Hive database not auto created when syncing

Describe the problem you faced
There is a new flag in hudi 0.7.0 or later - hoodie.datasource.hive_sync.auto_create_database. Based on documentation it defaults to true which would be consistent with previous behaviour that created hive dbs if they don't exist.
It seems that 0.7.0 and 0.8.0 will actually default to false when writing a dataframe as hudi if the flag is not specified at all because of how this code is written here https://github.com/apache/hudi/blob/release-0.7.0/hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/hudi/HoodieSparkSqlWriter.scala#L377  so we end up with the following error instead org.apache.hudi.hive.HoodieHiveSyncException: Failed to check if database exists your_db_name
To Reproduce
Steps to reproduce the behavior:

Write a dataframe to hudi without specyfing the new hoodie.datasource.hive_sync.auto_create_database flag but set other settings to sync to hive
it will not sync to hive and will give HoodieHiveSyncException exception instead

Expected behavior
On 0.6.0 this creates a hive db and in 0.7.0 it doesn't anymore. When the flag is explicitly provided and set to "true" it works fine so maybe docs could be updated to reflect it? This would still be a surprising behaviour as it wasn't required before.
Environment Description

Hudi version : 0.7.0 and 0.8.0

Additional context
Delta-streamer works fine with 0.7.0

@konradwudkowski  This was fixed recently as part of - https://issues.apache.org/jira/browse/HUDI-1942 , do you want to try out the latest source from master ?

hi @veenaypatil the version in master is indeed fixed - thanks. Any thoughts of updating the docs or releasing 0.7.1 and 0.8.1? I set the flag to "true" already so I'm not affected anymore but it may surprise someone else until 0.9.0

@konradwudkowski Thanks for bringing this up. We will be releasing 0.9.0 in few weeks. In the meantime, I think it makes sense to add this as a note in the configs. @veenaypatil Do you think you can take this ? JIRA here -> https://issues.apache.org/jira/browse/HUDI-1997

@n3nash  yes, I will pick it up and a note in documentation

@n3nash @konradwudkowski  https://github.com/apache/hudi/pull/3066 pls review is this note looks good

@veenaypatil looks good to me, thanks!

Closing this ticket since the issue is resolved. Thanks @veenaypatil !


Add Hyper-V firewall powershell cmdlet documentation

Add Hyper-V Firewall cmdlet documentation.

@officedocspr  - Is there any pending action items on my side? How does this PR get reviewed by a maintainer?

@tseward @scanum - Are you the right people to help this PR make progress?

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

close > reopen

close > reopen


EditOverlay for features is created multiple times

Context
Every time you switch between map view and table view, a new EditOverlay view is created
Steps to Reproduce
Please break down here below all the needed steps to reproduce the issue

Switch to table view
Switch to map view
Back to one a couple of times

Current Result

Expected result
That view is only created when new features added.

I've found a couple of issues related with this:

[ ] The EditOverlay view doesn't allow to click on the first widget's control.
[ ] Being in the feature edition view, if you change to another view instead of clicking on the back link (widget edition for example), the Builder breaks: https://my.trackjs.com/details/314b6da84e3844cc925bfd6a9cc5c88f

I'm going to fix them as part of this issue.

Works like a charm.

Shipped! :rocket:


espressif/esp32/esp32s2/esp32c3: Disable wdt on initialization.

Summary
This MR aims to:

Fix esp32s2 wdt disabling and wrap it under a function.
Wrap wdt disabling in esp32.
Implement wdt disabling in esp32c3.

Impact
From now on, it's not necessary to disable WDT in bootloader. Start initialization will take care of disabling wdt that was enabled by bootloader.
Test
I used a bootloader with RWDT enabled for the 3 SoCs and I commented the function that disables it and could see the chips rebooting then I uncommented and the nsh was kept.

CI for macOS(sim) fail to download the artifact, since this patch doesn't relate to sim or macOS, let's ignore the CI issue and merge it directly.


Track number of parts in multipart request

Description
Track number of parts in multipart request. RequestParts will be used to determine the # of ApiRequests to bill
We are billing for each part sent to us in STOW (aka - each instance) & each part returned in the response for WADO requests. For most requests this is the # of instances returned, except for /frames requests, where we bill for the number of frames returned.
Related issues
Addresses #89153.
Testing
Generate local nuget packages, deploy to test dicom service and try sending multipart and normal requests

How about multi-part response

Updated to bill for multipart WADO responses


change(state): Put chain data into an inner struct

Motivation
The non-finalized Chain contains many fields that we need to manually compare in eq_internal_state when comparing two chains. We can't use regular PartialEq because we already use that for comparing only the partial cumulative work between chains: https://github.com/zcashfoundation/zebra/blob/7348d080d6dd0bb2e7153569ce4158d86861524f/zebra-state/src/service/non_finalized_state/chain.rs#L2189.
We recently added new fields into Chain, and we forgot to compare them in eq_internal_state, which was a bug that is likely to happen again.
Solution

Put the fields used in eq_internal_state into an inner struct, and derive and use PartialEq on this struct in eq_internal_state.

Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

This is a draft because wrapping the fields of Chain into an inner struct causes ~ 290 errors. I would like approval of the current code before I fix all the spots where we access the fields. The fixes will be trivial, but tedious, so I would like to avoid them if they wouldn't be right anyway.

I labeled this PR as a draft because wrapping the fields of Chain into an inner struct causes ~ 290 errors. I would like approval of the current code before I fix all the spots where we access the fields. The fixes will be trivial but tedious, so I would like to avoid them if they wouldn't be right anyway.

The code looks fine to me.
If you impl Deref for Chain { Target: ChainInner ... } then most of those errors will go away, and we can see if the tests pass. You might also need to impl DerefMut for mutable accesses.
https://doc.rust-lang.org/std/ops/trait.Deref.html
Then we can do the mass replace in a later PR, because it's easier to review that way. (And the mass replace is likely to conflict with #7463 in this sprint.)

Failed due to issue #7659, I'll restart when it's finished.


DrefCreate Dref

References

https://github.com/npm/documentation/pull/583#issue-1656435669

.git-blame-ignore-revs
Removed semi-colons from the entire codebase
a8940f7fbddf7fad9d7d50014d4e8d46baf30592
Converted all JavaScript to TypeScript
69d029cec8337c616552756310748c4a507bd75

Space
Overview
Try for free
All-in-One Solution
for Software Teams and Projects
Space is a fully integrated toolchain that connects the software development pipeline, communication, and project management.
Everything your company needs in one tool
Covers the processes of the whole team
Scales with your organization
Give Your Software Team a Digital Home for Free
Try for free
Develop Software and
Collaborate in One Place
No matter the size of your business or how long you have been around, there are always things you need to manage. To help you focus on what's important, we’ve created Space, an integrated team environment.
Space unifies tooling and information across your company, bringing more effective collaboration to your team.
Manage all the aspects of your tech business from source control hosting to running meetings.
Communicate in chats, collaborate on projects, share knowledge, and build and deliver software together.
Preview image of space overview video
Software Development
Covering software development pipelines at every scale, Space includes everything you need to speed up production, giving developers the freedom to grow, discover, collaborate, and create:
Source code hosting (Git)
Code review, including merge requests and quality gates
Automation jobs for building and deploying applications
Package and container registries for publishing artifacts
Plugins to connect Space to your favorite IntelliJ-based IDEs
Software Development
Business Management
Manage and grow your business faster with Space. Collaborate, build software, and keep everything easily accessible in one platform.
Forget about having to rely on multiple tools and subscriptions. Space offers a broad toolchain for coordinating all aspects of your business, including:
Software development
Project management
Communications
Team management
Business Management
Project management
Manage projects faster and transparently with Space. Plan, schedule, collaborate, and meet deadlines easily.
Space helps you to optimize the whole project management workflow and collaboration process:
Checklists for planning
An issue tracker for task management
Issue boards for visualizing progress
To-do lists for personal task tracking
Project management
Communications
Streamline communication and keep everyone on the same page by sharing information with your team.
Space integrates various tools for easier communication:
Chats and channels
Collaborative documents and blogs
Meeting management
Shared team calendars
Communications
Team Management
Manage your team, absences, and your company’s organizational structure.
Space offers everything your company needs to ensure smooth and transparent collaboration:
Team directory with member profiles
Vacation and absence management
Managing personal and team calendars
Team Management
The Complete Solution for Software Teams and Projects
Perfect for Tech Companies
Space covers the entire software development life cycle. It has everything from source code hosting to package management.
Integration with JetBrains IDEs
Review and comment on code using the power of IDE navigation. Manage your merge requests and code reviews without leaving your IDE.
Scales with Your Business
You can easily scale Space as your company grows and more teams join the platform. Space can be customized and extended to meet your exact needs.
Cost Saving Solution
Instead of paying for multiple solutions, get one platform with the complete software development, project management, and communication toolchain your business needs.
Designed to Streamline Collaboration
Space removes the silos between different teams and specialists by connecting development, project management, and collaboration features.
Ideal for Remote Work
Space will help you run all your business processes online, keeping everyone informed and up to date without overloading them with irrelevant information.
Get started with the free cloud plan for an unlimited number of users, and then upgrade when you're ready.
Watch the Space Overview Video
Get started with Space today
Give Your Software Team a Digital Home for Free
Try for free
Discover How Other Software Teams Benefit From Using Space
Makery
Rather than using a multitude of individual tools, we’ve always preferred integrated environments. We started looking for a new digital home, and when we heard about Space at KotlinConf, we really liked the concept. So we decided to sign up. Read story
Andras Kindler
Andras Kindler,
Co-Founder,
Makery
AmberCore Software Ltd
Communication has become faster since we started using Space. I think the deciding factors in choosing to go with Space were its ease of use and versatility. Read story
Anna Vinogradova
Anna Vinogradova,
Chief Marketing Officer,
AmberCore Software Ltd
Get a Complete Solution for Software Development by Connecting JetBrains IDEs to Space
50% Discount on JetBrains IDEs and .NET Tools for Startup Companies
Is your company less than 5 years old? If so, you can benefit from our special offer for startups and get 50% off all your JetBrains tools.
With this discount, you can purchase up to 10 product licenses over a period of 60 months. Connect your JetBrains IDEs to Space to get access to your repositories, code reviews, and Automation (CI builds).
Startup Discount Requirements
Your company is privately held.
You can confirm that it was established less than 5 years ago.
Your company has a website or existing public references on the internet.
Get 50% off on all JetBrains Tools
Apply for Discount
Products
IDEs
.NET & Visual Studio
Team Tools
Plugins
Education
Languages
All products
Solutions
C++ Tools
Data Tools
DevOps
Education
Game Development
Software Development
Tools For Business
Quality Assurance
Initiatives
Kotlin
JetBrains Mono
JetBrains Research
Open Source Projects
Community
Academic Licensing
Open Source Support
User Groups
Events Partnership
Developer Recognition
Resources
Sales Support
Product Support
Licensing FAQ
Documentation
Early Access
Events and Webinars
Newsletters
Industry Reports
Blog
Desktop Art
Company
About
Contacts
Careers
News
Customers & Awards
Our Commitment
Brand Assets
Partners and Resellers
Saudi Arabia
English
Privacy & Security
Terms of Use
Trademarks
Legal
Genuine Tools
Copyright © 2000-2023 JetBrains s.r.o.
Developed with drive and IntelliJ IDEA

spam


Adds moo lexer to parser to dramatically improve speed and memory usage

Tokenizes integers, signal names, and whitespace into single chunks now resulting in much smaller parsing tables.
See: https://nearley.js.org/docs/tokenizers
This was motivated by noticing that as I was adding states to a reasonably sized program, it got slower and slower until the browser refused to allocate it any more memory and would crash the tab. The machine in question was only up to 14 states or so and about 70 lines long with several comments.

This is excellent. Thank you so much for doing this.


add elapsed time format

This adds a new elapsed format, which displays the compact representation of how long has elapsed between the provided datetime and the current date.
This is designed to replace an extended element we have in github/github called precise-time-ago, which aims to be like <time-ago format=micro> but will display all units, not just the largest unit.

:wave: Hello and thanks for pinging us! This issue or PR has been added to our inbox and a Design Infrastructure first responder will review it soon.

:art: If this is a PR that includes a visual change, please make sure to add screenshots in the description or deploy this code to a lab machine with instructions for how to test.
:fast_forward: If this is a PR that includes changes to an interaction, please include a video recording in the description.
:warning: If this is urgent, please visit us in #primer on Slack and tag the first responders listed in the channel topic.


Math flow parsed inline in the syntax tree

Here is a small reproducible example:
use markdown::{to_mdast, ParseOptions, Constructs};

fn main() {
    let options = ParseOptions {
              constructs: Constructs {
                math_text: true,
                ..Constructs::default()
              },
              ..ParseOptions::default()
            };
    let ast = to_mdast("$$math$$", &options).unwrap();
    println!("{:?}", ast);
}

I get
Root { children: [Paragraph { children: [InlineMath { value: "math", position: Some(1:1-1:9 (0-8)) }], position: Some(1:1-1:9 (0-8)) }], position: Some(1:1-1:9 (0-8)) }

This was not what I was expecting
After a few tests, It seems that math is parsed "Inline" when there is no newline between the $$ and the content, but I think it is almost never the expected behavior: a math equation between $$ on a single line should not be rendered "inline"
Does it add complexity to the implementation or was it a deliberate decision ?

This is intended, code works the same way. ``code``

ok
I just tried it on stackedit, and $$x$$ is not displayed inline.
Do you consider changing (or a least adding an option to change) that behaviour ?

No. There is no spec for math in markdown, so everyone is going to do their own thing.
It's best IMO, if there is no spec, to stick with how the spec works in similar cases

ok I somewhat agree
So I guess I will preprocess my markdown to add line breaks whenever there is $$

Ho no, I just realized that I needed the Positons on the syntax tree relative to the original source, so this solution does not work at all for me :disappointed:

@rambip similar to https://github.com/wooorm/markdown-rs/issues/62, I don't see this as something that would be a part of core markdown-rs.
But, a plugin could look at the positions of content, and convert inline math to math blocks based on your desired (though non-standard) rules. (depends on https://github.com/wooorm/markdown-rs/issues/32)
This could be done directly from rust with https://github.com/wooorm/markdown-rs/issues/32
Or in a more round about way working on the mdast tree (already supported in the API https://github.com/wooorm/markdown-rs#api)
Efforts towards supporting a plugin API, or enhancing the APIs around mdast handling like https://github.com/wooorm/markdown-rs/issues/64 are welcome!


Add more docs on stereo sound panning

In #4256, docs on how panning only works for mono sounds was added to the Sound#setPan() method.
However for anyone trying to use Sound#play() with the panning argument, they won't see this note in the Javadocs.
I'd like to suggest adding the same note to these method docs!

Is it true for all platforms?

Not sure; I don't have access to a Mac or Linux machine at the moment, only Windows.
But I suspect it's a limitation of OpenAL:

macOS and Linux behaviour is usually the same as Windows. MrStahlfelge will be getting at GWT, Android and iOS, the latter of which I'm not set up to test either.

For future readers: @Hangman noted on Discord that panning for stereo sounds can be enabled by an OpenAL extension on desktop:
AL10.alSourcei(sourceId, SOFTSourceSpatialize.AL_SOURCE_SPATIALIZE_SOFT, AL10.AL_TRUE);


Split Tendermint client into Tendermint33Client and Tendermint34Client

Creating a Tendermint client that makes the full Tendermint RPC API available to the user should be easy. Users need to do that to e.g. create custom quey clients.
Auto-detecting the Tendermint version of the backend does not work anymore starting with Tendermint 0.34. We should give it up. Also Client exposes and interface that tries to generalize between Tendermint 0.33 and Tendermint 0.34. If we split it into two clients, the interfaces can be more specific.
Right now, using the Tendermint client looks like
import { adaptor34, Client as TendermintClient } from "@cosmjs/tendermint-rpc";

// ...

const tmClient = await TendermintClient.connect(config.rpcUrl, adaptor34);

which has a few flaws:

Client is never a great name to import and always needs to be renamed. So it would be nice if we give it a useful name by default, even if slightly redundant with the package name
adaptor34 was designed to be private but was exposed later on. Not pretty to work with.
The async in the constructor is only needed for auto-detection. It can probably be sync now.

Instead, this could look like
import { Tendermint34Client } from "@cosmjs/tendermint-rpc";

// ...

const tmClient = new Tendermint34Client(config.rpcUrl);

Great idea. We can have different response values for them as well, rather than auto-converting.
0.33 can keep the present return values.
for 0.34, I would suggest importing the generated tendermint types: https://github.com/cosmos/cosmjs/tree/main/packages/stargate/src/codec/tendermint/types
They use protobuf internally and marshal to json for the rpc. There will need to be a bit of checks/conversion I think, especially things like oneof are different between go and js I think, but most of it is there. And returning these protobuf types makes us more future-compatible with 0.35 which plans a grpc interface.


AppLocker behaviour on Win 10 Pro managed via Intune

Hi,
I have a requirement to configure AppLocker EXE, script and MSI policies on a fleet of Windows 10 Pro machines. In the past, when using Group Policy, centralised management was only possible when using Win 10 Enterprise edition. However, this article states that AppLocker running on Win 10 Pro can be managed centrally if you are using an MDM solution to manage it (e.g. Intune). Based on this and other articles (including the documentation for the AppLocker CSP) I determined that my scenario where all machines are fully Azure AD joined (not hybrid joined) and enrolled in Intune should be fine.
However, when I have attempt to configure it, it seems to only partially work. This is the behaviour I am seeing:

EXE checks work absolutely fine in either audit or enforced mode. The relevant events can also be found in the AppLocker event log on the endpoint
Script and MSI checks do not work at all in audit mode and only partially in enforced mode. In audit mode, the relevant AppLocker event log remains empty. In enforced mode, the event log still remains empty, but some types of scripts are blocked. I cannot get MSI blocks to work at all and no events are present in the event log for this.

The endpoint I am testing on is fully up to date with the latest feature and security updates. All users are licensed for Intune (M365 BP). As far as I am aware, I have correctly configured the Intune side using the documented process to produce and export the AppLocker XML from an existing machine and then import it into a custom OMA-URI policy within Intune. The fact that my EXE policies are working without issue suggests I have done this correctly. Microsoft support were also unable to find any issues with the configuration.
I have done some investigation online to see if anyone else has experienced similar issues and have come across a handful of interesting posts that describe similar behaviour to mine (they seem to have more luck with MSI, but still very similar):
https://call4cloud.nl/2020/08/the-men-who-stare-at-the-applocker-event-log/
https://www.reddit.com/r/Intune/comments/gnzv2z/applocker_configuration_not_working_for_msi/
These posts are suggesting that the AppLocker CSP in Win 10 is actually using a mixture of AppLocker and Software Restriction Policies (SRP) to apply the configuration you have defined in Intune. This would makes sense based on the behaviour we are all seeing. However, I cannot find anything within the official documentation that mentions this. In fact, this article seems to suggest that all types of policies should be fully functional on Win 10 Pro if using Intune to manage, whereas the reality seems to be that there is potentially only limited support with a number of caveats.
I have a support case open to try and get to the bottom of this, but it is looking like it will take some time before it is escalated to the necessary level to get a definitive answer. As this is also a potential documentation issue, I have decided to raise it here with the hope you may have more luck getting clarification via your channels and the documentation can then be updated so that others aren't left wondering what's going on!
Thanks in advance!
EDIT: Just a small clarification - technically, the endpoint I am testing on is running Windows 10 Business rather than Pro. It was originally a Win 10 Pro image, but it is converted to Business edition upon enrolling it into an Intune tenant running M365 Business Premium (which this one is). I doubt this is significant as Pro and Business are very similar. Also, the AppLocker CSP is apparently supported on Business edition as well.

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

ID: e2218e80-12fa-9138-d074-3ebdc14113b0
Version Independent ID: 7fc2dbfb-2f46-5657-9e28-4a5ae7bbd301
Content: Requirements to use AppLocker (Windows)
Content Source: windows/security/threat-protection/windows-defender-application-control/applocker/requirements-to-use-applocker.md
Product: windows-client
Technology: itpro-security
GitHub Login: @vinaypamnani-msft
Microsoft Alias: vinpa

Hi @e0i,
Thanks for reviewing the issue. However, I'm not sure why it has been closed without any action? I can see you added the product-feedback tag before closing it. Am I right in thinking this means the issue has been closed because it's been deemed to be product feedback only and therefore not a documentation issue?
Although my issue does include what you could describe as "product feedback", it was raised as it is also a significant documentation issue. The "product feedback" is there as context for the documentation issue. All of my findings are suggesting that the actual behaviour of the product in this scenario is significantly different from what is documented. It also looks as though some of it is "by design", meaning the documentation should correctly describe the behaviour, which doesn't seem to be the case. This is especially important as some of the differences between the product behaviour and the documentation involve what appear to be limitations/caveats of the current implementation, which are especially important to document well.
The incorrect documentation will no doubt lead others to expect certain behaviour like I did and find themselves scratching in the dark when it doesn't behave as expected. As well as trying to reverse engineer what the correct behaviour is. Having it properly documented is important.
I'd appreciate it if the issue could be reviewed again. I'm happy to answer any questions or provide more detail regarding the above points - just let me know and I'll do my best.
Thanks again.

I can also confirm, I experience the exact same behaviour and have wasted a lot of time looking into this.
I did raise a Microsoft support case on this who actually referenced this post, telling me that they have raised this to the documentation team to make modifications.
I hope this gets resolved soon.

I can also confirm that I am affected by this issue. Subscribing and hoping this will be resolved soon.

Thanks all for feedback, apologies for premature closing. Reopening, assigning to Vinay to investigate. No promises on timelines, just noting that it's reactivated in our backlog.

@aczechowski thanks for re-opening this and giving it some attention - really appreciate it. It's actually reminded me that I need to update everyone with my findings since the original post. Lots of good news:

I had a support ticket open with MS regarding the fact that MSI/Script event logs do not appear to work on Pro edition with AppLocker is managed via CSP. It was escalated all the way to the product group, who requested a business impact statement before they'd investigate. Before doing that, I thought I'd re-rest the issue on 22H2 and it appears to be fixed! I rolled my lab back to the previous feature update and the issue re-occurred, so does appear to be something in 22H2 or possibly a monthly CU that's fixed it as I also installed those at the same time. Support were unable to confirm any specific fix in 22H2, but does appear to be there. It would be good if others are able to confirm the same. I haven't tested Win 11. As a side note, event logs have always worked on Enterprise edition for me.

My issues regarding some PowerShell scripts not getting blocked turned out to be PowerShell Constrained Language Mode. I won't go into the details here, but there's lots online about it. The short answer is that it's by-design behaviour. One specific case regarding right clicking PS1 scripts in Explorer and selecting "Run in PowerShell" turned out to be a known bug in Windows PowerShell. It's been fixed in PowerShell 7.x, but the fix may or may not be backported to Windows PowerShell 5.x. As 5.x is part of Windows, MS have to be much more careful with that than 7.x. I think there were some compatibility concerns. Happy to provide detail on this if it would be helpful, but it was an edge case that only applies in specific circumstances.

Regarding MSI block only partially working - I think this may have been user error on my part. We were eventually able to create MSI block rules that worked, although from memory, I think some MSIs will initially execute and then get blocked further along the wizard.

I hope the above helps others that have been struggling with any of these issues (particularly the issue re missing event logs). I think it would still be helpful to update the documentation with some "info" cards regarding:

Issues with event logs pre-22H2.
PowerShell Constrained Language Mode's interaction with AppLocker meaning that some scripts will execute, but in a restricted context.

I'll keep this issue open for @vinaypamnani-msft to review re AppLocker and other comments above, but regarding the point about Business SKU and the AppLocker CSP, see https://github.com/MicrosoftDocs/windows-itpro-docs/issues/9560#issuecomment-1382191251

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


R vignettes

Added vignette draft.

I know what this problem is:
Quitting from lines 94-101 (AIRR-standards.Rmd)  Error: processing vignette 'AIRR-standards.Rmd' failed with diagnostics: '' does not exist in current working directory ('/tmp/RtmpHiBSvd/Rbuild288d539cb64/airr/vignettes'). Execution halted
working on fix.

Seriously??? AAAAhhh!!

The problem is that the tests directory doesn't exist inside the built package. So, you'll probably have to move /tests/data-tests to /inst/data-tests and then reference it in the tests via system.file("data-tests/file.tsv", package="airr")

Ah, or maybe we just need to build with --install-tests to get system.file("/tests/data-tests") to work. That might be better, if it works.

I suspect system.file("data-tests/file.tsv", package="airr") works if the packages is installed.

Yeah, I got all mixed up. Sorry. Forgot we weren't running R CMD install during travis... One sec. Fixing.

No worries. I can't wait for the day I experience the joy of a problem free pull request. I am still getting used to this.


Llama index test

Pushed the code for the Llama-Index vector doc store and a notebook containing the experimentation with using a knowledge graph store


show update download progress in an item non valid

currently Alfred Kat sends a notification and starts the download, but Alfred itself disappear. it would be nice to keep Alfred open and show the progress of the download as a result item.

would need ScriptFilter rerun + launching a process in the background and somehow connect the background process to the rerun ScriptFilter (write progress in a cache file maybe?)

This would be a nice improvement. Hope to see this implemented any time soon.

i was gonna close this 😂
with the latest version the download happens in the background, you can keep using Alfred or do some other stuff. download is super fast anyways. i'm busy with other things. definitely not a priority.

but i'm curious. how as a user do you see this being better than having the thing downloaded in the background quickly and you going do your stuff without thinking about it?
although the update opening once downloaded is a bit annoying hmm. could break your flow.

that requires a major work. not just for Alfred KAT. but more like a whole system to handle updates. i've spent a couple of days already improving the Updater. now much better. gonna be it for this year.

yeah, true. but how often do you thing this is used?

does your workflow launch the update once downloaded too? or do you have a smarter system?

Oh gosh! I'm sorry! I think I misunderstood what this issue was about. Now I see it's about the  update download progress".  For a moment I thought it was about files we download using the workflow.

OOOOOOOOOHHHHHHH LMAO. now your comments make a lot of sense 😂😂😂
i never thought about showing the progress of the torrents in Alfred. hence the confusion. the title of the issue and the rest of my words are definitely confusing.

not even sure how it would be possible to do that with the torrents. i mean i'm not the one "downloading" them. i don't have any clean access to that info. reading the .part files? that'd be crazy. i mean maybe there's a way, but definitely not something i'm interested in implementing. although it would look super cool and probably would get me some dates with the ladies.

not even sure how it would be possible to do that with the torrents.

Maybe reading stdout from transmission-cli?

Maybe reading stdout from transmission-cli?

that was my first thought but i had to pull myself out of my own world and remind myself that Alfred KAT is client agnostic 😅
i mean sure, for PRO users you can set your own client and i could grab that. but each client would have its own format so yeah so no.

Yeah, maybe not worth it.

i could show an Alfred Result telling users to go see in their torrent client aww 😍


Issue with ngAnimate when using Selenium-TestNG for testing our application

Hello all,
I am currently facing to a random failure behavior on my Selenium which only occurs when my application has the ngAnimate module as dependency: angular.module('myApp', [ 'ngAnimate' ]);.
The angular-animate library seems to be too slow when Selenium is testing our application. We don't reproduce this behavior when we use our application manually.
When the input text failed with the ngMaxlength constraint, we can observe the ng-valid-maxlength class is removed and the ng-invalid-maxlength class is added but the ng-invalid class is not pushed in the dom and the ng-valid class is kept in the DOM instead of to be removed.
You will find below some parts of sources code, but you also can find the all projects (simple angular app and selenium/TestNG test) here:

Test with Selenium & TestNG: https://github.com/zann12/angular-animate-with-selenium.git
The application: https://github.com/zann12/angular-animate-with-simple-app.git

Libs:

angular: 1.3.13
angular-animate: 1.3.13

index.html
<!DOCTYPE html>

    <link rel="stylesheet" href="app.css">

    A simple Input with two constraints directives (ng-minlength and ng-maxlength)

    <input
            id="input#id1"
            ng-model="model.textvalue"
            class="text-value"
            ng-minlength="4"
            ng-maxlength="model.maxlength">

    <input
            id="input#id2"
            ng-model="model.maxlength">

Selenium Test
@Test(priority = 1, invocationCount = 100)
public void validatorsOnQuickTest() {
this.singleDriver.get("http://localhost:10000/Validator_bug/app/index.html#/view1");
	WebElement t = this.singleDriver.findElement(By.id("input#id1"));
	WebElement tMax = this.singleDriver.findElement(By.id("input#id2"));
	QuickTest.clearInputValue(tMax);
	tMax.sendKeys("10");
	QuickTest.clearInputValue(t);
	t.sendKeys("aaaaaaaaaaa");
	Assert.assertTrue(this.hasError(t));  // "aaaaaaaaaaa".length > maxLength (10) => An error is expected
}

private boolean hasError(WebElement element) {
    /*
     * An error must be visible on an invalid component only when:
     * #1. The 'ng-invalid' CSS class is set by angular
     * AND
     * #2. the model is not pristine (because in our case we don't need to show
     * 		the error if there is no action from the user).
     */
    String classes = element.getAttribute("class");
    if (classes == null || classes.length() == 0) {
      return false;
    }
    // split classes to avoid conflict with names!
    String[] cssClasses = classes.split(" ");

    if (ArrayUtils.contains(cssClasses, "ng-invalid") // #1.
        && !ArrayUtils.contains(cssClasses, "ng-pristine") // #3.
    ) {
      return true;
    }
    return false;
}

Selenium / WITH ngAnimate / with an input which contains the following two constraints: ngMaxlength & ngMinlength
App.js
angular.module('myApp', [ 'ngAnimate' ]);

Part of Index.html
<input
        id="input#id1"
        ng-model="model.textvalue"
        class="text-value"
        ng-minlength="4"
        ng-maxlength="model.maxlength">

Selenium Test Result
===============================================
Default test
Tests run: 100, Failures: 47, Skips: 0
Selenium / WITH ngAnimate / with an input which contains only one constraint: ngMaxlength
App.js
angular.module('myApp', [ 'ngAnimate' ]);

Part of Index.html
<input
        id="input#id1"
        ng-model="model.textvalue"
        class="text-value"
        ng-maxlength="model.maxlength">

Selenium Test Result
================================================
Default test
Tests run: 100, Failures: 6, Skips: 0
Selenium / WITHOUT ngAnimate / with an input which contains the following two constraints: ngMaxlength & ngMinlength
App.js
angular.module('myApp', []);

Part of Index.html
<input
        id="input#id1"
        ng-model="model.textvalue"
        class="text-value"
        ng-minlength="4"
        ng-maxlength="model.maxlength">

Selenium Test Result
===============================================
Default test
Tests run: 100, Failures: 0, Skips: 0
Selenium / WITHOUT ngAnimate / with an input which contains only one constraint: ngMaxlength
App.js
angular.module('myApp', []);

Part of Index.html
<input
        id="input#id1"
        ng-model="model.textvalue"
        class="text-value"
        ng-maxlength="model.maxlength">

Selenium Test Result
===============================================
Default test
Tests run: 100, Failures: 0, Skips: 0

Resume
I use Selenium to test an input which contains always the following directive attribute: ngMaxlength.

ngAnimate
ngMinlength
Test Result

Disabled
Not Used
SUCCESS

Disabled
Used
SUCCESS

Enabled
Not Used
FAILED (6 failures)

Enabled
Used
FAILED (47 failures)

When I use both the ngMaxlength directive attribute and the ngMinlength directive attribute, the number of failures is upper than if I do not used the ngMinlength directive attribute.
Why the angular-animate fails with our application, with our tests?
Can someone give me some light on why we have this behavior please?
Thank you very much.

Thanks for the report. Can you please test with the latest 1.3.20? Or even better 1.4.9? 1.3.x only gets critical fixes.

Thank you for your quick message. I test with angular 1.3.20 and angular-animate 1.3.20, and also with angular 1.4.8 and angular-animate 1.4.8. Unfortunately the same bad behavior occurs.

Hello Narretz. Have you reproduced this behavior? What is the status for this issue? Thank you.

I haven't tested this yet. So the app on github shows this behavior? I can clone and run it and the error will appear?

Yes I hope so.

Hi Narretz, What about this issue? Our team is currently blocked with this issue and we don't know how to override it.
Regards.

We have some (unrelated to this specfic problem) "speed" problem with selenium.
All our inputs to the browser from Selenium are followed by a sleep of 50ms and that solves everything for us.

@zann12 I'll take a look right now. But don't forget, this is not a support forum, this tracker is only for bugs and feature requests. Sorry if I gave of a different impression.
Regarding your project - I see two github repos, but I have actually no idea how to use them together. So unless you can give me these instructions, I can't test this.
What you can try is to disable animations in your selenium tests by adding a test-specific config. For example, with protractor it would look like this: http://stackoverflow.com/questions/26584451/how-to-disable-animations-in-protractor-for-angular-js-appliction

Sorry, I can't help without clear instructions on the project. Closing. Feel free to reopen if you have more info for me.


Create Assembly Dependency Diagram

Create a simple diagram showing the assemblies and their dependencies and post it on the Wiki.

https://github.com/RobDixonIII/Bloom/wiki/Technologies#assembly-dependencies

Re-opening to be updated.


ML House Price Prediction by KoNoHa Property

Project ML House Price Prediction using Linear Regression Model using different model.

niceee. keep up the good work!


DO NOT MERGE! Fast Forward release-0.1-stable to 1fbd679f06f549592da55621744a89fe23fcacdf.

This PR will be merged automatically once checks are successful.

Jenkins job pilot/postsubmit passed


refactor: db abstraction

Issue: https://github.com/omisego/elixir-omg/issues/628
The idea behind the PR is to provide a set of behavioral patterns that specific database implementation needs to follow so that we're able to plug whatever we would like to test out.

Coverage decreased (-24.5%) to 57.08% when pulling 151194e32913bf5718e8c27a6a79ee2d35f87438 on 628-db_abstraction into 985c64729f2b446be0fde99b66a50d98653b07bf on master.

@pdobacz, actually the 100% identical code is on purpose at the moment. Rocksdb implementation will change over time (multithreaded pool), and leveldb will get deprecated. So I would wait until we fully implement the desired approach.
Squashing the two commits.

So I would wait until we fully implement the desired approach.

:+1: Wait until that, or we make some business-logic-related update to a driver, whichever comes first.
In case the final implementation of RocksDB gets delayed, we should groom this and do an even rudimentary DRYing (delegate to some OMG.DB.CommonDB?).
Come to think of it, I think I'll suggest doing it right away at the Rocks PR. At 1st glance this looks like a quick refactor and it can save us some headache if there's a bugfix in the drivers.

Why common, if we're throwing away LevelDB?

Why common, if we're throwing away LevelDB?

Common until then. After LevelDB is no-more we can drop common and have only a single driver RocksDB, but till then I'd rather go from one clean state to another clean state (again - iif it is straightforward to do so, which it seems it is)


SPA: Lazy loading / code splitting components & code changes

Feature request
I'm using code splitting / lazy loading of components that are partly split based on which page you are at. However if I deploy a new version all the hashes changes and the old files are gone using this module. So then changing page just gives a 404 error when it requests that's page js.
https://webpack.js.org/guides/code-splitting/
I have tried webpack standard splitting:
const ProductViewContainerComponent = () => {
    return import(/* webpackChunkName: "ProductViewContainerComponent" */ 'ProductViewContainerComponent').then(({ default: ProductViewContainerComponent })).catch((error) => {
       window.location.reload();
    });
}

And loadable/component
const ProductViewContainerComponent = loadable(() => import(/* webpackChunkName: 'ProductViewContainer' */'../components/ProductViewContainer'), {
  fallback: '',
});

But the import command itself doesn't seem to be try-catchable, even when done it still breaks the page.
What is the expected behavior?
Some way to keep old files for a defined time.
What is motivation or use case for adding/changing the behavior?
Sites not being broken during new deployments
How should this be implemented in your opinion?
Allow an option to only delete files after x amount of days. So it can for example only delete files that haven't changed for 30 days.
For now I have to stop using this plugin :'(

I run into this issue a lot  Uncaught Error: Loading chunk # failed
Seems it can be caused by two issues.

A bad network, one can add retry logic for fetching lazy-loaded components
New deployments, the client will try to access the old chunk and you will see this issue

I think an option to keep the X previous build would be great.


Any plans for RTL (right to left) support?

Hi @JiHong88 , the editor is impressive and covers nearly all of our requirements. But I don't see a config option for RTL  (right to left) text direction in the editor.
Is this something on the roadmap or already existing option? (I didn't see this in the docs). The rtl configuration is defined as..

In a right-to-left, top-to-bottom script (commonly shortened to right to left or abbreviated RTL), writing starts from the right of the page and continues to the left. This can be contrasted against left-to-right writing systems, where writing starts from the left of the page and continues to the right.
Arabic, Hebrew, Pashto, Persian, Urdu, and Sindhi are the most widespread RTL writing systems in modern times.

Example Toolbar button on other editor:

Configuration Example:

Hi @BrianJVarley
Thank for your suggestion.
I'll add RTL option and button to next version.

Hey @JiHong88 Looking at the RTL support on different editors, the RTL button seems to be a common approach to flipping the editor content direction. But another thing to consider is order and direction of toolbar buttons.
When I've tested in this editor you can see that certain toolbar icons are re-ordered and flipped backwards on language change to arabic:

@JiHong88 I see the commit above for RTL. Is this available to test on a public released version? https://github.com/JiHong88/SunEditor/commit/c43b5c511507025c152a1ef4fc5b50355bc29369

@BrianJVarley I will upload it to the test server.
Check and giving me feedback will help for develop.

@JiHong88 sure, How can I check this change on the test server?

@BrianJVarley I'll notify you at the this week after update.

Hi, @BrianJVarley
I uploaded RTL options under development.
It is still being modified.
Modal window, the title at the bottom of the toolbar, and the table selector are being modified.
Please tell me know if there are any further modifications to be made.
Ex) Image modify box, etc...
http://suneditor.com/test/index.html

Hi @JiHong88 I tested the RTL feature above. The editor contents alignment and text typing direction look great for RTL.
But I notice some bugs with toolbar button direction, label direction and dropdown lists contents. When I compare it to this editor implementation of RTL - https://ckeditor.com/ckeditor-4/demo/(To test RTL change browser language to Arabic).
Here is some feedback on RTL feature that I think will improve it. I think if you can make the below changes it will make RTL feature it will be more complete:

Toolbar Dropdown Menus (Format, Size, Font, Paragraph Style, Text Style, Line Height, Align): These toolbar button dropdown menus contents should be LTR (left aligned), because contents are still in English and Numbers. The ▼ icon on dropdown should display on left side of button text:

Icon Direction: In RTL only some icons should be reversed. I have highlighted in green icons that should be reversed, also highlighted in red icons should not be reversed i.e, same as LTR mode:

Ordering of Icons: These icons should be ordered as, [Italic, Underline, Bold] the reverse of LTR mode. Also icons should not be flipped backwards. They should be same direction as LTR mode.

Undo Redo Buttons: These buttons should not be flipped. They are same in LTR mode as in RTL mode. See: https://material.io/design/usability/bidirectionality.html#mirroring-elements

Toolbar Labels: Because toolbars labels are still in English, they should still read LTR. Currently, they are backwards.

Image Modify Box:  URL input text should be LTR not RTL, Submit button should be left side of modal, close (x) button should
be left side of modal. "Link" and "Image" tab text should be normal LTR as its English text. Tab should be on right side of modal.

@BrianJVarley I updated to the test page.
And please comment on these components.

Also the color picker is modified

@JiHong88 The above changes look correct at first look and tollbar behaviour is correct after latest change. A good test to verify RTL is implemented correctly is to open Google Docs and change keyboard language to arabic.
I do notice some minor bugs on label and toolbar buttons as follows:

Undo / Redo button: These buttons should be pointing same direction as in LTR mode. SO non change to icon, (left arrow = undo, right arrow - redo) See correct implementation here:

Indent / Out-dent button: The English part of the label here is backwards, should be LTR:

Hi @JiHong88 ,
Since some time I keep an eye on your wonderful wysiwyg library, it's really great!
I consider the RTL support as a must-have in modern content editors, it's amazing you've decided to implement it for the SunEditor.
The suggestions from @BrianJVarley are quite precise 👍 .
During the implementation of RTL feature in my project I have found the following resources as comprehensively informative to refer to:

https://translate.google.com/translate?hl=en&sl=ru&tl=en&u=https%3A%2F%2Fhabr.com%2Fru%2Fcompany%2F2gis%2Fblog%2F358148%2F
https://translate.google.com/translate?hl=en&sl=ru&tl=en&u=https%3A%2F%2Fhabr.com%2Fru%2Fcompany%2Fabbyy%2Fblog%2F251579%2F
https://www.w3.org/International/talks/1602-oman/ -- a nice post from W3
https://steelkiwi.com/blog/right-to-left-rtl-website-development-design/

Several useful suggestions:

Following things should always remain LTR (as they don't have RTL equivalents): phone numbers (including all signs), URLs, email addresses, date&time (unless it's in the Arabic/Hebrew traditional format)
The default RTL locales for most languages (Arabic, Hebrew, Urdu) in the widely-used regular fonts (Times, Arial, Helvetica, etc) are quite poor and either just not intended for such purposes, or just appear unreadable for native speakers. Consider the example below -- such tooltip label has very small Arabic letters.

So I think it's better to reconsider the font-family. I have found the Noto Sans family as the great alternative.
BTW, on the screenshot with the tooltip above, the string slices are misaligned towards to each other: if in English you see something like "Shortcut description (Ctrl+F5)" then in Arabic you should see "(F5+Ctrl) Shortcut description" -- the part in the round braces moves from the string ending into the beginning (and vice-versa).

Don't hesitate to reach out in case of questions!
I'll try to have some more walkthrough meantime,
Thanks,
Andrey.

@BrianJVarley @impulsgraw
I updated RTL mode of the test page. http://suneditor.com/test/index.html
The creation of all controllers and tables has also been modified with RTL.
And the list of align buttons has been modified so that the right sort is at the top by default.
At the bottom of the test page is a list of buttons used when creating the editor.
I created a button module by reversing the order of the button list in RTL mode, but I am not sure if it is the correct operation.
And..  I'm confused if the Math plugin should also be RTL.

@BrianJVarley @impulsgraw
The 2.34.0 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.

@JiHong88 great! I'll take a run through your linked test page this week to verify.


app cr downward api to pass info as values into templating

Describe the problem/challenge you have
there have been a few cases where we want to pass info about App CR as values into templates:

App CR name and namespace so that resources can be customized

if i am creating multiple App CRs in the same namespace, i would like to avoid users having to manually specify unique name (App CR name would be unique within a namespace, or in cluster if it's namespace+name)

App CR annotations/labels

may want to propagate certain annotations to created resources

App CR uid
App CR fetch information (vendir.lock.yml)

may want to include git info about where content came from as an annotation on a resource

Describe the solution you'd like
something along these lines... (existing valuesFrom expanded)
valuesFrom:
- secretRef:
    name: secret-name
- configMapRef:
    name: cfgmap-name
- path: values/shared.yml
# new type downwardAPI
- downwardAPI:
    items:
    - name: namespace
      fieldPath: metadata.namespace
    - name: name
      fieldPath: metadata.name
    - name: somewhere.source
      vendirLock: {} # since it's not a field on App CR

Anything else you would like to add:
prior art for downward API in Pods -- https://kubernetes.io/docs/tasks/inject-data-application/downward-api-volume-expose-pod-information/

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

App CR annotations/labels

@neil-hickey this might be actually a pretty good solution to the problem you faced around version injection at build time. annotation on AppCR/PackageCR can hold artifact version, and App CR can slurp it down and inject it into Deployment etc. additionally we can also do some automatic annotation setting on underlying App CR based on some metadata of Package CR (e.g. what if author of Package CR wants to inject package version as a data value into App CR to show that somewhere or let some underlying software know what it is, may be for upgrade checks...)

@cppforlife
I'm going to try and split this enhancement into 2 separate PRs.

k8s app info provided as values (name, namespace, uid, labels, annotations)
vendir info provided as values (vendirLock)

wrt annotations / labels:
I think we should also provide the ability to allow subscripts as an accepted configuration. (I think its good practice to have ability to provide as little values into templating as the operator wishes, as well as good security practice too). lmk if I should re-think this, or if you think this is too much scope creep! (I don't see this adding too much complexity into the code fwiw)
i.e. "metadata.annotations['myKey']" would be a way to specify a specific annotation 'myKey'


[WiP] [FIXED JENKINS-28933] - Exclude unrelocated docker-traceability-api from HPI

https://issues.jenkins-ci.org/browse/JENKINS-28933
Prevents the classloading bug. https://github.com/jenkinsci/maven-hpi-plugin/pull/18 merge&release is required to maintain the backward compatibility.
@reviewbybees

:+1:

:-1: just build using not maven-3.3, also if you released bad shaded hpi, then it may cause issues for other plugins. Could you investigate published hpis?

I don't agree the change is invalid, but another maven version is a right shortcut

@oleg-nenashev not only this jars a redundant compare shading with 3.2 and 3.3

docker-traceability-1.1 has been released using maven 3.2.5. I've checked the HPI in the jenkinsci repo, now everything is OK there

:bee:

Closing as invalid


delete the segmentation but keep the bounding box

Hello all,
I have a dataset that is in coco format and contains segmentations and bounding boxes (see image below). I want to delete the segmentation from the JSON file but keep the BBox information. Do I need to use the filter option for this or a transform operation?

Hi. If I understand correctly, what you need to do is replace the "segmentation" annotation fields with "[]". I can suggest few options how to do this.

Manually by a simple script or text editor. This will work if there are filled "bbox" values in the file. The script can be the following:

import json

with open('path/to.json') as f:
  d = json.load(f)

for ann in d.annotations:
    ann["segmentation"] = []

with open('path/to.json', 'w') as f:
  json.dump(d, f)

With Datumaro:

datum import -i path/to.json -f coco_instances -o original_proj
datum transform -p original_proj -t shapes_to_boxes -o boxed_proj
datum export -p boxed_proj -f coco_instances


#54988 (Resource Request: Improvement)

Default: OrderBy Bill Account -> ThenBy Code
Add Filter Bill Account: All, Bill, No Bill
Add Filter Code: Multiple Select Code - Status

https://github.com/ncc-erp/ncc-erp-project/assets/153600705/155e31b9-1744-46e9-8a69-e121ba746f69


Fix formatting for hour+ long songs

Description
Formatting for hour long songs are messed up.
Versions

Rainmeter Version: 4.2.0 r3111
OS Version: Windows 10 Pro 1803
Skin Version: 2.0.2
Spotify Version: WebNowPlaying <IP_ADDRESS>

Additional notes/images

There should be no issue with that, what player are you using?


Manual menu select

Added an option for a manual option selection from the code.
useful for a smoother user experience (take the user to the relevant page instead of making them picking the option)
Included new README with a relevant usage

Thanks. Look good to me.

Of course. When I'll have the frame I'll reexamine it and leave a more
positive review, if only for your good customer service.
On Wed, 17 May 2023 at 11:43, Hengbin Yan @.***> wrote:

Would welcome more PRs that address problems raised in the issues. Thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/victoryhb/streamlit-option-menu/pull/34#issuecomment-1551635175,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB4JFX5SES4LSVLEGK2KKR3XGTWZJANCNFSM6AAAAAAYFEZI5E
.
You are receiving this because you authored the thread.Message ID:
@.***>


Edge - Download dialog

Download instructions on Edge should be "Save As" to save directly onto micro:bit drive - will eliminate 1 step.  Also the "Click here if the download hasn't started." link doesn't seem to do anything…

This is not a P1.

Yes "save as" directly into the drive is a nice trick. Works in Firefox too.

Download dialog has been dramatically simplified and auto-dissapears.


Support for customize transaction name for performance trace

With version 2.3.1 before_send event seems to no longer be raised for "traces sample" case.
Do you have suggestions on how I can rename the transaction in this case or add spans in event ?
Thanks
Originally posted by @illambo in https://github.com/getsentry/sentry-laravel/issues/415#issuecomment-761759121

In order to attach spans I used this e seems work:
$transaction = SentrySdk::getCurrentHub()->getTransaction();

if ($transaction instanceof Transaction) {

    $transaction->setName('Foo'); // doesn't work ..

    if (defined('LARAVEL_START') && request()->header('X-Request-Start')) {
        $xRequestStart = floatval(str_replace('t=', '', request()->header('X-Request-Start')));
        $spanContext = new SpanContext();
        $spanContext->setOp('queuing'); // queuing time
        $spanContext->setStartTimestamp($xRequestStart);
        $spanContext->setEndTimestamp(LARAVEL_START); // microtime(true)

        $transaction->startChild($spanContext);
    }
}

Is this the best best practice ? Why the setName method  doesn't work ?

In order to attach spans I used this e seems work:
$transaction = SentrySdk::getCurrentHub()->getTransaction();

if ($transaction instanceof Transaction) {

    $transaction->setName('Foo'); // doesn't work ..

    if (defined('LARAVEL_START') && request()->header('X-Request-Start')) {
        $xRequestStart = floatval(str_replace('t=', '', request()->header('X-Request-Start')));
        $spanContext = new SpanContext();
        $spanContext->setOp('queuing'); // queuing time
        $spanContext->setStartTimestamp($xRequestStart);
        $spanContext->setEndTimestamp(LARAVEL_START); // microtime(true)

        $transaction->startChild($spanContext);
    }
}

Is this the best best practice ? Why the setName method  doesn't work ?

Hi @illambo, thanks for raising this issue.
I think you are doing it right but the order of operations are messing up your efforts.
First, the before_send no longer works because of this PR: https://github.com/getsentry/sentry-php/pull/1158. The before_send should have never worked for transactions.

Transactions should not go through beforeSend. However, they are still processed by Event Processors. This is a compromise between some flexibility in dealing with the current implementation of transactions as events, and leaving room for different lifetime hooks for transactions and spans.

With that out of the way I can say that you are doing the right thing renaming transactions but I think the Laravel SDK is overwriting those values here:
https://github.com/getsentry/sentry-laravel/blob/c5af38e42f5cd513731613a64a2c6bc8039892a0/src/Sentry/Laravel/Tracing/Middleware.php#L155-L164
This code is called just before the transactions is finished (and sent):
https://github.com/getsentry/sentry-laravel/blob/c5af38e42f5cd513731613a64a2c6bc8039892a0/src/Sentry/Laravel/Tracing/Middleware.php#L68-L76
So your code will always be too early or too late.
I am discussing with the PHP team what is the best way to do this or that we want to change our order of operations in the SDK to allow you to set a custom transaction name.
I'll get back to you on that but I hope this explains why it's not working what you are trying to do at the moment!

Hi @illambo, thanks for raising this issue.
I think you are doing it right but the order of operations are messing up your efforts.
First, the before_send no longer works because of this PR: https://github.com/getsentry/sentry-php/pull/1158. The before_send should have never worked for transactions.

Transactions should not go through beforeSend. However, they are still processed by Event Processors. This is a compromise between some flexibility in dealing with the current implementation of transactions as events, and leaving room for different lifetime hooks for transactions and spans.

With that out of the way I can say that you are doing the right thing renaming transactions but I think the Laravel SDK is overwriting those values here:
https://github.com/getsentry/sentry-laravel/blob/c5af38e42f5cd513731613a64a2c6bc8039892a0/src/Sentry/Laravel/Tracing/Middleware.php#L155-L164
This code is called just before the transactions is finished (and sent):
https://github.com/getsentry/sentry-laravel/blob/c5af38e42f5cd513731613a64a2c6bc8039892a0/src/Sentry/Laravel/Tracing/Middleware.php#L68-L76
So your code will always be too early or too late.
I am discussing with the PHP team what is the best way to do this or that we want to change our order of operations in the SDK to allow you to set a custom transaction name.
I'll get back to you on that but I hope this explains why it's not working what you are trying to do at the moment!

Ok, it's clear, thanks.
I think this in graphql scenario it is usefull in order to differentiate the performance (in the meantime I use the tags).

Ok, it's clear, thanks.
I think this in graphql scenario it is usefull in order to differentiate the performance (in the meantime I use the tags).

May I ask what GraphQL library you are using, I am considering implementing "native" support for surfacing transaction names for Lighthouse (either from the Laravel SDK or their package). But if you are using another I might want to check that out too!

May I ask what GraphQL library you are using, I am considering implementing "native" support for surfacing transaction names for Lighthouse (either from the Laravel SDK or their package). But if you are using another I might want to check that out too!

I am currently using rebing/graphql-laravel.

I am currently using rebing/graphql-laravel.

Do you have a temporary solution ?

Do you have a temporary solution ?

And maybe is better move this as an issue because not match the provided documentation.
What do you think?
Thanks

And maybe is better move this as an issue because not match the provided documentation.
What do you think?
Thanks

I have created #442 to hopefully resolve your issue, if you are able to test it out that would be great.

I have created #442 to hopefully resolve your issue, if you are able to test it out that would be great.

I did a quick test, it works great and thanks!

I did a quick test, it works great and thanks!

Awesome, I don't want to release this today if it's okay since it's Friday here.
I'll try and get this out on Monday!
Thanks for your testing and patience 👍

Awesome, I don't want to release this today if it's okay since it's Friday here.
I'll try and get this out on Monday!
Thanks for your testing and patience 👍

Hey @stayallive, I was looking into setting this up for Lighthouse.
Did you do any work on it?

Hey @stayallive, I was looking into setting this up for Lighthouse.
Did you do any work on it?


Add initial KEP for external DNS.

I added the initial KEP for the external DNS migration. My goal is to get a review here and then create a more solid KEP into https://github.com/kubernetes/enhancements/tree/master/keps/sig-network.
I'm looking forward for your feedback @njuettner @linki @hjacobs

Any chance to get a review on that @linki @njuettner @hjacobs ? If not, I will wait two additional weeks and then merge this and submit it to the right repository.

Thanks @Raffo lgtm

/lgtm
/approve

@Raffo thank you for taking care!


feat: element, vnode, vm, and options are read-only

Since vm, element and options are read-only in the type definition, this changes them to read-only.
Since the element property is read-only, this removes unnecessary  checks.
This is related to #725.

Thank you for pointing out.
I changed it.


fix: server$ not throwing for errors above 500

What is it?

Bug

Description
Server functions only throw errors in the browser with 500 status code errors.
For errors that are greater than 500, such as a 504, the server functions will return the text which means you are unable to correctly catch and handle the errors.
Checklist

[X] My code follows the developer guidelines of this project
[X] I performed a self-review of my own code
[x] I added a changeset with pnpm change
[ ] I added new tests to cover the fix / functionality

@shairez @wmertens a gentle nudge, any chance this could get reviewed?


Update Dockerfile to pull latest version of web app

With this update, we should bump the version number from 1.2 to 1.3.

@bdwyer2 Do we need to bump the version number under this situation? Nothing in this repo has actually changed but the wrapped web app has a major update.

@xuhdev The version bump is nessary for the long running instance to pick up the change, but I would say this should be a patch update instead by your reasoning


BAD REQUEST (400)

I am getting BAD REQUEST (400) when I am trying to connect to any url ex: http://<IP_ADDRESS>:5000/api/v1/news/. What are the steps to resolve this?
Output of ./manage.sh up
Creating infrastructure...
Recreating mongo ... done
MongoDB shell version v4.2.17
connecting to: mongodb://localhost:27017/rss_news?compressors=disabled&gssapiServiceName=mongodb
Implicit session: session { "id" : UUID("59975a30-4c37-429f-a574-f5178dc7f588") }
MongoDB server version: 4.2.17
{
	"ok" : 0,
	"errmsg" : "command replSetInitiate requires authentication",
	"code" : 13,
	"codeName" : "Unauthorized"
}
bye
Initiated replica set
MongoDB shell version v4.2.17
connecting to: mongodb://localhost:27017/admin?compressors=disabled&gssapiServiceName=mongodb
Implicit session: session { "id" : UUID("22b505ad-c940-46bf-ba2b-2620b4bacd36") }
MongoDB server version: 4.2.17
2021-10-11T04:28:05.628+0000 E  QUERY    [js] uncaught exception: Error: couldn't add user: command createUser requires authentication :
_getErrorWithCode@src/mongo/shell/utils.js:25:13
DB.prototype.createUser@src/mongo/shell/db.js:1413:11
@(shell):1:1
2021-10-11T04:28:05.628+0000 E  QUERY    [js] uncaught exception: Error: command grantRolesToUser requires authentication :
_getErrorWithCode@src/mongo/shell/utils.js:25:13
DB.prototype.grantRolesToUser@src/mongo/shell/db.js:1635:15
@(shell):1:1
bye
MongoDB shell version v4.2.17
connecting to: mongodb://localhost:27017/admin?compressors=disabled&gssapiServiceName=mongodb
Implicit session: session { "id" : UUID("0e95023f-9a00-4201-8a07-7f13608ed26b") }
MongoDB server version: 4.2.17
{
	"ok" : 0,
	"errmsg" : "not master",
	"code" : 10107,
	"codeName" : "NotWritablePrimary"
}
2021-10-11T04:28:05.745+0000 E  QUERY    [js] uncaught exception: Error: couldn't add user: not master :
_getErrorWithCode@src/mongo/shell/utils.js:25:13
DB.prototype.createUser@src/mongo/shell/db.js:1413:11
@(shell):1:1
bye
Recreating postgres ...
   40 seconds ago      Up 39 seconds (health: starting)   <IP_ADDRESS>:8083->8083/tcp, 9092/tcp             connect
11abb03946fb        confluentinc/cp-schema-registry:5.3.1   "/etc/confluent/dock…"   41 seconds ago      Up 40 seconds                      8081/tcp                                     schema-registry
96ffc1d68f8b        dataengineeringproject_kafka            "./create_default_to…"   41 seconds ago      Up 40 seconds                      9092/tcp                                     kafka
cf36bccafb4b        dataengineeringproject_proxy            "/docker-entrypoint.…"   42 seconds ago      Up 40 seconds                      <IP_ADDRESS>:5000->5000/tcp, 8080/tcp             proxy
49266edc8a5f        dataengineeringproject_api              "./run_api.sh"           42 seconds ago      Up 42 seconds                                                                   api
3f776a6022ed        dataengineeringproject_airflow          "/entrypoint.sh webs…"   43 seconds ago      Up 41 seconds (healthy)            5555/tcp, 8793/tcp, <IP_ADDRESS>:8080->8080/tcp   airflow
2bfc84caa93f        redis:alpine                            "docker-entrypoint.s…"   43 seconds ago      Up 40 seconds                      <IP_ADDRESS>:6379->6379/tcp                       redis
ada4f43e8a2e        confluentinc/cp-zookeeper:5.3.1         "/etc/confluent/dock…"   43 seconds ago      Up 41 seconds                      2888/tcp, <IP_ADDRESS>:2181->2181/tcp, 3888/tcp   zookeeper
d9e5f3169391        dataengineeringproject_elasticsearch    "/tini -- /usr/local…"   43 seconds ago      Up 42 seconds                      <IP_ADDRESS>:9200->9200/tcp, 9300/tcp             elasticsearch
d9ce588ea654        postgres:9.6                            "docker-entrypoint.s…"   43 seconds ago      Up 42 seconds                      5432/tcp                                     postgres
bbb7edd6cf8d        mongo:4.2                               "docker-entrypoint.s…"   55 seconds ago      Up 54 seconds                      <IP_ADDRESS>:27017->27017/tcp                     mongo

Hi, did you try ./manage.sh down (deletes all containers) and then ./manage.sh up?

The health of "connect" container is showing "starting" event after 5 min from start.
CONTAINER ID        IMAGE                                   COMMAND                  CREATED             STATUS                            PORTS                                        NAMES
829efb7e52d0        dataengineeringproject_connect          "./register_connecto…"   5 minutes ago       Up 5 minutes (health: starting)   <IP_ADDRESS>:8083->8083/tcp, 9092/tcp             connect

Logs of this container are below:
	... 24 more
[2021-10-11 18:33:00,714] ERROR WorkerSinkTask{id=s3-minio-sink-0} Task is being killed and will not recover until manually restarted (org.apache.kafka.connect.runtime.WorkerTask)
[2021-10-11 18:33:06,214] INFO Checking current members of replica set at rs0/mongo:27017 (io.debezium.connector.mongodb.ReplicaSetDiscovery)
[2021-10-11 18:33:36,214] INFO Checking current members of replica set at rs0/mongo:27017 (io.debezium.connector.mongodb.ReplicaSetDiscovery)
[2021-10-11 18:33:38,803] INFO WorkerSourceTask{id=mongo-dbz-source-0} Committing offsets (org.apache.kafka.connect.runtime.WorkerSourceTask)
[2021-10-11 18:33:38,805] INFO WorkerSourceTask{id=mongo-dbz-source-0} flushing 0 outstanding messages for offset commit (org.apache.kafka.connect.runtime.WorkerSourceTask)
[2021-10-11 18:33:38,839] INFO WorkerSourceTask{id=mongo-dbz-source-0} Finished commitOffsets successfully in 35 ms (org.apache.kafka.connect.runtime.WorkerSourceTask)
[2021-10-11 18:34:06,214] INFO Checking current members of replica set at rs0/mongo:27017 (io.debezium.connector.mongodb.ReplicaSetDiscovery)


Cython attempt

@brendan-ward  Based on your work and our discussion in #197, the example project in http://github.com/caspervdw/cygeos and your working branch  https://github.com/brendan-ward/pygeos/tree/add_cython  I put together this PR that (at least locally) works.
And it also uncovers the major shortcoming of having "pygeos.c" in two Extensions: the GeometryObject in pygeos.lib does not equal the GeometryObject in pygeos.cythondemo.
So I guess we'll have to resort to C pointer arrays in PyCapsules for the objects that we need to share between the two extensions. (https://docs.python.org/3/extending/extending.html#providing-a-c-api-for-an-extension-module)
edit: Also this needs Cython >= 0.29

@brendan-ward  I copied your capsules approach from #197. It seems that adjusting the call signature of GetGeom from GeometryObject * to PyObject * solved the issue of having to define GeometryObject in Cython.

However, it struck me that #197 gives us a more complete testing and demonstration of using Cython, since it exercises both getting and setting geometries using the C API, and exposing those into Python. Since there are gotchas throughout the stack, I think our first roughout of Cython in pygeos probably should exercise the major parts (#197 does).

You're right. Let's focus the work back on #197, I'll close this and start reviewing #197.

I don't think there are issues with defining GeometryObject - the C struct - in Cython; it isn't the actual Geometry type, so it doesn't cause all the issues we saw around the type being created twice. Personally, I would keep GeometryObject in Cython because the C type helps make our variables and function signatures more clear - we are dealing in GeometryObjects rather than PyObjects where we have to track implicitly what type we're working with.

Not sure if I agree on this. Defining C structs on two locations is a liability, and I think that the struct itself might better be opaque outside of pygeom.c. No need to access the ptr if you can use the get_geom function as well. Concerning the get_geom function, I think its call signature should be refactored into (PyObject *, GEOSGeometry **), because we actually use it to typecheck any PyObject. Take for instance https://github.com/pygeos/pygeos/blob/32397d998d2fc7fe3afef673539dace9fd8efe7f/src/ufuncs.c#L142
Here, the PyObject is cast to a GeometryObject, but we actually can't know at that point if it is in fact a GeometryObject. So I think get_geom should not have GeometryObject in its signature.


Feature/HOLO 534 holograph treasury tests

Describe Changes

Add tests for HolographTreasury
Added MockExternalCall contract for testing external calls

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] Code styles have been enforced
[x] All Hardhat tests are passing

@ackvf i accidentally deleted your comment, thinking i was deleting my reply. :( sorry.

@sogoiii LOOOOL I didn't know you can do that 😂


Running "home" shows "10 minutes ago", "1 hour ago" instead of time and date.

I've done so that on login, an expect-script starts rainbowstream and runs the "home" command to display the latest tweets.
Its installed on an raspberry pi and connected to a monitor, and when rainbowstream is up and running, the tweets that is showed by the "home"-command is displayed with "10 minutes ago", "1 hour ago" instead of time and date.
The funny thing is that if I SSH into the raspberry, it will also run the expect script and then rainbowstream, but there it shows the whole time and date of the tweet.
Any explanation?
raspberry is set to auto-login
Code in .bashrc to run at login:
echo Starting FireWatch...
sleep 15
source venv/bin/activate
expect -f rainbowstream.exp

Code for expect script:
#!/usr/bin/expect -f

#The value of timeout must be an integral number of seconds. Normally timeouts $
set timeout 7200

# Now we start rainbowstream $
spawn rainbowstream

# Now we expect the twitter prompt to appear
expect "*Venter...*"

# And we write our home command to display latest tweets:
send -- "home 20"

# send blank line (\r) send command
send -- "\r"

#If twitter resets the connection
expect "*onnection reset*"

#Reconnect
spawn rainbowstream

#We have gave our "program" and now we expect that the remote server close the $
expect eof

.rainbow_config:
{
    // Turn to 'true' in order to disable extended tweets display (legacy mode)
    "DISABLE_EXTENDED_TWEETS" : false,
// After 120 minutes, the stream will automatically hangup
    "HEARTBEAT_TIMEOUT" : 500000,
    // Image on term
    "IMAGE_ON_TERM" : false,
    // Resize image to fit on terminal view
    "IMAGE_RESIZE_TO_FIT" : false,
    // Themes
    "THEME" : "monokai",
    // Ascii Art
    "ASCII_ART" : false,
    // Hide promt when receive a tweet
    "HIDE_PROMPT" : true,
    // Prefix
    "PREFIX" : "Venter...",
    // 'search': search type ('mixed','recent','popular')
    "SEARCH_TYPE" : "mixed",
    // 'search': search max result, number over 100 will fallback to 100
    "SEARCH_MAX_RECORD" : 5,
    // 'home': default number of home's tweets
    "HOME_TWEET_NUM" : 20,
    // 'allrt': default number of retweets
    "RETWEETS_SHOW_NUM" : 5,
    // 'conversation': max tweet in a thread
    "CONVERSATION_MAX" : 30,
    // 'quote' format
    "QUOTE_FORMAT" : "#comment https://twitter.com/#owner/status/#tid",
    // 'thread' meta format
    "THREAD_META_LEFT" : "(#id) #clock",
    "THREAD_META_RIGHT" : "#clock (#id)",
    // 'thread' frame's minimum width
    "THREAD_MIN_WIDTH" : 20,
    // 'Notification' format
    "NOTIFY_FORMAT" : "  #source_user #notify #clock",
    // 'inbox','sent': default number of direct message
    "MESSAGES_DISPLAY" : 5,
    // 'trend': max trending topics
    "TREND_MAX" : 10,
    // List home timeline max
    "LIST_MAX" : 5,
    // 'switch': Filter and Ignore list ex: ['@fat','@mdo']
    "ONLY_LIST" : [],
    "IGNORE_LIST" : [],
    // Autocomplete history file name
    "HISTORY_FILENAME" : "completer.hist",
    // Image margin
    "IMAGE_SHIFT" : 2,
    // Image max height
    "IMAGE_MAX_HEIGHT" : 90,
    // Seconds to wait before displaying another tweet, will drop all tweets while waiting.
    "STREAM_DELAY" : 0,
    // Stream config
    "USER_DOMAIN" : "userstream.twitter.com",
    "PUBLIC_DOMAIN" : "stream.twitter.com",
    "SITE_DOMAIN" : "sitestream.twitter.com",
    // Format
    "FORMAT": {
        "TWEET": {
            "CLOCK_FORMAT" : "%d/%m/%Y %H:%M:%S",
            "DISPLAY" : "\n  #name #clock \n  #fav\n  #tweet"
        },
        "MESSAGE": {
            "CLOCK_FORMAT" : "%Y/%m/%d %H:%M:%S",
            "DISPLAY" : "\n  #sender_name #sender_nick #to #recipient_name #recipient_nick :\n  #clock message_id:#id\n  #message"
        }
    },
    "POCKET_SUPPORT": false
}

Looks like a bug to me. I change draw() in draw.py to have the default Parameter humanize be False:
def draw(t, keyword=None, humanize=False, noti=False, fil=[], ig=[]):
That did the trick for me. Seems that the parameter humanize is a bit misleading here. It is used inside as
clock = fallback_humanize(date, clock_format, not humanize)
which renames the parameter to use_fallback:
def fallback_humanize(date, fallback_format=None, use_fallback=False):
The usage of use_fallback now is straight forward:
    if not fallback_format:
        fallback_format = '%Y/%m/%d %H:%M:%S'
    # Determine using fallback format or not by a variable
    if use_fallback:
        return date.datetime.strftime(fallback_format)

Since I don't have the time to fix it, I did the hack above, which solves my problem: don't show any relative date-time.
RandomCore


kubernetes: update response to reflect AdmissionReview metadata

addresses #34
nit: i had to use else to ensure safe defaults.

/cc @tsandall

LGTM


thanos query takes high cpu when added self as a store

Thanos: 0.5.0
go: go version go1.11.2 darwin/amd64
thanos, version 0.5.0 (branch: HEAD, revision: 72820b3f41794140403fd04d6da82299f2c16447)
  build user:       root@7d72e9360b09
  build date:       20190606-10:49:59
  go version:       go1.12.5

What happened
If thanos query is added itself as a store then it takes high cpu after running any query.
What you expected to happen
It should give warning about adding self or shouldn't allow adding itself
How to reproduce it (as minimally and precisely as possible):

start thanos query

./thanos-0.5.0.darwin-amd64/thanos query --http-address <IP_ADDRESS>:9091 --store=localhost:10901

access the ui and run any query, eg. up
wait for ~30-45 seconds
local machine's cpu boosts up and thanos process needs kill -9

Full logs to relevant components
Uncomment if you would like to post collapsible logs:

level=warn ts=2019-07-14T03:09:27.682971Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"
level=warn ts=2019-07-14T03:09:27.683081Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"
level=warn ts=2019-07-14T03:09:27.683321Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"
level=warn ts=2019-07-14T03:09:27.683494Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"
level=warn ts=2019-07-14T03:09:27.683669Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"
level=warn ts=2019-07-14T03:09:27.68383Z caller=proxy.go:349 err="failed to receive any data from Addr: localhost:10901 Labels: [] Mint: 0 Maxt:<PHONE_NUMBER>854775807: context canceled" msg="returning partial response"

Anything else we need to know

Yeah... that's to be expected 😕 , however, how would you solve this? Simply checking for <IP_ADDRESS> or localhost is not an answer since someone could a proxy in front and then we would be back to square one. We could print a message at best in such situations, IMHO. Also, Having something like "correlation IDs" on top of the queries would be nice however that would probably incur a heavy performance penalty and bring more harm than good.


String formatting no longer works with typed variables

Godot version:
3.1.dev
Commit: 4b549fa
OS/device including version:
Windows 8
Issue description:
String formatting using the % operator no longer works with typed variables.
Steps to reproduce:
The following code snippet
func random_function(value:int):
    print("Input received: %d" % value)

gives this error:
 built-in:24 - Parse Error: Invalid operand types ('String' and 'int') to operator '%'.
 modules/gdscript/gdscript.cpp:579 - Method/Function Failed, returning: ERR_PARSE_ERROR

The following two functions do work as they should:
func random_function(value:int):
    print("Input received: %d" % [value])

and
func random_function(value:int):
    var copy = value
    print("Input received: %d" % copy)

CC @vnen


feat(router): add gateway_status_map CRUD APIs

Type of Change

[ ] Bugfix
[x] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
add gateway_status_map CRUD APIs
Additional Changes

[x] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?
Tested Manually

GSM Create
CURL
curl --location --request POST 'https://sandbox.hyperswitch.io/gsm' \ --header 'api-key: {ADMIN_API_KEY}' \ --header 'Content-Type: application/json' \ --data-raw '{  "connector": "stripe",  "flow": "Authorize",  "sub_flow": "sub_flow",  "code": "incorrect_cvc",  "message": "incorrect_cvc",  "status": "failure",  "decision": "retry",  "step_up_possible": true }'

GSM Update
CURL
curl --location --request POST 'https://sandbox.hyperswitch.io/gsm/update' \ --header 'api-key: {ADMIN_API_KEY}' \ --header 'Content-Type: application/json' \ --data-raw '{  "connector": "stripe",  "flow": "Authorize",  "sub_flow": "sub_flow",  "code": "incorrect_cvc",  "message": "incorrect_cvc",  "decision": "do_default" }'

GSM Retrieve
CURL
curl --location --request POST 'https://sandbox.hyperswitch.io/gsm/get' \ --header 'api-key: {ADMIN_API_KEY}' \ --header 'Content-Type: application/json' \ --data-raw '{  "connector": "stripe",  "flow": "Authorize",  "sub_flow": "sub_flow",  "code": "incorrect_cvc",  "message": "incorrect_cvc" }'

GSM Delete
CURL
curl --location --request POST 'https://sandbox.hyperswitch.io/gsm/delete' \ --header 'api-key: {ADMIN_API_KEY}' \ --header 'Content-Type: application/json' \ --data-raw '{  "connector": "stripe",  "flow": "Authorize",  "sub_flow": "sub_flow",  "code": "incorrect_cvc",  "message": "incorrect_cvc" }'

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible
[ ] I added a CHANGELOG entry if applicable

In order to make it easier to use the gsm apis, a gsm list endpoint will be a good addition. This endpoint can have filters for connector, flow_type etc. Also the update endpoint should not allow the primary key fields to be updated and instead allow only the actions ( step_up, retry_action etc ) to be updated. Each gsm can have a unique identifier gsm_abcd which will act as a primary key to update / delete the gsm entry.
This can be added as a good first issue.

@sai-harsha-vardhan Can you add the cURL request?

are the errors due to changes in this PR?

are the errors due to changes in this PR?

@Narayanbhat166 This is due to APIEventMetrics which are added recently, fixing it!


Mac OS X 10.11.1
What did you do?
Saved this Go program into main.go:
package main

import "os"

func main() {
	os.Exit(3)
}

And then ran go run main.go:
$ go run main.go
exit status 3
$ echo $?
1

What did you expect to see?
$ echo $? should be 3 instead of 1
What did you see instead?
The exit code reported by $ echo $? is always 1 if the program exits with a non zero exit code.
If you compile the example with $ go build -o gotest it returns the expected exit code:
$ ./gotest
$ echo $?
3

In the general case we can't recreate the right exit status, but I suppose we could pass along an ordinary exit code.

I think the exit code would be enough, just to be able to use go run ... from other tools, without requiring a go build first. Context: we do these for smaller scripts, where we use Go like a scripting language, instead of Ruby / Python / etc.

Not for Go 1.6.

From code review:

What if it crashed due to a signal? Now it's exit(someRandomNumber).
I am not convinced this needs to be done at all.

It won't exit with some random number if it fails with a signal.
It will exit with status 1 as it did before.

Fair enough, let me try again:
What if it crashed due to a signal?
It used to print information about the exit reason and exit 1.
Now it's a silent exit(someRandomNumber).
The real question is whether 'go run x.go' is supposed to be just an interpreter for Go programs, like 'python x.py' or whether it is a tool that runs a subprocess and reports the results itself, like make. To date, the answer has been the latter. So it's not obvious the behavior is wrong, unless 'go run' is supposed to be some kind of interactive go command, which we've said in the past it is not.
$ cat makefile
exit5:
	exit 5
$ make exit5
exit 5
make: *** [exit5] Error 5
$ echo $?
2
$

(Make happens to use 2 instead of 1, but it's always 2 on my system.)
$ cat x.go
package main

import "os"

func main() {
	os.Exit(5)
}
$ go run x.go
exit status 5
$ echo $?
1
$

I agree with Russ for the reasons he stated.
An exit code is a single-dimensional value. When doing go run, there are 2 processes that run and 2 exit codes one may want to know.
However, go run is only able to report a single exit code value, not two. It's not possible to losslessly combine two exit codes into one. If it reported the exit code of the program it ran verbatim, then information about the go run exit code would be shadowed and effectively lost.
IMO, if one cares about the exact exit code of the program that is executed, they need to build it and execute it themselves. go run is a convenience feature for when your needs are not as demanding and you're okay with less information, because it's unable to communicate more information than it already does.

I think this is effectively closed.

I updated the CL to fix only windows. Still wrong?

@shurcooL how about to add new flag to return status code (non-1) of the exit code from go run?

Please, no more flags. Go run is for toy programs, if you have a serious
program then build and run it seriously.
On Wed, 11 May 2016, 13:26 mattn<EMAIL_ADDRESS>wrote:

@shurcooL https://github.com/shurcooL how about to add new flag to
return status code (non-1) of the exit code from go run?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/13440#issuecomment-218352428

@davecheney So do you think https://github.com/sno6/gommand this should do go build -o myapp &&  ./myapp it self if he want exit code of the program?

My advice that Go run is a toy, and adding more features to it to make it
more capable is moving in the wrong direction. The rest of the go tool is
focused on code placed in packages and built and tested as such.
On Wed, 11 May 2016, 13:32 mattn<EMAIL_ADDRESS>wrote:

@davecheney https://github.com/davecheney So do you think
https://github.com/sno6/gommand this should do go build -o myapp &&
./myapp it self if he want exit code of the program?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/13440#issuecomment-218353024

Okay, agreed. Thanks.


[BUG] 4.5: SNOWFLAKE : Lateral flatten failing parsing

Always check against the Latest SNAPSHOT of JSQLParser and the Syntax Diagram
Failing SQL Feature:

Snowflake LATERAL FLATTEN function fails
LATERAL FLATTEN net.sf.jsqlparser.JSQLParserException: Encountered unexpected token: "FLATTEN" "FLATTEN"

SQL Example:
WITH t AS (
  SELECT
    'ABC' AS dim,
    ARRAY_CONSTRUCT('item1', 'item2', 'item3') AS user_items
)
SELECT DIM, count(value) as COUNT_
FROM t a,
LATERAL FLATTEN(input => a.user_items) b
group by 1

Software Information:

JSqlParser version 4.5, 4.6, 4.7-SNAPSHOT
Database SNOWFLAKE

Tips:
Lateral flatten is similar to explode used to flatten a json structure.
https://docs.snowflake.com/en/sql-reference/functions/flatten
http://jsqlformatter.manticore-projects.com/jsqlformatter/demo.html?args=-c OoSQKgEgBALlCCBlKAKAUFKiCiAZbAwmFBplAOTwBCB5CyAJgJYC2ANCWQgErfwCaAfQIB5AHKIw3AKpEU5JjACmLAIzkOC5SwBMGiopUBmcgEp6UAK4BnJQCdBhltbSm0OfESgAREAFkOAGMAe0sAOxgUADcAQwAbSyVzGOsoUWkxMEE0ADFuET9YKBi2NFx4MGw+XCgc8rBKsRQmMIAHSzgAXgA+YoA6G3tHbWtzACM0AHM7UNaoMYBPKFUgA

Greetings,
only LATERAL subquery is supported: http://<IP_ADDRESS>:8080/jsqlformatter/JSQLParser/syntax_snapshot.html#lateralview although this certainly can be changed into a LATERAL expression which would cover Function and Select objects.

This one works already
-- subquery
select * from lateral (select 1)

-- expression
select * from lateral (select * from flatten(input => a.user_items))

Thanks that worked. Do you think adding support for lateral flatten would be added in the future?

Hi @manticore-projects
were you able to add it in? Is there a PR i can probably use for my local testing?

@manticore-projects quickly checking again if this is still being planned

Thank you for the reminders and sorry for taking that long.
Cheers!

Thanks a lot 🙏


register an iceberg table with the spark client

I would do a single PR where you also read the iceberg table 😄 let's break PRs but not too small


net: dns_sd, mdns: support service type enumeration

Support DNS-SD Service Type Enumeration in the dns library and mdns_responder sample application.
For more information, please see Section 9, "Service Type Enumeration" in RFC 6763.
https://datatracker.ietf.org/doc/html/rfc6763#section-9
Fixes #38673

The compliance failure is a false positive (bug in ./scripts/checkpatch.pl. @galak, you may need to merge this one manually after it's approved.

Works now with avahi-browse -r -t -a <img width="957" alt="Screen Shot 2021-09-26 at 6 58 52 AM" src="https://user-images.githubusercontent.com/983494/134804945-ea984f1f-3e71-4a8e-bd67-b9aa6dc9a8b5.png">

So service type enumeration is working now, in a way that wireshark agrees with. However, I do need to do a avahi-browse -r -t -a in order for avahi to send out the enumeration query. avahi-discover does send out the query but it seems that 'non-standard' services are not listed. I was able to test with BOARD=stm32f746g_disco and the macOS application discovery and _zephyr._tcp.local is indeed automatically discovered with this fix.

Also, note, that there is an enhancement ticket to provide an overall improvement of DNS-SD in v3.0.0, with emphasis on optional features like this (service type enumeration), domain enumeration, as well as complete name, subtype, and non-local domain support.
For more information, see #38846

@rlubos , @jukkar - please have a look when you have a few minutes.

@galak / @nashif - either of you want to ack? Compliance failure is a false positive, so this needs to bee merged manually.

Added workaround for bug in compliance checks.

When I performed  dns-sd -B _zephyr._tcp on my pc to query my device running Zephyr v2.6.0, I was able to get a response  like below containing all the necessary information.

If I try the same query with Zephyr v2.7.0-rc4 with the fix for the service type enumeration, my device seems to not respond the second packet anymore....


Reorganize config samples

Remove namespace openstack, to align with other operators
Ensure standalone and conductor groups samples have unique metadata
names so that kustomize will process them
Remove image tags from all samples, defaults now come from the
operator environment
Add a JSON6902 patch file which restates the image defaults.
It is provided as an example and for tooling to use it as a template
to convert to versioned tags.
Add the image patch to kustomization.yaml, just for validation -
these samples are not bundled with the operator.
Set the replicas attribute on the single service samples. They still
won't work but at least there is something valid.

/test ironic-operator-build-deploy-kuttl

ironic-operator-build-deploy-kuttl is green now, this can be review on it's content


[Android] Perspective transform styles not working

I am passing     transform: [       { perspective: 500 }     ] to one of my view's style. Works on iOS, throws following error on Android:
"undefined is not a function (evaluating 'MatrixMath.inverse3x3(perspectiveMatrix)')"

I saw MatrixMath.js and couldn't find it, maybe uncompeleted on Android

Just got hit by this.

I got it,it'a bug,look this issue


⚠ Bofrost.be has degraded performance

In 97ad034, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 806 ms

Resolved: Bofrost.be performance has improved in dc317c3 after 1 hour, 2 minutes.


[SwiftLanguageRuntime] Simplify Error dynamic type lookup.

We don't really need the witness table pointer, so don't bother
computing it.

@swift-ci test and merge


bgp nlri evpn not supported (junos)

Original issue napalm-automation/napalm#191 created by @fooelisa

print device.get_bgp_neighbors()
Traceback (most recent call last):
File "", line 1, in
File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/napalm-0.51.0-py2.7.egg/napalm/junos.py", line 307, in get_bgp_neighbors
peer['address_family'] = self._parse_route_stats(neighbor_data)
File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/napalm-0.51.0-py2.7.egg/napalm/junos.py", line 266, in _parse_route_stats
family = self._get_address_family(table)
File "/opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/napalm-0.51.0-py2.7.egg/napalm/junos.py", line 258, in _get_address_family
return address_family_mapping[family]
KeyError: 'evpn'

@fooelisa can we have more detail on this? The section "Steps to reproduce the issue" is what I am looking for.
Or is this obsolete?

Assigning this to @kderynski

Fixed via https://github.com/napalm-automation/napalm-junos/pull/47


Release 1.7.1

Descriptive summary
Version: 1.7.1
Release Manager: @hweng
Product Owner: @mcritchlow
Operations: @jhriv
Addresses Nokigiri security vulnerability via #108

@jhriv The new release has been merged into master branch. Could you please deploy it? Thanks


Failed to cd directory in start @ line 1498

Describe the bug
Upon running the script :
sudo ./reconftw.sh -d target.com -r

I'm getting an error saying Failed to cd directory in start @ line 1498
To Reproduce
Steps to reproduce the behavior:

Go to 'Reconftw' directory
Run the script using sudo command,
scan a target
See error

Expected behavior
The script should've start the scanning
Screenshots

Desktop (please complete the following information):

Architecture: x64
OS: Parrot oS
Version Debian 5.10.28-6

Fixed in dev, I will release a new version today :)