Slider reference design

Adds the reference design for the slider module in KiCad, along with some documentation about how the design and footprints were built.
This is largely unchanged from the internal v1.1 reviews. The two READMEs are new, see the footprint README and board README, and include how we generated fabrication data for JLC as well as mounting hole silk footprints from SVGs (that aren't included, though maybe we should make them available as part of the DDK?).
The actual hardware is still untested, though the schematic-equivalent version 1.0 worked fine.

exmaples -> examples

ouch, fixed now. 🍁

Looks good! Do we need to add an explicit license in the examples folder? Are we happy with Creative Commons 4.0?

Disclaimer: I'm not a lawyer, this is not legal advice.
The current CC license we have on the main repo seems to be CC 4.0 Attribution. Two potential issues:

Do we want to require attribution for what is supposed to be a reference design that others could directly build on in making their products?
Because CC does not distinguish between documentation (analogous to source code, like schematics) and products, if we keep Attribution, what is the scope of our intent (and does the text reflect our intent)? For reference, CERN and TAPR OHL (strong copyleft) both make the distinction explicit, as does Solderpad (permissive, Apache 2.0 based - in fact it is Apache 2.0 with expanded definitions for hardware). Note that CC does not recommend their licenses for computer hardware and software with this being one of the reasons.

Does the design pass JLCPCB DRC?

It should be - the board was designed to pretty loose design rules that are within JLC's listed capabilities, and while I don't remember if JLC had automated DRC as part of their ordering process, I didn't get any communications from them about bad design files (aside from a confirmation for tooling holes placement for assembly).

Both fixes done:

changed references from JD-PEC-01 to JD-PEC-02 (except in the 1.1 BoM files that were sent to the board house - I consider those files frozen)
added explanation that LCSC Part # was from an older sample BoM in the readme


Do we want users to be able to override search.followSymlinks?

Testing #223154
FindFiles2OptionsNew.followSymlinks seems to exist for this sole purpose. But search.followSymlinks defaults to true, so a user likely only ever changes it if it was a problem. So why are we letting extension developers override this?

That's a good point. A big use case for this would be where following symlinks slows down search a lot. If the extension knows that resolving symlinks is not necessary, then they could force-disable for better performance.

I still think that I'm leaning towards keeping it for the reason above. I'm taking the viewpoint where I want to keep the API as loose as possible within reason.


More efficient implementation of NCA

arikpoz proposed a simple improvement

This change was merged in gh-48, and affected ITML (not NCA).


Data Quality: Documentação da Stack

Description: Documentar toda a Stack de Data Quality:

[x] Plugin Data Quality Manager
[x] Plugin Data Quality Core
[x] Plugin Data Quality Distribution
[x] Library Data Quality
[x] Library Data Quality Core

https://github.com/stack-spot/hub-dq-plugin-core
https://github.com/stack-spot/hub-dq-plugin-api-manager
https://github.com/stack-spot/hub-dq-plugin-api-distribution
https://github.com/stack-spot/hub-data-quality-libs
https://github.com/stack-spot/hub-data-quality-core-libs


Minor copy edits in prep for beta

List of changes

Remove statement of values from about page - not really relevant
Added disclaimer to front page
Changing name to MAPLE
Remove old, erroneous pages from GGPv1

@nesanders looks like this is based on an old version of master. There are a few other mentions of MATT in the current version. I'll clean those up, push this, and merge.
Here are instructions for how to update your local master branch


nbdev_update_lib errors out when fname not supplied

https://github.com/fastai/nbdev/blob/f8d6ed294425a6b306eeff09a640714dc5ed252d/nbdev/sync.py#L126
When doing nbdev_update_lib without specifying the optional fname argument, it fails with an error as shown below instead of traversing the lib path directory.

I think its caused by this commit https://github.com/fastai/nbdev/commit/f2f4c32c385d3dc7920edd723d73f704bc00255d#diff-d8cd2fcd987f7884b494f7373cf50eb48cd2f925120114e153b08f779e326f95

I think its caused by this commit https://github.com/fastai/nbdev/commit/f2f4c32c385d3dc7920edd723d73f704bc00255d#diff-d8cd2fcd987f7884b494f7373cf50eb48cd2f925120114e153b08f779e326f95

Is a broken link.  Also I cannot reproduce your error make sure you are using the latest version fo nbdev
pip install -U git+https://github.com/fastai/nbdev.git

If upgrading doesn't fix it for you will be happy to reopen this issue please let me know.

If upgrading doesn't fix it for you will be happy to reopen this issue please let me know.

The problem still persists, I already have the latest version of nbdev

I think we need to add a None checker before the if statement in the code. If the default of fname is None and you execute the if statement fname.endswith, it will cause the error seen above.

Thanks.  I was testing the wrong command!  I was testing nbdev_update_lib instead 🤦🏽  sorry.   This is going to be fixed shortly via #416

Thanks. I was testing the wrong command! I was testing nbdev_build_lib instead 🤦🏽 sorry. This is going to be fixed shortly via #416

Thank you sir! 💯


fix docs crasing

Coverage increased (+0.03%) to 88.777% when pulling abe63fcf7de1477da55b8ff97c460880f7f402a8 on bugfix/documentation into f6b304b51f7196d24b452815f88b5f07777e0875 on main.


Build fixed/improvements and readme update

Should build out of the box.
There are still these warnings:
cc1plus: warning: /usr/lib/x86_64-linux-gnu/libGL.so: not a directory
cc1plus: warning: /usr/lib/x86_64-linux-gnu/libGLU.so: not a directory

I see no problems with merging this.


[CS] Use adjusted type for single curry thunks

This matches the double curry thunk logic and ensures that the resulting autoclosure matches the expected type of the reference, avoiding mismatches with parent expressions. This requires fixing concurrency checking such that it respects global actors on autoclosure thunks, but while here it seems like it ought to respect them on autoclosures more generally.
rdar://140212823

@swift-ci please test

@swift-ci please test source compatibility


Limit field when indexing

I have an old table with lot's of fulltext capable fields and cannot remove them. When I am trying to add the indexes I get the response that a maximum of 16 fields are allowed while I have 17. Only three of them contain real data to search on.
It would be nice to have a possibility to limit the fields to create an fulltext search index on and eventually to search in.
A nice way to do this is by adding a $searchable array to the model like this:
use Laravel\Scout\Searchable;

class Product extends Model
{
    use Searchable;

    protected $searchable = [
        'menu',
        'title',
        'text'
    ];
    ...
}

Forget this, I read a little more and it is arranged by toSearchablearray() ...
/**
 * Get the indexable data array for the model.
 *
 * @return array
 */
public function toSearchableArray()
{
    return array_only($this->toArray(), ['id', 'menu', 'title', 'text']);
}


#13 Rebrand

Replace Siempo launcher icon
Replace 'Siempo' with 'Focus launcher' in all texts
Remove 'purchase' section in settings
Replace Siempo's logo with Focus launcher icon + 'Focus launcher' text.

launcher3-alpha-debug.zip

Some issues I found after testing the APK from that .zip shared above:
Siempo verbiage found on this notification:

'Welcome to Siempo! Thank you for your trust'
Update to 'Welcome to Focus Launcher! ...'

The APK you attached still has the URLs and launch verbage (see screenshots)

Update URLs and emails from getsiempo.com to focuslauncher.io destinations

Link privacy policy button to link to focuslauncher.io/privacy
Link terms of service to focuslauncher.io/terms
Update feedback email from<EMAIL_ADDRESS>to<EMAIL_ADDRESS>

Testing tip
I had the original Siempo still installed. I wasn't able to successfully install this Focus Launcher APK until I uninstalled Siempo. I believe this has to do the two currently having the same package name. I documented this in #17

launcher3-alpha-debug.zip


Downloading the flank jars from releases in Github seems to return 403

We have a task to download the flank jar from github releases in our build and looks like it seems to be getting 403s now.
Execution failed for task ':app:downloadFlank'.

> de.undercouch.gradle.tasks.download.org.apache.http.client.ClientProtocolException: Forbidden (HTTP status code: 403, URL: https://github.com/TestArmada/flank/releases/download/v8.0.1/flank.jar)

If i manually click on any jar on the releases, i get a "SignatureDoesNotMatch" error on AWS.

Let me know if you need any other info. Thanks!

https://github.com/TestArmada/flank/releases/download/v8.0.1/flank.jar should be working again.
GitHub had an outage.

Also FYI the new URL is https://github.com/Flank/flank/releases/download/v8.0.1/flank.jar since we graduated to a top level org. You might want to update your scripts. 🙂

Thanks!


Query related to atleast_once & exactly_once

Hi @mohsenrezaeithe, @charles-tan
I am new to snowflake and flink and wanted to understand how are you achieving atleast once in snowflake connector.
I saw the code and understood that data will be flushed when buffer time expires or when a checkpoint is triggered(by invoking flush method of channel through reflection). Is this sufficient to achieve atleast once gurantee here?
I also saw an issue opened on atleast once, to check the offsets. Is the atleast once gurantee still valid with that issue?
Also, to implement it should we maintain a basic counter for offsets for each channel and pass it with insertRow API and check it after every insert that it has been updated at snowflake or not (but snowflake inserts data either at buffer expiry or checkpoint, so offset will not be updated immediately at snowflake)? So how can we do it?
Also, by any chance have you explored what changes do we need to make to this connector to support exactly once? Because flink needs TwoPhaseCommit supporting sink to support exactly once, but snowflake does not support commiting or rolling back transactions.
Any help is much appreciated, thanks in advance!!

hi @prateekkohli21
Yes, we're relying on the flush() call during checkpointing to ensure at least once. If the flush() fails, then the Flink checkpoint will fail and the Flink job will reprocess data starting from the last successful checkpoint.
I think your intuition is correct about checking the offsets. Based on our understanding of the Snowflake flush() mechanism, if it succeeds, then that means everything was written and is enough to satisfy at least once. Checking the offsets will give us further validation, which is why we've ticketed it as an improvement to this connector. Since we force a flush() during checkpointing, we can check for the offsets after the flush() call is complete and it should match our counter of records inserted into Snowpipe.
We have given some thought into exactly once but weren't satisfied with any of the the solutions we thought of. Exactly once semantics work in the Flink Kafka sink connector because Kafka supports creating transactions, as you mentioned, but Snowpipe only exposes the offset which is not enough to do two phase commit.

Thanks for the reply @charles-tan
But when I try to run atleast once I can see that number of records keep on changing in the snowflake table even after flushing. Which maybe means that it is still has some data in buffers which it is writing to the snowflake table after flush.
I captured below output after calling the flush and just counting the number of records in my table using JDBC:
No of records for checkpoint id : 3 is 45565
No of records for checkpoint id : 3 is 61505
No of records for checkpoint id : 3 is 91825
No of records for checkpoint id : 3 is 91825
No of records for checkpoint id : 3 is 91825
As you can see the it keeps on changing when I execute the count(*) query again and again and settles after sometime.

@prateekkohli21 additional validation for at-least-once was added in 1.0.11-1.18. Closing this as there isn't anything else that can be provided in this area.
Please follow up with new issues if you have further questions.


Need some more functional information

Hi John
I have seen your date paginator it working fine.
But i need some more functionality from you so can you please guide me?
I want to add date paginator with table ( responsive that table as per date paginator. )
For more information see attached image.
So please tell me how to do that.
Thanks
Vijay Maheriya

You could use the selectedDateChanged event to update the table when the paginator is clicked.
However, adding a table to the paginator or attempting is keep the two in sync is clearly out of scope.  It would be down to you, to implement this functionality.

Thanks for reply
Date paginator update after resize ( Means you are show hide cell on width of device ) so how to update my table after resize like date paginator?
I want to just resizing functionality for my table so please tell me if you have any suggestion
Thanks
Vijay Maheriya

Your best bet is to look in the date-paginator code, basically it's dynamically calculated.

ok john i will look into code and thanks for help
Thanks
Vijay Maheriya


AttributeError in scrape_with_playwright when response is None

Issue
Bug Description
The scrape_with_playwright method in the Scraper class raises an AttributeError when the response object is None. This occurs when a page fails to load, causing the code to attempt to access header_value on a NoneType object.
Steps to Reproduce

Use the Scraper class to scrape a URL that results in a failed or empty response.
Observe that an AttributeError is raised in the scrape_with_playwright method.

Expected Behavior
The method should handle the case where the response object is None gracefully, without raising an AttributeError.
Actual Behavior
When the response object is None, the following error is thrown:
AttributeError: 'NoneType' object has no attribute 'split'
Code Reference
The issue occurs in this section of the code:
response.header_value("content-type").split(";")[0] if response else None

Version and model info
Aider v0.54.6
Playwright v1.46.0

Thanks for trying aider and filing this issue.
The change is available in the main branch. You can get it by installing the latest version from github:
aider --install-main-branch

# or...

python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.


MVC filename spelling mistake in _Layout_Boostrap.cshtml

The .net nuget package for Bootstrap contains an MVC file whose name is misspelled - _Layout_Boostrap.cshtml should be _Layout_Bootstrap.cshtml

We don't own or manage that. File a bug against them please.


[GSSoC'23] Adding content to the Home Page

The home of the WebApp has no content and hence doesn't look pleasing.
I would like to add relevant content in it and hence enhance the user experience.
Please assign the issue to me.

Someone is already working on it, you can check it out later and enhance the the UI

Sure thanks, if any other I could help with please do let me know.


fix(connect): sometimes open sql console result in 404 error

What type of PR is this?
type-bug

What this PR does / why we need it:
use url base64 encoder or decoder to encode or decode the session id to avoid '/' char.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

https://stackoverflow.com/questions/43089541/difference-between-basic-and-url-base64-encoding-in-java-8

ref: https://stackoverflow.com/questions/43089541/difference-between-basic-and-url-base64-encoding-in-java-8


Copying to clipboard not working by default on OSX 10.13.6

May need to include this
bind -T copy-mode M-w send -X copy-pipe "reattach-to-user-namespace pbcopy" \; display-message "copied to system clipboard"
Reference: https://github.com/tmux/tmux/issues/909

Could not replicate it. Not sure what I was doing wrong.


use on_finalize for creating new mmr leaves

This produces new mmr leaves on_finalize so we can be sure leaf data is canonical.
closes: #11797

@acatangiu anyway we can reopen this?

@acatangiu anyway we can reopen this?

we need to do https://github.com/paritytech/substrate/issues/11799 first


[WIP] updating method to ingest external content

Signed-off-by: RinkiyaKeDad<EMAIL_ADDRESS>Fixes #93

Heyy @RinkiyaKeDad is there still any interest to get this PR in?
 I was working on this a few months ago but was having a hard time figuring out the implementation details. I currently don't have the bandwidth to take this on, so I'll close this PR. Thanks for the nudge! :)


Web UI searches not loading

System OS
Windows
Python Version
3.10 (CPython)
Install Source
pip / PyPi
Install version / commit hash
v4 rc2
Expected Behavior vs Actual Behavior
Web UI.

so i search up a song and it works fine. and then i search up for another song (different names) the page shows the first song results

Steps to reproduce - Ensure to include actual links!
After downloading song, search for second song which will not load sometimes.
On page reload, only json returned {"detail":"Not Found"}
Traceback
Updating web app
INFO:     Started server process [12240]
17:11:25 - INFO - Started server process [12240]
INFO:     Waiting for application startup.
17:11:25 - INFO - Waiting for application startup.
INFO:     Application startup complete.
17:11:25 - INFO - Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8800 (Press CTRL+C to quit)
17:11:25 - INFO - Uvicorn running on http://<IP_ADDRESS>:8800 (Press CTRL+C to quit)
INFO:     <IP_ADDRESS>:5829 - "GET / HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5829 - "GET /assets/index.fe127150.js HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5829 - "GET /assets/index.d96dcc94.css HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5829 - "GET /assets/vendor.7698a149.js HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5829 - "GET /api/settings HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5829 - "GET /assets/spotdl.5dcb64c0.svg HTTP/1.1" 200 OK
INFO:     ('<IP_ADDRESS>', 5836) - "WebSocket /api/ws" [accepted]
17:11:26 - INFO - ('<IP_ADDRESS>', 5836) - "WebSocket /api/ws" [accepted]
INFO:     connection open
17:11:26 - INFO - connection open
INFO:     <IP_ADDRESS>:5829 - "GET /favicon.ico HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5847 - "GET /api/songs/search?query=survival+tactics HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5847 - "POST /api/download/url?url=https:%2F%2Fopen.spotify.com%2Ftrack%2F4pNT8XTKWb2vhcsqwzdXCT&client_id=1779a702-6f1b-4056-b995-2e8e962a176d HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5870 - "GET /api/songs/search?query=flashing+lights HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5874 - "GET /search/flashing%20lights HTTP/1.1" 404 Not Found
INFO:     connection closed
17:13:25 - INFO - connection closed
INFO:     <IP_ADDRESS>:5874 - "GET /favicon.ico HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:5875 - "GET / HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:5875 - "GET /assets/index.fe127150.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:5875 - "GET /assets/index.d96dcc94.css HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:5875 - "GET /assets/vendor.7698a149.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:5875 - "GET /api/settings HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5875 - "GET /assets/spotdl.5dcb64c0.svg HTTP/1.1" 304 Not Modified
INFO:     ('<IP_ADDRESS>', 5882) - "WebSocket /api/ws" [accepted]
17:13:33 - INFO - ('<IP_ADDRESS>', 5882) - "WebSocket /api/ws" [accepted]
INFO:     connection open
17:13:33 - INFO - connection open
INFO:     <IP_ADDRESS>:5886 - "GET /api/songs/search?query=survival+tactics HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5886 - "POST /api/download/url?url=https:%2F%2Fopen.spotify.com%2Ftrack%2F4pNT8XTKWb2vhcsqwzdXCT&client_id=7f1a4190-8d20-44af-ba08-cb284b67e14c HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5898 - "GET /api/songs/search?query=flashing+lights HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5902 - "GET /api/download/file?file=Joey%20Bada$$,%20Capital%20Steez%20-%20Survival%20Tactics.mp3&client_id=7f1a4190-8d20-44af-ba08-cb284b67e14c HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5902 - "GET /api/songs/search?query=flashing+lights HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5906 - "GET /api/songs/search?query=flashing+lights HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:5914 - "GET /api/songs/search?query=Kanye HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:6059 - "GET /search/Kanye HTTP/1.1" 404 Not Found
INFO:     connection closed
17:28:27 - INFO - connection closed
INFO:     <IP_ADDRESS>:6059 - "GET /favicon.ico HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET / HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET /assets/index.fe127150.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET /assets/index.d96dcc94.css HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET /assets/vendor.7698a149.js HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET /api/settings HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:6086 - "GET /assets/spotdl.5dcb64c0.svg HTTP/1.1" 304 Not Modified
INFO:     <IP_ADDRESS>:6086 - "GET /favicon.ico HTTP/1.1" 304 Not Modified
INFO:     ('<IP_ADDRESS>', 6093) - "WebSocket /api/ws" [accepted]
17:30:53 - INFO - ('<IP_ADDRESS>', 6093) - "WebSocket /api/ws" [accepted]
INFO:     connection open
17:30:53 - INFO - connection open
INFO:     <IP_ADDRESS>:6090 - "GET /api/songs/search?query=Kanye HTTP/1.1" 200 OK

Other details
User reported on Discord https://discord.com/channels/771628785447337985/1007941869252116580

can confirm. trying v4.rc3 on a mac and the same happens. I will have to stop the process and rerun spotdl web

Looks like the issue is that, when entering the index page directly through the / path, as is done after executing spotdl web, the Vue app is initialized, and any subsequent navigation in the web UI is handled by Vue itself. When reloading the page, the navigation is not handled by Vue, routing the request to the FastAPI backend instead, which leads to the 404 error. I'm not quite sure what the solution would be, though :/

looks fixed on v4.0.1


Resize coordinate_transformation_mode=pytorch_half_pixel is not supported in Tensorflow

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):conda
TensorFlow version (use command below):2.2
Python version:3.8
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: None
GPU model and memory: None

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
Resize coordinate_transformation_mode=pytorch_half_pixel is not supported in Tensorflow
Describe the expected behavior
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@tensorflowbutler Nice

@Adhithya-tech,
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here and also the dataset you are using. Thanks!

@amahendrakar I think

This is a known limitation, as seen in notes section of https://github.com/onnx/onnx-tensorflow/blob/master/doc/support_status_v1_7_0.md.

This should solve it

Anyways, it's basically converting a faster RCNN model to tensorfow using onnx in between

Anyways, it's basically converting a faster RCNN model to tensorfow using onnx in between

Without a sample code it would be difficult for us to debug the issue. Hence, could you please provide a minimal code snippet to reproduce the issue or share the Python script/notebook you are running. Thanks!

@amahendrakar
Here is one text boundary detection model I'm using. Let me know if this helps -
Install ONNX-Tensorflow
git clone git@github.com:onnx/onnx-tensorflow.git && cd onnx-tensorflow
pip install -e .
Code snippet:
Model - Link
import cv2
import torch
import numpy as np
from skimage import io
import sys
sys.path.append("path/to/cloned/folder/onnx-tensorflow")
import onnx
from onnx_tf.backend import prepare

# Load the ONNX file
model = onnx.load('onnx_export/detect2.onnx')  #Model File is provided in this above link

# Prepare the Image Input

img = io.imread(img_path)      # Any google image with text on it will do.
img = cv2.cvtColor(img, cv2.COLOR_GRAY2RGB)
img = np.array(img)
img = img.copy().astype(np.float32)
img -= np.array([0.485 * 255.0, 0.456 * 255.0, 0.406 * 255.0], dtype=np.float32)
img /= np.array([ 0.229 * 255.0, 0.224 * 255.0, 0.225 * 255.0], dtype=np.float32)

x = torch.from_numpy(img).permute(2, 0, 1)  # Converts [h, w, c] to [c, h, w]
x = Variable(x.unsqueeze(0))                           # Converts [c, h, w] to [b, c, h, w]

# Import the ONNX model to Tensorflow
tf_rep = prepare(model)

# Run the forward pass
detections, features = tf_rep.run(x)   # This function give error as stated in title

Thanks !

@Adhithya-tech, have you resolved this issue with converting a Torch FasterRCNN model to TF through ONNX? Through some digging, the error is caused by the Resize op that the FasterRCNN model does regardless of the backbone. Buried within the GeneralizedRCNNTransform class is this interpolate method call:
image = torch.nn.functional.interpolate(image[None], size=size, scale_factor=scale_factor, mode='bilinear',
                                            recompute_scale_factor=recompute_scale_factor, align_corners=False)[0]

The key is to set align_corners to True and retrain the model.

I am also experiencing the same error. I am using mmsegmentation pytorch framework. And as @alexispascual pointed interpolate function is also the problem:
https://github.com/open-mmlab/mmsegmentation/blob/master/mmseg/ops/wrappers.py#L26
I fixed the operation to use those values:
mode='bilinear'
align_corners=True
return F.interpolate(input, size=size, scale_factor=scale_factor, mode=mode, align_corners=align_corners)

but I am still getting the same error:
RuntimeError: in user code:
    /opt/miniconda3/envs/tf_exporter/lib/python3.8/site-packages/onnx_tf/backend_tf_module.py:98 __call__  *
        output_ops = self.backend._onnx_node_to_tensorflow_op(onnx_node,
    /opt/miniconda3/envs/tf_exporter/lib/python3.8/site-packages/onnx_tf/backend.py:328 _onnx_node_to_tensorflow_op  *
        return handler.handle(node, tensor_dict=tensor_dict, strict=strict)
    /opt/miniconda3/envs/tf_exporter/lib/python3.8/site-packages/onnx_tf/handlers/handler.py:58 handle  *
        cls.args_check(node, **kwargs)
    /opt/miniconda3/envs/tf_exporter/lib/python3.8/site-packages/onnx_tf/handlers/backend/resize.py:125 args_check  *
        exception.OP_UNSUPPORTED_EXCEPT(
    /opt/miniconda3/envs/tf_exporter/lib/python3.8/site-packages/onnx_tf/common/exception.py:50 __call__  *
        raise self._func(self.get_message(op, framework))

    RuntimeError: Resize coordinate_transformation_mode=pytorch_half_pixel is not supported in Tensorflow.

I spent a lot on this issue,too.


Time Lapse photography

Thanks for this excellent API.
I haven't been able to work out how to set up a record time lapse photographs.
Basically I want to do the following.

Connect to goPro via wifi (this is fine)
programmatically start recording photos (not video) at a predefined time

eventually I will have multiple cameras that I want to synchronize

Continue taking images every 5 seconds
Manually stop when finished (this part is trivial)

This is for an underwater project where the gopro(s) will be on a rov. So the wifi setup is only possible during initialization and wifi control (and manual control) will be lost once the GoPro is underwater.
Any advice will be much appreciated
Alan Buchanan

GoPro cameras have timelapse feature, so you can do something like:

from goprocam import GoProCamera, constants
gopro = GoProCamera.GoPro()

def start():
    gopro.gpControlSet(constants.Multishot.TIMELAPSE_INTERVAL, constants.Multishot.TimeLapseInterval.I5)
    gopro.mode(constants.Mode.MultiShotMode, constants.Mode.SubMode.MultiShot.TimeLapse)
    gopro.shutter(constants.start)

def stop():
    gopro.shutter(constants.stop)

#Run start() to set up the camera to take a picture every 5 seconds
#Run stop() to stop the timelapse once the WiFi is available on the surface

Hi. Is it possible to get the GoPro to have a greater interval of 1 min between photos?  The standard max time on the timelapse feature is only 1 min? Could this be set much greater in the region of 10-20mins? I had looked at building an Intervalometer, but that bit advanced for me.  For the occasional long timelapse I do I dont want to purchase something like this [BlinkX] (https://cam-do.com/products/blinkx-time-lapse-camera-controller-for-gopro-hero5-6-7-8-cameras) which is $400 :(. Thanks in advance.


[ISO] (9660 at least) plugin creation

43 44 30 30 31         CD001 ISO         ISO-9660 CD Disc Image
This signature usually occurs at byte offset 32769 (0x8001), 34817 (0x8801), or 36865 (0x9001).
More information can be found at MacTech or at ECMA.
43 44 30 30 31 | CD001 | 0x8001 0x8801 0x9001 | iso | ISO9660 CD/DVD image file

"CD001", // ECMA_119
"NSR03", // ECMA_167
"NSR02", // ECMA_167_PREVIOUS
"BEA01", // ECMA_167_EXTENDED
"BOOT2", // ECMA_167_BOOT
"TEA01", // ECMO_167_TERMINATOR
"CDW02"  // ECMA_168

At the moment only Plain ISO is supported.

Add UDF format - @Cosmin765.
Universal Disk Format (UDF) is an open, vendor-neutral file system for computer data storage for a broad range of media. In practice, it has been most widely used for DVDs and newer optical disc formats, supplanting ISO 9660.
https://datatracker.ietf.org/doc/draft-hallambaker-udf/02/
http://www.osta.org/specs/


Can this method support deepdanbooru's caption?

I just want to know, did this method can support deepdanbooru's txt of tags, I can not determine, but it seems not really worked?

DreamArtist does not need to be trained with caption, unless you want to exclude some features from learning.


Problem with multi plugins.

@sgkim126 ,
Currently even though the following config is set in application.prod.conf like below, it doesn't override the one in application.conf.
In application.prod.conf:
  plugin {
    paths {
      logicServer = ["/Users/noraesae/Works/company100/beyond/logicServer"]
    }
  }

In application.conf:
  plugin {
    paths {
      example = ["plugins"]
    }
  }

It causes an error that main.js can't be found when an app is run as production mode and accessed with the plugin path specified in application.prod.conf. Can it be fixed with source modification, or should we reconsider the design of the paths configuration?
For the detail context, please feel free to ask me or @murmur76.

@sgkim126 have you checked this? Since our logic server has to be integrated with its client this week, i'd like you to look into this issue as soon as you can.

Sorry, I'll check it untll wednesday morning.

I think I got the reason.
It's because plugin.paths is not for overriden, it's for appended.
So although you launch it activated mode, beyond think you want example plugin, and fail to load example plugin, because production mode should use absolute path.

I pr #214

The patch merged. :+1:


ValueError: Invalid format string

Please read the following instructions carefully

Issues submitted without this template format will be ignored.
I can't help you with python / ffmpeg installation problems so issues related to those will probably not get a response from me.
Please read the questions asked and answer completely.
Put an x into all the boxes [ ] relevant to your issue (like so [x] - no spaces).
Do not post screenshots of text errors or logs.
Use the Preview tab to see how your issue will actually look like.

Before submitting an issue, make sure you have:

[Y ] Read the README
[Y] Read the FAQ
[Y] Searched the bugtracker for similar issues including closed ones
[Y] Make sure you have the latest version: v0.2.9

Purpose of your issue:

[Y] Bug report (problems/errors)
[Y] Question
[ ] Other

For a bug report, you must include the command used and a complete output/error log.
Command:
livestream_dl -u "myid" -p "<mypassword>" "livestreamid"

Complete Output/Error Log:
# Traceback (most recent call last):
  File "C:\Users\***\AppData\Local\Programs\Python\Python36-32\Scripts\live
stream_dl-script.py", line 11, in <module>
    load_entry_point('instagram-livestream-downloader==0.2.9', 'console_scripts'
, 'livestream_dl')()
  File "c:\users\***\appdata\local\programs\python\python36-32\lib\site-pac
kages\livestream_dl\__main__.py", line 5, in main
    run()
  File "c:\users\***\appdata\local\programs\python\python36-32\lib\site-pac
kages\livestream_dl\download.py", line 356, in run
    download_start_time = int(now.strftime('%s'))
ValueError: Invalid format string

Describe your issue
I ever can download the video file
but I don't know what happened
I try to uninstall and install again
but it's still show 'ValueError: Invalid format string'
I see from other Issues and do the same thing
but it's still show the massage like that
i try to use the command
livestream_dl -u "<username>" "<instagram-live-username>"
but I can't type or paste the password
and I have some question, how to make loop in .bat file
I try to search in google but I still don't understand
or can you tell me where I should start with
sorry for my english
and thank you very much that you make this :)

Thank for reporting the error. I've fixed it in the latest version so please update your install to v0.3.0 and let me know if it solves the ValueError.
The program will prompt you to enter the password. In the latest version, you will see a prompt like this:
Type in the password for <username> and press "Enter"
(Your password will not show on screen):

Just type in your password and press enter. Nothing will appear on screen but it is okay.
Regarding the .bat file, I don't have a Windows machine so I can't offer you any advice. Maybe in the future if I have time and can borrow a Windows PC, I will write an example.

To make a loop in a batch file you can use this:

cd c:\Livestream_dl
:loop
livestream_dl -u "myid" -p "mypassword" "livestreamid"
timeout 10 > nul
goto loop

I recommend you to cd to the directory first where you also save the livestreams to as shown in the batch example.
I've found that if you use a config file you'll need to cd to the directory where the config file is located before running the livestream_dl command.

thanks for answer! thank you so much :D
now i can download
but the result is mp4.tmp and m4a.tmp file
it's ok, i can change it to .mp4 and . m4a and take it together
but it's have a way to fix it?
(it's same as the first time that i download before i have that problem)
it was showed
Assembling files.... Traceback (most recent call last):   File "C:\Users\***\AppData\Local\Programs\Python\Python36-32\Scripts\live stream_dl-script.py", line 11, in <module>     load_entry_point('instagram-livestream-downloader==0.3.0', 'console_scripts' , 'livestream_dl')()   File "c:\users\***\appdata\local\programs\python\python36-32\lib\site-pac kages\livestream_dl\__main__.py", line 5, in main     run()   File "c:\users\***\appdata\local\programs\python\python36-32\lib\site-pac kages\livestream_dl\download.py", line 528, in run     dl.stitch(final_output, skipffmpeg=userconfig.skipffmpeg, cleartempfiles=(no t userconfig.nocleanup))   File "c:\users\***\appdata\local\programs\python\python36-32\lib\site-pac kages\instagram_private_api_extensions\live.py", line 356, in stitch     exit_code = subprocess.call(cmd)   File "c:\users\***\appdata\local\programs\python\python36-32\lib\subproce ss.py", line 267, in call     with Popen(*popenargs, **kwargs) as p:   File "c:\users\***\appdata\local\programs\python\python36-32\lib\subproce ss.py", line 707, in __init__     restore_signals, start_new_session)   File "c:\users\***\appdata\local\programs\python\python36-32\lib\subproce ss.py", line 990, in _execute_child     startupinfo) FileNotFoundError: [WinError 2] The system cannot find the file specified
about the batch file
it's WORK !!!!
/ for download from many id i can make many .bat file right?
or it have easier way?
thank you again :D ♥
(and sorry for my poor english lol)

Yes you can just have multiple batch files running at once for each id.

@nppht Did you install ffmpeg? From the command line, what happens when you enter ffmpeg?

@nppht It looks like you don't have ffmpeg installed, or if you did, you did not set it properly in %PATH%.
May be this will help http://adaptivesamples.com/how-to-install-ffmpeg-on-windows/
For the batch file, the easiest way is probably to just add lines for each additional user.
cd c:\Livestream_dl
:loop
livestream_dl -u "myid" -p "mypassword" "livestreamid1"
timeout 10 > nul
livestream_dl -u "myid" -p "mypassword" "livestreamid2"
timeout 10 > nul
livestream_dl -u "myid" -p "mypassword" "livestreamid3"
timeout 10 > nul
goto loop

@ping thanks for tell me about batch files!
now I make new account so my main account (maybe) don't get blocked from ig 😂
@taengstagram I forget to add it to the patch!!
now I can download as mp4 file!!
thanks!!

(maybe this is the (2+) last question)
for batch file, if 2 user do the live in the same time (I mean in 1 batch file)
it will have problem?
or is better if I make many batch file?
and if I increase the timeout it will be missed at the first part of livestream right?
(maybe it's just 4-5 second? I just ask for sure)

thank you so so so much !! ♥
and sorry again for my poor english

The IG block is not a block on your account, it's a temporary block on your IP address for about 15-30 minutes (can't remember exactly). So even if you switch to another account, as long as you're on the same IP, you won't be able to access IG at all until the block has ended.
If you have 2 or more users in one .bat file, the later user's stream will be downloaded when the earlier user's stream has ended.
If you increase the timeout, yes, you may miss more of the initial stream. There are always limits, if you want to check more users, you must increase the timeout. If you don't want to miss too much of a stream, don't check so many users.
The minimum safe timeout is 5 seconds. So if you use 2 .bat files, you must make timeout in each .bat file be 2*5 = 10 seconds.
For example: You have 3 .bat files running, so each .bat must have 3*5 = 15 seconds timeout
cd c:\Livestream_dl
:loop
livestream_dl -u "myid" -p "mypassword" "livestreamid3"
timeout 15 > nul
goto loop


Reinstate the specific create view methods in WindowManager.

More verbose but more intuitive.

this is far more intuitive!
I'm all for having them in the API
(despite they duplicate code in a way)
thank you @tinevez for adding them


should preserve the corresponding order of tasks and num_fewshot arguments

main.py won't preserve corresponding order of tasks and num_fewshot arguments.
in this PR, num_fewshot and limit args are provided to each task respecting the order.
if we run harness.sh like this:
MODEL_ARGS="pretrained=cyberagent/open-calm-1b"
TASK="jcommonsenseqa-1.1-0.2,jnli-1.1-0.2,marc_ja-1.1-0.2,jsquad-1.1-0.2,xlsum_ja,jaqket_v2-0.1-0.2"
python main.py --model hf-causal --model_args $MODEL_ARGS --tasks $TASK --num_fewshot "2,3,3,3,1,1" --device "cuda" --output_path "models/cyberagent-open-calm-1b/result.json"

we expect that num_fewshot is provided each task in the same order as tasks.
but, the harness.sh results in that the corresponding order of tasks and num_fewshot won't be preserved like that:
{
  "results": {
    "jaqket_v2-0.1-0.2": {
      "exact_match": 2.9116465863453813,
      "f1": 4.696147504882445
    },
    "jsquad-1.1-0.2": {
      "exact_match": 37.12291760468258,
      "f1": 47.16735348285718
    },
    "xlsum_ja": {
      "rouge2": 2.288077088085482
    },
    "jcommonsenseqa-1.1-0.2": {
      "acc": 0.26988382484361034,
      "acc_stderr": 0.013275885907507282,
      "acc_norm": 0.24754244861483468,
      "acc_norm_stderr": 0.01290758346346734
    },
    "jnli-1.1-0.2": {
      "acc": 0.33566146261298274,
      "acc_stderr": 0.00957358086224245,
      "acc_norm": 0.3331963845521775,
      "acc_norm_stderr": 0.009556042193601356
    },
    "marc_ja-1.1-0.2": {
      "acc": 0.8434736469755925,
      "acc_stderr": 0.004832701818807143,
      "acc_norm": 0.8434736469755925,
      "acc_norm_stderr": 0.004832701818807143
    }
  },
  ..
  "config": {
    "model": "hf-causal",
    "model_args": "pretrained=cyberagent/open-calm-1b",
    "num_fewshot": [
      2,
      3,
      3,
      3,
      1,
      1
    ],
  ..
  }
}

@kumapo hi thank you for reporting. Actually the didn't happen to me. @polm-stability could you also check this issue?

$ MODEL_ARGS="pretrained=rinna/japanese-gpt-neox-3.6b-instruction-ppo,use_fast=False"
$ TASK="jsquad-1.1-0.4,jcommonsenseqa-1.1-0.4,jnli-1.1-0.4,marc_ja-1.1-0.4,xlsum_ja-1.0-0.4"
$ python main.py  \
    --model hf-causal \
    --model_args $MODEL_ARGS \
    --tasks $TASK \
    --num_fewshot "2,3,3,3,1" \
    --device "cuda" \
    --limit "5,5,5,5,5"
WARNING: --limit SHOULD ONLY BE USED FOR TESTING. REAL METRICS SHOULD NOT BE COMPUTED USING LIMIT.
Selected Tasks: ['jcommonsenseqa-1.1-0.4', 'jnli-1.1-0.4', 'jsquad-1.1-0.4', 'marc_ja-1.1-0.4', 'xlsum_ja-1.0-0.4']
Using device 'cuda'
Running loglikelihood requests
0it [00:00, ?it/s]
Running greedy_until requests
0it [00:00, ?it/s]
{
  "results": {
    "jcommonsenseqa-1.1-0.4": {
      "acc": 0.2,
      "acc_stderr": 0.2,
      "acc_norm": 0.2,
      "acc_norm_stderr": 0.2
    },
    "jnli-1.1-0.4": {
      "acc": 0.8,
      "acc_stderr": 0.20000000000000004,
      "acc_norm": 0.6,
      "acc_norm_stderr": 0.24494897427831783
    },
    "marc_ja-1.1-0.4": {
      "acc": 1.0,
      "acc_stderr": 0.0,
      "acc_norm": 1.0,
      "acc_norm_stderr": 0.0
    },
    "jsquad-1.1-0.4": {
      "exact_match": 40.0,
      "f1": 40.0
    },
    "xlsum_ja-1.0-0.4": {
      "rouge2": 1.3559322033898304
    }
  },
  "versions": {
    "jcommonsenseqa-1.1-0.4": 1.1,
    "jnli-1.1-0.4": 1.1,
    "jsquad-1.1-0.4": 1.1,
    "marc_ja-1.1-0.4": 1.1,
    "xlsum_ja-1.0-0.4": 1.0
  },
  "config": {
    "model": "hf-causal",
    "model_args": "pretrained=rinna/japanese-gpt-neox-3.6b-instruction-ppo,use_fast=False",
    "num_fewshot": [
      2,
      3,
      3,
      3,
      1
    ],
    "batch_size": null,
    "device": "cuda",
    "no_cache": false,
    "limit": [
      5,
      5,
      5,
      5,
      5
    ],
    "bootstrap_iters": 100000,
    "description_dict": {}
  }
}
hf-causal (pretrained=rinna/japanese-gpt-neox-3.6b-instruction-ppo,use_fast=False), limit: 5,5,5,5,5, provide_description: False, num_fewshot: 2,3,3,3,1, batch_size: None
|         Task         |Version|  Metric   | Value |   |Stderr|
|----------------------|------:|-----------|------:|---|-----:|
|jcommonsenseqa-1.1-0.4|    1.1|acc        | 0.2000|±  |0.2000|
|                      |       |acc_norm   | 0.2000|±  |0.2000|
|jnli-1.1-0.4          |    1.1|acc        | 0.8000|±  |0.2000|
|                      |       |acc_norm   | 0.6000|±  |0.2449|
|marc_ja-1.1-0.4       |    1.1|acc        | 1.0000|±  |0.0000|
|                      |       |acc_norm   | 1.0000|±  |0.0000|
|jsquad-1.1-0.4        |    1.1|exact_match|40.0000|   |      |
|                      |       |f1         |40.0000|   |      |
|xlsum_ja-1.0-0.4      |    1.0|rouge2     | 1.3559|   |      |

the problem does not happen if you always put tasks in dictionary order.

@kumapo oh i finally got your point. We should definitely fix the issue. Thanks for finding!

if it seems to be much changes, just not to call sorted() is another option.

Are there any cons when remoing sorted? I believe it doesn't harm at all.

Thanks for fixing this, looks good to me!

@mkshing
removing sorted does not have any downsides, just may change task execution order as previous.
but it causes unexpected behaviors in the corner case that both nonexistent task and num_fewshot are given.
the nonexistent task is skipped without any errors but num_fewshot is not skipped. so, the num_fewshot is to be passed to another task. thats's why I did not remove sorted, to be on safe side.
but, for simplicity it's better just not to call sorted.

@kumapo that's good to know!

the nonexistent task is skipped without any errors
Can we remove sorted and raise an error in this case? cc: @polm-stability

Sounds fine. If there's a mismatch in the number of num_fewshot and task arguments we should definitely raise an error.
It's a bigger refactor, but I was thinking it would be better to include the fewshot number in the task spec to avoid all these issues with mismatches, something like: task-1.1:2,othertask-2.0:3, where the number after the colon is the fewshot number.

@polm-stability that's a good idea.

It's a bigger refactor, but I was thinking it would be better to include the fewshot number in the task spec to avoid all these issues with mismatches, something like: task-1.1:2,othertask-2.0:3, where the number after the colon is the fewshot number.

I'm wondering what the command line looks like in the case both limit and num_fewshot are given.
but, we can skip thinking about limit because it should be used only for testing for now.

So, at least, in this PR, we want to do the following things if I understand correctly.

fix the bug by removing sorted
raise errors when

number of fewshot != number of tasks
number of limit != number of tasks if limit is not None
nonexistent task are passed

@kumapo I know you're busy but I am wondering if you can fix further. Thanks!

sorry, I found this was a mistake. MultiChoice validates tasks and raise an error if nonexistent task given.

the nonexistent task is skipped without any errors

@kumapo Cool! Thank you for fixing!


Make docs anchorjs links darker on keyboard focus

Not just on mouse hover

Current look when focused with keyboard - low/failing contrast ratio

sorry, got mixed-up with the labels...meant for backport as well


MM-32246: fix welcome rhs as sysadmin

Summary
The welcome RHS incorrectly prompted sysadmins to create an incident if there were /any/ playbooks in the team, regardless of whether the sysadmin was a member. Fix this by adding an optional member_only parameter to the query and use it when testing for playbooks.
Ticket Link
Fixes: https://mattermost.atlassian.net/browse/MM-32246

/update-branch


Switch to xml for coverage reports

This branch provides a fix for the codecov failures. The previously generated .coverage file generated by running the coverage tool was in an invalid format for Github's codecov. This branch switches the format over to xml, which can be processed by codecov

Nice. I struggled with codecov and gave up. Thanks for fixing this.

My pleasure. It looks like the coverage reports are working now, but we still get the message "Waiting for CI to complete". This didn't happen on my fork, oddly enough. Should we merge this in and monitor what the CI does in develop?

Yes, please merge this and let's see what happens.


Parsing of grpc-web trailers is incorrect

See https://github.com/timostamm/protobuf-ts/blob/fefa69ba2bbad48aa6e28264b90b564b0f9846ec/packages/grpcweb-transport/src/grpc-web-format.ts#L303
And the docs for String#split()

Limit - A non-negative integer specifying a limit on the number of substrings to be included in the array. If provided, splits the string at each occurrence of the specified separator, but stops when limit entries have been placed in the array. Any leftover text is not included in the array at all.

This means that a response trailer that looks like this
grpc-status:9
grpc-message:Precondition failed for updated time (actual:<PHONE_NUMBER>, expected:<PHONE_NUMBER>)

Will result in an error being thrown like this: RpcError: Precondition failed for updated time (actual
The official grpc-web client parses the trailers with https://github.com/grpc/grpc-web/blob/3956560ad01b4af0a2a5c29c081f5bbd1424e85d/javascript/net/grpc/web/grpcwebclientreadablestream.js#L349-L358 where it finds the first index of :, everything before that is key and everything after is value.
This seems like an easy fix. I'll try to put up a PR to address this over the weekend.

Excellent find!

Also, if trailers are empty, but trailers header was provided (bytes 128, 0, 0, 0, 0), which is not restricted by grpc-web protocol error is throwing:
RpcError: Cannot read property 'trim' of undefined

This happens because of splitting empty string by \r\n returns array with empty string, and this should be checked before trimming:
for (let chunk of String.fromCharCode.apply(String, trailerData as unknown as number[]).trim().split("\r\n")) {
  if (chunk == "")
    continue

Since @wielski was kind enough to put up a PR to fix the issue I'm going to work on adding tests for this.

Thanks for the PR, @wielski.
@jcready, in case you come around to it, test coverage would be very welcome.

😅 I was working on it this weekend. I don't quite have a PR ready for you yet, but here are a few things I've noticed already:

Adding "sourceMap": true to tsconfig.test.json fixes the coverage report which made finding the areas missing coverage much easier.
This first check seems unnecessary since the same check was already performed a few lines up. Also, at least in the case of parsing headers out of a normal Response, any duplicate entries (keys) will have their values .join(', ')-ed automatically so checking for any array is only necessary for parsing the trailers. https://github.com/timostamm/protobuf-ts/blob/12098410d02c9af3c8f9cf5e477cc4dd576ffbfa/packages/grpcweb-transport/src/grpc-web-format.ts#L275-L277
As of yesterday I had the overall coverage of the grpcweb-transport package at:90.57% Statements 269/297
75% Branches 147/196
88.1% Functions 37/42
90.31% Lines 261/289

Though obviously coverage doesn't equate to meaningful tests so I've been trying to use the same tests that the official grpc-web client uses. For example this test looks like:it('handles multiple partial messages', async function () {
    const { frames, next } = getFrameRef();
    await next([0, 0, 0, 0, 2, 38]);
    assertFrames(frames, []);
    await next([39, 128, 0, 0]);
    assertFrames(frames, [
        [D, [38, 39]],
    ]);
    await next([0, 3, 40]);
    assertFrames(frames, [
        [D, [38, 39]],
    ]);
    await next([41, 42], true);
    assertFrames(frames, [
        [D, [38, 39]],
        [T, [40, 41, 42]],
    ]);
});

While writing tests for this I noticed something that didn't meet my personal expectations (even though the official grpc-web client does the same thing): For a unary call the response promise doesn't get resolved until the entire stream has finished.

Waiting for the trailer of unary responses is actually intentional. I want to prevent users from awaiting only the message, and have them ignore an error condition. For example:
const call = client.foo(...);
return await call.message;

gRPC servers are allowed to send a valid message and finish with an error status in the trailer. gRPC-web proxies like envoy will pass it through as is.
While this is a very nice feature for streaming responses, it makes little sense for unary methods. If you look at other implementations, you'll notice that they behave in the same vein (even gRPC clients, so this is not just a web thing). I was quite irritated by this situation as well.

It's especially surprising since it seems the condition where it throws unary call received 2nd message doesn't seem like it would be possible to occur.

Yes, this condition should never occur with a compliant server. Changing a rpc from unary to server streaming, and accessing the new server implementation with an outdated client could result in this condition, which is why this safe guard is in there.

Waiting for the trailer of unary responses is actually intentional. I want to prevent users from awaiting only the message, and have them ignore an error condition

I wonder then if we should avoid resolving defMessage until after the status code check here: https://github.com/timostamm/protobuf-ts/blob/12098410d02c9af3c8f9cf5e477cc4dd576ffbfa/packages/grpcweb-transport/src/grpc-web-transport.ts#L247-L249

Ha, so it doesn't even do as I was arguing?
From the manual:

Response trailers are a very useful feature, but for simple unary calls, awaiting two promises is cumbersome. For this reason, the UnaryCall itself is awaitable, and will reject if an error status is received.

Apparently not, sorry about the confusion. gRPC clients for C# do behave like I argued, and @grpc/grpc-js does so as well, but it looks like I just pondered changing our implementation, and didn't go forward with it. Changing the response promise to reject would definitely be a breaking change, so let's hold on moving this.
FYI: Going through your PR at the moment (huge thanks in advance).


Add unit tests for renderables

Codecov Report

Merging #11 (2229138) into main (49211f0) will increase coverage by 10.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main      #11       +/-   ##
===========================================
+ Coverage   33.02%   43.11%   +10.09%
===========================================
  Files           5        5
  Lines         218      218
  Branches       46       47        +1
===========================================
+ Hits           72       94       +22
+ Misses        146      124       -22

Impacted Files
Coverage Δ

src/humble_explorer/renderables.py
61.29% <0.00%> (+17.74%)
:arrow_up:

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


ERROR    Uncaught RangeError: Maximum call stack size exceeded

lodash.js:4114 Uncaught RangeError: Maximum call stack size exceeded

Sorry,  My bad,
I was passing NaN for x, y, width and height. :P


Bug: Handle PHP_INT_MAX overflow

Sorry for all the tickets I'm creating, but I am loving the package so far and want to help improve it :)
This bug has a pretty low chance of surfacing in actual use but the fix is simple enough so I thought I'd mention it.
Description:
Supplying a value larger than a signed 64bit integer to the create method causes a type error.
Steps to reproduce:
$generator = HumanoIDs::spaceIdGenerator();
$id = $generator->create(PHP_INT_MAX + 1);

Which throws the following error:
RobThree\HumanoID\HumanoID::create(): Argument #1 ($id) must be of type int, float given,

Expected output:
Since https://github.com/RobThree/HumanoID/blob/master/src/HumanoID.php#L116 checks for negative values and throws an InvalidArgumentException, something similar would be expected.
However, since the error happens before the check even happens I'm not sure how it would be implemented without changing the argument type. And it seems counter productive to allow floats just to throw an exception that floats are not allowed.

The PHP_INT_MAX is thrown by PHP itself since you're trying to fit a square peg in a round hole (i.e. a float into an int). This is what typing and compile-time-checking is for. This is, IMHO, normal and expected behavior and I don't see the need (nor a sane way) to "fix" this. This is gonna be a won'tfix I'm afraid unless someone more knowledgeable in PHP has a better suggestion.

Yeah, I kinda realized that myself while writing the issue so assumed this would not be changed so no hard feelings here. Just thought I could let you know while I had already written it up :)  I love the square peg in a round hole analogy by the way!


Implement deleteNode() function for a Singly Linked List

Taking reference from issue #12, implement the deleteNode() function for a Singly Linked List.
First step
Search, read, and understand information about lists, linked lists, and singly-linked lists, and how they are implemented.
Second step
Open a pull request with the data structure implementation, and wait for comments from @code-sherpas/kollections-maintainers.
Minimum acceptance criteria

The implementation must be well documented with proper Javadoc at Class and function levels.
The implementation must be tested with automatic tests that cover all relevant cases and situations.
The implementation must be generic.
The implementation must have the expected time complexity according to Computer Science. Here you can find a very helpful cheat sheet.

Related resources:

https://www.youtube.com/watch?v=v4cd1O4zkGw&ab_channel=HackerRank
https://www.youtube.com/watch?v=njTh_OwMljA&ab_channel=HackerRank
https://en.wikipedia.org/wiki/List_(abstract_data_type)
https://en.wikipedia.org/wiki/Linked_list#Singly_linked_list
https://www.bigocheatsheet.com/

Hey @ashikka  could i work on this?

Hey @ashikka could i work on this?

Sure thing!

Hey there, is this function for deleting a node at the end, beginning or anywhere?

Hey there, is this function for deleting a node at the end, beginning or anywhere?

My bad, I didn't specify that. The function deleteNode(position: Integer ) takes a position parameter and delete a node at the specified position. Thank you for pointing that out.

Hey i made a PR regarding this. #51


Nothing about Local Scale Module

Could you please advise why there is no effective information about the Local Scale Module in your open-source code?

Hi, LSM (Local Scale Module) is in yolc_head.py. Please refer to https://github.com/dawn-ech/YOLC/blob/3c498c8bfab284db66caa82607115580f5b4f2ee/models/dense_heads/yolc_head.py#L616.


View build log
TravisBuddy Request Identifier: bc1576a0-e3c0-11e9-b752-11f331f20c85

100th closed PR 🎉

This project grew far from I expected.

I've added #hacktoberfest to issues and it already proven advantages ✨


Response.data and Response.text

Response.data now returns the value of Content-Length (as number of byte)
Response.text take the stage instead.
Nothing goes wrong here, and I don't know whether it's an issue or not. Just to make sure you acknowledge the change.
Thanks

Ok, thanks. I'm pretty certain everything is working properly in version 3.0.1.


Support generic container registry authentication

Currently this action only supports authentication with an Azure Container Registry thats in the same subscription as the container app.
I'd like to be able to specify an imageToDeploy path and the credentials that are required to access the image. It could be an Azure Container Registry, a GitHub registry or a private registry that I'm hosting on an endpoint.
In my specific case, it is indeed an Azure Container Registry. However it is not in my subscription, and the subscription the ACR is homed in isn't even in the same tenant.
ERROR: The resource with name '***' and type 'Microsoft.ContainerRegistry/registries' could not be found in subscription 'Visual Studio Enterprise Subscription (***)'.

I'm providing these parameters and would expect it to work.
    - name: Build and deploy Container App
      uses: azure/container-apps-deploy-action@v0
      with:
        resourceGroup: containerapp
        imageToDeploy: mycontainerregistry.azurecr.io/azuredocs/azure-vote-7b3cb867c04385f583d1525cdc95912a2f54e52b
        acrName: mycontainerregistry
        acrUsername: ${{ secrets.REGISTRY_USERNAME }}
        acrPassword: ${{ secrets.REGISTRY_PASSWORD }}

@Gordonby Hey Gordon, apologies for the delayed response -- we added support for registryUrl, registryUsername and registryPassword arguments that can be provided for non-ACR scenarios when pushing/pulling images. When you have the opportunity, would you mind trying this out and letting me know if it's sufficient for your scenario when using the GitHub Action? Thanks in advance!

@Gordonby I'm going to close this issue as the registry* arguments added to this GitHub Action should support the desired use case, but please feel free to let me know otherwise and I'll gladly re-open the issue for further discussion. Thanks!


Draft: Basic luckyOrange library

Description
LuckyOrange Events
Related Ticket
Resolves #1086
Changes
NA
Checklist

[ ] I have run npm run build-all without errors
[ ] I have formatted JS and TS files with npm run format-all

@jabby @Atmosfearful I added base code for LuckyOrange custom tracking.
Calls to LuckyOrange library seem to be alright but no requests are make to the Luckyorange server. Plus their tag is included by GTM so I am a bit blind.
Question is: is luckyOrange configured so no calls are made when in developer mode?

@jabby @Atmosfearful I added base code for LuckyOrange custom tracking. Calls to LuckyOrange library seem to be alright but no requests are make to the Luckyorange server. Plus their tag is included by GTM so I am a bit blind. Question is: is luckyOrange configured so no calls are made when in developer mode?

@sy-zygy do you know the answer to this question?

hello @biwano I unsuscribed as I think I'm not the right person here. 😄

@biwano I'm jabby09
Can we release 1 or 2 of the events in production and then we can test that the events are firing?

@biwano I'm jabby09
Can we release 1 or 2 of the events in production and then we can test that the events are firing?

Sure. This PR should be now OK for this purpose.


Update gpu performance benchmark script

This PR updates the performance benchmark script to run the gpu example for 10 steps. Note that it runs for 20 steps total (the first call is to compile). cc @bloops.

Can we have it print the walltime it took when running the final benchmark step? Thanks!

Done, @bloops!

bors r+


Gizmo lags on mouse press

I have implemented the transform gizmo following the example. It works great, but I've noticed a minor annoying issue that I was not able to backtrace myself. When I click on the gizmo to start, to say, a drag operation, there is some sort of delay between when it starts moving and coords appear on the screen. If I keep holding the mouse and moving, the object and gizmo moves without any delay.

Can you repro this issue with the sample app? I tried it myself and could not repro. I get the delta coord on screen as soon as my mouse button is down.

Can you repro this issue with the sample app? I tried it myself and could not repro. I get the delta coord on screen as soon as my mouse button is down.

I sadly can't. The issue might be coming from the way Dear ImGui is installed for me. I am using inside QT's QOpenGL widget. I will investigate further if you confirm it does not happen on clean setup.

It is weird though, since hover on GIZMO reacts instantly. And it never looses any focus and works properly. The only issue is that it does not change its position right away and the coords text appears a little delayed. As soon as the coord text has appeared, it works smoothly.

the only thing I can think of is some code is running when a click is happening. Maybe the event is trapped by imgui (I doubt) or Qt (more probable to me)  and it takes time to complete.
Maybe you can try to profile your app and guess what's happening with the profile.

After debugging it appeared that is a Mac OS X issue in the end. The mouse press event for some reason is sent with delay to QT application. Works fine on Windows, and I am yet to test on Linux. So, not related to ImGuizmo. Can be closed, I guess.

Sounds good! Good luck with your app on macos :)


Add Entity Culling mod

Add Entity Culling v1.6.1. This is a small client-side mod which culls entities that are not in a direct line of sight. This may require some configuration tweaking for blocks introduced by other mods in the future.

Aside from the CI being unable to export the Modrinth modpack with this, it's not clear what our redistributive rights are with this modpack. We'll forego this one.


chore: emergency version upgrade to v1.3 Protocol Version 2.

Issue being fixed or feature implemented
This PR provides an emergency update to version 1.3 of Platform and Protocol V2.
What was done?
Created a way to change consensus parameters as an emergency update. This will fix the issue in Tenderdash as soon as we hit Epoch 3.
Contested usernames will go live on Epoch 4 instead of Epoch 3.
Evonodes that have not upgraded by Epoch 3 will stall.
How Has This Been Tested?
We will be testing this on testnet.
Breaking Changes
This is a breaking change.
Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added or updated relevant unit/integration/functional/e2e tests
[X] I have added "!" to the title and described breaking changes in the corresponding section if my code contains any
[X] I have made corresponding changes to the documentation if needed

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Before this is merged in we need to update the epoch on Testnet to test the immediate consensus params update.


Basic Export needs dotnetcore dependency checked

I am brand new to .NET, so maybe this is a silly thing to post. I downloaded your BasicExport example and loaded it in Visual Studio to experiment with it. I could get the VB.net/C# .NET framework projects to display in the debug console properly, but the DotNotCore would not, erroring with "specified module could not be found".
After tinkering I saw that the DotNotCoreLibrary was not checked under dependencies. Checking that allowed it to debug properly. I don't know if I did something to mess that up, but figured I'd post it here just in case.

Thanks for the information!
Looks like I forgot to add this dependency for a new added .NET Core project sample.
@Zinoh, Just open PR and I'll approve it.


eRFC: single-file packages ("cargo script") integration

This experimental RFC adds unstable support for single-file
packages in cargo so we can explore the design and resolve
questions with an implementation to collect feedback on.
Single-file packages are .rs files with an embedded
manifest.  These will be accepted with just like Cargo.toml files with
--manifest-path.  cargo will be modified to accept cargo <file>.rs as a
shortcut to cargo run --manifest-path <file>.rs.  This allows placing
cargo in a #! line for directly running these files.
Example:
#!/usr/bin/env cargo

//! ```cargo
//! [dependencies]
//! clap = { version = "4.2", features = ["derive"] }
//! ```

use clap::Parser;

#[derive(Parser, Debug)]
#[clap(version)]
struct Args {
    #[clap(short, long, help = "Path to config")]
    config: Option<std::path::PathBuf>,
}

fn main() {
    let args = Args::parse();
    println!("{:?}", args);
}

$ ./prog --config file.toml
Args { config: Some("file.toml") }

See cargo-script-mvs for a demo.
Rendered

Not even sure if eRFC's are still a thing but when we talked about this in the cargo team meeting, we felt this format would be better for closing out conversations on all of the various details rather than trying to track and hash them out in an upfront design.

I think it's important to have cargo add support for .rs files, otherwise adding dependencies is not very ergonomic (having to remember the syntax and lookup the version numbers)

Of note is that rustc already ignores a shebang at the beginning of a file despite it not being valid Rust code - I hadn't realised that.

As this is just agreement to do an experiment, interest seems high, and conversation has died down, I feel like its time:
@rfcbot merge

Huzzah! The @rust-lang/cargo team has decided to accept this RFC.
To track further discussion, subscribe to the tracking issue here:
rust-lang/cargo#12207

Just migrated from rust-script and it works like a charm. https://git.vtimothy.com/vtimofei/dotfiles/commit/5eaf73a001f86cda73e5332937a6c6187a0f47a5

I tried to use a library with the below structure:
#!/usr/bin/env -S cargo +nightly -Zscript

//! ```cargo
//! [dependencies]
//! hello_lib = { path = "./hello_lib" }
//! ```

use hello_lib::greet;

fn main() {

    println!("{}", greet());
}

$ tree
.
├── hello_lib
│   ├── Cargo.lock
│   ├── Cargo.toml
│   ├── src
│   │   └── lib.rs
└── script.rs

$ cat hello_lib/src/lib.rs
pub fn greet() -> String {
    "Howdy".into()
}

I think using libraries this way is a viable option, and support for [lib] wouldn't be required.

This RFC will appear in the Call for Testing section of the next issue (#501) of This Week in Rust (TWiR).
I have removed the call-for-testing label.  Please feel free to re-add the call-for-testing label if you would like this RFC to appear again in another issue of TWiR.

@victor-timofei

I think using libraries this way is a viable option, and support for [lib] wouldn't be required.

The idea for single-file lib packages is to reduce the boilerplate for writing [lib] packages, especially in workspaces.


Create reset/try again button

Create reset/try again button

Mistakenly identified pull request 93 as resolving this ticket, but it was actually pull req 113 that resolves this issue.

Go back and Try Again button added in pull req 113 as part of the improvements made to modal functionality.


pcap.Net

Anony123[CodePlex] Hi,
Can someone please tell me how to capture data packets over a WiFi network?
wireless interface?
Is there some inbuilt pcap.Net functionality that helps us do so?
Thanks!

Brickner[CodePlex] Questions regarding to how to use Pcap.Net should be in the discussions.
Issues should be opened for concrete bugs or feature requests.


https://bugs.launchpad.net/jio/+bug/1467502

When file size is an exact multiple of chunk_size, glance client
is processing EOF in image-data as a chunk and sends to
glance-api. The server treats this as the end of chunked
transmission and sends a http response.
When the actual last chunk is sent by the 'requests' library, the
server sends a 400 response and tracebacks with broken pipe
as the client has already closed the socket.
Closes-Bug: #1342080
Change-Id: Icdbff838450db1c252ddc919a230a7d3ca16765f
fixes: https://bugs.launchpad.net/jio/+bug/1467502

The required functionality works in the build
http://jiocloud.rustedhalo.com:8080/job/puppet_rjil_pkg_test/61
successfully tested. Recommend merging.

green build with this code change, merging.


raftstore: the invalid read delegates in local reader may not be removed forever

Bug Report
Although https://github.com/tikv/tikv/pull/4765 introduce the invalid flag to remove the invalid ReadDelegates on LocalReader, but the flag only be checked when handling a request to the ReadDelegate, if the ReadDelegate is marked as invalid and no incoming request to it, the ReadDelegate not be removed and consuming memory.
Although mem::size_of::<ReadDelegate>() is 122 bytes, but if this case happened the ReadDelegate will not be removed forever, and the memory occupied by these invalid ReadDelegate will only increase not decrease as TiKV running.

PTAL @hicqu @BusyJay

PTAL @hicqu @BusyJay

Perhaps it can also use LruCache instead of HashMap to maintain a reasonable size.

Perhaps it can also use LruCache instead of HashMap to maintain a reasonable size.

Perhaps it can also use LruCache instead of HashMap to maintain a reasonable size.

LruCache only remove one item each time when its capacity is full.  As TiKV run long enough, the capacity we set to LruCache will full and after that it may bring extract cost when handling request for a delegate not in the cache.
I think it may better to keep an unlimited HashMap and when we need insert new delegate to LocalReader, check delegates.len() > store_meta.readers.len() * 1.5 then remove all invalid delegates if it is true.

Perhaps it can also use LruCache instead of HashMap to maintain a reasonable size.

LruCache only remove one item each time when its capacity is full.  As TiKV run long enough, the capacity we set to LruCache will full and after that it may bring extract cost when handling request for a delegate not in the cache.
I think it may better to keep an unlimited HashMap and when we need insert new delegate to LocalReader, check delegates.len() > store_meta.readers.len() * 1.5 then remove all invalid delegates if it is true.

What's the extract cost comparing to a HashMap and a for loop?

What's the extract cost comparing to a HashMap and a for loop?

A LruCache's capacity is limited, after it is full, every insert comes with a remove. With the approach mentioned above, we only need to clear invalid delegate when we know there has a large amount of invalid delegate, it should not be frequent and will be efficient, also prevent the HashMap grow unlimitedly.

A LruCache's capacity is limited, after it is full, every insert comes with a remove. With the approach mentioned above, we only need to clear invalid delegate when we know there has a large amount of invalid delegate, it should not be frequent and will be efficient, also prevent the HashMap grow unlimitedly.

The usage in TiKV usually maintains LruCache's capacity the same as region count. A remove is an O(1) operation that only takes several nanoseconds. Removal occurs only when capacity is full and new delegate is inserted. However the time of looping over a HashMap is undetermined. If the count of hashmap exceeds 100000, the time probably can be tens or hundreds of millis.

The usage in TiKV usually maintains LruCache's capacity the same as region count. A remove is an O(1) operation that only takes several nanoseconds. Removal occurs only when capacity is full and new delegate is inserted. However the time of looping over a HashMap is undetermined. If the count of hashmap exceeds 100000, the time probably can be tens or hundreds of millis.


How to use command release spells？

How to use command release spells？

I’m sorry, I’m not sure what you mean. Could you please elaborate?

I want use command release spells
Don't use wands

You can use the /cast command, but it's not really meant for players.
I suggest you take a look at another plugin- MagicSpells, Heroes and SkillAPI are all good alternatives. This plugin really revolves around the wand items, and is not meant for players to use commands.

OK Thank you!


chore: use Linux container CPU quota

Go is not cgroup aware and by default will set GOMAXPROCS to the number of available threads, regardless of whether it is within the allocated quota. This behaviour causes high amount of CPU throttling and degraded application performance.
Fixes: #875

WDYT about using https://github.com/VictoriaMetrics/VictoriaMetrics/blob/master/lib/cgroup/cpu.go ?

Thank you for your time and PR.
Closing it with comment https://github.com/VictoriaMetrics/operator/issues/875#issuecomment-2272229935


[Public Catalog] Remove code / automation for manually fetching quickstarts

Description
Once we have fully swapped the landing page and the detail page over to being powered by Gatsby SSR functions, we no longer need to have a Github workflow that runs every few hours to make API requests and build up a JSON file.
This ticket is to clean up that functionality and the files we are no longer using.
Acceptance Criteria

[x] https://github.com/newrelic/developer-website/issues/1853 (landing page via SSR)
[ ] https://github.com/newrelic/developer-website/issues/1855 (search and filter via SSR)
[x] https://github.com/newrelic/developer-website/issues/1854 (details page via SSR)
[x] Remove the Github workflow
[x] Remove the fetch quickstarts script (and related tests)
[x] Remove the quickstarts JSON file
[x] Remove any code that references the quickstart JSON file (example)
[x] Edit the way we give related resources the quickstart slugs (we'll need to make an api call since we don't have quickstarts in the gatsby schema anymore)
[ ] Remove any secrets we no longer need (New Relic API key?)
[ ] Notify Virtuoso & Developer Experience teams that we are no longer hitting the Open Install Library service

PR #1968 should be the last code change cleanup wise.
Im not sure about the acceptance criteria that links to another ticket 🙈
The last two acceptance criteria should be blocked by the feature/gatsby-ssr being merged. We can't affect the current workflows (removing the api key) until we are ready to swap to using the api in site.

blocked by this lack of functionality of Gatsby Cloud not supporting private environment variables.


GR-PEACH does not implement SystemCoreClock

GR-PEACH (RZ_A1H target) does not implement SystemCoreClock variable which defined as CMSIS spec here: https://www.keil.com/pack/doc/CMSIS/Core/html/group__system__init__gr.html
If a program uses the SystemCoreClock variable, it obviously generate link error.

cc @TomoYamanaka

Hi
We implemented "SystemCoreClock" variable which defined as CMSIS spec.
PR is here: https://github.com/mbedmicro/mbed/pull/1848

ARM Internal Ref: IOTMORF-177

#1848 had been merged.


Fixes MCC during graph execution.

Description
Brief Description of the PR:
Fixes # (issue)
Type of change

[X] Bug fix
[ ] New Tutorial
[ ] Updated or additional documentation
[x] Additional Testing
[ ] New Activation and the changes conform to the activation contribution guidelines
[ ] New Callback and the changes conform to the callback contribution guidelines
[ ] New Image addition and the changes conform to the image op contribution guidelines
[ ] New Layer and the changes conform to the layer contribution guidelines
[ ] New Loss and the changes conform to the loss contribution guidelines
[ ] New Metric and the changes conform to the metric contribution guidelines
[ ] New Optimizer and the changes conform to the optimizer contribution guidelines
[ ] New RNN Cell and the changes conform to the rnn contribution guidelines
[ ] New Seq2seq addition and the changes conform to the seq2seq contribution guidelines
[ ] New Text addition and the changes conform to the text op contribution guidelines

Checklist:

[X] I've properly formatted my code according to the guidelines

[X] By running Black + Flake8
[X] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
If you're adding a bugfix or new feature please describe the tests that you ran to verify your changes:
*
As per the documentation, the values in batch_set_value should be a numpy array. Passing a tf.Tensor throws the following exception when not running eagerly:
TypeError: __array__() takes 1 positional argument but 2 were given
This can easily be reproduced by either disabling eager exeuction, or by subclassing MCC and decorating reset_states with tf.function like so:
class MyMCC(tfa.metrics.MatthewsCorrelationCoefficient):

    @tf.function
    def reset_states(self):
        """Resets all of the metric state variables."""
        reset_value = tf.zeros((self.num_classes,), dtype=self.dtype)
        tf.keras.backend.batch_set_value(((v, reset_value) for v in self.variables))

mcc = MyMCC(num_classes=1)
mcc.reset_states() # Throws TypeError

@autoih @marload
You are owners of some files modified in this pull request.
Would you kindly review the changes whenever you have the time to?
Thank you very much.

Added a test. It's not the prettiest, but while there's fixtures for running functions eagerly or not, there does not seem to be a fixture for eager/non-eager Tensors.

Sorry for the delay. Thanks for the fix :-)


chore(otel): upgrade otel to v1.28.0

The Float64Gauge method was added to otel/metric in v1.27.0, so we need to update our dependency to use it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


bump dependencies' versions

cc @susurrus

This compiles for me with the exception of missing the panic_fmt lang item when building with xargo. If I just run xargo build --example hello after cloning this branch I get a "can't find crate for std" error. If I follow the quickstart docs and delete build.rs and memory.x I get the same error. So I'm not sure what I'm doing wrong, but I can't fully test this right now.

@Susurrus You need to specify the --target otherwise you are building for the host.

@japaric Thanks, that was it. I had thought that was already specified in .cargo/config, but it looks like it's not. Since this is a board-specific crate, maybe the [build] target should be specified there as well?

Since this is a board-specific crate, maybe the [build] target should be specified there as well?

Unfortunately if we do that then we can't publish this crate because cargo publish / cargo package would fail.

Yeah, I figured there was a reason it couldn't be done.
Anyways, this appears to work here. This branch doesn't seem to work in my small example project with RTFM, however, which what made me post about this in the first place. I was trying to follow your blog post for the 0.2 release and use the f3 crate and its stm32f30x device crate instead of the stm32f103xx crate so I could work with my F3DISCOVERY board. but I seem to be missing something. I still get a missing panic_fmt error. My dependencies are cortex-m-rtfm, cortex-m-semihosting, and f3, but I didn't specify any features but that shouldn't be necessary by my understanding.

@Susurrus Did you start from quickstart v2? It enables the "abort-on-panic" feature of the cortex-m-rt crate and that provides the panic_fmt lang item.
If you already have that then maybe have CARGO_INCREMENTAL enabled? That doesn't work well with no-std applications, IME, and can produce bizarre errors. If you have that enabled then disable it and call cargo clean because you may be using some stale cache.

I'm having linking problems now, but they seem more a factor , but I think I'm past the whole Rust compilation process and it's just cross-compilation issues on my end. So I think this is fine to merge.

:pushpin: Commit 1d27248 has been approved by japaric

@homunkulus r+

:hourglass: Testing commit 1d27248 with merge 1d27248...

:broken_heart: Test failed - status-travis

@homunkulus r+

:pushpin: Commit ffbfe25 has been approved by japaric

:hourglass: Testing commit ffbfe25 with merge ffbfe25...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing ffbfe25b0f8ac80ed0b794656968d23af500205b to master...


python3Packages.aioesphomeapi: 10.14.0 -> 11.0.0

https://github.com/esphome/aioesphomeapi/releases/tag/v11.0.0
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 193703 run on x86_64-linux 1

4 packages built:

esphome
home-assistant-component-tests.esphome
python310Packages.aioesphomeapi
python39Packages.aioesphomeapi


Not related to cookiecutter-homeassistant-component

Hello, I could not find any other way of sending you a message so I will try here, is this tutorial still useable with all the changes from the last time the tutorial got updated? It says August 2, 2019
https://aarongodfrey.dev/home automation/creating-a-3d-floorplan-in-home-assistant/

Yes, that tutorial still works for the current version, 2021.8.8.  If you have other questions you can reach out on the community thread: https://community.home-assistant.io/t/3d-floorplan-using-lovelace-picture-elements-card/123357


Do not force specific Ruby versions

Remove .ruby-version files which were forcing developers
to use a particular version of Ruby.

2.2.3 was out-of-date and is not the latest stable 2.2.* version
This forced users with rbenv/rvm to install an older version of Ruby

@frankyn PTAL

+1 from the runtime team.

Fixes #66


[FEATURE] Improvement in Footer section . [GSSOC 2023]

Detailed description
Footer looks little bit off in comparative to the other parts  and even icons are not aligned properly,
Want to make some changes keeping the whole website on mind.
If changes req. do assign it to me under GSSOC 2023 .  :|)
Context
Improvement in design
Possible implementation
No response
Additional information
No response
Code of Conduct

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

@ManishaSwain8 can you assign this to me under gssoc23

@ManishaSwain8 sure you can work on this. I am assigning it to you.
Please make sure the footer looks like this.
Figma link -> https://www.figma.com/file/3DBLz1I6bI651lizseSQ0d/WebX-DAO---Website?type=design&t=Lg0Yk7hnG4tVBDnW-0

Got it   !!!

Ok

@Mustafiz04 Just example below, is background ok or should keep it as it is !

@Mustafiz04 Just example below, is background ok or should keep it as it is !

It is okay.


let's keep the data in YAML format and render into Markdown on every push

Maybe we can keep the entire dataset in a YAML file. Then, on each push we configure a GHA job, that will render this YAML file into a Markdown table, put it into the README.md (replacing the existing one), and then submit a pull request with the changes to the repository.
See how it works, for example, here: https://github.com/yegor256/micromap/blob/master/.github/workflows/up.yml

@uchitsa think about this, please

@yegor256 thanks, I'll do it

@uchitsa will be great, if we create a Bash script that does all that: compile.sh. It gets a path to .yml file as an input and prints a Markdown table as an output. We will call this script from GHA job, saving its output to a Bash variable. Then, inject it into the README.md

@uchitsa would be great to create a unit test (with a synthetic YAML input) for this Bash script, in order to make sure that it actually prints the right Markdown output.

@uchitsa I think, this is the most important task now, since our list keeps growing and it will be just more time consuming later to turn it into YAML

@yegor256 I prepare python script to parse a yaml source and create a result md file

@uchitsa don't forget to add pylint or pep8 (or both) to GHA

@uchitsa thanks a lot for implementing this feature!


IDToken Encryption is not supported for JWKS urls

An internal server error is thrown in oidc auth flow when a JWKS url is used when IDTokenEncryption is enabled.

moved to https://github.com/wso2-extensions/identity-inbound-auth-oauth/issues/1253


Seccomp Support

This is almost done, but file an issue here to track the progress.

Close since #219 is merged.