Don't create NSErrors with NSURLErrorDomain and bogus error codes

This code (in AFJSONRequestOperation.m) is incorrect:
error = [[[NSError alloc] initWithDomain:NSURLErrorDomain code:[response statusCode] userInfo:userInfo] autorelease];

We should not be creating errors with system domains (or at the very least, we shouldn't be assigning the HTTP status code as the error's code, since there are actual NSURLError error codes that handling code might expect).
One possible fix is to create AFNetworkingHTTPErrorDomain and document (in that declaration) that the codes for that domain are equivalent to HTTP status codes.

I'm finding this a little annoying.. For example in my rails app I pass down a status code.. I want to react to this in my app. Instead I get a string which contains my http code and a AFNetworkingErrorDomain code which represents, for example a 401 (unauthorised) as -1011
Maybe that's just me but I want to set the codes!

OK spoke to soon. If anyone else comes across this the lovely Gowalla's have put it in the NSHTTPURLResponse *response.
EG
failure:^(NSHTTPURLResponse *response, NSError *error) {

        NSLog(@"Response Code: %d",[response statusCode]);

        failureBlock(error);

    }];


Database Navigator - Does not show all SYSIBM tables for DB2 for z/OS

System information:

Win 10
DBeaver CE Version <IP_ADDRESS>002011957

Connection specification:

DB2 12 for z/OS - FUNCTION LEVEL (V12R1M505 )
IBM Data Server Driver for JDBC and SQLJ 4.15.105

Describe the problem you're observing:
DBeaver CE Version <IP_ADDRESS>002011957 (Win 10) seems to have an issue on DB2 12 for z/OS to display ALL DB2 catalog tables. We are using IBM Data Server Driver for JDBC and SQLJ 4.15.105. However issue seems not to be related to driver version (see later).
When displaying the tables for schema SYSIBM only 93 out of 302 objects are showed. The displayed objects are of Table Type equals ALIAS, AUXILIARY TABLE or GLOBAL TEMPORARY TABLE.
All the other system tables (for instance SYSIBM.SYSDATABASE, SYSIBM.SYSTABLES, SYSIBM.SYSTABLESPACE, SYSIBM.SYSINDEXES, …) are missing:

Same issue (with missing tables) is encountered when using table name auto completion when typing in an SQL stmt:

Having traced the communication between DBeaver and DB2 for z/OS it looks that DBeaver is using stored procedure SYSIBM.SQLTABLES to query the DB2 catalog tables. Stored procedure SYSIBM.SQLTABLES does then, query the catalog using following SQL stmts with a case clause to set the Table Type:

According to IBM product documentation (https://www.ibm.com/support/knowledgecenter/SSEPEK_10.0.0/odbc/src/tpc/db2z_fntables.html) table type SYSTEM TABLE did already exist in DB2 10 for z/OS and most probably already in the version before. Hence there was no recent change by IBM DB2 triggering this issue.
Steps to reproduce, if exist:

Connect to DB2 for z/OS
In Database Explorer search for schema SYSIBM and display the tables

Include any warning/errors/backtraces from the logs
No warning/errors found in the log files

thanks for the bug report

@db2zosuser Do you have "Show system tables" option enabled?

Verified in our that when activating "Show system objects" the regular system tables are viewable:
Activates Flag:

Deactivated Flag:
only ALIAS, AUXILIARY TABLE or GLOBAL TEMPORARY TABLE are shown:

Hi @serge-rider and @Sargul
Thanks for the hint and the confirmation. Yes, you are right when enabling "show system objects" in the config settings all SYSIBM objects are displayed.
However I wonder why this settings is not enabled by default and if the current distinction in system objects and non system objects is really correct. The differentiation is done based on the 'Table Type' returned by the stored procedure SYSIBM.SQLTABLES. However by design all objects in schema SYSIBM are system objects.
It is true that the tables SYSCONTROLS_DESC and SYSCONTROLS_DESC_H are auxiliary tables, but they are at the same time also system objects:

Hence that differentiation based on the table type is not completely correct ant might lead to misunderstandings. Hence I would appreciate if you consider to change/improve it ;-)

The classification comes from the driver meta data API. However for the db2 zos plugin it makes totally to display system objects as default as mainframe people make heavy use of the system tables in their daily work routine

@Sargul @db2zosuser thanks for testing.
In 7.0.2 we will add new navigator view configuration (#7387).
In Advanced mode (for now it is the default mode) system objects will be visible by default.
So, closing this ticket.


consider removing onDidChangeProperty

and instead expose properties on LocalPty and RemotePty so they can be accessed directly

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Various dev container improvements

Purpose

Install python dependencies after creation

This avoids the need for developers to install them manually
Aligned the version of streamlit between the base app and extensions to avoid a conflict

Install all recommended VSCode extensions

This avoids the need for developers to install them manually

Set default function path

This avoids a popup requesting this on startup

Set python interpreter path

This enabled VSCode to automatically detect installed dependencies

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Test the code

Open the project in the dev container

What to Check
Verify that the following are valid

all extensions are installed
python files open with no dependencies missing
no pop ups open

Other Information
Required by #69

Relates to #69


Browser demo link fails

Clicking the link from the README fails.

Thanks for the issue; fixed by #15


Dev drive insights and pip cache move

Summary of the pull request
Windows customization root page:

Dev Drive Insights: Listing the dev drives and the Pip cache optimizer card.

Folder picker content dialog to move the cache to dev drive location:

User inputs directory on dev drive:

Pip cache optimized card.

Currently the app needs to be relaunched for reflecting the change.
References and relevant issues
Detailed description of the pull request / Additional comments
Added Dev Drive Insights. User can move pip cache from non dev drive location to the dev drive for optimized performance.
Validation steps performed
pip install works after the change.
PR checklist

[ ] Closes #xxx
[ ] Tests added/passed
[ ] Documentation updated

FYI I think we should remove the list of list and instead do a single list for each viewmodel you have just like in this settings example

This will make coding it up simpler and also move the classes with dependency properties, to class with regular clr properties and use the observable property attribute if you need them to have one way binding.

As @bbonaby mentioned somewhere in here, it's important to try to update your feature branch, and your branch on it, to be on the most recent version of main. There is a lot of stuff in there that will affect what you have.
Also, if it's possible, it would be really helpful to try to break this up into smaller PRs. It's very exciting how much functionality you're bringing in, but there's so much code and UI that it's difficult to review.

Once I have all the comments addressed and ready to check in, I will rebase main, pull the latest changes and resolve conflicts if any. I am aiming for EOW to check this in, so, at this time I am not keen to break this into different PR.

In reply to:<PHONE_NUMBER>

@bbonaby @dhoehna @krschau I have addressed most if not all of your comments in this PR.
I have rebased to main to get Nick's Windows Customization changes. There were lots of conflicts from other components, so applied my changes manually and here is the new PR- https://github.com/microsoft/devhome/pull/2486.
Abandoning this.

@bbonaby @dhoehna @krschau I have addressed most if not all of your comments in this PR.
I have rebased to main to get Nick's Windows Customization changes. There were lots of conflicts from other components, so applied my changes manually and here is the new PR- https://github.com/microsoft/devhome/pull/2486.
Abandoning this.


Update README.md

Added additional hint for Linux users (such us Debian 9)

I'm not 100% sure, but I think mono-devel is all you need, maybe ca-certificates-mono.
I'll take a look later tonight.
Thanks.

It can be, the problem was that using Debian 9 i was unable to make it work this plugins.
After checking the official KeePass website. It says that for some plugins it may be required the mono-complete package. After install it the complement worked.
If only mono-devel if needed I suggest to edit my PR and change the package name but is would me great to have a hint for users like me that doesn't underestand what package is needed.
Kind Regards

I did a clean install on a Debian 9 vm.
The package mono-devel was enough.
The only problem with mono that is shipped with debian is that it is a little bit old.
If you want better results, you should upgrade it.
There is a few issues here where I explain about it.
Maybe create a wiki entry for it.
What do you think about?
[]'s

Sure ;)
I believe is enough for the majority of us.
Thanks!

Any suggestion?
https://github.com/navossoc/KeePass-Yet-Another-Favicon-Downloader/wiki/Linux-install


Make the circular preloader loop in one layer

I would like to know if there is a way to make the circular preloader loop through one color/layer only, without adding 4 spinner layers. If I only use one layer, it adds a pretty long time gap in the animation.

Added in 124780538a90a436031c6b3fbc8b9b552a9b3113


llvm installation error

With the latest llvm update, I got an error when executing vcpkg install llvm, below is the error I am getting
-- Performing post-build validation
There should be no empty directories in C:/dev/vcpkg/packages/llvm_x64-windows-dev
The following empty directories were found:

    C:/dev/vcpkg/packages/llvm_x64-windows-dev/include/llvm/BinaryFormat/WasmRelocs

If a directory should be populated but is not, this might indicate an error in the portfile.
If the directories are not needed and their creation cannot be disabled, use something like this in the portfile to remove them:

    file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/a/dir ${CURRENT_PACKAGES_DIR}/some/other/dir)

Found 1 error(s). Please correct the portfile:
    C:\dev\vcpkg\ports\llvm\portfile.cmake
-- Performing post-build validation done
Error: Building package llvm:x64-windows-dev failed with: POST_BUILD_CHECKS_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: llvm:x64-windows-dev
  Vcpkg version: 0.0.108-2018-04-16-a9b1541fd6d308a17848ab4eb5396598ffa1a240

Additionally, attach any relevant sections from the log files above.

I tried installing it for 2 times and get the same error

Thanks for the PR!


Test coderef handling change

This adds a test for https://github.com/dita-ot/dita-ot/commit/16bc43859c38862d921b33521b3ec288157f5d5c
If you undo that commit, the tests in this pull request will fail because referenced json is parsed as XML.
I am just adding this because it was code already existing in another pull request.
Note that it tests the generated output instead of the copied resources, e.g. tempDirBelow instead of e. If that is the right approach, maybe e should be changed to the generated output in the other methods and I can submit another pull request for that, or someone else can do it.
The method testFileContentBelow runs generate as a way to test that the json won't be parsed.
Also, TestUtils.getSourceDitaDir guesses that the root of build/resources/tests, on the classpath, is src/main, which could be wrong.

I've changed the test to use Job file info map to check for subtarget and reverted changes to TestUtils.java.

@queshaw could you squash the branch and force push to update the PR to a single commit?

Sorry, what I meant to say is that could you squash the branch to be on top the the latest develop code? (So that it doesn't have e.g. README.markdown or .gitignore changes.

I suspect that might not be what you want either. Sorry.
I think my branch now only differs from where it diverged by my changes. But, the other changes in develop now differ.

Okay, I squashed changes into develop on my fork.

I think I have fixed things and there are only my changes comitted.
I also noticed that I had named a directory "aux" which is a device name on windows. So, I changed that.

The change broadens the scope of a unit tests. Coderef should be tested in it's own unit test, not in one for gen-list.


MobiFlight-Customdevices repo should be archived

Since we now use CommunityDevices and CommunityTemplate, we should archive MobiFlight-Customdevices git repository, since it is very confusing to new people interested in the firmware development.
I mean it has fooled me several times already even though I know about the move :-)

That's related to issue #310 and PR #311 .
And yes, latest when the mentioned PR gets merged the MobiFlight-CustomDeiveces repo should be archived.


Fixed topics margin

Fixed topics margin for two or more rows
Before:

After:

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7ccaf54). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9248   +/-   ##
=========================================
  Coverage          ?   41.03%
=========================================
  Files             ?      556
  Lines             ?    72535
  Branches          ?        0
=========================================
  Hits              ?    29765
  Misses            ?    39025
  Partials          ?     3745

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 7ccaf54...0765c87. Read the comment docs.


Dynamic rowHeight and fixed aspect ratio on resize

Hi there,
i'm struggling with changing the rowHeight of the grid on a window resize event. I want to keep a fixed aspect ratio of the grid elements, even when the window changes size.
Does anyone have a good idea how to solve this? My first idea was to apply a callback to the window resize event and compute the rowHeight based on the current width of an element.
Any help or suggestion is appreciated.

I added a PR, which solves the problem for me: #101

Thanks for that PR.


Local embeddings function with Transformer.js API

Fixes # (issue)

I think you have to commit the updated Lockfile
Also, I think we've tried that specific package and it has some issues in most JS envs

Will try that. As for the node env. I can imagine it does not run on edge environment.
In fully fledged node environment in the context of a Next.js 13 app I have no promblem using this function to store document embeddings in ChromaDB. So would be great to have the capability to retrieve the embedded collections from LangChain ;)

Hey @l4b4r4b4b4, wasn't able to push commits onto your branch but opened #2288 with an integration test - it seems to just time out on my environment (Node 18.15.0).
Does it require some other setup to run? That's ok if so, but it should be documented.

as I wrote in the documentation mdx you need to install @xenova/transformers which I added to the peerDependencies. Will have a look at the test.

Yes, I have it installed but the test didn't run on node 18 - I remember having some difficulty in the past so was wondering if your environment was different?

That's the node version I am running on.
When looking at the CI/CD test that failed with your PR #2288 I had formatting test fail tool, when not leaving empty lines between certain lines e.g. defining class variables.

Merged in #2487


trying to add http://purl.obolibrary.org/obo/NCBITaxon_1319

@sabrinatoro feel free to close once you've had a chance to look at this

yes, this is exactly what is happening to me too. I can add the new taxon for http://purl.obolibrary.org/obo/NCBITaxon_1319 in the ncbitaxon_terms.txt, but it is not added to the ncbitaxon_import.owl file after the "run import".
@matentzn : I expected this to work once the obophenotype/ncbitaxon had been released, but you mentioned it has already been released. Is there another step we should do ? Thank you!


[kudu1] Failing to update/install system76-driver after upgrading to 18.04

Something is wrong on the dependency.
$ sudo apt-get purge system76-driver
$ sudo apt-get install system76-driver
[sudo] password for n130s:
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 system76-driver : Depends: system76-dkms but it is not going to be installed
                    Depends: system76-io-dkms but it is not going to be installed
		    E: Unable to correct problems, you have held broken packages.
		    $ sudo apt-get install system76-dkms
		    Reading package lists... Done
		    Building dependency tree
		    Reading state information... Done
		    Some packages could not be installed. This may mean that you have
		    requested an impossible situation or if you are using the unstable
		    distribution that some required packages have not yet been created
		    or been moved out of Incoming.
		    The following information may help to resolve the situation:

The following packages have unmet dependencies:
 system76-dkms : Depends: dkms (>= <IP_ADDRESS>) but it is not going to be installed
 E: Unable to correct problems, you have held broken packages.
 $ sudo apt-get install system76-io-dkms
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 Some packages could not be installed. This may mean that you have
 requested an impossible situation or if you are using the unstable
 distribution that some required packages have not yet been created
 or been moved out of Incoming.
 The following information may help to resolve the situation:

The following packages have unmet dependencies:
 system76-io-dkms : Depends: dkms (>= <IP_ADDRESS>) but it is not going to be installed
 E: Unable to correct problems, you have held broken packages.
 $ sudo apt-get purge dkms
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 The following packages will be REMOVED:
   dkms*
   0 upgraded, 0 newly installed, 1 to remove and 2 not upgraded.
   After this operation, 0 B of additional disk space will be used.
   Do you want to continue? [Y/n]
   (Reading database ... 464638 files and directories currently installed.)
   Purging configuration files for dkms (<IP_ADDRESS>-2ubuntu11.5) ...
   $ sudo apt-get autoremove
   Reading package lists... Done
   Building dependency tree
   Reading state information... Done
   0 upgraded, 0 newly installed, 0 to remove and 2 not upgraded.
   n130s@130s-kudu1:~$
   $ sudo apt-get install system76-io-dkms
   Reading package lists... Done
   Building dependency tree
   Reading state information... Done
   Some packages could not be installed. This may mean that you have
   requested an impossible situation or if you are using the unstable
   distribution that some required packages have not yet been created
   or been moved out of Incoming.
   The following information may help to resolve the situation:

The following packages have unmet dependencies:
 system76-io-dkms : Depends: dkms (>= <IP_ADDRESS>) but it is not going to be installed
 E: Unable to correct problems, you have held broken packages.
 $ sudo synaptic&
 [1] 4192
 n130s@130s-kudu1:~$
 (synaptic:4194): IBUS-WARNING **: 12:43:04.047: The owner of /home/n130s/.config/ibus/bus is not root!
 $ sudo apt-get install -f
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 0 upgraded, 0 newly installed, 0 to remove and 11 not upgraded.
 [1]+  Done                    sudo synaptic
 $ udu
 Get:1 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
 Hit:2 http://us.archive.ubuntu.com/ubuntu bionic InRelease
 Hit:3 http://ppa.launchpad.net/system76-dev/stable/ubuntu bionic InRelease
 Hit:4 http://us.archive.ubuntu.com/ubuntu bionic-updates InRelease
 Get:5 http://us.archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
 Fetched 163 kB in 1s (173 kB/s)
 Reading package lists... Done
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 Calculating upgrade... Done
 The following packages have been kept back:
   cpp libhdf5-dev
   The following packages will be upgraded:
     e2fslibs e2fsprogs e2fsprogs-l10n libcom-err2 libcom-err2:i386 libcomerr2 libcomerr2:i386 libext2fs2 libss2
     9 upgraded, 0 newly installed, 0 to remove and 2 not upgraded.
     Need to get 589 kB of archives.
     After this operation, 0 B of additional disk space will be used.
     Do you want to continue? [Y/n]
     Get:1 http://us.archive.ubuntu.com/ubuntu bionic-updates/universe amd64 e2fsprogs-l10n all 1.44.1-1ubuntu1.1 [4,224 B]
     Get:2 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 libext2fs2 amd64 1.44.1-1ubuntu1.1 [157 kB]
     Get:3 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 e2fsprogs amd64 1.44.1-1ubuntu1.1 [390 kB]
     Get:4 http://us.archive.ubuntu.com/ubuntu bionic-updates/main i386 libcom-err2 i386 1.44.1-1ubuntu1.1 [9,516 B]
     Get:5 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 libcom-err2 amd64 1.44.1-1ubuntu1.1 [9,228 B]
     Get:6 http://us.archive.ubuntu.com/ubuntu bionic-updates/main i386 libcomerr2 i386 1.44.1-1ubuntu1.1 [2,708 B]
     Get:7 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 libcomerr2 amd64 1.44.1-1ubuntu1.1 [2,712 B]
     Get:8 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 libss2 amd64 1.44.1-1ubuntu1.1 [11.1 kB]
     Get:9 http://us.archive.ubuntu.com/ubuntu bionic-updates/main amd64 e2fslibs amd64 1.44.1-1ubuntu1.1 [2,712 B]
     Fetched 589 kB in 1s (588 kB/s)
     (Reading database ... 464619 files and directories currently installed.)
     Preparing to unpack .../e2fsprogs-l10n_1.44.1-1ubuntu1.1_all.deb ...
     Unpacking e2fsprogs-l10n (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Preparing to unpack .../libext2fs2_1.44.1-1ubuntu1.1_amd64.deb ...
     Unpacking libext2fs2:amd64 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Setting up libext2fs2:amd64 (1.44.1-1ubuntu1.1) ...
     (Reading database ... 464619 files and directories currently installed.)
     Preparing to unpack .../e2fsprogs_1.44.1-1ubuntu1.1_amd64.deb ...
     Unpacking e2fsprogs (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Setting up e2fsprogs (1.44.1-1ubuntu1.1) ...
     update-initramfs: deferring update (trigger activated)
     (Reading database ... 464619 files and directories currently installed.)
     Preparing to unpack .../libcom-err2_1.44.1-1ubuntu1.1_amd64.deb ...
     De-configuring libcom-err2:i386 (1.44.1-1ubuntu1) ...
     Unpacking libcom-err2:amd64 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Preparing to unpack .../libcom-err2_1.44.1-1ubuntu1.1_i386.deb ...
     Unpacking libcom-err2:i386 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Preparing to unpack .../libcomerr2_1.44.1-1ubuntu1.1_amd64.deb ...
     De-configuring libcomerr2:i386 (1.44.1-1ubuntu1) ...
     Unpacking libcomerr2:amd64 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Preparing to unpack .../libcomerr2_1.44.1-1ubuntu1.1_i386.deb ...
     Unpacking libcomerr2:i386 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Preparing to unpack .../libss2_1.44.1-1ubuntu1.1_amd64.deb ...
     Unpacking libss2:amd64 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Setting up libcom-err2:amd64 (1.44.1-1ubuntu1.1) ...
     Setting up libcom-err2:i386 (1.44.1-1ubuntu1.1) ...
     Setting up libss2:amd64 (1.44.1-1ubuntu1.1) ...
     (Reading database ... 464619 files and directories currently installed.)
     Preparing to unpack .../e2fslibs_1.44.1-1ubuntu1.1_amd64.deb ...
     Unpacking e2fslibs:amd64 (1.44.1-1ubuntu1.1) over (1.44.1-1ubuntu1) ...
     Setting up e2fslibs:amd64 (1.44.1-1ubuntu1.1) ...
     Processing triggers for initramfs-tools (0.130ubuntu3.6) ...
     update-initramfs: Generating /boot/initrd.img-4.15.0-45-generic
     Processing triggers for libc-bin (2.27-3ubuntu1) ...
     Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
     Setting up libcomerr2:amd64 (1.44.1-1ubuntu1.1) ...
     Setting up libcomerr2:i386 (1.44.1-1ubuntu1.1) ...
     Setting up e2fsprogs-l10n (1.44.1-1ubuntu1.1) ...
     $ sudo apt-get purge ros-kinetic-
     ros-kinetic-catkin           ros-kinetic-rosbash          ros-kinetic-ros-environment  ros-kinetic-rospack
     $ sudo apt-get purge ros-kinetic-*
     Reading package lists... Done
     Building dependency tree
     Reading state information... Done
     Note, selecting 'ros-kinetic-catkin' for glob 'ros-kinetic-*'
     Note, selecting 'ros-kinetic-rospack' for glob 'ros-kinetic-*'
     Note, selecting 'ros-kinetic-rosbash' for glob 'ros-kinetic-*'
     Note, selecting 'ros-kinetic-ros-environment' for glob 'ros-kinetic-*'
     The following packages were automatically installed and are no longer required:
       google-mock googletest libgtest-dev libtinyxml-dev libtinyxml2.6.2v5
       Use 'sudo apt autoremove' to remove them.
       The following packages will be REMOVED:
         ros-kinetic-catkin* ros-kinetic-ros-environment* ros-kinetic-rosbash* ros-kinetic-rospack*
 0 upgraded, 0 newly installed, 4 to remove and 2 not upgraded.
 After this operation, 1,116 kB disk space will be freed.
 Do you want to continue? [Y/n]
 (Reading database ... 464617 files and directories currently installed.)
 Removing ros-kinetic-rosbash (1.14.4-0xenial-20180824-082251-0800) ...
 Removing ros-kinetic-catkin (0.7.14-0xenial-20180809-132632-0800) ...
 Removing ros-kinetic-ros-environment (1.0.0-0xenial-20180809-133925-0800) ...
 Removing ros-kinetic-rospack (2.4.4-0xenial-20180809-135522-0800) ...
 $ sudo apt-get autoremove
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 The following packages will be REMOVED:
   google-mock googletest libgtest-dev libtinyxml-dev libtinyxml2.6.2v5
   0 upgraded, 0 newly installed, 5 to remove and 2 not upgraded.
   After this operation, 7,961 kB disk space will be freed.
   Do you want to continue? [Y/n]
   (Reading database ... 464419 files and directories currently installed.)
   Removing google-mock:amd64 (1.8.0-6) ...
   Removing libgtest-dev:amd64 (1.8.0-6) ...
   Removing googletest:amd64 (1.8.0-6) ...
   Removing libtinyxml-dev:amd64 (2.6.2-4) ...
   Removing libtinyxml2.6.2v5:amd64 (2.6.2-4) ...
   Processing triggers for libc-bin (2.27-3ubuntu1) ...
   Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
   $ udu
   Get:1 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
   Hit:2 http://us.archive.ubuntu.com/ubuntu bionic InRelease
   Hit:3 http://ppa.launchpad.net/system76-dev/stable/ubuntu bionic InRelease
   Hit:4 http://us.archive.ubuntu.com/ubuntu bionic-updates InRelease
   Get:5 http://us.archive.ubuntu.com/ubuntu bionic-backports InRelease [74.6 kB]
   Fetched 163 kB in 1s (159 kB/s)
   Reading package lists... Done
   Reading package lists... Done
   Building dependency tree
   Reading state information... Done
   Calculating upgrade... Done
   The following packages have been kept back:
     cpp libhdf5-dev
     0 upgraded, 0 newly installed, 0 to remove and 2 not upgraded.
     $ sudo apt-get purge libhdf5-dev
     Reading package lists... Done
     Building dependency tree
     Reading state information... Done
     The following packages were automatically installed and are no longer required:
       gcc-6-base hdf5-helpers libaec-dev libaec0 libgfortran3 libhdf5-10 libhdf5-cpp-11 libsz2
       Use 'sudo apt autoremove' to remove them.
       The following packages will be REMOVED:
         libhdf5-dev* libhdf5-serial-dev*
 0 upgraded, 0 newly installed, 2 to remove and 1 not upgraded.
 After this operation, 30.5 MB disk space will be freed.
 Do you want to continue? [Y/n]
 (Reading database ... 463996 files and directories currently installed.)
 Removing libhdf5-serial-dev (1.10.0-patch1+docs-4) ...
 Removing libhdf5-dev (1.8.16+docs-4ubuntu1.1) ...
 $ sudo apt-get autoremove
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 The following packages will be REMOVED:
   gcc-6-base hdf5-helpers libaec-dev libaec0 libgfortran3 libhdf5-10 libhdf5-cpp-11 libsz2
   0 upgraded, 0 newly installed, 8 to remove and 1 not upgraded.
   After this operation, 5,597 kB disk space will be freed.
   Do you want to continue? [Y/n]
   (Reading database ... 463844 files and directories currently installed.)
   Removing libhdf5-cpp-11:amd64 (1.8.16+docs-4ubuntu1.1) ...
   Removing libhdf5-10:amd64 (1.8.16+docs-4ubuntu1.1) ...
   Removing libgfortran3:amd64 (6.5.0-2ubuntu1~18.04) ...
   Removing gcc-6-base:amd64 (6.5.0-2ubuntu1~18.04) ...
   Removing hdf5-helpers (1.10.0-patch1+docs-4) ...
   Removing libaec-dev:amd64 (0.3.2-2) ...
   Removing libsz2:amd64 (0.3.2-2) ...
   Removing libaec0:amd64 (0.3.2-2) ...
   Processing triggers for libc-bin (2.27-3ubuntu1) ...
   Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
   $ sudo apt-get purge cpp
   cpp      cpp-4.8  cpp-5
   $ sudo apt-get purge cpp
   Reading package lists... Done
   Building dependency tree
   Reading state information... Done
   The following packages were automatically installed and are no longer required:
     autoconf automake autotools-dev cpp-5 debugedit libdw1 libfile-stripnondeterminism-perl libfs6 libisl15 libmail-sendmail-perl
       librpmbuild8 librpmsign8 libsys-hostname-long-perl m4 po-debconf rpm x11-apps x11-xfs-utils xinit xorg-docs-core
       Use 'sudo apt autoremove' to remove them.
       The following packages will be REMOVED:
         alien* cpp* debhelper* dh-autoreconf* dh-strip-nondeterminism* libtool* x11-session-utils* x11-xserver-utils* xorg*
 0 upgraded, 0 newly installed, 9 to remove and 0 not upgraded.
 After this operation, 3,972 kB disk space will be freed.
 Do you want to continue? [Y/n]
 (Reading database ... 463778 files and directories currently installed.)
 Removing alien (8.95) ...
 Removing xorg (1:7.7+19ubuntu7.1) ...
 Removing x11-xserver-utils (7.7+7build1) ...
 Removing x11-session-utils (7.7+2build1) ...
 Removing dh-autoreconf (17) ...
 Removing debhelper (11.1.6ubuntu2) ...
 Removing dh-strip-nondeterminism (0.040-1.1~build1) ...
 Removing libtool (2.4.6-2) ...
 Removing cpp (4:5.3.1-1ubuntu1) ...
 Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
 (Reading database ... 463174 files and directories currently installed.)
 Purging configuration files for x11-session-utils (7.7+2build1) ...
 Purging configuration files for x11-xserver-utils (7.7+7build1) ...
 $ sudo apt-get autoremove
 Reading package lists... Done
 Building dependency tree
 Reading state information... Done
 The following packages will be REMOVED:
   autoconf automake autotools-dev cpp-5 debugedit libdw1 libfile-stripnondeterminism-perl libfs6 libisl15 libmail-sendmail-perl
     librpmbuild8 librpmsign8 libsys-hostname-long-perl m4 po-debconf rpm x11-apps x11-xfs-utils xinit xorg-docs-core
     0 upgraded, 0 newly installed, 20 to remove and 0 not upgraded.
     After this operation, 32.3 MB disk space will be freed.
     Do you want to continue? [Y/n]
     (Reading database ... 463170 files and directories currently installed.)
     Removing automake (1:1.15.1-3ubuntu2) ...
     Removing autoconf (2.69-11) ...
     Removing autotools-dev (20180224.1) ...
     Removing cpp-5 (5.5.0-12ubuntu1) ...
     Removing rpm (4.14.1+dfsg1-2) ...
     Removing debugedit (4.14.1+dfsg1-2) ...
     Removing librpmbuild8 (4.14.1+dfsg1-2) ...
     Removing libdw1:amd64 (0.170-0.4) ...
     Removing libfile-stripnondeterminism-perl (0.040-1.1~build1) ...
     Removing x11-xfs-utils (7.7+2build1) ...
     Removing libfs6:amd64 (2:1.0.7-1) ...
     Removing libisl15:amd64 (0.18-4) ...
     Removing libmail-sendmail-perl (0.80-1) ...
     Removing librpmsign8 (4.14.1+dfsg1-2) ...
     Removing libsys-hostname-long-perl (1.5-1) ...
     Removing m4 (1.4.18-1) ...
     Removing po-debconf (1.0.20) ...
     Removing x11-apps (7.7+6ubuntu1) ...
     Removing xinit (1.3.4-3ubuntu3) ...
     Removing xorg-docs-core (1:1.7.1-1.1) ...
     Processing triggers for install-info (6.5.0.dfsg.1-2) ...
     Processing triggers for libc-bin (2.27-3ubuntu1) ...
     Processing triggers for doc-base (0.10.8) ...
     Processing 2 removed doc-base files...
     Registering documents with scrollkeeper...
     Processing triggers for man-db (2.8.3-2ubuntu0.1) ...

OS re-installed and no problem since then.
$ sudo apt-get install system76-driver
[sudo] password for n130s:
Reading package lists... Done
Building dependency tree
Reading state information... Done
system76-driver is already the newest version (19.04.18~1568998391~18.04~651afba~dev).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.


Get involved page

Acceptance criteria

The sidebar logic discussed should take care of the sidebar completely (i.e. that is not part of this task)
The article content itself also matches articles from other sections
The only thing different is the bottom section, where the "Keep going" section currently appears

We don't want the standard "Related questions" to show up at the bottom of the pages, so we'll likely just want to delete Related questions in Coda
The bottom, instead, just shows it's children
If the article is the third tier (i.e. has no children), obviously this won't show

Links
https://www.figma.com/file/qqfeYAh6Z7QdyJXsQjIUfh/Stampy?type=design&node-id=2177%3A4109&mode=dev&t=VPCJqtZ3vZNguLmb-1

Blocked by keep going section

Update:
There should be one more set of children. Is this a copy thing or a dev thing?
The little arrows are cut off on some

The article content itself is not right but I think this is just blocked by the "main content on article page" which I hope to fix soon.

Same as before, that there should be one more set of children. Is this a copy thing or a dev thing?

Oh and actually for this section only, can we not open the children in a new tab? Remove the arrows too. I am sorry my fault.

Update: Melissa talk to editors about putting third tier

Looks good enough, but (another task) @mruwnik There are a couple CSS comments I have about the sidebar, to be addressed later


ultrack CLI documentation clarification

Hi,
I found this documentation for a CLI interface. However I don't understand how to use it, in particular it often contains the phrase "on napari", which I do not understand.

Hi @tischi,
We use the reader plugins from napari to support multiple file formats, and we index the data using the layer name (or index) they would get in napari viewer. We don't load the napari viewer, so it's safe to use in an HPC.
For example, assuming you have an ome-zarr dataset compatible with the napari-ome-zarr plugin.
And your image has three channels: Image, Foreground, and Edges.
So you would use ultrack segment --cfg config.toml -r napari-ome-zarr -dl Foreground -el Edges
To indicate it should use the napari-ome-zarr to open your dataset and the detection and edges channels are respectively Foreground and Edges.


chore: show loading spinner and disable submit button on submit artwork mutation

[Bug Bash]
This PR resolves ONYX-963
Description
This PR makes the following changes to the submit button on the existing submit flow

Disable the button when it's loading
Show loading spinner during the create mutation

https://github.com/artsy/eigen/assets/11945712/4199ed4d-d2ba-4f36-bf8e-01f73d8c4323
https://github.com/artsy/eigen/assets/11945712/60abf2cf-abc0-4d6c-a679-48bbaf39fbce

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

show loading spinner and disable submit button on submit artwork mutation - mounir

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (show loading spinner and disable submit button on submit artwork mutation - mounir)

  Generated by :no_entry_sign: dangerJS against 0ae3ea28a505baded9241382ea40ad80beb848a3

🙏


added script to node-local testing for running test.js via the action…

… container itself, so that actions using npm libraries can be tested more simply

Thanks @tysonnorris ! LGTM !

One more change - I found it did NOT work when using relative paths... fixed now

ugh, renaming variables...

@tysonnorris this is a great way to run integration tests locally for the developer who writes actions. It would be great if all openwhisk action containers would have built-in support for this.


how to solve it

Please file your issues with Web Starter Kit in their repo at https://github.com/google/web-starter-kit


calling CloudServiceManagementClient.Create on a pre-existing cloud service, an unexpected exception is thrown

The following code works fine the first time through - a fresh cloud service name and the service returns with an expected HttpStatusCode and CloudServiceOperationStatus.
On the second run (leaving the names the same to see what happens on a subsequent create with the same details), I expected the following code to print out "failed."
Instead, I received a CloudException instance with the message:
"BadRequest: The condition specified by the ETag is not satisfied."
var cloudServiceManagementClient = new CloudServiceManagementClient(new CertificateCloudCredentials(subscriptionId, certificate));
var response = loudServiceManagementClient.CloudServices.Create("myCloudService",
   new CloudServiceCreateParameters()
   {
        GeoRegion = "uswest",
        Label = "myCloudService",
        Description = "My Cloud Service Scheduler",
       Email = "[my-email-address-omitted]"
   });

if (response.HttpStatusCode == System.Net.HttpStatusCode.OK &&
    response.Status == CloudServiceOperationStatus.Succeeded)
    {Console.WriteLine("succeeded.");}
else
    {Console.WriteLine("failed.");}

https://github.com/WindowsAzure/azure-sdk-for-net/issues/328

Hi Stevehebert,
I am facing the same issue. First time it went through. Second time, getting the same exception you got.
Please let me know what you did to get it resolved.


RequestResponse Cancellation Test

Assert that a requestResponse can be cancelled.

Apparently running the unit tests locally didn't check all the build :-) Will fix and repost.


Add error handling for Plot Wizard plots with x fields

3/3 main<- #4787 <- #4797 <- this

If the user selects more than one x field, we check if there amount of x fields and y fields chosen are the same amount and we check that the user hasn't selected more than one field in a file

Demo
Check for only metric per file
https://github.com/iterative/vscode-dvc/assets/43496356/cc342b69-9647-44b1-a647-1cae75059063
Check for x and y having the same amount of metrics
https://github.com/iterative/vscode-dvc/assets/43496356/21beb866-db78-4345-8fe3-7eac69ce6cb6
Part of #4654

Going to merge and improve validation for the y metrics quick pick (https://github.com/iterative/vscode-dvc/pull/4798#discussion_r1355908616) in a followup pr!


Remove old ECR images from tooling account

Is your idea related to a problem? Please describe.
Amazon Inspector will scan and raise vulnerabilities against versions of ECR images that are not currently being used by data.all.
ECR image maintenance should be a responsibility of the data.all service.
Describe the solution you'd like
Automatically delete previous versions of ECR images when the tooling pipeline is run
Maintaining at least the previous version is desire-able for any CloudFormation stack rollback requirements.

@petrkalos @dlpzx is this still relevant or its fixed and we can close this?

Currently there is a lifecycle rule that keeps 200 images. I think the reqs here will be diverse, larger orgs might want to keep more for auditing purposes and smaller fewer as cost optimisation.
I think the best way to deal with it is to make the number of images to keep configurable via a context option. wdyt?

I agree, the requirements are very diverse. In the case of the original github issue purpose the idea was to reduce the number of stored images; but given that there is already a lifecycle rule in place I would fall to this default configuration. I don't think we should be investing time on this unless there is clear requirement to do so


FR: require_admin and /or other user view settings

currently, all logged users in my HA instance see the Music assistant side panel.
would love to be able to select which users can see that.. Not having my music being played by others feels like a fundamental security setting?
thanks for considering

I do not consider the ability of playing music a security setting but a feature but that's just personal I guess ;-)
The ability to show/hide panels is not something each integration should implement (not even custom integrations) but imo this should be implemented at core level, for example with user roles to which you assign certain functionality.
So, basically my answer is that I will not consider adding this feature request to Music Assistant but I'm willing to start the architecture discussion for core ;-)

clearly you dont have kids old enough to be little teasers... ;-)
thanks if you open the discussion though, that would be really helpful!

haha, oh yes I do, and I find it kind of fun to teach them how to handle IT instead of hiding it from them ;-)
Although I do not agree with an approach of hiding stuff because someone in my household may abuse it, I do agree that it is a good UX to hide stuff that is not relevant for some persons in the house. That's why I think that role bases access stuff is still a good idea but no so much for security (I want my HA to be secured) but for usability reasons.
For example I'm interested in nitty pitty details about power consumption and automations performance and my wife just needs a few big buttons to turn off a light.
I'll let you know once I've prepared something on the architecture repo.

thanks!
btw, I wasnt perse talking 'little' kids.....And I thought them all to well ;-)
when they're old enough to travel the world and party, I've learned that remote access to my home isn't perse a good thing... even though I understand the fun, I'd like to be in charge who controls my music playing in the middle of the night over the broadcast system.
For now I've used the generic setting on users requiring local access, but thats very drastic ofc. more fine grained control would be appreciated.
Feel free to close here.

haha I understand, it's a whole different level of joking around


Weird effect when drag element up or down

When I have the following css style on the parent element:
display: flex;
align-items: center;
flex-wrap: wrap;

When I drag from left to right (or right to left), it works fine. The problem is, when I drag from up to down (or down to up). I can't describe the exact behavior it has, but it's definitely not the expected result. Here's a gif explaining:

(For all the perfectionist out there like me, there is a grammar mistake at "The Rest doesn't Work". The 'd' isn't capitalized.)
What is wrong, and how can I fix it?
I can edit dragula.js; and, I can not use JQuery.
Sources:

Drag Function - Line #392 - el was lifted from source
Rearrange DOM - Line #441 - el, the visual aid shadow, was moved into container. May trigger many times as the position of el changes, even within the same container; el originally came from source

JSFiddle

Could you tell me how do you solve this problem

I have same problem too

this will be solve this
https://github.com/spartez/dragula/tree/mixed-direction

This is due to the way the DOM elements are rendered by the browser. Adjusting your html/CSS might help, perhaps use a CSS grid?


Feature Request: Add ReentrancyGuard

This is a long shot but i think it may be worth a try.
So the idea here is to have solady with all the "standard" contracts much like oz and solmate have.
This way when working with solady as your library of choice you can have all the tools at your disposal instead of having to import from different libraries (unless its very specific). This would make the project have less "bloat" and be cleaner overall.
The code bellow passes on all of solmate's tests as well as saving 100 gas on protected calls.
abstract contract ReentrancyGuard {

    uint256 private constant _UNLOCK = 1;
    uint256 private constant _LOCKED = 2;
    uint256 private constant _REENTRANCY_SELECTOR = 0xab143c06;

    uint256 private _locked = _UNLOCK;

    error Reentrancy();

    modifier nonReentrant() virtual {

        /// @solidity memory-safe-assembly
        assembly{
            if iszero(eq(sload(_locked.slot), _UNLOCK)) {
                mstore(0x00, _REENTRANCY_SELECTOR)
                revert(0x1c, 0x04)
            }

            sstore(_locked.slot, _LOCKED)
        }

        _;

        /// @solidity memory-safe-assembly
        assembly{
            sstore(_locked.slot, _UNLOCK)
        }
    }
}

I'd actually encourage devs DIY their own reentrancy guard (packed with other variables).
Hmm...

I agree with that statement, much like you did with ERC721A.
However not all devs will do it and this would be a option for them to use in case they need it.
If you think this goes against the philosophy defined in (#175) feel free to close this issue. But since solady tries to have optimized gas snippets on all areas I though this could belong here.

I think we might reopen this just to entertain the doubts of contract developers, particularly if an upgrade path (so v0 might have ReentrancyGuard, while v1 bakes). Whether we like it or not, RG is sticky af and often just tells external parties what cannot happen even if wasteful.


Bug with the decimal bar effects

Hi !
First , thanks for the excellent work ! It gives a nice addition to the wall clock !
I use 9 LEDs per strip, but I have a problem where the decimal bar (at 10 am for example) applies the palette of the downlights (not sure if i'm really clear! )
For example, when I set white for the downlight, it also applies to the "1" bar when it's 10:00 for example.
When I set red for the hours palette, the bar stays white and only the zero gets red.
I think the problem is somewhere here, but I can't pinpoint it ... Do you have any idea ?
Thanks a lot again !
`uint16_t WS2812FX::mode_clock(void) {
uint8_t timeHour = hour(local) % 12;
if(timeHour == 0) timeHour = 12;
uint8_t timeMinute = minute(local);
uint8_t numberOfSegments = 3;     //1.5x Hours, 2x Minutes, because 12 hour format
uint8_t ledsPerSegment = 9;       //5 LEDs per Segmentfraction (7 Segment with each 3 LEDs = 21 LEDs)
uint8_t digits[4];
digits[0] = timeHour / 10;
digits[1] = timeHour % 10;
digits[2] = timeMinute / 10;
digits[3] = timeMinute % 10;
uint8_t index = 0;
uint8_t usedIndex = 0;        //Used for better Color Palette usage
uint8_t counter = (now * ((SEGMENT.speed >> 2) + 2)) & 0xFFFF;
for (int i = numberOfSegments; i > 0; i--) {
for (int j = 0; j < 7; j++) {
for (int k = 0; k < ledsPerSegment; k++) {
if(segmentArray[digits[i]+1][j]) {
usedIndex++;
if(i == 3 || i == 2) {      //Minutes
setPixelColor(index, color_from_palette(usedIndexmap(SEGMENT.intensity, 0, 255, 0, 25)+(now / map(SEGMENT.speed, 0, 255, 400, 5)), false, true, 0));
}
else if(i == 1) {                      //Hours
setPixelColor(index, color_from_palette(usedIndexmap(SEGMENT.intensity, 0, 255, 0, 25)+(now / map(SEGMENT.speed, 0, 255, 400, 5)), false, true, 1));
}
}
else {
setPixelColor(index, 0);
}
index++;
}
}
}
if(timeHour >= 10) {
for(uint8_t i = 0;i < 2ledsPerSegment;i++) {
setPixelColor(index+i, color_from_palette(usedIndexmap(SEGMENT.intensity, 0, 255, 0, 25)+(now / map(SEGMENT.speed, 0, 255, 400, 5)), false, true, 1));
}
}
else {
index += 2*ledsPerSegment;
}
for(int i = index; i < SEGLEN;i++) {
setPixelColor(i, SEGCOLOR(2));
}
return FRAMETIME;
}`

Allright !
I understand your wiring, and indeed, you don't have the problem. Maybe I thinkered the settings too much, I'll reflash and tell you !
Thanks !!

This should be fixed already, please confirm

Thanks you !!
Sorry haven't had time lately to look about it, but I will try the fixes !
Thanks a lot again

I compiled and uploaded, can confirm, it works good now ! (decimal bar is not tied to the downlight color)
Cheers !


Arrays filling improvement and some tidy up.

Hi @dlwh,
the goals of this PR are:

removing some boilerplate when filling arrays
optimizing lenght and empty check on arrays
doing some general tidy up

thanks for doing this! My biggest concern is with Array.fillWith (which should just be array.tabulate, anyway). A lot of places where I used a while loop is near an inner loop.

replaced by https://github.com/dlwh/epic/pull/51


UI API called from background thread

IOS 11.4.1
Cordova 4.5.5
push plugin 2.2.3
I get the following error in Xcode:
UI API called from background thread: -[UIApplication setApplicationIconBadgeNumber:] must be used from main thread only
It should be fixed in version 2.1.0, as it written here:
https://github.com/phonegap/phonegap-plugin-push/issues/1988
but the issue still occurs.

Sorry to be a jerk but I'm now closing issues that don't follow the issue submission guidelines. This plugin takes a lot of work to maintain and I can't afford the time required to solicit the required info from everyone.
I'm closing this issue but please feel free to re-open the issue if you provide the requested details from the new issue submission guidelines.
https://github.com/phonegap/phonegap-plugin-push/blob/master/.github/ISSUE_TEMPLATE.md


lsif-java: cross-file references

Working for:

type identifier in variable declaration

Closing this issue for combined #59 as generic cross/same-file refs solution now exists


Make JavaTimeEncoders/Decoders public

So that they can be extended by custom helper objects.
See #976

Codecov Report

Merging #995 into master will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #995      +/-   ##
=========================================
- Coverage   85.35%   85.3%   -0.05%
=========================================
  Files          75      75
  Lines        2362    2362
  Branches      159     159
=========================================
- Hits         2016    2015       -1
- Misses        346     347       +1

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/JsonNumber.scala
88.46% <0%> (-1.29%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
90% <0%> (-1.25%)
:arrow_down:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.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 71d1c93...56dc41f. Read the comment docs.

Thanks!


Change titles on lines 51 and 77

The translation of the titles on lines 51 and 77 in portuguese is wrong. It should be changed for this version to help people to better understand the guide.

[x ] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[x ] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


[daw-json-link] update to v3.0.0

Fix #25470
Update [daw-json-link] version to 3.0.0
Update [daw-header-libraries] version to 2.46.2
Note: no feature need to test.

Looks like there is a typo, the old version of daw-json-link was 2.46 I think, https://github.com/microsoft/vcpkg/issues/24478, but this one coincidentally went to 2.64.0

Yah 67637711bc87f959b6d52adab56a819765b94ae1 is 2.64.2 :) yay, just the text had it swapped


Plaformio IDE Terminal

Hello,
Recently I installed Python IDE terminal in my atom to run python. Everything seems fine but I can only use the default command prompt which is like that:
For, print ("Hello World") code
the output instantly disappeared & don't want for keyboard key press.
I searched many places but didn't find, how to connect it to the python cmd.
So, can anyone face the same problem?
If yes, then how could you solve it?
Thanks in advance.

Please retest again with platformio-atom-ide-terminal 2.9.0.
These type issues all so far seem fixed with this release due to using node-pty-prebuilt instead unmaintained and outdated pty.js


Add tensorflow/theano setup to Travis config

Right now tests fail with an mhctools dependency issue:
Traceback (most recent call last):
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/nose/loader.py", line 418, in loadTestsFromName
    addr.filename, addr.module)
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/nose/importer.py", line 47, in importFromPath
    return self.importFromDir(dir_path, fqname)
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/nose/importer.py", line 94, in importFromDir
    mod = load_module(part_fqname, fh, filename, desc)
  File "/home/travis/build/hammerlab/vaxrank/test/test_shell_script.py", line 6, in <module>
    from vaxrank.cli import main as run_shell_script
  File "/home/travis/build/hammerlab/vaxrank/vaxrank/cli.py", line 25, in <module>
    from mhctools.cli import (
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/mhctools/__init__.py", line 10, in <module>
    from .mhcflurry import MHCflurry
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/mhctools/mhcflurry.py", line 22, in <module>
    from mhcflurry import Class1AffinityPredictor
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/mhcflurry/__init__.py", line 15, in <module>
    from .class1_affinity_prediction.class1_neural_network import (
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/mhcflurry/class1_affinity_prediction/__init__.py", line 3, in <module>
    from .class1_neural_network import Class1NeuralNetwork
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/mhcflurry/class1_affinity_prediction/class1_neural_network.py", line 8, in <module>
    import keras.models
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/keras/__init__.py", line 3, in <module>
    from . import activations
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/keras/activations.py", line 4, in <module>
    from . import backend as K
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/keras/backend/__init__.py", line 73, in <module>
    from .tensorflow_backend import *
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/keras/backend/tensorflow_backend.py", line 1, in <module>
    import tensorflow as tf
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/__init__.py", line 24, in <module>
    from tensorflow.python import *
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/__init__.py", line 51, in <module>
    from tensorflow.python import pywrap_tensorflow
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 52, in <module>
    raise ImportError(msg)
ImportError: Traceback (most recent call last):
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow.py", line 41, in <module>
    from tensorflow.python.pywrap_tensorflow_internal import *
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 28, in <module>
    _pywrap_tensorflow_internal = swig_import_helper()
  File "/home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 24, in swig_import_helper
    _mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
ImportError: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.17' not found (required by /home/travis/miniconda/envs/test-environment/lib/python2.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so)
Failed to load the native TensorFlow runtime.

(Addressed in #134)

This has been fixed


FTL coordinate disk command for admins: FTLdiskburner

About the PR

Expand NanoTrasen's dominion over all time and space once more.
Why / Balance

Let the admins play with shuttles across multiple maps with less micromanagement.
Technical details

Takes a list of EntIDs, checks if theyre a map, if they aren't, finds the map they are on, then makes sure the map is setup as a destination, and finally spawns a disk setup for the map.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:ADMIN:

add: new command "FTLdiskburner [EntID]" to make coordinate disks for shuttles to go to other maps.

Okay i cooked my master branch a bit apparently, but its fiiiiine, it'll squash.

IConsoleCommand is deprecated. Use toolshed for new commands.

IConsoleCommand is deprecated. Use toolshed for new commands.

I thought this was backtracked?

IConsoleCommand is deprecated. Use toolshed for new commands.

I thought this was backtracked?

Not serverside. Use toolshed for this.

Is this still being worked on?

Needs a completion helper.

Needs updating to code standards for commands.
I've looked at this for a couple weeks, and I have no idea what you mean lmao.

I swapped from "Map ID" -> "Map netID" since we have two schemas

Needs updating to code standards for commands.

I've looked at this for a couple weeks, and I have no idea what you mean lmao.

For new commands the standard is to use toolshed, unless there's a specific reason you need to use the old command system.
Do you need any help with porting this to toolshed?

Needs updating to code standards for commands.

I've looked at this for a couple weeks, and I have no idea what you mean lmao.

For new commands the standard is to use toolshed, unless there's a specific reason you need to use the old command system.
Do you need any help with porting this to toolshed?

If you want to be my guest. I looked into it a bit, and it seems it will need engine stuff I do not want to do, and every time I ask about it I get conflicting direction.

There are 3 issues you may encounter with the command, but they aren't issues with the command so much as they are issues with maps which 'mattered until now.
I'm going to try to fix them in separate PRs, but I'll list them here for reference.

The FTL map can't be FTL'ed to, probably due to the FTL console/system thinking that's a dumb idea (it kinda is, but idk maybe you want to do warp speed pirates or something).
Centcom has the FTLDestinationComponent, but its not setup properly and needs to have the component's Enabled and RequireCoordinatesDisk fields set to true. Its also got a generic "Map Entity" name for the map.
Gateway Destination maps need to spawn and name an FTL point along with the gateway entity. They're surprisingly hard to get to and require a lot of fiddling with F3 or the lsmap command to tp to them.

But don't worry! This just works tm for new maps you create and initialize or for your admin test area. So you could for example use this to have two stations on different maps and let the players ferry themselves between them via FTL.

Localized


(Windows) script to select which shell you want on startup

I've written it up here, complete with install instructions.
Not sure where to put it, 'cause it's not a plugin or a theme or anything.

This is really awesome.
If you want, I can help you to make a plugin around this.

That would be really cool. I don't really know much about NodeJS, so help would be appreciated.

To be honest, I've never made a Node project. I've used some tools available via npm, but I've never made anything directly with Node.

If you want, I can make this plugin, and give you credits and write access on its GitHub repo. I could explain you all steps etc...
If you are ok, DM me on Zeit's slack

Nah. This'll give me an excuse to learn Node. It's how I've learned other languages - pick a project and a language and learn as I go.


[Docs] Homebrew installation instructions using wrong mesheryctl command

Current State:
The homebrew instructions on this page say to run ./mesheryctl start
Desired State:
The instructions should say to run ./mesheryctl system start

Contributor Resources

Meshery documentation site
Meshery documentation source
Instructions for contributing to documentation

@iknowahra if you would, you can take this up too.

@Nikhil-Ladha I would like to do 👍

@Nikhil-Ladha I think it is already done :-)

Oops, seems like someone forgot to mention the issue😬. No worries you can work on any other issue you want. Also, feel free to join our slack workspace at slack.layer5io.com

Fixed by #1342


Screenshots HTML file should use relative paths

It currently uses absolute paths which is a problem for portability.

You are completely right, I'll fix this right now! Thanks!

I'll release a new version soon :+1:

Thanks!


Introduce Github Action workflows to run tests and coding style checks

Fixes #14
PHPUnit 4 is a fixed dependency which results in problems running tests using higher PHP versions. I use your code in a PHP 8 project without any problems though.
It runs test for PHP 5.4 until 7.4. PHP 8 is ignored.
Current status: https://github.com/k00ni/php-jsonpointer/actions (see upper two entries)

Is it still work in progress or an actual pull request?

Can you combine the commits into one i.e. Improvement, replace Travis with GitHub Actions.

Improvement, replace Travis with GitHub Actions

Thanks for this contribution.


chore(test): mock localStorage

I know that you are very busy, just resolve your problem.
Recently, I create the chore pull request(not the important change) so you can temporarily ignore it in this time.

Oh this is nice to have! lets do it :)
ive been working on things like this:
https://www.socialmediatoday.com/news/instagram-launches-stay-home-sticker-and-story-to-encourage-social-distan/574628/
that are all pretty rapid responses to COVID-19 so hence the business :P


What is current stable version?

Is the current stable 0.4.2 or 0.4.3?
I noticed that the link https://bintray.com/artifact/download/aragozin/generic/sjk-plus-0.4.3.jar doesn't exist yet, so the Homebrew formula cannot be upgraded, assuming 0.4.3 is now the current stable.
Thanks!

I have just uploaded 0.4.3 to bintary

@aragozin voilà: https://github.com/Homebrew/homebrew-core/pull/4126
Thanks!


feat(nvmx): allow configurable extended host ID

Allow the MAYASTOR_NVMF_HOSTID environment variable to override the
default randomly generated 128-bit extended host identifier used by
the nvmx NVMf client with an optionally-hyphenated UUID string.
Overriding that also overrides the host NQN as SPDK usually derives
the host NQN from the host ID, unless the host NQN has also been
overridden.
Extend the NVMe reservation acquire test to verify that the host
adding the child has indeed acquired the write exclusive reservation.
Fixes CAS-856

bors try

bors merge


validation and test

Hi, thanks for sharing your great work!
I found that the best mIOU of S3DIS is different for validation and set, but boh sets are Area 5.
Then I found that the test set did the data augmentation while the validation didn't.
I wonder why not unify them. Is this ok?

Hi, check readme here (https://github.com/Pointcept/Pointcept?tab=readme-ov-file#testing), we adopt a more precise yet time-consuming testing strategy, so we do not use it during validation.


dattcore: Msg type: Original content

It should be possible to create new "original content" which gets posted to the "front-page". We don't need to support multiple different categories for the prototype - just one "page" of content, the "front-page". Not that original content must involve signing the latest block hash to prove the content is new, to prevent users from spamming old content from other users.

Closed by https://github.com/dattnetwork/datt/commit/7c4b7d2d11bf7ccc153f8ce23e9a41d1556e4625


Improve company

This PR do the followings:

Add company-files, which provides completion for file paths, to company-backends. This completion is really useful, and doesn't introduce significant CPU overhead.
Set value of company-selection-wrap-around to t. This behavior is the default in most code editors I know, for good reasons.

Sure. I'll make those changes.
For the your decline of company-backends change, I'm not sure I understand your reasons above. As someone who switched from vanilla Emacs to evil/vim keybinding, I find having company-files completion automatically invoked much more convenient than having to press C-x C-f each time I want completion.
If you're still convinced the change to company-backends is not necessary, I suggest adding the configuration of company-backends to the documentation of company module.

For the your decline of company-backends change, I'm not sure I understand your reasons above. As someone who switched from vanilla Emacs to evil/vim keybinding, I find having company-files completion automatically invoked much more convenient than having to press C-x C-f each time I want completion.

There's a spectrum to completion in editors. On one end is the manual approach: where you invoke the precise backend you need, and only when you need it (where completion won't bother you otherwise); and on the other end is the automatic approach: inundate the user with all candidates from all backends as they type, all the time, whether or not they need them.
I'm staunched in the manual camp. IDEs tend to be the opposite. Our company module is designed for the former, and I don't intend to change that (at least, as the default), but I may add a feature flag later to support it (then make idle-completion opt-in). It'll be some time until I get around to it though, and until then, I'd rather not change it. What's more: it's trivial enough for users to change themselves.

If you're still convinced the change to company-backends is not necessary, I suggest adding the configuration of company-backends to the documentation of company module.

I've documented how to use set-company-backend! in the module's docs, but I'll look into make it clearer.


JointTrajectoryFollower not working

Hello,
I've installed the extension and it's dependencies and created a follower server, however ros2 action list doesn't return anything.
From what i've understood you need the ros_bridge extension you've also developed, which doesn't load because I am using ros2. Is there something I'm missing or have you not yet release the ros2_bridge ?

Hi @09ubberboy90
I have not implemented the extensions for ROS 2 yet...
It is on my working list

Thank you.
Is there anything i can help you with ?


charts: bump harvester-node-disk-manager to v0.5.0

- update CRD
- update values/daemonset for debug configuration

can we split the commit and backport the template and CRD files to the master first, thanks.

can we split the commit and backport the template and CRD files to the master first, thanks.

Sure, open the PR (https://github.com/harvester/charts/pull/187) to master


nodejs - jsbarcode - image generate different in docker container and in local

I'm realizing barre code based on doctor rpps in France. This just a eleven number code.
"canvas": "^2.6.1",
"jsbarcode": "^3.11.0",
In local the image is perfect.

Under docker :

Need help to understand

Does the docker image have any monospace fonts installed?


Update RSpec to 3.2

convert specs to RSpec 3.2
fix a few broken specs
tweaks for Travis
rspec tweaks
rspec-retry to rerun failures on Travis

@e2, it seems the bulk of this is to change the way tests are performed for the new version? Is it backward compatible with older versions?
And I notice several Celluloid internals were modified to remove space between method definitions in the class bodies. Not sure if this kind of compression is what we're looking for? What's the code standard convention here @tarcieri?

@digitalextremist - this is mostly to migrate to RSpec >= 3.0 (which was released ages ago). The files should be compatible with RSpec 2.99 (except maybe the RSpec 3 specific config), though RSpec 3.0 has better features and better checking (e.g. the mock validation could help turn many Celluloid "integration specs" into unit tests with proper mocking - and especially help with the flaky specs).
For Celluloid specifically, the difference is mostly avoiding deprecations.
Also, project with the monkey-patching disabled could not use the Celluloid RSpec helpers (because the shared examples in lib/celluloid/rspec depended on the monkey-patching).

And I notice several Celluloid internals were modified to remove space between method definitions in the class bodies.

I did whatever matches RuboCop settings - if there's any specific style guide, it should be set in RuboCop's config. The compressed spaces and quotes are mostly auto-correctable by RuboCop.
(RuboCop isn't run on Travis though).

I'm not sure why the failure happened - that should've been handled by rspec-retry...

Overall this looks good, but I'm also curious why rspec-retry didn't work...

Also I'm down to get the code to the state where we don't rely on monkeypatching...

I think rspec-retry ran out of retries (3 may not be enough - especially when test is rerun successively).
I'll work something out, especially since retries are needed in development too...

Feel free to mark any of the StackDumper tests that are flaking out as pending because they're flaky. They've been notoriously unreliable, and unreliable tests suck :wink:

@tarcieri - yes, I'll mark flaky tests with flaky: true, so only those get reruns - and so they can be easily reconfigure in the spec_helper.rb. (Flaky specs should be explicit, while normal tests shouldn't need reruns - it's confusing during dev).

Ok, I'm done on my side.

@e2 another build failure :cry:

LOL! 5 times is not enough!
RSpec::Retry: 2nd try ./spec/celluloid/stack_dump_spec.rb:58
RSpec::Retry: 3rd try ./spec/celluloid/stack_dump_spec.rb:58
RSpec::Retry: 4th try ./spec/celluloid/stack_dump_spec.rb:58
RSpec::Retry: 5th try ./spec/celluloid/stack_dump_spec.rb:58
    should have the correct roles (FAILED - 1)

@tarcieri - any idea why it's failing? (Why the first thread is not present in the results).

The entire stack dump spec is racy and not well-written. I'd say disable
all the flaky specs until someone can investigate/rewrite them.

--
Tony Arcieri

The fix looks dumb, but basically there are too many messages - and the internal_pool sometimes doesn't have a chance to end up with the 3 tested threads (2 busy + 1 idle).
The expected idle thread (nil) comes from an "initialize" message - the 3rd thread for that message doesn't happen if there isn't enough time after the blocking() call (the reason for the first sleep()).
I think the InternalPool should simply be mocked - because that's all the StackDump test is really using as a "fixture".

The StackDump tests are definitely the biggest culprits in the flakiness of the test suite, and having a flaky test suite really sucks. I definitely do appreciate you doing the work to shore it up!
My vote is to disable them until they can be fixed. Flaky tests are useless and detrimental. If you have ideas to fix the great, if not mark them pending.

Overall, threads combined with mutexes and timers are hard to unit test. The only way to do that theoretically is to implement a custom "scheduler" or "virtual machine" (e.g. you can't deterministically call code after a sleep has started).
Not worth the effort. There are some tests that can be quite elegantly turned into unit tests using queues - but that alone can be quite mind bending when combined with mocks.
This makes "integration specs" easier than trying to force Celluloid into being convenient for unit tests.
The best idea I have is to gradually make it impossible to create flaky specs in the first place - by not making the full Celluloid stack available in specs (e.g. by partially making threads, conditionals and timers unavailable during unit tests).
A first step would be to fail tests if threads are left hanging (every spec should explicitly clean up after itself). But this is probably done while fixing bugs, though.
Oh, and since RSpec 3.0 pending tests fail if they "pass" (pending tests are expected to fail, or that itself is a failure). So yeah, flaky tests are detrimental - even more so.
All MRI versions are green, btw.

This looks good to me now. Thanks!


fix: fixed the classname

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Make schema more sane using graphql-tools

cc: @jvoros

Not sure why travis is failing right now.. It works fine. I'll investigate later.

Codecov Report

Merging #209 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #209   +/-   ##
=======================================
  Coverage   59.83%   59.83%
=======================================
  Files          16       16
  Lines         244      244
  Branches       52       50    -2
=======================================
  Hits          146      146
  Misses         98       98

Impacted Files
Coverage Δ

app/pages/Home.tsx
100% <ø> (ø)
:arrow_up:

app/components/SearchResults.tsx
95% <ø> (ø)
:arrow_up:

app/cards/CardListContainer.tsx
100% <ø> (ø)
:arrow_up:

app/cards/CardList.tsx
90.9% <ø> (ø)
:arrow_up:

app/cards/Card.tsx
92.85% <ø> (ø)
:arrow_up:

app/categories/Category.tsx
92.85% <ø> (ø)
:arrow_up:

app/components/Announcements.tsx
0% <0%> (ø)
: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 fe0dd6e...3534b5a. Read the comment docs.


#1243 removed pre ICS code of trust manager

@dpreussler I want to merge your commits into https://github.com/Vorlonsoft/AndroidAsyncHTTP Could you please explain, minSdkVersion 9, why you remove pre ICS (sdk 14) code?


add vm e2e and integration testing framework

add vm e2e and integration testing framework

close as a duplicate of https://github.com/rancher/harvester/issues/45


Menu Component Placeholder/Text

We have the requirement to allow a short text beneath the Menu Icon

We tried with the :placeholder.prop but this had no effect.
https://parlamentsdienstecore.lambda-it.ch/?path=/story/layout--menu
Is this possible with this component already?
Also here: https://xd.adobe.com/view/727dd55b-1491-4151-8f5d-db8600161be8-272e/screen/08bb52a8-fc0f-492c-9528-6445ecb33b7d/specs/

The label for the Menu Icon like "Aktionen" is missing.


Update Erlang Guide to use Staged Joins

like the elixir guide does

done: https://riak-core-lite.github.io/blog/pages/getting-started/


PM2 cant be bundled

hi i have this code
let pmx = require('pmx').init({
    http: true, // HTTP routes logging (default: true)
    ignore_routes: [/socket\.io/, /notFound/], // Ignore http routes with this pattern (Default: [])
    errors: true, // Exceptions loggin (default: true)
    custom_probes: true, // Auto expose JS Loop Latency and HTTP req/s as custom metrics
    network: true, // Network monitoring at the application level
    ports: true  // Shows which ports your app is listening on (default: false)
});

and i get this error
Cannot read property 'slice' of undefined
Profiling.detectV8Profiler = function(cb) {
  var require_paths = require.main.paths.slice();

the code above is required for app logging!

Please, check 1.3.42

so far the error is gone, i will test on server soon :)


Landing page

I want a landing page that shows me an overview of important information (sumary of power consumption, list of connected plugs).

Hey team! Please add your planning poker estimate with Zenhub @euclid9099 @Flexla54

Definition of done

[ ] Design a Landing page (that contains account-settings, multiple-plug-analysis and list of all connected plugs) in Figma
[ ] The landing page should be implemented like in the Figma design file

Tasks

[ ] implement a front-end page only accessible to logged in users
[ ] have a side menu to fill in the future with links to subpages


Implement map_overlap

Planning on merging this soonish to unblock rolling work, but feedback welcome

The complexity is in class MapOverlapInterleavePartitions(Expr):, everything else is fairly straightforward, so a set of eyeballs would be appreciated there

I am merging this to unblock rolling work, but happy to iterate further


Very outdated mixed-reality/tree/docs/enthusiast-guide/immersive-headset-hardware-details.md

immersive-headset-hardware-details
Still mentions old HMD's as main specs

Resolved! Thank you


convert httpbin to go-based

Please provide a description of this PR:
Convert this to a slimmer Go-based httpbin that utilizes port 8080 by default. Also version locked tags to prevent breakages when a new version is released.

kong/httpbin                                  latest                                            8f751f486162   2 days ago      626MB
mccutchen/go-httpbin                          v2.5.0                                            fbabd38ea186   22 months ago   28.6MB

this will save a lot when pulling image

Also these still have the kong ref:

samples/extauthz/local-ext-authz.yaml
samples/httpbin/README.md

In response to a cherrypick label: #53315 failed to apply on top of branch "release-1.22":
Applying: convert httpbin to go-based
Using index info to reconstruct a base tree...
M	samples/httpbin/httpbin-vault.yaml
M	samples/httpbin/httpbin.yaml
Falling back to patching base and 3-way merge...
Auto-merging samples/httpbin/httpbin.yaml
CONFLICT (content): Merge conflict in samples/httpbin/httpbin.yaml
Auto-merging samples/httpbin/httpbin-vault.yaml
CONFLICT (content): Merge conflict in samples/httpbin/httpbin-vault.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 convert httpbin to go-based
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #53341

In response to a cherrypick label: #53315 failed to apply on top of branch "release-1.23":
Applying: convert httpbin to go-based
Using index info to reconstruct a base tree...
M	samples/httpbin/httpbin.yaml
Falling back to patching base and 3-way merge...
Auto-merging samples/httpbin/httpbin.yaml
CONFLICT (content): Merge conflict in samples/httpbin/httpbin.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 convert httpbin to go-based
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #53342


put VTTCue on window if it is null

Didn't realize you had this PR. What are your thoughts on https://github.com/gkatsev/vtt.js/pull/10, @BrandonOCasey ?


More convenient instantiating of View Controllers

Extension for view controllers with a known storyboard identifier that allows to instantiate view controller from specified storyboard;
It returns exact VC's subclass, so there is no need to make any casts after calling this method;
Before:
let myVC = Storyboards.Main.instantiateViewControllerWithIdentifier(CustomViewController.storyboardIdentifier!)
    as! CustomViewController

Now:
let myVC = CustomViewController.instantiateFromStoryboard(.Main)

This is great!


Video cacher fix

This patch fixes a problem with the PNVideoCacher that cached a 403 forbidden web page as a video.
For the solution I ensured that the downloaded data status is correct and the content type is a video type. Otherwise, an error is returned by the video.

Coverage decreased (-0.03%) to 54.86% when pulling 01b20cc22e3140bf77ed51665331204c3b90a754 on development_videoCacherFix into ca9276d8fc388219a72cb6b3b57b1c7691ea20c0 on development.


Updates for craft 4 requirements

Craft 4 has some typing specifications necessary for plugins to work. These changes should allow the plugin to work.

OK thx..  :)
will ask developer to check that asap..


"removeFromMemcached" is called after tomcat failover

From<EMAIL_ADDRESS>on December 11, 2012 00:30:26
Gist:
I am currently using this solution to scale our e-commerce application. I use sticky sessions and that all the calls to the server should update the session, even for js and images.
Here is my config

And that I see removedFromMemcached Called twice, where it succeeds on first and failed on the 2nd (Because it was deleted already.)
Environment:
2 Tomcat 6
1 Apache HTTP Server
1 Memcached (CouchBase server)
What steps will reproduce the problem?

Start the Tomcat 1 & 2
Launch the browser and sign in
Kill one tomcat instance.
The other tomcat should take responsibility of the session.

What is the expected output? What do you see instead?
I expect that the removeFromMemcached is called once.

I see 2 of them..

FINE: Found session with id 449A7CF0B4B06CA0B5B7184A1F212204-n1.node2
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.MemcachedBackupSessionManager remove
FINE: remove invoked, removeFromMemcached: true, id: 449A7CF0B4B06CA0B5B7184A1F212204-n1.node2
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.MemcachedSessionService deleteFromMemcached
FINE: Deleting session from memcached: 449A7CF0B4B06CA0B5B7184A1F212204-n1.node2
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.SessionIdFormat createSessionId
FINE: Creating new session id with orig id 'ping' and memcached id 'n1'.
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.NodeAvailabilityCache updateIsNodeAvailable
FINE: CacheLoader returned node availability 'true' for node 'n1'.
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.MemcachedSessionService loadFromMemcached
FINE: Loading session from memcached: 449A7CF0B4B06CA0B5B7184A1F212204-n1.node2
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.MemcachedSessionService loadFromMemcached
FINE: Session 449A7CF0B4B06CA0B5B7184A1F212204-n1.node2 not found in memcached.
Dec 10, 2012 4:36:41 PM de.javakaffee.web.msm.MemcachedSessionService createSession
FINE: createSession invoked: null

What version of the product are you using? On what operating system?
I am using the latest release you specified in the downloads. This is my testing from my environment.

Tomcat 1 --> RedHat
Tomcat 2 --> ubuntu 12.04
Apache HTTP Server --> Windows 7
CouchBase --> Ubuntu 12.04

Please provide any additional information below.

This is only happening for one environment going to another.

I've assigned Redhat as my node1 and primary server. If it goes down, tomcat 2 intermittently restores it from memcached.

If I reverse it, when ubunutu goes down, Redhat tomcat fails to initialize the session due to what I see above.

Maybe you have 2 cents, and any settings I need to set.

Thanks,
Jay

Original issue: http://code.google.com/p/memcached-session-manager/issues/detail?id=154

From<EMAIL_ADDRESS>on December 10, 2012 15:33:47
The title is incorrect, loading from memcached 2X where delete is called in between. The second load fails to load the session, thus the session is lost. Can we turn this "removeFromMemcached" off?

From martin.grotzke on December 11, 2012 13:21:20
The invocation of MemcachedBackupSessionManager.remove should be caused by a session expiration.
As long as your code does not call this, I wonder what's the reason for it.
Did you check that the server time is the same on both machines? That you only experience this from one machine to the other but not the other way around might be a hint on a server time mismatch, too.
Summary: "removeFromMemcached" is called after tomcat failover   (was: "removeFromMemcached" is called 2X)

From<EMAIL_ADDRESS>on December 11, 2012 15:16:14
Hello Martin,
Good point. There was a few lag between 2 machines. I was actually looking at the code and looking why that is called, and I noticed that there was a background task that sweeps through the sessions and cleans up.
Great stuff. I actually like this solution over the clustering between tomcats.
Thanks,
Melvin

From martin.grotzke on December 11, 2012 15:19:13
Thanx! :-) So it's working now?

From<EMAIL_ADDRESS>on December 12, 2012 13:42:15
Yes! It works very well. I would put more testing around this because I want to put this live on our production systems.
If I find more issues I'll let you know. :)
Cheers!
Melvin

From martin.grotzke on December 13, 2012 00:17:26
Ok, great!
Status: Invalid

From<EMAIL_ADDRESS>on May 24, 2013 06:12:26
Hi,
I am having clustering environment with 2 memcache instances and 2 servers and added the jars in tomcat lib directory which are available in the site
https://code.google.com/p/memcached-session-manager/wiki/SetupAndConfiguration

While disabling the cookies am getting the data from the session.

problem:
========
But with enabling cookies am not getting data from the session with multiple tabs on the same browser hitting at the same time.

Expected:
========
It should work with enabling/Disabling cookies.

Configuration in context.xml of tomcat:

<Manager className="de.javakaffee.web.msm.MemcachedBackupSessionManager"
memcachedNodes="n1:localhost:11211"
sticky="false"
operationTimeout="60000"
sessionBackupTimeout="300000"
sessionBackupAsync="false"
lockingMode="none"
transcoderFactoryClass="de.javakaffee.web.msm.JavaSerializationTranscoderFactory"
/>

Previous without clustering able to get the data in multiple tabs.

When am searching with different search criterias in multiple tabs am able to get the data in one tab and am getting session as null for other tab in same browser.

Thanks.


ow to disable throw out up and down?

In our App we need only left and right swipes, so we want to disable up and down swipes. now the App is submitting in up when user moves to the corner right or left.

we can disable it by adding only left and right direction to  allowedDirections property in stackconfig. Thanks


Add a SelectFunc(filter) selector that just delegates to the given filter function

This would let users create custom selectors (so that, say, they can combine their logic with other selectors) without having to create a custom Selector subclass.

I withdraw what I've said. The function signature is so simple that it is Ok to pass it as an argument. That is more Pythonic. Similar to the first argument of filter().


Support MaybeRefOrGetter as the type for passing props

What problem does this feature solve?
Getters are extremely useful to avoid the overhead of computed properties and to pass data that should be re-evaluated when accessed. So something like () => new Date() vs new Date(). #7997 is great and provides some nice tools for handling getters and normalizing values specifically the MaybeRefOrGetter type and the toValue and toRef utilities.
Props basically have the type MaybeRef because you can bind either a static value or a ref :value="new Date()" vs :value="myDateRef". However you cannot pass a getter. Currently in order to do so you'd have to make the type for your prop Date | (() => Date) and then normalize the value using toRef or toValue inside your component. Then you could pass :value="myDateGetter".
It would be a nice developer experience to allow all props to be MaybeRefOrGetter and automatically normalize the value. So you could bind myDateGetter while the prop type would simply Date.
What does the proposed API look like?
DateLabel.vue
<template>
  {{ value.toLocaleString() }}
</template>

When using the DateLabel component all three of these would pass type validation and render the date label.
<template>
  <DateLabel :value="dateConst" />
  <DateLabel :value="dateRef" />
  <DateLabel :value="dateGetter" />
</template>

<DateLabel :value="dateGetter()" />

<DateLabel :value="dateGetter()" />

@Shyam-Chen I could be wrong but I believe there is a distinct difference between that and what I'm asking for. A getter normalized to a ref using toRef calls the getter itself each time .value is accessed. Which would make this work differently in a few ways

If you never used the value the getter would never be called (possibly because another prop or user interaction is needed)
When you access .value on a getter that is normalized to a ref using toRef the getter is called each time. Which means in my example above if the label was "live" the outcome would be very different. In this example if you passed the getter () => new Date() the date could update every second. If you called the getter when assigning the prop like :date="dateGetter()" it would not.

<template>
  {{ value.toLocaleString() }}
</template>

Use computed ?

Props basically have the type MaybeRef because you can bind either a static value or a ref

This may be a misunderstanding. You cannot pass a ref, props do not have a MaybeRef type. The ref will aways be unwrapped in the parent's render function. What happens, essentially, is this:
render: () => h(DateLabel, {  date: dateRef.value }

So the child never receives the ref, it always receives the value - a Date. So a prop defined as type Date can only receive a Date, not a Ref. Unwrapping happens in the parent's render function/template, not in the child's props. (In a manually written render function, you would have to do the unwrapping yourself).
To do a similar thing with a getter, you would do what @Shyam-Chen initially proposed - you would evaluate the getter in the template.
<DateLabel :value="dateGetter()" />

Now, if you want the getter to be evaluated in the child, only when the prop is actually used there, then you would need to write your component to support that.

<template>
  {{ typeof date === 'function' ? date() : date }}
</template>

I kind of understand that you want this to happen automatically, but I'm not sure where and when.
We can't "unwrap" getters in the parent automatically like we do refs, because getters are just functions - and so far, we do allow you do pass a function as a prop value to a child. If we decided to suddenly "unwrap" (call) all functions we pass as props, we would certainly break lots and lots of userland code, so this is not going to happen.
Similarly, unwrapping it with some "magic" in the props in the child on each access would break existing userland code all the same.

@pleek91 Can you probvide feedback to what I layed out here? Otherwise I would close this request as it's not really possible in the way that I understand your request to work.

@LinusBorg thanks for the detailed response. That totally makes sense. I guess I'm suggesting that when the template is converted into a render function to use toValue rather than unwref. But you're right that would break things.
I was thinking about this from a pure types perspective where currently if you're using typescript the type for a prop that is a Date is effectively MaybeRef<Date> because you can bind Date | Ref<Date> and if you tried to pass () => Date that wouldn't be valid. So from that perspective allowing the type that you can bind to include the getter and using toValue rather than unref would be a non breaking change. BUT I wasn't considering non typescript users or typescript users who have a prop that is indeed supposed to be a function.
In my project I am currently doing what you suggested by making a MaybeGetter<T> = T | () => T that I use for props. Then I'm using toValue or toRef in my components rather than typeof date === 'function' ? date() : date. Which is working well but I did have to update quite a few components in order to support that functionality. So this feature request was maybe short sighted based on some frustration.
Thanks again for the detailed explanation!


Add LICENSE

As of now, tree-sitter-teal doesn't have any license, and so it's unclear how to properly integrate it in other projects (for example, adding support for teal in helix). It would be nice to have a LICENSE for that, maybe the same MIT license that teal itself has ?

Apologies for the slow response. I've just added the license file in 3db6559, thanks for opening the issue.


C-MOVE SCU/SCP not possible?

Due to the way the program structure works, its not possible to run a C-MOVE operation with the SCU as the move destination due to a C-MOVE invoking a separate Association request from the SCP for the C-STORE request (unlike C-GET which uses the same Association to receive the C-STORE request).
I'm not actually sure if this is a bug or if this is the desired behaviour. Why use a C-MOVE to fetch to the running AE instead of a C-GET?

From PS3.4 Section C.4.2:

The C-MOVE operation allows an application entity to instruct another application entity to transfer stored  SOP Instances to another application entity using the C-STORE operation.

So the current behaviour is correct, you can't use a C-MOVE to direct a peer AE to transfer SOP Instances to the local AE.

Why use a C-MOVE to fetch to the running AE instead of a C-GET?

I know Orthanc does not support C-GET.


fix: ui fix about publish & setting page

Description
UI fix about publish & setting page.  text truncate  instead of overflow & set column width wider

publish page

setting page

Task Item
fix #5246
Screenshots

Coverage decreased (-0.002%) to 55.37% when pulling a6c6750b23f40aa8adc544eefc8159bb71c494c5 on julong/publish-ui-fix into 286bef23a95fdcfba0e1fcb67791afaffd399dcb on main.