Add quickstart docs for dataset workflow

Linked issue(s):
KOL-4073
What change does this PR introduce and why?
Adds quickstart docs for the new dataset workflow
Please check if the PR fulfills these requirements

[x] Include reference to internal ticket and/or GitHub issue "Fixes #NNNN" (if applicable)
[ ] Relevant tests for the changes have been added
[x] Relevant docs have been added / updated

In particular, I would love some thoughts on

General structure
Suggested plotting, test cases, and evaluation metrics for keypoint detection

Will fill in the blank gifs/demo path details after receiving feedback.

In particular, I would love some thoughts on

General structure
Suggested plotting, test cases, and evaluation metrics for keypoint detection

After opening, remembered that there's another GIF of defining evaluation metrics (step 4) I haven't uploaded yet. Seems like you can't use model fields just yet. Ping me when it's in a good state and I can amend accordingly.

Oh one more thing -- flagging that checking out this branch is really slow for me. I'm not sure if there's something special to do with Git LFS
(unclear if that's tracking all of the new GIFs properly):
    ~/kolena/kolena/docs/assets/images  on   mb/quickstart-docs-v1 *2                                                                                  at 11:18:40 
❯ git grep lfs
Binary file metrics-iou-seg-mask-results.png matches
Binary file metrics-iou-seg-mask.png matches
Binary file quickstart-upload-dataset-workflow.gif matches // where are the other gifs?

It seems like LFS is tracking the .gifs - is there some subtlety to how the files are tracked that would make this slow?
From docs/assets/images:
> git lfs ls-files | grep ".gif"
2cb630713f - docs/assets/images/quickstart-compare-models.gif
ad5807d43b - docs/assets/images/quickstart-create-test-cases.gif
4b9c7a7f6a - docs/assets/images/quickstart-otf-plotting.gif
701d2133bf - docs/assets/images/quickstart-upload-dataset-workflow.gif
674ba99b90 - docs/assets/images/quickstart-upload-results-workflow.gif

Bumped up the git lfs settings for the org, hopefully that tracks the gifs better.


addding dotnet 7

Codecov Report
Base: 68.38% // Head: 68.38% // No change to project coverage :thumbsup:

Coverage data is based on head (6424524) compared to base (bb89ea8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   68.38%   68.38%
=======================================
  Files          67       67
  Lines        4659     4659
  Branches      446      446
=======================================
  Hits         3186     3186
  Misses       1278     1278
  Partials      195      195

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

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


doesn't work with google apps scripts

I'm using a deployed web app created with Google apps scripts (GAS). It has a model of putting the script code in an IFrame which Batarang seems not to handle. The panel in the developer tools is empty

There's a known race condition where the panel is empty regardless of app state.  Can you try a few more times & see if you can get the devtools panel to appear?

I could never get the dev tools panel to appear
On Sun, Jul 5, 2015 at 10:59 PM Randall Koutnik<EMAIL_ADDRESS>wrote:

There's a known race condition where the panel is empty regardless of app
state. Can you try a few more times & see if you can get the devtools panel
to appear?
—
Reply to this email directly or view it on GitHub
https://github.com/angular/angularjs-batarang/issues/238#issuecomment-118661805
.

Ah, re-reading this, the problem lies in the iframe.  Dupe of #14


Upgarde Quarkus to v1.3.0.Final

I would say this is already completed. Any objection to closing this ?

Yes, it's completed.


Alternative Design Ideas

It has been noted that other countries use different coaster sizes. There is a not a lot of flexibility in the current design, especially going to a larger diameter coaster.
The firmware can handle any size and probably accommodate other changes. If you are looking at a redesign, here are some thoughts.

The board does not need to be part of the design. A simpler existing board could be used remotely or packaged within the base. I sell some on Tindie.

The geared rotational bed is complex to model. Someone suggested a direct drive. That is probably doable without affecting quality. I would probably suggest a metal hub, rather that direct mounting of the 3D printed part. It is more likely to be concentric and could buffer the heat from the motor.

I actually thought about a parameterized OpenSCAD model. Maybe also the gears can keep their size and an extra plate on top for the coaster lead to a constant gear transmission ratio. Then the board can also work for machines with different diameter.

@fastriker You would need to increase the travel of the linear axis too. Maybe just move it out and increase the rod length.

I am actually currently trying to solve this. I increased the height of the "big gear" and added inserts for magnets. Further I modeled a platform which can hold 107mm coasters (pretty standard in Germany) which also has inserts for 3 magnets. This would allow for swappable platforms. The height of the "arm tower" is also increased by 10mm, but I probably need to increase it more.
I am currently waiting for the PCBs and other parts to try a real world assembly. If this works as expected I will publish the changes.

@fastriker You would need to increase the travel of the linear axis too. Maybe just move it out and increase the rod length.

You are right - I didn't thought about that. But maybe it is also not always necessary to use the full space of the coaster.

I am actually currently trying to solve this. I increased the height of the "big gear" and added inserts for magnets. Further I modeled a platform which can hold 107mm coasters (pretty standard in Germany) which also has inserts for 3 magnets. This would allow for swappable platforms. The height of the "arm tower" is also increased by 10mm, but I probably need to increase it more.

That is also a nice quick solution - and also interesting for me.


Wireless Rpi3 - nxdomain error

Just updated to latest nerves_* releases and built a fresh firmware for my RPI3.
bootloader           0.1.2
nerves               0.7.5
nerves_firmware_ssh  0.3.0
nerves_network       0.3.4
nerves_runtime       0.5.0
nerves_system_rpi3   0.17.0

My WiFi connection is correctly established and a IP address is assigned via DHCP to the wlan0 interface. The connection works as expected but the DNS resolution is not working (nxdomain error).
The DNS lookup works pointing to google DNS:
:inet_res.lookup('nerves-project.org', :in, :a, [nameservers: [{{8,8,8,8}, 53}]])
but not with the system default one:
:inet_res.lookup('nerves-project.org', :in, :a)
Nerves.Network.status "wlan0":
%{domain: "fritz.box", ifname: "wlan0", index: 3,
 ipv4_address: "<IP_ADDRESS>", ipv4_broadcast: "<IP_ADDRESS>",
 ipv4_gateway: "<IP_ADDRESS>", ipv4_subnet_mask: "<IP_ADDRESS>",
 ...
nameservers: ["<IP_ADDRESS>"],
...
}

Checked with a clean nerves_examples/hello_network build.
I think the issue is related to nerves_system_rpi3 0.17.0.
With nerves_system_rpi3 0.16.1, the DNS resolution works as expected:
  bootloader 0.1.2
  distillery 1.5.2
  elixir_make 0.4.0
  nerves 0.7.5
  nerves_network 0.3.4
  nerves_network_interface 0.4.2
  nerves_runtime 0.5.0
  nerves_system_br 0.13.9
  nerves_system_rpi3 0.16.1
  nerves_toolchain_arm_unknown_linux_gnueabihf 0.11.0
  nerves_toolchain_ctng 1.1.0
  nerves_wpa_supplicant 0.3.2
  system_registry 0.6.0

After upgrading nerves_system_rpi3 to 0.17.0, it doesn't work:
  bootloader 0.1.2
  distillery 1.5.2
  elixir_make 0.4.0
  nerves 0.7.5
  nerves_network 0.3.4
  nerves_network_interface 0.4.2
  nerves_runtime 0.5.0
  nerves_system_br 0.14.0
  nerves_system_rpi3 0.17.0
  nerves_toolchain_arm_unknown_linux_gnueabihf 0.11.0
  nerves_toolchain_ctng 1.1.0
  nerves_wpa_supplicant 0.3.2
  system_registry 0.6.0

Thanks for reporting this @visciang. I am able to reproduce the problem and it appears that the symlink for /etc/resolv.conf -> /tmp/resolv.conf is missing. When updating to the latest buildroot, we started to provide our own skeleton and it seems that this might have been missed.
I am building a test system now to see if this fixes the issue and should have this patched and out soon.
:heart:
Thanks!

This should be all fixed in nerves_system_rpi3 v0.17.1 as well as on all other systems. Please update and give it a try. Thanks!


support repeat_penalty config

it seems works at the sample phase. which helps reduce the repeativeness problem.
this is a description about repeat_penalty in kimi chat:

The function of repeat_penalty is to adjust the model's sensitivity to tokens that have already been generated when creating new tokens. By increasing the value of repeat_penalty, the model is encouraged to avoid reusing recently generated words, thereby increasing the diversity and novelty of the text. Conversely, reducing the value of repeat_penalty lessens the penalty for repetition, which may result in more repetitive content in the generated text.

related links: https://www.reddit.com/r/LocalLLaMA/comments/15s7ln1/potential_fix_to_the_repetitiveness_problem_of/

this is a description in GPT4:

Here's a simplified explanation of how the repetition penalty works:

When the model generates text, it calculates the probability of each word or token in its vocabulary to be the next one in the sequence.
If the repetition penalty is greater than 1, the probabilities of tokens that have already appeared in the current output are divided by this penalty factor, effectively reducing their chances of being selected again.
The model then samples the next token from the adjusted distribution of probabilities, taking the repetition penalty into account.

import numpy as np

def apply_repetition_penalty(probabilities, token_sequence, penalty=1.2):
    """
    Apply repetition penalty to the probabilities of tokens.

    :param probabilities: numpy array of original probabilities for each token
    :param token_sequence: list of tokens already generated
    :param penalty: repetition penalty factor (default 1.2)
    :return: adjusted probabilities
    """
    # If penalty is set to 1, no penalty is applied
    if penalty == 1:
        return probabilities

    # Apply penalty to the probabilities of tokens that have already appeared
    for token in set(token_sequence):
        token_idx = token_to_index[token]  # Assuming a function that maps tokens to their index
        probabilities[token_idx] /= penalty

    # Re-normalize the probabilities
    probabilities /= sum(probabilities)

    return probabilities

# Example usage
original_probabilities = np.array([0.1, 0.2, 0.3, 0.4])  # Probabilities for 4 hypothetical tokens
generated_sequence = ['token2', 'token3']  # Tokens that have been generated so far
token_to_index = {'token1': 0, 'token2': 1, 'token3': 2, 'token4': 3}  # Mapping tokens to their index

# Apply repetition penalty
adjusted_probabilities = apply_repetition_penalty(original_probabilities, generated_sequence, penalty=1.2)

# Sample the next token using the adjusted probabilities
next_token_index = np.random.choice(range(len(adjusted_probabilities)), p=adjusted_probabilities)
next_token = index_to_token[next_token_index]  # Assuming a function that maps indices back to tokens

print(f"Next token: {next_token}")


overlapblock Creator - Web design #35

https://images01.nicepagecdn.com/page/42/52/html-template-42520.jpg Design a overlapblock like the above image
Create a file "overlapblock_NUM_by_(YourName).html" in the modules/overlapblock folder.
Use simple HTML and CSS and JAVASCRIPT to create the overlapblock animation. You can use the interface development https://kwickerhub.com/ (This project) or you can write the lines of code yourself.
Step by Step Guide.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "overlapblock_NUM_YOUR_NAME_.html" in the "modules/overlapblock" folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/overlapblock/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
'''

'''
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Good-luck.

Hey @NtemKenyor , I want to work o this issue.

Hey @NtemKenyor , I want to work on this issue.


Improve validation error message

Description:
Improve the way validation errors are presented to the user. This is a backport of some features in esphomeyaml.

Replace unreadable OrderedDict(...) offending item summaries with JSON encoded ones.
If multiple validation errors occur, print all of them.
Print the path to the validation error for all errors
Better warnings for vol.Required validation errors.

Will add tests once this idea is accepted.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

I like it !


URGENT: pin nf-validation version

Description of the bug
To prevent breaking this pipeline in the near future, the nf-validation version should be pinned to version 1.1.3 like:
plugins {
    id<EMAIL_ADDRESS>}

Command used and terminal output
No response
Relevant files
No response
System information
No response

Thank you @subwaystation!

This is already live in the latest release.


Check COSMIC

Check if it's necessary for MuTect2

It has been removed in GATK4.

cf https://github.com/broadinstitute/gatk/issues/3207


Getting Zerto Resource Report - issue with URI

Greetings, @ChristopherGLewis -
Enjoying the module.
As I was starting to leverage the Get-ZertoResoureReport and Get-ZertoResoureReportAdvFilter cmdlets, though, I came across what may be a bug:  the URI is not valid, at least in the Zerto v5.0 u2 version that I am using -- it seems to be missing an "s".
That is, these two cmdlets are using /ZvmService/ResourceReport/ in the URI, but per the Zerto API docs for the Resources Report API, the URI is /ZvmService/ResourcesReport/ (notice the "s" in "Resources").
I tested by updating that URI in the function definitions in a copy of the .psm1 file, and, success.
So, I'm opening this GHub issue to track this one.
Thanks for all the work on making this module what it is.  Cheers

Thanks - I should have a new module version published today or tomorrow - I'm looking at correcting some of the Zerto Version and deprecated API issues.

Superb!  Thanks much

Udpated to version 1.0.3 to fix the ResourcesReport - note that the commands have changed to Get-ZertoResouresReport and Get-ZertoResourcesReportAdvFilter to reflect the actual API call.


Roll Flutter Engine from a1d513f78bbb to 7155881f346b (1 revision)

https://github.com/flutter/engine/compare/a1d513f78bbb...7155881f346b
2023-08-06<EMAIL_ADDRESS>Roll Skia from 72264deb9f05 to 9c4cfcd16529 (1 revision) (flutter/engine#44427)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


status script works not correct

A subscription is createted ( with Status Tab) and an user is assigned
step1: click on My Subscription
step2: click Tab Status  -> works fine
step3: click Details and execute an Operation -> works fine
step4: click Tab Status
[observed]
the status script will not be executed
[expect]
the status script will  be executed again

works fine


Do i need to use  Access Control Cluster.xml in my PICS? (CON-1244)

Description
In my project i defined the root node in this way:

node::config_t, in my esp_matter version, is defined inside esp_matter_endpoint in this way:

endpoint::root_node::config_t root_node creates an endpoint with these clusters:

As you can see the cluster access_control is defined, do i have to implement the Access Control Cluster.xml in my PICS even if I don't use it?
The access control is defined only with the cluster _revision attribute:
So in the test, when it tryes to read the attribute AccessContorlEntryChanged that is obviously not implemented but is Mandatory in the PICS if the Access Control Cluster as Server(ACL.S) is defined.

If you are performing Access Control or Access Control Enforcement Cluster's test case then you need to use Access Control Cluster.xml

The access control is defined only with the cluster _revision attribute:

It is header file.
You will get the actual implementation of the Access Control Cluster in esp_matter_cluster.cpp file.

Ok, so if the Access Control Cluster is defined but I don't implement it, I don't need to do the test with the Access Control Cluster.xml right?

The Root Node is mandatory for all device types, and the Access Control Cluster is a mandatory cluster of the Root Node. Therefore, it must be supported.

Root Node: This endpoint is akin to a "read me first" endpoint that describes itself and the other endpoints that make up the node.

Do I have to do the test or not?
The test checks all the mandatory attributes, but the SDK doesn't implement all of them.

Di I need to implement all of them or can I skip the test?

Hi @FMistrorigo, It is a header file. You will find the actual implementation of the Access Control Cluster in the esp_matter_cluster.cpp file.
All mandatory clusters, attributes, features, events, and commands are implemented in esp-matter. If you create a device type, you don't need to implement these mandatory elements yourself.

@FMistrorigo Please close this if your questions are answered

Thanks, I found the implementation and wrote some code to see the attribute of every cluster of every endpoint to check if the mandatory ones are present and now is fine.


Mixture of upstream/product folder names in the SDK

kafkamgmt - uses common world
srsmgmt - uses srs which is internal name, same as kas.
kafka vs kas
srs vs serviceregistry vs registry

It will strongly suggest renaming package to registrymgmt before we get more usage.

Yeah you are right.

Done in #130


13.1 Quick Draw Cat Autoencoder --'Model'object has no attribute '_get_distribution_strategy' --Issue and Solution

For Anaconda/miniconda3 Environment, there is an error for no attribute '_get_distribution_strategy' , I have found out the solution to the issue as follows.
Issue:
'Model'object has no attribute '_get_distribution_strategy'
Solution:
Go to the following directory
/home/user/miniconda3/lib/python3.7/site-packages/tensorflow_core/python/keras
Open callbacks.py and make the two changes as follows. Please remember to back up the original  callbacks.py in aother file for further reference.
Change the lines of code --callbacks.py --1530 line
    # TensorBoard callback involves writing a summary file in a possibly distributed settings.
    self._log_write_dir = distributed_file_utils.write_dirpath(
        self.log_dir, self.model._get_distribution_strategy())  # pylint: disable=protected-access

to the following lines of code.
    # In case this callback is used via native Keras, _get_distribution_strategy does not exist.
    if hasattr(self.model, '_get_distribution_strategy'):
      # TensorBoard callback involves writing a summary file in a
      # possibly distributed settings.
        self._log_write_dir = distributed_file_utils.write_dirpath(
            self.log_dir, self.model._get_distribution_strategy())  # pylint: disable=protected-access

Change the lines of code --callbacks.py --1728 line
# Safely remove the unneeded temp files.
distributed_file_utils.remove_temp_dirpath(
    self.log_dir, self.model._get_distribution_strategy())  # pylint: disable=protected-access

to the following lines of code
    # In case this callback is used via native Keras, _get_distribution_strategy does not exist.
    if hasattr(self.model, '_get_distribution_strategy'):
      # Safely remove the unneeded temp files.
        distributed_file_utils.remove_temp_dirpath(
            self.log_dir, self.model._get_distribution_strategy())  # pylint: disable=protected-access

Please take the following weblinks for reference.
github1: https://github.com/tensorflow/tensorboard/issues/3125
github2: https://github.com/tensorflow/tensorflow/pull/34870

after applying these changes to the lines of code , a new error is coming out  ;
AttributeError: 'TensorBoard' object has no attribute '_log_write_dir'

Hi Hadiaz1:
Since I upgraded TensorFlow 1.5 to TensorFlow 2.1, so the TensorBoard has separated from Tensorboad. That means that Tensorboard needs to be installed independently. That is Google's new practice. During running the scrip in Jupyter Notebook, the script has no any problem with the above-mentioned solution. I will try to install TensorBoard on Ubuntu 18.04 and see what happens with TensorBoard.
Best regards,
Mike

Hi Hadiaz1:
After installation of TensorBoard, I test the script again.

Add the magic line of code to load the TensorBoard notebook extension at the beginning of the lines of code.

%load_ext tensorboard

Start TensorBoard within the notebook using magics.

Please put them after the section of the first training. For example, it has the lines of code autoencoder.fit(...). It originally has the mark In [5]
%tensorboard --logdir logs
It shows the first tensorboard instance.

Start TensorBoard within the notebook using magics.

Please put them after the section of the first training. For example, it has the lines of code variantional_auto_encoder.fit(...). It originally has the mark In [11]
%tensorboard --logdir logs
It shows the second tensorboard instance.
If you have more findings, please feel free to contact me.
Notes:
I tied to add the following line of code in the the scripts but did not succeed.
file_writer = tf.compat.v1.summary.FileWriter('/path/to/logs', graph).
Without the line of code, Tensorboard catches the orginal logs infomration such as LeNet-MNIST-1. It is a compatibility issue between TensorFlow 2.1 and the original script of 13.1 Quick Draw Cat Autoencoder.  Google has not further improvement on the issue.
Reference:
tensorboard: https://github.com/tensorflow/tensorboard/blob/master/docs/tensorboard_in_notebooks.ipynb
tensorboard in notebook: https://www.tensorflow.org/tensorboard/tensorboard_in_notebooks
tensorboard v2.2.1: https://github.com/tensorflow/tensorboard
tensorboard issue: https://github.com/tensorflow/tensorboard/issues/3634

my problem was kinda solved suddenly with some restarts and runs .Thanks you so much anyways, those code changes may have made something !


Optimization for blob.Data method

Old Data method will read all blob to the memory and then wrap it as a io.Reader, this is unnecessary.

I think this can be closed since #94 was merged.

@mrexodia you are right, thanks!


Add unit tests for storage

The project is working but have a very big issue: there is no tests. Everything is tested by hand...
The goal here is to add unit tests to every storages

I can take this up, but can you tell me which classes exactly do you refer as Storage

@dearrudam already made a PR for the issue, but you can have a look on his PR if you want to add more tests :slightly_smiling_face:


Updated the installation page distributions table for 1.5

Update the distributions table on the installation page to reflect those distributions that have been upgraded for Kubeflow 1.5.
https://www.kubeflow.org/docs/started/installing-kubeflow/

@jbottum we'll want to ping on all the distribution teams on this ticket. Would you please tag everyone involved?

We need to know which distros will be updated for the 1.5 release.

@shannonbradshaw Hey Shannon - I believe that we are tracking on this issue, https://github.com/kubeflow/manifests/issues/2146.      @DomFleischmann do you have any suggestions on this requirement for the kubeflow.org docs ?  will this material be similar to the chart that you are creating for the 1.5 slide on distributions ?

@jbottum I think their readiness for 1.5 should be tracked in the issue that you mentioned. Regarding updating the distribution table in the docs, it seems to me like that should usually be part of the PR where they update their documentation for the next version and should be handled by each distribution on their own. I might be wrong with this but that seems the most reasonable as they know best if they are ready or not.

Created a PR to update AWS distro status #3191

OK. Let's the distributions update this page as they become ready.

No longer applicable.
/close


[Suggestion] Allow enchantment descriptions to be toggleable

As the title suggests, I would like to see the ability to toggle enchantment descriptions

If you wouldn't mind clarifying, you mean toggling them in the sense that they would be shown without holding shift?

If you wouldn't mind clarifying, you mean toggling them in the sense that they would be shown without holding shift?

No, I think they mean being able to disable the "Hold Shift, Hold Alt, etc." entirely in the config

Any updates? @kpzip

The feature is finished, still haven't gotten around to doing comprehensive tests though

The feature has been implemented and is available in v1.2.2.


Windows multiple Scan

Hi,
On Windows 10 desktop I can use barcode scanner one time but on second scan click the scan load as a black screen with red bar but video seems to be launched lately and then barcode close by itself.
Thanks

Happenning to me too..., scan works fine the first time, but on second scan onward it fails most of the time (not all). in my case I am using it with Ionic 2.. here is a simplified code and the error I am receiving..
import {BarcodeScanner} from 'ionic-native';

public scan() {
    console.debug("Initiating scanning.");
    this.platform.ready().then(() => {
        console.debug("Platform ready, scanning...");
        BarcodeScanner.scan().then(
            (result) => this.scanSuccess(result),
            (error) => this.scanError(error));
    });
}

private scanSuccess(result): void {
    console.debug("Scan success: " + JSON.stringify(result));
}

private scanError(error): void {
    console.error("Scan failed: " + JSON.stringify(error));

And the error..
Scan failed: {"asyncOpType":"Windows.Foundation.IAsyncOperation`1<Windows.Media.VideoFrame>","asyncOpSource":{},"asyncOpCausalityId":370}

This does not reproduce for me on 5.0.1 - could you please re-test, @yesiman?

Please reopen if this still occurs.


WIP: This test should fail, with the cloud-config being ignored due to low…

… memory
the test I was working on to track down #1625
doing this same thing using vmware and 768MB ram causes the failure.
Signed-off-by: Sven Dowideit<EMAIL_ADDRESS>

Note - part of the problem is that the docker-load fails, so hopefully this test won't even make it to running now that I've made failing docker-load cause a panic.

I solved this differently by making ros core dump if it had a failure during initialisation.


some accelerator enhancements

Hi,
My main code base have been switched to be based on accelerate for a while, it works grate! Thank you guys so much!
There are a few things I think it's a common need.
The first is .distsibuted attributes, which is basically .num_processes > 1. I mean, it's not a lot of works but .... I'm an extremely lazy person.
def distsibuted(self):
    """
    Runs in distributed mode
    """
    return self.num_processes > 1

Besides, these two decorators are often used in my code when saving models and downloading datasets, I think it could be useful.
def on_main_process(func):
    """
    Run func on main process
    """
    @wraps(func)
    def wrapper(self, *args, **kwargs):
        if self.is_main_process or not self.distributed:
            return func(self, *args, **kwargs)
    return wrapper

def on_local_main_process(func):
    """
    Run func on local main process
    """
    @wraps(func)
    def wrapper(self, *args, **kwargs):
        if self.is_local_main_process or not self.distributed:
            return func(self, *args, **kwargs)
    return wrapper

Nice idea! @sgugger WDYT of instead having this wrap around AcceleratorState directly, rather than having this be attached to Accelerator?

I don't mind having wrappers on the Accelerator for those. As long as the naming is clear, it's not a bad thing to have multiple properties/methods/context managers for different use cases :-)

I do think distsibuted should be in AcceleratorState, but for the decorators, I think it's better to put it in the Accelerator

There is no way to set it and make it work (you can't transform the training in distributed mode suddenly), so it should be a property.

There is no way to set it and make it work (you can't transform the training in distributed mode suddenly), so it should be a property.

Ohh, by property I meant the @property decorated function, and by variable I meant a value. So I assume variable is better.
Though as PyTorch is supporting elastic training, maybe other values (e.g. num_processes) should be set to a property?

I understood properly, and I still stand by the fact it should be a property, decorated by @property and not a variable.

Should we makee other variables (e.g. num_processes) property too?

@ZhiyuanChen I'd say it couldn't hurt I think!

Hi sorry for this late reply.... but do you mean if changing doesn't hurt or not changing doesn't hurt...

Changing it! 😄

Changing it! 😄

Haha will do

I believe everything has been merged now, so I'm gonna close this issue.
Thanks again for this grate work!


feat: support Health Check in BackendTrafficPolicy

What type of PR is this?

What this PR does / why we need it:
This PR introduces active health checking to BackendTrafficPolicy, supporting 3 types of health checkers: HTTP, gRPC, and TCP.
Which issue(s) this PR fixes:

relates to https://github.com/envoyproxy/gateway/issues/2126

/retest

thanks @lemonlinger , it's better to add some e2e test and user facing doc in the following PR.

Sure, Will do.

/retest

Hey @AliceProxy, would you like to take a review on this ?

/retest

/retest

/retest

/retest

/retest

@lemonlinger can you address the comments ?

@lemonlinger can you address the comments ?

Sure thing. I'll get it done today.

LGTM thanks !
can you also raise a separate GH issue for E2E and docs

Sure thing.


List of incompatible modules

For modules marked with ⚠, additional steps may be necessary. Modules marked with ❌ are incompatible.

[⚠] Font modules

To prevent crashes, install HusykDG's FontLoader.
If you're using Font Manager, try its Fix apps crashing feature.

[⚠] Modules mounting to /vendor (e.g., ViPER4Android)

Some browser apps may crash with an EGL error. If you don't experience this issue, no additional steps are needed.
Set ro.zygote.disable_gl_preload prop to 1 to fix this.

[⚠] Shamiko

Requires Zygisk Assistant v2.1.3+ and Shamiko v1.1.1+.

[❌] Modules with runtime resource overlays (e.g., Pixelify)

Currently, there are no plans to address this issue, but contributions are welcome.

先刷了这个模块，在刷shamiko，系统直接损坏

Is there any plan to make this module compatible with Shamiko?

Is there any plan to make this module compatible with Shamiko?

It should work now if you're using the latest version of both modules.

Another module that can't be used with Zygisk Assistance is Youtube RVX root module and youtube music RVX root module every time I tried to install Zygisk Assistance with this 2 module install abd enable after I reboot the Zygisk Assistance module will disappear like it have been remove. That's so sad

Another module that can't be used with Zygisk Assistance is Youtube RVX root module and youtube music RVX root module every time I tried to install Zygisk Assistance with this 2 module install abd enable after I reboot the Zygisk Assistance module will disappear like it have been remove. That's so sad

It is compatible and this was answered in #15 #40 #43 #67 #68. You have to whitelist the YouTube app, see #67 (comment) for a visual guide of this.

Another module that can't be used with Zygisk Assistance is Youtube RVX root module and youtube music RVX root module every time I tried to install Zygisk Assistance with this 2 module install abd enable after I reboot the Zygisk Assistance module will disappear like it have been remove. That's so sad

It is compatible and this was answered in #15 #40 #43 #67 #68. You have to whitelist the YouTube app, see #67 (comment) for a visual guide of this.

Thank you for your guide sir. The module I was used is from rvx team not the one that #67answer any way i have change to those module and now it work fine thank you very much sir.

[⚠] Modules mounting to /vendor (e.g., ViPER4Android)

Hi! Thanks for your work, it's really interesting to see how root gets hidden. I'm one of the devs working on the reverse-engineered ViPER4Android. What's the issue with it? Do you have some recommendations to fix it? :)

[⚠] Modules mounting to /vendor (e.g., ViPER4Android)

Hi! Thanks for your work, it's really interesting to see how root gets hidden. I'm one of the devs working on the reverse-engineered ViPER4Android. What's the issue with it? Do you have some recommendations to fix it? :)

It's not really related to ViPER4A, but to some versions of Adreno EGL drivers. Since the module mounts to /vendor, APatch and KSU will mount an overlayfs at /vendor.
This makes all file IO go through that device before it is unmounted and results in different inode and/or dev values for the same path (depending on the kernel config).
The EGL driver fails to load OpenGL 3.1+ extension if EGL was preloaded before the Zygote fork and the overlayfs was unmounted afterwards due to the effect I've mentioned. Unfortunately I don't have a device with such driver so I couldn't debug the issue myself.

[⚠] Modules mounting to /vendor (e.g., ViPER4Android)

Hi! Thanks for your work, it's really interesting to see how root gets hidden. I'm one of the devs working on the reverse-engineered ViPER4Android. What's the issue with it? Do you have some recommendations to fix it? :)

I have a bit of a workaround for this but this will require overlayfs.
you can inject something like this on module's post-fs-data.sh
MODDIR="${0%/*}"
# you can fake this to whaever, you can even
# mimic a vendor mount like my_bigball
modid="vendor_audio"

# make sure $MODDIR/skip_mount exists!
# this way root manager wont mount it
# we handle the mounting ourselves
[ ! -f $MODDIR/skip_mount ] && touch $MODDIR/skip_mount

[ -w /mnt ] && basefolder=/mnt
[ -w /mnt/vendor ] && basefolder=/mnt/vendor

mkdir $basefolder/$modid

cd $MODDIR

for i in $(ls -d */*); do
	mkdir -p $basefolder/$modid/$i
	mount --bind $MODDIR/$i $basefolder/$modid/$i
	mount -t overlay -o "lowerdir=$basefolder/$modid/$i:/$i" overlay /$i
done

I guess this is a vendor-mount-mimicking method
It will global mount things though so, it is a double edged sword.
Demo: https://github.com/backslashxx/weebu/


Client glitched heavily affects the usage of the client

https://github.com/runelite/runelite/wiki/Disable-Hardware-Acceleration

Appreciate ya mate


Add camera scrolling to the game

This would be likely the final feature added to ICM
When pressing a directional button (WASD), all points will need to be rendered (not have their position updated) with their position going in the opposite direction. For example, I press W to move the camera up, meaning the points will have to move down (point will have point.x + 0 and point.y + [n], where n is positive to reflect an increase in y and the point moving down)
Continuing from the last point, if the player places a new point, it will have the coordinates of mouse.x - 0 and mouse.y - [n]. I use - instead of + since in our example, we want to place a new point depending on the new camera placement (if it was up).

Also lines will need to have their positioning updated as well


make systemd unit file

this systemd unit file must ensure that roflcoptor starts after tor but before the various tor-control-using-apps start.

i added a rough systemd unit file... but we can fix it later when we do the packaging.


Update docs on .python-version file

Summary
Closes https://github.com/astral-sh/uv/issues/7027

When displaying the file structure of a uv-managed project show the .python-version file which is now created by default.
Mention the purpose of the .python-version file in Guides/Working on projects/Project structure
In Concepts/Python versions/Project python versions, changed sentence about .python-version file to reflect the fact it is included by default so is likely to be present.

@zanieb - Sorry haven't got npx set up yet. Did these edits on Github.com.


Make Order use fixer configurable and added sort by length option

@keradus please check changes

@keradus please check

please rebase on top of 1.12

@keradus already done

@DarkaOnLine
This branch has conflicts that must be resolved

maybe you have rebased on top of your local, outdated version of 1.12 branch ?

git pull --rebase upstream 1.12. You can use git remote add to add this repo as a remote (doesn't have to be called upstream, name it what you want).

@keradus yup, you are correct. Sorry. Hope know it is ok

There are no tests for new sorting method run on group import statements.

@keradus made requested changes

Travis is sad :(

Travis is sad :(

When is it not? :trollface:

@keradus it's green now :clap:

thanks @DarkaOnLine
@GrahamCampbell could you make a final review, please?

@keradus @GrahamCampbell any news on this ?

@GrahamCampbell could you make a final review, please?
@DarkaOnLine please check mine comment https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2148#discussion_r77135277

The length sorting still doesn't sort alphabetically for those of the same length.

@GrahamCampbell , it does:
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2148/files#diff-a9cd1458f8196198c2b7c5377bec09d3R1350

Looks OK to me. 👍

@keradus can we finally merge this ?

I would rather merge this into 2.x line instead of 1.x to avoid merge hell. What do you think @SpacePossum ?

hmm this going to be a merge issue with https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2235 as well, that one is bug fixing the Fixer creating invalid code + sorting items within group definition. The PR'ed version here doesn't deal fix any of those 2 issues.
With the new 2.x beta out and close to release I think it makes more sense to get 2235 merged and ready in 1.x and master, followed with this new feature merged in master only to make the merging as less hard as possible.

Please rebase on top of master

Closing this because it was easier to make separate PR #2280


scons: Remove acados Include Paths from Global

Removes the following global include paths from SCons and relocates them to the specific modules that require them:

#third_party/acados/include
#third_party/acados/include/blasfeo/include
#third_party/acados/include/hpipm/include

This change improves build performance by reducing the number of directories the compiler searches globally.

Can you post before/after for these PRs? There's a tradeoff being made here on using the same environment everywhere (connivence) and time. If it's costing a lot of time, then I think these are worth it.
I propose we do one big PR with all the SCons optimizations, then evaluate whether it's worth it. If it is and the PR is too big, we can split it up then.

I agree, it's not worth it for these small changes.


Feature/stf chain parallel reorder

Draft
Pending

[ ] Export types
[ ] Fix PR history (working on the wrong base branch)
[ ] Test Parallel rules
[ ] Test Retrocompatibility
[ ] Test Reordering

This PR adds three related features to the STF Execution

Create and Execute a Serial or Parallel Execution Plan for the STF.
Adds an optional game callback submittedDataReordering function to reorder Submitted Inputs.
STF Includes ChainData to allow STF to get additional information about the current block.

STF Chaindata argument
Adds new argument to STF invocations.
This call is retrocompatible and does not require changes from the game side.
Game STF signature
export default async function (
  inputData: STFSubmittedData,
  blockHeight: number,
  randomnessGenerator: Prando,
  dbConn: Pool,
  chainData: ChainData
): Promise<SQLUpdate[]> {

Reorder Function
Game STF Router can optionally return the submittedDataReordering function.
This is retrocompatible, the router can submit only stateTransition (does not require changes)
Game STF Router:
const reorderFunction = (chainData: { submittedData: { inputData: string }[] }) => {
  console.log('Reordering Submitted Data');
  return chainData.submittedData;
};
// This function allows you to route between different State Transition Functions
// based on block height. In other words when a new update is pushed for your game
// that includes new logic, this router allows your game node to cleanly maintain
// backwards compatibility with the old history before the new update came into effect.
function gameStateTransitionRouter(_: number): {
  stateTransition: Function;
  submittedDataReordering: Function;
} {
  return {
    stateTransition: gameStateTransitionV1,
    submittedDataReordering: reorderFunction,
  };
}

Parallel Execution
ENV variable GAME_STF_EXECUTION_MODE can be set to parallel or serial
Serial by default.
This allows to run STF functions that do not have dependencies in parallel.

----------------------------------------
| Execution Plan (parallel):
|   @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|     @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|       @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|         @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|   @a|i|300| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|     @a|i|300| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|       @a|i|300| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|         @a|i|300| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
----------------------------------------

----------------------------------------
| Execution Plan (serial):
|   @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|     @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|       @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|         @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|           @a|i|301| (0xdc3da0d7a6338039d79272653172bdd056ec0f7a)
|             @a|i|2| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|               @a|i|2| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|                 @a|i|2| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
|                   @a|i|2| (0xbcb878950cbcfc4a1d13fd481ffeec6c2f310ff1)
----------------------------------------

We're going to need to rebase this since the orderbook-dex PR has been merged,  and also we need to update the Paima Engine docs to mention these features


Please Exclude Compilation Product

first of all, this is a great library, but i think it would be great if you could exclude compilation product such as *.class, it will make the repository less cluttered and reduced in size. 😄

i would like to suggest usage of .gitignore in this repository

Ah yes, thank you
it is a really old project that I finally decided to just open it 😅


fix: navbar alignment in mobile devices

Related Issue
Closes: #645
Describe the changes you've made ✍

Fixes Navbar alignment and spacing in mobile devices.

Screenshots (if any) 📸

Hi there, please donot go around creating issues & making PRs WITHOUT LOOKING for the same first.
If you take a look at #596 , someone is working on it (2 days ago) - and i'd like to wait for a few days before I close the issue, and make another one.
For now I am closing this PR + Issue. We can create one later, when/if they decide not to work on it.
Thanks,
Tamal.

Hi there, please donot go around creating issues & making PRs WITHOUT LOOKING for the same first.
If you take a look at #596 , someone is working on it (2 days ago) - and i'd like to wait for a few days before I close the issue, and make another one.
For now I am closing this PR + Issue. We can create one later, when/if they decide not to work on it.
Thanks,
Tamal.

Okay i didnt see it, I raised the issue few days ago and i didnt notice it someone is already working on it, my bad


Support for Dynamic Messages

Currently, Translategram only supports static messages for translation within Telegram bot handlers. However, there is a need for translating dynamic messages, where the content of the message is generated dynamically based on user inputs or external APIs.
One example of such a case is ChatGPT telegram bot. The bot uses the OpenAI Chat API to generate responses based on user inputs. The translated responses from Translategram are required to provide a localized experience to users.
To address this limitation, we need to enhance Translategram to support dynamic messages. This would involve modifying the Translategram library to handle translation for messages generated at runtime, such as the responses generated by the chat function in the example.
By adding support for dynamic messages, Translategram will enable developers to create more interactive and multilingual Telegram bots that can provide real-time translations for user-generated content.

#5 is kinda did it.


Underscored names don't work with MatchNamesWithUnderscores = true

Hello. I have a table auth with fields id and discord_id. I have such a model:
[Table("auth")]
public class AuthDataModel
{
    [Key, Identity]
    public int Id { get; set; }
    public string DiscordId { get; set; }
}

Dapper is configured to ignore underscores: Dapper.DefaultTypeMap.MatchNamesWithUnderscores = true;
Now when I try to get all the records using var foundUsers = (await m_authRepository.FindAllAsync()).ToList(); I get an error: 42703: column auth.discordid does not exist
While this call works correctly:
await m_authRepository.Connection.QueryAsync<AuthDataModel>("SELECT * FROM auth")).ToList();
May be related to https://github.com/DapperLib/Dapper/issues/818
What is the possible solution other than marking all the underscore named columns with [Column("underscored_name")] ?
Thanks

@b2soft SqlGenerator doesn't read Dapper.DefaultTypeMap.MatchNamesWithUnderscores. Current solution only with [Column("underscored_name")]

Are there any plans to support such flag?

@b2soft Flag about mapping, not about sql generation...
I guess you want support naming conventions in SqlGenerator. If you have time feel free to implement it and raise a PR.


Incorrect enum coverage issue in Kotlin 2.0

Describe the bug
A clear and concise description of what the bug is.
Coverage of enum classes is incorrectly calculated when using kover in Kotlin 2.0.
Errors
If present, stacktraces or files from build/kover/errors directory
N/A
Expected behavior
A clear and concise description of what you expected to happen.
Incorrect display of the last ‘}’ string in the Enum class

Kotlin 1.9.23
Kotlin 2.0.0

Reproducer
A link to your project, if it is open source. Otherwise, please try to reproduce the problem on a small project.
If this is not feasible, give an example of the code on which the problem manifests itself.
Reports
If applicable, report files or screenshots.

Kotlin 1.9.23
Kotlin 2.0.0

Environment

Kover Gradle Plugin version: 0.8.0
Gradle version: 8.7
Kotlin project type: Kotlin/JVM (Spring boot)
Coverage Toolset (if customized in build script): Kover
Other context important for this bug: Windows 11

Fixed in 0.8.2


Add missing enum values

Fixes #278

Note: No idea if it build as my work PC can't build the solution at the moment, showing :

The package 'StyleCop.Analyzers/1.0.0' could not be found in the libraries section of the lock file. This may indicate your lock file is corrupted.

So i'll let appveyor decide :)

I'll use this PR to also fix #280

LGTM. After this, the policy to always specify values (perhaps by writing an analyzer) sounds even better.

Yes while I think we should allow still allow to not specify them when they aren't in the header files, we should have an analyzer and only suppress it after the header file have been checked and a comment is added.
// Sequential values are used in the original header file (WinUser.h)
#pragma warning disable XXXX

It's a lot more specific an would avoid "OOPS" cases like this


working on white screen issues

[x] Make an html element in the app that'll exist if the react system doens't load. had a title so if you hover over it it'll say 'loading...' and its slightly colored.
[x] Having the raw html scripts setup electron log forwarding so we can catch more errors earlier. stashing this original stuff in console._log etc.
[x] refactored the setupLocalLogs and console-helper stuff into one thing so this is all stored in one place. Removed unused pipeLogs etc

@keybase/react-hackers

@keybase/react-hackers bump

lemme know if that makes sense @gabriel

👍


split current-item-defaults per element type

When changing the default while a specific element is selected, or a tool type is selected, it changes the default for all future shapes.
This may be undesirable if you e.g. change a stroke color of a rectangle, you may not want that to apply to future text elements. Especially if you switch the color to a transparent color.
proposal
Cache defaults per element types, or type groups (e.g. rectangle + ellipse + diamond could use the same defaults fine).

Stroke color applying to text color is the unexpected behavior that affects me the most.
To be honest, I'd prefer the defaults always start as the same black and white shapes and text (at minimum in new documents, but also just in general).
There are already several options for copy styles quickly:

The contextual menu
The copy/paste styles hotkeys
Option-dragging elements
Copy-pasting items

To be honest, I'd prefer the defaults always start as the same black and white shapes and text (at minimum in new documents, but also just in general).

Remembering the last preferences is to help in creating shapes of the same style without needing to edit them afterwards.
But there is an argument to be made about changing the defaults if you just edit an existing shape, vs changing the setting while the tools is selected. Discussion: https://github.com/excalidraw/excalidraw/issues/6395


Extra tolerations for route daemonset

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #1844
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[x] Local K8S cluster

/rebase test=true

Unit test fails on go part, I think it's not related to my changes. Can someone help me?

Hi @Sharathmk99, we have some issues related to unit tests: sometimes they are failing for no reason, and when re-runned they work (as in this case). Your pr cannot be the cause of the fail

/merge


mdsd on bootstrap node

This PR is just for discussion, showing the direction it's going.
Changes made only in the /vendor/ subtree, not propagated to the openshift/installer repository yet.
Some of the MDSD parameters are still hardcoded - will be parametrised further.
Very much WIP.
cc @mjudeikis

Looks good. if @jim-minter  will not have any comments, lets raise a PR into https://github.com/jim-minter/installer/tree/release-4.3-azure tomorrow.

Closing -> #815 which has proper dep vendoring.


Support for Angular 17

Hi,
Today has been released Angular 17, can you increment the peer dependencies range to support this version?
https://blog.angular.io/introducing-angular-v17-4d7033312e4b
Thanks for your work!
Just in case anyone needs to update, by forcing the installation it should work
npm install --force

+1

Hi,
Please make this also compatible for angular V17

+1

+1

Alternative to force flag in package.json:
{
  // ...
  "overrides": {
    "angularx-qrcode": {
      "@angular/core": "$@angular/core"
    }
  }
}

#232 I created a PR for that!

Any news? @Muehli25 's PR should do it

+1

Hi @all,
thanks to @Muehli25  for his work on the upgrade for angular 17 and sorry for the delay.
Let me know, if the latest version works for you


WIP - Validation rule for duplicate security requirements

Addresses: #37 Validation Rule: Detect duplicate Security Requirements
@EricWittmann This is very much a WIP, I just want to get your opinion on if it functionally solves the problem
You can just run the validation test package on its own to see it working

The CI build is expected to fail for now also

@EricWittmann  I have addressed feedback and added tests, so I will mark this as ready for review

@EricWittmann  All done, Thanks!


Missing elements on 'Connect Dataset' menu in Firefox

Seems like a Firefox issue, on several versions of Firefox, both OSX and Win. Here is a link to a screenshot of the page after selecting a CSV for upload. The button is working fine in Chrome.
screenshot

Description
The Connect Dataset button and the Let CartoDB automatically guess... checkbox do not appear when trying to upload a file to import.
Those elements seem to be displaced to the bottom part of the page, where they are not reachable.
Also note that the closing 'X' button is displaced as well.
Context
The issue reproduces with Firefox versions earlier than 38.5.2 ESR
Screenshot

Please @matallo review

I've checked Firefox version 42 and 43 (latest ones) and seems like it appears correctly, what is your version?

Closing since v38 is the minimum version required


Add note about gitignoring react_bundle

Thank you @sethherr  :)
Currently we are working for better generators, this includes better entresies for .gitignore
merged #48


Improve Readme.md file

Please comment on this issue , I will assign to you !

I will work on this

Ill do this

Please comment on this issue , I will assign to you !

I would lkie to work on this

If this is still open, I would like to work on this, will complete it within a week.


md5 url, what if collision?

any problems if have collision? will new address be generated...?
i can accept that same image can have same url but if different image, will it detect?

what is your solution？


Fix logo link

Currently hardcoded to dev-dashboard.solarforecastarbiter.org.

It's currently hard coded to https://dev-dashboard.solarforecastarbiter.org/tep
It's the /tep that I am worried about.


deploying WebJob

How can this be used to deploy a WebJob project to an AppService?

@srijken - Web jobs are just regular file content, so there shouldn't be anything special required in order to deploy them.  Building them may be a different story of course, but for this repo which is specific to the deployment action, I don't see why it would be anything different.  Are you running into issues with it?

Here is a working example for you brother. Just setup the publish profile as a secret and your done.
And of course change the variables.
`name: Deploy WebJob
on:
workflow_dispatch:
push:
branches: [ develop ]
env:
WEBJOB_NAME: YourWebJobName
AZURE_APP_NAME: your-azure-app-service-name
PROJECT_ROOT: path/to/your/Project.Root
TEST_PROJECT_ROOT: path/to/your/Project.Root.Tests
DOTNET_VERSION: '3.1.x'
jobs:
build:
runs-on: ubuntu-latest
steps:
# Checkout the repo
- uses: actions/checkout@v2

# Setup .NET Core SDK
- name: Setup .NET Core
  uses: actions/setup-dotnet@v1
  with:
    dotnet-version: ${{ env.DOTNET_VERSION }}

- name: dotnet test
  run: |
    dotnet test ${{ env.TEST_PROJECT_ROOT }}

- name: dotnet publish
  run: |
    dotnet publish -c Release -o './publish/App_Data/Jobs/Triggered/${{ env.WEBJOB_NAME }}' ${{ env.PROJECT_ROOT }}

- name: Deploy to Azure App Service
  uses: azure/webapps-deploy@v2
  with:
    app-name: ${{ env.AZURE_APP_NAME }}
    publish-profile: ${{ secrets.PUBLISH_PROFILE }}
    package: './publish'`

Here is a working example for you my friend. Just setup the publish profile as a secret and your done.
(and of course change the variables)
`name: Deploy WebJob
on:
workflow_dispatch:
push:
branches: [ develop ]
env:
WEBJOB_NAME: YourWebJobName
AZURE_APP_NAME: your-azure-app-service-name
PROJECT_ROOT: path/to/your/Project.Root
TEST_PROJECT_ROOT: path/to/your/Project.Root.Tests
DOTNET_VERSION: '3.1.x'
jobs:
build:
runs-on: ubuntu-latest
steps:
# Checkout the repo
- uses: actions/checkout@v2

# Setup .NET Core SDK
- name: Setup .NET Core
  uses: actions/setup-dotnet@v1
  with:
    dotnet-version: ${{ env.DOTNET_VERSION }}

- name: dotnet test
  run: |
    dotnet test ${{ env.TEST_PROJECT_ROOT }}

- name: dotnet publish
  run: |
    dotnet publish -c Release -o './publish/App_Data/Jobs/Triggered/${{ env.WEBJOB_NAME }}' ${{ env.PROJECT_ROOT }}

- name: Deploy to Azure App Service
  uses: azure/webapps-deploy@v2
  with:
    app-name: ${{ env.AZURE_APP_NAME }}
    publish-profile: ${{ secrets.PUBLISH_PROFILE }}
    package: './publish'`

@joeystdio - perfect!  I've been trying for hours to get it to work.  The key for me was getting the files deployed to the right place, i.e under app_data/jobs/continuous.
I couldn't find a single piece of documentation that explained how to do the whole thing. Wish I'd hit your post above sooner- would've saved me going round the houses.
Note: the above also works with azure login rather than publish profiles which didnt work for me, ie:
      # ---------------------------------------------
      # Login using Service-Principal-type secret
      # See: https://docs.microsoft.com/en-us/azure/app-service/deploy-github-actions?tabs=userlevel#configure-the-github-secret
      # ---------------------------------------------
      - uses: azure/login@v1
        with:
          creds: ${{secrets.AZURE_WEBJOB_CREDENTIALS_DEV}}

      # Deploy to Azure Web apps
      - name: Deploy to Azure App Service ${{env.AZURE_APPSERVICE_NAME}}
        uses: azure/webapps-deploy@v2
        with:
          app-name: '${{env.AZURE_APPSERVICE_NAME}}'
          package: '${{env.PUBLISH_OUTPUT_ROOT}}'

      - name: logout
        run: |
          az logout

Hi. I am having a very difficult time getting this to work for .Net Core 7.0 WebJob for Azure App Service using GitHub Action Workflow.
I would appreciate any tips... Thank you so much ahead of time.
`name: Deploy WebJob
on:
push:
branches:
- master
workflow_dispatch:
env:
WEBJOB_NAME: DotNetWorker
AZURE_APP_NAME: 'webjob-xpert'
PROJECT_ROOT: ./DotNetWorker
DOTNET_VERSION: '7.0.x'
jobs:
build:
runs-on: windows-latest
#runs-on: ubuntu-latest
steps:
# Checkout the repo
- uses: actions/checkout@v2

# Setup .NET Core SDK
- name: Setup .NET Core
  uses: actions/setup-dotnet@v1
  with:
    dotnet-version: ${{ env.DOTNET_VERSION }}

#- name: dotnet test
#  run: |
#    dotnet test ${{ env.TEST_PROJECT_ROOT }}

- name: dotnet publish
  run: |
    dotnet restore
    dotnet build --configuration Release
    dotnet publish -c Release -o './publish/App_Data/Jobs/Continuous/${{ env.WEBJOB_NAME }}' ${{ env.PROJECT_ROOT }}

- name: Deploy to Azure App Service
  uses: azure/webapps-deploy@v2
  with:
    app-name: ${{ env.AZURE_APP_NAME }}
    publish-profile: ${{ secrets.AZUREAPPSERVICE_PUBLISHPROFILE_77DA0552C0F24762B786220FDECEA669 }}
    package: ./publish`


Add logout support

simply listens for logouts on a hardcoded path /logout
if Request contains the cookie for SessionId, then the Session is looked up and removed from the SessionStore
The client is redirected to default Service Id dictated by the subdomain (i.e. configured in customerIdentifier)

Fixes #114

Current coverage is 95.16%

Merging #115 into master will decrease coverage by -0.83% as of 5a827fb

@@            master    #115   diff @@
======================================
  Files           37      37
  Stmts          873     890    +17
  Branches        40      41     +1
  Methods          0       0
======================================
+ Hit            838     847     +9
  Partial          0       0
- Missed          35      43     +8

Review entire Coverage Diff as of 5a827fb

Powered by Codecov. Updated on successful CI builds.

@trane done. let me know if you are ok with this. If not, i will merge it


Relative images not working

When using relative paths to refer to an image or figure, it tries to resolve the path relative to the base URL of the page. I think this code from head.html is suspicious:
<base href="{{ .Site.BaseURL }}">
If I just remove it, everything works fine.
Is there any reason not to remove it?

I use <base> to ensure every relative path works as expected.
Can you provide an example of the relative path issue?

Sorry for being so vague. I have this project structure:
content
- posts
- - 2021-07-my-post
- - - index.md
- - - diagram.png

From the index.md I want to reference the diagram.png. Usually this should work using {{< figure src="diagram.png" >}}. That will be generated to an HTML img tag also using the relative src attribute. But having the base tag in the head of the HTML makes the relative src behave differently: The browser will try to resolve the image relative to the base URI and not relative to the location of the post. Instead of the correct resolution mydomain.com/posts/2021-07-my-post/diagram.png it will resolve to mydomain.com/diagram.png so the image will not be found.
When I remove the base from the theme, my post including the image works as expected. Also I cannot find anything else that does not work anymore. That's why I was asking, why exactly the base is set at all by the theme.

Thank you. I will check this issue. 👍

Short answer: If you want to add images in Markdown. Please put the image files in the static path.
Long answer: After studying the document, I think the Page Bundles feature fits your description.
And I also remember that why I did not use it:

For only adding images into posts. Page Bundles are too complex for me.
Even I use Page Bundles in Bingo. I still can not insert images in the middle of the post content with Markdown or shortcodes.

So I suggest you put the images in the static path. That is easier.
References:

(Official) Directory Structure
(Official) Static Files
(Official) Page Resources
(Official) Images in markdown relative to content
Hugo Page Resources | Regis Philibert

Thank you very much for looking into it. Indeed "page bundles" is what I am trying to achieve: Each of my blog posts comes with poorly named image files, which if thrown all together into the static directory would quickly become an unfathomable mess.
However I don't see why they are complicated. At work we use Hugo with the well-known Docsy theme and we also use page bundles, so I thought I'd look how Docsy handles this: They do no preprocessing on the "src" attribute on a "figure" (even though they overwrite figure to provide a fancy gallery and whatnot). The only difference between docsy and hugo-bingo concerning image sources is that docsy does not set a base tag in the HTML head, so relative paths for images just work.
I don't know if there is anything complicated to page bundles, which eludes me at the present time, but I have the feeling that showing images from page bundles does not have to be difficult.
As I said I removed the base tag from my hugo-bingo fork and nothing bad happened. And so many messages into this conversation I still don't know why the base tag would be useful in the first place.

I use <base> to ensure every relative path works as expected.

For me it looks like the opposite: the base tag causes relative paths to work in a (for me) unexpected way.
If you want to use images from the static folder, you should be able to do that simply by prefixing them with /, s.t. the browser knows, that these are absolute paths. The only time that is not possible is if the website is not at the root of the (sub-)domain, e.g. at mydomain.com/website. I don't know how one deals with those situations using Hugo, but even then using base seems pretty strong, because it interacts with all relative links.

I said "Page Bundles are too complex for me" because IMO it is much easier to put images in the static path. The feature is not complex technically.
Back to the base element. I found some of the Hugo themes have similar issues since my last comment. It looks like a known bug but undocumented. And it can be fixed by removing the base element as you said.
I added the base element to ensure relative paths work fine. But looks like that's not necessary today.
I will remove the base element from partials/head.html. Thanks for the reporting! 👍


⚠ gorzdrav.org has degraded performance

In 16b7e70, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 1663 ms

Resolved: gorzdrav.org performance has improved in 0f72035.


remove tcomb import detection

Since tcomb must already be requireable, this removes the now unnecessary machinery in order to detect tcomb imports from libraries exporting tcomb.
The gist is that the plugin now adds its own tcomb import using generateUidIdentifier in order to avoid name collisions

Made some tests, this plugin works nicely with tcomb-react:
https://github.com/gcanti/babel-plugin-tcomb/tree/polish#type-checking-react
/cc @giogonzo


[Market] Use reserved for collateral value instead of a pot account

make it to reserved

Maybe do it with #549. We can let everyone to unreserve his balances and reserve the new amount.


Kelsey's repository

here it is!

I still see merging conflicts, Kelsey!

yes, i'm not sure why i'm having such a hard time with it. i'm going to try again later.

no worries! I think that we should do it step by step with you sharing your screen next time

yay! it works now!


Records with certain data may irrecoverably break the database

An API call to create a new MST record with the following payload:
{
	"metadata": {
		"general_parameters": {
			"associated_publications": {
				"additional": [],
				"main": {
					"authors": [],
					"resource_type": "Article"
				}
			},
			"chemical_information": {
				"chemical_environments": [],
				"entities_of_interest": []
			},
			"depositors": {
				"contributors": [],
				"depositor": {
					"affiliations": [],
					"identifiers": []
				},
				"principal_contact": {
					"affiliations": [],
					"identifiers": []
				}
			},
			"derived_parameters": [],
			"funding_reference": [],
			"instrument": {
				"manufacturer": "Bio-Rad",
				"performance_test": {
					"published_test_protocol": {
						"authors": [],
						"resource_type": "Article"
					},
					"report": {
						"processing_steps": [],
						"size": {
							"unit": "bytes (B)"
						},
						"source": "Instrument software",
						"type": "text"
					},
					"sample_composition": []
				}
			},
			"physical_environment_at_sample_handling": {
				"atmosphere": {
					"composition": []
				},
				"humidity": {
					"controlled": false,
					"obtained_by": "Measurement",
					"unit": "%",
					"value_error": {
						"errors_are_relative": false
					}
				},
				"pressure": {
					"controlled": false,
					"obtained_by": "Measurement",
					"unit": "Pa",
					"value_error": {
						"errors_are_relative": false
					}
				},
				"temperature": {
					"controlled": false,
					"obtained_by": "Measurement",
					"operational_value": "Room temperature",
					"unit": "K",
					"value_error": {
						"errors_are_relative": false
					}
				}
			},
			"raw_data_information": {
				"collection_start_time": "2023-06-25",
				"file_information": []
			},
			"record": {
				"access_rights": "open access",
				"date_available": "2023-06-25",
				"deposition_date": "2023-06-25",
				"keywords": [],
				"metadata_access_rights": "open access",
				"project": {
					"owner": {
						"affiliations": [],
						"identifiers": []
					}
				},
				"publisher": "MBDB",
				"resource_type_general": "dataset"
			},
			"technique": "Bio-layer interferometry (BLI)"
		},
		"method_specific_parameters": {
			"data_analysis": [],
			"excitation_led_color": "RED (ex 605–645nm, em 660–730nm)",
			"experiment_type": "Affinity",
			"label_free": false,
			"measurements": [
				{
					"measured_data": {
						"x_data": {
							"id": "46cb7471-7128-4459-936f-c5c8bc35e79f",
							"values": []
						},
						"y_data": {
							"id": "1429bea7-9bd6-4758-8217-698b5ea48e5a",
							"values": []
						}
					},
					"sample": {
						"container": "Monolith Standard Capillary",
						"ligands": [],
						"preparation": [],
						"targets": []
					}
				}
			],
			"signal_type": "Initial intensity"
		}
	}
}

ends up with the server returning 500. A long backtrace pointing to the marshmallow package ends with
TypeError: 'builtin_function_or_method' object is not iterable

The offending part seems to be the measurements. If it's left empty, the otherwise same payload does not trigger any problems.
This would not be so much of a problem if the problematic data didn't get stored in the DB. They, however, apparently do because all subsequent attempts to display listing of all MST data fails with a 500 and the same backtrace.

According to @mesemus this was caused by: "the problem is inside the marshmallow schema - python implementation gets the value from a dictionary using [] at first and if it fails it uses getattr. If a name of a property is a name of a method (for example, "values" inside xdata) and it is not filled in inside the data, marshmallow will try to serialize the method instead and will fail. I'll add a workaround to the model builder to fix this. "
This should have been fixed with #83  so I'm closing it

According to @mesemus this was caused by: "the problem is inside the marshmallow schema - python implementation gets the value from a dictionary using [] at first and if it fails it uses getattr. If a name of a property is a name of a method (for example, "values" inside xdata) and it is not filled in inside the data, marshmallow will try to serialize the method instead and will fail. I'll add a workaround to the model builder to fix this. "
This should have been fixed with #83 so I'm closing it.


Use Coverity for static code analysis

Coverity offers its code analysis service for open source projects free of charge. Maybe we could integrate it into our Travis build.

I think we had this once and did not really give us meaningful hints. I know static code analysis can be helpful but you'll get many false positives and need to invest heavily before it turns out valuable. Maybe you have some time to share where with this help already some bugs are pointed out?


dev - Folders "clickable'

#447  created a situation where folders are clickable in the UI and it spits out an error. Plus the icon needs to be different. I am already working on this code. Folders should be at the top of the listing.

@Bugs5382
Has this problem not been resolved already?

Not solved, still having this issue in 1.9.0 and up. Folders show as files and cause errors when clicked.

This has since been solved with<EMAIL_ADDRESS>since we have added complete support for folders on the preview server.
See https://resend.com/blog/react-email-2


Added y measurements

Codecov Report

Merging #4 (87173ea) into master (b9e593f) will decrease coverage by 6.32%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   83.26%   76.94%   -6.33%
==========================================
  Files           4        4
  Lines        1297     1132     -165
==========================================
- Hits         1080      871     -209
- Misses        217      261      +44

Impacted Files
Coverage Δ

src/experimental/NoisyCircuits.jl
60.69% <20.00%> (-2.33%)
:arrow_down:

src/QuantumClifford.jl
80.02% <0.00%> (-7.00%)
:arrow_down:

src/experimental/quantikz_methods.jl
70.58% <0.00%> (-1.64%)
: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 b9e593f...87173ea. Read the comment docs.

Codecov Report

Merging #4 (87173ea) into master (b9e593f) will decrease coverage by 6.32%.
The diff coverage is 20.00%.

@@            Coverage Diff             @@
##           master       #4      +/-   ##
==========================================
- Coverage   83.26%   76.94%   -6.33%
==========================================
  Files           4        4
  Lines        1297     1132     -165
==========================================
- Hits         1080      871     -209
- Misses        217      261      +44

Impacted Files
Coverage Δ

src/experimental/NoisyCircuits.jl
60.69% <20.00%> (-2.33%)
:arrow_down:

src/QuantumClifford.jl
80.02% <0.00%> (-7.00%)
:arrow_down:

src/experimental/quantikz_methods.jl
70.58% <0.00%> (-1.64%)
: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 b9e593f...87173ea. Read the comment docs.


Near impossible to transpile into ESM

General information

Library version: master
Browser and OS: chrome-unstable 66.0.3355

Issue description
The "UMD" wrapper of your source makes it virtually impossible to transpile this from ES5 CJS to ESM. Notably, the module.exports are declared within an if statement inside an IIFE. Since ESM requires exports to be at the top level, a transform would have to be intelligent enough to unwrap both the if statement and the wrapping IIFE. This would be... difficult.
There are wonderful codemods like 5to6 that help bring code into the future developers want.
Ideally, this module would be converted into ECMAScript Modules according to the specification that governs the JavaScript language. The dist/framebus.js could continue to be a UMD bundle, perhaps created with tooling to generate that wrapper from the ECMAScript standard code that the library could be authored in.
Alternatively, availability of a CJS form of the module would be fine.

Thanks for reporting this. We'll look into this. Likely, in the next major version, we'll drop the UMD wrapper and the dist and just use npm to distribute.


'copy' action for rules disregards pre/post parameter for the destination device group

Hi Sven,
When I run a command along these lines:
pan-os-php in=api://<panorama_10.2> type=rule location=<source_device_group> ruletype=security 'actions=copy:<destination device group>,pre' 'filter=<my filter>'
All rules from <source_device_group>/POST (as per the filter) end up in the POST section of the destination group despite the PRE argument specified in the copy action.
Interestingly enough, as the tool executes it gives the correct output for each rule as follows:
- object '<rule name>'  passing through Action='copy' Args: location=<destination device group>, preORpost=pre,
(despite that, the rule gets placed into the POST section as it was in the source device group)
Can you investigate this one, please?

Hi,
thanks for finding this bug, which is "ONLY" available in API mode.
I just right now pushed a new develop Docker container version,
where I like to ask, if you can validate this new version and if this bug is also fixed for you.
I tested this against Panorama 11.1.x and 11.2.x
docker pull swaschkut/pan-os-php:develop
docker run --name panosphp --rm -v ${PWD}:/share -it swaschkut/pan-os-php:develop
the develop docker container is still on pan-os-php version 2.1.24
you can validate the behaviour if you add "debugapi" as an argument to your command:
pan-os-php in=api://<panorama_10.2> type=rule location=<source_device_group> ruletype=security 'actions=copy:<destination device group>,pre' 'filter=<my filter>' debugapi

I wish Palo TAC was that quick with their responses :)
I will test it later and let you know the outcome. THANK YOU!

bugfix published with #16


Use cvmfs_partsize as an integer in tests

Pull Request (PR) description
The rspec tests were setting the cvmfs_partsize fact as a string despite it being an integer for years.
Resolves:
Puppet::PreformattedError:
Evaluation Error: The string '10000000' was automatically coerced to the numerical value 10000000

My first Puppet 8 bug fixed.

This LGTM, thanks for the fix! :+1:


plugin: cannot load plugin when other dependent packages changed

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8.3
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/root/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build438394725=/tmp/go-build"
CXX="g++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
What did you do?
plugin fake code:
func OnPluginInit( ){

      pluginpkg.Foo( )
}

package 'pluginpkg' fake code
func Foo( ){
      println( "v1")

     // println("v2")
}

go build plugin to 'v1.so'
uncomment  // println("v2")  then go build plugin to v2.so
host loader fake code:
func main(){

    loadPlugin( "v1.so")

    loadPlugin( "v2.so")
}

What did you expect to see?
console output:
(after v1.so loaded)
v1
(after v2.so loaded)
v1
v2
What did you see instead?
console output:
(after v1.so loaded)
v1
(after v2.so loaded)
plugin was built with a different version of package v2
I'm using golang plugin system in MMO game server. We need use plugin system to hot fix program when server is still running.
I guess that plugin system can ONLY hot fix plugin's main package,BUT other depend packages
I've read #17832 issue, and know it's hard to make code compatible in plugin.
It's hard to make all code write in one plugin main package in order to hot fix
So, how can I make plugin and all it's depend packages hot fix with plugin system?
Thank you!

I see no way around this. Beyond the practical issue that using the dynamic loader to hot-swap code is extremely difficult, there are definitional issues. What if the init functions have changed between versions of a package? They can't be called again. What if a global variable already being used by running code changes type or is removed?
For your specific problem, consider introducing an interface type registered with a control package. When a new plugin is loaded, remove the implementation of the interface that comes from the old plugin, and add the implementation that comes from the new plugin. This lets you hot-swap running code (though you are slowly accumulating all previous versions into the process).


Wait until logged in

I am trying to dockerize a pytg bot. I encountered this problem:
Sample Code:
tg = Telegram(telegram = TG_CLI, pubkey_file = TG_PUBKEY)
tgSender = tg.sender
try:
    tgSender.send_msg("user#123",'Bot started')
except:
    print('ERROR sending start-msg')

The line tg = Telegram(telegram = TG_CLI, pubkey_file = TG_PUBKEY) will continue after the CLI has started, but before the CLI prompts first time user for phone number, which will result in an error in tgSender.send_msg("user#123",'Bot started')  in the following lines.
My question is: how to start the CLI, wait until the user authentication is done, and then proceed to send or receive messages?

The registration with the phone number and the SMS code must be made manually. After that pytg should work.
I don't see any purpose on doing that setup automated.

@luckydonald The purpose of automating the process is just to make deployment on remote servers easier (by using a single command).
I later used the following shell code as a workaround.
function checkauth() {
if [ -f /root/.telegram-cli/auth ];
then
  asize=$(wc -c < /root/.telegram-cli/auth)
  if (($asize > 0)); then
     ok=1
  else
     ok=0
  fi
else
  ok=0
fi

}

checkauth

while ((ok == 0)); do
  echo "[RUN] Not authenticated. Running telegram-CLI"
  echo ""
  $TG_CLI -k $TG_PUBKEY
  checkauth
done

echo ""
echo "[RUN] Authenticated. Starting pytg..."
pkill -f telegram-cli

python /app/p.py

echo ""
echo "[RUN] Stopping telegram-cli."
pkill -f telegram-cli

That works almost fine. The only problem is I need to press Ctrl-C to exit telegram-cli after the first login.
So if I can find a way to kill telegram-cli after /root/.telegram-cli/auth is created, the process will be more automated. :)

The only use-case I can see here, to do the setup automatic is, if you deploy a large scale of bots.
And creating a large amount of bots just screams "spamming" to me.

Oh, wait, are you still letting the user do the input?
Then I kinda misunderstood your purpose.
Maybe some startup-parameter does the trick?
--exec/-e <commands>                 make commands end exit
Maybe append --exec safe_quit to your commands?
like $TG_CLI -k $TG_PUBKEY --exec safe_quit?

I assume this is resolved?

Actually a docker would be great. I completely read over that. Sorry.
That happens when you do such stuff at 2am.

Sorry for being away for a few days. Thank you for your kind help.
In short the need is to have a single entrypoint for both the initial run and later runs. On the first run it does authentication and then starts the bot; on later runs the bot will just work normally.
I don't quite understand what --exec safe_quit does.

--exec does execute a command in the cli and then exits, and the command run is safe_quit which waits for all queries to complete, and then quits. Maybe a normal quit is sufficient, too.
So my idea,  though untested, is that this hopefully terminates the cli after a successful login.

have a single entrypoint for both the initial run and later runs.

If you are already logged in, it just should exit, so you can continue doing the rest of your routine.
If you get something working, maybe you can add a pull request.
As how I understand docker is something quite famous and useful?

@ztl8702:
I've been working on getting my bot running in Docker as well. It's not quite finished, but seems to work  well enough for testing with the following configuration: https://gist.github.com/justingood/020e24222fa1653f5cd0
The docker-compose file also lets you run the test code in-place
I ended up keeping the .telegram-cli directory outside the container for a couple of reasons:

Keeping secrets inside a container is generally a bad idea and Docker themselves recommends against it.
If the container dies, or is updated with a new version, it can be started and have access to the configuration without having to re-register.

Would a configuration similar to this fit your use-case? If you're already running a tool to spin up the bot on each server, maybe it could also pre-populate the .telegram-cli contents?
@luckydonald: Docker is pretty cool. The example I linked to shows how you can have code on your machine and run it inside the container without needing Python installed locally. You could do the same for Ruby, Go, and PHP without cluttering your machine with development libraries. So in this type of setup, all you need is Docker and a text editor and you're good to go.
For an actual deployment, you'd add the code into the container, and it will act similar to a single binary - run it, and it'll do its thing. Move it to another server, and it's ready to go. No extra configuration required.

I was just about to link you here, as I saw that in terribot

@ztl8702  I found your image at Docker Hub.
Care to share the Dockerfile?

I tried to get something working, which resulted in luckydonald/docker-telegram-cli
But I really can't remember if that even did run in the end.
I tried docker, but I couldn't get through it. Maybe someone else gets it working with all the login and stuff.


Upstream audio rocm rnnt merge

This PR has changes to merge upstream pytorch audio with rocm_rnnt branch. The changes are tested with DLM rnnt model.
CC: @amathews-amd

The conflicts diff file is at this link:
https://amdcloud-my.sharepoint.com/personal/pnunna_amd_com/_layouts/15/onedrive.aspx?id=%2Fpersonal%2Fpnunna_amd_com%2FDocuments%2FDocuments%2Fconflicts.diff&parent=%2Fpersonal%2Fpnunna_amd_com%2FDocuments%2FDocuments
All changes from HEAD and upstream/main are retained except libtorchaudio_decoder.so related


Upgraded to Python3

Most changes made automatically with Python 3's 2to3 tool.
Had to manually change syntax for importing a api_client module from within a sub-directory.
Had to specify UTF-8 encoding manually on line 92/94.

Hi @speedplane my apologies. Thank you for the resources. I'll start reading up on everything and fix this up.


LAPACKE: fix incorrect value of lda_t in lapacke_?laswp_work

This is a small subset (https://github.com/Reference-LAPACK/lapack/pull/118/commits/07e1fbd89752bed74d35c48e92287d467646a158) of changes committed recently to the Reference LAPACK (https://github.com/Reference-LAPACK/lapack/pull/118). Since I'm not sure whether there is any policy on when and which changes are merged from upstream to OpenBLAS, I'm submitting only changes that prevent our tests (https://github.com/gonum/lapack/pull/219) to pass.

I am not the one to set policy, but I would expect that bugfixes should be merged as soon as possible. In that sense most of Reference-LAPACK/lapack#118 (excepting just the purely cosmetic changes) looks important enough to me. Thanks for flagging this.


Length Convertor Added

Fixes Issue🛠24

Closes #
Description👨💻
I have added a Length Calculator, which converts the length between two specified units.
Type of change📄

[] New feature (non-breaking change which adds functionality)

How this has been tested✅

I have tested each unit and conducted manual conversions, and everything is functioning correctly.
Checklist✅

[] My code follows the style guidelines of this project
[] I have performed a self-review of my own code
[] I have commented my code, particularly in hard-to-understand areas
[] I have added demonstration in the form of GIF/video file
[] I am an Open Source Contributor

Screenshots/GIF📷

Pls mention the issue number this PR closes in Closes #
Delete the .vscode folder and its files.
Add a Readme.md file as well in the calculators folder which includes descripton of the calc, tech stacks used and screenshot of the calculator @JrG-One

Ok i'll do that

Looks nice, Merged!! Congrats on getting your PR merged to this repo under SWOC S4 🎉🎉 @JrG-One

Completed those specified changes

Looks nice, Merged!! Congrats on getting your PR merged to this repo under SWOC S4 🎉🎉 @JrG-One


HTTPResponse strict attribute gone in urllib3>=2.0.0

Operating System version: Debian 11
Firebase SDK version: 6.1.0
Firebase Product: auth
Python version: 3.11.3
Pip version: 23.1.2

Steps to reproduce:
firebase-admin==6.1.0 package requires urllib3<3,>=1.21.1 and so pip installs urllib3==2.0.2, but with urllib3>=2.0.0, which removed attribute 'strict' from 'HTTPResponse' object, firebase_admin.auth.verify_id_token(token) raises an error AttributeError: 'HTTPResponse' object has no attribute 'strict'.
Relevant Code:
import firebase_admin
from firebase_admin import auth
cred = firebase_admin.credentials.Certificate(GOOGLE_APPLICATION_CREDENTIALS)
firebase_admin.initialize_app(cred)
firebase_admin.auth.verify_id_token(token)

...
File "\Lib\site-packages\cachecontrol\serialize.py", line 54, in dumps
    u"strict": response.strict,
               ^^^^^^^^^^^^^^^
AttributeError: 'HTTPResponse' object has no attribute 'strict'

I have forced urllib3==1.26.15 in the meantime.

Thanks, urllib3 to 1.26.15 works well

Any updates on this issue? (We use urllib3<2.0.0) as a workaround)

This seems to have been an issue in the cachecontrol library and was tracked in https://github.com/psf/cachecontrol/issues/292. It was resolved in https://github.com/psf/cachecontrol/pull/301 and released in cachecontrol >= 0.12.13. We'll need to bump the minimum version to match that as well.


Status of Chart.Zoom

Hey, good job for this feature :).
Is the plugin stable/production ready?

@nux17 I don't think this is production ready but if you give it a try and like it I don't see why you couldn't use it.

I'm kind of a javascript noob so I have a dumb question about the setup.  For Chart.js, I can go into samples/ and do 'firefox <sample>.html' and it all works great.  But in a clone of Chart.Zoom.js this just brings up a blank page.  I suspect it's because the repo doesn't contain the paths to ../node_modules/....  I'm guessing I'm just missing a part of the setup.  What's the proper way to get the missing content in ../node_modules/?  Should I copy the Chart/hammer content into node_modules/ manually? Is there supposed to be a submodule configuration I'm unaware of?
Thanks!

@ddj116 run npm install

Thanks buddy :)

Love this feature, but I'm having a problem with resetZoom().  When I call that the chart resets, but then the page also reloads.  Is there a way to prevent that?

@jvolf-zolo please submit bug reports as new issues, and preferably include a codepen or something where we can see and reproduce the issue. Thanks!


How to resolve Error: Magic RPC Error: [-32603] Internal error: Unsupported Magic Connect method?

✅ Prerequisites

[ YES] Did you perform a cursory search of open issues? Is this question already asked elsewhere?
[ YES] Are you reporting to the correct repository (magic-sdk)?

❓ Question: Can I use Magic Link only in Web3 applications, not in my usual Next.js app, which I'm building for my school project? If yes, then why does it show the above mentioned error, i.e. "Unsupported Magic Connect Method" when I use loginWithMagicLink() method?
I am building a Next.js app for my school project and trying to integrate Magic as my password less authentication method. For this I want to use the loginWithMagicLink({email}) method. But I don't know why it shows the error.
I searched in the Docs where it shows that the RPC Error: [-32603] is "Internal JSON-RPC error. These can manifest as different generic issues (i.e.: attempting to access a protected endpoint before the user is logged in)."
I don't understand what's the issue what it's talking about, what is the protected endpoint here? What could be different generic issues? I'm building a very simple app and not yet implemented anything.
Could any one help me with this?
🌎 Environment

magic-sdk
^13.6.0

magic-sdk

Edge

npm

Windows

Hello @And4Web , this error shows up when your Magic Connect app is calling a method that is not supported for Magic Connect apps. The method loginWithMagicLink() is a Magic Auth method, part of the Auth module, and would require the API keys of a Magic Auth app. You can check out the Web API Reference for methods that can be used in a MAgic Auth app. You can tell which app you had made in the Magic developer dashboard, they will have the label  Magic Connect or Magic Auth on them.
The methods you can call in a Magic Connect app are the methods of the Wallet module.


[lnurl]: Request inbound payment channel using lnurl

Added support to request inbound payment channel using lnurl.
Files changed:
bos
commands/constants.json
lnurl/manage_lnurl.js
New File:
lnurl/channel.js
Spec: https://github.com/fiatjaf/lnurl-rfc/blob/luds/02.md
Issue: https://github.com/alexbosworth/balanceofsatoshis/issues/278
Testing: https://lnurl.fiatjaf.com/
There is one issue with testing this, when you make a GET request to the call back URL and it returns a pubkey and socket, looks like Fiatjaf's node is down or something so it fails to add it as a peer but the rest of the flow works. I just tested it skipping the add peer part.

had to rebase this


Added Tests to test for the JSON structure of the deepgram response

Fixed the '.' causing the tests to fail
Checking to make sure that result from the SDK is in the correct JSON format. These should not break unless there is a breaking change from the API itself.

Hey @SaarthShah I just merged a PR with a minor conflict. Hopefully it's easy to resolve though. If you want me to review this PR let me know.


iOS: 1Password from keyboard does not work

When logging in, when the password field has focus, the top row of the keyboard offers password managers, clicking on 1Password brings up 1Password authentication as expected, selecting Rainbow Monkey login in 1Password returns to Rainbow Monkey as expected, but password is not filled in and top row of keyboard (where password managers appear) is now gone.
Have not tried with iCloud password

This will need https://github.com/flutter/flutter/issues/13015


Is this implementation right?

Coming from this issue #1875
I was trying to implement some evaluations using HELM's API (which are not publicly exposed in the documentation properly) in a functional manner. Please let me know if I am doing this right (logically)

from helm.benchmark.augmentations.data_augmenter import DataAugmenter
from helm.benchmark.augmentations.dialect_perturbation import DialectPerturbation
from helm.benchmark.scenarios.scenario import Instance, Input, Reference, Output
from helm.benchmark.metrics.basic_metrics import quasi_exact_match

def quasi_exact_match_perturbation_dialect(
    input_text: Union[List[str], str], model: Model
):
    if isinstance(input_text, str):
        input_text = [input_text]

    data_augmenter = DataAugmenter(
        perturbations=[
            DialectPerturbation(prob=1.0, source_class="SAE", target_class="AAVE")],
    )
    instance_list = []
    # for now I do not know about the references and output
    for text in input_text:
        instance_list.append(Instance(
            id="id0",
            input=Input(text=text),
            references=[Reference(Output(text="Is this love?"), tags=[])]))
    instances = data_augmenter.generate(instance_list, include_original=True)
    instances = [instance.input.text for instance in instances]

    # in instances we have alternate pairs of actual and perturbed
    # so we will take all the pertubed

    perturbed_instances = instances[1::2]
    actual_instances = input_text

    model_generations_actual = model(actual_instances)
    model_generations_perturbed = model(perturbed_instances)

    gen_from_perturbed = [generation['text'] for generation in model_generations_perturbed]
    gen_from_actual = [generation['text'] for generation in model_generations_actual]

    scores = []
    for gold, pred in zip(gen_from_actual, gen_from_perturbed):
        scores.append(quasi_exact_match(gold=gold, pred=pred))
    return scores

Mostly all the fairness-based metrics use some perturbations. In HELM there is a separate class for different kinds of perturbations. So the idea is

Take a string
Perturb that string
Now input both the strings (actual and perturbed)
For the outputs generated for both cases, use different metrics like Quasi Excat match etc.

Am I right in this thought process? Also, I have another question on what references=[Reference(Output(text="Is this love?") does this mean and how to use this in more customized manner.
Some parts of the implementation is been taken from HELM's code base (example: test_perturbations.py)

Overall your script looks reasonable. Some high level comments:

We currently don't document the low-level internal API because it is subject to breaking changes in future versions and we don't want people to rely on them. But you are welcome to use this API, and you can pin the version of HELM to avoid breakages. We may make this API more official / public in the future.
You can look at Runner.run_one() if you want to see our pipeline for instance generation, augmentation, model inference and metric computation.
References are used for two things - to indicate "correct" answers for open ended generations, and to indicate the incorrect and correct options for multiple choice questions.
Your script computes the percentage similarity between the responses to the original input and responses to the perturbed input (because you treat the former as "gold"), which may not be what you intended (the response to the original input may not always be correct).
We compute the fairness metrics differently. Instead of computing a metric based on the difference between (response to original input, response to perturbed input) pairs, we compute accuracy over all the responses to the perturbed input only. This is discussed further in the paper in section 4.6.

It's not too difficult to write a new Metric that does the pairwise comparisons. However, we don't have any existing metrics in the codebase that do this.

That's quite helpful thank you for all the clarification @yifanmai


Fix Bug on Events Configuration Equal function

If Config.Events was nil, it would cause an null pointer exception

[X] Tests included

@harshavardhana  I've addressed it and added more tests for the function itself.


Rethink settings to be more robust

Settings should be

Durable across app updates via the Store
Portable

+1, also see https://github.com/TheJoeFin/Text-Grab/issues/401


Fix incorrect redundant output session finalization sequence

First terminate tasks that may be blocked on the underlying session, then close the session itself.
Fixes #204

Pull Request Test Coverage Report for Build 42165175

0 of 0   changed or added relevant lines in 0 files are covered.
15 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.05%) to 95.653%

Files with Coverage Reduction
New Missed Lines
%

/home/appveyor/projects/pyuavcan/pyuavcan/transport/can/media/socketcan/_socketcan.py
1
78.54%

/home/appveyor/projects/pyuavcan/pyuavcan/transport/can/_session/_input.py
1
98.6%

/home/appveyor/projects/pyuavcan/pyuavcan/transport/loopback/_loopback.py
5
92.62%

/home/appveyor/projects/pyuavcan/pyuavcan/transport/redundant/_session/_output.py
8
93.06%

Totals

Change from base Build 42125464:
-0.05%

Covered Lines:
13660

Relevant Lines:
14139

💛  - Coveralls