How do you use this without setuptools?

Basically, all the documentation assumes you're writing a stand-alone CLI application.
In my application, I have a python module with a optional CLI interface that exists basically for installation diagnostic purposes. As such, it really, really shouldn't be installed as a stand alone executable.
I assume somewhere is a function that I can put in my if __name__ == '__main__': that will call the appropriate decorated function, but apparently it's not documented.
There are a lot of use cases for non-installed cli tools.

That's more of a general Python question than click-specific...
If you add a __main__.py in your package you can run it as python -m yourpackage
Anyway, IRC or Stack Overflow are better places for this kind of support question.

What?
My issue is how do I invoke the click dispatcher from my code, not how do I add a __main__.
This is the issue tracker. It is exactly the correct location for this kind of question.

Oh, and if you don't want to use setuptools: python path/to/whatever.py

I have all that.

No, the issue tracker is for bugs and feature requests. Not for usage questions.

The issue I have is the documentation is missing details.
Basically, I have a file with multiple functions decorated with @cli.command('open').
What part of click do I call so it dispatches to the relevant function?

 It's completely unsearchable, and is therefore useless for answering the same question for anyone else.
Stack Exchange is somewhat more appropriate, if far far too general for this sort of thing.

That's your opinion. Usually IRC works very well for quick questions, especially if they need clarification.
For example, it is not clear how you intend to run your CLI. If it's just about running a python file to launch the CLI, then the very first page of the documentation shows you how to do that.

If the documentation had a useful example for the quickstart, it'd answer all my questions.
Basically, the examples are either so simple they're useless, or so baroque and overcomplicated they're impenetrable.
A better quickstart that shows probably 98% of what the average person needs

import click

@click.group()
def cli():
	'''
	CLI Documentation goes here

	calls:
		script.py func_1
		script.py func_2 [dest] [-r | --rev <value>]

	'''

@cli.command()
def func_1():
	'''
	Function 1
	'''
	print("Command 1")

@cli.command()
@click.argument('dest', required=False)
@click.option('--rev', '-r', default='HEAD', help='Clone a specific revision instead of HEAD.')
def func_2(dest, rev):
	'''
	Function 2
	'''
	print("Command 2: %s" % ((dest, rev), ))

if __name__ == '__main__':
	cli()

That example shows how to put a single CLI option in a file.
I want to have multiple commands.

And the docs explain that you use a click group like a normal command when it comes to invoking it.


WIP: Right click option POC

Here is an example implementation to solve #191

[ ] Add support for middle click
[ ] Add some default right click actions

[ ] Wifi menu on right click
[ ] Spotify next track
[ ] Remap space hover action to right click

[ ] Add support for user widgets

Hi, sorry I involuntary closed this as I removed the "add-user-widgets-creator" branch.
I'll implement your changes soon but I am not so sure about the middle click thing as a lot of people are working only with their mac book touchpad. What do you think?

I think it wouldn't hurt, I actually already implemented it here: https://github.com/rosenpin/simple-bar/blob/feature/right-click-action/lib/components/data/data-widget.jsx


Update TableReader interface to return (*Schema,error)?

It might make sense to update the interface to return the error too?
https://github.com/polarsignals/frostdb/blob/b2cc5b02f387b5a9a8da72cb20e01deaf13c3d3e/query/logicalplan/logicalplan.go#L133
Or should this error really just be handled outside the interface? Basically opening that as a question to everyone else as I'm not 100% sure right now.

This schema should be passed as configuration when instantiating the table, so it should just be a pointer in the table that's just always there, I don't think there is any way currently for this to error.

I currently have a way for this to error :P so this panic is currently earmarked in what I'm working on.

Did we happen to update this in one of the PRs? Just making sure we close it if we don't need it anymore.

I don’t think we’ve done this. I think what Thor referenced we managed to get to work without this.

Yea I ended up not needing to make this change, so I'm just going to close this issue for now


Monitoring REST + CLI

A very simple CLI should be able to connect to the current scheduling "daemon" and dump current state (resource and allocations, etc.). This ticket only requires a minimal output alike a proof of concept.
We can later extend this as we add more features. This CLI could be a great tool for integration testing

Initial implementation could use curl


Beta23 metallurgy

something broken in 23 or is it just with me?
after updating from beta22 to beta23 no longer able to proceed in alchemical metallurgy.
the recipe page for brass ingots remains blank ??
also some of the added sections doesnt seem to complete .. got the btn complete, but only displays "completing...".

rig has i7,gtx1070, plenty mem. using twitch, no extra mods.
what i tend to do after new update is start a new world, gather all vanilla stuff, only after collecting all vanilla items i mine crystals and go tru every Thaumonomicon entry.

going back to beta22 allows me to proceed in alchemy.
removing modpack and reinstalling didnt resolve either.

Cannot reproduce.
Made a new world in a pack with only a few clientside mods next to Thaumcraft 6 Beta 23. Everything worked as it should, and I was able to progress through Alchemical Metallurgy just fine. Recipe for brass ingots showed up normally.
Did you start the affected world in Beta 22 or Beta 23? Perhaps it only affects existing worlds that are upgraded, because new worlds in B23 are definitely fine. Or, something could be wrong on your end - that's possible too.

Drat. Tks for response.
Went tru update process twitch again, rushed towards Alchemical Metallurgy.
Same problem, so i can reproduce, on my rig, Will dust off old computer and see if i can reproduce again same account and such. (hope it still boots, will take some time)

++Attempting to make brass ingots in b23 similar to b22.
Lava beneath Crucible with water, toss flint, iron ingot, flint, iron ingot, bunch of crystal nuggets … iron ingots are gone, no brass, Inventory does show 5 instrumentum, but no 5 terra.
Normally also the instrumentum is gone also and you get the brass.
Mods conflicting, incomplete install ??

Which other mods have you installed? Maybe Thaumic Periphery? If so, do you have "Copper Brass Recipe" turned on in the config?

interesting ... yes, those mods came with default twitch/modpack update.
Me just a casual player, configuring mods usually i skip.
( where do i find the config ? )


PlanQK QUBO Solver

see #63 --> needs to be refactored to new backend.
Never made id to live instance.


Fix support for new dubs

Fix English/European Spanish dub downloads and add support for Hindi. European Spanish and Latin American Spanish still use the same download code.

Fixes most of #342


✅  Add a11y test for Tooltip

Resolves #2052

I suppose the tooltip doesnt actually exist until you fire mouseover event on the span?


Issue #488 Add docs_serve target to makefile

Fixes #488

This is very slow, takes around a minute for the container to get up and start serving... :(

This is very slow, takes around a minute for the container to get up and start serving... :(

Ah :( Initially, I wanted to ask why we were starting a container for that, but then I realized I don't have that 'http' python module installed by default on my machine ;) But if starting the container is too slow, maybe checking if the right module is available, and running this directly is a better solution?

But if starting the container is too slow, maybe checking if the right module is available

I think this module is part of python std library @praveenkumar?
So the container is using python3, for python2 you need to use python -m SimpleHTTPServer both are equivalent.

@anjannath where is the dockerfile for this serving container?

I think this module is part of python std library @praveenkumar?

yes it should be part of python std library but having container help if I want to test the docs on Mac or Windows where default python installation not available.

But if starting the container is too slow, maybe checking if the right module is available

I think this module is part of python std library @praveenkumar?
So the container is using python3, for python2 you need to use python -m SimpleHTTPServer both are equivalent.

Yup, this was my confusion, I tried python -m http.server 8088 which is still python2 on my machine. python3 -m htpp.server 8088 works as expected.

yes it should be part of python std library but having container help if I want to test the docs on Mac or Windows where default python installation not available.

You can probably directly open the html files in your webbrowser for this kind of testing?

@praveenkumar The dockerfile look as mentioned in https://github.com/code-ready/crc/issues/488#issuecomment-525650590

You can probably directly open the html files in your webbrowser for this kind of testing?

True, but having a command you can directly run and have the URL to copy paste is convenient, then going to browser, browse to the dir and opening the file :)

I'm personally fine with this change, feel free to send an updated version if the concerns which were raised need addressing.

(and feel free to merge this if you think it's good enough for now ;)


Underscores in Sentences

While reviewing Jingwei's example on GlassBR, I encountered issues with underscores being used in Sentences. GlassBR has variables named is_safe1 and is_safe2. When used in Sentences, Latex renders them in text mode and as a result it doesn't render properly, like this:

Escaping the underscores could fix this, but I think a better solution would be for Drasil to make Latex include the underscore package, if possible.

Can you point me to the exact point where this happens?

Sure, for example, this piece of code for the Table of Symbols:

comes out like this:

Also, in descriptions of Theoretical Models:

it causes this:

This is two separate issues. First off, the symbol for "is_safe" should have an underscore representation, not an underscore (so this still needs to be implemented, it's going to work similar to things like "Tau" or "alpha").
As for the theoretical model, that "is_safe" should be pulled from the chunk, not written directly, and thus will be fine once the symbol is fixed. All of this is also assuming the context printing is working appropriately.


Type information lost after Pick on object with [key: string]: any;

TypeScript Version:  2.8.3

Search Terms:  Pick
Code
type X = {
    [key: string]: any;
    x: string,
    y: string
}

type Y = Pick<X, keyof X>

Expected behavior: Y to be same as X
Actual behavior: Y = {[key: string]: any;}
Playground Link:
https://www.typescriptlang.org/play/#src=type X %3D {
    [key%3A string]%3A any%3B
    x%3A string%2C
    y%3A string
}

type Y %3D Pick<X%2C keyof X>
Related Issues:
My use case is to make a required field optional in type X
type X = {
    [key: string]: any;
    x: string,
    y: string
    ....another 20 params
}

type Omit<T, K> = Pick<T, Exclude<keyof T, K>>; // this works for type without index type
type Y = Omit<X, "x"> & { x?: string}
// expect: x becomes optional and rest remain the same
// actual: y is removed

@Bnaya yes your solution works for my example. But this issue is causing a problem on Omit. I updated description with my original use case

Got it.
I've created this example to make it more clear
type X = {
    [key: string]: any;
    x: string;
    y: string;
}

type X3 = {
    x: string;
    y: string;
}

type Omit<T, TR extends keyof T> = {
    [K in Exclude<keyof T, TR>]: T[K];
};

/**
     Y2 = {
        [x: string]: any;
    }
 */
type Y2 = Omit<X, "x">;

/**
    type Y3 = { y: string; }
 */
type Y3 = Omit<X3, "x">;

https://www.typescriptlang.org/play/#src=type X %3D {
    [key%3A string]%3A any%3B
    x%3A string%3B
    y%3A string%3B
}

type X3 %3D {
    x%3A string%3B
    y%3A string%3B
}

type Omit<T%2C TR extends keyof T> %3D {
    [K in Exclude<keyof T%2C TR>]%3A T[K]%3B
}%3B

%2F**
     Y2 %3D {
        [x%3A string]%3A any%3B
    }
 *%2F
type Y2 %3D Omit<X%2C "x">%3B

%2F**
    type Y3 %3D { y%3A string%3B }
 *%2F
type Y3 %3D Omit<X3%2C "x">%3B

The type has an index signature that allows it to have any string named property.. there are no way int eh language to define exclusion from infinite ranges, e.g. string - "x".. that said, not sure what you expect from Pick rely..

keyof X is string| number. string subsumes all the other property names, "x"  and "y".

The compiler will try to keep the properties as long as it can, e.g. Partial<X> does have x and y. the issue is once you get keyof, which is a union, the string subsumes all the other property names, and you end up with only string.
As i noted earlier, it does not make much sense to use operations like Pick, Exclude and Omit on a type like that. if you want to specify the types of properties other than x/y with the index signature, i think you are looking for https://github.com/Microsoft/TypeScript/issues/17867 instead.

will it make sense for keyof to return like string|"x"|"y" so the information will be kept?
my use case is to make one required field optional, which is common in React component with higher order components. The only solutions I can find is to use Extract which doesn't work here.


docs: markdown link fixes

Issue being fixed or feature implemented
I discovered a number of linking issues in our markdown files that broke navigation.
What was done?
Fixed the links.
How Has This Been Tested?

Clicked links
Breaking Changes

N/A
Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

There are currently no conflicts between this and v0.22-dev branch either so merging it quickly would be ideal so it can also be merged back in there.


Pick Your Poison incompatibility

Mod Name
Pick Your Poison
Mod Homepage
https://modrinth.com/mod/pick-your-poison
Minecraft version
1.20.1 (LTS)
Describe the bug
Crash on start
Steps to reproduce

...
Logs
https://pastebin.com/ryiYXtjV
Additional context
No response

As our LTS support target, 1.20.1 now only receives critical fixes related to Connector bugs. Please upgrade to the latest version for mod compatibility support.


Unskip unit tests that work on ROCm 5.6

As part of SWDEV-392298 unskipping the test that works on ROCm 5.6 and xfail for older versions.

rocm_5.6_test2.log.txt
rocm_5.4_test2.log.txt


Add cairo interaction

Closes #2.
cc @sdroege

Looks good to me. @GuillaumeGomez ?
This is now in sync with gdk3?

Looks good to me as well. Thanks @sfanxiang !


[FEATURE REQUEST] Status of internet connection

hi, is it possible to retrieve the status of the internet connection?
I would like to be able to start specific automations in the event of an internet outage (presence detection, for example, because no one connects via wifi and therefore the heating management is switch off, although the heating management works locally, so when I am there I can force locally a boolean input to simulate a presence, but when I'm not there... my wife is not happy :-) )
Thanks in advance
Best Regards.

@geobar78 It seems to be binary_sensor.miwifi_00_00_00_00_00_00_wan_state and displays the connection, doesn't it?

hooo thank you very much for your return, I'm ashamed to have been so blind, I'm going to hide in a very small hole.


Show file hierarchy on the work show page

Why was this change made? 🤔
Fixes #2943 - show file hierarchy on the work show page, as mocked up here: https://projects.invisionapp.com/share/YQ132HXFX2GR#/screens/471005875
Borrowing from the Argo display: https://github.com/sul-dlss/argo/pull/3913/files
Questions:

what does the empty vs filled in folder icon mean in the mockups? note: we will never have a folder with no files in it in H2 (and how to render an empty folder?)

How was this change tested? 🤨

Localhost
Existing tests
Updated/added tests

@astridu and @amyehodge ... this is an example of what the file hierarchy display looks like in the work show/detail page with this PR:

(the repeated filenames are expected, I just made copies of the files and put into folders for testing)
I can deploy to stage or qa if you want to see it live (screenshots are from my laptop).
One note...the mockup (https://projects.invisionapp.com/share/YQ132HXFX2GR#/screens/471005875) shows some empty folders with a different folder icon (white instead of black).  We won't ever had blank folders in our data model, so the current view only ever shows black folder icons (representing folders that either have sub-folders and/or files in them).

Just deployed to QA.  Best way to see it in action (if you don't already have an object with hierarchy to view) is to create a ZIP file with nested hierarchy and upload to a new draft object.  You don't need to deposit.

@peetucket yes, sorry. it's at https://sul-h2-qa.stanford.edu/works/351

yep @peetucket looks right to me now as well


VNET peering docs: overlapping IP CIDR blocks not allowed

This is a general Azure networking gotcha, but worth adding to docs: if you're building a cluster in a VNET that is peered with other VNETs, ensure that all VNET IP address space CIDR blocks are distinct, and have no overlapping IP ranges.

Azure portal starts giving the warning for overlapping subnets even before vnet creation or doing actual peering.
Example: We already have a k8s cluster with vnet address space <IP_ADDRESS>/8. If I just try to create a vnet using portal, it starts giving the warning for overlap.

The following screenshot shows vnet of our dev k8s cluster showing warning of overlap from qa k8s cluster even though they are not peered. I think its harmless and its a problem in azure portal.

The accepted answer at https://stackoverflow.com/questions/45836574/the-address-space-10-0-0-0-16-overlaps-with-10-0-0-0-16-in-virtual-network also says the same thing.

I have the same scenario, I have overlapped CIDR from two VNets which are in two different subscriptions. Now I cannot peer this VNET's..


Auto update widths of inputs on type

:nut_and_bolt: Description: What code changed, and why?
By default, the number inputs on data-lifecycle were only large enough to show two digits.  We will now auto update the width of the input upon typing so that the entire number is always visible.
:chains: Related Resources
Fixes: https://github.com/chef/automate/issues/4040
:+1: Definition of Done
Data Lifecycle input values are always visible to the user, regardless of how long.
:athletic_shoe: How to Build and Test the Change
build components/automate-ui-devproxy && start_all_services
make serve
navigate to https://a2-dev.test/settings/data-lifecycle
In any/all of the type-able number inputs, type as many valid numbers as you wish and view the input expand/shrink as necessary.
:white_check_mark: Checklist

[x] I have read the CONTRIBUTING document.
[ ] Tests added/updated?
[ ] Docs added/updated?
[x] All commits have been signed-off for the Developer Certification of Origin.

:camera: Screenshots, if applicable

added a small update to run after the for is initially populated.  Noticed that when a large number is saved in the database, the input width was too small.

Not to self - onChange wasn't totally covering what was necessary.  Opened to card to update when possible: https://github.com/chef/automate/issues/4175
Making a directive as Scott mentioned above could be great too.


Combine multiple IMAS db entries into single one [WIP]

This PR implements the feature to combine different IDS data into a single IMAS DB entry, setting the group error to the _error_upper / _error_lower keys.
Addresses #156

I'm pretty happy with where this is at currently. There is a bug where the standard deviation cannot be stored to *_error_upper. But since this PR is already getting quite long I will merge it as it is in a pretty good (working) state now state now and solve the remaining issues in a new branch.


Add 1.39.0 to wrapped lang interop matrix builds

Following the 1.39 release
The new docker images have been uploaded.

This is actually blocked on C# package uploads (b/195992135)
I'll wait on this to see if we can resolve that in the next couple of days.

This is actually blocked on C# package uploads (b/195992135)
I'll wait on this to see if we can resolve that in the next couple of days.

After resolving conflicts, this should be good to go (for C# I uploaded images for 1.39.1,  not for 1.39.0).

basic tests C# windows: https://github.com/grpc/grpc/issues/26609


Add paint to get the current value of the varbit for the selected quest

"Varbit: " + whatever the varbit is

Added this in latest commit or two


ThreatIntelligenceImport fix

Required items, please complete
Change(s):

Update to /V2/ThreatIntelligence api
fix on how to set workspace from the ARM template.

Reason for Change(s):

Issue with current version

Testing Completed:

Tested standalone installation and mainTemplate and template installation.

Checked that the validations are passing and have addressed any issues that are present:

Yes

@v-prasadboke can you handle this PR quickly? It's a fix to one of our main Playbook.

Hello @RecordedFutureOskbo, Thanks for raising this PR. I'll investigate this PR and try to close this earliest.
You can expect an update till 31 October, 2023.

Hello @RecordedFutureOskbo, Thanks for raising this PR. I'll investigate this PR and try to close this earliest. You can expect an update till 31 October, 2023.

It's a fix to one playbook, storing ThreatIntelligenceIndicators using the new upload API. Updating to the V2 API end-point and using WorkspaceID instead of retrieving the WorkspaceID from the WorkspaceName. The Name to ID ARM conversion was case-sensitive and did not work as expected.  We have customers wanting to upgrade running in to problems.

Can we connect for a minute

Can we connect for a minute

Did I miss the opportunity? I'm available on several platforms and teams


rpcserver: Add proveutxochaintipinclusion command

New rpc command proveutxochaintipinclusion is added.  It uses the
underlying utreexo proof index's ProveUtxos() method and return the
proof in json.
Currently there is no support for verifying the proof returned by the
rpc call.  It will be added in a future commit.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 110 (0.91%)  changed or added relevant lines in 3 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 54.59%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

btcjson/chainsvrcmds.go
1
7
14.29%

server.go
0
18
0.0%

rpcserver.go
0
85
0.0%

Files with Coverage Reduction
New Missed Lines
%

addrmgr/network.go
4
73.58%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
26137

Relevant Lines:
47879

💛  - Coveralls


Please add a license file and a screen shot

I'd like to add this to awesome-zsh-plugins, but I prefer to add themes with screen shots in the readme so that users can see what they look like without having to install them.
Also, please add a license file, some people won't use any code that doesn't have an open source license. If you don't have one in mind, https://choosealicense.com is a good resource to help you select an appropriate one.
Thanks!

I added a license file and updated the README with additional information as well as a screenshot.

One last thing - could you rename it to something less generic than zsh-theme? Sorry, I should have included that when I created the issue.


Update docs with long review date for Phoenix

What does this PR do?
Updates review dates/fixes for the outdated docs

mostly updating dates as docs are still fine imo


Why is customer in shared apps?

Hi so I've set up a django app according to the tutorial with multiple tenants with their own schemas. Why do I need the customer app in shared_apps? I've moved it to tenant_apps and nothing changed. Is it so that I as the superuser can view all my tenants and when I create users for the tenants I make it so that they can't view the client app?

You moved Client model , but you didn't removed this table from DB :) Try dropping DB, and start again, it won't work. It's not that you need customer app in shared_apps, you need TenantMixin model in shared app. Please familiarize with schema aproach and this article: https://web.archive.org/web/20160311212239/https://msdn.microsoft.com/en-us/library/aa479086.aspx
You want be able to access your tenants not knowing about them. And you won't know if you won't have TenantMixin in your public schema.


tolerance for Krylov solvers

Experimenting with the new skfem.utils.solver_iter_krylov using scipy.sparse.linalg.gmres with skfem.utils.build_pc_ilu for a three-dimensional version of ex25 ('Forced convection'), it's going quite well (up to a basis.N of 911 752 so far, solving in a few minutes).
But I found that the optional keyword argument tol to solver_iter_krylov didn't seem to be doing anything.  It turns out that this tol is passed in to the solver (here gmres) as atol and the gmres (relative) tol left as the default and this was dominating.
https://github.com/scipy/scipy/blob/0f5d8aab5d9990c433168ad3424b3f5c9440978e/scipy/sparse/linalg/isolve/lgmres.py#L168
Perhaps it should be possible to pass in tol as well as atol?
Maybe solver_iter_krylov should just pass on *args, **kwargs to the solver?  That would be simpler, if a little less explicit.
It might also facilitate using solvers from outside scipy.sparse.linalg that take different arguments (presumably though always taking the linear operator and right-hand side as first two positional arguments, and maybe the initial guess as the third?).

Maybe solver_iter_krylov should just pass on *args, **kwargs to the solver? That would be simpler, if a little less explicit.

I think this is fine.


Mekanism Power Distribution Bug

Hi,
I am having an issue using mekanism cables for my power infrastructure in Minecraft. It seems when you go away from your house a bit, teleport, go to sleep… power generation seems to stop and the cables do not transfer power properly.
I do not have an accurate cause for this, but it has affected players on my Minecraft Server.
In the pictures attached you will see:

The Windmills are charged and are not drained. Very high elevation, not prone to loss of wind speed. There is sufficient power in the grid to power my items at ~414.55 J/t or ~165.82RF/T per windmill. (2 Windmills running)
The cables indicate they have a charge and yet visually as you will notice there is no indication of that with the green glow.
The items that are powered in my house do not get replenished with power as the cables should indicate.

I do not know if this is a bug with the modpack I am using in combination with other mods or the mod itself. Would it be possible for someone on the dev team to help me debug it?
Minecraft Info:
Version: 1.12.1
Modpack: Beyond the New Frontier V4.8
Mekanism Mods:
Mekanism-1.12.1-<IP_ADDRESS>6
MekanismGenerators-1.12.1-<IP_ADDRESS>6
MekanismTools-1.12.1-<IP_ADDRESS>6

I apologize if my first post did not satisfy posting requirements for easy reading #4686 . That was indeed my fault.

In reply to whether I searched for similar issues... I did but the answer given was unclear. There was no indication of a resolution, rather a vague hint in #4676. Last Reply: "Your version is far too out of date and probably doesn't even include the initial half of the fix to this type of problem. The latest is <IP_ADDRESS>6 for 1.12". That does not stipulate that is is being looked into or resolved. Rather it leaves an impression that it is resolved via upgrading mekanism, which is not my issue. Now I am not trying to be an assh***. I realize people have lives away from virtual wonderland and I am not looking for a quick fix. I would appreciate it when you have the time to help me look into this. Please and Thank You.

You could have edited your issue rather than starting a new one.
My comment about a version update was not in  response to the op, which is why it is still open.
The original issue post is exactly the issue you are having. Earlier in the thread Tom stated "apparently it's not fixed completely..." which he has since tested and confirmed.
Please subscribe to that thread using the button in the sidebar.


There is no contact method provided in the Code of Conduct

The Code of Conduct does not include any description of who to contact and how to contact them, in case of CoC violations.
See https://github.com/executablebooks/.github/blob/master/CODE_OF_CONDUCT.md#enforcement

Instances of abusive, harassing, or otherwise unacceptable behavior may be reported to the community leaders responsible for enforcement at [INSERT CONTACT METHOD]. All complaints will be reviewed and investigated promptly and fairly.

Ah cheers you beat me to it: @ebp-bot now has the<EMAIL_ADDRESS>email address where these should be directed

closed in 9fbb9f8


Add missing error check in flatten example/stackAlgorithm

Stack Algorithm has a case where it errors but does not return a nullptr.  In the flatten example, it was only looking at whether the result was null or not, so it could create other bugs.
Additionally, the flatten example would write out the input timeline instead of the flattened one.

Codecov Report

Merging #1112 (c973720) into master (31b0e8d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1112   +/-   ##
=======================================
  Coverage   86.20%   86.21%
=======================================
  Files         191      191
  Lines       19028    19030    +2
  Branches     2105     2105
=======================================
+ Hits        16404    16406    +2
  Misses       2078     2078
  Partials      546      546

Flag
Coverage Δ

unittests
86.21% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/opentimelineio/stackAlgorithm.cpp
79.59% <100.00%> (+0.86%)
:arrow_up:

Continue to review full report at Codecov.

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


Callback of Model::handleMissingAttributeViolationUsing called when using isset to check if attribute exists

Laravel Version: 9.38.0
PHP Version: 8.1.3

Description:
When a custom callback is defined to handle missing attribute violations (introduced in https://github.com/laravel/framework/pull/44664), then that callback is also called when using the isset function to check if an attribute exists. That is not correct, because it is not a violation when using isset
Steps To Reproduce:
Model::shouldBeStrict(true);

Model::handleMissingAttributeViolationUsing(function ($model, $key) {
    dump('this should not be called!');
});

$user = User::factory()->create();

if (isset($user->fresh()->foobar)) {
  // ...
}

We have no plans anymore to change this behavior, please see the convo at https://github.com/laravel/framework/pull/44717

The discussion in that PR seems to be around something else. This issue is caused by the fact that the offsetExists method tries to catch an exception that will never be thrown. The catch makes sense when no missingAttributeViolationCallback is defined, but when there is callback defined the catch does nothing.
https://github.com/laravel/framework/blob/0de8734170de48926e5a65c44e898b7078a0da0e/src/Illuminate/Database/Eloquent/Model.php#L2239-L2246
https://github.com/laravel/framework/blob/0de8734170de48926e5a65c44e898b7078a0da0e/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php#L465-L478

Got bit by the same bug, see https://github.com/laravel/framework/pull/44642#issuecomment-1344964691. Was a really nice feature, but unusable unless you directly access the attribute.


Add nativeSearch in SearchQuery

Fixes #453 .
Changes proposed in this PR:

SearchQuery typing update
update MetadataCache and Assets query's to use new nativeSearch

thanks for going through this! This is great so far, and covers use cases where people use the nativeSearch. But what about everybody using the "old way" of searching? With this change we break it for them, although the functionality is still there.
To me the proper solution has always been to stick to only one way of searching instead of adding 2 different ways.
So we either:

figure typings allowing old & new way of search queries
remove old way from the core

For 2. this PR is then already perfect

core meaning Aquarius, cause it has 2 ways of searching:

https://docs.oceanprotocol.com/references/aquarius/#assetsddoesquery
https://docs.oceanprotocol.com/references/aquarius/#assetsddoquery

This is of course confusing but also not sure how much work it would be to remove the old way of searching in Aquarius. Could be fine deprecating the old way starting in ocean.js, and default to nativeSearch.

I think we should deprecate the old way, we don't plan to maintain the es drivers and whatever is needed

I agree.  we will only use ES with Aqua from now on

ok, cool. Which would then make a key like nativeSearch redundant if we update Aquarius.
So we would end up in here with only:
query: {
  query_string: {
    [property: string]: string | number | string[] | number[]
  }
}

On the other hand, does Aquarius support all full text queries from here, or just query_string? https://www.elastic.co/guide/en/elasticsearch/reference/current/full-text-queries.html
If it supports all of them then in this PR we might want to find some external package which has typings for Elasticsearch queries and rather use that

Since it's a pass-through, they should all work

So we could wait for https://github.com/oceanprotocol/aquarius/issues/372 to be able to finalize this here.
Or we just set nativeSearch in here by default when hitting /query, so users of library do not have to pass that. Then ocean.js would already deprecate all other ways, and once Aquarius is updated we remove passing that nativeSearch key

So I don't know exactly how can we use elasticsearchjs client typings (https://github.com/elastic/elasticsearch-js) in our SearchQuery interface and keep pagination related parameters, but I've added match and simple_query_string text queries.
Also the nativeSearch param is now optional in the SearchQuery interface and if not set will be set in our queryMetadata

unblocked. New Aquarius has been released where nativeSearch does not have to be passed anymore https://github.com/oceanprotocol/aquarius/pull/394
Meaning all those queries are supported under the query object https://www.elastic.co/guide/en/elasticsearch/reference/current/full-text-queries.html

superseeded by #662


Characteristic not found on device when using the sample app

We have a custom Bluetooth module we want to read and write to. I am able to connect to it but when I tap on TOGGLE, I get "Current status: Information not sent". Investigating this revealed that the getCharacteristic method in BluetoothConnection returns null right after this message is logged: Characteristic 4ac8a682-9736-4e5d-932b-e9b31405049c not found on device C6:2F:1D:80:A7:8A!
4ac8a682-9736-4e5d-932b-e9b31405049c is the characteristic in the sample app. How can I find the right characteristic to use with my Bluetooth device?

Hello there, thank you for your interest in the library.
Different devices will expose different characteristics, the one on the sample app is just an example.
If the device you have exposed the characteristics you can fetch them by using the library.
After connecting to your device, the connection instance will have the following properties:

notifiableCharacteristics
readableCharacteristics
writableCharacteristics

You can use them to list the available and exposed characteristics from your device. If they're empty that means nothing is being exposed, so you are better of trying to contact the manufacturer.

Did it work?

Closing due to inactivity.


Prepare repo transfer to public

Any Licenses we need to reference?

Hmm not sure, I think they are references in packages themselves when they are installed?


Firestore security image issues

Describe your environment

Xcode version: 10.1 swift 4.2.1
Firebase SDK version: 5.18.0
Firebase Component: firestore
Component version: 1.0.2

Describe the problem
When firestore security rules set to not allow write image of bytes length of more than 10k to firestore by using security rules, it work normally when write to firetore with image bytes length of less than 10k. If there is a one attempt of write image to firestore with 12k bytes length, the usual error occurred “Missing or insufficient permission” which is normal. However after this error, future write of image with less than 10k bytes length such as 5k or 6k bytes will also cause error “Missing or insufficient permission”, the error keep occur even I switch device. Below is the example security rules, hope it will help.
match /image/{documentID} {
        allow create: if request.auth.uid != null
                      && request.resource.data.size() == 3
                      && request.resource.data.keys().hasAll([ 'image', 'comment', 'timeCreated'])
                      && request.resource.data.image is bytes
                      && request.resource.data.image.size() < 10000
                      && request.resource.data.comment is string
                      && request.resource.data.timeCreated is timestamp

step to reproduce

Use above security rule
Upload a image with less than 10k bytes to prove it work normally
Upload a image with more than 10k to firestore, error will occurr which is normal
Try upload again the image with less than 10k bytes, error will keep occur which is unusual

Closed this issues due to inaccurate information on my part. Sorry for any inconvenience, feel free to delete this issues


Accessibility: Manage_Settings_DeveloperTool: The Clear search input button is wrapped in a  which is given a role toolbar.

"Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier."
Environment Details:
Version: 1.35.0-insider (user setup)
Commit: 443257e699bfa1d7402ee7e41f36cf3db1ac383f
Date: 2021-12-21T06:32:12.431Z
VS Code: 1.58.0
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000
Repro Steps:
Launch the application-Azure Data Studio
Tab till settings button and press enter.
Tab till search field.
Tab till clear search input button and inspect using Developer tool.
Actual:
The Clear search input button is wrapped in a <ul> which is given a role toolbar.

Expected:
The clear search input button should be given a role as presentation.
User Impact:
Screenreader users will get confused if the role is not correctly defined.
MAS Reference:
MAS 4.1.2 – Name, Role, Value

GitHub Tags:#A11yMAS;WCAG4.1.2;A11ySev2;#A11yTCS;#VisualStudioCode-Win32-Apr2021;#Visual Studio Code Client;

Usually, the toolbar designation makes sense because it indicates to the user that they are focused within an action bar, and that they must use the left and right arrows to navigate the buttons. An example of that case is in the keyboard shortcuts editor (accessible via Ctrl+K Ctrl+S).
For this issue, there is only one item in the action bar/toolbar. Therefore, the options I can think of are:

For consistency, we continue using the toolbar role on the ul element even when there is only one button in the toolbar, or
We only add the toolbar role onto the ul element when there is more than one action item.

Do you have any recommendation on which would be better to implement?
Also pinging @isidorn.

@rzhao271 I think that only option 1 makes sense.
We heard no user negative feedback regarding this, thus I suggest to not change anything here until we do. A one element toolbar is perfectly fine and it should not be treated as a special case.
Thus closing as designed


UniversalBot class is missing in botbuilder v4

Seems like botbuilder v4 or higher is not supported since there are missing classes there like UniversalBot, ConsoleConnector and etc.

You should be using the Bobuilder v4 test framework

@microsoftly I searched all over the documentation, and couldn't find a way to emulate a connector and inject messages (like sendMessageToBot) in botbuilder v4.
Can you please direct me to the right page?


Fixed: Navbar Overflow issue on smaller mobile devices.

Fixes Issue
Closes #7917
Changes proposed
Fixed the mobile navbar overflowing issue under width 300px, causing the Ui to break for smaller mobile devices.

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots
In production

After merging this pull request (PR)

Note to reviewers
There is a z-index problem in the mobile navbar but there already exists a issue on this. Refer #8507

There seems to be merge conflict, could some one confirm if
   <label htmlFor={name} className={`sr-only ${className}`}>     -> Current changes.
    <label htmlFor={value} className={`sr-only ${className} hidden md:block`}>  -> When I raised this PR

The difference is htmlFor = {name} or htmlFor = {value}, so could you confirm which change should I keep.
-Kartikeya Saini

I have resolved the merge conflict by keeping both changes.
     <label htmlFor={name} className={`sr-only ${className}`}>     -> Changes in production.
    <label htmlFor={value} className={`sr-only ${className} hidden md:block`}>  -> When I raised this PR.

    <label htmlFor={name} className={`sr-only ${className} hidden md:block`}>  -> After resolving the conflict.

Looks good to me


Fixed parameter names in Set-SPODocumentSetField samples.

Example 1:
SetAsSharedField -> SetSharedField
SetAsWelcomePageField -> SetWelcomePageField
Example 2:
RemoveAsSharedField -> RemoveSharedField
RemoveAsWelcomePageField -> RemoveWelcomePageField

Hi @andreashebeisen, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@andreashebeisen, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Thanks!


Posting data into magento.

Hello.
I am trying to post categories to magento using laravel. I found that this post, put, delete methods are protected. I can not use. Is there any other way? or why it is protected

$magento->api('categories')->post('/',$categoryArray);

  /**
     * Send a POST request with query parameters.
     *
     * @param  string  $path
     * @param  string  $parameters
     * @return \Illuminate\Http\Client\Response
     */
    protected function post($path, $parameters = [])
    {
        return $this->checkExceptions(Http::withToken($this->magento->token)
            ->post($this->apiRequest.$path, $parameters), $this->apiRequest.$path, $parameters);
    }

Hi @vimal-360,
We currently don't have active support on that post endpoint. Only to pull all categories; feel free to send a PR.
The protected post method is intentional; these are for custom endpoints. I would not recommend your method for making API calls. Please refer to the documentation. If you need a workaround you could use Laravel's HTTP client to make a post request to your endpoint.
Something like this:
$response = Http::post('http://my-magento-shop.com/rest/default/V1/categories/', $payload);

Thanks


Inconsistent naming between FEStateOptions and FiniteElementState

We should be consistent one way or the other.

It looks like FEStateOptions exists for the sole purpose of appearing as the second argument in the FiniteElementState constructor. If that's all it does, it feels like it should be defined inside the FiniteElementState,
https://godbolt.org/z/v5j6z6
struct Mesh{};

struct FiniteElementState{
    struct Options{
        int a;
        double b;
    };
    FiniteElementState(Mesh m, Options opt){}
};

int main() {
    FiniteElementState state(Mesh{}, {.a = 1, .b = 3.14});
}

On an unrelated note, it also looks like we may not need to explicitly name the FiniteElementState::Options type in the constructor to use the designated initializer syntax

It looks like FEStateOptions exists for the sole purpose of appearing as the second argument in the FiniteElementState constructor. If that's all it does, it feels like it should be defined inside the FiniteElementState,

I agree, though this actually relates to a compiler bug I found earlier present on both GCC and Clang.
In this case we can work around it by setting the defaults as an init-list in the FEState constructor instead of via non-static data member initialization.


add Azerbaijani localization

Overview

Azerbaijani translation completed
Link to index.html added

Review
@GasimGasimzada, please check if there are any translation or grammatical errors.

Wow! Thank you so much, @Maharramoff and @GasimGasimzada! ✨


C-sharp doesn't have a shape

I meant to assign it pentagon but I guess I missed that one.

Fixed in #26


Unable to load the requested language file: language/english/_lang.php

I was trying to implement language support in Codeigniter.
Any controller will throw this error.

This is a support question, and should be asked on the forum.
Why are you trying to load "_lang"? That isn't one of the language files.

See https://www.codeigniter.com/user_guide/libraries/language.html
"_lang" is a suffix for a package language file, and not used on its own.

This is a support question, and should be asked on the forum.
Why are you trying to load "_lang"? That isn't one of the language files.

I'm sorry, i will repost it in the forum.


Tab is misspelled in Advanced Component Communication section

In the Advanced Component Communication section in index.jade change
From:
"This allows tobs component to work with ngFor without any special knowledge of it."
To:
"This allows tab components to work with ngFor without any special knowledge of it."

Couldn't find that section, but since ngFor is involved, I assume this refers to either Angular 2 or http://angular.io.
This repository is for Angular 1. Please, reopen in either https://github.com/angular/angular or https://github.com/angular/angular.io.


featdisable join reorder if column stats is invalid (#41790) (branch-3.0)

Proposed changes
pick #41790
disable join reorder if any condition is matched:

any table row count is -1
any column, whose ndv is 0, but MinExpr or MaxExpr is not null
ndv > 10* rowCount

Issue Number: close #xxx

(cherry picked from commit e238a8705f25e2b5bc5a9e95b2025712f98dd682)
Proposed changes
Issue Number: close #xxx

run buildall


p2p: revisit the necessity of using libp2p connection manager

we are using off the shelf solution for limiting the number of active connections, it has three configuration parameters:

low-peers
high-peers
grace period for new connections

on reaching high-peers we prune all nodes that are connected for longer than grace period to a number specified in low-peers. note that high-peers is not an actual hardcap for the system, so it may go above that number.
it creates a loop where node periodically renews connections above low-peers number, not clear if there is any advantage.
the alternative is to introduce hardcap for inbound and outbound connections number and maintain it with connections gater.

one use case when connection manager makes sense is for bootnodes, they need to prune peers periodically to accept new peers and let them get connections. for regular node i can't justify using it, simple hardcap will lead to more stable connections.


Boolean and Text data simulator

This pull request enhances the data simulator by adding support for generating text and boolean data types. Additionally, it addresses a bug in the random generation of integers within a specified range.

Changes Made:

Added support for generating text data.
Added support for generating boolean data.
Fixed a bug in the random generation of integers within a specified range.

Bugfix: Resolve Infinite Loading Issue on LineChart Component

When the cursor go out of the widget the loading start and never stop during a request of new data
PickerPopUpComponents: manage if on drag on the chart, the element hasn't a parent


Include GnuPG

Trying to add a Debian repo's GPG key via apt-get results in this message:
gnupg, gnupg2 and gnupg1 do not seem to be installed, but one of them is required for this operation

Seeing as this is a very common thing to do, could you please include GnuPG in the Docker image?

This only effects stretch/sid and was reported in https://github.com/tianon/docker-brew-debian/issues/49.
From the Debian bug report:

Apt recommends gnupg | gnupg2 which is the correct choice so we can build minimal environments
without pulling in the whole GPG stack (agent, other daemons).

This allows us to build smaller images and then let users install, use, and remove gpupg to keep their images slim too.

Thanks for the info! Closing this as a duplicate of #49.


[ECard Settings] - Add “Report virtual card fraud” page and route

HOLD ON https://github.com/Expensify/App/issues/22873
Design Doc Section
https://docs.google.com/document/d/1rFxJ78vz5On6zSWzYa51B9v-tyLdC5pUsBeLOLig0t4/edit#bookmark=id.q9y9t2vy090m
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~017ab90cdfe61c6886
Upwork Job ID:<PHONE_NUMBER>779143680
Last Price Increase: 2023-07-14

cc @pac-guerreiro

Could you assign me to this @grgia ? 😄

@pasyukevich Any updates here?

@s77rt  working on it

Work in progress. This issue is connected with https://github.com/Expensify/App/issues/22873

In progress

Hi, I'm Julian from Callstack - expert contributor group - and I would like to ask you to reassign this issue to me as @pasyukevich is out of office and I have taken over his issues.

not overdue

cc @joekaufmanexpensify could you help with $500 C+ review payment to @s77rt

We need to pay @s77rt $500 for their C+ review of this PR. Paid via NewDot. @s77rt could you please submit a request for $500 and confirm here once that's complete?

@joekaufmanexpensify I'm still getting paid via Upwork.

Ah, my bad! I updated my comment above, and will send you an offer now.

original job was closed, so made a new one: https://www.upwork.com/jobs/~015daf95cf94d7dd28

@s77rt $500 offer sent via upwork!

Accepted! Thanks!

@s77rt $500 sent and contract ended!

Closing as payment has been handled!


Fix a bug where we were treating an int as a str.

The to_float function expects a str or None, but we were passing it an int. The function then tries to replace ';' and ',', but that's not a valid operation on an int. Instead, where the parameter is missing, we'll default to None which is handled by to_float and apply the int default afterwards.

@rmarianski could you review, please?

:+1: Great catch! :zap:


报错了

CompletedProcess(args=['edge-tts', '--voice', 'zh-CN-XiaoyiNeural', '--text', '天空蔚蓝', '--write-media', 'voices/1695236088-pollinations-ai.mp3', '--write-subtitles', 'voices/1695236088-pollinations-ai.mp3.vtt'], returncode=0)
voices/1695236088-pollinations-ai.mp3.vtt
Building prefix dict from the default dictionary ...
Dumping model to file cache /tmp/jieba.cache
Loading model cost 1.706 seconds.
Prefix dict has been built successfully.
[2023-09-21 02:54:54,993] ERROR in app: Exception on /convert [POST]
Traceback (most recent call last):
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 2190, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1486, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1484, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python3.8/dist-packages/flask/app.py", line 1469, in dispatch_request
return self.ensure_sync(self.view_functions[rule.endpoint])(**view_args)
File "app.py", line 21, in convert_text_to_video
video_path = convertTextToVideo(validate_model(model), text)
File "/root/tts/text2video/text_to_video.py", line 126, in convertTextToVideo
add_text_to_image(draw_text, image_path,
File "/root/tts/text2video/add_text_to_image.py", line 30, in add_text_to_image
if draw.textsize(current_line  + word, font=font)[0] <= target_size[0] - 2 * padding:
AttributeError: 'ImageDraw' object has no attribute 'textsize'
<IP_ADDRESS> - - [21/Sep/2023 02:54:55] "POST /convert HTTP/1.1" 500 -

edge-tts需要自己安装吗？

支持docker一键启动


Add types to ZodFirstPartySchemaTypes

Fixes #3246
Some first party schema types were missing from the union. This pull-request adds them, along with a test that tries to keep the union and ambient type declaration in sync.

This has landed in Zod 3.23.
https://github.com/colinhacks/zod/releases/tag/v3.23.0


Supporting other Python Frameworks Gracefully in Matrix Display

Proposed Patterns
This is to talk through support for other frameworks, and possible ways to handle it safely

[x] Spark
[x] Tensorflow

One approach which I've demo'ed is to use pd.get_options to control spark dataframes when pulling datasets through
spark_df.limit(<number>).toPandas()
For tensorflow variables you will inevitably pulling full matrices down.
Displaying Pandas and Numpy
One option for handling the display of data is to limit it to:
max(10**6, pd.get_option('display.max_rows')) if pd.get_option('display.max_rows') is not None else <show all>

We can do the same thing for the numpy equivalent. This should naturally flow-on to the default Pandas + Numpy variables.
Handling Imports
Using the approach shown in the Rodeo variable explorer I think the pattern for handling unknown imports (not everyone will have tensorflow/pyspark or even pandas installed):
try:
    import my_module
except ImportError:
    my_module = None

....

if my_module and isinstance(x, my_module.my_type):
    <do stuff>

If @lckr agrees, I'll ensure my PRs are aligned and cleaned up for review.

I like your proposal and I'm looking forward to review your PR.

@chappers Does this issue need more work or would you consider it done?

I'll consider it done for now. New issues can be opened if others want support


Maven Artifact Resolver

A very popular issue when using Recaf is the fact that some dependencies are missing which are causing failed recompilation.
To add users to find the corresponding jars to add as library, there should be a utility for downloading artifacts based on maven coordinates.
These are the features I would suggest being available either built-in or as plugin:

Search for dependencies, like IntelliJ allows it to.
A search of artifacts based on packages.
Resolved dependencies should being able to be downloaded and linked to the workspace.

Upon investigation, the Maven Artifact Resolver can be used, this is, as noted by some, a "beefy" library, so it should be considered if this feature is better off in a plugin.

https://maven.apache.org/repository/central-index.html
https://maven.apache.org/maven-indexer/indexer-core/
https://oss.sonatype.org/#nexus-search;quick~
https://oss.sonatype.org/service/local/lucene/search?_dc=1693511344333&cn=jakarta.annotation.Nonnull
could be sued for searching based on FQN of classes.


Add support for launching containers using spec file

Cherry picked from master
https://github.com/rdkcentral/Thunder/pull/961


Update index.md

Fixed formatting

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

In the future, can you please give your PRs a more descriptive title so it's easier to understand and keep track of. Every single change to the Guide could be called "update index.md".

In the future, can you please give your PRs a more descriptive title so it's easier to understand and keep track of. Every single change to the Guide could be called "update index.md".


Use upstream delimited stream

Which issue does this PR close?

Closes #.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

FYI any "no space left on device" CI errors should be fixed by merging up from master to get https://github.com/apache/arrow-datafusion/pull/5047

There seems to be a few errors in CI for this PR --

I hope to replace this with the new CSV and JSON Decoder abstractions in the next release of arrow, so closing this for now


Added Competitive Programming Roadmap

Pull Request
Description
Added the competitive programming roadmap inside of the DSA
Closes #1013
Type of change

[ ☑] Adding a new Resource
[ ] Adding a new Category
[ ] Documentation enhancement or fixes
[ ] Website enhancement or fixes

Checklist:

[☑ ] I read carefully CONTRIBUTING.md
[☑ ] I have performed a self-review of my own code
[☑ ] I have commented my code, particularly in hard-to-understand areas
[ ☑] I have made corresponding changes to the documentation

Screenshots (if appropriate):
Please add screenshots to help explain your changes.

Additional

Add the URL into this link


Get the cpu usage per core in Windows

The following function needs to be implemented.

func (self *CpuList) Get() error {
	return notImplemented()
}

Getting cpu usage per core under Unix systems is implemented in the PR: https://github.com/elastic/topbeat/pull/130

Requested by https://github.com/elastic/topbeat/issues/40 as well.

https://github.com/UhuruSoftware/gosigar/blob/master/sigar_windows.go is a possible implementation which we could go for.

@vjsamuel Thank you. I'll give it a try and let you know if it works.

How do you generally test in windows environment?

You can use the virtual machine from https://github.com/elastic/winlogbeat/blob/master/Vagrantfile and then install gcc by using choco install mingw64.

@vjsamuel FYI we have an updated Vagrantfile in the elastic/beats repo that has the development environment fully setup for compiling Topbeat and gosigar. It should be as simple as:
vagrant up
vagrant ssh
cd C:\Gopath\src\github.com\elastic\beats\topbeat
go build


[MM-18118] Check locale is valid before sending to intl_provider

Summary
If an invalid locale is specified in the database, upon refreshing the app a logged in user will experience a blank screen. This PR forces the locale to the default in this case, such that intl_provider is never fed an invalid locale.
Ticket Link
https://mattermost.atlassian.net/browse/MM-18118

Is this something that unit / e2e tests can and should be added for? @devinbinnie

@lindalumitchell Definitely, I'll add a unit test to ensure an invalid user locale returns the default locale. Thanks for reminding me :)

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3558.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


Outline tree flashing at every character written

Issue Type: Bug
New this morning, after no apparent change in VS Code.
Editing a markdown file. At every character entered, the outline tree is deleted and recreated.
In the output panel, plenty of errors as the following appears:
[2019-11-09 06:43:48.733] [renderer1] [error] TreeError [OutlinePanel] Tree input not set: Error: TreeError [OutlinePanel] Tree input not set
    at Q.updateChildren (file:///C:/Users/Mario Valle/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1047:30)
    at e._doUpdate (file:///C:/Users/Mario Valle/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:5269:515)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:89:5)

VS Code version: Code 1.40.0 (86405ea23e3937316009fc27c9361deee66ffbf5, 2019-11-06T17:02:13.381Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz (8 x 2904)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: enabledrasterization: enabledskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.81GB (6.10GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (35)

Extension
Author (truncated)
Version

Bookmarks
ale
10.5.0

spellright
ban
3.0.48

github-markdown-preview
bie
0.0.2

markdown-footnotes
bie
0.0.7

markdown-yaml-preamble
bie
0.0.4

vscode-toggle-quotes
Bri
0.3.2

refactorix
chr
0.3.7

bracket-pair-colorizer
Coe
1.0.61

vscode-eslint
dba
1.9.1

vscode-pandoc
Dou
0.0.8

eslint-disable-snippets
drK
1.2.0

gitlens
eam
10.1.2

vscode-great-icons
emm
2.1.47

vscode-todo-plus
fab
4.14.1

auto-close-tag
for
0.5.6

sass-lint
gle
1.0.6

todo-tree
Gru
0.0.162

tag-rename
kri
0.2.1

stack-tabs
kyl
0.0.2

git-graph
mhu
1.18.0

prettify-json
moh
0.0.3

notepadplusplus-keybindings
ms-
1.0.3

leaper
Onl
0.6.0

vscode-jscpd
pau
0.3.4

vscode-versionlens
pfl
0.24.0

bib
phr
0.3.0

vscode-javascript-booster
sbu
0.11.0

trailing-spaces
sha
0.3.1

markdown-preview-enhanced
shd
0.5.0

unique-window-colors
stu
1.0.51

highlight-matching-tag
vin
0.9.5

vscodeintellicode
Vis
1.2.0

highlight-bad-chars
wen
0.0.3

change-case
wma
1.0.0

markdown-all-in-one
yzh
2.5.1

(1 theme extensions excluded)

Some additional info.

I'm not sure to have edited any markdown file after 1.40.0 upgrade, so the problem could have started after the upgrade.
A reboot cleared the problem for few minutes, then it restarted.
The outline has "filter on type" and "sort: by position" checked
In explorer the workspace file list is collapsed, the outline is expanded

Did you try running with all your extensions disabled in case it is being caused by one of those?

Does repo for me... @crystalfp Does the error persist? Is that related to specific markdown extension? Can you try disabling them as suggested?

Let me arrive home and will check. Thanks!

Seems the culprit is Spell Right. If I disable it, the Outline does not flash. The strange thing is that the error message in the Log (Window) panel makes no mention of this extension.
Opened as spell-right issue 320

Sorry, it seems a red herring. The problem appears even with Spell Right extension disabled, but depends on the kind of editing I'm doing. I will shortly prepare a reduced example that shows the problem.

Strange, strange.
If I disable all extensions, the problem goes away. Then, if I reenable them except "Bracket Pair Colorizer" seems the problem is solved. Enabling it and the problem reappears. So this extension should be the culprit.
Not so fast. If I disable it again, the problem is back. If I disable all extensions except it, the problem does not appear.
At the end seems that disabling both "Bracket Pair Colorizer" and "Spell Right" solved the problem.
At the end I'm not inclined to make them guilty, due to the above incoherences, but seems some form of interaction between them and the latest VS Code version.
Thanks for helping!
mario

I have the same problem, and I don't use either of those extensions.
That said, the problem only occurs with one particularly long JavaScript file (13k+ lines of code and comments, 300+ functions).  No other files cause the problem where the "Outline" tree vanishes a brief moment after hitting one key, and then reappears a brief moment after hitting another key.  If I hit two keys in quick succession, then it does the same thing as hitting one.  (This happens even if I disable all of my extensions.)
Based on this, I'd guess that there is some code which is expecting something to be done within a certain amount of time, and when it takes too long (because of the large amount of code), it's failing, thus causing the outline tree to get blanked out.  Then, the next time, the tree is empty, so it gets done in time, and the outline tree shows properly again.
Anyways, it's really annoying, so if anyone has any ideas, a fix would be greatly appreciated.

Yes! My markdown file is quite long. This is why I was not able to create a small example

My markdown file: 2300 lines 256KB.
Strange enough, this morning with all extensions enabled, the problem does not reappear (for now). Maybe a good night sleep helped also VS Code :-)

My markdown file: 2300 lines 256KB.
My JavaScript file is 566KB, so yeah, it looks like it's large files that are the problem.

Strange enough, this morning with all extensions enabled, the problem does not reappear (for now). Maybe a good night sleep helped also VS Code :-)
It probably also depends on what other things you have running in the background, which could slow things down enough to cause the problem.

Can one you share those files with me?

Can one you share those files with me?

Here you go.

@HiEv Thanks! I could reproduce after the keystroke 😮

Alright, problem is that after 100ms we show a message saying "Loading symbols..." and that resets the tree so that the subsequent update finds the tree in an unexpected state...

This was introduced with bcaa2be1b51441fd11685e599b2704908f25a6f2

This will be fixed in tomorrows insider's build

Re-open so that we can cherry-pick this for the October Recovery release


cake-contrib user on Nuget

First of all, I wanted to thank you for adding to the Cake community by adding this addin.
I was just wondering if you had seen this blog post:
http://cakebuild.net/blog/2016/08/cake-contribution-organization
We are currently going through a process of increasing the visibility of addins, and also trying to ensure their long term maintainability.
To that end, we are asking addin creators to add the cake-contrib user on NuGet as a co-owner (this can be done through the NuGet website by clicking on Manage Owners on the package page).
Would you be interested in doing this for Cake.AWS.EC2?  If you have any questions about this, please let me know.  There was some initial concern that the Cake Team were trying to "take over" packages, and that couldn't be further from the truth, and if you have this concern, or others, I would like to address them.
Thanks!

Done 👍

Thank you!


tiles with dynamic image width and height

Hello guys,
I do have an issue implementing tiled image with dynamic width and height. Hopefully you can help me.
I have a clear response from my backend service:
{
    "tileResolution": {
        "width": 128,
        "height": 128,
    },
    "zoomLevels": [
        {
            "zoomLevel": "HD",
            "width": 1080,
            "height": 720
        },
        {
            "zoomLevel": "FULL_HD",
            "width": 1620,
            "height": 1080,
        },
        {
            "zoomLevel": "ULTRA_HD_4K",
            "width": 3240,
            "height": 2160
        },
        {
            "zoomLevel": "FULL",
            "width": 6000,
            "height": 4000
        }
    ]
}

Based on that I need to construct a tileUrl
basically, on different zoom levels (HD, FULL_HD, ULTRA_HD_4K, etc.) I need to provide different width and height). This is how image folders are structured. How could I changed this dynamically according to the structure.
Url should be like /images/${zoomLevel}/${height}/${width}.
For testing purposes, with static url everything works as a charm. Challenges comes, when I need to change this dynamically on zoom.
my goal:

on different zoom levels provide different zoomLevel name to url
on different zoom level provide different width and height of the image, so images are taken from correct url and width&height is accurate - tiles calculated correctly
Any wise advices?

currently, with static url ,my code looks like
  OpenSeadragon({
      element: this.$refs["photo-container"],
      showNavigator: false,
      blendTime: 0,
      constrainDuringPan: true,
      visibilityRatio: 1,
      homeFillsViewer: true,
      minZoomLevel: 1,
      wrapHorizontal: false,
      tileSources: {
        height: 1620,
        width: 1080,
        tileSize: 128,
        minLevel: 0,
        getTileUrl: function(level, x, y) { return `imageUrl/FULL_HD/${y}/${x}`}
       }
})

Hmm... I don't think OpenSeadragon is actually set up to support this scenario. If an image is just a pyramid, but not tiled, you can use the Legacy Tile Source:
http://openseadragon.github.io/examples/tilesource-legacy/
If each level of the pyramid is tiled (which it sounds like yours is), the assumption is that each smaller level will be exactly half the size of the level above it. I don't think we have any way to have non-doubling tiled pyramids.
Of course, this can always change... I'm not sure exactly what would be involved (probably modifying TileSource and TiledImage), but the LegacyTileSource could be a good guide for how to do it. If you're interested in taking a whack at it, let me know and I can help with some guidance.


Windows 7 Error

I try to run the program in a Windows 7 laptop and an error shows:
An error occured in the editor, please report to the creator: System.BadImageFormatException: Could not load file or assembly 'Presentation Framework.Aero2, PublicKeyToken=31bf3856ad364e35' or one of its dependencies. Reference assemblies should not be loaded for execution. They can only be loaded in the Reflection-only loader context. (Exception from HRESULT: 0x80131058) File name: 'Presentation Framework.Aero2, PublicKeyToken=31bf3856ad364e35' ---> System.BadImageFormatException: Cannot load a reference assembly for execution.
at System.Reflection.RuntimeAssembly._nLoad(AssemblyName fileName, String codeBase, Evidence assemblySecurity, RuntimeAssembly location Hint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forintrospection, Boolean suppress Security Checks)
at System.Reflection.RuntimeAssembly.nLoad(AssemblyName fileName, String codebase, Evidence assembly Security, RuntimeAssembly location Hint, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throw OnFileNotFound, Boolean forintrospection, Boolean suppressSecurity Checks)
at System.Reflection.RuntimeAssembly.InternalLoadAssemblyName(AssemblyName assembly Ref, Evidence assembly Security, RuntimeAssembly reqAssembly, StackCrawlMark& stackMark, IntPtr pPrivHostBinder, Boolean throwOnFileNotFound, Boolean forintrospection, Boolean suppress Security Checks)
at System.Reflection.Assembly.Load(AssemblyName assembly Ref) at System.Windows.Bam 2006.Bam 2006SchemaContext.Resolve Assembly(Bam Assembly bamlAssembly) at System.Windows.Bam|2006.Bam2006SchemaContext.ResolveBamlTypeToType(BamlType bamlType) at System.Windows.Bam|2006.Bam|2006SchemaContext.ResolveBamlType(BamlType bamlType, Int16 typeld) at System.Windows.Bam 2006.Bam 2006SchemaContext. GetXamlType(Int16 typeld) at System.Windows.Bam 2006.Bam 2006 Reader.Process_ElementStart at System.Windows.Bam 2006.Bam 2006 Reader.Process OneBamRecord at System.Windows.Bam 2006.Bam 2006 Reader.ReadObject(KeyRecord record) at System.Windows.ResourceDictionary.CreateObject(Key Record key) at System.Windows.ResourceDictionary.OnGettingValue(Object key, Object& value, Boolean& canCache) at System.Windows.ResourceDictionary.OnGettingValuePrivate(Object key, Object& value, Boolean& canCache) at System.Windows.ResourceDictionary.GetValueWithoutLock(Object key, Boolean& canCache) at System.Windows.ResourceDictionary.GetValue(Object key, Boolean& canCache) at System.Windows.ResourceDictionary.GetValueWithoutLock(Object key, Boolean& canCache) at System.Windows.ResourceDictionary.GetValue(Object key, Boolean& canCache)
at System.Windows.ResourceDictionary.FetchResource(Object resourceKey, Boolean allowDeferred ResourceReference, Boolean mustReturnDeferred ResourceReference, Boolean canCacheAsThemeResource, Boolean& canCache)
at System.Windows.FrameworkElement.Find ResourceInternal(FrameworkElement fe, FrameworkContentElement fce, DependencyProperty dp, Object resourceKey, Object unlinkedParent, Boolean allowDeferred ResourceReference, Boolean mustReturnDeferred ResourceReference, DependencyObject boundary Element, Boolean isimplicitStyleLookup, Object & source)
at System.Windows.FrameworkElement.FindImplicitStyleResource(FrameworkElement fe, Object resourceKey, Object & source) at System.Windows.FrameworkElement. GetRawValue(DependencyProperty dp, Property Metadata metadata, EffectiveValueEntry & entry)
ndows.FrameworkElement.EvaluateBaseValueCore(DependencyProperty dp, Property Metadata metadata, EffectiveValueEntry & newEntry) at System.Windows.DependencyObject.EvaluateEffectiveValue(EntryIndex entryIndex, DependencyProperty dp, Property Metadata metadata, EffectiveValueEntry oldEntry, EffectiveValueEntry newEntry, Operation Type operation Type)
at System.Windows.DependencyObject.UpdateEffectiveValue(EntryIndex entryIndex, DependencyProperty dp, Property Metadata metadata, EffectiveValueEntry oldEntry, EffectiveValueEntry& newEntry, Boolean coerceWith DeferredReference, Boolean coerceWithCurrentValue, Operation Type operation Type)
at System.Windows.DependencyObject.InvalidateProperty(DependencyProperty dp, Boolean preserveCurrentValue) at System.Windows.FrameworkElement.UpdateStyleProperty at System.Windows.FrameworkElement.OnInitialized(EventArgs e) at System.Windows.Controls.MenuItem.OnInitialized(EventArgs e) at System.Windows.FrameworkElement.TryFireInitialized at System.Windows.FrameworkElement.Endinit at System.Windows.Controls.ItemsControl.EndInit at MS.Internal.Xaml.Runtime.ClrObjectRuntime.InitializationGuard(XamlType xaml Type, Object obj, Boolean begin)
I would really appreciate if someone would share a solution to my problem

Rename the current PresentationFramework.Aero2.dll to PresentationFramework.Aero2.dll.bak.
Download the following zip PresentationFramework.Aero2.zip and extract the PresentationFramework.Aero2.dll from it along side the GTEventGenerator2.exe file.

Does it still give you the same error?

It doesnt give me the error now. Thank you very much!


how to cal     low threshold  and high threshold

       "{low            |20        | low threshold                 }"
                    "{high           |40        | high threshold

I also have the same question. I use the algorithm on images with different exposures. The default settings are not suitable for all exposures

Perhaps you can switch to a different edge detection method before the first step and before finding the contour points.


[API. Challenges] Incorrect response status is appears and challenge does not deleting when user tries to delete challenge

Environment: Windows 10 x64, Postman v9.0.7.
Reproducible: always.
Build found: last commit.
Preconditions

Run Postman
Create new request > Choose 'GET' in 'Method' drop-down
Enter request URL: https://speak-ukrainian.org.ua/dev/api/challenge/3
Navigate to 'Authorization' tab, choose 'Bearer Token' in 'Type' drop-down
Set 'accessToken' of user with admin rights (e. g. 'email'<EMAIL_ADDRESS>'password': "admin") in 'Token' field
Click on 'Send' button
Make sure that server responds with status 200 and info about task is appeared

Steps to reproduce

Choose 'DELETE' in 'Method' drop-down
Click on 'Send' button

Actual result
Server is respond with 500 status - 'Internal Server Error' and challenge does not deleting.
Expected result
Server should respond with status 200 and challenge should be deleted.

Screenshots:
Precondition - GET-request to make sure that challenge under /api/challenge/3 endpoint is created

Actual result - DELETE-request.

This issue interdependent with #926 issue, so we cannot check is it fixed till #926 will be fixed.


Create a secure API that will return a list of login attempts.

As an admin of GC form, I would like to get all the "login attempts" as well as for a given form id.
These logs should come from the "access_log" table.
Acceptance criteria:

[ ] Create an API route that accepts GET requests.
[ ] The API path should be  /api/login/attempt.
[ ] This API endpoint should allow requests with valid admin bearer token or session*.
[ ] This endpoint should return all the login attempts.
[ ] if there is no "login-attempts" an empty list should be returned. Otherwise a list of "login attempt".
[ ] A reasonable level of unit testing is required.
[ ] if the bearer token is invalid return a 403 status code.

** To be approved :  with query parameters**

[ ] with a query parameter user_id  '/api/login/attempt?user_id='.
[ ] if the user_id cannot be found in access_log a 404 status code should be returned. otherwise return a list of "login-attempt".

This information does not need to be exposed through an API.  The database call can be achieved through getServerSideProps.  If this was a pure React application this would have been the way to go however given that this is SSR (Server-Side Rendered) React we should take advantage of the platform.  Adding additional APIs, where the same functionality can be achieved purely server-side, exposes additional security attack paths.
Please modify the acceptance criteria.


elasticsearch plain text password in logs

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):
Setup elastic search.
Result:
Observe plain text password in /var/log/containers/fluent*
Other details that may be helpful:
{"log":"2019-02-25 13:54:20 +0000 [warn]: #0 failed to flush the buffer. retry_time=8 next_retry_seconds=2019-02-25 13:54:20 +0000 chunk="582b7ed065a06ad08bff99637b74457 f" error_class=URI::InvalidURIError error="bad URI(is not URI?): https://username:plaintextpassword@secret.host.com:3602"\n","stream":"stdout","time":"201 9-02-25T13:54:20.507771327Z"}
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): v2.1.3
Installation option (single install/HA): HA

Talked to @orangedeng , he has removed it in the log. Please test it in master

This issue has been fixed in https://github.com/rancher/rancher/pull/18543. The master build should be no long logging the password.

Same reason as https://github.com/rancher/rancher/issues/18389. Closing duplicate

@kingsd041 it's another issue

Version: master 2.2 (3/12/19)
Confirmed fix.

Config with a password with special characters, for example, Please@ChangeM#e. ES starts successfully and can receive logs
ES password will not be displayed in the fluentd log


Security Helper

This needs to be looked at in more detail, but is likely a port of the existing, while removing xss_clean.
Development Checklist:

[ ] Component(s)
[ ] ... with PHPdocs
[ ] Unit testing
[ ] ... with >80% coverage
[ ] User guide updated
[ ] Classmap integration?
[ ] Securely signed commits
[ ] Conforms to style guide

We seem to be missing a few methods in \CodeIgniter\Security\Security.php. Are these methods removed or moved somewhere?

getRandomBytes()
entityDecode()//??
encodePHPTags()//??
And the last one, xssClean(). This has a lot of features such as compactExplodedWords(), removeMaliciousAttributes(), sanitizeNaughtyHTML(), removeJSLink(), removeImageLink()

The xss_clean functions are not being included since Zend/Escaper that we ship with the framework does a better, more thorough, job of handling xss protections. So some functions aren't included in that library because they were only used by the xss_clean functions.

get_random_bytes is not needed because PHP7 has a crypotgraphically secure version of it natively now, in random_bytes().
entity_decode seems redundant since it's primarily a copy of html_entity_decode.

Nice. Thanks for the infos.


how to parse string

I have problem with parsing example like this:
auto result1 = scn::scan<std::string, double>(":x22x , 3.14", ":x{}x , {}");
Either I dont understand something or documentation is missing important case ?
the other case like this:
auto result1 = scn::scan<std::string, double>(":x22x , 3.14", ":x{}x , {}");
works as expected. How to tackle the string case ?

The default behavior for reading a string is to scan until a whitespace character: https://www.scnlib.dev/latest/group__format-string.html#type-string, which is the same behavior as with scanf and %s. Therefore, the following is what happens:
auto result = scn::scan<std::string>("x22x", "x{}");
// result->value() == "22x"

The format string following the {} is not inspected to determine, what characters to accept when reading a string, and when to stop reading. To determine what characters to accept, the character set syntax can be used:
// [^x]: accept everything but 'x'
auto result = scn::scan<std::string, double>(":x22x , 3.14", ":x{:[^x]}x , {}");
// std::get<0>(result->values()) == "22"
// std::get<1>(result->values()) == 3.14

Your second case is identical to the first, so I can't help with that.


Not working with certain tokenizer(s)

Describe the bug
The library works fine with google-bert/bert-base-cased but with thenlper/gte-small (and maybe others), it shows weird calculation output: the encoded output is always 128 in length, etc.
To Reproduce Tokenizer not working with gte-small
from semantic_text_splitter import TextSplitter
from tokenizers import Tokenizer

text = """
This is an example text This is an example text
"""

model_name = "thenlper/gte-small" # limit 512 tokens

max_tokens = 300 # Modify this to 5 tokens also lead to weird output with 3-chars chunks
tokenizer = Tokenizer.from_pretrained(model_name)
splitter = TextSplitter.from_huggingface_tokenizer(tokenizer, trim_chunks=False)

chunks = splitter.chunks(text, max_tokens)

print("Number of chunks =", len(chunks))
for i,v in enumerate(chunks):
    print(f"CHUNK {i}, length={len(tokenizer.encode(v))} tokens, {len(v)} chars")
    print(v)

Will produce just one chunk with number of tokens = 128. Feel free to replace the text with a longer one, the issue insists.
I included the sample notebook to reproduce.

@xtfocus thanks for reporting. It seems that this tokenizer does some padding, which is why it is always at 128, but I am also seeing some repeated chunks, which is odd...
I will definitely dig in and see what I can find out. I'll see if there is a fix, or if there needs to be some extra documentation at the very least because I can definitely see a problem for any tokenizer that uses padding.

Hi @xtfocus the fix for this was release in https://github.com/benbrandt/text-splitter/releases/tag/v0.9.0
Hope this helps!


Global player configuration

We're using MediaElement.js in several WordPress plugins and themes and need to register features for any player that exists on a page. With version 2.2.0, we were able to do something like this to proxy the init() method and register new features:
var mePlayerInit = MediaElementPlayer.prototype.init;
MediaElementPlayer.prototype.init = function() {
	if ( -1 === this.options.features.indexOf( 'featurename' ) ) ) {
		this.options.features.unshift( 'featurename' );
	}

	mePlayerInit.call( this );
};

Now that 4.x is bundled in WordPress 4.9, that method is no longer available and it doesn't look like it's possible to configure players globally. I haven't opened a ticket for WordPress yet and was hoping we could work out a solution here to have it included in 4.9. Otherwise, we have a lot of users whose sites are going to break.

OK I'll take a look at this soon. Thanks for reporting this

Looks like that took care of it. Thanks for the quick turnaround on this!

Thanks for reporting this. Closing this issue and please star the project to support it


Trezor API - Add support for passphrase

Third-party consumers of MultiBit Hardware require the advanced use case of the optional BIP44 passphrase support during wallet creation and recovery.

I'm missing hearth smiley here in github comments.

Merged in PR #30 by @martin-lizner. Requires verification.


clean up requests dependency

I think requests might only be used in the examples so it's not needed in the main list of dependencies. Investigate defining an examples dependency group (maybe re-define the jupyter group) to clean this up

Hi.
I fixed the issue.
I did some investigation but it didn't see usage of requests outside of examples.
Created a new group named examples.
If you think re-defining the jupyter group is better let me know.
https://github.com/resgroup/wind-up/pull/19

Thank you @izofat ! PR is merged


Bump buildpackapplifecycle

This PR was generated by the buildpacks releng team bot

Change applied via https://github.com/cloudfoundry/diego-release/commit/f2a8cbd652106d31e8e2c1c435938972982b2108


HTTP client redirects not working

Description:
Following redirect example gives the error
{message:"Remote host closed the connection without sending inbound response", cause:null, statusCode:-1}
In a browser, it returns "Redirect works!"
import ballerina/io;
import ballerina/net.http;
import ballerina/mime;

endpoint http:ClientEndpoint clientEP {
    targets:[{uri:"http://www.mocky.io"}],
    followRedirects : { enabled : true, maxCount : 5 }
};

function main (string[] args) {
    http:Request req = {};

    //Send a GET request to the specified endpoint
    var returnResult = clientEP -> get("/v2/59d590762700000a049cd694", req);
    match returnResult {
        http:HttpConnectorError connectorErr => {io:println(connectorErr);}
        http:Response resp => {
            match resp.getStringPayload() {
                mime:EntityError payloadError => {io:println(payloadError.message);}
                string payload => io:println("Response received for the GET request is : " + payload);
                any | null => io:println("null payload");
            }
        }
    }
}

Steps to reproduce:
Affected Versions:
0.964.1
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):
HTTP Client

Suggested Assignees (optional):

Tried the following on v0.970.0-alpha1-SNAPSHOT and it's working as expected. Hence closing this issue.
import ballerina/io;
import ballerina/http;
import ballerina/mime;

endpoint http:ClientEndpoint clientEP {
    targets:[{uri:"http://www.mocky.io"}],
    followRedirects : { enabled : true, maxCount : 5 }
};

function main (string[] args) {
    http:Request req = {};

    //Send a GET request to the specified endpoint
    var returnResult = clientEP -> get("/v2/59d590762700000a049cd694", req);
    match returnResult {
        http:HttpConnectorError connectorErr => {io:println(connectorErr);}
        http:Response resp => {
            match resp.getStringPayload() {
                http:PayloadError payloadError => {io:println(payloadError.message);}
                string payload => io:println("Response received for the GET request is : " + payload);
            }
        }
    }
}


remove unsupported comments from code example

remove unsupported comments from code example.
similar to https://github.com/fsprojects/Paket/issues/1123
also cleaned up the styling in that section to keep same information.

Hello,
I don't know if this file touches the web page for no content option as that is what people see first. If not, that would also need a to be changed. I can make an additional PR if needed. Thank you. Good day.

thanks!


Support websocket through forward http proxy ?

Hi,
Nice simple web socket client, easy to use.
Still, I am missing something: possibility to go through a forward http proxy.
Since the implementation uses a TCPsocket, then it does not honour the http_proxy env var, so the ruby http library way of handling proxies...
Maybe someone has this in the pocket ?
I wonder if it would be possible that the oimplementation uses Ruby's HTTP object (with connect, etc...) instead of raw TCPSocket...

Thanks,
no problem,
Since the core of the work is in the websocket gem...
I now use that lib directly, and dropped websocket-client-simple , but that was useful to speedup at the beginning.
the new code is more embedded in the app here:
https://github.com/IBM/aspera-cli/blob/develop/lib/aspera/fasp/agent_httpgw.rb#L78
The idea is:
http_socket = Net::HTTP.start(...)
# this is the BufferedIO equivalent to a socket
ws_io = http_socket.instance_variable_get(:@socket)
# and then it is similar to this gem, using ::WebSocket::Handshake
# and to finalize:
http_socket&.finish

The advantage of that is that if the user specifies a proxy in Net::HTTP.start, then the websocket is forwarded through the proxy...

Thanks,
no problem,
Since the core of the work is in the websocket gem... I now use that lib directly, and dropped websocket-client-simple , but that was useful to speedup at the beginning.
the new code is more embedded in the app here: https://github.com/IBM/aspera-cli/blob/develop/lib/aspera/fasp/agent_httpgw.rb#L78
The idea is:
http_socket = Net::HTTP.start(...)
# this is the BufferedIO equivalent to a socket
ws_io = http_socket.instance_variable_get(:@socket)
# and then it is similar to this gem, using ::WebSocket::Handshake
# and to finalize:
http_socket&.finish

The advantage of that is that if the user specifies a proxy in Net::HTTP.start, then the websocket is forwarded through the proxy...

Hi @laurent-martin The link you provided gives 404, Could you provide more detailed steps to make the websocket-client-simple proxy supported? I am using websocket-client-simple in my Gem and have similar requirement to make it capable of using the proxy.


Kluctl should warn about fields being added by other field managers which Kluctl can't remove

Command

[ ] check-image-updates
[ ] delete
[X] deploy
[X] diff
[ ] helm-pull
[ ] helm-update
[ ] list-images
[ ] list-targets
[ ] poke-images
[ ] prune
[ ] render
[ ] seal
[ ] validate
[ ] version

Who are you?
Maintainer of Kluctl
What do you want to do?
Currently, Kluctl does not warn the user when fields can't be removed from an object that got manually added via tools like kubectl or k9s. This is due to the second point in Using Server-Side Apply in a controller.
Kluctl should be able to guess which fields got added externally and can't be removed anymore and then warn the user about such fields.
Why do you need that?
Otherwise, the user might run into very unexpected behaviour, as a diff/deploy does not show that the object has diverged from the local declaration.

Hello! I was exploring kluctl for evaluation purposes and initially intended to pose a related question in an issue. However, I discovered that this question has already been addressed in this thread.
Is the issue focused on identifying manually added changes and then displaying them in the diff? For instance, if an object in Git is initially defined as follows:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: prometheus-pushgateway
  labels:
    app: prometheus-pushgateway

But then someone uses kubectl edit to add an additional label, making the object appear like this:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: prometheus-pushgateway
  labels:
    app: prometheus-pushgateway
    something: banana

Should kluctl be able to detect that metadata.labels.something: banana addition was made?
This would be a distinctive feature, as I haven't come across any Kubernetes deployment tool that can perform this task.

@mirsafari Yes, the long-term goal is to show all fields unknown to kluctl that were added by an external source. So your example would be covered as well. I plan to show this in diffs and in the Webui as well, as part of the drift shown on targets.
@mirsafari May I ask how you found out about Kluctl? This will help me to better understand how Kluctl is positioned and how people perceive it.

Awesome, thanks for fast reply.
I saw it maybe a year ago mentioned somewhere on Reddit, but I started looking more into it now. The two Rawkode Youtube sessions are very,very good for introduction, so this is where I got the most information. I follow the Rawkode channel, so probably they poped up on Youtube search in the first place when I was looking for some video materials for kluctl


[test](jdbc catalog) add some jdbc query tvf test

cherry-pick #45938

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall


Compiling help: Everytime i try and open CNE it just says ''process creation failure : attrib +h.temp.''

Which target are you using?
[x] Windows
[ ] Linux
[ ] Mac
Not sure if this is a ''bug'' or a ''compiling help'' but im PRETTY SURE its a ''compiling help'' so here's the case.
I have no idea why but every single time i try to open CNE it just display this error message, i've practically tried everything, change versions, looking in the files and tried anything to fix it. But what is strange is that the first time i downloaded and tested the engine it worked perfectly, but when i tried adding a mod into the mod folder it just broke completely and this error appears everytime.

#291


Integrate the ExP framework

Hey @daytonellwanger，
I have updated the PR. Please take a look when you have time. Thanks a lot.


Add seek behaviors

Pull request
Description
This PR adds two seek behaviors required for Pillarbox Apple, as the player is also an observable object and must deal with bindings on the progress property.
Changes made

Add two modes as an enum.
No UTs have been written as internals would have been tested in a way tightly coupled to the Player implementation details. We should better test this behavior with future player states we might add (seeking, buffering). I added special dedicated tasks to the corresponding backlog item to do it at a later time.

Checklist

[x] Your branch has been rebased onto the main branch.
[ ] APIs have been properly documented (if relevant).
[ ] The documentation has been updated (if relevant).
[ ] New unit tests have been written (if relevant).
[ ] The demo has been updated (if relevant).

🌙 Available in TestFlight nightlies
The most recent state can be tested in the following builds:

Platform
Version

iOS
1.0.0 (83)

tvOS
-


test: update spec tests target to v1.4.0-alpha.3

update version which has non substantive changes for current lodestar build

:tada: This PR is included in v1.9.0 :tada:


flutter command is blocked by Waiting for another flutter command to release the startup lock... on macOS if flock is installed

Steps to Reproduce

On macOS

brew install flock
Install flutter
Run any flutter command

Root cause: The macOS version of flock is not compatible.
Logs
 ~/flutter/bin (git::stable)]flutter doctor
Waiting for another flutter command to release the startup lock...

Closing this issue as it's caused by the older version of flock on macOS. brew upgrade can fix it.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v4.12 to before v5.15.22",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ded845a781a578dfb0b5b2c138e5a067aa3b1242",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=94cd597e20ed4acedb8f15f029d92998b011cb1a"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "ded845a781a578dfb0b5b2c138e5a067aa3b1242",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "94cd597e20ed4acedb8f15f029d92998b011cb1a",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "net: ieee802154: ca8210: Stop leaking skb's\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.22 by commit 94cd597e20ed4acedb8f15f029d92998b011cb1a, it was introduced in version v4.12 by commit ded845a781a578dfb0b5b2c138e5a067aa3b1242. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000586


Updated Playwright test to be in order

This is done by giving each test a letter. Playwright runs tests alphabetically

Nice simple fix


Adapt definition lists for mobile layout

Could you provide a screenshot of what this changes? I could not seem to see any difference when testing this.

I rebased it now, so it can be seen with the latest updates.
The issue can be seen in the information security compendium
Before:

After: