Is node installed?

I've enabled node in my docker-compose.yml
    workspace:
        build:
            context: ./workspace
            args:
                - INSTALL_XDEBUG=false
                - INSTALL_MONGO=false
                - INSTALL_NODE=true
                - INSTALL_DRUSH=false
                - COMPOSER_GLOBAL_INSTALL=false
                - PUID=1000
                - PGID=1000
        volumes_from:
            - volumes_source
        extra_hosts:
            # IMPORTANT: Replace with your Docker Host IP (will be appended to /etc/hosts)
            - "dockerhost:<IP_ADDRESS>"
        tty: true

But when I enter the workspace container
docker exec -it laradock_workspace_1 bash
root@090e4d583908:/var/www/laravel# node -v
bash: node: command not found

Using the latest LaraDock code.
Maybe it's installed but not on my path
root@090e4d583908:/var/www/laravel# echo $PATH
/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/var/www/laravel/vendor/bin

Where would the node installation be located?

@kauhat @appleboy

Hi @nojaf,
Can you paste the output of nvm current and nvm use node?

root@d1488e25daa8:/var/www/laravel# nvm
bash: nvm: command not found
root@d1488e25daa8:/var/www/laravel# nvm current
bash: nvm: command not found
root@d1488e25daa8:/var/www/laravel# nvm use node
bash: nvm: command not found
root@d1488e25daa8:/var/www/laravel#

I did see in the logs that it was getting installed. I should have node 6.4

Hmm, how strange. Can you try exec'ing into a login shell using bash -l. That may load source some additional things.
Also: can you try running su laradock inside the container and try nvm etc again.

Using bash -l:
PS C:\Users\nojaf\Projects\hello-laravel\laradock> docker exec -it laradock_workspace_1 bash -l
mesg: ttyname failed: Success
root@915f54093e00:/var/www/laravel# nvm
bash: nvm: command not found

su laradock:
root@915f54093e00:/var/www/laravel# su laradock
laradock@915f54093e00:/var/www/laravel$ nvm

Node Version Manager

Note: <version> refers to any version-like string nvm understands. This includes:
  - full or partial version numbers, starting with an optional "v" (0.10, v0.1.2, v1)
  - default (built-in) aliases: node, stable, unstable, iojs, system
  - custom aliases you define with `nvm alias foo`

Usage:
  nvm --help                                Show this message
  nvm --version                             Print out the latest released version of nvm
  nvm install [-s] <version>                Download and install a <version>, [-s] from source. Uses .nvmrc if available
    --reinstall-packages-from=<version>     When installing, reinstall packages installed in <node|iojs|node version number>
    --lts                                   When installing, only select from LTS (long-term support) versions
    --lts=<LTS name>                        When installing, only select from versions for a specific LTS line
  nvm uninstall <version>                   Uninstall a version
  nvm uninstall --lts                       Uninstall using automatic LTS (long-term support) alias `lts/*`, if available.
  nvm uninstall --lts=<LTS name>            Uninstall using automatic alias for provided LTS line, if available.
  nvm use [--silent] <version>              Modify PATH to use <version>. Uses .nvmrc if available
    --lts                                   Uses automatic LTS (long-term support) alias `lts/*`, if available.
    --lts=<LTS name>                        Uses automatic alias for provided LTS line, if available.
  nvm exec [--silent] <version> [<command>] Run <command> on <version>. Uses .nvmrc if available
    --lts                                   Uses automatic LTS (long-term support) alias `lts/*`, if available.
    --lts=<LTS name>                        Uses automatic alias for provided LTS line, if available.
  nvm run [--silent] <version> [<args>]     Run `node` on <version> with <args> as arguments. Uses .nvmrc if available
    --lts                                   Uses automatic LTS (long-term support) alias `lts/*`, if available.
    --lts=<LTS name>                        Uses automatic alias for provided LTS line, if available.
  nvm current                               Display currently activated version
  nvm ls                                    List installed versions
  nvm ls <version>                          List versions matching a given <version>
  nvm ls-remote                             List remote versions available for install
    --lts                                   When listing, only show LTS (long-term support) versions
  nvm ls-remote <version>                   List remote versions available for install, matching a given <version>
    --lts                                   When listing, only show LTS (long-term support) versions
    --lts=<LTS name>                        When listing, only show versions for a specific LTS line
  nvm version <version>                     Resolve the given description to a single local version
  nvm version-remote <version>              Resolve the given description to a single remote version
    --lts                                   When listing, only select from LTS (long-term support) versions
    --lts=<LTS name>                        When listing, only select from versions for a specific LTS line
  nvm deactivate                            Undo effects of `nvm` on current shell
  nvm alias [<pattern>]                     Show all aliases beginning with <pattern>
  nvm alias <name> <version>                Set an alias named <name> pointing to <version>
  nvm unalias <name>                        Deletes the alias named <name>
  nvm reinstall-packages <version>          Reinstall global `npm` packages contained in <version> to current version
  nvm unload                                Unload `nvm` from shell
  nvm which [<version>]                     Display path to installed node version. Uses .nvmrc if available

Example:
  nvm install v0.10.32                  Install a specific version number
  nvm use 0.10                          Use the latest available 0.10.x release
  nvm run 0.10.32 app.js                Run app.js using node v0.10.32
  nvm exec 0.10.32 node app.js          Run `node app.js` with the PATH pointing to node v0.10.32
  nvm alias default 0.10.32             Set default node version on a shell

Note:
  to remove, delete, or uninstall nvm - just remove the `$NVM_DIR` folder (usually `~/.nvm`)

laradock@915f54093e00:/var/www/laravel$ node -v
v6.4.0

Works!

@nojaf Yes, you need switch to laradock user. Please close this issue.
cc @Mahmoudz

I recently just encountered this problem
Junereys-MacBook-Pro:laradock junereycasuga$ docker-compose up -d nginx mysql phpmyadmin
Creating network "laradock_default" with the default driver
Creating laradock_volumes_data_1
Creating laradock_volumes_source_1
Creating laradock_workspace_1
Creating laradock_mysql_1
Creating laradock_phpmyadmin_1
Creating laradock_php-fpm_1
Creating laradock_nginx_1
Junereys-MacBook-Pro:laradock junereycasuga$ docker-compose ps
          Name                        Command             State                     Ports
-----------------------------------------------------------------------------------------------------------
laradock_mysql_1            docker-entrypoint.sh mysqld   Up       <IP_ADDRESS>:3306->3306/tcp
laradock_nginx_1            nginx                         Up       <IP_ADDRESS>:443->443/tcp, <IP_ADDRESS>:80->80/tcp
laradock_php-fpm_1          php-fpm                       Up       9000/tcp
laradock_phpmyadmin_1       /run.sh phpmyadmin            Up       <IP_ADDRESS>:8080->80/tcp
laradock_volumes_data_1     /true                         Exit 0
laradock_volumes_source_1   /true                         Exit 0
laradock_workspace_1        /sbin/my_init                 Up       <IP_ADDRESS>:22->22/tcp
Junereys-MacBook-Pro:laradock junereycasuga$ docker exec -it laradock_workspace_1 bash -l
mesg: ttyname failed: Success
root@5294369e43f2:/var/www/laravel# su laradock
laradock@5294369e43f2:/var/www/laravel$ nvm
bash: nvm: command not found

Im still getting node/nvm/npm command was not found. I switch to laradock but that did not help at all.
restarted the boot2docker vm and still did not work. even rebuilding the containers did nothing.

Hi @CreativeKoen - I think the setup process Laradock has changed quite a lot since this issue. Have you set WORKSPACE_INSTALL_NODE and WORKSPACE_INSTALL_YARN in your docker/.env file?

Yes I did. I figured out that I had to build my workspace container. but now It hanging on this.

On building for me it hangs for a little while at that point, but then prints out the whole npm tree with what seems to be the wrong encoding.

For the record I'm on Windows as well - All out of ideas I'm afraid!

Im gonna play around, thanks for the help!

@CreativeKoen I had a similar issue and restarting the containers fixed it (docker-compose down, and then up).
I am also on Windows.


METRON-1654: findOne request after an alert patch returns with the original state of the alert item

Contributor Comments
With the addition of https://github.com/apache/metron/pull/1190, the Alerts UI can now update state using the object returned from a REST update call.  This PR updates the Alerts UI to do that.
Testing
This has been tested in full dev.

Spin up full dev and navigate to the Alerts UI
Open an alert and change the status from "NEW" to "OPEN".  After closing the detail panel the status for that alert should show as "OPEN" in the list view.
Create a metaalert and search for the metaalert so that is shows up in the list view.  Add an alert to that metaalert by checking the checkbox next to an alert and using the "Add to Alert" option in the "Actions" dropdown.  The count next the the metaalert in the list view should increment by 1.
Remove an alert from the metaalert.  The count next to the metaalert should decrement by 1 and the alert should be removed immediately.
Other update functions in the UI should be tested to ensure there are no regressions:  adding/deleting a comment, changing a metaalert name, etc.

Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && dev-utilities/build-utils/verify_licenses.sh

[ ] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

@merrimanr When you get a chance, we're conflicted.

@merrimanr - can you deconflict this so we can re-test?

Done

+1 I am able to verify that when I add the alert in an "open" state to the meta alert the count increments by 1. then i remove it the count decreases by 1


oracle wallet magic number mismatch

I created a wallet with this version of orapki:
Oracle PKI Tool Release <IP_ADDRESS>.0 - Production
Version <IP_ADDRESS>.0
Copyright (c) 2004, 2021, Oracle and/or its affiliates. All rights reserved.

...and, for testing,  with:
Oracle PKI Tool Release <IP_ADDRESS>.0 - Production
Version <IP_ADDRESS>.0
Copyright (c) 2004, 2021, Oracle and/or its affiliates. All rights reserved.

In both cases when I do a hex dump of the start of the file I see:
$ od -t u1 ~/dataconnect2.wallet/cwallet.sso
0000000   161 248  78  55   0   0   0   6   0   0   0  33   6 121  74 196

When loading this wallet I saw the failure invalid wallet. I ran through the code in the debugger and found this code in wallet.go:
func (w *wallet) read() error {
	fileData, err := ioutil.ReadFile(w.file)
	if err != nil {
		return err
	}
	index := 0
	if !bytes.Equal(fileData[index:index+4], []byte{161, 248, 78, 54}) {
		return errors.New("invalid wallet")
	}
	index += 4
	if fileData[3] != 54 && fileData[4] != 55 {
		return errors.New("invalid magic version")
	}

The wallet file I have created by both versions I currently have access to create a wallet file that will not pass these tests.
I guess I have two questions:

Have I somehow created the wallet incorrectly?
If I have not created the wallet incorrectly, is there an issue that can be resolved?

I am connecting to Oracle 19c, and have already successfully used go-ora with unencrypted connections. This problem occurs when I create a wallet for using a TCPS connection, which I must do for security.

is this v1?
would you please try it with latest version v2

I believe this is v2. My go.mod file:
module ora_sample

go 1.18

require github.com/sijms/go-ora/v2 v2.3.2

require github.com/lib/pq v1.10.5

require github.com/google/uuid v1.3.0

require (
        github.com/jaswdr/faker v1.10.2
        gorm.io/driver/postgres v1.3.7
        gorm.io/gorm v1.23.5
)

require (
        github.com/go-logfmt/logfmt v0.5.1 // indirect
        github.com/go-logr/logr v1.2.3 // indirect
        github.com/godror/godror v0.37.0 // indirect
        github.com/godror/knownpb v0.1.0 // indirect
        github.com/jackc/chunkreader/v2 v2.0.1 // indirect
        github.com/jackc/pgconn v1.12.1 // indirect
        github.com/jackc/pgio v1.0.0 // indirect
        github.com/jackc/pgpassfile v1.0.0 // indirect
        github.com/jackc/pgproto3/v2 v2.3.0 // indirect
        github.com/jackc/pgservicefile v0.0.0-20200714003250-2b9c44734f2b // indirect
        github.com/jackc/pgtype v1.11.0 // indirect
        github.com/jackc/pgx/v4 v4.16.1 // indirect
        github.com/jinzhu/inflection v1.0.0 // indirect
        github.com/jinzhu/now v1.1.5 // indirect
        golang.org/x/crypto v0.0.0-20220525230936-793ad666bf5e // indirect
        golang.org/x/text v0.3.7 // indirect
        google.golang.org/protobuf v1.28.1 // indirect
)

v2.3.2 is very old please update

@einarnn ,
For the go-ora, do you have any problem to 23c wallet ?


Edge reversed animation

Please describe the feature that you want to propose
It would be awesome if there was a way of reversing the animated edge, without needing to change the source and target.

You can already do this with just CSS, this doesn't need to be an additional feature ^^
.react-flow__edge.animated path {
	animation-direction: reverse;
}


Feature: add option to read flags from file

This adds the following:

ability to chain With operators on flags.
read contents of complex flags (proto, JSON) from flags

@devnev

Codecov Report

Merging #23 into master will increase coverage by 0.57%.
The diff coverage is 85.1%.

@@            Coverage Diff            @@
##           master     #23      +/-   ##
=========================================
+ Coverage   78.12%   78.7%   +0.57%
=========================================
  Files          14      15       +1
  Lines         576     648      +72
=========================================
+ Hits          450     510      +60
- Misses         86      95       +9
- Partials       40      43       +3

Impacted Files
Coverage Δ

common.go
100% <ø> (ø)
:arrow_up:

dynstringset.go
91.83% <100%> (+0.34%)
:arrow_up:

dynduration.go
85.71% <100%> (+1.09%)
:arrow_up:

dynint64.go
87.87% <100%> (+0.78%)
:arrow_up:

dynfloat64.go
88.23% <100%> (+0.73%)
:arrow_up:

dynstringslice.go
94.11% <100%> (+0.36%)
:arrow_up:

dynstring.go
93.54% <100%> (+0.44%)
:arrow_up:

protobuf/dynproto3.go
64.51% <71.42%> (-0.71%)
:arrow_down:

fileread_flag.go
82.14% <82.14%> (ø)

dynjson.go
78.57% <85.71%> (+6.07%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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

Why not use the more conventional golang style of option arguments to a construcor, i.e. flags.StringFlag(...required stuff here..., WithNotifier(...), WithFileFlag(...))?


Enhancement: allow external beams (plus some minor things)

This pull request contains a few minor changes and one significant change. All the changes are backwards compatible. I’ve run all the tests in the tests/ directory and they all pass. I’ve added another test script.
The minor changes are:

Allow a random seed to be passed to construct_skymodel(), which then calls np.random.seed(seed). This means that a “flat_spec” sky model is repeatable.
Allow a telescope height to be specified, along with latitude/longitude.
Ignore sources below the horizon when calculating a beam value. This probably should be handled by “horizon_taper” but it is turned off. I added a simple statement that zeros pixels below the horizon, but doesn’t take pixel area into account. I needed to do this because I was using a beam that goes below the horizon.

The significant change is to allow the use of external beams derived from and including pyuvsim AnalyticBeam. We (Phil Bull’s group) want to use the Fagnoni-type beams that have been developed for hera_sim: https://github.com/HERA-Team/hera_sim/blob/polybeam/hera_sim/beams.py. The beams do not need to be the same for all antennas, and are passed as a list to set_beam(). UVBeam objects could also be used, but UVBeam has a lot of other parameters which I have not implemented, and also are not implemented by pyuvsim AnalyticBeam. All these beams probably need a consistent interface.
There is a slight issue with this enhancement because the beams defined within healvis (healvis AnalyticBeam) are baseline beams, but the external beams are antenna beams. Antenna beams for a baseline must be multiplied when generating visibilities. Code has been added to keep track of per-antenna beams and then multiply beams when visibilities are calculated in _vis_calc().

My test script is failing because it imports a couple of packages that are not available. I can work around that and will push an update.


[vcpkg baseline][pocketpy] Don't install include/pybind11/*

Detected by https://dev.azure.com/vcpkg/public/_build/results?buildId=104413&view=results
error: The following files are already installed in /mnt/vcpkg-ci/installed/x64-linux and are in conflict with pocketpy:x64-linux
Installed by pybind11:x64-linux
include/pybind11/embed.h
    include/pybind11/functional.h
    include/pybind11/operators.h
    include/pybind11/pybind11.h
    include/pybind11/stl.h

Broken by https://github.com/microsoft/vcpkg/pull/39510, in which pybind11 support was added.

[x] Changes comply with the maintainer guide.
[ ] SHA512s are updated for each updated download.
[ ] The "supports" clause reflects platforms that may be fixed by this new version.
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Tested usage successfully by pocketpy:x64-windows:
pocketpy provides CMake targets:

  # this is heuristically generated, and may not be correct
  find_package(pocketpy CONFIG REQUIRED)
  target_link_libraries(main PRIVATE pocketpy)

This of course raises the question if the port is vendoring pybind in some way.

This of course raises the question if the port is vendoring pybind in some way.

Is it appropriate for me to add pybind11 as a feature?

IIUC this is vendored just for convenience, so no feature or dependency required.

Thanks for fixing CI!

I asked the author of pocketpy's pybind11, pocketpy's pybind11 is not same as pybind11, but only the interface is same. So remove it may not be right?
https://github.com/pocketpy/pocketpy/issues/216

Hm, that might be bad for pocketpy. At least in vcpgk.

what about install it to pocketpy/pybind11

what about install it to pocketpy/pybind11

This would resolve the immediate file installation conflict, but not the conflict of the symbols behind the file names.

The reason they chose same interface may be that it's easy to change from pybind11 + cpython to pocketpy's pybind11 + pocketpy with the same code.
The conflict will occur if using this two pybind11 at the same time, but seems if someone choose pocketpy, he may be aware of the difference, scenarios where both versions of pybind11 (from pocketpy and the regular one) are used simultaneously may be rare. What about

install to pocketpy/pybind11
describe it in the usage


fleetctl: print error when units were not found in registry

Destroy logic was already improved so this issue could be closed: https://github.com/coreos/fleet/pull/1256
https://github.com/coreos/fleet/issues/710
But we still have to print some kind of error message when units were not found.

Hi @kayrus thank you for the patch! but since we agreed on ignoring non-existent units and don't fail we added unit tests for that case, and the patch just triggers those failures., and also findUnits() filters units based on what you have provided, if you did provided non-existent units, it will return an empty list and not an error. The callers assume an empty list if units do not exist, errors are only in case the Units() fail.

@tixxdz updated

ok thank you could you also please fix gofmt and push again ;-)
While you are it why is this related to #1256 #710 ?? am I missing something here ? when you call with wildcards! I guess it's has nothing to do with fleetctl command or Go, it's you shell which expands automatically and by accident to local valid unit services..., you should know that you are doing or escape ?!  are these really bugs related to fleet ? hmm ?

Btw, should we filter units when we would like to print only one unit? I.e.
$ systemctl list-unit-files etcd2.service
UNIT FILE     STATE
etcd2.service disabled

1 unit files listed.
$ systemctl list-units etcd2.service
UNIT          LOAD   ACTIVE SUB     DESCRIPTION
etcd2.service loaded active running etcd2

LOAD   = Reflects whether the unit definition was properly loaded.
ACTIVE = The high-level unit activation state, i.e. generalization of SUB.
SUB    = The low-level unit activation state, values depend on unit type.

1 loaded units listed. Pass --all to see loaded but inactive units, too.
To show all installed unit files use 'systemctl list-unit-files'.

$ fleetctl list-unit-files hello.service
UNIT                    HASH    DSTATE          STATE           TARGET
hello.service           3750b3e launched        launched        706e4d01.../coreos1
<EMAIL_ADDRESS>       79699e3 launched        launched        706e4d01.../coreos1
<EMAIL_ADDRESS>       79699e3 launched        launched        706e4d01.../coreos1
<EMAIL_ADDRESS>       79699e3 launched        launched        706e4d01.../coreos1
$ fleetctl list-units hello.service
UNIT                    MACHINE                 ACTIVE  SUB
hello.service           706e4d01.../coreos1     active  running
<EMAIL_ADDRESS>       706e4d01.../coreos1     active  running
<EMAIL_ADDRESS>       706e4d01.../coreos1     active  running
<EMAIL_ADDRESS>       706e4d01.../coreos1     active  running

@kayrus hmm IMO yes it make sense, not sure what others think
I guess the only thing to worry about is the exit codes, since list-unit-files always succeeded, this should not change. If unit tests are not there then this should also be added.

I'm not convinced of this change: it seems a bit arbitrary to me that it warns if no units were matched, but not if some of the supplied units were matched while others were not.
At the very least, the warning should be clearer, e.g. "Warning: No units matched."
I'm not sure I understand the remark about listing one unit. (Isn't that a separate issue?...)

LGTM.
I think there's no reason not to merge this. I'll merge it tomorrow.


docs: update github links

Updated the github links for the libraries that move to software mansion.
There were some broken links that pointed to the react-native-screens/native-stack that are now fixed as well.
I didn't include the links in the blog posts, I can add them too if you want.

thanks for the PR!


Security Threat found

Hi team,  while exploring the codebase found the hardcoded keys, cert etc.. it's a potential risk for the project like opensource. I will paste all the code block here for your references.
https://github.com/FrigadeHQ/remote-storage/blob/bfb4fa144f7198cfba1acb9d0723eafc5a76345a/apps/www/pages/_app.tsx#L14
https://github.com/FrigadeHQ/remote-storage/blob/bfb4fa144f7198cfba1acb9d0723eafc5a76345a/apps/remote-storage-server/certs/server.crt#L1
https://github.com/FrigadeHQ/remote-storage/blob/bfb4fa144f7198cfba1acb9d0723eafc5a76345a/apps/remote-storage-server/certs/server.csr#L1
https://github.com/FrigadeHQ/remote-storage/blob/bfb4fa144f7198cfba1acb9d0723eafc5a76345a/apps/remote-storage-server/certs/server.key#L1
If possible so port this hardcoding into env or somewhere else just like you did for redis configuration.

Hey there! These keys are all public information. The posthog key used for the demo website is publicly available as well on https://remote.storage


Error: remove_email_envelope() function

Describe the bug
The method mailparser.parse_from_bytes(bt) cannot create a Mailparser object, because the remove_email_envelope(message) function called inside fails. The function tries to use a string pattern on a bytes-like object.
To Reproduce
Steps to reproduce the behavior:

import mailparser
mail_obj = open('TEST_Mail.eml', 'rb')
test_mail = mail_obj.read()
mail = mailparser.parse_from_bytes(test_mail)
See error

Expected behavior
Getting a Mailparser object.
Raw mail
Delivered-To<EMAIL_ADDRESS>Received: by <IP_ADDRESS> with SMTP id a13cs354079rvf;
Fri, 21 Nov 2008 20:05:05 -0800 (PST)
Received: by <IP_ADDRESS> with SMTP id w15mr2254748ybj.98.1227326704711;
Fri, 21 Nov 2008 20:05:04 -0800 (PST)
Return-Path<EMAIL_ADDRESS>Received: from mail11.tpgi.com.au (mail11.tpgi.com.au [<IP_ADDRESS>])
by mx.google.com with ESMTP id 10si5117885gxk.81.2<IP_ADDRESS>.05.03;
Fri, 21 Nov 2008 20:05:04 -0800 (PST)
Received-SPF: neutral (google.com: <IP_ADDRESS> is neither permitted nor denied by domain of<EMAIL_ADDRESS>client-ip=<IP_ADDRESS>;
Authentication-Results: mx.google.com; spf=neutral (google.com: <IP_ADDRESS> is neither permitted nor denied by domain of<EMAIL_ADDRESS><EMAIL_ADDRESS>X-TPG-Junk-Status: Message not scanned
X-TPG-Antivirus: Passed
Received: from [<IP_ADDRESS>] (60-241-138-146.static.tpgi.com.au [<IP_ADDRESS>])
by mail11.tpgi.com.au (envelope-from<EMAIL_ADDRESS>(8.14.3/8.14.3) with ESMTP id mAM44xew022221
for<EMAIL_ADDRESS>Sat, 22 Nov 2008 15:05:01 +1100
Message-Id<EMAIL_ADDRESS>From: Mikel Lindsaar<EMAIL_ADDRESS>To: Mikel Lindsaar<EMAIL_ADDRESS>Content-Type: text/plain; charset=US-ASCII; format=flowed
Content-Transfer-Encoding: 7bit
Mime-Version: 1.0 (Apple Message framework v929.2)
Subject: Testing 123
Date: Sat, 22 Nov 2008 15:04:59 +1100
X-Mailer: Apple Mail (2.929.2)
Plain email.
Hope it works well!
Mikel
Environment:

OS: Linux
Docker: yes
mail-parser version 3.7.0

Additional context

Error:
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.6/dist-packages/mailparser/mailparser.py", line 116, in parse_from_bytes
return MailParser.from_bytes(bt)
File "/usr/local/lib/python3.6/dist-packages/mailparser/mailparser.py", line 243, in from_bytes
envelope_present, bt = remove_email_envelope(bt)
File "/usr/local/lib/python3.6/dist-packages/mailparser/utils.py", line 526, in remove_email_envelope
True if EMAIL_ENVELOPE_PATTERN.search(message) else False
TypeError: cannot use a string pattern on a bytes-like object

This breaks a lot of things for us as well.. I can't even do
ipdb> p mailparser.parse_from_bytes(orig.as_bytes())
*** TypeError: cannot use a string pattern on a bytes-like object
ipdb> p type(orig)
<class 'email.message.Message'>

The problem does not happen in 3.6.1

Fixed.

ah, this was my fault, sorry. @fedelemantuano I can create a fix to handle the python 3 bytes case.

@dfeinzeig for now we don't need it. If someone will have a problem, we can do that.

i need it :)
the problem is that the python 2 email library silently fails to parse
emails with an envelope. i included a link to the spot in the python stdlib
where this happens.
https://github.com/python/cpython/blob/2.7/Lib/email/feedparser.py#L226
On Wed, Nov 28, 2018 at 12:00 PM Fedele Mantuano<EMAIL_ADDRESS>wrote:

@dfeinzeig https://github.com/dfeinzeig for now we don't need it. If
someone will have a problem, we can do that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/SpamScope/mail-parser/issues/46#issuecomment-442523386,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABaBgqcsiYl6y9hKfL7OxJAYlH8EHqVqks5uzsE_gaJpZM4Yd4_A
.

--
David Feinzeig
<EMAIL_ADDRESS>508.353.4735
Site: http://david.feinzeig.com
Blog: http://david.feinzeig.com/blog

Can you send me a complete email with this problem? I want to see what's happen with mail-parser.

@fedelemantuano , i'm picking this back up. i need to add envelope checking and removal back for my purposes. let me know if you have any particular concerns about how you'd like this implemented. i'm thinking i'll likely add in the proper python version checking that i was missing before.

@fedelemantuano , i'm thinking something like this
https://github.com/dfeinzeig/mail-parser/commit/503f5027ed0eed9d8a84b188bed3e031a32d9618
And this time I've tested it with both python 2 and 3 :), including the exact bytes example the original poster mentioned.

Did you run any test?


Logout from LinkedIn

I am successfully able to login on linkedIn. How can I logout from it?

It doesn't seem to be implemented yet, even though it's a necessary function.

This lib is much more a way to get access token than call linkedin api. So yes logout in an app is necessary but you can do it by yourself just by calling linkedin API.

Was anyone able to figure out a way to logout of linkedIn?

tried to clear cookies by react-native-linkedin package but still it is managing session


Update installing-signed-packages.md

Fix broken link

Docs Build status updates of commit ea059d0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/consume-packages/installing-signed-packages.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Open Table bugs

🐛 Bug Report

[ ] Nested Columns do not read correctly with NVDA - haven’t tested with other screen readers. Not sure what bit is not announcing correctly. It reads out the column/row numbers
[ ] Focusing a cell then holding down arrow causes it to lag and jump while scrolling unlike FF and Safari Reproduced
[ ] When I tab out of Table many rows/columns, it does a page down. When I shift tab back into the table, it does another page down Reproduced
[ ] hiding columns story: if you rapidly change what columns are hidden/displayed, you get a error (might be related to transitions)? Reproduced
[ ] When I focus the checkbox cell and hold the down arrow to scroll down, it becomes very choppy in the scrolling, maybe disable checkbox focus animation? Reproduced
[ ] Table stories from "many columns" onwards don't fit on a mobile screen, prob should refine for testing (DL)
[ ] From the many column example. When the width of the table is greater than the frame/browser, you can't access the first column on two. In RTL on iphone, there is weird render artifacts or it not rendering correctly on the right. See screen shots. (KT)
[ ] using Option Tab in /story/table--hideheader will take you through buttons, if you go through a few of them, and then hit the 'up arrow', it'll put focus back above where the last non-Option-tab stop was (RS, Safari MacOS)
[ ] Table is sometimes blurry on initialization, goes away on scroll, see screenshot (DL, Firefox Windows)
[ ] When the select all checkbox switches from indeterminate to checked it appears un-emphasized briefly (DG)
[x] If i’ve scrolled part way down, then leave the table to do something else, come back, grab the scrollbar and try to drag it down, it jumps to the top (RS) Unable to reproduce, though I find that I am unable to grab the scrollbar without using the mouse to scroll since clicking interacts with the cell underneath the scrollbar
[x] Cannot shift+tab back out of the table once you enter it Unable to reproduce
[x] Async Loading button for ‘remove first item’ should probably aggregate removals in time with the table transitions
if you have a mouse plugged in so scroll bars are always visible, there are flashes of scroll bars appearing and disappearing whenever a table is loaded Can't reproduce
[x] Rapidly using arrow keys (down) in table, also holding down throws an error in createTreeWalker Unable to reproduce
[x] CRUD example: If you delete a row in the middle and immediately add another row, the row appears empty” Unable to reproduce

🧢 Your Company/Team
RSP

[x] Cannot shift+tab back out of the table once you enter it Unable to reproduce
[ ] Nested Columns do not read correctly with NVDA - haven’t tested with other screen readers. Not sure what bit is not announcing correctly. It reads out the column/row numbers
[x] Rapidly using arrow keys (down) in table, also holding down throws an error in createTreeWalker Unable to reproduce
[x] CRUD example: If you delete a row in the middle and immediately add another row, the row appears empty” Unable to reproduce
[x] If i’ve scrolled part way down, then leave the table to do something else, come back, grab the scrollbar and try to drag it down, it jumps to the top (RS) Unable to reproduce, though I find that I am unable to grab the scrollbar without using the mouse to scroll since clicking interacts with the cell underneath the scrollbar
[ ] Focusing a cell then holding down arrow causes it to lag and jump while scrolling unlike FF and Safari Reproduced
[ ] When I tab out of Table many rows/columns, it does a page down. When I shift tab back into the table, it does another page down Reproduced
[ ] hiding columns story: if you rapidly change what columns are hidden/displayed, you get a error (might be related to transitions)? Reproduced
[ ] Async Loading button for ‘remove first item’ should probably aggregate removals in time with the table transitions
if you have a mouse plugged in so scroll bars are always visible, there are flashes of scroll bars appearing and disappearing whenever a table is loaded Can't reproduce
[ ] When I focus the checkbox cell and hold the down arrow to scroll down, it becomes very choppy in the scrolling, maybe disable checkbox focus animation? Reproduced

Addition possible issues to discuss from https://github.com/adobe/react-spectrum/pull/1077#pullrequestreview-506844829:
Page down only takes the user to the last page, once on that last page you can't get to to the last item in the table by hitting page down again.
It seems weird that that in row selection or cell selection, the enter/return key can toggle select, but when you select the check box you can't toggle it with return.
It is interesting that the home end keys take the user to the start or end of a row when in cell selection and to the start or end of the table when in row selection. Page up and page down always move by a page of rows.
Should there be a story with long content in a cell and no wrap?
When there are nested columns, using the keyboard, I goto the second or last sub column, goto the parent, then back down to the sub column and I'm not at the first child, I'd expect to return to the child I entered from.

Closing this ticket in favor of the sub issues I made


[Geneva Exporter] Add support for exporting metrics to different combinations of account and namespace

Add support for exporting metrics to different combinations of account and namespace

reading and configure account and namespace from the attributes if present.
ensuring that the account and namespace is not exported as attributes. They should be used only for configuration.

Removed an optimization that was made earlier by having separate buffers for histogram and non-histogram metrics with pre-serialized account and namespace. All the metrics share the same buffer now since we have to serialize account and namespace on every export call now.

Does the separate buffer cause any issue?

No as we only do one export at a time, to optimise memory.


added a crucial comma to separate required packages

in file setup.py

Good catch, thanks so much! (I tend to use pyproject.toml so I did not see the issues with setup.py)


Update filename to include timestamp

https://trello.com/c/dq3Y7Cgn/575-add-timestamp-to-data-extract-filename
Change description
The download filename (either JSON or XLSX) is changed from download.extension to the format download-yyyy-mm-dd-hhmmss.extension
The format depcited in the above ticket includes spaces and colons , however I thought it would be best to leave these out of the filename and instead used hyphens. This can be updated to use spaces if neccessary (but colons and slashes are not permitted).

[X] Unit tests and other appropriate tests added or updated
[X] README and other documentation has been updated / added (if needed)
[X] Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test
Changes can be seen in the output file after running download

pre-commit.ci autofix


[Docs]: Update Maintenance window page for docker

Problem Statement
Update Maintenance window page for docker
Other Relevant URLs
Draft: https://www.notion.so/Update-Maintenance-window-page-for-docker-129fe271b0e28099ac71c2b756111f80
Slack/conversation:

Fixed as part of #1948


Add tooltip top position

Hi!
I have a problem whith the tooltip if the element is close to the bottom, similar to button of [Inbox] (https://inbox.google.com/), (next picture)
I added some lines code to resolve this issue. You need to add the class "mdl-tooltip--top" to the element

Check it!
Thanks

@EdgarVaguencia Thanks for the PR. Code-wise it looks good to me and I agree that we need a solution to this. Let me loop-in @santokii to see if this is acceptable as the MD spec itself has all their tooltips on the bottom.

As long as the padding / specs are the same having the tooltip appear above should be fine when it's not visible below. (Same would apply to the left and right). Thanks for pointing this out!

FYI there should be a PR coming up for automatically moving the tooltip to make sure it's visible. For now it only moves if it overflows on the left. But supposedly a PR is coming for when it overflows on the right and on the bottom. See #858 someone volunteered but the PR is not in yet...
This PR could still be fine though because some users might prefer to have the tooltip above at all time rather than below.

I think this raises the notion we should have (4) modifiers for tooltips to set which position they go into. Leaving no modifier for the component to decide on its own how to best display the tooltip.
Sound reasonable?

I think this raises the notion we should have (4) modifiers for tooltips to set which position they go into. Leaving no modifier for the component to decide on its own how to best display the tooltip.

This sounds reasonable to me.

We could basically have the classes as the menu component with automatic detection if none is present.

Closing this in favor of #1101 which provides all four of the positions required.


tf.Module doesn't see variables from keras layers

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.0.0-alpha0
Python version: 3.6.7
Bazel version (if compiling from source): N/A
GCC/Compiler version (if compiling from source): N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A

Describe the current behavior
User-defined subclasses of tf.Module do not recognize and include variables from keras layer object members.
Describe the expected behavior
Subclassing tf.Module should expose the variables (not to mention training_variables) from all sub-computations, including keras layers, so long as these computations are set as members of the class.
Code to reproduce the issue
This came from a TF2.0 port of the pytorch dynamic graph example here.
import tensorflow as tf
import numpy as np
import random

class DynamicNet(tf.Module):
    def __init__(self, D_in, H, D_out):
        self.input_linear = tf.keras.layers.Dense(H, input_dim=D_in)
        self.middle_linear = tf.keras.layers.Dense(H, input_dim=H)
        self.output_linear = tf.keras.layers.Dense(D_out, input_dim=H)

    def __call__(self, x):
        h_relu = tf.maximum(self.input_linear(x), 0)
        for _ in range(random.randint(0, 3)):
            h_relu = tf.maximum(self.middle_linear(h_relu), 0)
        y_pred = self.output_linear(h_relu)
        return y_pred

N, D_in, H, D_out = 64, 1000, 100, 10

x = np.random.randn(N, D_in)

model = DynamicNet(D_in, H, D_out)
model(x)
assert len(model.variables) > 0

(This particular example can be fixed by manually defining a @property variables which combines the variables members from each of the Dense layers into one list, but I think this partially defeats the point of using tf.Module).
Other info / logs
I'm not completely sure this is a bug/oversight, or if tf.Module was never intended to be compatible with keras layers. If that's the case, perhaps this is more appropriately categorized as a feature request. It seems like a natural thing to be able to do based on the the tf.Module RFC.

After tf.Module becomes the base class for tf.keras.layers.Layer this issue will be fixed, but I don't know when

Hi all, the tf.keras team are working on making Module the base class for Layer. I think @qlzh727 is working on it this quarter.
As a temporary hack you can make a reference to the layer variables inside the module:
class DenseModule(tf.Module):
  def __init__(self):
    super(DenseModule, self).__init__()
    self._layer = tf.keras.layers.Dense(64)

  def __call__(self, x):
    y = self._layer(x)
    # Keep a reference to layer variables so Module can find them.
    self._k_variables = self._layer.variables
    return y

@tomhennigan is there a timeline for it? will it get done in official 20. release?

Hi @tomhennigan, is there a timeline for it? will it get done in 2.0 release?

I hope so, I worked with the Keras team last month to remove known blockers (e.g. e866995aff7) and I think it should be possible to do before 2.0 if we don't find more blockers.

Yes, the plan is to have keras.layer use module as base class, and we plan to have this feature in 2.0 release. I am actively working on it right now.

https://github.com/tensorflow/tensorflow/commit/23c8fd4ca3452865ac9ef1359f74cd0039908b59 should fix this issue.

I've tested this at head and @qlzh727's fix does indeed make your example pass.


Adding a child page via the front end

Interested to get thoughts on this one.
Currently the Wagtail UI visible on the front end of the site to editors allows you to add a sibling page to the page you are currently viewing (along with editing the current page and accessing the Wagtail dashboard).
This means if you're at the end of the tree branch you can't add a child page via this interface. I know some of our clients prefer to navigate through the front end of the site rather than through the explorer ui but this presents a problem for their workflow.
Option 1. Change the 'add' button so that it adds a child page. This means you can still add siblings but you would have to add them as child pages of the parent. This could be disruptive to editors who are already familiar to with the existing workflow. We could make the label more explicit by changing it to 'add child page' but my experience is that editors don't always understand the parent/sibling/child terminology (though it is used elsewhere in the ui already). I know you can hover on the button for help text but generally people won't wait long enough to realise this.
Option 2. Add an additional stage to the workflow that allows you to specify adding either a sibling or child.

Hi Tom,
This is what #728 tackled. Can you test the latest master, please?

Apologies. Thought I was on the latest release. I'll take a look.

@tmsndrs did you get a chance to test the latest release?

Addressed by #728.


Allow for exponential backoff between retries for artifact uploads

We occasionally get rate-limited from GCS (where we upload artifacts to) due to high load on the bucket. The suggested retry policy in such cases is to use exponential backoff between retries. https://cloud.google.com/storage/docs/exponential-backoff. We add a cli flag to optionally specify backoff strategy.
Since the agent doesn't do this currently, we're having to workaround by using gsutil which already does this. However, we'd like to do this through the agent so that we can see the artifacts in UI and also allow sharing of artifacts between jobs (which we've lost with using gsutil).
Tested locally as follows with dummy values:
C02Y231UJHD2:agent sridhar.mocherla$ buildkite-agent artifact upload --job testjob --backoff-strategy exponential --endpoint localhost test.txt
2019-08-19 23:07:47 INFO   Found 1 files that match "test.txt"
2019-08-19 23:07:47 INFO   Creating (0-1)/1 artifacts
2019-08-19 23:07:47 WARN   Post localhost/jobs/testjob/artifacts: unsupported protocol scheme "" (Attempt 1/10 Retrying in 2s)
2019-08-19 23:07:49 WARN   Post localhost/jobs/testjob/artifacts: unsupported protocol scheme "" (Attempt 2/10 Retrying in 4s)
2019-08-19 23:07:53 WARN   Post localhost/jobs/testjob/artifacts: unsupported protocol scheme "" (Attempt 3/10 Retrying in 8s)
2019-08-19 23:08:01 WARN   Post localhost/jobs/testjob/artifacts: unsupported protocol scheme "" (Attempt 4/10 Retrying in 16s)

Thanks for this! We'll review and discuss.

We'd prefer to not have a different config for this, how about we just look for specific exit codes and use a backoff strategy in those cases?

@lox Sure, I'll update the PR accordingly.

Closing this as it's largely (though not entirely) superseded by https://github.com/buildkite/agent/pull/1588.


Disable generating SourceMaps?

The sourceMappingUrl comment that is added to the bottom of our styles (that appear in the HTML head) doesn't actually resolve to a proper source map.
I'm using babel-plugin-react-css-modules instead of react-css-modules, which means I simply import my styles without declaring a reference in my React components, and it maps the styleName prop to the imported SCSS stylesheet: import 'App.scss'
Can you tell me if there is a way to disable these source maps entirely? It basically means we end up with tons of "Failed to parse SourceMap" errors in the browser console:

As well as being able to disable these (I tried running Meteor in production mode and they are still generated), is there any way to resolve the broken source map references?
Thanks!

@nathantreid — Sorry... any help on the above? Thanks!

This isn't currently a customizable option; I'll look into it when I get a chance, but in the meantime if you want to change it in a local copy of the package, it's probably coming from here.

No worries @nathantreid — thanks for the reply. I'll look into running a customised local package at some point. Cheers.

Hey guys, I am getting this warnings too. How could i change it in local package? Don't u propose me to go to node_modules folder?))


Cloudflare 1020

I'm getting 403 for login with some cloudflare error saying "Please enable cookies". Have you encountered something like this?

i'll take a look later this day

От: Dorinel Panaite<EMAIL_ADDRESS>Отправлено: 24 августа 2020 г. 11:23
Кому: oleh2208/etoro-api<EMAIL_ADDRESS>Копия: Subscribed<EMAIL_ADDRESS>Тема: [oleh2208/etoro-api] Cloudflare 1020 (#10)
I'm getting 403 for login with some cloudflare error saying "Please enable cookies". Have you encountered something like this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/oleh2208/etoro-api/issues/10, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADHNNVT2GYFJYDOGIBXH7NLSCIWRLANCNFSM4QJJHVOA.

any chanse you have 2factor auth?

No, I have 2factor disabled. I cannot seem to reproduce it again, it happened in the first run, then it worked properly.

ok, it's probably some race condition on strart, i need to add some checks to fix it

От: Dorinel Panaite<EMAIL_ADDRESS>Отправлено: 24 августа 2020 г. 23:22
Кому: oleh2208/etoro-api<EMAIL_ADDRESS>Копия: oleh2208<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Тема: Re: [oleh2208/etoro-api] Cloudflare 1020 (#10)
No, I have 2factor disabled. I cannot seem to reproduce it again, it happened in the first run, then it worked properly.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/oleh2208/etoro-api/issues/10#issuecomment-679373342, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADHNNVR5MU7TJDDID5ULIU3SCLKZBANCNFSM4QJJHVOA.

I have the same issue ("Please enable cookies")

hi make sure you are using latest version of this project, this issue was fixed when i started using chrome instead of phantom.js


Execute rake tasks?

Is it possible for sidekiq-cron to execute rake tasks? Or should I convert them to Sidekiq Jobs?

Hi,
from my experience I am putting code into Service file as service pattern.
so in my rails app I have something like:

app

models
services

some_super_cool_task.rb

which be executed from rake task, sidekiq worker or direclty in app (controller action etc).
Best thing from this aproach is that you can test that code.
Rake task is untestable

I know this issue 4 years old, but I had the same question and ended up here. I figured I'd share my solution for others who can't easily convert their rake tasks to background jobs (for example rake tasks provided by other gems).
Create a generic background job or worker that can execute any rake task:
# app/jobs/invoke_rake_task_job.rb

require "rake"
Rails.application.load_tasks

class InvokeRakeTaskJob < ApplicationJob
  def perform(args)
    Rake::Task[args["task"]].invoke(args["args"])
  end
end

In order to run the sitemap:refresh rake task add the following to your config/schedule.yml:
sitemap_refresh:
  cron: 0 0 * * *
  class: InvokeRakeTaskJob
  args:
    task: sitemap:refresh

I know this issue 4 years old, but I had the same question and ended up here. I figured I'd share my solution for others who can't easily convert their rake tasks to background jobs (for example rake tasks provided by other gems).
Create a generic background job or worker that can execute any rake task:
# app/jobs/invoke_rake_task_job.rb

require "rake"
Rails.application.load_tasks

class InvokeRakeTaskJob < ApplicationJob
  def perform(args)
    Rake::Task[args["task"]].invoke(args["args"])
  end
end

In order to run the sitemap:refresh rake task add the following to your config/schedule.yml:
sitemap_refresh:
  cron: 0 0 * * *
  class: InvokeRakeTaskJob
  args:
    task: sitemap:refresh

I want to add a note to this comment for anyone running rake tasks inside workers.
When you use #invoke method on a rake task, it will be executed only once.
If you have Rake::Task[args["task"]].invoke(args["args"]) as a sidekiq-cron job, you will have a task executed for the first time and it will just return nil on the next scheduled execution (unless you restarted sidekiq-cron process in between those scheduled executions)
You have 2 options:

Reenable task before executing it, something like:

Rake::Task[args["task"]].reenable
Rake::Task[args["task"]].invoke(args["args"])

Use #execute instead of #invoke

Rake::Task[args["task"]].execute(args["args"])

Important! #execute and #invoke are not the same! See more info: https://sampatbadhe.medium.com/rake-task-invoke-or-execute-419cd689c3bd
TL;DR if you have nested tasks or another task as a dependency, #invoke will execute the whole chain, whereas #execute executes only the specified task.

@romankovt Thanks for sharing this Roman. I wasn't aware of this. I will update my code example in case someone misses your comment.

Hello!
How can I put an environment variable in the args of config/sidekiq_scheduler.yml ?

Latest Rails.application.load_tasks already requires rake:
# Load Rake and railties tasks, and invoke the registered hooks.
# Check Rails::Railtie.rake_tasks for more info.
def load_tasks(app = self)
  require "rake"
  run_tasks_blocks(app)
  self
end

So we may avoid requiring twice ;-)
https://github.com/rails/rails/blob/main/railties/lib/rails/engine.rb#L470


Make accessibility improvements for the AudioPlayer

Related to #8112

[x] update startTime when slider received focus.

Who is this for?
Screenreader users
What is it doing for them?
Improving what gets announced about the AudioPlayer

Closing for now. Going to add volume and radio buttons.


ModuleNotFoundError: No module named 'tensorflow'

i am using python 3.6.7 , in python 3.6.7 version not able to install tensorflow, please post the solution and guide to me.
error:         Could not find a version that satisfies the requirement tensorflow (from versions: )
No matching distribution found for tensorflow

It looks like you haven't used a template to create this issue. Please resubmit your issue using a template from here. We ask users to use the template because it reduces overall time to resolve a new issue by avoiding extra communication to get to the root of the issue. We will close this issue in lieu of the new one you will create from the template. Thank you for your cooperation.


Plugin for Detectify

Hello Everyone,
Just want to check if we have any plugin for Detectify to fetch report from Detectify to DefectDojo like Qualys or BurpeSuite.
Let me know if we have any.

If you just want to ask a question please come to our slack channel. Or in these case check the list of parsers in the docs: https://defectdojo.github.io/django-DefectDojo/integrations/import/ or the demo instance or in your own instance.

It seems that Detectify is an only service. So only API extract I think.
ref: https://developer.detectify.com/?_gl=18jxidj_gaNDAwMDU3NjA2LjE2MzE3MjQxNTQ._ga_F9HDKDJVKNMTYzMTcyNDE1My4xLjEuMTYzMTcyNDE3NS4w_fplc*NUpaV0lCNzFydXI5RW9NRW5BSmNkU1RHcG84QWxydzhpRzQ3VVQ1MFRjd2FZYk1PTnklMkJzRmFqVGYyZk5IeGUzZHRhb3dFYUpSUGR1aDlEYUdRdEtTTUFBaTNjMXZpNHBKYWklMkJnJTJGNGNyVVUxMEIzcTQwJTJGdmFMaTUyQyUyRllwdyUzRCUzRA..#operation/singleReport

If you just want to ask a question please come to our slack channel. Or in this case check the list of parsers in the docs: https://defectdojo.github.io/django-DefectDojo/integrations/import/ or the demo instance or in your own instance.
We can leave this open as feature request. Does detectify export xml/json reports or doesn it require import via detectify api? If you provide example we may be able to implement the import.

Detectify export the reports in PDF format.


ARIES-1780 - Redesign of zookeeper discovery using DS

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Aries-rsa-pr/31/


HBASE-26103 Deprecate BufferedMutatorParams#pool method

In HBASE-26103#comment, we discussed to remove BufferedMutatorParams#pool and BufferedMutatorParams#getPool methods.
But BufferedMutatorParams#InterfaceAudience is public and BufferedMutatorParams#pool and BufferedMutatorParams#getPool methods are both public so we can't remove them. We need to deprecate them for 1 version and remove in next.
@saintstack  Please review.

@saintstack @anoopsjohn  Could you please review ?


Readme example does not compile

The README has basically this as an intro example.
package main

import (
	"github.com/ShadowApex/godot-go/godot"
)

func init() {
	// Register will register the given class constructor with Godot.
	godot.Register(NewSimpleClass)
}

func main() {
}

// SimpleClass is a simple go struct that can be attached to a Godot Node2D object.
type SimpleClass struct {
	godot.Node2D
}

// X_ready is called as soon as the node enters the scene.
func (h *SimpleClass) X_ready() {
	godot.Log.Warning("Hello World!")
}

// NewSimpleClass is a constructor that we can pass to godot.
func NewSimpleClass() godot.Class {
	return &SimpleClass{}
}

However will not compile: undefined: godot.Node2D.   The docs appear to be out of date.

I recently merged in a large change that re-implements much of the library functionality, and hadn't yet updated the README.
I updated the README so the steps should compile if you have the latest version of godot-go.


Star+ support

Hey, Star+ uses the same technology of Disney+. It's literally the same player (UI, features, everything) and the same pattern for url: [language]/video/[id].
I still haven't tried to copy and paste codes from the Disney+ integration into a new template using that @trakt-tools/cli. Should be easy, but I don't what to do after with these TypeScript files, lol.
Anyway, if someone know how and wants to try and share here, it will be very great to all of us!

Ok, so I have made something very dumb, but it's working!
Until somebody adds the Star+ in a way not dumb, my ugly method is working. I don't know if I breaked any other service doing this, anyway... I just use Disney+, Star+ and HBO Max (broken by default) with this extension, so for me it's very very nice.
It's just an improvisation because, as I saied above, I have never worked with TypeScript and because of that I don't know what to do after with the TypeScript files generated by @trakt-tools/cli :/

Copy and paste the disneyplus.js and rename the duplicate to starplus.js.

I was using VS Code for this. Open starplus.js, do Ctrl + H, and use like this:

Open options.js, format the document and paste this code where there's the services' data:

new I({
          id: "starplus",
          name: "StarPlus",
          homePage: "https://www.starplus.com/",
          hostPatterns: ["*://*.starplus.com/*"],
          hasScrobbler: !0,
          hasSync: !1,
          hasAutoSync: !1,
        }),

Open manifest.json and add permission for *://*.starplus.com/*.

The last step is to open background.js, format the document and copy and paste this code:

var lr = new U({
        id: "starplus",
        name: "StarPlus",
        homePage: "https://www.starplus.com/",
        hostPatterns: ["*://*.starplus.com/*"],
        hasScrobbler: !0,
        hasSync: !1,
        hasAutoSync: !1,
      });
      new ((function (t) {
        function e() {
          return o(this, e), r.call(this, lr.id);
        }
        l(e, t);
        var r = (function (t) {
          var e = (function () {
            if ("undefined" == typeof Reflect || !Reflect.construct) return !1;
            if (Reflect.construct.sham) return !1;
            if ("function" == typeof Proxy) return !0;
            try {
              return (
                Boolean.prototype.valueOf.call(
                  Reflect.construct(Boolean, [], function () {})
                ),
                !0
              );
            } catch (t) {
              return !1;
            }
          })();
          return function () {
            var r,
              n = p(t);
            if (e) {
              var i = p(this).constructor;
              r = Reflect.construct(n, arguments, i);
            } else r = n.apply(this, arguments);
            return f(this, r);
          };
        })(e);
        return e;
      })(tr))();

6. Reload the extension.

7. Enable StarPlus in the settings.

Enjoy the Trakt integration with Star+!

Disney+ still working, so... Nice.


Fat32 format

Not sure if this is the way to note this, but I had to specifically format to Fat32 (from exFAT) by default to be able ot get the pruple light and have it work. Might be worth noting that in the documentation as what I could tell, most links eventually come here

Added formatting instructions in troubleshooting section.


org-pdftools slows down org-mode significantly

What did you expect to happen?
Given the following configs on debian buster (not a frankendebian) with emacs-27.1 (installed via nix derivation), I expect the org-capture pane to take < 20s to appear:
config.el:
;;; $DOOMDIR/config.el -*- lexical-binding: t; -*-

(setq user-full-name "Sam"
      user-mail-address "Sam's email")

init.el:
(doom! :input
       :ui
       doom              ; what makes DOOM look the way it does
       doom-dashboard    ; a nifty splash screen for Emacs

       :editor
       (evil +everywhere); come to the dark side, we have cookies

       :tools
       pdf      ;;;; <<< this is the problem

       :lang
       (org +capture)
       :config
       (default +bindings))

package.el: none
Empirically, this not only effects org-capture but a lot of org-functionality (collapsing a TODO, was another). It is also worth pointing out that this change happened after a recent update of doom.
What actually happened?
org-capture takes >20s (maybe more like 1-2min) to appear. Screen capture can be found here:
https://s3-us-west-1.amazonaws.com/static.stites.io/Screencast_fixed.mp4
Steps to reproduce:

nuke everything
add configs from above
doom purge && doom sync
open emacs
open capture with SPC X

profile-report points to `org-pdftools-store-link`

Function                                                                                CPU samples    %
- command-execute                                                                             33506  99%
 - call-interactively                                                                         33506  99%
  - funcall-interactively                                                                     33496  99%
   - org-capture                                                                              33492  99%
    - org-store-link                                                                          33472  99%
     - org-pdftools-store-link                                                                33462  99%
      - load-with-code-conversion                                                             33462  99%
       - eval-buffer                                                                          33462  99%
        - require                                                                             33460  99%
         - byte-code                                                                          33447  99%
          - require                                                                           33447  99%
           - byte-code                                                                        33354  99%
              char-displayable-p                                                              33336  99%
            + require                                                                            12   0%

System information:

((system
  (type . gnu/linux)
  (config . "x86_64-pc-linux-gnu")
  (shell . "/bin/bash")
  (uname . "Linux 4.19.0-10-amd64 #1 SMP Debian 4.19.132-1 (2020-07-24) x86_64")
  (path "/nix/store/fvkwfry7q5v9n422839qwdi5sn7j90k4-emacs-packages-deps/bin" "~/.nix-profile/bin" "/opt/anaconda/bin" "/opt/anaconda/condabin" "~/.cargo/bin" "~/.local/bin" "/usr/local/sbin" "/usr/sbin" "~/.tmuxifier/bin" "~/.emacs.d/bin" "~/.local/bin" "~/.nix-profile/bin" "~/.nix-profile/bin" "/usr/local/bin" "/usr/bin" "/bin" "/usr/local/games" "/usr/games" "/snap/bin" "/nix/store/i2afxpb95rl38h1a8362w0sby8gp5x8w-emacs-unstable-emacs-27.1/libexec/emacs/27.1/x86_64-pc-linux-gnu"))
 (emacs
  (dir . "~/.emacs.d/")
  (version . "27.1")
  (build . "Sep 25, 2020")
  (buildopts . "--prefix=/nix/store/i2afxpb95rl38h1a8362w0sby8gp5x8w-emacs-unstable-emacs-27.1 --disable-build-details --with-modules --with-x-toolkit=gtk3 --with-xft --with-cairo CFLAGS=-DMAC_OS_X_VERSION_MAX_ALLOWED=101200")
  (features . "XPM JPEG TIFF GIF PNG RSVG CAIRO SOUND DBUS GSETTINGS GLIB NOTIFY INOTIFY LIBSELINUX GNUTLS LIBXML2 FREETYPE HARFBUZZ M17N_FLT LIBOTF ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 XDBE XIM MODULES THREADS LIBSYSTEMD JSON PDUMPER LCMS2 GMP")
  (traits batch server-running))
 (doom
  (dir . "~/.config/doom/")
  (version . "2.0.9")
  (build . "HEAD -> develop 1456108d 2020-09-10 18:03:32 -0400")
  (elc-files . 0)
  (modules :ui doom doom-dashboard :editor (evil +everywhere) :tools pdf :lang (org +capture) :config (default +bindings))
  (packages "n/a")
  (unpin "n/a")
  (elpa "vterm")))

One minor issue:

Also char-displayable-p seems to check if a character can be found in your system fonts. What does find /usr/share/fonts/ -type f | wc -l give?

$ find /usr/share/fonts/ -type f | wc -l
4661

RE: broken image -- very weird! It looks like this works on chromium, but not firefox. I'll fix this in a bit.

Regarding the video, the pixel format is yuv444p and Firefox doesn't support these kinds of videos because even though ffmpeg can decode them just fine, the system decoders on macOS and Windows can't so they've disabled playback for these kinds of videos on all platforms.

Ah, the unexplored world of video encoding -- good to know! I think this version should do the trick. I'll edit the issue as well/
https://s3-us-west-1.amazonaws.com/static.stites.io/Screencast_fixed.mp4
Of note:

from 0:00 -> 2:30, this is just me showing the steps I took to reproduce the error
around 2:30 shows emacs becoming unresponsive
2:50ish shows some strace outputs -- this goes on until about 4:50
after this, pdf tools needs to compile its dependency.
at ~5:26 I basically just do everything all over again to demonstrate that every first capture after boot produces the same result and this isn't dependent on the pdf-tools dependency not being present.

I've narrowed down the issue to https://github.com/hlissner/doom-emacs/blob/ea3502376571b18c66c690e7cfda832e2bd850a3/modules/lang/org/config.el#L936 but I'm not sure what this function does and why it is being triggered on org-capture.

I cannot reproduce this by mirroring your config or steps, using the same emacs 27.1 build on nixpkgs stable. It invokes the org-capture popup instantly, and without issue.
It appears to hang on char-displayable-p, which would imply a font or encoding issue. Perhaps reinstalling Emacs is your best bet. Or running doom build. I'm afraid I don't know where else to look for this.

Thanks for looking into this @hlissner -- closing this since it can't be reproduced.
I'll look into font issues next and report back here if I find the problem.


Added rule for replacing toggle_disabled with false in test-resources

Added rule in test-resources

Checks for occurances of !ENUM_NAME.STALE_FLAG.isEnabled and replace them with false.

Stack from ghstack (oldest at bottom):

-> #344
#343

Yes, I can do that.


Normalize strides of cuDNN descriptors

Fix chainer/chainer#6644 and some other cuDNN issues.  Revert an ad-hoc fix.  _make_tensor_descriptor_array could be fixed similarly, but not in this PR.
Drawback:

Normalized stride is incorrect if an input array has 1-length axes and other contiguity is used.

Alternative fixes:

Use _ascontiguousarray_normalized_strides (in #2261) everywhere

It will create more ndarray objects.
It is overhead if (4-dim) tensor discriptor is used.  It is too unmaintainable to keep core._internal_ascontiguousarray for the safe cases.
It depends that the internal _reshape preserves normalized strides. https://github.com/cupy/cupy/blob/b9db374f4d0ccee358a998d7b283248f77cef351/cupy/cudnn.pyx#L1673-L1674

Make contiguous array and its cuDNN descriptor simultaneously.

Not easy to fix the relevant codes.

pfnCI, test this please.

I've checked where _make_tensor_descriptor_array  is called and suppose that the drawback above is acceptable. Would it be better to check pooling_forward's and pooling_backward's ys are c-contiguous? It is currently ensured in Chainer side.

Jenkins CI test (for commit 7efcc2d34e496ececa7b6aee5f1a1d0b6a778e5f, target branch master) succeeded!

pooling_forward: added the check
pooling_backward: added ascontiguousarray

pfnCI, test this please.

LGTM! Waiting for CI finish.

Jenkins CI test (for commit 6621f4876d8c4ec73c3ab1898b48cd80058d305b, target branch master) succeeded!


Fix undefined names in Python code

$ flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
./qlib/qlib/contrib/model/pytorch_tabnet.py:567:38: F821 undefined name 'inp'
            self.independ.append(GLU(inp, out_dim, vbs=vbs))
                                     ^
./qlib/examples/model_rolling/task_manager_rolling.py:75:18: F821 undefined name 'task_train'
        run_task(task_train, self.task_pool, experiment_name=self.experiment_name)
                 ^
2     F821 undefined name 'task_train'
2

Undefined names are usually a sign of a typo, missing imports, or code that has not been ported to Python 3.  These also would be compile-time errors in a compiled language but in Python, a NameError may be raised which will halt/crash the script on the user.

Description

Motivation and Context

How Has This Been Tested?

[ ] Pass the test by running: pytest qlib/tests/test_all_pipeline.py under upper directory of qlib.
[ ] If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

Pipeline test:
Your own tests:

Types of changes

[ ] Fix bugs
[ ] Add new feature
[ ] Update documentation

@cclauss  It looks great!
Thanks!


Support for NdConvolution and NdPooling

Hi !
If I am not mistaken there is no support for NdConvolution and NdPooling modules.
Am I right ?
I am trying to load the C3D network from facebook. Which is sequential, but has 3D convolution and pooling operations.
Do you know if those modules will be included in the future ?
Thank you for any information you would be able to provide :)

@hassony2 sorry for bothering you! I have the same issue, did you solve it?

Hi @liu666666,
Unfortunately I did not solve it at that time, and ended up working directly with the caffe implementation.
Good luck !


feat: support crossOrigin option

…full when user want to collect errors from gtm scripts into error reporting tools like appinsihgts, sentry etc

Thanks @aldarund sorry for late merge


Implement conversion Integration tests for Galley

Implements the conversion integration test for Galley.
Moves current validation tests that are at the "galley/" to a lower level folder at "galley/validation" for better test refactoring.
Implement a conversion test based on an input-output golden file model that tests Galley's MCP output, based on K8s resource inputs in YAML form.
Make Galley Server restartable. This is done by making the event registration restartable as well.
Add a new local-only Galley component in the new test framework.
Fix Makefile issue that causes test.integration.* targets to shadow test.integration.*.kube targets.
Introduce an in-memory mcp/client/Updater implementation for testing.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [ozevren]
pkg/OWNERS [ozevren]
tests/integration2/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Same as https://github.com/istio/istio/pull/10381. Recreated a new PR to for 1.1 branch.

Friendly ping on this. This is the same PR that was reviewed against the master branch, just rebased on 1.1.
I'd like to build on this.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: geeknoid, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

galley/OWNERS [geeknoid,ozevren]
pkg/OWNERS [geeknoid,ozevren]
tests/integration2/OWNERS [geeknoid,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Complete Pipeline Play, Pause, Stop

Initial implementation complete... correct behavior is dependent on new issue to refactor UnLink functions.


Bug Report: Text Renderer does not support removing /re-adding strings

Describe the bug
The free list of string handles does not work. The vertex shader currently works by checking all strings in-order. This means having strings that are not in order of offsets is wrong and breaks horribly.
Reproduction

Repros On: [tz_text_rendering_demo] (with local modifications to remove/add new strings over time)
Repro Type: unexpected behaviour
Repro Rate: every time
Repro Severity (how badly does it affect your bottom line): high
Repro Steps
cba.

Expected behavior
Users of text_renderer should be able to remove and then add new text without dodgy behaviour. They shouldn't need to worry about the internal sorting of strings having to be correct.
Screenshots
N/A
Conditions (please complete the following information):

OS: windows
Render API: vulkan
Build Config: debug
Version: master
Compiler: clang

Fixed at 68a643885423ebf626ff0e5e7e1b7eb1a3602c9a


LoadGen: Simplify atexit segfault fix.

This also seems to work without requiring exceptions to
be enabled.
There is a risk that std::unique_ptr's destructor
gets unloaded along with the loadgen module, but it doesn't
seem to be an issue in practice. Either it's optimized away
or it's still valid for some reason.

@guschmue , can we merge this too? It simplifies the segfault work around.

I see that you modified the sn config ... it this fix going to work when building from setup.py ?

No changes required to setup.py.
The previous approach required exceptions to be enabled. This approach works with or without exceptions. That's why I modified the BUILD.gn files.
If the previous approach broke setup.py for anyone, this will fix it. Although, now that I think about it, pybind11 requires exceptions to be enabled, so setup.py should work for everyone with either approach.


Label's "uppercase" property does not account for locale

Godot version:
3.0
Issue description:
Label currently has the "uppercase" property, but it doesn't work for any language, since the same small letter character may be represented by a different capital letter character depending on a language. For example, lowercase i must be uppercased into I for English, but into İ for Turkish and Azerbaijani.
There is at least one case when a single small letter is represented by multiple capital letters - in German ß is uppercased into SS.
This makes uppercase property of a Label a bit misleading, since people may not be aware of these specifics, use it for English, but then face problems when localizing their game.

Still valid in 4.0 15aea89
CC @bruvzg


Sunconure's sleep schedule is broken.

I do not want that. I know what a broken sleep schedule does to people and it's not a good thing. :(
Get well and no rush with the textures!

Its recovering, and I might be able to do copper daggers today. Keep in mind this happened partially because of working too hard with Growthcraft's Milk Module.

Ah, don't hurry! Just take the time you need.

I also tweaked some of my armor and weapon suggestions a bit. Go and check them out, to see if they fit better.

Intimidation stuff actually does stuff to mobs now too.

Yeah, I see :D I hope, that your sleep schedule is better now ^^ Thank you!

There's really no reason to rush, since it usually takes longer to code a new item than to draw or recolor it. ;)

Sleep schedule has recovered. Expect silver textures this weekend.

Good to hear. :D


Request for discussion: Priority of merges to master

Currently, bug fixes are merged to master early. There are bugs, but as long as nobody suffers and complains, IMO to fix them does not need that high priority. Those bugs in most cases do not even come to play, because that would require an edge case configuration. Those are rare, I think.
Several people including me think, that one of the most important things for a ground laying component like ServiceManager is performance. That important, that bad performance is seen as a bug. So enhancements here will often get recognized as a bug fix, IMO.
I suggest to rethink, what changes qualify for a quick deployment to master. Would do you think about that?

Bug-fixing only on master, everything else goes to develop, no exclusions.
Improvements, whilst welcome, do always go to minor/major as per SemVer,
which we adhere as closely as possible.
On 1 Feb 2018 04:18, "Frank Hein"<EMAIL_ADDRESS>wrote:

Currently, bug fixes are merged to master early. There are bugs, but as
long as nobody suffers and complains, IMO to fix them does not need that
high priority. Those bugs in most cases do not even come to play, because
that would require an edge case configuration. Those are rare, I think.
Several people including me think, that one of the most important things
for a ground laying component like ServiceManager is performance. That
important, that bad performance is seen as a bug. So enhancements here will
often get recognized as a bug fix, IMO.
I suggest to rethink, what changes qualify for a quick deployment to
master. Would do you think about that?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-servicemanager/issues/246, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAJakOTAtEz0y1atmVhKw3RgLG7txQZkks5tQS0AgaJpZM4R0_sr
.

Thanks. This versioning scheme is good practice.
I don't think, that there is urgency to apply #242 for the bug fixes, because those bugs seem to be not critical in the field. Otherwise there would be loud complains.
That brought me to question the classification scheme (i.e. not the versioning scheme). There seem to twilight zone, where it is difficult to say, if something should be classified as a bug or as an enhancement.
Imagine, ServiceManager's get() would take 10 minutes to return a service. This would be considered as a bug, I believe. Allthough it could be seen as an enhancement also.
On the other end of the scale is a bugfix for abstract factories defined via members. Allthough this is a bug definitely, it could be treated like an enhancement (or nice to have bugfix), because nobody defines abstract factories via member obviously.

@fhein Again, if a performance regression is reported due to previous changes in code within the same minor release, we'd consider a performance bugfix. Otherwise, performance improvements are generally features for minor releases.
Generally speaking, bugfixes are the result of reported unexpected behavior; features and enhancements are improvements that present new behavior. Improved performance is a new feature generally, unless it is a by-product of a bugfix.
I'd argue #242 is in fact a bugfix, and there are examples of pre-configured service managers in many components: specifically, any plugin manager. The bugfix is primarily around the aspect of properly resolving pre-configured service mappings; the performance improvement is a side-effect.

Agreed. In that particular case performance is a side-effect. And in general performance improvements are features (allthough I still believe, that a 10 min get() would be considered to be a bug ;).
I close that now. Thank you! :+1:


Build problems with SdkVersion 34 (Android 14) on Release

Hi, we are having some problems when updated the Android Gradle Plugin (AGP) to version v8.1.3 for sdkVersion 34. And had to rollback to v7.3.1 (kept sdkVersion 34, even that was not recommended).
I believe disabling the minimization in the release, it also works, as in Debug, it works with no problem.
We are using the latest version of 2020 v1.4.11.

Did you add the following proguard rule:
-keep public class cz.msebera.android.httpclient.*


added ikettle2

Yes please add .title, else there are issues with Admin, we reverted in adapter-creator, sorry
Some more review comments:

you may want to cleanup https://github.com/Jey-Cee/ioBroker.ikettle2/blob/master/admin/words.js
best call callback in unload in the callback from socket.end() to make sure it is really finished before callinf unload-callback
please check for ack flag and not from in https://github.com/Jey-Cee/ioBroker.ikettle2/blob/master/main.js#L122
you set the "ack" flag before even executing the action? https://github.com/Jey-Cee/ioBroker.ikettle2/blob/master/main.js#L126-L153

Please check the comments

you set the "ack" flag before even executing the action? https://github.com/Jey-Cee/ioBroker.ikettle2/blob/master/main.js#L126-L153

Yes, because the value will be changed inside the function. So if i want to see if the adapter has accepted the command this is needed.
Other things are done.

Sorry for the delay ... and welcome in the repo


Error in Edge Ember-cli from ember-pickr because of spread operator

This attached line throws an error in Edge when running locally.

I believe this is because object spread isn't supported in Edge yet. However, this must be handled by transpilation. Can you check if Edge is present in your browser targets?

@Benjy1979, were you able to address this by adding Edge to the browser targets?

Yes, thanks that did it. I wasn't aware of this feature in Ember - ignorance

No issues @Benjy1979, do let me know if you face any further issues. Will be happy to help out.


Update availability to make recap hold request status unavailable

Related to: https://github.com/pulibrary/voyager_helpers/issues/69
When an item has a status of "Hold Request":

[ ] Regular recap items should be unavailable
[ ] Rare "always requestable" recap items should still be available, see the aeon_location flag
[ ] All other locations should ignore this status

Are there any complications with the Pending Order, On Order, or In Process statuses that could arise here?


Add specs-report on CI

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.45%. Comparing base (0e23e5d) to head (3ba652f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #101   +/-   ##
=======================================
  Coverage   93.45%   93.45%
=======================================
  Files          36       36
  Lines        1345     1345
=======================================
  Hits         1257     1257
  Misses         88       88

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This is great


Issues when scanning as part of Jenkins Pipeline job

Hi folks, I'm having some odd issues with incorporating this into a Pipeline job.
When I run a scan on the Jenkins instance's command line, it completes with no problems, but when I put the scan command into a build step, it scans some of the layers before throwing a 400 error - this is on the same image. The Jenkins instance itself is in a container and is working normally and the Jenkins user has access to the host's Docker socket file (Docker in Docker.)
Does anyone have any ideas what might be causing this? It's the exact same command, only used in two different places - could it be an environment variable or something?

Two jobs running at the same time?
I haven't some across this problem. Is it possible for you to test this out on an empty/default Jenkins (in docker) with just a simpel job that scans an image and see what happens?

Not to worry - I rebuilt everything just as a test and it seems to be working fine now. Guess it was just a weird network hiccup!


Can't make code completion for the methods of class fields when developing chromium.

Can't make code completion for the methods of class fields when developing chromium.
But it's right to make code completion for the methods of local variables, and it's also right to make code completion for the methods of class fields when developing a simple CMake-based project.
Logs
The verbose logs of code completion for the methods of a class field:
 V[18:18:08.699] <<< {"id":76,"jsonrpc":"2.0","method":"textDocument/completion","params":{"context":{"triggerCharacter":".","triggerKind":2},"position":{"character":10,"line":46},"textDocument":{"uri":"file:///home/*****/message/dependencies.h"}}}
I[18:18:08.699] <-- textDocument/completion(76)
I[18:18:08.707] --> reply:textDocument/completion(76) 8 ms
V[18:18:08.707] >>> {"id":76,"jsonrpc":"2.0","result":{"isIncomplete":false,"items":[]}}
error_clangd.log
The verbose logs of code completion for the methods of local variables:
V[18:11:38.854] <<< {"id":176,"jsonrpc":"2.0","method":"textDocument/completion","params":{"context":{"triggerCharacter":".","triggerKind":2},"position":{"character":7,"line":75},"textDocument":{"uri":"file:///home/wxq/aha/aha/morpheus/message/dependencies.cc"}}}
I[18:11:38.854] <-- textDocument/completion(176)
I[18:11:38.881] --> reply:textDocument/completion(176) 26 ms
V[18:11:38.881] >>> {"id":176,"jsonrpc":"2.0","result":{"isIncomplete":true,"items":[{"detail":"const ::entities::ChatConfig &","filterText":"chat_config","insertText":"chat_config()","insertTextFormat":2,"kind":2,"label":" chat_config() const","score":1.2384182214736938,"sortText":"40617b83chat_config","textEdit":{"newText":"chat_config()","range":{"end":{"character":7,"line":75},"start":{"character":7,"line":75}", "***other items***"]}}
ok_clangd.log
System information
Linux XPS-8930 5.4.0-91-generic 102~18.04.1-Ubuntu
Output of clangd --version:
clangd version 13.0.0 (GitHub - llvm/llvm-project: Th... d7b669b3a30345cfcdb2fde2af6f48aa4b94845d)
Features: linux+grpc
Platform: x86_64-unknown-linux-gnu
Editor/LSP plugin:
emac/lsp-mode
Operating system:
Linux

There's not really enough information in the report or the logs to understand what's happening. If you can reduce it to a small example that would be helpful.
The one clue from the logs is there are lots of compile errors coming from inside the standard library, which sounds a lot like the compile command for the file isn't correct/working. Are there lots of diagnostics shown? Can you provide the output of clangd --check=$YOUR_FILE?

@sam-mccall  Thanks for the hint. After I move the lib/clang/13.0.0/include to the correct directory, clangd works well.


Procedure call blocks can't handle _updateDisplay being called multiple times

They never clear out the mutated fields before adding them again, so the number of fields will only increase.
Whoops.

Fixed by #1225


Reduce PEP-8 errors

See https://github.com/Senzing/g2-python/issues/123

Pull requests:

https://github.com/Senzing/g2configtables/pull/15


docs(genai)📄: Add a new document discussing the use of generative AI in Bible study courses

Introduce a new markdown document exploring the use of generative AI in Bible studies.
Discuss perspectives on AI in religious contexts, weighing pros and cons.
Provide examples of AI as a translation tool, content generator, and reviewer.

🚀 Deployed on https://deploy-preview-51--cbcgb-com-sgbs-training.netlify.app

我个人的看法是，可以用GenAI做secular tasks例如设计简单的观察类问题，summarize经文，summarize解经书，smooth out让查经问题更加interpersonal之类的，但是不能做theological tasks比如设计解释题，应用题，翻译圣经等等

不如你也加一两段吧！这个观点也是值得借鉴的。


fix(tilecatalog): fix flash between loading data asynchronously

Summary

Refix of my previous issue to fix the flash between loading data in the tile catalog.  Also cleaning up a console error in the batch actions

Change List (commits, features, bugs, etc)

fix(table): batch actions were throwing a React warning due to passing an extra parameter to the button

Related Issues

Closes IBM/carbon-addons-iot-react#197

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.40.4 :tada:
The release is available on npm package (@release-1.x dist-tag)
Your semantic-release bot :package::rocket:


feature/v5-asset-withdraw

implements the withdraw endpoint

@decanus Thank you for your commit!
It looks like it needs to be changed. Could you please confirm the contents?
Thanks again.

@hirokisan made the necessary changes


spelling fixes

patch contains some spelling fixes ( just in comments ) as found by a bot ( http://www.misfix.org, https://github.com/ka7/misspell_fixer ).

Coverage remained the same at 99.035% when pulling f7082f88468360638065c5df8e9c192b1b7de07a on ka7:spelling into 2b59b895119e2d33ee9f82765fe4e063fa23fc58 on pydoit:master.

thanks


Vague "Cannot find Primary Constructor" compile error

Got this error:
e: java.lang.IllegalStateException: Can not find primary constructor
	at io.realm.compiler.RealmModelSyntheticPropertiesGeneration$addNewInstanceMethodBody$1.invoke(RealmModelSyntheticPropertiesGeneration.kt:477)
	at io.realm.compiler.RealmModelSyntheticPropertiesGeneration$addNewInstanceMethodBody$1.invoke(RealmModelSyntheticPropertiesGeneration.kt:475)
	at io.realm.compiler.IrUtilsKt.blockBody(IrUtils.kt:98)
	at io.realm.compiler.RealmModelSyntheticPropertiesGeneration.addNewInstanceMethodBody(RealmModelSyntheticPropertiesGeneration.kt:475)
	at io.realm.compiler.RealmModelLowering.lower(RealmModelLoweringExtension.kt:77)
	at org.jetbrains.kotlin.backend.common.ClassLoweringVisitor.visitClass(Lower.kt:82)

Kinda hard to debug without more information. We should also add the Class being modified to the exception message.

The cause is this code like this:
class Person : RealmObject {
  constructor() { }
  constructor(input: String) { }
}

We still require a no-op constructor in order to create objects on the user's behalf, but forcing the no-op to be the primary constructor seems a bit odd.... Instead we should probably search all viable constructors and only then report the error?

Verify if this is fixed

Not fully on top of this, but seems like the work that I thought fixed this (#184/#673) was done prior to this issue being reported so assume there is still something to do.


Ensure AsyncQueue events have sufficient exception handling

so what are you looking for here?

Migrated to: https://dev.azure.com/Stratisplatformuk/StratisBitcoinFullNode/_workitems/edit/3052


Factory now returns rangy object. Fix for npm and browserify.

Thanks for this. You (or your text editor) even tidied some superfluous white space :)

Actually, this change makes the raw, unbuilt JS files (which I use in development) invalid, so I've had to revert. I'll apply the change in the build script instead.

I was not sure where is the proper place so I'd be glad if you could make changes in the way that they fit the exact flow. Cheers!


Load models external to calaccess_raw

We're creating additional models and want to leverage calaccess_raw commands to load those models from CSVs.
This patch allows you to specify an application name separate from calaccess_raw so that you can load your own modules. Here's an incomplete example:
https://gist.github.com/adborden/e4382d5c2d7a0654befb

This is a very interesting issue I hadn't considered before.
Another potential implementation that comes to mind (though maybe it's fundamentally flawed or missing the point) would be to break out the get_model line of the management command into its own method so that your subclass could override that one bit and we wouldn't have to add a kwarg to a big long complicated thing.
Am I getting what you're after?

@palewire yeah, that's the idea. I think that would work well. I'll refactor the patch.

I gave this some more thought and came back around to your style of the solving the problem. Since a command like yours might wish to load a mishmash of models from different apps, an override of the command wouldn't work because you wouldn't be able to switch it around line by line.
So what I've tried in the patch I just pushed is adding a management command option --app-name that allows you to pass in your own variable each time you run it, with calaccess_raw set as the default.
So you could do something like:
call_command(
   "loadcalaccessrawfile",
    your_model_name,
    verbosity=self.verbosity,
    app_name=your_app_name,
)

Or from the command line like:
$ python manage.py loadcalaccessrawfile MyFileName --app_name=my_app_name

How's that look?

Awesome, that's working well, thanks!

Glad to hear it's what you need. If you have any other requests or issues don't be shy.


Issue with Dragon 1 photo

There's an issue in regards to the second photo of the 'Dragon 1' vehicle.
Here's the updated section of the JSON for the vehicle's photos:
{
  "https://i.imgur.com/9fWdwNv.jpg",
  "https://live.staticflickr.com/8578/16655995541_078768dea2_b.jpg",
  "https://farm3.staticflickr.com/2815/32761844973_4b55b27d3c_b.jpg",
  "https://farm9.staticflickr.com/8618/16649075267_d18cbb4342_b.jpg"
}

Should be fixed


how to get only object not background

i want to get object only. not background
Are there any other options besides adding a foreground mask?

Hi, the easier way is to train the model with background and crop or extract the ROI after training.

Hi, the easier way is to train the model with background and crop or extract the ROI after training.

than you:)  But I don't know how to do it, can you recommend a way to cleanly remove only the mesh part I want?
Currently, when I remove it from meshlab, the bounds that touch the object I want are lowered, leaving the object open and empty.


Implement Tree sort

https://en.wikipedia.org/wiki/Binary_search_tree#Sort

Implemented in this release


feat(ci): add yaml validation to the pipeline

As requested: https://github.com/dyrector-io/dyrectorio/pull/580#issuecomment-1536402997
This PR adds YAML validation/linting to the pipeline.
PR #580 and #586 are required for this.

Codecov Report
Patch and project coverage have no change.

Comparison is base (1126dff) 18.04% compared to head (817fb76) 18.04%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #588   +/-   ##
========================================
  Coverage    18.04%   18.04%
========================================
  Files           54       54
  Lines         5082     5082
========================================
  Hits           917      917
  Misses        4138     4138
  Partials        27       27

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bump to latest codeclimate-test-reporter

Diff: https://github.com/codeclimate/ruby-test-reporter/compare/v0.4.0...v0.5.0
@alex-stone @joshlam

:+1: