Can I turn off auto transaction in delete/update/insert?

According to this doc: http://gorm.io/docs/transactions.html
GORM perform single create, update, delete operations in transactions by default to ensure database data integrity.
DB will auto add transaction to single operation. I want to let DB do it rather than give it to gorm in order to save some bandwidth usage.
Is there a switch to turn that off?

you can remove transaction callback process：
	db.Callback().Create().Remove("gorm:begin_transaction")
	db.Callback().Create().Remove("gorm:commit_or_rollback_transaction")
	db.Callback().Update().Remove("gorm:begin_transaction")
	db.Callback().Update().Remove("gorm:commit_or_rollback_transaction")
	db.Callback().Delete().Remove("gorm:begin_transaction")
	db.Callback().Delete().Remove("gorm:commit_or_rollback_transaction")


Fixes GlyphRunNode hit testing

What does the pull request do?

What is the current behavior?

What is the updated/expected behavior with this PR?

How was the solution implemented (if it's not obvious)?

Checklist

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

Breaking changes

Obsoletions / Deprecations

Fixed issues

Fixes: #11007

You can test this PR using the following package version. 11.0.999-cibuild0033205-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Update CardIO, BraintreeSDK, updates for cordova/8 & cordova-android/7+

This PR pulls in changes across all the forks:

Capture an iOS error on launching the Drop In UI
Prevent the framework build script from running on Cordova 7+
Add the <framework> tags for Cordova 7+ so it copies them automatically
Update CardIO
Update Braintree SDK
Update build gradle for cordova-android 7 (backwards compatible)

@Taracque hope you can take a look :)

@Taracque any chance you can take a look at this?

Looking forward to this...
Anybody knows if the card.io/braintree integration with this plugin is PCI compliant? thx

It seems like @Taracque is no longer maintaining this fork. I installed it in a Cordova 8 app and there are issues. Installing this PR fixed everything. @codinronan would it be worth it to just consider yours the "stable" version going forward? I'd love to have one repo to go to instead of 12 😄  Will help maintain!

Hi @jamesdixon we meet again! 🎉
I am fine with your proposal - I didn't want to have a bunch of disparate branches, but if mine is the one being maintained well then we go :)
Let me add you as a maintainer over there, keep an eye out for the email.

@codinronan it's been a while! Hope all is well!
Yeah, it's becoming a mess, but it appears yours may be the most up to date. This plugin is driving me insane. One day it builds, the next day it doesn't :smile: Curious, what versions of Cordova/cordova-android/cordova-ios are you using along with your branch? I can't seem to get a fully working build all of the sudden 😡

Team, @Taracque hasn't been on GitHub in some time. Is it time for another fork and attempt to support?
Ionic seem to reference this repo from their official documentation, so perhaps they'll throw some support

Hi @FuzzAU,
I forked this repo more than one year ago, I fixed some bugs and I've used it (in production) since that time.
It works, so if you want you can use it.
Let me know if you need support.

@fbognini that's a wonderful offer and I'll try it out. Thank you!
I was just testing out @codinronan's branch now however I see you've taken my and his previous commits in to your branch too. If it's working for you, and I can get it all glued in nicely with my build then I'll continue on with it.
Oh software development... building on the shoulders of giants, that then disappear (understandably, people have lives!)

Hey all,
I've made some slight improvements to my branch over at https://github.com/engineerapart/cordova-plugin-braintree, namely, improved the install script so it won't duplicate the shell script entry as well as added some notes about ensure the script is there. It seems like in later versions of cordova, the script won't execute on a normal cordova prepare command anymore, so that is addressed in the README.
I looked at @fbognini 's branch and I'm not sure that change is needed with<EMAIL_ADDRESS>there's a commit there that should update the bundle ID correctly. However I've also mentioned his branch as well as this thread in my updated README in case people need it.


GH-41233: [C++] Added an are_cols_sorted option to RowTableMetadata  for control column sorted

Rationale for this change
Fix the bug in grouper when set are_cols_in_encoding_order=false in below codes:
https://github.com/apache/arrow/blob/2979d69a05cb16012da06baaa801a1849e9110ce/cpp/src/arrow/compute/row/grouper.cc#L582
It will cause the num_group different with are_cols_in_encoding_order=true condition.

What changes are included in this PR?
Added an are_cols_sorted option to RowTableMetadatato control whether we enable column-sorted in RowTableEncoder.

Are these changes tested?
Will be added after https://github.com/apache/arrow/pull/40998 merged(grouper_test has added in this pr).

Are there any user-facing changes?
No.

GitHub Issue: #41233

I have a question about the necessity of this fix.
IIUC, though RowTableImpl supports both usages of columns "in encoding order" and "not in encoding order", the user (e.g. Grouper or SwissJoin) is free to choose either, in other words, the user is not mandatory to support both. For example, the current SwissJoin is using it the way that all columns are assumed "not in encoding order" and it is perfectly fine because there isn't a case that requires SwissJoin to do it the other way. The same goes to Grouper as well.
Is there a reason that Grouper must assume the columns are not in encoding order, or that Grouper can benefit in terms of performance/complexity from treating the columns not in encoding order?
Thanks.

I have a question about the necessity of this fix.
IIUC, though RowTableImpl supports both usages of columns "in encoding order" and "not in encoding order", the user (e.g. Grouper or SwissJoin) is free to choose either, in other words, the user is not mandatory to support both. For example, the current SwissJoin is using it the way that all columns are assumed "not in encoding order" and it is perfectly fine because there isn't a case that requires SwissJoin to do it the other way. The same goes to Grouper as well.
Is there a reason that Grouper must assume the columns are not in encoding order, or that Grouper can benefit in terms of performance/complexity from treating the columns not in encoding order?
Thanks.

This PR try to support column "sorted" and "not-sorted" modes for grouper.

"sorted" mode means do column sort for RowTable and compare with "are_cols_in_encoding_order=true"
"not-sorted" mode, contrary to the above logic, do not sort for RowTable and compare with "are_cols_in_encoding_order=false"

The performance of grouper in some "not-sorted" scenarios tested by benchmark(https://github.com/apache/arrow/pull/41036) is better than that of "sorted" mode.

The columns in this scenario are already sorted.
Do you think we should support not-sorted mode in grouper for user?

Try to summarize:

The current Grouper (before this PR) uses the "sorted" approach, has no bug, and performs 15% faster than the "non-sorted" approach (introduced in this PR) for "sorted"-friendly scenarios.
The current Grouper (before this PR) doesn't support the "non-sorted" approach (or in other words, has bug if force enabled).
This PR enables the "non-sorted" approach (or in other words, fixes the bug in 2), as an user faced option, and performs 2%~4% faster than the original "sorted" approach for "non-sorted"-friendly scenarios.

IIUC, then I think it might be more proper to treat this PR as an enhancement or even a small feature than a bug fix, because IMO it's not actually a bug if Grouper is not designed to use "non-sorted" approach from the beginning.
And the question comes to: do we need this new "non-sorted" approach for Grouper with the given performance number? Personally I think we can have it, but I'm not confident (nor authorized) to make the call. So I'll just leave it to others :) cc @pitrou @felipecrv @westonpace

Q1
Sorted mode implementation has two main parts:

Add the input cols to the RowTable using the RowTableEncoder pair according to the column sort in [1]. The main purpose of column sorting is to construct arrays of memory-alignment column_offsets to facilitate memory-aligned comparisons in subsequent column-Compare comparisons.

The [2] logic in CompareColumnsToRows decides, according to are_cols_in_encoding_order, whether to take the offset of the comparing columns from RowTable (rows) in order of part1.

Non-soretd mode is the opposite of the above logic, that is: column sorting is not performed in the part 1, and the column_offsets of the input original columns are used in the part 2.
Q2
I understand that the application scenario of non-sorted mode is that the input columns of RowTableEncoder are already sorted (can be accessed by memory alignment). When comparing, the input columns can be compared in the order of the input columns [2] without memory alignment check. This is also the reason for the improved performance in the non-sorted mode in the benchmark above.
[1] https://github.com/apache/arrow/blob/e254c43c095bd6e33d07129257e11760f885f299/cpp/src/arrow/compute/row/row_internal.cc#L88
[2] https://github.com/apache/arrow/blob/e254c43c095bd6e33d07129257e11760f885f299/cpp/src/arrow/compute/row/compare_internal.cc#L366-L368

So, to my question "are the two modes functionally different", is the answer "no"?

So, to my question "are the two modes functionally different", is the answer "no"?

Yes, the two modes functionally are different.

Is it worth keeping the two modes if the only difference is performance? Usually, it's better if users don't have to make low-level performance decisions themselves.

Usually, it's better if users don't have to make low-level performance decisions themselves.

Yes, this is what I have been thinking about recently. The RowTableEncoder and CompareColumnsToRows interfaces are too low-level and are currently only used in grouper and swiss-join. Choosing the optimal performance mode still requires some internal automatic decision-making, mainly because It is not clear to the user whether the order of the input columns has been sorted.
I will mark this PR as DRAFT first, and then start reviewing it when I can provide users with a more elegant way to use it (for example, if internal determines whether column sorting has been completed, the caller can directly use non-sorted mode).

cc @pitrou , the PR is ready for review.
We will make decisions automatically internal and use a mode with better performance.

We will make decisions automatically internal and use a mode with better performance.

I'm not sure I understand. Do you mean you'll remove the option in a later PR?
Also, this PR claims to both fix a bug and add an option. Can those two changes be made separate PRs?


[Feature] Flow integration

PR

Once this merge is in, would it be possible to put the flow config in its own repo so we can use across multiple projects? I'd love to use the same config here: https://github.com/Jam3/intern-webgl-react-app

@namniak @ameliemaia closing this ticket because we will likely move forward with TS instead in the near future.


Song not loading in fnf psych engine (crash)

Describe your bug here.
I was making a mod, when I stumbled across this. I searched everything, but I couldn't fix it.

This is the path that I was looking at.

I have no idea what happened, so reach out and help pls.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No.

Check if isn't corrupted, it won't work if it was .mp3 and you rename it to .ogg, you have to convert.
https://convertio.co/pt/mp3-ogg/


[core][cve] Bump cryptography package to 42.0.5

What changes were proposed in this pull request?

Bumping to latest version to remove multiple CVEs:

https://github.com/cloudera/hue/security/dependabot/215
https://github.com/cloudera/hue/security/dependabot/210
https://github.com/cloudera/hue/security/dependabot/189
https://github.com/cloudera/hue/security/dependabot/212
https://github.com/cloudera/hue/security/dependabot/206
https://github.com/cloudera/hue/security/dependabot/191
https://github.com/cloudera/hue/security/dependabot/187

@Harshg999 what kind of testing was done or planned for this?

Need to test building Hue and its normal working out on different form factors.

Did general Hue testing on both normal secure Hue cluster and FIPS cluster.

@Harshg999
have you checked it in ppc builds also?


Workspace - "It's not here" error when opening the subscriptions page

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

Version Number: 1.4.81-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Issue found when executing PR https://github.com/Expensify/App/pull/42990
Action Performed:

Log in with a new Gmail account
Navigate to https://staging.new.expensify.com/settings/subscription

Expected Result:
I should be navigated to the subscriptions page
Actual Result:
"It's not here" error when opening the subscriptions page without a workspace
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/da5de34f-a919-4ddd-a6c2-dfd6dd7f0628
View all open jobs on GitHub

@mallenexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@ikevin127 , plz review @Krishna2323 's proposal above.
Also, @ikevin127 @Krishna2323 , assuming the proposal is accepted:

Are there other places/links we should update for NotFoundPage surfacing when there's no workspace
Do either of you know where 'unvalidated' account discussions are happening? I don't see them on the road map doc. @ikevin127 , I see you reported this which might be similar?

https://github.com/Expensify/App/issues/43603

Also... checking here, looks like Niiki and Vivek might be working on unvalidated accounts
https://expensify.slack.com/archives/C049HHMV9SM/p1718648170515479?thread_ts=1718213571.738069&cid=C049HHMV9SM

@mallenexpensify The proposal looks good if we can all agree that the expected result here is to show the Create a workspace screen instead of the Not found page. Regarding the 2 points brought up:

Not sure, but if we move forward with the proposed solution then I'll make sure to double-check during PR review.
Not sure about this either, I think we could test this during PR. I found that issue in a similar fashion (during general testing on a PR review).

Clarifications
Context
The NewFeature issue and PR that introduced this issue:

https://github.com/Expensify/App/issues/38626
https://github.com/Expensify/App/pull/42990

Note: The NewFeature issue that implemented the subscription logic was UI focused and the feature is still not available to the user through UI yet.

This issue's expected result doesn't make sense since if you're not part of a workspace there shouldn't be content in the subscription page.
The only way the subscription page can be reached currently is for the user to paste the https://staging.new.expensify.com/settings/subscription URL and navigate to it manually, there's not button to reach this page.

Question

Should we HOLD this issue for when all parts of the NewFeature are deployed and actually made available to the user ? I'm asking this since the the PR author might want to fix this themselves before the feature is actually made available to the user as they mentioned in https://github.com/Expensify/App/pull/42990#discussion_r1631322823 regarding follow-up PRs related to any possible issues.

cc @amyevans (❌ OOO) Since she was the CME on both NewFeature issues / PRs.

If we really need to fix this right now, regardless, even though the page is not accessible to the user through UI yet -> then I will move forward with the existing proposal if we can all agree on the expected result proposed by the solution.

tl,dr: https://staging.new.expensify.com/settings/subscription is throwing an error if you don't have a workspace (but.. you can currently only access via the link so a normal user wouldn't know about).  @ikevin127 and I are wondering if this is going to be addressed soon and, if so, how to manage this issue.
Thanks for the detailed digging @ikevin127 .  I'm not well-versed in subscription settings details.   @trjExpensify @MitchExpensify @blimpich since y'all seem to be part of Build the ability to add and manage your billing card in NewDot

Should we put this on hold?
If so, what issue or PR should we be holding on?

I personally don't think this is a concern. We don't show the subscription menu item in the navigation pane unless the user has a subscription already.
I consider this expected behavior. @MitchExpensify feel free to re-open if you feel otherwise.


Ansible operator is munging keys

Bug Report
What did you do?
I have a CRD that defines properties named quotaName and quotaDefinition.
What did you expect to see?
I expected these keys to be exposed in the corresponding Ansible role as variables quotaName and quotaDefinition.
What did you see instead? Under which circumstances?
The keys have been renamed to quota_name and quota_definition, which  makes no sense: the original names were perfectly valid Ansible variable names.
Environment
Operator type:
/language ansible
Kubernetes cluster type:
OpenShift
$ operator-sdk version
operator-sdk version: "v1.22.0", commit: "9e95050a94577d1f4ecbaeb6c2755a9d2c231289", kubernetes version: "1.24.1", go version: "go1.18.3", GOOS: "linux", GOARCH: "amd64"

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.5", GitCommit:"c285e781331a3785a7f436042c65c5641ce8a9e9", GitTreeState:"clean", BuildDate:"2022-03-16T15:58:47Z", GoVersion:"go1.17.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.3+b93fd35", GitCommit:"3a0f2c90b43e6cffd07f57b5b78dd9f083e47ee2", GitTreeState:"clean", BuildDate:"2022-02-11T05:26:59Z", GoVersion:"go1.16.12", Compiler:"gc", Platform:"linux/amd64"}

Possible Solution
Stop renaming keys!

This is mentioned a few places in the docs: https://sdk.operatorframework.io/docs/building-operators/ansible/reference/watches/
You can set snakeCaseParameters to false in watches.yaml to disable the behaviour you're talking about

@loganmc10 thanks, that's helpful. Now that I know what to look for I do see the note in the tutorial, but since the behavior is surprising I would like to see it emphasized (and perhaps added to the quickstart as well.
I'll try to get around to make a docs pr. Thanks for your help!


[BUG] Naming inconsistency of model in ChatAgent and model_type in elsewhere

Required prerequisites

[X] I have read the documentation https://camel-ai.github.io/camel/camel.html.
[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[ ] Consider asking first in a Discussion.

What version of camel are you using?
0.1.0
System information
3.10.11 (main, May 15 2023, 19:29:30) [Clang 14.0.6 ] darwin
0.1.0

Problem description
The naming of model in ChatAgent is inconsistent with model_type in elsewhere. Let's change model in ChatAgent to model_type as well.
Reproducible example code
N/A
Traceback
No response
Expected behavior
No response
Additional context
No response

Would you like me to modify the model parameters in the init function of multiple agents in camel/agents to align with the model_type? This will ensure consistency with the model_type in the init function of various models in camel/model.

Would you like someone to modify the model parameters in the init function of multiple agents in camel/agents to align with the model_type? This will ensure consistency with the model_type in the init function of various models in camel/model.

Yes, just renaming the model parameters to model_type to  e consistent with other files.

Hi, I noticed that the initial function of ChatAgent in agents/chat_agent.py is not in the same format as in other agents, is that what you are talking about? If this hasn't been modified, I can help change this.

Hi @FUYICC. This issue is about renaming the argument model in agents to model_type since model_type is used in other modules like models. Changing the name will make it more consistent. Not sure if @riverfor is working on it. Otherwise, please feel free to have a try.

#366 is merged.


Bad documentation

The page is utterly useless, as it does not describe what an IconId is, does or where to create one from!

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

ID: 4d3ab438-a674-5009-85aa-584641721286
Version Independent ID: b65af069-1d07-0a84-73a9-da0840f1e7e4
Content: IconId Struct (Microsoft.UI) - WinUI
Content Source: winui-api-build/xml/Microsoft.UI/IconId.xml
Product: reunion
Technology: winui-reference
GitHub Login: @jwmsft
Microsoft Alias: jimwalk

It has been a year and the original link does not work anymore but the documentation here is just as useless as it was before. As you seem not interested in fixing this i close this issue,.


Register version v0.4.1

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/48356
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.1 -m "<description of version>" 77b4cd6e74e93a85ff296052ffe66b777cc3fc86
git push origin v0.4.1


Fix "geo-parser miss some out-of-ranges"

Fix #21

coverage: 100.0%. remained the same
when pulling 96b79edfb090835462796213346589f95fef993d on 21-60-minutes-do-not-throw-range-exception
into d7c6088c10e1ae5b89c90f42a25740073e91b199 on main.


Multiple vendors flagging exe as a virus

Multiple vendors flagging as a virus:
https://www.virustotal.com/gui/file/cf935364f4d69f8ace0f5564fdf05dc34809fd5773db57f23ba9bdfa48f5c2bd

There's not much I can do there. All I can suggest is to avoid using crappy anti-virus software.
You can always choose to get source and build .exe file yourself - the build process is quite trivial, just run build.cmd batch file.


IONICSwap

Integrated FIBER Engine RIAC check that categorise Refinery and Ionic Assets per the documentation here
Integrated FIBER Engine RIACCheck that checks if the assets are Refinery/IONIC.

 Incase of RefineryAssets, first RefineryAsset will be swapped to foundry on SourceNetwork then foundryAsset will be swapped to TargetAsset on TargetNetwork

 Incase of IONICAssets, first IONICAsset will be swapped to swapRoutePair (which has pair liquidity for SourceAsset) and then to FoundryAsset on SourceNetwork then foundryAsset will be swapped to TargetAsset through multiple swapRoutePairs on TargetNetwork

Improved the functionality of DEX Swap within FiberRouterContract to process all Refinery/IONIC assets Swaps

Please NOTE:

Foundry.js is specific to swapping only Foundry/Bridgeable assets
Refinery.js is specific to swapping Refinery assets
Ionic.js is specific to swapping Ionic assets

In my opinion, overall codebase looks fine as prototype implementation but I think it will need to be more organized for scalability. I think this could be done as part of Multiswap SDK implementation.


[Hubs] Create a method for adding cost management databeyond 90 days

📝 Scenario
Azure Cost Export has a limitation of collecting cost management data for up to 90 days in the past. Many customers need data for up to 1 year, and this can only be done through the API. As a possible workaround, I used the Power BI cost management connector and saved it as a CSV using DAX Studio. Then, I uploaded the CSV file to the msexport container, and Data Factory was able to collect it. However, I can't see the files for the other months in Power BI.
💎 Solution
Create a solution that allows analyzing cost management data beyond 90 days using the API or DAX Studio.

[heart]         Arthur Silvany (HE/HIM) reacted to your message:

From: Michael Flanakin @.>
Sent: Monday, December 4, 2023 9:50:15 AM
To: microsoft/finops-toolkit @.>
Cc: Arthur Silvany (HE/HIM) @.>; Author @.>
Subject: Re: [microsoft/finops-toolkit] Create a method for adding cost management databeyond 90 days (Issue #449)
FYI - This will come with FinOps hubs 0.2. It's mostly done already. The one thing I believe we're missing is kicking off the backfill process.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/finops-toolkit/issues/449#issuecomment-1838187368, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHPFGCZ36JJIUJYCMN32D7LYHWMFPAVCNFSM6AAAAAA67QRXLKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZYGE4DOMZWHA.
You are receiving this because you authored the thread.Message ID: @.***>


gcc_impl_<target> contains stuff it probably shouldn't contain?

Looking at the content of gcc_impl_linux-64 (and removing the SOVERSION distinctions), there's a bunch of things in that output that probably shouldn't be there?
x86_64-conda-linux-gnu/lib/libasan.so
x86_64-conda-linux-gnu/lib/libasan_preinit.o
x86_64-conda-linux-gnu/lib/libatomic.so
x86_64-conda-linux-gnu/lib/libgfortran.so       # !!!
x86_64-conda-linux-gnu/lib/libgomp.so           # !!!
x86_64-conda-linux-gnu/lib/libgomp.spec
x86_64-conda-linux-gnu/lib/libhwasan_preinit.o
x86_64-conda-linux-gnu/lib/libitm.so
x86_64-conda-linux-gnu/lib/libitm.spec
x86_64-conda-linux-gnu/lib/liblsan.so
x86_64-conda-linux-gnu/lib/liblsan_preinit.o
x86_64-conda-linux-gnu/lib/libquadmath.so       # ???
x86_64-conda-linux-gnu/lib/libsanitizer.spec
x86_64-conda-linux-gnu/lib/libtsan.so
x86_64-conda-linux-gnu/lib/libtsan_preinit.o
x86_64-conda-linux-gnu/lib/libubsan.so

In particular, libgfortran and libgomp have separate outputs themselves, and libquadmath is also used by gfortran, without ever bringing gcc into the game.

OK, I guess this works because gfortran_impl depends on gcc_impl
https://github.com/conda-forge/ctng-compilers-feedstock/blob/8853ee7c0d15a9a73b35503c384f5894cf38c379/recipe/meta.yaml#L382
And since it's in the target-specific folder, it shouldn't be picked up by accident. Still scratching my head a bit about the layering, but happy to close if there's nothing to do...


dump() relies on Object.toString() for type detection, which is legacy and may break some apps

I ran across this issue after modifying Object.toString() to get more verbose debug messages in my app. Suddenly, yaml.dump() stopped working with YAMLException: unacceptable kind of an object to dump. I checked the source code and discovered that, to my surprise, js-yaml applies toString() to an object in order to check its type, and then performs string comparisons like type === '[object Object]' (in function writeNode()). This looks like a very risky way to check a type of an object, esp. when the ECMA specification explicitly says that:

The above definition of toString preserves compatibility for legacy code that uses toString as a test for those specific kinds of built-in objects. It does not provide a reliable type testing mechanism for other kinds of built-in or program defined objects. In addition, programs can use @@toStringTag in ways that will invalidate the reliability of such legacy type tests.

(source: https://tc39.es/ecma262/multipage/fundamental-objects.html#sec-object.prototype.tostring)
In short, the spec says that using toString() for type tests is a legacy and should not be relied upon. In my app, I observed that this approach may break the core functionality of js-yaml in a very unexpected way, and is a blocker for me (and supposedly for some other users, too) to using js-yaml.

In JS, modifying prototype methods of base objects is like shutting in to leg. That's not problem of this package.

All polyfills rely on modyfing base objects, so it's not a rare thing, rather a standard and officially accepted practice in JS - unavoidable in some cases given how flawed the standard API tends to be (e.g., re toString()). Sure, the issue I raised is not critical: most users will not notice, and myself, I already switched to another package. However, staying with this legacy way of type testing will surely come back to you more often over time, so I would be careful saying this is not your problem at all.

Polyfills only exdend missing methods, but not modify behaviour of existing. Also, this package is very stable and support legacy browsers many years, without problems. I respect your right to do anything you wish, but can't consider things you do as good practice (=> no need efforts for support). Sorry if disappointed.


Use file extension from --merge-output-format= when remuxing

When --add-metadata  or --embed-subs options are used file extension from --merge-output-format= should be used like it is used for muxing DASH videos.

I think we should use another option name, it could be confusing.


Change in pluralize syntax

The earlier version of photon client used plural syntax to access models. However, the latest prisma client uses singular model names to access DB. However, the documentation is not updated.
Fixing the same and raising pull request.

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

Thanks a lot for catching this @SaiNageswarS 🎉


Fix topic writer infinite reconnections

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Hello!
I've created topic writer with StartTimeout 30 seconds and Default or Retry RetryPolicy.
Then with ip6tables I've blocked the YDB port to get transport error.
The retries are expected to stop after 30 seconds, but this does not happened.
The reason why this is happened is connectionTimeout*resetAttemptEmpiricalCoefficient overflow.
If connectionTimeout is time.Duration(math.MaxInt64) (that is always true, since we do not have options to set connectionTimeout), then its multiplication to empirical coefficient (=10) gives negative value (see playground https://goplay.space/#qlIeS6o3PCz).
As a result, function CheckResetReconnectionCounters always returns true. Since CheckResetReconnectionCounters gives true, startOfRetries in connectionLoop loop method always sets to w.clock.Now(). Thus, in CheckRetryMode there are no chance to get retriesDuration > settings.StartTimeout to stop reconnections. As a result, we always get infinite reconnections in Retry and Default modes.
Pull request type

Please check the type of change your PR introduces:

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Topic writer with StartTimeout set to X and Default or Retry RetryPolicy.
Topic writer gets any error in Retry mode or retryable error in Default mode.
After X time topic writer continue reconnections.
Issue Number: N/A
What is the new behavior?
Topic writer with StartTimeout set to X and Default or Retry RetryPolicy.
Topic writer gets any error in Retry mode or retryable error in Default mode.
After X time topic writer stop reconnections.

Other information

Condition startOfRetries.IsZero() is needed to set startOfRetries for the first time.
Since with infinite connection duration CheckResetReconnectionCounters will return false.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (adb5de8) 67.64% compared to head (00fb002) 38.87%.
Report is 5 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1006       +/-   ##
===========================================
- Coverage   67.64%   38.87%   -28.78%
===========================================
  Files         261      208       -53
  Lines       24686    21472     -3214
===========================================
- Hits        16700     8347     -8353
- Misses       7127    12771     +5644
+ Partials      859      354      -505

Flag
Coverage Δ

?

go-1.17.x
?

go-1.20.x
?

go-1.21.x
38.87% <100.00%> (-28.54%)
:arrow_down:

integration
?

macOS
38.87% <100.00%> (+0.05%)
:arrow_up:

ubuntu
?

unit
38.87% <100.00%> (-0.05%)
:arrow_down:

windows
?

ydb-22.5
?

ydb-23.1
?

ydb-23.2
?

ydb-23.3
?

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@art22m thanks for your PR, I have been see it and will check soon in details.
Fix broken test please.

Hello.
Thanks for your pr and a lot of work for research and describe the bug.
Function CheckResetReconnectionCounters used for detect when we have established stream and reset counters - for good logging.
The PR solve bug with stop reconnection, but break logging/metric ux.
What you mean about set some reasonable constant for check "connection established" state for case with connection timeout is infinite?
For example - 1 minute. It will be mean:
if last connetion attemp was earler, than 1 minute - then connection was fine established and worked. And reset attemp counter to zero.

Hi, thanks for feedback.

if last connetion attemp was earler, than 1 minute - then connection was fine established and worked. And reset attempt counter to zero.

May be I did not get your point, but should we check last connection was earlier then constant, not later?
I guess you want to compare constant with lastConnectionAttempt variable.
If the connection was fine stablished, and it worked for some time (reconnectReason = writer.WaitClose(ctx)), then lastConnectionAttempt will be larger then our constant and we should zero the attempts.
Contrariwise, if we do not established our connection, this variable always will be small due to this: prevAttemptTime = now

When reconnect timeout is infinite, then we should check: if duration since last attempt was more then constant.
I want detect a situation:

Successfully establish a stream session
Successfully receive init message
Failed on fist (or one of firsts) message

And count the failure as new attempt for logs and for retry policy.

I've made one minute constant, but I guess right constant should be find empirically.

@art22m Thanks for the fix:)


Use getOrNull for manifest file in 3.6.0

Fix for https://github.com/bugsnag/bugsnag-android-gradle-plugin/issues/185
Goal
Fixes exception when building with AGP 3.6.0 and Bugsnag Gradle Plugin version 4.7.0+
Design
Provider.get() throws an exception if there is no value; use getOrNull instead to avoid the exception.
Changeset
Changed one line in BugsnagPlugin.groovy
Tests
Installed locally and verified my app builds with AGP 3.6.0-rc01.
Linked issues

Review

For the submitter, initial self-review:

[ ] Commented on code changes inline explain the reasoning behind the approach
[ ] Reviewed the test cases added for completeness and possible points for discussion
[ ] A changelog entry was added for the goal of this pull request
[x] Check the scope of the changeset - is everything in the diff required for the pull request?
This pull request is ready for:

[ ] Initial review of the intended approach, not yet feature complete
[ ] Structural review of the classes, functions, and properties modified
[x] Final review

For the pull request reviewer(s), this changeset has been reviewed for:

[ ] Consistency across platforms for structures or concepts added or modified
[ ] Consistency between the changeset and the goal stated above
[ ] Internal consistency with the rest of the library - is there any overlap between existing interfaces and any which have been added?
[ ] Usage friction - is the proposed change in usage cumbersome or complicated?
[ ] Performance and complexity - are there any cases of unexpected O(n^3) when iterating, recursing, flat mapping, etc?
[ ] Concurrency concerns - if components are accessed asynchronously, what issues will arise
[ ] Thoroughness of added tests and any missing edge cases
[ ] Idiomatic use of the language

Note: there seem to be a lot of existing codeNarc violations. Not sure if that is expected.

This was fixed by a different commit.


add Serialization page to quick start, #422

I'm not sure if the Why not HTML section is good or not, let me know if you want it removed.

Cool, thanks!

Sorry to see this go unmerged, but since it's been inactive for a while I'm going to close this PR. Please feel free to reopen with the changes requested in the initial review. Thanks!


Subdirectories

Hi Elias, let me know if you want me to do anything!
Cheers, Cedd

Hey @ceddlyburge.
I've created the pull request by mistake while I was playing with my fork.
Sorry about that, everything is ok.

Ooops! No worries ....


Ambry compaction and uploading very large files

I've fallen into a bit of a rabbit hole when trying to deal with large files in Ambry, and specifically trying to compact them out of logs properly. Namely, compaction does not seem to recognize deleted blobs if they were uploaded in parts.
To illustrate, I have a 3.8GB test file which is too large to be read into memory for uploading. Using the -T flag with CUrl, I can successfully upload the file into the system:
curl -i -H "x-ambry-service-id: CUrlUpload" -H "x-ambry-owner-id: whoami" -H "x-ambry-content-type: text/plain" -H "x-ambry-um-description: Demonstration File" http://localhost:1174/ -T ./a_very_large_file.txt -X POST
The blob can be retrieved with a GET as expected, but something seems to be wrong since if I use HEAD, the request-cost header claims that the blob is 0.0GB:
x-ambry-request-cost: WRITE_CAPACITY_UNIT=906.0; STORAGE_IN_GB=0.0
Despite this, deletion seems to work properly since further gets return a 410: Gone.
The problem arises when compaction tries to run, and the Ambry server simply says the following:
[2023-08-31 13:30:06,291] INFO StoreId: Partition[0]. DataDir: /home/meno/testambry/db/0. Capacity:<PHONE_NUMBER>0 is not eligible for compaction due to empty compaction details (com.github.ambry.store.CompactionManager$CompactionExecutor)
Thus, the blob stays in the log, ostensibly permanently.
The default CUrl command provided in the quickstart guide works (the request-cost shows as non-zero and compaction will run successfully) but that uses the --data-binary flag which requires the blob to be small enough to be fully read into memory, which is not possible for many of the blobs I'm working with.
I've also tried using the requests library in Python to upload the file without CUrl, but that runs into the same issues with large files.
Additionally, this may or may not be related, all of the upload operations (whether using --data-binary, -T or the Python script) result in cryptic quota errors:
[2023-08-31 13:36:40,513] WARN Could not get recommendation for quota [READ_CAPACITY_UNIT, WRITE_CAPACITY_UNIT] due to exception: Could not recommend for request / with resourceid -1 due to Couldn't find quota for resource: -1 (com.github.ambry.quota.AmbryQuotaManager)
[2023-08-31 13:36:41,180] WARN Exception com.github.ambry.quota.QuotaException: Could not charge for request DefaultHttpRequest(decodeResult: success, version: HTTP/1.1)
POST / HTTP/1.1
Host: localhost:1174
User-Agent: curl/7.81.0
Accept: */*
x-ambry-service-id: CUrlUpload
x-ambry-owner-id: whoami
x-ambry-content-type: text/plain
x-ambry-um-description: Demonstration File
Content-Type: application/x-www-form-urlencoded
content-length: 385159 due to Couldn't find quota for resource: -1 while charging for [READ_CAPACITY_UNIT, WRITE_CAPACITY_UNIT] quotas. (com.github.ambry.quota.AmbryQuotaManager)
[2023-08-31 13:36:41,209] WARN Could not get recommendation for quota [READ_CAPACITY_UNIT, WRITE_CAPACITY_UNIT] due to exception: Could not recommend for request / with resourceid -1 due to Couldn't find quota for resource: -1 (com.github.ambry.quota.AmbryQuotaManager)

These errors repeat by the hundreds for large files. I don't know what to make of this, as the upload/download operations still work just fine—it's just the compaction that causes issues. Any insights?

wtf


Ability to have two bridges running at same time?

s it possible to have two bridges running simultaneously?
I have two computers and occasionally I need to turn them off but not simultaneously,  I would still like to have access 100% of the time aND not have to wait for next reboot

it is possible to run two Bridges at the same time. What you can't do is have them configured with the same devices with the same names.
Well you can but it confuses Alexa. A way around this would be to have the devices named slightly different maybe with A/B or 1/2 though Alexa has difficulty with numbers sometimes.  Amazon suggests avoiding them.

If you run the ha-bridge with systemctl you could keep one on "hot stand by".  Just stop the bridge on one with the $ sudo systemctl stop habridge command.  When you want to swap computers just run $ sudo systemctl stop habridge on the one coming out of service, and $ sudo systemctl start habridge on the other going into service.

@audiofreak9
Thank you for the reply, however I was hoping for a seemless integration.  While this is a great option and there probaby is a way to code all of the above you have stated and maybe even automate the codes it is a bit out of my league.
@tuicemen this is something that I can already do with Alexa and Vera.  Vera is just starting to come out of beta with a decent working firmware that would allow me to run both.   I don't want to completely ditch the bridge because I think it has way more control over how vera and alexa work.  Although I do have the vera/alexa skill working much better with the thermostat that I have.  I do understand that alexa gets confused with two of the same name or numbers in the name and this has been a crutch of Alexa.  For now I'll live.  Lol. Thank you both for the replies

Also @audiofreak is that code for Windows or rasp pi?  I should have been more clear and stated both are windows.  I haven't made the jump yet but really would love to soon since it is a much lower cost to run that instead of a windows htpc.

@michaelhinchey The systemctl command is for RPi (Linux) not Windows.  The ~$25 investment in an RPi will be well worth the 4watts it takes to run it vs. the typical 25watts of a PC (or more), plus you can just set it and forget it, no need to swap between Windows machines.  If you are invested in using the ha-bridge then the RPi is a no brainer purchase.

@michaelhinchey
I had two Windows PCs with the bridge configured the same on both PCs as long as only one is running the bridge you'll have no issues Alexa will see the bridge that is on line and report all the devices on it as on line. The devices on the bridge which is not running will be reported as off line.
So as long as you remember to only have one bridge running at a time you should have no problems.

@tuicemen Understood.  However, I rarely turn both my pc s off.  I've been looking for ways to do so but really haven't found any and still have the bridge running.  Sometimes my gf will be on pc in bedroom and I'll be in the living room so that is not an option.   It is in a perfect world though. Thx for the option and reply.

I did buy a rpi 3 about a yr ago but it sat because I has some issues with work I needed to get done and then didn't have time to work on it.  So I returned it.  I did buy a rpi zero at the time because it was .99 cents.  Still sitting.lol.
Maybe I will invest in the pi.  Is the original one ok and still will work?  I've never used the software for this and I guess it's command based.  Any help there would be great. Thx

I'd suggest an RPi B+ or newer.  Just follow the README for setup on the RPi.  Otherwise are you good to go on this issue.

Yes thank you. Sorry I didn't reply back.   I'm sure I will need more help whenever I goto build the bridge since this will be the first time I am working outside of windows.


generate.py does not work due to change DOM in document page

Hi,
Current company-terraform-data.el does not include new or updated AWS Services such as aws_eks_cluster resource, mixed_instances_policy parameter in aws_autoscaling_group resource.
So I tried to execute utils/generate.py to refresh my company-terraform-data.el but It couldn't.
That script causes this error:
$ python utils/generate.py
Traceback (most recent call last):
  File "utils/generate.py", line 294, in <module>
    get_providers()
  File "utils/generate.py", line 99, in get_providers
    table = div_inner.find_all('table')[0]
IndexError: list index out of range

I think the reason is the Terraform document page seems to have been updated and restructured DOM.
I tried to repair this script, but it is difficult for me because I don't know that page before update.
Could you please repair it to be executable?

I'd love to, but it's likely to be a lot of effort in vain.
Generally, gathering the list of resources and attributes by parsing documentation web pages is a terrible idea. The documentation is human-readable, and lacks a well-defined structure. Various providers use different formatting styles and different document structure. Which makes the documentation a terrible source for automated processing. Thus generate.py is full of wierd unmaintainable tricks, specific rules, handling exceptions and very specific cases, in an attempt to extract the structure of these documents. Completely bonkers idea.
But to be fair, terrible as it is, this seems to be the only possible way of extracting resource lists, their attributes and their descriptions. Some most prominent providers might have a more structured or consistent representation of their resources and attributes, but as long as some providers don't do that, this isn't a strategy worth pursuing. There are many providers that get updated very rarely, and I can't expect them all to adapt some shared machine-readable documentation format - though I believe this is an effort that should be pursued by terraform developers.
With that in mind, generate.py was never meant to be easily maintainable. Even small changes in docs format require a lot of fixes in this script; since a lot of documentation styles have changed since I originally wrote it, I believe at this point it would be easier to rewrite the script entirely rather than fix it... which obviously requires a lot of tedious work. Eventually I will have to do that, though!
I have also considered borrowing the data file from some other terraform plugin for other editors, but back when I last researched them, none of their data sets was comprehensive enough. I should revisit this idea.

@rafalcieslak Have you seen this repository maintained for the IntelliJ plugin? Seems like it could fit right in with some changes to the parsing?
This way you don't have to maintain your own parsing tool.

Hi @rize, this is an awesome suggestion! If there was indeed a repository that gathered resources/attributes/functions and other terraform keywords, I would be very happy to use it instead of maintaning my own tool. That terraform-metadata repository (or it's clones I've seen present in other editors' plugins) is a step in the right direction, but it's not quite there yet. Most importantly, it does not provide docstrings for any resource, which are available when using company-terraform. This is understandable, because documentation is not embedded within provider sources, it exists solely as html/markdown files in providers' repository - making it difficult to extract it automatically. Additionally, terraform-metadata is only partially automated, note that backends or interpolation functions list cannot be automatically updated, and you will see that, for instance, it is missing all functions introduced by terraform 0.12 back in May.
So while I really hate having to maintain my own tool, I feel that, at this point, switching the schema source from a web-scrapper to terraform-metadata would be replacing one half-arsed tool with another. I hope this will change in the future, but I don't imagine this happening without a lot of work done between terraform providers to standardize the documentation format.

Hi @rafalcieslak, thanks for the in-depth explaination! I really appreciate it.
I started scouring for other tools that have documentation and it looks like VSCode's plugin might have what you're looking for? It's important to note that the provider list is hopelessly incomplete, only having the most commonly used 6 providers (aws, azure, gcloud, etc...).
On that note, it might be better only supporting a subset of providers till Hashicorp comes out with a full API specification.


RadzenDataGrid crashes if DataGridSettings.CurrentPage exceeds PageNumbersCount

Describe the bug
The RadzenDataGrid crashes when loading DataGridSettings if the CurrentPage (index from 0) is greater than PageNumbersCount - 2 (which is page count + 1 for some reason). This only happens when loading dynamic data (e.g. date and time updated or state).
To Reproduce
Steps to reproduce the behavior:

Using the blazor.radzen DataGrid save/load settings example, store the DataGridSettings in local browser storage.
Select the last page of the DataGrid and check the CurrentPage value is set in in local browser storage.
Either delete some rows from the test data, or set a filter that means there will be less data on refresh (e.g. duration < 00:00:10).
Refresh the page and you'll see the whole UI freeze.

Expected behavior
An exception should be thrown without locking up the UI.
Desktop (please complete the following information):

OS: Windows
Browser Chrome and Firefox
"Radzen.Blazor" Version="4.15.7"

Additional context
The workaround is to clear the DataGridSettings current page index in LoadStateAsync to avoid trying to load a non-existent page.
if (settings.CurrentPage > 0)
    settings.CurrentPage = null;

I navigated to the last page, filtered EmpoyeeID = 1 which will result in single page and refreshed the page - no UI freeze, no errors:

This assumes you either have a stopwatch column or you dynamically calculate the difference between a given start time and the current time.

I'm afraid that I'm unable to reproduce the problem nor to understand what you are saying.


ReportProcessing node for goodtables validation

We want to add a new pipeline which runs goodtables validation on each source of the pipeline.

Result is a json report, stored as a non-tabular resource in the datapackage
Failure to validate doesn’t fail the pipeline!

Acceptance Criteria

[x] there is non-tabular resource on S3 named datapackage_report

Tasks

[x] new processingNode
[x] update node collectors

Analysis

Create new ReportProcessingNode with the following processors:

class ReportProcessingNode(BaseProcessingNode):
    def __init__(self, available_artifacts, outputs):
        super(ReportProcessingNode, self).__init__(available_artifacts, outputs)
        self.outputs = outputs
        self.fmt = 'json'

    def get_artifacts(self):
        datahub_type = 'derived/report'
        resource_name = 'datapackage_report'
        tabular_artifacts = [
            artifact for artifact in self.available_artifacts
            if artifact.datahub_type == 'source/tabular'
        ]
        output = ProcessingArtifact(
            datahub_type, resource_name,
            tabular_artifacts, [],
            [('assembler.validate_resource', {})],
            False)
        yield output

ReportProcessingNode will come first in node collector

FIXED


Follow Artist endpoint

When will the Follow Artist endpoint be implemented?? It's supported by the default Spotify API here: https://developer.spotify.com/web-api/follow-artists-users/
I need it for a project im working on.

@MileanCo This is already implemented in the method followArtists().


Enhancement request: Get value of control's property inside expression binding

Hi,
When I'm defining a new control inside an XML-View , I'd like to be able to use the reference to that control and use the value of its certain property inside the expression binding.
Something like this:
...
property="{= thisControl.valueState === 'Error' ? _doThis_ : _doThat_ }"
...
Of course, I could just define an extra JSONModel and store the controls properties there in order to use it later in the expression binding, but that concept doesn't work when I'm working with aggregation binding.

Hi boghyon,
Just a check if i understand the enhancement correctly:
I put up this scenario:
<Input value="{= thisControl.getValueState() === 'Error' ? 'Wrong' : 'All good' } valueState="Error" />"
Here are my Questions

is thisControl the Input?
When the property 'valueState' changes is the property 'value' is changed?
thisControl.valueState is a field did you mean the getter?

If this is the desired feature, it is probably a big change because the current change detection of bindings is currently only triggered by model Data. Control setters are not inside of the change detection. So i don't know if this enhancement will be done.
When you answered my 2 Questions i will forward it to the responsible developer.
a Note to the local JSON model: probably thisControl.valueState is probably also bound to a value of the model. Then you could repeat the binding inside of the valueState. But i totally agree that it is very ugly to spread a lot of complicated bindings across the VIews so JSON model is probably the way to go.
Best regards,
Tobias

Hi Tobias,
Sorry for the lack of information. Yes, you are correct. In this example, "thisControl" is the Input. When its property "valueState" changes, its "value" property should also be notified and change its value accordingly.
For the third question, I cannot give a definite answer. It doesn't have to be a getter or a field, since my example code was just a concept (it could be however it's easy for the responsible developers to implement this feature).
I understand that it would be a big change. I just hope that there is an easier way to get desired control's properties inside an expression binding (or forward that control's instance to a formatter) so that we don't have to rely on JSON models.

Hi boghyon,
i have forwarded it to the responsible developer and he will get back to you.
Best regards,
Tobias

Isn't this very similar to the Pull Request https://github.com/SAP/openui5/pull/727 ?

Yes, I agree. Your pull request provides an answer to Boghyon's question.
As I already commented, your suggestion is a general "enhancement to basic binding concepts which of course can be used inside expression binding, but not only there". This is a good thing! But it means the tests are misplaced and we need other people's approval as well, not just mine.
I like the idea of binding to another (or the own) control's properties via a syntax similar to "model>path". But: "@" is not a reserved character within a model name, e.g. "@i18n" is a perfectly valid model name as of today. I fear this change is somewhat incompatible. We need to find a better syntax.


Failed to find any device to open DMK23G445

Describe the bug
I was trying to verify the installation for two DMK23G445 cameras. They are responsive to tcam-gigetool. However, there is no device listed in tcam-capture.

Running gst-launch-1.0 tcambin ! videoconvert ! ximagesink returns the following error:

gst-launch-1.0 --gst-debug=tcamsrc:5 tcambin ! videoconvert ! ximagesink
Setting pipeline to PAUSED ...
0:00:00.033249189 52281 0x555ae5c6ec60 WARN                 tcamsrc gsttcamsrc.cpp:249:open_source_element:<tcambin-source> error: Failed to find any device to open.
ERROR: Pipeline doesn't want to pause.
ERROR: from element /GstPipeline:pipeline0/GstTcamBin:tcambin0/GstTcamSrc:tcambin-source: Failed to find any device to open.
Additional debug info:
/home/yac1nst/repos/tiscamera/src/gstreamer-1.0/tcamsrc/gsttcamsrc.cpp(249): open_source_element (): /GstPipeline:pipeline0/GstTcamBin:tcambin0/GstTcamSrc:tcambin-source
Setting pipeline to NULL ...
Freeing pipeline ...

tcam-ctrl -l returns nothing
ping successful for both cam
tcam-gigetool list shows both camera with correct info

tcam-gigetool list
Model Name           | Serial Number    | User Defined Name    | Current IP
DMK 23G445             216106**           A2                     <IP_ADDRESS>
DMK 23G445             216106**           A1                     <IP_ADDRESS>

tcam-gigetool check-control said that both cameras are not controlled by anyone.

computer used (please complete the following information):

OS: Ubuntu 20.04
Architecture/Platform: amd64, Desktop
Camera  model you use
DMK23G445
tiscamera version (tcam-ctrl --version, if possible)
Tcam:	1.1.1_master/0be2537f_rev_4143
Aravis:	0.8_version_e977fa4
Modules:	gst_aravis_arvu3v_v4l2_libusb_tools_doc

Just saw that the current version doesn't support the cam model. I have to use older version.

The current version can support your camera too:
export TCAM_DISABLE_DEVICE_BLACKLIST=1

(Source: https://www.theimagingsource.com/en-us/documentation/tiscamera/environment.html?highlight=environment#tcam-disable-device-blacklist)
Stefan


Command to keep characters the same on every new prompt?

This is an issue on most AI tools. The characters don't stay the same for consistency.
Is there a specific prompt that can be used every time?

a much needed feature

This is similar problem as next issue , now, Flux can generate consistent character pictures easily, so with those consistent character pictures as an initial images, we can develop many video clips with the character.

We recently released a LoRA fine-tuning script at https://github.com/genmoai/mochi/tree/main/demos/fine_tuner. It's now possible to fine-tune Mochi on a set of videos of yourself, then use the fine-tuned model to generate personalized videos. You can fine-tune on short videos and still retain the ability to generate longer videos.


Clean up some logging found while testing

While testing I found a few places where we our logging could be improved. Fixing those and deleting status.py which should have been removed when it got moved to common.

👍


I want to completely remove ZHCMessagesInputToolbar empty space

Hi , How to remove ZHCMessagesInputToolbar, so that i can use my own custom

If you want to remove ZHCMessagesInputToolbar, you can only modify the source code. You can fork the whole project and then modify InputMessageBarView in ZHCMessagesViewController. Of course, many other unforeseen problems may arise during the revision process, and if you are sure you want to modify it, you must understand the framework of the entire project.

Thanks i made it done and once more thanks for wonderful library


Route-appending for global modules

Consider a sidebar that can slide in/out with a module containing account-profile data.
Would it be possible to tell the router to append a global route /account to the current URL and state of the application? So when the the URL equals /home/account the state of the sidebar is set to active.
This should also apply for other routes that inherits from the account route and all child routes. Routes like /home/second/account and /info/account should activate the sidebar.

The best way to handle this is to make the sidebar work independent of routing. Simply have the sidebar subscribe to a message on the event aggregator. Whenever a screen needs to show the sidebar with some data, it simply publishes the message.


Bug: Routes with duplicate parameter names are not shown in OpenAPI rendering

Description
OpenAPI schema fails to generate if there's 2 parameters present with identical name, route is subsequently removed from OpenAPI schema.
URL to code causing the issue
No response
MCVE
from typing import Annotated

from litestar import Litestar, post
from litestar.params import Parameter

@post("/test")
async def route(
    name: Annotated[str | None, Parameter(cookie="name")] = None,
    name_header: Annotated[str | None, Parameter(header="name")] = None,
) -> str:
    return name or name_header or ""

app = Litestar(
    route_handlers=[route],
)

Steps to reproduce
1. Launch uvicorn server `uvicorn app:app`
2. Go to http://<IP_ADDRESS>:8000/schema/swagger
3. See error

Screenshots
No response
Logs
$ uvicorn app:app
INFO:     Started server process [8376]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     <IP_ADDRESS>:54807 - "GET /schema/swagger HTTP/1.1" 500 Internal Server Error
INFO:     <IP_ADDRESS>:54807 - "GET /schema/swagger HTTP/1.1" 200 OK

Litestar Version
2.3.2
Platform

[ ] Linux
[ ] Mac
[X] Windows
[ ] Other (Please specify in the description above)

Changed the title to be more descriptive of the actual issue. The routes continue to work, it's just that they are not shown on any OpenAPI renderers (swagger, elements etc) after the first load which raises the following exception (not attaching the stack trace).
500: OpenAPI schema generation for handler app.route detected multiple parameters named 'name' with different types.

I should have originally attached stacktrace, but I'm fairly new to litestar and didn't know how to configure debug/logging options 🙂

I should have originally attached stacktrace, but I'm fairly new to litestar and didn't know how to configure debug/logging options 🙂

No worries 🙂 I just do app = Litestar(route_handlers=[route], debug=True) and run the app.

Also on first request openapi.json returns status code 500 🤔

This is not limited to your issue per se, most issues in the schema generation exhibit the same behavior, it sort of gets "cached" which is why you only see the error on the first try and the "non erroneous" schema from the second.

I've opened #2788 to address this, it would now only be an error to have two openapi params declared in the same place (i.e., two dependencies that both declare a "name" header with different constraints).
I've got a slightly modified MCVE:
from __future__ import annotations

from typing import Dict, Optional

from typing_extensions import Annotated

from litestar import Litestar, post
from litestar.params import Parameter

@post("/test")
async def route(
    name_cookie: Annotated[Optional[str], Parameter(cookie="name", description="name via cookie")] = None,
    name_header: Annotated[Optional[str], Parameter(header="name", description="name via header")] = None,
) -> Dict[str, str]:
    return {"name_cookie": name_cookie or "", "name_header": name_header or ""}

app = Litestar(route_handlers=[route], debug=True)

And that renders:

So I suppose that satisfies this particular issue, however, when I try the route out:

Notice that I don't get a value for the cookie name. It doesn't seem to make any difference if I send the header name or not, the cookie name doesn't come through. Was this ever working for you @ThirVondukr?


CAMEL-10126: Refactored completionSize & completionTimeout expressions

completionTimeout & completionSize attributes and expression usages are confusing in xml dsl, so if we have to change the names, we need to change the names in camel-spring.xsd and camel-blueprint.xsd files.
With the below respective changes,
camel-spring.xsd
<xs:element name="completionTimeoutExpr" type="tns:expressionSubElementDefinition" minOccurs="0"/>
<xs:element name="completionSizeExpr" type="tns:expressionSubElementDefinition" minOccurs="0"/>
camel-blueprint.xsd
<xs:element minOccurs="0" name="completionTimeoutExpr" type="tns:expressionSubElementDefinition"/>
<xs:element minOccurs="0" name="completionSizeExpr" type="tns:expressionSubElementDefinition"/>

I tested the committed code with the above xsd changes and i am able to refactor code with 'completionTimeoutExpr' and 'completionSizeExpr' for the use in expressions. I can't commit or have access to camel-spring.xsd and camel-blueprint.xsd, because everytime, i build the camel-spring and camel-blueprint, the xsd changes.

@vrlgohel, FYI: you don't need to recreate PR in case of something changes, it's just push again in the same branch with new or squashed commits

We need to wait with this for Camel 3.0 as we cannot break the DSL

@dmvolod, thanks for the suggestion. Yes, i thought of doing that, but i wanted to ensure everything was in sync with master, hence created a new PR. No issues, i will squah the commit from on.
@davsclaus, no issues. Please review this and let me know if any more changes are required when we release Camel 3.0. I think this fix should do.

Lets wait until Camel 3.0 with doing code changes. We should avoid using xxxExpr in the Java DSL etc. I would suggest to close this PR and we look at this later.

Hi Claus,
Okay,  i will close this request then. I will wait for Camel 3.0 release and work on this issue, if more changes are required. Thanks.


DISTINCT in Aggregating functions

Is currently missing the support for DISTINCT in Aggregating functions.
For instance the following:
RETURN collect(DISTINCT this { .id }) AS data
or
RETURN count(DISTINCT this { .id }) AS data
Are valid Cypher.
Reference: https://neo4j.com/docs/cypher-manual/current/functions/aggregating/#_counting_with_and_without_duplicates

Aggregation functions already support DISTINCT (although it is not yet documented):
The following js should generate something similar to what you need
Cypher.collect(new Cypher.Node()).distinct();


update gtool

Codecov Report

Merging #46 into develop will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop      #46      +/-   ##
===========================================
- Coverage    38.37%   38.34%   -0.03%
===========================================
  Files          120      120
  Lines        10311    10311
===========================================
- Hits          3957     3954       -3
- Misses        5955     5957       +2
- Partials       399      400       +1

Impacted Files
Coverage Δ

gemmill/blockchain/pool.go
68.79% <0%> (-1.13%)
:arrow_down:


Optional one-to-one - code example not complete - missing the mentioned nullable foreign key property

It seems that in the in the expample of the "Optional one-to-one" the code does not show the mentioned (within the text below) nullable foreign key property. This might be somewhat missleading despite the minimal footprint of the example :)

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

ID: 93b48a8b-469b-1dbe-e6b9-66a58d29e841
Version Independent ID: 93b48a8b-469b-1dbe-e6b9-66a58d29e841
Content: One-to-many relationships - EF Core
Content Source: entity-framework/core/modeling/relationships/one-to-many.md
Product: entity-framework
Technology: entity-framework-core
GitHub Login: @ajcvickers
Microsoft Alias: avickers

@jsonserverkit Thanks--I submitted https://github.com/dotnet/EntityFramework.Docs/pull/4357 to fix this. The wrong code sample was referenced in the document.


minor optimize slice filter performance

FilterInPlace uses mutated in place, so it avoids allocating temporary space during execution.

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot_istio
fc6ffccca301649e837d06a2470e95e9d1afeebc
link
true
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
e4660b205d645683787ddba0dfb392d3b93d4be2
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test unit-tests

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot_istio
7095f4521be86462b431269901f4379cf59d2453
link
unknown
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Status - DOWN // Not able to use the scraper

Heya !
First, I'd like to thank you for making this project !
After thoroughly filling the info asked when running the script, I'm faced with this:
Traceback (most recent call last):
  File "C:\Program Files\Python39\Scripts\onlyfans-scraper-script.py", line 33, in <module>
    sys.exit(load_entry_point('onlyfans-scraper==2.0.0', 'console_scripts', 'onlyfans-scraper')())
  File "c:\program files\python39\lib\site-packages\onlyfans_scraper\scraper.py", line 330, in main
    process_prompts()
  File "c:\program files\python39\lib\site-packages\onlyfans_scraper\scraper.py", line 241, in process_prompts
    model_id = profile.get_id(headers, username)
  File "c:\program files\python39\lib\site-packages\onlyfans_scraper\api\profile.py", line 73, in get_id
    r.raise_for_status()
  File "c:\program files\python39\lib\site-packages\httpx\_models.py", line 736, in raise_for_status
    raise HTTPStatusError(message, request=request, response=self)
httpx.HTTPStatusError: Client error '400 Bad Request' for url 'https://onlyfans.com/api2/v2/users/toohsakaa'
For more information check: https://httpstatuses.com/400

Do you have any idea on what's causing this ?
Thanks in a million !

Status down generally means the login is invalid, are you sure you filled everything out correctly?

What's interesting is the 400 status.
400 Bad Request
The server cannot or will not process the request due to something that is perceived to be a client error (e.g., malformed request syntax, invalid request message framing, or deceptive request routing).

You must provide all the variables exactly as they are in your browser, including the user agent, otherwise, it will fail.


'hoodie.datasource.write.precombine.field' only supports a single field

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
Why does the configuration "hoodie.datasource.write.precombine.field" only support sorting on a single field? I think it is also possible to specify multiple fields for sorting
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Environment Description

Hudi version : 0.12.3

Spark version : 3.2.3

Hive version : 3.1.3

Hadoop version : 3.3.0

Storage (HDFS/S3/GCS..) : hdfs

Running on Docker? (yes/no) : no

Additional context
Add any other context about the problem here.
Stacktrace
Add the stacktrace of the error.

You are right, and we are planning to support this feature.


Further exploration of conditional visibility

This gets a few more scenarios working end-to-end. We can tweak the expression language and property arrangement.
Relates to https://github.com/dotnet/project-system/issues/6627.

Fixes calculation of TargetMultipleFrameworks (note target framework dropdown not populating)

Makes "Startup Object" conditionally visible based on "Output Type"

On the packaging page, group licensing options together and set up conditional visibility between them

Use conditional visibility for code signing options

Fixes calculation of TargetMultipleFrameworks (note target framework dropdown not populating)

I figured this one out. The drop down is getting the right set of supported values, and if you expand the drop down you will see the display names ".NET Core 2.0", ".NET Core 2.1", ".NET Core 3.0", etc. These display names and the corresponding values (.netcoreapp2.0, .netcoreapp2.1, .netcoreapp3.0, etc.) are ultimately coming from the SupportedTargetFrameworksProvider.
However, the current value is coming from the "TargetFrameworkMoniker" property, and that value has the form ".NETCoreApp,Version=3.0" or similar. So the current value doesn't match any of the admissible values.
I think the solution here is to bind directly to the "TargetFramework" property instead of "TargetFrameworkMoniker", and I've gone ahead and made that change.

Added hyperlinks.
9b6d0008099415cc0d06bd926fc2a449190e5046

b7ed4475727f296c83d8be099e9841862a991d4a

636144c6cf5cd438f25a24295799ffa2b93291c0


[MINOR] gpu memory fixes

Ping @niketanpansare

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2100/

Thanks @niketanpansare, I shall merge.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2109/


GlobalProtect Logs

It would be great if you included a GlobalProtect dashboard.  I have built a few for myself but I'm sure the community would appreciate having one out of the box that they can either use or build upon.

We are currently working on a dashboard which is nearly complete. Do you mind sharing your current dashboard for GlobalProtect? I am interested in what kind of information your showing.

Sure I can share it.  Which address should I send the images or do you want the source?
Kent Farries | Sr. Systems Analyst, Security Intelligence & Analytics
TRANSALTA CORPORATION
T: +1<PHONE_NUMBER> | C: +1<PHONE_NUMBER>
<EMAIL_ADDRESS>| Webhttp://www.transalta.com/ | Facebookhttp://www.facebook.com/TransAlta | Twitterhttp://www.twitter.com/TransAlta
From: Paul Nguyen<EMAIL_ADDRESS>Sent: Thursday, April 14, 2016 12:03 PM
To: PaloAltoNetworks/SplunkforPaloAltoNetworks<EMAIL_ADDRESS>Cc: Kent Farries<EMAIL_ADDRESS>Subject: Re: [PaloAltoNetworks/SplunkforPaloAltoNetworks] GlobalProtect Logs (#19)
We are currently working on a dashboard which is nearly complete. Do you mind sharing your current dashboard for GlobalProtect? I am interested in what kind of information your showing.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/PaloAltoNetworks/SplunkforPaloAltoNetworks/issues/19#issuecomment-210079088

images are fine<EMAIL_ADDRESS>


OSX error on example

Hi,
I'm just running the basic example from the GitHub page
using HttpServer
http = HttpHandler() do req::Request, res::Response
Response( ismatch(r"^/hello/",req.resource) ? string("Hello ", split(req.resource,'/')[3], "!") : 404 )
end
server = Server( http )
run( server, 8000 )
It launches the server instance but immediately throws the following error:
Listening on <IP_ADDRESS>:8000...
ERROR: type cannot be constructed
in ClientParser at /Users/marcstein/.julia/v0.3/HttpServer/src/RequestParser.jl:133
in handle_http_request at /Users/marcstein/.julia/v0.3/HttpServer/src/HttpServer.jl:224
in run at /Users/marcstein/.julia/v0.3/HttpServer/src/HttpServer.jl:305
in run at /Users/marcstein/.julia/v0.3/HttpServer/src/HttpServer.jl:309
I'm running Julia 0.3.11 on OSX El Capitan Beta 7. Has anyone gotten this to work on El Capitan?

This will be fixed in the next few days - thanks for your patience.

If you run Pkg.update(), this will work now. Please reopen if you're still having issues.

Jonathan,
Thank you SO much for the fast solution! That worked perfectly.
Best,
Marc
On Sun, Aug 30, 2015 at 8:06 AM, Jonathan Malmaud<EMAIL_ADDRESS>wrote:

If you run Pkg.update(), this will work now. Please reopen if you're
still having issues.
—
Reply to this email directly or view it on GitHub
https://github.com/JuliaWeb/HttpServer.jl/issues/61#issuecomment-136131233
.


Support both manifest v2 & v3

Since only Chrome support v3 for now: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/manifest.json/manifest_version

Blocked by https://github.com/w3c/ServiceWorker/issues/1577


chore: release 1.0.0

:robot: I have created a release *beep* *boop*
1.0.0 (2019-05-20)
⚠ BREAKING CHANGES

upgrade engines field to >=8.10.0 (#272)

Bug Fixes

DEADLINE_EXCEEDED no longer treated as idempotent and retried (1a0000c)
DEADLINE_EXCEEDED retry code is idempotent (#281) (9c00cf6)
deps: update dependency @google-cloud/pubsub to ^0.29.0 (#283) (b1f9ee1)
deps: update dependency google-gax to ^0.26.0 (#270) (68258a6)
deps: update dependency google-gax to v1 (#280) (2cbb997)

Build System

upgrade engines field to >=8.10.0 (#272) (233e814)

Release is at https://github.com/googleapis/nodejs-dlp/releases/tag/v1.0.0

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Results Reproducibility

Hello,
Thank you for sharing your work.
I'm trying to reproduce ImageNet-C results. I downloaded ImageNet-C from here, and created a dataset using torchvision.datasets.ImageFolder and modified the __getitem__ to allow for is_carry_index.
First, evaluating pretrained ResNet50 (torchvision weights ImageNet1K-V1) on ImageNet-C (level 5) reports 83.11% error rate which is higher than the 82.22% reported in this repo. The error rate on the snow corruption using run_ttac_no_without_queue.sh is around 50% instead of 46.64%. The seed is fixed everywhere in the code so I'm expecting to reproduce both numbers. Any insights as to what might be wrong? I'm using PyTorch 1.12.1 and CUDA 10.2.89.
I reproduced the results on CIFAR-10 successfully.
Your suggestions and insights would be really appreciated.
Best regards,
Hani

Hi Hani,
Thank you for your interest in this work.
The ImageNet-C used in our code is generated by create_corruption_dataset.py. In this script, we leverage the package from here to generate various corrupted sets and save them as "*.pth".  You can refer to Readme.md for this specific operation process.
Also, I again ran the approximate results as the values reported in this repo, as shown below.
82.22% is the directly testing error rate;
46.54% is the TTAC (without queue) error rate. The partial output results are as follows.

Finally, I hope this reply is helpful to you and I kindly suggest you could download and run this released code firstly.
Yongyi

Hello,
Thank you for your reply.
I generated the snow corruption from ImageNet and I was able to reproduce your numbers. I got 82.10% error rate using ResNet50 and 46.172% error rate after adaptation.
Thank you for your suggestion.
Best reagards,
Hani


Presets are not being passed properly to vueify

I've been using vue-brunch and got into the need to use vuex's mapGetters helper method, which in turn requires babel's transform-rest-object-spread plugin to work. It appears that, even though I was passing babel's "stage-3" preset as part of brunch's plugin config, vue-brunch is not passing said option to vueify, so I was forced to add a .babelrc with my options, forcing vueify to see it.
I'm not sure if this is a vue-brunch bug per se, as it might be vueify does not offer any interface to pass it babel plugins or presets other than via a .babelrc file.
Yet, I'd like to at least leave this reported here in case someone else stumbles upon this very same issue.
I know @nblackburn is working on a rewrite of this plugin and dropping vueify dependencies while at that, so this will certainly be solved then. However, until that happens, please be aware you will still need the .babelrc file with your babel presets and plugins for vueify to play nice along with brunch.

Only the "plugins.vue" part of the Brunch Config is passed to vueify, which kind of makes sense. I worked around this problem by added this to the bottom of my brunch-config.js:
// Pass Babel config to Vue
exports.plugins.vue = exports.plugins.vue || {};
exports.plugins.vue.babel = exports.plugins.babel || {};


artemis text logo

Description of PR
Add ARTEMIS text logo and adobe illustrator file.
Resolves #169
What component(s) does this PR affect?

[ ] Back-End (Database, Microservices, Containers, etc)
[x] Front-End (Flask, API, etc)
[x] Docs
[ ] Build System

Does the PR require changes on other components? If yes, please mark the components:

[ ] Back-End (Database, Microservices, Containers, etc)
[ ] Front-End (Flask, API, etc)
[ ] Docs
[ ] Build System

Related Issue

Resolves #
Solution

Type

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Docs update
[ ] None of the above

Checklist:

[x] I have read the contributing guide and my code conforms to the guidelines.
[ ] This change requires a change in the documentation.
[ ] I have updated the documentation accordingly.

@pgigis in the branch, could you also add it to the README file? (it is .md)


Support drain timeout in Helm chart

I'm working with a team that is using the helm chart for deployment.  They are very interested in the drain-timeout feature mentioned in the main readme.
      --drain-timeout duration              timeout after which the drain is aborted (default: 0, infinite time)

However, it looks like the helm chart has not been updated to support this option.  I'm familiar with Helm and it looks straightforward to add, would you be open to a pull request for this?

There is a open PR to add those flags. #360
We will try to merge this as soon as possible. I will close this in the meantime.


[FLINK-24501][table-runtime] Stores window progress into state in order to check whether an input element is late or not for window-tvf aggregate operator.

What is the purpose of the change
Problem description
After recover from savepoint or checkpoint, unexpected behavior of cumulate window aggregate for late event may happened.
Bug analyze
Currently, for cumulate window aggregate, late events belongs to the cleaned slice would be merged into the merged window state, and would be counted into the later slice.
For example, for a CUMULATE window, step is 1 minute, size is 1 day.
SELECT window_start, window_end, COUNT(USER_ID)
FROM TABLE(
CUMULATE(TABLE Bid, DESCRIPTOR(bidtime), INTERVAL '1' MINUTES, INTERVAL '1' DAY))
GROUP BY window_start, window_end;
When the watermark already comes to 11:01, result of window [00:00, 11:01) would be emitted. Let's assume the result is INSERT (00:00, 11:01, 4)
Then if a late record which event time is 11:00 comes, it would be merged into merged state, and would be counted into the later slice, for example, for window [00:00, 11:02), [00:00, 11:03)... But the emitted window result INSERT (00:00, 11:01, 4) would not be retracted and updated.
The behavior would be different if the job recover from savepoint/checkpoint.
Let's do a savepoint after watermark comes to 11:01 and emit (00:00, 11:01, 4).
Then recover the job from savepoint. Watermarks are not checkpointed and they need to be repopulated again. So after recovered, the watermark may rollback to 11:00, then if a record which event time is 11:00 comes, it would not be processed as late event, after watermark comes to 11:01 again, a window result INSERT (00:00, 11:01, 5)  would be emitted to downstream.
So the downstream operator would receive two INSERT record for WINDOW (00:00, 11:01) which may leads to wrong result.
This pull request aims to solve the problem by storing window progress into state in order to check whether an input event is late or not.
Brief change log

Add  OperatorState in SlicingWindowOperator to store watermark

Verifying this change

Update UT in SlicingWindowAggOperatorTest#testEventTimeCumulativeWindows to check the late event would still be processed as late event even if the job is restored from savepoint.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

LGTM for now， cc  @JingsongLi would you help do the final review since this is a change on table-runtime


Isolate scope does not 'isolate' for multiple directives

See plunker: http://plnkr.co/edit/YhR39jDt9Zfefx0Dovdm?p=preview
Setting the scope of "myCustomer" to true, effectively separates the scope. Isolating the scope of "myCustomer" with {}, does not separate the scope.

The template-one and template-two directives do not request a scope, thus they share the normal (non-isolate) scope of the elements (possibly inherited by an ancestor).
They can't see an isolate scope requested by another directive (even on the same element), because, you can't have more than one directive share an isolate scope.
(There are other ways to make your directives communicate/augment each other, usually involving one directive requiring the other directive's controller.)
--
That said, this is a general support question and would be better suited for one of the support channels.
GitHub issues are reserved for bug reports and feature requests.


Remove .setTitle from SHORT_TEXT and RANGED_VALUE complication types

Do this to all other Complication Services or I can do it. Please let me know :)

Done - https://github.com/SimpleAppProjects/SimpleWeather-Android/commit/a1fd62b47b6987ff5aa854464ef482ef37590156


Updating NS page

Adding a how-to for setup and explicit variable lines for OpenAPS.
I'm not sure whether the formatting for the .. note:: will work (saw it in other rtd sites and looked helpful?), nor am i totally sure that the picture formatting will work (but I sure hope so, i prefer the slightly smaller pics and having them centered.

Changed to dev branch to test out the img tags and notes


Update in DSC name, DSC ---> GDSC

According to new branding, Developer Student Club is now Google Developer Student Club, so the logo should be changed!!!

Thank you for the PR, I'll merge this along with the new updates to the guidelines


Wrong popups/tooltips offset in the editor (DisplayServer regression)

Bugsquad note: This issue has been confirmed several times already. No need to confirm it further.

Godot version:

OS/device including version:
Windows 10 x64
Using Vulkan Device #0: AMD - AMD Radeon R7 200 Series
Issue description:
When you click any tab (Project,Scene,Debug) or even right click to add node the popup appears far away form mouse:

Steps to reproduce:
Just interact with engine itself
Workaround
minimize window and maximize again!

happens only with custom zoom ftw

Related to #45200
This issue goes away when you resize the window.

I also experience this with default settings on Windows 10 with current master (b1eee24e2673b4a980a6f661511ff0f18c2fff8a).
I can reproduce this every time like in previous comments: open Godot, while a project loads switch focus to another window, wait, voilá, offsets are all weird. :)

I noticed that it's offset by the same offset that the project manager opens up offset by. Dragging the project manager around before opening a project doesn't affect this offset though.
1440p monitor

Popup Offset
Project Manager Offset

1080p monitor

Popup Offset
Project Manager Offset

The 20 px difference is probably due to the Windows Title bar being 20px high

Windows 10.  d6f972fad45d28222b1437a047241dd74f0d5b90
I'm trying to figure this out.
After a few steps I got to (and similar lines if I'm trying to click on the FileSystem dock):
https://github.com/godotengine/godot/blob/d6f972fad45d28222b1437a047241dd74f0d5b90/editor/filesystem_dock.cpp#L2512
The call to tree->get_screen_position() there is returning the odd values.
Following deeper
https://github.com/godotengine/godot/blob/d6f972fad45d28222b1437a047241dd74f0d5b90/scene/gui/control.cpp#L1717
w->get_position() seems to be the call that leads to the bad offset values.
When the editor window is resized, everything starts going to the right positions. My guess would be that https://github.com/godotengine/godot/blob/d6f972fad45d28222b1437a047241dd74f0d5b90/scene/main/window.cpp#L284 gets called and finally sets the right position for the window.
Could it be that a call to that is being "skipped" with the repro steps described in the comments above?

Still reproducible somewhat on v4.0.alpha.custom_build [afdae67cc], only in single window mode
On my left monitor (1080p):

On my middle monitor (1440p):

Menus are always displayed correctly for me though:

Project manager is also offset, but not by the same offset as any of the tooltips


Update README links

After the recent changes to gudmap/atlas-d2k config changes most of the plot links in the README.md are broken. We should update these and if we don't have any example link for them we should just remove the link (and keep the screenshots).
We should also avoid using links to our user directories as we keep changing them. Some of the links are throwing terminal errors which I'm assuming it's either because of an outdated config or code.

The readme has been updated to show working links and to remove links that were pointing to user directories when no example was in production.
We should consider copying data from facebase and atlas-d2k to dev.derivacloud.org. Once we've done this, we can define a plot-config.js using that data for our examples.


feat: remove deprecated code issues #133

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #133
Summary:

Resolved Issues with Deprecated Features

Removed the fixed attribute.
Eliminated legacy size options (xxxs to xxxl).
Simplified margin handling to numeric values (none, 25, 50, etc.).

Codebase Cleanup

Removed legacy utility classes and pixel-based rendering styles.
Deleted outdated examples from documentation and API.

.husky/pre-commit Fixes

Removed the following lines that would fail in version v10.0.0:#!/usr/bin/env sh
. "$(dirname -- "$0")/_/husky.sh"

Documentation and Dependencies Updates

Revised documentation files and examples for the updated implementation.
Updated package.json dependencies to the latest @aurodesignsystem/auro-library version.

Benefits of These Changes

Streamlined codebase for improved maintainability.
Removed outdated functionality to align with modern practices.
Ensured compatibility with future versions of the tools and frameworks.

On branch dsande/deprecatedFeats/#133
Changes to be committed:
modified:   .husky/pre-commit
deleted:    apiExamples/fixed.html
modified:   apiExamples/margins.html
modified:   apiExamples/margins2.html
modified:   demo/api.md
modified:   demo/index.min.js
modified:   docs/api.md
modified:   docs/partials/api.md
modified:   package-lock.json
modified:   package.json
modified:   src/auro-header.js
modified:   src/style.scss
modified:   test/auro-header.test.js
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Documentation:

Update documentation to reflect the removal of deprecated features and the new margin handling.

NOTE: in order to update API docs, I temp added back in the code needed to make that work. See following issue:

https://github.com/AlaskaAirlines/auro-header/issues/135


Should be able to call Get-PSSession -State Opened

When I call Get-PSSessions -State Opened I get this error that says the parameter set cannot be resolved. It should be valid to call Get-PSSession with just the -State parameter to get the sessions that match that state.
Steps to reproduce
Get-PSSession -State Opened

Expected behavior
Id Name            ComputerName    ComputerType    State         ConfigurationName     Availability
 -- ----            ------------    ------------    -----         -----------------     ------------
  1 WinRM1          Server1   RemoteMachine   Opened        Microsoft.PowerShell     Available
  3 WinRM2          Server2   RemoteMachine   Opened        Microsoft.PowerShell     Available
  2 WinRM3          Server3   RemoteMachine   Opened        Microsoft.PowerShell     Available
  4 WinRM4          Server4   RemoteMachine   Opened        Microsoft.PowerShell     Available

Actual behavior
Get-PSSession : Parameter set cannot be resolved using the specified named parameters.
At line:1 char:1
+ Get-PSSession -State Opened
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [Get-PSSession], ParameterBindingException
    + FullyQualifiedErrorId : AmbiguousParameterSet,Microsoft.PowerShell.Commands.GetPSSessionCommand

Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.2
PSEdition                      Core
GitCommitId                    v6.0.2
OS                             Microsoft Windows 10.0.16299
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The discussion point here is that the current implementation requires that you will provide another parameter. This is the current workaround:
Get-PSSession | Where State -eq 'Opened'

Get-PSSession -State Opened -ComputerName localhost

This doesn't work when connecting to a remote host, obviously. For scripts that run in loops for multiple hours (e.g. moving several hundred accounts to Skype online) it's useful to clear out the broken sessions on an interval.
The 'Get-pssession | where-object {$_.State -eq 'Opened'}' workaround is reliable, but not very efficient.


Does not comply with MIT licensing

This project is sourced from https://github.com/pmndrs/react-three-next, which is under the MIT license. You may dual-license, but the original license must be included unmodified in vendored works.

This is based on create-r3f-app as clearly stated here.
The original author has been credited for the foundational work in the README and package.json. Their MIT license did not include references to [react-three-next](https://github.com/pmndrs/react-three-next).
Once this project deviated from the original repo, I assigned "ownership" to me, since most code is or will be mine. To dual-license would make it unclear in this case, since it's not a 3rd party library or portion of the app. But I still want to respect the author and their work, which is why I've credited them.

No, this project is indeed based on https://github.com/pmndrs/react-three-next as you can see since b8e598e0bf8bcb6373784f21f97a5564859caf83. The original files, folder structure, and even assets are visible.
I appreciate the effort to credit the original author of these works, but it does not do so in a way that respects its respective licensing.