Correct the UarteWithIdle timeout math

Fixes #753
Tested on a custom board with an nrf52840 communicating via uarte to a sim7000g using hardware flow control. I also bumped the idle time to 8 bytes of idle since 2 bytes of idle cut it a bit short in my experience. Especially given rounding errors due to integer math.

Perhaps it would be better to leave the un-simplified formula in the comments so that it can be reconstructed to check for accuracy? Let me know.

As discussed in Matrix, if you're using RTS/CTS you shouldn't lose data independently of the idle timeout. For users not using RTS/CTS, there's no way to choose a timeout value that will suit everyone. Due to this, I'd rather keep the shorter timeout of 2 bytes, as it'll lead to less latency.
The original maths are correct:

Formula from converting BAUDRATE reg value to baudrate hz:
Source: https://devzone.nordicsemi.com/f/nordic-q-a/391/uart-baudrate-register-values
baudrate_hz = baudrate_reg * 16_000_000 / 2^32
Example: 0x01D7E000 * 16_000_000 / 2^32 =<PHONE_NUMBER>21875 = ~115200

1 byte on the wire is 10 bits worth of time (1 start bit + 8 data bits + 1 stop bit).
2 bytes is therefore 20 bits.

That is, 20 baudrate_hz cycles.
That is, 20 / baudrate_hz seconds.

Formula for calculating CC register value from a timeout in seconds,
when the timer is running at 16Mhz:

cc = timeout_s * 16_000_000.

Putting this all together:

cc = timeout_s * 16_000_000.
cc = (20 / baudrate_hz) * 16_000_000.
cc = (20 / (baudrate_reg * 16_000_000 / 2^32)) * 16_000_000.
cc = 20 / baudrate_reg / 16_000_000 * 2^32 * 16_000_000.
cc = 20 * 2^32 / baudrate_reg.

Now, to make the math fit in a u32:

cc = 20 * 2^32 / baudrate_reg.
cc = 20 * 2 * 2^31 / baudrate_reg.
cc = 40 * 2^31 / baudrate_reg.
cc = 2^31 / (baudrate_reg / 40)

Dividing baudrate_reg / 40 is slightly less precise, but the result is rounded
down, so the end result in `cc` is rounded up. Waiting for slightly more than 2 bytes is fine.

Your formula is also correct. You're replacing 2^32 with 0xFFFF_FFFF which I guess is slightly more accurate than having to divide by 40 instead of 20.
 I think it's better to always round up, so the timeout is always "at least 2 bytes".
If you want, change the formula to the more accurate cc = 0xFFFF_FFFF / (baudrate_reg / 20), otherwise I think we can close this.

I'm going to go ahead and close this given the silence, feel free to open a new PR if you think the formula with slightly better precision is worth it.


Fix(Breadcrumbs): Optional action/href in the last breadcrumb

Action and href are not used in the last bit, so specifying action: () => {} is very strange
For example (or just make it optional):
type BreadcrumbItem = {
text: string
href: string
}
type LastBreadcrumbItemOptionalHref = {
text: string
href?: string
}
type BreadcrumbItems =
| BreadcrumbItem[]
| [...BreadcrumbItem[], LastBreadcrumbItemOptionalHref]
const invalidItems: BreadcrumbItems = [
{ text: "Home" },
{ text: "Contacts", href: "/contacts" },
]
const validItems0: BreadcrumbItems = [{ text: "Home" }]
const validItems1: BreadcrumbItems = [
{ text: "Home", href: "/" },
{ text: "Contacts" },
]
const validItems2: BreadcrumbItems = [
{ text: "Home", href: "/" },
{ text: "Contacts", href: "/contacts" },
{ text: "Foo", href: "/contacts/foo" },
{ text: "Emails", href: "/contacts/foo/emails" },
{ text: "Some Email" },
]

@lu-perfect We're planning to replace current Breadcrumbs with the new one in the next major. The new component doesn't have this issue.


Determine if #showtooltip works on custom action buttons

If it does, it simplifies mount icon/tooltip handling, for instance.

It does not. At least, not in my testing of creating an ActionButtonTemplate/SecureActionButtonTemplate and setting type=macro and macrotext=something with #showtooltip.


Readme change

@saching13 I am not sure whether the ros_ws specified folder is correct, but for beginners like me it's useful to really have step-by-step tutorial:)

Hhhmm. it was under the assumption that they would have gone through basic ros tutorials.
Anyways we can make it as dai_ws


Cancellation semantics for async() and coroutine()

What should the following code return?
$promise = async(static function (): int {
    try {
        await(React\Promise\Timer\sleep(2));
    } catch (Exception $e) {
        return 1;
    }

    return 2;
})();

$promise->cancel();

var_dump(await($promise));

At the moment, the code would print 1. The same cancellation logic is triggered for fibers (#20) and coroutines (#13). Accordingly, this affects Async v3 and Async v4.
Perhaps we should reject the entire promise? (See also https://github.com/reactphp/promise/issues/56 for discussion about cancellation semantics in Promise v3).

Cancellation semantics are tricky. @WyriHaximus has set up this poll on Twitter: https://twitter.com/WyriHaximus/status/1539183900668440576
In accordance with this, it's currently my understanding each of these examples should behave exactly the same and only cancel the first sleep(2) and in effect keep executing sleep(3):
$promise = async(static function (): int {
    try {
        await(React\Promise\Timer\sleep(2));
    } catch (Exception $e) {
        await(React\Promise\Timer\sleep(3));
    }
})();

$promise->cancel();

$promise = coroutine(static function (): Generator {
    try {
        yield React\Promise\Timer\sleep(2);
    } catch (Exception $e) {
        yield(React\Promise\Timer\sleep(3);
    }
});

$promise->cancel();

$promise = React\Promise\Timer\sleep(2)->catch(function (Exception $e) {
    return await(React\Promise\Timer\sleep(3));
});

$promise->cancel();

Arguably, this not how we would recommend handling promise cancellations and it's also not something we're aware of in the wild at the moment. Consumers are advised to handle only appropriate rejections and reconsider if more operations are to be performed if either operation is cancelled.
I'll file PRs to adjust coroutine() and async() behavior to match the above logic by cancelling only the first operation for the outstanding v3 and v4 releases and link this against this ticket. This way, everything is in line with how promise cancellation has always worked ever since its inception in 2014.


Location of files

I am attempting to install the requirements.
It did not find the install.
I am using Windows 10 Powershell. New Python install. Pip has been upgraded.
Any suggestions where the directory where the scraper was put?

I am attempting to install the requirements. It did not find the install. I am using Windows 10 Powershell. New Python install. Pip has been upgraded. Any suggestions where the directory where the scraper was put?

Try altering the command line to pip3 and see if that works.
If that fails, try installing the requirements out of the text file manually.
The directory of the Scraper will be wherever "main.py" is located at and ran from. You determine this.
If you're still having issues, please feel free to follow up.
Attach any errors given by Powershell if possible.


Make components in template optional

Make possible to upload template with no components in it.

FE application crashing when I trying to deploy application and choosing template without components:


Krish Chopra added to contributers.md

Pull Request Sent

Thnx for contributing @KrishChopra22  ..do star this repo


Update some libraries to the latest patches

Akka 2.4.8 has some Http/Stream Improvements
Curator 2.11 has some good fixes.
ScalaTest 3.0 is out

Only fixed deprecations so far. The release notes
look really interesting.

Thanks!


issue when migrating from 0.3.9 to 0.4.2

Our existing code base uses version 0.3.9, and I am trying to upgrade it to 0.4.2. However, in the generated pb.h file, under maximum encoded size of message, there is something new:
union ControlEnvelope_payload_size_union {char f9[(8 + sizeof(Response_payload_size_union))]; char f10[(12 + Dialog_size)]; char f11[(26 + SpeechSettings_size + SpeechInitiator_size + Dialog_size)]; char f12[(14 + Dialog_size)]; char f13[(12 + Dialog_size)]; char f25[(13 + DeviceConfiguration_size)]; char f101[(13 + State_size)]; char f102[(13 + State_size)]; char f0[38];};
The compiler reports Error 'Response_payload_size_union' undefined.
0.3.9 doesn't have anything under maximum encoded size, so do I need to manually create this union somewhere else or the nanopb supposes to auto-generate this?
Thanks,

Sounds like issue #569.
You can try passing all .proto files to the generator at the same time, to try to make sure it finds them and doesn't have to resort to the compile time calculation of message sizes that is causing the problem.

this particular proto file import 11 other proto files, so is it just the sequence of the files generation is out of order? so ControlEnvelope_payload_size_union is passed to generator before Response_payload is passed to generator?
Do I need to combine all the proto into one file or re-arrange my build script to make sure response.proto is built first before controlEnvelope.proto?

You could try something like  protoc --nanopb_out=. response.proto controlEnvelope.proto with all the proto files listed. The order does not matter, but having them in the same command (instead of 11 separate commands) ensures that the generator finds them.

Thank you, it works.


missing maven central dependencies in 1.72

also see #167
some jenkins packages aren't on maven central, so resolving from maven fails

[warn] io problem while parsing ivy file: https://repo1.maven.org/maven2/org/jenkins-ci/annotation-indexer/1.4/annotation-indexer-1.4.pom: Impossible to load parent for file:/Users/user/.ivy2/cache/org.jenkins-ci/annotation-indexer/ivy-1.4.xml.original. Parent=org.jenkins-ci#jenkins;1.26
[warn] 	module not found: org.jenkins-ci#annotation-indexer;1.4
[warn] ==== typesafe-ivy-releases: tried
[warn]   https://repo.typesafe.com/typesafe/ivy-releases/org.jenkins-ci/jenkins/1.26/ivys/ivy.xml
[warn] ==== sbt-plugin-releases: tried
[warn]   https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/org.jenkins-ci/jenkins/1.26/ivys/ivy.xml
[warn] ==== local: tried
[warn]   /Users/user/.ivy2/local/org.jenkins-ci/jenkins/1.26/ivys/ivy.xml
[warn] ==== public: tried
[warn]   https://repo1.maven.org/maven2/org/jenkins-ci/jenkins/1.26/jenkins-1.26.pom
[warn] ==== repo.jenkins-ci.org: tried
[info] Resolving org.fusesource.jansi#jansi;1.4 ...
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::          UNRESOLVED DEPENDENCIES         ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: org.jenkins-ci#annotation-indexer;1.4: not found
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn]
[warn] 	Note: Unresolved dependencies path:
[warn] 		org.jenkins-ci:annotation-indexer:1.4
[warn] 		  +- com.infradna.tool:bridge-method-annotation:1.14
[warn] 		  +- org.kohsuke:github-api:1.72
[warn] 		  +- default:scala-build:0.1-SNAPSHOT (scalaVersion=2.10, sbtVersion=0.13)

Solution: resolvers += "repo.jenkins-ci.org" at "http://repo.jenkins-ci.org/public"

Your library is great. I checked out all of githubs recommendation and this one wins by far. Nice API and relatively typesafe :+1: . You should document the resolver above, so people don't get turned off when they can't resolve their deps :).

Thanks for the feedback!
pom.xml explicitly points to the jenkinsci repo, so the current state looks good to me

Hi, I try to use this GitHub api today and have this same issue. How can I resolve it?

add repo.jenkins-ci.org


fixed math expressions

equations rendered incorrectly in Guides
What does this PR do?
fixes math expressions rendered incorrectly in jax for the impatient and flax basics

Hey @banda-larga, thanks for doing this! You are missing the pre-commit hooks to sync the markdown and jupyter notebooks  as detailed in the How to Contribute guide. Please run:
pip install pre-commit
pre-commit install
pre-commit run --all-files

@cgarciae yes, sorry, done it!

Codecov Report

Merging #2392 (3d77db1) into main (890a391) will increase coverage by 0.54%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2392      +/-   ##
==========================================
+ Coverage   69.33%   69.87%   +0.54%
==========================================
  Files          63       62       -1
  Lines        5556     5513      -43
==========================================
  Hits         3852     3852
+ Misses       1704     1661      -43

Impacted Files
Coverage Δ

flax/optim/dynamic_scale.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thanks @banda-larga! I was wondering if you could fix an additional one? After

The linear approximation of f at point $x$ reads:

in jax_for_the_impatient.md we are also missing a new line.


The list under "You should be aware of the following constraints when creating alternate keys" is not complete

I have noticed that for some tables, it is not possible to create alternate keys. But, it is not mentioned anywhere in the documentation. You need a list of standard tables which do not support alternate keys. This can be mentioned under "You should be aware of the following constraints when creating alternate keys" section.

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

ID: 7b70f58f-1e86-4148-7f89-7dc9d21ee337
Version Independent ID: 32cb118b-4632-0e1d-915a-7d3f530dd7ef
Content: Define alternate keys for an entity (Developer Guide for Dynamics 365 Customer Engagement (on-premises))
Content Source: ce/customerengagement/on-premises/developer/define-alternate-keys-entity.md
Product: d365ce-op
GitHub Login: @mayadumesh
Microsoft Alias: jdaly

Thanks for your comment, @rezanid. This issue has been assigned to the doc author for review.

@rezanid  What tables? I just added a constraint for virtual tables. Are there others?

@rezanid I will keep this issue open until I have an official answer from the product team, then update both the CE on-premise and Power Apps topics as needed.

One that I recently encountered was Annotations.


server: fix wrong log

What have you changed? (required)
Fix the wrong log message when an operator timeout.
What are the type of the changes (required)?

Bug fix (non-breaking change which fixes an issue)

How has this PR been tested (required)?
None
Does this PR affect documentation (docs/docs-cn) update? (optional)
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

/run-all-tests


git-secrets triggers on every commit

A lot of code in cromwell-tools triggers our default patterns.

Closing since this could be addressed by updating the git-secrets config.


snowflake_procedure is not idempotent

Provider Version
$ terraform version
Terraform v1.6.1
on darwin_arm64
+ provider registry.terraform.io/snowflake-labs/snowflake v0.73.0

Terraform Version
$ terraform version
Terraform v1.6.1
on darwin_arm64
+ provider registry.terraform.io/snowflake-labs/snowflake v0.73.0

Describe the bug
snowflake_procedure's are not idempotent.
Expected behavior
snowflake_procedure's are idempotent.
Code samples and commands

Create a procedure resource like this one and do terraform apply:

resource "snowflake_procedure" "dev2837" {
  name     = "DEV2837"
  language = "JAVASCRIPT"
  database = snowflake_database.default.name
  schema   = snowflake_schema.share_config[0].name
  arguments {
    name = "VIEWNAME"
    type = "VARCHAR"
  }
  return_type = "VARCHAR"
  execute_as  = "OWNER"
  statement   = <<EOT
    const dbName = '${snowflake_database.default.name}';
EOT
}

Go to the Snowflake console and drop the procedure, out-of-band from Terraform:

DROP PROCEDURE DEV2837(VARCHAR);

Run terraform apply again. Terraform should detect that the procedure no longer exists and it should offer to recreate the procedure for you. But instead, Terraform just errors out:

│ Error: 002003 (02000): SQL compilation error:
│ Procedure 'MY_DB.MY_SCHEMA.DEV2837' does not exist or not authorized.
│
│   with module.sandbox4_application.snowflake_procedure.dev2837,
│   on ../../modules/aws/application/snowflake.tf line 1411, in resource "snowflake_procedure" "dev2837":
│ 1411: resource "snowflake_procedure" "dev2837" {

Additional context
The behavior noticed here is essentially the same thing as described for snowflake_schema's here: https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/845, though I don't know if both are caused by the same code paths or not.

Hey @jrobison-sb.
In v0.100.0, we introduced new procedure resources. They have this behavior fixed. Please verify it and let us know.


Ability to have exclusive tags for just the eks cluster resouces

Is your request related to a new offering from AWS? No
Is your request related to a problem? No
Describe the solution you'd like.
As a terraform user, I'd like to give my eks cluster resource a set of tags that are exclusive to just the cluster resource
Describe alternatives you've considered.
I have forked this project and I created a new variable in variables.tf, specific for the cluster tags
Additional context
Tags as placed on all the resources, which is verbose, and chews into limits of tagging in AWS.  I'd like to the ability to put tags only on the EKS cluster itself.

Hi @antonbabenko , let me know if this is something that can be looked at for a future release.  Basically, I have a use case where I just need to be able to tag the EKS clusters with very specific tags, and the other resources that are created along the EKS cluster do not need that tag.

Hi @mmerrill3 !
@barryib is the lead maintainer of this module and he will reply when time permits.
PS: I guess it is a summer break for many of us who maintain terraform-aws-modules, so don't expect an immediate response.

Fixed in #1569
Released in v17.9.0.


Clarified homepage examples

This is a bit speculative, but I've noticed that people have a hard time parsing the second example on the homepage -- the result looks good, but it's hard to see how the bits add up to it.  I've reorganized the code slightly and changed the explanation to match; what do you think?

I can't see why the push build would have failed...

The edits look good. It's a pretty complex thing to convey especially because some of the things it's doing aren't particularly meaningful and serve just to demonstrate freedom to express complex relationships.

Push build failure was a transient. They're very frequent again since I reintroduced something I accidentally deleted before but I know where they're coming from now, the matplotlib tight bounding box code on the MPLRenderer seems to fail sometimes on travis.

It's a pretty complex thing to convey especially because some of the things it's doing aren't particularly meaningful

I had to be very careful in how I worded it, because I didn't want to try to explain that the dictionary of Layout objects becomes a HoloMap of Layout objects and then is collated into a Layout of HoloMap objects, which will just make peoples' heads explode.  So I focused on the fact that it was a collection of items at different y values, that we then get a slider for. :-)

Ready to merge?

Sure; I thought it had been merged ages ago. :-)


Investigate the usage of Nan values for cohorts definition

If the cohorts are of different sizes, the queue is full of Nan values.
In NetCDF format, they take up a lot of memory, but when compressed with Zarr, they take up almost none.

To keep everything in the same Dataset I use cohort as a coordinate. There will be some nan values when fgroup have different cohort sizes.


brakeman: 4.8.0 -> 5.0.0

Motivation for this change
https://github.com/presidentbeef/brakeman/releases/tag/v5.0.0
Things done

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

[x] NixOS
[x] macOS
[ ] other Linux distributions

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

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110989 run on x86_64-linux 1

1 package built:

brakeman

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110989 run on x86_64-linux 1

1 package built:

brakeman


two questions about the code

Hi, thanks for sharing the code. I have two questions about the code. Can you please clarify them? thanks.

In file scalable-birl/AVRIL/models.py, why line 91 is td = q_values_next_a - q_values_a instead of td = q_values_a - q_values_next_a according to Eq.11 in the paper?
line 100, log_sds = np.take_along_axis(r_par,(2+targets[:,0,:]).astype(int),axis=1).reshape((len(inputs),)), what does 2 mean here? Shoube it be a_dim?

Thank you for pointing these out - they are mistakes and have been fixed.


The navbar is getting crowded any ideas?

All the new pages getting added is making the navbar messy.
My suggestion is to go back to the 3 standard navbars (Home,Helpful-Material, Contribution) and move the other navbars/feature somewhere else? Any suggestions?

may i help you with create a few drop down menu from your navbar menu ?

@syauqiahmd  Go ahead :)

Making drop-downs with sections like Tech, Entertainment,News would help i think.And put new pages into their related section.

@satya-vinay That sounds like a good idea

We now have a drop down but only for index.html anyone care to fix it on all the other pages?

Taking this up

@HJain13 Great looking forward to it ;)

hi @HJain13, I already done that

Awesome 👍


AOI size error message

Feedback from user testing: Feedback on reducing size doesn’t go away after you fix it.

Fixed https://github.com/hotosm/field-campaigner/pull/123


Options -Wtrampolines and -fstack-clash-protection not universal

In the compiler option hardening materials, running the new "make" test on clang on MacOS causes failures with clang:
make
cc -O2 -Wall -Wformat=2 -Wconversion -Wtrampolines -Wimplicit-fallthrough  -D_FORTIFY_SOURCE=3  -D_GLIBCXX_ASSERTIONS  -fstack-clash-protection -fstack-protector-strong  -Wl,-z,nodlopen -Wl,-z,noexecstack  -Wl,-z,relro -Wl,-z,now  -fPIE -pie -fPIC -shared     demo.c   -o demo
clang: warning: argument unused during compilation: '-fstack-clash-protection' [-Wunused-command-line-argument]
clang: warning: argument unused during compilation: '-pie' [-Wunused-command-line-argument]
warning: unknown warning option '-Wtrampolines' [-Wunknown-warning-option]
1 warning generated.
ld: unknown option: -z

The -pie problem is known, and resolved by using -fpie. But -Wtrampolines and -fstack-clash-protection are not universal.

The -Wtrampolines issue is fixed by #358. The `-fstack-clash-protection' may just be a compiler version issue.

Maybe the -fstack-clash-protection error you're seeing is because the aarch64 support was lacking? That has been added only recently: https://github.com/llvm/llvm-project/pull/66524

@siddhesh - No, not in my specific case, since it's an Intel-based Mac:
clang --version
Apple clang version 14.0.3 (clang-14<IP_ADDRESS>.1)
Target: x86_64-apple-darwin22.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

That's odd, it's been in clang since 2020, so I'm guessing clang 11-ish. I wonder why Apple hasn't enabled it.

My guess is that Apple is not deploying the latest and greatest versions. That's really common, which is why we identify compiler versions for things.

Has this been addressed by the C/C++ Compiler Hardening options guide?    @gkunz @thomasnyman @david-a-wheeler

The placement of the options in the tl;dr section with respect to the main list and situational options has been addressed. Closing after review in Compiler BP Call 2024-06-27.


Not insert events if already inserted

Check if a event needs to be insertet. This is  currently a bug, because every event is after each run added again.

closed


Unexpected Dry::Schema::MissingMessageError

Describe the bug
Dry validation combined with dry-configurable:0.13.0 raises a Dry::Schema::MissingMessageError exception when the  validated attribute is an enum and the actual value isn't one of the listed enums.
This doesn't happen with versions < 0:.13, and although I can't be certain, it stands to reason that the issue is in this gem.
To Reproduce
# frozen_string_literal: true

gem 'dry-configurable', '< 0.13' # passes
# gem 'dry-configurable', '= 0.13.0' # fails

require 'dry-validation'
require 'rspec/autorun'

module Types
  include Dry::Types()
end

class Contract < Dry::Validation::Contract
  params do
    required(:arg).value(Types::Coercible::String.enum('foo', 'bar'))
  end
end

describe 'dry-validation' do
  let(:contract) { Contract.new }
  let(:validation) { contract.(arg: nil) } # or arg: 'xxx'
  subject(:error) { validation.errors.first }

  it 'reports validation errors as messages' do
    expect(error).not_to be_nil
    expect(error.path).to eq [:arg]
    expect(error.text).to eq 'must be one of: foo, bar'
  end
end

relevant error output
 Dry::Schema::MissingMessageError:
       Message template for :arg under "" was not found. Searched in:
       "en.dry_validation.errors.rules.arg.included_in?.arg."
       "en.dry_validation.errors.rules.arg.included_in?"
       "en.dry_validation.errors.included_in?.failure"
       "en.dry_validation.errors.included_in?.value.arg"
       "en.dry_validation.errors.included_in?.value.string.arg."
       "en.dry_validation.errors.included_in?.value.string"
       "en.dry_validation.errors.included_in?.arg."
       "en.dry_validation.errors.included_in?"

Expected behavior
No exception raised. See above.
My environment
Shouldn't be environment-specific, but anyhow:

Ruby version: 2.6.5

I didn't manage to reproduce it, as this passes:
# frozen_string_literal: true

require "bundler/inline"

gemfile do
  source "https://rubygems.org"

  gem "dry-configurable", "0.13.0"
  gem "dry-validation"
  gem "rspec"
end

module Types
  include Dry::Types()
end

class Contract < Dry::Validation::Contract
  params do
    required(:arg).value(Types::Coercible::String.enum("foo", "bar"))
  end
end

require "rspec"

RSpec.describe "dry-validation" do
  let(:contract) { Contract.new }
  let(:validation) { contract.(arg: nil) } # or arg: 'xxx'
  subject(:error) { validation.errors.first }

  it "reports validation errors as messages" do
    expect(error).not_to be_nil
    expect(error.path).to eq [:arg]
    expect(error.text).to eq "must be one of: foo, bar"
  end
end


how to expose myself to the internet world without localtunnel, ngcrok or any framework?

my home ip is dynamic

Do it the old fashioned way: forward a port from your router to your local computer and set up Dynamic DNS. A free dynamic dns provider is available at: https://freedns.afraid.org/

@mfresonke , why do you recommand this freeedns website? how often do you use it ? anything else?

You need a public IP for that @anlexN

Outside of the scope of this project.


TcpSink doesn't have a finalizer

We sometimes see in our production that we have a lot of TcpSocketWriters and small number of TcpSinks. VM can go out of sockets, effectively preventing our application from writing logs or make outbound connections. Seems thats somewhere we have a lot of TcpSinks born and die w/o Dispose.
We understand that's probably bug in our code, and we are working on it, but I think that TcpSink will benefit from having finalizer and implementing old-style dispose pattern, disposing TcpSocketWriter on finalization. What do you think? If yes, we can provide a PR.
P.S. @pauldambra I wonder if the project is maintained still or it's recommended  to move to other sinks? We are happy to contribute and fix bugs — but only there is project has active mantainer and there are other users :-)

More than happy to receive a PR!
I'm not working with .Net any more so I'm not actively maintaining this but have seen other folk are using it... the dotNet standard upgrade was entirely not my work for example
It's a while since I've thought about TCP in detail - do we need to worry about performance if we're actively disposing of Sinks? Don't want to stand the cost of initialisation if it's a problem.... Not a leading question just a thought. Maybe we need a test that starts and uses many sinks wrapped in a timeout so we can be sure that we're not adding startup cost? (Also happy to be told that's not how TCP works and to not worry about it :D)

Hi @leotsarev, @pauldambra,
I've prepared pull request which should solve this problem.
Could you review it?

very old PR with no tests


endpoint for marking user as removed

This can be done by sending a PATCH request to the user url with the set_active field set to false.
A separate endpoint seems unnecessary imo.


Files by Folder not working

When we try to add a new file to a folder, it is not listed for that folder. Instead, it is showing all files or recent files from all the folders.

Can you tell us what are the steps to reproduce the issue?
Many thanks

Not reproduced

Hi, I'm seeing the same issue. I have three folders, and they all show all the documents.


Let Agents explizite target workflows by labels

ontop of https://github.com/woodpecker-ci/woodpecker/pull/4141:
 ### `WOODPECKER_AGENT_LABELS`

 > Default: empty

 Configures custom labels for the agent, to let workflows filter by it.
 Use a list of key-value pairs like `key=value,second-key=*`. `*` can be used as a wildcard.
+If you use `!` as key prefix it is mandatory for the workflow to have that label set (without !) set and matched.
 By default, agents provide three additional labels `platform=os/arch`, `hostname=my-agent` and `repo=*` which can be overwritten if needed.
 To learn how labels work, check out the [pipeline syntax page](../20-usage/20-workflow-syntax.md#labels).

Originally posted by @6543 in https://github.com/woodpecker-ci/woodpecker/issues/3483#issuecomment-2371702315

Why not to implement it like in Kubernetes?
https://kubernetes.io/docs/tasks/configure-pod-container/assign-pods-nodes/
https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/


removed Neo environment from tutorials

Hi Daniel,
as you mentioned in the mail. I have changed the turoials that we are not covering the neo environment any more.

The PR shows the tutorial being completely changed, yet it looks barely changed. What did you change in the sac-embed-story tutorial?

Hi Daniel,
i have removed the chapters where Neo was used.
Best regards Benedikt
From: Daniel Wroblewski<EMAIL_ADDRESS>Reply-To: SAPDocuments/Tutorials<EMAIL_ADDRESS>Date: Thursday, 27. August 2020 at 12:14
To: SAPDocuments/Tutorials<EMAIL_ADDRESS>Cc: "Michligk, Benedikt"<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [SAPDocuments/Tutorials] removed Neo environment from tutorials (#5734)
The PR shows the tutorial being completely changed, yet it looks barely changed. What did you change in the sac-embed-story tutorial?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SAPDocuments/Tutorials/pull/5734#issuecomment-681857627, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGKWOOU6FAHFI2RMIEOKOO3SCYWUNANCNFSM4QJMUZ4Q.

I have pushed the changes myself -- your PR showed one of the tutorials being completely changed.https://github.com/SAPDocuments/Tutorials/commit/f86998bb163a1afe2294d320b803602b9579c77b


update git installation on centos 7 to git 2.x

motivation: centos 7 ships with git 1.x which is too old for the toolchain usage
chages:

use RHL software collections to install git 2.x
add symlink since collections install in /opt (TBD better solution)

@shahmishal @drexin thoughts?

It turns out that simple symlinking the executable won't work because git has runtime dependency on httpd24.
I managed to get it work by:
RUN echo "source /opt/rh/rh-git227/enable" >> /etc/profile.d/sh.local
ENV ENV=/etc/profile.d/sh.local

though which the environment variables can be correctly set for both bash and sh. @tomerd @shahmishal


fix: opts.method=getPanelCompletions

Replace custom implementation with copilot.api module.
Fix integration for opts.method = "getPanelCompletions"

but unless your experience is very different from mine

Nah, it's the same. It's a hit or miss, mostly miss. Most of the times I don't get anything at all, the time to receive suggestion is way too long.

panel completions should probably just be removed at least from the docs for now.

Yep, done.


mutableStateMapOf doc copy paste mistake

Hi team.
Found that in documentation for mutableStateMapOf returned object called like MutableSet<K,V> it seems like copy paste mistake, cause it returns SnapshotMutableMap.
/**
 * Create a instance of MutableSet<K, V> that is observable and can be snapshot.
 *
 * @sample androidx.compose.runtime.samples.stateMapSample
 *
 * @see mutableStateOf
 * @see mutableMapOf
 * @see MutableMap
 * @see Snapshot.takeSnapshot
 */
fun <K, V> mutableStateMapOf() = SnapshotStateMap<K, V>()

Please create an issue in Google issue tracker, common code issue.

https://android-review.googlesource.com/c/platform/frameworks/support/+/1611116

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


About "conv_in" layer in the U-Net

In the training of cyclegan-turbo, conv_in was set to be trainable, but it seems that the weights of conv_in after training were not saved. Instead, only the LoRA weights of conv_in were saved. Why is that?
Similarly, in pix2pix-turbo, conv_in was replaced with a TwinConv layer, and the paper does not seem to explain the function of this part.
Thanks

Hello, I have the same question. Did you solve it?


Python 3.8 as default on Windows [AUTOZIP]

Download:
Environment: QT=5.12, ARCH=32, CONFIGURATION=release, QTIFW=C:\Qt\Tools\QtInstallerFramework\3.1


add address to contact entity

Contact models needs to have address structure.

couple of hints:
Flask restplus doc
Flask restplus model api
Blog on how to build restplus

https://www.programiz.com/article/python-self-why
On Fri, Dec 21, 2018 at 10:57 AM Alex Joyeux<EMAIL_ADDRESS>wrote:

couple of hints:
Flask restplus doc https://flask-restplus.readthedocs.io/en/stable/
Flask restplus model api
https://flask-restplus.readthedocs.io/en/stable/api.html#models
Blog on how to build restplus
http://michal.karzynski.pl/blog/2016/06/19/building-beautiful-restful-apis-using-flask-swagger-ui-flask-restplus/
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/skalpel-tech/skpcrm/issues/4#issuecomment-449470373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWJtA91uElZ8KZUyVubb4691Q9DvwvLKks5u7S78gaJpZM4Zectx
.

ahah excellent

close in pr #8


Fix crash when g_expr is used in lazy variable

Fix #861

@diorcety there  are a few regression :-)

Obsoleted by #883


error: process ID list syntax error (master broken)

I just realized that the current master version is broken. Version 4.3.6 works fine. Any idea what goes wrong? It looks it's only a x11/UI problem, because this error does not occur with --nothing.
Here is the output. What is this kind of Terminallist output? I guess the main error is error: process ID list syntax error.
x11docker note: You are running a beta version of x11docker.
  Beta versions change often and may introduce temporary new bugs.
  If you prefer latest stable release, run 'x11docker --update'.
  If you prefer to follow development, run 'x11docker --update-master'.

  You can contribute to x11docker if you find bugs and report them at:
    https://github.com/mviereck/x11docker

x11docker note: To allow GPU acceleration (option --gpu) with --hostdisplay,
  x11docker will share host resources with insecure option --hostipc
  and allow trusted cookies with option --trusted.

x11docker WARNING: To allow clipboard sharing with option --hostdisplay,
  trusted cookies and insecure option --hostipc will be enabled.
  No protection against X security leaks is left!
  Please consider to use another X server option.

x11docker WARNING: Option --hostdisplay with trusted cookies provides
      QUITE BAD CONTAINER ISOLATION !
  Keylogging and controlling host applications is possible!

x11docker WARNING: Security risk:
  Option --hostipc causes severe reduction of container isolation!
  Drawback: IPC namespace remapping is disabled.
  Advantage: X extension MIT-SHM is possible.

x11docker WARNING: Option --cap-default disables security hardening
  for containers. Granting docker's default capabilities is considered insecure.

x11docker WARNING: Option --sys-admin may be dangerous.
  It adds insecure capability SYS_ADMIN to container.
  It is needed to run debian 9 images with option --systemd.
  Debian 10 images run well without --sys-admin.

x11docker WARNING: Option --gpu degrades container isolation.
  Container gains access to GPU hardware.
  This allows reading host window content (palinopsia leak)
  and GPU rootkits (compare proof of concept: jellyfish).

realpath: '': No such file or directory
x11docker WARNING: You are using proprietary closed source nvidia driver.
  GPU acceleration will only work if you have installed the very same driver
  version in image. That makes images less portable.
  It is recommended to use free open source nouveau driver on host instead.
  Ask NVIDIA corporation to at least publish their closed source API,
  or even better to actively support open source drivers like nouveau.

x11docker note: x11docker can try to automatically install nvidia driver
  version 396.24 in container on every container startup.
  Drawbacks: Container startup is slower and its security will be reduced.

  You can look here for a driver installer:
    https://www.nvidia.com/Download/index.aspx
    https://http.download.nvidia.com/
  A direct download URL may be:
   https://http.download.nvidia.com/XFree86/Linux-x86_64/396.24/NVIDIA-Linux-x86_64-396.24.run
  If you got a driver, store it at one of the following locations:
    /home/skeil/.local/share/x11docker/
    /usr/local/share/x11docker/

  Be aware that the version number must match exactly the version on host.
  The file name must begin with 'NVIDIA', contain the version number 396.24
  and end with suffix '.run'.

  Automated installation fails on image systems not using glibc like Alpine
  and fails on openSUSE images with a self-extraction error.
  These issues cannot be fixed due to closed source policy of NVIDIA corporation.

  To avoid all this, use free nouveau driver on host
  instead of proprietary closed source nvidia driver.

x11docker WARNING: Option --pulseaudio allows container applications
  to catch your audio output and microphone input.

+ Terminallist='xterm lxterm lxterminal stterm sakura termit pterm terminator terminology Eterm konsole qterminal gnome-terminal mate-terminal mrxvt rxvt xvt kterm mlterm xfce4-terminal NOLUCK'
+ '[' -z :1:1 ']'
+ for Pullterminal in $Terminallist
+ command -v xterm
+ for Pullterminal in $Terminallist
+ command -v lxterm
+ for Pullterminal in $Terminallist
+ command -v lxterminal
+ for Pullterminal in $Terminallist
+ command -v stterm
+ for Pullterminal in $Terminallist
+ command -v sakura
+ for Pullterminal in $Terminallist
+ command -v termit
+ for Pullterminal in $Terminallist
+ command -v pterm
+ for Pullterminal in $Terminallist
+ command -v terminator
+ for Pullterminal in $Terminallist
+ command -v terminology
+ for Pullterminal in $Terminallist
+ command -v Eterm
+ for Pullterminal in $Terminallist
+ command -v konsole
+ for Pullterminal in $Terminallist
+ command -v qterminal
+ for Pullterminal in $Terminallist
+ command -v gnome-terminal
+ break
+ '[' gnome-terminal = NOLUCK ']'
+ '[' -z :1 ']'
+ case $Pullterminal in
+ Pullterminal='dbus-launch gnome-terminal -x'
+ '[' -z 'bash -c' ']'
+ Anyterminal='dbus-launch gnome-terminal -x'
+ '[' -z :1:1 ']'
+ set +x
error: process ID list syntax error

Usage:
 ps [options]

 Try 'ps --help <simple|list|output|threads|misc|all>'
  or 'ps --help <s|l|o|t|m|a>'
 for additional help text.

For more details see ps(1).
error: process ID list syntax error

Usage:
 ps [options]

 Try 'ps --help <simple|list|output|threads|misc|all>'
  or 'ps --help <s|l|o|t|m|a>'
 for additional help text.

For more details see ps(1).
[skeil@desktop1950x ~]$ /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 172: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/share/container.CMD.sh: No such file or directory
/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 218: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/share/container.CMD.sh: No such file or directory
/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 220: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/share/x11docker.log: No such file or directory
/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 222: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/container.log: No such file or directory
/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 64: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/share/message.fifo: No such file or directory
tail: cannot open '/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/container.log' for reading: No such file or directory
/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/dockerrc: line 70: /home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/share/message.fifo: No such file or directory
touch: cannot touch '/home/skeil/.cache/x11docker/X50-sandrokeil-arch-chromium-nvidia/xtermready': No such file or directory

The terminal list output is created by a set -x that I inserted for debugging and accidently uploaded to master.  It is already removed in latest master.
The ps error is something different, not sure now. Can you please update and try again?

Alright, just updated to latest master. The Terminallist is gone, but I still have the process ID error. Just quick lookup, maybe it's something like this?

It occurs since this commit https://github.com/mviereck/x11docker/commit/02657d8851253c29276d75a26fb51f3b75d67e28#diff-308fa5eac58a6b2ba23e0fbc6e033041 Previous revisions worked fine.

Can you run with option --debug and show me the output, please? It shows at lot of background process handling and gives good chances to find the false ps command. I assume it happens in finish().
Currently I don't have this issue, maybe it depends on special options.

It's Command at Line 5702 returned with error code 1: awk '{print $1}'
x11docker note: You are running a beta version of x11docker.
  Beta versions change often and may introduce temporary new bugs.
  If you prefer latest stable release, run 'x11docker --update'.
  If you prefer to follow development, run 'x11docker --update-master'.

  You can contribute to x11docker if you find bugs and report them at:
    https://github.com/mviereck/x11docker

DEBUGNOTE:(20:28:01) Command at Line 1878 returned with error code 1:
  tr -cd '[:alpha:][:digit:][:blank:]-_.'
  0 - ::main
DEBUGNOTE:(20:28:01) Command at Line 1878 returned with error code 1:
  Imagecommandbasename=$(basename $(echo $Imagecommand | cut -d' ' -f1) 2>/dev/null | tr -cd '[:alpha:][:digit:][:blank:]-_.')
  0 - ::main
DEBUGNOTE:(20:28:01) Dependency check for --hostdisplay: 0
DEBUGNOTE:(20:28:01) Dependency check for --hostdisplay: 0
DEBUGNOTE:(20:28:01) Using X server option --hostdisplay
x11docker WARNING: To allow clipboard sharing with option --hostdisplay,
  trusted cookies and insecure option --hostipc will be enabled.
  No protection against X security leaks is left!
  Please consider to use another X server option.

x11docker WARNING: Option --hostdisplay with trusted cookies provides
      QUITE BAD CONTAINER ISOLATION !
  Keylogging and controlling host applications is possible!

x11docker WARNING: Security risk:
  Option --hostipc causes severe reduction of container isolation!
  Drawback: IPC namespace remapping is disabled.
  Advantage: X extension MIT-SHM is possible.

DEBUGNOTE:(20:28:01) Running xinitrc
DEBUGNOTE:(20:28:01) --hostdisplay is ready
DEBUGNOTE:(20:28:01) Command at Line 5702 returned with error code 1:
  awk '{print $1}'
  0 - ::main
DEBUGNOTE:(20:28:01) Command at Line 5702 returned with error code 1:
  Xinitpid="$(pgrep -a xinit | grep "xinit $Xinitrc" | awk '{print $1}')"
  0 - ::main
error: process ID list syntax error

Usage:
 ps [options]

 Try 'ps --help <simple|list|output|threads|misc|all>'
  or 'ps --help <s|l|o|t|m|a>'
 for additional help text.

For more details see ps(1).
DEBUGNOTE:(20:28:02) Stored background pid PID of 1
DEBUGNOTE:(20:28:02) Watching
DEBUGNOTE:(20:28:02) watchpidlist: PID PID has terminated
DEBUGNOTE:(20:28:02) time to say goodbye (watchpidlist PID)
DEBUGNOTE:(20:28:02) time to say goodbye (watchpidlist)
DEBUGNOTE:(20:28:02) time to say goodbye (xinit)
DEBUGNOTE:(20:28:02) Terminating x11docker.
DEBUGNOTE:(20:28:02) List of stored background processes:
8466 watchpidlist
8472 watchmessagefifo
8528 containershell
PID 1
error: process ID list syntax error

Usage:
 ps [options]

 Try 'ps --help <simple|list|output|threads|misc|all>'
  or 'ps --help <s|l|o|t|m|a>'
 for additional help text.

For more details see ps(1).
DEBUGNOTE:(20:28:02) Running dockerrc
DEBUGNOTE:(20:28:02) Checking: PID (1):
DEBUGNOTE:(20:28:02) Checking: 8528 (containershell):   8528 pts/0    00:00:00 x11docker
DEBUGNOTE:(20:28:02) Terminating 8528 (containershell) x11docker:   8528 pts/0    00:00:00 x11docker
DEBUGNOTE:(20:28:02) Checking: 8472 (watchmessagefifo):
DEBUGNOTE:(20:28:02) Checking: 8466 (watchpidlist):
DEBUGNOTE:(20:28:02) time to say goodbye (finish)
DEBUGNOTE:(20:28:02) Exitcode 0

Thank you for investigating!
The bug itself has been two lines below and only appeared with --hostdisplay:
      echo $Xcommand | grep -q Xorgwrapper && Line="Xorg $Newdisplay" || Line="$Xcommand"
      Xserverpid=$(ps aux | grep "$(echo "$Line" | cut -d' ' -f1-2)" | grep -v grep | grep -v xinit | awk '{print $2}')
      [ "$Xserverpid" ] && setonwatchpidlist $Xserverpid && storepid $Xserverpid Xserver

With --hostdisplay variable $Line is empty. ps aux | grep "$(echo "$Line" [...] gives full output of ps aux and leads coincidentally in parsing to PID 1. Thanks god that x11docker failed to kill PID 1 accidentally. :-)
The fix is now not to parse for X server pid with --hostdisplay.


Can't connect with Azure AD account on 2nd computer

Issue Type: Bug
issue Type: Bug

Download Azure Data Studio, connect with Azure AD account (Mac).
Take a 2nd computer (both are Mac in my case).
Sign in with the same Azure AD account and try connecting. Login failed for user ‘<token-identified principal>’.

https://www.loom.com/share/09ce8e33d6724588b2a8c1f778bd1d92
Azure Data Studio version: azuredatastudio 1.39.1 (7553f799e175f471b7590302dd65c997b838b29b, 2022-08-30T01:12:48.559Z)
OS version: Darwin x64 21.3.0
Restricted Mode: No

System Info

Item
Value

CPUs
Apple M1 Pro (10 x 24)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
5, 5, 4

Memory (System)
16.00GB (0.02GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Hi @Lukydemboy
The computer that you logged in to, did you connect to a particular database or master? If it was a database, can you provide the same database name by typing it in the Database input box and try again? Otherwise, can you confirm you have access to master database from your first computer too?

Hi @cheenamalhotra
Tried connecting to both, the master and a particular database, both throw the same error. I can confirm that I can still connect to the database on the first computer (using the same procedure as shown in the Loom video).

Before gathering logs, please:

Clean token cache by running: Command Palette > Azure Accounts: Clear Azure Account Token Cache.
Remove all accounts by running: Command Palette > Clear all saved accounts.

Thanks!

Hi @Lukydemboy
Following up, do you have Logs we can look into, or you were able to resolve issue?

Closing as inactive


Mark previewable files in File Explorer

when we can preview kustomization/helm-values files directly from the file explorer (see #937) we could perhaps mark these files somehow - so they are easy to identify?

Let's simply add the Preview action before the three dot menu.


[Bug] Error loading plugin SQLTools Teradata Driver Plugin

Describe the bug
A clear and concise description of what the bug is.
After installing Teradata SQLTools extension, SQLTools will not properly load
To Reproduce
Steps to reproduce the behavior:

Open VisualStudio Code
Install SQLTools
Install SQLTools Teradata Driver
Open SQLTools to configure Teradata

Expected behavior
Should allow you to configure Teradata
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

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

Smartphone (please complete the following information):
Visual Studio Code version:
Version: 1.89.1 (user setup)
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
Date: 2024-05-07T05:13:33.891Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Windows_NT x64 10.0.22631
Additional context
Full error information provided by VS Code included below.

[1715697882741] INFO  (ext): SQLTools is starting
[1715697882743] INFO  (ext): initializing language client...
[1715697882744] INFO  (ext): Registering client for languages [{"scheme":"sqltools"},{"language":"sql","scheme":"untitled"},{"language":"sql","scheme":"file"},{"language":"sql","scheme":"sqltools"}]
    ns: "lc"
[1715697882750] INFO  (ext): loading plugins...
[1715697882750] INFO  (ext): registering Formatter Plugin.
    plugin: {
      "name": "Formatter Plugin"
    }
[1715697882751] INFO  (ext): registering Connection Manager Plugin.
    plugin: {
      "name": "Connection Manager Plugin"
    }
[1715697882751] INFO  (ext): registering CodeLens Plugin.
    plugin: {
      "name": "CodeLens Plugin",
      "registeredLanguages": []
    }
[1715697882756] INFO  (ext): registering Dependency Manager Plugin.
    plugin: {
      "name": "Dependency Manager Plugin",
      "installingDrivers": []
    }
[1715697882759] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1715697882759] INFO  (ext): registering History Manager Plugin.
    plugin: {
      "name": "History Manager Plugin"
    }
[1715697882760] INFO  (ext): registering Bookmarks Manager Plugin.
    plugin: {
      "name": "Bookmarks Manager Plugin"
    }
[1715697882761] INFO  (ext): registering Authentication Provider Plugin.
    plugin: {
      "name": "Authentication Provider Plugin",
      "isRegistered": false
    }
[1715697882761] INFO  (ext): registering Object Drop-in Provider (plugin).
    plugin: {
      "type": "plugin",
      "name": "Object Drop-in Provider",
      "isRegistered": false
    }
[1715697882762] INFO  (ext): SQLTools activation completed. 20.9901 ms
[1715697882762] INFO  (ext): Formatter Plugin registered!
[1715697882762] INFO  (ext): CodeLens Plugin registered!
[1715697882762] INFO  (ext): Dependency Manager Plugin registered!
[1715697882762] INFO  (ext): Connection Manager Plugin registered!
[1715697882762] INFO  (ext): History Manager Plugin registered!
[1715697882762] INFO  (ext): Bookmarks Manager Plugin registered!
[1715697882762] INFO  (ext): Authentication Provider Plugin registered!
[1715697882762] INFO  (ext): Object Drop-in Provider (plugin) registered!
[1715697882954] INFO  (ls): SQLTools Server started!
===============================
Using node runtime?: no
ExecPath: C:\Users\josep\AppData\Local\Programs\Microsoft VS Code\Code.exe
===============================
[1715697882961] INFO  (ext): LanguageClient ready
    ns: "lc"
[1715697882959] INFO  (ls): Initialized with node version:v18.18.2
[1715697883774] INFO  (ext): EXECUTING COMMAND => sqltools.refreshTree
[1715697883774] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1715697884765] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1715697900128] INFO  (ext): registering SQLTools Teradata Driver Plugin (driver).
    plugin: {
      "extensionId": "ScriptPup.sqltools-teradata-driver",
      "name": "SQLTools Teradata Driver Plugin",
      "type": "driver"
    }
[1715697900128] INFO  (ls): REQUEST RECEIVED => ls/RegisterPlugin
[1715697900128] INFO  (ls): request to register plugin: "c:\Users\username\.vscode\extensions\scriptpup.sqltools-teradata-driver-0.4.0-win32-x64\out\ls\plugin.js"
[1715697900306] ERROR (ls): Error registering plugin: {}
    error: {}
[1715697900307] ERROR (ext): ERROR: Error loading plugin SQLTools Teradata Driver Plugin Request ls/RegisterPlugin failed with message: Error in native callback, {"code":-32603}
    ns: "error-handler"
[1715697901132] INFO  (ext): EXECUTING COMMAND => sqltools.refreshTree
[1715697901132] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1715697904450] INFO  (ext): EXECUTING COMMAND => sqltools.showOutputChannel
[1715697962273] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1715697962274] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1715697963737] INFO  (ext): EXECUTING COMMAND => sqltools.openAddConnectionScreen
[1715697974399] INFO  (ext): registering SQLite Plugin (driver).
    plugin: {
      "extensionId": "mtxr.sqltools-driver-sqlite",
      "name": "SQLite Plugin",
      "type": "driver"
    }
[1715697974399] INFO  (ls): REQUEST RECEIVED => ls/RegisterPlugin
[1715697974400] INFO  (ls): request to register plugin: "c:\Users\username\.vscode\extensions\mtxr.sqltools-driver-sqlite-0.5.1\out\ls\plugin.js"
[1715697974431] INFO  (ls): Driver sqlite registered!
    place: "driver-map"
    ns: "ls-context"
[1715697974432] INFO  (ext): SQLite Plugin (driver) registered!
[1715697975434] INFO  (ext): EXECUTING COMMAND => sqltools.refreshTree
[1715697975434] INFO  (ext): Connection explorer changed. Will be updated.
    ns: "conn-man:explorer"
[1715697975638] INFO  (ext): EXECUTING COMMAND => sqltools.getConnections
[1715697975638] INFO  (ls): REQUEST RECEIVED => connection/GetConnectionsRequest
[1715697976194] INFO  (ext): getting extension mtxr.sqltools-driver-sqlite information.
    ns: "ext-util"
[1715697976194] INFO  (ext): loaded extension information mtxr.sqltools-driver-sqlite for driver SQLite.
    ns: "ext-util"
[1715697977532] INFO  (ext): EXECUTING COMMAND => sqltools.openAddConnectionScreen

This issue can be circumvented by setting sqltools.useNodeRuntime property to true. Basically, it tells SQL Tools to use an external nodejs instance to load the extension. As a result,  the extension doesn't the same memory space as Electron and avoids native errors in memory management.
I've created a PR that adds info about the workaround to the README: https://github.com/ScriptPup/sqltools-teradata-driver/pull/29

Thanks @adamtwo! @jsauerland, if you continue to receive errors after setting the sqltools.useNodeRuntime setting, please let us know.

Hi @ScriptPup, @adamtwo. Thanks for your response. Unfortunately, I couldn't get it to work even after trying the above solution.
When I set the usenoderunetime to true, I received this error:
PS C:\Users\username> node -e "require('fs').writeFileSync('C:\\Users\\username\\AppDatacode-sqltools\\Data\\.node-runtime', process.execPath)" ; if ($?) { exit 0 }
node : The term 'node' is not recognized as the name of a cmdlet, function,
script file, or operable program. Check the spelling of the name, or if a path
was included, verify that the path is correct and try again.
At line:1 char:1

node -e "require('fs').writeFileSync('C:\\Users\\username\\App ...

  + CategoryInfo          : ObjectNotFound: (node:String) [], CommandNotFoun
 dException
  + FullyQualifiedErrorId : CommandNotFoundException

Can you clarify what could cause this?

@jsauerland I'm guessing you don't yet have Node.js installed. Please see the Prerequisites section at https://vscode-sqltools.mteixeira.dev/en/drivers/sq-lite for guidance.

@gjsjohnmurray this seems a bit more involved than I at first thought it would be.... It seems like there's a pretty extensive setup process that a windows business laptop may not be able to achieve due to security restrictions?


Bot only notify Scene switch to Live (Brb,Offline.. no)

If the Scene switch to BRB or Offline it doesn't send message to the chat, only in Live.
And it doesn't write actual briltrate in the message (if it is possible)
sorry for my bad english

Hi @urkoperez16 ,
Are you testing the switcher by streaming to your ingest from OBS?
NOALBS will not send the automatic switch messages when you're not streaming from OBS. It will also only show the bitrate in the message when switching from live to low or the opposite of that.

Oh, thank you very much, I've tried streaming and everything is fine, one thing, in areas with poor coverage, the audio starts to get out of sync, it gets choppy or makes small ‘explosions’, is there any way to fix that? Is it a problem of the server, is it a problem of the player? I've tried !fix or !refresh and they don't always work.

Are you using the VLC Media Source or the OBS Media Source in OBS Studio?

Are you using the VLC Media Source or the OBS Media Source in OBS Studio?

I use OBS Media Source

if you're using the OBS Media Source then it's just chalked up to the connection, and there isn't much you can do but find better signal, I would highly suggest looking into SRT/SRTLA with SRT you can use the HEVC codec which requires less bitrate to produce a watchable image, and if you move into SRTLA you can bond multiple connections.  IRL Pro for Android and Moblin for iOS both support SRTLA/Bonding.


It seem to have a python version require.

My python version is python 3.8. It cause the problem that “TypeError: unsupported operand type(s) for |: 'type' and 'NoneType' “

Hi!
Yes, currently the package supports only python >= 3.10...but, since the package I'm using to run duckduckgo is compatible with python 3.8, I'll make some changes and fix this. Thank you!


Add support for manifest various files.

Fixes #834
Changes proposed:

[x] Add

Things I've done:

[x] My pull request fixes an issue, I referenced the issue.

@robertohuertasm It's intended. Personally, I'm not happy with the icons either but...
At first, I followed the art guidelines and placed the overlay icon at the bottom right. I wasn't happy with the result on all three icons.
I'll try again to see if I can come up with better ones.

@robertohuertasm How about those ones?


r0 calculation is different for BeamformerTime classes

BeamformerTime, BeamformerTimeSq: r0 = self._steer_obj.r0
BeamformerTimeTraj, BeamformerTimeSqTraj: r0 = self.steer.env._r(tpos)
when the .ref argument of class SteeringVector is changed, it does not effect BeamformerTimeTraj and BeamformerTimeSqTraj. This can be confusing and should be fixed in the future.

closed via commit e49c9587e0e143af40087c435a16421bb1d3bf90
By Gert Herold on 2020-02-27T11:10:56 (imported from GitLab)


Adding unit & map annotation sample links

See https://trello.com/c/lIBeSsQy/93-new-2015-01-samples
OME-XML samples for map annotations and units were only recently added for the 2015/01 schema. This PR adds specific links indicating they are now available.

Will be staged at https://www.openmicroscopy.org/site/support/ome-model-staging/ once built

One formatting comment. Overall reads fine. Given the specificity of the sample links in this context, is it worth linking to the individual files directly rather than the samples directory?

I did wonder about just linking to the files. I'll do that and fix the formatting now.

Should be sorted now @sbesson

Live docs updated.


CORS config parameter parsing

Hi @AnandChowdhary
It took me a while to get the app set up locally after the recent updates.
My .env config:
# Remove CORS headers without API key
DISALLOW_OPEN_CORS = false

Upon close inspection, it looks like the value false was coming in as a string so it failed the condition here: https://github.com/staart/packages/blob/master/packages/server/index.ts#L52

Also a side issue during compilation:
☐  pending   Compiling TypeScript
.staart/src/__staart.ts(2,34): error TS7016: Could not find a declaration file for module 'regenerator-runtime'. '/Users/smoujami/projects/dukketta/staart/migration/api/.staart/node_modules/regenerator-runtime/runtime.js' implicitly has an 'any' type.
  Try `npm install @types/regenerator-runtime` if it exists or add a new declaration (.d.ts) file containing `declare module 'regenerator-runtime';`
✔  success   Listening on 8080

Hi @simoami, thanks for opening the issue! If this is the case, I should handle other boolean environment variables too, like DISABLE_HELMET below it (thought they work right now because !"true" is false, but !"false" is not true), I'll fix it to be more precise.
The TypeScript warning is not an error, so it should compile, but that's something already on my todo and I'm going to fix it ASAP.

For now, if you remove the environment variable, it should work, because !undefined and !"" are both true, so CORS will be injected. This can be a temporary fix until I deploy an update to @staart/server.

I've fixed this, but I'll also check the codebase for other environment variables booleans.

I'll open another issue for the TypeScript warning so we can check on it

Thank you! I should point out that this was working before. Perhaps dotenv
isn't parsing properly?
On Mon, Mar 16, 2020, 2:15 PM Anand Chowdhary<EMAIL_ADDRESS>wrote:

I've fixed this, but I'll also check the codebase for other environment
variables booleans.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/staart/api/issues/1046#issuecomment-599688378, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRT6DQQPIACGGENB7R4XTRHZUEVANCNFSM4LMP5JNQ
.

Maybe something to do with the fact that it was in @staart/api before, and is now in a separate @staart/server package and the .env was directly used with process.env.VAR instead of from the config.ts file which had smart fallbacks.
If anything similar doesn't work again, I'll reopen this issue, but we should be good to go for now.

Ah got it. That must be it then. Thanks for fixing


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.9.0 -> 1.15.1

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.12.1 -> 1.15.1

terraform-ibm-modules/key-protect-all-inclusive/ibm (source)
module
minor
4.2.0 -> 4.4.0

terraform-ibm-modules/observability-instances/ibm (source)
module
minor
2.9.2 -> 2.10.0

terraform-ibm-modules/resource-group/ibm (source)
module
minor
1.0.6 -> 1.1.0

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.15.1
Compare Source
Bug Fixes

updated usage section in readme for all the modules (#321) (89e5cfe)

v1.15.0
Compare Source
Features

added pre-wired rule for IS (VPC infra) -> COS in fscloud submodule (#302) (2e8521d)

v1.14.0
Compare Source
Features

added pre-wired rule for IKS -> IS (VPC infrastructure) in fscloud submodule (#306) (4026d8c)

v1.13.0
Compare Source
Features

update terraform version constraints to >= 1.3.0, <1.6.0 (#319) (57f862a)

v1.12.2
Compare Source
Bug Fixes

added extra validation to the existing_serviceref_zone and existing_cbr_zone_vpcs variables in the fscloud profile submodule (#303) (f24ab4a)

v1.12.1
Compare Source
Bug Fixes

added a default value for the operations variable to make it backward compatible with recent CBR service updates around api types (#305) (2f89973)

v1.12.0
Compare Source
Features

added pre-wired rule for icd-databases in fscloud submodule (#294) (2d8d25b)

v1.11.0
Compare Source
Features

add activity tracker and logdna to list of custom rules (fscloud submodule)  (#297) (3eb5aa7)

v1.10.0
Compare Source
Features

add fscloud added pre-wired rule for AT -> COS (#296) (0e10af6)

v1.9.2
Compare Source
Bug Fixes

endpoint set to direct for VPC to COS in the fscloud submodule (#283) (4467165)

v1.9.1
Compare Source
Bug Fixes

update module references in README (#284) (321229b)

terraform-ibm-modules/terraform-ibm-key-protect-all-inclusive (terraform-ibm-modules/key-protect-all-inclusive/ibm)
v4.4.0
Compare Source
Features

update terraform version constraints to >= 1.0.0, <1.6.0 (#360) (426cc7a)

v4.3.0
Compare Source
Features

added new variable force_delete_key_ring which defaults to true (#359) (16b4144)

terraform-ibm-modules/terraform-ibm-observability-instances (terraform-ibm-modules/observability-instances/ibm)
v2.10.0
Compare Source
Features

updated the required terraform version range to >= 1.0.0, < 1.6.0 (#368) (0e33928)

terraform-ibm-modules/terraform-ibm-resource-group (terraform-ibm-modules/resource-group/ibm)
v1.1.0
Compare Source
Features

update terraform version constraints to >= 1.0.0, <1.6.0 (#508) (0a10be9)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

The bump to key_protect_all_inclusive changes the default behaviour of key_protect_key_rings force delete from false (not set) to true to make sure that the key ring is deleted even if it contains a key (or key in process of being deleted).
This is entirely contained within the module and is desired behaviour.
This causes the upgrade to test to fail with
Resource(s) identified to be updated Name: key_ring Address: module.key_protect_all_inclusive.module.key_protect_key_rings["cos-key-ring"].ibm_kms_key_rings.key_ring Actions: [update]
DIFF:
Before: {"force_delete":false}
After: {"force_delete":true}

This is the only difference detected, and since it is desired, I propose skipping the upgrade test on this PR.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 7.0.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add table lock to prevent race conditions

This is probably not ideal for the core library, but I wanted to open this PR for continued discussion of https://github.com/djreimer/sequenced/issues/12. I first tried to wrap #set_sequential_id in an advisory lock (using https://github.com/mceachen/with_advisory_lock), but there was a less-frequent race condition where a different row would be created in between #set_sequential_id and #save being called.

@ajb thanks for submitting this, I'll take a look

FWIW, we've been using this in production for a while now.

This is how its done - locking records.
https://coderwall.com/p/h0qura/scope-sequences-in-activerecord
http://stackoverflow.com/questions/17024447/add-auto-increment-with-scope-to-existing-column-in-migration-file-rails
The first link is based on this gem.
I think it should be noted that this gem is not thread safe in the readme, since without thread safeti its pretty much useless for anyone.

Thanks for the review! Closing in favor of #16, which looks :star2:


carry out removals of previously deprecated functions/kwargs

Carry out a scheduled removal of the inverse and compute_hessian_eigenvalues functions. These were not part of the top level submodule APIs, but not technically private previously so were on a deprecation cycle before their eventual removal.
A couple other minor fixes were made:

deprecated/removed versions were wrong in unsupervised_wiener
updated a docstring for Gaussian which indicated a kwarg default change would have already taken place. I think we can just make the actual change in the next release rather than this one. (It was not yet changed in the latest scikit-image release, but has been changed on scikit-image main)

/merge


Resolve a pair of session fixation bugs around multi-factor auth

Sessions should be reset between authentication levels, and a user should have a limit on how long to input their multi-factor authentication.

Coverage increased (+0.08%) to 87.297% when pulling 0b96e3f5ef3df38750bd93efc590197206e2434c on bug/security/session-fixation-in-mfa into 4cc9b2bc714622d28e465a1a2222b5b7f76726f0 on main.


Dismissing an ViewController runs the same animation as the one used for Presenting it.

When you present an Modal or push a ViewController with an animation, you would expect the reversed of the animation would play when you or dismissing the model or popping the view. But this is not the case.
Not sure if this was by design, but it would be great to have a reverseMode where you can tell Hero to do the reversed animation when dismissing or popping the view.

+1

@sam899 I created a pull request that creates this feature. You can check at the code I wrote to include this feature if yo like.

closing out old issues, feel free to reopen


navigator empty after data changed

Hello,
I encounter a problem using highstock.js in my angularjs app
Expected behaviour
Nagivator update with new data, like highchart plot.
Actual behaviour
When i update series data, navigator becames empty.
Live demo with steps to reproduce
In my html view
<highchart config="plot_cfg"></highchart>
In my controller:
function loadData(){

var res=requestService.getSeries(id).query().$promise;
        $scope.plot_cfg.loading = true;
        res.then(function(d){
            var res= [];
            $scope.plot_cfg.title.text = configService.record.timeinterval;
            $scope.plot_cfg.loading = false;

            $scope.plot_cfg.series=res;
            //$scope.rplot_cfg.series.showInNavigator=true;
            //reset zoom scale
            $scope.plot_cfg.getHighcharts().redraw();
            $scope.plot_cfg.getHighcharts().zoom();
            $scope.plot_cfg.getHighcharts().xAxis[0].setExtremes();

        });
}

loadData() is called first time when user click on show curves. But when we click on other curves and loadData() is called a second time, curves are refresh in main plot, but Navigator becomes empty.
Affected browser(s)
any idea ?

Could you try with the latest version (1.0.0)? if it is still broken please create a jsfiddle.


opae.io: fix reference to fn in pci.py

Signed-off-by: Rodrigo Rojo<EMAIL_ADDRESS>

Coverage increased (+0.03%) to 64.896% when pulling 6efb78f7a1090bae6981a046e85c30af9ae54e05 on rrojo/opae.io-fix-pcirefs into 2573451cc851fe4470ff0a12d77db3c96c1c1d2a on master.


Create 0000-01-02-ozibozkurt.md

I want to create a pull request for learning github

Thank you, it was basic team work for new learner.


Update lambda function with better error messages

per https://github.com/Senzing/aws-cloudformation-ecs-poc-simple/issues/145

PR merged


add content-type header

following my discussion with recurly's support, they require an explicit deceleration of content type and charset

Hey thanks for this PR.
Just wanted to clarify what lead your discussion w/ recurly support on that conclusion.
Was there an issue before that is now resolved with this (eg: a specific method) didn't work), or is this a precautionary measurement to follow the reculry API guidelines in a more precise manner?

Some API requests failed and after a small investigation we found out that the presence of accented letters in the XML fails the request with error about the validity of the XML although only UTF8 was used.
Their support said that they well support UTF8 and so we search for the origin of the error. their dev team said that the charset should be defined explicitly in the headers, and gave their PHP lib as an example.
IMO it's a fair requirement.

👍


Example usage with swc plugins

I'd like to use SWC with the official styled-components plug-in in Storybook.
How might I do this?

@kylemh
It should be possible to use plug-ins in SWC by writing the following configuration file.
However, as for the styled-components plugin, it seems that it has not been released to npm yet because package.json is not prepared.
I think you need to clone the repository and place it directly in your project or wait until it is released.
module.exports = {
  addons: [
    {
      name: 'storybook-addon-swc',
      options: {
        swcLoaderOptions: {
          jsc: {
            experimental: {
              plugins: [['plugin-name', {}]]
            },
          },
        },
      },
    },
  ],
};

reference: https://github.com/swc-project/swc/blob/c5a0c9a0ab2345aa9c41a40100d049a144dfb70f/node-swc/src/types.ts#L511

Added documentation about the swc plugins to README.md.
https://github.com/Karibash/storybook-addon-swc#swc-plugins

Thanks a ton, @Karibash for both resolutions


HV-1665 Support incremental annotation processing in Gradle

https://hibernate.atlassian.net/browse/HV-1665

As per Gradle doc on incremental APs it looks like it should be enough to register AP in META-INF/gradle/incremental.annotation.processors
with corresponding category separated by a comma.
Also added a small section to documentation on AP and Gradle. (that's how I "configured" the AP for testing the gradle builds and messages from AP). We have the doc sections for various build tools but nothing for gradle.

Hi Guillaume,
I thought it was about time I start opening some PRs if I wanted to get at least something done for free form validation till the new year 😄
I've applied the suggested changes and force-pushed a new commit. As for the testing of this ... I've just created a simple clean gradle project and added all the things we have in the tests folder for AP module. This way I'd get all kinds of errors/warnings. And here's my build file from that project:
plugins {
	id 'java'
}

group 'org.hibernate.validator'
version '1.0-SNAPSHOT'

sourceCompatibility = 1.8

repositories {
	mavenCentral()
	mavenLocal()
}

dependencies {
	annotationProcessor group: 'org.hibernate.validator', name: 'hibernate-validator-annotation-processor', version: '6.1.0-SNAPSHOT'

	// any other dependencies ...

	compile group: 'org.hibernate.validator', name: 'hibernate-validator', version: '6.1.0-SNAPSHOT'
	compile group: 'org.glassfish', name: 'javax.el', version: '3.0.1-b09'

	testCompile group: 'junit', name: 'junit', version: '4.12'
}

compileJava {

}

It compiles and gives warnings/errors  😃 Not sure what else to check here ...
As for

Wondering if we should have a Gradle example somewhere but not sure where. Probably a separate project?

I was thinking about this as well. Usually we give maven examples but never mention gradle anywhere in the doc. Don't get me wrong - I love maven 😄. Hence I would agree that having a gradle example would be nice. But I'm not sure where to put it, maybe under https://github.com/hibernate/hibernate-demos/tree/master/hibernate-validator ? But it would also be nice to be able to reference some examples in the doc as well...

Merged, thanks!


make code better

elijah is it your goal to make your code unreadable
like am i reading greek

yes, yes it is


Users/jscholtz/version

What does this Pull Request accomplish?
Add version resource file for gRPC Device Server (Windows). This adds version and product information to file properties. The version information is parsed from git describe --tags command and is used to generate the version.rc file at build time.
Why should this Pull Request be merged?
Currently we have no easy way to distinguish different version of the Windows executable downloaded from release artifacts. This will add some traceability of file version.
What testing has been done?
Building dev branch locally. Right click on file and review properties.

The CMakeList.txt file had several trailing spaces. My editor removes trailing spaces by default when I save a file. Let me know if there are any issues with removing trailing spaces.

Pipeline is currently failing because we do a shallow git clone which does not have enough history to resolve tags.

When building a release, I think GITHUB_REF is set to refs/tags/v1.2.3. Perhaps you could use GITHUB_REF if it is set and fall back to git describe --tags if GITHUB_REF is not set?

Here's the recipe for doing a full clone if you haven't found it: https://github.com/actions/checkout#fetch-all-history-for-all-tags-and-branches . I'd bet the performance cost of just adding that is tolerable if we don't have another solution.


Using in Acceptance testing

Hi.
I was trying to test that the flash is shown during an acceptance test..:
visit('/customers').then ->
  click('button:contains(\'Delete Customer\')').then ->
    equal currentRouteName(), 'customers.index'
    equal find('.flashMessage').text(), 'Customer was deleted'

It feels like maybe I'm just tired and totally overlooking something, but a pointer would really be helpful.
If i get this working I can write up a little tutorial and add it to the README...

@kimf What was the issue?

As soon as I added the line
equal find('.flashMessage').text(), 'Customer was deleted'
the test failed, and I tried alot of different things, wich I can't remember right now, but gave up and just skipped that assert for now.

Ah okay. Maybe these will help?

https://github.com/poteto/ember-cli-flash/blob/develop/tests/acceptance/integration-test.js
https://github.com/poteto/ember-cli-flash/blob/develop/tests%2Fdummy%2Fapp%2Froutes%2Findex.js

I will try that..
Closing this, as the examples is there and the issue is on my end.
Thanks for your help.


Updated Event Status Page

Now includes a list a of participants

magnuv

Fixed the layout of the RSVPs. Now the name is ellipsisized properly.


Crash after install compile-watch

After install https://atom.io/packages/compile-watch atom has freeze. After force close it and reopen I see message:
The editor crashed
Please report this issue to https://github.com/atom/atom
I was trying to keep it open (this dialog shows again), reload (same), close window (nothing happens)

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:

Try uninstalling compile-watch from the command line with apm uninstall compile-watch.  If Atom starts ok after that, then it seems like an issue with https://atom.io/packages/compile-watch, not Atom itself. It may be that there is still a bug in Atom. If so, we'll need the maintainer of the package that is causing the problem to help track that down. You should file this issue on their repository so they can either fix the problem or tell us where the problem is coming from. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.
If uninstalling compile-watch doesn't help and you still have trouble after that, feel free to post on the official message board and ping me in it: https://discuss.atom.io/.


Visitor class shouldn't take Parser instance as an argument

The current situation makes very little sense, given that (in theory) you can pass a Visitor instance to the Parser constructor. We need to iron out this API glitch.

Fixed in sort of a dopey way. I tried to implement a somewhat cleaner solution, but I was thwarted by the fact that Rhino's AstBuilder is mostly implemented in Java.


Add static analyzers to itevents

Add static analizers PMD, checkstyle and findbugs to itevents module. Analyzers must be connected, but all rules must be disabled.

@AndriyBaibak 1 hour

@AndriyBaibak 15 minutes spent


No KIC 3.x KongConsumer or KongPlugin guide

Where is the problem?
https://docs.konghq.com/kubernetes-ingress-controller/2.12.x/guides/using-consumer-credential-resource/
What happened?
The 2.x KIC docs included step-by-step guides for using KongConsumer, credential, and KongPlugin resources:
https://docs.konghq.com/kubernetes-ingress-controller/2.12.x/guides/using-consumer-credential-resource/
https://docs.konghq.com/kubernetes-ingress-controller/2.12.x/guides/using-kongplugin-resource/
The latest URLs for these redirect to https://docs.konghq.com/kubernetes-ingress-controller/latest/concepts/custom-resources/
https://docs.konghq.com/kubernetes-ingress-controller/latest/guides/using-consumer-credential-resource/
https://docs.konghq.com/kubernetes-ingress-controller/latest/guides/using-kongplugin-resource/
The concepts/custom-resources page is not a guide. It's a high-level overview that briefly describes the purpose of these resources. It does not show how to use them.
grepping for global='true' and kongCredType (or konghq.com/credential) indicates this content was not ported elsewhere.
https://docs.konghq.com/kubernetes-ingress-controller/latest/get-started/key-authentication/ roughly provides the equivalent information for consumers and credentials and would be a better redirect target, though its title indicates it's only for key-auth rather than a concept applicable to consumers in general.
We do have several plugin guides, but AFAICT none that cover the different means of attaching plugins and the precedence order that prioritizes, for example, a service+consumer plugin over a service plugin.
What did you expect to happen?
Content from those 2.x guides demonstrating usage of those resources is still available.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

@mheap Any update on this?
I think this one is closely related to my request in https://github.com/Kong/docs.konghq.com/issues/5324 which I submitted nearly two years ago.


docs: use async for instead of for for streaming FastAPI resp

Hi 👋🏻
I tried the docs snippet and found that it should use async for instead of for to avoid TypeError: 'async_generator' object is not iterable

:rocket: This PR description was created by Ellipsis for commit 5f3291dba2a8efab3bc87f5977ac8de089941fd7.

Summary:
This PR updates the fastapi.md documentation to use async for instead of for when iterating over an async generator in the generate function within the extract function.
Key points:

Updated fastapi.md documentation file
Changed for loop to async for in generate function within extract function
The change is necessary to avoid TypeError when iterating over users, which is an async generator

Generated with :heart: by ellipsis.dev

As a beginner, Still It's hard for me to get up and running with simple FastAPI & Instructor. Can you please add full runnable example file so we can check?

can you try this?
https://github.com/instructor-ai/cloud


Expense - Logo in Share code for transaction thread is not Expensify logo

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.10-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/39072
Action Performed:

Go to staging.new.expensify.com
Go to DM with no unsettled expense
Submit an expense
Click on the expense preview
Click report header > Share
Note that the logo in share code is Expensify logo
Create another expense
Go to transaction thread
Click report header > Share

Expected Result:
Logo in Share code for transaction thread is Expensify logo
Actual Result:
Logo in Share code for transaction thread is not Expensify logo, It follows the individual policy avatar
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/22ce232b-d900-4216-9df1-ca6571aae025
View all open jobs on GitHub

@Kicu @grgia @blazejkustra @luacmartins @sobitneupane Seems like this might be coming from your PR https://github.com/Expensify/App/pull/39072 can you have a look please?
Ultimately I think this could be demoted as its a minor UI issue that does not block the user

Hey thanks for calling us. Yes this is definitely coming from my PR https://github.com/Expensify/App/pull/39072
However you can take a look at this comment from @shawnborton https://github.com/Expensify/App/pull/39072#issuecomment-2236052607
Who suggested we move forward with switching the current expectation of using expensify logo to using the same logo as in header.
Now the app is not yet 100% ready to do this (we need 1 more PR) but there are parts of the app that will already show the avatar, so I think this should be deemed a not bug.
Please comment on this @shawnborton

Correct, this is not a bug - we want to move towards a world where the avatar found in the share code always matches either the user's avatar, the workspace avatar, or the avatar found in the chat header. I can close this out, this is not a deploy blocker.

@Kicu what are next steps to make this work everywhere though? Do we have that spelled out anywhere, and do we need a new issue to track this? Happy to make one if so, just let me know!

I believe the main thing I need is a new issue defined be someone so that I can get assigned to it.
I don't think I need any detailed definition other than "make the avatar inside QR code the same as in header".
The issue would also be a place to discuss any edge cases that might arise while doing that (I'm not yet sure simple this task will be, probably simple-to-mid-complex).

Sounds good, I'll make the new issue now.


Enable to filter challenges using date range

Options:

List challenges that with an activity date range that overlaps with the specified date range
List challenges that have an activity date rage that is fully contained within the specified date range
List challenges that have a start date that falls within the specified date range
List challenges created during the specified date range
Other

Which of the above option(s) do we want to implement?

Done


any plans to update this for el capitan?

would love to use this for the new emojis! thanks much—cc

FWIW I just tried it on El Capitan and it appeared to work flawlessly.  If anyone else can confirm that it works perhaps all that's needed is an update to the docs?


Add Azure.Communication to the allowlist of namespaces

Fixes #1053

Is it approved by the archboard?

@pakrym Yes it is :)

I kicked off the build https://dev.azure.com/azure-sdk/internal/_build/results?buildId=556324&view=results
We would need to update the version of analyzers in the main repo after it's done.


yii\jui\Spinner clientEvents

The only client event Yii2 listens to is 'spin'. The other events don't work as expected.
For instance, trying to catch the 'change' event by setting:
        'clientEvents' => [
            'change' => 'function(e, ui) { console.log(this); }'
        ]

Results in the following Js:
       jQuery('#xxx').on('spinnerchange', function(e, ui) { console.log(this); });

While it should be:
       jQuery('#xxx').on('spinchange', function(e, ui) { console.log(this); });

I suppose it has something to do with the protected property $clientEventMap.

Fixed https://github.com/yiisoft/yii2-jui/commit/3d4f3d7d0e86af0e0b051a3660517f3041623ffa. Thanks!


Incorrect sensor data?

The addon pulls through all the data which is great, however my SG8k with either the config for the 5kw or 8kw version reports no data to the sensors:

These are the entity attributes:

state_class: total_increasing inverter_size: 8
daily_power_yield: 100
total_power_yield: 0
inverter_power_on_hours: 2
internal_temp: 39.1
pv1_voltage: 0
pv1_current: 0
pv2_voltage: 0
pv2_current: 0
total_pv_power: 0
grid_voltage: 235.8
inverter_current: 0
total_active_power: 0
grid_frequency: 50.1
export_power: 493
export_power_indicator: 0
power_meter: 493
daily_export_energy: 0
total_export_energy: 0
daily_import_energy: 4200
total_import_energy: 4.2
daily_energy_consumption: 100
total_energy_consumption: 0.1
timestamp: 2022-03-17T19:41:14
unit_of_measurement: kWh
device_class: energy
icon: mdi:solar-power
friendly_name: Inverter Daily Energy Import

I can see the power generation in the app, anyone having the same issues? I can't select any other attributes for the HA energy component so it'd be good to get a fix for the addon

Edit: I can see the addon is getting all the correct data in the logs but constantly getting this error in Mosquitto broker:

1647560585: New client connected from <IP_ADDRESS> as ModbusTCP (p2, c1, k60, u'addons').
1647560607: Socket error on client ModbusTCP, disconnecting.

I've seen in other issues that a restart fixes it but I've tried restarting both the Addon, Mosquitto & HA itself multiple times and no fix

I've got this working now


API-1835: fix up minor comments from Lucasz

from https://github.com/openshift/library-go/pull/1794

great, but please check the other controllers as well. I think you need to check all of them. For example the comments also apply to https://github.com/deads2k/library-go/blob/a86ed6242ce1d886cd79fbb1785829208d4d4f35/pkg/operator/certrotation/client_cert_rotation_controller.go#L102 which you didn't change here.


Redo ability selection menu

Player abilities are currently chosen from a row of buttons in the bottom-right corner of the screen. This takes away focus from the player actor, and may look cluttered as more abilities are added.
Possible Solution: Abilities are chosen from a pie menu that appears when the player clicks on the player actor. Would require icon images for abilities.

Pie menu implemented as of 52b29cd

Pie menu implemented as of 52b29cd


tr: Illegal byte sequence

When trying to run the ./generateId.sh script I get:
tr: Illegal byte sequence D----
For now I found a solution? here:  https://www.real-world-systems.com/docs/tr.1.html
The fix seems to be to run export LC_ALL=C

Host:

VM: ubuntu/xenial64 Vagrant box (16.04)

you are right. same situation.I found an answer here: https://unix.stackexchange.com/questions/45404/why-cant-tr-read-from-dev-urandom-on-osx

I had the same issue on macOS.  The above fix corrects it.
run: export LC_ALL=C
then run: ./generateId.sh

@mgfield gedit is not mandatory, feel free to use any editor to make the necessary changes.


Delete S3 buckets older than t time

It would be fun to have this option too. We have a couple of labs accounts so... you see...
Thanks in advance,
F

I mostly have this working, however there's a small annoying thing. If I simply duplicate ami.go and change it to handle S3 buckets, it will list all buckets again and again as we iterate through regions. It does this because there's no way to only list buckets within a region.  I can call GetBucketLocation and get the region for each bucket so we can use it to exclude regions, of course doubling the amount of API calls.
There are, as I see it, two solutions.

Use the above approach and be fine with issuing a silly amount of redundant API calls, going through i.e. 75 buckets and 8 regions would yield 75*8 api calls.
Treat S3 differently, this would mean changing quite a few things I would imagine, and I have no idea how you'd want this implemented.

If solution 1 is what is preferred, then I will submit a PR within the next few days.

How about building a local cache?
Each time getAllS3Buckets is called:

Check if the cache contains buckets.
If it does, return the buckets for that region.
If it does not:

Fetch all bucket names.
Loop through and get the bucket location once for each bucket name.
Put it all into a cache (i.e., a map of lists) broken down by region name.
Return the list of buckets for the currently requested region.

This would require 75+1 API calls for 75 buckets, but there's no working around that.

Great suggestion, for some reason that didn't occur to me as an option, but it's pretty logical, thanks!
I'll submit a PR once I have this working :)

@lasseoe have you made any progress on this PR? We got bit by something similar and would love to have S3 added to CloudNuke.

@brikis98 @davidvasandani raised https://github.com/gruntwork-io/cloud-nuke/issues/58 to support S3 bucket deletion.

This is now released in https://github.com/gruntwork-io/cloud-nuke/releases/tag/v0.1.17