kfd2kgd: init_user_pages: Failed to get user pages: -14

Hi, I have run ROCM 1.7 into a Ubuntu 16.04 VM by passthrough GPU.
GPU: amd FirePro S7150
CPU: Intel(R) Xeon(R) CPU E5-2680 v2 @ 2.80GHz
When I run /opt/rocm/hip/samples/0_Intro/square sample, below error happend.
$ cd /opt/rocm/hip/samples/0_Intro/square && make
$ ./square.out

Backtrace:
0x00007f107e3e05e6:     Kalmar::HSAContext::initPrintfBuffer() + 0x126
0x00007f107e3c6378:     Kalmar::HSAContext::HSAContext() + 0xa68
0x00007f107e3c41b6:     _GLOBAL__sub_I_mcwamp_hsa.cpp + 0x5f6
0x00007f10816cc6ba:     _dl_rtld_di_serinfo + 0x706a
0x00007f10816cc7cb:     _dl_rtld_di_serinfo + 0x717b
0x00007f10816d18e2:     _dl_find_dso_for_object + 0xcf2
0x00007f10816cc564:     _dl_rtld_di_serinfo + 0x6f14
0x00007f10816d0da9:     _dl_find_dso_for_object + 0x1b9
0x00007f10814b8f09:     <unknown function> + 0x1b9
0x00007f10816cc564:     _dl_rtld_di_serinfo + 0x6f14
0x00007f10814b9571:     dlerror + 0x2c1
0x00007f10814b8fa1:     dlopen + 0x31
0x00007f10806ce661:     Kalmar::CLAMP::GetOrInitRuntime() + 0x261
0x00007f10806cf5ab:     Kalmar::KalmarBootstrap::KalmarBootstrap() + 0x8b
0x00007f10806cf4f9:     __hcc_shared_library_init + 0x29
0x00007f10816cc6ba:     _dl_rtld_di_serinfo + 0x706a
0x00007f10816cc7cb:     _dl_rtld_di_serinfo + 0x717b
0x00007f10816bcc6a:     <unknown function> + 0x717b

### HCC STATUS_CHECK Error: HSA_STATUS_ERROR (0x1000) at file:mcwamp_hsa.cpp line:3648
Aborted (core dumped)

dmesg show below.
dmesg  | tail -n 3
[  165.695012] kfd2kgd: init_user_pages: Failed to get user pages: -14
[  247.468531] kfd2kgd: init_user_pages: Failed to get user pages: -14
[  420.439295] kfd2kgd: init_user_pages: Failed to get user pages: -14

However, when I run /opt/rocm/hsa/sample is ok.
$ cd /opt/rocm/hsa/sample && make
$ ./vector_copy
Initializing the hsa runtime succeeded.
Checking finalizer 1.0 extension support succeeded.
Generating function table for finalizer succeeded.
Getting a gpu agent succeeded.
Querying the agent name succeeded.
The agent name is gfx802.
Querying the agent maximum queue size succeeded.
The maximum queue size is 131072.
Creating the queue succeeded.
"Obtaining machine model" succeeded.
"Getting agent profile" succeeded.
Create the program succeeded.
Adding the brig module to the program succeeded.
Query the agents isa succeeded.
Finalizing the program succeeded.
Destroying the program succeeded.
Create the executable succeeded.
Loading the code object succeeded.
Freeze the executable succeeded.
Extract the symbol from the executable succeeded.
Extracting the symbol from the executable succeeded.
Extracting the kernarg segment size from the executable succeeded.
Extracting the group segment size from the executable succeeded.
Extracting the private segment from the executable succeeded.
Creating a HSA signal succeeded.
Finding a fine grained memory region succeeded.
Allocating argument memory for input parameter succeeded.
Allocating argument memory for output parameter succeeded.
Finding a kernarg memory region succeeded.
Allocating kernel argument memory buffer succeeded.
Dispatching the kernel succeeded.
Passed validation.
Freeing kernel argument memory buffer succeeded.
Destroying the signal succeeded.
Destroying the executable succeeded.
Destroying the code object succeeded.
Destroying the queue succeeded.
Freeing in argument memory buffer succeeded.
Freeing out argument memory buffer succeeded.
Shutting down the runtime succeeded.

So please let me know how can I make ROCM work in KVM by passthrough amd GPU?
Thanks a lot.

Sorry for the late response, I followed the kvm-passthrough guide still get the the problem.
I have also test it on the host that run the kvm, dmesg still show:
[  444.255690] kfd2kgd: init_user_pages: Failed to get user pages: -14
Host CPU is 'Intel(R) Xeon(R) CPU E5-2678 v3 @ 2.50GHz', GPU is '[AMD/ATI] Tonga XT GL [FirePro S7150]', are thoese CPU and GPU supported by ROCM?

We have limited support for Tonga.


Add test CI: JNI

This part: https://github.com/apache/arrow/blob/main/dev/tasks/java-jars/github.yml

https://github.com/apache/arrow/blob/a6fe59592c643d591ee719041dd2b5d8b9ada7d6/.github/workflows/java_jni.yml#L59-L102 will not be needed because dev/tasks/java-jars/ covers it.


x/build: add a builder with oldest git supported by cmd/go to catch regressions

What did you expect to see?
The bug fix for breaking go on Ubuntu 16 would have a test or ci change to prevent a similar error in the future.
What did you see instead?
A code change with no test change.
Context
https://github.com/golang/go/issues/26501#issuecomment-407431340

Git 2.7.4 is notable(?) because it is the stock version in Ubuntu 16.04 LTS.¹
However, it's not obvious to me why we should draw the line at Ubuntu 16.04: the Ubuntu project claims to still support 14.04.5 LTS,² and the current release is 18.04 LTS. (Ubuntu 18.04 LTS provides git 2.17,³ which is much more recent.)
I don't actually see a minimum Ubuntu or Git version listed on https://tip.golang.org/doc/install. Perhaps we should add one.
(CC: @rsc @rogpeppe @bradfitz @myitcv)
¹https://packages.ubuntu.com/xenial/vcs/git
²https://wiki.ubuntu.com/Releases
³https://packages.ubuntu.com/bionic/vcs/git

I don't actually see a minimum Ubuntu or Git version listed on https://tip.golang.org/doc/install. Perhaps we should add one.

I think thats a wonderful idea!

Whether Ubuntu 14.04 is a sufficiently broad baseline for this sort of analysis I'm not 100% sure (it's pretty good on the basis it's used by Travis); it ships git version 1.9.1

My development VM was Debian Jessie for the longest time and its old git (https://packages.debian.org/jessie/git-core ... 2.1.4) was what finally forced me to update to stretch.

Note also https://github.com/golang/go/issues/26594#issuecomment-411379080, which suggests we might also need to add tests for different remote types.

The current version of the standard linux-amd64 builder has:
$ git version
git version 2.30.2

Closing as done. (To help keep the issues that are still open more accurate and actionable.)

Indeed, I had misread it, thanks for catching.
We can certainly reopen if it's still actionable as is, but as I understand 2.7.4 is relevant to Ubuntu 16.04 LTS, which passed End of Standard Support in April 2021 (understandably 3 years after this was filed) and so might not be.
Since #26746 is accepted by now, we should try to look at this issue when that proposal is implemented.

Per https://github.com/golang/go/issues/26746#issuecomment-1169102174, the oldest Git we support should be at least 2.17.6. (Versions older than that may be affected by CVE-2021-21300.)


make svg for trash can icon

nevermind fixed it after putting back the deleted font awesome cdn

if internet is out the icon will not be available. see if we can remake the icon with figma and export as an svg!


Set up Travis CI

As a [role]
I need [functionality]
So that [benefit]
Details & Assumptions:

...

Acceptance Criteria:
Given [some initial state]
When [some event]
Then [some measurable outcome]

Hey team! Please add your planning poker estimate with ZenHub @Fivetoes @jiuliwei @XinyuC97 @yx2512 @ZX-WANG

I added our repo at travis-ci.com, so all other steps should just work


Auto update and multi uuid

@davidmann4 Please add an auto update feature to this repo as it's very irritating to download zip every single commit and also allow server to manage multi uuid or better ip adresses...

whats auto update?
use git pull

@davidmann4 in Windows make it a batch so that when there's new version it updates itself

git pull


fix(autocomplete): fix while specified to_field

When use ForeignKey and set to_field,
the autocomplete_fields in admin also use pk in queryset yet.
Obvious, this will make some wrong filter result

i made a typo, closed


BUG: Wrong send screen when scanning not added addresses

Describe the bug
When scanning an address that is not added as a chain on the vault, the app just open the send screen with no address.
To Reproduce
Steps to reproduce the behavior:

remove chain
scan address of removed chain
See error

Expected behavior
Display a error message that this chain is not yet added to the vault.
Screenshots

-v14


Storing LP item data in database

This is my first PR that updates the database, so it might be worth getting Bear to review it before merging.

@bearthatcares - could you have a quick look at this one to see if I've done anything obviously wrong?


Feature add advanced search

[Advanced search] Prepare advanced search for the showcase
Description
Preparing the advanced-search components' example for the showcase

Gotta had a good capture screenshot before the merge

Waiting for the capture to merge then

Job done


[CT-1118] Materalize as Redshift external tables (Glue tables)

Describe the feature
A clear and concise description of what you want to happen.
With DBT we have only Table and View materialization which is fine for most of the engines. With Redshift Spectrum, it would be useful to be able to directly materialize models as external table to get a better integration of DBT with the lakehouse capabilities of Redshift.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
A workaround solution i use currently is by materializing the model as view and then use a post hook to create an external table from the view :
{{ config(
   alias='exter',
   materialized='view',
   bind=False,
   post_hook=[{"sql": "{{ insert_to_external_table(
       this.schema,
       this.table,
       'glue_catalog_test',
       'crwiris',
       ) }}",
   "transaction": False},
   after_commit("drop view dev.dbt.exter")]
   )
}}

select * FROM "dev"."glue_catalog_test"."crwiris"

Additional context
Please include any other relevant context here.
Who will this benefit?
What kind of use case will this feature be useful for? Please be specific and provide examples, this will help us prioritize properly.
Lakehouse approach has become attractive since a while and Redshift (with Spectrum) is a major player for handling lake and warehouse data with the same engine and query formats. Many companies would benefit from the ability of DBT to handle Glue tables through Redshift not only for reading but for creating(already possible through Macros) and inserting seamlessly.
Are you interested in contributing this feature?
Let us know if you want to write some code, and how we can help.

Did you try with using the existing external table and the incremental materialization (without unique_key)? This just inserts all data from the model into the target table if it exists.

mp24-git commented 6 days ago

Did you try with using the existing external table and the incremental materialization (without unique_key)? This just inserts all data from the model into the target table if it exists.

If I'm not mistaken, the goal here would be to create external table which does not exist yet.

Thanks for reaching out about external table support @Ans-hrf !
Have you tried the dbt_external_tables package (view on GitHub)?

The issue with bt_external_tables is that it works for sourcing external tables, however we'd like to have a SQL model materialized as external table.
I found the similar issue here: https://github.com/dbt-labs/dbt-redshift/issues/24

Hi @dbeatty10 @mp24-git  and @sanga8 !
Yes indeed, as @sanga8 mentioned, the purpose of my demand is to have a simple process to materialize data in external tables with Redshift. dbt_external_tables allows to create external tables (using sources) but when we want to save dbt models in this tables to the best of my knowledge we don't have a simple way (for example materialize as external_table) to save model result not as a Redshift internal table/view but as an external table in Glue catalog.
In the workaround i propose, i save the model as a view with DBT then i run a post-hook to put the content of the view in an external table leveraging spectrum.
I hope the demand is clearer now, don't hesitate to challenge if not yet clear.
Thanks

Hi all,
I'm closing this ticket and re-opening #24, as a lot more context is available there. Please let me know if you think this is not a duplicate and should deserve to stay open on its own - happy to reconsider.
Thanks!


Great Talk!

I've seen your talk as a last one and I had to go :(
Great talk, and the giveaway of Elixir gems went even better than I thought.
BTW. You can reach me at the same username at google's_mail_service_name dot com ;).
(you can close it after you read it :wink:)

:wink:


Errors while making Microwin

Describe the bug
Errors while making iso
3775 |          "WPFMicrowin" {Invoke-WPFMicrowin}
|                         ~~~~~~~~~~~~~~~~~~
| An unexpected error occurred: The volume of the specified mount path does not support reparse points.  Please
| mount to a volume that supports reparse points.
Unmounting Registry...
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
To Reproduce
Steps to reproduce the behavior:

Go to 'Microwin'
Click on 'select scratch folder'
Click on 'select iso Win11_23H2_EnglishInternational_x64v2.iso'
Click on 'Import drivers from current system'
Click on 'Start Process'
See error

Error```
Getting image information f:\Downloads\winutl-scratch\micro\sources\install.wim
Finding suitable Pro edition. This can take some time. Do note that this is an automatic process that might not select the edition you want.
Selected value '6 : Windows 11 Pro'.....
Done reading and unpacking ISO

Check the UI for further steps!!!
Target ISO location: f:\Downloads\windows_11_23h2_winutils.iso
Index chosen: '6' from 6 : Windows 11 Pro
Mounting Windows image. This may take a while.
Invoke-WPFMicrowin: F:\downloads\winutil.ps1:3775
Line |
3775 |          "WPFMicrowin" {Invoke-WPFMicrowin}
|                         ~~~~~~~~~~~~~~~~~~
| An unexpected error occurred: The volume of the specified mount path does not support reparse points.  Please
| mount to a volume that supports reparse points.
Unmounting Registry...
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
Cleaning up image...
Cleanup complete.
Unmounting image...
Dismount-WindowsImage: F:\downloads\winutil.ps1:4949
Line |
4949 |          Dismount-WindowsImage -Path "$scratchDir" -Save
|          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The request is not supported.
Exporting image into f:\Downloads\winutl-scratch\micro\sources\install2.wim
Remove old 'f:\Downloads\winutl-scratch\micro\sources\install.wim' and rename f:\Downloads\winutl-scratch\micro\sources\install2.wim
Windows image completed. Continuing with boot.wim.
Mounting boot image f:\Downloads\winutl-scratch\micro\sources\boot.wim into f:\Downloads\winutl-scratch\Scrach
Invoke-WPFMicrowin: F:\downloads\winutil.ps1:3775
Line |
3775 |          "WPFMicrowin" {Invoke-WPFMicrowin}
|                         ~~~~~~~~~~~~~~~~~~
| An unexpected error occurred: The volume of the specified mount path does not support reparse points.  Please
| mount to a volume that supports reparse points.
Unmounting Registry...
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
ERROR: The parameter is incorrect.
Unmounting image...
Dismount-WindowsImage: F:\downloads\winutil.ps1:5011
Line |
5011 |          Dismount-WindowsImage -Path "$scratchDir" -Save
|          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The request is not supported.
Creating ISO image
[INFO] Using oscdimg.exe from: C:\Users\m_a_s\AppData\Local\Temp\oscdimg.exe
OSCDIMG 2.56 CD-ROM and DVD-ROM Premastering Utility
Copyright (C) Microsoft, 1993-2012. All rights reserved.
Licensed only for producing Microsoft authorized content.
Scanning source tree (500 files in 43 directories)
Scanning source tree complete (945 files in 86 directories)
Computing directory information complete
Image file is<PHONE_NUMBER> bytes (before optimization)
Writing 945 files in 86 directories to f:\Downloads\windows_11_23h2_winutils.iso
100% complete
Storage optimization saved 4 files, 24576 bytes (0% of image)
After optimization, image file is<PHONE_NUMBER> bytes
Space saved because of embedding, sparseness or optimization = 24576
Done.

Done. ISO image is located here: f:\Downloads\windows_11_23h2_winutils.iso
Bye bye!```
Additional context
no error if dont select scratch folder

The new release 24.08.22. He fixed it yesterday

According to @SHR1SHAK the issue is fixed now, so I'm gonna close this out.
If the issue is still there for you fell free to reopen the issue :)

/close


[BUG] AttributeError: 'WindowsPath' object has no attribute 'read_text'

setuptools version
setuptools==60.7.0
Python version
3.8.10
OS
Windows 10
Additional environment information
No response
Description
There's regression in compare to previous setuptools==60.6.0  (which works)
E.g. installing latest setuptools and pip and then installing e.g. pathlib fails with following error: AttributeError: 'WindowsPath' object has no attribute 'read_text'
Expected behavior
it works
How to Reproduce

install latest pip (22.0.2)
install latest setuptools (60.7.0)
install pathlib using pip

Output
python -m pip install -U pip
(cut)
pip install -U setuptools
(cut)

(venv) C:\Temp\setuptools>pip install -U pathlib
Collecting pathlib
  Using cached pathlib-1.0.1.tar.gz (49 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [12 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 14, in <module>
        File "c:\temp\setuptools\venv\lib\site-packages\setuptools\__init__.py", line 16, in <module>
          import setuptools.version
        File "c:\temp\setuptools\venv\lib\site-packages\setuptools\version.py", line 1, in <module>
          import pkg_resources
        File "c:\temp\setuptools\venv\lib\site-packages\pkg_resources\__init__.py", line 74, in <module>
          from pkg_resources.extern.jaraco.text import (
        File "c:\temp\setuptools\venv\lib\site-packages\pkg_resources\_vendor\jaraco\text\__init__.py", line 227, in <module>
          lorem_ipsum: str = files(__name__).joinpath('Lorem ipsum.txt').read_text()
      AttributeError: 'WindowsPath' object has no attribute 'read_text'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Hi @cz6ace, that you very kcih for bringing this up.
Is this error related to the fact you are installing pathlib specifically?
Can you reproduce this error without installing an external pathlib (or any package with a dependency on it)?
pathlib is suppose to be provided by the stdlib. (I.e. it should come with the Python installation, not from a package in PyPI). Overwriting it with a 3rd party package is problematic.
I also noticed that the PyPI package has a warning about being no longer maintained: https://pypi.org/project/pathlib/. This is a sign that the API is no longer compatible with the API provided in the stdlib, and probably the origin of the error you are facing.

@abravalheri, Is it possible that a breaking change was introduced in 60.7.0? I haven't figured out how to work around the issue yet. I only encountered this issue today after 60.7.0 was released. I think I'm going to try re-creating the issue with a fresh environment and see if the issue still exists. There are only 2 reports on all of github with this stacktrace so I suppose it could be my environment.

Hi @parthea every release has a chance of breaking something for someone, so I don't discard the possibility, but the way this particular issue is described sounds like the problem is caused by a replacement of a module in the stdlib with a non-API compliant 3rdParty.
Either that or the Python docs are extremely misleading, since https://docs.python.org/3/library/pathlib.html seems to indicate read_text should be present (which I find very unlikely)
If anyone manage to create a reproduction without installing pathlib from PyPI (or a library that depends directly or indirectly from it), we can proceed with the investigation.

@parthea, for the problems you are facing, can you check if anyhow pathlib gets pulled from PyPI instead of being used from the stdlib?
If that is not the case, I think the best is to consider it has a different origin from the issue described here and open another ticket with a minimal reproduction.

No one should be using pathlib. It was abandoned years ago and it masks the stdlib module and it doesn't even install on Python 3.10. If you need backward compatibility for pathlib, you should use pathlib2.
The reason the issue triggered with Setuptools 60.7 was because this version vendors jaraco.text, which utilizes importlib.resources/importlib_resources, which utilizes pathlib.
I apologize for the incompatibility. Setuptools could work around the issue, but it would be preferable for users to fix the root issue that pypi:pathlib is deprecated and should not be used at all.

I'm using the built-in pathlib so it is a different issue.

Hi @cz6ace, that you very much for bringing this up.
Is this error related to the fact you are installing pathlib specifically?

We have the pathlib as a dependency, probably from hexrec package. Now I can see the most recent version got rid of it: "Removed dependency of legacy pathlib package; using Python’s own module instead."

Can you reproduce this error without installing an external pathlib (or any package with a dependency on it)?

No, just with pathlib
At the end - we can easily fix to upgrade the hexrec package which do not rely on obsolete pathlib any more.

No one should be using pathlib.

This ticket can be closed if I understand it correctly.


Catching terminal shortcuts

When I was trying the console, I noticed that it doesn't catch some bash shortcuts. For example:
Ctrl+L for clearing the screen
Alt+B for going to previous word in current command
Is it possible for these shortcuts to be catch when the focus is on the console?

Hi @gessha, do you mean providing those shortcuts in the Jupyter Console, or are they not working for you in the terminal emulator?

Hi @blink1073, I meant the Jupyter Console.
Btw, I haven't tried this outside of Chrome.

While we may over time be able to implement some of these, it's important to clarify that the console is not a terminal emulator.  It's much closer to an "append-only notebook without markdown"... The text editing experience should be identical to that of the notebook, the difference being that up-arrow retrieves the previous "cell" for re-execution rather than moving up, as there is no document to be edited.
I can see the case for clearing the screen, as that one already works in the qt console and is useful (we could even consider having a way to fully clear the console, not just moving the prompt to the top but discarding all previous entries).
So my suggestion is:  Ctrl-L for a basic clear-screen is in-scope for 1.0, Alt-B and full vt-100 terminal control characters are out of scope.  For text-editing, we'll just reuse the same machinery in the console as in the notebook and have it offer parity with that (after all, that's the point of using the same tools, to get benefits of code reuse :)
Does that sound OK?

Yeah, it sounds great.
Thank you for working on this awesome project! I'm using it on a daily basis and I've also introduced it to a couple of my friends and we all love it.

I'd like to add that this behavior should be exposed as commands for the console, so that users are free to customize the shortcuts via the shortcut registry. We can of course, provide a default shortcut for the time being.


Update NoSQL Dockerfile to use CE_4.3

Oracle NoSQL Database 4.3 (Community Edition) is now released. It would be useful to have the Dockerfile included in this repo for the latest version. Some of the key changes from docker perspective include -

The community edition is now licensed under Apache License v2.0
oracle nosql database is now secure by default. This affects how the kvlite instance can be started within the container.

I created a patch to add the Dockerfile with necessary changes to the Readme. Please review and let me know your feedback.
kv4_3_patch.TXT

Please submit this as a proper git pull request and not as an uploaded text file. We can then better review and test the submission.

Thanks @Djelibeybi
I tried that and get this,
$ git push --set-upstream origin kv4311
ERROR: Permission to oracle/docker-images.git denied to nirmay.
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
Do I need to obtain access? I am already part of Oracle org on github.

Please do not attempt to push to this repository. You need to fork this repo, make your changes and then submit a pull request:
https://help.github.com/articles/fork-a-repo/
https://help.github.com/articles/creating-a-pull-request-from-a-fork/

ah, thanks @Djelibeybi
have created a pull request. do let me know your feedback.

PR merged. Thanks!

Thanks @Djelibeybi for all your support!
Mayuresh


This seems like a licensing issue?

Probably. I guess it could be avoided by just renaming it to something like
"Martin Davidez", and the description as
"The Middlewalker legend. Remembered by a drink, forgotten by a drunk."

This seems like a licensing issue?

I thought you couldn't copyright a name

Recipes, cocktail or otherwise, cannot, as recipes are considered “facts” and a fact cannot be copyright protected

Names are not copyright protected and the sprite would likely fall under fair use as it is a remix of my own creation with enough difference to clearly be its own remix or spinoff

Names aren't but fictional characters are treated as a component of a copyrighted work. For example you cannot use the name Sherlock Holmes even though it's just a name. Though in this case it's more of a reference which is allowed. CDPR might never find out it was used, they might like it or they might sue. Who knows?

wouldn't they just send a cease and desist in which case we just remove it from the game?

wouldn't they just send a cease and desist in which case we just remove it from the game?

We're opting to license everything upstream rather than risking any sort of DMCA.

Almost done fixing stuff.  Just need to rename a file and test it, then we should be good to go.

@Shadowtheprotogen546 Merge master & fix tests/conflicts then I can merge

@Shadowtheprotogen546 Merge master & fix tests/conflicts then I can merge

yeah, sorry.  life's been busy.  I should be able to get it done this weekend tho.


Add a dependabot config

Dependabot will automatically file PRs for updatable dependencies. The
cargo section watches Cargo.toml. Patch releases are ignored since those
are semver compatible and cargo downloads the latest anyway.
The github-actions section watches the workflow files in .github/workflows
for possible updates to any of the Actions depended upon.
Kindly merge https://github.com/eminence/terminal-size/pull/39 first to avoid conflicts.

I'm new to dependabot configs, but this LGTM.  Thanks


unable to register core dump handler

Please paste the output of the following command here: sudo edit-chroot -all

chronos@localhost / $ sudo edit-chroot -all
name: sid
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/sid...
WARNING: Unable to register core dump handler.
crouton: version 1-20190902190258~master:9a594a66
release: sid
architecture: amd64
xmethod: xiwi
targets: xiwi,xorg
host: version 12371.46.0 (Official Build) beta-channel samus
kernel: Linux localhost 3.14.0 #1 SMP PREEMPT Tue Sep 3 00:54:58 PDT 2019 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/sid...
chronos@localhost / $
Please describe your issue:
During install:
sudo install -Dt /usr/local/bin -m 755 ~/Downloads/crouton
sh: 0: /tmp/crouton-installer-cache/crouton.Dk0/host-bin/crash_reporter_wrapper: warning: script from noexec mount; see https://chromium.googlesource.com/chromiumos/docs/+/master/security/noexec_shell_scripts.md
WARNING: Unable to register core dump handler.
If known, describe the steps to reproduce the issue:
xiwi blender&
fails - maybe related to the core dump issue?
Tried reinstalling crouton, tried updating sid, tried removing xorg from the target.
Any other suggestions?!

I have similar issues
chronos@localhost / $ sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
WARNING: Unable to register core dump handler.
crouton: version 1-20190120145639~master:5bf96711
release: xenial
architecture: amd64
xmethod: xiwi
targets: xiwi,gnome,extension,core,cli-extra
host: version 12371.41.0 (Official Build) beta-channel auron_yuna
kernel: Linux localhost 3.14.0 #1 SMP PREEMPT Thu Aug 29 00:55:12 PDT 2019 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

yeah, I gave up with crouton and went for chrx instead. There's no chance
of getting a graphics tablet working under crouton and it works out of the
box with chrx.
On Thu, 19 Sep 2019 at 17:37, qwertpi<EMAIL_ADDRESS>wrote:

I have similar issues
chronos@localhost / $ sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
WARNING: Unable to register core dump handler.
crouton: version 1-20190120145639~master:5bf96711
release: xenial
architecture: amd64
xmethod: xiwi
targets: xiwi,gnome,extension,core,cli-extra
host: version 12371.41.0 (Official Build) beta-channel auron_yuna
kernel: Linux localhost 3.14.0 #1 SMP PREEMPT Thu Aug 29 00:55:12 PDT 2019 x86_64 x86_64 x86_64 GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnschneid/crouton/issues/4135?email_source=notifications&email_token=ACTAOZUITQPOSDPQGENKA53QKO2G5A5CNFSM4IVHSTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7EIDFI#issuecomment-533234069,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACTAOZVBWF2W2H42XTZOCQLQKO2G5ANCNFSM4IVHSTJA
.

--
Dr. Kester Clegg
Department of Computer Science
University of York
York YO10 5GH
Mob. +44 (0)75 3815 3945
Work +44 (0)1904 32 5461
Email Disclaimer http://www.york.ac.uk/docs/disclaimer/email.htm

I don’t have a graphics tablet and have been happily using crouton for about 2 years

Same here. Crouton is great until you need some hardware support via a
kernel module. Then you're stuck.
Crouton also seems to have gone very quiet recently...
On Fri, 20 Sep 2019 at 08:17, qwertpi<EMAIL_ADDRESS>wrote:

I don’t have a graphics tablet and have been happily using crouton for
about 2 years
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnschneid/crouton/issues/4135?email_source=notifications&email_token=ACTAOZUOSGDLYR3U4FPNQYDQKR2JHA5CNFSM4IVHSTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7F2F7Q#issuecomment-533439230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACTAOZRMBOMJXXYGVOXSGILQKR2JHANCNFSM4IVHSTJA
.

--
Dr. Kester Clegg
Department of Computer Science
University of York
York YO10 5GH
Mob. +44 (0)75 3815 3945
Work +44 (0)1904 32 5461
Email Disclaimer http://www.york.ac.uk/docs/disclaimer/email.htm

For me it broke as soon as I upgrade chromebook to version 77.Nothing else changed.
this also breaks the crouton extension which never comes up which in turn breaks any X-based apps.

ok got this all working.
So the culprit of xiwi not working is the crouton extension.
Do the following.
1>Remove the existing crouton integration extension from chrome.
2>Reinstall it from https://chrome.google.com/webstore/detail/crouton-integration/gcpneefbbnfalgjniomfjknbcgkbijom
3>Retry xiwi or any other x-windows operation it will work fine.

yes, I found I had to disable and re-enable the extension. It stops working
after a while - after a reboot?
On Wed, 25 Sep 2019 at 16:18, patelbhavesh<EMAIL_ADDRESS>wrote:

ok got this all working.
So the culprit of xiwi not working is the crouton extension.
Do the following.
1>Remove the existing crouton integration extension from chrome.
2>Reinstall it from
https://chrome.google.com/webstore/detail/crouton-integration/gcpneefbbnfalgjniomfjknbcgkbijom
3>Retry xiwi or any other x-windows operation it will work fine.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnschneid/crouton/issues/4135?email_source=notifications&email_token=ACTAOZXWO7UEAGIKLPITL6LQLN6LBA5CNFSM4IVHSTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7SJBSI#issuecomment-535072969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACTAOZRXO2MZPOKMFK6KMC3QLN6LBANCNFSM4IVHSTJA
.

--
Dr. Kester Clegg
Department of Computer Science
University of York
York YO10 5GH
Mob. +44 (0)75 3815 3945
Work +44 (0)1904 32 5461
Email Disclaimer http://www.york.ac.uk/docs/disclaimer/email.htm

Yes you are right .after reboot it stopped working for me as well .

I am still getting this after I've removed and reinstalled the crouton integration extension and rebooted my machine. I've also deleted my chroots and attempted to reinstall a target.

Yeah, think it's something to do with Chrome's updates locking down parts of the kernel or filesystem or something. Reinstalling / deleting chroots won't help.

@biscuitlad I am trying chrx as you suggested above.

For myself on a Chromebook Pixel 2015 it's been great, very well supported.
Few annoyances for GalliumOS, such as no docs, resolution of login screen,
etc. It would be nice to be able to switch from the Pixel screen to my 4K
monitor with a script, but all of that is of minor importance.
The main thing is that my graphics tablet works for krita and blender!
Sadly it'll never work under crouton because of the Pixel's ancient kernel

I couldn't find the right headers to compile the modules. chrx just
worked out of the box once you burn the firmware from Mr Chromebook. Been
very good so far.

On Tue, 8 Oct 2019 at 10:28, Adrian Lewis<EMAIL_ADDRESS>wrote:

@biscuitlad https://github.com/biscuitlad I am trying chrx as you
suggested above.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dnschneid/crouton/issues/4135?email_source=notifications&email_token=ACTAOZXPW2VUUWGGDT6P5QDQNRHEZA5CNFSM4IVHSTJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEATREMQ#issuecomment-539431474,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACTAOZXSEGTHJJFF3PD2WYDQNRHEZANCNFSM4IVHSTJA
.

--
Dr. Kester Clegg
Department of Computer Science
University of York
York YO10 5GH
Mob. +44 (0)75 3815 3945
Work +44 (0)1904 32 5461
Email Disclaimer http://www.york.ac.uk/docs/disclaimer/email.htm

I didn't manage to boot GalliumOS through chrx. Used the Chrome recovery utility to delete the partition and I will return back to Crostini.

For me the following works 90% of the time.
Click on the big C icon on the right hand top corner.Right click on the C icon for Crouton integration.
Click on Manage extensions.Disable extension.wait a few seconds.Reenable it.Start your xiwi apps etc.Rinse/repeat as required.

For me I have to use the disable and then enable button within the crouton helper extension after reboots but other than that it works but does come up with this warning every time I run start-gnome or enter-chroot

latest upgrade to version 78 fixes the issue where in crouton extension had to be disabled/enabled periodically.xiwi works reliably hand in hand with the crouton extension.

I have not got the extension installed as a target or in the chrome browser.
crouton -V
crouton: version 1-20190902190258~master:9a594a66
sudo crouton -r sid -t cli-extra
....
Pruning /mnt/stateful_partition/crouton/chroots/sid mounts...
WARNING: Unable to register core dump handler.

I have neither the extension installed as a target or in the Chrome browser.
crouton: version 1-20190902190258~master:9a594a66
sudo crouton -r sid -t cli-extra
....
$ sudo startcli
Entering /mnt/stateful_partition/crouton/chroots/sid...
WARNING: Unable to register core dump handler

Is this a Chrome OS update issue? Or we talking about two different things?

I'm also having this issue, alongside all my applications being incredibly slow all of a sudden.


Contradicting wiki documentation on ArduCopter 4.0 upgrade for IMX firmware upgrade

The following page:
https://github.com/OpenSolo/OpenSolo/wiki/Install-via-SSH
gave instructions for upgrading both firmware and ArduCopter to the latest versions. One paragraph in it stated:

If you are upgrading a stock cube that still has the old 3DR 1.3.2 or 1.5.4 ArduCopter firmware, you need to install ArduCopter 4.0 first! The old firmware is so old, it will not play nicely during the full IMX update, so we need to do it separately first. Visit the ArduCopter Firmware for instructions.

Unfortunately, the link leads to a page where the phrase "ArduCopter 4.0" or even the version number "4.0" DOES NOT exist. In addition, The page also stated that ArduCopter-Solo v1.5.4 is already the latest version:

Open Solo 3.0.0 Release contains binaries specifically for the solo as used in Open Solo 3.0.0, with all the default parameters baked in. This includes ArduCopter-Solo v1.5.4 for the stock cube, and 3.5.4 for the Green Cube.

May I suspect that the link is now obsolete, and a new, more detailed paragraph is required for the intended "ArduCopter 4.0" upgrade?

PRs welcome

I'm not sure if GitHub support forking of wiki. So I'll just post my intended change here, if you have write permission to the wiki, could you help me reviewing and paste it?
Original:
----
=== ArduCopter 3.7 (''Development Beta'')===
'''ArduCopter 3.7 will be fully compatible with all Solos, whether you are totally stock or green cubed.'''  There will no longer be a requirement for different firmware versions.  All users will get all features and benefits of the latest and greatest ArduCopter firmware.

Beta testing of ArduCopter 3.7 is currently in progress in the [https://www.facebook.com/groups/617648671719759 Solo Beta Test Facebook Group.]

''Note: Older versions of ArduCopter used old NuttX RTOS based firmware and a *.px4 file extension.  This includes the ArduCopter versions included in Open Solo 3 and from 3DR.  ArduCopter 3.7 uses the new ChibiOS RTOS. The firmware files have a file extension of *.apj.  Due to the file extension change, the firmware will not load without some additional file changes.  The beta testing packages in Solex and SidePilot include this''

The upcoming Open Solo 4 will include ArduCopter 3.7 fully compatible with all Solos, marking a huge milestone for the Solo and Open Solo community.
----

Changed:
----
=== ArduCopter 4.0+ (''Formerly ArduCopter 3.7-dev'')===
'''ArduCopter 4.0.x will be fully compatible with all Solos, whether you are totally stock or green cubed.'''  There will no longer be a requirement for different firmware versions.  All users will get all features and benefits of the latest and greatest ArduCopter firmware.

Beta testing of ArduCopter 4.0.x/3.7-dev is currently in progress in the [https://www.facebook.com/groups/617648671719759 Solo Beta Test Facebook Group.]

''Note: Older versions of ArduCopter used old NuttX RTOS based firmware and a *.px4 file extension.  This includes the ArduCopter versions included in Open Solo 3 and from 3DR.  ArduCopter 4.0.x uses the new ChibiOS RTOS. The firmware files have a file extension of *.apj.  Due to the file extension change, the firmware will not load without some additional file changes.  The beta testing packages in Solex and SidePilot include this''

The Open Solo 4 will include ArduCopter 4.0.x fully compatible with all Solos, marking a huge milestone for the Solo and Open Solo community.

In most cases with Open Solo 4 and/or Solex/SidePilot, installation of ArduCopter 4.0.x will be fully automatic. Manual SSH-based upgrade to ArduCopter 4.0.x is only necessary if you are upgrading a stock cube that still has the old 3DR 1.3.2 or 1.5.4 ArduCopter firmware.
----

Original:
* [https://github.com/Pedals2Paddles/OpenSolo/releases/tag/OpenSolo-3.0.0 Open Solo 3.0.0 Release] contains binaries specifically for the solo as used in Open Solo 3.0.0, with all the default parameters baked in. This includes ArduCopter-Solo v1.5.4 for the stock cube, and 3.5.4 for the Green Cube.

Changed:
* [https://github.com/OpenSolo/OpenSolo/releases/tag/OpenSolo-4.0.0 Open Solo 4 Release] contains binaries specifically for the solo as used in Open Solo 4, with all the default parameters baked in. This includes ArduCopter-Solo v4.0.0 for all versions CubePilot.
* [https://github.com/OpenSolo/OpenSolo/releases/tag/OpenSolo-3.0.0 Open Solo 3.0.0 Release] contains binaries specifically for the solo as used in Open Solo 3.0.0, with all the default parameters baked in. This includes ArduCopter-Solo v1.5.4 for the stock cube, and 3.5.4 for the Green Cube.

No idea how that link to a forked repo skipped all the eyeballs for so long

Also, why this wiki doesn't contain info regarding pixhawk.py and uploader.py?
https://3drpilots.com/threads/update-1-3-1-to-open-solo-4-dev-via-ssh-sftp.15379/

the documentation is trivial to fix....   open a PR, get it reviewed, done.

Gents,
Have been absent from Drones for a couple of years, only to discover just how impressive you guys have been, regarding keeping all of our Solos well-clear from destination 'Landfill'.
Dramas encountered when blowing the dust off of a bone-stock, factory original Solo and Controller, and attempting an update to OpenSolo via SSH - when following the instructions located currently up on the Wiki.
Opened a topic over on ArduPilot - ( see here ) - but thought it wise to seek counsel from the legends themselves, as well, eh?
Short-form of the dramas encountered, as follows ;

Instructions currently on the Wiki for SSH-upgrade, ( see here ), present the first caveat -
'If you are upgrading a stock cube that still has the old 3DR 1.3.2 or 1.5.4 ArduCopter firmware, you need to install ArduCopter 4.0 first! The old firmware is so old, it will not play nicely during the full IMX update, so we need to do it separately first. Visit the ArduCopter Firmware for instructions.'

Moving across to the ArduCopter Firmware Page, reveals the problem at hand -
'Note: Older versions of ArduCopter used old NuttX RTOS based firmware and a *.px4 file extension. This includes the ArduCopter versions included in Open Solo 3 and from 3DR. ArduCopter 3.7 uses the new ChibiOS RTOS. The firmware files have a file extension of *.apj. Due to the file extension change, the firmware will not load without some additional file changes. The beta testing packages in Solex and SidePilot include this'

Any attempt to simply SSH the provided OpenSolo 'arducopter.apj' firmware into the prescribed location - inside a bone-stock, factory original Solo - results with the Solo simply deleting the file upon reboot...
Would love to know what the 'additional file changes' are, ( which Solex and SidePilot use to otherwise perform this upgrade ), and would be thrilled to update the Wiki with this information to assist others.
Warmest Regards,
Solo-Loco
( and Matt, I'm THRILLED to see that you're still here too! )

Yep,
Just found Matt's conversation, concerning this very issue, over on 3drpilots.com - ( see here )
Carefully read through the entire 'revised' process, and Boom-goes-the-Dynamite - my Solo and associated Controller are now fully OpenSolo-4-o-fied!

In any case, I'm going to submit for consideration the revision for the Wiki via a PR, as I feel that it's the least I can do.
Many thanks -
Solo-Loco.
(c;=

Please open a PR when u have improvement/s


whitespace use - resize columns

right now there's significant whitespace on pages like 'advanced settings'. can we make these columns resizable?

same for identities

The styles that are available here are not averse to size changes.. Would need a rewrite, will put off until we build a mobile view.


Removes Basic Auth for static files, enables SpA route handling

The changes are:

Removed Basic Auth for static files, allowing the SpA to handle authentication & routes protection. Addressed by fce2d93d01a3b452e9cab2b1f8bc192aab6fb6c0

Allow SpA to resolve routes. cfaae9c9988f35a7d083809effde5c4140cc0a8a

Aims to provide support for https://github.com/turing-machines/BMC-UI/pull/5

Regarding https://github.com/turing-machines/bmcd/commit/cfaae9c9988f35a7d083809effde5c4140cc0a8a, I'm not entirely sure this is the optimal approach to solve it. I have several examples:
https://github.com/rishadbaniya/Actix-web-SPA-react-js-example/blob/master/src/main.rs
https://github.com/calvinlauyh/rust-actix-angular-example/blob/master/actix-backend/src/main.rs
I honestly fought with this because the project dependencies are incompatible with macOS, so I tested it by triggering a complete build via BMC-Firmware, allowing me to iterate every hour. I got more fails than I'm willing to tell 😆 I'm just happy by having it do the expected behavior.
Could you give it a look? I'm mainly concerned about whether it's actually required to read the config every time that function is called.


SilverStripe security advisories should come from the official RSS feed

At the moment, this package bakes-in YML files comprising official security advisories for SilverStripe (and a bunch of other applications). This seems a pretty high-maintenance procedure for a kind of data that is by its very nature, constantly evolving.
Is there a reason why the package doesn't make use of web-APIs and RSS feeds where available for each of the supported packages? The SilverStripe one is here: https://www.silverstripe.org/download/security-releases/rss

web-APIs and RSS feeds where available

Those feeds are designed for human consumption, not for machine consumption.

I beg to differ. In a few lines of Python, I can parse that feed and compare with a project's composer.lock.
#!/usr/bin/python
#
# Russell Michell 2018<EMAIL_ADDRESS>#
# What is this?
#
# Parses the SilverStripe Security Releases RSS feed for vulnerabilities in the current project's
# composer.lock file.
#
# Requirements:
#
# Feedparser:    pip install feedparser
# BeautifulSoup: apt-get install python-bs4

from bs4 import BeautifulSoup
import json, sys, feedparser, re

feed_url = 'https://www.silverstripe.org/download/security-releases/rss'

# We use this pseudo constant when no package is given in the RSS advisories
DEFAULT_PACKAGE = 'silverstripe/framework'

# Clean up version constraints
def clean_version(input):
    return re.sub('[^[\d.]', '', input)

# Cleanup package-names
def clean_package(input):
    return input.strip(' :')

with open('./composer.lock', 'r') as composerLockFile:
    json = json.load(composerLockFile)

    feed = feedparser.parse(feed_url)

    if feed.channel == None:
        print 'No feed data found.'
        sys.exit(1)

    for package in json['packages']:
        c_package = package['name']
        c_version = clean_version(package['version'])

        for item in feed['items']:
            soup = BeautifulSoup(item['description'], 'html.parser')
            # All entity-encoded <dd> HTML elements found within an RSS <description> element
            allDDs = soup.find_all('dd')
            # TODO Can also use BS4 to access via class="foo" rather than as a dict/numeric-index
            severity = allDDs[0].string
            identifier = allDDs[1].string
            versionsAffected = allDDs[2].string
            versionsFixed = allDDs[3].string
            advisory_link = item.link

            # Advisories sometimes exclude the package name. Assume this means "silverstripe/framework"
            # Advisories are formatted slightly differently (with/without colon-separated spaces between <package><version>)
            package = re.sub("[\s:]?((>=?)?\s?)?(\d\.?)+(-rc\d)?,?", '', versionsFixed)
            version = re.sub("([^\/]\w+\/[\w-]+(?=[\s:]))+", '', versionsFixed)
            f_package = clean_package(package)
            f_version = clean_version(version)

            # Basic test to see if a package has been stipulated. If not, we assume: "silverstripe/framework"
            if f_package is None or '/' not in f_package:
                f_package = DEFAULT_PACKAGE

            if c_package != f_package:
                continue

            for fixed_version in f_version.split(','):
                is_vulnerable = c_version < fixed_version

                if is_vulnerable:
                    print '[ALERT] %s version %s has a security advisory: %s' % (c_package, c_version, advisory_link)
                    # Break so no further versions need to be checked
                    break
                    sys.exit(1)

Yes, but there is no guarantee from the publisher that the syntax stays
stable. It needs to be an official source, with stable DSL. This is what
PSR-9 is about, if you are interested in designing a proper XSD/DTD for
security-related ATOM feeds.
On Thu, 6 Dec 2018, 00:14 Russ Michell<EMAIL_ADDRESS>wrote:

I beg to differ. In a few lines of Python, I can parse that feed and
compare with a project's composer.lock.
#!/usr/bin/python

Russell Michell 2018<EMAIL_ADDRESS>
What is this?

Parses the SilverStripe Security Releases RSS feed for vulnerabilities in the current project's
composer.lock file.

Requirements:

Feedparser:    pip install feedparser
BeautifulSoup: apt-get install python-bs4
from bs4 import BeautifulSoup
import json, sys, feedparser, re
feed_url = 'https://www.silverstripe.org/download/security-releases/rss'
We use this pseudo constant when no package is given in the RSS advisories
DEFAULT_PACKAGE = 'silverstripe/framework'
Clean up version constraints
def clean_version(input):
return re.sub('[^[\d.]', '', input)
Cleanup package-names
def clean_package(input):
return input.strip(' :')
with open('./composer.lock', 'r') as composerLockFile:
json = json.load(composerLockFile)
feed = feedparser.parse(feed_url)

if feed.channel == None:
    print 'No feed data found.'
    sys.exit(1)

for package in json['packages']:
    c_package = package['name']
    c_version = clean_version(package['version'])

    for item in feed['items']:
        soup = BeautifulSoup(item['description'], 'html.parser')
        # All entity-encoded <dd> HTML elements found within an RSS <description> element
        allDDs = soup.find_all('dd')
        # TODO Can also use BS4 to access via class="foo" rather than as a dict/numeric-index
        severity = allDDs[0].string
        identifier = allDDs[1].string
        versionsAffected = allDDs[2].string
        versionsFixed = allDDs[3].string
        advisory_link = item.link

        # Advisories sometimes exclude the package name. Assume this means "silverstripe/framework"
        # Advisories are formatted slightly differently (with/without colon-separated spaces between <package><version>)
        package = re.sub("[\s:]?((>=?)?\s?)?(\d\.?)+(-rc\d)?,?", '', versionsFixed)
        version = re.sub("([^\/]\w+\/[\w-]+(?=[\s:]))+", '', versionsFixed)
        f_package = clean_package(package)
        f_version = clean_version(version)

        # Basic test to see if a package has been stipulated. If not, we assume: "silverstripe/framework"
        if f_package is None or '/' not in f_package:
            f_package = DEFAULT_PACKAGE

        if c_package != f_package:
            continue

        for fixed_version in f_version.split(','):
            is_vulnerable = c_version < fixed_version

            if is_vulnerable:
                print '[ALERT] %s version %s has a security advisory: %s' % (c_package, c_version, advisory_link)
                # Break so no further versions need to be checked
                break
                sys.exit(1)

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/FriendsOfPHP/security-advisories/issues/340#issuecomment-444687719,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakHYpfzWVGAXc9gi2Yc6Ne1CnWx2Fks5u2FM6gaJpZM4ZDoe9
.

I'm going to close this one as we don't have any resources to maintain such a script and it is not our responsibility to create entries for all PHP projects out there. It's a contribution based effort from the community. Now, if the Silverstripe community wants to automate the process of submitting their advisories info, they can do so of course.


Bottom border doesn't show on tabs

Looking at the CSS there should be a bottom border outlining inactive tabs, this is current look:

Making tab element's height 36px instead of 35px displays it:

Great catch.


Add seconds behavior to the Time module

It's defined in the tests.

@paf31 I think this one can be closed now :D


As an analyst, I want the @Elapsed moniker to be accessible as an Analyzer & Insight, to provide greater flexibility.

Background
The little known @Elapsed moniker has been available for quite some time. It is useful for identifying gaps in a timeline.
￼Acceptance Criteria

[ ] Analyzer: takes threshold as input from user, and publishes results in record Comment
[ ] Insight: add to Common library so that plug-ins can include it in their dashboard

Relates to: #126

Implementation Details
It turns out that this feature already existed in a slightly different form:

time gaps could be identified using the @Elapsed>3000 moniker

in this case 3000 represents a time period measured in milliseconds.

the built-in UI Responsiveness Analyzer was already comparing log entries generated by the UI thread (ThreadId=1) looking for periods of time where logging stopped

So a couple of small changes where made:

there is now a more generic implementation of the old UiResponsivenessAnalyzer called: TimeGapAnalyzer
Weevil's menu now has the following options:

Detect Time Gap
Detect Time Gap (UI Only)

It is worth noting that:

the TimeGapAnalyzer will write the elapsed time into the record's comment
whereas, the user still has the option to use the @Elapsed moniker if they do not wish to generate any comments


cloudflare_tunnel_route is not removed when contains virtual_network_id

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Cloudflare plugin: v3.20.0
Affected resource(s)
cloudflare_tunnel_route
Terraform configuration files
I'm using Pulumi wrapper. But the idea is
resource "cloudflare_argo_tunnel" "tunnel" {
	account_id = "accid"
	name       = "example"
	secret     = "AQIDBAUGBwgBAgMEBQYHCAECAwQFBgcIAQIDBAUGBwg="
}

resource "cloudflare_tunnel_route" "route" {
    account_id = "accid"
    tunnel_id = cloudflare_argo_tunnel.tunnel.id
    network = "<IP_ADDRESS>/24"
    virtual_network_id = "vnetidhere"
    comment = "Test"
}

Debug output

Panic output
No response
Expected output
When tunnel route is removed via terraform then tunnel route is removed from Cloudflare.
Actual output
Result is displayed as successful but tunnel route is still present in Cloudflare after removal via terraform
Steps to reproduce

Create tunnel route with vnet with TF
Remove tunnel route with TF
Check that tunnel route is still present.

Additional factoids
No response
References
No response

Did not check but should be fixed now.


Removed outdated information about creating PRs for stable branches.

All PRs should be against master.
Any backport will be performed by the merger.

This information is not outdated (see comment).


Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/db-for-postgre-sql/flexible-server
Module Details
No response
Do you want to be the new owner of this module?
No
Module Owner's GitHub Username (handle)
arnoldna
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Hi @arnoldna,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"

Hi @arnoldna,
Thanks for confirming that you wish to own this AVM module and understand the related requirements and responsibilities!
We just want to ask you to double check a few important things.
Please use the following values explicitly as provided in the module index page:

You must be the owner of the GitHub teams as outlined here.
Please check that your name has been updated in the module index page (this should happen shortly after you confirmed ownership).

You can now take ownership of this module and start improving it as needed! ✅ Happy coding! 🎉
Any further questions or clarifications needed, let us know!
Thanks,
The AVM Core Team


Errors

there are lot of errors in this version kindly Update this Repo....

there are lot of errors in this version kindly Update this Repo....

We are currently working on releasing a full stable version of this repository soon, where all the bugs will be fixed.


Refreshing data and keeping it in the original display position?

Hi @evant :
Whether it supports refreshing data and keeping it in the original display position?
When new data arrives, it is processed as "clear" and then "added," resulting in a change in the scrolling position. How can i resolve it.

This is handled by diffUtil, check out docs to how to implement it correctly.

Oh, thanks, I didn't notice this in the document.


fix for provisioning extensibility provider Enabled

Provisioning engine ignores Enabled attribute for extensibility providers

Nice catch Ivan! I'll merge this one


[ROAD-657] Fix: sentry invalid auth token provided on sast requests

Changes in this PR will

Check is token valid on send sast request.
If token not valid or empty in settings > return null result.
If token not valid or empty in settings > disable Run and Stop buttons in Snyk tool window
If token not valid or empty in settings > show Welcome view.

@michelkaporin could you please take a look?


Create Chinese_remainder_theorem

plss accept my  pr request for hactoberfest.

plz assign it to me...

pls star the repo

pls star the repo

could u merged it as it is not accepted yet in hactober


Improving and Fixing the Tests

Tests for the following functionality is missing:

Image encoding
State switching (depth, view etc)
SceneManager

There are some weird race conditions (?) in the tests.

Sometimes the "composite font" atlas will be missing glyphs.
The "CurrentTrack" towards the end of the PlaylistLogic AudioTest is sometimes null.

These are fixed

There are some weird race conditions (?) in the tests.

Sometimes the "composite font" atlas will be missing glyphs.
The "CurrentTrack" towards the end of the PlaylistLogic AudioTest is sometimes null.

These are fixed


Cleanup and fix typos in comments

Wrap comments at 80 cols, add missing periods and fix typos.

Ok for a merge @smyrman?

Yeah.


Add option for disabling tab row

Is your feature request related to a problem? Please describe.
I'm really used to switching between files with the cmd+p shortcut on both vscode and goland, so the tab row is redundant for me and it's just wasting space
Describe the solution you'd like
add an option for hiding the tab bar completely
as the breadcrumbs already include the filename there's no need for any other indicator for what file is being used, at least for me
Describe alternatives you've considered
no alternatives, is not something that has many choices I think 😅
Screenshots
I'm attaching screenshots of both my vscode and goland windows, where I have tabs hidden

Here's Emacs


[BUG] Stale Element Exception during Login

Bug Report
EasyRepro Version

[ ] Microsoft Dynamics 365 Online Version 9.1 (9.1.x) (DB 9.1.x) online

UCI or Classic Web

[ ] UCI (XrmApp)

Online or On Premise

[ ] Online

Browser

[ ] Chrome

Describe the bug
stale element reference: element is not attached to the page document
at Microsoft.Dynamics365.UIAutomation.Api.UCI.WebClient.Login(Uri orgUri, SecureString username, SecureString password, SecureString mfaSecretKey, Action`1 redirectAction)
xrmApp.OnlineLogin.Login(_xrmUri, _username, _password);

Expected behavior
Should Login without issues

Apart from Stale Element, we are not using MFA, and seems like sometimes we are getting the error:
"Something went wrong entering the OTC. Please check the MFA-SecretKey in configuration."
Can we make a validation if MFA is null, then it should not wait for anything to set the MFA.


Last Discussed on Stack Overflow Badge Proposal

:clipboard: Description
This badge is for repositories hosted on Github to demonstrate if the API/Service/Library etc. has
recent community support. Often times I'm picking a library that is outdated or
suggests that many contributors are not following anymore.
A dynamic badge to show the last date of the library has been talked about on a popular Q&A site, Stack Overflow would be helpful!
An example would be as below,

:link: Data
We can get the data via the StackExchange API using the StackOverflow parameters.
This does require an application key that can be thrown in together with the API url
for example, https://api.stackexchange.com/2.2/questions?order=desc&sort=creation&site=stackoverflow&tagged=SOMELIBNAME&key=apikey
Documentation: https://api.stackexchange.com/docs
I am also happy to try implementing this badge and make a PR to Shields!

Nice idea!
Other possibilities: questions in the last year, or questions in the last month. I think those figures would convey a better sense of the amount of activity, because the metric would be less sensitive to a single outlier question. Let's discuss and pin down the design.
Would be great to have your help to implement this! Our stackexchange badge is written using our legacy API and needs to be ported to a BaseJsonService subclass. The service also needs tests; it's one of relatively few at this point which doesn't have them. If you're up for it, what I'd suggest is doing a port of the existing functionality first, and adding the new badge in round 2.

@paulmelnikow Those are also good suggestions especially with your point regarding the outliers.
Would something like this design be more suitable?

In terms of figuring out what to do next to make sure i'm understanding correctly

Port existing Stack Exchange badge into the new BaseJsonService subclass
Write new tests for said badge above
Add new proposed badge?

Or a bit more concisely:

Yea! That's a good course of action. The best place to start is the tests – you can write tests for the old badge first, and we can even get them reviewed in a separate PR. It gives you something to test the rewrite against. And then, yea! You can add the new proposed badge in the last step.

Having some issues while porting StackExchange legacy to BaseJson and posting it here!
Writing the StackExchange endpoint handle function, although it is runnable i'm unable to get the data returned from this.__requestJson and even tried printing it right after but doesn't print anything to the console.
The API url is correct, my query parameters are fine. Not sure if my problems lies within the schema or elsewhere and am open to suggestions of where to look next.
In summary,
expected: requestJson return data should be some object containing numbers
actual: returning nothing? Unable to determine also the type of data response

This badge image is what I get overall

unparseable json response means you're getting back an HTTP response that isn't JSON. It suggests the endpoint isn't quite right.
You can use npm run test:services:trace or TRACE_SERVICES=true npm start and you'll get copious debugging output from BaseService. I bet that will help you find the error.
Another suggestion: start with a loose schema (Joi.any()), and add the validation once you get the basic thing working. 😀

You might have already worked it out, but I suspect you are hitting this: https://meta.stackexchange.com/questions/284416/the-stack-exchange-api-appears-to-ignore-accept-encoding-this-breaks-curl
The same thing is happening with request (which is used as the underlying HTTP client). It isn't expecting a gzipped response, but the SE API is sending one anyway. I haven't looked into it any further than than that, but have a look at the request docs: https://github.com/request/request/blob/master/README.md#requestoptions-callback - I suspect if you set gzip: true it will work :)

@chris48s Yep I got it figured out! It definitely was weird to be seeing a non-JSON response being returned back but at least now I learned something new 👍


Window freezes when using PhysicsServer in background thread

Godot version:
3.2.1 Mono
OS/device including version:
macOS 10.15.4
Issue description:
When I try to use PhysicsServer in different thread, game window eventually freezes and the only way to close it is with kill -9 -PID from terminal. I get no errors, no logs, only pointless mono dump.
It seems like it's related to the load, if I add items slowly (visuals with VisualServer and static bodies + shapes with PhysicsServer), crash happens later. If I add items fast, it happens much sooner.
Physics code is very basic:
RID body = PhysicsServer.BodyCreate();
PhysicsServer.BodySetMode(body, PhysicsServer.BodyMode.Static);
RID shape = PhysicsServer.ShapeCreate(PhysicsServer.ShapeType.Box);
PhysicsServer.ShapeSetData(shape, new Vector3(cube.Size.x / 2f, cube.Size.y / 2f, cube.Size.z / 2f));
PhysicsServer.BodyAddShape(body, shape, new Transform(Basis.Identity, new Vector3(0f, cube.Size.y / 2f, 0f)));
PhysicsServer.BodySetSpace(body, Space);
PhysicsServer.BodySetState(body, PhysicsServer.BodyState.Transform, cube.Position);

// and later
PhysicsServer.FreeRid(cube.Shape);
PhysicsServer.FreeRid(cube.Body);

Steps to reproduce:
Kind of hard to describe it here, I added reproduction project. But basic steps are the following:

Add a method that adds an item with VisualServer and PhysicsServer
Constantly run that method in a background thread

Minimal reproduction project:
Issue.zip
PS! Physics code is commented out to show that VisualServer works perfectly. These few lines can be found in Cubes.cs Add and Remove methods. Remove the comments to witness the crash.

I want to add that rendering/threads/thread_model is Multi-Threaded.

Also occurs on exported project.

Happens with both Godot and Bullet physics.

Maybe this helps?
Crashed Thread:        14  tid_d103

Exception Type:        EXC_BAD_INSTRUCTION (SIGILL)
Exception Codes:       0x0000000000000001, 0x0000000000000000
Exception Note:        EXC_CORPSE_NOTIFY

Termination Signal:    Illegal instruction: 4
Termination Reason:    Namespace SIGNAL, Code 0x4
Terminating Process:   exc handler [10580]

Thread 14 Crashed:: tid_d103
0   org.godotengine.godot         	0x000000010b690dcd BroadPhaseOctree::set_static(unsigned int, bool) + 157
1   org.godotengine.godot         	0x000000010b6979c6 CollisionObjectSW::_update_shapes() + 822
2   org.godotengine.godot         	0x000000010b68a54b BodySW::set_space(SpaceSW*) + 107
3   org.godotengine.godot         	0x0000000109950b1f godot_icall_2_633(MethodBind*, Object*, RID*, RID*) + 47
4   ???                           	0x000000013aaa8852 0 +<PHONE_NUMBER>
5   ???                           	0x00007fc052c72b40 0 +<PHONE_NUMBER>32832

Closing this as I'm not able to replicate it anymore on 4.0.rc.mono. Not sure what or which version fixed it.
Ran the minimal reproduction project (with few minor modifications) until 10,000 instances were added with static bodies.
In said project there's a 90% chance on every frame that cube + static body is added and 50% chance on every frame or when cube count is over set amount (10,000 in my case) that random cube is deleted. So it's quite volatile environment.


Check error before deferring calls that use file

Error checking should be done earlier, and if there's an error creating
a file we should not try to use the file.
Error creating a file is an exceptional condition, can happen for lack
of permissions, full disk, etc. Calling file.Close() on a nil File
pointer would panic.

@bparees @mfojtik PTAL

or @soltysh?

LGTM [merge]

LGTM [merge]


Solution to The DNA matching problem

Problem Given:A forensic lab wants to know whether a deceased person’s DNA matches a set of known relatives. It is known that the closest relatives will have most of the DNA matching, and as the relationship becomes farther, the matching genes will be lesser. For this, the DNA pattern of the deceased is given and the DNA pattern of 10 likely relatives are given. Write a program to identify the family of the deceased.

Thanks for contributing!


Implement TransferLogic

Reference -> https://github.com/hashgraph/hedera-services/issues/2416
Start with local branch in LimeChain -> feat/transfer-logic

PR: https://github.com/hashgraph/hedera-services/pull/2497


MaterialButton change after Navigation.PopModalAsync()

I use the MaterialButton for something, so when I click one button that a show ContenPage with the method Navigation.PushModalAsync that work fine, but when I call  await Navigation.PopModalAsync(), but before to call PopModalAsync(), I change a property of button like a BorderColor that change all the style of the button, is like the button not was a MaterialButton , because the modal dissapear but the button look like a native button and not like MaterialButton
That something I doing bad o something with the button, because i tried everything but nothing change, sorry is weird to me that happend

Will check on this. Is this in Android or iOS?


[WIP] Add zlib/gzip stream indexing and random access library

Add zlib/gzip stream indexing and random access library

bindings for the soci indexer
add magic number for blob, avoid crashes due to invalid blob
add test cases for soci indexer

#686
Signed-off-by: zyfjeff<EMAIL_ADDRESS>

I don't think it's related to SOCI. It's just another OCI random accessor.
Maybe Nydus-OCI blob?

We hope to use a pure rust version:)


Interface bug

Hi,
Also found out a crash bug on the desktop version, when you follow this flow:
File-->Load-->Cancel (after hit ok on the would be lost changes dialog) app exits.
Windows 10, version 095 desktop
thanks in advance! and Merry Christmas!
Jorge

good news ! can't reproduce it on Linux . Merry Christmas all

Lol
On Wed, Dec 26, 2018, 4:47 PM Philippe BENOIST<EMAIL_ADDRESS>wrote:

good news ! can't reproduce it on Linux . Merry Christmas all
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tentone/nunuStudio/issues/240#issuecomment-450031036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQLGlQnn1x7VQzfCHz3ig_T9d7IlanOqks5u8-50gaJpZM4ZiAKc
.

Please try the new windows release (updated 0.9.5) should be working fine now.

thanks Tentone, appreciate it!
On Thu, Dec 27, 2018 at 5:53 AM Tentone<EMAIL_ADDRESS>wrote:

Please try the new windows release (updated 0.9.5) should be working fine
now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tentone/nunuStudio/issues/240#issuecomment-450128482,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQLGlTJE9tQ1Maa_-a39gUNv4AmLdY-3ks5u9KaegaJpZM4ZiAKc
.


xdg-terminal is missing from xdg-utils

Describe the bug
xdg-terminal is missing from xdg-utils. It's a potentially very useful package as it provides a DE-independent way of launching a terminal.
Steps To Reproduce
Steps to reproduce the behavior:

Install xdg-utils
browse to the bin directory and you will see xdg-terminal is missing.

Expected behavior
xdg-terminal should launch the default terminal for any given DE.
Additional context
Not a bug per se. More of a feature request.
Notify maintainers

Add a :+1: reaction to issues you find important.

Does this need to go to staging or can it go directly to master? I presume staging given the number of rebuilds?

EDIT: Ok I've managed to add it to staging. Do I leave it as a Draft?

For more than 1000 rebuilds it should go to staging. You only leave it drafted, if the PR isn't ready for review.

in the interim, xdg-terminal-exec looks to provide very similar functionality for anyone who needs this immediately.


Existing Data in Table gets corrupted when more data imported with insufficeint disk space.

I tried to import data into a table which already had data. The disk space was running low and by mistake i tried importing more data. The import failed with error.
When the check the data in column some data partially imported, but when running aggeregate queries the queries started to fail & it could not read the full table.
Due to this the whole table needed to be replaced.

What table engine do you use?
This issue should not happed if you use MergeTree family of table engines. Appends are atomic and table should be always in physically consistent state; moreover, it will do automatic repair if data is rendered broken in filesystem.
But it may be considered normal if you use Log family of table engines, because writes are non-atomic, as documented.

@nadarkathiresh have you tried switching to MergeTree engine?


Open redirect issues

Hey,
There are numerous Open Redirect issues in Piranha.
Core\Piranha\Web\AuthController.cs
Controller action Logout() is using unvalidated parameter directly from Request object
string returl = Request["returnurl"];
Controller action Login() uses
Request["returnurl"];
Request["failureurl"];
Action NewPassword() consumes 2 unvalidated URLs from user input
Request["returnurl"];
Request["failureurl"];
To mitigate this issue Piranha should validate that redirect URL belongs to a whitelisted domain i.e. to the same domain where application is running.
You can read more about Open Redirects here:
https://www.owasp.org/index.php/Unvalidated_Redirects_and_Forwards_Cheat_Sheet

Sounds good! Feel free to make a fork and send me a pull request and I'll include it in the next service release! Otherwise I will take care of it as soon as I get the time.
Best regards
Håkan


Adding TCN Layers + Tutorial to TFA Layers

This PR adds TCN layer as well as the Residual Block layer to TFA layers. A colab with the basic usage of TCN layer with IMDB is also included.
TODO(shunlin):
Write more tests.
May I get some pointers on what kind of tests I should include? Thanks!
Associated Issues:
#661
philipperemy/keras-tcn#73
Source Implementations:
https://github.com/philipperemy/keras-tcn (In Pure Keras)
https://github.com/locuslab/TCN (In Pytorch)
Relevant Paper:
https://arxiv.org/pdf/1803.01271.pdf

Thanks for the contribution! My main concern is why we should instantiate layers in build instead of __init__. Could you elaborate the reason why?

Hi Tzu-Wei @WindQAQ, thank you so much for the review, you are right that there are some layers that are not depend on the input_shape should be instantiated in __init___, I will update the code to reflect that, thank you so much!

@WindQAQ I have resolved the comments you had earlier this week and updated the code, when you have time may you review the changes? Thank you so much!

@shun-lin The code looks good so far. A few changes requested.
Could you also expand the tests?

We need tests for both TCN and ResidualBlock.
Check that they can be serialized and deserialized. Both the layers are currently missing from_config.

I haven't taken a look at the notebook yet, but I'll do that soon.

got it, thanks @Squadrick !

@shun-lin The code looks good so far. A few changes requested.
Could you also expand the tests?

We need tests for both TCN and ResidualBlock.
Check that they can be serialized and deserialized. Both the layers are currently missing from_config.

I haven't taken a look at the notebook yet, but I'll do that soon.

@Squadrick I believe that we don't need to extend from_config method from tf.keras.layer's from_config as the from_config method that is inherited should work just fine, but will definitely write unit tests to confirm this :)

@shun-lin Thanks for making the changes. I think you've accidentally included an extra file: env/bin/tensorboard.
I'm adding a BLOCKED tag to this PR until the issue with layer_test is resolved.

@shun-lin Thanks for making the changes. I think you've accidentally included an extra file: env/bin/tensorboard.
I'm adding a BLOCKED tag to this PR until the issue with layer_test is resolved.

@Squadrick thanks for catching the extra file, I have removed env/bin/tensorboard in the latest patch. I will try to write more tests for TCN and wait on the issue with layer_test for test for residual block.

@Squadrick may you unblock this PR? I think the issue with layer_test should not be a blocking issue for this PR, I'll write more test for TCN layer and a TODO for adding tests for the internal residual block once the issue with layer_test is resolved. Thanks!

@shun-lin I'd prefer if we don't merge the PR until the tests are up and working. Three reasons for this:

Maintaining it will be a hassle since we can't check if any changes made to ResidualBlock are broken.
Any upstream changes in TF that cause ResidualBlock to change its behaviour won't be caught immediately.
We have no ETA on when the blocking issue will be resolved.

Will test_tcn catch any errors caused in ResidualBlock? If yes, then we can reconsider merging the PR with a TODO.

@Squadrick you have a good point, and the tensorflow team has assigned the blocking issue to me so I will look into resolving it when I have time. In the meantime I will write more tests for tcn and see if I can catch errors caused in ResidualBlock as you suggested. Thanks for the feedback!

@Squadrick so after digging inside Tensorflow Keras's codebase for tests for layers that are returning a list of Tensors, I found out that they are not using layer_test, but instead writing custom tests (see below for reference), and that layer_test does specifically said that it only works for "single-input single-output" layers and therefore at layer_test's current state I can not use it to test ResidualBlock, I will write custom tests for ResidualBlock without the use of layer_test and update it to layer_test when it finally support layers that have list of Tensors as output. Thanks!
tf.keras.layers.RNN's test when the layer is set (with 'return_sequences' set to 'True) to return a list of Tensors:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/keras/layers/recurrent_test.py#L330

@shun-lin Thanks for looking into this. Yes, that sounds like a good idea.

update it to layer_test when it finally support layers that have list of Tensors as output.

Make this a TODO in the code.

Thanks for this contribution @shun-lin. I am really interested in using TCN layers. If you need any help, let me know. I will be happy to help.

@shun-lin any update so far on this one? :)

Seems like the path forward here is already determined, but just wanted to mention that it was previously suggested we fork layer_test and expose it as a public API for other projects to use:
https://github.com/tensorflow/addons/issues/258

@pedrolarben @philipperemy @seanpmorgan
I apologize for the delay, the main issue I have right now is just testing the internal ResidualBlock layers, which returns a list of tensors, and since tf.keras's layer_test does not currently support testing layers that returns a list of tensors, I am working on extending layer_test to accomplish this, but it is taking longer than I expected. I'll try to put more time into this and contribute to update layer_test during the upcoming holiday breaks :)
Relevant Tensorflow Issue:
https://github.com/tensorflow/tensorflow/issues/34408
And @pedrolarben I think @philipperemy's Keras-TCN works with TF2 now as well, https://github.com/philipperemy/keras-tcn.
And @philipperemy if you have written any tests for TCN please let me know so I don't duplicate work, thanks!

@shun-lin I can confirm that it now works well with TF2. There are no unit tests at the moment for TCN. Just a quick training test to make sure that the training converges after a couple of epochs on a simple task.

This pull request had no activity for a while now. I'll be closing it. Feel free to open it again if there are changes.


Whisper user if they are ignored and trying to run a command

If a user is ignored, they should be whispered (to notify them that they're ignored) when they try and use commands (with a cd of course)

We should discuss this first. I think we shouldn't do this, but I'm happy to discuss the pros/cons.

The idea is that they're notified that the command is not working due to them being ignored. Otherwise they wouldn't know they were ignored I guess? That's also the idea of the whisper cooldown.


Add canary region and node LTS coverage to smoke test matrix

This PR adds a new entry to the smoke test matrix to add a public cloud test against a Canary/EUAP region (specifically eastus2euap), and to cover multiple node versions across the matrix.
Version coverage across the matrix taken from here
Azure/azure-sdk#1879

/azp list

@weshaggard yeah @mikeharder, @praveenkuttappan and I have been digging into several issues in a separate thread. The plan now is to start with another, simpler PR containing one of the test fixes, and then building a known good state from there. After that, if there are still lots of new failures, I'm going to split these changes up into PRs one by one (since it's actually adding a lot of new test scenarios with versions+regions).
The PR I mentioned: https://github.com/Azure/azure-sdk-for-js/pull/11826

/azp run js - smoke-tests

@mikeharder all public cloud smoke tests are green with https://github.com/Azure/azure-sdk-for-js/pull/11895. Once I merge that, I'll rebase this branch against it, and we can decide whether we want to merge this or wait until I fix the sovereign cloud 401 issue first.

@mikeharder all public cloud smoke tests are green with #11895. Once I merge that, I'll rebase this branch against it, and we can decide whether we want to merge this or wait until I fix the sovereign cloud 401 issue first.

I think it's fine to merge this PR as long as the public cloud tests (and canary) are green.

/azp run js - smoke-tests

@mikeharder so it looks like we are seeing some issues with both canary and node v8.x (https://dev.azure.com/azure-sdk/internal/_build/results?buildId=582696&view=results). So with that, I'm going to remove those two changes from this matrix, and if it's green, I'll merge this. Then I'll create two more PRs with each of those changes so we can deal with the failures one by one.

@mikeharder so it looks like we are seeing some issues with both canary and node v8.x (https://dev.azure.com/azure-sdk/internal/_build/results?buildId=582696&view=results). So with that, I'm going to remove those two changes from this matrix, and if it's green, I'll merge this. Then I'll create two more PRs with each of those changes so we can deal with the failures one by one.

It's not too surprising that both Node 8 and Canary are failing so this sounds good.  The failures on Node 8 are higher priority since this might be a real customer-impacting issue.

/check-enforcer override

Public cloud is green now, going to merge. https://dev.azure.com/azure-sdk/internal/_build/results?buildId=582816&view=results


Create EBWiki Organizations

I'm in the process of creating an EBWiki AWS Organization so that we can work together and pool resources under the name.  Unfortunately, adding users is a manual, multi day process, and even after receiving the go ahead from AWS, I find myself still unable to add a user as of this morning
.
I've sent a response and we'll see how it goes.

I am waiting on contact from AWS in order to finish this job up. Once this is done, I will add the EBWiki org members.

I am going to expand upon this to deal with all the services for which we could/should be organized. Here is a list of services:

[X]  Github
[ ] Heroku
[ ] AWS
[ ] Rollbar
[ ] New Relic
[ ] @BOWiki/participants - If you can think of any other organizations that apply, please let me know.

Here is the update from Amazon re: organization setup.
Hi,

Thank you for getting back to us.

I am sorry to hear you still experiencing issues when trying to add accounts to your organization.

I've reached out to our Organization team to investigate this again. I'll hold onto your case and update as soon as I hear back from them.

Thank you for your patience.

Best regards,

Ngoni N.

I just added Medium as a service that we'll use as an organization.  The assets I was using are on my Windows partition, so when I reset tonight I'll be able to add the publication.  I don't know that the credentials are listed on Trello.  @trystant can you invite me to that other board, so I can check and add them if need be?
Btw, I think it's really odd that I can just update your comment myself.

Marked Medium as completed.  @trystant can't we mark Heroku as complete as well?

We can mark Heroku as complete. We can mark AWS because we're no going to use it, and I haven't addressed the others yet.

@trystant at this point can we close this?

We're finished with this, as AWS organizations isn't a good fit for this open source priest as is.


GCS bucket and swap out S3

Overwrite a small parquet within GCS bucket.
Make sure it runs successfully in GCP, before migrating rest of S3 files over.

Opting to keep S3 bucket for this project. Future work will use GCS buckets.


tls: optionally allow expired TLS client certs.

Description: In attempting to replace HAProxy with Envoy, we need to be able to handle expired certs.  We want to require all connections have certs signed by our CA and any expired certs (that are signed by our CA) are redirected to another route.  The redirection is going to happen through a filter, but we need a way to let Envoy accept the connections.  Currently, Envoy enforces NotBefore and NotAfter so it will reject these connections.  Even if you leave require_client_certificate to default false, it will still reject expired certs when presented.  With this option enabled, it will allow expired certs.
In HAProxy, this functionality corresponds to:
use_backend expiredcert if { ssl_c_verify 10 }

or
redirect location ... if ...

Risk Level: Medium risk as the default is still to enable verification.  High risk if the logic is wrong.
Testing: Added test cases in this PR for the default case, explicitly allowing and explicitly denying cert expiration verification.  I also did manual testing with an envoy + golang backend + grpcurl client setup (since the latter can handle TLS certs with gRPC connections).
Docs Changes: N/A other than a new option added to the protos.
Release Notes: Added a note to the version history.

@doughd thanks for the contribution. @ggreenway @PiotrSikora can you take a first pass / sanity check on this?


Multiple WSL distro support

Hi! I have two Ubuntu WSL distro: Ubuntu and Ubuntu-20.04 for personal and working purposes respectively.
I set git executable in PhpStorm as C:\Users\yevhe\bin\wslgit.exe and git integration works fine with Ubuntu, but not with Ubuntu-20.04. It returns error Error updating changes:  not a git repository (or any of the parent directories): .git
Is it possible to get both of them working properly? If yes — how?
UPD: I have tried to download source code, replace Ubuntu-20.04 with Ubuntu and compile two different versions, but still no luck.

The problem is that wslgit always use the default wsl, so when accessing the filesystem from windows on another wsl instance the paths will translate to a Unix-path that exists on the other wsl instance, but wslgit will then run in the default instance where that path likely doesn't exist.
I'll push a PR soon that should make it possible to use wslgit with multiple distros, you just have to make sure git/ssh is setup correctly in all distros you intend to access repos in :)

@dixydo There is now an alpha build available with the changes from @carlolars :
https://ci.appveyor.com/project/andy-5/wslgit/build/artifacts
Does this work for your use case?

@andy-5 Thank you! Unfortunately I haven't possibility to test it right now as I'he already switched to single distro and route ssh config to repos manually to be able use  your tool. I will try to test it on weekend.

The changes have no been released as wslgit v1.2.0.


Can't use Plane Mode freely [2.0]

I'm on version 2.0 of the demo on Dolphin, I can only choose to go left or right, upon opening the save. If I choose left, both inputs send me flying left.

So sorry, it was my emulator bugging out. Not an actual issue with the mod!

Are you using the recommended version of Dolphin the Github states?

For me i use 5.O dolphin version and i have this problem

For me i use 5.O dolphin version and i have this problem


Update short title for using a matrix page

Why:
Matrix should be pluralized ”matrices” as in the body text.
Closes: none

What's being changed (if available, include any code snippets, screenshots, or gifs):
One letter.

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@alexanderchr Thanks so much for opening a PR! I'll get this triaged for review :zap:


Throw Error: Failed to load document on windows system

Failed to load document: assets/content/docs/getting-started.html. Error: Not Found

please provide details？
Angular Version, Docgeni Version, OS, steps to reproduce, If you can provide an online example.
Thanks！

Fixed


AmiGO should exclude regulates relationship from enrichment tool by default

Doesn't anyone else find this odd default behavior for the enrichment tool?
http://jira.geneontology.org/browse/GO-1335
Val
I heard back from MGI, and they think the difference is probably a result of the regulates relationship. When you use the enrichment tool on AMIGO2, they apparently exclude genes annotated to regulates children, but when you search AMIGO2 using the GO term it does return the genes annotated to regulates cild term. For example, 'regulation of somitogenesis' is a child term of 'somitogenesis', and this is a child of 'segmentation'. This regulation term would not be considered in the enrichment but would be included in the results when you search ith a term string. When I took the lists of genes returned from the enrichment (97) and compared them to the list associated with the term (103), the genes in the difference are all annotated to some kind of regulation of X where X is a child of segmentation, but they are not annotated to any isa or partof children of segmentation.
Some of the enrichment tools out there such as the GO Term Finder at Princeton (http://go.princeton.edu/cgi-bin/GOTermFinder) by default follow these terms, and if you look down the page to the advanced options, you see a toggle called ‘Follow regulation links’. So when it looks for all the genes annotated to a term, it includes all the genes annotated to regulation of that term. It appears that this is not the behavior of the AMIGO2 tool.

The AmiGO 1.x enrichment tool is no longer supported and it does not take into account pretty much anything. If you are referring to the PANTHER tool that we link out to, you'd probably want to contact them directly.

Pinging @ValWood @mugitty .