[Bug]: When puppeteering a remote browser running a different OS, page.uploadFiles does not work

Bug description
Steps to reproduce the problem:

Puppeteer a remote browser on a different OS.  (in my case my script is running on Linux, and puppeteering Windows).
Try to upload a file.
Fail.

The issue are the follwing lines in page.uploadFile
*const files = filePaths.map((filePath) => {
if (path.isAbsolute(filePath)) {
return filePath;
}
else {
return path.resolve(filePath);
}
})
path.isAbsolute() is not OS independent.  If you are on Linux, and pass C:\path\file, it returns false.   So then we resolve, and return something like
/home/user/dir/C:\path\file
Puppeteer version
14.3
Node.js version
16.13
npm version
8.5.0
What operating system are you seeing the problem on?
Linux, Windows
Relevant log output
No response

This is a pretty niche use case, but it would be nice to be able to somehow turn this check off.

@jrandolf - you are a machine, my friend!  Amazing work.


Revert "Issue#512, Empty area showing up before report"

Reverts loklak/loklak_webclient#524

Fomer version was successful building and now has issues with travis as it cannot connect for some reason. I am aborting this revert.


Rewind in debug controls

Currently, the speed options for debug mode go from 0x (paused) to 5x.  It would be nice to have a rewind option, as well.
The implementation would be trivial for animations, where you just want to subtract from the current time every frame.  For non-debug simulations, this is impossible anyway, so the slider should only go to zero.
Debugged simulations and interactions are the hard case.  We know the past states, but we don't know their times.  If we did know the times, this would be doable, but complex.  (The new debug state type would have to contain two lists of paired state-and-timestamp, and a current timestamp: ([(a, Double)], [(a, Double)], Double)  This also opens the question of whether you want the history slider scaled by frame, or by time.

FYI @kammitama5 : Chances are you won't be able to get to this, but it relates to your code, so I'm tagging you in case you have thoughts.


Falcon40b prefill CI coverage

Current CI coverage:

1 layer tests for seq len 128 and 2048 with Bfp8_b
1 layer tests for seq len 128 and 2048 for FP16 (PR waiting for approval)

We need to add 60 layers test to track e2e correctness and notice possible hangs. I am investigating what the options are.

https://github.com/tenstorrent/tt-metal/tree/sjovic/falcon40b-ci-e2e
I prepared 60 layer e2e tests to add to the unstable multi device CI pipeline. Haven't tried it out yet since we might need to do some weights/binaries initialization before trying out the test since 60 layers were never used on the CI. @tt-rkim should give some info on this.
cc @pavlejosipovic if you have time to tackle this next week


[Bug]

What happened
The topic of kafka is successfully created, but an error is displayed
What you expected to happen
The topic of kafka is successfully created, but an error is displayed，org.apache.kafka.common.errors.UnsupportedVersionException: Topic metadata and configs in CreateTopics response not supported，maybe is result.numPartitions() == CreateTopicsResult.UNKNOWN
How to reproduce
The topic of kafka is successfully created, but an error is displayed，org.apache.kafka.common.errors.UnsupportedVersionException: Topic metadata and configs in CreateTopics response not supported，maybe is result.numPartitions() == CreateTopicsResult.UNKNOWN
Environment
docker_compose
InLong version
1.4.0
InLong Component
InLong Dashboard
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@GYK123123 please try to use the 1.5.0 branch which has lots of optimizations for Kafka, and it will release recently.


Prepare iOS Home Page prompt

Closes #604

Hey @shelegdmitriy how do you suggest we test this?  I've tried with bos-loader, but nothing is rendered and no errors are thrown at https://beta.near.org/test/widget/NearOrg.Notifications.HomeScreenAlert.
Also, this is the new modal and we'll need the logic which triggers this modal before showing the Enable Notifications modal. Does that already exists somewhere?

Also, this is the new modal and we'll need the logic which triggers this modal before showing the Enable Notifications modal. Does that already exists somewhere?

Yes, you're right. We have to trigger this modal from near-discovery repository as we do for NotificationAlert component. So far there is no such logic. But as far as this PR is merged I can use isIOS variable to create such functionality for iOS devices and there we can test it.

Actually we can make a pause with merging it and let me create such logic on near-discovery side and then make a loom video with an example. @charleslavon WDYT?


VERY VERY VERY BASIC Minesweeper

Took me a ton of time to get the game and square files working. Now all thats left is the sad face, about box, difficulties, acheivements, and pixel perfect (tradmarked). And yes, this is horrible code.

:white_check_mark: Build TimeHACK AppVeyor-361 completed (commit https://github.com/TimeHACKDevs/TimeHACK/commit/32b4797179 by @jtsshieh)


Division by Zero in History, compare.py

Hi,
I'm getting errors when trying to sudo, evidently the history is missing.
File "/lib/security/howdy/compare.py", line 101, in
if float(hist[0]) / hist_total * 100 > float(config.get("video", "dark_threshold")):
ZeroDivisionError: float division by zero
Unknown error: 1
is the backtrace.
I'll try to take a look when I can.
Cheers

Found the issue: If every single pixel of the frame is exactly rgb(0,0,0), the sum of the histogram will be zero. I'll add a fix in the next patch but for now you can add
	if hist_total == 0:
		dark_tries += 1
		continue

on line 100 in /lib/security/howdy/compare.py


fix simplify range result wrong when reference is nullable (#41356)

pick from master #41356
if reference is nullable and simplify result is boolean literal. the real result should be:
IF(${reference} IS NULL, NULL, ${not_null_result})

run buildall


[FP]: CVE-2015-2575 False positive against AWS Aurora MySQL v1.1.2 library

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:mysql:mysql:*:*:*:*:*:*:*:*
CVE
CVE-2015-2575
ODC Integration
No response
ODC Version
7.3.0
Description
Seems to be looking for a different library, entirely

duplicate #5097


fix: add execute permission to script

Codecov Report
Base: 65.34% // Head: 65.33% // Decreases project coverage by -0.02% :warning:

Coverage data is based on head (680709d) compared to base (70de509).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1646      +/-   ##
==========================================
- Coverage   65.34%   65.33%   -0.02%
==========================================
  Files         127      127
  Lines       23118    23118
==========================================
- Hits        15106    15103       -3
- Misses       8012     8015       +3

Impacted Files
Coverage Δ

fedimint-server/src/net/peers.rs
88.62% <0.00%> (-3.00%)
:arrow_down:

modules/fedimint-mint/src/lib.rs
77.55% <0.00%> (-0.12%)
:arrow_down:

modules/fedimint-wallet/src/lib.rs
77.54% <0.00%> (+0.23%)
:arrow_up:

fedimint-server/src/net/connect.rs
94.63% <0.00%> (+0.24%)
:arrow_up:

fedimint-server/src/lib.rs
77.23% <0.00%> (+0.27%)
:arrow_up:

client/client-lib/src/api.rs
79.34% <0.00%> (+0.51%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added a JSON file to track all similar bugs and a script that updates config and markdown automatically

This PR solves this issue: https://github.com/zksecurity/zkbugs/issues/20

lgtm. Can you just move the script into the scripts directory as a runner script and the JSON into the dataset directory?

Fixed, ready to merge

Thanks!


Is it possible to use mirror as the way of private registry?

We have a requirement of avoiding configuring the Docker Daemon with "--registry-mirror" while using Mirror.
For example: we have a mirror address: mirror.abc.com, we want to pull image from it by the method of "docker pull mirror.abc.com/libary/ubuntu".
Is it possible?

This is possible - it's what a private registry is.  If you tag library/ubuntu as mirror.abc.com:5000/libary/ubuntu and push it, the docker engine will attempt to push to a registry on mirror.abc.com port 5000

@RichardScothern, sorry, what I mean is that the 'mirror.abc.com' is not a registry, it is a pull-through-cache instead. We don't push image to 'mirror.abc.com', instead, we pull images from Docker Hub through it.
Typically, for the mirror, we need to configure the '--registry-mirror' in Docker Daemon, then 'docker pull ubuntu' will pull image from it (proxy); however what I am asking is whether it is possible we pull image through the mirror by the way of 'docker pull mirror.abc.com/library/ubuntu' other than configuring '--registry-mirror' in Docker Daemon

mirror.abc.com/library/ubuntu is not the same as library/ubuntu.  The first resides on a host (mirror.abc.com) and the latter is the official image.  library/ubuntu can be cached to another registry but that registry must be specified with the --registry-mirror flag.

Running a private registry as a pull through cache should currently allow getting images with  docker pull mirror.abc.com:5000/libary/ubuntu and the image will show up in the docker cache as mirror.abc.com:5000/libary/ubuntu.  I do not suggest relying on this behavior since the pull through cache is not respecting namespace in this case, requested image in mirror.abc.com:5000/, got image from namespace docker.io/. I consider this a bug and one we want to fix by making the registry more aware of the namespace it is serving images from and caching for. The correct solution which we will continue to support in this project is using --registry-mirror daemon flag.


Animation in README and on RTD is inaccessible

Issue description
The animation referenced in the README and on Read the Docs currently doesn't show up.
Expected result
Seeing the animation.
Actual result
The response from S3 is as follows:
<Error>
  AllAccessDisabled
  <Message>All access to this object has been disabled</Message>
  <RequestId>BB41E4330F22C8C6</RequestId>
  <HostId>7Nm9ixlUsFQEkIJJcU+jdU/6m47oM5m3/SXvB/+hNSTlJsd4AdcaPSl0FLWB/aFHgMPQJrvsJPU=</HostId>
</Error>

Steps to replicate

Go to the main repo page or docs homepage
Scroll to where the animation is suppose to be
Observe the image not loading

Screenshots

On my side, the Github caches the GIF in githubusercontent so it can
display normally, but the GIF is broken on documentation.
Thanks

MS Edge-
window-
works on main repo page
broken on docs homepage
Inprivate window-
works on main repo page
broken on docs homepage

Bromite(https://www.bromite.org/)-
[after delete all cached images and tabs]
tabs-
works on main repo page
broken on docs homepage
incognito tabs-
works on main repo page
broken on docs homepage

main repo page or docs homepage

 Another issue?

testimonial is still visible. on the 8 instances above. https://github.com/pypa/pipenv/pull/3536/files
2 Apr 2019, 5:15 PM[`Asia/Kolkata`]
hope it helps.

[editiing]

Anyone can help this issue? I can't see gif file on homepage.

This issue has the "help wanted" label, but it seems that the error is tied to the AWS account hosting the image. I'd be happy to help if someone could provide guidance on acceptable alternative image hosting (perhaps GitHub via githubusercontent?).

Any update on this?
I noticed the issue too.
I get Invalid upstream response (403) for the 'image' in the README.md

Closed via #3850


Proposal: Add support for XAML codegen in Visual Studio for UWP/WinUI 2 on Modern .NET

Proposal: Add support for XAML codegen in Visual Studio for UWP/WinUI 2 on Modern .NET
Summary
By Modern .NET we mean .NET without builtin WinRT interop support, so .NET 5+.
With recent C#/WinRT developement, Windows.UI.Xaml projections are now possible. Bring full XAML support to it for the full experience.
Rationale
The XAML codegen build task (Microsoft.Windows.UI.Xaml.Build.Tasks) hardcodes to retrieve WinRT API declarations from winmds, which cannot be directly referenced in Modern .NET.
So this proposal updates the official XAML compile build task to properly support Modern .NET, with this we can:

Build System XAML island apps with Modern .NET
Build UWP apps with Modern .NET, targeting UWP-only devices like the Xbox

Scope

Capability
Priority

This proposal will allow developers to build WinUI 2 XAML apps with Modern .NET
Must

Important Notes
It is verified that the XAML codegen works resonably well with Modern .NET if this limitation is somehow worked around. An example is provided at https://github.com/driver1998/ModernNetUAP.XamlCompiler, where it swaps C#/WinRT projections DLL references with winmds during XAML codegen (pass 1), and swaps those back after XAML codegen (pass 2).
A demo app is included at https://github.com/driver1998/NetCoreApp, yes this is an UWP, with AOT support too.
This works because the code we compile in pass 1 will never be executed, only reflected to get type metadata for pass 2.
But these kind of efforts are finicky. We can't have any library that uses WinRT types in projections referenced in a project that need XAML compile, as that will lead to type mismatch during pass 1.

The prayers were answered 🎉
https://devblogs.microsoft.com/ifdef-windows/preview-uwp-support-for-dotnet-9-native-aot/

I guess this issue can be closed now.


Brain JS causes Google Chrome to become unresponsive

What is wrong?
When Brain JS is training, Google Chrome browser tab becomes unresponsive/crashes and only becomes responsive again after training is complete, which could take many minutes.
Where does it happen?
When running in Google Chrome from index.html file
How do we replicate the issue?

Copy contents from code example into a *.html file locally on Macbook: https://codepen.io/sts-ryan-holton/pen/RwweqEX
Run in browser

How important is this (1-5)?
5
Expected behavior (i.e. solution)
The browser tab should remain responsive during training as deploying Brain JS to a production site could cause the whole browser tab to crash on slower systems.
Other Comments

Training is computationally expensive, so you should try to train the network offline (or on a Worker) and use the toFunction() or toJSON() options to plug the pre-trained network into your website.

There are following ways to avoid this:

Train offline and then load exported network: https://github.com/BrainJS/brain.js#json
Use web-workers for training: https://developer.mozilla.org/en-US/docs/Web/API/Web_Workers_API/Using_web_workers

@mubaidr  I've already looked at both parts of the documentation, and have tried using the JSON feature in another issue I've opened and can't see any difference, please provide a code example on how to use this?

In your case it seems like data is not enough. Network is not able to reach error threshold. Please enable  log: true in training option, as specified here: https://github.com/BrainJS/brain.js#training-options


DiamondCutFacet.sol is NOT standard

At https://github.com/mudgen/diamond-1/blob/1.3.5/contracts/facets/DiamondCutFacet.sol#L13
it is referred that this is the // Standard diamondCut external function.
But actually this is NOT standardized. The standard makes no mention that this function be restricted to OWNER usage.

Thanks. I removed the comment.


Add support for migration command to worker

We currently are not able to deploy the latest version of MergeStat to Kubernetes because we have a new dependency between the mergestat and sqlq schemas we introduced in this migration
In k8s we run an initContainer as part of our deployment for the worker and it currently only runs the migrations for the mergestat schema. So, when it runs it currently breaks on deployments that previously did not have the `sqlq' schema added.
To address this we would like to introduce a command we can call from the worker cmd/migrate that will run both the sqlq and mergestat schema migrations.

Closing this out as we resolved by simply removing the init container: https://github.com/mergestat/helm-charts/pull/109 and are now only running migrations as part of the worker startup


Update Global Targeting Params and Ad Targeting Strategy

For this new ad targeting strategy the adzone will be moved out of the ad_path_name and into global targeting params. Additional targeting parameters will also be added to the global targeting structure such as channel, season, etc.

update ad_path_name structure
update global targeting params
update hashtag processing

@gdbrown please verify the changes you requested


node tuning: Fix docs and improve logs

What this PR does / why we need it:
Update the ConfigMap key in the docs from tuned to tuning, as this was missing in PR#1802. Improve logs when wrong key is used or the ConfigMap has no contents.
Which issue(s) this PR fixes:
Related to PSAP-742. Docs fix should have been included in #1802.
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes docs.
[ ] This change includes unit tests. (N/A)

/cc @jmencak @liqcui

@dagrayvid @jmencak Thank you,I will do pre-merge testing and will let you know once I'm done

/retest

@dagrayvid and @jmencak Verified by QE

/assign @enxebre

/lgtm
/approve


Changes to support checkpointing to BlockRDD as described in http://apac...

...he-spark-user-list.1001560.n3.nabble.com/java-lang-StackOverflowError-when-calling-count-td5649.html#a11970

Can one of the admins verify this patch?

You need a JIRA and to update the title here... see https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark

Can one of the admins verify this patch?

Do you mind closing this issue for now? This PR lacks a JIRA and is opened against a really old branch. If you still want to make this change, I'd recommend that you file a JIRA first, then open a new PR against the master branch.

@mtimper please file a JIRA at https://issues.apache.org/jira/browse/SPARK and reopen a new patch against the master branch. Would you mind closing this patch?


HTTP 500 response on save profile

The backend is returning a 500 when updating an existing profile.
It looks like this issue is caused by the backend.

Issue is fixed in backend


add karmor discover filter to specify policy type (network/file/process)

karmor discover --type network

@Ankurk99 I would like to work on this issue.


Display breadcrumbs when any are provided

Story: https://trello.com/c/omwNh8eA/41-breadcrumbs

I'm not sure whether this requires a unit test...


Assistance with basic beginner question

Hello,
I've created a stackoverflow topic here describing a problem. https://stackoverflow.com/questions/48273396/coinbase-api-python-library-getting-started
I'm trying to just get started with this library but it doesn't seam to work anymore?
Any help would be appreciated.
Thanks,
Wayne

Hey @wayneworkman, please try again with version 2.0.7 and open a new issue if you're still encountering problems. Thanks!


Create a development branch

I strongly suggest that you create a development branch for this repo. That way you can develop the next release on the development branch and maintain the master branch for stable releases.

Done: https://github.com/ResidentMario/missingno/tree/dev


Switch from setup.py to pyproject.toml

Primary Problem
The main problem is that https://github.com/j0057/github-release/pull/73 has been merged but has not yet been shipped as a packaged release to PyPI.
Secondary Problem
As a workaround, one might be tempted to install from current master in order to get the needed fix from https://github.com/j0057/github-release/pull/73:
python -m pip install git+https://github.com/j0057/github-release.git

However, the above command yields a fatal error because the version number generated by the (deprecated) setuptools-version-command package required in setup.py does not conform to PEP 440:
    File "/private/var/folders/r7/xxdlb6114759yn1rltsq8nwh0000gn/T/pip-build-env-40g3i5io/overlay/lib/python3.12/site-packages/setuptools/_vendor/packaging/version.py", line 198, in __init__
      raise InvalidVersion(f"Invalid version: '{version}'")
  setuptools.extern.packaging.version.InvalidVersion: Invalid version: '1.5.9-5-geaec743'

Unsuccessful Resolution in setup.py
I spent entirely too much time trying to fix this in setup.py, attempting to replace the deprecated setuptools-version-command package with setuptools-scm, but that just resulted in setup.py yielding a version number of 0.0.0 every time.
Solution: Replace setup.py with pyproject.toml
Ultimately, by far the easiest solution to the Secondary Problem of not being able to install the current master branch of this project was to switch from setup.py to pyproject.toml.
No other changes are present in this pull request, in an effort to keep the scope as focused as possible.

Thanks for the detailed report and contributions 🙏

Glad to help. Thank you for the quick merge! 😁
Echoing the request from December 2023, any chance a new release could be issued containing the fix from #73?

re: release
Back in December @j0057 and I exchanged few emails and agreed on transferring the project to the scikit-build GitHub organization.

I just followed with them about the transfer as well as shared maintenance  of the corresponding PyPi project.
As soon as this is completed, we will be moving forward with publishing a release.
Thanks for your patience 🙏


high cpu usage and iowait when idle

I'm using 2 server grade nvme in raid0 at 30% capcity running op-bnb/geth and op-bnb/node seeing 3% iowait and constant read access when runniung the archive node from the snapshot.
comparing that with optimism rollups of the same datadir size that's concerning as other rollups using hash state scheme and leveldb as backend are basically idle on the disk. I'm running the latest release 0.3.0 from last week.
I doubt that I can just switch out op-bnb for op-geth due to tuned block gas parameters.

another issue is that after a few days of running without issues it stops progressing for me. I checked the logs. no error. went though 4 GB of logs for 3 days. blocknumber is not advancing.

What's your node start-up command?

I use this docker-compose https://github.com/StakeSquid/ethereum-rpc-docker/blob/main/op-bnb.yml which is copied from the reference docker-compose with the variables filled in.

another issue is that after a few days of running without issues it stops progressing for me. I checked the logs. no error. went though 4 GB of logs for 3 days. blocknumber is not advancing.

Could you pls check if there's keyword "Database compacting, degraded performance" in geth log?

The keywords are not in the log
On Wed, 3 Apr 2024 at 15:15, krish-z @.***> wrote:

another issue is that after a few days of running without issues it stops
progressing for me. I checked the logs. no error. went though 4 GB of logs
for 3 days. blocknumber is not advancing.
Could you pls check if there's keyword "Database compacting, degraded
performance" in geth log?
—
Reply to this email directly, view it on GitHub
https://github.com/bnb-chain/opbnb/issues/173#issuecomment-2033870509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDFLZD2MWDYBGPMKWLTI3KLY3O3BRAVCNFSM6AAAAABFN2Y7IKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZTHA3TANJQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

@goldsquid could you paste the recent logs from both op-node and op-geth here so that we can analyze the issue?


Low performance (in 100 times more) Connect/Bind methods in Docker on Win11 with Linux containers

Hi, i am found bug https://github.com/dsbenghe/Novell.Directory.Ldap.NETStandard/issues/216
Possible you have background solution for this issue?

No, sorry, I don't.
Note that I did not really update the lib in quite a while.
This lib was first created to be cross-platform (mainly Linux or WSL2 + windows). The windows part would be to develop, while the Linux would be more for production. I would expect your project to deploy within a production Kubernetes cluster and if it's the case, the best remains in Linux, not Windows.
That said, if you want, you can also rewrite part of this to use the Microsoft lib to interact with the Ldap. At the beginning of this project was meant to show that it was possible to make an easy plugin to interface with Ldap + Identity Server 4.


Tab-Complete for nickname postfix

Hi,
it would be nice to have tab completion with some postfix that can be customized.
e.g. :
I want to type nickname "rsroka" so I will type "rs" and press TAB. Lounge will complete it to "rsroka".
I want to set postfix " :" or ", ". So Lounge should complete "rsroka :" or "rsroka, ".

Duplicate of #1176.


Move away from "global" leader election model

Assigning to Matt based on our discussion.
ContainerRuntime exposes leader election API (leader property, events "leader" and "notleader" events).
This presents "global" leadership, i.e. single point that different tasks are using to assign one client to run a task.
It's "global" because single client would be running all the tasks!
Instead I believe we should refactor code to move it into reusable class, and make LeaderTaskId = "leader" provided by each user, with no defaults, I.e. each sub-system (like spell-checking) would use its own key to select task-specific leader, which allows tasks to be distributes across more clients. While it's not given it will happen right away, at least it allows us to change Scheduler in the future to implement it, without need to change API in the future.
This change has some back compat implications (I believe) that we need to consider.

Moving to next as I'm not planning to work on this in January.

Moving to next as I'm not planning to work on this in January.

I'll be spending time on this in March.

Fixed by a series of changes, finished in #6542.  Specifically TaskSubscription is a good tool for achieving granular task assignment on AgentScheduler, or TaskManager can be used for a queued assignment strategy across granular tasks.


Read Airbrake config from environment

For the time being this file will still be overwritten at deploy time, but once we remove that functionality from govuk-app-deployment this new file will start to be used.

Merge away.


testutil: increase size of buffer for stack dump

Too many goroutines to fit all stack traces in 8kb.
(cf. https://semaphoreci.com/coreos/etcd/branches/master/builds/1403)

lgtm

lgtm

Current coverage is 63.81% (diff: 0.00%)

No coverage report found for master at bc2d471.

Powered by Codecov. Last update bc2d471...6ab8dcb


Can't Visualize filters with "Visualize spatio-chemical filters.ipynb"

On Windows (WSL Ubuntu20.04), constructed environment with conda and run pip install -r requirements.txt, still I stucked some bug relevant to pythreejs.
When I run the 3rd cell from the top, I encountered ValueError like below and visualize non-three-dimensional image (maybe only showed the flag of atom or acids).
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-3-c7d86697239f> in <module>
----> 1 renderer = show_3d_filters.plot_aminoacid_filter(filter_specificities,117,sg=sg);
      2 display(renderer)

c:\...\ScanNet\visualizations\show_3d_filters.py in plot_aminoacid_filter(filter_specificities, index, show_asa, show_ss, show_negative, threshold1, scale, list_additional_objects, sg, camera_position)
    467                                           camera_position=camera_position,
    468                                           key_light_position=[0.5, 1, 0],
--> 469                                           maxi=12,dpi=600,crop=False)
    470
    471

C:\...\ScanNet\visualizations\weight_logo_3d.py in show_ellipsoids(list_ellipsoids, list_colors, list_figures, list_texts, list_segments, list_additional_objects, level, sg, wireframe, show_frame, fs, scale, offset, camera_position, key_light_position, opacity, maxi, xlims, ylims, zlims, download, crop, render, tmp_folder, dpi)
    556         lam, U = np.linalg.eigh(inertia)
    557         sqrt_inertia = np.dot(U, np.sqrt(lam)[:, np.newaxis] * U.T)
--> 558         ellipsoids_V = level * np.dot(sphere_V, sqrt_inertia)
    559         ellipsoids_V = ellipsoids_V.tolist()
    560         sphereG = pythreejs.Geometry(vertices=ellipsoids_V, faces=sphere_F)

<__array_function__ internals> in dot(*args, **kwargs)

ValueError: shapes (0,) and (3,3) not aligned: 0 (dim 0) != 3 (dim 0)

Please tell me how to fix the bug.

I attempted a further challenge to get this code working.
sg = weight_logo_3d.make_sphere_geometry(30)
After running this code again after the error, I no longer get the above error, but the image is not rendered except in the "Visualize atomic neighborhood" cell.

Hi,
Try:

Resetting the notebook.
Running separately the first cell with make_sphere_geometry
Then run the others (do not use run all).

Let me know,
Jerome

On 20 May 2024, at 8:26, L40S38 @.***> wrote:
I attempted a further challenge to get this code working.
sg = weight_logo_3d.make_sphere_geometry(30)
After running this code again after the error, I no longer get the above error, but the image is not rendered except in the "Visualize atomic neighborhood" cell.
—
Reply to this email directly, view it on GitHub https://github.com/jertubiana/ScanNet/issues/10#issuecomment-2119693784, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACHSPJQE4QXRVYDGHOIQLZDZDGCRFAVCNFSM6AAAAABHK224VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJZGY4TGNZYGQ.
You are receiving this because you are subscribed to this thread.

I installed pythreejs extention to jupyter, I  can visualize on jupyter notebook, but can't on jupyter-lab.
I have no idea why failing on jupyter-lab, but I could visualize the all filter, so I'm going to use jupyter notebook from now on.
Thank you for your advice.


Add sound effects to the game

Play sound when:

snake eats an apple
game over

The path to the assets directory must be provided as command line argument for now.

@liweiyi88

@mrnakumar thanks for your PR, it looks cool!
I would make 2 suggestions based on the PR.

Let's use the standard package embed  to load the static file at compile time. Thus we do not need to pass the path argument to load the file at runtime. What you can do is basically create a file in the assets folder and export a var like the below snippet (assets/embed.go)

package assets

import "embed"

//go:embed *
var Assets embed.FS

It would be better to add a command option (--silent) to mute the background music. Afterall, there are some time when people want to play the game without any background sounds.

Please let me know your thoughts.

@liweiyi88 thanks for comments. I have made changes accordingly. Please let me know if this is okay.


windows version?

Not gonna happen unless someone else implements it. Volunteers?

Yeah, a windows version would be nice :)


MultiTenant + AssignPremiumPlan doesn't work

When using the New-NavContainer command, using a combination of '-multitenant' and '-assignPremiumPlan' does not work. The record does not get created for the premium plan.
My full command is below. If I remove the '-multitenant' parameter the user plan is added properly.
New-NavContainer -containerName $containerName                  -accept_eula
-accept_outdated                  -auth NavUserPassword
-Credential $credential                  -doNotExportObjectsToText
-imageName $imageName                  -includeCSide
-licenseFile $licenseFile                  -memoryLimit 6g
-updateHosts                  -useSSL
-alwaysPull                  -shortcuts Desktop
-restart no                  -includeTestToolkit
-assignPremiumPlan `
-multitenant

I have a fix for that - need to get things checked in and published.
Expect next weekend.

version <IP_ADDRESS> has been released


[WIP] PR: Refactor Spyder configuration propagation to use an observer pattern.

Description of Changes
This PR enables the Spyder configuration system to use an observer pattern in order to propagate changes across all the application actions/options/etc. This PR will supersede the current API configuration propagation logic

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: @andfoy

Thanks on tackling this area because I found the old API hard to work with. Where is the new design explained? I am lazy and don't want to search through 100+ commits and files changed. :smile:

@jitseniesen Basically the idea is to register any object that requires updates on a configuration option of a section via CONF.observe_configuration(observer, section, option). If the option is None, the object is notified on any change on the section. Likewise, since now we support tuple options to access nested dictionary keys, an object can subscribe to a partial tuple key. A registered object will be notified via the on_configuration_change(option, section, value) method each time a registered option/section changes via CONF.set.
This PR also introduces two decorators spyder.api.decorators.configuration_observer and spyder.api.decorators.on_conf_change to register automatically methods that receive changes on specific keys, e.g.,
from spyder.config.manager import CONF
from spyder.api.decorators import configuration_observer, on_conf_change

@configuration_observer
class ConfObserver:
    def __init__(self):
        # Register class to listen for changes in all the registered options
        for section in self._configuration_listeners:
            observed_options = self._configuration_listeners[section]
            for option in observed_options:
                CONF.observe_configuration(self, section, option)

    def on_configuration_change(self, option, section, value):
        section_recievers = self._configuration_listeners.get(section, {})
        option_recievers = section_recievers.get(option, [])
        for receiver in option_recievers:
            method = getattr(self, receiver)
            method(value)

    @on_conf_change(section='editor')
    def section_change_reciever(self, section):
          # section contains the new values of the editor section
          ...

    @on_conf_change(section='main_interpreter', option='custom_interpreter')
    def on_interpreter_change(self, interpreter):
          # interpreter contains the value of main_interpreter
          ...

Furthermore, to simplify things, this PR adds a new class mixin spyder.api.mixins.SpyderConfigurationObserver that already implements the aforementioned protocol. By inheriting from this mixin, the previous example can be simplified to
from spyder.api.mixins import SpyderConfigurationObserver

class ConfObserver(SpyderConfigurationObserver):
    @on_conf_change(section='editor')
    def section_change_reciever(self, section):
          # section contains the new values of the editor section
          ...

    @on_conf_change(section='main_interpreter', option='custom_interpreter')
    def on_interpreter_change(self, interpreter):
          # interpreter contains the value of main_interpreter
          ...

FInally, all new actions created using self.create_action of the new API now can specify a section and option that is read and written to, thus they are also subject to the new observer pattern, where they will change their state if a change is done on the configuration option observed.

Hello @andfoy! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/config/types.py:

Line 41:35: W292 no newline at end of file


Unify the blaze API to xxx$ synax

We need to following APIs.

state$
stateFn$
prop$
action$

Done.


Need a way to refresh the frontend of the attachment component

The problem with the current state of things
In certain cases we need to make changes to attachments in backend. E.g. in a form currently being made for Digitale Helgeland, where the user may be required to upload an attachment depending on their answer to a previous question.
If they change their answer after uploading the attachment so that the attachment is no longer required, it should be deleted in backend. However, after the attachment has been deleted, it will still be listed in the attachment component because there is no way to trigger the frontend of the component to refresh.
This can cause problems with confusing error messages and buggy behaviour.
What I want
I want a way to trigger a refresh of the attachment component frontend from backend.
I can refresh other components in e.g. DataProcessingHandler.cs by returning await Task.FromResult(true), but this does not currently work with the attachment component.
Examples of problems
By going back and forth between the question determining weather an attachment is required, my attachment has been deleted in the blob, but it doesn't look like it from a user perspective.
Not only is this confusing to a user, but the validatePage-trigger that should stop the user from progressing without a valid attachment is no longer working as wanted.

The user is stopped from sending in the form, but the error message is confusing for the user because it looks like the attachment is already there.

If there is no summary page or if the attachment component is not included in the summary page, there is no indication of what is wrong. Only the generic error message which provides little help to the user, especially in long forms such as this one.

When navigating back to the page where the failing attachment component is located (after having tried to send it in and gotten an error) the app fails and bluescreens.

Moving to app-frontend-react. I'm assuming this could be implemented by having a trigger that refreshes instance data (and re-maps attachments) after saving updates to that component.

This is now possible, see: https://github.com/Altinn/app-frontend-react/issues/1335#issuecomment-2506004466


feat(paid expense email): different money emoji for EUR and GBP

Coverage decreased (-0.09%) to 70.994% when pulling 2c2a430ec9af42d14df810ed0f15c65b7f960ab0 on moneyEmoji into a1b5006ac9f924b66083795b978a758a5d49d03e on master.


:host styles have specificity issues vs. *

If in my demo I have

And in my element that I import in this demo I have:

The * style always wins (the dev tools actually interprets it as *:not([style-scope]):not(.style-scope)), which means you need to pepper the :host styles with !important.

@notwaldorf, it works the same way under native Shadow DOM.
Styles applied to element from the outside have higher priority, nothing wrong here.
While nesting element inside other element should not propagate styles, and it works exactly the way it should:

<dom-module id="x-host">
	<template>

		<x-nested></x-nested>

	</template>
</dom-module>
<dom-module id="x-nested">
	<template>

	</template>
</dom-module>

<x-host></x-host>

x-host::shadow > x-nested in this example have blue border, not red as everything else.

Yep, :host always loses, because it is meant to give "default" styling, like the User-Agent stylesheet, like
div {
  display: block
}


Remove inventory attrs from interfaces

Inventory attributes should only be available on System objects.

Currently, the only attributes allowed on interfaces are inventory attributes. Should we just get rid of attributes on interfaces altogether?

I think so - I suppose there may be some day when we feel we need to add additional attributes for individual interfaces, but I don't see that at this time.

Resolved by https://github.com/OSU-Net/cyder/pull/914


Delete the content of the downlinkQueue

Hello brocaar,
i want to syncronize the realtime clock from my nodes with a generated downlink packet.
Now i have the problem that sometimes more than one packet is stored in the downlinkQueue.
The result is an old timestamp (the oldest packet in the queue , FIFO) and a wrong clocktime at my nodes.
So, i want to clear the downlinkQueue. I write a curl command direktly at the raspberry pi like this:
curl -X DELETE --insecure --header 'Accept: application/json' 'https://xxx.xx.xx.xxx:8080/api/downlinkQueue/0004a30b001a2c98'
The answer is:
{"Error":"strconv.ParseInt: parsing "0004a30b001a2c98": invalid syntax","Code":2}
after that the queue is not empty.
What is my mistake. Is there a better way to clean the queue?
It is possible to define only one entry size for the queue?
Thank you very much for help and for your nice work.

Hi @developy2, sorry for my late response I overlooked your issue. This might be a misunderstanding of the queue API. Instead of the devEUI, the id of the item in the queue is expected.
Maybe the API can be improved by:

Returning the queue item id after a create
DELETE /api/downlinkQueue/0102030405060708 = delete the queue
DELETE /api/downlinkQueue/0102030405060708/123 delete queue item 123

What do you think?

it should be like this ...Their should be one command to delete all queues related to particular device and selective queue item of that particular device.

Hi, I am wondering if there is any way to delete all message in downlink queue?

Not yet.

Thank you for your reply. I may writer a bash script to help.

I've just released LoRa App Server 0.15.0 + LoRa Server 0.23.0. This release includes a refactor of the downlink queue and also implements an API method to flush the entire downlink queue :-)


Layout issue when click on Account drop-down for pt_BR locale.

Framework Version: 0.1.5

Framework Version: 0.1.5
Fixed


Previous and Next links do not work

The Previous and Next links at the end do not work because there is a blank space between the closing square bracket and the open parenthesis.

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

ID: 7b506236-b211-eb9d-b571-5efecdde76c8
Version Independent ID: 1bb011dd-a559-c0d7-b0cf-984fbb2d4bd3
Content: Use HttpClientFactory to implement resilient HTTP requests
Content Source: docs/standard/microservices-architecture/implement-resilient-applications/use-httpclientfactory-to-implement-resilient-http-requests.md
Product: dotnet
GitHub Login: @CESARDELATORRE
Microsoft Alias: wiwagn

Thanks for your feedback. My PR addresses this issue: https://github.com/dotnet/docs/pull/9232.


Pairwise crypto

Detailed description:
Which issue(s) this PR fixes:
PR enables pair wise crypto
Relevant issue:#408 mostly fixed but nextKey not implemented
Relevant issue:#411 but no logging implemented
Relevant issue:#409 but remove, update not implemented
Fixes #401
Special notes for your reviewer:
Checklist

[ ] Documentation added
[ ] Changelog updated
[x] Readme updated
[ ] Tests updated

Hibernate does not work closing


need support for file groups

While enhancing our pipeline to download Gemfire at the moment it was needed we discovered that the pivnet-resource did not support file groups.
trying to download Gemfire 8.2.1, which I know I can use the PivNet API and goto https://network.pivotal.io/products/pivotal-gemfire#/releases/1753/file_groups/429  where the release # and filegroup # are the key factors...
Gemfire releases several archives per release targeted for various platforms, in development we need to be able to get a specific one, i.e. Pivotal_GemFire_821_b18207_Linux.tar.gz
We are using resource_types:
- name: pivnet
  type: docker-image
  source:
    repository: pivotalcf/pivnet-resource
    tag: latest-final

- name: gemfire-current
  type: pivnet
  source:
    api_token: {{token}}
    product_slug: pivotal-gemfire
    product_version: 8.2.1
    access_key_id: {{aws-access-key-id}}
    secret_access_key: {{awe-secret-access-key}}

- name: build-linux-debug
  serial: false
  plan:
  - do:
    - aggregate:
        ...
      - get: gemfire-current
        trigger: false

@echobravopapa thanks for this. Currently the pivnet-resource doesn't have any concept of file_groups - just product_files attached directly to the release.
I've created the following stories to allow you to download files in file-groups, as well as optionally filter the file-groups you want to consider:

pivnet-resource should download files in file-groups
pivnet-resource should support filtering file-groups when downloading files

I will close this issue when those stories are ready.

@echobravopapa the latest release will consider files in all file groups as well as directly attached to the release.
I defered the second story - please let me know if you think filtering on specific file groups wil be valuable to you.
Please let me know if you have any feedback on this feature!
Thanks,
Rob

Bringing @addisonhuddy into the conversation


Child attendance

We need a solution for child attendance. I need more details about how this is done, how it is used or reported, etc.

For now we're not going to implement this due to liability questions.


className copy(default delete space)

复制  Btn100  默认删除 了 className 左边的空格 这是不好的
Copying Btn100 defaults to deleting spaces on the left side of className, which is bad

@jackdizhu Not sure if I understand the issue. Can you please elaborate more

With detailed steps to reproduce
A video would be more helpful

@jackdizhu Please provide details here.

1. select active className  Press  Ctrl + C

2. After className content Press space key ,

and  Press  Ctrl + V.

3.The result is this

3. There is no space between content and active.

understand . . .

If you don't know, then I don't know how to describe it.
I would choose to use sublime.
Wait for someone else to submit this problem until you fix it.

@jackdizhu  Thank you for the detailed steps, but I could not reproduce. Are you perhaps using a specific keyboard layout where the problem occurs?
Here is me doing the steps:

@rebornix any ideas. Possibly related - #16647

I can't reproduce this on Windows 10, both IE11 and VS Code. @jackdizhu do you mind providing gif to help us troubleshoot? Can you share your extensions list or run code --disable-extensions and see if you can still run into this problem?

ok! This configuration may be the cause.


Provide QuarkusIntegrationTest for major features

Is your feature request related to a problem? Please describe.
Providing QuarkusIntegrationTest allows to test that the native executable. It is very useful to detect bad packaging of resources.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
first one (and very minimal) test added here https://github.com/KaotoIO/kaoto-backend/pull/338

Listing for api module:

Class
Status
Issue
Comments
Priority

CatalogWarmingUpExceptionTest

CapabilitiesResourceTest
Done
#426
CapabilitiesResourceIT
High

DeploymentsResourceTest

IntegrationsResourceTest

StepResourceTest
Done
#336
StepResourceIT
High

ViewDefinitionResourceTest

DeploymentServiceTest

KameletBindingDeploymentGeneratorServiceTest

LanguageServiceTest

GenericViewDefinitionParserServiceTest

Should be possible for other modules but need to investigate first and maybe having the tests are the API level would be enough?

Unfortunately there was no activity but it is a really important task for the maintainability and quality of the software. Reopening. I will let the people that will work on this task checking all link substasks if they need to be reopened or if they were closed because really implemented.


Callback is not triggered in iOS.

I implemented the plugin and used the example. All the settings for Android and for iOS were implemented as written in the docs. In Android it works great but in iOS the callback function is never triggered. Any idea?

Are you testing on real device or on simulator?
Receiving location on iOS depends on many things. Actually there is no interval for getting location like in android, in iOS the os decide when to send new location.

Tested on both simulator and real device. Not once the callback was triggered.

I'm experiencing the same issue....Any idea on how to solve this?

Hi, I have the last version, in Android works but in iOS don't work the callback, I read the wiki many times, I don't know why the error.


Added AF_XDP input/output. Added ctrl-C handling

This is not ready-to-use code - this is a proposal. Questions to solve:

libbpf is now a dependence. It should be downloaded (git clone https://github.com/libbpf/libbpf.git) and then installed in default location. How should we handle this?

AF_XDP setups BPF program to sockets. This program is not removed after finishing application, no new AF_XDP can be setup. This patch introduces handling application interruption by ctrl-C and release all used AF_XDP after this. However framework can fail with an error in other places. In this case user should explicitly run command "bpf_set_link_xdp_fd(interface, -1, XDP_FLAGS_UPDATE_IF_NOEXIST);" to release socket.

Current implementation is like "one AF_XDP functionality per port". AF_XDP queues are single producer/consumer. For receive send to one port we should introduce combined function. We need some documentation for this - no API documentation currently available.

This patch is combination of DPDK patch https://git.dpdk.org/dpdk/tree/drivers/net/af_xdp/rte_eth_af_xdp.c and Linux example samples/bpf/xdpsock_user.c which is under GPL license - need to do something with this.

This patch was created and tested with Linux kernel 5.1 (not released yet)

This patch doesn't contain error handling - need documentation for this

AF_XDP has a little bit unstable fluky behavior 5% cases. Wasn't investigated at this stage.

Can we submodule libbpf along with dpdk?

AF_XDP sockets support was implemented in pull request #643


is possible to use a cookie ?

Will be possible to use a cookie instead passing user and password?
Thanks

From v0.0.7 there is already an implementation of the cookie mechanism using a file "ig.json" to avoid re-login.
🥂


Merge pull request #1 from openstack/master

update master

Thank you for contributing to openstack/networking-l2gw!
openstack/networking-l2gw uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Problem with Client().database().getRelease()

Hi!
I have recently started to get multiple errors of type UnhandledPromiseRejectionWarning while using Problem with Client().database().getRelease(), even though I do have a catch block in place in my code.
This is the error:
(node:43691) UnhandledPromiseRejectionWarning: DiscogsError: Unknown error.
    at IncomingMessage.passData (/<path-to-my-project>/node_modules/disconnect/lib/client.js:212:23)
    at IncomingMessage.emit (events.js:187:15)
    at endReadableNT (_stream_readable.js:1094:12)
    at process._tickCallback (internal/process/next_tick.js:63:19)
(node:43691) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4)

And this is my code:
public getRelease({ id }): Promise<any> {
  return new Promise((resolve, reject) => {
    const db = new Client(this.token).database();

    db.getRelease(id)
      .then(response => resolve(response))
      .catch(error => reject(error));
  });
}

Am I doing something wrong? This has been working for months, it's only recently that I started to get this error ocasionally.
Thank you!

Actually, I was missing a catch() on my top function, and now I was able to identify the error as follows:
{ DiscogsError: Unknown error.
    at IncomingMessage.passData (/Users/jeromefaria/Work/dscgrm/node_modules/disconnect/lib/client.js:212:23)
    at IncomingMessage.emit (events.js:187:15)
    at endReadableNT (_stream_readable.js:1094:12)
    at process._tickCallback (internal/process/next_tick.js:63:19) statusCode: 502, message: 'Unknown error.' }

Apparently, Discogs is occasionally throwing a 502 for unknown reasons.

@jeromefaria - I'm glad I saw your post, as I'm also getting a lot of 502 errors and couldn't make sense of it since nothing has changed in my script. Is this a known issue / is there a thread you can point me towards?
Thanks!

@jeromefaria, you may be interested in this update. I've heard back from Discogs staff regarding recent uptick in 502s. Although, they didn't share details they told me that they were aware of the issue over the last few days, but that things should have stabilized as of this afternoon.
Thread on Discogs

Thanks, @michaelck! It does seem to have stabilised in the last few days; at least I didn't notice that problem anymore.
Closing this as it appears to have been an issue on Discogs' side all along.


Error: missing required field "serviceName"

Hi! I'm trying to install Flink on my Kubernetes cluster with this command:
helm install my-flink --namespace my-namespace riskfocus/flink --version 0.1.9
And I'm receiving the error below:
Error: unable to build kubernetes objects from release manifest: error validating "": error validating data: ValidationError(StatefulSet.spec): missing required field "serviceName" in io.k8s.api.apps.v1.StatefulSetSpec

How can I get rid from this error?

Hi @marquesds!
It was an issue in this version. Fixed in 0.1.10.

@vkropotko I've tried to install the 0.1.10 version but I'm still having an error, not the same as above, but the error below:
Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "ServiceMonitor" in version "monitoring.coreos.com/v1"


Include a basic CRUD in template

I don't have background with web components yet, but it would be ideal and useful if there is a sample application such as the TodoMVC that includes both Server/Client which has create, edit, delete functionality. I'm trying to implement it but I'm still reading through and understanding how it all works.
Maybe instead of just calling a simple rest API for displaying Cats, I believe it would be useful as a starter to have a Cat forms page that shows how it all connects.

I wrote a blog post tutorial that creates a todo with update using Fable.Remoting + Fable.React + SQLProvider. Does this help?
https://jordanmarr.github.io/fsharp/safe-stack-ssdt-starter/

It does. However, it's confusing to which one to use. Feels like if I'm using Lit then I'd mix it with Grapnel router library. If I want to use Elmish with Lit then what do I when picking router library? Do I want to mix it with react and leverage the Feliz library? I just want to kick start and get to writing code that builds the web app, and not having to connect the dots for different Fable libraries. I don't care if it's opinionated or not, but I believe if I'm using Lit, I really don't prefer to mix it with React (even if I could).

You should not be calling ChangePage ListCatFacts |> dispatch  in your click event handler. That should happen when initializing the Grapnel routes.
The click event should just call router.navigate:
<sl-button href="#" @click={fun _ -> router.navigate("/cat-facts")} variant={navLinkIsActive ListCatFacts} outline>

If you want to use Elmish instead of Hook.useEffectOnce, you should just initialize Grapnel router within the Elmish init.
Maybe something like this:

let init () =
    // init routes
    router.get("/", fun _ -> setCurrentPage Welcome)
    router.get("/cat-facts", fun _ -> setCurrentPage ListCatFacts)
    router.get("/cat-fact/:fact", fun (req: Req<{| fact: string |}>) -> setCurrentPage (Page.ViewCatFact req.``params``.fact))

    // init model
    let model = { ... }
    model, Cmd.none

Actually you can still initialize the Grapnel router using Hook.useEffectOnce even if you are using Elmish:
    Hook.useEffectOnce(fun () ->
        router.get("/", fun _ -> dispatch (SetCurrentPage (Page.Projects None)))
        router.get("/projects", fun _ -> dispatch (SetCurrentPage (Page.Projects None)))
        router.get("/projects/:client", fun (req: Req<{| client: string |}>) ->
            dispatch (SetCurrentPage (Page.Projects (Some req.``params``.client)))
        )
        router.get("/project-dashboard/:id", fun (req: Req<{| id: string |}>) ->
            dispatch (SetCurrentPage (Page.Dashboard req.``params``.id))
        )
        router.get("/users", fun _ -> dispatch (SetCurrentPage Page.Users))
    )

That's how I ended up doing, but do you mind to share how will we implement a parent child elmish in conjunction with Lit components?

I would just wrap the parent dispatcher in a function and pass it to the child as a command. Would that work for your scenario?

It sounds like you have two options:
1) Componentized Hook.useElmish
Your parent subscribes and updates its own model with latest message. Then the child component is passed the updated model.Message.
match model.CurrentPage with
| Page.ChildPage -> ChildPage.Page(model.Message)

Then within your child component, you would need to watch for the change like this:
[<HookComponent>]
let Page(webSocketMessage: string) =
    let model, dispatch = Hook.useElmish(init, update)

    Hook.useEffectOnChange(webSocketMessage, fun webSocketMessage ->
        dispatch (HandleWebSocketMessage webSocketMessage)
    )

2) Global Elmish
If you have a lot of child components that rely on shared state, then you may want to go back to using a global Elmish loop instead of using componentized Hook.useElmish.

I ended up doing Global elmish. :(
Do you have a sample how to do a quick one with <form>? Do I have to define all Msg if I go that route? I am aware of the Fable.Form library but I can't grok how to define my own custom http handler, so I just want to use a pure HTML from Fable with Lit.

Since there is not currently a useContext hook, I wonder if you could do something as simple as storing the shared context in a mutable static binding and in your child components with
    Hook.useEffectOnChange(AppContext.ctx, fun ctx ->
        dispatch (UpdateSignedInUser ctx.Username)
    )

I ended up doing Global elmish. :(
Do you have a sample how to do a quick one with <form>? Do I have to define all Msg if I go that route? I am aware of the Fable.Form library but I can't grok how to define my own custom http handler, so I just want to use a pure HTML from Fable with Lit.

Don't create a message to set every field -- you can do that but it's obviously a lot of clutter and maintenance hassle.
I just create a single message for updating each entity.
type Msg =
    | SetVehicle veh
    | SetOccupant occ
    // etc...

and then let the input changed event select the actual property:
fun e -> dispatch (SetVehicle { model.Vehicle with PlateNo = e.target.Value })

I don't use any form libs because they are confusing to me, and I like to keep things simple.
Also, I will be adding form validation to the template soon.

Same here.
I settled with pure html with Lit only. I don't like learning another library and it's not easy to follow with. Looking forward how you added form validation.
I encountered something with the <form> tag though, not sure if it's just lack of understanding of being a web developer, but with MVU approach. Would you leverage the form attributes such as action and method, or would you just bind the button inside the form to a dispatch message? Because if I use forms then obviously if I click the button then the page will reload/try to go to that action.

No need to leverate action/method since we will be handling everything ourselves via Fable.Remoting.
You could use a regular button click, but better to use the conventional form @submit event.
Here's an example:
Here is an example:
<form @submit={Ev (fun e -> e.preventDefault(); save())}>

    <sl-input
        label="Client"
        class="label-on-left"
        .value={project.Client}
        @sl-change={Ev (fun e -> setProject { project with Client = e.target.Value })}>
    </sl-input>

    <sl-input
        label="Name"
        class="label-on-left"
        .value={project.Name}
        @sl-change={Ev (fun e -> setProject { project with Name = e.target.Value })}>
    </sl-input>

    <div style="width: 245px">
        <sl-button type="submit" style="float: right; width: 120px;" variant="primary">Save</sl-button>
        <sl-button style="float: right; width: 120px;" variant="default" @click={Ev (fun e -> close())}>Cancel</sl-button>
    </div>

</form>

@JordanMarr Awesome. What about having a sidebar and main page implementation? How would you implement that? Would you consider using child routers or would you just rely on the MVU approach where Parent component has the states for both side/main page?
Think of a ListView on the left and when I select something it propagates the right side (main page)?

@leolorenzoluis, new stuff has been added:
https://twitter.com/jordan_n_marr/status/1553107245868748804

Great stuff! Will check out. Have you tried interfacing with charts? It's either I hand craft the code for canvas manipulation using Fable (which I don't prefer) or use an existing chartJS library (which I use), but there's no guidance when it comes to manipulating it in MVU pattern or even if either Lit/React owns the state.
What I did was to trigger a message that the dynamic manipulation (side effects) happen at the update. Not sure if I'm on the right direction.

Were you able to get it working? I guess it just depends on which library you are using.
The ideal graph library would be a Web Component which would allow you declaratively set properties via the lit .property={} notation. However, if you have to do it via code, then the side-effect approach you are using seems like it could work.

Yeah I got it working. I'm just not sure if it's the right pattern to put in the update. One can just rewrite similar functionality from d3 or other chart libraries in Fable to make it friendly with F# 👍


convert some service classes to static props, run [EclipseMarketplace Ecologi ElmPackage Endpoint]

Warnings

:warning:
This PR modified service code for endpoint but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for elm-package but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for ecologi but not its test code. That's okay so long as it's refactoring existing code.

:warning:
This PR modified service code for eclipse-marketplace but not its test code. That's okay so long as it's refactoring existing code.

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @calebcartwright!

  Generated by :no_entry_sign: dangerJS against 615ef4592ec209616ba213412277027513e3d44c


Integrate this FAQ

Dunno who compiled this, some of it is based on the official documentation, but it looks like there is some more stuff: http://khaidoan.wikidot.com/jquery-validation

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.
To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Goaccess opens files in write or append mode

See this, if an access log being monitored by goaccess tries to rotate the file, some bad things happens. This is probably because of goaccess opens log file in write mode or append mode, please see this https://github.com/mholt/caddy/issues/1363

goaccess does not open the log in in write mode nor append mode. It does it in read mode. Here's where it attempts to open it.
I monitor access logs with goaccess and have logrotate doing its job on the logs without any issues so far. As mentioned on the other thread, it sounds like it may be lumberjack?

Closing this. Feel free to reopen it if needed.


Updates archive_prepare to use local.python as the runtime

Description
Use the system default python runtime when building the lambda archive without docker
Motivation and Context

I'm using pyenv to manage python on my machine and without this change the python version must be explicitly set. Not sure if this is just a misconfig on my end but this change seemed simple enough and I don't think it alters core functionality - unless I'm missing something.
Breaking Changes
None?
How Has This Been Tested?

Running this from Ubuntu 18.04 LTS
relevant terraform apply output prior to this change:
module.lambda_function.null_resource.archive[0]: Provisioning with 'local-exec'...
module.lambda_function.null_resource.archive[0] (local-exec): Executing: ["python3" ".terraform/modules/lambda_function/package.py" "build" "--timestamp" "1592508494071385000" "builds/b548a6f33e3794ec3c272a4a41f0d1e164a2d5ea6084aa48d13270fdc24e74df.plan.json"]
module.lambda_function.null_resource.archive[0] (local-exec): zip: creating 'builds/b548a6f33e3794ec3c272a4a41f0d1e164a2d5ea6084aa48d13270fdc24e74df.zip' archive
module.lambda_function.null_resource.archive[0] (local-exec): Installing python requirements: ./lambda/requirements.txt
module.lambda_function.null_resource.archive[0] (local-exec): > mktemp -d terraform-aws-lambda-XXXXXXXX # /tmp/terraform-aws-lambda-5g2w50b2
module.lambda_function.null_resource.archive[0] (local-exec): > cd /tmp/terraform-aws-lambda-5g2w50b2
module.lambda_function.null_resource.archive[0] (local-exec): > python3.7 -m pip install --no-compile --prefix= --target=. --requirement=requirements.txt

relevant terraform apply output with this change:
module.lambda_function.null_resource.archive[0] (local-exec): Executing: ["python3" ".terraform/modules/lambda_function/package.py" "build" "--timestamp" "1592508766218947800" "builds/57a5116ab18759bf413e7039ea45741e4c9ae15a85311ffaa4e160dc4cf8aa9c.plan.json"]
module.lambda_function.null_resource.archive[0] (local-exec): zip: creating 'builds/57a5116ab18759bf413e7039ea45741e4c9ae15a85311ffaa4e160dc4cf8aa9c.zip' archive
module.lambda_function.null_resource.archive[0] (local-exec): Installing python requirements: ./lambda/requirements.txt
module.lambda_function.null_resource.archive[0] (local-exec): > mktemp -d terraform-aws-lambda-XXXXXXXX # /tmp/terraform-aws-lambda-c8b2g29o
module.lambda_function.null_resource.archive[0] (local-exec): > cd /tmp/terraform-aws-lambda-c8b2g29o
module.lambda_function.null_resource.archive[0] (local-exec): > python3 -m pip install --no-compile --prefix= --target=. --requirement=requirements.txt

runtime is not always matching executable. For example, runtime can be what AWS Lambda supports (nodejs, ruby, PowerShell, golang, custom...), but local.python defines a version of Python executable which should be used depending of the platform.
The current behavior is correct, so I am going to close this PR.

runtime is not always matching executable. For example, runtime can be what AWS Lambda supports (nodejs, ruby, PowerShell, golang, custom...), but local.python defines a version of Python executable which should be used depending of the platform.
The current behavior is correct, so I am going to close this PR.

understood, thanks for the clarification!


Update README.md

Description
I've replaced "repository-name" with "devFind", since the repository name will remain consistent for all contributors. It may be a minor contribution, but I wanted to address it.
Related Issues
[Include any related issues or pull requests that this PR addresses or is related to. Use GitHub's shorthand syntax to link to them, like #1234.]
Changes Proposed
[Describe the changes you've made in detail. Be specific and include any relevant code snippets.]
Checklist

[x] I have read and followed the Contribution Guidelines.
[x] All new and existing tests passed.
[x] I have updated the documentation to reflect the changes I've made.
[x] My code follows the code style of this project.
[x] The title of my pull request is a short description of the requested changes.

Screenshots
Note to reviewers

@Shiva-Sai-ssb Thank You for the contribution.


89 impl starkvest vestings

Fix #89

All changed are done @Bal7hazar and @tekkac :)
@Bal7hazar , I had error during generation of docs, so I reverted and did not generate the new docs. Can we merge this PR and update the docs in the next one (to not block the src repo).


merging import lines like isort

After some refactoring, several imports were removed and this was left:
from foo import (
    bar,
    baz,
)

bar()
baz()

With isort this is nicely formatted as:
from foo import bar, baz

bar()
baz()

But ruff does no formatting here and keeps the file as is.
Ruff 0.0.284 was executed as ruff check --fix --select I test.py with no config, isort as isort test.py with not config.

I think you’re looking for this setting: https://beta.ruff.rs/docs/settings/#isort-split-on-trailing-comma. Ruff defaults it to true while isort does not. (It’s a point of confusion as the isort docs state it defaults to true for their Black configuration, but doesn’t seem to do so in practice.)

Thanks, that's it. While reading through the doc, I didn't realize that something called split will actually toggle folding.
And isort documentation is indeed wrong on this, I've created https://github.com/PyCQA/isort/pull/2163 for that.


GeneTaggerExample_Learning broken

On master, the GeneTaggerExample_Learning notebook is failing for me on the first cell after the 'Learning' heading (i.e., DDL.train_model()) with an IndexError. Is it working for anyone else?

Found it--looks like there was an old feature pickle from a previous state of the notebooks that was being loaded that was of a different size. We good.


meta: ship 10 features and bug fixes created through public feedback

Summary
In October of 2020 the npm team launched the new public feedback process. A primary goal is to improve transparency and give our community a voice in determining the roadmap of npm. Starting in Q3 of 2020 the npm team will prioritize and ship 10 different features and bug fixes that were discussed and prioritized based on the feedback process.
Intended Outcome

Improvements to the CLI, Registry, and Website
The npm team builds trust in our new feedback process
The npm team is able to show how we prioritize work in a transparent fashion

How will it work
The community will engage in our discussions forum to make requests for changes to any of the npm products.
The npm team will respond and participate in discussion to help identify solutions to the problem statements identified in the discussion. The npm team will also host bi-weekly triage meetings to discuss the discussions at length.
Items that we choose to explore will be added to the future column of the roadmap. Once the requested item has been properly scoped it will be prioritized based on available engineering resources.

We got a number of new features launched and bugs fixed through public feedback! We definitely got more than 10, but here are 10 of the things we did so we can exit green on this roadmap item 🎉

website: show TypeScript information for packages that have type declarations
website + registry: Use GitHub Flavored Markdown for README rendering

Broken assets path in README after Github flavoured markdown
Render README.txt as plain text, not as markdown
Render text READMEs in a monospace font

registry: whoami endpoint for automation tokens
website: sign out flow appears to not sign you out
website: send fewer emails during sign up - and no marketing emails
New docs site search bar performance
cli: add set script command


Fix NRE when aborting connection or client disconects

@cesarbs @halter73

Please benchmark this change against the plaintext scenario.

Both this PR and clear run is  1,200,000 +- 50,000

Sweet.

:up::date:

:shipit:


4.1.1  Email init error

Hi Guys
Had a problem with update  4.1.0 to 4.1.1
It broke the site.
Had to rollback to 4.1.0
[10:27] Gilpin, Tony
Fatal error: Uncaught Error: Call to a member function set_account_name() on bool in /var/www/html/wp-content/themes/Divi/core/components/api/email/init.php:26 Stack trace: #0 /var/www/html/wp-content/themes/Divi/core/components/api/email/init.php(6): et_core_api_email_fetch_all_lists() #1 /var/www/html/wp-content/themes/Divi/core/functions.php(338): et_core_api_email_init() #2 /var/www/html/wp-content/themes/Divi/core/functions.php(75): et_core_initialize_component_group('api_email', '/var/www/html/w...') #3 [internal function]: et_core_autoloader('ET_Core_API_Ema...') #4 /var/www/html/wp-content/plugins/bloom/bloom.php(191): spl_autoload_call('ET_Core_API_Ema...') #5 /var/www/html/wp-content/plugins/bloom/bloom.php(167): ET_Bloom->maybe_load_core() #6 /var/www/html/wp-content/plugins/bloom/bloom.php(5349): ET_Bloom->__construct() #7 /var/www/html/wp-includes/class-wp-hook.php(287): et_bloom_init_plugin('') #8 /var/www/html/wp-includes/class-wp-hook.php(311): WP_Hook->apply_filters(NULL, Array) #9 /var/www/html/wp-includ in /var/www/html/wp-content/themes/Divi/core/components/api/email/init.php on line 26

Hi Tony 👋  I'm not familiar with the theme you're running there, but looking at the error, it appears to be doing some custom email registration process? Did you see this error when a specific action was done on the site, or was it on page load? Also can you tell me what WP and PHP versions? Thanks.

Fatal error: Uncaught Error: Call to a member function set_account_name() on bool
I have same error on my website. Can you help me please

Hi @fuadhn, I can help. Please create a new issue and describe your issue in detail. I'll need to know your WordPress and PHP versions, what version of the plugin you're using, if you're using some sort of custom theme, and under what conditions you see the error happening. Thanks!


chore(main): release 0.9.1

:robot: I have created a release beep boop
0.9.1 (2022-10-07)
Bug Fixes

Search: Type select on search page is broken (3c5beac)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/BLSQ/openhexa-frontend/releases/tag/0.9.1 :sunflower:


Question: Why does t.doesNotThrow have three arguments?

I do not understand why t.doesNotThrow(fn, expected, msg) has three arguments. It seems like the second argument, expected, is actually used as the message: https://github.com/substack/tape/blob/master/lib/test.js#L476.
Why doesn't the function just have the signature t.doesNotThrow(fn, message)?

It's an if - if you pass an error instance, it checks the type of the error as well.
It's meant to mirror the assert API.

@des-des that package is not the core module assert, it's something else that nobody can ever actually require in node. You're looking for https://nodejs.org/api/assert.html#assert_assert_doesnotthrow_block_error_message which also explains what the argument is used for. "does not throw a TypeError" is not the same as "does not throw anything at all" - something that throws a RangeError, for example, fails the latter but passes the former.


Parsed launch file as output

Thank you for the great package!
I guess the project is not actively worked on any more, but if development should continue, I think getting the parsed launch file as output would be a great addition to the package.
Rqt_launchtree is already quite useful for debugging. I think adding the ability to generate a monolithic launch file with all the argument substitution, evaluated python expressions, nested includes etc would make the package even better for debugging complex launch files.

This is an interesting idea, maybe we can add such a feature at some point.


Enhance and cleanup lexer-parser interface

This pull request cotains the following major changes:

Separate lex operations so developer can controls the behaviour of preprocessor aliasing, this also replaces preproc_aliasing functionality.
Refine control flow related preprocessor directives functions: Migrate if_elif_skip_lines and ifdef_else_skip_lines into single function due to the nature of same functionality and covers all possiblities of control flow combines. (In previous design, former one does not cover #else and later one does not cover #elif.)
Refine naming of lex related functions.

In addition, this pull request fixes hacks introduced in #106.
This pull request is part of #84.

Thank @ChAoSUnItY for contributing!


chore: Update dependencies to latest versions

Update dependencies versions in go.mod file
Address security vulnerabilities by updating third-party libraries
Improve performance and stability with updated dependencies

Proposed changes

Checklist

[x] Pull request is created against the dev branch
[x] All checks passed (lint, unit, build tests etc.) with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
375

Relevant Lines:
2009

💛  - Coveralls


Version/download info table overlaps with rendered PDFs

When rendering PDF files, the table on the right overlaps with the document even at max screen width (viewing on 13" macbook).

This does not happen when rendering word documents-- the text narrows to accommodate the table as you change the screen width.

@sloria @caseyrollins @samanehsan  I know this issue is closed, but I found something similar to be happening on production now with a word file I have uploaded. See image below:

Should I create a new issue for this? Or should we re-open this issue?

@caileyfitz, go ahead and create a new issue for this one since it's specific to word files, not PDFs. Thanks!

I suspect it's a genuinely new issue. I'd make a new one.

+1 to @samanehsan :)

Alright I'm about to create a new issue!


Add required Vale version note

Added a note to README.md that linting requires Vale version 2.16.0 or greater. There is a workaround, but in most circumstances it is likely best to be using a newer version. Also did a few quick edits to make the linters happy.
Signed-off-by: Katrina Prosise<EMAIL_ADDRESS>Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Vale 2.16.0 or greater is needed to use packages, as done so by docs. Added a note to README.md to reflect this.

[ ] follow best practices for commits.

[x] Descriptive title written in the imperative.
[ ] Include brief overview of QA steps taken.
[ ] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

[x] Request PR review by a technical writer and at least one peer.

Docs for 8c8201fba09fdd81c5bc63210009272308ed53cf are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1939/docs/artifacts/html/index.html

Docs for 4fabf88525b141956a3f828e9b0802078f1cac9f are browsable at: https://ci.foundries.io/projects/fio-docs/builds/1944/docs/artifacts/html/index.html

The link-check failure can be ignored.
The cause was an external link being temporarily down.

@kprosise there are 2 different sets of changes:
* a note about the Vale version.

* style and syntax fixes.

Wouldn't it be better to split the commit into 2?

Yes, that would be the proper way, albeit I was feeling a bit lazy. Perhaps a better choice would be to have the commit as an "Update README.md" one instead.  I'll edit it accordingly.


scipy implementation of Carathéodory-Fejér method for approximating with rational function

See http://eprints.maths.ox.ac.uk/926/1/NA-10-03.pdf. MATLAB implementation in https://github.com/chebfun/chebfun/blob/34f92d12ca51003f5c0033bbeb4ff57ac9c84e78/%40chebfun/cf.m (BSD-licensed); I tried to port (I know MATLAB reasonably well) but I'm stuck in the mud trying to understand what it's doing in places, like the vscale method.
See also http://www.chebfun.org/examples/approx/CF30.html which contains an earlier, less complicated version.

whee! I was able to get the CF30 function working, by comparing the Matlab code executed running in Octave (I don't have Matlab).
see https://gist.github.com/jason-s/81c57b337ebc39da9eaa30d239116a52 -- I hereby release this to scipy per the SciPy license, in case you find this useful.

I'm going to close this; I think it comes under the category of "things we would probably accept a PR for" (though it should still be proposed on the mailing list first), but we don't want to keep issues around for every such thing.


Why no pip package?

Working behind a proxy right now, trying to get this installed is a nightmare. Why not just publish pip packages for packages in this repo?
Expected behavior
pip install vsphere-automation-sdk-python

Actual Behavior
Lots of proxy issues with git, local installation dependencies via --extra-index-url flag

This looks like a duplicate of #38.

Right. This is duplicate of https://github.com/vmware/vsphere-automation-sdk-python/issues/38
We are looking into this.
We need to have clearance for dependent packages before publishing to PyPi.
Currently there is no ETA, which we can provide now


Implement vectored IO in parquet file format

We recently added a new feature called vectored IO in Hadoop for improving read performance for seek heavy readers. Spark Jobs and others which uses parquet will greatly benefit from this api. Details can be found here
https://github.com/apache/hadoop/commit/e1842b2a749d79cbdc15c524515b9eda64c339d5
https://issues.apache.org/jira/browse/HADOOP-18103
https://issues.apache.org/jira/browse/HADOOP-11867
Reporter: Mukund Thakur
Assignee: Steve Loughran / @steveloughran
Related issues:

Bump hadoop.version from 3.2.3 to 3.3.5 (is blocked by)
High performance vectored read API in Hadoop (relates to)
Vectored Read into off-heap buffer broken in fallback implementation (is related to)
Vector IO: consistent specified rejection of overlapping ranges (is related to)
Improve Parquet IO Performance within cloud datalakes (is superceded by)
Utilize Hadoop vectorized APIs (is depended upon by)

PRs and other links:

GitHub Pull Request #1103
GitHub Pull Request #1139
GitHub Pull Request #1330

Note: This issue was originally created as PARQUET-2171. Please see the migration documentation for further details.

Mukund Thakur:
CC<EMAIL_ADDRESS>

Timothy Miller / @theosib-amazon:
This might synergize well with the bulk I/O features I've been adding to ParquetMR. Some of the initial work is already in some PRs, and the rest of the plan can be found at https://docs.google.com/document/d/1fBGpF_LgtfaeHnPD5CFEIpA2Ga_lTITmFdFIcO9Af-g/edit?usp=sharing
I determined what to optimize from profiling, and I have run experiments on the new implementation. I glanced through your Hadoop commits, and I noticed that you use ByteBuffer a lot. I have found ByteBuffer to impose a nontrivial amount of overhead, and you might want to consider providing array-based methods as well.

Timothy Miller / @theosib-amazon:
The parquet reader has two phases of reading. One does the raw I/O and decompression. Someone is working on an asynchronous implementation of this, which should help a lot. The second phase works on the output of that, providing higher-level data types. My PRs improve on this by eliminating LittleEndianInputStream, which was super inefficient, plus some other improvements in the most critical paths. All of these improvements are incremental, of course, and we're happy to get contributions that improve on this further.

Steve Loughran / @steveloughran:

I have found ByteBuffer to impose a nontrivial amount of overhead, and you might want to consider providing array-based methods as well.

mixed feelings. its hard to work with but some libraries (parquet...) love it, which partly drove our use of it. if you use on heap buffers is just arrays with more hassle.
FwIW, i was looking at some of the parquet read code and concluding that the s3a FS should implement read(bytebyffer)  as a single vectored IO read. currently the base class implementation reads into a temp byte array and so breaks prefetching...the s3afs only sees the read(bytes) of the shorter array, not the full amount wanted

Mukund Thakur:
Not yet Steve. I plan to do it soon.

Steve Loughran / @steveloughran:
mukund, is there a PR up for this? even though it's not going to be merged, it needs to be shared for others to pick up


[5.4] TestResponse does not display a thrown exception

Laravel Version: 5.4.9
PHP Version: 7.1.1
Database Driver & Version: N/A

Description:
When testing an endpoint that returns an exception, Laravel does not show this exception and phpunit returns green for the test.
I believe that this is caused by Illuminate\Foundation\Testing\TestResponse::fromBaseResponse() which stores the exception on a TestResponse object but, by default, does not display it in the console.
I am not entirely sure if this is a bug. Although, when I test an endpoint that throws an exception, I'd like to know about it.
Steps To Reproduce:

$ laravel new app

Go to tests/Unit/ExampleTest.php and change testBasicTest() to:
public function testBasicTest()
{
    $this->get('/');
}

Then, in routes/web.php change default route to following (notice misspelled dd() function):
Route::get('/', function () {
    d(11);
});

run $ phpunit - you should see the tests passing. There is no indication that the function is misspelled.

Thanks.

Hello @JacobBennett :) I remember you had a solution for this, no?

@themsaid I guess this https://github.com/laravel/framework/blob/5.4/src/Illuminate/Foundation/Testing/TestResponse.php#L36 could be changed to something like?
if (isset($response->exception)) {
    try {
        throw $response->exception;
    } catch (SomethingExpected $e) {
        $testResponse->exception = $response->exception;
    }
}

@jerguslejko when calling $this->get('/') you aren't making any specific assertions against the response which is why you aren't getting any failing tests. If you instead did
$this->get('/')->assertResponseOk();

Then I believe your test will get an exception since the response will return a 500 instead of a 200.
@themsaid I think what you were referring to was this tweet which will re-throw your exceptions instead of having the framework handle them and return a valid response. This was warned against by Adam Wathan however so use with caution.

@JacobBennett calling d(11); causes Symfony\Component\Debug\Exception\FatalThrowableError. I think this kind of exception should be thrown no matter what.
If I do $this->get('/')->assertStatus(200); the only thing I see is:
1) Tests\Feature\ExampleTest::testBasicTest
Expected status code 200 but received 500.
Failed asserting that false is true.

It's really a philosophical thing I suppose. When you are calling get('/') I think the expectation is that you are doing this as an "acceptance" level test. You are making a mock request to the framework which is then passing it to the router > controller > view > etc... or in your case, just to the router. In the case that a user was visiting your site, would you rather them get a 500 error or would you rather them get a stack trace and an exception, or a white page? That is the level you are testing this at. The framework is handling this as if you are a client making a request to the server, and is responding as such. In this case, a 500 is exactly what you would be looking for. This is expected behavior.
Hopefully that helps. In the case that you run into such behavior and you need to figure out the ACTUAL exception that is being thrown, I would suggest following the tweet screen shot included above.

Well, as much as I agree with you I still think that it would be much more useful to see actual error "by default".
While your solution helps, it requires some additional work + there are situations, as Adam Wathan mentions in the tweet, where you want the exception to be handled by the framework.
The "philosophical thing" is whether a test should pass if the response status was 500.
Any opinions @themsaid @taylorotwell ? thanks

You are correct, the test should not pass if there is a 500. This is why the test should assertResponseOk(). This will make sure that the test fails when there is a 500 thrown.
Maybe you should test the controller method instead of a response getting routed to the controller method?
The potential problem with the proposed solution is that it would break any acceptance level tests that are looking for responses with status codes that depend on exceptions being thrown. In example, if you are checking to be sure that your validation is working in your controller, you might assertStatus('422'). If you were catching the exceptions and throwing them again, you would have to assert the exception being thrown rather than the status code that it would return when that exception occurs.

This is an actual test from my testsuite:
/** @test */
public function it_returns_all_tasks()
{
    factory(Task::class)->create();

    $this->get('/tasks')
        ->assertJsonStructure(['id', 'title', 'description']);
}

If there is a bug, it results in Failed asserting that a row in the table [tasks] matches the attributes.... Not descriptive at all.
To the matter of the proposed solution: Sorry, that's mistake on my part. In the example above I meant to only catch exceptions which do not extend ErrorException.
I cannot think of a situation where someone would want to catch these.

I guess this is now obsolete as https://github.com/laravel/framework/blob/master/src/Illuminate/Foundation/Testing/Concerns/InteractsWithExceptionHandling.php exists.
Thanks anyway!

@jerguslejko how do I enable the exceptions?

@morcegon call $this->withoutExceptionHandling() at the beginning of your test method. Or in the setUp method to disable exception handling for all your tests.

Hi @jerguslejko . Trying to add $this->withoutExceptionHandling() method at the beginning of my test, throws a call to undefined method on Lumen 5.6. Do i have any other option to activate this feature?

Hi @jerguslejko . Trying to add $this->withoutExceptionHandling() method at the beginning of my test, throws a call to undefined method on Lumen 5.6. Do i have any other option to activate this feature?

I wrote manual helper in abstract TestCase class
public function disableExceptionHandling(){
app()->instance(\App\Exceptions\Handler::class, new class extends \App\Exceptions\Handler{
public function __construct () { }
public function report ( Exception $exception ) {
parent ::report( $exception ); // TODO: Change the autogenerated stub
}
        public function render($request, Exception $exception)
        {
            throw $exception;
        }
    });
}

Hi @jerguslejko . Trying to add $this->withoutExceptionHandling() method at the beginning of my test, throws a call to undefined method on Lumen 5.6. Do i have any other option to activate this feature?

I'm having the same issue.

@BWoodfork hi, sorry I don't use lumen so can't help you out :/


Add a balancer tools to balance data between workers

Is your feature request related to a problem? Please describe.
After new workers join to exist alluxio cluster after several month from alluxio cluster started, the new worker will have less data and related io work load, but the exist workers are busy as it was, so we need a balance tool to transfer data from exist data to new worker.
Describe the solution you'd like
Add a balancer tool, something like hdfs balancer.
Describe alternatives you've considered
Add a module and related shell script to make it easy to start alluxio-balancer process.
Urgency
Normal
Additional context
No

maybe it can be a part of master, not just a tool. It means that when a worker is down , the cached data can be reload by another worker.


Return of RAR metadata caching

My previous attempt at this in #2192 had a bad side effect: it kept handles within AssemblyInformation objects live in the cache past the point of RAR returning, because nothing disposed them. They should have been cleaned up eventually, if finalizers ran after GC, but that wasn't sufficient to keep an internal test that did a rebuild in VS working--it failed copying from obj to bin because a the file in bin was held in use.
This is a cleaner approach: wrap the extraction process as it was done before, and cache an object consisting only of the results.
That "object consisting only of the results" seems like it should just be AssemblyInformation instead of a new type. I introduced the new type to minimize churn and maximize understandability of this change in the short term.
Relates to #2015.

    internal static FrameworkName GetTargetFrameworkAttribute(string path)

Shouldn't GetTargetFrameworkAttribute get cached as well?

Refers to: src/Tasks/AssemblyDependency/AssemblyInformation.cs:222 in d541b3f. [](commit_id = d541b3fde95571060de73d4ef91576029c729490, deletion_comment = False)

Shouldn't GetTargetFrameworkAttribute get cached as well?

Yes, but it doesn't seem to be called anywhere. So I didn't bother.

    internal AssemblyInformation(string sourceFile)

Are there any other usages of the AssemblyInformation constructor besides the static method you created? If yes, it might be worth beeffing up AssemblyMetadata with the new usages.

Refers to: src/Tasks/AssemblyDependency/AssemblyInformation.cs:65 in d541b3f. [](commit_id = d541b3fde95571060de73d4ef91576029c729490, deletion_comment = False)

No, no other uses of the AssemblyInformation constructor, so I think this is reasonably complete.

@dotnet-bot test Ubuntu16.04 Build for CoreCLR please

I did some benchmarks with and without this change on top of current vs15.3 (520d1aa).
Testbed is cli 2.0.0-preview2-006497 + private MSBuilds on incremental build of a project generated with dotnet new mvc.

scenario
mean
stdev

no-cache
1580.35
102.2840601

with-cache
1528.2
134.2231449

That's about a 3% gain, which is nothing to sneeze at but doesn't meet the bar for 15.3 at the moment, so I'll retarget this PR to master.

@dotnet-bot test Windows_NT Build for CoreCLR please

@dotnet-bot test Windows_NT Build for CoreCLR please

@dotnet-bot test this please


Add GetFeatureInfo Masking

This PR adds proper masking of GFI requests for all of the current grid types with fast checks that the request lng/lat position is within a grid polygon. The only issue right now is that the ROMs masking isn't perfect, and sometimes lng/lats outside of the data polygons still give values. However, overall it is a huge improvement, and might even be faster due to the simple numpy functions used to find the valid polygon.

Thanks for all the hard work on this nick!!


operator project-quay (3.9.0-nightly.20230511)

Add 3.9.0-nightly.20230511 to community operators

/merge possible

/merge possible