Add ability to ignore certs for Splunk in sink

From #23
In a development environment it would be helpful to ignore sinks.
An implementation similar to the Seq sink may be appropriate.

I too am in need of this. Is there any plan for it? Dotnet core doesn't seem to have a cert validation callback like the original dot net does.

@veccsolutions a new release is out with the HttpMessageHandler exposed.  I will update the wiki however it may be of use to your situation.

Thanks! I'll be checking this out in the next couple of days.

@merbla Thanks for exposing it. However, I can't specify whether to ignore it or not through the config files. I was able to write my own sink configuration method to do it using the one you have already created as a template.
My custom configuration method is identical to yours except for the following changes.

In project.json I increased the netstandard framework to 1.6 (I didn't try earlier versions, but the ServerCertificateCustomValidationCallback property wasn't in the netstandard 1.1 framework)
I added an ignoreSsl parameter, and the following between your null checks and newing up the eventcollectorsink object:         if (ignoreSsl)
         {
             //only ignore the certificates for correct host
             var uri = new Uri(splunkHost);
             var lowerHost = uri.Host?.ToLowerInvariant() ?? string.Empty;

 #if NET45
             ServicePointManager.ServerCertificateValidationCallback +=
                 (object sender, X509Certificate cert, X509Chain chain, SslPolicyErrors policyErrors) =>
                     ServerCertificateCallback((sender as WebRequest)?.RequestUri?.Host, lowerHost, policyErrors);
 #else
             HttpClientHandler client;
             //allow custom httpmessagehandlers
             if (messageHandler == null)
             {
                 client = new HttpClientHandler();
             }
             else
             {
                 client = messageHandler as HttpClientHandler;
             }

             //if the message handler is either HttpClientHandler or derived from it we can add to the callback
             if (client != null)
             {
                 client.ServerCertificateCustomValidationCallback +=
                     (HttpRequestMessage message, X509Certificate2 cert, X509Chain chain, SslPolicyErrors policyErrors) =>
                         ServerCertificateCallback(message?.RequestUri?.Host, lowerHost, policyErrors);

                 messageHandler = client;
             }
 #endif
         }

The ServerCertificateCallback method is:    private static bool ServerCertificateCallback(string requestHost, string splunkHost, SslPolicyErrors policyErrors)
    {
        //quick check for no error
        if (policyErrors == SslPolicyErrors.None)
        {
            return true;
        }

        return requestHost.ToLowerInvariant() == splunkHost;
    }

@veccsolutions you are correct.  The change was generally targeted at dotnet core and opening up the HttpMessageHandler for more than just the situation to ignore SSL certs.
We would have overload the config with an ignoreSSLValidation like param for the configuration from file to work.  I am a little reluctant to implement however I know it can be a pain in development.
Generally what frameworks are you targeting?

Just to add to @merbla's remarks about being reluctant to add this - I think we'd expose ourselves to some  criticism by dealing with SSL certificate policy in the sink. (Even in the case of self-signed certificates, it's usually better to add the certificate to the list trusted by the client machine rather than disable validation entirely.)

That's fine. I'll continue with my separate config method. I just need to be able to do it without managing the certs on the local system and still using the config file for different deployment targets like local development vs QA vs Production etc.

Closing this for now.


fix!(manifest): only poa is allowed to burn tokens

Only the POA admin should be allowed to burn tokens from its account.
I added a bunch of unit and e2e tests covering this functionality.
Fixes #65

Ill review this & merge after getting the payout refactor in (since that is larger).


⚠ Bofrost.fr has degraded performance

In f1d2d92, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 819 ms

Resolved: Bofrost.fr performance has improved in ee87019.


Switch Arc to Box and use Box::leak to avoid use-after-free

#449 introduced a potential use-after-free bug that would get triggered on the 2nd call to the httpsend opcode. This resolves that by instead using a Box.
The bug was caused by using an Arc to store a channel that gets used for sending HandlerMemorys to the http server that's running in the AVM. We store the Arc pointer in global memory, ensuring that it's accessible throughout the program's lifetime, but because Arc, Box, and Rc all use their Drop implementations to determine when to free their heap allocation, we have to ensure that those values don't get dropped. We could avoid this by calling Arc::incr_strong_count , but it's still an experimental API and we don't need to use Arc in the first place since its benefits over Box are non-existent in this use case. This PR changes that Arc usage to a Box instead to avoid the unnecessary overhead associated with Arc, and to avoid the use-after-free there's a call to Box::leak with the return value ignored to prevent the implementation of Drop for Box from getting called.

One quick question on all of this: How do we eventually clean it up once the HandlerMemory for the connection handler is freed?

I was under the assumption that the channel would be used many times across the runtime of the program, but in discussions yesterday realized that the channel is intended to be used exactly once, so this use-after-free bug only exists when a user calls the httpsend opcode the 2nd time in response to a single http request. To avoid this, I'll allow the Box to drop instead of leaking it, and instead remove the value from the HandlerMemory so that this bug won't exist.


EnvironmentConfig in solutions potentiall no depracted my actions.json

Ridiculous title, just delete solutions.grid_world.environment_config ❌🚮


WIP: Migrate to ghcr.io

Closes: #351
Signed-off-by: Michael Gasch<EMAIL_ADDRESS>Fixes #

Proposed Changes

Pre-review Checklist

[ ] At least 80% unit test coverage
[ ] E2E tests for any new behavior
[ ] Docs for any user-facing impact

Release Note

Codecov Report

Merging #434 (a8afd9b) into main (9018e18) will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #434      +/-   ##
==========================================
+ Coverage   81.62%   81.83%   +0.21%
==========================================
  Files          27       27
  Lines        1382     1382
==========================================
+ Hits         1128     1131       +3
+ Misses        226      223       -3
  Partials       28       28

Impacted Files
Coverage Δ

pkg/vsphere/adapter.go
64.73% <0.00%> (+1.57%)
:arrow_up:

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 9018e18...a8afd9b. Read the comment docs.


URLpilgrim issue : The tool is testing for non-links

When I test the tool for this  https://wiki.cdot.senecacollege.ca/wiki/Planet_CDOT_Feed_List url/html file the tool is testing for non-links

As you can see in the image
The tool is trying to test for Woosle and Wissmann which are not links.
This could be because the parser is going line by line instead of testing the complete document.
because in the link above there are urls that are on multiple lines.
Check out
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/match
The match command creates an array of all matches in a string

Thanks, @zg3d!
Everything should be working fine now.


Use Dask to optimize StochPy doStochSim

Gain an understanding of the Dask library by reading online documentation COMPLETED
Experiment with various Dask features on StochPy library COMPLETED
Implement Dask optimizations to doStochSim() and other StochPy functions as necessary OBSOLETE

Moving to Sprint 4, due to being unfinished in Sprint 3

After investigation into the viability of doing so and a discussion with the client this goal has been rendered obsolete and so is being closed manually.


[now dev] Functions request body is always base64 encoded

This has been reproduced using the now-python runtime but it happens with all function runtimes,
The problem is here, the function request is always encoded as 64
https://github.com/zeit/now/blob/49cba94178dd26ef13cb16d6e656e53d51b8b70b/packages/now-cli/src/util/dev/server.ts#L1335
This really adds a lot of overhead, i hope the production server doesn't do this.
The base64 encoding was probably done to avoid problems with binary data, but this can be solved encoding to base64 only when the content-type header is binary like

Thank you for creating this issue!
Please note that – in order for us to be able to handle each support request with care – we have to consume all of them through a unified pipeline that makes it easier for us to prioritize and track the progress.
In turn, the Issues tab on this repository is best only be used for reporting bugs in any part of our system, that we can immediately act on.
For detailed support requests (like the one you just posted), please contact us at<EMAIL_ADDRESS>so that our team can help you out directly.


How to use parameters with the runcommands

We had this query on the Bicep discussions.
Was thinking some of these examples might be useful in this page.

https://github.com/Azure/bicep/discussions/10553

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

ID: 04865202-8e09-967f-a7a7-b18ab6845b2e
Version Independent ID: 73268057-c2f4-c7be-6149-a8172bf68e35
Content: Run scripts in a Windows VM in Azure using managed Run Commands - Azure Virtual Machines
Content Source: articles/virtual-machines/windows/run-command-managed.md
Service: virtual-machines
GitHub Login: @nikhilpatel909
Microsoft Alias: erd

@brwilkinson
Thanks for your feedback! We will investigate and update as appropriate.

@brwilkinson
I'm going to assign this to the document author so they can take a look at it accordingly
@GabstaMSFT
Can you please check and add your comments on this doc update request as applicable.

#reassign @nikhilpatel909 who own this product.

acknowledge, Reviewing it .

Parameters usage examples are available at listed below URL, please review it and let me know if anything else is needed
https://aka.ms/runcommandmanaged

This thread was opened up against that page. There are no examples in that page. Please see the linked issue on the Bicep repo.

got it...Let me work on updating the doc and will update

Thanks @nikhilpatel909 I am sure if you decide to add some updates, it will be very helpful, since I have noticed the runcommands is popular for Bicep users.

Thanks for your contribution to our documentation
We sincerely apologize for the delayed response. Unfortunately, we have been unable to review this issue in a timely manner. However, we are making overall enhancements to our content. We are closing this issue for now as there has been no activity for a while. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Add invivo datasets

This would add the invivo datasets under a creative common license for easy re-use by interested parties.

Need to change the readme to indicate new data and add changelog for it when everybody agrees on the legal wording to be used.


npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed

I base on
    "@vue/composition-api": "^0.5.0",
    "@vueuse/core": "^2.0.24",

Can you try to run npm i again and see if the warning still preserves?

still the same
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

audited 5279 packages in 4.982s

30 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

Oh I see, semver treat 0.x version differently. https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004
I will make a change address to that. Thanks!

could close?

v0.0.26 released with the fix.

Wow still the same
npm i @vueuse/core@vue2 @vue/composition-api
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

+<EMAIL_ADDRESS>+<EMAIL_ADDRESS>added 2 packages from 2 contributors and audited 5279 packages in 6.706s

30 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

My package
"dependencies": {
    "@babel/runtime": "^7.9.2",
    "@quasar/extras": "^1.6.4",
    "@u3u/vue-hooks": "^2.0.1",
    "@vue/composition-api": "^0.5.0",
    "@vueuse/core": "^2.0.26",
    "apollo-cache-inmemory": "^1.6.5",
    "apollo-client": "^2.6.8",
    "apollo-link": "^1.2.14",
    "apollo-link-http": "^1.5.17",
    "apollo-server-express": "^2.12.0",
    "bcrypt": "^4.0.1",
    "chart.js": "^2.9.3",
    "decimal.js": "^10.2.0",
    "graphql": "^14.6.0",
    "graphql-load": "^0.1.1",
    "graphql-scalars": "^1.1.0",
    "graphql-subscriptions": "^1.1.0",
    "graphql-tag": "^2.10.3",
    "inputmask": "^5.0.3",
    "lodash": "^4.17.15",
    "merge-graphql-schemas": "^1.7.7",
    "meteor-node-stubs": "^1.0.0",
    "moment": "^2.24.0",
    "mongoose": "^5.9.9",
    "numeral": "^2.0.6",
    "perfy": "^1.1.5",
    "printd": "^1.4.1",
    "quasar": "^1.9.14",
    "shelljs": "^0.8.3",
    "simpl-schema": "^1.5.7",
    "to-css": "^1.2.1",
    "v-click-outside": "^3.0.1",
    "vee-validate": "^3.2.5",
    "vue": "^2.6.11",
    "vue-apollo": "^3.0.3",
    "vue-barcode-scanner": "^0.6.0",
    "vue-chartjs": "^3.5.0",
    "vue-clipboard2": "^0.3.1",
    "vue-count-to": "^1.0.13",
    "vue-json-pretty": "^1.6.3",
    "vue-meta": "^2.3.3",
    "vue-meteor-tracker": "^2.0.0-beta.5",
    "vue-router": "^3.1.6",
    "vue-types": "^1.7.0",
    "vuex": "^3.1.3",
    "vuex-router-sync": "^5.0.0"
  },

My bad. Fixed in 9ffd6e74386f91b029ebc226060be91021ec760e. Will ship in next release. Thanks

thanks again 👍


Add cstree

Now that this actually gets some downloads that are not me and several folks have pinged me on Discord, let's add it to the list.

Thanks!


Import fails when using node >= 7.3.0

Importing document-register-element in node fails with:
TypeError: Cannot assign to read only property 'prototype' of function 'class HTMLCollection {
      constructor(secret, element, query) {
        if (secret !== privates) {
          throw new T...<omitted>...
    }'

      at patchClass (../node_modules/document-register-element/build/document-register-element.node.js:1370:34)
      at polyfillV1 (../node_modules/document-register-element/build/document-register-element.node.js:1382:7)
      at installCustomElements (../node_modules/document-register-element/build/document-register-element.node.js:1393:48)
      at Object.<anonymous> (../node_modules/document-register-element/build/document-register-element.node.js:1433:1)

This seems to be because of: https://github.com/nodejs/node/commit/524f693872cf453af2655ec47356d25d52394e3d
They might revert it in Node: https://github.com/nodejs/node/pull/10920
Anyway, I thought I would mention this, since currently import fails when using Node 7.3.0 and 7.4.0.

wait ... what has this library to do with NodeJS exactly? I'm not sure I understand your heads up (thanks though)

Well, we use it with node, we are running tests with jest/jsdom. But yes, as you say, this isn't really your problem.


Exception calling "Save" with "0" argument(s)

I get this error when running the Import-Excel module in Powershell to append data to an Excel file containing 33 sheets, Pivot Tables and Charts :
Exception calling "Save" with "0" argument(s): "Error saving file C:\Users\nlsba\pivot.xlsx"
At C:\Program Files\WindowsPowerShell\Modules\ImportExcel\7.0.1\Public\Export-Excel.ps1:657 char:32
+             else           {   $pkg.Save() }
+                                ~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
    + FullyQualifiedErrorId : InvalidOperationException

It works when I try a clean document without any Pivot Charts. Did anyone experience a similar issue?
This is the code I use:
$excelSourcePath = $excelSourceTextBox.Text
$excelTargetPath = $excelTargetTextBox.Text
$excelItem = $excelSheetList.SelectedItem
$excelDate = $excelDateList.SelectedItem
$object = Import-Excel $excelSourcePath -WorksheetName $excelItem
$object | Add-Member -NotePropertyName Date -NotePropertyValue $excelDate
$object | Export-Excel $excelTargetPath -WorksheetName $excelItem -Append
}

Please post a repro. The smaller the better so it can be potentially debugged.

Solved the problem by removing a column with incorrect formulas!


Node 12 support and dependency/version updates

This PR adds support for Node 12 (tested with 12.13.0). The functionality of this package was also tested for Node 8 (8.16.2) and Node 10 (10.16.3).
Thanks to @abalabahaha for his changes regarding the Node 12 support.
The version of the NPM package was updated to 0.1.3 The reason is that the package published on NPM is currently on 0.1.2 and the last publish was 3 years ago. I plan on publishing a new version onto there and this time I would like to do it without a separate repository that introduces changes not existing in here.

This PR also makes #20 obsolete. The reason I chose to use abals changes is because I found them a lot cleaner and also made some improvements on using NAN more.


Bump @adobe/css-tools in the npm_and_yarn group across 1 directory

Bumps the npm_and_yarn group with 1 update in the / directory: @adobe/css-tools.
Updates @adobe/css-tools from 4.3.1 to 4.3.3

Changelog
Commits

updated-dependencies:

dependency-name: "@adobe/css-tools"
dependency-type: indirect
dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>

Thanks for your contribution.
Unfortunately, this won't be merged for the following reasons:

we don't use pnpm
we don't use this package directly


Remove python2 support

This PR standardises Python 3.6 as the lowest supported version of Python, dropping support for Python 2.7.
3.6 was chosen as that's where our CI runs tests, although it wasn't declared as such in the setup.cfg until I edited it. If this should actually be Python3.7 we should also remove 3.6 from CI.
I have removed all references to 2.7, mostly by removing all Python version checking code, and done some fairly trivial refactoring where appropriate.
This also adds some auto-formatting of whitespace/newlines on save to VSCode, as we don't use an autoformatter.
Closes #61

This PR is parked until #63 is merged and released, then this branch will need to be rebased.

@Araneidae I've made all the changes requested, and CI is now passing.
It seems the previous error is a transient problem with codecov that disappeared on re-runs.

Looks good to me.


revision for 0.12.0 stable release

hi @Jack-Ji just found your post on zig.news, nice approach, just revised for 0.12 (fyi trait was removed from meta https://github.com/ziglang/zig/pull/18061)

Thanks!


Multisite terms (>2)

Thanks for sharing this nice package!
Is it possible to define multisite terms beyond two sites?
For instance- Plaquette or star terms as in the Toric code model.
The currently available implementation only contains two site terms.
Thanks!
Snir

Hi Snir,
The multisite terms are implemented, similarly to how the twosite terms work:
operator = TensorMap(rand, ComplexF64, ComplexSpace(2)^4, ComplexSpace(2)^4)
H = @mpoham operator{i,i+1,i+2,i+3} for i in vertices(FiniteChain(20))

This should also work for the other lattice types. However, keep in mind that everything will get mapped back to an MPS, i.e. for a Toric code model on a torus this means that the interactions become long-ranged, so it might still become quite expensive.
Hope this helps!

Thanks for the quick reply!
With this approach, I would have to specify all tensor elements (right?).
Is there an equivalent (multisite) functionally to:
function S_xx(elt::Type{<:Number}, ::Type{Z2Irrep}; spin=1 // 2)
return contract_twosite(S_x(elt, Z2Irrep; spin=spin), S_x(elt, Z2Irrep; spin=spin))
end
to build the operator as a product over single site operators?
Thanks!
Snir

It depends a bit on the operator and the symmetries. It is always possible to join together factors that are symmetric by themselves, so in the trivial symmetry case you could just do:
S_xxxx = X \otimes X \otimes X \otimes X

For the case where you do have some symmetry, it becomes a bit more complicated, but if you can find separate factors that are symmetric, it more or less works the same:
S_xxxx = S_xx \otimes S_xx

This even works on the level of the operators as well:
H = @mpoham S_xx{i,j} * S_xx{k,l} [...]

the bottom line here is that the contract_twosite function is only necessary when there is a non-trivial charge involved for the single site operators, which then need an additional leg to compensate for this. The contract_twosite function then makes sure these get paired up correctly, such that the total operator is symmetric again.

This is currently not done. The only optimizations that are there are the following:

The mpo is stored sparsely, i.e. only the non-zero terms are multiplied.
The identity or scalar terms are stored separately, i.e. whenever you have a twosite term that acts on non-neighbouring sites, the transfer matrix is not computed by inserting a (braiding) identity tensor, but by implicitly leaving out that operator

You can play around a bit with the parallelization settings as well, there is a section in the documentation for this, where it allows parallelization over the different non-zero entries in the MPO

Can I close this? Are there any specific features you are missing?

Sure. Thanks again for your help!
It would be nice to have some automated compression of the MPO, but I guess it is too much to ask for :)

I agree, it is definitely somewhere on my list (although for now not too high up there ;))
In any case, this would definitely be a feature in MPSKit, so if you want to open an issue there as a reminder, that's definitely okay, but I will close this.
Out of curiosity, were you able to reproduce some results of the toric code?

Sorry for not getting back to you. I hope to get back to benchmarking the
toric code soon and will keep you posted
Snir
בתאריך יום ה׳, 13 ביוני 2024, 08:21, מאת Lukas @.***>:

I agree, it is definitely somewhere on my list (although for now not too
high up there ;))
In any case, this would definitely be a feature in MPSKit, so if you want
to open an issue there as a reminder, that's definitely okay, but I will
close this.
Out of curiosity, were you able to reproduce some results of the toric
code?
—
Reply to this email directly, view it on GitHub
https://github.com/QuantumKitHub/MPSKitModels.jl/issues/17#issuecomment-2164404562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABUFVYON7OZ4JW76BRHXRKLZHET6PAVCNFSM6AAAAABICYGKOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNRUGQYDINJWGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


1.3.x: Allow reporting Metrics on stop

Problem:
We use dropwizard extensively and are fans of how relatively quick it starts and stops.
When testing locally, we start our application from its all-in-one jar file and shut it down again after the tests, configuring metrics as local .csv files. However, we never see them :disappointed:
This is because dropwizard starts, we run tests and dropwizard stops again without ever getting the report() called from ScheduledReporterManager .
Solution:
Add a reportOnStop boolean setting in MetricsFactory that would cause one last report() by the ScheduledReporterManager when stopping.
Result:
This should have no other impact, as the default for this boolean is false to remain backwards-compatible.
This fixes #2557

I agree that the premise of flushing metrics (like logs) before shutdown is sound, though I think that this PR should be targeted towards the master branch. If it needs to be backported to a 1.x branch, it can be. Also can you include a test for deserialization, so we don't miss something obvious. Thanks!

Updated the Pull request to target master and made requested change + added unit test for deserialisation (missing property -> default false + minimum config with true)

Can you make sure that after merging to master, this also gets backported to 1.3.x?
We're still on the latest 1.3 version 😉

Can you update the javadocs and the configuration docs with this new value?

Can you make sure that after merging to master, this also gets backported to 1.3.x

No, I don't think that is how that works. We tend to reserve patch releases for security updates or major bugs

Isn’t 1.3 the official ‘current’ release though?

Op 16 nov. 2018 om 15:12 heeft Nick Babcock<EMAIL_ADDRESS>het volgende geschreven:
Can you update the javadocs and the configuration docs with this new value?
Can you make sure that after merging to master, this also gets backported to 1.3.x
No, I don't think that is how that works. We tend to reserve patch releases for security updates or major bugs
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This PR is good to go. Thank you for the contribution. I wouldn't think this would land in a 1.3.x release. A 1.4 (unplanned) may be better suited. 2.0 has been blocked for what seems like ages 😅


Provide a versioning information on the app for correlation

There should be a way to see an app version on UI. This has multiple benefits:

We know visually which app one is running.
It facilitates running multiple simultaneous experiments.

This is provided on the three dots menu that opens up the About modal.


Update filter.py

Add implementation equivalent to:
pfctl -t  -T test

Thank you very much @jauntysankey for this feature!


First simulation sphinx doc

Description of the problem
The explanation for launching the first simulation using Lethe was only available on the wiki.
Description of the solution
This PR migrates the explanation for launching the first simulation using Lethe to the new sphinx documentation.
How Has This Been Tested?
This does not require any test. The new section of the documentation compiles without warnings.
Future changes
None
Comments
None

One tiny change and this is ready for a merge.

Done :)


Change repo description: Python -> America

A simple change, the repo description should be changed to America, instead of Python,
Make Python great again -> Make America great again

The repo description is a play on Trump's tag line. America = Python

Ok, got it! :D

:+1:


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/114085

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/114365

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115066

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115075

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/115144


openrc init script

please add an init script for openrc
i created one, am not sure if it's all correct, and didn't know how to implement Restart=on-failure and RestartSec=3, probably not possible with openrc alone.
#!/sbin/openrc-run

description="pantalaimon daemon"
supervisor="supervise-daemon"
command="/usr/bin/pantalaimon"
command_args=""
command_background=true

depend() {
        after net
}

I can't get it to start as daemon for some reason, probably related to #37

Restart=<condition> and RestartSec=<secs> are implemented in systemd (the best PID 1 out there). Install systemd if you want real daemon support.

Restart=<condition> and RestartSec=<secs> are implemented in systemd (the best PID 1 out there). Install systemd if you want real daemon support.

Sorry, but i don't like binary logs or random vulnerabilities.
First, why does pantalaimon need X11 running? It's a daemon.
Second, why does it depend on systemd for daemonizing?
Third, why is it written in python and incredibly slow?
I don't want to tell someone else to do his job, but I'm telling someone else to do his job.
This is like making a program depend on a specific distribution.
And before you tell me to fix it myself, I don't bother with python and already looked into alternatives such as agitdl (WIP)

If your not interested why be so rude?
If you want to requested or talk about a new feature stay on point, spreading FUD doesn't help any software has bugs.

Second, why does it depend on systemd for daemonizing?

It doesn't need it but systemd helps to do some lifting like when to restart and so on.

First, why does pantalaimon need X11 running? It's a daemon.

A user daemon can do some things like sending graphical notifications..


[MM-48474]: E2E tests for the yearly subscriptions

Summary

Move the yearly subscription E2E tests to their own file.
MM-T5127

Checks that users are able to click on the yearly/monthly toggle and it updates the prices shown accordingly

MM-T5128

Check that the prices update accordingly when the "Seats" input field is changed

Ticket Link

https://mattermost.atlassian.net/browse/MM-48474
Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

NONE

@yasserfaraazkhan The deleted tests are the same as the "new" tests. They were basically just moved to a different file (with some slight adjustment to match the new UI).


Add support for ThunderCore (Mainnet and Testnet)

Website
https://docs.developers.thundercore.com/
ChainId
Mainnet: 108
Testnet: 18
RPC
Mainnet: https://mainnet-rpc.thundercore.com
Testnet: https://testnet-rpc.thundercore.com

Hi @nick8319 , deployment transaction is ready: https://github.com/safe-global/safe-singleton-factory/pull/98/files


Publish to crates.io

Before we publish, we need to decide who is owner of the crate.

As one of the owners of https://crates.io/crates/winrt, I think we can discuss whether you want to take over that crate name. From my perspective it makes no sense to develop two Rust language projections for WinRT in parallel (and also I don't have time to maintain https://github.com/contextfree/winrt-rust in a meaningful sense any longer).
I'm really happy that there is now an alternative that is supported by Microsoft directly 👍

Thanks Patrick, that would be ideal.

I would prefere if you could continue with the semver schema and release this project as 0.7.0.

I agree that it would be necessary to release the first version as 0.7.0 to be semver-compatible. And 7 looks almost like 1, doesn't it? 😉

Yeah, having this project take over the winrt crate name would definitely be fine.


[Bug] Downloaded Chapter is not treated as Downloaded Chapter

Device information

Tachiyomi version: Latest, <IP_ADDRESS>
Android phone & version: Google Pixel 3, Android 11

Steps to reproduce

Download any Manga
After Download finished, close the app. When I tried to reopen the App the downloaded manga is not treated as Downloaded, so when I try to read the chapter it stream again from mangadex (but the files are saved on the phone)

Expected behavior
Downloaded manga did not stream from mangadex again
Actual behavior
Downloaded manga is not treated as downloaded, so the app tries to stream again from mangadex
Other details
N/A

I assume the chapter has  a '
There is android 11 bug with files with '.

I see, I try to download Jojo's Bizzare Adventure though.
Thanks a lot for the confirmation @CarlosEsco! :D

<IP_ADDRESS>


Reduce

@Pragati1610 @mdhishaamakhtar reduce what? The PR title should be informative.

Reduce time for flushing the redis cache


Trino upgrade

Ticket Link #712
Description

Updated all apps that use prestige to use the new version that handles Trino
Updated pipeline to use the new version of prestige
Updated docker-compose files with the latest postgres, metastore, minio, and trino

Reminders:

[ ] If changing elixir code in an "app", did you update the relevant version
in mix.exs?
[ ] If altering an API endpoint, was the relevant postman collection updated?

[ ] If a new version of smart_city is being used (new fields on a struct), were the relevant postman collections updated?

@LtChae you'll need to update the forklift mix.exs


Security fix for Prototype Pollution vulnerability

:bar_chart: Metadata *
bouillon is vulnerable to Prototype Pollution.
Bounty URL: https://www.huntr.dev/bounties/1-npm-bouillon
:gear: Description *
Prototype Pollution refers to the ability to inject properties into existing JavaScript language construct prototypes, such as objects.
JavaScript allows all Object attributes to be altered, including their magical attributes such as __proto__, constructor and prototype. An attacker manipulates these attributes to overwrite, or pollute, a JavaScript application object prototype of the base object by injecting other values. Properties on the Object.prototype are then inherited by all the JavaScript objects through the prototype chain.
:computer: Technical Description *
Fix implemented by not allowing to modify object prototype.
:bug: Proof of Concept (PoC) *

Create the following PoC file:

// poc.js
const Bouillon = require('bouillon')

console.log('Before:', {}.polluted)
const bouillon = new Bouillon()
bouillon.set('__proto__.polluted', true)
console.log('After:', {}.polluted)

Execute the following commands in terminal:

npm i bouillon # Install vulnerable package
node poc.js #  Run the PoC

Check the Output:

Before : undefined
After : true

:fire: Proof of Fix (PoF) *

+1 User Acceptance Testing (UAT)

I've executed unit tests.
After fix the functionality is unaffected.

👋 Hello, @robertcorponoi. @arjunshibu has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above. If you want this fix in your repository, a PR will automatically open once you comment:
@huntr-helper - LGTM

☎ Need further support?
Come and join us on our community Discord!

❓@robertcorponoi - want more fixes like this?
Copy this snippet into your README.md for more vulnerability fixes in the future:
[![huntr](https://cdn.huntr.dev/huntr_security_badge_mono.svg)](https://huntr.dev)


VS17 Support

Is there a plan to make this extension available to VS17?

Working on this right now.

API for accessing project internal settings is public now[1], which is positive, but changed a bit so I've to learn it.
[1] https://github.com/Microsoft/VSProjectSystem

Done. No functional change was needed to support the new API. I've tested it with my usual simple test case (C++ only) and it works. No idea if the concept still works with C# projects: it used to work but I'm not using the extension actively.
https://marketplace.visualstudio.com/items?itemName=ceztko.SolutionConfigurationName


Well Done!

Ingenius!Your work has an excellent effect on my project!
tku~

Thanks a lot! I’m really glad I could help out.😊


tf.clip_by_norm documentation wrong

Click to expand!
Issue Type
Documentation Bug
Source
source
Tensorflow Version
TF2.8
Custom Code
Yes
OS Platform and Distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
clip_norm should be a 0-D (scalar) Tensor > 0 written on documentation https://tensorflow.google.cn/versions/r2.4/api_docs/python/tf/clip_by_norm#args. However, when the clip_norm is a negative integral, the code works.

Standalone code to reproduce the issue
import tensorflow as tf
results={}
try:
  arg_0 = tf.random.uniform([1, 5], dtype=tf.float32)
  results["res"] = tf.clip_by_norm(arg_0,clip_norm=-1)
except Exception as e:
  results["err"] = "Error:"+str(e)
print(results)
'''
{'res': <tf.Tensor: shape=(1, 5), dtype=float32, numpy=
array([[-0.54458696, -0.59545624, -0.31598726, -0.20027176, -0.4570559 ]],dtype=float32)>}
'''

Relevant log output
No response

@gadagashwini
The issue Was able to reproduce the issue on Colab using TF v2.9 and TF-nightly ,Please find the gist here for reference .
Thank you!

This seems to be a duplicate issue of https://github.com/tensorflow/tensorflow/issues/54414, I have submitted a fix internally to allow clip_norm value greater than 0.
Could you please close this issue and track the progress in the linked issue. Thanks!


Remove intermediary block bloom filters

This PR removes the computation of the intermediary bloom filters pre & post transaction execution.
The block bloom is kept as PI and consistency check is maintained as part of BlockMetadata::connect within aggregation proofs. The transaction bloom logic is unchanged.

PI is public inputs. We keep the bloom filter included in the block header (the final bloom), but we don't construct it internally anymore


fix labels list in templates

See issue #13615.
Since this is a P2 issue, let's take care of this issue sooner than later.

@kbhawkey this doesn't look to me like a 'test'... it looks more like a 'fix'. Am I understanding it correctly?
/lgtm

@tengqm, hoping this will fix the templates. Thanks!

/approve


Perform EDA to find best players

Perform EDA to find the top 10 best players according to:

HLTV Rating
K/D Ratio

Create different .csv files for the tasks and save them inside EDA and Feature Engineering Directory

May I work on this issue?

@RohanHBTU it's great to see you here. Absolutely!

I want to make a PR. kindly assign me this issue.

Sure @aarshgoel1502 !


Show only accepted AMI in new capacity advance

Currently we show both denied and accepted amis. We had a logic to filter out but looks like we use it wrongly. This change filtered out to only accepted AMIs when creating a new capacity.
Tested locally at env/canary/config/newcapacity/advanced/
before:

after:

I'm not sure if I can build a connection between the description and the changed code.

If we only show accepted, there are unknown and denied images. Should we not allow users to choose from those?

yeah let's keep this PR open and we can discuss more about that. We'll need to gather more context around that. But my current thought is AMI should be validated properly before being available to users. Note that this view is for creating a new capacity so using a declined AMI would be strange. Showing unknown and declined amis to users would introduce confusion instead (at least from not a newbie user like me).


Very low accuracy

I tried to compare the performance of this code 1:
from deepface import DeepFace DeepFace.stream(db_path='./', model_name='Facenet', detector_backend='mtcnn', 	time_threshold=1, frame_threshold=1)
run on windows 10, tensorflow 2.8 with no GPU support
with a similar framework with dlib and particularly with this example code 2 https://github.com/ageitgey/face_recognition/blob/master/examples/facerec_from_webcam.py (added my own pictures) on ubuntu 20.04
For each framework I added only 4 images (4 different people). MTCNN detects faces pretty good, but face recognition with code 1 works really really bad (similar performance or worse than opencv haar method), it can recognize myself only at particular lightning and position, while dlib (https://github.com/ageitgey/face_recognition) recognizes every frame really good with different lightning and position.
What could be the issue here? As I understand the face recognition performance with Facenet and dlib should be very similar.

You can try dlib within deepface.
DeepFace.stream(db_path='./', model_name='Dlib', detector_backend='dlib', time_threshold=1, frame_threshold=1)
Besides, those face recognition models are ready to use. I have not train them. Scores are shared in the original studies. Facenet researchers claim that they have 99% accuracy. If you cannot reach that score, nothing to do.


data dict updates

some updates to data dictionary spreadsheet to remove old "need to discuss" notes that are now resolved
stylistic changes corresponding to my PR in BFDR

This is not super important because we are not working off of this repo anymore, but updating sushi-config might be nice so it's easy to navigate to the updated links from the menu.

Should 'Observation - Input Race and Ethnicity Vital Records' be under coded content? It's technically input not coded, but is referenced in the coded content bundle


New cancel notifications methods

Add new methods:

cancelAllNotifications()
cancelNotification("message_id") (Android only). Not tested

closing in favor of https://github.com/arnesson/cordova-plugin-firebase/pull/403


[Feat]: Add support for $SHELLrc instead of assuming bashrc

Downloading https://go.dev/dl/go1.20.1.linux-amd64.tar.gz
Creating /root/.gvm/1.20.1
Updating symlink /root/.gvm/1.20.1/go => /root/.gvm
Adding /root/.gvm/go/bin to PATH
For ubuntu you must run [source ~/.bashrc] or logout/login to complete path installation
GVM is shadowed by /usr/bin/go in your PATH. Please update your PATH to use /root/.gvm/go/bin first
➜  packetgen git:(01942cd) go version
go version go1.18.1 linux/amd64
➜  packetgen git:(01942cd) source ~/.zshrc
➜  packetgen git:(01942cd) which go
/usr/bin/go
➜  packetgen git:(01942cd) go version
go version go1.18.1 linux/amd64
➜  packetgen git:(01942cd) echo $SHELL
➜  packetgen git:(01942cd)


Give more precise reasons for Subscriptions with no candidates.

A Subscription specifies certain criteria for selecting operators from
catalogs. The granularity of the Subscription criteria is, from least
to most specific: catalog, package, channel, startingCSV (if
applicable). If a user creates a Subscription and its criteria does
not match a single operator, the failure message now indicates the
highest level of granularity that excludes all available
operators. For example, if a Subscription specifies a package name
that doesn't exist in the referenced, catalog, the specified package
name will appear in the failure text.

/lgtm


Task mailbox

Convert Fast File Scan to use Task Queues
Flesh out ingest/outgest of task queues
Issue #1875

This change is

@utopiabound wrote this patch, and I've been through it quite a bit, so I'll +1


docs: fix README

:tada: This PR is included in version 4.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[FIX] Update Blog Thumbnail Images and Blog Footer

Overview
Our new blog implementation uses Ghost CMS and there are a few UI fixes to be made.

[x] Fix the blog thumbnail images.
[ ] Update the footer, use footer from webb-ui-components.
[ ] Update the webb logo link to point to webb.tools/

Thumbnail images have been addressed (see here: https://www.blog.webb.tools/).
I attempted to have logo redirect to webb.tool; but was unable to find a solution directly within Ghost; may require dev help.

Removed footer for now.
Updated the logo link to redirect to https://webb.tools/

https://github.com/webb-tools/webb-dapp/assets/53374218/ded6244d-951e-4226-8c68-8b4c2ae74766


add missing _fqin parameters

Description
Check each of the following when appropriate to help reviewers verify work is complete.
Modifying an existing version

[ ] I modified operator permissions in the OLM CSV and operator.yml
[ ] I modified the operator deployment in the OLM CSV and operator.yml
[ ] I modified operand permissions in the OLM CSV and ansible role
[ ] I modified CRDS in the OLM CSV and ansible role

Adding a new release version

[ ] I created a new z release directory in deploy/olm-catalog/konveyor-operator
[ ] I updated channels in the konveyor-operator.package.yaml
[ ] I created a new release directory in deploy/non-olm
[ ] I created or updated the major.minor link in deploy/non-olm
[ ] I updated the spec.skips parameter in the CSV
[ ] I updated downstream-prep.sh to only update the latest CSV patch version in the channel

Addresses https://github.com/konveyor/mig-operator/issues/377


Updgrade 1.23

Upgrade to Smithy version 1.23
Description of changes

Upgrade smithyVersion to 1.23.0 that adds support for IDL-2.0
Avoid emitting the available attribute for shapes deprecated in the prelude, this will keep the output unchanged.

Add the logic to support for enums shapes that now are their own type instead of being a trait over string shapes
Fix the recursive logic to annotate shapes to avoid infinite recursion

Scope

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Change looks good to me, please use conventional commit message when you merge your PR.
i.e. chore: update smithy version to 1.23.0


Disabling zniffer kills the app

Checklist

[X]  I am not using Home Assistant. Or: a developer has told me to come here.
[X] I have checked the troubleshooting section and my problem is not described there.
[X] I have read the changelog and my problem is not mentioned there.

Deploy method
Docker
Z-Wave JS UI version
9.13.0
ZwaveJS version
12.9.1
Describe the bug
enabling zniffer and then disabling it will cause the docker container to die.
zwave  | 2024-05-30 03:37:37.483 WARN STORE: scenes.json not found
zwave  | 2024-05-30 03:37:37.505 INFO LOGGER: Cleaning up log files...
zwave  | 2024-05-30 03:37:37.522 INFO APP: Listening on port 8091 protocol HTTP
zwave  | 2024-05-30 03:37:37.543 INFO ZNIFFERMANAGER: Zniffer is DISABLED
zwave  | 2024-05-30 03:37:37.545 INFO MQTT: MQTT is disabled
zwave  | 2024-05-30 03:37:37.554 INFO BACKUP: Backup job started with cron: 0 0 * * *. Next run: 5/31/2024, 12:00:00 AM
zwave  | 2024-05-30 03:37:37.555 INFO BACKUP: Backup job started with cron: 0 0 * * *. Next run: 5/31/2024, 12:00:00 AM
zwave  | 2024-05-30 03:37:37.566 INFO Z-WAVE: Connecting to /dev/zwave
zwave  | Logging to file:
zwave  |        /usr/src/app/store/logs/zwavejs_2024-05-30.log
zwave  | 2024-05-30 03:37:39.382 DEBUG SOCKET: New connection oXpeYRIRzo5Z8rS4AAAC
zwave  | 2024-05-30 03:37:39.383 INFO Z-WAVE: Setting user callbacks
zwave  | 2024-05-30 03:37:39.387 DEBUG SOCKET: New connection T-oFYwslkfKBvFNGAAAD
zwave  | 2024-05-30 03:37:39.428 DEBUG SOCKET: Event INITED emitted to oXpeYRIRzo5Z8rS4AAAC
zwave  |
zwave  | /usr/src/app/server/lib/ZnifferManager.js:23
zwave  |         return this.zniffer.active;
zwave  |                             ^
zwave  | TypeError: Cannot read properties of undefined (reading 'active')
zwave  |     at ZnifferManager.get started [as started] (/usr/src/app/server/lib/ZnifferManager.js:23:29)
zwave  |     at ZnifferManager.status (/usr/src/app/server/lib/ZnifferManager.js:115:27)
zwave  |     at Socket.<anonymous> (/usr/src/app/server/app.js:509:41)
zwave  |     at Socket.emit (node:events:529:35)
zwave  |     at Socket.emitUntyped (/usr/src/app/node_modules/socket.io/dist/typed-events.js:69:22)
zwave  |     at /usr/src/app/node_modules/socket.io/dist/socket.js:704:39
zwave  |     at processTicksAndRejections (node:internal/process/task_queues:77:11)

To Reproduce
enable zniffer in configs.
fill out keys and serial port.
then disable.
reload page
Expected behavior
should disable
Additional context
probable fix:
https://github.com/zwave-js/zwave-js-ui/blob/1c36405f0265a87acb5057ddef1782115570f66e/api/lib/ZnifferManager.ts#L74
	get started() {
		return this?.zniffer?.active
	}

No response

I think this happens only when zniffer is not connected for some reason, thanks for your issue. Fixed

Thanks for the quick fix. I think I'm also running into this currently on the zwavejs/zwave-js-ui:latest image from docker hub (I have zniffer disabled). The image is probably still one version behind. I'll await the new image.

All working now, thanks for the fix.


allow requirements to be specified on Mapper.resource

In OpenStack Nova we've used the prefix_path on Mapper.resource to
specify additional variables we want to capture (specifically
{project_id}). Project_id is a uuid. When trying to restrict
project_id to only valid uuid format a couple of issues were exposed.

#1 '/{project_id:[a-f0-9]{32}}/...' builds an incorrect regex
because of the nested {}

#2 the preferred method that works on connect() to pass
requirements doesn't work here (requirements are reset to only an
id match)

That leaves us with having to build a custom project_id match with 32
[a-f0-9] strings appended for every resource added to get the support
we need without effectively vendoring our own version of Mapper.resource.
This small change to allow requirements to pass through would make it
possible to get this tighter validation with much smaller regexes.

I'm fine allowing the requirements to make it through, but this PR is missing

CHANGELOG.rst entry
Test to verify it works as intended
Docstring regarding the new option

If you could update it with those, I'd be happy to merge.
Thanks!

Thanks, updated with I think all the feedback you provided.

Looks good, the only error is on an older Python where there is no assertIsNone apparently. Could just assertEqual to None for that.

Will update shortly with that fix.
-Sean
--
Sean Dague
dague.net
On Jan 13, 2016, 10:41 AM, at 10:41 AM, Ben Bangert<EMAIL_ADDRESS>wrote:

Looks good, the only error is on an older Python where there is no
assertIsNone apparently. Could just assertEqual to None for that.

Reply to this email directly or view it on GitHub:
https://github.com/bbangert/routes/pull/57#issuecomment-171334509

Thanks!


Temperature Humidity Sensor doesn't have pressure in REST API response

I'm trying to use SmartThings REST API to access Xiaomi Aqara Temperature Humidity Sensor.
But the response does not include pressure.
{
    "components": {
        "main": {
            "relativeHumidityMeasurement": {
                "humidity": {
                    "value": 42,
                    "unit": "%"
                }
            },
            "healthCheck": {
                "checkInterval": {
                    "value": 7320,
                    "unit": "s",
                    "data": {
                        "protocol": "zigbee",
                        "hubHardwareId": "000E"
                    }
                },
                "healthStatus": {
                    "value": null,
                    "data": {}
                },
                "DeviceWatch-DeviceStatus": {
                    "value": null,
                    "data": {}
                }
            },
            "temperatureMeasurement": {
                "temperature": {
                    "value": 19.9,
                    "unit": "C"
                }
            },
            "sensor": {},
            "battery": {
                "battery": {
                    "value": 83,
                    "unit": "%"
                }
            }
        }
    }
}

Would it be possible to add atmospheric pressure to the response?

I'm sorry to say that SmartThings does not officially support an air pressure capability when developing device handlers.
If you take a look at SmartThings capabilities reference page in the Classic Developer's Documentation and the new Developer's Documentation, you will see that no pressure capability is listed.
So, in order to generate air pressure value events, the Aqara Temperature / Humidity sensor device handler makes use of a custom attribute.
I do not use SmartThings' REST API personally, so I am not sure how to get custom attribute event values to appear in the response. I suggest searching the SmartThings Community Forums for more information.
I did find this thread where someone asked the same question.

Thank you a lot for the explanation, I'll follow the links!


Prevent crash on unnamed function calls [phab:D998]

Created by djrenren at 2016-08-05 00:11:51:
Cases where the function expression is not a path were causing crashes. Now ref/calls are only emitted on named functions.

zarko wrote at 2016-08-05 00:13:08:
You're missing function_unnamed.rs.

djrenren wrote at 2016-08-05 21:32:07:

Added test case

zarko commented inline (context lost during migration) at 2016-08-05 21:37:57:
nit:
too
much
vertical
whitespace

zarko accepted this revision at 2016-08-05 21:37:57

djrenren wrote at 2016-08-05 21:42:29:

Remove whitespace

djrenren wrote at 2016-08-05 21:49:01:

Added test case
Remove whitespace
Rebase onto master

djrenren closed via commit e5793da4b7eafe77f5d5a9e23b40590f8e42cafb at 2016-08-05 21:49:42


[Interface to MLJ] Interface to supervised MLJ models

MLJ is a machine learning framework that provides a large suite of common machine learning models, both supervised and unsupervised. This package can't currently be interfaced to MLJ due to differentiability issues: to be able to use any of our counterfactual generators to explain MLJ models, those models need to be differentiable with respect to features.
To solve this, we will take the following steps:

[x] Implement a basic interface to MLJ (an AbstractFittedModel for MLJ.Supervised)
[x] Investigate which models are compatible with the package, taking the steps outlined in issue #174
[x] Test all implemented functionality
[x] Document the entire process

Basic interface: #206
Implementation of EvoTrees from MLJ: #206
A report detailing our investigation into MLJ models: #206
Tests and documentation for EvoTrees: #206
Implementation of the DecisionTree and RandomForest models from MLJ: #195
The Feature Tweak generator for generating counterfactuals for DecisionTrees and RandomForests: #195
Tests and documentation for DecisionTrees, RandomForests and Feature Tweak: #195
The Growing Spheres generator for generating counterfactuals for EvoTrees, DecisionTrees and RandomForests: #243
Tests and documentation for the Growing Spheres generator: #243


with bank card images as input, i only get simple color blank images too, anything wrong?

Try to use square images for training, resize images might cause issues.


Mbox file won't import into new email

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Setup gyb
did a backup of new email account (worked correctly I believe as it made the folder and all of the email files that currently reside in new email)
Typed in command prompt: gyb --email<EMAIL_ADDRESS>--action restore-mbox --local-folder "C:\localfolder" --label-restored "name"

Expected outcome (what are you trying to do?):
exported all gmail emails from old account. Then downloaded the files from vault onto c drive. want to export files back to a different email address under the same domain.
Actual outcome (what errors or bad behavior do you see instead?):
Got the error:
File "gyb.py", line 1138, in get_db_settings
sqlite3.OperationalError: no such table: settings
Traceback (most recent call last):
File "gyb.py", Line 2203, in
File "gyb.py", line 1572, in main
File "gyb.py", line 1142, in get_db_settings
(AttributeError: 'OperationalError' object has no attribute 'message'
I've tried to delete backup folder. I've tried to move the location of the mbox file. I've tried to use another computer. I must be missing something.

Any chance you're trying to import mbox "folders" from Mac Mail? I'm having this exact issue.

I notice the problem was the folder need to be the "Takeout" with all the data not "Takeout/Mail"


[WB-3858] - Confusion matrix takes predictions as a default argument

Link to jira ticket:
https://wandb.atlassian.net/browse/WB-3858
This should allow users to either use predictions, or probabilties easily, so long as the probabilities are properly shaped
Result of standalone tests: https://wandb.ai/kylegoyette/imageTest/runs/iu2drzm4?workspace=user-kylegoyette

Thanks @KyleGoyette this looks great & is a much cleaner solution than I was originally thinking for the argmax default.

I think given how close "probs" and "preds" are, a usage example or two and maybe a bit more description on when to use which one could go a long way. I can also add this in myself.
is it ok to break backwards compatibility by placing "probs" as the first argument? or shall we just place "preds" as the first argument (less ideal) to preserve backwards compatibility with v0?


Wrongfully will match any word in the row

Requirements

[ ] Yes
[ ] No

WSL Version

[X] WSL 2
[ ] WSL 1

Version
Microsoft Windows [Version 10.0.22000.2538]
Kernel Version
No response
Repro Steps
Attempt to create a new user with user name matching with a word somewhere in the GECOS field will result in failure. Recommend adjusting the grep pattern to ^$username to look for matches in the username field only.
Expected Behavior
Allowing user to create new users with any user name, as long as it is not repeated actual user name.
Actual Behavior
Can't create users with user name matching word in GECOS field.

GitHub did not work as I expected; I am referencing this.

ok. will make the necessary changes in the next release


Drive-by fix for TODO

Description
Removes a TODO since the linked issue is long fixed.
Related Issues
https://github.com/dart-lang/sdk/issues/31543
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes tests for all changed/updated/fixed behaviors (See Test Coverage).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes).
[x] No, this is not a breaking change.


In light graph app, what kind of data receive?

Hello,
I have installed the driver-sensingkit and light graph app.
In this light graph,What data are shown in graph?

Light level over time, from the light sensor in your phone.

Light level means android phone's brightness level?
And only Light?
What is the purpose of the Mobile sensor data tab?

The sensingkit driver is general purpose, and can query any of those sensors, and write their data to its store. The light graph app only reads light sensor values from the store. Using the store API, you can create similar apps that read the data from one or more other mobile sensors in the store.
In other words, the driver can make data from any sensor available, but the app only asks for permission and uses the light sensor data. The driver can be used with any app that requires any mobile sensor data.

If I create new app that read the data from one or more other mobile sensors in the store,the sensingkit driver installation is also required even though when I have already installed the sensingkit driver for light graph app ?
Light sensor is where did receive the data from ?
I means the android phone of a screen brightness or light on/off ??

If I create new app that read the data from one or more other mobile sensors in the store,the sensingkit driver installation is also required even though when I have already installed the sensingkit driver for light graph app ?

No you don't need to install it again, that driver's store can be queried by many apps at the same time.

Light sensor is where did receive the data from ?
I means the android phone of a screen brightness or light on/off ??

In most phones, there's an ambient light sensor above the screen (next to speaker, front-facing camera, and proximity sensor). This is usually used to automatically adjust screen brightness depending on your surroundings, e.g. if it's a sunny day, the screen gets brighter. You can affect the value on the graph by covering your phone with your hands, or holding it against a lamp or window.

Thanks you :)
And Is there no export service in light graph app?
I want to export the data of light.

The light graph doesn't export anything, but if you want you can with a few lines modify it to do so and re-build it. The light graph app is built using our node library which has documented API endpoints for exporting data. I recommend starting there. You can also take the twitter sentiment app, or os monitor app as examples (since they export), and don't forget to add export permissions to the manifest.

hello, @yousefamar
About to export light data,
Just only adding the following code into light graph app's manifest.json file,right?
"export-whitelist": [    {
"url": "https://export.amar.io/",
"description": "Exports the calculated sentiment to amar.io"
}
],

Not quite; that manifest entry is only so that your app can have permission to export to that URL. You would also need to modify the source code to export the data you want. This is the function you want to use. E.g. in the most basic way, right before this line, you can add:
subs.on('data', (hostname, datasourceID, data) => {
	databox.export.longpoll('https://export.amar.io/', { data: data });
});

You can modify that as you see fit and rebuild the app — this will make it so that every time you get a subscription event, you export its payload (see app-twitter-sentiment does the same thing).


Addition of a "friendly" display name for Monitors

Hi,
First PR for this from me, more than happy if you deem it unnecessary and reject it since it's all for an issue I personally have.
I've added the ability for a user to assigned "friendly names" to the monitors listed for them.
I have a problem where I have two identical monitors, that show up with the exact same name in the drop-down menu.

My modification includes an editable cell in the preferences view where a user can specify a more meaningful name ("Left Screen" and "Right Screen" for example")

These names are stored in preferences and then propagated through the UI.

A couple of things that are probably worth noting:

The added logic in the DispayPrefsViewController increased the cyclomatic complexity of the tableView function to > 12 which triggers a linting warning. I didn't want to go disassembling things in there too much in case you rejected the entire pull request.
I haven't provided any localized strings for German/French. Happy to roll them in either part of the acceptance of this PR, or in a future PR.

It's been quite a while since I've done Prod level macOS development, so please feel free to critique anything you feel I need to address.

LGTM 👍 , we just need to update the translations and we'll merge


Importing FinetunerCalc leads to Segmentation Fault

I tried testing the QE example code (qe_gpu_online_al_example.py). However, Python quickly exits with only the message "Segmentation Fault".
When I try to run the code line by line in the interactive Python interpreter, I found that the following line causes the segmentation fault:
from finetuna.ml_potentials.finetuner_calc import FinetunerCalc
What could be causing this?
Some additional details:

I am on an AWS EC2 instance with GPU
I created my Python environment by calling conda env create -f env.gpu.yml. However, the code failed to run, and I found out that calling import torch; torch.cuda.is_available() returned False. So I then called pip3 install torch torchvision torchaudio to get a GPU-enabled Pytorch installed in my environment.

Turned out Pytorch was improperly installed


Master branch specs are failing with I18n::InvalidLocale

When I run the specs locally I get one failure:
Failures:

  1) Fill in user form finds and fills a spanish locale date field
     Failure/Error: I18n.locale = :es
     I18n::InvalidLocale:
       :es is not a valid locale
     # ./spec/features/fill_in_user_form_spec.rb:88:in `block (2 levels) in <top (required)>'

Master passed 'about a month ago' on Travis, so I assume a gem release since then has caused this breakage. I can't the inspect the Gemfile.lock created by the CI build to compare against my local version, so I'm not sure how to track down the cause of this.

Here's my Gemfile.lock.

I played around a bit more, and found that it passes intermittently. It looks like something the specs is order-dependent, since rspec --seed 31845 with this Gemfile.lock fails every time.

@calebthompson @andyw8 fixed in #53

@calebthompson since #53 is merged. We can close this issue.


Render view refactor & maxDataPoints=1

This PR refactors render/views.py to break up the monolithic renderView() function into a set of helpers for each output format, and to unify handling of response caching and render logging.
It also expands the render test suite to increase coverage for the new helper functions to 100% and cover more of the available query parameters.
  Previously this path used the same "nudging" logic as other maxDataPoints values, but that would often produce nonsensical results because it would "nudge" out a large number of data points.  With this change there is no longer any "nudging" with maxDataPoints=1 and all available points are simply consolidated.

Codecov Report

Merging #2054 into master will increase coverage by 1.08%.
The diff coverage is 90.41%.

@@            Coverage Diff             @@
##           master    #2054      +/-   ##
==========================================
+ Coverage   70.65%   71.73%   +1.08%
==========================================
  Files          78       78
  Lines        7555     7568      +13
  Branches     1483     1485       +2
==========================================
+ Hits         5338     5429      +91
+ Misses       1937     1882      -55
+ Partials      280      257      -23

Impacted Files
Coverage Δ

webapp/graphite/render/views.py
66.98% <90.41%> (+20.02%)
:arrow_up:

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 f6c262c...c1752ff. Read the comment docs.

Could you please elaborate what is "maxDataPoints=1", and why it could be processed that way?
I think this should be included also in documentation - https://graphite.readthedocs.io/en/latest/render_api.html#maxdatapoints

@deniszh the most common use for setting maxDataPoints=1 is the singlestat panel in Grafana, where doing that means that Graphite only needs to return a single data point to Grafana (instead of returning a list and having Grafana do the aggregation client-side, or hacking it in Graphite with |summarize("10y") or similar).
The problem with current master is that if you do that then typically you're going to throw out quite a lot of the points due to the "nudging", which makes the result more or less useless (and quite confusing to the end user).  If you look at the diff for c1752ff you can see that the tests show what happens to the values, the sum over the interval under the old scheme is quite a bit lower than you'd expect because most of the values are being thrown away.
I'm not really sure what we should put in the docs, to really explain it you have to first get into a discussion about how the "nudging" works to align the intervals when using maxDataPoints.  The way I look at it, with this patch the user is going to get back results that don't need to be explained, where current master you definitely need to understand what's going on "under the hood" to see why your value isn't what you'd expect.  Any suggestions?

OK, agreed. Does it really happens only for maxDataPoints=1 or just for low maxDataPoints?

The effect is there for other low maxDataPoints, but it's less pronounced and I can't think of any real-world scenarios besides 1 where you'd use less than 10 or so.


Reading a blockie

This pod looks really cool but is there a way to scan a blockie and get the data behind it?

No this is not possible. Blockies are generated with an algorithm which does a lot of xor operations to generate pseudorandom data with a seed. Reversing those operations is not possible.


More information

It would be very useful if the article had a section describing the differences between using text, buffer or stream. So one could see which one is best suited.

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: b36404b8-4385-81d9-2ee8-486f0a55fc37
Version Independent ID: 7b6ce4df-6ab2-3aa6-059f-b38aa9ea8062
Content: Create, write, and read a file - Windows UWP applications
Content Source: windows-apps-src/files/quickstart-reading-and-writing-files.md
Product: uwp
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for taking the time to let us know about this and we sincerely apologize for the delayed response. Due to the sheer volume of feedback in our repo, we are currently prioritizing new Issues. If you believe your feedback is still actionable, let us know either directly in this thread or by filing a new Issue and we’ll work to incorporate the suggestions into the documentation. Though the Issue will be marked as Closed, I will monitor any replies that come afterwards so feel free to treat this like an open Issue. Many thanks!


fix: :bug: Make sure queries which have dataUpdatedAt of 0 will be run

When initialDataUpdatedAt is a falsy numeric value (e.g. 0), it will cause it to be replaced with Date.now(). This leads to unexpected behaviour, as one would expect the query to be invalidated instantly, if staleTime is set to an amount less than the result of Date.now(). This change will check if the value is numeric, and if so, always use this numeric value.
BREAKING: This changes the behaviour of the plugin for those users that have specified initialDataUpdated at <= 0.

The problem is that a valid timestamp, 0 (number, 1970) is falsy, and
therefore gets converted to date now, the current timestamp (2020)
Op za 2 jan. 2021 18:33 schreef Dominik Dorfmeister <
<EMAIL_ADDRESS>
@TkDodo commented on this pull request.
In src/core/query.ts
https://github.com/tannerlinsley/react-query/pull/1556#discussion_r550902853
:

@@ -475,7 +475,11 @@ export class Query<
return {
data,
dataUpdateCount: 0,

 dataUpdatedAt: hasData ? initialDataUpdatedAt || Date.now() : 0,

that's true, but what else should it be? The types clearly state that it
should be number | undefined, and the docs say that it has to be a JS
timestamp in milliseconds.
I don't think that we need to check every option for every possible input

that's what TypeScript is for. Yes, there are also (lots of) JS
consumers, but if I put in a string ("1608412420052") - would it not be
weird if that were converted to Date.now() ?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-query/pull/1556#discussion_r550902853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOEZFJORWV5BUG4OH3NT5TSX5KEXANCNFSM4VRFT4GQ
.

The problem is that a valid timestamp, 0 (number, 1970) is falsy, and
therefore gets converted to date now, the current timestamp (2020)
Op za 2 jan. 2021 18:33 schreef Dominik Dorfmeister <
<EMAIL_ADDRESS>
@TkDodo commented on this pull request.
In src/core/query.ts
https://github.com/tannerlinsley/react-query/pull/1556#discussion_r550902853
:

@@ -475,7 +475,11 @@ export class Query<
return {
data,
dataUpdateCount: 0,

 dataUpdatedAt: hasData ? initialDataUpdatedAt || Date.now() : 0,

that's true, but what else should it be? The types clearly state that it
should be number | undefined, and the docs say that it has to be a JS
timestamp in milliseconds.
I don't think that we need to check every option for every possible input

that's what TypeScript is for. Yes, there are also (lots of) JS
consumers, but if I put in a string ("1608412420052") - would it not be
weird if that were converted to Date.now() ?

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-query/pull/1556#discussion_r550902853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIOEZFJORWV5BUG4OH3NT5TSX5KEXANCNFSM4VRFT4GQ
.

The problem is that a valid timestamp, 0 (number, 1970) is falsy, and
therefore gets converted to date now, the current timestamp (2020)

Yes, I think I got the problem, and using ?? instead of || would solve it with minimal changes. This is one of the prime reasons why nullish coalesce was introduced - so that you can easily get "fallback values" without trapping into the js falsy-ness of the number 0 / empty string. It will only convert undefined or null to new Date() and nothing else, which I think was the intention of the original code.

The problem is that a valid timestamp, 0 (number, 1970) is falsy, and
therefore gets converted to date now, the current timestamp (2020)

Yes, I think I got the problem, and using ?? instead of || would solve it with minimal changes. This is one of the prime reasons why nullish coalesce was introduced - so that you can easily get "fallback values" without trapping into the js falsy-ness of the number 0 / empty string. It will only convert undefined or null to new Date() and nothing else, which I think was the intention of the original code.

Ah sorry, I misunderstood nullish coalescing, and therefore your comment. I see now, it makes a lot of sense. I will change the PR accordingly.

Ah sorry, I misunderstood nullish coalescing, and therefore your comment. I see now, it makes a lot of sense. I will change the PR accordingly.

Alright, I updated the PR, was thinking whether I should have 0 or Date.now() on line 471, but ultimately this value would never be used anyway in the current implementation and is only used to satisfy typescript.

Alright, I updated the PR, was thinking whether I should have 0 or Date.now() on line 471, but ultimately this value would never be used anyway in the current implementation and is only used to satisfy typescript.

Nice! Could you add a test to cover this bug?

Nice! Could you add a test to cover this bug?

Test added

Test added

:tada: This PR is included in version 3.5.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 3.5.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Spark] Support taking UDT as pandas_udf input in Spark

Currently, Spark @pandas_udf does not support UDT for both input and output.
We should contribute to Spark to make the following code feasible.
df = spark.createDataFrame([Row(id=1, img=Image("s3://bucket/foo")), Row(id=2, img=Image("s3://bucket/foo"))]

@pandas_udf(ArrayType(StructType([
    StructField("score", FloatType()),
    StructField("box", Box2dType()),
    StructField("label", IntegerType()),
])))
def object_detection(iter: Iterator[pd.Series]) -> Iterator[pd.Series]:
    model = mlflow.pytorch.load_model("model://..../yolo")
    for series in iter:
         dataset = rikai.torch.SeriesDataset(series, batch=16)
         output_series = []
         for batch in dataset:
              predictions = model(batch)
              output_series.extend(predictions)
         yield output_series

df = df.withColumns("predictions", object_detection(df.img))
df.write.format("rikai").save("s3://to/results")f

The first task is be able to pass UDT (i.e., ImageType) into the pandas udf (i.e., object_detection)

https://github.com/apache/spark/pull/31735 is on returning UDT from pandas.  Would appreciate to help it moving forward.

I will create a pull request to https://github.com/eddyxu/spark/tree/lei/pandas_udf.

python version: 3.7.10
$ pip freeze
certifi==2020.12.5
numpy==1.20.1
pandas==1.1.4
pyarrow==2.0.0
python-dateutil==2.8.1
pytz==2021.1
scipy==1.6.1
six==1.15.0
xmlrunner==1.7.7

one of the error msgs of python/run-tests --testnames pyspark.sql.tests.test_pandas_udf_scalar
======================================================================
ERROR [0.326s]: test_vectorized_udf_varargs (pyspark.sql.tests.test_pandas_udf_scalar.ScalarPandasUDFTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/da/github/apache/spark/python/pyspark/sql/tests/test_pandas_udf_scalar.py", line 588, in test_vectorized_udf_varargs
    self.assertEqual(df.collect(), res.collect())
  File "/Users/da/github/apache/spark/python/pyspark/sql/dataframe.py", line 679, in collect
    sock_info = self._jdf.collectToPython()
  File "/Users/da/github/apache/spark/python/lib/py4j-<IP_ADDRESS>-src.zip/py4j/java_gateway.py", line 1305, in __call__
    answer, self.gateway_client, self.target_id, self.name)
  File "/Users/da/github/apache/spark/python/pyspark/sql/utils.py", line 117, in deco
    raise converted from None
pyspark.sql.utils.PythonException:
  An exception was thrown from the Python worker. Please see the stack trace below.
Traceback (most recent call last):
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/worker.py", line 601, in main
    process()
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/worker.py", line 593, in process
    serializer.dump_stream(out_iter, outfile)
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py", line 290, in dump_stream
    return ArrowStreamSerializer.dump_stream(self, init_stream_yield_batches(), stream)
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py", line 83, in dump_stream
    for batch in iterator:
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py", line 284, in init_stream_yield_batches
    batch = self._create_batch(series)
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py", line 226, in _create_batch
    arrs.append(create_array(s, dt, t))
  File "/Users/da/github/apache/spark/python/lib/pyspark.zip/pyspark/sql/pandas/serializers.py", line 184, in create_array
    array = pa.Array.from_pandas(s, mask=mask, type=t, safe=self._safecheck)
  File "pyarrow/array.pxi", line 887, in pyarrow.lib.Array.from_pandas
  File "pyarrow/array.pxi", line 292, in pyarrow.lib.array
  File "pyarrow/array.pxi", line 83, in pyarrow.lib._ndarray_to_array
  File "pyarrow/error.pxi", line 107, in pyarrow.lib.check_status
pyarrow.lib.ArrowTypeError: Did not pass numpy.dtype object

Switched to Python 3.8.8, and all of python/run-tests --testnames pyspark.sql.tests.test_pandas_udf_scalar passed.

For Apache Spark, from https://github.com/apache/spark/blob/master/.github/workflows/build_and_test.yml
Only Python 3.6 and 3.8 is tested in Github Action.

A bit confused, I can't find the output even with the --verbose option.
(spark) ➜  spark git:(da/pandas_udf) python/run-tests --verbose --testnames pyspark.sql.tests.test_pandas_udf_scalar
Running PySpark tests. Output is in /Users/da/github/apache/spark/python/unit-tests.log
Will test against the following Python executables: ['/Users/da/opt/miniconda3/envs/spark/bin/python3']
Will test the following Python tests: ['pyspark.sql.tests.test_pandas_udf_scalar']
/Users/da/opt/miniconda3/envs/spark/bin/python3 python_implementation is CPython
/Users/da/opt/miniconda3/envs/spark/bin/python3 version is: Python 3.8.8
Starting test(/Users/da/opt/miniconda3/envs/spark/bin/python3): pyspark.sql.tests.test_pandas_udf_scalar
Finished test(/Users/da/opt/miniconda3/envs/spark/bin/python3): pyspark.sql.tests.test_pandas_udf_scalar (36s)
Tests passed in 36 seconds
(spark) ➜  spark git:(da/pandas_udf) stat /Users/da/github/apache/spark/python/unit-tests.log
stat: cannot stat '/Users/da/github/apache/spark/python/unit-tests.log': No such file or directory

A WIP PR is created: https://github.com/eddyxu/spark/pull/1
I need more time to review the python part of the original PR. And move it forward.

No worries, please take your time @sadhen !  And thank you so much for taking care of this issue.

@eddyxu  IMHO, https://github.com/apache/spark/pull/31735 is too big to review.
How about splitting it into several PRs to Apache Spark.

Support UDT for DataFrame in PySpark
UDT input for pandas_udf
UDT output for pandas_udf

That makes total senses.  @sadhen . Thank you for owning it!

Just improved https://github.com/eddyxu/spark/pull/2
Still need to add more unit tests, and I think this PR is ready to review and merge.
If you think it's ok, please merge it and change the title of https://github.com/apache/spark/pull/31735: from SPARK-34600 to SPARK-34771， and let the Spark community do another round of code review.
Otherwise, let me add more tests and make more improvements according to your code review comments.

Let me create another PR to make PyLint happy.

Make PyLint Happy: https://github.com/eddyxu/spark/pull/3

Support for TimestampType in UDT is tricky and a common problem for SPARK-34600.
I plan to work on SPARK-34771 first and then handle the corner cases.

Hi @sadhen -- I've tried to follow the different PRs and JIRAs, so forgive me if I've missed it. Do you think at some point this work will support Pandas' extension API to enable exposing these UDTs to pandas without needing an intermediate conversion?


Some suggestions after using numerals Quests plugins that are available on Spigot

What does your feature do?
This is an excellent plugin, however it does have some shortfalls that make it really the one stop shop option.
Some suggest that would really make this plugin even more awesome:

Quest progress configurable notification on chat/textbox
(e.g., for Stone breaking quests, an option to notify the quester on the progress per interval).

Support for Scoreboard Quest tracking
(e.g., tracks for the accepted quests, tracks active quest progress)

Provide PAPI support so that other plugins can pick up quest progress and etc.,

Thank you
How would people use it?
For users/players to track their progress either on scoreboard or chat/text

Thanks for the input. I think you'll find these resources useful, in response to your list:

https://www.spigotmc.org/resources/questsbar.100634/
https://www.spigotmc.org/resources/placeholderexpansion-quests.69357/
https://www.spigotmc.org/resources/placeholderexpansion-quests.69357/


Beautify downloads/index.html

Overview
When you build this repository locally (or package it for viewing behind a firewall) the page at on-demand/downloads/index.html looks like this:

To improve the offline experience, we should add some formatting to this page to make it look more like the cheat sheets section at https://services.github.com/resources/#cheat-sheets
How to Contribute

[ ] Comment on this issue if you'd like to work/collaborate on it!
[ ] Fork the repository.
[ ] Use the GitHub Flow to make changes to your fork. This is a refresher course if you're unsure about how to make a change on GitHub.
[ ] Push your changes to your repository.
[ ] Submit a Pull Request

Base Dropdown: github/training-kit
Compare Dropdown: Your fork

Testing Locally
If you'd like to make and test changes locally (and see how they would look if merged), do the following:

In your command line:

Install Git
Install Ruby
[ ] Run script/setup
[ ] Run script/server.

When successful, the script will initiate a local server at http://<IP_ADDRESS>:4000/on-demand.

Questions?

Leave a comment on this issue! Make sure to use @ mentions if you want a specific person's attention!

.

@crichID Hey! I can work on this as my first issue!

@abdulhannanali That's fantastic, thank you! Feel free to fork and open a pull request, and let us know if you need or want any help along the way. 😄

Cool @brianamarie I'll start right away

@brianamarie Some questions before I start.

Should this downloads page use the layout similar to the other pages using Minimal Mistakes?
Checking out the link @crichID mentioned, on what basis are the given icons assigned.
What's better, An index file maintaining the file paths of all the resources within downloads for every locale or reading the directory to figure out the logic, the indexing based on directory  structure will be flexible but can cause problems, if anything with the path name or the meta data gets wrong, while maintaining a separate index with this data, will be better, but each file would have to be added separately in the index for a given topic?

If there's anything you think I am allowed to figure out myself do let me know. Don't want to bug you with unnecessary questions. Thanks a lot :)

@abdulhannanali These are all great questions! I'll do my best to answer, but any other @github/services-training members, please feel free to lend your expertise. 😊

Should this downloads page use the layout similar to the other pages using Minimal Mistakes?

Minimal Mistakes would be 💯 👍 great.

Checking out the link @crichID mentioned, on what basis are the given icons assigned.

Hm...these decisions were made before my time on the training team, so I actually don't know. @crichID Do you have any ideas on this?

What's better, An index file maintaining the file paths of all the resources within downloads for every locale or reading the directory to figure out the logic, the indexing based on directory structure will be flexible but can cause problems, if anything with the path name or the meta data gets wrong, while maintaining a separate index with this data, will be better, but each file would have to be added separately in the index for a given topic?

I have no strong opinion on this. Maybe @crichID, @stoe, or @hectorsector will have some thoughts?

What's better, An index file maintaining the file paths of all the resources within downloads for every locale or reading the directory to figure out the logic, the indexing based on directory structure will be flexible but can cause problems, if anything with the path name or the meta data gets wrong, while maintaining a separate index with this data, will be better, but each file would have to be added separately in the index for a given topic?

@abdulhannanali this depends on your comfort level. I will tell you that in the link @crichID shared, we don't get that fancy, and we just manually maintain the list 😉 If you're into Jekyll and can work through the logic, I'm game! If not (but you're willing to try it out later), I suggest 🚢ing a first iteration with manually added links, and then (if you'd like) we can collaborate on a second iteration using logic.

on what basis are the given icons assigned

@abdulhannanali that is a great question and it was a design decision more than a technical one. We can grab copies of those icons if you would like to reuse them here.

👋 @abdulhannanali !
Just wanted to check in. We're currently running through all of our stale issues and cleaning them up. Are you still interested/do you still have the bandwidth in helping us out with this one? 😄  We'd be happy to support you if you need anything!

Are you still interested/do you still have the bandwidth in helping us out with this one?

Actually @hollenberry I was just looking at this and I think the process is much simpler now.
The goal was to make this page better for offline viewing since resources/downloads lived on services.github.com instead of here on training-kit. Now that resources have been moved back to training-kit, I "think" we can just redirect anything from downloads/ to the resources/cheatsheets.md. @JasonEtco do you agree?

we can just redirect anything from downloads/ to the resources/cheatsheets.md

That would certainly be simpler; as far as I know, the content is the same so I have no problem doing that.

I've done downloads/index.html for better user experience (:


SNAT Port exhaustion

Version
4.16.0
Describe the bug
We're hosting bots created by the bot composer on a custom server in Azure, and our SNAT ports usage steadily climbs until we run out and start getting timeout errors.
To Reproduce
Steps to reproduce the behavior:

Create a bot (I believe that makes an external HTTP call, but not 100%)
Create a web app that will load the AdaptiveBot and run it (using bot.ContinueConversationAsync)
Run the bot over the course of a few hours/days in an Azure App Service, making sure to call the endpoint
Observe the SNAT port exhaustion using the 'diagnose and solve errors' tab in azure

Expected behaviour
SNAT ports should be reused
Screenshots

Additional context
I'm pretty sure that the issue isn't in the code we've written to host the bots, as the problem only appears in live and not our UAT/QAT environments. The only difference between the two are the bots that are hosted on them. I believe the problem might be in libraries/Microsoft.BotBuilder.Dialogs.Adaptive/Actions/HttpRequest.cs, where you're instantiating and disposing of a new HTTP client every time the dialog is called.
I'm going to try verifying this by writing a custom middleware, which injects a reused HTTP client into the TurnState. I'll update this issue if that solves it. I'd rather not rely on an internal implementation detail intended for testing, so a correct fix would be preferred.

Adding an HttpClient to TurnState via Middleware is the recommended course.


Identity parameter does not accept GUID

Connected to Exchange Online via PowerShell, I have been unable to pass a GUID to the identity parameter as this page indicates. I'm attempting to use the output of Get-MailPublicFolder (which contains the GUID) to retrieve the full path to the public folder with Get-PublicFolder.
With the following code:
$MailPF = Get-MailPublicFolder | where {$_.primarysmtpaddress -eq<EMAIL_ADDRESS>Get-PublicFolder -Identity $MailPF.GUID
get-publicfolder returns an error: Cannot process argument transformation on parameter 'Identity'. Cannot convert the "GUIDHERE" value of type "System.Guid" to type "Microsoft.Exchange.Configuration.Tasks.PublicFolderIdParameter".
The documentation should include an example of how to pass a GUID to the identity parameter, or the text indicating a GUID can be used should be removed if the Identity parameter only accepts the full path to the folder.

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

ID: c43e2506-2cce-f232-24b5-64755bfa2eaf
Version Independent ID: 7c604d12-69c0-e35b-683e-3e60226b0aad
Content: Get-PublicFolder (exchange)
Content Source: exchange/exchange-ps/exchange/sharing-and-collaboration/Get-PublicFolder.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@mschultz-bcbsne, apparently, there is no GUID property available on Get-PublicFolder. There's a MailRecipientGuid property whose value is the same as the GUID property value from Get-MailPublicFolder, but when I changed your command to:
Get-PublicFolder -Identity $MailPF.MailRecipientGUID
All I get as a result is:
Name                                                        Parent Path
----                                                        -----------
IPM_SUBTREE

as if I ran Get-PublicFolder by itself without any identity arguments. It looks like Get-PublicFolder demands an Identity value using the "\TopLevelPublicFolder\PublicFolder" syntax and nothing else.
I updated the Identity parameter description to remove GUID as an acceptable value in Get-PublicFolder, Remove-PublicFolder, and Set-PublicFolder.
Thanks for the feedback.


Is it possible to provide layouts, pages on site designs !?

On your documentation you say that you can create site designs to provide layouts, pages. But i see nothing about this somewhere.
Not enough available ? Error doc ?

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

ID: 54a433bf-b243-03ff-f733-484dca2be753
Version Independent ID: 7a173ed3-1700-b24f-ee78-417dd494ce16
Content: Get started creating SharePoint site designs and site scripts
Content Source: docs/declarative-customization/get-started-create-site-design.md
Service: sharepoint
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Good point ymihir
I would like to know how to provision a reusable modern page and set it as a homepage.
@vesa - Is this supported yet (as per the documentation) or is this in the dev roadmap? Alternately please suggest us what is the current recommended path to achieve this use case.
Thank You.
Jag

I have raised an issue on Github. https://github.com/SharePoint/sp-dev-docs/issues/3038
It seems that schema has option to create a page using verbs but, framework is throwing error when setting ClientSideApplicationId. The verb used in site scripts for creating page is "createPage". This might be addressed in future.

A comment from MSFT in the referenced issue #3038 states this is not currently supported at this time.


Add Logger

Codecov Report

Merging #2 into master will decrease coverage by 13.27%.
The diff coverage is 26.08%.

@@             Coverage Diff             @@
##           master       #2       +/-   ##
===========================================
- Coverage   76.15%   62.87%   -13.28%
===========================================
  Files           3        4        +1
  Lines         130      167       +37
===========================================
+ Hits           99      105        +6
- Misses         25       56       +31
  Partials        6        6

Impacted Files
Coverage Δ

log/logger.go
0% <0%> (ø)

log/packet.go
49.33% <33.33%> (-8.08%)
:arrow_down:

log/log.go
75.6% <42.85%> (-8.18%)
:arrow_down:

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 69ee31b...82d50b9. Read the comment docs.

Note: https://github.com/evalphobia/go-log-wrapper/releases/tag/v0.4.0


Update raidshamer

Add toggles for friends-only, outside raids, and fix TOB bug

raidshamer: 83baadd55cf853a7c53fbe4c102ae7a64dbbbeab...6f830decdc8a785a303ce59eedcb4b2c4727e7ad

new link:
https://github.com/jack0lantern/raidshamer/compare/83baadd55cf853a7c53fbe4c102ae7a64dbbbeab...ejedev:6f830decdc8a785a303ce59eedcb4b2c4727e7ad

I haven't thoroughly reviewed your PR, but please remove enabledByDefault = false in your plugindescriptor. Also, you still got raid shamer in a couple places. Might want to ctrl+shift+f that.

keep the original config name
Remove loadWhenOutdated = true


set the currency to paystack flutter plugin, but the dialog still display NGN

we set the currency to paystack flutter plugin, but still display NGN

My bad--I replicate and fix.

I believe the issue has been fixed. Please, reopen if you can still replicate it.

OK，I have verified and solved，very thanks,
But I found two other problems:
The first: Cannot delete by long pressing the delete button of the floppy disk on the card number input page;
The second: Error after entering invalid year or month

OK. Thank you.
Please, check the issues raised; I believe have addressed them.


Bad quality of icon in the titlebar of the window

When i set an image on Window it has a bad quality, though the real quality is good. I follow this guide, use painterResource("sample.png")
If i set the image using
Image(
    painter = painterResource("sample.png"),
    contentDescription = "Icon",
    modifier = Modifier.fillMaxSize()
)

it looks good

My OS: Fedora Linux

@eymar, yes, it works, the image has good quality, but it sets image only

Here. And it runs as separate application

I'm having the same problem.
I'm using Ubuntu LTS

Having similar (?) issues with 16 bit PNG images. Is there are any way to control filter quality of painter resources?

Similar issue in Ubuntu 22.04.2 LTS. I determined that regardless of the original resolution, the icon is always 32 pixels, if the system is set to 48px, then the icon of the KMM application is severely distorted.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Added action for android version code lookup

This is part of my attemt to implement an android versionCode auto-increment. This action uses supply (in a somewhat hacky way I admit) to lookup the highest version code of the current app on google play. It can then be used for something like
gradle(
  ...
  properties: {
    ...
    'versionCode' => lane_context[SharedValues::ANDROID_VERSION_CODE]+1
  }
)

See also #878

@aguenther Thanks for the PR. Using supply in this way isn't recommended since the client isn't intended to be accessed externally to the supply tool/action.
If you'd like to keep this functionality you can try using a local action instead.


Error while executing serverless invoke local -f

Error occured in serverless-plugin-chrome wrapper when trying to ensureChrome for default() handler. { flags: [ '--window-size=1280x1696', '--hide-scrollbars', '--headless' ],   functions: [ 'requestpdf' ],   chromePath: '/var/task/headless-chromium' } Error: Unable to start Chrome. If you have the DEBUG env variable set,there will be more in the logs.     at /Users/jesondias/Documents/oncore/oc-print/.build/.webpack/service/src/handlers/requestPdf.js:3208:13     at Generator.throw (<anonymous>)     at step (/Users/jesondias/Documents/oncore/oc-print/.build/.webpack/service/src/handlers/requestPdf.js:3142:191)     at /Users/jesondias/Documents/oncore/oc-print/.build/.webpack/service/src/handlers/requestPdf.js:3142:402     at <anonymous>     at process._tickDomainCallback (internal/process/ne
Tried using serverless-offline and chrome seems to throw an error too while i attempt a POST request on one of the handlers . could this be related ?

Possible duplicate of #89