Dispatch messages to the new consumer after exist consumer acknowledged all dispatched messages in Key_Shared Subscription.

Master issue #4077
Is your feature request related to a problem? Please describe.
Dispatch messages to the new consumer after exist consumer acknowledged all dispatched messages in Key_Shared Subscription.

Let me add a related issue and some discussion I and @codelipenghui had about this.

Key_Shared consumers with different subscriptions get messages for the same key in an inconsistent order
https://apache-pulsar.slack.com/archives/C5Z4T36F7/p1585527971073400

I think sticky consumers won't really solve it since consumers can be easily down in failure cases and it is pretty hard for applications to manage scalability of consumers.

How come this is marked as a feature and not a bug?
It violates the ordering guarantee of Key_Shared, and basically makes Key_Shared useless if you dynamically add consumers.
I ask, because I would like to ensure this bug gets the proper attention, and is not regarded as a nice-to-have feature.

I totally agree with @poulhenriksen .

@codelipenghui Could you share updates on this if any ?
Sorry for pushing, but I feel it is a really critical bug and Key_Shared users will have so much troubles if they just use it without doubt.

@feeblefakie We are already advancing this matter and I have drafted an approach. I will create a PIP soon.

@codelipenghui Thank you for sharing the update ! Good to know. :)

So, I think the problem here should be treated by having new consumers joining in a "paused" state.
For example, consider this sequence:

Subscriptions has c1 and c2 consumers
c3 joins. Some of the keys are now supposed to go to c3.
Instead of starting delivering to c3. We mark the current readPosition (let's call it rp0_c3) of the cursor for c3.
Any message that now hashes to c3 and that has messageId >= rp0_c3 will be deferred for later re-delivery
Any message that might get re-delivered (eg: originally sent to c1, but c1 has failed) to c3 and that has messageId < rp0_c3 will be sent to c3
When the markDelete position of the cursor will move past rp0_c3 the restriction on c3 will be lifted.

Essentially, c3 joins but can only receive old messages, until everything that was read before joining gets acked.

@merlimat
I'm not fully sure about the solution but does it guarantee key ordering ?
For example, a message X with key "A" is first consumed in c1, and another message Y with key "A" can possibly go to c3 ? (can we make sure message X is consumed and acked before message Y ?)

For example, a message X with key "A" is first consumed in c1, and another message Y with key "A" can possibly go to c3 ? (can we make sure message X is consumed and acked before message Y ?)

Yes, the above proposal takes care of that.
The important point is that X could be re-sent to c3, in case c1 fails just after c3 has joined.

@merlimat I like the approach you have mentioned.

@pouledodue @feeblefakie @merlimat  I have pushed a PR #6799 for this issue. Please help review, thanks.

Reopen it via https://github.com/apache/pulsar/issues/5819#issuecomment-644510797.

PR #7106 referred to by https://github.com/apache/pulsar/issues/5819#issuecomment-644543109 is marked as closed and merged, with milestone 2.6.0. Should this issue be closed with 2.6.0 milestone after all?

@rocketraman As discussed with @feeblefakie in slack channel, looks there are some problems with auto split mechanism. So I reopen this issue for tracking this problem. And @feeblefakie has verified consistent hash mechanism is work as expected. I will take a look at the auto split mechanism later.

@codelipenghui Should I create a open issue for this ?
I realized that I just created how to reproduce doc in my repo and shared it in only the closed issue.

@feeblefakie Yes, thanks.

@codelipenghui #7455  PTAL.

The issue will be tracked in #7455 so move the issue to release/2.6.2

It seems that #8292 has solved this problem, let us consider closing this issue.


Different background/selector for out of bounds dates

Hello.
I have a calendar displaying all days within a month and few days from previous and next month.
What I want to do is to create a DayViewDecorator that will set different background for a range of days, but if it is a day from the previous/next month I want the color to be a bit different to notify the user that this day isn't within current month.
Let's say that I have:
JUNE
27 | 28 | 29 | 30 | 1 | 2 |  3
4  |  5  |  6  |  7  | 8 | 9 | 10
...
and I have set a decorator for range 01.06 - 30.07.
I want decorator to set dark blue background for dates 27.06, 28.06, 29.06, 30.06 and light blue background for dates 01.07 - 30.07. After changing the view to july I'd like to have light blue on July's dates, and dark blue on June's dates.
How can it be achieved?

I have had the same idea! I did it by reloading decorators whenever month changed. You know, you can use onMonthChangedListener to do so.
However, reloading decorators takes some time, and visual stutter can be a pain.
Some serious fork needs to be done, to achieve this goal. I have this on my mind, hovewer not on the top priority,


fix: Correct Guidance Typo in UHD Bluray + WEB Template

Correct a typo in the HDR10+ Boost guidance on the UHD Bluray + WEB template.

@NiceTSY - this PR is failing because the French profiles have out of date Trash IDs in them - any chance you could have a look and update them in this branch?

@yammes08
Arf I was expecting to wait for the Language CFs to be fully implemented into the guide before making any update there.
Anyway, sorry for that. Let me see what I can do in the meantime so you can still push your update.

@yammes08
Arf I was expecting to wait for the Language CFs to be fully implemented into the guide before making any update there.
Anyway, sorry for that. Let me see what I can do in the meantime so you can still push your update.

Not to worry - this isn't an urgent update by any means!

Closing as another PR is taking precedence.


OverconstrainedError with Safari

OverconstrainedError: Invalid constraint

This was equivalent to "NotFoundError: Requested device not found" on Chrome.


Adding KubeShop as member

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you added your SVG to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~15 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No cached entry, and kubeshop (member) has issues with logo: kubeshop.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/kubeshop.svg'
No cached entry, and Valve Software (member) has issues with twitter: https://twitter.com/valveoficial, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
No cached entry, and Sosivio (member) has issues with twitter: https://twitter.com/SosivioLtd, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
Empty twitter for Rancher Federal (member): https://twitter.com/rancherfederal
Empty twitter for Banzai Cloud (KCSP): https://twitter.com/banzaicloud
Empty twitter for StackStorm: https://twitter.com/Stack_Storm
Item kubeshop (member) has no image_data


Feature/etherspot transaction buidler widget

Description
This PR includes Etherspot SDK (https://etherspot.io) implementation to enable cross-chain staking using Etherspot Buidler (https://buidler.etherspot.io) depdendency.
It enables users to stake Klima using any asset on supported chains and receive sKlima on Polygon.
It was QA tested on our end, but feel free to test it once again on your end.

Changes

Before
After

-->
Checklist

[x] I have run npm run build-all without errors
[x] I have formatted JS and TS files with npm run format-all

Approved app preview deployment will take a closer look soon.


int not passed from cli as int

if you pass an int to the cli it is interpreted as a string

fixed


[Dialog] Tooltip inside Dialog

Hey,
I have an issue with IconButton inside Dialog. Tooltip is hidden outside dialog.
Demo:

Any chance you can look through the dom and see what values for z-index the tooltip, dialog and the overlay have? This is magic :scream:

z-index
tooltip: 3000
dialog: 1500
overlay: 1500

now that's weird O.o @oliviertassinari What do you think can cause this? RenderToLayer maybe?

@alitaheri The Tooltip isn't using the Popover. That remembers me old issue before using the Popover.
I'm wondering if there is a overflow: hidden somewhere and if we can remove it to fix this issue.

No i didn't mean popover, I meant rendertolayer which dialog uses

I don't think so RenderToLayer isn't supposed to apply any style. What about https://github.com/callemall/material-ui/blob/master/src/dialog.jsx#L135-L136?

I'm wondering if there is a overflow: hidden somewhere and if we can remove it to fix this issue.

Yeah... I kinda forgot about overflow O.o Do we really need that?!

I believe that we can remove the overflowX.

Yeah, I guess :+1:

@mthenw Do you want to submit a PR?

sure, in a moment :)

This is still happening to me, no matter how high the z-index is.
Version<EMAIL_ADDRESS>


TypeError: Cannot read property 'pid' of null

Every time an application has been launched by desktop there is a script error on syslog:
Mar 22 12:20:42 5cd2512wtv kwin_wayland[4301]: file:///home/user/.local/share/kwin/scripts/kwin-system76-scheduler-integration/contents/code/main.js:5: TypeError: Cannot read property 'pid' of null
KDE Plasma 6.0.2

Thanks for your report! This looks like issue https://github.com/maxiberta/kwin-system76-scheduler-integration/issues/1 - but that's with 5.27.11. For KDE 6, this simple patch appears to work for me. Would you mind trying it? https://github.com/maxiberta/kwin-system76-scheduler-integration/commit/093a269670275feaa240d02c712f1ec8b812fd80. It can be applied on ~/.local/share/kwin/scripts/kwin-system76-scheduler-integration/contents/code/main.js, then restart it at System Settings -> KWin Scripts . Thanks!

I'm already using the "windowActivated" function

Hi @maxiberta ,
enabling the debug console I can see the getting of pid successfully. Probably there are many conditions when the script fail. A check on pid != null should be done :)
Thank you for your work.


xarray friendly ef.create_mask

Right now masks are created by hardcoded index selections. It would be nice to instead have a more versatile selection, for example depending on explicit latitude and longitude values.

For now only France mask is supported

It should be possible somehow to get country borders...


New MSVC projects. Using PropertySheets

pugixml/msvc/
Pull request
Compare This branch is 44 commits ahead of zeux:master.
Latest commit d2bae88 37 minutes ago @sevlat sevlat Added readme.txt
..
props 	Msvc14: IntDir and OutDir changed a little 	4 hours ago
.gitignore 	Added .gitignore to msvc directory 	a day ago
pugixml.props 	pugixml.props: support custom user property sheet 	4 hours ago
pugixml.s.props 	Support props for static /dynamic RTL 	23 hours ago
pugixml_vs2010.sln 	Updated Msvc10 solution 	3 hours ago
pugixml_vs2010.vcxproj 	Msvc10: cosmetic changes. Remove unnecessary empty tags 	7 hours ago
pugixml_vs2010_static.vcxproj 	Msvc10: cosmetic changes. Remove unnecessary empty tags 	7 hours ago
pugixml_vs2012.sln 	Added msvc11 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2012.vcxproj 	Added msvc11 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2012_static.vcxproj 	Added msvc11 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2013.sln 	Added msvc12 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2013.vcxproj 	Added msvc12 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2013_static.vcxproj 	Added msvc12 solution and projects: converted from Msvc10 	2 hours ago
pugixml_vs2015.sln 	added msvc2015_static.vcxproj (just by copying from msvc2015.vcxproj) 	4 hours ago
pugixml_vs2015.vcxproj 	msvc14: Cosmetic changes, removed empty tags 	4 hours ago
pugixml_vs2015_static.vcxproj 	msvc14: import pugixml.s.props to static project 	4 hours ago
readme.txt 	Added readme.txt 	37 minutes ago
readme.txt
A new approach to MSVC build system, using PropertySheets.
This approach is based on MsBuild ver 4, so it concerns only latest MSVS versions (starting from Msvs2010).
New projects, solutions and props files are located in msvc directory.
All properties and item metadata, such as compiler and linker settings are declared in PropertySheets (*.props files).
There are dozen props *.files in msvc/props directory. Each *.props file contain settings for some build configuration. The most part of *.props files are suitable for all VisualStudio versions.
Each project file should import pugixml.props file (located in msvc directory). Projects for static RTL should also import pugixml.s.props (before pugixml.props).
Advantages of this approach:

Each parameter stores in only one plase. So it is easier to explore the structure of projects. It's easier to change some setting.
Project files became smaller and easier to read.
It's easier to create new vcxproj files based on existing project. For example, projects for Msvc11 and Msvc12 was created using Msvc10 project.

Visual studio has a nice instrument for exploring and editing property sheets: Property Manager. If it not visible, turn it on by using menu View/OtherWindows/PropertyManager.
And yet another new feachure. User can customize building process by inserting his own property sheet into project structure (without modifying any file from pugixml directory).
For example:
MsBuild.exe /p:PUGI_Props_External=d:\MyProps.props pugixml_vs2010.vcxproj
Best regards,
Taymanov Sergey,
Moscow.

I'm sorry to say this given the amount of work here, but I don't see the value in merging this.
I played with this project layout a bit and as a long-time VS user it's confusing - I have to change the way I think about project settings from "need to change something - open properties" to "need to change something - open an obscure widget and guess which vsprops the setting is in".
In addition, this adds quite a few files to pugixml distribution.
Now, I'm not convinced of the advantages you listed either. I can see the settings being specified in one place as a good thing if you read/edit vcproj files by hand (I sometimes do), but this can be achieved by editing the vcproj by hand - MSBuild is fully capable of specifying settings shared by all platforms/configurations once and I'm somewhat surprised VS does not generate the file like that.
Something else to keep in mind - VS projects for pugixml are pretty much always trivial "create a default project and add 3 files to that". There is never any complicated setup - there is no real reason to change settings in the distributed vcproj files. Projects for new VS versions can be created by auto-upgrading the previous version or just creating one from scratch.
Ideally I'd like to auto-generate all project files - the existing project files (except VS2015) are all generated by premake - but if premake does not support all versions it's not hard to do it manually.
Finally, the user-extendable projects are interesting, but my understanding is that you can only do this at a command-line level - you can't add settings without modifying projects if you want to open them in Visual Studio. I'm not sure that custom MSBuild command-line settings for project files are a great idea - they make it very easy to lose build consistency.


Add CNI plugin dryrun test.

fix https://github.com/istio/istio/issues/33772

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@bianpengyuan: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

lint_istio
cad1929ce2d92973a55bd3cd0c7000dc92e5c7ba
link
/test lint_istio

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

@bianpengyuan: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Rerun command

lint_istio
cad1929ce2d92973a55bd3cd0c7000dc92e5c7ba
link
/test lint_istio

gencheck_istio
cad1929ce2d92973a55bd3cd0c7000dc92e5c7ba
link
/test gencheck_istio

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

Put a hold in case you want to use the diff package, if not feel free to merge

Surprised order is not stable, what part changes?
On Tue, Jul 13, 2021, 4:03 PM Pengyuan Bian @.***>
wrote:

@.**** commented on this pull request.
In cni/pkg/plugin/plugin_dryrun_test.go
https://github.com/istio/istio/pull/33992#discussion_r669166056:

	}

	os.Setenv(dryRunFilePath.Name, outputFilePath)

	_, _, err := testutils.CmdAddWithResult(

		sandboxDirectory, ifname, []byte(cniConf), func() error { return CmdAdd(args) })

	os.Unsetenv(dryRunFilePath.Name)

	if err != nil {

		t.Fatalf("CNI cmdAdd failed with error: %v", err)

	}

	generated, err := ioutil.ReadFile(outputFilePath)

	if err != nil {

		log.Fatalf("Cannot read generated IPTables rule file: %v", err)

	}

	generatedRules := getRules(generated)

	if os.Getenv("REFRESH_GOLDEN") == "true" {

Thanks for the pointer! I used the RefreshGolden function, Compare
function is a bit tricky because IPTables output tables order is not stable
so need a bit of custom logic. IPTables output is small and I think we can
live with raw got/want output.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/33992#discussion_r669166056, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXMQ6HLGYP457PULFJDTXTA3ZANCNFSM5AIJK6MQ
.

@howardjohn it is the order of tables. Some times nat table comes first, sometimes raw table comes first. IPtables logic internally has a map for tables, which makes output order of tables not stable.

Interesting, thanks
On Tue, Jul 13, 2021, 4:10 PM Pengyuan Bian @.***>
wrote:

@howardjohn https://github.com/howardjohn it is the order of tables.
Some times nat table comes first, sometimes raw table comes first. IPtables
logic internally has a map for tables, which makes output order of tables
not stable.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/33992#issuecomment-879463923, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXJINGIHRUREB5YADADTXTBWNANCNFSM5AIJK6MQ
.


changed the documentation link from v1.docusaurus.io(deprecated) to docusaurus.io(latest)

Pre-flight checklist

[x] I have read the Contributing Guidelines on pull requests.

Motivation
In introduction to docusaurus blog the hyperlink is directed to v1.docusaurus.io documentation which is deprecated so I changed it to docusaurus.io (v2 which is latest documentation on latest verion).
Link-
https://docusaurus.io/blog/2017/12/14/introducing-docusaurus#resources (-read our documentation)

This is the blog post for the V1 release, so it surely should link to the V1 docs for full historical context. Everything else about this post is outdated anyway.


Content change for ENIC description

Context
When a candidate needs to show how their international qualifications compare to their UK ones, they get a certificate from a particular agency. The agency used to be called the National Academic Recognition Information Centre (NARIC) but post Brexit it will be called UK ENIC
Work has been happening with the Digi comms team to change content on the start page (which is owned by GOV.UK). In the process of getting it approved, a last minute change was made which puts the GOV.UK content out of synch with the apply service.
Changes proposed in this pull request
Changing "UK ENIC (the UK agency for...)" to "UK ENIC (the UK agency that recognises...)"

Before
After

Guidance to review

Link to Trello card
https://trello.com/c/zAG23BXm/3356-enic-content-change-in-apply
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] This code does not rely on the addition/removal of Azure config environment variables in the same Pull Request
[ ] New environment variables have been added to the Azure config

Looks good - thanks @JR-G


nextcloud-desktop crashes on startup

Describe the bug
nextcloud (that is nextcloud-client) crashes on startup
$ nextcloud --version
Nextcloud version 3.2.1git
Git revision 5901a0f9835261a640cd8bc9397bcd3068f2acd8
Using Qt 5.15.2, built against Qt 5.15.2
Using Qt platform plugin 'xcb'
Using 'OpenSSL 1.1.1k  25 Mar 2021'
Running on NixOS 21.05 (Okapi), x86_64

Running in KDE plasma 5.21. Maybe related to #126929 .
To Reproduce
Steps to reproduce the behavior:

nextcloud in your terminal, results in segfault.

Notify maintainers
@caugner
Metadata

system: "x86_64-linux"
host os: Linux 5.4.125, NixOS, 21.05.git.1f91fd10406 (Okapi)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.3.12
channels(m): ""
channels(root): "nixos-20.03.3269.f05c380a51d"
nixpkgs: /var/src/nixpkgs

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

Just updated to latest stable (246502ae2d5ca9def252abe0ce6363a0f08382a7), issue still exists.

Happens for me as well, including a stacktrace from gdb.
#0  0x00007fffef7561d0 in QQmlType::QQmlType(QQmlType const&) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#1  0x00007fffef7bc6df in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#2  0x00007fffef7bf2b4 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#3  0x00007fffef7bf644 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#4  0x00007fffef7bf644 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#5  0x00007fffef7c0162 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#6  0x00007fffef7bae90 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#7  0x00007fffef70c1c5 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#8  0x00007fffef7122e2 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#9  0x00007fffef6ffd65 in QQmlDataBlob::tryDone() () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#10 0x00007fffef75ee75 in QQmlTypeLoader::setData(QQmlDataBlob*, QQmlDataBlob::SourceCodeData const&) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#11 0x00007fffef75f5c2 in QQmlTypeLoader::setData(QQmlDataBlob*, QString const&) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#12 0x00007fffef760970 in QQmlTypeLoader::loadThread(QQmlDataBlob*) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#13 0x00007fffef760d9c in QQmlTypeLoader::load(QQmlDataBlob*, QQmlTypeLoader::Mode) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#14 0x00007fffef761640 in QQmlTypeLoader::getType(QUrl const&, QQmlTypeLoader::Mode) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#15 0x00007fffef711366 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#16 0x00007fffef713f6d in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#17 0x00007fffef75f12a in QQmlTypeLoader::setData(QQmlDataBlob*, QQmlDataBlob::SourceCodeData const&) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#18 0x00007fffef75f5c2 in QQmlTypeLoader::setData(QQmlDataBlob*, QString const&) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#19 0x00007fffef760970 in QQmlTypeLoader::loadThread(QQmlDataBlob*) () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#20 0x00007fffef71bafd in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#21 0x00007fffef7d87df in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#22 0x00007fffef7d8fb2 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#23 0x00007fffee52474f in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Widgets.so.5
#24 0x00007fffed5a93ca in QCoreApplication::notifyInternal2(QObject*, QEvent*) () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#25 0x00007fffed5ac411 in QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#26 0x00007fffed601ec3 in ?? () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#27 0x00007fffed99f5ab in g_main_context_dispatch () from /nix/store/z3lf066csyma75624syd2vcklry8vqhd-glib-2.68.2/lib/libglib-2.0.so.0
#28 0x00007fffed99f858 in g_main_context_iterate.constprop () from /nix/store/z3lf066csyma75624syd2vcklry8vqhd-glib-2.68.2/lib/libglib-2.0.so.0
#29 0x00007fffed99f90f in g_main_context_iteration () from /nix/store/z3lf066csyma75624syd2vcklry8vqhd-glib-2.68.2/lib/libglib-2.0.so.0
#30 0x00007fffed601520 in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#31 0x00007fffed5a7d7b in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#32 0x00007fffed3b5eeb in QThread::exec() () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#33 0x00007fffef7d8475 in ?? () from /nix/store/xbcazjfz6fdq1i780d8bnciwzmhndwyb-qtdeclarative-5.15.2/lib/libQt5Qml.so.5
#34 0x00007fffed3b71b3 in ?? () from /nix/store/lxjazf4faixxfcyr4r045pfx2sxgaf6x-qtbase-5.15.2/lib/libQt5Core.so.5
#35 0x00007fffecdd3e9e in start_thread () from /nix/store/ikl21vjfq900ccbqg1xasp83kadw6q8y-glibc-2.32-46/lib/libpthread.so.0
#36 0x00007fffecee949f in clone () from /nix/store/ikl21vjfq900ccbqg1xasp83kadw6q8y-glibc-2.32-46/lib/libc.so.6

Are you using more than one display/monitor?

Just updated to latest stable (246502a), issue still exists.

Does this mean you're running 3.2.2 now?

Yes, I am using two monitors. If that's the version.that is installed on that stable commit, then probably yes.

I have a single monitor, and Nextcloud crashes for me. Using KDE Plasma as well.

If that's the version.that is installed on that stable commit, then probably yes.

I checked and release-21.05 still has nextcloud-client 3.2.1.
The issue could very well be fixed in 3.2.2, so you could try nixos-unstable (which has 3.2.2) to see if the issue is resolved there.

Using Nextcloud from channel:nixos-unstable doesn't make a difference.

Could you try if (backing up and) deleting the folders mentioned in #121594 solves the segfaults for you?

Today I finally switched to NixOS 21.05 myself, and also experienced a segfault when starting nextcloud. However, rm -rf ~/.cache/Nextcloud fixed it for me.

Worked for me, too.


CachetHQ Status Page

https://github.com/CachetHQ/Cachet
Good to have it in EE. I think the way EE works, it can setup everything automatically making it even more appealing.

@rahul286 I'd set up CachetHQ long back I guess in 2014
https://miteshshah.github.io/linux/centos/how-to-configure-status-pages-using-cachet/
CachetHQ is Good if we want to schedule downtime.
If you are just looking for display all Webserver/API/ uptime i think best way is use UptimeRobot status page with CNAME so https://status.easyengine.io display EE Website API and all other on one single page without much developer work :)
https://blog.uptimerobot.com/introducing-public-status-pages-yay/

CachetHQ API https://docs.cachethq.io/reference can record incidents I think.
I am not sure exactly how we are going to use it but this is not for EE website. Idea is to configure create a status page per site that EasyEngine creates. So as an example for a WordPress site, a status page can list incidents such as php, mysql, cron etc failures.

@mbtamuli will this be needed in docker world or we have alternatives.

Will be handled differently in v4 because of docker.


Test scripts use different ways to define the default Test Event Trigger Key

In the current version of the Test Scripts there are two different ways to define the default Test Event Trigger Key:

0x000102030405060708090a0b0c0d0e0f
This is used by the function send_test_event_triggers() in src/python_testing/matter_testing_support.py which is always called without the key parameter. Thus the default key 0x000102030405060708090a0b0c0d0e0f gets used or the value of the global parameter enableKey if it exists

0x00112233445566778899aabbccddeeff
This is hard-coded as a default in various YAML scripts like for DGGEN-2.1, DD-3.2, DD-3.6, DD-3.8, etc.

IMHO we should at least unify the hard-coded default keys to one value for all scripts to avoid confusion on the tester side.

@akrithika27 thoughts on these Test Event Triggers ?

@akrithika27 did you get a chance to look at this ?


Schema Diagramming Tool

Hi,
Is there an automatic way to generate an ORD / ERD from the waterline models ? Something on the lines of SchemaSpy ?
I am looking for a schema diagraming tool to visualise the Models which I created for my SailsJS app.
If there is such a thing!
Thanks In advance!

I've never heard of such a thing, but I think it would be great.  @tjwebb do you have any idea?

Yea, like a visual entity-relationship mapping tool. I think this would be really cool. It might be worth asking the folks at treeline.io if they have anything in the works.

One way i am trying to work around to getting something going is to utilise the ability to change adapters, here is what I did :

Installed MySql on my machine,
Changed the adapter from my sails application from mongodb to mysql
Ran schemaspy on mysql.

Schema Spy Re-verse engineers diagrams like this : http://schemaspy.sourceforge.net/sample/relationships.html
But I have run into a hickup,

when I lifted the sails application, it created the tables in mysql but it did not create the relationships (PK , FK etc), maybe there is a way to enforce that ?

So I was able to get the table description out, but the real nice stuff around reverse engineered relationships is missing.
What do you guys think ?

@gunish there's a feature request for this in the sails-mysql project: https://github.com/balderdashy/sails-mysql/issues/172
Currently you do have to add FK contraints manually, which I think is a bit of a shortcoming.  Would love a PR!

Closing this as inactive, please re-open if you feel this hasn't been fully answered. Thanks

Sorry just noticed the comments from @devinivy , yes thats cool, for now in my project, we resorted to doing it manually, but there should be an easy win if we were able to get a a simple drawing library to perform some static code analysis and draw out an ERD. schemaspy is old and very good, we want something basic and in the browser.


Doesn't work on Asia server

It doesn't work on Asia server
Is there any solution?
US and EU servers work well.

Not a script issue, Blizzards auth servers crapped out again.
Nothing I can do sorry but wait for them to fix. It happens sometimes and they're painfully slow at resolving, this is the longest I've seen a region down for.

I will use the Battle.net launcher until the server down is resolved.
thank you for the reply.


Missing required forwardable

@ptrykov what was breaking, without this explicit require?

Closing this one, there was a problem with the configuration of my dev machine 🤷♀


Support Infoflow(ruliu) notification.

@hardy4yooz thanks for contributing this feature, could you please address the failed checks so we can get it merged?

Hi, since this pr has not been updated for several month, I will close it. Please reopen it when necessary.


enable multiplex stream muxer

This allows js ipfs nodes to connect to go-ipfs nodes, which paves the way for browser openbazaar implementations 😛 (spdy stream muxer doesn't work in js ipfs at the moment) . multiplex is already enabled by default in ipfs 0.4.7 in the daemon (ie when run from command line), but is not enabled in default repo config when using ipfs as a go import yet.

Coverage remained the same at 29.547% when pulling c47e3eff44ac127dddb079ac7f9e41e67b825ed4 on jackkleeman:patch-4 into 8545e5a21694b461f6ae1a379990e75a67fb5d91 on OpenBazaar:master.


Expose 'metadata' field in transcription results

Proposed changes

Provide a detailed description of the change or addition you are proposing

Add the metadata field to transcription results (i.e. ASR API request responses).
Current streaming response via websockets:
{
    "type": "Results",
    "channel_index": [
        0,
        1
    ],
    "duration": 4.409999,
    "start": 14.56,
    "is_final": true,
    "speech_final": true,
    "channel": {
        "alternatives": [
            {
                "transcript": "And finally, here's my third utterance.",
                "confidence": 0.99952614,
                "words": [
                    {
                        "word": "and",
                        "start": 16.6,
                        "end": 17.0,
                        "confidence": 0.9989612,
                        "punctuated_word": "And"
                    },
                    ...
                    {
                        "word": "utterance",
                        "start": 18.2,
                        "end": 18.68,
                        "confidence": 0.99990237,
                        "punctuated_word": "utterance."
                    }
                ]
            }
        ]
    },
    "metadata": {
        "request_id": "9378f55d-513c-4d5f-9a55-3ca7530c084b",
        "model_info": {
            "name": "general",
            "version": "2023-02-22.3",
            "arch": "base"
        },
        "model_uuid": "96a295ec-6336-43d5-b1cb-1e48b5e6d9a4"
    }
}

The final metadata fields (request_id, model_info, model_uuid) do not seem to be currently exposed via the .NET SDK.
Context

Why is this change important to you? How would you use it? How can it benefit other users?

This change allows users to retrieve the full metadata from their request responses, especially the request ID, which is essential for looking up past requests and debugging failures with Deepgram support.
Possible Implementation

Not obligatory, but suggest an idea for implementing addition or change

Add the metadata field to the LiveTranscriptionResult and PrerecordedTranscriptionResult.
Other information

Anything else we should know? (e.g. detailed explanation, related issues, links for us to have context, eg. stack overflow, codepen, etc)

Will be taking a look at this!

This needs to be reimplemented. The merge of the v4-alpha branch in PR https://github.com/deepgram/deepgram-dotnet-sdk/pull/188 ended up removing this because of the refactor.

Merged PR https://github.com/deepgram/deepgram-dotnet-sdk/pull/205. Closing.


Add register event && Add friendship && Add FileBox

Add register event.
Add friendship.
Add FileBox.
In order to solve the circular reference, I newly added the impl directory as the implementation of the interface, and write the interface to the wechaty directory. This should not be the best solution, but it can solve the current problem.

Great to see all of us approved this PR to be merged!
I'd like to suggest that:

Please feel free to merge a PR after it has been approved and you believe everything is OK with it.
The best practice is to split the features to separate PRs instead put everything into one. So please create different branches for each feature, so that we can make it easy to discuss about it in different threads (Issues/PRs).

Thank you very much, I hope you all have a great weekend!

Can this pr be merged

@dchaofei Of course, please feel free to help yourself to merge this pull request because we have already two approvements on it!

I don't have merge permission. 😊

Perhaps GitHub does not permit the PR author to merge the PR by himself?
Ok, I'll merge this for you, and we can ask other contributors who have the write permission to help to merge in the future!


Jenkins: add git and openssh-client

Added git to the Jenkins image. This change is necessary as the GitHub push webhook wasn't functioning without the presence of git in the image.

this seems to align with official jenkins image https://github.com/jenkinsci/docker/blob/d92706534dc2f8b8f28967cae5de4e113b997da1/21/alpine/hotspot/Dockerfile#L23

As part of a Chainguard effort to reduce code duplication and provide higher quality images, we have stopped accepting pull requests for changes to images in this repository. Due to this, we are closing all pull requests today in order to start fresh.
If you think this pull request was closed in error, please feel free to re-open this pull request or open an issue and we will try to help you out. Thank you!


Feature/automate netvar testing

@zain-mecklai hey I did a bunch with MultiInstanceHelper here including adding a new helper method and wanted to get your thoughts

@zain-mecklai hey I did a bunch with MultiInstanceHelper here including adding a new helper method and wanted to get your thoughts

@mattwalsh-unity As I read through this it is clear to me that I don't understand enough to be able to provide any kind of useful feedback. I'll continue to work on developing my knowledge and understanding and then will be in a better position to provide feedback.


feat: prebuilt binaries with Ledger support

Closes https://github.com/celestiaorg/celestia-app/issues/2729
This PR removes checksum signing because https://github.com/goreleaser/goreleaser-cross/issues/58.
Testing
The assets on https://github.com/rootulp/celestia-app/releases/tag/v1.0.0-rc52

OS
Platform
Ledger support
Notes

Darwin
arm64
✅

Darwin
amd64
✅

Linux
arm64

Linux
amd64

I think this works b/c I get this error instead of an error claiming Ledger is unsupported.

I get this error instead of an error claiming Ledger is unsupported.

 I got the same but then it worked once I unlocked my ledger

I'll hold off on merging until @evan-forbes tests on Linux. You should be able to test via:
./celestia-appd keys add ledger-1 --ledger

Out of curiosity which architecture is your Linux machine? arm64 or amd64?


WebGL2: sampler

Part of WebGL2 to add.
https://developer.mozilla.org/en-US/docs/Web/API/WebGL2RenderingContext/createSampler

Closed by https://github.com/webmixedreality/exokit/pull/821, which includes this support


[SPARK-47500][PYTHON][CONNECT][FOLLOWUP] Restore error message for DataFrame.select(None)

What changes were proposed in this pull request?
the refactor PR https://github.com/apache/spark/pull/45636 changed the error message of DataFrame.select(None) from PySparkTypeError to AssertionError, this PR restore the previous error message
Why are the changes needed?
error message improvement
Does this PR introduce any user-facing change?
yes, error message improvement
How was this patch tested?
added test
Was this patch authored or co-authored using generative AI tooling?
no

Merged to master.


List run time dependencies of agent

I'm leaving this here because my instrumentation stopped when I packed my app into a small docker image, leaving build-time dependencies out. In my case, the ca-certificates package was missing (obvious, in hind-sight...)

This dependency was automatically pulled in by the build-time dependencies of the agent, which is why the issue did not apprear previously.


Add design note of flow testing extensions & GUI testing

This PR adds plugin extensions and GUI testing function to the flow testing design notes.

Is this a proposal to add UI testing to the existing flow tester plugin that is still being developed, or to create another plugin that extends the flow tester?
I am concerned that adding UI testing to the Flow Tester plugin would make it much larger and more complicated. For most users that want to test flows, this extra size and complexity would not be needed.
I believe we still have a lot of work on the Flow Tester to do - (which I acknowledge it held up by me at the moment) so I would also be concerned about trying to add more complexity before the core is done.
If this was proposed to be a second plugin that can be installed alongside the core Flow Tester to add-in UI testing capabilities, I think that would be better.

Sorry for the late reply.

If this was proposed to be a second plugin that can be installed alongside the core Flow Tester to add-in UI testing capabilities, I think that would be better.

Yes, it is a proposal as a plugin that can add functions to the Flow Tester plugin.
Users can use the UI testing feature by installing the UI testing add-on for the Flow Tester as needed.


Break README.md apart into docs/

Pull Request Test Coverage Report for Build 68

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

Totals

Change from base Build 65:
0.0%

Covered Lines:
144

Relevant Lines:
212

💛  - Coveralls


Kling-Gupta efficiency and percentage bias

I would like the following metric, statistical test or data processing tool to be considered for addition to the scores repository
In hydrology, the Kling-Gupta Efficiency (KGE) and percentage bias (PBias) are commonly used metrics. Including these metrics would enhance the appeal of the scores package to a broader hydrological audience.
Please provide a reference that describes the metric, statistical test or data processing tool
KGE: Gupta, H.V., Kling, H., Yilmaz, K.K., Martinez, G.F., 2009. Decomposition of the mean squared error and NSE performance criteria: Implications for improving hydrological modelling. J. Hydrol., 377(1-2): 80-91.
PBias: https://search.r-project.org/CRAN/refmans/hydroGOF/html/pbias.html

Thanks very much for raising those. I'd be happy to put both of those on our to-do list. Let us know if there is a particular deadline to consider and if we can accommodate it we will. Also, if it's something you would be interested in doing, we would welcome code contributions.
If I have understood correctly, percentage bias is the multiplicative bias of the error term, as a whole percentage number (i.e. 1-100 rather than 0-1). As such I would probably be inclined to implement it as percentage_error_bias (rather than percentage_bias) to avoid confusion. Does this seem reasonable to you?
In terms of our existing metrics, multiplicative bias is somewhat similar to percentage bias (https://scores.readthedocs.io/en/stable/api.html#scores.continuous.multiplicative_bias) - is that something you might use in the meantime?

Thanks very much for raising those. I'd be happy to put both of those on our to-do list. Let us know if there is a particular deadline to consider and if we can accommodate it we will. Also, if it's something you would be interested in doing, we would welcome code contributions.
If I have understood correctly, percentage bias is the multiplicative bias of the error term, as a whole percentage number (i.e. 1-100 rather than 0-1). As such I would probably be inclined to implement it as percentage_error_bias (rather than percentage_bias) to avoid confusion. Does this seem reasonable to you?
In terms of our existing metrics, multiplicative bias is somewhat similar to percentage bias (https://scores.readthedocs.io/en/stable/api.html#scores.continuous.multiplicative_bias) - is that something you might use in the meantime?

@tennlee: Thank you for your prompt reply and for considering our request to implement new matrices.
There is no strict deadline for implementing these metrics in scores, as we will use our verification scores until then. I am interested in contributing code.
Yes, percentage bias is the multiplicative bias of the error term (additive bias/mean(obs) * 100). Regarding naming,  I do not mind with percentage_error_bias or percentage_additive_bias.

@durgals If you're happy to contribute code, that would be warmly appreciated and would definitely speed things up. I'd suggest starting with reading though the following implementations (these should link directly to our code in GitHub for additive and multiplicative bias):
https://github.com/nci/scores/blob/b57325318b698e17a970a448f96f3adfe13f0eee/src/scores/continuous/standard_impl.py#L244
https://github.com/nci/scores/blob/b57325318b698e17a970a448f96f3adfe13f0eee/src/scores/continuous/standard_impl.py#L293
We have a contributor's guide available here : https://scores.readthedocs.io/en/stable/contributing.html
There is also a pull request checklist here: https://github.com/nci/scores/blob/develop/.github/pull_request_template.md
Those extra docs can seem lengthy and maybe a bit overwhelming but I wouldn't let that stop you - we'll happily help out along the way with anything and don't expect PRs to be perfect first time. I'd suggest making a fork of the repo, getting started, and letting us know how that goes.
If percentage_bias goes well, we can then move on to KGE.
How does that sound?

In terms of what to name the method in the module, I can think of:

error_bias_percentage
percentage_error_bias

Either seems fine to me. I might ask @nicholasloveday or @rob-taggart to put an opinion also if it's important to them

Thank you for the links to the implementations and the contributor's guide.
I'll fork the repository, begin working on the percentage bias, and reach out if I have any questions or need further assistance.
Regarding the naming, since percentage bias is a modification of additive_bias (additive_bias/mean(obs)), I prefer additive_bias_percentage to error_bias_percentage.

One thing to think about is whether pbias (or whatever it is renamed as) should fit within the additive and multiplicative bias tutorial or if it should fit in its own tutorial. My preference is to extend the existing bias tutorial with pbias rather than create a separate one. https://github.com/nci/scores/blob/develop/tutorials/Additive_and_multiplicative_bias.ipynb

Hi @tennlee , I was testing the additive_bias_percentage function, but it failed due to a precision error (see below).  The expected value is [-50,  -100., (0.5/3+0.5/3)/(-0.5/3)*100], which is [-50,  -100., -200.]. The test failed for both values due to precision issues with the -200.0 value. Then I used xr.testing.assert_allclose(result, expected,rtol=1e-10, atol=1e-10)  instead of xr.testing.assert_equal and the test passed. Is this ok to use assert_allclose?

Hi @durgals - we have used xr.testing.assert_all_close several times in the repository (e.g., https://github.com/search?q=repo%3Anci%2Fscores assert_allclose&type=code)
I think it is fine to use to deal with floating point precision issues.

In addition, while it's totally fine to wait until you're ready to create the pull request, you are also very welcome to create a pull request while your code is still under development. Our convention is to mark such things with (under development) at the start of the name of the pull request. Some developers find this helpful so they can ask questions about particular parts of the code as they go, but it's totally up to you whether you'd like to work that way. I just thought I'd mention it.

Hi @durgals. While the PR for percent bias is still undergoing review, it's looking really good. If you feel motivated, you would be welcome to open a PR for KGE in parallel rather than feeling like you need to wait. It's up to you, either way is fine, but you obviously have the hang of things so it would be reasonable to proceed with confidence.

Thanks, @tennlee. Yes, I've considered creating another KGE branch while percent bias is under review. I actually have working code; the only remaining task is to move it to scores and conduct rigorous testing.

@durgals  I am going to close this issue since it contains general discussions and a way forward is now agreed. Please create new issues for each new score you would like included.


Transition to new logo

The use of the Frictionless logo(s) is still not 100% consistent across the different tools pages, with the old logo still being used here and there. Here's a recap of the current status (check when the new logo is used):

frictionlessdata.io

[ ] Favicon
[x] Main icon

Framework

[ ] Favicon
[x] Main icon

Libraries

[ ] Favicon
[x] Main icon

Components

[ ] Favicon
[ ] Main icon

Repository

[ ] Favicon
[ ] Main icon

Livemark

[ ] Favicon
[ ] Main icon

Standards

[ ] Favicon
[x] Main icon

Please preserve this line to notify @lwinfree (lead of this repository)

Noting that we're working with Lissett on this; it is currently blocked, but will be unblocked in April


Inochi2D support

I see you're aiming to add Live2D support, would you be able to also look into supporting Inochi2D? Its a FOSS alternative to Live2D that's currently being developed, and the dev told me she has a rough ETA of a working version around summer of this year, 2021. Current progress is avalible on github already, and its all written in D.

I can definitely look into supporting Inochi2D but no promises until they have their first release. It might be a better solution than Live2D since Live2D isn't FOSS and seems to have a pretty restrictive license (i.e. I might not be able to release an integration without a license agreement?)

I can definitely look into supporting Inochi2D but no promises until they have their first release. It might be a better solution than Live2D since Live2D isn't FOSS and seems to have a pretty restrictive license (i.e. I might not be able to release an integration without a license agreement?)

If I'm not mistaken, I believe Live2D does require a paid license to use the SDK in programs

Going off of their release license page, I think this project would fall under the 'One-time Plan' category, as the program is free with no in-app purchases. I will have to confirm later, as I'm still working through VRM-related things.

Will add that the spec is seeing good progress, especially on the side of the rigging application. While I did not make it for summer 2021. Currently the aim is to have 1.0 out before April 2022


Add Art of Illusion

An open-sourced Java application for 3d modeling and rendering.

Failing the Sourceforge URL format check.

Sorry, I of all people should've know better. Fixed.

Thank you for the contribution. It was merged directly as 9a138cd9022b5975ba69fec4459b9e350ca655eb to keep commit history cleaner. Your contribution is still credited to you.


[Mbed-OS 5.13 OOB] mbed help command doesn't give ARMCC details

mbed toolchain --help

Set or get default toolchain
This is an alias to "mbed config [--global] toolchain [name]"

positional arguments:
  name                 Default toolchain name. Example: ARM, GCC_ARM, IAR

optional arguments:
  -h, --help           show this help message and exit
  -G, --global         Use global settings, not local
  -S, --supported      Shows supported matrix of targets and toolchains
  -v, --verbose        Verbose diagnostic output
  -vv, --very_verbose  Very verbose diagnostic output

suggesting to add ARMC5 and ARMC6 descriptions for the 'help' page.
also, mbed_settings.py doesn't include ARMC6_PATH, which might be confusing.

Internal Jira reference: https://jira.arm.com/browse/MBOCUSTRIA-1272

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.


Common code could be refactored from head and tail

The head and tail programs have similar command-line arguments, error handling, and display styles. These common pieces could be refactored into a common module.

I would advise against this. Better to have a better test rig for Clap grammars on all coreutils.
I would like to see more formal specs on the grammars:
https://github.com/clap-rs/clap/blob/a4b9bfc97b0c8e688f8e5745326684192c2e6e6d/tests/unique_args.rs#L20
I have yet to see a good tool that emits a Clap grammar so you can do more formal analysis on it.

Sorry, I may have been unclear. I was referring to some other parts of the code, including but not limited to code for

deciding whether an input is a file or stdin,
opening a file and handling errors, and
displaying a heading in --verbose mode.

There is a more general refactor here I think. Many of the utilities read input from a list of sources, each of which may be either stdin or a file. Each utility has its own unique way of handling this. This could be refactored into a common data structure in uucore. Something like enum DataSource { Stdin, File(String) }.


Add "show once only" feature

In its current state we have to handle the limiting of the displaying ourselves. It would be great if there was a feature in the api where we can define that the spotlight is displayed only once.

I thing this use case doesn't belong in the library itself. There are other ways of achieving this - e.g. https://github.com/jonfinerty/Once

for now, you have to save the flag on your side and control if you show the spotlight or not.
thank you for the request, but unfortunately there is no plan to implement that.
I will let you know when I do this.


[Exclusivity] Add deserialization callbacks from the optimizer.

AccessMarkerElimination now registers a callback so that any subsequently
deserialized function bodies will have access markers stripped for optimization.
rdar:31908496 Assertion failed: (isa(Val) && "cast() argument of
incompatible type!") in SILPerformanceInliner
(cherry picked from commit 195da33d1d8db87ee7a699d4b88a84d428d72c6b)

@swift-ci test and merge.


Fix hang on Windows 10 (issue #31)

This is the suggested fix/workaround for the hang on Windows 10 (issue #31).

The Windows 10 hang is fixed in https://github.com/rprichard/winpty/commit/d4640890cf00346d403c9688a057013b7cf2f3f4.


Mismatch of output when trying the model

I was looking for something related to speech recognition where they can use speech tool to achieve their day to day task such as save Mahesh as contact, please call sandeep if they have poor vision strength etc.
The linguistic culture is Indian, tried speechbrain/asr-wav2vec2-commonvoice-en
I spoke PLEASE CALL MAHESH got.

In order to resolve the difference, I thought to train on data available from here
raw speech datset
However upon running I faced this issue:
!python train.py hparams/train_en_with_wav2vec.yaml --data_folder=/content/nptel-pure got this error  speechbrain.core - Exception: Traceback (most recent call last):   File "train.py", line 310, in <module>     character_coverage=hparams["character_coverage"],   File "/content/speechbrain/recipes/CommonVoice/ASR/seq2seq/speechbrain/speechbrain/tokenizers/SentencePiece.py", line 170, in __init__     run_on_main(self._train_BPE)   File "/content/speechbrain/recipes/CommonVoice/ASR/seq2seq/speechbrain/speechbrain/utils/distributed.py", line 61, in run_on_main     func(*args, **kwargs)   File "/content/speechbrain/recipes/CommonVoice/ASR/seq2seq/speechbrain/speechbrain/tokenizers/SentencePiece.py", line 304, in _train_BPE     spm.SentencePieceTrainer.train(query)   File "/usr/local/lib/python3.7/dist-packages/sentencepiece/__init__.py", line 407, in Train     return SentencePieceTrainer._TrainFromString(arg)   File "/usr/local/lib/python3.7/dist-packages/sentencepiece/__init__.py", line 385, in _TrainFromString     return _sentencepiece.SentencePieceTrainer__TrainFromString(arg) RuntimeError: Internal: src/trainer_interface.cc(406) [!sentences_.empty()]
It would be great if anyone could help me translate the error language

As this this related to the tokeniser, I would suggest you to do some data preparation cleaning. E.g., from the error, consider checking that some sentences aren't empty or only containing character that are not decoded properly.

Hello,
The issue seems to be inactive and this comment is giving the right path to follow. I am therefore closing this issue. Please, feel free to reopen it if needed.
Best,
Adel


How to mock a standalone function?

I want to mock a standalone function (i.e. a function that's not a method of some object) but I can't find information on that in the readme.
Here is a simplified version of what I'm trying to do:
type Callback = (x: number) => void;

const caller = (callback: Callback) => {
    callback(Math.random())
}

let mockCallback: Callback = mock<Callback>();

caller(mockCallback);            // fails when using mock without instance
caller(instance(mockCallback));  // fails when using mock with instance
caller(x => console.log(x))      // works when not using a mock

As shown above, the first two caller(...) calls that use the mock would fail. They fail with the error:
TypeError: callback is not a function
 ❯ caller src/test.test.ts:7:5
      5|
      6| const caller = (callback: Callback) => {
      7|     callback(Math.random())
       |     ^
      8| }

I want to mock a standalone function so that I can capture its arguments and do stuff with them. Could you please help me out? Thanks

@mebble Hey,
This project is in maintenance mode as the original upstream isn't maintained at all, so in general there's no plan of adding new features (especially functionalities which are available in jest/vitest)  but there's a PR we can merge on that.
What do you think of the following?
let fn: (a: number, b: string) => number = fnmock();
when(fn(10, 'hello')).thenReturn(5);
instance(fn)(10, 'hello'); // returns 5
verify(fn(10, 'hello')).called();

I can port that and release another version

Yeah this API seems consistent with how we mock objects. You must be referring to PR https://github.com/TypeStrong/ts-mockito/pull/7?
I switched to vitest mocks as you mentioned. In case someone runs into this, here's how to mock and capture in vitest:
import { vi, Mock } from 'vitest';

type Callback = (x: number) => void;

const caller = (callback: Callback) => {
    callback(Math.random())
}

const mockCallback: Mock<Parameters<Callback>, ReturnType<Callback>> = vi.fn();

// inject the mock
caller(mockCallback);

// capture the mock's calls
const [randomNum] = mockCallback.mock.lastCall!

I think we can close this issue if the discussion on fnmock is in PR#7


fix(approval-signing-manager): hash large payloads

Do not squash
Description
Adds a fix for approval-signing manager, as seen in the vault and fireblocks signer
Breaking Changes

JIRA Link

Checklist

[ ] Updated the Readme.md (if required) ?

There is that merge commit in the middle of my two commits.
I am tempted to rebase it out since there is only one comment

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:

:tada: This PR is included in version<EMAIL_ADDRESS>:tada:
The release is available on:

npm package (@latest dist-tag) :package::rocket:
GitHub release :package::rocket:


Add a new pass to convert output chunks to ES Modules

Here's some early work to convert output chunks to ESModules and use import and export statements to reference symbols across chunks. It's definitely not ready for main use yet, but I wanted some early feedback on the strategy.
My main concern here is mutability. How do we guard against:
import {a} from './m0.js';
a = 4; // This didn't actually change A in m0, right?

Created internal issue http://b/132364485

Maybe you don't have to worry about mutability too much?
I just tried this little test example
<!DOCTYPE html>
<meta charset="utf-8">
<head>
<title>Page Title</title>
</head>
<body>
  <script type="module">
    import {a, getA} from '/modules/lib1.js';
    console.log(`a = ${a}, getA() => ${getA()}`);
    a = 15;
    console.log(`a = ${a}, getA() => ${getA()}`);
  </script>
</body>

// /modules/lib1.js
export let a = 1;
export function getA() {
  return a;
}

Chrome reported an error for the a = 15 assignment.

Chad is it worth keeping this PR open?

I finally was able to finish this and run real-world tests on a sizable project. The results were quite surprising. Across the board, this had almost no impact on the total size of the project. Some files did see significant changes in size, but overall that was not the case. A few files actually had a dramatic reduction in size - my guess is that the variable references vs the properties allowed more optimizations. I was prepared for this to increase the size so neutral was definitely a win.
Full results can be seen here: https://drive.google.com/file/d/16CfcUOrXdEZtezOPaA0tp30IgrPkLBHA/view?usp=sharing
The only optimization pass I have seen thus far that needed updated was ExtractPrototypeMemberDeclaration. The global variable reference that was shared threw exceptions as the imported symbol cannot be modified. I added an option for this pass to use a per-chunk variable rather than solely a global one.

Thanks for posting those results, @ChadKillingsworth
Have you you done any testing to confirm that the output code behaves correctly?

I haven't run the full end-to-end test suite, but I have run and manually tested the application locally. Everything worked as expected so far.
I added a check with an error for the case when an imported symbol is the LHS of an assign. My application had 4 errors for that - all in tests and were easy fixes.

@brad4d this should be ready for review now

All comments addressed. I ran this on my internal project again just to make sure things were still working.
As a reminder, #3724 must be merged as well or this pass doesn't work well. I get errors like:
node_modules/google-closure-library/closure/goog/uri/uri.js:1165:0: ERROR - [JSC_IMPORT_ASSIGN] Imported symbol "h" in chunk "js/bannoweb-login.js" cannot be assigned
  1165| goog.Uri.QueryData.prototype.getCount = function() {
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  1166|   this.ensureKeyMapInitialized_();
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  1167|   return this.count_;
        ^^^^^^^^^^^^^^^^^^^^^
  1168| };

imported and sent for internal review and testing

imported and sent for internal review and testing

imported and sent for internal review and testing

Sorry, that was intended for #3724

imported and sent for internal review and testing

Sorry, that was intended for #3724

imported, edited as needed to match internal requirements, sent to @blickly for internal review

I'm trying to submit this now after making quite a lot of changes.
Here's a couple of particularly interesting ones for educational purposes.

Replaced all uses of HashSet and HashMap with LinkedHashSet and LinkedHashMap - This is necessary to ensure that changes to the ordering of the Java hash implementation won't cause the compiler's output to change in subtle ways.
Had to add super source (j2cl fakes) for a couple of Path methods that weren't previously by our code.
Use computeIfAbsent() to add to a possibly-not-there-yet item in a map.
Prefer .isEmpty() over .size == 0

I'm trying to submit this now after making quite a lot of changes.
Here's a couple of particularly interesting ones for educational purposes.

Replaced all uses of HashSet and HashMap with LinkedHashSet and LinkedHashMap - This is necessary to ensure that changes to the ordering of the Java hash implementation won't cause the compiler's output to change in subtle ways.
Had to add super source (j2cl fakes) for a couple of Path methods that weren't previously by our code.
Use computeIfAbsent() to add to a possibly-not-there-yet item in a map.
Prefer .isEmpty() over .size == 0

This was merged, but somehow didn't get marked as such.
Maybe that's because I had to make such significant changes to land it?

This was merged, but somehow didn't get marked as such.
Maybe that's because I had to make such significant changes to land it?


Create an intro block

Make an intro, where you describe in 2-4 sentences what you do, what you like to do, what you want to do better, what is important to do, why you are here.

I am a 2nd-year student at Vilnius Gediminas Technical University, specialty in Information Technologies.
I like to create things and solve problems, and programming is the perfect tool to implement that.
However, at the moment some tasks are too complicated for me, which is why I want to enhance my coding skills, expand my database (know more libraries, algorithms, patterns, etc.), and, of course, put it all into practice to gain the skill of "thinking like a programmer". Why I am a good candidate? I can work in a team, open, creative, and friendly person, not being lost in stressful
situations, and often find good non-standard solutions, also if I am interested in
something I can spend a lot of time on it without noticing the fatigue.


Can't install google-cloud-pubsub on RPI

Trying to follow readme instructions and got stuck on this step.
pip install --upgrade google-cloud-pubsub
Note this is running on my RPI. I've updated the firmware and other packages as per the readme.
It fails with an install error that I need help resolving.
Something similar mentioned here but I don't know how to proceed.
pi@raspberrypi:~/iot/Cloud-IoT-Core-Kit-Examples/CPUTemp $ pip install --upgrade google-cloud-pubsub
Downloading/unpacking google-cloud-pubsub
Downloading google_cloud_pubsub-0.28.4-py2.py3-none-any.whl (79kB): 79kB downloaded
Downloading/unpacking grpc-google-iam-v1<0.12dev,>=0.11.1 (from google-cloud-pubsub)
Downloading grpc-google-iam-v1-0.11.4.tar.gz
Running setup.py (path:/tmp/pip-build-eqe101m8/grpc-google-iam-v1/setup.py) egg_info for package grpc-google-iam-v1
Downloading/unpacking google-gax<0.16dev,>=0.15.13 (from google-cloud-pubsub)
Downloading google-gax-0.15.15.tar.gz (109kB): 109kB downloaded
Running setup.py (path:/tmp/pip-build-eqe101m8/google-gax/setup.py) egg_info for package google-gax
warning: no previously-included files matching 'pycache' found anywhere in distribution
warning: no previously-included files matching '*.pyc' found anywhere in distribution
Downloading/unpacking psutil<6.0dev,>=5.2.2 (from google-cloud-pubsub)
Downloading psutil-5.4.0.tar.gz (406kB): 406kB downloaded
Running setup.py (path:/tmp/pip-build-eqe101m8/psutil/setup.py) egg_info for package psutil
Downloading/unpacking google-cloud-core[grpc]<0.28dev,>=0.27.1 (from google-cloud-pubsub)
Downloading google_cloud_core-0.27.1-py2.py3-none-any.whl (50kB): 50kB downloaded
Cleaning up...
Exception:
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/pkg_resources/init.py", line 2680, in requires
deps.extend(dm[safe_extra(ext)])
KeyError: 'grpc'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/pip/basecommand.py", line 122, in main
status = self.run(options, args)
File "/usr/lib/python3/dist-packages/pip/commands/install.py", line 290, in run
requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
File "/usr/lib/python3/dist-packages/pip/req.py", line 1266, in prepare_files
req_to_install.extras):
File "/usr/local/lib/python3.4/dist-packages/pkg_resources/init.py", line 2683, in requires
"%s has no such extra feature %r" % (self, ext)
pkg_resources.UnknownExtra: google-cloud-core 0.27.1 has no such extra feature 'grpc'
Storing debug log for failure in /home/pi/.pip/pip.log

Ah, google-cloud-pubsub depends on google-cloud-core now.
I've updated the documentation to read:
sudo -s
apt install build-essential libssl-dev libffi-dev python-dev
pip install pyjwt paho-mqtt cryptography
pip install --upgrade google-api-python-client
pip install --upgrade google-cloud-core
pip install --upgrade google-cloud-pubsub
exit
Let me know if that works for you. Thanks for reporting this!

This worked thanks but I ran into some more trouble.
If I recall, it complained about cffi library. So I did this and it worked:
sudo pip -vvv install --upgrade --force-reinstall cffi


Style comments, include Date / time posted, impose limit on number of comments shown.

Style the comments with a border and padding. Replace timestamp with Date variable, and impose a limit on the number of comments shown through a query in the fetch method.

Thanks for the further comments @bheff-work -- I haven't included class constants as much in the past, so I'll make sure to do that going forward.


ImportError: cannot import name

While running the code of example scene (or other file, e.g. basel.py), I get stuck into the ImportError:
File "extract_scene.py", line 15, in
from scene import Scene
ImportError: cannot import name 'Scene'
The error varies as I try to run different files, which the same error ImportError is encountered in different modules/classes in different py file.
I suppose that the cause is somehow related to location of files I should put. Now I just download the file , move nothing and compile the code. Is there any clue what might be the cause of the error?

The definition of scene.Scene is here https://github.com/3b1b/manim/blob/master/scene/scene.py#L28

Closing, as this issue is old and inactive. Feel free to reopen it if this is still an issue.


Typo fix in e2e test readme

The current E2E testing readme indicates that you need to have an etcd operator image created prior to testing the prometheus operator, which was a little confusing. It looks like this was just a copy and paste issue from the etcd operator E2E test readme. This PR updates the readme to reflect that it's a prometheus operator image that is required.

oops :) thanks!


non standard keyboard navigation

According to standard:
_https://raw.githack.com/w3c/aria-practices/issue57-date-picker-design-pattern/aria-practices.html#calendar-grid-keyboard-interaction_

Shift Page Up: Updates the calendar grid to the next year. instead of Alt+Page Up

Shift Page Down: Updates the calendar grid to the previous month. instead of Alt+Page Down

Home: Moves focus to first day of the current week. not to first day of current month

End: Moves focus to the last day of the current week. not to last day of current month

partial fix in #59


Add PowerLineDecoration

Adds ability to draw arrows, slants, curves as used for the powerline effect. Ricers will hopefully enjoy this one!
Closes #90

I tried to test it, but got this error: https://dpaste.org/tzkSy
Hmm... this doesn't happen on 159eb19b9e796b7b685c8806c40cca709ed25f59

qtile: 0.21.1.dev89+gc8d3aad1
qtile-extras: 0.1.dev141+g6b4510c

Hmm, can you share your config? I'm not sure that's an issue with this code (at least, I can't recreate on my system).

I also think it's my config: jx11r/qtile
But I'm not sure why, maybe the problem is extras/textbox.py

@elParaguayo I can't use the TextBox widget, I've tried this minimal config:
from libqtile import bar, layout
from libqtile.config import Click, Drag, Group, Key, Match, Screen
from libqtile.lazy import lazy
from libqtile.utils import guess_terminal

from qtile_extras import widget

mod = "mod4"
terminal = guess_terminal()

keys = [
  Key([mod], "space", lazy.layout.next(), desc="Move window focus to other window"),
  Key([mod], "Return", lazy.spawn(terminal), desc="Launch terminal"),
  Key([mod], "Tab", lazy.next_layout(), desc="Toggle between layouts"),
  Key([mod], "w", lazy.window.kill(), desc="Kill focused window"),
  Key([mod, "control"], "r", lazy.reload_config(), desc="Reload the config"),
  Key([mod, "control"], "q", lazy.shutdown(), desc="Shutdown Qtile"),
  Key([mod], "r", lazy.spawncmd(), desc="Spawn a command using a prompt widget"),
]

groups = [Group(i) for i in "12345"]

for i in groups:
  keys.extend([
    Key([mod], i.name, lazy.group[i.name].toscreen(toggle=True)),
    Key([mod, "shift"], i.name, lazy.window.togroup(i.name, switch_group=True)),
  ])

layouts = [
  layout.Max(),
  layout.MonadTall(
    single_border_width = 0,
    border_width = 0,
    single_margin = 15,
    margin = 15,
  ),
]

widget_defaults = dict(
  font = "sans",
  fontsize = 12,
  padding = 3,
)

screens = [
  Screen(
    bottom = bar.Bar(
      [
        widget.TextBox(text = 'test'),
      ],
      24,
    )
  ),
]

mouse = [
  Drag([mod], "Button1", lazy.window.set_position_floating(), start=lazy.window.get_position()),
  Drag([mod], "Button3", lazy.window.set_size_floating(), start=lazy.window.get_size()),
  Click([mod], "Button2", lazy.window.bring_to_front()),
]

dgroups_key_binder = None
dgroups_app_rules = [] # type: list
follow_mouse_focus = True
bring_front_click = False
cursor_warp = False
floating_layout = layout.Floating(
  float_rules = [
    *layout.Floating.default_float_rules,
    Match(wm_class="confirmreset"),
    Match(wm_class="makebranch"),
    Match(wm_class="maketag"),
    Match(wm_class="ssh-askpass"),
    Match(title="branchdialog"),
    Match(title="pinentry"),
  ]
)
auto_fullscreen = True
focus_on_window_activation = "smart"
reconfigure_screens = True
auto_minimize = True
wl_input_rules = None
wmname = "LG3D"

But the problem persists:
Exception in callback Qtile.call_soon.<locals>.f() at /usr/lib/python3.10/site-packages/libqtile/core/manager.py:850
handle: <Handle Qtile.call_soon.<locals>.f() at /usr/lib/python3.10/site-packages/libqtile/core/manager.py:850>
Traceback (most recent call last):
  File "/usr/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/lib/python3.10/site-packages/libqtile/core/manager.py", line 851, in f
    func(*args)
  File "/usr/lib/python3.10/site-packages/libqtile/bar.py", line 559, in _actual_draw
    self._resize(self.length, self.widgets)
  File "/usr/lib/python3.10/site-packages/libqtile/bar.py", line 450, in _resize
    offset += i.length
  File "/usr/lib/python3.10/site-packages/qtile_extras-0.1.dev141+g6b4510c-py3.10.egg/qtile_extras/widget/decorations.py", line 508, in length_get
    length = int(self.calculate_length())
  File "/usr/lib/python3.10/site-packages/libqtile/widget/base.py", line 527, in calculate_length
    return min(self.layout.width, self.bar.width) + self.actual_padding * 2
  File "/usr/lib/python3.10/site-packages/libqtile/drawer.py", line 81, in width
    return self.layout.get_pixel_size()[0]
  File "/usr/lib/python3.10/site-packages/libqtile/pangocffi.py", line 138, in get_pixel_size
    pango.pango_layout_get_pixel_size(self._pointer, width, height)
TypeError: initializer for ctype 'PangoLayout *' must be a cdata pointer, not NoneType

Interesting. I don't get the same error but I also can't create a textbox without a decoration (it works with the decoration).

Found the bug!
Have been making other changes to the code too (which you may have seen on Discord). Will push changes later but may not be until tomorrow.

Should be working now - NB customisation is different in the latest commit.

Well, it's working now, you've done an amazing job!

With nerd fonts:

With qtile-extras:

Is that last screenshot really with qtile-extras? I haven't done the padding yet on the powerline decoration to it should be drawing to the edges of the bar.

I'm not using the padding, it's a simple border with the same background color.
bar = {
  'background': color['bg'],
  'border_color': color['bg'],
  'border_width': 4,
  'margin': [10, 10, 0, 10],
  'opacity': 1,
  'size': 18,
}

Ah. Cool.


[Color Picker] Increase magnifier size and allow adjustable zoom levels

📝 Increase magnifier size and allow adjustable zoom levels
A larger magnifying glass allows greater coverage of the screen so that we can see where we are in relation to the zoomed in view. This applies nicely for very high resolution screens when the pixels are much smaller.
An adjustable zoom level in the settings would also help alleviate this issue, giving much better context on where the cursor is.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

can you do maybe a screenshot of what you are thinking about?  "larger" is relative :)

I can see 'more' of the surroundings and thus am able to better get a sense of where my cursor is pointing. This 'zoomed-out' look should be customizable as maybe some users prefer the current setting.

ShareX implements this well:

@ZipD-spec
is the magnifier in 0.29.3 large enough?

@ZipD-spec
is the magnifier in 0.29.3 large enough?

I don't get it, how is this large at all? It's literally showing just the pixel.

I don't get it, how is this large at all? It's literally showing just the pixel.

@ZipD-spec
use the mouse wheel to zoom https://docs.microsoft.com/en-us/windows/powertoys/color-picker#select-color

@ZipD-spec
use the mouse wheel to zoom https://docs.microsoft.com/en-us/windows/powertoys/color-picker#select-color

Works great! Thank you!

Works great! Thank you!

@ZipD-spec
can we close the issue?

@ZipD-spec
can we close the issue?

Yes, closing it.

Yes, closing it.


Compare outputs with tolerance

This adds a Java program that compares two output files with tolerance for accumulated double errors.
It also produces nice diagnostic output, indicating the whole entry where there's a problem.

Dang, all this is a bit of a mess. Turns out, I even had the correct behavior initially (see commented code in baseline), but then I got "smart" and refactored it into that round() method. Someone brought up the concern about missing creating false positives with this approach (i.e. masking actual errors). Seems kinda impossible to not make it suck in one way or another. Maybe, the least bad way could be this:

Fix the baseline
Allow tests to be run with two modes: "strict" (i.e. current diff-based impl, contenders must exactly match the result of the fixed baseline) and "lenient" (i.e. what's proposed here, contenders can under-report mean by 0.1 compared to the fixed baseline)
Leave existing entries as is (i.e. implicitly using "lenient" validation mode)
Use "strict" mode for new or updated entries going forward
When publishing the 10K key set test results for the 10 or so top entries (i.e. similar to your earlier experiment), add a column which says whether they pass in "lenient" or "strict" mode, encouraging entries which only pass in "lenient" mode to adjust their implementation

I feel that may be the best balance to ensure correctness while not moving goal posts too much at this point. In particular, it would not mess up any existing results so far, which feels important to me.
@mtopolnik, @royvanrijn, @hundredwatt, @thomaswue, @merykitty, any thoughts?

@mtopolnik, @royvanrijn, @hundredwatt, @thomaswue, @merykitty, any thoughts?

Maybe we can first assess the damage? How many solutions are failing? For example, with my dataset, I found only a single solution to have this off-by-one-decimal problem.

Sounds like a good plan. If there are entries failing, let’s mark that separately on the leaderboard and they’ll hopefully be fixed.

I can add to the script that it records such minor mistakes, but continues testing for larger ones. Then we know for sure the solution is correct module off-by-decimal.

Maybe an easier approach to not have to change testing too much:

Copy CalculateAverage_baseline to CalculateAverage_baseline_lenient
Copy CalculateAverage_baseline to CalculateAverage_baseline_strict
Fix strict version
Use diff first on strict output, then on lenient output if needed.

Maybe we can first assess the damage? How many solutions are failing? For example, with my dataset, among the first 20 I found only a single solution to have this off-by-one-decimal problem.

For the test case brought up by @tivrfoa here, about 1/3 of all implementations fail when comparing to the output of the fixed baseline. But out of the current Top 20, i.e. the set I'd run for the 10K keyset potentially, only 2 fail. So not too bad indeed.

Maybe an easier approach to not have to change testing too much...

Yeah, coming to the same conclusion actually. So I think just adding the new test case should be all we need here, and I'll follow up with updating the baseline implementation, keeping the old one around two for cases where the old output needs to be reproduced.

This seems to address #49

Ok, so as discussed I'm moving forward with this by only adding the new test case and adjusting the baseline implementation accordingly. I'm going to close this PR in favour of https://github.com/gunnarmorling/1brc/pull/392, which contains this work.
New or updated entries need to be compliant with this fixed behavior, i.e. pass the new additional test, whereas existing entries remain as-is, i.e. compliance at the point of original evaluation is relevant. Thanks a lot @mtopolnik for pushing on this, and to all for providing feedback.

Here's the summary of what I understood. Many solutions do this: accumulate an integer sum that is 10x the actual value, and then, to get the average:

Convert both sum and count to double;
Divide sum by count;
Round the result;
Divide by ten.

The baseline accumulates the sum as a double, and then:

Divide sum by count;
Multiply by 10.0;
Round;
Divide by ten.

The proposed fix is to instead:

Multiply the sum by 10.0;
Round;
Divide by 10.0;
Divide by count;
Round again.

This should align the result with the first approach, which accumulates the integers.

Yes, pretty much that. See the linked new PR for how it looks like from a "specification" point of view.


Generate hashes from pypiserver

Description
I have a local Pypiserver used in production to speed up installation. It has wheels for all the packages I use, so whenever pypi.org does not have a wheel for one, I build it myself and add it to my Pypiserver, but I just download it if it's available. Because I build some wheels, I would like to add their corresponding hashes to requirements.txt when I run :
python3 -m piptools compile --upgrade --generate-hashes --index-url <pypiserver_url> --extra-index-url https://pypi.org/simple

I also would like to keep that --extra-index-url https://pypi.org/simple to know if some packages have new releases that are not in our local index. From what I understand, pip-compile will always try to install the most recent version allowed by the input's constraints.
My main issue is that pip-compile always uses hashes from pypi.org even if my server has the same versions. I do not know much about pip-compile's implementation, but after some debugging I found out it's using this Warehouse JSON API and Pypiserver does not support it as far as I know (I tried with 1.2.1 and 1.4.2 which is also the latest).
What has been tried
I tried adding the hashes to my requirements.in, but they were not taken into account. I also tried manually adding hashes to requirements.txt, however they were removed when I re-ran the command (as expected).
Possible feature requests
I did not create this issue using the feature request template as pip-tools might have some functionality I'm not aware of that could support my intended workflow. However, I do have some things in mind if that's not the case and I can create a separate issue for any of them:

Add a flag to pip-compile to always generate hashes from files:
If I remove extra-index-url , I get several messages saying Couldn't get hashes from PyPI, fallback to hashing files. If I could somehow trigger this functionality directly instead of as a fallback, it would solve my issue.
Use hashes from input if available:
This would also work, but I would have to manually add new hashes for custom wheels, even for transitive dependencies.
Get hashes directly from Pypiserver:
Probably the cleanest solution and we already have #1211 describing it, but looking at the warehouse issue it points to, it does not look like it will be implemented soon.

Thank you for this great piece of software! I really appreciate your time and help.

Hello @AliGhahraei,
Thanks for the issue!

Add a flag to pip-compile to always generate hashes from files:
If I remove extra-index-url , I get several messages saying Couldn't get hashes from PyPI, fallback to hashing files. If I could somehow trigger this functionality directly instead of as a fallback, it would solve my issue.

Yeah, this might be a solution.

Hello @AliGhahraei,
Thanks for the issue!

Add a flag to pip-compile to always generate hashes from files:
If I remove extra-index-url , I get several messages saying Couldn't get hashes from PyPI, fallback to hashing files. If I could somehow trigger this functionality directly instead of as a fallback, it would solve my issue.

Yeah, this might be a solution.

From what I understand, pip-compile will always try to install the most recent version allowed by the input's constraints.

Well, it also checks for an existing output file, and doesn't upgrade higher than the version specs there if it doesn't have to, to meet the input requirements.
$ <<<'plumbum' pip-compile --no-header --no-annotate - -o reqs.txt
plumbum==1.7.0
$ <<<'plumbum<1.7.0' pip-compile --no-header --no-annotate - -o reqs.txt
plumbum==1.6.9
$ <<<'plumbum' pip-compile --no-header --no-annotate - -o reqs.txt
plumbum==1.6.9


Question about App Insights app version

Your sample (under A|B Testing) shows setting your application version using itemProperties.Properties["AppVersion"], but there is already a standard property of ITelemetry.Context.Component.Version or even globally set with ApplicationInsightsServiceOptions.ApplicationVersion.  Is there a reason this would use a custom property?

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

ID: edce8383-4e3b-425b-5f01-02f54385b7a4
Version Independent ID: bf51eb39-0bb9-262a-d4be-51b1bcb977f0
Content: Usage analysis with Application Insights - Azure Monitor
Content Source: articles/azure-monitor/app/usage-overview.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @bwren
Microsoft Alias: lagayhar

@atkulp Thanks for reaching out. We will investigate and update the thread accordingly.

@atkulp, great suggestion. You are right, the better approach would be to use the App Version standard dimension instead of custom dimensions. This would be a better example:
// Telemetry initializer class
public class ApplicationVersionTelemetryInitializer : ITelemetryInitializer
{
    // In this example, to differentiate versions, we use the value specified in the AssemblyInfo.cs
    // Make sure to set different assembly version when you deploy your application for A/B testing.
    static readonly string _version = System.Reflection.Assembly.GetExecutingAssembly().GetName().Version.ToString();
    public void Initialize(ITelemetry item)
    {
        item.Context.Component.Version = _version;
    }
}

I just updated the doc, it will go live shortly.

@vgorbenko - Thanks for providing your insights and raising the PR.
@atkulp - Thanks again for bringing this to our attention. We have updated the Doc and it will be live once its merged.  We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


docs: update README.md with some grammatical corrections

Motivation
The README could be explained more clearly with some minor grammatical changes.
Have you read the Contributing Guidelines on pull requests?
Yes, I have read it.
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

thanks


Mock dns resolving and speedup tests

If old time was ~5s on running all tests, now I have ~0.7s on all this.

P.S. This also allows run tests without internet, and on Termux (which cannot resolve DNS, because of it's own reasons).


Removed 1 space so asterisk work properly

@TerryWarwick,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


GPG key validity affected by non-expiring key

Can you reproduce the bug at https://try.gitea.io:

[X] Yes (provide example URL) Users setting, no way to show URL.
[ ] No
[ ] Not relevant

Log gist:

Description
See #1293 comments.

closed by #1628


[Tab] lock tab by default in the application

From https://github.com/Inria-Asclepios/music/issues/356
This PR locks tab in Meshing, Reformat and Segmentation.
:m:

This PR actually locks the tab everywhere, not just in Meshing, Reformat and Segmentation.
Is it what was meant?

There are no more workspaces where we don't want locked tabs.

Indeed, it was the idea to lock every workspace :)

Ok, why not.
Can we make an issue to remember that we have to either improve the tabs or remove them from MUSIC?

why? if we want to unlock a specific toolbox we can do it in the toolbox.

OK.
Now that I am thinking about it, can we keep the tabs in Visualisation Workspace?
First because I use the shortcut Ctrl+T quite often to display several datasets, then so that we (and future devs) know that this feature exist (sleeping features hardly wake up).

In my previous comment I forgot about the visualisation workspace where, I agree, tabs should be allowed.

I updated this PR. Visu is unlocked. Thanks (sorry for the commit name, it's a missclick :p)

Let's see if we don't miss this feature too much.
I merge.


Rename Config type

This is not all that descriptive a name, though we're not sure what a better name is. I suggest Builder. If nothing better comes up, pick that and we'll bikeshed it on the PR.
cc @alexcrichton @sfackler @dtolnay @kimundi @burntsushi bikeshedding.

+1 for Builder

I am not a fan of Builder because it makes me expect a build() method that gives me back some object that I used it to build.
Some alternatives to explore:

gcc::Orchestrator⸻https://en.wikipedia.org/wiki/Orchestration_(computing)
gcc::Invocation⸻like Clang's clang::CompilerInvocation
gcc::Build⸻in the noun sense like "build tool", "build automation"

Build sounds pretty reasonable to me, becuase that's what's happening here anyway!


User friendly settings assignments

Apart from following test case (line 409-410), all other are passing.

Apart from following test case (line 409-410), all other are passing.

Following are the usage of new settings assignment.

As mentioned before, we have observed following behavior.
settings assignment to True is working but assignment to False is not working.


KDC1 : New files as foundation for enhancements

This PR has no impact on existing code.  It introduces new files as foundation for RFE #1 and RFE #2.  The new defaults.yaml file will facilitate further enhancements and hiding complexity from pillars.

Required by #5 and #6

Hey @fintanmm thanks for taking the time for reviewing.
If you have time can you check the other 2 follow-up PR also.
Thanks!!!


Python: Recover connection refactor

Recovered refactor from 027fd3e in #5764.

LGTM, thanks @helderco!


Forest is a single crate, not a workspace of many crates

I'd like to make forest a single crate
Benefits

less hassle to manage dependencies

~E.g just looking at the top of our Cargo.toml, we have three three versions of aead - I suspect we've got a few duplicate dependencies like this.~

more optimization opportunities (within crate)
makes further refactoring easier

better dead code detection

shorter compile times (we don't need to do all that linking again)

Risks

you'll have to remember to do cargo check --all in the interim
Maybe longer compile times - some crates choose to break into sub-crates to parallelize compiles. If this happens, we should probably just write less code :)

How we do this incrementally

Convert our workspace from a virtual manifest to a root package with a workspace section
Work from the bottom of the dependency graph of our crates, interning each crate into the root package

This is mostly renames, which should be git-history friendly

Handle the renames in the other crates
Move tests
Repeat

see https://github.com/ChainSafe/forest/compare/main...aatifsyed/one-crate-to-rule-them-all

Great to have this as an issue. Such a reorganization would be a boon. I'm not convinced by all of the listed benefits, though. For example, some of our dependencies use the older aead when targeting wasm32-unknown-unknown. We have many duplicate dependencies (as witnessed by cargo tree --duplicates), but none of those will be fixed simply by using a single crate. I would also be surprised if there's a measurable effect on performance or a meaningful change in linking times. Nonetheless, the organizational benefits are more than worthwhile.
For context, Forest was made up of 49 crates last year and sits at 31 crates at present. Some crates should be pushed upstream, some should be spun out of the repository, and most should be merged into a single forest crate.

The only practical benefit of doing this, for me, is getting rid of circular dependencies when moving things around. We wouldn't have this issue with a single crate, and we could structure the modules however we deem appropriate. This may not necessarily be a good thing, though.
Hypothetically it would also help with a feature flag creep, but we don't have that many flags and don't indent to use them extensively.
As for the downsides that matter, it would (and any large refactor, for that matter) make git blaming difficult unless those aren't exclusively renames/moves that are automatically followed. I managed to find countless regressions by just looking at recently changed lines.
All in all, it seems like a cosmetic change. I'm not necessarily in favour of this, but I'm okay with going ahead with it.

As of https://github.com/ChainSafe/forest/pull/2973/commits/d2b598f249b49b3b2ca407caf79c531c7bc5f69b, most of the files are recognised as moved


loader added in manage admin

Issue that this pull request solves
Closes: #595

Please Check, I have updated the Loader.

Please Check and review, I have updated the Loader.

Perfect! Looks good to go!


Classes B and C for steelec2en material type carbon steel

I'm testing the difference for the hotformed carbon steel for different classes of ductibility but only class A is running on Safir. When I change it to classes B or C Safir freezes up. It seems like a quick review on the code might do.

You can ask this kind of question to<EMAIL_ADDRESS>

We could check and help you if you send us the files that do not run to<EMAIL_ADDRESS>

Here they are.
On Fri, Aug 18, 2023 at 12:12 PM JMFranssen @.***>
wrote:

We could check and help you if you send us the files that do not run to
@.***
—
Reply to this email directly, view it on GitHub
https://github.com/gmsafir/gmsafir/issues/82#issuecomment-1684063338,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAGNVTSSKETWMOZN4Y6N6G3XV6A7BANCNFSM6AAAAAA3S7IJ7U
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Luís Henrique A. A. de Queiroz.
(31) 99305-9217

Where are they?