Any way to rotate connection?

I'm trying to configure the run file for connect and disconnect periodically, but doesn't works..
Like -> while true do  protonvpn refresh; protonvpn $PVPN_CMD_ARGS; sleep 100; protonvpn disconnect; done </dev/null &>/dev/null &

Other than a missing semi-colon after true, and dropping the redirection and backgrounding, this works for me in a container.
I'm going to close this issue, though, since it's not really within the scope of this project.

Other than a missing semi-colon after true, and dropping the redirection and backgrounding, this works for me in a container.
I'm going to close this issue, though, since it's not really within the scope of this project.
Thanks for your answering!
Any code example?  I can't understand an for me is not working

From a shell in the container, this worked: while true; do protonvpn refresh; protonvpn $PVPN_CMD_ARGS; sleep 10; echo "DISCONNECTING!"; protonvpn disconnect; done

It not works for me..
#!/bin/sh

PVDIR=/root/.pvpn-cli

if [ -z "$PVPN_USERNAME" ] || [ -z "$PVPN_PASSWORD" ]; then
	echo "PVPN_USERNAME and/or PVPN_PASSWORD not set!"
	exit 1
fi

# Initialize config
if [ ! -f $PVDIR/pvpn-cli.cfg ]; then
	cp $PVDIR/pvpn-cli.cfg.clean $PVDIR/pvpn-cli.cfg
	sed -i \
		-e "s/PVPN_USERNAME/$PVPN_USERNAME/" \
		-e "s/PVPN_PROTOCOL/$PVPN_PROTOCOL/" \
		-e "s/PVPN_TIER/$PVPN_TIER/" \
		$PVDIR/pvpn-cli.cfg
fi

echo "$PVPN_USERNAME" > $PVDIR/pvpnpass
echo "$PVPN_PASSWORD" >> $PVDIR/pvpnpass
chmod 0600 $PVDIR/pvpnpass

# Connect to ProtonVPN
while true; do protonvpn refresh; protonvpn $PVPN_CMD_ARGS; sleep 10; echo "DISCONNECTING!"; protonvpn disconnect; done </dev/null &>/dev/null &

if ! ip link show proton0 > /dev/null; then
	echo "Failed to bring up VPN :("
	exit 1
fi

if [ -n "$DNS_SERVERS_OVERRIDE" ]; then
	# This needs to run at this point, because ProtonVPN will have changed the
	# DNS servers in /etc/resolv.conf.
	cp /etc/resolv.conf /etc/resolv.conf.bak
	echo "$DNS_SERVERS_OVERRIDE" | sed -e 's/^/nameserver /' -e 's/,/\nnameserver /' > /etc/resolv.conf
fi

# Setup route for host network
if [ -n "$HOST_NETWORK" ]; then
	gw=$(ip route | awk '$1 == "default" { print $3 }')
	ip route add "$HOST_NETWORK" via "$gw"
fi

# Start Privoxy
exec privoxy --no-daemon


NTR: SLAC complex

Hello,
I've a ntr for the slac complex, can anyone please help me with that?
thanks a lot
Livia
Term1:

Name: SLAC complex
Def: A protein complex that regulates Arp2/3 complex-mediated actin nucleation.
References:
GOC:lnp
PMID:
22973053
Relationships:
is_a GO:0032991 protein-containing complex
capable_of_part_of GO:0034315 regulation of Arp2/3 complex-mediated actin nucleation
DB xref:
IntAct: EBI-7506670

Hi @liviap - are there defining members of the complex that we could cite in the definition?  Thx.

in S. cerevisiae the components are LAS17 and SLA1 (ref: 22973053)

Sorry to bother...Any update on it?
Livia


No validation error is shown when not entering other researchers

Path: /proposals/other_researchers/
Occurred during practice application.
When selecting "yes" but not entering other researchers, no validation error is shown but the user cannot continue.
The same happens when I do add another researcher, but when I also remove myself.

Probably related to this, when first selecting "yes" and then "no" to the first question, it seems that the (now invisible) researcher list is still evaluated, so the same problem as above occurs.

Can be grouped together with #728


feat: Add support for warm_pool

Description
Adds support for aws_autoscaling_group's warm_pool configuration.
Motivation and Context
This isn't a hill I need to die on. If this change is risky for some reason that's non-obvious to me, we can forego a merge of this.
Breaking Changes
This is a relatively new feature to the underlying aws provider: https://github.com/hashicorp/terraform-provider-aws/pull/18734
How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

See supplied example.

hey thanks for the PR @curiositycasualty ! I think this looks great, I'll find some time to test and validate. one thing we'll need to update is the min AWS provider version to 3.37.0 per https://github.com/hashicorp/terraform-provider-aws/pull/18734#issuecomment-821296130

I just found this feature, and it might be just what I need. Any chance of this getting updated per review and added in?

+1 for seeing this revived, definitely would like to use this feature!

@bryantbiggs Have you had a change to poke this? I just rebased and got things "semantic".

@curiositycasualty What @bryantbiggs was saying is to update the aws minimum version requirement in versions.tf
      version = ">= 3.37"

Thanks @denraf. Sorry for the delay.

@bryantbiggs This is ready for review.

apologies, looks good if you can fix the conflicts and rebase master - 👍🏽 @antonbabenko

Thanks @curiositycasualty for the PR!
v4.6.0 has been just released.


[Impeller] GLES discard logic doesn't consider that stencil/depth might be same attachment.

If we need to store the result of the stencil buffer (though we should fix this) then we might just discard it today.

macrobenchmarks -> backdropfilter

If we fix https://github.com/flutter/flutter/issues/137302 , then we don't need to worry about immediately since we'll never store the stencil in Entities.


python312Packages.uamqp: 1.6.9 -> 1.6.11

Diff: https://github.com/Azure/azure-uamqp-python/compare/refs/tags/v1.6.9...v1.6.11

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 353152 at 0d0d64b1 run on aarch64-darwin 1

:white_check_mark: 2 packages built:

python311Packages.taxi
taxi-cli (python312Packages.taxi)

Can you try pulling in https://github.com/Azure/azure-uamqp-python/pull/411 and rebuilding? Currently, it fails for me on aarch64-darwin due to https://github.com/Azure/azure-uamqp-python/issues/386.

Can you try pulling in https://github.com/Azure/azure-uamqp-python/pull/411 and rebuilding? Currently, it fails for me on aarch64-darwin due to https://github.com/Azure/azure-uamqp-python/issues/386.

I accidentally deleted the patch that was there. Thanks for submitting the PR.

Result of nixpkgs-review pr 353152 at b8b2bef7 run on aarch64-darwin 1

:x: 2 packages failed to build:

python311Packages.uamqp
python312Packages.uamqp

Result of nixpkgs-review pr 353152 at b8b2bef7 run on x86_64-darwin 1

:x: 2 packages failed to build:

python311Packages.uamqp
python312Packages.uamqp


[BUG] Datepicker - manually change day in input

Bug, feature request, or proposal:
The input and the datepicker communicate incorrectly. If I select a date from the picker, it will be displayed DD/MM/YYYY.
Following this logic, if I try to manually modify the DD, when I open the datepicker, and if I set a value from 1-12, the corresponding month will be selected in the picker.
What is the expected behavior?
If the output date format of the datepicker is of format DD/MM/YYY, when I manually modify the day (DD) that day should be selected in the picker.
What is the current behavior?
If I modify the day, it modifies the month in the background.
What are the steps to reproduce?
Go to: https://material.angular.io/components/datepicker/overview
Select any date. current date for example. (03/08/20017)
Open the datepicker again. Now, the datepicker will display March 2017.
What is the use-case or motivation for changing an existing behavior?

The date format is MM/DD/YYYY in the material.angular.io examples. So 03/08/2017 means March 08, 2017, not August 03, 2017.

 Did you at least checked what I said?
Select from datepicker 10 august 2017. It will display 10/08/2017 (DD/MM/YYYY).
Open the datepicker again, it will display october as selected month.
I'll reopen this thread until someone actually tries to replicate the bug and fixes it.


Reorganizing gp bases and priors.

This adds a few of the functions from enterprise_extensions into the base package. It also reorganizes the Gaussian process functions out of utils.py and into their own sub modules. Backwards compatibility is retained by calling the functions explicitly in the namespace of utils, as suggested by @vallis.

So in order for the lint test not to fail I had to import * within utils.py which I don't like to do. I tried importing the various function, but it didn't like that I had imported unused functions, and I couldn't fit the ignore error commands in there without going over the line limit :-) I'll work up some more tests soon...

Thanks Jeff. Looks good, as soon as you can get the coverage up we'll merge. I don't mind the import star, although it means that all the bases, and not just the legacy ones, would be available from utils.
But re the import statement vs linting, could you do
from module import (function, another_function,
                    another_function)

or even
from module import function, another_function, \
                   another_function

and give the linting direction on the second line? Maybe that's what you tried already.

So in order for the lint test not to fail I had to import * within utils.py which I don't like to do. I tried importing the various function, but it didn't like that I had imported unused functions, and I couldn't fit the ignore error commands in there without going over the lint limit :-) I'll work up some more tests soon...

You should be able to do something like
from module import (function, another_function, # noqa: F401
                    another_function)

where F401 is just a place holder for whatever flake8 error you are getting. This will ignore those kinds of errors in that import statement

Thanks @jellis18 . I actually has to do
from module import (# noqa: F401
                                         function,
                                         another_function,
                                         another_function)

So that works.


Add environment variables to identify the binary and crate name

Closes #8251
This adds CARGO_BIN_NAME and CARGO_CRATE_NAME to rustc/rustdoc process env.
CARGO_BIN_NAME is added for binary compilation units, CARGO_CRATE_NAME is added for binary and library units.
The build::crate_env_vars test was updated to test for this. The test is currently only checking behavior for the binary compile unit.
Documentation was updated to reflect the added environment variables.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @ehuss (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Thanks! This looks good to me.  Just want to get final OK from the team:
@rfcbot fcp merge

Looks great, thank you!

One possible alternative to BIN_NAME might be BIN_STEM since "filestem" is typically already known to not include file extensions.

One possible alternative to BIN_NAME might be BIN_STEM since "filestem" is typically already known to not include file extensions.

Happy to amend the pr if we have a consensus on the name to go forward with. I personally find filestem a rather unknown word that has a lot in common with filesystem, so while technically being more precise it might be more confusing for the average user, so I would prefer clarifying in the docs and leave it as bin.

I don't have a strong opinion one way or the other.  Unless @alexcrichton wants it to change, I would just leave it.

Nah yeah that's a good point, I think just clarifying that the name doesn't include the extension should do the trick!

Agreed. Please update that one line in the documentation; I supplied some text that should work.

:+1:

@bors: r=ehuss

:pushpin: Commit 7ad7427ced99dadd76295963615dc9d903462802 has been approved by ehuss

:hourglass: Testing commit 7ad7427ced99dadd76295963615dc9d903462802 with merge 6f9d8083236d8b1ae8455a04782919326b06951c...

:sunny: Test successful - checks-azure
Approved by: ehuss
Pushing 6f9d8083236d8b1ae8455a04782919326b06951c to master...

I've tried last nighly and it looks like env!("CARGO_CRATE_NAME") is always set to "build_script_build", env!("CARGO_BIN_NAME") does not compile, and env::var("CARGO_CRATE_NAME") as well as `env::var("CARGO_BIN_NAME") are not set at all.

@A1-Triard CARGO_BIN_NAME is only set while building the binary.

@ehuss, I am building a binary. Two binaries in fact. They are not specified in Cargo.toml explicitly, but they lie in right place (src/bin/bin1.rs, src/bin/bin2.rs), thus they automatically detects as binaries.

From your description, it sounds like you are checking from the build script, though?  The following works for me when placed inside the binary:
// src/main.rs
fn main() {
    println!("Hello, {}", env!("CARGO_BIN_NAME"));
}

Are you checking it from the build script, or from the binary?

@ehuss, I have understood what you mean. Thank you for explanation.

This is an essential feature but the documentation for these environment variables are extremely confusing.
The documentation for CARGO_BIN_NAME says "The name of the binary that is currently being compiled". From that statement, it's clear that the variable is meant to be checked from the build script. But it's not available there? It's only available FROM inside the binary, AFTER being compiled? 🤷🏽 That doesn't make sense to me at all.
I have multiple bins and only one of them needs the build script. But I cannot check which binary is being compiled from the build script to do that. The only way I found so far is to manually set an env var before running the build and repeating that for each binary I have to build. That's tiresome. Why not have the CARGO_BIN_NAME available, where it's actually needed? I've also seen https://github.com/rust-lang/cargo/issues/1430 which was never resolved.


Implement interaction between backend and UI screen

Allow the user to add waypoints to the list of locations of the itinerary they are creating. This is done by searching for a point of interest in the location text field and proposing related locations on the map which could correspond to the search. When the user chooses a location, it is added to the list and forces a recalculation of the preview.
Strongly related to https://github.com/orgs/WanderWise-Inc/projects/1/views/1?filterQuery=&pane=issue&itemId=62490045

Resolved in PR #251


[Bug] Centrifuge doesn't accept Fuzzy Wildcard

This script would show up in NEI but if you try to put a sapling in, it wouldn't accept it. Removing the ":*" makes it work again.

For me it shows it correct in NEI, but the machine does not process it. What exactly (if you use) addon/version have you for NEI Forestry recipes?

Tonius' NEI Integration.

I am currently uncertain: can the normal Minecraft Itemstack handle the metadata/Damagevalues wildcard?

Similar behaviour in ThermicFabricator, Carpenter for example accepts wildcard. I assume that i implemented something different than jarred, so that my machines don't accept wildcards

OK i found something, but don't know if i am stupid or Forestry do something wrong.
@mezz
Compare the matching methods of Centrifuge and Fermenter (are used to check if valid recipe):
https://github.com/ForestryMC/ForestryMC/blob/dev/src/main/java/forestry/factory/gadgets/MachineCentrifuge.java#L75
https://github.com/ForestryMC/ForestryMC/blob/dev/src/main/java/forestry/factory/gadgets/MachineFermenter.java#L95
I changend Centrifuges matches logic to the one used in the Fermenter and now it seems working.

Is it now working? fixed they the Bug in Forestry.


how to set the count of  the scroll content child?

I dont know how to set the childcount, it looks like  the child count is determined by the scroll height.

You don't have to - please see step 11 of the Readme where you have to set RowCount, which is the number of items in the model. The view will determine how many visual children are needed at once.


Fix pane resizing issue

Check if there is more than one pane in the pane-container before resizing the pane
Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change
When a request to resize a pane is sent, atom never checked how many panes were present in the pane-container. This resulted in a situation where if a package or the user tried to resize the pane, and there was only one pane, the pane would resize itself though there was no other pane to take up the space it freed.
The PR adds a check to the pane model, to check the number of panes in its container, before resizing the pane. If there is only one pane in the pane's container, it does not try to resize the pane, otherwise, resize as before.
Alternate Designs
NA
Why Should This Be In Core?
Fixes incorrect behaviour when resizing panes.
Benefits
Fixes issue #11673
Possible Drawbacks
None that I can think of
Applicable Issues
#11673

Thanks for helping out!
This will need tests before it can be reviewed though.

Superseded by https://github.com/atom/atom/pull/16428


freezing

Describe the bug
Hi, my issue is about freezing android app. When i open it, I can normally click and do anything, after few minutes it freezes, and I have to restart it, than it remains normal.
To Reproduce
Steps to reproduce the behavior:

Go to 'app'
Click on 'anything few times'
3 wait a moment
it freezes

Expected behavior
normal behavior
Additional context
This is fairly old tablet, but i think it should run normally with this app

Hi, what android version do you use?
Does the app always freeze after a designated time or only when its beeing worked with actively?
Do the buttons still visually respond after it froze or does the app freeze completely?
Are your macro deck server and app at the newest version?
Have your tried if the issues also happen with the webclient?

Hi, my android version is 5.0.1, app always freezes after about minute
after i open it, then I have to restart it, and it works fine. Buttons are
not responding when app is freezed, it crashes if i try to click any
button. Server is the newest version, i havent tried web client suddenly
sob., 13 sie 2022 o 00:12 Xenox003 @.***> napisał(a):

Hi, what android version do you use?
Does the app always freeze after a designated time or only when its beeing
worked with actively?
Do the buttons still visually respond after it froze or does the app
freeze completely?
Are your macro deck server and app at the newest version?
Have your tried if the issues also happen with the webclient?
—
Reply to this email directly, view it on GitHub
https://github.com/Macro-Deck-org/Macro-Deck/issues/276#issuecomment-1213556793,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2BWGIUMZYQBQQIFI3R2FHDVY3D5VANCNFSM553YBEVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I suspect its your Android version causing those problems as its already pretty old.
As a workaround you can embed the new betaclient (https://betaclient.macrodeck.org) onto your homescreen, this will work simmilar to the app but requires an internet connection.

Did the problem go away since i posted my fix?

I am not very interested in web app, im waiting for updates for now
wt., 13 wrz 2022, 19:58 użytkownik Xenox003 @.***>
napisał:

Did the problem go away since i posted my fix?
—
Reply to this email directly, view it on GitHub
https://github.com/Macro-Deck-org/Macro-Deck/issues/276#issuecomment-1245766493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2BWGIUGWM2V23ROHDXHTHTV6C6EXANCNFSM553YBEVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

So the issue still occurs?

To be fair, i dont know, i think Yes, but i dont remember, ill check and
reply
wt., 13 wrz 2022, 20:17 użytkownik Xenox003 @.***>
napisał:

So the issue still occurs?
—
Reply to this email directly, view it on GitHub
https://github.com/Macro-Deck-org/Macro-Deck/issues/276#issuecomment-1245787992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2BWGIXKHYZ5OBHH5EKCLDTV6DAMNANCNFSM553YBEVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

yup, after checking Im sure, problem still occurs, but not as often
wt., 13 wrz 2022 o 20:24 I_am_Piter @.***> napisał(a):

To be fair, i dont know, i think Yes, but i dont remember, ill check and
reply
wt., 13 wrz 2022, 20:17 użytkownik Xenox003 @.***>
napisał:

So the issue still occurs?
—
Reply to this email directly, view it on GitHub
https://github.com/Macro-Deck-org/Macro-Deck/issues/276#issuecomment-1245787992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2BWGIXKHYZ5OBHH5EKCLDTV6DAMNANCNFSM553YBEVQ
.
You are receiving this because you authored the thread.Message ID:
@.***>


Ubuntu installation

How do you install this plugin on the Keepass2 for Ubuntu?

You can't: it is currently not compatible with Mono. See #18 for further details.


Corrected app_metadata and user_metadata retrieval.

 Issue by typhoon2099
Monday Apr 09, 2018 at 15:59 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/pull/72

app_metadata and user_metadata can't be returned by the userinfo endpoint any more (unless you signed paid for Auth0 before 1st September 2017). This pull request and the following Auth0 rule corrects this:
function (user, context, callback) {
  var namespace = 'http://catalogue/';
   if (context.idToken && user.user_metadata) {
     context.idToken[namespace + 'user_metadata'] = user.user_metadata;
   }
   if (context.idToken && user.app_metadata) {
     context.idToken[namespace + 'app_metadata'] = user.app_metadata;
   }
   callback(null, user, context);
}

typhoon2099 included the following code: https://github.com/catalogueglobal/datatools-ui/pull/72/commits

 Comment by skinkie
Tuesday Jul 10, 2018 at 19:48 GMT

Can confirm this works, at least ... the next problem is...
https://github.com/catalogueglobal/datatools-server/issues/102

 Comment by abyrd
Wednesday Jul 11, 2018 at 15:47 GMT

@landonreed have you tested out this pull request?

 Comment by landonreed
Thursday Jul 12, 2018 at 15:42 GMT

When attempting to add the rule to our Auth0 account and clicking 'Try this rule', I get: ERROR: Unknown error. Possible reason: you are not calling callback(null, user, context)

 Comment by landonreed
Thursday Jul 12, 2018 at 15:47 GMT

Ah, never mind. I was including an underscore in the rule description and Auth0 didn't like that.

 Comment by landonreed
Thursday Jul 12, 2018 at 16:09 GMT

Adding this rule does not appear to work for our (pre-September 2017) existing Auth0 account; however, it does work for a test account I created a couple of days ago.
In order to accept this PR, the assignment of the profile.app_metadata and profile.user_metadata to the catalogue-scoped values should be wrapped in a conditional, e.g.:
if (typeof profile.user_metadata === 'undefined' && typeof profile.app_metadata === 'undefined') {
  profile.app_metadata = profile['http://catalogue/app_metadata']
...

There should also be sufficient comments about what is going on here (the reason for the assignment and wrapping it in an if block) and a reference to this issue and any other Auth0 documentation about the rule that @typhoon2099 provided above.  I would also like to see a check for the existence of the scoped fields within this block and an informative error thrown about the Auth0 rule if it is not found. And an update to the docs regarding Auth0 setup would be greatly appreciated.
As @skinkie mentioned above, this also depends on a server fix, which we'd be happy to review if anyone has some code that's backwards-compatible.  Thanks so much for investigating this!

 Comment by landonreed
Thursday Jul 12, 2018 at 16:22 GMT

I've also just discovered that adding the above rule to a pre-September 2017 Auth0 account does not play nicely with applications running with the existing Auth setup.  So, I'm not entirely sure this rule/hack is a great path forward.
Edit: in other words, when I added the rule to our existing account, authentication seemed to stop working for our applications running the existing code.  I'm not sure if there's a way to isolate the application of rules to a single application within an Auth0 tenant.

 Comment by landonreed
Thursday Aug 30, 2018 at 16:29 GMT

I have just learned that it is possible to check the clientID on the context arg in the rule.  For example:
function (user, context, callback) {
  // Only apply rule to certain client
  if (context.clientID === 'ABC123') {
    const namespace = 'http://datatools/';
    if (context.idToken && user.user_metadata) {
      context.idToken[namespace + 'user_metadata'] = user.user_metadata;
    }
    if (context.idToken && user.app_metadata) {
      context.idToken[namespace + 'app_metadata'] = user.app_metadata;
    }
  }
  callback(null, user, context);
}

So for Conveyal's Auth0 tenant, we can apply this rule only to certain applications, which means it won't interfere with our other production applications.
Adding this rule and changing the client to get the user profile from a datatools-server endpoint (rather than using the Auth0 Lock getUserInfo endpoint, should be enough to fix the Auth0 issue.

 Comment by landonreed
Friday Oct 12, 2018 at 16:50 GMT

Closing in favor of #250


Upgrades to frees-rpc-common 0.3.0

It also releases a new patch version.

Codecov Report

Merging #23 into master will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           master   #23   +/-   ##
====================================
  Coverage       0%    0%
====================================
  Files           5     5
  Lines         126   126
  Branches        1     1
====================================
  Misses        126   126

Continue to review full report at Codecov.

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


Test examples/ping-*/README.md

Using cram to execute and verify instructions in the following files:

examples/ping-json/README.md
examples/ping-thrift-gen/README.md
examples/ping-thrift/README.md

Currently, all of the tests are failing due to incomplete instructions in the README, but opening the pull  request anyway to set the path forward to fixing them.

Coverage decreased (-0.05%) to 94.621% when pulling 44cbefce0ed6341423b520749b06f2289631d030 on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

Coverage remained the same at 94.668% when pulling 32aed6541ce6ae53100d81f82a47ce7fe346e90a on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

Coverage increased (+0.02%) to 94.691% when pulling 32aed6541ce6ae53100d81f82a47ce7fe346e90a on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

Coverage increased (+0.1%) to 94.785% when pulling 32aed6541ce6ae53100d81f82a47ce7fe346e90a on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

LGTM, happy to merge, though I do have this philosophical thought:
Is storing the tests in the README actually a good idea? I know it seems cool because it's less files or whatever, but it is sane/obvious/intuitive?
Note we have sacrificed formatting in the README to do this:
https://github.com/uber/ringpop-go/blob/32aed6541ce6ae53100d81f82a47ce7fe346e90a/examples/ping-json/README.md#running-the-example
I don't feel strongly enough about this to block the merge of this PR. Practically, I don't think this will cause us a huge issue/effort in the future. But I do think tests in the README is a bit strange and may increase the average WTFs/min in the codebase.

@dansimau thanks for the comment, and this is indeed a good philosophical discussion.
If we look at what we want to achieve:

Have clear instructions on how to build and use the project.
Make sure the instructions are sane (= automatically tested).

I see two approaches:

More common: separate the documentation generation and testing. Usually deliverable is a docbook- or a sphinx-like documentation artifact, which, in the back, nobody cares how it's generated.
Less common: test the actual final documentation.
Move the steps from README to a script, and test the script.

(1) is a standard practice. However, since github does not really allow it easily (I want the README to be readable as-is).
(2) gives more wtfs/minute, but it will work, as long as people keep adding the correct examples.
(3) seems to be least wtfs/minute, but leads to a least readable README.

cc @severb can you check out the examples?

Coverage decreased (-0.2%) to 94.481% when pulling 98552ea9878871148424ff9c4d71b56c02f01f7f on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

Coverage increased (+0.07%) to 94.738% when pulling 6042e48b3356128184271473dfe798f2da3bc489 on motiejus-glide-examples into 6a7722079e2c3c605e94a75bd628ea87b2d9115e on dev.

LGTM

@dansimau we actually fixed the formatting in README. Can you re-check?

LGTM

Coverage remained the same at 94.668% when pulling 1c5ab9546527126e117c7ac08da6d8e130ce079d on motiejus-glide-examples into 2abc28cc9cb854e7584b66c4a6e86e219fed4d1d on dev.


Fixed #27217 -- Fixed CreateModel optimization with models that use mixins.

https://code.djangoproject.com/ticket/27217

Is there a reason not to test CreateModel.references_model directly?

I pushed an alternate test and will remove the other one if you prefer. I wasn't sure about the best way to test it since I haven't seen any unit tests for operation methods.

The patch and the new test LGTM.


I don't cant get the id of original element (LI) that triggered de context menu.

i have UL with LI elements, and this script
var mn2 = [{
name: 'delete',
img: 'images/delete.png',
title: 'to delete this element',
fun: function (event) {
alert(event.target.id+' i am delete this element')
}}]
$('LI').contextMenu(mn2,{triggerOn:'contextmenu','displayAround': 'trigger'});
but i don't cant get the id of original element (LI) that triggered de context menu.

Duplicate to #14 , Check http://ignitersworld.com/lab/contextMenu.html#menuParameterDoc,
You will get data and event as parameters in fun method, where you can access the trigger using data.trigger


Click on the middle point for optimization

Click on the middle point, and the two newly added points can be directly dragged . This can greatly improve the efficiency of editing and drawing polygons ,
When dragging the middle point, two new points are automatically added, and both new points can be directly dragged。

@JamesLMilner  I suggest changing the onClick event at the midpoint to be triggered when keydown, rather than when keyup, so that you don't have to click before dragging

Thanks for raising this idea. A feature like this might be possible but requires a bit of thinking about how it works with the wider select mode without making the code harder to maintain. A feature like this would be behind a flag and which developers could opt into by turning it on. I will look into it and see if it's possible, but can't promise anything on time lines.

@JamesLMilner, thanks a lot.


target + compile for SDK 32, solve some deprecations, minor refactors

we need to use release candidate version of gradle wrapper and the plugin for SDK 32 at the moment. also solves log4j vulnerabilities in gradle wrapper (yes. really.). release candidate of Android Studio Bumblebee is recommended as well.
also some tiny refactors
Signed-off-by: June<EMAIL_ADDRESS>

We're not reading to move to API 32 since it's not released yet. This should be split up and appcompat, etc. should be updated across all our apps at once.


Integrate with github commandline tool for release creation

The github command line tool provides a release command to create a github release. It would be nice if you optionally support it in our process.
Related topics:

[ ] changelog #231
[ ] artifact/binary upload #201

@epage right, I think it's an optional one. Most binaries are platform dependent so the workflow should be using gh release upload to upload them individually.
I just changed prerequisites to related topics to clarify.

I would love this functionality even without any binary artifact handling.

For those interested, for now I've taken to creating the release, with notes, via Github Actions
See https://github.com/crate-ci/typos/commit/5c92dc6f8cc68b9d1c8cb1e8840b81e78cf7f65d

@epage Thanks for sharing!

Decided to handle this by documenting it in the FAQ


Fix the missing participated watcher in the subscriptions endpoint.

Users who participated in issue/PR and did not explicitly unsubscribe are also watchers, although their records are not kept in the issue_watch table. But the original endpoint ignored this part of the watcher.

Raw SQL

	SELECT id
	FROM `user`
	WHERE (
	    id IN (
	        SELECT `comment`.poster_id
	        FROM comment
	        WHERE `comment`.issue_id = ?
	        AND `comment`.type IN (?, ?, ?)
	        AND `comment`.poster_id NOT IN (
	            SELECT `issue_watch`.user_id
	            FROM issue_watch
	            WHERE `issue_watch`.issue_id = ?
	            AND `issue_watch`.is_watching = ?
	        )
	    )
	    OR id IN (
	        SELECT `issue_watch`.user_id
	        FROM issue_watch
	        WHERE `issue_watch`.issue_id = ?
	        AND `issue_watch`.is_watching = ?
	    )
	)
	AND `user`.is_active = ?
	AND `user`.prohibit_login = ?

Fix #26750

@puni9869
SELECT id
	FROM `user`
	WHERE (
	    id IN (
	        SELECT `comment`.poster_id
	        FROM comment
	        WHERE `comment`.issue_id = ?
	        AND `comment`.type IN (?, ?, ?)
	        AND `comment`.poster_id NOT IN (
	            SELECT `issue_watch`.user_id
	            FROM issue_watch
	            WHERE `issue_watch`.issue_id = ?
	            AND `issue_watch`.is_watching = ?
	        )
	    )
	    OR id IN (
	        SELECT `issue_watch`.user_id
	        FROM issue_watch
	        WHERE `issue_watch`.issue_id = ?
	        AND `issue_watch`.is_watching = ?
	    )
	)
	AND `user`.is_active = ?
	AND `user`.prohibit_login = ?

@puni9869
SELECT id
	FROM `user`
	WHERE (
	    id IN (
	        SELECT `comment`.poster_id
	        FROM comment
	        WHERE `comment`.issue_id = ?
	        AND `comment`.type IN (?, ?, ?)
	        AND `comment`.poster_id NOT IN (
	            SELECT `issue_watch`.user_id
	            FROM issue_watch
	            WHERE `issue_watch`.issue_id = ?
	            AND `issue_watch`.is_watching = ?
	        )
	    )
	    OR id IN (
	        SELECT `issue_watch`.user_id
	        FROM issue_watch
	        WHERE `issue_watch`.issue_id = ?
	        AND `issue_watch`.is_watching = ?
	    )
	)
	AND `user`.is_active = ?
	AND `user`.prohibit_login = ?

Could you optimized this query,
see the schema of the table and run the ANALYSE <query> you will see some hints on indexes and more.
https://thoughtbot.com/blog/reading-an-explain-analyze-query-plan
The query plan.

@puni9869

-> Filter: ((`user`.prohibit_login = 0) and (`user`.is_active = 1) and (<in_optimizer>(`user`.id,`user`.id in (select #2)) or <in_optimizer>(`user`.id,`user`.id in (select #4))))  (cost=1.45 rows=1) (actual time=0.088..0.097 rows=1 loops=1)
    -> Table scan on user  (cost=1.45 rows=12) (actual time=0.032..0.034 rows=12 loops=1)
    -> Select #2 (subquery in condition; run only once)
        -> Filter: ((`user`.id = `<materialized_subquery>`.poster_id))  (cost=1.07..1.07 rows=0.2) (actual time=0.014..0.014 rows=0 loops=1)
            -> Limit: 1 row(s)  (cost=1.05..1.05 rows=0.2) (actual time=0.013..0.013 rows=0 loops=1)
                -> Index lookup on <materialized_subquery> using <auto_distinct_key> (poster_id=`user`.id)  (actual time=0.013..0.013 rows=0 loops=1)
                    -> Materialize with deduplication  (cost=1.05..1.05 rows=0.2) (actual time=0.012..0.012 rows=0 loops=1)
                        -> Filter: ((`comment`.`type` in (0,1,2)) and <in_optimizer>(`comment`.poster_id,<exists>(select #3) is false))  (cost=1.02 rows=0.2) (actual time=0.010..0.010 rows=0 loops=1)
                            -> Index lookup on comment using IDX_comment_issue_id (issue_id=102)  (cost=1.02 rows=1) (actual time=0.009..0.009 rows=0 loops=1)
                            -> Select #3 (subquery in condition; dependent)
                                -> Limit: 1 row(s)  (never executed)
                                    -> Filter: ((issue_watch.is_watching = 0) and (issue_watch.issue_id = 102) and <if>(outer_field_is_not_null, (<cache>(`comment`.poster_id) = issue_watch.user_id), true))  (never executed)
                                        -> Alternative plans for IN subquery: Index lookup unless user_id IS NULL  (never executed)
                                            -> Single-row index lookup on issue_watch using UQE_issue_watch_watch (user_id=<cache>(`comment`.poster_id), issue_id=102)  (never executed)
                                            -> Table scan on issue_watch  (never executed)
    -> Select #4 (subquery in condition; run only once)
        -> Filter: ((`user`.id = `<materialized_subquery>`.user_id))  (cost=1.05..1.05 rows=1) (actual time=0.003..0.003 rows=0 loops=13)
            -> Limit: 1 row(s)  (cost=0.95..0.95 rows=1) (actual time=0.003..0.003 rows=0 loops=13)
                -> Index lookup on <materialized_subquery> using <auto_distinct_key> (user_id=`user`.id)  (actual time=0.003..0.003 rows=0 loops=13)
                    -> Materialize with deduplication  (cost=0.95..0.95 rows=1) (actual time=0.032..0.032 rows=1 loops=1)
                        -> Filter: ((issue_watch.issue_id = 102) and (issue_watch.is_watching = 1))  (cost=0.85 rows=1) (actual time=0.024..0.027 rows=1 loops=1)
                            -> Table scan on issue_watch  (cost=0.85 rows=6) (actual time=0.023..0.026 rows=6 loops=1)

@puni9869 I adjusted the order of the index fields of issue_watch,  the query selected by  index except the user table.


Multiple select.

Hi,
can we select multiple option using this directive  ?

Hello. No, that is not currently supported.

Any way to implement it


SkyFactory 4 NuclearCraft Alloy Machine crash

Was making Tough Alloy when this happened.
https://pastebin.com/WXHALeU7
Now when I load the world this happens.
https://pastebin.com/MsLpHHuJ
Am backed up but I want to use that Alloy Furnace.. Dangit.
Not sure if I posted this to the right Github as I was asked why I was posting it there.. I thought you would post issues in the pack github and then the Dev would seek the aid of the mod maker. Hopefully I won't be bouncing between Githubs

This is a Nuclearcraft bug, crashed in a fresh install with just Nuclearcraft and JEI.
Crashlog: https://pastebin.com/bAYgNRWa
Steps to replicate:

Have a powered Alloy Furnace (used Califwhatever RTGs on 5 sides)
Fully upgrade the Alloy Furnace with Speed and Energy Upgrades. (upgrades are important, doesn't seem to crash without upgrades.  It will crash with less but either didn't crash or I'm misremembering my tests when I only used 1 upgrade, and I don't have time to keep making new worlds to test right now.)
Place Lithium DUST in the first slot. (doesn't seem to crash in the second slot)
Place Ferroborn Ingot in the second slot.
Crash almost always happens before the first ingot is produced.

Thanks for the heads up - this was actually reported to me in our Discord server the other day, and I know how to fix it. I'll try to get a new build out as soon as possible, but right now I'm in the middle of a few mock-exam-type-things and it'll be a couple of days before I can sort it out!

Real Life comes first...glad you already know the fix, and look forward to seeing it out when you have time.

I should mention that, for now, you should be able to avoid crashes like this in general as long as the ingredients are positioned in the order they appear in JEI ;)

I have the same problem.
But i found out that it only happens with lithium dust in the false slot.
When you take lithium ingots it dosen't crash no matter which slot you use.

Yep, that's because the ingot is the first of the allowed lithium ingredients in the registered recipe. What's happening is that NC needs to know which ingredient in that list of possible inputs is actually being used, but it's getting them the wrong way round because it's incorrectly taking into account the ordering of the inputs in the machine, so when you put the dust in the second slot, which is specified as the second allowed lithium ingredient in the recipe registration, it tries to get the second allowed ferroboron instead instead because ferroboron is the second ingredient in the tough alloy recipe, which doesn't exist and throws that out of bounds error.

Hey, just wanted to report that this same issue happens with the chemical reactor. The small difference in my crash is that it's all fluid based and not item based:
java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
        at java.util.ArrayList.rangeCheck(ArrayList.java:657)
        at java.util.ArrayList.get(ArrayList.java:433)
        at nc.recipe.ingredient.FluidArrayIngredient.getMaxStackSize(FluidArrayIngredient.java:53)
        at nc.tile.processor.TileFluidProcessor.produceProducts(TileFluidProcessor.java:282)
        at nc.tile.processor.TileFluidProcessor.finishProcess(TileFluidProcessor.java:263)
        at nc.tile.processor.TileFluidProcessor.process(TileFluidProcessor.java:258)
        at nc.tile.processor.TileFluidProcessor.updateProcessor(TileFluidProcessor.java:127)
        at nc.tile.processor.TileFluidProcessor.func_73660_a(TileFluidProcessor.java:119)
        at net.minecraft.world.World.func_72939_s(World.java:1835)
        at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:613)
        at net.minecraft.server.MinecraftServer.func_71190_q(MinecraftServer.java:767)
        at net.minecraft.server.dedicated.DedicatedServer.func_71190_q(DedicatedServer.java:397)
        at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:668)
        at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:526)
        at java.lang.Thread.run(Thread.java:748)

-- Block entity being ticked --
Details:
        Name: nuclearcraft:chemical_reactor // nc.tile.processor.TileProcessor.ChemicalReactor
        Block type: ID #452 (tile.nuclearcraft.chemical_reactor_idle // nc.block.tile.processor.BlockProcessor // nuclearcraft:chemical_reactor_idle)
        Block data value: 5 / 0x5 / 0b0101
        Block location: World: (-3067,64,1029), Chunk: (at 5,4,5 in -192,64; contains blocks -3072,0,1024 to -3057,255,1039), Region: (-6,2; contains chunks -192,64 to -161,95, blocks -3072,0,1024 to -2561,255,1535)
        Actual block type: ID #453 (tile.nuclearcraft.chemical_reactor_active // nc.block.tile.processor.BlockProcessor // nuclearcraft:chemical_reactor_active)
        Actual block data value: 5 / 0x5 / 0b0101
Stacktrace:
        at net.minecraft.world.World.func_72939_s(World.java:1835)
        at net.minecraft.world.WorldServer.func_72939_s(WorldServer.java:613)

Hope this helps :)

Yep, exactly the same thing, but with fluids!

I have another crash report for ya just for record keeping :) https://pastebin.com/k4XjrJ8G

Fixed with version 2.16f ;)

Perfect!


How to add the PR tags in the Azure DevOps dashboard?

How can I add the PR tags to the DevOps Dashboard? Any help is really appreciated.

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

ID: 514ba817-c404-380b-6c8d-67ea982e9a22
Version Independent ID: 915a8f30-5192-8b0a-d6a4-43fe7a606ff6
Content: Catalog of widgets you can add to a dashboard - Azure DevOps
Content Source: docs/report/dashboards/widget-catalog.md
Product: devops
Technology: devops-analytics
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@SibeeshVenu - Thank you for your inquiry. I don't know of any widget that supports adding PR tags to the dashboard. You can request such a feature on our Developer Community page.


nodejs - Transcription status "NotStarted", no transcription id

When I submit my transcription job from nodejs, I am faced with a couple of issues. First, the api's createTranscription method returns nothing when doing an asynchronous call. Since this isn't working, and since it is a REST api, I tried sending a request from Postman with the appropriate body and headers to the Create Transcription endpoint. My file is in a Azure blob container. I am returned a 201 response where the job was created, however there is no transcription job id and the status reads "NotStarted" each time I submit.
Here is the returned body:
{
    "self": "https://westus2.api.cognitive.microsoft.com/speechtotext/v3.0/transcriptions/22014645-b005-4acb-9aaf-93e9960f2d38",
    "model": {
        "self": "https://westus2.api.cognitive.microsoft.com/speechtotext/v3.0/models/1cfa5e5e-6203-4c9f-b9b3-30142e84fac5"
    },
    "links": {
        "files": "https://westus2.api.cognitive.microsoft.com/speechtotext/v3.0/transcriptions/22014645-b005-4acb-9aaf-93e9960f2d38/files"
    },
    "properties": {
        "diarizationEnabled": false,
        "wordLevelTimestampsEnabled": false,
        "channels": [
            0,
            1
        ],
        "punctuationMode": "DictatedAndAutomatic",
        "profanityFilterMode": "Masked"
    },
    "lastActionDateTime": "2021-04-21T16:22:05Z",
    **"status": "NotStarted",**
    "createdDateTime": "2021-04-21T16:22:05Z",
    "locale": "en-US",
    "displayName": "Transcription using default model for en-US"
}

The transcription id is the id at the end of the self field in the response: "self": "https://westus2.api.cognitive.microsoft.com/speechtotext/v3.0/transcriptions/22014645-b005-4acb-9aaf-93e9960f2d38". To see the current status of your transcription, do a GET request on that route. Please also refer to these docs.

Thanks


Cannot handle space in file path when using multiple profiles

Windows Terminal version (or Windows build number)
10.0.19041.1052
Other Software
No response
Steps to reproduce
Try to open wt with muliple tabs and multiple profiles wt new-tab when there are spaces in the folder names.
Expected Behavior
Multiple tabs will be opened with multiple profiles
Actual Behavior
I can open multiple tabs with multiple profiles just fine when there are no spaces ion the folder names, but when I have spaces in the folder names and I have to enclose the path in quotes, I can only use a single profile in the beginning.

Can you give some example commandlines? wt -d "c:\program files" ; -d "c:\program files" seems to work just fine over here...

Thank you for looking into this. The problem must be that I have a dash in my user name.  This results in the error:
wt -d "C:\Users\Jordan - Regular\Files\Web\Pixelmatic\Projects\web-auth-backend" `; new-tab -p "Windows Powershell" -d "C:\Users\Jordan - Regular\Files\Web\Pixelmatic\Projects\web-auth-backend" `; new-tab -p "Windows Powershell" -d "C:\Users\Jordan - Regular\Files\Web\Pixelmatic\Projects\exo-auth"

Thank you for looking into this. The problem must be that I have a dash in my user name. This results in the error:
wt -d "C:\Users\Jordan - Regular\Files\Web\Company\Projects\web-auth-backend" `; new-tab -p "Windows Powershell" -d "C:\Users\Jordan - Regular\Files\Web\Company\Projects\web-auth-backend" `; new-tab -p "Windows Powershell" -d "C:\Users\Jordan - Regular\Files\Web\Company\Projects\back-auth"

Spaces and hyphens in the directory name seem OK.  These work fine at a CMD prompt and a PowerShell prompt, respectively,
wt -d "v:\a - b" ; new-tab -p "Windows Powershell" -d "v:\a - b" ; new-tab -p "Windows Powershell" -d "v:\a - b"
wt -d "v:\a - b" `; new-tab -p "Windows Powershell" -d "v:\a - b" `; new-tab -p "Windows Powershell" -d "v:\a - b"

What is "the error"?

The error is [error 0x8007010b when launching `powershell.exe']

@jordankittle what version of the Terminal are you on? Somewhere in the 1.8-1.9 timeframe, we improved the messaging for that specific error (see #9114, #10045), which might help clarify

That's ERROR_DIRECTORY (the directory name is invalid).  I get it if I use "-d" with the name of a non-existent directory.  I can only suggest you check those names:
"C:\Users\Jordan - Regular\Files\Web\Company\Projects\web-auth-backend"
"C:\Users\Jordan - Regular\Files\Web\Company\Projects\back-auth"

They are valid names. I have double and triple checked. The first one works fine, the second errors, even when using the same exact -d name.

I have verified that this error does not happen when I use the same exact script without the Jordan - Regular folder.  It is only when using this folder that I get the error. And it is always the second instance of new-tab, not the first, even when using the exact same directory name.

What I am trying to do is build a shortcut that opens tabs to all my working directories for a project. After chasing this for hours, I am very sure there is a bug here.

I had also tried it in the Run dialog (Win+R).  Just now I tried it as the target of a shortcut.  Both worked!  So ... beats me!  I'll be glad to hear what's going on.

I wish I could give you more information.  If anybody from Microsoft wants to have a meeting where I share my screen let's do it .

@jordankittle, fooling around with this, I just ran into an apparent 260 (MAX_PATH) character limit in a shortcut's command line.  You're command line exceeds 260 characters also.  And if cut off at 260, you're would be right in the middle of a (now bad) directory name.  That's exactly what happened to me (and I got the same error as you).  Did you try your command line at an actual command line (CMD ot PowerShell)?  I wouldn't be surprised if it works there.
If that's the problem, it may be with the shortcut dialog (and not with shortcuts, per se).  Maybe PowerShell or VBScript offers a way to create a shortcut that's not subject to that limitation.

Wow, I feel a little silly now. Sorry for wasting your time! In my shortcut I have been using it was indeed cut off! Thank you very much and again I'm sorry I didn't catch this. In my shortcut I was using it was the very end that was cut off so that's why when I switched from  "C:\Users\Jordan - Regular" to  "C:\Users" it would work.

FYI, the limitation is with the shortcut dialog, and not with shortcuts, per se.  My shell has a SHORTCUT command.  With it I made a working shortcut with a target well over 300 characters.  AS I said before, you might be able to do that with VBScript, PowerShell, or some other tool.


Add testcafe-community & Edit TestCafe

Add the new testcafe-community plugin which is a fork from the original plugin with community gathered rules.  It extends the original plugin which only provided the global environment variables for TestCafe. The original has not seen an update since 2016 and is less prepared to support PRs for additional rules.  Therefore, I moved the original TestCafe down to the globals category since that is all it provides from the actual implementation index.js.
Thanks again for this awesome repo!

Thanks very much for the PR!


feat(axios): handle MaxBodyLengthExceededError

Add error handler on fileContainer request to check if the error is a 'MaxBodyLengthExceeded'.
If that so, then add some mitigation step in the error message

We do not support the 'big files' yet which should prolly have its own strategy to match what's explained here:
https://docs.coveo.com/en/69/index-content/using-the-compressedbinarydatafileid-property
So I'm reducing the limit to 5MB to be accurate


panc: bump javacc

@ned21 @jrha some clarification: template is a reserved word in the recent javacc, so can't have methods or namespace with word template in the jjt.
so i chose ttemplate to replace it. that's what most of this PR is about.
also, no real improvements wrt speed or memusage (and i don't think that was ever an issue).
this PR is mainly so we can keep javacc up to date.

It's called ttemplate but has nothing to do with Template Toolkit... confusing?


Context of the include directive

Hi,
I'm using a include directive like this in my main ninja file:
include sources.ninja
The sources.ninja file is in the same directory than build.ninja
And I call ninja using path variables from eclipse. The problem is that the include directory seems to use the path context of Eclipse and not the one from ninja : the include file is not found.
If I'm executing ninja from the command line and from the same directory than build.ninja and sources.ninja , it's working well.
The only way I found is to use an absolute path in build.ninja for the include but I would find a cleaner way :-/

All paths in ninja should be relative to the directory containing the root build.ninja.  Does that help?

How does eclipse execute ninja?  Does it print a command line?

Yes , I change the default make in my project by ninja. After some researching , I found a solution in Eclipse. There's a field to set the Build directory for the builder , and by pointing to the ninja file, it's working well. So I guess Eclipse use this setting to change the context for ninja.

You can probably also pass -C folder/with/your/buildfile to ninja from Eclipse.


Natively support sparse registries

Motivation
With Cargo 1.68.0, sparse registries were made stable. With sparse registries, index metadata is stored in an HTTP api rather than in a git repository.
As relevant to Crane, the relevant changes are mostly that registries do not always start with registry+ and mostly start with registry+ or sparse+ depending on whether the registry is sparse or not.
This PR adjusts the core of Crane to differentiate between registry and sparse, and adds a new sparse registry factory to make things easy.
I'm new to Nix, so it's possible and likely that my code is incorrect.
Checklist

[X] added tests to verify new behavior
[X] added an example template or updated an existing one
[X] updated docs/API.md (or general documentation) with changes
[X] updated CHANGELOG.md

@j-baker I had originally assumed the build issues were due to using an older rustc, but after merging the latest master in, I can't seem to get the tests to pass locally
error: not sure how to download crates from registry+https://github.com/rust-lang/crates.io-index.
known registries are:
registry+https://github.com/rust-lang/crates.io-index/

I have a feeling that something about the trailing slash handling on URLs is tripping the evaluation up (registry+ URLs do not have a trailing slash in Cargo.lock so maybe that's related)

Hi, ivan! I discovered the same thing this week and had to use an older
commit. I think when fixing PR comments i broke the crates.io case. I’ll
take a look early next week.
On Sat, 10 Jun 2023 at 00:31, Ivan Petkov @.***> wrote:

@j-baker https://github.com/j-baker I had originally assumed the build
issues were due to using an older rustc, but after merging the latest
master in, I can't seem to get the tests to pass locally
error: not sure how to download crates from registry+https://github.com/rust-lang/crates.io-index.
known registries are:
registry+https://github.com/rust-lang/crates.io-index/
I have a feeling that something about the trailing slash handling on URLs
is tripping the evaluation up (registry+ URLs do not have a trailing
slash in Cargo.lock so maybe that's related)
—
Reply to this email directly, view it on GitHub
https://github.com/ipetkov/crane/pull/324#issuecomment-1585262712, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAKMJMZYCU7OC26UUCDC43XKOW6NANCNFSM6AAAAAAYBEHTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think I figured out the issue! Let's see if the CI passes


Redesign for Charts view to make it more functional

Description:
Discussed in meeting that we need a bit of redesign for Charts view on both Android and iOS to make it more functional and understandable.
Related cards:
https://github.com/ruuvi/com.ruuvi.station/issues/654
https://github.com/ruuvi/com.ruuvi.station.ios/issues/822

We've decided to make the clear/sync buttons easier to understand by renaming them and also adding an export button on the graph view. UI instruction is coming.
Clear - Sync
to be:
Clear View - Download History - Export History

Here:

"Synchronising.." text should use the same font and size as "Updated" text on the bottom.

1.15.0 @andreevdenis
Title is missing from "More..." popup
Title: Longer history
Text: Ruuvi Station mobile app supports maximum 10 days history. Ruuvi Cloud subscribers are able to view up to 2 years of historical data using web app at ruuvi.com/station (requires Ruuvi Gateway router).
Title FI: Pidempi historia
Text FI: Ruuvi Station -mobiilisovellus tukee maksimissaan 10 päivän historiaa. Ruuvi Cloud -tilaajat voivat tarkastella historiatietoja jopa 2 vuoden ajalta web-sovelluksessa osoitteessa ruuvi.com/fi/station (vaatii Ruuvi Gateway -reitittimen).

1.15.5 @andreevdenis I still see old popup text in More... popup

1.15.6

QA passed


Add new worker node

I've an existing cluster with 3 control planes + 1 worker node working. I added a new worker node but I'm not available to add them into the cluster. In the Terraform config file I incremented the count to 2, the script added the new node with no errors in the same subnet than the first worker node.
If I connect to the new node I can ping the control planes and the worker node.
I tried to add them manually with this config:
{   "kind": "Node",   "apiVersion": "v1",   "metadata": {     "name": "<IP_ADDRESS>",     "labels": {       "name": "agent-big-1"     }   } }
But the logs are:
kubectl describe node <IP_ADDRESS>
node <IP_ADDRESS> after 230.759893ms: hcloud/CreateRoute: hcops/AllServersCache.ByName: <IP_ADDRESS> hcops/AllServersCache.getCache: not found   Warning  FailedToCreateRoute  4m46s (x53 over 13m)  route_controller  (combined from similar events): Could not create route 244b6a6e-2e16-4e68-b98b-6024d3037846 <IP_ADDRESS>/24 for node <IP_ADDRESS> after 274.573378ms: hcloud/CreateRoute: hcops/AllServersCache.ByName: <IP_ADDRESS> hcops/AllServersCache.getCache: not found
How can I fix it?

@arkkanoid Which version of kube-hetzner are you using? See here https://github.com/kube-hetzner/terraform-hcloud-kube-hetzner/tags. You have to choose the same version as your cluster was deployed with.


Dummy gives incorrect result on sympy.sympify

Original issue https://github.com/sympy/sympy/issues/22141
>>> import sympy, symengine
>>> a = symengine.symbols('a', cls = symengine.Dummy)
>>> a
_a
>>> sympy.sympify(a)
__a

sympify should return a dummy without the additional underscore.
The problem is here-
https://github.com/symengine/symengine.py/blob/5d49ad3b093ddce56ed13568b256dd74b8eae61d/symengine/lib/symengine_wrapper.pyx#L1263-L1265
the Dummy conversion takes the dummy name of symengine type which gives it two underscores

Awesome. Thanks for the report. It seems that should be easy to fix?

Yes , it's an easy to fix issue
The optimal fix would be to ignore the underscore by doing something like
return sympy.Dummy(str(self)[1:])

It's not. PRs are welcome

@isuruf
Could you check if this is fixed !?
Currently on terminal-
>>> import sympy, symengine
>>> a = symengine.symbols('a', cls = symengine.Dummy)
>>> a
_a
>>> sympy.sympify(a)
_a

Fixed in #375


pyctr.crypto.engine: Load Boot9 keyblobs and store it globally instead of the resulting keys

Just an idea. This would help reduce the amount of global variables. It would also make it easier if, for example, testing requires resetting the engine global state; just unset the stored keyblobs instead of multiple variables with different values.

https://github.com/ihaveamac/pyctr/compare/b7fc25fdb75c...b572a15fe5f8


golint fixes for daemon/ package

One linter advice yet:
daemon/inspect.go:100:56: exported method ContainerExecInspect returns unexported type *daemon.execConfig, which can be annoying to use
Suggestions for addressing it?
There are a surprising (to me) amount of things that became non-exported.
There some comments with "perhaps" in them that  I was not sure if I should take action on.
There are some renames with *Locking because all the functions did was add locking around an existing unexported function.
Some of the comments need work as to the exact details.
A partial history of individual commits can be found in my forked repo in the "split-commits" branch.
for #14756

I'm going to temporarily turn the linter off until it's agreed on how to implement the daemon/inspect.go advice.

fix for windows. kind of concerned that the cross compile did not fail. It should have built everything for Windows, right?

I'd suggest exporting execConfig. Any type returned by an exported function should also be exported.

This is great. The new documentation comments are a pleasure to read.
There's a lot of stuff that got unexported. I'm not familiar enough with this code to weigh in on that. There may be certain functions that should be exported even though they're not currently used outside the package, to allow future flexibility. Hopefully people who understand this code well will comment on that.

Please rebase.

rebased.

@MHBauer needs another rebase, sorry :blush: :sweat_smile:.

Any other comments on KillSig?
I took out the comment on daemon to file a separate issue so there is discussion.

Hum, windows build is failing, setRestarting is re-defined somehow (state.go:217).

Yea, bad rebase on my part. Fixing.

@MHBauer would you mind to rebase. I promise to review!

LGTM
I think we can fix minor comments in other PRs, because this is really painful to rebase and it is indeed fixes all goling warnings.

LGTM, brb, rebasing all my PRs that are affected by me merging this! :tada:


{network} Add tests for five network commands

Description

Add tests for five network commands, below tests will be affected:
test_network_route_table_operation
test_network_traffic_manager
test_network_vnet_gateway_nat_rule
test_network_vpn_connection_ipsec
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

network


Does it work with chrome browser on macOS?

Hi, i'm looking for same bypass script but for my mac device running chrome browser. How do i make it work?

I haven't tested it on macOS but in theory it should work. You need to install it using pip and use it in your Python code. How it works? It will take a screenshot of your screen and use template matching to detect Cloudfalre Turnsile. Then it will control your mouse to automatically click.


Nothing to report, just want to say: THIS IS GORGEOUS! 😆

@imnapo  I wanted to thank you also ; this library was a missing spot in RN ecosystem for years!!

@flyinghawker could you please help me get value from editer with HTML tags so I can sent it to db with same? I am stuck add looking for solution from last 2 days. Please help me.
appreciating your help, because of this only I got stuck.
Input String: This is bold and italic text
output should be something like this or in format of editor:

This is bold and italic text


Add support for .basis texture file format

https://opensource.googleblog.com/2019/05/google-and-binomial-partner-to-open.html

Can't wait !

Thanks a lot @TrevorDev


Align packaging options for new modules

What does this PR do?
This change brings the consistency for the packagingOptions parameter among the new modules.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1353 (81f18b5) into feature/sdkv2 (6e29f7b) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           feature/sdkv2    #1353      +/-   ##
=================================================
- Coverage          81.69%   81.67%   -0.02%
=================================================
  Files                387      387
  Lines              13548    13548
  Branches            2234     2234
=================================================
- Hits               11067    11064       -3
  Misses              1812     1812
- Partials             669      672       +3

see 13 files with indirect coverage changes


Error - 404

Getting a 404 No Success error code when I followed the steps and downloaded the sample application. The exception is thrown at...
... using System;
using System.Threading.Tasks;
using Microsoft.Owin;
using Owin;
using Microsoft.Owin.Security.ActiveDirectory;
using System.Configuration;
namespace TodoSPA
{
public partial class Startup
{
public void ConfigureAuth(IAppBuilder app)
{
app.UseWindowsAzureActiveDirectoryBearerAuthentication(
new WindowsAzureActiveDirectoryBearerAuthenticationOptions
{
Audience = ConfigurationManager.AppSettings["ida:Audience"],
Tenant = ConfigurationManager.AppSettings["ida:Tenant"]
});
}
}

}

Manually installing the active directory NuGet from the console line resolved the issue.

Thanks for reporting @Neumeier, I'll see if I can repro & fix.


[Feature] Add comprehensive CI workflow with build, lint, test, e2e, benchmark, and size jobs

#3407  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @jnsdls and the rest of your teammates on  Graphite


Build script

How can i build dist? There is no build script ... I was trying to fix some mistakes...

@vizo Welcome here and thanks for you interest :) Webpacvk build config was located at /scripts
build file itself was accidentally ignored so i pushed it back now you can easily run npm run build

Thanks!


The new floating button to play video items ("+") is confusing

Hi,
The new feature in 2.4.0 that makes it possible to chose to play on Kodi or locally on the Android device is very nice.
However, it is also confusing to regular users: there are now 2 different "+" icons on the screen and they do different things. One adds to playlist (next to "download to device" and 🗸) and one opens a sub-menu that offers option about where to play.
Two suggestions:

Change the floating + button back to a play sign (⏵). If one wants to play something, one expects a play button, anything else is confusing (except possibly for a button clearly labeled "play", but we are lucky that ⏵is almost universally recognised).
Second idea: differentiate the default choice (play on Kodi) from the secondary choice (play locally on Android device) by making the two choices two different colours.
Signal does this for its "send message" button for instance: the normal option is the default blue, the other option is grey.

Looking at the colours, I was thinking "Play on Kodi" could be the same colour as the theme's default and  "Play locally" could be the same but less saturated. This might be the tricky bit, finding a colour type that works with the different themes.

Thanks!

We kind of mimicked other FAB speed dial designs. I wouldn't mind changing it to the play icon. @SyncedSynapse what do you think?
I'm not sure about the coloring. I don't think "Play locally" is less important than "Play on Kodi". Making it less saturated might give the idea it is.

Cool!
Regarding "play locally", thing is, it is less important. We've been able to play on Kodi since day 1, and only years later locally. It's a lovely bonus, but it's an extra, it's not as fundamental as playing back on Kodi, which is Kore's raison d'être ;)
But it could just be a different colour that shows it's an alternative, it doesn't need to imply it's less important really.

Regarding changing the play icon, i don't have a preference. There are pros/cons for each solution and i don't think one overwhelms the other so i'm a bit indifferent (+ icon signals that it is a menu, but is duplicated on the screen, play icon signals its function, but will confound users when it opens).
Anyway, if it changes, there should be a nice animation on the play icon, and i'm not sure which one would work. The current animation on the + icon is nice and standard.
As for the colors, i wouldn't bother with it.
By the way, there's an option on settings to disable local play if you don't use it (we should discuss if it shouldn't be named "enable local play" instead).

Closing, since the + button has been replaced by a traditional one


Force relationship include

Is there exist any way to force include relationship durning update a resource?
I have a case when I want to return, after update resource, all relationships that was changed by server.

@Exelord
If you submit
PUT /articles/1?include=comments

the comments are included in the response JSON, aren't they?

Yes but I want to include all relationship of fresh created object automatically. Is the include only one option for do that?

The only way I can think of is modifying include in controller
class ArticlesController < ActionController::Base
  include JSONAPI::ActsAsResourceController

  def create
    params[:include].to_s.concat(',comments')
    super
  end
end

Yep I done like that but I think I need to do that on the resource level. Sometimes I make a lot of operations eg in case of update or create some resource on that level and I need to force include some relationships to inform front end that something change what it doesn't request.
Eg. I need to change one relationship to default value after updating another one. And the problem is during returning them. It's not possible to do from resource :/

class MyResource < JSONAPI::Resource

  class << self
    def apply_includes(records, directives)
      super.includes(:comments)
    end
  end
end

Will always apply them by default.

@Fedcomp if I understand correctly, you're trying to configure the resource to include(side load) relationships by default? The code you posted is for eager loading relationships for performance reasons and will not affect the API response.
Check out #389 for more discussion on this

This is what worked for me (including resources for performance reasons)
class MyResource < JSONAPI::Resource
  def self.records(options = {})
    super(options).includes(:comments)
  end
end

I think some of the internals have been moved around and apply_includes did not do anything for me.

In my case I was calling an association on the resource to populate an attribute which caused N+1 queries.
Using records_for_populate solved my problem.
The same example as @Ross-Hunter can be used:
class MyResource < JSONAPI::Resource
  def self.records_for_populate(options = {})
    super(options).includes(:comments)
  end
end

You can force an include in a resource by directly adding it to the include_directives. The include directives object can be found in the options of many resource methods such as find_fragments. For me, I put the include directives into the context in my base controller before any operation processing so I can access it anywhere. With that you can do this in your resource:
context[:include_directives].include_directives[:include_related][:some_entity] = { include_related: {} }


fix NaN and 0.33 stock buy bugj

closes #83
closes #82

Codecov Report

Merging #84 into master will decrease coverage by 0.11%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #84      +/-   ##
==========================================
- Coverage   44.29%   44.18%   -0.12%
==========================================
  Files          57       57
  Lines         797      799       +2
  Branches       86       88       +2
==========================================
  Hits          353      353
- Misses        433      435       +2
  Partials       11       11

Impacted Files
Coverage Δ

src/views/Market/StockCard/BuyOrSellView.tsx
41.17% <ø> (ø)
:arrow_up:

src/views/Market/Market.tsx
52.38% <0%> (-5.52%)
:arrow_down:

Continue to review full report at Codecov.

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


Adaptation for 6 cameras e-CAM30_HEXCUTX2

Hello,
I have a six camera e-CAM30_HEXCUTX2 and I want to use opencv for some computer vision work. Is there any plan to adapt the code for a 6 camera synchronous acquisition?
cheers

Hi,
I have a four camera e-CAM20_CUXVR synchronized multiple camera solution and just like @Poutshi I was planning on doing computer vision work on the synchronized camera streams.
I landed here while trying to figure out how to bring camera frames into OpenCV Mat format.
How can I configure the cameras in the master-slave synchronized mode prior to opening the cameras with VideoCapture and starting to grab (synchronized) frames?
Thanks in advance!


Crash when using SwiftPM

I recently experienced a crash.
I installed it using SwiftPM but it crashed. It's release build, not debug build.
Here's the crashing part:
func createLayout <T> (layout: T.Type, options: PagingOptions)-> T where T: PagingLayout {
   return layout.init (options: options)
}

To test this, I ran a manual / Cocoapods installation. But there was no problem.
So I reinstalled with Cocoapods.
Some other libraries(e.g. SwiftyUserDefaults)  had same issue. Using generic seems to be the problem.

Experiencing the same problem. It looks like the problem is in SPM itself

You are right. From my testing, I'm having trouble with libraries that create instances using Generics in SwiftPM. As XCode is updated, the libraries generated by SwiftPM seem to be a problem. I had to move all the libraries back to Cocoapods.

Experiencing this Crash on Release Build too, also when using another library named Moya. I’ve found no way but to switch back to Cocoapods.

Please refer to the following link:
https://bugs.swift.org/browse/SR-11564?focusedCommentId=51285&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-51285

Thanks @ymkim50! Closing this issue as it’s not really much we can do about it.


JSLint using husky

This code was part of the https://github.com/do-me/SemanticFinder/pull/26 PR, but I'm splitting it out to reduce the size of each PR.
The goal is to keep code consistent between devs in terms of coding conventions and styling.

Defined a basic .eslintrc file, extending the eslint standard
Used husky ton run eslint --fix before each commit, fixing all basic issues such as indentation, unused imports etc.
Commits will fail if there's a deeper error, like inconsistent variable naming.

Thanks @lizozom that's a good idea.
I just checked the splitting functions where regex are used. It seems that the linter removed the backslashes. Could you maybe have a look at e.g. at the splitBySentences function and check how to avoid unwanted modifications in the future?

Yes, those seem unnecessary for the regex to work
https://regex101.com/r/26omjP/1

Great, thanks for checking!

I just noticed that there is something failing during build:
Run npm run build
  npm run build
  shell: /usr/bin/bash -e {0}

><EMAIL_ADDRESS>build /home/runner/work/SemanticFinder/SemanticFinder
> webpack --config webpack.config.js

[webpack-cli] Failed to load '/home/runner/work/SemanticFinder/SemanticFinder/webpack.config.js' config
[webpack-cli] Error: Cannot find module 'ajv/dist/compile/codegen'
Require stack:
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/definitions/typeof.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/keywords/typeof.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/keywords/index.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/index.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/schema-utils/dist/validate.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/schema-utils/dist/index.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/mini-css-extract-plugin/dist/index.js
- /home/runner/work/SemanticFinder/SemanticFinder/webpack.config.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/lib/webpack-cli.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/lib/bootstrap.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/bin/cli.js
- /home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack/bin/webpack.js
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:931:15)
    at Function.Module._load (internal/modules/cjs/loader.js:774:[2](https://github.com/do-me/SemanticFinder/actions/runs/5653876044/job/15315817765#step:5:2)7)
    at Module.require (internal/modules/cjs/loader.js:100[3](https://github.com/do-me/SemanticFinder/actions/runs/5653876044/job/15315817765#step:5:3):19)
    at require (internal/modules/cjs/helpers.js:107:18)
    at Object.<anonymous> (/home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/definitions/typeof.js:3:19)
    at Module._compile (internal/modules/cjs/loader.js:111[4](https://github.com/do-me/SemanticFinder/actions/runs/5653876044/job/15315817765#step:5:5):14)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1143:10)
    at Module.load (internal/modules/cjs/loader.js:979:32)
    at Function.Module._load (internal/modules/cjs/loader.js:819:12)
    at Module.require (internal/modules/cjs/loader.js:1003:19) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/definitions/typeof.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/keywords/typeof.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/keywords/index.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/ajv-keywords/dist/index.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/schema-utils/dist/validate.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/schema-utils/dist/index.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/mini-css-extract-plugin/dist/index.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/webpack.config.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/lib/webpack-cli.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/lib/bootstrap.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack-cli/bin/cli.js',
    '/home/runner/work/SemanticFinder/SemanticFinder/node_modules/webpack/bin/webpack.js'
  ]
}
npm ERR! code ELIFECYCLE
npm ERR! errno 2
npm ERR<EMAIL_ADDRESS>build: `webpack --config webpack.config.js`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/runner/.npm/_logs/2023-07-2[5](https://github.com/do-me/SemanticFinder/actions/runs/5653876044/job/15315817765#step:5:6)T07_24_47_40[6](https://github.com/do-me/SemanticFinder/actions/runs/5653876044/job/15315817765#step:5:7)Z-debug.log
Error: Process completed with exit code 2.

It seems like it's ajv codegen's fault and that there are some TS files. Do you know where the error could originate from @lizozom?

Unfortunately due to this bug none of the subsequent builds are working. To proceed for the time being I've opened a PR reverting Husky so that we can build.
https://github.com/do-me/SemanticFinder/pull/30


Box for show item in stash is off after new update

🐞 bug report
Describe the bug

When highlighting an item in stash that a buyer has whispered you for, the alignment of that box to show the item is now off after the stash affinity update yesterday.
To Reproduce

Buyer whispers for item
Click the eye button on the trade interface to highlight item
Box is not aligned properly

Expected behavior

Box is aligned properly
Screenshots

🌍  Your Environment

Version:  1.0.10
PoE:  Steam 3.12.4d English

I have similar problem. It is happen inside quad tab. I use QHD resolution (2560 x 1440).


Create user docs on how to use Git trailers for patch metadata

This is to document the changes introduced in packit/packit#1336.
Explain how Git-trailers can be used to influence how patches are generated and added to the spec-file when updating a dist-git repo from a source-git repo.
Explain how this patch-metadata is used during initializing a source-git repo, so that when the same content is synced back produces no or minimal changes in dist-git.

as part of this, we could also update the packit source-git init document

as part of this, we could also update the packit source-git init document

see #322

as part of this, we could also update the packit source-git init document

see #322

sorry, missed that one


v4.0 DRAFT mentions both !>= and !=> version_affected operators

I noticed that the version_affected section in the v4.0 DRAFT CVE JSON Schema document mentions both the !=> but lists the !>= operator in the table. Additionally, CVE files contain the !=> operator (ex: 2018/6xxx/CVE-2018-6339.json), while others contain the !>= operator (ex: 2019/1xxx/CVE-2019-1573.json. Is this a typo, or is !=> not fully documented?

!=> is a typo - will fix it.


update "Why CodeIgniter?" section icons

https://github.com/codeigniter4projects/website/issues/158

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
243

Relevant Lines:
320

💛  - Coveralls

cc @lonnieezell


Added mbedtls_net_close

When building a forking server the parent process should close the client socket and continue accepting, while the child process should close the listening socket and handle the client socket.
Overwriting the file descriptor does not close it, so mbedtls_net_init is not enough.
mbedtls_net_shutdown will close down the socket for writes which also affects the child process so this cannot be used either.
The forking server simply overwrites the socket file descriptor in userspace which does not disassociate the socket from the process as can be seen here:
$ ./ssl_fork_server >/dev/null &
[1] 25071
$ for i in {1..10}; do timeout .1 nc localhost 4433; done
$ ls -l /proc/$(pidof ssl_fork_server)/fd | grep socket
lrwx------ 1 user user 64 Aug 23 10:53 10 -> socket:[69247]
lrwx------ 1 user user 64 Aug 23 10:53 11 -> socket:[69248]
lrwx------ 1 user user 64 Aug 23 10:53 12 -> socket:[69249]
lrwx------ 1 user user 64 Aug 23 10:53 13 -> socket:[69250]
lrwx------ 1 user user 64 Aug 23 10:53 3 -> socket:[69233]
lrwx------ 1 user user 64 Aug 23 10:53 4 -> socket:[69234]
lrwx------ 1 user user 64 Aug 23 10:53 5 -> socket:[69240]
lrwx------ 1 user user 64 Aug 23 10:53 6 -> socket:[69241]
lrwx------ 1 user user 64 Aug 23 10:53 7 -> socket:[69243]
lrwx------ 1 user user 64 Aug 23 10:53 8 -> socket:[69244]
lrwx------ 1 user user 64 Aug 23 10:53 9 -> socket:[69245]
$

This is bad. When connecting the child process will inherit all these file descriptors and can thus communicate with all clients as if it was the server. All it takes is a RCE vulnerability and some socket finding shellcode.
After having applied this PR you get this result:
$ ./ssl_fork_server >/dev/null &
[1] 25192
$ for i in {1..10}; do timeout .1 nc localhost 4433; done
$ ls -l /proc/$(pidof ssl_fork_server)/fd | grep socket
lrwx------ 1 user user 64 Aug 23 10:54 3 -> socket:[71874]
$

I've added a ChangeLog entry for you in a new commit, to credit your contribution. Feel free to modify the commit to your liking.

Note: The incentive for this PR is in this question

This PR has been fully approved, so removing the "needs review" label


Edits to the Results section

Note to be careful using rich diff for this since the figure legends don't display in rich diff.

This looks good to me, especially the figure captions. Thanks!


Don't trigger event for hidden elements

Passes all tests
[ref] #6

@tiff What about merging this PR? I would really appreciate this one.

this would not trigger the inview event for elements that have zero height (since :visible is based on that). not sure if I want this :)