Implement File Format Reader/Writer

Description
This pull request enables DataFrames to be read from and written to various file formats (CSV, JSON, ORC, Parquet) using a set of predefined options implemented via the ConfigOpts trait. Key changes include:

DataFrameWriter:

Implemented functionality to write DataFrames to csv, json, orc, parquet and text formats.
Added support for configuration options for various file formats (i.e csv, json, orc, parquet and text).
ConfigOpts trait implemented for JsonOptions, OrcOptions, and ParquetOptions.
Unit tests added to validate configuration options, similar to TableOptions tests in DataFusion.

DataFrameReader:

Enabled reading DataFrames from csv, json, orc, parquet and text formats.
Supported predefined reading options using the ConfigOpts trait for flexible configuration.
Added advanced options like schema merging and recursive file lookup for Parquet files.
Developed tests ensuring correct functionality of the DataFrameReader configurations and file parsing.

Related Issue(s)

Closes #54

Documentation

PySpark IO
DataFusion

Hi @sjrusso8  I just updated the pr.

@lexara-prime-ai  LGTM! just update the README.md and mark these as closed. Then i'll merge in the change


ability to add gameplay footage to the bottom of the video

Is your feature request related to a problem? Please describe.
ability to put gameplay footage (minecraft, subway surfers, gta5 racing)
Describe the solution you'd like
have a check mark and put gameplay footage if enabled
Describe alternatives you've considered
idk
Additional context
i love ai brainrot

No. ❗


Add ability to bundle unischema fields for storage

I still need to add some more tests and improve documentation but it should be open for feedback.
Note one design choice I have included is that a field can be in multiple bundles, meaning you can include a small field multiple times in a row stored in different columns. This can be useful if you commonly need the same small piece of metadata with 2 large fields which can be read separately.
I know we had discussions on predicates on decoded data but I will do that in a separate PR since that change is unrelated to bundling.

Codecov Report

Merging #188 into master will increase coverage by 0.06%.
The diff coverage is 82.88%.

@@            Coverage Diff             @@
##           master     #188      +/-   ##
==========================================
+ Coverage   81.88%   81.94%   +0.06%
==========================================
  Files          65       65
  Lines        3462     3529      +67
  Branches      528      544      +16
==========================================
+ Hits         2835     2892      +57
- Misses        532      542      +10
  Partials       95       95

Impacted Files
Coverage Δ

petastorm/etl/__init__.py
62.5% <ø> (ø)
:arrow_up:

petastorm/utils.py
91.22% <100%> (+2.33%)
:arrow_up:

petastorm/reader_worker.py
95.18% <100%> (+2.23%)
:arrow_up:

petastorm/reader_impl/row_group_loader.py
93.02% <100%> (+0.25%)
:arrow_up:

petastorm/spark_utils.py
93.33% <100%> (-0.42%)
:arrow_down:

petastorm/reader_impl/row_group_decoder.py
58.33% <16.66%> (-41.67%)
:arrow_down:

petastorm/etl/rowgroup_indexing.py
66.66% <20%> (+0.53%)
:arrow_up:

petastorm/etl/rowgroup_indexers.py
42.18% <25%> (ø)
:arrow_up:

petastorm/unischema.py
96.82% <95.83%> (-0.8%)
:arrow_down:

Continue to review full report at Codecov.

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

Unit tests pass on my local machine, however things are always hanging on travis

Going to close for now as we might have better ways to improve performance with many columns


Index out of range during protocol negotiation when using host name

Win X client, Server 2008 R2 server.
Interestingly, if I use the IP instead of the domain name, I get issue #17  instead. On rare occasions, this will 'work', and I get issue #17 again.
(With debug:)
[] Spoofing arp replies...
[] Turning on IP forwarding...
[] Set iptables rules for SYN packets...
[] Waiting for a SYN packet to the original destination...
[+] Got it! Original destination is ****
[] Clone the x509 certificate of the original destination...
[] Adjust the iptables rule for all packets...
[*] Run RDP proxy...
Warning: The python3 module 'hexdump' is missing. Using hexlify instead.
Listening for new connection
Connection received from ****:34673
From client:
Listening for new connection
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python3.6/threading.py", line 916, in _bootstrap_inner
self.run()
File "/root/src/Seth/seth/main.py", line 37, in run
self.handle_protocol_negotiation()
File "/root/src/Seth/seth/main.py", line 100, in handle_protocol_negotiation
self.save_vars({"RDP_PROTOCOL_OLD":  data[-4]})
IndexError: index out of range
This would suggest that the socket read on line 98 is returning no data. Confirmed with a print(len(data)) check.

If no data is returned, this means the peer has reset the connection. Maybe a pcap file of the connection could tell us more.


Minecraft is ALWAYS on top of other windows, even in windowed mode (similar to #1)

in this image, my browser is the focused application

was this supposed to be fixed in #1 ?

This is not this mod's problem. It has nothing to do with windowed mode.
Here's the full code of this disappointment:
https://github.com/Kir-Antipov/cubes-with-borders/blob/6f2f7449a3349274dead712f875c63550b78f826/src/main/java/dev/kir/cubeswithborders/client/mixin/WindowMixin.java#L27-L54

hmmm...
hang on
after a restart its fixed itself anyways

so no i have no idea where its coming from :'p

#1 was a different beast - compositing was disabled while Minecraft was in fullscreen (it's called exclusive fullscreen for a reason). To enable it, you need to untether the window from a monitor with a few additional requirements, which I did. This change, by its nature, only affects the game while it's in fullscreen.
However, I have a feeling that you still have Cubes Without Borders installed, and this problem was triggered by it, because it explicitly changes the Z-order of the window on Windows. So, maybe some changes in that regard that shouldn't have stuck did indeed stick.

oh i did forget to disable that one..
does that mean i should make an issue about it there? :p

If you don't mind :)
I'll need to look into that when my Windows friend is available. I guess, HWND_NOTOPMOST is not reset whenever the game window leaves borderless fullscreen for some ungodly reason.

👍


Please move ansible-collections/edge.microshift to redhat-cop/edge.microshift

Type of repository request
migration of existing repository
Requested name for the repository
edge.microshift
Name of the repository to create or location of existing repository to migrate.
https://github.com/ansible-collections/edge.microshift/
Repository Description
Description applied to the newly created repository.
Validated Content collection for Microshift
Associated CoP
Automation CoP
Admin Access
Name of the team(s) and/or user(s) that should have admin access on the newly created repository.
maxamillion
resolutecoder
matoval
thom-at-redhat
validated-content-curators
Write Access
Name of the team(s) and/or user(s) that should have write access on the newly created repository.
Read Access
Name of the team(s) and/or user(s) that should have read access on the newly created repository.
Additional details
Any additional information or context to provide for this request.

Review request:
cc: @redhat-cop/container-cop-core

this has been added


Model training stalls forever after just a few batches.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution: Windows 10.0.19041 Build 19041
TensorFlow installed via pip
TensorFlow version: 2.4.0-rc4
TensorFlow Git version: v2.4.0-rc3-20-g97c3fef64ba
Python version: 3.8.5
CUDA/cuDNN version: CUDA 11.0, cuDNN 8.0.4
GPU model and memory: Nvidia RTX 3090, 24GB RAM

Describe the current behavior
Model training regularly freezes for large models.
Sometimes the first batch or so works, but then just a few batches later and training seems stuck in a loop. From my activity monitor, I see GPU CUDA use hovering around 100%. This goes on for minutes or more, with no more batches being trained.
I don't see an OOM error, nor does it seem like I'm hitting memory limits in activity monitor or nvidia-smi.
Describe the expected behavior
I would expect the first batch to take a bit longer, then any subsequent batches to take less than <1s. Never have a random batch take minutes (after previous batches took <1s) or stall forever.
Standalone code to reproduce the issue
Run through all the cells in the notebook shared below to initialize the model, then run the final cell just a few times. Eventually it will hang and never finish.
https://github.com/not-Ian/tensorflow-bug-example/blob/main/tensorflow error example.ipynb
Other info / logs
My intuition:
Smaller models train quickly as expected, however I think even then they eventually stall out after training many, many batches.
I had another similar, small VAE like in my example that trained for 5k-10k batches overnight before stalling. Could there be a memory leak somewhere?

@not-Ian,
Could you please update TensorFlow to the stable version v2.4 and check if you are facing the same issue.
Also, try setting a hard limit on the total GPU memory as per this guide and check if it helps. Thanks!

Thanks for the reply, but no luck. I ran:
> pip uninstall tensorflow-gpu
then
> pip install tensorflow-gpu==2.4.0
So now I'm on:
TensorFlow version: 2.4.0
TensorFlow Git version: v2.4.0-rc4-71-g582c8d236cb
Then I included this code at the top of my notebook:
gpus = tf.config.experimental.list_physical_devices('GPU')
if gpus:
  try:
    # setting memory limit to 1024MB * 23 = 23GB. I've got 24, but 23 to be safe.
    tf.config.experimental.set_virtual_device_configuration(
        gpus[0],
        [tf.config.experimental.VirtualDeviceConfiguration(memory_limit=1024 * 23)])
    logical_gpus = tf.config.experimental.list_logical_devices('GPU')
    print(len(gpus), "Physical GPUs,", len(logical_gpus), "Logical GPUs")
  except RuntimeError as e:
    # Virtual devices must be set before GPUs have been initialized
    print(e)

And training stalled again (100% GPU usage, nothing training) after 2 batches.

Also, for completion's sake, I got this error:
2020-12-16 15:50:23.428273: I tensorflow/core/platform/windows/subprocess.cc:308] SubProcess ended with return code:<PHONE_NUMBER>
I read once somewhere about a ptxas.exe compiler error when this happens. Something about using the CUDA 11.1 ptxas.exe file, as it supports -arch=sm_86 argument in this call:
2020-12-16 16:03:52.614874: I tensorflow/stream_executor/gpu/asm_compiler.cc:215] C:/Program Files/NVIDIA GPU Computing Toolkit/CUDA/v11.0/bin/ptxas.exe C:\Users\Ian\AppData\Local\Temp\/tempfile-DESKTOP-IDQ189A-1554-3668-5b69c0cfcfd7f -o C:\Users\Ian\AppData\Local\Temp\/tempfile-DESKTOP-IDQ189A-1554-3668-5b69c0cfd038e -arch=sm_86 -v
Anyways I uninstalled, and reinstalled Tensorflow 2.4.0 and used the 11.1 CUDA ptxas.exe file instead and it still didn't work. Instead I got SubProcess ended with return code: 0, and it still stalled after 2 batches.

Same with me

@amahendrakar any other ideas?

https://github.com/tensorflow/tensorflow/issues/45648

#45648
this fix solves the problem

but that is not a solution of course.

WOW I think this fixed it. Weird fix, but thanks a lot.
Do you know - did changing this remove 'CUDA' from the Task Manager?
Thanks again

don't close the issue.

Solution for Keras, remove activation='relu' from CONv2D
conv1 = Conv2D(denseList[0], 3, activation='relu', padding='same')(inputs)
to
 conv1 = Conv2D(denseList[0], 3, padding='same')(inputs)
conv1 = Activation("relu")(conv1)

@psycho2012 lmao, it is not a solution. Solution is to migrate to pytorch for example.

Was able to replicate the issue in TF 2.6.0-dev20210602,please find the gist here..Thanks !

@not-Ian I am not able to reproduce the issue with TF2.5. Please check the gist here and let me know if I am missing anything. Please note that i ran your code in colab (not on Windows).
Can you please check with TF2.5 and let us know whether the issue is persisting or not. Thanks!


add ability to set http timeout

Desired Outcome
The aim is to be able to set the http client timeout, which is currently set to 10 seconds in the code.
Implemented Changes
The modification consists in adding the "HttpTimeout" integer to the "Config" structure and using it when instantiating the "http.Client" object.
Test coverage

This PR includes new unit and integration tests to go with the code
changes

Hi @antoinemiquel,
Thank you for the pull request! I'll bring this up with the team today, but at first glance this looks good. The only thing we would probably need before being able to approve this would be some more thorough tests to ensure that if no timeout is provided, the timeout is set to 10 seconds by default.
Let me know if you need any guidance on this, and as always we appreciate your contribution!

Hi @szh,
Thanks for your quick feedback!
To follow up on your comments, I just pushed the following modifications:

remove the ability to load the HttpTimeout value through the .conjurrc file or through environment variables
add a constant HttpTimeoutDefaultValue set to 10 seconds
modify the logic linked to the config.HttpTimeout variable

Specifically, a config.GetHttpTimeout() function has been created. It manages this logic and retrieves the http timeout value as follows:
If config.HttpTimeout is

less than 0, GetHttpTimeout returns 0 (no timeout)
equal to 0, GetHttpTimeout returns the default value (constant HttpTimeoutDefaultValue)

Else, GetHttpTimeout returns the value of config.HttpTimeout
Thus, config.GetHttpTimeout() is now used when creating http.Client.
The following unit tests have been added:

TestConfig_GetHttpTimeout in config_test.go
TestClient_HttpClientTimeoutValue in client_test.go

I hope these changes are acceptable. Don't hesitate to let me know if you need any changes

/approve-build 65defdc2c303d250b4a5499dfd24edbe30b7a0a0

/approve-build 65defdc

thanks for the merge!!


Differences between screenshot in readme and what tool looks like

Hey, love what you guys are building, super cool.
I was curious about the differences between the screenshot shown in the repo:

And what I see when I run the tool:

I don't see the Action Console and Feedback tabs in the top right. I also don't see Agent Cluster, APM, and Embeddings in the left-most sidebar. The Action Console in particular would be really useful, would love to understand how to get that working.

@RickyEsclapon Action Console, Feedback,Embeddings and APM are currently under development you will soon see them in up coming releases.


chore(ci): Fix release workflow build step checking out wrong commit

actions/checkout checks out the latest commit at the time of the workflow trigger,
hence any commit done during the workflow is not included.
Fixing this to checkout the actual latest (release) commit.
Fixes: https://github.com/neoave/mrack/issues/273

Tested in https://github.com/dav-pascual/mrack/actions/runs/6417183083
Job checking out the actual latest release commit at https://github.com/dav-pascual/mrack/actions/runs/6417183083/job/17422433550#step:3:44
Commit done: https://github.com/dav-pascual/mrack/commit/c0fc82eec7878d45edbd95f4c219684f8747f19d

I don't understand how does this fix #273.  There, the issue seems to be:
404 Client Error: Not Found for url: https://github.com/neoave/mrack/releases/download/v1.16.0/mrack-1.16.0.tar.gz
I.e. the release process did not upload the archive here: https://github.com/neoave/mrack/releases/tag/v1.16.0
But how does the checkout fix it? I was looking at latest test&release run: https://github.com/neoave/mrack/actions/runs/6406506972 and that seems to fail on the fact the version was not bumped as the commit used "chore" type in commit (i think that "chore" doesn't bump version") it would need to be fix, feat or something else.
The fact that the workflow runs with the version it was executed on seems good to me. Or what is the issue with it?

I don't understand how does this fix #273. There, the issue seems to be:
404 Client Error: Not Found for url: https://github.com/neoave/mrack/releases/download/v1.16.0/mrack-1.16.0.tar.gz
I.e. the release process did not upload the archive here: https://github.com/neoave/mrack/releases/tag/v1.16.0
But how does the checkout fix it? I was looking at latest test&release run: https://github.com/neoave/mrack/actions/runs/6406506972 and that seems to fail on the fact the version was not bumped as the commit used "chore" type in commit (i think that "chore" doesn't bump version") it would need to be fix, feat or something else.
The fact that the workflow runs with the version it was executed on seems good to me. Or what is the issue with it?

@pvoborni Oops, didn't look at the packit logs and thought it was about the same issue. I'll unlink the issue.
The workflow run you looked at, indeed won't upload the new version to pypi because @Tiboris already did so manually.
In that workflow it also gets the right version (1.16.0) because the release commit was done in a previous workflow run, so the commit "is there" when the workflow starts: https://github.com/neoave/mrack/actions/runs/6406506972/job/17391355353#step:12:61
Look at this workflow run: https://github.com/neoave/mrack/actions/runs/6241434877
It's doing semantic release for 1.16 version, but it's trying to publish 1.15.1 because it didn't get the latest release commit done during the semantic release step.


Fix CITATION.cff YAML

Actually meant to open https://github.com/rrbuchholz/pm25_atmosphere_analysis_2023/pull/1

Looks good to me.


bpf: complexity-tests: add HAVE_SET_RETVAL and HAVE_NETNS_COOKIE for bpf_sock tests

Consider two kernel-dependent capabilities in the bpf_sock tests.

/test

/test


Concise view for dashboard

This is out of the blue, so feel free to take as much or as little as makes sense :)
Background
The developer has many windows open - IDE, terminal, browser, StackOverflow. Sometimes they are plugged into an external monitor with plenty of pixels, other times just a laptop. Screen real-estate is limited, and rearranging them takes valuable time.
Simple case
Developer uses Snowpack for hot-reloading with a simple config file. They minimize/cover the Terminal running Snowpack.
Advanced: Snowpack as dashboard
A team has feedback tools for static analysis in snowpack -- linting, type checking, unit tests. The snowpack dashboard shows information without scrolling or a lot of screen flicker. Snowpack reacts and shows feedback quickly on change. Developers find the fast-feedback loop valuable, so keep it onscreen and notice flashing / bright colors if they have errors in code.
When errors fire, the Snowpack window explodes with feedback, scrolling vertically from the 3 concurrent checks. The developer may need to click over to the window and scroll through output.
Proposal
I was thinking about Snowpack's dashboard in relation to Edward Tufte's data-to-ink ratio. Basically, make each pixel justify its existence (and intensity) by the value it provides.
Here are some screenshots--
Before: Success - 23 lines

Nicely formatted - underlines, bright colors, blank lines.
Before: Errors - 45 lines

Screen seems busy. Bright underlined task names compete with the tool output for attention.
Proposal: Success - 9 lines

Non-essential information is dropped. Lower contrast, less blank space. Plugin output is brighter, pops out.
Proposal: Errors - 27 lines

Errors stand out. Fewer lines, need to scroll.
Objections
Does not look as nice as before. True: the priority here is to convey information with as little screen real estate as possible for the dashboard scenario. It's opt-in.
Tests
Don't have any -- there was no existing infra I could find to hit this code path.

Aw man -- meant to do the PR against rc3! Will redo it if this looks interesting.

main is rc3 now!


Powertoys Run: search engine shortcuts

Description of the new feature / enhancement
In Edge, you can customize the search bar to work with shortcuts. For example, if I type wiki yodeling, Edge plugs the word "yodeling" into a predefined URL form corresponding to Wikipedia's search bar, then resolves the result. It seems like it would be fairly simple to add a similar plugin for Powertoys Run. The user would put in their own URL forms in the Powertoys settings.
Scenario when this would be used?
I do physics research, and I'm always jumping back and forth between my research and reference sources like Wikipedia, Google scholar, code documentations, etc. It would be nice to be able to do that faster.
Supporting information
No response

This would be done via macros. /dup https://github.com/microsoft/PowerToys/issues/15032

Here's a plugin https://github.com/Daydreamer-riri/PowerToys-Run-WebSearchShortcut


Add safeguard for execute-as group

As per the VyOS docs, running configuration changes as root breaks future configuration changes until the next reboot.
Not only a careless user may run this script as root, it is also the default configuration for the EdgeOS task-scheduler (see /etc/cron.d/vyatta-crontab).
The added snippet is taken directly from the VyOS docs linked above to prevent the script from running with the incorrect credentials.

@mvn23 thanks for catching this! It could help explain the problems in #14 as well.

Sorry, my edit accidentally changed bash to vbash 🤦♂, fixed with a follow up commit to master though.


Update Arch. with arm64

For nodeAffinity, additional arch. Option arm64 is added
What does this PR change?

It will enable the Paralus to run as Server or Client on Apple Mac M1 (arm64) and relay server will come up without any issue

Does the PR depend on any other PRs or Issues? If yes, please list them.

Checklist
I confirm, that I have...

[x] Read and followed the contributing guide in CONTRIBUTING.md
[ ] Added tests for this PR
[ ] Formatted the code using go fmt (if applicable)
[ ] Updated documentation on the Paralus docs site (if applicable)
[ ] Updated CHANGELOG.md

For issue #70

@sandeep540 can you add an entry in changelog.md

@niravparikh05 I added to unreleased tag, hope its correct


1363 - Largest Multiple of Three

Description of the Problem
Given an integer array of digits, return the largest multiple of three that can be formed by concatenating some of the given digits in any order.
Since the answer may not fit in an integer data type, return the answer as a string.
If there is no answer return an empty string.
Example 1:
Input: digits = [8,1,9]
Output: "981"

Example 2:
Input: digits = [8,6,7,1,0]
Output: "8760"

Example 3:
Input: digits = [1]
Output: ""

Example 4:
Input: digits = [0,0,0,0,0,0]
Output: "0"

Constraints:

1 <= digits.length <= 10^4
0 <= digits[i] <= 9
-The returning answer must not contain unnecessary leading zeros.

Code
class Solution:
    def largestMultipleOfThree(self, digits: List[int]) -> str:

Link To The LeetCode Problem
LeetCode

can you please assign it to me.


[feature]: support of Go for VScode extension

Feature Request: Go Support in VSCode Extension
👀 Is there an existing feature request for this?
No existing feature request found after searching the existing issues.
🔖 Enhancement description
Add support for Go in the Keploy VSCode extension.
🎤 Why should this be worked on?

Enables generation of unit tests for Go applications
Expands Keploy's utility to Go developers
Aligns with Keploy's goal of simplifying testing across multiple languages

Repository
keploy/vscode-extension
fixes: #29
No changes in frontend

open issue in keploy main repos  : https://github.com/keploy/keploy/issues/2250

seems like the support is already added


Make gableroux.com point to local repo

Also, not yet sure if it's possible to get https://gableroux.github.io/ to redirect to my domain.

🎉 sounds like redirect works directly when set in repo settings. Did the move, quite satisfied 🚀


CIM: Expose Network type configuration

included in https://github.com/stolostron/console/pull/2778


error: no such subcommand: add

Choose one: is this a 🐛 bug report

Expected Behavior

Current Behavior

The readme says:
Installation
$ cargo add memdb

but cargo says:
error: no such subcommand: `add`

	Did you mean `doc`?

$ cargo -vV
cargo 1.39.0-nightly (ab6fa8908 2019-09-25)
release: 1.39.0
commit-hash: ab6fa8908c9b6c8f7e2249231c395eebfbc49891
commit-date: 2019-09-25
Possible Solution

cargo install memdb ?
error: specified package `memdb v1.0.0` has no binaries

nope, guess that's not it.
Manually adding it to [dependencies] in Cargo.toml ?
Your Environment

Software
Version(s)

Package
?

Runtime
?

Package Manager
pacman

Operating System
Arch Linux 64bit

cargo add is part of cargo-edit; I kind of assumed this would've been merged with cargo in the last 2 years, but we're still not there.


No executable selection when Tool Bar Location is set to 'commandCenter'

When using the following setting:
"debug.toolBarLocation": "commandCenter"
I can no longer choose the executable like with the other options like "docked"
I really like the new possibility to put it in the command-center but this is a major flaw for me.

Here we can see the executable selection dropdown when "docked" which is missing when "commandCenter" is used

Please add this dropdown also to the command-center-variant.
Thanks

Discussed recenlyt in https://github.com/microsoft/vscode/issues/194881


Added "overfitting and underfitting"

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you @KidoVin01 for your contribution and congratulations on your first PR merge to this repo! 🎉
Hope to see more contributions from you in the future.


Step 87 of calorie counter too strict

See https://forum.freecodecamp.org/t/learn-form-validation-by-building-a-calorie-counter-step-87/720435/
The test will fail if there are extra spaces at the end of certain lines, this should not cause failing
It would be great if the test could check the effect of calling calculateCalories, instead of regex
This is the file: https://github.com/freeCodeCamp/freeCodeCamp/blob/4524a11e57c68602fdfa27cc7ff57261e4f69a19/curriculum/challenges/english/15-javascript-algorithms-and-data-structures-22/learn-form-validation-by-building-a-calorie-counter/63c9e8fe3a6f022a05a04675.md

Functionality-based testing:
Instead of checking for exact HTML formatting, test the effect of calling calculateCalories() by inspecting the output.innerHTML content after the function runs. This approach will focus on verifying the expected calorie values are displayed correctly without enforcing specific formatting.

@leaHaimovich does your comment add something to this issue?

@a2937 it's HTML, the newlines don't matter too much. But it's good practice to write human readable code


Consumers fixes

Fixes #113
We now notify the task system when new deliveries are available to make sure the consumers are correctly polled

Coverage increased (+0.3%) to 55.273% when pulling 10fc8a223ac553537d56854c449e7a3df742274f on consumers into 545ba8fa5d4240d74e80316409f0bdb1ae52e445 on master.


ASE support for Availability Zone GAed in December 2019, needs to be updated here

[Enter feedback here]
Per this Blog post,
https://azure.github.io/AppService/2019/12/12/App-Service-Environment-Support-for-Availability-Zones.html

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

ID: cca70845-100f-2cfd-aba3-34b7fa4a5264
Version Independent ID: 475b26e5-9e23-f769-3bd5-cbc4a678dbf0
Content: What are Azure Availability Zones?
Content Source: articles/availability-zones/az-overview.md
Service: azure
GitHub Login: @cynthn
Microsoft Alias: cynthn

@absheik Thanks for the Feedback. I have assigned the issue to content author for checking and updating the document as appropriate.

@cynthn Can you please check this and add your comments.

#reassign @cephalin

@VikasPullagura-MSFT can you tag this for App service?

This is already published. #please-close


Bins history is not available after logging in

I have just registered and logged-in at jsbin.com to try the bins history feature, but cannot ffind it anywhere. Could you please tell me whether this feature has been removed, or point me where to find my last created bins after logging-in?

I am from the Czech Republic and my profession is software engineer, but why is this important?

2. 6. 2022 v 19:55, Skywebb @.***>:
Where are you from
And what your profession???
Send directly to mail🛎🛎
On Thu, 2 Jun 2022 at 6:40 PM, Adam @.***> wrote:

I have just registered and logged-in at jsbin.com to try the bins history
feature, but cannot ffind it anywhere. Could you please tell me whether
this feature has been removed, or point me where to find my last created
bins after logging-in?
—
Reply to this email directly, view it on GitHub
https://github.com/jsbin/jsbin/issues/3489, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZBY6ASO6KYVCX2J5OK7563VNDWX3ANCNFSM5XVWI6MA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub https://github.com/jsbin/jsbin/issues/3489#issuecomment-1145150316, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAE66RHYT2DGMOSJUKKZE3LVNDYPJANCNFSM5XVWI6MA.
You are receiving this because you authored the thread.

I will look into this and get back to you.
On Thu, 2 Jun 2022 at 7:05 PM, Adam @.***> wrote:

I am from the Czech Republic and my profession is software engineer, but
why is this important?

2. 6. 2022 v 19:55, Skywebb @.***>:
Where are you from
And what your profession???
Send directly to mail🛎🛎
On Thu, 2 Jun 2022 at 6:40 PM, Adam @.***> wrote:

I have just registered and logged-in at jsbin.com to try the bins
history
feature, but cannot ffind it anywhere. Could you please tell me whether
this feature has been removed, or point me where to find my last
created
bins after logging-in?
—
Reply to this email directly, view it on GitHub
https://github.com/jsbin/jsbin/issues/3489, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AZBY6ASO6KYVCX2J5OK7563VNDWX3ANCNFSM5XVWI6MA

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub <
https://github.com/jsbin/jsbin/issues/3489#issuecomment-1145150316>, or
unsubscribe <
https://github.com/notifications/unsubscribe-auth/AAE66RHYT2DGMOSJUKKZE3LVNDYPJANCNFSM5XVWI6MA
.
You are receiving this because you authored the thread.

—
Reply to this email directly, view it on GitHub
https://github.com/jsbin/jsbin/issues/3489#issuecomment-1145160143, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AZBY6AXWZDAT4E4VFF4OMSLVNDZW7ANCNFSM5XVWI6MA
.
You are receiving this because you commented.Message ID:
@.***>


fix: make TransactionReference's inner public

There are no invariants to protect.

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.01%. Comparing base (3243dd2) to head (402a8a5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #301   +/-   ##
=======================================
  Coverage   77.01%   77.01%
=======================================
  Files          28       28
  Lines        1379     1379
  Branches     1379     1379
=======================================
  Hits         1062     1062
  Misses        262      262
  Partials       55       55

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


perf(commitments): generate sapling point outside the method

Motivation
Fix https://github.com/ZcashFoundation/zebra/issues/4783
Solution
Generate the sapling points once as a global for sapling (https://github.com/ZcashFoundation/zebra/commit/f79c438ca4096d08ce27d720ee464631db934597).
Orchard is already doing it buy the lazy_static is inside the method. Maybe this is ok but i changed it anyway for consistency (https://github.com/ZcashFoundation/zebra/commit/4cbe1fcd37f3d6035890c548ec47bfb6f946dace).
Review
Anyone can review.
Reviewer Checklist

[ ] Code implements Specs and Designs
[ ] Tests for Expected Behaviour
[ ] Tests for Errors

I admin-merged this PR because it is an urgent sync fix.


Use generic type signature for the decorator

Should fix #36 🤞

Fantastic. Brilliant save, Sir Morley. I must confess here... I use a battle-hardened CLI-oriented hand-built Vim as my IDE, which features about as much in-editor introspection as the Canadian bullfrog belching on a rotting lily pad outside our cottage. so not much
Thanks for both the detailed issue writeup at #36 and the working changeset. You the best Morley. :medal_sports:


Fix a few typos and links in the documentation

I've just fixed a few typos and links in the SBI documentation.
Thank you very much for the development and maintenance of this great package!

thanks @pitmonticone for fixing the links and for your support!

Codecov Report

Merging #549 (08177e1) into main (86256e0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #549   +/-   ##
=======================================
  Coverage   66.82%   66.82%
=======================================
  Files          56       56
  Lines        4181     4181
=======================================
  Hits         2794     2794
  Misses       1387     1387

Flag
Coverage Δ

unittests
66.82% <ø> (ø)

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

Continue to review full report at Codecov.

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


Fix clippy::doc_lazy_continuation lints.

These make it easier to read ordered and unordered lists by having more consistent indentation.

Also, if you're interested, we default allow all clippy lints in this workspace because it's found to be too noisy by default. These warnings showed up due to a misconfiguration issue. If you'd like though selectively adding lints to that [workspace.lints.clippy] block is something that's reasonable to do on a one-off basis. For example adding this lint to that block sounds like a good idea, although I think it will need to wait a bit because this lint was added in Rust 1.80 and we're using 1.79 in CI.


Analyze Documentation generation via automation

Possible Option: JSDocs

TypeDoc for typescript code
JSDocs seems a better option for the web application in js

Unknowns

How to setup JSDoc and TypeDoc together in the same repo?

better-docs allows document generation for both JS and TS files. This has been tested with sample files and documents were generated satisfactorily well.
A caveat though is that a particular format of descriptions and comments needs to be maintained to achieve this. The current code base does not adhere to these requirements and we would need to make modifications in that regard to generate decent documentation.
In conclusion, we have decided to go ahead with better-docs for now, certain formatting choices need to be kept in mind.


concore-editor does not run

The below output was thrown in the browser for both http://localhost:3000/concore-editor and http://localhost:3000/DHGWorkflow, running from the dev branch as well as the main branch. Tested with both Firefox and Chrome.
Failed to compile
src/App.jsx
Line 1:43:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 2:20:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 3:42:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 4:47:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 5:68:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 6:45:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 7:68:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 8:64:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 9:56:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 10:57:  Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 11:55:  Expected linebreaks to be 'CRLF'
........
.........
Search for the keywords to learn more about each error.
This error occurred during the build time and cannot be dismissed.

concore-Editor/DHGWorkflow is broken since pull request #4 is merged. The GitHub workflows fail with the same error, rendering https://controlcore-project.github.io/concore-editor/ unusable.
1m 58s
Run npm ci
added 1801 packages, and audited 1802 packages in 35s
2 packages are looking for funding
run npm fund for details
45 vulnerabilities (1 low, 7 moderate, 26 high, 11 critical)
To address issues that do not require attention, run:
npm audit fix
To address all issues (including breaking changes), run:
npm audit fix --force
Run npm audit for details.

<EMAIL_ADDRESS>build
react-scripts build

Creating an optimized production build...
Browserslist: caniuse-lite is outdated. Please run:
npx browserslist@latest --update-db
Why you should do it regularly:
https://github.com/browserslist/browserslist#browsers-data-updating
Failed to compile.
src/App.jsx
Line 1:43:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 2:20:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 3:42:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 4:47:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 5:68:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style
Line 6:45:   Expected linebreaks to be 'CRLF' but found 'LF'  linebreak-style

Please do not run the --fix command as it may corrupt the git changes history and will show a lot of one endings changes. The simple fix for this issue is as follows -
In the .eslintrc.json change line 53 to "linebreak-style": ["error", "unix"]  if using a unix system. I have tried this fix on a new unix build.

Can you pls make a pull request with the changes?

The issue still persists, even after merging the changes to the main and dev branches.


Fixing Unload Pod not being created on Module deletion

setting InProgress to false, when the load Pod is successful
adding unit test

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (228b1d8) 80.06% compared to head (96d8c7b) 80.07%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #539   +/-   ##
=======================================
  Coverage   80.06%   80.07%
=======================================
  Files          43       43
  Lines        4460     4461    +1
=======================================
+ Hits         3571     3572    +1
  Misses        732      732
  Partials      157      157

Files Changed
Coverage Δ

internal/controllers/nmc_reconciler.go
70.74% <100.00%> (+0.05%)
:arrow_up:

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

/test pull-kernel-module-management-unit-tests

/lgtm


Moved all file handling to one location and simplified file structure.

Codecov Report

Merging #18 into master will decrease coverage by 23.12%.
The diff coverage is 32.43%.

@@            Coverage Diff             @@
##           master     #18       +/-   ##
==========================================
- Coverage   75.22%   52.1%   -23.13%
==========================================
  Files           2       2
  Lines         109     119       +10
  Branches       17      19        +2
==========================================
- Hits           82      62       -20
- Misses         13      44       +31
+ Partials       14      13        -1

Impacted Files
Coverage Δ

src/wrp_interface.c
82.6% <0%> (ø)
:white_check_mark:

src/json_interface.c
32.87% <32.87%> (-36.97%)
:x:

Continue to review full report at Codecov.

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


fix: not clear promise of revalidate after mutating

Fix https://github.com/vercel/swr/issues/828.
See added test case for detail.

I think the root cause of the original issue might not be the dedupingInterval ?
import { mutate } from 'swr'
mutate('key')

When using the global mutate

If there are mounted hook instanced - calling mutate('key') will revalidate the data and update the cache using the hook's fetcher
If all the hooks are unmounted - calling mutate('key') will do nothing - you need provide the data or function to update the cache

https://github.com/vercel/swr/blob/f7876240094c02c9933757d3e8d568185122adfb/src/use-swr.ts#L391-L402
The dedupingInterval will be ignored by these conditions.

You are right. But The mutate('key') should support to clear revalidate promise although not providing the  second param.
Some backend post api doesn't return the data, so RD should use useSWR as follow:
fetch('/api/user/edit', { method: 'post', name })
mutate('/api/user')

and then they want to fetch new data if they need.

mutate('key') has different behavior according to  if there are mounted hook. This API should be better.

@promer94 Thanks for your review.
It's better to keep mutate() to revalidate when shouldRevalidate is true. It's a good API design.
I made a new commit, PTAL.


[Bugfix] Fix tensor parallel for qwen2 classification model

Fix tensor parallel for Qwen2ForSequenceClassification

Let me see if this PR can fix https://github.com/vllm-project/vllm/pull/10218#issuecomment-2472738855 as well...

Let me see if this PR can fix #10218 (comment) as well...

This change doesn't fix it.


memoize_articles=False still caches

 Issue by N8Brooks
Sun Dec 22 23:32:23 2019
Originally opened as https://github.com/codelucas/newspaper/issues/761

Setting memoize_articles to False still caches articles. The docs say that setting it to False shouldn't cache anything.
This can cause problems when scraping a site such as wayback machine. Multiple sites would have the same web.archive prefix and so the cache would consider them to have the same feeds. So grabbing another source would use the previous source's cached feed.
Setting memoize articles to False should prevent this, but cache is still being set.
Maybe just some code like this in source would allow memoize_articles=False to do this:
`
@utils.cache_disk(seconds=(86400 * 1), cache_folder=ANCHOR_DIRECTORY)
def _get_category_urls(self, domain):
"""The domain param is necessary, see .utils.cache_disk for reasons.
the boilerplate method is so we can use this decorator right.
We are caching categories for 1 day.
"""
return self.extractor.get_category_urls(self.url, self.doc)
def set_categories(self):
# changed code:
# only use the private cached function if memoize_articles is False
if self.config.memoize_articles:
urls = self._get_category_urls(self.domain)
else:
urls = self.extractor.get_category_urls(self.url, self.doc)
self.categories = [Category(url=url) for url in urls]
`

memorize article has another purpose (it's for keeping track what articles were downloaded in previous runs in order to skip them)
you were looking to disable the category url cache.
in v 0.9.2 i added a configuration option - Configuration.disable_category_cache that disables the category cache for that source.


Basic documentation for rules

Hi there!
Couldn't quite finish this today, but as will go on vacation wanted to open a PR anyway.
This adds some very basic documentation on the rules and closes #382
The rules that are missing detailed documentation were either very clear from naming (for example NeverForceUnwrap or I had trouble testing myself (such as UseWhereClausesInForLoops).
If there are any questions or improvements, please let me know. I won't be checking in for the next 1-2 weeks, but after that will look into comments if there are any.
Cheers!

Thank you for the effort here! However, we've decided to go with the approach merged in https://github.com/apple/swift-format/pull/615, which generates the documentation automatically as part of generate-pipeline.


MySQL: Drop trigger support

Hi,
this is feature request, please add drop trigger support from right click menu.
Thanks.

Drop and create trigger functions will be added in the next version.


Add full RegEx instructions to grep allowing complex exclusions

This allows users to include more complex RegEx in their exclusions pattern. The previous -e options allowed for one and only one expression (e.g. vendor/) but some projects might want to exclude specific files or folders (e.g. protobuf generated files).
The -E options just add RegEx instructions to the current used option. It's backward compatible.

How well supported is this for different distro's. Please check at least Alpine and BusyBox. Those are notorious for having different versions, which might not support -E.

According to the official GNU Grep documentation: "-E is specified by POSIX" which most distributions support it (not to say all).
Tried on ubuntu, busybox, alpine, debian, and centos.

@kokaz nice, :lgtm: then


chore(deps): update dependency tsup to v7

This PR contains the following updates:

Package
Change
Age
Adoption
Passing
Confidence

tsup (source)
^6.7.0 -> ^7.0.0

Release Notes

egoist/tsup
v7.0.0
Compare Source
Bug Fixes

external: match root paths on unix-like and windows (#891) (cfa6bba)
prefix terser log with name (#899) (2f41663)
preserve top-level when running terser for IIFE (#900) (2c0435a)
Remove SWC and Terser minification clash (#881) (23e25cd)
Update docs domain in schema.json (#901) (a036242)

Features

require nodejs 16 or above (45832d9)
support svelte-preprocess (#906) (3606e45)
Update esbuild version to latest for keeping unknown directives during bundle (#925) (e2e70e8)

BREAKING CHANGES

require nodejs 16 or above
update esbuild to 0.18

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - "after 8am and before 5pm every weekday" (UTC).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: actions/pnpm-lock.yaml
 ERR_PNPM_UNSUPPORTED_ENGINE  Unsupported environment (bad pnpm and/or Node.js version)

Your pnpm version is incompatible with "/tmp/renovate/repos/github/pleo-io/spa-tools/actions".

Expected version: 8.x
Got: 7.32.5

This is happening because the package's manifest has an engines.pnpm field specified.
To fix this issue, install the required pnpm version globally.

To install the latest version of pnpm, run "pnpm i -g pnpm".
To check your pnpm version, run "pnpm -v".

Your Node version is incompatible with "/tmp/renovate/repos/github/pleo-io/spa-tools/actions".

Expected version: 16.x
Got: v18.16.0

This is happening because the package's manifest has an engines.node field specified.
To fix this issue, install the required Node version.


feat(anvil) add "strict" mode that warns on common user errors

Component
Anvil
Describe the feature you would like
Make common mistakes easy to spot:

eth_calling empty contract
sending tx (with calldata) to empty contract
delegate calling empty contract
probably more

All of these, even though well within consensus rules, are most likely mistakes.
These actions could result in something as unintrusive as printing a warning in console or, alternatively bail the whole request.
I am curious what do you think? This could be ofc turned off/on with a command line flag.
Additional context
Already, cast call on empty contract results in an error but this is implemented in cast and works only when return type is provided and could not be decoded (because eth_call returned empty data). Implementing this in node directly could be much more solid. Also, more complex scenarios as delegatecalling empty contract would be possible to cover.

Delegate calling empty contact is something that I've recently spent couple of hours debugging and actually it prompted me to create this issue. Turns out that I missed broadcast in a foundry script when deploying governance spell so in a later (broadcasted) tx I was calling governance relay that was trying to delegatecall to not-deployed governance spell. It simply succeeded without doing anything of value. Again, I know this is correct according to consensus rules but I can't imagine scenario when delegatingcalling to empty contract makes sense in dev node.

Any learning materials about reth-anvil so i can dive deeper? is the plan to rewrite anvil to reuse parts of reth under the hood?

Any learning materials about reth-anvil so i can dive deeper? is the plan to rewrite anvil to reuse parts of reth under the hood?

Best to follow along in https://github.com/paradigmxyz/reth/issues?q=is%3Aopen+is%3Aissue+label%3AA-sdk where the goal is to create an SDK. anvil-reth will build on top of this SDK.


K8SPXC-172 Generate keyring

Test name
Status

scaling
failed

init-deploy
failed

big-data
failed

recreate
passed

self-healing
failed

storage
failed

scaling-proxysql
passed

self-healing-advanced
failed

operator-self-healing
passed

demand-backup
passed

upgrade
failed

limits
failed

monitoring
failed

one-pod
failed

monitoring-2-0
failed

affinity
failed

proxysql-sidecar-res-limits
passed

auto-tuning
passed

scheduled-backup
failed

upgrade-consistency
failed

Test name
Status

Test name
Status

Test name
Status

scaling
failed

big-data
failed

recreate
passed

init-deploy
failed

self-healing
failed

storage
failed

scaling-proxysql
passed

self-healing-advanced
failed

upgrade
failed

operator-self-healing
passed

demand-backup
passed

limits
failed

one-pod
failed

monitoring
passed

monitoring-2-0
failed

auto-tuning
passed

affinity
passed

scheduled-backup
failed

proxysql-sidecar-res-limits
passed

upgrade-consistency
failed

Test name
Status

recreate
passed

self-healing
failed

init-deploy
failed

scaling
failed

big-data
failed

storage
passed

scaling-proxysql
passed

limits
failed

self-healing-advanced
failed

upgrade
failed

operator-self-healing
passed

demand-backup
passed

monitoring
passed

one-pod
failed

monitoring-2-0
passed

affinity
failed

auto-tuning
failed

scheduled-backup
failed

upgrade-consistency
failed

proxysql-sidecar-res-limits
passed

Test name
Status

recreate
failed

self-healing
failed

big-data
failed

init-deploy
failed

scaling
failed

storage
passed

scaling-proxysql
passed

self-healing-advanced
failed

operator-self-healing
failed

limits
failed

upgrade
failed

monitoring
passed

one-pod
failed

upgrade-consistency
failed

demand-backup
failed

monitoring-2-0
passed

auto-tuning
passed

scheduled-backup
failed

affinity
failed

proxysql-sidecar-res-limits
failed

Test name
Status

scaling
failed

recreate
failed

big-data
failed

self-healing
failed

init-deploy
failed

storage
failed

scaling-proxysql
failed

self-healing-advanced
failed

upgrade
failed

operator-self-healing
failed

demand-backup
failed

limits
failed

upgrade-consistency
failed

monitoring
failed

one-pod
failed

scheduled-backup
failed

auto-tuning
failed

monitoring-2-0
failed

affinity
failed

proxysql-sidecar-res-limits
failed

Test name
Status

scaling
failed

big-data
failed

recreate
passed

self-healing
failed

init-deploy
failed

storage
failed

scaling-proxysql
passed

self-healing-advanced
failed

operator-self-healing
passed

upgrade
failed

demand-backup
passed

limits
failed

one-pod
failed

monitoring
passed

auto-tuning
passed

scheduled-backup
failed

monitoring-2-0
passed

affinity
passed

proxysql-sidecar-res-limits
passed

upgrade-consistency
failed

Test name
Status

self-healing
passed

scaling
passed

recreate
passed

init-deploy
passed

big-data
failed

scaling-proxysql
passed

storage
passed

self-healing-advanced
failed

upgrade
failed

operator-self-healing
passed

demand-backup
failed

limits
passed

monitoring
passed

one-pod
passed

monitoring-2-0
passed

auto-tuning
passed

scheduled-backup
passed

affinity
passed

proxysql-sidecar-res-limits
passed

upgrade-consistency
failed

Test name
Status

recreate
passed

self-healing
passed

init-deploy
passed

big-data
failed

scaling
passed

scaling-proxysql
passed

storage
passed

demand-backup
failed

self-healing-advanced
failed

limits
passed

operator-self-healing
passed

upgrade
failed

monitoring
passed

scheduled-backup
passed

monitoring-2-0
passed

one-pod
passed

auto-tuning
passed

affinity
passed

proxysql-sidecar-res-limits
passed

upgrade-consistency
failed

Test name
Status

Test name
Status

self-healing
passed

scaling
passed

big-data
failed

init-deploy
passed

recreate
passed

scaling-proxysql
passed

storage
failed

self-healing-advanced
failed

upgrade
failed

demand-backup
failed

operator-self-healing
passed

one-pod
passed

limits
passed

monitoring
passed

auto-tuning
passed

scheduled-backup
passed

monitoring-2-0
passed

affinity
passed

proxysql-sidecar-res-limits
passed

upgrade-consistency
failed

not in scope anymore, we will reopen PR if needed


Update validation in Inline DNS table (#11897)

Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>Signed-off-by: Shriram Rajagopalan<EMAIL_ADDRESS>Signed-off-by: Yuchen Dai<EMAIL_ADDRESS>

I don't think you need this in 1.14? @rshriram?

/wait

Nope. Just the release that corresponds to Istio 1.7

I feel we should fix in advanced before any one request for the fix at stable release.
WDTY? @lizan

@lambdai My feel is in 1.14 the DNS filter is premature and no one put in production though. I'll leave this up to you.


[share] p:share allow custom text/url for sharing based on user selection

Use case
when sharing a url, it will be nice to customize the url (text) to include the source.
for example:,  allow supply of customize the url link based on the user selection:
NSString *const UIActivityTypePostToFacebook;
NSString *const UIActivityTypePostToTwitter;
NSString *const UIActivityTypePostToWeibo;
NSString *const UIActivityTypeMessage;
NSString *const UIActivityTypeMail;

Proposal
Allow customization of the text (url), so that based on the user selection, it will use that customized url
Share.shareViaEmail('check out my website https://example.com&source=email', subject: 'Look what I made!');
Share.shareViaTwitter('check out my website https://example.com&source=twitter', subject: 'Look what I made!');
...

@pedromassangocode shouldn't this be a request for plus_share? see https://github.com/flutter/plugins/blob/master/CONTRIBUTING.md#important-note

Thanks @nt4f04uNd
I did missed that
@yz
Please file this issue at https://github.com/fluttercommunity/plus_plugins/pulls.


Service Topology Support

Is your feature request related to a problem? Please describe.
Our traffic cost if fairly high due to Interzone traffic. Service Topology solves this issue.
Describe the solution you'd like
Ability to annotate the service with service topology definition.
Describe alternatives you've considered
N/A
Additional context
N/A

@tarokkk @pepov  Can I pickup this issue and work on this ?

@pavan541cs yes, please go ahead!

Service Topology is deprecated in v1.21 and removed in v1.22.  The suggested replacement is topology aware hints


0.1.0

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/116301
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v1.1.0 -m "<description of version>" eed20ddb5b3ae714785b34608775376f6ccaa52f
git push origin v1.1.0


🐛 BUG: pages deploy "found" /functions directory when no such directory exists

Which Cloudflare product(s) does this pertain to?
Pages
What version of Wrangler are you using?
2.0.16
What operating system are you using?
cloudflare pages deployment
Describe the Bug
my cloudflare pages deploy is throwing this error:
Found Functions directory at /functions. Uploading.
 ✘ [ERROR] No routes found when building Functions directory: /opt/buildhome/repo/functions

but there is not a /functions directory in my repo.
my project build configuration is:

Framework preset: Nuxt.js
Build command: nuxt generate
Build output directory: /dist
Root directory: /
Build comments on pull requests: ``

this project previously deployed as configured without problems.

deployment log:  ruleznewmedia-com.d99637be-a531-49c4-b70b-3c48f0507514.log

I got the same error. I had to change the build command to:
nuxt generate && rm -rf /opt/buildhome/repo/functions
And the output directory to:
/output/public

hi @jefforulez :) thanks for flagging this and thanks @eduardo-marcolino for posting your solution 👍  in an effort to remove stale issues so we can have better signal on current friction points, we’re closing out issues that have not been updated within the last 6 months. if this is still a problem, please feel free to open a new issue referencing this one. please be sure to include an updated summary and any additional context and we will review. thank you!


Brief Error Displayed when displaying the indicators

Current behavior
When loading the indicators list an error is displayed briefly.
To Reproduce
Steps to reproduce the behavior:

Go to Indicator Page
A brief error is displayed before all the indicators are displayed

Expected behavior
The indicator list will be displayed without errors
Screenshots

I've also received this on staging and dev when I do a hard refresh within the browser. I noticed two things that happen:

I receive an error message that targets didn't load from the database:

I see squares around all the icons (seems they aren't loading):


Can not serialize a vector of shared_ptr

I tried to serialize a vector of shared_ptr. But it seems that only one element is serialized.
My code is as follows:
class CMyObject {
   public:
    CMyObject(): m_dwSize(0) {}
    void init(uint32_t _dwSize) {
        // ...
        m_dwSize = _dwSize;
        m_pData = new double[_dwSize];
        // ...
    }

    template <class Archive>
    void serialize(Archive &ar) {
        printf("== Serialize MyObject\n");
        ar(m_dwSize, cereal::binary_data(m_pData, sizeof(double) * m_dwSize));
    }

   private:
    uint32_t m_dwSize;
    double *m_pData;
}

int main() {
    std::vector<std::shared_ptr<CMyObject>> data;
    // push some object to data, like data.push_back(std::shared_ptr<CMyObject>(new CMyObject()))
    // some process, init
    // ...
    std::stringstream ss;
    {
        cereal::BinaryOutputArchive oarchive(ss);
        printf("== data's size: %ld\n", data.size());
        oarchive(data);
        printf("== Serialize data done");
    }
    return 0;
}

The output is as follows:
== data's size: 199
== Serialize MyObject
== Serialize data done

And when I deserialized it, throw std::bad_alloc error.
Is there any wrong usage in my code?

The problem had solved. The serialize function of CMyObject is wrong.


autoCleanupSourceBuffer buffer not working

I have autoCleanupSourceBuffer on. I have tested it thoroughly and can say it does not work properly. The streams still stop with console messages such as:
Maximum buffering duration exceeded, suspend transmuxing task
SourceBuffer is full, suspend transmuxing task
How can I keep the stream playing and clear the buffer?

I have autoCleanupSourceBuffer on. I have tested it thoroughly and can say it does not work properly. The streams still stop with console messages such as:
Maximum buffering duration exceeded, suspend transmuxing task
SourceBuffer is full, suspend transmuxing task
How can I keep the stream playing and clear the buffer?

hi,  did you solve the questions,Maximum buffering duration exceeded, suspend transmuxing task.

I also encountered the same problem, I wonder if you have a solution?


added link to tutorial and resources

[x ] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Hi @viviantran27, thank you for the PR!
However, we do not accept PRs that only add links. I'm going to close this PR, but feel free to open a new one with more content.
Happy coding!


Expand parser formats

Fixes #70

Looks good, try cleaning up some of the checkstyle issues also!

Fixed some checkstyle and tests to pass gradle building!

nice! makes it easier and better for CLI users - good job!

Great tweak!


Any chance of new development for the library?

I've been looking for a collision library with various collision primitives for a while now to escape Box2D, but Hardon Collider doesn't provide enough collision resolution info (like the hit normal), and bump.lua is AABB only, so Splash seems like a great fit for me.
However, it seems all progress has been ceased. I'd normally have no issue with this; I can fork it and resume progress on my own (I have), but there isn't enough code documentation for me to do anything very substantial.
Lots of sweep tests haven't been implemented (like AABB vs Segment, which I want for slopes), and Segment vs Segment, among other things.
Is there any chance of it resuming progress in the future, or at least code documentation? Thanks!

I haven't worked on this for a while, simply because I have moved on to other things. I would be willing to revisit and help add features if you would like.
The largest problem I ran into was resolving circle to circle collisions. The detection was simple, but resolution proved quite difficult for some reason, especially in situations with more than 3 circles.
The math for the sweep tests got a bit annoying , but that is doable.

I'm actually pretty fine with the state of the library as is, I'd just like some proper documentation for the sweep tests. How should a sweep test be resolved, what a sweep test function should return, etc. I could code the rest myself.


Sample code partially invisible in Google Chrome

Some code is invisible in Google Chrome, because the .line-highlight div is covering it.

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

ID: 431df4fc-4168-cc8f-d053-9b264d2971c1
Version Independent ID: cd157d04-d596-83fa-6edb-24d52193eb34
Content: Add, download, and delete user data to Identity in an ASP.NET Core project
Content Source: aspnetcore/security/authentication/add-user-data.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

This is a system wide problem. that's tracked in #13143


🐛 Scanning devices, it only finds my ble beacon on Android 11 (it doesn't on iOS 17.5.1 and Android 14)

Prerequisites

[X] I checked the documentation and FAQ without finding a solution
[X] I checked to make sure that this issue has not already been filed

Expected Behavior
The app finds my ble beacon. When it is found the scan stops and gives a warning.
Current Behavior
The app doesn't find my ble beacon.
Library version
^3.2.0
Device
Android 14 and iOS 17.5.1
Environment info
System:
  OS: macOS 14.5
  CPU: (8) arm64 Apple M2
  Memory: 78.36 MB / 8.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.15.0
    path: ~/.nvm/versions/node/v20.15.0/bin/node
  Yarn: Not Found
  npm:
    version: 10.8.1
    path: ~/.nvm/versions/node/v20.15.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.5
      - iOS 17.5
      - macOS 14.5
      - tvOS 17.5
      - visionOS 1.2
      - watchOS 10.5
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.4/15F31d
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 22.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.3
    wanted: 0.74.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Steps to reproduce

npx create-expo-app issue --template blank;
cd issue;
npm install react-native-ble-plx;
Copy and paste the code I give you in the app.js;
Change the device.id to find to the one you're using to test (or change it with device.name)
Inside app.json add "plugins": ["react-native-ble-plx"];
npx expo prebuild;
I don't know if I could do this but: go to /android/gradle/wrapper/gradle-wrapper.properties and change distributionUrl=https\://services.gradle.org/distributions/gradle-8.6-all.zip to distributionUrl=https\://services.gradle.org/distributions/gradle-8.8-all.zip ( I do this because I use Java 22.0.1 );
Connect an iOS device or and Android 14 (I think the problem can be found on Android 12 and 13 too, but I'm not sure);
npx expo run:android or npx expo run:ios --device --configuration Release

Formatted code sample or link to a repository
import { StatusBar } from 'expo-status-bar';
import { useEffect, useState } from 'react';
import { StyleSheet, Text, View, PermissionsAndroid, Platform } from 'react-native';
import { BleManager, ScanMode } from 'react-native-ble-plx';

const manager = new BleManager();

async function requestPermissions() {
  if (Platform.OS === 'android') {
    const apiLevel = Platform.Version;
    const permissions = [
      PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION,
      PermissionsAndroid.PERMISSIONS.BLUETOOTH_SCAN,
      PermissionsAndroid.PERMISSIONS.BLUETOOTH_CONNECT,
    ];
    const granted = await PermissionsAndroid.requestMultiple(
      apiLevel >= 31 ? permissions : [PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION]
    );

    return (
      granted[PermissionsAndroid.PERMISSIONS.ACCESS_FINE_LOCATION] === PermissionsAndroid.RESULTS.GRANTED &&
      (apiLevel < 31 ||
        (granted[PermissionsAndroid.PERMISSIONS.BLUETOOTH_SCAN] === PermissionsAndroid.RESULTS.GRANTED &&
        granted[PermissionsAndroid.PERMISSIONS.BLUETOOTH_CONNECT] === PermissionsAndroid.RESULTS.GRANTED))
    );
  }
  return true;
}

export default function App() {
  const [text, setText] = useState("Initial state");
	let i = 0;

  const startScan = async () => {
    const permissionGranted = await requestPermissions();
    if (!permissionGranted) {
      setText("Permission not granted");
      return;
    }

    manager.startDeviceScan(
			null,
			{
				allowDuplicates: true,
				scanMode: ScanMode.LowLatency,
			},
			(error, device) => {
      if (error) {
        console.error(error);
        setText(`Error: ${error.message}`);
        return;
      }
			if (device.id === "CD:4E:63:E7:78:63")
			{
				console.warn("found: ", device.id);
				setText("found: ", device.id);
				manager.stopDeviceScan();
			}
			else
			{
				i++;
				console.log(device.id);
			}
			setText("Scanned: " + i);
    });
  };

  useEffect(() => {
    const subscription = manager.onStateChange((state) => {
			if (state === 'PoweredOn') {
					startScan();
					subscription.remove();
			}
	}, true);

	return () => {
			subscription.remove();
			manager.stopDeviceScan();
	};
  }, []);

  return (
    <View style={styles.container}>
      <Text>{text}</Text>
      <StatusBar style="auto" />
    </View>
  );
}

const styles = StyleSheet.create({
  container: {
    flex: 1,
    backgroundColor: '#fff',
    alignItems: 'center',
    justifyContent: 'center',
  },
});

Relevant log output
Don't know how. I'm a beginner, I'm sorry.

Additional information
This app is a simple version of a component I've built on another app. In both apps the scan does start but doesn't find the beacon.id I'm searching. The real problem that is making me crazy and not knowing where to turn, is that on Android 11 both apps find the beacon I'm searching.
I know about the permission differences between Android ≤ 11 and Android ≥ 12, and I think this is not the problem because the scan shouldn't start at all if it was.
I've tried different scanMode(s), allowDuplicates: true/false and legacyScan: true/false. No success.
I've tried these apps on Android 11, Android 14 and iOS 17.5.1.
The beacon I'm using: amazon.it link

Hi @Feffe01
I did some research and it turns out that beacons won’t work on iOS with this library as it have different API. I’m not sure what’s happening with android 14, you might try https://github.com/Driversnote-Dev/react-native-kontaktio/issues/121#issuecomment-2098884380.
Overall I would recommend moving to different library that focuses on beacons like react-native-beacons-manager or react-native-kontaktio

Thank you very much! This solved the problem.
For those having the same problem, the solution is modifying the permission of bluetooth scan like this:
<uses-permission android:name="android.permission.BLUETOOTH_SCAN" tools:remove="android:usesPermissionFlags" />
Because without it, android:usesPermissionFlags="neverForLocation" is set causing some beacons not to be found.
In addition, if you have still problems, it's better to set:
import { BleManager, ScanMode } from 'react-native-ble-plx'; and  {   allowDuplicates: true,   scanMode: ScanMode.LowLatency, } to the settings of the scan. These allow to scan more beacons but with higher battery consumption.


Bump @solid/oidc-rp to 0.10.0 and Version Bump to 0.1.2

Update to @solid/oidc-rp as discussed in #10

I'll make another PR. The lock file needs to be updated too :(


[Feature Request] Tries and its implementation

🚀 Tries and its implementation
Is your feature request related to a problem? Please describe.
I'm frustrated when dealing with large datasets of strings where traditional data structures, such as hash tables or binary search trees, don't offer efficient prefix-based searching. For example, autocomplete systems, spell checkers, and IP routing benefit from a more specialized structure like a trie, but I haven't found a straightforward guide to implementing them effectively.
Describe the solution you'd like
I would like a detailed guide or module that covers the implementation of tries (prefix trees) in a programming language (such as C++ or Python). It should include:
Insertion of words
Searching for a word or prefix
Deleting a word
Traversal of the trie structure
Describe alternatives you've considered
I've considered using other data structures like hash tables, but they don't efficiently support prefix matching. I've also explored binary search trees, but they are not optimal for large datasets of strings when prefix operations are required.

Would you like to work on this feature?

[X] Yes
[ ] No

@ajay-dhangar  shouldn't it be (level2)


stream info: move upstream timings into separate struct

Signed-off-by: Michael Puncel<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: Track upstream request timings in struct separate from StreamInfo
Risk Level: low
Testing: unit tests
Docs Changes: N/A
Release Notes: N/A
This is a step toward #2784 and #5841 which both involve handling multiple simultaneous in-flight upstream requests in the router filter. This means that the StreamInfo on the callbacks_ cannot be used as scratch space for timing.

Michael, thanks for tackling this.  Mind checking out CI?  Something looks unhappy.
@snowp up for doing first pass?


How/Where to set the make/model etc data in the Zigbee HA Light On/off ? (TZ-490)

Question
Hi, using this example, where do I set the name, model and other data to "broadcast" so it can be recognised ? Thank You fro providing a clear example with those definitions !!!!
Additional context.
No response

@Tokn59 ，
Regarding this issue, I believe a more detailed description is necessary. Do you expect the data packet to be sent via the broadcasting way? Is the name or model an attribute of the basic cluster?

@Tokn59 ，
Regarding this issue, I believe a more detailed description is necessary. Do you expect the data packet to be sent via the broadcasting way? Is the name or model an attribute of the basic cluster?

Yes those should be part of the basic cluster. Meanwhile I managed to define and successfully configure them !

Hello @Tokn59 ,
If you intend to broadcast the ZCL commands, you can follow the code below:
Broadcast the ON/OFF toggle command:
        esp_zb_zcl_on_off_cmd_t cmd_req;
        cmd_req.zcl_basic_cmd.dst_addr_u.addr_short = 0xffff;
        cmd_req.zcl_basic_cmd.dst_endpoint = on_off_light.endpoint;
        cmd_req.zcl_basic_cmd.src_endpoint = HA_ONOFF_SWITCH_ENDPOINT;
        cmd_req.address_mode = ESP_ZB_APS_ADDR_MODE_16_ENDP_PRESENT;
        cmd_req.on_off_cmd_id = ESP_ZB_ZCL_CMD_ON_OFF_TOGGLE_ID;
        ESP_EARLY_LOGI(TAG, "Send 'on_off toggle' command to address(0x%x) endpoint(%d)", cmd_req.zcl_basic_cmd.dst_addr_u.addr_short, on_off_light.endpoint);
        esp_zb_zcl_on_off_cmd_req(&cmd_req);

Hello @Tokn59 ,
Please refer to https://github.com/espressif/esp-zigbee-sdk/blob/main/examples/esp_zigbee_HA_sample/HA_temperature_sensor/main/esp_zb_temperature_sensor.c#L136

@Tokn59 ，
If the issue had been fixed, please consider closing it. Welcome to discuss any question for esp-zigbee-sdk.


Error with ES6 code?

I have some ES6 modules, and this happens:
 ❯ doctor src/* -o docs                                                                                [21:29:09]
name:
SyntaxError
expected:
comment,whitespace
found:
i
message:
Expected comment or whitespace but "i" found.
offset:
227
line:
10
column:
1
file:
src/Calendar.js

My code is at http://github.com/infamous/infamous

Yeah, JS parsers have come a long way since I last worked on this project. Would love to get back to it and see if I could get rid of the custom parser and use something that supports ES6. Right now, I don't have any time to dedicate to it though.


Cannot ReadFile when using embed.FS on a directory

What version of Go are you using (go version)?

$ go version go1.16.2 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
GO111MODULE="on"
GOARCH="amd64"
GOBIN="/Users/andrewlee/go/bin"
GOCACHE="/Users/andrewlee/Library/Caches/go-build"
GOENV="/Users/andrewlee/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/andrewlee/go/pkg/mod"
GONOPROXY="*.amazonaws.com/v1/repos"
GONOSUMDB="*.amazonaws.com/v1/repos"
GOOS="darwin"
GOPATH="/Users/andrewlee/go"
GOPRIVATE="*.amazonaws.com/v1/repos"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.16.2/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.16.2/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16.2"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="0"
GOMOD="/Users/andrewlee/Projects/TestProjects/test_bindata/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/3k/x98sh83s31nb6kl86555q0c80000gn/T/go-build307906134=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
Created a simple example to see how I can replace https://github.com/gilgameshskytrooper/go-bindata with native 1.16 file embed functionality.
root/
├── files/
│   ├── hello.txt
└── main.go
└── go.mod

hello.txt
Hello World

go.mod
module test_bindata

go 1.16

main.go
package main

import (
	"embed"
	_ "embed"
	"fmt"
)

// Also tried "files"
// go:embed files/*
var embedFS embed.FS

func main() {
        // Also tried "hello.txt", "/files/hello.txt", "./files/hello.txt"
	fileData, err := embedFS.ReadFile("files/hello.txt")
	if err != nil {
		panic(err)
	}
	fmt.Println("string(fileData): ", string(fileData))
}

Most examples I see specify how to use the go:embed directive to point to a particular file, and reading that particular file using ReadFile(). Or defining a directory in the go:embed directive, and using http.FS() to host that folder. However, I could not find an example of using // go:embed files/* to embed the entire directory, then trying to directly read a file located inside that directory using ReadFile(). Therefore, I wanted to know if this is even possible.
What did you expect to see?
Successful file read.
What did you see instead?
panic: open files/hello.txt: file does not exist

Duplicate of #43698

Doesn't seem like the same issue to me. #43698 seems to be referring to a linting problem. Mine is pointing to the behavior of ReadFile().

Yes, and the proposed rule would catch the problem with your code: space in // go:embed

Gocha. Thanks


Add SetContext on Builder interface and GetContext on Biscuit struct.

To be able to set a context and retrieve it form a biscuit.

"context" is the second field of the block liked it is indicated into the spec:
message Block {
  repeated string symbols = 1;
  optional string context = 2;
  optional uint32 version = 3;
  repeated FactV2 facts_v2 = 4;
  repeated RuleV2 rules_v2 = 5;
  repeated CheckV2 checks_v2 = 6;
  repeated Scope scope = 7;
  repeated PublicKey publicKeys = 8;
}

And the goal is to be able to be equivalent to the java biscuit library (https://github.com/biscuit-auth/biscuit-java/blob/a863cd8c0bd45ad1e36390ce777741e49417da0e/src/main/java/org/biscuitsec/biscuit/token/builder/Biscuit.java#L119)

"context" is the second field of the block liked it is indicated into the spec:

I did see that field in the IDL before asking the question here, but I'm wondering what a typical value would be for such a "context", who sets it, who consumes it, and why.

"context" is the second field of the block liked it is indicated into the spec:

I did see that field in the IDL before asking the question here, but I'm wondering what a typical value would be for such a "context", who sets it, who consumes it, and why.

It is intended as a way to embed a payload outside datalog. so something freeform that would not be directly used in the authorization context. It’s a remnant of early biscuit days, meant as some kind of escape hatch.

It’s a remnant of early biscuit days, meant as some kind of escape hatch.

It sounds like something we'd like to get rid of, but can't now, because the libraries for other languages allow writing to and reading from this Protocol Buffer message field. This patch makes it more likely that we'll preserve this field for a long while.

That may be so. But if you want to break the backwards compatibility, you need to do it in a major release. This fix can be applied to a minor version and allow compatibility with older/current versions in other languages (which is not the case at the moment).


Update switch.md

It seems for some reason thumbTintColor has been renamed to _thumbColor according to this warning:
// react-native/Libraries/Components/Switch/Switch.js:112
'Switch: `thumbTintColor` is deprecated, use `_thumbColor` instead.',

thumbTintColor has been deprecated. Generally, props with a _ prefix are not considered part of the public API.
Instead of renaming this prop, can you simply add a deprecation warning to the docs?

Sorry I don't quite get it. Is there a reason why this has been deprecated? It still seems a valid property for iOS switches. Does this mean in the future we won't be able to control this property from RN?
Is there any example in the docs of how to document a property that has been deprecated?
Thanks!

You may usually find additional context behind a given change by looking up the commit that implemented it. Looking through the history of Switch.js, I found 965adee109b86cc36f39a713237ade9592f13dad which does say that, in fact, thumbTintColor has been renamed to thumbColor:
- Renamed `thumbTintColor` to `thumbColor`.

So I was wrong in claiming thumbTintColor had been deprecated.  I don't know why the message you saw used _thumbColor, as it should have said thumbColor. We should fix that.
I'm OK with renaming the prop in the docs here. I'll edit your change to use thumbColor and get this merged. Thanks for pointing this out and helping us keep the docs up to date!


CONCAT not implemented

see #859
https://qlever.cs.uni-freiburg.de/wikidata/U8lVrR
ParseException, cause: Clause "CONCAT("/event/",SUBSTR(STR(?event),32))" at line 8:22 Built-in function "concat" not yet implemented; if you need it, just add it to SparqlQleverVisitor.cpp::visitTypesafe(Parser::BuiltInCallContext following the already implemented functions there

@joka921  IMHO this is not an exact duplicate - it is a single issue of the multiple ones to be expected when working on #859

I am sorry, I was confused. It is a duplicate of #858 which had already singled out this issue.


DeprecationWarning in init.py:7: bad escape \s

When setting the warnings filter to show DeprecationWarnings, I get a warning when importing Pandas (which I discovered because mne-python changes the global warning settings, see https://github.com/mne-tools/mne-python/issues/2870):
import warnings
warnings.simplefilter("default")
import pandas as pd

This results in the following warning message:
/usr/local/lib/python3.5/site-packages/pandas/__init__.py:7: DeprecationWarning: bad escape \s
from pandas import hashtable, tslib, lib

I have no idea what's wrong with the code in line 7 though. Here's the output of pd.show_versions():
INSTALLED VERSIONS
------------------
commit: None
python: 3.5.1.final.0
python-bits: 64
OS: Darwin
OS-release: 15.3.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: en_US.UTF-8
LANG: en_US.UTF-8

pandas: 0.17.1
nose: 1.3.7
pip: 8.0.2
setuptools: 20.1
Cython: 0.23.4
numpy: 1.10.4
scipy: 0.17.0
statsmodels: None
IPython: 4.1.1
sphinx: 1.3.5
patsy: 0.4.1
dateutil: 2.4.2
pytz: 2015.7
blosc: None
bottleneck: None
tables: None
numexpr: None
matplotlib: 1.5.1
openpyxl: None
xlrd: None
xlwt: None
xlsxwriter: None
lxml: None
bs4: None
html5lib: None
httplib2: None
apiclient: None
sqlalchemy: None
pymysql: None
psycopg2: None
Jinja2: 2.8

This was present in 0.17.1, but has been fixed (I don't have an exact pinpoint, but I rewrote the __init__ a bit recently
You can try out the release candidate by installing vai
conda install pandas=v0.18.0rc1 -c pandas
Python 3.5.1 |Continuum Analytics, Inc.| (default, Dec  7 2015, 11:24:55)
[GCC 4.2.1 (Apple Inc. build 5577)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import warnings
>>> warnings.simplefilter("default")
>>> import pandas as pd
/Users/jreback/miniconda/envs/py3.5/lib/python3.5/site-packages/matplotlib/externals/six.py:80: PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  __import__(name)
>>> pd.__version__
'0.18.0rc1'

Python 3.5.1 |Continuum Analytics, Inc.| (default, Dec  7 2015, 11:24:55)
[GCC 4.2.1 (Apple Inc. build 5577)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import warnings
>>> warnings.simplefilter("default")
>>> import pandas as pd
/Users/jreback/miniconda/envs/py3.5/lib/python3.5/site-packages/pandas/__init__.py:7: DeprecationWarning: bad escape \s
  from pandas import hashtable, tslib, lib
/Users/jreback/miniconda/envs/py3.5/lib/python3.5/site-packages/matplotlib/externals/six.py:80: PendingDeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
  __import__(name)
>>> pd.__version__
'0.17.1'

Nice - thanks!


Request parameters as JWTs

Is it possible to add a feature to pass request parameters as JWTs to the authorization endpoint (as stated here: http://openid.net/specs/openid-connect-core-1_0.html#JWTRequests) ?

dup: #594


Add nightly job

This should permit us to quickly realize if a dependency is breaking the library, so that we can easily understand which change caused the problem.

fyi @CarlottaSartore @S-Dafarra @GiulioRomualdi

C.C. @fl-ferr


[Bug][Manager] MySQL binlog datasource job can not be add successfully in tke env

What happened
the binlog datasource job has been in the 'ToBeAdd' state
What you expected to happen
binlog datasource run successfully
How to reproduce
config a binlog datasource job
Environment
tke
InLong version
master
InLong Component
InLong Manager, InLong Agent
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

it does not happen again, close it.


Adding Frank Li

Merged via https://github.com/Cornell-CS5356-Fall2015/cs5356/commit/1d9297e4b479b35c81365ab513ebaa234ab83c9c.


Add Scheduling for cards

Pull Request Test Coverage Report for Build 619

44 of 62 (70.97%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 89.518%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/logic/parser/Parser.java
0
1
0.0%

src/main/java/seedu/address/model/deck/Card.java
21
22
95.45%

src/main/java/seedu/address/model/Anakin.java
5
7
71.43%

src/main/java/seedu/address/model/deck/UniqueCardList.java
0
2
0.0%

src/main/java/seedu/address/model/ModelManager.java
7
10
70.0%

src/main/java/seedu/address/logic/commands/PerformanceSortCommand.java
0
9
0.0%

Totals

Change from base Build 605:
-0.6%

Covered Lines:
2118

Relevant Lines:
2366

💛  - Coveralls

Please keep the coverage above 90%


serverless offline with angular cli application

Command
serve
Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I am attempting to utilize the application builder instead of webpack as per the instructions provided in the link - https://angular.io/guide/esbuild. However, we encountered the following error while running the application in serverless offline.
undefined ERROR Uncaught Exception {
"errorType": "Runtime.ImportModuleError",
"errorMessage": "Error: Cannot find module './dist/server/server'\nRequire stack:\n- /var/task/lambda.js\n- /var/runtime/index.mjs",
"stack": [
"Runtime.ImportModuleError: Error: Cannot find module './dist/server/server",
"Require stack:",
"- /var/task/lambda.js",
"- /var/runtime/index.mjs",
" at _loadUserApp (file:///var/runtime/index.mjs:1087:17)",
" at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)",
" at async start (file:///var/runtime/index.mjs:1282:23)",
" at async file:///var/runtime/index.mjs:1288:1"
]
}
Please help on this.
Minimal Reproduction
NA
Exception or Error
NA

Your Environment
17

Anything else relevant?
NA

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


Tighten-up a Foundation Deserialization Test

CocoaError.Code.fileReadUnknown conflicts with the CHECK-NOT line here.
We should be checking more specifically for UnknownBlock and UnknownCode
anyways.
rdar://53284293

@swift-ci smoke test and merge

@swift-ci please smoke test Linux platform

⛵


Dependencies don't match between requirements.txt and setup.py

Describe the bug
There are requirements listed in the requirements.txt file (such as shapely >=2.0.0) that are not present in the setup.py file.  Since   the requirements.txt file is not used by the setup.py file, I believe this can cause issues with enforcing dependencies.
Code To Reproduce
CLI commands or Python code snippet to reproduce the bug

Output
Compete output obtained with maximal verbosity.
Environment:

Python version: python --version
sertit-utils version: import sertit; sertit.__version__

Additional context
Add any other context about the bug here.

Hey,
Thanks for this, I'll update it 😄


test: Fix missed BeTrue() in testing

Fixes #N/A
Description
This fixes a missed BeTrue() that should have been present in our testing
How was this change tested?
make presubmit
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.07%) to 81.028%

Files with Coverage Reduction
New Missed Lines
%

pkg/test/expectations/expectations.go
2
95.51%

Totals

Change from base Build<PHONE_NUMBER>:
0.07%

Covered Lines:
8200

Relevant Lines:
10120

💛  - Coveralls


Include LICENSE file in wheels

The license requires that all copies of the software include the license.  This makes sure the license is included in the wheels.  See the wheel documentation here for more information.

Thanks!


Add error boundaries for analysis tabs

Fixes https://github.com/VEuPathDB/web-eda/issues/365

I added error boundaries to just the routes in the analysis panel that rendered tabs. Let me know if you think I should add boundaries to some of the other routes, too.

You can use the first commit above to test with an error; it has an artificial error in the subsetting component.


Detect more errors

Users make a variety of errors, such as this.setImage = ("blah.png");. Need to do as much as possible to detect these. JS has ways of making properties ... "immutable"(?). Research this and other techniques to help them out.

Another technique: more typeof checking on the arguments they pass to sgc.


Update README to reflect unmaintained status

@tmm1, it looks like this fork is not longer being used. The commit was pulled upstream. Do we still need to keep this around?
If we don't, I can delete this repository.
If we do, I'd like to propose that we make it explicit in the README.md that we are not maintaining this and transfer the repository to @github-archive.
Thoughts?

It looks like this fork is being used by our hubot.


Explicitly disconnect example app from domain upon window closing.

Without explicitly disconnecting, the user's avatar remains visible in the domain for 20s or so until the domain server decides that the user hasn't been heard from for too long and so removes the avatar.
Previously: Prevented domain server crash when close or refresh SDK example app window while connected to domain server, though this has since been fixed in https://github.com/vircadia/vircadia/pull/1429 .

We'll merge this upon being fairly confident we've got most of the unhandled crash cases.

I have been running this PR for the last few weeks and have been connecting and disconnecting multiple times with no crashes.
I approve this PR.

We should retest web functionality on Linux and Windows to determine if any crashing is still present before merging this. If there is crashing present, then we should attach a debugger to either instance and solve it, then merge.


Updated external-libraries.md file documentation for using @faker-js/faker

Thanks for taking care of this @cyrilgeorge153 !


[Bug]: Links Not Clickable on Website

What happened?
I found a BUG 🐞
Overview
I found out an issue in the Opportunities Hub Section in which the links which is provided aren't clickable in nature meaning that they are unable to redirect this to the respective site links Provided. Thus i can Fix the issue by identifying the key bug in this .
Add screenshots

What browsers are you seeing the problem on?
Brave
Code of Conduct

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

Kindly Look into this Issue and do let me assign me to fix this issue under HackOctoberFest and GSSOC-extd under any Level. Thanks in Anticipation

📢 Invitation to Join the DevDisplay WhatsApp Community!
We are excited to invite you to join the WhatsApp community of DevDisplay, where developers and contributors come together to collaborate and share their ideas.
Collaborative Support: If you are interested in working together with extensive support, we encourage you to join our community.
Open Source Contribution: If you would like to contribute to the DevDisplay open-source project, your participation is welcomed in our WhatsApp group.
By joining, you will have the opportunity to connect with like-minded individuals, discuss project developments, and contribute to our shared goals.
For more information and to join the community, please visit: Join DevDisplay WhatsApp Community.
We look forward to your participation!

Hey @AdityaP700
We're invited you to work on the issue on Opportunity page
make all the link clickable our team will help you in the providing the main link through you can make it clickable
Let's work on this issue and become a DevDisplay's core Team Member


Enhancement for store.get

Would you consider integrating a pull request that allows for set-ting the alt value of a store.get call in the case the requested key does not exist in the store? As far as I can tell this is currently not possible (in a single call). This enhancement would mean adding a an extra parameter store.get(key[, alt][, write]);.

Yeah, i'd definitely at least consider it. Is there a particular use case or are you just hoping to save a few lines of code? Answering yes to the latter won't keep me from considering it, i'm just curious about the motivation.

Still interested in making a PR for this? Does the new-ish transact function help alleviate need for this?

Shouldn't that kind of logic be handled by the callee?

I think it's better to have a separate api for that, something like store.getOrSet(key, alt).
You can create it yourself pretty simply via the plug-in system:
_.fn('getOrSet', function(key, alt) {
    const value = this.get(key);
    if (value === undefined) {
        this.set(key, alt)
        return alt
    }
    return value;
});

// usage
const value = store.getOrSet(key, 'default value')

I agree with evermediocre, this is probably best as a plugin.


Issue with echo blank lines

There is inconsistency:
term.echo('\n');

prints 2 blank lines, the same as this:
term.echo('\n\n');

but:
term.echo('');

prints nothing

Something to consider should extended commands after remove print the line? term.echo("[[ foo ]]").
Maybe just fix less plugin (that was used when founding this issue) to use:
str.replace(/^$/gm, ' ') to don't have blank lines, or maybe make it work in this way if string is empty don't print anything but if not print every line.

released in version 1.19.0


In order to make commands and interaction with chirp easier to process for devs, i can use external library for CLI interface

User story
parsing arguments are error prone. refactoring results in lower coupling.
Criteria

possible to interact with Chirp through cli after refactoring
arguments received from library not parsed from args[]

2.b)


Introduce and use namespace prefixes for common channels

As discussed in #125 with @kstreeter:

My preference is that we define both a namespace, and a prefix, for all channels. We will then use the prefix to define both metrics and optins.

This makes #124 obsolete.

@trieloff - is this something we document as a pattern to follow in the docs, or do you see a schema way of doing it.
If we want to do this we need to do it asap.


Chrome 17.0.963.46 complains about reserved bits in the websocket connection node 0.6.10

lib/server.js

   // Force xhr-polling if you have issues with

102

   // reserved bits in Chrome and websockets.

103

   //this.io.set('transports', ['xhr-polling'])

This is a work around, but we might want to let people specify the transports supported.

It appears that this is no longer an issue for me.  So I will close this until it reappears.
for the records I'm running: Software  Mac OS X Lion 10.7.1 (11B2118)