[DNM] samples: net: get SimpleLink wifi driver samples building on sanitycheck

Note: please wait to merge after #10389 resolved.
Previously, wifi samples (sockets/echo, net/wifi) were not built
by sanitycheck for the cc3220sf_launchxl.
This patch gets at least those two samples to (attempt to) build.
Signed-off-by: Gil Pitney<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master   #10392   +/-   ##
=======================================
  Coverage    52.9%    52.9%
=======================================
  Files         210      210
  Lines       25732    25732
  Branches     5672     5672
=======================================
  Hits        13614    13614
  Misses       9802     9802
  Partials     2316     2316

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 3687103...99f3f4c. Read the comment docs.

Commit message:

wifi samples (sockets/echo, net/wifi)

Can you please replace with "wifi-enabled samples" to avoid confusion.
And you'll need to decide with @jukkar whose sample.yaml is better.

OK, I removed my sample/net/wifi/sample.yaml, and also got socket offload and sockets/echo sample to build again with new logging updates.

OK, feedback resolved, and rebased.


Discussion: Enable 5 minutes of out-of-order by default in Prometheus

Proposal
This is an issue to kick start a discussion to switch the default configuration for the out-of-order window.
Today the default value of the window is 0 meaning no out-of-order samples are allowed into Prometheus by default. it's up to the client to allow this use case and thus its up to the client to toggle the window and set it to a certain amount of minutes.
If we use Prometheus in the traditional way were it is pull based and it does scrapes based on a the configured interval this is completely fine and probably the out-of-order feature won't be needed.
However if Prometheus is used with remote-write due to a specific use case, certain network topologies, OTEL ingestion, scale or other reasons then we might run into issues. It's a paradigm change where order is hard to maintain.
It is this second use case where enabling out-of-order by default to a small window like 5m or 10m starts making a bit of sense.

An example is using Prometheus with OTEL. Due to how OTEL clients work, encouraging users to batch up writes and then remote writing them all at once to improve compression rate, out-of-order is a natural outcome.

For any of these cases, the user needs to know about the existence of this feature and how to tweak it to avoid data loss. Maybe it does make sense to enable it to a small window to solve 99% of the race conditions between the writes?

I disagree that we should enable this by default. This is still very experimental, in regular usage of Prometheus when you need it it is showing that you have clocks instabilities, and it has bad side effects, such as those described in #11730 that would break some features in prometheus in regular usage, even with 5m delay.

Seems like https://github.com/prometheus/prometheus/issues/12631 is a prerequisite for this?
Could use case specific documentation and respective example configs be a path forward here?


improve performance of controller

What type of PR is this?
This PR is aimed at optimizing the performance of controller components in large-scale scenarios.
When there are many cr objects, the efficiency of index is much higher than that of list by selector, as shown in this performance test.
Here's the case:
func BenchmarkGetByList(b *testing.B) {
	for n := 0; n < b.N; n++ {
		if subs, err := subsLister.Subscriptions("clusternet-test").List(labels.SelectorFromSet(
			labels.Set{"6640f948-89af-4d1d-976d-08968895457d": "Subscription"})); err != nil || len(subs) != 1 {
			b.Fatalf("list fialed: %v %d", err, len(subs))
		}
	}
}

func BenchmarkGetByIndex(b *testing.B) {
	for n := 0; n < b.N; n++ {
		if objs, err := sInformer.GetIndexer().ByIndex("uid", "6640f948-89af-4d1d-976d-08968895457d"); err != nil || len(objs) != 1 {
			b.Fatalf("list fialed: %v %d", err, len(objs))
		}
	}
}

Benchmark results:
2023-11-01T14:49:23+08:00: list subscription
2023-11-01T14:49:23+08:00: total count of subs is 100000
goos: darwin
goarch: arm64
BenchmarkGetByList-10                 46          24414396 ns/op
BenchmarkGetByIndex-10          15271741                79.20 ns/op

When searching from a list of 100,000 cr objects, the efficiency of index is much higher than that of list.
What this PR does / why we need it:
This PR focuses on optimizing the base lookup process in the controller, which involves helmcharts, manifests, and subs processing.
Which issue(s) this PR fixes:

Fixes #753
Special notes for your reviewer:

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

Comparison is base (e2ffa5f) 13.60% compared to head (b0b1e22) 13.11%.
Report is 117 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #756      +/-   ##
==========================================
- Coverage   13.60%   13.11%   -0.50%
==========================================
  Files          66       76      +10
  Lines        7492     8672    +1180
==========================================
+ Hits         1019     1137     +118
- Misses       6391     7452    +1061
- Partials       82       83       +1

see 51 files with indirect coverage changes

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


Automatic Godot Camera access (iOS) causes crash and App rejection

Godot version:
3.2 Alpha, 3.2 Beta1, 3.2 Beta2
OS/device including version:
Tested on iOS 12/13
Issue description:
Godot automatically asks for camera access at startup wich causes the app to crash on certain devices and leads to the app rejection when you try to publish it.
I could reproduce the issue on an iPhone 5S using iOS12 (works fine on iPhone X and iPad Pro and iOS13). The App does not crash when you delete the "Privacy - Camera Usage Description" but Godot still tries to get permission and the App can not be uploaded to the App store.
I am trying to release a game for children and, apart from being unable to publish it, Godot asking for Camera access on startup is quite problematic.
My code does not contain any references to the camera, microphone or arkit. Everything is deselected and I have tried to exclude them using the entitlement description.
I have tried Godot 3.1, Godot 3.2 Alpha and both Betas. I have also tried to revert to 3.1.2RC1 but it does not recognize the .tscn files after they have been opened in 3.2
At the moment it is not possible for me to release the app due to this issue.
Steps to reproduce:
Upload any project to iOS App store
Minimal reproduction project:

Yes, this is really problematic, confirmed. A similar issue has been raised pre-beta2 related to the packaging of ArKit and has been addressed by @bruvzg and @BastiaanOlij in [this PR] (https://github.com/godotengine/godot/pull/33759) but more needs to be done to resolve this properly and unfortunately I am not very up to speed with iOS and the build chain. I can help with testing though.

Also happening here on 3.2 beta 2.

Godot should not request camera access when "Camera usage description" is empty or missing: https://github.com/godotengine/godot/blob/master/platform/iphone/camera_ios.mm#L401-L410 (added in #30374), so it must be requested by some other code.

If the "Camera Privacy" entry is not set (delete the key or leave the "value" field empty) the Xcode Debugger output reads: "No NSCameraUsageDescription key in pList, no access to cameras." and Apple will not to accept the build and send an email with the following statement:
"ITMS-90683: Missing Purpose String in Info.plist - Your app's code references one or more APIs that access sensitive user data. The app's Info.plist file should contain a NSCameraUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. Starting Spring 2019, all apps submitted to the App Store that access user data are required to include a purpose string. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. You can contact the developer of the library or SDK and request they release a version of their code that doesn't contain the APIs."

No NSCameraUsageDescription key in pList, no access to cameras

Yep, this is message from the Godot itself indicating camera init is skipped.

ITMS-90683: Missing Purpose String in Info.plist

Well, looks like just having camera code is enough to trigger this.
If this is the case we'll probably need to ship multiple versions of the export templates compiled with and without camera (and most likely microphone) support.
Or somehow split all sensitive the code into multiple static libs and load it dynamically.

Or somehow split all sensitive the code into multiple static libs and load it dynamically.

Here's basic mock-up:
AFAIK iOS dlsym doesn't work on RTLD_SELF and apps with .dylibs are not accepted to the App Store, so loading camera server code at the runtime is not an option.

1. Compile camera server code to the separate static lib (additionally make empty/stub lib version).
diff --git a/platform/iphone/SCsub b/platform/iphone/SCsub
index 85ba56165b..0c492f4553 100644
--- a/platform/iphone/SCsub
+++ b/platform/iphone/SCsub
@@ -14,12 +14,25 @@ iphone_lib = [
     'in_app_store.mm',
     'icloud.mm',
     'ios.mm',
+]
+
+iphone_camera_lib_real = [
     'camera_ios.mm',
 ]

+iphone_camera_lib_stub = [
+    'camera_ios_stub.mm',
+]
+
 env_ios = env.Clone()
 ios_lib = env_ios.add_library('iphone', iphone_lib)

+env_ios_camera_real = env.Clone()
+ios_lib_camera_real = env_ios_camera_real.add_library('iphone_camera_real', iphone_camera_lib_real)
+
+env_ios_camera_stub = env.Clone()
+ios_lib_camera_stub = env_ios_camera_stub.add_library('iphone_camera_stub', iphone_camera_lib_stub)
+
 def combine_libs(target=None, source=None, env=None):
     lib_path = target[0].srcnode().abspath
     if "osxcross" in env:
diff --git a/platform/iphone/camera_ios_stub.mm b/platform/iphone/camera_ios_stub.mm
new file mode 100644
index<PHONE_NUMBER>..f724171df3
--- /dev/null
+++ b/platform/iphone/camera_ios_stub.mm
@@ -0,0 +1,12 @@
+/*************************************************************************/
+/*  camera_ios_stub.mm                                                   */
+/*************************************************************************/
+
+#include "camera_ios.h"
+#include "servers/camera/camera_feed.h"
+
+void CameraIOS::update_feeds() { /*NOP*/ }
+
+CameraIOS::CameraIOS() { /*NOP*/ }
+
+CameraIOS::~CameraIOS() { /*NOP*/ }

2. On export include either real camera lib or stub (or generate stub code in place) to the Xcode project.

Been a bit preoccupied lately, sorry for jumping in late.
This is pretty horrible from Apple, it really makes it hard for generic
engines to operate. There are other features that fall under this type of
protection, having separate builds for each possible combination sounds
whack.
Seeing we do the final compile in xcode, would there be a way to
restructure this so we isolate the bits of logic in modules but instead of
compiling in the modules we add them as separate lib files? So we have a
dummy version and implemented version compiled and deployed in the template.
Then on export we decide which libraries to link in?
On Thu, 28 Nov 2019 at 11:51 pm, bruvzg<EMAIL_ADDRESS>wrote:

Or somehow split all sensitive the code into multiple static libs and load
it dynamically.
Here's basic mock-up:
AFAIK iOS dlsym doesn't work on RTLD_SELF and apps with .dylibs are not
accepted to the App Store, so loading camera server code at the runtime is
not an option.

Compile camera server code to the separate static lib (additionally
make empty/stub lib version).

diff --git a/platform/iphone/SCsub b/platform/iphone/SCsub
index 85ba56165b..0c492f4553 100644--- a/platform/iphone/SCsub+++ b/platform/iphone/SCsub@@ -14,12 +14,25 @@ iphone_lib = [
'in_app_store.mm',
'icloud.mm',
'ios.mm',+]++iphone_camera_lib_real = [
'camera_ios.mm',
]
+iphone_camera_lib_stub = [+    'camera_ios_stub.mm',+]+
env_ios = env.Clone()
ios_lib = env_ios.add_library('iphone', iphone_lib)
+env_ios_camera_real = env.Clone()+ios_lib_camera_real = env_ios_camera_real.add_library('iphone_camera_real', iphone_camera_lib_real)++env_ios_camera_stub = env.Clone()+ios_lib_camera_stub = env_ios_camera_stub.add_library('iphone_camera_stub', iphone_camera_lib_stub)+
def combine_libs(target=None, source=None, env=None):
lib_path = target[0].srcnode().abspath
if "osxcross" in env:diff --git a/platform/iphone/camera_ios_stub.mm b/platform/iphone/camera_ios_stub.mm
new file mode 100644
index<PHONE_NUMBER>..f724171df3--- /dev/null+++ b/platform/iphone/camera_ios_stub.mm@@ -0,0 +1,12 @@+//+/  camera_ios_stub.mm                                                   /+//++#include "camera_ios.h"+#include "servers/camera/camera_feed.h"++void CameraIOS::update_feeds() { /NOP/ }++CameraIOS::CameraIOS() { /NOP/ }++CameraIOS::~CameraIOS() { /NOP/ }

On export include either real camera lib or stub (or generate stub code
in place) to the Xcode project.

—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/33918?email_source=notifications&email_token=AAO262LSEZ4MJOGY7ZFND3TQV65FXA5CNFSM4JR2QPW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFMQSBI#issuecomment-559483141,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAO262M4TX7EIFFVV6ALBR3QV65FXANCNFSM4JR2QPWQ
.

--
Kindest regards,
Bastiaan Olij
https://www.facebook.com/bastiaan.olij
https://twitter.com/mux213
https://www.youtube.com/BastiaanOlij
https://www.youtube.com/channel/UCrbLJYzJjDf2p-vJC011lYw
https://github.com/BastiaanOlij

A bit more advanced mock-up - https://github.com/bruvzg/godot/tree/ios_modular_build:

[x] All camera access implementations are moved to the camera module.
[x] iOS build - export template have all modules as separate libgodot_{module_name}_module.iphone.{release/debug}.{arch}.a static libs.
[ ] iOS exporter - get available module list from the export template. (TODO)
[ ] iOS exporter - generate register_module_types.gen.cpp for selected modules on demand.  (TODO)
[ ] iOS exporter - add selected static libs to the Xcode project.  Go for it.


More Ownership: Switch enum, checked cast br, dynamic method br oh my!

Some more ownership changes.

adb85df: This change makes it so that we check for triviality in a part of the checker by comparing against ValueOwnershipKind::Trivial instead of using SILType::isTrivial(...). This is important since in the case of enums, getOwnershipKind() treats no payload and trivial payload as trivial, while SILType::isTrivial will return non-trivial.

4c7ec6e: This change fixes switch_enum, checked_cast_br, dynamic_method_br to be allowed to take trivial arguments as well as their normal non-trivial cases. The checked cast br and dynamic method br cases come up due to metatypes.

rdar://29791263

@swift-ci Please smoke test and merge


能告诉一下使用方法吗？

能告诉一下使用方法吗？

This library is incomplete, and I have no plans of continuing development. You can take a look at the example: https://github.com/1lann/go-sip/blob/master/example/example_server.go and the server code: https://github.com/1lann/go-sip/tree/master/server if you're planning to use the lower level primitives of the library.
There is also the godocs: https://godoc.org/github.com/1lann/go-sip/sipnet


Publish IBM Cloud Quick Start reference architecture

Publish content to:

IBM Cloud - https://github.com/IBM/automation-ibmcloud-infra-openshift

Dependencies

[ ] IBM Resource Group
[ ] IBM Access Group
[ ] IBM VPC
[ ] IBM VPC Subnets
[ ] IBM VPC Gateways
[ ] IBM OCP on VPC (ROKS)
[ ] IBM Log Analysis (LogDNA)
[ ] IBM Monitoring (Sysdig)
[ ] IBM Object Storage
[ ] Portworx storage on IBM Cloud
[ ] ODF storage on IBM Cloud

Definition of Done

[ ] Document installation instructions in README in Automation Solutions repo
[ ] Generate automation logic from BOM in Automation Solutions repo and publish to repo - https://github.com/IBM/automation-ibmcloud-infra-openshift

Dup of IBM/automation-ibmcloud-infra-openshift#2


How to call loadmore manually?

I want to call loadmore when the scrolls in the middle of the results
so for example, if the results are 20 and the user scrolled to the item 10 I need to request loadmore
how is that can be achieved

you can refer this https://github.com/CymChad/BaseRecyclerViewAdapterHelper/blob/master/app/src/main/java/com/chad/baserecyclerviewadapterhelper/PullToRefreshUseActivity.java


Please update Embedded-reader-import

Link is here

seems to work fine for me, can you please describe what your experience of the bug is?

Ok. I have updated the line 24 its curricullums  but it can not display new updats in the reader . So, I thought you havn't updated the embedded-reader-import yet.


Get-Pfa2Volume (2.2.272.0) appears to be hardcoded to use only API v2.2

In my attempts to use Get-Pfa2Volume against Purity/FA running 5.3.13, it is failing with the following error.
PS>Get-Pfa2Volume -Array $array
Get-Pfa2Volume: Response: Error calling Api22VolumesGet:  from Get-Pfa2Volume (__AllParameterSets) Array=blahblah
My attempts to force the use of API v2.0 or v2.1 where not successful.

Hi Craig,
We would need a bit more information on the error as well as the commands your trying to execute.  Here is what happens when I execute the following commands against a FlashArray with version 5.3.2 with a maximum API version of 2.0.
PS C:\Windows\System32> $array2 = Connect-Pfa2Array -Endpoint $ArrayEndpoint -Username $ArrayUsername -Issuer $ArrayIssuer -ClientId $clientId -KeyId $keyId -PrivateKeyFile $privateKeyFile -PrivateKeyPassword $privateKeyPass -IgnoreCertificateError -ApiClientName $ArrayClientname

PS C:\Windows\System32> Get-Pfa2ApiVersion -Endpoint <IP_ADDRESS> | Select -ExpandProperty "ArraySupportedAPIVersion"
1.0
1.1
1.2
1.3
1.4
1.5
1.6
1.7
1.8
1.9
1.10
1.11
1.12
1.13
1.14
1.15
1.16
1.17
2.0

PS C:\Windows\System32> Get-Pfa2Volume -ApiVersion 2.0

Id                      : 7277ca3f-4d83-e2a1-ff91-df9fbc0cfd3a
Name                    : GENIO-TST-01
ConnectionCount         : 0
Created                 : 12/21/2017 7:49:53 PM
Destroyed               : False
HostEncryptionKeyStatus : none
Provisioned             :<PHONE_NUMBER>00
Qos                     : class Qos {
                            BandwidthLimit:
                            IopsLimit:
                          }

Serial                  : 2EBD55E4363D498400011012
Space                   : class Space {
                            DataReduction: 1.6244731
                            Shared:
                            Snapshots: 0
                            System:
                            ThinProvisioning: 0.40274802
                            TotalPhysical:<PHONE_NUMBER>15
                            TotalProvisioned:<PHONE_NUMBER>00
                            TotalReduction: 2.7199123
                            Unique:<PHONE_NUMBER>15
                            Virtual:<PHONE_NUMBER>64
                          }

TimeRemaining           :
Pod                     : class Reference {
                            Id:
                            Name:
                          }

Source                  : class FixedReference {
                            Id:
                            Name:
                          }

Subtype                 : regular
VolumeGroup             : class Reference {
                            Id:
                            Name:
                          }

RequestedPromotionState :
PromotionStatus         :

Hi Mike,
-ApiVersion 2.0.  I wasn't setting this option on Get-Pfa2volume cmdlet but was
setting on the 'Connect-Pfa2Array' cmdlet.  Not sure how I missed since it is the first option in the syntax diagram.
Anyway, my bad. You can close out this issue.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Monday, December 20th, 2021 at 1:06 PM, Mike Nelson @.***> wrote:

Hi Craig,
We would need a bit more information on the error as well as the commands your trying to execute. Here is what happens when I execute the following commands against a FlashArray with version 5.3.2 with a maximum API version of 2.0.
PS C:\Windows\System32> $array2 = Connect-Pfa2Array -Endpoint $ArrayEndpoint -Username $ArrayUsername -Issuer $ArrayIssuer -ClientId $clientId -KeyId $keyId -PrivateKeyFile $privateKeyFile -PrivateKeyPassword $privateKeyPass -IgnoreCertificateError -ApiClientName $ArrayClientname
PS C:\Windows\System32> Get-Pfa2ApiVersion -Endpoint <IP_ADDRESS> | Select -ExpandProperty "ArraySupportedAPIVersion"
1.0
1.1
1.2
1.3
1.4
1.5
1.6
1.7
1.8
1.9
1.10
1.11
1.12
1.13
1.14
1.15
1.16
1.17
2.0
PS C:\Windows\System32> Get-Pfa2Volume -ApiVersion 2.0
Id                      : 7277ca3f-4d83-e2a1-ff91-df9fbc0cfd3a
Name                    : GENIO-TST-01
ConnectionCount         : 0
Created                 : 12/21/2017 7:49:53 PM
Destroyed               : False
HostEncryptionKeyStatus : none
Provisioned             :<PHONE_NUMBER>00
Qos                     : class Qos {
BandwidthLimit:
IopsLimit:
}
Serial                  : 2EBD55E4363D498400011012
Space                   : class Space {
DataReduction: 1.6244731
Shared:
Snapshots: 0
System:
ThinProvisioning: 0.40274802
TotalPhysical:<PHONE_NUMBER>15
TotalProvisioned:<PHONE_NUMBER>00
TotalReduction: 2.7199123
Unique:<PHONE_NUMBER>15
Virtual:<PHONE_NUMBER>64
}
TimeRemaining           :
Pod                     : class Reference {
Id:
Name:
}
Source                  : class FixedReference {
Id:
Name:
}
Subtype                 : regular
VolumeGroup             : class Reference {
Id:
Name:
}
RequestedPromotionState :
PromotionStatus         :
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

No problem. Thanks for filing the issue!

Closed.


firebase serve unable to serve https function

I was asked to file this bug here by the mods at the firebase-tools repo.
https://github.com/firebase/firebase-tools/issues/827#issuecomment-406106709
Version info
firebase v3.19.3
node v6.11.5
npm v3.10.10
nvm v0.33.2
firebase-admin 5.12.1 and
firebase-functions 1.0.3 and 1.1.0
Platform Information
Mac OS 10.12.6
Steps to reproduce
firebase serve fails to emulate functions even with the example function in index.js.
const functions = require('firebase-functions');

// Create and Deploy Your First Cloud Functions
// https://firebase.google.com/docs/functions/write-firebase-functions

exports.helloWorld = functions.https.onRequest((request, response) => {
 response.send("Hello from Firebase!");
});

Expected behavior
Expected to serve functions.
Actual behavior
➜  functionsTest firebase serve --only functions --debug
[2018-07-13T05:31:30.253Z] ----------------------------------------------------------------------
[2018-07-13T05:31:30.257Z] Command:       /Users/hectorlee/.nvm/versions/node/v6.11.5/bin/node /Users/hectorlee/.nvm/versions/node/v6.11.5/bin/firebase serve --only functions --debug
[2018-07-13T05:31:30.258Z] CLI Version:   3.19.3
[2018-07-13T05:31:30.258Z] Platform:      darwin
[2018-07-13T05:31:30.258Z] Node Version:  v6.11.5
[2018-07-13T05:31:30.258Z] Time:          Fri Jul 13 2018 13:31:30 GMT+0800 (+08)
[2018-07-13T05:31:30.259Z] ----------------------------------------------------------------------

[2018-07-13T05:31:30.268Z] > command requires scopes: ["email","openid","https://www.googleapis.com/auth/cloudplatformprojects.readonly","https://www.googleapis.com/auth/firebase","https://www.googleapis.com/auth/cloud-platform"]
[2018-07-13T05:31:30.268Z] > authorizing via signed-in user
[2018-07-13T05:31:30.272Z] >>> HTTP REQUEST GET https://admin.firebase.com/v1/projects/firecast-caaa6

 Fri Jul 13 2018 13:31:30 GMT+0800 (+08)
[2018-07-13T05:31:31.132Z] <<< HTTP RESPONSE 200 server=nginx, date=Fri, 13 Jul 2018 05:31:31 GMT, content-type=application/json; charset=utf-8, content-length=111, connection=close, x-content-type-options=nosniff, strict-transport-security=max-age=31536000; includeSubdomains, cache-control=no-cache, no-store
[2018-07-13T05:31:31.133Z] >>> HTTP REQUEST GET https://admin.firebase.com/v1/database/firecast-caaa6/tokens

 Fri Jul 13 2018 13:31:31 GMT+0800 (+08)
[2018-07-13T05:31:31.887Z] <<< HTTP RESPONSE 200 server=nginx, date=Fri, 13 Jul 2018 05:31:31 GMT, content-type=application/json; charset=utf-8, content-length=263, connection=close, x-content-type-options=nosniff, strict-transport-security=max-age=31536000; includeSubdomains, cache-control=no-cache, no-store
[2018-07-13T05:31:31.888Z] >>> HTTP REQUEST GET https://cloudresourcemanager.googleapis.com/v1/projects/firecast-caaa6

 Fri Jul 13 2018 13:31:31 GMT+0800 (+08)
[2018-07-13T05:31:33.071Z] <<< HTTP RESPONSE 200 content-type=application/json; charset=UTF-8, vary=X-Origin, Referer, Origin,Accept-Encoding, date=Fri, 13 Jul 2018 05:31:33 GMT, server=ESF, cache-control=private, x-xss-protection=1; mode=block, x-frame-options=SAMEORIGIN, x-content-type-options=nosniff, alt-svc=quic=":443"; ma=2592000; v="43,42,41,39,35", accept-ranges=none, connection=close

=== Serving from '/Users/hectorlee/code/functionsTest'...

i  functions: Preparing to emulate functions.
[2018-07-13T05:31:33.531Z] Fetching environment
[2018-07-13T05:31:33.533Z] >>> HTTP REQUEST GET https://mobilesdk-pa.googleapis.com/v1/projects/11896845545:getServerAppConfig

 Fri Jul 13 2018 13:31:33 GMT+0800 (+08)
[2018-07-13T05:31:34.187Z] <<< HTTP RESPONSE 200 content-type=application/json; charset=UTF-8, vary=X-Origin, Referer, Origin,Accept-Encoding, date=Fri, 13 Jul 2018 05:31:34 GMT, server=ESF, cache-control=private, x-xss-protection=1; mode=block, x-frame-options=SAMEORIGIN, x-content-type-options=nosniff, alt-svc=quic=":443"; ma=2592000; v="43,42,41,39,35", accept-ranges=none, connection=close
[2018-07-13T05:31:34.188Z] Starting @google-cloud/functions-emulator
[2018-07-13T05:31:34.209Z] Parsing function triggers
[2018-07-13T05:31:34.341Z] Error while deploying to emulator: TypeError: Cannot read property 'call' of undefined
TypeError: Cannot read property 'call' of undefined
    at Promise (/Users/hectorlee/.nvm/versions/node/v6.11.5/lib/node_modules/firebase-tools/node_modules/@google-cloud/functions-emulator/src/client/rest-client.js:34:42)
    at getService.then (/Users/hectorlee/.nvm/versions/node/v6.11.5/lib/node_modules/firebase-tools/node_modules/@google-cloud/functions-emulator/src/client/rest-client.js:33:16)
    at process._tickDomainCallback (internal/process/next_tick.js:135:7)
⚠  functions: Failed to emulate helloWorld
i  functions: No HTTPS functions found. Use firebase functions:shell if you would like to emulate other types of functions.

I can't seem to get any functions to emulate using new firebase-tools, firebase-functions, and firebase-admin. Even firebase-samples default examples fail, so do my previously working functions.
⚠  functions: Failed to emulate helloWorld
i  functions: No HTTPS functions found. Use firebase functions:shell if you would like to emulate other types of functions.
After 2 days of searching, I still find conflicting info on the status of what versions to use and I can no longer get a simple helloworld app to emulate:
Docs say (https://firebase.google.com/docs/functions/manage-functions#set_nodejs_version)  you can specify either node 6 or node 8 runtime. Great. I have installed 8. But errors now arise from the emulator which say: Warning: You're using Node.js v8.11.3 but Google Cloud Functions only supports v6.11.5.   I figured if I use the recommended Typescript to get all the async/await/es6 goodness shown in these training videos (https://firebase.google.com/docs/functions/video-series/) that it should transpile to es5, which I'd hope would overcome the need for an older node runtime environment. Apparently not the case despite these tutorial videos show he's running node 10 and deploying functions and encouraging us to emulate the functions before deployment. Do his function emulations actually work?  If so, what am I doing wrong?
So if I understand, I can use node 8 runtime, but can't use node 8 when emulating functions yet, even when I try a simple helloWorld straight from the firebase examples?
So I guess I have to uninstall all node 8 from my system and go back to 6.11.5?  Even though when I install firebase-tools it throws a warning that I need to update to the latest firebase-tools, firebase-functions and firebase-admin? Which of course, starts this whole process over again?  When I run npm install in the functions dir of default stock examples it wants me to update to
"firebase-admin": "^5.13.1",
"firebase-functions": "^2.0.2"
which I do, but still emulation doesn't work.
I must have lucked out on my previous app, which went smoothly and had me praising FB. Strangely, I was on Node 8 then and didn't even specify "engines: "8" on that one. It just worked.  I'm not an advanced Node or Firebase developer, in fact, trying hard to grok all of it, but this is crazy making right now. Is this all due to the latest updates (which came right in the middle of my current project and broke everything)?
Can someone specify exact versions of node, firebase, firebase-tools, firebase-functions, and firebase-admin I should use to get something done? Do I need firebase SDK too (I think it's not necessary without auth, etc. correct?)
Thanks in advance,
J
$ node --version
v8.11.3
$ npm --version
6.2.0
⚠  functions: Failed to emulate helloWorld
i  functions: No HTTPS functions found. Use firebase functions:shell if you would like to emulate other types of functions.
Warning: You're using Node.js v8.11.3 but Google Cloud Functions only supports v6.11.5.
[2018-07-28T18:54:08.421Z] ----------------------------------------------------------------------
[2018-07-28T18:54:08.424Z] Command:       /usr/local/bin/node /usr/local/bin/firebase serve --debug
[2018-07-28T18:54:08.424Z] CLI Version:   4.0.0
[2018-07-28T18:54:08.424Z] Platform:      darwin
[2018-07-28T18:54:08.424Z] Node Version:  v8.11.3
[2018-07-28T18:54:08.425Z] Time:          Sat Jul 28 2018 11:54:08 GMT-0700 (PDT)
[2018-07-28T18:54:08.425Z] ----------------------------------------------------------------------

Ping?

Any updates?


Update 'alias add' example

I don't really know the code or golang, but this seemed simple enough.

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

I didn't look at the diff in this PR since the DCO wasn't signed but based on the title I can guess what it does, so I'll send a branch which fixes this, including the needed make i18n output.

Opened a separate PR with all the bits needed for this fix.


Scrollbar are displayed when the calendar is displayed at the bottom of a form

When the datetime picker is displayed at the bottom of a form, scrollbar are displayed to show calendar. Is it possible that the DOM element is create outside the form to show up outside the div element without overflow?
Best Regards

@heyanctil
any suggestions to do that?

Right now, no hehehe


[SMALLFIX] Increase readSkipTest's timeout

All the other tests are at 25000 (25 seconds), but readSkipTest was at 20 seconds. This caused a failure in Jenkins: https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Master/957/PROFILE=hdfs1Test,label=centos/console

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4318/
Test PASSed.

LGTM


Bug:  'unterminated comment' error when you have /- in a docstring

Prerequisites
Please put an X between the brackets as you perform the following steps:

[X] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[X] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[X] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
Having /- appear in a docstring or block comment seems to be interpreted as starting a nested comment, triggering an unterminated comment error. This error confusingly is only shown at the very end of the file, even if the faulty comment is much higher up.
Furthermore, syntax highlighting in VSCode does show the comment as being properly terminated, making the error very confusing to track down.
/-- `x +/- y` -/
def test : String := "foo"

-- We get `unterminated comment` at the *end* of the file, even if we introduce other definitions after `test`

Context
Zulip post
Steps to Reproduce

Write a docstring that has /- in it
Be confused about the 'unterminated comment' error

Expected behavior: [Clear and concise description of what you expect to happen]
Having /- inside a docstring or block comment should not have any special meaning.
At the very least, the 'unterminated comment' error should be located at the faulty comment.
Actual behavior: [Clear and concise description of what actually happens]
Having /- inside a docstring or block comment seems to start a nested comment, that is expected to be terminated by a matching -/. This causes an unterminated comment error at the very end of the file
Versions
live.lean-lang.org version 4.12.0-nightly-2024-08-12
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

This sounds like working-as-intended: /- inside comments does in fact start a nested comment, lean's comment syntax has always allowed nested comments:
#check 1 /- /- -/ -/ + 1

Because this works at the lexer level, it does not know about markdown syntax and the fact that this is inside a code literal makes no difference. The error message occurring at the end of the file also makes some amount of sense, since this is an unterminated comment and it's pointing at the place where it knows there is a mistake, although I know many languages point instead at the open comment marker, or at the entire span starting there until the end of the file, both of which are clear and actionable indicators of what went wrong and where.

I'd strongly suggest we either change the syntax highlighting in the vscode extension (this is LSP-controlled, right?), or we change the location of the error to include the beginning of the comment as well.

and it's pointing at the place where it knows there is a mistake
This is kinda true, but cases where we have a comment at the end of the file that we forgot to terminate are vastly outnumbered by the cases where the comment was in fact intended to end (way) earlier. Having the error closer to the start (or indeed for the entire span, either are fine with me), is much more likely to give useful information about where the fix is actually supposed to be.

In my case, the faulty comment was a few hundred lines before the end. Combined with the fact that the highlighter did show the comment to end where I expected to, it took me quite a while to figure out what exactly the problem was.

I was also confused just now by a similar issue with an unterminated comment, where the tactic state from before the tactic is shown if there is an unterminated comment directly after it.
Showing the unterminated comment error at the start of the comment would help.
example : True := by
  simp -- unsolved goals ⊢ True

/-

-- unterminated comment


Add a facility filter

As I user, I would like to be able to:

be able to filter the results by facility
be able to have a list of facility included in the results set
be able to search a specific facility which is than passed to the federated search

Is it feasible? What would be the effort?

Do you want to select one facility or many?


Websocket frame interface issue

Hi,
first of all, thank you very much for great library. I really like that it is based on Boost.Asio, which makes it easy to understand and work with.
However I am having troubles with using websocket frame interface. Following is the minimal working example which demonstrates my problem.
#include <beast/websocket.hpp>
#include <boost/asio.hpp>
#include <iostream>
#include <string>
#include <array>

int main()
{
    // Normal boost::asio setup
    std::string const host = "<IP_ADDRESS>";
    boost::asio::io_service ios;
    boost::asio::ip::tcp::resolver r{ios};
    boost::asio::ip::tcp::socket sock{ios};
    boost::asio::connect(sock,
        r.resolve(boost::asio::ip::tcp::resolver::query{host, "42000"}));

    // WebSocket connect and send message using beast
    beast::websocket::stream<boost::asio::ip::tcp::socket&> ws{sock};
    ws.handshake(host, "/");
    ws.set_option(beast::websocket::message_type{beast::websocket::opcode::binary});

    // Send some random binary message
    std::array<char, 512> binary_message;
    ws.write_frame(false, boost::asio::buffer(binary_message));

    // THIS IS HACK, (needed to make this work)
    ws.set_option(beast::websocket::message_type{beast::websocket::opcode::cont});

    ws.write_frame(true, boost::asio::buffer(binary_message));

    // Receive WebSocket message, print and close using beast
    beast::streambuf sb;
    beast::websocket::opcode op;
    beast::error_code ec;
    ws.close(beast::websocket::close_code::normal);
    ws.read(op, sb, ec);

    if (ec != beast::websocket::error::closed)
    {
        std::cerr << "Communication terminated in unexpected way." << std::endl;
        return 1;
    }

    std::cout << "Successfully terminated." << std::endl;
    return 0;
}

I was forced to modify constructor of the class beast::websocket::message_type so it accepts beast::websocket::opcode::cont as argument. Then, before sending the second frame I have to call ws.set_option(beast::websocket::message_type{beast::websocket::opcode::cont}); on the websocket.
The problem is that if I do not do that, the server will report an error: Protocol violation. I am using the websocket_async_echo_server.hpp from your examples with slight (ad-hoc) modification that it uses async_read_frame() and it is not sending a response.
        void operator()(error_code ec)
        {
            using boost::asio::buffer;
            using boost::asio::buffer_copy;
            auto& d = *d_;
            switch(d.state)
            {
            // did accept
            case 0:
                if(ec)
                    return fail("async_accept", ec);

            // start
            case 1:
                if(ec)
                    return fail("async_handshake", ec);
                d.db.consume(d.db.size());
                // read message
                d.state = 2;
                //d.ws.async_read(d.op, d.db, d.strand.wrap(std::move(*this))); // ORIGINAL
                d.ws.async_read_frame(d.fi, d.fb, d.strand.wrap(std::move(*this)));
                return;

            // got message
            case 2:
                if(ec == beast::websocket::error::closed)
                    return;
                if(ec)
                    return fail("async_read", ec);
                // ORIGINAL
                // write message
                //d.state = 1;
                //d.ws.set_option(beast::websocket::message_type(d.op));
                //d.ws.async_write(d.db.data(), d.strand.wrap(std::move(*this)));

                if (!d.fi.fin)
                {
                    d.ws.async_read_frame(d.fi, d.fb, d.strand.wrap(std::move(*this)));
                }
                else // entire frame received
                {
                    std::cout << "Entire frame received" << std::endl;
                    d.ws.async_read(d.op, d.db, d.strand.wrap(std::move(*this)));
                }
                return;
            }
        }

I was tracking Protocol violation error on the server side and it seems to come from file: stream_base.hpp:
        if(rd_.cont)
        {
            // new data frame when continuation expected
            return err(close_code::protocol_error);
        }

Finally, I am using websocket_echo.cpp from your examples as a server, where I removed websocket::sync_echo_server part.
Thus in summary, when I use the HACK ws.set_option(beast::websocket::message_type{beast::websocket::opcode::cont}); everything works as expected (At least seems so). When I remove this line from client I get Protocol violation error on server side.
Could you please give me a hint what am I doing wrong?
Thank you.

I am reviewing and investigating your issue, thanks for being so thorough that will help a ton!

I believe this branch contains a commit that reproduces your issue:
https://github.com/vinniefalco/Beast/commits/b33
This test:
https://github.com/vinniefalco/Beast/blob/b33/test/websocket/stream.cpp#L808

@matinko Sorry for the delay, I don't have access to my desktop machine. I have located the defect and committed a fix, can you verify it works? Thanks!
https://github.com/vinniefalco/Beast/commits/b33

@vinniefalco Hi, just tested the branch b33 and I can say it fixed my problem. Thank you very much!


Style objects imported with css modules are empty

I'm trying to use this plugin with css modules. It seems to be creating my main css file correctly (I'm able to open it and it contains all my styles with the correctly generated css class names), but I'm not able to import any styles in my components. For example, I put the following in one of my components, and it logs an empty object.
import styles from './styles.css';

console.log('STYLES', styles);

Pretty much everything I've run across so far has been about the opposite issue (css classnames in their components, but no css file generated). I did run across https://github.com/webpack-contrib/extract-text-webpack-plugin/issues/229 , which suggested my problem might be solved by setting remove: false in the loader query, but that doesn't seem to do anything. If I keep what I have in ExtractTextPlugin.extract and just setting the disable flag to true, the styles show up again, so it seems the configuration I've put in there should work.
I'm on the latest version of both webpack and this plugin:
<EMAIL_ADDRESS><EMAIL_ADDRESS>
For what it's worth, npm-check also says I'm on the latest version of everything in my project.
My configuration is below (have been running in development mode with the dev middleware, but tried doing a production build and experienced the same issue).
const webpack = require('webpack');
const path = require('path');
const config = require('config');
const ExtractTextPlugin = require('extract-text-webpack-plugin');
const hotMiddlewareScript = 'webpack-hot-middleware/client?path=/__webpack_hmr&timeout=20000&reload=true';

const entry = process.env.NODE_ENV === 'production' ?
  ['babel-polyfill', 'whatwg-fetch', path.resolve(__dirname, 'client')] :
  ['babel-polyfill', 'whatwg-fetch', hotMiddlewareScript, path.resolve(__dirname, 'client')];

module.exports = {
  devtool: 'inline-source-map',

  entry: {
    app: entry
  },

  resolve: {
    alias: {
      components: path.resolve(__dirname, 'client/components'),
      containers: path.resolve(__dirname, 'client/containers'),
      redux_utils: path.resolve(__dirname, 'client/redux'),
      dux: path.resolve(__dirname, 'client/redux/ducks'),
      fixtures: path.resolve(__dirname, 'client/fixtures'),
      fragments: path.resolve(__dirname, 'client/fragments'),
      hocs: path.resolve(__dirname, 'client/hocs'),
      queries: path.resolve(__dirname, 'client/queries'),
      lib: path.resolve(__dirname, 'client/lib'),
      main: path.resolve(__dirname, 'client/main'),
      mutations: path.resolve(__dirname, 'client/mutations'),
      pages: path.resolve(__dirname, 'client/pages'),
      prop_types: path.resolve(__dirname, 'client/prop_types'),
      routes: path.resolve(__dirname, 'client/routes'),
      styles: path.resolve(__dirname, 'client/styles'),
      test_utils: path.resolve(__dirname, 'test_utils'),
    },
  },

  output: {
    path: path.resolve(__dirname, 'public'),
    filename: 'js/[name].js'
  },

  externals: {
    'react/addons': true,
    'react/lib/ExecutionEnvironment': true,
    'react/lib/ReactContext': true,
  },

  module: {
    rules: [
      {
        test: /\.(js|jsx)$/,
        include: [
          path.resolve(__dirname, 'client'),
          path.resolve(__dirname, 'render_pages'),
          path.resolve(__dirname, 'test'),
          path.resolve(__dirname, 'styleguide'),
          path.resolve(__dirname, 'node_modules/css-loader'),
        ],
        loader: 'babel-loader',
        query: {
          presets: ['es2015', 'react', 'stage-2'],
        },
      },
      {
        test: /\.css$/,
        include: [
          path.resolve(__dirname, 'node_modules/react-spinkit'),
          path.resolve(__dirname, 'client'),
        ],
        loader: 'style-loader',
      },
      {
        test: /\.css$/,
        include: [
          path.resolve(__dirname, 'node_modules/react-spinkit'),
          path.resolve(__dirname, 'node_modules/loaders.css'),
        ],
        loader: 'css-loader',
      },
      {
        test: /\.css$/,
        include: [
          path.resolve(__dirname, 'client'),
        ],
        loader: ExtractTextPlugin.extract({
          loader: "css-loader",
          query: {
            remove: false,
            localIdentName: '[path][name]__[local]--[hash:base64:5]',
            modules: true,
          }
        }),
      },
      {
        test: /\.woff(2)?(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        loader: 'url-loader',
        exclude: /node_modules/,
        query: {
          limit: 10000,
          minetype: 'application/font-woff',
        },
      },
      {
        test: /\.(ttf|eot|svg|ico)(\?v=[0-9]\.[0-9]\.[0-9])?$/,
        exclude: /node_modules/,
        loader: 'file-loader',
      },
    ]
  },

  plugins: [
    new ExtractTextPlugin({
      disable: false,
      filename: 'css/[name].css',
      ignoreOrder: true,
      allChunks: true
    }),
    new webpack.optimize.OccurrenceOrderPlugin(),
    new webpack.DefinePlugin({
      'CONFIG': JSON.stringify(config.get('public')),
    }),
  ].concat(process.env.NODE_ENV === 'production' ? [
    new webpack.optimize.UglifyJsPlugin()
  ] : [
    new webpack.HotModuleReplacementPlugin(),
  ])
};

Figured it out: the loader configuration I had earlier in the config for style loader was messing things up. Changing this:
{
  test: /\.css$/,
  include: [
    path.resolve(__dirname, 'node_modules/react-spinkit'),
    path.resolve(__dirname, 'client'),
  ],
  loader: 'style-loader',
},

to this:
{
  test: /\.css$/,
  include: [
    path.resolve(__dirname, 'node_modules/react-spinkit')
  ],
  loader: 'style-loader',
},

fixed the issue.


Incorrect pool token amount

when adding liquidity, you need to calculate the return of. pool tokeen amount.
currently, bnt pool token is worth ~1.02 bnt.
meaning,  if i add. 2 bnt, i should receive less than 2 pool tokens.

Are you saying the rates should reflect the pool token return?

@pingustar knows what's up, added to the plate.

@Velua
Data is needed of calculation for expected return in Pool Tokens. Or have you that available already?

Waiting on me to provide expectedReturn for v2 pools add liquidity.

@ashachaf we haven't seen a mockup that displays this, is providing the pool token return for a deposit in line with the design/approach?

@pingustar double check and see if there's a mockup that reflects this in case I'm blind.

@Velua @ashachaf
for V1 add liquidity we have this mockup:
https://app.zeplin.io/project/5d70f6905eefcd18e1c40502/screen/5f17d104bacb0478b55d7138
currently we show the balances to be deposited instead of the pool token amount returned.

for V2 add liquidity we have this mockup:
https://app.zeplin.io/project/5d70f6905eefcd18e1c40502/screen/5f17d1090847fb84fa1e127e
and currently we show the exact amount entered by the user as return, I believe it should be slightly less than that, not to sure what calculation that would be:

when adding liquidity, the summary page should indicate the amount of pool tokens received (not reserve tokens).
https://zpl.io/boqg4GE
also, the information we display in the gray box should be updated to fit the mockup.

phase out


Use libqrencode also on OSX

From @ThomasHabets on October 10, 2014 8:7
Original issue 257 created by Alexander.Schneider on 2013-03-31T11:36:02.000Z:
I have installed libqrencode on my OSX system (10.8.2) with brew, and therefor the needed library is available on the system.
Currently the google-authenticator does not use it, because of a different name of the lib.
Attached is a easy patch to solve this issue.
Copied from original issue: google/google-authenticator#256

Referenced patch is at https://storage.googleapis.com/google-code-attachments/google-authenticator/issue-257/comment-0/fix_libqrencode_osx.diff

Fixed in https://github.com/google/google-authenticator-libpam/pull/62


SingleChildScrollView viewport jumps around when scrolling.

Widgets inside a SingleChildScrollView jump around to the right as if the viewport keeps changing constraints. Something interesting that I have noticed is that the viewport doesn't seem to be using MSAA?
Normally when a widget is positioned in the centre where its width constraints are an odd number, both of its sides (left and right) become blurry. However with all of the scrolling widgets (SingleChildScrollView, ListView, GridView...) the edges are always sharp. I think this may be related to what I am experiencing here.
This is happening on both stable 3.7 and master 3.9.0-20.0
From the video below, not only are the containers always jumping around, but the inkwell for the MenuItemButton always goes out 1px further to the right.
Video:
https://user-images.githubusercontent.com/73116038/229033662-96e50d49-e789-46fd-bab5-e37bbce543a2.mp4

Repro code:
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: SizedBox(
            height: 200,
            width: 200,
            child: SingleChildScrollView(
              child: Container(
                color: Colors.red,
                child: Column(
                  children: [
                    SizedBox(
                      width: double.infinity,
                      height: 42,
                      child: Material(
                        color: Colors.blue,
                        child: InkWell(
                          onTap: () {},
                        ),
                      ),
                    ),
                    Container(
                      width: double.infinity,
                      height: 42,
                      color: Colors.green,
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                    MenuItemButton(
                      onPressed: () {},
                      child: const Text('Tap me'),
                    ),
                  ],
                ),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

CC @Piinks

Hello @whiskeyPeak. Please refrain from mentioning team members directly.
I have found a similar issue (https://github.com/flutter/flutter/issues/110957). Are you hitting this?

I don't think so. The issue you linked talks about widgets/text jumping around even when no scrollView is present. My case is specifically about the right side of the scrollView extending by 1 pixel when scrolling or using an inkwell widget.

Are you able to reproduce this issue on both desktop and web? On MacOS, I cannot seem to reproduce this issue at all

Whoops, sorry for the long reply. This issue isn't reproduceable on web

Are you able to reproduce this issue on both desktop and web? On MacOS, I cannot seem to reproduce this issue at all

It may be related  to system/browser scaling, from by experience widgets jiggling behaviour is different depending on the scaling value

So testing this on Linux and Windows, I am still not able to reproduce this. @whiskeyPeak If what @kirill-21 is saying is true, so you have any scaling set on your distro? Or even fractional scaling?


flip interpolator

Currently InteriorBasis.interpolator is curried, taking first the ndarray of a finite element function's degrees of freedom and then the ndarray of the coordinates of some abscissæ.
If one runs a long unsteady simulation with many time-steps on the same mesh (i.e. same InteriorBasis), it's common to ‘probe’ a fixed array of points and record the interpolated solution there; e.g. gdmcbain/fenics-tuto-in-skfem#26 which is a two-dimensional incompressible Navier–Stokes problem with 5000 time-steps in which the ElementTriP1 pressure is logged at two points.  Using InteriorBasis.interpolator would require a new interpolator at each time-step.  If one could flip interpolator so that it was defined for its InteriorBasis on a fixed array of points and then took a solution ndarray, it would just be a matter of calling the same flipped interpolator on each new snapshot.
I think most of the uses in the suite of examples to date just call it once, so it wouldn't make much difference to them whether the method were curried either way or uncurried, but I can imagine cases in which the current currying is desirable; e.g. tracing an integral curve on a vector solution.
The current version of the unsteady Navier–Stokes problem has a hardcoded flip of interpolate, a function probe, but since it seems like a reasonably common thing to want to do, I intend to think about how to refactor the existing method in scikit-fem to provide both orders of currying.

Cool! Is it enough to have two keyword arguments, e.g.,
def interpolator(self, solution: Optional[ndarray] = None, points: Optional[ndarray] = None):
    if solution is not None and points is None:
        # return original interpolator
    if solution is None and points is not None:
        # return "probe"
    raise Exception

Yeah, we could do that; type-wise it might be funny rhough. I mean literally in present Python the missing type would be the same either way, just ndarray, sunce that covers both solution and points but conceptually they're quite different. The new one with the solution missing returns a linear form on solutions. So I was thinking of giving it a different name ('probe'?) to make this distinction more plain, but to share internal implementations.

Sure that's probably better. I expect numpy.typing to improve in the future and at some point we can hopefully be more explicit about the dtypes and shapes of the inputs.


Clarify the scope of the data replication setting

Clarify the scope of the data replication setting

@feddovanede I've reviewed the PR. Let me know if I can publish, or if I should wait for a go ahead.


UHF-9440: Add new field called information for the service channels

UHF-9440

What was done

This thing was fixed

How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-0000_insert_correct_branch

Run make drush-updb drush-cr

How to test

[ ] Check that this feature works
[ ] Check that code follows our standards

Other PRs

Link to other PR

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

Project coverage is 13.01%.  Consider uploading reports for the commit 9d8c3b2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #703   +/-   ##
=========================================
  Coverage     13.01%   13.01%
  Complexity      242      242
=========================================
  Files            28       28
  Lines           945      945
=========================================
  Hits            123      123
  Misses          822      822

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


Health Checks

This is useful for current usage, but becomes much more required when added to concept of #31
At any time a container could stop, error out, or become unresponsive and it would be good to monitor those at least on a very basic level of up vs down. Later we could add metrics such as latency/packet loss, send notifications.. etc
My MVP proposal:

Add a status attribute with predefined values of:

online
offline
inactive (for when you want to turn it off on purpose, but not delete)

Add a healthcheck attribute to services, jsonb format:

{ 'interval': 30000, 'type': 'tcp', 'port': 6379, 'timeout': 5000 }
empty health check config will disable it for that service

Use clockwork to trigger a health check every X seconds (based on config)
Use sidekiq to actually execute the health check calls and update the database

Yeah, I like this idea. I do think we should leave the management of the containers to Docker at this point. For example, if one dies, we should not try and bring it back (docker will do that).
Would we just ping the container port for that service to check if it's active? Or does Docker have something built in?


Escape ` inside img alt descriptions

Should fix the following error locally:
Unexpected token (Note that you need plugins to import files that are not JavaScript)
file: /tmp/tmp2zqefn3k/kit/src/routes/guides/search/+page.svelte:202:169
200:   let t105;
201:   let p15;
202:   let textContent_20 = `<img src="https://huggingface.co/datasets/huggingface/documentation-images/resolve/main/search_text_classification.gif" alt="Animation exploring `model_args...

as you can see in let textContent_20 = , svelte/mdsvex compile is using `, which is why the alt description can't have `

@Wauplin worked !
https://github.com/huggingface/huggingface_hub/actions/runs/6295850155/job/17094132182

@mishig25 Awesome! Thanks for working of it. That will finally settle this regex issue for good. Great job!


Tags inside conditions (in a table)

Hello again :)
Another question, is it possible to add a simple text tag inside a condition? If yes, what´s the approach?
i want something like this

and i´ve tried using the #lines example and the other tags outside..

i get Node has no Parent Error
next i tried this

and this

same error.. any hint on how to achieve this?

found another way around.. since the {DataAssuntoEncerrado} wont be visible if dont have data just have the {#Invisible} around "Data Encerramento" and it worked..

The conditions issue is now solved with the new v3.0.0 (and this commit in particular) but unfortunately to completely fix this issue and make things works inside a table, further work is required. This will be solved when issue #50 is solved.

Fixed in v4.0.0


Batch count and size is multiplied by itself

While Dynamic prompts is active the Batch count and size is multiplied by itself resulting in exponential more images.
Batch count: 1 / size: 2 = 4 images
Batch count: 2 / size: 1 = 4 images
Batch count: 2 / size: 2 = 16 images
I have

Turned off all extensions
Run UI in safe mode
Resetting to default
Rebooted the computer
Removed cookies
narrowed it down to Dynamic prompts

UI: SDnext (Vlad's fork)
Version: 83d1ee77 Sun Aug 13 00:32:03 2023 +0200
Dynamic prompts inactive

Dynamic prompts active Batch count: 1 / size: 2 = 4 images

Best regards,
Olivmamman

Just letting you know, I am having the same problem too. Thanks Dev.

I can confirm that I have same problem also. Having Dynamic Prompts enabled by itself does it.

https://github.com/adieyal/sd-dynamic-prompts/commit/638d4b1784b95d19b69c54f93aeee987d2c5b0a2 is to blame.
Revert to 2.15.0 to workaround this.

i have also the problem and the errors i got brought me into another extension first. There i found out through another user, that apparantly this:
https://github.com/adieyal/sd-dynamic-prompts/commit/638d4b1784b95d19b69c54f93aeee987d2c5b0a2
should be the cause of that multiplying

638d4b1 is to blame.
Revert to 2.15.0 to workaround this.

how to revert?

are you using negative prompts? Re reverting, go into the sd-dynamic-prompts directory and type git checkout 2.15.0 - you can return to the latest version with git checkout main

i use A1111 local, so how do i have to do this exactly?

ok, nevermind XD i downloaded the previous version directly through the commits-section and installed 2.15.0 manually. do you have an idea, when you can fix it?


Remove preference for "installed" installer type

Fixes #2565
Change
Move (and rename) the InstalledTypeComparator from being a comparator to just a filter.  Given what we can (or rather, currently do) actually discern from system state, it doesn't make sense to give preference to the few types that we use to signify the compatibility sets.  If we truly want to preserve installer type selection within a compatibility set, we need to store the user selection into the tracking database similar to UserIntentArchitecture.
Validation
Add a test for the case as called out by the issue; regression tests cover it as well and the only one that needed updating was relying on the intentionally changed behavior.
While this change is arguably "correct", it will cause different choices to be made if there are packages with multiple compatible installers that are equal in the other relevant dimensions.  I'm working on determining the extent of impact to winget-pkgs via a custom validation pass.
If it turns out to be overly broad, the most likely solution will be to keep the existing behavior but drop from Strong to Weak for the case of equal installer types.
Microsoft Reviewers: Open in CodeFlow

The only issue I uncovered when searching for package versions with installers in the same compatibility set that would be indistinct for upgrades was with multiple versions of ToEverything.AFFiNE (since they added an Nullsoft version of their installer).  For example: https://github.com/microsoft/winget-pkgs/blob/master/manifests/t/ToEverything/AFFiNE/0.16.3/ToEverything.AFFiNE.installer.yaml
This is already an issue during upgrade, as for the same reason as the original issue we will prefer exe.  With my change, the two x64, user scope installers will be equivalent in priority for upgrade (as they are for install already).  We will still choose the exe as it comes first in the list rather than because it matches the detected type of exe.
This isn't really an issue, and it sets the likelihood of needing the UserIntentInstallerType to be implemented to very low.


Assembly entry

PR checklist

[x] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!

[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/metatdenovo branch on the nf-core/test-datasets repository.

[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker).
[x] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I added an option --assembly to allow entry with an assembly and skip the assembly step.
I also made some unrelated changes here and there! :-S

the PR is against the master branch, is it correct?


Concatenate list of streams with ffmpeg

I’d like to automatically rearrange the scenes of a video. How do I use ffmpeg-python to concatenate a list of n streams/scenes?
Unfortunately, I couldn’t figure out how to do it properly.
The status quo can be found here.

stream_list = [ … ]
stream ffmpeg.concat(*merge_list, v=1, a=1)


mat-table-extensions with server side data source

Hello,
First of all thank you so much for these libraries you are contributing with, they are of great help for many of us.
I'd like to know if mat-table-extensions is able to work with custom server side data sources.
I have a mat-table and a paginator, to feed mat-table with data I have my own custom DataSource (implementing DataSourceinterface) for fetching data from the server when we interact with paginator. The problem is when I try to export the table mat-table-exporter goes through all the pages of my table but it outputs the data of the first page on every page. Example:
Table page 1 contains rows A, B and C
Table page 2 contains row D and E
After exporting with mat-table-exporter my document has 6 rows: A, B, C, A, B, C.
I've checked a bit in the source and observed even though paginator was changing page the contents of the table did not change accordingly, they stayed the same as if the table was on page 1. Am I missing something here?
I'm using:

Angular 6.1.9
Angular material/cdk 7.0.2
mat-table-exporter 1.2.5

@dmbarreiro Have you got any good news for us? :)

We do not have the problem anymore with the newer version. Sorry for being irresponsive, haven't had time to reproduce it on stackblitz with the old version. From my point of view we can close the issue.

Thanks for the response. Closing then.

Hello, I have this problem with the most current version of the angular. Paging on the server side, in the same way that @dmbarreiro  reported.


fix: update to eye v2.7.3

fix: update to eye v2.7.3

:tada: This PR is included in version 2.9.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v2.9.3

Your semantic-release bot :package::rocket:


Array declaration fails in global scope

Can't put this in global scope
imm numbers [3; i32] = [3, 4, 5]

Fixed.
Please close if you agree.
(Note:  you don't need to specify the type after numbers. Inference will figure it out.)


Highlight green technologies

From Vicky Morley,
Where relevant, it would be useful to highlight the extent to which there are green technologies such as renewables (power) and electric cars (automotive) that can grow as part of the transition to a green economy

this is covered in the updated version


API does not work with the ChatMessage

Hayhooks fails with Chat Messages because the pydantic conversion to a dict remove ChatMessage class.
I had to work around like this:
        dict_local = pipeline_run_req.dict()
        dict_local["prompt_builder"]["prompt_source"] = pipeline_run_req.prompt_builder.prompt_source
        result = pipe.run(data=dict_local)

Hey!
I have a similar problem where I send a messages dict to a Multiplexer component in a pipeline. But it is not converted to a List[ChatMessage] but rather a dict is passed on.
So my serialized pipeline looks like this:
chat_pipeline = Pipeline()
chat_pipeline.add_component("msg_receiver", Multiplexer(List[ChatMessage]))
chat_pipeline.add_component("generator", chat_generator)
chat_pipeline.connect("msg_receiver.value", "generator.messages")

and in my API call
  "data": {
    "msg_receiver": {
      "value": [
        {
          "content": "Test",
          "role": "user",
          "name": null
        }
      ]
    },
    "generator": {},

this raises an error in the Generator:

File "/home/anaconda3/envs/hsenv/lib/python3.10/site-packages/haystack/core/pipeline/pipeline.py", line 864, in run
res = comp.run(**last_inputs[name])
File "/home/anaconda3/envs/hsenv/lib/python3.10/site-packages/haystack/components/generators/chat/hugging_face_api.py", line 194, in run
formatted_messages = [m.to_openai_format() for m in messages]
File "/home/anaconda3/envs/hsenv/lib/python3.10/site-packages/haystack/components/generators/chat/hugging_face_api.py", line 194, in
formatted_messages = [m.to_openai_format() for m in messages]
AttributeError: 'dict' object has no attribute 'to_openai_format

Grateful for any idea how to fix or workaround!


Documentation updates

Coverage decreased (-9.3%) to 52.725% when pulling 5b244918a061cccfa161ada31f3eb3eba20647c5 on kthSim:docs into 84e782ece441da31648ad963bdf71997280426db on CS2103-AY1819S2-W17-2:documentation.


Make follow-up improvements to validate_record

There's also a few more changes to make around this (as a follow up to #105):

rename the argument to validate_contents, and make it kw-only.
should validate that everything has a hash/size even if validate_contents is False.
should validate that RECORD signatures are not mentioned in the RECORD.
should validate that the RECORD file is mentioned without hash or size.
documentation needs to present an example and add cross-references to/from WheelSource.validation_error.

FWIW, I'm also not a big fan of the inconsistency within test_sources.py around function tests vs method tests and would prefer to unify things under the TestWheelFile class.

Thanks @BlueGlassBlock! You're awesome! ❤


TestAdapter 3.7 does not work with multiple TargetFrameworks

Describe the bug
I've got a class library project and a MSTest project that both have <TargetFrameworks>net8.0;net472</TargetFrameworks>. With MSTest.TestAdapter 3.6.4 (in the latest VS2022) two sets of unit tests are shown in the Test Explorer, one for each target framework, and tests from both sets can be selected and run.
After upgrading to 3.7.0 the net472 tests are no longer discovered. The test output states:
No test is available in ...\net472\mytestassembly.dll. Make sure that test discoverer & executors are registered and platform & framework version settings are appropriate and try again.

Steps To Reproduce
MSTestAdapter_364 - Example test project with MSTest.TestAdapter 3.6.4. The test can be run for both net472 and net8.0
MSTestAdapter_370 - Same project after "upgrading" to MSTest.TestAdapter 3.7. The net472 tests cannot be run anymore.

Thanks for reporting this @frobijn.
This looks like a duplicate of https://github.com/microsoft/testfx/issues/4426. I will go ahead and close. Meanwhile, you can subscribe to notifications on #4426.


add support of EPUB description field

It would be nice to be able to set the description field.

Coverage increased (+0.2%) to 81.733% when pulling 41e95ad1d91e441ff4beff71e0ce3d52836ec63f on propan:master into f15a3ab1ad2aa85e05a87ca7a1647f7c2067afaf on bmaupin:master.

Great work, thanks! I especially appreciate that you added tests :)


jbrowse: 2.11.0 -> 2.11.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/GMOD/jbrowse-components/releases.
meta.description for jbrowse is: The next-generation genome browser
meta.homepage for jbrowse is: https://jbrowse.org/jb2/
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.11.1 with grep in /nix/store/jkn0x1d3s88hjgryjp5xghm75by4rznm-jbrowse-2.11.1
found 2.11.1 in filename of file in /nix/store/jkn0x1d3s88hjgryjp5xghm75by4rznm-jbrowse-2.11.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

jbrowse

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jkn0x1d3s88hjgryjp5xghm75by4rznm-jbrowse-2.11.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A jbrowse https://github.com/r-ryantm/nixpkgs/archive/f56f4c9feca8c6f755186863a73afa3c0397468d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/f56f4c9feca8c6f755186863a73afa3c0397468d#jbrowse

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jkn0x1d3s88hjgryjp5xghm75by4rznm-jbrowse-2.11.1
ls -la /nix/store/jkn0x1d3s88hjgryjp5xghm75by4rznm-jbrowse-2.11.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

jbrowse

Maintainer pings
cc @benwbooth for testing.

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

1 package built:

jbrowse


Update CI OSes

Remove deprecated buster, centos 7 and bionic.

Hrm, CentOS 7 is supported until June 30, 2024.  However, CentOS 8 was supported until end of 2021: https://endoflife.date/centos — personally, I will miss 7, it helped us to catch many curious mistakes last year.

8 support will go until 2029 at least https://access.redhat.com/support/policy/updates/errata/
Yeah, we'll definitely miss CentOS 7, it was a good helper finding various bugs all those years.

Your link points to RHEL8 which probably shares the same assets but managed differently?


domain

[Enter feedback here]

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

ID: a2382361-1699-3beb-574e-0879c87f7442
Version Independent ID: 97f40a33-983d-f927-beb5-cb03aa3f82dd
Content: Known issues for application provisioning in Azure AD
Content Source: articles/active-directory/app-provisioning/known-issues.md
Service: active-directory
Sub-service: app-provisioning
GitHub Login: @kenwith
Microsoft Alias: kenwith

@sonnyng, Thanks for leveraging Azure docs feedback channel to raise this question.  We notice that you have closed this thread, if you still have questions or need help with this, could you please share the Azure documentation for which this feedback is applicable to?
This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements.


I cant use the oswald from W.I

Describe your bug here.
I cant use the oswald from W.I My psych engine crashes
Command Prompt/Terminal logs (if existing)
flixel/graphics/frames/FlxAtlasFrames.hx (line 246)
Character.hx (line 164)
PlayState.hx (line 940)
flixel/FlxGame.hx (line 637)
flixel/FlxGame.hx (line 724)
flixel/FlxGame.hx (line 677)
flixel/FlxGame.hx (line 545)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/ShadowMario/FNF-PsychEngine

> Crash Handler written by: sqirra-rng

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No

Not a bug
Missing character texture files

Oswald and many other W.I. characters use JSONs instead of XMLs, but not like an animation atlas JSON.

Not a bug Missing character texture files

I have the texture and the json (the oswald sprite use JSON not XML)

Oswald and many other W.I. characters use JSONs instead of XMLs, but not like an animation atlas JSON.
How i can fix it? i dont have adobe animator

I have the texture and the json (the oswald sprite use JSON not XML)

psych doesn't support json, you'll have to make an xml

I have the texture and the json (the oswald sprite use JSON not XML)

psych doesn't support json, you'll have to make an xml

It literally has support for texture atlas

which uses a completely different system you clown

which uses a completely different system you clown

yeah, but still, support for json characters, instead of just xml

they're not using texture atlas, your comment is pointless

they're not using texture atlas, your comment is pointless

still, they're using flxanimate, and here's the description for it Adobe Animate's texture atlases player for HaxeFlixel

Don't know what you're trying to prove here since texture atlas has nothing to do with this post, like I said

Don't know what you're trying to prove here since texture atlas has nothing to do with this post, like I said

that's what the characters in Wednesdays infidelity use, Oswald as well, they use an texture atlas, and in the source code, they use an texture atlas to load the characters, so yeah, it's probably possible to use Oswald's character file and sprites, and everything you said before was wrong

some proof of evidence

that's what the characters in Wednesdays infidelity use

No? They use texture packer just like XML but in json format

Know what i will Make a BETADCIU for other song

CLOSED

that's what the characters in Wednesdays infidelity use

No? They use texture packer just like XML but in json format It's literally in your screenshot?

look it doesn't even matter anymore, the issue's closed, so you can just piss off


GPL Ghostscript 9.53.3: Unrecoverable error, exit code 1

SUMMARY
I have a particular PDF document that triggers a CUPS Ghostscript error. Since the PDF viewer I use is KDE Okular, I first reported the issue there but I have been told that is not an Okular problem.
Since I cannot publish the PDF here, I will send it by e-mail to any of the Okular developers that need it.
I attach CUPS debuginfos that I got by using "LogLevel debug" in /etc/cups/cupsd.conf
STEPS TO REPRODUCE

In KDE Okular open the document, press CTRl + P to print
Open Options button, select "Pages" and write 544 (or any other number)
Click Print button

OBSERVED RESULT
Nothing happens, document stuck in CUPS print queue
EXPECTED RESULT
Print the page
SOFTWARE/OS VERSIONS
cups 2.3.3
poppler 0.90.0
Okular 20.08.3
KDE Frameworks 5.76.0
Qt 5.15.1
Fedora 33
cups_errors.txt

thanks for mentioning the bug @Germano0
I think the problem can be generalised a little bit more :-)
Can you please try printing from some other application like chrome or firefox and ensure whether same behaviour is observed there too or is this the problem with KDE ocular only.
Thanks :-)

Can you please try printing from some other application like chrome or firefox and ensure whether same behaviour is observed there too or is this the problem with KDE ocular only.

I tried using Evince and I do not experience the bug with that PDF reader

@Germano0  can you please show me your ppd file as well, it must be your "/etc/cups/ppd/Officejet_7110.ppd" file
thanks :-)

Hello @surajkulriya , I have just sent you the files by e-mail.
Thank you

@Germano0, please do all communication here in the bug report, also attach any files you get asked for here to this bug report instead of sending them in private e-mail, as not only @surajkulriya is reading here and trying to help you but also others. There are also other having the same problem as you and your files could help them.
If the system rejects your attachments make sure that you rename text files (like PPD files or log files) to have a .txt file name extension. Thanks.

Officejet_7110.txt (PPD)

hello @Germano0
actually I was not able to reproduce the bug. I tried the thing you mentioned above using the ppd file you provided and it got exited with no errors.
By the way problem seems to exist in your gstoraster filter. Maybe it's getting started before the exiting of pdftopdf filter.
You can try to reinstall the cups and cups-filters from the source code and check if the problem still exits.
In case the problem is not solved please do tell us and provide the file /var/log/cups/error_log
Thanks

hello @Germano0
actually I was not able to reproduce the bug. I tried the thing you mentioned above using the ppd file you provided and it got exited with no errors.
By the way problem seems to exist in your gstoraster filter. Maybe it's getting started before the exiting of pdftopdf filter.
You can try to reinstall the cups and cups-filters from the source code and check if the problem still exits.
In case the problem is not solved please do tell us and provide the file /var/log/cups/error_log
Thanks

@Germano0
Did you reinstalled the cups from source and retried the issue?
What were the results?

@Germano0
Did you reinstalled the cups from source and retried the issue?
What were the results?

You can try to reinstall the cups and cups-filters from the source code and check if the problem still exits.

I cannot install cups and cups-filters from sources. Note that Fedora CUPS is packaged by the same Red Hat employees that package CUPS for Red Hat Enterprise Linux, so high quality packaging is assured.
I enabled CUPS debug logging, restarted the service, triggered the problem and collected the following logs
cups.txt

You can try to reinstall the cups and cups-filters from the source code and check if the problem still exits.

I cannot install cups and cups-filters from sources. Note that Fedora CUPS is packaged by the same Red Hat employees that package CUPS for Red Hat Enterprise Linux, so high quality packaging is assured.
I enabled CUPS debug logging, restarted the service, triggered the problem and collected the following logs
cups.txt

Just found out another PDF that has a similar situation on Okular: open the PDF, print a single page from the PDF: nothing will happen, and print queue will be emptied immediately.
Evince instead prints without problems
affected file.zip
cups log


Some Trickle ICE related issues

Commit https://github.com/diasdavid/webrtc-explorer/commit/a5a3a785a6acbed2eb76f7acba83f08bc6fa5cbe removes bows and updates , as it's not compatible with node, everything else seems also to work with nodejs.
Sorry, there are also some other nodejs related changes incorporated.

Hi @areiter , thank you for showing up and contributing.
Could you explain me a bit more about "removing bows because it is not compatible with node"? webrtc-explorer was built to run on the browser.

I see that your goal is to have WebRTC explorer working both in the browser and in Node.js? That's pretty cool.
Have you looked at the Signalling Server? It is the rendezvous point for WebRTC peers to connect between themselves - https://github.com/diasdavid/webrtc-explorer-signalling-server

I'm aware of the signalling server as an integral part of the system.
As you noticed several times, my goal is to have it running in node-js.
Your main goal is to support browsers, although maybe some of the changes are also relevant for you.
From my point-of-view, an ideal solution would be to have a single module, running in the browser and in the node environment.

From my point-of-view, an ideal solution would be to have a single module, running in the browser and in the node environment.

I agree, and if we can work together on that, I'm all aboard. Nevertheless, I should inform you, that if you are looking for DHT for Node.js only, you probably don't want to use webrtc-explorer or webrtc at all, you could avoid all of the signalling stuff or even having to have several Data Channels open, since dealing a socket is way cheaper.

I opened a issue of where to go next - https://github.com/diasdavid/webrtc-explorer/issues/5

Browser <--> Node interoperability would be really nice :-)


Implement Download Button and Caption

We can also implement a caption feature like YT videos and a download button in every video just like a youtube video so it will be easy for the viewer.

We have that. Thank you!


added 'main' property to package.json for browserify

Without this, browserify won't work. Failing with:
Error: Cannot find module 'field-selection' from '...mypath...'

thanks


Attempt at fixing "Set up sphinx to build the documentation #31"

So, I followed the docs from Sphynx, and it sort of works.
With 'sphinx-build source build' command from the terminal, the html files for the .rst pages are generated. But I feel that this probably needs more work.

Hi @dujm, I was considering that, but I'm not sure whether it's a good idea or not. Since the content of build/ is generated, some may argue that it shouldn't be in the repo.

Just wondering if this the make file I would be calling in my azure-pipeline? @datapythonista

Good catch @bhavaniravi, you guessed well. :)
We'll in the future add more things to the CI, and I'll propose changing that make file by something else, but for the moment, you're totally correct.

Resolved the mentioned concerns. As for .gitignore, perhaps we should start another issue to add it and start populating with things?

Resolved the mentioned concerns. As for .gitignore, perhaps we should start another issue to add it and start populating with things?

sure, creating issues is cheap, and help not forget things, feel free to create as many as things come to your mind :)

Hi @MKhalusova! Thank you, this is great.
I tried building the docs using hub as suggested by @datapythonista & I came across pages that have something like {{ header }} on the top of the page. I'm not sure if it's just my build or not, and if it's not just me is it something that we need to configure in Sphynx?

@galuhsahid this is where it is implemented in pandas: https://github.com/pandas-dev/pandas/blob/master/doc/source/conf.py#L309

@galuhsahid Thanks! I didn't notice that. I'll try to fix it today.

Ok, I'm a little stuck.
In .rst files {{header}} is a jinja thing where header is a variable. Where is it defined? My guess is - in conf.py.
In my understanding, if I simply write header =" "  in conf.py that should replace all generated {{header}} lines in the html files to a white space. But that doesn't happen.

I thought sphinx was doing that, but looks like we implemented that ourselves in pandas: https://github.com/pandas-dev/pandas/blob/master/doc/source/conf.py#L695
I guess we want to do the same here (can surely be in a follow up PR), but we can also have the header variable being displayed

@datapythonista I think it deserves a separate PR, so we don't put too many eggs in one basket.

agree, with the level of complexity that we're dealing, I think a divide and conquer approach is preferred, and just iterate in small steps at every PR

@datapythonista Is it OK if I make a separate issue then?

@galuhsahid surely ok. I think it's a good practice to create issues for everything, so things are not forgotten, and also people can discuss, give ideas, coordinate. Thanks for offering!

Since the header thing is now a separate issue, is there something else stopping us from merging this PR?


udp: remove unnecessary string conversion in gso batch writer

Signed-off-by: Dan Zhang<EMAIL_ADDRESS>Risk Level: low, not in use
#15954

/assign @mattklein123

Can this be merged?


Feature: use @component test in sql tests

What this PR changes/adds
Adds the @ComponentTest annotation to all SQL-related tests.
Why it does that
Those tests are not unit tests (the use non-mocked collaborators such as H2) and therefore should only run when triggered explicitly to reduce test runtime on local machines.
Further notes

I also added a short test summary to the root buildfile, which produces output such as-----------------------------------------------------------------
|  Results: SUCCESS (32 tests, 32 passed, 0 failed, 0 skipped)  |
-----------------------------------------------------------------

Linked Issue(s)
Closes #1132
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[ ] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

#1151 implements @marcgs's suggestion of moving the summary out to a plugin. whichever PR we merge first - the other must remove the code from the build file.


Remove TypeScript

This removes TypeScript and all dependencies and only keeps a simple
Javascript file that does not have any dependencies and can be run
directly. There is not need for an NPM build any more and we just
call a shell script via Javascript.
With this, we no longer have to commit node_modules and Javascript
compiled from TypeScript code. Also, the shell script can easily be run
directly for local testing.
Signed-off-by: Reinhard Naegele<EMAIL_ADDRESS>

Oh, do you also want to update https://github.com/scottrigby/chart-testing-action/blob/master/.gitignore now that there's less to ignore? 😄

Never mind, let's clean that up separately. This is great as-is 😸


Content/multi model pipelines

Pull request for deliverable 4 (multi-model pipelines) which includes

New content on the landing page

Example notebook for multi-model pipelines

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

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

@yeandy @rezarokni

The diff won't render so I can't leave comments inline on the notebook unfortunately. Mostly, this LGTM, my only comments are:

In the Model handlers section

The key is used o keep a reference

Should be `The key is used to keep a reference

Could you please add the comment about issue #21863 to CLIPWrapper and PytorchNoBatchModelHandlerKeyedTensor
Could you please fix the whitespace precommit violations and resolve the merge conflicts?

@yeandy I know you had thoughts about using cascade vs ensemble, does that look right in this PR? Also feel free to drop in other comments of course :)

thanks for reviewing @damccorm. I resolved the issues mentioned above.


[CTI] Add hosting bundle installer tests

Closing as the ball is rolling in the tooling repo and will finalized by them. See https://github.com/aspnet/Tooling-ManualTests/issues/310


Fix over orders

Context
The existing implementation of school allocations overloads and mixes the meaning and role of raw_cap and order_state fields for schools with some undesirable consecuencies:

When a school order_state is changed, an automatic mechanism to auto-adjust the value of raw_cap is triggered before persisting the change. This way, we are persisting twice the fact that a school is set to 'orders for this school not allowed': order_state: :cannot_order and raw_cap: raw_device_ordered. This leads to eventual data inconsistencies in the system very difficult to detect, prevent and fix. The test suite is full of samples where schools are simultaneously in :cannot_order whilst device numbers mismatch that state:

 {
  order_state: :cannot_order,
  raw_laptop_allocation: 5,
  raw_laptop_cap: 3,
  raw_laptops_ordered: 1
 }

Another caveat due to the implementation above affects over-orders preventing us from being able to accurately store the cap stolen from sibbling schools in virtual caps:
If a school in a vcap has over-ordered devices, its raw_allocation and raw_cap values are changed to match the new raw_devices_ordered value. At the same time, sibbling schools in the vcaps get as many over-ordered devices stolen from their caps to compensate the overall vcap allocation. However, due to the current implementation:
If we decrease the raw_cap of a sibbling school to represent the stolen devices, it will automatically be set back to match their raw_allocation value if the school is in :can_order state (because of the auto-adjust mechanism described above).
To overcome this issue, we need to reduce both, raw_cap and raw_allocation. This means, we lose the initial allocation assigned to  the sibblings schools involved.

Changes proposed in this pull request
As we can see from above, there are multiple variables to take into account to compute the final effective cap of a school. Currently we are considering a few of them: assigned allocation, order_state, number of stolen/over_ordered devices, number of devices ordered... In the future we might add new ones.
The current implementation trying to persist the final result in raw_cap_x has proved not to be valid in all scenarios. We should be able to persist all the variables affecting a school cap and compute the result based on it whenever we need it. Only that way we can fully cope with all the requirements and leave it open for future extensions.
This PR is aimed to fix the issues above. The following changes are proposed:

Remove the cap auto-adjust mechanism when allocation numbers are going to be persisted.
Store devices gained/lost because of own/sibblings over orders in new fields :over_order_reclaimed_laptops and :over_order_reclaimed_routers
Store devices lost after changing the state of a school to :can_order_for_some_circumstances in new fields :circumstances_laptops and :circumstances_routers
Remove raw_laptop_cap and raw_router_cap from School schema.

School cap is now a function of:

order state,
allocation,
over order devices gained/lost
circumstances devices lost

if cannot_order?
  raw_devices_ordered
else
  raw_allocation + over_order_reclaimed_devices + circumstances_devices
end

This will allow us to:

Know and display the initial allocation assigned to any school anytime.
Know and update/give-back the number of devices gained/lost due to over orders transactions.
Know and update/revert the number of devices lost because of state changes to can_order_for_some_circumstances state and back.
Set back the previous cap of a school  when it is temporarily set to cannot_order and back to  a can order state with no loss of information.
Easily extend cap with new components always keeping/displaying/updating data related to other variables composing the allocation cap.

Guidance to review

 Over order handling

LGTM