AI Generated Fruits and Real Fruits Classification using Image Processing

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : AI Generated Fruits and Real Fruits Classification using Image Processing
:red_circle: Aim : The aim of this project to identify and predict the real fruits and AI generated fruits using Image Processing methods.
:red_circle: Dataset : https://www.kaggle.com/datasets/osmankagankurnaz/dataset-of-ai-generated-fruits-and-real-fruits
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Email ID :
Participant ID (if applicable):
Approach for this Project :
What is your participant role? (Mention the Open Source program)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

Name: Kunal Sharma
GitHub Profile Link: https://github.com/KunalSharmaGit
Email ID<EMAIL_ADDRESS>Approach for this Project: This can be implemented using various algorithms especially CNN in which we use Sigmoid activation function to classify the AI Generated and Real Fruits.
What is your participant role? Social Winter of Code 2023

Wait for the program to officially start. Issues will be assigned to you after the orientation.
@KunalSharmaGit

Hi @KunalSharmaGit welcome to SWOC 2024. What are the models you are planning to implement for this project?

Hey @abhisheks008, I think algorithms like Support Vector Machines(SVM), k-NN and CNN can be effective in implementing this project.

Can you suggest some of the Deep Learning methods for this project?

CNN, VGG and ResNet.

Full name : Titiksha Agrawal
GitHub Profile Link : https://github.com/AgrawalTitiksha
Email ID<EMAIL_ADDRESS>Approach for this Project : After data preprocessing and visualization, using PCA algorithm for normalizing and standardizing the images, then to use LDA , SVM (all 3 kernel) , and CNN (VGG16) for analyzing the images, it can also be used to predict for new input dataset (whether ai generated or not)
What is your participant role? (Mention the Open Source program) : @contributor 2024
need to mention: have worked on a similar project but topic, "Early stage Alzheimer's disease detection, classification and prediction" using the same model's above (excluding CNN).

Based on the given approaches I'll go with @AgrawalTitiksha. Try to use ResNet along with VGG16. Issue assigned to you. You can start working on it.

@abhisheks008 thankyou for assigning this, I'll hit the given requirements. Just to ask you what is the deadline.

You can take your time according to your availability. Just finish the issue before the program ends.

sure sure


Contact Page

Issue Number #88 Fixed (that was assigned to me)
Added a cool new contact us page with minimal design and matching the dark theme of the website. Fixed glitching navbar that was expanding abnormally on hover, and added link to contact us page on home navbar accordingly.

Nice @ekagrashukla.
But resolve conflicts so I can merge your PR.

It's resolved now


add all in one solar notebook

업스테이지 api 다 들어있는 노트북이예요

Can we include OCR and KIE features as well?

@chosh0615 They are already included.

@chosh0615 more user friendly version
on colab.


tags remain in generated markdown

I don't understand why https://github.com/domchristie/to-markdown/blob/50d28867bb05019ca20ec2803460f058dd0d950a/test/to-markdown-test.js#L121 is the expected result. Shouldn't something be done with
 tags with no  inside, maybe perhaps only removing it?

Just saw that other tags, such as
, also remain in the generated markdown. Why is that? These elements are not markdown, why not remove them?

Well, I imagine this issue is not really an issue, because clearly keeping these tags was a design choice.
For those who, like me, want "pure" markdown generated by a library from HTML, I have forked this project at https://github.com/fiatjaf/to-pure-markdown

@fiatjaf do you know the reason for this design choice?

@OKNoah I don't know the reason, it's probably because some Markdown editors accept HTML tags and this library assumes people will use it for filling these editors? Perhaps.
I only concluded that it is a design choice from reading the code.

@OKNoah @fiatjaf the reason for this design choice is to preserve the intention of the source HTML. HTML is perfectly valid in markdown and since pre has no direct markdown equivalent, there is a question as to what it should be converted to.
The options are

remove the pre tags and just include the text content. However this would then be interpreted as a paragraph, and therefore would lose the original intention.
convert to a markdown code block (i.e. indent it); but strictly speaking, it’s not a block of code, so the meaning changes.

I know that some developers are using this library in WYSIWYG editors and wish to strip all HTML tags, whereas others are using it to convert pages on content, where it maybe more important to preserve these tags. It’s a relatively common problem, so I hope to have a solution in the future.
Thanks for your interest :)

I'd think pre tags that are not preceded by line breaks could be replaced with back ticks  and ones that are could be code blocks.

@OKNoah Unfortunately the whitespace formatting of the original HTML has little significance outside of a pre block, so I feel it’s a step too far to assume that the author’s intention was to create a code block. Take this poem for example:

Jabberwocky

'Twas brillig, and the slithy toves
Did gyre and gimble in the wabe;
All mimsy were the borogoves,
And the mome raths outgrabe.

If pre blocks were converted:
# Jabberwocky

    'Twas brillig, and the slithy toves
    Did gyre and gimble in the wabe;
    All mimsy were the borogoves,
    And the mome raths outgrabe.

Which would then back to HTML as follows:

Jabberwocky

'Twas brillig, and the slithy toves
Did gyre and gimble in the wabe;
All mimsy were the borogoves,
And the mome raths outgrabe.

So we now have a poem marked up as code!

OK, thanks for the explanation. I've personally not seen anything like that marked up in pre but I'll take your word for it.

If you need to convert plain pre blocks, you can add a custom converter:
toMarkdown('
…
', {
  converters: [{
    filter: 'pre',
    replacement: function (content) {
      return '\n\n```' + content + '\n```\n\n';
    }
  }]
});

Got one set up, thanks :)

If you want to convert plain pre only,
You can use this filter/replacement combo. (note that uses ES6 arrow function syntax.)
toMarkdown(dirty, {
    converters: [{
      filter: (node) => node.nodeName === 'PRE' && (node.innerHTML.toString().toLowerCase().indexOf('') === -1),
      replacement: (content) => '```\n' + content + '\n```'
    }]
  });


Pickle works with the old API not the new one

Whenever Pickle loads a saved Shogun model, it expects the model to be implemented using the old API this can be demonstrated here
I guess this is related to #4463

Could you post a minimal standalone python example on synthetic trivial data instead of the notebook? Thx

@karlnapf I have updated the link, now it's a python file using Shogun's data however you're going to have to edit the path to point to your shogun dir

thanks. that does it.
(BTW standalone means no loading of files, or adjustment of paths, but one can run this straight away, but nevermind this is sufficient.)

Could you share Traceback if there is one please?

Could you share Traceback if there is one please?

@gf712 I have updated the gist

I think the offender is this hackery here:
if isinstance(cls, str) and cls.startswith('shogun.'):
            if base is object:
                import shogun
                return eval(cls+'()')

Which templated objects are you referring to? The template type is hidden from interfaces no?

Nevermind for now, I thought this could become a problem, but not sure anymore now, as the serialization and the new api have changed the picture. I guess someone will see when trying to fix this


feat(eager): generalized logic for splitting inputs and attributes

Generalize #368 to all eager backends
split happens in the evaluator instead of the Op or OnnxFunction call methods so that evaluators can control behaviors like whether to fill in default values for attributes.
Fixes #287, partially #305 (still need the scripted version)

@gramalingam I am merging this now. Please feel free to continue to add comments if there are and I will update in another PR.


Fix FIPS compliance issues for KubeRay Operator

We have the following FIPs compliance issues with the KubeRay Operator image
openssl library not present  |
 /manager        | go binary is not CGO_ENABLED |

We should resolve this via PRs to upstream first, if possible.

hmm, it's not letting me assign @tedhtchang to the issue.

With the wip  this is this scan result:
root@brinks1:~/check-payload# ./check-payload scan operator --spec quay.io/tedchang/kuberay-operator:latest
I1103 17:14:35.367067  373643 validations.go:363] rpm -qf error: exec: "rpm": executable file not found in $PATH (stderr=)
I1103 17:14:35.367176  373643 scan.go:325] "scanning failed" image="quay.io/tedchang/kuberay-operator:latest" path="/usr/sbin/ldconfig" error="executable is not dynamically linked" component="ubi8-minimal-container" tag="" rpm="" status="failed"
---- Failure Report
+------------------------+--------------------+--------------------------------------+------------------------------------------+
| OPERATOR NAME          | EXECUTABLE NAME    | STATUS                               | IMAGE                                    |
+------------------------+--------------------+--------------------------------------+------------------------------------------+
| ubi8-minimal-container | /usr/sbin/ldconfig | executable is not dynamically linked | quay.io/tedchang/kuberay-operator:latest |
+------------------------+--------------------+--------------------------------------+------------------------------------------+
F1103 17:14:35.580738  373643 main.go:259] Error: run failed
I don't think the test should be marked Error: run failed
The config.toml  has
[[rpm.glibc.ignore]]
error = "ErrNotDynLinked"
files = [ "/usr/sbin/ldconfig", "/sbin/ldconfig" ]

so I think this error should be ignored.

Upstream issue: https://github.com/ray-project/kuberay/issues/1632
Upstream PR: https://github.com/ray-project/kuberay/pull/1633

Ubuntu is not supported by the payload check program. The error was resolved by testing from a RHEL VM.

Upstream PR was merged in.


Can't find source code of downloaded version of Crystal plugin

Description of the problem or steps to reproduce
When installing the Crystal plugin with plugin install crystal, the version downloaded is the 1.1.0. Can see the following repo.json file inside .config/micro/plug/crystal:
[{
  "Name": "crystal",
  "Description": "adds helpful crystal tools",
  "Tags": ["crystal-lang", "crystal"],
  "Website": "https://github.com/ColinRioux/micro-crystal",
  "Versions": [
    {
      "Version": "1.1.0",
      "Url": "https://github.com/ColinRioux/micro-crystal/archive/1.1.0.zip",
      "Require": {
        "micro": ">=2.0.0-1"
      }
    },
    {
      "Version": "1.0.0",
      "Url": "https://github.com/ColinRioux/micro-crystal/archive/1.0.0.zip",
      "Require": {
        "micro": ">=1.0.3"
      }
    }
  ]
}]

But if I go to the original repository: https://github.com/ColinRioux/micro-crystal, the release 1.1.0 doesn't exist. Only the 1.0.0. Here's the repo.json of the repository: https://github.com/ColinRioux/micro-crystal/blob/master/repo.json
Am I missing something?
Specifications
Commit hash: 60846f5
OS: Linux (Ubuntu 20.04)
Terminal: Gnome-Terminal

Micro V2 installs plugins from here https://github.com/micro-editor/updated-plugins/tree/master/
source code for crystal plugin is here https://github.com/micro-editor/updated-plugins/tree/master/micro-crystal
looks like the release zip has the wrong info in the repo.json file

@tommyshem My local repo.json differs from the one in that repository.
See in my first message that the .zip of 1.1.0 points to the wrong repository, but the one in here points to the correct repository.

the source code has been updated but look at the release zip file contents on the github page https://github.com/micro-editor/updated-plugins/releases and it matches your repo.json file.

Ok, resolved then :) thanks


[Enhancement] report num of tablets to FE along with heartbeat response in shared-data mode

fix show backends/show compute nodes sql doesn't show the tablets number in shared-data mode

Why I'm doing:
What I'm doing:
Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[X] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[X] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[X] I have checked the version labels which the pr will be auto-backported to the target branch

[X] 3.2
[X] 3.1
[ ] 3.0
[ ] 2.5

how about getting this information from starmgr?
will the heartbeat between FE and BE be retained in the future?

how about getting this information from starmgr?
will the heartbeat between FE and BE be retained in the future?

It is possible to retrieve this info from StarManager, let me think about it.


Archivo de entidades preliminares del proyecto

Codecov Report

Merging #3 (01d8e50) into develop (544f72b) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             develop        #3   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity         5         5
===========================================
  Files              2         2
  Lines             10        10
===========================================
  Hits              10        10

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fix CPU use

Fix for #691
The argument parsers of the webui and the postapi default device to cuda but never checks if CUDA is actually available.
This is a very small fix to default to CPU if torch.cuda.is_available() is False.

LGTM


Partial rotate feature

Currently it seems like we can only rotate in increments of 90 degrees. It would be nice if we could partially rotate an image to help level it. This would obviously change the size of the photo since the aspect ratio would need to be maintained. An example UI for this could be the iOS implementation shown here: https://media.idownloadblog.com/wp-content/uploads/2015/02/rotate-photos-on-iPhone.png

closed in favor of #936


Upgrade & fix break change in jsonschema

Followed the information from https://github.com/Stranger6667/jsonschema/blob/HEAD/MIGRATION.md#upgrading-from-025x-to-0260
I don't know if we have any tests that make sure we get validation error? If not maybe something that can be added. Not sure how though 😄

Relates to #129

Codecov Report
Attention: Patch coverage is 33.33333% with 368 lines in your changes missing coverage. Please review.

Project coverage is 83.87%. Comparing base (71c5023) to head (2240d59).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
src/tests/schema_validation/v2_0_1.rs	33.33%	256 Missing ⚠
src/tests/schema_validation/v1_6.rs	33.33%	112 Missing ⚠
Additional details and impacted files
☔ View full report in Codecov by Sentry. 📢 Have feedback on the report? Share it here.

@tommymalmqvist Not sure why this happens. Do you?


Allow bosh lite LATs to run without the TrafficController

Allow link to be optional
Provide ops file to remove TC link from manifest
Have deploy-bosh-lite script use ops file to disable TC
Make scripts more consistent
Use LATs version that can skip TC tests

Related: https://github.com/cloudfoundry/logging-acceptance-tests/pull/2

Closing due to inactivity. Please re-open if you still see a need for this.


The attrs field cannot be defined in a FormHelper subclass.

For most fields the following syntax works:
class MyFormHelper(FormHelper):
    form_class = 'form-horizontal'

But the following does not work:
class MyFormHelper(FormHelper):
    attrs = {'some-attribute': 'some-value'}

The only way accomplish this in a FormHelper subclass appears to be:
class MyFormHelper(FormHelper):
    def __init__(self, *args, **kwargs):
        super(MyFormHelper, self).__init__(*args, **kwargs)
        self. attrs = {'some-attribute': 'some-value'}

This appears to be because FormHelper.init overwrites attrs (rather than it being defined at the class level).  See:
https://github.com/maraujop/django-crispy-forms/blob/dev/crispy_forms/helper.py#L208

I'm going to close this as Out of Scope for the moment. Happy to consider documented, tested pull requests. We may have capacity to reconsider in the future.


fix: things

Creating bookings only works in Göteborg and Stockholm for now; adding adress info to tests to make them succeed.

What's the deal with this PR?

Umm...

I am fairly sure that whatever we had in mind back then, is no longer relevant. 🧺


Add missing --mediator-connections-invite cmd arg info to docs

Signed-off-by: Dobrosław Żybort<EMAIL_ADDRESS>

Thanks!
@ianco -- can you please confirm and approve this?  Thanks!


[HELP] Making HTTPS calls using MBEDTLS

Description
I need help understanding what is wrong with this implementation attempt.
I am trying to make HTTPS calls in my project using MBEDTLS, and for that, I am using the default Wi-Fi configuration for the ESP32C6.
$ ./tools/configure.sh esp32c6-devkitc:wifi
After that, I enable the following configurations in the menu:
CONFIG_CRYPTO_MBEDTLS
CONFIG_MBEDTLS_APPS
CONFIG_MBEDTLS_APP_BENCHMARK
CONFIG_MBEDTLS_APP_SELFTEST
With this configuration, when trying to build, I get the following output:

Please let me know if I skipped any necessary steps for success.
Verification

[X] I have verified before submitting the report.

Adding to what has already been said:
It seems like the issue is related to missing configurations in NuttX concerning the Espressif HAL for the ESP32C6.
As a validation, I enabled the MBEDTLS features using the following configuration for the STM32F746:
$ ./tools/configure.sh stm32f746g-disco:netnsh

I also applied the same configuration for the ESP32:
$ ./tools/configure.sh esp32-devkitc:wifi

@fdcavalcanti, @eren-terzioglu can you PTAL?

Checknuttx/arch/risc-v/src/esp32c6/esp-hal-3rdparty/nuttx/esp32c6/include/sdkconfig.h.
You will see some symbols are already defined there, maybe it will help you better understand what's going on.

I was trying to deal with similar in the past in https://github.com/apache/nuttx/pull/11738
Seems like linking process is using CFLAGS with both app flags and flags from Wireless.mk

You are right @pkarashchenko , I did same thing than you and now we are able to compile.
@fdcavalcanti  can you check whether a did right process here: https://github.com/espressif/esp-hal-3rdparty/pull/5, please?

@engdavidiogo is everything ok in your side ?

Yes @FelipeMdeO , I can reproduce the build without any errors.
@fdcavalcanti  I continue testing and have no apparent problems

@fdcavalcanti  We are able to update firmware using secure link, please lets update 3rdparty in nuttx master.

Reference

We are testing the changes internally. As soon as our CI finishes, we can merge it. Thanks!

Hi @engdavidiogo and @FelipeMdeO ,
We merged it on release/v5.1.c branch:https://github.com/espressif/esp-hal-3rdparty/commit/26d4d1b57d54b63384b4e466b9e644ff0fe702b5
Can you please test it setting ESP_HAL_3RDPARTY_VERSION=26d4d1b57d54b63384b4e466b9e644ff0fe702b5 before the make command to build NuttX? (we'd still need to update the hash for the HAL on NuttX)

Hi @tmedicci thanks for the support with this!
Closing...


Replace all .. with ... instead

This PR replaces two dots in series with three instead as usually having a process log line would contain ... not ...
Feel free to reject this PR if you preference it but it does hurt to look at.
For example:
[+] Extracting "content-configs.zip"..
[+] Extracted "content-configs.zip"! (Packages left: 19)

becomes:
[+] Extracting "content-configs.zip"...
[+] Extracted "content-configs.zip"! (Packages left: 19)

Ignore the commit message, it doesn't like dots with code formatting.

Builds now fail. Bad PR.

This PR also adds in a hidden backdoor.

Builds now fail. Bad PR.
This PR also adds in a hidden backdoor.

This is a feature

DNLGTM
File size increased... 😠😡

Opinionated(..) Denied!
Reasoning: refer to Liker https://github.com/latte-soft/rdd/pull/2#issuecomment-2101572660

not to mention: refer to Matt https://github.com/latte-soft/rdd/pull/2#issuecomment-2067008464

I believe this should be reconsidered. The implications of .. indicates that it is a fast pause rather than a long pause (...). Since @regginator's code is incredibly slow it would be more accurate to properly indicate the length using ... rather than ...
The file size loss is required to improve UX and there is no such backdoors in the provided pull request.
@latte-soft


Fix less and sass files not reloading in debug mode due to not return…

…ing an appropriate file path

Thanks for your PR.
I've taken another approach to handle this case.
Your help is really appreciated.
Cheers,


Refactor Chef::Application::Client#reconfigure

This branch refactors the very complex method Chef::Application::Client#reconfigure.

Travis CI build appears to have failed due to a network timeout :crying_cat_face:

Working on increasing the test coverage on the refactored methods.

Added test coverage.


Runtime programs do not return results in Qiskit provider even after executing

Describe the bug
When running runtime programs with the QASM3 runner, the program will complete in the hardware and be registered/updated appropriately in the IQS. However, when calling job.result() with the default options (and different timeouts) the call will hang forever and never return. Even sending a kernel interrupt will not end the request.
See this thread for more details.
Steps to reproduce
Run this notebook with sim = False. It may take several runs to trigger.
Expected behavior
The notebook should execute.
Suggested solutions
Additional Information

qiskit-ibm-runtime version: 0.6.1
Python version: 3.9.5
Operating system: OSX

@jyu00


Improve and add to specializations section

See issue #5

Notifying @jpthiele and @jcohen02 about this changeset. Being direct output of the workshop, I'll merge it.


Response Header Modification: Add set-cookie caveat

I'm not sure about exact wording here (suggestions/rewrites very welcome), but I think it could be very useful to call out the fact that the set-cookie header is a special case (in terms of IETF standards), and that this results in caveats for its use with the current version of the Response Header Modification feature.
I think the Response Header Modification functionality may have been written with a more general header case in mind (RFC 2616), whereas set-cookie is a special case in which multiple headers of the same name can be sent AND can't be collapsed together into a single header. The exceptional behavior of set-cookie leads to design constraints for any feature like Response Header Modification, and necessarily involves tradeoffs. I think it would be very valuable to list this particular tradeoff in this section of the documentation, because it's (IMO) non-obvious.

Hi @bregenspan
Thank you for your PR and all the context you provided.
We already support the specific case of set-cookie headers through the "Add" operation, instead of using the "Set static" or "Set dynamic" operations. The "Add" operation is described in Developer Docs, namely in the Cloudflare dashboard instructions:

Add — Adds an HTTP response header with a static string value. This operation will not remove any existing response headers with the same name.

Therefore, to set cookies using the modify response headers functionality and keep other cookies being set in the response you must use the "Add" operation.

Hi @bregenspan Thank you for your PR and all the context you provided.
We already support the specific case of set-cookie headers through the "Add" operation, instead of using the "Set static" or "Set dynamic" operations. The "Add" operation is described in Developer Docs, namely in the Cloudflare dashboard instructions:

Add — Adds an HTTP response header with a static string value. This operation will not remove any existing response headers with the same name.

Therefore, to set cookies using the modify response headers functionality and keep other cookies being set in the response you must use the "Add" operation.

@pedrosousa I think set-cookie is a common enough use-case that it could still be useful to call this out specifically in the doc here. As you noted, it's not currently possible to set a dynamic header value via Add (related feature request). That's why people might be tempted to still try Set Dynamic, which can actually work OK if one is not relying on another set-cookie header returned by the origin or a Cloudflare produce.


Adds GroupName dependency property to facilitate submenu implementation

Summary

Updated version of PR #1114.
Adds a GroupName dependency property to facilitate submenu implementation;
Adds an extension to get a list of submenu NavButtons.

In XAML
parent NavButton  (the first in the Group and must be Command button):
<Controls:HamburgerButtonInfo GroupName="groupName" ButtonType="Command" Tapped="MenuTapped" ... >

submenu NavButtons (can be any button type):
<Controls:HamburgerButtonInfo GroupName="groupName" ... >

In Codebehind
private void MenuTapped(object sender, RoutedEventArgs e)
{
     var hbi = (sender as HamburgerButtonInfo);
     var subMenuButtons = hbi.ItemsInGroup();

     ... // do visibility toggling on submenus (see previous example) or other operations ...
}

hbi.ItemsInGroup(); extension returns a list of submenu items (in PrimaryButtons) as List<HamburgerButtonInfo>.
hbi.ItemsInGroup(IncludeSecondaryButtons:true); provides an option to include SecondaryButtons.
No need to pass on the GroupName string; the extension does that job.
No need to exclude the parent NavButton (of type Command) from the returned List; again, the extension does that job.

Great work. Good thinking to make it an extension method, too. But since there is a Template10Utils class, and since this is specific to T10 and not XAML, let's put it in the T10 Utils and not the Xaml Utils. Do you agree?

Agree. Will update that.


Header detection issue

I have some problem with headers.
I testes it with swaks, it sends valid headers, but Guerrilla can't parse headers and gives me error:
DEBU[0001] Client sent: DATA
DEBU[0001] Writing response to client: 54 Enter message, ending with '.' on a line by itself
ERRO[0001] parse headers error error=EOF

I tested it with swaks -f<EMAIL_ADDRESS>-t<EMAIL_ADDRESS>--server <IP_ADDRESS>:2525

Hi, thanks for the report.
Do you know what is the full input that is fed to the DATA command? Please
paste a sample here.
On Fri., 15 Mar. 2019, 02:12 Serhii Chaplia<EMAIL_ADDRESS>wrote:

I have some problem with headers.
I testes it with swaks, it sends valid headers, but Guerrilla can't parse
headers and gives me error:
DEBU[0001] Client sent: DATA
DEBU[0001] Writing response to client: 54 Enter message, ending with '.' on a line by itself
ERRO[0001] parse headers error error=EOF
I tested it with swaks -f<EMAIL_ADDRESS>-t<EMAIL_ADDRESS>--server
<IP_ADDRESS>:2525
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/flashmob/go-guerrilla/issues/139, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAnmP8hGnQfnZGbT_KoJo46q4IOnz33Qks5vWmbxgaJpZM4b0Zl8
.

@flashmob sure
=== Trying localhost:2525...
=== Connected to localhost.
<-  220 iwex-pc SMTP Guerrilla(unknown) #1 (1) 2019-03-15T15:26:53+02:00
 -> EHLO iwex-pc
<-  250-iwex-pc Hello
<-  250-SIZE 10485760
<-  250-PIPELINING
<-  250-ENHANCEDSTATUSCODES
<-  250 HELP
 -> MAIL<EMAIL_ADDRESS><-  250 2.1.0 OK
 -> RCPT<EMAIL_ADDRESS><-  250 2.1.5 OK
 -> DATA
<-  354 Enter message, ending with '.' on a line by itself
 -> Date: Fri, 15 Mar 2019 15:26:53 +0200
 -> To<EMAIL_ADDRESS> -> From<EMAIL_ADDRESS> -> Subject: test Fri, 15 Mar 2019 15:26:53 +0200
 -> Message-Id: <20190315152653.027193@iwex-pc>
 -> X-Mailer: swaks v20181104.0 jetmore.org/john/code/swaks/
 ->
 -> This is a test mailing
 ->
 ->
 -> .
<-  250 2.0.0 OK: queued as a04ac137fdce82ed46362ca1daf0399d
 -> QUIT
<-  221 2.0.0 Bye

I have the same problem. The error comes from here.
From the ReadMIMEHeader documentation it expects a blank line at the end but the header slice doesn't include it.
IMO the fix can be like this.
header := buf[0:headerEnd+2]

It won't be out of bounds because the Index call before it made sure that those two bytes exists.


ARROW-6965: [C++][Dataset] Optionally expose partition keys as columns

PartitionScheme now acts on individual segments of a path rather than the whole path. This allows easier extraction of partition information relative to a given parent
FileSystemDataSource applies modified scan options to each file fragment it emits, including simplified filters and projectors containing default column values derived from partition keys
PathTree is removed in favor of PathForest, which provides a lighter weight implementation and supports accessing nodes' parents
PathPartitions are removed in favor of SegmentDictionaryPartitionScheme (should resolve ARROW-7069)
RecordBatchProjector now provides a mutator for setting default column values, rather than requiring these be specified on construction
RecordBatchProjector and ExpressionEvaluator use the MemoryPool provided by a scan context
specialize IterationTraits<util::optional> to support Iterator when a reserved "end" value for T is difficult
add path utilities IsAncestorOf and RemoveAncestor

Could you remove L34 and L42 (and the preceding commas) from https://github.com/apache/arrow/blob/master/r/tests/testthat/test-dataset.R? Then the test at https://github.com/apache/arrow/blob/master/r/tests/testthat/test-dataset.R#L76 will exercise (parts of) this new code.

Follow up for PartitionSchemeDiscovery: https://issues.apache.org/jira/browse/ARROW-7366

passing appveyor
merging


Settings sometimes don't load when rapidly clicking save

Windows Terminal version (or Windows build number)
1.8.1521.0
Other Software
No response
Steps to reproduce
Rapidly click on save in the settings menu.
Discovered while reporting a memory leak issue #10609, at 13 seconds in the video below you can see this issue.
https://user-images.githubusercontent.com/2277504/125179830-a31f3a00-e1f2-11eb-8b6f-6f6aa07896c5.mp4
Expected Behavior
Settings stay the same
Actual Behavior
Settings fail to load (possibly already corrupted or overriding on next save? not sure)

Indeed cannot reproduce (only saw it the one time, it might be dependent on moon alignment), I did find this while trying #12333


Add release to central to travis

Release to central on every build to master.

@buzdin ; have you been able to find the certs and add this task to travis?

https://github.com/DozerMapper/dozer/commit/2483cb39aa7c2850a15db241252b3cc2ee6221a5


Switch form control components to using reactive forms

Currently, some of our form control components rely on ngModel internally, which has to be processed asynchronously. This makes testing more difficult and causes more complexity than necessary.   We should be using something like formControl internally to make the processing sync.

This is fixed


Surpress/Remove error message "$apply already in progress"

Not sure if I am doing this right. If i trigger data update outside directive, I need to use $scope.$apply() to force the grid to refresh. However, if the data update is triggered through directive, then it would refresh automatically but then the $scope.$apply() would cause an error message.
See: http://plnkr.co/edit/XcSoBxCF4IaM4NPANPeh?p=preview
In external.js, I added a row externally when the document is ready. Works, everything looks good.
Now go to the top and click "Add blank row". It will cause:
Error: [$rootScope:inprog] $apply already in progress

This is more an angular thing than a grid thing.  Basically if you call from outside a digest cycle, then you need an apply.  If you call from within a digest cycle then you can't start another apply.
There are two options:

refactor so that you are always within an apply
it is possible to check the $$phase variable to see if an apply is already in progress.  This is a bit bodgy, but works.

Would it be simpler for AngularJS to ignore the apply if it an apply has already started?
This way the user will be able to trigger the function regardless it is being called internally or externally.
At the place where AngularJS throws that "$apply already in progress", it could simply ignores it.

Perhaps, but you'd have to ask the angularJS developers.  I presume they have a reason that they didn't think that was a good idea.

Hum... right, my bad, shouldn't raised this under ng-grid. Thank you for your fast reply and insight Paul! Regards.


Propose sem-ver mechanism for dNation helm chart projects

Investigate well know helm chart project like prometheus or grafana

Proposed mechanism:
https://git.ifne.eu/infra/maintenance/-/wikis/GitHub


Support Infineon's TLE5012B SPI encoder

The accuracy of the tle5012b encoder is 16 bit or 15 bit, and the performance is very good! His SPI was originally 3 wires, and the official added 4-wire library, which is convenient to connect! Can add this encoder may give me some advice on code transformation! esteem it a favor!
https://github.com/Infineon/TLE5012-Magnetic-Angle-Sensor

There is support for the TLE5012B for STM32 now:
https://github.com/simplefoc/Arduino-FOC-drivers/tree/master/src/encoders/tle5012b
And there is a new GenericSensor class:
https://docs.simplefoc.com/generic_sensor
with which you should be able to integrate the official library and just use Tle5012MagneticAngleSensor.getAngleValue(d); on the hardware platforms it supports.

Since we now support the TLE5012B, at least on STM32 MCUs, I am going to close this ticket. Please feel free to re-open it or create a new one if the solution we have isn't meeting your needs.


Ability to select fields while using report

Can you include the ability to select fields while using the custodian report. Currently there is no way a user can specify the fields required for output to csv.

sounds good, at the moment our reports have some org specific tag structure to them. i think we need both adhoc cli extension of attributes in the report, and some config based format. For column definition in the csv, i'd like to leverage a key=jmespath  style syntax. so
custodian report --field="size=Size"

as an optimization it would be nice to eval the expression against the record collection instead of individual records, but that might be premature and automatable (expr preprend [].)

@kapilt - implementation questions
I assume you could specify --field any number of times to include multiple fields (correct?).  Questions:

If a user specifies some number of fields using --field, should the set of fields included be limited to just what the user specifies, or are there any fields that should always be included, such as the instance ID?

e.g. If a user specifies --field foo and --field bar, the CSV will have two columns, foo and bar

The report currently includes information from the tags.  Should there be a way to specify fields from the tags as well?  This could be a different flag such as --tag-field="name" or we could have a special prefix in the field name that indicates it is a tag such as --field="tag:name"

i think we should retain the intrinsics unless a user specifies via option otherwise (--no-defaults), we also want to preserve order on the user specified options. we're currently got some org specific stuff on tags that needs pulling ideally as its org specific (OwnerContact, etc).

We should have helper syntax for tags, i'd go with --field="tag:name" there's some stuff.

Other stuff to consider but not required, putting the default format spec on the model for each type, it already includes id/name fields. reusing value filter for expression retrieval, probably overkill.

@kapilt
I've completed this work for the report subcommand on my local branch.  I'll commit and push it shortly and provide you a link in slack to review.  I didn't create a PR yet in case you wanted to change any of my approach which I detailed below.
--field support two types of entries:

Giving a prefix of tag: will search the tags for the provided key.  E.g. tag:CustomId will find the CustomId tag.
Any other expression will be evaluated as a JMESPath expression against the resources.json dictionary.

Fields are added in the same order they are specified on the command line, and they are added to the end of the CSV file, except if --no-default-fields is specified, in which case the default format spec is not included.
I did not allow for the size=Size style of syntax which you mentioned above.  I assume the first part is to set the header name?  I was worried about cluttering the user interface.  (Would the corresponding entry for tags be size=tag:MySize?  Would the default if no = sign is included be to just use the field name as the header name?  That's what I do now - I just use the name of the field for the header)  If you think this is essential its not hard to put in - I was just trying to keep the UI cleaner.
I do not have any unit tests yet, but based on the coverage I don't think there are any unit tests for the report subcommand.  Do you need tests before creating a PR?  Do you need tests for the entire report subcommand - and is that something that we should be working on?
As for the other things you wrote above - I'd need some more details.  Which model are you referring to in "putting the default format spec on the model for each type"?

@scotwk cool.
wrt to size=Size, i guess the issue is that jmespath expresssions can get quite complex and we need some sort of relabel for complex expressions, ie. here's one that got handed to me in slack yesterday "max([].length(TargetHealthDescriptions))" (app elb). Re clutter, i think its fine if its optional, but that can be a followup.
wrt to tests, definitely would appreciate a test before merge, in part because of the lack of them to date in the reporting code. if you want to send the pr for initial review comments prior that's fine as well.
wrt to models, 99% of the resources are based on skew library's models, potentially with augmentations that we do for various things, rather than having a bunch of formatters divorced of the model definitions, it would be nice to just co-locate those with the model. as an example see the resources/efs.py. ie. we have some generic name/id fields for any given resource there, putting the default format for a resource on to those will allow more ready reuse in different front ends. again its deferrable, just noodling on what ideal target looks like.

@kapilt
Good point on the complex expressions.  I'll add in the ability to set the header tonight and submit a PR.  I am traveling thru the weekend and don't have time to write tests, but I'll pick that back up next week.

@kapilt - Tests are added.  The PR is passing all tests.  I can look at moving the models next if you think that takes priority over other work.  But if you have users wanting this feature then I think this can be taken as-is and that can be a next step.

@kapilt - This has been released and I think this issue can be closed.

Yeah thanks @scotwk and @kapilt I have already tested this 👍


Override config from environment variables

Summary
Allow users to override config from environment variables.
Motivation
Allowing users to layer different sources of config over top of each other improves ergonomics with running fatigue in deployed environments and against multiple environments.

Potential candidate: figment


fix: declare GlobalComponents

close #3268, close #3265

Not the right way to fix...?

Hi @rchl! I have pushed a new commit. Could you please check it out if it works?

I don't know if it's the right approach. As I said, I don't think it's OK to import packages that might not exist but maybe TS is fine with it...

I don't know if it's the right approach. As I said, I don't think it's OK to import packages that might not exist but maybe TS is fine with it...

Yes, the original logic is to ignore errors through @ts-ignore. In this case the imported type will be any, but I don't know why in 5.1.3 the type is still any but the behavior changed.

Oh, I see now that the original code also imported those so I guess this is not any worse.


⚠ Bofrost.lu has degraded performance

In e9a87f4, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1752 ms

Resolved: Bofrost.lu performance has improved in 272a949 after 20 minutes.


foldl-1.2.0

foldl-1.2.0 is out of bounds for:

[ ] pipes-extras-1.0.2 (>=1.0.1 && < 1.2). Gabriel Gonzalez<EMAIL_ADDRESS>@Gabriel439
[ ] turtle-1.2.5 (==1.1.*). Gabriel Gonzalez<EMAIL_ADDRESS>@Gabriel439

Fixed by pipes-extras-1.0.3 and turtle-1.2.6


Add Audit Log Reason Support (#386)

This pull request adds audit log reason support (#386).
The new OptionalRequestHeaders struct can be extended to cover future headers that may want to be set in the future.
Also added versions of functions so that they take a reason argument.

Maybe I missed something, but I think you removed the ability to set the Days to delete at the GuildBanCreateWithReason method, don't see days used anywhere.

My bad, I changed the method to set the header instead of set query parameters to pass the reason for a little consistency and missed it.

IIRC there is a difference between the audit log reason header and the reason query param in behaviour
The audit log reason wont show in the ban/kick event for example, i'm gonna do a couple tests in a little bit to see if this is still the case

I did test if the setting the header would work for the ban method before I made the PR and it worked:

I also tested a few of the other methods like unban, role edit, and role create:

Yeah i think i remember wrong, disregard what i said.

ah yes i remember now, does it show up here aswell? in the bans?

It does not show up there; I will revert the ban change.
Do you think I should do the same for the GuildMemberDeleteWithReason method? As far as I know, the only place the reason for a kick shows up in is the audit logs, but seeing as how setting the header and passing the query parameter do not do exactly the same thing for ban, I think maybe it should be changed back, too.

This PR looks pretty good but it drastically increases the complexity of the API. What do you think about making the API:
session.SetNextRequestHeaders(headers OptionalRequestHeaders)

RequestWithBucketID accepts an interface for the data struct.
You could modify just the request and requestWithLockedBucket functions
to accept a *RequestOptions struct to avoid changing most of the API.
Then do a type check to decide how to treat the data.
// RequestOptions allows you to supply additional information to
// the request functions
type RequestOptions struct {
	Headers [][]string
	Body    []byte
}


Terminal Crashes on opening

Environment
Windows Specs: Windows 10 Pro
Version: 1909
OS build: 18363.720
Processor: Intel Core i5-4210U CPU @ 1.70GHz 2.40GHz
Installed RAM: 16 GB (15.9 usable)
System Type: 64-bit operating system, x64-based processor
Windows Insider Program: Slow (Recommended)
Use developer features: Developer mode
Windows build number: [Version 10.0.18363.720]
Windows Terminal version (if applicable): 0.10.781.0
Any other software?

# Steps to reproduce
Open Windows Terminal from start menu
<!-- A description of how to trigger this bug. -->

# Expected behavior
Open Windows Terminal with Powershell command prompt

<!-- A description of what you're expecting, possibly containing screenshots or reference material. -->

# Actual behavior
Error starting both powershell.exe and cmd.exe files with Windows Terminal.

![image](https://user-images.githubusercontent.com/3514724/79055484-d2d93e00-7c4d-11ea-9f61-22346b505a71.png)

![image](https://user-images.githubusercontent.com/3514724/79055497-e2f11d80-7c4d-11ea-811b-5a7c8066bf72.png)

<!-- What's actually happening? -->

Fixed this issue by downloading Ubuntu from the Windows store and installing it, then changing the Terminal config file to include a profile for WSL.
This also fixed opening cmd.exe and powershell.exe in the Windows Terminal
Close this issue


Files crashes during launch

Description
Files crashes after showing the splash screen.
Steps To Reproduce
Open files.
Expected behavior
Normally Files would open.
Files Version
Version <IP_ADDRESS>
Windows Version
Windows 11 22526.1000
Relevant Assets/Logs
Can't access.

Uninstalling and Reinstalling fixed the bug. It appears as if there is now a new version of Files that fixed that Bug.


New version: EmoteInteractive.RemoteMouse version 4.503

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

[x] Does your manifest conform to the 1.2 schema?

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

Closes #92695

Microsoft Reviewers: Open in CodeFlow

Hi @mdanish-kh,
This PR appears to remove the 4.502 version's defaultLocale file. Can this be reverted?

@stephengillie That is actually intentional as I deleted the previous manifest. They both contain the same InstallerUrl, so there is no need for 4.502 as it will always result in hash mismatch. wingetbot does this in two PRs (one for update and one for deletion). I did it one PR (update and remove).
All the files for 4.502 were deleted in this PR, but the other ones contained slight changes so GitHub is smart enough to show them as replaced and not deleted. I made major changes in the locale file, so GitHub is showing it as deleted and not replaced.

@stephengillie That is actually intentional as I deleted the previous manifest. They both contain the same InstallerUrl, so there is no need for 4.502 as it will always result in hash mismatch. wingetbot does this in two PRs (one for update and one for deletion). I did it one PR (update and remove).
All the files for 4.502 were deleted in this PR, but the other ones contained slight changes so GitHub is smart enough to show them as replaced and not deleted. I made major changes in the locale file, so GitHub is showing it as deleted and not replaced.

Unfortunately, removing one version and adding another counts as changing more than one manifest per PR. The removal of 4.502 will need a separate PR.

@stephengillie, This still counts as change to one manifest as I've replaced the previous one in-place. The limit for one manifest per PR is because the Azure Pipelines cannot validate more than one manifest in the same PR but the Pipelines also agree that this counts as change to one manifest as Pull-Request-Error label was not added (you can see in #90939, #90715 where the label was added by the pipelines).
The in-place upgrades can also be seen in #83397, #89713, #89712 and #89287 just to give a few examples of similar PRs (that seemingly change more than one manifest) that got validated and merged without an issue.


Added eternal mosaic

@evias for some reason the latest npm release does not have the latest changes. After this PR is accepted I'll launch a new version.

Pull Request Test Coverage Report for Build 59

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.732%

Totals

Change from base Build 57:
0.0%

Covered Lines:
51

Relevant Lines:
59

💛  - Coveralls


[Problem] Doesn't display choices from related table

Looks great, but when I call to a related model, it only shows the code:
Vocabulary in General                [u'10']
and it doesn't expand the inline Questionaire.
Installed:
Django 1.8.6,
Python 2.7,
Grappelli 2.7.1,
ckeditor 4.4.8 - used for Questionnaire,
MultiSelectField 0.1.3 - used for Summary
Here's my admin.py:
class EvaluationAdmin(admin.ModelAdmin):
    fieldsets = [
        ('Scheduling Info',    {'fields': [('candidate'), ('user', ), ('status'), ('source'), ('sourcename')]}),
        ("Questionnaire", {"classes": ("placeholder evaluationresponse_set-group",), "fields" : ()}),
        ("Feedback", {"classes": ("placeholder evaluationscore-group",), "fields" : ()}),
        ('Comments', {'fields': [('comments')]}),
        ('Summary', {'fields': [('grammar', ), ('conversation'), ('pronunciation'), ('confidence'), ('level')]}),
    ]

Here are my models:
class Evaluation(models.Model):
    xRATING = (
       ("E", "Excellent "),
       ("VG", "Very Good "),
       ("G", "Good "),
       ("R", "Regular "),
       ("N", "Needs Improvement "),
       ("P", "Poor "),
    )
    grammar= models.CharField(choices=xRATING, max_length=30, blank=True, null=True, verbose_name='Applied Grammar')

class EvaluationScore(models.Model):
    xSCOREMSGS1 = (
        (10, "Excellent amount of vocabulary."),
        (11, "Good amount of vocabulary.  Always room for improvement."),
        (12, "Needs to acquire more vocabulary to express ideas better."),
        (13, "Used some false cognates."),
        (14, "Does not have sufficient vocabulary to express ideas; used several false cognates or invented words, used Spanish."),
    )
   vocabulary_general= MultiSelectField(choices=xSCOREMSGS1, max_length=30, blank=True, null=True, verbose_name='Vocabulary in General')

And here's my form:
class EvaluationResponseform(forms.ModelForm):
    response = forms.CharField(widget=CKEditorWidget(),
    	label=mark_safe("<normal>

Are you currently living in a house or in an apartment?"), required=False,)

    class Meta:
        model = EvaluationResponse
        fields = ('response',)

Hi @mykljohn,
Thanks for reporting the issue.
I looked the MultiSelectField and as I noticed, it doesn't contain any widget when this field is readonly.
For example, if you add in the admin.py at the ModelAdmin of the EvaluationScore the attribute
readonly_fields = ('vocabulary_general',)
again the default admin app, without the admin_view_permission app shows this field weird:
Vocabulary in General [u'10']
Maybe, when this field is readonly calls some method for CharField which inherits and return a simple text. I think, a possible solution is to override the MultiSelectField and find the method which called when this field is readonly.
Please, let me know for any comments.


DataTable - Preserving sort arrow on default sort

Basic information

Tiller version: 1.7.3
Module: -
Link to the complete executable reproducer if available (e.g. GitHub Repo): -

Additional information

none

Enhancement description

Keep the sort arrow displayed when the default sort is selected and the defaultSortBy prop is set.

Current behaviour

The arrow indicating the initial sort is cleared after no other sort is set.

Wanted behaviour

Always display the arrow indicating the initial sort when it's not sorted by any other column.

Possible workarounds

none

@mnovaciccroz pls update description in english

@mnovaciccroz I'm not sure if I understand your issue. As I know defaultSortBy prop is used only for declaring how DataTable should be sorted in it's initial state and in that case arrow will be displayed. After, you can sort table in another direction by clicking on column or even disable sorting. And if sorting is disabled, arrow will not be displayed which is wanted behaviour.


battop exits when plugging AC back in

Running battop 0.2.2, installed via yay -S battop on ArchLinux 5.1.5-arch1-2-ARCH.
Battery detected as SMP DELL GPM0365.
Steps to reproduce:

launch battop
plug AC out.
wait more than ~2/3 seconds.
plug AC back in.
battop exits apparently gracefully (running with RUST_BACKTRACE=1 does not report anything).

stderr collected via -vvvvv:
2019-05-29T11:50:25+00:00 - ERROR - Error occurred: Battery(Error { source: Os { code: 19, kind: Other, message: "No such device" }, description: None })
Error: Battery(Error { source: Os { code: 19, kind: Other, message: "No such device" }, description: None })

Hey, @razielgn! I think I've managed to fix this issue, is there any chance you can test it out?
I was not able to reproduce it locally and emulating the ENODEV is kinda hard to do properly :)
Since you are using a fresh enough Arch, I think there would be issues with running this binary: battop-0.2.3-alpha.1.gz

I tried it and couldn't reproduce the reported issue! Kudos! :smiley:

Yay! Many thanks for such a quick feedback!
I'll publish the new version properly in a next hour or so (into AUR also)

Great! Thank you!

And done: https://aur.archlinux.org/packages/battop/


org_settings: Fix color on hover in the code playgrounds table.

This PR attempts to fix the white background color of table headers
in the code playground table on hover by changing the CSS for
hoevring and adding the table-sticky-headers class to make the
background color consistent with the other similar tables in the
settings section.

Testing plan:
GIFs or screenshots:

Could you please post screenshots of both panels in dark mode and light mode, to confirm that everything looks as expected? Thanks!

@alya Here is a visual for the change.

Thanks! Why is there a hover behavior on "Actions" in dark mode in that recording? I don't think there should be one, right?

For next time, it would be great to use some a video format that's scrollable/stopable for such a long recording, so I don't have to rewatch the whole thing if I missed something. :)

@alya Thanks for pointing out the bug and sorry for the inconvenience.
Here's a screen-recording after fixing this.
https://user-images.githubusercontent.com/77766761/158747380-7b41f262-29d0-4a1b-85d8-030ef1cfdef8.mp4

This is great; the revised implementation that just adds missing classes fully explains the bug. I did a quick audit to confirm you'd caught everything affected by either bug, and it seems so. Merged, thanks @somesh202!


Sholz/orderflow tests

lot of the changes are just formatting
will mark non draft once snapshots are working properly

passing


[pddl_parser] Fix support for disjunctive-preconditions

This fixes a typo which affected the support of disjunctive preconditions

Hi @MostafaGomaa
Could you add any test to check if disjunctive-preconditions are really working?
Thanks!!

Merged!! :rocket:


Uncaught Error: Pty seems to have been killed already

Issue Id: ff4d91a4-d71d-8fc0-ac95-52f56d1f61f0Versions - 1.26.0-insider (7/19/2018 5:16:52 AM)Stack Error: Pty seems to have been killed already    at WindowsPtyAgent.kill (\node_modules.asar\node-pty\lib\windowsPtyAgent.js:83:13)    at WindowsTerminal. (\node_modules.asar\node-pty\lib\windowsTerminal.js:144:26)    at WindowsTerminal._defer (\node_modules.asar\node-pty\lib\windowsTerminal.js:155:24)    at WindowsTerminal.kill (\node_modules.asar\node-pty\lib\windowsTerminal.js:139:14)/vs/workbench/parts/terminal/node/terminalProcess.ts#97:20

I can repro it by killing terminals on Windows when using 64 bit system setup (not user set up)


Sudo powers

It's very common to need sudo rights in scripts. We might consider a SudoExec method, for example, or perhaps just a Sudo filter which obtains sudo permissions, if possible, for all subsequent pipe operations.

Interesting idea, would love to have a crack at this.

Go right ahead @mainawycliffe!

Excellent. I will look into it over the week and then we can discuss my idea on the implementation with what you have in mind.

after further play with the gomarkdoc tool, it's possible to generate the complete read me, but it will require moving the package intro section of the read me as a package comment, it works for the most part, but it's not compatible with markdown, but can be adopted to be somewhat decent.


Not all code available in sample code to perform successful test

There's no DocumentDBRepository referenced in the sample code to know where that piece comes from or how to set it up local to emulate the documentation.

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

ID: 617f002c-f40e-28a0-2ab7-e3227e08e37c
Version Independent ID: a06ba438-55c8-e77c-ec89-095d44fbf4f3
Content: Set up CI/CD pipeline with the Azure Cosmos DB emulator build task
Content Source: articles/cosmos-db/tutorial-setup-ci-cd.md
Service: cosmos-db
GitHub Login: @deborahc
Microsoft Alias: dech

I'm looking for 2 things. the cs file for Item, and the cs file for DocumentDBRepository. Although you initialize it with that line. Where is the declaration and the definition of those 2 classes?

@SnehaGunda Would this get assigned to you or @deborahc? Thank you...

Most likely- https://github.com/Azure-Samples/documentdb-dotnet-todo-app/blob/master/src/DocumentDBRepository.cs &
https://github.com/Azure-Samples/documentdb-dotnet-todo-app/blob/master/src/Models/Item.cs

@darthmolen sent an email to Deborah to find if the project is hosted somewhere in Git.

@darthmolen and @sridharanprasanna apologies for the long delay in getting back, most of us were on vacation over the past few weeks. The code sample that Sridhar pointed to are right, we have extended the sample and added test cases for CI pipeline. I updated the code in the repo to include test cases as well and the doc to point to this code sample. The following are the two files that you need:

https://github.com/Azure-Samples/documentdb-dotnet-todo-app/blob/master/src/Models/Item.cs
https://github.com/Azure-Samples/documentdb-dotnet-todo-app/blob/master/src/DocumentDBRepository.cs

#please-close


New package: jopohl.UniversalRadioHacker version 2.9.4

- [X] Have you signed the Contributor License Agreement?

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

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

Resolves #111416

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

/azp run

/azp run


command "ntc" returns error

Hello,
when I use ntc it returns

env: node\r: No such file or directory

Is there any way I can solve this?
Im running mac Os Sierra.
Thanks

I just tested on Sierra and is works fine for me.
I am not sure if you are talking about another project as this one does not create a command called ntc is creates one called name-that-color

Oh, I am sorry. I don't know why I thought the command is ntc, instead of 'name-that-color'.
Problem solved, I guess You can close this issue :-) Thanks


Unwanted move and stop events after beforestart event

From version 3.2.0 I have move and stop events between beforestart and start.
Code

Seems to be caused in this line selection/blob/master/packages/vanilla/src/index.ts#L176 since false isn't passed as second argument, I'll roll out a fix later!


Provide a way to not treat some requirements files as references

Let's say I have the following set of files:
base.in:
-r external/deps.in
a

prod.in:
-r base.in
b

external/deps.in:
c

What I want is for the output to look like this:
base.txt:
c==1.2.3
a==4.5.6

prod.txt:
-r base.txt
b==7.8.9

In other words, external/deps.in is not an environment, so it should not produce a requirements file of its own. Anything it contains should just be treated as if it's a part of base.in.
As far as I know, there's no way to do this with pip-compile-multi at the moment.
This would be helpful to reduce the number of pip-compile runs, as well as to permit more leeway with the version constraints. For example, base.in could constrain a package more tightly than external/deps.in, which doesn't work if deps.in is treated as a reference.

Have you tried --auto-resolve flag?

I have now. It's useful for the second problem, but not so much the first - in fact it adds an extra pip-compile run, so the whole thing becomes even slower.

The versions are resolved only on the first run. All subsequent runs are faster because they re-use the constraints from the first call.

In my practice, each run still takes a really long time, so the fewer of them, the better.
Plus, purely from a conceptual standpoint, I don't consider external/deps.in to be an environment, so I don't need or want to have a corresponding external/deps.txt.

--use-cache can improve compilation time significantly.
I don't follow the concept behind external/deps.in. It's a requirements file, and it's referenced from the base.txt. What's the issue with its existence?


High "latency" in "Received block"

My built version of nodeos (master branch): a97a49a0dd4f5051dc7de1de4bc654fedf49cadf
I've run the script ${EOS_INSTALL_DIR}/tutorials/bios-boot-tutorial/script_bios_boot.sh to deploy a testnet on some virtual server.
After that, I start a local nodeos that gets connected to this remote testnet.
Everything works fine.
However, after a long while, the local nodeos seems to be running in a situation that the latency value in Received block is very high. Consequently, all transactions that go via this local nodes are expired.
0|script_c | 2449775ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block fb6ac5f36e2513c9... #8000 @ 2018-06-19T05:31:19.000 signed by producer111d [trxs: 0, lib: 7865, conf: 0, latency: 198570775 ms]
0|script_c | 2460079ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block 6e46f9724a80e6e5... #9000 @ 2018-06-19T05:39:39.000 signed by producer111g [trxs: 0, lib: 8861, conf: 0, latency: 198081079 ms]
0|script_c | 2470377ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block 640c594a3d7690b9... #10000 @ 2018-06-19T05:47:59.000 signed by producer111j [trxs: 0, lib: 9857, conf: 0, latency: 197591377 ms]
0|script_c | 2481194ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block aefb8281daa941cf... #11000 @ 2018-06-19T05:56:19.000 signed by producer111d [trxs: 0, lib: 10865, conf: 0, latency: 197102194 ms]
0|script_c | 2491786ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block 49b7d5cb2d435ce9... #12000 @ 2018-06-19T06:04:39.000 signed by producer111g [trxs: 0, lib: 11861, conf: 0, latency: 196612787 ms]
0|script_c | 2505076ms thread-0   producer_plugin.cpp:290       on_incoming_block    ] Received block 88971c698b567def... #13000 @ 2018-06-19T06:12:59.000 signed by producer111j [trxs: 0, lib: 12857, conf: 0, latency: 196126077 ms]

This kind of situation will be resolved only if I stop the currently-running testnet and re-run the script ${EOS_INSTALL_DIR}/tutorials/bios-boot-tutorial/script_bios_boot.sh
I guess that the root cause is because the local nodeos is downloading a huge amount of produced blocks.

The latency reported is from the time the block was created, not network latency.

Sometimes negative value is observed for latency. Could this be due to the clock on the producer being ahead than the the receiver node?
info  2019-04-24T14:54:23.467 thread-0  producer_plugin.cpp:345       on_incoming_block    ] Received block c0bfc5dd5160ef18... #54664349 @ 2019-04-24T14:54:23.500 signed by eossv12eossv [trxs: 18, lib: 54664013, conf: 0, latency: -32 ms]

Producers can configure a block to be released ahead of its scheduled time which will show as a negative latency.


Unit Testing Infrastructure Upgrade

Addresses #46, #196, and #202.
Summary of Changes:

Restructured test_sw to use test data rather than download (#196).

Added relevant kp and f107 data to test_data directory
Unit tests use a fixed test_day rather than today for testing
Added test_data_path to pysat.init
Removed skip_tests, tempfile notation.

Reduced code duplication by adding functionality to pysat testing instruments (#202).

Added tags to pysat_testing to change the default behaviour of dummy1.
Old tag behaviour (total number of data points) for numeric strings migrated to sat_id
Various tests invoking numeric string as tag updated.
testaddx and testsmall instruments removed (replaced with pysat_testing)
_generate_fake_data function added to remaining 4 testing instruments (#46).
Updated test_constellation objects to use tag structure to determine dummy1 variable

General bug and style fixes

Fixed a bug in _meta.keep
Updated style in test_omni_data to make more readable
Updated some tests in test_ssnl_occur_prob to be more resilient

This is the second half of the testing infrastructure upgrade I started in #192.  Travis CI tests are down to 40 min.

Coverage increased (+1.4%) to 81.073% when pulling eb00ac220b4be7067b75ecebce8b49c8f634a84f on test_sw into 145a00a853fe5fd25cca0603baf9b5948ece5047 on develop.

Open Issue:  The calculate_imf_steadiness routine in omni_hro.py looks like the test object should be an omni_hro instrument, but unit tests currently use a pysat_testing object.  Not sure if I am interpreting the function correctly.

Another open issue: _generate_fake_data could be placed in a common area like utils rather than in every test file.

New updates to clean up code:

generate_fake_data has been moved to testing_methods.py to further reduce duplication.  This will be moved to methods.testing as part of #214.

Note that ~26,462 of the 26,769 added lines are the test dataset added to speed up the test time.

Thanks for all of the code!


Standardize file names

Modified file names of html-cheatsheet, java-cheatsheet, and numpy-cheatsheet to make them match the existing file name scheme.

This can't be accepted as a valid PR, Try adding some more cheatsheets/ updating the previous ones with new important commands.


[QUERY] How to download public key/certificate from Azure App Services Certificates

Library name and version
Azure.ResourceManager.AppService v1.0.0
Query/Question
I want to download every public key/certificate from Azure App Service Certificates. AppServiceExtensions.GetAppCertificates() only returns certificate metadata such as thumbprint, issuer, and expiration data. Per the /certificate endpoint documentation, certificate data is not available using the GET method. Is there another way to consume App Service Certificates aside from what's detailed in this article?
Environment
Platform:
.NET SDK:
 Version:   7.0.100
 Commit:    e12b7af219

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19044
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\7.0.100\

Host:
  Version:      7.0.0
  Architecture: x64
  Commit:       d099f075e4

.NET SDKs installed:
  7.0.100 [C:\Program Files\dotnet\sdk]

.NET runtimes installed:
  Microsoft.AspNetCore.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 3.1.31 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.11 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 7.0.0 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Other architectures found:
  x86   [C:\Program Files (x86)\dotnet]
    registered at [HKLM\SOFTWARE\dotnet\Setup\InstalledVersions\x86\InstallLocation]

Environment variables:
  Not set

global.json file:
  Not found

Learn more:
  https://aka.ms/dotnet/info

Download .NET:
  https://aka.ms/dotnet/download

IDE version:
Visual Studio 17.4.1

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'App Services:0.39276415,Storage:0.055857815,Extensions:0.044107005'

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Added Service Attention label as this should be answered by the service team and SDK doesn't have too much domain knowledge here.


Question about VideoConvMAE

Thank you for your impressive work! VideoConvMAE seems still lack a code release, can you update it?

Thanks for your interest in ConvMAE. We will try our best to release VideoConvMAE codebase within two week.

Sure, btw, can you release the code of backbone before the official version? That would be very helpful.

Hello, do  you use this model later?


Enable latitude / longitude searches

Hi, thanks for creating the plugin. It's really useful.
One thing that is limiting the use for me is that I HAVE to specify an address and cannot just enter latitude and longitude. This in no issue for locations in cities, but rather locations in rural areas or national parks that don't have an address. I would appreciate if you'd consider adding this feature. Thanks!

Hi, thanks! That's a great suggestion, I'll work on that in the next few days.


[14.0] ISPN-14492 Java 17 instead of 11

https://issues.redhat.com/browse/ISPN-14492

Merged, thanks


Symbol clash between Scipy and libsndfile

There is a symbol clash between scipy and libsndfile:
Scipy declares "sf_error" as follow (see https://github.com/scipy/scipy/blob/master/scipy/special/sf_error.h):
void sf_error(const char *func_name, sf_error_t code, const char *fmt, ...);

While libsndfile declare this function as follow (see https://github.com/erikd/libsndfile/blob/master/src/sndfile.c):
int sf_error (SNDFILE *sndfile);

What is the purpose of sf_error in Scipy? It looks like Scipy tries to hack the real "sf_error" function.
Of course having twice the same symbols coming from 2 different libraries create crashes.
Reproducing code example:
An application liked with libsndfile in which we import scipy and then start the unit tests (scipy.test())
Error message:
Crash.
Python version 3.5.2 |Enthought, Inc. (x86_64)| (default, Mar  3 2017, 18:53:01) [GCC 4.4.7 20120313 (Red Hat 4.4.7-17)]
nose version 1.3.7
...............................................................................................................................................................................................F..........................................................................................K.....................................................................................................................................................................................................................................................................................................................................K.............................................................................too many axes: 2 (effrank=2), expected rank=1
too many axes: 2 (effrank=2), expected rank=1
.....K...............F.......................................................................................................................................................................................................................................Segmentation fault (core dumped)

#0  0x000000385ec06f5f in sf_error () from /usr/lib64/libsndfile.so.1
#1  0x00007f3d1c05e6c1 in beta_negint (a=<value optimized out>, b=<value optimized out>) at scipy/special/cephes/beta.c:242
#2  0x00007f3d1c05e62f in cephes_beta (a=<value optimized out>, b=<value optimized out>) at scipy/special/cephes/beta.c:69
#3  0x00007f3d1bfec36d in __pyx_f_5scipy_7special_15orthogonal_eval_binom (__pyx_v_n=<value optimized out>, __pyx_v_k=<value optimized out>)
    at scipy/special/_ufuncs.c:31455
#4  0x00007f3d1bfea4d3 in __pyx_f_5scipy_7special_7_ufuncs_loop_d_dd__As_dd_d (__pyx_v_args=<value optimized out>, __pyx_v_dims=<value optimized out>, __pyx_v_steps=
    0x3960520, __pyx_v_data=<value optimized out>) at scipy/special/_ufuncs.c:9174
#5  0x00007f3d23f06aeb in iterator_loop (ufunc=<value optimized out>, op=<value optimized out>, dtype=<value optimized out>, order=<value optimized out>,
    buffersize=<value optimized out>, arr_prep=0x2, arr_prep_args=0x0, innerloop=0x7f3d1bfea470 <__pyx_f_5scipy_7special_7_ufuncs_loop_d_dd__As_dd_d>, innerloopdata=
    0x7f3d1c37b070) at numpy/core/src/umath/ufunc_object.c:1504
#6  0x00007f3d23f0a696 in execute_legacy_ufunc_loop (ufunc=<value optimized out>, args=0x7f3c00000001, kwds=0x2000, op=0x7ffe6653bba0)
    at numpy/core/src/umath/ufunc_object.c:1659
#7  PyUFunc_GenericFunction (ufunc=<value optimized out>, args=0x7f3c00000001, kwds=0x2000, op=0x7ffe6653bba0) at numpy/core/src/umath/ufunc_object.c:2625
#8  0x00007f3d23f0aebf in ufunc_generic_call (ufunc=0x2dee040, args=0x7f3cf9d35288, kwds=0x0) at numpy/core/src/umath/ufunc_object.c:4250
#9  0x00007f3d460328b6 in PyObject_Call (func=0x2dee040, arg=<value optimized out>, kw=<value optimized out>) at Objects/abstract.c:2165
#10 0x00007f3d460f3a27 in do_call (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:4936
#11 call_function (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:4732
#12 PyEval_EvalFrameEx (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:3236
#13 0x00007f3d460f5f8e in _PyEval_EvalCodeWithName (_co=<value optimized out>, globals=<value optimized out>, locals=<value optimized out>,
    args=<value optimized out>, argcount=1, kws=<value optimized out>, kwcount=0, defs=0x0, defcount=0, kwdefs=0x0, closure=0x0, name=0x7f3cf9fbc9f0, qualname=
    0x7f3cf9faef90) at Python/ceval.c:4018
#14 0x00007f3d460f4b9b in fast_function (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:4813
#15 call_function (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:4730

Scipy/Numpy/Python version information:
SciPy version 0.19.0
Python version 3.5.2

https://github.com/scipy/scipy/issues/8130

In short --- your program is loading dynamic libraries in a way that
they see each other's symbols. Not a bug in scipy.

libsndfile is a system library to play sound. How can I use Scipy and play sound at the same time?
A fix could be to prefix the "error" function by some more letters than only "sf". Indeed, "error" is quite a used token...

There's no principal problem having the same symbol name in two
different shared libraries. The problem is probably that you are
embedding Python in your application in a way that results to mixing up
symbols in the application and symbols from dynamic libraries loaded by
Python.

Yes, the issue comes from the fact that I link my program with 2 shared libraries:

One in charge of playing sound (libsndfile)
One in charge in computing (Scipy)
Both of the them declare le same symbol "sf_error".
If I first load Scipy, the application will crash as soon as the the library in charge of playing sound calls "sf_error".
If I first load libsndfile, the application will crash as soon as the library in charge of doing computation calls "sf_error".

PS: it's OK to have the same symbol defined twice if the signature is the same. This is not the case here.
Of course in addition to the signature the the function must to the same job from the 2 different libraries. I mean fulfill the contract it is supposed to 2 within the 2 different implementations.

That the signatures differ does not matter when the shared libraries are loaded with the correct visibility settings. If you are embedding Python in your program, you can arrange this is so.

Closing (duplicate of gh-8130)

That the signatures differ does not matter when the shared libraries are loaded with the correct visibility settings. If you are embedding Python in your program, you can arrange this is so.

I would be happy to know how. Could you detail?
Imagine the following scenario:

User launches a computation
The computation raises an error which is output through Scipy through "int sf_error (SNDFILE *sndfile);" (scipy is loaded first so the sf_error in memory is the one of Scipy so, it's OK.
After the output of the error, a sound is played to notify the user by a sound
While playing the sound, an error is raised by libsndfile (file is corrupted, not accessible, sound card is not accessible, whatever)

Don't you think it will raise a crash of the application? The system will not load the "sf_error" of the libsndfile because a symbol with the same name is already in memory.
Please let me know how "If you are embedding Python in your program, you can arrange this is so".
What you said in the other topic "These symbols however could be marked not exported (and generally so also for many other symbols), so that such workarounds are not needed." would be another real fix. Another one would be to rename "sf_error" to "scipy_sf_error" or something like that if symbol export is mandatory.


add bare environment CI tests

[x] Closes #727
[x] I am familiar with the contributing guidelines.
[x] Fully tested. Added and/or modified tests to ensure correct behavior for all reasonable inputs. Tests (usually) must pass on the TravisCI and Appveyor testing services.
[x] Updates entries to docs/sphinx/source/api.rst for API changes.
[x] Adds description and name entries in the appropriate docs/sphinx/source/whatsnew file for all changes.
[x] Code quality and style is sufficient. Passes LGTM and SticklerCI checks.
[x] New code is fully documented. Includes sphinx/numpydoc compliant docstrings and comments in the code where necessary.
[x] Pull request is nearly complete and ready for detailed review.

Ready to merge. Test failures are due to get_psm3.


It is easy to trigger null pointer exception

https://github.com/Netflix/eureka/blob/4d36f04849541df6c6b8738622544d1b26dfe8d4/eureka-core/src/main/java/com/netflix/eureka/util/StatusUtil.java#L59-L61
https://github.com/Netflix/eureka/blob/4d36f04849541df6c6b8738622544d1b26dfe8d4/eureka-core/src/main/java/com/netflix/eureka/util/StatusInfo.java#L105-L107

If the first step is not executed during construction, the method calling the second step will throw a null pointer exception. Can you change it to:
public boolean isHealthy() {       return  isHeathly == null? false:isHeathly.booleanValue();   }

SGTM, feel free to send a PR.

Push to that branch?Master?

You won't be able to push to origin/master, need to fork and send a PR which I see you did.


[flutter_plugin_tools] Include examples in test

Includes any Dart unit tests for examples when running Dart unit tests
via test.
Also updates the deprecated dart pub run test to dart run test.
Fixes https://github.com/flutter/flutter/issues/102845
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

Infra seems to be a little bit messed up:
            > Could not GET 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/3.3.0/gradle-3.3.0.pom'. Received status code 502 from server: Bad Gateway

Everything is passing now, but main is in a small pickle with dependabot


TypeError: func is not a function

Sorry I get stuck for a while already so i would really love to have a hand on this issue.
serverless-webpack works when i run sls deploy to the cloud but not for the local server.
I guess i must have missed something.
$ sls webpack serve
Serverless: Serving functions...
  GET - http://localhost:8000/test
TypeError: func is not a function
  at /home/eric/development/first-serverless/node_modules/serverless-webpack/lib/serve.js:114:7
  at Layer.handle [as handle_request] (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/layer.js:95:5)
  at next (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/route.js:131:13)
  at Route.dispatch (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/route.js:112:3)
  at Layer.handle [as handle_request] (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/layer.js:95:5)
  at /home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:277:22
  at Function.process_params (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:330:12)
  at next (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:271:10)
  at jsonParser (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/body-parser/lib/types/json.js:103:7)
  at Layer.handle [as handle_request] (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/layer.js:95:5)
  at trim_prefix (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:312:13)
  at /home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:280:7
  at Function.process_params (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:330:12)
  at next (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:271:10)
  at expressInit (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/middleware/init.js:33:5)
  at Layer.handle [as handle_request] (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/layer.js:95:5)
  at trim_prefix (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:312:13)
  at /home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:280:7
  at Function.process_params (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:330:12)
  at next (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:271:10)
  at query (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/middleware/query.js:44:5)
  at Layer.handle [as handle_request] (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/layer.js:95:5)
  at trim_prefix (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:312:13)
  at /home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:280:7
  at Function.process_params (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:330:12)
  at next (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:271:10)
  at Function.handle (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/router/index.js:176:3)
  at EventEmitter.handle (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/application.js:173:10)
  at Server.app (/home/eric/development/first-serverless/node_modules/serverless-webpack/node_modules/express/lib/express.js:38:9)
  at emitTwo (events.js:87:13)
  at Server.emit (events.js:172:7)
  at HTTPParser.parserOnIncoming [as onIncoming] (_http_server.js:528:12)
  at HTTPParser.parserOnHeadersComplete (_http_common.js:88:23)

my package.json file
{
  "name": "first-serverless",
  "version": "0.0.1",
  "description": "First Serverless",
  "main": "handler.js",
  "dependencies": {
    "babel-runtime": "6.11.6",
    "babel-polyfill": "^6.16.0"
  },
  "devDependencies": {
    "aws-sdk": "^2.6.6",
    "babel-core": "^6.17.0",
    "babel-loader": "^6.2.5",
    "babel-plugin-transform-runtime": "^6.15.0",
    "babel-preset-es2015": "^6.16.0",
    "bluebird": "^3.4.6",
    "serverless-webpack": "^1.0.0-rc.2",
    "webpack": "^1.13.2",
    "webpack-node-externals": "^1.4.3"
  },
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "Eric Chan",
  "license": "ISC"
}

webpack.config.js
var nodeExternals = require('webpack-node-externals');

module.exports = {
    entry: [
        'babel-polyfill',
        './handler.js'
    ],
    module: {
        loaders: [{
            test: /\.js$/,
            loaders: ['babel'],
            include: __dirname,
            exclude: /node_modules/,
        }]
    },
    target: 'node',
    externals: [nodeExternals()]
};

serverless.yml
service: aws-nodejs

provider:
  name: aws
  runtime: nodejs4.3

functions:
  hello:
    handler: handler.hello
    events:
      - http:
          path: test
          method: get

plugins:
  - serverless-webpack
custom:
  webpackIncludeModules: true

handler.js
export function hello (event, context, cb) {
    cb(null, { message: 'Go Serverless v1.0! Your function executed successfully!', event });
}

works after removing node_module and npm install...
pls close this ticket...

Interestingly... 2nd call works...
But if i make any changes in the code and save it, the server will be terminated with the following Error.
It means I need to rerun sls webpack serve everytime when i have changes and I need to call the endpoint twice to see the expected result.
/home/eric/development/webpack-serverless/node_modules/babel-polyfill/lib/index.js:10
  throw new Error("only one instance of babel-polyfill is allowed");
  ^

Error: only one instance of babel-polyfill is allowed
    at Object.<anonymous> (/home/eric/development/webpack-serverless/node_modules/babel-polyfill/lib/index.js:10:9)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.defineProperty.value (/home/eric/development/webpack-serverless/.webpack/handler.js:55:19)
    at __webpack_require__ (/home/eric/development/webpack-serverless/.webpack/handler.js:20:30)
    at Object.<anonymous> (/home/eric/development/webpack-serverless/.webpack/handler.js:47:2)
    at __webpack_require__ (/home/eric/development/webpack-serverless/.webpack/handler.js:20:30)
    at /home/eric/development/webpack-serverless/.webpack/handler.js:40:18
    at Object.<anonymous> (/home/eric/development/webpack-serverless/.webpack/handler.js:43:10)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at ServerlessWebpack.loadHandler (/home/eric/development/webpack-serverless/node_modules/serverless-webpack/lib/run.js:19:20)
    at Watching.handler (/home/eric/development/webpack-serverless/node_modules/serverless-webpack/lib/serve.js:24:39)
    at Watching._done (/home/eric/development/webpack-serverless/node_modules/webpack/lib/Compiler.js:81:7)

I get that error if you call the endpoints before webpack has finished. I normally wait 30seconds after starting to ensure webpack has finished.

I followed the babel example and I am also getting this error. 2nd call doesn't work for me

Fixed this here e07ab1dcfac592c28f9fa9c7c6f4a64b0dff0fa3 basically the function reload code is not great (because it doesn't provide a fresh node environment when reloading functions). The fix proposed is more of a workaround of conditionally loading babel-polyfill in the handler rather than the webpack config.
There might be better way to reload dependencies but this should hold for now.

this solution requires the addition of babel-polyfill import to all files if multiple entry points are used.


Roll Flutter Engine from f4fbabf1eb9f to b08de44649b7 (5 revisions)

https://github.com/flutter/engine/compare/f4fbabf1eb9f...b08de44649b7
2024-02-01<EMAIL_ADDRESS>Roll Skia from 19e5e8f089b2 to bcbc172f74da (2 revisions) (flutter/engine#50226)
2024-02-01<EMAIL_ADDRESS>Ignore CRLF endings when determining the names of header guards. (flutter/engine#50227)
2024-02-01<EMAIL_ADDRESS>[Impeller] make GPU tracing off by default. (flutter/engine#50215)
2024-02-01<EMAIL_ADDRESS>Multiview: Add view ID to _render and remove render rule skipping (flutter/engine#50220)
2024-02-01<EMAIL_ADDRESS>Remove migration flag and unused header files (flutter/engine#50216)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC matanl@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Add "robust" flag to the bit-stream

Instead of having OD_ROBUST_STREAM in internal.h

My patch to fix this issue is one its way for review!

I have had a look at your the patch for this issue.  My comments can be found in reitveld:
https://review.xiph.org/758/


Add Thomas Falk to Contributors list

I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

I just wanted to thank you for your work, it helped me a lot getting started and it inspiried me to dig deeper in git and github ( in open-source overall)
Greetings and best wishes 8-Ball

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


LSP Find all references does not find references in assembly

Would be also nice to implement Go to definition and Go to type definition for inline assembly external references

Fixed in #102.


bug(FirebaseList): Add ref.off to listeners

Right now the listeners registered inside of FirebaseList are never unsubscribed.

Fixed by 9b28d843793312248f25627cb692ac2716934c06


[YAML] 'log_level' is deprecated parameter name. use '@log_level' instead // parameter '$log_level' is not used

Describe the bug
log_level on plugin scope can't be correctly defined in yaml config, without fluentd throwing a warning and/or the parameter not being recognized
To Reproduce
# Config 1
- match:
    $tag: '**'
    $type: stdout
    log_level: debug

results in:
[warn]: #0 'log_level' is deprecated parameter name. use '@log_level' instead.

since @ is not supported in yaml and every other instance of it has been replaced by $, looking at the fluentd documentation, I've also tried:
# Config 2
- match:
    $tag: '**'
    $type: stdout
    $log_level: debug

however this results in:
[warn]: parameter '$log_level' in <match **>
...
</match> is not used.

Expected behavior
I expect Config 2 not to throw a warning message and instead set the log_level at plugin scope to the according value.
Your Environment
- Fluentd version: 1.16.2
- TD Agent version:
- Operating system: Ubuntu 22.04 LTS
- Kernel version: 5.15

Your Configuration
see To Reproduce

Your Error Log
see To Reproduce

Additional context
No response

@ng-bsy Thanks for your report!
It seems that YAML format fails to take log_level setting into account.

#3712

We will check it.

We have checked this report and it appears that YAML format doesn't support the $log_level.
We also confirmed that using log_level instead of $log_level results in warn messages, but the log_level setting works fine.
Although this may be improved by future modifications, you can use log_level (without $) for log_level setting for now.


fix: remove duplicated close buttons on mobile popups

Summary
Since the popup drawer on mobile has got a close button in general, some popups now have two close buttons.
Changelog
- Removed duplicated close buttons on backup and password popup

Relevant Issues
Testing
Platforms

Please select any platforms where your changes have been tested.

Desktop

[ ] MacOS
[ ] Linux
[ ] Windows

Mobile

[x] iOS
[x] Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...
Checklist

Please tick all of the following boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

The close button on Password is not on Figma design. There is a cancel button but is already implemented on desktop and we sould use the same.
We needed only a close button on Network status drawer. So I thinl Password should be restored to previous state.


1.0.10 and Deploy New Contracts

Pull Request Test Coverage Report for Build 8383

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 8374:
0.0%

Covered Lines:
1474

Relevant Lines:
1474

💛  - Coveralls


CRN Sprint Fixes

added DWI file stubs to ds005 demo
fixed crash in get_nearest_helper when no entries found
fixed bval/bvec getters
added test for bval/bvec getters

Test output:
$ pytest
======================================== test session starts ========================================
platform darwin -- Python 3.6.2, pytest-3.2.1, py-1.4.34, pluggy-0.4.0
rootdir: /Users/greg/code/gkiar/pybids, inifile:
collected 10 items

bids/grabbids/tests/test_grabbids.py ..........

===================================== 10 passed in 5.41 seconds =====================================

@adelavega and I modified regex to properly type top-level files that do not have an underscore (i.e. dwi.bval and dwi.bvec)
Having these files properly typed allows us to use the get_nearest_helper function again.

@gkiar do you mind including a fix for #60 in this PR, since you're already hacking on the JSON config? Totally fine if not, we can merge this and then fix that later.

Happy to do it @tyarkoni - made a comment on the issue to clarify what we think makes the most sense before I implement anything

@tyarkoni updates made based on discussion 😄

Awesome, thanks! The tests are still failing on Windows, but since this was a pre-existing bug that's only causing tests to fail now because of the new test, I'll merge anyway.

But now tests will start failing for all new PRs causing people to start ignoring them...

Good point. But this is on my agenda for tomorrow to fix. Plus I'm inspecting each PR pretty closely, and I'm paying more attention to the travis results than the appveyor results, so we should be okay...

Cool - fingers crossed, but if this cannot be fixed soon we should disable tests on windows. The noise in tests results will lead to poor experience for contributors.

If it isn't fixed soon, we should probably say pybids isn't supported on Windows :p

Yes if it takes too much effort to support Windows we should drop it.