Rostest catkin/meta-ros dependency

While building core-image-ros-roscore, from scratch, I get a catkin error.
TL;DR: The error is that catkin can't find the 'rostest' package and one of the packages (in this case 'base-local-planner').
Error:
ERROR: Function failed: do_configure (log file is located at /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/temp/log.do_configure.31782)
ERROR: Logfile of failure stored in: /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/temp/log.do_configure.31782
Log data follows:
| DEBUG: Executing python function sysroot_cleansstate
| DEBUG: Python function sysroot_cleansstate finished
| DEBUG: SITE files ['endian-little', 'bit-32', 'arm-common', 'common-linux', 'common-glibc', 'arm-linux', 'arm-linux-gnueabi', 'common']
| DEBUG: Executing shell function autotools_preconfigure
| DEBUG: Shell function autotools_preconfigure finished
| DEBUG: Executing python function autotools_copy_aclocals
| DEBUG: Python function autotools_copy_aclocals finished
| DEBUG: Executing shell function do_configure
| NOTE: cmake.bbclass no longer uses OECMAKE_BUILDPATH.  The default behaviour is now out-of-tree builds with B=WORKDIR/build.
| -- The C compiler identification is GNU 4.9.1
| -- The CXX compiler identification is GNU 4.9.1
| -- Check for working C compiler: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/arm-poky-linux-gnueabi/arm-poky-linux-gnueabi-gcc
| -- Check for working C compiler: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/arm-poky-linux-gnueabi/arm-poky-linux-gnueabi-gcc -- works
| -- Detecting C compiler ABI info
| -- Detecting C compiler ABI info - done
| -- Check for working CXX compiler: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/arm-poky-linux-gnueabi/arm-poky-linux-gnueabi-g++
| -- Check for working CXX compiler: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/arm-poky-linux-gnueabi/arm-poky-linux-gnueabi-g++ -- works
| -- Detecting CXX compiler ABI info
| -- Detecting CXX compiler ABI info - done
| -- Using CATKIN_DEVEL_PREFIX: /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/build/devel
| -- Using CMAKE_PREFIX_PATH: /opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/usr;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr
| -- This workspace overlays: /opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/usr;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr
| -- Found PythonInterp: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/python-native/python (found version "2.7.3")
| -- Using PYTHON_EXECUTABLE: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/bin/python-native/python
| -- Using default Python package layout
| -- Found PY_em: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/lib/python2.7/site-packages/em.pyc
| -- Using empy: /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr/lib/python2.7/site-packages/em.pyc
| -- Using CATKIN_ENABLE_TESTING: 0
| -- catkin 0.6.18
| -- Using these message generators: gencpp;genlisp;genpy
| CMake Warning at /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/opt/ros/indigo/share/catkin/cmake/catkinConfig.cmake:76 (find_package):
|   Could not find a package configuration file provided by "rostest" with any
|   of the following names:
|
|     rostestConfig.cmake
|     rostest-config.cmake
|
|   Add the installation prefix of "rostest" to CMAKE_PREFIX_PATH or set
|   "rostest_DIR" to a directory containing one of the above files.  If
|   "rostest" provides a separate development package or SDK, be sure it has
|   been installed.
| Call Stack (most recent call first):
|   CMakeLists.txt:4 (find_package)
|
|
| -- Could not find the required component 'rostest'. The following CMake error indicates that you either need to install the package with the same name or change your environment so that it can be found.
| CMake Error at /opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/opt/ros/indigo/share/catkin/cmake/catkinConfig.cmake:83 (find_package):
|   Could not find a package configuration file provided by "rostest" with any
|   of the following names:
|
|     rostestConfig.cmake
|     rostest-config.cmake
|
|   Add the installation prefix of "rostest" to CMAKE_PREFIX_PATH or set
|   "rostest_DIR" to a directory containing one of the above files.  If
|   "rostest" provides a separate development package or SDK, be sure it has
|   been installed.
| Call Stack (most recent call first):
|   CMakeLists.txt:4 (find_package)
|
|
| -- Configuring incomplete, errors occurred!
| See also "/opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/build/CMakeFiles/CMakeOutput.log".
| WARNING: /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/temp/run.do_configure.31782:1 exit 1 from
|   cmake ${OECMAKE_SITEFILE} /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/navigation-1.12.10/base_local_planner -DCMAKE_INSTALL_PREFIX:PATH=/usr -DCMAKE_INSTALL_BINDIR:PATH=/usr/bin -DCMAKE_INSTALL_SBINDIR:PATH=/usr/sbin -DCMAKE_INSTALL_LIBEXECDIR:PATH=/usr/lib/base-local-planner -DCMAKE_INSTALL_SYSCONFDIR:PATH=/etc -DCMAKE_INSTALL_SHAREDSTATEDIR:PATH=/com -DCMAKE_INSTALL_LOCALSTATEDIR:PATH=/var -DCMAKE_INSTALL_LIBDIR:PATH=/usr/lib -DCMAKE_INSTALL_INCLUDEDIR:PATH=/usr/include -DCMAKE_INSTALL_DATAROOTDIR:PATH=/usr/share -DCMAKE_INSTALL_SO_NO_EXE=0 -DCMAKE_TOOLCHAIN_FILE=/opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/toolchain.cmake -DCMAKE_VERBOSE_MAKEFILE=1 -DCMAKE_PREFIX_PATH='/opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/phyflex-imx6-1/usr;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/opt/ros/indigo;/opt/yocto/yocto_fsl/build/tmp/sysroots/x86_64-linux/usr' -DCMAKE_INSTALL_PREFIX:PATH='/opt/ros/indigo' -DCATKIN_BUILD_BINARY_PACKAGE=ON -DSETUPTOOLS_DEB_LAYOUT=OFF -DCATKIN_ENABLE_TESTING=0 -Wno-dev
| ERROR: Function failed: do_configure (log file is located at /opt/yocto/yocto_fsl/build/tmp/work/cortexa9hf-vfp-neon-poky-linux-gnueabi/base-local-planner/1.12.10-r0/temp/log.do_configure.31782)
NOTE: recipe base-local-planner-1.12.10-r0: task do_configure: Failed

Some more of my thoughts:
The rostest dependency must be listed in the package.xml and CMakeLists.txt for base-local-planner, but not in the recipe for base-local-planner. There is some strange stuff going on with rostest, I think, so I didn't look into it further. Doing this does "fix" the problem, but I defer to an expert because I don't think this is what you actually want to do:
echo "DEPENDS += "rostest"" >> $FSL_DIR/sources/meta-ros/recipes-ros/navigation/base-local-planner_1.12.10.bb
I don't know anything about the rostest package, but I think it's special somehow. I also get the impression that if I was building a different set of features, a different package might fail here instead of the base-local-planner, so just adding it to that recipe is probably not the fix. Hopefully, someone recognizes this, and has a good solution.

The issue you reported is well-known for two years now, it has been thoroughly discussed with the ROS core developers, and there is an commonly agreed solution to it.
If you look into the meta-ros git history, you will find these two interesting sets of changes:
Commit 9879db in pull request #161 [2] shows how we patch the ROS packages with CATKIN_ENABLE_TESTING to make rostest optional.
Pull request #239 [1] includes some more commits where we address the issue of making rostest dependencies optional. In fact, most patches (maybe even all) were provided back tothe upstream repositories and merged into their development. Hence, meta-ros only contains the patches for those ROS packages that have released new versions and recipes that have not been updated---which are only some very few by now
You can also find the discussion on this topic with Dirk Thomas, catkin's main developer, in the rosdistro issue tracker ros/rosdistro#3010 [3].
With all that information at hand, I hope that you can produce a patch that resolves the issue just as it was discussed and resolved in the past. We are happy to review any pull request on this matter.
[1] https://github.com/bmwcarit/meta-ros/pull/239
[2] https://github.com/bmwcarit/meta-ros/pull/161/commits/9879db3a97a028c383a321e068e3c1980cb37f6b
[3] https://github.com/ros/rosdistro/issues/3010
P.S.: I am on vacation, travelling through Europe and only have limited internet access; so I will not get much involved in lengthy technical discussions. Also I do not have access to any build server to check or confirm the functionality of any patch, and hence will rely on others confirming that patches work.


temporalite: 0.2.0 -> 0.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/temporalio/temporalite/releases.
meta.description for temporalite is: "An experimental distribution of Temporal that runs as a single process"
meta.homepage for temporalite is: "https://github.com/temporalio/temporalite"
meta.changelog for temporalite is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.0 with grep in /nix/store/m6ihhz7d2pmhk15l4pgnknvgrr6v667j-temporalite-0.3.0
found 0.3.0 in filename of file in /nix/store/m6ihhz7d2pmhk15l4pgnknvgrr6v667j-temporalite-0.3.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
temporalite

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/m6ihhz7d2pmhk15l4pgnknvgrr6v667j-temporalite-0.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A temporalite https://github.com/r-ryantm/nixpkgs/archive/f2f5ecb76bf72975390130f661768b4d40ad753c.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m6ihhz7d2pmhk15l4pgnknvgrr6v667j-temporalite-0.3.0
ls -la /nix/store/m6ihhz7d2pmhk15l4pgnknvgrr6v667j-temporalite-0.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

temporalite

Maintainer pings
cc @lucperkins for testing.

@ofborg build temporalite temporalite.passthru.tests


Feat/zarr io

add more functionality to MultiscaleGroup, and a lot of tests!

I peeled from_arrays and from_array_props off the MultiscaleGroup class and put it in the tests as over-engineered test utilities. If we need these in the library, we an always bring them back.

i think this is ready to go, any further thoughts / concerns @dstansby ?


Adjust references to the runner JAR for the fast-jar format

Thanks for spotting it, @jamesnetherton!


enhance: add sync.Pool for logMessage

Signed-off-by: zhangyue<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
As the title told, when consume and produce log message in pouch, the logMessage struct be deleted and constructed back and forth, this pr add a sync.Pool for logMessage to cache logMessage struct in memory to enhance gc performance.
Ⅱ. Does this pull request fix one issue?

None.
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
None.
Ⅳ. Describe how to verify it
go test -check.f PouchAttachSuite
Ⅴ. Special notes for reviews
None.

Any update for this? @ZYecho
Actually I recommend that more information could be added in pull request description, for example what will be achieved by this pull request, what benefit this would bring.
Not just what we did.
Thanks a lot.

Any update for this? @ZYecho
Actually I recommend that more information could be added in pull request description, for example what will be achieved by this pull request, what benefit this would bring.
Not just what we did.
Thanks a lot.

Thanks for your reply, I have updated the desc of this pr.


Migrate content from local files to contentful

[ ] Profile
[ ] Page
[ ] Experience
[ ] Training
[ ] Skill
[ ] Certificate

Optional: Use content management api

Is dependent on #12


remove unused xcode versions

Would it be possible to implement something which removes anything except the version i wanted to initialize. I created this workaround for now:
      - name: Initialize latest xcode
        uses<EMAIL_ADDRESS>        with:
          xcode-version: ${{ matrix.xcode }}

      - name: Remove old xcode versions
        run: |
          echo "Searching for Xcode versions:"
          find /Applications -name "Xcode_*" -maxdepth 1 -mindepth 1
          echo "Removing old Xcode versions..."
          find /Applications -name "Xcode_*" -maxdepth 1 -mindepth 1 | grep -v ${{ matrix.xcode }} | xargs rm -rf
          echo "Available Xcode versions after removal:"
          find /Applications -name "Xcode_*" -maxdepth 1 -mindepth 1

For reference https://github.com/actions/runner-images/issues/2840

+1 for this.
Lately GitHub runners have been failing due to space constraints (with no change on our part). And it seems GitHub's solution is to just delete stuff on the runner that we don't need.
Since each Mac runner has a bunch of versions of Xcode installed this would be nice.

@BrentMifsud i agree that space constraints are a issue, and this helps definitely to free up space in fact much more space than the initial 17G free space, but i think some of the underlying reasons are in my case for example:
<EMAIL_ADDRESS>uses 3G
<EMAIL_ADDRESS>uses 10G
Which is almost all of the 17G free macos-latest runner space.

for reference https://github.com/swift-actions/setup-swift/issues/682


feat(shared-data): json protocol schema 8

Add JSON protocol schema 8, which is similar in semantic concept to json protocol schema 7 but

Requires user specification of the schema for the command List
This allows json protocols to switch which commands they're interning without requiring further bumps to the protocol schema
Does the same thing for command annotations, liquids, and labware
For the same reasons.

This should be a big step forward in flexibility for our protocol schemas.
todo

[ ] make sure the schemas work
[ ] incorporate everybody's ideas

As discussed in the comments on 13854, we should add some sort of deckVersion field along with deckId to allow robot server/protocol runner to know whether to load fixed trash or not, rather than just relying on schema version moving forward.


'no android devices found'

adb is connected. I can run adb shell no problem.

I know we verified that this was working last week, so something's going on. Let me test this flow out on my end.

Ok I'm running into the same issue with the pairing workflow. We'll investigate on our end. In the meantime, using an emulator seems to work fine. We'll report back when we've root caused the issue.

Alright this was due to a silly mistake in the documentation. We were missing step (2) below:

Pair your device over wifi using the standard adb pair HOST:PORT CODE command (docs).

eg: adb pair <IP_ADDRESS>:46851 123456

Tell your device to listen for connections on port 5555.

adb tcpip 5555

Once your device is listening for connections on port 5555, Maestro can connect to your device using your device's IP address (same as HOST above).

eg: maestro --host <IP_ADDRESS> test flow.yaml.

I've updated the documentation as well.
@ericmigi Closing since I believe this is fixed with the update to the documentation, but please feel free to open back up if this still doesn't work for you!


Fix populatedb

I want to merge this change because...i wan to update populatedb

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-1229 Move collection availability to mulichannel solution


Incompatibility of F.conv1d with 4D tensors in getItem of HDF5Dataset in utils.py

Hello,
Thank you for an amazing paper and great code!
I came across an error that occurs when processing the dataset, which has dimensions [batch_size, channels, spatial_res, temp_res], using F.conv1d, in these parts of utils.py:
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/blob/76f07bb26a1829ac5ba6665956f51f645ad6e322/common/utils.py#L86-L87
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/blob/76f07bb26a1829ac5ba6665956f51f645ad6e322/common/utils.py#L104
https://github.com/brandstetter-johannes/MP-Neural-PDE-Solvers/blob/76f07bb26a1829ac5ba6665956f51f645ad6e322/common/utils.py#L108
It expects either 2D unbatched or 3D batched, while our data is 4D. The convolution weights of shape [1, 1, 1, 5] suggest a 1D-like convolution in a 2D context, which F.conv1d cannot do.
I suggest replacing it with F.conv2d, which fixes the issue while keeping the intended 1d convolution operation due to weights' shape.

Dear @pkhudov you are right. Please proceed an make a PR! Thanks a lot


(#280) Add test to exercise various existing / new print functions

This commit adds a new test case that exercises various functions to print different types of of pages, BTree innards, and RC-allocator innards.

Add a function to print the super-block
Modularize BTree printing function to sub-fns for diff page types
Modularize Trunk printing function to sub-fns for pivots, bundles etc
Improve layout of allocator details, showing extent-ID, addr and count
Rename several print-fns to _sprint*(), taking an output stream handle.
Minor corrections to some trunk print fns for proper alignment of output

All output from this test case will now be go to the /tmp file where tests redirect stdout.

Hi, @ajhconway , @rtjohnso - This oldish change-set which ran into print-function naming conventions flak has been overhauled now. Here is what I did:

Rebased v/s /main, picking up Alex's rework of platform_log() and related changes done under SHA 235fae2
Reworked the set of new print functions that I had previously carved out of existing code to conform to the new naming / interface conventions introduced recently.

As an example, here are some new interfaces after this rework:
trunk_print_super_block(platform_log_handle *log_handle, trunk_handle *spl)
trunk_print_memtable(platform_log_handle *log_handle, trunk_handle *spl)
+trunk_print_branches_and_bundles(platform_log_handle *log_handle,
+                                 trunk_handle        *spl,
+                                 page_handle         *node)

+btree_print_leaf_node(platform_log_handle *log_handle,
+                      btree_config        *cfg,
+                      uint64               addr,
+                      btree_hdr           *hdr)

... and so on.

No more will you see those _sprint_ interfaces, using stream handles.
I had to fuss around with the output widths as due to the addition of kv bytes in PIVOTS output block, the width becomes slightly longer. Had to fuss around this in a few diff places.

The best way to re-review this would be to take a look at the diffs in this commit. That is the one where the reconciliation of these new print interfaces w/ Alex's rework can be found.
There are still a couple of minor niggles, which I will comb through and annotate in the diffs. They are minor; if we don't resolve them in this round, it's ok. I'll poke you off-line when we meet.

NOTE: As an alert, ... I have added this page to our docs Diagnostics & Troubleshooting, and see that the href-links are a broken now.
There is a bit of docs cleanup to be done due to this code rework. Will fix them all before merging.
I want to use this page as a way to communicate to users "some" details of our internal layouts.
Thanks. ... If you can take a quick 2nd look, I am ready to integrate this pending item.

@ajhconway - Thanks for a careful tooth-comb re-review. I fixed the straggling pieces of prototypes and various other cleanup you suggested.
The docs file .... I will re-check the href-links one final time before I merge; leave them for now.
Let me know when you've had a re-re-look, and pass this. I'd rather wait till this is finalized, as this change set has been a bit controversial. And want to make sure that it finally conforms all the required naming conventions.

About the test itself: It is there to ensure that the print methods are always working reliably, at least for some big'gish input set.
Otherwise, if they seg-fault when you really need them, that will be very awkward. It's mostly a regression test against less-used print methods; as a defense check against bit-rot.

@ajhconway - I made all the corrections you pointed out.

Took out the comment about 'static' config for key-size, as expanding that would involve more re-work & re-review. And all this is anyway going to come out.

Am going in for the merge ... once CI passes.


Fix concurrent map writes error

What does this PR do?
Fix #5953 error and prevent similar issues in accesslog in future

Motivation
Fixes #5953

More

[ ] Added/updated tests
[ ] Added/updated documentation

Additional Notes

Looks like Travis build is failing on install step, it didn't even reach lint/testing.
https://travis-ci.org/containous/traefik/builds/622335753#L646-L678

Hi @sh7dm ,
First off, thank you for this PR.
We understand why you want to add a mutex, and that would have been my first reaction. However, we would like to avoid that as it should not be needed, because there should not be any concurrent access of the log, and we'd rather fix the origin of the issue. Also, we're concerned that adding a mutex would hide other concurrent accesses that we missed and that we would like to fix.
Which is why we did this other PR instead for now:
https://github.com/containous/traefik/pull/5967
We're going to leave your PR open for a while, in case it turns out we still need it.

okay, you're right.

as the user in #5953 confirm that the problem is fixed by #5967, I will close this PR.
Thank you anyway :+1:


Ricardo da Verdade Silva - Amaze Us Backend Challenge

Here is my test. I hope you enjoy this one as much as I enjoyed to do this one.

I'm closing this PR to not polute my list of open PRs. Thank you for the opportunity to try to be part of b-yond!


Broken link on Wiki page

Link for the 'component-router' on the Wiki section 'Gallery of third party React Motion demos' is broken (link: http://in-flux.github.io/component-router/example/)

Correct link: http://in-flux.github.io/component-router/

yeah, thanks. I also did not update component-router example to use react-motion... So probably better to remove link for now.


Navigation changes for V2

Keep Timeline and Docs in the LHS navigation. Disable all other navigation items and respective screens.

Done
https://github.com/argoproj/argo/pull/491
https://github.com/argoproj/argo/pull/492


Date Picker not work

in my html :

in javascript:
$('.datepicker').pickadate({
selectMonths: true, // Creates a dropdown to control month
selectYears: 15 // Creates a dropdown of 15 years to control year
});
And I give a error:

This usually means you aren't including jQuery correctly


fix: run cmd tests depend on func.yaml

:broom: run tests use client API rather than writing a func.yaml

Updates the run command's tests to not rely on func.yaml, which is an implementation detail of the client package, instead using the client API to create the function.
Adds an optional function manipulation "setup" method to the tests which allows the test to mutate the function, taking the place logically of providing this exact func.yaml.
/kind cleanup

Close as incorporated into #1695 as prerequisite


Update README.md

Add information about service to boost Eddi and correct a couple of typos.
Resolves #276

Thanks!


Update build_def.bzl

To resolve the following two issues on a linux_aarch64 system:

prohibited conversions between vectors (https://github.com/tensorflow/tensorflow/issues/26731#issue-421396111)
prohibited explicit use of frame pointer register x29 in asm (https://github.com/tensorflow/tensorflow/issues/26731#issuecomment-499361715)

Hi petewarden, could you review the code change at your convenience?

It looks like the underlying #26731 is fixed, so I'm assuming we can close this now?


Add firebase Auth for login

Login and signup using firebase would be very convenient
I would like to implement this

Login and signup using firebase would be very convenient I would like to implement this

Sure, please proceed @abhijithhegde
Happy Coding..!


Combine previewResourceId, previewHelmValuesId and previewConfigurationId into one field

Right now, in the main state we have multiple fields starting with the "preview" prefix denoting what is being previewed.
A better solution would be to have a single previewTargetId field and check for the previewType to determine the type.
This might also apply to the selectedPath, selectedValuesFileId and selectedResourceId.

This was done in 2.0


Roadmap/Proposal

v1.x

HTTP Digest Auth supports neon

Proposal

zero-copy send on ASGI #8
anonymous access

If you have any ideas, please open a new issue, Thanks.

Can we run this on termux on android? if yes, how?

Can we run this on termux on android? if yes, how?

sorry, i have not idea


Proper handling of ISO datetimes

We have a schema where a lot of our properties are defined as string().datetime(). The generated strings doesn't validate against Zods schema though.
https://zod.dev/?id=iso-datetimes
So if I have a schema like this:
const Schema = z.object({
  created: z.string().datetime()
});

The result is:
"created": "created-88a852ee-6b28-46a1-a551-9cdc808d9daf"
Running the tests against that gives an error:
[
  {
    "code": "invalid_string",
    "validation": "datetime",
    "message": "Invalid datetime",
    "path": [
      0,
      "created"
    ]
  }
]

Thanks for bringing this to our attention.
This needs to be a new customization, do you want to create a Pull Request for this?


relocation R_X86_64_PC32 cannot be used against symbol internal/cpu.options

go version ubuntu 18.04
go version go1.12 linux/amd64

go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/xm/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/xm/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go-1.12"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go-1.12/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build519519221=/tmp/go-build -gno-record-gcc-switches"

go build -buildmode=c-archive -o wasm.a   // this is ok
when i use cmake build error
ld: error: relocation R_X86_64_PC32 cannot be used against symbol internal/cpu.options; recompile with -fPIC
>>> defined in ../lib/linux/wasm.a(go.o)
>>> referenced by cpu.go:195 (/usr/lib/go-1.12/src/internal/cpu/cpu.go:195)
>>>               go.o:(local.internal/cpu.processOptions) in archive ../lib/linux/wasm.a

What is the actual compiler or linker command that is failing?  What C linker are you using?

@ianlancetaylor gcc
C linker ,I use llvm lld and gcc ld

What is the actual compiler or linker command that is failing?

all fail

mac ok

@helperShang Can you be more specific about the commands you are running with the full output of the command execution?
@ianlancetaylor added the WaitingForInfo tag since it doesn't sound like we have enough information yet, but please remove it if you feel that you have enough information to resolve this.

@katiehockman
cmake
cmake_minimum_required(VERSION 3.10)
project(wasm_java)

set(CMAKE_CXX_STANDARD 11)

set(SOURCE_FILES VM.cpp jni_wrap.cxx)

#add jhi include
find_package(JNI)
if (JNI_FOUND)
    INCLUDE_DIRECTORIES(${JNI_INCLUDE_DIRS})
elseif (JNI_INCLUDE)
    INCLUDE_DIRECTORIES(${JNI_INCLUDE})
endif ()
FIND_PACKAGE(SWIG)
if (SWIG_FOUND)
    MESSAGE(STATUS ${SWIG_EXECUTABLE})
endif ()

IF (WIN32)
    MESSAGE(STATUS "Now is windows")
    SET(WASM_OS_DIR windows)
    SET(WASM_OS_LIB wasm.lib)
ELSEIF (APPLE)
    SET(WASM_OS_DIR darwin)
    SET(WASM_OS_LIB wasm.a)
    MESSAGE(STATUS "Now is Apple systens.")
ELSEIF (UNIX)
    MESSAGE(STATUS "Now is UNIX-like OS's. Including aPPLE os x  and CygWin")
    set(WASM_OS_DIR linux)
    set(WASM_OS_LIB wasm.a)
ENDIF ()

INCLUDE_DIRECTORIES(${CMAKE_SOURCE_DIR}/lib/${WASM_OS_DIR})
LINK_DIRECTORIES(${CMAKE_SOURCE_DIR}/lib/${WASM_OS_DIR})
ADD_LIBRARY(VM SHARED ${SOURCE_FILES})

if (APPLE OR UNIX)
    ADD_LIBRARY(bar SHARED IMPORTED)
    SET_TARGET_PROPERTIES(
            bar
            PROPERTIES IMPORTED_LOCATION
            ${CMAKE_SOURCE_DIR}/lib/${WASM_OS_DIR}/${WASM_OS_LIB}
    )
    TARGET_LINK_LIBRARIES(VM bar)
elseif (WIN32)
	# TARGET_LINK_LIBRARIES(VM ${WASM_OS_LIB})
endif ()

make info
/usr/bin/cmake -H/home/wmt/wasm-lib-jni -B/home/wmt/wasm-lib-jni/build --check-build-system CMakeFiles/Makefile.cmake 0
/usr/bin/cmake -E cmake_progress_start /home/wmt/wasm-lib-jni/build/CMakeFiles /home/wmt/wasm-lib-jni/build/CMakeFiles/progress.marks
make -f CMakeFiles/Makefile2 all
make[1]: Entering directory '/home/wmt/wasm-lib-jni/build'
make -f CMakeFiles/VM.dir/build.make CMakeFiles/VM.dir/depend
make[2]: Entering directory '/home/wmt/wasm-lib-jni/build'
cd /home/wmt/wasm-lib-jni/build && /usr/bin/cmake -E cmake_depends "Unix Makefiles" /home/wmt/wasm-lib-jni /home/wmt/wasm-lib-jni /home/wmt/wasm-lib-jni/build /home/wmt/wasm-lib-jni/build /home/wmt/wasm-lib-jni/build/CMakeFiles/VM.dir/DependInfo.cmake --color=
make[2]: Leaving directory '/home/wmt/wasm-lib-jni/build'
make -f CMakeFiles/VM.dir/build.make CMakeFiles/VM.dir/build
make[2]: Entering directory '/home/wmt/wasm-lib-jni/build'
[ 33%] Linking CXX shared library libVM.so
/usr/bin/cmake -E cmake_link_script CMakeFiles/VM.dir/link.txt --verbose=1
/usr/bin/c++ -fPIC -O3 -DNDEBUG  -shared -Wl,-soname,libVM.so -o libVM.so CMakeFiles/VM.dir/VM.cpp.o CMakeFiles/VM.dir/jni_wrap.cxx.o  -L/home/wmt/wasm-lib-jni/lib/linux -Wl,-rpath,/home/wmt/wasm-lib-jni/lib/linux ../lib/linux/wasm.a
/usr/bin/ld: ../lib/linux/wasm.a(go.o): relocation R_X86_64_PC32 against symbol `internal/cpu.options' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status
CMakeFiles/VM.dir/build.make:124: recipe for target 'libVM.so' failed
make[2]: *** [libVM.so] Error 1
make[2]: Leaving directory '/home/wmt/wasm-lib-jni/build'
CMakeFiles/Makefile2:70: recipe for target 'CMakeFiles/VM.dir/all' failed
make[1]: *** [CMakeFiles/VM.dir/all] Error 2
make[1]: Leaving directory '/home/wmt/wasm-lib-jni/build'
Makefile:86: recipe for target 'all' failed
make: *** [all] Error 2

There is no current support for using -buildmode=c-archive to build a Go object and to link that Go object into a shared library.
I believe you can get it to work if you link the shared object with -Bsymbolic.  That is, something like:
/usr/bin/c++ -fPIC -O3 -DNDEBUG  -shared -Wl,-Bsymbolic -Wl,-soname,libVM.so -o libVM.so CMakeFiles/VM.dir/VM.cpp.o CMakeFiles/VM.dir/jni_wrap.cxx.o  -L/home/wmt/wasm-lib-jni/lib/linux -Wl,-rpath,/home/wmt/wasm-lib-jni/lib/linux ../lib/linux/wasm.a

However, note that -Bsymbolic will affect the non-Go code as well, and you must make sure that those effects will work for your case.
I'm going to close this because you're trying to do something that is not supported and there is no reasonable chance that it will be supported in the future.  Sorry.


Changed search to use the url

This should solve most of #261 (just not the update search with websocket part).

As it was with websockets search bar was not a good user experience. Red coloring for search failure was unclear and didnt tell the user what was happening. Also didnt put the user on a page where they could simply refresh to search again, or get the eventual websocket update to happen when the result becomes valid.

@chappjc I can't seem to see how it breaks the websocket, the websocket stays connected when you search and get the error page and also when you search and it redirects you to the desired page. Can you get an instance of when the websocket breaks?

I'm rebuilding my db to try this again, so I can't give an exact example for an hour or so, but what I am seeing is from a user perspective the websocket is always connected, but the websocket hub looses the connection and has to establish a new one.  What I was seeing yesterday in the dcrdata log with this change was that a search would cause the original websocket connection to be abandoned.
Where the action=/search part of the search form is used, the browser opens that page with the query.  This is a regular page load vs. the way turbolinks loads the pages with XHR (javascript).  When you initial switched the search to send on the websocket, that allowed the websocket connection to be reused.
I'll check this out again in the dcrdata terminal, and in the browser's developer tools (the Network tab in particular).

Oh, I see, I'll look into making it use xhr.

Since it's not a UX issue, but rather an architectural issue, I suppose the PR is solid as is.  I just wanted to identify a more elegant way to handle the websocket connections than having the client drop and server eventually realize it after a ping times out.
When this might be helpful is when the WebsocketHub needs to send client-specific messages, as is likely to be the case for the address page. (Say we implement subscription to address/whatever notifications). Right now it's broadcast only.  I suppose some other more persistent identifier like cookies could be involved there however.

I added something to make it use XHR (via turbolinks) but the "Connected" status text disappears if the search fails and if the search works the URL still shows /search?search=x and I have no idea how to fix either of those. If @gozart1 could chip it with some info on that, I'll appreciate it.

Function looks good. After merge it would be good for @gozart1 to do a face lift of the error pages. We probably do not even need to call it an ERROR like something went wrong, but could be more gentle about it with something along the lines of "Sorry, that transactions could not be found." As it is a normal expected occurrence when using the tool. I have opened https://github.com/dcrdata/dcrdata/issues/307 for tracking autorefresh when search items become available.

@RogueElement did you see this: https://github.com/turbolinks/turbolinks/blob/master/README.md#following-redirects

Since Search uses redirects in some cases now, it might be needed to set that location header.  Not sure about why the connected text disappears

Added lines to set the location header, now the url changes to the appropriate one (the connected text is back too but I have no idea why).

Good stuff.  Looks about ready to go.  Will test and merge ASAP


Create an Empty SAPUI5 Project, problem to create SAPUI5 freestyle from template with Northwind OData Service

Tutorial: https://developers.sap.com/tutorials/sapui5-101-create-project.html
Hello,
I have followed the guide until the step 3, section 8, and at the end of the generator, BAS returns an error ans the application is not generated
here's the error:
{"message":"fiori-project:app generator failed - env.on(error) Failed to consume services due to: undefined\nAbort ui5-simple creation.","stack":"Error: env.on(error) Failed to consume services due to: undefined\nAbort ui5-simple creation.\n at Ui5SimpleGenerator._abortGenerator (/extbin/generators/ui5-generators/generator-ui5-simple/generators/app/index.js:205:24)\n at /extbin/generators/ui5-generators/generator-ui5-simple/generators/app/index.js:126:26\n at processTicksAndRejections (internal/process/task_queues.js:97:5)"}
If I create an application without the OData, there are no issues, so I guess the Destination I created from the previous tutorial has a problem.
https://developers.sap.com/tutorials/cp-cf-create-destination.html#765bac60-947f-4be7-910c-7116197946cf
I have the right response when testing the destination alone, see the result screenshot.
So I don't know what is wrong with it, there's just one thing that is not clear, the guide mentions that
the parameter HTML5.DynamicDestination needs to be set manually, how can that be done?

Regards
Denis

Hi @DenisGalandSAP ,
For the additional parameter: Click the New Property button, and instead of selecting the property from the drop-down, type it manually.
Regards - Raz

Hi @DenisGalandSAP ,
For the additional parameter: Click the New Property button, and instead of selecting the property from the drop-down, type it manually.
Regards - Raz

oh yes, i didn't paid attention to the cursor, it was a combobox and not a select, i added the parameter and now the template can be generated.
Thank you for the hint, the ticket can be closed 👍

oh yes, i didn't paid attention to the cursor, it was a combobox and not a select, i added the parameter and now the template can be generated.
Thank you for the hint, the ticket can be closed 👍

oh yes, i didn't paid attention to the cursor, it was a combobox and not a select, i added the parameter and now the template can be generated.
Thank you for the hint, the ticket can be closed 👍

oh yes, i didn't paid attention to the cursor, it was a combobox and not a select, i added the parameter and now the template can be generated.
Thank you for the hint, the ticket can be closed 👍

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Implement it along with angular js

I need to find solution to implement it along with angularjs

This library works fine along with angular. You just need to wrap the lib for angular specific code.


update to 1.22.3

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Minor fixes for whiteboard-ps

Split SYNTAX entries for Get-Whiteboard. Added short descriptions for the two examples.
Specified syntax highlighting mode in code sections.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/MicrosoftDocs/office-docs-powershell, working directory: W:\g3hq-s, target branch: master, target branch commit: 335bd044e636ef0c5734c152fff0a894d96395ea, source branch: af7dddfb857342db62677c58ada1a4eb/whiteboard-ps-minor-fixes, source branch commit: .

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

:x: Validation status: errors

File
Status
Preview URL
Details

whiteboard/whiteboard-ps/whiteboard/Get-Whiteboard.md
:x:Error

Details

:x:Error

Details

whiteboard/whiteboard-ps/whiteboard/Get-Whiteboard.md

[Error] Unable to load file: whiteboard-ps/whiteboard/Get-Whiteboard.md via processor: AzureCliDocumentProcessor: Object reference not set to an instance of an object.

[Error] System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.GetInnerContexts(DocumentBuildParameters parameters, IEnumerable1 processors, TemplateProcessor templateProcessor, IHostServiceCreator creator) at Microsoft.DocAsCode.Build.Engine.SingleDocumentBuilder.BuildCore(DocumentBuildParameters parameters) at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.BuildCore(DocumentBuildParameters parameter, IMarkdownServiceProvider markdownServiceProvider, BuildInfo currentBuildInfo, BuildInfo lastBuildInfo) at Microsoft.DocAsCode.Build.Engine.DocumentBuilder.Build(IList1 parameters, String outputDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument(BuildJsonConfig config, TemplateManager templateManager, String baseDirectory, String outputDirectory, String pluginDirectory, String templateDirectory)
at Microsoft.DocAsCode.SubCommands.DocumentBuilderWrapper.BuildDocument()

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


initialize the extraInfo hashmap lazily

This makes a meaningful improvement in anyCompilationUnitToCeylon performance when extraInfo is not being used: 851ms vs 1050ms in one test.

Have you tested whether this has any significant impact on memory requirements? It’s an extra field per node… (though I guess so is data, and I just added that one :D )

Well, 1) fields are cheap, 2) it keeps what's probably many objects from being allocated when you consider the complexity of HashMap and the number of interfaces involved.

Good point, I forgot that “extra field costs memory” doesn’t really apply very well for non-primitive fields.

Should I rebase this on 1.2.0 before merging?

I forgot about that. It's now on 1.2.0.

Great, thanks!


No Windows pre built binaries

No pre-built binaries for Windows. Please add them. This would help in gaining users. I appreciate that it can be built on Windows, but this is an unnecessary speed bump. Thanks.

on windows make sure to invoke
cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=release -DCMAKE_EXPORT_COMPILE_COMMANDS=YES -DCMAKE_GENERATOR_PLATFORM=x64
I was following https://github.com/cquery-project/cquery/wiki/Building-cquery but after unsuccessful build I read that I have to add -DCMAKE_GENERATOR_PLATFORM=x64 ... because of that I lost some time

More than just a small speed bump, getting a compatible cmake is no small feat either.


Issue #3531 fixed

BUGS:
issue: #3531
For things other than bugs:

Describe that thing in very short line, word limit is 200.
Otherwise use issue #id if the issue was filed as feature request.

@vishal-webkul
Its not working properly if total is float like 329,45$US

@bhanu-webkul
Above mentioned issue solved please let me know if any other remains

Not working - getting exception while using filter total column and applying other the equals to condition

@bhanu-webkul
now fixed

Now working


Would it make sense to make GenerateParserTask and GenerateLexerTask outputs cacheable?

Is your feature request related to a problem? Please describe.
In --info mode, Gradle reports that both tasks have caching disabled :/
Describe the solution you'd like
Just mark the tasks as @CacheableTask?
Describe alternatives you've considered
Currently I'm doing sth like:
task generateMacheteParser(type: GenerateParserTask) {
  outputs.cacheIf { true }

  // ...
}

task generateMacheteLexer(type: GenerateLexerTask) {
  outputs.cacheIf { true }

  dependsOn generateMacheteParser

  // ...
}

Also! there's a caveat around GenerateParserTask targetRoot and GenerateLexerTask targetDir... the two folders should not be overlapping, or Gradle won't attempt to cache the outputs of either task :/ This isn't particularly obvious (and only gets reported on --info level in Gradle)... it would be good to mention in the docs that the generated Java sources of the tasks shouldn't go to a single directory, but instead two separate ones, like this:

See https://github.com/VirtusLab/git-machete-intellij-plugin/pull/946/files#diff-18fccbc3a3c0c8aff70a53e41afdd0cbd8b86cca800071243407fe2edf620400

I'm not sure the complexity to track all involved variables (including specific properties in .bnf/.flex files) is worth the efforts or even doable.
YMMV I enable these tasks only when actively working on lexer/parser with manual temporary dependsOn in Gradle. Why do you want to run them all the time?

YMMV I enable these tasks only when actively working on lexer/parser

Oh! so do I understand correctly that you keep the parser/lexer-generated code checked in to git and not git-ignored? 🤔

yes. I see no point in regenerating it from scratch all the time. Apart from that, not all setups are fully supported by the Gradle plugin so I have to generate Parser manually.

Okay, I'll reconsider my setup then... feel free to close the issue


Listeners don't work with functional components

Version
1.0.0-beta.25
Reproduction link
https://codesandbox.io/s/r7rn4q2qo
Steps to reproduce

Run test
Open sandbox console

What is expected?
Test passed
What is actually happening?
[Vue warn]: Invalid handler for event "click": got undefined.

How do I can pass them on?

It works.
import { shallowMount } from "@vue/test-utils";
import BaseButton from "../BaseButton.vue";

describe("BaseButtonIcon", () => {
  it("calls item-click when button is clicked", () => {
    const stub = jest.fn();
    const wrapper = shallowMount(BaseButton, {
      context: {
        props: {
          text: "click"
        },
        on: {
          'item-click': stub,
        },
      },
    });
    const button = wrapper.find("button.base-button-inc");

    button.trigger("click");

    expect(stub).toHaveBeenCalled();
  });
});


Bitmap Lockbits and Unlockbits

I just found out this project after few searches on google.
Readme on V3 Branch of this repo really suit on my needs.
I need an Image library for my game that compatible with NETCore (in other words: without System.Drawing.dll)
My game require to load general image formats (at least png and bmp) into OpenGL texture.
following code used to upload image data to OpenGL texture:
	// Lock Bits Bitmap Data
	BitmapData bmpdata = bmp.LockBits(
		new Rectangle(0, 0, bmp.Width, bmp.Height),
		ImageLockMode.ReadOnly,
		System.Drawing.Imaging.PixelFormat.Format32bppArgb);

        // Upload Texture Image
        GL.TexImage2D(TextureTarget.Texture2D, 0,
				PixelInternalFormat.Rgba,
				bmp.Width, bmp.Height, 0,
				OpenGL.PixelFormat.Bgra,
				PixelType.UnsignedByte,
				bmpdata.Scan0);

	// Pixels copied, Unlock it
	bmp.UnlockBits(bmpdata);

So my questions

Is this library implemented LockBits/UnlockBits like the code shown above?
Can library handling decode process from PNG to Bitmap? When I load a png image (using new Bitmap(Image.FromFile("....");) and uses code above to upload the image data to OpenGL texture, the transparency is still there, is ImageProcessor have same behavior?

Thanks in advance!

Hi Sirus,
No this version does not use LockBits/UnlockBits.
ImageProcessor images once loaded are format independent, storing a float array of pixels in the order rgba. Transparency is only lost when re-encoding the image.  You should be able to pass it around easily enough.
Here's an example of what you would like to do coded by @mellinoe at Microsoft.
https://github.com/mellinoe/BoxGame/blob/4b5ac397655aa16979fb2087c7fc5562e9ea08c4/src/EngineCore/Graphics/OpenGL/TextureBuffer.cs#L39

Thanks for clarification!
I don't know there is Pixels property there!
Keep up your good work!

No worries. Will try my best! :smile:

I'm sorry for reopening the issue, but it seems i cant compile the project
I'm using DNX 5.0 and got an error (the error seems occur when restoring dnx package)
Could not load file or assembly 'Microsoft.DNX.PackageManager' or one of its dependencies. The system cannot find the file specified. File name: 'Microsoft.DNX.PackageManager'
I'm not really familiar with CoreCLR actually but any ideas?

Looking at this similar issue  and solution on StackOverflow you are probably not using the latest tooling.
https://github.com/aspnet/dnx/issues/2426
http://stackoverflow.com/a/32357737
In answer to your next question you would have to do extra work to support old frameworks. System.Numerics.Vectors for example require DotNet5 to run.
https://www.nuget.org/packages/System.Numerics.Vectors
Personally I'm finding the support matrix increasingly difficult to understand. There's a lot of discussion here but I wasn't much wiser after reading it all. https://github.com/dotnet/corefx/issues/973

Thanks for your help!
I able to compile it now
hmm I see, easiest way to do this is clone the corefx and took their System.Numerics sources.
Or just use .NET Core haha
Thanks again!


WIP: Telldus Live: Implement support for covers and dimmable lights

Description:
Implement missing support for covers and dimmable lights
Related issue (if applicable): fixes #
https://community.home-assistant.io/t/trying-to-expand-tellduslive-support/3369/16
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Make sure to rename when done so we can see that it's ready for review 👍

I think it is ready for review now, @balloob (not sure about the missing check from coveralls)
While the original plan only was to add support for dimmable lights, I added some other changes as well and did some reworking of the component.
The only blocking method is in TelldusLiveClient.request - does it have to be converted to async as well?

This looks great 🐬


npm start throw Data path ".builders['app-shell']" should have required property 'class'

Issue description
Current behavior: Not able to start the project.

Expected behavior: Project started

Steps to reproduce: npm start

Other information:
npm, node, OS, Browser
Schema validation failed with the following errors:
  Data path ".builders['app-shell']" should have required property 'class'.
Error: Schema validation failed with the following errors:
  Data path ".builders['app-shell']" should have required property 'class'.

Node, npm: `v10.15.0` and `6.9.0`
OS: macOS (Mojave)

Exact same problem when trying to start an Angular project using npm start. My specs:
Node v10.15.1
NPM v6.7.0
Windows 10

Hey maybe you can try this fix, hope this helps :)
https://stackoverflow.com/questions/56393158/errors-data-path-buildersapp-shell-should-have-required-property-class

Thanks! I think I had a package conflict, which is one of the suggested areas to check which fixed my problem.

worked for me thanks! package conflict

How exactly did you resolve this guys?

Hey @bretonio, I've deleted the package-lock.json and then I ran npm install.
After that, everything looks fine, probably in my case It was update conflict
I hope it's help you

Easy, thanks. And yes! Good!

Angular CLI: 7.3.7
Node: 11.5.0
OS: win32 x64
Angular:
...
Package                      Version
@angular-devkit/architect    0.13.7
@angular-devkit/core         7.3.7
@angular-devkit/schematics   7.3.7
@schematics/angular          7.3.7
@schematics/update           0.13.7
rxjs                         6.3.3
typescript                   3.2.4
Showing error
Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
Error: Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.

Angular CLI: 7.3.7
Node: 11.5.0
OS: win32 x64
Angular:
...
Package Version
@angular-devkit/architect 0.13.7
@angular-devkit/core 7.3.7
@angular-devkit/schematics 7.3.7
@schematics/angular 7.3.7
@schematics/update 0.13.7
rxjs 6.3.3
typescript 3.2.4
Showing error
Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
Error: Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.

Changing from "@angular-devkit/build-angular": "^0.802.1"
to  "@angular-devkit/build-angular": "^0.13.9",
solved my problem

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
solved my problem, thank you

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
solved my problem, thank you

Works like a charm!

Angular CLI: 7.3.3
Node: 8.15.0
OS: linux x64
Angular: 8.2.2
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router
Package                           Version
@angular-devkit/architect         0.802.2
@angular-devkit/build-angular     0.802.2
@angular-devkit/build-optimizer   0.802.2
@angular-devkit/build-webpack     0.802.2
@angular-devkit/core              8.2.2
@angular-devkit/schematics        8.2.2
@angular/cdk                      8.1.3
@ngtools/webpack                  8.2.2
@schematics/angular               8.2.2
@schematics/update                0.802.2
rxjs                              6.5.2
typescript                        3.5.3
webpack                           4.38.0
----------------------- error --------------------------
Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
Error: Schema validation failed with the following errors:
Data path ".builders['app-shell']" should have required property 'class'.
at MergeMapSubscriber._registry.compile.pipe.operators_1.concatMap.validatorResult [as project] (/usr/local/lib/node_modules/@angular/cli/node_modules/@angular-devkit/core/src/workspace/workspace.js:215:42)
at MergeMapSubscriber._tryNext (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/mergeMap.js:69:27)
at MergeMapSubscriber._next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/mergeMap.js:59:18)
at MergeMapSubscriber.Subscriber.next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/Subscriber.js:67:18)
at MergeMapSubscriber.notifyNext (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/mergeMap.js:92:26)
at InnerSubscriber._next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/InnerSubscriber.js:28:21)
at InnerSubscriber.Subscriber.next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/Subscriber.js:67:18)
at MapSubscriber._next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/map.js:55:26)
at MapSubscriber.Subscriber.next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/Subscriber.js:67:18)
at SwitchMapSubscriber.notifyNext (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/switchMap.js:86:26)
at InnerSubscriber._next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/InnerSubscriber.js:28:21)
at InnerSubscriber.Subscriber.next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/Subscriber.js:67:18)
at /usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/util/subscribeTo.js:17:28
at Object.subscribeToResult (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/util/subscribeToResult.js:10:45)
at SwitchMapSubscriber._innerSub (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/switchMap.js:65:54)
at SwitchMapSubscriber._next (/usr/local/lib/node_modules/@angular/cli/node_modules/rxjs/internal/operators/switchMap.js:55:14)

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
.
.
.
The application works fine for me but a potential security vulnerability warning is found with high priority.

Hey @bretonio, I've deleted the package-lock.json and then I ran npm install.
After that, everything looks fine, probably in my case It was update conflict
I hope it helps you

Its work for me. Thanks alot

This worked for me:

Opened package.json
Changed "@angular-devkit/build-angular": "^0.800.0" to  "@angular-devkit/build-angular": "^0.10.0"
Run npm install
Run ng serve

This worked for me:

Opened package.json
Changed "@angular-devkit/build-angular": "^0.800.0" to
"@angular-devkit/build-angular": "^0.10.0"
Run npm install
Run ng serve

thanks! its work

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
.
.
.
The application works fine for me but a potential security vulnerability warning is found with high priority.

This was also my issue, I ran npm audit and found the vulnerability. I ran npm audit fix and then my app would not run because It reverted to "^0.803.6".

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
solved my problem, thank you

This really worked for me... Thanks

Changing from "@angular-devkit/build-angular": "^0.802.1"
to "@angular-devkit/build-angular": "^0.13.9",
.
.
.
The application works fine for me but a potential security vulnerability warning is found with high priority.

This was also my issue, I ran npm audit and found the vulnerability. I ran npm audit fix and then my app would not run because It reverted to "^0.803.6".
EDIT: Setting it to "^0.13.9" solved the vulnerability issue ! :)

Same, problems started after "npm audit fix". All is good now. I still have 3 low severity vulnerabilities - but will live with that for now.

jkklk

I mean what is this package and why are we reverting it back to a earlier version? The audit-fix does not revert to earlier version, it actually updates it. Latest version as of today is 0.803.22  https://www.npmjs.com/package/@angular-devkit/build-angular

No need to perform any changes in package-lock.json or other files just rum command npm install --save-dev @angular/cli@latest. See attached image for final result


Issue942 bettergetdata

Main changes:

interpolate_along_los()

Issues:
Fixes, in devel, issue #942

Hello @dvezinet! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tofu/data/_class02_sample.py:

Line 372:12: E225 missing whitespace around operator
Line 597:25: W292 no newline at end of file

In the file tofu/data/_class02_tangency_radius.py:

Line 219:5: E303 too many blank lines (2)
Line 451:27: W292 no newline at end of file

In the file tofu/data/_class08_interpolate_along_los.py:

Line 639:30: E225 missing whitespace around operator
Line 640:11: E271 multiple spaces after keyword
Line 673:5: E303 too many blank lines (2)
Line 817:18: W292 no newline at end of file

In the file tofu/data/_class8_los_data.py:

Line 254:16: W292 no newline at end of file


Hyphenated compound words are wrongly tokenized as negated

The tokenizer wrongly counts subterms in compound words as negated
64-bit is parsed as 64 -bit and therefore excludes any hit of bit
Surfaced by @discordier in this pr comment

Suggested fix at #115


Move flaky test check to an eventually

This also changes the comparison to use native Gomega assertions.
Relates:
https://issues.redhat.com/browse/ACM-4873

/cherry-pick release-2.9

/cherry-pick release-2.8


Tab navigation skips most background layers

URL
No response
How to reproduce the issue?
Use (keyboard TAB key, on the left side of your keyboard) to navigate around the Background menu.
At first TAB press we see:

The next jumps to here:

And the next:

So all the other background layer choices get skipped over, inaccessible via Tab navigation!
Further down the background layer menu every checkbox is accessible.
It is only the section I have mentioned above that has the bug.
Screenshot(s) or anything else?
No response
Which deployed environments do you see the issue in?
Released version at openstreetmap.org/edit
What version numbers does this issue effect?
No response
Which browsers are you seeing this problem on?
Chrome

it's a bit unintuitive, but it seems like it's working as intended^1 - once you've focussed on the list of layers using ↹ Tab, you then need to use the ⬆ Up and ⬇ Down keys.
I agree though, it would be nicer if you could ↹ Tab through the list...

I see. Different than checkboxes.


Message size upper bounds checked in unit test

Closes #3557.
Adds a test to find the biggest possible message.

Closing this in favor of a coming new PR with the chainspec feature already added.


Is it possible to mark a C++ type as Trivial?

I have a situation where there's a fairly complex C++ enum (below) I wish to be able to return to Rust as Opaque, but by value. IOW, I'd like to mark the type as cxx::kind::Trivial, but don't want a) to duplicate the definition as a shared type, or b) wrap it in a UniquePtr. Is this at all possible with the current cxx?
enum class Type

enum class Type
{
    None = 0,
    Unsigned8 = unsigned(BaseType::Unsigned) | 1,
    Signed8 = unsigned(BaseType::Signed) | 1,
    Unsigned16 = unsigned(BaseType::Unsigned) | 2,
    Signed16 = unsigned(BaseType::Signed) | 2,
    Unsigned32 = unsigned(BaseType::Unsigned) | 4,
    Signed32 = unsigned(BaseType::Signed) | 4,
    Unsigned64 = unsigned(BaseType::Unsigned) | 8,
    Signed64 = unsigned(BaseType::Signed) | 8,
    Float = unsigned(BaseType::Floating) | 4,
    Double = unsigned(BaseType::Floating) | 8
};

Yep, something like this should work:
#[cxx::bridge]
mod ffi {
    unsafe extern "C++" {
        include!("example/include/metasim.h");

        type Type = crate::Type;
        fn metasim() -> Type;
    }
}

#[derive(Debug)]
#[repr(transparent)]
pub struct Type(std::ffi::c_int);

unsafe impl cxx::ExternType for Type {
    type Kind = cxx::kind::Trivial;
    type Id = cxx::type_id!("Type");
}

fn main() {
    println!("{:?}", ffi::metasim());
}


👽 Add size parameter for lfsFiles when committing on the hub

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@Wauplin sorry I don't understand by the coverage decreased so much :exploding_head:

Thanks @coyotte508 @Wauplin!


Bug: Outdated crates.io index

When trying to build my docker image with cargo-chef, I get this error:
[3/4] STEP 3/6: RUN cargo chef cook --release --recipe-path recipe.json
    Updating crates.io index
error: failed to select a version for the requirement `serde_with = "^2.3.1"`
candidate versions found which didn't match: 1.14.0, 1.13.0, 1.12.1, ...
location searched: crates.io index

To me it seems that the crates.io index that cargo-chef is using is outdated, and somehow the "update" step is not doing what it's supposed to do.

The issue was caused by a mismatch of the Rust version being used. Just updated my docker file to match it.


Refactor inventory containers. Separate client and server container concept.

This PR refactors the inventory containers, it divides the container in two different layers. A server layer which will handle all our custom/vanilla click movements. And the client layer, which will "guess" what the client will update so we can revert these changes if they don't match our own ones. The client container concept is also being expanded that you can bind any server slot to a specific client slot, the indexes don't have to match (slots can be disabled and also put an icon in it).
This is still WIP, needs cleaning up in some places, some things still need to be moved and everything needs to be properly tested.
Features:

Refactor the shift clicking handling, the OpenableInventory defines now how shift clicking is handled, and not by the slots like it was before. It allows a greater customizability.
Separate server and client containers for complete client container control.
Added all the vanilla client container implementations.
Implemented furnace shift clicking.

@Meronat Shift clicking in furnaces is now a thing. ;)

This became a pretty big undertaking but from the looks of it will definitely be worthwhile.

@Meronat Could you give this another try? It should be working now.

@Cybermaxke Just saw this now. I'll give it a test :)

It's working now for furnaces it seems, nice job. The only problem I have found(not necessarily related) is that the progress bar for fuel does not retract(goes down/runs out?) as it runs out.
It also seems to work between my hotbar and main inventory, as well as with single chests but not double chests.
With double chests my items disappear then reappear in my inventory when I close the container.

Also I'm not sure if this can be helped, but when I shift click an item to go into the ingredient slot of the furnace, it seems to go to the part of the player inventory it was not in very quickly then go to the ingredient slot.
So hotbar -> main -> ingredient slot or main -> hotbar -> ingredient slot
This does not happen with the fuel

@Meronat What was the ingrediënt in the last issue? It was probably original client behavior.

@Cybermaxke It was a half stack or so of iron ingots for the steel recipe

@Meronat You can retry this PR, I fixed the issues you had. ;)

All of my issues seem to be fixed, nice job. I'll give it a more complete review soon.


fix: SC not returning tracks

Changes
Write here about the changes you've made
Added an extra condition so that only SoundCloud tracks will be returned on request.
Why
Write here why you think this should be merged
It fixes an improntant issue related to SoundCloud information retrieval.
Checkmarks

[ X ] The modified endpoints have been tested.
[ X ] Used the same indentation as the rest of the project.
[ X ] Still compatible with LavaLink clients.

Additional information
If you have any additional information, write it here

no


iOS 11 breaks slacktextviewcontroller

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Description
Cannot press the textview due to there being an invisible view over it.
Reproducible in:

[x] This is reproducible in the sample project.
SlackTextViewController version:
iOS version(s): 11
Device(s): All  iOS 11 compatible devices.

Steps to reproduce:

Install iOS 11
Run SlackTextViewController Demo
Try clicking the textview

Expected result:
Expected me to be able to send messages etc
Actual result:
Couldn't open textview
Attachments:

Confirmed 😮

@hipwelljo however it works on apps built pre iOS 11 which is wierd

It seems that this bug is caused by a new subview which is added only for iOS 11
On iOS 10, the subviews array for textInputbar looks like this:
(lldb) po self.textInputbar.subviews <__NSArrayM 0x170c5cbf0>( <_UIBarBackground: 0x15fdbc650; frame = (0 0; 375 48); userInteractionEnabled = NO; layer = <CALayer: 0x1746370a0>>, <UIView: 0x161a03f50; frame = (0 0; 375 0); clipsToBounds = YES; hidden = YES; layer = <CALayer: 0x174636340>>, <UIButton: 0x161a04fb0; frame = (0 48; 0 0); opaque = NO; layer = <CALayer: 0x174635e20>>, <UIButton: 0x161a05530; frame = (375 11; 0 30); opaque = NO; layer = <CALayer: 0x174635d40>>, <SLKTextView: 0x16085f400; baseClass = UITextView; frame = (8 5; 359 38); text = ''; clipsToBounds = YES; gestureRecognizers = <NSArray: 0x174851640>; layer = <CALayer: 0x174636ec0>; contentOffset: {0, 0}; contentSize: {359, 38}>, <UILabel: 0x161a05ab0; frame = (367 5; 0 0); userInteractionEnabled = NO; layer = <_UILabelLayer: 0x174482760>> )
On iOS 11, the subviews array for textInputbar has a new element, which is right at the top of view hierarchy.
(lldb) po self.textInputbar.subviews <__NSArrayM 0x60800287a9c0>( <_UIBarBackground: 0x7fabd172c630; frame = (0 0; 375 48); userInteractionEnabled = NO; layer = <CALayer: 0x608000a2fc20>>, <UIView: 0x7fabd17224e0; frame = (0 0; 375 0); clipsToBounds = YES; hidden = YES; layer = <CALayer: 0x608000e25f20>>, <UIButton: 0x7fabd1720590; frame = (0 48; 0 0); opaque = NO; layer = <CALayer: 0x608000e35ea0>>, <UIButton: 0x7fabd150e3b0; frame = (375 11; 0 30); opaque = NO; layer = <CALayer: 0x6000012323a0>>, <SLKTextView: 0x7fabcf864000; baseClass = UITextView; frame = (8 5; 359 38); text = ''; clipsToBounds = YES; gestureRecognizers = <NSArray: 0x608002662580>; layer = <CALayer: 0x608000c32640>; contentOffset: {0, 0}; contentSize: {359, 38}; adjustedContentInset: {0, 0, 0, 0}>, <UILabel: 0x7fabd153a1e0; frame = (367 5; 0 0); userInteractionEnabled = NO; layer = <_UILabelLayer: 0x60000068d570>>, **<_UIToolbarContentView: 0x7fabd17276d0; frame = (0 0; 375 48); layer = <CALayer: 0x608000a291e0>>** )
Notice the _UIToolbarContentView subview, right at the top!
A quick temporary workaround is to do something like this in your controller:
[self.textInputbar sendSubviewToBack:[self.textInputbar.subviews lastObject]];
Or:
[self.textInputbar bringSubviewToFront:self.textInputbar.rightButton]; [self.textInputbar bringSubviewToFront:self.textInputbar.textView];
Hope this is good enough until we have a proper fix

@claudiumatei - thanks for the workaround. If I add this code, I can now type in the input box (in the simulator using a keyboard) but the keyboard does not appear on on a device when you click on the text input box.  Do you have the same problem or know of a workaround to make the keyboard appear?

The workaround committed by paulcarpenter is working nicely for me.

This workaround is not going to play well with the newly announced iPhone X. You're not allowed to put any interaction elements in that area due to the rounded screen.
I think our days of putting a text field right at the bottom of the screen are over. You'll notice that Apple doesn't even do this in the Messages app anymore, they have that annoying iMessage apps bar.

It seems that inserting or deleting rows causes the entire tableView to stop responding as something is wrong with layout of the views.

@dzenbot Haven't tried it, but wanted to note a UIView subclass would be a solid background instead of the translucent blurred toolbar. (Though it seemed the toolbar wasn't actually translucent anyways, I didn't see elements through it in my app). Just wanted to be sure that's what is desired now.

I tested #624 and it works for our application - thanks

I programmatically called  becomeFirstResponder() on the textfield. Keyboard appeared, but the textfield was still on the bottom 😾

I am using cocoa pods. What do I need to do to get this update?

To solve the problem for iOS11 (compatible with lower versions) you only need
to make layoutSubview right after UIToolBar was added as a subview to UI hierarchy.
In this case _UIToolbarContentView lowers to the first subview of UIToolBar, and you can
add all your subviews higher as before.
For example in ObjC,
    UIToolbar *toolbar = [UIToolbar new];
    [self addSubview: toolbar];
    [toolbar layoutIfNeeded];

    <here one can add all subviews needed>

Hi Guys,
I am facing different issue. Contents drawn on the CGContext of the UIImageView got disappeared. I have attached the video which demonstrates the issue i was facing in my side. I am getting this issue only in iOS 11 iPad. I could not reproduce the issue in iPod/iPhone with iOS 11. Can you please let me know if you have any idea on this?
Video: https://www.dropbox.com/s/7nhnaqdhh8ha7u1/Untitled.zip?dl=0


[RemoteDebugging] To fix EnableRemoteDebugging bug

To fix EnableRemoteDebugging bug.
When EnableRemoteDebugging was called with same port number,
remote_debugging_server was stopped, because it was deleted by reset()
It does not seem to intended code.
If already remote_debugging_server was created with same port,
It should just return

Abandon this commit,
The reason of delete the remote_debugging_server was to prevent port already in use error


DelimitedFiles should be added to REQUIRE

readdlm function seems to be used in TableSchema internals (not only in example)
https://github.com/frictionlessdata/TableSchema.jl/blob/72f599c7505adcf89bf21f2c767527953320679a/src/table.jl#L15
so it should be added to REQUIRE

my mistake... that's not necessary. Sorry for noise

it's a module inside Julia base https://github.com/JuliaLang/julia/blob/master/stdlib/DelimitedFiles/src/DelimitedFiles.jl


Abbreviations: Atoms protrude beyond the expanded view for Functional Groups and Salts and Solvents

Steps to Reproduce

Launch Ketcher
Press 'Custom Templates' button and click on 'Functional Groups' tab
Select for example 'CO2tBu' and add to canvas
Press 'Custom Templates' button and click on 'Salts and Solvents' tab
Select for example 'methane sulphonic acid' and add to canvas
Expand the abbreviations and look at appeared images

Actual behavior
Expand abbreviation for Functional Groups and Salts and Solvents works incorrectly - atoms protrude beyond the expanded view
Expected behavior
Expand abbreviation for Functional Groups and Salts and Solvents works correctly - atoms not protrude beyond the expanded view. The whole structure fits into an expand square.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 108.0.5359.125 (Official Build) (64-bit)

Ketcher version
Version 2.7.0-rc.8
Additional context
Add any other context about the problem here.

Issue is reproduced in v2.7.0-rc.9, moving to v2.9.0-rc.1


Class 'Evenement\EventEmitter' not found.

Where this class should be? I looked at your bundle, there is nothing with this name.

For the reference only: I guess you've found https://github.com/reactphp/socket#install by now? :)


chore: update dependencies

Bump all dependencies.
This also includes the latest @commercelayer/react-components and @commercelayer/sdk

@gciotola already deployed on staging?

@malessani  just deployed


💃 Use pre-built containers

Now that containers are being built daily in GitHub Actions from
the container[0] repository, we can use those containers. That will
speed up the GitHub Actions for osbuild and reduce errors from
unreleased Fedora versions.
[0] https://github.com/osbuild/containers
Signed-off-by: Major Hayden<EMAIL_ADDRESS>

Very cool! Thanks a lot!
I am all fine with quay.io (yay, Red Hat!). Just FYI: You can also push to Github Packages. At least I thought reading that it supports pushing docker containers as package type. Just in case quay.io somehow gets discontinued.


Update home.html by change violin.JPG to violin.jpg

Update home.html by change violin.JPG to violin.jpg

Pls modify the code in branch "test-index.html-with-Live-Server"


fix: enr signature changes on restart even if enr content is the same

What was wrong?
I made a PR https://github.com/sigp/enr/pull/53 and Age Manning gave me the insight that the signature should always be the same in the client if the content is the same.
We had an issue before my increase seq pr and after that even though our enr could be the same, the signature would change which isn't good.
How was it fixed?
It was fixed by restructuring a bit of my code. I also cleaned it up a bit for how to compare them from suggestions from Age and Nick.
I also resolved a param name nitpik nick had in my last PR.

@njgheorghita I believe I resolved your concerns let me know what you think


add users, users-test, users-postgresql-simple

I've fixed the mtl issue in a new version of users-postgresql-simple.

Thanks!


Metadata tables don't show wap snapshots if there are no other snapshots

Steps to reproduce:

Create an empty table with wap enabled
Produce a new wap snapshot
Ensure the data is not yet accessible
Query the snapshots table

I would still expect to see the created snapshot.

Closing as it was addressed in #803.


[4.3.0] Improve configure forgerock document

Fixes https://github.com/wso2/docs-apim/issues/7799

Doc was revamped via [1]. Hence, closing this PR.

https://github.com/wso2/docs-apim/pull/8607


Hardly to show the comments in code diff

IntelliJ IDEA 2023.2.1 (Ultimate Edition)
macOS Ventura 13.4.1
In code diff

In editor

Oooh, that's gnarly.  I'll try to figure out what I borked on that and get it updated asap.

This should now be fixed by this commit: f92ef92c84941f90573cf3f77d08d12db03be9cf.  It usually takes ~24 hours to get approval, at which point it will become available to be updated.  If you want to update it manually before then, you can install it from here https://plugins.jetbrains.com/plugin/17938-zenburn/versions/stable/393485.  If this works, please let me know.  If I haven't heard back in the next day or two, I will close it.  Sorry for the inconvenience.  Thanks for letting me know!

Cool!  Thanks for checking it so quickly.  That makes my life a lot easier to keep organized!


Add the study id as a column in the mapping file

It will be useful for meta-analysis, in and outside Qiita

Thanks for adding this, Jose!

Hi guys, just to get a bit more info here. Currently when you do a meta-analysis in qiita this is added to the map files on the fly but I'm wondering if we want to add by default to the mapping file? Basically, add to the map file even if not use by meta-analysis.

I think it would be nice to have in the mapping file by default.

On Nov 9, 2016, at 11:39 AM, Antonio Gonzalez<EMAIL_ADDRESS>wrote:
Hi guys, just to get a bit more info here. Currently when you do a meta-analysis in qiita this is added to the map files on the fly but I'm wondering if we want to add by default to the mapping file? Basically, add to the map file even if not use by meta-analysis.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/biocore/qiita/issues/1866#issuecomment-259505767, or mute the thread https://github.com/notifications/unsubscribe-auth/AFa_ZrleVe76iAE91z87qTYEJJXpNAoPks5q8iFogaJpZM4I8EMw.

The previous decision on this was to add to the sample template.


Consider Streams instability

The Stream trait isn't stable like Future is, and hyper uses it in a couple places. Is it worth putting all usage behind some unstable feature flag?

impl Stream for Body: this exists for convenience so users can use stream combinators. Easily optional.
Body::wrap_stream: this exists to allow users to easily wrap stream combinators into an HttpBody. Easily optional.
hyper::Server::builder(impl Stream): this allows using a hyper server with a stream of incoming connections besides just TCP. This is pretty fundamental. An alternative could be to define an Accept trait that functions similarly. And then an optional AcceptStream<impl Stream> can exist unstably.

@seanmonstar I think the first two sound good. For the last one, I think the custom trait works well 👍.

This has been done.


Error in _find_unused_ops for operations with output size 0

Hi tfcoreml team,
I encountered a bug in _find_unused_ops in https://github.com/tf-coreml/tf-coreml/blob/master/tfcoreml/_tf_graph_transform.py#L107 . For operations with zero sized outputs (a corner-case, but relevant), this line fails. Suggested fix is to insert a check before:
if out1.size == 0 and out2.size == 0:
    is_skippable = True
    continue

Thank you for pointing that out. Do you want to put up a PR with a fix?
Otherwise I'll put up one soon. Thanks!

No problem! It would be great if you can put up the PR.
Thanks a lot!

@aseemw did you put up a PR regarding this issue?

I have created it now PR #350


Asp.net connection to and from aspnetcore

Hi folks,
Not sure if this is the correct place for this.
If its not please tell me.
I have a server running
<PackageReference Include="Microsoft.AspNetCore.SignalR" Version="1.0.0-preview1-final" />
And a client running
Microsoft.AspNet.SignalR.Client 2.2.3
Is it possible to connect the 2 together.
At the moment i'm getting 404s when i try. Both running on local host.
I found a stack overflow post (that i have now lost) that mentioned that they are no longer compatible.
Sorry if this is a bit of a dense question.

You can't mix and match versions of the server and client.

From the Alpha release blog post here

One of the consequences of this is that SignalR for ASP.NET Core is not compatible with previous versions of SignalR. This means that you cannot use the old server with the new clients or the old clients with the new server.

Closing this as the question has been answered. Feel free to comment further if you need clarification on something! We can always re-open the issue if there's something to fix here.

@anurse  @mikaelm12  thanks folks for the clarification.
I will just need to find another solution.


Don't concatenate method name and arguments in empty methods without method def parentheses

So we had some (unconventional) code in our repository that was incorrectly corrected by Style/EmptyMethod:
Before:
def some_method arg
end

After:
def some_methodarg; end

Even with Style/MethodDefParentheses enabled, the end result was the same (I suppose because of the order in which they are applied). This PR adds an extra space between the method_name and arguments if necessary:
def some_method arg; end

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

👍


Fix cosmos CI build failures caused by dev_setup.py changes

Simple change! From here on out will grep dev_setup.py through the repo on any change to the input strategy. This kind of thing shouldn't happen again.

Can one of the admins verify this patch?

@scbedd actually, are the Cosmos emulator tests running here? I don't seem to see them in the output above but maybe I am missing something?

@scbedd actually, are the Cosmos emulator tests running here? I don't seem to see them in the output above but maybe I am missing something?

Yep, needed to update the dev_setup.py call in two locations. Bad assumptions and half-attention.


Feedback for "Basic Platform Concepts"

https://help.sap.com/docs/btp/best-practices/basic-platform-concepts
Is the diagram still correct? A global account doesn't have a region according to my understaning (at least it cannot be chosen).
In addition, with hybrid global accounts it can also support two license types: subscription + CPEA/PAYG.

@annawenger20  yes this has been  updated and will be visible with the next release.


[Identity] Documentation improvements

Added a disclaimer to the VisualStudioCodeCredential class docstring to highlight its shortcomings. Users who glean information from the API docs might find it helpful.

Also slightly updated the EnvironmentCredential class docstring to include a missing envvar.
Minor updates to the README.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-identity


CSS Media Query Hierarchy

CSS group media query is not sorting the queries based on hierarchy (largest to lowest)

I assume that you've changed it to max-width, if that's the case you might have to sort the parameters accordingly.
@mixin break($point) {
  @if $point == desktop {
    @media (max-width: 425px) {
      @content;
    }
  } @else if $point == laptop {
    @media (max-width: 600px) {
      @content;
    }
  } @else if $point == tablet {
    @media (max-width: 768px) {
      @content;
    }
  } @else if $point == phablet {
    @media (max-width: 1024px) {
      @content;
    }
  } @else if $point == mobile {
    @media (max-width: 1366px) {
      @content;
    }
  }
}


GEOMESA-1544 Jupyter and Leaflet support

Add initial support for Jupyter and Leaflet

Signed-off-by: Anthony Fox<EMAIL_ADDRESS>

@anthonyccri @jnh5y looks like CQ check is failing - need to run ./build/calculate-cqs.sh and check in the resulting changes to build/cqs.tsv. I can open up CQs for the changes once they're identified.

@elahrvivaz Your wish is mah push:  The CQs diff is...
+org.apache.commons:commons-lang3       3.4     compile
I wonder if we can manage that to version 3.3.2 which is already in GeoMesa?

@jnh5y @anthonyccri Waiting on your guys to fix the geomesa versions in the README.md and open tickets for the TODOs, then I'll merge.


Add support for merge-sort

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-10451
Add kernel that merges two arrays according to a third vector of whether the item should come from the left array or the right array

A more generic form of this added in #2838


Unable to get comments data

I trying to integrate this module into my stapi project as per your documentation but i am getting {"data":null,"error":{"status":404,"name":"NotFoundError","message":"Not Found","details":{}}}  error when ever I am trying make rest api get call  for http://localhost:1337/comments/api::post.post:1
When I tried to  navigate in comments section of admin portal, I can see my console as,

with my admin panel as,

content of my package.json file is,
{
"name": "app-name",
"private": true,
"version": "0.1.0",
"description": "A Strapi application",
"scripts": {
"develop": "strapi develop",
"start": "strapi start",
"build": "strapi build",
"strapi": "strapi"
},
"dependencies": {
"@aws-sdk/client-s3": "^3.48.0",
"@strapi/plugin-i18n": "4.0.0",
"@strapi/plugin-users-permissions": "4.0.0",
"@strapi/provider-email-nodemailer": "^4.0.3",
"@strapi/provider-email-sendgrid": "^4.0.3",
"@strapi/provider-email-sendmail": "^4.0.0",
"@strapi/strapi": "4.0.0",
"@strapi/utils": "^4.0.0",
"aws-sdk": "^2.892.0",
"axios": "^0.24.0",
"bad-words": "^3.0.4",
"lodash": "^4.17.21",
"nodemailer": "^6.7.2",
"pg": "^8.7.1",
"sendmail": "^1.6.1",
"sqlite3": "5.0.2",
"strapi-plugin-comments": "^2.0.0-beta.3",
"strapi-provider-upload-aws-s3": "^3.6.8",
"strapi-provider-upload-aws-s3-advanced": "^4.0.3"
},
"author": {
"name": "A Strapi developer"
},
"strapi": {
"uuid": "unique id"
},
"engines": {
"node": ">=12.x.x <=16.x.x",
"npm": ">=6.0.0"
},
"license": "MIT"
}
installed node version is  15.14.0
npm version is  7.7.6
I have enabled all the setting for comments plugin for making a rest api call from setting tab> role for all users .
Content of my plugin.js is,
comments: {
enabled: true,
config: {
enableUsers: false,
},
badWords: false,
moderatorRoles: ["Public"],
approvalFlow: ["api::post.post"],
entryLabel: {
"*": ["Title", "title", "Name", "name", "Subject", "subject","id"],
"api::post.post": ["id"],
},
reportReasons: {
MY_CUSTOM_REASON: "MY_CUSTOM_REASON",
},
}
Can you please help me to fix this issue

@cyp3rius
We have upgraded with version rc.1.
In we have configured our entity on api::post.post we can access the same using http://localhost:1337/api/posts
Now we have tried to make a post a call for
http://localhost:1337/comments/api::post.post:1
with request body as
{
"author": {
"id": "11",
"name": "Joe Doe",
"email"<EMAIL_ADDRESS>"avatar": "sample url"
},
"content": "My sample response"
}
in the postman i am getting an error,
**{"data":null,"error":{"status":400,"name":"Error","message":"invalid JSON, only supports object and array"}}**

and in in the console I am getting error as,

Please help

Ok, so seems something with payload, how you are making the call? Which lib is used? Have you tried postman / any API test suite?
I would like to eliminate the data transform issues between API and client first.
Regards,
Matt

we are using postman. There is one thing that I have noted ...with graphql it is working pretty smoothly.  Not sure why it is failing with rest api

I've recognized what was wrong. Endpoints mentioned in the Readme.md were missing the api/ part so you've to use http://localhost:1337/api/comments/api::post.post:1 instead of http://localhost:1337/comments/api::post.post:1.
Documentation has been already fixed.

@cyp3rius Docs are still not fixed, as you can see in the image.

Will pick it up on Monday with some first commits


Fixes to ModernDialog.xaml

-Hide Minimize, Maximize and Restore(shows up when WindowState is set to Maximized) when ResizeMode is NoResize, same as a normal window does
-Added fix for when WindowState is Maximized and ResizeMode is NoResize, when both were set before you could start dragging the window and it would snap to the size of MinWidth and MinHeight.

@UpwindSpring01 the visual bug:

Style for window:
        <Setter Property="ResizeMode" Value="NoResize" />
        <Setter Property="SizeToContent" Value="WidthAndHeight" />
        <Setter Property="MinWidth" Value="300" />
        <Setter Property="MinHeight" Value="50" />

Thanks for the report

Good job!


Jest throws error when using Drawing Library

We are attempting to write Jest tests with a map component that is using the google maps api drawing library
<LoadScript googleMapsApiKey={api_key} libraries={['drawing']}>
              ..
</LoadScript>

<GoogleMap
              mapContainerStyle={containerStyle}
              zoom={20}
              center={center}
              options={{ streetViewControl: false, mapTypeControl: false }}
            >
              <Marker position={center} />
              <Polygon
                paths={parseBoundary(centreBoundary?.coordinates).map((pt) => ({ lat: pt[0], lng: pt[1] }))}
                options={storeOptions}
              />
              <Polygon
                paths={parseBoundary(centreBoundary?.coordinates).map((pt) => ({ lat: pt[0], lng: pt[1] }))}
                options={centreOptions}
              />
              <DrawingManager
                onPolygonComplete={onCentrePolygonComplete}
                options={{ drawingControl: false, drawingMode: 'polygon' as OverlayType }}
              />
            </GoogleMap>

but when the component is accessed in the jest test
test('Should show dialog when button is pressed', async () => {
    renderWithClient(<MapPage />);
    const btn = await screen.findByRole('button', { name: /Show Map/i });
    btn.click();
    expect(await screen.findAllByText(/Click on the map/)).toBeTruthy();
  });

it throws the following error:
Invariant Violation: Did you include prop libraries={['drawing']} in the URL? undefined

I agree, this library seems to be missing the entire DrawingManager in it's mocks?

A feature request to add the drawingLibrary was added as #521. Besides that I don't see anything actionable for us here.
As for the original bug-report, I can't really tell where that error-message is coming from, but it coming from the implementation of the <GoogleMap> component seems reasonable.  Unless that is the component you are developing and testing, I would advise to mock the entire component instead of trying to mock the Google Maps API and all it's interactions with the <GoogleMap> component.
The mocks provided here are really only useful if you are testing code that directly interacts with the Google Maps API, in which case there's no real alternative.


⚠ SkipTheTrailers has degraded performance

In fbbb44d, SkipTheTrailers ($STT) experienced degraded performance:

HTTP code: 500
Response time: 1082 ms

Resolved: SkipTheTrailers performance has improved in 55ca1f0.


Possible QuestaSim work-around?

I've tried to get around the recommended QuestaSim for co-simulation by instead using ModelSim 2019.1 and replacing the QuestSim path with my ModelSim 2019.1 path. I've strayed from the recommended QuestaSim installation as my academic institution currently does not have active licenses for QuestaSim. I've tried to make it work with the free 'alternative', but I seem to have run into an issue when running the Makefile in CEP/sims/cep_cosim.
It appears to be trying to invoke a vlog executable that does not exist in my current ModelSim installation directory (see screenshot attached.) My assumption is that this directory most likely exists in QuestaSim, but not in ModelSim. Would there be any possible workarounds for this and any future problems that I could run into by using ModelSim instead of QuestaSim? Thanks in advance.

From a quick search, Modelsim should support Verilog and SystemVerilog, but I am not sure if all of the verification capabilities of the CEP co-simulation environment is supported.
I would do some searching and ensure your PATH is setup correctly.  vlog should be there somewhere.

vlog exists in /home/cposada/intelFPGA/19.1/modelsim_ase/linux, but not in /linux_x86_64pe/ as the error suggests. The /linux_x86_64pe/ directory doesn't exist in my modelsim install directory, and vlog is within my PATH as well.

Update (or override) the QUESTASIM_PATH variable as defined in ./sims/cep_cosim/cep_buildHW.make to point to where your modelsim executables are installed.

Yessir I did that before I posted this discussion, but the -64 flag seems to invoke some alternate vlog binary that is located in the aforementioned /linux_x86_64pe/ directory. I've tried removing the -64 flag in the Makefile for the vlog command, which allows the make to run further than it did previously. However, it seems to lead to similar errors saying it cannot find the vlib binary and it points to a directory that does not exist within my modelsim install directory. Attached is a screenshot of the QUESTASIM_PATH variable I modified to point to my modelsim binaries.

Hello.  It seems like you have done due diligence.   I don't have that specific version of the tool installed, but could you attach the log when you remove the -64bit flag?
What happens if you try to run your modelsim manually with the -64 flag?

Attached is a screenshot of the change I made to cep_buildHW.make (just removing the -64 flag from COSIM_VLOG_ARGS), a screenshot of the error it returns, and the log itself. When I try to run vsim -64 manually, it returns a similar to what we saw previously saying how those specific binaries do not exist:
Failed to open executable /home/cposada/intelFPGA/19.1/modelsim_ase/bin/../linux/../linux_x86_64pe/vish in execute mode needed for the option -64.

execv: No such file or directory

** Fatal: Unable to exec the GUI /home/cposada/intelFPGA/19.1/modelsim_ase/bin/../linux/../linux_x86_64pe/vish.

cep_error_log.txt

Hello.  Unfortunately, I don't have the exact tool you are attempting to run with and thus am not able to reproduce your situation.  Please feel free to leave the issue open and I will try to find the tool and install when I can.