refactor(windows): port C# Windows implementation to C++

Description
Port Windows implementation from C# to C++.
Port Windows code in the example app to use a C++/winrt implementation.
Remove the C# implementation.
Add some Windows tests to the example app, and run those as the Windows CI, replacing old Windows test in GitHub workflows.
Many thanks to the authors of the C# implementation and to those who have been updating/maintaining it, since this work is based on their work.
Fixes #1073
Compatibility

OS
Implemented

Windows
✅

Checklist

[X] I have tested this on a device/simulator for Windows, testing on react-native versions 0.61, 0.62 and 0.63.
[X] I added the documentation in README.md

Oooh exciting :-), thank you @jaimecbernardo hopefully not too distasteful despite the Sync method signatures being objectively kind of awful (and yet removed at anyone's peril)...
I read through and other than a question about .h vs .cpp, going back to the related issue, I don't have a windows rig I trust to run this so may I politely ask that you confirm this appears to run fine for you?
You worked through the example as well so I'm assuming yes, but have to ask :-)

Oh - one other item, we were pinned in the example on 0.62.x because of react-native-windows, it appears that the example could be bumped to 0.63.x ? https://github.com/react-native-device-info/react-native-device-info/blob/7f1d2a66acc2b828ec5b68330682ac6f89fd7227/example/package.json#L15
Related, I would ❤  for this script to be usable again https://github.com/react-native-device-info/react-native-device-info/blob/master/refresh-example.sh - as a maintainer I feel that tracking current stable in the example is both important and the biggest waste of my time ever, but I can't refresh safely right now

Thank you for the review :)

I read through and other than a question about .h vs .cpp, going back to the related issue, I don't have a windows rig I trust to run this so may I politely ask that you confirm this appears to run fine for you?

Yes, I've added a github workflow that builds and runs the example app on Windows and checks if each tab in the example app is correctly loaded. You can check the reasults in the PR's checks.

Oh - one other item, we were pinned in the example on 0.62.x because of react-native-windows, it appears that the example could be bumped to 0.63.x ?

Indeed it could be bumped.

Related, I would ❤ for this script to be usable again https://github.com/react-native-device-info/react-native-device-info/blob/master/refresh-example.sh - as a maintainer I feel that tracking current stable in the example is both important and the biggest waste of my time ever, but I can't refresh safely right now

Interesting, and I've gone and added some more test code in there 😅 I'll take a look at the script and see what else needs to be copied around/ran. With auto-linking on Windows since 0.63, I suppose it'll be easier.
You have to run it in macOS, though, because of pod install. Is this accurate?

Yeah, it is not going to be easy to make that work across all three platforms (win/mac/linux) that it runs on now - I primarily run it on macOS because of the pod install yes. It may be an intractable problem - I want full multi-platform support more than I want the script so I'm prepared to accept that but I will be sad. Updating examples is normally the same time it takes to merge 1-2 large PRs and I don't typically have a lot more time than that regularly

I'll give it a try at adding some instructions to the script pertaining to the Windows code.
I see there are some other changes that are not yet contemplated in the scripts, though, such as a non Windows test and an "app-extension" for iOS.

Indeed - entropy takes hold quickly!

Hi @mikehardy
I've updated refresh-example.sh with the windows specific commands needed.
I've also updated the example to react-native 0.63.3.
Regarding the "app-extension" for iOS, all I've done is manually cherry-pick this commit https://github.com/react-native-device-info/react-native-device-info/commit/2e8d47c9c863e1795089e93b188cd65f1e8f3886 and resolve the conflicts. The app still runs, but I have no idea what the "app-extension" is supposed to do.
After updating to react-native 0.63.3, I've tested it successfully on a Windows device, an Android device and an iOS emulator.

Sweet! The app extension is just there to make sure getFontScale works in app extensions, it was a bug and recent pr

I suppose the app building and running for iOS is a sufficient test, then? :)

The test should've run successfully. It seems to have failed while getting a NuGet package.
I've re-run it in my repo to make sure it should run fine.
Could you please have the action re-run, @mikehardy ?

Thanks! Yes, if the iOS example app builds the extension should be fine, I believe that is sufficient
I re-ran the action and it appears to be reproducibly not working:
Build failed with message D:\a\react-native-device-info\react-native-device-info\example\node_modules\react-native-windows\ReactWindowsCore\ReactWindowsCore-Items.vcxitems(231,5): error : This project references NuGet package(s) that are missing on this computer. Use NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=322105. The missing file is D:\a\react-native-device-info\react-native-device-info\example\windows\packages\boost.<IP_ADDRESS>\build\boost.targets. [D:\a\react-native-device-info\react-native-device-info\example\node_modules\react-native-windows\ReactWindowsCore\ReactWindowsCore.vcxproj]. Check your build configuration.

I have literally zero experience with building windows items so even the stack overflow that comes up https://stackoverflow.com/questions/22909506/this-project-references-nuget-packages-that-are-missing-on-this-computer is mysterious but this appears to be semi-common and something in the csproj may have bearing

I've re-ran it on my repo: https://github.com/jaimecbernardo/react-native-device-info/pull/2/checks?check_run_id=1321362933
And it worked successfully.
So my theory is that the github action is saving some state between runs in the same PR. It has ran before when react-native was 0.62 . Now it's running after updating to 0.63. My guess is that it's trying to use the NuGet packages that are there from 0.62.
I'll look into it on my repo and try to replicate these conditions.
I'll add a commit to fix once I understand what's the issue here and am able to fix it.

Works for me on react-native 0.62 and react-native-windows 0.62.14. Thanks @jaimecbernardo !

Nice, thanks for the success report @exotexot

Hi @mikehardy ,
Ended up removing the caching of node_modules between CI runs, since that cache was big and contained build artifacts for the modules, which were causing the build failures on following runs.
It passes now 😄

I've added a commit so that the workflow uses two caches, one for ./node_modules and another for ./example/node_modules. This fixes the CI issue this PR was having, while still keeping a cache.

Fantastic! I don't think I had anything else then, let me re-scan this fresh but I believe it is just on me to go "yep" and merge. Really fantastic stuff @jaimecbernardo ! I'm excited to have an up to date example again as a side benefit and (I think - I will test) my handy refresh script back to keep it up to date :)

@jaimecbernardo with apologies, I'm about to head out for vacation for 1 week starting tomorrow, and I've been hard up against other deadlines (forward porting react-native-firebase to all the breaking changes released in concert with the Firebase Summit that just happened...). I'll have to shelve this till I got back but it is still my expectation that I just need to review and it's good to go. Thanks for your patience

Thank you for the heads-up. Have a nice vacation! 😄

Hi @mikehardy ,
Any news on this? 😄
Would love to get it in so I can properly open https://github.com/react-native-device-info/react-native-device-info/pull/1122


Build Vue Application with Django REST API and Axios

Proposed title of article
Build Vue Application with Django REST API  and Axios
Proposed article introduction
When building web applications one may want to consume and display data from an API. This also applies to Vue.js applications where Django, a backend framework based on python is purely used as an API using a package called Django rest framework to build web applications together with Axios, a HTTP client library for fetching and displaying data.
To understand this, we will cover as much in this article to equip the reader with the understanding of how to combine Django and Vue.js to build web applications using Django rest framework together with Axios . The reader will understand how to configure rest API, Axios and their integration in building Vue web applications.
Key takeaways
The reader will be able to:

Understand how to build Django rest API using Django REST Framework.
How to combine Django and Vue.js.
How to build Vue application using Axios.

Article quality
This article will be of high quality different from the others, as it not only shows how to combine Django with Vue.js but also its implementation on how one can build Vue web applications using Django rest api together with Axios.
References
N/A
Conclusion
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

There are articles that already exist about these topics. Can you alter your article to become a tutorial about how to build a tye of web app using these technologies that hasn't been covered elsewhere?

There are articles that already exist about these topics. Can you alter your article to become a tutorial about how to build a tye of web app using these technologies that hasn't been covered elsewhere?

@louisefindlay23 done with the changes

So can you tell me what type of web app your tutorial will cover?

So can you tell me what type of web app your tutorial will cover?

@louisefindlay23 the tutorial will cover a Tod application just to show how we can implement the three technologies Django API, Axios and Vue.js


Texture preview filter regression

Godot version
v4.0.alpha6.official [e4f0fc50f]
System information
Ubuntu, Windows
Issue description
For projects that use mainly pixel art assets, all previews are displayed blurry now.
Godot 3.5 to the left, and 4 alpha to the right.

Since the filter setting is now no longer stored with the imported resource, perhaps it could be solved via an editor setting.
Alternatively, perhaps it could use different filters based on texture resolution.
Steps to reproduce

In the FileSystem panel, double-click any low res pixel art sprite to preview it the inspecor
Notice it being blurry

Minimal reproduction project
No response

Related to https://github.com/godotengine/godot/issues/57550 and https://github.com/godotengine/godot/issues/52332.

A few extra areas to tack on so it doesn't get missed in testing juuuust in case!
The blurriness affects shaders that are pulling color from a sampler2D which is small in size, like say a pixel color palette in a color replacement shader because it pulls the color from the blurred image~  (Including global shader params)
Using edit region on pixel art is a bit of a "guess which pixel is the edge" game~ xD


Track Epoch Start Time in the ClusterHistoryAccount

Is your feature request related to a problem? Please describe.
In order to better track APY of validators and stake pools on chain, we need to know the real world time that epochs started. The ClusterHistory account is the logical place to store this network-level field.
Describe the solution you'd like
Use the epoch_start_ timestamp from the Clock sysvar to read this field on chain for the current epoch and store it in the ClusterHistoryEntry for this epoch. This can be done in the copy_cluster_info instruction.
Describe alternatives you've considered
N/A
Additional context
N/A

jm :coffee:, could I be assigned to this one?
Additionally, do you have any hints for resolving the test_copy_cluster_info failure related to memory allocation?

@butonium yes! we're currently working on a fix for that, expect it to be resolved sometime next week.
In the meantime, if you get to this before the fix is in, you can comment out the parts of the copy_cluster_info instruction and test related to the slot_history account and total_blocks. Then we can merge the fix into your PR and test everything properly


[Feature]: Animate Contest Page Using Framer-Motion Library

Is your feature request related to a problem? Please describe.
The home page is currently animated, but the contest page lacks similar animations. To provide a consistent and engaging user experience, the contest page should be animated using the Framer-Motion library. This includes animating all elements such as headings, contest cards, tags, etc., similar to the animations on the home page.
Feature Requirements:

Use the Framer-Motion library to add animations to the contest page.
Animate the following elements:

Headings
Contest cards
Tags
Any other relevant elements

Expected Behavior:
The contest page should have smooth animations similar to those on the home page, enhancing the visual appeal and user experience.
Current Behavior:
The contest page is static and lacks animations, which creates an inconsistent experience compared to the animated home page.
Describe the solution you'd like

Integrate the Framer-Motion library into the contest page.
Implement animations for all specified elements, ensuring they are consistent with the home page animations.

Alternatives
No response
Additional Context

The animations on the contest page should mirror the style and feel of the home page animations.
Ensure that the animations do not impact the page's performance negatively.

Are you interested in contributing?

[X] I'd like to work on this issue

Code of Conduct

[X] I have used the search function to check if an issue already exists
[X] I agree to follow this project's Code of Conduct

Which program are you contributing in?
ssoc'24

Hey @harmeetsingh11  I would like to work on the issue kindly assign it to me.


wordpress.test is missing from slic container's /etc/hosts

When running Codeception acceptance tests that need to access the wordpress.test site within slic, the slic container is unable to resolve the hostname because it is missing from /etc/hosts. The hostname is, however, present in the wordpress container.

Released with 1.1.0.


Can't not find Variable:requireNativeComponent.

an error occurred when  I run react-native run-android.
"react-native-modal": "^9.0.0",

React Native Environment Info:
System:
OS: Windows 7
CPU: (4) x64 Intel(R) Core(TM) i5-6400 CPU @ 2.70GHz
Memory: 11.15 GB / 15.88 GB
Binaries:
Node: 8.12.0 - F:\install\nodejs\node.EXE
Yarn: 1.9.4 - C:\Users\GDYT\AppData\Roaming\npm\yarn.CMD
npm: 6.4.1 - F:\install\nodejs\npm.CMD

Your sourcemaps are probabled wrong as we don't use any native component and, therefore, don't make such calls.
It's much likely an other file is the origin of the error, you should check the exact line in your debugger to find a culprit.
Closing for the moment.

@rewieer Facing the same issue on an upgrade to 59.5. An old Modal.js file is added when running yarn.

We are depending on react-native as a peer dependency which means that when you download react-native-modal it doesn't have a react-native folder it in node_modules.
I don't think we can do anything about it because that's a weird error (I admit I had it at least once). But if there's to be a culprit it's either react-native or metro.

For us this turned out to be caused by https://github.com/listenzz/react-native-modal-translucent/issues/21, not react-native-modal

I updated react-native-modal-translucent to version 3.0.1, removed node_modules and package-lock.json, cleaned npm cache and reinstalled the whole s*** again. Worked fine.


Go Protocol Buffers APIv2

The Go Team announced a new api for protobuf in March. Since then, a large scale migration has taken place that has left https://github.com/golang/protobuf as a legacy v1 implementation.
We are currently using namely's protoc-all docker image and would like to begin generating proto using the new v2 api, so I was wondering if you had any plans to upgrade this repository - or if we should fork and implement the changes just for us.
It will be a little tricky because the recommended install path is now:
go install google.golang.org/protobuf/cmd/protoc-gen-go

grpc-gateway was also updated to v2: https://github.com/grpc-ecosystem/grpc-gateway/releases/tag/v2.0.0

Hi @Rossiar , @owenthereal ,
I believe both of these issues were addressed in latest image - 1.32_4.
Please let me know if you find otherwise.

@ido-namely thanks! works very nicely for our Message definitions - it may take some time for me to verify that it works well with our full grpc binding (separate repo/purpose), so I'm happy for you to close this issue and I will open a new one if I find anything amiss.


Paimon connector error

Steps to reproduce the behavior (Required)

CREATE EXTERNAL CATALOG paimon_catalog_fs
PROPERTIES
(
"type" = "paimon",
"paimon.catalog.type" = "filesystem",
"paimon.catalog.warehouse" = "s3://paimon/namespace",
"aws.s3.enable_ssl" = "false",
"aws.s3.enable_path_style_access" = "true",
"aws.s3.endpoint" = "http://<IP_ADDRESS>:9199",
"aws.s3.access_key" = "admin",
"aws.s3.secret_key" = "Dpjg40uIJr"
);
CREATE DATABASE test_water_platform;
#DROP Catalog paimon_catalog_fs;
SHOW CATALOGS;
SHOW CREATE CATALOG paimon_catalog_fs;
SHOW DATABASES FROM paimon_catalog_fs;
SET CATALOG  paimon_catalog_fs;
SHOW DATABASES;
USE  ods;
CREATE TABLE IF NOT EXISTS tax_catalog.ods.ods_bn_fee_info_zg(
fee_id	BIGINT,
fee_date	BIGINT,
fee_state	STRING,
fee_state_desc	STRING,
fee_type	STRING,
fee_type_name	STRING,
cancel_debt_date	BIGINT,
cancel_debt_mode	STRING,
cancel_debt_mode_desc	STRING,
write_off_batch	BIGINT,
amount_received	DOUBLE,
amount	DOUBLE,
settle_mode	STRING,
settle_mode_desc	STRING,
settle_date	BIGINT,
reckoner	BIGINT,
cancel_debt_clerk	BIGINT,
previous_balance	DOUBLE,
current_balance	DOUBLE,
user_name	STRING,
user_no	BIGINT,
address	STRING,
user_type	BIGINT,
meter_no	STRING,
account_type	STRING,
account_type_desc	STRING,
account_no	BIGINT,
water_bill_id	BIGINT,
update_date	BIGINT,
region_code	BIGINT,
caliber	BIGINT,
fee_code	STRING,
read_time	STRING,
tenant_id	BIGINT)PARTITIONED BY (tenant_id,read_time) WITH (
'bucket'  =  '12',
'scan.manifest.parallelism'  =  '10',
'write-mode'  =  'append-only',
'sink.parallelism'  =  '12',
'file.format'  =  'parquet')
2. INSERT INTO '....'
3. select * from xxx limit 10
Expected behavior (Required)
Normal Query
Real behavior (Required)
org.jkiss.dbeaver.model.sql.DBSQLException: SQL 错误 [1064] [42000]: Failed to call the nextChunkOffHeap method of off-heap table scanner. java exception details: java.io.IOException: Failed to get the next off-heap table chunk of paimon.[com.starrocks.paimon.reader.PaimonSplitScanner.getNext(PaimonSplitScanner.java:211), com.starrocks.jni.connector.ConnectorScanner.getNextOffHeapChunk(ConnectorScanner.java:101)]
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:133)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeStatement(SQLQueryJob.java:582)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.lambda$1(SQLQueryJob.java:491)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:189)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.executeSingleQuery(SQLQueryJob.java:498)
at org.jkiss.dbeaver.ui.editors.sql.execute.SQLQueryJob.extractData(SQLQueryJob.java:934)
at org.jkiss.dbeaver.ui.editors.sql.SQLEditor$QueryResultsContainer.readData(SQLEditor.java:3919)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.lambda$0(ResultSetJobDataRead.java:123)
at org.jkiss.dbeaver.model.exec.DBExecUtils.tryExecuteRecover(DBExecUtils.java:189)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetJobDataRead.run(ResultSetJobDataRead.java:121)
at org.jkiss.dbeaver.ui.controls.resultset.ResultSetViewer$ResultSetDataPumpJob.run(ResultSetViewer.java:5101)
at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:105)
at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: java.sql.SQLSyntaxErrorException: Failed to call the nextChunkOffHeap method of off-heap table scanner. java exception details: java.io.IOException: Failed to get the next off-heap table chunk of paimon.[com.starrocks.paimon.reader.PaimonSplitScanner.getNext(PaimonSplitScanner.java:211), com.starrocks.jni.connector.ConnectorScanner.getNextOffHeapChunk(ConnectorScanner.java:101)]
at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:120)
at com.mysql.cj.jdbc.exceptions.SQLError.createSQLException(SQLError.java:97)
at com.mysql.cj.jdbc.exceptions.SQLExceptionsMapping.translateException(SQLExceptionsMapping.java:122)
at com.mysql.cj.jdbc.StatementImpl.executeInternal(StatementImpl.java:764)
at com.mysql.cj.jdbc.StatementImpl.execute(StatementImpl.java:648)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.execute(JDBCStatementImpl.java:330)
at org.jkiss.dbeaver.model.impl.jdbc.exec.JDBCStatementImpl.executeStatement(JDBCStatementImpl.java:131)
StarRocks version (Required)

3.1.2

3.1.2 docker image maybe not include the latest bugfix in branch main

The version of 3.1.2 image maybe not including the latest bugfix of some special type adaption in branch main
You can have a try in branch main and it works good to me.
There is a part of the table can be queried normally, but some table queries have corresponding problems. On Dingding, I have given feedback and consulted mr Wuqian, which has not been solved for the time being

Any progress for this, Please? I have met the same problem when using version of 3.1.2

Any progress for this, Please? I have met the same problem when using version of 3.1.2

No one's paying attention for a while

I find out the root cause of the problem, the error threw because the type of the index of target InternalRow is not the same as the input logicalTypes[i], the error should only happen when use select * or select all the column from paimon table. And it's caused by following code in PaimonSplitScanner, I think we can remove the if condition directly to fix the problem, I test query like select * from paimon_table works fine after the fix. However, I don't do the full test yet.

fee_id

Referring to your experiment, I tried to query only one field, but this problem still occurs

fee_id

 Referring to your experiment, I tried to query only one field, but this problem still occurs

This error is different from the issue we talked about above, it happened when method open was called, instead of getNext  being called. You can refer to jni.INFO.log under be/log/ folder to get more details

fixed by https://github.com/StarRocks/starrocks/pull/35774


Illegal instruction error...

...even after building with
cmake -DRWKV_AVX=OFF -DRWKV_AVX2=OFF -DRWKV_AVX512=OFF -DRWKV_FMA=OFF -DBUILD_SHARED_LIBS=ON .
cmake --build . --config Release

Also, trying to use quantize.py, gdb gives:
Starting program: /usr/bin/python3.10 rwkv/quantize.py /llms/models/RWKV-4b-Pile-171M-20230202-7922.bin /llms/models/RWKV-4b-Pile-171M-20230202-7922.rwkv.cpp-171M-Q4_1_O.bin 4
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".
Program received signal SIGILL, Illegal instruction.
0x00007ffff75fcb20 in ggml_init () from /home/e2/compile/rwkv.cpp/librwkv.so
(gdb) display /i $pc
1: x/i $pc
=> 0x7ffff75fcb20 <ggml_init+400>:	vpxor  %xmm1,%xmm1,%xmm1
(gdb)

How about removing the -mf16c option from the build? There are a lot of them and you need to remove them all.
In addition, it would be useful to have an entry in cmake to remove -mf16c.

Commenting out lines with "-mf16c" in CMakeLists.txt and ggml/src/CMakeLists.txt appears to have worked on a different machine which had the same issue.  Still need to verify on the N4200 machine.


chore: add not required input for manuall run + set default to generated access-token

Description

add one more option for the workflow run on manual trigger and on schedule should be defaulted to generic github-access-token.

not needed anymore... thanks @tomaszbarwicki for verification


justify text, possible or not?

I want to justify text, but don't have any idea, is it possible to do with this library?
thank you

Read the README. It supports the center tag.


Add files via upload

 Quality gate: Failed
11 Issues

 6 Code issues
 5 Anti-patterns

Details

 Quality gate: Failed
11 Issues

 6 Code issues
 5 Anti-patterns

Details


Draft: Fix a time string parsing bug

With ffprobe version
ffprobe version 3.4.1-tessus Copyright (c) 2007-2017 the FFmpeg developers   built with Apple LLVM version 8.0.0 (clang-8<IP_ADDRESS>)
It prints media duration string like 0:12:04.829792. Its hour field has only one digit.
If you put the string through from_time_string() to to_time_string(), you would get a different string out at the end.
Python 3.10.4 | packaged by conda-forge | (main, Mar 24 2022, 17:45:10) [Clang 12.0.1 ] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import opentimelineio as otio
>>> s = "0:12:04.829792"
>>> t = otio.opentime.from_time_string(s, 24.0)
>>> t
otio.opentime.RationalTime(value=2995.92, rate=24)
>>> otio.opentime.to_timecode(t, 24.0)
'00:02:04:19'
>>> otio.opentime.to_time_string(t)
'00:02:04.829792'
>>>

The fix should be detect the colon separator and determine the field string length accordingly.

Could I ask, in addition to the python test, could you throw this into tests/test_opentime.cpp so we get C++ coverage as well by running make test
    tests.add_test("test to_string", [] {
        std::string time_string = "0:12:04";
        auto t = otime::RationalTime(24 * (12 * 60 + 4), 24);
        auto time_obj = otime::RationalTime::from_time_string(time_string, 24);
        assertTrue(t.almost_equal(time_obj, 0.001));
    });

A thing I did for the older PR was to be robust against a varying number of colons. ie. being able to parse 1:23 to mean one second and 23 frames, or 1:2:3 to mean 1 minute, 2 seconds, 3 frames.
the bit in the middle of the parse was simple and looked like
    std::vector<int> fields;
    int days = 0;
    int hours = 0;
    int minutes = 0;
    int seconds = 0;
    int frames = 0;

    std::string residual = timecode;

    try {
        while (residual.length() > 0) {
            size_t colon_pos = residual.rfind(':');
            std::string part = residual.substr(colon_pos + 1);
            fields.push_back(std::stoi(part));
            residual = residual.substr(0, colon_pos);
            if (colon_pos == std::string::npos)
                break;
        }

        auto it = fields.begin();
        if (it != fields.end()) frames = *it++;
        if (it != fields.end()) seconds = *it++;
        if (it != fields.end()) minutes = *it++;
        if (it != fields.end()) hours = *it++;
        if (it != fields.end()) days = *it;
    }
    catch(const std::exception& e) {
        *error_status = ErrorStatus(ErrorStatus::INVALID_TIMECODE_STRING,
                                    string_printf("Input timecode '%s' is an invalid timecode, exception: %s",
                                                  timecode.c_str(), e.what()));
        return RationalTime::_invalid_time;
    }

with the rest of the calculation following in the obvious manner. Maybe we could do that here as well.
it also validated that the frames field was legal:
    const int nominal_fps = static_cast<int>(std::ceil(rate));
    if (frames >= nominal_fps) {
        *error_status = ErrorStatus(ErrorStatus::TIMECODE_RATE_MISMATCH,
                                    string_printf("Frame rate mismatch.  Timecode '%s' has "
                                                  "frames beyond %f", timecode.c_str(),
                                                  nominal_fps - 1));
        return RationalTime::_invalid_time;
    }

FWIW, here's how ffprobe constructs the time string:
https://github.com/FFmpeg/FFmpeg/blob/83e1a1de8833845224948e5d002355c03dd117d5/fftools/ffprobe.c#L412

Ah, good reference, the ffprobe time string is h:m:seconds with no frames.

Codecov Report

Merging #1293 (8bcbbc0) into main (fb0b86d) will decrease coverage by 0.02%.
The diff coverage is 68.42%.

@@            Coverage Diff             @@
##             main    #1293      +/-   ##
==========================================
- Coverage   86.14%   86.12%   -0.03%
==========================================
  Files         196      196
  Lines       19813    19831      +18
  Branches     2314     2314
==========================================
+ Hits        17068    17079      +11
- Misses       2181     2188       +7
  Partials      564      564

Flag
Coverage Δ

py-unittests
86.12% <68.42%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/opentime/rationalTime.cpp
82.60% <60.00%> (-3.00%)
:arrow_down:

tests/test_opentime.py
99.73% <100.00%> (+<0.01%)
:arrow_up:

src/opentime/errorStatus.h
87.50% <0.00%> (-12.50%)
:arrow_down:

Continue to review full report at Codecov.

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

Closing this in favor of @meshula 's change in https://github.com/AcademySoftwareFoundation/OpenTimelineIO/pull/1297


422 error should be form validation

 "message": "Validation failed: Name: greater than max"
When creating a new kube if the name is too long it returns a 422 error after closing the form. This should be a form validation or it should allow editing and resubmitting the form

Hi, @bfosberry, thanks for bringing this up, in the new beta, the form field no longer allows typing anymore than the max characters, and the regular expression restricting this is displayed below the text box, so this is no longer a problem. I apologize for the delay in getting to you; let me know if this fixes the issue!

Hi, @bfosberry, I'll go ahead and close this; let me know if anything else needs to be done.


Update policy assignment for deny public IP

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to a problem?
On the Enterprise scale repo a PR changes the policy and assignment for deny public ip. Please modify ALZ-Terraform to reflect this change.
Describe the solution you'd like
N/A
Additional context

Closed by #523


Update README to clarify project status

cc @Fishrock123?

Thanks for all the fish.

After a lot of works it seems Jenkins works ok for us, so... yeah.


Keyboard Shortcut for Paste as Plain Text

Microsoft PowerToys version
0.68.0
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
Paste as Plain Text
Steps to reproduce
In the latest Insider's Dev version 25309 of Windows 11, the default keyboard shortcut for the new Volume Panel is Win + Ctrl + V.
✔ Expected Behavior
Paste in Place
❌ Actual Behavior
Opened new Volume Panel
Other Software
Windows 11 ver. 25309

/dup #24491


Add a different backbone other than Mobilenetv2?

Hey, great work on MODNet!
I'm looking to try out modnet with a HRNet backbone for better results. Would you happen to already have the model function for HRNet that I can use instead of Mobilenetv2? If not, How do I go about adding this?

While you wait for the author to reply, I would suggest you to try Pytorch Image Models Library! (pip install timm)
The following link provides a super cool demo of how you can use different backbones (currently only CNN based) and get multi-scale features like the author used in his implementation :)
https://rwightman.github.io/pytorch-image-models/feature_extraction/#:~:text=Size([2%2C 1024])-,Multi-scale Feature Maps (Feature Pyramid),-Object detection%2C segmentation

Hi, @tarunn2799, thanks for your attention.
Sorry that we did not use HRNet as the backbone, but we have tried EfficientNet and obtained better results.
If you want to change the backbone, you have to retrain the model. You may try the library shared by @mylifeasazucchini to replace the following code that define the backbone:
https://github.com/ZHKKKe/MODNet/blob/b3b6964a58562c2220b0d6a0608689722f927abd/src/models/modnet.py#L216

Why not consider using the MobileNetV3 network as the backbone in the MODNet model instead of MobileNetV2?


How to implement nested function calling using VoicePipelineAgent?

Hello LiveKit team @davidzhao @dsa,
We’re working on a project that utilizes LiveKit and have encountered a couple of challenges that we were unable to resolve with the existing documentation and examples. Specifically, we’d appreciate your guidance on the following:
1. Nested Function Calling: We’re looking for an example or best practices on handling nested function calls within LiveKit. We’d like to understand how to manage the flow and error handling when one function needs to call another, especially with a focus on minimizing latency and optimizing performance.
2. Dynamic Function Addition in FunctionContext: Additionally, we’d like to see an example of dynamically adding functions in the FunctionContext class. This functionality is crucial for our application, but we’re unsure of the correct approach to implement it.
Thank you very much for your assistance and for all your hard work on LiveKit!

Hey @MartinKovachev, for 1. we do support nested function calls, you can increase max_nested_fnc_calls a higher value.
we left default low because most LLMs other than OpenAI do not handle it properly.
for 2. we are working on improvements in this API to make this simpler. this will be in 0.12.x, to be released later this month

Thank you @davidzhao


Improve location of large config chunks that support secondary data loading in edit pages

https://github.com/rancher/dashboard/pull/6860 brings in large configuration chunks to support it's mixin
this clutters up the data object

Closing due to dev-validate


how to use in react-native

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
v3.2.0
Please ask your question
How to use in react-native, have some demos?

I don't believe that anyone in the Shaka Player team has any experience with React Native.

Shaka Player uses the open web standards MediaSource Extensions and Encrypted Media Extensions, neither of which are available in React Native run-time since its not a browser environment. So, i'm pretty sure you wouldn't be able to use Shaka-player in React native.
It might be possible to write a React Native bridge for https://github.com/google/shaka-player-embedded but its currently only supported on iOS.

Thanks, Vasanth.  If React Native has no MediaSource Extensions, then Shaka Player is probably a poor fit for that environment.  @vaseala, does that answer your question?


Redirected you too many times error

I got this error
[...].execute-api.ap-southeast-1.amazonaws.com redirected you too many times.
when following the tutorial.
And the addressbar will redirect you to:
https://[...].execute-api.ap-southeast-1.amazonaws.com/prod/resize?key=//////////130x130/blue_marble.jpg
It turns out that the error is because in the code const key = event.queryStringParameters.key; already has the slash e.g.: \130x130\blue_marble.jpg and later the code redirect to the location that add one more slash ${URL}\${key}

Hi! Hmm, if I recall correctly, the redirect rule should pass the object without a leading slash. How do you have your redirect rule configured?

Sorry, I don't understand the fix or the issue. I just created the function from the provided SAM template and I did not encounter the same problem. Can you compare your setup to one created via the template and let me know what the difference is, please?


R2DBC API: Provide Statement.bindNull(…) accepting integer index

From @mp911de on June 7, 2018 15:23
The Statement API should accept null bindings using an integer index.
Copied from original issue: r2dbc/r2dbc-client#4

In this case, I agreed with something the ADBC team says, and that's that rather than putting a JDBC/ADBC abstraction around parameter binding, just use the existing parameter binding from the databases themselves, to be consistent with databases across languages rather than consistent with Java across databases.
So, PostgreSQL only supports $# binding parameters.  We could write a mapping from something else onto that, but I'd actually made the decision to support only that scheme since that's what PostgreSQL supports natively.
Let's definitely discuss this a bit.


pkg/endpoint: remove WaitGroup return value from TriggerPolicyUpdatesLocked

This wait group is no longer used since conntrack cleanup is not done based on
policy recalculation per endpoint.
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

test-me-please


Does not conform to protocol SMSegmentViewDelegate

Hi, my view controller is UIViewController but I cannot assign delegate to my UIViewController

@amycheong19 I think you should be able to get rid of the error by adding the following delegate method in your view controller:
func segmentView(segmentView: SMSegmentView, didSelectSegmentAtIndex index: Int) {
}

Thank you!


Single 97 KB texture loaded once results in 83 copies of the texture and 163 MB total memory usage in the debugger "Video RAM" tab

Godot version
4.0.2
System information
Windows 10
Issue description
Loading a texture causes dozens of copies of the texture to appear in the Video RAM tab in the debugger. The total reported Video RAM usage of 163 MB is 1600x the size of the 97 KB texture file.

Commenting out the preload() line in the repro project reduces the total Video RAM usage to 1.5 MB.
Steps to reproduce

Load a png resource
Run project and look at Debugger > Video RAM

Minimal reproduction project
texture-video-ram-usage-repro.zip

Duplicate of https://github.com/godotengine/godot/issues/76110 (this is fixed in 4.1.dev3 and 4.0.3.rc1).


nitweb: show mentities details

Add more content to mentities pages.
Packages list inner groups

http://nitweb.moz-code.org/package/core

Groups show parent and list inner groups/modules

http://nitweb.moz-code.org/group/core>
http://nitweb.moz-code.org/group/core>collection>

Modules show imported modules, introduced classes and redefinitions

http://nitweb.moz-code.org/module/core::array

Classes show super classes, introduced properties, redefined properties

http://nitweb.moz-code.org/class/core::Array

Bonus, all lists are filterable by raw string or visibility.

Very cool.
By the way, http://nitweb.moz-code.org/property/core::Object::SELF is very long to load because http://nitweb.moz-code.org/api/entity/core::Object::SELF is 12MB of JSON (c.f. #1563)

+1

Reroll: fixed whitespaces in html

@privat I have no out of the box solution for SELF.
Maybe mark all redefs as is_fictive but we still want the redef information for every type in the documentation.
One quick, but no so good, solution would be to create a special rule for SELF.
One other way to relieve the browser/server stress would be to paginate the results and load pages as you scroll down.

SELF is special, but http://nitweb.moz-code.org/api/entity/core::abstract_text::Object::to_s is still 250KB


Fix some misspellings, remove repeated: and

Signed-off-by: Piotr Kasprzyk<EMAIL_ADDRESS>

Looks good. Thanks!

Thank you!


Drag and drop trc mot and c3d files

Backend is needed only once to login

For your review @AlbertoCasasOrtiz

Thanks much


Leader board name search buggy

Hi, Jan and I were looking for users in the missingmaps team on the leader board but having trouble finding some users.
For example the user Piskvor is not found using 'piskvor' or 'Piskvor' but is found if I just use 'iskvor' see below.
We are not finding the users pschonmann and Datin which should be in the results for missingmaps team as well.

This is the same for my username "Giblet" maybe something to do with the uppercase character to start?

I thought maybe that too, but pschonmann for sure is an all lowercase osm username.
https://www.openstreetmap.org/user/pschonmann

@giblet22 I am not sure now, you can go to mm team listing and try and type in any of the names that start with capital letters and they are not found in the search.
I guess this is just the same as issue #382 but I thought that was fixed. Sorry just remembered that issue existed.

In order to improve performance, the leaderboards switched to using a paginated API endpoint (500 users at a time rather than the full 10k+, more as time goes on). Since search is implemented within the browser, if a user is not within the first 500 (more if you've scrolled and triggered pagination) results (according to whichever sorting criteria is active), s/he won't be found.
A potential fix to this is to create a "search within specific leaderboard" endpoint in the API which returns contextual results (+/- 50 users within the same sort order).

@mojodna I still think there is a case issue going on as you can see the listing of users and if you try to find any of them who have a capital letter as the first letter in their name, you can not find them via the search box. Just visit the leaderboard listing, pick any user with a capital letter first name and try to find them via the search box on the page. Hopefully the below gif illustrates that.

Ah, I think I know why that might be the case; usernames were previously forced to lowercase (with unintended consequences) and there's probably something sticking around that lowercases the search query. If so, it should be a quick fix.


Current priorities

Meta-issues
These are some project meta issues that would be nice to get before 1.0

CLA (#200, waiting for apinf/platform#2690)
License (#198) (cancelled)
Code of conduct (#193/#208) (merged)
Semicolon removal (#217/#219)

Post-1.0 priorities

Helping the user fill the form

Placeholders (#105) and help texts (#170)
Input validation (#171)

OpenAPI space integration, assuming we get it to a usable level (#120)
Togglable split UI (#92), maybe already for 1.1

This issue can be closed after the 1.0 release.

1.0 is nearly here, so I'll close this.


Add android shortcut to record a note

Is your feature request related to a problem? Please describe.
Other feature would be a widget button to speech to text, instead of having to say ok google
Describe the solution you'd like
A shortcut to record a note from android

Apparently there is a way to customize a phrase to open a shortcut so this becomes more important!


How do I get the same git gutter in Doom Emacs?

The git gutter on the left hand side looks amazing! I'm using my own configuration and would love to have the same visual effects. So far I'm using the diff-hl package but can't have it look as good as the indicators in doom emacs.
Which packages do I install and where can I find the corresponding configurations that you use?

Doom users the git-gutter and git-gutter-fringe packages. They are configured in the :ui vc-gutter module.
You'll find the fringe bitmap definitions at the bottom of the module's config.el
Hope that helps!

@hlissner  Thanks for the reply! I'll close this issue and look into your answer.


Streamline Solver Selection

Issue Description
Currently, the Simulation class handles solver selection and instantiation directly, leading to some code duplication and potential for errors. The goal of this enhancement is to improve flexibility, maintainability, and testability by introducing a factory pattern to abstract solver interactions.
Current Workflow
Simulation class tightly coupled to Gurobipy The current implementation heavily relies on Gurobipy, making it difficult to switch or add new solvers seamlessly. There is also a temporary file creation: writing and deleting an MPS file for HiGHs introduces unnecessary overhead and potential I/O issues.
Proposed Solution
The idea is to adopt the Factory pattern.

Create an abstract base class or interface (SolverInterface) that defines the common methods for interacting with solvers (e.g., add_variables, add_constraints, optimize). This probably is a lot of work.
Implement concrete classes (e.g., GurobiSolver, HighsSolver) that inherit from SolverInterface and provide the specific implementation for each solver
Create a factory class (SolverFactory) responsible for instantiating the appropriate solver object based on configuration or user input

Benefits of Factory Pattern

Centralized solver management makes it easier to add, remove, or modify solver implementations without affecting the core simulation logic
Eliminates conditional logic and temporary file handling within the Simulation class
Simplifies unit testing of the Simulation class
Adheres to the open/closed principle by allowing the addition of new solvers without modifying existing code.

Additional Considerations:

Not sure if this is the best way. Any ideas?

Closed with the following commits:

e1eb3d673b6abbae9924860ce638c0b11971a6c2
99e1e04f725f510df141177b2265dbb709634af7


!wip refactor(queue-manager): move pending swap to rango-types

We need to remove wallet-type RF-586 and then finalize and merge this pr.

fixed in https://github.com/rango-exchange/rango-client/pull/468


Add toml update support for langserver

Purpose

Add toml update support for langserver

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Merging without test due to broken test cases

Merging without test due to broken test cases


Style Shopping Cart Page

Here's the shopping cart page of Freshbey

The position of the elements is not perfectly aligned. Style it as you like. But follow the color theme!
React Bootstrap is used for styling. Also, the Zephyr theme is used from Bootswatch.

When raising your PR, attach the screenshot of the finished shopping cart page.

You can use img-fluid.

I'd like to work on this!

Sure, Go ahead!

Does this look good?

Nice! Whether it is responsive on smaller screens?

Yep! Here is how it looks on several different screen sizes.
XL:

L:

M:

S:

Hey i would like to style this cart page for you ....can you describe me how you want it too look ?

Seems like you enjoyed @emilylubkert work...  is this issue ready to be closed?  Or did you have more in mind @roopeshsn ?


[SW-1347] Move permissions vuex state to composable

@subhod-i

Existing permissions are not being considered.

Thank you for finding this out. The issue is fixed now. The reason was the lack of state synchronisation between the extension and the background. So background didn't see the permissions added within the extension app.

@CedrikNikita , @subhod-i - I requested regression testing for all stuff related to signing: https://github.com/superhero-com/superhero-wallet/issues/2419#issuecomment-1774711274


Roll Flutter Engine from b57a044ed10f to 29f30f0fdea7 (2 revisions)

https://github.com/flutter/engine/compare/b57a044ed10f...29f30f0fdea7
2024-07-01<EMAIL_ADDRESS>Roll Skia from e27104eaef37 to 8af866a56bce (4 revisions) (flutter/engine#53661)
2024-07-01<EMAIL_ADDRESS>Roll Skia from 34be7c830184 to e27104eaef37 (1 revision) (flutter/engine#53659)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Value Assignment by specific indices error in Tensorflow (Assign values using mask)

Click to expand!
Issue Type
Feature Request
Source
source
Tensorflow Version
tf 2.8
Custom Code
Yes
OS Platform and Distribution
Windows 10
Mobile device
No response
Python version
3.10.2
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
Hi! I'm trying to build a custom loss function for my model, but whenever I try to convert Tensors into .numpy() arrays with run_eagerly = True, it gives "WARNING: Gradients do not exist for variables ...". So I debugged other custom loss functions implemented using tensorflow. But in my case, I need to apply to mask and splitting index arrays and then use those arrays as indices to apply some sort of arithmetic functions using broadcasting. But I retrieved indices lists after masking, but I just have to access those indices and add specific functions. But I found no way in TensorFlow to implement that in a vectorized way.

Standalone code to reproduce the issue
In NumPy, I can access it using:
error[i, j] = error[i, j] * 5
But TensorFlow does not provide any support related to this.
Also, I could not implement the custom_loss function by converting arrays to NumPy because it makes Gradient None. And it gives "WARNING: Gradients do not exist for variables ...".

Relevant log output
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Input In [193], in <cell line: 1>()
----> 1 error[i, j]

File ~\AppData\Local\Programs\Python\Python310\lib\site-packages\tensorflow\python\util\traceback_utils.py:153, in filter_traceback.<locals>.error_handler(*args, **kwargs)
    151 except Exception as e:
    152   filtered_tb = _process_traceback_frames(e.__traceback__)
--> 153   raise e.with_traceback(filtered_tb) from None
    154 finally:
    155   del filtered_tb

File ~\AppData\Local\Programs\Python\Python310\lib\site-packages\tensorflow\python\ops\array_ops.py:899, in _check_index(idx)
    894 dtype = getattr(idx, "dtype", None)
    895 if (dtype is None or dtypes.as_dtype(dtype) not in _SUPPORTED_SLICE_DTYPES or
    896     idx.shape and len(idx.shape) == 1):
    897   # TODO(slebedev): IndexError seems more appropriate here, but it
    898   # will break `_slice_helper` contract.
--> 899   raise TypeError(_SLICE_TYPE_ERROR + ", got {!r}".format(idx))

TypeError: Only integers, slices (`:`), ellipsis (`...`), tf.newaxis (`None`) and scalar tf.int32/tf.int64 tensors are valid indices, got <tf.Tensor: shape=(4797,), dtype=int64, numpy=array([ 0,  0,  0, ..., 26, 26, 26], dtype=int64)>
I tried using other functions provided by TensorFlow too, but those did not work.

@Muhammad-Kaleem-Ullah
Can you please elaborate about your feature and please specify the use cases for this feature. Thanks!

I'm building a custom loss function while converting Tensors into NumPy arrays by using .numpy(), which results in "Gradients loss". So, I must have to use Tensorflow to manipulate data. Here is what I'm trying:
error = y_true - y_false
print(y_true.shape, y_pred.shape)
print(error.shape)
print("Error values: ", error)

Output:
Output: (10, 1000), (10, 1000)
(10, 1000)
Error values: <tf.Tensor: shape=(10, 1000), dtype=float64, numpy= array([[-10, 0, 8, ..., 3, -1.5, -2.5], ..., [ 2.5, 8 , 6.5, ..., 5.5, 3.5, -0.5]])>

Then, I apply the mask to retrieve indices where y_true > 5 & y_false <=0 to punish these errors.
mask = tf.where(y_true > 5)
i = mask[0]
j = mask[1]
print(i[:5])
print(j[:5])

Output:
(<tf.Tensor: shape=(5,), dtype=int64, numpy=array([0, 0, 0, 0, 0], dtype=int64)>,
<tf.Tensor: shape=(5,), dtype=int64, numpy=array([19, 26, 28, 35, 39], dtype=int64)>)

I'm facing a problem here below when I try to apply this trick (like in NumPy) to assign values against the mask.
error[i, j] = error[i, j] * 1.5
What I want is to replace with new values of error in a specified position after executing the above code and get values like:

Error values: <tf.Tensor: shape=(10, 1000), dtype=float64, numpy= array([[-10, 0, 16*, ..., 3, -1.5, -2.5], ..., [ 2.5, 16* , 13*, ..., 11*, 3.5, -0.5]])>
I'm trying to build a custom loss function for my model, but whenever I try to convert Tensors into .numpy() arrays with run_eagerly = True, it gives "WARNING: Gradients do not exist for variables ...". So I debugged other custom loss functions implemented using TensorFlow. But in my case, I need to apply to mask and splitting index arrays and then use those arrays as indices to apply some sort of arithmetic functions using broadcasting. But I retrieved indices lists after masking, but I just have to access those indices and add specific functions. But I found no way in TensorFlow to implement that in a vectorized way.

error = y_true - y_false
print(y_true.shape, y_pred.shape)
print(error.shape)
print("Error values: ", error)
Output: (10, 1000), (10, 1000)
(10, 1000)
Error values: <tf.Tensor: shape=(10, 1000), dtype=float64, numpy= array([[-10, 0, 8, ..., 3, -1.5, -2.5], ..., [ 2.5, 8 , 6.5, ..., 5.5, 3.5, -0.5]])>
mask = tf.where(y_true > 5)
i = mask[0]
j = mask[1]
print(i[:5])
print(j[:5])
Results:
(<tf.Tensor: shape=(5,), dtype=int64, numpy=array([0, 0, 0, 0, 0], dtype=int64)>,
<tf.Tensor: shape=(5,), dtype=int64, numpy=array([19, 26, 28, 35, 39], dtype=int64)>)
In NumPy, I can access it using:
error[i, j] = error[i, j] * 5
What I want is to replace with new values of error in a specified position after executing the above code and get values like:
Error values: <tf.Tensor: shape=(10, 1000), dtype=float64, numpy= array([[-10, 0, 16*, ..., 3, -1.5, -2.5], ..., [ 2.5, 16* , 13*, ..., 11*, 3.5, -0.5]])>
But when I try to execute this as Tensors, it gives the following error:
TypeError                                 Traceback (most recent call last)
Input In [193], in <cell line: 1>()
----> 1 error[i, j]

File ~\AppData\Local\Programs\Python\Python310\lib\site-packages\tensorflow\python\util\traceback_utils.py:153, in filter_traceback.<locals>.error_handler(*args, **kwargs)
    151 except Exception as e:
    152   filtered_tb = _process_traceback_frames(e.__traceback__)
--> 153   raise e.with_traceback(filtered_tb) from None
    154 finally:
    155   del filtered_tb

File ~\AppData\Local\Programs\Python\Python310\lib\site-packages\tensorflow\python\ops\array_ops.py:899, in _check_index(idx)
    894 dtype = getattr(idx, "dtype", None)
    895 if (dtype is None or dtypes.as_dtype(dtype) not in _SUPPORTED_SLICE_DTYPES or
    896     idx.shape and len(idx.shape) == 1):
    897   # TODO(slebedev): IndexError seems more appropriate here, but it
    898   # will break `_slice_helper` contract.
--> 899   raise TypeError(_SLICE_TYPE_ERROR + ", got {!r}".format(idx))

TypeError: Only integers, slices (`:`), ellipsis (`...`), tf.newaxis (`None`) and scalar tf.int32/tf.int64 tensors are valid indices, got <tf.Tensor: shape=(4797,), dtype=int64, numpy=array([ 0,  0,  0, ..., 26, 26, 26], dtype=int64)>

There are multiple reasons for Gradients to become None, in your case it might be due to the operation you are doing could be outside Tensorflow operation or tf.Variable gets replaced with tf.tensor which is used for gradient calculation.
More details on this can be found here.
Regarding updating specific index values, you can take a look into tf.Variable operation scatter_update which assigns tf.IndexedSlices values.
Let me know if this helps. Thanks!

I used run_eagerly in model compile and then call the custom loss function. Then, I just used the above calculation for Tensors. And I got stuck there. Regarding your suggestion to use scatter_update, it did not work here.

I think you want a combination of tf.gather_nd and tf.tensor_scatter_nd_update:
# Get the list of indices for some condition
mask = tf.where(y_true > 5)

# Extract the values from another tensor at those indices
vals = tf.gather_nd(error, mask)

# Update values at the supplied indices based on the old values
tf.tensor_scatter_nd_update(error, mask, 5 * vals)

@Muhammad-Kaleem-Ullah , Could you please refer the above comment and let us know if that helps.
For more details refer, tf.gather_nd, tf.tensor_scatter_nd_update

Or, you could use the multiplexing mask:
new_error = tf.where(y_true > 5, 5 * error, error)

An RFC to address this has been up for a bit.  Adding a link here in case anyone wants to provide input.
Comments are invited to the pull request. You can view the design doc here, and also leave comments inline on the document source.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Provide the ability to add custom health checks that precede and follow ClickHouse reconfiguration/upgrade

We had a recent discussion with a user who has the following use case. They have an operator that watches for replication lag on ClickHouse and only schedules upgrade when it's below a certain level. They were wondering if the operator could help them implement logic that amount to "guard conditions" on operator upgrades. It could work as follows:

Permit a ClickHouse replica upgrade operation to start when replication lag is below a "reasonable" level.
Confirm that the replication lag is back to a reasonable level before moving to the next replica.

In this case the guard condition is replica lag but it's likely that there would be others. They and other users would presumably want to customize the conditions.
It seems as if this could be implemented in a general way by adding a custom health check section to CHI instances that contain one or more SQL queries to run before and after upgrade. If the check(s) pass, the upgrade could proceed. If they fail the operator would wait.
There are some questions with this kind of guard that we need to contemplate.

How would other services learn the status of the cluster health checks?
What happens if replica lag increases suddenly during an upgrade? It could block an incomplete operation indefinitely.
Does it just apply to upgrade or to any reconfiguration that causes a pod restart?
Should we report these health check results in our exported metrics? It would be important to see these visually in, say, Grafana when kicking off an upgrade or diagnosing problems.

This is a very interesting topic that requires detailed discussion. However, these are my brief thoughts on the matter.

How would other services learn the status of the cluster health checks?

"Basic or k8s" health checks are run by k8s and clickhouse-operator does not know results of the health checks on-the-fly. These statuses can be fetched from each Pod, though, so "other services" can monitor Pods directly. List of clickhouse pods is provided by the clickhouse-operator in CHI.Status section
Some kind of "custom health checks" explicitly described in CHI manifests may be introduced, but this requires a good amount of design. Main questions to answer should be like "should SQL-only checks be allowed or custom shell commands as well" and "how to expose results of custom checks". As to results exposure, this can be done, for example, via REST endpoint, along with metrics being already exported by the operator. Speaking about metrics arises the question "do we need additional explicit health checks or metrics can be enhanced to serve this way?"

What happens if replica lag increases suddenly during an upgrade? It could block an incomplete operation indefinitely.

Clickhouse-operator has timeouts for host reconcile processes. Some additional or "general" timeout(s) should be introduced, if we are talking about waiting for external permission to reconcile a host.

Does it just apply to upgrade or to any reconfiguration that causes a pod restart?

There is no such a special case as "upgrade" at the moment, operator performs general reconcile. I'd rather avoid narrow cases and prefer to go with more generic approach. Introducing some kind of "external hooks" may be a proper way to go. Details are to be specified additionally.

Should we report these health check results in our exported metrics? It would be important to see these visually in, say, Grafana when kicking off an upgrade or diagnosing problems.

This question is the good starting point to think on how:

To enhance metrics functionality and introduce "custom metrics" that can be specified in CHI manifest and provide flexible mechanism to run complex tasks on the clickhouse host (having in mind questions like: SQL-only or to allow shell scripts as well)
Provide hooks (general external or to custom metrics only) that will control the operator's behavior during reconcile. Say to block host reconcile until specified metrics allows to proceed.

Anyway, this is a big topic to discuss and to plan, which we can start to.

Just a quick idea - option to launch k8s job from user provided template and wait for its completion before starting ch pod reconciliation (or some condition). We might augment this job with well-known env variables (such as pod to be reconciled, all cluster pods). Important advantage could be that we will move all the configuration (secrets, podFailurePolicies and much more) to k8s and have build-in way to watch/debug it. Job could even push metrics/logs to collector during its runtime.

👋 Hey folks! I'm an engineer over at Mux, the company asking y'all for this feature 🙌
Overall I think what y'all are saying makes sense, for some clarity I think we only need the ability to execute a single SQL statement before progressing onto the next pod to ensure our strict latency requirements are met.
For example, a query we'd want to run before a pod rolls is something like this (used a random value threshold since I'm not 100% sure what replication lag threshold we'd want to block progress on):
SELECT value FROM system.asynchronous_metrics WHERE metric = 'ReplicasMaxAbsoluteDelay' AND value > 60

I was thinking it could be implemented similar to how Prometheus alerts work, where any data returned means a failure: https://prometheus.io/docs/prometheus/latest/configuration/alerting_rules/

Whenever the alert expression results in one or more vector elements at a given point in time, the alert counts as active for these elements' label sets.

So you would simply check
It looks like we already have some logic that runs SQL before it proceeds with the reconcile here: https://github.com/Altinity/clickhouse-operator/blob/d5f265fb4773ec4622b2af13fa52858c9f1e8c15/pkg/controller/chi/worker-chi-reconciler.go#L704
Under the hood, that calls https://github.com/Altinity/clickhouse-operator/blob/d5f265fb4773ec4622b2af13fa52858c9f1e8c15/pkg/model/chi/schemer/schemer.go#L151-L154 which runs https://github.com/Altinity/clickhouse-operator/blob/d5f265fb4773ec4622b2af13fa52858c9f1e8c15/pkg/model/chi/schemer/sql.go#L244-L246.
I think all we really need is the ability to specify a single SQL statement in the custom resource, maybe a field like healthcheckSQL or something, since you can just simply chain multiple statements together in the event you need to halt reconciles based on multiple sources of data. And then in the operator we would simply check if any data was returned from the query and if so, halt progress in the operator until no data is returned (or some external timeout is hit like mentioned in a previous comment).
Curious what y'all think of this!

@com6056, thank you for your input. As an alternative, would it make sense to have this custom SQL as a part of the readiness probe? Or even a special condition to include host to the load balancer.
I am asking because syncing the replica may take quite a lot of time. Operator has no visibility what is going on when check is failing. What is a timeout value?
Alternatively, we may add replica wait logic to operator explicitly, without a custom SQL. In this case, operator will know what is going on. It will be able not just monitor the replica delay, but also check replication queue and see if there is any progress, or host is stuck.

I think the latter idea sounds best with the operator being in full control of the rolling based on those metrics. Ideally we don't want pods rolling if the replication delay is above a certain amount and I don't believe a readiness probe halts rolling of pods with this setup (not 100% sure how it all works in your operator since you have a special StatefulSet setup), the probe only prevents the pod from being served traffic.


Resolve a few simple rubocop identified performance issues

Use max instead of sort.last and key? instead of keys.include?

Not worth troubleshooting the failures


Loosing the correct index after re-render view

Which OS 11
Version
Which versions are you using:

react-native-swiper v1.5.13
react-native v0.49.3

Expected behaviour
Maintain the correct view index and switch between views correctly
Actual behaviour
The index is lost and I can't switch between views after re-render view (with setState or others). It seems that is adding new index when rendering again
Steps to reproduce

Rerender view (with setState() or forceUpdate)
Switch between swipe views by swipe or with scrollBy

+1. First,it works ok.but it will show nothing after using 'setState'.

i solve this problem just now. Just modify the file index.js of react-native-swipe
componentWillReceiveProps (nextProps) {
    if (!nextProps.autoplay && this.autoplayTimer){
      clearTimeout(this.autoplayTimer);
      this.setState(this.initState(nextProps, this.props.index !== nextProps.index))
    }
  }

i have two swiper pages and when i press any button on the second, the state rerenders and scrolls to the first page. This happens only on ios. Any ideas?


Make samples standalone

The samples are in the same repo as the libraries...
If I check out the entire repo it is not clear how do I use a given sample with a given release (HEAD is not released)
The pom refers to the parent sample that refers to the parent...
This would not be the case for real customers apps, and they are struggling to adapt to a correct pom with the correct references to a released version of SpringBoot and the GCP integration.

I think it's a fair criticism, as more people are trying to use the samples directly rather than just looking at the code.

@meltsufin, what's your recommendation to begin starting work on this? I guess we first have to figure out where to first create a new repository?

I would disconnect the samples from the project parent.
Make the parent of each sample the spring boot parent.
<parent>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-parent</artifactId>
    <version>...</version>
</parent>


Channels I closed with 'closechannel' are stuck in 'pending_force_closing_channels'

Background
I opened two channels a few days ago then I closed them normally via the cli using closechannel. Both of them are stuck in pending_force_closing_channels but I did not force close them. Unfortunately now all my btc is stuck. This is for mainnet.
    "pending_force_closing_channels": [
        {
            "channel": {
                "remote_node_pub": "020e56a13babec99abdc2c4afbe34e1e44230d79b234c059fd4ff1e367765fdb1b",
                "channel_point": "3dd317535b21fee66912860017d70e2cba79588b094d1c41d24206649238cf35:1",
                "capacity": "16777215",
                "local_balance": "16773742",
                "remote_balance": "0",
                "local_chan_reserve_sat": "0",
                "remote_chan_reserve_sat": "0"
            },
            "closing_txid": "d14b171932754fbd8644a85d3f0da77a62dfccb25cb86a7357b6429cd60dd426",
            "limbo_balance": "16773742",
            "maturity_height": 615234,
            "blocks_til_maturity": 1976,
            "recovered_balance": "0",
            "pending_htlcs": [
            ]
        },
        {
            "channel": {
                "remote_node_pub": "02ad6fb8d693dc1e4569bcedefadf5f72a931ae027dc0f0c544b34c1c6f3b9a02b",
                "channel_point": "769d2d197126d2e16cab80850490e97611c2dcc68b2791c9e72cfcf915fe6a82:1",
                "capacity": "13068664",
                "local_balance": "13052263",
                "remote_balance": "0",
                "local_chan_reserve_sat": "0",
                "remote_chan_reserve_sat": "0"
            },
            "closing_txid": "e4de8e3022eb07f00a865d89f5be7b24aae4ecbc48424bcb973c3e3825635688",
            "limbo_balance": "13060036",
            "maturity_height": 614629,
            "blocks_til_maturity": 1371,
            "recovered_balance": "0",
            "pending_htlcs": [
                {
                    "incoming": false,
                    "amount": "7773",
                    "outpoint": "965366e67f9ce459b9575056af6772d79e9be74ace36347c99b823e17ab1d788:0",
                    "maturity_height": 614630,
                    "blocks_til_maturity": 1372,
                    "stage": 2
                }
            ]
        }
    ],

Your environment

lnd version Version: 0.9.0-beta commit=v0.9.0-beta-rc2, build=production, logging=default
OS:Linux 3.10.0-957.12.2.el7.x86_64 #1 SMP Tue May 14 21:24:32 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux (CentOS)
bitcoind 190001

Steps to reproduce

Open a channel
Run closechannel

Expected behaviour

Channel should close normally?

Actual behaviour

Channels got stuck in this pending_force_closing_channels state

Log files
Here is the log file when I start the lnd node with --debuglevel=LNWL=trace,CNCT=trace,FNDG=trace,UTXN=trace
lnd.log

The funds have been released, so I do not have an issue anymore, though I still havent been able to figure out an alternative reason to the one above.


Increase the time waiting for a consul leader to be elected

Hi, thank you for your contribution to Mantl! Before we can accept any code into
master, we need it to meet the following criteria. If there are any you can't
satisfy yourself, go ahead and open the pull request anyway and we'll help you
test. Feel free to delete this message once you're done. Thanks again!

[ ] Installs cleanly on a fresh build of most recent master branch
[ ] Upgrades cleanly from the most recent release
[ ] Updates documentation relevant to the changes

When the consul leader is restarted by the consul-rolling-restart.sh, the unlocking of the locks/consul node can fail with 500 because the Consul cluster isn't quite ready despite the /v1/status/leader consul endpoint reporting that there is a leader. Increase the time waiting for the cluster to settle.

Attempts to address https://github.com/CiscoCloud/mantl/issues/1625

all builds passed (aws failed but only due to a timeout during cleanup - the actual build and health checks passed)

@ChrisAubuchon @ryane Nice!


Added check to see if source exists

Prevents too many errors in the log for every lost file.
Ticket: #
Reviewer: @
Issue

Solution

Impact

Usage

Considerations

Testing

Coverage remained the same at 93.235% when pulling 4493ddbf589474f6d51958a719c46cf377f3d4e2 on mgussekloo:patch-4 into a968a68ba21218c177618798248ed2477487f2ec on timber:master.

Codecov Report

Merging #1420 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #1420   +/-   ##
=========================================
  Coverage     93.78%   93.78%
- Complexity     1408     1409    +1
=========================================
  Files            45       45
  Lines          3396     3396
=========================================
  Hits           3185     3185
  Misses          211      211

Impacted Files
Coverage Δ
Complexity Δ

lib/ImageHelper.php
96.1% <100%> (ø)
81 <0> (+1)
: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 a968a68...4493ddb. Read the comment docs.

Looks good, thanks @mgussekloo (and sorry for the slow response, I just got back from a long GitHub-free vacation)


hook fails if comment tags not found

I would expect a tool like this to operate in a very standard mode, not trying to do anything like inserting between comments. This is your design decision to make but I'd much prefer a more standard execution of the tool like:
terraform-docs md ./ | cat -s | ghead -n -1 > README.md

This approach is markdown linter compliant and supports the standard of putting your documentation in the main.tf

My reasoning for making it using comments was that not everybody wants to overwrite the whole README.md but rather put it into specified section of that file.
Later we may have a possibility to specify which file to update instead of README.md as now.
Regarding the issue with sed... unexpected EOF. It is happening not because a comment tag was not found, but because sed command does not work on all platforms equally well (I tested it with brew'ed sed on Mac). The most platform agnostic way seems to be using Perl (#15).
I will keep this issue posted when a new pre-commit hook is released (during today or tomorrow).

v1.7.2 is using Perl and keeps a temporary file in a temporary directory.
Please give it a try and let us know how it goes.

v1.7.3 at least got me past this error. I wanted all of the documentation in main.tf to overwrite the README.md. The hook doesn't actually write the README contents like #7 asks for still, but this could probably be closed.


whitespace stays when the cursor is at it upon saving

Upon saving, any whitespace before the cursor should get removed (if given option is activated). However, this is not the case currently.
E.g. it should work like on Sublime (upon save):
http://inft.ly/35tktDb

This can be configured in the whitespace package.
Open Settings View (Ctrl+, or Cmd+,), navigate to Packages, then to the Whitespace package, and finally change "Ignore whitespace on current line" to false.


[tlul] Add some missing dependencies

These missing dependencies currently lead to lint errors in the tlul modules.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

Looks like a good idea, but I'm not sure exactly what was failing. Can you give the command line of the lint check that was broken?

My change created a cyclic dependency in the core deps. This is now fixed.

@rswarbrick can you take another look at this one (should be quick)?

Thanks!


ENH: Set ITK_TEST_OUTPUT_DIR with module path

Change the default output path for modules testing data to include the
module name as a directory.
This prevents the same file name in defferent modules clobbering one
another causing intermediate testing failures.

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

how thoroughly has this been tested?

It has not been thoroughly tested. It seemed to work locally. And using the CI to test what happens across systems on clean build seemed like the next step.

It has potential to cause minor disruption in some remote modules.

Yes indeed. Alternatively, a new CMake variable could be introduced, and every thing changed to used it.. but that is more work that I care to tackle especially with the remote modules.

how thoroughly has this been tested?

It has not been thoroughly tested. It seemed to work locally. And using the CI to test what happens across systems on clean build seemed like the next step.

It has potential to cause minor disruption in some remote modules.

Yes indeed. Alternatively, a new CMake variable could be introduced, and every thing changed to used it.. but that is more work that I care to tackle especially with the remote modules.

If remote modules do provide an appropriate value for the itk-module variable, this seems a fair solution to me.

If remote modules do provide an appropriate value for the itk-module variable, this seems a fair solution to me.

remote modules provide ... itk-module

I don't think this is an issue. I think there are some assumptions where the test temp dir is. Updating that should not be hard.

remote modules provide ... itk-module

I don't think this is an issue. I think there are some assumptions where the test temp dir is. Updating that should not be hard.

Looks like all the tests in the CI passed :)

Looks like all the tests in the CI passed :)

Probably best to check how it works for externally built remote modules (pick 2-3 to test). I expect that remotes built as part of ITK will behave the same as regular modules.

Probably best to check how it works for externally built remote modules (pick 2-3 to test). I expect that remotes built as part of ITK will behave the same as regular modules.

Seems fine to generally help with filename clobbering issues. I tend to use the consolidated testing output directory with rm -rf ITK-build/Testing/Temporary to clean up build trees that are used repeatedly over time and inflate disk usage. This would make it more difficult, but maybe we could also isolate more easily areas that require cleanup.

Seems fine to generally help with filename clobbering issues. I tend to use the consolidated testing output directory with rm -rf ITK-build/Testing/Temporary to clean up build trees that are used repeatedly over time and inflate disk usage. This would make it more difficult, but maybe we could also isolate more easily areas that require cleanup.

I tend to use the consolidated testing output directory with rm -rf ITK-build/Testing/Temporary to clean up build trees that are used repeatedly over time and inflate disk usage.

That will still work as the new test output directory for modules will be like ITK-build/Testing/Temporary/ITKCommon and not under the modules build path.

I tend to use the consolidated testing output directory with rm -rf ITK-build/Testing/Temporary to clean up build trees that are used repeatedly over time and inflate disk usage.

That will still work as the new test output directory for modules will be like ITK-build/Testing/Temporary/ITKCommon and not under the modules build path.

If as @dzenanz said rather than the module name issue this is likely to impact the output dir expected by the remotes, once that has been checked, I'd be for the change.

If as @dzenanz said rather than the module name issue this is likely to impact the output dir expected by the remotes, once that has been checked, I'd be for the change.

Brad, are we still targeting this for 5.3?


Break instruction into Paragraphs

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Hey @rahulasher110
This PR appears to make the same changes as the existing PR #40428. As such, we are going to close this PR.
Thanks again! 😊

If you have any questions, feel free to ask questions on the 'Contributors' category on our forum or the contributors chat room.


Debug: fivecarddraw print() statements not printing in notebooks

print() statements within the fivecarddraw modules class methods do not print into notebooks. Check to see if it's possible to change where print() can output. Perhaps even add a toggle if possible.  Consider deconstructing the methods instead within the notebook, to be illustrative. Consider how large the deconstruction of the FiveCardDraw object would have to be, in order to provide print statements that way.

This was a notebook kernel bug.


Ambari 23945 AMBARI-23945. Infra Solr migration: REQUESTSTATUS response content type is xml instead of json

What changes were proposed in this pull request?
with solr 5 the default response content type is xml, so the REQUESTSTATUS command can fail when we are reading the response object.
How was this patch tested?
manually
please review @zeroflag @fimugdha

retest this please


Need Cross Platform Example

.Net Core is a cross platform tool. You can create applications on Windows, Mac, and Linux. The example should not require Visual Studio, which is Windows only.

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

ID: 3d445808-e61d-ccc2-3696-ef3ea109a662
Version Independent ID: 8a9e706a-c350-1271-dcb8-883e166423b4
Content: Create a C# ASP.NET Core web app in Azure
Content Source: articles/app-service/app-service-web-get-started-dotnet.md
Service: app-service-web
GitHub Login: @cephalin
Microsoft Alias: cephalin

@TerryDiederich We will now close this issue. If there are further questions regarding this matter, please reply to @femsulu and we will gladly continue the discussion.


No release files on gihub for 1.18.3

There are only tag 1.18.3 but latest release with downloadable artefacts is 1.18.2

this is due to the desktop version of 1.18.3 not properly functioning despite passing existing unit tests
see #604


[MM-19728] Add loading indicator for compliance monitoring export that are in progress similar to Compliance Export

Summary

Add loading indicator for compliance monitoring export that are in progress similar to the one in Compliance Export.
Ticket Link

https://mattermost.atlassian.net/browse/MM-19728
Screenshots
Pre-Fix

Post-Fix

Creating a new SpinWick test server using Mattermost Cloud.

Please wait while a new kubernetes cluster is created for your SpinWick

Timed out waiting for the kubernetes cluster. Please check the logs.

Test server creation failed. See the logs for more information.

Test server destroyed

@srkgupta do you still need the test server?

@cpanato No. I tested the changes locally and have already approved the PR. Thanks for checking.

@hahmadia is this slated for a particular release? We should add the milestone.


docs(examples): demo apps setup

PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What has been added?
Examples of unifig usages with express and Nest.js.

Codecov Report

Merging #17 (f833ed8) into main (585b3c5) will increase coverage by 0.08%.
The diff coverage is 97.61%.

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   97.76%   97.85%   +0.08%
==========================================
  Files          34       34
  Lines         358      373      +15
  Branches       65       69       +4
==========================================
+ Hits          350      365      +15
  Misses          8        8

Flag
Coverage Δ

unittests
97.85% <97.61%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

...lib/adapters/class-adapters/plain/plain.adapter.ts
100.00% <ø> (ø)

packages/nest/lib/config.module.ts
84.21% <85.71%> (+2.96%)
:arrow_up:

...kages/core/lib/loader/decorators/from.decorator.ts
100.00% <100.00%> (ø)

...ges/core/lib/loader/decorators/nested.decorator.ts
100.00% <100.00%> (ø)

packages/core/lib/loader/loader.impl.ts
100.00% <100.00%> (ø)

...kages/core/lib/manager/container/container.impl.ts
100.00% <100.00%> (ø)

packages/core/lib/manager/manager.impl.ts
97.36% <100.00%> (+0.07%)
:arrow_up:

...core/lib/manager/source-group/source-group.impl.ts
100.00% <100.00%> (ø)

...ages/core/lib/validator/errors/validation.error.ts
100.00% <100.00%> (ø)

packages/core/lib/validator/validator.impl.ts
95.55% <100.00%> (+0.20%)
:arrow_up:

... and 3 more


Model ignores the speakers in the audio except for one

I noticed a weird behaviour when transcribing an audio from a news report that has a couple of interviews.
the model transcribes the speech of the host decently, but completely skips the parts where the guests speak. some segments are over 40 seconds long and are completely absent from the transcript.
I am using large-v2 model for Korean language transcription.
I played around with using the vad filter and even an initial prompt that mentions how many speakers will be in the audio, but nothing fixed it yet.
any idea what could be the cause?

yes it is the same issue with openai/whisper. I think it has to do with the fact that the audio has sections that are super loud.
I found this solution from HF transformers "Rescale layer in whisper processor"
so using do_normalize seems to help a bit.
I couldn't find an implementation of this in the faster-whisper repo so I did it manually using the code below and it helped.
`audio, sr = librosa.load("../audios/03.mp3", sr=None)
if sr != 16000:
audio = librosa.resample(audio, orig_sr=sr, target_sr=16000)
audio_data_normalized = (audio - np.mean(audio)) / np.std(audio)`
if it is implemented and I missed it, please let me know how to use it.

you could normalize it like so
ffmpeg -vn -c:a mp3 -b:a 32k -vbr off -ar 16000 -af dynaudnorm,aformat=channel_layouts=stereo input.mp3
even remove silence and filter out noise (hiss) if that's what you wanna do
ffmpeg -c:a mp3 -vbr off -b:a 32k -ar 16000 -af highpass=500,lowpass=1000,afftdn,aformat=channel_layouts=stereo,volume=12dB,"silenceremove=start_periods=1:stop_periods=-1:start_threshold=-50dB:stop_threshold=-50dB:start_silence=1:start_duration=2:stop_duration=5:detection=peak",dynaudnorm
but I recommend doing EBU R128 algorithm and it's two-pass and is superior to dynaudnorm.  Use videomass in AV-Conversions section.  After this is done you still might have to convert it to correct sample rate though


train loss

Thanks for your great work! I would like to ask why there are two sudden drops in this loss graph(copied from https://huggingface.co/O1-OPEN/OpenO1-LLama-8B-v0.1/blob/main/training_loss.png) Does this mean that the current stage of this work is overfitting on the O1-stype data format through sft?

That's likely to be 3 epochs and a high learning rate. 2nd and third time the model is exposed to the same data, it's familiar with it.


Remove known issue on ARCH-PRO

Remove known issue on ARCH-PRO by fix ARMmbed/mbed-os#4060 and https://github.com/ARMmbed/mbed-os-example-sockets/issues/17

cc @geky


Added typescript declaration.

Thanks!


Fix some typos in the schema doc. [phab:D1771]

Created by fromberger at 2017-07-12 03:15:05:
Specifically, the reference to the record type was wrongly written as
an anchor instead of a ref, and one of the subheading markers was wrong.

jrtom accepted this revision at 2017-07-12 04:18:04

fromberger closed via commit 4c1aba79635078f15ecefdcbd23c1282519daf7e at 2017-07-12 04:30:14


Remote Key sharing button getting cut in older Android

I have implemented the fix in the feature/remote-sharing branch of the public repo, where @cakesoft-vaibhav is working on remote sharing. It is not part of the private repo build, so it should not be tested there.

There is another issue regarding this.So closing this issue for now.
#5347


[Discussion] Some Observations on Florence2

Hello, I am testing Florence2 on a fork from another repo made by spacepxl. https://github.com/spacepxl/ComfyUI-Florence-2
I chose this repository because the approach with the XY coordinates, etc., can be more comfortably used later for inpainting/upscale purposes. It would be great if you could both collaborate with a single repository for Florence-2. This would surely help in pooling resources more effectively. @spacepxl @kijai
Regarding Issue https://github.com/kijai/ComfyUI-Florence2/issues/11
I noticed that lines appear when the model does not output the coordinates for the mask beyond a certain point. For example, with max_tokens, I was sometimes able to increase the max_tokens beyond 1024, and then the mask continued to be drawn. However, Comfy often crashes when the value exceeds 1024. Unfortunately, I have not yet found a way to "simplify" the masks from the model.
You can observe this behavior if you take an image with a small subject and use the prompt hair with the task <REFERRING_EXPRESSION_SEGMENTATION> with 1024 tokens, getting the full mask. Then, if you significantly reduce the max_tokens, you will see that the mask eventually gets cut off.
I have no idea why the model outputs incomplete data or if this process can be improved by batch processing.
Currently, the models do not seem fully mature to me. Even the fine-tuned models provide fewer coordinates if you manage to get the same mask. For example, when segmenting hair, fine-tuned models often show the whole character instead of just the hair, whereas the non-fine-tuned model works right away. I tried using num_beams to get the hair mask with the fine-tuned model, but it's not very reliable.
Maybe this will help in troubleshooting.
I also noticed problems with elements like eyes, arms, or legs when multiple parts are visible in the image. A workaround that has worked was using prompts like: both eyes or left eye, right eye, left arm, right arm
https://huggingface.co/microsoft/Florence-2-large/discussions/10
However, this is far from satisfactory.
I get the impression that the model is not very good at body segmentation or not adequately trained for it.
What segmentation tasks would you mainly use the model for? Perhaps it could be fine-tuned for such tasks, although I have never done that myself.
Currently, I am working on detecting and masking multiple objects "simultaneously" using the tasks <REFERRING_EXPRESSION_SEGMENTATION>, <REGION_TO_SEGMENTATION>, e.g., mouth, hair, dress .
This is working relatively well, but I wanted to clear this with my previous pull request with Spacepxl before committing it.
Processing image batches is next on my to-do list.
Do you have any ideas on how we can improve our experience with Florence2 for ComfyUI together?
Best regards,
dnl

I have the node outputting square masks for the detected boxes already, I did think about bboxes too but it really just depend what node it needs to pair with.
The segmentation is indeed disappointing, I had the same though of chaining the detections and using the region segmentation instead, but I'm afraid it still has same issue with the incomplete masks. I tested drawing just the points too to be sure the problem isn't the drawing of the polygons. I have also tried adjusting the max tokens and other generic LLM parameters, only observation I had was same as you: reducing the max tokens cuts it off earlier. Going past 1024 never crashed for me, but it didn't help any either.

Very interesting observation. My thought was already to integrate a SAM model for mask generation behind the boxes, as the box detection is already considerably faster and more accurate than some other models I've had so far. But from my perspective, that would be too much.
Yes, I can understand the concern with bounding boxes. I think a widely used library is the Impact Pack, which can also expect bounding boxes and SEGs for its detailers and upscalers.
I delved into Dr.Lt.Data's code to see how SEGs are structured. This seems to be a worthwhile approach to me, especially since Dr.Lt.Data will also be a core competency team at comfy.org. I believe that this type of approach could prevail in the future, considering how much is already included in a SEG (x, y, width, height, cropped image, etc.).
I think it would be beneficial to establish good connectivity with the other nodes in the Comfy universe at this point. Pictures are also fine. Just my two thoughts.
But thank you very much for your feedback, and it's a shame that the observations are similar. However, I am optimistic that Florence2 could establish itself. I find the masks it creates to be very good (compared to others like ClipSeg)

Few months later... (200 years in AI world...)
I find Florence2 also very random, I just type face, neck and select the hair, for example.
What tools are you using now for this task?


When You Find Me (Low Pitch) by Wist8nGD

Song Name
When You Find Me (Low Pitch)
Artist Name
Wist8nGD
Source
Youtube
Youtube Link
RQ3ZMYlSvqzgqM8i
Direct File Link
No response
Song ID
30
Extra
Created from Auto Nong.

video is not available, please update the url

accepted in another issue


fix ldb headline to be ldb and not beezcli

fixes: https://github.com/speedb-io/speedb/issues/482

@ofriedma I tried to build on my laptop from scratch and there are errors. I have sent you the information in Slack.

@udi-speedb ready for review


Fix/mqtt connection auth

What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds username and password to MQTT broker connection
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

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

:tada: This PR is included in version 2.17.0-rc.3 :tada:
The release is available on:

npm package (@pre/rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.20.0-development.1 :tada:
The release is available on:

npm package (@rc dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


update Craft CMS gitignore to follow Craft team's recommendations

Reasons for making this change:
The existing Craft CMS gitignore is not what is recommended in the official source referenced in the existing CraftCMS.gitignore. My first change is updating the ignore list to reflect the official recommendations.
Craft cache can refer to something else, so I dropped the word "cache."
The recently released v3 comes with gitignores out of the box, and does not require anything additional. I added a comment pointing this out.
I can see replacing
# Craft 2 Storage (https://craftcms.com/support/craft-storage-gitignore)
# not necessary for Craft 3 (https://github.com/craftcms/craft/issues/26)

with a simple
# Craft 2 Storage (Craft 3 requires no gitignores beyond what it comes with)

but thought I'd start by sticking with the existing format. Let me know if you'd like me to make that change!
Another way to go would have been renaming the file to CraftCMSv2.gitignore, but I don't think that would be as clear.
Links to documentation supporting these rule changes:

Craft 2 documentation: https://craftcms.com/support/craft-storage-gitignore
Craft team's confirmation that Craft 3 doesn't need additional ignores: https://github.com/craftcms/craft/issues/26

@olets thanks!


shell: kconfig: Put 'menuconfig SHELL' in top-level menu

The 'Shell Options' menu contains just the 'menuconfig SHELL' menu in
the menuconfig interface.
Remove the 'Shell Options' menu and put 'menuconfig SHELL' directly in
the top-level menu instead. Also change the prompt from "Enable shell"
to just "Shell", to make it consistent with "Logging".

Makes the top menu look like this instead:

Good chatch. This was needed when we had legacy shell and new shell.


Auto-generated: Update kuttl e2e test  files

Automated changes by create-pull-request GitHub action

Tests are failing on k8s 1.23 because the plugin uses operator v2.5.0 which does not have the following fix: https://github.com/storageos/operator/commit/7797c8a3d2c7363455e1e44264973ab9bdde6f54


add name to contributors.html and add small style to index.html

Hi, would you add small changes? :)

Yes for sure :)


Make a CLI interface and optional GUI compilation

That way this could come embedded into OpenWRT/LEDE Routers

@nicopace this version of name-bench has been essentially dead for quite a while, check out the forks if you want further development. The original namebench had a CLI interface (use the -x option) and is able to run on one of my LEDE Archer C7 routers.

what a pitty... implementing it in go instead of python would make it much more compact and distributable on smaller devices :)


Don't backup readme.md as notes.md

I've scanned the entire splunk-soar-connectors repo (>300 connectors) and notes.md wasn't generated for any apps.

Let's wait until I migrate the apps in https://docs.google.com/spreadsheets/d/1CeTW1MJIMIDQgtc3BMI1yuQbHx--qprxsTpGY9JSIak/edit#gid=0
I'll run build_docs.py during the migration

Let's wait until I migrate the apps in https://docs.google.com/spreadsheets/d/1CeTW1MJIMIDQgtc3BMI1yuQbHx--qprxsTpGY9JSIak/edit#gid=0
I'll run build_docs.py during the migration

Sounds good. We should merge this one instead then: https://github.com/phantomcyber/dev-cicd-tools/pull/25


False alarm: reportGeneralTypeIssues

Seem it still raises the error after setting python.analysis.typeCheckingMode': 'off
I have some query set with the related fields, and everything is ok, but today, when working time, the error reportGeneralTypeIssues appears.
example:

OS and version: Ubuntu 20.04.5 LTS
Python version: 3.9.15+
Pylance: v2023.2.30

Thanks for the issue but I can't reproduce it. Do you have a complete sample that reproduces it?
For example typing this for me causes a squiggle:
x =[4]
x.bar

turning off type checking makes the squiggle go away

Oh, it only appears error in my Django project, I try to open a new python file and type follow you, not any error happened.

I realize I don't change TypeCheckingMode on Workspace too, just changed it on User, so this problem has been resolved. Thank you.