with-apollo does not work after npm run build

Examples bug report
Example name
with-apollo
Describe the bug
The example runs with npm run dev, but when the project is run after npm run build then npm run start, the page load creates a 500 internal server error.  See below for the full error.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

npx create-next-app --example with-apollo with-apollo-app
cd with-apollo-app
npm run build
npm run start

Expected behavior
The example project should run correctly without any modifications.  As it is, there is an internal server error in the base project when the project is built for production rather than ran off of the dev server.
System information

OS: macOS
Browser: Chrome
Version of Next.js: 8.1.0

Error
Error while running `getDataFromTree` { Invariant Violation: Invariant Violation
    at new InvariantError (PATH/with-apollo-app/node_modules/react-apollo/node_modules/ts-invariant/lib/invariant.js:16:28)
    at Object.finish [as children] (PATH/with-apollo-app/node_modules/react-apollo/react-apollo.cjs.js:22:59)
    at a.render (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:44:64)
    at a.read (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:41:58)
    at renderToStaticMarkup (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:53:181)
    at process (PATH/with-apollo-app/node_modules/react-apollo/react-apollo.cjs.js:1043:20)
    at process._tickCallback (internal/process/next_tick.js:68:7) framesToPop: 1, name: 'Invariant Violation' }
{ Invariant Violation: Invariant Violation
    at new InvariantError (PATH/with-apollo-app/node_modules/react-apollo/node_modules/ts-invariant/lib/invariant.js:16:28)
    at Object.finish [as children] (PATH/with-apollo-app/node_modules/react-apollo/react-apollo.cjs.js:22:59)
    at a.render (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:44:64)
    at a.read (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:41:58)
    at renderToString (PATH/with-apollo-app/node_modules/react-dom/cjs/react-dom-server.node.production.min.js:53:83)
    at render (PATH/with-apollo-app/node_modules/next-server/dist/server/render.js:86:16)
    at renderPage (PATH/with-apollo-app/node_modules/next-server/dist/server/render.js:211:20)
    at Function.value (PATH/with-apollo-app/.next/server/static/LklYisY6_HqrmKOShcflL/pages/_document.js:812:41)
    at Object.loadGetInitialProps (PATH/with-apollo-app/node_modules/next-server/dist/lib/utils.js:42:35)
    at Object.renderToHTML (PATH/with-apollo-app/node_modules/next-server/dist/server/render.js:218:36) framesToPop: 1, name: 'Invariant Violation' }

Hi,
me too facing the same issue. its stopped my work. please suggest me an idea to resolve this.

Hi guys, I reproduced the issue and confirmed that it works in the latest release of next@canary, can you again there 🙏 ?

Running it with next@canary worked for me

@gibsonjoshua55 Amazing, I'm going to close this issue now because it's working in canary and the next release will be out soon.


New player character: breathing mechanic

[ ] Add more weapon sway when you're standing still
[ ] make it when holding sprint input action when you're standing still it will hold your breath so there should be no weapon sway
[ ] Only allow holding breath for 30 seconds (or less based on current breath meter value)
[ ] After you stop holding breath, breath meter will fill up slowly again. It will get to full after 10 seconds of not holding breath.
[ ] Make sure to follow the pattern with the new Human_Character Blueprint where we put mechanics in separate components, Blueprint function libraries, or Blueprint macro libraries.

Note we have the breathing mechanic already implemented in old PlayerCharacter_Deprecated BP which you can copy from.

@LeroyTechnologies where should I add the breathing mechanic to?


Add the ability to run Infra Client multiple times from .d configs

As a followup from https://github.com/chef/chef/issues/12475 we need the ability to run multiple Target mode runs using multiple client.rb definitions in the /etc/chef/client.d directory. We should load these files alphabetically and execute them one at a time to avoid concurrency issues.

This probably makes more sense to fire individual forked chef-client runs off one at a time and kill and cleanup the process afterwards to deal with global state issues.  Even though you're not running concurrently you still have the mess of global state.
You take the ruby startup cost then every time in between client runs, but I would argue that we need to again seriously stop accepting rubygems perf issues as being a force of nature and go fix them by wrapping rubygems with a cache of gem-to-filename mappings.  We're still stuck in the Denial/Bargianing stage of the 5 stages of grief over needing to do that, and we need to hit acceptance at some point and take the time to do it, because its going to cost more eventually to not do it.

Migrating to https://chefio.atlassian.net/browse/INFC-38


Add authentication and authorization

Fixes #125  ; refs #125
Implement SSO (authentication) + AD/LDAP (authorization).
@ucsdlib/developers - please review

Coverage decreased (-2.5%) to 89.939% when pulling e56ac06e18217366299b83dfa847ea789d5669bf on feature/ad_ldap into a98151cff9902fb5ae6e46089e8569e9a35a8e33 on develop.

Coverage decreased (-2.3%) to 90.126% when pulling b2b46640c31cf83bbb84a2e1d1d580eb0453c5f3 on feature/ad_ldap into a98151cff9902fb5ae6e46089e8569e9a35a8e33 on develop.

Coverage decreased (-2.2%) to 90.2% when pulling 7473a601122d6e222a5ad458cf9000c81f1f7f54 on feature/ad_ldap into a98151cff9902fb5ae6e46089e8569e9a35a8e33 on develop.


Page Still in Draft/Modified after Published

Affected Version
Composer settings: "silverstripe/framework": "4.2.x-dev",
Composer info: silverstripe/framework            4.x-dev c23d21a
Description

Edited pages still display as draft/modified after I published the page.
Steps to Reproduce

Create a page
Change title
Press publish

This issue wasn't around in 4.1 I think, I wanted to use the setContentCSS method in TinyMCEConfig so I had to upgrade SS framework. After I upgrade, this bug was introduced.

I can replicate the behavior in 4.1 and 4.x-dev.
What I think is going on is that when you click Publish straight after updating the title of a brand new page, a call is made to retrieve a suggested URL Segment, which blocks the publish request.
If you don't change the page title and click publish, it works.
If you focus on another form field before clicking publish, it works.

4.1@stable does not have this issue btw.


[MM-47391] Upstream branch to test e2e

Summary

Ticket Link

Related Pull Requests

Has server changes (please link here)
Has mobile changes (please link here)

Screenshots

Release Note

NONE

/e2e-tests

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

/e2e-tests

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard


Replace Tesseract

Discussed Ideas on this:

For number recognition, you can pretty safely use a low level classification algorithm (regularized logical regression NN) that should run on any computer.

Tesseract is designed for print, not handwriting - ergo the pre-trained models have seen print, which is why it works well here. If you really only want to read out the numbers and with as little computational effort as possible. Then I would neither touch Tesseract nor TensorFlow (neural networks), but work with pixel masks and regression methods (opencv, scikit-learn).

I just wanted to use TensorFlow and would do the forward pass with the pre-trained model weights in NumPy (doesn't have to be much more complex than MNIST with some dot products and ReLU / Sigmoid activations for a start). With my suggestion we would not need to add any new dependencies to our project, because we already have NumPy. But do what you think. I'm not angry with anyone if you have a better idea. Then maybe I learn something ^^

Met Maxim today and told him, that you guys contribute a lot and that you bring this project to a next level - he was happy to hear.
He had 2 ideas:

Adding a voting system for a certain combo/set (like Assassins) and auto-buy it, the moment the unit appears in the shop. Here we need a little more complex OCR-pipeline. If we still wanna get rid of Tesseract, we could use - as @Bonifatius94 already mentioned - MNIST and maybe some "Fonts" generated Texts to train some easy NN. A wordlist, with possible Champions and classes will help us, finding always the correct unit, so it's fine, if the character error rate (CER) of the OCR-pipeline isn't too good.
Additionally we need a Overlay, so people see the chosen combo. We discussed this to handle the time delay in the stream and giving the community the chance to play more competitive.

We should open new issue for that, but still I want to mention it here. Finding the items in the carousel and people can vote for the item, which should be picked (listing also the units will be too much of work in terms of labeling ... I guess) and the avatar is moving indipendendly to the unit with this item, without accidentally running into a Champion with a wrong Item.

Okay, the first feature will than influence this Issue here.
I guess I'll need some help to implement that.
So using MNIST and some easy NN to recognize numbers is not that hard.
I'm just not that sure how we should handle the rest.
How i would solve the problem:
Iterate over the champions in the buy section and process each image by champion names / class names.
The classification should then return the champion / set which matches the image. (So just OCR pipeline using nn which is trained with texts in different fonts)
Valid classes are then a combination of champion name + set  for all available TFT champions (champion / class wordlist)?
If this champion is in the combo or set it shall be bought.
If I'm completely wrong here please correct me :D
So should all recognition (Number & texts) be handled by a single nn or how would you approach that?
Any good datasets you have in mind for training the OCR?

Okay, the first feature will then influence this Issue here.
I guess I'll need some help to implement that.
So using MNIST and some easy NN to recognize numbers is not that hard.
I'm just not that sure how we should handle the rest.

I've had all kinds of AI projects at university for the last 1.5 years (last one was a self-driving car software including some traffic light detection NN, etc). I'm very confident to solve this with ease. TFT is a very constrained environment because units always look the same. The only difficulty is when a new TFT set appears because then the look of the units changes. But it's not too critical either as we would just need to train a recognition logic with new data. This it's more about the data really, the training and AI part is rather easy 😄


Support code gen using ? for wildcards

In Scala 3 it is possible to use _ for type lambda placeholders by setting the -Ykind-projector:underscores compiler flag. When this is enabled _ can no longer be used for wildcards, ? must be used instead. The code gen doesn't support this ATM so you end up with compiler errors from the generated code.

ScalaPB needs to support this as fs2-grpc piggy-backs on it. See https://github.com/scalapb/ScalaPB/issues/1316
Changing the code generator is not a big deal, but until ScalaPB supports it we have to wait to implement it. I have a PoC that feeds the code generator with scala target version, so it is doable.


Title and description don't show up when picture is portrait

Title and description don't show up when picture is portrait.
When the picture is a portrait, it takes up the entire screen and the title and description don't show up. How can I fix this?

If your image is to big, please use the scrollable(true) method as documented in the readme.


merge from arlac77/npm-package-template-kronos-module

package.json

chore(scripts): add ava --node-arguments '--experimental-modules' --timeout 2m (scripts.test)
chore(scripts): add c8 -x 'tests/**/*' --temp-directory build/tmp ava --node-arguments '--experimental-modules' --timeout 2m && c8 report -r lcov -o build/coverage --temp-directory build/tmp (scripts.cover)
chore(package): add ^3.4.0 remove ^3.3.0 (devDependencies.ava)

README.md

docs(README): update from template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #658   +/-   ##
=======================================
  Coverage   85.53%   85.53%
=======================================
  Files           8        8
  Lines         560      560
  Branches       67       67
=======================================
  Hits          479      479
  Misses         80       80
  Partials        1        1

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 b02ce28...7ffd68b. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Render empty whereIn as 1 = 2

Is your feature request related to a problem? Please describe.
Currently an empty whereIn clause will render as empty list which results in a MySQL error.
Describe the solution you'd like
An empty whereIn clause should render as 1 = 2 or some other false statement in order to replicate the expected behavior of WHERE x IN ()
Describe alternatives you've considered
Excluding the statement is another option but that's dangerous as sensitive data might be exposed that otherwise would have been filtered by the IN clause and doesn't really match the expected behavior of does the set () include element x.

Fixed in 1.0.2


Global UBO

Implemented global UBO, for paint parameters that are same for all shaders.
Optimised shaders UBOs by using the global UBO.
This will reduce the number of UBO binds and it will reduce also the bandwidth between CPU and GPU.
The shaders have the following UBOs:

paint parameters UBO, it's set at global level and contains parameters that are the same for all shaders.
evaluated properties UBO, it's set at layer group level and contains the layer properties.
drawable UBO, it's set at drawable level and contains the matrix and other properties that are drawable dependent.
interpolate UBO and tile properties UBO are set at drawable level in the render layer.

!benchmark android


Verify hashsums of files imported over HTTP

Stacker should do a HTTP HEAD call check the shasum of the files imported using HTTP.
There are several cases:

If the shasum is missing - keep current behavior, download the file every time regardless of cache
If the shasum is present and matching the shasum of the file cached locally skip the download
If the shasum is present and different from the local shasum, download the file

I've successfully used the method below with Artifactory in the past.
// getRemoteFileHash returns Sha256 for a file stored on a web server
func getRemoteFileHash(remoteURL string) (string, error) {

	// Make a HEAD call on remote URL
	resp, err := http.Head(remoteURL)
	if err != nil {
		return "", err
	}
	defer resp.Body.Close()

	// Get Sha256 from header
	hash := resp.Header.Get("X-Checksum-Sha256")
	return hash, nil
}

@hallyn I understood from @tych0 that your input is needed before proceeding here.

I just want to make sure that I can do builds, say, on an airplane, with everything cached.  So you can either ignore Head errors (after all, you already downloaded it once), or take a stacker --remoteisland argument (--no-net, --private?) to say "don't do these checks".  That's fine with me.

Fixed in https://github.com/anuvu/stacker/pull/56


All instantiate of framework services should be with "If"

Problem: I cant replace "auth.password" in own service provider becouse framework code has deferred service provider for "auth.password"
So, i should create one more deferred service provider in my package just only for replacing framework's code.
It will be better to define all framework core's services with
singletoneIf
bindIf
and etc function for easy developing packages.
Thank for answer.

Packages (generally) probably shouldn't be messing with that stuff, and if they are, only one package at a time should be, so you should extend the framework service provider and instruct users to replace the framework provider with the package one.

Thanks for answer


create seed file from  the existing data in tables

hi
i think it is good to also create seed file from  the existing data in tables

Xethron,
It would actually be helpful to create seeds in some cases that are not dummy data.
For example, switching from sqlite to mysql where you can't import the sql dump from one to the other because of syntax differences.


Warning: Unknown props trim, trimOnSubmit on  tag.

Warning appears in browser console<EMAIL_ADDRESS>

Thank you!


Ensure Chains works with the v1 version of TaskRuns and PipelineRuns

Pipelines APIs have been updated to v1 from v1beta1.
Make sure that these versions work well with Chains.

Context: today chains is only watching v1beta1 pipelines.

I wonder if it's possible for Chains to handle both v1beta1 and v1. That would facilitate adoption.

I wonder if it's possible for Chains to handle both v1beta1 and v1. That would facilitate adoption.

+1
In fact, we need a spin-off issue around handling cross-version compatibility with pipelines changes.

So, technically, as long as v1beta1 is served by tektoncd/pipeline, chains will work because the controller watches v1beta1 types, which are automatically converted, no matter how the user created them. This means that, today, if a user creates a v1 type, chains will handle it (because it will get it while watching v1beta1 objects).
I think, in a few release (most likely, more or less at the same time 0.47 gets release with a storage version change), chains should switch from watching v1beta1 to v1. This would make chains not working with older pipeline version (older than 0.43), but I think at some point this will be fine.
We may want to rename this issue to "switch chains controller to watch v1 types" instead 👼🏼

Immediate AIs:

[ ] future releases of chains need to include release notes on compatibility with pipeline versions
[ ] future releases of pipelines need to include release notes on compatibility with chains versions

TEP please:

[ ] enhance chains and pipelines such that installation of a version incompatible with the existing installation would fail or at least alert to the problem

@bendory definitely +:100: on your checklist ❤

/area roadmap

/lifecycle frozen

Here is the proposed action plan for this:

Switch to using v1  internally in Chains (throughout)
Convert all v1beta1 objects to v1  in the controller/object interface.
Wherever there are deprecated things like pipelineResources , we could partially convert back to v1beta1 where needed so that appropriate things can be assessed.

This would completely satisfy chains/v1  predicate.
The only issue is with v2alpha2  where we embed the complete runSpec. We could leverage the configMap for that and depending on the desired version, convert to v1 or v1beta1 before sticking it into the provenance.
Eventually, when v1beta1  is deprecated, we could remove support for the deprecated fields from Chains.

/assign

Starting on the work here, made some issues to track the subtasks:

[ ] https://github.com/tektoncd/chains/issues/985
[ ] https://github.com/tektoncd/chains/issues/986
[ ] https://github.com/tektoncd/chains/issues/987

If someone has comment edit access, feel free to make this an [Umbrella Tracking] issue and put this checklist in the top comment

/assign @aaron-prindle


Edit campaign route and description

We have updated the URL and description following user research.

@andrewgarner I've got the unpublish task working (tested on integration). Are you able to re-review?


fix(runtime-core): support deep: false when watch reactive

fix #9916

I added a depth option in #9572 to control the depth of watch listeners. When the object being watched is a shallowReactive, it should only listen to changes at the first level.

@Alfred-Skyblue I like the depth idea, but it is a feature so I refactored this PR to use your depth-based traverse so we can land the fix in a patch sooner.


feat: ca certificate is optional for https configuration

A PR for #1835
One thing worth mentioning and please correct me if I am wrong: previously verdaccio config https.ca expected a CSR(Certificate Signning Request), but from node.js doc, it should be a CA certificate.
I think the only use case to setup https.ca is when the client uses a self-signed certificate.
@juanpicado please review and comment, thanks!

Thanks @ddhp I'll take a look :) 💯

@ddhp heads up you might need to check the lint step is failing

I spent some time trying to figure out why linting is failing, and I found out it's because of the yarn.lock file, although I still don't know what's the exact reason. I updated the lock file because a new dependency is introduced, anyhow when I checked the lock file, there are many changes renaming registry.verdaccio.org to registry.npm.org.
May I ask what's the expected way to handle lock file in this scenario? Did I miss something about the lock file?
Thanks for your help!

Some modification needs to be done on the documents too

I spent some time trying to figure out why linting is failing, and I found out it's because of the yarn.lock file, although I still don't know what's the exact reason. I updated the lock file because a new dependency is introduced, anyhow when I checked the lock file, there are many changes renaming registry.verdaccio.org to registry.npm.org. Now I added the dependency with yarn add selfsigned --pure-lockfile
May I ask what's the expected way to handle lock file in this scenario? Did I miss something about the lock file?

Yeah, sorry for that. Long story sort, we have a private registry just for cache and few canary versions. Lock file points there but I don't expect everybody uses it ... we 🙃 cannot afford that.

Thanks for your help!

Thanks for your help :) welcome to Verdaccio.

@all-contributors please add @ddhp  for translation, code

Some modification needs to be done on the documents too

@ddhp feel free to update the docs :)

Hello, I'm still getting this error: "You have enabled HTTPS and need to specify either "https.key", "https.cert" and "https.ca" ....".
I've only set the key and cert values since I don't need to set the CA. Verdaccio version is at 4.7.2.

@vocafeuvre please try with v4.8.0!


Added missing word (...cannot BE null...).

Thanks @jeffa00 but there is already a verb in the sentence: contain. With your proposal we would have two verbs "contain be" and that would make the sentence incorrect.

Current: A property whose CLR type cannot contain null
Proposed: A property whose CLR type cannot contain be null


[release-v1.28] Pick 2325

#2325
Description

For PR author

[ ] Tests for change.
[ ] If changing pkg/apis/, run make gen-files
[ ] If changing versions, run make gen-versions

For PR reviewers
A note for code reviewers - all pull requests must have the following:

[ ] Milestone set according to targeted release.
[ ] Appropriate labels:

kind/bug if this is a bugfix.
kind/enhancement if this is a a new feature.
enterprise if this PR applies to Calico Enterprise only.

/merge-when-ready delete-branch

I'm sorry but I failed delete the branch after merging the pull request.


Typescript types pointing to wrong file

Just installed the client. VS Code can't find TS types probably because in package.json this field is incorrect: "types": "src/mistralai.d.ts" when it should be "types": "src/client.d.ts".

Ahh, I see a PR to fix this has been merged but a new release hasn't been issued yet.

Hi @brenc Have pushed the new version of the client. let me know if this fixing your issue!


Land Debate page mobile device: Overlap of text and images

Hi,
On the mobile device, the different information items of an online debate page (dates, status (open/closed) and facilitator profiles seem to be kind of overlapping:
http://landportal.info/debates/2017/responsible-large-scale-agricultural-investment-mekong-region-online-dialogue-9-20

I don't know what the order should be, but in any case this information should probably not overlap :)
Thanks!

fixed, see: https://beta.landportal.info/debates/2017/responsible-large-scale-agricultural-investment-mekong-region-online-dialogue-9-20


Missing dependencies

The game crashes upon loading.
Using Oculus PCVR with public_beta channel and newest Melon Loader
Relevant logs:
[18:50:30.881] Some Melons are missing dependencies, which you may have to install.
If these are optional dependencies, mark them as optional using the MelonOptionalDependencies attribute.
This warning will turn into an error and Melons with missing dependencies will not be loaded in the next version of MelonLoader.
- 'BoneLib' is missing the following dependencies:
    - 'SLZ.Marrow' v<IP_ADDRESS>
    - 'SLZ.Marrow.SDK' v<IP_ADDRESS>
    - 'UnhollowerBaseLib' v<IP_ADDRESS>
    - 'UniTask' v<IP_ADDRESS>
- 'LabFusion' is missing the following dependencies:
    - 'SLZ.Marrow' v<IP_ADDRESS>
    - 'SLZ.Marrow.SDK' v<IP_ADDRESS>
    - 'UnhollowerBaseLib' v<IP_ADDRESS>
    - 'UltEvents' v<IP_ADDRESS>
    - 'UniTask' v<IP_ADDRESS>
[18:50:30.896] ------------------------------
[18:50:30.900] BoneLib v2.2.1
[18:50:30.900] by Gnonme
[18:50:30.906] Assembly: BoneLib.dll
[18:50:30.907] ------------------------------
[18:50:30.927] [LabFusion] Data is at C:/Users/USER/AppData/LocalLow/Stress Level Zero/BONELAB/Fusion/.
[18:50:30.928] [LabFusion] Loading embedded resource data LabFusion.Core.resources.lib.x86_64.steam_api64.dll...
[18:50:30.929] [LabFusion] Done!
[18:50:31.037] [LabFusion] Successfully loaded steam_api64.dll into the application!
[18:50:31.040] Failed to register Mod 'F:\OculusGames\Software\stress-level-zero-inc-bonelab\Mods\LabFusion.dll': Melon failed to initialize!
[18:50:31.042] System.TypeLoadException: Could not load type 'SLZ.Rig.RigManager' from assembly 'Assembly-CSharp, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null'.
   at System.Delegate.GetMulticastInvoke()
   at System.MulticastDelegate.CombineImpl(Delegate follow)
   at LabFusion.Utilities.MultiplayerHooking.add_OnLocalPlayerCreated(RigManagerEvent value)
   at LabFusion.Utilities.PlayerAdditionsHelper.OnInitializeMelon()
   at LabFusion.FusionMod.OnEarlyInitializeMelon()
   at MelonLoader.MelonBase.Register() in D:\a\MelonLoader\MelonLoader\MelonLoader\Melons\MelonBase.cs:line 397
[18:50:31.043] ------------------------------
[18:50:31.044] 1 Mod loaded.

Turns out this was because I forgot to install Melon Loader using the provided Zip file for MelonLoader.


Update README.md

The README looks really comprehensive.
A couple of minor things I would suggest changing / adding:

I think it'd be helpful for the readers to know that we're interested in deploying the gRPC client of TF Serving via GitHub Actions.
A note on how to perform inference with the gRPC client deployed via Actions. Or better yet, include the pre-processing and post-processing handlers since we're not load-testing anymore.
Brief notes about the important numbers shown in the load-test plots.
Spec of the machine we used to perform load-testing.
How the load-test charts were generated.

@sayakpaul
merging your PR, and I will consider to add your suggestions!

A couple of minor things I would suggest changing / adding:

I think it'd be helpful for the readers to know that we're interested in deploying the gRPC client of TF Serving via GitHub Actions.
A note on how to perform inference with the gRPC client deployed via Actions. Or better yet, include the pre-processing and post-processing handlers since we're not load-testing anymore.
Brief notes about the important numbers shown in the load-test plots.
Spec of the machine we used to perform load-testing.
How the load-test charts were generated.


initialize the package with an empty table state

Hi!
I love the project, I am already using it! I'm on my way to develop a game in TTPG 🌱
With the use, I realized that the startup doesnt give you a initial assets/States/start.vts or something.
So when you create a new mod, and start the editor, a warning is shown saying
Could not load saved state
invalid save file

What do you think? Is it reasonable?
Thanks!

This is a sample .vts file

{
	"saveStateVersion": "1.1",
	"gameState":
	{
		"measureUnit": 2.5399999618530273,
		"rotationStep": 45,
		"notes": "You can enter notes for the game here. They will be visible to all players and are stored when the game is saved.\n\nYou can use [color=#ff0000]colors[/color], [b]bold[/b] or [i]italic[/i] text, [size=9]different[/size] [size=16]sizes[/size], and [color=#00ff00][size=8][b][i]combine text styles.[/color][/size][/b][/i]",
		"turnInfo":
		{
		},
		"slotTeams": [ 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 ],
		"globalScriptPackage": "00000000000000000000000000000000",
		"globalScriptName": "",
		"backgroundTexture":
		{
			"resourceName": "",
			"packageGuid": "00000000000000000000000000000000"
		},
		"phases": [],
		"spectatorsSeeEverything": false,
		"persistentData": "",
		"persistentKeyData":
		{
		},
		"permissions":
		{
			"delete": -1,
			"objectLibrary": -1,
			"copyPaste": -1,
			"cardPeek": -1,
			"cardExplorer": -1,
			"containerExplorer": -1,
			"changeOwner": -1,
			"changeTeam": -1,
			"editZones": -1,
			"draw": -1,
			"ground": -1,
			"throw": -1,
			"editLabels": -1,
			"saveGame": -1,
			"spectators": -1
		},
		"physicsLocked": false,
		"measureAngles": "None",
		"alwaysSnap": false,
		"liftOverRegular": true,
		"gravityMultiplier": 1,
		"slotIds": [
			"00028a7986104715ad744734c892c2b1",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			"",
			""
		],
		"mapName": "HDRI_Field",
		"currentTurn": 0,
		"bCanBeDamaged": false
	},
	"requiredPackages": [
		{
			"name": "General",
			"guid": "D74C7D5D6745CD565913DAA5FB3E9C93"
		}
	],
	"storedCameraSetups": [ false, false, false, false, false, false, false, false, false, false ],
	"playerCameraSetups": [ false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false ],
	"playerSlotNames": [
		"1",
		"2",
		"3",
		"4",
		"5",
		"6",
		"7",
		"8",
		"9",
		"10",
		"11",
		"12",
		"13",
		"14",
		"15",
		"16",
		"17",
		"18",
		"19",
		"20"
	],
	"customPlayerColors": [ false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false, false ],
	"lighting":
	{
		"intensity": 1,
		"specular": 1,
		"altitude": 90,
		"azimuth": 0,
		"color":
		{
			"r": 255,
			"g": 255,
			"b": 255
		}
	},
	"grid":
	{
		"type": 0,
		"snapType": 0,
		"visibility": 0,
		"size":
		{
			"x": 5.0799999237060547,
			"y": 5.0799999237060547
		},
		"offset":
		{
			"x": 0,
			"y": 0
		},
		"rotation": 0,
		"color":
		{
			"r": 0,
			"g": 0,
			"b": 0,
			"a": 179
		},
		"thickLines": false
	},
	"objects": [
		{
			"objectType": "Table",
			"transform":
			{
				"rotation":
				{
					"x": 0,
					"y": 0,
					"z": 0,
					"w": 1
				},
				"translation":
				{
					"x": 0,
					"y": 0,
					"z": 0
				},
				"scale3D":
				{
					"x": 1,
					"y": 1,
					"z": 1
				}
			},
			"simulatingPhysics": true,
			"primaryColor":
			{
				"b": 255,
				"g": 255,
				"r": 255,
				"a": 255
			},
			"secondaryColor":
			{
				"b": 0,
				"g": 0,
				"r": 0,
				"a": 255
			},
			"metallic": 0,
			"roughness": 1,
			"friction": 0.69999998807907104,
			"restitution": 0.30000001192092896,
			"density": 1,
			"surfaceType": "SurfaceType1",
			"objectName": "",
			"objectDescription": "",
			"collisionType": "CB_Static",
			"templateId": "F27689B0431FFA778EC9D6835FA2B8FC",
			"shouldSnap": true,
			"previousPosition":
			{
				"x": 0,
				"y": 0,
				"z": 0
			},
			"objectScriptPackage": "00000000000000000000000000000000",
			"objectScriptName": "",
			"persistentData": "",
			"persistentKeyData":
			{
			},
			"uniqueId": "nu2",
			"drawingLines": [],
			"objectTags": [],
			"objectGroupId": -1,
			"ownerIndex": -1,
			"lightsOn": true,
			"bCanBeDamaged": false
		}
	],
	"zones": [],
	"labels": []
}

</details

Sorry, I didn't notice this issue until now >.<
from a design standpoint: I'm a little hesitant to have the template create an initial Save State automatically. I see the logic, but consider the scenarios:

You need a save state for your mod, and the script provides a starter one: One would need to edit that initial state, or will end up deleting the initial state in favor of creating on themselves. It'd save you a mouse-click, maybe.
suppose one forgot about needing to create a save state (which I know I've done a few times) and go to publish. TTPG sees the default state and says "oh yeah, there's a state here, no problem" and you've published your mod with a blank start state.

You need a save state for your mod, and the script doesn't provide one: you're not really that worse off: you'd just need to create one. On the off chance that one forgets this step and you go to upload, TTPG will catch it and say "there is no save state, are you sure?" - saving you from uploading a mod without a save state. Otherwise, if you forget, and go to upload: you've got an empty save state because the template created the blank one, so TTPG never warns you.

You don't need a save state (which happens a lot for utility object-type mods; in fact 90% of mine don't have a save state at all, they're just meant to be imported individually) and the script provides one. At that point you have to remember to remove the initial state the script provides, otherwise TTPG will upload your mod without question and you've got a blank/empty state that serves no purpose and is, again, confusing for consumers.

You don't need a save state and the script doesn't provide one. When you go to upload, TTPG will ask "there is no start state: are you sure?" at which point you go "yup!"

Having the template not provide an initial state seems the least "dangerous" (for a given measure of dangerous) and has the most safety in terms of double-checks. Providing one by default seems fraught with potential accidents and false positives.
As far as the error you're getting? I haven't seen that, but I've never really gone looking since it doesn't seem to interfere with my ability to edit a package. Can you share a bit of your process and share where you see that warning?

Hello again! First, thank you for being very thorough!
I see that it is more reasonable not to create a save state than to create the package with an empty one.
As for the error I was getting, I tried with a new package and I couldn't reproduce it, so it was just some random error that sometime occurs. Don't worry!
Is it alright if we take a section on the README.md explaning this thing of the Save State?
I believe more than one dev will be confused if they also expect a Save State to be created with the package setup. What do you think?

Makes sense - I'll write something up in the coming days and add it to the readme :)


Sigmoid  for lsgan

Why sigmoid is not used in Discriminator last layer for lsgan, I do not understand

As far as I understand, least square loss is mainly used in regression model, and we don't need activation function (sigmoid/softmax) here. But in classification model, where cross entropy loss is mainly used, activation function works as converting output to probability of each class.
The maths behinds it is very complicated that I can't explain fully here but in practice just remember: least square → no sigmoid/softmax, cross entropy → need sigmoid/softmax.


Add note why we use Merge.ensureOnHandler in pre Union Merge

For the merge which might be necessary before the union merge, we use the
Merge.ensureOnHandler method. Execution on handler is not strictly required but
the union merge afterwards will be performed on the handler anyway.
Follow-up for #6644.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #6654   +/-   ##
=========================================
  Coverage          ?   83.12%
  Complexity        ?    13385
=========================================
  Files             ?     1525
  Lines             ?    49638
  Branches          ?     5922
=========================================
  Hits              ?    41263
  Misses            ?     5563
  Partials          ?     2812

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/io/crate/planner/operators/Union.java
87.83% <100%> (ø)
18 <0> (?)

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 ff497ad...63810f7. Read the comment docs.

@mfussenegger  If we have left and right relations of a union having both a merge phase to apply for example the final ordering + limit isn't it better in terms of more parallelism over nodes if those merge phases are not forced to execute on the handler?

@matriv On the one hand you're correct, the pre-union merge phases could be parallelized better. On the other hand, the data needs to be moved to the handler node for the actual union, so we save network traffic by moving everything to the handler node and performing the pre-union merge there, instead of moving it to a different node first and then moving it to the handler.
Will merge this for now but feel free to discuss. Also feel free to correct me @mfussenegger.


Log player login name changes

When a player logs in, if NameLayer is changing their name because they have changed their name with Mojang, please log it.
[NameLayer] ABC (uuiduuid-uuid-uuid-uuid-uuiduuiduuid) name changed to XYZ
ABC is their current Mojang name. XYZ is the name they will be forced to use on the server.
This is somewhat orthogonal to #80, as it would allow faster banning of the account's current Mojang name. Even if #80 is fixed, it will allow easier mapping since the initial login entry with their UUID contains their Mojang name and the very next "logged in" entry contains their NameLayer name with no UUID.

In newest build


Reduce the security bits for MD5 and SHA1 based signatures in TLS

This has as effect that SHA1 and MD5+SHA1 are no longer supported at
security level 1. This has as effect that TLS < 1.2 is no longer
supported at the default security level of 1, and that you need to set
the security level to 0 to use TLS < 1.2.
The first commit is the commit from #10785.
This will probably not get applied to the 1.1.1 branch, only to master.
This currently has a lot of failing test, because all tests using TLS < 1.2 or DTLS < 1.2 now need to run at security level 0. I need to look at those tests again.

Checklist

[ ] tests are added or updated

This should have a really obvious user-visible change notice (NEWS or CHANGES or ...)

I've updated the tests.

Can somebody review this?

So does anybody want to review this?

Please rebase so that CI tests will run...

rebased

I don't remember any discussion about making this configurable. If you think this should be configurable, how do you suggest we do that?

I don't remember any discussion about making this configurable. If you think this should be configurable, how do you suggest we do that?

Well, it would have to be in the configuration file, from there a new option in the SSL_CTX and/or SSL handle, and downward into X509_VERIFY_PARAM, because this is needed both in libssl to accept the TLS sigalg and in X.509 to accept certificate signatures.
Just implementing @AUTHLEVEL=0 (which has been requested in the past) is not quite what's needed, because on the one hand it permits too much (also weak public keys, ...), and on the other not enough (no effect on TLS sigalgs).

Any idea what the travis failure is about?

Any idea what the travis failure is about?

It's not related to this PR, I've seen in broken in other PRs too.

Note that this PR does not have any effect on X509, just the TLS sigalgs. If you think we need more configuration support for this, I suggest we do that in an other PR.

I forget, were we going to make this configurable in master, or unconditional?

I see no reason to make this configurable. We already have a configuration option to control this "@SECLEVEL". Having a configuration option to control how the configuration option behaves seems a bit nonsensical to me.

This seems like a significant breaking change. Should there be an OMC vote for it?

If the outcome is no, would you leave it insecure by default? ;) It would be really nice if you could vote about uplifting/backporting it into 1.1.1 by the end of the year - against original intent.
Debian 10 (Stable) and Ubuntu 18.04 (debian/patches/Set-systemwide-default-settings-for-libssl-users.patch) already enforce SECLEVEL=2 by default. That covers mail servers as well.
GnuTLS wants to do this change as well: https://gitlab.com/gnutls/gnutls/-/issues/910
TLS 1.0 and 1.1 have been re-enabled in Chrome Stable and Firefox Stable (so that desperate users can access every carelessly configured government website in this exceptional situation), but are left disabled in Beta and Dev/Nightly. These websites are getting fixed.
The last known DTLS 1.0 user was Jitsi, but they have fixed it and web browsers will probably disable it in the summer. Originally it was supposed to be disabled in Firefox and Chrome last month. It was left disabled in Firefox Nightly.
Chrome and Firefox want to disable the RSA_PKCS1_SHA1 signature algorithm in TLS 1.2 as soon as possible. Every announcement to nudge these few remaining large websites to update or replace their load balancers is highly appreciated.
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/yaJcs4p9LNI
https://bugs.chromium.org/p/chromium/issues/detail?id=658905
https://github.com/ssllabs/ssllabs-scan/issues/465#issuecomment-554669595

This seems like a significant breaking change. Should there be an OMC vote for it?

This question was asked by @mattcaswell  a month ago. IMHO it deserves at least a discussion, so I placed an omc hold label on his behalf as a reminder. Matt, if you changed your mind, please clear it again.

I'm lifting the OMC hold on this PR. The OMC have voted on this:

topic: The TLS security bit values for MD5, MD5_SHA1 and SHA1 should be set to
39, 67 and 65 respectively in OpenSSL 3.0. Consequently TLS < 1.2 will
be disallowed in the default security level

The result was:
+1: 4
0: 1
-1: 2
So the vote passed and this PR can go ahead.

So it's now waiting for someone to review it

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.


Convert Scrapy Spider to new pipeline: australia_nsw

The new pipeline has several benefits we want; better filenames, error logs, not hitting API end points twice sometimes ...

@odscjames is this still needed? I know we've done some work with this publisher recently

Nope; this now uses the new pipelines


LED state not updating with Beta 3

For some reason the LED state of keypads (and also when the box is checked, that translating into button state) is not updating. I do see the LED status messages show up in the debugging log, but the status isn't changed in indigo, and none of my former triggers based on button presses are working...

I need to do some tests. This is caused by either/both:

the beta 3 plugin
LEDs (set to control button status) on tabletop keypads

When I reverted to the prior plugin code and disabled those LEDs, everything went back to normal.
I haven’t had a chance to look, but I suspect there is some implicit expectation that the LEDs will be in the range of 81-87 somewhere in the monitoring code that is causing this problem. I do note that I had these LEDs disabled in indigo before, so I suspect this issue may have been in prior releases also (so not something that changed with the latest beta)

Ok, there's two bugs here.  One is in the new code, where I changed the internal key format for the keypad buttons.  Keypad 25, button 3 used to be '253', now it's '25.3' to match the external address.  That's an easy fix.
The second bug is in the original code, where it's converting the key for an LED (2583, for example) to the button key (253).  It's doing it by just removing the 8.  Which breaks a tabletop keypad because it might need to go from 2594 to 2514.  I'll have that fixed soon.

The second bug makes sense, that's what I figured was happening as I was thinking about the problem while in the shower (where I do all my best thinking).
I don't understand the first bug - is the issue that not all of the code is expecting the dot seperated notation? I would agree with you that it probably makes more sense to use a dot or some other separator (maybe even an array), especially as dual-repeater systems can have IDs above 100? (Is that the case? I am still only on a single repeater system)...

The external address (visible in the Indigo UI) used the dot notation for keypads, etc.  Internally, it was using a non-dot version as a key in a dictionary.  I changed that to always use dot notation.  Just because I wanted them to match.  That way I could use the external address for the key rather than building the key on the fly.  But I hadn't fixed all the places it used the key yet.
Phantom buttons now have the Integration ID for the repeater prefixed to them.  So you'll see buttons like 1.100, 1.101, etc.  My second repeater has an Integration ID of 13, so I also have phantom buttons 13.100, 13.101, etc.

Fixed in next release.


IOS 11 key name changes

Key name was changed from _requiresUserActionForMediaPlayback to requiresUserActionForMediaPlayback
Quick fix:

Open generated application in xcode
Open plugins/APPBackgroundMode.m

change
  [obj setValue:[NSNumber numberWithBool:NO]
               forKey:@"_requiresUserActionForMediaPlayback"];

to: (Remove underscore _)
  [obj setValue:[NSNumber numberWithBool:NO]
               forKey:@"requiresUserActionForMediaPlayback"];

Thank you for noting this. Since katzer has been away for a while, I have created my own fork of this, which fixes another issue described in my pull request.
I am interested to see if I also need to include your fix. What problem did you notice with the previous code? What exactly does this break? Could you please elaborate?

were struggling with that for a while #317 ...
@dimitriscsd does your fork runs fine with all states of the background mode,  on ios11 with WkWebView? thanks!

Hello. I haven't modified anything regarding WkWebView in my fork. I was asking the OP here to find out what's going on, thinking maybe I should add that.
The changes I have done on mine are regarding Android code for another issue. Everything else is currently same as master


Dont know why, but removing profiles does not work

this is kinda necessary

ea4e836a0c6b690c559a2fd0d6cef742373c90c8 fixes this
i dont know why i cannot have the onclick function to change the selected profile as shown in the commit, very annoying and can hopefully be changed so it can be done like that


script adding-new-user

Issue: adding new users when requested is simple enough that I've been doing it manually, but it's just complex enough, with the user, user_profile, and DISA-db users tables, that scripting it would be better.

Goal of Birkin with the four new researchers coming in week of 11/11/2020


Why using "force_fp32"?

Hi,
I notice that some of the functions are using "force_fp32" although trained under amp. What is the rationale of this?

torch_scatter supports fp32 only


suggestions open only on first caret even when it out of view

VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:43:53.668Z)
OS version: Windows_NT x64 10.0.17134
Steps to Reproduce:
1.Select multiple lines in file(more than screen can show, or just scroll down)
2.And now try to open suggestions popup
Isn't it needs to open on first visible caret?

It's not clear what you mean, can you post a screenshot?

So vscode needs to show suggestions on first visible line or jump to first selected line when you try to open them

Duplicate https://github.com/Microsoft/vscode/issues/6995


Patch 1

What type of pull request is this?

[ ] Non-breaking code change
[ ] Breaking code change
[X] Documentation change/addition
[ ] Tests change

Description

A bunch of grammar and spelling fixes for the docs
Changed token instructions to match new Discord Developer Portal design
Changes

oh boy

Checklist

[ ] I've formatted my code with Black
[ ] I've ensured my code works on Python 3.10.x
[X] I've tested my code

This targets master. Are you sure that's what you want?

I had no idea what to name it and I got a #404 when trying (heh, get it?)

I had no idea what to name it and I got a #404 when trying (heh, get it?)

As zev said, that's understandable! For the sake of transparency, I renamed this PR to something I think fits personally after struggling to decide what to name it for a bit.
In general, I'll admit much of our docs needs polishing with grammar and all, I've just kinda held off on doing it. Thanks for the PR - this is a great start!


flutter run does not terminate after failure to install

If I yank the cable from my device during installation (I know this isn't normal, but I'm simulating https://github.com/Dart-Code/Dart-Code/issues/329!) then flutter run sends an app.stop notification but does not terminate; leaving the process around.
Since this is not daemon and starts the app immediately at startup, I would expect that on app start it would terminate (and if I send a stop command, I believe it does).
PS M:\Coding\Applications\flutter\examples\stocks> flutter run --machine
[{"event":"app.start","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","deviceId":"ZY2224ZKJJ","directory":"M:\\Coding\\Applications\\flutter\\examples\\stocks","supportsRestart":true}}]
Launching lib/main.dart on XT1562 in debug mode...
[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"0","progressId":null,"message":"Initializing Gradle..."}}]
[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"0","progressId":null,"finished":true}}]
[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"1","progressId":null,"message":"Resolving dependencies..."}}]

// Cable yanked around here

[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"1","progressId":null,"finished":true}}]
[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"2","progressId":null,"message":"Running 'gradlew assembleDebug'..."}}]
[{"event":"app.progress","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","id":"2","progressId":null,"finished":true}}]
Built build\app/outputs/apk/app-debug.apk (21.6MB).

// Error comes back with a stop notification

error: device 'ZY2224ZKJJ' not found
[{"event":"app.stop","params":{"appId":"006f6f46-ba1e-4a68-aaf4-f3d53e9b8b80","error":"Exit code 1 from: M:\\Apps\\Android\\platform-tools\\adb -s ZY2224ZKJJ shell pm list packages io.flutter.examples.stocks"}}]

// Process is still alive

Is my understanding correct - flutter should terminate here?

@DanTup What's the status of this one, do you know?

Honestly, I had forgotten about this since I raised it (it was raised well before I did any work on Flutter tools). However, I wonder if it's a similar issue to #16183 which I'm hoping to look at very soon.

This is still a problem. In some cases it doesn't even send a stop notification.
Dart Code extension: 2.15.0-beta.1
Platform: mac
Workspace type: Flutter
Multi-root?: false
Dart SDK:
    Loc: /Users/dantup/Dev/Google/flutter/bin/cache/dart-sdk
    Ver: 2.0.0-edge.4b2d60cb18e418c5cb92cf22d2d959d8b70142a3
Flutter SDK:
    Loc: /Users/dantup/Dev/Google/flutter
    Ver: 0.5.6-pre.30

Thu Jun 21 2018 [14:25:38 GMT+0100 (BST)] Log file started
[14:25:44 GMT+0100 (BST)] [FlutterRun] Spawning /Users/dantup/Dev/Google/flutter/bin/flutter with args ["run","--machine","-t","lib/main.dart","-d","ZY2224Z9N9","--start-paused"]
[14:25:44 GMT+0100 (BST)] [FlutterRun] ..  in /Users/dantup/Dev/Google/flutter/examples/stocks
[14:25:46 GMT+0100 (BST)] [FlutterRun] <== [{"event":"daemon.connected","params":{"version":"0.3.0","pid":1650}}]
[14:25:46 GMT+0100 (BST)] [FlutterRun] <== [{"event":"app.start","params":{"appId":"4e00e576-9cb0-4b30-8a57-9b19880aa590","deviceId":"ZY2224Z9N9","directory":"/Users/dantup/Dev/Google/flutter/examples/stocks","supportsRestart":true}}]
[14:25:46 GMT+0100 (BST)] [FlutterRun] <== Launching lib/main.dart on XT1562 in debug mode...
[14:25:46 GMT+0100 (BST)] [FlutterRun] <== [{"event":"app.progress","params":{"appId":"4e00e576-9cb0-4b30-8a57-9b19880aa590","id":"0","progressId":null,"message":"Initializing gradle..."}}]
[14:25:48 GMT+0100 (BST)] [FlutterRun] <== [{"event":"app.progress","params":{"appId":"4e00e576-9cb0-4b30-8a57-9b19880aa590","id":"0","progressId":null,"finished":true}}]
[14:25:48 GMT+0100 (BST)] [FlutterRun] <== [{"event":"app.progress","params":{"appId":"4e00e576-9cb0-4b30-8a57-9b19880aa590","id":"1","progressId":null,"message":"Resolving dependencies..."}}]
[14:25:56 GMT+0100 (BST)] [FlutterRun] <== [{"event":"app.progress","params":{"appId":"4e00e576-9cb0-4b30-8a57-9b19880aa590","id":"1","progressId":null,"finished":true}}]
Thu Jun 21 2018 [14:26:31 GMT+0100 (BST)] Log file ended

However I don't think it's isolated to --machine.. I left this for many minutes:

@devoncarew The errors above come seem to come from AppDomain in  daemon.dart, where we have this in handleCommand:
https://github.com/flutter/flutter/blob/a18f5e84a1045551774a6c12fd40fa4997356931/packages/flutter_tools/lib/src/commands/daemon.dart#L193L195
This is catching errors and sending them back the the client, but not terminating. Does it makes sense to terminate here? Maybe in the flutter daemon we shouldn't, but for flutter run it seems like an error should probably be fatal?

@devoncarew Ok, I think I finally got to the bottom of why this is happening.. adb outputs message that look exactly like the ones in flutter_tools (device 'x' not found.); these messages are coming from there. We have many places that call adb and print its stdout/stderr.
In the case I was testing, this is occurring in this code:
https://github.com/flutter/flutter/blob/0fb209726f7c87da7079981fd8597315725f0240/packages/flutter_tools/lib/src/android/android_device.dart#L386-L387
It looks like as part of the build we try to stop the app (presumably in case it's already running). However, this line does not inspect the return value of stopApp (which is false in the case of an error) and just continues. If adb fails with "device not found" it just carries on anyway.
We could bail out here if it returns an error, however, since we're running this speculatively it probably fails normally a lot (because the app isn't running).
The thing that runs next is isAppInstalled which pretends the app is not installed if there are any errors:
https://github.com/flutter/flutter/blob/0fb209726f7c87da7079981fd8597315725f0240/packages/flutter_tools/lib/src/android/android_device.dart#L258-L261
Possibly this should bail out (throwToolExit?) in the case of an error; however I'm not sure if it's ever normal for this check to fail and we'd want to continue. I can try it and do some testing here, but I don't know if I can cover enough cases to be sure we're not going to start quitting on something we shouldn't.
LMK what you think is best.

@devoncarew @gspencergoog Trying to clean up my list of issues - any opinions on the last comment above?

Can we check and see if the app is running before trying to stop it?

I don't know if that'll help - if I'm remembering/reading correctly, the issue isn't that stopApp fails, it's that when it does, we just ignore that and carry on anyway. If we skipped over stop, we'd just be doing the same thing - continuing on and assuming the app is running.
I think we need to detect that the app failed to install/launch and abort (ultimately terminating with a message that we failed to install).

@DanTup is this still relevant?

I think so, it doesn't look like the code around here has changed.


Add multi-file coverage parsing

Merging cobertura XML is hard - there isn't a utility to do it.  Unfortunately, while working in a repo of mine, I had to use two different test harnesses (one for python, one for JS - see https://github.com/spresse1/remote_thermostat).  Both can output to xml, but have no other common formats.  The codacy API appears to only retain the last coverage report uploaded.
Based on this situation, I decided it would be best to give the python uploader the ability to merge multiple XML input files.
This implementation merges the python structure that is turned into JSON before uploading.  This means that if, in the future, the uploader adds additional supported input formats, this code can be used to perform merges over multiple formats.
This PR is fully backwards compatible (at least according to the tests).  I've also added tests for the new functionality.
Comments and requests for revision are absolutely welcome.

Hi @spresse1,
Codacy should allow you to send one coverage report by commit and by language. Meaning that you should be able to send us a coverage report for JS and another for python, and Codacy would take care of merging the coverage information.
If you think that is not happening, ping us on the support chat (as soon as you login you will see on the bottom right a help button), so that we can check what's happening please.

Hi @spresse1
Were you able to check my previous suggestion regarding the upload of one coverage report by language?


 should come before TOC in PDF output

Converted from SourceForge issue 1942252, submitted by hmichaud
In bookmap, if you insert content (for example, a copyright page) in the  element and generate a PDF, it comes after the TOC in the output, even if in the map it is placed before the  element. Typically in print formats such information comes before the TOC and is not listed in the Contents. If content is inserted before the  in the map, it should come before the contents page in the output.

Hi, can you please share the version of dita in which this issue was fixed?


Test each type for having the expected default operations

It is possible to write a test helper that checks for the type to have copy constructor, move constructor, and so on.
Then, for each type, we can have one extra unit test that checks whether a type has or hasn't copy constructor as expected and so on.
One way is to check it semantically, but then I think we can't test for move/vs copy and so on. But the benefit is that we can write a test that verifies that copying and assignment works exactly as we expect. Plus there's an argument that we really care about semantics and not the actual way it is implemented.
But it is not possible to generalize copy constructor and assignment verfication, for each class such testcases would need to be written individually.
Second approach - and we can combine both approaches at the same time - second approach, would be to write a templated test helper to which you tell what operations you expect the type to have, and it will verify that your expectations are true.
We can do so without a test helper, and just repeat most of the statements for each class, but still the sense is the same - to use type_traits.
Check out what type_traits have: https://en.cppreference.com/w/cpp/header/type_traits

is_constructible

is_trivially_constructible

is_nothrow_constructible

is_default_constructible

is_trivially_default_constructible

is_nothrow_default_constructible

is_copy_constructible

is_trivially_copy_constructible

is_nothrow_copy_constructible

is_move_constructible

is_trivially_move_constructible

is_nothrow_move_constructible

is_assignable

is_trivially_assignable

is_nothrow_assignable

is_copy_assignable

is_trivially_copy_assignable

is_nothrow_copy_assignable

is_move_assignable

is_trivially_move_assignable

is_nothrow_move_assignable

is_destructible

is_trivially_destructible

is_nothrow_destructible

has_virtual_destructor

is_swappable_with

is_swappable

is_nothrow_swappable_with

is_nothrow_swappable

BTW, I won't be surprised if some of our inheritable classes do currently have a default copy constructor and assignment, and we should =delete them to prevent slicing.

Hello, is this issue still available for assignment?

Hello, is this issue still available for assignment?

As far as I know, it is.

@tmrna, thank you for your interest!
@jnyfah has just recently made a PR #4627 to test it for a single type, but there is more to cover, plus to straighten out some methods to correctly detect things like "is destructible". If you want, I think there is a way to cooperate and cover the entire SDK and all its types, and there can be plenty of work for everyone. Let me know if I can help, maybe we should split this issue into a several smaller ones that are easier to close, divide, and assign: i.e.: "cover azure.core types", cover "azure.identity types", and so on.
All your contributions are appreciated, in the end we can make the issues to look right for the boundary of work that you did, it is not a problem to create as many of them as necessary, I only don't want you to do something only to find out that the very same work that was done by another contributor just days ago, as this must be disappointing, so please let me know you need my help to coordinate your efforts in that aspect; or you are free to come to that agreement without me, but I think we can make that all work.

Resolved by @jnyfah in #4627.
I will create a subsequent issue, marked with good-first-issue tag, to cover more SDK types. People are free to take as big chunk of work as they want - we'll create a sub-issue with corresponding smaller scope, and will mention contributors in the CHANGELOG.

Subsequent issue - #4657.


fix: #204

https://github.com/alibaba/designable/issues/204
Before submitting a pull request, please make sure the following is done...

[ ] Ensure the pull request title and commit message follow the Commit Specific in English.
[ ] Fork the repo and create your branch from main.
[ ] If you've added code that should be tested, add tests!
[ ] If you've changed APIs, update the documentation.
[ ] Ensure the test suite passes (npm test).
[ ] Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


isBetween with time zone is not correct

Describe the bug
I am trying to use the isBetween with a timezone but it returns the wrong output.
var time = moment.tz('Australia/Sydney');
var beforeTime = moment.tz('000500', 'HHmmss', 'Australia/Sydney');
var afterTime = moment.tz('200000', 'HHmmss', 'Australia/Sydney');

console.log("local time is " + time.format('HHmmss'));

if (time.isBetween(beforeTime, afterTime)) {
  console.log("is between");

} else {
  console.log("is not between");
}

I set the start hour at 12:05 AM and end time at 8:00 PM but it returns false when time is 2:50 AM
To Reproduce
go here: https://jsfiddle.net/4xurmogh/ for a demo
Expected behavior
Should show that time is between
Desktop (please complete the following information):
Chrome
Moment-specific environment
"Sat Mar 27 2021 11:49:07 GMT-0400 (Eastern Daylight Time)"
"2021-03-27, 11:49:07 a.m."
240
"2.29.1"

const timeSlot = moment(slot.format('HH:mm'))
const twoHoursFromNow = moment(moment.tz("Europe/Istanbul").add(2, "hours").format('HH:mm'))
const isBeforeTwoHoursFromNow = timeSlot.isBefore(twoHoursFromNow, "hours")
This is my workaround solution to bug. Format all the times then change them into moment again before compare.


python: Use installer script

Use installer script as main/python so there will not be ugly item in Uninstall Programs.

/verify

/verify

python-alpha link is 404

It's okay on my PC, which link do you mean?

I force-update all python* manifests and they are both well here.

This one:

Its download link doesn't contain a3, I'll fix them, thanks.

Done.

/verify


[Enum Request] - Table Reservation Entry - Field 11 "Source Subtype" ex #21263

Hello, I retry with my request.
I was told the the fields is integer but it's not. It's an option.
If Microsoft permit to extend Document Type for sales and purchase, I think that also this field, that represent the document type for reservation entry, must be an extendible enum or at least a pure integer.
DamianoCastagna commented 3 weeks ago
Please change the field into Enum so it can be extensible
I want to create a new document type extending the enum Sales Document Type with my values 50000,50001.
But when I use the reservation entry it's not possible to use these values because it's an option and not an integer
field(11; "Source Subtype"; Option)
{
Caption = 'Source Subtype';
OptionCaption = '0,1,2,3,4,5,6,7,8,9,10';
OptionMembers = "0","1","2","3","4","5","6","7","8","9","10";
}
@AlexanderYakunin
Contributor
AlexanderYakunin commented last week
Sorry, this field cannot be transformed to enum, it does not make any sense. We only use Integer value of this field.
@AlexanderYakunin AlexanderYakunin closed this as completed last week
@DamianoCastagna
Author
DamianoCastagna commented last week
Sorry but I don't understand.
I want to create a new document type extending the enum Sales Document Type with my values 50000,50001.
But when I use the reservation entry it's not possible to use these values because it's an option and not an integer
field(11; "Source Subtype"; Option)
{
Caption = 'Source Subtype';
OptionCaption = '0,1,2,3,4,5,6,7,8,9,10';
OptionMembers = "0","1","2","3","4","5","6","7","8","9","10";
}

Option is named Integer, you can assign Integer value to Option and Option value to Integer without any problem.
Please understand that this field cannot be enum, as it will have different values for different source tables. And this field is already Integer in some tables.

But I can't assign a value greater than 10

Why? You can assign any value, like "Source Subtype" = 50000;

I'll try to put the value value into the flow but if I remember well if the option is passed to a procedure with a value that is not a Optionmember value Business Central will trhow an error.
We'll start soon with this new app and I'll check.
Thank


Create one function that iterates over elements and returns edge dicts, node dicts, edge_dict idx and node dict idxs

There's a lot of redudancy in the package. Several functions iterate over elements, but this only has to be done once (preferably in updateElemets). The output can then be passed to all functions that need it.

related to #10. Might not be necessary anymore to do this, once #10 is solved?


Fix: ControlledInverter propagation fix

Fixes #2395
Makes ControlledInverter's propagation similar to TriState

Coverage remained the same at 82.319% when pulling a4ab8fbc7b45dd2da71066a6bbf156ca96e5d348 on gr455:simFix1 into d1c3e620884c2a49132ffddde03944875a1020cf on CircuitVerse:master.


KYLIN-3997: Add a health check job of Kylin

See: https://issues.apache.org/jira/browse/KYLIN-3997

Can one of the admins verify this patch?

Codecov Report

Merging #669 into master will decrease coverage by 0.05%.
The diff coverage is 2.94%.

@@             Coverage Diff              @@
##             master     #669      +/-   ##
============================================
- Coverage     25.78%   25.73%   -0.06%
  Complexity     6008     6008
============================================
  Files          1382     1384       +2
  Lines         82202    82374     +172
  Branches      11519    11550      +31
============================================
- Hits          21198    21197       -1
- Misses        58958    59129     +171
- Partials       2046     2048       +2

Impacted Files
Coverage Δ
Complexity Δ

...in/java/org/apache/kylin/cube/CubeDescManager.java
33.57% <0%> (-0.25%)
12 <0> (ø)

.../java/org/apache/kylin/common/KylinConfigBase.java
13.02% <0%> (-0.03%)
42 <0> (ø)

.../apache/kylin/metadata/model/DataModelManager.java
34.54% <0%> (-0.64%)
12 <0> (ø)

...a/org/apache/kylin/common/util/BufferedLogger.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../java/org/apache/kylin/common/util/SoutLogger.java
16.66% <0%> (-16.67%)
1 <0> (ø)

...org/apache/kylin/rest/job/KylinHealthCheckJob.java
0% <0%> (ø)
0 <0> (?)

...ava/org/apache/kylin/tool/KylinHealthCheckJob.java
0% <0%> (ø)
0 <0> (?)

...c/main/java/org/apache/kylin/cube/CubeManager.java
37.77% <100%> (-0.26%)
44 <1> (ø)

...che/kylin/metadata/cachesync/CachedCrudAssist.java
56.98% <60%> (+0.17%)
15 <1> (ø)
:arrow_down:

.../apache/kylin/cube/cuboid/TreeCuboidScheduler.java
63.84% <0%> (-2.31%)
0% <0%> (ø)

... and 6 more

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 4565

5 of 170 (2.94%)  changed or added relevant lines in 9 files are covered.
224 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.06%) to 28.222%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
0
1
0.0%

core-cube/src/main/java/org/apache/kylin/cube/CubeDescManager.java
0
1
0.0%

core-metadata/src/main/java/org/apache/kylin/metadata/cachesync/CachedCrudAssist.java
3
5
60.0%

core-metadata/src/main/java/org/apache/kylin/metadata/model/DataModelManager.java
0
2
0.0%

core-common/src/main/java/org/apache/kylin/common/util/BufferedLogger.java
0
3
0.0%

core-common/src/main/java/org/apache/kylin/common/util/SoutLogger.java
0
3
0.0%

tool/src/main/java/org/apache/kylin/tool/KylinHealthCheckJob.java
0
4
0.0%

server-base/src/main/java/org/apache/kylin/rest/job/KylinHealthCheckJob.java
0
149
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
1
78.12%

core-metadata/src/main/java/org/apache/kylin/metadata/cachesync/CachedCrudAssist.java
1
64.52%

core-cube/src/main/java/org/apache/kylin/cube/cuboid/TreeCuboidScheduler.java
2
68.46%

core-cube/src/main/java/org/apache/kylin/cube/CubeManager.java
220
42.6%

Totals

Change from base Build 4562:
-0.06%

Covered Lines:
23248

Relevant Lines:
82374

💛  - Coveralls

I think it is a good feature, by this feature, Kylin's administrator needn't check metadata error in kylin.log or find useless cube manually. Some value such as ExpansionRate or Out of date should be configurable.

Thanks for @hit-lacus 's review. I will update the pr asap.

Format the code and compress the commits. Please help review the pr one more time, thanks~
@hit-lacus


localError should take precedence over globalError

Describe the Bug
localError should take precedence over globalError, I use the custom translate module can not change the i18n language

Expected Behavior
localError should take precedence over globalError, should show chinese error label
https://github.com/bpmn-io/properties-panel/blob/c34fba48f145714a5f5729b60a22ed26aa6dbe3e/src/components/entries/TextField.js#L138
Environment

Host (Browser/Node version): Edge 18, Chrome 69
OS: Windows11
Library version: 3.21.0

Hi,
From the issue, I don't understand why we should implement your suggestion. The global error (better called "external error") precedence is required to support errors provided in https://github.com/camunda/linting/. In that case, we find it more relevant for the user that some property is not supported at all than that the value of the unsupported property would not have been valid if it had been supported.
If there's some problem in the translation labels that prevents i18n, please share it in another issue.


Add lyric_finder lib crate

Implemented the lyric_finder crate that retrieves a song's lyric using https://genius.com and its APIs

Is lyric_finder still working with its aggressive cloudflare captcha policy? I can't seem to be able to get the example working.

A possible solution - multiple API used by https://github.com/spicetify/spicetify-cli/tree/master/CustomApps/lyrics-plus

@slikie I never have a trouble with scrapping lyric from genius.com. Which command did you use to run the example? I use
cargo run --example lyric-finder -- 'run taylor swift'

I'm not familiar with the "aggressive cloudflare captcha policy" you're referring to.

same as in the log -
30:2023-03-04T12:08:13.305693Z ERROR client_request{request=GetLyric { track: "Kill V. Maim", artists: "Grimes" }}: spotify_player::client::handlers: Failed to handle client request: failed to get lyric for track Kill V. Maim - artists Grimes: error decoding response body: expected value at line 1 column 1: expected value at line 1 column 1
I have given up using genius with this popping up all the time.
curling genius.com having similar result.

I see what you mean. Will take a look into the other solution you send. Can you create an issue for tracking?


make build run dvc live tests

As a build step, we could clone dvc repo and run live integration tests.
It might be worth considering to move all integration tests to dvclive
Related to #28

I've sent a P.R. enabling dvclive integration tests on dvc C.I. I think that should be enough, not sure what benefits would have to move those tests to dvclive C.I.


Bring to the front

Bring to the front
How do I leave the component above all other layers?
Observe the illustration below. The vue-multiselect should, in this case, stand on top of all other screen elements. In fact, it should always stay on top of the other screen elements. I think this should be a standard match.
How do you solve this?
Thanks a lot for the help.

Well, this isn’t a problem related to staying above other elements. It happens because your container styles have the overflow property set to auto. You could probably change it to visible and it should work then :) Without showing the scrollbar.

Okay, but I want multiselect to be over the footer of the modal component of Bulma css.

Then do the change that I suggested. Otherwise, it won’t be possible.
It’s a problem of the container – you can’t get outside of it with anything you have inside. Not just multiselect.

<template lang="html">
  <transition name="fade">

          <form>

                Cidade

                <dm-form-select @event="getValueField" field-name="city"></dm-form-select>

        </form>

  </transition>
</template>

I would say try changing this class modal-card-body.

i tried:

  .modal-card {
    overflow: auto !important;
  }

  .modal-card {
    overflow: auto !important;
    z-index: 1 !important;
  }

  .modal-card-body {
    overflow: auto !important;
  }

  .modal-card-foot {
    overflow: auto !important;
  }

But I was not successful.

You have to change it to visible instead of auto.

Yeah! It's work!
  .custom {
    width: 80% !important;
    overflow: visible !important;
  }
  .modal-card-body {
    overflow: visible !important;
  }

Thanks a lot for the help.

Happy to help! :)

Looks great! 👍

Returning...
The problem using "overflow: visible;" In a Bulma modal is that I can no longer scroll when the form is larger than the screen layout.
What I effectively need is that scrolling within the modal works and that "vue-multiselect" is over all elements.
Look at the illustrations below:

IMHO this is not a Bulma Modal scrolling problem. This will happen with all scrollable containers. Wouldn't it be preferable to position the dropdown list absolute, given this would mimics the behaviour of a native select? codepen.

The element is positioned absolutely within the multiselect container. What you probably mean is positioning it absolutely within the whole document, which is a bit harder to accomplish.

Hi there,
I ran into the same issue, and it's definitely a problem of the absolute positioned list, rather than the overflow of a parent container. As the multiselect wants to "imitate" and extend a html select element, which is overlapping any overflow (it even reaches out over the Browser), there needs to be a fix.
A solution I was working on is to set the list to position: fixed; z-index: n; with a calculated position to an element that acts as – as I call it – anchor. I build a little helper:
const calculatePosition = function calculatePosition(anchor, element) {
    const anchorBounding = anchor.getBoundingClientRect();
    const height = window.innerHeight;
    const general = ` width: ${anchorBounding.width}px; left: ${anchorBounding.left}px;`;
    let pos = `top: ${anchorBounding.bottom}px;` + general;
    if (height - anchorBounding.bottom < 200 && height - anchorBounding.top > height - anchorBounding.bottom) {
        pos = `top: inherit; bottom: ${height - anchorBounding.top}px;` + general;
    }

    element.setAttribute('style', pos);
};

with a method:
recalulateList() {
    calculatePosition(this.$el.querySelector('input'), this.$el.querySelector('ul'));
}

Triggering on start and, on scroll, to recalculate the position, keeping the element attached.
Hope this helps finding a solution.
Regards,
Manuel

I found very interesting example with Popper.js, but it's uses old version of vue-multiselect.
https://codesandbox.io/s/0x608nxo10

Why is this closed? I have the same issue in conjunction with a fixed header.

The v3, once it finally releases... allows for the user to provide their own implementation of the list (or the built-in but wrapped in whatever you want). You could then use popper or any other solution for that. As for 2.x, I sadly don’t have the time to work on this and I don’t want to introduce any dependencies.
If you want to help, feel free to dig into the v3 branch, look for regressions compared the current version / see if what’s there solves this problem.

I am facing this in tailwind Dialog. How would i fix this?

Same problem in bootstrap modal, any solution ?


add sum so counters are displayed properly

When we view a counter, we get back this kind of data:
  "values": [
    {
      "numPoints": 75,
      "sum": 25,
      "timestamp":<PHONE_NUMBER>000
    },
    ...
   ]

The 'numPoints' and 'sum' attributes used to show the same exact value. However, for this PR:
https://github.com/rackerlabs/blueflood/pull/591
the semantic of 'numPoints' was changed to return sample count.
For displaying Counters in Grafana, people are mostly going to be interested in graphing the sum, not sample count.
This PR fixes the finder so that it picks up sum first before numPoints. This should be fine for other types of StatsD metrics. According to this:
https://one.rackspace.com/pages/viewpage.action?title=Sub+Metrics+or+Stats&spaceKey=cloudmetrics#SubMetricsorStats-BluefloodSubMetricsSupport
there is only one other metric, Timer, for which Blueflood returns sum. And for Timer, the average is going to take precedence over sum.

@izrik , @ChandraAddala , this PR is ready for review.
I had to comment out the test test_finder because this test makes calls to an external blueflood system and most likely will require credentials. We need to rework this as Travis recently changed the behavior of encrypted/secure environment variables. They no longer support accessing this encrypted/secure environment variables from a fork.
https://docs.travis-ci.com/user/environment-variables#Defining-encrypted-variables-in-.travis.yml
So we can not pass AUTH_API_KEY from Repository settings or encrypted in .travis.yml. We have to change the way we test this.


Publish to Maven Central

Anyway we can get this library published to Maven Central?

@filosganga This would be highly appreciated. If I can contribute please let me know.

With Bintray being EOL, is this being looked at again?

Hi @mrooding the library is still being published to an ovotech public repository if that helps, see the updated Readme.

Hi @joedarby I completely missed that update. Last I recall it was still being published to bintray. Thank you!


dev-timeseries-ontop

Test integration of time series data with ontop

Experimentation so far reveals that this only works with a small dataset, when there are multiple timesteps, ontop tries to create lots of tables and crashes