DE ELR Issue- Ordering Provider missing- resend from 7/12 to date

We are receiving ELRs from Milwaukee Brewers (MSH3/4 listed below) through Prime. The ELRs have no ordering Provider. Please correct this and resend all messages from 7/12/2022 to date.
Milwaukee Brewers LP^52D2188931^CLIA-CDC PRIME - Atlanta,^2.16.840.1.114<IP_ADDRESS>821^ISO

Any message_id for this one, or a report_id?

Emailed user to see if they could provide the messageids.

Thanks @sliu1000 Let us know when you receive the message ID.  I am going to move this to the icebox until we have that.

`ef86038c-d920-4164-baa9-4c2f438b0b1b
f8626cda-0183-4dec-9ac0-cc232c7bf1c7`

Per User, please see the message ids:
ef86038c-d920-4164-baa9-4c2f438b0b1b
f8626cda-0183-4dec-9ac0-cc232c7bf1c7
The user is requesting to correct and resend all messages from 7/12/2022 to date.

I'm on it, I'll have an update soon.

Update:
imagemover schema blanks out ORC-12-2 and ORC-12-3. OTC data was being submitted with the patient's name under those fields so imagemover requested for those fields to be blanked out in order to prevent a PHI/PII leak.

The issue with this is that test results without ORC-12-2 and ORC-12-3 fail DE's system. Due to these findings, we should reachout to both parties and make them aware of the situation.

@TaneaY Can you please reach out to the the relevant parties as Jorge described above?

Update @jorg3lopez and I met and here's next steps:

Sarah will email ImageMover to ask if they've fixed the PII concern with these two fields
Checking with Joel about other OTC senders and RADX MARS for these fields
Once we have those answers, we'll reach back out to DE to discuss their requirements on these fields.

@BerniXiongA6  BX to help contact ImageMover and DE

@sarahnw to review this before taking action, @BerniXiongA6 can backup if needed - cc: @jorg3lopez

Hi @sliu1000 could you send me any emails you have regarding this ticket? I don't think I've been on the email chains and I'd like to follow up on it in Sarah's place. I'll need to reach out to ImageMover and Delaware, but in the meantime would like to see any emails you might have on this before I do so. Thanks! cc: @jorg3lopez

@BerniXiongA6 Hi Berni - no I have not received any other emails from DE. The shared inbox should have all the communications under SLTL Support/Delaware/Support Queue

Thanks @sliu1000 for clarifying this. I’ll follow up and cc: RS shared email and @jorg3lopez when I do.

BX sent reply to Chris on 9/2/2022, waiting to hear back from them:

Hi Chris,
I'm the new Delivery Manager for ReportStream's Onboarding and Operations team. I'm hoping you can help me out with an open issue concerning a Delaware Dept. of Health ELR issue.
In July, they received ELRs from Milwaukee Brewers, which have no ordering Provider. Our engineer, Jorge, investigated this to determine that the ImageMover schema blanks out ORC-12-2 and ORC-12-3 in order to prevent a PHI/PII leak since OTC data is submitted with the patient's name under those fields. The issue with this is that test results without ORC-12-2 and ORC-12-3 fail Delaware's system. We wanted to make you aware of the situation and to find out whether there has since been a fix on your end? We're happy to reach back out to Delaware with any updates or insight you're able to share with us.
These are the Message IDs:
ef86038c-d920-4164-baa9-4c2f438b0b1b
f8626cda-0183-4dec-9ac0-cc232c7bf1c7
Thank you so much.
Best,
Berni

From Chris Cash -<EMAIL_ADDRESS>9/12 at 11:30am
Hi Berni,
Sorry for the late reply (I was out last week) - we're sending these as the orderingProviderLname and orderingProviderFname fields; are these blank for these records? If they are blank, is there any data populated for the orderingProviderNpi field?
Thanks,
Chris

@jorg3lopez Please see email from Chris Cash below.
Hi Jorge,
Yes, those fields are good to go now (no PHI, healthcare provider/physician's information when it's available; for OTC results these fields are blank); it was addressed on our side in November last year, so anything sent from December 2021 on should have non-PHI data for those fields.
Hope that helps,
Chris

@jorg3lopez @BerniXiongA6 create new ticket "allow for ORC-12-2 and ORC-12-3 to not be blank in Imagemover's schema" -- can close this ticket.


Expulsion Pylons keep the chunks loaded after removal

Modpack: ATM8 1.0.15
Mod version: 1.19.2-3.1.0

What happened:
I was experimenting with the player pylon, placed it, changed settings, had my friend enter the chunk and be teleported to bed. At the end I broke the pylon.
Expected: The chunk with the pylon unloads when I leave
Actual: The chunk was kept loaded (and I was wondering why). I had to move to another chunk nearby that unloaded properly.
Please verify.
I don't at all understand why this mod needs forced chunks? Even with the 5x5 expulsion range, Minecraft's default minimum view-distance of 4 should still be a valid compromise of losing out on 1 chunk of range in the edge case, yet without the forced loading.

Expulsion pylons do not force load chunks at all. The linked code is used for the infusion pylon: https://github.com/jeremiahwinsley/pylons/blob/813c8a22466b637cc2df00fd31a42f0aafd962d1/src/main/java/net/permutated/pylons/tile/InfusionPylonTile.java#L37
Are you using crash utilities to see the loaded chunks, or something else? Which ticket and load level are you getting, if you have that information?

Hm I missed that (didn't load in IDE), sorry.
This was on a public server so my options were very limited. The two chunks where we placed the pylons had CC: Tweaked computers that I expected to unload together with the chunks for testing. To figure out the point of unloads I let the computers write current os time to a monitor and output a redstone signal through a wireless transmitter/receiver pair (I was watching on the other end). Neither computer was getting unloaded after 10-20 minutes so I assumed it was the pylons...
I will close for now and check later on a fresh instance.


cannot read property 'hide' of undefined

Uncaught exception on Windows

How did adding an && fix the issue


create a support model info panel

An easily accessible panel with links to docs & community so users can find help quickly.

Added ... moving to pull-request #126


Incorrect parsing of comment lines following a line continuation

According to the newly released requirements file format spec:

A line that begins with # is treated as a comment and ignored.
A line ending in an unescaped \ is treated as a line continuation and the newline following it is effectively ignored.
Comments are stripped before line continuations are processed.

If a line with a line continuation is followed immediately by a comment line, then the comment line will get ignored and the following non-comment line will be appended erroneously onto the previous line.
E.g.
joblib==0.14.1 \
    --hash=sha256:0630eea4f5664c463f23fbf5dcfc54a2bc6168902719fa8e19daf033022786c8 \
    --hash=sha256:bdb4fd9b72915ffb49fde2229ce482dd7ae79d842ed8c2b4c932441495af1403 \
    # via scikit-learn
numpy==1.18.1 \
    --hash=sha256:1786a08236f2c92ae0e70423c45e1e62788ed33028f94ca99c4df03f5be6b3c6

is incorrectly converted into
joblib==0.14.1 --hash=sha256:0630eea4f5664c463f23fbf5dcfc54a2bc6168902719fa8e19daf033022786c8 --hash=sha256:bdb4fd9b72915ffb49fde2229ce482dd7ae79d842ed8c2b4c932441495af1403 numpy==1.18.1 --hash=sha256:1786a08236f2c92ae0e70423c45e1e62788ed33028f94ca99c4df03f5be6b3c6

Comments are stripped before line continuations are processed.

Are you sure that your example is not erroneous? Because according to the order of actions, if comment lines are stripped before line continuations, then hanging line continuation will join next line.... Current joinlines() implementation also does not yield line if line continuation is not followed by \n at the end of file. I thought that such cases are not correct requirements.txt :/

I think that even PIP does not parse the requirements.txt file according to spec. If you mix line continuations and comment lines, first appearing comment line stops pip from reading next line e.g. --hash option which is under the comment line....
What a mess :D


Use within uib-tabset from AngularUI

First off, love the directive! Trying to use it inside of 'uib-tabset' and it appears to break. For example:
<select chosen class="form-control" data-placeholder="timezone" ng-model="student.timezone" ng-options="name for name in timezones" ng-change="update()">
    <option value=""></option>
</select>
<uib-tabset>
    <uib-tab heading="Timezones> Timezone content </uib-tab>
</uib-tabset>

This shows the angular-chosen select perfect and has a tabset beneath it. However, if I move the chosen into the tab content, the chosen stops working (ie, it just shows an empty select). Like so:
<uib-tabset>
    <uib-tab heading="Timezones> Timezone content
        <select chosen class="form-control" data-placeholder="timezone" ng-model="student.timezone" ng-options="name for name in timezones" ng-change="update()">
            <option value=""></option>
        </select>
    </uib-tab>
</uib-tabset>

At first I thought it might not like nested directives, but I have several other directives that work fine within uib-tabset. If you had any thoughts, that'd be greatly appreciated and if you can point me in the right direction, I'm happy to take a crack at it myself.
Thanks!

Can you try this once
<uib-tabset>
    <uib-tab heading="Timezones> Timezone content

        <select chosen options="timezones"
                     class="form-control"
                     data-placeholder="timezone"
                     ng-model="student.timezone"
                     ng-options="name for name in timezones" ng-change="update()">
            <option value=""></option>
        </select>

    </uib-tab>
</uib-tabset>

options attribute was missing. This might have been the issue. Let me know, if it doesn't work.

Thanks for getting back to me! So adding the "options" value didn't solve the issue. I've done a lot of digging around and I think it might be tied to the 'restrict' value on the directive. It looks like when used in conjunction with just $scope it loses synch. I created a Plunkr to demonstrate the various $scope use cases:
http://plnkr.co/edit/qBjsswQimx9ebrfbp2hv?p=preview
Perhaps I should only use it with named controllers or $scope objects?

I think you are right. Using it without a named controller or without wrapping it around a $scope object seems to surface this issue.
I'm guessing, when we set the value in the outer ng-model="timezone" the one inside uibTabset is just referring to the value in the parent scope. Once it is set, the inner scope creates a timezone property of its own and is detached from the parent scopes' timezone property.
Maybe primitives are passed by value to the child scope. (Are string primitives, immutable :confounded: ?)
As of now, use a named controller or $scope object.
I will dig in and figure out if it's an issue with the directive or if it's just how Angular works.

This is how angular works apparently. Closing the issue.


Change Nightly to Beta

As by the time we run this Firefox 57 will be in Beta, we should change all mentions from Nightly to Beta including the links pointing to the download page.

@couci this will need your final decision

I think we need to stick with nightly, I understand the problem you're posing but right now the comms and the video is about nightly and we also want to increase the downloads of the program

Same issue, don't get it. The repo information says:
"This is the repo for the Firefox 57 sprint https://firefoxsprint.mozilla.community/"
Also multiple Events have Firefox 57 as the target of the Spring and they will be using the Firefox Nightly version that will be 58.

Hello,
yes you're right, nightly is now on 58 and beta is on 57 and our focus is 57. The reason we decided to go initially to Nightly was because we knew that the issues are going to be eventually uplifted. However, by asking the engineers we were told that nightly will be more unsustainable on 58 thus move to beta for better user experience.
@MichaelKohler for that reason I am reopening this one. Do you need to track the places we need change?

@MichaelKohler for that reason I am reopening this one. Do you need to track the places we need change?

As far as I know, this is only on the "Host" page for now.

This page is still suggesting Nightly: https://firefoxsprint.mozilla.community/participant/

You prefer to participate at the campaign from the comfort of your couch? That’s totally fine! Here is how:
Install Nightly for Desktop

This is done in the "newdesign" branch. I've also changed to the new video with the corrected phrase. This will be live once that branch is merged.


review: git change doesn't work on windows cmd.exe

using git bundled on msys2 on windows10 64bit.
git change failed because the git rev-parse doesn't work correctly on cmd.exe.
how to reproduce

install git bundled on msys2
goto some repository that already registered aliases to git change or others.
make new branch and modify some codes
git add and git change

This problem can be reproduced on cmd.exe. On msys2 shell, it doesn't. git change run git rev-parse --abbrev xxx@{u}. But this xxx@{u} is not passed into git command correctly.

This is cased of different behavior of parsing command-line arguments. As you know, on windows, the arguments are parsed from one-line string by the process it-self. So there are few implementations and behaviors.
args.c
#include <stdio.h>
int main(int argc, char *argv[]) {
  puts(argv[1]);
}

For example, if compiling code above with mingw native compiler, it works expectedly.
C:\>args @{u}
@{u}

And this works well on msys2 shell too. But if compiling this code with msys2 or cygwin compiler, it doesn't work correctly.
C:\>args @{u}
@u

I alredy confirmed this issue will be fixed with following patch.
https://go-review.googlesource.com/#/c/21018/2
BTW, please don't answer Use "bash on windows".

install git bundled on msys2
...
This problem can be reproduced on cmd.exe.

I would say, don't do it. You are looking for trouble. cmd.exe and bash are very different - it is difficult to please both at the same time.
Use standard git version that everyone else is using. If you want to use msys2, then stay inside msys2.
Alex


feature: conform-like floating command input

Did you check the docs?

[X] I have read all the snacks.nvim docs

Is your feature request related to a problem? Please describe.
Currently, notifer covers a lot of ground conform does. I'm using conform only for the floating command input.
Describe the solution you'd like
An option to get such input directly from snacks.
Describe alternatives you've considered
The alternative I considered and using is conform along with snacks.
Additional context
No response

conform is a formatter.
I assume you're referring to noice?
If so that is out of scope for now. Messing with ui_attach is a whole different beast.
And doing just the cmdline is unfortunately not possible. When you enable cmdline with ui_attach, you also need to implement ext_popupmenu and ext_messages.

conform is a formatter. I assume you're referring to noice? If so that is out of scope for now. Messing with ui_attach is a whole different beast. And doing just the cmdline is unfortunately not possible. When you enable cmdline with ui_attach, you also need to implement ext_popupmenu and ext_messages.

My bad. Noice.
Thank you for the clarification.


enh(assistant builder): use the callback pattern everywhere

Description
fixes https://github.com/dust-tt/tasks/issues/736
UI was updating state by spreading potentially stale state. We should always use setState callbacks when the new state is not fully replacing the old state, otherwise there are race conditions when calling the setState method in rapid successions (every setState is seeing the same stale state)
Risk

Deploy Plan

afk for lunch, will review asap when back at office


Adds pull-request command to create new pull

As a user, when I run the github-gem pull-request command, it creates a pull request for the current or specified branch.

LGTM

LGTM

lgtm


Testing BFCP.

Hi @alfredh ,
Would you please help me to test BFCP module?
Is there any documentation about test procedure?
Regards,
Ali

the BFCP module is not really complete. it is just a test module to test
the BFCP protocol and not very useful.
for example it does not do any floor request etc.
there is no documentation, sorry :)


Improve overriding dimens

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #1832
Description

Create new and rename properties to only handle the use case of that
property, so people can override the property without
affecting other unintended UI components
Remove un-used properties
Arrange properties alphabetically

Alternative(s) considered
N/A
Type
Feature
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

@jingtang10 this is ready for review

@jingtang10 I'll check all dimens and make sure people can override those safely

I'll resolve this once I'm back from exam leave.
Will return on April 17th.


add an option to limit the number of connections per port

Adds an environment variable to limit the number of forks socat creates at max and sets the default to 100.
Socat default is unlimited (risking pushing the server in a state where no threads can be started anymore).
Info @wt-io-it

And what happens if more than 100 are needed?

Also, any way to set to unlimited? That should be documented.

if more than 100 are needed one can set MAX_CONNECTIONS as mentioned in the README.md, unlimited is impossible as one system only can handle 32768 processes (per default and a few of those are already taken by the main soccat, docker and system processes).
However one can set it to 4194304 according to stackoverflow:
https://stackoverflow.com/questions/6294133/maximum-pid-in-linux
So unlimited parallel connections seems unreasonable to me.

I updated the documentation to contain as many details as seemed reasonable to me. Basically the behaviour depends on many things in the linux stack and it's pretty easy to kill your own dev system / bring your dev system into an unstable state when testing this things.
For future reference, to test the limits:

start a docker-whitelist that points to a firewalled port
run a script to hammer the ports (replace IP by the docker instance ip)

while [[ $(ps fax | wc -l) -lt 10000 ]] && bash -c 'echo "connect"; cat smartcat_regexes.txt | nc -v IP 514 &'; do echo "-"; done

if you are feeling lucky you can remove the connection process limit of 10000 or increase it (crashes the system or a few processes on the system most likely)

Unlimited subprocesses/connections is not a good idea therefore.

Thanks, I was trying to test it but it tells me smartcat_regexes.txt is empty. What am I supposed to write in that file?

i updated the original content of the comment, that was just a test file to not have an empty body in the the connect, a few kb should suffice, and please take care of your local process limit (not removing the 10000 check) i crashed my gnome session a few times while trying

Thanks, great addition. :blush:


ngx_pagespeed builds locally but not on PPA

I created a nginx-pagespeed package for my PPA and I hit something strange.
While debuild works fine on my local machine, on the PPA server it always fails with:
adding module in /«PKGBUILDDIR»/debian/modules/pagespeed
mod_pagespeed_dir=/«PKGBUILDDIR»/debian/modules/pagespeed/psol/include
build_from_source=false
checking for psol ... not found
./configure: error: module ngx_pagespeed requires the pagespeed optimization library.
Look in obj/autoconf.err for more details.
debian/rules:60: recipe for target 'override_dh_auto_configure' failed
make[1]: *** [override_dh_auto_configure] Error 1
make[1]: Leaving directory '/«PKGBUILDDIR»'
debian/rules:57: recipe for target 'build-arch' failed
make: *** [build-arch] Error 2
dpkg-buildpackage: error: debian/rules build-arch gave error exit status 2

but psol exists, otherwise local build wouldn't work (here output from local binary package build):
adding module in /home/claudio/nginx-1.10.1/debian/modules/pagespeed
mod_pagespeed_dir=/home/claudio/nginx-1.10.1/debian/modules/pagespeed/psol/include
build_from_source=false
checking for psol ... found
List of modules (in reverse order of applicability): ngx_http_write_filter_module ngx_http_header_filter_module ngx_http_chunked_filter_module ngx_http_v2_filter_module ngx_http_range_header_filter_module ngx_pagespeed_etag_filter ngx_http_gzip_filter_module ngx_pagespeed ngx_http_postpone_filter_module ngx_http_ssi_filter_module ngx_http_charset_filter_module ngx_http_xslt_filter_module ngx_http_image_filter_module ngx_http_sub_filter_module ngx_http_addition_filter_module ngx_http_gunzip_filter_module ngx_http_userid_filter_module ngx_http_headers_filter_module ngx_http_subs_filter_module
checking for psol-compiler-compat ... found
 + ngx_pagespeed was configured

I also verified that psol is included in the package sent to the PPA server with dput:
tar tf nginx-pagespeed_1.10.1-20161021.tar.xz
[...]
nginx-1.10.1/debian/modules/pagespeed/
nginx-1.10.1/debian/modules/pagespeed/test/
nginx-1.10.1/debian/modules/pagespeed/test/nginx_system_test.sh
nginx-1.10.1/debian/modules/pagespeed/test/run_tests.sh
nginx-1.10.1/debian/modules/pagespeed/test/valgrind.sup
nginx-1.10.1/debian/modules/pagespeed/test/pagespeed_test.conf.template
nginx-1.10.1/debian/modules/pagespeed/LICENSE
nginx-1.10.1/debian/modules/pagespeed/config
nginx-1.10.1/debian/modules/pagespeed/README.md
nginx-1.10.1/debian/modules/pagespeed/cpp_feature
nginx-1.10.1/debian/modules/pagespeed/config.make
nginx-1.10.1/debian/modules/pagespeed/psol/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Debug/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Debug/linux/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Debug/linux/x64/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Debug/linux/ia32/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/linux/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/linux/x64/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/linux/x64/pagespeed_js_minify
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/linux/ia32/
nginx-1.10.1/debian/modules/pagespeed/psol/lib/Release/linux/ia32/pagespeed_js_minify
nginx-1.10.1/debian/modules/pagespeed/psol/include/
nginx-1.10.1/debian/modules/pagespeed/psol/include/net/
[...]

Any ideas why this fails on PPA?

Any chances you could get your hands on that obj/autoconf.err file?
Basically it tried to link a test program against psol.a (actually can't see if you have that file or not due to the [...] --- do you?), and the obj/autoconf.err contains the error messages.

No, unfortunately not. I'd love to ;) However I think this is just a build-container starting up to build to package on the PPA server and then distroys itself. The only thing I see is the build log.
One way would be to make a "cat obj/autoconf.err" somewhere, maybe in the debian/rules file? But I'm not sure if that will work. Gotta try.
You can find the build log here, as long as I keep the PPA (it's just a test anyway): https://launchpadlibrarian.net/290276460/buildlog_ubuntu-xenial-amd64.nginx-pagespeed_1.10.1-20161021_BUILDING.txt.gz
Nope, "psol.a" does not exist. Just checked. Doesn't this come with the psol tar package? Do I have to download additional psol libraries/files?

I was now able to get my hands on "obj/autoconf.err" with the following trick in the pagespeed config:
if [ $ngx_found = no ]; then
  cat << END
$0: error: module ngx_pagespeed requires the pagespeed optimization library.
Look in obj/autoconf.err for more details.
$(pwd)
$(find . -name 'autoconf.err' -exec cat {} +)
END

Here's the relevant part of autoconf.err:
----------------------------------------
checking for psol

cc: error: /«PKGBUILDDIR»/debian/modules/pagespeed/psol/lib/Debug/linux/x64/pagespeed_automatic.a: No such file or directory

That file exists in my local source folder, but it seems it was not packed in the tar.xz file! No wonder it doesn't turn up on the PPA server. I know now this is not a pagespeed fault, but any idea why this file was not packaged by debuild?

Sorry, correction: pagespeed_automatic.a
(something like: psol/lib/Release/linux/x64/pagespeed_automatic.a)
I don't know which files you can easily edit based on how it's setup. Our
"config" file would be a potential spot, since it's what produces that
error message at this fragment:
if [ $ngx_found = no ]; then
cat << END
$0: error: module ngx_pagespeed requires the pagespeed optimization library.
Look in $PWD/$NGX_AUTOCONF_ERR for more details.
END
exit 1
fi
On Fri, Oct 21, 2016 at 9:54 AM, Claudio Kuenzler<EMAIL_ADDRESS>wrote:

No, unfortunately not. I'd love to ;) However I think this is just a
build-container starting up to build to package on the PPA server and then
distroys itself. The only thing I see is the build log.
One way would be to make a "cat obj/autoconf.err" somewhere, maybe in the
debian/rules file? But I'm not sure if that will work. Gotta try.
You can find the build log here, as long as I keep the PPA (it's just a
test anyway): https://launchpadlibrarian.net/290276460/buildlog_ubuntu-
xenial-amd64.nginx-pagespeed_1.10.1-20161021_BUILDING.txt.gz
Nope, "psol.a" does not exist. Just checked. Doesn't this come with the
psol tar package? Do I have to download additional psol libraries/files?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/ngx_pagespeed/issues/1301#issuecomment-255383153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADl1RNH0cZDp4G3LAv7fUX6ZKb_nEOuZks5q2MQTgaJpZM4Kc5VL
.

I think that's talking about the final package... or I guess they expect
the inputs to be either tarballs or other debian packages?
You could just build the entire thing from source, although that's a bit of
a waste of a CPU (and a lot more steps), but it is kinda nice to use the
system compiler for everything.
On Fri, Oct 21, 2016 at 2:45 PM, Claudio Kuenzler<EMAIL_ADDRESS>wrote:

I'm closing this now as it can be read in the Debian packaging guidelines,
that such static libraries (.a files) need to be created in a separate -dev
package. Bummer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/ngx_pagespeed/issues/1301#issuecomment-255440206,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADl1RHy99pzExo1-8gSG4xbQ4W3fYLWqks5q2QhQgaJpZM4Kc5VL
.


None of the hashes fuzztests succeed

We have fuzztests which try to produce a hash two ways and check that they're the same. Because our cfg(fuzzing) easily-broken hashes are implemented inconsistently, they don't pass.
RUSTFLAGS=--cfg=fuzzing cargo test

fails right now in hashes/fuzz/.

Is the solution to check each failing test and see if it makes sense to run it when fuzzing, then just conditionally compile in the test only when not fuzzing? For example sha256::tests::midstate seems not useful when fuzzing. Or do I need to further understand what the fuzz tests do and what the hobbled code does when fuzzing?

Now that #1821 merged exactly which cargo incantations should pass?

In rust-bitcoin/hashes: RUSTFLAGS=--cfg=fuzzing cargo test -> PASS
In rust-bitcoin/bitcoin: RUSTFLAGS=--cfg=fuzzing cargo test -> 12 tests FAIL
In rust-bitcoin/: RUSTFLAGS='--cfg=bitcoin_fuzz --cfg=hashes_fuzz --cfg=secp256k1_fuzz' cargo test -> 85 tests FAIL

This looks like its fixed,

cd fuzz;  RUSTFLAGS=--cfg=fuzzing cargo test - all pass
cd bitcoin; RUSTFLAGS=--cfg=fuzzing cargo test - all pass
cd hashes; RUSTFLAGS=--cfg=fuzzing cargo test - all pass

Yeah, I seem to remember fixing this but I don't recall when exactly. The old fuzztests would compare against RustCrypto and I think we dropped those. Not sure if we replaced them with anything.
Ultimately there is very little value in fuzzing the hash crate at all. It doesn't allocate memory or index into arrays in complicated ways, and if the hashing code is wrong it will be wrong 100% of the time, so even a single unit test is enough.
Since the tests currently pass I'm happy to close this.


Improve merge performance (Alternative)

Aims to improve: https://github.com/CycloneDX/cyclonedx-dotnet-library/issues/287
Alternative to https://github.com/CycloneDX/cyclonedx-dotnet-library/pull/300

Closing as https://github.com/CycloneDX/cyclonedx-dotnet-library/pull/300 has been merged.


unlock win 10

[Enter feedback here]

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

ID: 05519993-ce58-9656-a7fa-fced61dda068
Version Independent ID: 5186f2a1-4214-e419-b0e5-33b618afcbdb
Content: 4801(S) The workstation was unlocked. (Windows 10)
Content Source: windows/security/threat-protection/auditing/event-4801.md
Product: w10
GitHub Login: @Mir0sh
Microsoft Alias: justinha

@splendide2018 : you may want to add a description in your issue text if you want this issue kept open. Otherwise, it looks like it should be closed due to no issue or request reported.


how to modify code to support image_size>1 when training?

Hi, right now the ProposalTarget custom operator only supports single image each GPU, so that's where you might want to start.

I'm trying to modify op, thanks!

No bother, good luck!


[2.1.1] Bump package versions

I'll merge this PR once release/2.1 branches are open for 2.1.1 changes. Until then, leaving this PR open as "SHP: Approval pending".

@dotnet-bot test Windows Release x64 Build


Suggested Q&A entry: why not Reflect.*?

Maybe someone can clarify the purpose of Reflect to me because my understanding is that it was more a place to intercept implicit operations (defineProperty, setPrototypeOf) or create functions for syntax features (new or in). I don't think has / hasOwnProperty fits within that.

Reflect's purpose is indeed only to contain, 1:1, a method for each Proxy trap. Reflect.hasOwnProperty and Object.has are already identical, so adding it to Reflect wouldn't make much sense.

Reflect.hasOwnProperty and Object.has are already identical, so adding it to Reflect wouldn't make much sense.

Reflect.has already exists, and it is the same as in operator (own and inherited properties). Reflect.hasOwnProperty also exists, and it's inherited from Object.prototype, so it has the same requirement to .call.

ahhh yes, thanks.


⚠ SceneTime has degraded performance

In 6966747, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1134 ms

Resolved: SceneTime performance has improved in 35ef847 after 6 minutes.


Bug in Mac/Mamp

Hello,
I worked with archlinux, php7 ...etc it worked fine now I'm with:
Os : Mac sierra
MAMP : Php 7.1
when I run in console : php start.php I get nothing, I tried in browser I get : Fatal error: Uncaught League\Flysystem\Exception: Impossible to create the root directory "". in /Users/mac/code/laracasts-downloader/vendor/league/flysystem/src/Adapter/Local.php:106 Stack trace: #0 /Users/mac/code/laracasts-downloader/vendor/league/flysystem/src/Adapter/Local.php(78): League\Flysystem\Adapter\Local->ensureDirectory(false) #1 /Users/mac/Code/laracasts-downloader/start.php(14): League\Flysystem\Adapter\Local->__construct(false) #2 {main} thrown in /Users/mac/code/laracasts-downloader/vendor/league/flysystem/src/Adapter/Local.php on line 106
Why this issue ?

Hi @kossa,
After reading the exception, it seems a directory problem. Are the folders with the right write permissions?
Do you have valid paths in your .env file?

The path is correct look :

My bad I found the issue, PATH and not PAT
Sorry :(

No problem 😃


Usage with NodeJS Server

Please help me with the steps to intregate this with node webapp.
I'm getting this error when I use pkg .
:Error! Property 'bin' does not exist in /home/lt-101/16may/package.json

Add bin property to your package.json and make it's value point to your starting javascript file.

It worked and created following files

16may-linux
16may-win.exe
server-linux
server-win.exe

How do I run it?

16may-win.exe was executed on windows and worked fine.
Then a question again what is the use of  server-win.exe file
LINUX executing is still indark I dont know how to use it.

Please erase all 4 files and try running pkg again. I am sure it creates only 16may-linux and 16may-win.exe.


ci(docs): :ferris_wheel: fix workflow about type doc

:tada: This PR is included in version 1.0.3 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add upgrade note about having to remove mailer require

Adds upgrade note about removing mailer require which was causing uninitialized constant Devise::Mailer error.
Fix #34

Thank you for your contribution and apologies for the oversight!


Remove akka agent from MetricClock/MetricSystem

See https://github.com/tumblr/colossus/issues/529 for dan's approach


Walking in other direction moves player

When switching directions the player will change position, most likely the pivot point is not in the center of the player?

That would explain another hack I put in!

Fixed and deployed.


Changing route when in word model view broken

What went wrong?
The word model component used to have some logic to handle corpus changes, which did not play well with #1238 . So now, when switching corpora or logging out, the user gets to see a blank screen, instead of being rerouted.
What did you expect to happen?
When clicking about / manual etc., you would expect to be taken to that page. Instead, only a second click will trigger correct rerouting.
Screenshot
No response
Where did you find the bug?

[ ] https://ianalyzer.hum.uu.nl
[ ] https://peopleandparliament.hum.uu.nl
[ ] https://peace.sites.uu.nl
[ ] a server hosted elsewhere (i.e. not by the research software lab)
[ ] a local server

Version
5.2.0
Steps to reproduce
Choose a corpus with word models, then try to navigate to another part of the application. The only switching that works immediately is between search/ word model / info views.

It seems that in the current branch, this issue is fixed - switching corpora from navbar is not possible anymore, and switching from word models to manual works again.


SMTP settings for Office365 returns authentication unsuccessful

Description
SMTP configuration always seems to fail when using the same rails settings that are successful in Gitlab. Office365 does seem to be a little more sensitive than, for example, Gmail: it fails to authenticate if the sender and account addresses are different. Note that Gitlab is deployed with  their official docker image (Ubuntu). All other features in Portus appear to work.
Steps to reproduce
With the following in portus/config.yml:
email:
  from<EMAIL_ADDRESS>  name<EMAIL_ADDRESS>  reply_to<EMAIL_ADDRESS>  smtp:
    enabled: true
    address: smtp.office365.com
    port: 587
    domain: example.co.uk
    user_name<EMAIL_ADDRESS>    password: <PASSWORD>
    authentication: login
    enable_starttls_auto: true

...and the following result from rake portus:info:
email:
  from<EMAIL_ADDRESS>  name<EMAIL_ADDRESS>  reply_to<EMAIL_ADDRESS>  smtp:
    enabled: true
    address: smtp.office365.com
    port: 587
    domain: example.co.uk
    ssl_tls: ''
    enable_starttls_auto: true
    openssl_verify_mode: none
    ca_path: ''
    ca_file: ''
    user_name<EMAIL_ADDRESS>    password: "****"
    authentication: login

Expected behavior: Receive email from<EMAIL_ADDRESS>Actual behavior:

portus_1      | Processing by PasswordsController#create as HTML
portus_1      |   Parameters: {"utf8"=>"✓", "authenticity_token"=>"abcd1234=="<EMAIL_ADDRESS>"button"=>""}
portus_1      |   Rendered vendor/bundle/ruby/2.5.0/gems/devise-4.4.0/app/views/devise/mailer/reset_password_instructions.html.erb (0.5ms)
portus_1      |
portus_1      | Sent mail to<EMAIL_ADDRESS>(5698.0ms)
portus_1      | [Mailer] 535 5.7.3 Authentication unsuccessful [LO2P265CA0453.GBRP265.PROD.OUTLOOK.COM]
portus_1      | :
portus_1      | Redirected to https://docker.example.co.uk/users/password/new
portus_1      | Completed 302 Found in 5990ms (ActiveRecord: 2.3ms)

Deployment information
Deployment method: Using modified docker-compose example with Clair.
Portus version: 2.4.3@5a616c0ef860567df5700708256f42505cdb9952

Currently, the only difference I can see is that Gitlab use the Mail:Address class and its format method to return the sender string: app/mailers/base_mailer.rb
On the other hand, Protus appears to use a hard-coded format: app/mailers/devise_mailer.rb
I can't currently confirm whether this is the issue, but given Office365 is known to have issues with mis-matching sender and account addresses, this might be somewhere to start. I should also note that the above authentication unsuccessful message is returned when changing the from address in other apps like Gitlab while the other settings are otherwise correct.


Fix: Race Condition resolved with I18n files on server starts.

Fixes #1585
What does this PR do?
This PR fixes issue #1585 to resolve the race condition between file reset and configuration of i18n files.
Steps taken:
Two asynchronous functions configureI18n() and reloadI18nFiles() have been added to ensure that the file copying process finishes before configuring I18n.

@J12934
Thanks for reviewing..
I'll make the necessary changes and commit here..

@bkimminich
I'll go through the code again and do the needful.
Thanks.

@bkimminich
Can you tell me what purpose does that portion of code serve?
I think we'll have to wrap that up in a separate async function so that it occurs after i18n is configured.
So to name the function, I'll need the purpose.

That code is part of the i18n module, not of Juice Shop. We just use this as a feature. I guess wrapping the configuration of i18n into async had some side effect with Express. But your guess is as good as mine here.

@J12934
I've changed the function name and its nature as instructed.
@bkimminich
As for the approach, I tried using then() and it works perfectly.
I ran the tests locally and they passed.
So is this fine now?

Just tried it locally and so far works fine. I've merged latest develop back into your fork and will merge the PR if CI/CD passed. Thanks a lot!
PS: If you send me your post address I'll send you a few stickers as a thank you! for your first merged PR!

You can write me an email or DM me on Slack or Gitter, sure!

Great!
Thanks a lot..
I'll analyse the idea a bit more and reach out to you..


Propagate --target flag from cargo test invocation

Closes #61.
Tested with cargo test --target x86_64-unknown-linux-gnu inside a crate that has [build] target = "thumbv7m-none-eabi" in .cargo/config.

Thank you, this was the last bit that I need to get trybuild working in lpc8xx-hal!


Can't deploy MyWardData: Unexpected Response, [Help 1]

I'm getting the below error trying to deploy MyWardData.  It builds fine.

Deploying adapter to runtime mfp on https://mobilefoundation-myward2e-server.mybluemix.net:443/mfpadmin...
[INFO] Scanning for projects...
[WARNING]
[WARNING] Some problems were encountered while building the effective model for com.sample:MyWardData:adapter:1.0-SNAPSHOT
[WARNING] 'build.plugins.plugin.version' for com.ibm.mfp:adapter-maven-plugin is missing. @ line 68, column 12
[WARNING]
[WARNING] It is highly recommended to fix these problems because they threaten the stability of your build.
[WARNING]
[WARNING] For this reason, future Maven versions might no longer support building such malformed projects.
[WARNING]
[INFO]
[INFO] -----------------------< com.sample:MyWardData >------------------------
[INFO] Building MyWardData 1.0-SNAPSHOT
[INFO] ------------------------------[ adapter ]-------------------------------
[INFO]
[INFO] --- adapter-maven-plugin:8.0.2018082311:deploy (default-cli) @ MyWardData ---
[ERROR] logger.changeStateDeploymentException: FWLSE0321: State change failure. Caused by: java.lang.RuntimeException: User code thrown exception: java.lang.Exception: Unable to connect to Cloudant DB, check the configuration.  at com.sample.CloudantJavaApplication.init(CloudantJavaApplication.java:58)  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)  at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)  at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)  at java.lang.reflect.Method.invoke(Unknown Source)  at
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 17.424 s
[INFO] Finished at: 2018-09-29T10:31:51-04:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.ibm.mfp:adapter-maven-plugin:8.0.2018082311:deploy (default-cli) on project MyWardData: Unexpected response from https://mobilefoundation-myward2e-server.mybluemix.net:443/mfpadmin/management-apis/2.0/runtimes/mfp/adapters -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException
Error deploying adapter
undefined
Error: An error occurred during an attempt to deploy the adapter. See the preceding messages for details.

@shivahr FY


[BUG] [VSCode Extension] Custom Connections in AML Workspace Do Not Show Up in Drop Down

Describe the bug
After setting pf config set connection.provider="azureml:/subscriptions/<your-subscription>/resourceGroups/<your-resourcegroup>/providers/Microsoft.MachineLearningServices/workspaces/<your-workspace>", connection dropdown list shows nothing:

How To Reproduce the bug
Steps to reproduce the behavior, how frequent can you experience the bug:

Create a custom connection in AML workspace
Configure provider to AML workspace
Create an LLM node, connection dropdown list shows nothing.

Screenshots

On the VSCode primary side bar > the Prompt flow pane > quick access section. Find the "install dependencies" action. Please it and attach the screenshots there.
Please provide other snapshots about the key steps to repro the issue.

Environment Information

Promptflow Package Version using pf -v: [e.g. 0.0.102309906]
Operating System: [e.g. Ubuntu 20.04, Windows 11]
Python Version using python --version: [e.g. python==3.10.12]
VS Code version.
Prompt Flow extension version.
On the VS Code bottom pane > Output pivot > "prompt flow" channel, find the error message could be relevant to the issue and past them here. That would be helpful for our trouble shooting.
If your code to repro the issue is public and you want to share us, please share the link.

Additional context
Add any other context about the problem here.

Fixed with version 1.1.2.


sync a binary to an existing component

I want to be able to code locally and have it sync my changes to my running component and be smart enough to let me know when a restart of the container is required if dependencies have changed. This should work across PHP, Node.JS, Python, Ruby, and Java.
$ cd myfrontend
$ ocd create wildfly --binary=target/my-app.war
$ ocd push
$ ocd watch [--binary=target/mybetterapp.war]
                                         ^ override
Output:
Watching for changes on target/my-app.war.  To stop, use control-c
[13:45:10] target/myapp.war was pushed
[14:05:32] target/myapp.war was pushed

Subtask of #160
There's work in progress to demonstrate how this could work done by @grahamdumpleton and @jorgemoralespou

implemented in #408


Install from Homebrew is not working

I am using an M1 based Macbook

When installing from homebrew I had to manually add the execute bit.
 chmod +x  /opt/homebrew/Cellar/atac/0.11.3/bin/atac

The binary is for X64 rather than ARM so I can't run it

(PS -- This looks like a cool tool so will install from binary for now)

I've experienced this on both my intel mac book and my m1 mac book. Both of them returned the error
~ atac
zsh: exec format error: atac

# from m1 macbook
file atac
atac: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), static-pie linked, not stripped

Some tap are not using the binary but launch the cargo install
https://github.com/hendrikmaus/homebrew-tap/blob/main/Formula%2Frust-workflows.rb
Here is also an article https://superuser.com/questions/1793635/how-to-publish-binary-executables-as-homebrew-taps-for-both-macos-and-linux

I may have misunderstood something anyway.
Bit it seems to me there is something to around what they call a "bottle"

I think that problems comes from the binary I put in my homebrew script. Currently the only one that has been working for is unknown-linux-musl, but since I don't have a mac maybe it doesn't work at all on those devices. I have been searching how to put binaries for many distribution in my homebrew but I do not find anything at all... If you guys know, I am willing to understand how!
PS: heres the "script" https://github.com/Julien-cpsn/homebrew-atac/blob/main/Formula/atac.rb

It looks like this cargo tool could be useful https://github.com/axodotdev/cargo-dist

Indeed https://opensource.axo.dev/cargo-dist/book/installers/homebrew.html

Still not working for me (M1 Macbook)
$ file   /opt/homebrew/Cellar/atac/0.12.0/bin/atac
/opt/homebrew/Cellar/atac/0.12.0/bin/atac: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), static-pie linked, not stripped

:-(

@alecthegeek how can you have 0.12.0 while it's the 0.13.0 that is available ?
did you launch the following command ?
brew upgrade atac

@alecthegeek hmm, I've just udpdated the app version to 0.13.0 and your comments talks about the v0.12.0. Maybe try to uninstall entirelly and install it again? or just update it?

@Julien-cpsn for records, I just upgrade via brew, and I got the 1.13.0 and the binary is working

@Julien-cpsn for records, I just upgrade via brew, and I got the 1.13.0 and the binary is working

Yes same, I tried before posting. But well we use Linux 👀

@alecthegeek  If you want to replicate you could launch this
file $(realpath $(which))

Ah Ha.
A brew update fixed it!
Sorry about that.
Also not sure if this info is useful
https://docs.brew.sh/Formula-Cookbook#handling-different-system-configurations

Ah Ha.
A brew update fixed it!

@alecthegeek so everything works fine?? I am happily surprised haha

Also not sure if this info is useful
https://docs.brew.sh/Formula-Cookbook#handling-different-system-configurations

Yes I have red through that but the problem was handling many tar archives which is not documented

Looks like success to me. I will close the ticket.
Merci bien


:broom: Fix golang formatting in scan.go

gofmt is attempting to fix formatting in scan.go. Pull this out in to a
separate change and clean it up.

/test e2e-rosa


[IOTDB-1635] settle TsFiles and mods

Complete the settle tool, including online settleTool and offline settleTool. Also modify the online upgrade tool : remove the deleted data when rewriting tsfiles.

Coverage decreased (-0.01%) to 67.546% when pulling c0bedd2565679ef905ae433c7c3931bf02fcb481 on choubenson:settleTsFileAndMod into 1b7add158a99052e6dfaf1fb38314ead76d20d2b on apache:master.

First time to contribute for IOTDB，there may be many problems, including code format and specification, etc. Welcome to give me
correction !

hi , I feel you should write a Design Documents in wiki.
So  can review your code better.

Hi，i've writen the documents in wiki


0 Matic Balance Shown in USD and Can't Send

AW fails to read the Matic balance in USD and it doesn't allow it sending. If you change the connected wallet for another one and change back again you will see the balance...

I believe that this happens in all networks with their main native tokens. I only have assets on Mainnet so I can't check.

@Hoanib can you please provide video record for this issue?

However if you change the current wallet for another one and change back again you would see the entire balance and could perform the transaction. This temporary solution has allowed many users to carry out transactions.

I believe that this issue happens in all chains though I could not provide examples bc I have assets only in Mainnet and this small amount of Matic in Polygon.


Error es_ES

Google Translate version:
In the version es_ES/godmode9-usage#convert-a-file-3ds-a-cia, there is a link that redirects to #dumping-a-game-cartridge, and it should be #volcar-un-cartucho-de-juego...

Translation project issues should be directed to Crowdin.

The issue is now fixed and will be up with the next translations update. You can see the string in question here. You can also contribute with translations there!
(also, sorry for answering here, @Plailect)


Seemingly long time to reload

Hi-
I'm using supervisor, reload and chrome but it's taking about 6-7 seconds for the reload to actually occur. Is this normal? I've tried modifying the p value on supervisor  but it doesn't seem to matter.
What happens-

File is changed
Almost immediately the browser changes to 'This webpage is not available'
6-7 seconds elapse
Page is reloaded

I'm on OS X with Node 10.30.

Try using the new reload delay option true, that may fix your problem https://github.com/jprichardson/reload#reloadhttpserver-expressapp-delay

Had the same symptoms as OP; @alallier's issue resolved this for me.


not working on 1.20.6 forge/fabric

I tried pretty much everything at this point. at first I thought it was an issue with forge, then with tectonic, but both turned out to do nothing. the mod still doesn't replace anything.
Put your rules here in the format:
old_biome > new_biome

Examples (remove # in front of one to activate it):
minecraft:dark_forest > minecraft:cherry_grove
terralith:lavender_forest > aurorasdeco:lavender_plains
terralith:lavender_valley > aurorasdeco:lavender_plains
terralith:cave/infested_caves > minecraft:dripstone_caves
minecraft:ice_spikes > minecraft:snowy_plains
minecraft:mushroom_fields > minecraft_plains
minecraft:mangrove_swamp > minecraft_swamp
minecraft:deep_dark > minecraft:dripstone_caves
minecraft:frozen_ocean > minecraft:ocean
minecraft:lush_caves > minecraft:dripstone_caves
minecraft:dark_forest > minecraft:taiga
2024-12-24-5.log

That's very strange - according to the log, there are no rules in the config file? Make sure you are actually editing a correct file.
Meanwhile, I'll try backporting the never BR version, since it works very differently to 1.2 and may work out better for you.

Backport done - please try the new 1.7 version, and let me know wether it fixed the problem

installed the 1.7 version as listed on modrinth, but the issue still persists. I also tried deleting and rewriting the config file, but nothing seems to work. the only thing I can think of is that the mod is somehow incompatible with certain optimization mods, but I really cannot afford to
2024-12-27-3.log
2024-12-27-4.log

Weird question to start - did you remember to save the file after you edited it? Asking because I see a gray dot in your text editor tab, which usually indicates unsaved changes.
If that wasn't the problem, then I'm officially stumped. I've tried installing every mod you have down to the exact version (found from the logs), and could not reproduce the issue. It's unlikely to be a mod incompatibility anyway, since BR should at least log how many rules it loaded before breaking. In your case it's logging 0, which can't be right.
You can try to update your mods (many of them are outdated, I noticed) just to see if anything changes, but otherwise I'm out of ideas.

actually saving the file (forgive my stupidity) seems to have worked.... somewhat. I tested it on 5 clean worlds and 5 out of 7 biomes have apparently been eliminated.  however, mangrove swamps and mushroom fields keep consistently ignoring the set configs. as far as I'm aware there are no mistakes in the config, nor have I changed its contents after rewriting it to 1.7.

the logs however show that there are still no rules being applied, which does confuse me a little...
2024-12-27-5.log

Well, glad to hear the fix was simple, haha! I also see the issue with the last two biomes - when specifying replacements, you used a _ where a : should be - so instead of minecraft_plains it should be minecraft:plains, for example.
The fact that logs still don't work is even stranger, but I guess it's not a problem if the mod works as intended

oh, so it was a fault on my part :D thank you for the support, I'l much enjoy using this mod for a long time to come!


Prevent double refund for platform tip dept as well

I sof-deleted transactions #7836311 and #7836312 that were part of this bug. Prod data now looks clean:
select "TransactionGroup", kind, COUNT(*)
from "Transactions"
WHERE kind IN ('PLATFORM_TIP_DEBT')
AND type = 'DEBIT'
AND "isRefund" IS TRUE
AND "deletedAt" IS NULL
GROUP BY "TransactionGroup", kind
HAVING COUNT(*) > 1


feat: define chess board base models

Create base traits: ChessBoard, Move, Pawn, Piece, Position and File and Rank enumerations.

:tada: This PR is included in version 0.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Default list mode is not working.

Terms

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

Bagisto Version(s) affected
master
Issue Description
While changing the default list mode to list still it shows as the grid on the shop.
Preconditions

Admin should be logged in.
Admin should have at least one product.

Steps to reproduce

Go to the 'Admin panel'
Click on 'Edit products' inside the configuration/catalog
Scroll down to 'Storefront'
Set the default list mode as 'list' and save it
Go to the shop
Search for a product
See the error

Expected Result
It should be shown according to the selected option from the admin panel.
Actual Result
It is showing only in the grid. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Clarifications and additions to Apache Zeppelin README

Added additional details and clarifications to Apache Zeppelin readme file. Ultimate goal is to have README files follow an approximately similar format.

I signed it!


Switch Target Allocator strategies

Summary
Allocation strategy
Target Allocator has these 2 allocation strategies at the moment: least-weighted and consistent-hashing. least-weighted is the default one and is not capable of scaling out. It is not designed to re-distribute the scrape targets when new collectors are introduced (see OTel operator repo -> issue 2477). It is recommended to go for the consistent-hashing strategy which is more efficient and capable of scaling out.
Filtering strategy
Moreover, Target Allocator comes with a filtering strategy. At the moment, there is only one: relabel-config. This one filters all the targets discovered by the Prometheus discovery manager and then distributes the targets to the collectors. If this is not used, all of the discovered targets are assigned to collectors and the collectors filter the ones that are not relevant afterwards.
This default behaviour can cause the following issue. Let's say that we have 2 collectors and 10 targets in the cluster. However out of this 10, we only want to scrape 5 (due to relabel config). Target Allocator distributes all 10 targets to the collectors without considering which ones are to be scraped. So we might face such a scenario where 5 to-be-scraped targets are assigned to 1 collector and the rest to the other, meaning that 1 collector would do all the job.
Desired Behavior
Distribution of targets as evenly as possible in case of changes in to-be-scraped targets and in case of collector addition and removal.
Possible Solution
Target Allocator config:
allocationStrategy: consistent-hashing
filterStrategy: relabel-config

Implemented with #100.


Enable Python style checking during build and CI.

This PR enables Python style checking during build and CI.
It also reduces noise in Pylint's output (no chart displayed, no locally disabled warning displayed)
cross-reference: #587 #1072

👍


let users pass objects in msg.update()

Please describe the changes this PR makes and why it should be merged:
This PR lets users update messages by passing objects through message.update, for example a use case for this would be:
const embed = new Embed().setTitle('1');
const msg = await message.reply({ embeds: [embed] })
embed.setTitle('2')
msg.update({ embeds: [embed] })

Status

[x] Code changes have been tested.

Semantic versioning classification:

[x] This PR changes the library's interface (methods or parameters added)
[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR only includes non-code changes, like changes to documentation, README, etc.

the recent commit is about message.reply and how it has a bug when built

    /** Send a message that replies to this message. It mentions the user who sent this message. */
    reply(content) {
        var _a;
        if (typeof content === "string")
            content = { content };
        return this.client.messages.send(this.channelId, { content: content.content, replyMessageIds: (_a = content.replyMessageIds) !== null && _a !== void 0 ? _a : [this.id] });
    }

this for some reason makes it so that objects don't work in message.reply, for this reason, i got rid of the possibility of using replyMessageIds in the object for message.reply, it may/should be used in message.send instead

this for some reason makes it so that objects don't work in message.reply

what's the error you're receiving? it seems to work for me

this is when passing { embeds: [embed] }
{ content: 'hi' } works but solo embeds don't

this is when passing { embeds: [embed] }
{ content: 'hi' } works but solo embeds don't

This error is from Guilded's API. I guess content is required? Would be best to ask in the official server about this.
replyMessageIds shouldn't affect this though

content isn't required when passing an embed, your suggested commit fixed the problem, the issue was is that it was sending { content: content.content } instead of { ...content } for some reason

thank you 🙂


error at initializing zappa

Context
received error while initializing zappa
Expected Behavior

Actual Behavior

Possible Fix

Steps to Reproduce

Your Environment

Which version of Python are you running this under? Is it the same version you installed pip installed Zappa with initially?
I think you get this error if you run code for Python 2.7 under Python 3.6.
The correct syntax for this under Python 3.6 should be
os.chmod(temp_settings.name, 0o644)

@prashantbaditya  Which version of Zappa are you running?
I just checked the code for cli.py here: https://github.com/Miserlou/Zappa/blob/master/zappa/cli.py and that contains os.chmod(temp_settings.name, 0o644) and not  os.chmod(temp_settings.name, 0644) as in your case.
If you try a pip install -U zappa in your virtual environment - does that help with your issue?

current python version is 3.7 which is the latest and for zappa, I used below commands to install:
pip install django zappa
I tried pip install -U zappa as menioned above and received below error

Since AWS Lambda currently only supports Python 2.7 and 3.6 I would recommend that you install 3.6 instead (or alongside) of Python 3.7 and use that.
The error message you are getting also seems to indicate that there is something wrong with your Python installation.

Thanks @swedishmike ...It worked...

Glad to hear that it works for you now. If all's ok - are you good to close this issue?

yes


Test interactions between presenters and views

Say I want to test whether the view's displayError is actually called when the observable emits an error. How do I do that? I've been trying to write test codes and couldn't figure it out. I'd love if someone gives an example on it

https://github.com/android10/Android-CleanArchitecture/issues/223


Newline in VERSION file breaks maven-publish

When bumping the version number in the VERSION file if a newline is added the maven-publish workflow will fail with:
Invalid publication 'maven': version cannot contain ISO control character '\u000a'.

This might be a reasonable behavior.   But we could probably make this workflow more forgiving by trimming whitespace.

Note: vim inserts a new line by default.  Looks like maybe need to use:
:set nofixendofline

See: https://stackoverflow.com/questions/1050640/how-to-stop-vim-from-adding-a-newline-at-end-of-file

Fixed in each project build using trim()


Adjust roof codes

This is correct as-is. It's the "All Glass Panoramic Roof".

@timdorr there are more changes on the PR, did you see it?


solving the param issue

Instead of the sealed trait, it should be taking an executor context instead. We should distinguish between "spawn local" and "non-local spawn" executors, so that the generic param can be aligned and threaded through. This removes the need for the sealed trait + individual structs.

local spawning is not a great idea; dropping that (:


Add german translation of fastlane structures

This PR gives a german translation of a native speaker.

Okay. Where do I register the strings file for the german translation? I will always be able to translate new descriptions or strings.

Okay. Where do I register the strings file for the german translation? I will always be able to translate new descriptions or strings.

@fossdd You have to create res/values-de folder in app module, and translate the original strings.xml file from values folder here
But please fork your branch from the develop, because it's where most of the development work occurs.

@fossdd You have to create res/values-de folder in app module, and translate the original strings.xml file from values folder here
But please fork your branch from the develop, because it's where most of the development work occurs.


feat(server): remove addrstream struct

Remove AddrStream type, it provides no benefit over tokio::net::TcpStream.
Closes issue #2850
This is my first contribution to this project, so I may have missed something. I am happy to take any feedback. Thanks!
Notes

I'm not quite sure if my changes to AddrIncoming.poll_next_ method are correct, or if that method is actually used anywhere.
AddrStream is used in documentation related to make_service, but as I understand it, make_service will be removed anyway in a different PR. Ref issue #2854

Awesome, thanks for starting! I need to get the branch ready to start the new version, within the next day or two, and then I'll just start blasting merging :D

Cool! I'd love to work on some of the other easy issues from your stream tommorow.
I see the CI failed because AddrStream is imported in the  make_service examples. I'll fix it tommorow.

I updated the make_service doctests to use tokio::net::TcpStream as well, so now the tests pass. I'll make a new PR for #2854 and delete them. 😅


Tapo C225 connectivity issue

Version of the integration
2.11.0
Configuration
Device Name: Camera C225
Describe the bug
Tapo C225 can't be connected to
From the log, I can see it try to connect to device 80 port, which is closed. My C225 443 port is open.
Debug log

Failed to setup cannot connect
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 980, in _wrap_create_connection
    return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 1085, in create_connection
    raise exceptions[0]
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 1069, in create_connection
    sock = await self._connect_sock(
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/base_events.py", line 973, in _connect_sock
    await self.sock_connect(sock, address)
  File "/usr/local/lib/python3.11/asyncio/selector_events.py", line 628, in sock_connect
    return await fut
           ^^^^^^^^^
  File "/usr/local/lib/python3.11/asyncio/selector_events.py", line 668, in _sock_connect_cb
    raise OSError(err, f'Connect call failed {address}')
ConnectionRefusedError: [Errno 111] Connect call failed ('<IP_ADDRESS>', 80)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/config/custom_components/tapo/config_flow.py", line 215, in _try_setup_api
    await client.initialize()
  File "/usr/local/lib/python3.11/site-packages/plugp100/api/tapo_client.py", line 62, in initialize
    await self._guess_protocol()
  File "/usr/local/lib/python3.11/site-packages/plugp100/api/tapo_client.py", line 235, in _guess_protocol
    response = await self.execute_raw_request(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/plugp100/api/tapo_client.py", line 71, in execute_raw_request
    return (await self._protocol.send_request(request)).map(lambda x: x.result)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/plugp100/protocol/passthrough_protocol.py", line 42, in send_request
    response = await self._send_request(request)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/plugp100/protocol/passthrough_protocol.py", line 62, in _send_request
    await self._login_with_version(self._credential)
  File "/usr/local/lib/python3.11/site-packages/plugp100/protocol/passthrough_protocol.py", line 81, in _login_with_version
    session_or_error = await self._passthrough.handshake(self._url)
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/plugp100/protocol/securepassthrough_transport.py", line 70, in handshake
    response = await self._http.async_make_post(url, json=request_body)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/plugp100/common/utils/http_client.py", line 18, in async_make_post
    async with self.session.post(
  File "/usr/local/lib/python3.11/site-packages/aiohttp/client.py", line 1141, in __aenter__
    self._resp = await self._coro
                 ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/client.py", line 536, in _request
    conn = await self._connector.connect(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 540, in connect
    proto = await self._create_connection(req, traces, timeout)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 901, in _create_connection
    _, proto = await self._create_direct_connection(req, traces, timeout)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 1209, in _create_direct_connection
    raise last_exc
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 1178, in _create_direct_connection
    transp, proto = await self._wrap_create_connection(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/aiohttp/connector.py", line 988, in _wrap_create_connection
    raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host <IP_ADDRESS>:80 ssl:default [Connect call failed ('<IP_ADDRESS>', 80)]

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/config/custom_components/tapo/config_flow.py", line 118, in async_step_user
    tapo_client = await self._try_setup_api(user_input)
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/tapo/config_flow.py", line 220, in _try_setup_api
    raise CannotConnect from error
custom_components.tapo.errors.CannotConnect

just realized this is a duplicated issue with this: https://github.com/petretiandrea/home-assistant-tapo-p100/issues/578

This integration not supports tapo camera, you must refer to tapo-camera-control integration


BED-4622: Enforce goimports

Description

Fixes all formatting issues that diverge from normal goimports
Adds a new step to prepare-for-codereview to ensure goimports has run

Motivation and Context
This PR addresses: BED-4622
Why is this change required? What problem does it solve?
We've long used goimports as our source of truth for formatting preference, but there are some minor differences in how that tooling is used in different IDEs. This is the first step towards ensuring that code is actually formatted by the canonical goimports. There will be evaluation of this for our pipelines in the future.
How Has This Been Tested?
Ran through the new commands and ensured the new dependency is picked up properly
Types of changes

Chore (a change that does not modify the application functionality)

Checklist:

[x] I have met the contributing prerequisites

Assigned myself to this PR
Added the appropriate labels
Associated an issue: https://github.com/SpecterOps/BloodHound/issues/672
Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing

[x] I have ensured that related documentation is up-to-date

Open API docs
Code comments (GoDocs / JSDocs)

[x] I have followed proper test practices

Added/updated tests to cover my changes
All new and existing tests passed

For some reason the just deps command is not installing goimports for me.
➜  bhce git:(BED-4622) ✗ just stbernard deps
{"level":"info","time":"2024-07-25T13:49:21.272486-04:00","message":"Running go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.149043-04:00","message":"Finished go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.14916-04:00","message":"Running go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.767212-04:00","message":"Finished go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.767375-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/cmd/ui"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 258ms
➤ YN0000: Done with warnings in 0s 653ms
{"level":"info","time":"2024-07-25T13:49:23.833247-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/cmd/ui"}
{"level":"info","time":"2024-07-25T13:49:23.833442-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/bh-shared-ui"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 262ms
➤ YN0000: Done with warnings in 0s 661ms
{"level":"info","time":"2024-07-25T13:49:24.946727-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/bh-shared-ui"}
{"level":"info","time":"2024-07-25T13:49:24.946867-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/js-client-library"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 258ms
➤ YN0000: Done with warnings in 0s 650ms
{"level":"info","time":"2024-07-25T13:49:26.016716-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/js-client-library"}
{"level":"info","time":"2024-07-25T13:49:26.01673-04:00","message":"Command `deps` completed successfully"}
➜  bhce git:(BED-4622) ✗ just goimports
Running goimports, check for file drift
find: goimports: No such file or directory

For some reason the just deps command is not installing goimports for me.
➜  bhce git:(BED-4622) ✗ just stbernard deps
{"level":"info","time":"2024-07-25T13:49:21.272486-04:00","message":"Running go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.149043-04:00","message":"Finished go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.14916-04:00","message":"Running go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.767212-04:00","message":"Finished go install for /Users/jbarnett/dev/bloodhound-enterprise/bhce"}
{"level":"info","time":"2024-07-25T13:49:22.767375-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/cmd/ui"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 258ms
➤ YN0000: Done with warnings in 0s 653ms
{"level":"info","time":"2024-07-25T13:49:23.833247-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/cmd/ui"}
{"level":"info","time":"2024-07-25T13:49:23.833442-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/bh-shared-ui"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 262ms
➤ YN0000: Done with warnings in 0s 661ms
{"level":"info","time":"2024-07-25T13:49:24.946727-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/bh-shared-ui"}
{"level":"info","time":"2024-07-25T13:49:24.946867-04:00","message":"Running yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/js-client-library"}
➤ YN0000: ┌ Resolution step
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui doesn't provide @testing-library/dom (pa1bac), requested by @testing-library/user-event
➤ YN0002: │ bh-shared-ui@workspace:packages/javascript/bh-shared-ui [b226e] doesn't provide @testing-library/dom (p2a54b), requested by @testing-library/user-event
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @mui/system (pec2ad), requested by @mui/x-date-pickers
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/is (p33505), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @redux-saga/symbols (p0b773), requested by redux-saga-test-plan
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide @testing-library/dom (p99d58), requested by @testing-library/user-event
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides luxon (p5ac9e) with version 2.5.2, which doesn't satisfy what @date-io/luxon requests
➤ YN0060: │ bloodhound-ui@workspace:cmd/ui provides react (p74846) with version 18.2.0, which doesn't satisfy what @mui/styles requests
➤ YN0002: │ bloodhound-ui@workspace:cmd/ui doesn't provide redux (p6f2da), requested by redux-devtools-extension
➤ YN0002: │ sigma@npm:2.4.0 doesn't provide graphology-types (pe610a), requested by graphology-utils
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed
➤ YN0000: ┌ Link step
➤ YN0000: └ Completed in 0s 258ms
➤ YN0000: Done with warnings in 0s 650ms
{"level":"info","time":"2024-07-25T13:49:26.016716-04:00","message":"Finished yarn install for /Users/jbarnett/dev/bloodhound-enterprise/bhce/packages/javascript/js-client-library"}
{"level":"info","time":"2024-07-25T13:49:26.01673-04:00","message":"Command `deps` completed successfully"}
➜  bhce git:(BED-4622) ✗ just goimports
Running goimports, check for file drift
find: goimports: No such file or directory

There's a successful second go install. My best guess is that your GOPATH isn't on your PATH, which can happen on people's macs when they use brew I think. Let's take a closer look on your system to figure it out and see if there are any mitigations we can add to st bernard

The problem was with my environment. I have Go installed via homebrew, which does not automatically add the installation directory to $PATH. goimports was correctly installed to ~/go/bin, and once I added that to $PATH the just goimports command started working as expected.
This should be good to go!


Cannot terminate task when running using the regular task runner then switching to terminal runner

Testing #19057

VSCode Version: Code - Insiders 1.9.0-insider (4265f5e35cb3ea8ba7e0cf6b14ec606b6f8647ea, 2017-01-24T10:29:13.076Z)
OS Version: Darwin x64 16.3.0
Extensions:

Extension
Author
Version

EditorConfig
EditorConfig
0.3.4

lorem-ipsum
Tyriar
1.0.0

sort-lines
Tyriar
1.2.0

theme-glacier
Tyriar
0.0.10

vscode-svgviewer
cssho
1.3.0

gitlens
eamodio
1.4.3

git-project-manager
felipecaputo
1.0.1

md-navigate
jrieken
0.0.1

vscode-scss
mrmlnc
0.6.0

seti-icons
qinjia
0.1.3

Steps to Reproduce:

Setup and run a watch task that doesn't terminate
Add "_runner": "terminal" to tasks.json
Try run the task again
You will be asked to terminate, it doesn't appear to do it, repeat 3-4

Even when I remove _runner I keep getting this message:

The fact that you couldn't kill the build task even in the output runner is a dup of #19328. Hot switching the runner with running processes is currently not support I don't intend to do so (not a common use case since the Output runner will go away in 1 - 3 releases). I will add a warning that hot swapping is not supported.

What I see now is that on step 3, you get a dialog:
The task is already active and in watch mode. To terminate the task use F1 > terminate task
Which of course is not true.
I understand that this is a bit of a constructed example, with switching the runner while a task is running.
@dbaeumer What is the expected behaviour?

@aeschli hot switching is not supported and will never be. I added a message that if you try to hot switch the runner that informs you that you need to reload the window. So you should still be able to terminate the task. I will reopend and reinvestigate in Feb.

Not critical.

Fixed by two things:

you can't hot switch the runner anymore. It stays the same even if you change values in the tasks.json
if the terminal runner is used then you now get a list of active tasks to terminate (if more than one).

No further action planned since hot switching will not be supported.


Django 1.9 compatibility

Package will not work in Django 1.9 due to deprecated import in forms:
``django_json_field-0.5.7-py2.7.egg/json_field/forms.py:5: RemovedInDjango19Warning: The django.forms.util module has been renamed. Use django.forms.utils instead.```
There is no necessary to import django.forms.util(s), because you're using it to raise ValidationError. Just import forms and change util.ValidationError to forms.ValidationError, or import ValidationError from django.core.exceptions.

there is already a PR i've submitted for this. just waiting for it to be accepted ='(
https://github.com/derek-schaefer/django-json-field/pull/41


[FLINK-11837][table-runtime-blink] Improve internal data format

What is the purpose of the change
Introduce Decimal: Scale of this object is specified by the user, not automatically determined(like BigDecimal).
Introduce BinaryGeneric(GenericType): We don't care about specific types, we work through serializers.
Introduce LazyBinaryFormat: It can exist in two formats: binary or java object
Verifying this change
ut
Does this pull request potentially affect one of the following parts:
all no
Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (JavaDocs)

BTW, NestedRow has not been covered by any test case

BTW, NestedRow has not been covered by any test case

Yes, I will add test for NestedRow, JoinedRow, BoxedWrapperRow and GenericRow

@KurtYoung Thanks for your suggestion, I have fixed it.

LGTM, merging...


Add benches and test cases for perf context

PTAL @huachaohuang . And seems PerfContext is not thread local for all platforms. So should we add a feature gate for this?

@hicqu What is the specific platform that doesn't support thread local variable? If it is not common, I guess it is not necessary.


Easylist tracker detection

Asana: https://app.asana.com/0/361428290920652/471718733961915
Description:
Adds Adblock plus tracker blocking to the app. Uses the easylist and easyprivacy tracker lists (downloaded from our servers) to perform matching.
NOTE: Other than adblockplus-lib.cpp which is ours, most of the C++ code is third party and can be ignored. It will be packaged separately in the the future. A server sync service will also be added separately.
Steps to test this PR:
Test: Data Loading

Open the app.
In the console note the time it takes between the two calls to "Loading basic data" and "Loading complete"
Kill the app and open it a second time.
Now note the time between "Loading preprocessed data" and "Loading Complete". It should be significantly faster

Test: False positives

Navigate to duckduckgo.com
Observer that all requests are listed as not blocked in the console

Test: False negatives

Navigate to evanscycles.com
Observer that (amongst others) requests to https://manager.tagcommander.com/utils/IP and https://cdn.optimizely.com are blocked

Author Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and document any non-automated tests)
[ ] Consider implications (CPU, memory, database / preference migrations, statistics)

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

Thanks for the feeback @CDRussell, ready for rereview.

👍  when CI passing


--actAsUserId not working with uuid

Due this parse in $actAs, --actAsUserId is not working when using uuid.

same issue

Hi,
I couldn't reproduce the error following the steps given in this environment:
OS: macOS High Sierra
Laravel: 5.5
The running command was
php artisan api:generate --routePrefix="api/users  --actAsUserId=4317"

Fixed in #257


[v17] Replay votes in response to a confirm_req for an active block

Port of https://github.com/nanocurrency/raiblocks/pull/1382/

IIRC compute_rep_votes is never called, so we can get rid of that function entirely.
I was hoping that V17 would be a good opportunity to do some kind of LRU cache instead of tying it to active_transactions, but this is still a lot better than nothing, and we can improve it later if we need to. :+1:

If you have LRU cache in mind, we can merge this in 17.0 & not merge to master, wait for cache I guess
@rkeene ?

That's fine -- getting it fixed in V17.0 and doing something better in V18+ works


Source refactor. Abstracted common functionalities and removed redundant...

... code

Any news on the status of this pull request?

sorry, will not accept.

Thanks for the response. Is there a particular reason?


Enhancement and Optimization of smart contract

Hello ,
Myself Raja Kumar , a proud contributor at GSSOC23. I would like to contribute to this project.
### Optimization
1. Use uint256 instead of uint .
2. Consider using a mapping instead of an array
If you anticipate a large number of deployed Sal contracts, using a mapping instead of an array for deployedSal in the allemp contract can provide better efficiency for contract retrieval and management.
3. Consider using bytes32 for string parameters
If the string parameters have a fixed length, such as _FirstName, _LastName, _Country, _Position, and _image, use the bytes32 data type instead of string. This can save gas costs and improve efficiency.
4.Consider using bytes32 for string parameters
If the string parameters have a fixed length, such as _FirstName, _LastName, _Country, _Position, and _image, you can use the bytes32 data type instead of string. This can save gas costs and improve efficiency.
5.Consider using a modifier for access control
If certain functions should only be accessible by specific roles or addresses, you can use a modifier to enforce access control. This helps improve security and restrict unauthorized access to critical functions.

If you think it is legit then do assign me the issue under GSSOC23 with the level of PR as Well.

PLease share the code along  with the changes.

Hey bro @Raja543 .Please share the code where u have seen all the stuff so that i can rectify and assign u .Hope u get it.SHre the code here only and highlight all the things.

1 :  use uni256 instead of uni
for example : - uint indexed timestamp,
uint public recievedamnt;
2:- Consider using a mapping instead of an array
address [] public deployedSal;
event salcreated(
    string FirstName,
    string LastName,
    address indexed owner,
    address salAddress,
    string image,
    uint indexed timestamp,
    string indexed Position
);

3: -  Consider using a modifier for access control
modifier onlyOwner() {
require(msg.sender == owner, "Only the contract owner can call this function");
_;
}
4:- Consider using bytes32 for string parameters
string memory _FirstName,
string memory _LastName,
string memory _wallet_ddress,
string memory _Position,
string memory _Country,
string memory _image

here is the code @Vikash-8090-Yadav


Backport 2.16: Clarify that the full config enables everything that can be tested together

This is a partial backport of #3190.

Copy the general guidance around full and baremetal from config.py to config.pl, but not the per-symbol comments, and of course not the code structure.
Fix the build failures with options that are now enabled in full.
Make the code strict C99.
Reorganize the testing around deprecated features.

In the LTS branches, there are no deprecated features in full, so I didn't backport full_no_deprecated nor the corresponding all.sh components.
There may be some debate about exactly which parts should be backported. For some things I think that it's a close call between keeping things aligned in all branches and keeping things simple where only development needs the extra complexity (full_no_deprecated). So I'll wait until at least one approval on this PR to make the 2.7 backport, which I expect to be very similar to 2.16.

Following our chat earlier today, I excluded MBEDTLS_DEPRECATED_WARNING from full.

I realized that I'd made some mistakes with the *deprecated* components of all.sh, so I rewrote the last two commits.


Updates for adding status to push pools and their work queues

Previously work pool status and work queue status only referred to work pools which are polled by workers.  We have now introduced the idea of status for push style work pools and their work queues.  This PR updates the wording in the tooltips to remove mention of workers for push pools.
This PR also updates the workPoolQueueStatusArray component to show work queue status for push pool related work queues on the dashboard.
After:

Before:

Thanks @pleek91 - changes made