OCommandSQLParsingException: Error on parsing command at position #0: Encountered "  "from ""

Studio
Orientdb 2.1 SNAPSHOT
select from (traverse References from ( select from Node WHERE Email = 'julia@local'  ) WHERE @class = 'Node' ) where $depth = 1

 [ONetworkProtocolHttpDb]com.orientechnologies.orient.core.sql.parser.ParseException: Encountered " <FROM> "FROM "" at line 1, column 8.
   ...
    ":" ...
    "{" ...
    "true" ...
    "false" ...
    "-" ...
    <INTEGER_LITERAL> ...
    "?" ...
    ":" ...

        at com.orientechnologies.orient.core.sql.parser.OrientSql.generateParseException(OrientSql.java:12402)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.jj_consume_token(OrientSql.java:12275)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.Expression(OrientSql.java:4019)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.ProjectionItem(OrientSql.java:3244)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.Projection(OrientSql.java:3193)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.SelectWithoutTargetStatement(OrientSql.java:507)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.Statement(OrientSql.java:419)
        at com.orientechnologies.orient.core.sql.parser.OrientSql.parse(OrientSql.java:87)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLAbstract.preParse(OCommandExecutorSQLAbstract.java:227)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.parse(OCommandExecutorSQLSelect.java:220)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLSelect.parse(OCommandExecutorSQLSelect.java:100)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLDelegate.parse(OCommandExecutorSQLDelegate.java:76)
        at com.orientechnologies.orient.core.sql.OCommandExecutorSQLDelegate.parse(OCommandExecutorSQLDelegate.java:42)
        at com.orientechnologies.orient.server.network.protocol.http.command.post.OServerCommandPostCommand.execute(OServerCommandPostCommand.java:76)
        at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.service(ONetworkProtocolHttpAbstract.java:175)
        at com.orientechnologies.orient.server.network.protocol.http.ONetworkProtocolHttpAbstract.execute(ONetworkProtocolHttpAbstract.java:612)

checking right now

the parsing error message is not so clear, actually a problem exists in your query: you are using WHERE clause in a TRAVERSE, while it does not support it. Try to replace the WHERE with a WHILE and you'll see your test pass
select from (traverse References from ( select from Node WHERE Email = 'julia@local'  ) WHERE @class = 'Node' ) where $depth = 1

But 'where' was and is supported since many previous versions before 'latest 2.0.10'. Who tells me that this will be deprecated? :/

Hi @StarpTech
WHERE is supported (and will remain supported) for all the statements except TRAVERSE.
Maybe in old releases the parser was too permissive...
Take a look at the docs, they have not changed since a while
http://orientdb.com/docs/last/SQL-Traverse.html
Thanks
Luigi

Testing with 2.0.10 show you that where works and this release is 25days ago and I think also in 2.0.11 it works 3 days ago. But ok I will change my query and give feedback. thanks.

by the way, strict SQL parsing can be disabled in 2.1, you can do it from Studio -> DB tab -> Configuration, see the bottom of the page.
This way you will have complete backward compatibility (but you will also have all the defects of the old parser)

TRAVERSE accepted WHERE, but we replace it with WHILE because has more sense. If you see the server log you should notice WARNINGs about that usage. With the new parser this syntax is not supported anymore.


Pass Controller to httpserver.GetConfig

base on https://github.com/mholt/caddy/commit/a798e0c95191e07996c2a0d79087962d14e06464

Thanks for this but the job is not yet done.
./setup_test.go:20: not enough arguments in call to caddy.NewTestController
./setup_test.go:166: not enough arguments in call to caddy.NewTestController
./setup_test.go:194: not enough arguments in call to caddy.NewTestController

👍

Thanks.


Test time analyses csmith

This profiles the default static analysis pipeline against a set of random C programs generated by csmith.
It outputs the results in CSV and gnuplot format to produce plots like:

usage:
cd examples/csmith && python3 gen.py && cd ../..
mill test.testOnly '*TimeStaticAnalysis'
gnuplot src/test/analysisTiming/plot.gp
xdg-open  src/test/analysisTiming/analysisres.svg

The generation script and tests all run singlethreaded so this takes quite a while to run unfortunately. Tests could possibly be multithreaded via Future.sequence but I don't know of a good way to avoid the JVM running out of memory by loading everything at once.

We don't want to disable the MRA-related analyses when regions are disabled because they are still used for resolving indirect calls which is separate functionality.


Can nested list also be drag&droppable

I just wonder, can I make nested list also drag&droppable.... I have a drag&droppable list, which has listitems which on their own have nested lists...can I make that nested list drag&droppable too?

@AnoushSim having the same problem. Can you provide an example please?

@AnoushSim Your example would be most helpful!


Construction of html elements in controller layer

Problem
There are a few places where html is being built in the controller layer
...
const result = {
    company: `<a href="${companyUrl}">${serviceDelivery.company.name}</a>`,
    dit_team: getPropertyName(serviceDelivery, 'dit_team'),
...

Suggestion
It would be good to bring this out into the view layer in a component.

As we made the explicit decision not to use React, I'd tend to agree

This can happen as part of the move to new macro styles, the plan was to move to allow some things to pass either a string or an object, though @tyom had some issues with that I think, to do with it not being clear for developers to know how to use the components.


Formatting issue with screenshots

Issue Template
Description
Using Firefox, when I download the screenshot the formatting for substats is broken. Please see the screenshots. Cleared cache, didn't fix. Tried on google chrome with the same data and it worked fine, so looks like it might have to do with the browser.
Steps to Reproduce

Open the optimizer on Firefox
Go to characters
Click on the download button next to the "copy screenshot button"

Expected Behavior
Screenshot downloads and the formatting is correct
Actual Behavior
Screenshot downloads but the substat formatting is incorrect
Screenshots

fribbels-optimizer-save(1).json
Additional Information
Tried it on different screen resolutions as well but had no effect.

Omg that looks awful - thanks for letting me know

This issue should be resolved now - was a zoom scaling issue on firefox


RawKeyboardListener not work after upgrading hover to 0.45

In hover 0.43 just works, what might be the problem
hover doctor
hover: Hover version v0.45.0 running on windows
hover: Sharing packaging tools
hover: darwin-bundle is supported
hover: Packaging darwin-dmg is not supported on windows
hover: To still package darwin-dmg on windows you need to run hover with the `--docker` flag.
hover: Packaging darwin-pkg is not supported on windows
hover: To still package darwin-pkg on windows you need to run hover with the `--docker` flag.
hover: Packaging linux-appimage is not supported on windows
hover: To still package linux-appimage on windows you need to run hover with the `--docker` flag.
hover: Packaging linux-deb is not supported on windows
hover: To still package linux-deb on windows you need to run hover with the `--docker` flag.
hover: Packaging linux-pkg is not supported on windows
hover: To still package linux-pkg on windows you need to run hover with the `--docker` flag.
hover: Packaging linux-rpm is not supported on windows
hover: To still package linux-rpm on windows you need to run hover with the `--docker` flag.
hover: Packaging linux-snap is not supported on windows
hover: To still package linux-snap on windows you need to run hover with the `--docker` flag.
hover: To package windows-msi these tools are required: candle
hover: Install the WiX Toolset from https://wixtoolset.org/releases/
hover: To still package windows-msi without the required tools installed you need to run hover with the `--docker` flag.
hover:
hover: Sharing flutter version
Flutter 1.23.0-18.1.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 198df796aa (3 weeks ago) • 2020-10-15 12:04:33 -0700
Engine • revision 1d12d82d9c
Tools • Dart 2.11.0 (build 2.11.0-213.1.beta)
hover: Flutter engine commit: https://github.com/flutter/engine/commit/1d12d82d9cb54876f58044aa52198d53ec841c3d
hover: Finding out the C compiler version
gcc (tdm64-1) 9.2.0
Copyright (C) 2019 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

hover: Sharing the content of go.mod
module client/go

go 1.14

require (
        github.com/Xuanwo/go-locale v1.0.0 // indirect
        github.com/adrg/xdg v0.2.1
        github.com/go-flutter-desktop/go-flutter v0.42.0
        github.com/go-gl/glfw/v3.3/glfw v0.0.0-20201101223321-f087fae2c024 // indirect
        github.com/kr/pretty v0.1.0 // indirect
        github.com/pkg/errors v0.9.1
        golang.org/x/sys v0.0.0-20201101102859-da207088b7d1 // indirect
        golang.org/x/text v0.3.4 // indirect
        gopkg.in/check.v1 v1.0.0-20180628173108-788fd7840127 // indirect
)
hover: Sharing the content of hover.yaml
application-name: CH Trader
executable-name: ""
package-name: ""
license: ""
target: lib/main.dart
branch: ""
cache-path: ""
opengl: ""
engine-version: ""
hover: Sharing the content of go/cmd
go\cmd\main.go  go\cmd\options.go       go\cmd\path_provider.go

The hover version shouldn't interfere with the go-flutter version or how this specific feature works.
I don't see why changing the build tool reflects an change in how RawKeyboardListener works.
please run:
GO111MODULE=on go get -u github.com/go-flutter-desktop/hover
hover build windows --debug
In the https://github.com/go-flutter-desktop/examples/tree/master/keyboard_event example.
Does ./go/build/outputs/windows-debug_unopt/keyboard_event works ?

Thanks @pchampio , after reading the example code, I found the issue, maybe it's FocusNode related:
My code just put RawKeyboardListener in a StatelessWidget and without setting the autofocus = true. It works in 0.43.0 but not in 0.45.0
In 0.45.0, Setting autofocus = true fix this isse. Or put in a StatefullWidget with didChangeDependencies()


Use _data vs data / _mask vs mask more deliberately

I think it's worth rethinking all uses of _data, data, _mask, and mask attributes. Considerations:

I think it would be nice if these functions accepted with np.ma masked arrays as input
We need to be careful about accessing the data memoryview attribute of regular NumPy arrays. This suggests that use of _data and _mask should be preferred?

At this point it is almost exclusively .data and .mask in the code, so this can be closed. If we need to be more careful about accidentally accessing the data memoryview of regular NumPy arrays, it will come up as an issue.


Is it necessary to implement hash/compare trait and for Array?

[x] Compare (done in #1003) (but to be discussed)
[ ] Hash (to be discussed)

compare is needed, not sure about hash though

use case for hash: BPE tokenizer. In BPE tokenizer I need to maintain a Map[(Array[Int], Array[Int]), Array[Int]] when merging multi-byte pairs.

I have another use case: hash cons when compiling a computation graph.
The const in a computation graph is a FixedArray[Float] (or Array[Float]) and I need it to be hashable so that I can derive Hash for my graph data type.


Can not reach server

When I use a live account：

When I use a Demo account：

Where is the problem with this prompt, thank you

can you replicate you error


SCons: Print info message on build failures with werror=yes

This provides an informational message allowing people to compile the engine by bypassing werror build option (this does not change the defaults).
Using GetBuildFailures method provided by SCons to know whether a build has failed internally.
The condition is quite specific and should only affect development versions.
See some reports which justify this change:
#38829
#39648
#39901
#40286 https://github.com/godotengine/godot/pull/40355#issuecomment-658032776
#40342

#40286

This bug is not related to werror=yes, so that's an example of where the added info message might mislead the user.

Well I guess if we treat warnings as errors now, users should accept that, but this has to be documented in some way, perhaps this PR is just an attempt at resolving documentation issue.
I'll see whether this can be detected when a build fails due to treating warnings as errors specifically, so WIP then.

Alternatively, this could be converted to some kind of build summary/status, to remove the need to specify causality of the errors, users would still be able to guess what might have gone wrong during the build.

Well, I think the relative difficulty to eradicate all warnings for all compilers and all past, present and future compiler versions probably forces us to disable werror=yes by default. We'll keep it on CI of course, so it means devs will sometimes make PR that build locally and fail on CI, but that's already the case if they use a compiler which doesn't catch a specific warning that CI would. And there's still dev=yes as an option to force verbose output and -Werror.

Alternatively, this could be converted to some kind of build summary/status, to remove the need to specify causality of the errors, users would still be able to guess what might have gone wrong during the build.

I think some kind of build summary could be useful indeed, mainly to summarize what actual build options have been used (both user-specified, but most importantly ones with default values, as users don't necessarily know that e.g. scons p=windows will give them a tools=yes target=debug build.

Superseded by #40367.


chore: bump https-proxy-agent to mitigate a security issue

Context:

https://github.com/TooTallNate/node-https-proxy-agent/issues/78
https://app.snyk.io/test/npm/https-proxy-agent/2.2.2
https://hackerone.com/reports/541502

According to release notes, the bump should not bring any breaking changes for Puppeteer. I believe it can get shipped with the next patch release.

@mathiasbynens I'm not sure what the release schedule is for this lib, but could we get a patch for this?

Also can't wait for a patch release to stop seeing security warnings in Snyk. Some of my projects depend on puppeteer, which brings in vulnerable<EMAIL_ADDRESS>🙏

@bmeurer and @mathiasbynens it looks like dropping Node v6 support was merged to master since this went in. Will that require a major version bump of puppeteer?
Can I please request that a simple patch version of 1.20 be released with this security vulnerability fixed first?

Fixes https://github.com/GoogleChrome/puppeteer/issues/5055


bdist_nuitka produces wrong output for a nested namespace package

Nuitka version, full Python version and Platform (Windows, OSX, Linux ...)

$ python3 -m nuitka --version
0.6.7
Python: 3.6.8 (default, Aug 20 2019, 17:12:48)
Executable: /usr/bin/python3
OS: Linux
Arch: x86_64

How did you install Nuitka and Python (pip, anaconda, deb, rpm, from source,
what is a virtualenv ...), this is very important usually.

$ python3 -m pip install nuitka

Issue
I am trying to compile a Python package of mine to an extension module and package it into a wheel using Nuitka. The special thing about my case seems to be that this package resides inside an implicit namespace package nested two levels deep. The compile runs fine, the .so file gets created too, but when I install that wheel, my package cannot be imported.
Minimal example
Consider the following package layout:
|- a/b/pkg/__init__.py
|- setup.py

a.b.pkg is the package I want to compile to an extension module. a.b is the implicit namespace package (notice the lack of __init__.pys).
These are the contents of setup.py:
from setuptools import setup

setup(
    name="pkg",
    packages=["a.b.pkg"],
)

I now invoke bdist_nuitka:
$ python3 setup.py bdist_nuitka
running bdist_nuitka
running build
running build_py
creating build
creating build/lib
creating build/lib/a
creating build/lib/a/b
creating build/lib/a/b/pkg
copying a/b/pkg/__init__.py -> build/lib/a/b/pkg
Building: PyPackage(module_name=a.b.pkg, related_packages=['a.b.pkg'])
Nuitka:INFO:Recurse to import 'a.b.pkg' from 'a/b/pkg'. (Module is exact match of 'a.b.pkg' instructed by user to recurse to.)
Nuitka:INFO:Recurse to import 'a.b' from 'a/b'. (Containing package of recursed module 'a.b.pkg'.)
Nuitka:INFO:Recurse to import 'a' from 'a'. (Containing package of recursed module 'a.b'.)
Nuitka:INFO:Included compiled module 'a'.
Nuitka:INFO:Included compiled module 'a.b'.
Nuitka:INFO:Included compiled module 'a.b.pkg'.
Nuitka:INFO:Included compiled module 'pkg'.
installing to build/bdist.linux-x86_64/wheel
running install
running install_lib
creating build/bdist.linux-x86_64
creating build/bdist.linux-x86_64/wheel
creating build/bdist.linux-x86_64/wheel/a
creating build/bdist.linux-x86_64/wheel/a/b
creating build/bdist.linux-x86_64/wheel/a/b/pkg
creating build/bdist.linux-x86_64/wheel/a.b
copying /tmp/nuitka-test/build/lib/a.b/pkg.pyi -> build/bdist.linux-x86_64/wheel/a.b
copying /tmp/nuitka-test/build/lib/a.b/pkg.so -> build/bdist.linux-x86_64/wheel/a.b
running install_egg_info
running egg_info
creating pkg.egg-info
writing pkg.egg-info/PKG-INFO
writing dependency_links to pkg.egg-info/dependency_links.txt
writing top-level names to pkg.egg-info/top_level.txt
writing manifest file 'pkg.egg-info/SOURCES.txt'
reading manifest file 'pkg.egg-info/SOURCES.txt'
writing manifest file 'pkg.egg-info/SOURCES.txt'
Copying pkg.egg-info to build/bdist.linux-x86_64/wheel/pkg-0.0.0-py3.6.egg-info
creating build/bdist.linux-x86_64/wheel/pkg-0.0.0.dist-info/WHEEL
creating 'dist/pkg-0.0.0-cp36-cp36m-linux_x86_64.whl' and adding 'build/bdist.linux-x86_64/wheel' to it
adding 'a.b/pkg.pyi'
adding 'a.b/pkg.so'
adding 'pkg-0.0.0.dist-info/METADATA'
adding 'pkg-0.0.0.dist-info/WHEEL'
adding 'pkg-0.0.0.dist-info/top_level.txt'
adding 'pkg-0.0.0.dist-info/RECORD'
removing build/bdist.linux-x86_64/wheel

Now, when I install that wheel and try to import my package:
$ cd dist
$ pip install pkg-0.0.0-cp36-cp36m-linux_x86_64.whl
Processing ./pkg-0.0.0-cp36-cp36m-linux_x86_64.whl
Installing collected packages: pkg
Successfully installed pkg-0.0.0
$ python3 -c "import a.b.pkg"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'a'

My site-packages directory contains the compiled extension module at a.b/pkg.so. I think this is the reason why the import doesn't work. Shouldn't it be at a/b/pkg.so instead?

@tommyli3318 Can you have a look at this one, seems easy and where you made changes previously. The a.b is wrong, and we probably just need to handle multiple levels. Namespace packages for distutils are supported, but maybe only one level deep?

Fixed in #646.

The release 0.6.8 was just made and contains the correction.


Why 'codesearch' task in CodeBERT is 'classification'

Follow README at CodeBERT/codesearch, I fine-tuned the model.
But, it's task type is 'classification', so, when i want to search something by model, i got a classification result, like this:
[{'label': 'LABEL_1', 'score': 0.9671146273612976}]
I mean, i want to search code, not classify my input.

you can use unixcoder, which use retrieval setting.


How to use bigger fields

#[bitfield(u64)]
#[derive(PartialEq)]
pub struct Message {
    #[bits(8)]
    pub message_id: u8,
    #[bits(24)]
    pub time_of_reception: u32,
    #[bits(3456)]
    pub payload: [u8; 432],
}

if i remove (u64), throws unknown type.
How can I use something this crate for bigger fields?

Unfortunately, u128 is the biggest bitfield size supported. Everything beyond has to put into multiple bitfields, like:
#[bitfield(u128)]
pub struct A {
    // ...
}
#[bitfield(u128)]
pub struct B {
    // ...
}

#[repr(C)]
struct AB {
    a: A,
    b: B,
}


365 Hunting Query data refresh

Adding Powerbi Report to powerbi webapp causes data refresh to break due to "dynamic data sources" no documentation online.

Ok this seems to be due to how PowerBI Online handle refresh, I explain that here https://www.linkedin.com/pulse/story-powerbi-microsoft-graph-api-service-side-refresh-journaux/
But the solutions I found for the MS Graph does not apply here due to the fact that api.security.microsoft.com is not accessible directly without authentication and does not redirect to a website that return a HTTP 200.
Let's wait for a PG comment on that :)


chore(deps): update rust crate tracing-subscriber to 0.2.17

This PR contains the following updates:

Package
Type
Update
Change

tracing-subscriber
dependencies
patch
0.2.16 -> 0.2.17

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

bors merge

bors merge

bors merge

bors merge


New installation issue

I just installed ANTs on a new machine using the standard instructions, i.e., adding the Scripts folder to the path. When I try to run antsLongitudinalJointLabelFusion.sh I get the following errors:
--------------------------------------------------------------------------------------
 FILE /mnt/c/Users/dp/antsbin/bin/ANTSpexec.sh DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scripts/an
tsLongitudinalJointLabelFusion.sh will terminate.
--------------------------------------------------------------------------------------
 if the file is not executable, please change its permissions.

--------------------------------------------------------------------------------------
 FILE /mnt/c/Users/dp/antsbin/bin/waitForSGEQJobs.pl DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scri
pts/antsLongitudinalJointLabelFusion.sh will terminate.
--------------------------------------------------------------------------------------
 if the file is not executable, please change its permissions.

--------------------------------------------------------------------------------------
 FILE /mnt/c/Users/dp/antsbin/bin/waitForXGridJobs.pl DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scr
ipts/antsLongitudinalJointLabelFusion.sh will terminate.
--------------------------------------------------------------------------------------
 if the file is not executable, please change its permissions.

--------------------------------------------------------------------------------------

This seems to come from where some of the scripts set up the path to other scripts. Normally I don't move the scripts to the bin folder, they are all in their original Scripts folder. I just add that folder to the path. But at the beginning of antsLongitudinalJointLabelFusion is this, which seems to set up scripts inside the bin folder:
ANTS=${ANTSPATH}/antsRegistration
WARP=${ANTSPATH}/antsApplyTransforms
JLF=${ANTSPATH}/antsJointFusion
PEXEC=${ANTSPATH}ANTSpexec.sh
SGE=${ANTSPATH}waitForSGEQJobs.pl
PBS=${ANTSPATH}waitForPBSQJobs.pl
XGRID=${ANTSPATH}waitForXGridJobs.pl
SLURM=${ANTSPATH}/waitForSlurmJobs.pl

Question is: do we need to put all the scripts in the bin folder now? This does not seem to be in the official installation instructions.
P.s. Note also the inconsistent / at the end of the path settings after ${ANTSPATH}

The installations directions are on the wiki
Scripts should be copied to the Bin folder, it has been that way for quite
some time
On Tue, Aug 8, 2017 at 9:00 AM dorianps<EMAIL_ADDRESS>wrote:

I just installed ANTs on a new machine using the standard instructions,
i.e., adding the Scripts folder to the path. When I try to run
antsLongitudinalJointLabelFusion.sh I get the following errors:

FILE /mnt/c/Users/dp/antsbin/bin/ANTSpexec.sh DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scripts/an
tsLongitudinalJointLabelFusion.sh will terminate.
if the file is not executable, please change its permissions.

FILE /mnt/c/Users/dp/antsbin/bin/waitForSGEQJobs.pl DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scri
pts/antsLongitudinalJointLabelFusion.sh will terminate.
if the file is not executable, please change its permissions.

FILE /mnt/c/Users/dp/antsbin/bin/waitForXGridJobs.pl DOES NOT EXIST -- OR -- IS NOT EXECUTABLE !!! /mnt/c/Users/dp/antsbin/../ANTs/Scr
ipts/antsLongitudinalJointLabelFusion.sh will terminate.
if the file is not executable, please change its permissions.

This seems to come from where some of the scripts set up the path to other
scripts, at the beginning of antsLongitudinalJointLabelFusion is this:
ANTS=${ANTSPATH}/antsRegistration
WARP=${ANTSPATH}/antsApplyTransforms
JLF=${ANTSPATH}/antsJointFusion
PEXEC=${ANTSPATH}ANTSpexec.sh
SGE=${ANTSPATH}waitForSGEQJobs.pl
PBS=${ANTSPATH}waitForPBSQJobs.pl
XGRID=${ANTSPATH}waitForXGridJobs.pl
SLURM=${ANTSPATH}/waitForSlurmJobs.pl
Question is: do we need to put all the scripts in the bin folder now? This
does not seem to be in the official installation instructions
https://brianavants.wordpress.com/2012/04/13/updated-ants-compile-instructions-april-12-2012/
.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stnava/ANTs/issues/481, or mute the thread
https://github.com/notifications/unsubscribe-auth/AATyfs7AwTJRJ7ET7J9jI7NPWP_Ekk9Hks5sWFvogaJpZM4Owspl
.

--
brian

Ok, I updated the instructions for Windows installers. You may also want to update the main Readme page, it points to the instructions that don't mention the Scripts, unless one goes to read the comments below the instructions.

I updated the README https://github.com/stnava/ANTs/commit/bcc6a7873b5cf6f546d94034f1940ecbcb434cba
The scripts are not entirely consistent; some call other scripts as ${script} rather than ${ANTSPATH}${script}, so you might have run some scripts from other locations successfully in the past. Or, more troubling, you might have mixed ANTs versions inadvertently. To avoid this, it's best to prefix the calls within scripts with ${ANTSPATH} and locate the ANTs scripts in the ${ANTSPATH} directory.


More flexible authentication system

Hello again!
I'm attempting to use this client with AWS Keyspaces. This service has two mechanisms for authentication. Right now, I'm able to use the user/pass versions, and it's working! I'm amazed!
However, the "real" way to do auth with Keyspaces is via AWS SigV4 authentication. Doing this requires building a plugin that uses  the lower level auth systems of the C++ driver. These are not currently exposed. I haven't built a system that does this yet, so I'm kind of winging it based on what I see in the Go plugin.
Any thoughts on how to wrap up the driver's API?

I think adding a plugin system for auth is interesting direction. we would obviously need to expose a new API that allows the client to inject the alternative auth provider. would you like to to propose such?

cc @yim-lee @rnro

I did make a little progress. I was using the example here:
https://github.com/datastax/cpp-driver/blob/master/examples/auth/auth.c
However, I was working on this as a side-project, and have been distracted. I like the idea of an API to make this easier, but I don't think I have enough time or context to make a good suggestion.


detection window to roshost side

Dear,
I built and tested darknet_ros today on NVIDIA Jetson TX2.
It runs pretty well much more than I expected.
Then I want to display this detection window on roshost side, if possible.
And darknet running console might be created and displayed another terminal.
I am not able to monitor ROS logs.
Thanks,

It works!
I launch darknet_ros in other terminal,
$ roslaunch darknet_ros darknet_ros.launch
This clears two issues at a time!
Thanks,

Hello, I am also using TX2 to run Darknet, using command roslaunch darknet_ros darknet_ros.Launch, using Yolov3, , FPS is only about 0.4, when I use Yolov2-tiny, FPS is raised to about 4, but the effect is still not ideal, I also modified in Makefile, GPU=1, CUDNN=1, OPENCV=1, I used JetPack3.0, CUDA8,
It looks like Cudnn isn't working. Can we talk about it?

@alexis-pepe  I have the same issue the 0.4 fps, Let me know If you found a fix


Update to CDK v1?

I'm getting the ts error: Argument of type 'this' is not assignable to parameter of type 'Construct'.   Type 'PipelineStack' is missing the following properties from type 'Construct': dependencyRoots, apply
    const pipeline = new delivlib.Pipeline(this, 'MyPipeline', {
      repo: githubRepo,
      branch: 'jjacobs/KH-343/feature'
    })

regardless of what scope I apply. I've also tried copying the example from https://github.com/awslabs/aws-delivlib/blob/master/test/test-stack.ts but am getting the same error.
My versions of aws-cdk:
    "aws-cdk": "^1.1.0"
    "@aws-cdk/core": "^1.1.0"

Version of aws-devlib:
"aws-delivlib": "^3.9.5"

Thanks!

Looks like this is already being worked on. Apologies, I should have checked the PRs.


Remove version: RHash.RHash version 1.4.4

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://downloads.sourceforge.net/project/rhash/rhash/1.4.4/rhash-1.4.4-win32.zip
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


Acceptance Testing

Sets up the basis for running integration Acceptance tests.  These tests directly hit an opslevel account but perform cleanups.  We should beable to use these as a way to full flush water through the pipes before a release.

It looks like the go.mod and go.sum are off.  Probably because of the submodule thing, right?


2 ways relationship in documentation for switch

Description

Hey,
I was trying to do a 2 ways relationship for the switchers in my project and I was wondering how to modify the boolean values into other string values. While checking the doc of Oruga, I discovered the trueValue & falseValue.

<template>

Which Ice cream would you like ?

        <o-field>
          <o-switch
            v-model="state.example"
            true-value="chocolate"
            false-value="strawberry"
            variant="primary"
          >
            Chocolate
          </o-switch>
        </o-field>

        <o-field>
          <o-switch
            v-model="state.example"
            true-value="strawberry"
            false-value="chocolate"
            variant="primary"
          >
            Strawberry
          </o-switch>
        </o-field>

</template>

Why Oruga needs this feature
Would you mind to add an example of this feature in the doc instead of just sharing simple switcher examples ? It could be great to highlight the power of the trueValue and falseValue, don't you think ? :)

Hey, there is already such an example in the Base example section of the switch component.


Termite is deprecated

Issue description
As per this issue comment Termite is no longer developed/maintained and forks are discouraged. Wanted to mostly raise awareness about the issue, inclined towards the idea that it should be removed from nixpkgs.
That said, some comments on that same issue do mention the value of VTE-based terminal emulators, and how Termite in particular excels, so perhaps further discussion is warranted(?)

Is there a deprecated flag in meta like there is for meta.broken?

I don't think so :thinking:

There's probably no need to quickly react to this news, after all, the latest release was over 2 years ago. Perhaps it would be best to wait a few months to see if any community-maintained forks gain traction, and then flip it to that before the 21.11 branch, or remove it at that point?

That's a good point, should I close this issue or leave it open for further discussion in November?

The package has been dead for years now and no prominent fork emerged so far. And while it is possible that the announcement might trigger something, even against its recommendations, I do not consider anything but life support forks likely.
But we actually already have one such fork right in Nixpkgs. As long as @JJJollyjim or someone else wants to fix the occasional incompatibilities with newer versions of VTE and no security vulnerability is found, there is not much to do.
We could keep this open in case a new upstream appears but again, that is unlikely to happen so I would just close this and do nothing until the package stops building or otherwise breaks.

Oh! I wasn't aware of that. Fair enough, closing now :smile: .


Modifications of reboot script

The following modifications were discussed and should be implemented in the reboot script :

[x] The killing method should be the following :

kill -2 $(pgrep 'minetest')
sleep 30
kill -15 $(pgrep 'minetest')

[x] All repository cloning should use --depth=1 to (slightly) speed up cloning, and reduce data storage

Done and Done, the git clone xxx --depth=1 reduced drastically the number of file pulled ! (now 7000, before 41000)
Thank you for the suggestion

I just remembered that @cessna151 also proposed to move the hour at which the server update. Could we move it forward to 5AM (GMT+2)?

I scheduled the daily reboot to 6:31AM (the server is stopped between 6:29AM and 6:30AM)
The hypervisor VM backups are scheduled to 6:30AM
@LeMagnesium
Can you tweak the shutdown messages/mod for the new reboot time please ?

Je te renvoi le ticket @LeMagnesium
Le mod shutdown crsh le serveur à chaque message qu'il balance, voir cette issue : https://github.com/MinetestForFun/server-minetestforfun-skyblock/issues/137

Corrigé, je clos


Free shipping not applied on cart, if from action "Free Shipping" is selected and discount amount is given as zero.

Bug report
Title
Free shipping not applied on cart, if from action "Free Shipping" is selected and discount amount is given as zero.
Issue Description
Free shipping not applied on cart, if from action "Free Shipping" is selected and discount amount is given as zero.
Preconditions
2. Commit id.-f0e5bfdae76e77dc49c4acce05912c90cb01eecf

Steps to reproduce
1. step1- Create a cart rule and set discount amount to zero and select "Free Shipping" as Yes.
2. step2- Place an order

Expected result
Shipping charge should be zero
Actual result
Shipping charges get applied

Framework Version: 0.1.9

Fixed


Don't count NO_ERROR GOAWAY as a failure in HTTP health checker

Title: Don't count NO_ERROR GOAWAY as a failure in HTTP health checker
Description:
We've observed that the HTTP health checker implementation treats GOAWAY frames with the no_error code as an unhealthy health checks, which means that listener updates are not hitless if you use unhealthy_threshold of 1 for your http health check config
It looks like this was fixed for gRPC in https://github.com/envoyproxy/envoy/pull/11324, so we probably just want to implement that same logic for the HTTP health checker.

Closing as duplicate ofhttps://github.com/envoyproxy/envoy/issues/13544 which explains the issue more accurately


DropdownMenu support

Hi,
Is there any plans to support the new DropdownMenu from Flutter: https://api.flutter.dev/flutter/material/DropdownMenu-class.html
Or I can use it already and I'm missing anything?
Thanks.

Thank you very much @vasilich6107


css changes cause to refreshing page, can we apply css injection without browser refresh

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
changing css cause to refreshing page, its very hard for big projects.
Expected behavior
ionic solve that issue but i couldnt found also hot module replacement not useful for me.
thanks for help
Minimal reproduction of the problem with instructions

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular Seed Version: aaaaf75

Node: node --version =

No, applying CSS changes requires browser refresh as well.


Expanded access check to check application item

Checking app item itself, which covers webmap-based and group-based apps, in addition to the webmap for webmap-based apps

Looks good thanks @MikeTschudi


SQLite implementation

Re-basing and working on further improvements now

@sergeyshaykhullin , I have made all the improvements that were discussed. Could you give it a review?

Currently fixing the failing pipeline and trying to setup a scheduler test for the other database types

@sergeyshaykhullin Okay I have fixed the failing SQLite tests. I was also looking at https://dotnet.testcontainers.org/ (video about it) but maybe in a seperate PR since this one is already huge


Create Person does not work

The input for the GraphQL mutation on CREATE is invalid. The issue is definitely FE, as it works fine in Playground.

#36 Fixes


Video thumbnails are not generating

Describe the bug
My video previews are enabled in the config.js and I have the fluent-ffmpeg installed, however, video previews are just showing up as the missing image icon.
To Reproduce

Go to uploads
Observe.

Expected behavior
See a video preview.
Screenshots

Do you have ffmpeg installed in your system?

Yes, I will try to reinstall it and refresh my instance.

Yes, I will try to reinstall it and refresh my instance.

Fixed by reinstalling ffmpeg and refreshing instance.

Now that this is functioning correctly, is there a way to get thumbnails for previously uploaded files?

Not built-in, but you can easily do it by making a script that loops over all the filenames of your uploaded files and calls this method on each one

don't know if you have already found a solution @fangshang132 but i came upon this thread and had this same problem myself. made this simple script that should solve it.

Yup, that'll do nicely

From the looks of it, just make sure you put it in the controllers folder, or any direct child folder from root.

thank you!

very cool, thanks kanye!


Improve reliability of IPFS snapshot retrieval

The current strategy of just pinning to a self-hosted IPFS node seems to not be good enough - snapshot retrieval often times out at first attempt, leading to bad UX.
We should either switch to a default gateway where this doesn't happen (not sure if one exists) or add some pinning service.

our IPFS node had very narrow limits for ConnsInbound (79).
Can be checked with
ipfs swarm limit system

After increasing that with
ipfs config --json Swarm.ResourceMgr.Limits.System.ConnsInbound 2000

and restarting the service, and also increasing the limit for StreamsInbound, the resource warnings in the log almost disappeared and the network traffic went up a lot to several Mbps.
A quick test suggests that the snapshot availability issue may have been solved with that. If that's confirmed, this issue can be closed.

also switching default IPFS gateway: https://github.com/superfluid-finance/superfluid-sentinel/pull/158


Fix update fid failure issue

Another thought is, the deleted table doesn't matter in taking over file access account. So in my PR, I ignore such table not found exception and only throw exception when all tables are not updated.

Another thought is, the deleted table doesn't matter in taking over file access account. So in my PR, I ignore such table not found exception and only throw exception when all tables are not updated.

Thanks for your contribution. Let's simplify the implementation by just ignoring table not found exception due to table eviction. See #2215.

Thanks for your contribution. Let's simplify the implementation by just ignoring table not found exception due to table eviction. See #2215.


Promise type

Hello,
forgive my ignorance, this a very simple (and probably silly) question: how to annotate a function returning a Promise?
I've read most of the official documentation and I haven't found a specific page dedicated to Promises.
I've seen a couple of threads here on github but things are not clear to me.
Also, when we specify a Promises return type how to specify the resolve/reject cases?
Thanks

function demo(): Promise<string> {
    return 'I love cats!';
}

Thanks @lll000111 for the example!
Any link to the official documentation?

You are looking at it (you opened in issue in the repo that holds the source).
https://github.com/facebook/flow/blob/master/lib/core.js#L594

Yes, you are right, I was talking about https://flow.org/en/docs/
Probably it is still not there?

Why don't you go ahead and update it? Contributors welcome :-)

Yes I will, but having a very limited experience with Flow I will wait a bit before writing stupid things :]

Hey folks, this topic was extremely helpful to me. So I added a doc page per request - https://github.com/facebook/flow/pull/4927 - did I do that right?

Is there official documentation already? This is over 4months old issue but I still can not find the promises in docs :(.

@VladimirPittner You won't find a lot of things which is hard to duplicate in docs. Just look at lib defs
https://github.com/facebook/flow/blob/master/lib/core.js#L591

@TrySound sorry I do not understand what do you mean by 'hard to duplicate in docs'. If I would want to search for things in actual code, I would not look for documentation, but thanks.
PS: Why don't actually drop all docs and just read the code, its probably self explanatory ;).

how to specify the resolve/reject cases?

Reject: You don't - that's not a return type, that's an exception.

How should you specify what type of argument will be passed to .reject() then?

By convention reject should always get an Error. This is reflected in the standard definitions:
https://github.com/facebook/flow/blob/master/lib/core.js#L615

I've just realized that #4927 was never merged and Promises are still not very well documented in the docs page. Is there any reason for that?

Sure glad I found this issue, I wasted two hours of my day trying different things.

Suffering from this in 2021

@chmiiller considering the lack of interaction here it seems to me that not even at facebook they are using flow.
From a dev point of view I gave up on this

I've since moved onto TypeScript, it makes more sense.


Excessive database load retrieving users when using MySQL with millions of users

What version of UAA are you running?
The problem was found after upgrading from 77.1.0 to 77.3.0.
How are you deploying the UAA?
I am deploying the UAA as part of a commercial Cloud Foundry distribution.
What did you do?
A customer has millions of users in a MySQL database in production. We're getting queries like this that are taking more than 10 minutes, and they're stacking up and bogging down the database, causing CF commands to time out. They originated from calls to cf's /v3/users endpoint.

select count(*) from users u join identity_provider idp on u.origin = idp.origin_key and u.identity_zone_id = idp.identity_zone_id where idp.active is true and (((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((((LOWER(u.id) = LOWER('...id here...') OR LOWER(u.id) [...]

This caused an outage, so they rolled UAA back to 77.1.0 and saw the database load return to normal. They don't have any issues in their testing environment that only has a few thousands of users.
We see that LOWER(u.id) is indexed for postgres, but we don't create an index for mysql.

@swalchemist @bruce-ricard @Tallicia FYI, still you should think about the UAA option database.caseinsensitive.
If a MYSQL landscape has set database.caseinsensitive=false, then this still can happen because for MYSQL you dont have the needed indexes
For postgresql this parameter cannot be used , but for MYSQL it can be used.
See
database.caseinsensitive

@Tallicia Please decide whether you want to keep this open after PR #2859.

@torsten-sap, let's close this issue, as the original reported issue has been resolved now.


Fix Length and Exists properties in MockFileInfo

Fix issue #440 , also when doing that issue notice Exists property is set to true when we set path to directory. (should be false as in normal FileInfo).

Looks great, thanks a lot for your contribution!


DRILL-4474: Use varchar for default column when all_text_mode is enab…

…led.

Can you generate the test file as part of the test rather than check in static?

Yes, I can do that.

Could you also close this PR and open a new one? the JIRA number was wrong in your commit so this is posting to the JIRA about incorrect creation if direct scans. The correct number is 4479

oops ... sorry, closing this and will reopen against the correct JIRA.


fixed instance variable reference

This is a very minor change. Why are you don't like return new $this->class();? It's a correct syntax too.

:-1:

ah, disregard. I thought $this->class() would evaluate to a method call that doesn't exist. Never mind :)


The equations of motion in trace_relativistic are not correct.

When I attempted to solve those problems mentioned by #25, I found the equations of motion in trace_relativistic are wrong.
The equations of motion in trace_relativistic are:
$$
\frac{\mathrm{d}\mathbf{v}}{\mathrm{d}t} = \frac{q}{\gamma m}(\mathbf{E}+\mathbf{v}\times\mathbf{B}),
$$
where $\gamma = 1/\sqrt{1-v^2/c^2}$.
These equations are not Lorentz covariant. The equations of motion based on four-momentum should be written as:
$$
\frac{\mathrm{d}p^\mu}{\mathrm{d}\tau} = \gamma q(E^\mu+\epsilon^{\mu \nu l}v_\nu B_l),
$$
where $\tau$ is proper time and $p^\mu = \gamma m v^\mu$. These equations are Lorentz covariant. And after a simplification, they can be written as:
$$
\frac{\mathrm{d}(\gamma v^\mu)}{\mathrm{d}t} = \frac{q}{m}(E^\mu+\epsilon^{\mu \nu l}v_\nu B_l), \tag{1}
$$
or
$$
\frac{\mathrm{d}v^\mu}{\mathrm{d}t} = \frac{q}{m \gamma^3}(E^\mu+\epsilon^{\mu \nu l}v_\nu B_l). \tag{2}
$$
And, I have found the accuracy of  Eq.(1) in numerical calculation is better than Eq.(2).

That's a nice check! Could you share a reference to the equations?

That's a nice check! Could you share a reference to the equations?

You can see https://en.wikipedia.org/wiki/Four-force or Jackson's Classical Electrodynamics 3rd  chapter 11.9. It can be derived directly from the equations of motion based on four-momentum.

I have a question about the current choice of Eq.(2) over Eq.(1) @TCLiuu: Could you remind me of the reasons behind this choice?


[Bug] Discord Client Freezes Up

Bug:
The whole Discord client freezes.
To Reproduce:
The bug occurs usually when writing up messages or when navigating through settings or DMs/Servers.
Expected Behavior:
Discord freezes for a solid 30 seconds before resuming regular functionality. I'm able to move my cursor but cannot click anything within Discord as it's unresponsive. During the time that Discord freezes, incoming messages or animations within the application do not respond and update until the freeze is over.
Screenshots

I do not know if having a bunch of plugins is a problem. However, I'll be testing each and every one of them when I have time to try to see if any of the plugins causes the unresponsiveness.
Discord Version: Stable
Additional Context
This bug can sometimes occur for me without having any plugins installed at all.

I do not know if having a bunch of plugins is a problem.

Try disabling all of them, reload Discord, then see if the issue persists.

@rauenzi It still happens however less frequently this time. Should I do a clean install of both Discord and BandagedBD?

Worth a try

@rauenzi Oh wait. I think I may have solved my problem. I had an instance of Discord running in Docker and the Message Logger caused my Windows Installation of Discord to freeze up while the Instance in Docker crashed as the plugins were in the same directory. Both installations of Discord were using it at once and ig t causes confusion and Discord just freezes up.

Thank you for the help anyhow.

For the record, message logger is unsupported.

@rauenzi Well thank you for letting me know. I never knew that.


No known class method for selector 'reportNewIncomingCall'

I changed it to this

I changed it to this

[RNCallKeep reportNewIncomingCall:uuid
handle:handle
handleType:@"generic"
hasVideo:YES
localizedCallerName:callerName
supportsHolding:YES
supportsDTMF:YES
supportsGrouping:YES
supportsUngrouping:YES
fromPushKit:YES
payload:nil
withCompletionHandler:nil];
v3 to v4


[Terre2] Bugfix

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

correction des 10 qui explosent sur les jets de caractéristique
correction des textarea indépendants de la bio
correction du langage par défaut

Character Sheet Info Roll20 Internal Use only.

Hello,
I don't understand why the 'new sheet' label is automatically applied. This is not a new sheet, I just wanted to make an update. What did I do wrong ?


Assets/Examples/Example1/src/Program.cs(115,17): error CS0117: System.Console' does not contain a definition for KeyAvailable'

Assets/Examples/Example1/src/Program.cs(115,17): error CS0117: System.Console' does not contain a definition for KeyAvailable'

Example1 is a console app, which could be built & run under vs2015/vs2017.
I've update the source of Example1 to support Unity now.   Take a try?


Joining forces with other Obsidian draw.io plugin developer?

Hi there,
I wrote in the other developers account the same message. I just wanted to give a heads up, that there is another person doing the same thing. Maybe it would be a good fit to join forces?
https://github.com/jensmtg/obsidian-diagrams-net/issues/23
Thanks for your work so far!
Greetings from Germany.

This would be great. I love diagrams.net over excalidraw and Canvas for many things. But the fact that neither of these plugins are maintained is such a shame

Welcome to try my modified version of obsidian-diagrams-net, added several improvements

Changed how it opens on the TAB and now it will load into the modal window
Added a Dark mode, which follows the obsidian setting
Double-click to enter the edit
Fixed the issue of refreshing after editing in edit and preview modes


Joining your Linux instance to AD?

This does not work, in part because the hostname for the Linux instance is the same as your machine, and I think in part because of other failures. The last time I tried to do this I broke a lot of things so I'm reticent to try it again. Basically, I'd like to be able to use/log in as the same AD accounts I use in Windows as I do in Ubuntu on my machine, but sssd, etc. can't join correctly. Is there a plan to make this work or is there some other way of making it work that I'm not seeing?

Your Windows build number:  10.0.16299.15

If it's a hostname problem i'm pretty sure you can just use hostname to change it?


Added link to website

We're linking to the github repos directly in various marketing content, so we need to direct users back to the main site.
Description

Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @mrmrcoleman, thanks for this contribution! is it a duplicate of https://github.com/tinkerbell/pbnj/pull/53?
Also, looks like also you'll need to have the "Signed-off-by" line. if you're not familiar with this I usually just do that by adding a -s to my git commit command.
https://probot.github.io/apps/dco/
https://github.com/tinkerbell/pbnj/pull/52/checks?check_run_id=1558341252

@jacobweinstock This is it a duplicate of #53, thanks for catching that. I'll take a look at your comment above later on.

Hey @mrmrcoleman, looks like there has been a few changes to the README.md since you proposed this. Mind rebasing master?
Also, looks like you might need the DCO sign-off for the commit still.

@mrmrcoleman, please let me know if I can assist with this at all.

Cleaned up into #89. Closing this one.


wifiNetworks returns an empty list on macOS >= 14.4

Describe the bug
Before macOS 14.4, wifiNetworks worked correctly, and returned Wi-Fi network information. On macOS 14.4 and later, it returns an empty list.
This is because they dropped support for the airport utility which systeminformation is dependent on on macOS.
When running the airport command directly, it returns this:
/System/Library/PrivateFrameworks/Apple80211.framework/Versions/Current/Resources/airport -s -x
WARNING: The airport command line tool is deprecated and will be removed in a future release.
For diagnosing Wi-Fi related issues, use the Wireless Diagnostics app or wdutil command line tool.

See more discussion here: https://www.reddit.com/r/MacOS/comments/1bjjchk/rip_airport_cli_macos_sonoma_144_removes_the/
To Reproduce
Steps to reproduce the behavior:

Make sure you have a Mac running 14.4 or later.
Create a tiny script that runs wifiNetworks.
Observe that it returns an empty array.

Current Output
[]
Expected behavior
Should return list of available Wi-Fi networks.
Environment (please complete the following information):

systeminformation package version: 5.22.10
OS: macOS 14.4 or later
Hardware: Macbook

To get all needed environment information, please run the following command:
  npx systeminformation info

│  SYSTEMINFORMATION                                                     Version: 5.22.10 │
└─────────────────────────────────────────────────────────────────────────────────────────┘

Operating System:
──────────────────────────────────────────────────────────────────────────────────────────
Platform         : darwin
Distro           : macOS
Release          : 14.4.1
Codename         : macOS Sonoma
Kernel           : 23.4.0
Arch             : arm64
Codepage         : UTF-8
Build            : 23E224

System:
──────────────────────────────────────────────────────────────────────────────────────────
Manufacturer     : Apple Inc.
Model            : MacBookPro
Version          : 18,1
Virtual          :

CPU:
──────────────────────────────────────────────────────────────────────────────────────────
Manufacturer     : Apple
Brand            : M1 Pro
Family           : 458787763
Model            :
Stepping         : 4
Speed            : 2.4
Cores            : 10
PhysicalCores    : 10
PerformanceCores : 8
EfficiencyCores  : 2
Processors       : 1
Socket           : SOC

@VladislavGatsenko ... wow I did not know about that. This is awesome. I will try to come up with a solution.

@VladislavGatsenko @accnops Version 5.23.17 just released. Tested on different macOS versions ... works now again.


how to auto refresh when the token expired?

I want refresh the token by token or refresh_token if token is expired.but symfony not has any middleware like laravel.So,if token is expired ,it will dispatch a event named JWTExpiredEvent and return a response from that event.But how to return the old response like /users/me,/articles/create after I refresh_token in JWTExpiredEvent event?
Not by send refresh token uri by frontend
My code
  public function onJWTExpired(JWTExpiredEvent $event)
    {
        $event->setResponse($this->autoRefreshToken($event, [
            'code' => '50008',
            'message' => self::$errorMessages['error-token']
        ]));
    }
    protected function autoRefreshToken(JWTFailureEventInterface $event, $errorData = [])
    {
        $encoder = new JsonEncoder();
        $errorData['code'] = $errorData['code'] ?? 50002;
        $errorData['message'] = $errorData['message'] ?? 'Bad credentials.';
        /** @var JWTAuthenticationFailureResponse $response */
        $response = $event->getResponse();
        $this->setRefreshTokenFromRequestHeaders($encoder);
        /** @var JWTAuthenticationSuccessResponse|JWTAuthenticationFailureResponse $result */
        $result = $this->RefreshToken->refresh($this->request);

        if ($result instanceof JWTAuthenticationSuccessResponse) {
            // Here I want to return the response of 'users/me'
            // And add the token info by `$result->getContent()` to response header
            // How to do it?
        }
        $response->setData($errorData);
        return $response;
    }

    protected function setRefreshTokenFromRequestHeaders(JsonEncoder $jsonEncoder)
    {
        if ($this->request->headers->has('Refresh-Token')) {
            $RefreshToken = $this->request->headers->get('Refresh-Token');
            if (false !== strpos($this->request->getContentType(), 'json')) {
                $params = !empty($content) ? $jsonEncoder->decode($content,'json') : array();
                $params['refresh_token'] = isset($params['refresh_token']) ? trim($params['refresh_token']) : $RefreshToken;
                $this->request->initialize(
                    $this->request->query->all(),
                    $this->request->request->all(),
                    $this->request->attributes->all(),
                    $this->request->cookies->all(),
                    $this->request->files->all(),
                    $this->request->server->all(),
                    $jsonEncoder->encode($params,'json')
                );
            } else {
                $this->request->attributes->set('refresh_token', $RefreshToken);
            }
        }
    }

How to do it?please Help me,thx!

If you want to refresh the token manually on the request which dispatches jwt expired event then you don't need this library at all. You could use JWTManager from the lexik jwt to create a new token and clone original request and then replacing the jwt with the new one generated:
$request = $this->container->get('request_stack')->getCurrentRequest();
$request->attributes->set('Authentication', 'Bearer NEW_TOKEN');

$newResponse = $this->container->get('http_kernel')->handle($request->duplicate(), HttpKernelInterface::SUB_REQUEST);

$event->setResponse($newResponse);

@norkunas Yes, this is a great solution. But I have another way to solve this problem,
define a custom JWTTokenAuthenticator and handle it in getCredentials,and get the new token by refresh bundle,set the new token to request header.At last,get the token of request headers and set it to response headers in a event subscribe,I test it,it can work fine.Is this writing consistent with the convention?
the code is as follows:
class JWTTokenAuthenticator extends BaseAuthenticator
{
...
    public function getCredentials(Request $request)
    {
        $tokenExtractor = $this->getTokenExtractor();

        if (!$tokenExtractor instanceof TokenExtractorInterface) {
            throw new \RuntimeException(sprintf('Method "%s::getTokenExtractor()" must return an instance of "%s".', __CLASS__, TokenExtractorInterface::class));
        }

        if (false === ($jsonWebToken = $tokenExtractor->extract($request))) {
            return;
        }

        $preAuthToken = new PreAuthenticationJWTUserToken($jsonWebToken);

        try {
            if (!$payload = $this->jwtManager->decode($preAuthToken)) {
                throw new InvalidTokenException('Invalid JWT Token');
            }

            $preAuthToken->setPayload($payload);
        } catch (JWTDecodeFailureException $e) {
            if (JWTDecodeFailureException::EXPIRED_TOKEN === $e->getReason()) {
                $refreshToken = $this->container->get("gesdinet.jwtrefreshtoken");
                $refreshResult = $refreshToken->refresh($request);
                if ($refreshResult instanceof JWTAuthenticationSuccessResponse){
                    $encoder = new JsonEncoder();
                    $data = $encoder->decode($refreshResult->getContent(),'json');
                    $preAuthToken = new PreAuthenticationJWTUserToken($data['access_token']);
                    $payload = $this->jwtManager->decode($preAuthToken);
                    $preAuthToken->setPayload($payload);
                    $request->headers->set('token',$data['access_token']);
                    return $preAuthToken;
                }
                throw new ExpiredTokenException();
            }

            throw new InvalidTokenException('Invalid JWT Token', 0, $e);
        }

        return $preAuthToken;
    }
}

Your solution looks like overcomplicated :) But if it works for you then all fine :) But what's the reason of using gesdinet jwt refresh, when you are not storing the refresh token in client side? Backend can create a new token without it

@norkunas
The features I need:

check whether the request contains an Authentication token and is a valid token. If everything is ok ,application will try to authenticate with this token
If the authentication fails, it will check if the token expires, if token has expired, it will try to refresh the token
If the refresh fails, that refresh_token has expired too, and user have to re-input credentials to log in.
If the refresh is successful, return a new token
All of above will be run in backend api.
Client only send Authentication and get the error or new token to store.
So,How to code it will be better solution?

Better solution would be store refresh token in client side and do token refresh before token expiration or when token expired :)

The answer is perfect,and to do it in code is very difficultly.can you let the $refreshToken->refresh method independent with out return response?

Did you apply the security control on your get info endpoint? Because I think your subscriber shouldn't be reached at all if there is no token.

@norkunas
'Did you apply the security control on your get info endpoint?',whether or not I apply the security control ,the result of $this->tokenStorage->getToken() is still be none when $event->setResponse($newResponse);,but $refreshResult->getContent() can get the data of new token info. and $refreshResult instanceof JWTAuthenticationSuccessResponse
because of api platform,I config the security control like this:
        itemOperations:
            getInfo:
                method: 'GET'
                path: '/users/info'
#                controller: App\Entity\User\GetMe
                access_control: 'is_granted("user.view-own")'
                access_control_message: 'Sorry, only the user himself or super admin can view his own detail info.'

the event method like this:
public function onJWTExpired(JWTExpiredEvent $event)
 {
   $request = $this->request->duplicate();
   $refreshToken = $this->container->get("gesdinet.jwtrefreshtoken");
   $refreshResult = $refreshToken->refresh($request);
   if ($refreshResult instanceof JWTAuthenticationSuccessResponse) {
       $encoder = new JsonEncoder();
            $data = $encoder->decode($refreshResult->getContent(), 'json');
            $request->headers->set('token', $data['access_token']);
            $request->headers->set('Authorization','Bearer '.$data['access_token']);
            $newResponse = $this->container->get('http_kernel')->handle($request, HttpKernelInterface::SUB_REQUEST);
            $event->setResponse($newResponse);
   }else{
     ...
    }
}

subscriber event
public static function getSubscribedEvents()
{
...
       'kernel.request' => [
                ['getInfo', EventPriorities::PRE_READ],
            ],
}

and I get user in getInfo method by $user = $this->tokenStorage->getToken()->getUser();
Is the subscriber error that it can not get the token in kernel.request event?


libsnappy name change on ubuntu 16

The name of libsnappy1 changed in ubuntu 16

Thanks for the PR.
Travis is recommending to change to the following per ruby style guidelines:
when 'debian'
  pkgs += if node['platform_version'] == '16.04'
            ['libsnappy1v5', 'libsnappy-dev']
          else
            ['libsnappy1', 'libsnappy-dev']
          end

I might also suggest if node['platform_version'].to_i >= 16' assuming later ubuntu versions are also using 1v5.

Thanks for the contribution!
The travis mixlib-shellout error is fixed in #338.  Merging


Update groups-concept.md

Hello, I'm making this change to clarify customers that the eligible nested group assignment is the only supported scenario in the Privileged Access group feature and this has been clarified internally under ICM 361088910.
Hope this makes sense and please reach out to<EMAIL_ADDRESS>if you have any questions related to this change.

@rolyon
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Contacted vrviriya about this addition. We will handle this doc update in a separate pull request.

#please-close


feat: Initial Probuf changes Proto2 to Proto3

This doesn't look like it includes the new dependency

Hi can you give more insight , what do mean here ?

This doesn't look like it includes the new dependency

Hi can you give more insight , what do mean here ?

I mean checked out this branch and it still had the old protobuf library instead of the new one. I would expect there would be change with a .csproj file


[BUG] An unhandled exception occurred. Start index cannot be larger than length of string.

Cannot use toolkit at all. After the initial settings screen it shows error "Start index cannot be larger than length of string. (Parameter StartIndex)"
Version 1.4 windows, freshly downloaded.
I have added 4 folders, A111 output, standalone ComfyUI output, and an archive folders.
==============================================================================
09/11/2023 11.30.52: Initializing pages
09/11/2023 11.30.52: Loading models
09/11/2023 11.30.52: An unhandled exception occured: startIndex cannot be larger than length of string. (Parameter 'startIndex')
at System.String.Substring(Int32 startIndex, Int32 length)
at Diffusion.Toolkit.MainWindow.LoadModels()
at Diffusion.Toolkit.MainWindow.OnLoaded(Object sender, RoutedEventArgs e)
at System.Threading.Tasks.Task.<>c.b__128_0(Object state)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.DispatcherOperation.InvokeImpl()
at System.Windows.Threading.DispatcherOperation.InvokeInSecurityContext(Object state)
at MS.Internal.CulturePreservingExecutionContext.CallbackWrapper(Object obj)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
at MS.Internal.CulturePreservingExecutionContext.Run(CulturePreservingExecutionContext executionContext, ContextCallback callback, Object state)
at System.Windows.Threading.DispatcherOperation.Invoke()
at System.Windows.Threading.Dispatcher.ProcessQueue()
at System.Windows.Threading.Dispatcher.WndProcHook(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)
at MS.Win32.UnsafeNativeMethods.DispatchMessage(MSG& msg)
at System.Windows.Threading.Dispatcher.PushFrameImpl(DispatcherFrame frame)
at System.Windows.Threading.Dispatcher.PushFrame(DispatcherFrame frame)
at System.Windows.Threading.Dispatcher.Run()
at System.Windows.Application.RunDispatcher(Object ignore)
at System.Windows.Application.RunInternal(Window window)
at System.Windows.Application.Run()
at Diffusion.Toolkit.App.Main()

I have the same error on startup after update to 1.4 and fresh install

when I click OK the program shut down

Getting this too now.

Did you set up the path to cache.json?
Can you post your cache.json?

Did you set up the path to cache.json?
yes

Can you post your cache.json?
It's pretty huge, zipped it and attached.
cache.zip

I have some fixes, please try redownloading the same version
https://github.com/RupertAvery/DiffusionToolkit/releases/tag/v1.4

I have some fixes, please try redownloading the same version

It is working now, thank you for the fix!


About training time

Hi author, when I trained with the 3090, I realized that the training time is expected to reach about 20 days, and I would like to know how long your training time is.Thanks!

The number of iterations I chose is the 800,000

Actually you can determine the training time by yourself.
800，000 is an number by experience.


Referees

Sorry, dumb question: there's only one classification item, and it's based on crowdhuman at some initial point, so why doesn't it pick up the referee? is that at the backbone/classification (yolox) level or the tracking/MixViT level?

Hello,
In our SportsMOT dataset, we only focus on all the players, so the referees are not annotated.


No such file or directory: 'data/test'

Thanks for creating this.
I was trying this for the first time today.
I ran
cd data/
bash download_dataset.sh

then
cd ..
python3 test_benchmark.py -a srgan --pretrained --gpu 0 data

I get
FileNotFoundError: [Errno 2] No such file or directory: 'data/test'
I'm unsure on what should go into the test folder

closing as duplicate of https://github.com/Lornatang/SRGAN-PyTorch/issues/20


Challenge http://www.freecodecamp.com/challenges/waypoint-use-clockwise-notation-to-specify-the-margin-of-an-element has an issue

Directions say :Let's try this again, but with margin this time. Use Clockwise Notation to give an element a margin of 40 pixels on its top and left side, but only 20 pixels on its bottom and right side.
But criteria say
Your green-box class should give the top of elements 20px of margin.
Your green-box class should give the left of elements 40px of margin.
Your green-box class should give the bottom of elements 20px of margin.
Your green-box class should give the right of elements 20px of margin.
Accepts
margin:40px 20px 20px 40px;
So just need to correct the px in the criteria for match to 40px on top.
Cheers!

Thanks, closing this as a duplicate of #665


[#150010843] Upgrade cflinuxfs2 to 1.145.0

What
Upgrade cflinuxfs2 to 1.145.0 specifically for:

https://www.cloudfoundry.org/usn-3363-1/
https://www.cloudfoundry.org/usn-3363-2/

1.146.0 is available, but it has only been out for <24hrs and doesn't appear
to address any CVEs/USNs.
How to review
Deploy from a branch and check that all of the acceptance tests pass.
I have done this in a dev environment, so I think it would be acceptable to code review only and let CI catch any mysterious test regressions.
Who can review
Anyone.

Ready for review.


[discussion] Scala 2.12 support

This PR bumps dependencies to Scala 2.12 versions.
The output of running tests is:
[gkk@mbp ~/stripe/rules_scala (gkk-scala-2.12)]$ bazel test //test/...
INFO: Found 361 targets and 31 test targets...
ERROR: /Users/gkk/stripe/rules_scala/test/jmh/BUILD:23:1: Generating benchmark code for //test/jmh:test_benchmark_codegen failed: Process exited with status 1 [sandboxed].
Exception in thread "main" java.lang.RuntimeException
	at org.objectweb.asm.MethodVisitor.visitParameter(Unknown Source)
	at org.objectweb.asm.ClassReader.b(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.openjdk.jmh.generators.asm.ASMGeneratorSource.processClass(ASMGeneratorSource.java:60)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$3(BenchmarkGenerator.scala:140)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$3$adapted(BenchmarkGenerator.scala:139)
	at scala.collection.immutable.List.foreach(List.scala:389)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$2(BenchmarkGenerator.scala:139)
	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.withClassLoader(BenchmarkGenerator.scala:97)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$1(BenchmarkGenerator.scala:134)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$1$adapted(BenchmarkGenerator.scala:127)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.withTempDirectory(BenchmarkGenerator.scala:107)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.generateJmhBenchmark(BenchmarkGenerator.scala:127)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.main(BenchmarkGenerator.scala:43)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator.main(BenchmarkGenerator.scala)
Use --strategy=Generating=standalone to disable sandboxing for the failing actions.
FAIL: //test/aspect:scala_test (see /private/var/tmp/_bazel_gkk/bdf28b460d8a4ef6c8e29d02646a3035/execroot/io_bazel_rules_scala/bazel-out/darwin_x86_64-fastbuild/testlogs/test/aspect/scala_test/test.log).
INFO: Elapsed time: 2.312s, Critical Path: 1.30s
//test:HelloLibTest                                             (cached) PASSED in 7.0s
//test:HelloLibTestSuite_test_suite_HelloLibTest.scala          (cached) PASSED in 7.0s
//test:HelloLibTestSuite_test_suite_HelloLibTest2.scala         (cached) PASSED in 6.9s
//test:JunitFilterTest                                          (cached) PASSED in 1.4s
//test:JunitFiltersAbstractClassesAndInterfaces                 (cached) PASSED in 1.3s
//test:JunitFiltersClassesWithoutTests                          (cached) PASSED in 1.3s
//test:JunitFiltersInnerClasses                                 (cached) PASSED in 1.9s
//test:JunitIncludesClassesWithRunWith                          (cached) PASSED in 1.4s
//test:JunitIncludesClassesWithTestsFromParent                  (cached) PASSED in 1.4s
//test:JunitMultiplePrefixes                                    (cached) PASSED in 1.2s
//test:JunitMultipleSuffixes                                    (cached) PASSED in 1.3s
//test:JunitPrefixesAndSuffixes                                 (cached) PASSED in 1.3s
//test:JunitTestWithDeps                                        (cached) PASSED in 1.3s
//test:RunJavaBinary                                            (cached) PASSED in 0.9s
//test:RunJavaBinary2                                           (cached) PASSED in 1.0s
//test:RunJavaOnlySources                                       (cached) PASSED in 0.5s
//test:RunMixJavaScalaLibBinary                                 (cached) PASSED in 1.4s
//test:RunResourcesStripScalaBinary                             (cached) PASSED in 0.9s
//test:RunScalaBinary                                           (cached) PASSED in 0.9s
//test:RunScalaLibBinary                                        (cached) PASSED in 1.0s
//test:Specs2Tests                                              (cached) PASSED in 3.7s
//test:TestFilterTests                                          (cached) PASSED in 7.4s
//test:TestScalaBinaryInGenrule                                 (cached) PASSED in 0.1s
//test/aspect:aspect_test                                       (cached) PASSED in 0.1s
//test/aspect:scala_junit_test                                  (cached) PASSED in 2.7s
//test/aspect:scala_specs2_junit_test                           (cached) PASSED in 3.2s
//test:data_location_expansion                                  (cached) PASSED in 3.7s
//test:no_sig                                                   (cached) PASSED in 0.8s
//test/src/main/scala/scala/test/resource_jars:resource_jars    (cached) PASSED in 7.7s
//test/src/main/scala/scala/test/resources/strip:resouceStripPrefixTest (cached) PASSED in 4.7s
//test/aspect:scala_test                                                 FAILED in 1.3s
  /private/var/tmp/_bazel_gkk/bdf28b460d8a4ef6c8e29d02646a3035/execroot/io_bazel_rules_scala/bazel-out/darwin_x86_64-fastbuild/testlogs/test/aspect/scala_test/test.log

Executed 1 out of 31 tests: 30 tests pass and 1 fails locally.
There were tests whose specified size is too big. Use the --test_verbose_timeout_warnings command line option to see which ones these are.
[gkk@mbp ~/stripe/rules_scala (gkk-scala-2.12)]$

Can one of the admins verify this patch?

@ittaiz you might want to take a look.

@jeremydhoon any ideas about the jmh failure when we bump to 2.12?

Using this branch combined with --strategy=Scalac=worker, I have to bazel clean whenever the public interface in a file changes. For example, if object Foo {} in One.scala gets a new method bar, Two.scala cannot see Foo.bar until I bazel clean. It's a similar situation if the type signature changes.

Would it be possible to make 2.12 a configurable option or kept on a separate branch/tag until that's possible? I ask because the upgrade path for scala is kind of painful when it comes to 3rd party scala library dependecies for even minor version bumps in the language. I indend to pull in these bazel rules in more often that I upgrade scala versions and i dont have plans to upgrade to 2.12 soon. That may be just me though. I can see the inverse side of this with these rules locked into 2.11 being a blocker for those looking forward to to 2.12 upgrade ( and all of the fun 3rd party lib upgrades that come along with that :) )

Two things:

the bazel-deps tool I wrote made it pretty easy for us to try this
internally.

I don't think we should merge this until we can make scala version
selectable. Definitely not force everyone into scala 2.12, and I'd rather
not have two branches, although that may be the only tractable thing if
ijar is really broken on 2.12.

On Thu, Sep 7, 2017 at 19:54 doug tangren<EMAIL_ADDRESS>wrote:

Would it be possible to make 2.12 a configurable option or kept on a
separate branch/tag until that's possible? I ask because the upgrade path
for scala is kind of painful when it comes to 3rd party scala library
dependecies for even minor version bumps in the language. I indend to pull
in these bazel rules in more often that I upgrade scala versions and i dont
have plans to upgrade to 2.12 soon. That may be just me though. I can see
the inverse side of this with these rules locked into 2.11 being a blocker
for those looking forward to to 2.12 upgrade ( and all of the fun 3rd party
lib upgrades that come along with that :) )
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/251#issuecomment-328006039,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdtGjI9YfnlsJpkUERZt5qhikemwEks5sgNaAgaJpZM4OWWZP
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin

+1 to what @johnynek said
@jbalogh-stripe any chance you want to take a crack at seeing what happened to ijar with 2.12?
I'm guessing 2.12 will be a major deal for us in a few months but we're tied up in other matters
@softprops if you think you can find the time to help with #170 (scala toolchain) that could really advance us

the bazel-deps tool I wrote made it pretty easy for us to try this
internally.

The problem isn't necessarily bumping artifact identifiers, it's doing the work to find out if library authors have actually republished the same version for the newer scala version, and if they did making sure than that didn't accidentally come along with a new library version changes :). And if library authors haven't republished a library version for a newer version of scala, it's a problem being blocked until they do.

@jbalogh-stripe to be clear: is this only with workers enabled?

@jbalogh-stripe to be clear: is this only with workers enabled?

Right. Without Scalac=worker the code builds correctly when interfaces change.

Hi, sorry for dropping the ball on this. I was busy with other plots at Stripe. The 2.12 migration is back on my list of active threads now.
I think we should have a 2.12-specific branch where people interested in finding and fixing 2.12 upgrade issues can work together.
Independently, I agree #170 would be great to have and would help us folding 2.12-specific branch back to master.

Regarding interface changes, is ijar called only when scalac worker is involved?

I pushed a branch rebased on latest master. Still all tests except jmh one pass:
ERROR: /Users/gkk/stripe/rules_scala/test/jmh/BUILD:23:1: Couldn't build file test/jmh/test_benchmark_codegen.srcjar: Generating benchmark code for //test/jmh:test_benchmark_codegen failed (Exit 1).
Exception in thread "main" java.lang.RuntimeException
	at org.objectweb.asm.MethodVisitor.visitParameter(Unknown Source)
	at org.objectweb.asm.ClassReader.b(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.objectweb.asm.ClassReader.accept(Unknown Source)
	at org.openjdk.jmh.generators.asm.ASMGeneratorSource.processClass(ASMGeneratorSource.java:60)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$3(BenchmarkGenerator.scala:140)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$3$adapted(BenchmarkGenerator.scala:139)
	at scala.collection.immutable.List.foreach(List.scala:389)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$2(BenchmarkGenerator.scala:139)
	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:12)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.withClassLoader(BenchmarkGenerator.scala:97)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$1(BenchmarkGenerator.scala:134)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.$anonfun$generateJmhBenchmark$1$adapted(BenchmarkGenerator.scala:127)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.withTempDirectory(BenchmarkGenerator.scala:107)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.generateJmhBenchmark(BenchmarkGenerator.scala:127)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator$.main(BenchmarkGenerator.scala:43)
	at io.bazel.rules_scala.jmh_support.BenchmarkGenerator.main(BenchmarkGenerator.scala)
INFO: Elapsed time: 11.899s, Critical Path: 0.88s
//test:HelloLibTest                                             (cached) PASSED in 8.0s
//test:HelloLibTestSuite_test_suite_HelloLibTest.scala          (cached) PASSED in 7.2s
//test:HelloLibTestSuite_test_suite_HelloLibTest2.scala         (cached) PASSED in 7.5s
//test:JunitFilterTest                                          (cached) PASSED in 2.8s
//test:JunitFiltersAbstractClassesAndInterfaces                 (cached) PASSED in 2.6s
//test:JunitFiltersClassesWithoutTests                          (cached) PASSED in 2.4s
//test:JunitFiltersInnerClasses                                 (cached) PASSED in 2.5s
//test:JunitIncludesClassesWithRunWith                          (cached) PASSED in 2.4s
//test:JunitIncludesClassesWithTestsFromParent                  (cached) PASSED in 3.3s
//test:JunitJavaTest                                            (cached) PASSED in 2.7s
//test:JunitMultiplePrefixes                                    (cached) PASSED in 2.5s
//test:JunitMultipleSuffixes                                    (cached) PASSED in 1.0s
//test:JunitPrefixesAndSuffixes                                 (cached) PASSED in 2.6s
//test:JunitTestWithDeps                                        (cached) PASSED in 2.7s
//test:RunJavaBinary                                            (cached) PASSED in 0.6s
//test:RunJavaBinary2                                           (cached) PASSED in 3.1s
//test:RunJavaOnlySources                                       (cached) PASSED in 0.6s
//test:RunMixJavaScalaLibBinary                                 (cached) PASSED in 2.0s
//test:RunResourcesStripScalaBinary                             (cached) PASSED in 1.9s
//test:RunScalaBinary                                           (cached) PASSED in 1.8s
//test:RunScalaLibBinary                                        (cached) PASSED in 3.3s
//test:Specs2Tests                                              (cached) PASSED in 6.1s
//test:TestFilterTests                                          (cached) PASSED in 7.6s
//test:TestScalaBinaryInGenrule                                 (cached) PASSED in 0.2s
//test/aspect:aspect_test                                       (cached) PASSED in 0.1s
//test/aspect:scala_junit_test                                  (cached) PASSED in 1.4s
//test/aspect:scala_specs2_junit_test                           (cached) PASSED in 1.3s
//test/aspect:scala_test                                        (cached) PASSED in 7.1s
//test:data_location_expansion                                  (cached) PASSED in 3.6s
//test:no_sig                                                   (cached) PASSED in 1.0s
//test/src/main/scala/scala/test/resource_jars:resource_jars    (cached) PASSED in 6.5s
//test/src/main/scala/scala/test/resources/strip:resouceStripPrefixTest (cached) PASSED in 3.3s

Executed 0 out of 32 tests: 32 tests pass.

it seems like most, or all of these changes could be made to mainline by making separate repository loading functions suffixed by 2_12, which would make it possible to set a version by the user, but maybe not the standard bazel toolchain way.

on the surface, I agree. Would making  def scala_repositories()[0] parametric and take the Scala version as an input work in bazel? Or is that exactly what you have in mind?
[0] https://github.com/bazelbuild/rules_scala/blob/master/scala/scala.bzl#L1001

nice that you are getting closer. I think you have an asm mismatch. I wonder if scalac embeds a version of asm or if a transitive dependency does.
As far as making the functions take parameters, that could work too, but it is just going to be a giant if/else so I was suggesting just naming them: scala_repositories_2_12 ....
but either way if you like (not like we have compile safety in the rules).

Yep. There are 2.12 and 2.11 versions of later versions.
On Sat, Sep 9, 2017 at 00:48 Ittai Zeidman<EMAIL_ADDRESS>wrote:

@ittaiz commented on this pull request.
In WORKSPACE
https://github.com/bazelbuild/rules_scala/pull/251#discussion_r137923707
:

@@ -20,15 +20,15 @@ specs2_junit_repositories()

test adding a scala jar:
maven_jar(
name = "com_twitter__scalding_date",

artifact = scala_mvn_artifact("com.twitter:scalding-date:0.16.0-RC4"),
sha1 = "659eb2d42945dea37b310d96af4e12bf83f54d14"

artifact = scala_mvn_artifact("com.twitter:scalding-date:0.17.0"),

we can upgrade this one and the one below in mainline as well right?
guessing they have a 2.11 version
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/251#pullrequestreview-61660276,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdrn-rPta7wNmBRntL5dxBFZJZzzgks5sgm0UgaJpZM4OWWZP
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin

looks like this bug:
https://github.com/scala/bug/issues/10295
was hitting us internally when we tried this. I think scala 2.12.4 may have fixed it, but for now there is a work-around compiler flag that needs to be passed.

The bug is in classpath caching that can be disabled with -YdisableFlatCpCaching passed to the compiler. The #6064 has a fix for the caching logic. It's not part of an official release, though. 2.12.4 is planned to be out very soon: https://github.com/scala/scala/milestones
I'd like to verify that the bug is indeed fixed in 2.12 master. The concern I have is whether the cache invalidation supports symlinks properly that are used by bazel.

The problem with incremental compilation not working in 2.12 wasn't detected by the existing tests. I'm leaving a note: this points at the need to have some basic sanity checks for the scalac worker.
I'm wondering how we can move forward with enabling Scala 2.12 support. The refactor to make scala_repositories parametric or create 2.12 variant as oscar suggests would be the ideal solution.
I don't have the bandwidth to do the refactor and deal with the fallout at the moment (other Scala concerns eat up my time at Stripe).
My proposal is to create a 2.12-specific branch where people who care about 2.12 migration can work together and shake out the bugs like the one with incremental compilation.

thoughts?

actually #285 would have caught it.
We were not testing with workers, but workers are the "harder" case since they keep state. After that is merged CI should be faster and I think this would have been caught.
do you want to merge that branch in and see if it does catch this bug?

so this would catch the problem just in ci? what's being tested when I run bazel test //test/... locally?

I just verified that latest Scala 2.12 nightly build has classpath caching that works in bazel too.

If you turn on bazel workers in your bazel rc it would have also caught it.
On Fri, Sep 29, 2017 at 12:37 Grzegorz Kossakowski<EMAIL_ADDRESS>wrote:

so this would catch the problem just in ci? what's being tested when I run bazel
test //test/... locally?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/251#issuecomment-333255144,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJdsSJfP0R2bQWx8ytPdPk_SrJDMHuks5snXFFgaJpZM4OWWZP
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin

exciting!
So we don't have to give up all the performance I guess.
On Fri, Sep 29, 2017 at 12:39 PM, Grzegorz Kossakowski <
<EMAIL_ADDRESS>wrote:

I just verified that latest Scala 2.12 nightly build has classpath caching
that works in bazel too.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/251#issuecomment-333255306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEJds-xQD5ipSnTpIr5zZkNV25ySrzNks5snXGSgaJpZM4OWWZP
.

--
P. Oscar Boykin, Ph.D. | http://twitter.com/posco | http://pobox.com/~boykin

My proposal is to create a 2.12-specific branch where people who care about 2.12 migration can work together and shake out the bugs like the one with incremental compilation.

Sounds good. I estimate this will be a high priority for us in a month or two.

Excellent!
Ittai or Oscar, could you create a 2.12 branch off master? I’ll send a PR
with the latest iteration of bazel rules.
On 30 September 2017 at 11:16, Ittai Zeidman<EMAIL_ADDRESS>wrote:

My proposal is to create a 2.12-specific branch where people who care
about 2.12 migration can work together and shake out the bugs like the one
with incremental compilation.
Sounds good. I estimate this will be a high priority for us in a month or
two.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/pull/251#issuecomment-333326362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAQeftMA2vUjXNE1rttLVbI1hqQxnfLks5snoVugaJpZM4OWWZP
.

--
gkk
--
gkk

@gkk-stripe @gkossakowski https://github.com/bazelbuild/rules_scala/tree/scala2_12


Spec/implementation mismatch: Random bytes written to user_data in netcode_generate_connect_token

Based on the current spec, the user_data field in the private connect token should be user specified data (this means I assume the API should be able to receive user_data from an outside source). However, in netcode_generate_connect_token line 5081, a private connect token is generated with random bits filling the user_data field.
This seems like something that would be easy to fix with a user_data parameter in netcode_generate_connect_token. (I don't know C enough to be sure.)

Yes, the connect token generation is just for tests. You're welcome to update it if you wish.

Ahh, the RedpointGames netcode.io C# bindings uses this function as part of its API, so I got confused there. I don't know C too well, but I think I've studied the code well enough to make this very minor change (and run the tests).

Issue resolved in referenced pull request: #79


Fix links to examples in main README and landing page

Coverage remained the same at 84.281% when pulling d3707c22ad35673231dd30d9c519fc26134efbcf on mziccard:fix-example-links into f76f1c9916ef87c9fb1f213d28bd3f771d379bf1 on GoogleCloudPlatform:master.

/cc @lesv

LGTM - though you might wish to add https://github.com/GoogleCloudPlatform/java-docs-samples/tree/master/appengine/guestbook-cloud-datastore to the list.
(Didn't realize you were tracking these things, or I'd have fixed the links for you.

Coverage remained the same at 84.281% when pulling af7c0b31a613ac616439312706a7396f8fd59a06 on mziccard:fix-example-links into f76f1c9916ef87c9fb1f213d28bd3f771d379bf1 on GoogleCloudPlatform:master.

Coverage increased (+0.005%) to 84.286% when pulling 3a1fd9e15ba849ecb2475a5c0b95fac6b37bb87c on mziccard:fix-example-links into f76f1c9916ef87c9fb1f213d28bd3f771d379bf1 on GoogleCloudPlatform:master.


refactor(batcher): reduce the number of arguments to create_block_builder

This change is

#1934
#2011  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @dafnamatsry and the rest of your teammates on  Graphite


Using dropout makes training very slow

I tried using dropout and it slows down training a lot for me (like 10x slower).
Looking at net/bilstm-parallel-layer.h:177, this code is the only difference compared to training the network without dropout:
  if (drop_factor_ != 0.0) {
    drop_mask_.Resize(T*S, 2 * cell_dim_, kUndefined);
    drop_mask_.SetRandUniform();
    drop_mask_.Add(-drop_factor_);
    drop_mask_.ApplyHeaviside();
    YR_RB.RowRange(S,T*S).MulElements(drop_mask_);
  }

I can see that the dropout code calls SetRandUniform(), which in turn creates a new CuRand tmp; on every call. Might this slow down training because the GPU is reseeded every time? I also suspect that seeding is done on the host, as this comments in gpucompute/cuda-rand.cc suggests it:
// optionally re-seed the inner state
// (this is done in host, for good performance it is better to avoid re-seeding)
int32 tgt_size = probs.num_rows_ * probs.stride_;
if (tgt_size != state_size_) SeedGpu(tgt_size);

I will try to reuse the CuRand object and see what happens... meanwhile, if someone else encountered the same problem, let me know.

Reusing the CuRand object does help! I'm guessing that seeding the CuRand object was happening on the host, not on the GPU. However my quick fix will only work, if all the layers that use Dropout have the same size.


Fixed #1004

What does this changes
Fixed tone_detector bug
How this fixes it

Fixes #1004
Your checklist for this pull request

[x] Passed code styles and structures
[x] Passed code linting checks and unit test

coverage: 52.823% (-0.1%) from 52.954%
when pulling 6f1dc929f853ecd9910d1d25a56716b864b2ee98 on fix-1004
into 188904f22c4c80fd7406706850c798f611690a4a on dev.


SerialPortError: Bad file descriptor

Where can I find our more about the above error?
The program crashed when the above happened.

same here

This is funny because I already solved it. It is just because my SerialPort only can read, but I wrote canWrite in my code.


Clients making a connection to (questionable?) IP not added to host list

Hi!
I just started using Quasar a couple days ago. While installing a client to a computer, I had a program called "Process Hacker" open on the "Network" tab to make sure things were working as I was trying to tunnel it through my local home server to my laptop, and saw this unrecognized IP:

This IP is on AbuseIPDB.
I also built a client and ran it on my laptop, pointing it to itself (localhost), and the same IP showed up on that list right as the client prepared itself.
The confusing part is that none of the clients are configured to connect over the internet... just locally.
What may be happening here? Thanks?

I think I've asked this in the past and nothing to be concerned about but, I wouldn't mind a reminder.
I get a different connection each time
ec2-54-243-119-179.compute-1.amazonaws.com
ec2-50-19-243-236.compute-1.amazonaws.com
ec2-23-21-252-4.compute-1.amazonaws.com
Could be a stay alive thing or to get a geolocation

I think I've asked this in the past and nothing to be concerned about but, I wouldn't mind a reminder.
I get a different connection each time
ec2-54-243-119-179.compute-1.amazonaws.com
ec2-50-19-243-236.compute-1.amazonaws.com
ec2-23-21-252-4.compute-1.amazonaws.com
Could be a stay alive thing or to get a geolocation

These connections are HTTP requests to retrieve geo IP location, see here https://github.com/quasar/Quasar/issues/637

These connections are HTTP requests to retrieve geo IP location, see here https://github.com/quasar/Quasar/issues/637

Guess I didn't search enough terms on here before making this issue... sorry for the duplicate!

Guess I didn't search enough terms on here before making this issue... sorry for the duplicate!


Update Kustomize packages to v5.5.0

This PR updates the controller dependencies to Kustomize v5.5.0 and Kubernetes v1.31.3

@stefanprodan
Hi!
Could you also build the release candidate with Kustomize v5.5 for the helm-controller?
You’ve already built it for kustomize-controller, but we discovered that the bug we encountered is also caused by the helm-controller's embedded Kustomize.

Here are the RC images:

ghcr.io/fluxcd/helm-controller:rc-e8986629
docker.io/fluxcd/helm-controller:rc-e8986629

@stefanprodan
We're grateful for your help, everything is working now as a charm 🙌
I left the comment about the issue in crossplane kafka-provider repository as well.