Questions about Raftlib

Hi,
I'm making a study of stream processing tools, and I have some questions about Raftlib. I've been studying tools like Spark, Flink, Samza, etc... and I want to know the extent in which Raftlib is comparable to them.
In the thesis, it is briefly mentioned Raftlib capabilities in regards to distributed stream computing. Specifically, it is mentioned that there is an "oar" component that manages a network mesh.
I'd appreciate if you could answer these questions:
1 - Seems like the networking capabilities of Raftlib are poorly documented, or maybe not obviously available. Is there more information on this? Maybe I just didn't find it.
2 - Suppose I want a kernel to spit out a stream of a custom type. Like a class PurchaseEvent or something like that, with many properties. How should I deal with serialization in these cases, when I have multiple nodes in a network and some of these events travel between computers? Should my kernel just return a byte stream or something, and deserialize in the other side?
3 - Is my understanding correct that Raftlib automatically handles back-pressure? I understood that RaftLib automatically optimizes resource allocation for all kernels, and when this is detected, an automatic action takes place. Are there any chances that a node executing some raftlib kernels could suffer resource exhaustion because a kernel is misbehaving?
4 - Normally tools like Flink, Samza, Spark, etc... mention their guarantees regarding message processing or message delivery (is a bit of a muddy definition) in terms of "at-most-once/best effort", "at-least-once", "exactly-once", etc etc. I think this concern is more important in a commercial, enterprise setting than an academic one. Regardless, does raftlib make any guarantees regarding message delivery between network nodes?

Seems like the networking capabilities of Raftlib are poorly documented, or maybe not obviously available. Is there more information on this? Maybe I just didn't find it.

There's a TCP socket version of the FIFO. Over the years it has been removed as keeping it up would require significant work. If it's interesting enough, we can add back, although perhaps plugging in an RDMA style layer might actually be better. The SHM version is in the same boat, although I'm making an effort to bring that back given it's needed to even start on the TCP socket version.

Suppose I want a kernel to spit out a stream of a custom type. Like a class PurchaseEvent or something like that, with many properties. How should I deal with serialization in these cases, when I have multiple nodes in a network and some of these events travel between computers? Should my kernel just return a byte stream or something, and deserialize in the other side?

From a programmer perspective, you just write a struct or class into the stream, the runtime will take care of the rest.

Is my understanding correct that Raftlib automatically handles back-pressure? I understood that RaftLib automatically optimizes resource allocation for all kernels, and when this is detected, an automatic action takes place. Are there any chances that a node executing some raftlib kernels could suffer resource exhaustion because a kernel is misbehaving?

At the moment, yes, you could encounter user-limits if a kernel is misbehaving. That being said, there are limits as to what the runtime can allocate so in reality, the likelihood is slim. But, definitely something we're still working on.

Normally tools like Flink, Samza, Spark, etc... mention their guarantees regarding message processing or message delivery (is a bit of a muddy definition) in terms of "at-most-once/best effort", "at-least-once", "exactly-once", etc etc. I think this concern is more important in a commercial, enterprise setting than an academic one. Regardless, does raftlib make any guarantees regarding message delivery between network nodes?


Input form validation is working wrong

Describe the bug
I have problems with form validation on b-form-input elements, if value is linked by v-model.
Required and some other props are working. But for example minlength is not detected as invalid.
I can reproduce this behaviour and build an example.
Native input und b-form-input without v-model are working.
To Reproduce
https://codesandbox.io/s/bootstrap-vue-3-minlength-input-validation-bogcx3
Expected behavior
Form validation is working in same way as native forms
Environment (please complete the following information):

latest version

I'm getting all the same behaviors in your example.

Hmm, that wired. If you try it with string "Test", first input is valid, others are invalid, because minlength="10"

This is a conflict between Vue, and our script, and HTML standard of what accounts for a violation.
When I originally tested, I took a shortcut and attempted to simply call your checkValidity function on @input of all the  elements. Which causes wildly different answers.
However, your original example only runs when a button is clicked, far beyond a critical point...
You see, a constraint violation in HTML only occurs when the value was last changed by a user. It does not take into account script changes. It may seem not very obvious, but regardless if you have additional features like "format", some scripts of this nature still run, altering the value in the process.
So, if you use my method of adding @input to the  tag, and check for validity, it will immediately return the correct values... However, when you add in a setTimeout, things get more complex. When the @input is fired, the value has not yet been changed. However, when any setTimeout is enabled (in your example, this could be considered the time it takes to click the button), it then returns the wrong value because it has been modified already by script.
Needless to say, this is very complex and will take a lot of effort to get working correctly. It may not be fixed in the near future.

Perhaps you should try using a full Vue JS solution like Vuelidate instead of relying on html

Fixed 0.15


3Sass refactor

I started to work through some of the code and I am realizing that there are some styles i may be able to break down. Anyways, I have begun to refactor some of the scss and all has gone well thus far. let me know if there are any guidelines you'd like me to follow.
also, I think it might be worth integrating SassDoc into the project depending on how complex it gets.

Looks good so far, thanks! Couple of things I spotted:

[ ] you seem to have merged the cal-starts-within-day and cal-ends-within-day classes (on the week and day view stylesheets). These need to be separate. By default all events have square corners, then if it starts or ends on the current day only then the corners are rounded
[ ] can you rebase off master

I've been giving this some more thought and I've decided there isn't really a huge amount of benefit into breaking everything out into variables and mixins, I think it keeps everything cleaner and more maintainable to just leave things as they are.


Fix view indices with Android LayoutAnimation

Fixes issue #11828 that causes layout animations for removed views to
remove some adjacent views as well. This happens because the animated
views are still present in the ViewGroup, which throws off subsequent
operations that rely on view indices having updated.
This issue was addressed in #11962, which was closed in favour of a more
reliable solution that addresses the issue globally since it’s difficult
to account for animated views everywhere. @janicduplessis recommended
handling the issue through ViewManager.
Since API 11, Android provides ViewGroup#startViewTransition(View)
that can be used to keep child views visible even if they have been
removed from the group. ViewGroup keeps an array of these views, which
is only used for drawing. Methods such as ViewGroup#getChildCount()
and ViewGroup#getChildAt(int) will ignore them.
I believe relying on these framework methods within ViewManager is the
most reliable way to solve this issue because it also works if callers
ignore ViewManager and reach into the native view indices and counts
directly.
Test Plan
I wrote a minimal test app that you can find here:
https://gist.github.com/lnikkila/87f3825442a5773f17ead433a810d53f
The expected result is that the red and green squares disappear, a blue
one appears, and the black one stays in place. iOS has this behaviour,
but Android removes the black square as well.
We can see the bug with some breakpoint logging.
Without LayoutAnimation:
    NativeViewHierarchyOptimizer: Removing node from parent with tag 2 at index 0
    NativeViewHierarchyOptimizer: Removing node from parent with tag 4 at index 1
    NativeViewHierarchyManager: Removing indices [0] with tags [2]
    RootViewManager: Removing child view at index 0 with tag 2
    NativeViewHierarchyManager: Removing indices [1] with tags [4]
    RootViewManager: Removing child view at index 1 with tag 4

With LayoutAnimation tag 3 gets removed when it shouldn’t be:
    NativeViewHierarchyOptimizer: Removing node from parent with tag 2 at index 0
    NativeViewHierarchyOptimizer: Removing node from parent with tag 4 at index 1
    NativeViewHierarchyManager: Removing indices [0] with tags [2]
    NativeViewHierarchyManager: Removing indices [1] with tags [4]
->  RootViewManager: Removing child view at index 1 with tag 3
    RootViewManager: Removing child view at index 2 with tag 4

    (Animation listener kicks in here)

    RootViewManager: Removing child view at index 1 with tag 2

Here are some GIFs to compare, click to expand:

Current master (iOS vs Android)

With this patch (iOS vs Android, fixed)

Related PRs
Previously addressed in #11962, which wasn’t merged.
Tangentially related to my other LayoutAnimation PR #18651.
No documentation changes needed.
Release Notes
[ANDROID] [BUGFIX] [LayoutAnimation] - Removal LayoutAnimations no longer remove adjacent views as well in certain cases.

It seems similar to the technique I wanted to use in #16091 and that iOS uses but using a platform implementation which is way better (since my implementation had a bug left that I didn't have time to investigate 😆).

Thanks for taking a look! I’ll try it out with that gist first thing tomorrow.

Tried it out, works fine! I managed to get current master to crash due to a wrong index by removing lots of squares rapidly and then adding some, but this branch was stable.
There’s still a bug with the view opacities not updating when interrupting an ongoing animation (compared to iOS where they get the final value) but it looks like a separate issue to me.

@lnikkila I'm reverting this PR, as we found a java.lang.IndexOutOfBoundsException when removing views:
Exception in native call java.lang.IndexOutOfBoundsException
at android.view.ViewGroup.removeViewsInternal(ViewGroup.java:4795)
at android.view.ViewGroup.removeViewsInLayout(ViewGroup.java:4652)
at com.facebook.react.views.view.ReactViewGroup.removeViewWithSubviewClippingEnabled(ReactViewGroup.java:530)
at com.facebook.react.views.view.ReactViewManager.removeViewAt(ReactViewManager.java:285)
at com.facebook.react.views.view.ReactViewManager.removeViewAt(ReactViewManager.java:36)
at com.facebook.react.uimanager.NativeViewHierarchyManager.manageChildren(NativeViewHierarchyManager.java:381)

Hmm could be due to removeClippedSubviews, I’ll take a closer look with that enabled and see if I can figure out why that’s happening.


Prevent panic when loading an empty buffer

If for some reasons you try to load a truncated to zero file then you end up with a panic.
I'd like to prevent such case. What do you think?

LGTM. Thank you for the contribution.


uBlock Origin support

Hi, is there a way to detect also uBlock Origin, are you planning to make an updated version in the near future? Thank your for your work.

When you have uBlock active, BlockAdBlock don't work, you can't even see the debug in the console. Any update about uBlock? Thank you so much!


Update addon-installer.md

Overview
First: Addons now extend the Installer class, which takes advantage of array data in the upd.addon.php and ext.addon.php files. These two arrays, actions and methods, were missing commas. I added the commas to allow easy copy and paste straight into PHP.
Second: The addon installer class methods, Query_upd::install() and Query_upd::uninstall(), requires you to return a boolean to indicate whether or not the addons was installed properly. So, I added return TRUE to both of these methods.

[ x ] 🐛 Fixes a bug
[ ] 🚀 Implements a new feature
[ x ] 🛁 Rewrites existing documentation
[ x ] 💅 Fixes coding style
[ ] 🔥 Removes unused files / code

@intoeetive I have made the changes as requested above


DataContext in the middleware Invoke function

Description
[Enter feedback here]
First of all, thanks for this documentation, I was able to almost test one middle ware.
However, I was trying to write unit tests for another one which uses DataContext or is DI injested in the Invoke method. The DataContext is however resolved, but the model say Blog does not have any data entries in it. I am assuming this to be the case as I am using the UseTestServer() in the beginning.
Is there a way or any alternative way as well of injesting data to the DataContext so that the middleware can be tested?
Page URL
https://learn.microsoft.com/en-us/aspnet/core/test/middleware?view=aspnetcore-8.0
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/test/middleware.md
Document ID
8415adee-793e-2ba7-b97b-672b493ba42b
Article author
@Tratcher

@andrejpk I noticed you contributed to this article recently -- can you help with this question?


Patch 1

Summary
Changes to enable app_home_opened. This does not include the Manifest changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Provide default gateway setup

What type of PR is this?
feature
Which issue does this PR fix:
What does this PR do / Why do we need it:

Add DEFAULT_SERVICE_NETWORK env variable, on cluster startup create service network and associate VPC
Keeps CLUSTER_LOCAL_GATEWAY behavior (for conformance test) with new network override mode, unlike CLUSTER_LOCAL_GATEWAY, register all gateways to the same default service network.

If an issue # is not available please add repro steps and logs from aws-gateway-controller showing the issue:
Testing done on this change:

Automation added to e2e:

Will this PR introduce any new dependencies?:

Will this break upgrades or downgrades. Has updating a running cluster been tested?:
Does this PR introduce any user-facing change?:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you update doc and add instruction says that we need to pass DEFAULT_SERVICE_NETWORK to start the controller when run the make e2e-test? would be good if you could run e2e-test for this PR.
Overall LGTM.

Addressed the comments, thanks


Please add Concourse CI Pipeline Editor to code-server marketplace

For those of us that use Concourse CI, this would be very helpful:
https://marketplace.visualstudio.com/items?itemName=Pivotal.vscode-concourse

Done! 👍


just reissue the tokens and see what happens

Description
It doesn't look like the tokens are expired, but maybe the gitlab UI is faulty
Just reissue the gitlab access tokens for testing and see what happens
If curious, the process for this is in https://wiki.oicr.on.ca/display/DOC/Testing+Dockstore+Locally
Issue
n/a
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future

Stale credentials?

Possibly, the format looks different too.
Kinda wonder if they deprecated something behind the scenes and didn't let users know (or at least I didn't find anything quickly)


fix: offset filter wrongly set to next page

The offset value was defined with args.page * args.perPage, but should be (args.page - 1) * args.perPage (it should be equal to 0 for page 1).
Also improve typing of getQueryIncludeOf.

:tada: This PR is included in version graphql-gene-v1.1.0-beta.5 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @graphql-gene/plugin-sequelize-v1.1.0-beta.3 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


MaxDensityDataPointsFilter not working

I'm binding libpointmatcher to CloudCompare I'm trying to implement all the different filters.
However when implementing MaxDensityDataPointsFilter I never get any subsampling out. I'm using the SurfaceNormalDataPointsFilter in first place then the MaxDensityDataPointsFilter
such as:
`m_normalParams = PM::get().DataPointsFilterRegistrar.create(
"SurfaceNormalDataPointsFilter",
{
{"knn", "5"},
{"keepNormals", "1"},
{"keepDensities", "1"},
{"keepEigenValues", "0"},
{"keepEigenVectors", "0"},
{"keepMatchedIds", "0"},
}
);
filterParams = PM::get().DataPointsFilterRegistrar.create(
"MaxDensityDataPointsFilter",
{
{"maxDensity", "10"},
}
);
`
Any Advice?

My best advice would be to test your filter using our example code conver.cpp.  You can find it in example/demo_cmake and compile it from there.
It will take a yaml file and a point cloud as input. You can use the yaml file to test you configuration on a stand alone executable. In general, we use often MaxDensityDataPointsFilter, so it should be working. Tuning the parameter will depend on your point clouds.

My best advice would be to test your filter using our example code conver.cpp.  You can find it in example/demo_cmake and compile it from there.
It will take a yaml file and a point cloud as input. You can use the yaml file to test you configuration on a stand alone executable. In general, we use often MaxDensityDataPointsFilter, so it should be working. Tuning the parameter will depend on your point clouds.

Alright,
Must be on my side then, just wanted to make sure there's no known issues with the densities descriptors..
Thanks!

Alright,
Must be on my side then, just wanted to make sure there's no known issues with the densities descriptors..
Thanks!


make js files for every example

It might be easier for people to understand the implementation of the examples if they can run the code and play around with it. It would also be nice to have tests for each example.
I can help out with this if you think it might be helpful 😺

This is actually a very good suggestion. I'll get the first set of files and structures released soon :D Thanks for your suggestion!

We can use jsfiddle.net or codepen :D

Great suggestions! Going to use codepen for the simplicity, and already started

Created codepens. Closing this issue!


Ring compatibility w/out client middleware

In the absence of a :url key, it looks like client/req->domain assumes :host and :port keys in the request.  To support rock-bottom Ring requests, It seems like it might be reasonable to look at :server-name and :server-port?  If that makes sense, I'm more than happy to tackle this, though I don't have information on the necessity of continuing to support :host/:port.

This code is taken pretty much verbatim from clj-http, whichis what I'm trying to maintain compatibility with on the client side.  I'm fine with supporting Ring aliases, though.


Android: java.lang.Boolean cannot be cast to java.lang.String

Bug Information
Version Number of Plugin: 3.1.1
Device Tested On: Xiaomi Redmi Note 4 running Android 7.0
Simulator Tested On: N/A
Version of VS: 2017 15.4.2
Version of Xamarin: <IP_ADDRESS>779
Versions of other things you are using: N/A
Steps to reproduce the Behavior

// In Helpers/Settings.cs
public static string Pin
        {
            get => AppSettings.GetValueOrDefault(nameof(Pin), string.Empty);
            set => AppSettings.AddOrUpdateValue(nameof(Pin), value);
        }

// Somewhere on the app
Helpers.Settings.Pin = "somePinEncryptedHashWithAround450Characters";

Close the app.

Reopen the app.

// In App.xaml.cs
string result = Helpers.Settings.Pin;

Expected Behavior
result should have "somePinEncryptedHashWithAround450Characters"
Actual Behavior
Exception:
Java.Lang.ClassCastException: java.lang.Boolean cannot be cast to java.lang.String

Full Stack
Code snippet
N/A
Screenshots
N/A

In the past you must have saved it as a boolean. you should remove the setting and then re-add it.

Actually I'm doing that with this code on the App.xaml.cs:
try
{
    string result = Helpers.Settings.Pin; // Don't know why this happens...yet!
}
catch (Exception ex)
{
    Helpers.Settings.Pin = string.Empty;
}

This keeps the app running in order to try to fix the issue. Either way, the line in step 2. should have had fixed this by running later in the app:
Helpers.Settings.Pin = "somePinEncryptedHashWithAround450Characters";

Unfortunately, every time I start the app, it falls down on that catch bringing the exception.
I already cleared the cache of the app, uninstalled and reinstalled, and nothing new. Other settings are working normally, for my annoyance.

maybe "pin" is reserved. try something else perhaps.

I had this working and for some reason it stopped. Now, even if I change the "Pin" for "PinHash", it still gives me the error


Tests: improve memory usage

This simple change frees up GPU memory after each unit test. Prior to this change, I would experience a system crash when attempting to run the full unit test suite on RTX 3060 6GB. It now runs comfortably.

Important detail to get right for lower resource environments, thanks for the helpful contribution ❤


Must be last HOC applied to work

I didn't see this in the docs, but I just spent a couple hours trying to figure out why my handleClickOutside method wasn't getting called. I got an error from the package saying I didn't have it, but I most certainly did.
Turns out I had it running like this:
export default onClickOutside(withTheme(MyComponent));

Switched it to this:
export default withTheme(onClickOutside(MyComponent));

and everything works. Seems like either a bug, or something that really should be in the docs. It was interesting cause it was doing something. I would get an error, but the error made no sense, because I did have a handleClickOnOutside method.

To be fair: until Facebook formalizes how chaining should carry over into the original component, there's not much we can do here: with the two HOCs you show, the ordering withTheme(onClickOutside()) works, but there are plenty of other HOCs for which neither ordering works, due to the fact that both of them need to be looking at the original component, and Facebook has not defined an API that HOCs can rely on to get the instance they wrap, with that's directly, or indirectly ("through" other wrappers).
So as annoying as it is, I'm going to have close this issue: we can't update the docs to say "you have to apply this HOC first" because that's basically not true. You might have to, but you might also not, so there's no good advice we can give other than "try different orders if things don't work", which is already standard approach when using higher order components.


updated css file

As u have said i have slightly modified the menu button and its position also.
Hope u liked it.
Please let me know if any mistake is done by me.

As u have said i have slightly modified the menu button and its position also. Hope u liked it. Please let me know if any mistake is done by me.

Excellent Word!
Want to work on other issues?

yeah sure.
But to clarify one thing this repo is not participating in hacktober fest 2022, despite of that it has label onto it of hacktoberfest

Okay but one of my friend contributed to this repo and got accepted


Don't print message placeholder

If the message is empty, the message placeholder should be hidden in the print.

Done.


while installing admin password validation issue.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.2.2
Issue Description
When I installed the Bagisto on my machine, I set a password of 3 characters as an admin password.
and when I tried to login to admin panel, I got validation for the password which requests the password to be minimum of 6 characters.
I think it's better to remove the validation on login, because it doesn't make sense to validate an entered password for login, on creating a user it makes sense to have such validation rule.
Preconditions
--
Steps to reproduce

Install new App
Set a password of 3 characters length.
Navigate to admin login
Validation for password input requires 6 characters length.

Expected Result
Password validation shouldn't be required at login.
Actual Result
validation is set on password input field.
Screenshots
No response
Additional context
No response

Checked and Found that the CLI installation issue has been fixed, now validation is appearing on the CLI Installation for minimum 6 words for password.
Please check the screenshot for reference.
Screenshot


Updating to Guzzle 5 >

It would be great if the plugin requirements could meet a newer version for Guzzle.
We're getting so many conflicts because other SDKs require a newer version.

@tim-hanssen
In composer.json
  "require": {
    "eher/oauth": "1.0.*",
    "guzzle/guzzle": ">=3.1.0,<4"
  }

You can remove <4 to see if it is working. It is there probably for compatibility issue with php <=5.3.3

the newer versions of Guzzle have a different response handling and a different way of configuring requests that is incompatible. This should definitely be upgraded to current Guzzle version...


Question: Select any node through function call

Is there a way to select any node through a function call where the parent nodes will automatically expand, and that node will be selected?
Something look like this: treeRef.dispatchAction({type: 'SELECT', id: 15});?

@dgreene1 Done, I just use controlled properties, thanks:
selectedIds={typeof selectedId === 'number' ? [selectedId] : []}
expandedIds={expandedIds?.length ? expandedIds : []}

I’m glad you found a solution. :)


[GEP-26] ReferenceCleaner reconciler as part of the CredentialsBinding controller

How to categorize this PR?

/area security ipcei
/kind enhancement
/ipcei workload-identity
What this PR does / why we need it:
This PR introduces a new ReferenceCleaner reconciler which is responsible for cleanup of labels & finalizers of Secrets and WorkloadIdentitys that are no longer referenced by CredentialsBindings. This is no longer easily achieved by the "main" reconciler, because the reference to a credential resource is not immutable as it was in the SecretBinding API. Please see the linked issue below.
Which issue(s) this PR fixes:
Fixes #10160
Part of #9586
Special notes for your reviewer:
cc @rfranzke @vpnachev
Release note:

Labels and finalizers of `Secret`s and `WorkloadIdentity`s that are no longer referenced by a `CredentialsBinding` resource would be properly cleaned up. Please see [the documentation]( https://gardener.cloud/docs/gardener/concepts/controller-manager/#credentialsbinding-controller) for more details.

/retest

In an internal discussion we decided to try another approach w.r.t immutability in CredentialsBinding resources, hence I am closing this issue for now. I will share more details from the discussion and the new proposal in the upcoming days by updating https://github.com/gardener/gardener/issues/9586

/label ipcei/workload-identity


jupyter notebook output cells foreground colour control (notebook.outputContainerForegroundColor)

Since we have notebook.outputContainerBackgroundColor it would be cool to have
notebook.outputContainerForegroundColor

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Incorrect example at python readme

Hi,
I was executing the examples at the readme when I noticed a mistake.
print(py.eval('foo(a) + 10'), {a = 42})  -- prints 53
should be
print(py.eval('foo(a) + 10', {a = 42}))  -- prints 53
This will save future beginners (like me) some minutes ;)
Best,
Fréderic

thank you. fixed readme in commit: https://github.com/facebook/fblualib/commit/76e934fe42177451a03807940cc7a6de627f8dd1


[PyTorch] Mask R-CNN from Torchvision

https://pytorch.org/vision/master/_modules/torchvision/models/detection/mask_rcnn.html

For better efficiency, use box_detections_per_img option to limit maximum number of detected objects and rpn_post_nms_top_n_test to limit RPN's number of proposals.

@likholat, please review


Add Rambox 0.3.0

Adding a new cask

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

Thank you for the contribution. It needed some fixes, so they were made in 4e0641f13922d91f886776689c801f3bcd1aeb51. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

That was amazingly fast. Thank you!


[FX-5652] Migrate Dropdown to TailwindCSS

FX-5652
Description
This PR removes material-ui styles and replaces them with Tailwind CSS classes. It also adds the @mui/base package as a dependency to handle the import of the Click-Away Listener component. However, the @material-ui/core package will not be removed because Dropdown still uses the Grow component from mui@4 that will need to be migrated in this follow-up task.
How to test

Temploy
Check the production and compare it with the temploy.

Development checks

[x] Add changeset according to guidelines (if needed)
~Read CONTRIBUTING.md and Component API principles~
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
~Annotate all props in component with documentation~
~Create examples for component~
~Ensure that deployed demo has expected results and good examples~
~Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.~
[x] Self reviewed
~Covered with tests (visual tests included)~

Breaking change

~codemod is created and showcased in the changeset~
~test alpha package of Picasso in StaffPortal~

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers


Add error messages for SSL/web errors

Fixes #9754
HTTPS support was already working but the browser swallows any sort of certificate error. Added a wiki page for when a remote connect fails in the web extension:
https://github.com/microsoft/vscode-jupyter/wiki/Connecting-to-a-remote-Jupyter-server-from-vscode.dev

Codecov Report

Merging #9972 (8e8009e) into main (140fa6b) will increase coverage by 0%.
The diff coverage is 85%.

:exclamation: Current head 8e8009e differs from pull request most recent head a577260. Consider uploading reports for the commit a577260 to get more accurate results

@@         Coverage Diff          @@
##           main   #9972   +/-   ##
====================================
  Coverage    63%     63%
====================================
  Files       207     208    +1
  Lines      9890    9907   +17
  Branches   1571    1572    +1
====================================
+ Hits       6293    6307   +14
- Misses     3091    3093    +2
- Partials    506     507    +1

Impacted Files
Coverage Δ

src/platform/logging/outputCommandListener.ts
81% <81%> (ø)

src/platform/common/globalActivation.ts
92% <83%> (-2%)
:arrow_down:

src/platform/common/utils/localize.ts
98% <100%> (+<1%)
:arrow_up:

src/platform/serviceRegistry.node.ts
98% <100%> (+<1%)
:arrow_up:


parse_stream gives incompatible type "IO[bytes]"; expected "BinaryIO"

I'm parsing a stream from a zip file, and mypy shows the following error
Argument 1 to "parse_stream" of "Construct" has incompatible type "IO[bytes]"; expected "BinaryIO"  [arg-type] (11:32)
I believe that parse_stream should also be happy with IO[bytes] - see https://github.com/python/typeshed/issues/7843 for some discussion - but I'm a little out of my depth on this part of Python typing.
The following examples will reproduce this mypy error
from zipfile import ZipFile

from construct import Struct, GreedyBytes

s = Struct(
    'test' / GreedyBytes
)

with ZipFile('spam.zip') as myzip:
    with myzip.open('eggs.txt') as myfile:
        parsed = s.parse_stream(myfile)

Changing StreamType = t.BinaryIO to StreamType = t.Union[t.BinaryIO, t.IO[bytes]] in construct-stubs/core.pyi resolves this.
I'm happy to create a PR if that seems like the appropriate solution
Thanks so much for construct-typing :)

Thanks for the report and sorry for the late reply. I fixed this in v0.5.6.

Great. Thanks @timrid


Guide to the wiring

Hey,
This project looks awesome and am keen to try it myself. I however pretty much only have software skills and little to no hardware skills. It would be helpful if you could  outline the general wiring for the project, e.g. Connecting the mouse, relay/mechanical key wiring, LEDs, what you soldered etc.
Just a simple outline would be great as I can learn on the fly and fill in the gaps myself. Just struggling to get a start and assuming i am just missing the basics.
Thanks!
My disc is jordn24#5777 if you'd rather just add me there.

I'll complete the guide with some infos about that.
Here are some examples until then:
https://www.electronicshub.org/control-a-relay-using-raspberry-pi/ (instead of the light bulb connect to the pins of the clicking module inside the mouse)
https://raspberrypihq.com/use-a-push-button-with-raspberry-pi-gpio/ (same thing as a mechanical keyboard switch)
https://thepihut.com/blogs/raspberry-pi-tutorials/27968772-turning-on-an-led-with-your-raspberry-pis-gpio-pins (LEDs are optional, maybe do that last)
I added you in case you have specific questions, it might take a while until I have a proper guide.


Merge rc1.0.5 into master

Grant has updated lccal and provided updated documentation for ceph_tools, so rc1.0.5 is ready to be merged into master.

With the documentaion and lccal.py updated, I am going to merge rc1.0.5 into master.


Support slice of files and includes

Something that would be nice would be to allow includes(incs: &[PathBuf]) and something similar for files instead of just one include dir and one file at a time.
While it's not that hard to write
let some_include_dirs = [
  "dir0", "dir1", "dir2"
];

for dir in &some_include_dirs {
    build.include(dir);
}

If the user could supply an array/slice directly it would reduce the code a bit.

Sounds reasonable to me to support this!

files was implemented in #205. I've opened #550 for includes after having looked for this myself.


Documentation: Mixed Mode Booting with systemd-boot

Problem
While attempting to install NixOS on an older Windows Tablet, I ran across the issue that NixOS does not provide any 32bit EFI files for a 64bit install of the OS, and this tablet requires 32bit EFI files, and I would like to run the 64bit version of NixOS. I found this section of the bootloader wiki, but the issue with this is that after GRUB 2.06, booting ia32 EFI's is broken. That wiki page that I just linked seems to indicate that booting ia32 EFI's is possible with systemd-boot, but I could not find any documentation on how to do that. I'm not even sure that NixOS has built systemd-boot with mixed mode even enabled (I also don't know if you even need to enable it). I'm just hoping for some guidance on how to get this to work.
I'll ping the systemd-boot maintainer here in case he has any input.
@JulienMalka
Checklist

[x] checked latest Nixpkgs manual (source) and latest NixOS manual (source)
[x] checked open documentation issues for possible duplicates
[x] checked open documentation pull requests for possible solutions

Add a :+1: reaction to issues you find important.

This usecase is not supported currently with systemd-boot. From my understanding, installing systemd-boot from a 32bit bootctl version would do what you want but getting there might requires ugly hacks.

Oh, so is this implying that you need to do some workarounds to make it work? I thought it meant that systemd-boot supports it natively without any workarounds. I think I read that Arch is building with 32 and 64 efi's and systemd-boot, I wonder how they're doing it.


Support z.date()

Is your feature request related to a problem? Please describe.
Yes. The panel doesn't just like show a fallback text input, but just completely hides the option to fill in this field.
Describe the solution you'd like
Implementation of the z.date() type in TRPC-ui.
btw sorry if this sounds really negative haha, loving the project thusfar

This would be useful. I'll get on it after fixing #27. At some point I will try and implement most of the zod types that would only be compatible with superjson.


Creation of links for testing on localhost and tool labs

Looking at the db-connection file in php-scripts folder, you can clearly see that there are 2 configurations for connecting to the DB. There is one for localhost and another for tool labs (deployment env). Also, if you look at the shortener.php in the root directory, you will see that;
$LINK = "tools.wmflabs.org/durl-shortener/shortener.php/" which is only valid on tool labs. But when used on localhost, we need something like this:
$LINK = "localhost:3000/shortener.php" where 3000 is the localhost deployment port. So check files that have link that point to tool labs and create a local host version.
You can decide to make the tool labs link variable like this: $LINK_TL and the localhost variable like this: $LINK_LH.

Links have been handled both on localhost and tool labs.


Implement first-class continuations

See "A Theory of Substructural Types and Control".

Link is broken. PDF attached.
CtlURAL.pdf


Update README.md

Empty file is not good

Please open an issue here for that.  I agree empty isn't preferable.  :)

I changed it so its a more positive inviting message.  But like I said please open an issue for missing documentation.


Update apply-updates-and-hotfixes.md

Add important hotfixes.

Docs Build status updates of commit 2e4fa4b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamicsax2012-technet/apply-updates-and-hotfixes.md
:white_check_mark:Succeeded
View

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

Are you able to get these updates into tables?

Docs Build status updates of commit 81de08f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamicsax2012-technet/apply-updates-and-hotfixes.md
:white_check_mark:Succeeded
View

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


[now dev] Bump @now/build-utils to 0.9.10

[now dev] Bump @now/build-utils to 0.9.10

Codecov Report

:exclamation: No coverage uploaded for pull request base (canary@efe0ca5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             canary   #2715   +/-   ##
========================================
  Coverage          ?   8.27%
========================================
  Files             ?     261
  Lines             ?   10536
  Branches          ?    1345
========================================
  Hits              ?     872
  Misses            ?    9604
  Partials          ?      60

Continue to review full report at Codecov.

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


Add i18n research and demos

Criteria: i18n
Candidates:

[X] angular
[ ] react
[X] vue.js

Vuejs code samples added for the vue-i18n plugin.

Will be looking into Angular support. Appears to have good support out of the box, but there are wrappers/extensions to make it better. Want to understand better what needs improving in the default implementation.

Something to consider: can Date and Text content be translated separately from number/currency localization?
Since we display money values based on "selected" currency, or base currency of individual products, we would not want to automatically format and change currency symbols on all money values.

Angular code samples added for out-of-the-box compile-time and lazy-loading with ngx-translate.

Guess i'll take a look at React support too


general educational fixes in some files

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

There is a whole lot of whitespace changes, not sure if this is intentional. If yes would like some context.

Tbh it is unintentional. My text-editor messed up. I tried my best to fix the indentation in code blocks.

No worries, we would need you to undo those. If you need help with this let us know or reach us in the Contributors chat.

I think I need help. I'm messing it more trying to fix it.

I think I need help. I'm messing it more trying to fix it.

No worries, you should reach on the forum or the chat here: https://chat.freecodecamp.org/channel/contributors

@giridhar7632 if you are still having errors with resolving these conflicts and rendering bugs, let me know - I would be happy to assist.

@nhcarrigan I was hoping to undo the unintentional formatting and resolve the conflicts, but please feel free to beat me to it. I have been working on other infra stuff for the moment.

@giridhar7632 I've gone ahead and given this a rebase, and reverted the spacing changes to allow for a cleaner diff tree. Also, it appears that the rendering issues are resolved. 🙂

I tried to fix all the unintentional white space changes today 😁😅

@giridhar7632 You should really be testing these changes locally by building the docs and previewing the files.  You have added some addition whitespace on lines that were originally blank lines.  @nhcarrigan already fixed the whitespace issues for you earlier.  @nhcarrigan I think we possibly need to revert @giridhar7632's last commit.

Sorry 😅


Plugin Manager - False "Last version"

There is a sort of problem in the version numbers. Sometimes the proposed version is not the latest version.
Exemple with TernJs.
Last version in LightTable : 0.1.9
Real last version : 0.1.10
Metadata repository is correct.

@Armaklan Thanks for the report. This should be fixed by my PR. Hopefully it will be included in the next release.

@Armaklan This should be fixed on master.


Default to user_config.current_lang when auto_language is 'document' and document.documentElement.lang is not set

Expected behavior
When using auto_language: 'document' and document.documentElement.lang is not set (<html> without lang) default to user_config.current_lang before defaulting to _config.current_lang.
Example:
<html> <!-- without lang -->

const cc = initCookieConsent();

 cc.run({
  current_lang: 'pt-BR',
  auto_language: 'document',
  languages: {
    'pt-BR': {/*...*/},
    'en': {/*...*/},
  },
})

Current behavior
When using auto_language: 'document' and document.documentElement.lang is not set current_lang has no effect.

Hi @rayronvictor,
thanks for taking your time to create the issue + pr. Unfortunately, I don't plan on releasing new v2.x.x versions.
A better, broader approach is to check if the value returned from document.documentElement.lang (which in this case is an empty string '') is defined as a property key of the language's object. If it is not, use default language. This was implemented in v3.0-beta.


yum needs-restarting -r or needs-restarting -r not available in RHEL 8.0

So the reboot required detection fails on RHEL 8.0

See also https://access.redhat.com/solutions/3847481 .

solved by pull request #106 .


[TC-192] Java TM and Go TM get build with exact same RPM Name

When building Traffic Control, goTM and javaTM get build with the exact same rpm name.  This means that if you are building all projects you only get one TM RPM and its whichever one built last.  The names should be updated so that they are not the same.
Author: David Neuman
JIRA Link: https://issues.apache.org/jira/browse/TC-192
Found Version: 2.1.0

moving to 2.2.0.  There is a proposal to remove java TM which will solve this issue.


Typo with midiSRD-style.css

Module.json has
"styles" : ["scripts/midiSRD-styles.css"],
but the file is scripts/midiSRD-style.css
Note the non plural version ie missing 's'.

+1. Prevents installation of the module via Foundry module manager.

If anyone needs a temporary installation manifest for v10, until this is fixed: https://github.com/thatlonelybugbear/midi-srd/releases/tag/10.0.3


yolo instance segmentation format?

Hi again,
Thanks for implementing annotation generation scripts.
After running the code to generate the .txt mask label, I tried to import image/label/labelmap to see if the annotation is generated well.
However, it seemed like the annotation is somewhat messed up.
I checked with Roboflow tool, and could see the image like below.

Have you tried to train the yolo model with the generated labels?
Thank you so much,

@mhyeonsoo yes i tried it able to train the model also? are you still getting issue?

@akashAD98
Yes, still getting an issue. Can you show me a sample txt file for generated annotation?

I'll try with that script. Thanks!

@mhyeonsoo try it & let me know if you get any problems. im closing this issue right now ,open it agin if you get issue


build: create gh actions workflow

Pull Request Test Coverage Report for Build 527

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 87.879%

Totals

Change from base Build 519:
0.0%

Covered Lines:
27

Relevant Lines:
29

💛  - Coveralls


[Bug]: Test bug

Contact Details
No response
Tell us the project / group you are associated with
Community (Default)
What happened?
A bug happened!
Describe the expected behavior
No response
What version are you running?
No response
How critical is this bug to you?
Blocker - solution is unusable
How can we recreate the bug?
No response
Any debug data that you are able to share?
No response
What is your operating system?
No response
Any additional / relevant info
No response

New JIRA Created with ID: https://jira.eng.vmware.com/browse/NPT-694


Amplitude progress bar not working with stream

Issue description
Amplitude progress-bar selector not working well. I implement the docs.
Environment

What's My Browser Support link: https://www.whatsmybrowser.org/b/PTGHA

Amplitude.js Version: 5.0.2

Steps to reproduce the issue

angular mean stack project
file : player.component.html
create an account and add a music with the user interface (it's working perfecly)
route : localhost:4200/songs

What is expected?
the progress bar need to go to the percentage of the music when i click on her
Link to where issue can be reproduced
https://github.com/AlexisLapeze?tab=repositories : frontend and backed repo
Additional details / screenshots

no console logs, no output provided like everything working perfectly
video of the problem : https://streamable.com/hcd655

Thank you for your detailed bug report, that saves us a ton of time! 🙏
This will be something that @danpastori will have to answer. He is a bit busy, but hopefully he will have some free time soon!

no nbody else can looked at the bug ?

@AlexisLapeze If you configured from the sample, you might need to check your byte-range headers are sent back from your server. This allows you to navigate the audio.
I took a quick glance at the code, I'm not too familiar with Angular, but you write some of the headers here: https://github.com/AlexisLapeze/backend/blob/master/controllers/song.controller.js#L41. Make sure you don't overwrite the byte-range header for navigation.

Thank's for your response ! 🙂I change the operation of my website. I'm using an amazon S3 bucket now.
BTW, your player is awesome !! 😎


Arrange VinylController and VinylModel

Organize vinyl controller for search and pagination

Searching and Pagination were made on the side of Backend


The video of the dancing man

Could the author provide the video of the dancing man?

The little chicken is showing its black feet!

Could the author provide the video of the dancing man?

The little chicken is showing its black feet!

@ppanzx  your video is here:-
link 1:- https://github.com/showlab/Tune-A-Video/blob/main/data/car-turn.mp4
link 2:- https://github.com/showlab/Tune-A-Video/blob/main/data/man-skiing.mp4
link 3:- https://github.com/showlab/Tune-A-Video/blob/main/data/man-surfing.mp4
link 4:- https://github.com/showlab/Tune-A-Video/blob/main/data/rabbit-watermelon.mp4
Press download button to proceed further
thanks

6

Thank you! But the video I want is not in the links above.

@ppanzx your video is here

Thank you! But the video I want is not in the links above.

6

little black zi?

True ikun? 🐔 🐔 🐔
https://www.bilibili.com/video/BV1ct4y1n7t9/?spm_id_from=333.337.search-card.all.click


Private Registry support when creating swarm services

Is it possible to create a swarm service by providing the parameter "--with-registry-auth"? currently if I create the service it cannot pull the image from the private registry?

Sorry this actually works correctly when passing the AuthConfig into ServiceCreateParameters


[BUG REPORT] TypeError: 'rdcc_nbytes' is an invalid keyword argument for this function

Describe the bug
" TypeError: 'rdcc_nbytes' is an invalid keyword argument for this function" on saving an image to Hangar repository
Severity

Select an option:

[ ] Data Corruption / Loss of Any Kind
[  ] Unexpected Behavior, Exceptions or Error Thrown
[ ] Performance Bottleneck

To Reproduce
import hangar
repo=hangar.Repository('.')
repo.init(user_name='chandnika'<EMAIL_ADDRESS>co=repo.checkout(write=True)
import numpy as np
co.arraysets.init_arrayset('images' ,shape=(10,10,3) , dtype=np.uint8)
co.commit('arrayset init')
images=co.arraysets['images']
img=np.random.random((10,10,3)).astype(np.uint8)
images[0]=img
Expected behavior
TypeError                                 Traceback (most recent call last)
 in ()
----> 1 images[0]=img
5 frames
/content/hangar-py/src/hangar/arrayset.py in setitem(self, key, value)
500             sample name of the stored data (assuming operation was successful)
501         """
--> 502         self.add(value, key)
503         return key
504
/content/hangar-py/src/hangar/arrayset.py in add(self, data, name, **kwargs)
646             existingHashVal = self._hashTxn.get(hashKey, default=False)
647             if existingHashVal is False:
--> 648                 hashVal = self._fs[self._dflt_backend].write_data(data)
649                 self._hashTxn.put(hashKey, hashVal)
650                 self._stageHashTxn.put(hashKey, hashVal)
/content/hangar-py/src/hangar/backends/hdf5_00.py in write_data(self, array, remote_operation)
675                     self._create_schema(remote_operation=remote_operation)
676         else:
--> 677             self._create_schema(remote_operation=remote_operation)
678
679         srcSlc = None
/content/hangar-py/src/hangar/backends/hdf5_00.py in _create_schema(self, remote_operation)
540                                   rdcc_nbytes=rdcc_nbytes_val,
541                                   rdcc_w0=CHUNK_RDCC_W0,
--> 542                                   rdcc_nslots=rdcc_nslots_prime_val)
543         self.w_uid = uid
544         self.hNextPath = 0
/usr/local/lib/python3.6/dist-packages/h5py/_hl/files.py in init(self, name, mode, driver, libver, userblock_size, swmr, **kwds)
309             name = filename_encode(name)
310             with phil:
--> 311                 fapl = make_fapl(driver, libver, **kwds)
312                 fid = make_fid(name, mode, userblock_size, fapl, swmr=swmr)
313
/usr/local/lib/python3.6/dist-packages/h5py/_hl/files.py in make_fapl(driver, libver, **kwds)
107             msg = "'{key}' is an invalid keyword argument for this function"
108                   .format(key=next(iter(kwds)))
--> 109             raise TypeError(msg)
110         return plist
111
TypeError: 'rdcc_nbytes' is an invalid keyword argument for this function
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: Google Colab Debian based Linux
Python: Python 3.6.8
Hangar: 0.3.0

Additional context
Add any other context about the problem here.

Hi @ChandProj, thanks for the bug report. Sorry you are having issues!
Thanks for the detailed bug report, but I was unable to recreate the bug as you described it (and have previously tested hangar in colab).
I copy/pasted the example code you provided in a colab project, and it runs as you can see at this link.
https://colab.research.google.com/drive/1dUYURo18UzNHp_59Fd0gYjtnadEJiAlD
Did you do anything different? Is that code able to run for you?

Hi ,
Thanks for the response.
Intsead of pip install hangar , I did git clone hangar-py.git and ran python setup.py develop
The link to colab notebook : https://colab.research.google.com/drive/1EoGWwalUGOO4R875rKfBd67dyZPfGKiD
The link to

However, when I did pip install hangar, no issues exist.

Hey so I just tried it out, apparently google colab has an installation of h5py already available. However, it is version 2.8.0 and Hangar requires 2.9.0. If you retry your original notebook, but first !pip install h5py==2.9.0 everything works!


Don't display facecast field on character#show unless the character has one

I'm not really seeing why this change?


docs: fix link to moonshot-chat.adoc from comparison.adoc

thanks @dafriz

rebased and merged at e83bddfb37e67fd33053b8ec48036d4fd66c9494


Reload master configuration without restarting it

Describe the bug
It is not possible to reload the master's configuration without restarting it. While the master is offline, clients will experience errors.
For example, configuring storage backends needs a change in the configuration file.
System Setup

SeaweedFS 3.00 on Ubuntu 20.04
version 30GB 3.00 b1dac20c704cf5322c6c516e5f064eb00701aec9 linux amd64

Expected behavior
I can do some configuration changes, such as changing the storage backends (config actually only used by volume servers, but sent from the master) without making the cluster unavailable for a time.
Additional consideration

Maybe the master being unavailable is not that big of an issue?
Maybe running multiple masters is the intended way to support this? However restarting the master in this configuration still means a leader failover, which takes 10s.

@remram44 In terms of the life of a master in k8s where any change results in a restart of the container, this is unnecessary.

@kmlebedev Sorry, I don't understand

@kmlebedev Sorry, I don't understand

you don't really need a reload master

You do, I have checked this myself, for example if you change the [storage.backend] section.

Zero step https://github.com/seaweedfs/seaweedfs/pull/3755


Question on permissions

My scanner got a few new features in January, and now reported on today's release:
! repo/com.lagradost.quicknovel_54.apk declares sensitive permission(s):
 android.permission.READ_EXTERNAL_STORAGE android.permission.REQUEST_INSTALL_PACKAGES

While the storage one is easily explained, I wonder what apps a novel reader wants to install?

Oh, if you wonder about that DEPENDENCY_INFO_BLOCK, that's easily eliminated:
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains.

Ummm… Is the self-updater opt-in? And does it give proper explanation of the implications? From the IzzyOnDroid App Inclusion Criteria:

The app … must not download additional executable binary files (e.g. addons, auto-updates, etc.) without explicit user consent. Consent means it needs to be opt-in (it must not be harder to decline than to accept or presented in a way users are likely to press accept without reading) and structured in a way that clearly explains to users that they’re choosing to bypass the checks performed in this repo if they activate it.

(basically the same rule as F-Droid.org has). So while ideally it is not present at all, you need not necessarily to remove it altogether, but having it opt-in with the proper explanations would suffice.

Ummm… Is the self-updater opt-in? And does it give proper explanation of the implications? From the IzzyOnDroid App Inclusion Criteria:

If search update check is enabled (can be disabled/enabled in settings) it will search the github releases every startup for an update, and if it finds one it will show the user a dialog with the following options: update, cancel, dont_show_again. So the update is optional but it searches by default. It will never update without user consent.

While you are already here, the same goes for cs3 (https://apt.izzysoft.de/fdroid/index/apk/com.lagradost.cloudstream3) and cs3 removed acra by default

If search update check is enabled (can be disabled/enabled in settings) it will search the github releases every startup for an update, and if it finds one it will show the user a dialog with the following options: update, cancel, dont_show_again. So the update is optional but it searches by default. It will never update without user consent.

Hm, the check should also be opt-in. Let me give you two example screenshots from an app that recently implemented this:

The first popup occurs when no choice was made yet (e.g. right after the first start following the initial installation or update to the version that introduced this, ad by default the value for that setting is NULL). Both buttons use the very same design, so no "nudging". The second popup of course never appears if the check was disabled. It's still not ideal as it forgets to mention the implications (such updates would bypass the additional checks performed in the resp. F-Droid repo).

While you are already here

Thanks! I removed the Tracking AF right now (effective with the next sync around 7 pm UTC, as usual) and configured the scanner to not report it. As for "the same goes" – yes, then please carry over the same solution from here. It was not yet triggered for CS3 as most likely there was no update yet since those new features got added to my scanner. I didn't yet run a full scan on existing APKs as I still have more than enough to follow up by incoming updates :see_no_evil:

The warning popped up again today, so let me show you the new "screenshot 1" I just received:

@LagradOst sorry to be a bother, but I just received the warning once more:
! repo/com.lagradost.quicknovel_55.apk declares sensitive permission(s): android.permission.REQUEST_INSTALL_PACKAGES
! repo/com.lagradost.quicknovel_55.apk contains signature block blobs: 0x504b4453 (DEPENDENCY_INFO_BLOCK; GOOGLE)

So may I ask you again if the update check is disabled by default?
As for DEPENDENCY_INFO_BLOCK, that can easily be avoided with a minor adjustment to your build.gradle:
android {
    dependenciesInfo {
        // Disables dependency metadata when building APKs.
        includeInApk = false
        // Disables dependency metadata when building Android App Bundles.
        includeInBundle = false
    }
}

For some background: that BLOB is supposed to be just a binary representation of your app's dependency tree. But as it's encrypted with a public key belonging to Google, only Google can read it – and nobody else can even verify what it really contains. More details can be found e.g. here: Ramping up security: additional APK checks are in place with the IzzyOnDroid repo.

DEPENDENCY_INFO_BLOCK will be removed, just forgot. I also want to say that update check will not be disabled by default as github is the primary platform to download and update this app from, and this will not change.
However if you compile this app yourself I might be able to include a build config to change or remove the build in github updater on FDroid/IzzyOnDroid builds. Is there any environment variables or similar I can read in the build step to indicate that the computer is building the fdroid ver?

DEPENDENCY_INFO_BLOCK will be removed, just forgot.

Thanks!

I also want to say that update check will not be disabled by default

That's sad. Those downloads would bypass the extra checks implemented in the repo here.

However if you compile this app yourself

I don't; the IzzyOnDroid updater just fetches the APKs the developers provide. But if you could establish such a flavor, maybe you could also provide the resulting APK – in addition to the current one? That would also make direct updates possible, as that build then would be signed by the same key.

I don't; the IzzyOnDroid updater just fetches the APKs the developers provide. But if you could establish such a flavor, maybe you could also provide the resulting APK – in addition to the current one? That would also make direct updates possible, as that build then would be signed by the same key.

Bruh that makes no sense. If IzzyOnDroid only grabs the APK from this github repo then why is inapp updater wrong, they both serve the exact same APK in the end.

Because there are additional checks in place here. Folks get the results presented or, in the worst case, the APK is held back from being published.
I didn't say "in-app updater is wrong". It should be the conscious decision of those using the app. If I install an app from F-Droid or IzzyOnDroid, I expect the updates to come from there. People might think the update offered would be downloaded from the repo, and they might not be aware that the scans have been skipped.
Further, as it reaches out to Github by default, I might have to add the NonFreeNet anti-feature. Not so if it's opt-in.

@LagradOst the updater is still there, the permission still requested. It's half a year now, so we need to solve this. If we can't, we'll have to remove the app from IoD: a policy that's not enforced is useless. So which way to go here?

have a build flavor without the updater
make it clearly opt-in with informed consent
remove the app from IoD

(in order of my preference; removing the app I count as last resort if no choice is left).
I'll set myself a reminder there now to check back in about 4 weeks. By then we must have decided, sorry.

Read https://github.com/recloudstream/cloudstream/issues/1215. Remove QuickNovel and Cloud Stream 3, if the updated being turned off is critical, as github is the primary and only official source for both apps.

OK, as you wish. A specific build flavor would have done the job, proper hints too. A pity.

Both apps are removed now, effective with the next sync around 6 pm UTC. Well, almost exactly at their 3rd anniversary at IoD…
All the best then!


Find If a Value Exist in a Sorted Array

How could one expand the Binary Search in order to get a simple binary answer, does the input values exist in the given sorted array?
Thank You.

Take linear_search_sse function, replace _mm_cmplt_epi32 with _mm_cmpeq_epi32.
At the end of the function, return _mm_cvtsi128_si32(cnt); will return how many times given value occurs in the array.

What about a specific code for the case of no power of 2?
What about the existence for Binary Search?


Fix owner to fix 404 when trying to comment

Details
Fixes broken commenting on staging qa checklists like this: https://github.com/Expensify/App/runs/3424743113?check_suite_focus=true#step:3:44
Fixed Issues
N/A
Tests

Close a QA checklist with an item unchecked, verify it comments on the issue correctly

🚀 Deployed to production by @roryabraham in version: 1.0.88-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


replace deprecated golang/gomock with uber/gomock

Provide a description of what has been changed
Checklist

[ ] When introducing a new scaler, I agree with the scaling governance policy
[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Tests have been added
[x] Changelog has been updated and is aligned with our changelog requirements
[ ] A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
[ ] A PR is opened to update the documentation on (repo) (if applicable)
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #5440

Relates to #

/run-e2e internal

Could you solve the changelog conflict?

Could you solve the changelog conflict?

done!

/run-e2e

/run-e2e

/run-e2e

Only us can trigger the e2e tests because they use real infra 😄

/run-e2e

/run-e2e kafka


Appearance properties to adjust the header title

I needed a layout like shown at the screenshot. So I've implemented some configuration properties.

I need this too. Why wasn't it merged?

Oh dear god, Yesss!! Merge it soon please.

This feature would be amazing! Please merge it soon please.

Please merge this


ehcache 3.9.1 strange jaxb-runtime version dependency / compile error

after upgrade 3.9.0 -> 3.9.1 i get strange compile error:
Failed to collect dependencies at org.ehcache:ehcache:jar:3.9.1 -> org.glassfish.jaxb:jaxb-runtime:jar:2.3.0-b170127.1453 -> org.glassfish.jaxb:jaxb-core:jar:2.3.0-b170127.1453 -> javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438: Failed to read artifact descriptor for javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438: Could not transfer artifact javax.xml.bind:jaxb-api:pom:2.3.0-b161121.1438 from/to releases.java.net (http://maven.java.net/content/repositories/releases/): Transfer failed for http://maven.java.net/content/repositories/releases/javax/xml/bind/jaxb-api/2.3.0-b161121.1438/jaxb-api-2.3.0-b161121.1438.pom: PKIX path validation failed: java.security.cert.CertPathValidatorException: validity check failed: NotAfter: Fri Feb 19 13:00:00 CET 2021

In 3.9.1 the jaxb-runtime dependency changed from a specific org.glassfish.jaxb:jaxb-runtime:2.3.1 to the version range org.glassfish.jaxb:jaxb-runtime:[2.2,3). This means rather than terminating it's search upon finding 2.3.1 your build system will now search all configured repositories to find the latest available version that satisfies the range. This means it's now looking at your configured maven.java.net repository and it appears the SSL setup there is broken: https://www.ssllabs.com/ssltest/analyze.html?d=maven.java.net

that's true, but first i never used maven.java.net (i also grep -r "maven.java.net" in the whole project without result) and second why is that really old beta version choosed?

I don't think it's necessarily choosing that version. I think it's simply pulling the pom for that version as part of dependency resolution. Regarding why it's accessing maven.java.net, assuming this is a maven build system I would check the effective-pom: mvn help:effective-pom and see if you have anything in your ~./.m2/settings.xml that could be causing that.

Thanks for the help, i couldn't find anything about jaxb or java.net in that output too,
i mean i could always fix that via explicit version and/or excludes, just trying to understand.
The compile problem is gone, they fixed their ssl certificate.
And also that missing package is not added to the shaded jar, but too many different
jaxb variants via jackson, hibernate, restassured and ehcache (jaxb-api-2.3.1, jaxb-core-3.0.0-M5/jax-runtime-3.0.0M5,
jax-impl-2.3.3, and also jakarta.xml.bind-api-2.3.2).

If you're running on Java 8 (or on Java 9 and want to add the internal jaxb to the module path) then you can exclude the jaxb-runtime dependency. This is only here to avoid failures on Java 11 environments.

If you're running on Java 8 (or on Java 9 and want to add the internal jaxb to the module path) then you can exclude the jaxb-runtime dependency. This is only here to avoid failures on Java 11 environments.

Like so?
    <dependency>
      <groupId>org.ehcache</groupId>
      <artifactId>ehcache</artifactId>
      <exclusions>
        <exclusion>
          <groupId>org.glassfish.jaxb</groupId>
          <artifactId>jaxb-runtime</artifactId>
        </exclusion>
      </exclusions>
    </dependency>

Yes, that should work.

We have a similar issue with Ehcache 3.10.2:

Failed to collect dependencies at org.ehcache:ehcache:jar:3.10.2 -> org.glassfish.jaxb:jaxb-runtime:jar:2.3.0-b170127.1453 -> org.glassfish.jaxb:jaxb-core:jar:2.3.0-b170127.1453 -> javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438: Failed to read artifact descriptor for javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438: Could not transfer artifact javax.xml.bind:jaxb-api:pom:2.3.0-b161121.1438 from/to netbeans (http://bits.netbeans.org/nexus/content/groups/netbeans): Not authorized -> [Help 1]

We have analysed the root cause:
The effect that the Maven build is accessing repositories like http://maven.java.net or http://bits.netbeans.org/nexus/content/groups/netbeans is not caused by the local Maven configuration. In fact the parent POM of org.glassfish.jaxb:jaxb-runtime defines those repositories, for example see https://repo1.maven.org/maven2/com/sun/xml/bind/mvn/jaxb-parent/2.3.0/jaxb-parent-2.3.0.pom. This has been removed in a later version, but is still present in the old beta version 2.3.0-b170127.1453 which is now being checked.
@chrisdennis, you may be right that this old beta version will not actually be "chosen" in the end, but since it is defining some invalid repositories, the version range processing fails at this point.
A workaround which is also valid for Java 17 is to define an explicit version for org.glassfish.jaxb:jaxb-runtime.
Wouldn't it be a good idea to at least restrict the version range so that only versions without repository definitions are included?

This or a very similar error still happens for anyone relying on the latest version of CXF security that still supports the javax namespace instead of the jakarte namespace, specifically CXF 3.6.x (they all use ehcache 3.10.8):
Failed to collect dependencies at org.apache.cxf:cxf-rt-ws-security:jar:3.6.1
  -> org.ehcache:ehcache:jar:3.10.8
    -> org.glassfish.jaxb:jaxb-runtime:jar:2.3.0-b170127.1453
      -> org.glassfish.jaxb:jaxb-core:jar:2.3.0-b170127.1453
        -> javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438:
        Failed to read artifact descriptor for javax.xml.bind:jaxb-api:jar:2.3.0-b161121.1438:
        The following artifacts could not be resolved: javax.xml.bind:jaxb-api:pom:2.3.0-b161121.1438 (absent):
        Could not transfer artifact javax.xml.bind:jaxb-api:pom:2.3.0-b161121.1438 from/to maven-default-http-blocker (http://<IP_ADDRESS>/):
        Blocked mirror for repositories: [releases.java.net (http://maven.java.net/content/repositories/releases/, default, releases+snapshots), shapshots.java.net (http://maven.java.net/content/repositories/snapshots/, default, releases+snapshots), jvnet-nexus-staging (http://maven.java.net/content/repositories/staging/, default, releases+snapshots), netbeans (http://bits.netbeans.org/nexus/content/groups/netbeans, default, releases)]

Since CXF pulls in a different version of the jaxb-runtime (cxf-core directly requires jaxb-runtime 2.3.8) anyway, i think that it can be safely excluded in CXF security if there exist other CXF dependencies in the POM, like shown by previous commenters:
<dependency>
    <groupId>org.apache.cxf</groupId>
    <artifactId>cxf-rt-frontend-jaxws</artifactId>
    <version>${version.cxf}</version>
</dependency>
<dependency>
    <groupId>org.apache.cxf</groupId>
    <artifactId>cxf-rt-ws-security</artifactId>
    <version>${version.cxf}</version>
    <exclusions>
        <exclusion>
            <groupId>org.glassfish.jaxb</groupId>
            <artifactId>jaxb-runtime</artifactId>
        </exclusion>
    </exclusions>
</dependency>


Merging dev to main for documentation deadline

This merge includes

results page update
contributing.md
meeting notes(sprint related)
.prettier

Closed for naming change.


How to change the file name in the top header?

How to change the file name in the top header?
I used render file "url: file://${url}
and tried with field fileNameOptional and fileName but it not works.

it should be fixed in the master branch, i upload the npm library on weekend


AttributeError on scheduler

Describe the bug
Got this error in sentry from scheduler command dispatch scheduler start
AttributeErrordispatch.nlp in extract_terms_from_text
'spacy.tokens.span.Span' object has no attribute 'merge'

Full exceptions
AttributeError: 'spacy.tokens.span.Span' object has no attribute 'merge'
  File "dispatch/decorators.py", line 37, in wrapper
    result = func(*args, **kwargs)
  File "dispatch/incident/scheduled.py", line 69, in auto_tagger
    extracted_tags = list(set(extract_terms_from_text(text, matcher)))
  File "dispatch/nlp.py", line 43, in extract_terms_from_text
    token = doc[start:end].merge()

I believe https://github.com/Netflix/dispatch/pull/851 fixed this issue.


Create class for family income

Definition proposed by zhengj2007 in issue #62
family income: no definition yet (parent: IAO_0000030 information content entity)

Does this refer to the income of a family that lives together?

Yes this refers to a household (people living together). Could label / define as that instead of family if it helps clarify.


Adding query parameters to URL route?

Is it possible to pass additional query parameters on a route, and if so, how can we capture those parameters in tandem with the route parameters?
For example, if you have the route:
// search books
$router->get('/books/title/:title', 'Books@searchBooksByTitle');

Could append the URL to include some additional parameters like:
https://localhost/books/title/Harry?sort=publish_date
I am running into issues when I attempt that it says the requested route is not found, and I believe that is because its explicitly looking for a route to match with the query parameters on it.

@bradical1379 The issue is fixed. Now you can access your query params in the controllers:
$this->request->get('sort')

@bradical1379 The issue is fixed. Now you can access your query params in the controllers:
$this->request->get('sort')


Support config directory / split fogg.yml

Adopt Linux config directories pattern.
Add new field config_directory to the root config file, which will be used to read, parse and merge into a single fogg.yml config

fixed in https://github.com/vincenthsh/fogg/pull/218


Using Karibu together with Testbench in a pure Vaadin Flow App

Dear Martin,
I would like to use Karibu together with Testbench, but during test execution this exception is thrown:
java.lang.IllegalStateException: Polymer 3 webjar is on the classpath, indicating compatibility mode:
jar:file:/C:/Users/stefan.sator/.m2/repository/org/webjars/bowergithub/polymer/polymer/2.7.0/polymer-2.7.0.jar!/META-INF/resources/webjars/polymer/polymer.html. Karibu-Testing 1.2.x doesn't support Vaadin 14 Compatibility mode; please use Karibu-Testing 1.1.x instead. Alternatively, if you're not using compatibility mode, please exclude all webjars from your vaadin/vaadin-core dependency; please see the Skeleton Starter or Karibu10-helloworld-app on how to do that.
at com.github.mvysny.kaributesting.v10.VaadinMeta.checkNotVaadin14CompatMode(VaadinVersion.kt:122)
at com.github.mvysny.kaributesting.v10.VaadinMeta.isCompatibilityMode(VaadinVersion.kt:97)
at com.github.mvysny.kaributesting.v10.MockVaadin.setup(MockVaadin.kt:78)
at de.hahnunternehmensberatung.domain.main.ApplicationTest.setup(ApplicationTest.java:57)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:64)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:564)
at org.junit.platform.commons.util.ReflectionUtils.invokeMethod(ReflectionUtils.java:688)
at org.junit.jupiter.engine.execution.MethodInvocation.proceed(MethodInvocation.java:60)
at

This can be fixed be changing the pom like this and thereby remove the polymer jar:
<dependency>             <groupId>com.vaadin</groupId>             <artifactId>vaadin-testbench</artifactId>             <!-- <exclusions>             <exclusion>                 <groupId>*</groupId>                 <artifactId>polymer</artifactId>             </exclusion>             </exclusions> -->             <scope>test</scope>         </dependency>
But I suppose that Testbench will not work any longer.
Can you give me any hints what could be possible solutions to this? The app is a pure Flow app based on the vaadin starter.
If I understand this correct then polymer should be removed or is already removed on newer vaadin version. Maybe I just should update?
Currently I used:
<vaadin.version>14.6.5</vaadin.version> <groupId>com.vaadin</groupId>         <artifactId>vaadin-testbench</artifactId>         <version>14.6.5</version>
Best regards
Stefan

Hi Stefan, i think testbench should not depend on the polymer library and thus excluding polymer should not affect the testbench itself. Also polymer webjar was the old way of creating vaadin apps and is not used anymore, so it should be completely safe to exclude polymer. I can take a look at some point, but I'm on a vacation right now and trying to stay away from the computer 😁

There is a demo project for TestBench here: https://github.com/mvysny/testbench-simplest-demo unfortunately it's for Vaadin 8 and doesn't use Karibu-Testing. I can add KT there; and also I can create a Vaadin14 version of the project and also use TB and KT there as well, but it has to wait until the end of this month.

I've created the https://github.com/mvysny/testbench-simplest-demo-14 example project which demoes both TestBench and Karibu-Testing in the same project, please take a look and let me know what you think :+1:

Hello Martin,
I switched to Vaadin 20 already and therefor will not check this because I dont need it any longer.
Is your vacation already over?
Best regards
Stefan

Von: Martin Vysny @.>
Gesendet: Montag, 26. Juli 2021 13:35
An: mvysny/karibu-testing @.>
Cc: satorstefan @.>; Author @.>
Betreff: Re: [mvysny/karibu-testing] Using Karibu together with Testbench in a pure Vaadin Flow App (#78)
I've created the https://github.com/mvysny/testbench-simplest-demo-14https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmvysny%2Ftestbench-simplest-demo-14&data=04|01||82c311b2cc8342c63e8808d950298470|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637628961501658183|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Pu8dePw%2F7aUOX4QMUbHu3xBh4uPpnnJy0Ah%2FzT6fYy8%3D&reserved=0 example project which demoes both TestBench and Karibu-Testing in the same project, please take a look and let me know what you think 👍
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmvysny%2Fkaribu-testing%2Fissues%2F78%23issuecomment-886624139&data=04|01||82c311b2cc8342c63e8808d950298470|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637628961501668141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=PEvFG%2BskRfUgpHEE8OE8O3jRnUBsUCw9t8czdNCPlKo%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGDJJ4HUQQT6S242LC65ACDTZVCBJANCNFSM474FFVCA&data=04|01||82c311b2cc8342c63e8808d950298470|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637628961501668141|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2BODfl5Xci1BCiTcTcqf0m7%2FXQCtADV37zWFvC2PO2Bk%3D&reserved=0.


Using / for division is deprecated and will be removed in Dart Sass 2.0.0

Hi guys,
can you please fix the warning:

This is fixed in latest version.

@NicolasCARPi - in which version was this fixed?  I'm still seeing the error when using 1.13.18 and the 1.14-betas
Thanks!

It is fixed in dev branch. Hopefully a new beta will be released soonish.


Only wait for the AniDB ban protection if actually getting from AniDB

Rather than HTTP cache. I did an import as I'm migrating to SQL Server from MySQL, and it still waits, even though it's not actually pulling from AniDB. I checked that it wasn't pulling AniDB through my router. Because, well, last I checked, it doesn't take 2 seconds to retrieve a few kb from an ssd...

@maxpiva do you think you can look into this for your db cache pr? It'd make importing so much faster

The Cache is a 2nd level memory cache it stores common database queries and
entities in memory. So there is no round trip to the db. I don't think
it'll accelerate the import only marginally.
El Wednesday, May 18, 2016, da3dsoul<EMAIL_ADDRESS>escribió:

@maxpiva https://github.com/maxpiva do you think you can look into this
for your db cache pr? It'd make importing so much faster
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/japanesemediamanager/jmmserver/issues/252#issuecomment-219945499

When you import/rebuild database, it scans your AniDB HTTP  cache. The problem is that it waits the 2 seconds to prevent getting banned, even though it never attempted to pull from AniDB. In the 2 seconds it waits, it could be done with 10 more files. That is why it would make it faster. Only the first run after updating to your new system would be faster by much, but that's the one that'll take awhile. If you don't think it'll be a big deal, then okay.

Upon closer inspection, it still pulls using udp for file import, so this is not applicable


Add self observability metrics for GraphQL query

[x] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #. NO
[x] Update the CHANGES log.

We should add this to so11y dashboard.


Legal Content Changes

For the BCeID.csproj file - I had a conflict when I did a merge upstream/master. Could you please double check to make sure I resolved it correctly? (I don't work in that file at all, so please make whatever changes are necessary.)

We will need to ammend this pull request to exclude the BCeID files, and the .gitignore


Initialized BE API Project

#38  #54
BE API Project is created with basic code structure. Also the Dockerfile is also created.

Typo in the file DBRepositoryFactroy.cs.

Typo in the file DBRepositoryFactroy.cs.

Fixed in this PR #105


Update test: total_cummulative_cases

Updated the test as discussed. Please review and approve.

hi Valli, you may want to modify the code like my part.
Test correct data is returned when date is specified
df = ccm.total_cumulative_deaths(loc='BC', date='2021-01-18')
assert df['cumulative_deaths'].sum() == 1078
assert df['deaths'].sum() == 31


[windows] Right click menu appears on wrong monitor

Under Windows with multi-monitor support, the right-click menu doesn't appear at the mouse cursor's location, instead it appears on another monitor.

Set up multiple monitors.
Add a monitor to the left of the primary, giving it a negative X axis (alternatively, one above the primary, for a negative Y axis)
Move the window to this other monitor.
Right-click within the window to bring up the context menu
Menu appears on primary monitor, instead of monitor where the window is currently located

Hah, interesting bug. Thanks for erporting, we'll take a look at it!

Yeah! In the earlier days of multi-monitor support in Windows, this was a very common bug to see in countless applications. They just all assume that a negative coordinate is impossible (or they use unsigned integers for coordinates). Its been a while since I've seen this is a desktop application! Pretty easy to fix though usually.

Hm, I'm actually not able to reproduce this with Yubico Authenticator 4.2.0 in a Windows 10 VM. Did I understand correctly that this is what you did?
The context menu does appear on the rightmost (primary) display if the Yubico Authenticator window is straddling the border in such a way that the window peeks out less to the left than the width of the context menu, but it doesn't seem like that corner case was what you meant.

Closing this for now, please reopen when you respond!


Add security information

Scheduled commands have additional security concerns, if somebody may do something today doesn't necessarily mean they may a month from now. Also, when the command is executed, it's impossible to do validation based on logged on users. Documentation on these caveats and tips on how to avoid them should be added.

ef604d2 closed #d3


ServerInstances loading from Skungee folder

I bought the Addon ServerIntance for SKungee and get this error:
Could not load plugin from file plugins/Skungee/ServerInstances.jar
java.lang.NullPointerException
at net.md_5.bungee.api.plugin.PluginManager.detectPlugins(PluginManager.java:353)
at me.limeglass.skungee.bungeecord.serverinstances.Premium.check(Premium.java:21)
at me.limeglass.skungee.bungeecord.Skungee.onEnable(Skungee.java:56)
at net.md_5.bungee.api.plugin.PluginManager.enablePlugins(PluginManager.java:226)
at net.md_5.bungee.BungeeCord.start(BungeeCord.java:280)
at net.md_5.bungee.BungeeCordLauncher.main(BungeeCordLauncher.java:56)
at net.md_5.bungee.Bootstrap.main(Bootstrap.java:15)
Thank you :smile:

The ServerInstances.jar is not within the Skungee folder right? Make sure the only jar of ServerInstances.jar, is in the plugins folder on Bungeecord.


Window title area extends over boxy-style controls

Describe the bug
When using the boxy window controls, the image title text-area extends over the controls. This is hard to read, and doesn't look nice.
To Reproduce
Steps to reproduce the behavior:

In settings, change the window controls type to the last option (boxy).
Click towards the top of the window in the image.
Type, and watch the text run into the window controls.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Info (please complete the following information):

OS [e.g. macOS, Linux, Windows, iOS]: Linux (xubuntu 20.04)
Browser [e.g. chrome, safari]: chrome 83

@b3u would widening the container a bit  work for you?

You can do this by dragging the right edge of the Carbon container.

@b3u would widening the container a bit work for you?

I think that's a great temporary fix. By the way, the carbon container never lets go of my cursor. I have a gif of it. I'll open a new issue.
I was playing around with the styles. Adding a max-width to the input should eliminate the issue completely. If 54px is the set size of the window controls svg, then the input should have a max-width of calc(100% - 54px).
input {
    max-width: calc(100% - 54px);
}

@b3u feel free to open a PR with this change. If it's simple, and doesn't break existing styles, I'll probably merge it 🙂

@mfix22 I noticed that the contributing guidelines is doesn't show the complete command for adding a contribution. It says yarn contrib:add <your GitHub username> instead of yarn contrib:add <your GitHub username> <contribution>. Would you like me to open a new issue?

I added it 👍
No need to open a new issue for a change as small as that one, you can just open a PR for consideration instead.

@faraz16iqbal there have already been 2 PRs submitted for it, but they have not yet merged, so the issue is still open.


Strategies for reducing resource utilization by NFTs

NFTs impose a RAM burden on witness and API nodes by virtue of the fact that asset objects, including description field in which the NFT payload is embedded, are kept in RAM.  Each NFT will consume a larger than ordinary amount of RAM for this for all nodes, including block-producing witness nodes, which can otherwise run on lean hardware since they don't serve an API to clients.  Token creators should do their utmost to reduce the storage footprints of their artwork, however, a storage burden in some form is inescapable.
As a quick calculation, it would take 2,621 NFT's utilizing the entire 400KB max_transaction_size allotment to consume 1 GB of witness node RAM.   This is not actually a lot of NFT's, if this becomes popular.  At present, BP-only witness node consumes around 4GB of RAM, so this would be a 25% increase for for BP's for a mere 2,621 NFTs.
Aside from offloading storage requirement by storing links to content instead of actual content data, which has a disadvantage of making the NFT reliant on hosting through, e.g., ipfs, one option to reduce the RAM hit might be to publish the NFT with full artwork, but then in a subsequent block to replace the nft_object field contents and replace with an object that refers to the block number in which the original publication occurred.  This means that witness and API nodes would no longer need to keep the full NFT object in RAM (they would instead keep the truncated form), and yet the full data would still be in the blockchain.  A viewer wishing to retrieve the data would no longer find it in the asset description, but would be directed to retrieve the relevant block instead, and to extract the data from there.  In this scheme, the NFT signature would still be in the asset description, but the NFT object would not.  However, it would be retrievable without having to rely on off-chain storage.

Some questions to pursue:

Are asset descriptions in fact stored in RAM?

Are asset description important to consensus?   My suspicion is they are irrelevant to consensus.

If they are not important to consensus, this would mean that they could be dropped from RAM for non-API (e.g. BP) nodes, thus reducing the RAM burden without having to implement a scheme of replacing the description with a truncated form.  This would be good for BP nodes.  However API nodes, including end-user personal nodes, would still need to store this data in RAM, although these nodes are already expected to have higher requirements due to storing history.

Would implementing disk cacheing for "large" description fields be a solution?  (As in, a simpler or more elegant solution than establishing a convention of removing payload from NFT objects and replacing with ref to block:tx:op.)

@abitmore — would be particularly interested in your commentary here.

Thank you @christophersanborn. My comments are below.

Some questions to pursue:

Are asset descriptions in fact stored in RAM?

Yes.

Are asset description important to consensus?   My suspicion is they are irrelevant to consensus.

Not very important IMHO.

If they are not important to consensus, this would mean that they could be dropped from RAM for non-API (e.g. BP) nodes, thus reducing the RAM burden without having to implement a scheme of replacing the description with a truncated form.  This would be good for BP nodes.  However API nodes, including end-user personal nodes, would still need to store this data in RAM, although these nodes are already expected to have higher requirements due to storing history.

Correct.

Would implementing disk cacheing for "large" description fields be a solution?  (As in, a simpler or more elegant solution than establishing a convention of removing payload from NFT objects and replacing with ref to block:tx:op.)

Maybe. I think perhaps it's too early to think about this. Hardware is cheap nowadays. I think we will have time to react when it's getting serious.

Thank you for the information @abitmore. Instead of replacing the description after creation, we could keep the NFT data in the description as long as we can drop it from RAM on the BPs. I am guessing this would require a core change?

Yes, a change in core because the witnesses are running the core software. But it's not a hard fork. So it can be developed and deployed easily and quickly when needed.

As the traditional art world suggest, an NFT might enter the market at $10... over the course of time become worth $10,000,000.   This leaves the original sale by the artist of very low value, which fails to compensate the creator for their work.
To me the very purpose of NFT market is to reward creativity of artists; not to create a modern tulip market.
It would require BSIP/Hardfork... but:
My suggestion to the NFT protocol is that the original creator of the work is entitled to a fee that is percent of trade value every time the NFT changes hands.   That % fee should be set by the original creator and accrue indefinitely into his possession upon every subjective change in value upon transfer between NFT holders.

Few remarks:

Adding asset ownership and details to witnesses endorse violation of GDPR and other privacy laws.
Adding asset details to a witness directly impose legal responsibility towards witness and the data of someone who is 1:1 owner of an item.
Why not just store .json on the website.hosting for all Different implementations of NFT.  That case witness has no responsibility and KYC and rest that will come to that ownership part from regulators was judged already yesterday :)

Chee®s

What about encouraging uploading JSON to IPFS which describe the images/multimedia, to reduce the fees and the compute load? Rather than include 50+ strings in media_png_multihashes and potentially reaching the max blocksize, I could do media_json_multihash & the contents of the json at the multihash are then processed.
I believe this is what a lot of ETH projects do to reduce costs


Inlining styles makes all elements inside mj-raw tags lowercase

Describe the bug
When using <mj-style inline="inline"> all html elements inside <mj-raw> and <mj-table> have their tag names and parameters set to lowercase, even if they aren't related to the css selectors.
My intended workflow is using MJML to render a freemarker template, which our application can process to dynamically generate unique emails.  The freemarker tags include references to variables, so casing is important.
To Reproduce
Steps to reproduce the behavior:

Render the following MJML using the cli application, or the try-it-live editor:

<mjml>
   <mj-head>
    <mj-style inline="inline">
      .myClass{
        background: red;
      }
    </mj-style>
  </mj-head>
  <mj-body>
    <mj-raw>
      <#if someCheckHere()>

${someValueHere}

      </#if>
    </mj-raw>
  </mj-body>
</mjml>

Expected behavior
Body output is:

    <#if someCheckHere()>

${someValueHere}

    </#if>

Ideally, the styles would be inlined with the
 tag and the unrelated <#if> would remain unchanged.  Or, since it's inside an  mj-raw tag, everything including the
 would remain untouched.
Actual behavior
Body output is:

    <#if somecheckhere()="">

${someValueHere}

    </#if>

The style is applied, but the <#if> tag's attribute somecheckhere is lowercased
MJML environment (please complete the following information):

MacOS Mojave
mjml-core: 4.2.1
mjml-cli: 4.2.1

Might be related to juice rather than mjml here, @kmcb777 can you check there's an option on juice side ?

And I haven't seen that it has been converted to <#if somecheckhere()="">. There's a really low change that we'll be able to do anything on that part (even juice)  because it's a parser issue with htmlparser :/
Thanks for reporting this @wdonnell with an example 👍

Your comments set me on the right track.  juice has a configuration option called codeBlocks that let you specify which tags you want ignored when inlining css.  It includes a couple defaults.  I was able to get this working by editing the source in juice/lib/cheerio.js and adding in the freemarker tags:
module.exports.codeBlocks = {
  EJS: { start: "<%", end: "%>" },
  HBS: { start: "{{", end: "}}" },
  freeMarker: { start: "<#", end: "</#" }
};

Exposing this configuration as a parameter would be a more permanent solution.

Maybe open a PR on juice side so we can update juice version for the next release ?