Allow to configure slack token

Currently, there's no way to configure slack token as described in this SO question.

I want to reevaluate this soon.

I believe the only issue here is that the portal doesn't easily allow us to set a key. I'll test this and create a Portal UX issue to track. Keeping this open just in case there are additional issues that pop up.

Steps to test slack token

Create Generic WebHook (I used Node) and have the sample just log data
via scm site, go to ./data/Functions/secrets/{function}.json
replace that token with the Slack provided token
Trigger WebHook and watch the log

So there are two issues before we can light up proper support for Slack in Functions.

Parsing www-form-urlencoded bodies into objects - https://github.com/Azure/azure-webjobs-sdk-script/issues/317
Portal UX allowing for 2 way key management - https://github.com/projectkudu/AzureFunctionsPortal/issues/360

Once those two have lit up, we can add it back to the bindings.json - https://github.com/Azure/azure-webjobs-sdk-templates/blob/dev/Bindings/bindings.json
So that said, this is going to become a feature planning issue, and I'm going to triage it to next considering that is where its dependencies are triaged.

I'd like to include this in "should-have" since I hope it's relatively low hanging fruit which results in big user impact. It's basically a bug of an existing feature.

Maybe we need a way to tag such 'should have' in GitHub issues?

I have one rc:should-have. I just thought I should discuss it first. :)

Will be addressed by this: https://github.com/projectkudu/AzureFunctionsPortal/issues/360

This is done now! :)


feat: impl LongRefFromWasmAbi for T where T: Tsify + FromWasmAbi

Implement #34
I tried to make a test of the behavior by building a test very similar to #33 using async function for validate and it fails 😧...
which is expected because Rust must be responsible for dropping the reference to the closure here so you cannot invoke the closure twice and we have no way to check it in Rust according to rustwasm/wasm-bindgen#3178 so this suggest my implementation is right actually 😃  ...
But it also fails if I use the same implementation as RefFromWasmAbi for the LongRefFromWasmAbi 😧, which normally does not drop the reference at the end of the first Rust invocation and so allow the second invocation to be processed.
In summary, I am not sure how to test that my implementation is the correct one 😕
I leave this PR here for people who are maybe more knowledgeable building a test for this. My main issue is that we cannot take an async closure as function argument to reproduce a test like in aaaa3982d2e0a5959acd162d10cb0ef6105ea9fa where validation is async

It looks like the expansion tests are the only ones that are failing? Or did you remove a different failing test?

Yeah I forgot to change expansion results in test, I am on it !

All good for the CI now
I did not include the test using an async function for now because I am not really sure about what it is really testing 😅 (so tests did not change at all, I just completed expansion results)
If my implementation is bad, the worst case should be that we leak memory in the ABI heap because Rust may not clean the imported JsValue when dropping it (obviously, that's not what I have done), this may happen if I copy-paste the implementation of RefFromWasmAbi in the implementation of LongRefFromWasmAbi but I am not able to make a test that show this kind of mistake clearly.
Leaking memory is safe but ... yeah we should avoid it so it would be nice to test that a correct implementation avoid it 😒


[BEAM-8893] Fix issues with state and multiple workers.

Also avoid worker pool when there is no actual parallelism.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @chamikaramj


PMID:18346214

normal plasma membrane sterol distribution
actin cortical patches present in decreased numbers
inviable swollen pear-shaped vegetative cell

normal plasma membrane sterol distribution FYPO:0004963
actin cortical patches present in decreased numbers FYPO:0004964
inviable swollen pear-shaped vegetative cell FYPO:0004965

edit file: 8ad92e08174157cfad7138045500f7652ec97855
release: f3fa940c3e2b1d7c71bc1e4dc7d51baff3b8160d


support for Azure.authenticate(File propertiesFile)

supports loading credentials via Java's standard Properties format, using the following settings:
subscription=########-####-####-####-############
tenant=########-####-####-####-############
client=########-####-####-####-############
key=####
managementURI=https://***
baseURL=https://****
authURL=https://****

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

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

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

TTYL, AZPRBOT;

Can one of the admins verify this patch?


Inconsistent strconv.Atoi behavior for 32bit release, lead to error or data corruption.

strconv.Atoi has different behavior:
$ GOARCH=386 go run test-int.go
2147483647 (0x80bee40,0x8c8e000)

$ GOARCH=amd64 go run test-int.go
4294967295 (0x0,0x0)

$ cat test-int.go
package main

import "strconv"

func main() {
	println(strconv.Atoi("4294967295"))
}

For example, in XORM:

A user has reported that Gitea 1.17 failed with error:
...
2022/06/28 11:47:32 routers/common/db.go:34:InitDBEngine() [I] Backing off for 3 seconds
2022/06/28 11:47:35 routers/common/db.go:27:InitDBEngine() [I] ORM engine initialization attempt #9/10...
2022/06/28 11:47:35 routers/common/db.go:33:InitDBEngine() [E] ORM engine initialization attempt #9/10 failed. Error: sync database struct error: strconv.Atoi: parsing "4294967295": value out of range
2022/06/28 11:47:35 routers/common/db.go:34:InitDBEngine() [I] Backing off for 3 seconds
2022/06/28 11:47:38 routers/common/db.go:27:InitDBEngine() [I] ORM engine initialization attempt #10/10...
2022/06/28 11:47:38 routers/init.go:68:mustInitCtx() [F] code.gitea.io/gitea/routers/common.InitDBEngine(ctx) failed: sync database struct error: strconv.Atoi: parsing "4294967295": value out of range

I'm having the same issue using gitea-1.17-linux-arm-6 on a Raspberry Pi 4 Model B (32-bit ARMv7)

func Atoi(s string) (int, error) {

on 32bit systems int is most likely to be int32 rather than int64.
The problem is that xorm *schemas.Column uses signed int for column length and should either: a) use int64 or b) uint or even uint64 - however this is a breaking change in xorm.

https://gitea.com/xorm/xorm/pulls/2160

If you have met this bug, please download latest 1.17 from https://dl.gitea.io/gitea/main/ , hopefully everything goes well know.


Add validation for Kraus operators

Many of the methods involving noise/channels mention that the matrices making up a mixture or channel must satisfy \sum_{k=0}^{r-1} A_i^\dagger A_i = I. This is never enforced, which allows the user to simulate unphysical systems and get invalid density matrices (see example below).
Validation is easy enough (assert np.sum([m.T @ m for m in object._channel_()) but I'm not sure where to put this @Strilanc ? Ideally this will be called exactly once, before a call to simulate.
Code Example:
from typing import Iterable

import cirq
import numpy as np

class UnnormalizedChannel(cirq.GeneralizedAmplitudeDampingChannel):
    """Overwrite the correct amplitude damping channel with a bad one."""
    def _channel_(self) -> Iterable[np.ndarray]:
        p0 = 1
        sqrt_g = np.sqrt(self._gamma)
        return (
            p0 * np.array([[1., 0.], [0., 1.]]),
            p0 * np.array([[0., sqrt_g], [0., 0.]]),
        )

def unnormalized_channel(gamma):
    return UnnormalizedChannel(1.0, gamma)

q0 = cirq.LineQubit(0)
bad_noise_inst = unnormalized_channel(0.3)
c = cirq.Circuit.from_ops(cirq.X(q0), bad_noise_inst.on(q0))
dm = cirq.DensityMatrixSimulator().simulate(c).final_density_matrix
kraus_sum = np.sum(m.T @ m for m in bad_noise_inst._channel_())
print(np.allclose(kraus_sum, np.eye(2)))
print(np.isclose(np.trace(dm), 1.0))

which prints
>>> False
>>> False

One first suggestion is that we should add a testing method that checks this.
My opinion is just as we don't enforce unitary returns a unitary (because checking it each time would be costly), I think we just need to make it easier for people to test that their channel is OK.

We have routines in cirq.linalg.predicates that check for a variety of potential qualities. I would suggest adding some additional methods like is_cp and is_cptp there for the users to apply on their constructions themselves. We could potentially add a flag in simulate and run which when true performs quantum computing validity checks on the circuit but I'm not convinced whether that would add much benefit if we had a more complete list checks for individual operations in linalg.

+1 for using predicate and I like the is_cp, is_cptp checks.

@peterse are you still planning to work on this?

oops, sorry for taking so long. I'm struggling to find time to contribute so I'm going to unassign myself for now

Keeping this before-1.0 as new validation will tighten the API. cc @95-martin-orion


build: initialize const vector explicitly

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>Description:
Fixes #6584
Risk Level: Low
Testing: CI
Docs Changes:
Release Notes:

@mattklein123 I'm not pretty sure why clang-8 is throwing error on this (and seems only on Fedora), but it doesn't hurt to have this change.


Feature - CLI Tutor Mode

Adds the CLI Tutor Mode, which can help with learning the IPFS Command Line Interface from the Webui/Desktop. This feature was requested in #1421
Done

Option to enable/disable CLI Tutor Mode from the settings page
If enabled, a copy-icon 📋 is shown next to the buttons/menus where those actions can be performed via IPFS' CLI
This icon is visible at:

Files Page -> inside + Import menu (Add File, Folder, Copy from IFPS)
Files Page -> File Menu (three dots next to a file) -> (Delete, Rename, Pin/Unpin, Download)
Peers Page -> next to + Add Connection
Settings Page -> next to Reset button in config edit box
Setting Page -> next to Submit button in API edit box

Pending Work:

I recently discovered a state management issue with the modals, I couldn't find the root cause so far, it started happening after the refactor today (1st Aug 2020)
After rebasing from master today, there's a small UI glitch (text in the modal actions is left aligned, I'll work it out before making this PR reviewable)
I have added a bunch of TODO's in the code, which can potentially help with the code cleanup
Files component kind of looks like a mess right now, a lot of state is being shared across mulitple components (maybe we can refactor it to make it look neat)

Here's a visual for how it looks in action: [Jessica note: these are a bit outdated now since commits for visual tweaks on 12 Aug]
Settings Page | Option to enable CLI Tutor Mode

Files Page | CLI Tutor Mode Enabled

Settings Page | CLI Tutor Mode - Open Modal

Signed-off-by: Jasdeep Singh<EMAIL_ADDRESS>

@jessicaschilling here's the draft for cli tutor mode, the place which would need more attention is files component, since it has a lot of shared state, please check this out and suggest any refactors/changes for places where i might have broken any acceptable coding patterns. I'll update this PR tomorrow with little of code cleanup and modal actions alignment fix

Thanks, @jay-dee7! I'd like @rafaelramalho19 to look through this in more detail ... but just a heads up that he's on holiday for the next week.  In-fact apologies for the delay. This should've been up way before than it did. In the meantime I'll keep  on iterating  over this, just to see if anything can be improved upon 😄

sure @jessicaschilling i'll make these changes ASAP. There are more things that I didn't notice initially. ContextMenu in files pages needs some work, after rebasing, the copy icon and the text aren't aligned symmetrically. I'll try to get them fixed before the review starts

@jessicaschilling I rebased from master just a while ago, one of the incoming changes were in ApiAddressForm component and since it was written using hooks (I believe), I was not able to overwrite props easily (I'm not sure what exactly broke the prop sharing, i believe its hooks but i have no idea.). So just for the ApiAddressForm component, I've managed the state for cli-tutor-modal locally to the component, instead of using the regular redux actions. Rest of the stuff stays almost same. I think this PR is ready for review 🤞

Thanks, @jay-dee7!
@rafaelramalho19 - are you able to take a look at this PR once you are back in and settled? Thank you!

After you remove the transforms, LGTM 😄

thanks for the amazing review @rafaelramalho19, i've change the css class to use just height and no transforms :)

@lidel -- do you mind going over this when you're able? I think we're very close to this being done.
NOTE: I want to get the "copy to clipboard" icon into ipfs-css before we ship. I can do that while you're reviewing 😊
Thanks!

hey @jessicaschilling I just saw the changes that you made and I think they really addch the finishing touches to the feature, left aligned text in the modal looks great plus the new icon actually looks pretty amazing :) However, I did notice once thing though, the Checkbox on CLI Tutor Mode is slightly moved to bottom lh-copy class is doing the most of it. Can we do some sort of workaround for that?
I was going through the ipfs documentation and realised that a couple of commands are not correct I think, namely

Files Page -> + Import Button -> New Folder
Files Page -> individual file -> three dots -> Rename
I'll push the change for both of these commands just now but please let me know if rest of them are correct too

@jay-dee7 @lidel I'd vote for closing the dialog onclick of the "Copy" button. @jay-dee7, can you please change that?

Sure @jessicaschilling I'll make the change ASAP :)

@jay-dee7 And thanks for spotting the misaligned checkbox! Fixed it. 😊

@jessicaschilling @lidel i've added both the requested points.
Now by clicking the copy button in the modal, we copy the contents (command) and then close the modal.
Also, I've added a fix for rename command. I realized that we need to preserve the file path while renaming and we should wrap the full path path with quotes (""), because it'll throw errors if the full path contains spaces (for example a file named cat image.png would throw error in cli but if we wrap it with quotes as "cat image.png") it won't. Please let me know if we're good to go with this :)

@lidel - I'm happy but will wait for your OK, please feel free to merge 😊

@lidel -- Thanks for calling the subdirectory detail out. I'm inclined to merge this as-is and address that in a separate issue (https://github.com/ipfs-shipyard/ipfs-webui/issues/1601), since I suspect we may want to make some tweaks to tutor mode overall after this gets into the hands of users for a bit.
I've also opened https://github.com/ipfs-shipyard/ipfs-webui/issues/1600 for consolidating repeat-use i18n keys, which I can probably take care of in the next week or two.
Thanks all (especially @jay-dee7!) for all of the work!


line 26 compares itself causing the condition never to fire

In line 26, we see that there is a loop that never gets fired. Needs to compare with the last scan or the cut off value...

Thanks for pointing this out! I have modified the script!

Awesome... Lightning fast reply...


Setting up artifactory-oss Master key problem

Hi,
I am pretty new to docker. So I wanted to try to use artifactory with docker. after I start up artifactory-oss-postgress.yml the image starts. But in the Logs it shows me this error
Failed resolving master key: failed resolving 'shared.security.masterKey' key; file does not exist: /opt/jfrog/artifactory/var/etc/security/master.key
and when I try to use the prepareHostEnvironment.sh I am getting this error.
Are you sure you want to remove existing /data [y/n]: y Removing old /data Creating /data First setup. Setting FIRST_SETUP=true Copying needed files to directories Artifactory configuration files cp: cannot stat './../../files/access': No such file or directory
And because of that I can not open artifactory via my browser. Can someone maybe help me out with this problem? My System is Ubuntu

Hi @Haso58 . This repository is deprecated and not maintained anymore.
We have now an official Docker compose distribution you can download and use from the official Artifactory OSS download page.


[sable/signalsciences] Update SignalSciences agent to 3.13.0

Signed-off-by: Marc Campbell<EMAIL_ADDRESS>What this PR does / why we need it:

Update the SignalSciences agent version to 3.13.0.
Adds an OWNERS file.

Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @mattfarina

@mattfarina bumping this so it's doesn't get stale. What can we do to get this released?

@mattfarina this is just a patch release for the agent. Do you have any suggestions on how we can get this marked as ok to test?

@mattfarina is there anything needed from me to get this patch validated and merged?

@mattfarina I'm hoping to get this version bump merged. Can you assist?


How to See the Data in an App ?  & Simple Wiring to ESP8266

I have the Code running OK on The ESP8266 D1 Mini in the VINDRIKTNING.
The Serial Port is streaming measurements to my IDE Serial Monitor.
What do I have to  use to see the data on an App ? (or a browser)
The Config.h is untouched, I dont know what to enter !
It has somehow managed to connect to my WiFi despite not giving it an SSID or Password.
I installed Home Assistant on my Android Tablet, it asks for a URL, I dont have one
The Blog does say this code  works with auto discovery, but no joy.
This must be a pretty basic issue, but I cant find any tutorial related to it.

Hmm, no takers.
Its a shame that for the sake of a basic tutorial on setting this up that such a complex program is likely to be abandoned.
Looking at the other Issues, I am not the only person at a loss to getting this finished.
I have done numerous other ESP projects using Blynk and Thingspeak, so I will have a go at using the  data in the serial feed to  feed Thingspeak - has anyone else done this ?
I want to use a simple manual configuration of ssid/password for the wifi, so first task is to clear out all the code around this.
I added a Piezo buzzer between port D6 and +3.3V, very useful to indicate the WiFi has connected or whatever you want.
I attach a few pics of a simple way of wiring this up using normal 1/10th  inch pitch headers, tweaked so they surface mount and then supported with epoxy resin (not hot melt). I used double sided tape to fix the board (with a 3mm spacer) with the socket available for external  connection, easy to reflash with it all screwed together.

After  remming out about half the code and adding in Thingspeak, its working.

Probably worth doing some reading on MQTT....
Homeassistant, nodered....

I accept that, I did look at the mqtt site and the "quick start" seemed anything but !
I have used Thingspeak for a few projects and it works very well, I prefer it to Blynk.
The code in this project is well beyond my capability, I am not a natural programmer.


Discussion: avoid periodical logs

          > BTW that line is unpin snapshot with RPC. It's normal for an idle cluster to only have this log.

From my perspective, in production we should never include logs that are triggered automatically and periodically, unless they're user-facing. Also applies to the DEBUG logs in hummock event handler.
Originally posted by @BugenZhao in https://github.com/risingwavelabs/risingwave/issues/12610#issuecomment-1752311968
These seems verbose even for developers.

+1. Previously I opened such an issue but nobody cares 🥹
https://github.com/risingwavelabs/risingwave/issues/7872


Full Uninstall Option [no trace files]

The Issue
No Complete Uninstall Option
Steps to reproduce
Option 1

Select the Uninstall.exe
Leaves Practically Every File if not Run As Admin
Manually go through and delete and then use windows uninstall option to get rid of hidden junk
still leaves trace files after all this

Option 2

go to windows uninstall in control panel
select uninstall lbry
leaves files in program folders and the %appdata%

Expected behaviour
the uninstall should have 2 options

first way keep it like you have it
second way have it delete every last bit of LBRY stuff as if the user wouldn't want it on that PC any longer or just doesn't want to use the app any further but also have pop up warning to backup default_wallet to ensure funds are not lost and if they don't already have it backed up and want to an option to select a new directory and copy the default_wallet over to it

Actual behavior
uninstall.exe doesn't really uninstall anything that would be deemed as an uninstall, even using the windows feature to uninstall leaves scrap files around that wouldn't be need if the Expected Behavior statements didn't play a role
Once a user installs you have to manually delete to get rid of everything LBRY it seems a bit intrusive

It was unexpected to find a wallet backed up after removing it for the development build from source. Should I bother using the backup I made?

Yes, the lbry data and wallet are not deleted during Uninstall, only the program related files. This is to ensure that you still have your lbry data and credits in case you reinstall lbry or bring it to another pc.
Migration guide here:
Https://lbry.io/faq/backup-data

@tzarebczan are we not planning on implementing this feature?


Doorbell ring action

Hi, thanks for the plugin.
Can you pls explain what the options mean. Also, is it possible to include when the doorbell rings ?
Thanks

ttl is the polling interval (in seconds)

verboseP determines whether the pluging should log verbosely

in general, you shouldn't need to change either of these options (and since they are the defaults, you can leave them out of your config.json file).
the plugin will report ringing to homekit, but only if it happens to catch that during a poll. the ring app has a better way of getting notifications than polling, but that api isn't available to the plugin...

Thanks for that. I assume they also haven't made the Live View api available.

cf., https://github.com/homespun/homebridge-platform-ring-video-doorbell/issues/6#issuecomment-284774326 ... there's a SIP URL, but i'm looking for a volunteer to do the work...


crypto/x509/pkix: FillFromRDNSequence does not preserve multi-value RDNs

What version of Go are you using (go version)?
This issue affects Go 1.8 and later as a consequence of https://github.com/golang/go/commit/809a1de1ac1ccc76f7a4faf630017626f2f68231 to fix  #16836 (and #12342).
Prior to Go 1.8 and that change, Go did not process multi-value RDNs -- it always assumed just one attribute and value (the first) per RDN.
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
The OS and architecture should not matter.
What did you do?
https://play.golang.org/p/7OIuhautCe
What did you expect to see?
The multi-value nature of any RDN is preserved.  That is, the length of the subject (and issuer) RDNSequence before pkix.FillFromRDNSequence and after pkix.ToRDNSequence should match.
What did you see instead?
The multi-value nature of any RDN is not preserved by pkix.FillFromRDNSequence.  That is, the length of the subject (and issuer) RDNSequence before pkix.FillFromRDNSequence and after pkix.ToRDNSequence do not match.
As previously noted in comments on https://github.com/golang/go/commit/809a1de1ac1ccc76f7a4faf630017626f2f68231, the original Gerrit CR, and #12342, if this will not be fixed, then the behavior should be documented at a minimum.

CC @FiloSottile

Clearly missed for Go 1.11.

This is bad, also because it will lead to a parse-serialize cycle returning a semantically different result, but I can't see how to fix this without making a whole new set of APIs, which is not an option for such rare certificates.
Unless anyone has any better idea, we should just document this behavior, maybe in FillFromRDNSequence.


Tournament on own implementation(s)

Is there an easy way to automatically pit a set of implementations against each other and evaluate them, other than manually changing the implementation of EvilBot each time?

I'm afraid there's no built-in way to do this, you'd need to modify the framework yourself.


Closed

Well nothing.Sorry for bother.

hey if you want a move + navi joiner I can help with that https://www.ebay.com/itm/302828605984


Add lower case codec string for IMSC1 Text profile (#2378)

Fixes #2378

register TTML handler for stpp.ttml.im1t codec
adjusted HLS test case to comply with rfc8216bis section <IP_ADDRESS>
This change keeps the legacy support for codec string stpp.TTML.im1t. Note that while unit tests pass without this, there is good reason to assume that there is still plenty content out there using the legacy content string.

To test:

HLS: https://demo.unified-streaming.com/video/tears-of-steel/master_hls_fmp4_clear.m3u8
DASH: https://demo.unified-streaming.com/video/tears-of-steel/tears-of-steel-ttml.ism/.mpd

@googlebot Yup, I signed it

@porcelijn LGTM, let me run your PR through our build bot

Looking good, thanks for the PR!


[Social] Twitter: unable to render Card preview

Twitter is unable to render card preview.
Open https://cards-dev.twitter.com/validator and enter an article url:
INFO:  Page fetched successfully
INFO:  17 metatags were found
ERROR: No card found (Card error)

Twitter is unable to render card preview.
Open https://cards-dev.twitter.com/validator and enter an article url:
INFO:  Page fetched successfully
INFO:  17 metatags were found
ERROR: No card found (Card error)

I'm having this exact same issue and my metatags are fine. They're the same as my other working sites. I'm trying in a NextJS app. #104 isn't helping me.

Twitter is unable to render card preview.
Open https://cards-dev.twitter.com/validator and enter an article url:
INFO:  Page fetched successfully
INFO:  17 metatags were found
ERROR: No card found (Card error)

I'm having this exact same issue and my metatags are fine. They're the same as my other working sites. I'm trying in a NextJS app. #104 isn't helping me.

you should use 'open graph protocol' for having twitter card and on any social media


feat(.github): add docker-build-push.yml

Build op-node/op-batcher/op-proposer and push images to  ghcr.io/

Replaced by https://github.com/node-real/optimism/pull/14


OKE and Mysql always created in okit deployment compartment

Create OKE and Mysql and deploy to OCI, both are stored in the root compartment and not in the one specified in the designer.

Fixed 0.44.0 Release


Default to Java 11 instead of 8

For Confluent Enterprise Support customers, we would strongly advise you to open a Support ticket which will be addressed within your Support contract SLA at https://support.confluent.io
Describe the issue
The default java version is 8, but I think there are significant performance benefits to using java 11, especially with ssl/tls enabled.

@chuck-confluent we have had this on our internal roadmap for a while, however there are a lot of caveats with running anything other then Java 8 currently. For example, the HDFS connector doesn't work with newer Java versions due to limits in some of the Hadoop libs.
Additionally the project is designed to keep with our current recommended best practices, and as per the documentation, we still recommend Java 8: https://docs.confluent.io/platform/current/installation/versions-interoperability.html
We will actually need the platform itself to start recommending Java 12 as the default before we can move down this path.

Additionally the project is designed to keep with our current recommended best practices, and as per the documentation, we still recommend Java 8: https://docs.confluent.io/platform/current/installation/versions-interoperability.html

@JumaX I do not read the documentation as Java 8 is still recommended. Could you elaborate, please? All the Confluent images are now based on UBI 8 with Azul Zulu Java 11. So I would support changing the default JDK to Java 11 in cp-ansible, as both Java 8 and Java 11 is supported in general. When using the HDFS connector, you would have to take care. Are there any other components not supporting Java 11?

@erikgb You are completely right!  I mis-read that matrix! The formatting is not the easiest to follow.
@chuck-confluent I take my previous response back, we should be able to default to Java 11 then.  Apologies, misread the matrix.

Java 11 will be the default in our next major release!


Propagate INSTANCE_PROFILE_6_SLICE constants from NVML

NVML added GPU_INSTANCE_PROFILE_6_SLICE and COMPUTE_INSTANCE_PROFILE_6_SLICE constants, but these weren't propagated through mig-parted, meaning that 6-slice MIG profiles were not supported. This PR adds those constants to internal/nvml/consts.go and exposes them in pkg/types/mig_profile.go.

Hi @vineel-cruise. This is a read-only mirror of our development repository, so please open a merge request against https://gitlab.com/nvidia/cloud-native/mig-parted instead.
Thanks.

Hi @vineel-cruise As Evan said, this is just a mirror and all changes should be opened as MRs against: https://gitlab.com/nvidia/cloud-native/mig-parted

Since this is such a small change and obviously correct, I've gone ahead created the MR myself (with your authorship on the commit retained): https://gitlab.com/nvidia/cloud-native/mig-parted/-/merge_requests/78
Will approve / merge once CI finishes.

Thanks @elezar and @klueska for the help!


↩ [release/1.9] DCOS_OSS-981: Move ports auto assign checkbox at the top level

Detailed explanation: https://jira.mesosphere.com/browse/DCOS_OSS-981
Turned out we were wrong assuming that ports auto assign works per endpoint, in fact it works on the app level. This PR moves the checkbox from the endpoint form to the network form and changes the way this checkbox affects port field in a portDefinition. When auto assign is checked JSONReducer replaces any value for all the ports with 0, when the checkbox is not checked it emits previous value or 0.
We might want to create a custom validator that will force a user to enter a port in case the auto assign feature is off.

/cc @timcharper

Assigning a specific port is not working 😢

2 tests are still failing

Hold merge because it causes a regression for other network types. I'd assume we should emit requirePorts only for HOST network. Though I'm not sure 😩

network != HOST issue should be addressed now.

Hold merge once again to address:

Host networking: all ports are auto-assigned, or none of them are.
Bridge networking: host ports can individually be specified as auto-assigned, or not
Container networking: host ports can individually be specified as auto-assigned, or not

@orlandohohmeier @Poltergeist I addressed all the issues and rewrote it 🙇

@tobilg @leemunroe can you please have a look and check if the wording is ok?

Great Job! ⚒ 💯


Behavior of inbound listeners during graceful drain

I have found some conflicting answers online, so I am hoping for an authoritative answer.
When Envoy is drained via  POST /drain_listeners?inboundonly&graceful:

Are inbound listeners closed immediately, or will new inbound connections be accepted until the graceful drain period ends?
Does it make a difference whether the number of inbound connections reaches zero before the graceful drain period ends?

My understanding is that inbound connections will continue to be accepted until the drain period ends no matter what, but I would like to confirm.
https://www.envoyproxy.io/docs/envoy/latest/operations/admin#operations-admin-interface-drain

Draining stops new connections to be accepted. Graceful drain waits for existing connections to be completed until the drain timeout expires, after which they are forcibly closed.

@yanavlasov Sorry, I'm still not quite clear on the matter. With a graceful drain, are listeners closed at the start of the drain period, or at the end?

New connections stop to be accepted at the beginning of the drain.

@yanavlasov If that's true, then the Envoy docs for POST /drain_listeners?graceful need to be fixed.

When draining listeners, enter a graceful drain period prior to closing listeners.

@yanavlasov Is that true that envoy stops accepting new connections at drain start? If yes as suggested by @rittneje we may have to update the docs?

@mattklein123 I thought drain?graceful does not stop new connections from accepting and that is the difference between that and normal drain (without graceful) - Looks like it is not case based on above^^?

From looking at the existing code I think the docs are correct:
https://github.com/envoyproxy/envoy/blob/fe5d87c44114f8ca64c594b39bd7921d8ac67023/source/server/admin/listeners_handler.cc#L24-L31
Whether this is the right thing to do or not I'm not quite sure. I don't recall the intent of this feature as without some external mechanism to stop incoming connections it seems not that useful.

@mattklein123 In the case of Kubernetes, the envoy drain is happening in parallel to the load balancer updating, so eventually new connections will stop coming. Hence the documented behavior is desirable in such use cases.

I'm not sure why this is closed since it affects a lot of people and causing errors on production systems. We need a proper delay before rejecting new connections while k8s updates its networking/routing configuration (unfortunately for all of us in parallel to our apps).
@rittneje Have you found the solution except hacks with PreStop?

@rittneje we experience some race conditions where new connections still go to the Envoy in Kubernetes since the controller to update the endpoint IP is async.

From looking at the existing code I think the docs are correct:
https://github.com/envoyproxy/envoy/blob/fe5d87c44114f8ca64c594b39bd7921d8ac67023/source/server/admin/listeners_handler.cc#L24-L31
Whether this is the right thing to do or not I'm not quite sure. I don't recall the intent of this feature as without some external mechanism to stop incoming connections it seems not that useful.

Just wanted to confirm - does the code suggest that it stops accepting connection at the end of drain?


[bug]: generate entity panic

sea-orm-cli generate entity -o src/entity

thread 'main' panicked at 'called Result::unwrap() on an Err value: Database(MySqlDatabaseError { code: Some("42S22"), number: 1054, message: "Unknown column 'generation_expression' in 'field list'" })', /Users/jacksoom/.cargo/registry/src/github.com-1ecc6299db9ec823/sea-schema-0.2.9/src/mysql/discovery/executor/real.rs:32:14

Hey @jacksoom, can I know the MySQL database version that you are using? I think this issue is specific to a database version.

mysql version: 5.5.62-log
verbose backtrace
[2022-01-13T02:35:37Z INFO  sqlx::query] SELECT column_name, column_type, is_nullable, …; rows: 0, elapsed: 31.262ms
SELECT
  `column_name`,
  `column_type`,
  `is_nullable`,
  `column_key`,
  `column_default`,
  `extra`,
  `generation_expression`,
  `column_comment`
FROM
  `information_schema`.`columns`
WHERE
  `table_schema` = ?
  AND `table_name` = ?
ORDER BY
  `ordinal_position` ASC

thread 'main' panicked at 'called Result::unwrap() on an Err value: Database(MySqlDatabaseError { code: Some("42S22"), number: 1054, message: "Unknown column 'generation_expression' in 'field list'" })', /Users/jacksoom/.cargo/registry/src/github.com-1ecc6299db9ec823/sea-schema-0.2.9/src/mysql/discovery/executor/real.rs:32:14
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
Is it related to the mysql engine?  because there are tables with different engines （myisam and innodb）

Thanks!

So the column GENERATION_EXPRESSION  was added since 5.7 any MySQL version below that won't have the column.
The INFORMATION_SCHEMA COLUMNS Table:

MySQL 5.6: https://dev.mysql.com/doc/refman/5.6/en/information-schema-columns-table.html
MySQL 5.7: https://dev.mysql.com/doc/refman/5.7/en/information-schema-columns-table.html

Thanks!

there is no generation_expression field in my database table information_schema.columns. should i update the mysql version . can you give me some advive for this

I'm not sure is there any breaking changes when upgrading from 5.5.62-log to 5.7.x. But I'll check why we need GENERATION_EXPRESSION in a minute.

Hey @tyt2y3, I want to know why we need the expression: Option<ColumnExpression>? Seems that it's not being used in the writer.
https://github.com/SeaQL/sea-schema/blob/bd742757bd2dc609c822daccd3ab426144a9804b/src/mysql/def/column.rs#L6-L25

same problem. any update?

[help4@dev23 rhuax]$ sea-orm-cli generate entity -u mysql://root:sanysany@<IP_ADDRESS>:3306/huaxrobot -o entities
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: Database(MySqlDatabaseError { code: Some("42S22"), number: 1054, message: "Unknown column 'generation_expression' in 'field list'" })', /home/help4/.cargo/registry/src/github.com-1ecc6299db9ec823/sea-schema-0.10.2/src/mysql/discovery/executor/real.rs:29:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

The expression: Option<ColumnExpression> column is not essential (and not used anyway). But SeaSchema was meant to be a faithful representation, so I tried to include that.
We might be able to fill in None if such column does not exist. I remember such "try" logic exists somewhere with the SeaSchema codebase.

MySQL version based selection. This? I guess.
https://github.com/SeaQL/sea-schema/blob/6b7ab04c891cf4d48418c2bfda689ef06d9fd5e4/src/mysql/query/index.rs#L58-L66

Exactly

Hey everyone, this will be patched on the next sea-orm-cli release.
https://github.com/SeaQL/sea-schema/pull/86

This bug has been patched on sea-schema 0.10.3.
Please re-install sea-orm-cli by executing cargo install sea-orm-cli --force in order to generate entities on MySQL 5.6


Issue/5041 fix documentation build issues iso5

Description
Follow up PR to https://github.com/inmanta/inmanta-core/pull/5239 to merge changes into iso5
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Processing this pull request

Merged into branches iso5 in f2dfbf817c6301629d394b85025186ff1489e6d7


Package pythonPackages.elpy: please reistate

The packages pythonPackages.elpy was dropped without any apparent explanation in commit 4bd7e963b349d4f98e8047509e3fd9b6d4b009cf . As a user of this package, can I ask it to be reinstated?
cc @costrouc,  @jonringer, @FRidh

From my understanding, it was abandoned upstream and replaced with stub setup.py to prevent installations from failing. Does the package actually give you any real value?
It was even removed from pypi at one point.
see jorgenschaefer/elpy#1583

Your're right and I'm sorry for the noise but why couldn't you add your very useful comment to the commit?

You're right, I should have written the intent of removing the pack into the commit message. Next time I will :)


Develop

Codecov Report
Patch coverage: 90.33% and project coverage change: +0.41 :tada:

Comparison is base (1e42821) 90.84% compared to head (8256355) 91.26%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #16      +/-   ##
==========================================
+ Coverage   90.84%   91.26%   +0.41%
==========================================
  Files          23       34      +11
  Lines        1519     1901     +382
==========================================
+ Hits         1380     1735     +355
- Misses         82      108      +26
- Partials       57       58       +1

Impacted Files
Coverage Δ

distributed/queue.go
70.76% <70.76%> (ø)

downloader.go
93.18% <80.00%> (+0.82%)
:arrow_up:

distributed/dupefilter.go
83.33% <83.33%> (ø)

request.go
91.12% <84.44%> (-0.28%)
:arrow_down:

response.go
88.00% <85.29%> (-12.00%)
:arrow_down:

distributed/utils.go
85.71% <85.71%> (ø)

service/service.go
86.59% <86.59%> (ø)

distributed/components.go
88.37% <88.37%> (ø)

distributed/distributed.go
88.38% <88.38%> (ø)

distributed/stats.go
89.47% <89.47%> (ø)

... and 20 more

... and 1 file with indirect coverage changes
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 in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Fix gnome3.libgit2-glib build.

The libgit2-glib build currently fails with:
theba :: ~/nixpkgs ‹master› » nix-build . -A gnome3.libgit2-glib
these derivations will be built:
  /nix/store/p27j6wkyfi8r02h1k1s5anb928s2cjjx-libgit2-glib-0.0.24.drv
building path(s) ‘/nix/store/g07gdf12qcirpb2m7bs8m8917w7x0rb8-libgit2-glib-0.0.24’
unpacking sources
unpacking source archive /nix/store/nnxrwfddfrln6dnzxp1fh6qfx9wz57id-libgit2-glib-0.0.24.tar.xz
source root is libgit2-glib-0.0.24
patching sources
configuring
fixing libtool script ./ltmain.sh
grep: sh: No such file or directory
grep: ./autogen.sh: No such file or directory
grep: sh: No such file or directory
grep: ./autogen.sh: No such file or directory
configure flags: --prefix=/nix/store/g07gdf12qcirpb2m7bs8m8917w7x0rb8-libgit2-glib-0.0.24
sh: ./autogen.sh: No such file or directory
builder for ‘/nix/store/p27j6wkyfi8r02h1k1s5anb928s2cjjx-libgit2-glib-0.0.24.drv’ failed with exit code 127
error: build of ‘/nix/store/p27j6wkyfi8r02h1k1s5anb928s2cjjx-libgit2-glib-0.0.24.drv’ failed

Apparently, the upstream tarball doesn't contain autogen.sh; this PR removes the reference to autogen.sh and uses the default configure instead.

Travis passed, thanks.


Can you call GeoJSON.addData and not render the changes?

I'm trying to reduce the amount of blocking JS that occurs as I load the data for my map. I've moved JSON parsing and TopoJSON to GeoJSON conversion into a Web Worker, and the final thing I want to do gradually add the features to my map. However, each time I call GeoJSON.addData it redraws the map.
Is there a nice way to suspend redrawing? This is the hack I'm using (don't laugh :smiley:) https://github.com/wpf500/Leaflet/commit/e9acf302adbe109ed9d28173145f7da38e466351

https://github.com/tmcw/leaflet-geojson-stream may help.

Yeah, I'm not sure we'd want to support a super-rare case like this in the core. You can do it with monkey patching though — e.g. instead of adding a line of code in the source, just override a method, calling the original one inside with .apply(this, arguments).


Add className & template decorators

I was going to start a repo and I saw yours!
This add className & template props. I use the template prop as a string, not sure that match everyone use cases.

Hi @posabsolute sorry for the slow response.  There's already a template decorator.  But I'd be happy to add the className decorator if you update the PR to remove the extra template decorator.

@posabsolute any updates on this?  I'm going to close unless you want to push this forward


Conflict to Revitlookup

Hi, I am new to the revit development, shout-out to your template. When I am building the MultiProjectSolution sample. It works perfectly. However, the revit lookup pops up the following errors:

May I know how to resolve it, thank you!
Revit version tested out:
2020,2021,2022,2023,2024

do you have the latest Lookup version installed? Please install the latest and it will be okay https://github.com/jeremytammik/RevitLookup/releases

do you have the latest Lookup version installed? Please install the latest and it will be okay https://github.com/jeremytammik/RevitLookup/releases

yes, it is solved. Thank you!


fix(messaging)!: Fix mix up messaging in window and custom event messengers

fix #57
Firstly
It may be hard to review, so take your time.
Also, if approved, it might be safer to make it a major bump. Because the messaging path is narrowed and pseudo-channeled, it is possible that a user's application, through which the message previously happened to pass, could be broken.
Changes
message
First, has a unique instance ID. This is to check whether the message came from its own instance or not. Then, namespace, instanceId, and message type are checked strictly in the req/res listeners.
messaging-demo

Add CustomEventMessage demo
Add multiple bi-directional messaging demo

About vitest browser mode
I was annoyed by the flakey test in page.test, so I partially switched to browser mode. I understand that the window(jsdom global context) on nodeVM is common between tests, but can be made isolated when in browser mode.
I really wanted to split the test environment nicely with workspace configs like in this example, but I gave up because of a bug that broke the existing unit test. The bug that cjs mocking (webextension-polyfill) fails when using browser mode together. They seem to have reworked the whole mock mechanism.
This Issue (#48) may have been resolved. But the link is already dead, I don't know which test Flakey was testing at the time.
Diagrams

mermaid live editor link

Here is the understanding I have in my head and its diagram.
sequenceDiagram
    box Content-Script

instance1
{ namespace: foo }
    participant ins1_on as onMessage
    participant ins1_send as sendMessage
    end
    box Injected/WebSite

instance2
{ namespace: foo }
    participant ins2_on as onMessage
    participant ins2_send as sendMessage
    end
    box AnotherContext

instance3
{ namespace: bar }
    participant ins3_on as onMessage
    participant ins3_send as sendMessage
    end

    %% namespace B
    Note over ins3_on,ins3_send: COMMUNICATION FLOW
SAME AS LEFT

    par
        ins1_send->>+ins2_on: send/type:"ping1"
        ins1_send--xins1_on: ignore own send
        ins2_on--xins2_send: ignore own res
        ins2_on->>-ins1_send: res/type:"ping1"
    end
    par
        ins2_send->>+ins1_on: send/type:"ping2"
        ins2_send--xins2_on: ignore own send
        ins1_on--xins1_send: ignore own res
        ins1_on->>-ins2_send: res/type:"ping2"
    end

    %% no communicate lines
    Note over ins2_on,ins3_send: No communicate in
different namespaces

@aklinker1 PR is ready for review. But there's really no need hurry. Because at least in my project, I'm resuming work on this content with a local patching. 😃

Waiting for merge....

No problem, thanks for the hard review!

Released v2.0.0

@aklinker1 It seems that the published 2.0.0 contents are still in 1.4.0. There may be a problem with the build process 🤔 .
https://www.npmjs.com/package/@webext-core/messaging?activeTab=code

@1natsu172 fixed, https://github.com/aklinker1/webext-core/releases/tag/messaging-v2.0.1

Thanks, confirmed that the correct update came 🚀


CTRL_R doesn't pick up DEFAULT_OPTS

Category

[x] fzf binary
[x] Key bindings

OS

[x] Mac OS X

Shell

[x] zsh

I have --reverse in my FZF_DEFAULT_OPTS but CTRL_R doesn't respect this anymore in the latest release. CTRL_T picks it up though

The code: https://github.com/junegunn/fzf/blob/0.16.1/shell/key-bindings.bash#L59
I decided to override --reverse in $FZF_DEFAULT_OPTS by setting --no-reverse after it as it seemed more fitting for the use case. But as you can see, you can re-override the config using $FZF_CTRL_R_OPTS.

I decided to override --reverse in $FZF_DEFAULT_OPTS by setting --no-reverse after it as it seemed more fitting for the use case.

But it just makes $FZF_DEFAULT_OPTS more confusing. Because people (or at least myself) will assume that $FZF_DEFAULT_OPTS will affect every fzf action. But applying it to some commands only is not really consistent IMO.

I can see that it can be controversial or confusing, but let me explain. Defaults are just defaults and it should be possible to override some of the options. For example, imagine that you have --multi in $FZF_DEFAULT_OPTS, should we still allow multiple selections in CTRL-R? It will break the function processing the output which expects just a single line from fzf. Likewise, one can have --preview 'cat {}' in $FZF_DEFAULT_OPTS, but fzf may be not filtering files but a list of processes, host names, etc.

But applying it to some commands only is not really consistent IMO.

To be clear, $FZF_DEFAULT_OPTS is applied to every fzf process, it's that we can override a subset of the options by explicitly stating it as the arguments to fzf.
export FZF_DEFAULT_OPTS='--height 50% --reverse'

# Override --height, but --reverse is still applied
fzf --height 30%

Anyway, back to the subject, the case of --reverse option is not equivalent to the examples I gave above, in that it does not affect the correctness, but only affects the presentation style, which should be probably, primarily decided by the user?

To be clear, $FZF_DEFAULT_OPTS is applied to every fzf process, it's that we can override a subset of the options by explicitly stating it as the arguments to fzf.
export FZF_DEFAULT_OPTS='--height 50% --reverse'

# Override --height, but --reverse is still applied
fzf --height 30%

I understand, but this doesn't work with $FZF_CTRL_R_OPTS for example, which breaks the predictability of the settings. For something like --multi I understand that it can break some commands, maybe docs to indicate which flags are allowed in $FZF_DEFAULT_OPTS?

Anyway, back to the subject, the case of --reverse option is not equivalent to the examples I gave above, in that it does not affect the correctness, but only affects the presentation style, which should be probably, primarily decided by the user?

Totally agree, it's just that I expect that when I override it in $FZF_DEFAULT_OPTS that it will work as expected. For example my preference is to use --reverse as a default but now I need to add it in multiple places. Which feels a bit odd IMO.
My main issue is already fixed & I understand why, so I'll close this.
Just to be clear, I'm not complaining or anything. I do appreciate your work & thanks for the great tools 😊

I understand, but this doesn't work with $FZF_CTRL_R_OPTS for example, which breaks the predictability of the settings.

Agreed, we should carefully order the options so that it makes most sense. To summarize:

$FZF_DEFAULT_OPTS

Options that are context-free (regardless of the type of the entities).
Mostly options that determine presentation style.

$FZF_[BINDING_NAME]_OPTS

Per-binding overrides. e.g. Have --reverse in $FZF_DEFAULT_OPTS, and --no-reverse only in $FZF_CTRL_R_OPTS

Options that can affect the correctness should come last

And now I agree that it is more straightforward that we don't override --reverse in CTRL-R.

Updated, thanks for the comments.


Generic symbolic dimensions

The default symbols used to represent the different spatial and temporal dimensions work fine for our current examples and test cases, but restrict users that may want to implement other, more complicated equations or time-stepping schemes. A more dynamic and customisable approach will facilitate further implementations and should allow us to significantly reduce the complexity of the Propagator. This, however, requires careful design which can be discussed in this issue.
My current thinking is based around the idea of introducing some type of Dimension class that, similar to our SymbolicData objects, associates SymPy dimension symbols (x, y, z, t) with meta-data about dimension sizes and loop iteration spaces. Such a model might work something like this:

A Propagator maintains a list of Dimensions, which it can query to generate the appropriate C loop code, reducing the complexity of the Propagator code.
A Dimension might yield multiple IterationSpace objects that represent sub-loops for loop blocking. These can be created according to various blocking schemes and across multiple dimensions.
Individual stencil equations coming into the Operator can be scanned for Dimension symbols, which allows us to capture free and bound variables and will allow us to automatically insert the stencil at the right depth in the loop nest. This will help with sanity checks for operator merging and inserting sparse operations, ie. source and receiver terms, more gracefully.
Additional SymPy expressions might be used to define custom stepping schemes for individual iteration spaces, which will allow user to implement custom time-stepping algorithms.

As stated above, this issue is intended for discussing and drafting a possible implementation, so comments and suggestions are highly welcome.

Should a Dimension object represent a loop on top of the ones defined by the Iteration objects?
If yes, I was thinking of a structure like this:

self.iterations list of Iteration objects for the sub loops
self.stencils list of stencils with the main code for the Dimension loop
self.pre_loop_stencils list of stencils to be executed before the dimension loop
self.post_loop_stencils list of stencils to be executed after the dimension loop

No, a dimension in this case is a purely symbolic object that represents an iteration space, ie. x or y or t in our current examples. Thus it behaves like a symbol that we we use to index our symbolic data / function objects, but it knows about the bounds in a certain spatial dimension for example, including things like padding layers. As such it would be able to generate Iteration objects that represent actual loops in the code over a certain iteration space.
The core idea I am pursuing with this is that if our function indices are smart enough, we can simply pull all the free indices out of a stencil expression and generate the loop objects that each expression needs to encapsulated in. Once we know that and we get the data dependencies between variables, we can match loops/iteration spaces and thus auto-generate the exact loop nest require, without the need to pre-define loops like pre_loop_stencils or post_loop_stencils.

self.stencils list of stencils with the main code for the Dimension loop
It still would be good to be able to have possibility to split the stencils in different dimension loop with intermediate Pre/post loops

Ok, so I believe PR #156 brings us a good step closer to doing this in a much nicer way. After that all that is missing, at least as far as dimension handling is concerned, is the ability to switch between loops with open or pre-defined "baked in" limits at will (preferably by setting limits on the dimension objects).

The new 3.0 API allows generic switching between pre-defined and open loop bounds, so I think we got all of this now in. So closing...


Fail to get HTML source inside popup step - limitation with attaching to popup target

I'm trying to get the HTML source of a page inside popup steps, but it is not successful. For example, I have two files "without_popup.tag" and "with_popup.tag" with content as follows:
without_popup.tag:
http://tebel.org/gist/about_tebel
read page to html
echo `html`

with_popup.tag:
http://tebel.org
click ABOUT
wait

popup about_tebel
 read page to html
 echo `html`

It turns out that

tagui without_popup.tag -h can display the HTML source, but
tagui with_popup.tag -h shows nothing.

Not sure is there any workaround to get the HTML source inside popup? Thanks for help in advance.

Hi @queyintik, the issue is due to complication in JSON parsing for retrieving the HTML source in the popup tab. The contents will be parsed and then stringified on the websocket level before being consumed by TagUI, in order to access data on a separate tab through attaching and detaching from target tab. But this leads to the following error.
jSON Parse error: Expected '}'

Can you tell me more about your use case? A workaround is get the URL of the popup window, and access it directly. Here's another issue with example on visiting URL dynamically (https://github.com/kelaberetiv/TagUI/issues/936) and a related issue on creating a macro visit (https://github.com/kelaberetiv/TagUI/issues/960).
If you are able to access the popup page directly, then read page to html would work.

Got it. Thanks for the explanation.


missing link (due to wrong Institution ID?)

I am trying to attribute this record (https://www.gbif.org/occurrence/2434489393) to this collector (https://bionomia.net/Q61390). I verified that this is correct attribution. However, the occurrence record does not show up in Bionomia. Is it because "Institution ID" cannot be matched? Is Bionomia looking for a different code? Thanks for looking into this.

Hi @sharifX. institutionID is not a term included in the Bionomia download from GBIF, whereas institutionCode is. There was no particular reason for this other than the presentation of such values in aggregate on eg https://bionomia.net/Q61390/deposited-at seemed like a tricky thing to do for human consumption because sometimes these appear to be URLs and other times not. I see that the institutionID term was used for this record with a value, "Naturalis Biodiversity Center" rather than perhaps a more appropriate value, http://biocol.org/urn:lsid:biocol.org:col:34992 or even https://ror.org/0566bfb96. I see the institutionCode is RMNH, at least according to https://www.gbif.org/grscicoll/institution/0e919a55-08d3-4bd2-aec1-200858fafd92 but this term is not populated. Would it be possible to repair these at your end? You had in fact attributed this particular record on the 21st of last month, https://bionomia.net/occurrence/2434489393.

Hi @dshorthouse
thanks for looking into this. I understand the issue now. Yes, we can try to fix it in our end. And we also have several institutionCode for Naturalis in GBIF. We will need to fix that also.


Trying to adpat DLRM but encounter api incompatibility

Describe the feature
DLRM is taking multiple input of different shape as default dataloader design.  When trying to adapt it using engine api, I meet a

Can we allow tuple input for this case as it need less effort compared to turning it into dict?

Can you provide more information? A code snippet will be more helpful.

Hi, I understand your question. This dict setting is designed for pipeline and we will support tuple in later versions.

We have updated a lot. This issue was closed due to inactivity. Thanks.


Make builtin window function output datatype to be derived from schema

Which issue does this PR close?

Closes #.
Rationale for this change
There are scenarios when output datatype for builtin window functions are not the same as hardcoded in DataFusion.
For instance Apache Spark requires some functions like ROW_NUMBER, RANK, etc to be Int rather than UInt which is hardcoded in DF. It would be great to derive builtin window function output type from the incoming schema rather than having the type hardcoded. Having this derivation from the schema increases DF extensibility to be used in other hybrid systems.

What changes are included in this PR?
Changed physical planner for window builtin functions to derive the output data type from schema.

Are these changes tested?
Yes

Are there any user-facing changes?
No

@mustafasrepo @alamb @viirya appreciate for your thoughts


Update eth-block-tracker dep and bump version

The branch we were using (3.0.1-subscription-fix) has been deleted and now you can't run npm i gaba

Codecov Report

Merging #24 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #24   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          18     18
  Lines         889    889
  Branches       97     97
=====================================
  Hits          889    889

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 bb6f68b...8596792. Read the comment docs.


Keyboard shortcuts not activated until a menu opens once

Symptom: Keyboard shortcuts (e.g., ctrl-c, -v, -x) cannot be used until a popup menu opens once.
How to reproduce:

Start a new session
Try ctrl-N and confirm that nothing happens
Open File > New like the screenshot below
Try ctrl-N again

Workaround: not found yet.

Resolved by 131a753126abedf0c4e85dacd6e7221108073db2.


ip4r extension must be installed for migrations to apply

Description
Since https://github.com/stackernews/stacker.news/commit/a5e50821b7e0dba0400a5f8ada6d3741a92de9d3, migrations no longer apply cleanly. You now get an error that the ip4r extension is missing
Environment variables loaded from .env
Prisma schema loaded from prisma/schema.prisma
Datasource "db": PostgreSQL database "stackernews", schema "public" at "localhost:5431"

Error: P3006

Migration `20231212235400_ofac` failed to apply cleanly to the shadow database.
Error:
ERROR: could not open extension control file "/usr/share/postgresql/13/extension/ip4r.control": No such file or directory
   0: sql_schema_connector::validate_migrations
           with namespaces=None
             at schema-engine/connectors/sql-schema-connector/src/lib.rs:324
   1: schema_core::state::DevDiagnostic
             at schema-engine/core/src/state.rs:270

Steps to Reproduce

Run postgres via docker using container defined in docker-compose.yml
Run migrations

Expected behavior
Migrations apply

I tried to fix this for now in my local setup by simply installing the extension as mentioned in this article inside the container:
$ docker exec -it db /bin/bash
$ apt-get update
$ apt-get install postgresql-13-ip4r

But then I get this error if I run the migrations:
Error: P3006

Migration `20210427152429_init` failed to apply cleanly to the shadow database.
Error:
ERROR: could not load library "/usr/lib/postgresql/13/lib/plpgsql.so": /usr/lib/postgresql/13/lib/plpgsql.so: undefined symbol: EnsurePortalSnapshotExists
   0: schema_core::state::DevDiagnostic
             at schema-engine/core/src/state.rs:270

To not get blocked by this, I will just "downgrade" my local git repository to a commit before https://github.com/stackernews/stacker.news/commit/a5e50821b7e0dba0400a5f8ada6d3741a92de9d3 which is https://github.com/stackernews/stacker.news/commit/6763ef7bf046a7680a9949419a6fbbc02f94a629

Sorry about that. There might be a postgres container with the extension already installed.

Sorry about that. There might be a postgres container with the extension already installed.

No worries, worst case, I think we can just update our docker-compose file to create such a container.


name argument added to cross_section function

This PR fixes the issue where the cross_section function never passes the 'name' argument to the CrossSection class.

Thank you Lawrence,
can you run pre-commit install and submit again after running ?
pre-commit run --a

https://github.com/gdsfactory/gdsfactory/actions/runs/3912673569/jobs/6687613950


quick start SetupAWS.py option to use existing Thing/Policy/Cert

This is like day two for me so I might be missing something.

Set up a new device with SetupAWS.py setup
git clean and lose all your local state.
Try to setup again, and get errors because the Thing already exists.  Read through the other available options and scratch your head.

Is there a way to fetch details for an existing Thing, and use those?

As part of git clean, if you have not deleted the generated certificate and private key (i.e. if you have not deleted the untracked files), you can populate the configure.json again and use the following command to configure your project:
SetupAWS.py update_creds

If you have deleted the generated certificate and private key, you need to delete the resources and recreate them using the following commands:
SetupAWS.py cleanup
SetupAWS.py setup

Thanks.

I tried! cleanup seems to depend on generated artifacts:
jrr@jrrmbp ~/r/r/a/d/c/t/aws_config_quick_start ((413f30f6…))> python SetupAWS.py cleanup
Traceback (most recent call last):
  File "SetupAWS.py", line 193, in <module>
    cleanup()
  File "SetupAWS.py", line 147, in cleanup
    delete_prereq()
  File "SetupAWS.py", line 118, in delete_prereq
    cert_id_file = open(cert_id_filename, 'r')
IOError: [Errno 2] No such file or directory: u'john-test-thing_cert_id_file'

The current set of scripts rely on some locally generated files and we will see how we can improve them. For now, would you please delete the created certificate and policy using AWS IoT Console:
https://us-west-2.console.aws.amazon.com/iot/home?region=us-west-2#/certificatehub
https://us-west-2.console.aws.amazon.com/iot/home?region=us-west-2#/policyhub
Note that your links may vary depending on the region you are creating your resources in (change us-west-2 with the region your exists in).
Then run the setup command again:
SetupAWS.py setup

Apologies for the inconvenience.

Yeah, that's where I wound up 😒.
As you look to improve the process: it'd be great if generated sources weren't versioned / versioned sources weren't rewritten by automation.
A developer on a team, finishing up a feature branch, shouldn't have to tease out out their code changes from their I-edited-this-file-so-that-I-can-flash-my-device changes.

Thank you for your feedback. These are really good suggestions and we will take them into consideration while improving these scripts.
Thanks.


do not udevadm trigger, ceph-disk does it

There is no need for osd create to call udevadm trigger action=add. By
running all actions associated with add for all block devices, it may
also trigger some race conditions. For instance, the following happens
about 10% of the time, when using dmcrypt:

ceph-disk prepare --dmcrypt /dev/vdb
/dev/vdb1 starts activating via udev
ceph-deploy triggers action=add
/dev/vdb2 udev chown root /dev/vdb2
/dev/vdb1 activate fails because of permission denied on /dev/vdb2
/dev/vdb2 udev action chown ceph /dev/vdb2

http://tracker.ceph.com/issues/13299 Fixes: #13299
Signed-off-by: Loic Dachary<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/ceph-deploy-pull-requests/1798/
Test PASSed.

Nice catch!  Can you retarget this at infernalis branch?
Which udev rule is doing the root chown?  Is that anything that is in our control?  It seems like we avoid the situation most of the time now but the possibliity is always there that we'll get unlucky...

in any case, Reviewed-by:

Which udev rule is doing the root chown? Is that anything that is in our control? It seems like we avoid the situation most of the time now but the possibliity is always there that we'll get unlucky...

It is my understanding that when a udev rule starts, it chown root (or recreate the file maybe, with default owner ?) and then the chown happens shortly afterwards but that leaves a window of opportunity.

Perhaps the "fix" is to not blindly trigger events on all devices, but only on the specific device that changes?

When looking at some of the ceph-disk changes recently, I saw that it seemed to do all the necessary udevadm calls needed, and that made me think of places I had seen it in ceph-deploy and wonder if those were still necessary.
Glad to see this cleanup!  Thanks a bunch.

Perhaps the "fix" is to not blindly trigger events on all devices, but only on the specific device that changes?

I think that's what ceph-disk does now.

When looking at some of the ceph-disk changes recently, I saw that it seemed to do all the necessary udevadm calls needed

It always did the necessary udevadm / partprobe. The bugs were when it did too many and it created race conditions. For instance calling partprobe actually removes devices before adding them again. I let you imagine what that can do to an ongoing osd activation :-)


"Syntax candy" inaccessible

The extensions for creating Durations from Int and Double haven't been marked public.

Fixed #29


Refactor weight combination schemes

This PR aims to address issue #28, which includes the following tasks:

Removing the functionality relevant to rmse_cutoff
Removing the option final from the YAML parameter w_combine
Deprecating the schemes of multiple swaps.

For the reason why these functionalities are removed/deprecated, please refer to the issue.

Closing the PR since it might be easier to merge the master branch first.


SONAR-19541 Enable users to install an oracle jdbc driver

Hello! Just a passer-by that happens to be having problems with installing the Oracle driver when deploying 10.7.0!
As part of this PR, would it be possible to add functionality to provide credentials when downloading the Oracle driver? (This could be similar to the netrc functionality that currently exists for plugins.) To provide some context, we have Sonarqube deployed in an airgapped environment so we can't download the driver from public unauthenticated mirrors, and the mirror we do have within the airgap environment requires authentication.

Hello @9numbernine9 👋

As part of this PR, would it be possible to add functionality to provide credentials when downloading the Oracle driver?

Thanks for the suggestion. We discussed and gonna add this feature to the current PR 💯
On a side note. The 10.7 charts are out already. For the DCE edition, we added the possibility to configure extraVolumes. This can be used as a workaround to pre-download the driver in an external volume in the cluster and mount it in the right folder. Feel free to leverage our community to post issues that you might encounter.
The current PR's changes will get shipped with 10.8.


Update for iOS 10/Swift 3

Review APIs and apply NS_SWIFT_NAME as appropriate

The demo and test applications have been updated to Swift 3. The documentation and wiki have not been updated to reflect the changes yet.

The updates have been documented.


Color there for tabs

Prerequisites
affected version: 1.16.2
Active / inactive tabs should be more different from color/style point of view

style was changed with 1.7.1


./PalServer.sh not found

Describe the bug
Starting server doesn't succeed due to
sh: 1: ./PalServer.sh: not found
A clear and concise description of what the bug is.
To Reproduce
Run the docker image with documented parameters
Expected behavior
Appropriate bash files to be present in image and server to start
Desktop (please complete the following information)

OS: Debian

Do you have some more information?

docker-compose.yml or docker run command
hardware
full log

Tried both the docker compose and docker run command you provide in readme (with the volume path changed to a real folder)
On an e2-standard-4 compute engine instance on GCP running on Debian, no logs unfortunately since I've deleted the instance but image would start fine and just error on startup with that after steam client init

Maybe a far fetch, but could it be related to: https://github.com/thijsvanloef/palworld-server-docker/issues/93?
I do not have a GCP account which I can test this on

Issue has not been active for a couple of days, reopen if necessary


Permission Denied while creating a temporary dir on AWS EKS

This is virtually the same as #947 and #915.
$>kubectl logs camel-k-kit-bm91v5tosnk1eokq4a10-builder {"level":"info","ts":1569857766.097141,"logger":"camel-k.builder","msg":"Go Version: go1.12.2"} {"level":"info","ts":1569857766.097169,"logger":"camel-k.builder","msg":"Go OS/Arch: linux/amd64"} {"level":"info","ts":1569857766.0971766,"logger":"camel-k.builder","msg":"Camel K Version: 1.0.0-M1"} {"level":"error","ts":1569857766.1634266,"logger":"camel-k","msg":"Unexpected error while creating a temporary dir","error":"mkdir /workspace/builder-394057232: permission denied","stacktrace":"github.com/go-logr/zapr.(*zapLogger).Error\n\t/home/nferraro/gopaths/camel-k/pkg/mod/github.com/go-logr/zapr@v0.1.1/zapr.go:128\ngithub.com/apache/camel-k/pkg/util/log.Logger.Error\n\t/home/nferraro/gopaths/camel-k/src/github.com/apache/camel-k/pkg/util/log/log.go:75\ngithub.com/apache/camel-k/pkg/util/log.Error\n\t/home/nferraro/gopaths/camel-k/src/github.com/apache/camel-k/pkg/util/log/log.go:196\ngithub.com/apache/camel-k/pkg/builder.(*defaultBuilder).Build\n\t/home/nferraro/gopaths/camel-k/src/github.com/apache/camel-k/pkg/builder/builder.go:66\nmain.main\n\t/home/nferraro/gopaths/camel-k/src/github.com/apache/camel-k/cmd/builder/main.go:80\nruntime.main\n\t/opt/go-1.12.2/src/runtime/proc.go:200"}
Kamel: 1.0.0-M1
eksctl: 0.6.0
AWS EKS: 1.14
Thanks much. I see the marker in both referenced issues that this is milestoned for 1.0.0-M2. Thanks much.

This seems to have been fixed in M2. Now it's all just kaniko and hardcoded Google, but I see that issue #613 referenced for the 1.0.0 milestone. Closing.


Possibly conflicting with Ultisnips?

My testing minimal init.vim
call plug#begin('~/.vim/plugged')
    Plug 'neovim/nvim-lsp'
    Plug 'nvim-lua/completion-nvim'
    Plug 'SirVer/ultisnips'
    Plug 'honza/vim-snippets'
call plug#end()

lua require'nvim_lsp'.rust_analyzer.setup{on_attach=require'completion'.on_attach}
au Filetype rust setl omnifunc=v:lua.vim.lsp.omnifunc

" Note: UltiSnips defaults to <tab>
" let g:UltiSnipsExpandTrigger = "<tab>"

let g:completion_enable_snippet = 'UltiSnips'

" Use <Tab> and <S-Tab> to navigate through popup menu
inoremap <expr> <Tab>   pumvisible() ? "\<C-n>" : "\<Tab>"
inoremap <expr> <S-Tab> pumvisible() ? "\<C-p>" : "\<S-Tab>"

" Set completeopt to have a better completion experience
set completeopt=menuone,noinsert,noselect

How to reproduce
Attempt to cycle through or complete something, the tab on the completion menu seems to be overridden by Ultisnips

Expected behavior
Pop-up including both snippets and completions and tab to cycle through
By changing g:UltiSnipsExpandTrigger from <tab> to something else, it seems to fix the issue.

I can't do anything with this issue though because Ultisnips overwrite the keymaps... Please consider changing the expand keys for ultisnips or using \<c-n> and \<c-p> to navigate through popup menu.


Style guide: .editorconfig and .prettierrc.json in source folder

Just an example style guide for us to consider. Most files are 4-space and CRLF, so I followed that.
If we can agree on this style then I'll pull it into my working branch.
My personal opinion: Tabs Are Clearly Superior

Watching this from afar, but tabs are the superior indentation method.


I need to buy you a drink!


ARROW-16811: [C++] Remove default exec context from Expression::Bind

13368

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: Passed
 Everything looks good
Details


Helm does not support Postgres over SSL

Hello,
I have been attempting to set this up on Azure Kubernetes Service using Azure PostreSQL as storage. However, my company mandates SSL communication to databases, and I do not see a way to specify that option on the helm chart.
Is PostgreSQL over SSL supported?

Hi @lwblackledge  sorry for the late response. Yes it's supported out of the box since the <IP_ADDRESS>

for the previou


Docs!

Wrote the standards into two separate documents:

Rendered View standards.md wraps the Parameter name / javadoc sections from the temp. working doc. Some minor wording was changed to target the document towards the audience of "people who want to submit mappings"
Rendered View review-merge-process.md wraps the "Review / Merge process" from the temp working doc. (real.. real shocker that one). It's intended as a codification of the review and merge standards and procedures the team will adhere to and is worded as such.

I intentionally excluded the "Quality of Life tools" section as that's basically just brainstorming and dreams, and the CLA, because, well, it's already in the CLA bot.

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

I would make these ordered lists so we can refer to them on the fly (like if someone violates use american english names for spelling, I could say violates Params 1 instead of the actual requirement).


Angular4 support

Version info
<EMAIL_ADDRESS>Angular:
4.0.3
Firebase:
3.8.0
AngularFire:
<EMAIL_ADDRESS>How to reproduce these conditions
npm install
yarn install
Steps to set up and reproduce
npm install
yarn install
Debug output
warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>My 2 cents
I noticed that the package.json has been upgraded to grant Angular4 support (even if I would suggest to leave Angular2 back-support enabled with:
"@angular/core": ">=2.0.0",
...

In any case, it would be great if you can release a latest version to get rid of that annoying npm warning saying:
warning<EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>
Thanks

See #843.
We plan on releasing soon, but for now you can use the next tag.
npm i angularfire2@next --save


Multiple rsync paths in shell provisioner

copy_from and copy_to is a nice shortcut and we don't want to break existing users in a change, but perhaps we can make it just be a friendly syntactic-sugar version of an insert into something like this:
commands = [
    { type: rsync, from: foo, to: bar },
    "cat this_is_shell_command"
 ]

(And any raw string is just a command).
This would trivially allow multiple rsyncs, and makes way for other command types, like a scp/exec of an inline script or something similar.

done, see new striderfile in repo for example.  Called it 'copy'


Need to update notify service redoc with correct syntax for python code

Currently for all examples the syntax for python shows to use the following
`import http.client
conn = http.client.HTTPConnection("undefinedundefined")
It should be import http.client
conn = http.client.HTTPSConnection("server_url")
`
Using HTTPConnection throws a 302 error

HTTPConnection or HTTPSConnection is decided by
https://github.com/Kong/httpsnippet/blob/71eec07c833c6421270c417a9d6e311b0fb34f27/src/targets/python/python3.js#L22
  const protocol = source.uriObj.protocol   if (protocol === 'https:') {     code.push('conn = http.client.HTTPSConnection("%s")', source.uriObj.host)       .blank()   } else {     code.push('conn = http.client.HTTPConnection("%s")', source.uriObj.host)       .blank()   }
which check current host uri when the app been started.
while in the helm OCP deploy, by default the app tls is not enabled. And with OCP route use tls redirection, the url shows as https. But inside the app when run the code snip script it still will be recognized as tls not enabled.
To fix this and also the url undefined issue, need remove run the code snip generate from the backend and update redoc script on frontend.

Does that mean similar logic will need to be applied on the teflo plugin side? Or we will have the D&O's notify service always resolved to https

With current helm deploy as default, it will be https, and you could found the clue in the URL when you open Swagger UI or Redoc.

The server url issue is resolved.


influx secret list returns misleading error message when user isn't authorized to view secrets [backport 2.0.x]

Backport the fix for #20054 to the 2.0 branch

Duplicate of #20378

Duplicate of #20378


Issue 16, config file format changed

The config file has been changed from dictionary-based access to attribute-based access. This provides a better control of ownership across the code since module attributes are members of singleton "class" (that a python module is an equivalent of). There is also file database file download implemented.
I found some items that seemed to be obsolete and were included in this PR.
Note: Actually this is the first time that I submit a pull requests that branches of a different base. Before working on this, I merged from speed_optimization. Then made changes then merged the last using_cupy commit and then filed the PR. It should just work, but please be careful.
This will close #16

I made this from prince_config import to actually better see what used where. But I'm actually not so convinced anymore. The moving of the Co fig into the prince folder.. I forgot, sorry.
It would be great if you can do this.

I moved the config to ./prince/config.py also changed the imports, such that now always import prince.config as config
A few minor changes, for which I did not want to make a separate PR:

removed examples, now in https://github.com/joheinze/PriNCe-examples
moved tests to ./prince/tests.py


feat: ExpressionMatch on select()

Allows ExpressionMatch on select statement.
Also removed deprecation warnings in sqlalchemy==1.4.47 & 2.0.4 tests.
This does require a bunch of package checking which isn't super nice, not sure if there is a better/nicer way to do that, but this should work fine.
Closes #329

Regarding testing this with real code.
I have tested this against one of my projects.
That project has 29 session.execute(select(...)) statements in it.
All 29 of those statements are being mocked in my tests successfully.
It definitely isn't an exhaustive test as a lot of the statements are fairly similar, but I am having no issues within that project once I modify my local mock-alchemy to support select() statement matching


Error Code : 521-Server Rejected the request

I keep getting this message "Error Code : 521-Server Rejected the request" please help! #

Me too, it happened to me since today, it doesnt let me buy manually through the web app neither.

Error Code : 512-Server Rejected the request
It also gives me this error, but I suppose its the same.

Think its soft Bann?

Have it too now for 16h. In Android app everything works and in Game

its a soft ban...

How can I avoid IT when it works again?
Its annoying 😂

I had it also just now, mine lasted 30 mins....

Thats ok, but now 16h for me :(

Yeah I figured out it's a softban but it lasted for 23h :(

Yeah I figured out it's a softban but it lasted for 23h :(

Did you test it every hour and then after 23h it worked again?

Yeah, went to bed for about 9 hours but when I wake up nothing happened

Works now again, But still have my bug see issues

I have this with almost every search, I put now the amount of buys to 1 instead of 3. Also get a lot of user did too much requests

les code erreur 521 / 512 c'est des blocage par le serveur Fifa, ban provisoire c'est à dire que votre compte est en cours de vérification par le système de Fifa, il faut impérativement fermé la console et ne plus ressayé jusqu'a 6H minimum. C'est la moyenne ça peut durer plus au moins. veuillez attendre et ne ressayer qu'après 6H ça devrait fonctionner et assurez vous de bien régler les paramètres " COMMON SETING" avant de ressayer comme sur la capture par sécurité pour ne pas bannir vos compte.


TCP client send timeout

I'm trying to use socks for my network layer in iOS application and method client.socket.setSendingTimeout(_:) appears to be broken. Client reading forever if network is unreachable. Timeout works well for receive, but not for read. I looked at tests and noticed that they don't cover this case at all:   https://github.com/vapor/socks/blob/master/Tests/SocksCoreTests/TimeoutTests.swift#L82

My current solution (⚠ it's really bad 🚧):
let timeout: DispatchTimeInterval = .seconds(4)
let sendWork = DispatchWorkItem {
  guard let client = self.client else { return }

  do {
    try client.send(bytes: bytes)
    // send success to callback
  } catch let error {
    // send error to callback
  }
}

writingQueue.async(execute: sendWork)
DispatchQueue.global(qos: .background).asyncAfter(deadline: DispatchTime.now() + timeout) { [weak work=sendWork] in
  guard let work = work else { return }
  work.cancel()
  DispatchQueue.main.async {
    // send time out error to callback
  }
}

    public func setReceivingTimeout(_ newValue:timeval) throws {
        if isClosed { throw SocketsError(.socketIsClosed) }
        try descriptor.setOption(level: SOL_SOCKET, name: SO_RCVTIMEO, value: newValue)
    }

    public func setSendingTimeout(_ newValue: timeval) throws {
        if isClosed { throw SocketsError(.socketIsClosed) }
        try descriptor.setOption(level: SOL_SOCKET, name: SO_SNDTIMEO, value: newValue)
    }

That's strange to hear as the implementations for both send/recv have identical implementations. I have no idea what this could be.


duplicate schema names in database navigator panel

This behavior on all 6.1.1 versions (linux, windows,eclipse with dbeaver plugin)
Database PostgresSQL (11.2.1 postgrespro build)

This seems to be a bug in schem list reading.
DBeaver uses following query to read schemas:
SELECT n.oid,n.*,d.description FROM pg_catalog.pg_namespace n
LEFT OUTER JOIN pg_catalog.pg_description d ON d.objoid=n.oid

How many rows it returns? I assume that your database has dup rows in pg_description table (generally this shouldn't happen ever).
What if you will add distinct in the beginning:
SELECT distinct n.oid,n.*,d.description FROM pg_catalog.pg_namespace n
LEFT OUTER JOIN pg_catalog.pg_description d ON d.objoid=n.oid

It looks like a problem in the "description" column, randomized comments are loaded, and then we get duplication
In other utilities (pgadmin, omnidb and etc) the list of schemes is displayed correctly.

I think I found the bug.
Could you test the fix in the latest Early Access version (https://dbeaver.io/files/ea)?

Sorry,bug is still there

Sorry, I've published wrong version. Please test the latest one.
Now schema read query is:
SELECT n.oid,n.*,d.description FROM pg_catalog.pg_namespace n
LEFT OUTER JOIN pg_catalog.pg_description d ON d.objoid=n.oid AND d.objsubid=0

Duplication Fixed

But where the schemas comments came from (description column) is not yet clear :) They should not be at all (comments are only to the tables)

Indeed. In addition to objsubid we need condition on column classoid. Now fixed (EA is ready).
Thanks for testing it

Great, thanks!


Use init-test as all jobs parents

Using init-test jobs to ensure workspace exist in pre,
and copy log files on node in post.

looks good to me +1


[Bug]: OpenCLIPEmbeddingFunction() function is not working without openai api key in Chroma Multi-Modal llamaindex Demo

Bug Description
This is about the demo at https://docs.llamaindex.ai/en/stable/examples/multi_modal/ChromaMultiModalDemo.html
OpenCLIPEmbeddingFunction() is a open source model from open_clip which doesn't require openai api key. OpenCLIPEmbeddingFunction is used to create vector database
(chroma_collection = chroma_client.create_collection(
"multimodal_collection",
embedding_function=OpenCLIPEmbeddingFunction(),
data_loader=image_loader,
).
But it is using default service_context when index is defined:
(index = VectorStoreIndex.from_documents(
documents,
storage_context=storage_context,
)
which implies it is invoking openai models for query embedding. Ideally your embedding model for vector database creation and for retrieval (i.e., for query embedding) should be same, i.e. OpenCLIPEmbeddingFunction(). The service _context in the index should take OpenCLIPEmbeddingFunction() as embed_model which is currently not possible.
Version
0.9.27
Steps to Reproduce
Just follow the steps in the llamaindex tutorial page:
https://docs.llamaindex.ai/en/stable/examples/multi_modal/ChromaMultiModalDemo.html
Relevant Logs/Tracbacks
No response

@saimunikoti actually, with multimodal, clip sucks at text-to-text retrieval.
What is happening here is, images are embedded and retrieved with clip, and text is embedded and retrieved with whatever other embedding model (openai in this case)
@hatianzhang is our multimodal guy, he can maybe confirm this 😁

@saimunikoti actually, with multimodal, clip sucks at text-to-text retrieval.
What is happening here is, images are embedded and retrieved with clip, and text is embedded and retrieved with whatever other embedding model (openai in this case)
@hatianzhang is our multimodal guy, he can maybe confirm this 😁

Thanks for responding. But even when we are trying to use different embedding models for text (i.e., BAAI/bge-large-en) and image (openclipembedding), it seems to not working. Its tightly coupled with openai api functions. Is there any workaround to skip openai functions and use open source models like BGE and openclipembedding ?

@saimunikoti Yea, just define the service context with the embedding model you want right?
service_context = ServiceContext.from_defaults(llm=llm, embed_model=HuggingFaceEmbedding(model_name="BAAI/bge-base-en-v1.5"))
index =  VectorStoreIndex.from_documents(...., service_context=service_context)


volume control in settings web ui

https://github.com/MiczFlor/RPi-Jukebox-RFID/commit/54d2b25b6f68186264de2c093149ec833dfc4061#commitcomment-32145669

I think there is something more to fix (links on the top of the settings page, class and anchor "language"). I fixed it in my installation but have to read how to make pull requests :)


(regression) computedFrom is only applied the first time the ViewModel is bound

I've prepared a Gist demo, however since there isn't a version after the computedFrom refactor, I'll just present what used to work and now is broken:
https://gist.run?id=ce252a8c24716e5d3620
To see what I mean switch between "computed" and "else". In the previous Aurelia version (the one running the Gist) this works well, as random is only computed once, every time when the route is bound. However if you run the same code using the latest version of Aurelia, when you switch to "computed" then back to "else" then back to "computed", the binding will become dirty-checked and a new Random value will be seeded every few hundred ms.
Might this be related to the recent computedFrom refactor by @jdanyow?

thanks @niieani - good catch!


Update index.html

CNCF Mentoring - open source program

Hi @AshaHolla , Thankyou for creating pr but I can't merge it as CNCF is already present in the website.

I am closing this pr @AshaHolla


Sileo Not Installed After Jailbreak...!

[] Patching and signing iBSS/iBEC
main: Starting...
iOS 15 iBoot detected!
getting get_debug_enabled_patch() patch
getting get_sigcheck_patch() patch
applying patch=0x1800c3324 : 000080d2
applying patch=0x1800c3374 : 000080d2
applying patch=0x1800c4f38 : 200080d2
main: Writing out patched file to iBSS.patched...
main: Quitting...
main: Starting...
iOS 15 iBoot detected!
getting get_debug_enabled_patch() patch
getting get_sigcheck_patch() patch
applying patch=0x1800c3324 : 000080d2
applying patch=0x1800c3374 : 000080d2
applying patch=0x1800c4f38 : 200080d2
main: Writing out patched file to iBEC.patched...
main: Quitting...
none
none
[] Patching and signing kernelcache
krnl
[] Signing DeviceTree
dtre
[] Patching and signing trustcache
trst
none
[] Booting device
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[==================================================] 100.0%
[] Tweaks enabled, running postinstall.
[!] Please install OpenSSH, curl, and wget from Sileo (repo is mineek.github.io/repo). Then, press any key to continue
As you can you see above, jailbreak is done completely. But I don't see Sileo pre installed in the device. I need help, what should be done further in order to see sileo or cydia in the device?

Did u check Tips app?

@Evoal I didn't get you, Sileo will be found inside Tips app?

Just open the tips app and install sileo, if it doesn't show anything, install pogo From IPA File then try to install sileo

@Evoal I didn't get you, Sileo will be found inside Tips app?

Thanks for the help. When I tapped on Tips app I didn't see any pogo app. So I downloaded IPA separately from nightly link site. I signed the app and installed in iOS 15.1. When I tap Tools>Do All nothing works, and also I tapped install it says "Error installing Bootstrap -1"

how you install sileo ? the same error happends to me (when I open tips nothings shows up to me just the tips of ios !!)

same error any fixes ?????????

try to use sshrd and use that command ./sshrd.sh $(yourversionIOS) TrollStore TV after that you use palera1n to install pogo and boot, after that open TV and install ldid and trollstore when trollstore is installed you need to download pogo ipa https://nightly.link/elihwyma/Pogo/workflows/build/main/Pogo.zip and install it with trollstore

Hey @edwin170 Thanks for the suggestion. I tried your method. I used SSHRD trying to install TrollStore TV, still nothing I can see and also I tried same process of palera1n, still no luck.

Hey @edwin170 thanks for the solution. I fixed it and working fine now. It's not because of trollstore, I had to reinstall palera1n again.

Hey @edwin170 thanks for the solution. I fixed it and working fine now. It's not because of trollstore, I had to reinstall palera1n again.

Sileo works ?


Re-render and remove the pin to fiona 1.9.3

Pinning fiona does not solve the issue of the getting the broken fiona from the default channel. Right now the only workaround is to ensure our is the latest version with a higher build number than the default channel.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

What is the prevailing issue. If we can track that down, we can either have a package on defaults fixed, or fix something in conda-forge.

I don't think there is an issue now. The fiona pinning was a remnant of the IOOS channel that leaked here.


Remove deprecated metadata.updates.* configuration

Since CDAP-5108 introduced audit publishing, the old metadata updates stuff was deprecated.
Removing it, to simplify the configuration.

LGTM


Number sections in the book chapters

Note: on the web, this just numbers the sections 1, 2, 3, etc. for latex/book.pdf this shows as "1.1" for chapter 1 section 1, just like the copy edited book.


locale_t etc are unusable

18:51 <Hello71> how do I check if a libc::locale_t is null?
18:52 <Hello71> it's just an enum, so I can't call is_null on it
18:52 <Hello71> as *const u8 == null? :/
18:53 <talchas> Hello71: you should never have a locale_t
18:53 <talchas> you should only ever have *const locale_t
18:53 <Hello71> then how do I use uselocale and newlocale and so on
18:53 <Hello71> those return locale_t, not *const locale_t
18:54 <Hello71> because C locale_t is really a pointer to a struct. it has to be, because (locale_t)0 is used everywhere
18:56 <talchas> hmm, that would be a bug in the libc crate then
18:56 <talchas> yeah, that's wildly incorrect
18:56 <talchas> having a value of enum Foo {} is UB
18:57 <Hello71> sounds about right
18:57 <Hello71> I don't like how POSIX likes to define x_ts as primitive types sometimes and structs other times and sometimes pointer types
18:58 <occultus> That means uselocale is completely uncalleable from safe code. Is newlocale actually bound?
18:58 <Hello71> actually I guess locale_t can be an integral type, but I'm willing to wager a guess that it's a pointer to struct everywhere
18:59 <talchas> occultus: yeah, it's just a bug in libc, and given that the current definition is 100% worthless it's probably safe to just change its definition of locale_t to something useful
19:00 <occultus> oh wait, newlocale is also uncallable becuase it takes a locale_t for base. heh.
19:02 <occultus> talchas: does it qualify as a bug or just not yet implemented?
19:04 <talchas> occultus: given that it's providing the type and fns, I'd call it a bug
19:13 <Hello71> https://github.com/rust-lang/libc/blob/master/src/fuchsia/mod.rs#L73 lmao
19:14 <Hello71> talchas, occultus: ^
19:14 <occultus> my guess is they just thought "if they're uninhabited nobody can use them, so nothing bad can happen"
19:14 <occultus> except that something bad does happen, people see them in the docs but can't use them
19:15 <talchas> Hello71: well, that's even just weighing in on whether you want *const empty_struct or *const empty_enum
19:16 <talchas> (ie, it probably wants to be struct DIR(()); or equivalent so that accidentally dereferencing a *const DIR is useless rather than UB)

seems reasonable to me to change! It sounds like no one's likely relying on the current implementation, so we can probably tweak it.


Customize Error Messages

What is the best way to customize error messages coming from the FX as an application developer

Hi,
I'm sorry if this is a silly question as you guys may not be in touch with the non-ASP.NET 5 of ASP.NET Identity.
Do you know if there's any chance that this could be backported in the initial ASP.NET Identity - the one on CodePlex? It's very hard to identify errors in the IdentityResult class since it's only based on the resource messages.
Thanks

At least please provide a static method like IdentityResult.SetResourceProvider(IIdentityResultResourceProvider provider). Is there any plan to incorporate this to version 2.x?
Thanks

Any update on this issue?

Same here, looking to translate IdentityResult.Errors in Identity 2.x


CLI-like hidden virtualenv

Update user docs to reference client installation into a 'hidden' virtual env, similar to https://docs.globus.org/cli/installation/virtualenv/

Fixed with documentation PR#320