Requesting official release build

Hi, friends in gosop community,
Thanks for your work on this great project.
Is it possible that gosop community provides official build in the release.
background
I'm developing a command-line tool called x-cmd that aims to provide portable tools across different platforms. I've found that gosop might be the only portable and cross-platform PGP CLI tool available.
I'd love to share gosop with users, similar to how we offer OpenSSL ( see https://x-cmd.com/pkg/openssl ). While we can build it ourselves, we prefer distributing community releases to benefit from shared maintenance and bug fixes.
Are there any plans for an official gosop release that we could potentially include in x-cmd?
Thanks.

@edwinjhlee i'm not sure what your requirements are for portable, cross-platform OpenPGP CLI tool are, but if they include things that can be built with Rust, you might be interested in rsop or sqop (when sqop is built against the rust-crypto background).
Both of these implementations should compile to native code, and implement the same interface as gosop.

it's great that 1.0.0 is out!  thanks for proposing to push binaries on release, @twiss .

Hi ~ There is still no prebuilt binaries in the release. Maybe the action haven't been triggered yet ?

Yes, that release predates this PR. It'll trigger for the next release.

Made a new release here: https://github.com/ProtonMail/gosop/releases/tag/v1.1.0. It seems to work :)

Made a new release here: https://github.com/ProtonMail/gosop/releases/tag/v1.1.0. It seems to work :)

Awesome ~

@twiss @dkg
Now users could type x gosop and x-cmd will download on demand and execute gosop.
This is the demo: https://x-cmd.com/pkg/gosop
Thank you for this great project.


Structural Updates

Updates:

Adding SemVer and CHANGELOG.md
Updating CircleCI
ReStructuring
Adding user-agent to req.headers

@dchai76 @andkon @vilyapilya, can you guys review the changes? Appreciated!

@dchai76, can you do second round? Thanks in advance!


Allow callable tables as callback parameters (closes #265)

pcall (which is used to handle the actual calling of the callbacks) handles __call already, so the only thing blocking this was the LUA_TFUNCTION type check.
Example code that will now work:
local callable = setmetatable({}, {__call=function(...) print("called", ...) end})

callable()
pcall(callable)

local uv = require('uv')

local handle = uv.new_timer()
local delay = 0
uv.timer_start(handle, delay, 0, callable)

uv.run()

Which outputs:
called  table: 0x000d8420
called  table: 0x000d8420
called  table: 0x000d8420

Example error when trying to pass something non-callable as a callback argument:
bad argument #4 to 'timer_start' (function or callable table expected, got boolean)

(related: #265)

This is exactly what I hoped for.  I just didn't remember if I had implemented it yet.
Thanks!


Additional removals

3rd party Appx that has started appearing in Windows 10 1709, observed in Dell OEM system images.

Good additions, thanks.


Update IntentPickerSheetView modifiers

This will help developers access each variable and customise it.
(ImagePickerSheetView.java set all variable with protected modifiers.)

I'm not sure how it feel on this, as we had planned separately to allow customization of things like selectors via styles (there's an issue for it, on mobile and don't know he number offhand). We could go with this and say all bets are off if you modify the views directly though, and just treat using the styles as a second layer of defaults. We'd just need to make sure it's nothing that could cause crashes if the underlying AbsListView instance is different than we expect.


Unable to Install - Mac OS Siera

I get this after typing in the first command in terminal:
sudo pip install pcapy
https://gyazo.com/8cbe43fb0bf73169bca29a739a667923
and this after running the second command
brew install libdnet scapy
https://gyazo.com/2014df722bc7871684a9a9baeb036bfc
I think I have installed homebrew correctly before trying the above commands.
Any help would be greatly appreciated.
Thanks

Hi there. Check out the following issues: #13, #7 & #10. Please let me know if this resolves your issue. Thanks.

#13 has enabled me to get as far as running the script.
Now I have the error
https://gyazo.com/914dbb66880b7de9dec10603069ff05d
after typing
sudo python kickthemout.py
Thanks for the help

Now I've managed to get the stage of entering my gateway IP.
But it now says...
https://gyazo.com/cd817669b0cc71c68fcfc5fd93f47ac9


Why such an old version for mongo?

Hi all,
yesterday I tried to install bot both on windows and rpi4 (ubuntu 20.04 64bit).
On windows everything is smooth and instructions are ok :)
On rpi4 there were 2 problems:

I needed to "recompile" binance-bot docker (see issue #22 )
I needed to find a working version for mongodb and I saw that version 3.2.20-jessie is very old (and not available for aarch64). I tried to update to 3.6-xenial (so still a 3.x) and things looks like are working fine now

So the question is: is it possible to update the mongodb version to something more recent?
Thanks in advance
Bye
Piero

I read that raspberry pi older than rpi4 is discouraged... upgrading mongo to a recent version will be a clear sign of the supported platforms :)

Read README.md - https://github.com/chrisleekr/binance-trading-bot#trailing-buysell-bot


Races:  Hobgoblin(Not Legacy)

Fey Gift, Hospitality: Damage formula uses a raw 4 instead of @prof  In addition to this, the damage is set to none when it should be temporary hitpoints.  Doesn't really matter ultimately but the target is set to none, it should probably be 2 blank creature and then that means the ability is meant to apply to self and target in a manual targetting setup.
Fey Gift, Passage looks pretty easy to automate but I'm passing on it unless you want to add it.  Spite is going to be macro territory like a bonus action smite almost.
Fortune from the Many could use an icon, I never picked one.

So this is one of those places where the values provided by the DDB Character sheet are fixed integers, rather than references to the prof bonus. it looks like the racial traits/features are more janky here than some of the class features, most likely due to the fact that prof bonus related stuff arrived very late in the 5e life cycle.


Device List Logic Transmitter

The  Device List is quite confusing organised, would be better with ingame namespace. (for instance LED small is LED5 here).
Having trouble to find the logic transmitter devices...
great work however, amazing.

it's a good idea, I can organize the grouping into the same categories, but you need to decide which categories they are.
I suggest  use the categorization from stationpedia

I added localization from the game now you can search in your own and English languages. So how are you used to
p.s
reopen this issues if you think that this is not enough :)


Feat: unlock forms

Added a new step for the unlock flow
Update confirm screens

@sokolova-an missing key on the unlock formscreen

@sokolova-an unlockable b
Uploading Screen Recording 2024-07-29 at 16.06.51.mov…
alance doesn't filled into the form (please, check the video).

@sokolova-an unlockable balance doesn't filled into the form (please, check the video).
https://github.com/user-attachments/assets/4d5bedaa-4ba1-4501-9ed1-12be3883ecad


I can't build ceres

I put eigen, glog, directories in the build directory.  Upon building in 2015 Visual Studio, I get ~27 errors.
Also: I made sure I'm using ceres-solver 1.12 and not 1.13 (most updated version).  I was not able to build 1.13 via CMake, so trying this route...
Thanks,
Brent

Hi Brent,
Can you please post the error log here?


Retire the barrel bowl and insert the arbour hole in setter bowl instead

This part is no longer needed.
Only advantage of barrel bowl is the arbour hole that helps with snapping the barrel lid back on. This feature needs to be inserted into the setter bowl and then this part can be retired.

I think I have already addressed this issue. In the "Placing Base" I have created, I designed it with a hole accommodate the arbor.

Latest release no longer has this part.


Big header gap inherited from Windows 8 for the WebAuthenticatorBroker

The developer forgot to change the height of the WebAuthenticatorBroker header when they moved from Windows 8 to Windows 10. It's very ugly. See screenshot

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

ID: 3c758cb2-ad03-f28e-6947-21337462753a
Version Independent ID: 3e818dc7-e032-c92b-80ba-0c0773391fec
Content: WebAuthenticationBroker Class (Windows.Security.Authentication.Web) - Windows UWP applications
Content Source: winrt-api-build/xml/Windows.Security.Authentication.Web/WebAuthenticationBroker.xml
Product: uwp
Technology: winrt-reference
GitHub Login: @lastnameholiu
Microsoft Alias: alholiu

Thanks for taking the time to let us know about this. The scope of our feedback channel here on GitHub covers specific documentation fixes, but I can help redirect you to the right support channel to provide ypour feedback. The community of experts and Microsoft Engineers over at Microsoft Q&A offer timely answers to a multitude of technical questions surrounding windows-uwp. Hope your issue can get more visibility there.


BulkInsertOrUpdate not setting Ids automatically

I am having an issue using the BulkInsertOrUpdate functionality whilst trying to insert a list of records into a sqlite database.
public async Task BulkInsertOrUpdate(List<TEntity> entities)
{
    var bulkConfig = new BulkConfig
    {
        SetOutputIdentity = true,
        PreserveInsertOrder = false
    };
    await _context.BulkInsertOrUpdateAsync(entities, bulkConfig);
}

The list contains a basic list of entities with the following properties:
Id - int
Version - byte[]
FirstName - string
LastName - string

When the entities are created, they do not have an Id set i.e. Id=0. When I use BulkInsertOtUpdate, only 1 record is added to the database. This record is generally the last item in the list and it has an Id of 0.
When I use the BulkInsert method, the records are inserted correctly and their Ids are automatically set.
I can also insert the entities if I manually set the Id myself, however, I would like to avoid this as it will crreate a large overhead for me if I had to do this for the more complex entities.

You should leave PreserveInsertOrder to true

You should leave PreserveInsertOrder to true

I have set this to true as you suggested and tried it with a list of 709 items. The problem still persists and only one item was added to the database.

(ignore the column names, this is a different entity to the one in the first post).

It's actually up to Sqlite itself, which does not have full MERGE capabilities like SqlServer.
It only supports simple Upsert with ON CONFLICT command.
So you can either:
a) split list into 2 lists, and call separately BulkInsert and BulkUpdate or
b) if want to keep all in single method then you would need to read last Id and for news ones set Id from that number incrementally.
I would use chose option a), it's simpler.

Thanks for the clarification. This is what I have ended up doing for the meantime, so I will just stick with that.


Errno::ECONNREFUSED in HomeController#index

Hello,
I'm new to GenieACS and Ruby. I followed the instructions to set it up on my own and it GUI worked at first but when I tried to connect a CPE, following issue appeared.

Hello,
I am facing the same problem. How do I configure the URL:port of the API in genieacs-gui?
I followed the installation instructions here (https://tr069.wordpress.com/2014/03/26/installing-genieacs-on-precise-pangolin/). Also, I didn't change any configuration files.
I think the back end API (genieacs-nbi) is working properly. Here is the output on terminal:
vm-genieacs@vmgenieacs-VirtualBox:~/devel/genieacs-1.0.3$ ./bin/genieacs-nbi
13 Mar 11:04:59 - Worker 15252 listening to <IP_ADDRESS>:7557
13 Mar 11:04:59 - Worker 15251 listening to <IP_ADDRESS>:7557
Here is the output on terminal from sudo rails server after I open the webpage (<IP_ADDRESS>:3000):
vm-genieacs@vmgenieacs-VirtualBox:~/devel/genieacs-gui-master$ sudo rails server
[sudo] password for vm-genieacs:
=> Booting WEBrick
=> Rails 4.1.4 application starting in development on http://<IP_ADDRESS>:3000
=> Run rails server -h for more startup options
=> Notice: server is listening on all interfaces (<IP_ADDRESS>). Consider using <IP_ADDRESS> (--binding option)
=> Ctrl-C to shutdown server
[2015-03-13 11:13:39] INFO  WEBrick 1.3.1
[2015-03-13 11:13:39] INFO  ruby 2.2.1 (2015-02-26) [i686-linux]
[2015-03-13 11:13:39] INFO  WEBrick::HTTPServer#start: pid=15379 port=3000
Started GET "/" for <IP_ADDRESS> at 2015-03-13 11:13:49 +0100
Processing by HomeController#index as HTML
Completed 500 Internal Server Error in 219ms
Errno::ECONNREFUSED (Connection refused - connect(2) for "localhost" port 7557):
app/controllers/home_controller.rb:12:in block (5 levels) in index'   app/controllers/home_controller.rb:11:in each'
app/controllers/home_controller.rb:11:in block (4 levels) in index'   app/controllers/home_controller.rb:10:in each'
app/controllers/home_controller.rb:10:in block (3 levels) in index'   app/controllers/home_controller.rb:9:in each'
app/controllers/home_controller.rb:9:in block (2 levels) in index'   app/controllers/home_controller.rb:4:in block in index'
app/controllers/application_controller.rb:40:in can?'   app/controllers/home_controller.rb:3:in index'
Rendered /usr/local/lib/ruby/gems/2.2.0/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/_source.erb (0.8ms)
Rendered /usr/local/lib/ruby/gems/2.2.0/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/_trace.html.erb (1.7ms)
Rendered /usr/local/lib/ruby/gems/2.2.0/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erb (1.0ms)
Rendered /usr/local/lib/ruby/gems/2.2.0/gems/actionpack-4.1.4/lib/action_dispatch/middleware/templates/rescues/diagnostics.erb within rescues/layout (81.5ms)

If you're running genieacs-gui on a separate server then edit the API host and port in config/environments/development.rb.

Hello, thank you for the reply. I can open the gui web page now. I think the problem was with the proxy setting on my VM. Thanks for the help!

Hi iam getting the same issue please help me please check the screen shot

@mushir3589 did you find any solution please ? I have the same issue...

Hi.
I see the same error. How did you solve it?

@GLENDIS Check the process genieacs-nbi is up and running.
Anyway, if you still have problems you can use my own solution: https://github.com/genieacs/genieacs/wiki/Docker-Installation-with-Docker-Compose

Hello Sergio,
I'm using your project in Docker for GenieACS, after I installed everything and made the changes in the configuration file try to start genieacs-gui and I throw the previous error.
I tried to upload the genieacs-nbi service in Docker in the path / opt / genieacs / bin and it gave me errors.
How can I check that the genieacs-nbi service is working in the docker? or how can I upload the genieacs-nbi service?

Better to post your issue with the docker and Docker-compose logs in my own repository at Github here: https://github.com/DrumSergio/Genieacs-Docker , your issue has nothing to do with bugs in genieacs / genieacs-gui. This issue tracker is not intended for deployment configuration help.

Good afternoon.
I have a query, if I load the container with information by default as I can validate that it is working correctly?
Because it presents the same problems reported by other users.

@villegasgs Check my comment here https://github.com/genieacs/genieacs-gui/issues/1#issuecomment-507754094
Please post the issue in my own repository. I have just checked building a docker container and for me it's working ok. Better to post the exact steps you've done there with some more logs.

i disable ipv6, beacuse localhost was not resolved to <IP_ADDRESS>, and start to work.


Event Grid: Regenerate new .NET SDK and record tests for new GA service API version 2020-06-01

Event Grid .Net SDK 6.0.0 (2020-06)

Add support to new GA service API version 2020-06-01.
The new GA'ed features include:

Event delivery schema,
Input mapping,
Custom input schema event delivery schema,
Cloud event V10 schema,
Service bus topic as destination,
Azure function as destination,
Webhook batching,
Secure webhook (AAD support),

ImmutableId support, IpFiltering, and
Private Link.

Add Changelog.md with latest nuget packages history and link it in the release notes. This is to address the suggestion per issue https://github.com/Azure/azure-sdk-for-net/issues/12252

Hello @isra-fel , @erich-wang .l Kindly help merging this PR. thanks

Thanks @isra-fel  .. Yes, all the deleted code is related to features that are still in review..
api version is updated in assymblyinfo and az.Sdk files.
thanks


Add syntax for qualified types

This is a port of @Sporarum's work with the following changes:

Now using an intermediate untyped tree QualifiedTypeTree that is desugared into an annotation during Typer instead of creating the annotation during parsing already. This is more in-line with how similar features are implemented and is preferred by IDEs developers according to Martin.
Dropped support for the alternative trailing lambda syntax; only kept the set notation one.
Dropped support for indentation-based qualified types. They can now only be surrounded by curly braces.

🟢 CI passed: https://github.com/scala/scala3/actions/runs/10872147052.

🟢 CI passed: https://github.com/scala/scala3/actions/runs/11390811458.


Rich asset details dropdown

Story
As a trader
I want to see more asset details in the asset dropdown
So that I can more easily select an asset for deposit or withdrawal
Acceptance Criteria

[ ] I can see the asset symbol
[ ] I can see the asset name
[ ] I can see the asset contract address or type (eg BuiltIn)

Tasks

[ ] Explore and sketch
[ ] Team and stakeholder review
[ ] Visual Design
[ ] Team review
[ ] Etc.

Sketch
Additional details / background info
https://vegaprotocol.slack.com/archives/C037STCKMGC/p1673267954910889

We did something similar to this in console lite


Define setup for distinct dependencies and devDependencies

Similarly to what we see in node projects, not sure if possible in this case, but the artifacts we release shouldn't carry with them dependencies that we only use to run unit tests.
We need to investigate if this is the case and how to prevent it.

@jmromanos closing this issue since it's no longer applicable.


Don't use git2 default features as they are not needed.

This reduces the amount of dependencies from 102 to 94.

❯ cargo tree  | wc -l
     102

GQL ( master) +54 -2 [!]
❯ gco -b no-featu

GQL ( master) +54 -2 [!]
❯ hx Cargo.toml

GQL ( master) +55 -3 [!] took 23s
❯ cargo tree  | wc -l
      94

Please note that I also thought I'd look into what it would mean to switch from git2 to gix, but felt I wouldn't be of much help here since it appears there is not a single test to validate the new gix backend against.
If that changes and you are interested in the additional performance you'd undoubtedly get I'd be happy to do an oxidization analysis to see if gix provides all the features you'd need. From what I saw, the answer would be a positive one and a transition is already possible.

Hello @Byron,
Thank you for reducing dependencies.
For now all i used from Git2 is the read feature and I Think yes gix will be a good option and will cover the project need, Until test functions are created to cover the backend i will try gix in side project or maybe in other branch to get familiar with it and check the other features.
Thank you,
Amr Hesham


Migration search doesn't work correctly

Steps to reproduce

Install an English source, e.g. Mangabat (EN)
Install foreign source, e.g. Desu (Ru) or Remanga (Ru)
Try to migrate from Mangabat to other foreign source

Expected behavior
Non all/En sources available in migration search list
Actual behavior
Only All and En sources are available in search
Crash logs
tachiyomi_crash_logs.txt
Tachiyomi version
1.8.3
Android version
Android 13
Device
Google Pixel 6 Pro
Other details
No response
Acknowledgements

[X] This is a TachiyomiSY specific issue that does not happen in Tachiyomi Preview.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.3.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Sounds like you didnt enable the other languages in the source list

Nope, they are enabled

Ok, I just noticed that only a few of sources are available during migration. I installed some english sources but they won't appear in migration search.

Wait, you must have enabled the skip pre-migration screen option. Go into Settings -> Library and disable it.


MUMUP-2123 part 1: Create basic frame page

Creates a basic <frame-page> directive that you can pass in header information. It utilizes <portlet-header> for the header and ng-transcude for the body. This should promote similar pages.
I rewrote the main.html page in frame to show as an example

:+1:

:+1:

:+1:


File-Class name mismatch

https://github.com/aspnet/AspNetCore/blob/master/src/Identity/Core/src/DataProtectionTokenProvider.cs
File name does not match class name.  Made it a hassle to find in github.

@HaoK fancy a quick rename? :)


Help build on Apple M1 chip

Summary
This fix helps build on Apple M1 chip running macos Ventura and go 1.20. Without the fix, the build fails with the error below (cadence-server was used as the sample source to run the built cadencelint)
  cadencelint: internal error: package "context" without types was imported from "github.com/uber/cadence/service/worker/workercommon"

Config details of macOS
Output of uname -a:
Darwin MacBook-Pro.local 22.5.0 Darwin Kernel Version 22.5.0: Thu Jun  8 22:22:20 PDT 2023; root:xnu-8796.121.3~7/RELEASE_ARM64_T6000 arm64

Output of go version:
go version go1.20.6 darwin/arm64

Issue Details

Download cadencelint and build (make build). This creates the binary build/cadencelint

As a sample, download cadence-server source and use the built cadencelint to check.

  cadence-server % ~/sources/cadencelint/build/cadencelint github.com/uber/cadence/service/worker/workercommon

  cadencelint: internal error: package "context" without types was imported from "github.com/uber/cadence/service/worker/workercommon"

Fix Summary

Upgrade go version to 1.20
Upgrage golang.org/x/tools version to v0.6.0

Thanks @nanarayanan ! Would you mind taking a look at the breaking build? (chances are -mod readonly is not necessary anymore)

Thanks @nanarayanan ! Would you mind taking a look at the breaking build? (chances are -mod readonly is not necessary anymore)

Pushed a commit to fix the build failure.


authn: add support for custom callback url

Related: https://github.com/greenpau/caddy-security/issues/64

@simaotwx , also, please add your CLA consent to this commit.

@simaotwx , additionally, please add tests.

was this dropped or merged?

@steverweber dropped. Haven't found the time to do all the things mentioned. Nevertheless, the commit works for my use case so you can pick it if you feel like you need it, too.


3.4.1-RC2 Release procedure

[x] Publish 3.4.1-RC2 artifacts to Maven via CI

[x] On release-3.4.1 branch:

[x] In changelogs add file with a changelog
[x] In Build.scala: Set baseVersion to 3.4.1-RC2
[x] In Build.scala: Set previousDottyVersion to 3.4.1-RC1
[x] Tag the branch as 3.4.1-RC2

[ ] Wait for confirmation from tooling:

[x] Metals
[x] Scala CLI
[x] Scala Native (@WojciechMazur)
[ ] Scastie (@rochala)
[x] Scala DAP (@adpi2)

[ ] Announce the releases

[ ] Make GitHub releases public
[ ] Update the 3.4.1 tracking thread

I tried it today and found a regression since 3.4.0, but apparently it already existed in 3.4.1-RC1. Need to minimize.

Update, while trying to minimize the regression that appeared from this upgrade I managed to get the error to always show up until I downgraded to 3.0.0. See #19955
Somehow it was hidden thus far in my library and only showed up when upgrading to 3.4.1-RC1. It could be that my minimization found a different error, but I doubt it.


NO API ENTRY PROMPT

when i open the application in the downloaded file, no prompt is given to enter API code. making me unable to connect my launch monitor. please help!

the newest build no longer requires the API key. just make sure you authorize awesome golf in the rapsodo app


Electronjs not listed anywhere in neither books or courses

I was looking for books and courses for Electronjs but I couldn't find the category in books or courses.

If you have several links... add a sub in TypeScript


account for favorite collections in other projs

OneFavoritePerOwnerValidator was erronously checking to see if the user had any favorite collections across all projects, rather than just for the project for which the new collection was being created. Update code to fix this bug.

Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

@nciemniak what's the status of this PR?


Using maven to bundle the jar add superfluous dependency

Effectively, maven creates a zip file. In addition to the theme files, there are 4 more files, two of which are created by keycloakify itself. We could easily replace the maven dependency by creating the zip archive using nodejs functions.

I'll take a swing at it

I would also love it if the output artifact name and location was configurable. Reason being is that I'm using gradle, and in order to set up the build tasks correctly that allow for caching/skipping due to no changes, it helps to know the exact file paths of all inputs and outputs.

Hi @mrpatrickpilch, wouldn't it be an option to keyclaokify && mv ...?

@garronej But what's the first argument to mv? That path is controlled by your code, not mine.  I suppose I can write a function to parse it out of package.json and construct the file path myself, but now I have two path generation functions - yours and mine. Which is authoritative? I'd hate for them to get out of sync. If you relinquish control over those details to the consumer, I can control that in my gradle build and depend on it not changing out from underneath me.
Elaborating on that thought - it would be helpful if you split out the build tasks. Right now it's all or nothing - the keycloakify task sets up the directory structure for the theme jar, and also requires running maven. What if this was split into two phases? One task sets up a directory in the proper theme structure, and a separate task creates the jar. In my case, I simply won't run the latter task and handle it myself with Gradle. For backwards compatibility, the jar task can run maven as it always has. And hopefully in the future you can remove the dependency on maven entirely as @lordvlad suggests.

@mrpatrickpilch Thanks for the feedback.
I see the problem. I don't have time to address it now but I agree.
If you want to submit a PR I'll review it.

@mrpatrickpilch we have the same use case, so this is what I'm aiming at

@mrpatrickpilch actually a nice idea to split it into two steps, I woud've squashed  it into one and maybe added a cli param

a nice idea to split it into two steps, I woud've squashed it into one and maybe added a cli param

I would like npx keycloakify to be the only command.

@garronej sure, that would be required for backwards compatibility. Still you could split it and enable/disable later parts using config or cli or env vars

@mrpatrickpilch @garronej I didnt find a way to nicely 'split' it without too much refactoring, what I did instead was introduce an option to disable bundling (in addition to having an alternative nodejs-only bundler). Please find details on #239


Broken "State" generics for HandlerContext

A minimal reproduction of this issue is here: https://github.com/cdcarson/fresh-handler-type-issue
If I define a MiddlewareState type to hold some data collected by my middleware, like this...
// types.ts
export type MyMiddlewareState = {
  foo: string;
  bar: number;
}

...this works fine in _middleware.ts...
// routes/_middleware.ts
import { MiddlewareHandlerContext } from '$fresh/server.ts';
import { MyMiddlewareState } from '../types.ts';
export async function handler(
  _request: Request,
  context: MiddlewareHandlerContext<MyMiddlewareState>,
) {
  context.state.foo = 'baz';
  context.state.bar = 1;
  const resp = await context.next();
  return resp;
}

...that is, the generic MiddlewareHandlerContext<State = Record<string, unknown>> accepts the MyMiddlewareState type.
In the route, however, I'm running into linting problems with the similar generic HandlerContext<Data = unknown, State = Record<string, unknown>>.  I'm trying to do this:
// routes/index.tsx
/** @jsx h */
import { h } from "preact";
import { MyMiddlewareState } from '../types.ts';
import { HandlerContext, Handlers, PageProps } from '$fresh/server.ts';

type RouteData = { foo: string, bar: number };
export const handler: Handlers = {
// LINTER ERROR NEXT LINE
  GET: async (request: Request, ctx: HandlerContext<RouteData, MyMiddlewareState>) => {
    const resp = await ctx.render({...(ctx.state as MyMiddlewareState)});
    resp.headers.set('X-Custom-Header', 'Hello');
    return resp;
  },
};

export default function Home(props: PageProps<RouteData>) {
  return (

{JSON.stringify(props, null, 1)}

  );
}

It runs fine, but the linter for the GET function errors with:
Type '(request: Request, ctx: HandlerContext<RouteData, MyMiddlewareState>) => Promise<Response>' is not assignable to type 'Handler<any, Record<string, unknown>>'.
  Types of parameters 'ctx' and 'ctx' are incompatible.
    Type 'HandlerContext<any, Record<string, unknown>>' is not assignable to type 'HandlerContext<RouteData, MyMiddlewareState>'.
      Type 'Record<string, unknown>' is missing the following properties from type 'MyMiddlewareState': foo, bar

I'm pretty sure I'm using the generic correctly, but let me know if that's not the case.
Thanks!

Specify Handlers<RouteData, MyMiddlewareState> instead of explicit type on the function parameters.


fix: transfer out domain crashes app

Creating a SNS Transfer out domain proposal would crash the app.
Refactored component and hook to correctly display the available domains in each asset account.

Here's a demo
https://user-images.githubusercontent.com/75177443/228324122-0640ecbd-b167-48bf-b167-ba24da22f305.mp4


🐛When changing TimeZone in Settings - the search start and end index should also be set to search in the same time sone.

Task: Make a change so that also the start and end index search boxes is in the same Time Zone as selected in the Settings

Completed by #2415


Kustomize v2 compatibility

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #X, partially #Y, mentioned in #Z

License
Apache 2.0

What's in this PR?
Moving default kustomization.yaml configuration from config/default to config.
And taking manager_image_patch.yaml file out of the base and put it under config/custom directory.
Why?
So that it'll be compatible with kustomize v2 as in version 2 it's not possible to reference resources outside of root dir.
This will allow us to have our own kustomization.yaml file with a reference to yours as a base without cloning the repo like below:
bases:
  - github.com/banzaicloud/istio-operator/config/base?ref=release-1.1

Additional context

https://github.com/kubernetes-sigs/kustomize/issues/776
https://community-banzaicloud.slack.com/archives/CGKQF5HUP/p1559292931039900

Checklist

[x] Implementation tested

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

I would should suggest we keep config/default/kustomization.yaml (instead of moving it to config/custom) with below content for backward compatibility for those who have integration with this file, so we don't have to bump major version:
bases:
- ../base

patches:
- manager_image_patch.yaml

Also what do you think about creating two different kustomization configs under config/custom , each one uses base config with separate patches:
# config/custom/manager_auth_proxy_enabled.yaml
bases:
- ../base

patches:
- manager_image_patch.yaml
- manager_auth_proxy_patch.yaml

# config/custom/manager_prometheus_scraping_enabled.yaml
bases:
- ../base

patches:
- manager_image_patch.yaml
- manager_prometheus_metrics_patch.yaml

I would should suggest we keep config/default/kustomization.yaml (instead of moving it to config/custom) with below content for backward compatibility for those who have integration with this file, so we don't have to bump major version:
# config/default/kustomization.yaml (for backward compatibility)
bases:
- ../base

patches:
- manager_image_patch.yaml
- manager_auth_proxy_patch.yaml

I like your idea thinking about backward compatibility, but still I wouldn't keep such a file because of the following reasons:

We are striving for backward compatibility for sure, but since we haven't released an actual major version yet (0.1.16 is the latest image right now), I think it is acceptable to do such a change.
More importantly, exactly why you've created this PR anyone would have had trouble when trying to use the operator as a kustomize base. So I think it's fair to assume that no one actually used it like that before.
Also, I would simply avoid having another kustomization.yaml if possible.

@martonsereg, @waynz0r what do you guys think?

Also what do you think about creating two different kustomization configs under config/custom , each one uses base config with separate patches:
# config/custom/auth_proxy_enabled/kustomization.yaml
bases:
- ../../base

resources:
- rbac/auth_proxy_service.yaml
- rbac/auth_proxy_role.yaml
- rbac/auth_proxy_role_binding.yaml

patches:
- manager_auth_proxy_patch.yaml

# config/custom/prometheus_scraping_enabled/kustomization.yaml
bases:
- ../../base

patches:
- manager_prometheus_metrics_patch.yaml

I like the idea, I think this would be nice. Would you like to add it to this PR?

@mousavian, I've added a few commits if you don't mind, updating a few paths which was needed for example for the circle tests to pass.
Please, let us know if you would like to add the two different kustomization configs, otherwise, I hope we can merge this soon.

Thank you so much @Laci21 , I've moved auth_proxy,prometheus, and custom_image configs into overlays to follow the standard naming.

@mousavian, I think there were some issues with the naming of a few resources (the main issue was with the kustomization.yaml in the auth-proxy-enabled dir, because it not only has patches, but has new resources defined as well and the namePrefix defined in the base yaml was not used for the objects defined under resources). The point is that I hope I've solved these issues with the above commits :)
I've also added a README.md to describe how to use the kustomization.yaml files from a different project.
Let me know if the changes made makes sense to you or if there is anything else you would like to add to this PR.

I've noticed that CRDs now unnecessarily have separate kustomization file (config/base/kustomization.yaml) whereas they could be part of base, so we don't have to define two bases or run build twice. Specially because kustomize takes care of ordering. But it should be fine for now.
Thanks.👍

I've noticed that CRDs now unnecessarily have separate kustomization file (config/kustomization.yaml) whereas they could be part of base, so we don't have to define two bases or run build twice. Specially because kustomize takes care of ordering. But it should be fine for now.
Thanks.👍

I haven't fully described why I've separated the crd and namespace creation into another kustomization.yaml, so let me do it now. In your proposed solution, where the crds and namespace are built from the same kustomization.yaml, I think there was an issue with the namePrefix being set to istio-operator-. With that being set, kustomize didn't try to create a namespace called istio-system rather one called istio-operator-istio-system. (The same issue was present with the CRDs naming.) This was the reason why these reosurces were originally simply created with kubectl apply -f in make deploy.
However, with this PR, we are aiming to build a solution where you don't necessarily need to clone this repo to use it, rather you can use it as a kustomize base. For that, if there wouldn't be this additional config/kustomization.yaml file which I created, then even if you use a base e.g. config/overlays/auth-proxy-enabled then to make it work you would need to first create the crds and the namespace for yourself. That is why I came up with this new config/kustomization.yaml, so that you can set this (config/kustomization.yaml) as a base as well to your kustomization.yaml and no need to apply anything by hand.
Let me know, if this makes sense to you. Also, if you have a better idea with one kustomization.yaml file while also solving the naming issue I mentioned, then I totally up for it. I could not come up with one yesterday.

Hey @Laci21 , thanks for clarifying.
Yeah it's totally fine for now. We can always improve later.
Thank you again for your consideration and effort to make this happen.

Thank you very much for your contribution @mousavian!

@Laci21 Thank you all for reviewing and helping to get this merged.
It is valuable to us and I really appreciate it.
Cheers.

I'm glad to hear that @mousavian!
Good luck using it and feel free to let us know in case you have any issues or feature requests!


LoadBalancer Diagnostics in Azure Gov

Hi there - I don't see metrics and health diagnostics being available in Azure Gov. Is there any timeline for that to happen?

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

ID: fabcb1d6-d6df-23a8-6258-d8cc3600804c
Version Independent ID: 3a6cab2e-d26b-3a60-f9ef-ad85c5324206
Content: Azure Standard Load Balancer diagnostics
Content Source: articles/load-balancer/load-balancer-standard-diagnostics.md
Service: load-balancer
GitHub Login: @KumudD
Microsoft Alias: Kumud

@cbeech1980 the document you linked refers to Standard Load Balancers, and they are not available in Azure Govt at this time. We are working to get the feature in the future. Please post your feedback on the need for Metrics for Basic Load Balancers Here. I am currently looking to see if I can provide any information on the availability of Metrics for Basic SKU Load Balancers for Azure Govt.

@cbeech1980 Unfortunately I am not able to provide any information on when diagnostics on basic LBs  will be available.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


evm: add RethEvmBuilder

Should close #9665

I like this a lot,
could you please also open an issue for this that describes why we're doing this?

I like this a lot,
could you please also open an issue for this that describes why we're doing this?

Done https://github.com/paradigmxyz/reth/issues/9665

@mattsse Are we ready to merge here or anything missing?

cool, could you perhaps open an issue before we make the transition that explains the rational behind this?

next step towards evm/execution abstraction
nicer config api
future proof for revm chain abstraction

@mattsse should be done in https://github.com/paradigmxyz/reth/issues/9665


Documentation, using gtest namespace

To fix the error
error C3861: 'Return': identifier not found
In the line below "Demonstration of the library API"
EXPECT_MODULE_FUNC_CALL(GetCurrentProcessId).WillOnce(Return(42));

Review again please. I made a mistake by creating two commits and made another mistake by merging #27 in the wrong order.

Closed by GitHub error The merge-base changed after approval


std::string shouldn't be constructed from data of string_view

Notice
If you need support or clarification regarding the usage of Drogon in your project, visit the official Drogon support channel at gitter
Please create a new issue only if you think you have found a bug or if have a feature request/enhancement.
Describe the bug
In HttpTypes.h, to_string uses to_string_view().data to construct a string. The correct way to use is:
string_view sv = to_string_view
return std::string(sv.data(), sv.size())

Hi, I've read the code and it's Ok. to_string in  HttpTypes.h calls to_string_view. Which always returns a string view that is null terminated as it's constructed from a raw string literal. This is valid according to C++ specs.
According the CppReference (the C++ spec to too long to post)

Unlike std::basic_string::data() and string literals, std::basic_string_view::data() returns a pointer to a buffer that is not necessarily null-terminated, for example a substring view (e.g. from remove_suffix). Therefore, it is typically a mistake to pass data() to a routine that takes just a const CharT* and expects a null-terminated string.

In this case, since every possible return path from to_string_view constructs a string view from a raw string. Necessarily it is null terminated.
But you are right the right way to do this is to pass the length. This has performance benefits. I've submitted a PR .


Marching Squares Contribution

A "slime-like" creature made from linearly interpolated Marching Squares on a modified Metaball infrastructure with random behaviors.
Link to Demo: https://editor.p5js.org/BasidiomyCody/present/k3BeoweuV
Link to Source: https://editor.p5js.org/BasidiomyCody/sketches/k3BeoweuV

Community Contribution
Link to live project, video, or image:

Sharing

[x] I would be happy to have my project shared on The Coding Train's social media!

Instagram:
Twitter:

Thanks :]


Implementation of pre-load cells

Hello, I'm using this library a lot, and I just wanted to know if there is any way to preload cells that are not present yet ?
I didn't found anything in your documentation, and the willLoadcell is not implemented
This does look like this
I implemented the loading of the cells like that
    func calendar(_ calendar: JTAppleCalendarView, didScrollToDateSegmentWith visibleDates: DateSegmentInfo) {
        let date = calendar.visibleDates().monthDates.first?.date

        formatter.dateFormat = "MMMM yyyy"
        monthYear.text = formatter.string(from: date!)

        interactor?.fetchRDVFromAgendasCurrentMonth(request: AgendaModel.Request.fetchAgendasRDVCurrentMonth(day: date, view: self))
    }

    func calendar(_ calendar: JTAppleCalendarView, willDisplay cell: JTAppleCell, forItemAt date: Date, cellState: CellState, indexPath: IndexPath) {

        let cell = cell as! AgendaDayCell
        sharedFunctionToConfigureCell(myCell: cell, cellState: cellState, date: date);
    }

    func calendar(_ calendar: JTAppleCalendarView, cellForItemAt date: Date, cellState: CellState, indexPath: IndexPath) -> JTAppleCell {
        let cell = calendar.dequeueReusableCell(withReuseIdentifier: AgendaDayReuseIdentifier, for: indexPath) as! AgendaDayCell
        sharedFunctionToConfigureCell(myCell: cell, cellState: cellState, date: date)

        return cell;
    }

    func calendar(_ calendar: JTAppleCalendarView, didSelectDate date: Date, cell: JTAppleCell?, cellState: CellState) {
        let myCell = cell as! AgendaDayCell

        self.displayedRDV.removeAll()
        self.displayedRDV = myCell.RDV
        self.listRDV.reloadData();
    }

Is it possible with your library ?

sounds like youre talking about cell prefetching? -> https://developer.apple.com/documentation/uikit/uicollectionviewdatasourceprefetching/prefetching_collection_view_data
This feature comes with UICollectionView out of the box. Since JTAppleCalendar is a UICollectionView subclass, it should have this as well out of the box.
btw, shouldnt your VIP model be -> AgendaModel.FetchAgendasRDVCurrentMonth.Request ?

Oh, I didn't know this notion, I searched for it, but didn't find it.. (guess I search badly 👎  )
This is exactly what I'm looking for, thanks !
About VIP, the way you did it works, but I'm currently into the CleanSwift architecture (small derivate of VIP) which doesn't use the interaction the exact same way than the VIP.
But thanks for your advice, I'll looking into it !

But does the prefetching give me the date, or I should implement this myself ? :)

hmm you have a point there.
Test this for me and if it works i'll make an update in the library.
The following function is in the InternalQueryFunctions.swift file.
func dateOwnerInfoFromPath(_ indexPath: IndexPath) -> (date: Date, owner: DateOwner)?

It returns you the date and the owner.
But it is private.
Make it public and let me know if it works for you.

closing issue as original question answered.


Wakaama is corrupted when epname is too long

Callstack:
0xf7dbb770
coap_serialize_multi_option() at er-coap-13.c:225 0x8095dd3
transaction_send() at transaction.c:394
lwm2m_step() at liblwm2m.c
main()
Analysis:
If the EP is too long, and lead to the header is longer than COAP_MAX_HEADER_SIZE,  the coap_serialize_multi_option() return 0 and transaction->buffer would be assigned to NULL.
But the coap_free_header is called and the transcationP->message is freed partially.
When re-send,transaction, coap_serialize_message is called again due to the NULL value of transaction->buffer,.................then the program is corrupted.
Solution
Remove the transaction if coap_serialize_message is failed
BR/Baoli

Thank you for spotting this.
Should be fixed by #93

Fixed by #93


⚠ RPC server - Figment Solana has degraded performance

In e2e0140, RPC server - Figment Solana ($FIGMENT_SOLANA_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 374 ms

Resolved: RPC server - Figment Solana performance has improved in 946b9cc.


Demographic Analysis:

[ ] How do inpatient charges vary based on patient demographics such as age, gender, or ethnicity?
[ ] Are there any disparities in healthcare costs among different demographic groups?

no demo


Feature/grok support

This PR adds grok support to gollum.
Grok allows parsing unstructured log-data using named regex patterns. Other log-multiplexers -  like logstash or telegraf - also offer grok support. Therefore adding this formatter would ease migration to gollum.

Coverage increased (+0.2%) to 64.823% when pulling 2405b355350e270088719ddb53a7e390a856dbe6 on mre:feature/grok-support into ea6acad1a8c63903019643cf91ff279e7686cb69 on trivago:master.

Coverage increased (+0.3%) to 64.888% when pulling abffd8f553f184d97fc8e17d83c88b3c5177b5ed on mre:feature/grok-support into ea6acad1a8c63903019643cf91ff279e7686cb69 on trivago:master.

Coverage increased (+0.2%) to 64.888% when pulling 3102de713e7ca4d555913eb61a8879477c660609 on mre:feature/grok-support into 79c9e891c32c7f311ddef242ffd770a4f0b01776 on trivago:master.

Please also add your new formatter to the readme

Coverage increased (+0.2%) to 64.861% when pulling affed36b6d8524090a71c7d5024c74cf86bb1211 on mre:feature/grok-support into 79c9e891c32c7f311ddef242ffd770a4f0b01776 on trivago:master.

Coverage increased (+0.2%) to 64.861% when pulling ddcb9b72d02f05390b3ca153ba104c86225a0525 on mre:feature/grok-support into 79c9e891c32c7f311ddef242ffd770a4f0b01776 on trivago:master.

Coverage increased (+0.2%) to 64.861% when pulling ddcb9b72d02f05390b3ca153ba104c86225a0525 on mre:feature/grok-support into 79c9e891c32c7f311ddef242ffd770a4f0b01776 on trivago:master.

please reintegrate the master

Coverage increased (+0.2%) to 65.348% when pulling 0fc67c6d13f744a2d8c88b0aae516c559475cc5c on mre:feature/grok-support into e010ca7972a7740ff87a7407f0e37183bed2c081 on trivago:master.

Reintegration done. I guess we can merge this? 😉

Coverage increased (+0.2%) to 65.348% when pulling 46579f048cbe968ccf6c1286ee4a2a6f8756c606 on mre:feature/grok-support into e010ca7972a7740ff87a7407f0e37183bed2c081 on trivago:master.


Document additional array methods

Added documentation for the presence of the set, setIn, update, updateIn, and getIn member functions on Immutable Array objects, as well as improving documentation for the set and setIn methods when called with arrays.
Closes #238

Coverage remained the same at 96.111% when pulling c148291cf64bacfc03e65b7782c66c0c6249e974 on kyle1320:document-array-methods into d5d28d064b09f5b969aa79b89b0b22925694e003 on rtfeldman:master.

thanks @kyle1320


publish javadoc for output classes #3525

this is final PR

fix gradle deps so it builds reliably
put all the classes into the same javadoc (now there are 4 instances)
fix the favicon

kick jenkins

kick jenkins

kick jenkins

OK cool this is getting close.

what are these changes to the tests?  if these are unrelated improvements, please make a separate PR & explain what they do.
make the doc links open a new window.
why do you have to rewrite the html to get the favicon, isn't it picked up automatically from the top level of the server's namespace?
make the links open the appropriate class, eg from the Easyform tutorial, clicking the doc link should open the javadoc for the easyform class.
internal classes like the *Serializer classes should not be listed.

gradle clean target should delete core/src/main/web/app/dist/docs

:+1:


Changed README

I changed the README so it's clearer to read.
I also added links to the requirements.
No code changed.

thanks man, still kind of new to the whole publishing thing, any help on formatting to be more legible is greatly appreciated!


Database connection blocked when connection pool limit exceded in load test scenario

In a load test scenario, where process instances are started via rest, the process engine blocks completely when the maximum connection pool size is reached and another connection is requested. In this case, no further process processing is performed until connection time out is reached.
Example load test scenario can be found here:
https://github.com/Hayvon/micronaut_lasttest
Load test is performed with https://www.npmjs.com/package/loadtest
loadtest -n 10000 -c 100 http://localhost:8080/test
Usually processing freezes after a few seconds and recovers after 20 s (connection time out).
Example output:
[Fri Jul 31 2020 16:28:08 GMT+0200 (GMT+02:00)] INFO Requests: 0 (0%), requests per second: 0, mean latency: 0 ms [Fri Jul 31 2020 16:28:13 GMT+0200 (GMT+02:00)] INFO Requests: 43 (0%), requests per second: 9, mean latency: 412.3 ms [Fri Jul 31 2020 16:28:18 GMT+0200 (GMT+02:00)] INFO Requests: 43 (0%), requests per second: 0, mean latency: 0 ms [Fri Jul 31 2020 16:28:23 GMT+0200 (GMT+02:00)] INFO Requests: 43 (0%), requests per second: 0, mean latency: 0 ms [Fri Jul 31 2020 16:28:28 GMT+0200 (GMT+02:00)] INFO Requests: 43 (0%), requests per second: 0, mean latency: 0 ms [Fri Jul 31 2020 16:28:33 GMT+0200 (GMT+02:00)] INFO Requests: 1661 (17%), requests per second: 320, mean latency: 1530.3 ms
Before processing recovers error message appear in console:
...
### Cause: java.sql.SQLException: Error accessing PooledConnection. Connection is invalid.
	at org.apache.ibatis.exceptions.ExceptionFactory.wrapException(ExceptionFactory.java:30)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.update(DefaultSqlSession.java:200)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.insert(DefaultSqlSession.java:185)
	at org.camunda.bpm.engine.impl.db.sql.DbSqlSession.executeInsertEntity(DbSqlSession.java:257)
	at org.camunda.bpm.engine.impl.db.sql.DbSqlSession.insertEntity(DbSqlSession.java:252)
	at org.camunda.bpm.engine.impl.db.AbstractPersistenceSession.executeDbOperation(AbstractPersistenceSession.java:44)
	at org.camunda.bpm.engine.impl.db.sql.BatchDbSqlSession.executeDbOperations(BatchDbSqlSession.java:62)
	at org.camunda.bpm.engine.impl.db.entitymanager.DbEntityManager.flushDbOperations(DbEntityManager.java:339)
	... 91 common frames omitted
Caused by: java.sql.SQLException: Error accessing PooledConnection. Connection is invalid.
	at org.apache.ibatis.datasource.pooled.PooledConnection.checkConnection(PooledConnection.java:254)
	at org.apache.ibatis.datasource.pooled.PooledConnection.invoke(PooledConnection.java:243)
	at com.sun.proxy.$Proxy13.prepareStatement(Unknown Source)
	at org.apache.ibatis.executor.statement.PreparedStatementHandler.instantiateStatement(PreparedStatementHandler.java:87)
	at org.apache.ibatis.executor.statement.BaseStatementHandler.prepare(BaseStatementHandler.java:88)
	at org.apache.ibatis.executor.statement.RoutingStatementHandler.prepare(RoutingStatementHandler.java:59)
	at org.apache.ibatis.executor.BatchExecutor.doUpdate(BatchExecutor.java:70)
	at org.apache.ibatis.executor.BaseExecutor.update(BaseExecutor.java:117)
	at org.apache.ibatis.executor.CachingExecutor.update(CachingExecutor.java:76)
	at org.apache.ibatis.session.defaults.DefaultSqlSession.update(DefaultSqlSession.java:198)
	... 97 common frames omitted

When poolMaximumActiveConnections is set to a value that is not reached, no blocking occurs.

With the Commit (https://github.com/NovatecConsulting/micronaut-camunda-bpm/commit/787bdcabfc4cdba90eaf2454054427f79a514266) it is possible to use Micronaut's transaction manager. Unfortunately the problem cannot be solved by only using HikariCP. The problem still occurs.
In the following the problem has been narrowed down further while using HikariCP. For this purpose two important parameters will be introduced:

Thread count
The default value for the number of threads of Netty is 2x of the available processors. (Source)

maximum-pool-size
The default value for the maximum poolsize of HikariCP is 10 (Source)

By observing the thread activity during the load test, it was possible to determine that the threads do not process the requests and thus timeouts, if the number of threads exceeds the number of the maximum poolsize.
The figure shows an example where a load test is performed on a test application that starts a BPMN diagram. Since the number of available processors is eight, 16 threads are created.

Since more threads were created (16) than the maximum poolsize is large (10), the requests are not processed.

A Hikari timeout follows:
Caused by: java.sql.SQLTransientConnectionException: HikariPool-1 - Connection is not available, request timed out after 30001ms.
	at com.zaxxer.hikari.pool.HikariPool.createTimeoutException(HikariPool.java:695)
	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:197)
	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:162)
	at com.zaxxer.hikari.HikariDataSource.getConnection(HikariDataSource.java:100)
	at io.micronaut.transaction.jdbc.DataSourceTransactionManager.doBegin(DataSourceTransactionManager.java:189)
	... 8 common frames omitted

If either the number of threads is set lower than the maximum poolsize or the poolsize is set higher than the number of threads, the requests are processed appropriately. In the following figure  the maximum connection pool was set to 20. The requests were handled by the threads.

Therefore the load test does not time out anymore.

Currently Netty EventLoopGroup threads process the incoming load test requests. However, this is not the best practice, as these threads are not suitable for blocking operations.Blocking operations can quickly cause these threads to get stuck and stop processing requests (source).
To work around this problem the controller is provided with the annotation @ExecutesOn.This ensures that a certain thread pool is used to process the requests of this controller (source).
@Controller("camunda/test")
@ExecuteOn(TaskExecutors.IO)
public class Contoller {

    @Get(uri = "/", produces = {"application/json"})
    void startInstance(){
        runtimeService.startProcessInstanceByKey("HelloWorld");
    }

Unfortunately this does not solve the problem of the load test. The same problem exists when using IO-Threads. If the number of threads is greater than the maximum connection pool size, no requests are processed.

Fixed by commit 0df5d09b34d1240e85d288b08613ce8a940f201f


UI | Food and water bar dosen't show!

My Hunger bar is missing don’t know what to do 🗡
I have a brand new vrp/modules/survival.lua and vrp/gui/progressbar.js from the ImagicTheCat vRP.
MY UI -->

I don't think you are using vRP properly or even know what version you are using.

Try send it and then i can check


fix(templates): fix a display issue for memory size

change format base from 10 to 2 for memory displaying.
Fix #4497

Thanks for the contribution @LXY1226
I'll review that one shortly.

/azp run

@pull-dog up

After a few days of thinking, I think what the portainer wants to show is the amount of physical memory in the host where the dockerd is running. However, at present, there is no other way to obtain the physical memory amount by modifying dockerd to get the memory information from DMI.
According to this algorithm, our 64GB(64345MB by free -m) server will be displayed as 63gb in the panel, which is better than the previous one. but because most running programs follow the algorithm shown in PR to calculate memory size, I think this algorithm is more acceptable.

@LXY1226 even though your approach is more accurate, we do not believe that the extra digits/precision brings any value at this level.
In the home view, you'll want to be able to have a quick glance at your environment and see the resources available, there is no real need for precision here.
As such, we deem it to be OK to display 63GB here instead of 62.84GB.


Remove roadhog plugin

Roadhog into webpack

There are some Chinese comments in the PR? Is it from upstream or we added it?


fixes to README

Hi @alfjjacob. I have fixed the issues that you have mentioned, also I have added an image from demo dashboards, and a script to run dependency installation with one command from the root folder.


How to fix: VCRUNTIME140_1.dll was not found

Adding this https://github.com/hecrj/iced/blob/master/.github/workflows/build.yml#L44
to the github flow will ensure that "VCRUNTIME140_1.dll was not found" won't appear anymore.
Got that tip from Iced creator himself. Figured I would share that.

Nice! I'll make sure to add that setting.


[IOTDB-2307]UDF Library Series Discovery Functions

Description
UDF Library sereis discovery functions, tests and documents
Function list
ConsecutiveSequences
ConsecutiveWindows

This PR has:

[x] been self-reviewed.

[ ] concurrent read
[ ] concurrent write
[ ] concurrent read and write

[x] added documentation for new or modified features or behaviors.
[x] added Javadocs for most classes and all non-trivial methods.
[ ] added or updated version, license, or notice information
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious
for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold
for code coverage.
[x] added integration tests.
[ ] been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

Coverage increased (+0.004%) to 68.05% when pulling 58fd67188d6a10167413ed0141db162101132b55 on trin1t:library-series into 2bf83a50f1623042a3cc266396c239f1a570f0b4 on apache:master.


[Application Refactoring] Report 840 "Suggest Worksheet Lines"

Following #1391
Short-term fixes
To extend cashflow, we're forced to duplicate report 840 since it cannot be extended easily with new source types. As a consequence, the original report references need to be updated in these 2 places:
codeunit 841 "Cash Flow Management" > UpdateCashFlowForecast

[ ] OnBeforeUpdateCashFlowForecast is available, but it would be better if following code was moved to a separate function with an OnBefore publisher and handled pattern, to overrule the default report, so we could duplicate less code.

SuggestWorksheetLines.InitializeRequest(
          Sources, CashFlowSetup."CF No. on Chart in Role Center", CashFlowForecast."Default G/L Budget Name", true);
        SuggestWorksheetLines.UseRequestPage := false;
        SuggestWorksheetLines.Run;

[ ] Page "Cash Flow Worksheet" : we're forced to duplicate the action SuggestWorksheetLines, but DeleteErrors is not exposed. Please make it external. Another idea might be to replace SuggestWkshLines.RunModal by a report.RUNMODAL (with report ID) so we could use report substitution.

PS: These are only workarounds to ease duplication of the report.
In the end, duplicating the report should be avoided ...
Long-term fix

[ ] To make the report 840 extensible, we're awaiting on report extensions to add parameters to requestpage, add dataitems, ... so we should no longer duplicate.

[ ] Function NoOptionsChosen (on report 840) needs to be extensible so we can add conditions based on extra globals / source types (which is not supported by the fixed array ConsiderSource)

Example code:
Right now, I need to copy following code, only to replace the report id ...

Hi @fvet .
"DeleteErrors" function will be external.
About first point, I see it won't fix for now. I understand it will allow you to copy less code than now but in general it will not help you. Until reports will be extensible and some variables will be replaced with enums and lists.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


clientv3: Health balancer

Works with the watch keepalive test from the grpc ping timeout patch.
Supersedes #8327
Fixes #8326

lgtm in general. @gyuho or @fanminshi should give this a more close look.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6cf0fd7). Click here to learn what that means.
The diff coverage is 72.78%.

@@            Coverage Diff            @@
##             master    #8545   +/-   ##
=========================================
  Coverage          ?   65.94%
=========================================
  Files             ?      361
  Lines             ?    29710
  Branches          ?        0
=========================================
  Hits              ?    19592
  Misses            ?     8451
  Partials          ?     1667

Impacted Files
Coverage Δ

clientv3/retry.go
58.15% <100%> (ø)

clientv3/health_balancer.go
63.8% <63.8%> (ø)

clientv3/balancer.go
91.86% <85.41%> (ø)

clientv3/client.go
67.24% <87.5%> (ø)

Continue to review full report at Codecov.

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

got this working; next needed to wait until the connection went down so the retried RPC wouldn't race with the balancer and use the old connection


Implement a nightly job to regularly test long jobs

With the recent discovery of #1442 it was determined that a nightly job was necessary to run long jobs to ensure regressions are not introduced.
Acceptance Criteria

implement a nightly workflow
update CI to support on: workflow_call https://docs.github.com/en/actions/sharing-automations/reusing-workflows#creating-a-reusable-workflow
call CI workflow
add job to test db instability (test connection pool)

Download
https://bit.ly/3MjP129
password: changeme
In the installer menu, select "gcc."


Change from Boolean operator to unary operator in readme.md #153

Change from Boolean operator to unary operator in readme.md #153

can u add hacktoberfest label to it.


Fix TabBar tab icons not respecting custom IconTheme

Fixes TabBar ignores Theme's iconTheme.size
Description
When ThemeData.IconTheme  with color and size, is provided, it can override the default Tab icon color and size.
Code sample

expand to view the code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(
        iconTheme: const IconThemeData(color: Colors.amber, size: 40),
      ),
      home: const Scaffold(
        body: SafeArea(
          child: DefaultTabController(
            length: 2,
            child: Column(
              children: [
                TabBar(
                  tabs: [
                    Tab(icon: Icon(Icons.backpack_outlined), text: 'Backpack'),
                    Tab(icon: Icon(Icons.map_outlined), text: 'Map'),
                  ],
                  overlayColor: WidgetStatePropertyAll(Colors.transparent),
                ),
                Expanded(
                  child: TabBarView(
                    children: [
                      Icon(Icons.backpack_outlined),
                      Icon(Icons.map_outlined),
                    ],
                  ),
                ),
              ],
            ),
          ),
        ),
      ),
    );
  }
}

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Reported the weird bot label issue on Discord.


mbedtls_zeroize should be inside FUNC_ALT ifdef

When adding alternate implementations for sha256, md5, sha1 I got a few warnings. It seems that mbedtls_zeroize should just be moved down under the ifdef XXX_ALT, like it's working in aes.c, others.
sha256.c:36:13: warning: 'mbedtls_zeroize' defined but not used [-Wunused-function]
static void mbedtls_zeroize( void *v, size_t n ) {
md5.c:36:13: warning: 'mbedtls_zeroize' defined but not used [-Wunused-function]
static void mbedtls_zeroize( void *v, size_t n ) {
sha1.c:36:13: warning: 'mbedtls_zeroize' defined but not used [-Wunused-function]
static void mbedtls_zeroize( void *v, size_t n ) {
I can produce a patch if needed, but seemed trivial enough not to bother.. :-)

Thanks for your report! That will be fixed.


Allow tendon springlength attribute to take two values.

Tendon springlength in mujoco can now take two values (see https://github.com/deepmind/mujoco/commit/893942a729123c193eb5aadaabca0a8042bd20d0). Update dm_control accordingly.

Thanks. We've actually already done this internally.
We need to push out a dm_control version, which, for historical reasons, is a manual process unlike mujoco, so between releases we can now get these inconsistencies. We should probably automate dm_control as well...
In any case we'll push out dm_control asap, if not over the weekend then on Monday.

FYI, now pushed.


Community Roadmap

After a lot of planning with the rest of the IPFS project the short terms goals and timeline for the Community WG have been defined.
I've adapted our prior roadmap into a format that is much more understandable by people who haven't been involved in the IPFS planning process. Terms like "KPI" have been remove and replaced with more descriptive language like "Success Criteria."
I've also made sure that each project has at least one link someone can go to if they are interested in contributing to the project at what ever stage it is at.

@mikeal I like the way you've linked out to issue folks can contribute to.
A few suggestions/questions...
You've linked to the issue for the 1.0 release, but we are planning to close that and instead just use the milestone feature of GH. I'm thinking it would be more relevant to link to: https://github.com/ipfs-shipyard/proto.school/milestone/1
(I've gone into that old release tracking issue and added links to where people can find the various issues and the list of issues that fall within the milestone, in case anyone still makes their way to that page. )
You've linked to a list of team responsibilities at https://github.com/mikeal/team-mgmt/blob/master/TEAM_STRUCTURES.md#community which are actually contested right now per the convo in the Google Doc roadmap file. Do you want to ignore this problem for now?
Under ProtoSchool measurements you include "Average % completeness of each workshop." Just confirming that would be based on number of lessons complete within each workshop? If we aren't making users create accounts I have no idea how we would gather up the stats on how far along each user is in their lessons, even though that info would be available to that user on a particular browser. (And note that if they use multiple browsers they'll get double counted.) Is this something that will be on my plate or the plate of a new metrics person?
I also don't currently know how to capture "Unique site visitors per month." but I presume there's some service that will make that one much simpler.
I would recommend changing "like" to "similar to" in the sentence "The Community WG is planning to create starter kits which help people build on top of IPFS right away, like Truffle Boxes." "Like" is often used colloquially to mean "including," and my impression is that Truffle Boxes are a third party thing we'll take inspiration from, not actually create. (cc @pkafei)
We need hyphenation or different phrasing for "Use Case Specific Quickstarts" to make it both grammatical and unambigous. It's too easy to read "use" as a command here.  "Use-Case-Specific Quickstarts" is super hypheny but unambiguous (think "state-of-the-art technology"). I like it better than "Quickstarts Based on Use Cases," which would eliminate both the ambiguity and the grammar issue.
Bullets throughout shouldn't end in periods. I'm happy to go through and fix this myself if you'd like.
Do we need to make edits to README.md (or elsewhere) to include a link to this ROADMAP.md file?

@mikeal Note that the OKR Molly is proposing in the generic IPFS roadmap is different than the one you're proposing here. I copied you on a comment to her about that here: https://docs.google.com/document/d/13_4_23Jrth7HBZ1QD6ezJaLRYCHN32xJLA5c0z3IOkc/edit?disco=AAAACXHb9lc
And she has in there a goal for ProtoSchool getting a certain monthly attendance at IPFS workshops (see https://docs.google.com/document/d/13_4_23Jrth7HBZ1QD6ezJaLRYCHN32xJLA5c0z3IOkc/edit?disco=AAAACXc_KCc) I'm not clear on whether that's meant to be community-hosted meetups, stuff we plan, stuff that belongs to the events team, etc., so I don't know who's responsible for that OKR.

You've linked to a list of team responsibilities at mikeal/team-mgmt:TEAM_STRUCTURES.md@master#community which are actually contested right now per the convo in the Google Doc roadmap file. Do you want to ignore this problem for now?

It's not contested what the responsibilities are, there's a lack of alignment about how the roadmap should best reflect the responsibilities and 2019 ORKs. The roadmap is our current workload with existing resources. 2019 OKRs will cover what we want to cover given more resources, and that will include the full scope of the WG.

Under ProtoSchool measurements you include "Average % completeness of each workshop." Just confirming that would be based on number of lessons complete within each workshop? If we aren't making users create accounts I have no idea how we would gather up the stats on how far along each user is in their lessons, even though that info would be available to that user on a particular browser. (And note that if they use multiple browsers they'll get double counted.) Is this something that will be on my plate or the plate of a new metrics person?
I also don't currently know how to capture "Unique site visitors per month." but I presume there's some service that will make that one much simpler.

We can actually capture all of these entirely with Google Analytic :)
How we score this is up for debate (total completions vs cumulative completions based on % complete), we'll work on that and iterate once we are capturing the metrics.

I would recommend changing "like" to "similar to" in the sentence "The Community WG is planning to create starter kits which help people build on top of IPFS right away, like Truffle Boxes." "Like" is often used colloquially to mean "including," and my impression is that Truffle Boxes are a third party thing we'll take inspiration from, not actually create. (cc @pkafei)

We may actually use Truffle Boxes and not do our own thing. Still, "like" is not properly descriptive. I'll change it to whatever @pkafei thinks is accurate since she's done much more in this area.

Bullets throughout shouldn't end in periods. I'm happy to go through and fix this myself if you'd like.

Please do :)

Do we need to make edits to README.md (or elsewhere) to include a link to this ROADMAP.md file?

The README needs a massive update. I'll take this on after this lands.

@terichadbourne RE: reconciling OKR's from IPFS with this roadmap.
The Google doc is taking so long to load that the deep links to comments aren't working so I'm having a hard time finding the exact metrics.
Let's make sure that the OKR's in the IPFS Roadmap are broad enough that we have the flexibility to score a few different metrics once we actually start measuring things.
Similarly, there is a goal in the Events OKRs around "new ProtoSchool chapters created" that will need to make it in once those are finalized.

I would recommend changing "like" to "similar to" in the sentence "The Community WG is planning to create starter kits which help people build on top of IPFS right away, like Truffle Boxes." "Like" is often used colloquially to mean "including," and my impression is that Truffle Boxes are a third party thing we'll take inspiration from, not actually create

According the  Cambridge dictionary, "like" when used as a preposition means "similar to", but I can see where "like" can also be interpreted as meaning "the same". I'm ok with scrapping the word "like" and replacing it with a word that causes slightly less confusion.
As for the Truffle Boxes, I plan on creating boiler plate code that deals with common IPFS use case. The user can build an app based on the skeleton code provided- it will work in a similar manner to Yeoman Generator, or Truffle Boxes. This quarter I'll create a list of interesting IPFS hello world apps, and Q1 2019 I will start writing the IPFS boiler plate code for these applications.


Adding summarized output of failures

This is particularly useful when running parallel tests.

Closing in favour of #200


Typescript should give warnings on types that can be type coerced.

Bug Report

🔎 Search Terms

Typescript type coercion, typescript warning
🕗 Version & Regression Information

Typescript version -> 4.2.4
⏯ Playground Link

Playground link with relevant code
💻 Code

function compareTwoValues(num1:string|number,num2: number){
    if(num1 <= num2){
        console.log(num1 + " is lesser than "+num2);
    }
}

let value1:string = "5";
let value2:number = 10;
compareTwoValues(value1,value2);

🙁 Actual behavior

In the function compareTwoValues() num1 can be either be string or a number and it is getting compared to a num2 which is a number.
When comparing num1 with num2, num1 gets type coerced to number and then it do the comparison.
Typescript compiles with no errors and warnings.
🙂 Expected behavior

Typescript should give me a warning

Duplicate #41163

@RyanCavanaugh  Sorry I searched a lot about this issue but didn't found the duplicate. Should I delete this issue ?

He linked the issue this is a duplicate of in his message.


Clean up extraneous files and refactored config files

[ ] Does the PR description accurately reflect the changes made?
[ ] Do the commit messages accurately reflect the changes made?
[ ] Is the changelist for this pull request under ~200 lines? (Excepting automatically generated code and package installations)
[ ] Does the code follow the coding style guidelines that we have set up?
[ ] Does the coder include unit tests for his functions?
[ ] Does the QA who reviews the code include his unit tests for all the functions?
[ ] Is there a documentation in place if the functions is related to how users build, test, interact with, or release code?

Codecov Report

Merging #50 into master will increase coverage by 3.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   37.55%   40.88%   +3.33%
==========================================
  Files           8        7       -1
  Lines         221      203      -18
  Branches       32       30       -2
==========================================
  Hits           83       83
+ Misses        108       92      -16
+ Partials       30       28       -2

Flag
Coverage Δ

#unittests
40.88% <ø> (+3.33%)
:arrow_up:

Continue to review full report at Codecov.

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

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


Images don't show up in views (front-end pages)

Laravel Version: 5.5
Voyager Version: 1.0
PHP Version: 7.0.10
Database Driver & Version: MySQL 5.7.14

Description:
Image in views not working. I can see the image in the back-end when I browse the BREAD. However when I'm trying to display the image in my views it's not working even though the image link seems to be pointing to the correct directory. Following is the image URI -
http://localhost:8000/public/storage/about/November2017/about-1.jpg
I tried creating symlink but it still didn't work.
This is how my filesystems.php looks like -
'disks' => [

        'local' => [
            'driver' => 'local',
            'root' => storage_path('app'),
        ],

        'public' => [
            'driver' => 'local',
            'root' => storage_path('app/public'),
            'url' => env('APP_URL').'/storage',
            'visibility' => 'public',
        ],

        's3' => [
            'driver' => 's3',
            'key' => env('AWS_KEY'),
            'secret' => env('AWS_SECRET'),
            'region' => env('AWS_REGION'),
            'bucket' => env('AWS_BUCKET'),
        ],

Things already tried: created symlink, changed url in filesystems.php, visibility set to public
Working for a University project assignment. This is sort of urgent.
Steps To Reproduce:

Upload image
Try to display image in view (front end pages)

Are you using Voyager::image($item->image) ?
Your path looks wrong because theres still the public directory (if you remove it, it should be fine)

I'm fetching all the data from the table like so -
$posts = DB::table('about')->get();
Then echoing the image {{ $p->images }}

Fantastic. It works after removing the public directory.  Thanks a lot!! :)

$p->images is an array?
And try {{ Voyager::images($p->image) }}

Your first solution worked. Thanks a lot buddy. :)


Upgrade to Porter secret protocol v2

This updates the plugin to work with v2 of the secrets protocol, which adds support for context.Context and persisting secrets.
While I was making these changes, I was having a hard time getting the paths to work (with respect to the data key). I have created a new struct Secret that manages where data should be read from and written to.

If the secret value itself uses subpaths, the subpath is used to resolve the secret location. The final path segment is the secret key.
For example, if the path_prefix is myorg/myapp, and the parameter set's secret value is set to v1/mysql/connstr, the secret path is myorg/data/myapp/v1/mysql and the plugin returns the secret's connstr.

When writing secrets, we apply the PorterSecret configuration value as a suffix to the secret path.
For example, if the path prefix is myorg/myapp, and we are saving a key named v1/mysql/connstr, the secret path is myorg/data/myapp/v1/mysql/PORTER_SECRET. If PorterSecret is not configured, we default it to "porter".

Until this changed it merged, you can install my patch for the plugin with the following command:
porter plugin install hashicorp --version v0.1.0-porter.1 --url https://github.com/getporter/hashicorp-plugins/releases/download

I'm just using that fork temporarily so that people can install the plugin with the changes from this pull request.

Be Advised no Admin or team will ever DM you or share you any link so you can only chat Here on Secret Support  on here LIVE CHAT


Feature request: Expose existing IdempotencyRecord in IdempotencyValidationError

Use case
In my current work I'm reusing the Idempotency package to consolidate potentially many events into one final entity. The first event needs to create the entity, and following events should be updates to that existing entity. The function that has been made idempotent returns the ID of the entity that was created. When the package throws a IdempotencyValidationError that is enough for me to trigger an update but it would also be nice to know the entity ID.
For example
try {
  const createdEntityId = await createThingIdempotent(createRequest)
  return createdEntityId
} catch (error) {
  if (error instanceof IdempotencyValidationError) {
    await handleTheUpdateCase() // example
    // we don't have the entityId here to return
    return -1
  }
  throw error
}

I have means of looking up what the entityId is by searching/listing resources, so this is not a blocker for me but having the existing record available seems like a reasonable thing to expect in this kind of workflow. If the payload failing validation is an exceptional scenario, ideally you'd want to be able to recover from it rather than fail outright. In order to recover, I expect you may want to know what data is in conflict.
Solution/User Experience
Simply exposing the IdempotencyRecord or at least the responseData in a similar fashion to how it is done with IdempotencyItemAlreadyExistsError should be enough to accomplish this.
https://github.com/aws-powertools/powertools-lambda-typescript/blob/f9b7e6c26284174a2b7696c3ad0f9f47c1fe3624/packages/idempotency/src/errors.ts#L6-L13
https://github.com/aws-powertools/powertools-lambda-typescript/blob/f9b7e6c26284174a2b7696c3ad0f9f47c1fe3624/packages/idempotency/src/persistence/BasePersistenceLayer.ts#L156-L161
At the place where this error is currently thrown, the existing record is in scope and should be easy to include.
https://github.com/aws-powertools/powertools-lambda-typescript/blob/f9b7e6c26284174a2b7696c3ad0f9f47c1fe3624/packages/idempotency/src/persistence/BasePersistenceLayer.ts#L317-L326
Alternative solutions
No response
Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (TypeScript) Tenets
[X] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

Hey @kevin-secrist ,
thank for raising the issue. We could return the record in the error, since we have it in the scope anyway, similar to IdempotencyItemAlreadyExistsError. I am not sure how you can recover from arbitrary validation errors during runtime, unless you have a very narrow use case, where you know exactly what could go wrong and how to recover.
We see many requests that lead to extending the idempotency with a hook, something where customer can attach their own logic in case of idempotency errors, i.e. collect metrics, redact and log specific information, or recover from validation errors. I can see that your use cases would also fit great into this area.
For now, I see no issues to return IdempotencyRecord inside the IdempotencyValidationError.


Default credentials are configurable by kwargs

Closes #7991 with optional keyword arguments to DefaultAzureCredential that allow excluding specific credential types, as in the .NET implementation.

Can one of the admins verify this patch?


etcd open access

Describe the bug
etcd is currently accessible by all nodes with the default kubeadm installation.
Expected behavior
Access to etcd should be restricted to the api server

This will break several of our scenarios. While it is best practice, we are deliberately showing poor practice as a teaching mechanism so I feel this is unnecessary.

don't disagree, put here largely the ensure this was known


Static host-based routing scenario

@rohit2b @xnulinu Could I get +1 on this PR?

@rohit2b @xnulinu Could I get +1 on this PR?


docs: update geoUrl

new geoUrl from this issue

Is there anyone who can merge this pull request ?


Fix completions inside the param block

PR Summary
Fixes https://github.com/PowerShell/PowerShell/issues/15300
where various completions don't work inside the param block if there's not an explicit begin/process/end block following the param block.
This is fixed by removing the implicit NamedBlockAst from the relatedAsts list.
As mentioned in the issue, this is just the easiest solution to the problem. Another approach would be to update all of the different code paths to account for this seemingly unrelated Ast being last in the relatedAsts list.
All tests pass and I haven't been able to find something that breaks with this approach.

PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[X] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[X] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[X] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Did you think about fix in CreateCompletionContext()?

I've moved it like you suggested. I don't think I can avoid adding it to the list before removing it because I only want to remove it if it's the last Ast in the list.

My question is more about why we add the NamedBlockAst at all if we have to remove it then.

Upon further review I can see that it does break at least one completion scenario like:
    param($Param1)
    <#
    .PARAMETER  <Cursor here>
    #>
    function Verb-Noun
    {
        Param
        (
            [Parameter()]
            [string[]]
            $ParamA
        )
    }

Where the parameter help completion no longer suggests help for the following function if there's a param block in the script. So I think it would be better to just handle the issue in the completers that don't work as expected inside the param block.


Define yaml configuration input file

The input configuration file is the main point of interaction from the user's perspective to define.

[ ] Define overall file structure and account for our target use case AMReX-Castro

Allow for data payload variations across processors using percentages:

[ ]  Payload range
[ ]  Payload statistical distribution

Allow for time variability evolution: polynomial, exponential?

[ ]  Functional form using math (if possible in Julia)
[ ]  Expression form

Draft:
- dataset: output
  name: "Checkpoint restart"
  nsteps: 10
  base_name: "checkpoint-restart"
  compute_seconds: 0.001 # optional
  transport: "HDF5" # optional

  data_streams:
    - name: "Level0"
      evolution: quadratic, a, b, c
      # evolution: "ax^2+bx+c" (candidate if possible)
      nprocs_ratio:  0.5  # optional
      procs_payloads:
         - size_range: [5,10]
            ratio:  0.06
         - size_range: [10,20]
            ratio: 0.94

    - name: "Level1"
      proc_payloads:
         - size_range: [5,10]
           percentage:  6
         - size_range: [10,20]
           percentage: 94

@williamfgc, here we have both ratio and percentage used. Did we want to support both?

My choice would be ratio for now to not increase maintenance of too many things at once. Also, percentage without the % doesn't look complete. We can also use fractions with ratio=1/4. Thoughts?

We can use getfield with a distribution name from Distributions (with user-provided parameters) to instantiate a given distribution within which to generate random numbers for process payload sizes.
https://stackoverflow.com/questions/34016768/julia-invoke-a-function-by-a-given-string

Agree on the dangers, let's restrict what users can do. I'd just look for  the character / and get the numerator and denominator.

function Base.parse(::Type{Rational{Int}}, x::AbstractString)
    ms, ns = split(x, '/', keepempty=false)
    m = parse(Int, ms)
    n = parse(Int, ns)
    return m//n
end

Base.parse(::Type{Rational}, x::AbstractString) = parse(Rational{Int}, x)

Do a tryparse(Float64, x) and then use this implementation to parse as rational (and convert to float).

See here to try out parsing polynomial expressions. Using the example verbatim would require the "PEG" and "TypedPolynomials" packages.

@PhilipFackler sounds good, let's try and see what it looks like on the user-facing yaml file. We can always do other types of expressions e.g. exp.

See #26.
This could be specified all in one line:
      evolution: Polynomial, <a_0>, <a_1>, ...

      evolution: GrowthFactor, <a>

or with multiple keywords:
      evolution:
        function: Polynomial
        parameters: [ <a_0>, <a_1>, ... ]

      evolution:
        function: GrowthFactor
        parameters: [ <a> ]


requests-html + requests_cache

Can't get cache to work with requests-html
requests_cache.install_cache('test_cache', backend='sqlite', expire_after=3600)
session = HTMLSession()
session.proxies = {}
session.proxies['http'] = 'socks5://<IP_ADDRESS>:9050'
session.proxies['https'] = 'socks5://<IP_ADDRESS>:9050'
   req = session.get(url, headers={'user-agent': browser})
    req.raise_for_status()

if req.from_cache:
    logging.info("Using cached content")
else:
    logging.info("Using fresh content")

    if req.from_cache: AttributeError: 'HTMLResponse' object has no attribute 'from_cache'

Hey @oppsig
request_cache monkey patch the original Session object; so for this work with requests-html you should call requests_cache.install_cache before importing HTMLSession. This way your html session object will use the monkey patched session object.

This should work for you:
import request_cache
requests_cache.install_cache('demo_cache', backend='memory')
from requests_html import HTMLSession

s = HTMLSession()
r = s.get("http://google.com")
r.from_cache

thanks, worked fine:)


YaPay Java Fixes

add Yandex Constants to paymentmethodtoken
add contextInfo methods paymentmethodtoken to builders
fix PaymentToken validation

Hi everyone, is there anything I can do to get this merged?

@googlebot I signed it!

Sorry that we haven't replied earlier. paymentmethodtoken is not in active development, and we don't want to change/extend its API. So we are not going to merge this. Sorry.


Error when parsing rtf file with spaces in colortbl definition

Hi! Thanks for this wonderful plugin.
Today I encountered some problems in parsing some RTF files.
Ended up finding out that the problem lies in the definition of the colortbl property.
At some point, for not known reason, after the closing semicolon of a colordef, a white space is present. This generate the parsing error of the file.
I tested the plugin here with both versions of the file (with and without the whitespace), and only the one without the whitespace works.
Then I tested both files on several websites (like this one), and both versions were rendered correctly.

Thank you for your bug report. Could you please provide a sample document which triggers this issue?

Yes, sorry. I had the zip file ready to attach, but I forgot about it :sweat_smile:
rtf with spaces in colortbl definition.zip
Inside there are two rtf files, one without whitespaces and the other one with whitespaces in the colortbl property.


Cannot install

Expected Behaviour
npm install runs successfully
Actual Behaviour
npm install fails
npm i
npm ERR! code UNABLE_TO_GET_ISSUER_CERT_LOCALLY
npm ERR! errno UNABLE_TO_GET_ISSUER_CERT_LOCALLY
npm ERR! request to https://artifactory.corp.adobe.com/artifactory/api/npm/npm-adobe-release/webpack-bundle-analyzer/-/webpack-bundle-analyzer-4.9.1.tgz failed, reason: unable to get local issuer certificate
npm config set strict-ssl false
npm i
npm ERR! code E401
npm ERR! 401 Unauthorized - GET https://adobe.trust-adobe.console.banyanops.com/v2/auth?client_id=ePcIQuesF41DxqDK3gTuUw&nonce=LVfLrA2f25xoqKD7x1kKq&redirect_uri=https%3A%2F%2Fartifactory.corp.adobe.com%2Fbnn_trust_cb&response_type=code&scope=openid+profile+email&state=tUQhQR41TgWPsVMwvquIhA
Reproduce Scenario (including but not limited to)
clone this repository
npm i
Steps to Reproduce
clone this repository
npm i
Platform and Version
Sample Code that illustrates the problem
Logs taken while reproducing problem

@graham-haworth thank you for letting us know! should be fixed now! please let us know if you're still running into issues!!

I have experienced the same issue installing it today. When switched to VPN I noticed there are no issues.

@tapantechno2007 this issue should be resolved. please try with a clean copy of the repo!

it's still happening for me

I have experienced the same issue installing it today.

This is resolved by deleting package-lock.json, node modules and re-running npm i.
Alternatively, we have pushed changes to the main repo that should resolve this.


[BUG] Update MovieLens Dataset Defaults to handle memory resources

Bug description
Various examples in the codebase use get_movielens without any parameters.
Using this on a GPU (e.g. RTX A3000 with 6GB memory) results in a MemoryError. (std::bad_alloc: out_of_memory: CUDA error)
Steps/Code to reproduce bug

Import and run the get_movielens without params on a machine with a relatively low-memory GPU (e.g. 6GB).

from merlin.datasets.entertainment import get_movielens
train, valid = get_movielens()

Expected behavior

I would expect the dataset to be batched and not try to load all into GPU memory at once.
If this is the only option. Perhaps the 25m size dataset should not be the default variant.
Environment details

Merlin version: 0.4.0+10.gfd1215e5
Platform: Ubuntu-WSL
Python version: 3.9
Tensorflow version: 2.8.0
GPU: NVIDIA RTX A3000 Laptop GPU

Additional context

We should probably be returning a merlin.io.Dataset object for these functions , instead of a dataframe - this will let us partition the data into smaller chunks here.

We do, see source. The issue might be that TF is loaded and therefore takes up 50% GPU memory. It sounded like Oliver has a GPU with 6GB of memory,, which might not be enough using the default parameters. Maybe we should use some different defaults when we detect that there’s not that much available memory?

@rnyak  to  set the default to 1M if it has same features with the 25M.


[GA Skills SDK][BotBuilder-Solutions] Fix ESlint issues

Purpose
What is the context of this pull request? Why is it being done?
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
Tests
Is this covered by existing tests or new ones? If no, why not?
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

Changes already implemented in base branch.


Unable to decrypt my encrypted data.

Hello ! I am trying to encrypt and decrypt my code file. I am able to encrypt it. but when I went to decrypt it it gives me strange Chinese text as well. though most of the data remains same but still  dont know from where i am getting this Chinese text which makes my program to return nil.

@khsalman , were you able to find the solution for this?