Build fails after updating to cloud_firestore 0.8.0

Steps to Reproduce

After updating cloud_firestore from 0.7.4 to 0.8.0 by VSCode builds failed (on the BETA channel)
(VSCode -> Start Debugging)
Tried a clean and rebuild (flutter clean), no difference
Tried with Android Studio instead of VSCode - still fails
Updated to DEV Channel. Still fails
Updated to Master - still the same
Switch back to 0.7.4 - builds and runs.

Logs

Launching lib\main.dart on SM N910G in debug mode...
registerResGeneratingTask is deprecated, use registerGeneratedFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedFolders(FileCollection)
registerResGeneratingTask is deprecated, use registerGeneratedFolders(FileCollection)
C:\Users\cborn\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cloud_firestore-0.8.0\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java:159: error: cannot find symbol
        query = query.whereArrayContains(fieldName, value);
                     ^
  symbol:   method whereArrayContains(String,Object)
  location: variable query of type Query
C:\Users\cborn\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cloud_firestore-0.8.0\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java:643: error: cannot find symbol
        return FieldValue.arrayUnion(readValue(buffer));
                         ^
  symbol:   method arrayUnion(Object)
  location: class FieldValue
C:\Users\cborn\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cloud_firestore-0.8.0\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java:645: error: cannot find symbol
        return FieldValue.arrayRemove(readValue(buffer));
                         ^
  symbol:   method arrayRemove(Object)
  location: class FieldValue
Note: C:\Users\cborn\AppData\Roaming\Pub\Cache\hosted\pub.dartlang.org\cloud_firestore-0.8.0\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
3 errors

FAILURE: Build failed with an exception.
* What went wrong:
Execution failed for task ':cloud_firestore:compileDebugJavaWithJavac'.
> Compilation failed; see the compiler error output for details.
* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
* Get more help at https://help.gradle.org
BUILD FAILED in 13s
Gradle build failed: 1
Exited (sigterm)

[√] Flutter (Channel master, v0.8.1-pre.50, on Microsoft Windows [Version 6.1.7601], locale en-AU)
    • Flutter version 0.8.1-pre.50 at D:\usr_chronos\Sandbox\flutter
    • Framework revision 3c58195350 (2 hours ago), 2018-09-07 09:13:13 +0200
    • Engine revision 58a1894a1c
    • Dart version 2.1.0-dev.3.1.flutter-760a9690c2

[√] Android toolchain - develop for Android devices (Android SDK 27.0.2)
    • Android SDK at D:\Users\cborn\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.2
    • ANDROID_HOME = D:\Users\cborn\AppData\Local\Android\sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
    • All Android licenses accepted.

[√] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 28.0.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] VS Code (version 1.27.1)
    • VS Code at C:\Users\cborn\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 2.18.0

[√] VS Code, 64-bit edition (version 1.26.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 2.18.0

[√] Connected devices (1 available)
    • SM N910G • 0ab5d05b • android-arm • Android 6.0.1 (API 23)

• No issues found!

Just a guess. You could create a new Flutter project, and compare the build.gradle files (android/build.gradle and android/app/build.gradle) of the old and new project and update the old accordingly.

@zoechi
Good guess, it gave me a direction to look in!
In order to build originally, I had added to by build.gradle a suggestion given here: https://github.com/flutter/flutter/issues/20486
I removed that code, then the missing symbols were resolved, but the build failed on a dex error.
I then added
multiDexEnabled true

and my build completed and is running, thank you!
I then created a new project and compared the build.gradle files, there are some changes which I'll try merging into my project tomorrow, as it getting late here now.
I don't know why just updating the plugin version caused the crash, or what I've done to fix it, being somewhat new to this build system, should I close the issue now, or leave it in case others come across it with the update?

Glad to hear!

I changed the build.gradle settings in my project to match the ones in the newly created one, basically
classpath 'com.android.tools.build:gradle:3.0.1' changed to 3.1.2, and the two androidTestImplementation dependencies got a version bump, after that my build failed saying I needed to upgrade gradle from 4.1 to 4.4
I did that in the gradle-wrapper.properties file as suggested, and the resulting build completed (with a lot more warnings than before) and runs successfully in debug, haven't tried release yet.
I still have to have multiDexEnabled true to build or it complains about more than 64K methods, which seems strange for a simple 3-page test app?
After all that I can now access FieldValue.serverTimestamp() from the new plugin, but unfortunately I don't think it does what I had hoped, which was to get the current time on the server as we do in our SQL code! And I haven't been able to figure out how to access SnapshotMetadata to see if the data I'm getting is fresh or stale (.fromCache), so lots to learn!

My android gradle build is working now. However iOS build is still failing on my Mac. Does anyone know how to fix this issue for iOS?

What exactly is the issue on iOS? Any error output?

Flutter beta channel,
Launching lib/main.dart on iPhone X in debug mode...
Starting Xcode build...
Xcode build done.
Failed to build iOS app
Error output from Xcode build:
↳
** BUILD FAILED **
Xcode's output:
.......
.......
/Users/Vic/Development/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-0.8.0/ios/Classes/CloudFirestorePlugin.m:50:22: error: no visible @interface for 'FIRQuery' declares the selector 'queryWhereField:arrayContains:'
query = [query queryWhereField:fieldName arrayContains:value];
~~~~~ ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.....
.....
19 warnings and 3 errors generated.
Could not build the application for the simulator.
Error launching application on iPhone X.

Does your iOS build work? Have you tried it?

I think it would be better to create a new issue.
This one is closed and about Android.

Ok. Thanks @zoechi

The new version of all official Google plugins swaps the use of Android support to use android x instead.
You need to migrate to Android X
how to fix it ??
here the solution
set minSdk and targetSdk on build.gradle
        minSdkVersion 21
        targetSdkVersion 28

and then add
android.useAndroidX=true
android.enableJetifier=true

in gradle.properties

28

also set compileSdkVersion 28 at build.gradle

C:\Program Files\flutter.pub-cache\hosted\pub.dartlang.org\firebase_messaging-4.0.0+3\android\src\main\java\io\flutter\plugins\firebasemessaging\FirebaseMessagingPlugin.java:19: error: cannot find symbol
import com.google.firebase.iid.InstanceIdResult;
^
symbol:   class InstanceIdResult
location: package com.google.firebase.iid
C:\Program Files\flutter.pub-cache\hosted\pub.dartlang.org\firebase_messaging-4.0.0+3\android\src\main\java\io\flutter\plugins\firebasemessaging\FirebaseMessagingPlugin.java:120: error: cannot find symbol
new OnCompleteListener() {
^
symbol:   class InstanceIdResult
location: class FirebaseMessagingPlugin
C:\Program Files\flutter.pub-cache\hosted\pub.dartlang.org\firebase_messaging-4.0.0+3\android\src\main\java\io\flutter\plugins\firebasemessaging\FirebaseMessagingPlugin.java:122: error: cannot find symbol
public void onComplete(@NonNull Task task) {
^
symbol: class InstanceIdResult
C:\Program Files\flutter.pub-cache\hosted\pub.dartlang.org\firebase_messaging-4.0.0+3\android\src\main\java\io\flutter\plugins\firebasemessaging\FirebaseMessagingPlugin.java:118: error: cannot find symbol
.getInstanceId()
^
symbol:   method getInstanceId()
location: class FirebaseInstanceId
I am  facing the same problem,
Followed all the recommended procedures but no luck .
Followed procedures.

Flutter clean
flutter packages pub cache repair
Restart Android
Remake emulator
nothing is working

2. flutter packages pub cache repair

same here, no luck also followed https://flutter.dev/docs/development/packages-and-plugins/androidx-compatibility
seems like only firebase messaging plugin is giving me issues


Homematic IP Cloud integration Cover drive in the wrong direction

Home Assistant release with the issue:

0.87.0
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Component/platform:

Homematic IP Cloud Cover
Description of problem:
Since version 0.87.0 covers are supported.
I believe the problem exists because in the Hame Assistant 100% means open and 0% means close. In Homematic, 100% means closed and 0% means open.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Additional information:

I think this is solved and could be closed.

Yes this is solved and can be closed


Re-export Comparator and LexicographicComparator traits

Pull Request
Related issue
A minor followup on https://github.com/meilisearch/heed/pull/212
What does this PR do?

Reexport 2 traits such that it'll be easier to use them; now I have to explicitly include heed-traits in my Cargo.toml

PR checklist
Please check if your PR fulfills the following requirements:

[x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[x] Have you read the contributing guidelines?
[x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!
@Kerollmops I forgot this in my last PR, and didn't realize this until I started to use the new version of heed just now, haha

The v0.20.0-alpha.6 is now available on crates.io 🎉


Fix parsing of version number for libraries with a digit as part of the name

... (such as "d3-selection")
This should be done before we merge DefinitelyTyped/DefinitelyTyped#10741

👍


[Enhancement⚡] Add #[inline] for SendResult struct methods

Before Creating the Enhancement Request

[X] I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary
No response
Motivation

Describe the Solution You'd Like
No response
Describe Alternatives You've Considered
No response
Additional Context
No response

I want to try

I want to try

@ad-source-bit  sure!


Agile Learner gives warning if you trade your only basic power/spell in a class for an advanced power/spell

When the user purchases Agile Learner for a class that only has a single basic power, Agile Learner will throw a warning when selecting that class to swap powers, that it is no longer valid.
As a separate, but similar issue, Agile Learner retains your class selection if you remove it, and then later add it back, even if the previous selection is no longer an option.  For example I added agile learner, to Sourcerer (level 2), removed it, removed sorcerer entirely, added artisan level 2, added agile learner back, and the selection was still on sourcerer when it was added back even though it wasn't even on the sheet.

Splitting off the issue of choice data persisting across purchases as #152.


Additional Linter Options

I have one linter config setup for our shop and share it between our developers in order to keep our code consistent.
This works, but I see an opportunity for additional code formatting options in the rpgle linter. Here are several suggestions...
- Include space after "(" and before ")" ;
         example:   Int(10)    would be converted to   Int( 10 )

-  Include space [ before | after | nospace ] with the parameter separator ":" character
         example:    nospace     Procedure( Parm1:Parm2:Parm3 )
                            before         Procedure( Parm1 :Parm2 :Parm3 )
                            after            Procedure( Parm1: Parm2: Parm3 )

-  Require or remove Const keyword on Dcl-C statements
          example:     require      Dcl-C  SOMECONSTANT  Const( 'XYZ' );
                              remove      Dcl-C  SOMECONSTANT   'XYZ';

I suggest adding an additional option surround for the parameter separator:
-  Include space [ before | after | surround | nospace ] with the parameter separator ":" character
         example:    nospace     Procedure( Parm1:Parm2:Parm3 )
                     before      Procedure( Parm1 :Parm2 :Parm3 )
                     after       Procedure( Parm1: Parm2: Parm3 )
                     surround    Procedure( Parm1 : Parm2 : Parm3 )


Update cloudformation-base.yml

Changed LisBucket to ListBucket
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@odellus Was there a reason you closed this? It certainly looks wrong,

It does, doesn't it. I was debugging some changes I made to the
cloudformation, saw this and thought it might have been the problem. It
wasn't so I closed.
I closed about 16 hours into a 22 hour shift yesterday, so I might not have
been as clear headed as I am right now.
I can reopen if you'd like. It does seem to be a typo though I don't see it
producing any errors (oddly enough).
On Wed, Oct 14, 2020, 10:41 PM Tom Faulhaber<EMAIL_ADDRESS>wrote:

@odellus https://github.com/odellus Was there a reason you closed this?
It certainly looks wrong.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/sagemaker-run-notebook/pull/6#issuecomment-708910847,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDYI3E6HJZCEFNJYKQ3HITSK2DRHANCNFSM4SRN4GEA
.

OK, thanks. I'll dig a little deeper. I don't know if the right solution is to fix it or delete it. I also don't understand why it isn't erroring out when CloudFormation runs.
Thanks for surfacing this. If I decide that the right thing to do is fix it, I'll just reopen this PR and accept it. Otherwise, I'll point it to the corresponding commit that does whatever seems right.


feat: added support for feeding custom annotations to the pods spun by scan jobs

closes #539

/kind feature

With this PR, starboard (in both cli and operator) will be able to annotate the pods, spun up by scanjobs, with user-provided custom annotations.
The custom-annotations which would be provided by the user must be in the format: key1=value1,key2=value2,key3=value3

When operating in CLI mode, starboard looks into the data.starboard.custom-annotations field of starboard configmap in starboard namespace to get know what are user-provided custom annotations.

When operating in Operator mode, starboard looks into the data.starboard-operator.custom-annotations field of starboard configmap in starboard namespace to get know what are user-provided custom annotations.

Note: As of now, this PR does not contain a dedicated functionality/sub-command around allowing the user input the custom-annotations in the first place. Hence, to make use of this functionality, as of now, the user would have to manually patch the starboard configmap with data.starboard.custom-annotations / data.starboard-operator.custom-annotations fields accordingly.

We also prefer smaller / incremental PRs so we can remove / merge faster. For example, deal with one scanner type at a time. Otherwise changes impact all packages and in case of regression we may break everything

Sure :) From next time onwards I ll ensure to segregate the PR into multiple commits (each dedicated to one sub-functionality) instead of squashing all the commits in the first place itself by creating the PR.

Ohh I pushed by mistake haha. Still have to clean some bits. Meanwhile, I ll address your comments as well :)

Quick overview of the current functionality:
I fed this configmap:
apiVersion: v1
data:
  configAuditReports.scanner: Polaris
  kube-bench.imageRef: docker.io/aquasec/kube-bench:0.5.0
  polaris.imageRef: quay.io/fairwinds/polaris:3.2
  scanJob.annotations: foo=bar,Creedthoughts.Gov.Www/creedthoughts=creed.blog. ### the scan job annotations
  trivy.httpProxy: ""
  trivy.httpsProxy: ""
  trivy.imageRef: docker.io/aquasec/trivy:0.16.0
  trivy.mode: Standalone
  trivy.noProxy: ""
  trivy.severity: UNKNOWN,LOW,MEDIUM,HIGH,CRITICAL
  vulnerabilityReports.scanner: Trivy
kind: ConfigMap
metadata:
  name: starboard
  namespace: starboard-operator

And that's how a pod spun by vulnerability reports looked
apiVersion: v1
kind: Pod
metadata:
  annotations:
    foo: buddysup
    Creedthoughts.Gov.Www/creedthoughts: creed.blog
  creationTimestamp: "2021-05-20T21:11:30Z"
  generateName: scan-vulnerabilityreport-59c955884-
  labels:
    app.kubernetes.io/managed-by: starboard
    controller-uid: c034578e-77be-4351-b8e8-a8bf5a4c8ca3
    job-name: scan-vulnerabilityreport-59c955884
    pod-spec-hash: 694c58c54
    starboard.resource.kind: Pod
    starboard.resource.name: nginx
    starboard.resource.namespace: default
    vulnerabilityReport.scanner: "true"
  name: scan-vulnerabilityreport-59c955884-rxnxx
  namespace: starboard-operator
  ownerReferences:
  - apiVersion: batch/v1
    blockOwnerDeletion: true

Same thing happened for the pods spun by configauditreports, etc.
Same for CLI mode as well on execution starboard scan ...

Noice, looking at the green flags, I ll make the relevant changes to settings.md as well


Users are contributing

The readme says you don't contribute; please fix the users

Ah I noticed this too. I think there is a bug in the current model of the humans, infact this bug seems to have gone unnoticed for millions of years.

I disagree, while the README says "you don't" for contributing, the CONTRIBUTING.md says you can contribute as long as it doesn't involve any code, I do see this as a discrepancy in the repo, someone should open a PR


use OpenAI's gpt-3.5-turbo model, an Http request error 404

When I use OpenAI's gpt-3.5-turbo model, an 404 error will be reported.
To Reproduce
Steps to reproduce the behavior:

Go to 'Skills' Directory
When I selected the gpt-3.5-turbo AI ENDPOINTS,and then Click to run any Function.
Error: InvalidRequest: The request is not valid, HTTP status: 404.

The following is the log of two different model executions, one error and one normal work
Terminal Output

AI Provider: OpenAI
Model: gpt-3.5-turbo
Execute: WriterSkill.Acronym
Parameters:
input: Organization to protect the freedom and security of its member countries in North America and Europe.
Prompt:
Generate a suitable acronym pair for the concept. Creativity is encouraged, including obscure references.
The uppercase letters in the acronym expansion must agree with the letters of the acronym
Q: A technology for detecting moving objects, their distance and velocity using radio waves.
A: R.A.D.A.R: RAdio Detection And Ranging.
Q: A weapon that uses high voltage electricity to incapacitate the target
A. T.A.S.E.R:  Thomas A. Swift’s Electric Rifle
Q: Equipment that lets a diver breathe underwater
A: S.C.U.B.A: Self Contained Underwater Breathing Apparatus.
Q: Reminder not to complicated subject matter.
A. K.I.S.S: Keep It Simple Stupid
Q: A national organization for investment in space travel, rockets, space ships, space exploration
A. N.A.S.A: National Aeronautics Space Administration
Q: Agreement that governs trade among North American countries.
A: N.A.F.T.A: North American Free Trade Agreement.
Q: Organization to protect the freedom and security of its member countries in North America and Europe.
A: N.A.T.O: North Atlantic Treaty Organization.
Q:Organization to protect the freedom and security of its member countries in North America and Europe.
fail: Microsoft.SemanticKernel.IKernel[0]
Something went wrong while rendering the semantic function or while executing the text completion. Function: WriterSkill.Acronym. Error: InvalidRequest: The request is not valid, HTTP status: 404. Details: This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions?
Status: 404 (Not Found)
  Content:
  {

"error": {
"message": "This is a chat model and not supported in the v1/completions endpoint. Did you mean to use v1/chat/completions?",
"type": "invalid_request_error",
"param": "model",
"code": null
}
}
  Headers:
  Date: Thu, 11 May 2023 08:02:40 GMT
  Connection: keep-alive
  Access-Control-Allow-Origin: REDACTED
  openai-organization: REDACTED
  openai-processing-ms: REDACTED
  openai-version: REDACTED
  Strict-Transport-Security: REDACTED
  x-ratelimit-limit-requests: REDACTED
  x-ratelimit-limit-tokens: REDACTED
  x-ratelimit-remaining-requests: REDACTED
  x-ratelimit-remaining-tokens: REDACTED
  x-ratelimit-reset-requests: REDACTED
  x-ratelimit-reset-tokens: REDACTED
  X-Request-ID: REDACTED
  CF-Cache-Status: REDACTED
  Server: cloudflare
  CF-RAY: REDACTED
  Alt-Svc: REDACTED
  Content-Type: application/json
  Content-Length: 227

  Microsoft.SemanticKernel.AI.AIException: InvalidRequest: The request is not valid, HTTP status: 404
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.RunRequestAsync[T](Func`1 request)
     at Microsoft.SemanticKernel.Connectors.AI.OpenAI.AzureSdk.ClientBase.InternalCompleteTextAsync(String text, CompleteRequestSettings requestSettings, CancellationToken cancellationToken)
     at Microsoft.SemanticKernel.Orchestration.SKFunction.<>c__DisplayClass18_0.<<FromSemanticConfig>g__LocalFunc|0>d.MoveNext()

fail: Microsoft.SemanticKernel.IKernel[0]
Function call fail during pipeline step 0: WriterSkill.Acronym. Error: InvalidRequest: The request is not valid, HTTP status: 404
Result:
Organization to protect the freedom and security of its member countries in North America and Europe.
Tokens:
Input tokens: 265
Output tokens: 17
Total: 282
========== Function execution was finished. ==========
AI Provider: OpenAI
Model: text-davinci-003
Execute: WriterSkill.Acronym
Parameters:
input: Agreement that governs trade among North American countries.
Prompt:
Generate a suitable acronym pair for the concept. Creativity is encouraged, including obscure references.
The uppercase letters in the acronym expansion must agree with the letters of the acronym
Q: A technology for detecting moving objects, their distance and velocity using radio waves.
A: R.A.D.A.R: RAdio Detection And Ranging.
Q: A weapon that uses high voltage electricity to incapacitate the target
A. T.A.S.E.R:  Thomas A. Swift’s Electric Rifle
Q: Equipment that lets a diver breathe underwater
A: S.C.U.B.A: Self Contained Underwater Breathing Apparatus.
Q: Reminder not to complicated subject matter.
A. K.I.S.S: Keep It Simple Stupid
Q: A national organization for investment in space travel, rockets, space ships, space exploration
A. N.A.S.A: National Aeronautics Space Administration
Q: Agreement that governs trade among North American countries.
A: N.A.F.T.A: North American Free Trade Agreement.
Q: Organization to protect the freedom and security of its member countries in North America and Europe.
A: N.A.T.O: North Atlantic Treaty Organization.
Q:Agreement that governs trade among North American countries.
Result:
A: C.A.F.T.A: Central American Free Trade Agreement.
Tokens:
Input tokens: 306
Output tokens: 19
Total: 325
========== Function execution was finished. ==========

@joisonwk , thanks for using the VS Code Extension to do prompt engineering.  Please make sure you update to the latest version of the extension.  Turbo chat completions are not supported currently for this. But we are working on it.

you use wrong model
Details: This is a chat model and not supported in the v1/completions endpoint.

you should use  text-davinci-003 for TextCompletionService
#650

@joisonwk were you able to get unblocked?

I tried this locally, no longer repros. Please re-open if still having issues.


Publish Static Documentation to Pages

This pull request publishes source documentation to GitHub Pages using Swift DocC.

Some paths in the docs directory make the project incompatible with Windows. Docs will live on this branch and a good system for updating them will be needed for future changes.


[SP-5616] -Backport of PDI-18757-

Removed empty debug System.out.println()
This commit is a cherry-pick from master commit 13e8a4804e

:white_check_mark: Build finished in 17m 34s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1096, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Add resx for string resources to MS.ANC.Razor.ProjectEngineHost

This PR just adds an empty resx file to the new ProjectEngineHost project in a similar fashion as other tooling projects.

You're up late

It was either get up and code or lay in bed with ideas. :smile:


I'd like to support bitbirds in a project I'm launching

Reach out if interested<EMAIL_ADDRESS>Thanks

Hello @charlesmarino ,
i used this creative repo and develop an script where any one can generate a data set for bitbirds and can use this dataset for Machine Learning, Supervised learning Etc.  User need to input the no. of rows that they need in the datasets. The dataset will be in csv format files.
i am generating a pull request if you find it interesting please accept it.
Regards,
Aagam jain
<EMAIL_ADDRESS>

Please accept my merge request if you find it interesting as soon as posible. as it is a part of important assignment as well.
Thanks


NetworkSDK download multi file from multi url

Now, NetworkSDK download request only can download single file, Next version will add download multi file and upload multi files.
This is an milestone

Multi download request can be send by NetworkMultiRequest

自己玩的挺嗨啊 -。-

自己写库嘛，写个里程碑约束一下自己。


[BUG] Early termination caused by spdlog::debug

Reported by @emekBaris
spdlog::debug commands in disable functions or destructors (functions that are called after ctrl + c) doesn't allow the lines after spdlog::debug to be executed before program terminates.
If I comment out spdlog:debug, destructors and disable functions work as intended. Else, lines after spdlog:debug doesn't run.
This could be due to lifetime of global spdlog.

Fixed here #30.


How to upgrade from V1 to current (V5)?

We seem to have version 1 installed and want to upgrade to the current version. The migration guides only tell to got one step at the time (1->2, 2->3 etc). Do we really have to do all those steps (and keep bothering our site admin with grant requests) or could this be done somehow in a single step?

Hi @mplwork,
Yes, there is a way that can be done i.e., you have to go for a fresh deployment of specific version. If data loss is major concern, then migration is a better path as it follows incremental approach (1->2,2->3....).

Hi @mplwork,
You have to delete previous resource group and create a new resource group for fresh deployment.

Hi @mplwork,
We are closing this request as we have provided the inputs for the query raised. If you face any issue with company communicator please raise a new issue for further assistance.


Smart Select re-init page history -1, when option select and  close

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: 1.6.4

Platform and Target:  CORDOVA

What you did
I have page_1  wich calls page_2 via a simple   href=''
page_2 has an smartselect like this:

Datos de la cabecera

		<form class="list-block inputs-list">

			            <select name="cmb_comprobante_tipo2" id="cmb_comprobante_tipo">
                                             <option value='1'> val 1</option>
                                             <option value='2'> val 2</option>
                                             <option value='3'> val 3</option>
                                             <option value='4'> val 4</option>
			            </select>

receipt

Tipo de comprobante

FACTURA C

MyApp is defined like this, so when you select in any option, it close the smartselect
var myApp = new Framework7({
    pushState: false,
    swipePanel: 'left',
    material: true,
    materialPageLoadDelay:0,
    materialRipple:true,
    materialPreloaderHtml:true,
    precompileTemplates: true,
    cache:true,
    template7Pages: true, // enable Template7 rendering for Ajax and Dynamic pages
    fastClicks:true,
    dynamicPageUrl: 'content-{{name}}',
    smartSelectBackOnSelect: true,

    // ... other parameters
});

BUT, when I click in an option and close the smartselect page, it refresh me page_1 executing the onPageInit method.
I don't know if it's the expected behavior but for me is not. ¿why re-init me the first page?

Expected

ok, it's rare that behavior. Page 1 has nothing to see with that smart-select.
If I need to capture the change of smart-select in page2 to assign that value and work with the result, when the back refresh me page1, it erase the content.
In my case I couldn't use it, and I had to change it to a normal select. Maybe someone happens to.
Could be optional this refresh.

This is unexpected

In all seriousness this is not a bug, it's a feature

chit
<EMAIL_ADDRESS>เมื่อ 17 มิ.ย. 2017 03:35 "VOUSYS"<EMAIL_ADDRESS>เขียนว่า

This is a (multiple allowed):

bug
enhancement
feature-discussion (RFC)
Framework7 Version: 1.6.4
Platform and Target: CORDOVA
What you did
I have page_1 wich calls page_2 via a simple
page_2 has an smartselect like this:

Datos de la cabecera

 val 1
 val 2
 val 3
 val 4

receipt

Tipo de comprobante
FACTURA C

MyApp is defined like this, so when you select in any option, it close the
smartselect
var myApp = new Framework7({
pushState: false,
swipePanel: 'left',
material: true,
materialPageLoadDelay:0,
materialRipple:true,
materialPreloaderHtml:true,
precompileTemplates: true,
cache:true,
template7Pages: true, // enable Template7 rendering for Ajax and Dynamic pages
fastClicks:true,
dynamicPageUrl: 'content-{{name}}',
smartSelectBackOnSelect: true,
// ... other parameters

});
BUT, when I click in an option and close the smartselect page, it refresh
me page_1 executing the onPageInit method.
I don't know if it's the expected behavior but for me is not. ¿why re-init
me the first page?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nolimits4web/Framework7/issues/1645, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AT6XhQrg02ls65_m5YaevOQajotiX_xJks5sEub3gaJpZM4N86Dx
.


Just in time decoding

Should give us a perf boost, and lays the foundation for configurable valueEncoding (per namespace fx)

/cc @andrewosh @Frando

3.4.1


RegistryNormalizer and SystemInformationNormalizer needs a check for null values.

In RegistryNormalizer:44, the value of $data->{'Mirrors'} can be null, making the following code throw an error.
foreach ($data->{'Mirrors'} as $value) {
     $values[] = $value;
}

Same goes for SystemInformationNormalizer:86 where the value of $data->{'SystemStatus'} can be null
foreach ($data->{'SystemStatus'} as $value_2) {
    $values_3 = [];
    foreach ($value_2 as $value_3) {
        $values_3[] = $value_3;
    }
    $values_2[] = $values_3;
}

A simple check solves the issue:
if (property_exists($data, 'SystemStatus') && ! is_null($data->{'SystemStatus'})) {

So, I figured out that docker-swagger.json needs to be modified and the type needs to be changed to "type": ["array", "null"],.
Same goes for Labels and Env in ContainerConfigNormalizer.php by the way.

Thanks to @Synpase791 it should be good now, version 1.22.2 and 1.21.3 has been tagged with the fix and other small changes

I will close this, don't hesite to reopen it if you still have this issue

Seen it! Thanks @joelwurtz and @Synapse791 !

No problem!
@joelwurtz Awesome package =]


Observer Arrow Bug

The triangular arrow on the bottom face of observers is facing the opposite direction that it should be. The pointed end of the triangle should face the direction of the signal output. The arrow on the top of the observers is correct.

Fixed in structura 1.3


dsc2 doesn't like R NA objects

Files
dsc_na_error.zip
Code
dsc_na_error$ dsc exec settings.dsc
INFO: Constructing DSC from settings.dsc ...
INFO: DSC exported to settings.html
INFO: Building output database dsc_result.rds ...
INFO: Elapsed time 0.384 seconds.
ERROR: 'list' object has no attribute 'shape'
ERROR: 'list' object has no attribute 'shape'

Ideally, foo can either be a numeric or NA. See settings2.dsc.

Converted to NaN and saved to the final output. More details see comment at #25.


Envoygateway wasm controller

What
Controller to configure kuadrant wasm-shim when EnvoyGateway is the gateway provider.
It enables kuadrant rate limiting
Verification Steps
①  Setup (Persona: Cluster admin)
make local-setup GATEWAYAPI_PROVIDER=envoygateway

Request an instance of Kuadrant in the kuadrant-system namespace:
kubectl -n kuadrant-system apply -f - <<EOF
apiVersion: kuadrant.io/v1beta1
kind: Kuadrant
metadata:
  name: kuadrant
spec: {}
EOF

② Deploy the Toy Store sample application (Persona: App developer)
kubectl apply -f examples/toystore/toystore.yaml

kubectl apply -f - <<EOF
apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
  name: toystore
spec:
  parentRefs:
  - name: eg
    namespace: envoy-gateway-system
  hostnames:
  - api.example.com
  rules:
  - matches:
    - method: GET
      path:
        type: PathPrefix
        value: "/toys"
    backendRefs:
    - name: toystore
      port: 80
EOF

Wait for the deployment:
kubectl wait --timeout=5m deployment/toystore --for=condition=Available

Export the gateway hostname and port:
export INGRESS_HOST=$(kubectl get gtw eg -n envoy-gateway-system -o jsonpath='{.status.addresses[0].value}')
export INGRESS_PORT=$(kubectl get gtw eg -n envoy-gateway-system -o jsonpath='{.spec.listeners[?(@.name=="http")].port}')

Test the route with traffic generator of 1 request per second.
while :; do curl --write-out '%{http_code}\n' --silent --output /dev/null --resolve api.example.com:$INGRESS_PORT:$INGRESS_HOST "http://api.example.com:$INGRESS_PORT/toys" -i | grep -E --color "\b(429)\b|$"; sleep 1; done

All responses should be 200 OK.
③ Protect the Toy Store application (Persona: App developer)
In other terminal, create the RateLimitPolicy for all the traffic defined for the HTTPRoute, max 5 request every 10 seconds.
kubectl apply -f - <<EOF
apiVersion: kuadrant.io/v1beta2
kind: RateLimitPolicy
metadata:
  name: toystore
spec:
  targetRef:
    group: gateway.networking.k8s.io
    kind: HTTPRoute
    name: toystore
  limits:
    basic:
      rates:
      - limit: 5
        duration: 10
        unit: second
EOF

Check the response of the traffic being run in the previous terminal. It should now be rate limited to 5 request every 10 seconds.
④ Verification of kuadrant managed EnvoyGateway resources
EnvoyExtensionPolicy
Check EnvoyExtensionPolicy resource defined by kuadrant
kubectl get envoyextensionpolicy -n envoy-gateway-system kuadrant-wasm-for-eg -o yaml | yq e -P

apiVersion: gateway.envoyproxy.io/v1alpha1
kind: EnvoyExtensionPolicy
metadata:
  annotations:
    kuadrant.io/namespace: kuadrant-system
  creationTimestamp: "2024-09-09T14:32:01Z"
  generation: 1
  name: kuadrant-wasm-for-eg
  namespace: envoy-gateway-system
  ownerReferences:
    - apiVersion: gateway.networking.k8s.io/v1
      blockOwnerDeletion: true
      controller: true
      kind: Gateway
      name: eg
      uid: e361aaf2-83e8-4cb6-b9e7-eba5b8870744
  resourceVersion: "3274"
  uid: e1f2a276-bf6f-4d7b-a759-8ddccd1f872d
spec:
  targetRefs:
    - group: gateway.networking.k8s.io
      kind: Gateway
      name: eg
  wasm:
    - code:
        image:
          url: oci://quay.io/kuadrant/wasm-shim:latest
        type: Image
      config:
        failureMode: deny
        rateLimitPolicies:
          - domain: default/toystore
            hostnames:
              - api.example.com
            name: default/toystore
            rules:
              - conditions:
                  - allOf:
                      - operator: startswith
                        selector: request.url_path
                        value: /toys
                      - operator: eq
                        selector: request.method
                        value: GET
                data:
                  - static:
                      key: limit.basic__48de662e
                      value: "1"
            service: kuadrant-rate-limiting-service
      failOpen: false
      name: kuadrant-wasm-shim
      rootID: kuadrant_wasm_shim
status:
  ancestors:
    - ancestorRef:
        group: gateway.networking.k8s.io
        kind: Gateway
        name: eg
        namespace: envoy-gateway-system
      conditions:
        - lastTransitionTime: "2024-09-09T14:32:03Z"
          message: Policy has been accepted.
          reason: Accepted
          status: "True"
          type: Accepted
      controllerName: gateway.envoyproxy.io/gatewayclass-controller

Few things to highlight:

The envoyextensionpolicy targets the gateway.
Wasm filter defined with a reference to the kuadrant's wasm-shim OCI image oci://quay.io/kuadrant/wasm-shim:latest.
The wasm configuration is exactly the same as the one defined for Istio's wasmplugin resource.
EnvoyGateway fills the status, so kuadrant controller can test status for the rate limit policy enforce's condition (missing, to be done in other PR).

EnvoyPatchPolicy
Check EnvoyPatchPolicy resource defined by kuadrant
The EnvoyPatchPolicy is used to tell Envoy the address of limitador, which is being used by the wasm module to open a grpc connection. Basically, it needs to tell: Envoy cluster kuadrant-rate-limiting-service is the address limitador-limitador.kuadrant-system.svc.cluster.local:8081
kubectl get envoypatchpolicy -n envoy-gateway-system kuadrant-patch-for-kuadrant-wasm-for-eg -o yaml | yq e -P

apiVersion: gateway.envoyproxy.io/v1alpha1
kind: EnvoyPatchPolicy
metadata:
  creationTimestamp: "2024-09-09T14:32:01Z"
  generation: 1
  name: kuadrant-patch-for-kuadrant-wasm-for-eg
  namespace: envoy-gateway-system
  ownerReferences:
    - apiVersion: gateway.envoyproxy.io/v1alpha1
      blockOwnerDeletion: true
      controller: true
      kind: EnvoyExtensionPolicy
      name: kuadrant-wasm-for-eg
      uid: e1f2a276-bf6f-4d7b-a759-8ddccd1f872d
  resourceVersion: "3277"
  uid: cf7d60fb-fb44-4b5e-b891-5257fda80038
spec:
  jsonPatches:
    - name: kuadrant-rate-limiting-service
      operation:
        op: add
        path: ""
        value:
          connect_timeout: 1s
          http2_protocol_options: {}
          lb_policy: ROUND_ROBIN
          load_assignment:
            cluster_name: kuadrant-rate-limiting-service
            endpoints:
              - lb_endpoints:
                  - endpoint:
                      address:
                        socket_address:
                          address: limitador-limitador.kuadrant-system.svc.cluster.local
                          port_value: 8081
          name: kuadrant-rate-limiting-service
          type: STRICT_DNS
      type: type.googleapis.com/envoy.config.cluster.v3.Cluster
  targetRef:
    group: gateway.networking.k8s.io
    kind: Gateway
    name: eg
  type: JSONPatch
status:
  ancestors:
    - ancestorRef:
        group: gateway.networking.k8s.io
        kind: Gateway
        name: eg
        namespace: envoy-gateway-system
      conditions:
        - lastTransitionTime: "2024-09-09T14:32:03Z"
          message: Policy has been accepted.
          reason: Accepted
          status: "True"
          type: Accepted
        - lastTransitionTime: "2024-09-09T14:32:03Z"
          message: Patches have been successfully applied.
          reason: Programmed
          status: "True"
          type: Programmed
      controllerName: gateway.envoyproxy.io/gatewayclass-controller

@adam-cattermole ready for review


Kusama to main

closing this for now with no merge. we eventually need to send such a pull request to sync both branches, but probably after the Kusama theme work is done and Kusama is stable.


Update does not consider unique fields in relations

This simple test is failing due to the fact that the account Model only have id as unique. However ownerUser email field is unique. Should not that field be allowed in where clause in the update?
test('account update', async () => {
    await prisma.account.update({
      where: { ownerUser: { email<EMAIL_ADDRESS>}},
      data: { auth_id: "xxx", auth_token: "xxx" }
    })
  });

model Account {
  id String @default(uuid()) @id
  dateCreated DateTime @default(now())
  dateUpdated DateTime @updatedAt

  auth_id String? @default("")
  auth_token String? @default("")
  ownerUser User @relation(fields: [ownerUserId], references: [id])
  ownerUserId String
}
model User {
  id String @default(uuid()) @id
  dateCreated DateTime @default(now())
  dateUpdated DateTime @updatedAt

  email String @unique
}

Environment
npx prisma --version
Environment variables loaded from ./prisma/.env
@prisma/cli          : 2.10.1
@prisma/client       : 2.10.2
Current platform     : darwin
Query Engine         : query-engine 7d0087eadc7265e12d4b8d8c3516b02c4c965111 (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli 7d0087eadc7265e12d4b8d8c3516b02c4c965111 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core 7d0087eadc7265e12d4b8d8c3516b02c4c965111 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt 7d0087eadc7265e12d4b8d8c3516b02c4c965111 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Studio               : 0.304.0

You will need to updateMany if you want update via the relation. Since you haven't specified the opposing side in the schema, it is a one to many relation and so two users can have same account.
See updateMany: https://www.prisma.io/docs/concepts/components/prisma-client/crud#updatemany

@pantharshit00 effectively many users can have the same account. I discovered that I was able to do a updateMany but I still don't fully understand why I can't do update since the email is @unique
To me the semantics of what I have written would be something like:
update the account that have an user with email
the other sounds to me like
update all the accounts that have an user with email

@pantharshit00

is a one to many relation and so two users can have same account.

But Im updating just only one account?

@DavidGOrtega This is still not possible as multiple Account rows in your database can point to same user row as per your schema ( one to many relation)
Example records:
Account table
id   |  ownerUserId
1    |  1
2    |  1

User table
id   | email
1    |<EMAIL_ADDRESS>2    |<EMAIL_ADDRESS>
Now definition of update in sense of Prisma is update one row of the database. Now if you seek account rows by<EMAIL_ADDRESS>we will get 2 account rows and we can't choose which one we should update. So that is why you need to use updateMany
If you want to allow this make this relationship a one to one relation by marking the ownerUserId foreign key as @unique in the schema.

Closing due to inactivity


Adding icon on login page

There is no icon in the Login Page

@thestarsahil Please assign me this issue under Gssoc'24.


TypingDNA API deployment failure on Apache Tomcat

Description
api#identity#typingdna#v_.war fails to start on Apache Tomcat 8 with the following logs,

0-Aug-2022 10:24:29.659 INFO [http-nio-8080-exec-1] org.apache.jasper.servlet.TldScanner.scanJars At least one JAR was scanned for TLDs yet contained no TLDs. Enable debug logging for this logger for a complete list of JARs that were scanned but no TLDs were found in them. Skipping unneeded JARs during scanning can improve startup time and JSP compilation time.
10-Aug-2022 10:24:29.660 SEVERE [http-nio-8080-exec-1] org.apache.catalina.core.StandardContext.startInternal One or more listeners failed to start. Full details will be found in the appropriate container log file
10-Aug-2022 10:24:29.661 SEVERE [http-nio-8080-exec-1] org.apache.catalina.core.StandardContext.startInternal Context [/api/identity/typingdna/v1.0] startup failed due to previous errors

How to reproduce:

Download api#identity#typingdna#v_.war from WSO2 store.
Copy the file into webapps directory of Tomcat or use the manager interface to deploy the war file.

Expected behavior:

To deploy the war file successfully and start the application.

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS-6.0.0-rc1
Tomcat: 8.5.81
OS: MacOS
Database: PostreSQL

Issue caused due to misconfiguration. Closing.


Add maxRequestSize env var to merlin-server to set Javalin HTTP config

Users are running into a case where they need to upload data that is larger than the accepted current default size for Javalin. This ticket is to expose an environment variable to merlin-server so users can set the Javalin HttpConfig maxRequestSize based on their use case. We should make the default larger than 1 Mb. We should test a large payload (> 10 Mb) works after this update.
For example when using the addExternalDataset action, Hasura makes a POST request to the merlin-server /addExternalDataset endpoint. The default value for the maxRequestSize config is 1 Mb and external datasets will often be much larger than 1 Mb. Payloads larger than 1 Mb returns a 413 Payload Too Large.
Additionally we should investigate if there are env vars on the Hasura side to support larger payloads. What even is the upper limit of payload size for Hasura actions? I can't find this documented anywhere.
More questions to think about when working on this ticket are:

Can we support streaming very large payloads into a Hasura action and  into merlin-server?
If we cannot support streaming, can we support submitting data in batches? This would involve either updating the addExternalDataset action to accept batches, or adding an updateExternalDataset action to include batch semantics.

See the internal Slack thread here.

As a thought, would it be difficult to set up a sidecar API for managing file uploads? I think this alternative was raised quite some time ago (over a year ago, certainly), but it didn't seem to be necessary at the time. I remember reading articles on this back then, like this.
The proposed scheme was something like:

Create a small service that manages uploads (creation, deletion, download) on the filesystem.
Add an uploads table to Postgres with id and path columns (at minimum).
Change Hasura queries that reference paths directly to reference uploads IDs instead.

@Twisol This is not for file uploads actually. The gateway still handles file uploads using the scheme you describe. The addExternalDataset action does not accept files, but instead just a (potentially) very large JSON blob which then writes to the plan_dataset.sql table (among others).

@Twisol Yeah it's definitely a possibility! In fact I think users would love being able to just add a raw CSV as an external dataset since that's the main format they are using. It's a little out of scope for this ticket tho since it would require an API change first (refactoring addExternalDataset). It would also slightly complicate the API a bit since two requests would be required to create an external dataset instead of one. I'll bring this up when we bring this ticket into a sprint tho and we can decide to make an additional ticket then.

For sure :+1: This issue definitely seems to be focusing on the proximal pain point, I get that. Just trying to think about what an ideal fix might be, aside from any immediate patch 😃

Yeah I appreciate the additional perspective!


JSON deserializing doesn't work with other charsets

Hi,
I'm trying to write a front-end for a website which delivers json with Content-Type: application/json; charset=windows-1252, but reqwest::Response::json() fails with Err(Serialize(Syntax(InvalidUnicodeCodePoint, 34, 37))).
I guess it didn't do the conversion by itself and chokes on that charset.

This issue has intrigued me since it was opened. I hadn't even considered that JSON would be passed around as anything but Unicode until now.
RFC7159 has this to say in Section 8.1:
JSON text SHALL be encoded in UTF-8, UTF-16, or UTF-32.  The default
encoding is UTF-8, and JSON texts that are encoded in UTF-8 are
interoperable in the sense that they will be read successfully by the
maximum number of implementations; there are many implementations
that cannot successfully read texts in other encodings (such as
UTF-16 and UTF-32).

Section 11 also says:
 Note:  No "charset" parameter is defined for this registration.
 Adding one really has no effect on compliant recipients.

The older RFC4627 even suggested looking at the first 4 octets to see which Unicode encoding was in use rather than specifying it explicitly. Now they seem to just suggest using UTF-8 all the time because some implementations can't handle anything else and don't even try the other Unicode encodings.
What do other HTTP libraries generally do here? If other libraries support non-standard character sets on application/json, are those that do following some specific extension in order to be compatible with each other?

I don't know exactly. I just wanted to use reqwest on a site's API giving out json, and it unconditionally encodes it in windows-1252, no matter the amount of Accept or other headers I throw at it. The site is itself pretty mainstream (the french equivalent to craigslist) and their API is sort of private to their phone app, so there's no way I can make them fix it.
Anyway I can live without that and do the decoding manually, if it's just a bug on their side.

That JSON is returned in a format other than UTF-8 (or its wider cousins) is a spec violation, definitely.
It's possible, though very unlikely, that serde_json could be improved to detect odd encodings.

To fix it yourself, you could wrap the response in a streaming windows-1252-to-utf-8 converter, and pass that to serde_json. If such a streaming implementation doesn't exist, you could buffer the full thing in memory first.

No worries I can work through this. I just didn't realize the API was wrong spec-wise and just thought it was a hole in your code.
I think you can safely close this.


Qiskit 0.4.9 Docker build error

Building an alpine docker image of qiskit-0.4.9 results in a build error.
Expected Behavior
pip install 'qiskit==0.4.9' should succeed.
Current Behavior
The installation fails with a build error. See the attached logfile for the output log.
log.txt
Building pip install qiskit==0.4.8 does succeed. The docker image is located at https://github.com/eddieschoute/python3-qiskit
Possible Solution
Provide build requirements so that I can figure out what dependencies are missing.
Or output missing dependencies that cause the build to fail.
Steps to Reproduce (for bugs)

git clone https://github.com/eddieschoute/python3-qiskit.git
cd python3-qiskit
make sure you are on the right commit: git reset --hard e2f71f73e39fcfde8db43860ee305e95f1502c46
docker build .

Context
My CI build system relies on the docker image and I recently bumped my version to qiskit-0.4.9. The CI system is now broken.
Building locally on macOS 10.13 did succeed.
Your Environment

Version used: qiskit-0.4.9
Environment name and version (e.g. Python 3.6.1): python3.6.4
Operating System and version: Alpine Linux, docker image

Hi @eddieschoute,
Seems like your docker image must be missing some of the components needed to build the C++ Simulator.
Anyway, that shouldn't be a problem at all, you are actually hitting a bug that we have already fixed in the master branch, so it will be fixed for the next  release, that should happen soon.
If you can install using pip in verbose mode: pip install -vvv 'qiskit==0.4.9' and paste the traces here, I could see what's failing on your build and try to sort it out.

Hi @atilag , I have attached the verbose log. I don't see any trace of why the error happens but maybe you have more insight: log-verbose.txt

@eddieschoute I just noticed that Alpine build-base package doesn't include CMake. Since 0.4.9, CMake (>=3.5) is required to build non-python code. Try to install it on your docker image, and send me the verbose log again if it fails.

Adding CMake to the docker packages has resolved the issue.


Adding continuous integrations with Travis CI

As discussed in issue #15, this PR adds the .travis.yml setup as well as simplifies the mechanism for running tests.
The result of the Travis build is avalable at https://travis-ci.org/PyAbel/PyAbel
Unit tests can be run either from inside the module with,
nose -s abel/tests/

or if the module is installed, from any location with,
python -c 'import abel.tests; abel.tests.run_cli(coverage=True)'

This latter part is used in Travis CI, and also show tests coverage of the code base (to use it locally one might have to install pip install coveralls).
For now, only the test for generating the basis set is run.
P.S: sorry this PR is a bit messy as there are parts related to other issues: there were just to many simultaneous changes in the repo today ))

It passed the tests! Looking good! :+1:


react-native-image-cropper alternatives

Well, as some of you have noticed this module appears to be abandoned and currently has not had many updates. At the moment neither TypeScript support is offered. And as said in https://github.com/st0ffern/react-native-image-cropper/issues/71 there is a lack of maintainers to take care of this project yet. The idea of this type of module for react-native is very good, as some types of applications need to crop images in a customizable way. So while this module is no longer active or maintained by someone with time to dedicate it, I felt the need to look for some alternatives and perhaps to know other packages with similar functionality.
Some alternatives:

react-native-simple-image-cropper
react-native-image-crop-picker

There is also https://github.com/DuDigital/react-native-zoomable-view but you need to implement the crop function manually.


Import meta graph followed by save overwrites the previous checkpoints

NOTE: Only file GitHub issues for bugs and feature requests.  All other topics will be closed.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Environment info
Operating System: Ubuntu 14.04
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*): using CPU version of tensorflow 1.0.0
If installed from binary pip package, provide:

A link to the pip package you installed: pip install tensorflow
The output from python -c "import tensorflow; print(tensorflow.__version__)". 1.0.0

If installed from source, provide

The commit hash (git rev-parse HEAD)
The output of bazel version

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
import tensorflow as tf

graph = tf.Graph()
with graph.as_default():
    initializer = tf.random_uniform_initializer(minval=-0.5, maxval=0.5, seed=42, dtype=tf.float32)
    var1 = tf.get_variable('var1', shape=(1,), dtype=tf.float32, initializer=initializer)
    saver = tf.train.Saver(tf.trainable_variables(), max_to_keep=20)
    init_op = tf.global_variables_initializer()
    graph.finalize()

with tf.Session(graph=graph) as sess:
    sess.run(init_op)
    saver.save(sess, 'sample_graph', global_step=0)

graph = tf.Graph()
with tf.Session(graph=graph) as sess:
    saver = tf.train.import_meta_graph('sample_graph-0.meta')
    saver.restore(sess, './sample_graph-0')
    saver.save(sess, 'sample_graph', global_step=1)
    print(saver.last_checkpoints) # lists only ['sample_graph-1'] does not preserve the previous checkpoint sample_graph-0

Essentially I am checkpointing a graph and then importing it. On trying to save the next checkpoint, the saver overwrites the previous checkpoint in the checkpoint file (the actual meta, index and data files are not overwritten) and only the last saved checkpoint is present in the checkpoint file. Is this the intended behavior? Is there any way to preserve the checkpoints across multiple saves of the graph.
What other attempted solutions have you tried?
Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).

Thanks for a clear issue report.

It has been 14 days with no activity and this issue has an assignee.Please update the label and/or status accordingly.

Nagging Assignee: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 107 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 139 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 154 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 169 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 184 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 199 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 214 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 229 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 243 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 258 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @sherrym: It has been 273 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Is this still an issue ?
@sherrym - Could you please look into this ?

Reassigning to Allen, Sherry has left the team.
I think what is happening is that the list of checkpoints is reset by import_meta_graph. The old checkpoints still exist on disk, but the index structure is overwritten by import_meta_graph.
We won't do any work on this before 2.0, and I'm concerned that we won't be able to really fix this since it would be a behavior change (and it's not clear that this is a bug -- import_meta_graph often does invalidate old checkpoints since you may add new variables).
I will close this issue. This should be much less of a problem in 2.0.

There is a write_state Boolean argument to Saver.save, so you can prevent it from updating the Checkpoint file. And there is also Saver.set_last_checkpoints_with_time, which you can use along with tf.train.get_checkpoint_mtimes so that the Saver writes a correct-ish Checkpoint file instead of just not overwriting.
I do think the 2.x story will be better here. Still working out the details, but there should be a design review soon for importing SavedModels. I'll include an import-and-train workflow there.


always need to rebuild chainstate after restarting omnicored

I am running a omnicore in a docker file, i was doing a backup over the blockchain data, and stopped the omnicored via omnicore-cli stop method (which is docker exec -it omnicore /omnicore/bin/omnicore-cli stop, executed from host machine.)
After stopping, here's my debug log,
2019-01-10 09:18:36 tor: Thread interrupt
2019-01-10 09:18:36 torcontrol thread exit
2019-01-10 09:18:36 addcon thread interrupt
2019-01-10 09:18:36 scheduler thread interrupt
2019-01-10 09:18:36 msghand thread interrupt
2019-01-10 09:18:36 net thread interrupt
2019-01-10 09:18:40 opencon thread interrupt
2019-01-10 09:18:40 Shutdown: In progress...
2019-01-10 09:18:40 StopNode()
2019-01-10 09:19:15 Shutdown: done

When it restarted, here's the log,
2019-01-10 09:28:51 Omni Core version v0.3.1
2019-01-10 09:28:51 InitParameterInteraction: parameter interaction: -whitelistforcerelay=1 -> setting -whitelistrelay=1
2019-01-10 09:28:51 Default data directory /root/.bitcoin
2019-01-10 09:28:51 Using data directory /omnicore/data
2019-01-10 09:28:51 Using config file /omnicore/data/bitcoin.conf
2019-01-10 09:28:51 Using at most 125 connections (1048576 file descriptors available)
2019-01-10 09:28:51 Using 16 threads for script verification
2019-01-10 09:28:51 scheduler thread start
2019-01-10 09:28:51 libevent: getaddrinfo: address family for nodename not supported
2019-01-10 09:28:51 Binding RPC on address :: port 8332 failed.
2019-01-10 09:28:51 HTTP: creating work queue of depth 16
2019-01-10 09:28:51 Config options rpcuser and rpcpassword will soon be deprecated. Locally-run instances may remove rpcuser to use cookie-based auth, or may be replaced with rpcauth. Please see share/rpcuser for rpcauth auth generation.
2019-01-10 09:28:51 HTTP: starting 4 worker threads
2019-01-10 09:28:51 Using BerkeleyDB version Berkeley DB 4.8.30: (April  9, 2010)
2019-01-10 09:28:51 Using wallet wallet.dat
2019-01-10 09:28:51 init message: Verifying wallet...
2019-01-10 09:28:51 CDBEnv::Open: LogDir=/omnicore/data/database ErrorFile=/omnicore/data/db.log
2019-01-10 09:28:52 Bound to [::]:8333
2019-01-10 09:28:52 Bound to <IP_ADDRESS>:8333
2019-01-10 09:28:52 Cache configuration:
2019-01-10 09:28:52 * Using 1024.0MiB for block index database
2019-01-10 09:28:52 * Using 8.0MiB for chain state database
2019-01-10 09:28:52 * Using 15352.0MiB for in-memory UTXO set
2019-01-10 09:28:52 init message: Loading block index...
2019-01-10 09:28:52 Opening LevelDB in /omnicore/data/blocks/index
2019-01-10 09:29:01 Opened LevelDB successfully
2019-01-10 09:29:01 Using obfuscation key for /omnicore/data/blocks/index:<PHONE_NUMBER>000000
2019-01-10 09:29:01 Opening LevelDB in /omnicore/data/chainstate
2019-01-10 09:41:19 Opened LevelDB successfully
2019-01-10 09:41:19 Using obfuscation key for /omnicore/data/chainstate: c3af64913ad8eb65
2019-01-10 09:41:50 LoadBlockIndexDB: last block file = 1490
2019-01-10 09:41:50 LoadBlockIndexDB: last block file info: CBlockFileInfo(blocks=12, size=8374351, heights=557854...557865, time=2019-01-10...2019-01-10)
2019-01-10 09:41:50 Checking all blk files are present...
2019-01-10 09:41:50 LoadBlockIndexDB: transaction index enabled
2019-01-10 09:41:50 LoadBlockIndexDB: hashBestChain=0000000000000000001f1ab17d82075248eb9791dcb0e626c4245d5b81bf0ff9 height=557331 date=2019-01-06 18:18:26 progress=0.999361
2019-01-10 09:41:50 init message: Rewinding blocks...
2019-01-10 09:41:54 init message: Verifying blocks...
2019-01-10 09:41:54 Verifying last 6 blocks at level 3
2019-01-10 09:41:54 [0%]...[16%]...ERROR: ApplyTxInUndo: undo data overwriting existing transaction
2019-01-10 09:41:54 ERROR: ApplyTxInUndo: undo data overwriting existing transaction
2019-01-10 09:41:54 ERROR: ApplyTxInUndo: undo data overwriting existing transaction
......
2019-01-10 09:41:59 ERROR: ApplyTxInUndo: undo data overwriting existing transaction
2019-01-10 09:41:59 ERROR: VerifyDB(): *** coin database inconsistencies found (last 7 blocks, 0 good transactions before that)

2019-01-10 09:41:59 : Corrupted block database detected.
Please restart with -reindex or -reindex-chainstate to recover.
2019-01-10 09:41:59 Aborted block database rebuild. Exiting.
2019-01-10 09:41:59 Shutdown: In progress...
2019-01-10 09:41:59 scheduler thread interrupt
2019-01-10 09:41:59 StopNode()
2019-01-10 09:41:59 Shutdown: done

and here's my getinfo, after running with -reindex-chainstate
{
  "version": 130200,
  "protocolversion": 70015,
  "walletversion": 130000,
  "balance": 0.00000000,
  "blocks": 263752,
  "timeoffset": 0,
  "connections": 8,
  "proxy": "",
  "difficulty": 189281249.2810329,
  "testnet": false,
  "keypoololdest":<PHONE_NUMBER>,
  "keypoolsize": 100,
  "paytxfee": 0.00000000,
  "relayfee": 0.00001000,
  "errors": ""
}

I've been stuck at this step for over a week now, any help will be appreciated, thanks!!!

Hello, please upgrade to the newest version v0.4.0. If the issue still remains, please let me know.


Enable accessibility service to use a password manager

The possibility to enable accessibility seevice by demand (maybe), to use a password manager like password androis store

Thanks for the suggestion. We have had a number of requests for password manager integration and hope we can add this in the future.

@ismaelpuerto we have been checking this request, and I can use my password manager to fill in login forms.

We are closing this feature request. Create a new discussion if your request was about something different. thanks.


Upgrade to sphinx 8

I wanted to use linkcheck_anchors_ignore_for_url which is available sphinx>=0.7.1. I thought I just update directly to sphinx 8.

I am not sure what to do with this not working link, seems rather important, maybe their page has a hicks

I just stumbled on the CI failing for my latest blog-post PR #72. Weird, because this wasn't an issue for the other one (PR #69). So if there are no arguments against the updates here, I'd say LGTM. Any objections, @edan-bainglass, could you look into it?


Removed the executable bit on files that should not be executable

What has been done

All files are no longer executable, as per PermCheck, only this is not a PHP project ;-)

Acceptance criteria

Files that should not be executable are not executable

@eXistenZNL are you sure the app.js doesn't have to be executable in order to write the statuses.json file to the folder?

Yep! The file public/js/app.js is just a thing living in your browser, this should never by an executable file :+1:

Oh mistaked it for the server.js :+1:


Data Validation

Set up validators for data returned by importers
Referenced in #14

Basic things that will need to be checked for a dataframe:

DataFrame is not empty
Required Columns exist
Required columns do not contain only nulls


Add compose metrics build.gradle.kts

https://github.com/lhoyong/ALogViewApp/actions/runs/4697086463/jobs/8327782610


confidence score of predicted word

In the following line if the word has 2 characters, the confidence variable is returning 3 values. Can anyone please tell me what are there values
label, confidence = parseq.tokenizer.decode(pred)
What I understood from the paper is 2 values are character confidence, but what about the third value.

Hello. The additional value is for the [EOS] token.


Override SUBMENU_OPEN_DELAY

Is there a way to set SUBMENU_OPEN_DELAY? I'd like there not to be a delay at all.

Hi @R4VANG3R , currently the submenu delays are using pre-defined values. Will add two props that allow them to be configurable.

Nice that would be a big help, especially when using no animations.

Thank you! 🎉


Add SelectionListener/SelectedContentRange

Adds:

SelectionListener, allows a user to listen to selection changes under the subtree it wraps given their is an ancestor SelectionArea or SelectableRegion. These selection changes can be listened to through the SelectionListenerNotifier that is provided to a SelectionListener.
SelectionListenerNotifier, used with SelectionListener, allows a user listen to selection changes for the subtree of the SelectionListener it was provided to. Provides access to individual selection values through the SelectionDetails object selection.
SelectableRegionScope, allows the user to listen to when a parent SelectableRegion is changing or finalizing the selection.
SelectedContentRange, provides information about the selection under a SelectionHandler or Selectable through the getSelection() method. This includes a start and end offset relative to the Selectables content.

Original PR & Discussion: https://github.com/flutter/flutter/pull/148998
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

This is ready for another review, the main change is that a Selectable/SelectionHandler now only return a single SelectedContentRange instead of a list of ranges and other comments have been addressed.

Talked to @Renzo-Olivares offline, the current idea is to use a listen/notify API something like TextEditingController to listen to selection changes, Ideally this can also get rid of the SelectionFinalizedEvent if the controller can listen to the SelectableRegion's state changes above the current widget, and it sounds good to me. Since I will be out for another 2 weeks, this PR will LGTM if it can be implemented without SelectionfinalizedEvent and also LGT @justinmc and @LongCatIsLooong .

@Renzo-Olivares This will go into production (we have an estimated date), it is a very good feature.

Friendly bump for review.


#396: Add missing links to next chapters in testing docs - better way

Fixed #397.

More typo fixes.


Can't put objects with special characters in key name

For example, when the key contains a space:
await s3.putObject({
  Bucket: 'some-bucket',
  Key: 'some key',
  Body: 'some text'
}).promise();

The logs show:
OBJECT PUT               /s3/some-bucket/some key      REJECTED
01/23/2019 12:06:08                                    2 Milliseconds

The only non-alphanumeric characters which I've got to work are . - _
Using scireum/s3-ninja:4

See the specs here. Probably best to avoid characters which need special handling.
https://docs.aws.amazon.com/AmazonS3/latest/dev/UsingMetadata.html

:+1: some useful information there; although if this wants to emulate S3, users might expect it to behave in the same way as the real S3.

Feel free to contribute a fix (or an option) to change the behaviour. As we only used generated object IDs we have no benefit in investing in this request. Contributions are however always much appreciated.
I'll leave this open so that other users learn about the incompatibility.

A solution would be to not just replace special characters by "_" but by something like "__UTF-BYTES" like " " -> "__0020" or the like. This could then be replaced back when enumerating objects etc.
Feel free to implement this and provide a PR - and also feel free to ask for help.


Bug Report: Option not working?

Environment
Windows 10 Pro 18956 Insider Preview
Steps to reproduce
The option "closeOnExit" will be invalid with only a tab running, and it could be reproduce with multi-tabs running occationally.
Expected behavior
Fix it
Actual behavior
There is an option named "closeOnExit", but it seems to be invalid after inputting exit command, and the windows would not be closed.
Following content is a part of my configuration file about powershell, it might help you to resolve it.
{
            "startingDirectory": "%USERPROFILE%",
            "guid": "{95e05ea2-6bad-4896-bcdf-a4f96c885cc8}",
            "name": "PowerShell",
            "background": "#002731",
            "colorscheme": "Campbell",
            "historySize": 9001,
            "snapOnInput": true,
            "cursorColor": "#FFFFFF",
            "cursorShape": "bar",
            "commandline": "powershell.exe",
            "fontFace": "Consolas",
            "fontSize": 12,
            "acrylicOpacity": 0.5,
            "useAcrylic": true,
            "closeOnExit": true,
            "padding": "0, 0, 0, 0",
            "icon": "ms-appdata:///roaming/icons/powershell_32px.png"
        }

Okay I'm going to need an actual walkthrough of repro steps, and expected/actual behavior. Right now it's really hard to understand what you're doing and what's wrong vs what's expected. For example:
Repro steps

set closeOnExit to true for the "powershell" profile
Open a powershell tab
type exit

Expected Behavior
The tab closes
Actual Behavior
The entire UI hangs.
(if what you're trying to report is exactly what I've described above, then this is a dupe of #2185.)

Okay, you are right, I am confused by this problem
When will you fix it?

/dupe #2185


Callback after successfully Shared

I want to get callback if the page has been successfully been shared or not. I have used before and after but it only functions when the sharing windows opens, not when exactly the contents has been shared.
 return FB.ui({
        method: 'feed',
        name: this.config.networks.facebook.title,
        link: this.config.networks.facebook.url,
        picture: this.config.networks.facebook.image,
        caption: this.config.networks.facebook.caption,
        description: this.config.networks.facebook.description
      },
      function(response) {
      if (response && response.post_id) {
      console.log('Post was published.');
      } else {
      console.log('Post was not published.');
      }
}

Managed to get callback for facebook, but unable for twitter and others

Yeah this depends on whether each individual network provides a callback. As we add more networks, this will become difficult to make consistent as a feature. If you could look into the support other networks have for a post-share callback and post it here, that would help a ton!

As far as I know Facebook, Twitter and Linkedin provides callback if you use their JS APIs. As you have use FB feed API so it was easy for me to put a callback function for facebook but for others you have use their share url. If you use their api instead of Share url it would become easier to putt callback functions for them also.

From this Article http://projs.hackhat.com/how-to-make-a-custom-share-tweet-button-and-know-when-the-user-clicked/
I used this as
var tweetUrlBuilder = function(o){
    return [
        'https://twitter.com/intent/tweet?tw_p=tweetbutton',
        '&url=', encodeURI(o.url),
        '&via=', o.via,
        '&text=', o.text
    ].join('');
};

$('.entypo-twitter').on('click', function(){
    var url = tweetUrlBuilder({
        url : 'http://bit.ly/OChT65',
        via : 'ProJavaScript',
        text: 'How to make a custom share tweet button and know when the user tweeted'
    });
    window.open(url, 'Tweet', 'height=500,width=700');
})

// Called by the Twitter, many times during it's
// life-cycle. We only want to know when the user
// has tweeted, so the following filters are required.
var callback = function(e){
    if(e && e.data){
        var data;
        try{
            data = JSON.parse(e.data);
        }catch(e){
            // Don't care.
        }
        if(data && data.params && data.params.indexOf('tweet') > -1){
            alert('Thanks for the tweet!');
        }
    }
};

window.addEventListener ? window.addEventListener("message", callback, !1) : window.attachEvent("onmessage", callback)

Now, need to call this function in your Share.prototype.network_twitter

Hey, @TaqiOfficial - we've been looking into this. If you find anymore like the twitter one, let us know! Otherwise in v1.0.0 we're going to have the option to load a platforms JS lib for this and other functionality

I am also working on it, As I have no prior exp. contributing to git repo via pull, merge etc. so i will share my code here, else I will contribute to this repo.

Hi, I tried this, it does not work, I also tried the original stuff on JSFIlldel, not either

HI @taqiofficial  the above script does not work

any update on this? I wanted to get a successful callback , so I can offer a user discount code

how we can get response after sharing content from Facebook in asp.net....:)


Review utils

See which ones we still need or which can be cleaned up.

[ ] Can we remove combineSpecWithDimension? I'm not clear what purpose it serves.
[ ] updateMultipleDatasetsInView only appears in Vega but not Vega-Lite. That seems suspicious as Vega-Lite compiles to  Vega.
[ ] updateSingleDatasetInView can probably be inlined since it's only used in a single method

I would combine all utils functions in a single file. There is no need to a file per perfection.

Regarding updateMultipleDatasetsInView, our VegaLite component simply sets the mode to vega-lite, and then renders a Vega component. Thus, this function gets called when something updates for both Vega and Vega-Lite.

Regarding combineSpecWithDimension, for vega-embed, the dimensions can be set in both the VisualizationSpec or the EmbedOptions. When the options change, we always recreate the view. However, we dont want to do that for a dimension change.
Thus, dimensions are handled in the spec changes. For ensuring that the dimensions in options are considered, we need to add those to the spec if applicable.

Regarding updateMultipleDatasetsInView, our VegaLite component simply sets the mode to vega-lite, and then renders a Vega component. Thus, this function gets called when something updates for both Vega and Vega-Lite.

I see. Does it make sense to move the logic to the embed component then? This way the Vega and Vega-Lite component could almost just be aliases.

Regarding combineSpecWithDimension, for vega-embed, the dimensions can be set in both the VisualizationSpec or the EmbedOptions. When the options change, we always recreate the view. However, we dont want to do that for a dimension change.
Thus, dimensions are handled in the spec changes. For ensuring that the dimensions in options are considered, we need to add those to the spec if applicable.

I see. So it's a bit of a hack. We just want to make sure that no matter ow dimensions are specified, we don't re-create a view if that's the only thing that changes.

Regarding combineSpecWithDimension, for vega-embed, the dimensions can be set in both the VisualizationSpec or the EmbedOptions. When the options change, we always recreate the view. However, we dont want to do that for a dimension change.
Thus, dimensions are handled in the spec changes. For ensuring that the dimensions in options are considered, we need to add those to the spec if applicable.

I see. So it's a bit of a hack. We just want to make sure that no matter ow dimensions are specified, we don't re-create a view if that's the only thing that changes.

Yes it kind of is a hack to make things easier when comparing states. Do you want that to be changed or should we leave it?

I think dimensions are important enough (and cause issues like https://github.com/vega/svelte-vega/issues/6) that we should think about this a bit.

Let's close this once you removed combineSpecWithDimension @Sparkier

I may have misunderstood, but I thought I should only move updateMultipleDatasesInView. I thought we wanted to keep combineSpecWithDimension for keeping the rerender decision simpler.

Discussed and sounds good.


add pre-built file on npm

hi, recently we add your lib on https://cdnjs.com by manually building, could you please consider adding pre-build file on npm package so that we can use npm auto-update to add later released versions automatically.
thank you very much!
https://cdnjs.com/libraries/politespace
https://github.com/cdnjs/cdnjs/issues/7407

Included as of v0.1.16

thanks a lot!


[FEA] Add signoff check to gh-pages

fix #604
After this got merged, we should update branch rule on gh-pages to enable this check.
e.g.

Is the plan to move the existing sign-off check on the dev branches to Github actions as well?  It's a maintenance burden to be doing it two different ways.

Is the plan to move the existing sign-off check on the dev branches to Github actions as well? It's a maintenance burden to be doing it two different ways.

Yeah, that's my plan here if this one looks good. So we are trying to use Github actions in all trivial GitHub-related workflows if possible. And our internal Jenkins will be a dedicated service for our building and testing since it requires GPU resources

I added the signoff-check to the gh-pages branch.


[Bug] create table failed when string type has a default value which length greater than 1

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
doris-master after doris 1.1.2
What's Wrong?
create table failed when string type has a default value which length greater than 1

CREATE TABLE `tb6` (
  `id` int(11) NULL DEFAULT "10",
  `name` varchar(32) NULL DEFAULT "a varchar 32",
  `c_text` text NULL DEFAULT "a text",
  `c_str` string  NULL DEFAULT "a string"
) ENGINE=OLAP
UNIQUE KEY(`id`)
COMMENT 'OLAP'
DISTRIBUTED BY HASH(`id`) BUCKETS 4
PROPERTIES (
"replication_allocation" = "tag.location.default: 3",
"in_memory" = "false",
"storage_format" = "V2",
"disable_auto_compaction" = "false"
);

What You Expected?
create table successful
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

https://github.com/apache/doris/pull/12187


contributing guide

A guide on how to add new entries to the zoo should be presented here https://niftynet.readthedocs.io/en/dev/

Looks good. I am reopening this as it would also be good to point to the documentation (https://niftynet.readthedocs.io/en/dev/contributing.html#submitting-model-zoo-entries) from here https://niftynet.readthedocs.io/en/dev/extending_net.html#share-the-network-and-trained-weights rather than say

Please consider submitting the design to our model zoo (contact<EMAIL_ADDRESS>if interested).


Cherry-Pick: Update nuget.config file (#1486)

Cherry-Pick: Update nuget.config file (#1486)

Update the nuget.config file to add a  directive to unblock some internal build pipelines work.

Hi @hanzhang54, Hi @guomingrui, Could you help review this PR? Thanks.


add append to target ability for includes

Allow user to add appendToTarget: true to include front-matter

added with commit 5b0e68f83bc45d0a9e1ba94175ece78dc971466b


[function] refine file io connector

(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation

FileListingThread doesn't need to do the permission check since the config.validate() has already done it. Also, as a ListingThread, it doesn't need the Write permission on the directory.
write permission is needed only when keepFile option is set to false.

Modifications

only do file writable permission check when keepFile is false.

Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[X] no-need-doc
internal fix

[ ] doc
(Your PR contains doc changes)

[ ] doc-added
(Docs have been already added)

/pulsarbot run-failure-checks

@freeznet Could you help take a look?

/pulsarbot run-failure-checks


bump grpc to 1.7.0

This version contains keepalive configuration we need to fix pubsub streaming pull hanging.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #382   +/-   ##
=========================================
  Coverage     74.11%   74.11%
  Complexity      522      522
=========================================
  Files           135      135
  Lines          2712     2712
  Branches        194      194
=========================================
  Hits           2010     2010
  Misses          624      624
  Partials         78       78

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 b79b7dd...dc93a28. Read the comment docs.

LGTM after you verify that integration tests pass locally


fix aggregation send/receive

fixes #624

Coverage remained the same at ?% when pulling 551a3a50ab8a087e55522e1c5dab58d6ebf6df40 on fix_aggregation_send_receive into 4d0c369b530f7b5bbea68454ffdaf4238b939a93 on master.


[Bug] [kompile] - macro for record production is applied to itself

K Version
Tell us what version of K you are using.
Example:
$ kompile --version
K version:    v5.1.188-4-g6f2b343a8f
Build date:   Tue Sep 21 16:51:08 UTC 2021

Description
A macro rule for a record production leads to kompile trying to attempt macro expansion on the macro itself.
Input Files
File test.k:
requires "domains.md"

module TEST
    imports INT

    configuration <k> $PGM:Int </k>

    syntax Int ::= f ( a: Int, b: Int )
 // -----------------------------------
    rule f(X, Y) => X +Int Y [macro]

endmodule

Reproduction Steps
Kompile the definition:
$ kompile --backend haskell test.k --main-module TEST
[Error] Compiler: Illegal function symbol _+Int_ on LHS of rule. Consider
adding `simplification` attribute to the rule if this is intended.
        Source(/home/dev/src/k/test.k)
        Location(10,21,10,29)
[Error] Compiler: Had 1 structural errors after macro expansion.
  while executing phase "expand macros" on sentence at
        <none>
        <none>
[Warning] Compiler: Could not find main syntax module with name TEST-SYNTAX in
definition.  Use --syntax-module to specify one. Using TEST as default.

Expected Behavior
I expect either (i) the kompiler gives us a better error, saying "macros are not allowed for record productions", or (ii) it handles this correctly, and does not attempt to apply the macro to itself.

The macro is being applied on a rule generated by GenerateSortProjections. I will investigate further tomorrow.

We need a helper function that tells if a symbol is a macro. Something attached to a Module.
Don't generate the sort projections if the symbol is a macro.


error in linux

NPM version (npm -v):5.4.2
Node version (node -v):8.8.0
Node Process (node -p process.versions):
Node Platform (node -p process.platform):
Node architecture (node -p process.arch):
node-sass version (node -p "require('node-sass').info")4.5.3:
npm node-sass versions (npm ls node-sass):4.5.3

@xzyfer @andrew who can help me?

operating system ：ubuntu 17.04


HSEARCH-4435 Custom Elasticsearch settings/mapping require the backend to be started

https://hibernate.atlassian.net/browse/HSEARCH-4435

Merged, thanks


Const assertions

Proposal
https://devblogs.microsoft.com/typescript/announcing-typescript-3-4/#const-assertions
const foo = ([1, 2, 3]: const) // Type is [+1, +2, +3] (how to do read-only elements on tuple?)
const bar = ('foo': const) // Type is 'foo'
const baz = ({foo: 1}: const) // Type is '{| +foo: 1 |}'

Use case
More explicit version of https://github.com/facebook/flow/pull/7607
Also would eliminate need for Object.freeze as tool for casting to singleton strings
Object.freeze is also a bit broken with regards to subtyping #7964
Not sure how sound it is, given that it is type cast essentially
Alternative
const foo = Object.freeze('foo') // Type is 'foo'
const baz = Object.freeze(1) // Type is 1
const bar = Object.freeze([1, 2, 3]) // Type is [+1, +2, +3] (how to do read-only elements on tuple?)

This would be really handy for objects as well.  Some of our redux code at work defines constants objects like so:
const Constants = {
   Foo: ("Foo": "Foo"),
   Bar: ("Bar": "Bar"),
   ...
};

except the constants are much longer.  It would be nice to be able to do:
const Constants = ({
   Foo: "Foo",
   Bar: "Bar",
   ...,
}: const);

Thia is currently possible with Object.freeze, but only $Values supports it

Does this mean $Values would return a tuple of type ["Foo", "Bar", ...]?

@kevinbarabash, no, union

something new here? this static type checker really has some potential but these missing typescript equivalent features prevent many of us from trying flow seriously...

as const support is now enabled by default


Warn too many staged chunks at configure

Output plugin with many chunk keys will generate too many staged chunks.
It doesn't cause crashes directly, but make performance worse and may cause any problems with file buffers.

This change also includes a commit to separate basic calls/classes about testing from lib/fluent/test/base, because that file is about deprecated Fluent::Test::TestDriver.


.gitconfig rewrites and forge proxies break straight.el

What's wrong
https://github.com/hlissner/doom-emacs/issues/2679
Any changes to the url, whether due to .gitconfig rewrites or forge proxies require manual intervention to rewrite the URL. This is something that should probably be handled transparently.
Directions to reproduce
Add harmless .gitconfig referenced in the URL above, then start Emacs with at least one straight-managed package that has not yet been installed/updated.
Version information

Emacs version: 27.2
Operating system: Linux

Related: #505

What benefit does straight.el extract from tracking redirects? I'm genuinely asking since I might be missing something.
Since the redirects are "lower in the stack" than straight.el, it seems to me that proxy.github.company.io can basically be treated as github.com in that environment, so there should be no need to alter the default mapping. Vim-plug (an admittedly much simpler piece of software) does things this way for example.

I'll have to look into this more deeply when I have time, but the issue probably lies in either straight-vc-git--urls-compatible-p
or straight-vc-git--ensure-remote.
In the latter, we determine the remote's URL by running:
git config --get remote.REMOTE.url

I believe running:
git ls-remote --get-url REMOTE

Instead would factor in any rewrite rules in your git config.
Not sure if that is the appropriate change or not.
@raxod502 Thoughts?

@progfolio You hit the nail on the head. The issue is that your suggested fix does not address proxies.
My question still stands though - What benefit do we extract from performing this check? I'm sure there is one, I just don't know what it is.

What benefit do we extract from performing this check?

If by "this check" you mean straight-vc-git--urls-compatible-p, the idea is that when you've updated the Git URL for a package in its recipe, you want straight.el to be able to tell, and update it in the repository git-remote. However, for this to work, we need a way to check whether the current remote URL in the repository matches the one the recipe specifies.

Not sure if that is the appropriate change or not. @raxod502 Thoughts?

Yeah, that looks right to me.

it seems to me that proxy.github.company.io can basically be treated as github.com in that environment

If I understand correctly, in your environment you're using Git remote URLs that have proxy.github.company.io embedded literally in them, and the issue is that straight.el wants to change them to use github.com instead. Is that right?
I think what you actually want is to use a rewrite rule in your Git configuration, like
% git config --global url."https://proxy.github.company.io".insteadOf "https://github.com"

which would make all of this work transparently. Otherwise, it seems to me like you'll run into issues where MELPA recipes will refer to github.com when you actually want them to refer to proxy.github.company.io.

Not sure if that is the appropriate change or not. @raxod502 Thoughts?

Wait, no. I've read through again more thoroughly and I think I misunderstood what was happening here. When doing the recipe normalization we have 2 options for which URL to look at for a remote:

the URL configured directly in Git's config
the URL as amended after any rewrite rules are applied

I'm pretty sure we want to always use (1) rather than (2). And I suspect that there is some place where we're accidentally using (2) when creating a remote or reading its value, for example, instead of (1). Having a rewrite rule in place shouldn't affect any of the values that straight.el reads or writes in Git configuration.

@wurosh:

I think what you actually want is to use a rewrite rule in your Git configuration, like

% git config --global url."https://proxy.github.company.io".insteadOf "https://github.com"

which would make all of this work transparently. Otherwise, it seems to me like you'll run into issues where MELPA recipes will refer to github.com when you actually want them to refer to proxy.github.company.io.

Does this address the issue for you?

Unfortunately no. I already had that in my .gitconfig

@wurosh: Any updates on this?


limits cpu shares

The documentation for deis limits:set claims it is setting cpu shares.  That is not the case it is requesting whole cpus.
    With --cpu, units are represented in the number of cpu shares. For example,
    'deis limit:set --cpu cmd=1024' will restrict all "cmd" processes to a
    maximum of 1024 cpu shares.

Reading that I would believe
deis limit:set --cpu cmd=1024 would set the pod definition to contain
    Limits:
      cpu:	1024m

The actual definition is
    Limits:
      cpu:	1024

that is correct. From the kubernetes documentation:

How Pods with Resource Limits are Run

When kubelet starts a container of a pod, it passes the CPU and memory limits to the container runner (Docker or rkt).
When using Docker:

The spec.container[].resources.limits.cpu is multiplied by 1024, converted to an integer, and used as the value of the --cpu-shares flag to the docker run command.
The spec.container[].resources.limits.memory is converted to an integer, and used as the value of the --memory flag to the docker run command.

Are you seeing different behaviour on your nodes?

@bacongobbler yep, I had a similar problem a little while back between different k8s versions.  The value deis is currently supplying is x1024 to high.

@bacongobbler This is not a docs issue.  If the expected behavior of is to set 1024 cpu shares for the kubernetes pod that is not the current behavior. deis limit:set --cpu cmd=1024
The current actual result at least in Kubernetes 1.2.4 is to set 1024 cpus

We are just passing the value through to kubernetes. If they now multiply it by 1024 to denote cpus, then we need to document deis limits:set --cpu cmd=0.5 will limit your process to half of a cpu.

The pod definitions for limits accepts units.  Currently Deis is supplying a raw number of X you could also supply Xm to specify mili for the unit.

https://github.com/deis/controller/blob/master/rootfs/api/serializers.py#L18 - the validation we allow for is both passing in straight up numbers and milli units - The [m] part is optional.
So to me the question becomes, do we always want to convert to milli units and forgo the ability to pass in of raw k8s or do we only want to fix docs in this situation. Or ask people to input milli units via deis limits

Personally I think it's better to fix the documentation, given that we should allow both types here as kubernetes does. Users of kubernetes will want to supply both a static number of CPUs via a float value or in milli units.

I don't feel strongly as long as we are consistent.  I will say that for usability reasons, most cluster managers (i.e. Mesos, Swarm, Kubernetes) have moved away from CPU shares -- an arcane concept -- to number of CPUs, which is a lot more understandable.

Ahh if you already support raw units and m I would vote for just fixing the docs.

ok, I'll take a crack at that.