DoNothing() + Upsert.ID() returns low-level SQL error in Postgres

Current Behavior 😯
id, err := entc.User.Create().
  SetEmail(email).
  OnConflict(
    sql.ConflictConstraint("user_email_key"),
  ).
  DoNothing().
  ID(ctx)

The above code results in an error if there's a conflict. Postgres's RETURNING combined with ON CONFLICT DO NOTHING doesn't return any rows, and the internal attempt at scanning the expected returned rows causes a low-level SQL error to leak from the ent API and be returned here.
Using Ignore() instead of DoNothing() works as expected. Here's an example of the difference in behavior.
Expected Behavior 🤔
One of two things should happen: Either usage of Postgres as a driver should cause a call to ID() combined with DoNothing() to immediately return an error, or the error returned from the SQL should be wrapped nicely into something that can be dealt with using ent's API.
This odd edge case should also probably be documented in the DoNothing() and/or ID() godoc comments, along with a recommendation of taking a look at Ignore() as a potential workaround.
Your Environment 🌎

Tech
Version

Go
1.16.7

Ent
v0.9.0

Database
Postgres 12, Postgres 13

Thanks for reporting this @DeedleFake.

One of two things should happen: Either usage of Postgres as a driver should cause a call to ID() combined with DoNothing() to immediately return an error, or the error returned from the SQL should be wrapped nicely into something that can be dealt with using ent's API.

I prefer the first option. The same check applies to SQLite as well.

This odd edge case should also probably be documented in the DoNothing() and/or ID() godoc comments, along with a recommendation of taking a look at Ignore() as a potential workaround.

Agree. Would you like to contribute this change? Thanks 🙏

Agree. Would you like to contribute this change? Thanks 🙏

Certainly. What error should it return?

Adding another DoNothing failure reported by @ernado.
https://github.com/ernado/ent-upsert-no-rows/blob/master/bug_test.go#L33-L51

Is there a way to insert a row only when there’s no conflict and determine if the row is inserted afterwards? This should be possible with ON CONFLICT DO NOTHING in postgres and sqlite. However, neither ID nor Exec would work in this case.

any updates on this? would really appreciate an upsert that returns ID. or is there an alternative approach?

Current Behavior 😯
id, err := entc.User.Create().
  SetEmail(email).
  OnConflict(
    sql.ConflictConstraint("user_email_key"),
  ).
  DoNothing().
  ID(ctx)

The above code results in an error if there's a conflict. Postgres's RETURNING combined with ON CONFLICT DO NOTHING doesn't return any rows, and the internal attempt at scanning the expected returned rows causes a low-level SQL error to leak from the ent API and be returned here.
Using Ignore() instead of DoNothing() works as expected. Here's an example of the difference in behavior.
Expected Behavior 🤔
One of two things should happen: Either usage of Postgres as a driver should cause a call to ID() combined with DoNothing() to immediately return an error, or the error returned from the SQL should be wrapped nicely into something that can be dealt with using ent's API.
This odd edge case should also probably be documented in the DoNothing() and/or ID() godoc comments, along with a recommendation of taking a look at Ignore() as a potential workaround.
Your Environment 🌎
Tech	Version
Go	1.16.7
Ent	v0.9.0
Database	Postgres 12, Postgres 13

using Ignore uses ON CONFLICT DO UPDATE and that could lead to deadlock?

FYI, DoNothing().Exec(ctx) also returns &errors.errorString{s:"sql: no rows in result set"} but IMO shouldn't


Remove redundant :'' from sources without url

It's effectively the same thing, just more confusing.

Thanks for the cleanup @mvdan. LGTM

retest this please


Input-Translation

Ref: https://www.gnu.org/software/screen/manual/html_node/Input-Translation.html

https://en.wikipedia.org/wiki/ANSI_escape_code#Sequence_elements


Sprockets resolve cache

I think this is the real fix to https://github.com/jekyll/jekyll-assets/issues/228 (I don't think https://github.com/jekyll/jekyll-assets/commit/db4cd7f8e04aa985fdcef97b9f3203b16164cb17 fully addressed the problem).
This PR brings down the build time of our Jekyll site from about 80 seconds to about 40 seconds on my machine. The big one is the resolve method. If your site contains the same asset in multiple pages, putting a debugger in there will show you that that method it is being called repeatedly with the same arguments, trying to resolve the same stuff over and over again.
Fixing it by caching the result and by overwriting the Sprockets superclass method.
This is probably not the best way to fix this, but it works and we verified that the generated output of our site is identical.
Is there a better way to do this?
Is there something we need to do for incremental regeneration?
Thoughts?
@envygeeks @rafaelfranca @knverey / cc @parkr @admhlt

Is resolve that expensive in jekyll application? In a normal sprockets application we don't cache resolve and it is just a few ms.

@rafaelfranca for us, the thing that had the most impact was the lack of a manifest, which I resolved last night, I've never really experienced this problem on my own sites but I can foresee it happening with very large asset chains (this might be why Discourse takes 20 years to build on our servers?)

On the incremental regeneration part, there is nothing that needs to be done.  We actually work outside of Jekyll independently and ask Core not to interfere with us and we don't interfere with it.  We do tie into the regenerator in the tag but that's about the most that needs to be done, as we let Sprockets deal with deciding whether an asset has changed or not (it's the one that caches and busts on our behalf.)

FWIW I tried our site on current master (after you merged your changes last time) and this PR still gives a substantial (!) performance improvement on top of that.

@envygeeks: Do you have any intention of merging this? Anything left you want me to do or clean up or anything?

@fw42 the pull request has conflicts and is doing something that we already do with @cached in lib/jekyll/assets/env.rb so I was waiting for those to be resolved so I could merge or I was going to manually patch with git when I had time.

@fw42 if you have a second to fix it, I'll get it merged today and add some other stuff that's been waiting and get a new stable pushed out

Rebased

:heart:


chore(deps): update terraform github.com/terraform-ibm-modules/terraform-ibm-landing-zone to v6.1.1

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/terraform-ibm-landing-zone
module
patch
v6.1.0 -> v6.1.1

Release Notes

terraform-ibm-modules/terraform-ibm-landing-zone (github.com/terraform-ibm-modules/terraform-ibm-landing-zone)
v6.1.1
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the landing zone das (#903) (69ff383)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline


Request for control over the text in the center of Doughnut charts

Is your feature request related to a problem? Please describe.
I need to show a different center text in a doughnut chart instead of the out of the box message shidoka gives.

Describe the solution you'd like
If the out of the box plugin can be made optional then center space gives me a clean slate for my plugin to write any text over there.
I am referring to this out of the box plugin for center text that shidoka uses: https://github.com/kyndryl-design-system/shidoka-charts/blob/main/src/common/plugins/doughnutLabel.js#L6

@bikashp2023  can you explain the use case for needing to overwrite the center content?

@cunningham-kyndryl I need to present numbers with commas to enhance readability, both in the peripheral and at the  center. I am not able to do it in the center as, for shidoka donut  charts center text is not optional for me to have a clean slate to write my own text.
As demonstrated below, the center text needs to have commas in the number.


initial preact implementation

Unsure what's going on with the CI tests - they seem ok locally but preact tests are flakey in CI.  Will look into it more another time

Any update on this PR?

@IshanKBG I believe we were waiting on a suspense fix in preact so defer would work right, but there's probably no reason that needs to hold up an initial merge here.  I'll see if I can find some time this week or next to maybe remove defer/Await from this initial PR and get a working version out and then we can focus on those in a separate PR

Oh nice
Thanks for notifying me

I'd love to use this if possible!


[SDP-1074] make add user cmd multi-tenant

What

Make add-user cmd multi-tenant by adding --tenant-id flag.
Remove migrate cmd from stellar-auth package. We're performing all migrations in one place.

Why
https://stellarorg.atlassian.net/browse/SDP-1074
Known limitations
[TODO or N/A]
Checklist
PR Structure

[ ] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[ ] This PR title and description are clear enough for anyone to review it.
[ ] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[ ] This PR adds tests for the new functionality or fixes.
[ ] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[ ] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[ ] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[ ] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[ ] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[ ] This is not a breaking change.
[ ] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[ ] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr228.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr228.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr228.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr228.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr228.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr228.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr228.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr228.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr228.previews.kube001.services.stellar-ops.com


[Bug] Won't start. Dependency issues maybe.

Version
1.1.0
Reproduction link
https://www.creative-tim.com
Operating System
macOS Big Sur
Device
Macbook Pro
Browser & Version
firefox 90.0.2
Steps to reproduce
npm -v 7.20.5
node -v 16.4.0

Download repo
$ npm install
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^16.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-swipeable-views
npm ERR<EMAIL_ADDRESS>from the root project

// also happens with react-chartist

delete package-lock.json and /node_modules
run npm install --save --legacy-peer-deps
still breaks

Sometimes the install passes, sometimes it hangs.
Haven't been able to get npm run dev to work.
What is expected?
Run npm install and npm run dev.
What is actually happening?
npm install fails

Solution
Additional comments

Try
npm install --force
or
npm install --legacy-peer-deps

The force is required because deprecated packages are being brought in. Can we get those updated because ummm. we can't build with those.

Still an issue

is still an issue?

Still an issue for me. I'm using node 18.13.0 and npm 9.2.0


Docs for socket based UDFs in Kapacitor

@rossmcdonald or @gunnaraasen can you review this?

On it.

@nathanielc Updated with small fixes around grammar/punctuation and wording. Let me know if you think the changes look okay.

@rossmcdonald +1


Decide if we want to use release branches

Decide if we want to use release branches

hm not sure if we need this - only thing would to update a release for a given kyverno release/version

Yep, would be nice to support multiple versions in the future.

so we want to organize release branches for kyverno versions? But we then coupled to kyverno versions for playground version?
Not sure how we organize it the best way

Probably just adopt the same semver strategy and go 1:1 with Kyverno.
Kyverno 1.10 = Playground 1.10
Kyverno 1.10.1 = Playground 1.10.1

Hm  i don’t think that will work all the time. We might want to release fixes or features between kyverno releases

No problem to use strings after those versions.


Accessing validation data within a custom callback ??

Describe the current behavior
Hi, how can I access validation data within a custom callback ?
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
from __future__ import absolute_import, division, print_function, unicode_literals

import os
import matplotlib.pyplot as plt
try:
  # %tensorflow_version only exists in Colab.
  %tensorflow_version 2.x
except Exception:
  pass
import tensorflow as tf
import datetime

print("TensorFlow version: {}".format(tf.__version__))
print("Eager execution: {}".format(tf.executing_eagerly()))

from tensorflow.keras.layers import Dense, Flatten, Conv2D
from tensorflow.keras import Model

mnist = tf.keras.datasets.mnist

(x_train, y_train), (x_test, y_test) = mnist.load_data()
x_train, x_test = x_train / 255.0, x_test / 255.0

model = tf.keras.models.Sequential([
  tf.keras.layers.Flatten(input_shape=(28, 28)),
  tf.keras.layers.Dense(128, activation='relu'),
  tf.keras.layers.Dropout(0.2),
  tf.keras.layers.Dense(10, activation='softmax')
])

class MyCustomCallback(tf.keras.callbacks.Callback):
    def on_epoch_end(self, epoch, logs=None):
        print('Hello')
        print(self.validation_data[0].shape)

def customLoss3(y_true, y_pred):
    return tf.reduce_mean(tf.keras.losses.sparse_categorical_crossentropy(y_true, y_pred))

#model.compile(optimizer='adam', loss=customLoss3, metrics=['accuracy'])

model.compile(optimizer='adam', loss=customLoss3, metrics=[tf.keras.metrics.SparseCategoricalAccuracy()])

model.fit(x_train, y_train, epochs=5, validation_split=0.2, callbacks=[MyCustomCallback()])

model.evaluate(x_test, y_test)

Other info / logs
TypeError: 'NoneType' object is not subscriptable

Can you please try the solution mentioned in this issue and let me know if it helps. thanks!

How can we access the self.model._targets[0] and self.model.outputs[0] inside a Custom Callback at on_batch_end?

Hi
I am also facing this issue while defining my own custom callback. I raised it in a wrong prf earlier this morning . Quoting the reference here [(https://github.com/keras-team/keras/issues/10472.)]
Here is the test case description to reproduce the issue:
`I am executing the code on google colab and tf version is 2.2.0-rc2 , i have built a custom callback for calculating the F1 score and AUC score.
I am getting the same error with fit. I am not using fit_generators as data augmentation in not necessary in my code.
`from tensorflow.python.util.tf_export import keras_export
@keras_export('keras.callbacks.Callback')
class MyCustomCallback(tf.keras.callbacks.Callback):
def on_train_begin(self, logs={}):
self._data = []
def on_epoch_end(self, epoch, logs={}):
print(type(self.validation_data[0]))
print(type(self.validation_data[1]))
X_val, y_val = self.validation_data[0], self.validation_data[1]
y_predict = np.asarray(model.predict(X_val))
y_val = np.argmax(y_val, axis=1)
y_predict = np.argmax(y_predict, axis=1)
self._data.append({
'val_microF1Score': f1_score(y_val, y_pred, average='micro'),
'val_rocauc': roc_auc_score(y_val, y_predict),
})
print('The f1 score {:7.2f} and auc {:7.2f}  for epoch {} .'.format( logs['val_microF1Score'], logs['val_rocauc'],epoch))
return
def get_data(self):
return self._data
mycallback = MyCustomCallback()`
I am invoking fit with these arguments
history = model.fit(X_train, y_train, epochs=nb_epoch,batch_size=batch_size,validation_split=0.2,validation_data = (X_cv,y_cv),callbacks=[mycallback])
Could you please suggest any workaround temporary for this as my work is held up due to this
Thank you


Ask some questions

Have you ever tried to solve the dynamic vehicle scheduling problem with this model？

We have not had the chance to tackle the dynamic versions of VRP, but it would certainly be very interesting to see how it goes.


[DOC] Typos

Codecov Report

Merging #21 (ce6f6f6) into main (050f7f2) will decrease coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
- Coverage   94.22%   93.92%   -0.31%
==========================================
  Files          21       21
  Lines         658      658
  Branches      110      126      +16
==========================================
- Hits          620      618       -2
- Misses         21       22       +1
- Partials       17       18       +1

Impacted Files
Coverage Δ

src/manifestoo/main.py
76.92% <ø> (ø)

src/manifestoo/core_addons/__init__.py
92.85% <0.00%> (-7.15%)
:arrow_down:

Continue to review full report at Codecov.

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


Add the CMB

@BumseBine Please provide us with: a proof showing that the school recognizes the domain cmb-portal.de, which you are submitting as an official email domain for the students.

@BumseBine You might miss my previous comment. Please react.

@BumseBine I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for the understanding!

Sorry for the late answer. My email program didn't display that I got
messages. Can you send a email to<EMAIL_ADDRESS>which is my
student email or is there any other method to confirm the email?
Sincerely
Mauritz Funke
Philip Torchinsky<EMAIL_ADDRESS>schrieb am Mo., 2. Dez. 2019,
10:21:

@BumseBine https://github.com/BumseBine I am sorry, this request
remains unanswered for a while. I have to close it without merging. Please
reopen and comment if you have an update regarding the aforementioned
issue. Thank you for the understanding!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/7200?email_source=notifications&email_token=AC7GDDUZHDA3MPRMYMK6FUDQWTHSJA5CNFSM4JQ3QA42YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFSZ2NA#issuecomment-560307508,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC7GDDR2MSJMTDDZNAJRF63QWTHSJANCNFSM4JQ3QA4Q
.


Update news-box width

taking the width from 90% to 100% will be better on mobile.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
246

Relevant Lines:
323

💛  - Coveralls


Add a DagOptimizer test

We are using the DagOptimizer at stripe before planning to reduce the size of some online graphs (went from 115 storm bolts or so to 69 in one example).
However, even in the case where we reach 69, there are rules that don't seem to be fully applied and I have not yet found out why.
Anyway, more test coverage never hurts.

@ttim can you take a look?

Codecov Report

Merging #745 into develop will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #745      +/-   ##
===========================================
+ Coverage    72.23%   72.28%   +0.05%
===========================================
  Files          154      154
  Lines         3742     3742
  Branches       209      209
===========================================
+ Hits          2703     2705       +2
+ Misses        1039     1037       -2

Impacted Files
Coverage Δ

.../twitter/summingbird/memory/ConcurrentMemory.scala
97.14% <0%> (+1.42%)
:arrow_up:

.../main/scala/com/twitter/summingbird/Producer.scala
77.27% <0%> (+1.51%)
:arrow_up:

Continue to review full report at Codecov.

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

okay, this now fails for me (both the fanOut and the idempotency test).
cc @non

actually, I can't get idempotency to fail now... maybe it is just fanOut being broken being the issue, which some of the rules use...

Okay, I don't see a bug actually. It was an issue with the test actually, and an inconsistency between how fanOut was defined in Dependants and ExpressionDag.
This test actually seems to pass for me now. I'll try to find any counter example next week, but so far, I guess everything looks correct still.

okay, idempotency failure:
 arg0 = Summer(IdentityKeyedProducer(NamedProducer(IdentityKeyedProducer(MergedProducer(IdentityKeyedProducer(FlatMappedProducer(LeftJoinedProducer(IdentityKeyedProducer(NamedProducer(IdentityKeyedProducer(NamedProducer(IdentityKeyedProducer(OptionMappedProducer(IdentityKeyedProducer(FlatMappedProducer(Source(List(-483916215,<PHONE_NUMBER>, -1,<PHONE_NUMBER>, -1094725117, -1107074050,<PHONE_NUMBER>, -894421428, 438074163, -1618870436,<PHONE_NUMBER>, -927761159,<PHONE_NUMBER>, 1,<PHONE_NUMBER>, 1, -941762984, 1, -2147483648,<PHONE_NUMBER>, -2147483648, -1360150793, -1774434453, -1968350011, -1999846660, 1, 940697649, -561677987,<PHONE_NUMBER>, 734987219, 0, -1731750202, 782845389, 0, 158805678, -1, 1, -560692419, -1, 879491512, 1, 685151277, 333970011, -2147483648, 961509966, -2147483648, -1, 870342926,<PHONE_NUMBER>,<PHONE_NUMBER>, -178556721, 0, -1028446598, -2147483648, 1,<PHONE_NUMBER>, 1, -1915898801,<PHONE_NUMBER>, 519019579,<PHONE_NUMBER>, 0,<PHONE_NUMBER>, -1, -877887704,<PHONE_NUMBER>, -224941626, -675798454, 1,<PHONE_NUMBER>, -487281773, 638669148, -2147483648, 1,<PHONE_NUMBER>, -2147483648, 1, -567098335, -1, 795549827, -1995849024, -577239589,<PHONE_NUMBER>, 1, 0, -1839762855, 0,<PHONE_NUMBER>, -385664255,<PHONE_NUMBER>,<PHONE_NUMBER>, -1701772912, -1, -1,<PHONE_NUMBER>, 330749634, 1,<PHONE_NUMBER>, -2147483648,<PHONE_NUMBER>)),org.scalacheck.GenArities$$Lambda$3183/1815517983@3d533ae3)),org.scalacheck.GenArities$$Lambda$3183/1815517983@368308bf)),tjiposzOlkplcu)),tvpwpdyScehGnwcaVjjWvlfuwxatxhdjhozscucpbq)),Map(1122506458 -> -422595330, 985940285 -><PHONE_NUMBER>, -1012102957 -> 577191710,<PHONE_NUMBER> -> 0,<PHONE_NUMBER> -> -1, -1571538327 -> -106754684, -883421086 -> -1843831487, -1186741108 -> -1297188435, 511615026 -> 892838107,<PHONE_NUMBER> -> 0, -1 -> 0, 654842167 -><PHONE_NUMBER>, 766280486 -> 0, 46050994 -> -2147483648, -146388742 -> -1080181663,<PHONE_NUMBER> -><PHONE_NUMBER>, -1170327451 -> 41927314, -1265823639 -><PHONE_NUMBER>, -1531812404 -> -1, -1426771473 -> 0, 107420900 -> 1, -778321474 -> -613855929, 730121132 -> -1824061783, -193486826 -> 0, -1603389086 -> -1, -1663776702 -><PHONE_NUMBER>,<PHONE_NUMBER> -> -1, -249195648 -> 0,<PHONE_NUMBER> -> 470882984, -1635961338 -> 995590274, -1315404816 -> 1, 303482491 -> 1, -1120629917 -> -913091299,<PHONE_NUMBER> -> 0, 1 -> 431843073, -1055837163 -> -428420920,<PHONE_NUMBER> -> -2147483648, -248886982 -> -2003982440,<PHONE_NUMBER> -> -1, 796959542 -> 1, -1658050660 -> -1870329784,<PHONE_NUMBER> -> -1878450200, 992361458 -> -1, -315984592 -><PHONE_NUMBER>, -1137239383 -> 1, -102255667 -> -1, -734906869 -> -611180863, -473252927 -><PHONE_NUMBER>, 886258153 -> 1, -916284450 -> 1,<PHONE_NUMBER> -> -1740554658, -2147483648 -> -1363669838, 0 -> -1)),org.scalacheck.GenArities$$Lambda$3183/1815517983@2bb51e67)),IdentityKeyedProducer(OptionMappedProducer(IdentityKeyedProducer(FlatMappedProducer(Source(List(-483916215,<PHONE_NUMBER>, -1,<PHONE_NUMBER>, -1094725117, -1107074050,<PHONE_NUMBER>, -894421428, 438074163, -1618870436,<PHONE_NUMBER>, -927761159,<PHONE_NUMBER>, 1,<PHONE_NUMBER>, 1, -941762984, 1, -2147483648,<PHONE_NUMBER>, -2147483648, -1360150793, -1774434453, -1968350011, -1999846660, 1, 940697649, -561677987,<PHONE_NUMBER>, 734987219, 0, -1731750202, 782845389, 0, 158805678, -1, 1, -560692419, -1, 879491512, 1, 685151277, 333970011, -2147483648, 961509966, -2147483648, -1, 870342926,<PHONE_NUMBER>,<PHONE_NUMBER>, -178556721, 0, -1028446598, -2147483648, 1,<PHONE_NUMBER>, 1, -1915898801,<PHONE_NUMBER>, 519019579,<PHONE_NUMBER>, 0,<PHONE_NUMBER>, -1, -877887704,<PHONE_NUMBER>, -224941626, -675798454, 1,<PHONE_NUMBER>, -487281773, 638669148, -2147483648, 1,<PHONE_NUMBER>, -2147483648, 1, -567098335, -1, 795549827, -1995849024, -577239589,<PHONE_NUMBER>, 1, 0, -1839762855, 0,<PHONE_NUMBER>, -385664255,<PHONE_NUMBER>,<PHONE_NUMBER>, -1701772912, -1, -1,<PHONE_NUMBER>, 330749634, 1,<PHONE_NUMBER>, -2147483648,<PHONE_NUMBER>)),org.scalacheck.GenArities$$Lambda$3183/1815517983@3d533ae3)),org.scalacheck.GenArities$$Lambda$3183/1815517983@368308bf)))),ncn)),Map(),com.twitter.algebird.IntRing$@4deb204c),

 arg1 = com.twitter.summingbird.planner.DagOptimizer$RemoveNames$@7d710ac9.orElse(com.twitter.summingbird.planner.DagOptimizer$RemoveIdentityKeyed$@2a6e2829).orElse(com.twitter.summingbird.planner.DagOptimizer$FlatMapFusion$@728d18e6).orElse(com.twitter.summingbird.planner.DagOptimizer$OptionMapFusion$@5dba444a).orElse(com.twitter.summingbird.planner.DagOptimizer$OptionToFlatMap$@28e82a76).orElse(com.twitter.summingbird.planner.DagOptimizer$KeyFlatMapToFlatMap$@731c6022).orElse(com.twitter.summingbird.planner.DagOptimizer$FlatMapKeyFusion$@69cc1c8f).orElse(com.twitter.summingbird.planner.DagOptimizer$ValueFlatMapToFlatMap$@66db9de8).orElse(com.twitter.summingbird.planner.DagOptimizer$FlatMapValuesFusion$@f09a0df).orElse(com.twitter.summingbird.planner.DagOptimizer$FlatThenOptionFusion$@1f04dc6c).orElse(com.twitter.summingbird.planner.DagOptimizer$DiamondToFlatMap$@16258367).orElse(com.twitter.summingbird.planner.DagOptimizer$MergePullUp$@2c2b7896).orElse(com.twitter.summingbird.planner.DagOptimizer$AlsoPullUp$@6ee63a94)

)

I'll try to reproduce that failure and fix.

This seems to fix the issue.
The bug was with fanOut. It was working in the Expr[_, _] and Id[_] space, which is not meaningful to the rules, which operate on the original N[_] space. The problem is that rewrites can give the same node two different Ids. When that happens, it looks like the fanOut is greater than it is, and this can prevent rules from applying.
If you jump back to the N[_] space, and then back into the optimizer, you reset the Ids and you may have a chance that the rule would apply then. This seems to be the problem.
Along the way I cleaned things up slightly.
Can you take a look @ttim

had a storm flake. Restarted a 2.12 build.

I'd like to cherry pick this when we merge and publish a version of 0.10.1 which includes this fix. We are using the optimizer and it would be helpful to us.
Also, I'm considering breaking this code out into a standalone library. I copied it into scalding, but it is inconvenient to have to publish these large projects to update this totally generic DAG rewriting tool.

 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.


Make parsing of piped data in speed.c more robust

Fixes openssl#19050

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@mattcaswell One reasonable way to test this is to run it before and after the changes and compare the result numbers. Would you have any other idea?

This pull request is ready to merge

Merged to master branch. This does not apply cleanly to 3.1. Closing


Wrapping cmake macro simplification

Python wrapping code is simplified (inline single use functions) and reduce contamination in the global namespace.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

@thewtex   ITK_WRAP_DOC:BOOL=ON succeeded (after doxygen and graphviz installs made).
Generating docs for compound itk::Statistics::RandomVariateGeneratorBase...
Generating graph info page...
Generating directory documentation...
Generating dependency graph for directory Core
Generating dependency graph for directory include
Generating dependency graph for directory Mesh
Generating dependency graph for directory Modules
Generating index page...
Generating page index...
Generating module index...
Generating namespace index...
Generating namespace member index...
Generating annotated compound index...
Generating alphabetical compound index...
Generating hierarchical class index...
Generating member index...
Generating file index...
Generating file member index...
Generating example index...
finalizing index lists...
writing tag file...
Generating search index
Running plantuml with JAVA...
Running dot...
Patching output file 1/1
lookup cache used 7517/262144 hits=58690 misses=8020
finished...
[2144/2144] Linking CXX shared module Wrapping/Generators/Python/itk/_ITKMetricsv4Python.so

Build finished

@dzenanz Squashed many like topics together.  Did not change the codebase.


MSPSDS-514: Remove validation and style text in reporter flow

In line with comments in here: https://regulatorydelivery.atlassian.net/browse/MSPSDS-514

Pull Request Test Coverage Report for Build 771

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.3%) to 88.619%

Files with Coverage Reduction
New Missed Lines
%

../controllers/investigations/report_controller.rb
1
93.75%

test/system/reporter_flow_test.rb
2
94.87%

Totals

Change from base Build 755:
-0.3%

Covered Lines:
1129

Relevant Lines:
1274

💛  - Coveralls


The form helpers are rendering an extra '%s'

Describe the bug
Printing an element in the traditional manner, with:
{{ formElement(profileForm.get('email') }}

Results in the printing of an extra %s character.
<div class="form-group">
    <input type="email" name="email" maxlength="254" required="required" autofocus="autofocus" placeholder="Email" class="forms_field-input form-control" value="">%s
</div>

To Reproduce
Steps to reproduce the behavior:

Load TwbsHelper in application.config.php
Create a simple form with this init method

    public function init()
    {
        $this->add([
            'name' => 'email',
            'id' => 'email',
            'type' => Email::class,
            'options' => [
                'label' => _("Your Email Address"),
                'label_attributes' => ['for' => 'email'],
            ],
            'attributes' => [
                'maxlength' => 254,
            ],
        ]);
    }

Load the form object into a template, and print the form.

Expected behavior
The form is printed as expected.
Screenshots
https://imgur.com/a/X69TVNX
Desktop (please complete the following information):

OS: OSX Mojave
Browser Any
Version Any

Additional context
The error is caused by this block in TwbsHelper\Form\View\Helper\FormElement
    // Add a sprintf directive for correct render of errors
    if (!in_array($sElementType, $this->options->getIgnoredViewHelpers()) &&
        !($oElement instanceof Collection)
    ) {
        $sMarkup .= "%s";
    }

Fixed, thanks your PR


Is it possible to provide related information about reduce rule?

Hi, I am a beginner of set covering problem. The repo is very excellent, and I learned a lot from it. But there are some confused concepts like mandatory set, identical element, etc. in reduce rules.  Is it possible to provide related information about reduce rule? Thanks.

Hi,
These are classical reductions for the set covering problem.
For example

If an element is covered by a single set, then it is mandatory to use this set
If tow sets or two elements are identical, then one of them can be removed from the problem

Thanks for your reply.


Integrate Users classes

Adds necessary classes (with tests) to be compliant with Tatter\Users.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
58

Relevant Lines:
58

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
58

Relevant Lines:
58

💛  - Coveralls


hotfix(usos): fixed USOS login not working

:tada: This PR is included in version 1.5.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Allow git to update line endings, based on the new gitattributes.

When I do a repository checkout on Windows, it modifies the line endings of most of the files in the repository. This is do to the gitattributes which was added about six months ago.
I don't know if this is due to my git configuration (or not).
I followed the instructions at https://help.github.com/articles/dealing-with-line-endings/ in order to create a new commit which "fixes" the line endings to those specified in the gitattributes file.
The other option would be to delete the gitattributes file. I don't know which is better, but it is annoying to have git marking all the files as modified after a checkout.

@pigrew, I have removed the .gitattributes file - see 2697eaa


Chain Sync Improvements: Phase 1

Goals:

Align derivation paths with iOS (this is in a dashj PR)
Align keys uses in Identities with iOS

Key alignment with iOS

Identity registration transitions are still signed by the key that has its P2PKH in the credit burn output

Identity public keys are now taken from the blockchain identity derivation path, from wallet.blockchainIdentityKeyChain, currently only index 0.

This PR depends on #14

This was merged manually.


When Device location is 'off'  ,  "Enable Location"  button isn't working  - ios App v1.0.1 issue

Version

Type of phone:  iPhone
OS Version: ios 13.3
Model: iPhone XR

Steps to reproduce

Device location is set to "off" .

launch  COVID SafePath app

Click on "Get Started"

Accept the Licensing agreement

Cross the 3 onboarding screen

Next Screen will be " Location access " and "Allow notifications " options.

clicking on "Enable Location"  button at bottom  - isn't working
8  After clicking "Skip the step"

Next screen will be "Enable notifications ".

Expected Behavior
"Enable Location"  button working

Can you please assign it to me. I will look at this issue.

Hi @ShreyaaSahu Can you please verify on ANdroid also?

This problem does not exist in Android.  In Android there is a different problem - if location settings are "off" on the device, the app allows location services to be set up & reports them all as working.  But in fact they are not as locaton services are "off".
We should check this out as part of re-testing #719 - I will add a note there.

Please assign this to me

We have four kind of cases for location permission
UNAVAILABLE,
DENIED,
BLOCKED,
GRANTED
and 3 states for location permission(UNKNOWN, Granted, Blocked).
In app we are managing only two cases with 2 states(Granted and Blocked), we have a state called UNKNOWN which is not managed in app.
In this specific scenario we have UNAVAILABLE case from location permission, i would like to ask what will be expected behaviour?
Does app continues to next state i.e notification permission and updating location permission state to UNKNOWN.
What will happen in case of location service denied as we are not handling this case.

Suggest you ask in #app_dev for expected behavior.

Couldn't resolve this issue due to blockers
https://github.com/Path-Check/covid-safe-paths/issues/890
https://github.com/Path-Check/covid-safe-paths/issues/911
Once above issues are resolved, i can commit changes to Onboadring5 file to resolve this issue.

We can close this issue as it's not happening


Sign-out of External Identity Providers

https://identityserver4.readthedocs.io/en/release/topics/signout_external_providers.html

done


docs: add example of deleting an array element

Reading this doc made me think it wasn't including the situation where delete was used to remove an array element - which does not update the array's length which is often unexpected. However, trying it, it does flag this as an error, so I think it would be useful to have it in the docs.
  3:1  error  Modifying an existing object/array is not allowed  functional/immutable-data

Maybe it could use to more explanation, but I'm not sure where that would do in this document.
Fixes: #(Issue Number)
Proposed Changes

Thanks for the PR 😄


Integration without account constraint

Hi,
is it also possible to integrate the security cams without an eufy account? Currently I'm using the cams only with Homekit and now I want to integrate them into my home system but I don't want to stick them into any cloud.
Would be great if this could be possible.
Cheers

It is not possible afaik.

As @schliemann has already said.


Update ZK storage version at the beginning of a migration

Before starting a ZK storage migration, Marathon should update the storage version, to prevent older versions from running with an incompatible state (see #3004).
It also has to set (and persist) a migrationInProgress boolean flag, so that on case of errors the migration is resumed.

Like we discussed should the indicator migrationInProgress is enough to decide if the marathon instance if operating on a possibly incompatible state.

I feel like this one is dead. Only the leader can migrate and that blocks all other operations (more or less)


RBAC: Allow "all" operator in the clusters' list

Issue submitter TODO list

[X] I've searched for an already existing issues here
[X] I'm running a supported version of the application which is listed here and the feature is not present there

Is your proposal related to a problem?
No response
Describe the feature you're interested in
Hello,
I want to assign view or admin permissions automatically to new Kafka clusters. For that reason, it would be great if there was an "all" or "*" operator under clusters' list.
For example, see the following configuration sample:
rbac:
  roles:
    - name: "readonly"
      clusters:
        - all (or *)
      subjects:
        - provider: ldap
          type: group
          value: "XXX"
      permissions:
        - resource: XXX
          actions: [ "view" ]

Describe alternatives you've considered
No response
Version you're running
83b5a60
Additional context
No response

hi, please see #4255
we have an 'all' rbac operator at https://github.com/kafbat/kafka-ui

Thanks a lot, Haarolean!


python: add a C-wrapper based on python-cffi

CFFI provides the ability to use the same C wrapper
for python2, python3 and pypy.

@sbinet apologies for the late reply.  Thanks for submitting this!!!


fix disable the field rather than using label

Not a huge fan of label in a form, so tried this to display the field as disabled.
See if that works for you.
Nevertheless this is not working for me, trying to figure out why:

Now it works nicely :partying_face:


Setting text color in a table cell

I'm trying to dynamically set the text color in a cell based on a value, and am struggling to find the right way to do this via XML.  @davidjb was great in helping figure out how to set the background for a cell:
||<w:tcPr><w:shd w:val="clear" w:color="auto" w:fill="00FF00"/></w:tcPr>||
That works as advertised.  Any advice on setting the color of the text?

To set the colour of the text, use the XML:
<w:color w:val="FFFFFF"/>

This can be combined with the background colouring to do both for a given cell:
||<w:tcPr><w:shd w:val="clear" w:color="auto" w:fill="00FF00"/></w:tcPr><w:color w:val="FFFFFF"/>||Your cell value here

Thanks.  I tried this and it's not working for me.  I tried it both with and without the background changes and while the background changes worked the text color didn't.  This is the function I'm using to return the XML string:
styledText: (value: string, color: string) => {
return ||<w:tcPr><w:color w:val="${color}"/></w:tcPr>||${value};
}
I've tried hard-coding the color as well, to no avail.   Any thoughts?

Thanks for your reply, @davidjb !


Add error returns to average area functions, etc

Coverage decreased (-0.03%) to 98.173% when pulling a294e67b74a7af2f13fd2defde46ff0b396c04ab on isaacbrodsky:error-returns-last into 017e9b45b7b2bb747e31cad70fcea0939dc9550b on uber:master.


[CENNSO-192] Adjust tool for our use-case

CENNSO-192
Related PR: https://github.com/travelping/cennso/pull/376

Also maybe adjust readme a bit, stating that it's a fork of XYZ repo?

When this is needed, we can always do this in a separate PR as it will probably lead to more review feedback, e.g. from technical writers, further postponing the merge of the functionality itself.


Fix helm

@trevorbox Thanks for the hard work and pushing through all of the changes. This is a great addition!


add stage-releases methods

---- Autogenerated Waffleboard Connection: Connects to #434

I created the "follow-up" issue:
https://github.com/resin-io/resin-sdk/issues/573

I've ended up with the following methods:

application.isTrackingLatestRelease
application.getReleaseHash
application.setToRelease
application.disableLatestReleaseTracking
application.enableLatestReleaseTracking
device.isTrackingApplicationRelease
device.getTrackedReleaseHash (to not confuse the hash with the value of is_on__commit)
device.setToRelease
device.enableApplicationReleaseTracking

Which make me thinking:

should we rename application.getReleaseHash to application.getTrackedReleaseHash for symmetry?
should we drop application.disableLatestReleaseTracking for symmetry?
This would mean that in order to stop tracking the latest release, you should use application.setToRelease which makes sense especially if we also agree the following bullet
should we rename application.enableLatestReleaseTracking & device.enableApplicationReleaseTracking to application.trackLatestRelease & device.trackApplicationRelease respectively?
should we rename setToRelease to pinToRelease?

@thgreasi
ad1: makes sense
ad2: I would say application.disableLatestReleaseTracking should stay for users that are on current release and don't want to update the release in the future. Making them to use setToRelease with the current release they are already set sounds counterintuitive.
ad3: what about application.trackLatestRelease and  device.trackLatestApplicationRelease ?

@amdomanska WRT point 3 enableApplicationReleaseTracking/trackApplicationRelease clears the should_be_running__release field of the device, making it to track the release that the application commit points to. device.trackLatestApplicationRelease does sound different than what was described above (tracking the latest is different than tracking whatever the app tracks).
PS: Afaik, the API doesn't have support for device.trackLatestApplicationRelease atm even if we wanted to provide that in the SDK.

@resin-ci retest

@amdomanska and @thgreasi whats the status on this PR, is it blocked on something?

@shaunmulligan It's waiting for reviews. I will reping everyone in FD.
I was trying this branch on the dashboard's console yesterday, and it was working nicely. It was great beng able to do staged releases w/o the .sh scripts (using this js api).

@shaunmulligan now with npm pre-publishing of resinCI you can npm i the result of this branch right away :)
npm i<EMAIL_ADDRESS>


Winning claims with supports fail to resolve due to amount formatting error

Does this fix all the issues described here? https://github.com/lbryio/lbry/issues/804

I believe it should fix that although you may need to clear out your cache for the changes to take place

Closing as this appears to be fixed, @jackrobison re-open if I'm misunderstanding.


Add server-side integration

Problem
This is how I do things now:

I call identify(userId, traits) from the back end.
I then only call identify(userId) from the front end

..and because there is no server side integration, the first identify call with traits doesn't work. Leaving me without traits in Sentry.
Solution
Add Server-side integration

Cheers,
Victor

Hi @pudgereyem, as part of the monorepo migration, this issue has been moved to new issue. Our engineers have been notified and will prioritize and work on it ASAP. Thank you!
For more information, see README.md.


Referencing non-existent uc-integration release version

The referenced uc-integration-api-0.0.7.tgz version in package.json doesn't exist. There's no release yet.
Use a direct git reference instead: "uc-integration-api": "git+https://github.com/unfoldedcircle/integration-node-library.git#658552bf6a9836766e3df315c1546043e3060656"
https://docs.npmjs.com/cli/v9/configuring-npm/package-json#git-urls-as-dependencies

Sorry, wrong repository


migration failed with error: show fields from proxy_host where field = 'forward_ip'

I'm receving the following error while doing the normal set-up of NGINX proxy manager.
migration failed with error: show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:14 PM] [Global   ] › ✖  error     show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:15 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
[7/12/2020] [3:34:15 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
migration file "20181019052346_forward_host.js" failed
migration failed with error: show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:15 PM] [Global   ] › ✖  error     show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:16 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
[7/12/2020] [3:34:16 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
migration file "20181019052346_forward_host.js" failed
migration failed with error: show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:16 PM] [Global   ] › ✖  error     show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
[7/12/2020] [3:34:17 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
[7/12/2020] [3:34:17 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
migration file "20181019052346_forward_host.js" failed
migration failed with error: show fields from proxy_host where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only

I am also getting the same issue with a brand new build today:
app_1  | [8/27/2020] [1:58:46 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
app_1  | [8/27/2020] [1:58:46 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
app_1  | migration file "20181019052346_forward_host.js" failed
app_1  | migration failed with error: show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
app_1  | [8/27/2020] [1:58:46 PM] [Global   ] › ✖  error     show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
app_1  | [8/27/2020] [1:58:47 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
app_1  | [8/27/2020] [1:58:47 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
app_1  | migration file "20181019052346_forward_host.js" failed
app_1  | migration failed with error: show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
app_1  | [8/27/2020] [1:58:47 PM] [Global   ] › ✖  error     show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
app_1  | [8/27/2020] [1:58:48 PM] [Migrate  ] › ℹ  info      Current database version:<PHONE_NUMBER>4513
app_1  | [8/27/2020] [1:58:48 PM] [Migrate  ] › ℹ  info      [forward_host] Migrating Up...
app_1  | migration file "20181019052346_forward_host.js" failed
app_1  | migration failed with error: show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only
app_1  | [8/27/2020] [1:58:48 PM] [Global   ] › ✖  error     show fields from `proxy_host` where field = 'forward_ip' - ER_OPEN_AS_READONLY: Table '/var/tmp/#sql_1_0.MAI' is read only

Any suggestions?

This is a mysql database issue. The MySQL database files have somehow changed permissions so that MySQL daemon is unable to open the file. Check the data permissions, if using a mysql/mariadb docker image for the database, the files need to be a specific user as well.

This is a mysql database issue. The MySQL database files have somehow changed permissions so that MySQL daemon is unable to open the file. Check the data permissions, if using a mysql/mariadb docker image for the database, the files need to be a specific user as well.

Permissions were fine. Owned by MySQL:
[root@rcn-ops-hg1-dev nginxproxymanager]# docker exec -it nginxproxymanager_db_1 ls -la /var/lib/mysql
total 122924
drwxr-xr-x    5 mysql    mysql          243 Aug 27 14:11 .
drwxr-xr-x    1 root     root            19 Mar 10 08:20 ..
-rw-rw----    1 mysql    mysql            2 Aug 27 14:11 718a022dde16.pid
-rw-rw----    1 mysql    mysql        24576 Aug 27 14:11 aria_log.00000001
-rw-rw----    1 mysql    mysql           52 Aug 27 14:11 aria_log_control
-rw-r-----    1 mysql    mysql         1940 Aug 27 14:11 ib_buffer_pool
-rw-rw----    1 mysql    mysql     50331648 Aug 27 14:11 ib_logfile0
-rw-rw----    1 mysql    mysql     50331648 Aug 27 13:54 ib_logfile1
-rw-rw----    1 mysql    mysql     12582912 Aug 27 14:11 ibdata1
-rw-rw----    1 mysql    mysql     12582912 Aug 27 14:11 ibtmp1
-rw-rw----    1 mysql    mysql            0 Aug 27 13:54 multi-master.info
drwx------    2 mysql    mysql         4096 Aug 27 13:54 mysql
drwx------    2 mysql    mysql         4096 Aug 27 13:54 npm
drwx------    2 mysql    mysql           20 Aug 27 13:54 performance_schema

Ended up moving to MySQL and container started and connected without any issues. I could replicate this on multiple nodes within our environment.
Here was my docker-compose.yml:
[root@rcn-ops-hg1-dev nginxproxymanager]# cat docker-compose.yml
version: '2'
services:
  app:
    image: 'jc21/nginx-proxy-manager:latest'
    restart: always
    ports:
      - '6080:80'
      - '6081:81'
      - '6443:443'
    volumes:
      - /opt/proxymanager/config.json:/app/config/production.json
      - /opt/proxymanager/data:/data
      - /opy/proxymanager/letsencrypt:/etc/letsencrypt
    depends_on:
      - db
    environment:
      DISABLE_IPV6: 'true'

  db:
    image: 'jc21/mariadb-aria:10.4'
    restart: always
    environment:
      MYSQL_ROOT_PASSWORD: 'npm'
      MYSQL_DATABASE: 'npm'
      MYSQL_USER: 'npm'
      MYSQL_PASSWORD: 'npm'
    volumes:
      - /opt/proxymanager/data/mysql:/var/lib/mysql

Apart form the MariaDB hiccup the tool is working perfectly without a hitch


A way to update the build in jre

Hey Edvin,
could you think of any way to make it possible updating the build in jre too?
Besides customizing the updater ui (which was mentioned in another issue) and passing in some jmv arguments, it's the one thing left on my list :)

Hi! No, unfortunately that is outside the reach of FXLauncher in it's current form. For version 2.0 this is definitely on the list, but I need to concentrate on TornadoFX 100% until we finish the book and wrap up the last of the features we have in the pipe line. After that I will finish the updater ui (I've done a lot of it already but got hit with a lot of new features in TornadoFX). Sorry this is taking so long :)


"Incorrect ack received" error while controlling device via Google Home.

Seeing an occasional Incorrect ack received crash controlling my commissioned "device" via Google Home. Log attached. You will see some output from "curl" interspersed with the node-mattter output because I am controlling my physical device via a simple http call using the "-on" and "-off" parameters - works great, very handy to expose control this way.
GoogleHome.txt

I experienced a comparanle issue with iOS ... see #181 ...

Thanks for reporting this!
This seems to be an issue in the MessageExchange logic when there is a packet retransmisison:
10:19:20.834 sending id:25313/30302/2011971280 t:1/5 reqAck:true -> original message
10:19:20.844 sending id:25313/30302/2011971280 t:1/5 reqAck:true -> retransmission (not sure why after only 10ms)
10:19:20.857 onMessageReceived id:17869/30302/51617566 t:1/1 acked:2011971280 reqAck:true -> 1st Ack for<PHONE_NUMBER>
10:19:20.867 sending id:25313/30302/2011971282 t:1/5 acked:51617566 reqAck:true -> new msg sent
10:19:20.991 onMessageReceived id:17869/30302/51617569 t:0/16 acked:2011971280 -> 2nd Ack for<PHONE_NUMBER>, but it was expected to receive an ack on the new message,<PHONE_NUMBER>
Two issues here:

the re-transmission after 10ms is too fast. The default is 300ms, it might be changed during the setup but seems really too short
unexpected acks should be ignored rather than causing a crash

This portion of the code had been written before the Matter specs were released so probably the business logic is not 100% correct....

Here is a MR that hopefully fix this issue: https://github.com/mfucci/node-matter/pull/182

Seems to be fixed by this PR

I can also ack that


[pangolin] Refix dependency python and fix dependency glew

Use @ras0219-msft suggestion to fix dependency python.
Fix find dependency glew.

Related: https://github.com/microsoft/vcpkg/pull/12856#issuecomment-672958395

Temporary close this PR, will continue this work in the future.

In favor of https://github.com/microsoft/vcpkg/pull/20006.


Wrangler is unable to publish w/ authentication failure, using env vars

$<EMAIL_ADDRESS>CF_API_KEY="xxxxx" ~/.cargo/bin/wrangler publish
⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
✨   Built successfully, built project size is 517 bytes. ✨
Error: Something went wrong! Status: 400 Bad Request, Details {"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}

Trying to publish a project (w/ no changes, directly from wrangler generate) using env vars, Wrangler is unable to authenticate. Similar error to #367, but w/ env vars

could not repro with current master. closing. please reopen if you still have trouble... your problem might be the quotes

I'm having a similar issue.
My wrangler.toml file looks like:
account_id = "my-account-id-went-here-with-quotes"
name = "my-project"
type = "webpack"
# route = ""
workers_dev = true
# zone_id = ""

The account_id has been substituted.
The project I was building was from:
wrangler generate my-project https://github.com/cloudflare/worker-template

I've verified the API key in ~/.wrangler/config/default.toml is correct using:
curl -X GET "https://api.cloudflare.com/client/v4/user/tokens/verify" \
     -H "Authorization: Bearer api-key-went-here" \
     -H "Content-Type:application/json"

and the response appears to be successful:
{"result":{"id":"id-was-here","status":"active"},"success":true,"errors":[],"messages":[{"code":10000,"message":"This API Token is valid and active","type":null}]}

I also tried using the command below, without having account_id in the wrangler.toml:
wrangler preview

This was the result:
⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
✨  Built successfully, built project size is 517 bytes.
⚠  Your wrangler.toml is missing the following fields: ["account_id"]
⚠  Falling back to unauthenticated preview.
👷  GET https://00000000000000000000000000000000.cloudflareworkers.com
👷  Your worker responded with: Hello worker!

I'm having a similar issue. I installed wrangler using cargo install wrangler on a MacBook Pro running macOS Mojave 10.14.6
For reference, here are the toml files I used:
~/.wrangler/config/default.toml:
email =<EMAIL_ADDRESS>api_key = "api-key-went-here-with-quotes"

my-project/wrangler.toml:
account_id = "account-id-went-here-with-quotes"
name = "my-project"
type = "webpack"
# route = ""
workers_dev = true
# zone_id = ""

I tested the API key from the cloudflare workers dashboard:
curl -X GET "https://api.cloudflare.com/client/v4/user/tokens/verify" \
     -H "Authorization: Bearer api-key-went-here" \
     -H "Content-Type:application/json"

Result:
{"result":{"id":"id-went-here","status":"active"},"success":true,"errors":[],"messages":[{"code":10000,"message":"This API Token is valid and active","type":null}]}

I used an example template:
wrangler generate my-project https://github.com/cloudflare/worker-template

I ran preview, before I modified the account_id from account_id = ''
wrangler preview

⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
✨  Built successfully, built project size is 517 bytes.
⚠  Your wrangler.toml is missing the following fields: ["account_id"]
⚠  Falling back to unauthenticated preview.
👷  GET https://00000000000000000000000000000000.cloudflareworkers.com
👷  Your worker responded with: Hello worker!

Then I modified account_id and set it to the account id on the cloudflare worker dashboard. I expected this might have resulted in an error:
wrangler preview

⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
✨  Built successfully, built project size is 517 bytes.
Error: https://api.cloudflare.com/client/v4/accounts/8ebb8d81a9b45eb8b6d4a42008072281/workers/scripts/my-project/preview: Client Error: 400 Bad Request

Then I tried publish and it resulted in a 400 Bad Request. The account_id was populated was still set to the account id on the cloudflare worker dashboard:
wrangler publish

⬇ Installing wranglerjs...
⬇ Installing wasm-pack...
✨  Built successfully, built project size is 517 bytes.
Error: Something went wrong! Status: 400 Bad Request, Details {"success":false,"errors":[{"code":10000,"message":"Authentication error"}]}

Happy to help with further debugging

hey @alexmcroberts, thanks for the thorough repro!
so this is an interesting thing! Cloudflare's API actually supports two types of authentication: an email/global api key, and an api token. based on the curl command you ran to check yours, you are actually trying to use an api token in place of the global api key. support for api tokens is on our radar (you can follow #471 for progress), but not yet implemented. In the meantime, I found my global API key on this page https://dash.cloudflare.com/profile/api-tokens UNDER the API tokens section.
Hope this helps!

Thanks @ashleymichal !
If wranger ony works with a global API key right now, can we have wrangler config throw an error when you try to use it with a non-global key?

CLI Quick Start https://github.com/cloudflare/wrangler/issues/cli  has a button called "Get your API token" , not "Global API Key" . So this is confusing.

@oleksandrborniak you are 100% right! sorry about that and we'll get it fixed right away

PayPal me directing at<EMAIL_ADDRESS>last years start of contract
On Wed, Oct 9, 2019, 6:11 AM ashley williams<EMAIL_ADDRESS>wrote:

@oleksandrborniak https://github.com/oleksandrborniak you are 100%
right! sorry about that and we'll get it fixed right away
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cloudflare/wrangler/issues/371?email_source=notifications&email_token=AMOCTOY6ON35ATPRBFYRRGLQNXJ6RA5CNFSM4IH6LMXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAX2OZA#issuecomment-539993956,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMOCTO6O6F7VS7W644H77UDQNXJ6RANCNFSM4IH6LMXA
.

Just run into this.
Also you could change Enter api key: for Enter Global API Key and even print the https://dash.cloudflare.com/profile/api-tokens link

I'm still getting this error, despite using my global API key. Unable to preview, dev, or publish.

"Unable to preview, dev, or publish" till now in China internet. and while I got an VPN which Legal and compliant in China, they told me that, should add some while list, so, how do I know the while list hostname or ip that wrangler use. Thank you very much!


kScrollMultiplier should be configurable in Windows

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
The scroll is junky, scrolling large chunks of the page instead of smooth from default browser
Expected Behavior
Smoth scrolling, making this variable default as 1 (or even better, customizable), fixes the problem
Steps with code example to reproduce
N/A
Stacktrace/Logs
N/A
Flutter version
latest
Operating System, Device-specific and/or Tool
Windows 11 23H2
Plugin version
v6.1.5
Additional information
No response
Self grab

[ ] I'm ready to work on this issue!

https://github.com/pichillilorenzo/flutter_inappwebview/blob/f67ae1f34868c7660c16a2473f53a7a7bfb6f784/flutter_inappwebview_windows/windows/in_app_webview/in_app_webview.cpp#L1792

same problem, jumps by 500 pieces with one mouse scroll

Yep, it probably should be something configurable. I can add it as a new parameter in InAppWebViewSettings 👍.
By the way, what could be the best default value for it? 1 should be ok?

Yep, it probably should be something configurable. I can add it as a new parameter in InAppWebViewSettings 👍. By the way, what could be the best default value for it? 1 should be ok?

yes I think 1 is the best solution, because everyone expects the same behavior as in the browser

Released new version 6.2.0-beta.1. The default value is 1 and you can change it using InAppWebViewSettings.scrollMultiplier.


[DCOS-44605] Filter sensitive environment keys and add file based secret support for frameworks

Introduce regex based blacklisting of log-sensitive environment variables
Add file based secret handling support for SDK. We still support environment based secrets for backwards compatibility.
Update frameworks mustache files with file based secrets.

Thanks for review @elezar and @kaiwalyajoshi
Backporting (cherry-picked only the first commit from this PR) via #2751


[Candidate 1v1 [RANK] Melting Snow ZH v4] Are map.ini changes safe?

Issue Description
Are map.ini changes safe?
WaterSet MORNING
SkyTexture = TSsunny_s.dds
WaterTexture = TSWater.dds
UScrollPerMS = 15.00
VScrollPerMS = 15.00
SkyTexelsPerUnit = 3.2
WaterRepeatCount = 32
End

WaterTransparency
TransparentWaterMinOpacity = 0.7 ;opacity of water at maximum depth - 1.0 is opaque, 0 transparent.
TransparentWaterDepth = 4.0 ;depth below water surface at which it becomes opaque (unless clipped by TransparentWaterMinOpacity)
StandingWaterColor = R:107 G:200 B:243 ;material color used to tint the water surface.
StandingWaterTexture = TWWater01.tga ; main texture used for water (usually something blue that looks like water).
AdditiveBlending = YES; use additive blending instead of alpha blending?
RadarWaterColor = R:107 G:200 B:243 ; color of water on radar display.

SkyboxTextureN = TSSunny_N.dds
SkyboxTextureE = TSSunny_E.dds
SkyboxTextureS = TSSunny_S.dds
SkyboxTextureW = TSSunny_W.dds
SkyboxTextureT = TSSunny_T.dds
End

Expected behavior
Map.ini changes do no carry over to other maps.

It's all about water colours and transperency which slightly changed to fit to the map. All fine to me.


backup reversible blocks into portable format and import reversible blocks from the portable format

Output reversible blocks in portable format anytime the reversible block database is recovered.
Add tool to chain_plugin to create a reversible blocks database from the portable reversible blocks file.
This allows users to share their reversible blocks in a portable format so that devs can do forensic analysis.

The documentation below assumes that PR #4680 has also been merged in:

Exporting and Importing the  Reversible Block Database using a Portable Format
The reversible block database, which is normally located in the "blocks/reversible" directory within the blockchain data directory, contains all the blocks in the current chain that are not yet irreversible. This database is stored as a shared memory file similar to the chain state database, which means it is not generally portable across machines.
If a node operator wants to move the reversible block database from one machine to another, they normally must use the --export-reversible-blocks and --import-reversible-blocks options on nodeos to, respectively, extract the reversible block database from the source machine into a portable file format and then import the blocks from that portable file into a newly constructed reversible block database on the target machine.
This feature is also incredibly useful in providing EOSIO blockchain developers with the necessary data that they need (in addition to the blocks.log file) to assist in debugging a problem that may have occurred on a live EOSIO blockchain.
Exporting the Reversible Block Database
The reversible block database can be exported into a portable file using the --export-reversible-blocks option. For example:
nodeos --export-reversible-blocks path_to_exported_reversible_blocks_file

will save the reversible blocks into the file path_to_exported_reversible_blocks_file.
Automatic Exportation of Reversible Block Database on Recovery
Whenever the reversible block database has to be recovered, either explicitly due to the --fix-reversible-blocks option or perhaps implicitly during a --hard-replay-blockchain, nodeos will automatically export the recovered reversible blocks into portable file format which is stored as the file portable-reversible-blocks-<timestamp> under the blocks directory where <timestamp> is the time at which the recovery occurred. This automatically generated file should be identical to the file that would have been generated with the explicit --export-reversible-blocks option.
Importing the Reversible Block Database
Given a file containing the reversible block database in the portable format, it is possible to use the --import-reversible-blocks option to import those blocks into the reversible block database that nodeos can use. Using this option will first clear out any existing reversible block database before importing the blocks from the specified file. Here is an example of how nodeos can be used to import the reversible blocks that were previously exported from another node:
nodeos --import-reversible-blocks path_to_exported_reversible_blocks_file


fix typo in readme

Changes and Fixes

Fix typo in readme

Thank you!

@samantha-wong Thank you!


Connecting to twitter instructions missing

I tried to connect to my twitter account but I'm missing some more instructions.
As I already had an up and running twitter app for another web application, I tried to reuse these consumer key and secret values.
At least it opens now a popup when I try to fiend friends. I accept the connection to that twitter app of myself and then it displays a numeric core to be entered ... where?
I thing I used that oauth stuff before but it never provided me with only a code but just redirected back to the website which wanted to connect. I'm sure missing something.

Oh good, I'm glad you also posted this here. We need to add instructions for setting up OAuth integration with Twitter.
This will be part of #811, since self-hosting users will need to also connect all of these services. Maybe write a INTEGRATIONS.md readme file in the top-level?
Here's the thread: https://forum.newsblur.com/t/initial-setup-of-twitter-connect-for-self-hosted-instance/9308/6


Feature/adding logging points for 3DS2

Summary
Making preparations for use of the new analytics endpoint within the 3DS2 process:

Introducing a greater level of granularity when detecting errors.
Identifying points at which logs/errors should be sent out

Tested scenarios

Fixed issue:

We in PVH, faced 3DS2 popup overlay issue last Feb.
Some of shoppers, during 3DS2 journey, were seeing blank-page. It affected our conversion rate to great extent.
Since then we're chasing Adyen for help.
Though we disabled 3ds2-popup since then, but shoppers' user-experience is not that great, when they gets redirected to not-so-great adyen-hosted page instead for native 3DS2 popup. Currently its disabled for all (more than 50) online stores.
We are looking forward to reintroduce popup on trial basis, so at least before next black-friday, we will have it enabled.
Since this logging is not in place, we cant even an experiment on live for small set of shoppers to get to the bottom of root-cause and fix it.
Can you please prioritize this and let us know when this logging can be enabled on live?

We in PVH, faced 3DS2 popup overlay issue last Feb. Some of shoppers, during 3DS2 journey, were seeing blank-page. It affected our conversion rate to great extent. Since then we're chasing Adyen for help.
Though we disabled 3ds2-popup since then, but shoppers' user-experience is not that great, when they gets redirected to not-so-great adyen-hosted page instead for native 3DS2 popup. Currently its disabled for all (more than 50) online stores.
We are looking forward to reintroduce popup on trial basis, so at least before next black-friday, we will have it enabled.
Since this logging is not in place, we cant even an experiment on live for small set of shoppers to get to the bottom of root-cause and fix it.
Can you please prioritize this and let us know when this logging can be enabled on live?

Hi @yogi-pvh What was the issue that you were facing in Feb? Did you try the recent adyen-web version, it could be that the issue is fixed. If not, feel free to reach out our support team or create an issue in the Github with steps to reproduce.
About the Analytics, there is a recent PR #2234 which is under review and we are also waiting for the backend to finalize some technical improvements. This analytics will be used internally, I don't know how it would assist you to analyze the root cause.


Review required vs. notBlank validator constraints

Do we think there's a real-world use-case for the required validator allowing empty string?
This is somewhat informed by the fact that one or more of our inputs appear to take a null value to "" when tabbing through or setting then manually clearing the value, but in general asking if the distinction is helpful.
GORM has allowBlank but that's in contrast with the more explicit nullable constraint - not sure if that informed the current setup or not. In our case with the name required I think we could be more general but writing up this ticket to discuss. 🤔

I think you are right w.r.t to emptyString being rejected by 'required'.
But what about a couple of empty 'spaces'?  Note that notBlank is actually doing a trim() before rejecting.   I guess simplest would be to fail that as well

Yes I think so - can't think of any valid use case for accepting "    ", only possible bug reports 😄
A user who really didn't want to type something but still wants that form to be valid can go with "xxx"


The background of the textarea grabber (-webkit-resizer) changes when the scroll bar appears

That's basically it, reproduced in Chrome and Safari (both light and dark themes), does not happen in Firefox.

Can you explain it bit more.

@pm-patel12 see the screenshots and notice how the resize handler in the bottom right corner changes color according to the presence of the scrollbar. Are you able to reproduce it?

No actually I think the expected behaviour is to not change the background at all, namely, the resize handler in the bottom screenshot should be the same as the top one.

ohh , your mean whan scroll bar appears than resize handler should be same as before , right ?

s

yep got it , i can do it. Check out screen shot below
and tell me if there is any chances
.
Before

.
After

Now chack it i Have make same as you say sir , can i make pull request ?

I don't see the bug in your screenshots, but that doesn't look like to be the Water.css style either...

Now chack it i Have make same as you say sir , can i make pull request ?

I'm not the owner of this repo.

Hello everyone! Is this issue still open?
I tweaked the styles of -webkit-resizer and -webkit-scrollbar-corner a bit, and these are the results
Empty Textarea - Light

Empty Textarea - Dark

With Scrollbar - Light

With Scrollbar - Dark

Open to all design suggestions and changes,
And thank you for this amazing library!

please assign me this issue. I want to start working on it immediately.


Improvements for Principal Component Analysis

A plain Principal Component Analysis algorithm was added in https://github.com/rust-ml/linfa/commit/7b6075e2dc9cc1c56ad7cd956bf996d69ce51d20. The next steps should improve upon edge-cases and features.

[ ] implement sparse PCA. By adding a sparsity constraint (like LASSO) only certain principal components are selected to represent the data
[ ] implement robust PCA to improve robustness to outliers
[ ] (?) implement non-linear PCA (should be similar to diffusion maps except for scaling)
[ ] add tests for edge-cases for very large, sparse or ill-behaving datasets

sparse PCA depends on #46

It seems like some Sparse/Robust PCA tests use Yale face dataset. It might be too big for linfa-datasets though (as it is a lot of image data), but I think it's way too nice of a "real-world" example to let this one slip away. Maybe we could have a separate repository for this test?

Perhaps, you can take a look at the mnist crate and implement something similar for the face dataset? There is also an open issue here to replace the downloader: https://github.com/davidMcneil/mnist/pull/8


Support for SSL certificates in keystore

Feature Request:
This docker image has added instructions and support for using your own ssl certificates in a Tomcat image.
https://hub.docker.com/r/chrisipa/tomcat/
https://hub.docker.com/r/chrisipa/tomcat/~/dockerfile/
I think it would be very beneficial for the official tomcat library to include support for SSL/TLS certificates as well.

Sorry for the really long tail here :disappointed:
We definitely "support" TLS certificates here, as much as Tomcat itself does.  The downside is that we require you to configure Tomcat to use them appropriately yourself.  Unfortunately, Tomcat's configuration is XML-based, so doing automated changes to it is more complicated than it really should be, and we try to provide a "vanilla" Tomcat experience as much as possible.


resolv.conf: nameserver gone regression

On my VMs resolv.conf does not properly generate the nameserver entry anymore. Works with 15.1.7, but not 15.1.8.

Related to #47 switching tickets....


Header fix for RN 0.40

Update for
https://github.com/facebook/react-native/commit/e1577df1fd70049ce7f288f91f6e2b18d512ff4d
yet?

I can see that a fix has been merged. Any plans on when a new release will be available with the fix?

You must update the version in package.json.
Publish the package again
You Will update the package on NPM

@KinanKazmi

npm remove react-native-pdf-view --save

npm install https://github.com/cnjon/react-native-pdf-view.git --save

maybe you can try this first

I created a PR that dumps the version, so that we can get a new release that supports RN >= 0.40

Any word on this? Just downloaded Version 0.3.2 from master.


Cannot find module 'babel-runtime/helpers/typeof'

Greetings,
I tried to install chromy, but got some warnings during the process. When I try to require('chromy') I'm getting the error Cannot find module 'babel-runtime/helpers/typeof'. Is babel-runtime a dependency?
Here's the output of the npm install and the error...
csears$ npm install chromy
><EMAIL_ADDRESS>install /Users/csears/Code/node_modules/chromy/node_modules/sharp
> node-gyp rebuild

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

xcode-select: error: tool 'xcodebuild' requires Xcode, but active developer directory '/Library/Developer/CommandLineTools' is a command line tools instance

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
  TOUCH Release/obj.target/libvips-cpp.stamp
  CXX(target) Release/obj.target/sharp/src/common.o
  CXX(target) Release/obj.target/sharp/src/metadata.o
  CXX(target) Release/obj.target/sharp/src/operations.o
  CXX(target) Release/obj.target/sharp/src/pipeline.o
  CXX(target) Release/obj.target/sharp/src/sharp.o
  CXX(target) Release/obj.target/sharp/src/utilities.o
  SOLINK_MODULE(target) Release/sharp.node
  TOUCH Release/obj.target/win_copy_dlls.stamp
<EMAIL_ADDRESS>../../../node_modules/chromy
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>├──<EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
csears$ node
> const Chromy = require('chromy')
Error: Cannot find module 'babel-runtime/helpers/typeof'
    at Function.Module._resolveFilename (module.js:325:15)
    at Function.Module._load (module.js:276:25)
    at Module.require (module.js:353:17)
    at require (internal/module.js:12:17)
    at Object.<anonymous> (/Users/csears/Code/node_modules/chromy/dist/index.js:3:16)
    at Module._compile (module.js:409:26)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Module.require (module.js:353:17)
>

Thanks!

Hi @csears, thank you for reporting!
Could you tell me your node version and OS?

You bet. Node is v4.5.0. OS is MacOS Sierra 10.12.5.

I tried updating node to v8.4.0. That seemed to fix the Cannot find module 'babel-runtime/helpers/typeof' error, but it left me with a different error.
I suspected I may have multiple out-of-date node modules, so I decided to try installing Chromy with yarn instead...
yarn add chromy
That seems to have fixed my issue.
I would say this is not a Chromy problem, so I'm closing the issue. Sorry for the false alarm.


Molecule Ring-Based Lifting (Graph to Cell)

Based on Jin et al. (2018), this implementation aims to condense a molecule ring into a single element. Utilizing the QM9 dataset, a benchmark dataset for molecule prediction, we develop a method to treat rings as 2-cell structures by identifying them through the provided SMILES representations of each molecule.
Here, the elements are the following:

Nodes: Atoms in the molecule.
Edges: Bonds between atoms.
2-cells: Rings in the molecule.

Additionally, attributes inspired by those used in (Battiloro et al., 2024) are incorporated into the elements, enhancing the representation of the molecule.
The attributes are:

Node: Atom type, atomic number, and chirality.
Edge: Bond type, conjugation and stereochemistry.
Rings: Ring size, aromaticity, heteroatoms, saturation, hydrophobicity, electrophilicity, nucleophilicity, and polarity.

This pull request is done under the team formed by: Bertran Miquel Oliver, Manel Gil Sorribes, Alexis Molina.

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

Please upload report for BASE (main@497947b). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage        ?   69.13%
=======================================
  Files           ?       17
  Lines           ?      554
  Branches        ?        0
=======================================
  Hits            ?      383
  Misses          ?      171
  Partials        ?        0

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

Hello @bertranMiquel! Thank you for your submission. As we near the end of the challenge, I am collecting participant info for the purpose of selecting and announcing winners. Please email me (or have one member of your team email me) at<EMAIL_ADDRESS>so I can share access to the voting form. In your email, please include:

your first and last name (as well as any other team members)
the title of the method you implemented
the input domain of the method you implemented
the output domain of the method you implemented
your pull request number (#14)

Before July 12, make sure that your submission respects all Submission Requirements laid out on the challenge page. Any submission that fails to meet this criteria will be automatically disqualified.

Hi Guillermo,
Answering to fulfill the submission requirements:

Name and last name of the participants: Bertran Miquel-Oliver, Manel
Gil-Sorribes, Alexis Molina, Victor Guallar (all of them in cc)
Title: Molecule Ring-Based Lifting
Input domain: Graph
Output domain: Cell Complex
Pull request: #14
https://github.com/pyt-team/challenge-icml-2024/pull/14

Best,
Bertran
Missatge de Guillermo Bernárdez @.***> del dia dl., 8
de jul. 2024 a les 19:11:

Hello @bertranMiquel https://github.com/bertranMiquel! Thank you for
your submission. As we near the end of the challenge, I am collecting
participant info for the purpose of selecting and announcing winners.
Please email me (or have one member of your team email me) at
@.*** so I can share access to the voting form. In
your email, please include:

your first and last name (as well as any other team members)
the title of the method you implemented
the input domain of the method you implemented
the output domain of the method you implemented
your pull request number (#14
https://github.com/pyt-team/challenge-icml-2024/pull/14)

Before July 12, make sure that your submission respects all Submission
Requirements laid out on the challenge page
https://pyt-team.github.io/packs/challenge.html. Any submission that
fails to meet this criteria will be automatically disqualified.
—
Reply to this email directly, view it on GitHub
https://github.com/pyt-team/challenge-icml-2024/pull/14#issuecomment-2214740528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHTIUTRANFPUQS3OOJSPPG3ZLLB33AVCNFSM6AAAAABJJ7P3T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJUG42DANJSHA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Reload on NDB will cause address in use on debugging applications serving ports

I'm using ndb v1.0.37, and locally have Chrome Version 70.0.3538.110 (Official Build) (64-bit).
Trying to use ndb to attach to a local copy of Ranvier mud.
https://github.com/RanvierMUD/ranviermud
I start it with ndb ./ranvier
Which works fine. However, when I try and CMD + R in ndb to restart the server, it throws
 unhandledRejection Execution context was destroyed, most likely because of a navigation.
And the server restarts, but it never seems to attache after that until I totally kill ndb.  Any thoughts?

Okay, I'm pretty sure this issue is actually the same as this one.
#77
I'm running node version v11.1.0
<EMAIL_ADDRESS>OSX 10.12.6 Sierra
And the application is running a telnet server on port 4000.

I checked<EMAIL_ADDRESS>with ranvier and it works just fine to me. It looks fixes, feel free to comment this issue or open new one if it is still reproducible for you.


Segmentation fault on Raspberry Pi Zero W

Specifications

Image hash or version: 1.25.0
Platform: Raspberry Pi Zero W (armv6l)
Subsystem:

Expected Behavior
Planefinder and opensky feeder start successfully
Actual Behavior
Segmentation faults appear in console when either opensky or planefinder feeder is enabled
./run: line 21:   146 Segmentation fault      (core dumped) /planefinder/pfclient --connection_type=1 --address="<IP_ADDRESS>" --port="30005" --data_format=1 --sharecode="${PLANEFINDER_SHARECODE}" --lat="${HTML_SITE_LAT}" --lon="${HTML_SITE_LON}" --pid_file=/run/pfclient.pid --config_path=/planefinder/config/pfclient-config.json --log_path=/planefinder/log 2>&1
       147 Done                    | mawk -W interactive '{printf "%c[36m[planefinder]%c[0m %s\n", 27, 27, $0}'

./run: line 9:   145 Segmentation fault      (core dumped) /usr/bin/openskyd-dump1090 2>&1
       146 Done                    | mawk -W interactive '{printf "%c[34m[opensky-feeder]%c[0m %s\n", 27, 27, $0}'

I am not sure if this is expected on this platform (Raspberry Pi Zero W). It's probably underpowered and I'm not sure if these feeders are even supposed to run on RPi Zero. I'm running all other feeders on it no problem though.
Steps to Reproduce the Problem

Enable either the opensky or planefinder feeder

It might probably what you say. I have a Pi Zero W, and I'm only running dump1090 on it, pf was failing so I started the other service on another computer (amd64).
You can try to disable the service and only run /planefinder/pfclient --version manually. If it fails we'll amost be sure we cannot do anything about it.

You're right, it also displays the Segmentation fault, so I guess these feeders do not run on a Zero.
I've setup the Pi to only get the dump1090 and to feed that to another server that does the feeding using your example config here.
Thanks for making such a versatile and easy all in one image!


Mixed JUnit4 and JUnit5 project results in duplicate targets

My demo project shows that when your project contains JUnit4 and JUnit5 tests, duplicate targets are exported in the BUILD.bazel files.

@D0rmouse  Thanks so much for the repro!
I will run it and think what can be done


Update scalafmt-core to 3.7.17

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.7.16 to 3.7.17
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

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

Comparison is base (4ad9d0a) 56.80% compared to head (719bc45) 56.80%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #431   +/-   ##
=======================================
  Coverage   56.80%   56.80%
=======================================
  Files          35       35
  Lines        1750     1750
  Branches      233      233
=======================================
  Hits          994      994
  Misses        756      756

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


Private messages sounds are not implemented yet.

Do you know which sounds are played when a PM arrives? It would be nice if you could add that to the Index of Sounds

@SungMatt Same as raindrops

Cheers, @TiTo-418 .

Should be fixed with #926.

implemented in 1e380db


Incorrect Spelling in Jeremiah 31:15

It is the correct spelling for KJV. See https://www.biblegateway.com/passage/?search=Jeremiah+31%3A15&version=KJV


TypeError on yargs when using concurrently to start services

Hello, we're running into an intermittent issue where we get a
TypeError: shim$1.Parser.lookslikeNumber is not a function error when trying to start up our services with: concurrently "npm run local-server" "npm run l10n-watch". It looks like its coming from yargs.
We currently need to npm uninstall concurrently && npm install concurrently to "fix" it
The stacktrace:
/app-repo/node_modules/concurrently/node_modules/yargs/build/index.cjs:2804
        if (shim$1.Parser.lookslikeNumber(arg) &&
                          ^

TypeError: shim$1.Parser.lookslikeNumber is not a function
    at Object.Yargs.self._parsePositionalNumbers (app-repo/node_modules/concurrently/node_modules/yargs/build/index.cjs:2804:31)
    at Object.Yargs.self._postProcess (app-repo/node_modules/concurrently/node_modules/yargs/build/index.cjs:2787:25)
    at Object.parseArgs [as _parseArgs] (app-repo/node_modules/concurrently/node_modules/yargs/build/index.cjs:2774:21)
    at Object.get [as argv]] (app-repo/node_modules/concurrently/node_modules/yargs/build/index.cjs:2651:25)
    at Object.<anonymous> (app-repo/node_modules/concurrently/dist/bin/concurrently.js:170:5)

Version<EMAIL_ADDRESS>

Said new release is out now -- do you want to give it a try and report back, @jchli?

Closing as I assume this has been fixed  Please open a new issue if the problem still exists

I'm having the same problem with yargs >= 17.6.1, the last version that works is 17.6.0.


handle deleted taxonomy elements in datamap export

‼ This should be hotfixed for 2.12.1
Closes https://github.com/ethyca/fidesplus/issues/858
Code Changes

treat defunct data_use or data_subject references from a privacy (data use) declaration as empty rows in the datamap export
ensure at least one "placeholder" system row is returned if no rows are produced from declarations

Steps to Confirm

requires integration with fidesplus to properly test (will be a corresponding PR there with increased test coverage)

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[x] Relevant Follow-Up Issues Created
[x] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

Description Of Changes
A broader improvement around taxonomy element deletion is covered in https://github.com/ethyca/fides/issues/3119.

@SteveDMurphy or @ThomasLaPiana, my datamap friends...would you mind taking a look at this quick bugfix to handle an edge case that we can hit since our handling of taxonomy element deletion is still a bit immature throughout the application?
@pattisdr FYI, since you found the bug :) if you want to dig into the datamap export code, please be my guest and i would be very happy to get your review! @Kelsey-Ethyca also want to loop you in for release logistics


macos automatic installation fails

Hi,
On Macos Mojave the installation instructions lead to an error. Namely, when sudo sh ./kextload.sh calls cp -R kextbuild/Release/PcmMsrDriver.kext /tmp/. the file PcmMsrDriver.kext is not found.
When I run make kext, the file gets build. I believe the issue is that when calling make kext library, the build step for library overwrites the generated kext file - I was able to install pcm manually when I backed the kext file before running make library.
I think that a sufficient fix would be to let xcodebuild operate in different directories.
Cheers,
Pavel

thanks for reporting this. We will address it soon

this has been addressed in the latest version of the master branch. Closing the issue.

Thank you!


chore: include nodejs benchmarks

Moved the existing factors.js and loop.js to use process.hrtime for high-resolution time measurements. It's been in Node since 0.x so is safe to assume all node users have it in their current Node version(s).
Also fixed loop.js, which had sum and list variable references that were not declared. This negatively impacts performance & is very heavily discouraged across the language.

Related #124

Thanks! I was tempted to replace the benchmarks.py file with a Node script, but that just shifts the dependency from one to another haha. Could probably be done with a Makefile since benchmarks self-report elapsed time


Autoedits Context Improvements

Context
The PR makes the following changes:

Removes the jaccard-similarity and recent-copy retriever from data logging and auto-edits, which is not getting used. These increases the context a lot without much utility. Since, jaccard-similarity does not prove to be valuable as per offline eval, Although recent-copy is useful information, its volume is <2%.
Since, auto-edits retriever are mostly dependent on user actions in the editor, Added a field timeSinceActionMs to track the age of context item. We log this field in our logging pipeline to experiment offline, to see if a different prompt rendering could work better. (Offline eval script)
Add a context ranker strategy to use timeSinceActionMs for sorting for autoedits, which was earlier using default RRF and messing up the order of context items.
Fix recent-view-port retriever, when the file was not tracked if we simply opened the file without any scrolling event.
Fixed the issue where previous if one of the context snippet was big, than other context snippets were not used in the prompt.
Refactor recent-edits-retriever to add add different diff algorithms. Currently the model suggests frequently to delete the most recent added line. Hypothesis is that adding the most recent line separately at the end of the prompt, which help model pay attention to recent short term changes.

Test plan
Added CI tests

To support our ongoing focus on improving release stability and ensuring more rigorous PR reviews, I suggest dividing PRs into smaller, targeted updates whenever feasible.
This strategy offers key advantages: if a problem occurs, we can revert just the affected change rather than a whole PR. This would ensure that your other changes stay in the release candidate, and we can get most of the required telemetry data despite addressing a potential release blocker. Also, it simplifies the review process significantly.
Your PR description already mentions several areas affected by this PR. They are pretty independent and can be updated in parallel PRs.

Sure @valerybugakov That totally makes sense !!
Would it be possible to review the current PR as is, I will keep in mind to create smaller PRs going forward, else I would have to redo all the changes. Also, since the current PR is for auto-edits, it shouldn't affect any prod features, and the majority of the diff is because of this file where I shifted the whole code under describe to the right, which created this big diff.