Feature Request: add BDfR as a new extractor for archiving Reddit content

Discussed in https://github.com/ArchiveBox/ArchiveBox/discussions/754

Originally posted by BlipRanger May 24, 2021
Just wanted to make a quick mention of BDfR as a cool project that might make for a good starting point for the unrolling of reddit comments/posts as mentioned in the roadmap. They currently support grabbing a variety of media types from the post as well as the comments/text in a separate (json) file. I've been working on an addon for it lately and I think it's a pretty great project with well-maintained code. If nothing else, they have really good examples of working with reddit data which could be useful! Just wanted to bring that to your attention!
I'd love to add BDfR as an extractor for Reddit content (and something similar for Twitter too https://github.com/ArchiveBox/ArchiveBox/issues/345) but am somewhat swamped with work and travel for the near future.
If you @BlipRanger or anyone else wants to add it as an extractor (matching the style of our other extractors, e.g. archivebox/extractors/media.py is a great example to copy), I'd be happy to review PRs!
We have some good instructions for contributing a new extractor and getting started with ArchiveBox development in general:

https://github.com/ArchiveBox/ArchiveBox/blob/dev/README.md#contributing-a-new-extractor
https://github.com/ArchiveBox/ArchiveBox/blob/dev/README.md#archivebox-development
https://github.com/ArchiveBox/ArchiveBox/blob/dev/.github/CONTRIBUTING.md

We use Mercury (recently renamed postlight) as an extractor already, and they're rapidly adding extractors on their side for many different kinds of sites, so we should get these improvements with no effort required on the archivebox side:

Reddit threads: https://github.com/postlight/parser/pull/746
HN threads: https://github.com/postlight/parser/pull/745
Twitter threads: https://github.com/postlight/parser/pull/622

It looks like the postlight project has no recent activity unfortunately (no PR reviews at least)...
Is there any plan to replace it with something else, or integrate the existing Reddit and HN PRs in a different way?


Items with same name all show same cover icon regardless of actual contents

All instances of an item with same name show the last retrieved cover icon regardless of what the first image actually was or if there even was an image. All cover icons are changed to the last cover image.
This may no longer be an issue if completed items are changed to final file name as this would prevent duplicate item references?

I can't recreate this problem, works for me. Though items with the same name can't be added to the list simultaneously (only if one of them is archived), which is a bug.

More of a pain to replicate than I thought. I saw it twice in a row in testing and thought it was repeatable. In trying to find a thread that reproduces it repeatedly, but I haven't found the trigger. I'll keep trying to find a thread that works repeatedly, but something is happening with the 2.5.1 (03F1) build.

I recreate it at will with 2.5.1 (043C).

Interestingly the previous item(s) cover images changes before the compression is complete on the new item.

All right, now I understand. The problem is that I don't cache the images like before, instead it fetches them each them it updates the view from the folder. If you change the name of the folder and then add it to the list, the compressed items icon will disappear. I remember now why I did the cache thing from the beginning. It's an easy fix.

Fixed in latest build.


I have googled it, but it has not been solved yet. I have the following problem, please help me

I only used MSCOCO for train.
Trying the 1st: 1st: pre-train Hand4Whole and run
python train.py --gpu 0-3 --lr 1e-4 --continue
Getting the following error
Traceback (most recent call last):   File "train.py", line 82, in <module>     main()   File "train.py", line 53, in main     loss = trainer.model(inputs, targets, meta_info, 'train')   File "/home/wansiqi1/.custom/pyrender78/envs/train/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in _call_impl     return forward_call(*input, **kwargs)   File "/home/wansiqi1/.custom/pyrender78/envs/train/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 168, in forward     outputs = self.parallel_apply(replicas, inputs, kwargs)   File "/home/wansiqi1/.custom/pyrender78/envs/train/lib/python3.8/site-packages/torch/nn/parallel/data_parallel.py", line 178, in parallel_apply     return parallel_apply(replicas, inputs, kwargs, self.device_ids[:len(replicas)])   File "/home/wansiqi1/.custom/pyrender78/envs/train/lib/python3.8/site-packages/torch/nn/parallel/parallel_apply.py", line 86, in parallel_apply     output.reraise()   File "/home/wansiqi1/.custom/pyrender78/envs/train/lib/python3.8/site-packages/torch/_utils.py", line 461, in reraise     raise exception KeyError: Caught KeyError in replica 0 on device 0.
Anyone else seen this? Thanks!

what is pyrender78?

It is the name I used to customize the image on the cloud server.
I find that I get no error when I use dataset Human36M+MSCOCO+MPII, but I get this error when I use only one of them.

How do you set trainset_3d and trainset_2d in config.py?

trainset_3d = []
trainset_2d = ['MSCOCO']

I have just downloaded current repo and ran without any problem.

trainset_3d = [] trainset_2d = ['MSCOCO']

It is the name I used to customize the image on the cloud server. I find that I get no error when I use dataset Human36M+MSCOCO+MPII, but I get this error when I use only one of them.

Hello, do you know what's causing this problem


chore(changelog): move changelog files to correct location

Summary
move changelog files to kong/changelog/unreleased/kong/
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

Fix #[issue number]

Better link to the original PR.


hw/mcu/dialog: Make 1V8, 1V8P and 1V8F rails configurable

This adds some configuration options to control 1V8x rails. For now,
only LDOs are selectable with possibly more options to come, if needed.

@JustineKH ^^


Change headers on HTTP reponse through ReponseFilter

Hi,
I would like to change the content-type of a response using a ReponseFilter.
My implementation looks like this:
proxy.addResponseFilter(new ResponseFilter() {
            @Override
            public void filterResponse(HttpResponse response, HttpMessageContents contents, HttpMessageInfo messageInfo) {

                if (contents.getContentType().equals("application/pdf")) {
                       //I would like to do:
                       contents.setContentType("text/plain");
                 }
...
}

Is there any way to accomplish that?
I am using the embedded Java API version 2.1.0-beta-5.

Yes, this is quite simple: the response object in the filter method contains a .headers() method, which allows you to examine and set the headers.

Thank you @jekh! For some reason I didn't notice the .headers() method. That worked for me.


ssh pulling doesn't works cause  no matching method

Hello.
We have a lot of Cisco ASA  (5506-X\ASA5505)
Software Version from 7.2(4)  till 9.8(4)12
It can't be polled arp.  asa supports only
dh-group1-sha1   Diffie-Hellman group 2
dh-group14-sha1  Diffie-Hellman group 14

So, if I check in console
$netdisco-do  macsuck -d <IP_ADDRESS> -D 99
[99] 2020-07-31 17:46:57  info App::Netdisco version 2.046001 loaded.
[99] 2020-07-31 17:46:58  info macsuck: [<IP_ADDRESS>] started at Fri Jul 31 17:46:58 2020
[99] 2020-07-31 17:46:58 debug macsuck: running with timeout 600s
[99] 2020-07-31 17:46:58 debug => running workers for phase: check
[99] 2020-07-31 17:46:58 debug -> run worker check/_base_/0
[99] 2020-07-31 17:46:58 debug is_macsuckable: <IP_ADDRESS> has no layer 2 capability
[99] 2020-07-31 17:46:58 debug macsuck skipped: <IP_ADDRESS> is not macsuckable
[99] 2020-07-31 17:46:58  info macsuck: finished at Fri Jul 31 17:46:58 2020
[99] 2020-07-31 17:46:58  info macsuck: status info: macsuck skipped: <IP_ADDRESS> is not macsuckable

$ssh -l user <IP_ADDRESS>
Unable to negotiate with <IP_ADDRESS> port 22: no matching key exchange method found. Their offer: diffie-hellman-group1-sha1
(and if changed):
$ssh -l user <IP_ADDRESS>
Unable to negotiate with <IP_ADDRESS> port 22: no matching key exchange method found. Their offer: diffie-hellman-group14-sha1
$cat /etc/os-release
PRETTY_NAME="Alpine Linux v3.12"

Your Environment

Software
Version

App::Netdisco
2.46.1

SNMP::Info
3.70

DB Schema
63

PostgreSQL
12.00.2

Perl
5.30.3

docker engine version:
19.03.8

docker-compose version:
1.25.5

host operating system
CentOS Linux 8

netdisco/netdisco
latest-backend

netdisco/netdisco
latest-web

netdisco/netdisco
latest-postgresql

Config info (deployment.yml and docker env settings)
- tag: 'ASA arp check'
  driver: cli
  platform: ASA
  username: user
  password: pass
  enable_password: pass
  only:
     - '<IP_ADDRESS>'

see https://github.com/netdisco/netdisco/wiki/Configuration#device_auth
ssh_master_opts option.

Hi @Dees7
Looks like you need to enable DSS key exchange algorithms. You can either put this into the ssh_master_opts as @inphobia says, or in the ssh_config file
https://www.openssh.com/legacy.html
Host somehost.example.org
    KexAlgorithms +diffie-hellman-group1-sha1

ssh -oHostKeyAlgorithms=+ssh-dss user@legacyhost

As a side note only the arpnip action uses SSH, macsuck like in your example is SNMP-only

Thank all.
Docker doesn't used ~/.ssh/ssh_config
I edit /etc/ssh/ssh_config  (mapped from host to keep after update). and  now all ok. works


Create Tries DataStructure

Hey @laroshan !
Great work!
You can star this repository for further updates.
Have fun!


@mentions unlinked after re-editing post

From @jeroenpraat on July 24, 2014 20:32
When you edit a post after publishing (users have to enable this in feature settings) @mentions will be unlinked and become only text.
Copied from original issue: friendica/red#543

From @ghost on July 24, 2014 21:10
A bit more detail:  When you create a post, the mention is stored as @[zrl=whatever]whatever+[/zrl].  So far so good.  When you come to edit it, this is stripped to just @whatever.  Not 100% certain, but I think this is happening in undo_post_tagging().  We either need to not do that, or redo that on submit.


upload: use smaller reads when doing client-side checksumming

So as to not try to read an entire large file into RAM.

Codecov Report

Merging #319 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #319      +/-   ##
==========================================
+ Coverage   84.97%   85.03%   +0.06%
==========================================
  Files          37       37
  Lines        1730     1731       +1
==========================================
+ Hits         1470     1472       +2
+ Misses        260      259       -1

Impacted Files
Coverage Δ

hca/upload/lib/client_side_checksum_handler.py
91.17% <100%> (+0.26%)
:arrow_up:

hca/util/__init__.py
87.42% <0%> (+0.28%)
:arrow_up:

Continue to review full report at Codecov.

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


feat: How To Install paragraph in the readme

Feature Request
Readme file should have How To Install section.

Cairo 1 doesn't require installation :)


Add manual test cases

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
see 26 files with indirect coverage changes


Mrtk dev editor update to 2017.2.0f3 MRTP

Fixes #1114

Going to close this and see if we need to use the official release.

Seems like Unity didn't include the MRTP fixes in the official release, so this is the correct editor version to use.


Cabal 3.4 in Github Actions

Includes a fix to the way executables are invoked which needs to be different when using cabal-3.4.

The change decreases test times to:

ghc-8.6.5 on ubuntu: 18m 5s
ghc-8.6.5 on macos: 18m 29s
ghc-8.6.5 on windows: 22m 37s
ghc-8.10.2 on ubuntu: 18m 43s
ghc-8.10.2 on macos: 29m 0s
ghc-8.10.2 on windows: 25m 29s

bors merge


Machine win-15 went dead from network monitoring website.

Machine win-15 went dead from network monitoring website.

Network adapter connection disabled by itself lack of activities by rebooting machine to restore connection.


Shop sign causing ticking world crash on placement

Minecraft 1.18.1
Forge 39.0.55
Builders Addition 20211208a
I tried to place the shop sign & the game crashed with this error:
The game crashed whilst exception ticking world
Error: java.lang.NoSuchMethodError: 'net.minecraft.nbt.CompoundTag com.mrh0.buildersaddition.tileentity.ShopSignTileEntity.save(net.minecraft.nbt.CompoundTag)'
Exit Code: -1
I suspect something in Forge changed since I had placed them just fine in a test world running Forge 39.0.45

Fixed in 20220123a.


Update private-endpoint-dns.md

Changed resource type for Cosmos

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

#sign-off


Short-cut for "Next replay move"

It's very long-winded to show the moves of a replayed game, because you always have to do 2 menu steps for that. I suggest, to introduce at least for menu points "Skat table - Next replay move" a keyboard short-cut, so that you can go faster through the replayed moves.

Hmm, there are those buttons already in the toolbar:

Have you deactivated the toolbar in the settings? Or isn't it visible at all. Maybe a problem of mixing Java Swing and JavaFX together in one UI.

The toolbar is also a problem for me, because it doesn't work on my computer correctly. But it's true, I had deactivated the toolbar for that reason and had forgot about that fact.

Because toolbar is sufficient, I close this issue.


Missing Frameworks

More PHP frameworks that should be included (in no particular order):

Magento  29a5ba91a8d7cf8ce6743ae4bb89f09a2f7be9cc
Yii 85e44424e7c64421904806b06a36244e52cd90f6
Solar/Aura See #24
Zikula
PPI  c50ae51d3c938b261c542289a13db2611e91d69a
Agavi
Chisimba
ezPublish
Phorum
MicroMVC
Kohana 59bb85073c92d77c07112188726546e3958d16d0
TYPO3 a02480ae3785e88e26527d21abe50c31fe2edd7d
FLOW3

you should also allow a "generic" type which allows me to add a library which just clones to a folder of my choosing and does nothing special to it.

@christhomas That generic type exists and is library which can be configured through the vendor-dir option. Or if you need further control you could write a custom installer.

Slim ?

Slim ?

Isn't Slim composer native?

Closing as I think we got a decent coverage now ;)


Bilateral scene graph linking

Collections will now be set as the parent of any node added to them.  This also permits nodes to add themselves to and remove themselves from collections via the new attach() and detach() methods.

This has been superseded by Pull Request #21.


refactor: migrates onboarding components to composition api

Migrates all onboarding-related components to composition API.
Streamlines color usage within the onboarding module to use variables where necessary (mainly replacing hexadecimal colors in templates).
Replaces all Tailwind utility classes in the onboarding module with either custom CSS or utility classes that are also present via Kongponents (which already override the Tailwind ones).
Signed-off-by: Philipp Rudloff<EMAIL_ADDRESS>

@johncowen I’ve centered the welcome view content now.
I also added another commit to replace hard-coded colors in the graphs with custom properties as discussed earlier.

Sounds good 👍


Add compression to FicTracDataInterface

Adding compressio for FicTracDataInterface

I agree that it is mildly annoying to add this only for it to be immediately deprecated after the backend refactor (only 2 PRs away I think now...)
But deadlines are deadlines and this data really should be compressed

Not a lot of work but is fine. To be honest I did not concerned mysefl because it is 30 MB of data. Do we have any threshold for when compression makes sense?

Do we have any threshold for when compression makes sense?

The word from Satra and Yarik on the archive is that every byte that can be compressed, should be compressed.
If anyone was actually paying $$$ for the bucket it would be an even bigger deal (hard enforced through validation), because why would you pay for storage you don't need [but this is then an indirect implication with the AWS Open Data team because someone is paying for it somehow; a hidden opportunity cost perhaps]
Also bandwidth cost. When the data is ready hundreds, thousands of times over the years by users, each unnecessary byte that gets sent is wasted time and theoretically also transfer cost if using cloud resources
So yeah, even 30 MB of data, if it gets streamed 1,000+ times, adds up. So if there was something that could have been done to easily reduce that amount, we should try

Thanks for explaining the rationale, makes sense.


Customize the separator / and define \ as separator too

Related to #55

Also, let's remove any duplication. E.g. // === /, /\ == /, \\ == /, \/ ==

Abandoned


Add log

Pull Request
Description
Add logging
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration

[ ] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[ ] Yes

Checklist:

[ ] My code follows OCF's coding style guidelines
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

@all-contributors please add @jacobbieker  for code

@all-contributors please add @JackKelly for code

@all-contributors please add @AyrtonB for code

@all-contributors please add @Rabscuttler for code

Good work, @peterdudfield!

@all-contributors please add @notger  for doc

@all-contributors please add @peterdudfield  for doc


[PE-D][Tester E] Example commands in UG

This is a bit nitpicky but the example commands in the quick start section of the UG should contain each feature available, it is missing the seller and meeting functions.

Labels: type.DocumentationBug severity.VeryLow
original: rynlwhh/ped#8

Thank you for that suggestion! However, I believe we did mention "some example commands you can try" so we would like to leave this portion as just "samples" they could try, but the full command usages will be presented in the feature list below.


[Feature Request]: MySQL on ARM/Macbook M1

Requested Feature
Currently, if I install MySQL (Homebrew) feature on MacBook M1, it will have an error:
Architecture aarch64 unsupported

So it would be nice to have the MySQL version with ARM/M1 support.
Full error:
#0 0.120 ===========================================================================
#0 0.120 Feature       : MySQL (via Homebrew)
#0 0.120 Description   : MySQL is an open-source relational database management system (RDBMS)
#0 0.120 Id
[2023-06-24T06:45:40.682Z]           : ghcr.io/devcontainers-contrib/features/mysql-homebrew
#0 0.120 Version       : 1.0.18
#0 0.120 Documentation : http://github.com/devcontainers-contrib/features/tree/main/src/mysql-homebrew
#0 0.120 Options       :
#0 0.120     VERSION="latest"
#0 0.120 ===========================================================================
#0 0.122 bash: cannot set terminal process group (1): Inappropriate ioctl for device
#0 0.122 bash: no job control in this shell
#0 3.265 nanolayer
#0 8.282 cd /tmp/tmp9ik8okqd && chmod +x -R . && _REMOTE_USER="vscode" _REMOTE_USER_HOME="/home/vscode" PACKAGE="mysql" VERSION="latest" INSTALLATION_FLAGS="" NANOLAYER_VERBOSE="" NANOLAYER_FORCE_CLI_INSTALLATION="" NANOLAYER_PROPAGATE_CLI_LOCATION="1" NANOLAYER_CLI_LOCATION="/tmp/nanolayer-1MGh3iyOTl/nanolayer" bash  -i  +H ./install.sh
#0 8.282 + source ./library_scripts.sh

#0 8.283 + PACKAGE=mysql

#0 8.283 + VERSION=latest

#0 8.283 + INSTALLATION_FLAGS=

#0 8.283 + '[' -z mysql ']'

#0 8.284 ++ id
[2023-06-24T06:45:40.682Z] -u

#0 8.288 + '[' 0 -ne 0 ']'

#0 8.288 + install_via_homebrew mysql latest ''

#0 8.288 + package=mysql

#0 8.288 + version=latest

#0 8.288 + installation_flags=

#0 8.288 + type brew

#0 8.288 + echo 'Installing Homebrew...'

#0 8.288 Installing Homebrew...

#0 8.288 + ensure_nanolayer nanolayer_location v0.4.29

#0 8.288 + local variable_name=nanolayer_location

#0 8.288 + local required_version=v0.4.29

#0 8.288 + [[ v0.4.29 == v* ]]

#0 8.288 + local nanolayer_location=

#0 8.288 + [[ -z '' ]]

#0 8.288 + [[ -z /tmp/nanolayer-1MGh3iyOTl/nanolayer ]]

#0 8.288 + '[' -f /tmp/nanolayer-1MGh3iyOTl/nanolayer ']'

#0 8.288 + '[' -x /tmp/nanolayer-1MGh3iyOTl/nanolayer ']'

#0 8.288 + nanolayer_location=/tmp/nanolayer-1MGh3iyOTl/nanolayer

#0 8.288 + echo 'Found a pre-existing nanolayer which were given in env varialbe: /tmp/nanolayer-1MGh3iyOTl/nanolayer'

#0 8.288 Found a pre-existing nanolayer which were given in env varialbe: /tmp/nanolayer-1MGh3iyOTl/nanolayer

#0 8.288 + [[ -z /tmp/n
[2023-06-24T06:45:40.682Z] anolayer-1MGh3iyOTl/nanolayer ]]

#0 8.288 + local current_version

#0 8.288 ++ /tmp/nanolayer-1MGh3iyOTl/nanolayer --version

#0 8.432 + current_version=0.4.39

#0 8.432 + [[ 0.4.39 == v* ]]

#0 8.432 + current_version=v0.4.39

#0 8.432 + '[' v0.4.39 == v0.4.29 ']'

#0 8.432 + echo 'skipping usage of pre-existing nanolayer. (required version v0.4.29 does not match existing version v0.4.39)'

#0 8.432 skipping usage of pre-existing nanolayer. (required version v0.4.29 does not match existing version v0.4.39)

#0 8.432 + nanolayer_location=

#0 8.432 + [[ -z '' ]]

#0 8.432 ++ uname -sm

#0 8.433 + '[' 'Linux aarch64' == 'Linux x86_64' ']'

#0 8.434 ++ uname -sm

#0 8.435 + '[' 'Linux aarch64' == 'Linux aarch64' ']'

#0 8.435 ++ mktemp -d -t nanolayer-XXXXXXXXXX

#0 8.436 + tmp_dir=/tmp/nanolayer-iUJ43VETYF

#0 8.436 + trap clean_up EXIT

#0 8.436 + '[' -x /sbin/apk ']'

#0 8.436 + clib_type=gnu

#0 8.437 ++ uname -m

#0 8.438 + tar_filename=nanolayer-aarch64-unknown-linux-gnu.tgz

#0 8.4
[2023-06-24T06:45:40.683Z] 38 + clean_download https://github.com/devcontainers-contrib/cli/releases/download/v0.4.29/nanolayer-aarch64-unknown-linux-gnu.tgz /tmp/nanolayer-iUJ43VETYF/nanolayer-aarch64-unknown-linux-gnu.tgz

#0 8.438 + url=https://github.com/devcontainers-contrib/cli/releases/download/v0.4.29/nanolayer-aarch64-unknown-linux-gnu.tgz

#0 8.438 + output_location=/tmp/nanolayer-iUJ43VETYF/nanolayer-aarch64-unknown-linux-gnu.tgz

#0 8.438 ++ mktemp -d

#0 8.439 + tempdir=/tmp/tmp.K1iZinpDRm

#0 8.439 + downloader_installed=

#0 8.439 + type curl

#0 8.440 + downloader=curl

#0 8.440 + '[' -z curl ']'

#0 8.440 + '[' curl = wget ']'

#0 8.440 + curl -sfL https://github.com/devcontainers-contrib/cli/releases/download/v0.4.29/nanolayer-aarch64-unknown-linux-gnu.tgz -o /tmp/nanolayer-iUJ43VETYF/nanolayer-aarch64-unknown-linux-gnu.tgz

#0 11.80 + '[' -z ']'

#0 11.80 + tar xfzv /tmp/nanolayer-iUJ43VETYF/nanolayer-aarch64-unknown-linux-gnu.tgz -C /tmp/nanolayer-iUJ43VETYF

#0 11.80 nanolayer

#0 12.24 + chmod a+x /t
[2023-06-24T06:45:40.683Z] mp/nanolayer-iUJ43VETYF/nanolayer

#0 12.24 + nanolayer_location=/tmp/nanolayer-iUJ43VETYF/nanolayer

#0 12.24 + declare -g nanolayer_location=/tmp/nanolayer-iUJ43VETYF/nanolayer

#0 12.24 + /tmp/nanolayer-iUJ43VETYF/nanolayer install devcontainer-feature ghcr.io/meaningful-ooo/devcontainer-features/homebrew:2.0.3 --option shallow_clone=true --option update=true

#0 16.55 cd /tmp/tmpve1ofyit && chmod +x -R . && _REMOTE_USER="vscode" _REMOTE_USER_HOME="/home/vscode" SHALLOW_CLONE="true" UPDATE="true" SHALLOWCLONE="false" NANOLAYER_VERBOSE="" NANOLAYER_FORCE_CLI_INSTALLATION="" NANOLAYER_PROPAGATE_CLI_LOCATION="1" NANOLAYER_CLI_LOCATION="/tmp/nanolayer-1MGh3iyOTl/nanolayer" bash  -i  +H ./install.sh

#0 16.57 (!) Architecture aarch64 unsupported

#0 16.61 Traceback (most recent call last):

#0 16.61   File "<string>", line 1, in <module>

#0 16.61   File "nanolayer.__main__", line 50, in main
[2023-06-24T06:45:40.683Z] #0 16.61   File "typer.main", line 328, in __call__

#0 16.61   File "typer.main", line 311, in __call__

#0 16.61   File "click.core", line 1130, in __call__

#0 16.61   File "typer.core", line 778, in main

#0 16.61   File "typer.core", line 216, in _main

#0 16.61   File "click.core", line 1657, in invoke

#0 16.61   File "click.core", line 1657, in invoke

#0 16.61   File "click.core", line 1404, in invoke

#0 16.61   File "click.core", line 760, in invoke

#0 16.61   File "typer.main", line 683, in wrapper

#0 16.61   File "nanolayer.cli.install", line 64, in install_devcontainer_feature

#0 16.61   File "nanolayer.installers.devcontainer_feature.oci_feature_installer", line 131, in install

#0 16.61   File "nanolayer.utils.invoker", line 56, in invoke

#0 16.61 nanolayer.utils.invoker.Invoker.InvokerException: The command 'cd /tmp/tmpve1ofyit && chmod +x -R . && _REMOTE_USER="vscode" _REMOTE_USER_HOME="/home/vscode" SHALLOW_CLONE="true" UPDATE="true" SHALLOWCLONE="false" NANOLAYER_VERBOSE="
[2023-06-24T06:45:40.683Z] " NANOLAYER_FORCE_CLI_INSTALLATION="" NANOLAYER_PROPAGATE_CLI_LOCATION="1" NANOLAYER_CLI_LOCATION="/tmp/nanolayer-1MGh3iyOTl/nanolayer" bash  -i  +H ./install.sh' failed. error: Return Code: 1. see logs for details.

#0 16.71 Sentry is attempting to send 2 pending error messages

#0 16.71 Waiting up to 2 seconds
[2023-06-24T06:45:40.683Z] #0 16.71 Press Ctrl-C to quit

#0 16.81 + clean_up

#0 16.81 + ARG=1

#0 16.81 + rm -rf /tmp/nanolayer-iUJ43VETYF

#0 16.83 + exit 1

#0 16.87 Traceback (most recent call last):
#0 16.87   File "<string>", line 1, in <module>
#0 16.87   File "nanolayer.__main__", line 50, in main
#0 16.87   File "typer.main", line 328, in __call__
#0 16.87   File "typer.main", line 311, in __call__
#0 16.87   File "click.core", line 1130, in __call__
#0 16.87   File "typer.core", line 778, in main
#0 16.87   File "typer.core", line 216, in _main
#0 16.87   File "click.core", line 1657, in invoke
#0 16.87   File "click.core", line 1657, in invoke
#0 16.87   File "click.core", line 1404, in invoke
[2023-06-24T06:45:40.683Z]
#0 16.87   File "click.core", line 760, in invoke
#0 16.87   File "typer.main", line 683, in wrapper
#0 16.87   File "nanolayer.cli.install", line 65, in install_devcontainer_feature
#0 16.87   File "nanolayer.installers.devcontainer_feature.oci_feature_installer", line 134, in install
#0 16.87   File "nanolayer.utils.invoker", line 56, in invoke
#0 16.87 nanolayer.utils.invoker.Invoker.InvokerException: The command 'cd /tmp/tmp9ik8okqd && chmod +x -R . && _REMOTE_USER="vscode" _REMOTE_USER_HOME="/home/vscode" PACKAGE="mysql" VERSION="latest" INSTALLATION_FLAGS="" NANOLAYER_VERBOSE="" NANOLAYER_FORCE_CLI_INSTALLATION="" NANOLAYER_PROPAGATE_CLI_LOCATION="1" NANOLAYER_CLI_LOCATION="/tmp/nanolayer-1MGh3iyOTl/nanolayer" bash  -i  +H ./install.sh' failed. error: Return Code: 1. see logs for details.
#0 16.97 Sentry is attempting to send 2 pending error messages
#0 16.97 Waiting up to 2 seconds
#0 16.97 Press Ctrl-C to quit
#0 17.08 ERROR: Feature "MySQL (via Homebrew)" (ghcr.io/devcontainers-contrib/fea
[2023-06-24T06:45:40.683Z] tures/mysql-homebrew) failed to install! Look at the documentation at http://github.com/devcontainers-contrib/features/tree/main/src/mysql-homebrew for help troubleshooting this error.
------
Dockerfile.extended:24
--------------------
  23 |
  24 | >>> RUN --mount=type=bind,from=dev_containers_feature_content_source,source=mysql-homebrew_4,target=/tmp/build-features-src/mysql-homebrew_4 \
  25 | >>>     cp -ar /tmp/build-features-src/mysql-homebrew_4 /tmp/dev-container-features \
  26 | >>>  && chmod -R 0755 /tmp/dev-container-features/mysql-homebrew_4 \
  27 | >>>  && cd /tmp/dev-container-features/mysql-homebrew_4 \
  28 | >>>  && chmod +x ./devcontainer-features-install.sh \
  29 | >>>  && ./devcontainer-features-install.sh \
  30 | >>>  && rm -rf /tmp/dev-container-features/mysql-homebrew_4
  31 |
--------------------
ERROR: failed to solve: process "/bin/sh -c cp -ar /tmp/build-features-src/mysql-homebrew_4 /tmp/dev-container-features  && chmod -R 0755 /tmp/dev-container-features
[2023-06-24T06:45:40.684Z] /mysql-homebrew_4  && cd /tmp/dev-container-features/mysql-homebrew_4  && chmod +x ./devcontainer-features-install.sh  && ./devcontainer-features-install.sh  && rm -rf /tmp/dev-container-features/mysql-homebrew_4" did not complete successfully: exit code: 1
[2023-06-24T06:45:40.690Z] Stop (21134 ms): Run: docker buildx build --load --build-context dev_containers_feature_content_source=/var/folders/dm/_fz_0qds5ys81lc_7x3gc5pw0000gn/T/devcontainercli/container-features/0.42.0-1687589114879 --build-arg _DEV_CONTAINERS_BASE_IMAGE=mcr.microsoft.com/devcontainers/base:jammy --build-arg _DEV_CONTAINERS_IMAGE_USER=root --build-arg _DEV_CONTAINERS_FEATURE_CONTENT_SOURCE=dev_container_feature_content_temp --target dev_containers_target_stage -t vsc-metapals-server-lite-78db92461b8c779f2a9953f96619ddf5f314c7481cd33851ba4a7d3aeb6e0b18-features -f /var/folders/dm/_fz_0qds5ys81lc_7x3gc5pw0000gn/T/devcontainercli/container-features/0.42.0-1687589114879/Dockerfile.extended /Users/tonysong/Library/Application Support/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/empty-folder
[2023-06-24T06:45:40.690Z] Error: Command failed: docker buildx build --load --build-context dev_containers_feature_content_source=/var/folders/dm/_fz_0qds5ys81lc_7x3gc5pw0000gn/T/devcontainercli/container-features/0.42.0-1687589114879 --build-arg _DEV_CONTAINERS_BASE_IMAGE=mcr.microsoft.com/devcontainers/base:jammy --build-arg _DEV_CONTAINERS_IMAGE_USER=root --build-arg _DEV_CONTAINERS_FEATURE_CONTENT_SOURCE=dev_container_feature_content_temp --target dev_containers_target_stage -t vsc-metapals-server-lite-78db92461b8c779f2a9953f96619ddf5f314c7481cd33851ba4a7d3aeb6e0b18-features -f /var/folders/dm/_fz_0qds5ys81lc_7x3gc5pw0000gn/T/devcontainercli/container-features/0.42.0-1687589114879/Dockerfile.extended /Users/tonysong/Library/Application Support/Code/User/globalStorage/ms-vscode-remote.remote-containers/data/empty-folder
[2023-06-24T06:45:40.690Z]     at mte (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2003:3286)
[2023-06-24T06:45:40.690Z]     at IN (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2003:3222)
[2023-06-24T06:45:40.690Z]     at process.processTicksAndRejections (node:internal/process/task_queues:96:5)
[2023-06-24T06:45:40.690Z]     at async Ote (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2020:3660)
[2023-06-24T06:45:40.690Z]     at async Hf (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2020:4775)
[2023-06-24T06:45:40.690Z]     at async Qre (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2151:10371)
[2023-06-24T06:45:40.691Z]     at async Zre (/Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js:2151:10112)
[2023-06-24T06:45:40.693Z] Stop (26178 ms): Run: /Applications/Developer Tools/Visual Studio Code.app/Contents/Frameworks/Code Helper (Plugin).app/Contents/MacOS/Code Helper (Plugin) --ms-enable-electron-run-as-node /Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js up --user-data-folder /Users/tonysong/Library/Application Support/Code/User/globalStorage/ms-vscode-remote.remote-containers/data --container-session-data-folder /tmp/devcontainers-9302aed5-9e56-4bf4-b56a-ad42b358c3501687589113508 --workspace-folder /Users/tonysong/Projects/metapals/micro-services/metapals-server-lite --workspace-mount-consistency cached --id-label devcontainer.local_folder=/Users/tonysong/Projects/metapals/micro-services/metapals-server-lite --id-label devcontainer.config_file=/Users/tonysong/Projects/metapals/micro-services/metapals-server-lite/.devcontainer/devcontainer.json --log-level debug --log-format json --config /Users/tonysong/Projects/metapals/micro-services/metapals-server-lite/.devcontainer/devcontainer.json --default-user-env-probe loginInteractiveShell --remove-existing-container --mount type=volume,source=vscode,target=/vscode,external=true --skip-post-create --update-remote-user-uid-default on --mount-workspace-git-root true
[2023-06-24T06:45:40.693Z] Exit code 1
[2023-06-24T06:45:40.695Z] Command failed: /Applications/Developer Tools/Visual Studio Code.app/Contents/Frameworks/Code Helper (Plugin).app/Contents/MacOS/Code Helper (Plugin) --ms-enable-electron-run-as-node /Users/tonysong/.vscode/extensions/ms-vscode-remote.remote-containers-0.295.0/dist/spec-node/devContainersSpecCLI.js up --user-data-folder /Users/tonysong/Library/Application Support/Code/User/globalStorage/ms-vscode-remote.remote-containers/data --container-session-data-folder /tmp/devcontainers-9302aed5-9e56-4bf4-b56a-ad42b358c3501687589113508 --workspace-folder /Users/tonysong/Projects/metapals/micro-services/metapals-server-lite --workspace-mount-consistency cached --id-label devcontainer.local_folder=/Users/tonysong/Projects/metapals/micro-services/metapals-server-lite --id-label devcontainer.config_file=/Users/tonysong/Projects/metapals/micro-services/metapals-server-lite/.devcontainer/devcontainer.json --log-level debug --log-format json --config /Users/tonysong/Projects/metapals/micro-services/metapals-server-lite/.devcontainer/devcontainer.json --default-user-env-probe loginInteractiveShell --remove-existing-container --mount type=volume,source=vscode,target=/vscode,external=true --skip-post-create --update-remote-user-uid-default on --mount-workspace-git-root true
[2023-06-24T06:45:40.695Z] Exit code 1

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


python312Packages.pysnmp-lextudio: 6.0.10 -> 6.0.11, python312Packages.atenpdu: 0.6.1 -> 0.6.2

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 296572 run on x86_64-linux 1

14 packages built:

home-assistant-component-tests.brother
home-assistant-component-tests.snmp
python311Packages.atenpdu
python311Packages.atenpdu.dist
python311Packages.brother
python311Packages.brother.dist
python311Packages.pysnmp-lextudio
python311Packages.pysnmp-lextudio.dist
python312Packages.atenpdu
python312Packages.atenpdu.dist
python312Packages.brother
python312Packages.brother.dist
python312Packages.pysnmp-lextudio
python312Packages.pysnmp-lextudio.dist


global and local installation of knex-migrator need to be compatible

It can happen that your global installation does not support the version Ghost has installed locally. And then the database management won't work as expected.
Steps to reproduce:

Install global version 0.1.2
Install local version 0.0.7
knex-migrator init - works
npm start -> still throws an error

To avoid getting problems: use for global and local the same versions.

@kirrg001 something that might be good to take a look at is how Ember-CLI solves the exact same problem. Basically the bin file checks if there's an ember-cli instance in the current directory and loads that if there is rather than the global one.
Their code is here: https://github.com/ember-cli/ember-cli/blob/master/bin/ember
I'm fairly certain it could be adapter to work for this lib.

@acburdine Thanks!

see https://github.com/TryGhost/knex-migrator/pull/20

This is fixed, tested and released. Closing.


Improve parallelism of ir_dist

Closes #468

[ ] CHANGELOG.md updated
[ ] Tests added (For bug fixes or new features)
[ ] Tutorial updated (if necessary)
[ ] test and compare performance with old implementation and full wu2020 dataset.

In general this works well (also out-of-machine computing with dask), but:

the tqdm progressbar is not compatible with the dask backend, or in fact pretty much any backend except for loky

instead, one can set the verbosity to ~5 to get some updates printed to the console. To make this nicer, maybe I can check if the backend is supported.

for some reason it takes ~15 seconds to spawn the loky worker threads. When rerunning the function, this is instant. Not sure what the threads are doing initially.

for some reason it takes ~15 seconds to spawn the loky worker threads. When rerunning the function, this is instant. Not sure what the threads are doing initially.

the reason was that the worker threads all import scirpy from scratch. It turned out to be quite slow because of import airr. I now deferred importing the airr package until it's needed (only in some IO functions) and this brought down the import time to ~1s which is acceptable and mostly due to scanpy.

@felixpetschko fyi
In this PR I change how the ParallelDistanceCalculator handles parallel processing. It now uses joblib.Parallel that (amongst others) allows to use dask as a backend for multi-machine computing.
But I think your reimplementation of the hamming distance calculator anyway doesn't need this.


FABRIC_E_IMAGEBUILDER_UNEXPECTED_ERROR: The remote server returned an error: (404)

I have using a ps1 script to deployment my SF setup.  But I am getting this error and not really sure what it means.  Any idea here?
PowerShell : New-AzureRmResourceGroupDeployment : 12:05:40 PM - Resource Microsoft.ServiceFabric/clusters/applicationTypes/versions
At C:\Users\Owner\Source\Repos\Xbox.Ambassadors\RnR.AzureDeploymentTemplate\RnR.AzureDeploymentTemplate\RnR.AzureDeploymentTemplate\Deployments\DeployMixerPreview.ps1:1 char:1
+ PowerShell -File ..\Deploy-AzureResourceGroup.ps1 `
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (New-AzureRmReso...Types/versions :String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

'mixer-pre/PatchOrchestrationApplicationType/1.2.2' failed with message '{
  "status": "Failed",
  "error": {
    "code": "ResourceDeploymentFailure",
    "message": "The resource operation completed with terminal provisioning state 'Failed'.",
    "details": [
      {
        "code": "ClusterChildResourceOperationFailed",
        "message": "Resource operation failed. Operation: CreateOrUpdate. Error details: {\r\n  \"Details\":
\"FABRIC_E_IMAGEBUILDER_UNEXPECTED_ERROR: The remote server returned an error: (404) Not Found.\"\r\n}",
        "details": []
      }
    ]
  }
}'
At C:\Users\Owner\Source\Repos\Xbox.Ambassadors\RnR.AzureDeploymentTemplate\RnR.AzureDeploymentTemplate\RnR.AzureDeploy
mentTemplate\Deploy-AzureResourceGroup.ps1:322 char:5
+     New-AzureRmResourceGroupDeployment -Name ((Get-ChildItem $Templat ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [New-AzureRmResourceGroupDeployment], Exception
    + FullyQualifiedErrorId : Microsoft.Azure.Commands.ResourceManager.Cmdlets.Implementation.NewAzureResourceGroupDep
   loymentCmdlet

New-AzureRmResourceGroupDeployment : 12:05:40 PM - Resource operation failed. Operation: CreateOrUpdate. Error
details: {
  "Details": "FABRIC_E_IMAGEBUILDER_UNEXPECTED_ERROR: The remote server returned an error: (404) Not Found."
}
At C:\Users\Owner\Source\Repos\Xbox.Ambassadors\RnR.AzureDeploymentTemplate\RnR.AzureDeploymentTemplate\RnR.AzureDeploy
mentTemplate\Deploy-AzureResourceGroup.ps1:322 char:5
+     New-AzureRmResourceGroupDeployment -Name ((Get-ChildItem $Templat ...

here are details from my deployment template in regards to PatchOrchestrationApplication.
 {
      "apiVersion": "2017-07-01-preview",
      "type": "Microsoft.ServiceFabric/clusters/applications",
      "name": "[concat(parameters('clusterName'), '/', variables('patchOrchestrationName'))]",
      "location": "[resourceGroup().location]",
      "tags": {
        "resourceType": "Application",
        "clusterName": "[parameters('clusterName')]",
        "displayName": "PatchOrchestrationApplication"
      },
      "dependsOn": [
        "[concat('Microsoft.ServiceFabric/clusters/', parameters('clusterName'), '/applicationTypes/', variables('patchOrchestrationTypeName'), '/versions/', variables('patchOrchestrationTypeVersion'))]"
      ],
      "properties": {
        "typeName": "[variables('patchOrchestrationTypeName')]",
        "typeVersion": "[variables('patchOrchestrationTypeVersion')]",
        "upgradePolicy": {
          "forceRestart": false,
          "rollingUpgradeMonitoringPolicy": {
            "healthCheckWaitDuration": "00:02:00.0",
            "healthCheckStableDuration": "00:05:00.0",
            "healthCheckRetryTimeout": "00:10:00.0",
            "upgradeTimeout": "01:00:00.0",
            "upgradeDomainTimeout": "00:20:00.0"
          },
          "applicationHealthPolicy": {
            "considerWarningAsError": "false",
            "maxPercentUnhealthyDeployedApplications": "50",
            "defaultServiceTypeHealthPolicy": {
              "maxPercentUnhealthyServices": "50",
              "maxPercentUnhealthyPartitionsPerService": "50",
              "maxPercentUnhealthyReplicasPerPartition": "50"
            }
          }
        }
      }

Just checking with you to see if the issue still exists. To mitigate, retrying the deployment should take care of it. In order for us to root cause, could you provide the resource id of the cluster (You can get it from the properties link on the left side) and date of this issue (timestamp is in the pasted log).

im running into the same issue.
was this ever resolved?
{"status":"Failed","error":{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"Conflict","message":"{\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\",\r\n    \"details\": [\r\n      {\r\n        \"code\": \"ClusterChildResourceOperationFailed\",\r\n        \"message\": \"Resource operation failed. Operation: CreateOrUpdate. Error details: {\\r\\n  \\\"Details\\\": \\\"FABRIC_E_IMAGEBUILDER_VALIDATION_ERROR: DOWNLOAD PATH SANITIZED\\\"\\r\\n}\",\r\n        \"details\": []\r\n      }\r\n    ]\r\n  }\r\n}"}]}}


Support encryption on overlay networking

Requesting encrypted networking support in 2.0.
Options:
optional ipsec backend for flannel
https://github.com/coreos/flannel/blob/master/Documentation/backends.md#ipsec
add support for weave-net
https://www.weave.works/oss/net/

ipsec backend for flannel is still experimental: https://github.com/coreos/flannel/issues/6
@leodotcloud will have an answer on the direction for this.

It's on my TODO list.

@superseb @leodotcloud
Since there's already pull request for weave network encryption:
https://github.com/rancher/rke/pull/781
Can you just integrate weave network to Rancher 2.0 at earlier?
We're running Rancher1.x on production, and Rancher 1.x has builtin ipsec mode on network, the network encryption feature is now the only one reason to stop us moving to Rancher 2.x. Thanks.
https://github.com/rancher/rancher/issues/15194
https://github.com/rancher/rancher/issues/12909

I'd still vote to have it very soon in flannel/calico ;-)

The option we would use will be to add in the ability to use weave in Rancher 2.0, which is this issue: https://github.com/rancher/rancher/issues/12909


Make #2547 merge-able with dev

Description of the pull request:
This pulls #2547 up to be merge-able with dev. I don't think there's any reason not to include this in 0.9.
Fixes the following issues:
#2545
Reviewers:
@guilhermeaiolfi @fskreuz

I'm not proud of the three merges there, but I gave up on rebasing because every commit had painfully interwoven conflicts...


Fix memory leak due to hessian2、RpcContext、FutureContext

What is the purpose of the change
Fix memory leak due to hessian2、RpcContext、FutureContext
More detail from #7770
Verifying this change

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

I'm not sure there's a more elegant way to remove RpcContext and FutureContext

Codecov Report

Merging #7771 (f621817) into master (fc1c1a3) will decrease coverage by 0.19%.
The diff coverage is 76.92%.

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


Detect false recombinations by checking if we have mutations with very few inheritors

This example from the Long ARG (node 544949) shows a clear example where mutations have been subsequently reverted, and end up being inherited by very few descendants:

Similar to #174

(These mutations might not actually be on that node though, I haven't looked properly)


Added SIGNL4 as Alertmanager Webhook Receiver

The initial idea was to add a deeper, code-based integration. However, I was told to use the simple webhook right now.

The DCO needs to be on all commits, squashing is generally easiest.

I think it looks fine now. Thanks @brian-brazil for all your support.

Thanks!


use_local_storage Not Storing Values as Expected

Describe the bug
I'm experiencing an issue with use_local_storage. It appears that values are not being stored or retrieved correctly.
Reproduce

cargo leptos new --git https://github.com/leptos-rs/start-axum
Add leptos-use to Cargo.toml

leptos-use = {version = "0.9.0", features = ["ssr"]}

Replace app.rs with

use crate::error_template::{AppError, ErrorTemplate};
use leptos::*;
use leptos_meta::*;
use leptos_router::*;
use leptos_use::storage::{use_local_storage, StringCodec};

#[component]
pub fn App() -> impl IntoView {
    // Provides context that manages stylesheets, titles, meta tags, etc.
    provide_meta_context();

    view! {


CSL locales with just a two-letter code (ar, eu) do not work

CSL locales with just a two-letter code (ar, eu) do not work.
pandoc -s -F pandoc-citeproc -t plain << EOT

@item1

# References

---
locale: ar
references:
- id: item1
  type: book
  editor:
  - family: Doe
    given: John
  issued:
  - year: 2015
  title: XXX
...
EOT

Output:
pandoc-citeproc: /Users/nick/Library/Haskell/share/ghc-7.8.3-x86_64/pandoc-citeproc-0.7.1/locales/locales-ar-AR.xml: openBinaryFile: does not exist (No such file or directory)
pandoc: Error running filter pandoc-citeproc

It seems https://github.com/jgm/pandoc-citeproc/blob/master/src/Text/CSL/Data.hs#L74 needs to be updated.
As to locale: eu (Basque), this seems to be silently ignored.
pandoc -s -F pandoc-citeproc -t plain << EOT

@item1

# References

---
locale: eu
references:
- id: item1
  type: book
  editor:
  - family: Doe
    given: John
  issued:
  - year: 2015
  title: XXX
...
EOT

Actual:
Doe, John, ed. 2015. _XXX_.

Expected:
Doe, John, arg. 2015. _XXX_.

If eu needs to be configured explicitly then la for Latin should be added, too (la doesn't have a CSL locale file yet but this might change.)

+++ nickbart1980 [Jun 04 15 06:10 ]:

CSL locales with just a two-letter code (ar, eu) do not work.

I've updated the locale files, and I get proper results with both
your examples.  However, I don't think the changes I made would have
affected these cases, so I'm wondering whether there's some other
problem on your end (e.g. with installation).
Is your version installed from the binary package or from
Can you try with the latest?  (Be sure to cabal install, not just
cabal build, to make sure that the data files are installed in the
right place.)

I had been using the latest, installed with cabal install but reinstalled just in case.
Now, it works as expected if ~/Library/Haskell/share/ghc-7.8.3-x86_64/pandoc-citeproc-0.7.1/locales/locales-ar-AR.xml exists – but this file should not be there in the first place (ar-AR is an incorrect tag, that would be Arabic/Argentina). Note that no such file exists in https://github.com/citation-style-language/locales. The only correct language tag for Modern Standard Arabic is ar.
Now with locales-ar.xml but without locales-ar-AR.xml I still get the error from my original post. Same for eu (no error, but “ed.” rather than “arg.”).
I still suspect this is because neither ar nor eu are listed in https://github.com/jgm/pandoc-citeproc/blob/master/src/Text/CSL/Data.hs.


This is more a javascript question though - https://github.com/blueimp/Gallery


Type argument incorrectly inferred from union type

🔎 Search Terms

generics
type argument
inference
union type

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about type parameters/arguments and inference.

⏯ Playground Link
https://www.typescriptlang.org/play?#code/CYUwxgNghgTiAEYD2A7AzgF3gMwFzwB4BBAPgAoN8BtIgXQEp4BeE+ANyQEtgBuAKFCRYCZOiwAPaphicUAc1rwAPvCooArgFsARiBi1+fbGXH0eQA
💻 Code
declare const f: <A>(t: [A]) => void;
declare const x: [string] | [number];

f(x);

🙁 Actual behavior
TypeScript infers the type argument as string.
🙂 Expected behavior
TypeScript infers the type argument as string | number.
Additional information about the issue
This issue appears to describe a similar problem with functions: https://github.com/microsoft/TypeScript/issues/52295.
Reduced test case:
type A = { a: string };
type B = { b: string };

// Parameter types
declare const f: <A>(ff: (x: A) => void) => void;
declare const x: ((x: A) => void) | ((x: B) => void);
// Inferred type argument: A
// Expected: A & B
f(x);

// Return types
declare const g: <A>(gg: () => A) => void;
declare const y: (() => A) | (() => B);
// Inferred type argument: A
// Expected: A | B
g(y);

The difference is that in those other examples the type parameter appears in what it considers a naked~ position. It unionify candidates from such positions.
The union of tuples is different since the position is not naked, it’s an element inside a tuple type.
This isn’t incorrect. How different inferences work is often just heuristic-based. As far as I know, this is working as intended - it’s not a bug. It can be treated as a feature request though.

If it's a feature request then it duplicates #44312

Thanks for the responses. If it provides more context, below is the original real world problem I encountered before reducing it down to the examples you see above.
export interface Reader<R, A> {
  (r: R): A;
}

declare const chain: <A, R, B>(
  f: (a: A) => Reader<R, B>
) => (ma: Reader<R, A>) => Reader<R, B>;

declare const boolean: boolean;
declare const a: Reader<string, number>;
declare const b: Reader<number, number>;

// Type argument `R` inferred as `string`.
// Ideally this would be `string | number`.
chain(() =>
  // Error: Type 'string' is not assignable to type 'number'.
  boolean ? a : b
);

and
interface Left<E> {
  readonly _tag: "Left";
  readonly left: E;
}

interface Right<A> {
  readonly _tag: "Right";
  readonly right: A;
}

export type Either<E, A> = Left<E> | Right<A>;

declare const chain: <E, A, B>(
  f: (a: A) => Either<E, B>
) => (ma: Either<E, A>) => Either<E, B>;

declare const boolean: boolean;
declare const a: Either<string, string>;
declare const b: Either<string, number>;

// Type argument `B` inferred as `string`.
// Ideally this would be `string | number`.
chain(() =>
  // Error: Type 'number' is not assignable to type 'string'.
  boolean ? a : b
);

There’s also the fact to consider that, in general, G<T | U> cannot be safely substituted for G<T> | G<U> for arbitrary G, so TS won’t even attempt inferences that imply such a substitution.

The "Actual Behavior" doesn't match the actual behavior upon opening the play ground link - which is this
declare const f: <A>(t: [A]) => void;
declare const x: [string] | [number];

f(x); // error
//~
// Argument of type '[string] | [number]' is not assignable to parameter of type '[string]'.
//   Type '[number]' is not assignable to type '[string]'.
//     Type 'number' is not assignable to type 'string'.(2345)
// const f: <string>(t: [string]) => void```

That error makes sense because [string] | [number] cannot be equal to [something].
It could argued that TypeScript should infer the A=[string | number] because it's smallest inference that satisfies, (although I think it's kinder to be strict).
No problem here:
declare const g: <A>(t: A) => void;
declare const y: [string] | [number];
g(y); // no error

The description "Actual Behavior" doesn't match the actual behavior

@craigphicks The OP claims that Actual Behavior is:

TypeScript infers the type argument as string.

which is consistent with the error message:

Argument of type '[string] | [number]' is not assignable to parameter of type '[string]'.

i.e. TS infers A = string for the call to f and subsequently produces an error upon trying to pass a [string] | [number] (that is, x) to a parameter typed as [A] = [string].  So the description of actual behavior looks correct to me.

It doesn't seem right that the inferred type depends upon the order, but as long as it fails, then it really isn't a problem.


Session does not persist L5.1

Hello folks, good morning.
I have a problem I can not save my user authentication sessions.
I could just adding index.php in the session_start ();
What I think has something strange, someone could help me solve this problem?

@GrahamCampbell I did not understand.


[rm nixpkgs] make HEAD request to BinaryCache to ensure binary is cached

Summary
This PR makes a HTTP HEAD request to https://cache.nixos.org/<hash>.narinfo to verify that the binary is indeed cached.
If not, we fallback to the legacy path that is dependent on downloading nixpkgs.
How was it tested?
I could do devbox shell in a project that has hello package. I inserted some print statements to verify that the HEAD
response was 200 and that the caching for the boolean value of isNarInfoInCache was working.

needs revision so marking as Draft.

Current dependencies on/for this PR:

main

PR #1318   👈

This comment was auto-generated by Graphite.

Hmm, I need to make some fixes...moving to draft.

@gcurtis PTAL

Gah, I really dislike the implementation in this PR. It makes the code brittle and harder to reason about. Inclined to try other approaches.


Log message when Tablet has been unloading for over 15 minutes

Related to #4539

I left a few comments, primarily around how the cache is implemented and used I think should be changed. But the other big thing is I am thinking this custom logic would be better implemented using the Filter API that is part of log4j2. I think we could create a custom filter and put the conditional logic in there instead of having to implement a new ConditionalLogger class. The nice thing about a filter too is it is configurable at runtime easily.

But we are coding to the slf4j api. A user using log4j2 is a deployment decision, someone could swap it out for logback or something else.

But we are coding to the slf4j api. A user using log4j2 is a deployment decision, someone could swap it out for logback or something else.

That's a fair point, I'm not sure it's very likely someone would replace that too often but it's possible. I guess in that guess it wouldn't break per se, but it would just start logging a lot of extra stuff of course.

The changes here caused a compilation issue in my dev environment because apparently log4j-api is a transitive dependency of log4j-1.2-api and is needed indirectly because some of the classes that Category requires are in log4j-api and the 1.2 Logger class extends Category. So, by adding an entry for log4j-api for the test scope, it was prevented from being added transitively to the compile scope for log4j-1.2-api.
It did work in some environments, though... because log4j-api leaks into the compile time classpath from a lot of places (including Maven itself), but it's not strictly correct, and does break the build in some environments.
I'm working on a fix for this, as well as some other cleanups.


"export midi" only exports bass clef

"export midi" gives midi file in which only left hand can be heard (using windows media player). musescore crashes when opening exported midi. midi is read correctly by synthesia

Duplicate of #22


feat: support mvcc

tracking:

[ ] support tombstone.
[ ] support snapshot read api

It seems that support MVCC for cache storage layer is not a good idea. It makes consistency confusing.


chore(angular): add to index

👋 Thanks for creating a pull request! 🚀 Checkout the storybook we've created for it: angular


Take Bazel's patches to source artifacts into account

Patches to source artifacts are not included in the Analyzer result and hence not downloaded or scanned.

Instead of adding patches to the ORT result, we should consider doing this similar to like planned for the BitBake integration: To upload patched source artifacts to a custom storage, like S3, from where the scanner would use them.

Somewhat relates to isModified detection (also see https://github.com/oss-review-toolkit/ort/issues/6493) as packages with patches should always be treated as modified packages.


Remove FromString method of enumerations

close #98

Do we want to completely remove String()? It can be useful in some cases.

Do we want to completely remove String()? It can be useful in some cases.

they must be saved, issue #98 didn't mention their removal


Make compute assists async and fix message

Fixes #44730.
Fixes #44731.
For "fixes" there were similar bugs that I've fixed as well.

Thanks for the PR! It look great.
As you might be aware, PRs for the sdk repository are automatically copied into a different system that we use named Gerrit, and that's where we do reviews. The link to the CL in Gerrit is https://dart-review.googlesource.com/c/sdk/+/180740.

Thanks for the PR! It look great.
As you might be aware, PRs for the sdk repository are automatically copied into a different system that we use named Gerrit, and that's where we do reviews. The link to the CL in Gerrit is https://dart-review.googlesource.com/c/sdk/+/180740.


Remove support for old combattag

@rourke750

Can one of the admins verify this patch?  Type 'ok to test' to test.

@Maxopoly Try to do "ok to test"

ok to test

implying the bot ever listens to me

Eh I think it should be fine. I had a wrong version of CT+ in my local pom that didnt have the changes rourke made to CT+. So when I saw the red errors after removing the old combat tag imports I thought the listener was from the old combattag.

Good to merge?

yes


Add TLS event

Note
Because the main branch of piglet still works with giganto-client 0.7.0, this branch starts from 0.7.0.
This branch can be merged if the following jobs are finished.
Pending

Merge aicers/piglet#540 (Then, piglet can work with giganto-client 0.9.0)
Manually rebase this branch to main

Codecov Report
Patch coverage has no change and project coverage change: -2.11 :warning:

Comparison is base (2f0ac47) 43.86% compared to head (b06cbfd) 41.76%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
- Coverage   43.86%   41.76%   -2.11%
==========================================
  Files          11       11
  Lines         497      522      +25
==========================================
  Hits          218      218
- Misses        279      304      +25

Impacted Files
Coverage Δ

src/ingest.rs
100.00% <ø> (ø)

src/ingest/network.rs
5.29% <0.00%> (-0.70%)
:arrow_down:

src/publish/range.rs
11.11% <0.00%> (-0.66%)
:arrow_down:

src/publish/stream.rs
0.00% <0.00%> (ø)

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


Documentation on stub runner boot, spring cloud services, and opening multiple ports

It's already documented in the cf migration guide here https://cloud.spring.io/spring-cloud-pipelines/single/spring-cloud-pipelines.html#_3_8_enable_stubs_for_smoke_tests


error-reporting: Add deleteEvents and getEvents module methods

The cloud-errors-nodejs repository has PR #99 that already contains some preliminary work.

This issue was moved to https://github.com/googleapis/nodejs-error-reporting/issues/9.


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2024-06-30)
Features

Implemented a new route that allowa to delete a product in a shelf (70a4f44)
Implemented route that allow to add a new product in a shelf (5ce48be)
Implemented route that return all shelf generated (72967d7)
Implemented route to add product in a shelf (8d65055)
logistic: completed temperature socketio notification (9e4b4e2)
logistic: completed temperature socketio notification, added env range configuration (c995d83)
logistic: product event on socket-io, first demo version (005002f)
logistic: socketio temperature alert example implementation (df84ae1)
Validate product existence when creating and modifying shelves (3adbe73)

Bug Fixes

logistic: product alert simplified (05b4ca4)
UI: variable name shelf corrected (39494f2)
Updated method that modified shelf data (ba03240)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/WMS-Corporation/wms-logisticControl-service/releases/tag/v1.1.0 :sunflower:


Two questions about vcs command and base_interface.hpp missing

I find problems when i install lbr_fri_ros2_stack:
The first problem I meet is that when i follow the steps . The command ''vcs'' is not defined but i cannot find an avaliable vcs-installing command for Ubuntu 20.04.

Most of installing methods shows

    ``Collecting vcs
      Using cached vcs-0.4.0.tar.gz (482 kB)
        ERROR: Command errored out with exit status 1:
         command: /usr/bin/python3 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-2_kpq018/vcs/setup.py'"'"'; __file__='"'"'/tmp/pip-install-2_kpq018/vcs/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-install-2_kpq018/vcs/pip-egg-info
             cwd: /tmp/pip-install-2_kpq018/vcs/
        Complete output (6 lines):
        Traceback (most recent call last):
          File "<string>", line 1, in <module>
          File "/tmp/pip-install-2_kpq018/vcs/setup.py", line 11
            except IOError, err:
                          ^
        SyntaxError: invalid syntax
        ----------------------------------------
    ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.``

2.When i skip the vcs installation and repos installation, I git clone the fri lib according to the repos.yml.  After colcon, the bash showed '' No such file or directory
13 | #include "hardware_interface/base_interface.hpp"''

Thanks @Tianhuanyu, try installing vcstool here: http://wiki.ros.org/vcstool

I find now when i try ''rosdep install --from-paths src --ignore-src -r -y''
ERROR: the following packages/stacks could not have their rosdep keys resolved
to system dependencies:
lbr_description: No definition of [joint_state_publisher_gui] for OS version [focal]
lbr_bringup: No definition of [gazebo_ros2_control] for OS version [focal]
lbr_hardware_interface: No definition of [ros2_control] for OS version [focal]
lbr_moveit: No definition of [moveit] for OS version [focal]
Continuing to install resolvable dependencies...
#All required rosdeps installed successfully

should i install the corresponding libs respectively?

Which directory are you running this command from?

I am assuming that installing vcstool worked?

yes, vcs worked

I run the command in workspace of ROS2.  it looks like this in terminal ~/ros2_ws$.

Try running $ sudo apt update

Thanks. It works!

Great, closing this issue now. It will be fully resolved when #64 is merged.


Expected lua number

I got that with Troubletoggle lsp_references

Development on the main branch is EOL.
Trouble has been rewritten and will be merged in main soon.
This issue/feature either no longer exists or has been implemented on dev.
For more info, see https://github.com/folke/trouble.nvim/tree/dev


Move console apis to tier 2, per what our docs say

and update generator to tolerate it
see our docs here: https://docs.openshift.com/container-platform/4.9/rest_api/understanding-api-support-tiers.html#mapping-support-tiers-to-openshift-api-groups_understanding-api-tiers

@spadgett fyi
@openshift/api-reviewers ptal

@openshift/api-reviewers bump


Extension requires me to continually create new contexts

I started Podman Desktop, and the sandbox instance I was using yesterday has 'unknown' status. Ok, no problem, it must just be logged out. But to log in again I had to:

Find sandbox website
Login to RedHat
Navigate through 3 or 4 pages to get to oc login command
Open terminal
Execute oc login

... which created a new context instead of using the one I had created yesterday. I can set this context inside of Podman Desktop using the system tray, but my Sandbox instance still says 'unknown'. I can't edit the instance to change the context; I can't create a new instance with the new context b/c it already exists. The only way to show the sandbox connection as running again is to create a new instance with a third context.

It's painful to have to go to the sandbox repeatedly to get the (re-)login command; this should be possible within PD.
I should be able to continue using one Sandbox instance indefinitely.

See https://github.com/redhat-developer/podman-desktop-sandbox-ext/issues/20#issuecomment-1527108951

For now it is manual process. Going to improve it next sprints.


Replacing the file in the zip makes the position of the file move to the bottom of the list

Hey hay!
So, I used .addFile() to replace a file into a zip.
But after the process is complete and I refresh the FileHeaders list, the file position moves to the bottom.

Yeah, when an entry in a zip file is replaced, it basically removes that entry from that position in the zip file, and appends it to the end of the zip file. Zip4j cannot override in the existing location of the entry because the size of the entry might change.

Oh okay. Thanks!


How to debug tests ?

Hi,
Maybe this arguments is discussed yet but I don't have found anything related to my question.
I have a test that fail and I want to understand what happen, so I have run the test with gdb and set a break point to the point of interest. The test run but the break point is ignored! Here my gdb output
(gdb) r
Starting program: /my_build_folder/uv_tests ipc_send_recv_pipe_inprocess
[Thread debugging using libthread_db enabled]
[New Thread 1 (LWP 1)]
[New LWP    2        ]
[LWP    2         exited]
[New Thread 2        ]
not ok 1 - ipc_send_recv_pipe_inprocess
# exit code 6
# Output from process `ipc_send_recv_pipe_inprocess`:
# Assertion failed: fd > STDERR_FILENO, file /home/libuv/src/unix/core.c, line 539, function uv__close

Program exited with code 06.
(gdb) bt
No stack.
(gdb) b core.c:537
Breakpoint 1 at 0xfffffd7c441c8e8e: file /home/libuv/src/unix/core.c, line 537.
(gdb) r
Starting program: /my_build_folder/uv_tests ipc_send_recv_pipe_inprocess
[Thread debugging using libthread_db enabled]
[New Thread 1 (LWP 1)]
[New LWP    2        ]
[LWP    2         exited]
[New Thread 2        ]
not ok 1 - ipc_send_recv_pipe_inprocess
# exit code 6
# Output from process `ipc_send_recv_pipe_inprocess`:
# Assertion failed: fd > STDERR_FILENO, file /home/libuv/src/unix/core.c, line 539, function uv__close

Program exited with code 06.

for sure I ignore something important! How are debugged the test? Some tips for my case?

@Zikoel could you open an issue @ https://github.com/libuv/help. I would gladly answer there. We use this only for issues in the code. Thanks


last overhaul of ZJSON

This commit implements one last overhaul to the ZJSON spec
to bring it into alignment with where the Zed design landed.
Importantly, types and type values are consistently encoded
using JSON structure so that clients can have an easier time
parsing and managing the Zed type context for a stream of
ZJSON records.  This also means that type-values are encoded
with JSON structure instead of using ZSON type strings that
in turn needed to be parsed.
This change follows the compiler AST pattern for Zed types
but keeps the simple encoding of values based on nested
JSON arrays of strings.  The types are now more verbose but
easier to parse at the client.  Also, the aliases hack was
removed in favor of Zed's model of embedded typedefs.
We also eliminated the use of package joe and removed it
as this was the last remaining vestige of joe-ness.
Closes #1780
Closes #2185
Closes #2376

I put this up as a draft since we need to update the python client (hence failing tests) and the zealot lib in the app.   Otherwise (aside from enum), ZJSON is officially done!

Maybe make the Map type return an array of arrays in the values? Right now it's flat:
    {
      kind: 'map',
      key_type: { kind: 'primitive', name: 'string' },
      val_type: { kind: 'primitive', name: 'int64' }
    } [ 'best', '1', 'worst', '0' ]

What do you think about making the Map type return an array of arrays in the values? Right now it's flat:
    {
      kind: 'map',
      key_type: { kind: 'primitive', name: 'string' },
      val_type: { kind: 'primitive', name: 'int64' }
    }

    [ 'best', '1', 'worst', '0' ]

So, you want   [ ['best', '1'],['worst', '0'] ], right?


Python 3 support

2buntu in its current form has not been tested against Python 3 at all. This bug consists of three steps:

checking each individual component to verify Python 3 compatibility
implementing a transition package (like six) to enable both Py2 and Py3 compatibility
exhaustive testing to ensure proper functionality

Nothing got broken. Closing.


Exclude project icon from gitignore

Task/Issue URL:
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

#4876  👈
develop

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

Join @mikescamell and the rest of your teammates on  Graphite


Pass session data to torii close

Related https://github.com/simplabs/ember-simple-auth/issues/295
This PR passes the session information to torii using the options argument for the close method. This is useful for doing things like revoking access tokens.

Can you retarget this on the ember-chli-addon branch?

Closing to retarget ember-cli-addon branch on https://github.com/simplabs/ember-simple-auth/pull/585


Add support for sub-directory for stacked create view

Hello @FilledStacks,
Many thanks for continued development of this stack of package.
I just started using the cli and I love it.
I would like to suggest addition of sub-directory support to stacked create view to enable grouping of views.
e.g if i wanted to create a view in the folder lib/ui/views/users/profile_view/
I would want to use the cli with command like stacked create view users/profile
This can also apply to other stacked create commands

@Henriquedn thanks for the issue. I like this request, I think we've now had it a few times.
I'll mark as a new feature and we'll tackle this when we free up. We're currently all hands on deck with the updates to get stacked ready for Flutter Web.
This is a noted request. I'll comment here as soon as it put it into the pipeline.

We have added #870 to our Improvements project and this request will be tracked there. We'll start working on this soon.
Thanks for creating the issue @Henriquedn


chore: add Muhamad Nur Hanif  as a contributor

I want add my name  Muhamad Nur Hanif as a contributor

Keren


Reusable suite-local fixtures is allocated after suite's beforeAll()

Update: munit v1.0.0-M6.
The following code
class TestMUnitResource extends munit.FunSuite {
  val resource = new Fixture[Double]("Double resource") {
    private var d: Double = Double.NaN

    def apply() = d

    override def beforeAll(): Unit = {
      println("Acquiring resource")
      d = 4.0
    }

    override def afterAll(): Unit = {
      println(s"Releasing resource $d")
      d = Double.NaN
    }
  }

  override def munitFixtures = List(resource)

  test("test1") {
    println(s"Using resource [${resource()}]")
  }

  override def beforeAll(): Unit = {
    println(s"Suite beforeAll: ${resource()}")
  }

  override def afterAll(): Unit = {
    println(s"Suite afterAll: ${resource()}")
  }
}

produces
Suite beforeAll: NaN
Acquiring resource
Using resource [4.0]
Suite afterAll: 4.0
Releasing resource 4.0

It looks that suite-local fixture is allocated after beforeAll and released after afterAll . Is this the desired behavior?

Is this matching the behavior described in the breaking changes for munit 1.x?
https://github.com/scalameta/munit/releases/tag/v1.0.0-M1

Is this matching the behavior described in the breaking changes for munit 1.x? https://github.com/scalameta/munit/releases/tag/v1.0.0-M1

It does indeed.

Done.

But yes, this is the desired behaviour in the v1.0 release line.

What is the rationale of such change, if I may ask?
Example against such behavior: I would like to use suite-local fixture to create a shared api client, login some users in beforeAll, logout the users in afterAll. Not possible, as beforeAll runs before the fixture is ready.

Looking at the "issue" I realized that the fixtures (including the suite-local fixture) are teared town in the exact same order they are initialized.
Wouldn't it make  more sense to call the afterAll and afterEach function on the reverse order of the fixtures ?
So that the last created resources are destroyed first ?
So that FixtureOrderFrameworkSuite would look like (for only 1 test):
beforeAll(ad-hoc)
beforeAll(a)
beforeAll(b)
beforeEach(ad-hoc, 1)
beforeEach(a, 1)
beforeEach(b, 1)
test(1)
afterEach(b, 1)
afterEach(a, 1)
afterEach(ad-hoc, 1)
afterAll(b)
afterAll(a)
afterAll(ad-hoc)

@kamilkloch The motivation for this behavior is based on the (perhaps false?) assumption that fixtures should be independent of each other and it was therefore a nice simplification to remove the special-case for the override def beforeAll/afterAll methods. With the new logic, suite-local (override def beforeAll/afterAll) methods are implemented as regular fixtures. You have full control over the order of fixtures by overriding override def munitFixtures, and override def beforeAll/afterAll are pure convenience methods to implement a normal fixture.

So that the last created resources are destroyed first ?

@mzuehlke the motivation for this change was based on the assumption that fixtures are independent and the order matter should therefore ideally not matter. If you have multiple fixtures that dependent on each then you have two options:

merge the fixtures into a single fixture
separate each fixture into two fixtures that each contain only the before/after parts, and then explicitly specify the reverse order

Is that a suitable solution for your use-case? I'm on the fence about merging your PR since it introduced a third ordering that has never been used before. In the 0.x series, there was only a special case for the suite-local fixtures that made before methods runs first and after methods run last.
Writing this comment, I realized that we can avoid the v1.x breaking change by using the "separate each fixture into two fixtures" trick for suite-local methods. This would preserve the ordering from the 0.x series while giving users full-control over how to define the ordering of fixtures.

I like your approach, of keeping munit simple on the one hand and
having multiple fixtures that rely on each other is probably not leading to more readable test.
 Maybe they are worth to be but somewhere in the Fixture documentation.
When preparing my PR especially one comment nearly stopped me:
https://github.com/scalameta/munit/blob/a2f1b913c7b04532572cde32af39f7e88329bb59/tests/jvm/src/test/scala/munit/AsyncFixtureSuite.scala#L82-L88

I've opened a new PR to revert to the v0.7.x style ordering: https://github.com/scalameta/munit/pull/724
By reverting we avoid a breaking change from v0.7.x to v1.0.x (once released).
However, the v1 milestones have been around for so long, this is effectively a second breaking change for the ecosystem that did migrate to the v1 milestones.
That is unfortunate, no doubt.
However, in this situation, I believe there have been enough reports about this being surprising behaviour, that it is worth reverting to the old, and perhaps more intuitive, ordering.

Should we close this? Are we ok having the change for 1.0.0?


Mixup returns different combinations for labels and images

The implementation of mixup seems to be wrong. We are supposed to use the same permutation for the labels and images. But the use of 2 separate random_shuffle calls will return a different combination of images and labels.
https://github.com/xudonmao/VMT/blob/39476e4520a915e30327d190867c3122206b7243/codebase/models/extra_layers.py#L76

@kdkalvik
Thanks for pointing this out. We have fixed the bug to have the same combination of images and labels. After fixing this bug, the performance of VMT can be further improved for almost all the tasks.
By the way, we need to change the values of some hyperparameters for the updated version. For example, for MNIST->SVHN with instance normalization, we change lambda_s and lambda_t to 1 and 0.08 respectively.


Organize listChildren() into separate methods

The point of this commit is to make the listArchives() method as simple as possible and to pull out any reusable code. This will help with any redesign DistributionAnalyzer, e.g., for parallelization.


V2 - Need API to edit CLA Group name and descirption

As of now there is no option to edit CLA Group name and description. Need an API to CLA Group name and descirption and update the name where ever applicable

Created PR: https://github.com/communitybridge/easycla/pull/1920/files

Update API: https://api-gw.dev.platform.linuxfoundation.org/cla-service/v4/api-docs#operation/updateClaGroup

Tested and observed below issues
1. Endpoint is not added to ACS file and hence only system-admin has access to the endpoint
https://github.com/LF-Engineering/acs-cli/blob/master/services/11-cla-service.yaml
2. When no body is provided 500 is returned

3. Able to update the CLA Group name with already existing CLA Group name.

Existing CLA Group Name

Added ACS-CLI PR to add permissions to the API-GW/ACS: https://github.com/LF-Engineering/acs-cli/pull/318

Merged PR - deploying to DEV.

Creating PR for duplicate or missing parameters: https://github.com/communitybridge/easycla/pull/1926

Issue3 is still reproducible and observed another issue. All other issues are fixed. Verfied Issue with only project manager role. Other roles will be verified in #1933

Able to update the CLA Group name with already existing CLA Group name

4. Duplicate description is not allowed. Description should not be restricted

Sorry, @nirupamav. I'm unable to reproduce the duplicate name allowed scenario. Can we review this issue during our next standup? I want to review your test scenario details.

@dealako
I am still able to reproduce the issue. Below is the data in cla-dev-projects table and getCLAGroup is fetching data from cla-dev-projects table. However in cla-dev-projects-cla-groups old cla group name is displayed

@nirupamav this logic was updated after discussing with @asithade. Can you please re-test?

Issue is still reproducible
CLA Group taht exist for another project

Able to edit with existing cla group name

Added additional logic. See PR: https://github.com/communitybridge/easycla/pull/1998

Verified that the issue is fixed. However observed an issue where I was trying to update description including cla group name of the project, error is displayed. Logged issue #2000


Bamtofastq generate duplicated reads - latest version - 2.30.0

Hi,
When I apply bamtofastq to a bam file (illumina single-end reads aligned with STAR), I do get duplicated reads in the fastq.
Checking the former version -  2.29.2 - it works as expected (without duplicates). Would it be possible that the update (to 2.30.0) "broke" bamtofastq?
Thanks !
Best,
AndreiR

Example:
$ bedtools --version
bedtools v2.31.0

$ echo -e "beholder_mucus-1130/1\t4\t*\t0\t0\t*\t*\t0\t0\tCCAAGGTATCTTTTTCGTATACGATATGAACTGGCAAACCATCTTCGTACTTGGTCATAGTCGCCATATACAAGGATAGTTGTTCTCGTTTAAAACCATCAAAGTCAGGCAACACTAAGGCTTTTGTAAGCTCACGCCCACGGTCTGGAC\tCCCGGGGG=GGGGJGJJ=JJGGJJJJJJJJJJGJGC=JG=JJJJCJJJJCGCGGC=8J1JGGGGGJGGG8GGG=CGG=CGCGGG=GGCGGGGCG=GGGG8J=CGCG=8GCG8=GGCGCCGGCGGGGGCGCCGCGGGC8GCCCCGGGGCGG\tRG:Z:FASTQ\tPL:Z:Illumina\tPU:Z:pu LB:Z:lb SM:Z:sm PG:Z:SNAP\tNM:i:255" > tmp.sam

$ bedtools bamtofastq -i tmp.sam -fq tmp.fq
$ cat cat tmp.fq
@beholder_mucus-1130/1
CCAAGGTATCTTTTTCGTATACGATATGAACTGGCAAACCATCTTCGTACTTGGTCATAGTCGCCATATACAAGGATAGTTGTTCTCGTTTAAAACCATCAAAGTCAGGCAACACTAAGGCTTTTGTAAGCTCACGCCCACGGTCTGGAC
+
CCCGGGGG=GGGGJGJJ=JJGGJJJJJJJJJJGJGC=JG=JJJJCJJJJCGCGGC=8J1JGGGGGJGGG8GGG=CGG=CGCGGG=GGCGGGGCG=GGGG8J=CGCG=8GCG8=GGCGCCGGCGGGGGCGCCGCGGGC8GCCCCGGGGCGG
@beholder_mucus-1130/1
CCAAGGTATCTTTTTCGTATACGATATGAACTGGCAAACCATCTTCGTACTTGGTCATAGTCGCCATATACAAGGATAGTTGTTCTCGTTTAAAACCATCAAAGTCAGGCAACACTAAGGCTTTTGTAAGCTCACGCCCACGGTCTGGAC
+
CCCGGGGG=GGGGJGJJ=JJGGJJJJJJJJJJGJGC=JG=JJJJCJJJJCGCGGC=8J1JGGGGGJGGG8GGG=CGG=CGCGGG=GGCGGGGCG=GGGG8J=CGCG=8GCG8=GGCGCCGGCGGGGGCGCCGCGGGC8GCCCCGGGGCGG


Outdated Screenshot

@K232 commented on Wed Feb 05 2020
Screenshot at https://docs.microsoft.com/de-de/azure/includes/media/functions-run-function-test-local-vs/functions-run-browser-local-vs.png in https://docs.microsoft.com/de-de/azure/azure-functions/functions-create-your-first-function-visual-studio#run-the-function-locally is outdated. It says, the return value should be: "Hello Glenn". In fact, it is: Hello, Glenn
I was not able to update the screenshot as https://github.com/MicrosoftDocs/azure-docs.de-de/blob/master/includes/media/functions-run-function-test-local-vs/functions-run-browser-local-vs.png is not found, and you of course have a global guideline how to mark screenshots etc. so it's probably better you update the screenshot.
Many thanks!

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: 6e88ea08-81ef-2b5a-b9d6-539e85592672
Version Independent ID: f2d8b7b4-906a-d1f5-5201-fc22a7890955
Content: Erstellen Ihrer ersten Funktion in Azure mit Visual Studio
Content Source: articles/azure-functions/functions-create-your-first-function-visual-studio.md

@srvbpigh commented on Wed Feb 05 2020
Hello, @K232
Thank you for your feedback.
We are actively reviewing your comments and will get back to you soon.
Kind regards,
Microsoft DOCS International Team

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

ID: 6e88ea08-81ef-2b5a-b9d6-539e85592672
Version Independent ID: f2d8b7b4-906a-d1f5-5201-fc22a7890955
Content: Create your first function in Azure using Visual Studio
Content Source: articles/azure-functions/functions-create-your-first-function-visual-studio.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi Team, could you please check the issue from user?
Kind regards,
Microsoft DOCS International Team

@K232  Thanks for your feedback! We will investigate and update as appropriate.

@ticianaciminari I have assigned this to content author for further review and update as appropriate.

@ticianaciminari sorry for the delay in fixing this. I've updated it in the private repo. Please reopen if this doesn't address the issue to your satisfaction.  #please-close


Foreach-Object Parallel serialization issue

Description
Powershell omits certain fields in foreach-object -parallel code block, for example Get-ADComputer correctly displays computer name in code block, but return empty string for any extra fields requested in Properties argument. When calling without -Parallel argument everything works as expected.
Steps to reproduce
Get-ADComputer w2 -Properties OperatingSystem | % -Parallel { $_.OperatingSystem }

Expected behavior
Windows 10 Pro

Actual behavior
<nothing>

Environment data
PSVersion                      7.1.0-preview.1
PSEdition                      Core
GitCommitId                    7.1.0-preview.1
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The Parallel parameter doesn't actually serialize anything since it's thread based instead of process based.
This is due to the ActiveDirectory module defining it's own PSPropertyAdapter and registering it on module import.
@PaulHigin I know RunspaceFactory.CreateRunspace has some overloads that take a TypeTable.  I'm guessing no, but is there a publicly accessible way to obtain the current TypeTable? Eh that's probably not feasible anyway without some method of removing things like script members.

So basically I could use something like this, as workaround
Get-ADComputer w2 -Properties OperatingSystem | % -Parallel { $_["OperatingSystem"] }

It is working, but it's inconsistent.

@somescout Yeah that should work.  You said it's inconsistent, do you have an example that doesn't work? Or can otherwise elaborate? (I may be able to suggest a more consistent alternative)

This is another consequence of the fact that each foreach -parallel loop iteration runspace is intialized (or reset) to its default state, rather than the state of the current runspace running the cmdlet.
I have created an Issue (#12240) to track this, and have linked this issue to that one.
Closing this issue.


NPM release wrong source

Seems latest NPM simplecrawler release doesnt contain #288 changes, can you confirm?

Hey, thanks for filing an issue! I don't believe there's any difference between the source at GitHub and npm ATM, but #288 was never merged. Is there a particular problem you're experienced that you believe is related to that PR?

I'm closing this for now, let us know if you're experiencing any specific issues @bymaximus


Enable ensemble forecast output

As per title, along with the mean Sea Ice Concentration (SIC) and standard deviation, we should also output the SIC generated from each ensemble member, this will be of use in the future when doing more uncertainty analysis.

@bnubald feels like something that should be very much optional, this will make the forecast NetCDFs huge and the individual members, unlike numerical models, shouldn't be counted on individually too much

@bnubald feels like something that should be very much optional, this will make the forecast NetCDFs huge and the individual members, unlike numerical models, shouldn't be counted on individually too much

Agreed.


feat: include the default extrinsics on the latest blocks list

This update ensures that default extrinsics are now included when displaying blocks in the latest blocks scroll area.

There is one more route explorer/extrinsics where want to filter out the first two extrinsics of each block

Done


fix(CodeModal): Adjust height of code editor

Had default from PF examples in but it looks bad on larger office monitor.

Preview: https://virtual-assistant-pr-virtual-assistant-129.surge.sh


Node example

An example to support the openid-client-helper package.

Great work, as usual!

Hi Lori, per your request, the writing has been posted at: https://developer.forgerock.com/docs/platform/how-tos/building-oauth-20-client-openid-client-and-openid-client-helper-libraries
Thanks again for all your comments!

@lorig135 ^^^

WELL DONE!!!!


Implement cypress tests for Groups feature

Is your suggestion related to an experience ? Please describe.
Needs to implement a cypress integration test for the Groups component in console application.

Duplicate https://github.com/wso2/product-is/issues/8865


Fix link

Fix link of "Provision and catalog tutorial" in "Next steps"

@yoichinak : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yoichinak - thanks for fixing this!
#sign-off


Bump up version to 1.5.2(gax) and 0.22.2(gax-grpc)

You should be using the next minor version, not patch version, since you're making breaking changes to @BetaApi features.

Codecov Report

Merging #352 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #352   +/-   ##
=========================================
  Coverage     76.65%   76.65%
  Complexity      494      494
=========================================
  Files           114      114
  Lines          2540     2540
  Branches        185      185
=========================================
  Hits           1947     1947
  Misses          522      522
  Partials         71       71

Continue to review full report at Codecov.

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

PTAL

LGTM


change link in readme

Codecov Report

Merging #320 (663aaa1) into master (6eac99b) will decrease coverage by 0.53%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##           master     #320      +/-   ##
==========================================
- Coverage   82.83%   82.31%   -0.53%
==========================================
  Files          60       60
  Lines        4410     4410
==========================================
- Hits         3653     3630      -23
- Misses        757      780      +23

see 7 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


(MAINT) Cleanup acceptance testing

This PR removes a number of acceptance testing nodesets. These were due to the following reasons:

Ruby 1.8 was present on the nodes, but the HOCON gem only supports 1.9 and above (all CentOS nodesets, SLES 11)
The nodes did not contain the gem command, and thus, the hocon gem could not be installed for testing (Debian 6, Ubuntu 10 and 12)
The distro is no longer supported by FOSS Puppet (Fedora 18)
The provisioner for the VM was not available to the general public (Windows)

Furthermore, there were previously some acceptance tests pending due to there not being validation to ensure the setting, value, and path` parameters were assigned. This validation has been added, and the relevant tests are now active.

@cprice404 This is some polish I wanted to get into the module before we do a release.
Once this and https://github.com/puppetlabs/puppetlabs-hocon/pull/8 get merged, I'm thinking this module will be ready for a release. Do you think this acceptance testing coverage is sufficient? There are some things being tested in the spec tests, but not here (setting hashes, arrays, and text, for example).

@fpringvaldsen looks like this needs a rebase now?
LGTM, but I would maybe create a couple of Jiras: one for adding more OS's back to the matrix, maybe linked to that ruby 1.8 ticket, and one for adding at least a basic hash/map test to these tests.
I'm +1 on this so if I'm not online by the time you get it rebased, feel free to have @jpinsonault or @demophoon or someone review and merge.

looks good to me :+1:


status: fix return code for stopped units

Fixes #1021 on github.
If the exit code of systemd is not 0, then return with that exit code.
Previous behavior was to break and return 0.
The github.com/coreos/fleet/ssh test fails on master (at least on arch linux), and my PR does nothing to change that.
I don't see any tests for status. Will any be necessary?

@jonboulle Thanks for getting back to me. I made the change you suggested.

@iankronquist awesome, thanks! LGTM


Hotfix #173033902 – Improve gene search performance

It turns out that having a clause in Postgres of the form SELECT ... WHERE foo IN (SELECT ...)) incurs in a big performance hit. Even if both the main query and the nested one separately complete in the order of 0.01 seconds, combining the two results in times ranging from 0.5 to 3 seconds. This was the case when looking for the marker gene profile of a gene ID (i.e. for which k and cluster ID in an experiment the gene ID is a marker gene, or none).
In the case of popular genes that appear in many experiments, we were looking for these marker gene profiles without caching and doing a SELECT query for each experiment, resulting in gene searches that made more than 40 queries (of 0.5-3 seconds each). Also, as part of this step, the IDF file was parsed each time to find the preferred K of the experiment.
To solve the above:
• I added caching to avoid parsing the IDF each time
• Instead of finding the minimum marker probability of each gene ID in an experiment every time, I added a method that creates (and caches) a map of gene ID to minimum marker probability
I thought of adding the cache in the service level, but that would mean that other genes in the same experiment would suffer long first access times, and gene searches are very variable.
When this PR is accepted we must remember to cherry-pick it in master as well.
There is a detailed report with times and benchmarks at https://www.pivotaltracker.com/story/show/173033902.

I have gone through the code to know what magic you did to boost SQL query performance. Now I got that magic :).
Everything looks good to me.


mac-18 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-18 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

mac-18 is up and running fine, machine is out of sync with Salt master. Run command "salt-call state.apply" to re-sync Salt master again.
flutter-devicelab-mac-18:~ flutter$ uptime
16:22  up  6:07, 3 users, load averages: 1.12 1.20 1.28


Continuous positioning (with navigator.geolocation and watchPosition) does not work properly with tap events

Stopping the thing, at least, does not work properly. It leaves infoWindows hanging around.
It remains to be seen whether this depends on click vs tap, or too persistent a cache.
Until resolved, I consider this as a bug.

Removed bugs and also changed click events to pointerdown events. Code cleanup.


Behavior on closure

What will happen? Does it work in a closure?
class Resource {
    [@Symbol.dispose]() { this.disposed = true; console.log('disposed') }
    log(c) { console.log(c, this.disposed) }
}

{
    let out
    function f() {
        using const inner = new Resource()
        out = inner
        console.log('inner scope ended')
        return () => inner.log(0)
    }
    let result = f()
    out.log(1)
    result()
    result = undefined
}

inner will be disposed when f() completes evaluation, as inner's lifetime is scoped to the function block. This lifetime is explicit regardless of whether you've captured inner in a closure.
The result of the above would be:
inner scope ended
disposed
1 true
0 true

@Jack-Works does this answer your question? If so I'd like to close this issue.

Yes