IfNull method

DBFlow Version: 3.1.1
Issue Kind (Bug, Question, Feature): Feature
Please note if you are using Instant Run, there may be bugs where generated classes are not created. Ensure you are using
the apt or kapt plugins and that incremental compilation is off.
Description:
It would be great if you can add support for the IFNULL method. I was able to create a new Method on-the-fly for IFNULL below. We could support this method by default in DBFlow.
new Method ("IFNULL", [property], [property])).as ([alias])

I also need to use IfNull method. How can I do this?

in 4.0.0-beta4

Awesome!!!! I have more coming as well. 😉
On Sat, Dec 31, 2016, 3:46 PM Andrew Grosner<EMAIL_ADDRESS>wrote:

in 4.0.0-beta4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Raizlabs/DBFlow/issues/911#issuecomment-269881310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7C_Z0lJ9cKUx40QezhBpCSnhr37BDaks5rNr8QgaJpZM4JKwSc
.

Also, do you want me to write usage file for the loaders?
On Sat, Dec 31, 2016, 3:47 PM James Hill<EMAIL_ADDRESS>wrote:

Awesome!!!! I have more coming as well. 😉
On Sat, Dec 31, 2016, 3:46 PM Andrew Grosner<EMAIL_ADDRESS>wrote:
in 4.0.0-beta4
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Raizlabs/DBFlow/issues/911#issuecomment-269881310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB7C_Z0lJ9cKUx40QezhBpCSnhr37BDaks5rNr8QgaJpZM4JKwSc
.


Added Heap sort in C language

heap sort is already covered in c++. Moreover, PR's coded in c are now not being merged.

Can you please let me know why you have tagged me as spam?

Please remove the spam label.

@manan-shxrma remove the spam label


How do I disable pageable sort param from showing in docs?

Version: 2.9.2
What kind of issue is this?

[x] Question.

How do I disable pageable sort param from showing in docs?
I am using Cassandra database, where sorting doesn't apply like with relational databases, so I want to use page and size only with pageable param. How do I do this?

Interesting, are you using spring data JPA/rest?


500 Internal Server Error when docker push localhost image for windows-amd64.debug

repro on : arm64-preview
1.set python version in VSCode settings
"azure-iot-edge.version.pythonmodule": "arm64-preview"
2.create python module
3.build and Push python module
received unexpected HTTP status: 500 Internal Server Error when  push localhost:5000/samplemodulepython:0.0.1-windows-amd64.debug
does windows docker image support localhost?

python module can't be connected when run solution in simulator

It's expected. Windows container does not have local registry.


cfg(doc) doesn't seem to be passed down to dependencies

I have in cairo-sys the following:
#[cfg(any(feature = "pdf", all(not(doctest), doc)))]
pub type cairo_pdf_version_t = c_int;

And in cairo:
#[cfg(any(feature = "pdf", all(not(doctest), doc)))]
#[doc(hidden)]
impl From<PdfVersion> for ffi::cairo_pdf_version_t {
    fn from(val: PdfVersion) -> ffi::cairo_pdf_version_t {
        match val {
            PdfVersion::_1_4 => ffi::PDF_VERSION__1_4,
            PdfVersion::_1_5 => ffi::PDF_VERSION__1_5,
            PdfVersion::__Unknown(value) => value,
        }
    }
}

But when running cargo doc, I get:
error[E0412]: cannot find type `cairo_pdf_version_t` in crate `ffi`
    --> cairo/src/enums.rs:1565:32
     |
1565 | impl From<PdfVersion> for ffi::cairo_pdf_version_t {
     |                                ^^^^^^^^^^^^^^^^^^^ not found in `ffi`

So I assume that the cfg(doc) isn't passed down to dependencies, contrary to what's expected.
Extra info wih cargo -v:
     Running `rustdoc --edition=2018 --crate-type lib --crate-name cairo_sys cairo/sys/src/lib.rs -o /home/imperio/rust/gtk-rs/target/doc --cfg 'feature="freetype"' --cfg 'feature="glib_ffi"' --cfg 'feature="use_glib"' --error-format=json --json=diagnostic-rendered-ansi -L dependency=/home/imperio/rust/gtk-rs/target/debug/deps --extern glib_ffi=/home/imperio/rust/gtk-rs/target/debug/deps/libglib_sys-1f466c66096ae0c3.rmeta --extern libc=/home/imperio/rust/gtk-rs/target/debug/deps/liblibc-87e5377408253f4a.rmeta --crate-version 0.13.0`

It was already discussed in https://github.com/rust-lang/rust/issues/76252#issuecomment-685825534

It was already discussed in rust-lang/rust#76252 (comment)

And dup of issue #8811 ?

The problem seems similar but not sure if it's exactly the same...

Can you clarify what appears to be different?  AFAICT, it is the same, that dependencies do not set --cfg doc.

Then it's the same. Closing this one then!


Add installation of sshd service.

This includes a copy of sshd (and support tools) and a script to install it into the MIB2 unit.
It should be possible to run from the Startup section of green screen toolbox menu - though this hasn't been tested yet.
Alternatively it can be run from console: on -f mmx ksh /fs/sda0/Toolbox/scripts/sshd_install.sh
If a sshd rsa login public key is on the SD card in Custom/id_rsa.pub then it will be copied onto the unit to allow password-less login of the root user.
ssh login will then be available on wifi connections after a unit reboot.
Behind the scenes the service is configured to be run by inetd, the same as the existing telnet service.

We should also add this new path into the uninstall script. So if someone wants to delete the Toolbox this should also be deleted.

Good idea

I believe this is finished now, working well. I've tested on my standalone unit configured through GUI to be wifi client.


PLT-623: verify cost model page accuracy

Deleted cross-reference to technical report that doesn't yet contain cost model information.
I've verified the accuracy of this page with Plutus Core team members. The only change is to remove a cross-reference from the top part of the page that points to the technical report that does not yet contain any information about the cost model.

My recommendation is to remove it, and if someone complains and provides a legitimate reason, we can then implement and put up an auto-generated version.

We had this conversation a few times, and Joseph says that Kevin wants it. I also think we should delete it.

But removing the cross-reference is fine.

We had this conversation a few times, and Joseph says that Kevin wants it.

I was suggesting that we remove it in spite of that. And bring and automated version if someone really complains with a good use case.
If someone actually needs it, having a version up that was last updated in June and that we don't know whether it's accurate or not can be more problematic.


[Bug]: Exception Occurred while executing   ScriptHookVDotNet.asi (0x00007FFFE19625B0), id 44

What modules are you seeing the problem on?
ScriptHookVDotNet.asi (runtime)
SHVDN Version
d955e2a
Game Version
v1.0.3095.0
Bug Description
CORE: An exception occurred while executing
"ScriptHookVDotNet.asi (0x00007FFFE19625B0), id 44
Press OKto continue.
Steps To Reproduce
1.) Just installed Script hookv and nightly vdotnet, and simple hud and lemon Ui, when i launch LSPDFR it says the error code.
Link To Code Example
No response
The Current Behavior
When i go to launch LSPDFR it says CORE: An exception occurred while executing
"ScriptHookVDotNet.asi (0x00007FFFE19625B0), id 44
Press OKto continue.
then simplyhud wont work
The Expected Behavior
To load ScriptHookVDotNet.asi

Probably the same bug as #1401, but I'll leave this open for now as I'm not completely sure.

so in his instance he said he disabled the upscaler? Im not running a graphics pack atm i deleted it and redid everything after for lspdfr again and still getting this error with ScriptHookVDotNet.asi erroring.

so in his instance he said he disabled the upscaler? Im not running a graphics pack atm i deleted it and redid everything after for lspdfr again and still getting this error with ScriptHookVDotNet.asi erroring.

I don't think the upscaler is not really relevant now, even if removing it happened to stop causing the error in his game. I believe there's still other multithreading bugs, but ones that involves data races or race conditions are non-deterministic and I haven't reproduced this issue once (this doesn't prove there's no bugs involving them though)…

Yeah i got it orginally, so i deleted GTA complelety and just resintalled and added LSPDFR and SimpleHud and error still remains, so with that any idea on how to fix the issue, the very first time i started it worked then when i closed and reopened its been this error ever since.

i installed NVE and then it started randomly working and then once i reopened the game right after it got the same error.

I've got the exact same issue. Tried both Nightly-60 and -59. The game worked just fine the first time I launched it with Nightly-60 and ever since then I keep getting the repeat of the error during loading:
Although the exception code is slightly different in numbers, no idea if that matters:
[21:40:12] CORE: An exception occurred while executing 'ScriptHookVDotNet.asi' (0x00007FF9B21B2680), id 38

Alright, just quick update. I was suggested at RAGE Discord to try nightly-37 and that one worked, not sure if this bit of info helps you in any way though.

Alright, just quick update. I was suggested at RAGE Discord to try nightly-37 and that one worked, not sure if this bit of info helps you in any way though.

37 didn't do anything related to the core. In fact, the majority of commits around 37 were all related to documentation, not code. As a result, you should be able to run Nightly-44 just as easily as Nightly-37. Nightly-45 introduced the first Core update since -37, so my guess is that's where you'll actually see a difference.
Feel free to test it yourself.

I believe this issue is already resolved as how #1454 got resolved.


Merge from https://github.com/EOS-distribution-collective/genesis

https://github.com/EOS-distribution-collective/genesis

merged 👍 please @dskvr if you see something wrong, report 👍


[BEAM-10480] Add splittable dofn as the recommended way of building connectors.

In this PR, I deleted the guide to use BoundedSource and UnboundedSource to develop IO and added the guide to use Splittable DoFn.
r: @rosetn
cc: @robertwb @chamikaramj @aaltay

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Updated the PR with suggestions.


Update ksonnet-lib printer to 0.1.10 to handle unescaping newlines

#670
Signed-off-by: bryanl<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 1098

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

Totals

Change from base Build 1095:
0.0%

Covered Lines:
11002

Relevant Lines:
15459

💛  - Coveralls


Use TensorDim for databuffer

change databuffer class to use TensorDim Class for input
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: jijoong.moon<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #111. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ec2-54-180-96-14.ap-northeast-2.compute.amazonaws.com/.


Get reactions.js to 100% line coverage.

Automated message from Dropbox CLA bot
@showell, it looks like you've already signed the Dropbox CLA. Thanks!


Update virtual-networks-vnet-faq-include.md

With the launch of ASE (App Service Environment) it is possible to launch Web Apps inside a VNet

Hi @rhossi, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

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

TTYL, AZPRBOT;

Resolved merge conflict in https://github.com/Azure/azure-content/pull/4686 - I've passed that PR along for review to the networking content team.


Update of branch io sdk for Xamarin-IOS error

We tried to update your nuget package of Branchio sdk link for Xamarin. Its build up fine on android, but on iOS it shows following compile time error:
Error MT2101: Can't resolve the reference 'BranchUniversalObject', referenced from the method 'System.Void AgriSync.App::InitSessionComplete(BranchUniversalObject,BranchLinkProperties)'
nuget version: https://www.nuget.org/packages/Branch-Xamarin-Linking-SDK
Branch-Xamarin-Linking-SDK -Version 4.0.3
So any thought on how to solve this?

@softsan , which version did you use before updating?

3.0.18

please update xamarin sdk


[numpy] array ufunc and array function protocols

Description
This PR implemented NumPy's array ufunc and array function protocols so that mxnet.numpy.ndarray can be fed into a subset of official NumPy operators and get dispatched to the corresponding MXNet operators for execution. See the following for examples.
import os
# this is required for NumPy version >= 1.16 while < 1.17
# In 1.17, this defaults to 1.
os.environ['NUMPY_EXPERIMENTAL_ARRAY_FUNCTION'] = '1'

import numpy as onp
from mxnet import np as mx_np

a = mx_np.random.uniform(size=(1, 3))
b = mx_np.random.uniform(size=(2, 1))
assert type(onp.concatenate([a, a])) == mx_np.ndarray
assert type(onp.sum(a)) == mx_np.ndarray

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

@szha This PR is ready for merge if there are no more comments. Thanks.

LGTM. @leezu would you mind taking a look too?

@leezu Thanks for the review. I tried the commands you showed, but it works for me. Could you confirm that you are using NumPy with version >= 1.17?
>>> from mxnet import npx, np
>>> a = np.arange(10)
>>> npx.set_np()
>>> a = np.arange(10)
>>> import numpy as onp
>>> print(onp.__version__)
1.17.1
>>> onp.dot(a, a)
array(285.)
>>> type(onp.dot(a, a))
<class 'mxnet.numpy.ndarray'>

@leezu Re: Can test be improved? Yes, absolutely. This PR is for building the skeleton of testing suite. It enables mxnet.numpy operator developers to easily add test cases from the official NumPy package to MXNet (OpArgMngr in test_numpy_interoperatibility.py is for storing those test cases. We will send in those test cases in the follow-up PRs.

@leezu Could you confirm that you are using NumPy with version >= 1.17?

You're right, I was using numpy 1.16 and it ran into the well-known issue (I guess there is some issue tracking it, but not sure) that numpy doesn't correctly convert ndarrays to numpy arrays.
Would it make sense to support Numpy 1.16 when NUMPY_EXPERIMENTAL_ARRAY_FUNCTION=1 environment variables is defined? If not, how about bumping the required minimum numpy version in setup.py to 1.17?
https://github.com/apache/incubator-mxnet/blob/e87995db6f8ac7e8d4424001e1db5296398562f6/python/setup.py#L33

@leezu I prefer to bump the required version to 1.17 to use this feature for two reasons. However, I'm not sure if we should bump the numpy version requirement in this or a separate PR. Maybe @szha can provide some insights.

This functionality was claimed as experimental in 1.16. There might be bugs that we are not aware of, and consequently would have to deal with unexpected behaviors. I tried using numpy 1.16 during the development. However, there is some issue with NumPy correctly dispatching calls to MXNet even when the environment variable is set (I printed the environ value and verified it's 1) using nosetests. In 1.17, there is no such problem.

Asking users to set the environment variable manually to enable this functionality may open the door to unknown issues, especially under multi-threaded circumstances (the one I saw may be related to this). To close the possibilities of using it incorrectly and make the user experience straightforward without surprises, we can stick with numpy versions >= 1.17.

The downside of upgrading is the potential conflict of the numpy dependency version between mxnet’s and other packages’. Shall we just produce proper error message for older numpy versions instead?

I will add proper error message when it is used with numpy 1.16 versions. Thanks.

@leezu I'm referring to the situation where another package on an end user's environment declares a dependency on numpy 1.16, in which case the control of upgrading numpy isn't in the hands of the end user.

This seems quite hypothetical / uncommon to me. I'm not convinced if we need to take consideration of this, given the user experience impact of not declaring the correct numpy version dependency and our current strategy of throwing runtime errors or having numpy throw errors such as ValueError: dot: too many dimensions in result (which is not very helpful for users).
I don't want to block this PR though.

@leezu @szha How about we merge this PR first and later consider if it's necessary to bump the numpy version to 1.1?

I'm OK with bumping the version and merging the PR. The error message during installation is sufficiently clear.


[Reddit Premium Feature] Support reddit themes / subreddit css

A limitation of using a WebExtension is that WebExtensions are not widely adopted on mobile. Using Reddit themes can be a possible workaround.
While I did not tested it, (I don't have premium) In theory it allows using a subreddit's theme globally except the user preferences page.
I am wondering if it's viable to make a css-only variant of this addon. I did a test and here are the limitations using this approach:

No javascript: It may be impossible to implement some of the functionality this WebEx does using only CSS.
No custom fonts: Reddit does not allow @font-face. This means no fontawesome and we must use png sheet for icons.
No vars: Reddit does not support css vars: unknown function "var". This is not a big issue since we can preproccess css.

There are 2 extra issues on top of that:

Addon requires viewport meta HTML code, without it website will look like a desktop site
Sidebar is handled via JS, not sure how can this be ported to CSS

In addition to the reddit themes requiring premium - this requires donating to reddit, which is a very questionable choice in current situation

Yeah, this is why I couldn't test if it works. Donating reddit $$$ right now is a very bad idea. But it might be useful to people who might already have premium. (Or after June 30, if everything else fails.)

There are 2 extra issues on top of that:

Addon requires viewport meta HTML code, without it website will look like a desktop site with everything being small

It seems like this is the biggest issue so far. We could use @viewport rule instead of the meta tag, but of course reddit does not allow this as well.
[line 1] @viewport is not allowed
@viewport { width: 320px; }

I guess we can partially mitigate this by designing the overall ui to be bigger? Most mobile browsers use 960px width layout in desktop mode. Or maybe the solution is to not use pixels in the design and instead use rem or even in or mm I've never used those in a web design, and they will come with their own unique challenges, but it might be a interesting solution.

Sidebar is handled via JS, not sure how can this be ported to CSS

Hiding the sidebar can be handled in other ways. The easiest method comes to my mind is putting it outside of the viewport and allow vertical scrolling, so the user can scroll "swipe right" to show the sidebar
body {
  overflow: scroll;
}
.side {
  top: 0;
  right: -310px;
  position: absolute;
}

In addition, @viewport is IE/EdgeHTML-only:
https://caniuse.com/?search=%40viewport
On Fri, Jun 9, 2023, 18:22 Mustafa Can Elmacı @.***>
wrote:

In addition to the reddit themes requiring premium - this requires
donating to reddit, which is a very questionable choice in current situation
Yeah, this is why I couldn't test if it works. Donating reddit $$$ right
now is a very bad idea. But it might be useful to people who might already
have premium. (Or after June 30, if everything else fails.)
There are 2 extra issues on top of that:

Addon requires viewport meta HTML code
https://developer.mozilla.org/en-US/docs/Web/HTML/Viewport_meta_tag,
without it website will look like a desktop site with everything being small

It seems like this is the biggest issue so far. We could use @viewport
https://www.w3.org/TR/css-device-adapt/#atviewport-rule rule instead of
the meta tag, but of course reddit does not allow this as well.
[line 1] @viewport is not allowed
@viewport { width: 320px; }
I guess we can partially mitigate this by designing the overall ui to be
bigger? Most mobile browsers use 960px width layout in desktop mode. Or
maybe the solution is to not use pixels in the design and instead use rem
or even in or mm I've never used those in a web design, and they will
come with their own unique challenges, but it might be a interesting
solution.

Sidebar is handled via JS, not sure how can this be ported to CSS

Hiding the sidebar can be handled in other ways. The easiest method comes
to my mind is putting it outside of the viewport and allow vertical
scrolling, so the user can scroll "swipe right" to show the sidebar
body {
overflow: scroll;
}
.side {
top: 0;
right: -310px;
position: absolute;
}
—
Reply to this email directly, view it on GitHub
https://github.com/OctoNezd/oldlander/issues/7#issuecomment-1584759224,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADUZM5DJ5WMHY5KTPQM2DQDXKM5SZANCNFSM6AAAAAAZAUM4H4
.
You are receiving this because you commented.Message ID:
@.***>

Sadly, while this would been nice to implement if reddit were to give free custom CSS to all, it still will be too much work, and cause too many issues - especially considering custom CSS on subreddits.


TH rotation: minimize DNS churn

Do not update all *.th.ooni.org DNS entries on new VM spawn - aka make the A records static during the VM lifetime.
Makes https://grafana.ooni.org/d/Dn1R7QEnz/new-test-helpers?orgId=1 more readable.

Duplicate of https://github.com/ooni/backend/issues/578


Synchronize GitHub Issue templates

Follow the example of the new Member/Committer request templates and add templates to ease the process of opening bugs or feature requests.

https://github.com/submariner-io/submariner/tree/master/.github/ISSUE_TEMPLATE

There is some overlap with https://github.com/submariner-io/submariner-website/issues/9 -- let's try to collaborate on this together.

Sent to the main submairner-io/submariner repo first, to get feedback and iterate before sending parallel PRs to other repos.
https://github.com/submariner-io/submariner/pull/534/files

@dfarrell07, with https://github.com/submariner-io/submariner/pull/534 merged, I think it makes sense to follow the same template across the rest of the repos. Wdyt?

I haven't done this because I'm not sure if it's something we really want, if folks are happy with the current templates. I've added this to the agenda of the next Automation Sync.

Per discussion on the Automation call today, the consensus was that we find these templates helpful and do want to add them to the other repos. I'll work on it. :+1:

bump

@nyechiel made substantial improvements to the templates in many repos recently. I think the main task here is really to make sure all the templates are present/updated for all the relevant repos.
For example, I see Admiral is quite: https://github.com/submariner-io/admiral/tree/devel/.github/ISSUE_TEMPLATE
I think basically we need to copy all the templates from https://github.com/submariner-io/submariner/tree/devel/.github/ISSUE_TEMPLATE to the parallel directores in all repos.

@nyechiel made substantial improvements to the templates in many repos recently. I think the main task here is really to make sure all the templates are present/updated for all the relevant repos.
For example, I see Admiral is quite: https://github.com/submariner-io/admiral/tree/devel/.github/ISSUE_TEMPLATE
I think basically we need to copy all the templates from https://github.com/submariner-io/submariner/tree/devel/.github/ISSUE_TEMPLATE to the parallel directores in all repos.

FWIW, when I pushed the bug template updates, I thought that submariner, submariner-operator, lighthouse and cloud-prepare are the most important ones. admiral is not really user-facing and I were a user I don't think I would even know what to report there. The same is probably true for shopyard. submariner-charts is user facing, but specifically for the latest bug template update I decided to skip it because the template itself is really focused on subctl. coastguard is also quite empty now, but we don't support it at the moment.

I guess it's a question of if we want consistent templates for all repos, or if that doesn't really matter as long as we have the important templates for the important repos.

I guess it's a question of if we want consistent templates for all repos, or if that doesn't really matter as long as we have the important templates for the important repos.

I think we should strive to be as consistent as possible whenever it makes sense, but not just copy everything from Submariner.

Reopening and assigning to @Jaanki per discussions at https://github.com/submariner-io/submariner-operator/pull/1492#issuecomment-883431260.

https://github.com/submariner-io/cloud-prepare/pull/73

https://github.com/submariner-io/submariner-operator/pull/1492


Issue with jenkins build of conda package

The Jenkins build of the conda package seems to be failing:
https://jenkins.choderalab.org/job/conda-omnia-release-linux-vagrant/56/
*** Error compiling '/home/vagrant/miniconda/envs/_build/lib/python3.3/site-packages/gaff2xml-0.1-py3.3.egg/EGG-INFO/scripts/processAmberForceField.py'...
  File "/home/vagrant/miniconda/envs/_build/lib/python3.3/site-packages/gaff2xml-0.1-py3.3.egg/EGG-INFO/scripts/processAmberForceField.py", line 13
    print stream.read()
               ^
SyntaxError: invalid syntax

Fixed by #88


Make core constructors protected

Is your feature request related to a problem? Please describe.
Per #2073 , we are moving away from direct instantiation of Post, Term, etc. objects in favor of Timber::get_* methods. Using Class Maps, we want to make sure that the correct classes are instantiated for their respective post types, taxonomies, etc. To that end, we need to make sure these objects aren't instantiated directly, but are created in such a way that the proper hooks are applied before instantiation.
To this end, we need to disable instantiating calling these constructors.
Describe the solution you’d like

[ ] Define a static Timber\Post::build() method that takes a WP_Post instance and returns an instance of the called class, e.g. a Timber\Post.
[ ] Make Timber\Post::__construct() protected.
[ ] Evaluate whether Comment, Image, Attachment, and PostQuery need to have their own special build methods and protected constructors.
[ ] Repeat for analogs Term, Comment, Image, Attachment, PostQuery, Menu, and User as appropriate.

Describe alternatives you’ve considered
@gchtr threw out the question of whether it was possible to throw a doing-it-wrong exception if the constructor is called directly (presumably from outside Timber core). I don't know of a way to do this without either ephemerally setting global variables or checking the call stack (i.e. debug_backtrace()) from within the constructor, both of which would be very hacky.

Whoa, @acobster is on 🔥

I wouldn’t go down a hacky road that involves debug_backtrace(). Your proposed solution makes sense to me 👍.

TODO: also protect Menu[Item] constructors in #2292


Upgrade Tailwind to 2.1.1 keeping jit (Just in Time) engine enabled per https://blog.tailwindcss.com/tailwindcss-2-1

Nothing as exciting in tech as upgrading to a much better solution and then immediately having to reconfigure said solution in a different way for no particular reason, right @timlrx ? Haha

yup, that's what I thought too when I saw the announcement haha


Help about parameter setting

Hello, would you like to know that when training cvusa, a gpu is used, gou is set to 1,lr is set to 0.0001, batch-size is set to 32, did-URL is set to 'tcp://localhost:10001' and world-size is set to 1. rank set to 0, epochs set to 100, op set to sam, wd set to 0.03, dataset set to cvusa, cos set to True,dim set to 1000, asam set to True, rho set to 2.5. But the result of the first stage is very bad, I would like to ask if I made a mistake, I took a screenshot of the specific parameter Settings, thank you

The image is not uploaded correctly, so I can not see the parameters. Why not directly run the scripts following the instructions?

Since I only have one gpu, running sh run_CVUSA.sh directly seems to require multiple Gpus, and when I run sh run_cvusa.sh directly, I get an error. The error is work = _default_pg.barrier().
RuntimeError: NCCL error in:  /opt/conda/conda-bld/pytorch_1607370172916/work/torch/lib/c10d/ProcessGroupNCCL.cpp:784, unhandled system error,  NCCL version 2.7.8

You may need to specify the GPUs for training in "train.py". Remove the second line if you want to train the simple stage-1 model. Change the "--dataset" to train on other datasets. The code follows the multiprocessing distributed training style from PyTorch and Moco, but it only uses one GPU by default for training，The readme paragraph should mean using a single GPU parameter, but the content of the command line should run in a distributed manner with multiple Gpus

It does not require multiple GPUs. If the train.py, we use os.environ["CUDA_DEVICE_ORDER"] = "PCI_BUS_ID" and os.environ["CUDA_VISIBLE_DEVICES"] = "0" to assign one GPU.


[NEW ICON] MySQL

Skill Name
MySQL
Why?
MySQL is widely used across industry for relational database management.
Reference Image

It seems that MySQL is already added as an icon, make sure to double-check the available icons before adding an issue!
![](https://skillicons.dev/icons?i=mysql)


Feature Request: fzf color scheme

I manually create this to customize fzf.
export FZF_DEFAULT_OPTS=$FZF_DEFAULT_OPTS'
--color=fg:#9ea3c0,bg:#222433,hl:#545c8c
--color=fg+:#535f98,bg+:#2a2c3f,hl+:#929be5
--color=info:#545c8c,prompt:#929be5,pointer:#73c1a9
--color=marker:#73c1a9,spinner:#b5ae7d,header:#87afaf'

Maybe we can generate it automaticaly or just put this in README/WIKI? 😀

I apologize for the late reply!
I've created a theme for fzf and added it to the README. While it's not yet automated, I hope it can be helpful as a reference :dog:
https://github.com/wadackel/vim-dogrun#fzf


Add kvrocks controller to README

Refer to kvrocks_controller.

I do not add kvrocks-operator since I do not know whether it is ready for use or not.


a project set up with yarn workspaces won't resolve local imports

Hi! Sorry if this is not the right place to ask this.
I am trying to set up a POC with yarn workspaces in order to reuse components and logic across webapps, RN, and TV. The webapp (created with CRA) resolves the test component (from a package within the monorepo) and renders it without a cough. On the other hand, the RN-TV app crashes: the tv apps renders the following error message in the simulator:

alt: "bundling failed: Error: Unable to resolve module components/App from index.js: components/App could not be found within the project."
I think my nohoist config is appropriate since the "components" symlink is here in the native's project node_modules folder. I think what is missing is some config on Metro's end, but I don't have much experience there, and could not find proper documentation on this topic (or missed it).
React Native version:
    OS: macOS 10.15.2
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
    Memory: 28.88 GB / 64.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 13.4.0 - /usr/local/bin/node
    Yarn: 1.19.2 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.2, DriverKit 19.0, macOS 10.15, tvOS 13.2, watchOS 6.1
  IDEs:
    Xcode: 11.2.1/11B500 - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native-tvos:  0.61.5-0

Steps To Reproduce

clone my repo
cd packages/native
follow those instructions to clean the xcode project
cd in the root folder, then yarn workspace native ios
the simulator will open. You might have to hit cmd+R to see the component rendered.
open the native index.js file, coment line 9, uncomment line 14
see the error message. The component should be rendered but seems to not even be detected in the node_modules. Does react-native have restrictions about symlinks?

This might be a bit cumbersome to get through, but you have no idea how grateful I'd be about any help on this.... It's been days 😅

I’m afraid I can’t help you.... I agree that this is probably a Metro config issue.


Added drawerPosition prop into DrawerView (left - right)

Resolves #87.
I also added an entry to the .gitignore regarding IntelliJ.

Are apps with a drawer on the right common? I don't remember seeing one.

Hey @ferrannp, how did you test this doesn't break any existing code using React Navigation?

@mkonicek I think it'll be useful for RTL languages.
@ferrannp can you post a screenshot?

Good point about RTL!

Here the example working with it:

I did not find unit tests for the UI like a test for the drawer. Do you plan to add them in the future? Maybe Jest snapshots?


GitHub redirects giving 403

Example:

https://api.github.com/repos/scout-app/scout-app/zipball/v2.18.16

In a browser this URL results in downloading a 3MB ZIP file. However when using easydl the redirects fail and it throws an error logging 403.

const EasyDl = require('easydl');

async function downloadZip (downloadUrl) {
  const zipFilePath = 'output.zip';
  const downloadOptions = {
    existBehavior: 'overwrite',
    maxRetry: 3,
    reportInterval: 1250
  };

  return new EasyDl(downloadUrl, zipFilePath, downloadOptions)
    .on('progress', (progress) => {
      console.log({ progress });
    })
    .wait()
    .then((completed) => {
      console.log({ completed });
    })
    .catch((error) => {
      console.log(error);
    });
};

downloadZip('https://api.github.com/repos/scout-app/scout-app/zipball/v2.18.16');

Error: Got HTTP Response code 403
  at Object.<anonymous> (/test/node_modules/easydl/dist/request.js:130:23)
  at Generator.next (<anonymous>)
  at fulfilled (/test/node_modules/easydl/dist/request.js:24:58)
  at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

Seems to be related to not setting a user agent.
Request forbidden by administrative rules. Please make sure your request has a User-Agent header (https://docs.github.com/en/rest/overview/resources-in-the-rest-api#user-agent-required). Check https://developer.github.com for other possible causes.

After using https://www.npmjs.com/package/nodejs-file-downloader it revealed more error details that helped solve it.
In the example above adding this in fixed it:
  const downloadOptions = {
    existBehavior: 'overwrite',
    maxRetry: 3,
    reportInterval: 1250,
    httpOptions: {
      headers:{
        'User-Agent': 'my-library-name'
      }
    }
  };


Package extent.exe (and configs) in the release

Hello,
Is it possible to add the extent.exe (and maybe configuration part as well) in a release? Currently there is only the source code.
Thanks,
Eniko

I would like to see this as well.


Update copyright in license description

What this PR does / why we need it:
As discussed on TSC, this PR will update all the license description.
Reference: https://github.com/opensds/tsc/blob/master/2019-06-03
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

LGTM


Issue with grel:string_substring

I'm iterating a csv and I'm trying to get the 3 first letters of each country , turn them into lowerCase and apply the output in a URI template. So far I have succeded getting the 3 letters but I cannot apply the result in a template, instead I get the 3 letters code. This is what I've tried
rr:predicateObjectMap [
    rr:predicate example:country;
    rr:objectMap [
        fnml:functionValue [
            rr:predicateObjectMap [
               rr:predicate fno:executes ;
               rr:objectMap [ rr:constant grel:string_substring ]
               ] ;

            rr:predicateObjectMap [
             rr:predicate grel:valueParameter ;
             rr:objectMap [ rml:reference "Country of origin" ]
             ] ;

            rr:predicateObjectMap [
             rr:predicate grel:param_int_i_from ;
             rr:objectMap [ rr:constant "0" ]
             ] ;

            rr:predicateObjectMap [
             rr:predicate grel:param_int_i_opt_to ;
             rr:objectMap [ rr:constant "3" ]
             ] ;

            rr:predicateObjectMap [
             rr:predicate grel:stringOut ;
             rr:objectMap [
             rr:template "https://example.com/origin#{Country of origin}"
             ]
          ];
        ]
     ]
];

Is it possible to get this triple
 <http://example.com/1> <http://example.com/country>  <http://example.com/origin#fra>
instead of
<http://example.com/1>  <http://example.com/country>  "Fra";

Templates don't work out of the box with functions, you'll need to 'recreate' the template functionality by nesting this result into a grel:array_join function (see here for an example on nesting). Then, you'll probably need to add add rr:termType rr:IRI to the objectmap to make sure the array_join result is actually outputted as an IRI and nota a string :)


Show warning if select_multiple value has spaces

Pyxform will check for this but there is always a chance that a form that builds a choice list from external data contains such options, so it needs to be checked on the client.
https://github.com/opendatakit/javarosa/issues/263#issuecomment-369019465

@MartijnR, Can you elaborate more about the "builds a choice list from external data contains" and "checked on the client"? and if possible steps to testing it

hi @theywa,
Enketo Express can load external data files that will be used in the XForm (referred to with <instance src="..."/>) (e.g. clinicaldata.xml, users.xml as OC uses). One way they can be used is to build choice lists for select_multiple questions. Since form validators do not know the content of this external data file (which can change as well), they cannot show warnings for values that contain spaces (and are not allowed).
In Enketo Core, the ability to load external files is not included. Instead the XML content of such external data files can be passed during form instantiation with the data property.
I'll create a form and a form instantiation snippet you could use to develop this check (and create an automated test from).
Let's add this feature only to itemset.js (i.e. dynamic choice list generator for both internal and external data). So it won't be available to hardcoded non-dynamic choice lists (we can rely on form validators to check for those). I think the only feasible way to do this is to add the check at the time a new dynamic list is built, so that means that a warning may only pop up halfway through the data entry session.

Here is a test form (added .txt to extension just to fool github) which you can add to the /test/forms folder:
external-data.xml.txt
To instantiate with external data in app.js (temporarily) add:
modelStr: modelStr,
        external: [ {
            id: 'mydata',
            xml: new DOMParser().parseFromString(
                `   <root>
                        <item>
                            11
                            <group>a</group>
                            <name>1 1</name>
                        </item>
                        <item>
                            22
                            <group>a</group>
                            <name>22</name>
                        </item>
                        <item>
                            33
                            <group>b</group>
                            <name>33</name>
                        </item>
                        <item>
                            44 (value has spaces!)
                            <group>b</group>
                            <name>4 4</name>
                        </item>
                    </root>
                `, 'text/xml' )
        } ]

Both select_multiple questions contain a value with a space, so both should prompt a warning. Let's make the warning helpful by mentioning the value. Eg. something like "select multiple question has an illegal value "a a" that contains a space"

@MartijnR - Thank you for the details instruction, it's really helpful and I'm really sorry for the late response.  I create a PR, also include the screenshot example.

Hi @theywa, Thanks! Here is a newer version of the test form that also tests a select minimal (with external data). So this form should show 3 errors.
external-data-v2.xml.txt


add decompression to QL LC and Compressed/SummedPixelData

Codecov Report

Merging #80 (cfbc8d6) into master (9cf2732) will decrease coverage by 43.09%.
The diff coverage is 28.33%.

@@             Coverage Diff             @@
##           master      #80       +/-   ##
===========================================
- Coverage   73.97%   30.87%   -43.10%
===========================================
  Files          43       43
  Lines        2628     2643       +15
===========================================
- Hits         1944      816     -1128
- Misses        684     1827     +1143

Impacted Files
Coverage Δ

stixcore/processing/decompression.py
36.00% <0.00%> (-56.00%)
:arrow_down:

stixcore/processing/engineering.py
23.91% <0.00%> (-63.05%)
:arrow_down:

stixcore/products/level0/quicklook.py
22.09% <0.00%> (-54.00%)
:arrow_down:

stixcore/products/common.py
20.51% <20.00%> (-17.65%)
:arrow_down:

stixcore/products/level0/science.py
14.86% <23.80%> (-2.04%)
:arrow_down:

stixcore/tmtc/packets.py
34.94% <27.27%> (-45.06%)
:arrow_down:

stixcore/products/product.py
31.93% <43.75%> (-44.71%)
:arrow_down:

stixcore/products/__init__.py
100.00% <100.00%> (ø)

stixcore/calibration/compression.py
12.06% <0.00%> (-87.94%)
:arrow_down:

stixcore/idb/idb.py
20.15% <0.00%> (-76.23%)
:arrow_down:

... and 28 more

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 9cf2732...cfbc8d6. Read the comment docs.


Move ParallelismConfiguration to a public package

This moves ParallelismConfiguration to a public package so that StartParameter is not leaking an internal type.

No, it didn't exist.  I wasn't aware of the aversion to "api" and that package just seemed to make sense.  I'll change it to something else public.

Package changed to "org.gradle.concurrent".

ParallelismConfiguration was introduced to communicate the max worker count to the console as we needed to process the CLI flags earlier with the logging flags. Now that we grow the number of work in progress items dynamically, we don't really have a need to communicate that information. Depending on where we want to move in the future for the console, we could get rid of the changes around ParallelismConfiguration and revert to what we had previously.
@eriwen @bmuschko Do you have an idea if this is required for the future? More importantly, how do we plan to shrink the number of work-in-progress items (if we plan to do so)?

For reference, I just saw https://github.com/gradle/gradle/issues/2411 which this work could be part of if we decide to revert the change. PR https://github.com/gradle/gradle/pull/1554 is where the ParallelismConfiguration and all the piping to connect to the console was introduced. If we revert, we should also revert most of the change in the PR.

@lacasseio this is no longer only used for the console, but also for the worker API.

Unfortunately, there aren't really any existing public packages in baseServices that are not under "api".  I've added the new package to docs.
Frankly, I don't see a lot of advantage or disadvantage to this being a public interface (i.e. we need to expose these methods on StartParameter either way) so I'm not highly opinionated about whether we should instead just remove the extends.  If you feel strongly about it, let me know your reasoning.


validate_block now checks if a federation node created the block.

resolves #229
related to #228
This pull request adds code to check if a block was created by a federation node.
To do so it checks if block['block']['node_pubkey'] is a federation node and also checks if the signature is correct.
The validation seems to be split between core and consensus. I decided to add the code to the consensus module but during the planned refactor we should address this code split.

Almost done here.  Not quite sure how to check if a particular transaction ended up back in the backlog

:+1:


Issue folding python code

There is some strange behavior with the code folding in python when folding a function with a multi-line declaration.   Declaring a function across multiple lines is recommended by pep8 for long declarations, but in such cases only the declaration folds - not the entire function block.

I wanna work on this issue. Please allow me to work on this issue.

It turns out that this can be resolved through the core atom settings by disabling "Tree Sitter".  If anyone still wants to resolve this behavior with Tree Sitter enabled go nuts, but I am closing the issue.


Rename Pico to Helidon Injection

Module Names
All modules names will replace "pico" with "inject".

helidon-pico-project -> helidon-inject-project
helidon-pico-api -> helidon-inject-api
helidon-pico-configdriven-* -> helidon-inject-configdriven-*
helidon-pico-maven-plugin -> helidon-inject-maven-plugin
helidon-pico-processor -> helidon-inject-processor
helidon-pico-runtime -> helidon-inject-runtime
helidon-pico-testing -> helidon-inject-testing
helidon-pico-tools -> helidon-inject-tools

Package Names
All package names will replace "pico" with "inject".

io.helido.pico.* -> io.helidon.inject.*

Class Names
All package names will replace "Pico" with "Injection".

PicoServices -> InjectionServices
PicoServicesConfig -> InjectionServicesConfig
PicoServicesHolder -> InjectionServicesHolder
PicoException -> InjectionException
...

Method Names, Variable Names
All released artifacts will have all "pico" variables and methods changed to remove and/or replace the word "pico" in its name to a suitable alternative (i.e., "inject" or "injection", etc).
Configuration

Currently "pico" is a reserved top-level configuration name. This name will be changed to "inject".
All attributes with "pico" in the name will be replaced with "inject".

General

All documentation (README, etc.) will rename "Pico" to either "Helidon Injection" for short, and in some cases "The Helidon Injection Framework".
Some of these test modules and test names will leave "pico" in the names (these are not released in the bom)

tests/resources-pico

Blocked behind these PRs:

https://github.com/helidon-io/helidon/pull/7155
https://github.com/helidon-io/helidon/pull/7156
https://github.com/helidon-io/helidon/pull/7160


jupysql interactive notebook added

Describe your changes
According to the original thread, I implemented a simple example of an interactive notebook using ipywidgets.
It allows getting an input from the user and using this input to run jupysql function and display the results.
I wasn't able to run this in the jupyter-book so I attached this example.
A simple notebook with a textbox and a button attached
interactive_nb.zip
Docs:
https://jupysql--146.org.readthedocs.build/en/146/interactive-demo/interactive_nb.html
Issue ticket number and link
Closes #136
Checklist before requesting a review

[ ] I have performed a self-review of my code
[ ] I have added thorough tests (when necessary).
[ ] I have added the right documentation in the docstring and changelog (when needed)

@yafimvo please update so we can review.

The example looks nice!
It wasn't too intuitive viewing the results, maybe you can go for a window on top of the main jupyter vs a side view on the bottom. Also if the query is on a large dataset (I saw only 4 raws), we should add a limit to it.

This was just a simple example to see how the interactive notebook works. I think we can create a more complex example with an interactive plot. what do you think?
@idomic @edublancas

this is a good first step! I think the next step would be to figure out how to listen for event in custom widgets and push updates to these widgets.
The first interactive feature we can ship is something like this: https://github.com/mwouts/itables/tree/main/itables but perform the queries in the underlying SQL engine.
e.g.,
%sqlcmd explore --table table --schema schema

should display a table in HTML with the first X rows of the table, then by clicking on the arrows next to the column name:

we should sort the contents of the table, by running a SQL query:
SELECT * FROM table ORDER BY column DESC

To enable this, I believe we cannot rely on the ipywidgets library since it probably only contains a few built-in widgets, and in our case, we want to generate the HTML/CSS/JS and display it, but still have the ability to listen for events and update the widget.
let me know if this explanation is clear @yafimvo


Only prefetch text not images (e.g for OCR'd files)

When using the search function the entire file is downloaded even with auto fetch disabled. This causes a problem with large OCR'd files. It would be great to be able to just download the text for searching, and the page images are grabbed only when required to view.

It would be great to be able to just download the text for searching, and the page images are grabbed only when required to view.

Unfortunately ignoring images would lead to missing content when extracting text!
Images, or rather XObjects as they are called, can come in various subtypes. Specifically in the case of the Form subtype, the XObject may contain (almost) arbitrary PDF content, please see the PDF specification[1]. Hence if we ignored images when extracting text, then text-selection and searching would be partially broken for many existing PDF files.
Closing as wontfix.

[1] http://www.adobe.com/content/dam/Adobe/en/devnet/acrobat/pdfs/PDF32000_2008.pdf#G7.1852124


Detached script editor can become frozen with multiple, differing resolution monitors on Windows

Godot version
4.1.stable
System information
Windows 11, geForce 3070 TI
Issue description
I have 3 monitors:

Monitor1 - leftmost, windows primary monitor, no windows open on it, 2K resolution
Monitor2 - center, godot window, 4K resolution
Monitor3 - rightmost, detached script editor window, 4K resolution

After I press F1 key, then clicking on the script editor it can now no longer be used. Clicking on it doesn't do anything, can't access the File/Edit/Search etc menus. Pressing escape doesn't help. Pressing the cross at the top right doesn't do anything.
Only way to fix is to close and re-open Godot.
Steps to reproduce
Have multiple monitors with differing resolutions. Make your lower resolution monitor your primary monitor then open Godot and detach the script editor. Leave them on your higher resolution monitor. Press F1 then click on the script editor. If the F1 menu didn't appear on your primary monitor, you are now out of luck.
Minimal reproduction project
No project. I can confirm resolution is the problem with the F1 menu though, as if I DONT detach the script editor and press F1, I see this:

Clicking and drawwing the incorrectly sized F1 window at all resizes it to the correct resolution:

It really should open with the correct resolution and in the center of the same monitor as the script editor.

A simple fix for this could also be not allowing the editor window to take focus whiel the F1 window is open.

A simple fix for this could also be not allowing the script editor window to take focus whiel the F1 window is open.

I think we should strive towards making less popups exclusive, not more :slightly_smiling_face:

The "Search Help" are modal, as well as all the settings windows.
I can understand the need for the settings windows to be modal.
But the "Search Help" might not need to be modal but a modeless window.

This is caused by the following lines: https://github.com/godotengine/godot/blob/6588a4a29af1621086feac0117d5d4d37af957fd/editor/editor_help_search.cpp#L163-L165
That code doesn't take into account that the screen size may change, allowing saved_size to cross the screen borders, as happen in OP.
The simplest way to fix this is to discard the saved_size if it goes out of the screen (I'll PR this solution soon).
Another solution is to apply the same algorithm used for floating panels, which has the nice advantage of keeping the window rect ratio, but it's probably overkill in the help dialog.
I checked only the help dialog, but I think there are other places where this happens.

In general, I'd argue that the Window code should check this automatically (at least of modal windows) and raise an error, or does fallback in some way.

Related to https://github.com/godotengine/godot/issues/74261.

This should have been fixed by #79617, could you test 4.3-dev5 or later to confirm?

This screenshot shows my 2k primary Monitor on left with secondary 4k monitor on right. Pressing F1 on the script editor brings up the 'Search Help' popup on my primary monitor (bottom left).
There are two issues the latter of which is a showstopper:

It's scaled wrong (2x larger than it should by the looks of it). Moving the window in any way causes it to rescale itself so it appears correct.
It's so far down and left, I can't actually move the window because the minimize/maximize/close buttons extend to the very edge of the window. There's no titlebar I can click on to move it.


Params vs Context

Hello!
Tell me why you parameters of the URL is transmitted via ps fasthttprouter.Params instead of func (*RequestCtx) UserValue in the context of.

fasthttprouter is a fork of httprouter, and this Params struct come from here:
https://github.com/julienschmidt/httprouter/blob/master/router.go#L86

@claygod there is a PR on this by the way https://github.com/buaazp/fasthttprouter/pull/8

Now UserValue is available in master and releases.


Add toc-page theme

toc-page is a new theme that was submitted a few days back to the old repository

Hey,
Could you tag a new release with the images (which I'm guessing is the reason your theme does not show; we have since then improved the build script to fail the build)?


Added Bitdeer as in the block 691271

I think this can be helpful as a fall back if the payout address is unknown. The block 691271 paid to 1Bf9sZvBHPFGVPX71WX2njhd1NXKv5y7v5 which receives both BTC.com and Bitdeer payouts. Payout address matching should be preferred over coinbase tag matching.
https://github.com/0xB10C/known-mining-pools/blob/60cfefeb7a1977c07b95da865735fbb293fe129f/pools.json#L849-L852


Extend Documentation of Calculated Log Metrics

Improve documentation of the workaround needed for the calculated log metrics
configuration.
There is an additional issue in the current API workaround, that leads to failures for
dependent configurations on first upload of a log metric - Bug to be created.

Interesting, your integration test runs timed out after 10m.

@didiladi  could you give this another review please?
Rewrote the samples a bit when resolving a conflict the the change that introduced the  type for code blocks
Also it seems the Table of Contents wasn't up to date anymore (I use a VS code plugin for markdown that auto-generates the toc)


FIX-#5737: BUG: String columns are converted to Categorical, if exported from HDK

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #5737
[x] tests added and passing
[ ] module layout described at docs/development/architecture.rst is up-to-date

The to_pandas part looks good, but I'm really not sure we should try to make a similar thing for SQL queries. Trying to match column names in the original frame and the result to borrow dtype might produce really unexpected results. If SQL is used to deliberately produce a column with a new type keeping the old column name, then the introduced code would change the type back to the original.

If SQL is used to deliberately produce a column with a new type keeping the old column name, then the introduced code would change the type back to the original.

No, it shouldn't changed back to the original. This code changes dtypes from category to object for the dictionary fields only. Additionally, it checks - if the field exists in the original frame and the type is categorical, then dtype is not changed for this field. An unexpected result could be if a categorical field is returned with a different name. In this case, it will be casted to string.

No, it shouldn't changed back to the original. This code changes dtypes from category to object for the dictionary fields only. Additionally, it checks - if the field exists in the original frame and the type is categorical, then dtype is not changed for this field. An unexpected result could be if a categorical field is returned with a different name. In this case, it will be casted to string.

That's good and reduces the possibility of an undesired cast. But still, it's so easy to get not converted dict simply because names don't match (and any operation on a column modifies its name). Also, we can always come up with a case where category becomes a string with no obvious reason. E.g. columns renaming with a filter:
df = pd.DataFrame({"a": ["str1", None, "str2"], "b": ["str3, "str4", None]}
df["b"] = df["b"].astype("category")
query("SELECT b as a, a as b FROM t WHERE b IS NOT NULL;", t=df)

I know it's kind of an artificial example, but the same as "SELECT *".
So, does it worth trying to set up dtypes just to cover some cases, break some other cases, and do nothing in probably most of the real scenarios?

Yes, I understand it's quite easy to get an unexpected dtype. This is mentioned in the comments. But what is the best solution? I see at least 3:

Do nothing here. In this case all string columns are converted to categorical.
Convert all dictionaries to string. In this case we miss all categorical columns.
Convert all dictionaries to string, but try to keep categoricals.

In my opinion, the first solution is just broken. Here is an example:
df = pd.DataFrame({"a": ["a"]})
res = query("SELECT a FROM t;", t=df) + "_suffix"

TypeError: unsupported operand type(s) for +: 'Categorical' and 'str'

The second one brokes all categorical dtypes.
The last one should work properly in most? real cases, where the column names are not changed, otherwise, we are getting the second scenario.
I think, we should choose between 2 and 3.

I would choose option 2 for now because it is simple and predictable. Then maybe switch to option 1 when we have reasonable string support in HDK.

I would choose option 2 for now because it is simple and predictable. Then maybe switch to option 1 when we have reasonable string support in HDK.

Thanks, implemented the option 2.

There are also several category-hacks that still exist in HDK tests, could you please raise an issue to remove the hacks in a separate PR?

https://github.com/modin-project/modin/issues/5818


fix(autoware_scenario_selector): fix bugprone-branch-clone

Description
This is a fix based on clang-tidy bugprone-branch-clone error.
/home/emb4/autoware/autoware/src/universe/autoware.universe/planning/autoware_scenario_selector/src/node.cpp:155:40: error: repeated branch in conditional chain [bugprone-branch-clone,-warnings-as-errors]
    if (is_in_lane && is_goal_in_lane) {
                                       ^
/home/emb4/autoware/autoware/src/universe/autoware.universe/planning/autoware_scenario_selector/src/node.cpp:157:6: note: end of the original
    } else if (is_in_parking_lot) {
     ^
/home/emb4/autoware/autoware/src/universe/autoware.universe/planning/autoware_scenario_selector/src/node.cpp:159:12: note: clone 1 starts here
    } else {
           ^

Related links
Parent Issue:

Link

How was this PR tested?
Notes for reviewers
None.
Interface changes
None.

Effects on system behavior
None.

@rej55 @satoshi-ota @shmpwk @taikitanaka3 @TakaHoribe @takayuki5168 @tkimura4
Please confirm the PR. Thank you very much.


chore: upgrade symfony/console for datastore/tutorial

Closing because we should upgrade to the new sample format instead (see #1389)


Use Semantic Release by Default

Overview
I think we should adopt Semantic Release as the default mechanism to release in Project Helix (at least for all libraries, services are a different discussion) and adjust the starter repo to include it.
Details
I've been using Semantic Release for https://github.com/adobe/fastly-native-promises simply because the original project used it, and I have grown to like it.
Cons:

you have to commit using npm run commit and fill out a questionnaire
it encourages bigger commits than I prefer

Pros:

the release can be fully automated, in fact semantic-release only works in a CI environment
you get proper NPM releases like<EMAIL_ADDRESS>(without doing anything)
you get proper GitHub releases like https://github.com/adobe/fastly-native-promises/releases/tag/v1.3.2 (without doing anything)
you get a proper CHANGELOG.md like https://github.com/adobe/fastly-native-promises/blob/v1.3.2/CHANGELOG.md#132-2019-01-22 (without doing anything)
issues get a nice status update once they have been fixed in a release like this: https://github.com/adobe/fastly-native-promises/issues/27#issuecomment-456381534 and they get tagged as released (without doing anything)
same for PRs: https://github.com/adobe/fastly-native-promises/pull/28#issuecomment-456381505

Proposed Actions

[ ] adjust this repo to use semantic-release
[ ] implement semantic-release for helix-shared
[ ] implement semantic-release for helix-pipeline
[ ] implement semantic-release for htlengine
[ ] implement semantic-release for helix-cli
[ ] implement semantic-release for git-server
[ ] implement semantic-release for helix-simulator
[ ] implement semantic-release for vscode-helix (there is a plugin for the VS Code Marketplace)

fixed in 615a56f81a5d9eec53e484ef8ee465fe9d7fda25


hbase 2 integration tests don't run.

The hbase 2 integration tests get this error:
Caused by: java.lang.IllegalArgumentException: port out of range:-1
The HBase integration tests complete with a status of 0, but don't actually run any tests.

This seems to be an HBase bug.

I think that we should wait for an official fix from HBase when we upgrade to the next 2.* release.


yarn install error

I forked the repo to work on it, when I run yarn I get this error:
https://github.com/okonet/listr-update-renderer/tarball/upgrade-log-update: Integrity check failed for "listr-update-renderer" (computed integrity doesn't match our records, got "sha1-Bgc/qTFmJ3YHp4FPTh+DlgCBQUw=")
I searched a bit online and this appears to be related to a specific version of a dependency for lint-staged.
Running yarn upgrade lint-staged fixes it!
Would it be worth including your yarn.lock file in the repo?
Thanks!

If you make a PR I'll merge it!


Feature/factor fix gnu tests with num prime

this addresses #6109 which is about failing gnu test tests/factor/factor.pl
changes:

replace custom implementation for factorisation with third party crate num_prime (Apache-2.0 license).
this allows for input numbers > u128
adapt gnu-test expected error messages with custom ones by applying patch

impressive. how many lines it remove ?
did you look at performances ?
See: https://github.com/uutils/coreutils/blob/main/src/uu/factor/BENCHMARKING.md

@sylvestre It removed ~1400 lines of code.
As you requested, I also looked into performance now.
Reference main branch:

PR branch with num_prime crate:

At first glance it might look promising, but actually there is a factor 1000 difference (us vs. ms) :-/ thats sad

are you sure you built in release mode ? thanks

OK, I found the issue with the performance comparison.
The existing benchmark was measuring only a part of the full factorization step.
I misunderstood this and thus the comparison was unfair.
I adapted the performance test such that it now compares the full facorization of uutils with num_prime in the same run such that also compiler optomization is guaranteed to be the same.
The result shows that the performance is comparable. num_prime even seems to be a bit more performant in average.
Therefor I think its fine to do the replacement.

@sylvestre what can I do about this error in the cargo-deny step?

I've never had this issue before.

can I ignore the warning?
do I need to fix the issue with the duplicated entries by aligning the versions? How do I do this?

Ignore the warning in deny.toml
you can also try to contribute to lru to hashbrown

The existing benchmark was measuring only a part of the full factorization step.

maybe update the benchmark accordingly ?
cc @nbraud

I submitted the changes I did for the direct comparison here:
https://github.com/cre4ture/coreutils/tree/test/factor_compare_different_implementations
You can see that I also tested different other third-party crates:
prime_factorization = "1.0.4"
prime-factor = "0.4.6"
primeshor = "0.2.0"
factor-rs = "1.2.1"
slow_primes = "0.1.14"

But none of it had better performance results than num_prime

Cool! For performance, it would be interesting to see if we can use factorize128 and factorize64 too if the input number is small enough. But maybe for another PR and maybe the library does that already internally.

Just a humble user of uutils' factor here, but I would not recommend removing all of this code. @nbraud put years of work into this and it looks like the num_prime library you are replacing it with has not been updated since 2022.

The result shows that the performance is comparable.

Considering that the performance of uutils' factor is already up to 15× slower than GNU factor and over 20× slower other programs (see the recent benchmarks in #1456), if the performance of this num_prime library is only "comparable" to the existing code, that likely means uutils' factor would never be able to match the performance of GNU factor if this library is used. According to the documentation, this library does not even implement all of the algorithms of GNU factor, let alone any modern algorithms like ECM used by other tools.

@tdulcet You raise some good points. The work by @nbraud is awesome, as shown by the fact that most of the libraries out there are slower than their implementation. However, I don't think anybody currently maintaining the project has the expertise/time to really improve this further within this repository. Therefore, I think we actually should rely on another library stands more on its own. The way I see it, there are several options:

Use a num-prime or another third-party library. Ideally one that is well-maintained and very fast.
Extract the factorization code into a separate library. Set up even better benchmarking and start improving it there. And then hope we can attract some contributors who want to improve it.
Leave it in and keep it until someone magically comes along and improves it.

I really hope that someone in the Rust ecosystem comes along who can write factor algorithms on par with GNU, but I'm pretty sure it's not going to be us.

if the performance of this num_prime library is only "comparable" to the existing code, that likely means uutils' factor would never be able to match the performance of GNU factor if this library is used.

This is why I mentioned factorize64. That might be a big improvement. Also remember that it's comparable in performance, but supports numbers larger than u128. So it's probably much faster than if we would convert all u128 in uutils to big decimals.

@cre4ture as another data point, have you tried primal?

I'm honestly still surprised that no good rust library has emerged yet. It feels like an obvious thing to tackle.

@cre4ture as another data point, have you tried primal?

I added primal now. But it seems to be quite slow. I canceled the execution of the performance test because it takes so long. And it only supports usize which is u64 on most modern PCs.
Meanwhile I stumbled accross another crate called number-theory, which does support >u128, but is super slow.
In general I need to state that I'm far away from beeing an expert in primal factorization. So maybe I'm doing something wrong in my comparison. So everybody is welcome to do reviews/suggestions.  My current branch for performance comparison is here: https://github.com/cre4ture/coreutils/tree/test/factor_compare_different_implementations .
I'm with @tertsdiepraam. I think the coreutils crate is not an ideal place to maintain a factorisation algorithm implementation. There are many usecases out there (I assume) which could make use of a smart implementation such that its clearly worth to have it in an own crate. Of course, I respect the authors of the uutils implementation. I could never have done it like this.
In addtion to the options mentioned by @tertsdiepraam , I see also the option that someone could merge the implemenation of uutils and num-prime to get best from both crates.
The good thing is, that the interface to all those libraries that I tested is small and always similar. An exchange of the library for the implementation of the algorithm is low effort and easy. So even if we drop the uutils implementation for now, we could easily re-introduce it in future if someone improved it.
So I hope nobody is angry at me if I still suggest to do the replacement now.

So I hope nobody is angry at me if I still suggest to do the replacement now.

nope, we love this kind of work, thanks :)

this is terrific!

@sylvestre how do I ignore the issue with the duplicate "hashbrown" crate?
I think to fix we would need to upgrade both sides as both are using a outdate version of hashbrown. Lates is 14.3.

@cre4ture you can add it to the skip list in deny.toml.

@sylvestre can we merge this PR? or is there something open to do?

yeah, let's take it
but let's see the result of the last run of the CI

Congrats! The gnu test tests/factor/factor is no longer failing!

and CI fully green!

well done!


"Matcher pwned already exists"

Hello,
Using the pwned matcher as the Vue example shows  in the docs (code below), results in the console message Matcher pwned already exists.
import matcherPwnedFactory from '@zxcvbn-ts/matcher-pwned'

// optional
const matcherPwned = matcherPwnedFactory(fetch, zxcvbnOptions)
zxcvbnOptions.addMatcher('pwned', matcherPwned)

This only happens if add the matcher in a functions that is executed multiple time. I can see that you are using vue and the composition API. So i would assume you are either using it inside a function or your component is rendered multiple time.
If it is rendered multiple time you need to to move the zxcvbn option handling into a place which is only executed once like a plugin

@MrWook Thanks for the reply. However, this is the only component I'm using zxcvbn in and I only have one instance of the component in use in my app. This all the zxcvbn code in my component:
import { zxcvbnAsync, zxcvbnOptions, debounce } from "@zxcvbn-ts/core";
import zxcvbnCommonPackage from "@zxcvbn-ts/language-common";
import zxcvbnEnPackage from "@zxcvbn-ts/language-en";
import matcherPwnedFactory from "@zxcvbn-ts/matcher-pwned";

const matcherPwned = matcherPwnedFactory(fetch, zxcvbnOptions);
zxcvbnOptions.addMatcher("pwned", matcherPwned);

zxcvbnOptions.setOptions({
  translations: zxcvbnEnPackage.translations,
  graphs: zxcvbnCommonPackage.adjacencyGraphs,
  dictionary: {
    ...zxcvbnCommonPackage.dictionary,
    ...zxcvbnEnPackage.dictionary,
  },
  useLevenshteinDistance: true,
});

const debouncedZxcvbn = debounce(async () => {
  if (passwordValue.value) {
    passwordStrength.value = await zxcvbnAsync(passwordValue.value);
  } else {
    passwordStrength.value = null;
  }
}, 300, false);

watch(passwordValue, debouncedZxcvbn);

This is in the root of my setup function.

...I will try moving the options code to a plugin to see if that helps

I just did the same thing in a nuxt basic setup and it is working fine. So i guess your page is rendered multiple times and on this note... the error would be thrown any way if someone is leaving the password page and access it again. So i need to update the documentation to use a plugin in vue or use it in the app.vue 🤔
But don't worry too much about the console message, its just a message and it wont break anything. It's just informs you that you are updating the options multiple times which is wasting resources.
The plugin approach would be something like

export default {
  install: () => {
    const matcherPwned = matcherPwnedFactory(fetch, zxcvbnOptions)
    zxcvbnOptions.addMatcher('pwned', matcherPwned)

    const options: OptionsType = {
      // recommended
      dictionary: {
        ...zxcvbnCommonPackage.dictionary,
        ...zxcvbnEnPackage.dictionary,
      },
      // recommended
      graphs: zxcvbnCommonPackage.adjacencyGraphs,
      // recommended
      useLevenshteinDistance: true,
      // optional
      translations: zxcvbnEnPackage.translations,
    }
    zxcvbnOptions.setOptions(options)
  }
}

@MrWook I'm using nuxt 3. This is my plugin code. Still throwing the error :/
import { zxcvbnOptions } from "@zxcvbn-ts/core";
import zxcvbnCommonPackage from "@zxcvbn-ts/language-common";
import zxcvbnEnPackage from "@zxcvbn-ts/language-en";
import matcherPwnedFactory from "@zxcvbn-ts/matcher-pwned";

export default defineNuxtPlugin((nuxtApp) => {
  const matcherPwned = matcherPwnedFactory(fetch, zxcvbnOptions);
  zxcvbnOptions.addMatcher("pwned", matcherPwned);

  zxcvbnOptions.setOptions({
    translations: zxcvbnEnPackage.translations,
    graphs: zxcvbnCommonPackage.adjacencyGraphs,
    dictionary: {
      ...zxcvbnCommonPackage.dictionary,
      ...zxcvbnEnPackage.dictionary,
    },
    useLevenshteinDistance: true,
  });
});

AH finally i got the same error let me check it a little bit

@MrWook Great, thanks! Just good to know I'm not going crazy :)

I got it to work with a nuxtModule and a ready hook:
export default function zxcvbnModule(moduleOptions, nuxt) {
  nuxt.hook('ready', async () => {
    const matcherPwned = matcherPwnedFactory(fetch, zxcvbnOptions)
    zxcvbnOptions.addMatcher('pwned', matcherPwned)

    const options: OptionsType = {
      // recommended
      dictionary: {
        ...zxcvbnCommonPackage.dictionary,
        ...zxcvbnEnPackage.dictionary,
      },
      // recommended
      graphs: zxcvbnCommonPackage.adjacencyGraphs,
      // recommended
      useLevenshteinDistance: true,
      // optional
      translations: zxcvbnEnPackage.translations,
    }
    zxcvbnOptions.setOptions(options)
  })
}

But be aware that you need to remove all the build folders otherwise the previous code is somehow stuck in there 🤔

@MrWook Nevermind in regards to the fetch issue. I was still running off of node 16. Switching to 18 fixed that.
However, I'm still not seeing any fetches to the pwned API.
Now, for some reason I'm also getting a score of 2 when typing in password whereas before I was getting a score of 0

@MrWook It appears that setting the options in the Nuxt module as per your most recent PR doesn't carry across to runtime. For example, I've set useLevenshteinDistance: true in the Nuxt module and if I console.log(zxcvbnOptions) afterwards (from within the module), it is correct. However, if I do console.log(zxcvbnOptions) from my component, it shows useLevenshteinDistance: false .

This makes sense as the module is only executed once on the server side.
So we need to work with client only plugins. I forgot about them...
The users are not typing any password on the server side rendering anyway. I updated my PR

I've put the options back in a client only plugin. This eliminates Matcher pwned already exists, but this issue then arises when inputting a password:
haveIBeenPwned.ts:33 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'digest')

I guess it's because you are working under an unsecure url:
https://stackoverflow.com/questions/57146558/typeerror-digest-of-undefined-in-development-environment
Nuxt prints out multiple urls to checkout the website like:

Local:    http://localhost:49537/
Network:  http://<IP_ADDRESS>:49537/

Only the localhost address is secure for the browser

You are a genius. That did the trick. I was accessing it on my local network. Everything seems to be working as intended now :) Thank you very much for the info.


error on final GlobalKey

I am getting error on  final GlobalKey _fbKey = GlobalKey(); The FormBuilderState is not a type so it can't be used as a type argument.  Also controls and on submit is not added.
Examples is not working.  Any idea? or do you have any working examples?

ok my mistake, forget to add flutter_form_builder: ^3.5.4. Thanks


feat: knex version 0.20 support

@jbrumwell I'm just opening this in the hopes you can merge and publish, to allow us to bump to knex 0.20. LMK if you need anything else adding (there were no new methods added in 0.20 so there isn't anything extra to actually add to the mock).

@bbeesley Thank you for your contribution released in<EMAIL_ADDRESS>:+1:


fix merge_asof usage in read_raw_eyelink()

closes #12480
the new parameter set is also not perfect:
direction=nearest can lead to duplicating samples (as a sample on an uneven ms is equally distanced to the previous and subsequent millisecond). you can see this behavior running the simplified example code in the issue.
but i think this is better than forward/backward which does not duplicate samples, but works worse with sfreqs < 500 Hz, as it can shift samples in the wrong direction

Thanks again! - I haven't looked at the code yet, but if merge_asof is still giving us some trouble I'm just going to crossref to my comment in the associated issue about refactoring the code to more closely the pattern in read_raw_neuralynx:
https://github.com/mne-tools/mne-python/issues/12480#issuecomment-1977293530


Page layout

add general html for grid boxes and header.
style header to system ui font, center to page and add solid black line at bottom of border.

resolves #2


Aliens Colonial Marines FPS problem

when you play the Aliens Colonial Marines game as 2 players player 2 runs at less fps but if you make a change with the alt + tab the fps player 1 is inverted with less fps and player 2 with 60 fps then in moments they go down a lot the fps is in player 1 or player to make the change with the alt + tab only happens to the one that touches less fps instead does not happen.

Strange, Aliens Colonial Marines is one of the games that I tested the least.
I'm assigning this as bug that needs fixing in the Colonial Marines code, will have to take a look.

@lucasassislar test with 3 players works almost perfect if slowness as in 2 players it seems that the game tubiera running less than 15 fps is unplayable I will try to make a video so that you can see the problems.

I close this as this is related to #64


cert-manager.com (Sectigo Network Agent) integration

What provider would you like to see added to NPM?
We would like to see an automatic certification renewal via the sectigo network agent.
https://cert-manager.com/
The Network Agent acts as "ACME Server" and renews all the cert's for us. For Apache, NGINX and Tomcat it can also automatically update the configuration. It would be nice to have such a function also for NPM. Because currently we use "Custom SSL Cert" and manually need to update our entries.
Have you checked if a certbot plugin exists?
yes

This would be great.
Need this, too
We provide with email, eab-kid and eab-hmac-key for generating and renew our certificates.

I'm up for this too!
May I add some reference material here
https://www.sectigo.com/resource-library/what-is-acme-protocol


Compare validator enhancement

Compare validator should be triggered on the field itself and also on the compareAttribute change event
['password', 'required'],
['repassword', 'required'],
['repassword', 'compare', 'compareAttribute' => 'password', 'message' => Yii::t('app', "Passwords don't match")],

Fill in the password field
Fill in the repassword filed
Go back and change password field - nothing happens.
The idea is to trigger the validation on both field.

Q
A

Yii version
2.0.9

PHP version
5.4.16

Operating system
Centos 7 x64

I guess fixing #12864  will fix this too.

Yes, you are right!
The nature of the issue is the same.
But in my opinion, single compare validator must compare both fields and must be triggered for both comparable fields instead of making two rules like in that issue:
['cred_rendimentos', 'compare', 'compareAttribute' => 'cred_despesas', 'operator' => '>=' ],
['cred_despesas', 'compare', 'compareAttribute' => 'cred_rendimentos', 'operator' => '<' ],

Yes, it makes sense.

Related to https://github.com/yiisoft/yii2/issues/12636

Won't be implemented in 2.0.


Arnej/use consensus map 1

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@havardpe please review

wait with this until later


Repopulating committees and bidlists takes a long time if blockchain is longer than 250,000 blocks

On testnet nodes, an incredibly long delay was observed anywhere between 2-5 minutes to re-initialize a node after shutting it down, when the blockchain was longer than 250k blocks. This is due to the fact that bidlists and committees and kept seperately between components, meaning that there will be a good handful of committee stores and bidlists re-populating on startup, and also because none of this information is persisted on disk, and so the node needs to walk up the blockchain many times when it restarts.

@julesdesmit I believe that the most efficient solution would be to create checkpoints (once every 5,000 blocks) containing, but not limited to the bid and stake lists, and persist those to the hard drive.

Closing this issue for now. With the merging of #130, provisioners are now only reinstantiated once, and block generator values are kept and updated on the database, which should cut down our initialization time tremendously.


Negative character_width shoud disable it

character_width: -1 shoud disable this specific formatting
Amazing project btw ❤

Don't quite follow. That is an invalid value. If anything the formatter should either ignore it and use the default value, or not even format anything.
If you want everything one lined you can pick an high value.

That is an invalid value

Alright, then integrate this behavior and it becomes a valid value! Magic isn't it?


unresolved external symbol "void __cdecl boost::throw_exception with VS2019

I get the compiler error unresolved external symbol "void __cdecl boost::throw_exception in VS2019 even if I have the custom function
namespace boost{
#ifdef BOOST_NO_EXCEPTIONS
template
void throw_exception(E const& e) {
//do nothing
}
#endif
}

There are two issues with your function:
First, it may not be a template. It should just take a std::exception const &.
Secondly, it is not allowed to return, so can't "do nothing". Call terminate or throw an exception, those are your choices.


How to upload file?

Hi,
I try my best so I couldn't get upload file.
Anyone show me how to get it?
Thank!

Upload file feature is not available in the current version.

Is it possible to autoclick the file chooser? This seems plausible, in view of this SO answer.

OK, it could be triggered by MouseEvents::click.


Relocate some yaml files to the proper directory (1.10)

Fix mislocation of yaml files happened in #8635 on release-1.10 branch.
Closes #10218
cf. patch for release-1.11: #10220
Note: this is not on master any more due to code->codenew work (#9363)

/assign @zacharysarah

/milestone v1.10

/assign @chenopis

Critical update to a frozen branch: https://github.com/kubernetes/website/issues/10218#issuecomment-420372458
/lgtm
/approve


Cannot delete from array?

package main

import (
	"fmt"

	"github.com/ohler55/ojg"
	"github.com/ohler55/ojg/jp"
	"github.com/ohler55/ojg/sen"
)

func main() {
	data := map[string]any{"a": 1, "b": []any{0, 1, "2", 3, 4, 5}, "c": 3}
	if err := jp.MustParseString("$.b[2]").Del(data); err != nil {
		panic(err)
	}
	fmt.Println(sen.String(data, &ojg.Options{Sort: true}))

}

I would have expected the output to be:
{a:1 b:[0 1 3 4 5] c:3}

But instead it is
{a:1 b:[0 1 null 3 4 5] c:3}

Is this a misunderstanding on my part or a bug?

That is expected behavior. If you want to not just remove a value but also collapse the array to fill in the removed item the jp.MustModify() is the function you are looking for.

And what about jp.Remove()?

Ah yes, jp.Remove() also works. Probably easier in this case too.

Are there any examples for MustModify? I find the documentation a bit confusing for someone who just discovered JSONPath.
Also there are some typos. pinic instead of panic for example.

There are unit tests for all the functions in the same directory as the code (ojg/jp). That probably the best place to start.
There are some links to the JSONPath spec in the README.md although that can get pretty overwhelming.

The docs for jp.MustModify were updated in develop if you would like to check it out.

Okay to close this?


Added Python package

This PR allows to install thunderSVM as a Python package for Python users.

Thanks! Your work is very useful.

Thank you for the great work in thundersvm :)


⚠ Jenkins Plugins Artifactory has degraded performance

In cdddcc8, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Jenkins Plugins Artifactory performance has improved in e3eeba2 after 12 minutes.


Code completion show defined conditions

Conditions defined with define-condition should show up as completion options.

Fixed in new release


FR: Drag/drop from pane into active note as link

Firstly, thanks a lot creating this very useful plugin which changed my workflow a lot. One feature I kept thinking about and would use a lot if available is to drag a note in the pane into active note to create a link there, much like what File Explorer is capable of doing. Right now, to archive the same it would take a lot of more extra steps. Hopefully you can consider it being implemented for the plugin.

Thank you for this request! I thought it might be very cool!
I did not know how to drag the items on Obsidian. (#73 is also the same).
However, I have noticed that we could handle that by using the internal APIs, accidentally.
It will be implemented soon!

@joellti
This feature has been released on 0.16.10!
Could you please try the latest version and check the behaviour?

Just tested and it worked great. Thanks so much for your quick turnaround to have it implemented.


[deps] add rspec ~>3.0 dependency

Custom matcher syntax was introduced in rspec-expectations 3.0
Addresses https://github.com/mojotech/dill/issues/134

I'm not sure if rspec should be explicitly added as a dependency since dill also supports minitest. This means everyone will be required to import rspec into their projects whether they use it or not...

@david I agree, it's not ideal. Is there another way to ensure that for people using the RSpec/Cucumber features of Dill that they have the correct version of RSpec?

Maybe just add it to the readme and wiki?

Either that or a quick check for RSpec::VERSION, if that's implemented.


Search all with CP850 encoding does not work

VSCode Version:1.42.0
OS Version: windows 10.0.15063

Steps to Reproduce:

Set default encoding to CP850 in the settings
Search all (ctrl + shift + f) in my workspace.

It starts searching and displays good results but almost instantaneously disappear to show the message "Unknown encoding : cp850". It works fine with windows1252 and UTF8 though.

Does this issue occur when all extensions are disabled?: Yes
EDIT:
I know an issue has already been opened and then closed but the problem still exists.
Thank you,
Guillaume.

https://github.com/microsoft/vscode/issues/68237


OnEndReached and pull to refresh in RkGallery

How can I pass onEndReached and RefreshControl to the RkGallery? I need to be able to page images as users scroll? thanks much

Hi @adamsolomon1986
Thanks for report. That's a framework bug and currently there is no way to pass custom props to RkGallery. But you can try modify existing code directly in your node_modules with following code
Try this and welcome to create PR if everything is okay for your case

This stays actual for v3.*
Check out the brand new v4 update!


Updated outdated description of bunit.template

Pull request description
Updated outdated description of bunit.template on index.html of bunit.
Is available for NUnit and MSTest too ;-)

Perfect - thanks for the PR. I will close the other one and merge this one. Thank you @HSchwichtenberg


Community Agreement Form

Progress:

Added has_agreed field for user and profile models on frontend and backend to keep track of whether or not they accepted the community agreement
Updated necessary API calls
Added community agreement widget to shared folder
Added community agreement widget to profile page where user can click on box to read community agreement
Added working accept button in the pop up box
FOR EXISTING USERS: When accessing home page of the csxl website, a pop up will show up with the agreement asking to accept if the user has not accepted yet


Added 3184-count-pairs-that-form-a-complete-day-I

Related Issue
Added 3184-count-pairs-that-form-a-complete-day-I
[Cite any related issue(s) this pull request addresses. If none, simply state "None”]
Description
Added 3184-count-pairs-that-form-a-complete-day-I
[Please include a brief description of the changes or features added]
closes #3580
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Security enhancement
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)
[Attach any relevant screenshots or videos demonstrating the changes]
Checklist

[ x] I have performed a self-review of my code.
[ x] I have read and followed the Contribution Guidelines.
[ x] I have tested the changes thoroughly before submitting this pull request.
[ x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ x] I have commented my code, particularly in hard-to-understand areas.
[ x] I have followed the code style guidelines of this project.
[ x] I have checked for any existing open issues that my pull request may address.
[ x] I have ensured that my changes do not break any existing functionality.
[ x] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.
[ x] I have read the resources for guidance listed below.
[ x] I have followed security best practices in my code changes.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Please read the following resources before submitting your contribution:

[ ] Code Harbor Hub Community Features
[ ] Docusaurus Documentation
[ ] React.js Documentation
[ ] Markdown Guide
[ ] MDX Documentation
[ ] Mermaid Documentation

Resources for DSA Features on Our Site:

[ ] Tabs: Switch Languages
[ ] Line Numbering
[ ] Math Equations

Contribution Guidelines
Thank you for considering contributing to our project! To ensure smooth collaboration and effective contribution management, please adhere to the following guidelines:
Issue Creation

Limit on Issues:

Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently.

Contribution Levels

Basic Contributions:

This project is primarily focused on documentation. Most of the setup has been completed, so contributors will generally need to work on basic code tasks, such as writing tests.
For these tasks, issues will be assigned the level1 label.

Acknowledging Hard Work:

If a contributor puts in significant effort on a task, the issue will be upgraded to level2. This is our way of recognizing and appreciating extra effort.

Feature Additions and Component Work:

Contributors working on new features or components using JSX/TSX will be assigned a level based on the complexity and quality of their work.
The more complex and valuable the contribution, the higher the level assigned.

Level Definitions

Level1:

Tasks are straightforward, such as fixing minor bugs, writing tests, or making simple documentation updates.

Level2:

Tasks require more effort, such as addressing complex bugs, improving existing features, or making substantial documentation improvements.

Level3:

Tasks are highly complex and involve significant new feature development, major refactoring, or extensive contributions to the project’s core components.

We look forward to your contributions and appreciate your effort in helping us improve the project!

@Ajay-Dhangar Please review the pull request soon


Feature:custom user agent

The default go http client user agent can be blocked by several tool (cloudflare, internal WAF, etc.).
This PR add a default custom user agent for the tool that can be configured from the config file.

Nice to know! Settings a custom user agent is not sufficient? I’m not sure allowing to customize the user agent through the configuration is required.

Settings a custom user agent is not sufficient?

For my use case it's enough, but leaving the option for people to customize it for their need seem like a good idea.

And the be spec compliant the user-agent should be privatebin-cli/1.2 (source; https://github.com/gearnode/privatebin)

Settings a custom user agent is not sufficient?

For my use case it's enough, but leaving the option for people to customize it for their need seem like a good idea.

Let’s remove the configuration for now. If someone ask for this we can think to introduce it.

As you may not have the time to update the patch, I've introduce a similar patch.

I was away since the PR, thanks for the patch.
I will update the package for https://github.com/gearnode/privatebin/pull/3


Implement a Arm Sprite animated by code

Requirements:

[ ] As a placeholder, apply a shader on Bob to hide one of his arms whenever a gun is being held
[ ] When a gun is acquired, a Arm sprite must be displayed. It's anchor will be the same as the Nodes that hold the gun
[ ] Some guns will not use this system, so there must be a way to turn it off depending on the gun's type

@vitormaduro I believe this was tested, so feel free to close it if you feel nothing else is to be done here.