Installation fails on Ubuntu 23.10 failing on libgl1-mesa-glx

installation failing on Ubuntu 23.10 at step where it tries to install libgl1-mesa-glx
$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 23.10
Release:        23.10
Codename:       mantic

+ for i in "${REQUIRED_PKG[@]}"
++ dpkg-query -W '-f=${Status}' libgl1-mesa-glx
++ grep -c 'ok installed'
+ '[' 0 -eq 0 ']'
+ apt install --no-install-recommends -y libgl1-mesa-glx
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Package libgl1-mesa-glx is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source

E: Package 'libgl1-mesa-glx' has no installation candidate

Looks like this issue is my host is running Minimal Ubuntu

I was able to get passed this step
diff --git a/install-librephotos.sh b/install-librephotos.sh
index f2a90dd..a8298c0 100755
--- a/install-librephotos.sh
+++ b/install-librephotos.sh
@@ -86,7 +86,7 @@ fi
 # LIBREPHOTOS : BACKEND

 REQUIRED_PKG=( wget swig ffmpeg libimage-exiftool-perl libpq-dev  curl libopenblas-dev libmagic1 libboost-all-dev libxrender-dev \
-liblapack-dev git bzip2 cmake build-essential libsm6 libglib2.0-0 libgl1-mesa-glx gfortran gunicorn \
+liblapack-dev git bzip2 cmake build-essential libsm6 libglib2.0-0 libgl1 libglx-mesa0 gfortran gunicorn \
 libheif-dev libssl-dev rustc liblzma-dev python3 python3-pip imagemagick )
 for i in "${REQUIRED_PKG[@]}"; do
 [ $(dpkg-query -W -f='${Status}' $i 2>/dev/null | grep -c "ok installed") -eq 0 ] && apt install --no-install-recommends -y $i


Fixed Pumpkin website links

Description
Changed the links in the README.md to link to the actual Pumpkin Website url (https://snowiiii.github.io/Pumpkin/ -> https://snowiiii.github.io/Pumpkin-Website/)
Testing
Click the links and they dont lead to a 404 page.
Checklist
Things need to be done before this Pull Request can be merged.

[x] Code is well-formatted and adheres to project style guidelines: cargo fmt
[x] Code does not produce any clippy warnings: cargo clippy
[x] All unit tests pass: cargo test
[ ] I added new unit tests, so other people don't accidentally break my code by changing other parts of the codebase. How?

Thanks @LieOnLion, I plan to get a domain soon


SCons: Explicitly define our C (C11) and C++ (C++14) standards

On GCC and Clang, we use C11 and C++14 with GNU extensions (std=gnu11
and std=gnu++14). Those are the defaults for current GCC and Clang,
and also match the feature sets we want to use in Godot.
On MSVC, we require C++14 support explicitly with /std:c++14, and
make it strict with the use of /permissive- (so features of C++17 or
later can't be used).
Moves the definition before querying environment flags and platform
config so that it can be overridden when necessary.

I think this could be cherry-picked for 3.2, since even though we don't formally use C++14 there yet, we're currently defining -std=c++11, which is more restrictive than the GCC and Clang default of -std=gnu++14 (and thus more restrictive than 3.1 which doesn't specify anything).

Note to self if I cherry-pick for 3.2: the logic was moved too far up and doesn't work for MSVC, #36463 should also be included (without the C++17 part).

Cherry-picked for 3.2.1. (+ removal of Android gnu++14 from #36682).


Feature/momspacebhchain

Momentum space Bose Hubbard chain
This PR adds the new Hamiltonian BoseHubbardReal1D and functionality around it. Other minor changes that should be irrelevant for script are renaming LinearOperator to AbstractHamiltonian and a change in nearUniform to make things more consistent.

The CI succeeded just by re-running. It must have been some glitch preventing the installation of MPI when they failed.


Require linked-hash-map v0.5.3 (or a newer patch version)

Hi @stusmall,
I found an issue with the linked_hash_map dependency on <= v0.5.2.  linked-hash-map can panic due to the UB checks implemented in Rust 1.48 (https://github.com/rust-lang/rust/issues/66151, as it uses std::mem::uninitialized:
v0.5.3 fixes the issue (https://github.com/contain-rs/linked-hash-map/pull/100).  This PR adds a requirement on v0.5.3 (or a newer patch version).

If a library consuming ttl_cache as is runs cargo update they will pick up the 0.5.3 version of linked-hash-map. If they aren't updating their dependencies then any change here won't help them anyways.  Ultimately updating to the latest patch version of a dependency should be the responsibility of the final consumer of the library, and not intermediate libraries in the dependency graph.
Tell every at ThreatX I said hi :)


refactor(workflow): configure jest to directly check coverage to schlanken the test workflows

🍰 Pullrequest
We can configure jest to directly check coverage at the end of unit testing.
This makes the separate coverage check steps in our workflows obsolete.
Todo

[X] add coveragaThreshold to all jest.condig.js
[X] remove separate coverage check steps from github test wirkflows
[ ] remove the coverage checks from the branch protection, @ulfgebhardt

Remove the --coverage option in all package files

Done in https://github.com/gradido/gradido/pull/2790/commits/c2de11e399aaae66824c26c9b6cee16e1a6387b8


[SPARK-10177] [SQL] fix reading Timestamp in parquet from Hive

We misunderstood the Julian days and nanoseconds of the day in parquet (as TimestampType) from Hive/Impala, they are overlapped, so can't be added together directly.
In order to avoid the confusing rounding when do the converting, we use 2440588 as the Julian Day of epoch of unix timestamp (which should be 2440587.5).

Merged build triggered.

Merged build started.

Test build #41466 has started for   PR 8400 at commit e96f92f.

Merged build triggered.

Merged build started.

Test build #41474 has started for   PR 8400 at commit 9b1b9ce.

Test build #41466 has finished for   PR 8400 at commit e96f92f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41466/
Test PASSed.

Test build #41474 has finished for   PR 8400 at commit 9b1b9ce.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41474/
Test PASSed.

Test build #1688 has started for   PR 8400 at commit 9b1b9ce.

Test build #1688 has finished for   PR 8400 at commit 9b1b9ce.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Can you explain what "they are overlapped" means?

I feel like Impala and Hive did Julian day conversion in the wrong way at first, but left it as is and made the wrong conversion logic the de facto "standard" way? I don't see an intuitive reason why we should store a Julian timestamp 12 hr later than the real Julian timestamp.

Merged build triggered.

Merged build started.

@yhuai The Julian day of epoch of unix timestamp (1970-01-01 0:0:0) is 2440587.5, if we use integer for julian days, then it will be cut off at noon (12pm) of a day. But the nanoseconds is counted from 12am, so they are overlapped from 12am to 12pm. In Hive, it create a calendar from Julian day (which will have hours as 12pm), it then re-set the hours from nanoseconds.
In this patch, we use a trick to shift Julian days by 0.5 day then rounding 0.5 down to zero (In Hive, it rounds 0.5 up to 1), then we can add these two parts together.

Test build #41520 has started for   PR 8400 at commit 05437e1.

LGTM now

Test build #41520 has finished for   PR 8400 at commit 05437e1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merging to master and branch-1.5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41520/
Test PASSed.

Just would like to point out that our original DateTimeUtils doesn't do Julian date conversion right either. Here is a spark-shell snippet executed against https://github.com/apache/spark/commit/053d94fcf32268369b5a40837271f15d6af41aa4:
import java.sql._
import java.util._

import org.apache.hadoop.hive.ql.io.parquet.timestamp._
import org.apache.spark.sql.catalyst.util._

TimeZone.setDefault(TimeZone.getTimeZone("GMT"))
val timestamp = Timestamp.valueOf("1970-01-01 00:00:00")

val hiveNanoTime = NanoTimeUtils.getNanoTime(timestamp, false)
val hiveJulianDay = hiveNanoTime.getJulianDay
val hiveTimeOfDayNanos = hiveNanoTime.getTimeOfDayNanos

println(
  s"""Hive converts "$timestamp" to Julian timestamp:
     |(julianDay=$hiveJulianDay, timeOfDayNanos=$hiveTimeOfDayNanos)
   """.stripMargin)

val (sparkJulianDay, sparkTimeOfDayNanos) =
  DateTimeUtils.toJulianDay(DateTimeUtils.fromJavaTimestamp(timestamp))
println(
  s"""Spark converts "$timestamp" to Julian timestamp:
     |(julianDay=$sparkJulianDay, timeOfDayNanos=$sparkTimeOfDayNanos)
   """.stripMargin)

The result is:
Hive converts "1970-01-01 00:00:00.0" to Julian timestamp:
(julianDay=2440588, timeOfDayNanos=0)

Spark converts "1970-01-01 00:00:00.0" to Julian timestamp:
(julianDay=2440588, timeOfDayNanos=43200000000000)

while the correct answer should be:
(julianDay=2440587, timeOfDayNanos=43200000000000)

So Hive is 12 hours later than the expected Julian timestamp, while we were 24 hours later :)

Asked for more background about Hive's 12 hr offset on [HIVE-6394] 1, which is the original JIRA ticket for introducing timestamp support for ParquetHiveSerDe.

@liancheng Thanks for the comments, but you may made a mistake somewhere, I got this WITHOUT this PR:
scala> TimeZone.setDefault(TimeZone.getTimeZone("GMT"))
scala> val timestamp = Timestamp.valueOf("1970-01-01 00:00:00")
timestamp: java.sql.Timestamp = 1970-01-01 00:00:00.0

scala> val (sparkJulianDay, sparkTimeOfDayNanos) =
     |   DateTimeUtils.toJulianDay(DateTimeUtils.fromJavaTimestamp(timestamp))
sparkJulianDay: Int = 2440587
sparkTimeOfDayNanos: Long =<PHONE_NUMBER>0000

scala> println(
     |   s"""Spark converts "$timestamp" to Julian timestamp:
     |      |(julianDay=$sparkJulianDay, timeOfDayNanos=$sparkTimeOfDayNanos)
     |    """.stripMargin)
Spark converts "1970-01-01 00:00:00.0" to Julian timestamp:
(julianDay=2440587, timeOfDayNanos=43200000000000)

The result you posted for Spark is totally wrong (you can't get the timestamp back after one round trip), fortunately this is a fake alarm :)
For the Hive one, it bad but not wrong (different rounding method, you can still read the timestamp back).
In general, maybe using Julian day here is the root of bad things, use signed integer for number of days since epoch should be enough (could be negative).

@davies Yeah, you're right. Verified that I was actually running the posted snippet against a local revision which I used to debug the Julian conversion logic. Forgot to rebuild after checkout out the code back to the earlier master revision.
Anyway, behavior of the current master branch is consistent with Hive.


Putting together a tutorial using some standard data sets from the literature?

Description
I've been experimenting with GluonTS and it has been a real pleasure working with this framework. I've tried it on a few text book time series (Air Passengers, Car Sales, Australian Tourism data) - and I was wondering if there was any benefit to putting that all together in a tutorial, especially with the Australian Tourism data set, since it has become almost a canonical example of grouped times series that can be approached via hierarchical forecasting. The tutorial would demonstrate how to use the hierarchy information as static features for the estimator.
Do you think there is any benefit to such a tutorial or would it be redundant with the quick start and extended tutorials? Or if you think any other data sets would be better for such a tutorial, I'de be happy to work on contribute that as well.
References

Air Passengers data: Box and Jenkins (1976): Times Series Analysis: Forecasting and Control
Australian tourism data: Athanasopoulos, G., & Hyndman, R. J. (2008). Modelling and forecasting Australian domestic tourism. Tourism Management, 29.

I believe that is a very nice idea to illustrate the functionality of GluonTS. Let me add that we should have a notebook for each estimator available, e.g., DeepAR, DeepGP, DeepState, DeepFactor. I have been using the Australian tourism data to test some of these models. I can volunteer and share them so as to use them as a working template.

It is a very good idea, it would be very nice to write wrapper so that we can access those datasets with:
from gluonts.dataset.repository.datasets import get_dataset
dataset = get_dataset("tourism")
# ... train your favorite estimator

We currently added m4 datasets and a bunch of others in this wrapper, this would make it easily available for benchmarks.

@StatMixedML I'm curious if you would share how you take the original Australian tourism CSV and parse it into a GluonTS data set format?
I myself have using some very hacky and not at all elegant Pandas manipulations, along the lines of:

Load csv into Pandas
splint into two data frames, one with static features and one with time series targets.
merge them back into a list of dictionaries
Transform the list of dictionaries into a GluonTS dataset object.

I'm wondering if there is a more straightforward and elegant way to do so?

@SkanderHn I hope that I can share the notebook some time next week, didn`t have time during last week.
@geoalgo What is the best way of sharing notebooks? Via PR?

@StatMixedML Any progress with the notebooks?


tag:with toolshed command

About the PR

Adds the toolshed command tag:with that filters a list of entities to only those with the given tag.
Why / Balance

This was requested by a few admins. In general, working with tags as a admin is
a massive pain.
Technical details

New toolshed command. It just does a LINQ filter in the input.
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: Aquif
ADMIN:

add: tag:with toolshed command. It takes in a list of entities via pipe and returns the entities from the list with the given tag.

That might be an issue with ValueRef parsing, which is blissfully nuked in https://github.com/space-wizards/RobustToolbox/pull/5455, but that's not been merged in so far.
Ideally the command should allow you to have these values WITHOUT the quotes for consistency. But seeing as that's already an issue in tag:add, not much to do here.


Firelocks can be built on top of each other

Description

You can stack an infinite amount of firelocks in 1 tile by just building them. This can make passing through areas near impossible if it's spaced.
Reproduction

Build firelock.
Place new construction ghost on firelock.
Build second firelock on top of the first.
You now have a stacked firelock.

Reproducible, the same can be done with airlocks if you build the second one while the first is opened. Notably only one airlock can be closed at a time.

Likewise only one firelock can be closed at a time. While spacing will initially trigger multiple firelocks to successfully close, once they are opened, again only one firelock can be closed at a time.


feat(github-actions): add support for GHES hosted repos

Hey, I just made a Pull Request!

Addresses TODOs in GitHub Actions plugin to resolve the repository's hostname from entity annotations. This enables the GitHub Actions plugin to work with self-hosted GitHub instances on domains other than github.com.
:heavy_check_mark: Checklist

[X] A changeset describing the change and affected packages. (more info)
[X] Added or updated documentation
[X] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

It's not clear to me if the error in E2E Windows 18.x workflow is related to my changes. Any help/feedback would be greatly appreciated.

Thanks for the review @jhaals! I believe the other GitHub plugins are lacking support for GHES hosted repositories. I was planning to open separate PRs for each plugin or I can lump them all together in this PR. Do you have a preference?

Thanks for the review @jhaals! I believe the other GitHub plugins are lacking support for GHES hosted repositories. I was planning to open separate PRs for each plugin or I can lump them all together in this PR. Do you have a preference?

Let's take this per plugin but I would be curious to know the other plugins/places that also lack the same functionality as there might be room for some shared utility that we might be missing.


Include RSA as a dependency

Sorry for any inconveniences
Fixed from https://github.com/openpesa/php-pesa/releases/tag/v0.1.0


add code coverage support

Codecov Report

:exclamation: No coverage uploaded for pull request base (trunk@f30dc87). Click here to learn what that means.
The diff coverage is 0%.

@@           Coverage Diff            @@
##             trunk     #243   +/-   ##
========================================
  Coverage         ?   58.15%
========================================
  Files            ?      101
  Lines            ?     4354
  Branches         ?      652
========================================
  Hits             ?     2532
  Misses           ?     1580
  Partials         ?      242


kapacitord ignores commands following the -config flag

kapacitord ignores command line verbs like help, version, and downgrade if a configuration file is passed with the -config flag.  This means that the downgrade flag cannot be used for any kapacitor.db file that is not in the default location.

This appears to be the case when the flags is before the command on the command line. It may not be the case when the command is first, e.g. kapacitord downgrade -config <path>.  Testing is underway.


Client.postMessage should return a Promise

Client.postMessage can fail for various reasons, for example:

user or JS closed the tab
user or JS navigated* the tab to a non-same-origin URL
tab is still open, but browser evicted it from RAM

As suggested in https://github.com/slightlyoff/ServiceWorker/issues/609#issuecomment-74920652 and https://github.com/slightlyoff/ServiceWorker/issues/626#issuecomment-74918907, it would be useful if Client.postMessage returned a Promise that would be resolved once the message event has been fired on the client, or rejected if the client ceases to exist before the message event can be fired.
Example use case from @jakearchibald:

After a push message, a SW may find a focused & visible client and post a message to it, but that fails because the tab has since been closed. Developers should be able to catch that case and show a notification instead.

@annevk is concerned that this would expose GC, but it remains unclear why it's bad for JS to be able to tell which tabs are still open/alive - especially since this can already be achieved by polling. So far, these concerns don't seem to outweigh the utility of the use case above.
*: some browsers have a back-forward cache (a.k.a. fastback) feature like https://wiki.mozilla.org/DocShell:Fastback - in such cases navigation doesn't destroy a client, it merely pauses all timers. In such cases where timers are paused, I guess the promise should neither resolve nor reject, but instead wait until the timers are resumed (e.g. because the user returns to the cached page) or the client ceases to exist (e.g. because the client is evicted from the back-forward cache).

*: some browsers have a back-forward cache (a.k.a. fastback) feature like https://wiki.mozilla.org/DocShell:Fastback - in such cases navigation doesn't destroy a client, it merely pauses all timers. In such cases where timers are paused, I guess the promise should neither resolve nor reject, but instead wait until the timers are resumed (e.g. because the user returns to the cached page) or the client ceases to exist (e.g. because the client is evicted from the back-forward cache).

As an implementer I'd rather that this just failed.

Apologies to have closed #609 without this issue being addressed.
So the use case is clear, but I'm a bit unclear about how to spec this bit:

tab is still open, but browser evicted it from RAM

even after having gone through #626. Some kind of "Note" will do for it?

Tracked in crbug.com/461416  but needs spec decision.

@jungkees if we expose evicted clients, I think they should be another type that doesn't have postMessage https://github.com/slightlyoff/ServiceWorker/issues/626#issuecomment-95207076

@annevk would it be wrong to make this return a promise given it doesn't elsewhere on the platform?

Setting this as v1, as we can't go from a method that throws to one that returns a promise, so this needs deciding soon.

How can you figure things out by polling without the other side cooperating?

I used postMessage in many ways for various multi-screen projects and experimentations : window.open, SharedWorker, BroadcastChannel, manual MessageChannel ports... It always felt like UDP to me and when I needed some kind of "TCP ACK", I had to build it myself.
I would prefer the postMessage API to behave the same as the rest of the platform. I see three possibilities :

Make SW postMessage return a promise and update other specs (there's a lot of them) to behave the same way.
Use a different function name in SW and this one would return a promise. IIRC, the presentation API renamed postMessage to something more like the WebRTC API.
Just do nothing. Stay low level and let devs build stuffs like this.

F2F: it isn't worth changing this one instance of postMessage. You can do this pretty easy in library land.

Yo: https://github.com/nolanlawson/promise-worker 😃
Not saying this solves every use case (e.g. it can't send ArrayBuffers or raw JSON), but I think it demonstrates this is indeed pretty easy to solve in userland.

Not dissimilar https://github.com/jakearchibald/svgomg/blob/master/src/js/page/worker-messenger.js


Is there any class imbalance problem in loss calculation ?

Thanks for your excellent work.
 I just wonder that for a sample A in batch, since "take a cross-entropy objective with in-batch negatives", take unsupervised scenario as an example, there is one positive sample and (batch size -1) negative samples ? So, will it raises  class imbalance problem ?
Correct me if I think wrong, thanks a lot.

Hi,
This is not a "classification" problem, and each time the positive sample is different, so there is no "imbalance problem".

Hi,
This is not a "classification" problem, and each time the positive sample is different, so there is no "imbalance problem".

Ok, thanks.  For a sample A in batch, take unsupervised scenario as an example, there is one positive sample and (batch size -1) negative samples, is this right ?

Yes that is correct.


[SPARK-45896][SQL] Construct ValidateExternalType with the correct expected type

What changes were proposed in this pull request?
When creating a serializer for a Map or Seq with an element of type Option, pass an expected type of Option  to ValidateExternalType rather than the Option's type argument.
Why are the changes needed?
 SQLSTATE: 42846
...
Caused by: java.lang.RuntimeException: scala.Some is not a valid external type for schema of array<int>
  at org.apache.spark.sql.catalyst.expressions.GeneratedClass$SpecificUnsafeProjection.MapObjects_0$(Unknown Source)
...

However, this code works in 3.3.3.
 SQLSTATE: 42846
...
Caused by: java.lang.RuntimeException: scala.Some is not a valid external type for schema of timestamp
...

As with the first example, this code works in 3.3.3.
SerializerBuildHelper#validateAndSerializeElement will construct ValidateExternalType with an expected type of the Option's type parameter. Therefore, for element types Option[Seq/Date/Timestamp/BigDecimal], ValidateExternalType will try to validate that the element is of the contained type (e.g., BigDecimal) rather than of type Option. Since the element type is of type Option, the validation fails.
Validation currently works by accident for element types Option[Map/<primitive-type], simply because in that case ValidateExternalType ignores that passed expected type and tries to validate based on the encoder's clsTag field (which, for the OptionEncoder, will be class Option).
Does this PR introduce any user-facing change?
Other than fixing the bug, no.
How was this patch tested?
New unit tests.
Was this patch authored or co-authored using generative AI tooling?
No.

Merged to master/3.5.
Could you make a backport for branch-3.4? There is a conflict on branch-3.4, @bersprockets .

Could you make a backport for branch-3.4? There is a conflict on branch-3.4, @bersprockets .

@dongjoon-hyun I will work on it shortly.

Thank you so much, @bersprockets !


Profile - NO cursors in Profile tabs

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the App
Login with any account
Go to Settings -> Profile -> Display name
Go to Settings -> Profile -> Legal name/Home address
Pay attention to the cursors in the input fields

Expected Result:
Сursors are present the in input fields on profile tabs
Actual Result:
NO cursors in the input fields on profile tabs
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/93399543/228913746-e26f426e-f0b9-41ab-8152-0e6104b93506.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @NicMendonca (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@kbecciv are these supposed to be supporting images?

Isn't this a cursor?

@kbecciv bump, please!

@NicMendonca Please create a new account and navigate to Profile. Select Display Name option.
User expected to see a cursor on First Name field.
https://user-images.githubusercontent.com/93399543/229921799-cc2107c5-7c31-4133-b79c-d5bbe70805e0.mp4

@kbecciv got it, thanks! Your OP says Login with any account so would be helpful to know this is happening to a new account only.

@NicMendonca It's easy reproduce with new account, but happening for both (new and existing accounts)

@kbecciv but I can't reproduce it with existing accounts -

assigning @Christinadobrzyn to watch over this while I am OOO!

asked about this here: https://expensify.slack.com/archives/C01SKUP7QR0/p1680717186187219

It sounds like this is expected. Maybe we can ask eng for a second opinion but I think we might want to close this without action.

Triggered auto assignment to @flodnv (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@flodnv, @NicMendonca, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick!

@flodnv, @NicMendonca, @Christinadobrzyn Whoops! This issue is 2 days overdue. Let's get this updated quick!

navigate to Profile. Select Display Name option. User expected to see a cursor on First Name field.

I disagree with this. You should see a cursor if you tap to focus into the field.
Please let me know if I misunderstood something.


Search and Count integration tests

In gitlab by @sfalquier on Apr 26, 2017, 15:40

In gitlab by @sfalquier on Apr 27, 2017, 09:02
closed

In gitlab by @sfalquier on Apr 26, 2017, 15:40
changed title from Search and Count inte{-r-}gration tests to Search and Count integration tests

In gitlab by @sfalquier on Apr 27, 2017, 09:02
removed ~1918368 label


Feature: Analysis shape viewer should be able to show individual subjects

This should behave similarly to Studio to maintain parity

This should look like the Samples tab in Studio:

Notable features:
options -

All Samples: renders all reconstructed samples in the dataset with particles. Should look and behave similarly to the standard shapeviewer.

Single Sample (with number field for sample #): Show single selected subject with particles.

Median: shows the median subject

Show difference to mean: shows the difference to mean. When All Samples is selected, shows a uniform scale. When Single Sample is selected, scale changes depending on the subject shown

Questions:

How is median sample calculated? Is this simply the median subject number?
How are the scales for the mean difference calculated?

How is median sample calculated? Is this simply the median subject number?

The median sample is the sample in the shape model with the smallest total particle to particle difference.  It's computed by taking each shape and for each shape, for each particle, adding up the particle to particle distance.

How are the scales for the mean difference calculated?

Usually just the min and max.  There are some cases in studio where it auto adjusts as you go through the data though, meaning the min is the min of all values seen so far and max likewise.


How to deal with this error?

/home/myshare/VIO_SLAM/src/snap_ros_examples/src/nodes/SnapdragonRosNodeVislam.cpp:160:13: error: ‘struct mvCPA_Configuration’ has no member named ‘startExposure’
cpaConfig.startExposure = param.camera_config.exposure;
I have tried many method, but it doesn't work. I have move the mvCPA.h to the /usr/include.

And there are many same errors, as follows:
/home/myshare/VIO_SLAM/src/snap_ros_examples/src/nodes/SnapdragonRosNodeVislam.cpp:161:13: error: ‘struct mvCPA_Configuration’ has no member named ‘startGain’
cpaConfig.startGain = param.camera_config.gain;
^~~~~~~~~
/home/myshare/VIO_SLAM/src/snap_ros_examples/src/nodes/SnapdragonRosNodeVislam.cpp:162:13: error: ‘struct mvCPA_Configuration’ has no member named ‘filterSize’
cpaConfig.filterSize = 1;
^~~~~~~~~~
/home/myshare/VIO_SLAM/src/snap_ros_examples/src/nodes/SnapdragonRosNodeVislam.cpp:163:13: error: ‘struct mvCPA_Configuration’ has no member named ‘exposureCost’
cpaConfig.exposureCost = 1.0f;
^~~~~~~~~~~~
/home/myshare/VIO_SLAM/src/snap_ros_examples/src/nodes/SnapdragonRosNodeVislam.cpp:164:13: error: ‘struct mvCPA_Configuration’ has no member named ‘gainCost’; did you mean ‘legacyCost’?
cpaConfig.gainCost = 0.3333f;

Is there any update on this working with the RB5?


Update header and theme style

Minor updates to header styles and theme to match mocks and DP/AnVIL portal header.

Completed with #549.


Adding accepted papers.

Should be checked by Sara first before merging.

Ready for review. If you want to toy around with the buttons, I'm sure it could be improved. Also no tags listed on the papers.


Suggestion: Skip downloading if the file already downloaded

Downloading the file that already on the path will make redundant download.
I hope there will be an option to handle this both on CLI and Python package.

I have a function that downloads the metadata first then calls the normal download if the data is valid and the file doesn't exist. Might be useful either way for ideas...
url_set contains a list of file id's
dest_path contains the target directory with trailing dir separator already appended, and the dir already exists
def fetch_file_metadata_from_google_drive(id):
    URL = "https://drive.google.com/uc?id="+id+"&authuser=0&export=download&confirm=t"
    headers = {
        "User-Agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36"  # NOQA
    }
    verify=True
    url_origin = URL
    session = requests.Session()
    meta={}
    meta['id']=id
    meta['valid']=False
    meta['filename']=''
    meta['url']=URL

    response = session.get(URL,allow_redirects=True, headers=headers, stream = True,verify=verify)
    token = get_confirm_token(response)
    meta['token']=token
    for k in response.headers:
        meta[k] = str(response.headers[k])

    if ('Content-Disposition' in response.headers):
        # Get the filename from the response header 'Content-Disposition'
        match = re.search(r'filename="(?P<filename>.+)"', response.headers['Content-Disposition'])
        # Make sure we found the filename field inside Content-Disposition
        if match is None:
            print('\n\nERROR: Unable to retrieve `dest_path` from `file_id`, please set it manually.')
            return meta
        if platform == 'win32':
            # Make it Windows safe, stripping: \/<>:"|?*
            remove_characters = dict((ord(char), None) for char in '\\/<>:"|?*')
        else:
            # Make it macOS and linux safe, stripping: /
            remove_characters = dict((ord(char), None) for char in '/')

        dest_path = match['filename'].translate(remove_characters)
        meta['filename']=dest_path
        meta['valid']=True
        print(json.dumps(meta,indent=4))
    else:
        # Need to redirect with confirmation
        try:
            url = get_url_from_gdrive_confirmation(response.text)
        except RuntimeError as e:
            print("Access denied with the following error:")
            error = "\n".join(textwrap.wrap(str(e)))
            error = indent(error, "\t")
            print("\n", error, "\n", file=sys.stderr)
            print(
                "You may still be able to access the file from the browser:",
                file=sys.stderr,
            )
            print("\n\t", url_origin, "\n", file=sys.stderr)
            meta['browser_url']=url_origin
            return meta
        print("redirect_url:",id,"-->",url)
        meta['redirect_url']=url
    return meta

for file_id in url_set:
    meta = fetch_file_metadata_from_google_drive(file_id)
    if (meta['valid']):
        epath = dest_path + meta['filename']
        if (not os.path.exists(epath)):
            gdown.download(meta['url'],output=dest_path,quiet=False)
        else:
            print("HAVE:", meta['filename'])


Missing License

The project is missing a licence file, which makes it difficult to use it in any projects. A few proposals:

MIT: Simple license, attribution only.
GPL: Can be freely used/modified, but changes should be published upstream if distributing the software.
AGPL: Same as GPL, but it closes the webserver loophole.

Or maybe the Author, just like me and many others, don't give a flying #$%$#%$# about licenses.

WTFPL is always an option!

WTFPL still falls under the category of license umbrella, even if you can do the @##@$ you want :) no license == freedom.

See the first few lines of jsmpg.js for license info.
I'd like it to be under MIT license, but I'm really sure if I can do this. It was based on a GPLd Java project. Any lawyers around?

@phoboslab, in your license remarks "who still cares about MPEG1" - funny thing is that, it is the BEST way to decode video on the web currently. Your implementation completely broke HTML5's awfule and un-capable mp4  tag player. I can do more with jsmpeg than any other player available. Thank you for this awesome contribution!!

@phoboslab Thanks, completely missed it. I agree that licensing this is pretty tricky. Wikipedia suggests that no patents apply for MPEG1. Stack Overflow suggests that you have to license it under the same license as the GPL project as it is probably a derivative work.


WIP: devcontainer for codespaces

List of current issues:

[x] nxdk's cmake does not build libraries for unknown reason: (workaround, call make then cmake)
[cmake] lld: error: could not open '.../nxdk/lib/.lib': No such file or directory
- libwinapi
- libxboxrt
- libpdclib
- libnxdk_hal
- libnxdk
- nxdk_usb
[x] rebuild container will trigger failure due to nxdk already existed from git clone via onCreateCommand.
[ ] CMake: find_package(SDL2 REQUIRED) failed

Using PKG_SEARCH_MODULE(sdl2 REQUIRED sdl2) fix little part of the issue.

[ ] CMake: find_package(Threads REQUIRED) failed

CMake is failing for CXX's features

[cmake] -- Detecting CXX compiler ABI info
[cmake] -- Detecting CXX compiler ABI info - failed
[cmake] -- Check for working CXX compiler: /usr/src/nxdk/bin/nxdk-cxx
[cmake] -- Check for working CXX compiler: /usr/src/nxdk/bin/nxdk-cxx - works
[cmake] -- Detecting CXX compile features
[cmake] -- Detecting CXX compile features - failed

Modified portion without find_package failures:

nxdk/lib/sdl/SDL2/include/SDL_stdinc.h:282:10: fatal error: 'sal.h' file not found x2

NOTE: missing __XBOX__ define from nxdk's cmake support
Somehow definition did not get included, have to manually include via add_compile_options for now.

Sources/config.cpp:10:10: fatal error: 'lwip/inet.h' file not found

NOTE: includes seem is not there from nxdk's cmake side?

nxdk/lib/sdl/SDL2/include/SDL_config_windows.h:46:22: error: typedef redefinition with different types ('unsigned int' vs '_PDCLIB_uintptr_t' (aka 'unsigned long'))

NOTE: missing __XBOX__ define from nxdk's cmake support
Somehow definition did not get included, have to manually include via add_compile_options for now.

UPDATE: After some modification from app's cmake, it only create exe file whilst xbe creation is absent. Why?

[ ] nxdk libraries aren't included? Need more research...
[ ] nxdk's $(NXDK_DIR)/lib/libjpeg is incorrect when it should be ${NXDK_DIR}/lib/libjpeg (from SDL2 Image pkgconfig)
[ ] xbe file creation did not occur, only exe file creation did.

In need of ACTUAL example for cmake usage. If cmake example does indeed work, then what modification needed in order to get it compile xbe or are we even using nxdk compiler?

Following nxdk's end fixes require:

nxdk/share/cmake/Modules/FindThreads.cmake:# NOTE: custom FindThreads is a requirement due to CMake's FindThreads is looking for pThreads and
# nxdk's toolchain is using system named Generic instead of "Windows" which could had work but inaccurate name.
# xboxkrnl and pdclib have C11 thread support out of the box
set(CMAKE_HAVE_THREADS_LIBRARY 1)
set(Threads_FOUND TRUE)
add_library(Threads::Threads INTERFACE IMPORTED)

nxdk/share/cmake/Modules/FindSDL2.cmake: TODO: Make this file align with SDL2 compatibility, like SDL2::SDL2include(FindPkgConfig)

pkg_check_modules(SDL2 REQUIRED sdl2)

if(NOT TARGET SDL2)
  add_library(SDL2 INTERFACE IMPORTED)
  set_target_properties(SDL2 PROPERTIES
    INTERFACE_INCLUDE_DIRECTORIES "${SDL_INCLUDE_DIRS}"
    # NOTE: pkg_check_modules' Cflags definition for "XBOX" did not get included and was passed to CFLAGS_OTHER for some reason...
    INTERFACE_COMPILE_OPTIONS "${SDL2_CFLAGS}"
  )
endif()

add line below into toolchain-nxdk.cmake file
set(CMAKE_MODULE_PATH "${NXDK_DIR}/share/cmake/Modules")
nxdk/lib/pkgconfig/libjpeg.pc
replace bracket to { } instead of ( )

At this time, both experimental package, nxdk's cmake-fixes branch, & changes made from this branch now has full CMake support.


Missing validation for instances

Compiler missing checks:

Instance overlap ( Show (List a) and Show (List Number) for example overlap )
Verify completeness of instances ( that all methods are implemented )

Linkage: https://github.com/madlib-lang/madlib/issues/12

I'll close #12 in favor of this one as this one is slightly more detailed.

Also I'd still pin interfaces as being experimental. I think they mainly work ok but we need more experience with them to see how they behave on the mid term. Then we also need to rethink a few bits of it as for now I had to do a few things that seem to derive from the theory, but also that make it behave as it should.


update refer links to other docs with utm_source

Goals:

Track visits from docarray to other docs in jina eco-system

@Roshanjossey  I don't think this is a sustainable way to add the query string parameter to clicked links. We can easily forget to add it on existing or new links. In order to give reliable information, is it possible to modify the website so that it adds utm_source=docarray to all the clicked links ? for example https://blog.miguelbernard.com/how-to-dynamically-add-a-query-string-to-all-links-in-a-page

@alaeddine-13, good point. I've made that change

@alaeddine-13, good point. I've made that change

Thanks, I just added small comments. Is it possible to apply this logic to other PRs that target other documentation websites if there are any ? Thanks

@alaeddine-13, I'm made changes you suggested. I'll do the same with other docs.


List inconsistent among object types.

Events takes a field selector but the other ones don't.
Suggest adding fields labels.Selector and labels labels.Selector to events type and all other corresponding types, so that a person can call:
c.Pods(ns).List() to get all pods or c.Events(ns).Select(someSelector).Project(someOtherSelector).List()
or various other combinations.
$ grep 'func.*List(' pkg/client/*.go | grep -v fake

pkg/client/endpoints.go:func (c *endpoints) List(selector labels.Selector) (result *api.EndpointsList, err error) {
pkg/client/events.go:func (e *events) List(label, field labels.Selector) (*api.EventList, error) {
pkg/client/minions.go:func (c *minions) List() (*api.MinionList, error) {
pkg/client/pods.go:func (c *pods) List(selector labels.Selector) (result *api.PodList, err error) {
pkg/client/replication_controllers.go:func (c *replicationControllers) List(selector labels.Selector) (result *api.ReplicationControllerList, err error) {
pkg/client/services.go:func (c *services) List(selector labels.Selector) (result *api.ServiceList, err error) {

Closing as dupe of #1362.


facing problem while deploying to heroku.

remote:
remote:        In ProviderRepository.php line 208:
remote:
remote:          Class 'Barryvdh\LaravelIdeHelper\IdeHelperServiceProvider' not found
remote:
remote:
remote:        Script @php artisan package:discover --ansi handling the post-autoload-dump event returned with error code 1
remote:  !     WARNING: There was a class not found error in your code
remote:
remote:  !     ERROR: Dependency installation failed!
remote:  !
remote:  !     The 'composer install' process failed with an error. The cause
remote:  !     may be the download or installation of packages, or a pre- or
remote:  !     post-install hook (e.g. a 'post-install-cmd' item in 'scripts')
remote:  !     in your 'composer.json'.
remote:  !
remote:  !     Typical error cases are out-of-date or missing parts of code,
remote:  !     timeouts when making external connections, or memory limits.
remote:  !
remote:  !     Check the above error output closely to determine the cause of
remote:  !     the problem, ensure the code you're pushing is functioning
remote:  !     properly, and that all local changes are committed correctly.
remote:  !
remote:  !     For more information on builds for PHP on Heroku, refer to
remote:  !     https://devcenter.heroku.com/articles/php-support
remote:  !
remote:  !     REMINDER: the following warnings were emitted during the build;
remote:  !     check the details above, as they may be related to this error:
remote:  !     - There was a class not found error in your code
remote:
remote:  !     Push rejected, failed to compile PHP app.
remote:
remote:  !     Push failed

This is not a ide-helper issue/bug.
Please tread the link provided by Heroku => https://devcenter.heroku.com/articles/php-support , it says

Heroku will not install development dependencies from the require-dev section of composer.json

Very likely, and correctly so, you've installed ide-helper as a dev dependency and thus it's not available in production.
You have multiple options, if you are not using any ide-helper functionality in production:

remove hardcoded reference to the service provider, the latest version of ide helper uses Laravels auto-discovered
or only conditionally reference it when APP_ENVis not production
or move the composer dependency into require instead of require-dev

This list is weighted from most preferred to least preferred solution IMHO :-)

I went with a combination of 1 and 2:
In composer.json:
    "extra": {
        "laravel": {
            "dont-discover": [
                "barryvdh/laravel-ide-helper"
            ]
        }
    },

and then in AppServiceProvider.php:
    public function register()
    {
        if ($this->app->environment() !== 'production') {
            $this->app->register(IdeHelperServiceProvider::class);
        }
    }

@barryvdh solution was provided, there's nothing this package can do if people manually register and and then don't install dependencies, issue can be closed :}

Had this problem as well, the solution was deleting those cache files right before installing composer dependencies and using artisan cache commands:
- rm ./bootstrap/cache/services.php
- rm ./bootstrap/cache/packages.php

just composer update solved me


switch openAI API to other open source LLMs

I wonder if it is possible to switch openAI API to other open source LLMs.
Thanks

yea, that's totally possible.
specifically which ones did you have in mind? there's a new one every 3 days ;)

@paulpierre , thanks for your prompt reply.
How about quantized llama 7b that has been in town for a bit time. :)

of course and thanks @spinning27
what is your host operating system? i can look into creating a LLaMa branch over the weekend because I'm actually curious on the implementation.
some questions before exploring:

base LLaMa isn't fine-tuned for QA/chat AFAIR. would you be open to other optimal options:

StableVicuna13b
Pygmalion6b
other options on HF

would it make more sense to run remote inference on HF vs local. i think most devs stand to benefit from this convenience

let me know your thoughts 👍

Actually, anyone @paulpierre mentioned would do as long as it is free (the cost of openAI API access could go up quite quickly with lots of query).
ATM, I mainly play it for out of curiosity. I have access to a GPU server machine or run locally on my M1 machine.
StableVicana-13B seems memory hungry. Not managed to run it on the machine yet, except quantized (compressed) version via llama.cpp.

Langchain currently develops at break-neck speed. Not sure sometimes it does not work like this one.

The best solution to beat the pricing of OPENAI is use your own deployed llms using fastChat or textgen-ui, they have nice openai api
But if you have M1 laptop you can easily run llm locally and this can be used for any prototype thing.
Sample code.
llm = LlamaCpp(model_path="models/llama-7b.ggmlv3.q4_0.bin", n_ctx=2048, verbose=True)
embeddings_model = LlamaCppEmbeddings(model_path="models/llama-7b.ggmlv3.q4_0.bin")

Been looking to build this myself and there we go -  a fine-crafted project already there.
I've made a replacement for ada embeddings @paulpierre  - perhaps hosting it locally would cut on running costs = https://github.com/proitservices/elmo_embedding_api
also, GPT4ALL is a great choice and with DB + Langchain the vicuna13b model would do just amazing (mistral 7b is also a good choice).
Would love to see this project grow into a fully featured and capable 'Jarvis' with memory and math capabilities + a rain of APIs extensions oob.
Happy to help with those, Peter


Stop Vienna from Sorting Folders on the Left Panel

Describe the bug
Usually, I never had any issue with Vienna for more than 15+ years. But, this time, it's really annoying.
Vienna always auto-sorts my folders in the alphabetical order on the left panel, although I set it manually. I do not want it to do that. Is there any way to force Vienna not to auto-sort folders on the left panel. I just want it to leave my manual sort alone.
It seems like Vienna reverts back to auto-sort after I set it manually.
To Reproduce
Actually, I don't know how to reproduce it, because it happens the second time, and seems to do it a whim.
Please complete the following information:

Vienna: 3.9.0 :7ad9df044: (8175)
OS version: 12.7.1

Thank you.

This happens to me as well - very annoying.  Have to either restore from Time Machine or re-import from my subscription export.
Mac Sonoma 14.4 Version 3.9.0 :7ad9df044: (8175)

It did it again. Switching back to an older version of my backups.

I've had this happen a few more times.  It seems to happen after every OS update.  Updated to 14.6 today and after restart Vienna had again reset View->Order By->Manual setting to View->Order By->Name.
To get my subscriptiions back in order I have to restore my subscriptions from an opml backup.

Vienna was non responsive this morning and I had to force quit it.  When it started it again reverted the the setting View->Order By->Manua to View->Order By->Name.

It has been happening for at least 6 months.  I have seen it happen after a force-quit and after an update reboot.  I let the system kill all running apps when I reboot.  I will try a user requested reboot after I post this and will let you know if it causes the issue.

I have been experiencing this for at least 6 months.  I have seen it happen after a force-quit and after update reboots.  I can't answer about crashes - I don't know that I have seen any.
I just did a normal user requested reboot (Black Apple->Restart...) and the setting stayed fine.
I let the reboots (both the user requested and the update reboots) kill all the running apps including Vienna.

There is work in version 3.9.1 related to this issue. Can you confirm / infirm if there is an improvement?

There is work in version 3.9.1 related to this issue. Can you confirm / infirm if there is an improvement?

I installed 3.9.1 and it did not change the Order By setting!  Yay! Thank you

Closing this as #1768 is more up to date.


The automated release is failing 🚨

:rotating_light: The automated release from the alpha branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the alpha branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

Unfortunately this error doesn't have any additional information. Feel free to kindly ask the author of the @semantic-release/exec plugin to add more helpful information.

Good luck with your project ✨
Your semantic-release bot :package::rocket:

:rotating_light: The automated release from the alpha branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the alpha branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

Cannot push to the Git repository.
semantic-release cannot push the version tag to the branch alpha on the remote Git repository with URL<EMAIL_ADDRESS>This can be caused by:

a misconfiguration of the repositoryUrl option
the repository being unavailable
or missing push permission for the user configured via the Git credentials on your CI environment

Good luck with your project ✨
Your semantic-release bot :package::rocket:


[Feature request] Add a flag to have error logs

It would be nice to have flag like --error-log /path/to/errors.txt to see what gave path/payload gave an error than just having the of numbers of errors at the bottom.

Hi @0xatul, have you take a look in -debug-log?
OUTPUT OPTIONS:
  -debug-log       Write all of the internal logging to the specified file.

Closing the issue in order to keep the issues backlog readable, and I do consider the issue either resolved or there's no communication. If you have something to add to the issue, please comment and we'll consider reopening it.


Old objects are not deleted

When deploying again, old objets in the bucket are not removed
https://github.com/brandonweiss/discharge/blob/cc81ccb0b24f5c9bc880022d0de7fbf162d36f7c/lib/deploy/synchronize-website.js#L89
Should this not be calling s3.deleteObjects() ?

🤦♂ Yes, you’re totally right. I must’ve copy-pasta-ed. Fixing now!


 :-(

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27593548-cta-percentage-is-149?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

def update_cta(website):
nusers = website.db.one("""
    SELECT nusers FROM paydays
    ORDER BY ts_end DESC LIMIT 1
""", default=0)
nreceiving_from = website.db.one("""
    SELECT nreceiving_from
      FROM teams
     WHERE slug = 'Gratipay'
""", default=0)
website.support_current = cur = int(round(nreceiving_from / nusers * 100)) if nusers else 0

https://github.com/gratipay/gratipay.com/blob/f9749d98d2853701968adf437e9436deca1741aa/gratipay/utils/__init__.py
I think the math looks ok, I'm guessing something went wrong in the db?

Looks like nusers FROM paydays doesn't include the people who haven't been charged because of the new arrears system - and nreceiving_from FROM teams does.

Closed in #3848


Remove direct Instant::now() calls

Quiche currently uses Instant::now() directly in various places.
Moving time management from quiche to the applications would make the quiche state machine more deterministic, which would improve testability.
Also, the time granularity could be controlled by the application, potentially reducing the number of syscalls.
For example the signature of send(&mut self, out: &mut [u8]) could be changed to send(&mut self, out: &mut [u8], now: Instant).
Automated tests that require timed events would be simpler.
Example:
     #[test]
     /// Tests that old data is retransmitted on PTO.
     fn early_retransmit() {
         let mut buf = [0; 65535];
+        let mut now = EPOCH;

-        let mut pipe = testing::Pipe::new().unwrap();
+        let mut pipe = testing::Pipe::new(now).unwrap();
-        assert_eq!(pipe.handshake(), Ok(()));
+        now = pipe.handshake(now, Duration::from_millis(1)).unwrap();

         // Client sends stream data.
         assert_eq!(pipe.client.stream_send(0, b"a", false), Ok(1));
-        assert_eq!(pipe.advance(), Ok(()));
+        now = pipe.advance(now, Duration::from_millis(1)).unwrap();

         // Client sends more stream data, but packet is lost
         assert_eq!(pipe.client.stream_send(4, b"b", false), Ok(1));
-        assert!(pipe.client.send(&mut buf).is_ok());
+        assert!(pipe.client.send(&mut buf, now).is_ok());

         // Wait until PTO expires. Since the RTT is very low, wait a bit more.
-        let timer = pipe.client.timeout().unwrap();
+        let timer = pipe.client.timeout(now).unwrap();
-        std::thread::sleep(timer + time::Duration::from_millis(1));
+        now += timer + Duration::from_millis(1);

-        pipe.client.on_timeout();
+        pipe.client.on_timeout(now);

         let epoch = packet::Epoch::Application;
         assert_eq!(
             pipe.client
                 .paths
                 .get_active()
                 .expect("no active")
                 .recovery
                 .loss_probes(epoch),
             1,
         );

         // Client retransmits stream data in PTO probe.
-        let (len, _) = pipe.client.send(&mut buf).unwrap();
+        let (len, _) = pipe.client.send(&mut buf, now).unwrap();
         assert_eq!(
             pipe.client
                 .paths
                 .get_active()
                 .expect("no active")
                 .recovery
                 .loss_probes(epoch),
             0,
         );

         let frames =
             testing::decode_pkt(&mut pipe.server, &mut buf[..len]).unwrap();

         let mut iter = frames.iter();

         // Skip ACK frame.
         iter.next();

         assert_eq!(
             iter.next(),
             Some(&frame::Frame::Stream {
                 stream_id: 4,
                 data: stream::RangeBuf::from(b"b", 0, false),
             })
         );
         assert_eq!(pipe.client.stats().retrans, 1);
     }

Passing in now externally would be nice for making the state machine deterministic to aid testing.
In addition, hard-coding the std::time::Instant type for deadlines is a bit unfortunate, because std::time::Instant does not expose its internal fields. This means that it is unsuitable for use with e.g. io_uring::Timeout with TimeoutFlags::ABS. It is possible to work around this by using Connection::timeout() instead of Connection::timeout_instant(), but that means there is an additional call to std::Instant::now() (inside Connection::timeout), which could be avoided with a time type that exposes the data that the kernel needs.


Error Running Code with UBNORMAL and STC Datasets

I am experiencing an issue when trying to run the code with the UBNORMAL and STC datasets, using the provided config.yaml file and pre-trained checkpoint. The problem occurs in the post_processing function of the mocodad.py file.
Error:
In the post_processing function, the code loops over different transforms. However, in some cases, the cond_transform variable is an array of all False values. This causes the filter_vectors_by_cond function to return empty arrays for out_transform, gt_data_transform, meta_transform, and frames_transform.
As a result, the figs_ids variable, which is calculated as sorted(list(set(meta_scene_clip[:, 2]))), is also empty. This causes an error when trying to loop over figs_ids and append elements to the error_per_person list.
Finally, when trying to execute the line clip_score = np.stack(error_per_person, axis=0), an error occurs because error_per_person is an empty list.
Steps to Reproduce:
Run the code with the UBNORMAL and STC datasets, using the provided config.yaml file and pre-trained checkpoint.
The error should occur in the post_processing function of the mocodad.py file.
The code produces an error when trying to execute the line clip_score = np.stack(error_per_person, axis=0), due to error_per_person being an empty list.:
ValueError: need at least one array to stack

Hi!
This error seems similar to the one in the issue here: https://github.com/aleflabo/MoCoDAD/issues/4#issuecomment-2224957007
Is the use_hr parameter set to true in the config?

I double-checked the settings and adjusted them to both true and false; however, I encountered an issue. I downloaded the data from the provided Google Drive file.
There are three datasets: ubnormal, avenue, and hr-shanghitech. In your code, when we pass "avenue," it indicates that it is "not usable yet," which means we need to download hr_avenue. Where can we find this?
Additionally, I downloaded the pretrained checkpoints, but the code did not execute successfully with any of the provided configuration files, regardless of whether hr is set to true or false. Could you please verify this?
I kindly ask you to clone your project with the uploaded dataset and checkpoint to ensure everything is functioning correctly. I suspect there may be an issue with the provided data or the checkpoints. I have spent a considerable amount of time on this, but it still does not work.


⚠ ClamAV has degraded performance

In d6bebf0, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 217 ms

Resolved: ClamAV performance has improved in 0bfd5b7.


Event trigger 'OnOpenPageEvent' can only be used if the page specifies 'SourceTable'

We are currently converting an adaptation that responds to the "OnOpenPageEvent" in the Classic C/AL. This also applies to pages of the type "NavigationPage". However, they do not have a "SourceTable"with us. For this reason we are not able to subscribe to this event, which was no problem in the C/AL. Would it be possible to subscribe to this event in the Extension 2.0 world?

Could you please check if which build you have? Several improvements in this area were made in the anniversary update. You can now subscribe to many events even if the target page does not have a source table.

Good clue. I only noticed the error in the extension development of NAV 2018. I'll check it out, and then let you know again!

Closing due to inactivity. Please let us know in case this is still an issue


Documentation site seems borked

For example, https://nlog-project.org/config/?tab=layout-renderers#layouts presents empty screen:

The other screens appear borked too.
Javascript console shows:
Failed to load resource: net::ERR_BLOCKED_BY_CLIENT
itemList.js?c=2023-10-29%2011:31:02%20+0000:53 Uncaught ReferenceError: Vue is not defined
at itemList.js?c=2023-10-29%2011:31:02%20+0000:53:1
searchPage.js?c=2023-10-29%2011:31:02%20+0000:16 Uncaught ReferenceError: Vue is not defined
at searchPage.js?c=2023-10-29%2011:31:02%20+0000:16:11

actually hang on,  i just noticed when I pasted the "ERR_BLOCKED_BY_CLIENT"

Just found out its the "DuckDuckGo Privacy Essentials extension" blocking cdn.jsdeliver.net because apparently they are using tracking?


Crash when loading a big code file

MipsAsmParser.zip
Issue Type: Bug

Load the file MipsAsmParser.cs attached to VSCode.
Crashed.

VS Code version: Code 1.44.2 (ff915844119ce9485abfe8aa9076ec76b5300ddd, 2020-04-16T16:36:23.138Z)
OS version: Windows_NT x64 10.0.18363

PS:
I also checked with extensions off. It seems that it is the issue of VSCode.

(Experimental duplicate detection)
Thanks for submitting this issue. Please also check if it is already covered by an existing one, like:

Visual studio code is getting crashed (#94932)

It is not the same issue.

What is the file's size and how many lines does it have?

What is the file's size and how many lines does it have?

I have uploaded the file in the first line of the issue.

@Banyc Thank you, this is all caused by the following line in your file:
			} while ( (((_la) & ~0x3f) == 0 && ((1L << _la) & ((1L << T__0) | (1L << T__7) | (1L << T__9) | (1L << T__10) | (1L << T__11) | (1L << T__12) | (1L << T__13) | (1L << T__14) | (1L << T__15) | (1L << T__16) | (1L << T__17) | (1L << T__18) | (1L << T__19) | (1L << T__20) | (1L << T__21) | (1L << T__22) | (1L << T__23) | (1L << T__26) | (1L << T__27) | (1L << T__28) | (1L << T__29) | (1L << T__30) | (1L << T__31) | (1L << T__32) | (1L << T__33) | (1L << T__34) | (1L << T__35) | (1L << T__36) | (1L << T__37) | (1L << T__38) | (1L << T__39) | (1L << T__40) | (1L << T__41) | (1L << T__42) | (1L << T__43) | (1L << NEWLINE) | (1L << COMMENT) | (1L << NUM) | (1L << ALPHA))) != 0) );

There are multiple regular expressions that take a long time. I found them with some newly added support via F1 > Developer: Start Text Mate Syntax Grammar Logging:
- searchOnigRegExp: (?x)
(?:
  (?:(\bref)\s+(?:(\breadonly)\s+)?)?(\bvar\b)| # ref local
  (?<type-name>
    (?:
      (?:ref\s+(?:readonly\s+)?)?   # ref local
      (?:
        (?:(?<identifier>@?[_[:alpha:]][_[:alnum:]]*)\s*\:\:\s*)? # alias-qualification
        (?<name-and-type-args> # identifier + type arguments (if any)
          \g<identifier>\s*
          (?<type-args>\s*<(?:[^<>]|\g<type-args>)+>\s*)?
        )
        (?:\s*\.\s*\g<name-and-type-args>)* | # Are there any more names being dotted into?
        (?<tuple>\s*\((?:[^\(\)]|\g<tuple>)+\))
      )
      (?:\s*\?\s*)? # nullable suffix?
      (?:\s*\[(?:\s*,\s*)*\]\s*)* # array suffix?
    )
  )
)\s+
(\g<identifier>)\s*
(?!=>)
(?=,|;|=|\))
|- did not match after 3834.285 ms

- searchOnigRegExp: (?x)
(\()\s*
(?<type-name>
  (?:
    (?:
      (?:(?<identifier>@?[_[:alpha:]][_[:alnum:]]*)\s*\:\:\s*)? # alias-qualification
      (?<name-and-type-args> # identifier + type arguments (if any)
        \g<identifier>\s*
        (?<type-args>\s*<(?:[^<>]|\g<type-args>)+>\s*)?
      )
      (?:\s*\.\s*\g<name-and-type-args>)* | # Are there any more names being dotted into?
      (?<tuple>\s*\((?:[^\(\)]|\g<tuple>)+\))
    )
    (?:\s*\?\s*)? # nullable suffix?
    (?:\s*\[(?:\s*,\s*)*\]\s*)* # array suffix?
  )
)\s*
(\))(?=\s*@?[_[:alnum:]\(])
|- did not match after 2960.280 ms

- searchOnigRegExp: (?x)
(?<tuple>\s*\((?:[^\(\)]|\g<tuple>)+\))\s*
(?!=>|==)(?==)
|- did not match after 3877.615 ms

- searchOnigRegExp: (?x)
(?:
  (?:(\bref)\s+(?:(\breadonly)\s+)?)?(\bvar\b)| # ref local
  (?<type-name>
    (?:
      (?:ref\s+(?:readonly\s+)?)?   # ref local
      (?:
        (?:(?<identifier>@?[_[:alpha:]][_[:alnum:]]*)\s*\:\:\s*)? # alias-qualification
        (?<name-and-type-args> # identifier + type arguments (if any)
          \g<identifier>\s*
          (?<type-args>\s*<(?:[^<>]|\g<type-args>)+>\s*)?
        )
        (?:\s*\.\s*\g<name-and-type-args>)* | # Are there any more names being dotted into?
        (?<tuple>\s*\((?:[^\(\)]|\g<tuple>)+\))
      )
      (?:\s*\?\s*)? # nullable suffix?
      (?:\s*\[(?:\s*,\s*)*\]\s*)* # array suffix?
    )
  )
)\s+
(\g<identifier>)\s*
(?!=>)
(?=,|;|=|\))
|- did not match after 3979.295 ms

Moved to the grammar's repo: https://github.com/dotnet/csharp-tmLanguage/issues/180

Closing in favor of the upstream issue.


Demo mode - log possible trades

I have blackbird running one week in demo mode and csv is empty (only header). Is there or can be added option to log possible trades to csv in demo mode? It will be helpful to see some results before run bot with real money.

Personally, I wouldn't want to nest action critical functions (like send order) in any loops for a feature like this. Without doing that, I think adding the feature to the current codebase would be a lot of work for very little payoff.
That being said, I don't want to discourage anyone from submitting a PR for a feature like this -- I'm sure it's in demand.


When softDelete, getDirty is empty in deleted event

Laravel Version: 7.12.0
PHP Version: 7.4.6
Database Driver & Version:

Description:
When soft-deleting a model changes are not visible to deleted event handlers as syncOriginal is called before firing the event.
In real and forced delete DB record is deleted, syncOriginal is not needed and everything is ok.
In soft-delete:

Model::delete() calls $this->performDeleteOnModel() (goes to SoftDeletes trait method)

SoftDeletes::performDeleteOnModel calls runSoftDelete
SoftDeletes::runSoftDelete updates DB record and calls $this->syncOriginalAttributes

Model::delete() fires deleted event

Result is that in deleted event handlers model getDirty() is empty but it should contain at least deleted_at attribute.
Proposal:
Make the order similar to update which soft-delete in reality is. (as in Model::finishSave)
Move $this->fireModelEvent('deleted', false); from Model::delete to the end of Model::performDeleteOnModel and add to softDeletes::runSoftDelete between update and syncOriginal
$query->update($columns);

$this->fireModelEvent('deleted', false);

$this->syncOriginalAttributes(array_keys($columns));

This way all the changes done in softDelete logic is visible to deleted event handlers.
Note: This creates BC breaks if someone has overwritten performDeleteOnModel or runSoftDelete in their app.
PR or not to PR ?

I'm curious what the use case would be for dirty attributes in a soft delete operation? Aren't you just calling $foo->delete()? What is being updated?

Yes, I call ->delete() and in runSoftDelete() I have more changes to the model (ex: deleted_by).
Then in deleted event handler change set is stored for auditing. This requires usage of getDirty().


Vickel: Watch priestoguname(?:@gmail.com)?

Vickel requests the watch of the watch_keyword priestoguname(?:@gmail\.com)?. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

priestoguname(?:@gmail\.com)? has been seen in 3 true positives, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


Convert photos that don't match their extension

There are some, e.g. select id, created_at, file_file_name, medium_url, file_content_type from photos where file_content_type = 'image/png' and medium_url LIKE '%.jpg%' limit 10;
We should probably also fix all the photos that have no file extension. Most of these have type octet-stream. I don't think any of these get created these days, so this is probably just a cleanup mission.

For the record, cleaned up jpg-not-jpgs with
Photo.where( id: [3545620, 2362669, 2362691, 2362716, 1662209, 2362739, 2362771, 1583083, 2931574, 3144540, 3144543, 3144537, 3144541, 3144539, 3144542, 3281937] ).each do |photo|
  puts photo.id
  LocalPhoto.turn_remote_photo_into_local_photo( photo )
end

As a note to myself, there are a lot of photos lacking extensions, and the above technique of downloading the photo and then uploading it just to change the name is pretty slow. Probably more efficient to just move the files with the S3 API and rename them locally.


Unexpected language change from English to Hungarian

I am a student at Wyncode Academy.
Our instructor Auston Busen suggested your awesome game as a way to learn Flexbox.
At level 14, I answered correctly and pressed "next".
The language changed to Hungarian as described here.
I answered correctly and pressed "next" again. The language remained in Hungarian.
I pressed the "English" link at the bottom. Still Hungarian. I pressed the "French" link,  and the language changed to French. I manually entered http://flexboxfroggy.com/#en into Chrome's location and still the language was Hungarian.
I finally tried pressing "Magyar" and then "English" and the language was then in English.
Thanks for making Flexbox Froggy!

Hi @dm-wyncode, the Hungarian translation was recently added. A bug is causing the new Hungarian instructions to overwrite English for levels 14 and 15. Thanks for your detailed report — I was able to track it down and will push a fix shortly.


pipewire + alsa is broken

Describe the bug
With pipewire.alsa.enable = true, alsa output doesn't work.
aplay -l works correctly, but pactl list cards outputs empty list.
mpv --ao=alsa ... doesn't play the file, it just hangs in the beginning at -00:00 file position:
Steps To Reproduce
Steps to reproduce the behavior:

Configure services.pipewire { enable = true; alsa.enable = true; };
Try pactl list cards # empty output
Try playing a file with alsa. # hangs

Expected behavior
pactl list cards shouldn't be empty; playing a file should work.
Additional context
switching from pipewire to pulseaudio fixes the sound
Notify maintainers

@Kranzes
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.72, NixOS, 22.05 (Quokka), 22.05.3620.e06bd4b64bb`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.05, nixos-unstable"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I've tried with unstable channel off, same result.

@sshilovsky have you tried if deleting ~/.asoundrc would help?

@sshilovsky have you tried if deleting ~/.asoundrc would help?

I don't have any, I've also tried it with empty user home dir.

Ok. so then your problem must be about something else. I had problem with one of my setups and got it solved by removing content from .asoundrc.

Came across this, can confirm that pipewire+alsa is fine

And I ran pipewire + alsa for a short while on nixos 22.11 and got working audio output from qmmp music player.
@sshilovsky: please comment if something still doesn't work and we can re-open.


Add test for the scraper service

Description
Add the test for scaper service function. For the test axios request has been mocked so that cheerio can parse the mocked file and test whether the scraper function works properly or not.
Issue
#3

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


TTS is ducked and does not recognize voice when VoiceOver is running

Moved the issue from here.
https://github.com/hulop/NavCogIOSv3/issues/61
@hiroshi-shin 's comment

The original issue:
miraikan-research/NavCog-Miraikan#18
Additional description:
I have recognized the low volume problem at the beginning. It was not happening on NavCog3 that time, and I thought there was something wrong with my code.
However, after the last system update, both my customized app and NavCog3 are in low volume, while the voice recognition is not started.
Please let me know if there is any updates required. Thank you.

https://github.com/hulop/HLPDialog/commit/c7c81aac3ad454cc07b9e95aaa2069a93acce0e0
Commit the bugfix for this issue.


"libbson-static-1.0", but CMake did not find one.

feng@feng-B250-HD3P:~/mongo-cxx-driver/build$ cmake -DBUILD_SHARED_LIBS=OFF -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr/local ..
-- The CXX compiler identification is GNU 5.4.0
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- The C compiler identification is GNU 5.4.0
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Auto-configuring bsoncxx to use MNMLSTC for polyfills since C++17 is inactive
CMake Error at src/bsoncxx/CMakeLists.txt:86 (find_package):
By not providing "Findlibbson-static-1.0.cmake" in CMAKE_MODULE_PATH this
project has asked CMake to find a package configuration file provided by
"libbson-static-1.0", but CMake did not find one.
Could not find a package configuration file provided by
"libbson-static-1.0" (requested version 1.9.2) with any of the following
names:
libbson-static-1.0Config.cmake
libbson-static-1.0-config.cmake

Add the installation prefix of "libbson-static-1.0" to CMAKE_PREFIX_PATH or
set "libbson-static-1.0_DIR" to a directory containing one of the above
files.  If "libbson-static-1.0" provides a separate development package or
SDK, be sure it has been installed.
-- Configuring incomplete, errors occurred!
See also "/home/feng/mongo-cxx-driver/build/CMakeFiles/CMakeOutput.log".

I solved this problem through the eosio_build.sh script.
I'm from China,when the eosio_build.sh script is used, the basic access to the Mongo link is not basically accessible, causing the step to be interrupted.


Hashing the API token (like Laravel Sanctum) before storing it in the database

Hello,
because the tokens are pretty much passwords, Laravel Sanctum hashes tokens before saving them to the database in the following format: <id>|<token> - the ID is there for finding the token faster (by primary key lookup).
Here's their implementation for creating a new token, as seen in HasApiTokens.php (line 46)
    public function createToken(string $name, array $abilities = ['*'], DateTimeInterface $expiresAt = null)
    {
        $token = $this->tokens()->create([
            'name' => $name,
            'token' => hash('sha256', $plainTextToken = Str::random(40)),
            'abilities' => $abilities,
            'expires_at' => $expiresAt,
        ]);

        return new NewAccessToken($token, $token->getKey().'|'.$plainTextToken);
    }

... and here's the way they check it in PersonalAccessToken.php (line 58):
public static function findToken($token)
    {
        if (strpos($token, '|') === false) {
            return static::where('token', hash('sha256', $token))->first();
        }

        [$id, $token] = explode('|', $token, 2);

        if ($instance = static::find($id)) {
            return hash_equals($instance->token, hash('sha256', $token)) ? $instance : null;
        }
    }

I think it'd be a great idea for laravel-keyable to do the same thing. The user would receive a plainTextToken which can only be displayed once.

This will be added in the next release shortly, thanks for the suggestion.


[iOS] Request notification permission right after app is initialized and get denied response immediately

Describe your environment

React-Native version: 0.59.5
Platform: iOS, Android, both? iOS
Device (which one?), simulator? iphone XR
OS version: 12.1
react-native-permissions version: 1.1.1
Devtools: Xcode? Version 10.1 (10B61)

How to repeat issue and example

call Permissions.request('notification') in componentDidMount method of app.js
show notification permission system dialog for user confirmation and receive denied immediately before use click any button.

Solution
For now, i use PushNotificationIOS to check and request notification permission.

I am closing this since this issue does not exists anymore with the 2.0.0 release.

This is still happening in v2.0.9 of the app with "react-native": "0.61.5" and "react-native-permissions": "^2.0.9" using the following request to obtain permissions:
    requestNotifications([
      'alert',
      'badge',
      'sound',
      'criticalAlert',
      'carPlay',
      'provisional',
    ]).then(({ status, settings }) => {
      Alert.alert('requested', JSON.stringify({ status, settings }));
    });

@katesowles  I was seeing the same thing until I took out provisional and carPlay.

Ah yes! I found the same yesterday after a bit of tinkering but failed to follow up. Good catch, @davrosull! 👍


Language Not Changing

Hello Team
Greetings of the Day!!
i have integrate the localisation in application, Its integrated successfully, but when i try to change the language then it's not translated
Can you please tell me how i can resolve it
I am change the language from other screen, like from profile screen..
Here is my main.dart code
import 'dart:ui';
import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';
import 'package:flutter_localizations/flutter_localizations.dart';
import 'package:flutter_translate/flutter_translate.dart';
import 'Home.dart';
final GlobalKey navigatorKey = GlobalKey(debugLabel: "Main Navigator");
Future main() async {
WidgetsFlutterBinding.ensureInitialized();
var delegate = await LocalizationDelegate.create(fallbackLocale: 'en_US', supportedLocales: ['en_US', 'ar']);
runApp(LocalizedApp(delegate,
new MaterialApp(
debugShowCheckedModeBanner: false,
navigatorKey: navigatorKey,
home: new MyApp()
),
),
);
}
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
var localizationDelegate = LocalizedApp.of(context).delegate;

new Future.delayed(const Duration(seconds: 10), () {
  Navigator.pushReplacement(context, MaterialPageRoute(builder: (context) => Home()));
});

return LocalizationProvider(
  state: LocalizationProvider.of(context).state,

  child: MaterialApp(
    localizationsDelegates: [
      GlobalMaterialLocalizations.delegate,
      GlobalWidgetsLocalizations.delegate,
      localizationDelegate
    ],
    supportedLocales: localizationDelegate.supportedLocales,
    locale: localizationDelegate.currentLocale,

    debugShowCheckedModeBanner: false,
    theme: ThemeData(
      backgroundColor: Color.fromARGB(1, 138, 51, 147),
    ),
    home: Scaffold(
      backgroundColor: Colors.white,
      body: Container(
        child: new Center(
            child: new Image(image: new AssetImage('assets/images/logo.png'), fit: BoxFit.fill)
        ),
      ),
    ),
  ),
);

}
}

The issue is probably related to how you perform the navigation.

@bratan what's the solution for that? I have the same issue and I do a normal transition


Fix: Add custom annotation add for application edit page

Description of your changes

Fixes #665
I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run yarn lint to ensure the frontend changes are ready for review.
[ ] Run make reviewableto ensure the server changes are ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

Special notes for your reviewer

@yikuaibro Maybe you don't submit all the changed codes.

@yikuaibro Maybe you don't submit all the changed codes.

@barnettZQG done


Miscellaneous considerations for request tests

Thanks for offering this tool to the community! I've been playing around with it to see if it satisfies my needs and I've hit a few minor snags.
Basic:

Cannot use $gt, $gte, $lt, $lte operators on array size. Arrays can only be asserted for exact sizing. Ideally I'd like more flexibility on testing array sizes. $.items: { $exists: true, $type: array, $size: { $gt: 5 } }
I'd like to see status and $h tests run first, always. Currently if you define those in common they appear to be appended to the endpoint test list and run last. Perhaps prepend the common tests (in the order defined by the user) to the endpoint test list OR perhaps sort status & header tests first, regardless of where they are defined?
If status or $h tests fail, the $. body tests are likely to fail since the response shapes or types will be different than expected. Perhaps do not run $. tests if the status and/or $h. tests fail? I have example endpoints where 400 or 403 responses return strings instead of application/json and this creates noise when zzzapi attempts to parse JSON in these cases.
I am using $regex, but have you considered $startsWith, $endsWith, and $contains as simpler alternatives?

Advanced:

Have you considered a declaration to skip tests? Could be as simple as skip: true or more sophisticated like skip: { $os: { $eq: "Windows" } }
ThunderClient offers some compelling array indexing tricks. I'm not sure if those are JSONPath compliant or not, but they are convenient.
Have you considered a mechanism to drop down to javascript asserts/tests when basic ones do not suffice?

Thanks. Each of the points makes a lot of sense. I had some of these in mind but had not acted.
I will add a separate issue for each of the points because we can address some of them immediately but some others may take time.

Need clarification for #7

Excellent! Thanks for the consideration @vasan-agrostar!


[WIP]Add new backlog-dedup command

Signed-off-by: metonymic-smokey<EMAIL_ADDRESS>fixes #2348

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Added a MetaFilter to separate blocks based on max compaction level - blocks with a compaction level lower than max compaction level are deleted.
Added a new command backlog-dedup in tools bucket to perform backlog deduplication on blocks with max compaction level.

Verification
Ran this command: sudo ./thanos tools bucket backlog-dedup --deduplication.replica-label=cluster  --debug.max-compaction-level=1 --data-dir=./test --objstore.config-file=local-bucket.yaml --deduplication.func=penalty --consistency-delay=0 --delete-delay=0.
This is a part of the logs:
ts=2021-10-10T05:06:20.80301Z caller=log.go:168 level=info msg="Found overlapping blocks during compaction" ulid=01FHM9GFYTAHYEZSY90BFHZA17 ts=2021-10-10T05:07:26.663905Z caller=log.go:168 level=info msg="compact blocks" count=2 mint=1633636815003 maxt=1633644015001 ulid=01FHM9GFYTAHYEZSY90BFHZA17 sources="[01FHJFVFPH69V0JXYC67Y6AAVM 01FHJF773MF6T0QS6JH98EMG03]" duration=1m5.901067503s level=info ts=2021-10-10T05:07:26.665113Z caller=compact.go:817 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="compacted blocks" new=01FHM9GFYTAHYEZSY90BFHZA17 blocks="[test/compact/0@9176828345592865883/01FHJFVFPH69V0JXYC67Y6AAVM test/compact/0@9176828345592865883/01FHJF773MF6T0QS6JH98EMG03]" duration=1m5.902304504s duration_ms=65902 overlapping_blocks=true level=info ts=2021-10-10T05:07:26.842571Z caller=compact.go:855 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="uploaded block" result_block=01FHM9GFYTAHYEZSY90BFHZA17 duration=140.508929ms duration_ms=140 level=info ts=2021-10-10T05:07:26.843991Z caller=compact.go:877 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="marking compacted block for deletion" old_block=01FHJFVFPH69V0JXYC67Y6AAVM level=info ts=2021-10-10T05:07:26.844435Z caller=block.go:209 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="block has been marked for deletion" block=01FHJFVFPH69V0JXYC67Y6AAVM level=info ts=2021-10-10T05:07:26.845831Z caller=compact.go:877 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="marking compacted block for deletion" old_block=01FHJF773MF6T0QS6JH98EMG03 level=info ts=2021-10-10T05:07:26.846549Z caller=block.go:209 group="0@{cluster=\"deduped\"}" groupKey=0@9176828345592865883 msg="block has been marked for deletion" block=01FHJF773MF6T0QS6JH98EMG03 level=info ts=2021-10-10T05:07:26.856134Z caller=compact.go:999 msg="start sync of metas" level=warn ts=2021-10-10T05:07:27.173761Z caller=fetcher.go:739 msg="block has no labels left, creating one" cluster=deduped level=warn ts=2021-10-10T05:07:27.173804Z caller=fetcher.go:739 msg="block has no labels left, creating one" cluster=deduped level=warn ts=2021-10-10T05:07:27.173822Z caller=fetcher.go:739 msg="block has no labels left, creating one" cluster=deduped

@yeya24 this is a WIP PR for the new command.
ToDo:

E2E Tests
Update docs and changelog


Heated seats constantly report as "On" even when vehicle is switched off

Region and Brand of car
United Kingdom / Kia (Sorento)
Describe the bug
My vehicle has heated front & rear seats. These are reported as 4 sensors (Front Left Seat Heater etc) via the integration, however all 4 constantly report their status as "On" even though they are not. For example; the vehicle is currently parked with the ignition off and they are still showing as "On" in Home Assistant.
Debug logs if an error occurred
N/A
To Reproduce

Enable the integration
Sign in
Navigate to Configuration >> Devices >> {Vehicle Name}
Inspect status of sensor(s)

Expected behavior
Sensors should report "Off" when the seat heaters are off and only report "On" when they are switched on.
Screenshots
Not really needed, but can provide if required.
Additional context
Status is reporting correctly in the official Kia UVO app.

Are you able to confirm if this corrects after next scan?

Please also share your Data sensor from Developer Tools.

Hi,
It happens the same to me.
My car is a Tucson HVE just bought in Spain. The Bluelink app reports all the seats heaters are off as these actually are, but HA reports all on (Encendido).
Here is a screenshot.

Are you able to turn them on, force a update and post your data again? The data shows a "2" maybe that means off and a different number means on?  Please also try turning the car off while heated seats are on and see if the value changes from whatever "on" is.

As well what does the Kia app report?

I've done a quick test. The data I sent previously was with the car's ignition off.
Now, I started the car, and switched on the driver's seat heater to max power (Three orange leds lighting), and switched on the copilot seat air conditioner/fan to min power (One blue led lighting).
In this Tucson, seats can be heated or cooled with three power levels. The first picture shows the switches for both front seats for heating and cooling with the orange and blue leds for the test condition.
I saw the status changed in the Bluelink app (Bluelink picture), but HA reminded unchanged (value 2 for those sensors, HA picture). I made sure HA was updated.

Perhaps the value of these sensors is not well parsed. (?)
I appreciate your work pretty much! Thanks a lot.

I will also check with my Kia Niro PHEV with heated and cooled seats and publish my findings here.

Hm. My Kona doesn't report seat heat settings at all, let alone allow to set it ... :{

Could you do some mapping for us?  Looks like @fuatakguns doesn't report these values so we can't use his car to test.   We wrongly assumed this reported on / off but I think it actually follows something like this: https://github.com/fuatakgun/kia_uvo/issues/225#issuecomment-1006112071
1: Unknown
2: Off
3: Low Cool
4: Medium Cool
5: Full Cool
6: Low Heat
7: Medium Heat
8: High Heat
To test you will need to set the car to one of the above and run a force update via the developer update menu and then check the data in the data element.

Another question for everyone.  How should we show this in home assistant?  Do you want it to just show on off or do you want it to show cool, heat and the levels?

Could someone test the above PR for me and provide feedback?

I have checked the values in the latest version  of HA (2022.2.2)

The screenshots show the status in the niro.data component show status 2 for the seat heaters. The specific entity of the seat heater shows on. Do the dara is read correctly by niro.data but conversion to the seat heater entity is wrong.
(I hope the pictures will show as pictures and not as link in my comment)

Please test in 2.0 if you get a chance.  If not solved it is the place we can now fix this.


����������

The package has successfully installed on machine

@GF-Huang   can you please share the version you are using of .NET Interactive?  Enter #!about into a cell and run it?


[ADAP-416] [PR Tracking]Ignore nulls in partitions to be processed #552

Tracking for
PR #552
Issue # 554

Merged


Add a new trait to handle post types

Context

Summary
A refactor to remove LOTS of repeated code and add a cleaner way of handling post type objects in our code.
Relevant Technical Choices
A trait is used, as an object doesn't make a tonne of sense here, as it does handle state and these are just helper functions.
To-do

User-facing changes

Testing Instructions

QA

[ ] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

UAT

[ ] UAT should use the same steps as above.

This PR can be tested by following these steps:

Reviews
Does this PR have a security-related impact?

Does this PR change what data or activity we track or use?

Does this PR have a legal-related impact?

Checklist

[ ] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[ ] I have verified accessibility to the best of my abilities (docs)
[ ] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[ ] I have added a matching Type: XYZ label to the PR

Fixes #

@swissspidy If you don't like naming, just create a PR to fix it. Commenting on a merged PR, doesn't make a lot of sense to me.

I'm currently going through all the notifications and review requests from the past two weeks and leave feedback where I see fit. Better than not saying anything at all :-) I don't have the bandwidth to create PRs for everything though.

I don't have the bandwidth to create PRs for everything though.

Okay then, what should be done about this feedback?


Move test data to CCBRPIPELINER testdata location

Describe what enhancement you'd like to see.
Need to store all testdata in standard location
Update test scripts to pull data from this location

completed: /data/CCBR_Pipeliner/testdata/CARLISLE


Inserting snippet that starts with - inserts two --

Clone https://github.com/dbaeumer/vscode-snippet-insert.git
start the extension
open a text file
type -r
this should bring up code compete with one entry -record ....
select that item
you get

--record(name, {field = Value :: Type()}).

Observe the two -- at the beginning.

@jrieken is this cased by word delimiters (e.g. - treated as such)

@jrieken is this cased by word delimiters (e.g. - treated as such)

Yeah, likely caused by the word definition. There is no range defined for the item and it's likely that - isn't part of a word or a word start. You can tell by the highlights, I guess it only highlights r, not -r

Yeah, likely caused by the word definition. There is no range defined for the item and it's likely that - isn't part of a word or a word start. You can tell by the highlights, I guess it only highlights r, not -r

Yes, only r is highlighted. Thanks for the clarification.

Yes, only r is highlighted. Thanks for the clarification.


Infinite values in generated images

For generating new images, I sample z from a zero mean and 0.6 standard deviation normal distribution and feed it to the network with reverse=True argument.
But in many images, there are plenty of values greater than 1, even Inf value!
How can I handle this issue? What is the problem?
Thanks.

Sorry, I'm not sure what caused the problem.
What does the generated images with abnormal values looks like?

Actually, I find the part that value explosion occurs.
It happens at module.py at line 53, when it scales the input by torch.exp(logs).
The Inf value often happens at layer around 80 during forward pass (reverse=True).
then the generated image with negative inf would be something like it (clamped between [0,1]):

I am getting the similar high values. @isharifi have you been able to resolve the issue?

I am getting the similar high values. @isharifi have you been able to resolve the issue?

Unfortuntely, the problem has not been resolved.
@chaiyujin, Do you have any idea?

@isharifi Sorry for the delay. I'm busy with my own project.
It may be caused by some invalid numerical operation, I think.

There may arise a numerical issue of division by zero in https://github.com/chaiyujin/glow-pytorch/blob/487a6b149295f4ec4b36e408f63604c593ff2031/glow/models.py#L93 when there are zero elements in the sigmoid output.
For me the following code snippet triggers the division by zero (running an unconditional generation):
pt.cuda.manual_seed_all(16)
pt.manual_seed(16)
glow(reverse=True)

I couldn't reproduce it running conditional generation though.
A possible fix would be elementwise adding a small value to the scale before division


The example only works when you are not usign PKCE

The OktaProvider should support PKCE, since its the most secure way to implement an OpenID-OAuth flow for an SPA. The only way to implement that using nextAuth is by setting a custom provider.

@AgusMarquezUala good catch. Yet we still can't get it to work here, even with a custom provider. Every time the session doesn't seem to get set properly. Did you get it to work? If yes, do you mind sharing your solution?


Name convention changes

For collecting changes related to the naming conventions

@Jazcash something changed and now there are conflicts I don't understand. I don't want to end up undoing your work so input is welcome :)

Should've merged it when you had the change! 😛 I'll take a look


Texture Pack Issues

Sorry if this isnt the right place for this, but I thought this might be the best place to get an reliable response. Im trying to run Sphax x64 on RR3 and it works with the majority of the other mods, but it dosent seem to be affecting certain machines and blocks from mekanism. Attached are pictures of what the machines look like, and a couple of shots of the texture pack folder itself

https://gyazo.com/c2f68c142fbb82fe2693d8ed51d3cad5
This is the texture that should be displaying for the enrichment chamber
https://gyazo.com/feacd55ce8eb44e1ffcdd5f2ed075f9d
Many Thanks for any help

Texture pack issues are not my fault and should be reported to the pack's owner.

They just haven't updated the UV maps / models to Mek 9. It needs to be updated, probably.


Upgrade protobuf to version containing ObjectiveC and csharp

This time to a version that actually contains those (version in #1654 turns out to be too old).
python will still depend to the latest released version 3.0.0a2 pip package, but csharp and objectiveC need to be able to access changes that were done after the last release of protobufs, so I set third_party/protobuf to head of prototobuf upstream.

Commit log message has a too-long subject line (chris.beams.io/posts/git-commit/#seven-rules).
Tell me more about this? It looks like it updates only a subset of what was updated in #1654? So some points of the codebase will depend on 3.0.0-alpha-2 and others will depend on 3.0.0-alpha-2+plus-some-unversioned-forward-progress-taken-off-master-branch-of-protobuf? That's okay, but should be described in detail in your commit log message.

changed the commit message and updated the PR comment to explain what I am doing.
Travis is green now.