[bug] DEFAULT_FILE_STORAGE environment var not always respected

Idiscovered a new bug in Pulp that i believe might be a dynaconf issue. The issue revolves around DEFAULT_FILE_STORAGE setting.  In our codebase we have the following import from django.core.files.storage import default_storage . When DEFAULT_FILE_STORAGE is set in settings.py, default_storage reports it correctly. If we set PULP_DEFAULT_FILE_STORAGE as an environment variable, the default_storage does not reflect the value of the env var. At the same time dynaconf list reports the correct value set in the env var.

I figured out that on Django 4.2 (the LTS version) when you set DEFAULT_FILE_STORAGE, Django performs some Magic when setting it https://github.com/django/django/blob/stable/4.2.x/django/conf/init.py#L339 and https://github.com/django/django/blob/stable/4.2.x/django/conf/init.py#L291
So it looks like the variable DEFAULT_FILE_STORAGE should not be used anymore but replaced with STORAGES
https://github.com/django/django/blob/d26c8838d0a447d5cba03a0b4eebd5fc2d27e9df/django/conf/__init__.py#L45C6-L45C75
The DEFAULT_FILE_STORAGE setting is deprecated. Use STORAGES instead

I will add those same checks to Dynaconf, but in any case it is recommended for Pulp to replace DEFAULT_FILE_STORAGE with STORAGES


examples: exchange entire RTCSessionDescription

Currently we only exchange the SDP between the browser (JSFiddle) and the example, hard-coding the session description type on both sides. This should be changed so the entire RTCSessionDescription is exchanged. In addition, all examples should be adapted so the offer/answer direction can be changed.
TODO

[ ] Adapt gstreamer-receive
[ ] Adapt gstreamer-send
[ ] Adapt save-to-disk
[ ] Adapt data-channels

Hi @evgeniuz. Awesome, glad to have you! Don't hesitate to ask any questions you may have. Please conciser joining the #pion channel of the gopher slack. It makes discussions a little easier plus we've got a great community in there.

This issue still opened?

Hi @millnitzluan. Yes, it is still open. JSON (un)marshaling of the RTCSessionDescriptions was added in #137 but the examples have not yet been adapted to exchange the entire RTCSessionDescription.


Use JSON to detail data?

As suggested by @mattbettcher here https://docs.google.com/document/d/1ztZMiwmc8K0bMAI4FOQ0IzniEZdJgPg6SASTbRjdeuQ/edit

To be honest it seems crazy verbose compared to just leaving it as code, but it does have all the info needed to gen the md.

yeah.. maybe it's better just have it as code really? Say that we have a config for 000/010/020/etc which is just code. Also the description would be the same in each file so maybe that can be general for shared instructions? Also with the latest changes the code is a bit more compact now also.

I agree, I feel it would be easier to add the extra items directly in the source.

Lets do that!

I suggest something like this then
struct Description<'a> {
  operation: &'static str,
  assembler: &'a [&'static str],
  attributes: &'static str,
}

Then in each instruction we add
struct Instruction<'a> {
  ...
  description: &'a Description,
}

That we we don't need to duplication the data for each table.


chore: Fix broken if-else logic in JSON schema

Reason for This PR
There's a problem with the if-else logic in the schema here.
Description of Changes
I replaced if-else with a oneOf for a valid combination of options. I have tested this locally so it should be the last PR for this repo.
Also, address is not required. I looked at the driver and it defaults to localhost:7233. I changed it only for consistency with the prometheus package; I'm aware localhost and <IP_ADDRESS> is the same.
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.
PR Checklist
[Author TODO: Meet these criteria.]
[Reviewer TODO: Verify that these criteria are met. Request changes if not]

[x] All commits in this PR are signed (git commit -s).
[ ] The reason for this PR is clearly provided (issue no. or explanation).
[ ] The description of changes is clear and encompassing.
[ ] Any required documentation changes (code and docs) are included in this PR.
[ ] Any user-facing changes are mentioned in CHANGELOG.md.
[ ] All added/changed functionality is tested.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 40.06%. Comparing base (9e19b6e) to head (c5c2e9a).
Report is 4 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #572   +/-   ##
=======================================
  Coverage   40.06%   40.06%
=======================================
  Files          14       14
  Lines         936      936
=======================================
  Hits          375      375
  Misses        518      518
  Partials       43       43

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


No able to run android app AppsFlyerLib.getInstance().setSharingFilterForPartners(null);

node_modules/react-native-appsflyer/android/src/main/java/com/appsflyer/reactnative/RNAppsFlyerModule.java:607: warning: non-varargs call of varargs method with inexact argument type for last parameter;
AppsFlyerLib.getInstance().setSharingFilterForPartners(null);
^
Report
Plugin Version
6.4.0
On what Platform are you having the issue?
Android
What did you do?
ℹ Please replace these two lines with what you did.
e.g. Run react-native run-android
What did you expect to happen?
ℹ Please replace these two lines with what you expected to happen.
should run successfully
What happened instead?

what version of RN do you use? it's just a warning or the app crashes also?

I got the same error. I'm using RN 0.63.4. For me the app build fails on android. Downgrading to the plugin version 6.3.50 fixed the issue.

@tiagobnobrega @vinaysgl can you please share your code where you call appsFlyer.setSharingFilterForPartners?

In m1 (monerey 12.0.1)
The same error occurred when using npx react-native run-andeoid using RN 0.66.0 and  plugin version 6.5.20
And I've never called appsFlyer.setSharingFilterForPartners

@amit-kremer93 I got the same issue, app build fail on android, on both 6.4.x and 6.5.20. Downgrading to the plugin version 6.3.50 fixed the issue.

@CuongNVietCetera which JDK version do you use?

@amit-kremer93 oh I got error with my old jdk 8. Just update to jdk 11 and it works now, thank you!

Any fixes yet ? getting same issue after upgrading react-native to 0.70

@vinaysgl  I got the same issue
- Downgrading to the react-native-appsflyer version : 6.3.50
- Downgrading to the react native moengage version  : 8.1.2
fixed the issue.


Removed dummy scene during saving

Fixes #43156
Might reintroduce #7976, but to me it looks very similar to #7984, which isn't fixed, so not sure ¯\_(ツ)_/¯

I've opened #49570, that addresses the root cause –if my hyptohesis is correct–, but also has some risk of adding regressions (different from the ones that may be introduced by this one).
My suggestion on how to proceed would be try to validate mine and only if that's not possible or it's found to be wrong, come back to this one and try to confirm it doesn't add regressions.

Superseded by #49570.


Substitution type not being recognized

According to the example on the README file (and I tested it too),
var start = 'This is a test for see how work the library',
    end   = 'This is a test for know how work the new library';

console.log(showResult(greedyInstance.differences(start, end)));
/*
result:
This is a test for (-see)(+know) how work the (+new )library
*/

The see and know words are detected as deletion and insertion. Aren't they both supposed to be under the substitution type, as they are placed on the same position in the string, between equal portions?
This is a test for (-+see,know) how work the (+new )library

If you check the initial table where the algorithms are compared, you can see that the operations available for the Gready algorithm are only: insert and delete, the substitution is only available in the Levenshtein algorithm.
Sorry for the late reply.


Allow specifying OpenWhisk dir in docker/build.sh

Previously, specifying an OpenWhisk directory when running
docker/build.sh would cause the script to silently exit 1. Now, a
directory may be specified when the script is run.
ICLA signed and submitted on 2017/06/01
Related-Issue: apache/incubator-openwhisk-deploy-kube#6
Signed-off-by: Matt Langbehn<EMAIL_ADDRESS>

@mlangbehn again, good catch...  Fixes https://github.com/apache/incubator-openwhisk-deploy-kube/issues/6


feat: additional x/leverage queries

Description
Adds more queries (and CLI support) for x/leverage
Progress towards #93 and #94

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[x] added appropriate labels to the PR
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch (see PR Targeting)
[x] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[x] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[x] reviewed "Files changed" and left comments if necessary
[x] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #320 (7cd159e) into main (c36f58c) will decrease coverage by 0.47%.
The diff coverage is 20.88%.

@@            Coverage Diff             @@
##             main     #320      +/-   ##
==========================================
- Coverage   32.78%   32.30%   -0.48%
==========================================
  Files          74       74
  Lines        9258     9640     +382
==========================================
+ Hits         3035     3114      +79
- Misses       5930     6233     +303
  Partials      293      293

Impacted Files
Coverage Δ

x/leverage/client/cli/query.go
0.00% <0.00%> (ø)

x/leverage/keeper/grpc_query.go
21.00% <0.00%> (-18.68%)
:arrow_down:

x/leverage/types/query.pb.gw.go
0.00% <0.00%> (ø)

x/leverage/client/tests/suite.go
100.00% <100.00%> (ø)

x/oracle/keeper/keeper.go
68.01% <0.00%> (-0.15%)
:arrow_down:

R4R again - note that the tests in client/tests/suite.go are still at a bare minimum - the next PR will flesh them out (table-driven, additional setup, etc).


Add plot_ridge

Split the current plot_forest in to plot_ridge (mountain-range kde-like) this issue and plot_forest (intervals) #6 }
For list of dataset/inferencedata input we might need another loop strategy
NOTE: __variable__  only works for facetting right now, it is needed for this to work with aesthetics when there are multiple variables.

It will look very similar to plot_forest (already implemented) so it might be a good idea to start from there and once it is working try to refactor them to extract common parts and avoid duplication


fctl tool

It would be great to have a way to "command line" query FAUCET controllers and present state in a familiar way. For example, to show which MACs have been learned on what ports/VLANs. The tool fctl would just use Prometheus queries to FAUCET and Gauge and wouldn't modify the controller's state.

Hi,
Just thinking that if the tool does not control Faucet and only makes queries, perhaps the name should not be fctl but maybe something like fquery ... ?
Cheers,
Vlad

I was thinking that we should restrict monitoring like tools for Faucet,
but enable them for Gauge.  Specifically, I am ok with fctl/fquery running
on either Faucet or Gauge - very useful for debugging purposes to exactly
understand what is seen by that particular controller.  I would be more
inclined to remove Prometheus integration for Faucet and focus only on
Gauge for monitoring.  Today, this functionality is simple and cool.  But,
maintaining it over a longer period of time is going to be hard.
Thanks
On Thu, Sep 7, 2017 at 6:37 AM, Vladimir Mencl<EMAIL_ADDRESS>wrote:

Hi,
Just thinking that if the tool does not control Faucet and only makes
queries, perhaps the name should not be fctl but maybe something like
fquery ... ?
Cheers,
Vlad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/947#issuecomment-327801603, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABmf899-zrwZQB9iCWtwZwARMrjJR3vmks5sf_GmgaJpZM4PPM73
.

Because FAUCET and Gauge are separate (a good thing for reliability, etc),
they need to both be monitored separately (for example, Prometheus can
monitor that both/all processes have good connections to dataplanes, which
is an essential and basic monitoring requirement). Therefore, I think
Prometheus has to be in FAUCET ongoing for at least these reasons. It does
not hurt anything, quite the reverse.
On Fri, Sep 8, 2017 at 9:42 AM, shivarammysore<EMAIL_ADDRESS>wrote:

I was thinking that we should restrict monitoring like tools for Faucet,
but enable them for Gauge. Specifically, I am ok with fctl/fquery running
on either Faucet or Gauge - very useful for debugging purposes to exactly
understand what is seen by that particular controller. I would be more
inclined to remove Prometheus integration for Faucet and focus only on
Gauge for monitoring. Today, this functionality is simple and cool. But,
maintaining it over a longer period of time is going to be hard.
Thanks
On Thu, Sep 7, 2017 at 6:37 AM, Vladimir Mencl<EMAIL_ADDRESS>wrote:

Hi,
Just thinking that if the tool does not control Faucet and only makes
queries, perhaps the name should not be fctl but maybe something like
fquery ... ?
Cheers,
Vlad
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/947#issuecomment-327801603,
or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABmf899-
zrwZQB9iCWtwZwARMrjJR3vmks5sf_GmgaJpZM4PPM73>
.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/947#issuecomment-327934963, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgZ15xIzdp6yBUgX6cyZCT4RGY2xWpYks5sgGMqgaJpZM4PPM73
.

That is so, today it doesn't modify state. One can imagine in the future
though it might include convenient helpers (for example, request a config
reload).
On Fri, Sep 8, 2017 at 1:37 AM, Vladimir Mencl<EMAIL_ADDRESS>wrote:

Hi,
Just thinking that if the tool does not control Faucet and only makes
queries, perhaps the name should not be fctl but maybe something like
fquery ... ?
Cheers,
Vlad
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/REANNZ/faucet/issues/947#issuecomment-327801603, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEgZ15Cia-yIexQ54MxJelzO3R_YNwJJks5sf_GmgaJpZM4PPM73
.

@anarkiwi - OK, if it's not completely restricted from modifying state, I'm happy for it to be called fctl :-)

I have an early prototype for folks to play with here: https://github.com/simeonmiteff/faucet/commit/6418f9627e6c243068b27cb44e13c30b674e2efc

https://github.com/faucetsdn/faucet/commit/aab5649891cebed4cced5d87bd07fca6c286fffc
josh@windscale:~/tmp$ ./fctl.py  -h
Retrieve FAUCET/Gauge state using Prometheus.
./fctl.py [-n] <-e|--endpoints=http://prometheus:port> [-m|--metrics=prometheus_metrics,] [-l|--labels=name:value,]

-n: Don't report 0 values
-e|--endpoints: list of Prometheus endpoints to query (comma separated)
-m|--metrics: list of Prometheus variables to query (comma separated)
-l|--labels: filter list of Prometheus variables by labels that must be present (comma separated)

Examples:
MACs learned on a DP.

./fctl.py -n --endpoints=http://<IP_ADDRESS>:9244 --metrics=learned_macs --labels=dp_id:0xb827eb608918

Status of all DPs

./fctl.py -n --endpoints=http://<IP_ADDRESS>:9244 --metrics=dp_status

josh@windscale:~/tmp$ ./fctl.py -n --endpoints=http://<IP_ADDRESS>:9244 --metrics=dp_status
dp_status	{u'dp_id': u'0xb827eb608918'}	1.0
dp_status	{u'dp_id': u'0xeccd6df72de7'}	1.0
josh@windscale:~/tmp$ ./fctl.py -n --endpoints=http://<IP_ADDRESS>:9244 --metrics=learned_macs --labels=dp_id:0xb827eb608918
learned_macs	{u'dp_id': u'0xb827eb608918', u'vlan': u'2004', u'port': u'2', u'n': u'0'}	9cf387b5f4c6
learned_macs	{u'dp_id': u'0xb827eb608918', u'vlan': u'2004', u'port': u'2', u'n': u'1'}	a45e60c55ced
learned_macs	{u'dp_id': u'0xb827eb608918', u'vlan': u'2004', u'port': u'2', u'n': u'2'}	da0204d4a435

This has been implemented and improved with some further contributions.


TacO cannot find GW2MumbleLink

Hey guys,
I've been encountering some issues with TacO and LB recently. Not sure whether or not this is your part of the code or an issue related to TacO but since you're working on it together I'll just dump what I found here.
So when I start a single client using Launchbuddy and afterward start TacO using the button in Launchbuddy, everything works fine. However, if I try to do the same with multiple clients running, it works sometimes. I'm currently still trying to find a way to reliably reproduce this problem and keep you update If I find it.
TacO states the following problem in the log:
Closing TacO because GW2 with mumble link GW2MumbleLink2 was not found in under a minute
The number (in this case 2) changes according to the account I tried to start it on.
I'm currently using d912pxy as well as ReShade and did not encounter any other problems with this setup. Everything is updated to the most recent version, running with admin permissions and in windowed fullscreen. Versions can be found below:

Software
Version

d912pxy
2.3.0

ReShade
4.8.2

LaunchBuddy
2.2.1

TacO
47.2802r

Game
109065

Looks like someone had a similar problem here.
That's what I gathered so far. If you require any additional details regarding my setup or any logs, feel free to reach out.
Best regards
Felix
Edit: Table was broke

After inspection using ProcessExplorer I found out that the -mumble parameter is not set. If I'm not wrong this line should be responsible for adding it. Any idea why this might not be working?

Reverting back to 2.2 resolves this. Seems to be a problem with the line mentioned above.

Hi,
https://github.com/TheCheatsrichter/Gw2_Launchbuddy/releases/tag/2.2.1 states:

Only use custom mumblelink on multiboxed clients to further support software which does not support multiboxed mumble links

For this reason LB starts all first selected GW2 accounts without a mumble link !
Only if already at least one GW2 client is running and another GW2 account is started later in LB, they will get started with a mumble link.
The issue is now: TacO starts everytime with the mumble link for each GW2 account resulting in TacO times out trying to bind to non-existing GW2 mumble link.
I think, it would be an idea to put an option to de-/activate mumble links at all or separated for each account.

This issue should be resolved with https://github.com/TheCheatsrichter/Gw2_Launchbuddy/commit/ed2914ae66a3957fe5921b4620dde1733f752d27
Tested it on my machine and seems to work. First acc now gets the correct Mumble Link.

This issue should be resolved with ed2914a
Tested it on my machine and seems to work. First acc now gets the correct Mumble Link.

Thanks TheCheatsrichter. At first I was confused, because no account started with a mumble link but then I found the option in "Account settings > {account} > Advanced Settings: Use Custom Mumble Link" and this worked like a charm now.


Concave polygons not colliding

Concave polygons are rendered and collide with the border but dont collide with other objects.

Caused by not updating the sub polyongs bounding boxes.


Orthographic projection not working with rasterize=True

I'm not sure if this is a bug or if I'm doing something wrong, but when I try to use cartopy's crs.Orthographic() projection, I get this warning:
/mamba/envs/viz/lib/python3.10/site-packages/datashader/glyphs/quadmesh.py:659: RuntimeWarning: invalid value encountered in cast
  xs = (xscaled * plot_width).astype(int)
/mamba/envs/viz/lib/python3.10/site-packages/datashader/glyphs/quadmesh.py:660: RuntimeWarning: invalid value encountered in cast
  ys = (yscaled * plot_height).astype(int)
/mamba/envs/viz/lib/python3.10/site-packages/datashader/glyphs/quadmesh.py:659: RuntimeWarning: invalid value encountered in cast
  xs = (xscaled * plot_width).astype(int)
/mamba/envs/viz/lib/python3.10/site-packages/datashader/glyphs/quadmesh.py:660: RuntimeWarning: invalid value encountered in cast
  ys = (yscaled * plot_height).astype(int)

And my map looks like this:

However, if I remove rasterize=True, everything seems to work:

Here's the code I used to create the first map:
import hvplot.xarray
import numpy as np
import xarray as xr
from cartopy import crs

ds = xr.tutorial.open_dataset('ersstv5').isel(time=0)["sst"]

ds.hvplot.quadmesh(
    project=True,
    geo=True,
    rasterize=True,
    dynamic=True,
    projection=crs.Orthographic(),
)

Versions:

hvplot : 0.8.4
numpy : 1.24.4
xarray : 2023.8.0
cartopy : 0.22.0
datashader : 0.15.2

Thanks for reporting this.
Do you need to specifically use QuadMesh? If not, a workaround is just using an image:
import hvplot.xarray
import numpy as np
import xarray as xr
from cartopy import crs

ds = xr.tutorial.open_dataset('air_temperature').isel(time=0)

ds.hvplot(
    project=True,
    rasterize=True,
    projection=crs.Orthographic(central_longitude=-88),
    coastline=True,
    global_extent=True,
)

Okay, seems like a bug with project=True + rasterize=True
import hvplot.xarray
import numpy as np
import xarray as xr
from cartopy import crs

ds = xr.tutorial.open_dataset('air_temperature').isel(time=0)

ds.hvplot.quadmesh(
    geo=True,
    rasterize=True,
    dynamic=True,
    projection=crs.Orthographic(),
)

This seems to work.

Without project=True, it works until move the plot or zoom in. If I do, half of the map goes blank like this:

Huh, I remember encountering this before...

Quadmesh would be better, but using an image seems to work for now. Thanks @ahuang11 for your help!


Failed to connect to server on first connect

I try to connect to mlab database by using the link provided when i run thehttp://localhost:3000/api/feedbacks then i got this error bellow from the console
MongoError: failed to connect to server [ds113566.mlab.com:13566] on first connect [MongoError: connect ETIMEDOUT <IP_ADDRESS>:13566]
at Pool. (C:\Users\Barry\Desktop\TaxiMoto\server\node_modules\mongodb-core\lib\topologies\server.js:336:35)
at emitOne (events.js:96:13)
at Pool.emit (events.js:188:7)
at Connection. (C:\Users\Barry\Desktop\TaxiMoto\server\node_modules\mongodb-core\lib\connection\pool.js:280:12)
at Connection.g (events.js:292:16)
at emitTwo (events.js:106:13)
at Connection.emit (events.js:191:7)
at Socket. (C:\Users\Barry\Desktop\TaxiMoto\server\node_modules\mongodb-core\lib\connection\connection.js:187:49)
at Socket.g (events.js:292:16)
at emitOne (events.js:96:13)
[nodemon] app crashed - waiting for file changes before starting...
so any help

Did you get help ?
I have the same problem since 5 days

I am also facing the same problem

i just found out mongojs is not connecting to mlab has anyone found a way to fix it


[DO NOT MERGE] Hdfs-17531 rebase #7176

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
12m 13s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  1s

The patch appears to include 16 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 38s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
32m 56s

trunk passed

+1 :green_heart:
compile
18m  8s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 19s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 35s

trunk passed

+1 :green_heart:
mvnsite
3m 51s

trunk passed

+1 :green_heart:
javadoc
3m 14s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 33s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
6m 58s

trunk passed

+1 :green_heart:
shadedclient
38m  4s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 31s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 20s

the patch passed

+1 :green_heart:
compile
16m 55s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
16m 55s

the patch passed

+1 :green_heart:
compile
16m 12s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 12s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 20s

root: The patch generated 0 new + 76 unchanged - 26 fixed = 76 total (was 102)

+1 :green_heart:
mvnsite
3m 55s

the patch passed

+1 :green_heart:
javadoc
3m  9s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 39s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
7m 30s

the patch passed

+1 :green_heart:
shadedclient
36m 23s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 27s

hadoop-common in the patch passed.

+1 :green_heart:
unit
2m 54s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
31m 35s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt
hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
1m  5s

The patch does not generate ASF License warnings.

307m 34s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterClientRejectOverload

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7177

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 41bdce45853b 5.15.0-125-generic #135-Ubuntu SMP Fri Sep 27 13:53:58 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / ea3c4c84ba278089485d9242e3a1d8e2e9cf3e90

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/1/testReport/

Max. process+thread count
3898 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 16 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m  6s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
31m 56s

trunk passed

+1 :green_heart:
compile
17m 32s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 22s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 25s

trunk passed

+1 :green_heart:
mvnsite
3m 57s

trunk passed

+1 :green_heart:
javadoc
3m 15s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 41s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
6m 51s

trunk passed

+1 :green_heart:
shadedclient
35m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 32s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 21s

the patch passed

+1 :green_heart:
compile
16m 49s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
16m 49s

the patch passed

+1 :green_heart:
compile
16m 24s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 24s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 17s

root: The patch generated 0 new + 76 unchanged - 26 fixed = 76 total (was 102)

+1 :green_heart:
mvnsite
3m 56s

the patch passed

+1 :green_heart:
javadoc
3m  7s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 37s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
7m 34s

the patch passed

+1 :green_heart:
shadedclient
36m 22s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 22s

hadoop-common in the patch passed.

+1 :green_heart:
unit
2m 51s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
31m 39s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt
hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
1m  6s

The patch does not generate ASF License warnings.

292m 22s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterClientRejectOverload

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7177

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 1c20eb09fd60 5.15.0-125-generic #135-Ubuntu SMP Fri Sep 27 13:53:58 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk /<PHONE_NUMBER>9b0dcd2b0dde697ef96c961325d54a

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/2/testReport/

Max. process+thread count
3799 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 16 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
15m  8s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
31m 43s

trunk passed

+1 :green_heart:
compile
17m 25s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 15s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 28s

trunk passed

+1 :green_heart:
mvnsite
3m 53s

trunk passed

+1 :green_heart:
javadoc
3m 16s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 44s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
6m 58s

trunk passed

+1 :green_heart:
shadedclient
37m  6s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 32s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 18s

the patch passed

+1 :green_heart:
compile
16m 56s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
16m 56s

the patch passed

+1 :green_heart:
compile
16m 13s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 13s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 16s

root: The patch generated 0 new + 76 unchanged - 26 fixed = 76 total (was 102)

+1 :green_heart:
mvnsite
3m 57s

the patch passed

+1 :green_heart:
javadoc
3m  5s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 41s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
7m 34s

the patch passed

+1 :green_heart:
shadedclient
36m 36s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 21s

hadoop-common in the patch passed.

+1 :green_heart:
unit
2m 52s

hadoop-hdfs-client in the patch passed.

+1 :green_heart:
unit
31m 49s

hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
1m  7s

The patch does not generate ASF License warnings.

293m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7177

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 0f67fcd334c4 5.15.0-125-generic #135-Ubuntu SMP Fri Sep 27 13:53:58 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / ea3c4c84ba278089485d9242e3a1d8e2e9cf3e90

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/3/testReport/

Max. process+thread count
4276 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 16 new or modified test files.

_ trunk Compile Tests _

+0 :ok:
mvndep
14m 59s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
32m  4s

trunk passed

+1 :green_heart:
compile
17m 33s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
compile
16m 14s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
checkstyle
4m 28s

trunk passed

+1 :green_heart:
mvnsite
3m 55s

trunk passed

+1 :green_heart:
javadoc
3m 11s

trunk passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 34s

trunk passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
6m 57s

trunk passed

+1 :green_heart:
shadedclient
36m  8s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 33s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
2m 20s

the patch passed

+1 :green_heart:
compile
17m 51s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javac
17m 51s

the patch passed

+1 :green_heart:
compile
16m 11s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
javac
16m 11s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
4m 17s

root: The patch generated 0 new + 76 unchanged - 26 fixed = 76 total (was 102)

+1 :green_heart:
mvnsite
3m 52s

the patch passed

+1 :green_heart:
javadoc
3m  7s

the patch passed with JDK Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04

+1 :green_heart:
javadoc
2m 43s

the patch passed with JDK Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

+1 :green_heart:
spotbugs
7m 35s

the patch passed

+1 :green_heart:
shadedclient
36m  6s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
20m 21s

hadoop-common in the patch passed.

+1 :green_heart:
unit
2m 53s

hadoop-hdfs-client in the patch passed.

-1 :x:
unit
31m 44s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs-rbf.txt
hadoop-hdfs-rbf in the patch passed.

+1 :green_heart:
asflicense
1m  6s

The patch does not generate ASF License warnings.

293m 10s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterClientRejectOverload

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/7177

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux a917975667cf 5.15.0-125-generic #135-Ubuntu SMP Fri Sep 27 13:53:58 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f7ad2fca1791fc65a2778c68e8bc6b75e9cc70ec

Default Java
Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.25+9-post-Ubuntu-1ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_432-8u432-ga~us1-0ubuntu2~20.04-ga

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/4/testReport/

Max. process+thread count
3651 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-hdfs-project/hadoop-hdfs-client hadoop-hdfs-project/hadoop-hdfs-rbf U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-7177/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.


Fix CVE-2024-7254

chore: Fix CVE-2024-7254:
Overview
com.google.protobuf:protobuf-java is a Google's language-neutral, platform-neutral, extensible mechanism for serializing structured data.
Affected versions of this package are vulnerable to Stack-based Buffer Overflow via the parsing of nested groups or series of SGROUP tags as unknown fields with DiscardUnknownFieldsParser or Java Protobuf Lite parser, or against Protobuf map fields. An attacker can cause infinite recursion by sending malicious Protocol Buffer data.

cc @njhill

Thanks @spolti


Coinzoom - Incorrect fees calculation

Describe the bug
Fees calculated when trades are executed using Coinzoom is incorrect. Currently it uses a fee of 0.2(maker) and .26(taker) on fees being deducted instead of 0.36(maker) and .44(taker). Fee reference can be seen here:
https://www.coinzoom.com/fees/
Steps To Reproduce

Run a pure_mm bot using coinzoom
Execute trades and compare fees from the client and exchange website

Screenshots

Release version
development-0.44.0
Attachments
Logs, config and csv: coinzoom.zip
┆Issue is synchronized with this Clickup task by Unito

Closing ticket since Coinzoom doesn't provide fee information thru WebSocket. We will continue using fee estimates (https://github.com/CoinAlpha/hummingbot/issues/2084#issuecomment-9925999332)

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://github.com/Jostino/RimWorld-it/releases/tag/fix
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


.close() will trigger a reconnection after socket is closed.

With fastClose being true, which is the default case, calling .close() after socket is closed will trigger reconnection.

Hi, I've just released a major version of the library. It's a full rewrite and many bugs have been fixed.
Please, if you still reproduce the problem, reopen the ticket.


CI CD Fixed

Fixed an issue with the first attempt at CI/CD. Script reference wrong

I hope

I hope


[SPARK-20773][SQL] ParquetWriteSupport.writeFields is quadratic in number of fields

Fix quadratic List indexing in ParquetWriteSupport.
I noticed this function while profiling some code with today. It showed up as a significant factor in a table with twenty columns; with hundreds of columns, it could dominate any other function call.
What changes were proposed in this pull request?
The writeFields method iterates from 0 until number of fields, indexing into rootFieldWriters for each element. rootFieldWriters is a List, so indexing is a linear operation. The complexity of the writeFields method is thus quadratic in the number of fields.
Solution: explicitly convert rootFieldWriters to Array (implicitly converted to WrappedArray) for constant-time indexing.
How was this patch tested?
This is a one-line change for performance reasons.

Can one of the admins verify this patch?

ok to test

Can you also make sure that we do not use a Seq for struct writing?

Test build #76983 has started for PR 18005 at commit e13ce9f.

Test build #76983 has finished for PR 18005 at commit e13ce9f.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76983/
Test FAILed.

Merged build finished. Test FAILed.

It might be nice to explicitly use the type IndexedSeq[ValueWriter] for rootFieldWriters (up on line 61 of this file) since that would capture the intent behind using an Array and would maybe help prevent regression during refactoring.

Yeah, I can change that - I do hate the standard IndexedSeq implementation (Vector) though, and want to make sure that the collection is actually a WrappedArray.
I've actually done more than make a one-line change using the Github UI now, and will update with performance benchmarks + a successful build.

LGTM pending jenkins

Addressed comments.
I tried to get some benchmark stats for this code:
spark.read.csv(text_file).write.mode('overwrite').parquet(parquet_path)

I wanted to see the performance improvement for files with various numbers of columns/rows that were all 1.5G. However, I didn't see much of a difference with <30 columns and catalyst blew up when I tried ~50 columns (I wanted to go up to several hundred)

What do you mean by catalyst blew up?

I used this script to generate random CSV files:
import uuid
import sys

try:
    print('args = ' + str(sys.argv))
    filename = sys.argv[1]
    cols = int(sys.argv[2])
    rows = int(sys.argv[3])
    if len(sys.argv) != 4 or cols <= 0 or rows <= 0:
        raise RuntimeError()
except Exception as e:
    raise RuntimeError('Usage: gen_text_file.py <filename> <cols> <rows>')

rand_to_gen = (cols + 7) / 8

with open(filename, 'w') as f:
    f.write(','.join('col%d' % i for i in range(cols)))
    f.write('\n')
    for i in range(rows):
        if (i % 10000 == 0):
            print('wrote %d lines' % i)
        rands = [x[i:i+4] for i in range(8) for x in [uuid.uuid4().hex for _ in range(rand_to_gen)]]
        f.write(','.join(rands[:cols]))
        f.write('\n')

I generated files that were all the same size on disk with different dimensions (cols x rows):
10x18M
20x9M
30x6M
60x3M
150x1200K
300x600K
Here's what I tried to do to them:
>>> spark.read.csv(text_file).write.mode('overwrite').parquet(parquet_path)

The 10, 20, 30-column files all took between 40s to 1m to complete on 2 cores of my laptop. 60 and up never completed, and actually crashed the java process -- I had to kill it with kill -9.
At one point for the 60-column table, I got a "GC overhead limit exceeded" OOM from the parquet writer (the error suggested that parquet was doing something silly trying to use dictionary encoding for random values, but I haven't figured out how to turn that off). I could be conflating this crash with one we encountered a few months ago, where Spark crashed because Catalyst generated bytecode larger than 64k for dataframes with a large schema.

Others on my team suggest that the >64k bytecode issue has been fixed already (and ported to a 2.1 maintenance release as well)

Test build #77018 has started for PR 18005 at commit 72c5487.

Test build #77018 has finished for PR 18005 at commit 72c5487.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77018/
Test PASSed.

LGTM - merging to master/2.2. Thanks!


Fix product name

Microsoft Flow -> Power Automate

@changeworld : Thanks for your contribution! The author(s) have been notified to review your proposed change. @preetikr

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change. @preetikr

@ecfan The author @preetkir needs to join the Microsoft Docs domain so they can receive notifications and reviewer/assignee status when their article is updated in the public repo. Thanks!

The update looks good; verified. Thanks @changeworld


Excercise 6 Calling Exception rather than printing any string

Summary
I have passed level1 and finished modification of level2 and 3, however, when I try to run the code in jshell, it returned only Exception but not any String, did anyone face the same problem? Appreciate for any help!
Code Snippets

Screenshots

you may want to look at the video prof uploaded on canvas by the videos/panopto folder and digest it

you may want to look at the video prof uploaded on canvas by the videos/panopto folder and digest it

okie thanks a lot! I have dealt with the prb
but tbh the vedio is just like a model answer and makes my previous 1 day thinking like a joke hhh


Enhance highlights tree style

Resolves #2635 to differentiate between various levels of a tree grid. Supports up to 6 levels in light and dark mode.
Adds new Colors and Colors w/Highlights tree styles.
Toolbox PR to add new tree styles to GridOptions.js: https://github.com/xh/toolbox/pull/532
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[N/A] Added CHANGELOG entry, or determined not required.
[N/A] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[N/A] Updated doc comments / prop-types, or determined not required.
[N/A] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

Petra, I converted this to a DRAFT (and removed WIP from the name) according to our new standards.
But let us know if you have updated this to the latest and greatest and want to try and push this over the top.  I think its a clear improvement.  We can always tweak the colors and  the API change seems quite minimal.
I guess one question to answer is, if an app  did want to tweak the colors, how straightforward would that be?

I am seeing the group borders on my local machine for those 2 w/Border options in the example. I'm not sure why you can't!


Provide API for extacting type of the v option used in dashboard Flux scripts

The platform allows for users to define variables which are then accessible in any Flux script via the v option.
For example I can define a variable bucket that is backed by a query to get buckets and when writing a dashboard query I can reference that variable like so:
from(bucket:v.bucket)
   |> range(...)
   |> filter(...)

The v option is defined externally by the UI when the query is submitted.
In order to export a dashboard with all of its dependency we need to know which variables a given Flux script references otherwise we do not know which variables to include in the export.
This can be done using Flux's type system, we can infer the type of v and then determine which variables (i.e. fields) of v were accessed. This gives us an exact and robust method for determine which variables are referenced from within a Flux script.
The API for such a function would be something like:

// Find all the referenced variables on the `v` option in the Flux script.
func FindVariableReferences(script string) []Variable

The implementation details are open but the general way I expect we would approach this is define the option v as a type variable with a constraint that it is a record and then let type inference determine what fields are required on v. Then we can map those fields back to variables defined in the platform.

After some further investigation we have learned that the UI (i.e. browser) will also need access to a similar API endpoint. The UI wants to know which variables are being referenced in a given query so it can prompt the user to provide the value for those variables.
As such we need to determine a design that allows us to expose this API to both Go code and the node module. Naturally the implementation will live in Rust, which is fine since all of the logic we need already lives on the Rust side.
The questions then revolve around where and how to expose this to both systems:

Does it make sense as part of the libflux API or is it too specific?
Should we create an influxdb specific wrapper around the flux node module that contains this InfluxData specific logic? Or should it all live in the flux node module for simplicity?
Should we make the API a bit more generic so that its more generally applicable? Instead of specific to the needs of the v option.
How do we encode type information? Currently the serialization of type information has remained a private detail of the API, this would expose it to some degree depending on how we decide to design the API.

Should we make the API a bit more generic so that its more generally applicable? Instead of specific to the needs of the v option.

I wonder if we should center this new API around the use of options?  Since they are a special part of Flux where we can allow clients to learn about them before execution, and we provide a way for clients to set them.  The action of the new API could be build a map of all the options used in the Flux script file, and their types.

How do we encode type information? Currently the serialization of type information has remained a private detail of the API, this would expose it to some degree depending on how we decide to design the API.

Would it be crazy to return something like a FlatBuffers representation of map[[2]string]MonoType, where the map is keyed on [package,option_name]?  Note that there is FlatBuffers target for JavaScript, which could allow use in the browser.
Otherwise we would need some other serialization of the option types, which can be arbitrarily complex.

NOTE: The scope for this issues does NOT include exposing the new API to the UI via WASM or the network. We will have a separate issue to determine a design for how to expose these types of API to the UI.

AIUI:

We need to be able to extract options from any Flux script so that Template authors can sensibly parameterize Tasks
We need to be able to extract options for a Task so that we can store the options without running the script
The ui needs to be able to extract Platform Variables used in Dashboard scripts (Platform Variables just being a special case of options)

Further, I believe that the UI team would like to extract the Platform Variables/Options via the wasm, though I am not certain why they need to extract Platform Variables from scripts at all.

We decide to build the following libflux API
// given a flux AST, and the name of the variable to look for, we return a flatbuffer containing the type of
// the designated variable.
struct flux_error_t *flux_find_var_type(struct flux_ast_pkg_t *, const char* vname, struct flux_buffer_t *)

I want to add some more depth to the above updates.

We need to be able to extract options from any Flux script so that Template authors can sensibly parameterize Tasks

Correct, this issue will cover that use case https://github.com/influxdata/influxdb/issues/17386

We need to be able to extract options for a Task so that we can store the options without running the script

Correct, this issue will also cover that use case https://github.com/influxdata/influxdb/issues/17386

The ui needs to be able to extract Platform Variables used in Dashboard scripts (Platform Variables just being a special case of options)

Correct, this current issue solves that problem, hence why it is being proposed as part of the libflux API (i.e. something we can expose via WASM)
Finally there is one more piece missing but the UI will also like to be able to interact with options, for that see https://github.com/influxdata/influxdb/issues/18834
@yzhang1991 This means that part of the DOD for this issue is to expose the flux_find_var_type function via the NodeJS WASM module we publish.


@ngtools/webpack Lots of duplicate modules

Bug Report or Feature Request
- [X] bug report
- [ ] feature request

Versions.
node v.6.10.2
npm v.4.5.0
Windows 10 1703 build 15063.138
Repro steps.
// webpack.common.js
        new CommonsChunkPlugin({
            name: 'polyfills',
            chunks: ['polyfills']
        }),
        new CommonsChunkPlugin({
            name: 'vendor',
            chunks: ['main'],
            minChunks: module => /node_modules/.test(module.resource)
        }),
        new CommonsChunkPlugin({
            name: ['polyfills', 'vendor'].reverse()
        }),

//webpack.prod.js
    new AotPlugin({
      tsConfigPath: "./tsconfig.aot.json",
      typeChecking: false,
      entryModule: helpers.root("src/app/app.module#AppModule")
    }),

The log given by the failure.
In the resulting bundles' stats I see many shared modules present in each lazy route chunk (so far 41!). The same does not happen when AoTPlugin is not present (dev build).
Desired functionality.
Trying adding one more CommonChunkPlugin instance in the production configuration yields no effect. Additionally, in the past we used to have an "extras.aot.ts" file for AoT build to manually include (via require.include) to list all hint from Webpack Analyze tool that actually worked. In latest releases it seems not to have any effect anymore.
Mention any other details that might be useful.
N/A

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
Can you provide a repository with a simple repro where this happens? It's also worth mentioning that, if you're only using the AoTPlugin in production, then you're just not using the same build process in dev.

I am seeing my shared @NgModules() show up in both my lazy chunks and my eager chunks.  It definitely started happening after moving off of angular2-load-children-loader and into @ngtools/webpack.  I validated this with a git bisect and was able to find the commit that caused it.
Is there a setting to ensure that shared @NgModules() don't get placed into lazy and eager chunks?

Same for me w/ ng5 + ngtools 1.9.8. Previously, I used a custom AoT w/ ngc + webpack, but the recent ng5+ changes to the compiler emit have led me to try this.

Let's see what happens here once webpack 4 merges into CLI.  I think that one will fix this any many other problems


Target Agora SIM identity

At some point we are going to be including bootstrapping creds on the SIMs we ship with Agora kits.
Any thoughts on how we might integrate support for that for this example?
Or should it just have it's own example?

Internal Jira reference: https://jira.arm.com/browse/IOTCLT-3852

@40Grit that's a nice feature, but probably should be included as part of Mbed OS and make it available in the configuration but it shouldn't require changes in the example app.
Anyway, that will take some time.
In the meantime, if you have a solution available that may require customization of the example app, I'd suggest you introduce it in your own example.


Layout event create

added administrator.html and createEvent.html

The layout admin page and the event creation page are created with nice code!
One suggestion to give: I prefer external CSS more than inline CSS.
You can consider my suggestion.

The code is well-structured and easy to read.
The layout is user-friendly, especially the date and time inputs, which are great.


How to custom llm in langchainjs?

compared to py version: https://python.langchain.com/docs/modules/model_io/models/llms/how_to/custom_llm
How can I custom llm in js version?

We should definitely add docs for this!

I also need this. Is there any update?

The above is accurate @ansarizafar. Will leave this open until we have a guide ready but that should be enough to get started!

Here's my example. It's a little simple. It also demonstrates calling an API to retrieve answer.
import { LLM } from 'langchain/llms/base'

export default class MyLLM extends LLM {

  constructor() {
    super({})
  }

  async _llmType() {
    return 'My LLM'
  }

  async _call(text) {

    const url = process.env.LLM_API_URL
    const data = {
      prompt: text,
      max_new_tokens: 250,
      temperature: 0.7,
    }

    const response = await fetch(url, {
      method: 'POST',
      body: JSON.stringify(data),
      headers: {
        'Accept': 'application/json',
        'Content-Type': 'application/json'
      }
    })

    const obj = await response.json()

    return obj['results'][0]['text']
  }

}

And usage:
const llm = new MyLLM();

Does this work well with other langchainjs features such as Tools?

Can it be used with streaming as well?

We've added a guide for this:
https://js.langchain.com/docs/guides/extending_langchain
Thanks for the enthusiasm!

We've added a guide for this:
https://js.langchain.com/docs/guides/extending_langchain
Thanks for the enthusiasm!

how set the API_KEY and URL, could you show a case ?


BackdropFilter on LinearPercentIndicator's background feature

Is there a way currently?
Even by editing the code. I would need to blur what's behind the widget's background only not the colored bar itself.

Hmm what about using Stack widget
On Sun, Aug 18, 2019, 7:33 AM Danaus<EMAIL_ADDRESS>wrote:

Is there a way currently?
Even by editing the code. I would need to blur what's behind the widget's
background only not the colored bar itself.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aeyrium/bezier-chart/issues/43?email_source=notifications&email_token=ABFL3UHDOGEQE2WUBUTDXCTQFE6SZA5CNFSM4IMSVH2KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HF2SDAA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFL3UFCNWUBT22HDAVFNJLQFE6SZANCNFSM4IMSVH2A
.


Enhancement: Dashboards routing

Please consider to add angular routing for dashboards. Right now all of them have url /dashboard and in case if some of us have auto page refresh in browser it is always loading first dashboard after refresh even if another has been opened. That will be nice to have dashboard id in url also.

@manusovich Thanks for reporting this. we does not have url with dashboard id. however dashboard was designed to store last selected dashboard in local cookies and displays selected one, until we logout. But it is not working. we have to fix this.
https://github.com/mycontroller-org/mycontroller/blob/0.0.3-alpha1/src/main/package/www/controllers/dashboard.js#L27


[Enh]: Use offical astral-sh/setup-uv@v2 github action and enable caching in the CI

We would like to learn about your use case. For example, if this feature is needed  to adopt Narwhals in an open source project, could you please enter the link to it below?
No response
Please describe the purpose of the new feature or describe the problem to solve.

We should use an official github action to install uv onto the running CI platform
We can speed up the uv install CI step by caching the dependencies, since requirements-dev.txt does not change so often.

Suggest a solution if possible.
...
        with:
          python-version: ${{ matrix.python-version }}
-     - name: Install uv (Unix)
-       if: runner.os != 'Windows'
-       run: curl -LsSf https://astral.sh/uv/install.sh | sh
-     - name: Install uv (Windows)
-       if: runner.os == 'Windows'
-       run: powershell -c "irm https://astral.sh/uv/install.ps1 | iex"
+     - name: Install the latest version of uv
+       uses: astral-sh/setup-uv@v2
+       with:
+         version: "latest"
+     - name: Enable caching
+       id: setup-uv
+       uses: astral-sh/setup-uv@v2
+       with:
+         enable-cache: true
+         cache-suffix: "reqs-cache"
      - name: install-reqs
...

If you have tried alternatives, please describe them below.
No response
Additional information that may help us understand your needs.
No response

sounds good, thanks - were you interested in making a pr with that suggested change?

Yeah, thanks, I'll draft a PR on these changes. I also need to read on cache-suffix, and see if I need a unique suffix for each workflow.


Allow environment variables to execution of flatc command

Allow setting of LD_LIBRARY_PATH to reach execution environment of flatc so it can load correct version of libstdc++ and so avoid build failure when built with non-system gcc
Fixes #46549

@googlebot I signed it!

@googlebot I signed it!

Is it proper to apply the change to the r2.4 branch as well?

For those suffering from the flatc issue with other branches, a manual fix of the following change works.
https://github.com/tensorflow/tensorflow/pull/46550/files/60aea05061a01b18ff02582341ab374abf253437

Please open a cherrypick PR and tag me.


Updated functions

Coverage remained the same at 89.952% when pulling cc69f004ad4898a017535f040ebf54161a6a74db on carloscanova:master into 5b9f75b1f44a0168eb79ba74e09b490883bfe890 on NeuralEnsemble:master.


Feature: Add more tests

We should add more tests (demos) to test everything added in #17 and all features that existed before #17.

Test all sort of property configuration (unreliable, reliable, interval, onchange, join, one shot)
Add test for buffered values by adding stuff to the buffer before starting the game (Text box that adds any message to the text)

I'm going to set up a MDFramework-Test repo that we can use for CI and examples (#24)
This should maybe be done there.

Moved to https://github.com/DoubleDeez/MDFramework-Examples/issues/1


Add cross validation for item-based cf

Based on our conversation today with Dr. Chi we will want to do K-Folds cross validation with K=14.  for each user in the test set we will randomly select an attribute as y and attempt to predict it.  I can take this on.  I'll try to get to it at some point this weekend.  I have an algo exam next week on Tuesday, so worst case I'll work on this after that.

So I've started on this tonight, I used what you've done so far, but there's some weirdness.  There was a use of a generator in utility.py for the gen_chunks function that messed up a bunch of other stuff, so I just switched it to a list.  I tried just putting dots in to show progress, but I'm not sold that it's really useful to do that.  I tend to like having something that shows whether a program is hung or not.
I made enough other small changes to the point that on each fold the program should spit out some output to indicate the ratio of successes to the number of test elements.  It takes a long time to run each one prob 5-10 minutes.  I just ran the first one for testing purposes and got an odd value though: 65%.  Not sure completely what to think of that.
Probably more bugs in there.

This is mostly done and now includes a logger for running test cases.  As soon as #10 is closed I think we can run these and finish this out.  I have a node on Digital Ocean running right now that I'm running the tests on.

I think this is completely done.  Let me know if anyone sees any issues.  If not I'll close.

Closing this.


Replace overrides in kernel recipes with machine/SoC settings

I put these changes together to make it easier to add a new target and to add bbappends to customize these recipes where needed, and to do some cleanup to eliminate redundant code and make version changes detectable at build time when SRCREV changes (by using SRCPV in PV).
I've been able to test these changes with my db410c, but couldn't test with others as I don't have the hardware.

thanks for the patch.. I really like where it's going. i tried to make a few comments. let me know what you think.

Thanks for the review.  I'll take a whack at making the changes more incremental and see how to address the specific issues you pointed out.

Closing this, as I've started on a new, more incremental, series.


Fix HTTPoxy

https://httpoxy.org/#mitigate-haproxy There is no use for the Proxy http
header, and is a security issue.

Old haproxy versions are crapping out on this so I'm just going to stop trying.


Add a new required permission to bypass read-only repository protection

Closes #7207

@nopcoder @guy-har This is only a draft of course, but wanted to know what you think about this direction.

prefer not to mix the two. if we like to model read-only as permission we should not pass it as a flag on operation.

prefer not to mix the two. if we like to model read-only as permission we should not pass it as a flag on operation.

@nopcoder why? I think it makes sense. If I'm ain't wrong, in Github, for some of the actions you need both- special permissions and use a force flag.

@arielshaqed I Forgot to tag you. Any thoughts?

in GitHub it is part of the branch protection option. to allow force for admin.

in GitHub it is part of the branch protection option. to allow force for admin.
ok I see other actions which come with permissions - like force push pr. this is what you thought we should model the force flag?

@nopcoder I've meant for the Bypass branch protections permission which is different from the Push commits to protected branches permission.

in GitHub it is part of the branch protection option. to allow force for admin.
ok I see other actions which come with permissions - like force push pr. this is what you thought we should model the force flag?

@nopcoder I've meant for the Bypass branch protections permission which is different from the Push commits to protected branches permission.

@idanovo so it is part of the branch protection configuration - there is no special permissions ACL/RBAC that enable you to bypass branch protection, you have configuration that enable or require confirmation/force to bypass branch protection (in this case it will check your are part of admin group)

in GitHub it is part of the branch protection option. to allow force for admin.
ok I see other actions which come with permissions - like force push pr. this is what you thought we should model the force flag?

@nopcoder I've meant for the Bypass branch protections permission which is different from the Push commits to protected branches permission.

@idanovo so it is part of the branch protection configuration - there is no special permissions ACL/RBAC that enable you to bypass branch protection, you have configuration that enable or require confirmation/force to bypass branch protection (in this case it will check your are part of admin group)

@nopcoder So it's different from the current behavior of read-only repositories in lakeFS.
You only need write permissions to the repository and no special permission is needed to use the Force flag.

@idanovo my concern in adding permission vs branch protection is the behavior in which I can create a repository with a read-only flag but I can't force write to the repository.
What do you suggest to allow for existing roles in terms of creating a repository and updating data on the repository.

@idanovo my concern in adding permission vs branch protection is the behavior in which I can create a repository with a read-only flag but I can't force write to the repository.
What do you suggest to allow for existing roles in terms of creating a repository and updating data on the repository.

@nopcoder We have the same behavior in lakeFS today! the fs:CreateRepository permission doesn't give you permission to commit to or delete the repository you created.
I don't think it's an issue, we have to give the right permissions to the right roles.

@idanovo true. but we map the permission to action. when I commit I need permission based on the action.
if we map force to the same principal it means we need to create one for each operation - CommitForce.
Guide the uses that these are different operations.

not saying that having a single permission to perform Force will not work. but you don't know which future actions will require force and having multiple permission checks can make it hard working with this feature.
as it currently implemented it enables safety for the caller without limiting the action by permission. so unless we need another level of control I don't see why we like to add it now.


minus sign in property name.

Hi. I have to consume a JSON that has a minus sign in one of the name fields. example:
{
"task-id": "df994c9b-bb5d-4081-a9d9-788face71efb",
"result": {
"commission": 0.0
},
"state": "SUCCESS"
}
can i override the field name? I'm using Delphi 7.
Thanks.

Did you manage to solve your problem?


Option to set LDAP server port

Add an option to set the LDAP server port.

We will be keeping it default to 1389, there isn't really any reason to change it. Thanks.


Show online player

Built server structure and can now see online players

A new user left a comment. This user must be approved by a Netlify team owner before comments can be displayed.
Approve this user


TrafficPortal DNS Bypass TTL storing empty string

When I add something to DNS Bypass TTL and then delete it. It will end up with 5xx internal server error when I do submit to create Delivery Service.

This is because the create/delete delivery service api can't handle empty strings where an integer is expected. The API should convert empty strings to null so it can insert a null into the database instead. This should be done globally in my opinion.

or better yet, the api should reject empty strings where integers are expected.


Reusing Tracer across your code disables patching boto3

What were you trying to accomplish?
I am have my shared code include @tracer but i don't want to double patch boto3 etc..
Expected Behavior

tracer = Tracer(auto_patch=False) should not disable patching in the main lambda handler
Current Behavior

When i go into the console now of the boto3 activity appears
# main.py
from aws_lambda_powertools import Tracer
from app import shared

tracer = Tracer(service="payment")

@tracer.capture_lambda_handler
def handler(event, context):
     pass

# shared.py
from aws_lambda_powertools import Tracer

tracer = Tracer(auto_patch=False)

Possible Solution

I think because the shared.py - tracer = Tracer(auto_patch=False)  code is called before the main.py. The patching has already been disabled.
Environment

Powertools version used:  1.1.1

That’s exactly it Michael - Could you try removing auto_patch from your
shared?
Under the hood, a module is only patched once by X-Ray SDK as it uses the
importlib package.
I’ll leave this open to add a clarification note in the docs about this
On Sat, 15 Aug 2020 at 05:02, Michael Brewer<EMAIL_ADDRESS>wrote:

What were you trying to accomplish?
I am have my shared code include @tracer but i don't want to double patch
boto3 etc..
Expected Behavior
tracer = Tracer(auto_patch=False) should not disable patching in the main
lambda handler
Current Behavior
When i go into the console now of the boto3 activity appears
main.pyfrom aws_lambda_powertools import Tracerfrom app import shared
tracer = Tracer(service="payment")
@tracer.capture_lambda_handlerdef handler(event, context):
pass
shared.pyfrom aws_lambda_powertools import Tracer
tracer = Tracer(auto_patch=False)
Possible Solution
I think because the shared.py - tracer = Tracer(auto_patch=False) code is
called before the main.py. The patching has already been disabled.
Environment

Powertools version used: 1.1.1

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/issues/108, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBGD7VEEB7EREACBVJLSAX3CZANCNFSM4QABHJ4A
.

Ok. Then I must have misunderstood the docs. I thought if you don't want to double patch you turn it off in the shared code but it will patch in the main handler file.
Currently I don't include it and it works. I guess it is a matter looking closely at the docs.

Nope, it’s on us - I had a second look at the docs and the wording isn’t
right; I‘ll fix it
On Sat, 15 Aug 2020 at 06:54, Michael Brewer<EMAIL_ADDRESS>wrote:

Ok. Then I must have misunderstood the docs. I thought if you don't want
to double patch you turn it off in the shared code but it will patch in the
main handler file.
Currently I don't include it and it works. I guess it is a matter looking
closely at the docs.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/issues/108#issuecomment-674349127,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBHCQW32UW4MCPZA47LSAYIH7ANCNFSM4QABHJ4A
.

@michaelbrewer When you can, could you please review the docs statement to make sure it's clearer?

Just released :) https://awslabs.github.io/aws-lambda-powertools-python/core/tracer/#reusing-tracer-across-your-code


Tracking: document --prod and add minification flag

We should document what --prod really does.
It's actually a sort of meta-flag that adds a bunch of defaults<EMAIL_ADDRESS>Besides setting these defaults, it also adds minification and a few more things<EMAIL_ADDRESS>Minification should perhaps be opt-in however, via a flag.

I was able to figure out that --prod seem to have done AOT and -environment=prod but wasn't sure what else it did.
The really shocking part was actually that I find the documentation missing and that filled me with worry.
Thanks for clarifications on twitter.

@filipesilva we can add that in build command wiki ?

@sumitarora I'm adding the docs bit in https://github.com/angular/angular-cli/pull/6232, can you add the minification flag while you're fixing the UglifyJS problems you are working on?


alt and title tags

Is it possible to get correct alt- and title-tags får the image?

@Nannica Sorry for the late response. This feature should be included in version 2.2.0


[POC PYTHON API] Implement read_network

Tickets:

51219

@jiwaszki Can you please review this PR? It seems I have no permissions in this repo to add you to the reviewers.


Initial team compass docs

Borrowed from the Jupyter Server Team Compass repo.
This PR doesn't name any team members. That can happen in a separate PR.

Pinging @tonyfast, @jtpio, and @afshin for review.

Oops! Thanks @willingc and @jtpio. I didn't mean to checkin the _build dir. It looks like I lost the .gitignore at some point, which would have ignored that directory. I'll amend the last commit to drop that directory from the history.

Looks nice,  @Zsailer! Thanks for pushing on this one!

This page is now live!
https://jupyter-notebook-team-compass.readthedocs.io/en/latest/index.html


SFC generate wrong code when use onUpdate:value as defineProps key

Vue version
3.3.1
Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-qly3ag?file=src%2FApp.vue,package.json&terminal=dev
Steps to reproduce
Expected "}" but found ":"
  4  |    __name: 'App',
  5  |    props: {
  6  |      onUpdate:value: { type: Function, required: true }
     |                    ^
  7  |    },
  8  |    setup(__props: any, { expose: __expose }) {

What is expected?
Generate correct code
What is actually happening?
Generate wrong code
System Info
No response
Any additional comments?
I want to extends Select component of ant-design-vue, then I try to use like this:
import type { SelectProps } from 'ant-design-vue'

export interface ApiSelectProps extends SelectProps {
  api?: () => Promise<SelectProps['options']>
}

const props = defineProps<ApiSelectProps>()

Workaround:
import type { SelectProps } from 'ant-design-vue'
import { selectProps } from 'ant-design-vue/es/select'

export interface ApiSelectProps extends SelectProps {
  api?: () => Promise<SelectProps['options']>
}

const props = defineProps({
  ...selectProps(),
  api: {
    type: Function as PropType<NonNullable<ApiSelectProps['api']>>,
  },
})

have the same problem

I found this issue earlier, don't know if this PR will fix it

I found this issue earlier, don't know if this PR will fix it

https://deploy-preview-7803--vue-sfc-playground.netlify.app/#eNpNjLEOwjAMRH/F8lJY6F5CJBZmFrYsUWNQpOBYjZOl6r+TiqXLnU5371a8i1xaJZzQlHmJolBIq0Dy/Lk51OLQOg70jkzPJUsxq2OAIfNLgleamk+VhgkelWeNma+ON3s6dzPj/7HzPSh9JXWgJwATYrNm3HXfHTrcfipHNDU=


Bokeh optimizations

This PR is meant to provide various optimizations for the bokeh backend. In particular we have the static data optimization in the bokeh backend preventing the data source from being updated and sent to the frontend. However since styling options may still change, the get_data method still has to be executed to modify color mappers and other style-data mappings. However certain data columns are guaranteed to never change and therefore can be skipped. This can provide significant improvements particularly for large paths, and geoviews geometries.

Other than one comment above, it looks fine to me and the tests are green. Is this PR ready to review/merge?

I think I'm going to go through and look for other opportunities for optimization, so not quite yet.

Might actually take this opportunity to remove one of the older optimizations which was the empty to BokehPlot.get_data methods.

@jlstevens Ready to merge once tests pass.

Tests are green. Merging.


Update feedback-support.md

Add "https://" to URL : aka.ms/automation-kit-issues

Learn Build status updates of commit 78d3878:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/guidance/automation-kit/feedback-support.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


fix button calculation on draft edit

When clicking on the "Edit" button on the UI, the buttons are not greyed out.
Reason is that our handler was only registered for "READ" events so far.
Unfortunately, we would also need to change the implementation of the handler as the query options ($select and $expand) for actions are evaluated in the protocol layer and therefore the format of the data in the after-handler is flat.
@danjoa I don't like that but that is the only way I can think of for now.
Maybe FE sent an additional GET request before or it has never worked before.
Alternatively, we could annotate a side effect to trigger the additional GET request.

@rbrechter Do you know which UI test fails here?

Not yet, from the console logs in the builds it looks like the journey passes without error. There seems to be another issue outside the UI tests

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: johannes-vogel:x: rbrechterYou have signed the CLA already but the status is still pending? Let us recheck it.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 5a7a4ae, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 10526 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in adb3fd2.


fix: repeat period starts when job deletion starts

What this PR does / why we need it:
Currently the repeat period starts after the ImageJob is deleted. If the repeat period is 24 hours, and the ImageJob delay is 24 hours, the job will only repeat after 48 hours. This PR fixes that to have it repeat after the 24 hours.
Which issue(s) this PR fixes (optional, using fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when the PR gets merged):
Fixes #
Special notes for your reviewer:

LGTM, needs a rebase and then merge when tests pass

Codecov Report

Merging #586 (d0715e4) into main (3c40e98) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #586   +/-   ##
=======================================
  Coverage   15.43%   15.43%
=======================================
  Files          13       13
  Lines        1600     1600
=======================================
  Hits          247      247
  Misses       1325     1325
  Partials       28       28

Flag
Coverage Δ

unittests
15.43% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


any update on hash-buster?

hello , i'm getting request timed out with hash-buster.. looks like some of the services used are down or no longer active?
will there be any updates?

Check #35


Dropdown for years in GSoC

As years go by the GSoC page will keep on increasing and will be very lengthy. We can have each year as a list hidden and clickable which upon clicking unveils the list. By default the latest year would be unveiled on load. Each dropdown will be like an accordion.
Idea is to something like this (not the design) just a reference for the concept.

THis should also be done for Hall Of Fame.

@kpbot claim

@abhigyank @vsvipul
I made the changes in #119, kindly review them.

Yes @kpbot, the issue has been addressed in #119 and now waiting for a review and merge by @kartik1000.

This issue need to be closed.


Incorrect types is reported when there is no new line between types

I just installed the extension, but it doesn't appear to show the correct type when there are no new lines between types.
In this example, the wrong type is displayed next to each of the types.
type IsNumber<T> = T extends number ? true : false;

type A = IsNumber<undefined>; // => type A = false
type B = IsNumber<null>; // => type A = false
type C = IsNumber<number>; // => type B = false
//   ^? type B = false
type D = IsNumber<number | string>; // => type D = boolean
type E = IsNumber<0>; // => type D = boolean
type F = IsNumber<0 | 1>; // => type E = boolean
type G = IsNumber<0 | "0">; // => type F = boolean

However, when new lines are added the issue is resolved.
type IsNumber<T> = T extends number ? true : false;

type A = IsNumber<undefined>; // => type A = false

type B = IsNumber<null>; // => type B = false

type C = IsNumber<number>; // => type C = true
//   ^?
type D = IsNumber<number | string>; // => type D = boolean

type E = IsNumber<0>; // => type E = true

type F = IsNumber<0 | 1>; // => type F = true

type G = IsNumber<0 | "0">; // => type G = boolean

TypeScript version: 5.1.3.
Visual Studio Code version: 1.79.2.
Extension version: 1.2.0.
I have inlay hints shown when I hold Ctrl + Alt.
EDIT(1): Shouldn't the TypeScript language server implement inlay hints for type definitions. I couldn't find an option in my Visual Studio Code settings, and I couldn't find any relevant issues on the TypeScript repository. I guess I should raise my own issue?
EDIT(2): To clarify, I did find your issue to upstream this extension. However, couldn't TypeScript just support type A /* = false */ = IsNumber<undefined>; in a similar manner that it works for parameters, variables, etc. The twoslash query is not required, but could still be provided by this extension.

@orta This seems to have been a regression in version 1.2.0.
Previously //^? comments worked as expected in VSCode in WSL Ubuntu.
After, I'm seeing a lot of incorrect results that seem to be the same root cause as those @scottwillmoore  described. I'm confused about all the inlay hints stuff as that seems unrelated to me, but this version does seem to be broken at the moment.

Yeah, this seems like a bug with the lookup code changes in https://github.com/orta/vscode-twoslash-queries/pull/23 to me 👍🏻


Release KV cache memory occupation when not needed

Motivation
This PR depends on https://github.com/sgl-project/sglang/pull/2586, thus there are much more code in the diff panel than its actual content.
Please see https://github.com/sgl-project/sglang/issues/2542

Modifications

Checklist

[ ] Format your code according to the Contributor Guide.
[ ] Add unit tests as outlined in the Contributor Guide.
[ ] Update documentation as needed, including docstrings or example tutorials.

@merrymercy Thank you! I posted in https://github.com/sgl-project/sglang/issues/2542#issuecomment-2563168115

I will temporarily close this since https://github.com/sgl-project/sglang/issues/2542#issuecomment-2563641647 seems more promising :)


unify collapsiples toggle button (#4286)

Pull request opened by the merge tool on behalf of #4286

Merged in e419d7f7ced4e358a4dc250fe65f6d9f411d72f1


Pagination not working on Route53 ListResourceRecordSetsAsync

Expected Behavior
listResourceRecordSetsResponse.NextRecordIdentifier should have value when there are records left to process.
Current Behavior
Pagination Not working
listResourceRecordSetsResponse.NextRecordIdentifier always returns null even though there are records left to process.
Steps to Reproduce (for bugs)
           var hostedZonesResponse = await _amazonRoute53.ListHostedZonesAsync();
            foreach (var hostedZone in hostedZonesResponse.HostedZones)
            {
               // Test pagination
                var r = new ListResourceRecordSetsRequest(hostedZone.Id) {MaxItems = "2"};
                do
                {
                    var listResourceRecordSetsResponse = await _amazonRoute53.ListResourceRecordSetsAsync(r);
                    r.StartRecordIdentifier = listResourceRecordSetsResponse.NextRecordIdentifier;
                   Process(listResourceRecordSetsResponse.ResourceRecordSets)
                } while (!string.IsNullOrEmpty(r.StartRecordIdentifier));
            }

Your Environment

AWSSDK.Route53 version used: 3.3.13

.NET Core Info

.NET Core version used for development: 2

This seems like an issue on the service side. If I can repro it, I'll let the Route53 team know.

It actually looks like you are misusing the API.  You probably want to use NextRecordName instead of NextRecordIdentifier:

IsTruncated
A flag that indicates whether more resource record sets remain to be listed. If your results were truncated, you can make a follow-up pagination request by using the NextRecordName element.

NextRecordIdentifier
Weighted, latency, geolocation, and failover resource record sets only: If results were truncated for a given DNS name and type, the value of SetIdentifier for the next resource record set that has the current DNS name and type.


dictate python functions from list

right now I only have a short list of python functions, once I get approval for the structure of this me and the rest of python users can continue adding to it

I’m thinking we could make these commands generic (programming.talon) with context-specific captures for supported languages.
This would be similar to the initial snippet support
https://github.com/knausj85/knausj_talon/blob/master/apps/vscode/snippets/csharp_snippets.py
https://github.com/knausj85/knausj_talon/blob/master/code/snippets.py
The capture could probably be declared in code.py
I’m happy to help with this, if it sounds reasonable.
A step towards making the ideas here
https://github.com/ma-anwar/knausj_talon/blob/master/code/c.py
generic and cross-language

I would love to extend this to other programming languages, but I'm a little lost on how to do it so your help would be greatly appreciated. I also liked the idea of selecting the functions by index!

@RodrigoChT  - I took a first pass at making this something that might work reasonably across languages. Sorry for the formatting changes. I'm switching the depot to using black for formatting needs, so a lot of noise in this one 👎
Hopefully you don't mind my use of the same PR!
@fidgetingbits @barrcodes  - you two might be interested in taking a peek at this.
Meat of the idea is really in these two, with whatever supporting code in code.py:
https://github.com/knausj85/knausj_talon/blob/de6c69a22e3a2e9b8900f400a4bc4177733d3ada/lang/csharp/csharp.py
https://github.com/knausj85/knausj_talon/blob/de6c69a22e3a2e9b8900f400a4bc4177733d3ada/lang/python/python.py
I would be to potentially extend this to include common libraries and types for each language as appropriate. Haven't had enough time to think about those just yet.
toggle fun: user.code_toggle_functions()
fun <user.code_functions>: user.code_insert_function(code_functions)

indeed black certainly adds a lot of noise at first, but totally worth it :D
I like this idea, and certainly goes well with the recent C stuff I hope to PR soon. I think the  selection window idea will be really useful too
Since you mentioned types, one the thing I have been wanting to do with types is have the ability to toggle between default types used by the C stuff, since for instance stdint (ex: uint8_t) vs default types (unsigned char) are two common used but different typing styles across many code bases (i constantly work on different codebases). originally I tried to do this with just updating a talon setting, but it seems like you can up to a setting on the fly, so I guess we need some way of maybe standardizing non talon setting 'settings' for different programming languages. may be suitable for a different issue if you want me to file it, but I thought I'd raise it since you mentioned types here if you plan on thinking about it. I suspect this might be something ran into other languages (cpp, windows (DWORD, etc)), so may also be worth having some quasi generic method of selecting the default type set

Is that changing the typeset something that could be done cleanly per-project? I have my doubts, but...
For example, the default type list could be the std_*, or whatever’s considered vanilla C. Then for projects using some other set, the list could be changed via a more specific python context.
So a python file with something like, plus a modified list.
ctx.matches = r"""
mode: user.c
mode: command
and code.language: c
tag: ctypes_fancy

Then, elsewhere opt-in:
title: \some project\
title: \some other project\
-
tag: ctypes_fancy

This looks great, what has to be done before it is ready to be merged?

I think with my latest changes, it's pretty reasonable and ready to go in. Types and libraries and stubbed in code.py once we're ready for those.
I switched to copy and paste for these, so we can hopefully reliably wrap selections without fighting the IDE(s).
toggle funk: user.code_toggle_functions()
funk <user.code_functions>:
    old_clip = clip.text()
    user.code_insert_function(code_functions, "")
    clip.set_text(old_clip)
funk cell <number>:
    old_clip = clip.text()
    user.code_select_function(number - 1, "")
    clip.set_text(old_clip)
funk wrap <user.code_functions>:
    old_clip = clip.text()
    edit.copy()
    sleep(100ms)
    user.code_insert_function(code_functions, clip.text())
    clip.set_text(old_clip)
funk wrap <number>:
    old_clip = clip.text()
    edit.copy()
    sleep(100ms)
    user.code_select_function(number - 1, clip.text())
    clip.set_text(old_clip)


Paket add writes invalid XML namespace into project file

Description
Paket 3.36.0 adds invalid namespace to project file element.
Repro steps

netcoreapp or netstandard app in F#
Add some dependency, e.g. paket add nuget Argu project MyProject
Build project.

Expected behavior
Build should run successfully. Choose tags in fsproj should not have xmlns attribute.
Actual behavior
You get:
error MSB4066: The attribute "xmlns" in element <Choose> is unrecognized.
because Paket creates choose tags like this:
<Choose xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
Known workarounds
Remove xmlns attribute manually.

yoe need to use Paket 4 beta for netcoreapp


On-Prem Password Complexity GPO Guidance

I'm curious what the recommendation is for "Password must meet complexity requirements" in the on-prem group policy.  Should we leave this enabled or disable it after Azure Password Protection has been implemented?

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

ID: bd810b22-4e2a-5a66-db90-dd39b1b5c565
Version Independent ID: 8e36df09-f7f1-dcbd-dc3d-93cbeed1b040
Content: On-premises password protection FAQ - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-password-ban-bad-on-premises-faq.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

Thanks for the ask  . At this time we are reviewing the ask and we will reply back as soon as we can .

@jvplopper the best practice is to have it Enabled. The caveats, individual configurations, and best practice details are covered in this article. https://docs.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/password-must-meet-complexity-requirements