Dupe glitch

I don't know where to post this, but my users have found that if they open, loot, and close rapidly on chests that are naturally spawned, it duplicates the gear over and over. Only gear that dupes is any gear modded by the plugin.

Hmm, that's not good and I'll definitely look into it.

What plugin version and Minecraft version are you running?
does it only happen if you do it really quickly, or always?
does it duplicate the exact same items again, or it just creates more and more different items?

Ok, yeah, sorry about posting there and here.  Found the link after I posted there.  Anyway, I am using AcuteLoot 1.0.2 and Paper 1.16.1 build 128. It happened with another user, I will see if he can reproduce while recording it for you. He described it as being done very rapidly and it was different drops I believe. I have messaged him but he is asleep right now. Since he knows the exact steps to reproduce it.  Also, I can get you anything you need.  Just let me know! I am always happy to help.

Thanks for your willingness to help out with this! I'm trying to wrap my head around what could be happening here. You wouldn't happen to have any gamerules or plugins that affect chest loot would you?

No other plugins for that no. And as for gamerule, only modified rule is to stop firetick. Everything else is default.

Hi, I am the user that found out the bug, It was whenever i opened a chest the was in a village or Temple, it would randomly spawn items anywhere from seeds to Netherite Armour and tools, It has slowed down but that could just Wvbigdave changing something. I don't have a recording of the happening but I am not sure what is going on.

Thank you both for more information. I'm still looking into it.

Did it happen specifically in villages and temples, and no where else like dungeons or sunken ships?

Was it specifically when you closed/open the chest really fast (like spam clicking)? Or any type of clicking, regardless of speed?

It would happen in villages and temples, I am not sure about sunken ships because I have not tried it. and when I spammed open and close is when you usually happened, I have not done it for a day or two, but If i hop on today and get it on record for you to see.

That might prove very helpful. Thank you!

I've tried opening chests in multiple villages now, opening and closing as quickly as I can and I'm still unable to reproduce the bug... I'm really not sure what be going on and I'm really curious to see your clip!

When I’m back at my computer in 1 hour I’ll record and try to get the dupe glitch

When I’m back at my computer in 1 hour I’ll record and try to get the dupe glitch

Any more word on this? I have an idea for what it could maybe be, but I'm still unable to reproduce it myself which makes it impossible to test adequately. It would be very helpful to see it in action.

I am soooo sorry this didn't get replied to.  I also have an idea because I have ran into a couple of other plugins that had and issue and I tracked it down to Drop2Inventory. But I would love to hear what you have.

I am closing this issue due to non-response. I have done extensive testing and have never been able to recreate the issue. If the issue occurs again, please provide detailed steps to recreate it. A video demonstration would be very helpful as well.


Show numeric values too

And one input field with alpha too.
Would allow for more fine grained control.
The picture is just a screenshot of the color picker in old photoshop

Hi, and thanks for the suggestion. Sadly, I don't feel like this fits with the aesthetic and idea behind the color-picker. If you would like to use an existing color value (eg. from a designer) you could just plop it into the Atom editor.

I am a designer myself and as many other designers I find myself designing and tweaking more and more directly in the code.
Those adjustments are often very small but getting e.g. 1degree hue color accuracy when adjusting is difficult with a slider.
Another solution could be to show the delta change as a tooltip when dragging.

I get that. However, colors, to me, are how they look – not the code value. Hence the design of the color-picker and the exclusion of direct code value tuning. If you would like to directly tune the code value, convert the color to a format of your liking, and do it directly in the editor. HSL and HSV, in particular, are great for doing just that.

I know - I only use HSL but I often find myself working on projects where people use e.g. hex :hurtrealbad:

Convert it all to HSL! Jokes aside, I'll definitely think about ways to improve the precision of color-picker.

Hehe...I would if I could ;)
I think tooltips when dragging would be a nice touch - could always be an option in settings.
You could also "go big or go home" - obviously as a settings option.


Add safe build tag to avoid all unsafe usage.

Even on platforms where unsafe is permitted, it is not always
desirable to include additional unsafe usage.  This change introduces
a 'safe' build tag which forgoes the terminal width detection on all
platforms to force a memory-safe build.
This needs testing on windows but it does cross compile for windows with and without the safe build tag.

I also came across https://godoc.org/golang.org/x/crypto/ssh/terminal#GetSize.  Is there a reason this isn't used instead of implementing the detection directly?

Ah, there is already a pull request for the terminal package: #308.
Either this PR or that one could be merged but it wouldn't make sense for both.  308 is probably the better solution but if no more deps should be introduced, this PR makes sense.

I'm going with #308, thanks for the effort though!


Automatic renewal of Certificates

I have installed this on my server fro 3 of my domains
Will I need to repeat this whole process every 90 day?
Can I set up some scheduler job, so that it will run daily and renew any certificate that needs renewal?

Yes of course, that's the whole point of Let's Encrypt and this program. A scheduled task for that is automatically set up during the creation of the first renewal.

Agha, so now I don't have to do anything, right?
The schedule task will automatically keep renewing the certificates, right?

Correct, your certificates will get renewed automatically every 55 days.

Wow, this is great Sir,
Thank a lot for this great job

You're welcome!


can't install the module

Hello,
i want to test your module with my arduino, but i can't install it :
meteor add donaldaverill:serialport
 => Errors while adding packages:

While checking for<EMAIL_ADDRESS>error: No compatible binary build found for this package. Contact the package author and ask them to publish it for
your platform.

I'm on a ubuntu 64 bits

Hi,
Thanks for the heads up. When I previously tried publishing with the newer version of serialport, I ran into this issue:
https://github.com/meteorhacks/npm/issues/76
I then accidentally dropped my arduino in a pint of beer and promptly abandoned the package update and marked the package on atmosphere as unmigrated(ie, broken). Anyway, I just now reverted to serialport 1.3.1 and republished and it seemed to work with publish-for-arch on OSX and Linux versions but not for Windows (got a similar shrinkwrap error).
I haven't had a chance to test it out, but give it a shot and let me know if it works for you or not.
Since this package is just a simple wrapper for the npm module, another approach you can try is to use the meteorhacks:npm package to add serialport to your app :
https://atmospherejs.com/meteorhacks/npm
Also, looks like there's another serialport package available:
https://atmospherejs.com/spectrum/serialport
Cheers!

Thanks for your answer. How is it possible to drop an arduino in a beer :) ? It was a small one ?
I've already tried with npm and it works just fine with version 1.4.6.
I will try with yours soon.
Thanks for your work !

It's an uno and as it turns out fits nice and snuggly in a standard pint glass. The arduino dried out and started working again a few days later. So, all is good there except for the stickiness.
I work in a brewpub and set the beer down on the ground to get it away from the electronics which were on a table I was sitting at, but the usb cable I was using at the time was about 10 feet long and super unwieldy. While tinkering other things attached to it, the uno fell off the table and dipped right into the beverage. Bullseye!

I'm on the wishey now and can confirm that it is definitely moving back to 1.3.1 of serialport. Cost me all night to figure out this topic gave me the answer...


Allow SASS/SCSS compiling

https://dev.to/mathieuhuot/processing-sass-with-11ty-5a09
https://www.belter.io/eleventy-sass-workflow/

TODO

[x] Finish Gulp SCSS setup!
[x] Move all static asset directories and files to the structure shown above (JS, favicons, fonts, img)
[x] Fix up URL paths in templates (they’ll be probably a bit everywhere, might need to Regex that), ensure all of these static files still work, and test them visually
[x] Rename all .css files to be .scss partials prefixed with _ — i.e. homepage.css becomes _homepage.scss_
[x] Create main.scss in the new css/directory
[x] Basing yourself on the structure and order of the current main.html (found in oa-works/src/), import all individual scss in that main.scssfile using the @import rule

Just created a branch where I installed node-sass (will add documentation) to package.json, converted the existing CSS into SCSS, then ran npm run scss to compile the SCSS files back into CSS. I've taken a look around running it locally with the changes and I don't think there's an issue with any misplaced CSS during the compiling — do let me know if you are able to see, @sssoz! Otherwise, I'll be sending a PR your way. :)

Added the list as TODOs in original comment.

Just some additional notes from the above TODO

[x]  Rename all .css files to be .scss partials prefixed with _ — i.e. homepage.css becomes homepage.scss
[x]  Create main.scss in the new css/directory
[x]  Basing yourself on the structure and order of the current main.html (found in oa-works/src/), import all individual scss in that main.scssfile using the @import rule
@import has been deprecated; SASS recommends using @use instead

TODO

[x]  Finish Gulp SCSS setup!
[x]  Move all static asset directories and files to the structure shown above (JS, favicons, fonts, img)
[x]  Fix up URL paths in templates (they’ll be probably a bit everywhere, might need to Regex that), ensure all of these static files still work, and test them visually
[x]  Rename all .css files to be .scss partials prefixed with _ — i.e. homepage.css becomes homepage.scss
[x]  Create main.scss in the new css/directory
[x]  Basing yourself on the structure and order of the current main.html (found in oa-works/src/), import all individual scss in that main.scssfile using the @import rule
[ ]  Test that all CSS output still works. Ghost Inspector might come in handy with this — we can do this together on our next call
Saving this last one so we can look at it together @sssoz!

Additional tasks:

[ ] Write documentation

@thinkbulecount2 Just checking in on this task. When we spoke on Friday, you mentioned wanting to finish this up (or finishing the a11y issues). Are there any updates on either?

I'm just adding in some documentation for this and then pushing (with instructions on how to deploy on Netlify) -- the a11y I will be finishing up EOD tomorrow.

Hi @sssoz, something came up yesterday so will push today and do a11y Wednesday. Sorry for the trouble!

@thinkbulecount2 No problem; do leave an update if possible.

@thinkbulecount2 I’ll let you close this one :)

⭐✨😊


Actor vs. Agent

In xAPI the Actor property can take an Agent (or Group) object. Agents can also be the object of the statement and can appear in the context.instructor property.
People often get mixed up and I myself often find myself using the wrong word and having to correct it!
In your syntax, you've used Agent where I think you mean Actor.

Yeah, I think the mixup happened because the syntax was originally only going to support Actor objectType Agent, but slowly the scope of this project is growing as I share it with others and receive feedback. I've updated Agent to Actor for the first part of the Remark / Statement.


Deprecated library warnings upon inaugural install

Describe the bug
My inaugural installation produced a few deprecated library warnings:

deprecated<EMAIL_ADDRESS>CircularJSON is in maintenance only, flatted is its successor.
deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax.
deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

To Reproduce
It was my first time installing:  npm install -g @aws-amplify/cli
Expected behavior
Consistent with OWASP Top 10 A9, I would expect dependencies to be kept up to date to minimize security flaws, and ensure maintainability.  It is also surprising to see an actively maintained project emit package deprecation warnings.
Screenshots
$ npm install -g @aws-amplify/cli
npm WARN deprecated<EMAIL_ADDRESS>CircularJSON is in maintenance only, flatted is its successor.
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to kleur@3 or migrate to 'ansi-colors' if you prefer the old syntax. Visit <https://github.com/lukeed/kleur/releases/tag/v3.0.0\> for migration path(s).
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
...
+<EMAIL_ADDRESS>added 1950 packages from 1311 contributors in 56.741s

Desktop

OS: Mac OSX 10.14.2
NodeJS 10.15.0
NPM 6.4.1

Good answer.  Sorry that I didn't think to check that.  I was just sharing my alarm.


Module 4 Assignment

@dgomes You are missing the module 4 assignment.

https://education.github.community/t/module-4-exercise-1-student-programs/26002/271


Update steps to create repo secrets

Why:
Closes #17791

What's being changed:

Everything mentioned by @github-actions in Content Changes table. In Dependabot article, removed one step which was not needed.
233604b - Updated the reusable to the latest steps
Check off the following:

[x] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[X] For content changes, I have completed the self-review checklist.

@thispsj Thanks so much for opening a PR! I'll get this triaged for review ✨

I'll review this as I need to check the behavior on GHES 3.3, 3.4 and 3.5.

Ok
On Tue, May 31, 2022, 2:02 AM mc @.***> wrote:

@.**** commented on this pull request.
@thispsj https://github.com/thispsj 👋🏻 - thanks for your contribution
✨
I've made some suggestions as I tested the behavior on dotcom, GHES 3.3,
GHES 3.4 and GHES 3.5.
I'd like to edit the {% data reusables.actions.sidebar-secret %} reusable
(found in data/reusables/dependabot/dependabot-secrets-button) but I wasn't
able to fetch your branch and add a commit.
I'd like to edit it so it contains the following:
{% ifversion fpt or ghec or ghes > 3.2 or ghae-issue-5658 %}

Inside the "Secrets" section of the sidebar, click {% data
variables.product.prodname_dependabot %}.
[image: {% data variables.product.prodname_dependabot %} secrets
sidebar option]
http:///assets/images/enterprise/3.3/dependabot/dependabot-secrets.png
{% endif %}

Can you help with that? (Not sure why I can't get your branch). Thanks!
In
content/code-security/dependabot/working-with-dependabot/managing-encrypted-secrets-for-dependabot.md
https://github.com/github/docs/pull/18175#discussion_r885256682:

@@ -49,7 +49,6 @@ The name of a {% data variables.product.prodname_dependabot %} secret:

{% data reusables.repositories.navigate-to-repo %}
{% data reusables.repositories.sidebar-settings %}
⬇ Suggested change
-{% data reusables.repositories.sidebar-settings %}
+{% data reusables.repositories.sidebar-settings %}
+{% data reusables.actions.sidebar-secret %}

In data/reusables/actions/sidebar-secret.md
https://github.com/github/docs/pull/18175#discussion_r885261219:

@@ -1,5 +1,5 @@

{% ifversion fpt or ghec or ghes > 3.4 or ghae-issue-5658 %}
-1. In the left sidebar, click {% octicon "key-asterisk" aria-label="The key-asterisk icon" %} Secrets.
+1. In the "Security" section of the sidebar, select {% octicon "key-asterisk" aria-label="The key-asterisk icon" %} Secrets, then click Actions.
Reverting as this just documents us getting to the Secrets option. Also,
this reusable is used in 4 different other files so this update would
impact these files too.
⬇ Suggested change
-1. In the "Security" section of the sidebar, select {% octicon "key-asterisk" aria-label="The key-asterisk icon" %} Secrets, then click Actions.
+In the left sidebar, click {% octicon "key-asterisk" aria-label="The key-asterisk icon" %} Secrets.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/18175#pullrequestreview-989911485,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASJB3XNC22GHGMFNP757FNDVMW2RVANCNFSM5WZMA3CA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

{% data reusables.actions.sidebar-secret %} is used in 4 files:

content/actions/security-guides/encrypted-secrets.md (3 occurrences)
content/actions/using-workflows/sharing-workflows-secrets-and-runners-with-your-organization.md (1 occurrence)
content/code-security/dependabot/working-with-dependabot\managing-encrypted-secrets-for-dependabot.md (2 occurrences)
/content/codespaces/managing-codespaces-for-your-organization/managing-encrypted-secrets-for-your-repository-and-organization-for-codespaces.md (1 occurence)

1 and 2 relate to Actions, 3 to Dependabot and 4 to Codespaces.
@thispsj  👋🏻 - so my recommendation would be to:
A- Update {% data reusables.actions.sidebar-secret %} so that it mentions clicking "Actions" inside "Secrets" (like you initially did in this PR). This takes care of 1 and 2. We can no longer use this reusable for 3 and 4 as it now specifically mentions Actions.
B. Add 2 new reusables, one in data/reusables/dependabot called sidebar-secret (so you'd use {% data reusables.dependabot.sidebar-secret %} when calling the reusable) and the other in data/reusables/codespaces (so you'd use {% data reusables.codespaces.sidebar-secret %} when calling the reusable), which should be used in the files involved in 3 and 4. In one, we instruct the user to select "Secrets" on the left sidebar, then click "Dependabot" inside the "Secrets" section, whereas for the the other one, we tell them to select "Secrets" on the left sidebar, then click "Codespaces"  inside the "Secrets" section.
What do you think? Do you feel comfortable with implementing this?

@thispsj 👋🏻 - I'm going to be away for a few days so once your new updates are ready, would you mind re-adding the waiting for review label to this PR, so that it gets added to our review board? Thank you 🙂

Hi @mchammer01 ,
How can I re label waiting for review ? I don't have triage access to this repo.

@janiceilene / @cmwilson21 - could you put this up for review once @thispsj has made the suggested changes? Thank youuu 🙇🏻 😄

@mchammer01 good idea, I'll just tag one of them once I'm done.

@cmwilson21 :wave:
This PR is ready for a review now !
Please add the waiting for review label so that it gets added to the review board.

Sorry for the delay on this one! I've resolved the conflicts and all looks good. I'll get this merged! 🚀

@lucascosti no problem 🙂

hhhhhmmmmm     ^^     and     $$


Can use in Unity?!!

You can use it with current Unity but it wouldn't be that useful as it does not support Mono, so all call stacks from Mono would appear with a raw address without a function name. Ultra is expecting .NET with CoreCLR.


The NUS-WIDE dataset

Hello! I have an question that I can get the NUS-WIDE.mat from your matlab implement but there is no nus-wide images dataset for pytorch implement. Can the author share the Nus-wide image dataset ?
Thank you very much !

I have update the links and password. You can download nus-wide dataset now.

You're so nice. Thank you very much.

Hi @jiangqy,
Where can I download NUS-WIDE dataset for pytorch?

@josianerodrigues Hi, please contact the author of NUS-WIDE. Mr. Xindi Shang, (shangxin AT comp.nus.edu.sg) from nus-wide homepage

Thank you!

nus-wide homepage is not opening. how can I download NUS-WIDE dataset.

I don't know is there any other resources. Maybe you can contact the authors of NUS-WIDE? @scpmail


feat(idempotency): Add raise_on_no_idempotency_key flag

Description of changes:
Add an option to raise an error when idempotency key value is None
GIVEN a persistence_store with event_key_jmespath = body and option raise_on_no_idempotency_key
WHEN getting the hashed idempotency key with an event without a body key value
THEN raise IdempotencyKeyError
Checklist

[X] Meet tenets criteria
[X] Update tests
[X] Update docs
[X] PR title follows conventional commit semantics

Breaking change checklist
Raising any kind of error when not passing in the data for the idemptency data is not what we currently do

@cakepietoast  @heitorlessa  - you might want to debate this.
For example when event_key_jmespath = "[body, queryStringParameters]"
and the event is {} then the data is [None, None]

Codecov Report

Merging #297 (8308fe1) into develop (f1a8832) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #297      +/-   ##
===========================================
- Coverage    99.71%   99.65%   -0.07%
===========================================
  Files           86       86
  Lines         3143     3145       +2
  Branches       149      150       +1
===========================================
  Hits          3134     3134
- Misses           5        7       +2
  Partials         4        4

Impacted Files
Coverage Δ

...wertools/utilities/idempotency/persistence/base.py
99.20% <100.00%> (+0.01%)
:arrow_up:

aws_lambda_powertools/utilities/parser/types.py
71.42% <0.00%> (-28.58%)
: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 f1a8832...8308fe1. Read the comment docs.

@cakepietoast @heitorlessa - you might want to debate this.
For example when event_key_jmespath = "[body, queryStringParameters]"
and the event is {} then the data is [None, None]

Also if we do raise an error, should it be a new type for Idempotency key errors?

@heitorlessa @cakepietoast  - i added a raise_on_no_idempotency_key option and a new new exception IdempotencyKeyError

I’ll look into this tomorrow morning and update here as soon as I can.
On Tue, 2 Mar 2021 at 14:49, Michael Brewer<EMAIL_ADDRESS>wrote:

@heitorlessa https://github.com/heitorlessa any concerns or changes
needed for this PR ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-python/pull/297#issuecomment-788922138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBGI5BSLL64OLRLETILTBTUHDANCNFSM4X7V2Q3A
.

Checking this now

Merging it - Thanks a lot @michaelbrewer 🎉


macOS support

Currently, we only support iOS and Android.
[snowplow-objc-tracker] also supports iOS 8.0+, macOS 10.9+, tvOS 9.0+ and watchOS 2.0+. Since the release of Flutter 2.0, Flutter also supports building macOS applications. I think we should therefore add support for macOS.
The underlying plugin code should be the same as an iOS. Is there any way to enable platform plugins for multiple platforms?

I believe its possible. This repo is probably the best to model the structure of: https://github.com/fluttercommunity/plus_plugins

The idiomatic approach to building support for multiple platforms is to adopt the Federated Plugin approach, where you have one platform interface package explaining what functions each platform should support/implement.
And for each platform. then a package that adds the support for this package.


Use Diffusers 0.30.0 / Enable offline single file model loading

Summary
This PR bumps diffusers to v0.30.0 and reenables the ability to load main checkpoint models without a connection to the internet.

Related Issues / Discussions
Closes #6623 with caveats.
Caveat 1: Diffusers 0.30.0 does not correctly load Stable Diffusion 1.5 models with prediction type v_prediction. See this diffusers issue.
Caveat 2: Except for canny 1.5, single file controlnet models fail to load properly under diffusers 0.30.0. An issue has been filed.

QA Instructions

Test generation with sd-1, sd-2 and sdxl models.
Test controlnet guidance
Test ti-adapter guidance
Test ip-adapter guidance
Test model loading when internet not connected.

Merge Plan
Given that this introduces the two regressions noted above, we should discuss whether to merge now or wait until the issue is fixed in diffusers.

Checklist

[X] The PR has a short but descriptive title, suitable for a changelog
[X] Tests added / updated (if applicable)
[X] Documentation added / updated (if applicable)

I'm going to try something a little different...

Are you still following this PR @lstein ? I can take it over and get it across the line if you'd like

I’m very happy to see renewed activity on this PR. I thought it was comatose. I’ll take it across the finish line.

Are you still following this PR @lstein ? I can take it over and get it across the line if you'd like
@brandonrising
I've removed the absolute path name from the controlnet json file and have corrected a merge conflict. Should be ready for a re-review.


Frontends to 'make all-depends-list' etc.

Poudriere has already 'poudriere options' that can be used for querying the set options or the defaults. I think it would make sense to provide similar frontends to 'make all-depends-list', 'make build-depends-list' and 'make run-depends-list' so that you could easily find the dependencies of a port using just poudriere. I don't have a ports tree at /usr/ports because I want keep the build host as clean as possible and running 'make all-depends-list' on a port would require for me either to nullfs(5) mount the default ports tree at /usr/ports or set custom ports tree and /var/db/ports locations in make.conf(5).

poudriere options is for setting options, not viewing options. What you're asking for does not deserve a sub-command. Just use testport -i to drop into a clean shell and check the depends from there.
make all-depends-list is also safe to run from the host.

My reasoning for these additional features is that if I have a heavily customized set of options under poudriere the dependencies can be very different from the defaults. I'd like quick way to review them without having to start a poudriere jail (requires root privileges) or go trough the hoops of setting custom PORTSDIR and PORT_DBDIR.


[Test] [FormRecognizer] Tests: ensure functional test parity with other languages

Goals

Read through other language's SDKs test projects and implement test scenarios that are missing in the .NET SDK. Using the same forms across languages is preferred.
Make sure that new tests pass and that any ignored tests are being tracked by GitHub issues.

Notes:

A test can be implemented differently or split into smaller tests as long as the same scenarios are being validated.
Tests that are language-specific or simply not applicable to our client can be skipped.

Missing tests have already been identified and will be added as part of #12438. Closing this issue.


Adds command spo user ensure

Closes #4497

Thank you @MathijsVerbeeck, we'll try to review it ASAP!

Thanks for the review, Martin. My apologies for the copy pasting misstake on --id 😄

Merged manually, thanks!! 👏


The app bundle of Arm64 architecture is 0.0mb.

I want to compile an app bundle for Arm64 architecture.
And get it.
flutter build appbundle --target-platform android-arm64
Initializing gradle...                                              1.2s
Resolving dependencies...                                           2.0s
Running Gradle task 'bundleRelease'...
Running Gradle task 'bundleRelease'... Done                        37.1s
Built build/app/outputs/bundle/app.aab (0.0MB).

flutter doctor -v
flutter doctor -v
[✓] Flutter (Channel stable, v1.5.4-hotfix.2, on Mac OS X 10.13.6 17G65, locale zh-Hans-CN)
    • Flutter version 1.5.4-hotfix.2 at /Users/cai/fluttersdk/flutter
    • Framework revision 7a4c33425d (4 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/cai/Library/Android/sdk
    • Android NDK at /Users/cai/Library/Android/sdk/ndk-bundle
    • Platform android-Q, build-tools 28.0.3
    • ANDROID_HOME = /Users/cai/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    • ios-deploy 1.9.4
    • CocoaPods version 1.6.1

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Ultimate Edition (version 2018.2.5)
    • IntelliJ at /Users/cai/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 31.3.3
    • Dart plugin version 182.5215

[✓] VS Code (version 1.34.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.0.2

[!] Connected device
    ! No devices available

! Doctor found issues in 1 category.

I use example url is example 's example.

I found #29224.
I don't know if it's repeated.


Suggest types - fixes #1446

@forki
Test errors:
+++ Warnings (SuggestTypesInModule.fs) +++
Microsoft (R) F# Compiler version 4.1
Copyright (c) Microsoft Corporation. All Rights Reserved.
(6,36): error FS0039: The type 'Lst' is not defined in 'System.Collections.Generic'.

Maybe you want one of the following:

   List

   IList

   Comparer

   Dictionary

   IComparer

*** The following necessary lines were never matched:
***	error FS0039:.+The value, constructor, namespace or type 'Lst' is not defined

*** The following necessary lines were incorrectly matched:

Unexpected Compiler Output
FAIL
``

Yep I'm on it

@enricosada @KevinRansom this is a bit tricky. the build doesn't come to neg2 test on my machine because other stuff is failing.
So I can't really check what's wrong.

ok this is ready for review

@forki
Okay looks good to me.  Pulling ...
Thanks for taking care of this :
The text "Maybe you want pne of the following" seems hokey to me.  Could we consider changing it to something more like "Consider one of the following:"
Kevin

Yes we should change that. I'm not native speaker so I hope others find a
good solution
Am 23.08.2016 10:12 nachm. schrieb "Kevin Ransom (msft)" <
<EMAIL_ADDRESS>
Merged #1449 https://github.com/Microsoft/visualfsharp/pull/1449.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/visualfsharp/pull/1449#event-765052864, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AADgNAtYqlUneurmEZt2fxwQv4m2htuuks5qi1Q3gaJpZM4JlY51
.


License clarification

Hi,
The metadata says MIT:
[metadata]
name = pip-requirements-parser
license = MIT

# description must be on ONE line https://github.com/pypa/setuptools/issues/1390
description = pip requirements parser - a mostly correct pip requirements parsing library because it uses pip's own code.
long_description = file:README.rst
long_description_content_type = text/x-rst
url = https://github.com/nexB/pip-requirements-parser

author = The pip authors, nexB. Inc. and others
author_email =<EMAIL_ADDRESS>
classifiers =
    Development Status :: 5 - Production/Stable
    Intended Audience :: Developers
    Programming Language :: Python :: 3
    Programming Language :: Python :: 3 :: Only
    Topic :: Software Development
    Topic :: Utilities

keywords =
    utilities pip requirements parser dependencies pypi

license_files =
    mit.LICENSE
    AUTHORS.rst
    CHANGELOG.rst
    README.rst

but you also carry Apache-2.0 license file in the repo. Is this an error?
Thanks.

@eclipseo good catch!
This is actually not an error, but this is is an error not to have updated the declared top level license in the setup metadata ... We had to extract a subset of an old version of the packaging library that is under BSD-2-Clause OR Apache-2.0 because of changes in this library and to ensure we could continue parsing legacy PyPI versions.
You could almost say that this is a cobbler's so problem of sorts ;)
See https://github.com/nexB/pip-requirements-parser/issues/10 for some history as of why we did have to backport this, thanks to @wellshs
You could almost say that this is a cobbler's so problem of sorts ;)
The definitive primary license for this package is MIT AND (BSD-2-Clause OR Apache-2.0) leaving aside the secondary licenses used in etc/ and tests/ files that are not used at runtime.
See https://github.com/nexB/pip-requirements-parser/blob/main/src/packaging_legacy_version.py.ABOUT and other ABOUT files that we use to document all origins and licenses too.


[minor] DeadLink in Docs

I'm opening this issue because:

[x] I have found a bug

The webpage https://claudiodangelis.com/qrcp/ has a LICENSE link at the end: https://claudiodangelis.com/qrcp/LICENSE. It returns a 404 error in Github Pages.

Good catch! Will fix it soon :-)

Fixed, thanks! :joy:


Cloning and changing name

Thank you, for creating this 👍
Just a quick issue I am having with NPM:

I cloned the repo (actually tried both ZIP and git clone)
Renamed the repo to something else for my project
Ran npm install and got following error:
"npm WARN<EMAIL_ADDRESS>No repository field."

Basically, unless I leave the name of the folder the repo is in the same as 'victor-hugo' I cannot build anything.

That's very weird, which version of NPM are you using?

Hey @axelsegebrecht. I'm closing this issue because we haven't seen any movement here. Other people are using victor-hugo without bumping into this issue. Please, feel free to reopen if you still is worth investigate it again.


Correctly set performance job metadata on kokoro

In the grpc-testing:performance_test.performance_experiment biquery dataset,
the metadata fields (e.g. metadata.jobName) are currently not being set because the upload script hasn't been updated since we migrated off of Jenkins.
Updating the upload scripts to use env variables set by Kokoro.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,951,564      Total (=)      1,951,564

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,662,105      Total (>)     10,662,098

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

Known failures:
https://github.com/grpc/grpc/issues/13727
https://github.com/grpc/grpc/issues/15599


Social Links need Emphasis

Description
The links to our community's activity sites are rather lost in the text - I'd suggest using Icons (like the one in my profile) or badges (shield.io or the like) to draw attention to these. If centred, it would also help reduce asymmetric whitespace

Can I work on this?

Go for it @krishdevdb!


Use OnBlockCommitted handler instead of OnBlockConnected

#84  👈
feature/proof-of-stake

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

Join @lazynina and the rest of your teammates on  Graphite


sway-audio-idle-inhibit: unstable-2023-08-09 -> 0.1.2

Description of changes
ChangeLog: https://github.com/ErikReider/SwayAudioIdleInhibit/releases/tag/v0.1.2

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 343530 run on x86_64-linux 1

1 package built:

sway-audio-idle-inhibit

Thanks!


nfs

Could I ask how you manage the NFS? Manual or by helm?

Hey guys, any update on the NFS? do you have to create a separate NFS to use with this chart? is there a way around this where you can just create the mounts in the deployment itself?

this is an example project. In this case I create the NFS pv/pvc in the chart. But you must operate the NFS server separately.


Crash Report

Issue Description:
What happens:
What you expected to happen:
Steps to reproduce:
1.I don't know. All I know is I got a pipe

Affected Versions (Do not use "latest"):

EnderIO:  EnderIO-1.12.2-5.2.66.jar
EnderCore:  EnderCore-1.12.2-0.5.76.jar
Minecraft:  1.12.2
Forge: 2855
SpongeForge  no
Optifine no
Single Player and/or Server?   Server

Your most recent log file where the issue was present:
https://paste.ubuntu.com/p/vH9stvgQRW/

pls describe your issue your having and what happen and how to reproduce the issue

pls explain your issue what happened and how did it crashed steps to reproduce it crash log does help but we also need to know how it crashed

Farming Station  ！！！！
Turn on the electricity and insert the capacitor will collapse indefinitely
(My English is very poor, I do not know if you understand!)  ):

Your "forge                             | <IP_ADDRESS>88           | CatServer-1.12.2-2019-11-27-server.jar" is not compatible with Forge 2847.

what forge 8888?

Your "forge | <IP_ADDRESS>88 | CatServer-1.12.2-2019-11-27-server.jar" is not compatible with Forge 2847.

Version degradation can solve this problem


predictive sliders improve parser

This provides a simpler parser. This is possible because now we are using the relevant variables from the signature and instantiating distributions at their default values. But this is not just a refactor, this also provides more flexible (and "natural") way of specifying the model. For example, this is a valid syntax b = pz.Normal(np.exp(a)*x, c).rvs(), when previously the np.exp would have needed to be applied on a separate line.
This also tries to be more "clever" and determine the boundary information (the info in parentheses next to the sliders name) of the parameters not only from the information of the parameter itself (e.g. a scale parameter, restricted to be positive), but also of the functions applied to them.
For instance a = pz.Normal(0, c).rvs(), c is restricted to be positive here, but it can be negative here a = pz.Normal(0, np.exp(c)).rvs() . Providing a truly general solution will probably be too hard, and it may require a different approach to parse the function. Something easier may be to solve a few common cases and for those that can not be solved return "(unk, unk)", signaling that the function is not able to guess the boundaries of the parameter. This will be complemented by allowing the users to modify the min and max values of each slider directly using text boxes, and/or passing a dictionary.

Codecov Report

Merging #173 (c60cddb) into main (915be0c) will decrease coverage by 0.20%.
The diff coverage is 79.48%.

@@            Coverage Diff             @@
##             main     #173      +/-   ##
==========================================
- Coverage   86.29%   86.09%   -0.21%
==========================================
  Files          26       26
  Lines        2474     2488      +14
==========================================
+ Hits         2135     2142       +7
- Misses        339      346       +7

Impacted Files
Coverage Δ

preliz/internal/plot_helper.py
93.20% <50.00%> (-2.70%)
:arrow_down:

preliz/internal/parser.py
94.44% <92.30%> (-2.53%)
:arrow_down:

preliz/predictive/predictive_sliders.py
92.30% <100.00%> (ø)

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


Pipfile not aligned with requirements

I got fooled when try to install dependencies with pipenv when I saw the project is containing a Pipfile, but it lacks all dependencies defined in requirements.txt

pip3 is what you need. Did you try that?

Should be fixed now


Order classes on @apply on .css and .sccs files

Is your feature request related to a problem? Please describe.
No problem.
Additional context
Just do exactly the same, but in front of @apply in .css (and friends) files.
I can try to help if you point me in the right direction!

I would expect we could do something like this…
"headwind.classRegex": {
    "css": "(?<=apply).*[^\\;]",
    "postcss": "(?<=apply).*[^\\;]",
 }

…but to be clear: this does not work.
I'm guessing that Headwind is specifically looking inside of HTML elements and not even looking side just a plan line of code (or in this case a CSS definition).

Works fine for me in *.pcss after I've duplicated css regex as postcss and restarted VS Code
"headwind.classRegex": {
  "css": "\\B@apply\\s+([_a-zA-Z0-9\\s\\-\\:\\/]+);",
  "postcss": "\\B@apply\\s+([_a-zA-Z0-9\\s\\-\\:\\/]+);",
}

Works fine for me in *.pcss after I've duplicated css regex as postcss and restarted VS Code
"headwind.classRegex": {
  "css": "\\B@apply\\s+([_a-zA-Z0-9\\s\\-\\:\\/]+);",
  "postcss": "\\B@apply\\s+([_a-zA-Z0-9\\s\\-\\:\\/]+);",
}

I don't understand how this works. How is "postcss" mapped to "*.pcss"?
In my case I want it to work on *.sass files (which it doesn't). What should I add?

I don't understand how this works. How is "postcss" mapped to "*.pcss"? What is the list of possible file type keys?

https://code.visualstudio.com/docs/languages/identifiers

In my case I want it to work on *.sass files (which it doesn't). What should I add?

In that case I think you should use scss. Not 100% sure but you won't waste much time trying it out :wink:


New Unit JoJo

A new unit —— JoJo
Shape: XuKun Cai
Bullets: basketball and chicken

The second commit w.r.t. https://github.com/Yao-class-cpp-studio/battle_game/issues/36.

The third commit w.r.t. https://github.com/Yao-class-cpp-studio/battle_game/issues/126.


Clean up automated tests and code for topic history

Following ef92fcbe2b7e580df17c5fa89415bcfdbcf20f00, our topic history feature works as intended, but the unit tests are kinda messy and repetitive, and I'm not super happy with having two long database queries written out.  I think the DB queries could potentially be cleaned up with either the Django or sqlalchemy ORM; meanwhile, the test code doesn't use our normal ways of sending messages and just feels a bit fragile structurally.
@showell this may be a good thing for you to knock out, since you have expertise on the code in question.

Closing, since stream_topic_history.js now has full test coverage.


Dennis Batiste- Sprint 2

Join and Reshape datasets

Join Data Practice

First, write down which columns you need and which dataframes have them.

It's a good idea to do this for future you, or anyone else looking at your notebook.

Next, merge these into a single dataframe.

Make sure to concat the order_products_prior and order_products_train tables together to get an accurate count.
What columns do you really need to answer the question? Writing down what you need is the first step for a reason :)
Try simplifying things by just merging first, then using groupby later. You've got a lot of long chains of functions that are hard to follow and the operations could be done separately. Most of them are not necessary to answer the question.

Then, use pandas functions from the previous lesson to get the counts of the top 10 most frequently ordered products.

Again, very creative with the groupby attempt. I'm pretty sure you could get there by that route.
You got the correct order, but your counts are off from not including the order_products_train.
Keep it simple, just use merge and value_counts.

Reshape Data Section

✅ Nice and clean. Some commenting is always good.

Replicate the lesson code

✅ Same as above. Good job having cells that tell a story. Good flow.

Complete the code cells we skipped near the beginning of the notebook:

Table 2 --> Tidy ❌ Nice work with table1, have a go with table2. :)
Tidy --> Table 2 ❌

Stretch goal

You can use groupby to answer the first one here!

Nice work, Dennis!


Renewing SSL cert using digitalocean DNS challenge fails

I have a setup using the digitalocean DNS challenge to get SSL certs from letsencrypt, specifically a wildcard cert for my domain, which is only available when using the DNS challenge.
I could generate the cert using nginx-proxy-manager without any problems, however I couldn't renew the certs. The webui just gives me a "timeout" error, in the docker logs for the container I found the following:
[2/3/2021] [8:58:38 AM] [SSL      ] › ✖  error     Error: Command failed: /usr/bin/certbot renew --non-interactive --quiet --config "/etc/letsencrypt.ini" --preferred-challenges "dns,http" --disable-hook-validation
Renewal configuration file /etc/letsencrypt/renewal/npm-2.conf (cert: npm-2) produced an unexpected error: 'Namespace' object has no attribute 'dns_digitalocean_credentials'. Skipping.
0 renew failure(s), 1 parse failure(s)
    at ChildProcess.exithandler (child_process.js:308:12)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1051:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:287:5)

I have absolutely no idea where to start debugging, as the issuing of the certificate was no problem at all. Any help would be really appreciated!

Having same issue here. While i can delete  then re add the cert i can't renew them.
Not wildcard but subdomains.  Skipping.

Could you add what the content of your configuration file /etc/letsencrypt/renewal/npm-2.conf is?


LaRocheSurYonEventsBridge: initial version

A site for a city where I live nearby.
I’m a bit unsure about 2 point:

The dates. There is no "post-time", and the start time of some event is 1970, certainly a placeholder. Thus, post time may not represent the way it is sorted (it isn’t explicit which one is used, but it seems to be roughly chronological)
Should I parse all pages, or only the first one? As of today, thay would make 5 pages to parse (including the first one). That seems reasonable, but I’m still a bit unsure that’s necessary. Will think a bit more about that, likely doing another PR if I actually implement that.

(also, thanks for this software. It was incredibly easy to write this despite my lack of experience with PHP)

since these are events you need to decide how you want to structure the feed.
one option is to include in the feed only those events which are happening with 7 days.

I'd suggest to provide event date as text inside $item['content'], not as item['timestamp'] because item timestamp is supposed to indicate when the entry is published on the website, not the event date itself. Since it's not stated on the site, better leave it empty.

agreed with @ORelio
should probably not populate the timestamp field at all here. Unless you can find the actual publication date of event announcement.

let know if u still here

Sorry, I tend to loose focus too quickly. I'll ptobably return on it sooner or later.
16 oct. 2023 03:00:58 Dag @.***>:

let know if u still here
—
Reply to this email directly, view it on GitHub[https://github.com/RSS-Bridge/rss-bridge/pull/3704#issuecomment-1763571856], or unsubscribe[https://github.com/notifications/unsubscribe-auth/AFMKJZBUJ2YJX6WYS4WIW73X7SBMPAVCNFSM6AAAAAA5FDZTYKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRTGU3TCOBVGY].
You are receiving this because you authored the thread.
[Image de pistage][https://github.com/notifications/beacon/AFMKJZCLSSQ3GF5ZUIXKM5LX7SBMPA5CNFSM6AAAAAA5FDZTYKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTJDX4JA.gif]


Round 2 data is not uploaded

Title.
Round 2 data below:
prices_round_2_day_0.csv
prices_round_2_day_-1.csv
prices_round_2_day_1.csv

As announced on Discord, my backtester won't support round 2 data:


Update MPI & Gromacs for cache hit and better compatibility w/ Rocky8 default image

Updating MPI version to 2021.x which will work better with Rocky 8 default Slurm GCP v6 image. Selected 2021.9 to produce cache hit for Google Spack cache. Also updated Gromacs version for cache hit.
Tested using the water benchmark.
Submission Checklist
Please take the following actions before submitting this pull request.

Fork your PR branch from the Toolkit "develop" branch (not main)
Test all changes with pre-commit in a local branch #
Confirm that "make tests" passes all tests
Add or modify unit tests to cover code changes
Ensure that unit test coverage remains above 80%
Update all applicable documentation
Follow Cloud HPC Toolkit Contribution guidelines #

Waiting for gromacs test to run. I expect to see a speedup in test time. Will post results before merging.


Bump wholesale webapi nuget version

Description
Bumps version on wholesale webapi  nuget.
This changes batchNumber from the databricks runId long type to the batchId Guid

References

#0000

Codecov Report

Merging #1045 (475c72f) into main (83295a4) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1045   +/-   ##
=======================================
  Coverage   69.96%   69.96%
=======================================
  Files          18       18
  Lines         263      263
  Branches       25       25
=======================================
  Hits          184      184
  Misses         69       69
  Partials       10       10

Impacted Files
Coverage Δ

...Hub.WebApi/Controllers/WholesaleBatchController.cs
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Using bubblewrap breaks Darwin compatibility

The bubblewrap package added here in https://github.com/Gerg-L/mnw/commit/8d5af5289aeb8c177ad746ca7d5c835c59b03057 does not support Darwin and therefore mnw no longer builds on Darwin machines. This breaks Darwin support for https://github.com/NotAShelf/nvf upstream, where the flake.lock was updated to include this change in NotAShelf/nvf@37dbdbb (#353).

Ahhhhh, i didn't even think about darwin.
Well, nvf's mnw input should be pinned before that change while I figure out an alternative solution

Well, nvf's mnw input should be pinned before that change while I figure out an alternative solution

You updated it yesterday in nvf (commit linked above) so this change is affecting nvf Darwin users who have updated the nvf input. Thanks for looking into it :)

https://github.com/NotAShelf/nvf/commit/fd90cf7fd5c0457d9ceece133832aac924f48118
Fixed in nvf for now

Great thanks :)

bubble wrap is gone


The case with 768 can pass the accuracy test in the rocm-4.1.0 but fails in the rocm-4.5.2, Do you know why？

What is the expected behavior

What actually happens

How to reproduce

Environment

Hardware
description

GPU
device string

CPU
device string

Software
version

ROCK
v0.0

ROCR
v0.0

HCC
v0.0

Library
v0.0

In case anyone's wondering, this was caused by a compiler bug that was introduced in 4.5.x and fixed in 5.0.


Update link-branding.md

Replaced old promo with implementation service promos
Description of the change:
Reason for the change:
Link to original source:

Closes #

Hello @negarzipan,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Logger container not being created for explainer spec

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
Deployed the AIX explainer and added a logger
apiVersion: "serving.kubeflow.org/v1beta1"
kind: "InferenceService"
metadata:
  name: "aix-explainer"
  namespace: default
spec:
  predictor:
    containers:
    - name: predictor
      image: aipipeline/rf-predictor:0.4.1
      command: ["python", "-m", "rfserver", "--model_name", "aix-explainer"]
      imagePullPolicy: Always
  explainer:
    logger:
      url: http://message-dumper.default/
      mode: all
    aix:
      type: LimeImages
      config:
        num_samples: "100"
        top_labels: "10"
        min_weight: "0.01"

But when I check on the pods I notice the aix-explainer-explainer-default-dv98t-deployment-68d8755f6dlrr9 has 3/3 with no logger deployed
bash-3.2$ kubectl get pods -w
NAME                                                              READY   STATUS    RESTARTS   AGE
aix-explainer-explainer-default-dv98t-deployment-68d8755f6dlrr9   3/3     Running   0          3m13s
aix-explainer-predictor-default-rn6rw-deployment-75bcf8c8fn9ch9   3/3     Running   0          3m12s
message-dumper-j85sv-deployment-6db5fcfd47-mq8jq                  3/3     Running   0          101m

And after querying the aix-explainer, no logs can are in the message dumper
bash-3.2$ kubectl logs $(kubectl get pod -l serving.knative.dev/service=message-dumper -o jsonpath='{.items[0].metadata.name}') user-container
bash-3.2$

What did you expect to happen:
Similar to what we see when the logger is deployed to the predictor
apiVersion: "serving.kubeflow.org/v1beta1"
kind: "InferenceService"
metadata:
  name: "aix-explainer"
  namespace: default
spec:
  predictor:
    logger:
      url: http://message-dumper.default/
      mode: all
    containers:
    - name: predictor
      image: aipipeline/rf-predictor:0.4.1
      command: ["python", "-m", "rfserver", "--model_name", "aix-explainer"]
      imagePullPolicy: Always
  explainer:
    aix:
      type: LimeImages
      config:
        num_samples: "100"
        top_labels: "10"
        min_weight: "0.01"

bash-3.2$ kubectl get pods -w
NAME                                                              READY   STATUS    RESTARTS   AGE
aix-explainer-explainer-default-c9wfk-deployment-8cbbcbdb7qc9d2   3/3     Running   0          4m50s
aix-explainer-predictor-default-psd8g-deployment-db8f584fdzhxgj   4/4     Running   0          4m49s
message-dumper-j85sv-deployment-6db5fcfd47-mq8jq                  3/3     Running   0          123m

And checking the message dumper after querying the predictor results in receiving the cloudevents created from the requests
bash-3.2$ kubectl logs $(kubectl get pod -l serving.knative.dev/service=message-dumper -o jsonpath='{.items[0].metadata.name}') user-container
☁  cloudevents.Event
Validation: valid
Context Attributes,
  specversion: 1.0
  type: org.kubeflow.serving.inference.request
  source: http://localhost:9081/
  id: 2062b33a-74ed-4b36-b512-9975ff4fe99f
  time: 2021-02-23T23:44:18.143788921Z
  datacontenttype: application/json
Extensions,
  endpoint:
  inferenceservicenam
...

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Istio Version:
Knative Version:
KFServing Version:
Kubeflow version:
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm]
Minikube version:
Kubernetes version: (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"19", GitVersion:"v1.19.0", GitCommit:"e19964183377d0ec2052d1f1fa930c4d7575bd50", GitTreeState:"clean", BuildDate:"2020-08-26T14:30:33Z", GoVersion:"go1.15", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"17", GitVersion:"v1.17.17+IKS", GitCommit:"8f82fcbe37a8086624922025f586f48b302f0f28", GitTreeState:"clean", BuildDate:"2021-01-15T02:17:32Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

OS (e.g. from /etc/os-release): macOS Big Sur 11.2

@drewbutlerbb4 currently explainer is not injected with logger or batcher, we could replicate the logic from predictor if that’s needed
https://github.com/kubeflow/kfserving/blob/master/pkg/controller/v1beta1/inferenceservice/components/predictor.go#L70

Thanks @yuzisun, it would be a nice to have feature and if anyone has time to pick it up that'd be great, but this isn't a high priority for us and the only reason I reported this issue was because it seemed like a bug.

Thanks @yuzisun, it would be a nice to have feature and if anyone has time to pick it up that'd be great, but this isn't a high priority for us and the only reason I reported this issue was because it seemed like a bug.

Agreed! Thanks for reporting !


Remove migration logic after migration is done

Remove the migration logic since it has been verified, need to update the config in close source as well.

Codecov Report
Patch coverage: 75.75% and project coverage change: -5.20 :warning:

Comparison is base (8183b3e) 72.75% compared to head (3654cf7) 67.55%.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2493      +/-   ##
============================================
- Coverage     72.75%   67.55%   -5.20%
+ Complexity    11097    10340     -757
============================================
  Files           787      787
  Lines         62745    62654      -91
  Branches       7674     7653      -21
============================================
- Hits          45649    42329    -3320
- Misses        14587    17880    +3293
+ Partials       2509     2445      -64

Impacted Files
Coverage Δ

.../ambry/account/CompositeAccountServiceFactory.java
0.00% <0.00%> (ø)

...thub/ambry/account/HelixAccountServiceFactory.java
29.16% <ø> (-58.34%)
:arrow_down:

...ithub/ambry/account/JsonAccountServiceFactory.java
0.00% <0.00%> (ø)

.../github/ambry/account/mysql/MySqlAccountStore.java
0.00% <0.00%> (-93.75%)
:arrow_down:

.../ambry/account/mysql/MySqlAccountStoreFactory.java
7.14% <ø> (-64.29%)
:arrow_down:

...github/ambry/config/MySqlAccountServiceConfig.java
100.00% <ø> (ø)

...ain/java/com/github/ambry/account/AccountTool.java
0.00% <0.00%> (ø)

...in/java/com/github/ambry/account/DatabaseTest.java
0.00% <0.00%> (ø)

.../com/github/ambry/account/MySqlAccountsDBTool.java
0.00% <0.00%> (ø)

.../com/github/ambry/account/MySqlAccountService.java
40.29% <22.22%> (+4.35%)
:arrow_up:

... and 4 more

... and 156 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


0.0.4 version of gem has broken things

It appears that the 0.0.4 version of the gem has broken the integration with the blinky plugin as well as the watch plugin. I have not tested the stopplicht or growl plugins.

I just uninstalled and reinstalled it, and it's working fine for me. 0.0.4.  I'm staring at a green blinky light now.
What specifically are you experiencing?

The blinky light always turns green at the end of the tests whether my tests passed or not.

Which tests are you running?  Is it a something for your company, or is it something that’s public?
When I did my own tests, I ran them on this public project:  https://github.com/crmckenzie/richmond
—
Sent from Mailbox
On Fri, Mar 27, 2015 at 12:52 PM, Ross Fuhrman<EMAIL_ADDRESS>wrote:

The blinky light always turns green at the end of the tests whether my tests passed or not.
Reply to this email directly or view it on GitHub:
https://github.com/rossfuhrman/starting_blocks-rspec/issues/5#issuecomment-87032648

Hmmm. I had tried it on two of my projects which aren't public and got the same result where it was always green.
However, it is behaving as expected when running the tests for richmond. I will dig a bit deeper to see what the difference is.

Fixed the issue in this commit:
https://github.com/rossfuhrman/starting_blocks-rspec/commit/21870dc094bf4d0ed979f1f7f8516971c2c44d70
Handle singular and plural forms of test results.
In the rspec results, both example and failure are correctly pluralized.
This was causing a problem with the parsing of the results as it was
expecting example and failure to always be plural.

This fix is in the 0.0.5 version of the gem.


Client certificate for downstream call

Hello,
We have a scenario when some services behind the gateway require certificate-based authentication. Is it possible to add client certificate to the downstream call somehow?

@AlexKuriatnyk you could do it with delegating handlers docs http://ocelot.readthedocs.io/en/latest/features/delegatinghandlers.html
Or maybe just have a piece of middleware before the Ocelot pipeline.
Let me know if that helps.

Looks like it is not possible to do this just with delegating handlers. I need to setup client certificate on HttpClientHandler that is used for HttpClient.
Is it possible to add some setting for this or just provide possibility to customize HttpClient creation logic? The only workaround I see for now is to replace entire HttpClientBuilder and HttpClientHttpRequester with DI.

@AlexKuriatnyk this should be possible to implement in Ocelot. How do you set the certificate on the HttpClientHandler? If should just be a case of getting the certificate into Ocelot somehow and then passing it through to the HttpClientBuilder to HttpClientHandler.
How does the certificate get passed to the downstream service? Is this like part of the TCP/IP handshake or as a header or something? If it is a header then we might have other options.

I set it in this way: httpclientHandler.ClientCertificates.Add(certificate);. I believe, it is passed as part of the TCP/IP. I my case, I read it by Subject from certificate storage on machine.
It will be great if it could be possible, for example, to read it on startup and inject into Ocelot with some named key, so then we can use this key in JSON config to indicate that we need to use a certificate for some downstream call.

@AlexKuriatnyk ok cool I will take a look at it ASAP!

Hello! I have a similar scenario. How I can deal with it?

We tried to adjust HttpClientBuilder class and add client certificate from the initial request to the request to the target URL
var handler = CreateHandler(context);
handler.ClientCertificates.Add(context.HttpContext.Connection.ClientCertificate);

However, our API is still not getting the client certificate. The issue might be with the fact that the client certificate from context.HttpContext.Connection.ClientCertificate does not have private key (https://serverfault.com/questions/709812/client-certificate-authentication-with-no-access-to-private-keys).
Certificate on our API side is read in the following way
var clientCertificate = await Request.HttpContext.Connection.GetClientCertificateAsync();

Any thought how client certificate can be convey to the target API?

temporary workaround:
    class X509Hanlder: DelegatingHandler
    {
        private CertificateRepository _certificateRepository;
        public X509Hanlder(CertificateRepository certificateRepository)
        {
            _certificateRepository = certificateRepository;
        }
        protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, System.Threading.CancellationToken cancellationToken)
        {

            var inner = this.InnerHandler;
            while(inner is DelegatingHandler) {
                inner = ((DelegatingHandler)inner).InnerHandler;
            }
            // inner is HttpClientHandler
            if(inner is HttpClientHandler)
            {
                var httpClientHandler = (HttpClientHandler) inner;
                if(httpClientHandler.ClientCertificateOptions != ClientCertificateOption.Automatic)
                {
                    await AddCerificates(httpClientHandler, request);
                }

            }

            var clientHandler = new HttpClientHandler();
            return await base.SendAsync(request, cancellationToken);
        }

        private async Task AddCerificates(HttpClientHandler httpClientHandler, HttpRequestMessage request)
        {
            X509CertificateCollection ceritificates = await this._certificateRepository.GetAllAsync();
            httpClientHandler.ClientCertificates.AddRange(ceritificates);
            httpClientHandler.ClientCertificateOptions = ClientCertificateOption.Automatic;
        }
    }

Hi,
I am trying to get downstream certificates to work. I simply want to add a certificate to the request received by ocelot and have the downstream service verify the certificate.
I have registered a DelegatingHandler with almost the same code as presented by @g00fy. I see the call to the SendAsync method adding the certificate to httpClientHandler.ClientCertificates. The downstream service is called without issue, but the certificate is not present. i.e. HttpContext.Connection.ClientCertificate returns a null value.
I have created a certificate for secure.local and added imported it into Cert:\LocalMachine\Root
The ocelot api gateway is accessible on: https://secure.local:12000
The downstream service on: https://secure.local:12010
Calls to https://secure.local:12000/U/V1/Testing is setup to reroute to https://secure.local:12010/V1/Testing. This is working, only as mentioned, the certificate added to the request is not being received by the downstream service.
Is there something I am missing?
powershell - used to create self-cert
        [string]$certificateName = "*.secure.local"

        Write-Host "`tProcessing the certificate $($certificateName)..."

	    (Get-ChildItem -path Cert:\LocalMachine\My -Recurse | Where { $_.Subject -eq "CN=$($certificateName)" -and $_.NotAfter -lt  (Get-Date).AddDays(14) }) | Remove-Item
	    (Get-ChildItem -path Cert:\LocalMachine\Root -Recurse | Where { $_.Subject -eq "CN=$($certificateName)" -and $_.NotAfter -lt  (Get-Date).AddDays(14) }) | Remove-Item

        $certificate = (Get-ChildItem -path Cert:\LocalMachine\My -Recurse | Where { $_.Subject -eq "CN=$($certificateName)" })
        $trustedCertificate = (Get-ChildItem -path Cert:\LocalMachine\Root -Recurse | Where { $_.Subject -eq "CN=$($certificateName)" })

        if ($certificate -eq $null)
        {
            Write-Host "`tCreating trusted certificate for $($certificateName) within cert:\LocalMachine\My"
            $certificate = New-SelfSignedCertificate -Subject $($certificateName) -DnsName $($certificateName) -CertStoreLocation cert:\LocalMachine\My -KeyusageProperty All -KeyUsage CertSign, CRLSign, DigitalSignature -KeyExportPolicy Exportable

            Write-Host "`tExporting private/public key trusted certificate for $($certificateName) within cert:\LocalMachine\Root"
            $certificatePwd = ConvertTo-SecureString -String "aPpaJP/0Pup/JuT2QQgqYw==" -Force -AsPlainText
            $certificate | Export-PfxCertificate -FilePath "$($this.RootDir)\Dependencies\trustbox.developer.pfx" -Password $certificatePwd
        }

        if ($trustedCertificate -eq $null)
        {
            Write-Host "`tCreating trusted certificate for $($certificateName) within cert:\LocalMachine\Root"

            $exportedCerFile = Export-Certificate -Cert $certificate -FilePath "$($this.RootDir)\Dependencies\secure.local.cer"

            $exportedCerFile | Import-Certificate -CertStoreLocation Cert:\LocalMachine\Root

            #I then create the PFX by calling Export-PfxCertificate
        }

ocelot.json
"ReRoutes": [
        {
            "DownstreamPathTemplate": "/{everything}",
            "DownstreamScheme": "https",
            "DownstreamHostAndPorts": [
                {
                    "Host": "localhost",
                    "Port": 12010
                }
            ],
            "UpstreamPathTemplate": "/U/{everything}",
            "UpstreamHttpMethod": [ "Options", "Get", "Post", "Put", "Delete" ],
            "DangerousAcceptAnyServerCertificateValidator": true,
            },
            "DelegatingHandlers": [
                "X509CertificateDelegatingHandler"
            ]
        }

Delegating Handler
    public class X509CertificateDelegatingHandler : DelegatingHandler
    {
        private readonly X509CertificateCollection _certificates = new X509CertificateCollection();

        public X509CertificateDelegatingHandler()
        {
            // testing...
            byte[] rawData = GetFileAsBytes("secure.local.pfx");
            _certificates.Add(new X509Certificate2(rawData, "password-used-to-create-pfx"));
        }

        protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, System.Threading.CancellationToken cancellationToken)
        {
            var inner = InnerHandler;
            while (inner is DelegatingHandler)
            {
                inner = ((DelegatingHandler)inner).InnerHandler;
            }
            // inner is HttpClientHandler
            if (inner is HttpClientHandler httpClientHandler)
            {
                if (httpClientHandler.ClientCertificateOptions != ClientCertificateOption.Automatic)
                {
                    httpClientHandler.ClientCertificates.AddRange(_certificates);
                    httpClientHandler.ClientCertificateOptions = ClientCertificateOption.Automatic;
                }

            }

            return await base.SendAsync(request, cancellationToken);
        }
        private static byte[] GetFileAsBytes(string filePath)
        {
            if (!File.Exists(filePath))
                filePath = Path.Combine(AppDomain.CurrentDomain.BaseDirectory, filePath);
            if (!File.Exists(filePath))
                throw new ApplicationException("Path missing: " + filePath);

            using (FileStream f = new FileStream(filePath, FileMode.Open, FileAccess.Read))
            {
                int size = (int)f.Length;
                byte[] data = new byte[size];
                size = f.Read(data, 0, size);
                f.Close();
                return data;
            }
        }
    }

Kind regards,
Rik

@AlexKuriatnyk ok cool I will take a look at it ASAP!

Any update on this issue?

Has there been any status update on this issue?  I'm in a similar situation where I want to forward the certificate but no approach so far has worked

@AlexKuriatnyk commented on May 12, 2018

Hello Alex!
Are you still with Ocelot?

@bkardisco commented on Jan 22

Let us know your user scenario please!

@g00fy- commented on Jul 2, 2019
@forfront commented on Apr 5, 2020

Hi guys!
Your draft solutions are based on Delegating Handlers. So, they should be reused in real feature development...
Why not to create a PR?
Do you have an intention to contribute in 2023?...

@k-u-s commented on Jan 19, 2019

Your feature branch doesn't exist!
Moreover, the Ocelot fork doesn't exist!
Can I remove your message from this thread?

Hello,
Has there been any update regarding this.
In my case I want to validate the downstream certificate in the ocelot API gateway, so I need to add a custom certificate validator, can anyone provide a solution for this.
Thanks.

@nikhillkumar

SSL Errors

@AlexKuriatnyk commented on May 12, 2018

We are patient but still wait for your contributions.
Because you are Software Engineer at Microsoft, we believe you have enough expertise to deliver this feature.
Sure we will help you and support during development.
@ggnaegi @RaynaldM  FYI

@raman-m, setting DangerousAcceptAnyServerCertificateValidator is something which will cause all certificates to be trusted and hence bypassed, whereas I want to do introduce a custome callback which will do validation based on my custom set of rules, something like ServerCertificateCustomValidationCallback.
Is there something of this sort which I could use ?

@nikhillkumar
You can search for some ASP.NET docs regarding client certificates on Microsoft Learn portal.
Ocelot doesn't support its own server SSL certificates to send to downstream endpoints. But regular Asp.net app can be configured for this. How to create certificates, how to manage them, how yo attach to a route? A lot of questions...
This is absolutely new feature!
I will convert this issue soon to discussion thread, because issue isn't ready for development. And nobody wants to contribute.

@nikhillkumar @raman-m you still have an option, it's adding your own logic here:
https://github.com/ThreeMammals/Ocelot/blob/develop/src/Ocelot/Requester/MessageInvokerPool.cs#L85
You could even go a step further and use DI.

@nikhillkumar

did you mean that you are gonna help with creating a PR ?

Sorry, which PR are talking about?
Will you work on #357 user scenario (new feature request)?
Or do you want that we will help you to write code for your current tasks of your current employer? )) You want to much, my dear!
Anyway, we are not going to rewrite our generic validator which covers all validation scenarios of all self-signed certificates because it simply returns true without any analysis! We will not change this embedded validator because it covers everything including even your scenario!
https://github.com/ThreeMammals/Ocelot/blob/d54836d51f6ea29a2013967e3649a23e19db28b3/src/Ocelot/Requester/MessageInvokerPool.cs#L87
We will not accept and approve any PRs with changes of this line!
@ggnaegi  FYI !

@raman-m Yes, I just gave a solution to @nikhillkumar

@ggnaegi

the feature should be implemented imo.

By whom? By Mr. Kumar?

@ggnaegi

the feature should be implemented imo.

By whom? By Mr. Kumar?

I think we should first convert this to a discussion and talk about the approach we would like to take. It's an important feature, that is not implemented "out of the box" in other projects (for the known reverse proxies, you will need to compile your own library too),


Pinball not working

Game reboots to the title after loading a first level, and unplayable and tried it FCEUX on my PC?......

This was reported almost 30 days ago on the nesdev forums. The problem has been listed on the compatibility sheet ever since. Please read the compatibility sheet before reporting bugs.
At the time this was reported, I was already looking into the reset bug the day prior. The fix is done and will be available when v1.4 is released.


Assign types from is_a series of functions

The is_a (is_string, is_array, etc.) series of functions guarantee,
when used in an if block, that the type of the variable matches the type
checked for. Add a simplistic check that works for the simple usage of
checking only the type. Only works for simple function call, not
expressions that include the is_a check.

What a patch. Thanks @ebernhardson!


Release 0.2.0 version

Since all the tests pass 🎉and the code went through some internal testing (thanks Dominik!) it's time to mark (release) it as a stable version and start a new SNAPSHOT version.

Tag created --> https://github.com/jaceklaskowski/splice-machine-spark-connector/releases/tag/v0.2.0

Fixed with https://github.com/jaceklaskowski/splice-machine-spark-connector/commit/ea17533067234191440595f84c460e2f57e1d3f7


Request support for SQLEXPRESS for db-type = mssql

I have a (MS SQL Server) SQLEXPRESS instance that I use for development purposes.
I can't connect to it with pynonymizer.
I was able to connect using the pyodbc package with appropriate credentials and server=(local)\SQLEXPRESS.
I would like to be able to connect to a SQLEXPRESS instance.
It could be an option to convey that I want to connect to a SQLEXPRESS instance. maybe
--mssql-named-instance = SQLEXPRESS
while I'm not that familiar with the details of MS SQL Server, they do have "named instances" that are not SQLEXPRESS.
The alternative approach I took to see if I could "make it work" was to wrap the initial connection attempt (in mssql/init.py, in __db_connection ) with a try/except and in the except, replace the server string with "(local)\SQLEXPRESS".  That worked.
And that would certainly be good enough for me (it adds a bit of a delay, but in the scheme of things, that's probably not a big deal).
I also tried
server=f"{self.db_host}\\SQLEXPRESS,{self.db_port}",
and
server=f"{self.db_host}\\SQLEXPRESS",
but those didn't work.
I did this on Windows 11 with SQL Server 16.0.1000.

Is this something that could be solved with access to the connection string directly? There is a feature request at #135 that might solve this.
I'm hoping we can kill two birds with one stone here as this is something that I think we can implement. Can you take a look and see if it meets your needs?

Is this something that could be solved with access to the connection string directly? There is a feature request at #135 that might solve this.
I'm hoping we can kill two birds with one stone here as this is something that I think we can implement. Can you take a look and see if it meets your needs?

I think so.
I added the following to an except block after the initial connection code:
                print("Failed initial attempt at SQL Server connection, retrying with SQLExpress")
                server_string = "(local)\\SQLEXPRESS"
                print(server_string)
                self.__db_conn = pyodbc.connect(
                    driver=f"{{{self.__driver}}}",
                    database=self.db_name,
                    server=server_string,
                    uid=self.db_user,
                    pwd=self.db_pass,
                    autocommit=True,
                )

so if "everything" that goes in to the connect function call is extracted from the connection string (or is the connection string), and at least the server key/value pair gets set with that, I'm perfectly happy passing in a complete connection string (with a named instance).

hi, this change is in the v2 milestone and will be released this week:
You can now pass "(local)\SQLEXPRESS" to the --db-host option and it should override other options. you can also customize the connection stribg using --mssql-connection-string


AttributeError: 'Select' object has no attribute '_simple_int_clause'

Trying to use dataset via Athena Database. Using the latest docker image. I am getting below error and 500 on UI page.
How to reproduce the bug

Go to Datasets
Click on +Datasets
Select athena as database and select appropriate schema and table.
Click on newly created dataset. you should see error.

Expected results
Should be able to create chart from newaly added dataset (athena)
Actual results
500 Internal server error.
'Select' object has no attribute '_simple_int_clause' Traceback (most recent call last):   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1516, in full_dispatch_request     rv = self.dispatch_request()   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1502, in dispatch_request     return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)   File "/usr/local/lib/python3.8/site-packages/flask_appbuilder/security/decorators.py", line 133, in wraps     return f(self, *args, **kwargs)   File "/app/superset/utils/log.py", line 245, in wrapper     value = f(*args, **kwargs)   File "/app/superset/views/core.py", line 888, in explore     datasource_data = datasource.data if datasource else dummy_datasource_data   File "/app/superset/connectors/sqla/models.py", line 871, in data     data_ = super().data   File "/app/superset/connectors/base/models.py", line 293, in data     "select_star": self.select_star,   File "/app/superset/connectors/sqla/models.py", line 860, in select_star     return self.database.select_star(   File "/app/superset/models/core.py", line 485, in select_star     return self.db_engine_spec.select_star(   File "/app/superset/db_engine_specs/base.py", line 1129, in select_star     sql = database.compile_sqla_query(qry)   File "/app/superset/models/core.py", line 465, in compile_sqla_query     sql = str(qry.compile(engine, compile_kwargs={"literal_binds": True}))   File "<string>", line 1, in <lambda>   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 481, in compile     return self._compiler(dialect, bind=bind, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 487, in _compiler     return dialect.statement_compiler(dialect, self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 592, in __init__     Compiled.__init__(self, dialect, statement, **kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 322, in __init__     self.string = self.process(self.statement, **compile_kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 352, in process     return obj._compiler_dispatch(self, **kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/visitors.py", line 96, in _compiler_dispatch     return meth(self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 2201, in visit_select     text = self._compose_select_body(   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 2320, in _compose_select_body     text += self.limit_clause(select, **kwargs)   File "/usr/local/lib/python3.8/site-packages/pyathena/sqlalchemy_athena.py", line 90, in limit_clause     if limit_clause is not None and select._simple_int_clause(limit_clause): AttributeError: 'Select' object has no attribute '_simple_int_clause' 2022-05-24 06:46:36,378:ERROR:superset.views.base:'Select' object has no attribute '_simple_int_clause' Traceback (most recent call last):   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1516, in full_dispatch_request     rv = self.dispatch_request()   File "/usr/local/lib/python3.8/site-packages/flask/app.py", line 1502, in dispatch_request     return self.ensure_sync(self.view_functions[rule.endpoint])(**req.view_args)   File "/usr/local/lib/python3.8/site-packages/flask_appbuilder/security/decorators.py", line 133, in wraps     return f(self, *args, **kwargs)   File "/app/superset/utils/log.py", line 245, in wrapper     value = f(*args, **kwargs)   File "/app/superset/views/core.py", line 888, in explore     datasource_data = datasource.data if datasource else dummy_datasource_data   File "/app/superset/connectors/sqla/models.py", line 871, in data     data_ = super().data   File "/app/superset/connectors/base/models.py", line 293, in data     "select_star": self.select_star,   File "/app/superset/connectors/sqla/models.py", line 860, in select_star     return self.database.select_star(   File "/app/superset/models/core.py", line 485, in select_star     return self.db_engine_spec.select_star(   File "/app/superset/db_engine_specs/base.py", line 1129, in select_star     sql = database.compile_sqla_query(qry)   File "/app/superset/models/core.py", line 465, in compile_sqla_query     sql = str(qry.compile(engine, compile_kwargs={"literal_binds": True}))   File "<string>", line 1, in <lambda>   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 481, in compile     return self._compiler(dialect, bind=bind, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/elements.py", line 487, in _compiler     return dialect.statement_compiler(dialect, self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 592, in __init__     Compiled.__init__(self, dialect, statement, **kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 322, in __init__     self.string = self.process(self.statement, **compile_kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 352, in process     return obj._compiler_dispatch(self, **kwargs)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/visitors.py", line 96, in _compiler_dispatch     return meth(self, **kw)   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 2201, in visit_select     text = self._compose_select_body(   File "/usr/local/lib/python3.8/site-packages/sqlalchemy/sql/compiler.py", line 2320, in _compose_select_body     text += self.limit_clause(select, **kwargs)   File "/usr/local/lib/python3.8/site-packages/pyathena/sqlalchemy_athena.py", line 90, in limit_clause     if limit_clause is not None and select._simple_int_clause(limit_clause): AttributeError: 'Select' object has no attribute '_simple_int_clause'
Screenshots

Environment

browser type and version: Chrome
superset version: superset version : Docker Image digest : 1d0a0c2923e5
python version: python --version  : Python 3.8.12
node.js version: node -v : NA
any feature flags active: NA

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Browser Endpoint : superset/explore/table/154/
Let me know if additional information is required from metadata db.
Code Snipet from SQLAlchemy:

def limit_clause(self, select, **kw):
text = []
offset_clause = select._offset_clause
if offset_clause is not None and select._simple_int_clause(offset_clause):
text.append(
f" OFFSET {self.process(offset_clause.render_literal_execute(), **kw)}"
)
limit_clause = select._limit_clause
if limit_clause is not None and select._simple_int_clause(limit_clause):
text.append(
f" LIMIT {self.process(limit_clause.render_literal_execute(), **kw)}"
)
return "\n".join(text)

Yes, This worked Thank you @czammar.

Hi @azadsagar I'm facing the same error with Superset .
I've tried adding
"PyAthena[SQLAlchemy]>=1.0.0, <2.0.0"

in my helm chart values for config bootstrapScript but I got some errors about version mismatch with PyAthena (2.9.2). Then I changed it to
"SQLAlchemy>=1.0.0, <2.0.0"

Per the documentation https://pypi.org/project/pyathena/#sqlalchemy... the package was installed successfully but still getting the error with helm chart 0.6.1.
Did you have to do something else to make it work?
Thanks in advance!


Use Persian digits

It would be great to have an option to use Persian digits like: ۱۳۹۸/۰۷/۰۹
jalaliMoment.local('fa',{usePersianDigit: true})

Thanks!

The main goal of this repository is to facilitate converting any library using moment.js to be compatible with jalali calendar system, if we generate persian digits we have to parse them too, and it would be hard to keep it update.
For this you can use the following package https://www.npmjs.com/package/persian-number
or use persian fonts also you can create your own fork then we will add a link to your repository.


Add eslint 7 to CI

The relevant breaking changes in eslint 7 are:

Test cases that contains unrecognized properties are now failed.
Test cases that have fixed code but missing output are now failed.
The method sourceCode.getComments is deprecated.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

Coverage remained the same at 97.797% when pulling 5c67f17aa87e836d0007c8cc064f447903155e03 on golopot:eslint-7 into 71ca88f0a1e7e1270f1c1f9633d3ae8f136f58e1 on benmosher:master.

@ljharb Any idea when this one will be released? 🤔

@benmosher @ljharb Any idea when this one will be released? 🤔

Ping @benmosher @ljharb

@MichaelDeBoey it will be released, like every version, when I have time to do it. There's a number of outstanding PRs i'm hoping to merge first.

Let me know if and how I could help getting them merged/fixed 🙂


:robot: Triggering CI on branch 'release-next' after synching to openshift/master

/test 4.6-e2e

/test 4.6-e2e

OCF Webhook is merging this PR

OCF Webhook is merging this PR


python3Packages.reolink: 0.53 -> 0.56

Motivation for this change
Update to latest upstream release 0.56
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 150008 run on x86_64-linux 1

2 packages built:

python38Packages.reolink
python39Packages.reolink


Update API documentation

to include register_subscriber_on_close()

I'll get this one -- you can check if I got it right :-)


TypeError: socialMedia is undefined

This is happening when I run yarn dev
To Reproduce
Steps to reproduce the behavior:

yarn dev

Screenshots

OS: [Windows]
Browser [firefox]

Additional context
Here's the full console output:

Please refer https://github.com/harshgoel05/another-portfolio/issues/104#issuecomment-984719901

I realized afterwards that that's what you need to do. Sorry 😶

No worries, glad that its working. Closing the  issue.


Iterate through batch generator in benchmarks

Description of proposed changes
In this PR the benchmarks iterate through the batch generator for a more complete evaluation of performance now that batch generation is lazy. Further improvements could include separate metrics for creating the batch selectors versus indexing on batches, but this is a step forward.

Fixes #

Codecov Report

Merging #140 (9fd005c) into main (13c69fe) will decrease coverage by 2.32%.
The diff coverage is 91.74%.

:exclamation: Current head 9fd005c differs from pull request most recent head 55b5c77. Consider uploading reports for the commit 55b5c77 to get more accurate results

@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
- Coverage   98.37%   96.04%   -2.33%
==========================================
  Files           6        6
  Lines         246      329      +83
  Branches       49       70      +21
==========================================
+ Hits          242      316      +74
- Misses          4        8       +4
- Partials        0        5       +5

Impacted Files
Coverage Δ

xbatcher/testing.py
100.00% <ø> (ø)

xbatcher/generators.py
94.67% <91.74%> (-5.33%)
:arrow_down:

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


Upgrade OpenSSL to 1.0.2k

OpenSSL hasn't been upgraded in a while, and according to https://www.openssl.org/news/vulnerabilities.html 1.0.2g suffers from 25 vulnerabilities at the moment, two of which labeled high severity.
This PR should help improve the security of the three macOS users out there who don't use Homebrew.

Thanks for taking care 👍

@yyuu You're welcome. By the way 3.6.1rc1 (just merged) needs to go through this replacement too.


Dev guide update

Update dev guide for the tabbing between books and creating new books function

Codecov Report
Base: 57.30% // Head: 57.30% // No change to project coverage :thumbsup:

Coverage data is based on head (5d57e8c) compared to base (65b86e2).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master      #71   +/-   ##
=========================================
  Coverage     57.30%   57.30%
  Complexity      476      476
=========================================
  Files            96       96
  Lines          1916     1916
  Branches        200      200
=========================================
  Hits           1098     1098
  Misses          772      772
  Partials         46       46

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Inconsistency in API design.

Shouldn't this
var isArray = require('callbaxx').isArray;

be more like
require('callbaxx').isArray(isArray => { /* ... */ });

?

Hi Nathan, I'm not sure I follow. Could you please elaborate?

Ah I understand now. You make some good points. How does the API prevent this usage though? Could you provide an example of how lib/add.js should look?
As for require, we could simply redefine the require function. I think that would be within the scope of this project since #7 is also on our roadmap.

This works:
global.__require = require.bind();

global.require = require = function require(...args) {
  if (args.length === 1) return __require(args[0]);
  return args[1](null, __require(args[0]));
};

require('callbaxx', function (err, callbaxx) {
  callbaxx.map([5, 15, 25, 35], callbaxx.add, function (e, r) {
    console.log(r);
  }, true, 5);
});

Beautiful.


Power Automate Create a custom connector changes

Hello, I tried to replicate your solution but in Power Automate, I can't find the "Import an OpenAPI from URL" option

What is the new way to do this?. Thank you

Looks like you might be using a Preview version of Power Automate (or you're in a region that's received the latest train).  In my test org in NAM, I don't see the same screen you do.
Either way, you can access the Custom Connectors under the "Data" tab on the left.  From there you can import the OpenAPI file.


Error occurs when updating vim header

I have encountered an error when updating header for my .vimrc/vimrc file.
Executing :AddHeader successfully adds header lines if there is no existing header; please see an example below:
" File: .vimrc
" Author: Firstname Lastname<EMAIL_ADDRESS>" Date: d.m.Y
" Last Modified: d.m.Y

But when executing :AddHeader to update the header, it fails but delete the "File" and "Last Modified" lines, where updates are supposed to occur; please see an example below:

" Author: Firstname Lastname<EMAIL_ADDRESS>" Date: d.m.Y

I suspect that this error is caused due to the fact that Vim's comment char is(") when calling update_header() function.
I am not good at Vimscript so I don't know how to fix this.
I hope you can replicate this error.

Hi @shoichiaizawa,
Thanks for the report. I committed a quick fix for this. You were right, it's because of vim's comment char "

@alpertuna That was a quick fix! Thanks for the update :D


Key Binds API

A built-from-scratch API that covers Fabric Key Binding API's functionality and more
The PR was planned in two stages: Blue and Pink. The implemented feature set of Blue was:

Key binds now support being dynamically disabled, making them completely invisible to the game's eyes
The Key Binds screen now shows a tooltip on conflicting keys showing the other key binds that use the same key, and adds a (!) indicator to the key text itself for accessibility reasons
The API exposes some methods meant to help with mod intercompatibility

Currently, work is being done on the Pink feature set:

[ ] A custom config file at ./config/qsl/key_binds.json allowing for our own key bind extensions (WIP)
[ ] Key binds now accepts key chords, expanding the possibilities for modded key binds (WIP)
[ ] Key binds will contain key predicates, which essentially points all the use cases where it will be used. This will allow multiple key binds that never overlaps to share the same key

Now is the time for any last-minute bikesheds on where to put the keybindings API, since I know people are unhappy with gui

gui isn't perfect, but its the closest large group in my opinion.
you could make an input or client group, but then we end up with 30 different modules

A custom config file at ./config/qsl/key_binds.json allowing for our own key bind extensions (WIP)

Now that I'm thinking of it, probably should be ./config/quilt/key_binds.json.

A custom config file at ./config/qsl/key_binds.json allowing for our own key bind extensions (WIP)

Now that I'm thinking of it, probably should be ./config/quilt/key_binds.json.

I thought about it (after all, FAPI used ./config/fabric/), however, I don't know; I'd think ./config/quilt/ would instead be used by something like the Quilt Loader, and not QSL


fix poolboy example in README.md

Hey @zhyu, thanks a lot for conrtibuting! Great job spotting those errors, they often go unnoticed :) thanks!