Add SemVer stability badge to README

First of all, thanks for Requests!
Would you be up for adding a badge that shows how SemVer compliant / bug free new releases are? I was looking through the data we gather at Dependabot and realised we could put one together, so threw together the below:

If you click through then there's a description of how it's calculated - basically it takes all of the relevant updates Dependabot has done for projects that use Requests and checks what percentage of the time specs pass on the upgrade PR.
The score is slightly lower than 100% because some projects have flaky specs, but I'm working on filtering them out from the data, too :octocat:.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4707   +/-   ##
=======================================
  Coverage   66.79%   66.79%
=======================================
  Files          15       15
  Lines        1563     1563
=======================================
  Hits         1044     1044
  Misses        519      519

Continue to review full report at Codecov.

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

Hi there @greysteil
I'm not sure I fully understand the value of this badge. Could you share more what people should understand when they see this?
As it stands, I'm not familiar with dependabot and since it's not a service the project uses, I'm not sure if we should be including it.

@sigmavirus24 - of course.
The idea is to show users how stable / bug-free new releases are. Dependabot creates update PRs for lots of Requests users (you can see them here), and we can use the data on whether their CI passes on the PR base and head branches to check whether new versions of Requests are backwards compatible and/or bug-free.
The information is hopefully useful for a couple of reasons:

It's obviously a good thing to aim for for new releases to be bug free and backwards compatible if they're non-major. Measuring what happens when users update is a good way to check it.
With Pipenv it's possible (and sensible) to specify compatible release requirements, but you probably want to know what the backwards compatibility strategy of the library is before doing so. Having a badge showing that compatibility strategy is SemVer, and that it's stuck to, lets users specify better requirements.

Shouldn't matter at all that requests doesn't use Dependabot itself - it's just the data source. 🙂

Interesting. That data sounds helpful. I'm wondering if you can share what aggregation and tracking you perform on your API endpoint.
Our downstream redistributors will care about this and any security minded users will ask this as well. Keep in mind, most of them will attempt to discover it themselves and will tell us, very rudely, if you're tracking anything.
Further more, it sounds like Dependabot is a company that could easily disappear tomorrow (as most start-ups and companies are want to do). To indicate SemVer (which isn't strictly correct anyway), we could just as easily use a shields.io badge and I'm sure most dependabot users can look up the stability of Requests on your website.
All of this is to say that I'm leaning against adding the badge for my own future peace of mind.

@sigmavirus24 - sorry, it sounds like I've not been clear about where the data is coming from. No data comes from badge endpoint itself. When people sign up for Dependabot they give us permission to create pull requests for them and to monitor the status of those pull requests - that's where we get the data from. Looking at the badge on Request's readme, or clicking through to it to get a description of what it means, doesn't produce any data that we're interested in (although to fully answer your question, we do have Google Analytics on the show view that the badge links to).
I've not heard anyone complain that this in unclear, and the badge is included on several high-profile repos for other languages (webpack, sinatra, jest, rubocop). Hopefully it's clearly explained by the first paragraph if you click on the badge?

Dependabot has updated requests between SemVer compatible versions 121 times across 70 projects so far. 98% of those updates passed CI.

On our backend we're using shields.io to generate the badges, and then caching them for a week to reduce load on their servers. That said, we're planning to move the URLs to be shields.io ones as soon as they're able to deploy again (the PR to add Dependabot to shields.io is merged here. Unfortunately their resources are limited and they've been unable to deploy for the last few months.)
Finally, on Dependabot disappearing, we're a bootstrapped, profitable company, but I can of course understand your concern. The worst thing that would happen if we did, however, would that this badge stopped working and that I or some other contributor put in a PR to remove it. Hopefully that's not too serious a worry.
Would you be happier with this badge if the link was a shields.io one? I can understand the concern that introducing an image from a foreign source to your README might allow us to be doing some nefarious tracking of your README views that using the shields.io link, when deployed, would fix.

👍 - the next thing I'm planning to add to the view is a way to see the PRs on which the update has broken CI. I'll put in a new PR then. Also planning to add a way for it to be used for beta releases (where Dependabot won't generate PRs for users, but will generate a commit on a new branch and check their CI) - so you'd be able to see whether a pre-release is breaking anyone's tests before releasing a new version.


Improve streaming vs. downloading UX

Many users get confused that streaming also downloads the file.
Some users would prefer to stream only and not download.

We need to both make this clearer and add options to allow people to automatically delete content after streaming.

we could create a separate cache folder that gets enabled when the option is selected and runs a garbage collection on that cache folder

@tzarebczan please open daemon side issue

@liamcardenas after talking to Grin about this, think it will most likely be just app side changes. The daemon API should provide everything we need to clear out files based on the logic we decide.

We now have the streaming vs downloading (and hosting) options. We should continue to look to improve the UX for a first time user. My suggestion was that we pop up something the first time it downloads so that the user is aware + lead them to settings if they want to change it. Sooner the better, but maybe a good time to do so would be once we have storage options.


Give collaborators commit access

I prefer a pull request based workflow where the PR originates in the same repo as the target, since then feedback can be given by committing changes to the PR branch
@mention me if you want commit access, and have signed the CLA

@drnic, @rkoster, @longnguyen11288, @malston - I'm giving you commit access to this repo.
Please leave commiting to master to me; but feel free to commit to branches and raise a PR to get those changes merged into master

Welcome @simonjohansson!

Danke!

Simon,
Have you accepted the invite yet?  Github says it is pending, and Gitter
won't let me add you to the chatroom until you are a member of the
logsearch org
On 1 April 2015 at 10:56, Simon Johansson<EMAIL_ADDRESS>wrote:

Danke!
—
Reply to this email directly or view it on GitHub
https://github.com/logsearch/logsearch-for-cloudfoundry/issues/15#issuecomment-88421343
.

--
David Laing
logsearch.io - build your own open source cloud logging cluster
http://davidlaing.com

@mrdavidlaing just accepted.

Cool - Gitter has let me invite you - see https://gitter.im/logsearch
On 1 April 2015 at 11:19, Simon Johansson<EMAIL_ADDRESS>wrote:

@mrdavidlaing https://github.com/mrdavidlaing just accepted.
—
Reply to this email directly or view it on GitHub
https://github.com/logsearch/logsearch-for-cloudfoundry/issues/15#issuecomment-88427974
.

--
David Laing
logsearch.io - build your own open source cloud logging cluster
http://davidlaing.com

@shinji62, I've just invited you to join the  logsearch-for-cloudfoundry.
Please accept the invitation so you can join us in the https://gitter.im/logsearch chat.


[BUG]: [SQLITE] Push command show SQL to apply with no schema changes

What version of drizzle-orm are you using?
0.31.2
What version of drizzle-kit are you using?
0.22.7
Describe the Bug
On previous drizzle version there was this bug with sqlite integer field in boolean mode did not handle default value correctly so I used 0 as never as a workaround. Now that it correctly handles false for a default value, drizzle-kit push wants to apply SQL commands to the database, (supposedly) successfully finishes and then running push shows the same SQL again.
Old schema on older drizzle version:
export const talentTrees = sqliteTable(
	'talentTree',
	{
		id: text('id', { length: 36 })
			.primaryKey()
			.$default(() => nanoid(10)),
		name: text('name', { length: 255 }).notNull(),
		public: integer('public', { mode: 'boolean' }).default(0 as never).notNull(),
		notes: text('notes'),
		class: integer('class').default(0).notNull(),
		icon: text('icon', { length: 255 })
			.default('inv_misc_questionmark')
			.notNull(),
		tree: text('tree', { mode: 'json' })
			.default('[]')
			.notNull()
			.$type<TalentTreeT>(),
		createdById: text('createdById', { length: 255 }).notNull(),
		createdAt: integer('createdAt', { mode: 'timestamp' }).notNull(),
		updatedAt: integer('updatedAt', { mode: 'timestamp' })
	},
	example => ({
		createdByIdIdx: index('trees_createdById_idx').on(example.createdById),
		nameIndex: index('trees_name_idx').on(example.name),
		publicIndex: index('trees_public_idx').on(example.public)
	})
);

New schema on current drizzle version:
export const talentTrees = sqliteTable(
	'talentTree',
	{
		id: text('id', { length: 36 })
			.primaryKey()
			.$default(() => nanoid(10)),
		name: text('name', { length: 255 }).notNull(),
		public: integer('public', { mode: 'boolean' }).default(false).notNull(),
		notes: text('notes'),
		class: integer('class').default(0).notNull(),
		icon: text('icon', { length: 255 })
			.default('inv_misc_questionmark')
			.notNull(),
		tree: text('tree', { mode: 'json' })
			.default('[]')
			.notNull()
			.$type<TalentTreeT>(),
		createdById: text('createdById', { length: 255 }).notNull(),
		createdAt: integer('createdAt', { mode: 'timestamp' }).notNull(),
		updatedAt: integer('updatedAt', { mode: 'timestamp' })
	},
	example => ({
		createdByIdIdx: index('trees_createdById_idx').on(example.createdById),
		nameIndex: index('trees_name_idx').on(example.name),
		publicIndex: index('trees_public_idx').on(example.public)
	})
);

SQL generated by push (seen with --verbose):
CREATE INDEX `trees_createdById_idx` ON `talent-builder_talentTree` (`createdById`);
CREATE INDEX `trees_name_idx` ON `talent-builder_talentTree` (`name`);
CREATE INDEX `trees_public_idx` ON `talent-builder_talentTree` (`public`);
ALTER TABLE `talent-builder_talentTree` RENAME TO `__old_push_talent-builder_talentTree`;
CREATE TABLE `talent-builder_talentTree` (
        `id` text(128) PRIMARY KEY NOT NULL,
        `name` text(256) DEFAULT 'New talent tree' NOT NULL,
        `public` integer DEFAULT 0 NOT NULL,
        `class` integer DEFAULT 0 NOT NULL,
        `icon` text(256) DEFAULT 'inv_misc_questionmark' NOT NULL,
        `tree` text DEFAULT '[]' NOT NULL,
        `createdById` text(255) NOT NULL,
        `createdAt` integer NOT NULL,
        `updatedAt` integer,
        `notes` text
);

INSERT INTO "talent-builder_talentTree" SELECT * FROM "__old_push_talent-builder_talentTree";
DROP TABLE `__old_push_talent-builder_talentTree`;

Expected behavior
It should not detect this as a change OR fix whatever it is detecting as a change and work again properly.
Environment & setup
No response

Upon some further investigation on a fresh local DB, it all seems to work, just the turso DB I'm connecting to does not want to get fixed by push.

Also I was getting a warning about the table needing to be dropped because it's not null and missing a default value when I tried adding a new column although the column definition was orderIndex: integer('orderIndex').default(0).notNull()

Update: I've found a workaround. With strict and verbose flags enabled, I was able to see the SQL that the drizzle-kit is trying to apply and then manually apply relevant queries myself directly.

Kit applied changes due to the difference in versions. Since it's currently not 1.0 yet, these things are bound to happen and the best course of action is to simply alter the generated migration to suit your needs.


Add example showing how existing vulkan graphical engine can integrate Kompute

Currently all the examples show how to use Vulkan Kompute from scratch, but it will be important to show how vulkan kompute can be used on an existing vulkan application. A potential way to explore this may be by converting some of the Sacha Willems Vulkan examples but using Kompute for the computational aspect https://github.com/SaschaWillems/Vulkan

Seconded! I am working on this and would like to add some machine learning features in the future. Would I be able to pass my existing Vulkan instance to Kompute?
Once I have figured out how this works I could contribute a tutorial on f.e. how to use Kompute in a Qt application...

Seconded! I am working on this and would like to add some machine learning features in the future. Would I be able to pass my existing Vulkan instance to Kompute?
Once I have figured out how this works I could contribute a tutorial on f.e. how to use Kompute in a Qt application...

@ttddee that project looks really interesting - I would be very keen on providing guidance / pointers on how this could be done. As of today you can actually just initialise a kp::Manager by passing your existing Vulkan Device/Instance, you can see the constructor in the documentation here https://kompute.cc/overview/reference.html#manager
Is this along the lines of what you're looking for?

@ttddee that project looks really interesting - I would be very keen on providing guidance / pointers on how this could be done. As of today you can actually just initialise a kp::Manager by passing your existing Vulkan Device/Instance, you can see the constructor in the documentation here https://kompute.cc/overview/reference.html#manager
Is this along the lines of what you're looking for?

@axsaucedo Yes thank you that looks like a good starting point. I will see if I can get something working alongside my renderer. Just for testing right now but it would be great for me if I can integrate Kompute and avoid having to ship some larger library.
I haven't had much time to look at this, so it might be a silly question, but are you planning to have support for convolutional networks in the future?

@axsaucedo Yes thank you that looks like a good starting point. I will see if I can get something working alongside my renderer. Just for testing right now but it would be great for me if I can integrate Kompute and avoid having to ship some larger library.
I haven't had much time to look at this, so it might be a silly question, but are you planning to have support for convolutional networks in the future?

Definitely, something that I would be really keen to explore! As of today, it would have to be implemented as a set of shaders - I would be very keen to work towards providing a set of kp::Operators that specialised for this, but it should be possible as of today to write these on top of Kompute, and would decrease the amount of boiler plate required for compute purposes.

Definitely, something that I would be really keen to explore! As of today, it would have to be implemented as a set of shaders - I would be very keen to work towards providing a set of kp::Operators that specialised for this, but it should be possible as of today to write these on top of Kompute, and would decrease the amount of boiler plate required for compute purposes.

Being able to do things like this in a simple manner would be amazing for image processing: https://github.com/kritiksoman/GIMP-ML/blob/master/gimp-plugins/PyTorch-Image-Dehazing/net.py

Being able to do things like this in a simple manner would be amazing for image processing: https://github.com/kritiksoman/GIMP-ML/blob/master/gimp-plugins/PyTorch-Image-Dehazing/net.py


Failing at differently sized groups

Thank you for a great library. There seem to be a bug for some numbers patterns with Indian style grouping (the default in Indian English, for instance):
> format("#,##,##0.###",2000000)
'2,000,000'

(Excepted: 20,00,000)
For reference, here's how it looks in Google Closure: https://github.com/google/closure-library/blob/master/closure/goog/i18n/numberformat.js#L892

Hi @rotsee!
Oops, yes... the library is only checking the size of the last group. It'll need to be refactored to format the Indian format grouping. Thanks for reporting the problem. I don't know when I'll have time to work on it. 😞


Small improvements to enable buffer aliasing/importing.

This adds the flags and populates them but the first real usage will come in #5472.

(you reviewed this in #5472 originally - I just pulled these independent bits out so that they can land now while work on that PR continues)


Dock clicked event in OS X

Is there a way to detect when the application's dock icon is clicked?

There is a activate-with-no-open-windows event.


Fix documentation parsing error

the documentation incorrectly tries to use the success function when rendering the objectClass
this makes the display of the parameter more consistent

Current coverage is 78.27%

Merging #110 into master will decrease coverage by -0.25% as of 78391e9

@@            master    #110   diff @@
======================================
  Files           37      37
  Stmts         2822    2822
  Branches       677     677
  Methods          0       0
======================================
- Hit           2216    2209     -7
- Partial        193     196     +3
- Missed         413     417     +4

Review entire Coverage Diff as of 78391e9

Powered by Codecov. Updated on successful CI builds.

Good catch, appreciate the PR!

Is there any way I can test if it actually looks ok?
I figured making it match what's in other doc blocks (e.g. the ones for
Relations and Objects) would work, but no idea about testing it... feeling
like a bit of a noob (;
Thanks
2015-12-04 17:03 GMT+00:00 Andrew Imm<EMAIL_ADDRESS>
Good catch, appreciate the PR!
—
Reply to this email directly or view it on GitHub
https://github.com/ParsePlatform/Parse-SDK-JS/pull/110#issuecomment-162022464
.


Pin reno to 1.9.0

This commit pins reno version to 1.9.0
For more information, please see the related card
Connects https://github.com/rcbops/u-suk-dev/issues/869

👍  perhaps we add >=1.8.0 like OSA does for newton

👍 LGTM


Roll engine 7f753f8f528a..3b1bebf2898b (3 commits)

https://github.com/flutter/engine/compare/7f753f8f528a...3b1bebf2898b
git log 7f753f8f528a1c92cc4f23957694623ad45104eb..3b1bebf2898b8ed8767eac53f189d237ba71f12f --no-merges --oneline
3b1bebf28 Roll src/third_party/skia e178c0586cd6..d7082a5805c8 (1 commits) (flutter/engine#8766)
5846027dc Roll src/third_party/skia 3349fb0195ff..e178c0586cd6 (1 commits) (flutter/engine#8763)
3493dcfba Rename tightWidth to longestLine (flutter/engine#8706)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Enable endpoint resolution 2.0 for all services

Remove conditionals that limited ruleset based endpoint resolution to only s3, s3control, and eventbridge. After this PR, ruleset based endpoint resolution will be in use for all services/versions for which an endpoint-rule-set-1.json file exists in the service's botocore/data/ subfolder.
Context:

#2785 introduced logic for ruleset based endpoint resolution and included rulesets for s3 and s3control.
#2801 added rulesets for all services
#2788 added eventbridge to the list of services for which ruleset based endpoint resolution is enabled.

Codecov Report
Base: 93.45% // Head: 93.40% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (fe39df0) compared to base (3d3f40c).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2802      +/-   ##
===========================================
- Coverage    93.45%   93.40%   -0.05%
===========================================
  Files           63       63
  Lines        13308    13301       -7
===========================================
- Hits         12437    12424      -13
- Misses         871      877       +6

Impacted Files
Coverage Δ

botocore/endpoint_provider.py
99.35% <ø> (-0.01%)
:arrow_down:

botocore/client.py
97.85% <100.00%> (ø)

botocore/compat.py
91.33% <0.00%> (-1.34%)
:arrow_down:

botocore/httpsession.py
91.40% <0.00%> (-0.43%)
:arrow_down:

botocore/utils.py
78.26% <0.00%> (-0.20%)
:arrow_down:

botocore/retries/adaptive.py
98.46% <0.00%> (-0.03%)
:arrow_down:

botocore/retries/base.py
100.00% <0.00%> (ø)

botocore/retries/throttling.py
100.00% <0.00%> (ø)

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

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


Unlock semver for babel-loader to fix dependency tree

babel-loader's version is locked to 8.1.0
After installing other packages that depend upon a later -- but still semverv-compatible -- version (such as @Storybook which currently requires version 8.2.2) running react-scripts results in the following error:
There might be a problem with the project dependency tree.
It is likely not a bug in Create React App, but something you need to fix locally.

The react-scripts package provided by Create React App requires a dependency:

"babel-loader": "8.1.0"

Don't try to install it manually: your package manager does it automatically.
However, a different version of babel-loader was detected higher up in the tree:

/Users/john/Projects/kovo/take-home/node_modules/babel-loader (version: 8.2.2)

Manually installing incompatible versions is known to cause hard-to-debug issues.

If you would prefer to ignore this check, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That will permanently disable this message but you might encounter other issues.

To fix the dependency tree, try following the steps below in the exact order:

1. Delete package-lock.json (not package.json!) and/or yarn.lock in your project folder.
2. Delete node_modules in your project folder.
3. Remove "babel-loader" from dependencies and/or devDependencies in the package.json file in your project folder.
4. Run npm install or yarn, depending on the package manager you use.

In most cases, this should be enough to fix the problem.
If this has not helped, there are a few other things you can try:

5. If you used npm, install yarn (http://yarnpkg.com/) and repeat the above steps with it instead.
   This may help because npm has known issues with package hoisting which may get resolved in future versions.

6. Check if /Users/john/Projects/kovo/take-home/node_modules/babel-loader is outside your project directory.
   For example, you might have accidentally installed something in your home folder.

7. Try running npm ls babel-loader in your project folder.
   This will tell you which other package (apart from the expected react-scripts) installed babel-loader.

If nothing else helps, add SKIP_PREFLIGHT_CHECK=true to an .env file in your project.
That would permanently disable this preflight check in case you want to proceed anyway.

P.S. We know this message is long but please read the steps above :-) We hope you find them helpful!

This PR allows later, non-breaking, versions of babel-loader to be accepted.

Hi @johnhenry, thanks for this - this will be addressed in #10797, and we expect it to be in the next minor release (in the coming weeks).


update-account branch issue

Hi, getting this error when running a simple ERC20 ownable transferOwnership on the update-account branch
The below is a snippet from my deploy.py
out = owner.send(erc20_contract_address, 'transferOwnership', [int(new_owner_address, 16)])  # due to int(, 10)

Using nile run --network 'goerli' scripts/deploy.py it spits out a tx hash but when I check its status below I get this bizarre error.
╰─ starknet get_transaction --network 'alpha-goerli' --hash 0x61e017ff4e30cb5151d86a0bf95b4da8eba6f9192302422b70e06c98ab34574                                        ─╯
Error: ValidationError: {'transaction': {'calldata': {0: ['Expected int string, got: "0x1".'], 1: ['Expected int string, got: "0x5816f092a90175976e5ddf92746f4245bf49196a064bcc8ee4389c9e1607201".'], 2: ['Expected int string, got: "0x14a390f291e2e1f29874769efdef47ddad94d76f77ff516fad206a385e8995f".'], 3: ['Expected int string, got: "0x0".'], 4: ['Expected int string, got: "0x1".'], 5: ['Expected int string, got: "0x1".'], 6: ['Expected int string, got: "0x1573f855755959b21b69a38805d406a45a64954a47df6ba226494684851697f".'], 7: ['Expected int string, got: "0x0".']}, 'max_fee': ['Unknown field.']}, 'transaction_failure_reason': {'tx_id': ['Missing data for required field.']}}

Any ideas?

Hey, thanks for reporting this. That is a strange error! I rechecked the tx hash and the error message that I'm seeing is an ENTRY_POINT error. It seems that the transferOwnership method is being called on the account contract and not on the erc20 contract. Based on the small snippet, it's hard to see where the error is coming from. Could you provide some more details on the script you're running?

@andrew-fleming sure thing. Here's my deploy.py script which I run using nile run --network 'goerli' scripts/deploy.py
import sys
import os

current = os.path.dirname(os.path.realpath(__file__))
parent = os.path.dirname(current)
sys.path.append(parent)

def run(nre):
    from tests.utils import (str_to_felt)

    owner = nre.get_or_deploy_account("PKEY1")

    eth_contract_address = ""
    try:
        eth_contract_address, _ = nre.deploy(
            "ERC20_Mintable_Capped",
            alias="eth",
            arguments=[
                str(str_to_felt("Ethereum")), str(str_to_felt("tETH")),
                "18",
                owner.address,
                "1000000",  # token supply cap
                "0"         # Uint256(low=a, high=b)
            ]
        )
    except:
        eth_contract_address, _ = nre.get_deployment("eth")

    print(f'ETH Token - Address: {eth_contract_address}')

    erc20_contract_address = ""
    try:
        erc20_contract_address, _ = nre.deploy(
            "ERC20_Mintable_Capped",
            alias="erc20",
            arguments=[
                str(str_to_felt("Serity")), str(str_to_felt("tSER")),
                "18",
                owner.address,
                "1000000",  # token supply cap
                "0"         # Uint256(low=a, high=b)
            ]
        )
    except:
        erc20_contract_address, _ = nre.get_deployment("erc20")

    print(f'Serity Token - Address: {erc20_contract_address}')

    curve_contract_address = ""
    try:
        curve_contract_address, _ = nre.deploy(
            "BondingCurve",
            alias="curve",
            arguments=[
                owner.address,
                "1000000",  # curve supply cap
                "0",
                "5",        # scale
                "0",
                "1",        # buffer
                "0",
                "1",        # discount
                "0",
                eth_contract_address,
                erc20_contract_address
            ]
        )
    except:
        curve_contract_address, _ = nre.get_deployment("curve")

    print(f'Bonding Curve - Address: {curve_contract_address}')

    out = owner.send(erc20_contract_address, 'transferOwnership',
                     [int(curve_contract_address, 16)])  # due to int(, 10)
    print(f'{out}')

@0xRho  excellent, thank you!
So I have a couple ideas of what it could be. Since the error message is a ENTRY POINT error, can you confirm the transferOwnership method isn't transfer_ownership or some variation thereof in your contract?
Also, I had issues with setting contract address arguments as base 16 integers in Nile's send method. Instead, I use and recommend int(x, 0). If it's not the former, this might be the culprit.

@andrew-fleming thanks for the response.
Yes, can confirm the method is called transferOwnership, see below.
@external
func transferOwnership{syscall_ptr : felt*, pedersen_ptr : HashBuiltin*, range_check_ptr}(
        new_owner : felt) -> (success : felt):
    Ownable_transfer_ownership(new_owner)
    return (TRUE)
end

Followed your suggestion and used
    out = owner.send(erc20_contract_address, 'transferOwnership',
                     [int(curve_contract_address, 0)])  # due to int(, 10)

this is the output of running nile run --network 'goerli' scripts/deploy.py
ETH Token - Address: 0x05e74ac4b8bf8cf8c4a1173db0b1d7b7ddc069b9da6d9863dffb8d7441f9d1ff
Serity Token - Address: 0x05ec8f09eb39045aa66435ebc93faa0ba3dd64fa3553ab3f74c637955f88bdfb
Bonding Curve - Address: 0x0766196e2177b37b5f89b2d11e9806803993f09b09987210ab4174c383f7d333
Invoke transaction was sent.
Contract address: 0x066b9ba7d4d17806b5849a474ffdfe2b8d25a6447811aeb3e499459c711ce851
Transaction hash: 0x1323c07965ba82837f557318f2f603691fb967ee2edb019d4d64f010d3f1d0b

running get transaction again returns
╰─ starknet get_transaction --network 'alpha-goerli' --hash 0x1323c07965ba82837f557318f2f603691fb967ee2edb019d4d64f010d3f1d0b
Error: ValidationError: {'transaction_failure_reason': {'tx_id': ['Missing data for required field.']}, 'transaction': {'calldata': {0: ['Expected int string, got: "0x1".'], 1: ['Expected int string, got: "0x5ec8f09eb39045aa66435ebc93faa0ba3dd64fa3553ab3f74c637955f88bdfb".'], 2: ['Expected int string, got: "0x14a390f291e2e1f29874769efdef47ddad94d76f77ff516fad206a385e8995f".'], 3: ['Expected int string, got: "0x0".'], 4: ['Expected int string, got: "0x1".'], 5: ['Expected int string, got: "0x1".'], 6: ['Expected int string, got: "0x766196e2177b37b5f89b2d11e9806803993f09b09987210ab4174c383f7d333".'], 7: ['Expected int string, got: "0x0".']}, 'max_fee': ['Unknown field.']}}

@andrew-fleming how do you know this is an entry point error? and any other ideas

@0xRho hmmm when I run get_transaction, this is what I'm seeing:
(py_venv) andrewfleming@Andrews-MacBook-Pro cairo-contracts % starknet get_transaction --hash 0x1323c07965ba82837f557318f2f603691fb967ee2edb019d4d64f010d3f1d0b --network alpha-goerli
{
    "status": "REJECTED",
    "transaction": {
        "calldata": [
            "0x1",
            "0x5ec8f09eb39045aa66435ebc93faa0ba3dd64fa3553ab3f74c637955f88bdfb",
            "0x14a390f291e2e1f29874769efdef47ddad94d76f77ff516fad206a385e8995f",
            "0x0",
            "0x1",
            "0x1",
            "0x766196e2177b37b5f89b2d11e9806803993f09b09987210ab4174c383f7d333",
            "0x0"
        ],
        "contract_address": "0x66b9ba7d4d17806b5849a474ffdfe2b8d25a6447811aeb3e499459c711ce851",
        "entry_point_selector": "0x15d40a3d6ca2ac30f4031e42be28da9b056fef9bb7357ac5e85627ee876e5ad",
        "entry_point_type": "EXTERNAL",
        "max_fee": "0x0",
        "signature": [
            "1371163946402037437721147194656151008959442413830890024041243938426228295232",
            "1580786845979697852904815929486029443920238206724216404724579276763989038370"
        ],
        "transaction_hash": "0x1323c07965ba82837f557318f2f603691fb967ee2edb019d4d64f010d3f1d0b",
        "type": "INVOKE_FUNCTION"
    },
    "transaction_failure_reason": {
        "code": "ENTRY_POINT_NOT_FOUND_IN_CONTRACT",
        "error_message": "Entry point 0x15d40a3d6ca2ac30f4031e42be28da9b056fef9bb7357ac5e85627ee876e5ad not found in contract with address 0x66b9ba7d4d17806b5849a474ffdfe2b8d25a6447811aeb3e499459c711ce851."
    }
}

Assuming that the get_transaction data that I'm looking at is correct, I was thinking that the issue might be with transferOwnership because it's the only method I see called in the script. Its selector however is 0x14a390f291e2e1f29874769efdef47ddad94d76f77ff516fad206a385e8995f (different from the error message). One route I would consider would be to break this script down, deploy each portion separately, and build up from there to pinpoint exactly where the issue is coming from.
Let me know if there's something wrong with the error message I received and if you can reproduce it.

Found out what those entry points are
# 0x14a390f291e2e1f29874769efdef47ddad94d76f77ff516fad206a385e8995f
print('transferOwnership', hex(get_selector_from_name('transferOwnership')))

# 0x15d40a3d6ca2ac30f4031e42be28da9b056fef9bb7357ac5e85627ee876e5ad
print('__execute__', hex(get_selector_from_name('__execute__'))) # meant to be in Account.cairo

I think the Account.cairo contract deployed was outdated so now I'm verifying all my pckgs and versions.
@andrew-fleming can you dump your installed packages and their versions? specifically cairo-lang, cairo-nile, starknet-devnet.
This is what I have now
cairo-lang==0.7.0 # latest is 0.8.0
cairo-nile==0.5.2 # previously 0.4.0
starknet-devnet==0.1.13 # latest is 0.1.19

When I update my cairo-lang and starknet-devnet to its latest - running nile setup <alias> fails so I left it as is but updating cairo-nile to 0.5.2 (previously 0.4.0) fails to run owner = nre.get_or_deploy_account("PKEY1").

Update - works now after nuking and refreshing all my pckgs and versions. Closing this issue now. Thanks for the help @andrew-fleming.

@0xRho ahhhh I see haha excellent! I'm glad you got it working :)


batch scanning feature is available?

Dear team,
I have a project requirement with Batch scanning / Continuous document scanning of document.
Just curious to know whether this library features this?
Appreciate your support.
Thanks

captureMultiple from feature section
I hope this is what I am looking for. Closing this for now.

captureMultiple from feature section
I hope this is what I am looking for. Closing this for now.


Added more details to Particle2D visibility_rect reference

Added more details to better describe what the visibility_rect can be used for.

Hey! Thanks for this. Looks good, but should be indented one more tab.
Also, I'd try to add [code] tags:
 		 Visibility Helper
		 If the [code]visibility_rect [/code] is not in view the particles will not be rendered.
		 Change the size of the [Rect] with  [code]w [/code] and  [code]h [/code].
		 Change the position with  [code]x [/code] and  [code]y[/code].
		 Example: To match a box particle shape of  [code]x:100 [/code]  [code]y:200 [/code]
		  [code]visibility_rect [/code] would be  [code]w:100 [/code]  [code]h:200 [/code]  [code]X:-50 [/code]  [code]Y:-100 [/code]
		 Tips:  [code]visibility_rect [/code] can be automatically generated using the UI:
		  [code]Particles > Generate Visibility Rect [/code]
		 Which appears on the toolbar when a [Particles2D] node is selected.


ROX-17702: GetByQuery product usage

Description
Make use of the newly implemented query by timerange.
Switched the signatures from *types.Timestamp to time.Time due to the new query interface.
Based on #7179.
Proto: #7136.
Planned optimization: make use of the pagination + sorting with Limit(1) to query Max values instead of computing in the backend.
Checklist

[x] Investigated and inspected CI test results
[x] Unit test and regression tests added
[ ] ~Evaluated and added CHANGELOG entry if required~
[ ] ~Determined and documented upgrade steps~
[ ] ~Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Unit tests.


Add saving and loading to text file

Currently saving and loading is done through print statements and modifying code variables directly. It would make more sense and be more extendible if these saves were recorded as files.

This was implemented in 8fa484d6d424b277d4f2b9510579fa462f85d6f4.
This could be improved further by adding a dropdown/file selector in the gui to choose your radpath, but I think this should be a separate issue.


#1717

Hi iBotPeaches,
Maybe you could explain your answer to this in a bit more detail:
Help / Feature Request : Any way to add resources to an existing app? #1717
How to do this? Links? etc?
explicitly generate a resource ID, abiding by the rules of that and use public.xml to maintain that ID for the resource(s) in question.

I'm not quite sure what it is that you're trying to find out. The issue you linked to talked about adding new resources to an existing APK and referencing those in the smali.
If you add a new resources to a decompiled APK then, when you recompile, public.xml will be updated with hexadecimal references to those resources. Those references can then be used in the smali.
There's a good example of this sort of thing here: https://forum.xda-developers.com/showthread.php?t=1484578&page=4 (copy and paste the link - I can't get links to work properly in github for some reason..)
XDA is a great place for learning about modding Android. If you have any further questions that aren't about using apktool itself, you should look there.

With the bug tracker being as large and behind times, unforunately can't really hold a support dialog here. As mentioned above, XDA has plenty of threads and one quick Google search led me to - https://stackoverflow.com/q/52742731/455008
Which basically explains the procedure in detail

Find last resource id in type you are interested in
Increase by 1 (remember it is hex, so go back to decimal +1 then back to hex if you can't work in base 16)
Add that resource to the R${type}.smali file
Use that resource id in the source file for what you are trying to do.


Cannot turn on/off my accessories

Hi, i've uploaded code into my ESP32.
I've seen the accessory in my Home Application, but when i try to turn on/off my light, it didn't work
Do you guys have the guide for my issue.
Thank you

i used example 05 : workingLED btw

I assume that the Accessory is functioning correctly in the Home App such that when you turn the light on/off it changes to on/off in the Home App?  If so, and it is only the LED itself that is not turning on/off, this suggests a problem with the LED, the way you've connected it your ESP32, or the pin that you used.  Have you verified the LED operates correctly on the pin you chose without HomeSpan (to test this you can write a simple script that turn on/off the LED every second)?


Create Order Panic Error

I am using the correct test key and secret and getting error
map[]
%!v(PANIC=Error method: runtime error: invalid memory address or nil pointer dereference)

package main

import (
	"fmt"

	"github.com/razorpay/razorpay-go"
)

func main() {
	client := razorpay.NewClient("rzp_test_9lsYF6ARKRbVbs", "l4KsqSMSB02smJBnVJgZPRmE")
	data := map[string]interface{}{
		"amount":          1234,
		"currency":        "INR",
		"receipt_id":      "some_receipt_id",
		"payment_capture": 1,
	}

	extraHeaders := map[string]string{}

	body, err := client.Order.Create(data, extraHeaders)
	fmt.Println(body)
	fmt.Println(err)

}

Getting the same error:
map[]
%!v(PANIC=runtime error: invalid memory address or nil pointer dereference)

Is it a working repository or just wasting our time?

@salsanej Repo is working but they have bug in error reporting so it will not reveal you actual error from the api. So you just have to make sure are you posting the right data to the api.
If you really want to see the error then you have to make edit to this file for now vendor/github.com/razorpay/razorpay-go/requests/request.go in doRequestResponse func

@venkatvghub
I have added a fix for this and created pull request.

hi can we get this issue fixed on priority

Seems to be fixed in #58

@khanakia Thanks for reporting this issue . It has already been fixed in one of recent releases. Please update to the most recent version to resolve this issue.
Closing this issue for now, Please don't let that stop you from re-opening if the issue is persisting for you.


tx error with Github action

l try to push and pull through github action automatically, but it goes wrong and below is the wrong message .
tx INFO: Updating /home/runner/.transifexrc file...
tx INFO: Using TX_TOKEN environment variable. Credentials won't be saved to .transifexrc.
tx ERROR: Authorization Required
tx ERROR:
Error: Authentication failed. Please make sure your credentials are valid.
For more information, visit:
https://docs.transifex.com/client/client-configuration#-transifexrc.

l have checked my .transifexrc file and it is correct since l can use tx push -s from the local development environment. So what’s wrong with that? And here is my repo: mendax1234/6940Swerve-docs: FRC Team 6940’s Swerve Documentation Powered by Read the Docs (github.com)

Hey @mendax1234,
I believe you are still using the Python client.
I would suggest closing this issue and start a new discussion in the repo for the old Python version of the client.

Hey @mendax1234, I believe you are still using the Python client. I would suggest closing this issue and start a new discussion in the repo for the old Python version of the client.

Sure , would you please give me some suggesstions on how to fix this bug? Thanks!

I have created a new issue. https://github.com/transifex/transifex-client/issues/342


Jira create allow to pass summary and description

I'd expect jira create --issuetype=Task --summary="Test" --description="My description" to create an issue with the Summary Test and My description as the description, however the --summary and --description option are not being recognized.
Is there any reason for this or just an oversight?

There is a generic --override or -o to set arbitrary key=value to use in your templates.  So this should work:
jira create --issuetype=Task -o summary="Test" -o description="My description"

--issueType and --project are special because they are used to formulate what is exposed in the template, so different issueTypes or projects will create different template documents to edit.

That's neat. I've ended up writing a custom command for this:
- name: ctask
  help: creates a task in the given project
  options:
    - name: project
      short: p
      default: $JIRA_PROJECT
    - name: description
      short: d
      default: ""
  args:
    - name: summary
      required: true
  script: |-
    {{jira}} create --issuetype=Task -o summary="{{args.summary}}" -o description="{{options.description}}" --noedit

When I used it like this: jira ctask "Blub" -d="Description" everything worked. However in the actual description of the title in jira I saw a = prefixed before my description.
Do you happen to know the reason for this?

the short option will be in the form of -dDescription or -d Descrption where the = is not expected/required with short options. This is from the getopt parser we use, so I don't have any control over that one.  For long options the form would be --description Description or --description=Description.  I am not sure why the short form does not allow for an =.

Huh interesting. Can we change the behavior in getopt parser?
In any case I'm going to close this. Thanks for the help.

-d "Description" does the trick and works correctly.


[FAB-13460] Add Support for TLS1.3

Add support for TLS1.3 and adds a tests to ensure these versions work on the comm stack.
Signed-off-by: Brett Logan<EMAIL_ADDRESS>

This was meant to be a draft, the test you commented on I'm aware of the problem I have in it. But I'll take the other comments under advisement. My opinion there was, keeping it as it was (limiting the supported TLS types) and setting the max version prevents unintended consequences when upgrading go versions that support new TLS versions in the future. But removing the default limitation also reduces the maintenance we have to do. I'm ok with either method, but both were used and I chose the former. If we agree the latter is the way we want to go, I'll remove the max version.

Correct me if I'm wrong, but - in TLS 1.3 the certificates are encrypted (unlike in 1.2).
Doesn't this make troubleshooting harder?
Can anyone explain the benefits of using TLS 1.3 in Fabric where we have long lasting TLS connections between nodes? The TLS handshake being faster now doesn't help us much...

Correct me if I'm wrong, but - in TLS 1.3 the certificates are encrypted (unlike in 1.2).
Doesn't this make troubleshooting harder?
Can anyone explain the benefits of using TLS 1.3 in Fabric where we have long lasting TLS connections between nodes? The TLS handshake being faster now doesn't help us much...

I don't think the troubleshooting you can do from within the Fabric context is any worse here.
There are really two reasons to allow TLS 1.3:

it's supported by Go and gRPC so no reason not to allow it
some users might have requirements to to TLS 1.3 in the near future

I don't think the troubleshooting you can do from within the Fabric context is any worse here.

Well in TLS 1.2 I see the certificates in the clear so I can analyze traffic and catch validation chains that do not appear in the channel config, or mismatching SANs. With TLS 1.3 I don't have that because it's encrypted.

/azp run


[move-analyzer] basic function name completion

Motivation
The current move-analyzer has limited code completion support, this PR adds primitive types and function name completion.
Note: the IdentType in FunctionIdentTypeMap is not used in this PR. However, it will be used for function signature help in a future PR.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)

Open the extension in debug mode
There should be a code completion suggestion for primitive types if you start typing a prefix of any primitive types (e.g. u64, bool etc.)
There should be a code completion suggestion for any function name prefix inside the same file

PRs to add after the current one

Add more intellisense support. (e.g. show function signature help, module method completion, auto imports etc.) Each of these is likely going to be a separate PR for easier review.

In the code above you have only one definition of some_fun but two UseDefs for this function.
More generally, you don't seem to be using any information in the UseDef and FunctionUseDefMap is used more like a set. Perhaps it makes sense to use the set structure instead (unless I am missing something).
Finally, could you please add a few integration tests in a similar style as #328 does?

Correct, I am not using any information in UseDef in this PR. The idea is in a following PR, I would add signature help for each function, so I will need the function's signature.
Perhaps storing the IndentType is more semantically correct than UseDef.
Yep, I will add some integration tests.

In the code above you have only one definition of some_fun but two UseDefs for this function.
More generally, you don't seem to be using any information in the UseDef and FunctionUseDefMap is used more like a set. Perhaps it makes sense to use the set structure instead (unless I am missing something).
Finally, could you please add a few integration tests in a similar style as #328 does?

Correct, I am not using any information in UseDef in this PR. The idea is in a following PR, I would add signature help for each function, so I will need the function's signature.
Perhaps storing the IndentType is more semantically correct than UseDef.

It may be, particularly since you are introducing a new map. If there is future use of the data you are not yet using, please at least mention this in comments to avoid confusion.

Yep, I will add some integration tests.

Thank you!

It may be, particularly since you are introducing a new map. If there is future use of the data you are not yet using, please at least mention this in comments to avoid confusion.

Apologies, will do so next time!

@awelc added all the suggested changes!

hey @awelc, do I need to create a new PR because my current PR has merge conflicts?

hey @awelc, do I need to create a new PR because my current PR has merge conflicts?

Please don't. You need to rebase on top of the main branch tip, resolve conflicts, and force push your changes (that's the typical flow used with GitHub).

hey @awelc, do I need to create a new PR because my current PR has merge conflicts?

Please don't. You need to rebase on top of the main branch tip, resolve conflicts, and force push your changes (that's the typical flow used with GitHub).

I asked, because in CONTRIBUTING.md, it says PRs with merge commits will be rejected.

hey @awelc, do I need to create a new PR because my current PR has merge conflicts?

Please don't. You need to rebase on top of the main branch tip, resolve conflicts, and force push your changes (that's the typical flow used with GitHub).

awesome, I misread the CONTRIBUTING.md doc

Running cargo test in language/move-prover/move-docgen throws this error, any idea how to fix it? I didn't change anything in the language/move-prover/move-docgen folder.

@awelc should be good to go now

Looks great! Just a quick typo fix please before I merge.

awesome, done!


Textures are loading twice in memory causing doubled CPU spike and memory usage

Hi @hybridherbst @pfcDorn
I'm loading a GLB through code using default settings provided by the API. I wonder why is there loading of a texture is twice when a GLB is loaded. Can we change it to use single texture instead?

There are 2 crucial issues because of this.

CPU spike gets doubled because it first calls LoadImage() and then Object.Instantiate() both in a same frame.
Memory usage gets doubled because both textures resides in memory until a GLB is destroyed.

Looking forward for your reply.... Thankyou

Hey,
that is as intended. At line 505, it will only be created when the previous created texture has other sampler settings (like filterMode or WarpMode).
When they have both the same sampler settings, it's likely a bug.

Can we just change sampler settings of first one and assign it to the second one? As the first one (I suppose) has no use here after...

So the first created Texture is not in use? Maybe you can share the file here?

It is the same case for any GLB it'll load the texture twice.... First one has only 1 reference which is (I guess) ImageCache list... and the second one is assigned to materials and has a proper use

i will check! that's very likely a bug :)

It is a very old bug since first build which I once resolved as well but wasn't able to share at that time unfortunately.
Basically, the problem is that when you load any GLB from code using GLTFComponent in a build and then if you use memory profiler by Unity then it'll show that every texture is loaded 2 times. First texture has only 1 reference and the second has proper use of it. Also if you use CPU profiler then you can see big spikes when a texture is loaded because it once load a texture through LoadImage() and then Object.Instantiate() on same frame. Above mentioned places are the exact points which I found after profiling.
Hope these information help.

Still needs some tests, but should prevent this behaviour now: 73f7b66cb2311630445999ccff71177d1649b5af

Hi @pfcDorn... Thankyou for sharing the implementation
I managed to brought back my implementation of the issue.
I would prefer older implementation because of many reasons

If isFirstInstance is null then there might be issues to call Object.Instantiate(null) in that case
isFirstInstance can't be null because of if (!source) return; at line 427
No if-else needed for Object.Instantiate so it is a simpler version of it
I've tested this on few test cases and was working fine on previous version

Here is the commit I made.

With your solution you just removed completly the support for multiple sampler settings on a single texture. In Unity we need to duplicate a texture when we have in the gltf different sampler settings for the same texture.
Lets say we use Texture A in Material 1 and in Material 2. In Material 1 the texture will have WrapMode Repeat, and in Material 2 it will set to Clamp.  So in this case we must dublicate the texture on import to support this. Otherwise one material will not be displayed correctly.

Yes but this statement is true for both of the above solutions I suppose.
The issue was that if you've 1 texture and 1 material it was still loading the texture twice.

thats the reason why i added the isFirstInstance, so it only create the texture twice or more when it's needed (in case of multiple sampler states)

But I think isFirstInstance might always be true because of the lines on 429 and 430 ???
var source = _assetCache.ImageCache[sourceId];
if (!source) return;

Which means matchSamplerState on line 491 will always be false

@pfcDorn did you manage to find a work around?

Oops! I was missing a point there... Thankyou


Most used links

Is it possible for the links to be the top 6 used websites?

Still considering this. Trying to keep the size of the extension and the amount of resources minimal as well as permissions. I'm unclear at the moment whether this would affect that or not. Will look into it later on however. Thanks for the support!

This should be a low priority item, as there is the ability to add favorites. This should hold me over :)

I want too this feature. Six links too few. May be make responsible design? If more links added set text smaller or add new line for links? or ex. if 7 links added, set in first line 4 and second line 3

This has been completed in CaretTab v3 which is currently in Beta at the moment, so I'll be closing this as fixed. If you would like to check it out, you can sign up for the beta here: https://github.com/bluecaret/carettab/issues/104


wsl Ubuntu18.04安装失败,无法下载插件

[100%] Linking CXX shared library /home/kratos/.vim/plugged/YouCompleteMe/third_party/ycmd/ycm_core.so
[100%] Built target ycm_core
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:--  0:00:02 --:--:--     0
curl: (60) SSL: no alternative certificate subject name matches target host name 'raw.githubusercontent.com'
More details here: https://curl.haxx.se/docs/sslcerts.html
curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.
Error detected while processing /home/kratos/.vimrc:                                                                    line  109:
E117: Unknown function: plug#begin
line  111:
E492: Not an editor command: Plug 'chxuan/cpp-mode'
line  112:
E492: Not an editor command: Plug 'chxuan/vim-edit'                                                                     line  113:
E492: Not an editor command: Plug 'chxuan/change-colorscheme'
line  114:
E492: Not an editor command: Plug 'chxuan/prepare-code'
line  115:
E492: Not an editor command: Plug 'chxuan/vim-buffer'
line  116:
E492: Not an editor command: Plug 'chxuan/vimplus-startify'
line  117:
E492: Not an editor command: Plug 'chxuan/tagbar'                                                                       line  118:
E492: Not an editor command: Plug 'Valloric/YouCompleteMe'
line  119:
E492: Not an editor command: Plug 'Yggdroot/LeaderF'
line  120:
E492: Not an editor command: Plug 'mileszs/ack.vim'
line  121:
E492: Not an editor command: Plug 'easymotion/vim-easymotion'                                                           line  122:
E492: Not an editor command: Plug 'haya14busa/incsearch.vim'
line  123:
E492: Not an editor command: Plug 'iamcco/mathjax-support-for-mkdp'
line  124:
E492: Not an editor command: Plug 'iamcco/markdown-preview.vim'
line  125:
E492: Not an editor command: Plug 'jiangmiao/auto-pairs'
line  126:
E492: Not an editor command: Plug 'scrooloose/nerdtree'
line  127:
E492: Not an editor command: Plug 'tiagofumo/vim-nerdtree-syntax-highlight'
line  128:
E492: Not an editor command: Plug 'Xuyuanp/nerdtree-git-plugin'
line  129:
E492: Not an editor command: Plug 'godlygeek/tabular'
line  130:
E492: Not an editor command: Plug 'tpope/vim-fugitive'
line  131:
E492: Not an editor command: Plug 'tpope/vim-surround'
line  132:
E492: Not an editor command: Plug 'tpope/vim-commentary'                                                                line  133:
E492: Not an editor command: Plug 'tpope/vim-repeat'
line  134:
E492: Not an editor command: Plug 'tpope/vim-endwise'
line  135:
E492: Not an editor command: Plug 'octol/vim-cpp-enhanced-highlight'                                                    line  136:
E492: Not an editor command: Plug 'vim-airline/vim-airline'
line  137:
E492: Not an editor command: Plug 'vim-airline/vim-airline-themes'                                                      line  138:
E492: Not an editor command: Plug 'ryanoasis/vim-devicons'                                                              line  139:
E492: Not an editor command: Plug 'junegunn/vim-slash'                                                                  line  140:
E492: Not an editor command: Plug 'junegunn/gv.vim'                                                                     line  141:
E492: Not an editor command: Plug 'kana/vim-textobj-user'                                                               line  142:
E492: Not an editor command: Plug 'kana/vim-textobj-indent'                                                             line  143:
E492: Not an editor command: Plug 'kana/vim-textobj-syntax'                                                             line  144:
E492: Not an editor command: Plug 'kana/vim-textobj-function'                                                           line  145:
E492: Not an editor command: Plug 'sgur/vim-textobj-parameter'                                                          line  146:
E492: Not an editor command: Plug 'Shougo/echodoc.vim'                                                                  line  147:
E492: Not an editor command: Plug 'terryma/vim-smooth-scroll'                                                           line  148:
E492: Not an editor command: Plug 'rhysd/clever-f.vim'                                                                  line  149:
E492: Not an editor command: Plug 'vim-scripts/indentpython.vim'                                                        line  151:
E117: Unknown function: plug#end
Press ENTER or type command to continue                                                                                __                __
__   // ___  ____  / /_  _______
\ \ / / / __ `__ / __ / / / / / /
\ V / / / / / / / // / / // (_  )
_/// // // ,//_/__/
//                     ...is now installed!
Just enjoy it!
p.s. Follow me at https://github.com/chxuan.
It takes 11 minutes.

curl failed to verify the legitimacy of the server and therefore could not
establish a secure connection to it. To learn more about this situation and
how to fix it, please visit the web page mentioned above.

上面报错大概意思是说通过curl来下载plugin.vim失败了，你可以尝试多次运行install.sh脚本试试

我没有在wsl版本的系统里面测试过，不保证能正常安装成功，建议使用虚拟机安装系统来体验vimplus或者直接下载vimplus的docker版本来体验。

确实是这样
我手动安装了plugin.vim，然后再在里面安装一下就好了

@Kratos0902 好的，欢迎使用vimplus。


chore: add blobfuse2 support to 24.04

What type of PR is this?
This PR adds blobfuse2 installation script for Ubuntu 24.04, previously, the scripts skipped it since the package wasn't available upstream, it is now.
https://packages.microsoft.com/ubuntu/24.04/prod/pool/main/b/blobfuse2/

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3044

Relevant Lines:
3962

💛  - Coveralls


bootstrap.sh fails on FreeBSD

Expected behavior
Actual behavior
GOBIN is now /usr/github/Harvey-OS/harvey/freebsd_amd64/bin
Building harvey-os.org commands into freebsd_amd64/bin
# github.com/insomniacslk/dhcp/dhcpv4/client4
/usr/home0/bakul/go/src/github.com/insomniacslk/dhcp/dhcpv4/client4/client.go:127:25: undefined: unix.AF_PACKET
/usr/home0/bakul/go/src/github.com/insomniacslk/dhcp/dhcpv4/client4/client.go:127:68: undefined: unix.ETH_P_IP
/usr/home0/bakul/go/src/github.com/insomniacslk/dhcp/dhcpv4/client4/client.go:135:12: undefined: unix.SockaddrLinklayer

Steps to reproduce

Run bootstrap.sh on a recent FreeBSD

QEMU output

```

```

https://github.com/Harvey-OS/harvey/wiki/Getting-Started has the following section. Perhaps it should be removed?
FreeBSD
(Regulary tested with latest release version)
If you are in a FreeBSD, you should do:

pkg install git go bison
export PATH=/usr/local/bin:$PATH


R script in zip folder for download is incomplete

The ancom_v2.1.R script contained in the scripts file in the zipped folder for download does not match the same script listed here in the github. It's missing key components to properly include out and is completely missing the commands to create and include fig in the res output when running ANCOM.

Hi @pricesdr,
Did you mean the zip folder of "Third release of ANCOM"? You are right, that zip folder is not up-to-date. I would recommend downloading the whole repository directly.
The ANCOM function in this repository is not actively maintained due to the limited resources I currently have. It is planned to be incorporated into the ANCOMBC R package later.
Best,
Huang


Support for inserting instructions in expression instrs: wast crate

I have been trying insert instructions dynamically to the array of instructions of a function, but as the Instruction enum does not implement clone, I am not being able to do it, is there any way of doing this?

I think it'd be reasonable to implement Clone for instructions, if you're up for it a PR would be most welcome!

Sorry for taking so long answering, I agree and I am up to do it, as I am relatively new to rust can you guide me please? @alexcrichton

I believe you'd start by adding Clone here and then playing whack-a-mole with compiler error messages to add Clone to more things too

I believe you'd start by adding Clone here and then playing whack-a-mole with compiler error messages to add Clone to more things too

Ok I will, thank you 🙂


Allow for custom record grouping

Introduce a new property file.record.grouper.builder to specify a builder for a grouper
enable the grouper to define additional properties and associated documentation
Minor refactors of the 'File' common configuration shared between S3 and GCS introduce some more validators

This still needs tests

Hi aiven-maintainers

Can I get a review of this PR please

hi @jjaakola-aiven @AnatolyPopov @gharris1727 @eliax1996
apologies - I am not sure what group shall I ping, could you pls take a look at this PR?
your help is much appreciated!
Thanks!


Is there a way to setup beaker without sudo permissions?

I tried installing dependencies of the beaker on my ubuntu machine, but it requires sudo access. I don't have sudo access on my machine. Is there a way to set it up locally? I can do that easily for jupyter.

yes the core dependencies are just nginx and java8.  while the script for installing them automatically uses apt-get (https://github.com/twosigma/beaker-notebook/blob/master/launcher/ubuntu/install-runtime-core-dependencies.sh), any way that you can download and get them into the PATH should be fine.


Refactor node agent

Release note:

@myidpt: Adding do-not-merge/release-note-label-needed because the release note process has not been followed.

One of the following labels is required "release-note", "release-note-action-required", or "release-note-none".
Please see: https://github.com/kubernetes/community/blob/master/contributors/devel/pull-requests.md#write-release-notes-if-needed.
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@myidpt: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/auth-presubmit.sh
9ddfa680496db2c1215c93f5c82ed05f2e081094
link
/test auth-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


RC #121 - Upgrade dependencies

This pull request makes the following changes:

Upgrades the dependencies for React, Webpack, React-Google-Maps, and more!
Fixes Flow type and ESLint errors
Sets up a GitHub Action to deploy the Storybook site to GitHub pages: https://performant-software.github.io/react-components

@dleadbetter Question: can we configure the accessibility storybook addon specifically to include WCAG 2.1 level AA? That's the standard we're aiming for in this next project. I see that wcag is in the list, but not sure if it's the level they need. Thanks!

@dleadbetter Question: can we configure the accessibility storybook addon specifically to include WCAG 2.1 level AA? That's the standard we're aiming for in this next project. I see that wcag is in the list, but not sure if it's the level they need. Thanks!

@jamiefolsom - That's a great question. I've updated the Google Doc to include A11y as a separate line item and we'll tackle that as it's own task.


Riato 582 get stats

Summary: Get stats for the number of rendered and dropped frames
Type: Fix
Test Plan: UT, CT, test that wpe-webkit can get stats on XiOne US
Jira: RIALTO-582

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56242/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56276/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56280/

Please make PR title more informative

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56370/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56373/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56374/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56375/

Blackduck scan failure details
Summary: 0 violations, 0 files pending approval, 1 file pending identification.

Protex Server Path: /home/blackduck/github/rialto/318/rdkcentral/rialto

Commit: 0f21c89bd480a98ae4ddfb6f53d95a5d9342f4e6

Report detail: gist

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56393/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56397/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56485/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56486/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56488/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56554/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56555/

Job for AI/job/AI_BUILDS/job/AI-Trigger-PR-Builds started: https://pv-jen015.uk2.scp.bskyb.com/job/AI/job/AI_BUILDS/job/AI-Trigger-PR-Builds/33/

Job for AI/job/AI_BUILDS/job/AI-Trigger-PR-Builds started: https://pv-jen015.uk2.scp.bskyb.com/job/AI/job/AI_BUILDS/job/AI-Trigger-PR-Builds/19/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56785/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/56787/

Sanity tests for ['RPI4 IPSTB'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/57039/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/57047/

Sanity tests for ['BCM 7216OTT Refboard VA'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/57050/


Introduce ZOrderCoveringIndex

What is the context for this pull request?

Tracking Issue:
Parent Issue:
Dependencies:

What changes were proposed in this pull request?

TBD
TODO: add PR description / comment
Does this PR introduce any user-facing change?

TBD
How was this patch tested?

Didn't have time to review it thoroughly, but it seems it has some potential code duplication with CoveringIndex?
To me, existing CoveringIndex and this new ZOrderCoveringIndex are both CoveringIndex, with different sorting schemes. It might take more time to do, but it would be architecturally better if we first refactor CoveringIndex so that it has common covering index related stuff and can be specialized with different sorting schemes, then implementing hash bucketing/Z-ordering sorting schemes.

The change will be merged via #516 #517 #518


Update license to GPLv3+

I just heard about Sparkling and it looks really great, I'd love to start using it. However, it looks like there is a licensing problem, and I can't use it until it's been cleared up.
Currently, the code is licensed as 2-clause BSD. However, readline was added almost 2 years ago in November 2013; because readline is GPL-licensed and Sparkling links to libreadline, Sparkling also must be GPL-licensed to comply with the license.
Readline used to be GPLv2+ licensed but is GPLv3+ licensed as of version 6.0, so I've changed the license to GPLv3+ to to comply regardless of the version of readline.
There are also some other files which ideally should get license headers, but AFAIK can be released under any license (they do not need to be GPL):

All of the example code under examples/
All of the tools under tools/
The documentation under doc/ (I would suggest a documentation-specific license)

Hi!
First of all, thanks for your kind words and pointing out this issue. However, I really would like to avoid any forms of GPL whatsoever. The problem is real, however, and I don't want to get into a lawsuit. So, do you think it would be a viable alternative to switch to something non-GPL (e.g. Libedit, which has a 3-clause BSD license) instead?
I agree with adding licensing to the non-source files, though.

Yes, removing readline completely and replacing it with libedit or another similarly licensed alternative should be fine. I believe for libedit you'll need to note its licensing terms in the documentation to comply with its license.
I will admit to be a bit of a GPL fan. Looking forward to using Sparkling!

:+1: for avoiding GPL software and just using BSD-licensed software like libedit (which is excellent btw).

Okay, thanks everyone for the comments. I will completely replace libreadline with libedit soon.

Switched to libedit in https://github.com/H2CO3/Sparkling/commit/8cb40826bca64ad16a0226347a7e69d10a2bd026.


JP Damage Change

https://www.reddit.com/r/FFBraveExvius/comments/9a7grr/jp_new_damage_formula_uncovered/
Reddit has figured out the new JP damage formula.

this was in the comments.
"it would be great if you'd make a checkbox for the new formula to (not) be used."
lyrgard was tagged in the comment, but i wanted to copy it to here for visibility, and so that it didn't get lost.

Added in latest commits: https://github.com/lyrgard/ffbeEquip/commits/master


Restic won't find/restore files using --include containing square brackets in name (Windows)

Output of restic version
restic 0.12.1 compiled with go1.16.6 on windows/amd64
How did you run restic exactly?
restic restore -t C:\temp --include "/G/Restictest/Folder 2 [square brackets]"

What backend/server/service did you use to store the repository?
local file system
Expected behavior
restic restores the folder Folder 2 [square brackets]

Actual behavior
nothing is restored

Steps to reproduce the behavior
Add the files of the attachment into a repo and try to restore with the steps above.

Do you have any idea what may have caused this?
No
Do you have an idea how to solve the issue?
It works with restic restore -t C:\temp --include "/G/Restictest/Folder 2 *" latest
However it took me some minutes to figure it out that the problem is obviously caused by square brackets in file name.
Did restic help you today? Did it make you happy in any way?
Restic makes me feel happy and save every day because for me it is the best backup program on the market right now. I trust my data is safely backuped with restic. It makes me happy to improve restic.

Restictest.zip

The path matching is done as described in https://pkg.go.dev/path/filepath#Match. Unfortunately, in windows there's no way to escape square brackets.

[[]
Works as expected, meaning it will all find/restore all files with on opening square bracket. However for closing squarebracket []] it won't work since it is not recognized as character class.

Shouldn't [[]] work? [[] results in an opening square bracket. And ] on its own does not require escaping.

[[]] finds only []. So only files that have an opening square bracket followed by a closing one.

Yes, so in summary this means that you only have to escape an opening square bracket in a filename as [[] and that's it. Closing square brackets don't have to be changed if the opening square brackets are properly escaped.

Closing in favor of https://github.com/restic/restic/issues/4867 .


Question/Suggestion: Allowing to override what failure case throws in AssertJ api.

Dear AssertJ team,
So, in the end AssertJ throws AssertionError. And that is, I guess, to imitate Java assert feature. Junit does it too. Testing frameworks expect nothing less.
I really want to use AssertJ api in the prod code, as I think it is awesome and can help with readability and reusability of validation layer. One would think, just catch AssertionError and bob's your uncle. But that is a code smell, as you are not supposed to catch this Throwable in prod code.
It would be nice if we could leverage the plethora of validating code before AssertionError is thrown and then throw something else. I did a bit of a search for this and I don't think this is either possible at the moment in the code or anyone has asked for this before. I could be wrong on both accounts. In any case, what do you think?
I realise this probably belongs in Discussions, but before I move this item, I wanted to get an idea if I'm on to something here or barking up the wrong tree :)

AssertJ only throws AssertionError and there is no plan to throw anything else.
I think that AssertionError can be used production code (although it is not a prevalent usage), java actually comes with the assert construct to to verify pre and post design-by-contract conditions and invariants. If an assert expression is not met, an AssertionError is thrown.

Hi @joel-costigliola,
Thanks for the response. Makes sense. One thing I would like to make - assert failures, usually, are not caught. That means if assert fails, it is the end of the thread's execution. That is how I read it.
Anyway, your response also probably means, that AssertJ is designed to be used in test code, mainly. I would probably use AssertJ in prod code for my personal projects, but in enterprise environment an architect would object :)
Thanks for your time anyway and thanks for amazingly useable framework, that AssertJ is. Cheers!


std::map<Key, Value> serialiaze

Hello.
Maybe I didn't read the documentation carefully, but I didn't see an opportunity to serialize the following entity:
std :: map <Key, Value>;
where
struct Key {...};
struct Value {...};
Your implementation works if Key is a "simple entity" such as a string or an integer.
In general, you should provide programmers with the ability to write a function to convert a custom class / structure to and from a string. Your implementation should use this function to convert the Key element to a string and use the resulting string in a json object.

you should provide programmers with the ability to write a function to convert a custom class / structure to and from a string

That function already exists, it is a conversion operator. e.g. operator T::std::string() for your T.

I have comments:

Is this documented in your library?
What about the reverse transformation? std::string -> "Key" object?
This does not work in general. Imagine that there are structures or classes written by another person / company (for example, this is the Windows SDK) and you have no right to modify header files. For this case, external functions (not class members) are needed for converting Type->std::string ang std::string-> Type.
A couple of years ago I already had to write something similar for jsoncpp and then I used external functions.

Of course, you can make your classes inherited from foreign ones (Windows SDK) and define operator std::string () const; for a new class, but I think it's ugly.

value_to converts maps  to objects when the key is is_string_like.  If not, it is converted to array.
If your key type is not is_string_like, but you can "stringize" it and you want the map to be converted to object, then you can go for contextual conversions. The linked section has an example of exactly such conversion.
So, I'm closing this for now. If you feel like your case isn't addressed still, please reopen and explain why.


go.mod has replace directories causing "go install" to show warning

If a user wants to use Lens inside of another go project, they will need to use go install github.com/strangelove-ventures/lens@latest
This command produces the following error:
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

Is it necessary to use these replaces?
https://github.com/strangelove-ventures/lens/blob/69827eda8e577296f191cf7a49f84d22747df4ed/go.mod#L115-L117

we can remove the grpc replace tag with 0.46 of the sdk. Also I don't think this repo needs the second replace tag, right @fdymylja

we can remove the grpc replace tag with 0.46 of the sdk. Also I don't think this repo needs the second replace tag, right @fdymylja

correct. just make sure to upgrade to latest sdk version, otherwise if you use proto or grpc you might incur in panics without the replace directive.


Steps after step 3 are missing db creation

Unlike the developing-microservices walk-through, this one requires bits and pieces from various steps in order to run the previous steps solutions. The one that's obvious from the fuge shell output is the missing temperature database.
Try removing your existing docker images and running the solutions.

Merged with #23 Scripts are being converted to bat, we'll add them with this. CC @mihaidma


[Feature] Live demo can be open source?

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Live demo can be open source
Use case
No response
Related issues
no
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

The demo is a deployment of showcase, https://skywalking.apache.org/docs/skywalking-showcase/next/readme/

thanks very much!
---Original---
From: "吴晟 Wu @.>
Date: Mon, Mar 20, 2023 11:30 AM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [apache/skywalking] [Feature] Live demo can be open source?(Issue #10560)
The demo is a deployment of showcase, https://skywalking.apache.org/docs/skywalking-showcase/next/readme/
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


[$1000] Desktop -  User stucked on notification on the app icon even though there are no unread messages

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

Action Performed:

Launch the app
Log in with your account A
Log in with account B on mobile
Send a message from mobile to Desktop

Expected Result:
User able to read the message
Actual Result:
User stucked on notification on the app icon even though there are no unread messages
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
n/a
Expensify/Expensify Issue URL:
Issue reported by: @rafecolton
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692904986751459
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c2cb438a23a5f8f3
Upwork Job ID:<PHONE_NUMBER>757332480
Last Price Increase: 2023-09-27

Note that this has happened before and usually logging out and back in fixes it, but I have been stuck at one notification for weeks at least

FWIW, I've been at 99+ notifications for as long as I can remember, and I never got to the bottom of it.

I'm not sure if that is a proper reproduction, though. Taking to slack thread

Real-life reports of this here: https://github.com/Expensify/Expensify/issues/315115

Moving forward per the linked GH and slack thread

It does not appear in focus, I have tried so many times

I don't think this is same as the issue you linked, what led you to conclude that it was?

This got Help wanted recently. We're still just waiting on proposals.

Just noting that the app is now stuck at three notifications for. So it has been mysteriously increasing. Still no unread messages that I can see, so no idea how to reproduce.

Awaiting proposals

doubling!

Can someone post a demo video here?
I am unable to repro this

Issue not reproducible during KI retests. (First week)

Closing unless we can reproduce


Update frr.inc correcting OSPF & OSPFv3 rules

Corrected so that outbound multicasts that originate from self to the multicast address (224.0.0/24, ff02::5/128, ff02::6/128) are allowed.

@AdSchellevis Does this help?

@billgertz yep, that helps., thanks No problem about the confusion, things happen :)

@AdSchellevis Glad that it worked out in the end.


Add withdrawal message relayer

Description
Questions

Metadata
Fixes

Fixes # [Link to Issue]

Contributing Agreement

[ ] I have read and understood the Optimism Contributing Guide and Code of Conduct and am following those guidelines in this pull request.

This project now lives in the optimism-ts-services


[CLocal-Azure] Write unit tests cases for the project

This PR contains changes for two different tasks. Make PRs for each one of them individually.


Treecko wont evolve

No matter all the fixes I try, suicide then candy it just won't evolve but keeps leveling up.

Where did you get the treecko?

Hi moist, the next time you are online if you still have this issue feel free to contact anyone that is a moderator or higher, they will be able to force evolve it.

Hi moist, the next time you are online if you still have this issue feel free to contact anyone that is a moderator or higher, they will be able to force evolve it.

Closing as it's necessary for the player to come online. Please tag us in #support, on https://discord.gg/B3XcYcR  for quick support issues, like this one!

Closing as it's necessary for the player to come online. Please tag us in #support, on https://discord.gg/B3XcYcR  for quick support issues, like this one!


Unresolved method AutoscalingAPIGroupDSL.v1() error in Kubernetes native image generation

Quarkus master.
Camel Quarkus master.
Graal 20.1.x.
OpenJDK 11 (not labs).

Detailed message:
Error: com.oracle.graal.pointsto.constraints.UnresolvedElementException: Discovered unresolved method during parsing: io.fabric8.kubernetes.client.dsl.AutoscalingAPIGroupDSL.v1(). To diagnose the issue you can use the --allow-incomplete-classpath option. The missing method is then reported at run time when it is accessed the first time.
Trace:
        at parsing org.apache.camel.component.kubernetes.hpa.KubernetesHPAConsumer$HpaConsumerTask.run(KubernetesHPAConsumer.java:94)
Call path from entry point to org.apache.camel.component.kubernetes.hpa.KubernetesHPAConsumer$HpaConsumerTask.run():
        at org.apache.camel.component.kubernetes.hpa.KubernetesHPAConsumer$HpaConsumerTask.run(KubernetesHPAConsumer.java:93)
        at app//com.oracle.svm.core.jdk.RuntimeSupport.executeHooks(RuntimeSupport.java:125)
        at app//com.oracle.svm.core.jdk.RuntimeSupport.executeStartupHooks(RuntimeSupport.java:75)
        at app//com.oracle.svm.core.JavaMainWrapper.runCore(JavaMainWrapper.java:141)
        at app//com.oracle.svm.core.JavaMainWrapper.run(JavaMainWrapper.java:184)
        at com.oracle.svm.core.code.IsolateEnterStub.JavaMainWrapper_run_5087f5482cc9a6abc971913ece43acb471d2631b(generated:0)

Longer output.

Can you assign this to me?

Are you working on this one, @galderz ? I could have look now.

@ppalaga Sure go ahead! Let me know if you need a hand :)

https://github.com/quarkusio/quarkus/pull/9878 is one part. The second part is that, we will probably need to upgrade to kubenetes client 4.10.2 as well. This should happen in Camel Quarkus quarkus-master branch (cc @jamesnetherton ) once https://github.com/quarkusio/quarkus/pull/9878 is merged.

Thx again for helping with this @ppalaga :))


chore(requirements): update Django to 1.9.5

See https://docs.djangoproject.com/en/1.9/releases/1.9.5/

Current coverage is 83.36%

Merging #596 into master will not affect coverage as of de6d3f2

@@            master    #596   diff @@
======================================
  Files           36      36
  Stmts         2387    2387
  Branches       347     347
  Methods          0       0
======================================
  Hit           1990    1990
  Partial        116     116
  Missed         281     281

Review entire Coverage Diff as of de6d3f2

Powered by Codecov. Updated on successful CI builds.


Add BTCPayServer.Lightning.Tests

Add BTCPayServer.Lightning to Ptarmigan's test.
Perform the following test using  BTCPayServer.Lightning.
1.  BTCPayServer.Lightning Ptarmigan operation check
In order for the latest Ptarmigan to work stably on BTCPayServer.Lightning.
2. Ptarmigan, LND, clightning, Eclair interconnection test
Test interconnection of Ptarmigan, LND, clightning, Eclair using BTCPayServer.Lightning.
3. Divide Travis CI and Circle CI
Since the tests performed by each CI are largely different, We decided to divide them.
Travis CI tests Ptarmigan alone, and Circle CI tests BTCPayServer.Lightning.

Thanks!


Feature Request: Hand Segmentation

System information

MediaPipe Solution: Hands
Programming language : Python
Are you willing to contribute it: No

Describe the feature and the current behavior/state:
I would be awesome if the hands solution would feature an segmentation like recenlty implemented in the pose tracking.
Will this change the current api? How?
Probably a new parameter called enable_segmentation  for the hand model.
Who will benefit with this feature?
Everyone that wants to have the border of the hands.
Please specify the use cases for this feature:
We want to do hand size measurements and therefore need to know the border of the hand and not only the landmarks.
Any Other info:

Yes that would be very useful.

@flobaader,
Pose Landmarker for MP Tasks contains segmentation mask.

This would indeed be useful.

Yeah, a Hand Segmentation would be very much appreciated.


Fix settings of audio recording not to interfere with others

@kanekin now to the right branch... :)
Fix for the noise sensor, should be QAed on another phone as well I think so would be good if you can run it as well on dogeprober or so while testing visit location coach.
QA and merge please :+1:

There is a short pause every time the audio recording starts.
When I use master branch, the pause does not happen except for the first time.

Confirmed that the pause is fixed.


Which is the true license?

http://dinbror.dk/blog/blazy/#license

bLazy is licensed under the DWIWYWBPVMS license. (Do With It What You Want But Please Visit My Sponsor).

https://github.com/dinbror/blazy/tree/fae860857d58764d84222ef981305f0ea9c90b24#license

MIT

Both :)

This project dual-licensed, right? And I recommend not using DWIWYWBPVMS license since it's a non-SPDX license. https://docs.npmjs.com/files/package.json#license


Fork on strict-pipe

Added fork on strict-pipe
Not tested, but follows similar logic to fork on linear-pipe

@nholland94 and I discussed a figured out a mental model for dealing with pipes. If you think in CSP style, you should be fine if you only use iter_without_pushback on the underlying pipes.
This is reflected in this PR, and the implementation of fork should still be correct in that universe


Add self generated certificate as SSL provider

I am trying to expose some HTTPS services to LAN only.
But when adding a proxy, Nginx Proxy Manager only offer Letsencrypt  or None as provider.
Is it possible to add a self generated certificate as SSL provider ?
Thank,

When switching to the "SSL Certificates" tab and clicking on the "Add SSL Certificate" button in the upper right corner of the card (Not the big one in the center) you can choose to upload a custom certificate.
This certificate can then be selected for the host.

However it seems since version 2.5.0 the custom certificate upload is broken. See https://github.com/jc21/nginx-proxy-manager/issues/594

@chaptergy: I'm unsure if the author meant a third option besides LE an Custom (before step three) like "Own CA". OpenSSL could generate a quick self signed certificate with the given name. I think with this scenario we could even leave out the whole renewal management, just let it generate a new cert eg. after year. Only give the CA certificate to download that it could be added to clients.
Besides that it's a little bit confusing if two buttons with the same name do different things ("Add SSL Certificate").

It was a conscious decision to make the main path for SSL certificate generation use Let's Encrypt because that was the goal of this project. Custom SSL certificates were added by request later in the game.
That said I agree it would be pretty easy to integrate mkcert into the project to generate self-signed certs on the fly and then make it part of the Host creation as an option.

Duplicate of https://github.com/jc21/nginx-proxy-manager/issues/593


Nothing happens

When I select Canvas2document (either step) from the command prompt, nothing happens.

You have loaded a canvas file and there is no notification (in the top right corner of Obsidian)?
If so, could you please send me a screenshot of obsidian's developer console (to open with Ctrl-Shift-I) after the problem occured?
Thank you!

Im getting the same, though it did work with the first canvas i loaded. The last two though nothing happens -
Says this in the developer log issues (see attached

)
plugin:canvas2document:344 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '3')
at Canvas2DocumentPlugin.writeCanvDocFile (plugin:canvas2document:344:34)
at Object.callback (plugin:canvas2document:59:35)

This look a lot like an error from 1.2.1. Do you have the latest version 1.2.2?

This look a lot like an error from 1.2.1. Do you have the latest version 1.2.2?

I do - recently installed
Just did another test with a different canvas and that worked fine - so I think maybe the error with the others, was that they contain embedded canvas' which is fine to workaround if that causes issue with transfer to document

Ok, I will check that. Embedded canvas is something that should get supported in one of the next releases, anyway.

confirmed, that crashes on embedded canvas under some circumstances, needs a fix release

released with 1.2.3

Great work. Thanks @sinsys

released with 1.2.3

thank you! ID is @slnsys :-)


Need to have wake words that are longer - 3 terms in a row

Longer awake work doesn't work?
no detection of 3 words in a row...
Is there any way to detect 3 words in a row? please? can the model be expended a bit?

You should be able to use three words as long as it is not too long. For example "a lazy cat" would work. What are the words you are trying to use?

for example: "OK OPEN WINDOW". I get it it's too long currently. I guess my question is - is it technical challenge to make the detection work on longer phrases or it's something that can be developed?

Yes, that is too long for the current version of optimizer available here. Although you can use "open window". I tried it and it works. Would that work for your use case?
Having such long wake-words is not common. But if more developers start asking for it we will consider adding it to our next release.

Thanks, guys for the feedback. We will add this to our feature list for v1.3 release.

@abramovi  and @davidgur111
We've got lots of request for wake-words that are longer than what we can do right now. We decided we would address this need by building an ASR on-device. Just giving you guys a heads-up.

What do you mean by ASR on device? the code will be able to run as it is
right now with the new logic?
On Sun, Jun 10, 2018 at 7:39 PM, Alireza Kenarsari<EMAIL_ADDRESS>
wrote:

@abramovi https://github.com/abramovi and @davidgur111
https://github.com/davidgur111
We've got lots of request for wake-words that are longer than what we can
do right now. We decided we would address this need by building an ASR
on-device. Just giving you guys a heads-up.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/Porcupine/issues/18#issuecomment-396062894,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlbWzhuUf_RYiTcJSebblhsU5NOY3PdJks5t7Uu8gaJpZM4T86g5
.

Is it still planned for v1.3

ASR: Automatic speech recognition. It will be a different repo/product.
We are going to try to expand the wake-word engine to be able to do longer wake-words. That being said there will be limits. As mentioned wake-word engines are not designed for sentence-like command detection. So if you need to detect "ok robot open the door" this cannot be done with wake-word. Therefore we are building a separate speech recognition engine for it. makes sense?

closing due to inactivity for more than 30 days.
comment: currently it is possible to create wake phrases with short words. If they get too long then it is not. Detection of multiple long words can be done using a speech recognition engine as wake-word engines are not suited for this task.


The receive() function cannot receive a message until the seek() function is called

I find that if I don't call seek(messageId: MessageId): Unit or consumer.seekEarliest() , consumption,consumer.receive() will block.
My scala tests are as follows:
val client = PulsarClient("pulsar://localhost:6650")
val topic = Topic("persistent://sample/standalone/ns1/test_" + UUID.randomUUID)

val producer = client.producer(ProducerConfig(topic))
val messageId = producer.send("wibble")
producer.send("wobble")
producer.send("wubble")
producer.close()

var consumer = client.consumer(ConsumerConfig(topics = Seq(topic), subscriptionName = Subscription.generate,
  subscriptionType = Some(SubscriptionType.Exclusive)
))
consumer.seek(messageId.get)
var msg = consumer.receive
new String(msg.get.data) shouldBe "wibble"
consumer.acknowledge(msg.get)
consumer.close()

consumer = client.consumer(ConsumerConfig(topics = Seq(topic), subscriptionName = Subscription.generate,
  subscriptionType = Some(SubscriptionType.Exclusive)
))
consumer.seek(messageId.get)
msg = consumer.receive
new String(msg.get.data) shouldBe "wibble"
consumer.close()
client.close()

Is this the right way to do it ?

In those examples, the messages are sent first, so you need to seek earliest in order to go back to before the messages were sent.

If you seek to a message id, the next message to be received will be the next message after that message on the topic. So this behavior is not a bug.


kbuild should put definitions in a local folder by default

Currently artifacts built by kbuild are placed in a folder structure like this:
~/.kbuild
├── hoon-semantics
│   └── d2f5ce0
│       ├── include
│       │   └── hoon-semantics
│       └── target
│           └── v5.6.110-0-gf2f25ce798
│               ├── haskell
│               └── llvm
...

The idea is that the ~/.kbuild folder serves as a user-local repository for kompiled definitions, arranged by source of installation (d2f5ce0) and K version (v5.6.110-0-gf2f25ce798).
However, for semantics projects, this seems to be more of an inconvenience than a feature. There, .kbuild is only used to produce the definitions but they are distributed in some other way. But due to the complex folder structure, the artiacts can only be accessed through kbuild which.
So by default kbuild just should output targets in a local folder, e.g.:
./kdist
└── hoon-semantics
    ├── include
    │   └── hoon-semantics
    ├── haskell
    └── llvm

Maybe the repository thing can even be dropped altogether.

@geo2a, @virgil-serbanuta, what's your opinion?

I personally am always in favor of directory local builds, and only if the user does something like kbuild install ... does it go to the user-level or system-level.

I also like local builds.
I think that it's a good idea to somehow rebuild the semantics when the k version changes, so I like having separate builds for each k version (although this is not the only way to solve this issue - writing the version in a file and checking it would also work).