[FLINK-17385][jdbc][postgres] Handled problem of numeric with 0 precision

What is the purpose of the change
Fix error when handling numeric field types with precision 0, such as:
CREATE TABLE orders(
    ord_no integer PRIMARY KEY,
    ord_date date,
    item_name character(35),
    item_grade character(1),
    ord_qty numeric,
    ord_amount numeric,
    CONSTRAINT unq_ordno_itname UNIQUE(ord_qty,ord_amount)
);

This address the same problem of SPARK-26538 (https://github.com/apache/spark/pull/23456), using the same fix.
Brief change log
In the PostgresCatalog.fromJDBCType, handle Decimal and Numeric types as follow:
if (precision > 0) {
	return DataTypes.DECIMAL(precision, metadata.getScale(colIndex));
}
return DataTypes.DECIMAL(DecimalType.MAX_PRECISION, 18);

Verifying this change
This change is already covered by existing tests, such as PostgresCatalogITCase.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving):  no
The serializers:  no
The runtime per-record code paths (performance sensitive):  no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper:  no
The S3 file system connector:  no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented? not documented

cc @bowenli86 , would you like to review this?

LGTM, except that can you also add array tests in PostgresCatalogITCase#testArrayTypes()?

@bowenli86 I'll add it within tomorrow hopefully

@flinkbot run travis
@flinkbot run azure

@flinkbot run travis

LGTM.
BTW, next time, please remember to write jira number and tags in the commit message, e.g. "[FLINK-17385][jdbc][postgres]"


Market card status

closes #313

I was going to adjust how the layout + scroll works market cards in this PR but I think it will require a larger overhaul and have opened a separate issue for it


Credentials path needs tweaking

we're storing things on a path credentials-pat.
We should tweak that.

Resolved by: https://github.com/alphagov/di-solid-prototype/pull/114
and... better resolution pending in https://github.com/alphagov/di-solid-prototype/pull/117


Create a dataflow, Step 4: There is no "Select load target" page, probably renamed into New Dataflow

Article: https://docs.microsoft.com/en-us/powerapps/maker/common-data-service/create-and-use-dataflows#create-a-dataflow
As is:
4. On the Select load target page, select the destination storage where you want entities to be stored. Dataflows can store entities in Common Data Service or in your organization's Azure Data Lake storage account. Once you select a destination to load data to, enter a Name for the dataflow, and then select Create.
Suggestion:
4. On the New Dataflow page enter a Name for the dataflow. By default dataflows store entities in Common Data Service. Select Analytical entries only if you want entities to be stored in your organization's Azure Data Lake storage account. Select Create.

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

ID: b3abf73d-60b9-675f-61e7-1c2dc622d055
Version Independent ID: 352c1196-0631-9f40-e382-bba13d5e27e9
Content: Create and use dataflows in Power Apps - PowerApps
Content Source: powerapps-docs/maker/common-data-service/create-and-use-dataflows.md
Service: powerapps
GitHub Login: @Mattp123
Microsoft Alias: matp

@AndreiPanko - thank you for your feedback. The article has been updated per your suggestion and to reflect the current feature UI. Closing issue.

@AndreiPanko - thank you for your feedback. The article has been updated per your suggestion and to reflect the current feature UI. Closing issue.


Go to definition not active in Go menu.

Describe the bug
Go to definition (F12) is available from the Editor context menu but the very same option is disabled in the Go menu.
To Reproduce
Click on the Go menu. The option is grayed out. Right click and the option is available.
Expected behavior
Should be active in both menus.

Thank you for this info! I can repro this issue.

This may not be our bug to fix, I have the described effect using TypeScript too.

Yes, this is a minor issue.


Request for contributor ORCIDs

📰 Custom Issue

Dear awesome python-stratify contributors!
python-stratify will soon be citable and contain a CITATION.cff file.
If you'd like to be recognised as a citable contributor then please share your ORCID details in a comment in this issue, and I'll add them in batch to the CITATION.cff. Alternatively, if you want to raise a pull-request yourself instead after the CITATION.cff file goes up, then go for it... whatever you prefer 👍
If you don't want to add your details, then that's also totally fine too.
Regardless, thanks for the interest and contributions to python-stratify.
Hopefully, I've not missed anyone from the list:

[x] @bjlittle
[ ] @pelson
[x] @lbdreyer
[x] @trexfeathers
[x] @bouweandela
[x] @evertrol
[ ] @tjtg
[ ] @bayliffe
[ ] @ocefpaf

@HGWright Awesome! Thanks for doing this.
My ORCID is 0000-0002-1345-9465 👍

Also, take a peek at https://github.com/SciTools/nc-time-axis/blob/main/CITATION.cff
There are several contributors to python-stratify that have also contributed to nc-time-axis 👍

Mine is 0000-0001-9005-8940

0009-0000-6857-560X

0000-0001-8357-4453
Thanks!

Mine is 0009-0006-6310-3658

Thanks! Mine is
0000-0003-4165-2913

This has been up for almost a month, I am going to close this now. But if you would still like to be added, feel to either create an issue or a pr with your info.
Thanks everyone!


Use importlib.resources to load colormap files

We currently use __file__ to get the path of files inside the Python module.
https://github.com/cogeotiff/rio-tiler/blob/4bd9be5db98827e40d4222627c3af59e9063bd95/rio_tiler/colormap.py#L14-L17
This does not always work:

This will not work if the files are in a distribution (IE. egg). Use pkg_resources to get at the data file

Historically the workaround was to use pkg_resources, however as we know this is very slow. I just learned that there's a new, more performant way to reliably access package data: importlib.resources. That was added in Python 3.7, but there's a backported library for earlier versions.
So you can import with:
try:
    import importlib.resources as pkg_resources
except ImportError:
    # Try backported to PY<37 `importlib_resources`.
    import importlib_resources as pkg_resources

E.g.
import importlib.resources
import rio_tiler
print(importlib.resources.read_text(rio_tiler, 'reader.py'))
# """rio-tiler.reader: image utility functions."""
# ...

See this very good SO answer for more info. We could probably also use this in other places. Where else do we have local package data? Titiler?

🎉

See this very good SO answer for more info. We could probably also use this in other places. Where else do we have local package data? Titiler?

morecantile
titiler
rio-viz
cogeo-mosaic-plugins
...
😬

To access a file inside the current module, set the package argument to __package__, e.g. pkg_resources.read_text(__package__, 'temp_file')

👀

@kylebarron not sure what the previous message is about


Accessories not appearing in Homekit

Describe Your Problem:
I have been able to setup the plugin as a child bridge. I have added the bridge to homekit as a child bridge and can see the bridge there.  The # of accessories associated with the bridge is listed in homekit, but when drilling in, no accessories display. I cannot add any of the accessories to homekit using the "add accessories" function as they don't appear.


I am facing the same issue... please suggest

    I am facing the same issue... please suggest

Originally posted by @deepak21singh in https://github.com/wso2/carbon-apimgt/issues/4953#issuecomment-599178951
Description:
API Gateway is not populating/appending x-forwarded/forwarded headers. Currently APIM simply forward those headers as is. Hence what back-end receives as the x-forwarded (say x-forwarded-host) information is, that of the client, not of the API gateway. We need to discuss this across team and come to conclusion about behavior. Logging this issue for recording purpose.
Suggested Labels:
apim,3.0.0
Suggested Assignees:
Affected Product Version:
2.2.0
OS, DB, other environments details and versions:
Steps to reproduce:
Related Issues:

This issue is moved from product microgateway to api manager.


45-Create end point to update trainees details -Olena

in traineeController.js
line #8 const { traineeId } = req.params;
may need to be replaced by
const { id} = req.params;
line #12
const result = await db.query(q, [traineeId]);
may need to be replaced  by
const result = await db.query(q, [id]);
endpoint updates trainees' details  according to the table schema (id, gitHub_user_name, full_name, cohort_id)

@PeteLindsell review please


train time

How long does it take to train the coco dataset? @lufficc Thank you.

vgg_ssd512_coco_trainval35k: 4 GPU, about 2 days, 0:12:19
vgg_ssd300_coco_trainval35k: 2 GPU, about 1 day, 10:14:12


[Snyk] Upgrade @graphprotocol/graph-cli from 0.65.0 to 0.67.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @graphprotocol/graph-cli from 0.65.0 to 0.67.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 15 versions ahead of your current version.
The recommended version was released 21 days ago, on 2024-01-18.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fb77ee4) 77.83% compared to head (7b5e713) 77.83%.

:exclamation: Current head 7b5e713 differs from pull request most recent head e2d6d8f. Consider uploading reports for the commit e2d6d8f to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2093   +/-   ##
=======================================
  Coverage   77.83%   77.83%
=======================================
  Files          40       40
  Lines        3090     3090
  Branches      629      629
=======================================
  Hits         2405     2405
  Misses        479      479
  Partials      206      206

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


Table - Tie selectedRowKeys to selectedRows Prop

If the prop updates by an external force, we need the table to mimic the outside property if possible. I think a watcher will fix the issue, but I'll need to try it out to be sure that it doesn't cause a weird side-effect.

If you use the original array and perform a splice to remove the element instead of a filter, the table updates which is a viable workaround so I'm going to de-prioritize this from 1.6.2 unless I find it ultimately necessary to modify.


make our api able to provide authorised transactions

maybe on top of authenticating we also provide an authorisation interface.

this issue is closed because it is not executable (lack use-cases where this is clearly defined.)


packer-rocm: refactor, multi-version

Refactoring the rocm playbook and adjacent parts in Packer. Moving away from single-release installation layout to multi.
We assume the latest release for amdgpu, allowing as many ROCm releases as the VM builder can hold. The provided disk limit may be fitting. Please consider raising this when adding more ROCm releases.

It is possible we regressed on proxy support with this; I haven't tested since introducing apt-key/the {apt,yum}_repository modules, effectively replacing amdgpu-install

Ready for review, I believe. Most of the commits are noise. Can be left out of the summary if we mind history 😄
Further work may lead towards making this play an actual role, separating the OS tasks by files. We could define an execution environment that makes this even more reproducible


pull request for e2e test at 01/30/2017 09:56:12

:white_check_mark: Validation status: passed
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.


Some problem when used this extension with NflxMultiSubs extension

When I enabled netflix-1080p and NflxMultiSubs extension together.
The NflxMultiSubs extension its can't working
but netflix-1080ps extension still working.
I try with google chrome and microsoft edge. It's still same problem.
Link NflxMultiSubs extension (developer doesn't open the Issues) : https://github.com/dannvix/NflxMultiSubs

It's work for me in Microsoft Edge.
https://github.com/lkmvip/netflix-4K-DDplus/releases/tag/1.0


Removes trait AsDataLakeClient

This is a follow-up that came out of #358.

@rylev this is the follow-up I promised in #358 and as discussed yesterday. I tried keeping this first commit as small as possible but I did include removing the use of Arc to match the CosmosDB crate. I'm new to Arc so particularly interested in feedback there; I should probably remove Arc from data_lake clients more broadly.

I attempted to remove AsStorageClient but this impacts areas I don't want to change in this PR, e.g. queue. Reverted those changes and limiting PR to data_lake only.


gb vendor fetch git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git fail

go get git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git
works, but
FATAL command "git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git" failed: unknown repository type
FATAL command "vendor" failed: exit status 1```

reading the code:
https://github.com/constabulary/gb/blob/master/cmd/gb-vendor/vendor/repo.go#L62
looks like it only support GH and bitbucket

This seems to be a dup of #178 ?

Not quite. Does the go tool support this import path ?
On Wed, Jun 3, 2015 at 6:10 AM, Luna Duclos<EMAIL_ADDRESS>wrote:

This seems to be a dup of #178
https://github.com/constabulary/gb/issues/178 ?
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/182#issuecomment-108082225.

yes the import path is comical, but works with go tools:
go get git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git
then
import "git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git"
http://godoc.org/git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git

Ok, this is different. We will add support for that form.
On Wed, Jun 3, 2015 at 6:21 AM, Julien Vermillard<EMAIL_ADDRESS>wrote:

yes the import path is comical, but works with go tools:
go get git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git
then
import "git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git"
http://godoc.org/git.eclipse.org/gitroot/paho/org.eclipse.paho.mqtt.golang.git
—
Reply to this email directly or view it on GitHub
https://github.com/constabulary/gb/issues/182#issuecomment-108086895.

Something similar with apache's git repo:
FATAL command "fetch" failed: Get https://git.apache.org/thrift.git/lib/go/thrift?go-get=1: dial tcp <IP_ADDRESS>:443: connection refused
FATAL command "plugin" failed: exit status 1```

Hello, I took a look at this on the weekend. It shouldn't be that hard to
implement.


password authentication failed for user "api"

2023-07-07 10:41:29.083 UTC [6134] STATEMENT:  SELECT organization.id, organization.uuid, organization.display_name, organization.namespace, organization.bot_url, organization.status, organization.created_at, organization.updated_at FROM organization WHERE organization.namespace = 'openai';
07/07/2023 16:15:15
2023-07-07 10:45:15.963 UTC [26] LOG:  checkpoint starting: time
07/07/2023 16:15:21
2023-07-07 10:45:21.386 UTC [26] LOG:  checkpoint complete: wrote 55 buffers (0.3%); 0 WAL file(s) added, 0 removed, 0 recycled; write=5.411 s, sync=0.006 s, total=5.424 s; sync files=42, longest=0.004 s, average=0.001 s; distance=157 kB, estimate=157 kB
07/07/2023 16:22:42
2023-07-07 10:52:42.014 UTC [7501] ERROR:  operator does not exist: character varying = integer at character 233
07/07/2023 16:22:42
2023-07-07 10:52:42.014 UTC [7501] HINT:  No operator matches the given name and argument types. You might need to add explicit type casts.
07/07/2023 16:22:42
2023-07-07 10:52:42.014 UTC [7501] STATEMENT:  SELECT organization.id, organization.uuid, organization.display_name, organization.namespace, organization.bot_url, organization.status, organization.created_at, organization.updated_at
07/07/2023 16:22:42
	FROM organization
07/07/2023 16:22:42
	WHERE organization.status = 2
07/07/2023 17:07:02
2023-07-07 11:37:02.307 UTC [11304] FATAL:  password authentication failed for user "api"
07/07/2023 17:07:02
2023-07-07 11:37:02.307 UTC [11304] DETAIL:  Connection matched pg_hba.conf line 100: "host all all all scram-sha-256"
07/07/2023 17:07:14
2023-07-07 11:37:14.658 UTC [11321] FATAL:  password authentication failed for user "api"
07/07/2023 17:07:14
2023-07-07 11:37:14.658 UTC [11321] DETAIL:  Connection matched pg_hba.conf line 100: "host all all all scram-sha-256"
07/07/2023 17:20:16
2023-07-07 11:50:16.619 UTC [26] LOG:  checkpoint starting: time
07/07/2023 17:20:16
2023-07-07 11:50:16.933 UTC [26] LOG:  checkpoint complete: wrote 4 buffers (0.0%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.303 s, sync=0.003 s, total=0.314 s; sync files=4, longest=0.003 s, average=0.001 s; distance=0 kB, estimate=141 kB````

what is the reason for this error and how to resolve it

update DB_* from .env before install


Add validation for "Primary key already exists"

When the entity id is not automatically generated, it's possible that an insert will trigger a primary key violation. That's a pretty common case, and a custom validator would be needed in order to avoid reporting it as an "unknown error".
Should we provide some support for this scenario in commons-backend? (either as an ad-hoc validator, or as some exception postprocessing.)

Detecting this while trying to save an existing ID could be useful to force the user to use the right method (save or update) for the action that he's trying to do, because if you use save with an existing ID, the entity will be updated. Maybe the user tried to persist a new entity but ended up updating an existing one, without noticing it.

I'm not sure if Commons Backend should do something in this scenario. Whether the entity id is automatically generated or not is a JPA mapping decision. Implementing this would require to "configure" the library in a "automatically generated id" mode so the validation is useful or not.


Use unstable libc++ ABI

This means users can benefit from all the latest libc++ improvements and bugfixes. For more information see
https://libcxx.llvm.org/DesignDocs/ABIVersioning.html
It is already the case that different versions of LLVM Embedded Toolchain for Arm are not expected to be ABI compatible.

@smithp35 good point, thanks. Done, and added an entry to the changelog as well.


bigchain -> BigchainDB

As @r-marques suggested on Slack, maybe just change bigchain -> BigchainDB in line 506 for now (i.e. in the GENESIS block). We can consider changing other instances of "bigchain" in the future.

PR #82 is the new version of this PR so I'll close this one without merging.


[Discussion] Why gdscript and not something else?

Wouldn't the speed be better using C# or something else? I'm not trying to offend anyone, but I'd really like to know what was the benefit of picking gdscript. Read multiple times that iteration is faster in other languages etc.

Sure, it would work much faster in almost any other language.
Not sure, how to answer this question. I guess, I can start by describing how I use Godot - it's mostly on game jams (specifically, Ludum Dare) and small experimenal projects. So, I didn't spend time learning and setting up everything necessary to use any better language. Also, I don't know C# and don't have plans to learn it yet. This addon was started as one of small experimental projects - as a demo for a lecture about WFC algorithm. But it has grown too complex too quickly, so I had to make a simpler demo.
To summarize:
Why not C#?

I don't know it, and don't want to learn it yet
It doesn't work for web builds at least with current version of Godot. GDScript works, just fine, even with multithreading.

Why not any other language using GDExtensions:

Difficult to setup both for me and users of the addon

Maybe, later I will rewrite this addon, or, at least, it's most critical parts in different language. Most likely, it will be Rust. But for now it will stay with GDScript despite how much it (GDScript) sucks (and not just in performance).


404 when trying to http a json

Hi,
my former code allowed me to http a json but now I use this seed, I get 404's
I am not sure if it is a webpack issue or something
this.http.get('app/translations/en.json') does not work anymore
any hint/help ?
thanks

stupid of me, I have put things in public, now it's working

tried leaving out app/?
On Thu, Mar 30, 2017 at 2:37 PM, phil123456<EMAIL_ADDRESS>wrote:

Hi,
my former code allowed me to http a json but now I use this seed, I get
404's
I am not sure if it is a webpack issue or something
this.http.get('app/translations/en.json') does not work anymore
any hint/help ?
thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/preboot/angular-webpack/issues/326, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ACWx9lzSoLZBAk1yz25JXQBhVCqFmzE-ks5rq6IigaJpZM4MuTB1
.

yes I did that too
but I try to debug it and I cant see the ts files in chrome, I cant open any files, wonder if the .map files went through

adding # did the trick...(I think) I think it's an issue with that version of webpack
https://github.com/webpack/webpack/issues/2145
if (isProd) {
config.devtool = '#source-map';
}
else if (isTest) {
config.devtool = '#inline-source-map';
}
else {
config.devtool = '#eval-source-map';
}


Change KestrelServerOptions.AllowSynchronousIO default to false

KestrelServerOptions.AllowSynchronousIO is going to default to true for 2.0.0. Hopefully we can change this to false in our next major release.
https://github.com/aspnet/Announcements/issues/252#issuecomment-312058065
@muratg @Tratcher  We'll probably want to file a similar issue for HttpSysServer.

@Tratcher could you file a bug on HttpSysServer to track this for 3.0.0?

https://github.com/aspnet/HttpSysServer/issues/373

We need to do this again in 3.0 https://github.com/dotnet/corefx/issues/32665


Bug on supporting bit_cast<uint64_t>(double)

Following code will cause internal error:
RWStructuredBuffer<uint64_t> out;

uint64_t cast(double x)
{
    return bit_cast<uint64_t>(x);
}

[shader("compute")]
[numthreads(1,1,1)]
void computeMain()
{
    double x = 1.0;

    out[0] = cast(x);
}

note, there is no problem just doing
  out[0] = bit_cast<uint64_t>(1.0lf);

The error message is:
test1.slang(6): internal error 99999: unimplemented feature in Slang compiler: unexpected IR opcode during code emit
    return bit_cast<uint64_t>(x);

I tried other targets
It has no problem on cpp, cuda, spirv.

on all the shader language targets, we don't handle double base type during processing kIROp_BitCast.

There is a TODO in hlsl target
// TODO: There is an asdouble function
// for converting two 32-bit integer values into
// one double. We could use that for
// bit casts of 64-bit values with a bit of
// extra work, but doing so might be best
// handled in an IR pass that legalizes
// bit-casts.
//

We should work on top of https://github.com/shader-slang/slang/pull/5020 for this issue.


Installation with pip without conda?

Can we install this without conda?
I was trying pip install kalpy-kaldi that didn't work, also tried to build this with pip3 install .
also python3 setup.py build  which was most helpful since it complained about not seeing headers from kaldi:
/opt/kalpy/extensions/chain/pybind_chain.h:6:10: fatal error: chain/chain-datastruct.h: No such file or directory
    6 | #include "chain/chain-datastruct.h"

despite having set
export KALDI_ROOT="/opt/kaldi"
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:"$KALDI_ROOT/src/lib"

I'm not using conda because I want to build this inside docker for aarch64, so conda doesn't work very well...

Hmm so to install it, I would need latest OpenFST, which main branch of Kaldi doesn't support?


Add basic support for SteelSeries Rival 100

First commit fixes report rate changing for version 1 devices.
Second commit adds support for Rival 100. As stated in the commit message, neither button nor LED configuration changing doesn't work. DPI changing works almost flawlessly (can't switch between first and second resolution), but polling rate changing doesn't work fully for now because of #1126.
Edit: forgot to mention: it doesn't support reading any data, including report rate and resolutions.

What a mess I made here... I didn't think at first that it would make no sense to force push if it's not before merging...
So, I think this is as far as I can get. I still didn't implement buttons.
To support LEDs I had to add an option to device configuration file, and I'm not sure it has a proper name please check it. Other than that, I think PR is ready for a proper review.

I will remake this patch later today taking into consideration comments I made.


Can it be more ergonomic for maintainers to modify PRs

This extension is great at what it does, it lets me checkout PRs locally however the branch naming is pr/[user]/[PRNum].
As a maintainer, this means its not easy for me to make quick changes and push them back up to the PR (assuming the user allowed modification from the maintainer).
Is there a way to make this process smoother? Like set the default remote to the users remote, and set the branch name to match what their branch name is on the PR? I understand this could cause a clash if the user just uses "master" for example.
Thanks

As a maintainer, this means its not easy for me to make quick changes and push them back up to the PR (assuming the user allowed modification from the maintainer).

Even with the differing branch name you can still push changes to the PR branch. The command is
git push <pr user's remote> HEAD:<pr user's branch>

Ideally though we'd set that for you in the extension.


run docs nightly too

This PR configures kvikio docs builds to also run nightly (not just on PR merges only)

Thanks AyodeAwe

/merge


USB printer support

Currently this plugin only supports networked printers however the brother_ql library supports USB printers as well. I have tested this by changing the send_label function.

@timmyhadwen interesting, would you be able to submit a PR to provide USB functionality?

I have a fork with just hardcoded values for device address etc. More than happy to complete that implementation with settings etc however just struggling to get the plugin installed in a development environment? Do you have any tips for easy dev of plugins etc? I am currently using Docker and have the plugin cloned into the plugins folder but alas it won't install properly and can't see a nice way to get logs out. Any help or pointing in the right direction would be great :)

If you have the plugin cloned directly into the external "plugins" directory, it won't detect and load, as it is buried too "deep" in the directory structure.
Put the invenrtee_brother subdirectory into the "plugins" directory, and then it should load. Can you check if this works for you?

The easiest thing for me is using devcontainers in combination with editable installs. I described them on the docs a while ago: https://docs.inventree.org/en/stable/develop/devcontainer/?h=devcont#plugin-development

@timmyhadwen did you get it working?

Thanks for your help guys. Work has been busy but hoping to get into a few PRs over the Christmas break. Will update soon.

Hello. I needed the USB connection as well and implemented an additional setting for the plugin. Seems to work fine. You find the changes in my fork, if you're still interested I can create a PR.

@RealNagus a PR would be greatly appreciated!


Add To F-Droid (FOSS Repository)

If you're not familiar, F-Droid is a repository of pre-built FOSS Android apps. You should consider adding this project!

Latest releases are downloadable from
https://github.com/yukuku/androidbible/releases
Unless you want to maintain the app in F-Droid, in that case I will let you handle the updates there.

For F-Droid inclusion, you'd need a FOSS build flavor. Currently, your app includes 2 proprietary analytics modules (Crashlytics, Google Analytics), plus GMS, which are not acceptable by F-Droid. F-Droid builds apps only from source.
Could you provide suh a FOSS build, @yukuku? Currently, thanks to the APKs you provide, your app is in my repo. But even I would prefer at least those trackers gone. Honestly, you don't need two of them, do you?

@IzzySoft  Thank you for having the APK on your repo. Let me check if we can have a build flavor that does not include Google Play Services libraries.

Thanks @yukuku – preferably without the two Analytics modules as well. Then we might try for f-droid.org if you wish.
PS: If you attach a second APK, please let me know so I can tell my updater what "name pattern" to pick. And if you need some Analytics, there's e.g. ACRA which is FOSS and (by default) privacy-friendly (opt-in).

@yukuku more than 4 years later, still no chance?

That looks really bad for an app dealing with quite personal things – and I might have to remove it from the IzzyOnDroid repo, where it is still listed.


  Same thing happening with neko-rooms.  If I create gpu docker container by neko rooms WEBRTC not reachable and docker inspect showing
            "DeviceRequests": [
                {
                    "Driver": "",
                    "Count": -1,
                    "DeviceIDs": null,
                    "Capabilities": [
                        [
                            "gpu"
                        ]
                    ],
                    "Options": {}
                }
            ],
            ```
            I tried to manually attach /dev/nvidia0. Unfortunately  didn't work too

Currently known issue for Google Chrome that it does not work well with GPU. https://github.com/m1k1o/neko/issues/279


Stock Visual enhancements

Hello,
I've tried removing the clouds.cfg file stated below (and restarted CKAN) but I still get this error:

Oh no! We tried to overwrite a file owned by another mod!
Please try a ckan update and try again.
If this problem re-occurs, then it maybe a packaging bug.
Please report it at:
https://github.com/KSP-CKAN/CKAN-meta/issues/new
Please including the following information in your report:
File           : GameData/StockVisualEnhancements/EVE/Atmosphere/clouds.cfg
Installing Mod : StockVisualEnhancements 1:0.6.4
Owning Mod     : SVE-UltraResolution
CKAN Version   : v1.16.1-0-g2e91715 (beta)

I've added my .ckan - any suggestions?
mods.zip

StockVisualEnhancements 1:0.6.4 is for KSP 1.1
SVE-UltraResolution doesn't exist for KSP 1.1.
You need to uninstall SVE-UltraResolution completely then update StockVisualEnhancements

Yep. There are no configuration options now. Use all the memory!


Test for EKS with AMI

I have written a test for check of new version of Kontain and the kubernetes daemonset yamls with EKS launched with original Amazon Linux 2 AMIs.  This is mainly so that we can test our daemonset and new version of Kubernetes yamls with every release.
But once this is tested, we need to generate new AMI with KKM pre-installed and launch EKS nodes with that AMI.
This issue identifies this need.


BUG - in navigation bar

Describing the bug
When navigating from the navigation bars on index.html page to the menu.html or services.html or cart.html or contact.html the user is not directed to the particular section as expected.
Expected behavior
Clicking on the navigation bars  on the index.html page should take the user back to the particular section to the other menu.html , contact.html , cart.html.
Screenshots :

I am contributing under GSSoc24 open source Program .Please assign me this issue.

Hey @khushi-joshi-05 I'm ready to tackle the bug in the navigation bar. Could you assign me this task with the GSSOC tag?

I can fix this issue. Please assign this to me under GSSoC 24. Also, the links open in a new tab each time we click a navbar link which creates a mess in the tab bar of user's browser which does not feel good. I can take care of that too.

@khushi-joshi-05 please assign me this issue  as i have discussed and issued above.

This issue has been solved, Try to contribute to other recent issues!


RaftOptions.java missed several methods

Hi Wenweihu,
Thanks for your effort. It's really a good and interesting project. However, there are several methods missed in the RaftOptions.java in the com.github.wenweihu86.raft.aftNode.java.
I have added the following methods but it seems still not sufficient. An interesting thing is it seems that those missed methods exist in the corresponding .class file.
public void setDataDir(String dataPath) {
	dataDir = System.getProperty(dataPath);
}

public String getDataDir() {
	return dataDir;
}

public void setSnapshotMinLogSize(int size){
	snapshotMinLogSize = size;
}
public void setSnapshotPeriodSeconds(int timer){
	snapshotPeriodSeconds = timer;
}
public void setMaxSegmentFileSize(int size){
	maxSegmentFileSize = size;
}

public int getMaxSegmentFileSize() {
	return maxSegmentFileSize;
}

public long getCatchupMargin() {
	return catchupMargin;
}

public int getSnapshotMinLogSize() {
	return snapshotMinLogSize;
}

public int getSnapshotPeriodSeconds() {
	return snapshotPeriodSeconds;
}

public int getRaftConsensusThreadNum() {
	return raftConsensusThreadNum;
}

public boolean isAsyncWrite() {
	return asyncWrite;
}

public int getMaxAwaitTimeout() {
	return maxAwaitTimeout;
}

Please install Lombok plugin in your IDE.


Add mdbook tests to CI

See: https://rust-lang.github.io/mdBook/cli/test.html

Can probably just add to CI's main.rs

Could not get working: https://github.com/rust-lang/mdBook/issues/394 no longer seems to work for me as dependencies cannot be found.
This can be re-opened if someone figures out how to get it working.

Looks like a bug in upstream. Leaving this open with low importance; if anyone feels like tracking down and fixing this issue for upstream I'd be happy to add this functionality.


Running the test script causes this error

This happens when I run "yarn test" in my normal Windows folder structure with my create-react-app created project under my DropBox folder:
ReferenceError: [BABEL] C:\Users\Bruce\Dropbox\Projects Learning\Blue-Duck-Beta\src\containers\Appnav\tests\navbar.test.js: Using removed Babel 5 option: C:\Users\Bruce\Dropbox.babelrc.stage - Check out the corresponding stage-x presets http://babeljs.io/docs/plugins/#presets
If I move the create-react-app created project outside of the DropBox folder, the error goes away.
Googling this error shows many people asking about this problem, but there is no answer that I can apply to the hidden build created by your program (BTW, I'm loving it after trying many boilerplate frameworks). Is there anything you can help with? I would like to have my project backed up on DropBox.

This might be related to #949.
I have a PR to fix this: #1017.

Please delete C:\Users\Bruce\Dropbox\.babelrc.

Indeed this will be solved in a future version, but it's still going to bite you in other projects.

On Mon, Nov 14, 2016 at 7:16 PM, Dan Abramov<EMAIL_ADDRESS>wrote:

delete C:\Users\Bruce\Dropbox.babelrc.

I did that. It fixed my problem. Thanks for the reply.


Question: where to define inflectors?

I had an issue where I had put
$this->container->inflector(ContainerAwareInterface::class)
            ->inflect('setContainer', [ContainerInterface::class]);

inside the register method of a ServiceProvider. The register method wasn't called because the resolved object wasn't referenced in the container as a ContainerAwareInterface implementation object.

Container 2 will have a BootableServiceProvider which will allow you to do this.

so there will be a BootableServiceProviderInterface. but what if I'd have this:
class AppServiceProvider extends \League\Container\ServiceProvider implements \League\Container\ServiceProvider\BootableServiceProviderInterface
{
    public function register()
    {
        //
    }

    public function boot()
    {
        $this->container->inflector(ContainerAwareInterface::class)
            ->invokeMethod('setContainer', [ContainerInterface::class]);
    }
}

class EventingServiceProvider extends \League\Container\ServiceProvider implements \League\Container\ServiceProvider\BootableServiceProviderInterface
{
    public function register()
    {
        //
    }

    public function boot()
    {
        $object = $this->container->get(ContainerAwareObject::class);
        // ...
    }
}

What if these are registered in the opposite way? There's shouldn't be a order constraint between serviceproviders.

@hannesvdvreken do you have a suggestion on how to achieve this? As it is, for eager loading there has to be an order constraint.

Nah, I don't know. When you want to simply define this SP and make it work:
class AppServiceProvider extends \League\Container\ServiceProvider
{
    protected $provides = [ContainerAwareInterface::class];

    public function register()
    {
        $this->container->inflector(ContainerAwareInterface::class)
            ->invokeMethod('setContainer', [ContainerInterface::class]);
    }
}

You'd need to go up the inheritance tree for every resolved object to see which interfaces it implements and which classes it extends. That's a potentially expensive reflection operation.
Maybe, just maybe:
class ServiceProvider
{
    protected $inflectors = [ContainerAwareInterface::class];

    public function hasInflectorsFor($object) {
        foreach ($inflectors as $inflector) {
            if ($object instanceof $inflector) {
                return true;
            }
        }
    }

    return false;
}

Once you start looking in to bypassing order constraints more than I already have with the lazy loading (I know this doesn't help for inflectors) then the complexity really starts to grow.
I'm improving inflector functionality this week and that might help the situation from a user land perspective so I'll give you a shout once I've pushed that stuff.

Great. No pressure. Just letting you know how I use it and what real life problems arise ;-)

Appreciated :+1:

If it stays like it is now, also fine.

What I'm thinking is that there is now a service provider aggregate that stores all service providers, a second argument could be passed to the addServiceProvider method that are the dependency service providers of the one you are registering. That way, before register is invoked, the register method of the dependencies will be invoked (if they haven't been already).

Or they could be defined in the service provider itself in a similar way to the provides array/method. Thoughts?

Not a fan of that. Service Providers don't really have to know which other SP can register something. If I want to hand the container my own Service Provider which builds a certain type of object in a much better way with my own dependencies...

Yeah, that's my thoughts, it's too much, but it is the only way around the order constraints.


Visio export fails to produce a working .vsdx file

I imported a simple Visio .vsd file, heavily modified it and tried to export it again, but the resulting .vsdx file cannot be open by the official MS Visio 2016 viewer (reported error "The Visio viewer did't succeed to complete the operation").
Moreover, if I try to re-import the .vsdx file in draw.io, I find it a little bit messed up: some objects disappear behind other ones, text alignment is often wrong, line spacing seems to be based on Helvetica 12 size even if the actual text is smaller (Helvetica 10), position of text in the boxes is not respected (always set to center), and some other minor issues.
I attach here my original draw.io file and the exported .vsdx file for your review; I tried both Mozilla Firefox Quantum rel. 68.0.1 (64 bit) and Microsoft Edge as browser to run draw.io, but the result has been the same.
Example.zip

Also have a problem with exporting to vsdx, would be great if you fix it 😊

Please avoid “me too” comments. They create noise in the thread without adding anything to the issue. Thumbs up on the first comment is the convention in github to indicate this.

The corrupted vsdx output is fixed. We are still working on improving the export quality


Replace the --defsym linker argument with an alias in code

This has the same effect as the
defsym=__CFConstantStringClassReference=_T010Foundation19_NSCFConstantStringCN
option but does it in code which allows simplifying the linker arguments.

This will help with the -static-stdlib option since less linker arguments will need to be provided by swiftpm

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test

Ive made the alias conditional on it not being on macOS which worked when I built it on xcode and linux

There is one other symbol to watch out for: __CFSwiftGetBaseClass
As it stands there is actually a android specific define for __CFConstantStringClassReference in CFInternal.h; perhaps we can elide that if check and convert it to something like this:
#if DEPLOYMENT_RUNTIME_SWIFT
#if DEPLOYMENT_TARGET_LINUX
#define __CFConstantStringClassReference _T010Foundation19_NSCFConstantStringCN
#else
#define __CFConstantStringClassReference _T015SwiftFoundation19_NSCFConstantStringCN
#endif
#endif

Since CF is only used in Foundation via swift we should be ok by defining the compiler reference as listed and wholesale remove the symbolic aliases file definitions.
SwiftPM should not need to access the __CFConstantStringClassReference since no CF code should ever be used outside of Foundation on linux (in the realms of swift projects)
Moreover it might be nice to have them as defined values since we could perhaps somehow have a define that matches the toolchain emission (as a future enhancement) to prevent this breaking if the mangling changes ever again.
#ifndef __CFSwiftGetBaseClass
#if TARGET_OS_LINUX
#define __CFSwiftGetBaseClass _T010Foundation21__CFSwiftGetBaseClasss9AnyObject_pXpyF
#elif TARGET_OS_MAC
#define __CFSwiftGetBaseClass _T015SwiftFoundation21__CFSwiftGetBaseClasss9AnyObject_pXpyF
#endif
#endif

I dont really want to do too many changes in this PR especially since I cant test the android one, Id rather do a separate one for that.
Is the CoreFoundation//Base.subproj/SymbolAliases only used for the one __CFConstantStringClassReference alias and the rest are ignored? Also, what is the difference between that file and DarwinSymbolAliases, they currently have the same contents.

@phausler I redid the PR to use the #define for all targets as you suggested and removed the entry from the SymbolAliases file. I couldnt find a way of extracting the mangled name since libFoundation cant be built until CoreFoundation is compiled. Perhaps there needs to be a swift mangle command added in the future.
@modocache Do you still run the CI for Android? If so could you check that my change hasnt broken it on Android - I dont think it has since its still the same #define just in a different #if
I tested it on both Linux and Xcode and didnt see any issues

@phausler Do these changes look ok now? I merged the linux and android #define

Yea, ideally we will want to migrate to something similar to @_cdecl in the future but for now I think this is fine

@swift-ci please test

@phausler I think the test worked but didn't update GitHub. Can this be merged now?


Go live on 5/15 with new Windows Terminal docs

Docs Build status updates of commit e944448:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/terminal, working directory: W:\2zqc-s, target branch: HEAD, target branch commit: fcac84384bb0367295664a2d3da6db5b2461393a, source commit: e944448036c97faa3879014e16f0854aa6faafa4.

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.

Docs Build status updates of commit d4c829c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

TerminalDocs/new-terminal-arguments.md
:warning:Warning
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

TerminalDocs/Customize settings/custom-keybindings.md
:white_check_mark:Succeeded

TerminalDocs/TOC.yml
:white_check_mark:Succeeded
View

TerminalDocs/cascadia-code.md
:white_check_mark:Succeeded
View

TerminalDocs/command-line-arguments.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/frosted-glass-theme.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/powerline-in-powershell.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/raspberry-ubuntu.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/retro-command-prompt.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/color-schemes.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/global-settings.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/key-bindings.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/profile-settings.md
:white_check_mark:Succeeded
View

TerminalDocs/dynamic-profiles.md
:white_check_mark:Succeeded
View

TerminalDocs/get-started.md
:white_check_mark:Succeeded
View

TerminalDocs/images/acrylic-opacity.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/acrylic.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/always-show-tabs.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/antialiasing-mode.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/background-image-alignment.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/background-image-stretch-mode.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/campbell-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/campbell-powershell-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/close-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/confirm-close-all-tabs.png
:white_check_mark:Succeeded
View

TerminalDocs/images/dropdown.png
:white_check_mark:Succeeded
View

TerminalDocs/images/duplicate-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/dynamic-profiles.png
:white_check_mark:Succeeded
View

TerminalDocs/images/experimental-retro-terminal-effect.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/first-run.png
:white_check_mark:Succeeded
View

TerminalDocs/images/frosted-glass-theme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/navigate-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/one-half-dark-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/one-half-light-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/open-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/overview.png
:white_check_mark:Succeeded
View

TerminalDocs/images/padding.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/powerline-powershell.png
:white_check_mark:Succeeded
View

TerminalDocs/images/programming-ligatures.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/raspberry-ubuntu.png
:white_check_mark:Succeeded
View

TerminalDocs/images/requested-themes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/resize-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/retro-command-prompt.png
:white_check_mark:Succeeded
View

TerminalDocs/images/search-case-match.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search-direction.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search.png
:white_check_mark:Succeeded
View

TerminalDocs/images/show-tabs-in-title-bar.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/snap-to-grid-on-resize.gif
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.

TerminalDocs/new-terminal-arguments.md

Line 10: [Warning] Markdown content is not allowed before H1.
Line 11: [Suggestion] H1 is required. Use a single hash (#) followed by a space to create your top-level heading. NOTE: This Suggestion will become a Warning on 05/15/2020.

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.

Docs Build status updates of commit de5f9a4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

TerminalDocs/Customize settings/custom-keybindings.md
:white_check_mark:Succeeded

TerminalDocs/TOC.yml
:white_check_mark:Succeeded
View

TerminalDocs/cascadia-code.md
:white_check_mark:Succeeded
View

TerminalDocs/command-line-arguments.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/frosted-glass-theme.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/powerline-in-powershell.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/raspberry-ubuntu.md
:white_check_mark:Succeeded
View

TerminalDocs/custom-terminal-gallery/retro-command-prompt.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/color-schemes.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/global-settings.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/key-bindings.md
:white_check_mark:Succeeded
View

TerminalDocs/customize-settings/profile-settings.md
:white_check_mark:Succeeded
View

TerminalDocs/dynamic-profiles.md
:white_check_mark:Succeeded
View

TerminalDocs/get-started.md
:white_check_mark:Succeeded
View

TerminalDocs/images/acrylic-opacity.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/acrylic.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/always-show-tabs.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/antialiasing-mode.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/background-image-alignment.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/background-image-stretch-mode.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/campbell-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/campbell-powershell-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/close-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/confirm-close-all-tabs.png
:white_check_mark:Succeeded
View

TerminalDocs/images/dropdown.png
:white_check_mark:Succeeded
View

TerminalDocs/images/duplicate-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/dynamic-profiles.png
:white_check_mark:Succeeded
View

TerminalDocs/images/experimental-retro-terminal-effect.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/first-run.png
:white_check_mark:Succeeded
View

TerminalDocs/images/frosted-glass-theme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/navigate-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/one-half-dark-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/one-half-light-color-scheme.png
:white_check_mark:Succeeded
View

TerminalDocs/images/open-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/overview.png
:white_check_mark:Succeeded
View

TerminalDocs/images/padding.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/powerline-powershell.png
:white_check_mark:Succeeded
View

TerminalDocs/images/programming-ligatures.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/raspberry-ubuntu.png
:white_check_mark:Succeeded
View

TerminalDocs/images/requested-themes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/resize-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/retro-command-prompt.png
:white_check_mark:Succeeded
View

TerminalDocs/images/search-case-match.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search-direction.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search-panes.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/search.png
:white_check_mark:Succeeded
View

TerminalDocs/images/show-tabs-in-title-bar.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/snap-to-grid-on-resize.gif
:white_check_mark:Succeeded
View

TerminalDocs/images/solarized-dark-color-scheme.png
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
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.


chore: added SmallDevTools

Fixes Issue
Closes #844

Changes proposed
Added SmallDevTools

Screenshots

Note to reviewers

Made the required changes @CBID2 :)


Docker::Container#exec passing self.connection to Docker::Exec.create

Suggested solution for #303

Looks good.  Got a couple lines with trailing whitespace though. Will merge when cleaned up!

Done
--
Adam Ladachowski
On Tuesday 25 August 2015 at 15:32, Todd Lunter wrote:

Looks good. Got a couple lines with trailing whitespace though. Will merge when cleaned up!
—
Reply to this email directly or view it on GitHub (https://github.com/swipely/docker-api/pull/304#issuecomment-134586305).

Released in v1.22.3! Thanks @aladac :)


Rename forked version to @labshare/express-jwt and add Github Action to publish to NPM

:tada: This PR is included in version 6.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


DO NOT MERGE - Codespace eqxwow congenial space disco x59r7rr6gr6cv465

Created only for reference. Closing this PR.


Enable claim validation by calling AuthenticationRequest#validateClaims() Issue

Checklist

[X] The issue can be reproduced in the auth0_flutter sample app (or N/A).
[X] I have looked into the README and have not found a suitable solution or answer.
[X] I have looked into the API documentation and have not found a suitable solution or answer.
[X] I have searched the issues and have not found a suitable solution or answer.
[X] I have searched the Auth0 Community forums and have not found a suitable solution or answer.
[X] I agree to the terms within the Auth0 Code of Conduct.

Description
I get this message E/BaseAuthenticationRequest(24058): The request is made without validating claims. Enable claim validation by calling AuthenticationRequest#validateClaims().
when I use the auth0.api.login method from the sdk on Android. Although the request goes through, however I am concerned as I don't know what this means.
I have been able to trace this to this file on the android-sdk auth0/src/main/java/com/auth0/android/authentication/AuthenticationAPIClient.kt to a comment on line 67 about the method on line 79.
Reproduction

Make a call to auth0.api.login() with required parameters.
immediately after request is initiated check console for this E/BaseAuthenticationRequest(24058): The request is made without validating claims. Enable claim validation by calling AuthenticationRequest#validateClaims().

auth0_flutter version
auth0_flutter: ^1.0.2
Flutter version
Flutter version 3.3.10 on channel stable
Platform
Android
Platform version(s)
Android 13 (API 33)

Hi @philip6560, thanks for raising this.
auth0_flutter wraps the Auth0.Android and Auth0.swift native SDKs. Since v2.8.0, Auth0.Android supports validating the claims of ID tokens obtained through the Authentication API client. It is an opt-in feature, which is not enabled in auth0_flutter because it's not supported in Auth0.swift yet.
You can implement the logic for validating the ID token claims yourself, and disregard the The request is made without validating claims. Enable claim validation by calling AuthenticationRequest#validateClaims() message: https://auth0.com/docs/secure/tokens/id-tokens/validate-id-tokens

@Widcket Thanks a lot, I'd check it out.


New Rails app with --skip-javascript doesn't get a bin/vite file when running bundle exec vite install

[x] I have tried upgrading by running bundle update vite_ruby.
[x] I have read the troubleshooting section before opening an issue.

Description 📖
New Rails app with --skip-javascript doesn't get a bin/vite file when running bundle exec vite install
Reproduction 🐞
rails new YourAppName --skip-javascript --database=postgresql && cd YourAppName  && git add . & git commit -m "initial no-js Rails app" && bundle add vite_rails && bundle install && bundle exec vite install
Result:
• No bin/vite file is generated
Expected result:
• bin/vite file is created as per "Adds the bin/vite executable to start the dev server and run other commands"
Logs 📜
Screenshots 📷

I see this doesn't work because this app has no package.json, and therefore no dependencies

This is caused by using yarn instead of npm
Fixed by using npm init -y instead of yarn init -y:
rails new YourAppName --skip-javascript --database=postgresql && cd YourAppName  && git checkout -b main  && git add . && git commit -m "initial no-js Rails app" && npm init -y && git add . && git commit -m "initalizes npm" && bundle add vite_rails && bundle install && bundle exec vite install


Question: Can you install cachetool on 1 specific domain/subdomain instead of entire server

Hey this is more a question than an idea or bug.
I'm not that advanced in this kinda stuff, but is it possible to use this for 1 specific domain instead of the entire server?
I have problems with the cache and reloading of the site after Envoyer deployments, but I am afraid if I install it on the entire server and I run the reset command, it'll apply it to all domains and I do not want that. And even if that's the only possible way, will this have negative effects on the other domains?
Thanks in advance

If you do run the reset command it will clear the opcache.
What you can do instead, is invalidate the files of the directory related to that specific subdomain, which you can do by running:
$ cachetool opcache:invalidate:scripts /path/to/subdomain

Conversation continued in https://github.com/gordalina/cachetool/issues/226


Added registration for helper functions (not magics)

This PR fixes #21
ATM the magic manager only registers magics, which are full blown functions to help out with various tasks. There is however a need to create smaller helpers, which could be used to filter out panda results, or tweak few things. These helpers don't need to be converted to magics or anything like that, but they need to be discover-able.  That's the goal of this registration mechanism, to simply register a function as a helper to make it discoverable.
The helpers themselves would then be added inside picatrix/helpers or something similar. Into various files there. This would also make that discoverable if people prefer to just install from picatrix import helpers or to be able to auto-generate documentation for these helpers.

[x] Unit tests added
[ ] End-to-end tests added
[x] Appropriate changes to documentation is included
[x] If additional dependencies are needed, are they added into dependency files.

Almost there :) left a small comment.

done,  PTAL @bladyjoker


[Question] which RN version(s) is(are) supported yet?

Hi, this project seems to be what I need.
I'm relying on native events from a native modules coming through the bridge at a high rate. In my application right now, processing those events is slowing down the UI.
So I guess that offloading this to another JS engine instance makes sense.
Is this module working with RN 0.57.4?
Are we limited to strings exchanged in messages both ways? If so I guess I would have to stringify my objects.
Thanks in advance.

Hi, not the maintainer but I'm happy to answer. I found this to work in 0.57 and 0.58 (so assuming it'll work with 0.57.4). Looking at the native source code for this, it seems that we are indeed limited to strings.

Also just tried it on 0.59.0 and it's working.

Sorry should have said working on 0.59.0 for iOS. Have not tried other platforms yet.

@cjroth does not work for me on 0.57.4 (latest version at least).

error: package com.facebook.react.jscexecutor does not exist

Hmm is this the same thing? https://github.com/joltup/react-native-threads/issues/63

Yep I managed to solve it going back to the previous version.
A version matrix would be a plus.

@cjroth can you please tell how are you initializing and command you used to build Thread.jsbundle Thread from your component in iOS as I tried the same and followed the instructions given but for me it going into infinite loading loop.

@sja1n I experience an infinite load loop if I had the remote debugger turned on. I had to turn it off entirely while using threads and instead use Reactotron. To get around that in development I basically have my code able to run either in a thread or not in a thread, depending on whether or not I need to use the remote debugger. This worked elegantly by using Redux actions that basically can either dispatch to the thread or just to the regular store.

oh yes, I was using remote debugging and disabling it off worked for me. Thanks @cjroth

is it possible to get this to work without reverting to a previous version? I'm getting the same error described in #63 with RN  0.57.8 and I'm kinda stuck with that version..

Is this library compatible with react-native 0.60.4 ?

@eharason
downgrading mentioned in #63  means downgrading the version of react-native-threads not react-native.
for convenience, I repeat my post here:
According to my test (test only on android), react-native 0.57.1 is compatible with react-native-threads 0.0.15, not the latest 0.0.16 (which will lead to the error below).

error: package com.facebook.react.jscexecutor does not exist

besides, below is a brief summary for this question:
(for future community members, pls complete this table according to your own test, so that in the future we will have a more complete answer.)
react-native version--------react-native-threads version--------platform--------reference/who tests
0.55.4--------0.0.13--------both--------react-native-threads/examples/SimpleExample/package.json
0.57.1--------0.0.15--------android--------@ChenhuaWANG22
0.57.4--------0.0.15--------android--------@omatrot
0.57.7--------0.0.15--------android--------@SokWonJeremyKim
0.57.0--------0.0.16--------iOS--------@cjroth
0.58.0--------0.0.16--------iOS--------@cjroth
0.59.0--------0.0.16--------iOS--------@cjroth


test: add unit test for the online snapshot prepare method

E2E tests running on EDB fork https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/7194531699


postgresql_16: init at 16.0

Description of changes
In the hope of helping the maintainers, here's an initial PR to add PostgreSQL 16, which was just released yesterday. Due to an upstream change (https://github.com/postgres/postgres/commit/11a0a8b529caeab101206ec4a33af95bb4445746), one of the existing patches was obsoleted, and required an alternate formulation.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[x] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255446 run on x86_64-linux 1

9 packages marked as broken and skipped:

postgresql16JitPackages.age
postgresql16JitPackages.cstore_fdw
postgresql16JitPackages.pipelinedb
postgresql16JitPackages.plv8
postgresql16JitPackages.promscale_extension
postgresql16Packages.age
postgresql16Packages.cstore_fdw
postgresql16Packages.pipelinedb
postgresql16Packages.promscale_extension

24 packages failed to build:

postgresql16JitPackages.apache_datasketches
postgresql16JitPackages.pg_auto_failover
postgresql16JitPackages.pg_bigm
postgresql16JitPackages.pg_hint_plan
postgresql16JitPackages.pg_hll
postgresql16JitPackages.pg_ivm
postgresql16JitPackages.pgaudit
postgresql16JitPackages.tds_fdw
postgresql16JitPackages.temporal_tables
postgresql16JitPackages.timescaledb
postgresql16JitPackages.timescaledb-apache
postgresql16JitPackages.timescaledb_toolkit
postgresql16Packages.pg_auto_failover
postgresql16Packages.pg_bigm
postgresql16Packages.pg_hint_plan
postgresql16Packages.pg_hll
postgresql16Packages.pg_ivm
postgresql16Packages.pgaudit
postgresql16Packages.plv8
postgresql16Packages.tds_fdw
postgresql16Packages.temporal_tables
postgresql16Packages.timescaledb
postgresql16Packages.timescaledb-apache
postgresql16Packages.timescaledb_toolkit

65 packages built:

postgresql16JitPackages.jsonb_deep_sum
postgresql16JitPackages.periods
postgresql16JitPackages.pg_cron
postgresql16JitPackages.pg_ed25519
postgresql16JitPackages.pg_net
postgresql16JitPackages.pg_partman
postgresql16JitPackages.pg_rational
postgresql16JitPackages.pg_relusage
postgresql16JitPackages.pg_repack
postgresql16JitPackages.pg_safeupdate
postgresql16JitPackages.pg_similarity
postgresql16JitPackages.pg_topn
postgresql16JitPackages.pgjwt
postgresql16JitPackages.pgroonga
postgresql16JitPackages.pgrouting
postgresql16JitPackages.pgsql-http
postgresql16JitPackages.pgtap
postgresql16JitPackages.pgvector
postgresql16JitPackages.plpgsql_check
postgresql16JitPackages.plr
postgresql16JitPackages.postgis
postgresql16JitPackages.postgis.doc
postgresql16JitPackages.repmgr
postgresql16JitPackages.rum
postgresql16JitPackages.smlar
postgresql16JitPackages.tsearch_extras
postgresql16JitPackages.wal2json
postgresql16Packages.apache_datasketches
postgresql16Packages.jsonb_deep_sum
postgresql16Packages.periods
postgresql16Packages.pg_cron
postgresql16Packages.pg_ed25519
postgresql16Packages.pg_net
postgresql16Packages.pg_partman
postgresql16Packages.pg_rational
postgresql16Packages.pg_relusage
postgresql16Packages.pg_repack
postgresql16Packages.pg_safeupdate
postgresql16Packages.pg_similarity
postgresql16Packages.pg_topn
postgresql16Packages.pgjwt
postgresql16Packages.pgroonga
postgresql16Packages.pgrouting
postgresql16Packages.pgsql-http
postgresql16Packages.pgtap
postgresql16Packages.pgvector
postgresql16Packages.plpgsql_check
postgresql16Packages.plr
postgresql16Packages.postgis
postgresql16Packages.postgis.doc
postgresql16Packages.repmgr
postgresql16Packages.rum
postgresql16Packages.smlar
postgresql16Packages.tsearch_extras
postgresql16Packages.wal2json
postgresql_16
postgresql_16.debug
postgresql_16.doc
postgresql_16.lib
postgresql_16.man
postgresql_16_jit
postgresql_16_jit.debug
postgresql_16_jit.doc
postgresql_16_jit.lib
postgresql_16_jit.man


Sidenav 'openedChange' event is fired when default value is 'opened=true'

Reproduction
Here is an StackBlitz reproducing the issue: https://stackblitz.com/edit/angular-t2dksh
Expected Behavior
I expect Sidenav openedChange event to not be fired when default value is opened=true.
Actual Behavior
openedChange event is fired when the Input property opened is set to true on init.
Explanation
I don't know if that functionality is by design, but in my opinion, this shouldn't be the expected behaviour. If we set that the init value is 'opened=true', for me there is no reason that openedChange event emits... it has not changed, simply the start value is opened=true. It has not changed from opened=false to opened=true.
Environment

Angular: 9.0.0-rc.6
CDK/Material: 9.0.0-rc.5
Browser(s): Chrome (latest)
Operating System (e.g. Windows, macOS, Ubuntu): Windows

The default state is false so setting it to true is a change. If you want to respond only to changes from the user, you should be able to implement it yourself since you're the one controlling which elements can toggle the sidenav.

Yes, I could fix easily on my code, is not big deal, but I simply found strange the behavior.
I am thinking about the possible use cases of listening openedChange, and I think most library users that listen to it are doing it for resizing manually some content when open/close animation ends (since openedChange emits when the animation finishes). For this use case is useless that openedChange emits when the default value is opened=true.
I actually I can't figure out any use case where one library user would like to listen and react to that openedChange event that emits for the default value opened=true.
Actually, non emitting the first default value is a practice that some components implement, like MatTabGroup with selectedIndex. The selectedIndex default value is null and it doesn't emit any change event for the first start value.
Code example (from MatTabGroup):
const isFirstRun = this._selectedIndex == null;
if (!isFirstRun) {
      this.selectedIndexChange.emit(indexToSelect);
}

As I said, is not big deal (it can be easily patched), but for me this behavior is not the expected one.
I could make a PR if you think that this behavior could be changed.

Closing since it's working as intended.


Use the training (or validation) loss value for early stopping by default

I think we should have scoring=None for all Gradient Boosting estimators by default and still enable early stopping.

if self.validation_split is not None, call self.loss_(y_validation, raw_predictions_validation) instead of a Scorer instance.
if self.validation_split is None`, find a way to incrementally update an estimator of the full training loss without adding any additional computation cost (I think it should be doable but I have not checked in detail).
if n_iter_no_change is None or n_iter_no_change == 0 disable early stopping and go to max_iter.

@NicolasHug WDYT?

+1 this is a more natural strategy.
For the second point, you mean updating the training loss while we update the gradients to avoid redundant computations? This seems doable for LS at least.
I'll go for a simple version first (separate updates) and see what it looks like.

For the second point, you mean updating the training loss while we update the gradients to avoid redundant computations? This seems doable for LS at least.

Yes.

I'll go for a simple version first (separate updates) and see what it looks like.

Alright.


Package stylish-haskell is marked as broken, refusing to evaluate

Describe the bug
Trying to install stylish-haskell fails.
https://github.com/NixOS/nixpkgs/blob/bf4169178ec8c9d8c9947ae4ee7f018093a6e77f/pkgs/development/haskell-modules/configuration-hackage2nix.yaml#L1022
To Reproduce
Steps to reproduce the behavior:

$ nix-env -i stylish-haskell
watch it fail

Expected behavior
It should install stylish-haskell
Screenshots

Additional context
stylish-haskell has a dependency on >=0.2.0 && <0.3, but it looks like HsYAML is set to <IP_ADDRESS> in configuration-hackage2nix.yaml
Metadata
λ nix-shell -p nix-info --run "nix-info -m"
these paths will be fetched (1.46 MiB download, 7.88 MiB unpacked):
  /nix/store/3lvcb4hxzax99zvb33ryk4whcv7zpjq2-bash-interactive-4.4-p23-info
  /nix/store/czg8zzd1qib1yafggnf5jsd14cmcfsdv-bash-interactive-4.4-p23-doc
  /nix/store/gr67mfj18zmyb0w7510ihi1qbivnjbaa-bash-interactive-4.4-p23-man
  /nix/store/kk512ydg1i1r0h5m9fizdrl9qxlrazrf-bash-interactive-4.4-p23-dev
  /nix/store/l5zsd7mphr5gp8p5y46ba6ahwlna1gnb-DarwinTools-1
  /nix/store/lq6lcv42lf7cp35lgxmxgs6qikqhvgg9-nix-info
  /nix/store/x3vwmx1y5l9ssy07i67395g1xzba1cyk-readline-7.0p5
  /nix/store/y27b4rnmiwzvpznb2hbqmvdzzk4a5472-bash-interactive-4.4-p23
copying path '/nix/store/czg8zzd1qib1yafggnf5jsd14cmcfsdv-bash-interactive-4.4-p23-doc' from 'https://cache.nixos.org'...
copying path '/nix/store/l5zsd7mphr5gp8p5y46ba6ahwlna1gnb-DarwinTools-1' from 'https://cache.nixos.org'...
copying path '/nix/store/3lvcb4hxzax99zvb33ryk4whcv7zpjq2-bash-interactive-4.4-p23-info' from 'https://cache.nixos.org'...
copying path '/nix/store/gr67mfj18zmyb0w7510ihi1qbivnjbaa-bash-interactive-4.4-p23-man' from 'https://cache.nixos.org'...
copying path '/nix/store/lq6lcv42lf7cp35lgxmxgs6qikqhvgg9-nix-info' from 'https://cache.nixos.org'...
copying path '/nix/store/x3vwmx1y5l9ssy07i67395g1xzba1cyk-readline-7.0p5' from 'https://cache.nixos.org'...
copying path '/nix/store/y27b4rnmiwzvpznb2hbqmvdzzk4a5472-bash-interactive-4.4-p23' from 'https://cache.nixos.org'...
copying path '/nix/store/kk512ydg1i1r0h5m9fizdrl9qxlrazrf-bash-interactive-4.4-p23-dev' from 'https://cache.nixos.org'...
 - system: `"x86_64-darwin"`
 - host os: `Darwin 19.3.0, macOS 10.15.3`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.1`
 - channels(rpearce): `"home-manager, nixpkgs-20.03pre211437.e3a9318b6fd"`
 - nixpkgs: `/Users/rpearce/.nix-defexpr/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
  - stylish-haskell
# a list of nixos modules affected by the problem
module:

Feel free to send a PR overriding the dependencies of stylish-haskell in configuration-common.nix.
Alternatively, getting stylish-haskell into Stackage would also help out here.

@cdepillabout Thank you for the :eyes:! I'll see about both of these

Also, I've written a couple places about how Haskell packages are updated in nixpkgs.
Maybe this thread will help you:
https://discourse.nixos.org/t/haskellpackages-stm-containers-fails-to-build/5416/4

While the fix isn't available to me yet on the unstable channel, that fix looks good, so I'll close this to get it out of your hair!


Crash at Startup

Mod Version: 1.10.2-<IP_ADDRESS>
Forge Version: <IP_ADDRESS>2
Crash: https://pastebin.com/qUnta4F2
This just happened one time.

If I am interpreting the trace correctly the sound manager was loaded before the proxy for my mod was created.  Which is outright strange.

Pushed v<IP_ADDRESS> BETA version that has a fix for this.


Tabs showing green line above them.

Does this issue occur when all extensions are disabled?: Yes/No
This occurs regardless of whether all the extensions are disabled. I don't think it's an extension issue.

VS Code Version: 1.80.1
OS Version: 10.0.22621 Build 22621
Microsoft Windows 11 Home

Steps to Reproduce:
I don't know what caused it. I just opened it today and it looked like this now.

Check your settings (user and wrkspace) for this colorCustomization:
    "tab.activeBorderTop"  // looks like you have this one set to green

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


PQueue#deq is NOT an alias of #push

Fixing incorrect alias. deq is an alias of pop

Yep. You are right. I am amazed that bug persisted for so long! Someone else reported it in the issues, so I had just fixed this myself before I noticed your pull request. B/c of that I can't get a clean merge. Nonetheless, thank you for submitting it. New release 2.1.0 should be out tonight.

:+1: Thanks!


Test-specific definition

Hello. Ceedling is awesome project and we use this system everyday.
I appreciate all of your hard effort :)
By the way, we did a customize to specify test-specific definition to compile the test.
Though it was tentatively realized by following pull-request but I think it is a kind of dirty hack.
https://github.com/TE-HiroakiYamazoe/Ceedling/pull/2
I also wrote what we want to do in the comment of the pull-request.
Does anyone have an idea to make it possible properly?

Hi.
A fairly recent Unity branch has added the ability to pass command line arguments. These arguments include the ability to filter to a specific test or group of tests, or to reject certain tests even. It's going to be merged to the main branch very soon.
When that happens, the best way to handle this feature request would be to make use of that built-in functionality and just have ceeding make those calls. :)


FFmpeg VVENC version cannot use more threads

Hello Martin, the CPU of my computer is AMD Ryzen 9 7945HX and the operating system is Windows11 23H2.
It only seems to support a maximum of 8 threads, but FFmpeg shows a maximum of 16 threads, and I'm not sure whether 8 or 16 processes are used.
My computer supports up to 32 processes, how can I maximize its use?
In addition, the Two-pass function cannot be used normally. Do I have to give up this version?

Hi, @jiangzemin-xudamin!
Thank you for providing your photos for the issue.
On third image, you used command to encode: .\ffmpeg_vvceasy -threads 32 -i 1.mp4 -c:v libvvenc -preset 4 -b:v 3760k -c:a libopus -b:a 64k 2.mp4, but it was wrong swap command.
The first thing is input goes first before you use threads command, not threads goes first before input.
Here's quick fix command:
.\ffmpeg_vvceasy -i 1.mp4 -threads 32 -c:v libvvenc -preset 4 -b:v 3760k -c:a libopus -b:a 64k 2.mp4

If you want to use two pass, you need to encode first pass before second pass command:
The first pass will use to create stats.json file on your located folder for only video. This will skip audio by using -an.
The second pass will read stats.json file, changing from pass 1 to 2 and you can encode audio too.
I added vvenc params command to make two pass correct.
.\ffmpeg_vvceasy -i 1.mp4 -threads 32 -c:v libvvenc -preset 4 -b:v 3760k -an -vvenc-params passes=2:pass=1:rcstatsfile=stats.json -f null NUL
.\ffmpeg_vvceasy -i 1.mp4 -threads 32 -c:v libvvenc -preset 4 -b:v 3760k -vvenc-params passes=2:pass=2:rcstatsfile=stats.json -c:a libopus -b:a 64k 2.mp4

You can find out more information about two passes: https://github.com/fraunhoferhhi/vvenc/wiki/FFmpeg-Integration#rate-control-vbr-mode.
If you have any question or issue, feel free to reply me. :)
Thank you! :)
Regards

Martin Eesmaa

Thank you very much for your correction! Martin.

Hi, @jiangzemin-xudamin!
You're welcome and it's okay if you make a mistake.
Also wishing you and everyone for Merry Christmas and Happy New Year!

Martin Eesmaa


Pipeline not triggered when comment is made by a github bot

We recently configured our pipeline with ghprb to trigger only when the trigger phrase is commented. We have a bot who comments on the trigger phrase when a change in our specified files has occurred. I tried adding the bot name to the admin list as well as the whitelist names but still, the pipeline is not triggered when the comment is done by the bot.

build


Added link to landing page for Java examples

Description
Modification to readme.md of /examples folder to add link to Java language binding landing page on mxnet.io

@lanking520 , could you please review and merge?


Darin McLain - Ready to get started

Forked the repo, cloned it local, ready to get started

welcome!

welcome!


Fix Ember version for default dashless helper support.

Automatic resolution of dashless helpers landed in Ember 1.13.0 (none of the betas included it), this PR updates the version check added in #5.

@stefanpenner any chance of a new release that incorporates this fix?
Cheers

thanks for the reminder 1.0.2 is released


Silent Fail when running AION account creation under a no-console environment

I'm running:

Which Aion version?: https://github.com/aionnetwork/aion/commit/c4f31463407d8a88c62ade310064daf230920ab7
Which operating system?: Linux
Which distributor and version?: Ubuntu 16.04.3 LTS
How installed?: source (IDEA)
Are you fully synchronized?: no
Did you try to restart the node?: yes

In an IDE environment, there may not be a console available, when this happens System.console() will be null and our CLI will throw an NPE. I was able to recreate this behaviour by running the aion account creation command within IDEA. (Surprisingly this worked previously but no longer works).
At the very least we should be warning people by logging something out when we detect that there is no console environment available.

#494

issue fixed by #494


Fix(15506) - Disallow non-any/number/string operands on comparisons

This PR aims to fix the #15506 ticket, here I based myself and learned from the suggested changes on previous PRs that tried to resolve this same issue in the past (so I would say that it was co-authored by @feloy and @FabianLauer).
From what I understand, the main idea is that we should flag an error on every comparison (<, >, <= and >=)  that is not between a string, number, or any values. Doing this, triggered a lot of new errors on the baselines (that seem correctly triggered assuming that I understand the issue correctly), but I would like to confirm that I got this
right because it seems like a huge change to me.
I'm submitting the PR as a spike for now, just to gather suggestions and confirm that I'm following the right path here since this issue is kinda old and this is my first time contributing here.

@c0sta please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@microsoft-github-policy-service agree company="Microsoft"

Contributor License Agreement

@microsoft-github-policy-service agree

This last commit fixes the Self Check action error by allowing compare Objects, Union, TypeParameter, and Intersection. That's because TS has some internal comparisons that use these types (already discussed here in the past). But I'm unsure if that's a good idea since it goes against the main issue purpose to disallow non-any, number, and string operands.

This last commit fixes the Self Check action error by allowing compare Objects, Union, TypeParameter, and Intersection. That's because TS has some internal comparisons that use these types (already discussed here in the past). But I'm unsure if that's a good idea since it appears to go against the main issue purpose to disallow non-any, number, and string operands.

I would expect that we would be recursing into those types to verify, and not ignore them. I'm pretty sure we have this sort of checking for other operators, so it feels weird that we are adding a whole new function to check this...

Just some updates, I've searched for checkers that could make this comparison but I couldn't find it, so basically what I did was start using this isTypeAssignableTo. I'm just struggling a little bit to trigger the error on never and number comparisons but I'm working on this.
The Self Check error will come back now and after fixing the never and number comparison I'll look at them.
This last push brings a lot from the main branch with it, not sure if I'm doing something wrong here.

It seems like you incorrectly merged main, I'm afraid.

It seems like you incorrectly merged main, I'm afraid.

Sorry about that, I did a reset on my 875169e commit, but if you think it's a better idea to open another PR I can do that too.
I changed my configs to use merge instead of rebase, it seems to be merging properly now.

So I've worked to fix some of the self-check errors, most of them were related to Date comparisons so I started using .getTime() to convert the Date objects to numbers.
But I'm not sure how to go on this last error:  Operator '<' cannot be applied to types 'string | number' and 'string | number'.. Since string and number are valid types for this operand, should this union keep triggering an error? and should it be possible to compare string with number?

How is this PR different from #52807?
compareComparableValues is also flagged by that PR for the same reason; it's unsafe but cannot actually be safe.

How is this PR different from #52807?
compareComparableValues is also flagged by that PR for the same reason; it's unsafe but cannot actually be safe.

I don't think they are, they look the same to me. In that case, it's a better idea to just close this current PR?

Maybe; #52807 did not mention #15506 but was instead in the context of better handling valueOf which I don't think this PR does (but likely should, as it is part of a way to make it less breaky). I don't recall where we were with #52807 but @RyanCavanaugh may remember. I'm personally still hoping that's merged.


base64urlencode as optional

@klerick Thank you for submitting this, it is great! I have one request for change:

If you can just add this use case as a separate test with a properly defined test case, something like "auth server succeeds when skipping base64encode"

I will get this merged in and tagged as the next version

done, if i understand you correctly :)


Add fuji

Description
Include Avalanche testnet (fuji) in hardhat config.
also fixes the 'unknown' network name, and adds the proper snowtrace URLs.
Test Coverage

[x] My code follows the Style Guide.
[ ] My code requires a Documentation Update.

updated.


Re-organize folder structure

Remove CSharp folder

It's really annoying to go yet another level deeper all the time IMHO. We may decide to put C++ in the same sln anyway, and we can probably put other language project as siblings of C# projects whether they are in a sln or not.

Rename Test folder to Tests

It's more consistent with project/namespace names.

Move Benchmarks to Tests

This is a natural generalization of "Tests" meaning IMHO to include perf testing. It further allows one directory for non-product build props.

Move *.md from source to new root devdocs folder

These have language-agnostic info and shouldn't be in the C# project.

Rename projects for consistency and brevity:

CommonAnnotatedSecurityKeys -> Cask
Tests.CommonAnnotatedSecurityKeys -> Cask.Tests

We still use CommonAnnotatedSecurityKeys as the namespace.

I have a big PR coming that builds on this structure. I want to ensure the history gets the renames and the diff in the big PR isn't too hard to review with renames getting lost.


perf(trie): use available pre-loaded prefixsets

Description
StateRoot::incremental_root_* methods internally create a PrefixSetLoader which walks the updated accounts and storages within the block range, hashes their keys and constructs corresponding prefixsets. In insert_hashes and get_or_take_block_and_execution_range::<true> provider methods, this results in double work hidden behind the incremental root abstraction, since we already walk and hash the same keys in order to update hashing tables.

I would like to test this before merging.

tested on sepolia, been successfully syncing live for a couple days


Static Content

Hi, thank you for the work on the package, I was wondering if theres a way to generate Static content instead of a Cache php file, No php or variables just plain html, then a function can be created that refresh/renders the page every day, this could improve speed drastically
Thanks

See https://jigsaw.tighten.co/


docs(README): add catwalk image as preview

Updated the preview with a catwalk to match with other repos, also changed the install command to use latest version of the script instead of old one and being able to pass arguments.

Actually nevermind, Stacked layout looks better


Hasan/DPROD-3065/lazy loading

Changes:

added lazy loading for live market, trade types and payment section

Type of change

[ ] Bug fix
[ ] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

Closing in favor of https://github.com/binary-com/deriv-com/pull/6966/


Echo /mode +b messages

I was trying to moderate in #mojo and I was unable to see if/when I had successfully banned a user. There was no output from sending the op +b command.
Addtionally, it might be useful to have some kind of moderation tools to help craft the ban command.

Not sure what "moderation tools" is, but first impression is that this is too low level for convos. I could be wrong.

I wonder if this is fixed in master now.
Please re-open if it is not.


Automate PyPI Update on GitHub Release

Automatically update PyPI on new GitHub releases.

Hey do you know if there is a way that I can pass the PyPI ownership to you. I have been having problems updating this repo since they changed over to api tokens. I had it working for a while and now I am having difficulties again. If you make an account I can add you as a maintainer and you can follow this https://packaging.python.org/en/latest/tutorials/packaging-projects/
Though you probably already know how to do it.

Before we get started, I did a little research and found that the easiest way to implement this feature is by using GitHub Actions. I've used GitHub Actions before, but I haven't set it up for this specific purpose yet. You can try this on your own while you work on resolving the issue, and this feature can wait. Alternatively, we can begin by adding me as a maintainer, as you suggested. We can learn together through experimentation. My PyPI username is FastFingertips.

It would be great to create a test repository and learn this feature as soon as possible. I’ll work on it in my spare time.

Hey awesome, it does look like GitHub Actions is a good way to go. I added you on PyPi as well. I have been busy since I began my PhD so I am sorry if I have not been active. I appreciate all the work on this project.


Add support for multiple indexes seperated by commas for issue #884

The indexes work as before but you can now add multiple by comma separating them.
If you leave one of the comma separated values empty it is assumed you want to create the default index.

Thank you for your PR.


Simplify minirent consumption

This change simplifies the integration with this library. It refactors
#if guards to automatically include <dirent.h> on non-WIN32
platforms.
Therefore, usages of this library can be changed from this:
 #ifdef _WIN32
 #  define MINIRENT_IMPLEMENTATION
 #  include "minirent.h"
 #else
 #  include <dirent.h>
 #endif

to this:
 #define MINIRENT_IMPLEMENTATION
 #include "minirent.h"

No fuss, no wuss. This should be backwards compatible. README and CI
were updated to reflect it.

@m4c0 good idea! :+1: Thank you!


feature: Hope to provide upgrade features

I hope to upgrade the curvefs cluster that has been installed, but I have found that there are no relevant steps and effective upgrade methods.

+1

BS also needs this feature

BS also needs this feature

It will coming soon, you can run curveadm upgrade -h for some preview.

now we support upgrade all services, run curveadm upgrade -h for more information.


chore: reformat code

What was the problem/requirement? (What/Why)
Reformatted the code using hatch fmt. This is the built in hatch formatter (ruff) which is not run as part of hatch run fmt (which formats using black).
Also updated a job bundle output test to increase test coverage and avoid a SonarCloud duplicated code error.
Did you run the "Job Bundle Output Tests"? If not, why not? If so, paste the test results here.
I ran the physical job bundle output test, since it was modified.
Was this change documented?
No, N/A. I decided not to include hatch format in DEVELOPMENT.md yet because it outputs a number of formatting errors e.g. test\test_copyright_headers.py:72:5: S101 Use of 'assert' detected which are false positives.
Is this a breaking change?
No

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

If this tool generally gives false positives, should we remove it? I say this because of 2 reasons:

I think the issue is that based on our current (lack of) ruff configuration, the tool doesn't have enough information to determine what to flag. If we were to add some ruff configuration, that would remove the false positives

This can reduce the amount of dependencies (i.e. ruff) that we may need to update.

hatch fmt (including ruff) is built into hatch, and we use hatch for various reasons, so we can't remove it altogether

As I understand, we may need to manually reformat this again later to ensure the formatting is correct? So we need to reformat the code periodically every few days?

Correct, the tooling right now doesn't run automatically in our CI workflows. We would need to add some configuration to get this to run, including to remove those false positives. There are also some legitimate code style issues flagged by hatch fmt that can't be automatically fixed.

How about other repositories? How do they manage this?

As far as I am aware, they only use hatch run fmt instead of hatch fmt. hatch run fmt has a subset of rules. Ideally, we should improve the linting of all repositories by running hatch fmt on them, then adding hatch fmt --check to the CI runner definition, but that will require a lot of refactoring to all codebases, so we can start by simply applying the rules here with hatch fmt.

If this tool generally gives false positives, should we remove it? I say this because of 2 reasons:

I think the issue is that based on our current (lack of) ruff configuration, the tool doesn't have enough information to determine what to flag. If we were to add some ruff configuration, that would remove the false positives

This can reduce the amount of dependencies (i.e. ruff) that we may need to update.

hatch fmt (including ruff) is built into hatch, and we use hatch for various reasons, so we can't remove it altogether

As I understand, we may need to manually reformat this again later to ensure the formatting is correct? So we need to reformat the code periodically every few days?

Correct, the tooling right now doesn't run automatically in our CI workflows. We would need to add some configuration to get this to run, including to remove those false positives. There are also some legitimate code style issues flagged by hatch fmt that can't be automatically fixed.

How about other repositories? How do they manage this?

As far as I am aware, they only use hatch run fmt instead of hatch fmt. hatch run fmt has a subset of rules. Ideally, we should improve the linting of all repositories by running hatch fmt on them, then adding hatch fmt --check to the CI runner definition, but that will require a lot of refactoring to all codebases, so we can start by simply applying the rules here with hatch fmt.

Cool, that makes sense. Let's start with Cinema 4D then. Approving the PR.


Refactor initialRender and addNodes in NodeListManager

There's a bit of duplicated logic in these two methods. I should check em out and see if I can simplify that at all.

Done


Adding support for "Command" to control Voicemeeter

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Adding support for "Command" with Voicemeeter
For exemple Restart, Shutdown, Eject,...

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

For a future note, please reach out before randomly doing a PR without any sort of contact whatsoever.
Some of the content of your PR, such as commands, is duplicate work of what is already being worked on which you would have realised if you had raised an issue or feature request.

Hi,
Thanks I will close this PR and let you drive the project.
This PR was just to share potential features without any pretention it could be accepted
There is no development branch visible that could warn me you were already working on this part
Have a great day
Regards

For single developer modules, such as this, it's common for work to be done locally and pushed in a finished state rather than work-in-progress code always being pushed unless the testers/coders are remote, which is why there wasn't any development branches as it has been local.
For some larger modules, such as the Twitch API module for example that has development done on a private repo because some of the content is under NDA so can't be pushed to the public repo yet.
It is for reasons like these that generally if you wish to contribute to a module to post feature request, or bugs, as a GitHub Issue, and then if it is something already being worked on by others then that can be stated, or if it's something that hasn't been worked on and you would like to contribute then it can be assigned to you and when complete submit a PR and it can be timed to line up with releases and such. This can also help speed things up such as getting bug fixes out quickly while feature updates can be looked over more.


Class attributes aren't present in subclasses

This breaks the Params __init__ function when you subclass it twice
def __init__(self, **kwargs):
        cls = type(self)
        for key, value in kwargs.items():
            if not key in cls.__dict__:
                raise KeyError(f'{cls.__name__} does not take {key} as an argument')
            setattr(self, key, value)
        _check_for_required_arguments(cls, kwargs)

ex.
class A(Params):
    x = BoolParams(default=True, help='')

class B(A):
    y = IntParam(help='')

# raises a KeyError
b = B(x=False)

As much as I don't like deep inheritance chains,
I feel like preventing nested inheritance isn't particularly future-proof.