feat: track invalid headers hit counter

Closes #3327
keeps track how often an invalid header was Hit.
remove the entry from the cache if that number exceeds a threshold, to allow for reprocessing, this will make false positive inserts non fatal

it's this check:
https://github.com/ethereum/go-ethereum/blob/2754b197c935ee63101cbbca2752338246384fec/eth/catalyst/api.go#L54-L57
this is a safety mechanism so that if the CL continues to send us the same block over and over again we will remove it from the invalid set and try again


URL points to malloc rather than ralloc docs

The repository's URL points to https://doc.redox-os.org/doc/malloc/ when it should be https://doc.redox-os.org/doc/ralloc/

Thanks, fixed.


Recent redesign broke API?

New user here, did not use API before T212 redesigned website, so can't be sure that is why API fails for me - whenever I try to get the result of a ticker, I get error:

r = act.result("USCR")
File "/home/user1/.local/lib/python3.8/site-packages/Trading212/api.py", line 179, in result
return float(self.position_info(stock, "ppl"))
File "/home/user1/.local/lib/python3.8/site-packages/Trading212/api.py", line 188, in position_info
return self.driver.find_elements_by_xpath(
IndexError: list index out of range

I did a little experimenting with api.py, and noticed that the section with this label is throwing an exception:

# Navigate to the old app if the new app is opened by default

So, maybe API is ending up on the new site, and can't get to the "old" site? Makes sense that that breaks the code. But the pre-redesign site is still available, at https://live.trading212.com/legacy. But I don't know Selenium to experiment.

Hey, Sorry for the late response. I know that the API is somewhat broken now, But I don't have the time to fix it.
I have to close the issue for now. If someone wants, He's welcome to contribute to the project and fix it.


Add profile and hacktoberfest stats link

You need to update your fork first and then make changes.
Refer to these -
Configuring a remote for a fork
Syncing a fork

@Man-Jain please resolve the conflicts

@maddhruv Done


Use torch setting

Hey,
I'm trying to use the torch in my app using ZXing. This works for me.
But now people are requesting a feature which is also available in the ZXing android app, which is the setting to set Torch setting to On/Off/Automatic.
Is this possible?
Kind regards,
Jeroen Korsten

Unfortunately I too have had issues with the torch settings on the latest version of Zxing.net.mobile and my Samsung Galaxy S7 running API 24.
No matter what I tried Zxing gave me a false result for ZXingScannerPage.HasTorch.
I decided to determine how to do this myself within the droid xamarin project using DI from the PCL.  It seems that the Android.Hardware.Camera class is obsolete/deprecated in later versions of android and we have to now use Android.Hardware.Camera2
It's a nice simple call using the new method - create an instance of CameraManager then using that you can issue CameraManager.SetTorchMode("0", true)
Loaded my app, activated the torch hit the scan button to bring up the ZXingScannerPage created earlier.. flash goes off!!
Investigating the code it appears that this Zxing library implements the deprecated Android.Hardware.Camera class which has limited (seems ok turning torch off) functionality on later versions of the Android OS.
ZXing.Net.Mobile/Source/ZXing.Net.Mobile.Android/CameraAccess/Torch.cs
It may be possible to issue a pull request, I'm quite new to github so not fully sure how that works.  If we decide to use it ourselves I'm considering forking my own copy of the code to implement both versions of the camera/torch api.

Thanks for reporting this issue!  Unforunately it took me way too long to respond 😭.  Sorry, I apologize!  Recently the source code for this project was completely refactored to modernize it.  Many PR's were included in this effort, and many bugs were hopefully fixed.  Please try out the latest 3.x series of NuGet packages (currently in prerelease).  To try and make the project more maintainable in my spare time going forward, I've decided to close all existing issues to start with a clean slate.  If you're still experiencing this issue on the newest version, please open a new issue with as much detail as possible.  Thank you for your patience and understanding!  Happy scanning!


Change font families

Use Linux Libertine instead of Latin Modern.

So, the most recent version makes a difference of 25 pages vs 27 pages. To spell out the obvious, for this to work you need the right truetype Linux Libertine fonts installed, here(arch) it's called ttf-linux-libertine.
@jngrad A minor thing: In your Makefile, in the non-PDF/A branch I encountered this error:
Package pdfx Error: CreationDate is not properly supported;
a bit more over here: https://github.com/ndlib/nddiss/issues/4
But it could also have been me, having outdated build files...
To summarise:
I like the new look better.

@jngrad A minor thing: In your Makefile, in the non-PDF/A branch I encountered this error:
Package pdfx Error: CreationDate is not properly supported;

Good catch! I'll fix that in a separate PR.


GitOps UI tries to authenticate user with wrong git host provider

Describe the bug
Given I've added an app from a gitlab repo to my cluster, if I try to authenticate myself to view app commit list, I'm taken to GitHub auth page. Since my app repo is originating from GitLab, the auth process fails and I'm not able to see the app commit list.
What is the severity of the bug

severity/Major
Weave Gitops functionality is broken, there is a workaround, and the workaround requires significant effort

Environment

gitops: [v0.5.0]

To Reproduce

Run `gitops install --config-repo=git@gitlab
Run gitops add app --config-repo=<git@gitlab> --url=<git@gitlab>
Run gitops ui run
Go to UI page and see app listed
Click on app link and see app status page
At the bottom of the page, see button Authenticate with GitHub
As the button title states, auth flow will be from github which is irrelevant for a gitlab workflow

Expected behavior
Ui should detect which git host provider is being used and provide relevant authentication method.
Screenshots

@rokshana-b and @jmickey, I won't both github and gitlab manually tested before we merge this story.


Fix typo on House of Tangerine POC

Thanks for pointing it out.

@gfelber, I think the fix is correct so I merged it already. But can you please check whether this is what you meant?

oh yeah my mistake, btw i guess we should change this for all versions then?

@gfelber I already updated all the versions at https://github.com/shellphish/how2heap/commit/702114479b53f0b708ad9835b834c0a24da4f490
Thanks for the confirmation!


Adding k8 28 support for powermax

Is this a new chart?
No
What this PR does / why we need it:
Adding k8 28 support for powermax
Updating helm/driver version
Which issue(s) is this PR associated with:

https://github.com/dell/csm/issues/947

Special notes for your reviewer:
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] Chart Version bumped
[ ] Variables are documented in the chart README.md
[ ] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

Please update the chart.yaml and values.yaml files of container storage modules chart.


Help to configure my index

Hi, something weird is happening so I`m not sure if I need to configure something more I made all kinds of tests, I will really appreciate if anyone can help me:
Just as an example I have this model:
| name | cientific_name |
| Chinche Apestosa | Diaphania Nitidalis |

When i search for example queries like:  apestosa it returns the model, the same if I search for  Diaphania, Nitidalis,  even if I write bad some words for example: Nitidlais (wrong char order) but if I search chinche (the first work of the name) I cannot able to get the results, actually I have several models that starts with the same name and I don't have any results, I'm desperate i made all kinds of tests not sure what its happening, with some words I have the problem and with other ones no.
This is my current configuration: (Its not the only configuration I tested but its the one with better results yet)
SearchRule:
public function buildQueryPayload()
    {
        return [
            'must' => [
                'multi_match' => [
                    'query' => $this->builder->query,
                    'fuzziness' => 'auto',
                ]
            ],
        ];
    }

IndexConfigurator:
protected $settings = [
        'analysis' => [
            'analyzer' => [
                'es_std' => [
                    'type' => 'standard',
                    'stopwords' => '_spanish_'
                ],
            ],
        ]
    ];

In the model:
protected $mapping = [
        'properties' => [
            'name' => [
                'type' => 'text',
            ],
            'cientific_name' => [
                'type' => 'text',
            ],
        ]
    ];

    public function toSearchableArray()
    {
        return  [
        	'id' => $this->id,
        	'name' => $this->name,
        	'cientific_name' => $this->cientific_name,
        ];
    }

What I want is that the user starts to write in a search box and find any results even if only writes one character, it's that possible to configure?
Thanks!

Update: I completily removed the index and now it works, in case someone wondered this its my current configuration for the index:
protected $settings = [
    	"analysis" => [
            "filter" => [
                "autocomplete_filter" => [
                    "type" => "edge_ngram",
                    "min_gram" => 3,
                    "max_gram" => 20,
                ]
            ],
            "analyzer" => [
                "autocomplete" => [
                    "type" => "custom",
                    "tokenizer" => "standard",
                    "filter" => [
                        "lowercase",
                        "autocomplete_filter"
                    ]
                ],
            ]
        ]
    ];

    protected $defaultMapping = [
        'properties' => [
            'name' => [
            	"analyzer" => "autocomplete",
                'type' => 'text',
            ],
            'cientific_name' => [
            	"analyzer" => "autocomplete",
                'type' => 'text',
            ],
        ]
    ];

And to competily remove my index i run:
php artisan elastic:drop-index App\\Scout\\PlagueIndexConfigurator
and to create it again
php artisan elastic:create-index App\\Scout\\PlagueIndexConfigurator


test(infra): improve e2e test stability

Summary
closes #7683
It seems that the issue is caused by the CI attempting to perform a hot update on the index.js module, but the update fails because it is the entry module.
Checklist

[x] Tests updated (or not required).
[ ] Documentation updated (or not required).

The first pipeline test passed, I will rebase and try again.

But why does it only have a small probability of failure, and can only be reproduced in CI?

But why does it only have a small probability of failure, and can only be reproduced in CI?

I'm not sure yet.

If this doesn't resolve the issue, we can investigate further.


Pull Request?

Hi there,
I was wondering if you'd consider submitting this to be included in the next version of Drill?  I am the PMC Chair and would be happy to assist with the code review.

hey, I'm glad to have this chance to share my code and contribute to Drill. I'm still refining the code. And I'm looking forward to further communication with you. thanks!

Thanks for the response.  When you're ready, I'm happy to assist with the pull request process.  When do you think you will feel comfortable starting a submission?    If you'd like, I can create a JIRA to start the process and when you're ready, you can submit a pull request.  Please let me know.


Update Composer dependencies (2021-07-22-00-09)

Visual regression test passed!
View the visual regression test report


Added support for setxy

I have added support for setxy from Logo
I also included support for passing args to the commandLookUp functions such at setxy(x, y)

Great stuff!


Feedback for: compose/compose-file.md

File: compose/compose-file.md
The update_config subsection under deploy still has two fields with description "TODO"

@caervs thanks for the feedback! I've added these descriptions to the file and republished it, so you can check it out here: https://docs.docker.com/compose/compose-file/#/deploy
Much appreciate it, closing the issue. Please ping us again if you find more problems.

Looks good. Thanks for the quick response, @londoncalling!


Wordpress uploads without S3 ?

Hi,
is it possible to create a custom plugin which replaces S3 in order to use my own hosting for assets storage ?

@gnud yes that is possible WordPress is very pluggable I recommend taking a look at https://github.com/humanmade/S3-Uploads as a starting point in building a plugin for switching out the S3 static file backend.


Add zeux/meshoptimizer to third_party.

This also adds it as a dependency to filameshio. This does not seem to
increase the size of the WebGL build even though filameshio is a
dependency, perhaps because we are not using it yet.

Good catch, yes we should add the license.

Thanks for catching that Ben, I added the license.


HAWKULAR-450

changes to make standalone charting work with Hawkular Metrics directly.

@mwringe can you move those changes over to the Typescript file (the javascript file is generated from the typescript file):
https://github.com/hawkular/hawkular-charts/blob/master/src/chart/hawkular-chart-directive.ts
The js file will get clobbered the next time any of the*.ts files change. You can type 'gulp watch' and it will watch your files and recompile anything that changes. Or just 'gulp' will build the js file from the ts files.

Ok, updated the ts file and updated the hawkular-metrics.js file to the latest generated one.
There are a lot of changes in the latest generated hawkular-metrics.js, mostly due to white space issues but also some code changes.
The whitespace issue may be due to some configuration on my machine, if there is a particular step I need to do for the whitespace to be generated differently let me know.
The code differences (other than what is new in the ts file) maybe due to the generated hawkular-metrics.js file being a bit out of sync with the latest from the ts file?


[RFC] jsoncomm: transparently handle huge messages via fds

[draft as this needs a syscall implementation for os.memfd_create so that it works with python3.6 and also to get feedback if this is too much magic]
[this would make https://github.com/osbuild/osbuild/pull/1833 and https://github.com/osbuild/osbuild/pull/1836 obsolete]
The existing jsoncomm is a work of beauty. For very big arguments however the used SOCK_SEQPACKET hits the limitations of the kernel network buffer size (see also [0]). This lead to various workarounds in #824,#1331,#1836 where parts of the request are encoded as part of the json method call and parts are done via a side-channel via fd-passing.
This commit changes the code so that the fd channel is automatically and transparently created and the workarounds are removed. A test is added that ensures that very big messages can be passed.
[0] https://github.com/osbuild/osbuild/pull/1833

I love this! :)

@mvo5 Is this still draft as the text implies? :)

@mvo5 Is this still draft as the text implies? :)

My bad, sorry that I forgot to update the PR description. The last commit implements the "memfd_create" via our "libc" module to make this work on python3.6 and the tests are passing so this should be good now :)


bump send version and also npm audit fix

per title
first add "send": "0.19.0" in package.json overrides section, then run npm install, then run npm audit fix, then remove "send": "0.19.0"
test locally:
pnpm dev then go to localhost:3000, click through some pages.

#2344  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

🚀 Deployed on https://6737a353a56ce5407eec96b4--cozy-fox-0defec.netlify.app

Merge activity

Nov 15, 2:40 PM EST: A user merged this pull request with Graphite.


Websocket package version seems to be incompatible [Samsung Smart TV]

Home Assistant release (hass --version):
0.56.2
Python release (python3 --version):
Python 3.5.3
Component/platform:
samsungtv
Description of problem:
Errors in the log
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
mediaplayer:
  - platform: samsungtv
    host: ...
    port: 8001

pip3 install websocket-client
press any button in samsung widget

Traceback (if applicable):
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity.py", line 204, in async_update_ha_state
    yield from self.async_device_update()
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/helpers/entity.py", line 309, in async_device_update
    yield from self.hass.async_add_job(self.update)
  File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/media_player/samsungtv.py", line 121, in update
    return self.send_key('KEY')
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/media_player/samsungtv.py", line 134, in send_key
    self.get_remote().control(key)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/media_player/samsungtv.py", line 127, in get_remote
    self._remote = self._remote_class(self._config)
  File "/srv/homeassistant/lib/python3.5/site-packages/samsungctl/remote.py", line 10, in __init__
    self.remote = RemoteWebsocket(config)
  File "/srv/homeassistant/lib/python3.5/site-packages/samsungctl/remote_websocket.py", line 21, in __init__
    self.connection = websocket.WebSocket()
TypeError: __init__() missing 3 required positional arguments: 'environ', 'socket', and 'rfile'

Additional info:

Seems to be a duplicate of #5455


SHA2

Under the following bullets, SHA2 is not a valid hash. The closest options are MD2, SHA1, or SHA256.
Minimum server certificate requirements

Configure the server certificate with the required cryptography setting:

Click the Cryptography tab and make sure to configure the following:

Hash Algorithm: SHA2

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

ID: 4de79f23-cacb-9d4c-c3cb-266346d043ac
Version Independent ID: e858df36-4e46-d232-fb88-e011329cf989
Content: Configure Certificate Templates for PEAP and EAP Requirements
Content Source: WindowsServerDocs/networking/technologies/nps/nps-manage-cert-requirements.md
Product: windows-server-threshold
Technology: networking
GitHub Login: @shortpatti
Microsoft Alias: pashort

Theoretically, SHA2 is a family of hashing algorithm that includes SHA256, SHA384, SHA512.
In practical implementation, yes, it should be specified as 1 of the above hashing algorithm,


Mac notification doesn't show

Hello,
I'm using:
macOS Catalina 10.15.2
node v10.16.3
node-notifier v6.0.0
I tried the quick start and is not working

Hi @gabrielmoncea! Could you explain more of what isn't working? When I create a small sample project it looks to work just fine on my computer: https://github.com/jnielson94/node-notifier-simple
Does that sample work for you? If not, can you share more of what isn't working? :)

@jnielson94 it's working now. I'm not sure why it wasn't.
Sometimes a restart of the mac and a good night sleep for the brain solves things 👍

I am in the same limbo right now, I guess I should sleep then. 🤣


Customisable sea levels

Might be a good idea to allow sea levels to be changed in the mod, since it could help traversability and allow terrain to generate a bit faster.
Specifically the following levels:

Overworld water level (+ where the generator thinks the water level is, so it know where to stop generating grass and instead generate dirt/gravel)
Nether lava level
Overworld cave lava level
Nether cave lava level

Probably out of the scope of this mod, may fit better into the proposal in #44 though

moving request to netherhigher


Request help for a similar case

Hello, and thanks in advance.
I have a question, can you help me?
How did you discover the API to get the APK download link?
I need this for Myket, can you help me?
Myket Android app is obscured and I could not find anything with the reverse engineer.

I used these tools to capture the traffic and bypass the certificate pinning:

Genymotion (Android < 7)
Xposed Framework

Module: SSLUnpinning 2
Module: Inspeckage

MITMProxy

you can also use Frida.

Thanks a lot for your quickly reply.
Thank you for generously giving me your knowledge.
I'm not familiar with these but I'll try to learn them soon.
Thanks again

Hi again,
Unfortunately after trying so hard, I could not configure it;
I need a complete guide on this.
Can you give me a complete reference for this?
My ecosystem is:

Operating system: Windows 7
Android Emulator: Nox App Player (with Android version 4.4.2 - with root access)
Xposed Framework installed with these two Modules

SSLUnpinning 2.0
Inspeckage 2.3 (2.4 is incompatible for Android 4.4)

mitmproxy 5.3.0

Apparently many details need to be configured that I do not know


Fixes #818

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
fixes #818

License
Apache 2.0

What's in this PR?
Use the spec.deployment.image from IstioMeshgateway in Deployment.
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline
[x] User guide and development docs updated (if needed)

Closed in favor of #820.


Fix an error for Style/HashSyntax when using Prism::Translation::Parser

This PR fixes the following error Style/HashSyntax when using Prism::Translation::Parser as a parser:
$ bundle exec ruby -rprism/translation/parser/rubocop $(bundle exec which rubocop) \
  --only Style/HashSyntax -d lib/rubocop/cop/style/not.rb
(snip)

An error occurred while Style/HashSyntax cop was inspecting
/Users/koic/src/github.com/rubocop/rubocop/lib/rubocop/cop/style/not.rb:23:27.
undefined method `loc' for an instance of AST::Node
/Users/koic/.rbenv/versions/3.3.0/lib/ruby/gems/3.3.0/gems/parser-<IP_ADDRESS>/lib/parser/builders/default.rb:1875:
in `join_exprs'

Other ways to resolve this might exist, but using RuboCop::Cop::Base#parser is not necessary.
Currently, no repro tests have been added as it is sufficient to verify that it is not broken with the Parser gem. And, this is for refactoring aimed at integration with Prism, no changelog entry will be added.
cf. https://github.com/ruby/prism/blob/v0.21.0/docs/parser_translation.md#rubocop

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!
Might be good to keep track of the Prism compatibility work somewhere (e.g. what are the current problems with it). I'm happy to see you're looking into this!

Yeah! Such localized issues with Prism integration only need to be fixed, but the overall design, including compatibility, is likely to be discussed and progressed with Prism and RuboCop, while being documented along the way :-)

@koic I can't seem to replicate the error you experienced. I just want to make sure there isn't an issue with the compiler that this is working around.

@kddnewton Here are the detailed steps I would reproduce.
First, please obtain the latest stable version (v1.60.2) of this rubocop repository:
$ cd path/to/rubocop
$ git checkout v1.60.2

Next, apply the following diff:
$ git diff .
diff --git a/.rubocop.yml b/.rubocop.yml
index bdae9cc7e..14d536658 100644
--- a/.rubocop.yml
+++ b/.rubocop.yml
@@ -15,7 +15,7 @@ AllCops:
     - 'tmp/**/*'
     - '.git/**/*'
     - 'bin/*'
-  TargetRubyVersion: 2.7
+  TargetRubyVersion: 80_82_73_83_77.33
   SuggestExtensions: false

 Naming/PredicateName:
diff --git a/Gemfile b/Gemfile
index 137bfd97b..25cf09159 100644
--- a/Gemfile
+++ b/Gemfile
@@ -8,6 +8,7 @@ gem 'asciidoctor'
 gem 'bump', require: false
 gem 'bundler', '>= 1.15.0', '< 3.0'
 gem 'memory_profiler', platform: :mri
+gem 'prism'
 gem 'rake', '~> 13.0'
 gem 'rspec', '~> 3.7'
 gem 'rubocop-performance', '~> 1.20.0'

Finally, this can be reproduced by executing the following command:
$ bundle exec ruby -rprism/translation/parser/rubocop $(bundle exec which rubocop) --only Style/HashSyntax -d lib/rubocop/cop/style/not.rb


Applying MaxNumberOfAttributes for stackdriver exporter has no effect

Please answer these questions before submitting a bug report.
What version of OpenCensus are you using?
0.24.0
What JVM are you using (java -version)?
openjdk version "1.8.0_212"
OpenJDK Runtime Environment (build 1.8.0_212-b04)
OpenJDK 64-Bit Server VM (build 25.212-b04, mixed mode)
What did you do?
The code below adds 10 annotation attributes to the span iwth max allowed =32
StackdriverTraceConfiguration stackdriverTraceConfiguration =
                        StackdriverTraceConfiguration.builder().setProjectId("xxx").build();
        StackdriverTraceExporter.createAndRegister(stackdriverTraceConfiguration);
        Tracing.getTraceConfig().updateActiveTraceParams(TraceParams.DEFAULT.toBuilder().setMaxNumberOfAttributes(32)
                        .setSampler(Samplers.alwaysSample()).build());
        final Scope test = Tracing.getTracer().spanBuilder("test").startScopedSpan();
        try {
            final Span currentSpan = Tracing.getTracer().getCurrentSpan();
            Map<String, AttributeValue> spanAttributes = new HashMap<>();
            for (int i = 0; i < 10; i++) {
                spanAttributes.put("attr" + i, AttributeValue.stringAttributeValue("attr" + i));
            }
            currentSpan.addAnnotation("Custom", spanAttributes);
        } finally {
            test.close();
        }

What did you expect to see?
I expect to see all ten added attributes at stackdriver
What did you see instead?
I see only 5 attributes. Four of them are randomly picked attributes from my list and the fifth one is the dropped label added by stackdriver witch is equal to 6.
Additional context
Result of this code execution at stackdriver:

It seems that it only affects span attributes, not the annotation attributes. Looks there is no way to configure annotation attributes, but it is out of scope of this one.


Edit hdinsight-administer-use-powershell.md

Edit complete.
The link "Install and configure Azure PowerShell" didn't work when I tried it.
The link "Use Azure Blob storage for HDInsight" doesn't seem to be rendering correctly.
Line 41 refers to a "publishsettings file." Please confirm that "publishsettings" is correct as is because it's a specific name. Also, some information may be missing here about what the publishsettings file is and where readers can find it.
Per naming guidelines, I changed instances of "PowerShell" by itself to "Azure PowerShell." Please make sure that all mentions are technically accurate and shouldn't be "Windows PowerShell" instead.
Please confirm that the screenshot doesn't reveal any potentially sensitive information.

Hi @ShawnJackson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


docs(cn): translate why-create-typescript into Chinese

The path seems wrong...

The source file is https://github.com/microsoft/TypeScript-Website/blob/v2/packages/typescriptlang-org/src/templates/pages/why-create-typescript.tsx
Not .md file
I don't know how to support translations.
cc: @orta

https://github.com/microsoft/TypeScript-Website/issues/2528

@hexiaokang请阅读以下贡献者许可协议 (CLA)。如果您同意 CLA，请回复以下信息。
@microsoft-github-policy-service agree [company="{your company}"]

选项：

（默认 - 未指定公司）我对我的提交拥有知识产权的唯一所有权，并且我不会在为我的雇主工作的过程中提交提交。
（当给定公司时）我正在为我的雇主工作（或我的雇主根据合同或适用法律在我的提交中拥有知识产权）。我已获得我的雇主的许可，可以代表我的雇主提交材料并签订本协议。通过在下方签名，定义的术语“您”包括我和我的雇主。

贡献者许可协议

@hexiaokang please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@hexiaokang please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]

Options:

(default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.
(when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

Contributor License Agreement

@microsoft-github-policy-service agree [company="{no company}"]

@microsoft-github-policy-service agree


Overlay default list

-- need to determine overlay list with defaults -- what order, what’s on upon open, and to what scale

[ ] Scale is fine, but Display Important Fossil Areas and parks

@LeannePyle - Which parks should be displayed? There are Provincial, Federal and Municipal to choose from. Any or all?

All please


Prepare for OOPS#1199

Description
Make changes needed to deal with https://github.com/JCSDA-internal/oops/pull/1199

@guillaumevernieres you're not doing a very good job of taking the day off!

@guillaumevernieres you're not doing a very good job of taking the day off!

Not the real @guillaumevernieres , just a cloned AI version. It's cheap too, runs on 3 neurons only.


Added Sieve of Erastosthenes in C++

This code finds all the primes till N and also computes the summation of those primes.

Also added segment tree implementation


Using src/app as the app folder for Expo Router with Tamagui spams logs

Current Behavior
I am setting up a simple project using Expo Router and Tamagui, and I am using src/app/ as the root app directory rather than app/. Whenever I run the app with Expo Go on IOS or Android, the following log message spams infinitely until I stop the app (this only happens when I add the TamaguiProvider): Using src/app as the root directory for Expo Router
Expected Behavior
The log message should not print over and over again.
Tamagui Version
1.90.6

Platform (Web, iOS, Android)
Android and IOS

Reproduction
1. Clone the following repository: https://github.com/mhamill2/tamagui-expo-router-bug
2. Run npm i
3. Run npm start
4. Open the app in either an Android or iPhone simulator and see the log spamming

This directory has almost the same configuration as the tamagui expo router starter template, the main difference being that I am not using yarn. I get the same error on the template if I refactor it to use npm instead of yarn, and move the app dir into src/.

System Info
System:
    OS: macOS 14.1
    CPU: (12) arm64 Apple M2 Max
    Memory: 479.48 MB / 32.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 21.6.1 - /opt/homebrew/bin/node
    Yarn: 1.22.21 - /opt/homebrew/bin/yarn
    npm: 10.2.4 - /opt/homebrew/bin/npm
    pnpm: 8.15.1 - /opt/homebrew/bin/pnpm
  Browsers:
    Chrome: 121.0.6167.184
    Edge: 122.0.2365.52
    Safari: 17.1
  npmPackages:
    @babel/core: ^7.20.0 => 7.23.9
    @expo/metro-runtime: ^3.1.3 => 3.1.3
    @tamagui/config: ^1.90.6 => 1.90.7
    @tamagui/metro-plugin: ^1.90.6 => 1.90.7
    @types/react: ^18.2.57 => 18.2.58
    babel-preset-expo: ^10.0.1 => 10.0.1
    expo: ~50.0.7 => 50.0.7
    expo-constants: ~15.4.5 => 15.4.5
    expo-font: ~11.10.3 => 11.10.3
    expo-linking: ~6.2.2 => 6.2.2
    expo-router: ~3.4.7 => 3.4.7
    expo-status-bar: ~1.11.1 => 1.11.1
    react: 18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    react-native: 0.73.4 => 0.73.4
    react-native-safe-area-context: 4.8.2 => 4.8.2
    react-native-screens: ~3.29.0 => 3.29.0
    react-native-web: ^0.19.10 => 0.19.10
    tamagui: ^1.90.6 => 1.90.7
    typescript: ^5.3.3 => 5.3.3

What's the log? Does it happen when you use yarn?

What's the log? Does it happen when you use yarn?

It's in the description, but this is the log message that prints infinitely: Using src/app as the root directory for Expo Router
No, it does not happen when I use yarn as it's setup in the expo router template repo. Only when using npm.

I noticed the log doesn't print if I don't import the tamagui css file, i.e. if I remove the following:
import '../../tamagui-web.css';
Not sure why though

@natew It will keep spamming a bunch of things like this:

@mhamill2 if you import the CSS file with a name then the logs will be gone
// eslint-disable-next-line @typescript-eslint/no-unused-vars
import _styles from '../../tamagui-web.css';

@octopitus is it still happening?

try changing outputCSS: './tamagui-web.css' in your metro.config to outputCSS: './src/tamagui-web.css'
and then import it one level up
import from '../tamagui-web.css';
anyway that seems to be an issue related to directory structure and expo-router this one might help https://github.com/expo/router/issues/41 if the problem persists


Minimal FileChooser example breaks on Ubuntu 14

Example code:
from gooey import Gooey
from gooey import GooeyParser

@Gooey(advanced=True)
def main():
    parser = GooeyParser(description="My Cool Gooey App!")
    parser.add_argument('filename',
                        help="name of the file to process",
                        widget='FileChooser')
    parser.parse_args()

if __name__ == "__main__":
    main()

Traceback
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/gooey/gui/widgets/widget_pack.py", line 82, in onButton
    self.text_box.SetLabelText(result)
AttributeError: 'TextCtrl' object has no attribute 'SetLabelText'

I think you may be running an out of date version of Gooey. This is a duplicate of issue #76, which was fixed in commit d83ab61bff9c5.
Try cloning a fresh copy of the repository (from git, not pip!) and you should be good to go. Feel free to reopen if the issue persists.

It works perfectly now! Sorry, you were correct, I was using the pip version. Thanks for making the library!

My fault for not keeping the pypi project up to date with the current state of the repo.
I really need to figure out how to tie the two together. Out of date pip bites people all the time!


feat: add Idempotency-Prior to ResponseMetadata

Its useful to know whether a response has the Idempotency-Prior header.
This change simply catches that header and adds it to the ResponseMetadata.
Please let me know if this PR is to the wrong branch.

No problem @douglasmiller.
You're probably right, it would actually be better to just allow someone to get all the headers. Then you wouldn't require code changes going forward.
I'd say their probably isn't any harm in just exposing the whole Response object within ResponseMetadata? People could then use that as required. Headers, or other bits.


[HELP]  arch/arm/src/armv6-m/arm_svcall.c There seems to be a problem

Description
There may be a bug in this code.

#ifdef CONFIG_LIB_SYSCALL
static void dispatch_syscall(void) naked_function;
static void dispatch_syscall(void)
{
asm volatile
(
" push {r4, r5}\n"                              /* Save R4 and R5 /
" sub sp, sp, #12\n"                            / Create a stack frame to hold 3 parms 这条指令调整堆栈指针（sp），为传递的三个参数腾出空间。每个参数占 4 字节，总共是 12 字节，创建了一个新的栈帧。/
" str r4, [sp, #0]\n"                           / Move parameter 4 (if any) into position /
" str r5, [sp, #4]\n"                           / Move parameter 5 (if any) into position /
" str r6, [sp, #8]\n"                           / Move parameter 6 (if any) into position /
" mov r5, lr\n"                                 / Save lr in R5 这条指令将链接寄存器（lr，Link Register）的值复制到 r5 中。lr 存储的是当前函数返回时的地址。保存它是为了在系统调用返回后恢复。/
" ldr r4, =g_stublookup\n"                      / R4=The base of the stub lookup table /
" lsl r0, r0, #2\n"                             / R0=Offset of the stub for this syscall 这条指令将寄存器 r0 的值左移 2 位，相当于乘以 4。由于每个函数指针占 4 字节，这样的操作可以计算出当前系统调用的偏移量。/
" ldr r4, [r4, r0]\n"                           / R4=Address of the stub for this syscall 从 g_stublookup 数组中加载对应系统调用的处理函数的地址到 r4 中*/
" blx r5\n"                                     /* Call the stub (modifies lr) 这条指令使用 blx 指令跳转到寄存器 r5 中保存的地址，并调用系统调用处理函数。blx 是带有链接的跳转指令，它会将返回地址保存在 lr 中，跳转到 r5 保存的地址。/
" mov lr, r5\n"                                 / Restore lr 这条指令将 r5 中保存的返回地址恢复到 lr 中。这是为了确保在系统调用完成后能够正确返回到调用点。/
" add sp, sp, #12\n"                            / Destroy the stack frame 这条指令恢复栈指针（sp），释放之前为保存参数所分配的 12 字节空间。/
" pop {r4, r5}\n"                               / Recover R4 and R5 这条指令将之前保存的 r4 和 r5 的值从栈中恢复到寄存器中，恢复调用前的状态。/
" mov r2, r0\n"                                 / R2=Save return value in R2 将返回值（存储在 r0 中）保存到 r2 中。这是为了在之后的系统调用中使用。/
" mov r0, #" STRINGIFY(SYS_syscall_return) "\n" / R0=SYS_syscall_return 这条指令将 SYS_syscall_return 的值加载到 r0 中。SYS_syscall_return 是一个表示系统调用返回的常量。/
" svc #" STRINGIFY(SYS_syscall) "\n"            / 相当于 int 0x80; Return from the SYSCALL 这条指令触发一个软件中断（svc），调用系统调用返回机制，并将 r0 中的值传递给内核。这表示系统调用完成后要返回到用户空间。*/
);
}
#endif

blx r5\n"                                     /* Call the stub (modifies lr) */

This should be blx r4 ,but not r5
Verification

[X] I have verified before submitting the report.

@guoshichao @xiaoxiang781216 could you please take a look?

@guoshichao @xiaoxiang781216 could you please take a look?

@patacongo In the submission with the identifier bd1488bdabb2aa6fcd76a187b1cd0de4f311fbad, the process of calling the sub-routine in the implementation of dispatch_syscall is as follows:
+  __asm__ __volatile__
+  (
+    " push {r4-r6}\n"             /* Save R4, R5 and R6 */
+    " mov r6, r14\n"              /* Save LR in R6 */
+    " ldr r4, =g_stublookup\n"    /* Get the base of the stub lookup table */
+    " lsl r3, r0, #2\n"           /* Get the offset of the stub for this syscall */
+    " ldr r3, [r4, r3]\n"         /* Load the entry of the stub for this syscall */
+    " blx r3\n"                   /* Call the stub */
+    " mov r14, r6\n"              /* Restore R14 */
+    " pop {r4-r6}\n"              /* Restore R4, R5, and R6 */
+    " mov r2, r0\n"               /* Save the return value in R0 in R2 for now */
+    " mov r0, #3\n"               /* R0=SYS_syscall_return */
+    " svc 0"                      /* Return from the syscall */
+    :::
+  );

However, in a subsequent submission with the identifier 686f21dcd3132270902639a6d4662d243a43946e, the implementation of calling the sub-routine within dispatch_syscall was modified to:
-    " push {r4-r6}\n"             /* Save R4, R5 and R6 */
-    " mov r6, r14\n"              /* Save LR in R6 */
-    " ldr r4, =g_stublookup\n"    /* Get the base of the stub lookup table */
-    " lsl r3, r0, #2\n"           /* Get the offset of the stub for this syscall */
-    " ldr r3, [r4, r3]\n"         /* Load the entry of the stub for this syscall */
-    " blx r3\n"                   /* Call the stub */
-    " mov r14, r6\n"              /* Restore R14 */
-    " pop {r4-r6}\n"              /* Restore R4, R5, and R6 */
-    " mov r2, r0\n"               /* Save the return value in R0 in R2 for now */
+    " push {r4}\n"                /* Save R4 */
+    " ldr r4, =g_stublookup\n"    /* R4=The base of the stub lookup table */
+    " lsl r0, r0, #2\n"           /* R0=Offset of the stub for this syscall */
+    " ldr r4, [r4, r0]\n"         /* R4=Address of the stub for this syscall */
+    " blx r5\n"                   /* Call the stub (modifies R14) */
+    " pop {r4}\n"                 /* Restore R4 */
+    " mov r2, r0\n"               /* R2=Saves return value in R0 */
     " mov r0, #3\n"               /* R0=SYS_syscall_return */
     " svc 0"                      /* Return from the syscall */

The process of calling the sub-routine has changed from
blx r3  # Here, the value stored in r3 is still the entry point of the sub-routine
to
blx r5 # Here, the value stored in r5 is the value of lr.
Why has it been changed like this?

@patacongo Please help to take a look, in the commit of bd1488b, the process of calling the sub-routine in the implementation of dispatch_syscall is as follows

That change was from a very long time ago.  I don't remember much in detail.
The original was in C.  That was converted to assembly in bd1488b.
The commit you reference just fixes some assembly language errors.


Allow failed event handler to return Promise or number

The type for the failed event only allowed you to return a Promise<number> or void.  However, you might want to return a number synchronously, or return void asynchronously.  This change allows those other cases in the failed event handler.

Yes, this was an oversight on my part. Thank you for fixing it 👍
I applied your change to the EJS file bottleneck.d.ts.ejs as it generates the bottleneck.d.ts.ejs file. I also added a few typings tests in test.ts and finally allowed users to return null (sync and async). https://github.com/SGrondin/bottleneck/commit/712db2f736fd243bf262760e053714acb5535ead
It's been released in v2.19.1 🎉  https://github.com/SGrondin/bottleneck/releases/tag/v2.19.1
Thank you for your contribution!


Missing dismiss() method

Hi, I've created an instance of material dialog, but I can't invoke the dismiss() method when it isn't no longer needed.

MaterialDialog.Builder is not a dialog, MaterialDialog is.

I added a new section to the README since so many cannot figure it out: Dismissing Dialogs

Thank you, for everything.


[eas-cli] Unify error display method

Overriden the Command's catch method to catch all the errors thrown by the commands, log the error message there the same way as it was previously handled in src/build/build.ts and re-throw generic Error instance with predefined base error message. To avoid duplication, old Log.error() calls have been removed from src/build/build.ts. Also added tests to see if the catch() method works as intended.

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
There used to be a difference between how the expected/handled errors had been displayed by eas-cli when compared to these that were not expected. This change is meant to unify this behaviour
Expected:

Unexpected:

How
Utilised the catch() method of oclif's Command class, which is extended by custom base class EasCommand, which in turn is extended by all the commands in eas-cli. EasCommand's catch() now logs the caught error the same way as it was previously done in case of expected errors, and then re-throws a generic error with pre-defined base error message. Code fragments which used to log the errors previously and re-throw generic error now just throw the error and let the catch() do the logging for them.
Expected:

Unexpected:

Test Plan
Added automatic tests to check the catch() methods behaviour

Plus tested manually by breaking the project to intentionally cause errors and check how they are displayed

/changelog-entry chore Unify how the command errors are displayed


Add created sequence object to the extension

Without adding the sequence pg_dump crashes. This happens because the sequence's table is ignored during dumping objects, unlike the sequence. It leads to inconsistency in pg_dump's metadata of objects.
Here is the backtrace from Postgres 15.10:
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x000000010001d140 pg_dump`dumpTable [inlined] dumpSequence(fout=0x000000015b6049e0, tbinfo=0x00000001600c0040) at pg_dump.c:16978:15 [opt]
   16975                                                          fmtQualifiedDumpable(owning_tab));
   16976                        appendPQExpBuffer(query,
   16977                                                          "ALTER COLUMN %s ADD GENERATED ",
-> 16978                                                          fmtId(owning_tab->attnames[tbinfo->owning_col - 1]));
   16979                        if (owning_tab->attidentity[tbinfo->owning_col - 1] == ATTRIBUTE_IDENTITY_ALWAYS)
   16980                                appendPQExpBufferStr(query, "ALWAYS");
   16981                        else if (owning_tab->attidentity[tbinfo->owning_col - 1] == ATTRIBUTE_IDENTITY_BY_DEFAULT)
warning: pg_dump was compiled with optimization - stepping may behave oddly; variables may not be available.
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x000000010001d140 pg_dump`dumpTable [inlined] dumpSequence(fout=0x000000015b6049e0, tbinfo=0x00000001600c0040) at pg_dump.c:16978:15 [opt]
    frame #1: 0x000000010001d04c pg_dump`dumpTable(fout=0x000000015b6049e0, tbinfo=0x00000001600c0040) at pg_dump.c:15102:4 [opt]
    frame #2: 0x00000001000061a0 pg_dump`main [inlined] dumpDumpableObject(fout=0x000000015b6049e0, dobj=0x00000001600c0040) at pg_dump.c:10045:4 [opt]
    frame #3: 0x00000001000060e4 pg_dump`main(argc=<unavailable>, argv=<unavailable>) at pg_dump.c:912:3 [opt]
    frame #4: 0x00000001993ac274 dyld`start + 2840

Also I forgot to DROP the sequence within pgmq.drop_queue.
BTW I wonder why queue tables are added to the extension? Would it make sense to allow users to dump those tables?

BTW I wonder why queue tables are added to the extension? Would it make sense to allow users to dump those tables?

The tables are added to the extension so that when you remove the extension from your database, the objects created by the extension (including the queue tables) will also be removed. The detach_archive function was added as a way to keep the archive table.

I pushed latest change which

adds ADD SEQUENCE/DROP SEQUENCE to all relevant functions pgmq.drop_queue, pgmq.create_non_partitioned, pgmq.create_unlogged, pgmq.create_partitioned
adds the migration script pgmq--1.5.0--1.5.1.sql which redefines the functions and adds relevant sequences to the extension

@ChuckHend I returned the version back to 1.5.0 and the migration script uses pgmq.meta now. I used more complex query before to make sure that the migration is idempotent. But I guess this might be an issue only if a user added sequences manually.
BTW it seems that the migration script skips 1.4.5. Is this intentional? It looks like a typo and it might fail during upgrading from 1.4.5 to 1.5.0.

@za-arthur, correct we will need an upgrade script for 1.4.5 to 1.5.0 before we release 1.5.0, thank you for bringing that up. I think we may need to rename https://github.com/tembo-io/pgmq/blob/main/pgmq-extension/sql/pgmq--1.4.4--1.5.0.sql to ..1.4.5--1.5.0.sql then put the changes from this PR into that migration.

@ChuckHend I created a PR to rename the script:
https://github.com/tembo-io/pgmq/pull/355

Rebased the PR on latest changes.


Run Tests for wasm32-unknown-unknown

This can be done using cargo-web: https://github.com/koute/cargo-web#using-cargo-web-on-travis

Now that wasm-bindgen support we can use wasm-pack to run our tests without too many complications (though all the tests that read from and write to files will fail). We likely just want to include_bytes! all the splits files and layouts to solve this.

Additionally for WASI we might be able to run the tests as well via something like wasmtime. There the files shouldn‘t be a problem.

Looks like it's super easy to do
[target.wasm32-wasi]
runner = "wasmtime"

in the .cargo/config
You can just do cargo test --target wasm32-wasi then and it just works.


gracefully handle 429: Too Many Requests http result codes (e.g., Zenodo)

Web APIs may have activity quota. For instance, Zenodo's API currently has the quota and they are made visible in the "x-ratelimit-*" HTTP response headers :
HTTP/1.1 200 OK
server: nginx
date: Fri, 30 Aug 2024 21:40:45 GMT
content-type: application/json
content-length: 5010
vary: Accept-Encoding
x-ratelimit-limit: 133
x-ratelimit-remaining: 132
x-ratelimit-reset:<PHONE_NUMBER>
retry-after: 60
permissions-policy: interest-cohort=()
x-frame-options: sameorigin
x-xss-protection: 1; mode=block
Currently, Preston retries failed http requests, even if a 429 error code is returned.
Suggest to find a more graceful way to handle rate limits.

after deploying Preston v0.9.3, the following error logs are observed:
[main] WARN bio.guoda.preston.RateLimitedRetryStrategy - server signalled [429: too many requests]: retrying request after waiting for [3]s

signalling that Preston is now gracefully handling the Zenodo server signals to slow down requests as signaled in the "Reply-After" http response header.


Radio-image doesn't work in IE11

Hi,
is there anyone capable/willing to confirm that radio-image select doesn't work in IE11? It's all right in IE9 and IE10 but for some strange reason I can't get it work in IE11.
Debugger / console shows no error or whatsoever and I'm pullin my hair off for the past couple of hours.
Radio-image is just not clickable at my end. Also checked in FF, Safari and Chrome on MacOS and everything seems to be fine.
Thanks!

@dameer hey there!
The only problem here is that I don't have a windows machine so I can't test this... I'll have to just take your word for it!
If you do manage to figure this one out though, please do let us know what's wrong exactly so that we may fix it (or better yet, you can submit a pull request against the develop branch with a fix)

Basically, I'm using VirtualBox on Mac to test websites in Internet Explorer which doesn't let me resolve the problem cause. Developer tools for IE is the only choice I've got. Beside, none of my comrades and coworkers is Windows user. Strange how come everyone abandoned Windows. Tho, I don't wonder why.
So, I'm 100% sure that radio-image select doesn't work in IE11. No matter what I do (considering limited resources) takes no improvement.
I'll just switch from radio-image to the "regular" select because it works all right.
Hopefully there will be someone capable to puzzle out what's going on with radio-image in IE11.

OK, I think I've got it. The most probably solution is here : http://stackoverflow.com/questions/20198137/image-label-for-input-in-a-form-not-clickable-in-ie11
Hopefully someone can give it a shot and eventually submit a pull request...

Thanks for your research!
I'm really surprised this issue came up... 'cause I do remember it in the "old days".
I'm currently working on refactoring the typography control, this one is next on my list. :+1:

I believe this was just fixed...
Can you please checkout the develop branch and let me know if the issue is fixed for you there?
(don't test from the master branch, master has been reset to <IP_ADDRESS>. The develop branch where work on version 2.2.0 is being done)

I'm not quite skilled here at GitHub but I guess we are talking about this file: https://github.com/aristath/kirki/blob/7d2f4e86212aaa1a26746f7ce17176b45a173781/assets/css/customizer.css
Unfortunately, I can't confirm it works, the problem still persists in IE11. I'll try something else and let you know the result.

OK, here's why it didn't work...
There's actually no checked input in HTML for non-selected radio-image:
.customize-control-radio-image input:checked + label .image-clickable { ... }
...however, if we just use:
.customize-control-radio-image input + label .image-clickable { ... }
things are back to normal in IE11. So, this one worked fine on :
MacOS - FF (latest), Chrome (latest), Safari (latest)
Windows 7 (VirtualBox) - IE9, IE10, IE11 (edge)
Hopefully it helps, thanks for sorting this out!

Excellent!
Fixed, thank you for your assistance! :+1:


Spreadsheets are empty when opening in "Spreadsheet compare" Office tool

Hello! First of all, thank you for the amazing tooling, metrics are really great and writes are so performant! I'm able to generate large xlsx files using .NET.
However, I have faced the issue when trying to compare the file generated with SpreadCheetah, with other xlsx files using the Spreadsheet Compare 2016 tool (part of Office). The file generated using SpreadCheetah is treated as empty for some reason by this tool, so I can't compare it with others. At the same time, the file in Excel looks good and has the data. I had no similar issue when opening files created using NPOI package.
So, is it possible to make the SpreadCheetah-generated xlsx files compatible with Spreadsheet Compare?
Could you please let me know if you can plan it, so that I can realize whether I can fully migrate to SpreadCheetah?
This is how the issue looks like in Spreadsheet Compare (left file that looks empty - SpreadCheetah, right file - created in Excel):

This is how the file looks like in Excel - it's not empty:

Minimal code:
using var tempStream = new MemoryStream();
await using var spreadsheet = await Spreadsheet.CreateNewAsync(tempStream, null, cancellationToken);
await spreadsheet.StartWorksheetAsync("Report", null, cancellationToken);

var columnNames = new List<string> { "column1", "column2" };
var values = new List<string> { "value1", "value2" };

var headerRow = new List<Cell>();

foreach (var columnName in columnNames)
{
     headerRow.Add(new Cell(columnName));
}

await spreadsheet.AddRowAsync(headerRow, cancellationToken);

var valuesRow = new List<Cell>();
foreach (var value in values)
{
    valuesRow.Add(new Cell(value));
}
await spreadsheet.AddRowAsync(valuesRow, cancellationToken);

await spreadsheet.FinishAsync(cancellationToken);

Environment:
Spreadcheetah 1.9.0
.NET 6

Hi, and thanks for the feedback!
Not sure what is causing this issue, but I can look into it.
In the meantime, if you open the file created by SpreadCheetah in Excel, and then save the file from there, does it still show up as empty in Spreadsheet Compare?

Morning @sveinungf! When I open the file in Excel and save it, then it starts displaying in Spreadsheet Compare. The same relates for files generated by both Source Generators and Basic Usage scenarios. I'm attaching both Generated and GeneratedThenSaved files if it can help to troubleshoot the reason:
Generated.xlsx
GeneratedThenSaved.xlsx

It seems this is due to SpreadCheetah not writing the explicit cell reference for each cell. In an XLSX file, each cell element can have an explicit reference (or "r") attribute. According to the Open XML specification this attribute is optional, but it seems it is required for Spreadsheet Compare to do a comparison. SpreadCheetah doesn't write this attribute to achieve better performance, but I'm considering adding an option to write this attribute, something like SpreadCheetahOptions.WriteCellReferenceAttributes. The option would default to false so that performance wouldn't be affected for users that don't need this feature.

Sounds great!

The option is now available in version 1.10.0.


Update entrypoint.sh for console authentication

@lionelraymundi jetty realm is merged into a single JAAS realm in 6.x. This change appears to revert that.
A single JAAS realm simplifies management overall, and we shouldn't need completely separate JAAS realms to do access control across services.
If you want to separate access for web layer services, simply change the roles in the conf/jetty.xml file and conf/users.properties.

Thanks for your quick feedback!
I had to add it because the user passed by environment variable ACTIVEMQ_WEB_USER (and its password) was not  having any effect when accessing the web console, as the configuration file jetty.xml points to jetty-realm.properties and not to the file which the sed command was modifying...
   <bean id="securityLoginService" class="org.eclipse.jetty.security.HashLoginService">         <property name="name" value="ActiveMQRealm" />         <property name="config" value="${activemq.conf}/jetty-realm.properties" />     </bean>
I dont need separate access, I just need to change the default user and password. Should I do something different?

@lionelraymundi you should take a look on the Dockerfile from activemq-5.18.x branch.


Add max_children to Nested Sort

This commit adds max_children to nested sort.
Closes #4595

LGTM, will this need a SkipVersion when backporting to 6.x?

Yes, it needs a
[SkipVersion("<6.5.0", "introduced in 6.5.0")]


helm create - Make it easier to do the right thing, part ii: securityPolicy

Hello,
Helm create should add options to create security policies, enabled by default imho, as it is a known best practice to enable security policies, notably, to run as non root, to have a read only file system , drop linux capabilities and so on.
By adding the option to create security policies (enabled by default imho) we are making it easier to do the right thing.
cc @bacongobbler @unguiculus @scottrigby @rimusz

If we add it, would should be enabled by default?
So far, runAsNonRoot is a good option.
What about?
Drop all linux capabilities?
runAsUser? default value to 1000, or 10001?
readOnlyFilesystem? on or off by default?


Windows installation - db colation missing

in http://opencats-documentation.readthedocs.io/en/latest/install-windows.html#launch-phpmyadmin there is step describing creation of database.
Missing is information, what collation is adviced to be setup, it hsall be `utf9-general-ci' or similar (see discussion in opencats/OpenCATS#44)

Documentation updated.
https://app.gitbook.com/s/FMP3hgt72kl23HAv7iDu/readme/installation/install-on-windows
and
https://app.gitbook.com/s/FMP3hgt72kl23HAv7iDu/readme/installation/install-on-ubuntu


Added option to quantize minutes step

Adds the ability to round minutes to nearest step size.
Does not change the graphical handle position, just the computed minutes value.

The last commit is wrong


Remove protected characteristics item

Removes protected characteristics 'to do' line, as it's replaced by the Equality information pattern
https://github.com/alphagov/govuk-design-system/issues/1545

You can preview this change here:
Built with commit 7431dbbe28860a195d81c69f7eca98e213201b4b
https://deploy-preview-1583--govuk-design-system-preview.netlify.app

Hi @vanitabarrett - would you mind reviewing to make sure I haven't broken anything with this commit?
It covers the 'Removed protected characteristics from backlog page on Design System website' item on issue 1545.


Multiple Milestones per task causes incorrect group height

If you have multiple milestones on one task, the group height increases with the total number of milestones, which makes it too tall. You only need to increase the height once per task with a milestone.
I hacked in this in group.py, which seems to fix the issue
milestone_count = 0
for task in self.tasks:
milestone_count += 1   #len(task.milestones) only need to increase height once per milestone
if len(task.milestones) == 0:
for parallel_task in task.tasks:
milestone_count += 1    #len(parallel_task.milestones)
break  ### We only need to know whether there milestone exists in the parallel tasks

Hi, can you provide your code sample that I can use to recreate the issue?

This has been resolved in v1.3.1

Tested it and it is fixed.
Thank You.


run experimental tests on nightly ydb image

Pull request type

Please check the type of change your PR introduces:

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

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

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

Project coverage is 43.20%. Comparing base (43625f8) to head (ed26da4).
Report is 6 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1383      +/-   ##
==========================================
+ Coverage   41.87%   43.20%   +1.32%
==========================================
  Files         351      293      -58
  Lines       35226    25768    -9458
==========================================
- Hits        14751    11133    -3618
+ Misses      19993    14183    -5810
+ Partials      482      452      -30

Flag
Coverage Δ

go-1.20.x
43.18% <ø> (-0.50%)
:arrow_down:

go-1.21.x
43.16% <ø> (+0.01%)
:arrow_up:

go-1.22.x
?

macOS
43.12% <ø> (+1.70%)
:arrow_up:

ubuntu
43.20% <ø> (+1.81%)
:arrow_up:

unit
43.20% <ø> (+1.32%)
:arrow_up:

windows
?

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

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


[Bug Report] RemoveTag in ElCascader Is Not Functioning Well

Element UI version
2.15.1
OS/Browsers version

Vue version
2.6.12
Reproduction Link
https://codepen.io/tomcloud/pen/WNRLEMK
Steps to reproduce

open the codepen link checkout bug example;
remove tag '2';

What is Expected?
tag '2' removed
What is actually happening?
tag '1' removed

    deleteTag(index) {       const { checkedValue } = this;       const val = checkedValue[index];       this.checkedValue = checkedValue.filter((n, i) => i !== index);       this.$emit('remove-tag', val);     },
this function needs rewrite...

#20939 已修复，应该是还没合并到master并上线

@kai475 this was fixed in 2.15.2, confirming that I had the same issue and that it's working for me now


How to display data:image/png;base64 image on the icon property

How would I go about to bind a base64 encoded image to the icon property of win-split-view-command?
Is there any other helpers than url() I can use?

Appears to work when I do this, though I was not earlier able to bind to the icon attribute:
var profileButton = document.getElementById('profileButton');
profileButton.winControl.icon = 'url("' + state.buddyIcon + '")';

Problem though is that my icon is a bit bigger than the small one, and I want to render a bigger one, so here is another workaround that works pretty well:
#profileButton {
    background-repeat: no-repeat;
    background-size: 32px 32px;
    background-position: 8px center;
}

Some JavaScript code to inject a new CSS rule:
document.styleSheets[0].addRule('#profileButton', 'background-image: url("' + state.buddyIcon + '");');

And this is how the markup looks:
<win-split-view-command ng-show="authenticated" label="flickr.state.userName" id="profileButton" ui-sref="profile({userId: flickr.state.userId})"></win-split-view-command>

The end result looks like this:

It appears I was doing it wrong with the binding syntax, this code works just fine:
<win-split-view-command label="'Settings'" icon="'url(' + state.buddyIcon + ')'" ui-sref="settings"></win-split-view-command>

Though my other suggestion can be used if you want to render something bigger than the default icon sizes.


[PPP-3876]-Use of vulnerable component spring-security-core 4.1.9 cve-2017-4995

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

Build Skipped
:white_check_mark: There was nothing to build. The changes are probably excluded by the build rules. If this is an error, please contact your system administrator.

@bmorrise @mbatchelor @pamval Could you please take a look?


posterior.trees in Grollemund et al. 2015 seems broken

The format of posterior.trees in Grollemund et al. 2015 looks suspicious:

Trees are listed right in the translate block.
There's no end marker for the Trees block.

Actually, easier to fix this by hand, PR in #235


Update datepart-transact-sql.md

Text revisions to tighten and optimize the reading flow of the material.
Note: the Return Value block now says
"The return value depends on SET DATEFORMAT if date is a string literal of some formats"
and 'some formats' needs explanation and definition. I looked at earlier versions of this file and they have the same issue. I Googled but I did not find a clear answer. If someone could give me some guidance, I'll revise and send a new PR.
Thank you!

@fbsolo : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

Thanks @fbsolo #sign-off

@fbsolo : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.


poc demo needs to be updated

I tried to run the poc in directory observ-vol-mgt/contrib/end2end/poc/insight_based_poc. I ran the commands make start and the containers all came up. I then ran make perform_analysis and got an error:
curl -X POST http://<IP_ADDRESS>:5010/api/v1/analyze
{
  "error": "[Errno 2] No such file or directory: 'extract/tsfel_conf/limited_features.json'"
}

Something seems to have changed in the build or in the versions of containers running.

@KalmanMeth this has been fixed. Please verify and reopen the issue if needed. Thanks.


Is it possible to run Alexnet with DarkneTZ on Rpi3?

Hi, Mofanv
I would like to run the AlexNet with Cifar 10 dataset on Rpi3 B+.
The OPTEE setup is fine and other models like LeNet with mnist works well.
But when I tried to run the Inference(not training) AlexNet with the Cifar-10 dataset, I got trouble with it.
the command I tried is as below
 darknetp classifier predict -pp_start 9 -pp_end 13 cfg/cifar.data cfg/alexnet.cfg models/alexnet/alexnet.weights data/cifar/train/988_dog.png
and the error is as below

I/TA: secure world opened!
Begin darknet
layer     filters    size              input                output
    0 conv     96 11 x11 / 4   227 x 227 x   3   ->    55 x  55 x  96  0.211 BFLOPs
    1 max          3 x 3 / 2    55 x  55 x  96   ->    27 x  27 x  96
    2 conv    256  5 x 5 / 1    27 x  27 x  96   ->    27 x  27 x 256  0.896 BFLOPs
    3 max          3 x 3 / 2    27 x  27 x 256   ->    13 x  13 x 256
    4 conv    384  3 x 3 / 1    13 x  13 x 256   ->    13 x  13 x 384  0.299 BFLOPs
    5 conv    384  3 x 3 / 1    13 x  13 x 384   ->    13 x  13 x 384  0.449 BFLOPs
    6 conv    256  3 x 3 / 1    13 x  13 x 384   ->    13 x  13 x 256  0.299 BFLOPs
    7 max          3 x 3 / 2    13 x  13 x 256   ->     6 x   6 x 256
    8 connected                            9216  ->  4096
    9 dropout_TA    p = 0.50               4096  ->  4096
   10 connected_TA            D/TC:0 0 abort_handler:531 [abort] abort in User mode (TA will panic)
        E/TC:? 0
     409E/TC:? 0 User mode data-abort at address 0x0 (translation fault)
6  ->  4E/TC:? 0  esr 0x92000045  ttbr0 0x200001019d000   ttbr1 0x00000000   cidr 0x0
096
E/TC:? 0  cpu #0          cpsr 0x60000100
E/TC:? 0  x0<PHONE_NUMBER>000000 x1<PHONE_NUMBER>0946d3
.
.
.

This works well when I was tried on QEMU v8
To mitigate this issue, I found several solutions and the TA memory size should be set up like this

#define TA_STACK_SIZE                   (1 * 1024 * 1024)
#define TA_DATA_SIZE                    (10 * 1024 * 1024)

but when I tried this, the 'darknetp' command doesn't works with kernal panic
so I use the memory setting like this

#define TA_STACK_SIZE                   (1 * 1024 * 1024)
#define TA_DATA_SIZE                    (5 * 1024 * 1024)

It works fine with small size model, but I can't run some models like AlexNet that I mentioned above. And I got the same Kernal panic when I set up the TA memory size with over 6MiB.
So the main question is that is it possible to set the 10 MiB or more size of TA memory on RPI3 B+?
Thanks in advance,
Cheers

HI：
I am following https://optee.readthedocs.io/en/latest/building/devices/rpi3.html OPTEE3.5.0 was built on Raspi 3B+. When I inserted the SD card into the card, a red light flickered for 4 times. How can I build a relevant environment on Raspi 3B+


README does not explain how to fuzz

Hi! How can the system be used to fuzz? Can I create a randomised XML workflow with e.g. Peach and then this could execute it? Is there a set of allowed verbs (and required structures) in the workflow XML? I am happy to write a generative system to create XMLs and then execute them with TLS Attacker and make it open source -- i.e. I am happy to contribute back. I just want to know what's the best way to do this.
Note that I was one of the reviewers for the ACM CCS'16 paper and I'd like to now use the paper :D

Would it be possible to have an XML that has all the possible options in a structured way? Like for Heartbeat, we'd have:
<Heartbeat>
<payloadLength/>
</Heartbeat>

So we would know that that's the only thing we can change in Heartbeat. Obviously, there'd be a lot more in ClientHello, etc. That would allow me to create a generator for the XML :)

Hey,
in the most recent version we removed the fuzzing code since we moved it to a different project (after TLS-Attacker 1.2) which we heavily extended since CCS'16 with more advanced fuzzing strategies and anomaly detection mechanisms in a more structured and generic way. We plan to submit our advances in this field to CCS'18 (hopefully).  I guess the current version could somehow be used with peach fuzzer, but the upcoming fuzzer will probably make this unnecessary :)
If you still want to use the old fuzzer we are happy to assist you in this process.

Hey, I am sorry, the newest fuzzing code is not available yet...it is not stable and we do not want to support it for now.
The best way to implement your own fuzzer is to use it directly from the java code. However, I can understand you want to use XML. This can be done in some "hacky" way if you do not have that much time you want to invest in the project.
My fast and hacky approach with the newest TLS-Attacker version would be to generate some basic XML flows (see the TraceTool.jar). You can insert some expliciteValue modifications to the values of the fields. Then, you can use Peach to generate values for these modifications and let TLS-Attacker to execute the workflow.
I assume this will be very slow but can be used to execute some basic evaluations.
With TLS-Attacker 1.2, you can use fuzzing out of the box. The code is buggy and does not support so many TLS features so you will probably not receive such a good test coverage. But we can give it a try if you want :)

Hey! How does 1.2 support fuzzing out of the box? What is the options I must use? I can't seem to find the option :S Thanks!

(Also, thanks for the detailed answers, super-useful! I wrote you an email, too :) Thanks again for the detailed descriptions, really kind of you!)

I could find the old wiki entry for fuzzing. Maybe this explains how to get started.
You can get the old version here:
git clone https://github.com/RUB-NDS/TLS-Attacker.wiki.git
cd TLS-Attacker.wiki
git checkout 4a8f92d2369aeb0a933e4b216cbd8d4030f7e9b0

Fuzzing
TLS-Attacker provides currently basic fuzzing capabilities. The best way to use TLS fuzzing is to compile the analyzed code with address sanitizer, for example, OpenSSL as follows:
cd openssl
./config -fsanitize=address
make -j10

This ensures that the server crashes once a boundary violation is detected. See [[TLS Server Compilation|TLS Server Compilation]]
For fuzzing execution you need to download sources and compile TLS-Attacker since you have to rely on some resource files (which are not included in the release version). See [[TLS-Attacker Compilation|TLS-Attacker Compilation]] for more details.
You can then define the properties of the server you are fuzzing. This is done in an XML file:
<startupCommandsHolder>
    <serverCommand>[link-to-openssl]/apps/openssl </serverCommand>
    <serverPort>51000</serverPort>
    <workflowFolder>resources/fuzzing/workflows</workflowFolder>
    <outputFolder>/tmp/</outputFolder>
    <modifiedVariableTypes>TLS_CONSTANT,LENGTH,COUNT,PUBLIC_KEY,PADDING,SIGNATURE,PLAIN_PROTOCOL_MESSAGE</modifiedVariableTypes>
    <startupCommandsList>
        <startupCommands>
            <fuzzerCommand>simple_fuzzer -connect localhost:$PORT</fuzzerCommand>
            <serverCommandParameters>s_server -quiet -accept $PORT -key resources/rsa1024key.pem -cert resources/rsa1024cert.pem </serverCommandParameters>
            <shortName>openssl-rsa</shortName>
        </startupCommands>
        <startupCommands>
            <fuzzerCommand>simple_fuzzer -connect localhost:$PORT </fuzzerCommand>
            <serverCommandParameters>s_server -quiet -accept $PORT -key resources/ec256key.pem -cert resources/ec256cert.pem </serverCommandParameters>
            <shortName>openssl-ecdhe-ecdsa</shortName>
        </startupCommands>
    </startupCommandsList>
</startupCommandsHolder>

If you use this file as an input, TLS-Attacker starts two OpenSSL instances with the given parameters. In this case, one server is configured with RSA and the second with ECDSA.
Set the appropriate reference to your OpenSSL folder, and store the file in config-multifuzzer-openssl. Then, you can start TLS-Attacker with a reference to your config file as follows:
$ cd TLS-Attacker
$ java -jar Runnable/target/TLS-Attacker-1.2.jar -loglevel ERROR multi_fuzzer -startup_command_file config-multifuzzer-openssl

TLS-Attacker then starts the TLS servers and then proceeds in two stages. In the first stage, it attempts to execute cryptographic attacks. In the second stage, it attempts to fuzz the server with randomized variable modifications and randomized protocol flows.
You can see the output of the fuzzing in the folder defined by outputFolder. This folder contains invalid TLS flows accepted by the server, or TLS flows causing boundary violations. If you want to re-run a specific TLS flow, you can simply the given workflow file, e.g.:
$ java -jar Runnable/target/TLS-Attacker-1.2.jar client -workflow_input /tmp/terminated408204.xml

Note that the fuzzing implementation is still rather experimental...

Thanks, this really helped! Thanks for all, cheers!


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In 239a480, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 7343 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in eca76da.


bug: undefined variable ref in doc source - ks_kind_port_num

Describe the bug
When I look at https://docs.kubestellar.io/main/Coding Milestones/PoC2023q1/environments/dev-env/#create-a-kind-cluster-with-a-port-mapping right now, the displayed port mapping section looks like this:
  extraPortMappings:
  - containerPort: 443
    hostPort: {{ no such element: mkdocs.config.defaults.MkDocsConfig object['ks_kind_port_num'] }}
    protocol: TCP

Steps To Reproduce
See above
Expected Behavior
No doc built complaints shown
Additional Context
No response

ahh, when #1116 merges, this issue will go away.

fixed now that #1116 is merged


Input Needs InputLeftElement/InputRightElement

Similar to how we have InputLeftAddon, InputRightAddon we also need InputLeftElement, InputRightElement
More info here https://chakra-ui.com/docs/components/input/usage

We also need better docs for these feature but I can make a separate ticket for that.

Would like to work on this @Alek99 . Can you assign it to me?

Its very similar to add-on see how its done here https://github.com/pynecone-io/pynecone/blob/d9e4d6d13089f84a72db10c36d85029f62f0bfef/pynecone/components/forms/input.py#L79-L88
Should only be a couple new lines to add left and right element, you can add them in the same file as the link.

Unfortunately, someone has added it.


Allow http command to keep initial query parameters

Refers to #319

Looks like build failed because of bundler issue (https://github.com/bundler/bundler/issues/3558). We need to update bundler before next build.


fix: handle query exceptions gracefully

SUMMARY
Currently expected exceptions in visualization queries are not correctly caught, leading to a large number of unnecessary false positives on monitoring systems. This catches typical exceptions (e.g. TemplateError) and reraises them as QueryObjectValidationError (extends SupersetException) with more context, which are later caught and sent to the frontend without exception logging in the backend.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

Local testing with both legacy and new charts + ensure that no exceptions are logged
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #10548 into master will decrease coverage by 4.44%.
The diff coverage is 24.07%.

@@            Coverage Diff             @@
##           master   #10548      +/-   ##
==========================================
- Coverage   63.75%   59.30%   -4.45%
==========================================
  Files         767      767
  Lines       36269    36299      +30
  Branches     3426     3426
==========================================
- Hits        23122    21528    -1594
- Misses      13033    14578    +1545
- Partials      114      193      +79

Flag
Coverage Δ

#cypress
?

#javascript
59.89% <ø> (ø)

#python
58.96% <24.07%> (-0.41%)
:arrow_down:

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

Impacted Files
Coverage Δ

superset/viz.py
57.52% <12.50%> (-0.39%)
:arrow_down:

superset/common/query_context.py
83.75% <20.00%> (-1.51%)
:arrow_down:

superset/connectors/sqla/models.py
88.04% <30.00%> (-2.24%)
:arrow_down:

superset/views/core.py
74.32% <33.33%> (-0.23%)
:arrow_down:

superset-frontend/src/SqlLab/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/App.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/explore/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/dashboard/index.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

superset-frontend/src/setup/setupColors.js
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 155 more

Continue to review full report at Codecov.

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


Support archives backend

If tar.gz, zip, files can be supported as opendal backends, then we can read, walk, add-files-to-archive, etc seamlessly.
In go world viant/afs supports such abstraction, which was very convenient.

Interesting idea. We have a similar issue at https://github.com/datafuselabs/opendal/issues/312
But I hesitate about the correct API that opendal should expose to users.

Closing in favour of #312


Custom bidding and scoring logic behind a flag

Not totally clear on whether this is needed, since if you're vendoring this code and you've got logic that you don't want to open source, you can just put it in your local copy. But if you're using npm then that might be trickier. There are various implications to the different possible API shapes here, so will wait to investigate further until we know what users need.

In order to utilize this shim in something resembling a real-world scenario, multiple buyers and publishers will need to leverage the same deployment / iframe domain but will not want to share the same bidding / scoring logic. In this scenario, buyers and publishers will need to provide their own scoring logic which cannot reasonably be pre-packed with the deployment.

@brodrigu We have not been able to figure out a way to execute arbitrary JavaScript with the current web platform while preventing exfiltration.  This means we can't just implement decision_logic_url/bidding_logic_url as provided in the spec in cases where buyers and sellers need the spec's privacy guarantees.
We're planning to take two paths:

In cases where buyers/sellers are comfortable open sourcing their decision/bidding logic, we will accept PRs.  In many cases, we think the tricky decisions will still be on the server, and the client side logic will be a relatively simple combination of server-provided signals.  For example, someone might factor a bidding model into a vector computed based on publisher info (provided in per_buyer_signals) and a vector for ad info (provided in trusted_bidding_signals) with the bidding logic being a safe-to-open-source dot product computation.

For internal testing the code would have the option to use decision_logic_url/bidding_logic_url as spec'd, but without any security/privacy guarantees.

Since (2) needs careful thought we are planning to put it behind a flag.

@taymonbeal I do think this should be in the official repo, because one of the things people will likely want to test is fetching/caching, and that won't be realistic if fully vendored.

Changed issue title to reflect new approach for securing this.


Update whats-new-2306-preview.md / added warning about usage of the Supplemental Package

Added remark that Supplemental package is not supported and also not allowed as a base to upgrade from 22H2.
With this, the user should be made aware to avoid attempts using the supplemental package.
Despite the clear "preview" remark, the versioning 2306 - current version might indicate it is not outdated or not considered a blocker.
As of October 2024, there are use cases for new Azure Stack HCI 22H2 deployments or cluster / node re-installations.
Such as:

Stretched Clusters
customers that cannot accept the current limitations with their BCDR solutions
clusternodes that are affected by "out-of policy" and could not be recovered otherwise.

@alkohli
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@Jak-MS I will close this. Release is of no relevance. Hope for chance of faster review next time.

@Karl-WE  - These documents were removed with the Ignite release as these are past the support date.

OK I will drop the PR.


Dev/profile

Functionality:
added profile page
Solution:
added profile page and edit option to update user data
Risk level:

[ ] high
[x] medium
[ ] low

How to test:
ng serve
firebase emulators

Let's take quick action on this for tomorrows release.


Set poster image fallback src to undefined, not empty string

When loading a Poster image using the React API, I noticed react-dom (19.0.0-beta-26f2496093-20240514) presented this warning:

Warning: An empty string ("") was passed to the src attribute. This may cause the browser to download the whole page again over the network. To fix this, either do not render the element at all or pass null to src instead of an empty string.

I found a writeup of the empty string src problem from back in 2009(!). So I guess that kind of problem is what the warning is about.
This warning was triggered even if the Vidstack poster component was only ever rendered with a string, so seemed to me unworkaroundable.
I traced this back to this line in the Poster component, which at some point in rendering would fall back to setting src to an empty string.
It appears undefined is always preferable to an empty string here, so this change would appear to be low risk.
Note that I have picked undefined instead of null as the warning advises, since null appears not to typecheck. This also aligns with the approach in the SliderVideoProvider component (which also falls back to undefined for src on its video tag)
Thanks again for all of your work on vidstack and I hope this PR is useful.

Thanks!


TypeScript type definition

Hello!
It seems like this library is missing .d.ts definitions, and likewise didn't seem to find anything in @types. This would be great to have!

Hi, thanks for the suggestion. We don't have plan for typescript definition at the moment, but you are welcome to open a PR for this, thanks.


error: cannot use e (type *Endpoints) as type "k8s.io/apimachinery/pkg/apis/meta/v1"

Hi guys,
I am having trouble compiling the project. When adding the plugin in the plugin.cfg list I get the following error:
# github.com/coredns/multicluster/object
/go/pkg/mod/github.com/coredns/multicluster@v0.0.0-20211130154347-488880e11fc0/object/endpoint.go:104:2: cannot use e (type *Endpoints) as type "k8s.io/apimachinery/pkg/apis/meta/v1".Object in return argument:
	*Endpoints does not implement "k8s.io/apimachinery/pkg/apis/meta/v1".Object (missing GetZZZ_DeprecatedClusterName method)
/go/pkg/mod/github.com/coredns/multicluster@v0.0.0-20211130154347-488880e11fc0/object/endpoint.go:152:2: cannot use e (type *Endpoints) as type "k8s.io/apimachinery/pkg/apis/meta/v1".Object in return argument:
	*Endpoints does not implement "k8s.io/apimachinery/pkg/apis/meta/v1".Object (missing GetZZZ_DeprecatedClusterName method)
/go/pkg/mod/github.com/coredns/multicluster@v0.0.0-20211130154347-488880e11fc0/object/namespace.go:31:2: cannot use n (type *Namespace) as type "k8s.io/apimachinery/pkg/apis/meta/v1".Object in return argument:
	*Namespace does not implement "k8s.io/apimachinery/pkg/apis/meta/v1".Object (missing GetZZZ_DeprecatedClusterName method)

go version go1.18.2

What is the coredns version?

What is the coredns version?

v1.9.2

Test it with v1.8.7. It's works fine. Thanks!

@runakash I think PR https://github.com/coredns/multicluster/pull/8 is fixing the Issue https://github.com/coredns/multicluster/issues/7 .
To fix the error missing GetZZZ_DeprecatedClusterName method, we should implement methods SetZZZ_DeprecatedClusterName and GetZZZ_DeprecatedClusterName on Namespace and Endpoints, or we can update apimachinery vendor to this commit https://github.com/kubernetes/apimachinery/commit/430b920312ca0fa10eca95967764ff08f34083a3

I did have this issue while first fixing #7, and as it seems this was a breaking change in 1.24 (see also https://github.com/coredns/coredns/pull/5364) that I didn't want to deal with at the time, I worked around it then by pinning to the prior 0.23.5 apimachinery version in coredns while developing.
Oddly I'm not having this issue in my latest tests today though I'm now using<EMAIL_ADDRESS>from my latest coredns master branch rebase; which is earlier, and without the change in, https://github.com/kubernetes/apimachinery/commit/430b920312ca0fa10eca95967764ff08f34083a3, even though that change does look like the one we need 🤔

@lauralorenz Thanks for clarify this. 😃
<EMAIL_ADDRESS>should not work in anyway, that's odd, might be a dirty vendoring issue?


Carousel Caption and its animation

Feature request is related to

[ ] embla-carousel (core package)
[ ] embla-carousel-react
[ ] embla-carousel-vue
[ ] embla-carousel-svelte
[ ] embla-carousel-autoplay
[ ] embla-carousel-auto-height
[ ] embla-carousel-class-names
[ ] embla-carousel-docs (documentation)
[ ] embla-carousel-docs (generator)

Is your feature request related to an issue?

There is currently no option to include caption text for the carousel

Describe the solution you'd like

A clear and concise description of what you want to happen.

Describe alternatives you've considered

A clear and concise description of alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

Hi @kenbinta,
Please always create a new discussion before jumping into issues. You may copy paste your issue description to the discussion.
Best,
David


Support for virtualbox?

Not really a pull request, but how should I run the os via virtualbox?

Nah use qemu, and use mac or linux to run it, windows isnt good

You could use windows, as qemu works with it

you can also run it here (go to custom, choose hard disk image, click boot.iso and perma link to original repo:
https://github.com/lucianoforks/tetris-os/tree/856224d466f29de0cdbaa55686ca59769e4b34d8


Simplify directory recursive_size function

:notebook_with_decorative_cover: Description

Problem:

recursive_size has its own logic for "recursing" directories to get
the total size of files, but this can be implemented directly more or
less using std::filesystem.

Solution:

Use std::filesystem::recursive_directory_iterator to do the heavy
lifting of recursing into directories.

Note:

This requires linking explicitly with stdc++fs flag for GCC < 9.1.0.

:memo: Checklist

[X] All user-facing changes have changelog entries.
[X] The changes are reflected on docs.tenzir.com/vast, if necessary.
[X] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions

File-by-file.

Now you're getting to the meaty pieces! VAST started with its own header in vast/filesystem.hpp way before there was a standard library implementation available. In the long run, we want to get rid of all the functionality in there that duplicates things from std. So the obvious candidate is path. directory is also superfluous because it can always be replaced with what you just did: std::recursive_directory_iterator.
Our goal is to take one piece at a time from our own header and replace it with the corresponding piece from the standard library. However, this requires a little bit of discussion first, because we use the caf::expected<T> as error monad in our code base. We still need to come up with a reasonable proposal on how to take, e.g., std::create_directories and lift it such that we can use it like this:
if (auto result = attempt<std::create_directories>(path))
  f(*result); // work with result
else
  g(result.error()); // handle error

This is just an ad-hoc idea for composition. We still need to discuss with @tenzir/backend how to go after this. Maybe @tobim and @dominiklohmann can lead the discussion from here.

Our goal is to take one piece at a time from our own header and replace it with the corresponding piece from the standard library. However, this requires a little bit of discussion first, because we use caf::expected<T> as error monad in our code base. We still need to come up with a reasonable proposal on how to take, e.g., std::create_directories and lift it such that we can use it like this:
if (auto result = attempt<std::create_directories>(path))
  f(*result); // work with result
else
  g(result.error()); // handle error

This is just an ad-hoc idea for composition. We still need to discuss with @tenzir/backend how to go after this. Maybe @tobim and @dominiklohmann can lead the discussion from here.

I talked about this with @mavam earlier and we came to the conclusion that it's best not to wrap the standard interface at all, but rather forward the error to the call-site by manually wrapping it using caf::make_error(ec::filesystem_error, ...). We also don't use the filesystem API in many places—mostly the lsvast utility and within a few helper functions.

The macOS CI seems to be failing because of a missing formatter, maybe you can adjust the log message to make it work so we're not trying to print internals of std::filesystem.

Yeah, I saw that last night but hadn't  dug too deep into the spdlog -> fmt code path to see why it was failing on macOS only. fmt works fine for std::filesystem::directory_entry types by delegating to its string representation AFAIK. In any case, not a high priority to chase down. Instead, I changed the call site to get a string representation of the underlying path to keep things simple. Another alternative would be to introduce a fmt specialization which inherits from fmt<std::string> class template. That approach would be useful if we end up logging in more call sites with std::filesystem types when we rework path and directory to use their std counterparts.

Our goal is to take one piece at a time from our own header and replace it with the corresponding piece from the standard library. However, this requires a little bit of discussion first, because we use caf::expected<T> as error monad in our code base. We still need to come up with a reasonable proposal on how to take, e.g., std::create_directories and lift it such that we can use it like this:
if (auto result = attempt<std::create_directories>(path))
  f(*result); // work with result
else
  g(result.error()); // handle error

This is just an ad-hoc idea for composition. We still need to discuss with @tenzir/backend how to go after this. Maybe @tobim and @dominiklohmann can lead the discussion from here.

I talked about this with @mavam earlier and we came to the conclusion that it's best not to wrap the standard interface at all, but rather forward the error to the call-site by manually wrapping it using caf::make_error(ec::filesystem_error, ...). We also don't use the filesystem API in many places—mostly the lsvast utility and within a few helper functions.

Seems reasonable to use the error code interface of std::filesystem and forward it with our own error type where appropriate. That's what I was thinking as well when perusing the code for path/directory in vast.  As @mavam pointed out, the observation I had is "can we replace these vast types with their std equivalents"?  It seems there isn't a fundamental reason why this shouldn't be possible -- just work that can be done to simplify the codebase.

It seems there isn't a fundamental reason why this shouldn't be possible -- just work that can be done to simplify the codebase.

Yes, that's exactly what we concluded as well.
Note that we won't be merging this PR until tomorrow; I'd like to avoid merging until after this month's release.

It seems there isn't a fundamental reason why this shouldn't be possible -- just work that can be done to simplify the codebase.

Yes, that's exactly what we concluded as well.
Note that we won't be merging this PR until tomorrow; I'd like to avoid merging until after this month's release.

Sounds like a plan.  The pipeline is passing now across all compilers, so it's ready to be merged once you cut the release. Thanks!

ping @mavam @dominiklohmann -- this MR compiles and builds fine locally with your proposed changes in regard to error handling. CI is currently failing for Debian builds due to some broken packages from what I can tell (outside the scope of this MR).
I'm seeing errors like:
Processing triggers for libc-bin (2.28-10) ...
Warning: apt-key output should not be parsed (stdout is not a terminal)
OK
deb http://apt.llvm.org/buster/ llvm-toolchain-buster-9 main
E: Unable to correct problems, you have held broken packages.
Error: Process completed with exit code 100.

Yeah, looks an unrelated change. Overall, everything seems to be in place. I'll let @dominiklohmann going to do the final merge.

Merged despite the CI failure, which indeed is unrelated.


[TCA] Get swift build Compiling on Windows

This gets the source files building successfully on Windows via the swift build command.
I pulled in some changes from develop like the .vscode directory, the ci.yml addition, etc., but most of it was just adding the conditional imports and blocking out Apple-specific code.

@brianmichel for reference, we're reapplying the Windows compatibility changes on top of 1.3 since there were so many file changes. Would love a quick 👀 from you if you get a chance.


EntryPointNotFoundException: libvlc_media_tracks_get

Just wonder if anyone else had the same error?
I am using 2019.4.11, and have tested my rtsp link on a pc version vlc player to ensure if its working.
EntryPointNotFoundException: libvlc_media_tracks_get
bosqmode.libvlc.VLCPlayer.GetVideoTrack () (at Assets/UnityVLCPlayer/VLCPlayer.cs:168)
bosqmode.libvlc.VLCPlayer.TrackReaderThread () (at Assets/UnityVLCPlayer/VLCPlayer.cs:111)
System.Threading.ThreadHelper.ThreadStart_Context (System.Object state) (at :0)
System.Threading.ExecutionContext.RunInternal (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at :0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state, System.Boolean preserveSyncCtx) (at :0)
System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext executionContext, System.Threading.ContextCallback callback, System.Object state) (at :0)
System.Threading.ThreadHelper.ThreadStart () (at :0)
UnityEngine.<>c:b__0_0(Object, UnhandledExceptionEventArgs)
bosqmode.libvlc.LibVLCWrapper:libvlc_media_tracks_get(IntPtr, IntPtr&)
bosqmode.libvlc.VLCPlayer:GetVideoTrack() (at Assets/UnityVLCPlayer/VLCPlayer.cs:168)
bosqmode.libvlc.VLCPlayer:TrackReaderThread() (at Assets/UnityVLCPlayer/VLCPlayer.cs:111)
System.Threading.ThreadHelper:ThreadStart()

Thanks for getting back! I tried using 2021.2.11 as well, and did setup the folder as per the readme, not sure if there is a different with the plugin folder for 2019 and 2021.

Looking through the documentation here: https://www.videolan.org/developers/vlc/doc/doxygen/html/group__libvlc.html, it looks like the method libvlc_media_tracks_get() does not exist anymore.

Downloading the release version of libvlc seemed to work: https://artifacts.videolan.org/vlc/release-win64/. Also upgraded the project to 2021 and still seems to be functioning fine.