Permissions missing from GKE quick-start

Page: /setup/quickstart/halyard-gke/
Feedback: I think the GCS service account section might be missing the command to give it the storage admin role, such as:
gcloud projects add-iam-policy-binding $PROJECT \
    --role roles/storage.admin --member serviceAccount:$SA_EMAIL

...in this Downloading Credentials documentation. Following the instructions yields:
+ Get current deployment
  Success
+ Get persistent store
  Success
- Edit persistent store
  Failure
Problems in default.persistentStorage.gcs:
! ERROR Failed to ensure the required bucket
  "spin-f6c2f8aa-82a2-45eb-a381-30cd94899142" exists:
  com.google.api.client.googleapis.json.GoogleJsonResponseException: 403 Forbidden
{
  "code" : 403,
  "errors" : [ {
    "domain" : "global",
"message"<EMAIL_ADDRESS>does not have
  storage.buckets.get access to spin-f6c2f8aa-82a2-45eb-a381-30cd94899142.",
    "reason" : "forbidden"
  } ],
"message"<EMAIL_ADDRESS>does not have
  storage.buckets.get access to spin-f6c2f8aa-82a2-45eb-a381-30cd94899142."
}

- Failed to edit persistent store "gcs".

@betandr , the doc you link that gives the error is /setup/storage/gcs/ , but the quickstart instructions you're using do include the storage.admin policy binding in this section. Did you follow the quickstart, or did you skip over to the gcs setup at some point and try to do it there?

Oh you're quite right. That's odd because I followed those instructions and the service account didn't have the permissions until I used the credentials I created from other instructions worked. It's probably PEBKAC, I guess, but I raised it because somebody else mentioned it on Slack and had the same problem. I'll try to go through the instructions again and post something more appropriate if I still have an issue.

Crazy enough, I'm using a script from an old run through of a spinnaker install and I'm bumping into this.  I've re-checked to ensure I've setup permissions, re-created service accounts and keys and still bump into this.
 I double check in the GCP console and roles/service-account/key looks correct.

Even with a manually created bucket, still cannot execute the hal command.
kross@halyard:~/setup$ hal config storage gcs edit \
>     --project ${GOOGLE_CLOUD_PROJECT} \
>     --bucket ${GOOGLE_CLOUD_PROJECT}-spinnaker \
>     --bucket-location us \
>     --json-path ${SPINNAKER_SA_DEST} \
> \
> && hal config storage edit --type gcs
+ Get current deployment
  Success
+ Get persistent store
  Success
- Edit persistent store
  Failure
Problems in default.persistentStorage:
- WARNING Your deployment will most likely fail until you configure
  and enable a persistent store.

Problems in default.persistentStorage.gcs:
! ERROR Failed to ensure the required bucket
  "my-project-unique-name-spinnaker" exists:
  com.google.api.client.googleapis.json.GoogleJsonResponseException: 403 Forbidden
{
  "code" : 403,
  "errors" : [ {
    "domain" : "global",
"message"<EMAIL_ADDRESS>does not have
  storage.buckets.get access to my-project-unique-name-spinnaker.",
    "reason" : "forbidden"
  } ],
"message"<EMAIL_ADDRESS>does not have
  storage.buckets.get access to my-project-unique-name-spinnaker."
}

- Failed to edit persistent store "gcs".

Also having the same problem as @rosskevin but with AWS

I had an idea over the weekend. I have deleted/recreated the service account several times. I narrowed this issue to a GCP issue alone by trying to use the service account to read/write to a bucket (gsutil -e). I recreated the permission problem seen with the hal command.
My theory (not confirmed yet) is the delete/recreate of the service account is the root cause, and my service account credentials of the same name are not being propagated/re-propagated...e.g. cached.  Waited over this weekend and will try tomorrow.
One way to circumvent the cache to prove this right now would be to change the service account name. I'm AFK so can't until tomorrow.

I have now confirmed the problem and a workaround.  I retried the service account I created on Friday (3 days ago) and it still yields 403 forbidden.
Cause (GCP bug):
Creating and deleting/recreating a service account with the same name
Symptom
403 forbidden when using for an operation
Workaround
Change the name of the service account var for the script, recreate, and test with:
# revoke any current credentials to be paranoid
gcloud auth revoke
# configure your new spinnaker key
gsutil config -e
# try a bucket operation
gsutil cp foo.txt gs://my-project-unique-name-spinnaker

It seems GCP has a credential problem when the generated email is identical.


Attack Tests, dep updates, license badge

Working tests for both attacks on macOS and Ubuntu
Minor dependency updates
Fixed bugs in installation test
Add license badge to README

Tests failing on account of test helpers not being present (added as part of the PR).


Autoscale formula metrics for number of task slots

Feature Request Description
My Azure Batch-based application uses a constraint-based scheduling convention wherein task slots represent vCPUs. If I have a pool using a 4-vCPU VM size then I set "Task slots per node" to 4, and when I submit a task that needs access to 2 vCPUs I set the task's "Required slots" to 2.
I have been unable to get this scheme to work reliably with autoscaling, because the autoscale formula language is unaware of tasks' task slot requirements. Information on task slots per node is available in the $TaskSlotsPerNode variable, but there does not appear to be any information on the slot requirements of existing tasks. This means that any autoscaling formula implicitly bakes in the assumption that every task requires exactly one slot.
Describe Preferred Solution
I think the ideal solution is to introduce a task-slot-wise version of each task metric variable. This might look like:

Existing task-wise metric
New task-slot-wise metric

$ActiveTasks
$ActiveTaskSlots

$RunningTasks
$RunningTaskSlots

$PendingTasks
$PendingTaskSlots

$SucceededTasks
$SucceededTaskSlots

$FailedTasks
$FailedTaskSlots

Describe Alternatives Considered
The current task-wise metric variables could be changed to reflect task slots instead of whole tasks. This would be a breaking change.
Additional Context
The proposed addition of task-slot-wise metrics would be analogous to the addition of the TaskSlotCounts object to the response of the Job_GetTaskSlots operation made in API version 2020-09-01.12.0.

Hi @alfpark
Does "known issue" mean this will not get fixed? Sounds more like a feature request to me and a very valueable one!
I have another use case where I run into the exact same problem.
I have a Job which has tasks which depend on each other, there are one task per job which is more compute intense then others, I usually set that one to max slots per node so that it runs on one VM with all the resources available.
The dependent tasks later are 10x more tasks but they run faster and only need one CPU, so, in that case I set slots to 1 for those tasks.
Now, its impossible to calculate how many nodes are actually needed with auto scaling...
So, my question would be, will this feature be added soon, to use task slot variables within the formula instead of tasks?
Or is there another solution or workaround for those use cases I should consider instead of auto scaling?
Thanks,
Michael


[Spike] Investigate having an automated way to generate client SDKs for TRE APIs

Investigate an automated way to generate a client SDK, that can execute calls to the TRE API, including workspace specific API endpoints.
Given that the TRE API, will gradually have new features, this client SDK, should be autogenerated using tools like AutoRest or OpenAPI Generator
This process should be documented. Also consider/investigate having SDK automatically genereted with GH Actions

Addressed in #1879


Creating Flutter Project: ProcessException: No such file or directory

After creating a new Flutter project I always get the following error:
Creating spring_queen_0369...
/home/dmta/flutter/spring_queen_0369/.gitignore
/home/dmta/flutter/spring_queen_0369/pubspec.yaml
/home/dmta/flutter/spring_queen_0369/README.md
/home/dmta/flutter/spring_queen_0369/lib/main.dart
Running pub get...
ProcessException: No such file or directory
  Command: pub get
Exception:
dart:io                                                         Process.start
package:sky_tools/src/commands/init.dart 48                     InitCommand.run.<async>
dart:async                                                      Future.Future.microtask
package:sky_tools/src/commands/init.dart                        InitCommand.run
package:args/command_runner.dart 178                            CommandRunner.runCommand.<fn>
dart:async                                                      Future.Future.sync
package:args/command_runner.dart 131                            CommandRunner.runCommand
package:sky_tools/src/commands/flutter_command_runner.dart 108  FlutterCommandRunner.runCommand
package:args/command_runner.dart 105                            CommandRunner.run.<fn>
dart:async                                                      Future.Future.sync
package:args/command_runner.dart 105                            CommandRunner.run
package:sky_tools/executable.dart 60                            main.<async>.<fn>.<async>
package:stack_trace                                             Chain.capture
package:sky_tools/executable.dart 59                            main.<async>

in a red box in the top right corner of the screen.
And below that a blue box saying:
Pub has never been run for project spring_queen_0369/. Run 'pub get'?

And when I click Run pub... it always works...
So, is there something wrong with my setup, preventing the red box from finding the pub executable?

I'm not sure exactly what's going on here, but it does look path related.
In the first case, atom is able to launch the flutter process. That process is not able to launch pub however. So the flutter process for some reason didn't get your system PATH setting.
For the 2nd case, we execute the pub tool out of the sdk directly. So it's not dependent on a path setting.

Like in #528 starting atom from bash resolved the problem!
Sorry again!

I'll close this and keep the other open. Thanks for the reports!


[Event Request] table 4812 "Intrastat Report Line" - OnSetIntrastatReportHeaderFilters

Describe the request
Please add a new event OnSetIntrastatReportHeaderFilters to table 4812 "Intrastat Report Line", to method SetIntrastatReportHeaderFilters.
  procedure SetIntrastatReportHeaderFilters(var IntrastatReportHeader2: Record "Intrastat Report Header")
  var
      IntrastatReportHeader3: Record "Intrastat Report Header";
  begin
      IntrastatReportHeader3.Get("Intrastat No.");
      IntrastatReportHeader2.SetRange("Corrective Entry", false);
      ...
      IntrastatReportHeader2.SetRange(Type, IntrastatReportHeader3.Type);
+     OnSetIntrastatReportHeaderFilters(IntrastatReportHeader2,IntrastatReportHeader3);
  end;

+ [IntegrationEvent(false, false)]
+ local procedure OnSetIntrastatReportHeaderFilters(var IntrastatReportHeader2: Record "Intrastat Report Header"; IntrastatReportHeader3: Record "Intrastat Report Header")
+ begin
+ end;

Additional context
Just to add additional filters.
Internal work item: AB#554335

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 27480.


yo

[Enter feedback here]
is loqi

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

ID: 9a275bee-cee3-f0f4-407f-f4d5971f4a73
Version Independent ID: 918a9e07-6982-630c-2d27-c5d5561f2157
Content: Override webpage resources with local copies (Overrides tab) - Microsoft Edge Development
Content Source: microsoft-edge/devtools-guide-chromium/javascript/overrides.md
Product: microsoft-edge
Technology: devtools
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Closing as incomplete.


[CRYPTIC]: using an arc in a pattern results in an internal engine error

Paste minimal KCL source that produces a cryptic error
startSketchOn('XY')
|> startProfileAt([10, 0], %)
|> arc({
angle_start: 180,
angle_end: 180,
radius: 1.5
}, %)
|> close(%)
|> patternCircular2d({
arcDegrees: 360,
center: [0, 0],
repetitions: 5,
rotateDuplicates: true
}, %)
Expected Behavior
No response
Additional Context
No response

core issue here is the angle start/end is 180, which is 0.


readsignal fails to retrieve data

The function fails to retrieve data with the following error:
read -1, less than 921600 requested!!!

The edf file for which this happens is readily available from physionet.org (chb01_01.edf)

Are you certain the file is an error-free EDF file?
Please provide a link to a compliant EDF file.

Found a chb01_01.edf file in the CHB-MIT Scalp EEG Database.

This works for me:
import pyedflib

signals, signal_headers, header = pyedflib.highlevel.read_edf("chb01_01.edf")

How to reproduce the error?

That was the file I was using, I see pyedflib.highlevel.read_edf works but I just wonder why readSignal on the EdfReader class won’t work.
So, if I create an instance of EdfReader with that file and I try to use the readSignal(chn=1) method it fails.
I have switched to using the “mne” library anyway which has similar functionalities but I would recommend attempting to solve this issue.
Best regards,
Ruben Ruiz-Mateos Serrano
MRes+PhD Connected Electronic and Photonic Systems
Department of Engineering
University of Cambridge / UCL
From: Dimitri Papadopoulos @.>
Sent: 27 June 2023 11:15
To: @.>
Cc: Ruiz-Mateos Serrano, @.>; @.>
Subject: Re: [holgern/pyedflib] readsignal fails to retrieve data (Issue #228)
This email from @.*** originates from outside Imperial. Do not click on links and attachments unless you recognise the sender. If you trust the sender, add them to your safe senders listhttps://spam.ic.ac.uk/SpamConsole/Senders.aspx to disable email stamping for this address.
This works for me:
import pyedflib
signals, signal_headers, header = pyedflib.highlevel.read_edf("chb01_01.edf")
How to reproduce the error?
—
Reply to this email directly, view it on GitHubhttps://github.com/holgern/pyedflib/issues/228#issuecomment-1609209529, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKSEWTKEX6I67TNUM5B2MDLXNKXCHANCNFSM6AAAAAAZULQ4VU.
You are receiving this because you authored the thread.Message ID: @.***>

I cannot reproduce the issuse, I assume the problem is with your code.
import pyedflib
file_name = '/home/simon/Downloads/chb01_01.edf'
with pyedflib.EdfReader('chb01_01.edf') as f:
    n = f.signals_in_file
    signal_labels = f.getSignalLabels()
    print(n, signal_labels)
    f.readSignal(1)


[pre production/stable]   better names?

Should the name of anything change before we transition to production/stable?

Are some public class names too generic? e.g. API and Error.
Could keyword argument names be better?
Could field names in the JSON file be better?
How about method names?
Is the library Pythonic?
Does it adhere to PEPs.?
Would Guido approve?

I am satisfied now, how about you?

No feedback received.


make the builds great again

to fix the travisci build + make it more representative of production
TODO:

[x] add a travis image to README.md

@BetterWorks/enterprise

changes lgtm 🇺🇸


[feature] Add Player::getDevice

Completed in https://github.com/EndstoneMC/endstone/commit/da46104e6a747a37ce1e7cd72c0ccb83b51696fa.


Updated C/C++ workload name

The workload for C/C++ development in VS Build Tools 2019 installer is called Desktop development with C++.
The old nave is probably from the previous release.
A small update that might help avoid confusion.

@ainrd : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Rentals functionality

Still needs a sign off on the strings and some testing, seems to work well for me though


See the filesystem together with the Sas Content

Is your feature request related to a problem? Please describe.
I need to access some sas file that are within the filesystem instead of the Sas Content
Describe the solution you'd like
See in the EXPLORER the filesystem together with the Sas Content, as Sas Studio does.
Describe alternatives you've considered
NA
Additional context
NA
Environment
SAS version Viya 3.5

VS Code by itself can connect to a remote location and show remote file system in it's Explorer. Refers to https://code.visualstudio.com/docs/remote/remote-overview
Will it meet your need?

@scnwwu Unfortunately I don't think so, because the file system shown in SAS Studio alongside the SAS Content is very hard to be connected to being on some volume in the kubernetes cluster in Viya 4. I don't think there is a secure and easy way to convince the Sas Cloud cysec to directly expose that internal volume on ssh in the network.

Ben Gow (a customer) wanted to make sure that when we add this feature that it also honors the fileNavigationCustomRootPath and serverDisplayName config properties, same as SAS Studio. I agree, this makes sense, so that there is a consistent experience between the extension and SAS Studio. So, please make sure to consider this in the implementation.
I am debating whether it makes sense to have separate config properties with the same names, but specific to the extension, or whether the extension should just re-use the existing SAS Studio config properties. The former would allow admins to set the properties separately for the extension and SAS Studio if they wanted them to be different for some reason. However, that would also force them to set both sets of properties if they wanted them to be consistent in the extension and SAS Studio (unless one overrode the other). For this reason, I think the latter makes more sense for now... re-use the SAS Studio config properties, so the extension and SAS Studio are consistent by default. If we get a request later, we could consider adding extension-specific properties that (if set) would override the SAS Studio config properties.

I agree that this would be a good feature for the extension. It can help with determining whether issues are isolated or not to SAS Studio when working with the filesystem or configured NFS mounts by having another application that can also access those locations.

This should likely be done in coordination with the same feature request for V9 (#889), since they should have a largely consistent experience.

This is currently a work in progress (see https://github.com/sassoftware/vscode-sas-extension/pull/1203). Since this work is pretty dense, we've decided to split things up in the following way:

Implement SAS file system for viya
Implement SAS file system for ITC-based connections (IOM/COM)
Implement favorites SAS file system with viya connections

Additionally, a bug was found when working on this feature set around downloading a mix of files/folders. A separate PR will be created to resolve that.

Delivered as part of #1203 in milestone 1.12


_WD_SetElementValue - Add remark

Pull request
Proposed changes
Closes #466
Checklist
Put an x in the boxes that apply. If you're unsure about any of them, don't hesitate to ask. We are here to help!
This is simply a reminder of what we are going to look for before merging your code.

[x] I have read and noticed the CODE OF CONDUCT document
[x] I have read and noticed the CONTRIBUTING document
[ ] I have added necessary documentation or screenshots (if appropriate)

Types of changes
Please check x the type of change your PR introduces:

[ ] Bugfix (change which fixes an issue)
[ ] Feature (change which adds functionality)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (functional, structural)
[x] Documentation content changes
[ ] Other (please describe)

What is the current behavior?
When using $_WD_OPTION_Advanced in _WD_SetElementValue, $sValue is not escaped
What is the new behavior?
Behavior is unchanged. Added remark to document that the user is responsible for formatting the data prior to calling _WD_SetElementValue in Advanced mode.
Additional context
Add any other context about the problem here.
System under test
Please complete the following information.

OS: [e.g. Windows 10]
OS Arch.: [e.g. X64]
Browser [e.g. firefox]
Browser version [e.g. 96.0.3]

Fine here


Fixes docblock for transaction function

After https://github.com/propelorm/Propel2/pull/1654 was introduced, docblocks in interface and implementation were not aligned with each other.

Yes, I missed that place, thanks 👍


[EXTENSION] getting revolut data

Allow the extension sidebar to redirect the user to revolut. then extract the Revolut ID from the webpage to make it accessible for the zkRamp app

can I work in this one?

Yeah I think you can get it, I'm having some personal problems to finish this issue.
I already implemented some parts, if you want some help please tag me on telegram (same user there)

Ok bro

Hey @0xChqrles , I am interested in solving this issue


Disabled anchor buttons are still usable

Bug
A Button that becomes an anchor element (with a href prop) will open a link even when disabled.
Expected behavior: href be removed to disable the anchor functionality if the disabled $prop is true
Reproduction
https://stackblitz.com/edit/sveltejs-kit-template-default-z697yf?file=src%2Froutes%2F%2Bpage.svelte

Thanks. The next version will fix this.

Solved in v0.46.0.


Does robots.txt exclusion code maintain different records for http:// and https:// protocols?

Per https://developers.google.com/search/reference/robots_txt

The short answer is yes — this has been handled by writing a robots.txt parsing library:
https://github.com/benjaminestes/robots


Since update to 0.5 AWS EU-Requests-Tier1 increased

Since update to v0.5.0 I see a increase of the request (EU-Requests-Tier1) in my S3 AWS Cost explorer by factor 10.

Version: v0.5.0

Oh, it's likely due to the new backup related features. We currently polling backup target every 5 minutes (by default) to update the latest backup.
You can set the settings BackupStorePollInterval to a large number to delay the polling. Though if you're on disaster recovery setup for multicluster, this won't be favorable. We can add a way to disable the auto polling e.g using value 0 to disable the feature for single cluster users.
Sorry for the incurred additional cost, we will make sure next time we change things may impact the S3 bill, we will put it in our release note.

I alreay set this value to 3600.
It wasn't a real problem for my bill. It just increased from 0.01 cent/day to 0.1 cent/day. It's just a test cluster with 7 volumes so it doesn't really matter but for people with a lot of backups this may be relevant.
I don't understand the point with "disaster revocery setup".
I already tested in version v0.4.1 a recovery from S3 on a new clean rancher-cluster with a new longhorn install. It worked fine for all the 7 volumes I have. Without this "BackupStorePollInverval". I did all the steps manually.

Start rancher docker
login
install driver
create cluster and some VMs (3 nodes)
after it was ready I deplyoed longhorn from apps
after it was deployed I opend the longhorn GUI -> settings
configured the S3 storage like before
go to "Backup" in longhorn GUI
Select image to restore and define the name of the volume (I set the same as before on the old cluster)
attach the volume to a node to start restore
after restore deployed the deployment on rancher/k8s

@vistalba Yeah, the disaster recovery feature is an upgraded version of the current process. The restoration is copying the data from the backup store, which can take a long time. It's undesirable if the volume is very big and needs quite some time (e.g. hours) to be copied over.
The disaster recovery feature is designed so the volume in the backup cluster will constantly poll the backup store, and if any new backup for the volume appears, the volume in the backup cluster will automatically start the restoration process, and it would be incremental so it only copies the data that changed. In this way, once you have an accident, you just need to activate the volume in the backup cluster, the time spent on restoring the volume should be much less or none (depends on if there is any data hasn't be restored yet), so you can start your workload much faster. This is designed to achieve a better RTO by reducing the volume restoration time.

@yasker thanks for explaination. Nice feature.
So I think it is useful if this feature isn't used (e.g. there is no backup cluster in place) it can be disabled.
Maybe it is also a good idea to better explain what the setting is doing. My backup-plan creates one backup per day - if I understand you right - there is no sense to poll the store more often. So maybe there is a way to calculate a good/the best value from the sheduled backups. (something like the shortest timespan between two backups of the same volume and the timeplan over all volumes).
e.g.:
If all backups are done within 01:00am and 05:00am just once a day. Poll just once in 24h maybe at 05:30am
If all backups are done all around the clock every hour. Poll just every 3600 seconds.

@vistalba In your case, this setting should be disabled. The setting BackupStorePollingInterval is useful in the backup cluster. If it's in the same cluster, any backup and list backup operation has a side effect to update the related Last Backup field, so you don't need to poll the store if you're making backups from the same cluster.

We will add a disable option to the BackupStorePollingInterval. If it's 0, it's disabled.

Steps to test:

Create 2 K8S clusters (A & B), install longhorn, and create longhorn StorageClass
Create an AWS S3 bucket, and configure backup target to use it for both clusters.
On Cluster B Settings page:  set Backupstore Poll Interval to 0
On Cluster A: Create Longhorn volume XA, attach it to a node, and create a backup.
On Cluster B, backup page, click Create Disaster Recovery Volume and create a DR volume XA_DR

Expected result: XA_DR's Last Backup At value on Volume page will not be updated since Backupstore Poll Interval is disabled on Cluster B,

On Cluster B Settings page:  set Backupstore Poll Interval backup to nonzero value
On Cluster A: Create a new backup of volume XA.

Expected result: XA_DR's Last Backup At value on Volume page will not be updated

On Cluster B Settings page: set Backupstore Poll Interval backup to nonzero value
On Cluster A: Create a new backup of volume XA.
Expected result: XA_DR's Last Backup At value on Volume page will not be updated

Actually when you set the backupstore poll interval to nonzero value, the backup polling should be enabled again and the XA_DR's Last Backup At value on volume page should be updated

it was a typo, i updated it, thank you.


Create cicdServerSetup.sh

intended to be a script that could be run to set up a bare Linux Mint machine for running CICD tasks

going to be a while before I am ready to merge this. Closing until then.


docker.ENTRYPOINT in shell form doesn't work with docker.CMD

I believe docker.ENTRYPOINT in shell form doesn't properly escape spaces (and maybe other special symbols) in resulting example string: sh -c echo test1 test2
Steps to reproduce:
$ cat > werf.yaml <<EOF
project: test
configVersion: 1
---
image: test
from: alpine
docker:
  ENTRYPOINT: echo
  CMD: ["test1", "test2"]
EOF
$ werf build
$ docker run --rm werf-stages-storage/test:$TAG_OF_BUILT_STAGE
# echo nothing
$ sed -i 's/echo/["echo"]/' werf.yaml
$ werf build
$ docker run --rm werf-stages-storage/test:$TAG_OF_BUILT_STAGE
test1 test2   # works when docker.ENTRYPOINT in exec form

Tested on werf v1.1.9+fix2
Related #1680

werf behavior is similar to docker’s (https://prnt.sc/rzyjqt)
ENTRYPOINT: echo
$ cat werf.yaml
project: test
configVersion: 1
---
image: test
from: alpine
docker:
  ENTRYPOINT: echo
  CMD: ["test1", "test2"]

$ docker inspect 4875326acdcd --format="entrypoint: {{ .Config.Entrypoint }} cmd: {{ .Config.Cmd }}"
entrypoint: [/bin/sh -c echo] cmd: [test1 test2]

$ docker run --rm 4875326acdcd # /bin/sh -c echo

ENTRYPOINT: ["echo"]
$ cat werf.yaml
project: test
configVersion: 1
---
image: test
from: alpine
docker:
  ENTRYPOINT: ["echo"]
  CMD: ["test1", "test2"]

$ docker inspect 4e3039c626cc --format="entrypoint: {{ .Config.Entrypoint }} cmd: {{ .Config.Cmd }}"
entrypoint: [echo] cmd: [test1 test2]

$ docker run --rm 4e3039c626cc # echo test1 test2
test1 test2

Wow, that's stupid, Docker docs indicate just that:

The shell form prevents any CMD or run command line arguments from being used

Would be nice to indicate that the exec form can be used at all and would be specified here by default:
https://github.com/flant/werf/blob/7dadcf3f0f7b0360994fa79092a51b400156c868/docs/_data/stages.yml#L285-L286

I agree with you it is ambiguous behavior.
However, this is the logic of interaction between ENTRYPOINT and CMD instructions in Docker and werf just repeat that.

The issue is just your docs show entrypoint and cmd used together in shell form, which doesn't work. Also, in Docker docs it is explicitly specified that the exec form is preffered and they have no (working) examples with both entrypoint and cmd in shell form.

Thanks for feedback, we'll add notes in comments soon 👌🏼


Change from MFEM's command line arguments to CLI11

Axom has built in support for CLI11.  We should switch to that and away from MFEM's.  CLI11 is more industry standard and is C++ vs C (ie std::strings vs char*).

There is a slight difference in functionality: CLI11 only supports single-character short names (those preceded by a single -).   From a usability standpoint, does it make more sense to shorten the short names, or to make them long names, e.g., for refining the mesh in serial, should -rs change to -s or --rs?


[Bug?]: useSession().data value inconsistent

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
Sometimes session.data.userId is undefined when expected to be defined.
Expected behavior 🤔
Expecting session.data to be consistent e.g. logged in user always has data.userId.
Steps to reproduce 🕹
I've made a repository based off the with-auth example/template that reproduces the bug. I've just added a "create project" function and "list project" function, the latter is what causes the issue.
And I've got a video in the readme with a demonstration.
https://github.com/metruzanca/solidstart-getSession-bug-repro
Context 🔦
Trying to build my next app with solidstart beta 2, but this issue is making it impossible to reliably get any data from the server.
Had good success with beta 1 and wanted to use the latest. I would like to keep this project using beta 2 and not have to fall back to v1.
Your environment 🌎
Tested on a fresh Ubuntu LTS 20 desktop with latest node v20.10 and pnpm v8.13, https://github.com/metruzanca/solidstart-getSession-bug-repro/blob/main/package.json

I played around with this a bit and could replicate. No idea what is happening tho

Learned that whichever createAsync is executed second, is the one that has the unreliable session.data value.

I think from what I saw it was just the 2nd call to getSession regardless of createAsync but I could be wrong

Out of curiosity I dug into useSession and found that this might be an issue with h3 since if we jerry-rig this out of its internals, we seem to reliably get sessionData.
async function getSessionData<SessionDataT>() {
  const event = getRequestEvent()!
  const cookie = getCookie(event, "h3")!
  const { data } = await unsealSession(
    event as H3Event<EventHandlerRequest>,
    { password: SESSION_SECRET },
    cookie
  )
  return data! as SessionDataT
}

By no means have I done any rigorous testing, nor do I really know what I'm doing with h3.

If this workaround is comprehensive, I see some other stuff has been left out, is there no reason to have that? We should keep whatever's stable in solid-start's flow. So if this works we can use this as the exported version of useSession()

@nksaraf I actually have made a more complete version, since getCookie does seem to not return expected value on fresh users.
export async function getSessionData(): Promise<Record<string, any>> {
  const event = getRequestEvent()!
  const cookie = getCookie(event, "h3")
  if (!cookie) return {}

  const { data } = await unsealSession(
    event as H3Event<EventHandlerRequest>,
    { password: SESSION_SECRET },
    cookie
  )
  if (data) return data
}

This seems to work pretty reliably in my app, though I'm still using useSession for updating the session, as that seems to work as expected.

This may be offtopic but I'm trying to save data into session:
export async function signin({accessToken, profile}: { accessToken: string, profile: string }) {
    try {
        const session = await getSession();
        await session.update((d: UserSession) => (d.accessToken = accessToken));
        await session.update((d: UserSession) => (d.profile = profile));
    } catch (err) {
        return err as Error;
    }
}

And I get response back from api with the error in Set-Cookie header:
Set failed, cookie was too large.
Actual length of access token is 244, profile is 334.
Do I have an option to compress data?

And I get response back from api with the error in Set-Cookie header:
Set failed, cookie was too large.
Actual length of access token is 244, profile is 334. Do I have an option to compress data?

moved this to new issue #1260.
The existing issue should be fixed when #1258 is merged and release.


[wayland-client error] Attempted to dispatch unknown opcode 0 for wl_shm, aborting

On Fedora gnome-shell-3.38.1-2.fc33.x86_64
$ git clone https://github.com/BrandtM/colourado.git
$ cd colourado
$ cargo run --example preview pastel 16 adjacent
[wayland-client error] Attempted to dispatch unknown opcode 0 for wl_shm, aborting.
Aborted (core dumped)
$ cargo tree -i wayland-client
wayland-client v0.21.13
├── glutin v0.19.0
│   └── pistoncore-glutin_window v0.52.0
│       └── piston_window v0.87.0
│           [dev-dependencies]
│           └── colourado v0.2.0 (/home/harald/git/colourado)
├── smithay-client-toolkit v0.4.6
│   └── winit v0.18.1
│       └── glutin v0.19.0 (*)
├── wayland-protocols v0.21.13
│   └── smithay-client-toolkit v0.4.6 (*)
└── winit v0.18.1 (*)

Update to the latest wayland-rs.

Update to the latest wayland-rs.

I am just a user... so, it boils down to pistoncore-glutin_window still using an old version because of:
https://github.com/PistonDevelopers/glutin_window/pull/179
If only there were bugfix releases of glutin-0.22 using newer wayland-client versions.
Or maybe even wayland-client v0.21.14 with a bugfix for this :smiley:

There's no way to fix it with just bugfix release, it requires breaking release, so we can't do anything about it. Suggest the mentioning project to update to the latest winit/glutin, they have all of that fixed.

Thanks!


FIX Debian+Ubuntu: install bash-completion if missing

fixes https://github.com/dokku/dokku/issues/4538
Note: If this PR is just doc changes, please put [ci skip] in the body that way tests do not run.

Mind sending this PR to the right place? :)

done: https://github.com/dokku/dokku/issues/4538


[SPARK-12542] [SQL] support except/intersect in HiveQl

Parse the SQL query with except/intersect in FROM clause for HivQL.

@hvanhovell This patch can't be compiled, failed with:
[error] ~spark/sql/hive/target/scala-2.10/src_managed/main/org/apache/spark/sql/parser/SparkSqlParser_IdentifiersParser.java:11056:  ERROR: too long
[error] 	static final String[] DFA5_transitionS = {
[error] 	                      ^

Merged build finished. Test FAILed.

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

@davies I haven't seen this one before. We can take a look at the generated source...

@davies I just tried this. If you add this rule the identifier parser grows beyond 64K rows which the JVM can't handle. The short term solution is to split the identifier parser (see @viirya's PR).
In the long term we might need to check if some rules aren't blowing things out of proportion.

@hvanhovell Thanks, I will wait for @viirya's PR go first.

Test build #48882 has started for PR 10622 at commit 4ae0ccf.

Test build #48882 has finished for PR 10622 at commit 4ae0ccf.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test FAILed.

Test build #48891 has started for PR 10622 at commit a83d4a2.

Test build #48891 has finished for PR 10622 at commit a83d4a2.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class CatalystQlSuite extends PlanTest

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

Merged build finished. Test PASSed.

LGTM. Would be great for @hvanhovell to take a quick look too.

The following query does not work:
(select 1 as a) union (select 2 as a)

I had not figure out a way to fix this.
PS: It seems that HiveParse is not a good stand point. lots of corner cases do not work.

Let's open a jira ticket to track that and see if some open source contributors can pick it up.

https://issues.apache.org/jira/browse/SPARK-12687

LGTM

Merging this in. Thanks!


2 commits: lowLevelError + testExpectEncodeStg

feat(stdError): add lowLevelError
Motivation
Cover INVALID opcode cases, such as trying to directly access an element of an array in another contract that is oob, or when there was an error while decoding the error message (0x).
Showcase with arrays
contract C {
    bool[] public bools;

    constructor() {
        bools.push(true);
    }
}

contract TestC is DSTest {
    function testLowLevelError() public {
        Vm vm = Vm(HEVM_ADDRESS);
        C c = new C();
        // P.S. No, we cannot use `indexOOBError` here
        vm.expectRevert(stdError.lowLevelError);
        c.bools(1);
    }
}

test(stdError): add testExpectEncodeStg
Add a test for encodeStorageError. This was marked as TODO.
/// INSIDE TESTING CONTRACT

function testExpectEncodeStg() public {
    vm.expectRevert(stdError.encodeStorageError);
    test.encodeStgError();
}

/// INSIDE DUMMY CONTRACT

bytes someBytes;

function encodeStgError() public {
    assembly {
        sstore(someBytes.slot, 1)
    }
    bytes memory b = someBytes;
}

lmk wen ready for review

if it isn't draft, it is ready :)

if it isn't draft, it is ready :)

plz request reviews going forward regardless :)

i can't formally request reviews as a non-collaborator
will @ u in the future, if that's what you meant


bug: Keyboard Scrolling Issue

Bug Report
Ionic version:

[x] 4.x
Current behavior:
If I have a page that has a bunch of inputs on it in the list, with nothing below it, and I click on the last input or two in the list it will be hidden by the keyboard and will not scroll into the viewable area until I start typing.
But, if I pad out the area under the list of inputs, say with a few paragraphs of cheesy text, I don't have the same issue. In that case, the final input scrolls into view immediately upon clicking it.
It seems like perhaps there is some issue with the code that tries to handle this scrolling, but only if the area that needs to be scrolled to is towards the physical bottom of the page.
Expected behavior:

Scrolling would work fine, even in the first scenario.
Related code:
https://github.com/kensodemann/test-inputs-scrolling
Steps to reproduce:

clone
npm i
ionic cordova build ios
run it on a device
scroll until "Description" is at the bottom of the screen
tap into it to show the keyboard
note that the view did not scroll
type something
now "Description" has scrolled into view

Now uncomment the cheesy text: https://github.com/kensodemann/test-inputs-scrolling/blob/master/src/app/home/home.page.html#L56-L71
Repeat the above steps. When tapping into "Description" to show the keyboard note that the scrolling takes place right away.
Ionic info:

Ionic:

   Ionic CLI                     : 5.2.8 (/Users/kensodemann/.nvm/versions/node/v10.16.3/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.9.0
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.0.0

Cordova:

   Cordova CLI       : 9.0.0<EMAIL_ADDRESS>   Cordova Platforms : ios 5.0.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.1, (and 4 other plugins)

Utility:

   cordova-res : 0.6.0
   native-run  : 0.2.8

System:

   Android SDK Tools : 26.1.1 (/Users/kensodemann/Library/Android/sdk/)
   ios-deploy        : 1.9.4
   ios-sim           : 8.0.2
   NodeJS            : v10.16.3 (/Users/kensodemann/.nvm/versions/node/v10.16.3/bin/node)
   npm               : 6.9.0
   OS                : macOS Mojave
   Xcode             : Xcode 10.3 Build version 10G8

I can tell that I keep my app outdated because I've found a combination of package version that doesn't have that bug...

Still the above issue exist in iOS devices.
Ionic Info:
Ionic:
Ionic CLI                     : 5.2.8 (/usr/local/lib/node_modules/ionic)
Ionic Framework               : @ionic/angular 4.11.2
@angular-devkit/build-angular : 0.801.3
@angular-devkit/schematics    : 8.1.3
@angular/cli                  : 8.1.3
@ionic/angular-toolkit        : 2.0.0
Cordova:
Cordova CLI       : 9.0.0<EMAIL_ADDRESS>Cordova Platforms : android 8.1.0, ios 5.0.1
Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.2, (and 4 other plugins)
Utility:
cordova-res : not installed
native-run  : not installed
System:
Android SDK Tools : 26.1.1 (/Users/apple/Library/Android/sdk)
ios-deploy        : 1.9.4
ios-sim           : 8.0.2
NodeJS            : v10.16.3 (/usr/local/bin/node)
npm               : 6.9.0
OS                : macOS Mojave
Xcode             : Xcode 10.3 Build version 10G8

Possibly related... this issue is a Cordova app, but this Capacitor one sounds very similar: https://github.com/ionic-team/ionic/issues/16797

Thanks for the issue. This was fixed via https://github.com/ionic-team/ionic/pull/20742 and will be available in an upcoming release of Ionic Framework.


AJ-1629 Improve configuration fail-fast behavior

This PR's primary intention is to introduce fail-fast behavior to the Spring runtime when our assumptions about the deployment mode are violated.  This is best done as early as possible in the bean configuration lifecycle, and so happens as part of initializing TwdsProperties.  Due to the very central role of bean config in the codebase (and the way something completely unrelated can fail to run when beans aren't configured properly), and the fact that tenancy + WORKSPACE_ID + data-plane vs. control-plane are somewhat tightly coupled, this PR ended up absorbing much more scope than initially intended, from the following three tickets:
AJ-1629: Require control-plane or data-plane to be set throughout. (Also: Default to data-plane in tests)).
AJ-1630: Respect allow-virtual-collections config setting
AJ-1650: Clean up properties classes
New fail-fast mechanics introduced:
WORKSPACE_ID:

If in single-tenant mode, WORKSPACE_ID is required, and WDS should crash instead of allowing any code to run without it.  This is enforced by restricting all access to this environment variable through the InstanceProperties.workspaceId() accessor, which is designed to never return null.  In fact, InstanceProperties can't be resolved as a Bean at all unless they have a valid WorkspaceId, so this is enforced in depth*** when injected via InstanceProperties.
If in multi-tenant mode, InstanceProperties cannot be auto-configured, and so all beans that depend on it need to be marked as running in @DataPlane only.  Note: this might need to be adjusted to look instead at tenancy-properties if we move to WDS in the LZ, see TODO below
A convenience @Bean for the WorkspaceId annotated as @SingleTenant is provided for direct injection, but it is also subject to the same constraints.

allow-virtual-collections:

This is barely used at the moment and enforced by configuring mutually exclusive @DataPlane PostgresCollectionDao and @ControlPlane VirtualWorkspaceIdDao beans rather than honoring the TenancyProperties; however, in tandem with the WORKSPACE_ID enforcement, a bunch of logic around the virtual collections can be removed in favor of trusting that the presence/absence of InstanceProperties indicates whether we're in a singleTenant deployment or not.  I think this may be addressed by AJ-1655

TODO:

[ ] The coupling between deploymentMode + tenancy + instance properties is probably exacerbated by using the @DataPlane and @ControlPlane annotations to selectively enable/disable beans.  We might be able to reduce this coupling by correctly honoring the booleans in tenancy and instance props, maybe with @ConditionalOnProperty?
[ ] ***Direct access to the workspace-id using @Value('twds.instance.workspace-id') throughout the codebase still circumvents the validation and enforcement that it's non-null, non-blank, and a valid UUID, followup is required to eliminate these references, which will cause unexpected misbehavior in control-plane (AJ-1656)

I'm going to take this back to the drawing board and try to shrink it's scope to improve reviewability and avoid risk of regressions.

Breakaway PR to clean up props and start introducing WORKSPACE_ID guarantees: https://github.com/DataBiosphere/terra-workspace-data-service/pull/583

Closing for now, subsumed by smaller PRs.


[CARBONDATA-1432] Fixed bug when column name in default.value property is not equal to column name in alter command

SDV Build Fail , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/504/

Retest this please

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/521/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/522/

LGTM


logBloom should show 0x0 if empty

not '0' which is what it does now for getBlock 50004. Plus  it's both int he block and the receipt.t

Moved to https://github.com/Great-Hill-Corporation/quickBlocks/issues/48


cifar10-resnet.py: Segmentation fault (core dumped)

Hi,
When I was training the model of cifar10-resnet.py, the code always stopped at this place:
tensorpack/dataflow/dataset/cifar.py, line 45 read_cifar() .
When the loop--for fname in filenames:--gone to the fifth file data_batch_5 and the subloop--for k in range(IMG_NUM): gone o k=5136, something wrong occurred:
...
k = 5135
k = 5136
Segmentation fault (core dumped)

(Here, I print the value of k during the loop). Then the program was over.

I couldn't reproduce this problem. Note that there are multiple threads running so it's unlikely it is a problem with data reading, but more likely to be in training.
Could you try if the simple examples (mnist-convnet.py, cifar-convnet.py) can run?
And how do you install opencv? Are you using anaconda?

I tried the another example cifar-convnet.py, but the same problem occurred.  Before the cifar,  I have test the example HED, it runs well.
I re-downloaded the file cifar-10-python.tar.gz  to exclude that the file is damaged.

I have installed OpenCV 3.1, CUDA 8.0, cuDNN 5.1( without using anaconda ).

How do you install opencv?
And could you try running without GPU to see if the error occurs? (use CUDA_VISIBLE_DEVICES=)

In fact, I traced debug the problem, it's just a problem of data reading. (The program was still in the data loading stage before the training phase, when it occurred.)

I installed the OpenCV according to the guide: OpenCV: Installation in Linux.

Could you post the full log so I can see which stage it was in?

I added some print in the original code:
# tensorpack/dataflow/dataset/cifar.py

...
        for k in range(IMG_NUM):
            img = data[k].reshape(3, 32, 32)
            img = np.transpose(img, [1, 2, 0])
            ret.append([img, label[k]])
            #print k
        print fname, len(ret)
    print "read_cifar done"

Output:
...
/home/swoda/tensorpack_data/cifar10_data/cifar-10-batches-py/data_batch_1 10000
/home/swoda/tensorpack_data/cifar10_data/cifar-10-batches-py/data_batch_2 20000
/home/swoda/tensorpack_data/cifar10_data/cifar-10-batches-py/data_batch_3 30000
/home/swoda/tensorpack_data/cifar10_data/cifar-10-batches-py/data_batch_4 40000
Segmentation fault (core dumped)

Then, I printed the k:
...
5105
5106
5107
5108
5109
Segmentation fault (core dumped)

It seems that the the location of the error is different sometimes.

If this is where it crashes I have no clue now.
python2 -c "from tensorpack import dataset; c = dataset.Cifar10('train')"

should also crash for you if it really breaks there.

It works by running:
python2 -c "from tensorpack import dataset; c = dataset.Cifar10('train')"

Output:
...
9998
9999
/home/swoda/tensorpack_data/cifar10_data/cifar-10-batches-py/data_batch_5 50000
read_cifar done

Did the crash happen when loading training data or validation/test data?
If the latter, could you remove the prefetch (or even the augmentors) in cifar-convnet.py to see if it segfaults?

It happened when loading training data.
It made no effect with removing the prefetch and augmentors.

Then cifar-convnet.py should be equivalent to the following:
import tensorflow as tf
import argparse
import numpy as np
import os
from tensorpack import *
import tensorpack.tfutils.symbolic_functions as symbf
from tensorpack.tfutils.summary import *
from tensorpack.utils.gpu import get_nr_gpu
os.environ['CUDA_VISIBLE_DEVICES'] = '0'
with tf.Graph().as_default():
    logger.auto_set_dir()
    ds = dataset.Cifar10('train')

If the above code crashes, could you import cv2 at the first line and try again?

After adding import cv2 at the first line of cifar10-resnet.py and cifar1-convnet.py, the problem was solved!
Thank you very much!

OK. I thought this problem was fixed by tensorflow already. Turns out it still happens on some systems.
I'll update some examples to avoid this.


[Documentation] Propose a way to have both ApolloBundle and HTTPBundle working side by side

Currently, the port is conflicting. We therefoe use the apolloBundle.app to add routes, but that prevents from using the HTTPBundle.addRoute method and utilities

ApolloBundle comes packed with its own 'HTTP' modules. I do agree with your approach, ApolloBundle should use HTTPBundle.


Error when trying to view Preview build

The preview flow works almost completely except I encounter an error at the end. I am loading what appears to be a valid URL based on API description:
http://localhost:6557/preview/demo.toitoi.co/65a193e5-18ed-44cc-81a3-5fd67ebb3304/
Let me know if it points to a deficiency at my end because I'm testing locally, e.g. could the preset not be loaded or available locally?
Error response:
{"error":"NotFoundError: No such file exists.
at /Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/libs/generator.js:243:11
at Layer.handle_error (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/layer.js:71:5)
at trim_prefix (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:310:13)
at /Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:280:7
at Function.process_params (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:330:12)
at next (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:271:10)
at /Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:618:15
at next (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/router/index.js:256:14)
at /Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/response.js:417:7
at SendStream.onerror (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/express/lib/response.js:993:5)
at emitOne (events.js:77:13)
at SendStream.emit (events.js:169:7)
at SendStream.error (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/send/index.js:245:17)
at SendStream.onStatError (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/send/index.js:356:12)
at next (/Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/send/index.js:658:28)
at /Users/fivetwelve/Documents/Projects/Work/Schmopera/toitoi/toitoi-generate/node_modules/send/index.js:666:23
at FSReqWrap.oncomplete (fs.js:82:15)"}

Error was caused by theme files not available on the server for rendering. Even when testing it is preferable to select the user's preset through the -cms UI rather than updating the value in -admin interface so that the selection triggers the retrieval of the theme from remote server and its unpacking/installation.


Extended debugging too verbose?

Seems to be logging the entire XRegExp.js code to the Zotero log...
Is this the intended behaviour?
Do you want me to attach a log or do you have enough to go by?

It's not intended, but it's not soemthing I can prevent; Zotero for some reason logs the entire source of the translator if it's not one of their own. which in the case of BBT means 5 times, as I bundle 5 translators. The log lines added by BBT are always prefixed by [better-bibtex, all the rest is Zotero itself or other extensions. For less verbose logging, you can set extensions.zotero.debug.level to 3, but as Zotero logs the translators to loglevel 3 when it saves them, there's not much I can do about it.

I understand. 5? I only noticed XRegExp in the logs. If the logging wasn't so verbose I'd nearly leave it on the whole time. As it is I'll just enable it when I really need it. Should I file an issue with the Zotero folks? Great tool by the way, I'm using it in conjunction with the Atom.io citations package. Happy user here.

I wouldn't necessarily count it as a Zotero bug; I don't really even know
why the translation installer is included as they don't use it themselves.
I'll just write my own translator installer, that will get rid of the
source in the logs, should be done tonight. The zotero logs are indeed
quite chatty; setting the log level at 3 trims them down substantially.

Super.  And I'll check from my end? Where do I set  extensions.zotero.debug.level to 3? Browser JS Console?

Yes, please leave it open. You can configure the debug level in
about:config; it would be easy enough to add a slider to the preferences
screen, but any change to the debug level would also affect logs sent to
zotero itself, and I'd rather not provide a mechanism by which a user could
casually (and unwittingly) meddle with the diagnosis process of zotero
itself. BBT is already invasive enough that people should really disable it
while trying to track down problems in zotero itself.

Will be out in the next release

Great. Thanks for the about:config tip!


[Fix] cert-manager vulnerabilities of high and critical severity

What type of PR is this?
/kind feature

[x] Unittest
[x] Coverage 80%+
[x] Build Success

What this PR does / why we need it:
fix cert-manager vulnerabilities of high and critical severity

https://artifacthub.io/packages/helm/kubservice-charts/kubeservice-namespace-node-affinity/1.1.1?modal=security-report


Comments for export test sources

Description
Comments for export test sources
Type of change

Documentation update

Testing steps
N/A

Codecov Report

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

@@           Coverage Diff           @@
##           master     #222   +/-   ##
=======================================
  Coverage   70.95%   70.95%
=======================================
  Files          12       12
  Lines         730      730
=======================================
  Hits          518      518
  Misses        208      208
  Partials        4        4

Continue to review full report at Codecov.

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


EF flowables whose CAS do not match SRS EPA

Hi! Here I am adding flowables from the PEF impact method whose CAS numbers (and names) are not found in SRS EPA. Previously I addressed PEF flowables that were already in the masterlist but lacking context (#147) or that were already in the SRS EPA (#148).
There's 66 of them, 59 I added to Chemicals and 7 to Groups. Instead of using their IUPAC names I tried as much as possible to adopt a more "user-friendly" name. In doing so, I retrieved their PubChem name instead of the original flowable name from EF. In some cases PubChem did not have an entry for the EF flowable, or I could not be confident about the entry referring to exactly the same substance. For those cases, I picked the ECHA name instead, here they are:
Groups:
2-Oxetanone, 3-C12-16-alkyl-4-C13-17-alkylidene derivs.
Alkenes, c10-13
Hydrocarbons, c4
Hydrocarbons, c4, 1,3-butadiene-free, polymd., dibutylene fraction, hydrogenated
Hydrocarbons, c4, 1,3-butadiene-free, polymd., dibutylene fraction
Hydrocarbons, c4, 1,3-butadiene-free, polymd., triisobutylene fraction
2-Oxetanone, 3-C12-16-alkyl-4-C13-17-alkylidene derivs.
Chemicals:
2-Butene, 1,1,4,4-tetramethoxy-
Phenol, 4,5-dimethyl-2-(1-methylethyl)-
8-ethylidene-tetracyclo[<IP_ADDRESS>(2,5).1(7,10)]dodeca-3-ene

In these cases I did not fill Formula or Synonyms, not sure how appropriate this would be since there is no SRS entry for these flowables. I could use the PubChem synonyms, but preferred not to, since sometimes they seem to have synonyms that are not exactly synonyms but close substance matches; however, I am quite out of my comfort zone here to be able to assess whether that's the case or not.
Some of those I added under Groups I am quite unsecure about the classification:
Hydrocarbons, c4, 1,3-butadiene-free, polymd., dibutylene fraction, hydrogenated
Hydrocarbons, c4, 1,3-butadiene-free, polymd., dibutylene fraction
Hydrocarbons, c4, 1,3-butadiene-free, polymd., triisobutylene fraction

These three seem quite similar to each other by name.
This one also seems to be quite specific to deserve to be a Group:
2-Oxetanone, 3-C12-16-alkyl-4-C13-17-alkylidene derivs.

But I think it can include many derivates of 2-Oxetanone, 3-C12-16-alkyl-4-C13-17-alkylidene?
Anyways, me and @ercollao @ganorris will be very happy to discuss this

tests passed, 198 new flowables with primary contexts (59 chemicals + 7 groups, 3 contexts for each) ✔


Lecture 24: Integrative Paradigms of GM: Regularized Bayesian Methods

Hi, could you attach a link to the rendered scribe note? I didn't find one. Thanks a lot!

Hi, could you attach a link to the rendered scribe note? I didn't find one. Thanks a lot!

https://github.com/wngzhiqi/pgm-spring-2019/blob/master/_posts/2019-04-15-lecture-24.md

Hi, could you attach a link to the rendered scribe note? I didn't find one. Thanks a lot!

https://github.com/wngzhiqi/pgm-spring-2019/blob/master/_posts/2019-04-15-lecture-24.md

Sorry about the confusion. I meant the rendered website link rather than the raw markdown file. It would be easier to check equations before we accept the pull request. Please see e.g. https://github.com/sailinglab/pgm-spring-2019/pull/22

Hi, could you attach a link to the rendered scribe note? I didn't find one. Thanks a lot!

https://github.com/wngzhiqi/pgm-spring-2019/blob/master/_posts/2019-04-15-lecture-24.md

Sorry about the confusion. I meant the rendered website link rather than the raw markdown file. It would be easier to check equations before we accept the pull request. Please see e.g. #22

Sorry for the misunderstanding.
The rendered link is here:
https://wngzhiqi.github.io/pgm-spring-2019/notes/lecture-24/

@xunzheng, can please you review this set of notes ASAP? Thanks.


[HOLD for payment 2024-07-24] [$250] Review duplicates - "Unhold" option appears for expenses that are pending duplicate reviews

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

Version Number: 9.0.4-0
Reproducible in staging?: Y
Reproducible in production?: Y
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat.
Submit two expenses with the same amount and merchant.
Go to transaction thread of any expense.
After Review duplicates button appears, click on the report header.
Click Unhold.

Expected Result:
"Unhold" option should be appear for expenses that are pending review for duplicates.
Actual Result:
"Unhold" option appeara for expenses that are pending review for duplicates.
Clicking Unhold does not unhold the expense.
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/11941eb6-be2e-4628-9293-a43a26e9767b

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e9c4febbfea1864e
Upwork Job ID:<PHONE_NUMBER>699118867
Last Price Increase: 2024-07-09
Automatic offers:

ahmedGaber93 | Reviewer | 103060862

Issue OwnerCurrent Issue Owner: @zanyrenney

We think this issue might be related to the #collect project.

Reviewing today

@bernhardoj's proposal LGTM!
I agree "we shouldn't allow unhold if it's hold because of duplicate expenses violation" because unhold action will not affect on anything as it is not a real hold with transaction.comment?.hold value
🎀 👀 🎀 C+ reviewed

PR is ready
cc: @ahmedGaber93

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x] [@ahmedGaber93] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/40153
[x] [@ahmedGaber93] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/40153/files#r1685718142
[x] [@ahmedGaber93] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] [@ahmedGaber93] Determine if we should create a regression test for this bug.
[x] [@ahmedGaber93] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again. Yes

Regression Test Proposal

Submit 2 expenses with the same amount and merchant to a workspace
Open the transaction thread of the duplicated expense
Press the chat header
Verify there is no "Unhold" option

Do we agree 👍 or 👎

Requested in ND.

@zanyrenney Could you please pay out this when you have a chance?

payment summary
@ahmedGaber93 requires payment automatic offer (Reviewer) - paid $250 via upwork.
@bernhardoj requires payment through NewDot Manual Requests - owed $250

$250 approved for @bernhardoj


"Licence" title has a typo

Section title "Licence" has typo in it.

Hi @Laptopmini !
See https://en.wiktionary.org/wiki/licence . This is the British spelling of the noun and it is not a typo.

Next time please look it up.


Question about Instrumentations

I am getting stuck in various bugs and think better tooling might help myself and others.
What tools are people using out there ?
I am thinking a dashboard as a CLI or other on all the grain s etc.
A first stab would be something like the golang docker watcher. It's elegant and simple and tells me what all the grains are up to.
Curious if there are thoughts on this. Happy. To start a test harness but looking for feedback first :)

We do have injection points where we can hook in e.g. mailbox statistics, or middleware for in and outbound messages.
There is however no dashboards or anything like that available yet.
We did talk early on about making a CLI took to query the state of cluster members

Thanks. I can now see what direction it's going.
A CLI watcher sounds like a great initial tool.
This is a nice golang CLI tool for real time watching of all your dockers.
https://github.com/bcicen/ctop
It's well done imho and can be adapted for watching actors / grains.
Also I use this alot. It's a GUI interface for debugging golang with delve. It was good concurrency support.
https://github.com/aarzilli/gdlv
It's really impressive and is pure golang.
It could be adapted to be a kick arse tool for seeing and even helping write actors.
Have a look if your curious. Both are very high quality imho, and I am interested in helping to push this area forward but don't want to branch off from what you feel is needed.


DFS clique, filters descent on back links, proof of concept, alpha code.

Please note I don't consider this quality work because the testing was just copied from test_matching.py with minor edits and no corner case tests, additional testing, and so on.  However I'm happy to contribute this to your project.

Thank you for the contribution. This looks good to me. Now we just need to use it.

No problem, hit me up if you have any more puzzlers.


MultiselectBelongsToSupport importing inexistent class

I'm getting Class 'Laravel\Nova\Util' not found error after I upgraded from v2.3.3 to v2.3.11. The class is being imported in the trait MultiselectBelongsToSupport, I checked the supposed class base on it's namespace and it doesn't exist in Laravel Nova's vendor directory.
I'm using these atm:
Laravel Nova: v3.22.0
Laravel Framework: 8.67.0

Hi! Apparently \Laravel\Nova\Util was added in version 3.25. I added support for older versions in the new 2.3.12 release.
Thanks for reporting this!


Any specific optimization did in kserve to support LLM inference?

Hi community,
I am wondering any specific optimization did in kserve to support LLM applications?  Is there a feature list?

/kind question

Hi @Jeffwan ! KServe's position is still on LLM inference orchestration and serverless autoscaling, so we are focusing on inference protocol like supporting OpenAI specification,  improving LLM container cold startup time and enabling autoscaling based on custom metrics like the number of input/output tokens. We will update our roadmap with a list features we have planned.


Support of embedded keys pair for tendermint private validator inside burrow instead of monax.keys

This PR allows you to configure inmemory crypto validator if you do not want to depend on an external keys API to sign messages for private tendermint validator:
By API:
        conf := config.DefaultBurrowConfig()
...
	conf.Crypto.KeysServer = nil // Disable external call to sign and load public key
        // Generate keys OR load your own pair programatically
	conf.Crypto.InMemoryCrypto, err = inmemory.DefaultInMemoryCryptoConfig()
...

Or by Config (not safest approach to store private key in file) - example:
[Crypto]
  [Crypto.InMemoryCrypto]
     PrivateKey = "{\"type\":\"ed25519\",\"data\":\"YOUR PRIVATE KEY\"}"
     PublicKey = "{\"type\":\"ed25519\",\"data\":\"2e4bbbffad9325db4e18725c6f31234ee303ac43f8ff4bee167b6e83a4c49197\"}"

Fixes #702

Thanks for this. I'll reply now rather than waiting until I've fully considered everything:

I'm inclined to include this functionality - but not expose it in config (yet) - we need a way of splitting up config in general and I think this probably encourages behaviour we don't want to encourage (i.e. keys in config file). I think local keys need to be in a properly permissioned separate file and for there to at least a warning if they are not.

I'm inclined to butcher some of what you have done and rework it so that we have something like a self-hosted set of keys. We will shortly be adding 'co-located signing' as in you can get Burrow to sign transactions if it has access to the signing key via keys. It might make most sense to pull in monax-keys wholesale for this - or at least a library portion of it to just make it 'something Burrow can do'.

We need to support PrivateKey not just ed25519 keys

Thank you for your reply. I agree with all 3 points above, feel free to rework this small contribution  if needed.
I am pleased to help if required, for example I may recode private key access to an external file.
We are waiting this feature for my project, happy to see embedded mona-keys inside burrow the soonest.

@phymbert just to let you know we are going to add burrow-hosted keys as a feature - this can support this use case and also with address-based delegated signing that I am working on also allows for running Burrow as a signing delegate - useful as a local 'wallet' or in the cloud behind a service.

Thanks @phymbert for embedding monax.key to burrow. I'd like to suggest if it is possible using an interface pattern for signer object. Burrow can easily implement any curve in a modular way, not only ed25519.  Look here: https://github.com/hyperledger/burrow/pull/707/files#diff-06dc382f674b6310ff1f5382daad00b7

We now have support for your use case and more with: https://github.com/hyperledger/burrow/pull/776


Add metrics for keeping track of off-heap allocation for consuming segments

Codecov Report

Merging #1874 into master will decrease coverage by 20.81%.
The diff coverage is 78.26%.

@@             Coverage Diff             @@
##           master    #1874       +/-   ##
===========================================
- Coverage    70.1%   49.29%   -20.82%
===========================================
  Files         813      813
  Lines       38918    38936       +18
  Branches     5139     5142        +3
===========================================
- Hits        27282    19192     -8090
- Misses       9841    18106     +8265
+ Partials     1795     1638      -157

Impacted Files
Coverage Δ

...com/linkedin/pinot/common/metrics/ServerGauge.java
95% <100%> (+0.26%)
:arrow_up:

...a/manager/realtime/RealtimeSegmentDataManager.java
100% <100%> (ø)
:arrow_up:

...data/manager/offline/AbstractTableDataManager.java
86.48% <100%> (-4.43%)
:arrow_down:

...n/pinot/core/io/writer/impl/MmapMemoryManager.java
74.6% <33.33%> (-7.37%)
:arrow_down:

...pinot/core/io/writer/impl/DirectMemoryManager.java
71.42% <66.66%> (-28.58%)
:arrow_down:

...eaderwriter/RealtimeIndexOffHeapMemoryManager.java
92.85% <83.33%> (-7.15%)
:arrow_down:

...ot/core/query/scheduler/SchedulerQueryContext.java
0% <0%> (-100%)
:arrow_down:

...ot/core/query/scheduler/TableBasedGroupMapper.java
0% <0%> (-100%)
:arrow_down:

...din/pinot/pql/parsers/pql2/ast/OptionsAstNode.java
0% <0%> (-100%)
:arrow_down:

...edin/pinot/pql/parsers/pql2/ast/HavingAstNode.java
0% <0%> (-100%)
:arrow_down:

... and 423 more

Continue to review full report at Codecov.

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


Fix form in saving email schedule

CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY

Fix NoneType bug while saving empty test email recipients
Fill the test recipients with original recipients if the field is empty

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

REVIEWERS
to: @michellethomas @mistercrunch

Codecov Report

Merging #7365 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #7365      +/-   ##
==========================================
- Coverage   64.91%   64.91%   -0.01%
==========================================
  Files         428      428
  Lines       20951    20953       +2
  Branches     2330     2330
==========================================
  Hits        13601    13601
- Misses       7226     7228       +2
  Partials      124      124

Impacted Files
Coverage Δ

superset/views/schedules.py
58.03% <0%> (-1.06%)
:arrow_down:

Continue to review full report at Codecov.

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


improve documentation

Created a better documentation

Thank you for participating @jesper137  - I will close this PR (and this repo) now, as the course is over :)


fix: remove vault_datacenter

This removes the unused variable vault_datacenter.
We don't understand the full history of this variable (why it was used initially), but it might originate from older installations where a Consul storage backend was required (the term datacenter makes more sense in the Consul realm).
The terminology in the Vault configuration is "cluster name", which is already configurable with the vault_cluster_name variable.

I would expect checks to pass, did you rebase on master today?

There appears to be an issue with the ACME plugin installation (KeyError). I had a quick look but was unable to fix it right away, needs more work


Added ignore for Jetbrains Rider

Reasons for making this change:
Rider is becoming more and more popular with Unity developers, especially with the latest changes to the Unity plugin. Adding the idea cache folder to the ignore provides convenience in the same way ignoring the visual studio cache does.
Links to documentation supporting these rule changes:
Not really much to support but with the continued Unity integration improvement, it is worth adding.

This is a bit different to the VS temporary directory.
We have a detailed set of rules for all the JetBrains IDES over in Global\JetBrains.gitignore, which I recommend adding to your repository .gitignore.


lock value of cli we are using

here too? https://github.com/keybase/client/blob/b199ec2c91944cd32ea9e31dead4f72c695b94d9/go/service/gregor.go#L480

maybe this should just hold the lock and be called in these places https://github.com/keybase/client/blob/4f2f4add4d82b77724a9fe542a0d9cb6f2aa6c3e/go/service/gregor.go#L407


I use an RTX 3090 and hence am very interested in the 2.4.0 version of Tensorflow.
However, when I use the 2.4.0-rc4 and attempt to import keras...
from tensorflow import keras
... I get the error:
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-9-256f42a18d91> in <module>
     22 print('ok0')
     23
---> 24 from tensorflow import keras
     25
     26 print('ok1')

~/anaconda3/envs/tsne/lib/python3.8/site-packages/tensorflow/keras/__init__.py in <module>
     17 from . import callbacks
     18 from . import constraints
---> 19 from . import datasets
     20 from . import estimator
     21 from . import experimental

~/anaconda3/envs/tsne/lib/python3.8/site-packages/tensorflow/keras/datasets/__init__.py in <module>
     12 from . import cifar100
     13 from . import fashion_mnist
---> 14 from . import imdb
     15 from . import mnist
     16 from . import reuters

~/anaconda3/envs/tsne/lib/python3.8/site-packages/tensorflow/keras/datasets/imdb/__init__.py in <module>
      9 import sys as _sys
     10
---> 11 from tensorflow.python.keras.datasets.imdb import get_word_index
     12 from tensorflow.python.keras.datasets.imdb import load_data
     13

~/anaconda3/envs/tsne/lib/python3.8/site-packages/tensorflow/python/keras/datasets/imdb.py in <module>
     23 import numpy as np
     24
---> 25 from tensorflow.python.keras.preprocessing.sequence import _remove_long_seq
     26 from tensorflow.python.keras.utils.data_utils import get_file
     27 from tensorflow.python.platform import tf_logging as logging

~/anaconda3/envs/tsne/lib/python3.8/site-packages/tensorflow/python/keras/preprocessing/__init__.py in <module>
     21 # TODO(mihaimaruseac): remove the import of keras_preprocessing and injecting
     22 # once we update to latest version of keras_preprocessing
---> 23 import keras_preprocessing
     24
     25 from tensorflow.python.keras import backend

ModuleNotFoundError: No module named 'keras_preprocessing'

Is this an expected behaviour for this rc -- and I should have known that keras does not work with rc versions?
Is there a simple way for me to get keras to work with 2.4.0-rc4?

We don't officially support conda here.
See if  this could help you https://stackoverflow.com/questions/64102020/modulenotfounderror-no-module-named-keras-preprocessing

You may try https://stackoverflow.com/a/64104744/11127923

@ymodak Is your different?

My bad.. I didn't check your comment. I will retract.

Nevermind, thanks.

Does it matter that I installed it with pip --just in a conda environment?

See https://github.com/tensorflow/tensorflow/issues/45458#issuecomment-740199045

Tested this in a virtualenv with pip-only and it works. So it seems that the fault lies in conda.

Thank you, everyone!


🚧Improve ReadMe

Complete The Item Below

[X] I have updated the title without removing the 🚧 emoji.

Description
Fix indenting issues with any of the bullet items.
Fix broken link.
Find any areas were text content can be improved by converting to a Note or Warning block
Should look like this:

Note

Acceptance Criteria

[x] README improved with Notes and/or Warnings
[x] Bullet indentation fixed
[x] Broken link fixed

ToDo Items

[X] Change type labels added to this issue.  Refer to the Change Type Labels section below.
[X] Priority label added to this issue.  Refer to the Priority Type Labels section below.
[X] Issue linked to the correct project (if applicable).
[X] Issue linked to the correct milestone (if applicable).
[x] Draft pull request created and linked to this issue (only required with code changes).

Issue Dependencies
No response
Related Work
No response
Additional Information:
Change Type Labels

Change Type
Label

Bug Fixes
🐛bug

Breaking Changes
🧨breaking changes

New Feature
✨new feature

Workflow Changes
workflow

Code Doc Changes
🗒documentation/code

Product Doc Changes
📝documentation/product

Priority Type Labels

Priority Type
Label

Low Priority
low priority

Medium Priority
medium priority

High Priority
high priority

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

PR #427  merged.  This issued is closed.


Fix ArrayElementPropagation handling of negative index

Convert an incorrect assert into correct code.
Fixes rdar://80444607 (Swift compiler segfault on “[1][-1]”)

@swift-ci test

@aschwaighofer please review for 5.5... or let me know if you don't think it's worth fixing on the release branch


Roll Flutter Engine from 53517772a5b0 to e57b440ec4ee (8 revisions)

https://github.com/flutter/engine/compare/53517772a5b0...e57b440ec4ee
2024-09-26<EMAIL_ADDRESS>Reland "[canvaskit] Further improve overlay optimization by splitting pictures" (flutter/engine#55464)
2024-09-26<EMAIL_ADDRESS>[Impeller] remove entity pass and re-combine canvas implementations. (flutter/engine#55019)
2024-09-26<EMAIL_ADDRESS>Roll Skia from e986bd33fb33 to 9ebb7c3640a1 (3 revisions) (flutter/engine#55466)
2024-09-26<EMAIL_ADDRESS>Roll Skia from 5f6ea5ff8401 to e986bd33fb33 (1 revision) (flutter/engine#55465)
2024-09-26<EMAIL_ADDRESS>Remove and inline dependencies on build_root for a constant. (flutter/engine#55441)
2024-09-26<EMAIL_ADDRESS>Reverts "Reverts "Add SurfaceProducer#onSurfaceAvailable, deprecate onSurfaceCreated. (#55418)" (#55450)" (flutter/engine#55463)
2024-09-26<EMAIL_ADDRESS>cipd: add mobileprovision CIPD package (flutter/engine#55449)
2024-09-26<EMAIL_ADDRESS>Roll Skia from b2c5f640cb44 to 5f6ea5ff8401 (1 revision) (flutter/engine#55462)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC bdero<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


mousepad: backport fix of issue 12134, resolves #14862

Motivation for this change
Resolves #14862. This should also be ported to stable.
https://bugzilla.xfce.org/show_bug.cgi?id=12134
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

It seems that you should fetchpatch https://git.xfce.org/apps/mousepad/patch/?id=309f6177d2c47070676d6c733e29952c63c110ba

No: some hunks of the patch had to be rewritten on top of 0.4.0, some hunks didn't apply because it was referencing a not-yet-existing code (revert), etc.


Focal length of the realsense D435

| Camera Model                       | {D435} |
| Firmware Version                   | V2.50.0 |
| Operating System & Version |   Ubuntu 18.04 |
| Kernel Version (Linux Only)    |  4.9.253                                       |
| Platform                                 | Nvidia Jetson xavier  |
| Language                            |  {Python}                          |
| Segment			|  {Robot }                          |
With the two gray cameras from the real sense, I can get the disparity. But when I want to convert it the depth image, It seems like it doesn't give me the correct depth.
I use depth = (baseline*focal_length)/disparity
where baseline = 50mm = 0.05m
focal_length= 942.8px, this one I am not, but I saw a guy comment it is this value.
when I use this value of the focal_length, the depth seem too far, when I convert it to the point cloud,
but when I random choose the value of 300, It seems gives me a better result.
So I'm not sure if the problem come from this one or another thing that I miss about?

Thank you very much for the information @MartyG-RealSense. I founded it.

You are very welcome, @monsterITC - thanks very much for the update!

Hi @monsterITC You can find the focal length for your particular camera at a particular resolution using the calibration information mode of the RealSense SDK's 'rs-enumerate-devices' tool. The tool can be launched in calibration mode with the command rs-enumerate-devices -c
If you look at the intrinsics for a particular resolution and stream type then it will list 'fx' and 'fy'. This value is the focal length. Using either fx or fy will be fine as they are the same value.

Sorry for disturbing, I came across the similar prob, I searched for "Intrinsic of 'Color' / 640x480 ....", the Fx value is 608.6419, while Fy is 607.1061, which should i consider?

Hi @by011  It's no trouble at all  )
On my own D435, for the 640x480 color intrinsics profile Fx is 607.5303 and Fy is 607.0724 (each camera unit has its own unique intrinsics determined during manufacturing at the factory).  Whilst Fx and Fy are commonly the same value, they are allowed to be different.  So both values are valid.  The focal length of the image will be calculated from a multiplication of pixel width by pixel height: Fx x Fy.
https://github.com/IntelRealSense/librealsense/wiki/Projection-in-RealSense-SDK-2.0#intrinsic-camera-parameters


Show number of characters next to translation input field and translation reference

Widget elements usually have a fixed size making the text they contain limited in length. As a first step it would be good to see the number of characters that the translation referebce and the translation (also if it is currently being typed) contains.
It would be good to somehow show these numbers. However, one should not think that they might be part of the texts.

added


On close animation

Added few helper animation classes to solve issue #11 .

Closed in favour of e09f07d419da023994deeba88a7b8d21dfd555a1


Separate demo site for elegant under dev version

@iranzo currently we made all our changes in master. Once 2.0 and 2.1 is out of the door, we will move to next branch so that master is used only for releases. This makes sense when you consider the potentials bugs that may creep in during dev.
The documentation site is also serving as a testing ground for new features and fixes. Is it possible and feasible to have a separate site to host the documentation but from next branch. Preferably hosted as a subdomain of pelican-elegant.github.io like next.pelican-elegant.github.io.
Let me know what are our options and what you think of it?

Seems that we can only have one 'master' which is the current pelican-elegant.github.io but github allows to create a subfolder in it (virtual)
What we do put in 'NEXT' repository will show up as https://pelican-elegant.github.io/next/

About google we can write a 'robots.txt' that disallows all bots when generating 'next' site

@talha131  can we create already a 'next' branch forked from any of the dev ones for testing the documentation part?

@iranzo here you go. https://github.com/Pelican-Elegant/elegant/tree/next
I suggest we first make a commit that updates the site title or description. "This hosts under development version of documentation".
This way we would know if travis is working correctly or not.

https://github.com/Pelican-Elegant/elegant/pull/228

@talha131 should we close this?

@iranzo thanks a lot for your contribution.
When I visit, https://pelican-elegant.github.io/next/ it shows me the site. But all the links on it point to the master release.

Fixed in https://github.com/Pelican-Elegant/documentation/pull/45 (modifying publishconf before push to next)

@iranoz to the best of my knowledge the solution didn't work.
https://travis-ci.org/Pelican-Elegant/documentation/builds/468399192

I think it was missing '' for string in the publishconf, I've submitted one to master directly to check output once it finishes

Something strange in top menu bar, but articles seem to be ok!


Armature animation export problem

Hello all,
I am trying to use Blender 2.80 beta (latest as of today) to export a simple animated character to GLTF. It has only 1 mesh using 1 armature skinning animation and 1 mesh parented to a single bone. The skinning animation export worked correctly in Blender 2.79 using the old export script, but the bone parented object was incorrectly transformed, but now neither works in the new 2.80 version.
girl_sword.zip
I tried to load with the windows 10 model viewer, where the model shows up incorrectly animated and jittery and the bone parented object doesn't animate at all. Also tried with the Cesium gltf viewer, which fails to open the file: https://www.virtualgis.io/gltfviewer/ and an other online viewer which also fails: https://gltf-viewer.donmccurdy.com/
And in my own loader ( https://github.com/turanszkij/WickedEngine ) I get completely broken skinning animation but the bone parented object is also animated.
So to recap, the Blender 2.79 exporter is working kind of well (except bone parenting), but the same project in Blender 2.80 fails with bone parenting and skinning too.

What result were you seeing on today's beta? The latest version of this repo (which is not yet in the Beta, I don't think) has several related fixes and is looking close...

... but the animation isn't playing quite correctly, there are some buggy cross-axis flips coming from somewhere. The bone parenting looks correct now.

Hello,
I'm having a similar issue: the attached model is completely flat, but once exported, all animated joints are rotating in axes they shouldn't. Tested the file with Don McCurdy's gltf viewer. Using blender 2.79. I actually had the same issue with a previous version of the exporter, from this repository: https://github.com/KhronosGroup/glTF-Blender-Exporter , that's why I updated to glTF-Blender-IO today.
bonhomme.zip

I have made some GIFs to visualize the problem. I should mention, that I tried to copy the latest gltf exporter scripts to Blender 2.80 scripts/addons folder too, but the results are the same as yesterday.
This is the reference as it looks in Blender:

This is how it looks in Windows 10 model viewer:

This is how it looks in my gltf loader:

Notice that the Windows 10 viewer doesn't animate the "sword" that is parented to bone, but not skinned. The skinned model animation also looks wrong. The first frame geometry is completely blown up, and the last frame is completely collapsed. The intermediate frames are also wrong.
In my gltf loader, the first and last frames match the windows 10 loader if I am just eyeballing it, however, the intermediate frames are different (as in completely messed up), but the sword is animated too.
I understand that the windows 10 model viewer and my gltf loader are maybe not acceptable reference implementations, but they can generally handle every gltf model fine that I throw at it (eg. from Sketchfab).

New update: If I change the animation's interpolation mode to linear and re-export it, I get better results, but still not perfect. The blender gif in my previous post was actually using linear interpolation mode, but didn't re-export it with those settings. The legs are weird, and seems like only half of the animation is actually exported.
Windows 10 viewer:

My gltf loader:

When I try exporting a more complex character, the exporter crashes with this call stack:
Traceback (most recent call last):
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\__init__.py", line 424, in execute
    return gltf2_blender_export.save(context, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_export.py", line 40, in save
    json, buffer = __export(export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_export.py", line 54, in __export
    __gather_gltf(exporter, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_export.py", line 69, in __gather_gltf
    scenes, animations = gltf2_blender_gather.gather_gltf2(export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather.py", line 36, in gather_gltf2
    animations += __gather_animations(blender_scene, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather.py", line 62, in __gather_animations
    animations += gltf2_blender_gather_animations.gather_animations(blender_object, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animations.py", line 38, in gather_animations
    animation = __gather_animation(blender_action, blender_object, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animations.py", line 55, in __gather_animation
    channels=__gather_channels(blender_action, blender_object, export_settings),
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animations.py", line 89, in __gather_channels
    blender_action, blender_object, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_cache.py", line 51, in wrapper_cached
    result = func(*args)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_channels.py", line 35, in gather_animation_channels
    channel = __gather_animation_channel(channel_group, blender_object, export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_channels.py", line 52, in __gather_animation_channel
    sampler=__gather_sampler(channels, blender_object, export_settings),
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_channels.py", line 85, in __gather_sampler
    export_settings
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_cache.py", line 51, in wrapper_cached
    result = func(*args)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_samplers.py", line 41, in gather_animation_sampler
    input=__gather_input(channels, blender_object_if_armature, export_settings),
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_cache.py", line 51, in wrapper_cached
    result = func(*args)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_samplers.py", line 71, in __gather_input
    export_settings)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_cache.py", line 51, in wrapper_cached
    result = func(*args)
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_sampler_keyframes.py", line 174, in gather_keyframes
    for c in channels
  File "C:\Program Files\blender-2.80.0\2.80\scripts\addons\io_scene_gltf2\blender\exp\gltf2_blender_gather_animation_sampler_keyframes.py", line 174, in <listcomp>
    for c in channels
ZeroDivisionError: float division by zero

location: <unknown location>:-1

location: <unknown location>:-1

Havoc.zip
The attached blender file has many animations both skinning and bone attached as well.

I´m getting errors as well when exporting animations:

I've uploaded the zip. It works fine if I don't export animations.
avatar.zip

NLA issue is probably linked to #461
I have to test again others cases, as some animation fixed was committed since the last comment

Everything seems to work now.
Don't hesitate to comment, open again this ticket, or create a new one if you found some (other) issue.

bonhomme.zip
These are my last results:
=>In blender 2.79, animations are exported but rotations are slightly wrong (see bonhomme_blender2_79.glb)
=>In blender 2.8, I can see the animations' names in https://gltf-viewer.donmccurdy.com/ but they don't play at all. (see bonhomme_blender2_8.glb)
The source .blend file is included.

@alexGuntha This seems to be a different problem, I will open a new issue about that.
Note that your armature object have some none default rotation and scale, that is always a very bad idea in Blender (don't know yet if the is related or not)

See #525


MultiModal pretraining issues

After pretraining the model on WebVid, the MSRVTT evaluation results dropped to below 1%. Similarly, when pretraining from the provided pretrained weights, the results also dropped below 1% after the first step of training. We have verified our evaluation setup by using the pretrained weights and successfully replicated the results shown in the paper.
Any idea what could causing these issues?

Does the lossed seem to be normal when running?

Does the lossed seem to be normal when running?

yes the loss seems fairly reasonable

It's interesting and I did not explore it before.