Consider client type renames

Rename client::Service -> client::Connection. This is the actual, established, connection. It is backed by a single TcpStream and implements Service. Another option is to name this Client.
Then, the simple case to establish a connection would be: Connection::connect(&addr) (or Client::connect(&addr).
client::Client -> client::Connect. this implements tokio_connect::Connect` and is essentially the connection factory.

And completely remove any type named Client?

@seanmonstar for now, yes. I think a Client type would be best reserved for something that establishes a configured connection with a socket address or host name (dns resolution)
Basically, reserve Client for later.

I agree with the rename.
I have some thoughts on things we might want in a proper Client interface.  Something like finagle's StackClient may not be particularly well-suited for Rust; but we may find some nice middle-ground with macros, etc...  Finagle started with a ClientBuilder type, but this type became overly complex and not generic across protocols.  Eventually, we mostly replaced the builder types with Stack-based types that support:

a generic type-map for configuration

a list of, effectively:

trait Module {
  type Request;
  type Response;
  type Error;
  type NewService: NewService<Request = Self::Request, Response = Self::Response, Error = Self::Error>;

  fn make<N>(params: Params, next: N) -> Self::NewService
      where N: Module<Request = Self::Request, Response = Self::Response, Error = Self::Error>;
}

i know this is nowhere near compiling, just illustrative...
Anyway, even if we don't end up with stacks, clients should try to be generic over discovery, stream classification (success/failures), retry policy, load balancing, sharding, etc etc

This issue has been resolved by #4.
There is also #6, which adds Connection::handshake. To add a connect fn would require depending on tokio-core, which I would like to punt for now.


remove the unpin epoch logs or move to Grafana metrics

Quite verbose:
2023-02-13T08:42:20.095855Z  INFO risingwave_frontend::scheduler::hummock_snapshot_manager: Unpin epoch<PHONE_NUMBER>628352 with RPC
2023-02-13T08:42:40.094707Z  INFO risingwave_frontend::scheduler::hummock_snapshot_manager: Unpin epoch<PHONE_NUMBER>354816 with RPC
2023-02-13T08:42:50.096468Z  INFO risingwave_frontend::scheduler::hummock_snapshot_manager: Unpin epoch<PHONE_NUMBER>316352 with RPC
2023-02-13T08:43:10.094423Z  INFO risingwave_frontend::scheduler::hummock_snapshot_manager: Unpin epoch<PHONE_NUMBER>977280 with RPC

It may be better to be removed or displayed as a Grafana metric or somewhere, keeping the console output clean.

Just as a possible solution and fyi: We could also use Loki queries that parse the logs to create the metrics OR use tools like logstash to create metrics based on our logs.

Ping?


Move guest-agent stable branch to topic-stable.

Since the "stable" ref name's been historicaly used for purposes of marking the latest stable release we are kind of limited of using it, for the purpose of stable releases we are using the topic-stable name.

/lgtm


[dubbo-1521]add java 9 in travis yml

What is the purpose of the change
add java9 in ci
Brief changelog
use jacoco
fix reflection issue
Verifying this change
ci pass
coverage in codecov.io
close #1521

Codecov Report

Merging #1575 into master will increase coverage by 2.26%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1575      +/-   ##
============================================
+ Coverage     32.17%   34.43%   +2.26%
- Complexity        0     3669    +3669
============================================
  Files           682      618      -64
  Lines         32972    30823    -2149
  Branches       6597     5445    -1152
============================================
+ Hits          10608    10615       +7
+ Misses        20468    18378    -2090
+ Partials       1896     1830      -66

Impacted Files
Coverage Δ
Complexity Δ

...a/com/alibaba/dubbo/common/utils/ReflectUtils.java
44.7% <100%> (+0.63%)
112 <0> (+112)
:arrow_up:

...ng/transport/dispatcher/WrappedChannelHandler.java
30.3% <0%> (-12.13%)
2% <0%> (+2%)

...com/alibaba/dubbo/common/logger/LoggerFactory.java
28.26% <0%> (-7.16%)
3% <0%> (+3%)

...ing/transport/dispatcher/ChannelEventRunnable.java
77.08% <0%> (-6.25%)
9% <0%> (+9%)

...ubbo/common/compiler/support/AbstractCompiler.java
45.45% <0%> (-4.55%)
3% <0%> (+3%)

...bo/remoting/transport/netty/NettyCodecAdapter.java
53.12% <0%> (-3.6%)
3% <0%> (+3%)

...ibaba/dubbo/rpc/protocol/thrift/ThriftInvoker.java
56.81% <0%> (-3.19%)
5% <0%> (+5%)

.../com/alibaba/dubbo/rpc/filter/ExceptionFilter.java
19.04% <0%> (-2.39%)
2% <0%> (+2%)

...rpc/protocol/thrift/ext/MultiServiceProcessor.java
12.76% <0%> (-2.13%)
3% <0%> (+3%)

...ubbo/remoting/transport/mina/MinaCodecAdapter.java
56.89% <0%> (-2.12%)
3% <0%> (+3%)

... and 199 more

Continue to review full report at Codecov.

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

When you have tested the program error or the test case doesn't pass, will the CI process terminate? @htynkn

@lovepoem please refer this build: https://travis-ci.org/diamondblack/incubator-dubbo
If test fail, CI will fail too. logs here: https://api.travis-ci.org/v3/job/364388877/log.txt

Look good to me


chore(main): release pixels 0.27.0

:robot: I have created a release beep boop
0.27.0 (2024-11-05)
Features

pixels: add initial ButtonGroup (826054d)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

pixels-v0.27.0
:sunflower:


fix(globals): do not share globals between skate versions

This fixes #493 for 0.12.x
refs #494
@treshugart @jpnelson @seancurtis PTAL

lgtm

lgtm

<EMAIL_ADDRESS>released


Shubhusion/issue98

Summary of Changes

Redesigned the Readme.md
Added Badges
Added Troubleshooting Tips Section Under Running
Added Connect with Us Section

Related Issue
Closes #98
Checklist

[x] I have read and followed the project's contribution guidelines, including code style and commit message conventions.
[x] My code is well-documented, and I've updated relevant documentation.
[x] I have added or updated test cases to ensure the code's functionality.
[x] I have tested these changes on my local environment.
[x] All tests pass, and there are no new linting errors.
[x] I have reviewed and proofread my code and the changes.
[x] The branch is up-to-date with the base branch.

Screenshots (if applicable)
Before Changes:

After Changes:

Additional Context
Add any additional context or information that might be helpful for reviewers.
Reviewer(s)
@hexsecs @jhdalek55

@shubhusion Could you please change the text 'built with' to a badge instead

something like this :
you can use : https://forthebadge.com/#/generator for this purpose

Hi @shubhusion instead of forthebadge can you please use https://shields.io/badges

@shubhusion Could you please change the text 'built with' to a badge instead

something like this :
you can use : https://forthebadge.com/#/generator for this purpose

Hi @shubhusion instead of forthebadge can you please use https://shields.io/badges

@Abhijay007 let me know if this works?

@shubhusion Could you please change the text 'built with' to a badge instead

something like this :
you can use : https://forthebadge.com/#/generator for this purpose

Hi @shubhusion instead of forthebadge can you please use https://shields.io/badges

 @Abhijay007 let me know if this works?

ya it LGTM

That's perfect. If you can update to that, I can merge the PR today.


fix(RHINENG-8253): Update risk label feature flag

I wasn't aware this feature is behind a flag so adding it.
Includes a test that checks for labels and api requests if the feature is disabled.
How to test:

Run the app in stage stable with the flag ocp-advisor.upgrade-risks.enable-in-stable disabled in Unleash
Go to Clusters
Check that you don't see any labels with Update risk next to the Cluster name and that there's no api requests on update-risks-prediction endpoint
Enable the flag and check if you can see some clusters with the label, if not check the network tab if there's a request on the update-risks-prediction endpoint
If you don't have any Update risk labels, try the prod-beta environment, there might be some there

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


fixed Tree, Human, and Snake. Each one of them didn't have enough 'lose' entries.

Thanks @slowrance ! I do believe it's possible I made a mistake on that spreadsheet. I had to do it manually by reading the picture. :)


Improve build logic

I have been giving some lovin' to the build system.  I've managed to reduce a bit the final image size as well as the build time:
branch:	improve-build-logic
build-target:	pixiecore-git
build time:	1:00.07
size:		15.81 MB

branch:	improve-build-logic
build-target:	pixiecore
build time:	1:02.85
size:		45.25 MB

branch:	master
build-target:	pixiecore-git
build time:	1:17.88
size:		21.76 MB

branch:	master
build-target:	pixiecore
build time:	1:22.74
size:		63.81 MB

I've tried to keep the commits as simple as possible, so that the changes could be properly explained.
Feedback welcome!
Cheers,

Awesome, thank you for all the improvements!

Doh, I didn't mean to squash all the commits... Sorry :(. But thank you for making it easy to review!

Really glad to be of service!
No worries, they served their purpose.
Cheers!


correctly implement gRPC health checking, or remove it

The gRPC health check is also a standard interface, but I'm afraid it's not correctly implemented yet.

Originally posted by @BugenZhao in https://github.com/risingwavelabs/risingwave/issues/17771#issuecomment-2244305973
Currently we always return Serving.
The priority might be low since

we already have cluster-level health checking through SQL interface, i.e., rw_recovery_status()
it might not be that meaningful to define "whether healthy or not" separately for different nodes in a cluster
currently there's no usage of this

Or, if we find it not useful, we can remove the support to avoid confusion.

We should first define node-level health. For example, a compute node is considered healthy in terms of responding batch queries even when the cluster is under recovery, while it's not in terms of making DDL changes.


Slack link is no longer active

Describe the bug
When I click on the slack icon in the website footer, the page appears with a message prompt "This link is no longer active"
To Reproduce
Steps to reproduce the behavior:

Go to https://hitk-tech-community.netlify.app/
Scroll down to the footer (in the social section)
Click on the slack icon
See error

Expected behavior
I expected to click the slack icon in the website footer, which would lead me to join the slack server of HITK Tech Community.
Would you like to work on this issue?
Yes.
Have you read the Code of Conduct?
Yes.

The same issue appears in the above bot comment when I click on Slack Community.


Include ambient types (declare keyword) as exported types.

Ambient reflection flag is added to reflections that are explicitely declared using the declare keyword.
This isn't only possible in declarations (*.d.ts) but also in source files.
The context's isDeclaration flag is pushed onto the scope stack to keep track of ambient types. All types inside an isDeclaration context are exported.

I did some testing and have concluded that the current behavior is correct. declare type does not export that type. Trying to import it from another file and use it doesn't work. The only case when it is available is when the file is not a module (no import/export) in the first place (in which case it still isn't exported from some scope, everything is global)
Sorry this took so long to get to, if I'm wrong about the behavior please feel free to open an issue with some code samples showing me why :)

Coming back to this, I was partially correct. Just because something is declared doesn't mean it is exported, but some declared symbols without an export may be "exported" so long as they appear in a global file with no imports/exports


Allow .from_csv to accept URL

The DataFrame.from_csv method only accepts file paths currently. Extend it to accept URLs.

Just found a pretty cool thing that could be in help somehow: https://github.com/seamusabshere/remote_table

We can use open-uri
content = open('link/file.csv')
File.write('file.csv', content.read)

and then we can use file.csv,
df = Daru::DataFrame.from_csv "file.csv"

right ?

@Shekharrajak This issue is already worked on https://github.com/SciRuby/daru/pull/300 -- though, the last movements was 2 weeks ago... So, I am not sure.
@anshuman23, do you plan to finalize your PR or what?..

Thanks for info.

@zverok - Sorry for not informing you earlier, but it completely slipped my mind. My college exams had started just a day after I last committed changes. However, they get over by Monday the coming week. So I will finalize the PR by Tuesday or at most Wednesday positively. Most of the requested changes are almost ready on my end.

I think this issue can be closed, as this has been fixed by PR #334.


Markdown view

@ritch can you purge .chroma from the PR? Would be nice to avoid the extra MBs of data sticking around in clones forever.

I amended the commit without the file so I think it should not be in the history.


Allow custom Account naming

Hub thread: https://powerofus.force.com/0D58000002atMIl
Description from the existing item in the internal backlog that has now been closed as duplicate: "Allow users to, for example, add the Contact's student ID to the name of the Administrative Account."

**lurch: add

Tracking W-009736

Closing this Issue as it is not related to an Idea.

Closed #436 as duplicate . Reopening to continue discussion for PUB issue https://powerofus.force.com/hub-ideas/1158

Power of US Idea Owner: Jennifer Sanchez
Power of US Idea
Enhancement Request: Whether the default HEDA account model is set to "Administrative" or "Households" it would be very useful to provide the ability to customize the account naming setting. Similar to NPSP, provide a setting within the HEDA settings to be able to specify the account name format. Here is the NPSP documentation for customizing household names: https://powerofus.force.com/articles/Resource/NPSP-Customize-Household-Names#!#nameformat
Requirements:
Ability to customize the the "Administrative" Account name with the following formats:

{!LastName} Administrative Account:  The current option for Administrative Accounts.

{!{!FirstName}} {!LastName} Administrative Account: Generates an Administrative Account name such as “John Smith Administrative Account”.

{!LastName} ({!{!FirstName}}) Administrative Account: Generates an Administrative Account name such as “Smith (John) Administrative Account”.

You can also select other and specify your own naming convention.

Ability to customize the "Household" Account name with the following formats:

none: Does not generate a Household Name, leaving it up to the user to assign one.

{!LastName} Household:  Generates a Household name such as “Smith Household” or “Smith and Jones Household,” depending on the number of Contacts in the Household.

{!{!FirstName}} {!LastName} Household: Generates a Household name such as “John Smith Household” or “John and Jane Smith Household”.

{!LastName} ({!{!FirstName}}) Household: Generates a Household name such as “Smith (John) Household”, “Smith (John and Jane) Household” or “Smith (John) and Doe (Jane) Household”.

{!LastName} Family: Generates a Household name such as “Smith Family” or “Smith and Jones Family”.

{!{!FirstName}} {!LastName} Family: Generates a Household name such as “John Smith Family” or “John and Jane Smith Family”.

{!LastName} ({!{!FirstName}}) Family: Generates a household name such as “Smith (John) Family”, “Smith (John and Jane) Family” or “Smith (John) and Doe (Jane) Family”.

You can also select other and specify your own naming convention.

Included in production release rel/1.49


Rename Control getters and setters to match properties

Part of #16863, originally identified by @AlexHolly here.
Includes renaming some properties to match the getters and setters where appropriate.
 Otherwise it would be a pain to refactor PRs after this one.

Do you mind listing the renames here directly?

Done.

rect_min_size -> rect_minimum_size

Why this needs to be changed? We already use min/max for math functions, so it's not like the property is unclear. Also if we are expanding, rectangle_minimum_size would be more correct/consistent, but it's even longer. While it's ok to make methods more explicit, property names take inspector space, so we shouldn't elongate them unnecessarily.

Such changes where already made in the past. I think this is where it originated.
https://github.com/godotengine/godot/commit/9bc534896135322a578118f433f0dc6265839082#diff-e31be202f4d69f30dffc794ce6dc2de6cb4d5e77e22fa55552a7cae936ab34ccL327

rect_min_size -> rect_minimum_size

Why this needs to be changed?

It's probably to match the internal methods, which were set/get_custom_minimum_size and became set/get_rect_minimum_size. I guess we can rename methods to set/get_rect_min_size instead.

I guess we can rename methods to set/get_rect_min_size instead.

Yep, IMO that would be better. The PR is about renaming these methods anyways.

rect_min_size -> rect_minimum_size

As suggested by @pycbouh, in this instance the property name was changed to better match the methods. Not just the exposed methods, but also Control's internal methods. Using minimum makes things consistent.

rectangle_minimum_size would be more correct/consistent

I agree that rectangle_* would be the better name, and I have changed the display name in the Inspector. However, unless there is some consensus on renaming Rect2 to Rectangle, it's probably better to keep it at rect for now.

property names take inspector space, so we shouldn't elongate them unnecessarily.

I agree, but Minimum Size is not currently Control's longest property name, and it fits in the default Inspector width; so I think it's okay as it is:

Rebased following merge of #38054, #45571 and #47166.

Rebased following merge of #43155, #47230 and #47491.

Rebased following merge of #46273 and ran --doctool ..

Rebased following merge of #37966 and #46340.

Rebased following merge of #46593 and #47448.

Given that some of these renames are obsolete because we renamed the properties instead, and that this PR has been inactive for over a year now, and we're at the end of the beta cycle, I'm going to close it. Thanks for your contribution nonetheless! Perhaps we'll be able to do more API cleanups in Godot 5 🙂


Upgrade from 5.7.2 -> <IP_ADDRESS> is broken

Hi,
I'm currently in the process of upgrading from concrete 5.7.2 to <IP_ADDRESS> and having some issues with the migrations running. ( I'm using the upgrade script /index.php/ccm/system/upgrade )
concrete/src/Updater/Migrations/Migrations/Version20141219000000.php -> calls Page::getByPath which generates the following exception
An exception occurred while executing 'SELECT t0.cPath AS cPath1, t0.ppID AS ppID2, t0.cID AS cID3, t0.ppIsCanonical AS ppIsCanonical4, t0.ppGeneratedFromURLSlugs AS ppGeneratedFromURLSlugs5 FROM PagePaths t0 WHERE t0.cID = ? AND t0.ppIsCanonical = ? LIMIT 1' with params ["209", 1]: SQLSTATE[42S22]: Column not found: 1054 Unknown column 't0.ppGeneratedFromURLSlugs' in 'field list'

concrete/src/Updater/Migrations/Migrations/Version20150504000000.php -> defines the column ppGeneratedFromURLSlugs but this migration is never ran as the previous migration throws that exception.
I've solved this locally by adding the following code to the first migration ``concrete/src/Updater/Migrations/Migrations/Version20141219000000.php` but that seems a bit hacky.
		$db = \Database::get();
		$pp = $schema->getTable('PagePaths');
		if (!$pp->hasColumn('ppGeneratedFromURLSlugs')) {
			$db->Execute('alter table PagePaths add column ppGeneratedFromURLSlugs tinyint(1) unsigned not null default 0');
			// we have to do this directly because the page path calls below will die otherwise.
		}

Yes, this is a known issue that needs to be documented. You should update from 5.7.2 to 5.7.3 first, then up to <IP_ADDRESS>.

@aembler I always found it quite a hassle to do upgrades like that. I never looked into it, but since every version contains the upgrade scripts for all older versions: Shouldn't it be possible to skip those versions in between? Is it worth looking into this?

I think the issue is that when we attempt to do multiple migrations, we're
currently using the most recent code base to do each migration. Going from
5.7.2 to 5.7.4 loads the 5.7.4 code base and runs through 5.7.3 and 5.7.4
migrations. So Version 5.7.3's migration script uses the PagePath's table
(likely because it's adding a single page.) Unfortunately, it uses versions
5.7.4's script to do this, and version 5.7.4's single page class references
a column that doesn't exist in the Version 5.7.3 migration – but Version
5.7.4 hasn't been migrated yet.
I'm not sure if there's an easy way to solve this – thoughts?

Okay I see what you mean. I guess the only approach that would work is if we'd avoid the framework as much as possible or write code that works with two different database structures. The latter would lead to ugly code we could hardly ever remove and avoiding the framework won't always work and isn't nice either but it's what most systems do as far as I've seen.

What's the recommdend approach after upgrading from 5.7.2 to <IP_ADDRESS> and then running into this issue, other than truncating the database and starting from anew?

I would recommend replacing the core with 5.7.3 - upgrading to that - and then going to <IP_ADDRESS>. That should be fine.
Sent from my iPhone

On Jun 13, 2015, at 1:04 PM, Joost Rijneveld<EMAIL_ADDRESS>wrote:
What's the recommdend approach after upgrading from 5.7.2 to <IP_ADDRESS> and then running into this issue, other than truncating the database and starting from anew?
—
Reply to this email directly or view it on GitHub.

Pretty much how I've handled this in other projects is to be strict in what
classes migrations can use, for example in this case, migrations should
only be able to use the doctrine sql classes (not ORM) and then your
migrations will pretty much be "standalone".
This does introduce some complexity because you can't reuse functionality
from your entities, but It ensures your migrations always function.
On 14 June 2015 at 08:11, Andrew Embler<EMAIL_ADDRESS>wrote:

I would recommend replacing the core with 5.7.3 - upgrading to that - and
then going to <IP_ADDRESS>. That should be fine.
Sent from my iPhone

On Jun 13, 2015, at 1:04 PM, Joost Rijneveld<EMAIL_ADDRESS>wrote:
What's the recommdend approach after upgrading from 5.7.2 to <IP_ADDRESS> and
then running into this issue, other than truncating the database and
starting from anew?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/concrete5/concrete5/issues/2455#issuecomment-111757926
.

I've actually wondered if we shouldn't have a property on the migration classes that indicate a minimum core version needed to run the migration. This way, if someone tried to run an update and they were trying to make to big of a jump we could indicate to them which version they needed to use instead of just erroring out.

In the future if you use the Dashboard to update, we inspect every update through concrete5.org, and will tell you if you can go from update A to update B, and even can check out potentially overrides problems

Nice! Sounds very promising!


Fix light source behind the camera

Is it possible to fix the light source behind the camera, so the highlighted side of the data is always the side nearest the viewer?
If this is not possible, how can I create a second light source, on the opposite side?
Thanks.

@SteveEdson

Is it possible to fix the light source behind the camera, so the highlighted side of the data is always the side nearest the viewer?

Are you using the lightSettings props? Have you tried just updating the light settings as your viewState changes?

If this is not possible, how can I create a second light source, on the opposite side?

If you look at lightPosition setting, you'll see that it is an array, you should be able to list more than one point position (x, y z).
   const LIGHT_SETTINGS = {
      lightsPosition: [-125, 50.5, 5000, -122.8, 48.5, 8000],
      ambientRatio: 0.2,
      diffuseRatio: 0.5,
      specularRatio: 0.3,
      lightsStrength: [1.0, 0.0, 2.0, 0.0],
      numberOfLights: 2
    };

Lighting system being reworked in 7.0 for #2096


Unable to drag horizontal scrollbar

It is possible to drag the vertical scrollbar but not the horizontal one. (on Mac)
~On Windows systems horizontal scrolling might be tricky so having a way of dragging the scrollbar is helpful.~
Update: Looks like on Windows scrollbars are always visible.
Sandbox: https://codesandbox.io/s/8z7j1wjok2

I don't believe this has anything to do with react-virtualized. I can reproduce the behavior you're describing with the following HTML:

Looks like on Windows scrollbars are always visible.

You're describing the default OS browser style. On Macs, scrollbars are hidden by default, but you can also make them always visible in settings (General > Show scrollbars > Always) if you prefer.

@bvaughn you're right.
Yes, scrollbars are hidden by default, although looks like the behavior on Chrome (Mac) is very inconsistent; I did a quick "test" by shrinking the browser and the body horizontal scrollbar works fine (can be dragged), but not the one in a overflow-hidden div 🤦🏼♂.  Safari and Firefox work just fine.
Definitely not related to react-virtualized.
Thank you so much for your time.

No problem :)


Autodoc integration / docs

Since .. code-refs:: is already recommended to be included in reference documentation, we ought to hook into autodoc (if possible) to inject the directive to docstrings. If not possible, we should at least document a way to do it.

Well that was easier than I thought 😅


Duplicate code in evm and starky crate?

It seems there are lots of duplicated code between evm and starky crate, like constraint_consumer.rs, vanishing_polynomials.rs, get_challanges.rs and so on.
Also it seems cross_table_lookup is also generic so it can also be put under starky crate.
Any specific reason why above is not done yet?

Hi! There was indeed a lot of code duplication, initially because we needed additional features that starky didn't have and that seemed too specific to the evm crate.
We're proceeding to a major code migration to update starky and leveraging it within the plonky2_evm crate. The remaining associated PR is this one: #1503, that should be resolving this issue.


Feature: Add startup script

While this is a "faulty" assumption, one may expect that APM would provide a startup script (similar to pm2).

This is actually a nice feature that could be added.
If you want you can provide a PR or I will do it myself.
What do you think?

Thanks but I cannot do it at this time.
On Monday, October 17, 2016, Mateus Dantas<EMAIL_ADDRESS>wrote:

This is actually a nice feature that could be added.
If you want you can provide a PR or I will do it myself.
What do you think?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/topfreegames/apm/issues/5#issuecomment-254254296, or mute
the thread
https://github.com/notifications/unsubscribe-auth/APEcmGs6uJ7yI6YmqUJG7M6YN9JUuodhks5q055YgaJpZM4KT2ih
.

--
Sincerely,
Travis Smith | Web Platform Architect
w : 770–226–2284
m : 678–506–0803
e<EMAIL_ADDRESS>

[ ] Add startup script to APM


refactor: cherry pick 11400 to v1.1 rc

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
close #11410
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[x] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR contains user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note

Wrong target branch.


NuGet Error NU1107 -- improve message and help text

We are trying to improve error messages to ensure that NuGet users can easily understand the best approach to fix it.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/errors-and-warnings/NU1107.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

This message would be even better if it were telling which project is having a problem deciding which package to use.


Rename conversation_workspace_name_unique in scripts/workspace_deploy.py to something like 'conversation_workspace_deploy_by_name'

I would just suggest to change name of this param to something like: conversation_workspace_deploy_by_name, it is the real action we tried to do and we test if the name is unique... when we change it... we just need to slightly change log output and it will make more sense:

'conversation_workspace_name set to unique' -> something like 'conversation_workspace_deploy_by_name is set to true -> trying deploy workspace by name'
"'conversation_workspace_name' set to unique but not defined." -> 'conversation_workspace_deploy_by_name is set to true but no workspace name is provided (conversation_workspace_name)'

Done in #317
Parameter is named conversation_workspace_match_by_name and it is used for all uploading, deleting and testing workspaces.


[FEATURE REQ] - Service bus processing handler and error handler under the same logical thread

Service Bus
Is your feature request related to a problem? Please describe.
When registering to the processor via ProcessMessageAsync and ProcessErrorAsync it appears that the library wrap the invocation of ProcessMessageAsync in a logical thread context which does not follow to fire ProcessErrorAsync when error occur. Instead the library internally catch and will move on to fire ProcessErrorAsync in a different async flow which is not the child of the async flow in ProcessMessageAsync
Since there are a lot of approaches in the eco-system to use AsyncLocal for things like logging, tracing, etc... It might be
confusing to some why those does no longer work.
The error handler signature does not include the original args from the receiver handler, which for me imply that this design is intended since there are not helpers there to settle the message. However, it is not clear at first sight and might make it harder for developers to integrate.
If it's possible to invoke the error within the logical context of the message handler it will be great.
However, I suspect it won't be the case, so I suggest more clear documentation on this subject, explaining the relationships between both handler and making it more clear for users to use.
If I've misses such documentation, my apologies in advance.
Thanks

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Hi @shlomiassaf,
If I'm understanding this correctly, you are wondering why the exception handler is not being triggered as a descendant of the message handler invocation. The reason for this is that the exception handler is not triggered only as a result of errors from the message handler. The exception handler can also be triggered due to errors that occur while the processor receives messages, completes messages, and other parts of the processor lifecycle (see https://docs.microsoft.com/en-us/dotnet/api/azure.messaging.servicebus.servicebuserrorsource?view=azure-dotnet). As such, it wouldn't be possible to have the kind of structure that you are looking for, or to include the settlement args (as there may not even be a message to settle).
If you want to handle errors that might occur in your message handler, the best way to do that would be to catch exceptions within the message handler and handle them as you see fit. Please let us know if you have further questions about this.

Hi @jsandoval-msft
Thank you for the time and the detailed information.
I assumed this will be the case, I only suggested to add this to the docs so users will know that both handlers originate from different async flows.
Anyway, thank you for clarifying :)


Minroot circuit

For minroot-vdf we need to enforce x_i_plus_1 = (x_i + y_i)^{1/5}, which can be more easily checked with x_i_plus_1^5 = x_i + y_i. And we want x_i_plus_1 as a public output. I think we can't do the constraint check other way round in generate_step_constraints fn. Is there a workaround without adding a custom gadget? @arnaucube

Hi, I'm not sure I'm understanding the question, from what you described (I haven't checked the minroot) you can compute x_i_plus_1 ^ 5 and enforce it to be equal to x_i + y_i directly in the circuit code, right?
I'm checking now the microsoft/Nova example on minroot and seems that similar code logic could be done in sonobe using arkworks.
Btw, I understand that you're preparing benchmarks comparing folding libraries, take in mind that sonobe is not yet optimized since we've focused our efforts into having the onchain verifier that can verify Nova proofs in the Ethereum EVM's.


React example is not working

When I run the 'react' example code. The only out put I get is
xxx:react xyx$ npm test

@ test /Users/dgrabov/workspace/jest/examples/react
node ../../bin/jest.js --verbose --coverage

Using Jest CLI v0.4.15
Waiting on 1 test...xxx:react xyx$
However all works as expect with non React tests such as 'getting_started' which produces the output below as expected.
xxx:react xyx$ npm test

@ test /Users/dgrabov/workspace/jest/examples/getting_started
node ../../bin/jest.js

Using Jest CLI v0.4.15
PASS  tests/sum-test.js (0.015s)
1 test passed (1 total)
Run time: 0.279s
Anyone know what might be causing it?

Having the same issue.

> @ test /Users/daniel/Sites/jest/examples/react
> node ../../bin/jest.js

  == NOTICE: ==
    On August 1st, 2015, Jest v0.5.x will work only on io.js
    Legacy v0.4.x for Node 0.10.x will still be available on
    on npm and from the "0.4.x" branch on GitHub.

Using Jest CLI v0.4.15
Waiting on 1 test...%

However this only happend when testing react components, when testing a simple example (like on the homepage using the sum method) it does work:
npm test                                                                                (do.jul.09)─┘

> @ test /Users/daniel/Sites/jest-example
> jest

  == NOTICE: ==
    On August 1st, 2015, Jest v0.5.x will work only on io.js
    Legacy v0.4.x for Node 0.10.x will still be available on
    on npm and from the "0.4.x" branch on GitHub.

Using Jest CLI v0.4.15
 PASS  __tests__/CheckboxWithLabel-test.js (0.063s)
1 test passed (1 total)
Run time: 0.283s

Mac OSX node v0.12
Also, when running the test of the jest package in general I'm getting a failing test:
><EMAIL_ADDRESS>test /Users/daniel/Sites/jest
> jshint --config=.jshintrc --exclude=src/coverageTemplate.js src && node bin/jest.js

  == NOTICE: ==
    On August 1st, 2015, Jest v0.5.x will work only on io.js
    Legacy v0.4.x for Node 0.10.x will still be available on
    on npm and from the "0.4.x" branch on GitHub.

Using Jest CLI v0.4.15
 PASS  HasteModuleLoader/__tests__/HasteModuleLoader-currentTestPath-test.js (0.041s)
 FAIL  __tests__/TestRunner-test.js
Error: Worker process exited before responding! exit code: null, exit signal: SIGSEGV
stderr:

A worker process has quit unexpectedly! This is bad news, shutting down now!
npm ERR! Test failed.  See above for more details.

I'm seeing the same issue on OS X with;
node v0.12.4
jest v0.4.15
<EMAIL_ADDRESS>Again simple tests are fine but the meant I require react the tests exit without an error.

Some combination of requiring react/addons and current jest version being funny with node >0.10.
Option a:
nvm use 0.10 then npm rebuild
Option b:
assuming you can use iojs, npm install facebook/jest#0.5.x should let it run
There is an option c to get it working with 0.12, but can't recall what I did.
Related issues https://github.com/facebook/jest/issues/347 (closed) and https://github.com/facebook/jest/issues/243

@psimyn  Thank you very very much! Going back to node 0.10.35 solved this for me (for now).

Jest doesn't work on node 0.12. You need to upgrade to node 4 (use jest 0.5+) or use node 0.12 (use jest 0.4 or below).


RTSP smart recording is broken due to changes to the NVIDIA recordbin to support Audio

Unfortunately, this has been broken for some time... automated tests were/are insufficient to catch this.
Changes to the recordbin plugin required changes to the way the depay, parser, and recordbin are linked.  NVIDIA's documentation and common RTSP source bin code still have this wrong.. i.e. have not been updated... only the Smart Recording example has been.
With the support of Audio, the parser must be moved between the pre-decode tee and recordbin ... i.e. can no longer be before the pre-decode tee.  The decoder is able to decode after depay

Fixed and merged into v0.26.alpha


Project doesn't work with cocoapods 1.0.1

I can't properly install the dependencies.
Tried to migrate the podfile but still getting many errors.
Any fix for this?
Thanks

@avnerbarr Please try building the latest commit and report back. It was necessary to update Google Analytics SDK.


Feature/cait dataset data model and import process

I'm making the PR so that I can get some feedback on the work. Let's not merge this for now, I still might have changes to make depending on the feedback from the frontend team.

I talked with the frontend guys and I'm going to do some deep changes to the serialization. After that we should be good to do the full review and merge.

👍


Address some type-stability issues

We had a few type-stability issues. I've introduced tests and fixes that are largely inspired by Seth's work on FiniteDifferences's to_vec function.

Hmm, maybe we should switch this repo over to GH actions?

Hmm, maybe we should switch this repo over to GH actions?

Yes, we should. Probably a job for tomorrow.

@rofinn @oxinabox tests are failing on 1.3 because cumsum wasn't defined on Tuples until 1.5.
I took a look at using Compat for it -- no luck unfortunately. Also looked at the definition in Base on 1.5 -- looks like a reasonable amount of the code to make this work just wasn't available pre-1.5. Any thoughts on the easiest way to proceed?

FWIW, I'd probably just add a version check and do for cumsum(collect(lengths)) for julia releases pre 1.5


Access Postgres database object without permissions causes lockup in DBeaver

System information:

Operating system (distribution) and version:
-- OS Name	Microsoft Windows Server 2016 Datacenter
-- Version	10.0.14393 Build 14393

DBeaver version: <IP_ADDRESS>108291732

Additional extensions: None

Connection specification:

Database name and version:  Postgres 11.12
Driver name:
Do you use tunnels or proxies (SSH, SOCKS, etc)?  No

Describe the problem you're observing:
If I double click on a database object for which I have no permissions, DBeaver hangs and cannot recover. No message etc. seems to pop up and I have to reboot DBeaver.  Also, works does not get saved. (quite frustrating).
Steps to reproduce, if exist:

Create a database object as a superuser (e.g. create table)
logout and login as a different account with NO privileges on the object
attempt to access the database object by double-clicking on it in the navigation pane on the left
you should see that DBeaver hangs
It must be forced to close and rebooted to get back to the DB

Include any warning/errors/backtraces from the logs
!ENTRY org.jkiss.dbeaver.model 4 0 2021-09-15 14:05:56.020
!MESSAGE SQL Error [42501]: ERROR: permission denied for materialized view mv_procedure
!SUBENTRY 1 org.jkiss.dbeaver.model 4 0 2021-09-15 14:05:56.020
!MESSAGE SQL Error [42501]: ERROR: permission denied for materialized view mv_procedure
!SUBENTRY 2 org.jkiss.dbeaver.model 4 0 2021-09-15 14:05:56.020
!MESSAGE ERROR: permission denied for materialized view mv_procedure
!STACK 0
org.postgresql.util.PSQLException: ERROR: permission denied for materialized view mv_procedure
	at org.postgresql.core.v3.QueryExecutorImpl.receiveErrorResponse(QueryExecutorImpl.java:2440)
	at org.postgresql.core.v3.QueryExecutorImpl.processResults(QueryExecutorImpl.java:2183)
	at org.postgresql.core.v3.QueryExecutorImpl.execute(QueryExecutorImpl.java:308)
	at org.postgresql.jdbc.PgStatement.executeInternal(PgStatement.java:441)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:365)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:307)
	at org.postgresql.jdbc.PgStatement.executeCachedSql(PgStatement.java:293)
	at org.postgresql.jdbc.PgStatement.executeWithFlags(PgStatement.java:270)
	at org.postgresql.jdbc.PgStatement.execute(PgStatement.java:266)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:330)
	at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:130)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeStatement(SQLQueryJob.java:513)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.lambda$0(SQLQueryJob.java:444)
	at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:171)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:431)
	at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.extractData(SQLQueryJob.java:816)
	at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:3440)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:118)
	at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:171)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:116)
	at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer$ResultSetDataPumpJob.run(ResultSetViewer.java:4718)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)

@rnemana unfortunately, cannot reproduce an issue. permission denied error is shown, but dbeaver doesn't hang. probably some special steps are needed?

hmm.... interesting. Let me see if I can reproduce and I'll get back to  you.  Meanwhile, I'll close this. I don't want it to pollute your open ticket  queue.


firefox crash when audio plays

Hi aws chime recording support.
I'm experiencing an issue when passing a url like a youtube video link, as soon as the audio starts to playing I get firefox crash screen.
wondering if you guys have any insight around that ?
I'm creating a localhost service based on your recording mechanism.
Thanks
-Oliver

Try to see if you can find and capture the Crash Report out of the /tmp/foo4 profile directory to see what may be causing the tab crash. Given that it occurs when you try to play a video, it's possible that there is a codec setting that may need to be either enabled or disabled.

found the crash report but it's not encoded with utf-8 strings. the file name is /tmp/recorder/crashes/store.json.mozlz4
I changed the user name from foo4 to recorder in run.sh file, that's why it's under /tmp/recorder.
based on my observation, it's fine when playing a video muted, but when playing a video unmuted, it will crash consistently. assuming it's audio encoding issue ?
weird that if use aws ec2 ecs , the recording with audio seems fine.
however, if local host based on the recording folder,  it doesn't seem to work with audio.
PFA for an example recorded mp4 file.
https://ucarecdn.com/601f3fae-d7a8-4246-adf9-5dc05a83ac20/1.mp4

Thanks @OliverwengFiltered. Would it be possible to retrieve the crash report file and attach to this issue? From the extension, it looks to be encoded in a Mozilla-specific mozlz4 encoding, but there are publicly available tools for decompressing files in this encoding.

here is the decoded crash report json:
https://ucarecdn.com/a1299250-af83-42ae-aa2c-8f2d0b31f0b5/search_extracted.json

Based on that crash report, it appears the main thread in Firefox is crashing, potentially when it tries to load OpenH264 for playing the video. We will try to reproduce this and see what can be done about this.

Hi @aburnson ,
thank you for the support.
and not sure if it helps to reproduce but here is what I have done to reproduce it on my local consistently.

get recording folder and Dockerfile from master branch.
use docker image build to build local image.
use docker run to run the image and attach to shell command.
in shell command run run.sh like this: ./run.sh https://www.youtube.com/watch?v=lgUDuZS9MVQ bucketName (run.sh has been slightly modified to support run with args)
after recording for like 10 seconds, stop recording by executing kill pid
watch recorded video, it will show firefox crash.

I'm using mac os latest version. same Dockerfile from ubuntu:18.04

Thanks for the repro steps, @OliverwengFiltered. That is helpful.

Hi @aburnson
Wondering if there is any updates about this one ?
Thanks.
-Oliver

@OliverwengFiltered For ECS, we have this configuration to stop the FF tab from crashing. I guess the same config should be considered if running the docker on localhost machine.
Meanwhile, we are trying to reproduce the issue without running on ECS.

Tried to reproduce the issue by running the recording docker container both on a local Mac and a cloud9 instance against the URL https://www.youtube.com/watch?v=lgUDuZS9MVQ.
No crash was observed during ~10 tests.
If you are still having this issue, you can try to explicitly specify --shm-size parameter in docker run command. For example. We have applied the SharedMemorySize parameter to ECS task definition in this demo, rather than in the running time of docker container.
It may explain the reason why the recording in ECS is fine which you mentioned above.
Hope it helps.

yup, it might be just a firefox v78 issue, never seen that again after using v83. please close this one now. thank you.


SAA-247 tightening validation around allocation of prisoners e.g. checking agaisnt prison api.

Codecov Report
Base: 68.08% // Head: 67.92% // Decreases project coverage by -0.16% :warning:

Coverage data is based on head (eb9ac75) compared to base (96ca529).
Patch coverage: 56.09% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #120      +/-   ##
============================================
- Coverage     68.08%   67.92%   -0.17%
- Complexity      567      570       +3
============================================
  Files           132      134       +2
  Lines          2701     2734      +33
  Branches         78       81       +3
============================================
+ Hits           1839     1857      +18
- Misses          808      822      +14
- Partials         54       55       +1

Impacted Files
Coverage Δ

...nagementapi/converter/StringToTimeSlotConverter.kt
0.00% <0.00%> (ø)

...esmanagementapi/service/ActivityScheduleService.kt
22.44% <21.42%> (-1.24%)
:arrow_down:

...pps/hmppsactivitiesmanagementapi/common/PayBand.kt
63.63% <63.63%> (ø)

...psactivitiesmanagementapi/common/PrisonerNumber.kt
81.81% <81.81%> (ø)

...activitiesmanagementapi/entity/ActivitySchedule.kt
89.70% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Re-export secp256k1_zkp

Re-exports secp256k1_zkp since a lof of the types are leaked in the API and just makes it easier to get from rust-dlc instead of having to import the library yourself. Needed to enable the rand-std flag for external use too.
Also makes read_dlc_message public so you can manually parse dlc messages externally.

Needed to enable the rand-std flag for external use too.

I think re-exporting secp-zkp is a good idea (bitcoin could be as well), but I'm not sure about adding features that are used only for outside purpose. Though it's inconvenient I think the proper way is for you to import it in your project and enable the features you need.

tACK
Master, nor 0.4, would compile without this


how to build a helmet camera?

Hi! I'm a student and I'm trying to use MuJoCo for building a sport simulator.
 I have thought of creating a void MjModel instance and use its built-in camera. Is there a more straightforward way to do that by modifying the XML file?
BTW, thanks for officially maintaining the MuJoCo Python API! Looking forward to hear any advice!

You can attach cameras to bodies. Here is an example.


Does increased accuracy result in imrpoved tracking performance?

Hello ZQPei,
Does the performance of tracking improve if the accuracy of classification model is increased?

Sure, the higher accuracy will lead to a better performance.

Will it also accelerate tracking? @ZQPei


hidden label on Samsung galaxy s7

hidden label, also the reward scale example is touching the edge of the box

screen zoom set to default, font size set to 3rd smallest option.

everything looks ok again if you set screen zoom to smallest and reduce font size.


How to concat multiple static files?

Description
I'm building a plugin for concatenation of markdown files (without parsing them into HTML), but when I try to compile them, they're generated individually in /public folder.
This is the plugin code I'm trying:
'use strict';
class ConcatCompiler {
  compileStatic({data, path}) {
    return new Promise((resolve, reject) => {
      // Should concat all ".md" files, but is compiling them individually instead
      resolve(data);
    });
  }
}

ConcatCompiler.prototype.brunchPlugin = true;
ConcatCompiler.prototype.type = 'template';
ConcatCompiler.prototype.extension = 'md';

module.exports = ConcatCompiler;

I'm trying to use 'template' as the plugin "type" because I believe this would fit better, but it generates HTML files instead of a single .md concatenated one.
Anyone have any idea of how can I concat multiple files into one?

Hey @kazzkiq, thanks for bringing this up.
Currently, there is no way to achieve this except for doing it manually in onCompile hook.
.html gets in because it is default staticTargetExtension value.
compileStatic was intended to make it possible for a single plugin to compile js runtime and static html templates (so there will be less bloat in plugin ecosystem). A better API for processing assets is coming in #1664.
However, no joinTo for assets yet. This is the thing we are considering. To speed this up, replying with cool use cases in this thread is appreciated.

I think one example is the very reason I opened this issue: modularization when writing markdown files.
Instead of writing a big piece of documentation, users could split the assets into multiple files for better organization and ease of maintenance. The final single-file being generated only for production:
static/
| - 01-introduction.md
| - 02-first-steps.md
| - 03-concerns.md
| - 04-how-to.md
| - 05-faq.md

would be "compiled" to:
public/
| - output.md

The same could apply to simple static websites/SPAs where for some reason there isn't any modular framework and/or template being used. Lets say I want to make a static webpage with lots of content in it. Perhaps it would be useful if I could split the pieces of my HTML into single files, and then using Brunch to generate the final index.html file:
static/
| - 01-header.html
| - 02-hero-intro.html
| - 03-features.html
| - 04-faq.html
| - 05-clients.html
| - 06-our-team.html
| - 07-talk-to-us.html
| - 08-footer.html

would be "compiled" to:
public/
| - index.html

Another reason to have this is SVG sprites. But for SVG, we can't just join files, we should wrap them. So this should happen in plugin, not in config (for users not to copy/paste ugly xml).


Add more filters for spawn protection

Describe your idea
Spawn protection should be more configurable. It should not only add a ResourceList for entities, but also for Blocks, and Items.
All of these values should be filled with default values for the most common gravestone mods. To name 3 of them:

Corpse
Corail Tombstone
GraveStone Mod

@MelanX You have better to chance to use the tag: "tombstone:graves" (or "tombstone:player_graves" if you don't want to allow the decorative graves)

Thank you for the input, I’ll consider changing it a bit. Since this is only the default config option which can be modified by any pack creator, it’s not that important.
But it’s a great idea to think about supporting tags in these and similar configs as well :)


fix(ruby)!: Avoid dropping pre-release version

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #1886 🦕

This alters the behaviour of release-please for ruby. It will create a release for 1.0.0.alpha.1 when 1.0.0-alpha.1 is specified to release-please. Ruby gems use a . instead of a - as the pre-release separator.
I have also removed the useDotPreReleaseSeparator boolean as I haven't seen it used anywhere in the code, making the . pre-release separator the default.
I had a look through the README and didn't find anywhere to update the documentation.


Updated Instructions

Visual Code 2022 is considerably different looking now. Would be nice if instructions showed updated screenshots.

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

ID: ad2eb047-525a-63b7-2d6e-1536a374d688
Version Independent ID: 2b8e3e9d-c957-d43e-ceeb-c75da0857d2d
Content: Tutorial: Write and register a plug-in (Microsoft Dataverse) - Power Apps
Content Source: powerapps-docs/developer/data-platform/tutorial-write-plug-in.md
Service: powerapps
Sub-service: dataverse-developer
GitHub Login: @divkamath
Microsoft Alias: jdaly

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

@jsdanie1 Yes, thanks for pointing that out. I will get to work on updating the screenshots.


Refactor with better organization, now with specs and templates for scaffolding

Changes Unknown when pulling 04ea6f977ff76000ae5483860e839ee3eae86b1f on refactor into ** on master**.

Changes Unknown when pulling a215271aa8b69040ab2486febc105b011f1a36f0 on refactor into ** on master**.


kindly remind: register the domain

Hi! I notice that you are building the website for kubewarf and other related components.
I noticed that the common-used .io domain like kubewarf.io, kubebrain.io and kubezoo.io were still available.
Please be harry to register them if you want to use these domain. :)

@STRRL Sorry for missing this message.
So kind of you to remind. Much hanks for your advice.~


Autoplay for cl.audio

Is your feature request related to a problem? Please describe.
We are currently trying to build a conversational AI app with voice output (reading out the response), and ideally, the player should start automatically. Currently, the user needs to click on the play button (of the CL audio player) first, interrupting the conversation flow.
Describe the solution you'd like
A parameter in cl.audio(autoplay = true) such that the audio (player) automatically starts.
Describe alternatives you've considered
We tried implementing the autoplay ourselves in the React frontend and Javascript backend, but it didn't work well.

This is already implemented since version 1.1.0 and the issue should be closed.


Composing selectors: lack of memoization

This example is given in the docs:
const makeMapStateToProps = () => {
  const getVisibleTodos = makeGetVisibleTodos()
  const mapStateToProps = (state, props) => {
    return {
      todos: getVisibleTodos(state, props)
    }
  }
  return mapStateToProps
}

However, calling makeGetVisibleTodos() will always return a new selector. Where does the memoization come into play here? The resulting getVisibleTodos will always operate on a new state, nothing is memoized. Or what am I missing here?

That's an example of the "factory function syntax" supported by the React-Redux connect function.  If a mapState function returns another function the first time it's called, that returned function is used as the "real" mapState.  So, that example creates a selector function instance that is unique to that closure, and therefore unique to the current component instance.  That means it will memoize correctly, even if multiple instances of the same component type are being rendered, whereas just re-using the same selector instance across multiple instances of a component would not memoize because they'd likely all pass in different props values.
I wrote up an example of using the factory mapState syntax here: https://www.reddit.com/r/reactjs/comments/5dxasp/any_deepdiveadvanced_tutorials_on_reselect/ .


Twitter API Restricted

The Twitter API url used in SocialCount works when only minimal calls are made to the Twitter API. After throwing several hundred requests at it (probably 150, the logged-out API limit) I began to receive errors.
After checking out the dev documents, it was stated several times that using this method to get Tweet counts is not authorized and that developers should not use it.
They stated that the proper way is to set up an Application in your account and then use the Streaming API to get these counts.
Of course, that is all well and good, but it is considerably more difficult than currently so I'm not sure what a good solution to this is. Just wanted to open an issue to let others know about this, because it will begin to fail if you throw a lot of requests at it.

Closing in favor of #54. Thanks!


build failure on CentOS 7

Attempting to build from source (binary doesn't work on CentOS 7), eventually got stuck on:
[btorpey@bt-brix infer]$ clear;./build-infer.sh

initializing opam...
# Run eval $(opam env) to update the current shell environment

installing infer dependencies; this can take up to 30 minutes...
[WARNING] Lock file opam.locked is outdated, you may want to re-run opam lock:
          Dependencies present in opam file not in lock file:
            - utop

The following actions will be performed:
  \u2217 install elina 1.1

<><> Gathering sources ><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[elina.1.1] found in cache

<><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[ERROR] The compilation of elina failed at "/home/btorpey/.opam/opam-init/hooks/sandbox.sh build make -j7".

#=== ERROR while compiling elina.1.1 ==========================================#
# context     2.0.5 | linux/x86_64 | ocaml-variants.4.07.1+flambda | https://opam.ocaml.org#5f5f316b
# path        ~/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1
# command     ~/.opam/opam-init/hooks/sandbox.sh build make -j7
# exit-code   2
# env-file    ~/.opam/log/elina-4180-6d07ae.env
# output-file ~/.opam/log/elina-4180-6d07ae.out
### output ###
# (cd elina_linearize; make all)
# make[1]: Entering directory `/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1/elina_linearize'
# make[1]: warning: jobserver unavailable: using -j1.  Add `+' to parent make rule.
# /build/share/gcc/5.3.0/bin/gcc -c -Wcast-qual -Wswitch -Wall -Wextra -Wundef -Wcast-align -Wno-unused -U__STRICT_ANSI__ -fPIC -O3 -DNDEBUG -Werror-implicit-function-declaration -Wbad-function-cast -Wstrict-prototypes -Wno-strict-overflow -std=c99 -g  -DNUM_LONGLONGRAT -DNUM_DOUBLE -DTHRESHOLD=0.75 -DTIMING -D_GNU_SOURCE -pthread -fno-tree-vectorize -m64 -march=native  -DHAS_APRON    -I/home/b[...]
# make[1]: /build/share/gcc/5.3.0/bin/gcc: Command not found
# make[1]: *** [elina_generic.o] Error 127
# make[1]: Leaving directory `/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1/elina_linearize'
# make: *** [c] Error 2

<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
\u250c\u2500 The following actions failed
\u2502 \u03bb build elina 1.1
\u2514\u2500
\u2576\u2500 No changes have been performed

*** `install_opam_deps` failed
*** Updating opam then retrying

<><> Updating package repositories ><><><><><><><><><><><><><><><><><><><><><><>
[default] no changes from https://opam.ocaml.org
[WARNING] Lock file opam.locked is outdated, you may want to re-run opam lock:
          Dependencies present in opam file not in lock file:
            - utop

The following actions will be performed:
  \u2217 install elina 1.1

<><> Gathering sources ><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[elina.1.1] found in cache

<><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[ERROR] The compilation of elina failed at "/home/btorpey/.opam/opam-init/hooks/sandbox.sh build make -j7".

#=== ERROR while compiling elina.1.1 ==========================================#
# context     2.0.5 | linux/x86_64 | ocaml-variants.4.07.1+flambda | https://opam.ocaml.org#5f5f316b
# path        ~/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1
# command     ~/.opam/opam-init/hooks/sandbox.sh build make -j7
# exit-code   2
# env-file    ~/.opam/log/elina-5172-6d07ae.env
# output-file ~/.opam/log/elina-5172-6d07ae.out
### output ###
# (cd elina_linearize; make all)
# make[1]: Entering directory `/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1/elina_linearize'
# make[1]: warning: jobserver unavailable: using -j1.  Add `+' to parent make rule.
# /build/share/gcc/5.3.0/bin/gcc -c -Wcast-qual -Wswitch -Wall -Wextra -Wundef -Wcast-align -Wno-unused -U__STRICT_ANSI__ -fPIC -O3 -DNDEBUG -Werror-implicit-function-declaration -Wbad-function-cast -Wstrict-prototypes -Wno-strict-overflow -std=c99 -g  -DNUM_LONGLONGRAT -DNUM_DOUBLE -DTHRESHOLD=0.75 -DTIMING -D_GNU_SOURCE -pthread -fno-tree-vectorize -m64 -march=native  -DHAS_APRON    -I/home/b[...]
# make[1]: /build/share/gcc/5.3.0/bin/gcc: Command not found
# make[1]: *** [elina_generic.o] Error 127
# make[1]: Leaving directory `/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/elina.1.1/elina_linearize'
# make: *** [c] Error 2

<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
\u250c\u2500 The following actions failed
\u2502 \u03bb build elina 1.1
\u2514\u2500
\u2576\u2500 No changes have been performed

*** ERROR: `install_opam_deps` failed
[btorpey@bt-brix infer]$ /build/share/gcc/5.3.0/bin/gcc
gcc: fatal error: no input files
compilation terminated.
[btorpey@bt-brix infer]$ bash -x -c `which gcc` --version
+ /build/share/gcc/5.3.0/bin/gcc
gcc: fatal error: no input files
compilation terminated.
[btorpey@bt-brix infer]$

Note the error message "/build/share/gcc/5.3.0/bin/gcc: Command not found", but invoking from command line confirms file exists, and that it is also on PATH.
TIA for any ideas ...

Opam has a sandboxing mechanism which I suspect is to blame here. You can try disabling it by removing the last 6 lines in your ~/.opam/config, which should look like this:
wrap-build-commands:
  ["%{hooks}%/sandbox.sh" "build"] {os = "linux" | os = "macos"}
wrap-install-commands:
  ["%{hooks}%/sandbox.sh" "install"] {os = "linux" | os = "macos"}
wrap-remove-commands:
  ["%{hooks}%/sandbox.sh" "remove"] {os = "linux" | os = "macos"}

Or alternatively by starting over from a new opam directory with sandboxing disabled:
mv ~/.opam ~/opam-old
opam init --no-setup --disable-sandboxing
./build-infer.sh

Thanks @jvillard !
Tried the second approach and still no joy (see below), get similar (incorrect) error: "/build/share/gcc/5.3.0/bin/gcc not found", when it is in fact at that location (and executable).
I know all the "cool kids" are on Ubuntu or Debian, but some of us are stuck on platforms (like RH/CentOS) that require some finagling to get more modern toolchains.  And since we want to maintain a "clean" OS install, those more modern toolchains end up somewhere other than
/usr/bin.  It would be nice if the install script was able to work with compilers that are NOT installed in the system-default location.
I assume that this is a problem in opam itself, and not infer per se?  FWIW, I couldn't care less about opam or OCaml for that matter, I would just like to get infer running on CentOS 7.
=> Jbuilder has been renamed and the jbuilder package is now a transition package. Use the dune package instead.

The former state can be restored with:
    opam switch import "/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/backup/state-20190805151806.export"
+ echo

+ printf '*** `%s` failed\n' install_opam_deps
*** `install_opam_deps` failed
+ echo '*** Updating opam then retrying'
*** Updating opam then retrying
+ opam update

<><> Updating package repositories ><><><><><><><><><><><><><><><><><><><><><><>
[default] no changes from https://opam.ocaml.org
+ install_opam_deps
+ local locked=
+ '[' yes == yes ']'
+ locked=--locked
+ opam install --deps-only infer /shared/infer --locked
[WARNING] Lock file opam.locked is outdated, you may want to re-run opam lock:
          Dependencies present in opam file not in lock file:
            - utop

The following actions will be performed:
  \u2217 install zarith 1.7

<><> Gathering sources ><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[zarith.1.7] found in cache

<><> Processing actions <><><><><><><><><><><><><><><><><><><><><><><><><><><><>
[ERROR] The compilation of zarith failed at "/home/btorpey/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/zarith.1.7/./configure".

#=== ERROR while compiling zarith.1.7 =========================================#
# context              2.0.5 | linux/x86_64 | ocaml-variants.4.07.1+flambda | https://opam.ocaml.org#794d1907
# path                 ~/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/zarith.1.7
# command              ~/.opam/ocaml-variants.4.07.1+flambda/.opam-switch/build/zarith.1.7/./configure
# exit-code            2
# env-file             ~/.opam/log/zarith-5854-d2c37b.env
# output-file          ~/.opam/log/zarith-5854-d2c37b.out
### output ###
# binary ocaml: found in /home/btorpey/.opam/ocaml-variants.4.07.1+flambda/bin
# binary ocamlc: found in /home/btorpey/.opam/ocaml-variants.4.07.1+flambda/bin
# binary ocamldep: found in /home/btorpey/.opam/ocaml-variants.4.07.1+flambda/bin
# binary ocamlmklib: found in /home/btorpey/.opam/ocaml-variants.4.07.1+flambda/bin
# binary ocamldoc: found in /home/btorpey/.opam/ocaml-variants.4.07.1+flambda/bin
# binary ar: found in /usr/bin
# binary perl: found in /usr/bin
# binary /build/share/gcc/5.3.0/bin/gcc: not found
# required program /build/share/gcc/5.3.0/bin/gcc not found

<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
\u250c\u2500 The following actions failed
\u2502 \u03bb build zarith 1.7
\u2514\u2500
\u2576\u2500 No changes have been performed
+ echo

+ printf '*** ERROR: `%s` failed\n' install_opam_deps
*** ERROR: `install_opam_deps` failed
+ exit 1
[btorpey@bt-brix infer]$ /build/share/gcc/5.3.0/bin/gcc
gcc: fatal error: no input files
compilation terminated.

Just to be sure: can you paste the contents of your ~/.opam/config after the second attempt? (I'm not 100% sure the opam command I suggested works as advertised)
Re: centos7: we also build on that platform internally :)

Als tried with system compiler (gcc 4.8.5) and get another dependencies error on CMake:
CMake Error at CMakeLists.txt:3 (cmake_minimum_required):
  CMake 3.4.3 or higher is required.  You are running version <IP_ADDRESS>

I put a more recent version of cmake on PATH and then I get this:
CMake Error at cmake/modules/CheckCompilerVersion.cmake:40 (message):
  Host GCC version should be at least 5.1 because LLVM will soon use new C++
  features which your toolchain version doesn't support.  Your version is
  4.8.5.  You can temporarily opt out using
  LLVM_TEMPORARILY_ALLOW_OLD_TOOLCHAIN, but very soon your toolchain won't be
  supported.

So I seem to be stuck -- it wants a newer gcc, but it won't use it.

Is there a reason you're using your own (personal) build of gcc (5.3.0) for this?
I've recently (in the last week) bootstrapped a whole new CentOS 7 VM to build infer without difficult -- however, I did use scl to get me access to a recent gcc and clang.
To install:
yum install centos-release-scl
yum install devtoolset-6 llvm-toolset-7

and then (in your bashrc):
source scl_source enable devtoolset-6 llvm-toolset-7

and afterwards:
# which gcc
/opt/rh/devtoolset-6/root/usr/bin/gcc
# gcc --version
gcc (GCC) 6.3.1 20170216 (Red Hat 6.3.1-3)

# which clang
/opt/rh/llvm-toolset-7/root/usr/bin/clang
# clang --version
clang version 5.0.1 (tags/RELEASE_501/final)

can you paste the contents of your ~/.opam/config

opam-version: "2.0"
repositories: "default"
installed-switches: ["ocaml-variants.4.07.1+flambda" "default"]
switch: "ocaml-variants.4.07.1+flambda"
jobs: 7
download-jobs: 3
eval-variables: [
  sys-ocaml-version
  ["ocamlc" "-vnum"]
  "OCaml version present on your system independently of opam, if any"
]
default-compiler: [
  "ocaml-system" {>= "4.02.3"}
  "ocaml-base-compiler"
]

however, I did use scl to get me access to a recent gcc and clang.

Hmmm -- didn't know about that, thanks!  I'll give that a whirl.  It would certainly be nice to be able to run more modern toolchains without having to build from source, which is a major hassle.

I probably should have said recent-in-quotes ("recent") rather than just recent ...

@WallStProg taking a step back: do you care about compiling infer from source? If not, would it make more sense to try and diagnose why you claim that you cannot run the pre-compiled infer binaries on CentOS 7 ("binary doesn't work on CentOS 7")?

Guess you're back to building your own :)

@jvillard : Re: centos7: we also build on that platform internally :)
How?

Have you tried using the scl versions of gcc/clang?
All I have to do is install gcc/clang from scl, install opam via its .sh method, and then I can do something like ./build-infer.sh -y clang (and I have to fill-in any holes that need -devel libraries).
Just an FYI though: your current build issue appears to be opam-related and not infer related.

Have you tried using the scl versions of gcc/clang?

Not yet, that's next.  Thanks for the suggestion!

Just an FYI though: your current build issue appears to be opam-related and not infer related.

Yeah, mentioned that earlier.  Doesn't help me, though -- that just makes the rabbit-hole deeper.


Update the version of ganache-cli

Resolve #291

Codecov Report

Merging #294 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #294   +/-   ##
=======================================
  Coverage   96.96%   96.96%
=======================================
  Files          80       80
  Lines        3131     3131
  Branches       24       24
=======================================
  Hits         3036     3036
  Misses         92       92
  Partials        3        3

Impacted Files
Coverage Δ

...c/zos-deployer/zos-jsonrpc-private-key-deployer.ts
100% <100%> (ø)
: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 95f679a...0baa5a8. Read the comment docs.


Is there a multiline message support?

Does DebugViewPP support multiline messages shown as a single message?
That's what I really lacked in DebugView (that I have just switched from), yet seems like DebugViewPP works the same way: it cuts each line into a separate message.

Yes, just uncheck 'View->Auto newline'

Uhm, that didn't help: the messages are still chopped into multiple ones.

Also, what's <flush>?

Also, there's a ~5sec lag between 'message 1 line 1' and the next message, although they are sent at the same time (yes, I'm sure + tested with outputdebug).

yes, I understand, I was wrong, I the auto-newline automatically accepts messages that do not have a newline but you want to ignore embedded newlines in a message.
The  message is generated when the process dies and messages not ending in a newline and still in the buffer.
The way it is currently implemented you can construct lines from multiple messages, which kindof the reverse of what you are asking.

I've put up a link for you:
http://www.myquest.nl/sites/debugview/DebugView++Drugoy_release.zip
this is an experimental version that does not split up message by their embedded newlines, however, it messes up the .dblog file. How would you want to deal with the newlines? convert them to spaces?

I'm not sure I understand you right, but if I did - I'd like to criticize it :)
Let's discuss things slowly and with examples, please.

I was wrong, I the auto-newline automatically accepts messages that do not have a newline but you want to ignore embedded newlines in a message.

Sorry, I don't understand. Could you give an example of what you mean?

The "flush" message is generated when the process dies and messages not ending in a newline and still in the buffer.

Do you mean that normally the debugger expects each message to end with a new line character (CR/LF/CRLF)?
Is this some kind of a standard/common advice or where did you take that from?

I've put up a link for you:
http://www.myquest.nl/sites/debugview/DebugView++Drugoy_release.zip
this is an experimental version that does not split up message by their embedded newlines, however, it messes up the .dblog file.

Err, again I'm not sure you got me right, or it doesn't work the way it should.
It works like this:

(the first result is with 'auto new line' option enabled, and the second one is with that option disabled).

How would you want to deal with the newlines? convert them to spaces?

No, I'd better have them shown somehow like this:

Ok, clear, @djeedjay will take a look on how to implement it.
About the newline behaviour that the 'debugger expects ', I'm not sure there are any 'rules' defined for this behaviour.

Is there a chance this issue would get actually fixed?
I see that the development has stalled, which saddens me a lot, because if not this issue and issue #208 - I'd be so happy to switch from DebugView to DebugViewPP.

@djeedjay had some thoughts on how to implement it, I myself do not know how to get this behaviour into the UI framework we use, but I will ask if he is interested to create it...


a small example how to use this package with angular & electron

could you help me with a small example how to add this package to angular project use electron js
thank you so much,you made a great package

how can use this package with angular?


setMapSize not working

Hi,
I have the following code,
		Env env = new Env();
		env.open(storage_path,Constants.MAPASYNC|Constants.WRITEMAP);
		long mapsize = 100*1024*1024*1024;  //size 100 Gb
		env.setMapSize(mapsize);
		EnvInfo envinfo = env.info();
		System.out.println("Map size = "+envinfo.getMapSize());

I get in the standard output,
Map size = 1048576
as soon as operations start, I get
org.fusesource.lmdbjni.LMDBException: MDB_MAP_FULL: Environment mapsize limit reached
at org.fusesource.lmdbjni.Util.checkErrorCode(Util.java:44)
at org.fusesource.lmdbjni.Transaction.commit(Transaction.java:82)
at org.fusesource.lmdbjni.Database.put(Database.java:384)
at org.fusesource.lmdbjni.Database.put(Database.java:374)
I verify that the data file is indeed 1048576. Any idea on how to overcome this?

This is how Java works. You're using an int to multiply then cast to long. The latest lmdbjni have a method setMapSize(size, ByteUnit)

Oh well, thanks for pointing it out. Now it works :-). I am closing this now.


fix: do not mismatch SendData requests that are received out of sequence

fixes: #1144

Wow, this one was tricky...


Support for LilyGo T-CAN485

https://github.com/Xinyuan-LilyGO/T-CAN485
more challenging design

Can you identify which poles corresponds to which GPIO ?

https://github.com/Xinyuan-LilyGO/T-CAN485/blob/main/doc/schematic.pdf
This PDF contain them page3

Not implemented yet.


Library crashes when no recording device is attached

When there are no active recording devices (disabling every recording device does the trick) the library dies on require and crashes node.

This also happens with speakers.


Openupm Support

This package is really great and could benefit from openupm. To be able to integrate the package to openupm, git tags with semantic versioning is required. If you could create a tag and release for the current version (currently: 0.3.6), this repo can be added to openupm.

I just added it, let me know if there is any issue ;) !

Thank you! I've left the URL to this package on OpenUPM below. I think it matches the versions there via git tags so you'll have to create a new tag for each new version.
OpenUPM URL


Improve overflow handling

Fix #2911

@stas00, FYI

So just to verify this PR:

refactors some code
separates the warning into 2 parts, only the 2nd part relevant to fp16
correct?

That looks great to me - thank you for improving it, Tunji!

Yes, and I have copied your apt description.
What remains are improvements to the error message:

Distinguishing the type of overflow: nans and/or infs

on transformers CI tests you're running into disc space issues:
Traceback (most recent call last):
  File "/tmp/actions-runner/_work/DeepSpeed/DeepSpeed/transformers/examples/pytorch/language-modeling/run_clm.py", line 625, in <module>
    main()
  File "/tmp/actions-runner/_work/DeepSpeed/DeepSpeed/transformers/examples/pytorch/language-modeling/run_clm.py", line 329, in main
    raw_datasets = load_dataset(
  File "/tmp/actions-runner/_work/DeepSpeed/DeepSpeed/unit-test-venv/lib/python3.8/site-packages/datasets/load.py", line 1731, in load_dataset
    builder_instance.download_and_prepare(
  File "/tmp/actions-runner/_work/DeepSpeed/DeepSpeed/unit-test-venv/lib/python3.8/site-packages/datasets/builder.py", line 586, in download_and_prepare
    raise OSError(
OSError: Not enough disk space. Needed: Unknown size (download: Unknown size, generated: Unknown size, post-processed: Unknown size)


Ef.update connections add account

Made changes to DB-API to pass account_name to SDK. No new tests were written.

This was replaced by #59.


ambiguous version info

version := 'whoami (V coreutils) 0.0.1'

Is the whoami v0.0.1?  Or Vcoreutils?
I'd like to suggest 1 of the following; hopefully the first:
// whoami version
version := 'whoami 0.0.1 (V coreutils)'

// V coreutils
version := 'whoami (V coreutils 0.0.1)'

Thanks for starting on this project!

Going by GNU style, we should use the second one. It also makes sense since all the coreutils are bundled together

I'm used to JavaScript land, where if there are many modules in a meta repo each module has its own version, since they can be required individually.  Perhaps you see this as only being distributed all together, I can see projects only using & requiring/repackaging a few commands....

True, but these are more user commands, so they would be packaged and used together. Even if they are used programmatically, they won't be downloaded/updated separately.


329 multicolour accordion

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--cn-website--netcentric.hlx.page/
After: https://329-multicolour-accordion--cn-website--netcentric.hlx.page/drafts/sdhekale/multicolor-accordian

hi changes requested are done have a look @nc-andreashaller.Thanks

A couple of concerns.

We need to make a more mobile friendly version when viewed on smaller viewports. @Gonzalo-CG Can you have a look at this please? Currently only a short text displays on mobile and we'd want a good experience for those visiting from mobile too.
I tried to add a link in one of the drawers to verify that works, but once I edited the accordion no longer shows properly on the page. Is there a trick to getting it display as an accordion after editing @sonaldhekale ?

Thanks!
Attaching mobile view

@sonaldhekale @alexnerdrumhansen as for the mobile view, Joel recommended reducing the font size & eliminating the padding on the left side.

Changes made make it look much better on mobile.
Can now also verify that links will open in new tabs.
Thanks @sonaldhekale !


Add accounts view.

Adds the account view.
Still needs work:
1). Needs the balance (acquired from mosaics).
2). Needs to last activity (acquired from the activity buckets).

@AnthonyLaw Thanks, good catch.


Adds Environment column to exported data

Hi @Bertverbeek4PS, first all of thank you for keeping the project alive and maintaining it!
We are currently using bc2adls for a couple of our customers where we have the requirement to extract data from multiple companies and environments and consolidate it across them all.
Therefore we simply added a $Environment column to the exported data.
Even if our requirement might not be that common I still consider this a good addition without any drawbacks.
I also considered simply editing the CopyBusinessCentral notebook to add the environment name by passing it via a parameter, but I think it's cleaner this way.
A little extra info:
I am aware that it is recommended to export data from different environments to different lakes, but we opted to not do it because it will involve creating additional pipelines/notebooks/sjd to handle the consolidation. To solve this instead I modified the CopyBusinessCentral notebook to handle data from multiple environments coming into the same lake. I didn't add it to the PR because it might not be relevant for everyone, but can do so if people might find it useful.

I see your point. There can't be duplicate companies in a single environment, but as far as I know there is no guarantee that the companies are unique across different environments.
I believe nothing is stopping a customer from doing something like this:
Environment-A / Company-A
Environment-B / Company-A

I see your point. There can't be duplicate companies in a single environment, but as far as I know there is no guarantee that the companies are unique across different environments. I believe nothing is stopping a customer from doing something like this: Environment-A / Company-A Environment-B / Company-A

@y0m0 indeed you can have the situation accross all environments. But in my eyes that would be very rare. Because it isn't possible to have that also for the taxes etc.
But if that is a use case maybe we can put it on a setting to export also the environment and also in a setting in the notebook. In that case the user can choose what he wants.
I can see that is a point for including the environment fields to it but then for consolidation reason.

Sounds like a sensible idea, I can modify my PR accordingly.

Sounds like a sensible idea, I can modify my PR accordingly.

Environment and Company GUID in the export would be extremely useful. These are needed in the situation where you refresh from Prod to Sandbox. When doing this since the extension data for BC2ADLS migrates in an active state, you run the risk of data from the new copy (refresh into Sandbox) pushing data to your configured (Prod) destination (ADLS or OneLake).

I see your point, but could we be overcomplicating things here? We have one customer with three environments, resulting in three lakehouses that are then merged into a single warehouse.
I'm thinking specifically about custom fields that vary across environments—these aren't always the same per table and could differ due to localization. Within a single environment, fields are consistent across all companies, so that isn't an issue, but across different environments, we do have these variations.

I see your point, but could we be overcomplicating things here? We have one customer with three environments, resulting in three lakehouses that are then merged into a single warehouse.
I'm thinking specifically about custom fields that vary across environments—these aren't always the same per table and could differ due to localization. Within a single environment, fields are consistent across all companies, so that isn't an issue, but across different environments, we do have these variations.

That is indeed a very good point @bjarkimg! Environments aren't always the same in schema and we cannot check that when exporting it. So I will close this pull request.


feat: fix missing titles

The idea is to let i18next handle determining which language needs to be returned which also allows for setting a fallback in case no value has been set for that variable.
What remains to be done: set the English (or French?) title to be required

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.391%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3269

Relevant Lines:
5696

💛  - Coveralls

@pierluca this PR shows two small changes, but in fact the entire validation/internalisation code needs to be updated
however, I prefer to integrate small gradual changes that fix visible problems to

avoid spending time on dead code (I'm pretty sure there's some)
limit testing to one functionality (and making sure that works now)
make quickly visible improvements instead of passing a lot of time on big changes

the idea being to move gradually in the right direction, is that OK for you?
Cc @ineiti @lanterno

sounds good. We'll discuss best next steps this Thursday...


Log Injection

since it is possible to inject information into the log (stdout), this is not secure
https://github.com/talkdai/dialog/blob/941ca28f851d4cbd56e875afa88631b89418fb6f/src/dialog/routers/dialog.py#L38-L38
https://github.com/talkdai/dialog/blob/941ca28f851d4cbd56e875afa88631b89418fb6f/src/dialog/routers/openai.py#L47-L47
References
OWASP: Log Injection.
Common Weakness Enumeration: CWE-117.
Tracking issue for:

[ ] https://github.com/talkdai/dialog/security/code-scanning/1
[ ] https://github.com/talkdai/dialog/security/code-scanning/2

@avelino is this a vulnerability? I've seen it on the scan, but as it's not identifiable, I thought it wasn't worth fixing it sometime.

Fixed!

Fixed!

@vmesel in which pr or commit?
we need to be easy to work with, for people to get to the solution in a simple way - the premise of being an open source solution

Sure @avelino, you are totally right!
Commits are:
Hotfix: Langchain version bumping for fixing https://github.com/advisories/GHSA-3hjh-jh2h-vrg6 - bumping dialog-lib to <IP_ADDRESS>
Bump urllib3 from 2.2.1 to 2.2.2 in the pip group across 1 directory
hotfix: Removes log injection vulnerability


Get 401 response whereas token is provided

In a Spring Boot application, I setup the configuration. Here is the configuration:
@Configuration
public class LogzMetricsConfig {

    @Bean
    LogzioConfig logzioConfig() {
        return new LogzioConfig() {
            @Override
            public String get(String key) {
                return null;
            }

            @Override
            public String uri() {
                return regionsUri().get("us");
            }

            @Override
            public String token() {
                return "XXXXXXXXXXXXXXXXX";
            }

            @Override
            public Duration step() {
                return Duration.ofSeconds(30);
            }
        };
    }

    @Bean
    LogzioMeterRegistry logzioMeterRegistry(LogzioConfig logzioConfig) {
        return new LogzioMeterRegistry(logzioConfig, Clock.SYSTEM);
    }
}

The token is passed as a Bearer in the authorization header
C:/Users/sydney/.m2/repository/io/logz/micrometer/micrometer-registry-logzio/1.0.1/micrometer-registry-logzio-1.0.1-sources.jar!/io/micrometer/logzio/LogzioMeterRegistry.java:132
                httpClient
                        .post(uri)
                        .withContent("application/x-protobuf", Snappy.compress(writeRequest.toByteArray()))
                        .withHeader("Content-Encoding", "snappy")
                        .withHeader(HttpHeaders.AUTHORIZATION, "Bearer " + config.token())

But I get the following response
Shipping token is missing. You must supply a token using one of the following ways: header, query string param, url path, message body. Please refer to Logz.io documentation to see which ways are suitable for your shipping method and how to implement it.

I doubled checked the token in my account, and it's correct.

Hey @sydneyhenrard can you please try to use:
@Override
public String uri() { return "https://listener.logz.io:8053"; }

Instead of :
@Override
public String uri() {
return regionsUri().get("us");
}

That should solve your issue.
I will check the regionsUri() function on our side.

It's working with the uri you provided


add infos about docker setups

/approve

Hey @remyleone,
Seems you have open the branch in the wrong repo.
Could you delete this remyleone-patch-1 branch from k/website? Thanks.


download issue

the program download only half of the video from the course the other half of the video is just viewing image where it stopped in the middle of the video

I'm having the same issue as Vrecce.. Is this great app still maintained? :(


Cells without size classes should span to the full width

Well, instead of setting a height on .cell, a new size class .cell-full (including its corresponding responsive size classes) should be introduced to keep separation of concerns.


Can we remove XML Settings file Support?

To the best of my knowledge, no one is using our XML settings files.  (I could be very wrong.)
And if no one is using them, I would really like to remove the functionality. There is a LOT of code and cruft in our codebase, and it would be a significant reduction in complexity to remove the functionality.
If no one is using them.

@jakehader @keckler @ntouran Does anyone use XML settings files in ARMI?

Not anymore. I use the xml geometry, but not settings.

Not anymore. I use the xml geometry, but not settings.

@jakehader
OH! Damn. I am most of the way through removing XML Geometry from the code base. I take it you missed this discussion earlier in the week?
https://github.com/terrapower/armi/issues/450#issuecomment-1069654528
Okay, well, I'll switch to removing XML settings.

Nah remove the xml geom and I'll just update the code that uses it! It's not that bad to switch to yaml.

Nah remove the xml geom and I'll just update the code that uses it! It's not that bad to switch to yaml.

Well, it sounds like I should remove the XML Settings first. If no one is using it, that's better to remove first. NBD
Also, if you have code that supports XML Geom, that'll be a bigger change than what I thought it would be.
No worries. Thanks for the info!

I agree that we can deprecate xml settings support.


chore: Added description for rule 942450

Description for SQLi hex evasion detection rule

Thanks @Xhoenix.

Good stuff.

Sorry for the delay, I didn't get around to merging this one in time as promised :laughing:
@Xhoenix Was this your first PR? If so, we'll need to add you to the list of contributors :tada:

Far from it.
$ git log | grep -e "Merge pull request.*Xhoenix"
    Merge pull request #3219 from Xhoenix/v4.0/dev
    Merge pull request #3215 from Xhoenix/v4.0/dev
    Merge pull request #3214 from Xhoenix/v4.0/dev
    Merge pull request #3213 from Xhoenix/v4.0/dev
    Merge pull request #3212 from Xhoenix/v4.0/dev


Snipping tool fails to load if Screen ruler is enabled

Microsoft PowerToys version
0.62.0
Running as admin

[X] Yes

Area(s) with issue?
Screen ruler
Steps to reproduce
Fire Screen ruler and then try to use Snipping tool.
Thank you for the great job! Screen ruler is an awesome tool!
✔ Expected Behavior
Use Screen ruler in combination with Snipping tool
❌ Actual Behavior
Snipping tool just crashes
Other Software
No response

@mccreadylemon, thank you for the report!
We've experienced this a couple of times. Unfortunately, it's likely an external API issue which we cannot directly affect, i.e. either GraphicsCapture.dll or Snipping Tool itself.

What's your Windows version?
Does rebooting help? (or logout->login)
Does switching between continuous and non-continuous mode in the settings help?

/needinfo

@yuyoyuppe  I've been experiencing this as well
To provide the information you requested earlier:

I'm running windows 11 21H2.
Both rebooting and logging out and back in fix the snipping tool
I've tried with both continuous and non-continuous modes and it happens with both

I can't figure out exactly what triggers the issue, as it doesn't happen every time I open and use the screen ruler tool.
When it does happen, the following error shows in event viewer:
Faulting application name: ScreenClippingHost.exe, version: 421.19500.0.0, time stamp: 0x60f735de
Faulting module name: ScreenClipping.dll, version: 421.19500.0.0, time stamp: 0x60f73587
Exception code: 0xc000027b
Fault offset: 0x00000000000211db
Faulting process id: 0x72b8
Faulting application start time: 0x01d8d1dcebf4a43c
Faulting application path: C:\Windows\SystemApps\MicrosoftWindows.Client.CBS_cw5n1h2txyewy\ScreenClippingHost.exe
Faulting module path: C:\Windows\SystemApps\MicrosoftWindows.Client.CBS_cw5n1h2txyewy\ScreenClipping.dll
Report Id: f2482dcf-b8b0-46d8-8347-4e73dd3badfc
Faulting package full name: MicrosoftWindows.Client.CBS_1000.22000.1042.0_x64__cw5n1h2txyewy
Faulting package-relative application ID: ScreenClipping

This is still an issue @yuyoyuppe
I have the same event viewer fault as the last comment.
Running PowerToys v0.66.0 and MS Version 10.0.22000.1335.
Opening the snipping tool and clicking 'new' just displays a mouse spinning circle then nothing launches.
Using the shortcut (WIN + Shift + S) does the same spinning circle then nothing.

I am experience the same issue
Faulting application name: ScreenSketch.exe, version: 10.2008.2008.14007, time stamp: 0x5f372032
Faulting module name: KERNELBASE.dll, version: 10.0.19041.2364, time stamp: 0x5b7d4d22
Exception code: 0xc000027b
Fault offset: 0x000000000010fe32
Faulting process id: 0x2ff8
Faulting application start time: 0x01d929ddfe41c578
Faulting application path: C:\Program
Files\WindowsApps\Microsoft.ScreenSketch_10.2008.2277.0_x64__8wekyb3d8bbwe\ScreenSketch.exe
Faulting module path: C:\WINDOWS\System32\KERNELBASE.dll
Report Id: f3a3b735-64b4-438c-b096-1a0625d58c68
Faulting package full name: Microsoft.ScreenSketch_10.2008.2277.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

Something interesting to note is that this issue occurs when i use WIN+SHFIFT+S to take a screenshot, and then open up the snipping tool by clicking the notification. Opening the snip & sketch directly, then taking a screenshot and using the ruler works properly, and fixes this issue for a bit.

I am experience the same issue
Faulting application name: ScreenSketch.exe, version: 10.2008.2008.14007, time stamp: 0x5f372032
Faulting module name: KERNELBASE.dll, version: 10.0.19041.2364, time stamp: 0x5b7d4d22
Exception code: 0xc000027b
Fault offset: 0x000000000010fe32
Faulting process id: 0x2ff8
Faulting application start time: 0x01d929ddfe41c578
Faulting application path: C:\Program
Files\WindowsApps\Microsoft.ScreenSketch_10.2008.2277.0_x64__8wekyb3d8bbwe\ScreenSketch.exe
Faulting module path: C:\WINDOWS\System32\KERNELBASE.dll
Report Id: f3a3b735-64b4-438c-b096-1a0625d58c68
Faulting package full name: Microsoft.ScreenSketch_10.2008.2277.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

Something interesting to note is that this issue occurs when i use WIN+SHFIFT+S to take a screenshot, and then open up the snipping tool by clicking the notification. Opening the snip & sketch directly, then taking a screenshot and using the ruler works properly, and fixes this issue for a bit.

I'm having the same issue as well, but it just does the loading circle on the mouse pointer. I recently updated and this is the result.
Faulting application name: ScreenClippingHost.exe, version: 622.32703.0.0, time stamp: 0x637de5ad
Faulting module name: ScreenClipping.dll, version: 622.32703.0.0, time stamp: 0x637de5a6
Exception code: 0xc000027b
Fault offset: 0x00000000000302d2
Faulting process id: 0x0x5B5C
Faulting application start time: 0x0x1D92A93A787F976
Faulting application path: C:\WINDOWS\SystemApps\MicrosoftWindows.Client.CBS_cw5n1h2txyewy\ScreenClippingHost.exe
Faulting module path: C:\WINDOWS\SystemApps\MicrosoftWindows.Client.CBS_cw5n1h2txyewy\ScreenClipping.dll
Report Id: e795d76d-789c-4701-bcc7-84eaffe9379c
Faulting package full name: MicrosoftWindows.Client.CBS_1000.22638.1000.0_x64__cw5n1h2txyewy
Faulting package-relative application ID: ScreenClipping

Windows 10 here. Please reopen. Author did provide feedback in time.

I'm facing the same issue on Windows 11 22H2 (22621.1992) with PowerToys v0.71.0


Triple-repeated statements

mgmt_core.v: 3342, 3374

@jeffdi :  Is this the same as issue #34 (pull request #35)?

I think these are different.  I believe the issue reported is due to a new version of litex than was used by us.  I can't find the issue being reported.  I will test further.


rename the project so it's findable

this project is called obsidian-zettlelflow which makes it impossible to find because the correct spelling is ZETTEL so suggest renaming the project to obsidian-zettelflow or just zettelflow for simplicity and to help people searching for zettelkasten plugins to find this one.
Good luck.

thanks for your suggestion, I will check those kind of typos

Changed!

Thanks.  You probably want to change the mis-spelling everywhere since it's still showing up in the docs.  👍🏾

Also was changed inside the code =P thanks again