SNOW-1506546: Add support for INCLUDE_METADATA copy option for df.copy_into_table()

I'm using Snowpark Python 1.16.0.
I'd like to add metadata columns to my COPY statement using the new include_metadata copy option,   but there's no way to pass in the value in such a way that it doesn't get quoted:
        my_df.copy_into_table(
            "raw_table",
            include_metadata='(FILE_SCAN_TIME=METADATA$START_SCAN_TIME)')

I think it comes down to this code in AnalyzerUtils:
def convert_value_to_sql_option(value: Optional[Union[str, bool, int, float]]) -> str:
    if isinstance(value, str):
        if len(value) > 1 and is_single_quoted(value):
            return value
        else:
            value = value.replace(
                "'", "''"
            )  # escape single quotes before adding a pair of quotes
            return f"'{value}'"
    else:
        return str(value)

def get_options_statement(options: Dict[str, Any]) -> str:
    return (
        SPACE
        + SPACE.join(
            f"{k}{EQUALS}{convert_value_to_sql_option(v)}"
            for k, v in options.items()
            if v is not None
        )
        + SPACE
    )

And convert_value_to_sql_option doesn't support a list of key values pairs.

Actually there is a bigger issue.    I was able to bypass the issue with string quoting like this:
     my_df.copy_into_table(
            "raw_table",
            include_metadata=RawSqlExpression('(FILE_SCAN_TIME=METADATA$START_SCAN_TIME)'))

Using this custom class:
class RawSqlExpression:
    def __init__(self, expression):
        self.expression = expression

    def __str__(self):
        return self.expression

But then the SQL fails:
SQL compilation error: include_metadata is not supported with copy transform
Is it possible to generate a COPY statement in Snowpark that does not explicitly list columns,   and instead relies on schema evolution?  That may be required for INCLUDE_METADATA.

Hello @sfc-gh-kgaputis ,
Thanks for raising the issue.
At present INCLUDE_METADATA with a copy transform is not supported and thats why its not working with Snowpark as well
Example
copy into  (c1) from @JSON_STAGE
include_metadata = (
filename           = METADATA$FILENAME,
file_row_number    = METADATA$FILE_ROW_NUMBER,
relative_path      = METADATA$RELATIVE_PATH
);
The purpose of the copy transform is to transform while loading, so it expects the column name which you wants to transform, so you have to explicitly list columns with copy transform.
I hope its clear now.
Regards,
Sujan

Hello @sfc-gh-kgaputis ,
Thanks for raising this issue.
Include_metadata is supported for copy into table command but its not supported for copy transform because include_metadata has to be used with The MATCH_BY_COLUMN_NAME and copy transform doesn't support it.
https://docs.snowflake.com/en/release-notes/2024/8_17#new-copy-option-include-metadata
The purpose of copy transform is to transform any column while loading, so it expects explicit column names, so we can't skip column names in copy transform.
Regards,
Sujan

Hello @sfc-gh-kgaputis ,
Will check and update further if we have any plans to support it.
Regards,
Sujan

Hello @sfc-gh-kgaputis ,
After discussion with the team:
As mentioned in previous update: The INCLUDE_METADATA is only for COPY INTO TABLE statement, its not for COPY TRANSFORMATION and the error message you hitting is because the syntax you using eventually landing with COPY TRANSFORMATION internally and thats why its throwing expected error
SQL compilation error: include_metadata is not supported with copy transform
Example
Jypyter UI:
copied_into_result=df.copy_into_table(
"copied_into_table",
target_columns=target_column_names,
force=True,
include_metadata=RawSqlExpression('(FILENAME=METADATA$FILENAME,FILE_LAST_MODIFIED=METADATA$FILE_LAST_MODIFIED,FILE_SCAN_TIME=METADATA$START_SCAN_TIME)')
)
Query history:
COPY  INTO copied_into_table(“SEQ”, “FIRST_NAMELAST_NAME”, “FILE_LAST_MODIFIED”, “FILE_SCAN_TIME”, “FILENAME”) FROM @mystagetest/File1.csv FILE_FORMAT  = ( TYPE  = CSV  FIELD_DELIMITER = ‘,’ SKIP_HEADER = 1  )  force = True include_metadata = ‘(FILENAME=METADATA$FILENAME,FILE_LAST_MODIFIED=METADATA$FILE_LAST_MODIFIED,FILE_SCAN_TIME=METADATA$START_SCAN_TIME)’
At present, it's not supported, I will take this as a feature request.
Regards,
Sujan

Hello @sfc-gh-kgaputis ,
You can try the following code to get the metadata. Its working fine and fetching all metadata
`from snowflake.snowpark.column import METADATA_FILENAME, METADATA_FILE_ROW_NUMBER, METADATA_FILE_LAST_MODIFIED, METADATA_START_SCAN_TIME
schema_for_data_file = StructType([
StructField("seq", IntegerType()),
StructField("last_name", StringType()),
StructField("first_name", StringType()),
StructField("FILE_LAST_MODIFIED", TimestampType(), nullable=True),
StructField("FILE_SCAN_TIME", TimestampType(), nullable=True),
StructField("FILENAME", StringType(), nullable=True)
])
Read the CSV file into a DataFrame
df = session.read.schema(schema_for_data_file)
.option("field_delimiter", ",")
.option("SKIP_HEADER", 1)
<EMAIL_ADDRESS>#df.show()
target_column_names = ["seq","first_name" "last_name","FILE_LAST_MODIFIED","FILE_SCAN_TIME","FILENAME"]
Drop the table if it exists
session.sql("DROP TABLE IF EXISTS copied_into_table").collect()
user_schema = StructType([
StructField("SEQ", StringType()), StructField("FIRST_NAMELAST_NAME", StringType())])
df = session.read.with_metadata(METADATA_FILENAME, METADATA_START_SCAN_TIME.as_("SCAN_TIME"), METADATA_FILE_LAST_MODIFIED.as_("FILE_LAST_MODIFIED"), METADATA_FILE_ROW_NUMBER.as_("ROW<EMAIL_ADDRESS>df.show()`
Output
`|"METADATA$FILENAME"  |"SCAN_TIME"                       |"FILE_LAST_MODIFIED"  |"ROW NUMBER"  |"SEQ"  |"FIRST_NAMELAST_NAME"  |
|File1.csv            |2024-07-10 15:34:49.941119+00:00  |2024-07-09 06:53:53   |1             |seq    | first_name            |
|File1.csv            |2024-07-10 15:34:49.941119+00:00  |2024-07-09 06:53:53   |2             |1      |AAAAAAA                |
|File1.csv            |2024-07-10 15:34:49.941119+00:00  |2024-07-09 06:53:53   |3             |2      |AAABBB                 |
|File1.csv            |2024-07-10 15:34:49.941119+00:00  |2024-07-09 06:53:53   |4             |3      |newuser                |
|File1.csv            |2024-07-10 15:34:49.941119+00:00  |2024-07-09 06:53:53   |5             |4      |forthuser              |
`
Regards,
Sujan

I am running into a similar issue:
`df = snowpark_session.read.option("PARSE_HEADER", True).option("error_on_column_count_mismatch", False).options({ "FIELD_OPTIONALLY_ENCLOSED_BY":'\042'<EMAIL_ADDRESS>df.copy_into_table('DEV_CORE.RAW.BROADCAST_UNP', force=True,  MATCH_BY_COLUMN_NAME='CASE_INSENSITIVE', include_metadata='(CHECKSUM=METADATA$FILE_CONTENT_KEY)')`
Without the '' around (CHECKSUM=METADATA$FILE_CONTENT_KEY)') the code does not compile, and with the quotes, SNowflake gives following:
COPY  INTO DEV_CORE.RAW.BROADCAST_UNP FROM @DEV_CORE.ADMIN.S3_DEV_☺☺☺☺☺ FILE_FORMAT  = ( TYPE  = CSV  PARSE_HEADER = True ERROR_ON_COLUMN_COUNT_MISMATCH = False FIELD_OPTIONALLY_ENCLOSED_BY = '"'  )  force = True MATCH_BY_COLUMN_NAME = 'CASE_INSENSITIVE' include_metadata = '(CHECKSUM=METADATA$FILE_CONTENT_KEY)'
And the following error message: SQL compilation error: invalid value ''(CHECKSUM=METADATA$FILE_CONTENT_KEY)'' for property 'INCLUDE_METADATA'
For reference following SQL, when executed from Snowflake directly works:
......ce = True MATCH_BY_COLUMN_NAME = 'CASE_INSENSITIVE' include_metadata = (CHECKSUM='METADATA$FILE_CONTENT_KEY')
Any thoughts on how do I go around this ? Thanks

Got redirected from support and been thinking on how making it work
Somehow this
.with_metadata(METADATA_FILENAME.alias("FILENAME"))
works when printing the dataframe - but when using this within the copy into - the transformation is not being sent to the query (can have a look on the Activity Monitoring)
Plus depending on where this fields is located, it can mess with the table structure (my filename is at the end for example)
I came with this solution
result = raw_files.copy_into_table(destination_table,pattern=rf'.*/REDACTED',target_columns=source_tbl.columns,transformations=[*[f'${col}' for col in range(1,len(source_tbl.columns))], 'METADATA$FILENAME'],**copy_options)

with raw_files being my DataframeReader
Only tweaking the transformation argument helped me to achieve adding the filename in my copy into processes


AOT build fail if there's no semicolon at the end of ViewChild() declaration

OS?

Windows 10

Versions.

beta.19-3

Repro steps.
Doesn't work
@ViewChild('some') some
Works
@ViewChild('some') some;
The log given by the failure.

not useful

Mention any other details that might be useful.
Not using semicolon at end of declaration for @ViewChild causes AOT build to fail.
Found this information on stack-overflow after hours...
Can something be done about that please ?
For those of us who don't use semicolon.

@thibaultsavary I doubt this is a cli issue. it may be a angular compiler issue. @filipesilva I seem to recall this issue popping up before, was it the aot compiler?

Yeah, I guess it make sense that the problem may be coming from the AOT compiler itself. It may not have ASI built-in, it also can be some kind of manipulation of the code by the cli before compilation. Honestly absolutely no idea of the internals of the build chain.
I'll wait an answer from someone in the cli team, then if it's confirmed that this is a problem from the compiler I'll close this issue and open a new one on the angular repository.

I don't know what caused this originally, but I can't repro it in the latest CLI nor outside of it. I suspect it was a problem in a older version of ngc that has been fixed since.
This is the component I used to test:
import { Component, ViewChild } from '@angular/core';

@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  @ViewChild('some') some: any
  title = 'app works!';
}


"Mondelez" company - scraper doesn't work

URL: https://admin.peviitor.ro/
Browser: Chrome
Device: Laptop
OS: Windows 11
STEPS TO REPRODUCE:

Open the URL in the browser.
Click on "Scraperi".
Click on "scrapers.js".
Enter "Mondelez" in the search bar.
Select "mondelez.js".
Click on "Ruleaza".

EXPECTED RESULTS:
The scraper should run without issues.
ACTUAL RESULTS:
An error occurs.

fixed

Fixed!


New Package: OpenTX.OpenTXCompanion version 2.3.15

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113355&drop=dogfoodAlpha

The download links are provided at the bottom of the release notes.
I also put a link to the release notes in the manifest.


innerJoin not working, throwing not allowed

I need to do a simple join to populate the datatables table. If I would write a query in my repository, I would write this:
public function findAllBySupplier($supplierId)
    {
        return $this->createQueryBuilder('t')
            ->innerJoin('t.stocks', 's')
            ->andWhere('t.isDeleted = 0')
            ->andWhere('s.supplier = :supplier_id')
            ->setParameter('supplier_id', $supplierId)
            ->getQuery()
            ->getResult();
    }

Doing this in the custom query part, it fails on Iterate with fetch join in class App\Entity\Stock using association tyre not allowed. Probably the same issue as in this open issue: https://github.com/omines/datatables-bundle/issues/30
I understand that the join might result in multiple rows, but I don't care about it. And if I would, I could allways use a group by.  How would I be able to get this working, I need the join on s.supplier_id.
Tyre.stocks is OneToMany relation
@ORM\OneToMany(targetEntity="App\Entity\Stock", mappedBy="tyre")

I understand that the join might result in multiple rows, but I don't care about it.

But that is the point why it's not allowed by default, it's an open invitation to get weird bugs and crashes. I think best route here would be to provide a custom adapter implementation providing the necessary transformations.
A demonstration of what you can achieve can be found in the tests at https://github.com/omines/datatables-bundle/blob/master/tests/Fixtures/AppBundle/DataTable/Adapter/CustomORMAdapter.php


Decide how to handle JSON serialization errors

The following code currently panics because JSON is not able to represent maps with non-string keys.
extern crate reqwest;

use std::collections::BTreeMap;

fn main() {
    let mut map = BTreeMap::new();
    map.insert(vec![true], "");

    let client = reqwest::Client::new().unwrap();
    client.post("").json(&map); // "serde to_vec cannot fail"
}

If the method is changed to return a Result, it will need to be documented with an "Errors" section like #111.
If the method continues to panic on invalid input, it will need to be documented with a "Panics" section according to this guideline.

I suppose I naively assumed JSON can represent anything in Rust, but completely forgot that you can put composite values into keys in a map. The panic seems inconvenient, and with ? available now, json should just return a Result.


missing bin files are reported as warning instead of error

failing to open is an error, so I believe this should be an error as well
in addition, the warning string is not even used here, so I had to step through the code to see what went wrong

Thank you for PR.
LoadExternalFile is also used for loading texture files. For texture file, I think it should be better to report it as an warning so that people can at least load .gltf file without textures. .bin is mandatory thus it should report as an error.
So I think it would be nice to add must_exist or mandatory argument to LoadExternalFile and decide making warning or error for missing file.

required perhaps, for consistency?

Yeah, required should be nice!

this will do?

Super awesome! Merged!


checkbox list tile focus node

Hi @vasilich6107,
Is this PR based on the latest 11.0.0 version? I want to make sure that everything that is in v11 is also here.

@joanpablo yes you are safe to merge it

@joanpablo could you merge this PR


Shulker boxes not always sorted to top

Still seems to be alphabetical by item name? 🤔

redstone_torch, shulker_box, sticky_piston… might just have been lucky during testing that they always appeared first alphabetically as well.

Ah, I think I thought I added a "AAA" prefix for all shulkers, just like I did the "ZZZ" one for empty boxes. Needs fixing here.

Fixed in https://github.com/manuelgrabowski/KeepInventorySortedSimple/commit/e680b10acc5b62f04c6d37d0d534774488badb13 – only for ID mode, though.


How to update cloudflared?

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
The Uptime Kuma includes cloudflared v2023.10.0 for now.
But the latest version is v2024.1.4, is it possible to update the one?
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.11
💻 Operating System and Arch
Linux omv 6.1.0-rpi7-rpi-v8 #1 SMP PREEMPT Debian 1:6.1.63-1+rpt1 (2023-11-24) aarch64 GNU/Linux
🌐 Browser
Google Chrome v120.0.6099.234
🐋 Docker Version
Docker version 24.0.7, build afdd53b
🟩 NodeJS Version
No response

We update these kinds of dependencys like apprise or cloudflared before every release.
We currently ship with the latest at the time of our last release: https://github.com/cloudflare/cloudflared/blob/master/RELEASE_NOTES
Is there a reason that requires a patch release for this?

Please check this issue for the recommended method of running an external cloudflared with your chosen version.
https://github.com/louislam/uptime-kuma/issues/3993#issuecomment-1803129833

Thanks for your help.
Maybe the best way for me to get the latest version would running cloudflared outside of the Uptime Kuma container.
I just want the notification in the CF tunnel away, because it is little noisy.
Thanks again for your informations.


Hyphen and Minus symbol show up identically.

Compare:
-10 # normal hyphen (U+002D)

−10 # Minus U+2212

-− You can see one is generally narrower and vertically at a different position.
Would really like to be able to tell them apart. Thanks!

Thank you, works for tewihm, will you add it to the other fonts as well (I prefer tewi2a). I will use tewihm for now. :)

See #29. It'll be sort of annoying to do properly so it might take a while.

Okay thanks 😄


addons and missing resources

I still have to add the following dependencies:
compile group:'com.vaadin', name:'vaadin-cdi', version:'1.+'
compile group:'com.vaadin', name:'vaadin-client-compiled', version:'7.4.+'
Is there no way to handle this by the plugin automaticly?
is it possible to add addons through vaadinAddons or another method?
Wildfly is still complaining about missing valo resources, e.g. spinner.gif

compiled gets added automatically if you don't use a widgetset

As @christoph-frick says, the vaadin-client-compiled dependency is automatically added if you haven't set the vaadin.widgetset property. If you define a widgetset the client-compiler (and its dependencies) will get added instead so your widgetset gets automatically compiled.
The vaadinAddons task will only list addons from the directory, it will not add anything to your build files. Build files can contain so much custom logic that it would be unsafe for the plugin to start adding things into it.
Not sure why Wildfly would complain about missing Valo resources. Might be related to GZip compression being enabled (as stated here http://stackoverflow.usinfonow.com/questions/27864923/vaadin-missing-icons-of-all-components). Anyway, I doubt it has something to do with the plugin.


RangeError: Maximum call stack size exceeded

i fixed it by make qreal return promise and edite $async function

link of tweet what i saw in example of qreal after edite


No way to set the token for all requests

It doesn't look like there is a way to set a token in the client when making an instance of the class, stopping you from having to put the variable everytime

This has been fixed in the latest version in development, should be released soon:tm:


nixos/networkd: update options

Motivation for this change
To fix missing option DNSDefaultRoute= (see #91761) and other missing options.
Things done
I have not tested all options that I touched.

[ ] Tested using NixOps
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I will update the PR following this comment.

This needs another rebase.

This needs another rebase.

Done
This is the second time I rebased my PR on master after more recent PRs than mine were merged. What guides the order in which PRs are merged?

Thanks!

This is the second time I rebased my PR on master after more recent PRs than mine were merged. What guides the order in which PRs are merged?

There's no specific order, but this PR is quite big, which is why it's harder to review.

Networking-Tests are looking good.
/nix/store/4y8k9isgglvhycvq59kg50hx62rw08kn-vm-test-run-Bond-Networking-Networkd
/nix/store/0kmcb3mwlm63hjkbd50h075d61ncqf0i-vm-test-run-Bridge-Networking-Networkd
/nix/store/dcz18v4qz82sx9ynr3wyqimzrmaigpgg-vm-test-run-OneInterfaceDHCP-Networking-Networkd
/nix/store/1g9p0kgpd8nzbv6kqkkcq08nh8hmvmxi-vm-test-run-SimpleDHCP-Networking-Networkd
/nix/store/3014l21l3qrg01s3x33ys08fgjqrs9lw-vm-test-run-Link-Networking-Networkd
/nix/store/jllmqcvalzn1imsb6x0r57dfcdw2v7cz-vm-test-run-Loopback-Networking-Networkd
/nix/store/431izfl3mb82crr6kaw143ky0r04c9yl-vm-test-run-MACVLAN-Networking-Networkd
/nix/store/p7vj2zfcl228a9p5ri27ycrdm1yix6v9-vm-test-run-Privacy-Networking-Networkd
/nix/store/fs1rk6x92h83gfm1f8pm4ps8158g17ph-vm-test-run-routes-Networking-Networkd
/nix/store/wvh4cha8ssx3ypplwqp75wf1phiy46yx-vm-test-run-Sit-Networking-Networkd
/nix/store/blbh606jdzsbgdadbavj8nl0ic00c4d0-vm-test-run-Static-Networking-Networkd
/nix/store/mgv2b09h658glqn8lgz2s3akxdwzf7d7-vm-test-run-Virtual-Networking-Networkd
/nix/store/8kfj6r2kh6ab2w5g6ix304fr21zlw2vz-vm-test-run-vlan-Networking-Networkd

I tried reviewing this, but it's really hard to do.
e9d13d3 added some commented-out checks due to Nix' <= 2.2 integer size, and 70407f0 commented them back in. Could this be squashed together, so we only add the checks in a single commit?

As much as I could try, each of these 2 commits do one logical thing and one thing only, as described by the commit messages

e9d13d3: update options for systemd 245. That commit synchronizes the code with the man page of networkd and, for homogeneity with the rest of the code, keeps avoiding the use of assertRange with 64bits integers.
70407f0: use assertRange with 64bits integers. This commit takes into account this comment. Perhaps it would better fit as a separate PR after this one would have been merged. Since the PR is about cleaning up the networkd module, I included this commit here.

I apologize if my commit messages were not clear enough.

Also, e9d13d3 does a bit too much at once. For example, it removes OriginalName. Is this intended?

Again, that commit synchronizes the code with the man page of networkd. The best way to review that I would suggest is to read the man page. For the specifics of OriginalName, refer to man systemd.link. I also try to double check by contacting the author here, without success. However, you kind of confirmed it yourself here. To facilitate our work, let's discuss specific code issues/questions with the review feature of Github.

Could you revise the commits, and make it these changes easier to review? Maybe something like this also deserves a separate commit.

I do not think it is beneficial because the commit does one clear logical thing. If you disagree, please describe what you mean by "these changes" because, as you may have noted, this is not the only thing I removed in this commit e9d13d3.

Thanks again for your patience! I'll try to react quicker on further updates.

Thank you as well for reviewing this.

Could we move on with this PR?

I gave this another close look, and while it is pretty hard to review due to the amount of changes, this seems to be fine, and should make things more readable and understandable in the longer run.
Thanks for the work you put in this, and your patience with me :-)


Question about U3DPW and H36M

Hi, thank you very much for your excellent job!
Could you please explain how to use U3DPW as an augmented dataset for H36M? The keypoints on the H36M dataset and the U3DPW dataset are different. The visualization results of the 3d skeleton on two datasets are not totally the same.
Thank you very much for your help!

Hi, thank you very much for your excellent job!
Could you please explain how to use U3DPW as an augmented dataset for H36M? The keypoints on the H36M dataset and the U3DPW dataset are different. The visualization results of the 3d skeleton on two datasets are not totally the same.
Thank you very much for your help!

Hi, the U3DPW dataset was only used for validation and not used during training. We did not use U3DPW as augmented data. In addition, we only do data augmentation for 2D-to-3D networks. We did not do data augmentation when training the high-resolution heatmap regression model.
I think it is possible to use U3DPW as augmented data for 2D pose estimation. However, if a few key-points does not match, the user need to manually correct them. The annotation style is quite similar to COCO or MPII.

Thank you so much for your reply! Is it possible to refine the annotation style from coco style to H36M style? I am glad to have a try if the aforementioned refinement is feasible.

Thank you so much for your reply! Is it possible to refine the annotation style of your proposed tools from the coco style to the H36M style? I am glad to have a try if the aforementioned refinement is feasible.

I'll mark this issue as a feature to add. I will discuss with other contributors @kp-kepra about adding H36M style annotation.

Thank you very much for your help! I'll close the issue.

Thank you very much for your help! I'll close the issue.

H36M style 2D annotation for U3DPW is now available.


Setting SMTP notification to seznam

I can not set email notifications for the Email account "Seznam". The settings are: smtp.seznam.cz, 465, Requires authentication (Yes), Smtp account and password filled in (Yes), Requires TLS (Yes), Requires SSL (Yes). The email is not the same as the Zonky account.
LOGY:
21: 23: 44.323 ERROR c.g.r.i.EmailSettingsValidator Uncaught exception.
What am I doing wrong?

Honestly? I don't know. I've seen Seznam work for some people, and fail for others. Google works reliably for everyone.

@ondrejFuchs I used these settings in the installer and the testing e-mail was sent. Can you please review and let me know what you did differently? (I tried experimenting with the SSL/TLS checkboxes and it works as well when both are checked.)

OK. It is possible that something I did recently actualy fixed the problem - I'll close this issue now and feel free to reopen when 4.8.0 is out later this week.
Thanks for reporting!


RTMP - camera2 Streamed video stability issue while moving camera

@pedroSG94 bro,
The video stream distorts while the camera is even moving slightly.
But video quality is perfect if the camera is still.
Is there a way to control this distortion?
I've used
 rtmpCamera2.enableOpticalVideoStabilization();
 rtmpCamera2.enableVideoStabilization();
but nothing seems working.
I'm attaching a screenshot of the stream result of both the still and moving camera stream for reference.

Before closing, I would like to extend my heartiest gratitude towards you and the tremendous open-source effort you and other contributors have put behind this wonderful module.

I'm not sure if you still need it but this seems a low bitrate or a high iframeinterval value.

Thanks bro... Low bitrate was the issue...
But high bitrate has its consequences like more battery drainage, more
network data usage and on cellular. Network connection mobile gets heated
more quickly.. because of high processing and bit through
On Mon, Mar 14, 2022, 2:52 PM Pedro Sánchez @.***>
wrote:

I'm not sure if you still need it but this seems a low bitrate or a high
iframeinterval value.
—
Reply to this email directly, view it on GitHub
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/issues/1069#issuecomment-1066581565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG3Q7ANJDJ7TOHQPWXX2OPDU74D63ANCNFSM5QPHPV5A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you modified the open/close state.Message
ID: @.***
com>


feat: check for new releases and auto-update

Version checking is implemented on frontend and autoupdated is not necessary for this project.


onClick calls within modal body not firing

Summary:
I have a button within my modal body, identical to the examples in the readme and docs. When I click the button, nothing happens. I cannot even console log.
I am trying to use this button to close the modal. It is calling the same function that is able to successfully close the modal on overlay click.
My modal does close the modal on overlay click if this causes a conflict.
My best guess is some logic that prevents the modal from closing when the modal body is clicked, is preventing my button from being clicked as well.
Steps to reproduce:

Create a modal similar to those outlined in the docs
Add an element that handles onClick events (button, div, a)
Try and click the button.

Expected behavior:
I would expect that my button would be able to call functions as indicated by the docs.
Additional notes:

@micodel can you provide more information about this issue? (like version, browser...)

@netto89 @wschaeferiii @micodel react-modal is current on version v2+. (preferable version v2.0.6 - latest). If you can, please update to this version.
If you can provide more information, please reopen this PR.
Thanks.


Source-built TF version

When built Tensorflow from source, the version appears as "0.12.head".  This will fail various test conditions:
https://github.com/fchollet/keras/blob/master/keras/callbacks.py#L526
https://github.com/fchollet/keras/blob/master/keras/callbacks.py#L531
https://github.com/fchollet/keras/blob/master/keras/callbacks.py#L541
Can I modified these lines to
if parse_version(tf.__version__.rpartition('.')[0]) >= parse_version('0.12.0')
or possibly something in the more elegant way? Thank you.

Works well for me, you will also need to alter the summaries like so:
if parse_version(tf.__version__.rpartition('.')[0]) >= parse_version('0.12.0'):     tf.summary.image(weight.name, w_img) else:     tf.image_summary(weight.name, w_img)


added documentation + roadmap

Closes #16

documentation looks great! what if we put it in the wiki rather than as md files?


Added documentation for swagger page

@nmanovic
Maria is our student from NNSTU

@Marishka17
Can you please write unresolved issues in comments to your PR?

Unresolved issues:

request(load_data_proxy) is not displayed on swagger page

Type commands below into the terminal window to generate a typescript client according to Swagger API specification.
git clone https://github.com/swagger-api/swagger-codegen
cd swagger-codegen
./run-in-docker.sh mvn package

wget http://localhost:7000/api/swagger.json --header "Authorization: Token ${your-token}" -O modules/swagger.json
./run-in-docker.sh generate -i modules/swagger.json -l typescript-fetch -o /gen/out/typescript-cvat -DpackageName=cvat

The generated client part will be located in ./out/typesckipt-cvat/
- api.ts
- configuration.ts
- custom.d.ts
- git_push.sh
- index.ts
- .gitignore
- .swagger-codegen/VERSION
- swagger-codegen-ignore

@Marishka17
Please, fix the codacy issue: "Unused import"

@Marishka17 ,
Welcome to our team! Great job! Please give me a couple of days to try the PR.


TypeScript handling

I'm currently using this command to build + minify a TypeScript project with esbuild:
esbuild ./index.ts --bundle --minify --outfile=index.js

How, I also have CSS as well as HTML. The index.js is basically just referenced in the index.html.
Is it possbible to use minify-html and still keep the TypeScript support that esbuild offers?
If so, how? Does something like this work and will the reference changed to .js?
<script src="index.ts">

Closing this for now. Feel free to reopen if there are further questions.

In case someone has a similar problem: I'm currently testing Vite, which seems to do the thing above: https://vitejs.dev


Swift 3.0 support question

Hi, I'm new to this ProtocolBuf Swift project.
I think I've updated all the tools to latest version(home-brew, cocoapods, ProtoBuf...), but still I will get the conflict syntax err w/ Swift 3.0 on Xcode, could you please help me to on this?

my Pods install using latest protobuf

Podfile:

brew already installed latest protobuf

proto.swift file are also generated by latest compiler
by the way, got this no such module issue also, not sure if it's related to syntax conflict issue

What version of cocoapods are you using?

pod --version
1.0.1

Cocoapods versions of 1.0.1 not supporting Swift.3.0. You should use version 1.1.0.rc.2.
sudo gem install cocoapods --pre

thanks for reply, I've tried update cocoapods, and that's worked!
one more question, is this also workable by using Carthage?

I think, carthage supports Siwft 3.0.


PTVS-LS Integration: Do not allow Analyzer add modules with invalid case in names

For example, Package/SubPackage/Module.py should be added as package.subpackage.module and not as Package.SubPackage.Module

Probably no longer applies as PTVS is moving to new LS?


Gui crash

the guide says that it crashed and a code appears, does anyone know how to solve it?

Not OP, but have the same issue. Tried with just the mod and api and with other mods. This is for version 1.20 though.
crash-2023-06-09_21.41.53-client.txt
crash-2023-06-09_21.34.02-client.txt
Had problems with the game crashing immediately in 1.19.4 with Optifabric installed as well, gives the ExitCode 1 without a crash log despite all mods being up to date.

@firstdegreecam You are running an outdated version, the latest for 1.20 is 0.6.8a

Sorry, posted the wrong crash logs. These are the right ones.
crash-2023-06-10_14.42.27-client.txt
crash-2023-06-10_14.43.54-client.txt

Thanks, sorry about that. It does in fact work. Was getting a lot of my mods mixed up trying to get things updated to 1.20.


Cyrillic alphabet translations of Feathercoin - Биткоин (Bitkoin)

Биткоин   stands for Bitkoin in cyrillic alphabet  kyrgyz.
Translations are only for English.  https://github.com/FeatherCoin/Feathercoin/search?utf8=✓&q=Биткоин&type=
канат  Feather
тыйын  Coin
тыйын Koin

канат translates to rope for me. There is a bigger issue here that we need native speaks of each language in the locale directory to work on the translations.

Do we still have issues with translations?

@wellenreiter01 basically yes, translations are best guess, but we'd need native speakers to translate each locale properly.


Updated the chatbot with Visible close button, better styling and size

Please check.

Add screenshots of the changes made. Mention your issue number too.

Here are the screenshots

issue #22

Hi, can you please add a level tag as well? for this is my first merged pull request of gssoc 2024.


restic mount fails on FreeBSD 13.1

Output of restic version
restic 0.14.0 compiled with go1.19.2 on freebsd/amd64
How did you run restic exactly?
restic mount /mnt/remote/
parameters are injected via a small python wrapper that create a proper environment for the process with the following values
RESTIC_REPOSITORY = "s3:https://myserver:9000/restic"
AWS_ACCESS_KEY_ID = remote_key
AWS_SECRET_ACCESS_KEY = remote_secret
RESTIC_PASSWORD = passphrase
RESTIC_CACHE_DIR = /store/maintenance/restic/cache
RESTIC_PACK_SIZE = "128"

output is the following:
repository 70565c28 opened (repository version 1) successfully, password is correct
Now serving the repository at /mnt/remote/
Use another terminal or tool to browse the contents of this folder.
When finished, quit with Ctrl-c here or umount the mountpoint.
resource temporarily unavailable

and the process exits unexpectedly.
What backend/server/service did you use to store the repository?
minio, deployed on a remote physical server.
Expected behavior
the mountpoint should show the content of repository
Actual behavior
the command fails without explaining what is the problem.
adding --verbose=3 does not change the output and does not provide other informations.
on the same platform and reposity i regularily use commands backup, restore, forget and prune, and they work like a charm
Steps to reproduce the behavior
on FreeBSD 13.1-RELEASE-p4 run:
pkg install restic
export RESTIC_REPOSITORY ...
restic init
restic mount /mnt

Do you have any idea what may have caused this?
I don't know.
I used the mount command succesfully many times in the past, but i can't remember what was the latest working combo of OS/restic version that worked.
The only thing that comes to mind is that rclone had recently issues with a change in FreeBSD kernel fuse module, but seems they are fixed now. This could be related.
Do you have an idea how to solve the issue?
I wish I had.
Did restic help you today? Did it make you happy in any way?
restic is an amazing piece of software, I use it on all my machines with extreme satisfaction and I will never thank you enough for it.

The fix for the issue still seems to be work in progress: https://github.com/golang/go/issues/54100
From what I can tell, https://github.com/golang/go/commit/b15c399a36a38509ae56dd69670974566f7b0d52 is not part of any released go version so far.

The bug is apparently fixed in FreeBSD 13.2, see #3550.


[META] Ensure 2.11 branch readiness for 2.11.1 release

Coming from https://github.com/opensearch-project/opensearch-build/issues/4161 this issue tracks all child issues for ensuring branch readiness for 2.11.1 release.
What is happening?
We are planning 2.11.1 release in upcoming few weeks. Following the semver guildelines and branching strategy, this release will be based on 2.11 as a reference branch
Why?

We need this release because of a critical breaking bug in the security plug-in. This bug impacts everyone using logstash and the only work around is to turn off compression (which is expensive)

As a refresher, our normal process is to commit changes to main, then backport to 2.x. Since the 1.0 release, we only backport bug fixes and security changes to the current 2.x branch (in this case 2.11) so that if we need to we can quickly do a patch release.

There are two reason we don’t back port features to the current release branch (right now 2.11):

Because we want patch release to be as thin as possible, and the more changes you make, the more verification we need to do
Adding features in a patch breaks semver.

However, looking at the list of changes for 2.11, we currently have a mix of features that have been merged in. After providing several options and discussing with all the component teams as well as community we came to the conclusion of reverting all feature related changes from 2.11 branch.

[ ] opensearch-project/alerting#1297
[x] opensearch-project/anomaly-detection#1097
[x] opensearch-project/asynchronous-search#509
[ ] opensearch-project/common-utils#559
[x] opensearch-project/reporting#926
[x] opensearch-project/index-management#1032
[x] opensearch-project/job-scheduler#538
[x] opensearch-project/k-NN#1294
[ ] opensearch-project/performance-analyzer#592
[ ] opensearch-project/performance-analyzer-rca#516
[x] opensearch-project/security#3650
[ ] opensearch-project/sql#2417
[x] opensearch-project/observability#1765
[x] opensearch-project/cross-cluster-replication#1283
[x] opensearch-project/ml-commons#1592
[x] opensearch-project/geospatial#582
[x] opensearch-project/notifications#812
[x] opensearch-project/neural-search#486
[ ] opensearch-project/security-analytics#714
[x] opensearch-project/custom-codecs#80
[ ] opensearch-project/sql#2418
[x] opensearch-project/alerting-dashboards-plugin#792
[x] opensearch-project/notifications#813
[x] opensearch-project/security-dashboards-plugin#1643
[x] opensearch-project/index-management-dashboards-plugin#914
[x] opensearch-project/anomaly-detection-dashboards-plugin#646
[ ] opensearch-project/reporting#927
[ ] opensearch-project/dashboards-observability#1230
[x] opensearch-project/dashboards-visualizations#289
[x] opensearch-project/dashboards-search-relevance#348
[x] opensearch-project/dashboards-maps#559
[ ] opensearch-project/security-analytics-dashboards-plugin#779
[x] opensearch-project/ml-commons-dashboards#283
[x] opensearch-project/OpenSearch#11094
[x] opensearch-project/OpenSearch-Dashboards#5431

Looks like observability got two issues (1765 and 1766), I assume one of those is meant for dashboards-observability

Looks like observability got two issues (1765 and 1766), I assume one of those is meant for dashboards-observability

Looks like it. I used the meta tool here to create issues. Maybe check if there is some issue with re-direct of GitHub repo naming there? I'll move one of the issue to dashboards-observability.
Thanks!

Sql also has duplicated issues 2147 and 2148, plus we are missing query workbench and dashboard reports.

Sql also has duplicated issues 2147 and 2148, plus we are missing query workbench and dashboard reports.

You can move the issues from sql to those repos. Taking care of it of it for now.

Closing this issue as all related action item sub-issues are now addressed.
Thanks!


Update Travis Matrix

Coverage remained the same at 92.105% when pulling 6cab0bf9bd1d1b4a29849a131e2dde1758aa27cb on update_travis into bc8dfab94b95b14d56a9dcf133b5a19ace49c911 on master.


Allow data: URIs in icons

This pull request aims to accept data: URIs as custom icons.
In the future, the extension may accept local files and can convert them to data: URIs. That would be a nice solution for the local file limitation. Still, that's a suggestion of a future step.
(Please read the commit message.)

Hi,
Could you provide an example with a working data: value?
Bests

Sure! Try these:

http://www.patternify.com/ → Interactively draw and create images.
http://dataurl.net/#dataurlmaker → Drag and drop any file there (for our purposes, use a tiny PNG image).

Thanks @denilsonsa, planning this to 0.15.1.


Team 15 VnV Plan Review: Fit Criteria Not Very Specific

For the test NFR-T18, the output of the test is that "the system operates without significant delays, errors, or crashes", but does not specify in the test what is an acceptable range for the test error to be significant. In some of your other tests (like NFR-T14) you mention that the test will be considered successful if the delay is under "MAX_DELAY" which clearly sets a parameter for the test to stay within to be successful. Consider adding that to this test to make it more verifiable.
TA: @Lyuyang0130

resolved in: ed7d3ea9de2c8ecd01221f43b2461f2fbd82e2d2


docs Amend JS Foundation to OpenJS Foundation (#3971)

Description of the Change
JS Foundation has been modified to remain unaltered as OpenJS Foundation. (#3971)

Coverage increased (+0.07%) to 92.761% when pulling b7754f3268d9da80ca9c607bcd5e71b12690b16b on leehwarang:issue/3971 into ec17f6315e0817bfb8e37279d31affc4ec108623 on mochajs:master.

Thanks!  But ... we still don't have direction from the OpenJS Foundation on updating copyright notices, so we will need to hold off on this.
I think it's probably best to close this for now, and then re-open when we have the required information from the Foundation.


fix: Authorization header use base64

:tada: This PR is included in version 2.0.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

nice catch, thanks


[GOBBLIN-849] Connection to Oracle using service name and update meta…

…data query for OracleExtractor
Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[ ] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-849

Description

[ ] Here are some details about my PR, including screenshots (if applicable):

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2705 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2705      +/-   ##
============================================
- Coverage      44.9%   44.88%   -0.02%
+ Complexity     8713     8709       -4
============================================
  Files          1879     1879
  Lines         70079    70083       +4
  Branches       7703     7706       +3
============================================
- Hits          31466    31454      -12
- Misses        35702    35719      +17
+ Partials       2911     2910       -1

Impacted Files
Coverage Δ
Complexity Δ

...pache/gobblin/configuration/ConfigurationKeys.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

...rg/apache/gobblin/source/jdbc/OracleExtractor.java
43.83% <0%> (-1.24%)
7 <0> (ø)

...bblin/cluster/GobblinHelixJobLauncherListener.java
70% <0%> (-30%)
3% <0%> (-2%)

...lin/util/filesystem/FileSystemInstrumentation.java
85.71% <0%> (-7.15%)
3% <0%> (ø)

.../gobblin/cluster/HelixRetriggeringJobCallable.java
60.41% <0%> (-3.48%)
9% <0%> (ø)

...ache/gobblin/couchbase/writer/CouchbaseWriter.java
66.27% <0%> (-2.33%)
11% <0%> (ø)

...pache/gobblin/cluster/GobblinHelixJobLauncher.java
81.25% <0%> (-2.09%)
26% <0%> (-2%)

...ache/gobblin/cluster/GobblinHelixJobScheduler.java
39.21% <0%> (-1.31%)
6% <0%> (ø)

...in/java/org/apache/gobblin/cluster/HelixUtils.java
41.83% <0%> (ø)
13% <0%> (-1%)
:arrow_down:

.../org/apache/gobblin/cluster/GobblinTaskRunner.java
66.19% <0%> (+0.46%)
29% <0%> (ø)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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

+1, LGTM


[PSM Interop] Legacy tests: fix xDS test client build (v1.51.x backport)

Backport of #33796 to v1.51.x.
https://github.com/grpc/grpc/pull/33699 incorrectly changed the legacy builds to not just use the test driver from the master, but also to build from it. This PR fixes the issue, and also updates the python job to work use the driver from master.

[ ] grpc/core/v1.51.x/branch/linux/grpc_xds_v3
[ ] grpc/core/v1.51.x/branch/linux/grpc_xds_v3_python


Create marketplace-products.html

Here is the component I have created-

@rohan9024 good job
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


Bug 1869613: Swift: add a unit test for bulk deleting failure

This test emulates Swift behavior if it fails to delete one of the given objects in bulk mode.

/bugzilla refresh

/test verify

/lgtm

/retest

/test e2e-openstack

/lgtm

/lgtm

/hold
@Fedosin is it expected that e2e-openstack fails that often?

@dmage yes, openstack ci is highly unstable. it's not related to this PR, since it create a unit test only.

/hold cancel
Let's see if it manages to land.

e2e-openstack job was missing the optional attribute, that's the reason why prow it testing it over and over even though it's not shown as Required in the github UI.
Patch at https://github.com/openshift/release/pull/11521 should fix it.


Transcribing MP3 Files to Turkish

Hi,
I am working with monolingual Turkish, monolingual German and bilingual Turkish-German speaking children, and would love to use batchalign to analyze the language samples of those children. When I want to transcribe the audio file of monolingual Turkish speaking children with the help of Whisper, I get an error and the analysis cannot go forward. The error is below:
(batchalign) SS2:~ ag$ batchalign transcribe --lang=tur --whisper /Users/ag/Music/trans /Users/ag/Music/trans
Mode: transcribe; got 1 transcript to process from /Users/ag/Music/trans:
Device set to use cpu
You have passed task=transcribe, but also have set forced_decoder_ids to [[1,
None], [2, 50360]] which creates a conflict. forced_decoder_ids will be
ignored in favor of task=transcribe.
The attention mask is not set and cannot be inferred from input because pad
token is same as eos token. As a consequence, you may observe unexpected
behavior. Please pass your input's attention_mask to obtain reliable results.
WhisperModel is using WhisperSdpaAttention, but
torch.nn.functional.scaled_dot_product_attention does not support
output_attentions=True or layer_head_mask not None. Falling back to the
manual attention implementation, but specifying the manual implementation will
be required from Transformers version v5.0.0 onwards. This warning can be
removed using the argument attn_implementation="eager" when loading the model.
⠧ XXX.mp3 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% 0:00:54 Running: ASRKilled: 9
(batchalign) SS2:~ ag$
Thank you very much for your work that enables researchers and clinicians to save time with this program. I will be looking forward to seeing your response.

I'm also trying to transcribe Turkish kids, and I'm getting the same error.
(venv) (base) PS C:\Users\dellvostrolpc\Desktop\BATCH\batchalign2> batchalign transcribe --lang=tur --whisper "C:\Users\dellvostrolpc\Desktop\ba_input" "C:\Users\dellvostrolpc\Desktop\ba_output"

Mode: transcribe; got 1 transcript to process from C:\Users\dellvostrolpc\Desktop\ba_input:
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50360]] which creates a conflict. forced_decoder_ids will be ignoredin favor of task=transcribe.
The attention mask is not set and cannot be inferred from input because pad token is same as eos token. As a consequence, you may observe unexpected behavior.
Please pass your input's attention_mask to obtain reliable results.
WhisperModel is using WhisperSdpaAttention, but torch.nn.functional.scaled_dot_product_attention does not support output_attentions=True or layer_head_masknot None. Falling back to the manual attention implementation, but specifying the manual implementation will be required from Transformers version v5.0.0
onwards. This warning can be removed using the argument attn_implementation="eager" when loading the model.
From v4.47 onwards, when a model cache is to be returned, generate will return a Cache instance instead by default (as opposed to the legacy tuple of tuples
format). If you want to keep returning the legacy format, please set return_legacy_cache=True.
DenizYilmaz.mp3 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% 0:02:46 FAIL
ERROR on file dd.mp3: _extract_token_timestamps() got an unexpected keyword argument 'num_input_ids'

I'm also trying to transcribe Turkish kids, and I'm getting the same error.
(venv) (base) PS C:\Users\dellvostrolpc\Desktop\BATCH\batchalign2> batchalign transcribe --lang=tur --whisper "C:\Users\dellvostrolpc\Desktop\ba_input" "C:\Users\dellvostrolpc\Desktop\ba_output"

Mode: transcribe; got 1 transcript to process from C:\Users\dellvostrolpc\Desktop\ba_input:
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50360]] which creates a conflict. forced_decoder_ids will be ignoredin favor of task=transcribe.
The attention mask is not set and cannot be inferred from input because pad token is same as eos token. As a consequence, you may observe unexpected behavior.
Please pass your input's attention_mask to obtain reliable results.
WhisperModel is using WhisperSdpaAttention, but torch.nn.functional.scaled_dot_product_attention does not support output_attentions=True or layer_head_masknot None. Falling back to the manual attention implementation, but specifying the manual implementation will be required from Transformers version v5.0.0
onwards. This warning can be removed using the argument attn_implementation="eager" when loading the model.
From v4.47 onwards, when a model cache is to be returned, generate will return a Cache instance instead by default (as opposed to the legacy tuple of tuples
format). If you want to keep returning the legacy format, please set return_legacy_cache=True.
Ddz.mp3 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━   0% 0:02:46 FAIL
ERROR on file Ddz.mp3: _extract_token_timestamps() got an unexpected keyword argument 'num_input_ids'

and I would like to ask one more question: Do I have to use Whisper or Rev.AI to transcribe because they enable transcription in CHAT format? Or can I transcribe the files with other programs or myself? Since I could run the morphotag function, I wanted to know if the problem with the function "transcribe" cannot be solved quickly.


SPIKE: fix pullImage for when source already contains digest

This fixes the current implementation to correctly pullImage when the source image from annotations already contains the image digest.
(note this was generated from dt annotate command where the values specify the image digest)
values.yaml
image2:
  registry: example.com
  repository: deploy/aaaaaa/service-a
  pullPolicy: IfNotPresent
  # Overrides the image tag whose default is the chart appVersion.
  tag: 0.3.0
  digest: sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867

generated Images.lock
apiVersion: v0
kind: ImagesLock
metadata:
  generatedAt: "2024-09-24T07:19:33.65058Z"
  generatedBy: Distribution Tooling for Helm
chart:
  name: test-chart1
  version: 0.1.0
  appVersion: 1.16.0
images:
  - name: service-a
    image: example.com/deploy/aaaaaa/service-a:0.3.0@sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867
    chart: test-chart1
    digests:
      - digest: sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867
        arch: linux/amd64

Failure
    ✘  failed to pull images: failed to pull image "service-a": parsing reference " example.com/deploy/aaaaaa/service-a:0.3.0@sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867@sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867": could not parse reference:  example.com/deploy/aaaaaa/service-a:0.3.0@sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867@sha256:f43290f7f2e3c3c8281bee512d216b7c604fd92501b3b4522df9333b1921f867

Good find @machecazzon . Thanks!


changed acquireToken methods to return promises issue #24

@rohitnarula7176,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:shipit:


tasks/R/easy/hello_world.R

Pull Request Type
What kind of change does this PR introduce?
[ ] Solved an issue/task
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:
What is the current behavior?

Issue Number: [Insert Issue Number (if applicable)]
What is the new behavior?

Other information

You have already done this task.


feat: added sub-module network interface

Added sub-module nic ( network interface card) and terratest pass 🎉
On the complete example the subnet id for nic, has been resourced from the module network hubs, this was done because the module network has delegation on its subnet ids
Fixes #25

Will have a closer look ASAP 👀

I'd suggest renaming the PR to something that would make sense in a changelog, e.g. feat: create network interface

the PR has been renamed and is now ready for a new review

A conflict had to be resolved, i noticed it a little late and have now resolved the conflict aswell as merge the main branch to the PR branch. Ran a new test, it passed and now this PR is ready for review


[206] Remove 'hide your profile' section

Revert this commit to put it back when ready post-MVP.
Screenshots of UI changes:
Before

After

 Remove "Hide your profile" section from Candidate Profile

@elliotcm happy with this one, good to go 👍


Conductor API for updateWorkflowDef doesn't support updating onStateChange values

Conductor API for updateWorkflowDef doesn't support updating onStateChange values.
        task.on_state_change = {
            "onScheduled,onStart,onSuccess,onFailed,onCancelled": [
                {
                    "type": "task_status_changed",
                    "payload": {"task": "${%s}" % task.task_reference_name, "workflow": "${workflow}"},
                }
            ]
        }

but updated workflow is missing this data because client doesn't have getter/setter for this newer attribute.

Hi @matteius this has been addressed in the latest version.  See https://github.com/conductor-sdk/conductor-python/blob/main/examples/task_status_change_audit.py for example.


Enables mssql support

Port of @dlob work in gogs (gogits/gogs#3772)
Enables options in index.js
Enables MSSQL as a database option in go.
Sets ID to 0 on initial migration. Required for MSSQL insert  #statements.
Signed-off-by: Beau Trepp<EMAIL_ADDRESS>

LGTM.

@btrepp  Maybe you have to govendor add github.com/denisenkom/go-mssqldb, since Gitea use govendor to manage dependencies.

Added a new commit with what is needed for govendor (never used that before, sorry).
Do you want me to fix up the missing space and squash the commits into one?.

Please fix the format and keep the vendoring in a separate commit. Than I'm fine with that.

Works surprisingly well.
LGTM, but should be merged only on 1.1.0

There is also another remaining issue related to that, if I remember correctly on the explore page


Upgrade netty to 4.0.36.Final

Upgrade netty to 4.0.36.Final and add build profile for netty 4.1.0

Implemented.


error on the examples

Error on example 1
Enable-ClusterStorageSpacesDirect : A parameter cannot be found that matches parameter name 'cluster'.
At line:1 char:35
Error on example 2
Enable-ClusterStorageSpacesDirect : The input object cannot be bound to any parameters for the command either because
the command does not take pipeline input or the input and its properties do not match any of the parameters that take
pipeline input.
At line:1 char:15

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

ID: 1028227e-51ad-854a-ca48-e176894cdb34
Version Independent ID: 1967653d-ebb8-7a06-af67-8359e8a85bdd
Content: Enable-ClusterStorageSpacesDirect (failoverclusters)
Content Source: docset/windows/failoverclusters/enable-clusterstoragespacesdirect.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

@officedocsbot unassign @e0i
@officedocsbot assign @yogkumgit

Hello @tofkamp thanks for your feedback, I created a pair of pool requests to correct the examples.
Thank you

@tseward and @dariomws306 please review PR Removed non working examples #1139 and Removed non working examples #1140  and kindly provide your feedback

Looks good.

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @tofkamp Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


Added the ability to make "chip in" charges.

Most of what is needed for wevote/Campaigns/issues/100
But parallel changes are needed in the WebApp

👍


It seems that the lib does not support union types (or discriminatedUnion) on querystring

I'm using it with:
import fastifySwagger from "@fastify/swagger";
import fastifySwaggerUI from "@fastify/swagger-ui";

As it is described in the documentation of this lib.
Try the following:
  app.get(
    "/",
    {
      schema: z.union([z.object({ x: 1 }), z.object({ a: 2 }) ]),
    },

I believe that z.union should be mapped to anyOf while z.discriminatedUnion should be mapped to oneOf
My complete setup is more or less the following:
import 'zod-openapi/extend';
import fastifySwagger from "@fastify/swagger";
import fastifySwaggerUI from "@fastify/swagger-ui";
import Fastify from "fastify";

import {
  type FastifyZodOpenApiTypeProvider,
  fastifyZodOpenApiPlugin,
  fastifyZodOpenApiTransform,
  fastifyZodOpenApiTransformObject,
  serializerCompiler,
  validatorCompiler,
} from 'fastify-zod-openapi';
import { type ZodOpenApiVersion } from 'zod-openapi';

import { logger } from "@/logger";

export async function buildFastifyApp() {
  const app = Fastify({
    loggerInstance: logger,
  }).withTypeProvider<FastifyZodOpenApiTypeProvider>();

  // Add schema validator and serializer
  app.setValidatorCompiler(validatorCompiler);
  app.setSerializerCompiler(serializerCompiler);

  await app.register(fastifyZodOpenApiPlugin, {
    documentOpts: {
      unionOneOf: true,
    }
  });

  await app.register(fastifySwagger, {
    openapi: {
      openapi: '3.0.3' satisfies ZodOpenApiVersion,
      info: {
        title: "My Project",
        description: "My Project API",
        version: "1.0.0",
      },
      servers: [
      ],
    },
    transform: fastifyZodOpenApiTransform,
    transformObject: fastifyZodOpenApiTransformObject,
  });

  await app.register(fastifySwaggerUI, {
    routePrefix: "/documentation",
  });

Any thoughts?

Hey, openapi does not support unions for parameters. If you can find me some examples where it does I'd be happy to help you out

Though, it may be possible to "flatten" the properties in the headers out into an "optional"-esque main object.

I'm referring to this: https://swagger.io/docs/specification/v3_0/data-models/oneof-anyof-allof-not/
From the OpenAPI specs:

And the link to it: https://github.com/OAI/OpenAPI-Specification/blob/main/versions/3.0.3.md#openapi-object
Like I said

I believe that z.union should be mapped to anyOf while z.discriminatedUnion should be mapped to oneOf

Does that make sense?

Yes but you mentioned query strings which are parameters which are cannot use oneOf or anyOf.
https://learn.openapis.org/specification/parameters.html
https://github.com/samchungy/fastify-zod-openapi/blob/375656feac7e3cecce3473e0a4b42491bebf5a9a/src/transformer.test.ts#L962
Here's oneOf working.


Refactor project to use smaller models

Proposed changes

Each module now uses its own model for NLP
CLI now runs a model that checks the text and chooses the best suited module to handle that text
Many minor changes that I forgot

Related issue
Fixes #57

Note that this is not the most beautiful refactoring. In another PR the loading of models should be done through the settings instead of hard coded.


Integrate LLVM at llvm/llvm-project@ba38640b9901

Build possibly broken by https://reviews.llvm.org/D148493 ?
May need to add some explicit cast<T>(...) calls


Fix CSG intersection

This isn't for final review as it needs a bit more work, but it resolves the issue and demonstrates that it doesn't require changes to the DynamicNodeManager.
As you identified @jmlait, the issue is that for the root node in CSG intersection mode, if any trees being merged don't have a particular key, that element needs to be deleted from the resulting tree, meaning unlike CSG union and CSG difference, absence of a root node child takes precedence over outside/inside tiles or child nodes. This doesn't apply to InternalNodes in CSG intersection mode as every element must be either a tile or a child.
I took a slightly different approach to yours here and first detect any keys that need deleting from the target tree, then delete them. Adding tiles or children in CSG intersection mode is then only permissible if they already exist in the root node. The root node has a pretty strict API so it's a bit of convoluted logic to even delete root node children - I found the easiest solution was to convert any elements I wanted to delete into background tiles and then call root.eraseBackgroundTiles(). There's definitely an argument for extending the RootNode API to make this kind of operation easier, but I don't think that should hold up this fix necessarily.
My next step is to modify the unit tests to run through the original CSG visitors to make sure I haven't baked in any other incorrect assumptions (as a test only - Merge.h should remain upstream of Composite.h). I also need a few additional unit tests to test the dropping of tiles and children in CSG Intersection mode.

This works both ways for my test case.
In line 630/732 (sadly I can't comment on lines not touched in github!) we have
if (!mergeNode)     continue;
This read to me suspicously like a similar potential problem.  Where does this flag trigger, and why doesn't it have the same implications?

This works both ways for my test case.
In line 630/732 (sadly I can't comment on lines not touched in github!) we have
if (!mergeNode)     continue;
This read to me suspicously like a similar potential problem.  Where does this flag trigger, and why doesn't it have the same implications?

This works both ways for my test case.
In line 630/732 (sadly I can't comment on lines not touched in github!) we have
if (!mergeNode) continue;
This read to me suspicously like a similar potential problem. Where does this flag trigger, and why doesn't it have the same implications?

I understand your concern.
Firstly, I'll just highlight that the DynamicNodeManager only caches at the level it is about to execute which you can see through the interleaving of the "init" methods and the "foreach" methods here:
https://github.com/AcademySoftwareFoundation/openvdb/blob/master/openvdb/openvdb/tree/NodeManager.h#L1592
so any manipulation of a target node directly impacts how the subsequent source children are merged into the target children. For example, replacing a target node with a target tile eliminates any more recursion down this branch before caching happens.
Having made the change in this PR, if you are evaluating a node below a root node (ie InternalNode1, InternalNode2, LeafNode) then all of your source merge trees must contain an element in the root node hash table that has the same key as the one that this node is rooted under. That means that if a source merge tree has no node at this level of the tree, it is because one of it's ancestors is a tile. With CSG intersection, if that case arose, the tile could only be an inside tile because an outside tile would have taken precedence and forced termination before recursion reached this level of the tree. A child node takes precedence over an inside tile so it can be safely skipped.

This works both ways for my test case.
In line 630/732 (sadly I can't comment on lines not touched in github!) we have
if (!mergeNode) continue;
This read to me suspicously like a similar potential problem. Where does this flag trigger, and why doesn't it have the same implications?

I understand your concern.
Firstly, I'll just highlight that the DynamicNodeManager only caches at the level it is about to execute which you can see through the interleaving of the "init" methods and the "foreach" methods here:
https://github.com/AcademySoftwareFoundation/openvdb/blob/master/openvdb/openvdb/tree/NodeManager.h#L1592
so any manipulation of a target node directly impacts how the subsequent source children are merged into the target children. For example, replacing a target node with a target tile eliminates any more recursion down this branch before caching happens.
Having made the change in this PR, if you are evaluating a node below a root node (ie InternalNode1, InternalNode2, LeafNode) then all of your source merge trees must contain an element in the root node hash table that has the same key as the one that this node is rooted under. That means that if a source merge tree has no node at this level of the tree, it is because one of it's ancestors is a tile. With CSG intersection, if that case arose, the tile could only be an inside tile because an outside tile would have taken precedence and forced termination before recursion reached this level of the tree. A child node takes precedence over an inside tile so it can be safely skipped.

Just as a note to this ongoing work - a "valid" levelset is expected to have only inactive tiles and for those tiles to have either positive or negative background values. I've found that the visitor-based CSG intersection method doesn't extrapolate in a predictable way to "invalid" levelsets. This means that if you provide any "invalid" levelsets as the input, the resulting levelset is not always a "valid" levelset itself and that the CSG intersection method isn't always commutative. I'm treating this as "undefined behavior" and only strictly matching the visitor-based implementation for valid inputs, so really I'm just highlighting that there may be differences in the result of these new methods compared to the visitor-based CSG methods if the inputs are not valid levelsets.

Just as a note to this ongoing work - a "valid" levelset is expected to have only inactive tiles and for those tiles to have either positive or negative background values. I've found that the visitor-based CSG intersection method doesn't extrapolate in a predictable way to "invalid" levelsets. This means that if you provide any "invalid" levelsets as the input, the resulting levelset is not always a "valid" levelset itself and that the CSG intersection method isn't always commutative. I'm treating this as "undefined behavior" and only strictly matching the visitor-based implementation for valid inputs, so really I'm just highlighting that there may be differences in the result of these new methods compared to the visitor-based CSG methods if the inputs are not valid levelsets.

I've extended and improved the unit tests and validated against the original composite methods now. This highlighted a couple of additional differences between the implementations that I've now addressed.
There is one change in behavior that I've intentionally made, which is to make the root node CSG intersection and CSG union commutative in one specific case where it was not previously as I believe this is more correct. This is demonstrated by the following:
FloatGrid::Ptr grid1 = createLevelSet<FloatGrid>();
grid1->tree().root().addTile(Coord(0, 0, 0), 10.0f, false);
FloatGrid::Ptr grid2 = createLevelSet<FloatGrid>();
grid2->tree().touchLeaf(Coord(0, 0, 0));
tools::csgIntersection(grid1->tree(), grid2->tree());

In this case, the resulting grid has one inactive tile with value 10.0. However, if you reverse the order of the inputs, the tile value is mapped to the background value on merge and then gets removed from the grid (as do all background tiles at the root node level). With the change in behavior, the resulting grid from the above routine now also re-maps the tile value to the background value and removes it from the grid so the result is identical irrelevant of the order of inputs. This remapping of non-background values is only performed at the root node so there is no change in behavior in the rest of the tree.

I've extended and improved the unit tests and validated against the original composite methods now. This highlighted a couple of additional differences between the implementations that I've now addressed.
There is one change in behavior that I've intentionally made, which is to make the root node CSG intersection and CSG union commutative in one specific case where it was not previously as I believe this is more correct. This is demonstrated by the following:
FloatGrid::Ptr grid1 = createLevelSet<FloatGrid>();
grid1->tree().root().addTile(Coord(0, 0, 0), 10.0f, false);
FloatGrid::Ptr grid2 = createLevelSet<FloatGrid>();
grid2->tree().touchLeaf(Coord(0, 0, 0));
tools::csgIntersection(grid1->tree(), grid2->tree());

In this case, the resulting grid has one inactive tile with value 10.0. However, if you reverse the order of the inputs, the tile value is mapped to the background value on merge and then gets removed from the grid (as do all background tiles at the root node level). With the change in behavior, the resulting grid from the above routine now also re-maps the tile value to the background value and removes it from the grid so the result is identical irrelevant of the order of inputs. This remapping of non-background values is only performed at the root node so there is no change in behavior in the rest of the tree.

Approving as this now passes @jmlait's tests


How to parse the artists metadata array as a single string separated by comma?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
In some youtube music songs, there is no artist tag but artists tag. Json dump considers this an array of strings, but is there a way to parse the metadata to make it human readable separated by commas?
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp --print "%(artists)s" "https://music.youtube.com/watch?v=GCpo61azoYE&si=Nr9NbTqU5Z0isqoY"
['Mayumi Fujita', '田中怜子', 'ATLUS']

%(artists)l

see 7. More Conversions


[Payment card / Subscription] make declined credit card have better error message

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Reproducible in staging?: yes
Reproducible in production?: yes
Email or phone of affected tester (no customers):
Issue reported by: @anmurali
Slack conversation: https://expensify.slack.com/archives/C036QM0SLJK/p1726609164016559
Action Performed:
Break down in numbered steps

create a new account with a collect subscription
go to the subscription page and try to add a billing card using Stripe's test card that will decline due to insufficient funds<PHONE_NUMBER>009995)
Try to add it

Expected Result:
The error provided to the user would be well worded and let them know what went wrong and what they can do to fix it
Actual Result:
It says "Auth CreateFund returned an error"
Screenshots/Videos

View all open jobs on GitHub

Ah, I went and looked at the design doc and this is a error case I think we anticipated, but for some reason it isn't working like we originally designed it. Maybe it never worked to begin with and we're only now discovering it because real users are running into it 🤷

Still investigating

I suspect the backend isn't properly updating the billing status onyx key

Well maybe the original hypothesis was correct. It looks like those special errors screens are for when we bill users who were already able to add valid funds to their subscription, whereas this is about being rejected straight from the get go of trying to add a card. So I think it makes sense that we should just provide a generic-ish error message to the user trying to add a bad billing card.

I think I got the solution for this in my head but I ran out of time today. Will come back to it tomorrow. Basically we want to modify web to show a generic error here instead of the Auth CreateFund error. The only thing that is kind of throwing me for a loop is that I'm getting different errors in the Old Dot version of this even though they both call the same CreateFund command under the hood. Need to look into it a bit more.

PR out for review: https://github.com/Expensify/Web-Expensify/pull/43590

This was deployed to production


Support for 3.13.x

Is your feature request related to a problem? Please describe.
RabbitMQ 3.12 is out of support since June 1st
Describe the solution you'd like
Support 3.13.x
Describe alternatives you've considered
No response
Additional context
No response

I was also searching the availability of 3.13 in a BOSH release but did not find it.
Is this planned? Or will 3.13 be skipped?

3.13 is not available up to at least BOSH release v613
No Branch in GitHub that shows any active work on this

Moreover, according to the RabbitMQ release page, 3.13.7 was the last one in the 3.13 stream. As we are now after "End of Community Support", does this mean there will be no more 3.13.8?


user spring authentication and authorization

Can you  please resolve your conflicts

I was implementing this issue, and there is an important note. The pom.xml needs to be changed on this part: "1.4.2.RELEASE to 1.4.4.RELEASE". Because there is some features that does not work right with spring-boot-starter-security.


[incubator/kafka] fail to start metrics sidecar

Describe the bug
fail to start kafka pod (metrics sidecar), when prometheus.jmx.enabled = true
Version of Helm and Kubernetes:
k8s:  1.10
helm: 2.10
Which chart:
kafka
What happened:
Fail to start kafka pod, when prometheus.jmx.enabled = true
How to reproduce it (as minimally and precisely as possible):
set prometheus.jmx.enabled = true and helm create
Anything else we need to know:
metrics container is started before broker container yet pending on broker

which among the two brokers of kafka statefulset's pod get ready ? Try checking logs for the container


timer countdown

?will each correct question add time back to the countdown or just keep things from getting worse? (jewels)

I imagined that each problem would be a stage, just like spaceteam

solving a problem before timer expires, next stage

yeah, it'd be a bit sad if the user can never fix a reactor lol

cool :)


Remove inline from DOM.attr

It doesn't seem necessary to inline here. It's also better not to inline big functions because this creates code duplication and can increase the bundle size.

Merged, thank you. I can't remember why that was inline in the first place!

Merged, thank you. I can't remember why that was inline in the first place!


youtube-dl --verbose

Traceback (most recent call last):
File "/usr/local/lib/python3.11/site-packages/jishaku/features/invocation.py", line 168, in jsk_debug
await alt_ctx.command.invoke(alt_ctx)
File "/home/user_203869495486316544/.local/lib/python3.11/site-packages/discord/ext/commands/core.py", line 1023, in invoke
await injected(*ctx.args, **ctx.kwargs)  # type: ignore
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/user_203869495486316544/.local/lib/python3.11/site-packages/discord/ext/commands/core.py", line 238, in wrapped
raise CommandInvokeError(exc) from exc
discord.ext.commands.errors.CommandInvokeError: Command raised an exception: DownloadError: ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

#30839 !


feature(core): allow implementers to override the action done when a message is failed to be consumed #216

…
:loudspeaker: Type of change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring

:scroll: Description
This implementation gives the ability to perform different actions when a message consumption fails, by providing a default implementation for the fail cases that can be overridden by the implementer
:bulb: Motivation and Context
#216
:green_heart: How did you test it?
Locally due to the lack of a testing suite
:pencil: Checklist

[x] I reviewed the submitted code
[ ] I added tests to verify changes
[ ] All tests passing
[x] No breaking changes

Looks nice. Have you tried overriding the onFailure to see whether it really works?


[ISSUE #2366] remove warning codes in shenyu-disruptor module

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2369 (059fe4a) into master (0e826ce) will decrease coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2369      +/-   ##
============================================
- Coverage     58.13%   57.96%   -0.17%
+ Complexity     3952     3945       -7
============================================
  Files           629      629
  Lines         18059    18056       -3
  Branches       1695     1693       -2
============================================
- Hits          10498    10466      -32
- Misses         6725     6748      +23
- Partials        836      842       +6

Impacted Files
Coverage Δ

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
82.35% <0.00%> (-11.77%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
69.90% <0.00%> (-11.66%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
54.73% <0.00%> (-9.48%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
8.69% <0.00%> (-4.35%)
:arrow_down:

...che/shenyu/admin/model/vo/SelectorConditionVO.java
96.07% <0.00%> (-1.97%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
54.54% <0.00%> (-1.82%)
:arrow_down:

...rg/apache/shenyu/plugin/hystrix/HystrixPlugin.java
74.07% <0.00%> (-0.93%)
:arrow_down:

.../plugin/grpc/loadbalance/AbstractLoadBalancer.java
69.56% <0.00%> (-0.33%)
:arrow_down:

...plugin/resilience4j/build/Resilience4JBuilder.java
47.61% <0.00%> (+2.16%)
:arrow_up:

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 68bbc7d...059fe4a. Read the comment docs.


Image resize for srcset

Sorry if this is silly question. I'm completely new in Laravel world.
Would be possible to resize images in Jigsaw to have ability to use them for img srcset tag?
Something like this one: https://medium.com/codefield-community/responsive-images-and-progressive-image-loading-with-laravel-a67d5d1f42c5 or here: https://docs.spatie.be/laravel-medialibrary/v7/responsive-images/getting-started-with-responsive-images/#generating-the-necessary-images but not on upload but... Before build?

Welcome to the Laravel world, @realcgslav 👋
This is not a silly question. Yes, this is possible with Jigsaw using its built-in Event Listeners. Within the bootstrap.php file you can add:
// bootstrap.php
$events->beforeBuild(function ($jigsaw) {
   // your code here that resizes images
});

ImageMagick is a popular Open Source tool with decades of great documentation and examples out there. Once you determine your file sizes and their location, you will be able to use them with the srcset attribute.
Here is a little more information about the srcset attribute

Hi @anthonyterrell,
I'm also wondering how to do this. I created a function that loops trough my collection of posts (which are .md files) and I want to replace all images with responsive images.
I want to replace  with
I wrote this:
$events->afterCollections(function (Jigsaw $jigsaw) {

    $posts = $jigsaw->getCollection('posts');

    foreach($posts as $post) {

        $dom = new DOMDocument();
        $dom->loadHTML($post->getContent());

        foreach ($dom->getElementsByTagName('img') as $img) {

            $src = $img->getAttribute('src');

            $tiny_src = str_replace('/assets/images/', '/images/tiny/', $src);

            $medium_src = str_replace('/assets/images/', '/images/medium/', $src);

            $img->setAttribute('src', $tiny_src);
            $img->setAttribute('srcset', $medium_src);
        }

        dd($dom) // Outputs the html I want
    }

});

But how can I publish my modifications to the build?

Publishing modifications can be done using the method ->setContent($domstring), i.e.:
$events->afterCollections(function (Jigsaw $jigsaw) {

    $posts = $jigsaw->getCollection('posts');

    foreach($posts as $post) {

        $dom = new DOMDocument();
        $dom->loadHTML($post->getContent());

        foreach ($dom->getElementsByTagName('img') as $img) {

            $src = $img->getAttribute('src');

            $tiny_src = str_replace('/assets/images/', '/images/tiny/', $src);

            $medium_src = str_replace('/assets/images/', '/images/medium/', $src);

            $img->setAttribute('src', $tiny_src);
            $img->setAttribute('srcset', $medium_src);
        }
        $post->setContent($dom->saveHTML());
    }

});


Install nightly builds

It would be nice if we could install the nightly builds instead of the stable ones. In our case we'd use it to ensure that we always are compatible with the latest Node master.
The nightly are available here: https://nodejs.org/download/nightly/index.json

Thanks.  I'll look into this along with the other version work like "LTS", "latest".  I suppose "nightly" would be a version option?

Is there any way for me to use nightlies today, perhaps by specifying a URL to the tar.gz or something?

Hello everyone. We released and updated the major tag with changes to install nightly builds. Could you please confirm that everything works as expected.
jobs:
  build:
    runs-on: ubuntu-latest
    name: Node sample
    steps:
      - uses: actions/checkout@v3
      - uses: actions/setup-node@v3
        with:
          node-version: '16-nightly'
      - run: npm ci
      - run: npm test

Hello everyone. For now I'm going to close the issue.


⚠ Bofrost.de has degraded performance

In 574d3db, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: Bofrost.de performance has improved in a6baca8 after 25 minutes.


cask-repair: use brew reinstall

Use brew reinstall instead of brew cask reinstall since the
latter is now deprecated.

With this change the cask installed properly and no warning was shown:
…
Submitting…
Updating cask locally…
==> Downloading https://musicbrainz.osuosl.org/pub/musicbrainz/picard/MusicBrain
Already downloaded: /Users/trolley/Library/Caches/Homebrew/downloads/d3eea81ef629bf43c9173d3f211b5bb24d4b2a3d94e864e5c005c77b47cf73ac--MusicBrainz-Picard-2.4.4.dmg
…


Docs: example in "DTO: Excluding fields" is not clear

Summary
Example:

Docs: https://docs.litestar.dev/latest/usage/dto/1-abstract-dto.html#excluding-fields
Code:

config = DTOConfig(
    exclude={
        "id",
        "address.id",
        "address.street",
        "pets.0.id",
        "pets.0.user_id",
    }
)

Later is specifies:

and "pets.0.id" and "pets.0.user_id" represent fields of the Pets objects nested within the list of User.pets.

and

Given a generic type, with an arbitrary number of type parameters (e.g., GenericType[Type0, Type1, ..., TypeN]), we use the index of the type parameter to indicate which type the exclusion should refer to. For example, a.0.b, excludes the b field from the first type parameter of a, a.1.b excludes the b field from the second type parameter of a, and so on.

But I still don't understand what pets.0.id means :)
Here are my questions:

Is pets.0 an index of an object? For example in [Pet(id=1, name="Dog"), Pet(id=2, name="Cat")] will both of them not have an id? Or only Dog?
This sentence confuses me even more:

Given a generic type, with an arbitrary number of type parameters (e.g., GenericType[Type0, Type1, ..., TypeN]), we use the index of the type parameter to indicate which type the exclusion should refer to

Because neither User or Pet is generic. They don't have any type params.
I don't have any proposals on how to fix this, because I don't understand how it works :)

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

Adding a partial explanation with a sample (and incomplete) model.
class Pets(Base):
    name: Mapped[str]
    user_id: Mapped[UUID] = mapped_column(ForeignKey("user.id"))

class User(Base):
    pets: Mapped[list[Pets]] = relationship(info=dto_field("read-only"))

UserDTO = SQLAlchemyDTO[User]
config = DTOConfig(
    exclude={
        "id",
        "address.id",
        "address.street",
        "pets.0.id",
        "pets.0.user_id",
    }
)
ReadUserDTO = SQLAlchemyDTO[Annotated[User, config]]

Is pets.0 an index of an object? For example in [Pet(id=1, name="Dog"), Pet(id=2, name="Cat")] will both of them not have an id? Or only Dog?

Both of them.

This sentence confuses me even more:

The 0 in pets.0.id is not indicative of the index in which it has to exclude an attribute, rather it indicates the index of the type in list[Pets] (because this is the type of User.pets), so 0 here refers to Pets.


Method mergeNewTranslations not work for me (Symfony 5.0.5, PHP 7.4.1)

Old project code (not work for new projects):
    public function create(array $names):Country
    {
        $entityManager = $this->getEntityManager();
        $country = new Country();

        foreach ($names as $locale => $name){
            $translation = $country->translate($locale)->setName($name);
        }

        $country->mergeNewTranslations();
        $entityManager->persist($country);
        $entityManager->flush();

        return $country;
    }

Now (working way):
    public function create(array $names):Country
    {
        $entityManager = $this->getEntityManager();
        $country = new Country();

        foreach ($names as $locale => $name){
            $translation = $country->translate($locale);
            $translation->setName($name);
            $entityManager->persist($translation);
        }

        $entityManager->persist($country);
        $entityManager->flush();

        return $country;
    }

Hi uginroot, thanks for your workaround.
I wanted to use that trait for a personnal symfony 5 project tonight, I will let you know if I face the same problem with mergeNewTranslations.
Regards,
t0FF

@uginroot may it be a problem with missing persist:cascade definition of your translation relation?
It is a wild guess as I still did not upgrade to v2. Hopefully Ill find a time to do it soon.

@uginroot could it be a problem with missing persist:cascade definition of your translation relation?

Maybe, but the cascade = {"persist"} setting is not specified in TranslationPropertiesTrait->translatable and TranslatablePropertiesTrait->translations property. And before, these settings were not indicated. I think this is due to the transition to Symfony 5. But I did not find out the reason. The solution above works for me.


ATSAME70Q21 - custom bootloader fails -jumps to application.

Discussed in https://github.com/zephyrproject-rtos/zephyr/discussions/60366
I am also working on similar application on  ATSAME70Q21.
I am using mcuboot as bootloader and helloworld as my application. Now I am facing similar kind of issue discussed in above thread. In my case, Bootloader is Jumping to the corresponding address of the application , after that  I am facing memory usage issue,

The following is (.lst) file (where it gets blocked)
/* do byte-sized copying until finished */
while (n > 0) {

4437ae:    1e43          subs    r3, r0, #1
4437b0:    440a          add    r2, r1
{
4437b2:    b510          push    {r4, lr}
while (n > 0) {
4437b4:    4291          cmp    r1, r2
4437b6:    d100          bne.n    4437ba <memcpy+0xc>
*(d_byte++) = *(s_byte++);
n--;
}
return d;

}
4437b8:    bd10          pop    {r4, pc}
*(d_byte++) = *(s_byte++);
4437ba:    f811 4b01     ldrb.w    r4, [r1], #1
4437be:    f803 4f01     strb.w    r4, [r3, #1]!
n--;
4437c2:    e7f7          b.n    4437b4 <memcpy+0x6>
can any help m out ?
@pavithvishnu are you be to jump to the application.
@Laczen can you help me out

@Aravindh1910, for my understanding: when you compile and flash hello_world without mcuboot everything works as expected ?

@Laczen  Yes, If I compile hello_world and flashed it without mcuboot it works!

@Laczen  and @pavithvishnu,
I have a findings below,

able to compile hello_world and Flash it without mcuboot it works.
able to compile and flash hello_world with mcuboot (without USB enabled) it works!.
able to compile and flash hello_world with mcuboot (with USB enabled) it doesn't worked!.

To summarize, if I enabled USB in Configuration, jump from mcuboot to Application doesn't work.
But, For my requirement I need to enable USB. USB configured as below.
filename : board_defconfig
CONFIG_USB_DEVICE_STACK=y
CONFIG_USB_DEVICE_HID=y
CONFIG_ENABLE_HID_INT_OUT_EP=y
CONFIG_HID_INTERRUPT_EP_MPS=256
May I know How to disable this before jump to application.
(FYI, usb_disable API also not works)

@jfischer-no, could there be something missing in mcuboot that causes this malfunction ?

@Aravindh1910  you are right, I have also tried with the same configuration  which you have mentioned above. I'm also facing the same issue. if usb stack is enable, jump to application is not working.


Polymorph Staging Site

Elegantt data. What's this?

➤ Craig MacGregor commented:
https://polymorph.navcoin.org:3000
needs this header editor plugin:
https://chrome.google.com/webstore/detail/modheader/idgpnmonknjnojddfkpgkljpfnnfcklj
import config:
{"title":"Polymorph","hideComment":true,"headers":[{"enabled":true,"name":"Authorization","value":"Basic c3RhZ2luZzoyelpyVVRJSm9rMll0Y0xOT1BkSlBwZU4=","comment":""}],"respHeaders":[],"filters":[],"appendMode":""}


Update GooseFS:  Support fuse lazy start and change worker daemonset to statefulset.

Ⅰ. Describe what this PR does
Support fuse lazy start and change worker daemonset to statefulset.
Ⅱ. Does this pull request fix one issue?

#1182
Ⅴ. Special notes for reviews
cc
@TrafalgarZZZ
@cheyang
@yangyuliufeng

done. cc @cheyang


Support for more device nodes with EBS volumes

Currently libstorage supports device nodes for EBS volumes within range /dev/xvd[f-p] which is in line with Amazon's recommendations. However, this is just a recommendation and typically it is not feasible for environments where volumes are used a lot more extensively. This is the case with any job scheduler that uses persistent storage such as Kubernetes or Nomad.
Kubernetes AWS device allocator uses a much broader device namespace /dev/xvd[b-c][a-z]. Related source code reference here: https://github.com/kubernetes/kubernetes/blob/master/pkg/cloudprovider/providers/aws/device_allocator.go#L80-L92
Now, this issue has been discussed previously at least in codedellemc/rexray#773. This pull request changes the device range from /dev/xvd[f-p] to the much larger one /dev/xvd[b-c][a-z]. And in order to prevent ghost device issues such as codedellemc/rexray#410 the device allocator iterates the devices in a random order.

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

@vtorhonen Any idea when will this get merged? This fix is of importance to us. 10 volume restriction for one agent node in a cluster is a hampering limitation. [Our agent nodes already have some number of EBS volumes assigned.]

@vtorhonen Any idea when will this get merged? This fix is of importance to us. 10 volume restriction for one agent node in a cluster is a hampering limitation. [Our agent nodes already have some number of EBS volumes assigned.]

Hello @kshitizbakshi, I haven't heard anything from the maintainers. I too would like to know when this can be merged.

@vtorhonen The libStorage project is being merged back into the REX-Ray project for the 0.10 release.  See notes on main project page.
We are open to discussing the proposal, but it would be great if you could rebase this over to the head of the RR project under /libstorage.
It might be more broadly applicable if the decision about the device paths was made an option. I think the decision for a smaller range was based on compatibility with more EC2 instances. I do recognize that K8s does support the broader range that you defined.

Hi @vtorhonen,
If you would file this PR against REX-Ray per @clintkitson's guidance and make your change one that can be enabled via a configuration property then I will have zero issue merging it.

Sure thing! It'll probably take a day or two to test it, but I'll create a PR once that's done.

@vtorhonen can you suggest how to use your build for testing?

I have now created a new PR for REX-Ray: https://github.com/codedellemc/rexray/pull/996
@miry If you want to test this I suggest you check out the latest REX-Ray build I made: https://github.com/vtorhonen/rexray/releases/tag/v0.10.0-lds. Just update the binaries on your environment and add useLargeDeviceRange: true to both client and server configs.
I'll close this PR so we can continue on the RR side.