Type Error: 'setting_kwargs'

Hello,
I am facing the same issue as the one described here. I ran this command python3 src/train.py   --algorithm svea   --seed 0 but got this error TypeError: load() got an unexpected keyword argument 'setting_kwargs'.
I did install dm_control which comes with mujoco 2.2.0 and I dont see any way to install mujoco 2.0.0 from the mujoco download page.
@nicklashansen Any help would be appreciated.

Hi, MuJoCo 2.0.0 is still available for download from the legacy site: https://www.roboti.us/download.html
Note that you will need a license for this version, which is provided for free on the same site. Let me know if this does not solve your problem!

Hi, thanks for the reply. I downloaded MuJoCo 2.0.0 from the legacy website above with the license. Currently, on my system, I have MuJoCo 2.2.0 (this came when I installed dm control with pip). I tried removing mujoco 2.2.0 using pip3 uninstall mujoco and then I moved the downloaded mujoco 2.0.0 folder from the legacy website into the directory /home/anaconda3/envs/dmcgb/lib/python3.7/site-packages (where the original mujoco install was) and when I ran  python3 src/train.py   --algorithm svea   --seed 0, I got this error ModuleNotFoundError: No module named 'mujoco' Is this the correct way to use the newly downloaded Mujoco 2.0.0 or am I missing something?
Thanks!

I got it to work, thanks! @nicklashansen

Hi, I also encountered this problem. Have you handled it successfully? load() got an unexpected keyword argument 'setting_kwargs' #23. @nicklashansen


Don't set Content-Length

This header is set automatically by the HTTP module (much more efficient) and does not need to be set in the library. Setting Content-Type is fine since that is not something the HTTP module can infer.

It appears that the HTTP node module isn't setting the content-length when koa doesn't set it explicitly.

yeah, the only time it could be set is if you do .end(buf) without a .write(). however, it's impossible to know the content length is if the response body is streamed with .write()s

This can probably be closed then.


Initial Commit: Store update and resource config refactor

Issue #: /bcgov/entity#6953
Description of changes:

Store refactor.
Created configuration file.
Removed/refactored older resources after integrating them into config object.
refactored component usage of resources, where applicable.
Updated / parameterize unit tests, see working examples.
Apply steps/components to configuration.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Looks great. Woot!

woo hoo! Thanks sir. Your input here was extremely valuable!


Clippy fixes

Some of these are arguably not strictly better, but didn't really seem worse to me either.
I tried changing all &Unit to Unit, but I felt like the code was worse with more frequent use of * and &.

r? @Eh2406
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 6b07c8da63c0896cc31fa6cb8b47a71fe29c88e2 has been approved by Eh2406

:hourglass: Testing commit 6b07c8da63c0896cc31fa6cb8b47a71fe29c88e2 with merge 13e43aab292c68879e62d1a5b11b3d3c56665d97...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: Eh2406
Pushing 13e43aab292c68879e62d1a5b11b3d3c56665d97 to master...


Adding transition  to Catalog navbar

Please Make pr with base branch <27-adding-transition-to-catalog-navbar>

the catalog section is not working can you tell does it require any api or data

ya of course you have to setup backend which is not possible
so if you can make make dummy data for it

ok but i did no back end so can you assigned to anyone else sorry for that

Ya no problem you can take other issues too


batch undelegations - sort by unbalance ratio

Context
Changed the sorting back to being by unbalance ratio now that we can support multiple batches.
This is how the sorting was originally done as it's more fair. We had changed it temporarily to avoid the 32 message constraint.

Looks great! For what it's worth, I think the main advantage of using the UnbalancedRatio is it helps simplify the final delegations Stride has (e.g. if any validators become zero weight, we immediately undelegate from them). It also feels a bit more fair to me too, I'm on board to reverting to this algorithm!


SharingWithUUNET in MixedRealityToolkit references -Examples project

Overview
The Player prefab in the SharingWithUUNET folder of MixedRealityToolkit has a PlayerController script that only exists in the MixedRealityToolit-Examples/SharingWithUUNET/Scripts folder
Expected Behavior
No references from MRTK to MRTK-Examples
Actual Behavior
Reference that breaks if you do not include the Examples project
Unity Editor Version
Any
Mixed Reality Toolkit Release Version
Dev_Working_Branch

We should probably check the stabilization branch as well.

Just checked. The same issue exists in the Stabilization branch.

Closing issues older than 180 days. If this is still an issue, please reactivate with recent information.


attempt2: Solve the problem with faulty WaitForControlledPods

Fix #1259
/assign @mm4tt
/hold

Density test failed on unrelated issue (pod startup time latency: https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/perf-tests/1267/pull-perf-tests-clusterloader2/1261620111752564736/)
/retest

/hold cancel
In case of issues, we should revert it, but it seems to work fine.
@jprzychodzen as oncall - FYI


Fix FreeBSD compatibility

The header <malloc.h> is deprecated / non-standard. Instead, <stdlib.h> should be used, which is defined by C99 and later.
On FreeBSD, using <malloc.h> generates an error:
In file included from ./modules/imgcodecs/src/rgbe.cpp:50:
/usr/include/malloc.h:3:2: error: "<malloc.h> has been replaced by <stdlib.h>"
#error "<malloc.h> has been replaced by <stdlib.h>"
 ^

This PR fixes the issue for FreeBSD specifically (since it currently generates an error), but it could also be fixed for all systems by removing the #ifdef altogether, since on __APPLE__, <stdlib.h> is also available. The entire include block would then become:
#include "precomp.hpp"
#include "rgbe.hpp"
#include <math.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

However, this should be properly tested on all platforms first, which is why I first propose this FreeBSD-only fix.

Thanks!
Could you please leave #include <stdlib.h> only (without "#ifdefs")?

Just changed the commit accordingly, getting rid of <malloc.h> altogether, replacing it with <stdlib.h> for all systems.

:+1:


Added option to load extra properties on list extension GetListByTitle and support for lookup fields, which isn't a site column.

When using list extension GetListByTitle, I often have a need for loading extra properties.
Eg.
var list = web.GetListByTitle("ListTitle", l => l.ContentTypes);
This small change will make it possible to do so while keeping backward compability.

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

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

TTYL, MSBOT;

Export has become really slow because Get Security is called extremely many times.
I've fixed this bug by moving the GetSecurty method outside the FieldCollection loop within the ListInstance object handler.

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


Bug: The right-click menu is missing items that are only in the new version of Win11 menu

Description

These things are missing from the right-click menu of Files
Steps To Reproduce

Find a file
Right-click

Files Version
<IP_ADDRESS>
Windows Version
10.0.26100.2161
User ID
18f8c936-5e4e-438c-9082-948a200d4eb0
Log File
debug.log

Thank you for the feedback.
We're tracking on in #8251. Click Subscribe to get notified of further news.


False display of the ranking as Unknown

Hi, I was looking at one of my Games played recently, while i noticed that the rankings of 3 players (Axe Legend 1, Undying -> Legend 0, Wraith King -> Legend 1) were displayed as Unknown although they had a ranking already (in original Replay). Is it because of Steam account privacy or maybe a mistake somewhere else, like a parsing mistake?

match ID?

Damn your fast in replying =),<PHONE_NUMBER>.

The ranks are not linked to the replay but to the account ID.  Since these players are anonymous in the match we don't display their ranks either.

Thank you very much for the explanation.


Span-of-buffer and better error messages at eof

So, if a proc-macro expects some input (in this case a trailing ;) which is missing at the end of input, this can result in bad error messages:
error: expected `;`
   --> crates/kas-widgets/src/label.rs:189:1
    |
189 | / impl_scope! {
190 | |     /// A label supporting an accelerator key
191 | |     ///
192 | |     /// Accelerator keys are not useful on plain labels. To be useful, a parent
...   |
304 | |     }
305 | | }
    | |_^

Um... where exactly is that ; missing from? (Hint: not the end of the whole input. The relevant line isn't even shown.)
Why? Because parsing a token group such as { .. }works by constructing a new ParseBuffer over that group's content. Generating an error message invokes ParseBuffer::span (and thus Cursor::span), which must return something when the buffer is empty, and so returns Span::call_site().
So how could we solve this?
I first looked at a specific case where this error is omitted: an input group (or "item") is expected to be terminated with ;. Idea: I can highlight the whole item, which is part of the current ParseBuffer. To do this, I would want fn ParseBuffer::span (or another method) to return the span of the whole buffer, not just the next token. This functionality is missing (though possible to emulate by joining spans, as done in Tokio here).
More general solutions may be possible without external changes (i.e. only changing syn, probably changes may be considered API breaking (more specifically, behaviour of existing methods changes)):

Make Cursor::span at eof return an empty span at the end of the buffer. (Are empty spans supported? Could they be?)
Make ParseBuffer::span at eof return the span of the next token. Problem: the buffer does not know about the next token. Solution: add the next token's span to the ParseBuffer as a field (somehow), added at construction time. Probably at the end of the whole macro's input this will still need to resolve to Span::call_site (or an empty span at the end).

Syn already behaves pretty much exactly how you described. See below for demo.
I skimmed through your macro and got a little lost, but I suspect the behavior you are seeing is just your macro being poorly behaved — is it converting #[widget{ derive = self.0 }] (where the {} is correctly spanned with the incoming span) into some other representation that throws out the {} tokens entirely?
// [dependencies]
// syn = "1"

use proc_macro::TokenStream;
use syn::parse::{Parse, ParseStream, Result};
use syn::{braced, bracketed, parse_macro_input, DeriveInput, LitInt, Token};

mod kw {
    syn::custom_keyword!(derive);
    syn::custom_keyword!(widget);
}

#[proc_macro]
pub fn demo1(input: TokenStream) -> TokenStream {
    parse_macro_input!(input as DeriveInput).attrs[0]
        .parse_args_with(|input: ParseStream| {
            input.parse::<kw::derive>()?;
            input.parse::<Token![=]>()?;
            input.parse::<Token![self]>()?;
            input.parse::<Token![.]>()?;
            input.parse::<LitInt>()?;
            input.parse::<Token![;]>()?;
            Ok(())
        })
        .unwrap_err()
        .to_compile_error()
        .into()
}

#[proc_macro]
pub fn demo2(input: TokenStream) -> TokenStream {
    struct Manual;
    impl Parse for Manual {
        fn parse(input: ParseStream) -> Result<Self> {
            input.parse::<Token![#]>()?;
            let bracketed;
            bracketed!(bracketed in input);
            bracketed.parse::<kw::widget>()?;
            let braced;
            braced!(braced in bracketed);
            braced.parse::<kw::derive>()?;
            braced.parse::<Token![=]>()?;
            braced.parse::<Token![self]>()?;
            braced.parse::<Token![.]>()?;
            braced.parse::<LitInt>()?;
            braced.parse::<Token![;]>()?;
            unreachable!()
        }
    }
    parse_macro_input!(input as Manual);
    unreachable!()
}

repro::demo1! {
    #[widget{ derive = self.0 }]
    struct S;
}

repro::demo2! {
    #[widget{ derive = self.0 }]
    struct S;
}

fn main() {}

$ cargo check

error: expected `;`
 --> src/main.rs:2:31
  |
2 |     #[widget{ derive = self.0 }]
  |                               ^

error: expected `;`
 --> src/main.rs:7:31
  |
7 |     #[widget{ derive = self.0 }]
  |                               ^

is it converting #[widget{ derive = self.0 }] (where the {} is correctly spanned with the incoming span) into some other representation that throws out the {} tokens entirely?

Yes, actually. This was intentional to mirror the behaviour of #[proc_macro_attribute] — although forms like #[doc = "Some doc"] are legal from the point of view of attributes, proc_macro_attribute accepts only #[PATH GROUP] forms then passes only the contents of the GROUP token — the impl can't even detect whether {}, [] or () is used.
I'm not sure whether this design is an oversight, but I copied it for use by impl_scope — see ScopeAttr::apply and parse_attr_group (which should refer to #[proc_macro_attribute]. I guess I should use Attribute::parse_args (_with); I wasn't aware of these methods.

Thanks for the suggestions; I've modified ScopeAttr::apply to take the Attribute as an argument, allowing usage of Attribute::parse_args.
There's one minor issue: parse_args does not consider #[my_attr] an acceptable alternative to #[my_attr()], which is the case for #[proc_macro_attribute]. Is this a bug (as parse_args supposed to emulate #[proc_macro_attribute]? It would certainly help me if this were the case.

I prefer the current behavior of parse_args. For attributes that want to support #[my_attr] as a synonym for #[my_attr()], that can be implemented by matching on Meta.
let attr: &Attribute;

match &attr.meta {
    Meta::Path(_) => Default::default(),  // or syn::parse2(TokenStream::new())?
    Meta::List(meta) => meta.parse_args()?,
    Meta::NameValue(meta) => {
        return Err(Error::new_spanned(meta, "expected #[my_attr] or #[my_attr(...)]"));
    }
}


Fix path, update versions

LGTM! can you try an build a version v0.0.18?

It will never work with versions that doesn't have the path: infrastructure/ami. It will only work with the branches which will be now branched out from main.

LGTM! can you try an build a version v0.0.18?

It will never work with versions that doesn't have the path: infrastructure/ami. And the branch v0.0.18 won't work because it is branched out from a previous timestamp. So the packer files aren't there in v0.0.18 or any other branch before. https://github.com/huggingface/optimum-neuron/tree/v0.0.18-release
It will only work with the branches which will be now branched out from main.

Okay but what if, e.g. make release 0.0.19 and noticed the AMI pipeline fails since there was some change in the base AMI or we need to update a version or another file in infrastructure/ami?
How can we then release 0.0.19. From what you shared we would not be able to release this version since it would check out the AMI code from the release which would not have the fixed or changes.

@philschmid with this change, if any change is made in the repo and a PR is made to the main, the workflow will be triggered and the ami will be built.
run: packer validate -var "optimum_version=${{ github.event.inputs.tag || github.head_ref || github.ref_name }}" -var "region=${{ env.AWS_REGION }}" hcl2-files

When someone manually specifies it will use that tag and during pull request it will use the name of the source branch (gtithub.head_ref is only set during PR) and if pushed to main it will use github.ref_name which will be main.

So from now on whenever we will make the release, we would be sure that the build-ami was successful.

Thats not a dependency we should have! The AMI is not directly related to the code. IMO we should be able to build AMIs for a specific optimum version, but the AMI code doesn't need to in the library code.
Why can't we use the inputs.tag and pass that into the packer script and based on this either install from pypi or install from git. We could remove all the with in the checkout here
            uses: actions/checkout@v4
            with:
              # If the workflow is triggered manually or by schedule, uses the tag, otherwise uses the current branch
              ref: ${{ github.event.inputs.tag || github.ref }}

this would then always check out the "branch/event" which triggered the workflow and we pass the inputs.tag into  packer validate -var "optimum_version=
And then in the packer bash we simply do
# Check if OPTIMUM_VERSION starts with 'v'
if [[ $OPTIMUM_VERSION == v* ]]; then
    # If it starts with 'v', install that version using pip
    pip install optimum-neuron==$OPTIMUM_VERSION
else
    pip install git+https://github.com/huggingface/optimum-neuron.git@$OPTIMUM_VERSION
fi

^ you have to validate it works
This will allow us to build an AMI based on the "dispatch" + "tag" and if it is triggered by action, e.g. pull request we us that branch?

@philschmid I removed all the checkout code. I agree we should use the branch that triggered the workflow.
We don't have to do this logic:
# Check if OPTIMUM_VERSION starts with 'v'
if [[ $OPTIMUM_VERSION == v* ]]; then
    # If it starts with 'v', install that version using pip
    pip install optimum-neuron==${OPTIMUM_VERSION#v}
else
    pip install git+https://github.com/huggingface/optimum-neuron.git@$OPTIMUM_VERSION
fi

As this thing is always taken care by:
optimum_version=${{ github.event.inputs.tag || github.head_ref || github.ref_name }}"

It will always be the value which triggers the workflow.
Also as we are copying some files from the git repo to the local machine while building the image the pip logic won't copy the files for that we will have to clone the repo anyhow.

Once merged you will also be able to build with v0.0.18, I tested it with the current workflow


[PORT] Update ESRP authenticode JSON per SHA1 Digest Deprecation requirement.

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/3824

N/A


Update to the latest Godot Engine master.

I think this is going to fail because of godot-cpp.
Checking.

godot-cpp needs to be set to "4.0" in an env variable


Add pydantic

I added the pydantic scheme.

@yaroslavMain looks like the tests are still unhappy. You can run poetry run pytest ./tests locally to test them on your machine


Update app ratings API examples

New app ratings endpoint doesn't have rating_summary and store_fronts attributes anymore.

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context

Inaccurate documentation.

I've followed the code in https://github.com/fastlane/fastlane/blob/master/spaceship/spec/tunes/app_ratings_spec.rb to update the examples.
Description

Latest Spaceship::Tunes::AppRatings doesn't provide rating_summary attribute, so the examples of how to use the API were outdated.
ratings.store_fronts is also not available, so I've removed line 350.

I signed it!

@DraXus ohhh thx, good catch 🙏 ! i missed that when changing spaceship

Wohoo, thanks everyone 🚀


Enemy chariot attacked my trireme from land

The AI logic checked whether the unit was boardable, but not the owner of the unit. So it tried to board you trireme.


Filtering tags based on a logical query

Originally posted in Issue #299
Tags are very useful for differentiating between runs, but are currently not directly visible when displaying runs and no advanced filtering based on combinations of tags is possible. The idea is to add the following features:

Show tags in a separate column when runs are displayed.
A new filter (--tags QUERY) for filtering tags based on a logical query.

An example on how the filtering using a query works. A query (apple & not orange) | pear will be evaluated as ('apple' in run_tags & 'orange' not in run_tags) or 'pear' in run_tags internally. This allows for filtering runs on complex combinations of tags. None can be used to match runs without a tag (e.g., (apple | None)). Instead of the logical character | and &, the operators or and and can be used respectively (e.g., (apple and not orange) or pear).

Thanks for this, and sorry for the long delay in response. I'm going to slate this for the next release (the one after the 0.8 release that already has an RC). What tests would be good to add to ensure that this is working properly and that we don't accidentally break this feature? Are there edge cases that you came across while developing this?


Error parsing JSON data as at 07-Feb-2022 during startup.

HA v2022.2.4
HACS v1.22.0
BOM Integration v1.1.1
Last couple of days have started getting this error during startup.
Error setting up entry Ashburton for bureau_of_meteorology
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 327, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/bureau_of_meteorology/init.py", line 35, in async_setup_entry
await collector.async_update()
File "/config/custom_components/bureau_of_meteorology/PyBoM/collector.py", line 85, in async_update
self.locations_data = await resp.json()
File "/usr/local/lib/python3.9/site-packages/aiohttp/client_reqrep.py", line 1103, in json
raise ContentTypeError(
aiohttp.client_exceptions.ContentTypeError: 0, message='Attempt to decode JSON with unexpected mimetype: text/html', url=URL('https://api.weather.bom.gov.au/v1/locations/r1r0t1')
Has the BOM JSON data changed?  It seems valid.
{"metadata":{"response_timestamp":"2022-02-09T03:32:17Z"},"data":{"geohash":"r1r0t1","timezone":"Australia/Melbourne","latitude":-37.87261962890625,"longitude":145.0689697265625,"marine_area_id":"VIC_MW005","tidal_point":"VIC_TP027","has_wave":false,"id":"Ashburton-r1r0t1","name":"Ashburton","state":"VIC"}}

I have just started helping with this project. Are you still having this issue?

Closing as no response forthcoming.


[BEAM-8646] Restore original behavior of evaluating worker host on Windows until a better solution is available.

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.
[ ] 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.

R: @violalyu @chadrik

Thanks @tvalentyn !


Use fractional coordinates for CSS positioning

Especially on HiDPI devices, ol.Overlay overlays wiggle on top of the map when panning. This can be avoided by not rounding the css coordinates.

Thanks Guys!

Nice stuff! Even better would be to use translate(x, y, 0) to improve the performance. Tricky to keep compatibility with older browsers though.

I've been testing this for a while now and it did not fix the overlay wiggle. It can be most seen on an Android mobile device running Chrome browser. Does anyone have any other ideas of how to solve this issue? We get considerable wiggle on our label overlays.
Video Located Here
Thanks

@wayofthefuture It looks like the wiggle in your video is caused by something else. It's more than a pixel. Maybe you're using some custom placement options? Or maybe you apply css transform in additon to the ol.Overlay positioning?

Yes we are definitely using css transforms for sure. The element we are passing into the ol.Overlay constructor options has a transform (rotate) and transform origin applied. So maybe it has something to do with that.

Hi,
We have a performance problem trying to migrate from version 4.2.0 to version 4.3.0, and I think this commit is the culprit.
Our app can display a lot of overlays on screen, and I see a few problems with this commit :

Safari on IOS gets a performance hit just because of this change
If overlays are used to display text on a low dpi screen (like a standard PC) then it will not be able to get "pixel perfect" alignment of text, and it will become fuzzy. However it seems like chrome does some rounding internally anyway making this change pointless. However I didn't measure if chrome gets a performance hit with this change, nor did a test with Firefox
Not rounding css positions is not consistent with the behaviour or geometries in OL. Many OL styles have a propery called "snapToPixel" which defaults to "true".

So I suggest to revert back to the old behaviour to :

Restore performance to what it was before
Be consistent with what is done in styles

An I also suggest to add a "snapToPixel" (defaulting to "true") to ol.Overlay() constructor for people that would like to have non rounded css positions.

Sounds reasonable @ogo-adp. Pull request welcome.

@ogo-adp @ahocevar You should be able to get rid of your performance problems by sub-classing ol.Overlay. Here is an example in Typescript:
import olOverlay from 'ol/overlay';

export class Overlay extends olOverlay {
  private element_: HTMLElement;

  updateRenderedPosition(pixel: [number, number], _: any) {
    const x = Math.round(pixel[0]);
    const y = Math.round(pixel[1]);
    this.element_.style.transform = `translate(${x}px, ${y}px)`;

    (this as any).setVisible(true);
  }
}

@hccampos I'm not sure it is feasbile to subclass anything when using the npm package.
@ahocevar Currently i'm not able to prepare a pull request (i'd love to ...). Would you accept to make the modifications ?

@ogo-adp Not a priority for me. I don't recommend to use ol.Overlay for anything other than tooltips or info popups, with only on visible at a time, and for that use case there is no performance issue. For anything else, you should be using vector layers and text styles. #4577 might be of interest in this context.

@ahocevar Well we have a true use case for overlays (stickers for moving objects, and will need HTML for that, and even #4577 would not be enough for us). Could it be possible to at least revert #7098 in next release because it make performance worse for no actual benefit ?

Maybe someone else interested in this can submit a pull request?

@ogo-adp you can also send me an e-mail, maybe we can figure out a way.

If this change didn't fix an issue that anybody was running into, and it is causing a new issue, it seems like reverting is the right thing to do.  If that sounds ok to you @ahocevar, I can cut another patch release with the change.

Looks like @wayofthefuture's issue was not fixed with this change anyway. I'll revert.

#7258 created. @tschaub, I will merge as soon as Travis is green. Then you can create a patch release. Thanks!

Many thanks @tschaub and @ahocevar :)

Published<EMAIL_ADDRESS>(website and full-build artifacts up soon).


Add ClusterGroup reference in ClusterNetworkPolicy AppliedTo

Allow ClusterNetworkPolicies to select ClusterGroups in appliedTo in spec and rules instead of standalone labelSelectors.
ClusterNetworkPolicy spec/rules can now be set with a list of ClusterGroups in appliedToGroups in spec or ingress rules and
egress rules instead of setting stand alone selectors. These ClusterGroup references in rules will be converted to corresponding control plane AppliedToGroups with the UID matching that of it's ClusterGroup.Group members calculated by the internal Group corresponding to the ClusterGroup will be used to determine workloads and set in AppliedToGroup and
Agents will continue to consume these appliedtos in the form of AppliedToGroup messages.

Codecov Report

Merging #1750 (7badff6) into master (9d3d10b) will decrease coverage by 28.08%.
The diff coverage is 24.21%.

@@             Coverage Diff             @@
##           master    #1750       +/-   ##
===========================================
- Coverage   63.31%   35.23%   -28.09%
===========================================
  Files         170      192       +22
  Lines       14250    16241     +1991
===========================================
- Hits         9023     5723     -3300
- Misses       4292     9671     +5379
+ Partials      935      847       -88

Flag
Coverage Δ

kind-e2e-tests
35.23% <24.21%> (-20.15%)
:arrow_down:

unit-tests
?

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

The diff coverage is 24.21%.

@@             Coverage Diff             @@
##           master    #1750       +/-   ##
===========================================
- Coverage   63.31%   35.23%   -28.09%
===========================================
  Files         170      192       +22
  Lines       14250    16241     +1991
===========================================
- Hits         9023     5723     -3300
- Misses       4292     9671     +5379
+ Partials      935      847       -88

Flag
Coverage Δ

kind-e2e-tests
35.23% <24.21%> (-20.15%)
:arrow_down:

unit-tests
?

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

Impacted Files
Coverage Δ

.../agent/apiserver/handlers/networkpolicy/handler.go
58.33% <ø> (ø)

pkg/agent/apiserver/handlers/ovsflows/handler.go
19.51% <0.00%> (-59.44%)
:arrow_down:

...gent/controller/noderoute/node_route_controller.go
40.15% <0.00%> (-6.32%)
:arrow_down:

.../agent/flowexporter/connections/conntrack_linux.go
0.00% <0.00%> (-29.17%)
:arrow_down:

pkg/agent/nodeportlocal/k8s/annotations.go
0.00% <0.00%> (ø)

pkg/agent/nodeportlocal/k8s/controller.go
0.00% <0.00%> (ø)

pkg/agent/nodeportlocal/npl_agent_init.go
0.00% <0.00%> (ø)

pkg/agent/nodeportlocal/portcache/port_table.go
0.00% <0.00%> (ø)

pkg/agent/nodeportlocal/rules/iptable_rule.go
0.00% <0.00%> (ø)

pkg/agent/nodeportlocal/rules/rules.go
0.00% <0.00%> (ø)

... and 189 more

/test-all

simple rebase

/test-all

resolved merge conflicts

/test-all


Fix significant rounding error in drawing elliptical arcs

In the ellipse example, it is clear from inspection that the major and minor axes and the elliptical arc are in error by a small margin--compare with the positions of the blue dots.

I thought I just fixed this but in fact the yellow and orange arcs are now twice as wide as intended.

The arc function arguments changed from diagrams 1.0. The second arg is now a sweep instead of an ending angle.


Backport HHH-14828 to branch 5.4 - Bytecode enhancement generates invalid bytecode for final fields

https://hibernate.atlassian.net/browse/HHH-14828
Backport of #4231
I did not intend to backport this initially, but I think it will be necessary to fix https://github.com/quarkusio/quarkus/issues/20186 in old versions of Quarkus.

@Sanne Could you please confirm it's fine to backport this to 5.4?

@Sanne Could you please confirm it's fine to backport this to 5.4?

yes it's fine but I wouldn't rush it for non-trivial patches. WDYT of leaving this here for a couple weeks and see feedback from 5.5+?

Works for me. But I'll bet we will just forget about it :D Let's find out!


add qx translation

to extract *.po files.
Similar to generate translation

That is already done automatically. The .po files are extracted and updated by the compiler.

Then there is missing some doc.
qx create test -I
cd test
add "de" to locales:  "locales": ["en", "de"],
run qx compile
i can't find any po file?

see https://github.com/qooxdoo/qooxdoo-cli/issues/38#issuecomment-346944443

You need to be calling this.tr("myMessageId"), that's how the compiler finds the message and know to generate a translation for it

ok, got it. I'll change the text in readme.txt of the skeletons

@johnspackman has noted that "de", "en" etc. won't work anyways because the compiler requires "real" locale codes such as "de_DE" etc.
a) This seems to be not BC with existing applications, which use only two-letter locales
b) we need to validate locales in qx create and ideally, use the same locale data that qxcompiler uses to validate entries. @johnspackman, is there a way to use the qxcompiler API to get this data easily?

@cboulanger existing applications use both, two-letter and combined locale definitions. de is for de* (i.e. it catches de-AT and de-DE if they don't exist.

@cajus Ah ok, thank you. I didn't know that.

Just checked with an existing one. Current correct spelling would be en_US for american english. So dash vs underline.

But won't two-letter code apps break with the new compiler, @johnspackman?

I don't know the state of the Part support in qx-compiler, but that's also a must have:
"i18n-as-parts"    : true
is present in the current generator. It can bundle locales as parts - which makes sense if you've 25 locales with ~200k each.

@cboulanger No, what happens is that if you just specify the language you will only get the default localisation settings for that language; this means that "en" will always return a date format "m/d/y" because the defaults happen to be US-centric (for dates anyway).  It works but it's wrong (unless you happen to be in the US).  If you specify "en_GB", the GB localisation data for "en" is loaded on top of the "en" data, overriding those "en" defaults.
It means though that you have to know in advance which language/locale combinations you want to support, so that the generator/compiler knows to extract the data it requires and insert it into the boot loader.
@cajus the i18n-as-parts is not in compiler at the moment, however it only loads a tiny subset of locale data is provided with the app - the entire "en" language is 4,698 bytes (pretty printed, un-minimized JSON data), and the "en_GB" specialization is another 1,858 bytes.  When minimized, "en" and "en_GB" combined come to 4,782 bytes.

@johnspackman yes, but if you have large translations, it's a problem. Or at least a waste of resources. For one application I've more than 4MB of translations. If this is pulled in at a time, 1/25 is used. 24/25 are wasted and would be loaded.
@hkollmann just deleted my de.po and it's getting re-generated upon qx compile. The contents is not valid in the moment, because qooxdoo-compiler needs an update on npm AFAIK.

ah, yes I see what you mean!  That's not supported yet but it needs to be. I'll create an issue

Maybe it would be nice to have a seperate command to extract the translations (i.e. qx translate). Most of the time, you don't want qx compile make that.


Normalize storage_cost and storage_rebate in gas summary

as in https://github.com/MystenLabs/sui/issues/7734
the idea here is to make storage_cost and storage_rebate 0-based.
In the simplest example one would get storage_cost and storage_rebate with the same value and it may be a confusing experience. So we are trying to normalize the values and in the previous simple example report 0 for both.
I had to fix some test and there are tests in rosetta that I am investigating.
In general though it is interesting that if the 3 values computation_cost, storage_cost and storage_rebate are not applied in the proper order one may get an improper amount of gas to apply. In other words if I am trying to find out how much gas I need to provide, I am wondering whether it would be safer to provide computation_cost + storage_cost without regards to rebates. I am not even clear yet if the failing test is relying on storage_cost non normalized...
comments welcome....

also to better contextualize what I said in the main comment, this PR adds the normalization just before returning transaction effects in execute_transaction_to_effects. And I am not sure if there are other ways to get effects that may create inconsistencies...

@sblackshear @lxfind talking to Move and wallet/sdk people it seems this feature can get problematic.
Particularly with programmable transactions, but even with regular ones, a user can run out of gas if normalized values are used. Simple example:
programmable transaction
1- execution cost: 100 storage cost: 10 rebate: 20
2- execution cost: 100 storage cost: 10 rebate: 20
normalized value would sum up to 180
non normalized would consider only cost (no rebate) and be 220
if a user were to provide 180 for total transaction cost (which is what the normalized value would give) the second transaction would fail execution and the experience be horrible.
Maybe this can be made a front end issue but at the moment I do not see how to move this forward

so this is likely not happening for now if ever.
I'll leave it open for a bit to let people comment if they have any, then I'll close it

I am going to close this soon, please comment if you think otherwise

it seems storage cost is all computed at the end so the match above is incorrect and we could do this.
Keeping it open then...
I will investigate why the rosetta test is failing and see if there are issues of some kind with the normalized model

too old closed in favor of https://github.com/MystenLabs/sui/pull/8666


Make text in panes (inspector, output, watch) copyable

added native-key-binding class and tabindex:-1 to panes to allow copying via ctrl-c from them
addresses #989 and #845 (related: #954)
inspired by rgbkrk/atom-script/pull/79

@ttor Do you want to help us maintain Hydrogen? We'd be very happy to add you to the nteract organization.

Oh, nice! Thanks @ttor!

@lgeiger I'd love to help out with Hydrogen. Let me first try to get a little more comfortable with the codebase and maybe solve a few issues.

@ttor That's awesome! Welcome to nteract 🎉


[component] description of issue

Detailed description
Describe in detail the issue you're having. Please include whether or not this is part of Sketch Libraries or the larger Sketch Kit file.

Which version of the kit are you using?

Is this a bug or feature enhancement request?

Is this issue related to a specific component? If so please give the page, artboard, and layer/symbol name.

If a bug, what did you expect to happen? What happened instead? What would you like to see changed?

Is this issue being filed based on a discrepancy between the website/code and the kit? If so what is the inconsistency?

Additional information

Screenshots or Sketch file links
Notes

i want carbon design kit Adobe XD file.

how can i download that kit??

@Chaewon0416 You can download the different themed kits here https://github.com/IBM/design-kit/tree/master/Adobe XD. We also have an Icon kit now.


Windows doesn't find TestHarness directory for plugins

Defect Description
Describe the defect
What did you expect to see happen?
Windows to work.
What did you see instead?
https://civet.inl.gov/job/471011/
Do you have a suggested fix for the development team?
Describe how to Reproduce
Steps to reproduce the behavior:
1.
2.
3.
4.
Screenshots and Input Files
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.
Platform (please complete the following information):

OS: [e.g. iOS]
Version: [e.g. 22]
Dependencies Installation: [CONDA or PIP]

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or task?
[x] 2. Is it tagged with a priority: critical, normal or minor?
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements?
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users.
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.)

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest release branch? If yes, is there any issue tagged with release (create if needed)?
[x] 5. If the issue is being closed without a pull request, has an explanation of why it is being closed been provided?

Closure checklist passed.
This issue is a defect but no "wrong results" are possible.
Closure approve via #1160


Know Sure Thing (KST) is different to that of stockcharts.com/school

Thank you for this package.
I found that kst implementation is different to
http://stockcharts.com/school/doku.php?id=chart_school:technical_indicators:know_sure_thing_kst
As for me, I'm adding another function which makes difference between kst and sig.
def kst_sig(close, r1=10, r2=15, r3=20, r4=30, n1=10, n2=10, n3=10, n4=15, nsig=9, fillna=False):
....
return pd.Series(kst-sig, name='kst-sig')

Hi @saroun74 ,
The implementation of ROC (Rate Of Change) is different than your link, but it's right. The result is the same if you use:
close / close.shift(r1) - 1
or
(close - close.shift(r1)) / close.shift(r1)
So, I think we can change the implementation from:
    rocma1 = (close / close.shift(r1) - 1).rolling(n1).mean()
    rocma2 = (close / close.shift(r2) - 1).rolling(n2).mean()
    rocma3 = (close / close.shift(r3) - 1).rolling(n3).mean()
    rocma4 = (close / close.shift(r4) - 1).rolling(n4).mean()

to:
    rocma1 = ((close - close.shift(r1)) / close.shift(r1)).rolling(n1).mean()
    rocma2 = ((close - close.shift(r2)) / close.shift(r2)).rolling(n2).mean()
    rocma3 = ((close - close.shift(r3)) / close.shift(r3)).rolling(n3).mean()
    rocma4 = ((close - close.shift(r4)) / close.shift(r4)).rolling(n4).mean()

Perhaps the code is more clear in this way.
What do you think?

I'm not mentioning the implementation of ROC.
This package returns the 'sig', but the sockcharts.com link guides 'kst - sig' as indicator.

Ok, I understand your suggestion now.
So, I think we could get 3 indicators: KST, KST Signal, and KST - KST Signal.

What do you think?

Library updated to new version 0.1.5.
Now,  we get 3 indicators from KST: KST, KST Signal, and KST - KST Signal.


Add convenience methods for use in Ethereum libraries

Add constructor to initialise a uint256 from a hex string.
Add export_bits to uint256 to export fixed length 32 byte vector.
Add export_bits_truncate to export fitted byte vector.

Thankyou for your review! My C++ is very rusty (circa 2004/5) as you can see. I've made the changes you requested. Mostly, the unusual choices to do things that way were dictated by trying to use as little memory as possible, but in hindsight the compiler will be producing the same code and that style is better.
I did some other tidying up so it looks the same as your code formatting style. Hopefully it looks a bit better now. Please let me know if there's any other changes I can do and thanks again for your review. This library has been a life saver, as Ethereum uses uint256 for a lot of values and has some very large numbers for handling the small 'Wei' unit.

@JamesSmartCell Can I also bother you to make a pull request for the uint128_t changes in the uint128_t repo?

Updated to use reference and deduplicate the two constructor paths.


[mxnet 2.0][item 4.11] Gluon Distributed Training Usability Enhancement

Scope:

more flexibility for communication with kvstore UDFs
add distribution strategies to estimator
plugin for communication backends (horovod, byteps, parameter server) for data parallel training
data sharding/sampling/streaming enhancement for distributed training

RFC for 3: https://github.com/apache/incubator-mxnet/issues/16795
Initial PR for the plugin interface: https://github.com/apache/incubator-mxnet/pull/17010


Add deleted datetime field to models with is_deleted [PLAT-512]

Purpose
Add a deleted field to all models that have is_deleted, change other models that have similar fields with different names. Also add a deleted field to preprints.
Changes

Add deleted fields of type NonNaiveDateTimeField to models with is_deleted Boolean field
Also add deleted field to the PreprintService model
Change naming for models that had fields like deleted_on or deleted_date
Set deleted datetime when object is deleted and unset when undeleted
Migrations to add fields and back populate where possible
If a node was deleted and doesn't have a
Update preprint detail's perform_destroy method to just set the deleted field to the current datetime instead of actually deleting it from the database
Update get_object for PreprintMixin filter for preprints that have null in the deleted field

QA Notes

This change most likely doesn't need a lot of QA, as it's mostly backend related. The new deleted field isn't yet for the most part used for anything at the moment.
Side Effects
Preprints used to be actually deleted when sending a delete request through the API, but no longer are and simply marked deleted - this is the only significant possibly breaking change.
For preprints, the deleted field will largely remain unpopulated - the migration here only adds a deleted datetime from the log when the preprint's node has been marked as deleted. This was done, instead of relying on all the other various ways a preprint could be considered "deleted" (node made private, is an orphan, not verified_publishable, etc) as changes in the upcoming Node Preprint Divorce will hopefully make this a lot clearer and more straightforward.
Ticket
https://openscience.atlassian.net/browse/PLAT-512

Replaced by #9133


Linking issue on FreeBSD

Describe the bug
When compiling from source on FreeBSD 11.2-RELEASE, the linker is called with -llibmaxminddb and it makes it fail. The correct option would be -lmaxminddb.
Logs and dumps
/usr/bin/ld: cannot find -llibmaxminddb

Hi @granalberto
Thanks for the report. I will have a look. Just installed a FreeBSD machine here to run a couple of tests.

Build bots are running on v3/dev/issue2131. A small change but may broken compilation for others, so let's see what the buildbots are going to tell us.

I can't reproduce this issue.
./configure --without-lmdb --enable-parser-generation --enable-mutex-on-pm --prefix=/usr
...
ModSecurity - v3.0.3-95-gd5b93c10 for FreeBSD

 Mandatory dependencies
   + libInjection                                  ....v3.0.3-95-gd5b93c10
   + SecLang tests                                 ....d5b93c10

 Optional dependencies
   + GeoIP/MaxMind                                 ....found
      * (MaxMind) v1.3.2
         -lmaxminddb , -DWITH_MAXMIND -I/usr/local/include
   + LibCURL                                       ....found v7.65.1
      -L/usr/local/lib -lcurl, -I/usr/local/include -DWITH_CURL_SSLVERSION_TLSv1_2 -DWITH_CURL
   + YAJL                                          ....found v2.1.0
      -lyajl , -DWITH_YAJL -I/usr/local/include
   + LMDB                                          ....disabled
   + LibXML2                                       ....found v2.9.9
      -L/usr/local/lib -lxml2 -lz -L/usr/lib -llzma -L/usr/lib -lm, -I/usr/local/include/libxml2 -I/usr/include -DWITH_LIBXML2
   + SSDEEP                                        ....not found
   + LUA                                           ....found
      -llua-5.3 -lm  -L/usr/local/lib , -DWITH_LUA -I/usr/local/include/lua53

 Other Options
   + Test Utilities                                ....enabled
   + SecDebugLog                                   ....enabled
   + afl fuzzer                                    ....disabled
   + library examples                              ....enabled
   + Building parser                               ....enabled
   + Treating pm operations as critical section    ....enabled

additional info's:
## --------- ##
## Platform. ##
## --------- ##

hostname = freebsd
uname -m = amd64
uname -r = 11.2-RELEASE
uname -s = FreeBSD
uname -v = FreeBSD 11.2-RELEASE #0 r335510: Fri Jun 22 04:32:14 UTC 2018     root@releng2.nyi.freebsd.org:/usr/obj/usr/src/sys/GENERIC

/usr/bin/uname -p = amd64
/bin/uname -X     = unknown

Of course, libmaxmindb had been installed:
# pkg info libmaxminddb-1.3.2_1
libmaxminddb-1.3.2_1
Name           : libmaxminddb
Version        : 1.3.2_1
Installed on   : Thu Jul 11 11:37:54 2019 UTC
Origin         : net/libmaxminddb
Architecture   : FreeBSD:11:amd64
Prefix         : /usr/local
Categories     : net
Licenses       : APACHE20
Maintainer<EMAIL_ADDRESS>WWW            : https://github.com/maxmind/libmaxminddb
Comment        : Library for the MaxMind DB file format used for GeoIP2
Shared Libs provided:
	libmaxminddb.so.0
Annotations    :
	FreeBSD_version: 1102000
	repo_type      : binary
	repository     : FreeBSD
Flat size      : 93.6KiB
Description    :
The libmaxminddb library provides a C library for reading MaxMind DB
files, including the GeoIP2 databases from MaxMind. This is a custom
binary format designed to facilitate fast lookups of IP addresses while
allowing for great flexibility in the type of data associated with an
address.

WWW: https://github.com/maxmind/libmaxminddb

And finally, all test passed.

@airween the configure stage passes OK for me too. Then, I run gmake. Default make doesn't work (you need to install gmake, of course). After a few minutes, the stage fails with the reported error.

@granalberto - as I wrote, the build flow and all test (gmake/make test) are successfully passed.
[airween@freebsd ~/src/ModSecurity]$ time gmake
Making all in others
gmake[1]: Entering directory '/usr/home/airween/src/ModSecurity/others'
gmake[1]: Nothing to be done for 'all'.
gmake[1]: Leaving directory '/usr/home/airween/src/ModSecurity/others'
Making all in src
...
gmake[1]: Entering directory '/usr/home/airween/src/ModSecurity'
gmake[1]: Nothing to be done for 'all-am'.
gmake[1]: Leaving directory '/usr/home/airween/src/ModSecurity'

real    0m13.788s
user    0m10.908s
sys     0m1.557s

(Note, this time is not the correct value, the binaries are built now...)
[airween@freebsd ~/src/ModSecurity]$ time gmake check
....
============================================================================
Testsuite summary for modsecurity 3.0
============================================================================
# TOTAL: 4842
# PASS:  4821
# SKIP:  18
# XFAIL: 0
# FAIL:  3
# XPASS: 0
# ERROR: 0
============================================================================
See ./test-suite.log
Please report to<EMAIL_ADDRESS>============================================================================

The failed tests caused by SSL error (Github needs sslv3, and looks like this release doesn't support it) - I didn't investigate it...
So, I think you forgot to install libmaxmindb, or built from source and linker doesn't found....?

Could you share with us your configure summary (as you can see mine above)?

The second attempt to the fix is on the go.

@airween I don't think the configuration summary is going to help here. The configuration script shall not link (or use) the wrong library regardless of how broken (or not) is the environment of the user. The main reason for having a configuration script is to detect and avoid nuances such as this. That said, works for me is not applied in this scenario.
Something that the configure does, for instance, is to compile o piece of code with the desired library being used. If the compilation fails, it marks the library as not supported.
If I didn't have a suspicious I would ask for the conifiguration log files, but I do have; Is on the buildbots -
Likely @granalberto has libmaxmind being detected by the files on the file system, not pkg-config (or alternatives), therefore he has the lib- prefix in the front of the library.
Nevertheless, I would go for LLVM on FreeBSD not gcc.

hi @zimmerle

@airween I don't think the configuration summary is going to help here. The configuration script shall not link (or use) the wrong library

yes, it doesn't help to detect the library,

Nevertheless, I would go for LLVM on FreeBSD not gcc.

but can helps to detect other parts of environment :).

hi @zimmerle

@airween I don't think the configuration summary is going to help here. The configuration script shall not link (or use) the wrong library

yes, it doesn't help to detect the library,

You meant: if it helps to identify if the library exists or not?
If so, let me explain how it works -
The build has a set of scripts at the build directory, here.
In this specific case, the m4 script that looks to the libmaxmind is here -
https://github.com/SpiderLabs/ModSecurity/blob/v3/master/build/libmaxmind.m4
The logging will help us to identify whenever there was a conflict of geoIP vs Maxmind or even garbage left for a not workable Maxmind. The summary on another hand will just present just a summary.

Nevertheless, I would go for LLVM on FreeBSD not gcc.

but can helps to detect other parts of environment :).

Yes, most certainly yes. As far as you go from what is considered standard as like you be to identify not tested scenarios, therefore possible issues.

Just an update: the script is looking for the library in /usr/local and /opt which may or may not be on the library path.

This is the summary of configure script:

ModSecurity -  for FreeBSD

 Mandatory dependencies
   + libInjection                                  ....v3.9.2-30-gbf234eb
   + SecLang tests                                 ....5d85f36

 Optional dependencies
   + GeoIP/MaxMind                                 ....found
      * (MaxMind) v
         /usr/local/lib//libmaxminddb.so, /usr/local/include, -DWITH_MAXMIND -I/usr/local/include
   + LibCURL                                       ....found v7.65.1
      -L/usr/local/lib -lcurl, -I/usr/local/include -DWITH_CURL_SSLVERSION_TLSv1_2 -DWITH_CURL
   + YAJL                                          ....not found
   + LMDB                                          ....not found
   + LibXML2                                       ....found v2.9.9
      -L/usr/local/lib -lxml2 -lz -L/usr/lib -llzma -L/usr/lib -lm, -I/usr/local/include/libxml2 -I/usr/include -DWITH_LIBXML2
   + SSDEEP                                        ....not found
   + LUA                                           ....not found

 Other Options
   + Test Utilities                                ....disabled
   + SecDebugLog                                   ....enabled
   + afl fuzzer                                    ....disabled
   + library examples                              ....enabled
   + Building parser                               ....disabled
   + Treating pm operations as critical section    ....disabled

These are the installed packages:

# pkg info
apache24-2.4.39_1              Version 2.4.x of Apache web server
apr-<IP_ADDRESS>.6.1_1              Apache Portability Library
autoconf-2.69_2                Automatically configure source code on many Un*x platforms
autoconf-wrapper-20131203      Wrapper script for GNU autoconf
automake-1.16.1_1              GNU Standards-compliant Makefile generator
binutils-2.32_1,1              GNU binary tools
ca_root_nss-3.44.1             Root certificate bundle from the Mozilla Project
curl-7.65.1                    Command line tool and library for transferring data with URLs
db5-5.3.28_7                   Oracle Berkeley DB, revision 5.3
expat-2.2.6_1                  XML 1.0 parser written in C
gcc8-8.3.0_2                   GNU Compiler Collection 8
gdbm-1.18.1_1                  GNU database manager
gettext-runtime-0.20.1         GNU gettext runtime libraries and programs
git-lite-2.22.0                Distributed source code management tool (lite package)
gmake-4.2.1_3                  GNU version of 'make' utility
gmp-6.1.2_1                    Free library for arbitrary precision arithmetic
indexinfo-0.3.1                Utility to regenerate the GNU info page index
libmaxminddb-1.3.2_1           Library for the MaxMind DB file format used for GeoIP2
libnghttp2-1.39.1              HTTP/2.0 C Library
libtool-2.4.6_1                Generic shared library support script
libxml2-2.9.9                  XML parser library for GNOME
m4-1.4.18_1,1                  GNU M4
modsecurity3-3.0.3_2           Intrusion detection and prevention engine
modsecurity3-apache-0.0.9.b1.19 Intrusion detection and prevention engine / Apache Wrapper
mpc-1.1.0_2                    Library of complex numbers with arbitrarily high precision
mpfr-4.0.2                     Library for multiple-precision floating-point computations
pcre-8.43_1                    Perl Compatible Regular Expressions library
perl5-5.28.2                   Practical Extraction and Report Language
pkg-1.11.1                     Package manager
readline-8.0.0                 Library for editing command lines as they are typed
sudo-1.8.27_1                  Allow others to run commands as root
yajl-2.1.0                     Portable JSON parsing and serialization library in ANSI C

I need to go through the whole compilation process so eventually I can make the modsecurity3-nginx port and package for FreeBSD and perhaps Pkgsrc.
Let me know if you need something else. I cannot respond fast these days but I'll do my best.

@granalberto it doesn't visible how do you pass the arguments to the configure script.
Here is how I did:
./configure --without-lmdb --enable-parser-generation --enable-mutex-on-pm --prefix=/usr
I didn't pass any maxmindb info. It's interesting that your script detect it as without version, and shows full path instead if linker flag.
Note, that you don't need to pass the --withoud-lmdb if you didn't installed, and I think you also don't need the --enable-parser-generation nor --enable-mutex-on-pm. The prefix is your choice :).
Anyway, as you can see @zimmerle gave a possible solution - you can grab it from here.

Thank you all very much. Branch v3/dev/issue/2131 works when you pass --prefix=/usr/local to ./configure.

I don't understand why libmaxminddb was removed from MAXMIND_POSSIBLE_LIB_NAMES. libmaxminddb.pc is the pkg-config filename used by upstream libmaxminddb: https://github.com/maxmind/libmaxminddb/blob/main/src/libmaxminddb.pc.in. So, as a result, build is broken with libmodsecurity 3.0.5 and any up-to-date buildsystems such as buildroot.

It's because MAXMIND_POSSIBLE_LIB_NAMES is used both for pkg-config and for matching the library name itself.
I think the solution is not to use it for pkg-config, because it was never called anything other than libmaxminddb. So replace
                for x in ${MAXMIND_POSSIBLE_LIB_NAMES}; do
                    if ${PKG_CONFIG} --exists ${x}; then
                        MAXMIND_PKG_NAME="$x"
                        break
                    fi
                done

with
                if ${PKG_CONFIG} --exists libmaxminddb; then
                    MAXMIND_PKG_NAME="libmaxminddb"
                    break
                fi


Edit Cron job - Next run time: Unknown

I have a cron job set to run Monthly (on the 1st) but the Next run time will show Unknown. I don't know if this is a Cron job, Authentic theme or Webmin issue.

Thanks for pointing this out - it will be fixed in the next release.


define KERN_TIMEOUT_STATS and incr KERN_MAXID on OpenBSD

KERN_TIMEOUT_STATS has been defined in sys/sysctl.h (1.194)

r? @gnzlbg
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit 7ddda6fd3bcba4d8eb8f011c609b9d10684aa2c8 has been approved by gnzlbg

:hourglass: Testing commit 7ddda6fd3bcba4d8eb8f011c609b9d10684aa2c8 with merge 7e4ece1a97d6805b663d940f6ed9dca0a3bc73d6...

:sunny: Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, status-azure
Approved by: gnzlbg
Pushing 7e4ece1a97d6805b663d940f6ed9dca0a3bc73d6 to master...


(Angular-Meteor Tutorial) Retrieve the lastest document inserted to MongoDB

In helper, how can I get the lastest document inserted to MongoDB? I have tried:
this.helpers({     newestParty: () => Parties.find({}, {sort: {_id: -1}, limit: 1}) });
but it does not work.

Finally I found that we cannot query for the lastest document based on neither _id or $natural key. It should be:
this.helpers({ newestParty: () => Parties.find({}, {sort: {createdAt: -1}, limit: 1}) });

thanks for updating!

have you try this : Parties.findOne({},{sort:{$natural:-1}});

Sorry. It doesn't work. The error is:
angular.js:12520 Error: unsupported sort key: $natural
It's seems like Meteor MongoDB is quite different to the original one although they said it just a wrap around Mongo cursor. You also cannot chain methods like this:
Parties.find().limit().sort() // You cannot do this too

It works for me on server side. I don't know client side


Add support for CRAN+Bioconductor

https://github.com/ossf/osv-schema/pull/176

[x] Initial import
[x] Ecosystem enumeration support (https://github.com/google/osv.dev/blob/1d395c94739bd50b919b21a1642f8fe4d697b539/osv/ecosystems/_ecosystems.py#L28) to enable OSV.dev API queries.

@tylfin we may need some assistance with the second point above (similar to #1463).

CRAN & Bioconductor is now on production: https://osv.dev/list?ecosystem=CRAN&q=
(API queries work too!)


Feature request: Encode protobuf

Encoding JSON into protobuf without schema/proto definition as it exists for decoding.
Thanks by advance and good job for all has been done.

Added in v9.32.0


@manual tag does not ignore the scenario, it runs it anyway

So I am not sure, but I think when we put @manual tag above some scenario, it should completely ignore the test from execution, right? Apparently serenity launches the scenario anyway and then in the report there are statuses like "Passing (manual)" or "Failing (manual)" depending on the result of automated run, which is very odd to me.

Hi @emosGambler! It's actually Cucumber (rather than Serenity/JS) that executes the scenario, so in order to avoid running the @manual scenarios you'd need to tell Cucumber to ignore them:
// protractor.conf.js
    cucumberOpts: {
        require:    [ 'features/**/*.ts' ],
        format:     'pretty',
        compiler:   'ts:ts-node/register',
        tags:       ['~@manual']           // that's the important bit
    },

Hope that helps!
Jan

In this case, Serenity report won't have the live documentation of them. I would expect to have those scenarios in the report but not executed.

If the scenario is manual, why does it have step definitions? Manual scenarios are not intended to be automated, otherwise they wouldn’t be manual.


why values change dramatically it may measure 90 then 60 then 40 ?

@mohabinventor It'll change drastically depending on the pressure you put onto the lens. It'll also change drastically at the beginning and then get somewhat consistent after a couple of seconds.


refCount with delay

Feature Request
Is your feature request related to a problem? Please describe.
I have a Observable<Array<Observable<T>>> which I want to map to Observable<Array<T>>.
When a new array is emitted, the inner observables should unsubscribe/subscribe as follows:

If Observable exists in previous array and the new/current array, retain pre-existing subscription
If Observable did not exist in previous array but does exist in new/current array, create new subscription
If Observable existed in previous array but does not exist in new/current array, unsubscribe from pre-existing subscription

I hoped to achieve this using switchMap on the outer observable and then passing Array<Observable<T>> into combineLatest. However, switchMap will unsubscribe from its previous inner Observable before subscribing to the new inner Observable, which means inner subscriptions are not retained as desired.
Example:

https://stackblitz.com/edit/typescript-b4wgr1
https://stackoverflow.com/questions/51875557/combinelatest-switchmap-and-retaining-inner-subscriptions/51901191

Describe the solution you'd like
To offset the fact that switchMap unsubscribes to the previous inner observable before subscribing to the new inner observable, I would like the refCounting to be delayed, thereby ensuring that subscriptions to the inner observables are retained.
This is the solution suggested in this article. An alternative refCount with delay operator can be found here.
Describe alternatives you've considered
(If this is new operator request) describe reason it should be core operator
From my research this seems like something people want but it's not easy to do. I discovered issues/articles/examples on the internet, but there doesn't seem to be anything in the core to help with this.
Additional context

Related discussion: https://github.com/ReactiveX/rxjs/issues/171
Issue which needed this: https://stackoverflow.com/questions/51875557/combinelatest-switchmap-and-retaining-inner-subscriptions/51901191#51901191
Example: refCountWithDelay implementation: https://gist.github.com/marinho/3637210b13c0f298e1692a0b7b104e64
Example: delayedRefCount implementation: https://medium.com/@volkeron/rxjs-unsubscribe-delay-218a9ab2672e

Thank you for writing this up. I ran into the same thing. For now, I'm actually memoizing the slow operation (it was an http call) and allowing the stream to reset. It's not perfect, but it seems to work in my case.
The operator (or flag to switchMap) you're proposing here would be great.

@aaronjensen IIRC, I ended up working with @cartant on a combineLatestHigherOrderArray operator, which felt like a better solution at the time: https://github.com/cartant/rxjs-etc/blob/master/source/observable/combineLatestHigherOrderArray-spec.ts. That might help in your case too!

That's helpful, thank you @OliverJAsh. It looks like that package has refCountDelay as well and some other handy operators. I'll try it out.

This has come up a few times, and now that we've revamped the share-type operators in 7.x it might be time to have a look to see if we can accommodate this with that new API somehow.
Off the top of my head, maybe something like (not at all set on this, it requires some thought):
source$.pipe(
  share({
    resetOnRefCountZero: () => timer(1000) // Where it could be a boolean or a reset timer factory?
  });
)

I'll add this to agenda items.


Added an example command for cloning in the docs

Contributor Comments
[INSERT DESCRIPTION]
Pull Request Checklist
Thank you for submitting a contribution to our project!
In order to streamline the review of your contribution we ask that you review and comply with the below requirements:

[ ] Rebase your branch against the latest commit of the target branch.
[ ] If you are adding a dependency, please explain how it was chosen.
[ ] If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
[ ] If there is an issue associated with your Pull Request, link the issue to the PR.
[ ] Validate that documentation is accurate and aligned to any project updates or additions.

Don't forget our more detailed contribution guidelines
here.

@Scriptornaut is this ready for another review?  If so, don't forget to keep it up-to-date, and re-request my review via the button in the top right.

Thanks for the guidance @JonZeolla. Just wrapped up your suggested changes.


RuntimeError: Exception with an error code: 0x9 (SPXERR_UNEXPECTED_EOF) [CALL STACK BEGIN]

Hello Everyone,
I have been trying to transcribe 6000 audio files for a while using Microsoft cognitive speech sdk but I am getting the error listed below.
Note: I passed one audio file into the code and it worked. However, when I passed in the 6000 audio files into the code, it throws the error listed below:
RuntimeError: Exception with an error code: 0x9 (SPXERR_UNEXPECTED_EOF)
[CALL STACK BEGIN]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x16892d) [0x7fa843e6f92d]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x179ffc) [0x7fa843e80ffc]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0xef26c) [0x7fa843df626c]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x15c23f) [0x7fa843e6323f]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x11694b) [0x7fa843e1d94b]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1df622) [0x7fa843ee6622]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1c2879) [0x7fa843ec9879]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1d6ffa) [0x7fa843eddffa]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1c87e2) [0x7fa843ecf7e2]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1e13f0) [0x7fa843ee83f0]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x1e1759) [0x7fa843ee8759]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(+0x21071f) [0x7fa843f1771f]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/libMicrosoft.CognitiveServices.Speech.core.so(recognizer_create_speech_recognizer_from_config+0x109) [0x7fa843f1910b]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/_speech_py_impl.so(+0x1086e0) [0x7fa8447c06e0]
/usr/local/lib/python3.7/dist-packages/azure/cognitiveservices/speech/_speech_py_impl.so(+0x92c69) [0x7fa84474ac69]
/usr/bin/python3(_PyMethodDef_RawFastCallKeywords+0x1d7) [0x5580e885ec47]
/usr/bin/python3(_PyCFunction_FastCallKeywords+0x20) [0x5580e885ea50]
[CALL STACK END]
Kindly, help me to fix this.
Here is the source code:
import time
import wave
import string
def speech_recognize_continuous_from_file():
"""performs continuous speech recognition with input from an audio file"""
#
speech_config = speechsdk.SpeechConfig(subscription='.............', region='.......')
audio_config = speechsdk.audio.AudioConfig(filename='/content/gdrive/MyDrive/Colab Notebooks/Google_STT_Notebook/tem/tbd_convert/recordings/')
speech_recognizer = speechsdk.SpeechRecognizer(speech_config=speech_config, audio_config=audio_config)
done = False

def stop_cb(evt):
    """callback that signals to stop continuous recognition upon receiving an event `evt`"""
    print('CLOSING on {}'.format(evt))
    nonlocal done
    done = True

speech_recognizer.recognizing.connect(lambda evt: print('RECOGNIZING: {}'.format(evt)))
speech_recognizer.recognized.connect(lambda evt: print('RECOGNIZED: {}'.format(evt)))
speech_recognizer.session_started.connect(lambda evt: print('SESSION STARTED: {}'.format(evt)))
speech_recognizer.session_stopped.connect(lambda evt: print('SESSION STOPPED {}'.format(evt)))
speech_recognizer.canceled.connect(lambda evt: print('CANCELED {}'.format(evt)))
# stop continuous recognition on either session stopped or canceled events
speech_recognizer.session_stopped.connect(stop_cb)
speech_recognizer.canceled.connect(stop_cb)

# Start continuous speech recognition
speech_recognizer.start_continuous_recognition()
while not done:
    time.sleep(.5)

@geniusrasaq Thanks for the report, could you please provide Speech SDK traces as instructed here?
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-use-logging
Also looking at your source code, for AudioConfig construction you need to provide filename, not the directory path.
Something like below:
speech_config = speechsdk.SpeechConfig(subscription='****', region='****')

entries = os.listdir('files/')
for audiofilename in entries:
    audio_config = speechsdk.audio.AudioConfig(filename='files/'+audiofilename)
    speech_recognizer = speechsdk.SpeechRecognizer(speech_config=speech_config, audio_config=audio_config)

    done = False

    def stop_cb(evt):
        print('CLOSING on {}'.format(evt))
        global done
        done = True

    speech_recognizer.recognizing.connect(lambda evt: print('RECOGNIZING: {}'.format(evt)))
    speech_recognizer.recognized.connect(lambda evt: print('RECOGNIZED: {}'.format(evt)))
    speech_recognizer.session_started.connect(lambda evt: print('SESSION STARTED: {}'.format(evt)))
    speech_recognizer.session_stopped.connect(lambda evt: print('SESSION STOPPED {}'.format(evt)))
    speech_recognizer.canceled.connect(lambda evt: print('CANCELED {}'.format(evt)))
    # stop continuous recognition on either session stopped or canceled events
    speech_recognizer.session_stopped.connect(stop_cb)
    speech_recognizer.canceled.connect(stop_cb)

    # Start continuous speech recognition
    speech_recognizer.start_continuous_recognition()
    while not done:
        time.sleep(.5)

@jhakulin, thank you for your reply. I tried the code you provided above and it worked. However, the code is splitting each audio files in the directory into subsections. I increased the sleeping time to higher number but it still did not work.

@jhakulin , what can I do to make the code finished transcribing each audio files before moving to the next one.

@geniusrasaq The script I provided earlier will transcribe each audio file at the time, so that first file is transcribed before starting the second one and so on. Could you please provide more information what do you mean by "code is splitting each audio files in the directory into subsections" ? Also could you attach your audio files to this case so that I could reproduce the possible issue you are seeing?

@jhakulin, thank you for your prompt response. Attached is the copy of some of the audios that I am transcribing and the output that I got from Microsoft speech sdk and how I want to be.
Sample_record (3).zip

@geniusrasaq If you need to transcribe a large set of WAV files, we recommend using batch transcription. You can find Python sample code here. Speech SDK is best for streaming live audio to the cloud or for processing a small number of short WAV files.
Regarding your attachment Sample_record (3).zip. Recognition results will always be given one "sentence" at a time. There is no way to configure the Speech Service to produce one recognition result for the whole file using Speech SDK. It's up to you (the app developer) to concatenate the recognition results into one string if that is what you need.
If you need help with batch transcription, please don't hesitate to open a new GitHub issue here.
Thanks,
Darren


fix: resolve the right config file and args from cli

Description
This PR contains two fixes.
The first is that the document here says that the config name is bumpp.config.ts, but the name is actually bump.config.ts. Maybe it's better to support both of the names? This will fix #6.
And because of adding the --no-commit and --no-tag options in v9.2.0, this will set the commit and tag from cli args to true instead of undefined according to https://github.com/cacjs/cac#negated-options. And this will overrides the config from config file. I made some fixes so it can now read the right args from cli. This will fix #13 and #19.
Linked Issues
#6 #13 #19
Additional context

LGTM


Update README for the 0.12 release

It should include that only Angular 1.4.0+ is supported since you are using angular.merge now which doesn't exist in Angular 1.3.x and before.

@mprentic-harris Good point, I will need to update bower.json dependencies version as well.

hmmm, how many companies out there are on Angular 1.4.0+? I know most are still on 1.2 and 1.3, can we make it backward compatible with 1.2, at least 1.3?

The angular.merge is a breaking change, indeed. I am using Angular 1.3.15 because it's still more recommended than 1.4.x in conjunction with angular-material.

I understand that since some people are stuck with 1.3.x, moving forward with 1.4.x just for the angular.merge doesn't make much sense. I am going to add a polyfill for the time being. That function won't incur any performance penalties, as it won't be executed on AngularJS 1.4.x. It will increase the minified file size by 463 bytes, which I think is an acceptable trade off.
Meanwhile, if you have a suggestion on how to avoid using angular.merge to achieve the same result I'd like to hear your opinion. If I don't find a better solution, I'll push out a patch (0.12.2) with a polyfill later today.
Need to do a deep merge of these two objects. The problem with just using angular.extend occurs with responseParams (nested object) being overwritten, instead of extended.
var defaults = {
  defaultUrlParams: ['response_type', 'client_id', 'redirect_uri'],
  responseType: 'code',
  responseParams: {
    code: 'code',
    clientId: 'clientId',
    redirectUri: 'redirectUri'
  }
};

and
facebook: {
  name: 'facebook',
    url: '/auth/facebook',
    authorizationEndpoint: 'https://www.facebook.com/v2.3/dialog/oauth',
    redirectUri: (window.location.origin || window.location.protocol + '//' + window.location.host) + '/',
    scope: ['email'],
    scopeDelimiter: ',',
    requiredUrlParams: ['display', 'scope'],
    display: 'popup',
    type: '2.0',
    popupOptions: { width: 580, height: 400 },
  responseParams: {
    clientId: 'client_id',
      redirectUri: 'redirect_uri'
  }
}

Satellizer 0.12.2 is now available and it no longer uses angular.merge.

Another item for the read me would be the fact that redirects no longer happen automatically.
I'm looking at the commits, and it seems there's quite a bit changing. Anything else that might be worth putting in the documentation?

We upgraded to 0.12.2 and realized redirect were removed, will be great to update readme to reflect these changes (very much like https://github.com/sahat/satellizer/releases/tag/0.12.0)

@jayzeng Will do that tonight.

The entire README has been updated top to bottom, so be sure to check it out!

Awesome job on the polyfill and the README looks great!


Parse vobjects with parse tree grammar instead of RegExp

A regular expression parser is hard to maintain. Using some parse tree grammar can help. Some grammar parser library available in Ruby, such as treetop (https://rubygems.org/gems/treetop/versions/1.6.8).

Rsec. Done. Treetop arguably easier to maintain because it is so much better known;  but I couldn't resist Rsec being 6 times faster. :-)


Using ASP.NET Core 2.2 Identity LockoutEnabled always set to true using UserManager CreateAsync

Hello I am having a problem with the ASP.NET Core 2.2 Identity

I am creating a new user using the IdentityUser. I set the new user class LockoutEnabled = false;
But the UserManager CreateAsync method always set it to true and it overrides the value
I set for the LockoutEnabled property. Is this normal ?

Hello I think I found the code that have the issue here it is set to a static true value
https://github.com/aspnet/AspNetCore/blob/9e788650b091875e4bf15eb7ae7f05283845c5b0/src/Identity/Extensions.Core/src/UserManager.cs#L477
@HaoK Hi bro could you please help me thanks.

Did you try turning off IdentityOptions.Lockout.AllowedForNewUsers = false?  That is what determines whether identity turns it on within a Create
https://github.com/aspnet/AspNetCore/blob/master/src/Identity/Extensions.Core/src/LockoutOptions.cs#L19


'str' object has no attribute 'dim'

I got ' 'str' object has no attribute 'dim'' by simply running the notebook. Any idea where is the error?
Epoch 1 / 10
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-31-c5138ddf6b25> in <module>
     12
     13     #train model
---> 14     train_loss, _ = train()
     15
     16     #evaluate model

<ipython-input-21-a8875e82e2a3> in train()
     25
     26     # get model predictions for the current batch
---> 27     preds = model(sent_id, mask)
     28
     29     # compute the loss between actual and predicted values

~/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py in _call_impl(self, *input, **kwargs)
    725             result = self._slow_forward(*input, **kwargs)
    726         else:
--> 727             result = self.forward(*input, **kwargs)
    728         for hook in itertools.chain(
    729                 _global_forward_hooks.values(),

<ipython-input-16-9ebdcf410f97> in forward(self, sent_id, mask)
     28       _, cls_hs = self.bert(sent_id, attention_mask=mask)
     29
---> 30       x = self.fc1(cls_hs)
     31
     32       x = self.relu(x)

~/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py in _call_impl(self, *input, **kwargs)
    725             result = self._slow_forward(*input, **kwargs)
    726         else:
--> 727             result = self.forward(*input, **kwargs)
    728         for hook in itertools.chain(
    729                 _global_forward_hooks.values(),

~/anaconda3/lib/python3.8/site-packages/torch/nn/modules/linear.py in forward(self, input)
     91
     92     def forward(self, input: Tensor) -> Tensor:
---> 93         return F.linear(input, self.weight, self.bias)
     94
     95     def extra_repr(self) -> str:

~/anaconda3/lib/python3.8/site-packages/torch/nn/functional.py in linear(input, weight, bias)
   1686         if any([type(t) is not Tensor for t in tens_ops]) and has_torch_function(tens_ops):
   1687             return handle_torch_function(linear, tens_ops, input, weight, bias=bias)
-> 1688     if input.dim() == 2 and bias is not None:
   1689         # fused op is marginally faster
   1690         ret = torch.addmm(bias, input, weight.t())

AttributeError: 'str' object has no attribute 'dim'

Figured it out by looking at this. It seems either specifying 'return_dict=False' or calling 'values()' method could force the model to return a tuple.

It's working with the older version.
Try with transformers==3.02 version.

It's working with the older version.
Try with transformers==3.02 version.

Yeah, you are right, thanks!

Yeah, you are right, thanks!

@JoKerDii  Did you specify return_dict=False where bert model was downloaded?
I mean
bert = AutoModel.from_pretrained('bert-base-uncased', return_dict=False)

From this stackoverflow question,
After some 3.X version syntax there has got little changed.
Old code
#pass the inputs to the model
_, cls_hs = self.bert(sent_id, attention_mask=mask)
x = self.fc1(cls_hs)
New code
#pass the inputs to the model
cls_hs = self.bert(sent_id, attention_mask=mask)
x = self.fc1(cls_hs[1])


PoolingProcess._format_picklist() produces incorrect output if populating >15 destination wells (on a 384-well plate)

For the test data for PoolingProcess(3) in the test db, the volume in every well of the 384-well input plate is 1 nL except for wells 0 23 and 24 and P 23 and 24, which are empty.  In this, case, when entering a max_vol_per_well of 1 nL, one would expect each input well to go into a separate output well (except for the 4 noted above), so there should be 380 output wells (A1-P20).  See attached expected output:
expected_poollist.txt
However, the code in fact produces output that works as expected up to A15 and then sends output to B16-B23, then B0[sic]-B7, then C8-C23, and so on in unexpected ways until X20 (spoiler: not a real 384-well-plate well :) See attached buggy output:
buggy_poollist.txt

I have hunted down the cause of this behavior, which is due to the interaction of three separate bugs in the below section of the method:
https://github.com/jdereus/labman/blob/46241e448edd4fc615aeea914e8dd80bf88aad79/labcontrol/db/process.py#L2534-L2549
Issue 1:
https://github.com/jdereus/labman/blob/46241e448edd4fc615aeea914e8dd80bf88aad79/labcontrol/db/process.py#L2548
This line is intended to find what integer to add to the ascii code for the letter "A" to find the row letter of the destination well (e.g., ascii for A is 097, so if this line returns the integer 1, we add 1 to 097 and get 098, which is the ascii code for B, etc)--let's call this the row increment.  d is the number of the current destination well.  We need to increase the row increment every time d indicates we have filled another row--that is, every time d goes up by the number of COLUMNS contained in a row.  Unfortunately, the number of COLUMNS on the destination plate is dest_plate_shape[1], and this code is erroneously using the number of ROWS on the plate (dest_plate_shape[0]).  That is why the buggy output mistakenly "turns over" to a new row on the 16th destination well (because the number of rows on a 384-well plate is 16).
Issue 2:
Even after correcting issue 1 above, the code doesn't work QUITE right: it turns over to a new row on the 24th destination well, whereas in fact it shouldn't turn over to a new row until the 25th (e.g., the 24th well should be A24, not B1).  This is most easily solved by treating d as the destination well index (0-based), starting at d = 0 rather than d = 1.  With this change, the 24th destination well has d = 23 and thus has a row increment of np.floor(23/24), or 0, so it remains part of row A.
Issue 3.
https://github.com/jdereus/labman/blob/46241e448edd4fc615aeea914e8dd80bf88aad79/labcontrol/db/process.py#L2549
This line is intended to find the column number of the destination well--as in., the "1" in "A1".  However,  this modulo calculation comes out to zero (not a valid column number) every time d is an exact multiple of the number of columns on the plate (e.g., 24, 48, etc).  The fix for this, in conjunction with the fix above to issue 3 that starts d at zero, is to simply add 1 to the modulo results.
Thus, the corrected section of the method code reads as below, with the modified values shown surrounded by double asterisks:
        d = **0**
        for i in range(rows):
            for j in range(cols):
                well_name = "%s%d" % (chr(ord('A') + i), j + 1)
                # Machine will round, so just give it enough info to do the
                # correct rounding.
                val = "%.2f" % pool_vols[i][j]
                # test to see if we will exceed total vol per well
                if running_tot + pool_vols[i][j] > max_vol_per_well:
                    d += 1
                    running_tot = pool_vols[i][j]
                else:
                    running_tot += pool_vols[i][j]
                dest = "%s%d" % (chr(ord('A') +
                                 int(np.floor(d / dest_plate_shape[**1**]))),
                                 (d % dest_plate_shape[1]) **+1**)

Please note: it looks like the LabControl method being discussed here may come verbatim from https://github.com/tanaes/metagenomics_pooling_notebook/blob/cf93b57d70f3a36f1321bd2b7a78275e358f7178/metapool/metapool.py#L451-L496 , and thus the fixes described above may need to be propagated to that code if it is still in use.

Also note that even with the corrections above, this code will output garbage in either of the two edge cases below:

you don't have enough wells to pool all the input volumes, given the max vol per well
an individual input well has > the max vol per well

@AmandaBirmingham I think I can close this one as well?

Not yet (but hopefully soon!) Just put in the PR for this one: https://github.com/jdereus/labman/pull/515


Add info to pair device

Description

Add guiding message pointing the user to either correct android documentation on how to actually connect to a device

Related issues

https://github.com/flexn-io/renative/issues/1067

Npm releases
n/a

? Please go to Settings, enable debugging and enter the ip/port required for pairing:
I think, there should be "enter IP address and Port" or something like that, because if I entering only IP I'm getting this error ->
? Please go to Settings, enable debugging and enter the ip/port required for pairing: <IP_ADDRESS>
Enter pairing code: 369484
error: protocol fault (couldn't read status message): Undefined error: 0
error: ⨯ ○ run: COMMAND:

/Users/gabriele/Library/Android/sdk/platform-tools/adb pair <IP_ADDRESS>

FAILED with ERROR:

Error: Command failed with exit code 1: /Users/gabriele/Library/Android/sdk/platform-tools/adb pair <IP_ADDRESS>
    at makeError (/Users/gabriele/Desktop/tests/renative/node_modules/execa/lib/error.js:60:11)
    at handlePromise (/Users/gabriele/Desktop/tests/renative/node_modules/execa/index.js:118:26)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

┌─────────────────────────────────────────────────────────────────┐
│ ⨯ SUMMARY | 4/30/2024, 2:15:34 PM |<EMAIL_ADDRESS>           │
│ $ npx rnv run -p android -d -t <IP_ADDRESS>:38321              │
├─────────────────────────────────────────────────────────────────┤
│ Project<EMAIL_ADDRESS>                         │
│ Workspace: rnv                                                  │
│ Platform (-p): android                                          │
│ Engine: engine-rn                                               │
│ Template: @rnv/template-starter                                 │
│ App Config (-c): harness                                        │
│ Build Scheme (-s): debug                                        │
│ Target (-t): <IP_ADDRESS>:38321                                │
│ Supported Platforms: ios, android, androidtv, firetv, androidwear, web, tizen, tvos, webos, macos, tizenwatch, kaios, chromecast, linux
│ Env Info: darwin | arm64 | node v18.19.0                        │
│ Executed Time: 0h:0m:38s:448ms                                  │
│ warn: ○ run: You'll need to pair your device before installing app.
For more information: https://developer.android.com/studio/run/device
│ error: ⨯ ○ run: COMMAND:

/Users/gabriele/Library/Android/sdk/platform-tools/adb pair <IP_ADDRESS>

FAILED with ERROR:

Error: Command failed with exit code 1: /Users/gabriele/Library/Android/sdk/platform-tools/adb pair <IP_ADDRESS>
    at makeError (/Users/gabriele/Desktop/tests/renative/node_modules/execa/lib/error.js:60:11)
    at handlePromise (/Users/gabriele/Desktop/tests/renative/node_modules/execa/index.js:118:26)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)

│ Project location: ./platformBuilds/harness_android              │
└─────────────────────────────────────────────────────────────────┘
➜  app-harness git:(feat/info_to_pair_device)


Windows 1809 alpha support

OS:

[x] Support 1809

Bugs:

[x]  nginx-proxy container sometimes fails to start: https://github.com/rancher/rancher/issues/16074
[x] kubelet (Kubernetes 1.12.) wont start on windows: https://github.com/rancher/rancher/issues/17341
[x] Node scheduling problem Linux/Windows for Rancher System components: https://github.com/rancher/rancher/issues/17423
[x] Support upgrade: https://github.com/rancher/rancher/issues/19048
[x] Windows Nodes crashing after 2-3 days: https://github.com/rancher/rancher/issues/17499

Enhancement:

[x] Refactor Windows logic
[x] Flannel VxLan mode support
[x] Windows CI support

[x] Same as Linux logic
[x] Package manifest support, for example, rancher/rancher:v0.0.0

rancher/rancher:v0.0.0-linux-amd64
rancher/rancher:v0.0.0-linux-arm64
rancher/rancher:v0.0.0-windows-1809

[x] Windows command manifest support

Related PR:

https://github.com/rancher/types/pull/750

[x] Support Kubernetes 1.14.1

https://github.com/rancher/rke/pull/1290

[x] Support RKE VxLan mode

https://github.com/rancher/hyperkube/pull/39

[x] Support Drone to package rancher/hyperkube Windows image.

https://github.com/rancher/kubernetes-windows-dockerfiles/pull/11

[x] Support Drone to package rancher/kubelet-pause Windows image.

https://github.com/rancher/flannel-cni/pull/3

[x] Support Drone to package rancher/flannel-cni Windows image.

https://github.com/rancher/dapper/pull/70

[x] Support Windows package via dapper
[ ] Need to change maiwj/dapper:windows-1809 to rancher repo in .drone.yml

Issues:

https://github.com/rancher/rancher/issues/16460 (Track)
https://github.com/rancher/rancher/issues/19173

LGTM

Other than the integration test issue, just a couple minor nitpicks, so, basically, its LGTM from me except for the minor nits, which could be addressed in a follow up PR if need be

LGTM

Link to https://github.com/rancher/rancher/issues/17102


Add README contents to show usage and capture an editing scenario

I needed to work around a bug in the Arris web interface related to configuring two firewall access control time ranges per day, and I discovered this XMLC utility. I successfully used the Go implementation to help me through my configuration editing scenario. Thank you!!
While working through this, I authored some README contents for your consideration.

I saw reference to an /etc/defaults.cfg file, but I don't know how to fetch it from the router. If you know how to, we could add that into the README as well.

This was re-eng in part to help facilitate persistent root access on the AT&T Humax BG320-500 and Nokia BGW320-505 and I'm glad to see it has further reach.
I don't have physical access to any Arris hardware so I'm inclined to assume that the details of this pull are factual as the firmware(s) don't include config data. I hope you don't mind that I asked some AT&T land cohorts to read over it.
I'll merge soon afterwards. Also, thank you for taking your time to write this up, I very much appreciate it 👍🏻

Very cool; thank you, @up-n-atom! I appreciate you creating this tool and I also think it's great it had farther reach. It helped me set up household internet access control to meet our family's needs when the router web interface otherwise wouldn't have.


Eco-CI Collapse function is broken

Old broken PR: https://github.com/green-coding-berlin/green-metrics-tool/pull/527

verified. Looks good.
Actual PR to fix this was https://github.com/green-coding-berlin/eco-ci-energy-estimation/pull/53#pullrequestreview-1759494618


Build with tiny-go

Test the resulting size of the .wasm file when build with tiny-go.

Currently the build failure:
% docker run -w /src -v $PWD:/src -v $GOPATH:/go -e "GOPATH=/go" tinygo/tinygo:0.26.0 tinygo build -o /src/wasm.wasm -target wasm --no-debug /src/pkg/wasm

# github.com/elastic/elastic-agent-libs/file
../go/pkg/mod/github.com/andrewkroh/elastic-agent-libs@v0.0.0-20220713130552-0fef594e5ae3/file/fileinfo_unix.go:39:35: Stat_t not declared by package syscall
../go/pkg/mod/github.com/andrewkroh/elastic-agent-libs@v0.0.0-20220713130552-0fef594e5ae3/file/helper_other.go:31:13: Rename not declared by package os

../go/pkg/mod/github.com/andrewkroh/beats/v7@v7.0.0-20221123043252-d8687c3ddb79/libbeat/common/net.go:40:8: impossible type switch case: *net.IPAddr
        addr (variable of type net.Addr) cannot have dynamic type *net.IPAddr (missing method Network)

https://github.com/tinygo-org/tinygo/blob/b5ad81c884981f42c1ac491cc1d195aea886ddbe/src/net/iprawsock.go
https://github.com/golang/go/blob/ee56b3c51016f1db83150d08db4fdb9921230c71/src/net/iprawsock.go
stdlib has the method, but tiny-go does not. I'm not sure what version of Go (e.g. 1.19?) that tiny-go claims to support.

It does not seem possible at the moment.


define sethostname under OpenBSD and Bitrig

factorize the definition with NetBSD

@bors: r+
Thanks!

:pushpin: Commit 7adbc75 has been approved by alexcrichton

:hourglass: Testing commit 7adbc75 with merge e183f93...

:sunny: Test successful - status-appveyor, travis
Approved by: alexcrichton
Pushing e183f93b9dcb692c1fd00c87d68a67722d1994e2 to master...


ValueError on unexpected hyprctl version

I run nixos and the output of my hyprctl -j version is, for some reason:
{
    "branch": "",
    "commit": "",
    "dirty": true,
    "commit_message": "",
    "commit_date": "",
    "tag": "",
    "commits": "",
    "flags": []
}

Which causes the following error when launching wlrui:
Traceback (most recent call last):
  File "/nix/store/azi273kz8q49caqj8nwjpjbnnfrwd1sa-wlr-layout-ui-1.6.11/bin/.wlrlui-wrapped", line 9, in <module>
    sys.exit(main())
             ^^^^^^
  File "/nix/store/azi273kz8q49caqj8nwjpjbnnfrwd1sa-wlr-layout-ui-1.6.11/lib/python3.11/site-packages/wlr_layout_ui/__init__.py", line 86, in main
    load()
  File "/nix/store/azi273kz8q49caqj8nwjpjbnnfrwd1sa-wlr-layout-ui-1.6.11/lib/python3.11/site-packages/wlr_layout_ui/screens.py", line 85, in load
    major = int(version[0])
            ^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: ''

Would it be possible to also assume a new_hyprland when running into ValueError while parsing (i.e. by adding in the except below) ?
https://github.com/fdev31/wlr-layout-ui/blob/1a43a87fec97d3ae21628973a292f5ac82e27817/src/wlr_layout_ui/screens.py#L56-L63

Hi, I'm maintaining the nix package of wlr-layout-ui.
There was a bug in the older hyprland nix package that lead to hyprctl not providing the version information. (see this issue)
This is fixed as of hyprland version 0.41.1 (in nix channel 24.11 and unstable) and I am currently doing an upgrade of the wlr-layout-ui package to support this fixes.
It should only take a few days, I guess.

Thank you very much ! I'll close the issue when the upgrade goes through.

It's solved now, thank you!


Is it required to validate the whole chain of schemas?

If the leaf JSON Schema document includes the full schema, it might be easier to just validate against the leaf and not against all schemas in the chain?

Schema validator always validates the full schema.
But note that SD-JWT renders the JSON schema validation useless as it breaks the (original) structure.


Fixes a  Typo in Template

Thanks for this fix.

Thank you! This project has really helped out with our angular documentation needs.

My pleasure, I am glad that I could be a help. Could give this project a star if you like it, so that other people can use it too.


Add sign up dialog for beta tester list

Code-wise LGTM
I think the content of the dialog looks somehow a bit off somehow 🤔
Maybe having the hammer on the left is better and the input field the width of the text and image? What do you think @microbit-matt-hillsdon ?

Version with the hammer on the left is definitely an improvement.

Seems hard to make it look good with an icon like this.
So I think we should consider removing it or a more substantial change.