the Italian language has disappeared from the hotword

Hello, since the beginning of the month, I and other users, we found that the app assistant, you can no longer set the Italian language for hotword.
Can we kindly know the reason? since the google home and google home mini have also arrived in our country !!
Thanks to those who will give us a plausible explanation

please give an explanation of why using sdk library we do not have the Italian language?
Until the end of May it worked now no longer appears in the app assistant

Encountered the same problem but seems that using pushtotalk example code italian language appears...

@S1m0n3
we do not confuse the 2 bookstores !!! pushtotalk, uses Google Assistant Service and Italian is also present.
While hotword uses the Google Assistant Library that from 8 March to 30/05/2018 also worked in Italian and now no longer and we do not know yet the reason after almost a month from the report.

@aandroide Yes, I know! But how you saying both support the Italian language from documentation of Google, so just want to know the reason because the Italian language disappears only when we using  Google Assistant Library! ;)

@proppy @S1m0n3 @aandroide @Fleker
sorry but is not that by chance google has made an agreement with amazon alexa? because look at the case in early June alexa also speaks in Italian and google assistant sdk instead no.

I am really sorry to hear you've been having trouble with the Google Assistant Library and just wanted to follow up and let you know that this has been brought to my attention and we are actively investigating the regression. I hope we can get this resolved quickly, thank you for your patience!

@JordanJTW
Ok today I noticed the new update from the google app playstore, but the Italian language does not appear yet. moreover, even the voice match to set the word ok google on the phone no longer works

@JordanJTW  it would seem that the problem manifests itself in Android devices. on iOS, other users have confirmed to me that they display the Italian language.
so at this point, if so, who should we ask? I will have sent a thousand feedback from the app assistant but I have never received an answer

I confirm that on ios it works, I tried today. so I guess it's not your problem but the google app developer on the android playstore

@JordanJTW
hello today the problem has not yet been solved on android devices.
how much longer do we have to wait?

solved


[Do not review; Testing AppVeyor] adding logging to failing test

Will create new PR for actual fix.


feat(*): upgrade golang image to 1.12-security

What this PR does / why we need it:
Add your description
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to # http://jira.caicloud.xyz/browse/CPS-8648

Special notes for your reviewer:
/cc @hicaistar

Release note:

NONE

/lgtm


Added SQL-on-FHIR v2 repo tests

Description of what I changed

Fixes #954, barring some issues that need to be fixed/discussed in upstream repos (marked as TODOs).
As a side effect we get a nice increase on the test coverage of the view package.
E2E test

TESTED:
This is all about tests!
Checklist: I completed these to help reviewers :)

[x] I have read and will follow the review process.

[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.

[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.

[x] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)

[x] I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

[x] All new and existing tests passed.

[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command git pull --rebase upstream master

Codecov Report
Attention: Patch coverage is 70.00000% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 50.33%. Comparing base (dda80a4) to head (103b1ef).

Files
Patch %
Lines

...com/google/fhir/analytics/view/ViewApplicator.java
73.33%
2 Missing and 2 partials :warning:

...com/google/fhir/analytics/view/ViewDefinition.java
60.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #988      +/-   ##
============================================
+ Coverage     49.56%   50.33%   +0.76%
- Complexity      631      645      +14
============================================
  Files            87       87
  Lines          5201     5211      +10
  Branches        664      666       +2
============================================
+ Hits           2578     2623      +45
+ Misses         2362     2339      -23
+ Partials        261      249      -12

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


Null pointer dereference in mrb_obj_iv_get

The following input demonstrates a crash, when supplied to mirb:
def a
end
lambda{ v{}.call
rescue
module Y
z = z
end

It seems the problem was caused by abed375ea669c2b8863cd0b253e6b9508b4950bb, and looks related to #3676.
ASAN report:
ASAN:DEADLYSIGNAL
=================================================================
==61072==ERROR: AddressSanitizer: SEGV on unknown address 0x000000000018 (pc 0x00010be60567 bp 0x7fff53edf9f0 sp 0x7fff53edf880 T0)
==61072==The signal is caused by a READ memory access.
==61072==Hint: address points to the zero page.
    #0 0x10be60566 in mrb_obj_iv_get variable.c:473
    #1 0x10bd5edac in mrb_class_path class.c:1595
    #2 0x10bd605ef in mrb_class_name class.c:1644
    #3 0x10bd610d2 in mrb_obj_classname class.c:1656
    #4 0x10bdfc23a in mrb_any_to_s object.c:442
    #5 0x10bd20750 in main mirb.c:562
    #6 0x7fffe5638234 in start (libdyld.dylib:x86_64+0x5234)

==61072==Register values:
rax = 0x0000000000000018  rbx = 0x00007fff53edf8e0  rcx = 0x000061400000a440  rdx = 0x00001fffea7dbf10
rdi = 0x00007fff53edf8a0  rsi = 0x0000000000000000  rbp = 0x00007fff53edf9f0  rsp = 0x00007fff53edf880
 r8 = 0x00007fff53edf8c0   r9 = 0x00000000000002a2  r10 = 0x0000000000000018  r11 = 0x0000100000000003
r12 = 0x00007fff53edfa40  r13 = 0x00007fff53edfa60  r14 = 0x0000100000000000  r15 = 0x00007fff53edfa20
AddressSanitizer can not provide additional info.
SUMMARY: AddressSanitizer: SEGV variable.c:473 in mrb_obj_iv_get
==61072==ABORTING
Abort trap: 6

This issue was reported by https://hackerone.com/ahihi

It's fixed along with #3676


Add main menu

Add main menu

resolved by 7c1c187 which adds a splash screen with "press Space to start" text


KAFKA-8037: Check deserialization at global state store restoration

This change aims to prevent records that cannot be deserialized from going into a global state store during restore. Currently such records are filtered during normal operations but will be processed during restore and will cause an exception when trying to access the value in the store.
The change copies the logic from the GlobalStateUpdateTask and builds a list of deserializers to use during restoration.
The GlobalStateManagerImplTest was extended to cover the case that a StreamsException is expected when a record is processed that can't be deserialized with the default LogAndFailExceptionHandler.
GlobalStateManagerImplLogAndContinueTest was added and contains one new test which uses the LogAndContinueExceptionHandler and verifies that a record can be ignored during restoration.
Copying all tests is not ideal, but I found no easy way to override the DefaultExceptionHandler just for the one case.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mjsax @vvcephei @guozhangwang May I kindly ask for a short review of my proposed changes for the global state stores?
If they are ok I can start adapting them for the local state stores too.

ping @guozhangwang @mjsax @vvcephei @ableegoldman

GlobalStateManagerImplLogAndContinueTest was added and contains one new test which uses the LogAndContinueExceptionHandler and verifies that a record can be ignored during restoration.
Copying all tests is not ideal, but I found no easy way to override the DefaultExceptionHandler just for the one case.

I'm ok with this approach, we'll see what others have to say though.

Hi @pkleindl ,
Sorry for the long delay. I'm coming up to speed on this, and I have a high-level question, and please bear in mind that I don't know much about global stores to begin with...
It seems like global stores can only be populated from input topics, is that right? In that case, they probably never need a changelog and just instead always populate and restore from the input. If that's true, then why do global stores need a concept of "restore" at all? It seems at first brush like populate and restore are functionally equivalent.
Maybe the restore path is more efficient for some reason, like it polls in larger batches or something?
I'm asking all this because it seems like there's some risk and complexity involved in maintaining two separate paths to populate the store. This is already the case with our pre-existing restore path, which is the ultimate cause of the bug you're fixing.
Thanks! I had a couple of small remarks, which I'll also send, but I wanted to ask for clarification on the big picture.

@vvcephei @bbejeck Sorry for asking directly, but I think I messed up when updating my branch.
I didn't mean to include all other changed from trunk here.
Do you know if there is a simple way to remove those changes? Or should I rather start over again in a new PR?

@pkleindl , no worries. You can use "rebase" to get rid of those commits. Just look in the commit history to find the commit that this whole branch is based on, and run git rebase -i ${THAT_COMMIT}.  Afterwards, you do a force-push to your PR branch.
Tread with caution, obviously. You can create a new branch or tag at the head of this branch before you do the rebase, so you can "undo" if things go sideways, git checkout KAFKA-8037; git checkout -b KAFKA-8037-backup; git checkout KAFKA-8037; git rebase -i .... Then, if the rebase messes everything up, you just git checkout KAFKA-8037-backup; git branch -D KAFKA-8037; git checkout -b KAFKA-8037 to restore.

Moving this back into the main thread:

It seems like global stores can only be populated from input topics, is that right? In that case, they probably never need a changelog and just instead always populate and restore from the input. If that's true, then why do global stores need a concept of "restore" at all? It seems at first brush like populate and restore are functionally equivalent.

This problem is not specific to global stores, I just did this part of the implementation first and would extend it to the local stores afterwards.
The point is that any unserializable record in the source topic can be handled during normal operations but will be loaded in the store upon restore, potentially causing a problem when accessing the record from the store.
And without this kind of protection there is not a lot you can do except try to clean up the topic itself or try to catch deserialization problems anywhere the store is accessed.

So, for local stores, there are two modes... In general, the store has a changelog, and we restore from the changelog. Since we only write to the changelog after record processing, we know it only contains valid data. In special cases, as an optimization, we don't create a changelog and instead restore from the input topic. In that latter case, it is indeed the same problem as what you're addressing here.
My question is, in the cases where we're "restoring" from the input topic, rather than from the changelog, why bother with the "restore" code path at all, why not just process the input topic normally? The RecordQueue would ensure that we read all the older records from this store's input before processing newer records from other topics anyway. This effectively means that we'd "restore" the state of the store in question before trying to use it for joins, etc., which I think is what really matters in the end.

@vvcephei Thanks for the help, will try the rebase tomorrow
As for your suggestion, the idea sounds very good to me and more elegant than my approach.
Some possible restrictions/drawbacks:

If this case is handled by normal processing, wouldn't this be quite limited in speed when the normal consumer is used?
This would possibly limit the performance for this kind of "restore" as the restore consumer can be tuned differently than the normal consumer
I am not sure if this is equally applicable for the global stores because their restore is different from the local ones and they have to be completely restored before any other processing (including restoration of local stores) can start

Should be take this back to the mailing list or try to get some opinions here?
We aren't really talking about implementation "details" at the moment ;-)

Hey @pkleindl ,
Sorry for the silence. I'm not sure if I saw that mailing list message; just re-starting it here.
I do think that performance is a concern for "restoring" a store from its input instead of from a changelog, but it seems like this could have a pretty straightforward solution of just configuring the consumer and store for bulk during the initial "bulk" phase. It's possible, but not clear to be whether also skipping some of the processing/handling logic also meaningfully affects performance or not.

Just to chime in here, I think we actually don't have a separate restore consumer for global stores -- this makes sense for the case of local stores, where the "normal" consumers use group subscription to subscribe to source topics while the restore consumers are assigned directly to specific changelog topics.
However for global stores we are restoring and "processing" from the same input topic, so the only real difference is the code path. Which as John has pointed out should not be meaningfully different between the "restore" and "initial loading" cases.

@vvcephei  @ableegoldman
Thanks for your feedback. It is correct that the global stores have their own logic now, although it would be very favorable if this could be changed to allow parallel restore/initialization at least.
Maybe this problem can be reduced to: Any store based on a source topic can run into this problem during initialization if the records are sent straight to the store without deserialization.
I am not sure if I understand the above comment correctly, but using the normale consumer instead of the restore consumer would run into other problems when building up the stores, right?
Unless everyone is using Avro I think more people will run into this kind of problem which is pretty nasty and workarounds are not really easy.
But I can give it another try if we find solution which is safe and fast (enough).

Thanks, @pkleindl !
I do agree it's a nasty problem, and to be honest, I've always been a little hesitant about using the input instead of a changelog topic. It's definitely a disk/network saver, but it also seems like a big risk to me.
Looking forward to see what you come up with!

Sorry for reviving old PR, but I have one high-level question about this, sorry if i am missing something, but if the aim is to stop populating records that can't be deserialized into the global store, why can't we have something like this in the GlobalStateManagerImpl#restoreState
            while (offset < highWatermark) {
                try {
                    final ConsumerRecords<byte[], byte[]> records = globalConsumer.poll(pollTime);
                    final List<ConsumerRecord<byte[], byte[]>> restoreRecords = new ArrayList<>();
                    for (final ConsumerRecord<byte[], byte[]> record : records.records(topicPartition)) {
                        if (record.key() != null) {
                            final String topic = topicPartition.topic();

                            final SourceNode source = topology.source(topic);
                            // try to deserialize key and value, if not use provided
                            // deserelization exception handling strategy to either fail topology or continue
                            final Object key = source.deserializeKey(topic, record.headers(), record.key());
                            final Object value = source.deserializeValue(topic, record.headers(), record.value());

                            restoreRecords.add(recordConverter.convert(record));

                        }
                    }
                   ...
                } catch (final InvalidOffsetException recoverableException) {
                   ...
                }
            }

During restoration we can try to deserialize key and value with the provided deserializers, if not successful we can use provided DeserializationExceptionHandler to either continue or fail altogether?

Hi, that was my basic idea, see https://github.com/apache/kafka/pull/6398/commits/8ea4d944bb5389d2014e4f3b4a617255a7179843#diff-e870eab2dedeee93442ba87dc12609a4
But I did not have the time to continue working on it in the last months, so the progress stalled.
This was meant as a basis to use the process during restoration too, but all this would need a bit of discussion because of possible performance impacts.

@lkokhreidze I looked at another JIRA ticket related to your observation and left some comments there (I think they are relevant :P): https://issues.apache.org/jira/browse/KAFKA-7663 LMK what do you think.

I'll close this PR as the differences are too big after this time.
A new PR will be opened


Rewrite import paths from mholt/archiver -> airplanedev/archiver

This appears to break Renovate upstream

Sorry, meant to open this on our fork. Nevermind us!


nonce empty on client

Getting following error on browser console
Warning: Prop nonce did not match. Server: "0d1781b142423cac737af4b3e38a32ea" Client: ""
Getting started with the following instructions
git clone https://github.com/epicweb-dev/epic-stack.git
cd epic-stack
npm install
cp .env.example .env
npm run setup
Tested with firefox and chromium. Both displayed same error.
Did i missed something? Thx for response.

Hi @curler7,
I can't reproduce this on the deployed version of the stack: https://www.epicstack.dev


Thread panics when running multiple waSCC demos on the same port

I ran into this problem when I tried running 2 waSCC demos at once. Since two examples are running on the same port (8080), the thread panicked and failed. After identifying the issue, I tried recreating the problem and one of the two scenarios would occur.

Use kubectl apply to create 2 pods from 2 examples. Only the 1st pod is running and the 2nd pod failed. Deleting the 2nd pod and running a curl localhost:8080 on the 1st pod produces Failed to handle request.
Run one demo following the instructions, then start running a second demo. curl localhost:8080won't produce any error, it will just show the output of the first demo. Deleting the pod created in the first demo, run curl localhost:8080 again, it says curl: (7) Failed to connect to localhost port 8080: Connection refused.  In the just run-wascc terminal, this log can be found: wascc actor executing ' panicked at 'called Result::unwrap() on an Err value: Os { code: 48, kind: AddrInUse, message: "Address already in use" }'.

Currently, the only way to avoid this problem is to run one demo and delete the pod created,  before running another demo.
Looking for suggestions to be able to run multiple examples concurrently on different ports automatically. @thomastaylor312 had mentioned in docker, an arbitrary port can be mapped to a port in the container. Kubernetes uses automatic mapping behind the scenes. It might be possible to take advantage of k8s and use it here.

When waSCC binds an HTTP server capability provider to an actor, you can supply a PORT configuration variable. If this isn't supplied, then the HTTP server binding defaults to port 8080.
If krustlet were to choose an arbitrary port number or assign port numbers with a monotonically increasing port number, this would prevent the panic due to trying to bind to an in-use port.

Yeah, for now the workaround is probably to change the pod definition to specify a different port. I think that we should make the waSCC provider use the containerPorts part of Pod spec to specify the port (instead of the env key) and then map it to a config underneath the hood. Combined with @autodidaddict's idea, I think it would be a good solution

Ok, I did some more thinking here. So I think the containerPort key doesn't mean the same thing in a wasm module context. There is no port to map to in the module, only on the host it is running on. So I think the behavior should be like this:

If containerPort is specified and hostPort is not, then automatically assign a port number
If hostPort is specified (whether or not containerPort is specified), then try to assign that port number. If it is taken, the pod will error

I'll work on figuring out how to tell kubernetes which port you are listening on. The data is in a Service but I am not sure what sets that. I think it is the kubelet, but I will double check

Ok so I looked more in to networking. I think that is probably a much bigger thing to tackle and is out of scope of this issue as it involves figuring out how to implement IP per pod networking. So for now, I would just figure out the automatic port mapping and then we can just rely on NodePort services as a workaround


only log flush/reap when it does something

Current coverage is 55.30% (diff: 66.66%)

Merging #70 into develop will decrease coverage by 0.20%

@@            develop        #70   diff @@
==========================================
  Files            23         23
  Lines          1962       1962
  Methods         249        249
  Messages          0          0
  Branches        240        242     +2
==========================================
- Hits           1089       1085     -4
- Misses          720        722     +2
- Partials        153        155     +2

Powered by Codecov. Last update 57c19b5...f6973aa


Inline equations will insert newlines

I'm not sure if I misconfigured something, but some short inline equations will insert a new line before and after the equation.
Example:

instead of

And

instead of

Adding a new line after  tag fixes the issue?
https://github.com/andrelimzs/github-latex-readme/commit/52de607c9fd4981fe8f67827b0c2a7a65a42f77b


Fix spelling of downloaded

Correcting of spelling error in a comment.  No functional change.

hey @ttk , thanks for contributing the fix, can you amend your commit to add a signature. (required for DCO)
more details here: https://github.com/open-policy-agent/opa-idea-plugin/pull/104/checks?check_run_id=2494239801

the failure:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'opa-idea-plugin'.
> Failed to notify project evaluation listener.                                                                                > Could not resolve all files for configuration ':detachedConfiguration3'.
      > Could not resolve com.jetbrains.intellij.idea:ideaIC:2021.1.1.
        Required by:
            project :
         > Could not resolve com.jetbrains.intellij.idea:ideaIC:2021.1.1.
            > Could not get resource 'https://www.jitpack.io/com/jetbrains/intellij/idea/ideaIC/2021.1.1/ideaIC-2021.1.1.pom'.
               > Could not GET 'https://www.jitpack.io/com/jetbrains/intellij/idea/ideaIC/2021.1.1/ideaIC-2021.1.1.pom'.
                  > Read timed out

is unrelated to the change in this PR.
jitpack.io is having issues currently.
we will wait till that resolves.


Typescript support

Hey there, thanks for the lib!
I noticed there were no typings for this lib. I am new to Typescript so i can't really create them myself. I have started but I am not sure how to import the proptypes and extend from TouchableOpacity
If anyone can help that would be great
import { Component } from "react";

export = Button;

type ButtonProps = {
    allowFontScaling?: boolean;
    accessibilityLabel?: string;
    containerStyle?: object;
    disabledContainerStyle?: object;
    disabled?: boolean;
    style?: object;
    styleDisabled?: object;
    childGroupStyle?: object;
}

/*~ Write your module's methods and properties in this class */
declare class Button<P = {}, S = {}> extends Component<ButtonProps, S> {
    constructor(props: object);
}

for temporary resolving you could follow this rules:

add typings.d.ts under the root folder with this code:

declare module 'react-native-button';

Hey, I am planning to add a PR for the typescript support.
Any prort information is appreaciated.

Any update on this?


visible probability prediction  of keypoints

Hi Xingyu,
I want to predict the visibility score of 21 keypoints on image space based on CMR or MobRecon. However I don't get a clean idea about how to creating ground truth label of keypoints visibility on FreiHAND dataset. What I want to do is when some keypoints are occluded by others (like inner keypoints in wist pose) or out of image, then set its score to a low value.  Could you give any hints on this? Thanks in advance.

Hi, Yunfeng. I have no good idea to automatically annotate visibility. It feels like the visibility, or confidence, can be estimated with likehood method. For each joint landmark, we can estimate its distribution with mean and variance.  Then, use negative log-likelihood to train. That is, put GT into the distribution, and maximize the probability. In this way, we do not need extra annotation, and the predicted variance could reflect the confidence/visibility.

Hi Xingyu, thanks for your solution. I don't know much about probability-related deep learning models thus I will do further search about it. Thanks again for your generous help


dApp Lifecycle enhancement

Area
[x] Plutus Foundation Related to the GHC plugin, Haskell-to-Plutus compiler, on-chain code
[x] Plutus Application Framework Related to the Plutus application backend (PAB), emulator, Plutus libraries
[] Marlowe Related to Marlowe
[x] Other Any other topic (Playgrounds, etc.)
Describe the feature you'd like
Our ideal dApp workflow is:

Write haskell "program" in separate files
Invoke haskell-to-plutus compiler
Deploy result to local Cardano or testnet
Invoke program transactions in testing

We can not find any documentation that supports this so: Is this being considered? If so, what ETA should we think about?
Regards
@FrankC01
@arthurgreef

Yep, that's possible. Please checkout https://github.com/input-output-hk/Alonzo-testnet/blob/main/Alonzo-exercises/alonzo-purple/3_Alonzo-purple-exercise-3.md how to use cardano-cli.


Ability to create additional color codes

Bug, feature request, or proposal:
Feature request/proposal
What is the expected behavior?
Providing more than 3 color codes.
What is the current behavior?
There are only 3 color codes:
Primary
Accent
Warn
What are the steps to reproduce?
N/A
What is the use-case or motivation for changing an existing behavior?
Other UI frameworks have more than 3 color codes available, but they are also limited to a predefined number. Being able to define custom color codes would serve the ease of use (and possibly some more superiority over other frameworks).
As an additional motivation, currently only way to achieve this functionality looks like changing the mixins in the _theming.scss file, which wouldn't be carried over between releases. So any customization would go in wains after updating the package.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
All
Is there anything else we should know?
While defining the mixins the scss list type might be helpful as mentioned in the below link:
https://gist.github.com/jareware/4738651#5-the-list-data-type

@alirizadikici I've checked out your stackblitz to see what you were getting at, but I'm unable to figure out what you exactly did that's supposed to help in being able to differently style certain elements according to the material template?

@alirizadikici I've checked out your stackblitz to see what you were getting at, but I'm unable to figure out what you exactly did that's supposed to help in being able to differently style certain elements according to the material template?

I think he is redefining the mat.light-theme function to return more colors. But then you need to go over all the components (or at least the ones you need) and refactor to also use the new colors

@shiamalon you actually got the idea and the pain point.
When using Bootstrap 4 it is very easy to create the color codes and they are available all around, on all the components, backgrounds, borders etc. This is the main reason for the feature request.


Can't get the last(newest) recorder from series

I want to get last recorder from series use “ select * from cpu limit 1 ”like in influxdb 0.8.
But in influxdb 9.0, it seems save the data as AES, and can't change the order.
Is there any way to realize?
"select last(value) from cpu"  just get a value, not all.

This need is covered by #1577 and/or #2022.


V2 Doc Overhaul: Part 1 - Intro

to: @ryandeivert and @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: medium
Background
As we get ready to release StreamAlert V2 🙌, the documentation needs some serious updating. Each PR will focus on a specific section; this one focuses on the landing page and the introduction.
Changes

Landing Page (index.html)

Replaced the high-level schematic with one that is more accurate and easier to read
Rewrote the list of features (which replaces the overview and benefits pages)

Architecture (new page)

Added a more complete architecture diagram and a step-by-step overview of the StreamAlert data lifecycle

Getting Started

Migrated details from the requirements page
Added steps for testing StreamAlert with an example after it is deployed. The goal is that the Getting Started page is a 1-page tutorial for a brand-new user to go from nothing to seeing an alert generated by their StreamAlert deployment.

Removed the roadmap page, which just points to GitHub milestones. We already link to GitHub elsewhere for issues and source code, I don't think this needs its own page of documentation.
Disable Kinesis and CloudTrail by default. These are expensive to run, and generate lots of error messages in CloudWatch logs in a fresh deployment that may make it harder to understand what is happening for first-time users.

Testing

Built the documentation locally: sphinx-build -W docs/source docs/build (this is also how I recommend the reviewers look at it)
I tested the steps in "Getting Started" from start to finish with a new deployment

Coverage remained the same at 97.392% when pulling 482a859ac3cb798a1487011e7f9977973a681ea8 on austin-docs into fae89fb2df439a8b63e781d61362d54303485319 on master.

this is great - thank you for leading this charge @austinbyers !!! I'll get your branch checked and and build locally to review

@chunyong-lin @ryandeivert awesome feedback, thanks for the thorough review! I've addressed it, but since there were lots of changes and comments, I'll wait until at least one of you approves again.
Note that I removed even more unnecessary stuff from the default cluster, stripping it down to almost the bare minimum (cloudwatch metrics, rule processor)

@austinbyers back to the versioning thing - this should work and will ensure we're in constant version parity with docs/code:
in docs/source/conf.py:
# The short X.Y version.
version = re.search(
    r"^__version__ = ['\"]([^'\"]*)['\"]",
    open('../../stream_alert/__init__.py', 'r').read(),
    re.MULTILINE
).group(1)

# The full version, including alpha/beta/rc tags.
release = version

great changes, still LGTM @austinbyers :)


Can you be more specific with the list of domains to whitelist for NuGet connections to work?

In the Nuget Connections section, Microsoft includes:
https://azurewebsites.net
Our security colleagues noted that there is all sorts of random user created content hosted on azurewebsites.net and they're not comfortable whitelisting a domain which has such wide use cases.  At the moment we're attempting to workaround this using suboptimal hacks.  Can you track down a more specific subnet of https://azurewebsites.net that we can work with our security colleagues to whitelist?
Thank you!
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dc5a4cfa-aa84-9f0f-55f4-71ec4861c070
Version Independent ID: d621620a-c924-03b1-8b43-f99c33e58bb8
Content: Add IP addresses and URLs to allow list - Azure DevOps
Content Source: docs/organizations/security/allow-list-ip-url.md
Product: devops
Technology: devops-security
GitHub Login: @chcomley
Microsoft Alias: ChComley

Hi @MarkIannucci, Thanks for reaching out. I'm following up on this with the PM and will be in touch. Thanks!

Hi @MarkIannucci, I wasn't able to track down a more specific subnet of https://azurewebsites.net. I will be publishing an updated version of this article, which will include  more IPs and links to more information. This issue will close automatically, once the pull request is completed. I highly recommend reaching out to the Developer Community for assistance. I quickly searched current issues and didn't see anything relating to this subject. Please go to https://developercommunity.visualstudio.com/spaces/21/index.html. Select Report a problem, and then Azure DevOps.
If you have any further questions about the article itself, please don't hesitate to @mention me. Thank you!


Feature: Loading when scroll to top

Thanks for you library bringing convenience to my job. I encounter a situation that need support of loading from top of scroll element.As lack of feature in this library, I can only implement it by my own,really time wasting.
I think this is quite a useful and necessary functionality.I just hope this library can be better and add it. Thanks again for your effort.

Glad you like this plugin.
There has the seem issue #8 already. I thought a lot after that, but I cannot imagine any situation which need an infinite scroll up plugin, because of such the plugin more like pull-refresh plugin rather than infinite scroll plugin. Perhaps I missed some situations, what do you think about it?

+1
About my situation, I have to load more pages of a book where the user can go directly to the page 10 (for example) in this case I have to load the page 11 if the user scroll down and the page 9 if the user scroll top...
Thanks for this component :-)

@julianfox Thanks for your comment, I understood your situation and it does exist, but in my mind, we need to pull down or pull up to load another page rather than scroll down or scroll up directly when using some apps, is that right?

I will close this issue until we found some situations that really needed the feature in this component, thanks everyone :D

Scroll up infinitely is required in messenger/chat apps, where you scroll up to load old messages.

@znck well, I think you're right, the iMessage App of iOS is like that, it will load old messages automatically when I scroll up.
I will reopen this issue and add this feature in the todo list, thank you!

Thanks for the great component! I am also desperately looking a way to reverse distance so that infinite-loading would be triggered if clients scroll is over a certain amount of pixels from the bottom or under the certain amount of pixels from the top. I am developing a chat application and old chat messages are loaded when user hits the top of the chat area. So there is a need for this feature :)

:+1: we could also use this feature!

Hello everyone, I released a new version v1.3.0 to support two direction load feature, the document is bottom of this page, please try it with npm, if it's ok, I will add it to the v2.0.0 version.
Sorry, this feature was late for too long, thank you a lot : )

@PeachScript Does that mean it's only compatible with Vue 1.0? How hard would it be to backport (well, forward-port, I guess) the changes?

@charliewolf Right, I plan to support Vue.js2.0 after this feature being stable, and it's very simple, see file changes, if you need, I can add it to v2.0.0 recently.

@PeachScript Got it--well, I can only speak for myself obviously, but the app I could test this with is using Vue2.

@charliewolf Hello, I've added this feature into the latest stable version v2.0.0, you can test it in your app now.

Thank you so much for this new feature! Works like a charm!

@vlahde You're welcome : )


read/write -time_field

connects to #25
Now the point that the adapter will use as a timestamp for documents in
ES is configurable by the time_field option. Juttle points still have
"time", this just determines the name of the field that the adapter will
put pt.time in when writing and put into pt.time when reading.
@demmer @VladVega @go-oleg @dmehra @rlgomes @henridf

is it time_field or -timeField ? https://github.com/juttle/juttle-sql-adapter-common/blob/master/lib/read.js#L69

I agree with @henridf that we should be consistent and I think the camelCase timeField would be more in line with other adapters including the builtins:
https://github.com/juttle/juttle/blob/master/lib/adapters/file/read.js#L30
https://github.com/juttle/juttle/blob/master/lib/adapters/http/read.js#L45
The same should also apply to -index_prefix.

I thought there was a big bikeshed and underscores won over camelCase? Or was it the opposite?

I guarantee it was one of the two.
On Tue, Jan 5, 2016 at 2:49 PM, Dave<EMAIL_ADDRESS>wrote:

I thought there was a big bikeshed and underscores won over camelCase? Or
was it the opposite?
—
Reply to this email directly or view it on GitHub
https://github.com/juttle/juttle-elastic-adapter/pull/35#issuecomment-169159430
.

--
Michael Demmer
Jut - VP Engineering
<EMAIL_ADDRESS>

The camels are in charge of this one now. I'll change the README in https://github.com/juttle/juttle-elastic-adapter/pull/30.

The code looks good to me. One additional test would be nice.


"Cannot read property 'fields' of undefined" while using multiple upload with onError callback

Promblem/Feature Request Summary
"Cannot read property 'fields' of undefined" while using multiple upload with onError callback.
And I found line 84 to 109 of source code 'index.js' cause the problem.
Environment Information

koa-body: 4.0.4
Koa: 2.6.1
Node.js: 8.12.0

Current Behavior
TypeError: Cannot read property 'fields' of undefined
Steps to Reproduce

Part of the code

...
router.post('/', koaBody({
    multipart: true,
    formidable: {
        uploadDir: config.uploadDir,
        maxFileSize: 1 * 1024 * 1024,
        keepExtensions: false,
        multiples: false,
    },
    onError: (err, ctx) => {
        console.log('Failed');
        ctx.body = "Failed";
    },
}), async (ctx, next) => {
    const files = ctx.request.files || {};
    for (let file in files) {
        console.log('file.name: ', file.name);
    }
    ctx.body = "Done";
});

Use curl to upload a file with size larger than 1M.

Expected Behavior
curl got 'failed' as response.
Possible Solution
Line 84 to 109 of source code 'index.js'
    ...
    return bodyPromise.catch(function(parsingError) {
     ...
      return next();
    })
    .then(function(body) {
          ...
          ctx.request.body = body.fields;
          ctx.request.files = body.files;
          ...
      }
      ...
    })
    ...

The 'then' callback may be called with undefined as parameter 'body', because bodyPromise.catch's callback function return next(), while next here I return is nothing.
Should I change catch's callback function return next() to return next() || {}
or just give parameter 'body' a default value like ... function(body = {}) { ... and change the order of then and catch ？

I don't think we should change the codebase to return next() || {} because next() is how the next middleware is called in Koa. I'm not even sure what would happen if you returned an object instead of a promise.
I see two options:
1. Custom error handler
The downside with this approach is that you have to figure out how to differentiate a koa-body error (actually formidable under the hood) VS other errors.
// other app code above here

// error handline middleware
app.use(async function (ctx, next) {
  console.log('loaded error handling middleware');
  try {
    await next();
  } catch (err) {
    // this is where you handle - Error: maxFileSize exceeded, received 1113965 bytes of file data
    // I would map this to a 4xx error code
    console.log('error', err);
  }
});
app.use(router.routes());
app.listen(port);

2. Custom error handler and throw in .onError
Add the same error handling middleware above but change your onError handler to throw. Ideally, you would re-cast this error as something you can recognize in the error handling middleware from step 1. I use Boom to help with this.
    onError: (err, ctx) => {
      throw err; // optionally recast as something we can recognize in the error middleware
    },


.cntnt empty and meta tags missing when content is empty or short

Hello Pascal,
when crawling pages in some occasion the .cntnt files are empty and the meta tags are not getting extracted.
To reproduce the behavior please have a look the following files

https://storage.googleapis.com/sascha-issue-reproduction/emptyBody.html
Contains an empty body, which obvious leads to an empty .cntnt file. But the existing meta tag is not extracted.

https://storage.googleapis.com/sascha-issue-reproduction/smallBody.html
Contains a small amount of text in the body, but still the .cntnt file is empty and the meta tag is still not extracted.

https://storage.googleapis.com/sascha-issue-reproduction/largeBody.html
After adding more content to the body the content and the meta tag is extracted properly.

It seems that the parsing / extracting is somehow related to the length of the content.
Any suggestions / known issues?
used version: 2.8.1
Best regards
Sascha

Hello Sacha,
I just tried with the latest snapshot release and it works fine, as you can tell by this attachment:
fs-committer-files.zip
So I encourage you to try with the latest snapshot, or share your config in case there is something else going on I missed.

Hello Pascal,
thank you for your support, I can confirm the behavior is not related to Norconex itself.
Rather the issue is related to a Committer Plugin which causes some dependency issue within the lib folder.
Best regards
Sascha


Add build number generator action

https://github.com/zyborg/gh-action-buildnum

@ebekker your changes have been merged, thanks for your contribution 👍


Have the "subject:wikipedia" tag display the same way as "subject:wikidata"

Description
Currently, only subject:wikidata pop when the tag in inputted. I recommend doing the same for subject:wikipedia.
I attached an image of how only subject:wikidata is being displayed.
Screenshots

I've moved this request to id-tagging-schema at https://github.com/openstreetmap/id-tagging-schema/issues/947
I'm closing this issue here, please track and comment on it over there. Thanks!


Where are the sources and project files of the GoboLinux mascot and documented complete mascot definition?

Here are some relevant resources:

icon_mascot.svg at the https://github.com/gobolinux/gobolinux.org repo
que.png at the https://github.com/gobolinux/Installer repo (an older version of the mascot, apparently)
From gobolinux.org/faq.html:

Who is the mascot? Is that Fibo?
No, the mascot is called Que. He's a penguin wearing Fibo's clothes.

From gobolinux.org/releases.html:

GoboLinux 011 was launched to coincide with the V International Forum on Free Software,
which take place in June 2004 in Porto Alegre, Brazil. Hisham designed a T-shirt to be
sold at the event. He wanted a design that reminisced the World War II propaganda posters
(one of André's subjects of interest). The result was a vectorized mascot in a "We want
you" pose, and the line "GoboLinux - join the revolution". Rafael adapted it into the
release icon.
For GoboLinux 012, the vectorized Que is featured again. André asked for
a suggestion for the release image, and Hisham said "I don't know, something that
took a long time but now is complete", in an allusion to the long delays in the
development of that particular release. André took matters with his own hands and
did the cool sith mascot. Nothing could be more apropriate, since GoboLinux 012
was released around the time Episode III came out, and indeed the Star Wars saga
took a long time to be done but was now complete.
Version 013 was almost released without an image, as it was late in the night while we tried to
match our self-imposed deadline and everyone was so tired. Also, the site was redesigned for this
release, like on 010, so we even had a suitable excuse. ;) But since we worked so hard to make
this release as complete as possible, such detail could not be left behind. So Hisham made a
sketch of the mascot in the style of the new icons of the website.

I think @waldyrious has answered the question? In the future consider asking in our forums :)
Closing for now, but feel free to reopen if the question has not been answered to your satisfaction.


Redux saga with proper hot reload

[x] I have searched the issues of this repository and believe that this is not a duplicate.

I've been following the https://github.com/zeit/next.js/tree/master/examples/with-redux-saga example and noticed when I change the code in the saga files, the app is hot-reloaded, although not the saga files. The new saga code starts working after F5 reload.
Expected Behavior
When saga files changed, the changes are propagated with the rest of the app and replaced the existing saga code.
Current Behavior
client.js?3cc2700:199 [HMR] bundle rebuilding
client.js?3cc2700:207 [HMR] bundle rebuilt in 439ms
process-update.js?3cc2700:27 [HMR] Checking for updates on the server...
process-update.js?3cc2700:100 [HMR] Updated modules:
process-update.js?3cc2700:102 [HMR]  - ./saga.js
process-update.js?3cc2700:102 [HMR]  - ./store.js
process-update.js?3cc2700:102 [HMR]  - ./pages?entry
process-update.js?3cc2700:107 [HMR] App is up to date.

But saga code is still the former one, until next server render.
Steps to Reproduce (for bugs)

clone the with-redux-saga example
Run with next
Add some changes, e.g. console.log at the saga beginning https://github.com/zeit/next.js/blob/master/examples/with-redux-saga/saga.js#L31
save, watch the code replace, but not the saga.

Is this even possible? The saga file is executed after change, but the saga code in the generators does not.
Your Environment

Tech
Version

next
v3.2.1

node
v5.12.0

OS
Windows 10 x64

browser
Google Chrome, Version 60.0.3112.113 (Official Build) (64-bit)

It probably has to module hot accept, same as redux requires.


Survey on globally distributed software engineering

Hey there.
We are doing a survey on globally distributed software engineering focused on open source projects. If you have some spare time we would really appreciate your response. The form takes about 10 minutes. Thanks in advance!
Survey can be found here

Hi @AsWali , I'm closing this since you've never been a contributor, and are here solely to advertise your survey. I recommend that you reach out to repo maintainers prior to posting surveys like this, otherwise it's an abuse of the issues board.


Calendar

I would love to start renting a lot of stuff I don't use often (tools, Sport goods: surfboard, MTB, skies)
Thus, I would recommend to add a calendar, so one could start renting stuff on openbazaar.
One idea to properly  incorporate renting, would be to sell the thing you rent, with a buy offer (buyback). So the person seeking the rent has the assurance that, they will get ride of the good after using it, for a certain rent fee. Even better you can have multiple buyback options, doesn't not need to be the original owner, making it a market.
For the time being, one needs to be able to afford the stuff one rents, but in future decentralized insurance markets, could cover the capital needed for a rent.
Even cars or real estate could be rented ;)
Please share you thought about this feature and tell me if it's in the scope of openbazaar?
This may be marked as Feature Request/Enhancement.

That is certainly something that would be interesting to explore in the future, but we don't have any plans to add rentals in the near to mid-term future.


Camunda stops working if loosing connection with Keycloak

If for some reason the connection with keycloak fails, it looks to stay in the broken state and only a restart of Camunda fixes this.
Is there a way around this? auto recovery / reconnect retries?

@Fireblade954:
Typically - in case of errors with authentication etc. - it should automatically reconnect.
Can you tell me a bit more about your setup? Keycloak version, whether you'r using Spring Boot | Camunda Run | ... etc.?
Do you have any log file?

I just tried to reproduce this and indeed its picking up again after starting keycloak again. We just update everything to the latest version so maybe that also fixed this issue. Ill create a new issue if it occurs again.

I just ran into a situation where Camunda crashes. When keycloak is not running and u start camunda it will crash / stop running the docker container. (Camunda Run 7.20)
So when camunda is already started it indeed tries to reconnect  if a keycloak connection fails.
But if keycloak is nog running and Camunda is started it will crash immediately without retries.
Have a workaround for now by using the InitContains option for kubernetes. Which will wait for Keycloak to responde before spinning up Camunda to prevent it crashing.
 *   initContainers:       - args:         - while [ $(curl -sw '%{http_code}' http://keycloak/health/ready -o /dev/null)           -ne 200 ]; do sleep 5; echo 'Waiting for the keycloak...'; done         command:         - /bin/sh         - -c         image: quay.io/curl/curl:latest         name: wait-for-keycloak

I just ran into a situation where Camunda crashes. When keycloak is not running and u start camunda it will crash / stop running the docker container. (Camunda Run 7.20)

This is indeed true. You need a running Keycloak Container in case you want to start Camunda with the Identity Provider plugin. At startup Camunda checks / creates authorizations for the admin user or admin group. Sorry, but this is part of the design of Camunda 7.
In case you want to avoid this dependency, you have to stop using the Keycloak Identity Provider plugin and take care of SSO with Keycloak using it for Cockpit / Tasklist access only. This should be possible, but you won't have users and groups within Camunda itself. Depends on your use case whether this fits or not.
Hope this helps.

i understand its not the issue that i don't want to use it, but it can happen that the keycloak container hasn't started yet and that will make Camunda crash / stop. But with my workaround above it works fine now thanks for the explanation.


Use consistent terminology when talking about pluralisation

What
Prompted by Ollie's comment on the CharacterCount macro options
Have consistent words for refering to:

the "zero", "one", "two", "few", "many", "other" keys
the translated messages attached to these when providing translations
the object associating the keys to the message

We don't want to invent new terms, just get some consistency in the codebase so we always refer to those in the same way.
There are a few specifications/documents that deal with the topic and will likely have terms we can reuse:

TC39's specification of Intl.PluralRules' select method, used to transform a number into the one,two, few... string: https://tc39.es/ecma402/#sec-pluralruleselect. That's the JavaScript API shipped in browser and node to help with localisation and pick the right plural forms. And its documentation on MDN
The Unicode specification about documentation: https://cldr.unicode.org/index/cldr-spec/plural-rules
Mozilla's documentation of translation best practices: https://mozilla-l10n.github.io/documentation/localization/dev_best_practices.html#use-proper-plural-forms
Libraries using a similar system referenced in @querkmachine's initial document about internationalisation:

banana-i18n
i18next
Fluent
Lingui.JS

Why
That's help us be clear about what exactly our code and documentation refers to
Who needs to work on this
Content designers, Developers
Who needs to review this
Content designers, Developers
Done when

[ ] Terms have been agreed upon
[ ] Code and docs have been updated to be consistent with the chosen terms

We're going to close this and wait to see if we see issues with people not understanding our documentation relating to pluralisation.


Fix: Deamon.js

I added the proper logging of the host, because it is very confusing to always have it log localhost even when you have assigned it another host.

This project seems kinda dead so I'll be closing this PR.


Helidon DB Client - Add JDBC Batch support

For larger inserts it is important to avoid network roundtrips per row inserted.
This can either be done generating new sql strings per row count or using the JDBC PreparedStatement.addBatch() feature and relying on the driver to do that in the background.
I would propose adding API to expose this JDBC feature or something equivalent.
Environment Details

Helidon Version: 2.4.1
Helidon SE

Some brainstorming to support batch.

Batch of unrelated statements

        dbClient().batch(b -> b
                .createInsert("INSERT INTO pokemons VALUES (?,?)")
                        .addParam("name1").addParam("type").add(b)
                .createNamedUpdate("update1")
                        .addParam("name", "name1")
                        .addParam("type", "type1").execute())
                .thenAccept(count -> response.send("Modified: " + count + " values"))
                .exceptionally(throwable -> sendError(throwable, response));

Batch of same statement with multiple param settings

        dbClient().batch(b -> b
                .createInsertMulti("INSERT INTO pokemons VALUES (?,?)", getPokemons())
                .prepare((s, p) -> {
                    s.addParam(p.getName());
                    s.addParam(p.getType());
                })
                .execute())
                .thenAccept(count -> response.send("Modified: " + count + " values"))
                .exceptionally(throwable -> sendError(throwable, response));

Some initial thoughts and design available here https://github.com/spericas/helidon/tree/dbclient-batch

Hey, thanks for exploring this improvement.
I moved the project that prompted this request away from helidon dbclient for now and can't really provide feedback on the development any more.
Wrangling the reactive model was just a tad too brittle and hard to debug, especially considering loom on the horizon.
The problem was real however and I would recommend keeping this issue around at low priority.
Thanks again for the quick responses and your work on this project.

Closing for now


just want to thank you for providing this nice tool and it works well!

just want to thank you for providing this nice tool and it works well!

😁


Add api version 2022-10-01

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month.
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month.
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[ ] I commit to follow the Breaking Change Policy of "no breaking changes"
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[ ] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[ ] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

approve breaking change because it has been approved in previous PRs: https://github.com/Azure/azure-rest-api-specs/pull/21625

/azp run

Hi, @utsrkr, For review efficiency consideration, when creating a new api version, it is required to place API specs of the base version in the first commit, and push new version updates into successive commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki. Or you could onboard API spec pipeline

Please follow this guide for ARM review. Thanks

@utsrkr - Your PR is labeled with ARMChangesRequested, which means it is not on ARM review's radar. Please address the previous reviewer's comment and resubmit your PR with the correct format.
Note also, you haven't filled out the submission form above. This is a hard requirement for reviewers and automation to help handle your PR correctly, so be sure to do so when you resubmit.
Be sure to follow these guidelines for preparing your PR for ARM review:
https://armwiki.azurewebsites.net/rp_onboarding/process/api_review.html and carefully read and complete the entire submission checklist above. Skipping any of these requirements may significantly slow down the review process or require reworking your PR.


The Migration record in Orchard_Framework_DataMigrationRecord should not be prefixed with the namespace where the migration class is created

With this prefix the records depends of the namespace and any change in namespace means an error in startup. Because the tables are not prefixed with the namespace but use the Part/Record class names, and migration try to run for the new namespace but the tables already exist and it crashes.
All this should be sync, the best would be to not use the migration namespace but a fixed name.
Changing this would consist in creating a property with same name as its namespace.

So if I understand correctly your argument against the current practice is that it prevents you from moving records across namespaces? You do know that records can only be in the Models namespace (and nothing else, not even a sub-namespace of this) in an extension, so this is only a case if you move a record from one extension to the other (including renaming the extension)?
But how would you add a custom prefix? Let's try an example: you create ModuleA and Record inside it. Now you need to choose a prefix for Record so it doesn't clash with anything else. What would you prefix it with if not the module name as an obvious choice? Because if you do that then it's the same issue again.

You don't understand correctly.

Then clarify?

Although I see Chris' point, I don't think it's compelling enough to change any of this.The convention where the namespace is used to build the table name is highly convenient in the majority of the cases. The only case where it might be inconvenient is when you are refactoring and the namespace changes, or even the class name, but having to manually try and specify a unique table name doesn't outweigh the benefit of that happening automatically in my opinion.
That being said, I believe you can already control the exact table name mapped to your entity classes by implementing ISessionConfigurationEvents, which allows precise control over table and column mapping. I did this on a few occasions with column name mapping, but never tried with table names. Sounds like it be worth a shot.

Thanks Sipke for understanding better this point. But not totally.
What I did has been to have a bad namespace name for the migration class and discovered it month later.
I tried to fix the namespace name but then entered in errors on start as explained.
I do not propose to change any table Name, simply to change the name in the migration record. Get rid of the namespace prefix and replace it by a  string property.
This way changing the namespace where is the migration class will have no impact.
Is it more clear to toi?
Envoyé à partir de mon sanyo phc 25

De : Sipke<EMAIL_ADDRESS>Envoyé : 19/01/2016 22:54
À<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Objet : Re: [Orchard] The Migration record in Orchard_Framework_DataMigrationRecord should not be prefixed with the namespace where the migration class is created (#6284)
Although I see Chris' point, I don't think it's compelling enough to change any of this.The convention where the namespace is used to build the table name is highly convenient in the majority of the cases. The only case where it might be inconvenient is when you are refactoring and the namespace changes, or even the class name, but having to manually try and specify a unique table name doesn't outweigh the benefit of that happening automatically in my opinion.
That being said, I believe you can already control the exact table name mapped to your entity classes by implementing ISessionConfigurationEvents, which allows precise control over table and column mapping. I did this on a few occasions with column name mapping, but never tried with table names. Sounds like it be worth a shot.

Reply to this email directly or view it on GitHub:
https://github.com/OrchardCMS/Orchard/issues/6284#issuecomment-173000166

Just to clarify, @CSurieux is talking about the value of the field which stores the name of the class that was run. Not the Table name.

Ideally, wouldn't it be a better solution to have the required API in migrations to fix existing data and move it to the new names, rather than a runtime override that fixes usage to point to the bad name?

Ah, yes I see now what you mean. I've run into that issue a few times myself. Easily fixed manually once you realize what's going on, but probably best to have the migration fix that data automatically once as per Bertrand's suggestion.

Seems more complicated than removing the link with the namespace where is the class.

De : Sipke<EMAIL_ADDRESS>Envoyé : 19/01/2016 23:18
À<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Objet : Re: [Orchard] The Migration record in Orchard_Framework_DataMigrationRecord should not be prefixed with the namespace where the migration class is created (#6284)
Ah, yes I see now what you mean. I've run into that issue a few times myself. Easily fixed manually once you realize what's going on, but probably best to have the migration fix that data automatically once as per Bertrand's suggestion.

Reply to this email directly or view it on GitHub:
https://github.com/OrchardCMS/Orchard/issues/6284#issuecomment-173006018

More complex to implement for sure, but also more correct.

Correct for you.
Envoyé à partir de mon sanyo phc 25

De : Bertrand Le<EMAIL_ADDRESS>Envoyé : 19/01/2016 23:38
À<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Objet : Re: [Orchard] The Migration record in Orchard_Framework_DataMigrationRecord should not be prefixed with the namespace where the migration class is created (#6284)
More complex to implement for sure, but also more correct.

Reply to this email directly or view it on GitHub:
https://github.com/OrchardCMS/Orchard/issues/6284#issuecomment-173010780

Do you want to substantiate that maybe?

No need les jeux sont pipes.
Envoyé via Outlook Mobile
On Tue, Jan 19, 2016 at 2:40 PM -0800, "Bertrand Le Roy"<EMAIL_ADDRESS>wrote:
Do you want to substantiate that maybe?

Reply to this email directly or view it on GitHub:
https://github.com/OrchardCMS/Orchard/issues/6284#issuecomment-173011276

I think the current behavior is fine, changing it would just push the issue somewhere else, defining your custom string would not prevent you from deciding to change it later and make existing migrations fail.


vault: 1.18.1 -> 1.18.2

Diff: https://github.com/hashicorp/vault/compare/v1.18.1...v1.18.2
Changelog: https://github.com/hashicorp/vault/blob/v1.18.2/CHANGELOG.md
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 358001

x86_64-linux

:white_check_mark: 2 packages built:

vault
vault-bin


Config branch

in this PR i have set up the linters

Hi @ANTOINE1128, Congratulations on making an effort in this milestone. You can merge this branch.
Your Pull request is approved.


[FR] Be nice if the panning event for tall/wide images that happen with mouseover kicks off when navigating to a toot with j/k too

Looks like included in dev.phanpy.social


New Crowdin translations

Enjoy!

@bkimminich there are also some changes to review in the crowdin interface for the french translation.


Misleading warnings: "local variable referenced before assignment"

Starting with Cython 0.15, constructions like
def setxattr(path, name, value):
    '''Set extended attribute'''

    cdef int ret
    cdef Py_ssize_t len_
    cdef char *cvalue, *cname, *cpath

    PyBytes_AsStringAndSize(value, &cvalue, &len_)
    #...

produce warnings of the form
warning: /home/nikratio/projekte/llfuse/src/fuse_api.pxi:58:42: local variable 'cvalue' referenced before assignment
warning: /home/nikratio/projekte/llfuse/src/fuse_api.pxi:58:49: local variable 'len_' referenced before assignment

...but obviously the code is correct. I'm not sure if warning about uninitialized values when one just passes a pointer to that variable really catches more bugs than it produces false positives.
Migrated from http://trac.cython.org/ticket/714

The same applies to the usage of C varargs, e.g. when calling va_start() on an "unininialised" va_list variable. Seems really not easy to fix.

Do we then perhaps need a "virtual initialize" that marks the variable as initialized without generating code? Perhaps a virtual storage class modifier? Somethng like "cdef noinit char *s" ?

Something like "cdef noinit char *s" ?

I think we're generally trying not to introduce new syntax (without a really good feature to justify it)


Sample Photo App Automated

I have created this PR for UI test Automation, iOS sample "SamplePhotoApp".

@Rajneesh360Logica this is not the right repository for UI tests. Please create new PR to QualityAssurnace repo.
Closing this one.


Docs: Improve documentation for Azure CLI authentication

Description
It is unclear how to configure deltalake to authenticate using Azure CLI credentials. It seems like it should be supported (rust documentation). However, the delta lake documentation doesn't show how it is done. It only shows key based examples for AWS.
All the other options in the rust documentation are key-value based, but the UseCLI doesn't seem to be, so what do you put, when you are supposed to pass a dictionary?
I have tried to figure it out myself, but without any success:
> az login
> python
>>> from deltalake import DeltaTable
>>> dt = DeltaTable(abfs_path, storage_options={
    "AZURE_TENANT_ID": tenant_id,
    "AZURE_USE_AZURE_CLI": "true"
})
PyDeltaTableError: Failed to read delta log object: Generic MicrosoftAzure error: At least one authorization option must be specified

Use Case
For data scientists/data analysts who are not knowledgeable about authentication it is easy to sign into the CLI but hard to manually featch tokens or manage service principal credentials for storage_options.
Related Issue(s)
This might be related, if the mentioned bug is also affecting CLI auth: https://github.com/delta-io/delta-rs/issues/1153

Hi @kvedes,
Our docs link to the latest config for object store, but the current python release pre-dates the azure cli support. That said, we are looking to cut a new release soon, which will then support azure cli auth.
Your configuration is already correct, personally I would set if up with keys a bit more pleasing to the eye, but thats just cosmetics 😆.
dt = DeltaTable(abfs_path, storage_options={
    "tenant_id": "<tenant_id>",
    "use_azure_cli": "true"
})

Hi @roeap
Thanks for your swift reply. Will the new release also enable SAS key auth? - It's a little unclear which ones will be supported and which ones I'm just not configuring correctly:
key = "sp=r&st=2023-03-08T08:15:31Z&se=2023-03-08T16:15:31Z&spr=https&sv=2021-12-02&sr=c&sig=SECRETVALUE"
url = "https://MY_STORAGE_ACCOUNT.blob.core.windows.net/MY_CONTAINER?sp=r&st=2023-03-08T08:15:31Z&se=2023-03-08T16:15:31Z&spr=https&sv=2021-12-02&sr=c&sig=SECRETVALUE"
dt = DeltaTable(url, storage_options={
    "sas_key": key,
})

Gives error:
PyDeltaTableError: Failed to read delta log object: Generic MicrosoftAzure error: Container name must be specified

The container is already a part of the URL, so it is unclear from the error, where the container name is missing. Also there doesn't seem to be an option that lets you point at a specific table when using the SAS key, so I suppose it has to be added to the URL in some manner?

So I managed to figure it out for the SAS token. Instead of using the https path given in the portal when creating the token, one has to do the following:
key = "sp=r&st=2023-03-08T08:15:31Z&se=2023-03-08T16:15:31Z&spr=https&sv=2021-12-02&sr=c&sig=SECRETVALUE"
container_name = "my_container"
storage_account_name = "my_storage_account"
table_name = "my_table"
abfs_path =<EMAIL_ADDRESS>
dt = DeltaTable(abfs_path, storage_options={
    "sas_key": key,
})

Might do a PR to add these auth options to the python docs at some point.


official binaries?

why torture users with a UI IDE? since this is DONE maybe help potential users with official binaries?
in six years from now it would be much easier to have binaries instead of trying to find a matching version of the IDE

What's wrong with STM32CubeIDE?

Nothing is wrong with STM32CubeIDE today.
I am speaking about

usability for end users
long term risk

in 6 years from today there is a good chance that newer STM32CubeIDE would not open current project. in 6 years there is a good chance that older version would be either hard to get or would refuse to install along side current version. C'mon, compiler warning/error behavior for corner cases changes every couple of years, Windows vs Linux versions of same compiler are themselves compiled with different flags! In my opinion a better way to open source is a combination of source code AND official binaries.

ST will always provide support to migrate old projects to new projects.
But I'm posting the source code?

@DanielMartensson you are placing too much faith in ST. They are humans, they have defects and sometimes they do not fix those defect for years.

@rusefillc  Yes I know. But at least they trying to build tools that are very easy to use. Also ST mcu are very cheap.


nipapd.service wont start - python error

Hello,
i try to get nipapd start working but no luck. The following error occurs
root@nipap-demo:/usr/lib/python2.7/dist-packages# Traceback (most recent call last):
File "/usr/sbin/nipapd", line 219, in
import nipap.xmlrpc
File "/usr/lib/python2.7/dist-packages/nipap/xmlrpc.py", line 16, in
from flask.ext.compress import Compress
ImportError: No module named ext.compress
so no socket is listening.
Install on Debian 10 latest (stable) Build
Has anyone a hint?

Hmm it seems the project is dead

I think you should consider using docker versions instead - that way you don't need to bother with all the install issues

Open /usr/lib/python2.7/dist-packages/nipap/xmlrpc.py
on line 16, replace:
from flask.ext.compress import Compress
with:
from flask_compress import Compress
That is a dirty "fix", and there might be other bugs laying around.

Ok, i'll give it a try
I wonder why it's called the best IP management tool and why it doesn't even install. No error is displayed during the installation and it does not start. No, it's much worse, it starts (according to systemd) and ends silently.
I've been working in Linux since Slackware 2.1 and there were projects that came and went, but they all worked ... 'out of the box'
It wouldn't be a problem to pay for it either, but to leave it online like that is unkind, then take it offline, it's more honest.
regards

If they took it online no-one would be able to fix things - I assume you mean the packages? sure, if they don't build.
If its not a problem to pay, you could pay developers to fix things?

This issue has been fixed, closing.


Cannot access cookies

https://github.com/grevory/angular-local-storage/blob/master/angular-local-storage.js#L325 accesses cookies from $document and not $document[0] or from $cookies via ngCookies

+1


[DO NOT MERGE] Add UK aid logo to DFID's page

Adds the UK aid logo section to a specific org page.

This PR exists only if we really need it. In order to add this to the org page in question some very specific logic has been written and this is not the way that GOV.UK should be built - if there is a requirement for something on an org page it should be an option available to all orgs to configure as they wish, not hard coded into a template like this.
This PR will not be merged.
Trello card: https://trello.com/c/1W0ySgAs/57-uk-aid-component-missing
Example: https://govuk-collections-pr-764.herokuapp.com/government/organisations/department-for-international-development

We haven't been asked to add this, so I'll close it.


Sourcemaps not working in new version of Scout-app v2.18.16

OS:
Scout-App Version:
Information from Dev Tools:
Uninstalled v2.18.16 and installed v2.12.12 and everything working fine again.
(Attach screenshot below)

Closing this as duplicate of #402


removing label names and confidence in predicted picture

💡 Your Question
I have trained yolo nas model using custom dataset and it is showing desired result but i want to remove label names  and confidence in predicted photos only want bounding box in picture.
Versions
No response

@rabion1234 There is no such option if you are using the .predict().


Fade is not working on Iphone

The Fade effect is not working on my Iphone, however when I use Chrome toggle device toolbar and select Iphone, it does seem to be working, although there's a strange behavior with the effect.
You can check the site here https://dashboardbox.netlify.com/#/ and select one of the movies and you will see what the behavior I'm talking about (in mobile mode). In my Iphone I never get to see the content inside the  component. Maybe it's related to my css?

I have the same problem in here, I was using it in my startup, but I am launching my business right now,  so I remove this lib. =( But I really appreciate this job!


License

Hi there! Could you please add a license to the project? I noticed basedmypy is MIT but that's probably primarily because it's a fork. I couldn't find any license information on the basedtyping project.

Mit?

Is that you questioning me mentioning the MIT, or you suggesting basedtyping is/may be released under the MIT?  :)

Haha, sorry. Do you think MIT would make sense?

Under the condition it's allowed (I don't know the history of the basedtyping codebase) then yes, the MIT is a great license for interoperability and compatibility with other free software licenses.

When I say "allowed", that means by the authors. Assuming none of the repo code was copied from elsewhere, that appears to be you and @DetachHead

https://github.com/KotlinIsland/basedtyping/commit/404b7df1852c90e55d955f5a450b2d86851535a7
Do you need me to release it?

Excellent! A release is not a hard requirement for me at this time. I can wait until the next one naturally comes around.
Thanks!


Added ISecretsListParser, ISecretsListValidator interfaces

What issue does this PR address?
SecretParser & SecretValidator classes will be implementations of interfaces, and the constructor of the ClientSecretValidator class now reflects this.
Does this PR introduce a breaking change?
Yes. ValidateAsync parameters are now in a different order.

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

Any word on this change?

We are currently in the process of prioritising feature requests. We'll know more soon.

This PR looks ok, but there is a new (minor) conflict that needs fixing. Can you please do that, and then we can merge it in. Thanks.

Have resolved the conflict. Thanks.


Trouble composing several elements

Not sure If I am just misusing ratatouille or it is not possible but anyway I would like to clarify some behaviour
If I have the following for rendering two panels
def render_first do
  column(size: 12) do
    panel(title: "First", height: 4)
  end
end

def render_second do
  column(size: 12) do
    panel(title: "Second", height: 4)
  end
end

I can do it like this and both rows show up as expected:
def render(model) do
  view do
    row do render_first end
    row do render_second end
  end
end

However, if I do it like this, only the last defined row will ever show up:
def render_both do
  row do render_first end
  row do render_second end
end

def render(model) do
  view do
    render_both
  end
end

It works if I change it to the following:
def render_both do
  panel do
    row do render_first end
    row do render_second end
  end
end

def render(model) do
  view do
    render_both
  end
end

but then I have a panel and the border, which I don't want.

This is a (surprising, but logical) consequence of how Elixir return values and the Ratatouille DSL interact:
Elixir functions return the last statement from their do blocks; Ratatouille view macros extract all child macros from their do blocks.
So what you are seeing is:
def render(model) do # returns the last expr, `view`
  view do # extracts both `row`s
    row do render_first end
    row do render_second end
  end
end

vs
def render_both do # returns the last expr, the second `row`
  row do render_first end
  row do render_second end
end

def render(model) do
  view do
    render_both # inserts the last expr from `render_both`
  end
end

As you call attention to, this semantic difference in how do blocks are interpreted, while core to the DSL, does mean you have to context-switch how to correctly compose components depending on where you are doing it.
The best way I have found to work around this is by return an array of components:
def render_both do
  [
    row do render_first end,
    row do render_second end
  ]
end

To my surprise, using macros did not work. I would have imagine this succeeding, but also only renders the last row:
defmacrop render_both do
  quote do
    row do render_first end
    row do render_second end
  end
end

the issue lies in these two lines: https://github.com/ndreynolds/ratatouille/blob/cc7b6a37e0b1757cd89cc5084343814a79dd86dc/lib/ratatouille/view.ex#L308-L309
the workaround i've found is to wrap things in a viewport (I think at least, I just discovered this)
This code will not work
view do
  if true do
    row do
      column size: 4 do
        panel(title: "Col1")
      end

      column size: 4 do
        panel(title: "Col2")
      end

      column size: 4 do
        panel(title: "Col3")
      end
    end

    row do
      column size: 4 do
        panel(title: "Col1")
      end

      column size: 4 do
        panel(title: "Col2")
      end

      column size: 4 do
        panel(title: "Col3")
      end
    end
  end
end

This code will work
view do
 if true do
   viewport do
     row do
       column size: 4 do
         panel(title: "Col1")
       end

       column size: 4 do
         panel(title: "Col2")
       end

       column size: 4 do
         panel(title: "Col3")
       end
     end

     row do
       column size: 4 do
         panel(title: "Col1")
       end

       column size: 4 do
         panel(title: "Col2")
       end

       column size: 4 do
         panel(title: "Col3")
       end
     end
   end
 end
end


HDDS-10246. Remove readLock from KeyValueHandler.checkContainerIsHealthy

What changes were proposed in this pull request?
As described in HDDS-10146, all of the ChunkReader threads are blocked in the KeyValueHandler#checkContainerIsHealthy method, waiting for the KeyValueContainer#readLock.
Remove readLock from KeyValueHandler.checkContainerIsHealthy to improve performance
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-10246
How was this patch tested?

No new test and pass exist unit tests
Works well in our cluster

@ChenSammi PTAL, Thanks !

as this getState  is fully synchronized method. We need to see if this method is really needed to be synchronized. I feel it is not need to be synchronized.

Not sure to what extent it affects performance. At least in practice, we haven't encountered any bottlenecks caused by synchronized getState yet. Maybe we can revisit it in the future if the problem occurs ?


Make UniformGrid account for layout rounding.

What does the pull request do?
As described in #17699, UniformGrid was ported from WPF but Avalonia's layout rounding is slightly different. Update ArrangeOverride to account for this.
Instead of arranging each child without considering whether it was snapped to the pixel grid, layout each each child with it's top-left aligned against the previous column/row, with the bottom-right placed in the ideal position. This ensures that there are no overlapping/spaces between controls, and also ensures that the overall uniform grid is preserved.
Fixed issues
Fixes #17699

An alternative fix might be to simply apply the layout rounding to the cell size. Conceptually, it's not really different from having a panel for each cell, which should work "naturally".

My first attempt started out doing this, but I was finding that you wound up in situations where the children didn't fill the panel completely, or went outside the panel bounds. However thinking more about it, it might because i was using LayoutHelper.RoundLayoutSizeUp. It may work better if we add a LayoutHelper.RoundLayoutSize which rounds to the nearest pixel (like WPF).

We can't rely on the previous child's bounds here: it might effectively be completely different (try any child with non-stretch alignment) from the cell's size, breaking the uniformity.

Good catch! Yes I should have thought of this ;) We need more unit tests for this control obviously.

One thing we need to decide: when snapping to pixels, there are situations where we have to choose between having the children not completely fill the panel, or having the children sizes not be completely uniform. Need to confirm what WPF does (from looking at the code, I think it will sacrifice uniformity but need to check this in practise).

It may work better if we add a LayoutHelper.RoundLayoutSize which rounds to the nearest pixel (like WPF).

I think we should keep RoundLayoutSizeUp(), because that's what every control in Avalonia uses, and we need to be consistent.
Imagine that instead of having a UniformGrid responsible for layouting the controls, we're generating the matching cells instead (with each cell being its own container) and stacking them using a standard StackPanel.
In that case, the cells would naturally "compose" without overlap, because they're already using RoundLayoutSizeUp in MeasureCore/ArrangeCore.
With code, I expect the following two panels to have the exact same layout at any DPI size:
<StackPanel Orientation="Horizontal" Width="99" Height="100" Background="Black">
  <Border Width="33" Background="Red" />
  <Border Width="33" Background="Green" />
  <Border Width="33" Background="Blue" />
</StackPanel>

<UniformGrid Width="99" Height="100" Columns="3" Background="Black">
  <Border Background="Red" />
  <Border Background="Green" />
  <Border Background="Blue" />
</UniformGrid>

That won't work if we apply a different algorithm just for UniformGrid. Note that in this case, the combined width of the 3 borders (100.8px) gets larger than the width of the StackPanel (99.2px) at 125% DPI.

I tested your two examples on WPF with 125% scaling, modifying the code to make the panel width 100 (i.e. not divisible by 3) and to also make clear the child bounds in relation to the parent:
<Window x:Class="WpfApp1.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        Title="MainWindow" Height="450" Width="800"
        UseLayoutRounding="True">
  <Grid>
    <Grid.ColumnDefinitions>
      <ColumnDefinition Width="*"/>
      <ColumnDefinition Width="*"/>
    </Grid.ColumnDefinitions>

    <Border Background="Beige" Padding="0 4" HorizontalAlignment="Center" VerticalAlignment="Center">
      <StackPanel x:Name="stack" Orientation="Horizontal" Width="100" Height="100">
        <Border Width="33" BorderBrush="Red" BorderThickness="1"/>
        <Border Width="33" BorderBrush="Green" BorderThickness="1"/>
        <Border Width="33" BorderBrush="Blue" BorderThickness="1"/>
      </StackPanel>
    </Border>

    <Border Background="Beige" Padding="0 4" Grid.Column="1" HorizontalAlignment="Center" VerticalAlignment="Center">
      <UniformGrid Name="uniform" Width="100" Height="100" Columns="3">
        <Border BorderBrush="Red" BorderThickness="1"/>
        <Border BorderBrush="Green" BorderThickness="1"/>
        <Border BorderBrush="Blue" BorderThickness="1"/>
      </UniformGrid>
    </Border>
  </Grid>

</Window>

We get the following results
StackPanel
0,0,32.8,100
32.8,0,32.8,100
65.6,0,32.8,100

UniformGrid
0,0,33.6,100
33.6,0,33.6,100
66.4,0,33.6,100

And look what happens. The UniformGrid has exactly the same overlapping problem as Avalonia! (the right-most green border is obsured).

The docs for the WinUI UniformGrid say (highlighting added):

The UniformGrid control is a responsive layout control which arranges items in a evenly-spaced set of rows or columns to fill the total available display space. Each cell in the grid, by default, will be the same size.

So the two constraints for its behavior is that

The children fill the panel
The children are the same size

Closing as the current behavior matches WPF.


Rule to check Error handler implementation for Flows in the Mule application

There are two distinct approaches for error handling in a Mule application to catch exceptions.

Global Error Handler (Application Error Handling)
Error Handling within main flows

It will be helpful to have general "error handling" rule with configurations to check for error handler implementation for Mule application. Rule would need to check that all the flow must have error handler, when global error handler is not configured.

To implement this rule we will do the following:

Check if there exists Global Error Handler exists in global-config.xml.
When global error handler is not configured, Find all the flow components in the configuration files and verify that error-handler configuration exists in the flow.


add delete/update in spark sql

I created the pull request to add delete & update grammars in spark sql, which has following drawbacks:

It adapts all grammars of spark sql with corresponding code, for it is hard to modify or extend the static fields of org.apache.spark.sql.catalyst.parser.SqlBaseParser.

It requires customized DataWritingCommand to delete/update rows of Hive transactional tables, to avoided repeated work as I thought in orc_write branch.

It doesn't have test suite, even I have tested some complicated delete/update sql in local to make sure LogicalPlan resolved, it's hard to write test cases in lack of DataWritingCommand.

It doesn't support sub-query in delete/update statements just as Delta Lake, but it should support expressions with functions & field references in delete/update statements.

Thanks for the PR!
cc @citrusraj @prakharjain09

Herbert,
see SparkSessionExtensions.injectParser
https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/SparkSessionExtensions.scala#L215
to extend the parser instead of copying entire parser.
-- Raj


Add the ability to calculate the interval between two dates

Now to calculate this data, I am using the date calculation that comes with Windows.

Well, thanks for the answer. Although I don't think there are many cases where someone just happens to come across the format “XXXX/XX/XX/XX-XXXX/XX/XX”. But it's not really a good idea to rule it out.
I can't think of any good solutions right now. Should this issues be turned off?

Feel free to close it for now while I explore options to improve the date time calculations.  Thanks for the feedback!


Typo Fixes - Fallout VNDPNP

Changes / Comments
Fixed a few typos and size errors.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Hi @Skyeris
Unfortunately, it appears as though there are conflicts between this and another pull request that was merged. You'll need to resolve these issues before we can merge this content in.

@nmbradley The conflict was resolved.