Kapacitor-generated Kafka alert messages have incorrect or invalid Kafka timestamp

OS: Centos 7.6
Kapacitor: 1.5.3 (installed from pre-built package)
Kafka 2.12-2.3 (but I've also tried with other versions)

When using the Kafka event handler to send alerts to a Kafka topic, I am able to generate and send alerts successfully to the topic, but something about the metadata in the messages causes them to be deleted by Kafka's log cleaner prior to the retention time. They will be deleted the next time the log.retention.check.interval comes up, with the following message in the logs:
Found deletable segments with base offsets [5] due to retention time 604800000ms breach (kafka.log.Log)
This is logged and the messages are deleted even if they were produced within the last minute.
However, if I have Kapacitor write messages to a log file and then use the Apache kafka-console-producer to add those messages to the kafka topic, they are maintained for the expected retention time period. As far as I can tell, the kafka-console-consumer seems to indicate that the contents of these messages are the same, so there seems to be some issue with the way that Kapacitor itself is setting (or not setting) the timestamp or some other aspect of the metadata when producing its messages.
This is my kafka config section from the kapacitor.conf file:

[[kafka]]
enabled = true
id = "localhost"
brokers = ["localhost:9092"]
timeout = "10s"
batch-size = 100
batch-timeout = "1s"
use-ssl = false
ssl-ca = ""
ssl-cert = ""
ssl-key = ""
insecure-skip-verify = false`

(note that I also changed Kapacitor to run on port 9094, since it would otherwise conflict with the default Kafka port)
And this is my TICKScript:

dbrp "telegraf"."autogen"
// Parameters
var period = 10s
var every = 10s
// Dataframe
var data = stream
|from()
.database('telegraf')
.retentionPolicy('autogen')
.measurement('cpu')
.groupBy('host')
|window()
.period(period)
.every(every)
|mean('usage_idle')
.as('stat')
// Thresholds
var alert = data
|alert()
.id('{{ index .Tags "host"}}/cpu')
.message('{{ .ID }}:{{ index .Fields "stat" }}')
.warn(lambda: "stat" > 95)
// Alert
alert
.kafka()
.cluster('localhost')
.kafkaTopic('kapacitor')`

I'm also running Telegraf 1.11.4 and Influx 1.7.7, both installed from pre-built packages. Other than the changes to Kapacitor.conf noted above, everything in the TICK stack is running with standard configurations.
And just for reference, here are the definitions of the kafka topic I was using:

Topic:kapacitor    PartitionCount:1    ReplicationFactor:1  Configs:segment.bytes=1073741824
Topic: kapacitor    Partition: 0    Leader: 0    Replicas: 0    Isr: 0`

And the log retention policy section of the kafka server.properties file (all default settings):

log.retention.hours=168
#log.retention.bytes=1073741824
log.segment.bytes=1073741824
log.retention.check.interval.ms=300000

With a little bit more investigation, I was able to verify that Kapacitor is generating messages with a 0 value for the Kafka timestamp. I did this using the Kafkacat utility:
Here are the timestamps from two otherwise identical messages, the first one generated using Kapacitor, and the second generated using the apache-console-producer using input from a message that Kapacitor had written to a log file on the system: https://github.com/edenhill/kafkacat

$ kafkacat -b localhost -C -t kapacitor -f 'Timestamp: %T\n' -p 0 -o 2
Timestamp: 0
Timestamp:<PHONE_NUMBER>098

Huge plus one on this, this bit us pretty hard while trying to implement kapacitor in production. It seems to be fixed on master, pending release. Any word on when that may be?

Wasn't this done with this commit? https://github.com/influxdata/kapacitor/commit/5d93d01cf2e4e16edf12989050c8e8f56b5eaa7f
And that was included in the 1.5.4 release, as far as I can tell. Is someone from Influx supposed to to close this ticket, or am I supposed to be the one to close it?


New monitoring brownfield scenario

Overview/Summary
Add monitoring and logging brownfield portal experience
This PR fixes/adds/changes/removes
Adds a new portal experience for Montioring and Logging
Breaking Changes
No breaking changes

@rvandenbedem  request for your review and sign off


[BFCL] Skip Executable Categories When API Keys Missing

This PR improves the behavior of the generation and evaluation pipeline. When executable categories are involved and API keys are not provided in the .env file, instead of throwing an error, the affected categories will now be skipped. This enhancement provides a smoother experience for first-time users.

What will happen to overall score? What would be the difference between score on BFCL official leaderboard vs. without Executable?

If the API Key is not provided, that category will not be evaluated and will be treated as 0 by default in the overall score calculation, which means the overall score (and the one on the leaderboard) will be hurt if the API Keys are not supplied. PR #849 should make things more clear.

What percentage of executable are there?

310 in total, out of 4751 entries.

Code Change LGTM. Please add in PR description:

What will happen to overall score? What would be the difference between score on BFCL official leaderboard vs. without Executable?
What percentage of executable are there?

Updated.


ReturnValues on Update of an entity?!

Hi, how do set the 'ReturnValues on an update of an entity?

found it, it is in the advanced operations.

Same question with Delete. advancedOperations.delete does not return the item.

Hi @obazoud . If you're using advancedOperations you can get the raw / unparsed version of the item - see https://github.com/symphoniacloud/dynamodb-entity-store/blob/main/documentation/AdvancedSingleEntityOperations.md#return-values-for-write-operations - but there is no current functionality in entity store to return the parsed item during a delete.


Simply use the oklog/run SignalHandler

Less code to maintain and using what upstream provides :)

Awesome! Thank you!


Decide whether to retire eval_internal_iterative

Assuming all dice are truncated from the max side, this is used in the case that direction is set to -1 and truncation is present. I guess it could happen for a RISK-like mechanic with step dice? Some variants of actual RISK do have 8-sided dice, though there the number of dice is too small to matter computationally.
The annoying part of keeping this is that the layers could become desynchronized when there are step dice and all of the dice of a certain size get eliminated early, causing a skip.

I'm going to retire it for now. The lack of persistent caching is what tips it over for me.

It looks like the performance can be not so good. I'm going to try restoring this.

Solving layer desync doesn't actually seem too bad, though it turns out it's currently not necessary because we retain dice with zero quantity (other than the final skip when we run out of nonzero count_dice).


Is it possible to set input language with whisper-base.tflite with this code

Here's the thing, I speak Chinese but the result is in English with similar meaning. So I am wondering if it is possible to set the input language so that the output will be in the same language as the input.

@SchweitzerGAO Hello! May I ask you about the language? How did you do to set the input language , so that the output will be in the same language as input?

@YangMing6401 Does apk shared in release directory work on your device?

@vilassn Yes, it can work on my device
And I want to ask you that Is it possible to set input or output languages?


Checking out a remote PR causes an error: "A non-recoverable error occurred during a database lookup"

Extension version: v0.44.0
VSCode Version: 1.68.1 30d9c6cd9483b2cc586687151bcbcd635f373630 x64
OS: Windows 10 Pro Version 21H1 (OS Build 19043.1766)

Steps to Reproduce:

Go to the All Open tab
Right click on a PR and click on Checkout Pull Request

Additional information:

The remote branch is called master, which is the same as one of my local branches.
I'm using SSH to push to the repo, which may or may not be an issue.

Error (in pop-up notification):
fatal: unable to look up git@github.com:PyreticDev (port 9418) (A non-recoverable error occurred during a database lookup. ) . Please check git output for more details

Error (in git output):
[2022-06-27T21:06:22.736Z] > git config --local -l [36ms]
[2022-06-27T21:06:22.785Z] > git status -z -uall [45ms]
[2022-06-27T21:06:22.821Z] > git symbolic-ref --short HEAD [33ms]
[2022-06-27T21:06:22.821Z] fatal: ref HEAD is not a symbolic ref
[2022-06-27T21:06:22.857Z] > git rev-parse HEAD [33ms]
[2022-06-27T21:06:22.898Z] > git remote --verbose [37ms]
[2022-06-27T21:06:22.902Z] > git for-each-ref --sort -committerdate --format %(refname) %(objectname) %(*objectname) [43ms]
[2022-06-27T21:06:22.939Z] > git config --get commit.template [33ms]
[2022-06-27T21:06:22.975Z] > git for-each-ref --format=%(refname)%00%(upstream:short)%00%(objectname)%00%(upstream:track)%00%(upstream:remotename)%00%(upstream:remoteref) refs/heads/pr/PyreticDev/85 refs/remotes/pr/PyreticDev/85 [34ms]
[2022-06-27T21:06:23.015Z] > git remote add PyreticDev7 git://git@github.com:PyreticDev/splitscreenme-www [37ms]
[2022-06-27T21:06:23.065Z] > git status -z -uall [46ms]
[2022-06-27T21:06:23.100Z] > git symbolic-ref --short HEAD [33ms]
[2022-06-27T21:06:23.100Z] fatal: ref HEAD is not a symbolic ref
[2022-06-27T21:06:23.134Z] > git rev-parse HEAD [32ms]
[2022-06-27T21:06:23.171Z] > git remote --verbose [32ms]
[2022-06-27T21:06:23.175Z] > git for-each-ref --sort -committerdate --format %(refname) %(objectname) %(*objectname) [38ms]
[2022-06-27T21:06:23.210Z] > git config --get commit.template [32ms]
[2022-06-27T21:06:23.244Z] > git config --local remote.PyreticDev7.github-pr-remote true [32ms]
[2022-06-27T21:06:23.294Z] > git status -z -uall [48ms]
[2022-06-27T21:06:23.329Z] > git symbolic-ref --short HEAD [33ms]
[2022-06-27T21:06:23.329Z] fatal: ref HEAD is not a symbolic ref
[2022-06-27T21:06:23.364Z] > git rev-parse HEAD [33ms]
[2022-06-27T21:06:23.404Z] > git remote --verbose [34ms]
[2022-06-27T21:06:23.408Z] > git for-each-ref --sort -committerdate --format %(refname) %(objectname) %(*objectname) [41ms]
[2022-06-27T21:06:23.442Z] > git config --get commit.template [31ms]
[2022-06-27T21:06:23.494Z] > git fetch PyreticDev7 master:pr/PyreticDev/85 --depth=1 [48ms]
[2022-06-27T21:06:23.494Z] fatal: unable to look up git@github.com:PyreticDev (port 9418) (A non-recoverable error occurred during a database lookup. )
[2022-06-27T21:06:24.598Z] > git status -z -uall [52ms]
[2022-06-27T21:06:24.633Z] > git symbolic-ref --short HEAD [33ms]
[2022-06-27T21:06:24.633Z] fatal: ref HEAD is not a symbolic ref
[2022-06-27T21:06:24.667Z] > git rev-parse HEAD [32ms]
[2022-06-27T21:06:24.705Z] > git remote --verbose [33ms]
[2022-06-27T21:06:24.711Z] > git for-each-ref --sort -committerdate --format %(refname) %(objectname) %(*objectname) [41ms]
[2022-06-27T21:06:24.744Z] > git config --get commit.template [31ms]

@Snailedlt can you share the output of git remote -v?

Sorry for the late response.
I have not been able to reproduce this behaviour lately. It seemingly got fixed by a newer version of VSCode, or was dependent on some other environment variable.
Anyways, it does not happen for me anymore, so feel free to close the issue if you find it appropriate. I'll make a comment here again if it starts happening again

👍 please do comment if you see again!

The same problem occurred...
what should I do?
$ git fetch
fatal: unable to look up new.url.heregit@github.com:MH-Otsutsuki (port 9418) (A non-recoverable error occurred during a database lookup. )

Thanks for the git fetch output @MH-Otsutsuki. Since the error happens when you run git from the command line that indicates that it's not an issue with the extension and is instead a problem with either your local git configuration or your remote repository.


Rework ShipStats page

collapsed subsections by default
move settings link to regular navigation
move secondary range into main secondary expander
Remove detection from subs at maximum depth from concealment data container
swap positions of concealment and maneuverability datacontainer
consider adding icon for shell type, alternatively: prefix shell name with shell type
share expander state between tabs within one session (enabled by default, maybe optional)
show "add new ship" when removing last ship on ships page (evaluate embedding selector in page vs. opening popup as usual)
check if tabs header can be made sticky

add secondaries max range dispersion


Word warp doesn't work on plain text

If you create a paste that has a very long line (to the point where it goes off screen and you get a scrollbar) and set the options to expire in one hour (I don't think this has any effect, but its what I tested with) and the type to plain text, then the word wrap checkbox doesn't do anything, and theres no error in the console.

Could you post an example? Is this about a very long text or just one ridicolous long word?
The wrap checkbox does indeed not work in plain text mode, its enabled by default.

It was some regex (that I'm not gonna share because it was for a profanity blacklist), but I managed to recreate it with some text by removing all the spaces, see https://dpaste.de/JAGO

I think thats what to expect. Its not ideal but I dont see a proper solution for this. I can't arbitrarily add linebreaks into those words otherwise you couldn't copy/paste it anymore.

Re-opened because I figured that theres a CSS property word-wrap: anywhere; which does what it says. It would work in these situations. Will look into it.

Re-opened

The issue still says its closed for me?

This is fixed.


Fix text wrapping behavior

What does the pull request do?
Modify the conditional expression to determine if a new line is required to display a string.
What is the current behavior?
TextFormatterImpl.SplitTextRuns treats a one-character Run at the end of a line as a character that should be displayed on the next line, even though it is included in the character count.
After the string splitting, this causes character at the end of the first line to be displayed twice : where it should be, but also at the beginning of the following line, overwriting the following character and thus resulting in an incorrect rendering.

↓

example xaml:
<Window xmlns="https://github.com/avaloniaui"
        xmlns:x='http://schemas.microsoft.com/winfx/2006/xaml'
        x:Class="Sandbox.MainWindow">
    <StackPanel>
        <TextBlock Text="0123456789☃0123456789" TextWrapping="Wrap" FontSize="50" />
    </StackPanel>
</Window>

What is the updated/expected behavior with this PR?
Characters will not be overwritten and lines will be split correctly.

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes
None expected.
Obsoletions / Deprecations
None
Fixed issues
None

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


Works very well! [Adding basic web server]

Wow! hotspotd works out of the box (I tried it on Dragonboard 410c). Very nice.
Now a question - what's the right way to add a simple web server, so when a I browse from my phone to any url, a default web page will be shown? (This will be used for the wifi provisioning of the device)

Thanks a lot!
No, I don't think a web-interface can be easily added as the access point works through NAT forwarding using iptables.
It has been a long time since I had a look at the code, though. Once I get some time, I'll review it again and you know if some way to achieve this.

Thanks. Once the device is connected to the AP we just created using hotspotd, why should it be complicated to simply listen to incoming TCP requests, like a regular web server, and answer them?
If I install a simple web server, wouldn't it be easy?

Btw - I just installed Apache (web server) and it worked out of the box, so hotspod + apache can help doing the provisioning screen easily. (The captive portal)

I'm closing this since its a long time, reopen a new case if you are still using hotspotd and have this issue.


API not working on production server

My website only appears to work whenever I'm hosting via localhost but not on actual production servers
The URL for the website is here.

But the thing is, the thing just stops working as soon as it gets to actually getting the stations. Here's what the output is supposed to look like:

And here is what the output on the production server looks like (Notice how it doesn't output "Done!" or the station data):

I know that this page is kinda dead tho

Please post actual errors, not screenshots.


Update antivmdetect.py

Response to Mikael Keri from June 25 2019.
Comments -- embedded at modified points

@oaustin I will cherry pick from the fixes.

Reference: https://www.virtualbox.org/manual/ch09.html#changedmi

The 0.1.9 release contains some parts of what's included in this PR, thank you so much for your effort!


Salesforce source documentation

Description of the change

Salesforce source documentation

Type of documentation

[x] New documentation
[ ] Documentation update
[ ] Hotfix

Notion ticket link

Singer Salesforce extract source

@chaitanyapraveen the doc is ready for your review: https://deploy-preview-768--rudderstack-docs-staging.netlify.app/sources/extract/salesforce-v2/ You can preview the doc here.


feat: lemmas about BitVec.getLsb and bitwise ops (and/or/xor)

Some more lemmas that rewrite toFin applied to a bitwise operation (and/or/xor)

awaiting-review


Not working without ng-repeat

Code:

Only apply lazyload in last img

at ng-if="true" to your img tag. not the nice way, better would be a scope:true in return of the angular lazyImg directive

Thaks for you response.
I inserted the image in object and applyed a ng-repeat and with those it works. It seems that only works with ng-repeat.


Repository doesn't have a quick start section in the README

Is your change request related to a problem? Please describe.
As described in the documentation section of the Library Guidelines, Contributors libraries are expected to have in their README a short summary of the library's purpose, installation instructions, a quick start with a minimal usage example, and links to the documentation and contributing guide.
This library currently doesn't have a quick start in the README. While the test/Examples.purs file exists, it's not in readme form and something simpler should exist as a quick start.
Describe the solution you'd like
The library needs a quick start section after the installation instructions. argonaut-codecs is one example of a library with a quick start.
Additional context
See the Governance repository for more information about requirements in the Contributors organization.

May I work on this?

Go for it!

I'd recommend reading through this pull request for extra context on the philosophy of this library.

Thank you for the advice!!

Oh no, I wrote this in the wrong tab 😅 I meant that for the issue of the same name on the Nullable repository:
https://github.com/purescript-contrib/purescript-nullable/issues/31
That PR is not relevant to this library. Sorry about that!


Add an alternative proc macro to allow documenting arguments in a "postfix" position

Allows this nicely aligned doc format with ///! separating the last valid comment between the last 2 arguments
#[argdocpos]
fn sum_image_rows_pos( /// sum the rows of an image
  image_data: &    [f32], /// the image data in row-major format
  nrows     :       u32 , /// the number of rows in the image
  ncols     :       u32 , /// the number of columns in the image
                          ///! an out buffer into which the resulting
                          /// sums are placed. Must have space
                          /// for exactly `nrows` elements
  sums      : &mut [f32],
) -> Result<(),String> {
    todo!()
}

Closes https://github.com/geo-ant/roxygen/issues/13
Not sure how to block both, but from a couple of checks they seem to fail as is when used together
I've copied some tests, but then the current tests fail for me due to a missing expand command, so can't test the new ones either

First of all, thank you for the work. However, I don't want to merge the PR for the reasons I explained in the linked issue that I had already closed. In short:

This crate is passively maintained, that means I'll fix bona fide bugs, like the one you filed in a different issue. But I don't want to add features. Adding features means adding complexity, which means increased maintenance cost. I don't want to spend a lot of time maintaining this crate. I ask to you to please respect that. That's the most important reason and that's true whether or not a new feature is the next best thing since sliced bread.

Our visions about good Rust documentation don't align. My vision (if you even want to call it that) for this crate is to act as a polyfill: it allows a style of documentation which is rarely needed while using a familiar syntax.

I encourage you to publish your own fork and ideas to the rust community.


Cannot save map file in TrenchBroom while Godot is open

If i try to save my map inside TrenchBroom after i pressed "Build" inside Godot it just says "Cannot open file: ".
Maybe this is related to another script error #42 and therefore does not close the file after generating the map?

Can you post a screenshot of what you're trying to do? Build as in Build Map? Or Build as in the top-right Build button in the Mono editor?
Import issues are common enough in Godot but usually restarting the engine addresses them.

I'm getting the same issue on the latest master branch of Godot, after building the map once inside Godot, Trenchbroom won't let it save anymore:

There aren't any errors showing up in my Godot log, the only thing I can think of is that Qodot is (for whatever reason) failing to close its file handle after building as OP suggests.

Sorry for the late reply, I am building the map inside Godot ("Build" Button of your addon) and after that the file is locked (still opened inside Godot) so i cannot open it for writing (saving) again in TrenchBroom.

Getting this also, a workaround is to reload the project and then save while Godot is still opening, still a bit hacky and tedious though

Edit: after adding a cheeky file.Close(); into MapParser.cs:80, the problem has gone away, but allegedly FileAccess instances are refcounted so this shouldn't be required, probably an engine bug.

Actually, in C# specifically, currently, RefCounted objects are not freed automatically, but checked periodically with a garbage collector.
From Godot docs:
Note: In C#, reference-counted objects will not be freed instantly after they are no longer in use. Instead, garbage collection will run periodically and will free reference-counted objects that are no longer in use. This means that unused ones will linger on for a while before being removed.

same issue I fixed it, plugin forgots to close the file, gonna make merge request

Resolved by #62


How to fine-tune the LLaVA-7b model ?

Question
Hi, thanks on your great work!
 And I get error when  I load these fine-tuned weights.  How to fine-tune the  LLaVA-7b ?  Thanks so much!

OSError: Unable to load weights from pytorch checkpoint file for 'LLaVA-main/checkpoints/llava-7B_new/checkpoint-5/pytorch_model-00003-of-00003.bin' at 'LLaVA-main/checkpoints/llava-7B_new/checkpoint-5/pytorch_model-00003-of-00003.bin'. If you tried to load a PyTorch model from a TF 2.0 checkpoint, please set from_tf=True.
I found that the third model was not saved completely. When saving, it was OOM, but the training did not stop..  Thanks.

Me too!
After I finetune 7B, the model I got is three bin files, but what you release is two bin files. The files I get from finetune are all very large, and the total_size in "pytorch_model.bin.index.json" is<PHONE_NUMBER>8, while what you release is only<PHONE_NUMBER>0.

Hi @Chen-Song, you may notice that the size of your trained model is roughly 2x the size of the released checkpoints.  This is because transformers saves the model weights with float32.  When I release the weights, I convert them to float16 to save storage space / bandwidth.
@yunh-w Can you share the size of your trained model weights with ls -lt like @Chen-Song does?  Thanks.

@haotian-liu What is the process to convert float32 to float16?  I have a 13B fine-tuned model that is 50G.

@codybum You can use this script for compressing the model. Please make sure to set two different paths for the model and delete the fp32 source model after verifying the model is working properly.  Thanks.

How can we fine tune it on custom data, and what's the format of dataset to feed-in ?

@anonymous-atom Here is an example dataset: https://huggingface.co/datasets/liuhaotian/LLaVA-Instruct-150K/blob/main/detail_23k.json
You just need to take your data and make it conform to this set.  You can then use the build scripts, substituting your datasets as the training set.

@yunh-w Hi, what hardware did you use?


If no secure sources, grey it out

Take a look at the icon packs, some to most of them have no secure sources ...
So what if there was a options ... Say ... "Grey out unsecure sources"
And it greys it out, if there are no valid sources, greys out the app entirely ... You can still see it in the list, though ... It should be tied to the risky toggle in settings ( install unsecure )

I love this idea and will definitely include it in the next release.
When implementing, I'll consider if it needs an additional settings option for more user control. Maybe the "Install Unsecure" toggle tie will be enough.

I think it should be an "opt-in" option that is disabled by default, but can be enabled in the settings page. Because some mods are treated as "malware" even though they aren't (false positive).


Text field

Add password entry and textarea

This change is

Apart from the two issues pointed out, I think this is ready to be merged.


GDScript 2.0 : When using Named Enums, accessing invalid members won't be caught while editing

Godot version
v4.0.alpha6.official [e4f0fc50f]
System information
Windows 10
Issue description
If you access an invalid member of a named enum, the editor won't catch the issue, but an error will be thrown at runtime.
If you do the same thing with an unnamed enum, the editor will flag it immediately.
I understand this is likely because named enums are a Dictionary, but it seems to me that it would be an improvement having named enums get flagged in the editor similar to unnamed enums.
Steps to reproduce
See code in repro section
Minimal reproduction project
enum GAME_STATE {WIN, LOSE}
var state1_good = GAME_STATE.WIN
var state1_bad = GAME_STATE.DRAW  #This will throw an error at runtime
enum {STATE_WIN, STATE_LOSE}
var state2_good = STATE_WIN
var state2_bad = STATE_DRAW #This will be flagged in the editor

Related to https://github.com/godotengine/godot/issues/59696.

Added a fix for this one in #69590! :)


Decide how to store habits and their data

What is the most suitable way to store data in Electron is still a mystery to us.
Figure out the database needs of the app and explore possible solutions.

[ ] How are we modelling our data
[ ] What database do we use and why?

DB Design

No relationships among the data so NoSQL
Thinking of storing name and other things we won't query in a json like already done.
Storing habits and their commits in a database
Collections: habits, commits.
habits: title, desc, created_at, type (pos/neg), weight
commits: date: {habit: count, habit: count} *Love my brain for this :dancer: *

@TigerApps ^

LokiJS is looking good. We only need to keep the activity of last month (or some other fixed amount). Other data can be stored in as file or thrown away.
Can even add some compression while storing in file if we go that way.

Yo, can be done later on!


Enhance Error Handling for TENSOR_META in Output Shape Calculation for YOLOv8 Quantization

Pull Request Description:
This PR includes a small update to the code that handles the extraction of output_shape based on the type of the _node.meta[TYPE].
The old code assumes that _node.meta[TENSOR_META] is always present when _node.meta[TYPE] is a list or tuple. This assumption could lead to a KeyError if TENSOR_META is missing from _node.meta.
The new code uses _node.meta.get(TENSOR_META, []) instead, which returns an empty list if TENSOR_META is not found, thereby preventing potential errors and ensuring the code handles such cases more gracefully.
This change is necessary to handle scenarios specific to YOLOv8 quantization, where TENSOR_META may not always be present in _node.meta.
Checklist before requesting a review:

[ ] I set the appropriate labels on the pull request.
[ ] I have added/updated the release note draft (if necessary).
[ ] I have updated the documentation to reflect my changes (if necessary).
[X] All function and files are well documented.
[X] All function and classes have type hints.
[X] There is a licenses in all file.
[X] The function and variable names are informative.
[X] I have checked for code duplications.
[ ] I have added new unittest (if necessary).

Hi @ambitious-octopus , thanks for your PR, can you explain the specific case that reproduce this issue?

Hey @Idan-BenAmi, you can reproduce the error with this script. This script closely resembles the notebook tutorial located at tutorials/notebooks/imx500_notebooks/pytorch/pytorch_yolov8n_for_imx500.ipynb. However, instead of using the custom-modified YOLOv8n, it utilizes the original YOLOv8n model from the Ultralytics package with minor adjustments for compatibility with torch.fx. To install this specific version, you can refer to this pull request or simply install it directly via pip:
pip install git+https://github.com/ultralytics/ultralytics.git@quan

Running the script without this fix results in an error when invoking the mct.core.pytorch_resource_utilization_data function.
Anonymized output
Traceback (most recent call last):
  File "/home/me/repos/ultralytics/test.py", line 3, in <module>
    model.export(format="mct", batch=2)
  File "/home/me/repos/ultralytics/ultralytics/engine/model.py", line 744, in export
    return Exporter(overrides=args, _callbacks=self.callbacks)(model=self.model)
  File "/home/me/miniconda3/envs/ultralytics/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/home/me/repos/ultralytics/ultralytics/engine/exporter.py", line 331, in __call__
    f[12], _ = self.export_mct()
  File "/home/me/repos/ultralytics/ultralytics/engine/exporter.py", line 143, in outer_func
    raise e
  File "/home/me/repos/ultralytics/ultralytics/engine/exporter.py", line 138, in outer_func
    f, model = inner_func(*args, **kwargs)
  File "/home/me/repos/ultralytics/ultralytics/engine/exporter.py", line 1077, in export_mct
    resource_utilization_data = mct.core.pytorch_resource_utilization_data(in_model=self.model,
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/resource_utilization_data_facade.py", line 83, in pytorch_resource_utilization_data
    return compute_resource_utilization_data(in_model,
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/common/mixed_precision/resource_utilization_tools/resource_utilization_data.py", line 64, in compute_resource_utilization_data
    transformed_graph = graph_preparation_runner(in_model,
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/graph_prep_runner.py", line 72, in graph_preparation_runner
    graph = read_model_to_graph(in_model,
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/graph_prep_runner.py", line 207, in read_model_to_graph
    graph = fw_impl.model_reader(in_model,
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/pytorch_implementation.py", line 149, in model_reader
    return model_reader(_module, representative_data_gen, self.to_numpy, self.to_tensor)
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/reader/reader.py", line 154, in model_reader
    graph = build_graph(fx_model, to_numpy)
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/reader/reader.py", line 65, in build_graph
    nodes, inputs, outputs, fx_node_2_graph_node = nodes_builder(model, module_dict, to_numpy)
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/reader/graph_builders.py", line 238, in nodes_builder
    input_shape, output_shape = _extract_input_and_output_shapes(node)
  File "/home/me/repos/model_optimization/model_compression_toolkit/core/pytorch/reader/graph_builders.py", line 151, in _extract_input_and_output_shapes
    output_shape = [list(m.shape) for m in _node.meta[TENSOR_META]]
KeyError: 'tensor_meta'

Hi @ambitious-octopus, I'm trying to reproduce the issue using the branch you sent to me (quan), both via pip (as you mentioned) and also by cloning the ultralytics repository and switching quan branch.
I got a torch FX error (see below), which it seems you resolved already.
Can you advise? am I using the correct branch/version?
Thanks

2024-08-26 13:25:24 idanb-ub20 Model Compression Toolkit[2611824] CRITICAL Error parsing model with torch.fx
fx error: symbolically traced variables cannot be used as inputs to control flow
Traceback (most recent call last):
File "/data/projects/swat/users/idanb/repository/git4/model_optimization/model_compression_toolkit/core/pytorch/reader/reader.py", line 90, in fx_graph_module_generation
symbolic_traced = symbolic_trace(pytorch_model)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/_symbolic_trace.py", line 1193, in symbolic_trace
graph = tracer.trace(root, concrete_args)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/_symbolic_trace.py", line 793, in trace
(self.create_arg(fn(*args)),),
^^^^^^^^^
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/tasks.py", line 0, in wrapper
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/tasks.py", line 129, in forward
return self.predict(x, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/tasks.py", line 147, in predict
return self._predict_once(x, profile, visualize, embed)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/tasks.py", line 168, in _predict_once
x = m(x)  # run
^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/_symbolic_trace.py", line 771, in module_call_wrapper
return self.call_module(mod, forward, args, kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/_symbolic_trace.py", line 495, in call_module
ret_val = forward(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/_symbolic_trace.py", line 764, in forward
return _orig_module_call(mod, *args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1532, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1541, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/modules/head.py", line 60, in forward
y = self._inference(x)
^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/users/idanb/repository/git4/ultralytics/ultralytics/nn/modules/head.py", line 98, in _inference
elif self.dynamic or self.shape != shape:
^^^^^^^^^^^^^^^^^^^
File "/data/projects/swat/envs/idanb/venv_np_1_26/lib/python3.11/site-packages/torch/fx/proxy.py", line 443, in bool
return self.tracer.to_bool(self)
^^^^^^^^^^^^^^^^^^^^^^^^^


Added response on freq request from Gpredict

Minor bug fix and added response to frequency from Gpredict.
The connector should be stable now. (running for more than 48hours without a problem)
I've also compiled the latest Gpredict version for Windows so the log from Gpredict isn't flooded with error messages anymore. See https://github.com/Jansemar/Gpredict_win32

Thanks for contributing! I will add this functionality in the next release!


Fix #10292: Remove DB & Schema Usage Filters

Describe your changes :

Fix #10292: Remove DB & Schema Usage Filters

Type of change :

[x] Bug fix
[x] Improvement

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@open-metadata/ingestion

will this have any impact on the queries that we can actually ingest?

@pmbrull except clickhouse no other db provides this info so without parsing we cannot know if it will be "ingestable"


Add popups with information to required fields in vuln. proposal

Also please note, that we should have following affected projects:

standalone projects
dependencies/libraries


feat(retro): adding reaction-bar

Adding a reaction bar to provide the ability to express Feelings about the sprint.
Solves: #107
Functionality

Client sends WebSocket -> WebSocket sends to all clients -> Client add the Reaction to a list -> List will be shown respectively
Reactions can be turned off completely by the moderator

Boundaries

The List of Reactions will be cleared when no reaction is currently send. In a worst case scenario, this can lead to performance issues, when reactions are send all the time. But atm I don't see that these cases will happen a lot.

Possibilities

In the Future, the Reaction Icons could be changed by the moderator to fit the team, but atm this solution would be sufficient.
The Reactions could be added to the PokerPage, further Refactoring would be needed therefore

Thank you very much for this PR! I hope to be able to review it on this weekend. The reactions shouldn't be needed in the Planning Poker, in my opinion.

@all-contributors please add @sillydomnom for code


Squash migrations and migrate db to utf8mb4

This PR squashes all migrations into a single .sql file, enabling faster migration and a cleaner migrations folder. As we were performing some inserts into the migrations, I relocated them to seeds (base.yml).
This can be merged immediately, but it's advisable to ensure that we won't add new migrations until the release, as it would be preferable to have an empty migrations folder in the release.
Commands:
# For testing
php artisan migrate --database=testing # to have a sqlite db to dump
php artisan schema:dump --database=testing
# then rename testing-schema.sql to memory-schema.sql

# For mysql (use --prune to remove the migrations from the migrations folder)
php artisan schema:dump --prune

Closes #1731

Okay, what's great about the PR regarding internal fields is that it allowed me to test what happens when we re-dump the db after adding a migration. Verdict: it works perfectly; Laravel keeps the old dump and adds the migrations present in the migrations folder to create a single dump.

Important Note Regarding This PR
Firstly, this PR squashes migrations. It is crucial to note that anyone who wants to upgrade to this version must first upgrade to the version immediately before this one and run all the migrations. This is because ALL migrations including those that were never run are now added to the dump, which is executed only when the database is empty. Therefore, it is VERY IMPORTANT that users upgrade to the previous version before upgrading to this one.
Additionally, this update changes the database charset to utf8mb4. As discussed with Nabeel, this will enable the addition of new features in the future, and it is better to make this change now so that all databases created from this release will not need to be converted later.
Before performing this update, it is VERY IMPORTANT to back up your database. Everything is automated and should go smoothly, but you can never be too careful.
I have tested with different databases containing varying amounts of data and did not encounter any issues with the migration. However, I would greatly appreciate feedback from @nabeelio and @FatihKoz . If you could test it locally with some of your databases and let me know if it works well, that would be great. Even though the change is fairly straightforward in theory.
Reference:
https://stackoverflow.com/questions/42704637/laravel-5-4-upgrade-converting-to-utf4mb-from-utf8

How you will provide the previous version ?
What will happen if people do NOT follow that logic ?
What if a user is already using utf8mb4 ?
Earliest i can test will be during weekend, I can replicate a live server
locally which is on latest dev (Turksim, THYVA) and a server is behind like
maybe 1-2 months (SagaVA, not sure when I last updated it)
Sent from mobile device / Mobil cihaz ile gönderildi
10 Tem 2024 Çar 00:40 tarihinde Arthur Parienté @.***>
şunu yazdı:

Important Note Regarding This PR
Firstly, this PR squashes migrations. It is crucial to note that anyone
who wants to upgrade to this version must first upgrade to the version
immediately before this one and run all the migrations. This is because
ALL migrations including those that were never run are now added to the
dump, which is executed only when the database is empty. Therefore, it is VERY
IMPORTANT that users upgrade to the previous version before upgrading to
this one.
Additionally, this update changes the database charset to utf8mb4. As
discussed with Nabeel, this will enable the addition of new features in the
future, and it is better to make this change now so that all databases
created from this release will not need to be converted later.
Before performing this update, it is VERY IMPORTANT to back up your
database. Everything is automated and should go smoothly, but you can never
be too careful.
I have tested with different databases containing varying amounts of data
and did not encounter any issues with the migration. However, I would
greatly appreciate feedback from @nabeelio https://github.com/nabeelio
and @FatihKoz https://github.com/FatihKoz . If you could test it
locally with some of your databases and let me know if it works well, that
would be great. Even though the change is fairly straightforward in theory.
Reference:
https://stackoverflow.com/questions/42704637/laravel-5-4-upgrade-converting-to-utf4mb-from-utf8
—
Reply to this email directly, view it on GitHub
https://github.com/nabeelio/phpvms/pull/1804#issuecomment-2218776461,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARXKVMJE4V5YG7QPHCH4AK3ZLRKD3AVCNFSM6AAAAABHS6R37KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJYG43TMNBWGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

By previous version, I mean having run all the latest migrations that were merged into the dev branch before this PR. I was thinking about the zip on Discord. What I mean is that you shouldn't catch up on 15 versions at once by applying this PR, and the thing is, I don't have much of an alternative for that...
Our last migration was just last week... So basically, if they are on the current dev and move to this dev, there won't be any problems. If people don't do this, they will end up missing migrations...
I'd suggest maybe posting a message on Discord, clearly explaining that they need to upgrade to the latest in two steps if people are catching up from a long time ago.
Regarding what happens if the table is already in utf8mb4, there is a check at the beginning of the migration that performs the update. It will not attempt the update if it's already utf8mb4 :)

Everything worked fine here on my production data (in test env), there are no problems.

So after discussing it with @nabeelio, the final solution we decided on is to keep the migrations in addition to the squash for now (which solves the issue of catching up many updates at once), and we will remove them at the beginning of version 7.1 (to allow compatibility with Laravel 11).


DoubleTab and zoom

Hi
i have a large image .
so can u tell me how can i zoom in custom point (increase scale) with double tab?
_doubleTapGesture = new TapGesture(_container);
_doubleTapGesture.numTapsRequired = 2;
_doubleTapGesture.addEventListener(GestureEvent.GESTURE_RECOGNIZED, onGestureTap);
_zoom = new TransformGesture(_container);
private function onGestureTap(e:GestureEvent):void
{
var matrix:Matrix = _container.transformationMatrix;
var gestureTab:TapGesture = e.target as TapGesture;
		if (gestureTab == _doubleTapGesture)
		{
			???????????
		}

	}

Gesture#location will give you point (in global coordinate space) of your tap, so you can scale it around that point:
scaleAroundGlobalPoint(_container, gesture.location, /*scale:*/ 2);

function scaleAroundGlobalPoint(obj:DisplayObject, point:Point, scale:Number):void
{
    const matrix:Matrix = obj.transform.matrix;
    const transformPoint:Point = matrix.transformPoint(obj.globalToLocal(point));
    matrix.translate(-transformPoint.x, -transformPoint.y);
    matrix.scale(scale, scale);
    matrix.translate(transformPoint.x, transformPoint.y);

    obj.transform.matrix = matrix;
}

Great, thank you for the response.
its perfect but how can i tween it?
I'm using TweenMax but when i write this code my object doesn't scale!
TweenMax.to(obj, 0.25, {transformationMatrix: matrix, ease: Linear.easeOut});

http://lmgtfy.com/?q=tweenmax+scale+around+point
2nd link
._.


How to sent empty parameters

I'm having trouble with a site I need pass empty arguments on the query. example:
http://thehost.com?Subject=Hello+World&submit=
i trying to make

$client = new Client();
$response = $client->get("http://thehost.com", ['query' => ['Subject' => 'Hello World', 'submit => null']]);

but guzzle trim the submit parameter, how can solve this issue?

There are two different ways to do this:

Set a query value to an empty string (e.g., '') if you want the trailing "=". (i.e., a=)
Set a query value to null if do not want the trailing "=" (i.e., a).

<?php
require 'vendor/autoload.php';
use GuzzleHttp\Client;
$client  = new Client();

$request = $client->createRequest('GET', 'http://httpbin.org/get', [
    'query' => [
        'foo' => 'bar',
        'baz' => '', // sends trailing =
        'qux' => null // no trailing =
    ]
]);

echo $request->getUrl();

Outputs: http://httpbin.org/get?foo=bar&baz=&qux

thanks @mtdowling

And what about POST request? It should be always with trailing =.

@mtdowling  When I set 'qux' => null, the qux param is not added at all. Has something changed during time? :) Thanks!

I had the same issue as @simPod said. Can we add some docs for this behavior?

Same issue... Any idea ? :)

Having the same issue right now
'query' => [
                'input' => $search,
                'location' => $this->eventParameter['location'],
                'radius' => $this->eventParameter['radius'],
                'language' => $this->requestStack->getCurrentRequest()->getLocale(),
                'key' => $this->apiKey,
                'strictbounds' => NULL
            ]

The parameter strictbounds does not affect the result of the request.

Hi, is there any work around with this issue?

It seems, you have to build the query string manually (in guzzlehttp/guzzle since at least version 6.5.5), e.g.
<?php

require_once dirname(__FILE__) . '/../bootstrap.php';

use GuzzleHttp\Client;

$query = join('&', [
  http_build_query(
    [
      'foo' => 'bar',
      'baz' => '', // sends trailing =
    ],
    null, '&', PHP_QUERY_RFC3986
  ),
  'qux', // no trailing =
]);
echo $query . PHP_EOL; // foo=bar&baz=&qux

$client = new Client();
$response = $client->get("https://httpbin.org/get", ['query' => $query]);
echo $response->getBody();

since AFAIK neither Guzzle nor http_build_query support query parameters without value and no trailing =.
@mtdowling

Has something changed during time?


7pm d basic google search

In this pull request, we enabled the user to perform a basic google search using a global search token and added an implementation of the amount of remaining searches.  Admin users also have a table available to them with user information.  This pull request closes #36, #39, and #42.
Users can do a Google Search:

Admins can list all of the users that have done Google Searches:

When this is merged, this will be 100 points.


Advanced frame innerBackground binding not working

Hi seems the binding is not working for the inner background property of the advanced frame, i'm creating a binding between a custom ui component and my view model to update the inner background color of the advanced frame in a property changed delegate. When however i change the backgroundcolor property of the advanced frame I get the updated color change as expected, but not currently for the innerbackground color.

Resolved issue in my own code


Release 1.4.4

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Looks like the windows build is trying to pull down grep-3.0-2-x86_64.pkg.tar.xz but that doesn't exist on the mirrors, only these versions do:
grep-1~3.0-3-x86_64.pkg.tar.zst                    17-Oct-2021 18:51    243K
grep-1~3.0-5-x86_64.pkg.tar.zst                    22-Oct-2022 18:27    244K
grep-1~3.0-6-x86_64.pkg.tar.zst                    15-Nov-2022 17:32    244K
grep-3.6-1-x86_64.pkg.tar.zst                      17-Jul-2021 16:01    237K

Resolves https://github.com/mamba-org/mamba/issues/2536

Seems like a vcpkg bug - not realising versions and compression schemes have changed. Maybe relevant:
error: building libiconv:x64-windows-static failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with `git pull` and `vcpkg update`.

Vcpkg inside of a conda package is awful. Please look at some of the changes we made in #116 to always get the most recent version of vcpkg. This should resolve it.

This is not a vcpkg bug but simply the conda Forge version of vcpkg not being up to date and therefore not having the most recent mirrors.

Alright, then let's do that!

Thanks @mikemhenry! Closing in favor of up to date #127


Add Apple Silicon compatibility disclosure.

The Metal backend support in non Apple Silicon devices is limited. This should be made more clear in the README documentation.
Suggestion from Twitter.

Added further clarification about not supporting Metal on Intel-based Macs on the README.

This will be released on the v1.0.0-alpha3 version.


Ubuntu CI test only

PR for the purpose of debugging PR #905 CI tests.

Just to be clear, I'm primarily using this PR as a vehicle to identify the cause of the test case failure, since it only seems to be a problem on the CI system.


Change interactive window to also wrap

Reported on StackOverflow here:
https://stackoverflow.com/questions/54743688/using-visual-studio-code-python-extension-with-jupyter-interactive-window-on-a-r
We are not supporting the retina figure format correctly. We are getting the 2x res image but also displaying it at 2x size it appears.

change interactive window to be wrap.

Validated. Example from the top now wraps into the space available instead of pushing out a h-scroll. Also handled screen size changes and shrinking / growing vscode window.


v1.3.0 not published in NPM

I can see 1.3.0 tagged and released in github at https://github.com/tessel/node-usb/releases/tag/1.3.0 , but https://www.npmjs.com/package/usb displays 1.2.0 as the latest available.

Using git dependency is not reliable with npm 5 for now. It'd be better to have it published in npmjs.
npm/npm#16907
npm/npm#16898

@tcr any news here, please?

bump. I think version 1.3.0 might fix my problem.

Published 1.4.0


If metadata flag is set complain if there is none

It is fair to assume that if the flag is set, we are expecting metadata in the input file and should complain and abort if it isn't present.

Added in https://github.com/simonpoole/mapsplit/commit/d5d7eefd06b66642682e0236a0827b2285feeaaf


Allow invalid url

This PR needs some explanation.
allow_invalid_url option
I wanted to manage attachment like wordpress does it so I set url doing so :
'url' => '/uploads/attachments/' . date('Y') . '/' . date('m') . '/:style.:filename',

I got an exception cause this url does not contain any :id / :hash so I chose to add a new option for developper willing to take the risk to break things allow_invalid_url
FileSystem Storage Issue
Your filesystem storage emptyDirectory on file removal. This is a problem when you plan to have multiple images in the same folder so I tweaked it a bit. My method works like this :

Delete the image
Check if the directory containing the image is empty
If it is empty remove the directory and check if the parent directory is empty too (recursively deleting every parent empty directory)

Do not hesitate to tell me if something is wrong with my ideas I can still do some edits.

I like this idea. I set the secure_hash as a property on my table on saving event, and want to use this as a key for accessing the file to keep the URL unique/secret. The current implementation forces me to use the :id or :hash where these won't actually be necessary in my implementation, and it either bloats the URL or exposes database IDs.


Adding link to the Logo of navbar to get back to the home page

Assign this to me under gssoc

@itsmohit097 I would love to solve this issue, please assign me this issue


sort JSON plans

It would be convenient if the JSON plan files generated by terraformer plan were more stable in their output. I did two runs, one with -z global and one with -z us-west1 (this is on GCP), which came up with nearly-identical content (disks and instances were null in the former), but it was really hard to tell, since many of the lists and objects were ordered arbitrarily. The ordering isn't stable even across two runs in the same zone.
Obviously there could be some intention to the ordering, but I think that at least with GCP, the places where ordering counts are handled by having key names with indices in them (foo.0.key, foo.1.key, etc).
The places where I saw it was in IgnoreKeys (lists of strings), Item.access (lists of objects related to roles), Item.role_entity (lists of strings), and service account scopes (lists of strings).

Definitely desirable as we want to put our reverse engineered files into git for tracking purposes.


Short solution needed: "How to handle errors in Rust" (rust)

Please help us write most modern and shortest code solution for this issue:
How to handle errors in Rust (technology: rust)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

Example of anyhow error usage in Rust

How to get error as string in Rust

How to assert errors in Rust

How to assert error type in Rust

Using alloc_error_handler in Rust

How to get error backtrace in Rust

How to use boxed errors in Rust

Using error chain in Rust

How to define error in Rust

How to catch error in Rust

How to derive error in Rust

How to downcast an error in Rust

How to display error in Rust

An example of error in Rust

How to format error in Rust

How to get error message in Rust

How to get error line number in Rust

Rust error message example

How to match multiple error types in Rust

How to match error type in Rust

New error in Rust

How to print error in Rust

Using question mark errors in Rust

How to return error in Rust

How to return error from main in Rust

Error with custom struct in Rust

How to get error stack trace in Rust

How to define error type in Rust

How to throw error in Rust

How to unwrap error in Rust

How to wrap error in Rust

Using error with context in Rust


Feat:Store Newsletter Email and Send Thank You Email

Describe the feature
Implement functionality to store the email address of a user who subscribes to the newsletter in the database, and send a thank you email to the registered email address upon successful registration.
-Create an endpoint to accept the email address and store it in the database.
-After successful registration, trigger an email to the registered address.
-The email should thank the user for subscribing to the newsletter.
I will use nodemailer to send the mail to registered email
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a gssoc-24-extd contributor
[X] I want to work on this issue

@RamakrushnaBiswal can you assign me this issue

@17arindam good idea you can work upon it

solved via #174


Issue #357: How to easily test flask API?

Fixes #357 by adding basic test for high coverage of api/server.py
What does this implement/fix? Explain your changes.

Creates basic flask client
Tests the creation of client
Tests the "/run" route
Tests the "/stop" route

Thank you!


External Community Call - 29th January 2025

Sign up: https://msit.events.teams.microsoft.com/event/6def41cd-16c0-4b44-9612-0faebc2b90a8@72f988bf-86f1-41af-91ab-2d7cd011db47
Please leave comments below on topics you'd like covered

Hey @cjtous1, this should be coming in January now, we are just finish docs for the release. Stay tuned and join the next community call.


In which winston decides he does not have to follow the laws of time and space

Description
Winston was wrong.
Fixes # (issue)
#250
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Confirmed Winston is now following the laws of time and space.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have updated changelog.md with changes made

Hooray Winston..

Well that's a fun one... I guess when minute = 0, MM defaults back to month?

I guess so, SS looks like it's milliseconds was counting up to 100 then resetting


Improve REPL to allow for clients that always execute from a fresh state

normally, bb automatically contains the fun calls that were executed

BUT for clients that "always execute from the beginning", client needs a way to tell REPL: "please reset blackboard" - as if no funs were generated.

could be a return from executeEnd():

ExecutionResult.MarkNewFunctionsAsPrevious // the behaviour for most clients
ExecutionResult.KeepNewFunctionsAsNew // the behaviour for "always executes from the beginning" clients
.

OR:

pass bbAccessor in to executEnd()

So REPL client can decide.

done


Add support for Message Queue java clients

Adding message queue support requires a new type of node handling. While it is similar to how nodes are generated from ServiceTypeProperty.TERMINAL and ServiceTypeCategory.RPC typed span events, a message queue node differs from the former in that transactions may continue after the generated node, and from the latter in that the target node comes one depth after the generated node.
Thus in order to handle this, a new ServiceTypeProperty will be introduced named QUEUE.
Message queue plugins that defines a ServiceType with the QUEUE property should meet the following conditions:

Producer span event must be able to provide enough information to generate the target queue node
Consumer span must be able to have enough information to generate it's source queue node
The source/target queue node from above must be identical if they are part of the same transaction

In order to satisfy above, we should set the destinationId of the producer's span event, and the acceptorHost of the consumer's span to the same value - the queue name being a strong possibility. Other attributes such as endPoint or remoteAddress should remain the same as other RPC client plugins.
There should also be a way for the producer to send trace headers over the wire to the consumer (similar to trace headers injected into HTTP header for HTTP clients).
Once the message queue client plugin is set, the rest is up to the collector/web. There are 2 main areas to look at:

Server map (constructed using call statistics)
Transaction view (constructed using actual trace data)

Current plan is to generate extra call statistics for the virtual queue node at the collector level for the main server map, and to add extra logic to handle queues for the trasaction view.
Current limitations

Our trace data model currently does not support single producer / multiple consumer messages (such as when broadcasting messages). These will show up correctly in the server map, but the call stack will only show a single consumer span. Multiple producer / single consumer (batch processing?) messages will manifest themselves similarly only reversed. These issues must be addressed with the planned Trace table overhaul.

👍


Allow completely disabling/hiding default templates

Context

Summary
Give developers the ability to disable default templates, by adding a field to the editor / dashboard settings object.
Relevant Technical Choices

To-do

User-facing changes

Before
After

Testing Instructions

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[x] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #10124

I think we can put some notes in the docs/ folder in a follow-up PR 👍
Would also be a good opportunity to mention the similar flag to disable the media3p library.

The should not render default templates e2e test now needs updating it seems


[LiveComponent] Force POST method when model is changed

Hello,
I've built my first LiveComponent and it works well. But I've noticed that backend requests are sent as GET with all props being appended as URL parameters. As my component includes sensitive data, I want to change the request method to POST. According to the code this could be achieved by adding an action (see https://github.com/symfony/ux/blob/d3ce11f105579c95cab72d5967ff7875c9e3da6b/src/LiveComponent/assets/src/Backend/RequestBuilder.ts#L38). But when I tried to add an data-action="live#update" attribute to my form element, I got the following error:
Error: Since LiveComponents 2.3, you no longer need data-action="live#update" on form elements. Found on element: <select data-model="mySelect" data-action="live#update">
What's the correct way to force POST for backend requests?
    <select data-model="mySelect">
        <!-- options...  should be sent as POST requests on change -->
    </select>

Clever solution :). You're the first person to ask about this. We could add an option on AsLiveClomponent for this - e.g. AsLiveComponent(forcePost: true).

I could try to implement this.
But I'm curious, what is the reason to send the requests using GET by default?

But I'm curious, what is the reason to send the requests using GET by default?

Theoretical HTTP caching :). And also, it just seems more correct - you have a repeatable URL that you can use to render in different states.


k8s Deployment

can anyone help me write suitable deployment?

Sure, just run:

Did you reed the documentation?

Another good implementation of it would be:

or:

If that fits your workflow better.

Why on earth would you use the first examples? That type of no code is getting really aged, not to mention the security implications.


first draft on how to support MirageOS4

this is only a draft, it compiles but haven't tested it yet.
also, the rebuild command is not there...

still open: opam-graph support (embed dependencies of duniverse stuff as well in the opam file)

squash-merged manually


Add Rage Module and start work on blitz

Rage works fine.
Started work on blitz - the fundamentals and logic works, the messages need to be tided up and the scoreboard needs to be implemented as well as the win condition.

We'd planned to make the life counter using LazyMetadataValues, do you think you could switch it to that?

If you need a reference, the KillStreak module uses them.

Working on it as well as completing the blitz module

The map system works fine for blitz, it wasn't how I would have done it but if it works fine I don't see a reason why it is "wrong."

How is it redundant? @twizmwazin

I have already re-written it with the LazyyMetaData, I was unaware of it before, I prefer it to the hasmap

getChild returns any elements that have the name specified. Just check if != null.

I did that before but It would not find the rage tag

Or, sorry, getChildren does that. getChild just returns the first element I believe.

How about get children("rage").size() > 0?

@CaptainElliott The redundant line was getting an element (or null) named "rage," and then if it was named rage do
@iEli2tyree011 Either getChildren("rage") in a for, or if getChildren("rage").size > 0 would be possibly safer, duplicate modules here shouldn't be an issue though.

Ready to merge the TODOs still need to be done