Error: gyp failed with exit code: 1

Hi,
I would like to try your package but I encountered an configure error when I try to install him.
gyp ERR! stack Error: 'gyp' failed with exit code: 1
The cli commands work but not the projects generated. When the server is started, he doesn't find her dependencies.
Error: Cannot find module 'gl-vec2' from '/Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/util'
I tried some things without success :

npm install manually into the penplot package.
use penplot into a local npm project and try to install the missing dependencies.
following this node-gyp issue #809, remove the .node-gyp/ directory.
...

node version
7.4.0
npm version
5.2.0
Complete stack trace :
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
WARN registry Unexpected warning for https://registry.npmjs.org/: Miscellaneous Warning EINTEGRITY: sha1-UWbihkV/AzBgZL5Ul+jbsMPTIIM= integrity checksum failed when using sha1: wanted sha1-UWbihkV/AzBgZL5Ul+jbsMPTIIM= but got sha512-yJHVQEhyqPLUTgt9B83PXu6W3rx4MvvHvSUvToogpwoGDOUQ+yDrR0HRot+yOCdCO7u4hX3pWft6kWBBcqh0UA==. (11423 bytes)
WARN registry Using stale package data from https://registry.npmjs.org/ due to a request error during revalidation.
/Users/jboulay/.nvm/versions/node/v7.4.0/bin/penplot -> /Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/bin/index.js

><EMAIL_ADDRESS>install /Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/node_modules/canvas
> node-gyp rebuild

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
./util/has_lib.sh: line 31: pkg-config: command not found
gyp: Call to './util/has_lib.sh freetype' returned exit status 0 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:336:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Darwin 15.6.0
gyp ERR! command "/Users/jboulay/.nvm/versions/node/v7.4.0/bin/node" "/Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/node_modules/canvas
gyp ERR! node -v v7.4.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok

><EMAIL_ADDRESS>install /Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/node_modules/fsevents
> node install

[fsevents] Success: "/Users/jboulay/.nvm/versions/node/v7.4.0/lib/node_modules/penplot/node_modules/fsevents/lib/binding/Release/node-v51-darwin-x64/fse.node" is installed via remote
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><=4.x but none was installed.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/penplot/node_modules/canvas):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1

+<EMAIL_ADDRESS>added 641 packages in 14.443s

Same issue in 1.1.2.
node 7.1.0
npm 5.3.0
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue

><EMAIL_ADDRESS>install /Users/walpole/DEV/genart/src/node_modules/fsevents
> node install

[fsevents] Success: "/Users/walpole/DEV/genart/src/node_modules/fsevents/lib/binding/Release/node-v51-darwin-x64/fse.node" is installed via remote

><EMAIL_ADDRESS>install /Users/walpole/DEV/genart/src/node_modules/canvas
> node-gyp rebuild

gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
gyp WARN download NVM_NODEJS_ORG_MIRROR is deprecated and will be removed in node-gyp v4, please use NODEJS_ORG_MIRROR
./util/has_lib.sh: line 31: pkg-config: command not found
gyp: Call to './util/has_lib.sh freetype' returned exit status 0 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/Users/walpole/.nvm/versions/node/v7.1.0/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:336:16)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Darwin 14.5.0
gyp ERR! command "/Users/walpole/.nvm/versions/node/v7.1.0/bin/node" "/Users/walpole/.nvm/versions/node/v7.1.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/walpole/DEV/genart/src/node_modules/canvas
gyp ERR! node -v v7.1.0
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS><=4.x but none was installed.
npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/canvas):
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm WARN optional SKIPPING OPTIONAL DEPENDENCY: Exit status 1

+<EMAIL_ADDRESS>added 640 packages in 22.419s

By the looks of things, you're missing the dependencies for canvas (node-canvas).
Take a look at the node-canvas repo for dependency installation instructions per your OS node-canvas as the extent of what is needed can vary greatly.
Also appears you both may be missing a global install of browserify.

You are right,
After following the installation of node-canvas penplot works! Even if I have some depreciated packages and some other errors.
Thank you :)

I’ve fixed some issues regarding gl-vec2. You shouldn’t need globally installed browserify for this module, and it should still work even if the node-canvas install fails (it’s an optional dependency only necessary if you want to generate prints without a browser). For those with node-canvas errors, does penplot not work at all after installing?
Following the node-canvas instructions in their readme will probably help clear up those errors.
Sent from my iPhone

On Aug 14, 2017, at 9:05 AM, Jérémie Boulay<EMAIL_ADDRESS>wrote:
You are right,
After following the installation of node-canvas penplot works! Even if I have some depreciated packages and some other errors.
Thank you :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I got the node-canvas dependencies to install, but node-canvas itself presents many errors when trying to install. However, I went from only being able to run the starter code, to now having the examples work except for the generative-paint.js example because of the missing color-converter module.


Syntax Checker errors with "missing owner" on valid CODEOWNERS

Description
It's totally valid to list a directory/file/pattern and leave off an owner as a way of specifying "no owner".
See the docs: https://github.com/github/docs/blob/209c340039acc4c2f73c2eb1fcd607af00911118/content/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners.md#L103
# In this example, @octocat owns any file in the `/apps`
# directory in the root of your repository except for the `/apps/github`
# subdirectory, as its owners are left empty.
/apps/ @octocat
/apps/github

Notice the last line has no owner.
Expected result
No error when leaving off owner.
Actual result
Error :disappointed:
Steps to reproduce
Run the tool on a CODEOWNERS file which leaves off the owner
Troubleshooting

Hi @joshua-cannon-techlabs
Thanks for reporting this issue. I will fix that. Do you maybe know the use-case for that?
Because here is also a valid issue https://github.com/mszostok/codeowners-validator/issues/70 which mention that the validator should inform you about valid but problematic part of your CODEOWNERS file.
In this case such thing IMO shouldn't be an error as it's valid from the CODEOWNERS point on view but at least be it should be displayed as warning. WDYT?

(FWIW I would fix it and make a PR but we're still working with legal on open source contributions :sob: )
I think it all depends on your policies. If you don't have the strict owners policy than you can still merge changes to /apps/github.
If this kind of validation (and #70) was an optional check I can opt into or out of then thats good. But putting them in syntax check is wrong IMO as the file is syntactically correct.

Actually, I might be able to take a stab at this. Good excuse to learn some Go

We have a use case for this.  In our repo, we have one file which does not need a code owner (anyone with access to the repo should be able to review/approve it.) However, the rest of the files in the directory in which it is in should be owned by a specific team.
The file to be unowned is a configuration file for the oncall-rotator.  The other files in the directory are an app that uses the configuration file.
/infra/oncall-rotator/                                  @sre-team
/infra/oncall-rotator/oncall-config.yml

I have the implementation ready to go, but it'll need to be reviewed internally which could take time.
The implementation itself is quite simple, as we're just removing code so I wouldn't be brokenhearted if someone else beats me to this and makes a PR.

We have a similar use-case in our repo - of wanting to have an explicitly unowned file in an otherwise owned directory.
It would definitely be encouraging if codeowners-validator can handle this directly - so we can avoid other compromises (intentionally changing our file location/structure solely to allow codeowners-validator to pass).

I have a PR open that addresses this issue: https://github.com/mszostok/codeowners-validator/pull/101
@mszostok If you could take a look when you have some time!

Sorry guys!
Currently, I'm quite busy, and this project was developed 100% in the spare time, which I almost don't have right now.
The mentioned PR is quite problematic for me as it's only removing a given check instead of making it optional, so maybe I will adjust it a bit, so you can set a given env variable to disable strict checking. Configuring the codeowners  validator in a more readable/customizable way will be handled by https://github.com/mszostok/codeowners-validator/issues/73. Additionally, I need to check whether this is still valid https://github.com/mszostok/codeowners-validator/pull/78 as it already landed on main branch.
Once again, sorry for such long waiting time. I also respect your time that you spent on it, so I will reserve some time this weekend to take care of this and do the release.
FYI: I plan to do the 1.0.x release when I will be able to address all reported bugs + https://github.com/mszostok/codeowners-validator/issues/73 which will change the way how validator is configured and https://github.com/mszostok/codeowners-validator/issues/50. Additionally, I would like to add an option to format CODEOWNERS files and an option to add/remove a given owner and create PRs automatically e.g. with such (draft) config:
updateRepositories:
  - path: "/tmp/cloned/repository"
  - url: "capactio/capact"
  - path: "capactio/website"

insert:
  - owners: [ "@hakuna", "@matata" ]
    condition:
      pattern: "*.go"
  - owners: [ "@baz", "@bar" ]
    condition:
      associatedWithOwner: "@ghost"

pullRequest:
  titleFormat: "Add {{ .NewOwner }} as a code owner in {{ .RepoName }}"

so maybe this project will be changed to sth like codeowners-manager than codeowners-checker.

Hi all!
I fixed it in a way that the new release v0.7.0 won't report missing owner as an error anymore 🙂
Nevertheless, I wanted to keep it, so I moved this check under owner checker and made it optional. As a result, validator may work in a picky mode when needed, see new option:

Name
Default
Description

OWNER_CHECKER_ALLOW_UNOWNED_PATTERNS
true
Specifies whether CODEOWNERS may have unowned files. For example:   /infra/oncall-rotator/                    @sre-team  /infra/oncall-rotator/oncall-config.yml    The /infra/oncall-rotator/oncall-config.yml file is not owned by anyone.

Additionally, I changed it from error to warning:
==> Executing Valid Owner Checker (1.2s)
    [war] line 23: Missing owner, at least one owner is required

To enable it back, set:
      - name: GitHub CODEOWNERS Validator
        uses<EMAIL_ADDRESS>        with:
          owner_checker_allow_unowned_patterns: "false"

Additionally, you can change the level on which the application should treat reported issues as failures. By default, it's set to warning, which treats both errors and warnings as failures, but as the missing owner is reported as warn you can also ignore it:
      - name: GitHub CODEOWNERS Validator
        uses<EMAIL_ADDRESS>        with:
          owner_checker_allow_unowned_patterns: "false"
          check_failure_level: "error"

In this case, you will see it in logs, but the action will exit with 0 status code.


Look in to adding external RAM for holding flow arrays

As we build up flow scripts we may want to store arrays for the scripts in non volatile memory. Can we add more non-volatile memory to the ESP32? Find out what memory we can add that will do this.

Another alternative might be adding a USB interface to a thumb drive. That would let us pre-load the storage device from a PC, and avoid the downloading process, or the need to include scripts in compiled code.


GROOVY-9465: groovydoc: show class declaration header for all types

It seems a mistake that the template required that the type should implement at least one interface before showing the declaration.

Good catch. Merged. Thanks!


Link to  Cloud collection in the README is broken

The Link mentioned in the Readme file for the Cloud  collection is incorrect.
Link mentioned is for file Xray Cloud GraphQLAPI.postman_collection
The correct file name is   Xray_Cloud_GraphQL_API.postman_collection.json

thanks for reporting @i-ivd . fixed


check if version works

To @dpiscia, in order to set the first milestone it should be checked if everything works fine

Done, it seems to work


TextField widget is using a high cpu on iOS compared to native one

While TextField is focused and soft keyboard is up in iOS simulator the backboardd daemon
shows using a high cpu in activity monitor. I've compared the behaviour with native iOS:
In native TextField when I stopped typing the backboardd daemon cpu usage decreases a lot but in flutter cpu usage remains very high even when I was not typing anything.

Are there any updates on this? It's such a significant problem that when developing on my MacBook Pro the battery life is cut in half while I'm developing/testing apps with TextFields. The TextField(s) being displayed uses ~100% CPU, and having one in focus uses over 200% on a 2018 MacBook Pro 15" with the 2.6 GHz i7, which is insane...

I have the same problem, when I run some app with TextField, CPU of my MacBook Pro 13 2017 go 100% usage

I have the same problem on my MacBook Pro 2014.

Same here

Same issue here. Dev Tools performance overlay shows continuous cpu usage as well while the soft keyboard is up.

Can you please confirm if this issue also happens on real iOS devices, or if this is simulator-only? CC @GaryQian to see if there's any quick thought as I believe Gary has looked into some text edit performance issues recently.

I don't have iOS device please somebody test this on device

https://github.com/flutter/flutter/pull/46720 This may help with the performance. The caching of the caret was incorrect and was being recomputed.

Can confirm the same. When in focus, CPU usage gets around 50%

We have recently landed #46720 which address one cause of abnormal CPU usage.
Can someone confirm whether this change has helped/if the CPU usage is still high. The change is available on master/dev channel as of now.

@monajafi can you confirm that this still happens on the master channel? We'd like to know where #46720 fixes the problem. Thanks.

I'm still having high CUP when TextField is showing.
flutter doctor
[✓] Flutter (Channel stable, v1.12.13+hotfix.7, on Mac OS X 10.15.2 19C57,
    locale en-CN)

@zhjuncai : the stable channel Flutter (v1.12.13+hotfix.7) doesn't have the fix (https://github.com/flutter/flutter/pull/46720) yet. Can you try to switch to the master channel, and see if that fixes the high CPU usage? If not, can you please attach flutter doctor again?

How is this still a thing? It's really annoying. My laptop is draining battery like crazy when working on forms.
Please fix this


ERROR ITMS-90054: "This bundle is invalid. The bundle identifier cannot be changed from the previous version.

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
We have a build pipeline for several iOS apps. It works fine. In fact, it has worked for this same iOS app previously. It's the pilot command that crashes:

Command executed

pilot(
  skip_submission: true,
  skip_waiting_for_build_processing: true,
  team_id: ENV["ITC_ID"],
  apple_id: ENV["APPLE_ID"]
)

Complete output when running fastlane, including the stack trace and command used

Expected behaviour is that uploads a build to the app store. Again. I have checked the bundle id, its consistent everywhere. It is now consistently crashing. See image. Same pipeline for other applications doesn't crash
Environment

I cannot ssh to github actions

it was my mistake


Create a main executable, with --pipe option

the pipe option will disable logging on standard out (move to stderr?) and prints BlockExtra serialized blocks into stdout, allowing Unix style composition

implemented in https://github.com/RCasatta/blocks_iterator/pull/24


Candy Crush, Cooking Fever, etc keep reappearing

As of Monday 8/6/18 I keep having consistent issues of Candy Crush and other bloat games not being removed. This wasn't a problem previously. It stays in the start menu, I right click and uninstall it, and it goes away until the next mouse click.
Just so you know, I'm doing everything in this order

Clean install Windows 10 1803 July .iso
Windows updates
Firmware/bios/drivers/etc
Software installs (chrome, firefox, ms office, java, silverlight, adobe reader)
this script

What are your thoughts?

Duplicate of #101


App 'pinax.notifications' could not be found. Is it in INSTALLED_APPS?

"Why, yes it is!"  So, exactly why can't manage.py find it?
INSTALLED_APPS = [
[...]
    'pinax.notifications',

@SundialServices you'll have to provide a bit more information. This sounds like a project configuration issue. Perhaps you can show your settings.py INSTALLED_APPS?

I found that by changing the INSTALLED_APPS entry to pinax, instead of pinax.notifications as the documentation states, solved the problem.  makemigrations then found all of the migrations, for the notifications component and otherwise.
I'm using Django 1.19 for other unrelated reasons, but I don't think that matters.
I'm closing the ticket because a solution or work-around has been found, but could this be a (wide-spread) documentation error?


Revert "Updated createNewFormat function for Parse and Format library."

Reverts xdan/datetimepicker#218

Hi. Do you mind explaining the issue with the 218 patch?
The reason for the pull was if the funtion was called with "" as the format string, it attempts to eval "retu;" which is invalid.

Hi.  Sorry. Can you resend your pull request again?

Shouldn't there be a revert for the revert?


Added a new darker theme

Thank you so much for your suggestion, but I am sorry I won't add a new version.
Sequoia has only these two versions and that was the idea from the beginning, but feel free to keep it for yourself.


[SPARK-25680][SQL] SQL execution listener shouldn't happen on execution thread

What changes were proposed in this pull request?
The SQL execution listener framework was created from scratch(see https://github.com/apache/spark/pull/9078). It didn't leverage what we already have in the spark listener framework, and one major problem is, the listener runs on the spark execution thread, which means a bad listener can block spark's query processing.
This PR re-implements the SQL execution listener framework. Now ExecutionListenerManager is just a normal spark listener, which watches the SparkListenerSQLExecutionEnd events and post events to the
SQL execution listeners.
How was this patch tested?
existing tests.

cc @gatorsmile @brkyvz

Test build #97122 has started for PR 22674 at commit 1701f3b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3799/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3800/
Test PASSed.

Test build #97123 has started for PR 22674 at commit 28f64d0.

Test build #97122 has finished for PR 22674 at commit 1701f3b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #97123 has finished for PR 22674 at commit 28f64d0.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3810/
Test PASSed.

Test build #97140 has finished for PR 22674 at commit a456226.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3813/
Test PASSed.

Test build #97145 has started for PR 22674 at commit 436197b.

Test build #97145 has finished for PR 22674 at commit 436197b.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3822/
Test PASSed.

Test build #97159 has started for PR 22674 at commit 436197b.

Test build #97159 has finished for PR 22674 at commit 436197b.

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

Merged build finished. Test PASSed.

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

Test build #97161 has finished for PR 22674 at commit 642ddd3.

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

Merged build finished. Test FAILed.

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

Test build #97200 has started for PR 22674 at commit a25524b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3852/
Test PASSed.

Test build #97200 has finished for PR 22674 at commit a25524b.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3853/
Test PASSed.

Test build #97203 has started for PR 22674 at commit 3ffa536.

Test build #97203 has finished for PR 22674 at commit 3ffa536.

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

Merged build finished. Test FAILed.

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

retest this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3869/
Test PASSed.

Merged build finished. Test PASSed.

Test build #97231 has started for PR 22674 at commit 3ffa536.

Test build #97231 has finished for PR 22674 at commit 3ffa536.

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

Merged build finished. Test FAILed.

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

hmm, seems it failed at the same test.

I couldn't reproduce it locally, let me try again

retest this please

Test build #97251 has started for PR 22674 at commit 3ffa536.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3880/
Test PASSed.

I would just up the timeout in that suite. Now that we're pushing a bunch more stuff to the LiveListenerBus, it may not be draining quickly enough. On slow jenkins' it could likely cause flakiness.

Test build #97251 has finished for PR 22674 at commit 3ffa536.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3891/
Test PASSed.

Test build #97269 has started for PR 22674 at commit b3e546d.

Test build #97277 has started for PR 22674 at commit 0bfc240.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3897/
Test PASSed.

Test build #97269 has finished for PR 22674 at commit b3e546d.

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

Merged build finished. Test PASSed.

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

Test build #97277 has finished for PR 22674 at commit 0bfc240.

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

Merged build finished. Test FAILed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3910/
Test PASSed.

Test build #97291 has started for PR 22674 at commit 6e3a345.

Test build #97291 has finished for PR 22674 at commit 6e3a345.

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

Merged build finished. Test PASSed.

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

Test build #97397 has started for PR 22674 at commit c42b499.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3989/
Test PASSed.

Test build #97397 has finished for PR 22674 at commit c42b499.

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

Merged build finished. Test PASSed.

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

LGTM, do you have any other concerns @hvanhovell @brkyvz @dongjoon-hyun ?

since there is no objection, I'm merging it to master, thanks!


ReadOnly and WriteOnly are getting ignored

Hi I have written a json schema which uses readonly and writeonly but the attribute are getting ignored by the schema. I noticed it while reading the properties filed of the schema and then the readonly field. Is it a error on my side? Could you present an exmaple with read only?
Cheers.

Compiler.ExtractAnnotations must be set to true to read those

Thank you! I dont know if it is documented anywhere but I think it should be for clarification. Is it wanted that a readOnly attribute can still be provided via json which means the validation is still true?

List of annotations https://json-schema.org/understanding-json-schema/reference/annotations

Annotation keywords has no impact on validation

Alright. Thank you. Currently I'm implementing an API and would like to validate the given input JSON data. For exmaple that an application is not allowed to overwrite the id or set the id. Any idea how to implement it using readOnly?

you can think of readOnly as javadoc in java program. the compiler ignores the java doc. whereas java doc tool used them to generate html documentation. similarly jsonschema validator ignores those keywords, where as some editors might use those as hints while editing json.
what you are trying to achieve is not possible using readOnly. But you can try the following:
following schema does not allow to set id:
{
    "properties": {
        "id": false
    }
}

following schema allows either id is not set or id has a specific value say 1234:
{
    "properties": {
        "id": {
            "constant": 1234
        }
    }
}


Abstract constracts isn't allowed?

Looks like doxity doesn't work with abstract notation?

And, it also doesnt with intereface


go install -v github.com/tofuutils/tenv/cmd/tenv@latest installing version 1

Describe the bug
Since you have tagged v2 version of the package, can you also please update go.mod file, with
module github.com/tofuutils/tenv/v2

To make it go install-able again.
To Reproduce
Steps to reproduce the behavior:
$ go install -v github.com/tofuutils/tenv/cmd/tenv@latest
go: downloading github.com/tofuutils/tenv v1.11.19

Expected behavior
go install -v github.com/tofuutils/tenv/cmd/tenv/v2@latest will install latest v2.
Environment (please complete the following information):

OS: any
tofuenv version: v2.x.x

https://github.com/tofuutils/tenv/pull/177

looks like that quick fix didn't help. Will check it later

Original command from the description had /v2 in the wrong place. Actual command looks like this:
$ go install -v github.com/tofuutils/tenv/v2/cmd/...@latest
go: downloading github.com/tofuutils/tenv/v2 v2.1.7
github.com/tofuutils/tenv/v2/pkg/reversecmp
github.com/tofuutils/tenv/v2/pkg/winbin


HTTP Client example not working

https://zigbyexample.github.io/http-client
Is not working since the request method needs headers and the method (e.g., GET)

sorry. being busy for a while. will fix everything in a few minutes


Unit Tests for Cloudtrail/Support modules in Baselines

User Story
As a MP Engineer
I need to write some unit tests for the Cloudtrail and Support sub-modules sin the Baselines module
So that I can increase the confidence in the modules
Value / Purpose
As a follow-on from https://github.com/ministryofjustice/modernisation-platform/issues/6490 this ticket is to add missing unit tests for some of the sub-modules in the MP Baselines module.
There are roughly 9 resources for unit testing.
Useful Contacts
@richgreen-moj
Additional Information
Follow-on from https://github.com/ministryofjustice/modernisation-platform/issues/6490
Proposal / Unknowns
No response
Definition of Done

[ ] Missing unit tests added
[ ]  New unit tests pass
[ ]  Another team member has reviewed

cloudtrail ✅ - write a follow-up story to test (8 resources) https://github.com/ministryofjustice/modernisation-platform/issues/7307
support ✅ - write a follow-up story to test (1 resource) https://github.com/ministryofjustice/modernisation-platform/issues/7307
These are the resources we need to test.

I'm picking up this one as I've not done much with unit tests & go before. If anyone wants to pair with me on it then do shout. :)

Branch containing the completed support module unit test - https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/tree/issue/7307

Added unit tests for the Support and CloudTrail modules.
https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/pull/547

just checked and the unit test is failing

Go issue resolved with PR https://github.com/ministryofjustice/modernisation-platform-terraform-baselines/pull/551

All looks good


CC module navigation

The README.org in the CC module mentions code navigation with irony, as far as i know, irony doesn't provide this feature.
Can you just briefly mention on how you are doing code navigation for C++. I personally had been using rtags till now, it works good, but i am not really happy with it, but you don't seem to be using it.
You seem to be using xref, but how are the tags getting built for it? I have never used it, so if you have used rtags how good is it comparatively?

I have a forked version of cc module with only rtags enabled
https://github.com/amosbird/abdoom/blob/master/cc/config.el
It works pretty well.

Cool, i will check it out, i am already on my way of doing it myself. Do you have any tips for speeding up rtags, especially some places where it pretty much takes 5-10 seconds for go-to-definition?
It is pretty much the best indexer that is available for emacs in terms of C++ support but damn, it's slow.

Yup, rtags can be quite CPU intensive. The tip is to get more hardware. I use rtags on an 40 cores machine with 256GB memory ;-)

Right now, xref in Doom uses etags and gnu global as backends, but they are less than stellar. They will ask you where the TAGs/global db file is if they can't find it on first invocation.
In fact, I rely on dumb-jump for most of my C/C++ code navigation needs (basically, set (setq xref-backend-functions '(t)) in C/C++ buffers and use +jump/definition and +jump/references). This is not shown in Doom, since I've considered this "experimental" for a while.
It's fast and reasonably accurate, minus the need to generate tags file or manage a daemon. But I understand if that isn't enough. I've considered rtags integration for a while.
I'll look into @amosbird's rtags implementation and see what i can do.

I've added rtag support to develop, loosely based off of @amosbird's work. It needs more testing, but it'll have to wait until after new years.

@amosbird What're your experiences with flycheck-rtags? Is it better than flycheck-irony? I haven't found any compelling reason to use the rtags backend over the irony one.

Hi, flycheck-rtags always shows the same errors as my project building system. I remember I didn't have much luck with flycheck-irony, and I never use auto-completion in c++. It's slow.

I remember I didn't have much luck with flycheck-irony

That's odd. Irony is configured to pick up on the same compile_commands.json file as rtags, so flycheck-irony ought to produce the same errors as your build system.


Nested menu drop downs not working in Roots 5.0

Hey,
I'm beginning development on a site and when I changed the theme use from 4.0 to 5.0, I noticed menu dropdown functionality stopped.
dev site is located at aboutaaron.com/dev
The dropdown indicator appears in the menu, but clicking it doesn't open the nested menu.
When I checked the console for errors I saw this:
Syntax error, unrecognized expression: /dev/sample-page/ (<-- this is name of the page link)
f.event={add:function(a,c,d,e,g){var h...hild(d);return this},empty:function()
http://ajax.googleapis.com/ajax/libs/jquery/1.7.1/jquery.min.js
Line 3
I'm not entirely sure why this is an issue, but any help into the matter would be great.

the bootstrap docs show the anchor for the top level of a dropdown as #
currently how i'm doing it over on http://demo.rootstheme.com/

Thanks for the speedy reply Ben.
And I see what you're saying.
So you cannot have the menu item clickable with content. It has to act as a dropdown only. My client wants to have the items in the navbar link to content. Would there be a way around that?

You can make the sub menu show on hover instead of click, it'd require some tweeks in the bootstrap css files though

@javiervd - do you the tweaks handy to make the sub menu show on hover instead of click. Much Appreciate if you can share your tweaks

@baritoneuk the parent menu element needs to have an href of ''#". So, in the WordPress admin, under Menus, the Primary Navigation Parent page would actually me a Custom Link with the value of #. The child elements underneath will then pop up when the parent element is clicked. The markup would more less look like this:

		My Parent Element

		    	Child 1

		    	Child 2

@aboutaaron That's not the way Roots worked in the past. I'd have thought you'd have the parent element as the parent page and each sub page as the dropdown. How else will you be able to get to the parent page? I'm really wanting to have the dropdown to appear on hover over. Isn't that how it's supposed to work out of the box?

@baritoneuk if you don't want to use the topbar like it's meant to be used in bootstrap then just don't use the navbar nav walker.
edit header.php to remove the navbar classes and remove the call to the custom walker

Thanks, @retlehs
Do you know why Bootstrap is implemented like this? I'm sure a lot of thought went into this way of doing it, but it's not making sense to me. Where should the parent page go in the list? Should it be at the top of the dropdown menu? I don't think  using # is semantic or good for accessibility.

@baritoneuk That was the "out-the-box" functionality with Roots, but per Bootstrap's style, it was removed. As @retlehs said, you can also just remove the Bootstrap Nav class (.nav-collapse) from header.php , remove the walker and then roll your own:

  <?php wp_nav_menu(array('theme_location' => 'primary_navigation', 'walker' => new Roots_Navbar_Nav_Walker())); ?>

to:

  <?php wp_nav_menu(array('theme_location' => 'primary_navigation')); ?>

As far as your comment about why Bootstrap choose this menu choice check https://github.com/twitter/bootstrap/issues/424 and https://github.com/twitter/bootstrap/issues/983.
If you want mouseover dropdowns, check http://stackoverflow.com/questions/8878033/how-to-make-twitter-bootstrap-menu-dropdown-on-hover-rather-than-click/8878666#8878666

Found a solution to this which worked for me, Terribly sorry that I forgotten the link to the very nice chap who solved it!
I have not tried this using Roots yet....
Add this to your functions.php file:
\n";

			}

			function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {

				$indent = ( $depth ) ? str_repeat( "\t", $depth ) : '';

				$li_attributes = '';
				$class_names = $value = '';

				$classes = empty( $item->classes ) ? array() : (array) $item->classes;
				if ($args->has_children){
					$classes[] 		= 'dropdown';
				}
				$classes[] = 'menu-item-' . $item->ID;
				$classes[] = ($item->current) ? 'active' : '';

				$class_names = join( ' ', apply_filters( 'nav_menu_css_class', array_filter( $classes ), $item, $args ) );
				$class_names = ' class="' . esc_attr( $class_names ) . '"';

				$id = apply_filters( 'nav_menu_item_id', 'menu-item-'. $item->ID, $item, $args );
				$id = strlen( $id ) ? ' id="' . esc_attr( $id ) . '"' : '';

				$output .= $indent . '';

				$attributes  = ! empty( $item->attr_title ) ? ' title="'  . esc_attr( $item->attr_title ) .'"' : '';
				$attributes .= ! empty( $item->target )     ? ' target="' . esc_attr( $item->target     ) .'"' : '';
				$attributes .= ! empty( $item->xfn )        ? ' rel="'    . esc_attr( $item->xfn        ) .'"' : '';
				$attributes .= ! empty( $item->url )        ? ' href="'   . esc_attr( $item->url        ) .'"' : '';
				$attributes .= ($args->has_children) 	    ? ' class="dropdown-toggle" data-toggle="dropdown"' : '';

				$item_output = $args->before;
				$item_output .= '';
				$item_output .= $args->link_before . apply_filters( 'the_title', $item->title, $item->ID ) . $args->link_after;
				$item_output .= ($args->has_children) ? ' ' : '';
				$item_output .= $args->after;

				$output .= apply_filters( 'walker_nav_menu_start_el', $item_output, $item, $depth, $args );
			}

			function display_element( $element, &$children_elements, $max_depth, $depth=0, $args, &$output ) {

				if ( !$element )
					return;

				$id_field = $this->db_fields['id'];

				//display this element
				if ( is_array( $args[0] ) )
					$args[0]['has_children'] = ! empty( $children_elements[$element->$id_field] );
				else if ( is_object( $args[0] ) )
					$args[0]->has_children = ! empty( $children_elements[$element->$id_field] );
				$cb_args = array_merge( array(&$output, $element, $depth), $args);
				call_user_func_array(array(&$this, 'start_el'), $cb_args);

				$id = $element->$id_field;

				// descend only when the depth is right and there are childrens for this element
				if ( ($max_depth == 0 || $max_depth > $depth+1 ) && isset( $children_elements[$id]) ) {

					foreach( $children_elements[ $id ] as $child ){

						if ( !isset($newlevel) ) {
							$newlevel = true;
							//start the child delimiter
							$cb_args = array_merge( array(&$output, $depth), $args);
							call_user_func_array(array(&$this, 'start_lvl'), $cb_args);
						}
						$this->display_element( $child, $children_elements, $max_depth, $depth + 1, $args, $output );
					}
						unset( $children_elements[ $id ] );
				}

				if ( isset($newlevel) && $newlevel ){
					//end the child delimiter
					$cb_args = array_merge( array(&$output, $depth), $args);
					call_user_func_array(array(&$this, 'end_lvl'), $cb_args);
				}

				//end this element
				$cb_args = array_merge( array(&$output, $element, $depth), $args);
				call_user_func_array(array(&$this, 'end_el'), $cb_args);

			}

		}

	}

endif; ?>
Then add this to the header.php file:

      <?php
			$args = array(
				'theme_location' => 'top-bar',
				'depth'		 => 2,
				'walker'	 => new Bootstrap_Walker_Nav_Menu()
			);
			wp_nav_menu($args);
		?>

        <?php //wp_nav_menu(array('theme_location' => 'primary_navigation', 'walker' => new Roots_Navbar_Nav_Walker())); ?>

Hope this does solve the issue.

http://www.markdotto.com/2012/02/27/bootstrap-explained-dropdowns/

As mentioned in @retlehs link, it was a delibrate design decision by the developers of bootstrap.
If anyone wants to change it so that the parent link is an actual link AND for it to dropdown on hover, you can use CSS
    display: block;
}```

As I was the person who posted the question on Stackoverflow that Mark Dotto referred to, I thought I'd chip in here.
I can see a major advantage in the Bootstrap implementation of dropdowns- and that is that they work on mobile devices. Showing a dropdown on hover is very difficult on a mobile- how do you "hover"?!
However there are still big issues in my (very) humble opinion with the implementation they have adopted. I emailed Mark about this, although I've yet to hear a response. Perhaps I am being overly pedantic or missing something. If so, I hope to be corrected!
It's usual for a navigation menu to mirror the site architecture. The top level pages being the main links, and 2nd level pages being in a dropdown menu under each top link. It's also usual to use lists and sub lists for this:

Home
About

The Team
Others

Contact

Here you can click the top level pages, and traditionally you'd see the dropdown appear when you hovered over it.
The issue I have with the implementation in Bootstrap is that you have to set the top page as an anchor tag. @retlehs refers to the Bootstrap docs above by saying you have to show the anchor for the top level of a dropdown as "#" This has a number of issues in my mind:

How do you link through to the parent page now that it's an anchor link?
You could put the parent link in the sub list above the sub pages. But this isn't mirroring the architecture of the site. In my view this also makes it less accessible.
It's not very semantic. An blank anchor tag doesn't mean anything.
It relies on javascript. OK, most people have Javascript on, but I'm a great believer in graceful degregation. It should work as expected with Javascript switched off, but you can do even more cool things with it switched on.

This is more of a Bootstrap issue, but it does affect Roots. It makes creating menus in Wordpress much more complicated. It will be difficult to explain to clients that they need to create a dummy top level element with a "#" for each dropdown.

@baritoneuk I'm on a mixed-thoughts on the "implementation they have adopted". While i do agree with you - we need to remember that bootstrap was initially created as a style guide to 'show off' and document all the styles they use on twitter.
i personally think that Roots SHOULD eventually modify some of bootstrap's functionality to better suit WordPress functionality. I don't think it needs to be done right now, though.


tea --magic=install; tea --magic=uninstall

I don't like the ergonomics of --magic=install but it was quick n’ easy. Will revise when I have better ideas

--add and --rm?

I considered a --magic that “magically” determines shell (by testing syntax or magic variables), but it's not worth it probs


Initial version of Wear OS support

My main concerns is how to handle dialog which isn't directly supported in Wear OS.
Fixes https://github.com/raamcosta/compose-destinations/issues/264
device-2022-12-23-120817.webm

Hi @yschimke ! Thank you for doing this!
I will accept the PR and make some changes right after, I will post here links to those commits so you can check what I did on top of your changes :)
Again, thank you so much!

https://github.com/raamcosta/compose-destinations/commit/d5fe7086fbdf0713820f5132db383a35640b87f4
This one ☝
Also did very minor renaming of the "shared" module which is unrelated 😋


No image on Home Assistant

Description
Hi,
Things were working perfectly untill monday and suddenly (update of core HA and update of TAPO control) I don't get any image on HA where I have all images on TAPO app on my phone. Please help
Best regards
Edouard
Reproduction Steps
Connect to HA as in the past
Expected behavior
To get clear images
If applicable, add error logs.
Logger: custom_components.tapo_control
Source: helpers/update_coordinator.py:300
Integration: Tapo: Cameras Control (documentation, issues)
First occurred: 14:47:12 (1 occurrences)
Last logged: 14:47:12
Unexpected error fetching Tapo resource status data: 'bool' object is not subscriptable
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 300, in _async_refresh
self.data = await self._async_update_data()
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 256, in _async_update_data
return await self.update_method()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/tapo_control/init.py", line 482, in async_update_data
entity["entity"].updateTapo(
File "/config/custom_components/tapo_control/sensor.py", line 183, in updateTapo
self._attr_state = camData["connectionInformation"]["ssid"]
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^
TypeError: 'bool' object is not subscriptable
Device Firmware
1.3.9 Build 231024 Rel 62920n (4555)
Integration Version
5.4.14
Using stream component
Yes
Does camera work via official integrations?
Yes
Camera has all attributes filled out in developer tools
Yes
HASS Environment
HACS
Search for similar issues
Yes
Additional information
No response

I have the same issue.

See FAQ regarding multiple rtsp stream limitation.
Regarding error ssid, this is a duplicate: https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/506


How to display an empty field?

I'm trying to display an empty field when there's no date selected.
How do I do that?
Any help will be appreciated.

In the examples directory is an example with the prop defaultText="..." Set that to an empty string.

thanks @cranesandcaff


Lambda needed for add_updater

Description of bug / unexpected behavior

add_updater doesn't work with a function* passed in as an argument unless it is in lambda form
*that is part of the class and has the self argument
Expected behavior

add_updater successfully working without throwing an error and not needing to use a lambda as an intermediary.
How to reproduce the issue

Code for reproducing the problem
class A(Scene):
    def updatingFunction(self, t):
        t.set_height(self.value.get_value())

    def construct(self):
        r = Rectangle().add_updater(self.updatingFunction) # does not work
      # r = Rectangle().add_updater(lambda t: self.updatingFunction(t)) does work
        self.value = ValueTracker(r.get_height())
        self.play(ShowCreation(r))
        self.play(self.value.set_value,10)

Additional media files

Images/GIFs

Logs
Terminal output

https://pastebin.com/8sCPtuED

System specifications
System Details

OS Win 10
RAM: 12 GB
Python version (python/py/python3 --version): Python 3.9
Installed modules (provide output from pip list):

Package           Version
----------------- -------
cairocffi         1.2.0
cffi              1.14.3
colorama          0.4.4
colour            0.1.5
commonmark        0.9.1
grpcio            1.33.2
grpcio-tools      1.33.2
manimce           0.1.0
numpy             1.19.3
pangocairocffi    0.3.2
pangocffi         0.6.0
pathtools         0.1.2
Pillow            8.0.1
pip               20.2.4
progressbar       2.5
protobuf          3.13.0
pycairo           1.20.0
pycparser         2.20
pydub             0.24.1
Pygments          2.7.2
rich              9.2.0
scipy             1.5.4
setuptools        50.3.2
six               1.15.0
tqdm              4.51.0
typing-extensions <IP_ADDRESS>
watchdog          0.10.3

LaTeX details

LaTeX distribution (e.g. TeX Live 2020): TeX Live 2020
Installed LaTeX packages: manim-latex

FFMPEG
Output of ffmpeg -version:
ffmpeg version 4.3.1 Copyright (c) 2000-2020 the FFmpeg developers
built with gcc 10.2.1 (GCC) 20200726
configuration: --enable-gpl --enable-version3 --enable-sdl2 --enable-fontconfig --enable-gnutls --enable-iconv --enable-libass --enable-libdav1d --enable-libbluray --enable-libfreetype --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libopus --enable-libshine --enable-libsnappy --enable-libsoxr --enable-libsrt --enable-libtheora --enable-libtwolame --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libzimg --enable-lzma --enable-zlib --enable-gmp --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvo-amrwbenc --enable-libmysofa --enable-libspeex --enable-libxvid --enable-libaom --enable-libgsm --enable-librav1e --disable-w32threads --enable-libmfx --enable-ffnvcodec --enable-cuda-llvm --enable-cuvid --enable-d3d11va --enable-nvenc --enable-nvdec --enable-dxva2 --enable-avisynth --enable-libopenmpt --enable-amf
libavutil      56. 51.100 / 56. 51.100
libavcodec     58. 91.100 / 58. 91.100
libavformat    58. 45.100 / 58. 45.100
libavdevice    58. 10.100 / 58. 10.100
libavfilter     7. 85.100 /  7. 85.100
libswscale      5.  7.100 /  5.  7.100
libswresample   3.  7.100 /  3.  7.100
libpostproc    55.  7.100 / 55.  7.100

Additional comments

Very similar to #761

Thanks @protonlaser91. I believe this is because the r.updaters list contains a direct reference to the object's method, and apparently that cannot be deepcopyed effectively. Using the lambda fixes this because then the r.updaters list contains a reference to an entirely different function instead.
I don't see how to fix this other than completely changing the way that updaters work. In my ideal world, updaters would not just be a list stored in the object itself, but we would use some sort of event-based system instead.
Perhaps @kolibril13 has some other thoughts

I actually can't get either version of the code to work. Both fail with AttributeError: 'DraftScene' object has no attribute 'value'. But that aside it doesn't seem like the code would work properly even if that was fixed, because the first argument to an updater is supposed to be the mobject being updated.

I actually can't get either version of the code to work. Both fail with AttributeError: 'DraftScene' object has no attribute 'value'. But that aside it doesn't seem like the code would work properly even if that was fixed, because the first argument to an updater is supposed to be the mobject being updated.

Where did DraftScene come from?

I just always give my Scenes that name so I can easily reuse command lines.

I confirm I can run the code in the OP and I get a similar error to @protonlaser91.
@eulertour are you on latest master? It sounds like the updater is being called before the ValueTracker has been created. In the past, each updater was ran once as soon as it was added, but that changed a while ago I think.

@leotrs That was true, there was some path resolution running an old version. The first argument to an updater has to be the mobject being updated (not a Scene), so this version of the code works.
from manim import *

class DraftScene(Scene):
    def construct(self):
        def updatingFunction(r):
            r.set_height(self.value.get_value())

        r = Rectangle().add_updater(updatingFunction)  # does not work
        # r = Rectangle().add_updater(lambda t: self.updatingFunction(t))  # does work
        self.value = ValueTracker(r.get_height())
        self.play(ShowCreation(r))
        self.play(self.value.set_value, 10)


Line highlighting not working with Find

As mentioned in https://github.com/Microsoft/vscode/issues/2789#issuecomment-181394595, line highlighting works with Go to Line but not with Find.

@alexandrudima Could this be targeted for a milestone or put into the backlog?

It is super confusing to try and find my cursor without line highlighting.  Would love to see this!

@alexandrudima Not working in 1.3.0-insider. Perhaps it missed the boat?

@alexandrudima Verified in 1.4.0-insider.


Better feedback for compilation errors

Custom reporter for webpack middleware to check if there were any compilation errors. If yes,  show it to a user.

A wild gif appears!

This functionality already exists in Hual. If you don't have it, clean cache and reinstall node_modules

Are you sure it exists?
For example, I am using TypeScript and I would like to have the red error screen (or something like that) in the device/AVD is there's a compilation/linting error with webpack. And yes, I do have webpack.NoEmitOnErrorsPlugin() in the config.


Add RateLimitPolicy create page

Closes #31
Screenshots below.
A few fields have been purposefully left out of the form view as the API will be changing for kuadrant v1.
Future changes to the from are captured in https://github.com/Kuadrant/console-plugin/issues/68
In the interim, an info message is shown to direct the user to the yaml view for more complex limits configuration beyond simple limits.
Create form

Add Limit modal

### Limits in form

### Yaml editor

@jasonmadigan thanks for reviewing.
I've pushed up recommended changes.
Also added i18n strings.

Changes look good, let me give it a quick spin again

Works good, noticed one small thing - the edit action in the RLP list view doesn't seem to navigate to edit

the edit action in the RLP list view doesn't seem to navigate to edit

I'd say that's the same for all Edit buttons?
There's a separate issue to figure out reusing the create view as an edit view #59

Must've had a fever dream where I thought some of these worked - nevermind so!
/lgtm


[$500] [P2P Activation] Chrome mWeb - scan button provides no feedback

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

Version Number: v1.3.85-0
Reproducible in staging?: Y
Reproducible in production?: Y
Slack conversation: https://expensify.slack.com/archives/C05RECHFBEW/p1697647198716039
Action Performed:

Logged in on mWeb chrome
Selected Scan
Went through scan flow on this account and scanned a receipt via mobile web
Landed back in this flow
Tried to tap scan again

Expected Result:
The camera should work as we already granted permission, OR we should minimally ask for permission again
Actual Result:
I'm stuck on being unable to access the camera, and tapping the scan button doesn't prompt me to allow camera access or do anything
Platforms:

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

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

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

Upwork Job URL: https://www.upwork.com/jobs/~01c10a1b4074c90859
Upwork Job ID:<PHONE_NUMBER>013912576
Last Price Increase: 2023-10-19

Unable to reproduce on Chrome, iOS. @kevinksullivan @Ollyws , how about you?

There are a few flows where I'm unable to reproduce but there is one where I am, it's something like.

On iOS Chrome, go to new.expensify.com and sign in.
Tap the green + sign in the bottom left corner (global create button)
tap request money
Tap Scan at the top, if needed.
Allow access (if needed) and scan photo
after scanned, click the back button to go to LHN
Refresh page (by clicking on the link in the address bar then tapping go
Tap the green + sign in the bottom left corner (global create button)
tap request money
Observe that camera doesn't work

Also tagging @lukemorawski who might have insight here.

@mallenexpensify works for me. I followed all your steps

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01f0e41431eb01dbdd

That's the email I'm currently logged in with

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01f0e41431eb01dbdd

This a comment from Lucas of Callstack

Thanks for confirming my reproduction steps worked @alexander01202 . @Ollyws can you please test too? If you can also confirm I'll update the OP to be more details.
I raised the price to $1k to get more eyes, since this is time sensitive.
@jasperhuangg can you confirm this can be worked on externally?  (I'll remove your assignment after you confirm)

@mallenexpensify as you mentioned ios chrome, does it means it only reproducible on physical device?

@mallenexpensify we have a PR in review for this: https://expensify.slack.com/archives/C05RECHFBEW/p1697779669659709

@mallenexpensify Good shout! It looks like this can be worked on externally.

Assigned @situchan as the C+ as he's been reviewing this.

@Samueljh1 navigator.permissions.query({ name: 'camera' }) will not ask for permissions. It'll just give you the current state of them (as name suggests).

@Samueljh1 navigator.permissions.query({ name: 'camera' }) will not ask for permissions. It'll just give you the current state of them (as name suggests).

You are correct here. Internally, the permission is requested when calling navigator.mediaDevices.getUserMedia({video:true}). When doing the reproduction steps, the browser returns Unhandled Promise Rejection: NotAllowedError: The request is not allowed by the user agent or the platform in the current context, possibly because the user denied permission.. No prompt is shown. Manually reloading the page causes this prompt to appear again, offering a workaround.

Proposal
Please re-state the problem that we are trying to solve in this issue
User is not able access the camera on Request money > Scan.
Tapping the Scan button again doesn't prompt user to allow camera access on Scan tab.
What is the root cause of that problem?
TLDR: The camera permission prompt does show but due to a race condition of the react tab navigation the prompt is dismissed which automatically sets camera permission as denied.
https://github.com/Expensify/App/blob/e99c297332c738475cfe5701e0a00ba0b5d58029/src/pages/iou/ReceiptSelector/index.js#L194-L204
Issue comes from ReceiptSelector because due to the race condition that react tab navigation creates, the NavigationAwareCamera is rendered before the transition to the Scan tab is completed which dismisses the camera permission prompt. I've been able to reproduce this both on staging and local with current main code:
I added some slow-motion to the videos belod where the prompt can be seen briefly.

iOS mWeb: Chrome

staging

https://github.com/Expensify/App/assets/56457735/f20b03d9-48e8-47a3-a300-d5e4ba6aa22a

local

https://github.com/Expensify/App/assets/56457735/b46e1dcf-376b-4f70-8d17-7f85ca3fac47

The way I've been able to test locally is by being on the same wi-fi both on the mac and on a real iPhone device and setting up fully trusted root certificates for <IP_ADDRESS> (local ip) and adding the certs in webpack.dev.config before starting the app locally in order to be able to have secure https:// locally on iPhone.
What changes do you think we should make in order to solve the problem?
Unfortunately I couldn't find any other solution then using setTimeout for this edge case related to the react navigation which works because it was implemented in a similar fashion here:
https://github.com/Expensify/App/blob/e99c297332c738475cfe5701e0a00ba0b5d58029/src/components/TabSelector/TabSelector.js#L89-L94
So my working solution is the following:

add a state variable called isPermissionReady which is set to initial value !Browser.isMobile()
within a useEffect with navigation.isFocused() as dependency, check if (Browser.isMobile() && navigation.isFocused()) is true
where a setTimeout with timeout of CONST.ANIMATED_TRANSITION (300ms) will call setPermissionReady(true)
and in the return of the useEffect we set setPermissionReady(false) and we clear the timeout

By this we ensure that:

if Browser.isMobile() is true, the NavigationAwareCamera is only rendered when isPermissionReady becomes true after the navigation.isFocused() and when the timeout is called
this way the camera permission prompt is not displayed before the Scan tab is fully rendered (including animations)
finally we make sure to set isPermissionReady back to false and clear the timeout when unmounting

Caveats:

Once the camera works and user then decides to manually disable the camera permission, the camera will become black until user navigates between tabs then back to the Scan tab or performs browser refresh while on Scan tab in order for the permission prompt to be shown again and accepted.

iOS mWeb: Chrome - Caveat 1
https://github.com/Expensify/App/assets/56457735/c20ea0c6-2b57-4cdc-bf82-5decabd23c60

This implementation does not work if attempted from a desktop computer in the Chrome browser on an iOS simulator, or any mobile simulator for that matter.

MacOS - Desktop: Chrome mWeb Simulator - Screenshot

Code changes:
// src/pages/iou/ReceiptSelector/index.js

+    const [isPermissionReady, setPermissionReady] = useState(!Browser.isMobile());

+    useEffect(() => {
+        let timeoutID;
+        if (Browser.isMobile() && navigation.isFocused()) {
+            timeoutID = setTimeout(() => {
+                setPermissionReady(true);
+            }, CONST.ANIMATED_TRANSITION);
+        }

+        return () => {
+            setPermissionReady(false);
+            clearTimeout(timeoutID);
+        }
+    }, [navigation.isFocused()]);

+                {isPermissionReady && (
                    <NavigationAwareCamera
                        onUserMedia={() => setCameraPermissionState('granted')}
                        onUserMediaError={() => setCameraPermissionState('denied')}
                        style={{...styles.videoContainer, display: cameraPermissionState !== 'granted' ? 'none' : 'block'}}
                        ref={cameraRef}
                        screenshotFormat="image/png"
                        videoConstraints={{facingMode: {exact: 'environment'}}}
                        torchOn={isFlashLightOn}
                        onTorchAvailability={setIsTorchAvailable}
                        forceScreenshotSourceSize
                    />
+                )}

Videos

iOS mWeb: Chrome
https://github.com/Expensify/App/assets/56457735/7955cd5a-bf7d-4731-ad83-624096c16457

@ikevin127 , someone is already hired for this job, sorry.  Please search for issues with the Help Wanted label. This link should show all.
@lukemorawski , can you please address @situchan comment in the PR?

@lukemorawski please fix conflict when you get a chance

working on some refactorings

Reviews are wrapping up and I'm seeing about getting this merged today.

@Ollyws @situchan please fill out the checklist above when you get a chance!

Bump @Ollyws and @situchan

I think the bug always existed after implementing scan feature.
We should create regression test for this since this bug is critical and main user flow.
Regression Test Proposal (mWeb only)

Go to FAB > Request money

Tap Manual or Distance tab (not Scan)

Refresh app

Tap Scan tab

Verify that app asks for camera permission and grant permission (optional)

Verify that camera view shows

Click back button

Refresh app

Go to FAB > Request money

Verify that app asks for camera permission and grant permission (optional)

Verify that camera view shows

Refresh app

Verify that app asks for camera permission and grant permission (optional)

Verify that camera view shows

Contributor: @situchan paid $1000 via Upwork
Contributor+: @Ollyws owed $1000 via Upwork
@Ollyws can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~01a0e882ab00ed78a6
(the old job closed so I created a new one)
Testrail update GH created

https://github.com/Expensify/Expensify/issues/340448

@mallenexpensify we can close this.
PR author is expert agency, C+ reviewer is just me

Ah... thanks @situchan , I didn't catch that, I just saw @Ollyws was assigned so I assumed they were due payment.
I updated the payment comment above to be accurate.


Epic: Improve Profile UX

Description of Problem:
We want to make the DataHelix as easy and intuitive to use as possible.   Related to #1243
Some Areas to Consider:

Typing: Currently must be done in constraints.

Can have conditional types, which is complex for user and developer.
Takes up a lot of space in rules.

Profile is currently quite verbose.

Especially true for the new datetime relations.

Investigate whether using Lua would be helpful.

Changes ready to be written up and implemented:
•	Put types, nullability, uniqueness into field definitions. Also consider putting the following in:
o	Formatting
o	Granularity
o	Min/max length
o	Numeric range allowed
•	Make the datetime format for value consistent.
•	“isNot” as well as “is” https://github.com/finos/datahelix/issues/1001.
•	Once type is a field definition, allow quotes around numbers.
Changes wanted that need more discussion:
•	Custom, user-defined generators.
o	Consider Lua
•	Add a defaults section to apply to multiple fields.
o	See the attached proposal.
•	Consider how to support multi-layer JSON and multiple tables.
o	See the attached proposal.
•	Don’t use JSON?
o	It doesn’t seem suited to be concise.
o	It doesn’t seem suited to describing inter-field relations.
o	It doesn’t allow comments.
•	Consider being able to name blocks of JSON for reusability.
•	Reduce repetition, e.g. “field is value”
o	From “field x is lessThan value y” to “field x lessThan y”
•	Consider how to make conditional blocks less verbose and confusing.
o	See the attached proposal.
•	Combine the concept of rules and constraints.
o	See the attached proposal.
•	Constraints that are similar can be far apart in the profile.
o	Evaluate after more information is added to the field definition.
•	Types can be hard to understand, and are potentially inconsistent.
Changes/Comments that we may address later:
•	Schema Validation not working well. Disable?
•	GUI based approach?
•	Unintuitive syntax, ambiguous in how it should be extended.
•	Null behaviour is unintuitive and inconsistent: https://github.com/finos/datahelix/issues/1115
•	Different way to set up table definitions to how to set up rules.
o	See the attached proposal.
•	“Not” is inconsistent e.g. ISINs.
•	Writing profiles is slow and error-prone.
•	Datetime offsets are ambiguous.
•	Imperative vs Declarative?
•	Entirely expressive language?
•	Have JSON tuple/list to contain contents of “field is value”?
•	CSV input for simple parts of profile (instead of JSON).
•	Go onto a different project and forget about it.

closing as duplicate as now capturing in a milestone (milestone 5)


Multiple variants

User can now add and delete individual variants,
Will need further work on making validation work seamlessly although this can be tackled in the next PR,
Failing tests currently being fixed in #29 , once they are fixed we can merge #29 into this PR
closes #22

closes #22. Tests passing locally but failing in actions. The confirmation page required even more data manipulation to render the content, we should decide whether the confirmation page is worth keeping around and whether the current way it is rendered is satisfactory.


Update Mapper tab

Added explanation the tab should not be closed or reloaded.
Updated the mapper tab page to look nice and clean.

Tested manually. I don't think it makes sense to automate compression, as we don't expect the markup to be changed often.

Imgur
Imgur

Screenshots look great! One possible future improvement (definitely don't block yourself in this PR for that) is to make the JSON pretty / formatted in the last container.


Fixed bug streaming SpreadPlot data

[x] Fixes https://github.com/ioam/holoviews/issues/2917
[x] Adds unit test

Ready to merge.

Looks good. The comment would be clearer if it said '# Plot does not support continuous boke streaming' (or similar) but that is a very minor complaint. Merging.


The "iOS run device" does not select the correct device

using the button "iOS run device" run the command:
"/opt/homebrew/opt/node@14/bin/npx react-native run-ios --device"
Unfortunately in my case, it try to use my mac mini, not my iphone
"info Using first available device named "Patrick’s Mac mini" due to lack of name supplied."
Is there any setup that I've missed?
How could I specify a specific device?

This logic is controlled by the react native command it self.
I noticed the real device list function is comment out in some version, I’ve uploaded a new version to the Jetbrains plugin center, you can download it then install it from disk and have a try first or waiting for the Jetbrains’ admin to approve it.
https://plugins.jetbrains.com/plugin/download?rel=true&updateId=126671
Then with the new version you can use the button next to it:
There should have your iPhone in the list’s top(please connect your iPhone by USB and enable developer mode).
The finally command will like this:
/usr/local/bin/npx react-native run-ios --device "your iPhone name here"

This update has been published, please take a try with the latest version.

First of all Thank you,
the improvement is greatly appreciated.
However, it's really frustrating that we cannot control which device is selected by the button "iOS run device". Some people are lucky enough to get the right one!
So, since we often use the same physical device, how about a button to run directly by the last device we used ?

Great advice and I'll change this command's behavior by only showing physical devices and auto select the last used one, please wait for the next version. Update will first posted here.

The new release had fixed this issue and allowing remember last chosen iOS device, please have a try.


Multi selection of torrent !?

Hello !
It will be great if the user can select few torrents (by using Ctrl & Shift keys), in order to suspend or remove torrent file to many files in one action !
And thanks for this little pearl (I mean PicoTorrent) : it work very well and nice ! (Adios Vuze !)
You can be proud of you ! (I'm an old web programmer, but I did programs since my 7 years old ... so, it's a very deep and sincere compliment !!) So, a big thanks from me ... and a lot of anothers users, I'm certain of this !

There's an issue for this already. Also, what's with the PR? 😄 Closing this since there doesn't seem to be any code here.


How can i get the image in different layers?

Hi , i want to check images on different CNN layers. How can i output the images,just like Figure 1 (a,b,c,d,e) in Gatys's paper

How can i do it?
Many thanks

Only set '-style_layers' separately as 'relu_1_1', 'relu_2_1'  etc.

Thanks.


occurrencesHighlight keyboard shortcut

I disabled occurrencesHighlight, but sometimes I need it and I can't find any keyboard shortcut to trigger it, so my suggestion is to add it inside right mouse click list and as a keybinding.
If needed I can create a PR, but I need a help with as much info as possible.
Thanks!

I changed my mind, it's really helpful feature and without it, it's really hard 😄


Extend Pages Functions Redirects docs to mention file extensions

Existing documentation URL(s)

https://developers.cloudflare.com/pages/configuration/redirects/#placeholders

What changes are you suggesting?
Mention support for file extensions in dynamic redirect rules with placeholders, like so:
/assets/:name.svg https://static-assets.example.org/svg/:name.svg 301
/assets/:name.png https://static-assets.example.org/png/:name.png 301
/assets/:name https://static-assets.example.org/other/:name 301

Additional information
No response

After consulting with the team, it was determined that this is too much of an edge case to add at this time, but thank you so much for your feedback @marcelgerber and we will certainly add it in the future if we receive additional similar feedback.


Go router - redirect issue!

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
Go router redirect holds the history after force redirect.

Login page -> (after successful login) Home page.

when i use redirect method to force the user to homepage from login page. I can able to go back and forth. Meaning I can still able to view the login page again.
How to implement push.replacement in this scenario?
Please use the default any sample app to reproduce the same result!
Expected results
no history in the navigation stack after redirect method.
Actual results
navigation history remains same and not cleared!.
Code sample
  redirect: (context, state) {
  final auth = bloc.state.status == AuthenticationStatus.authenticated;
  var userRole = false;
  if (bloc.state.status == AuthenticationStatus.authenticated &&
      bloc.state.user.message != null &&
      bloc.state.user.message!.data != null &&
      bloc.state.user.message!.data!.roles != null) {
    for (final element in bloc.state.user.message!.data!.roles!) {
      if (element.role == adminRoleName || element.role == adminRoleNameFullname) {
        userRole = true;
      }
    }
  }
  if (userRole) {
    debugPrint('yes admin role found and re-directing to admin role');
  }

  /// current location
  final locationClick = state.location;
  // debugPrint(auth.toString() + locationClick);

  if (auth && !userRole && (locationClick == '/$loginURLPagePath' || locationClick == welcomeURLPagePath)) {
    return homeURLPagePath;
  } else if (auth && userRole && (locationClick == '/$loginURLPagePath' || locationClick == welcomeURLPagePath)) {
    return adminhomeURLPagePath; //adminFeeSetupPath; // communicationPath;// schoolSetupPath;
  //adminhomeURLPagePath; //; //classWisePaymentPath; //adminFeeSetupPath; // homeWorkPath; //communicationPath;
  //adminhomeURLPagePath; //adminHelpDeskURLPagePath; //adminhomeURLPagePath; // adminFeeSetupPath;
  //adminFeeStructureDetailPath;//
  } else if (!auth) {
    return loginURLPagePath;
  }
  return null;
},

Screenshots or Video
na
Logs
na
Flutter Doctor output
flutter doctor -v
[✓] Flutter (Channel stable, 3.10.0, on macOS 13.2.1 22D68 darwin-arm64, locale en-IN)
• Flutter version 3.10.0 on channel stable at /Users/puspharaj/Documents/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 84a1e904f4 (4 weeks ago), 2023-05-09 07:41:44 -0700
• Engine revision d44b5a94c9
• Dart version 3.0.0
• DevTools version 2.23.1
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at /Users/puspharaj/Library/Android/sdk
• Platform android-33, build-tools 33.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
• All Android licenses accepted.
[!] Xcode - develop for iOS and macOS
✗ Xcode installation is incomplete; a full installation is necessary for iOS and macOS development.
Download at: https://developer.apple.com/xcode/download/
Or install Xcode via the App Store.
Once installed, run:
sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
sudo xcodebuild -runFirstLaunch
• CocoaPods version 1.12.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.15+0-b2043.56-8887301)
[✓] VS Code (version 1.75.1)
• VS Code at /Users/puspharaj/Downloads/Visual Studio Code.app/Contents
• Flutter extension version 3.64.0
[✓] VS Code (version 1.78.2)
• VS Code at /Applications/Visual Studio Code 2.app/Contents
• Flutter extension version 3.64.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-arm64   • macOS 13.2.1 22D68 darwin-arm64
• Chrome (web)    • chrome • web-javascript • Google Chrome 114.0.5735.90
[✓] Network resources
• All expected network resources are available.
! Doctor found issues in 1 category.

@Puspharaj  Which go_router version are you using? I checked this issue with redirection.dart but it works as expected. The login screen is cleared after navigating to home screen. You can check this by pressing back button on Android device on Home screen. It will quit app as expected.

How about flutter web! Thanks
On Mon, Jun 5, 2023, 3:18 PM huycozy @.***> wrote:

@Puspharaj https://github.com/Puspharaj Which go_router version are you
using? I checked this issue with redirection.dart
https://github.com/flutter/packages/blob/main/packages/go_router/example/lib/redirection.dart
but it works as expected. The login screen is cleared after navigating to
home screen. You can check this by pressing back button on Android device
on Home screen. It will quit app as expected.
—
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/128166#issuecomment-1576478004,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDOL5ZTHNXHKVTPP2REE5DXJWTN5ANCNFSM6AAAAAAYZFYZRU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think this is limitation of underlying RouterDelegate #97993

@Puspharaj As I can see, it also works as expected on Web platform:

Demo Web
https://github.com/flutter/flutter/assets/104349824/f4ae4d37-aa55-43a1-9582-ef5380718546

It may be caused by your logic inside redirect callback. I would recommend you double-check your implementation there and refer to sample code I linked above as well.
I will close this issue from here. If you still face this issue, please provide a completed and minimal reproducible code sample that doesn’t include 3rd party plugins or complex production code so that we may verify this. Thank you!

@huycozy would you please share the code as well? thanks!

You can try it here: redirection.dart

cool, but the things if i use shellroute, this doesn't allow me to navigate within the shell routes!

That's another open issue here: https://github.com/flutter/flutter/issues/114559. Please follow up on it for further updates.


How to install and run codpan

Coned it. Now how to instal and run it.

See https://github.com/egoist/codepan#development, basic yarn dev or npm run dev to run it in a dev server.
If you want a production build, run yarn build and it will be available in ./dist

sh: 1: poi: not found when I run npm run dev

Have you run yarn to install deps?

I installed yarn, run yarn build successfully. Got a dist folder. Now how to run it?

I successfully run it on dev using yarn dev. How to run it on production server?

it's a static website so you can run it locally with serve, eg: serve dist --single (single page app mode)

OK. What we want is, want to many urls like jsbin available .How to do that?

want to many urls like jsbin available

what do you mean?

Actually we want to install in a site and embed many html/css/js examples in various pages.
for example we have created an example
https://codepan.net/gist/0ffcc5fa021aa45b87c0792d8362b5b5
Now we want to embed the above in various pages of our site and the url will be
https://example.com/html-css-js/0ffcc5fa021aa45b87c0792d8362b5b5
Thanks for your response.

I just define more clearly after creating the above url i.e.  https://example.com/html-css-js/0ffcc5fa021aa45b87c0792d8362b5b5
will be embed in  https://example.com/js-examples page or any other pages under https://example.com.

I think you mean this https://github.com/egoist/codepan/issues/11

Yes. Almost. Let me try.

Well, for running it on production, I want to run it like 'forever somefile' so that it runs in the background and I can use this to render multiple urls, each presenting one code snippet. Is that possible. serve does not serve my purpose.

this is just an SPA, so I think it would be tricky even if it's possible.

So, unlike jsbin I can not run the app in the background and then create multiple instances (urls) each presenting one url fetched from Github you say?

it's a static website so you can run it locally with serve, eg: serve dist --single (single page app mode)
I want to use node.js to run the ./dist folder. Could you please tell me how can I do this?

it's a static website so you can run it locally with serve, eg: serve dist --single (single page app mode)<
I want to use node.js to run the ./dist folder. Could you please tell me how can I do this?


Update skip level section

Updated skip level section with purpose/process
@nicksnyder I didn't commit to a specific cadence, but am thinking we should give people an option to schedule a meeting with you if they want to meet more frequently/sooner than what you propose

Key points

As our department continues to grow, I also want to regularly check in with teammates to keep a pulse on what is happening.
These meetings are an opportunity to build deeper relationships, for me to listen, gather your feedback and insights, and discuss anything top of mind for you.
Happy to talk about anything that is top of your mind or answer any questions that you might have.
These are intended to be casual conversations where we can chat informally
If I send you an invite to chat but the timing is bad or you don't have anything you want to discuss, please decline.
I don't expect status updates, that is not the intent of these

Updated based on all your feedback @nicksnyder
I am having difficulty linking to my bio correctly now that this new team.yml file was created. It keeps directing to the github team page.


#2585 Change behaviour of "Show hydrogen labels" setting so the "on" value works the same way as "terminal and hetero"

Had to re-open the pull request, code was updated for new requirements

Currently the schema will reset to default value if the provided one is not in the enum. So users who have "on" in the saved settings will have the value set to default from the schema, which is "Terminal and Hetero" at the moment.

Looks good


works on msi z270 gaming m3

all working, from power off to startup, scripts running fine. :D thank you

Thanks!


Add info on how to build KEDA and deploy it local cluster

A small paragraph on how to build KEDA and deploy it on a local cluster.
This will bring developers one step closer to contributing to KEDA. Of course there will be more effort in writing the docs for the scalers themselves.

I never found much use of KedaScaleController.yaml, i was using helm template instead.
Separate yamls tends to lead to it being stale.
Wdyt?

I agree, I will change it once I reach my PC.
On another front, if no one should use that deployment file then we should remove it. I might create another PR for that to keep concerns separate.

Thanks @balchua for the input, done :).

This PR is complete waiting for review. Upcoming documentation is going to be in another PR.

I am abandoning this PR in favour of new one based on latest 'master' #443 .


Added onInit event to the StateManager.

Description
Please describe your pull request:

Why is it necessary?
** It created a standard way to register custom jQuery plugins.
What does it improve?
** You don't have to use $(document).ready() anymore.
Does it have side effects?
** No.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
None

How to test?
Register a jQuery plugin by using $.subscribe('StateManager/onInit', addMyPlugin);

This event could be useful to register custom jQuery plugins to the StateManager. Instead of using $(document).ready() a plugin could then be registered by $.subscribe('StateManager', addMyPlugin);

@shopware/shopware-webdev Any observations on this PR?

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-17255.
Please use this issue to track the state of your pull request.


Template clone can fail in windows machines

What version of mystjs are you using?
0.1.11
How did you install myst?
npm
What operating system are you using?
Windows
Which area is this feature request for?
Command Line Interface
Describe the Bug
During myst start a default book-theme template cloned onto the user's local machine. This can fail on some windows machines but succeed on others. The machine that fails has an up-to-date git` installation accessible on the path.
An excerpt of the error message is:
📚 Built 12 pages for Eindrapport in 36 s.
🔍 Querying template metadata from https://api.curvenote.com/templates/site/myst/book-theme
🌎 Cloning template from https://github.com/curvenote/book-theme.git
fatal: Too many arguments.

usage: git clone [<options>] [--] <repo> [<dir>]

...

Command failed: git clone --recursive --depth 1 --branch main https://github.com/executablebooks/myst-book-theme.git _build\templates\site\myst\book-theme
fatal: Too many arguments

If the user then runs the git command independently in the same command prompt window, the theme is correctly cloned, and a subsequent myst start succeeds.
Expected Behavior
The git clone of the template should succeed when issues by the myst cli.
To Reproduce
Start from scratch on a windows machine with a fresh myst project.
Note: I am reporting this as another user had this error persistently and I tried to help them with it. HOwever, I have not been able to reproduce this on a Windows 10 machine.

cc @choldgraf looks like you ran into this, if you have time to let one of us debug this with you that would be awesome.

Note that this wasn't a windows machine per se, it was WSL within Windows. I'm pretty busy this whole week unfortunately but maybe next week?

I think for this, we just take out the git clone and instead download the zip from github directly.
Previously we needed a recursive clone because the themes required myst-theme as a submodule. Now, though, the themes are built into nice standalone bundles, so the zip will work.

I believe this should be fixed in the next version @choldgraf. We have switched to zip download and just avoided the code path. I will let you know when it is live!

This has been released in v1.1.4!


java.sql.Driver missing from classpath in macro in presentation compiler

Original issue: https://github.com/scalameta/metals/issues/6494
Compiler version
3.4.2
Minimized code
//> using scala "3.4.2"
//> using dep "com.h2database:h2:2.2.224"

import scala.quoted._

class LoadStuff extends Selectable:
  def selectDynamic(name: String): Any = name

object LoadStuff:
  transparent inline def make = ${ makeImpl }

  private def makeImpl(using Quotes): Expr[Any] =
    import quotes.reflect.*

    Class.forName("org.h2.Driver") // problematic line

    val refinement = Refinement(TypeRepr.of[LoadStuff], "name", TypeRepr.of[String])

    refinement.asType match
      case '[refined] => '{ LoadStuff().asInstanceOf[refined] }

Output
Interactive compiler reports error diagnostics:
java.lang.ClassNotFoundException: java.sql.Driver
        at java.base/java.net.URLClassLoader.findClass(URLClassLoader.java:445)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:593)
        at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:526)
        at java.base/java.lang.Class.forName0(Native Method)
        at java.base/java.lang.Class.forName(Class.java:421)
        at java.base/java.lang.Class.forName(Class.java:412)
        at a.LoadStuff$.makeImpl(LoadStuff.scala:13)
        at a.LoadStuff$.inline$makeImpl(LoadStuff.scala:10)

Expectation
Should compile fine and diagnostics should be empty.

Hi,
Just adding some further info from the original issue.
Thinking about it, this can be further minimized, as this has nothing to do with Selectable. Apparently just being a transparent inline with class loading is enough to trigger this behavior:
object LoadStuff:
  transparent inline def make: Unit = ${ makeImpl }

  private def makeImpl(using Quotes): Expr[Unit] =
    Class.forName("org.h2.Driver")

    '{()}

This too triggers a loss of type-information at the call-site of make.
Removing transparent brings back type information.

Hi,
I'd like to take a stab at figuring out this issue.
Can anyone please point me where to start looking in the repo?
Thanks

Hello!
Macro methods (the previously compiled .class code run on runtime) are called from the compiler/src/dotty/tools/dotc/quoted/Interpreter.scala class. There, a separately constructed classLoader is used to run that previously compiled code (the Interpreter name might be misleading, the only thing that is being interpreted is the ${ makeImpl } part, then makeImpl is run via invoking a method from a classfile). Since that class loader is constructed separately I suspect it might be the thing causing the issue. Previously we already had a problem with it in the cli, caused by the fact that calling java -cp SomeClass:compilerClasses -classPath OtherClasses replMain would not join SomeClass with OtherClasses in that class loader (but worked for non macro compilation). I do not know much about the presentation compiler, so I cannot help with that aspect unfortunately.
Making any progress here or observation would be of great help, I recall taking a stab at this some time ago but not being able to make any progress

Thank you for the detailed response!
Is there a guide on how to develop and diagnose the presentation compiler with Metals?
E.g.:

How do I see the exception in the issue description? It doesn't seem to be visible as a regular Metals user.
How do I plug my local Metals setup to use a local version of the compiler?

Thanks

How do I see the exception in the issue description? It doesn't seem to be visible as a regular Metals user.

That should be visible in .metals/reports or in .metals/metals.log

How do I plug my local Metals setup to use a local version of the compiler?

You only have to publish locally the presentation compiler module, it should be picked up automatically.

Thanks for the prompt response.
Trying to reproduce the bug now, and it's still missing type information, but also, I'm not seeing anything in the logs at all (only successful compilation).
In the original issue I reported in the Metals repository, I did get an exception visible somewhere, though I don't recall now where I got it from.
But even back then, as can be seen in the discussion there, I wasn't seeing the actual exception thrown by the presentation compiler, but rather some subsequent exception coming from Metals. It was mentioned there by Kasia, that it should be possible to get diagnostics directly from the presentation compiler. Can you please point me to where these are visible?
Thanks

Curious, I can't seem to reproduce either:
https://github.com/user-attachments/assets/2a02d862-c70f-4b2f-948e-25f398cbbe8e
The only issue I see is that no hover information is shown in some places, but not actual exception. We could try to fix that and use Hover tests for that.

Maybe it got somehow fixed or maybe there was some fallback in metals, so there is no error visible. The error I posted was only visible when logging diagnostics from pc compile run, it did not surface.

Although I've no idea where the exception has gone, the full issue at the usage site is still present:

No hover
No autocompletion
No "insert type annotation"

Which I would imagine indicates that the presentation compiler decided to quit at this point.

Also, how can I enable diagnostics from the presentation compiler?
Thanks

Playing a bit more with this issue. It seems that there's a definite improvement in 3.5.x compared to 3.4.2.
In 3.4.2 the whole usage file completely loses type information in Metals. In 3.5.x it's only the results of the transparent inline that lose information, the rest of the file continues to function.

Also, how can I enable diagnostics from the presentation compiler?

If you are expecting Metals to display those diagnostics it won't. You have to change the code. Basically after type checking in run on compilation unit in pc you can call diagnostics (or something similar) method, and log that information. This is as far as I remember from the top of my head, I'd have to check for details.

@kasiaMarek, got it, thanks.

#22101 Confirms that the issue is actually not in compiler, but in the way how metals load presentation compiler.

I just tried the example again with the version 1.4.1+73-c291bd28-SNAPSHOT of the Metals server, and I confirm it works! 🥳

Thank you so much for fixing this!
I can confirm as well that both the example and the real code that triggered the error both work correctly.