Slow Performance on Incremental Runs (Project References)

Current behavior
Incremental typechecks are very slow.
We have a project that leverages Project References in Typescript.  As the latest release of your plugin supports references we tried to upgrade.  We are seeing an unusable drop in performance when type-checking this code.

For comparison, here is the same code change ran with tsc --build --incremental

Here is our config:
      new ForkTsCheckerWebpackPlugin({
        async: true,
        typescript: {
          mode: 'write-references',
          configFile: path.resolve(APP_ROOT, 'tsconfig.json'),
          configOverwrite: {
            compilerOptions: {
              incremental: true
            }
          },
          profile: true,
          build: true
        }
      }),

Expected behavior
Incremental typechecks should be quick.
Environment

fork-ts-checker-webpack-plugin: 5.0.0
typescript: 3.7.2
eslint: n/a
webpack: 4.39.3
os: macos

Hi,
Thanks for reporting this issue. Could you check timings using ‘tsc --build --incremental --watch’? This is the API we use and the performance drop is not in the initial compilation but in the next one.

Sure thing.  I've attached the comparison.
tsc --build --incremental --watch

Webpack, showing both initial compilation and then after a file change.

Is there any chance of getting a reproduction repository?

https://github.com/nathanforce/monorepo-ts-projects-webpack
@piotr-oles Here's my best attempt.  It is very simplistic but follows the same structure/patterns as the real repo, just with far less files.  Even in this tiny example I am seeing slower numbers on the webpack side when running the two commands:
webpack:
npx webpack-dev-server --mode=development --config ./apps/alpha/webpack.config.js

tsc:
npx tsc --build ./apps/alpha/tsconfig.json --extendedDiagnostics --incremental --watch

As is expected in such a contrived reproduction, the differences here are much much smaller but maybe it can help you spot something.  Sorry for the delay!
Cheers

I will try to take a look at this in a few days :)

@piotr-oles #494 seems to have helped a bit but the typecheck times using Webpack/plugin are still much higher than using tsc.  Below screenshot shows the two running side by side for the same change.
tsc:

Cool that there is a progress - I will profile it again :)

Thanks for looking into this. I just upgraded to this version on a branch and I believe we're experiencing the same problems.

Hi guys!
I have an application built with create-react-app which uses fork-ts-checker-webpack-plugin v3.1.1. I have noticed that lately the fork-ts-checker-webpack-plugin portion of the build process (both in dev/incremental and production build) take a huge amount of time to complete. I think there is some kind of dependency that the compiler "doesn't like" that got added at some point in time. E.g. I saw people that reported that adding "material icons" introduced this kind of behavior.
I'd like to debug the compilation phase, and some pointers would help me a lot. E.g. where could I insert a console.log() so that I can see, file by file, the files being processed? Or maybe I'm being naive, and the plugin doesn't actually have such fine grained control (i.e. at file level), and in this case I should somehow work at the level of tsc?
Thanks a lot in advance!
Cristian.

We don't process file-by-file - we pass configuration and host to the TypeScript API and we use this API to check for errors. I would suggest using debugger or console.log in the node_modules/typescript/lib/typescript.js file :)

Thanks Piotr.
The problem may be around the TS code; i.e. some corner case that the TS compiler doesn't "like". However, in a create-react-app (CRA) TS is practically invoked twice.

The first time during the compilation phase (via babel), where TSC actually transpiles the code, outputting JS code
The second time via fork-ts-checker.

The first step is executed reasonably fast. It's the second step that seems to put TS to the knees.
I will somehow extract the exact parameters which are passed to the TS API. CompilerHost.js seems promising. So that I can try to translate them into a tsc ... command. Maybe the configuration has something dubious and/or somehow a lot of files are sent to processing (maybe stuff from node_modules, etc?).
ESLint is also a player in the game; I didn't yet understand which is its exact relation to TS. Maybe the issue it's not in the TS code actually, but rather related to ESLint.

* The first time during the compilation phase (via babel), where TSC actually transpiles the code, outputting JS code

Just FYI: babel is not running tsc, but just stripping away type information, so when you start debugging in the typescript compiler, this step will never hit anything there.

ESLint is also a player in the game; I didn't yet understand which is its exact relation to TS

In the case of create-react-app, fork-ts-checker-webpack-plugin is not used for ESLint purposes; eslint-loader is. ( fork-ts-checker-webpack-plugin was added to create react app only once CRA opted to support TypeScript - ESLint support was already implemented prior to this)
https://github.com/facebook/create-react-app/pull/7036

Thanks guys for your kind hints!
It does seem to be related to tsc. Running tsc => about 10 sec. Running tsc --incremental => 70 sec on a small project. And 500 sec on a bigger one. Unfortunately these issues don't seem to be rare. I have commented here. This issue seems to be also similar.
To be honest, coming from the Java world I was amazed by the power of the types system in TS. And I was very suspicious at first, having concerns related to compiler performance. It seems that at some points the "type magic" does have a non negligible cost.

fwiw, i did some extensive benchmarking[1] and fork-ts-checker struggles the most with reference files in project references (but not main files which import the references)
[1] https://github.com/TypeStrong/ts-loader/issues/1157#issuecomment-681080972

Interesting. Because I just discovered the reference / composite structure recommended by TS as a performance improvement (+ reusability flow) mechanism.

performance improvement (+ reusability flow) mechanism.

what do you mean exactly? that you are more productive? yes absolutely, it's TS killer feature and if you change reference a lot pure tsc is the fastest followed by ts-loader

@piotr-oles do you have already any ideas how to fix fork-ts-checker's perf issues with project references or is it something which can't be fixed because of architectural reasons?

I don't have any ideas yet - first we need to find the root cause of this issue. I'm not sure why it's slower for references. If someone has some free time, feel free to profile it to find where is the bottleneck :)

My (temporary ?) solution for my issue:

running tsc --incremental => 70 sec

was to monkeypatch, within CRA, the creation of ForkTsCheckerWebpackPlugin, and to force
useTypescriptIncrementalApi = false;

CRA uses an older version of the ForkTsCheckerWebpackPlugin which is no longer developed. I don't think this version supports project references at all :/ I think we should drive CRA to update to a newer version

There is indeed https://github.com/facebook/create-react-app/issues/6799
To be honest, given the time I invested in CRA for monkeypatching => personally I don't find CRA a robust enough solution. Because of our attempt to reuse a common library among several projects, we were pushed towards using some symlinking. And because of this absolutely every new feature that we added: was done with a lot of pain.
Initially I saw in CRA a set of best practices; a reference. But even the Jest testing mechanism is slow and we were forced to ... more monkeypatch for mocha, which has the advantage of executing tests instantaneously.

feel free to profile it to find where is the bottleneck

@piotr-oles  profiles are below, interestingly there's almost no output from your profiler with reference files, so i guess fork-ts-checker is not even touching those files, hence not properly supporting project references.
what should i do next?
change in non-reference file:
Poll And Invoke Created Or Deleted: 0.25 s
I/O Read:                           0.00 s
Parse:                              0.11 s
ResolveModule:                      0.00 s
ResolveTypeReference:               0.00 s
Program:                            0.11 s
Bind:                               0.04 s
Check:                              0.86 s
transformTime:                      0.00 s
Emit:                               0.03 s
Semantic Diagnostics:               0.91 s
Queued Tasks:                       1.10 s

change in reference file:
Poll And Invoke Created Or Deleted: 0.21 s
Queued Tasks:                       0.00 s

fyi/fwiw, IDK but this slow ref transpile times might come from tsloader and not from fork-ts-checker, I did more benchmarking and added babel to the mix as a comparison and something is wrong with tsloader ref handling

As a profile I mean basically debugging/finding the root cause of this behaviour using whatever method you like. It will require digging into typescript's codebase which can be a time-consuming task :)

ok got it, however is it really ts or ts-loader because what I find strange is that fork-ts is not even touching reference files (if you look at the profile data above).
IDK if I can help, I guess I am as busy as you are. However let me know if I can do something, otherwise I'd move on.

https://github.com/microsoft/TypeScript/issues/40808
Might be related?

I've added support for a "generateTrace" option available in typescript 4.1.0-beta in 6.0.0-alpha.2. It should help to debug performance issues :)

@nathanforce The latest TypeScript Beta version contains a fix that could potentially fix this issue - could you test it?

Just faced the same issue with the project freshly migrated to ts (ejected CRA + SSR build + updated fork-ts-checker-webpack-plugin).
With TypeScript 4.2.3 the build was terribly slow but seems that problem is fixed with Typescript 4.3-beta.

It seems that the bug was in the TypeScript and as the new version of TypeScript has been released, I'm closing this issue :)

Is it possible that this issue is back? I'm using Typescript 4.4.2 and after introducing a change in a package that is listed in the project tsconfig.json references my webpack-dev-server is incredibly slow (it recompiles couple of times before serving the newest files in the browser). This is my ForkTsCheckerWebpackPlugin setup:
new ForkTsCheckerWebpackPlugin({
      async: isEnvDevelopment,
      typescript: {
        configOverwrite: {
          compilerOptions: {
            sourceMap: isEnvProduction ? shouldUseSourceMap : isEnvDevelopment,
            skipLibCheck: true,
            inlineSourceMap: false,
            declarationMap: false,
            noEmit: true,
            incremental: true,
            tsBuildInfoFile: paths.appTsBuildInfoFile,
          },
        },
        context: paths.appPath,
        diagnosticOptions: {
          syntactic: true,
        },
        mode: 'write-references',
        build: true,
        profile: true,
      },
      issue: {
        // This one is specifically to match during CI tests,
        // as micromatch doesn't match
        // '../cra-template-typescript/template/src/App.tsx'
        // otherwise.
        include: [{ file: '../**/src/**/*.{ts,tsx}' }, { file: '**/src/**/*.{ts,tsx}' }],
        exclude: [
          { file: '**/src/**/__tests__/**' },
          { file: '**/src/**/?(*.){spec}.*' },
          { file: '**/src/**/?(*.){stories}.*' },
          { file: '**/src/setup-tests.*' },
        ],
      },
      logger: {
        infrastructure: 'silent',
      },
    }),

I'm using "fork-ts-checker-webpack-plugin": "6.3.3", version. My project is an ejected create-react-app with some updates to support webpack 5.


Allow deleteion of external managed cks nodes

Description
The user might create the cluster manually. And since the cluster is being managed by some external provider, it's possible that provider doesn't have context of CKS and deletes the VM. This can lead to failures.
This PR allows deletion of CKS node if the CKS cluster is Externally managed.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@blueorangutan package

@vishesh92 a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9808

What happens to the cluster-vm mapping? Do we need to remove it?

I don't think we need to remove it. Worst case an entry is left over in the table.

@blueorangutan package

@blueorangutan test

@rohityadavcloud a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-10549)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 46955 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr9183-t10549-kvm-centos7.zip
Smoke tests completed. 130 look OK, 1 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

test_02_trigger_shutdown
Failure
336.69
test_safe_shutdown.py


reference to deprecated dev library - blocking upgrade to eslint 9

When stylelint was upgrade from 14-15, post-css-in-js was deprecated - instead, there are custom parsers. https://github.com/stylelint/stylelint/blob/main/docs/migration-guide/to-15.md#breaking-changes
One of those is https://www.npmjs.com/package/postcss-lit - Is there a reason we shouldn't switch to using this?

It's probably something worth looking into. Not all of our CSS is in a Lit component though. For example:

in a css template within a module: typography styles.css; (currently linted)
in a style block within a module colors.js (not sure if this is currently linted)

So we'll want to play around with it to ensure we'd be getting the coverage we want/need.


landuse for everyone

This project is about to create an app or website to let non-programmer

show landuse overlay for thailand
post landuse map that can be share via a link

Displays the information required for cultivation
Analyze weather data for cultivation
Illustrate marketing data for other agriculture products


Why is first client stored in global Raven variable?

Hello,
I am confused why is first Raven Client is stored in global Raven variable in base.py module. It is not used anywhere as far I can tell from code and is not even exported from module. It is commented as "singelton", which it is not as it does not prevent creation of new Client instances.
base.py
# singleton for the client
Raven = None
..

class Client(object):
    def __init__(self, dsn=None, raise_send_errors=False, transport=None,
                 install_sys_hook=True, **options):
        global Raven
...
        if Raven is None:
            Raven = self

Relevant commit from 2013:
https://github.com/getsentry/raven-python/commit/2c3acebf9ff882de0ea3b2db4b3bd2b1c5b3ed62

@hiir its a convenience tool so the client can be referenced automatically in some systems. I'm not sure its very widely used.

Ok, thanks for clarification. I was confused as all did not export it, but if it is imported explicitly then it indeed has convenience value.
I also noticed that if one creates multiple Client instances, then the Raven variable is assigned first Client instance, but install_sys_hook uses last Client instance which was created. Due this there is always two Client instances left to hang around even when otherwise not used anymore (out of scope) - one via global Raven variable, another via closure assigned to sys.excepthook.


Implement Serde::{Serialize, Deserialize}

Hi Guys,
currently is it available somehow to implement Serde Serialize and Deserialize traits for the tonic-build generated rust types?
Currently I have a project where I would use the rust types for RPC communication, but I would use the same objects for the public rest API, and for that it would be nice the serde auto implementation.
Now I need to create my public api request response structs and convert the rpc request response and vica versa.
Thank you in advance.
Best wishes,
Peter Mezei.

You can use https://docs.rs/tonic-build/0.6.2/tonic_build/struct.Builder.html#method.type_attribute which delegates to https://docs.rs/prost-build/0.9.0/prost_build/struct.Config.html#method.type_attribute which has an example for deriving Serialize and Deserialize.


Jasmine tests not passing

Three Jasmine tests for unformat are not passing in master.

These seem to be because of brackets, i.e. '()' being present in the string.
So a fix for issue #121 would fix  this issue as well.

I've proposed PR#136 to fix this.


[Change] Get personal best from api

Get personal best from api instead of local storage

Implemented with c2f756ff75431dd51fedd3d754a70d8437b12e14


More fixing of release script

Changes

extra cleanup of release script

/lgtm

/cherry-pick release-0.5


Remove rails dependency

I use an app that is Sinatra based but uses ActiveRecord. I'd like to add this in, but I don't want the overhead of needing all of rails. That and the fact that Sinatra 1.4.7, currently the latest, forces Rack at 1.5. I am using Active Record 5, so its using Rails 5, which needs actionpack 5, which needs rack 2.0, which Sinatra does not support.
If you only required ActiveRecord and Active Support, it would be just as functional, but more compatible, and you'd be requiring less, which is good.

Actually wrote a PR for you.

closed by #14


Set direction in constructor?

Many other GPIO libraries require you to set the direction at the same time the pin is declared for use - either via the constructor or some other initialization function.  This makes sense, because you're probably never going to change the direction of a pin during execution without rewiring your project.
Would it make sense to do the same thing with the Pin class?  Basically change its constructor to include a $direction parameter?  Or alternatively create separate InputPin and OutputPin classes, where the setValue() method is only available on the latter?
I'd be happy to submit a PR for either change if you're interested.

@colinodell - I like this idea, and I think there is an overlap here with the other issue that you created (https://github.com/PiPHP/GPIO/issues/4).
At the moment the library is a very thin wrapper over the sysfs GPIO driver, but I think it can become a lot easier to use with some of the changes you mention.
I'd gladly accept a PR on this if you're willing :)
I can see scenarios where you might want a pin to change (if you created plugs for the GPIO and had different modules that might be plugged in or not) - but it wouldn't stop you from making those changes. The calling code could easily just call the factory again asking for a new pin.

What do you think about having:
PinFactoryInterface::getInputPin(): InputPinInterface;
PinFactoryInterface::getOutputPin(): OutputPinInterface;

Then we could put the suitable methods onto each interface.

Yeah that would be perfect!


Revert "Merge pull request #97 from qurator-spk/420-namespace-package"

This reverts commit fd56b86acf55677dc7a8bfb9e2737c3cc167327a, reversing changes made to ea792d1e4ac4a722770b82dc91e71f84d5beb212.
This is the second attempt, same reasoning as in #107 (creating an upstream ref for ocrd_all), but different target: We want to avoid #86 (entirely), but also #97. We cannot just revert to v0.2.0 though, because we do need #100. So this is #100 plus a revert of #97.

This is already merged it seems.

Yes, the changes introduced by this PR so far are all in main already - but also the PR is still marked as draft.
@bertsky what do you consider the status of this?

Oh my... confusion on all levels!

This is already merged it seems.

No, it's not. You can clearly see a diff here.
Also, it's not even meant to be merged (only provide a reference for ocrd_all as long as problems caused by #86 and #97 are not fixed on main), hence the draft status. See reasoning in #107.

Yes, the changes introduced by this PR so far are all in main already - but also the PR is still marked as draft.
@bertsky what do you consider the status of this?

Again: #86 is broken for OCR-D. So is #97. But we still do need #100 (which obviously came later). So this is effectively a state before #86 plus cherry-picked #100.

@bertsky I see...sorry for the confusion and thanks for clarifying (again).

Unfortunately, I'll have to add new changes to this PR, which will likely be needed on master as well: Since https://github.com/OCR-D/core/pull/1079, we must correctly distinguish between local_filename and url.
If all else fails, please consider just cherry-picking from here on.

Moreover, since @kba says local_filename will now officially be : Path, we need yet another fix here.

I think I have cherry-picked all the commits to #130


ARROW-13816: [Go][C] Implement Consumer APIs for C Data Interface in Go

This supercedes some portions of #10995 as I think it's better to centralize the cdata interface interactions in a specific module that is part of the arrow module rather than it being a part of the dataset module. So this should get reviewed and merged first.
CC @pitrou @emkornfield

The failure in the integration test is an issue with the rust implementation that is unrelated to this change it seems.

Yes, the integration test failure is unrelated.

Hmm, I feel incompetent to review Go code.
What would be interesting would be to validate this using PyArrow, for example:

have a Go test module that exposes some C APIs that import a Schema or Array and test something with it (like you did with panic calls)
have a Python test module that loads the Go test module using ctypes, export a Schema or Array, and gives the exported pointer to the Go test APIs

@pitrou I'm rather happy with myself at the moment since I've managed to actually pull your suggestion off, though i'm not sure the best way to actually incorporate such a test into the current build system. For posterity, here's some snippets showing what I did:
Created a go file with the following:
package main

import (
    "fmt"
    "unsafe"

    "github.com/apache/arrow/go/arrow/cdata"
)

// #include <stdint.h>
import "C"

//export importSchema
func importSchema(ptr C.uintptr_t) {
    sc := (*cdata.CArrowSchema)(unsafe.Pointer(uintptr(ptr)))
    fmt.Println(cdata.ImportCArrowSchema(sc))
}

func main() {}

In order to build a c-shared library it apparently needs to be a main package. Anyway, the above file will export a function "importSchema" which takes a uintptr_t which is a pointer to an struct ArrowSchema. Running the command go build -buildmode=c-shared -o gotest.dll . if on windows or using gotest.so for linux. Then in python I did the following:
>>> import pyarrow as pa
>>> from pyarrow.cffi import ffi
>>> c_schema = ffi.new("struct ArrowSchema*")
>>> ptr_schema = int(ffi.cast("uintptr_t", c_schema))
>>> pa.schema([('ints', pa.list_(pa.int32()))], metadata={b'key1': b'value1'})._export_to_c(ptr_schema)
>>> from ctypes import *
>>> gotest = cdll.LoadLibrary('gotest.dll') # or gotest.so
>>> gotest.importSchema(c_ulonglong(ptr_schema))
schema:
   fields: 1
     - ints:  type=list<item: int32>, nullable
   metadata:  ["key1": "value1"]
0

So all in all, really cool. But I don't know where in the tests would make sense for how I should incorporate such a test where I build the go dynamic lib and then run the a python script with pyarrow to call the function. Any ideas?

Well, I don't know how Go runs tests, so I'm not able to answer that question. Perhaps @sbinet or @fsaintjacques have an idea.

Note that the CI runs the scripts https://github.com/apache/arrow/blob/master/ci/scripts/go_build.sh and https://github.com/apache/arrow/blob/master/ci/scripts/go_test.sh as part of its Docker routine: https://github.com/apache/arrow/blob/master/docker-compose.yml#L1196-L1199

so, the go tests just run by running go test which executes the tests. It's more that I'm not sure the best way to use pyarrow in the tests here. Should I modify the docker so it actually builds the entire c++ -> python toolchain to build pyarrow? Or should I just add python to the docker image and then just pip install pyarrow in the go_test.sh script?

Ah, you can just pip install pyarrow, fortunately.

Ok, i'll muck around with the tests and see if i can easily put something together to do that as an integration test then.

@pitrou I've updated the test cases to add a test which installs pyarrow and runs a python script against a built golang shared object module to test and confirm exporting a schema and record batch from python to Go and Go importing it correctly.  :smile:

@pitrou @emkornfield @sbinet @fsaintjacques
Any chance I might be able to get an approval on this? I have POC for implementing the producer APIs but it relies a lot on this same consumer code so I'd rather file it as a subsequent PR rather than make this one bigger.

I don't really read Go. I can try to give general advice but some Go developer will have to review the implementation.

@zeroshade I pushed some changes to improve the Python tests and try to fix some allocation issues. Can you double-check them?

@pitrou I'm updating the comments on the exported Import functions in interface.go so that they explicitly state that ArrowSchemaRelease will be called on the passed in schema regardless of error response or not.

@emkornfield @sbinet @fsaintjacques would one of you be willing to just give this a once over and approve? While I am a committer now, I'm not sure I wanna go merging this without at least one approval on it haha. :smile:

@pitrou I know you're not completely comfortable reviewing the Go code, but given that i've been unable to get someone else to come comment on it, do you have any particular qualms with me merging this as it's holding up some other development that I want to get PRs up for with the Go library.

@zeroshade I'm fine with you merging this. Go ahead, and let's move forward :-)


support 3D slide

This is a (multiple allowed):

[ ] bug

[ ] enhancement

[x] feature-discussion (RFC)

Swiper Version: EXACT RELEASE VERSION OR COMMIT HASH, HERE.

Live Link or JSFiddle/Codepen or website with isssue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

What you did
support 3D slide.
Expected Behaviorsupport 3D slide
Actual Behavior
support 3D slide

There is a cube transition effect, see docs

cube

no,It didn`t support touch event,when I touch stop,it will have a angle,when I touchend ,then flip


Bump AV2P version

I'm not sure I like where I ended up here. I ran in to an issue with a different project I'm doing receiving ZMQ from acars_router's ZMQ Pub Server.
I found two problems:

I was improperly binding the socket to loopback instead of the interface.
It appears either the library I'm using in Node to Sub to acars_router Pub socket cannot handle a blank topic, or Rust's tmq lib cannot handle blank topics.

I'm not sure which it is. What I ended up doing was having the ZMQ Server publish each message with and without a topic....I don't know if this breaks any sub's listening to the blank topic or not. I suspect not, but, who knows.

After playing around with it, I'm not really sure if my issue was nothing more than just simply not binding the interface correctly. I've removed the double sending of a message (which broke any Sub consumer listening on "" topics....they'd get two copies of the message) and it now will send all messages on the ZMQ Pub socket with a topic of acars. This SEEMS like it fixes the issue, and my testing with python and node Sub consumers seems to indicate that they will process any message sent on any topic if they are subbed to a blank topic.


Ensure --with-vendor-url is not blank, so plain hotspot can be built

Ensure --with-vendor-url  is set to a value.
Also disabled shellcheck globbing SC2086, as many warnings and most needed.
Fixes: https://github.com/adoptium/temurin-build/issues/3283

jdk8u failures, are known upstream problem, unrelated to this.


Accept Python 3 when NODE_GYP_PYTHON3 is defined

Checklist

[x] npm install && npm test passes
[x] tests are included
[x] commit message follows commit guidelines

Description of change

This depends on https://github.com/nodejs/node-gyp/pull/1812, I'll rebase when that PR lands.
This allows for Python 3 to be tested without breaking node-gyp for users.
This uses two new environment variables: NODE_GYP_FORCE_PYTHON is the only Python binary checked when it is defined (so we can be sure of what version is being tested in Travis), and NODE_GYP_PYTHON3 conditionally enables Python 3.
When Python 3 is fully supported, enabling it unconditionally should be as simple as removing NODE_GYP_PYTHON3, which is only used on one line.
This includes two commits. One just moves Python detection to a new file. The other contains the rest of the changes. Separately, they should be easy to review.
The tests that check for a "Python version too new" were deleted. Hopefully we won't need them again.

CI: https://ci.nodejs.org/view/All/job/nodegyp-test-pull-request/141/

When Python 3 is fully supported

I thought we were there already? Don't we have Travis tests for this?  The sooner we get to gyp.js the better.

I thought so too. Node-gyp was doing what it's supposed to do, find a
compatible version of Python somewhere and run it. The tests don't show
info output so we had no confirmation of version. All the Linux machines
have Python 2 installed, so the tests pass with no errors.
Thia shows that when actually using Python 3 the tests don't pass, but I
don't know how hard those issues are to fix.

Thia shows that when actually using Python 3 the tests don't pass, but I don't know how hard those issues are to fix.

At least https://travis-ci.com/nodejs/node-gyp/jobs/214237690#L2234-L2241
    +Traceback (most recent call last):
    +  File "/home/travis/build/nodejs/node-gyp/gyp/gyp_main.py", line 13, in <module>
    +    import gyp
    +  File "/home/travis/build/nodejs/node-gyp/gyp/pylib/gyp/__init__.py", line 10, in <module>
    +    import gyp.input
    +  File "/home/travis/build/nodejs/node-gyp/gyp/pylib/gyp/input.py", line 7, in <module>
    +    from compiler.ast import Const
    +ModuleNotFoundError: No module named 'compiler'

looks like it would be fixed by https://github.com/refack/GYP/commit/f989ef9f1c41f04267eb18bed24b880ad0f8b9ec.

@cclauss I don't understand which variable you'd like to change. Changing NODE_GYP_FORCE_PYTHON would break the point of having it in the first place, and changing NODE_GYP_PYTHON3 would break node-gyp for end users. My goal with these variables is:

NODE_GYP_FORCE_PYTHON: To have a way of forcing node-gyp to use a certain Python binary and not trying anything else. This is useful in Travis to specify a Python binary to use. I understand this adds some lines to .travis.yml, but when we're sure there is no other Python version in the machines we can remove this (hopefully next year Python 2 will start to disappear). I also plan to use this to add a test where python is a bat file, which is common in Windows. I think this is a good mechanism to have, even in the future when there is only Python 3.
NODE_GYP_PYTHON3: Python 3 needs to be opt-in for now, because it doesn't work. If we make this NODE_GYP_PYTHON2, then every user who wants to use node-gyp would have to set this before running! I purposefully used this variable only in one line: https://github.com/nodejs/node-gyp/blob/7bcc33b25f435be2da47457bba7fd4fb76351f55/lib/find-python.js#L21 So, when Python 3 works we can simply change that line to semverRange: '>=2.7.0', remove the variable from .travis.yml and be done with it.

I like the idea behind NODE_GYP_FORCE_PYTHON.  It will not be needed on Travis CI Linux because those jobs run in virtualenvs where by definition there is only one Python which is determined by the python: tag.  On Travis CI Mac, NODE_GYP_FORCE_PYTHON plays a very useful role because both python2 and and python3 will be present and you know the Windows story better than I do.
I would like to resist the NODE_GYP_PYTHON3 variable unless we have an agreed upon method of deprecating it.  My sense is that once we introduce new configuration vocabulary, it is nearly impossible to drop it.  For example, our README.md clearly states that we no longer support Python 2.6 yet contributors continue to try to push it back into our scripts.
As I wrote above, Python 3 should the default and Python 2 should be the opt-in.  Python 3 has been available for 8+ years and is the present and future of the language.  Python 2 is legacy and is no longer supported at yearend.  Therefore, I would much rather see us introduce a variable called NODE_GYP_PYTHON2 and force those who wish to remain on legacy Python to set that variable.  I advocate that continuing to rely on a soon-to-be-EOLed platform should to be a deliberate, conscious, and documented choice.

@cclauss I might be missing something here, but I don't understand where you're trying to get to. Node-gyp does not work with Python 3, as Travis shows. By making Python 3 the default, node-gyp will be broken by default for every user. How is that acceptable?

This PR is supposed to make node-gyp Python 3 compatible.  We already thought it was with all the tests passing until you demonstrated to us that find_python.js was actually preventing us from finding Python.  I have been on this mission for a year now...

This PR does not make node-gyp Python 3 compatible, it only allows testing with it.
find-python.js is doing what it's supposed to do, produces a lot of logging at several levels and clearly indicates what version of Python it chose and why. If you run node-gyp locally on any project, you'll see that output. I also added it to Travis here.
The only test that actually runs node-gyp end-to-end is test-addon.js. Those tests are not supposed to have any output, so they're not suitable to test this kind of change. With this PR we can explicitly select an executable to run, which should allow us to reliably test different versions of Python from now on.
@cclauss it's great that you can put some effort into making node-gyp Python 3 compatible, thank you for moving that forward. I can't look into it myself, but I think this is a very important goal and that's why I'm trying to help where I can. With all changes that happened so far, I was not finding it surprising that node-gyp was already compatible until I started looking into the Windows failures. I hope that even if this is a bit disappointing, there won't be much left to do.

The error ModuleNotFoundError: No module named 'compiler'  is caused by the fact that the compiler package was removed in Python 3 in favor of the ast module that exists in both Python 2 and Python 3.
https://github.com/refack/GYP/commit/f989ef9f1c41f04267eb18bed24b880ad0f8b9ec

I renamed NODE_GYP_PYTHON3 to EXPERIMENTAL_NODE_GYP_PYTHON3, I hope this helps it be clear that the variable is not to be used by end users, only for testing. Also excluded outdated versions 3.0-3.4 of Python if it is used.
@cclauss when this lands (or even before by depending on this), fixing that error should require only picking those changes and adapting for any possible difference.

Thanks for our WebEx on this topic and the resulting changes that you have made.  If you can fix the Travis CI Py2 issues then I will give this PR a LGTM.  It is a bummer that our Py3 tests now fail but at least with PR we are able to run the desired version of Python.  In this PR, we can leave th Py3 Travis tests in allow_failures  mode and then we can work to make those tests green in future PRs.

@cclauss fixed, thanks!
CI: https://ci.nodejs.org/view/All/job/nodegyp-test-pull-request/142/

Landed in https://github.com/nodejs/node-gyp/compare/bb92c76...66ad305

Why have we created a new NODE_GYP_FORCE_PYTHON environment variable when there is already a well documented gyp --python option?  Which one takes precedence?

The --python option is only a hint, it will be used if it can be used. If it can't be used, node-gyp will look for something else. I can elaborate on the rationale of how this works a bit:

Node-gyp also checks the PYTHON environment variable. As I understand it, this variable is quite common and might be defined for some other project that users are working on, or might even be a forgotten variable from some past project that was never undefined. Therefore, I don't think it makes sense for node-gyp to fail immediately if it points to an invalid version of Python, it should keep looking for a Python of the right version.
The --python option is the same as defining NPM_CONFIG_PYTHON or setting python in npm config. This is a harder case to make, but I think the same logic should (and currently does) apply here. We have plenty of users who are not familiar with node-gyp, and at some point in the past might have set the npm configuration for some reason (actually, we still recommend this in the README, and shouldn't remove until v5.0.0 becomes the default). If this Python is of the versions that we accept, we use it. If not, just look for something else. I think most users want node-gyp to work and not have to worry about this.
The NODE_GYP_FORCE_PYTHON is not a hint. It's either this or nothing. This variable is only used for node-gyp.

Note that the verbose log output shows clearly what is used, what can't be used, by what order and why.
Precedence works like this:

If NODE_GYP_FORCE_PYTHON is defined, check it's version and use it, or fail. Stop here, don't look for anything else.
Check --python/NPM_CONFIG_PYTHON.
Check PYTHON.
Look for a python in the PATH.
Look for a python2 in the PATH.
Try to use the py launcher on Windows.
Try the default locations on Windows.

The code is here: https://github.com/nodejs/node-gyp/blob/66ad30577594a37b08939948db1d4f1018d85335/lib/find-python.js#L47-L125 I wrote that hoping that at least that part would be easy to read and extend.


Reset Password not working Laravel 5.4.

Hi Guys,
I've tried to implement password reset, also with 'Jenssegers\Mongodb\Auth\PasswordResetServiceProvider::class' , but there are certain mistakes
First: Jenssegers\Mongodb\Auth\PasswordBrokerManager::class is big mistake with missing parameter in function, should look like this :
protected function createTokenRepository(array $config)
{
return new DatabaseTokenRepository(
$this->app['db']->connection(),
$this->app['hash'],  // this is missing
$config['table'],
$this->app['config']['app.key'],
$config['expire']
);
}
And after that is problem with: Cannot use object of type MongoDB\BSON\UTCDateTime as array
It would be helpful to fix it,
Thanks

Same problem here.. Any solution?

Same problem here

Any solution?

Same problem here

Same issue here as well. Any fix planned?

Test the above and working for me.

Try this solution but I still get the same error: Can not use object of type MongoDB \ BSON \ UTCDateTime as array
Someone else found some other solution for this error.

My dirty hack for reset issues:
Add the following to Auth/ForgotPasswordController and Auth/ResetPasswordController to make it work (requires php 7):
/**
 * Get the broker to be used during password reset.
 *
 * TODO: remove this method once Jenssegers\Mongodb reset password issues are fixed
 *
 * @return \Illuminate\Contracts\Auth\PasswordBroker
 */
public function broker()
{
    return new class(app()) extends \Jenssegers\Mongodb\Auth\PasswordBrokerManager {

        protected function createTokenRepository(array $config)
        {
            $c = $this->app['db']->connection();
            $h = $this->app['hash'];
            $t = $config['table'];
            $k = $this->app['config']['app.key'];
            $e = $config['expire'];

            return new class($c, $h, $t, $k, $e) extends \Illuminate\Auth\Passwords\DatabaseTokenRepository {

                protected function getPayload($email, $token)
                {
                    return ['email' => $email, 'token' => $this->hasher->make($token), 'created_at' => new UTCDateTime(time() * 1000)];
                }

                protected function tokenExpired($token)
                {
                    // Convert UTCDateTime to a date string.
                    if ($token instanceof UTCDateTime) {
                        $date = $token->toDateTime();
                        $date->setTimezone(new DateTimeZone(date_default_timezone_get()));
                        $token = $date->format('Y-m-d H:i:s');
                    } elseif (is_array($token) and isset($token['date'])) {
                        $date = new DateTime($token['date'], new DateTimeZone(isset($token['timezone']) ? $token['timezone'] : 'UTC'));
                        $date->setTimezone(new DateTimeZone(date_default_timezone_get()));
                        $token = $date->format('Y-m-d H:i:s');
                    }

                    return parent::tokenExpired($token);
                }
            };
        }
    };
}

As you see there are 2 issues:

token should be stored as a hash ($this->hasher->make($token))
tokenExpired gets $token which doesn't require $token['created_at'] anymore

any update ?

Thank you so much @mikekamornikov  this solution working very well
it just need to add
use MongoDB\BSON\UTCDateTime;
also
new \DateTimeZone
if someone tried this solution and not worked , you just need to make a new reset request
example.com/password/reset
##then
try the new token

Hey @mikekamornikov, I made all you wrote, but doesn't work.
I'm getting this error: DateTime::__construct(): Failed to parse time string<PHONE_NUMBER>000) at position 11 (0): Unexpected character
Help me please!

@joerecra I need more context. Where does this error come from (trace)?

In Config/App.php replace Illuminate\Auth\Passwords\PasswordResetServiceProvider on Jenssegers\Mongodb\Auth\PasswordResetServiceProvider
In App/Http/Controllers/Auth/ResetPasswordController.php add use MongoDB\BSON\UTCDateTime;
Timezone: Europe/Moscow
Laravel: 5.7.*
Jenssegers/mongodb: "^3.4"


Make it possible to require custom profile fields

Organization administrators may want to require users to fill out some profile fields. To support this, we should:

Add a "Required field" checkbox to the bottom of the "Add a new custom profile field" and "Edit custom profile field" modals.
Add a "Required" column after "Card" in the SETTINGS / CUSTOM PROFILE FIELDS panel.
If a user has any empty required custom profile fields, show a warning banner at the top of the app:

Your profile is missing the following required fields: X, Y. Edit your profile

We should discuss how dismissable this banner should be when we are implementing this feature.

CZO thread

@zulipbot claim

@zulipbot claim

@zulipbot  claim

@haroon10725, are you working on this issue?

@zulipbot abandon

@haroon10725, are you working on this issue?

You can work I have abandon the issue

@zulipbot claim

@kartikbhtt7, i have already made some progress on this one, can you pick something else

@notlelouch okays, you can work on this
@zulipbot abandon

@zulipbot claim

@zulipbot abandon

@zulipbot claim

@zulipbot claim

@zulipbot claim

Hi,
I am contributing for the first time so I was troubleshooting environment related issues.
I will take this up further.

@zulipbot claim


gzip: database//mgnify/mgy_clusters_2018_12.fa.gz: No such file or directory

I got this error when running download_all_data.sh, which runs download_mgnify.sh.
I looked into the latter and these lines were causing the error.
pushd "${ROOT_DIR}"
gunzip "${ROOT_DIR}/${BASENAME}"
popd
It went to the directory to unzip the file, but the path was wrong. The ${ROOT_DIR}/ should be removed.

Did you run the command line like this. I had the same error but it worked when I assigned a directory.
download_pdb70.sh directory_path

Did you run the command line. I had the same error but it worked when I assigned a directory. /home/usrn/alphafold/scripts/download_pdb70.sh directory_path
If you are in the alphafold directory then it should look like this scripts/download_pdb70.sh directory_path
One thing also is that the directory_path should have a lot of storage. I got an error when the directory I chose was sub 2TB, I had to get a 8TB SSD to not run across the same issue using the full database.

Sorry for the typo. I meant download_mgnify.sh should be edited.

Oh it should still work just edit the .sh file and your directory_path. Hopefully it works for you.

Did you use full path or relative path? You should use the full directory path for this to work.

Did you use full path or relative path? You should use the full directory path for this to work.

I see. I used a relative path. Thank you!
Since an absolute path is required, the pushd and popd statements are actually removable because gunzip decompresses a zip file in the same place and you don't need to go to that directory.
Because many people prefer to feed a relative path, I think it is better to make it accept relative path by removing the "${ROOT_DIR}/" from the gunzip "${ROOT_DIR}/${BASENAME}" while keeping the pusd and popd. It will work for both absolute and relative paths.


Fixed bundler incorrectly printing "the gem may have been yanked" in local installs

Closes #5022.

Thanks!
@homu r+

:pushpin: Commit 05d6f50 has been approved by segiddins


Problem saving dates

I'm trying to change the raw.info['subject_info']['birthday'] field in some files and am unable to do so. Here is the code I'm using.
raw = Raw(f, preload=False, verbose=False) raw.info['subject_info']['birthday'] = (01, 01, 1900) raw.save('new_fif_raw.fif)
when I open up new_fif_raw.fif to check if it worked I still see the original birthday. I change other fields and they update successfully. What is going on?

birthday is not a standard key in the measurement info. We don't write all
the keys to disk but just the standard ones.

raw.info['subject_info']['birthday'] = (01, 01, 1900)

This line has a SyntaxError in it -- 01 isn't a valid integer, you probably want 1. Perhaps you are hitting this error and not noticing (running the .save() line anyway)?
Also, note that the order is Y/M/D not M/D/Y, so you'd want to do:
raw.info['subject_info']['birthday'] = (1900, 1, 1)

Verifying:
>>> import os.path as op
>>> import mne
>>> raw = mne.io.Raw(op.dirname(mne.__file__, 'io', 'tests', 'data', 'test_chpi_raw_sss.fif')
>>> raw.info['subject_info']['birthday']
(1983, 12, 19)
>>> raw.info['subject_info']['birthday'] = (1900, 1, 1)
>>> raw.save('test_raw.fif')
>>> mne.io.Raw('test_raw.fif').info['subject_info']['birthday']
(1900, 1, 1)

Can you confirm?

@agramfort it actually does live in subject_info :)

@agramfort https://github.com/agramfort it actually does live in
subject_info :)

oups...

@gcwarner1, am I right in thinking you noticed this field doesn't get changed in the anonymize function? Do you have code that fixes that?

@dgwakeman no, it should be deleted (in both MNE-C and mne-python) because the entire info['subject_info'], which ends up being written as its own block in the FIF file, gets deleted, see:
https://github.com/mne-tools/mne-python/blob/master/mne/io/meas_info.py#L373

Looks like the anonymize function works for this. However, it does not edit or delete raw.info['meas_date']. Shouldn't that be included in the anonymization function as it is technically PHI?

Looks like the anonymize function works for this. However, it does not edit or delete raw.info['meas_date']. Shouldn't that be included in the anonymization function as it is technically PHI?

I'm -.5 on this, the rec date isn't an individual feature, and is critical when there's a mixup in subjects filenames.

although it definitely qualifies as PHI see #3:
http://cphs.berkeley.edu/hipaa/hipaa18.html

Fair enough

@mshamalainen do we need to add the measurement date to mne_anonymize?
It's easy enough to add it to mne-python in any case. It could be an argument to .anonymize(meas_date=None) with a deprecation cycle False->True behavior. @agramfort do you think we should add it?

mne-c does not do it but that seems legit
@mshamalainen thoughts?

When I wrote me_anonymize, I didn't think the measurement date would be PHI.

Ok. Can you take care of amending the mne anonymize function and we ll do the same in the python code?

me_anonymize now removes FIFF_MEAS_DATE as well

@mshamalainen shoudl we also remove the SECS, MSECS, and USECS from FILE_ID and MEAS_ID to remove traces of the recording date? That's what I did in  #3041


Audio files not appearing or dragging into ONLYM

Describe the bug
A clear and concise description of what the bug is.
THE BUG
Tried adding a WAV FILE (.wav) and an M4A FILE (.m4a) but it hasn’t worked. The files appear in the media folder but not on the application.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Windows version
Other important information about your environment

@NorthwichVerdinPark Both audio formats added in latest pre-release (<IP_ADDRESS>)


New version: Jsoftware.J.9.5 version 9.5.0-beta4

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

@microsoft-github-policy-service agree company="Jsoftware"


Add jsxFrag pragma

Description
If a component is specifying a jsx pragma and is using <></> Fragment shorthand, they must also specify a fragment pragma.
https://babeljs.io/docs/en/next/babel-plugin-transform-react-jsx.html#fragments
Without this, i'm getting an error when using docz in conjunction with a plugin:
node_modules/gatsby-theme-docz/src/components/Sidebar/index.js: transform-react-jsx: pragma has been set but pragmafrag has not been set
Review

[ ] Check the copy
[ ] ...
[ ] ...

Pre-merge checklist

[ ] ...
[ ] ...

Screenshots

Before
After

Image
Image

Thank you @tamagokun for catching this and fixing it !

Released in 2.0.0-rc.76


Safely allow multiple image capture requests to be serially processed

Don't want to take this approach.


Sign up / Sign in errors messages

Show to user sign up and sign in errors

Process sign in / sign up error messages int alert

Resolved


Actual PowerShell version of scripts

The commands proposed are valid in Az Cli, not PowerShell.
I'm using Az* PowerShell module, including Connact-azAccount, that does not work with the az cli commands, that require az login.
Please provide valid code samples.

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

ID: 78e65bf3-59d7-4b6d-f45e-884f4ea16cbd
Version Independent ID: 49fbcedf-e5ab-5a83-edba-3609f955756f
Content: Resize a virtual machine - Azure Virtual Machines
Content Source: articles/virtual-machines/resize-vm.md
Service: virtual-machines
GitHub Login: @ericd-mst-github
Microsoft Alias: cynthn

@ericd-mst-github
Thanks for your time on my issue. I'm using PowserShell on my computer.
I'm testing it locally before including that in our scripted deployment environment.
NB: I found a way updating the disk Update-azDisk, but I'm still disappointed to not have powershell commands in that doc :-(

Thank you for your response. We will work on updating this current doc for adding the valid local PowerShell samples. I'll relay your feedback for the Update-azDisk doc as well.

@fauberon I have the updated article staged but wanted to check with you on the updated local script sample that was tested recently.
Does the following PS script work for you locally? Please feel free to update the relevant variables as you see fit also.
# Import the Azure module
Import-Module Az
# Login to your Azure account
Connect-AzAccount
# Set variables
$resourceGroup = 'myResourceGroup'
$vmName = 'myVM'
$size = 'Standard_DS3_v2'
# Select the subscription
Select-AzSubscription -SubscriptionId '<subscriptionID>'
# Get the VM
$vm = Get-AzVM -ResourceGroupName $resourceGroup -Name $vmName
# Change the VM size
$vm.HardwareProfile.VmSize = $size
# Update the VM
Update-AzVM -ResourceGroupName $resourceGroup -VM $vm

@ericd-mst-github
Thanks.
Yes, this is the PS commands I executed. Can you add this precision: this will require the VM to restart to take effect.
On my side: I stop the VM then restart it.
NB: I mentioned about the AzDisk, mixing my topics...


box webbrowser sdk crushed after lunch.

I heard BoxDev 2015 keynote and Release iOS Web browser SDK.
i tried demo code.

    [BOXContentClient setClientID:@"My Client ID"
                     clientSecret:@"My Client Secret"];

    BOXContentClient *contentClient = [BOXContentClient defaultClient];
    ...
    ...

I compled and run code, after crush. log is
2015-04-22 12:51:08.073 boxbrowser[20630:768377] -[__NSCFData hasPrefix:]: unrecognized selector sent to instance 0x7fa2f6804a00
2015-04-22 12:51:08.080 boxbrowser[20630:768377] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[__NSCFData hasPrefix:]: unrecognized selector sent to instance 0x7fa2f6804a00'
*** First throw call stack:
(
	0   CoreFoundation                      0x000000010965ec65 __exceptionPreprocess + 165
	1   libobjc.A.dylib                     0x00000001092f7bb7 objc_exception_throw + 45
	2   CoreFoundation                      0x00000001096660ad -[NSObject(NSObject) doesNotRecognizeSelector:] + 205
	3   CoreFoundation                      0x00000001095bc13c ___forwarding___ + 988
	4   CoreFoundation                      0x00000001095bbcd8 _CF_forwarding_prep_0 + 120
	5   boxbrowser                          0x00000001087345d3 +[BOXOAuth2Session usersInKeychain] + 787
	6   boxbrowser                          0x0000000108702c5e +[BOXContentClient users] + 46
	7   boxbrowser                          0x00000001087017fc __33+[BOXContentClient defaultClient]_block_invoke + 60
	8   libdispatch.dylib                   0x000000010d943614 _dispatch_client_callout + 8
	9   libdispatch.dylib                   0x000000010d92f881 dispatch_once_f + 565
	10  boxbrowser                          0x000000010870179d +[BOXContentClient defaultClient] + 141
	11  boxbrowser                          0x00000001086b785b -[ViewController viewDidLoad] + 123
	12  UIKit                               0x0000000109cdb210 -[UIViewController loadViewIfRequired] + 738
	13  UIKit                               0x0000000109cdb40e -[UIViewController view] + 27
	14  UIKit                               0x0000000109bf62c9 -[UIWindow addRootViewControllerViewIfPossible] + 58
	15  UIKit                               0x0000000109bf668f -[UIWindow _setHidden:forced:] + 247
	16  UIKit                               0x0000000109c02e21 -[UIWindow makeKeyAndVisible] + 42
	17  UIKit                               0x0000000109ba6457 -[UIApplication _callInitializationDelegatesForMainScene:transitionContext:] + 2732
	18  UIKit                               0x0000000109ba91de -[UIApplication _runWithMainScene:transitionContext:completion:] + 1349
	19  UIKit                               0x0000000109ba80d5 -[UIApplication workspaceDidEndTransaction:] + 179
	20  FrontBoardServices                  0x00000001102a35e5 __31-[FBSSerialQueue performAsync:]_block_invoke_2 + 21
	21  CoreFoundation                      0x000000010959241c __CFRUNLOOP_IS_CALLING_OUT_TO_A_BLOCK__ + 12
	22  CoreFoundation                      0x0000000109588165 __CFRunLoopDoBlocks + 341
	23  CoreFoundation                      0x0000000109587f25 __CFRunLoopRun + 2389
	24  CoreFoundation                      0x0000000109587366 CFRunLoopRunSpecific + 470
	25  UIKit                               0x0000000109ba7b42 -[UIApplication _run] + 413
	26  UIKit                               0x0000000109baa900 UIApplicationMain + 1282
	27  boxbrowser                          0x00000001086b7b9f main + 111
	28  libdyld.dylib                       0x000000010d977145 start + 1
)
libc++abi.dylib: terminating with uncaught exception of type NSException

maybe it is my environment problem. Because, I reset ("Reset content and setting" ) iPhone simulator, then it worked.

@salexkidd Thanks for this report. Glad it worked once you reset your simulator. Please do let us know if you encounter it again. This crash looks to have happened while the SDK was trying to read from the keychain, so as you say, it might have been caused by an environmental issue.

The bug causing this was found to be in the Content SDK.
We will apply a fix there: https://github.com/box/box-ios-content-sdk/pull/19

Closing since it has been fixed with Content SDK 1.0.2


Crashes using Aldec Riviera on Ubuntu 16.04 LTS

Dear developer(s),
When using Aldec 32 bit under Ubuntu 16.04 LTS, Cocotb crashes often with a segmentation fault. I have been able to trace down the problem and fixed it in my local repository.
Problem was caused by faulty pointer passing between C and Python. Under Ubuntu 16.04 LTS, the C
library alloc (or loader?) causes virtual addresses to be used in the upper half of the 32 bit address space. When using a 32-bit simulator and 32-bit Python with Cocotb, the integer value in Python does not convert correctly back to a C struct pointer. The negative integer cannot be converted using an "Unsigned" conversion function. The resulting pointer is -1, which causes a SEGFAULT.
I fixed this by changing the cast back to a C struct as follows  (occurs at various places in simulatormodule.c):
    pSihHdl = PyTuple_GetItem(args, 0);
    hdl = (gpi_sim_hdl)PyLong_AsLong(pSihHdl);

.. instead of PyLong_AsUnsignedLong(...)
I hope this helps out other people as well. IMHO I think it would be better not to pass pointers to "C" objects directly into Python as integers, but keep them inside the realm of "C" and use some kind of lookup mechanism instead.
Best regards,
Gideon

Fixed by merging our pull request. Thank you!


Porting of the Grails Runtime-Datasources Plugin to Grails 3

I ported the plugin from Grails 2 to Grails 3. It can be published to bintray as seen here: https://bintray.com/alexanderfranke/plugins/alexanderfranke.plugins%3Aruntime-datasources/view
Also the tests run through on travis-ci, see here: https://travis-ci.org/alexanderfranke/runtime-datasources/builds/67339569

Thanks again for the contribution, I'll try to merge and publish it this evening and post a comment here once it's been completed.

I merged this manually on the command-line and have released it to bintray
https://bintray.com/domurtag/plugins/domurtag.plugins%3Aruntime-datasources/view
Thanks again.


Nicer output from SNMP check

Make it easier to identify whats going on by simply outputting the value for the SNMP returned and whatever expectations failed to be met:
CheckSNMP CRITICAL: 2.66 >= 2
CheckSNMP CRITICAL: 2.49 failed to match pattern 2.53
CheckSNMP OK: 2.65 matched pattern 2.[0-9]+

So what do we do about failures in other plugins when they're not anything to do with the change proposed ?


Ensure that var:declare works in combination with var:use to declare default scope

It doesn't but the question is whether it should..


Setup monitoring system

Deploy the first version of a system collect metrics from both browser and backed services.
I would recommend just look at https://docs.microsoft.com/en-us/azure/azure-monitor/azure-monitor-app-hub as a simple cloud native solution.
Do not need to automate the creation of a motoring service at this point. Lets get more azure experience before we try to automate everything.

Have chosen to likely use Azures own monitoring service. This requires no backend set up so ticket can be considered complete.
Keeping open until restapi and frontend log and monitor is done


Ignore Button Missing

I have a few entities that are unavailable until they are used (in the custom emby component)
Spook finds these entities, but I cannot ignore them. If I click submit, the repair goes away but reappears shortly afterwards:

Yeah, this is a bit of the "meh" part of the HA repairs.
This one is marked as "Submit will mark it fixed" (until the next detection/inspection run for Spook). However, the ones you can mark fixed in HA, you can't ignore? Which is a bit odd actually.
The reason why I allowed marking it as fixed, is because I haven't tried hooking into the updates of dashboards yet (which I would need to know when something changed and this determines if the issue is solved).
As I'm already hacking on this, I'll take a look if I can hook into listening for Lovelace changes.

I have a few entities that are unavailable until they are used (in the custom emby component)

PS: This is caused by the lack of unique IDs on those entities. Should generally be an easy fix for upstream...

OK thanks, I'll mention it to the emby custom component dev.

Found a way to automatically detect dashboard changes and let Spook clean up / update the raised repair issues by itself in #202 🤘
This behavioral change causes HA to enable the ignore button.
../Frenck

That did it, many thanks


Show high quality image version even for non selected images in gallery

What
We got a review:

Elle fait beuguer l'application photo

I (with help of a translator) guess that he things that the app is uploading these pixelated images.
We should definitely show the high res images when opening the full view of any image

@VaiTon can you help with that

Dublicate of #2960


DOP-5046: Prioritize PyMongo in wayfinding component

Ticket
DOP-5046
Notes

Seems like we want to prioritize PyMongo instead of Python. This accomplishes that.
Here's a staging link for comparison with prod.

README updates

[ ] This PR introduces changes that should be reflected in the README.md and/or HACKING.md, and I have made those updates.

[x] This PR does not introduce changes that should be reflected in the README.md and/or HACKING.md

Makes sense!


[Http] Removed URI Null Tests

Ignore this, closed after: https://github.com/slimphp/Slim/pull/1733

Closed for https://github.com/slimphp/Slim/pull/1733


[java] Add generics example

One of the recent merges to the jvm branch have broken the C# build. I'll work on a fix on Tuesday, Pacific time.

Fix for cs.sln project is ready for review:
https://github.com/Microsoft/bond/pull/563


How can i edit the pre-phish already created

Hello everyone, i am trying to edit the pre-phish already created to something of my choice instead of "download" or "*** shared a file with you" i'd prefer something else... i've tried editing the code and saved it, but when i click my link it doesn't seem to change, it still remains as the example. Please your help will be very much appreciated. Thanks in advance

We have given you the solution to your problem, currently the help here on Github is slow because very few people help, we have a private community that is updated and assisted 24/7 if you are interested in joining find me on Telegram @TheEvilGinx2 or by email at<EMAIL_ADDRESS>

alright i'm good now. thanks for the help @Support-1535


[WIP]: Multi namespace watch mode

[WIP]
Depends on https://github.com/aws-controllers-k8s/runtime/pull/125
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/hold

If we start supporting multi namespace we should also add in OwnNamespace SingleNamespace and MultiNamespace modes in the CSV. Code that would need to be updated is here

/unhold

/retest


signOutUser

Feature request

Is posible that method signOutUser removes all Google cache files? When you sign out and then sign in again, the account chooser doesn't show.. Next time you sign in, the login is silent with the previous account.
Account chooser only is showed at the first time you sign in. I don't know if this is a bug or is something that is not implemented yet.
Thank you in advange!

This looks to be a bug - well, a shortcoming in Firebase's use of Google SignIn anyway...
I had assumed Firebase would take care of signing out of Google when you signed out of Firebase, but apparently not: it seems you must manually sign out of both on Android and iOS.
I will test this scenario and implement a fix in the next plugin release if needed.

Hi!!
Thank you very much for your response and apreciate support!
B.R
El mar., 31 mar. 2020 8:51, Dave Alden<EMAIL_ADDRESS>escribió:

This looks to be a bug - well, a shortcoming in Firebase's use of Google
SignIn anyway...
I had assumed Firebase would take care of signing out of Google when you
signed out of Firebase, but apparently not: it seems you must manually sign
out of both on Android
https://github.com/firebase/FirebaseUI-Android/issues/187#issuecomment-225625136
and iOS https://stackoverflow.com/a/40107017/777265.
I will test this scenario and implement a fix in the next plugin release
if needed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dpa99c/cordova-plugin-firebasex/issues/353#issuecomment-606435652,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHANXLJWFGOYCCMWP4JS7DRKGHIFANCNFSM4LW6UH3Q
.

How can I test this changes?? Can I overwrite the commited file srcc/android/FirebasePlugin.java and build again? Thank you

You can install directly off the master branch until this is released in the next version to npm:
cordova plugin add https://github.com/dpa99c/cordova-plugin-firebasex

You can install directly off the master branch until this is released in the next version to npm:
cordova plugin rm cordova-plugin-firebasex --nosave && cordova plugin add https://github.com/dpa99c/cordova-plugin-firebasex --nosave

Hi!
It works like a charm!
But I had problems uninstalling the plugin and added with the above command, I think It was for version conflict with other plugins but I'm not sure and I didn't want "another plugin war" jajaja...
Finally I solved adding changes manually to  FirebasePlugin.java file existing in "plugins" and "node_modules" folders and rebuild until new version is released.
Thank you @dpa99c
Regards


JWT Support

@arekkas Since i'm a big fan of Hydra and also know that Hydra uses JWT, it would probably be good to add support in Fosite for JWT as well. Since JWT is gaining a lot of focus these days you know ;)
I really like the stateless behaviour of JWT and the ability to put arbitary data into it and lastly that you can validate it without a call to the database. That makes it more robust and easy to use in a multi-service environment, where multiple services need to validate the authenticity of a resource owner.
I know that with enigma, there should really be no hussle to do it quickly. Especially since you basically just can drag and drop the JWT folder of Hydra into Fosite and then modify enigma a little to support pluggable token generators.
What do you think? If you need help implementing anything, just shout out. I can help you! :+1:

I'm curious if developers understand my code :) The enigma interface is pretty much finalized so if you'd like give it a shot!
I think JWT support is a good idea! :) We might have to adapt the enigma API a little bit to get the request data into enigma, but just give it a shot and if you encounter problems ping me :)

I'll work on it after work the coming 2 days then! Expect it to be "working" by Wednesday!

Nice, looking forward :+1:

By the way, if you want to chat real time we could think of setting up a fosite or ory chat on https://gitter.im/

A quick update: JWT is now working in conformance to enigma interface. All unit tests pass. Will add this to the example tomorrow (a little sparse on spare time atm.) so a developer easily can switch between standard hmac256 or JWT (using RSA256, certificates). After that I'll create a PR so we can discuss further :)

nice! looking forward :)

Closed as per #11


Port existing functionality for creating a graph from c_api.h to a NIF.

So the description of the library is for utilizing models in elixir, not certain if you also want to head in the direction of being able to construct graphs as well. If so I'd like to try and add some functionality regarding building arbitrary operation graphs, nothing super fancy just construct and execute some operations with run_session afterward. Then if that goes well I could build up from there.

I believe we can't build graphs because there is no tensorflow C API for it? Or am I mistaking it by something else? 🤔

Uhh let me double check myself I just spent some time poking around it for a bit and I thought I saw the necessary functions.

So, ```
TF_CAPI_EXPORT extern void TF_GraphCopyFunction(TF_Graph* g,
const TF_Function* func,
const TF_Function* grad,
TF_Status* status);

Allows to add arbitrary functions to a graph and those functions appear to be a set of operations.

And https://github.com/tensorflow/tensorflow/blob/r1.10/tensorflow/python/framework/ops.py, lines 1537-1581 would suggest we may have the necessary functionality to create arbitrary ops for the graph from the c-api. Line 1576 TF_FinishOperation actually adds it.

Awesome, thanks! :)

Hello @GrandathePanda and @josevalim
Yes you can construct custom graphs with generated op functions in the C API (look in Current Status on this webpage: https://www.tensorflow.org/extend/language_bindings). But I have a question: how does this add to Tensorflex in a useful way and help users do anything related to ML/DL?
The problem is users still can't do ML or DL because there is no gradient computation support for training on custom created graphs in the Tensorflow C API. Again look under Current Status here: https://www.tensorflow.org/extend/language_bindings
If there are any relevant use cases for having simple custom graphs and ops then porting such functionality from the C API makes sense, otherwise I have a feeling this might complicate Tensorflex unnecessarily, without adding any value to it.
What do you think?

My impression was TF_CAPI_EXPORT void TF_AddGradients allows gradient computation and allows us to begin to build towards constructing actual ml/dl models It just doesn't seem to be documented on that status page.

There is also in the current c-api.h file TF_CAPI_EXPORT void TF_AddGradientsWithPrefix

Yes, while they have been added in some fashion, using them is not straightforward and requires linking some gradient C++ API functionality for successful compilation (which is not how compilation is referenced on the C API installation page). I did not find any documentation on doing that and only some arbitrary mentions on some issues on the tensorflow repo. Since they don't seem to work right out of the box, I think that's why they have not updated the status page.
However, if you can get gradients to work then I think custom graphs would make a lot of sense! :)
Would you be interested in taking a look at this? Even running gradient computation ops on a simple ML graph constructed in C via the C API would be a successful start to this. I am not sure if using them will be as easy as you say, though.

Certainly I would love to take a look!

Awesome! :)
I will try again sometime next week to see if I can get them to work as well, but my previous attempt had not worked.

As a quick update, it was a busy work week last week but I'm close to having creating a new graph and adding basic operations to it finished. I will then attempt the gradients portion and If all goes well will put up a PR hopefully by end of week.

Awesome, sounds great! I haven't been getting much time lately too but I'll let you know if I come up with something as well.

I have a question: adding those APIs mean we no longer need to build the models in Python or is this API just a small subset of what is necessary to build a model?

Hello @josevalim, if all this works as intended, then we might get some basic models to train completely in Elixir. I couldn't find support for all gradient types in the actual C API, so not all types of models (RNNs etc. maybe) might be directly trainable.
Basically we might be able to do some things that initially we would do in Python, but not all.

That's very nice to hear then. To me one of the biggest concerns with Tensorflow in Elixir is that you still have to write the models in Python, so you still need to learn Python and the main reason for using the Elixir bindings is because you want to integrate with a live Elixir system. But by being able to create some graphs in Elixir, it means we can have tutorials that are fully handled by Elixir. :+1:

I completely agree. Integrating these APIs might not be a straightforward process but it will be very good to have independent Elixir training-to-inference support in Elixir itself so it is well worth it.
I have got caught up with packing and last minute things lately but I will try this out once I'm a little settled. :)

I have not forgotten this, and speaking to the lack of straightforwardness, I ran into an issue with operation definitions. I have a stack overflow question that has been open for a bit and no one has answered so I figured I would link to it here and see if you @anshuman23 might be able to shed some light,
link

Sorry for the late reply @GrandathePanda. I'll take a look at this in the next few days and let you know!


[WIP] Fix encoding issue for publisher

Summary of Changes
Include a summary of changes then remove this line
Tests
What tests did you add or modify and why? If no tests were added or modified, explain why. Remove this line
Documentation
What documentation did you add or modify and why? Add any relevant links then remove this line
CheckList
Make sure you have checked all steps below to ensure a timely review.

[ ] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[ ] PR includes a summary of changes.
[ ] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does

[ ] PR passes make test

we have a py2 test which will fail. We may just drop the py2 test completely given no one is really use py2 at this point...

I think you can use io.iopen if you want to support both. It allows the encoding argument on py2 and is an alias for the built-in open on py3.

@astahlman updated :)
👀 @jinhyukchang

👍


[PE-D] If 2 companies in the JSON save file has the same name, all data is deleted.

If the user manually edits the JSON save file, all data are deleted.
While this is something I noticed, I personally think that it is unfair to be penalised for as it is something the user would have to maliciously do.

Labels: severity.VeryLow type.FeatureFlaw
original: TypeDefinition/ped#8

We can further test this behaviour.
Then a warning/caution against users on tempering with the internapply.json file could lead to loss of data which will be unrecoverable and that we won't be taking responsibility for such situations.

Please read - https://github.com/nus-cs2103-AY2122S2/forum/issues/243


[Bug]: Can't start extensions without updating them

What happened?
after you pushed the v3.5.4 and the updates for the other extension they will be disabled until you update to v3.5.4.
But there is a loading bug in it.
So i reinstalled my snapshot with v3.5.3 but the extensions with updates can't be started anymore so the website won't work
You can’t start the extensions anymore. they will be disabled automatically
I deleted the extensions and downloaded them from the marketplace. then it will download the newest version automatically. After that i can start the extensions again. and the website in working again.
The only update pending is the v3.5.4 update
What did you expect to happen?
website stopped working after updates
Version
3.5.x
What browser are you seeing the problem on?
Chrome, Safari
Relevant log output
No response

This issue will be closed and archived in 3 days, as there has been no activity in the last 60 days. If this issue is still relevant or you would like to see it actioned, please respond and we will re-open this issue. If this issue is critical to your business, consider the Dedicated Support Service where a Service Level Agreement is offered.


Can only do 30fps in basic cube example

Just started getting into this library. Was profiling one of the basic cube animation examples and I'm only getting about 30fps. I saw in the readme that even for very large scenes the author is able to get close to 60fps. Trying to understand what the issue could be.
My computer has a Quadro T2000 and an i9, and I'm on Ubuntu 21.04. glxgears runs at 1736fps. Unsure if this is something inherent to pygfx, wgpu, or something to do with vulkan support in my drivers. Any guidance would be much appreciated.

All canvas classes take a max_fps kwarg which defaults to 30. :) Try setting it to something really high!

Ah, that's what I get for not reading the examples closely. Ok now it runs at a very suspicious locked 60fps no matter how high I set the max_fps. Is there something else implicit controlling some sort of vsync? I tried grepping through the source code for references to something like that to no avail.

Good question. Could be the window toolkit you are using. Try using the GLFW canvas, I'm pretty sure that should work.

Hmm, tried that and was still stuck at 60fps. Ensured the max fps stuff is being set correctly and passed properly to the wait function. At least now I know where to poke around so I'll keep messing with it. Thanks.


Questions about point clouds

Hello, I'm just a beginner of the area and I'm a little confused of your impressive project.
Is your raw data the 2D coordinates of the point cloud or the 3D coordinates? When I read the paper, I thought the model was dealing with the xyz coordinates of the point cloud, however, what is provided in the code and the raw data files seem to be 2d coordinates?
If it is 2D coordinates, is a certain dimension directly discarded or other special processing is done?

The point cloud data provided are indeed 3D coordinates. Which part of the code are you referring to?


I suggest not del the curl tool

this issue: https://github.com/anapsix/docker-alpine-java/issues/8

Thanks, but this repo is deprecated, and remains for reference only. See README.
I suggest using the images still available in Docker Hub (generated from this repo) as a base, adding curl, and whatever other tools you require


Perl errors prevent SSLyze from working; Same errors with Nmap.

When running a SSLyze or Nmap scan on my Ubuntu Server v16.04 box with Seccubus v2.28, I got the following errors
chown: cannot access '/tmp/seccubus.10256.*': No such file or directory
Scanning done, converting .xml to ivil
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.
Importing ivil
Use of uninitialized value in join or string at /usr/share/perl5/XML/Simple.pm line 1284.
Use of uninitialized value $key in hash element at /usr/share/perl5/XML/Simple.pm line 1290.

The errors prevent the SSLyze from generating an IVIL XML file. Nmap works but I don't know how relialbe it will be.

@bislipur am I correct in thinking that this is the output of the nmap scanner? The SSLyze scan script doesn't contain any chown command, the nmap scan script does.
The chown only happens when you run the scan with the --sudo parameter.
For debug reasons, can you run the following command and send attach the output?
bin/do-scan -w <workspace_name> -s <scan_name> -v -v -v
If I ready through the script and see the chown: cannot access '/tmp/seccubus.10256.*': No such file or directory error, it appears that the actual nmap command you are running doesn't actually produce any output files but if you send me the output I requested above I can be more sure.
The Use of uninitialized value $key in hash element at and Use of uninitialized value in join or string at are warnings produced by the XML::Simple library on newer perl versions. THey are warnings and shouldn't actually be stopping the script.

@seccubus It is true that the errors listed above have been taken from the nmap run, but it got a lot of similarities (with a quick look). For the convenience I wil give you the (anonymized) data for bove nmap and SSLyze. After rerunning SSLyze, the "Findings" tab and the correspondending ivil file are empty.
output_Seccubus_nmap.txt
output_Seccubus_sslyze.txt
If needed I am willing to provide more information.

Just updated to Seccubus v2.32. Problem with SSLyze still exists.


Drab resets scroll position when updating DOM

Whenever I change any variables in Commander and I have some nested scroll which is in different then default position I will have DOM reloaded hence scrollTop will reset to 0. Is there any workaround to keep the scroll position?

I had the same issue creating naive chat, please take a look https://tg.pl/drab#chat_example. Simply, I just sent the JS:
broadcast_js socket, "document.querySelector('#chat').scrollTop = document.querySelector('#chat').scrollHeight"


Argument comments alignment uses spaces

It would be great if the alignment could be switched from settings to use tabs instead of spaces.
/**
 *	(tab)Does stuff
 *
 *	(tab)@param labels   (spaces)The labels.
 *	(tab)@param selector (space)The method.
 */

Currently the tab and space is mixed used when you uncheck the "use space instead of tab" option.
I will take a look at it soon to see what I can do later.

Ok, I updated the comment to fix an error and add an example to make it more clear.

Should be fixed in #123


temp fix for error while loading shared libraries: libprelude

temp fix for error while loading shared libraries: libprelude
Bug ticket: https://jira.cvent.com/browse/SRE-22166

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

closing


5 minute aggregates

Included weighted speed and imputed speed!

Please add @britt-allen to this review as well.


Reading a rar stream from a rar file gets NotSupportedException Specified method is not supported

Hi,
I'm using latest library version 0.37.2, at this moment of creating issue.
I have multi volume archive protected by password. That archive contains a subset of other archives, rar files especially.
There is my code:
var folder = @"D:\myfolder";
var rars = Directory.GetFiles(folder, "*.rar");
var pass = "abcdef1230213";

using var archive = RarArchive.Open(rars.Select(File.OpenRead), new SharpCompress.Readers.ReaderOptions { Password = pass });
foreach (var entry in archive.Entries.Where(c => !c.IsDirectory))
{
  Console.WriteLine(entry.Key);

  if (entry.Key!.EndsWith(".rar"))
  {
        using var stream = entry.OpenEntryStream();
        var r = RarArchive.IsRarFile(stream);
        using var subArchive = RarArchive.Open(stream);
        if (subArchive.Entries.Count > 0)   // here is the exception occurrence
        {
            foreach (var subEntry in subArchive.Entries.Where(c => !c.IsDirectory))
            {
                Console.WriteLine(subEntry.Key);
            }
        }
    }
}

What I'm doing wrong ?

Maybe the inner rar needs a password?  I can't tell from the usage,  A full stacktrace would help with the password issue


Update Timeline “2022-06-16t15-40-38-831z-campaign”

Automatically generated by Netlify CMS

👷 Deploy Preview for dynamic-cheesecake-5fbd8b processing.

Name
Link

🔨 Latest commit
0950a489c1350cb6a09792f6e11da613f310ceb9

🔍 Latest deploy log
https://app.netlify.com/sites/dynamic-cheesecake-5fbd8b/deploys/62eb29da89a423000966fc45


Fixes #366 - Allows options to be passed to mergeStreams

Fixes #366
mergeStreams now works with options passed in, which allows objectMode to be set to true, allowing an interesting use case where a SitemapStream uses an existing source file stream and another stream that allows more items to be added without having to hydrate and maintain the entire map in memory

@ekalinin - Are you available to review?  Thanks!


[adding] more ads domains

completesatisfaction.ltd
lp.mydirtyhobby.com
de.betano.com
makmoney.club
fgddl9szdof7.com
448ff4fcfcd199a.com
landings.twin.com

are another ads site.
Please see also
ww2.sponsorflyr.com
from https://github.com/StevenBlack/hosts/issues/1344

Thanks for this Brew @beerisgood.
Do you have any documentation on why we should include each of these hosts? Where do these come from?

Beer is good :D
I got all of these on different streaming sites and they open itself and are real annoying.

Thanks Beer @beerisgood.  These are added in commit 0f046591.


Tests are discovered with Pytest plugin in VSCode, but when launched in debug mode they are not found anymore.

When i try to launch a test from pytest plugin in VSCode, it shows me the error:
ERROR: not found path_to_the_test
(no name 'path_to_the_test' in any of [])
This usually happen when i have a test with multiple parameters and i try to launch the test with a specific one, but launching the test normally has always worked until 10th of October.
I have all init.py required, and didn't change the code.
Python version: 3.10.9
VsCode version: 1.83.1

Hello! Could you try opting out of the testing experiment and see if that fixes it? If so then it is an issue with our rewritten design. You can opt out by setting "python.experiments.optOutFrom": ["pythonTestAdapter"],. Secondly could you set your log level on vscode to trace (can be done via the command palette) and send over your logs from the python output channel and the python test results output channel?
Thanks!

Hello! Thank you for the option, after setting it all tests are now running through pytest plugin as usual.


Springer link

Using specific web elements to grab the the links to the separately stored tables and adding them on to the HTML that will be saved for that specific paper

I am quite confused why the checks are failing.

@aldensyi this looks fine now. I don't know if we need that many identifiers but it doesn't hurt. As long as there's no false positives (i.e. another journal article shows up as Springer) then I think we're good.


Application windows snap to FancyZones on system wake-up

FancyZones:
Using an Ultrawide and 3 monitors total, I often split my browser/view pane on the ultrawide using approximately 2/3 of the main display. The other 1/3 is used for Zoom/Teleconference/etc. I noticed that I can have a chrome window full screen, and if I let the system go idle and blank the screen when I return and re-awaken the system, chrome is snapped to the 2/3 FancyZone from earlier in the morning.
I'm not sure exactly how or why this is happening, but I suspect it is a session state restoration bug on the screen restoration of some kind.
ℹ Computer information

PowerToys version: v0.25.0
PowerToy Utility: FancyZones
Running PowerToys as Admin: Y
Windows build number: 1909 (OS Build 18363.1139)

📝 Provide detailed reproduction steps (if any)

Have application window fullscreen(chrome), but previously have used it in a FancyZone earlier in the day.
Let system go to sleep/screensaver/blank.
On return to active state with mouse movement, full screen application is reverted to a FancyZone setting

✔ Expected result
The application window preference needs to remain whatever the view was when the system blanked the screen. In this case, the view prior to screen blanking was a full screen window.
❌ Actual result
The application window is snapped to a FancyZone on system wake up.

@iankoetter
please do the following test: before letting the PC go to sleep, quit PowerToys. After the PC wakes up check the Chrome windows is still full screen.


Add warning about Citizens Discord

At this time, Citizens is applying a DoNotSupport role to those who are running Floodgate. This warns users running Citizens and Floodgate that support will be denied for them.

We decided against adding this feature.


add locale to word module

ref book module

Hi  @cieslarmichal, I would like to collaborate on this issue. After looking the code (the books and words modules), I noticed that right now this generate only English words . I am thinking in generate the Spanish(ARG) arrays in the word_store.h, new functions and tests. If it's ok(only for Spanish and English locations), please assign it to me.

Yeah you can do only with those two locales and it locale doesnt exist return default english

Hi @cieslarmichal, I've done something (added to word module es_AR)... how could I get a code review? Just generate a pull request? If everything is ok, I will continue with another languages. I am considering using std::vector instead of std::array in the structures in "word_store.h".

@gustavobastian yup just generate a PR, why would you like to change it to vector?

I am generating a map with the key each faker::Locale, and the value, the array. If we use const vector, instead of const array, we can let the compiler to calculate the size of each language structure. Right now, we are forced to store the same number of words for each language, and I don´t know if other languages has enough words for each kind, as the map is (key,array<string_view>xxx):

We can repeat some words in order to get the desired number... There are multiple solutions for the situation.


Bump tink worker

Issue #, if available:
Description of changes:
Bump tink worker from v0.10.0 to v0.10.1. v0.10.1 has tink worker changes needed for embedded images.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/test tink-tooling-presubmit

/cherry-pick release-0.20


Add Galarian Slowpoke to Pokemon Sword/Shield Pokedex

https://swordshield.pokemon.com/en-us/expansionpass/pokemon/slowpoke/
There will be new expansion pack coming soon with more as well. Right now it doesn't have a dex # so it just additional.
Ah here is more too.
https://swordshield.pokemon.com/en-us/expansionpass/pokemon/kubfu/
https://swordshield.pokemon.com/en-us/expansionpass/pokemon/urshifu/
https://swordshield.pokemon.com/en-us/expansionpass/pokemon/gigantamax-urshifu/
https://swordshield.pokemon.com/en-us/expansionpass/pokemon/calyrex/

thanks @Chamberly!
we actually won't be adding galarian slowpoke to the regional swsh dex just yet because it doesn't have a galarian dex number, even tho he is catchable. unfortunately in our system, without a galarian dex number, the system won't work very well. kubfu, urshifu, and all other pokemon who will become available with the expansion passes will be added all at once when those are released (https://github.com/pokedextracker/pokedextracker.com/issues/350)
we will however be including galarian slowpoke in the national dex update (https://github.com/pokedextracker/pokedextracker.com/issues/349)


Rewrite Chapter 10 CSRF section

This rewrites the portion of Chapter 10 focused on cross-site request forgery, and re-focuses it on the SameSite cookie attribute. The implementation is, as far as I can tell, compliant with the draft RFC and matches what browsers currently do. Note that this PR is dependent on #225 but I think the text can be reviewed independently.

I'll merge this once #223 is merged.


[10.x] add prefix and suffix to str random function

this pull request aim to add prefix and suffix to Str::random function if needed
$random = Str::random(5, 'foo_', '_bar')

should output something like this : foo_jndUh_bar

yes @nunomaduro i kinda agree, in general this will be handy when generating a lot of random strings and avoid extra code that may be involved.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Can you please support sortable as well?

It's a pity that the jquery-ui team doesn't support cross-window dragging [1] and it's great that you fix that for many people who need that!
I wonder how hard it is to make your solution work for sortables as well. I tried different possibilities now but didn't find a well working solution.
This fiddle shows the offset between the mouse pointer and the dragged element without your library included:
http://jsfiddle.net/e2kgbcbz/10
This is with your library included:
http://jsfiddle.net/e2kgbcbz/13
What do you think?
Best,
Sandro
[1] - just as a reference: https://bugs.jqueryui.com/ticket/7925

There is also a Stackoverflow question about that: http://stackoverflow.com/questions/16731203/invalid-mouse-offset-for-cross-frame-draggable-sortable

+1


AppImage: add option to specify application name

Right now the application name is taken from the binary name:
https://github.com/Nuitka/Nuitka/blob/6d868e6d6c371f46ce627f8b774ad3db3d14cab3/nuitka/freezer/Onefile.py#L154-L164
an option to specify a more human readable application name (which makes the marketing department happy) would be nice as binary_basename tend to be a technical name.

instead of a name option, maybe a more generic solution regarding .dektop files can used which would also solve #1130

That's correct, I am working to universalize the product information that is currenly Windows only, and then will work to produce Desktop files for Linux with it. I would welcome a PR that makes use of the getWindowsCompanyName there even now, but for now I have to defer it from my side until the automatic updating and code signing hits Linux too.

For Linux we do not generate desktop files anymore, and we may well add that, but AppImagine does not exist anymore, therefore I am closing this.