Recast nav mesh verticies go outside bounding box?

Overview of the Issue: Navmesh generation has high walls.
It seems with some inputs, the navmesh has points outside the shown bounding box in the Recast Demo application. Which is odd because the input mesh does not have points up there.
Windows 7, VS 2017
The agents do navigate over this spot briefly and go very high up.
The height is larger than Max Climb of 10.0 so why does it exist?
Callstack: N/A
Screenshots: (Have not figured out debug draw yet sorry)

Logs:
Reproduction steps:
I supply an example gset and mesh.
Example.zip
Interestingly changing tiles off 80 can make it go away.
Decreasing the Max edge Error can make it go away.
But it can still show up in other scenarios/meshes.
Note: It is possible that the mesh data is wrong/bad.
Recast version(s) and/or git commit hash: 2c85309280dbc9c82029e7ab16dfb01b9235c74e
Environment: Windows 7,
VS 2017
Related issues:
Suggest a Fix:
I wonder if this is caused by overflow or signed/unsigned mismatch.
So I compiled with optimization and higher warning level.
------ Rebuild All started: Project: Recast, Configuration: Release Win32 ------
Recast.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\Recast.cpp(401): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\Recast.cpp(448): warning C4365: 'initializing': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\Recast.cpp(448): warning C4365: 'initializing': conversion from 'unsigned int' to 'const int', signed/unsigned mismatch
RecastAlloc.cpp
RecastArea.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
RecastAssert.cpp
RecastContour.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(41): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(50): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(58): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(68): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(76): warning C4365: '=': conversion from 'int' to 'unsigned int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(156): warning C4365: '=': conversion from 'int' to 'unsigned char', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(177): warning C4365: '=': conversion from 'int' to 'unsigned char', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(728): warning C4365: 'argument': conversion from 'int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(774): warning C4365: 'argument': conversion from 'int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastContour.cpp(897): warning C4365: '=': conversion from 'int' to 'unsigned char', signed/unsigned mismatch
RecastFilter.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastFilter.cpp(111): warning C4365: 'initializing': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastFilter.cpp(112): warning C4365: 'initializing': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(39): warning C4820: 'rcLayerRegion': '1' bytes padding added after data member 'rcLayerRegion::neis'
..\..\..\Recast\Source\RecastLayers.cpp(376): warning C4365: 'initializing': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(376): warning C4365: 'initializing': conversion from 'int' to 'const unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(396): warning C4365: 'argument': conversion from 'int' to 'const unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(515): warning C4365: 'initializing': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(515): warning C4365: 'initializing': conversion from 'unsigned int' to 'const int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(517): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(523): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(525): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(531): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(533): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(539): warning C4365: 'argument': conversion from 'const int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(611): warning C4365: 'initializing': conversion from 'int' to 'unsigned char', signed/unsigned mismatch
..\..\..\Recast\Source\RecastLayers.cpp(632): warning C4365: '=': conversion from 'int' to 'unsigned char', signed/unsigned mismatch
RecastMesh.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastMesh.cpp(1029): warning C4365: 'argument': conversion from 'int' to '::size_t', signed/unsigned mismatch
..\..\..\Recast\Source\RecastMesh.cpp(1421): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastMesh.cpp(1444): warning C4365: 'initializing': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
RecastMeshDetail.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
RecastRasterization.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
RecastRegion.cpp
recastnavigation\Recast\Include\Recast.h(174): warning C4820: 'rcContext': '2' bytes padding added after data member 'rcContext::m_timerEnabled'
recastnavigation\Recast\Include\Recast.h(392): warning C4820: 'rcContour': '1' bytes padding added after data member 'rcContour::area'
recastnavigation\Recast\Include\Recast.h(1075): warning C4365: 'return': conversion from 'unsigned int' to 'int', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(88): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(97): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(108): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(117): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(142): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(151): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(162): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(171): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(262): warning C4365: 'initializing': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(423): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(467): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(534): warning C4820: 'rcRegion': '1' bytes padding added after data member 'rcRegion::connectsToBorder'
..\..\..\Recast\Source\RecastRegion.cpp(537): warning C4625: 'rcRegion': copy constructor was implicitly defined as deleted
..\..\..\Recast\Source\RecastRegion.cpp(537): warning C5026: 'rcRegion': move constructor was implicitly defined as deleted
..\..\..\Recast\Source\RecastRegion.cpp(537): warning C4626: 'rcRegion': assignment operator was implicitly defined as deleted
..\..\..\Recast\Source\RecastRegion.cpp(537): warning C5027: 'rcRegion': move assignment operator was implicitly defined as deleted
..\..\..\Recast\Source\RecastRegion.cpp(1390): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1391): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1392): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1393): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1565): warning C4365: 'initializing': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1580): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1581): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1582): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1583): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1591): warning C4365: '=': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1708): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1709): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1710): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch
..\..\..\Recast\Source\RecastRegion.cpp(1711): warning C4365: 'argument': conversion from 'int' to 'unsigned short', signed/unsigned mismatch

I am unable to reproduce the issue with your provided .gset. The terrain does not look like your exa
This issue used to happen when the detail mesh sampling sampled a span that was not walkable. This would happen because of edge simplification which would make some polygons go through terrain when the agent radius is very low (as in your example).
Most likely this is another case of #153 (you can see a similar situation happening in #151, which #153 was split out of).
However I do see that the .gset uses some very "extreme" settings. In general the lib is mostly tested of common "sane" settings, so I am not surprised that there might be issues with your particular configuration.

The gset+mesh is an example that has the problem. Just a chunk of the same mesh.
With agent radius (2.3), max edge error (1.5) and Max sample error (1) the problem appears to be gone.
It turns out the error's are not a bad thing as I had initially thought, they allow for smoothing in-between / not matching the input mesh exactly.

I am going to close this issue since I could not reproduce it. Please reopen if you can provide the steps to reproduce it.


FileNotFoundError: [WinError 2] The system cannot find the file specified on windows

File "c:\users<username>\appdata\local\programs\python\python37\lib\subprocess.py", line 1207, in _execute_child
startupinfo)
FileNotFoundError: [WinError 2] The system cannot find the file specified
i am trying to run bw_plex --url  -t  watch but everytime i run it i get the above error.

sorry i found a similar issue and the solution here solved it: https://github.com/Hellowlol/bw_plex/issues/76
i just installed ffmpeg and it works now


Model build error when training

Hi Team,
I got the following error when trying to train a model with my sample data.
It seems like a data error and there is no option to clean up or categorize encoding data in VS MLModel.mbconfig. To ensure model accuracy, it would be nice to pick the options for data clean-up, label encoding, choosing an applicable ML algorithm, and so on in VS configuration.

Thank you,
Srini

It would be good to have a validation and its validation summary result before training the data.

After I modified the 'Loan_Status' field type as Single and its values, model training got completed.

and prediction works as well

It's cool!


reusing the function injectIPEntry() inside injectEntries()

A "gnmi.Watch()" call is needed whenever we are executing a "gribiClient.AddIPv4()" call. But in the test, inside the function call "injectEntries()" that was missing. So, the test case was pumping a lot of traffic before all routes are up. As a result, there was a huge traffic loss and the test case was failing. This PR contains a fix for that.

Pull Request Functional Test Report for #1957 / 30951d73fdcb205758c61d0865c42765f1fefd68
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

TE-6.2: Route Removal In Non Default VRF

Cisco 8000E

TE-6.2: Route Removal In Non Default VRF

Cisco XRd

TE-6.2: Route Removal In Non Default VRF

Juniper cPTX

TE-6.2: Route Removal In Non Default VRF

Nokia SR Linux

TE-6.2: Route Removal In Non Default VRF

Openconfig Lemming

TE-6.2: Route Removal In Non Default VRF

Help


HSEARCH-2285 Explicit dependency on the javax.annotations to allow co…

…mpiling on JDK9

https://hibernate.atlassian.net/browse/HSEARCH-2285

thanks! merged.
Good point on the version, but I think it's more likely that we'll delete this dependency soon: it's just to workaround a bug in JDK9, which is hopefully resolved soon.


Chef-client removes array items in --local-mode with chef-zero

Description
We found a possible bug in chef-client when we define a json-attribute as an array with two identical strings Like:
{"onearray":["a","a"]}
At the first run chef-client does the correct thing and writes two "a" but at some point it seems to convert that array into a hash and upload one "a" to chef-zero
The 2nd run results in that it removes that 2nd "a"
Chef Version
Chef-client 12.17.44, 11.14.2
Platform Version
Linux, tested on both debian wheezy and arch-linux
Replication Case
cd /tmp
wget http://devsn.se/testcase.tar.gz
tar xf testcase.tar.gz
# First run
/opt/chef/bin/chef-client --local-mode --config /tmp/kitchen/client.rb --log_level auto --force-formatter --no-color --json-attributes /tmp/kitchen/dna.json --chef-zero-port 8889
# Second run
/opt/chef/bin/chef-client --local-mode --config /tmp/kitchen/client.rb --log_level auto --force-formatter --no-color --json-attributes /tmp/kitchen/dna.json --chef-zero-port 8889

Client Output
First run:
+# This file is generated by Chef for arraytest1-wheezy
+
+a
+a

Second run:
# This file is generated by Chef for arraytest1-wheezy

a
-a

testcase.tar.gz
Added test-case

it saves it correctly to the nodes/ directory with the array with two values.  i suspect that on the second run its doing array deep merge between the json attributes and the normal attributes in the node json and getting confused.

yeah deep merge algorithm is broken:
[1] pry(main)> require 'chef/mixin/deep_merge'
=> true
[2] pry(main)> include Chef::Mixin::DeepMerge
=> Object
[5] pry(main)> require 'chef/mash'
=> true
[7] pry(main)> merge({ array: ['a', 'a'] } , { array: ['a', 'a'] })
=> {"array"=>["a"]}

https://coderanger.net/arrays-and-chef/
offhand IDK if this get fixed via https://github.com/chef/chef-rfc/pull/238 or if it needs to be fixed before that RFC can proceed.

dropped in accepted major so its on my radar for Chef 13

This is still buggy in deep-merge, but I think I removed the use of deep merge in the actual way that node attributes are being merged as part of all kinds of refactoring that got done since chef 12 so this may not be an applicable bug any more.

Whatever it is this is still a bug in Chef-15-latest:
% chef-client -z  -j dna.json  -c client.rb
Starting Chef Client, version 14.13.11
resolving cookbooks for run list: ["test::test"]
Synchronizing Cookbooks:
  - test (0.0.1)
Installing Cookbook Gems:
Compiling Cookbooks...
[["default", :not_present],
 ["env_default", :not_present],
 ["role_default", :not_present],
 ["force_default", :not_present],
 ["normal", ["a", "a"]],
 ["override", :not_present],
 ["role_override", :not_present],
 ["env_override", :not_present],
 ["force_override", :not_present],
 ["automatic", :not_present]]
Converging 1 resources
Recipe: test::test
  * test_test[test] action run
    * ruby_block[test] action run
      - execute the ruby block test

Running handlers:
Running handlers complete
Chef Client finished, 2/2 resources updated in 03 seconds
git: rvm:ruby-2.6.1
15:21 coredump ~/zero % chef-client -z  -j dna.json  -c client.rb
Starting Chef Client, version 14.13.11
resolving cookbooks for run list: ["test::test"]
Synchronizing Cookbooks:
  - test (0.0.1)
Installing Cookbook Gems:
Compiling Cookbooks...
[["default", :not_present],
 ["env_default", :not_present],
 ["role_default", :not_present],
 ["force_default", :not_present],
 ["normal", ["a"]],
 ["override", :not_present],
 ["role_override", :not_present],
 ["env_override", :not_present],
 ["force_override", :not_present],
 ["automatic", :not_present]]
Converging 1 resources
Recipe: test::test
  * test_test[test] action run
    * ruby_block[test] action run
      - execute the ruby block test

Running handlers:
Running handlers complete
Chef Client finished, 2/2 resources updated in 03 seconds

This is actually correct behavior for deep_merge otherwise arrays would grow without bound.
On every execution of the client we need to take the value we saved and deep_merge it with the -j arguments.  If we use += to merge the two arrays, then on every execution the old array gets added to the new array.  So as a result it has to use |=.
The purpose is so that you can add a port to a firewall ruleset specified as an array or something like that.
But [ 80, 443 ] merged with [ 80, 443 ] is supposed to result in [ 80, 443 ] and not [ 80, 443, 80, 443 ] by design.
Without a compelling use case in here -- that is substantially MORE compelling that this firewall port number kind of case I think I'm closing this as working as designed.
We could simply not do array merging from the -j attributes and switch to using hash_only_merge instead of deep_merge, which would fix this issue, but if anyone out there is relying on the merging of port numbers above then they'll be broken hard, so that's a major breaking change, and would really need to be justified by something better than a simple example.  I don't see the compelling use case here, even though it feels buggy on the surface, and it would certainly hurt people to change the behavior so I think this needs to simply be closed as operating as designed.


Adds gemfile back to attempt Netlify build fix (again)

Builds are failing (again) in the Netlify preview.
This PR attempts to fix those builds by adding back Gemfile dependency management.

@rentry I get these errors when I attempt to build the branch in this PR.

Try running
bundle exec jekyll serve
On Mon, Jan 13, 2020 at 1:50 PM Shannon McHarg<EMAIL_ADDRESS>wrote:

@rentry https://github.com/rentry I get these errors when I attempt to
build the branch in this PR.
[image: image]
https://user-images.githubusercontent.com/12865238/72295059-ce888f00-3624-11ea-9a00-65c1bc148b3f.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenMaine/maineballot/pull/77?email_source=notifications&email_token=ADTN2IQQQMNXASYLCTXLG3TQ5TO2ZA5CNFSM4KF5ILVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEI2NPGA#issuecomment-573888408,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTN2IWGBEEFQQQVGXKELOTQ5TO2ZANCNFSM4KF5ILVA
.

I get this error when I do that.

In case the highlighted version is easier to read:

You probably need to run bundle install and then bundle exec jekyll serve.
Cross your fingers...this is where my dev environment broke and had to be reassembled.
We could delay this Netlify stuff until after the election, so you can get on with that, if you prefer.

Let’s put it off until after the election.

Sounds good.


Upload of 10MB file sync fails

Environment info

NooBaa Version: master-20210718
Platform: OCP 4.7

Actual behavior

Upload fails for multi-part upload

Expected behavior

Upload should not fail

Steps to reproduce

Upload attempt failed for folder for 10MB file
upload failed: akshat/file-size/size_10MB_99 to s3://bucket-1/file-size/size_10MB_99 An error occurred (NoSuchKey) when calling the CompleteMultipartUpload operation: The specified key does not exist.

Logs:
2021-07-19 09:05:09.867689 [PID-8/TID-8] [L1] FS::FileStat::OnOK: FileStat _stat_res.st_ino=110943119 _stat_res.st_size=10240000  2021-07-19 09:05:09.867755 [PID-8/TID-8] [L1] FS::FSWorker::Begin: FileClose _wrap->_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5-89 8e-2b83e61cd180/final _wrap->_fd=63  2021-07-19 09:05:09.867787 [PID-8/TID-16] [L1] FS::FSWorker::Execute: FileClose _wrap->_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5 -898e-2b83e61cd180/final _wrap->_fd=63 _uid=5000 _gid=800 _backend=GPFS  2021-07-19 09:05:09.867950 [PID-8/TID-16] [L1] FS::FSWorker::Execute: FileClose _wrap->_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5 -898e-2b83e61cd180/final _wrap->_fd=63  took: 0.130852 ms 2021-07-19 09:05:09.867991 [PID-8/TID-8] [L1] FS::FSWorker::OnOK: undefined FileClose _wrap->_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62 d-4ae5-898e-2b83e61cd180/final _wrap->_fd=63  2021-07-19 09:05:09.868016 [PID-8/TID-8] [L1] FS::FSWorker::Begin: Rename _old_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5-898e-2b8 3e61cd180/final _new_path=/nsfs/nsfs-nsr-1/bucket-2001/size-10mb/size_10MB_176  2021-07-19 09:05:09.868037 [PID-8/TID-18] [L1] FS::FSWorker::Execute: Rename _old_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5-898e- 2b83e61cd180/final _new_path=/nsfs/nsfs-nsr-1/bucket-2001/size-10mb/size_10MB_176 _uid=5000 _gid=800 _backend=GPFS  2021-07-19 09:05:09.868089 [PID-8/TID-18] [L1] FS::FSWorker::Execute: Rename _old_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5-898e- 2b83e61cd180/final _new_path=/nsfs/nsfs-nsr-1/bucket-2001/size-10mb/size_10MB_176  took: 0.02388 ms 2021-07-19 09:05:09.868134 [PID-8/TID-8] [L1] FS::FSWorker::OnError: Rename _old_path=/nsfs/nsfs-nsr-1/bucket-2001/.noobaa-nsfs_60f53a335038a50027af8e7c/multipart-uploads/57c9fa1c-d62d-4ae5-898e-2 b83e61cd180/final _new_path=/nsfs/nsfs-nsr-1/bucket-2001/size-10mb/size_10MB_176  error.Message()=No such file or directory  Jul-19 9:05:09.868 [Endpoint/8] [ERROR] core.sdk.namespace_fs::  [Error: No such file or directory] { code: 'ENOENT' } Jul-19 9:05:09.868 [Endpoint/8] [ERROR] core.endpoint.s3.s3_rest:: S3 ERROR <?xml version="1.0" encoding="UTF-8"?><Error>NoSuchKey<Message>The specified key does not exist.</Message>< Resource>/bucket-2001/size-10mb/size_10MB_176?uploadId=57c9fa1c-d62d-4ae5-898e-2b83e61cd180</Resource><RequestId>kraejweo-67jmb3-w73</RequestId></Error> POST /bucket-2001/size-10mb/size_10MB_176?u ploadId=57c9fa1c-d62d-4ae5-898e-2b83e61cd180 {"accept-encoding":"identity","user-agent":"aws-cli/2.1.29 Python/3.8.8 Linux/4.18.0-305.7.1.el8_4.x86_64 exe/x86_64.rhel.8 prompt/off command/s3.sync" ,"x-amz-date":"20210719T090509Z","x-amz-content-sha256":"6dc64d1f2999fb5dd50122890da22c3e4dbc0e6c9749a8d596557063396571f3","authorization":"AWS4-HMAC-SHA256 Credential=0muZI65pnSYg2HAIZsL1/2021071 9/us-east-1/s3/aws4_request, SignedHeaders=host;x-amz-content-sha256;x-amz-date, Signature=a6b5ad8632624effa674a14db2e36a0d6af492ff32ad50f2086c41604df3ee42","content-length":"271","host":"s3-nooba a.apps.ocp-akshat-1.cp.fyre.ibm.com","x-forwarded-host":"s3-noobaa.apps.ocp-akshat-1.cp.fyre.ibm.com","x-forwarded-port":"443","x-forwarded-proto":"https","forwarded":"for=<IP_ADDRESS>;host=s3-noo baa.apps.ocp-akshat-1.cp.fyre.ibm.com;proto=https","x-forwarded-for":"<IP_ADDRESS>"} Error: No such file or directory
More information - Screenshots / Logs / Other output

noobaa_diagnostics_1626687788.tar.gz

Investigating the issue, I saw that we open an FD for a final file on complete multipart upload and aggregate data into it.
Then we close the FD successfully and try to rename the final file for the actual key that we want to upload and fail on the rename due to ENOENT.

This bug seems to be fixed in master-20210802. Closing this.


Building: Add modules with DEPENDs to GENERATEd files

For files GENERATEd from templates (.in files), any perl module (.pm
file) that the file depends on will automatically be used.
This means that these two lines:
GENERATE[foo]=foo.in
DEPEND[foo]=whatever.pm

will emit this command in a Makefile (or corresponding):
foo: foo.in whatever.pm configdata.pm
$(PERL) -I. -Ipathto -Mwhatever -Mconfigdata $(SRCDIR)/util/dofile.pl \\
    foo.in > foo

Note that configdata.pm is automatically added, since util/dofile.pl
itself depends on it.

@richsalz, this should interest you in particular.

#10159 needs this :)
I can't understand all the perl, but I hope this gets merged soon.
Are there any internal doc changes needed?

Are there any internal doc changes needed?

Ah, yes, I totally forgot to adjust the build.info doc.

I still haven't adjusted the docs...

Documentation added

Merged.
70d9675342efbc9fe518e082c44f3ad80a1e4671 Building: Add modules with DEPENDs to GENERATEd files


Add client side validation to prevent entering non numbers

Add validation to the front end to prevent a user entering non numbers.

This issue was moved to sakaiproject/sakai#1348


Update README.md

add information about Meteor

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Okay, you're not interested.


Google: error importing instances created by GKE

Seeing this error trying to import instances created by GKE: (1668c152b3c558cedaf24641b0f3e1fdb4a1983a)
$ terraformer import google --resources=instances --projects=<p1> --regions=<r1>

2020/02/18 00:03:39 error formatting HCL: At 133:153: illegal char

looking at line 133:
133|       configure-sh = "#!/usr/bin/env bash\n\n# Copyright 2016 The Kubernetes Authors.\n#\n# Licensed under ....

Know issue
options:

Skip GKE instances. You really cannot manage single GKE instance with terraform
Convert configurate-sh(and other sh tags) to base64 and use base64decode for decode

WDYF?

(1) seems ideal; I agree completely that we shouldn't be managing individual GKE nodes with terraform; the node pool and cluster resources are sufficient for that

fixed


Expanding climate datapipes to include cosin zenith, land sea mask, geopotential and latlong

Expanding climate datapipe
Description
Refactor ERA5 datapipe so that it contains cosin zenith, lsm, geopotential and latlong stuff. This also generalized the loader to work with HRMIP data as well. Ultimate goal is to unify all weather modeling training scripts
Closes: https://github.com/NVIDIA/modulus/issues/77

Hey @mnabian, this is not finished yet but should unblock us for now. Basically a refactor of the current datapipe that allows us to get all needed fields to train graphcast and other models. If you could take a quick look to make sure this works for our desired problems that would be great!

Hey @mnabian and @ktangsali, This is ready for final review. I just merged the static data files needed so hopefully it will pass the CI tests. A couple of notes on how we can move forward with this in regards to getting the unified recipe together. My suggestion is to review this PR and merge is ASAP if it look good. Then I will work to get the new data loader working with @mnabian's new train script. There might (but hopefully not) be some small issues that come up along the way that will may require modifications to unified datapipe. I suggest we make them in the PRs for the unified recipe instead of delaying this PR because it will results in an a super large PR and get unmanagable. When we do the PR with the unified recipes then we can also clean stuff up like remove the old ERA5HDF5Datapipe. Does this sound good?
Also, should I still be setting the merge to main or is there a different development branch now?

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

Hey @mnabian and @ktangsali, This is ready for final review. I just merged the static data files needed so hopefully it will pass the CI tests. A couple of notes on how we can move forward with this in regards to getting the unified recipe together. My suggestion is to review this PR and merge is ASAP if it look good. Then I will work to get the new data loader working with @mnabian's new train script. There might (but hopefully not) be some small issues that come up along the way that will may require modifications to unified datapipe. I suggest we make them in the PRs for the unified recipe instead of delaying this PR because it will results in an a super large PR and get unmanagable. When we do the PR with the unified recipes then we can also clean stuff up like remove the old ERA5HDF5Datapipe. Does this sound good?
Also, should I still be setting the merge to main or is there a different development branch now?

That sounds good to me. And yes, please merge to main

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci

/blossom-ci


XH.prompt initialValue does not work

The MessageInput typedef doc refers to a initialValue prop being supported, however MessageModel is only looking for a value prop on the input config, and it is setting it to the defaultValue prop on the field config for the FormModel which it creates, which is also the wrong property.
It should be changed to initialValue in all places.

Looking through the develop branch, it looks like the only remaining fixes needed are:
to change this line:
https://github.com/exhi/hoist-react/blame/741c70fdd1850437331182fe572278203aaf55d4/core/appcontainer/MessageModel.js#L63
from:
const {value, rules} = input;
to
const {initialValue, rules} = input;
and to change this line:
https://github.com/exhi/hoist-react/blame/741c70fdd1850437331182fe572278203aaf55d4/core/appcontainer/MessageModel.js#L67
from:
initialValue: value,
to
initialValue,
Does that seem right?

Yep looks correct to me, noticed it on 25.2.0 where defaultValue was being used so looks like we are already halfway there!

Agreed. Then we're using initialValue across all consistently.
On Tue, Aug 6, 2019 at 5:06 PM Colin Rudd<EMAIL_ADDRESS>wrote:

Looking through the develop branch, it looks like the only remaining fixes
needed are:
to change this line:
https://github.com/exhi/hoist-react/blame/741c70fdd1850437331182fe572278203aaf55d4/core/appcontainer/MessageModel.js#L63
from:
const {value, rules} = input;
to
const {initialValue, rules} = input;
and to change this line:
https://github.com/exhi/hoist-react/blame/741c70fdd1850437331182fe572278203aaf55d4/core/appcontainer/MessageModel.js#L67
from:
initialValue: value,
to
initialValue,
Does that seem right?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/exhi/hoist-react/issues/1298?email_source=notifications&email_token=AASJA7ARB3HRI3UFERFIBLDQDHRWLA5CNFSM4IJZ55ZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3WO7DQ#issuecomment-518844302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASJA7AVEB54ULWTUY3X43LQDHRWLANCNFSM4IJZ55ZA
.

I fixed the assignment within the actual FormModel to get value working, but didn't notice that the typedef was specifying initialValue, which I think is better all around.
I will update now to initialValue consistently, including the one usage I know of.  I will add a breaking change note to the CHANGELOG as, while this brings the API in line with the docs, it will break any existing usages.
Thanks for noting.


Get Unit Tests passing on iOS

in CI.

98% done on iOS 9!

Clean test pass for iOS here: https://github.com/ThaliTester/TestResults/blob/549702610b97681_Switch_to_Unit_Tests_tobybrad/test_log.md

Review here: https://reviewable.io/reviews/thaliproject/thali_cordovaplugin/393#-


Usage in while in extended is incorrect

To have a task run at 20:16(:40) every evening:
time = ~N[2018-02-26 20:07:00]

# A basic non-extended call works as you would expect
~e[16 20 * * * *] |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
# +> [~N[2018-02-26 20:16:00], ~N[2018-02-27 20:16:00], ~N[2018-02-28 20:16:00]]

# However an extended call does not
~e[40 16 20 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
# => [~N[2018-02-26 20:16:40]]

Perhaps my mental model of this is incorrect, but shouldn't the second call return an enum that includes dates beyond the next invocation? (ie: [~N[2018-02-26 20:16:40], ~N[2018-02-27 20:16:40], ~N[2018-02-28 20:16:40]])?
If this isn't intended behaviour I'm happy to work up a PR to fix, but I want to make sure I'm understanding the intended use case properly.

@mtrudel This seems to be a bug. I'd be happy to accept a PR!

So we're in agreement, you'd see intended behaviour as being [~N[2018-02-26 20:16:40], ~N[2018-02-27 20:16:40], ~N[2018-02-28 20:16:40]] in that case?

@mtrudel Yes, that should be correct.

Trying to characterize the bug, and noticed something most curious:
 time = ~N[2018-02-26 20:07:00]
~N[2018-02-26 20:07:00]
iex(19)> ~e[21 16 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:16:21], ~N[2018-02-27 21:16:21], ~N[2018-02-28 21:16:21]]
iex(20)> ~e[22 16 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:16:22]]
iex(34)> ~e[22 15 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:15:22], ~N[2018-02-27 21:15:22], ~N[2018-02-28 21:15:22]]
iex(35)> ~e[23 15 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:15:23]]
iex(36)> ~e[23 14 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:14:23], ~N[2018-02-27 21:14:23], ~N[2018-02-28 21:14:23]]
iex(37)> ~e[24 14 21 * * * *]e |> Crontab.Scheduler.get_next_run_dates(time) |> Enum.take(3)
[~N[2018-02-26 21:14:24]]

There's a varying number of seconds into the start of the minute that work as expected, which seem to vary depending on the minute. The number of seconds after the minute which works doesn't seem to vary based on the hour or larger units in the cron expression, nor does it seem to matter what the value of time is.

I haven't started digging into this (hopefully later today), but that looks as good of a place to start as any. Will keep you posted.

@mtrudel I could verify that this is a performance / max cycles problem. The scheduler tries 10'000 times to find a date and will then stop. The date in your example is not found in that amount of cycles.
I did not yet check if it searches for the date less efficiently than other dates. So it's either a performance improvement or bug hunt.
 Maybe we could somehow make guesses to find the next date more efficiently.

@mtrudel This test passes with 100_000 maximum cycles:
test "max runs" do
  time = ~N[2018-02-26 20:07:00]

  run_times = ~e[22 16 21 * * * *]e
  |> Crontab.Scheduler.get_next_run_dates(time)
  |> Enum.take(3)

  assert run_times == [~N[2018-02-26 21:16:22], ~N[2018-02-27 21:16:22], ~N[2018-02-28 21:16:22]]
end

@mtrudel If we let all the tests from Crontab.FunctionalTest run in extended mode with second 0, there's 26 failures. How could I not see that :S

@mtrudel The python croniter package seems to make more intelligent guesses than this library does. See https://github.com/taichino/croniter/blob/master/src/croniter/croniter.py

Leaving myself a breadcrumb that the implementation in https://github.com/siebertm/parse-cron/pull/34 ought to work here

Is this issue still open in light of #50?

@mtrudel I’m currently improving the test suite to make sure that it is fully functional
now. I’ll close the issue with the new test cases.


profile Added - jinx-vi-0

Everything seems alright @jinx-vi-0 . About to merge 👍
Make sure to drop a star ⭐ to the repository


[SPARK-6339][SQL] Supports create CREATE TEMPORARY VIEW tableIdentifier AS query

What changes were proposed in this pull request?
This PR support new SQL syntax CREATE TEMPORARY VIEW.
Like:
CREATE TEMPORARY VIEW viewName AS SELECT * from xx
CREATE OR REPLACE TEMPORARY VIEW viewName AS SELECT * from xx
CREATE TEMPORARY VIEW IF NOT EXISTS viewName AS SELECT * from xx
CREATE OR REPLACE TEMPORARY VIEW IF NOT EXISTS viewName AS SELECT * from xx
CREATE TEMPORARY VIEW IF NOT EXISTS viewName (c1 COMMENT 'blabla', c2 COMMENT 'blabla') AS SELECT * FROM xx

How was this patch tested?
Unit tests.

work in progress.

Can one of the admins verify this patch?

@clockfly for work in progress pr, put WIP in the title.

test this please

add to whitelist

Test build #57693 has started for PR 12872 at commit 8af667e.

Test build #57693 has finished for PR 12872 at commit 8af667e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57693/
Test PASSed.

Test build #57707 has started for PR 12872 at commit a497cf3.

One thing that we've decided offline is that we should deprecate Dataset.registerTempTable() by Dataset.createTempView() in a follow-up PR. It's becoming confusing that we call the same thing "table" in Dataset API but "view" in SQL DDL.
I think the essential difference between a view and a table is that a view is basically a lineage, while a table is always materialized to disk. For example, data of temporary tables in Hive are written to scratch folder of the current session.

Test build #57710 has started for PR 12872 at commit 1e20bb0.

Test build #57707 has finished for PR 12872 at commit a497cf3.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57707/
Test FAILed.

Test build #57710 has finished for PR 12872 at commit 1e20bb0.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57710/
Test PASSed.

Test build #57756 has started for PR 12872 at commit 85d121c.

LGTM

Test build #57756 has finished for PR 12872 at commit 85d121c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57756/
Test PASSed.

Merged build finished. Test PASSed.

Would like to ask @yhuai to have a look at this.

LGTM

Merged to master and branch 2.0.

Currently, the existing DDL behaviors for temporary views when users do not specify the database name:

CREATE OR REPLACE TEMPORARY VIEW view_name alters the TEMPORARY view if the temporary view already exists.

DROP VIEW view_name OR SELECT... FROM view_name will be always first applied to a TEMPORARY view, if existing. If the temporary view does not exist, we will try to drop/fetch the PERSISTENT view, if existing.

ALTER VIEW view_name is only applicable to the PERSISTENT view, even if the temporary view with the same name exists.

@clockfly @rxin @yhuai @liancheng @hvanhovell @cloud-fan  Should we make them consistent?

Can you be more specific on the inconsistency? Seems ALTER VIEW view_name is the only inconsistent command?

Yeah, ALTER VIEW should be consistent with DROP VIEW, if we use the same naming rule. Should I submit a PR for it?
Another potential issue to users is the behaviors of CRAETE VIEW and CREATE TEMPORARY VIEW when users do not specify the database names. See the following example:
sql(s"CREATE TEMPORARY VIEW $viewName AS SELECT * FROM $tabName WHERE ID < 3")
sql(s"CREATE VIEW $viewName AS SELECT * FROM $tabName")

When we processing the second statement, we simply add CURRENT_DATABASE to make it a fully qualified view name. However, if users do not specify the fully qualified name in the subsequent SELECT/DROP, the persistent view is shadowed by the temporary view with the same name. The returned results might be a surprise to the Spark users, because they might not realize there exist a temporary view in the existing session.
Of course, the existing behavior is right, but I think the better way is to force users to specify the database name when creating a persistent view if there exists a temporary view with the same name. That means, we can issue an error message here in this specific case.


Bug with draft & subject in Gmail

Reproduce

Open two drafts in Gmail
Insert a template that has a subject, in one of those drafts
The two drafts will have a subject inserted

User says he'd pay if we fix it

Fixed in 3.4.5


Color preview doesn't work when value contains a zero padded number

When i hover over rgb(255, 129, 159) a box appears and displays the color, but when i hover over something like rgb(255, 176, 008) it does not show up, in the other hand if i change 008 to just 8 it works.

I've got a fix, PR will be up momentarily.

I have a fix up at #10705.


Feature/ws technologies doc

List of technologies applied in WS
Description
A table with technologies that DS Team has used in WS.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue) labeled with bug
[x] New feature (non-breaking change which adds functionality) labeled with enhancement
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] My code follows the code standards of this project.
[ ] My change requires updating the documentation. I have updated the documentation accordingly.

recheck

I have read the Dynamic Solutions Individual Contributor License Agreement (CLA) Document and I hereby sign the CLA

recheck


[FEATURE] Mobile View Profile Pic Issue

Description
In the mobile view of the Community Events page, there is a discrepancy regarding the display of profile pictures. On desktop, profile pictures are visible, but they are missing in the mobile view.
Steps to Reproduce:

Access the Community Events page on a desktop browser.
Observe that profile pictures are displayed correctly.
Access the same page on a mobile device.
Notice that profile pictures are missing in the mobile view.

Screenshots

Desktop view showing profile pictures

Mobile view without profile pictures

Do you want to work on this issue?
Yes
Additional information
Expected Behavior:
Profile pictures should be displayed consistently across both desktop and mobile views of the Community Events page.
Labels: enhancement

Thank you for raising.  This was a design choice as there is a lack of space in mobile view, so I will be closing this issue.


Add fonts

Codecov Report

Merging #18 into master will decrease coverage by 1.63%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master      #18      +/-   ##
==========================================
- Coverage   99.56%   97.93%   -1.64%
==========================================
  Files           8        8
  Lines         465      484      +19
==========================================
+ Hits          463      474      +11
- Misses          2       10       +8

Impacted Files
Coverage Δ

src/czml3/types.py
97.45% <57.14%> (-2.55%)
:arrow_down:

src/czml3/properties.py
97.7% <61.53%> (-2.3%)
:arrow_down:

Continue to review full report at Codecov.

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

Thanks @Sedictious! :tada:


Admin overlays hold a party around 0,0

Description

By sheer chance this round spawned saltern close to 0,0 and i noticed.

Havent seen this reported in a long while and cant repro locally.


HAR Viewer Integration

The main set of metrics we require to be rendered in the application UI (for MVP), is a 'waterfall' chart of assets from the tested page - in the same vain as the 'Network' tab in Chrome DevTools.
Background
The test agent will stream a fully-qualified JSON representation of the assets through a TCP socket into the application UI server. This will in turn be available to the browser through a WebSocket. The frequency of these messages is dictated by the speed in which the test agent's Chrome extension can resolve them, and pass them on.
Therefore, multiple re-renders of the 'HAR viewer' will occur - one on every new message from the WebSocket.
MVP

A non-interactive 'waterfall' chart of the assets from the test endpoint.

Questions

Since the 'HAR viewer' should be it's own standalone application, how can we embed it into the application UI?

For future reference and HAR parsing, the format specification: https://dvcs.w3.org/hg/webperf/raw-file/tip/specs/HAR/Overview.html


Add first class support to Compose

Version 2.5.0 of Lifecycle introduces a new CreationExtras parameter which would allow us to provide better support for SavedStateHandle on Compose.
What does that mean to Retained? In addition to the current APIs we currently support (ComponentActivity, Fragment and NavBackStackEntry) we could add a composable function that knows how to resolve the parent lifecycle owner, and connects to the right SavedStateProvider. Everything out of the box
The current API of Retained would remain the same, and you should be able to use any of the currently supported APIs with Compose, with the addition of a new function (to be defined):
@Composable
fun View() {
    val vm = retain { entry: RetainedEntry ->
        MyViewModel(entry.savedStateHandle)
    }
    // Other stuff.
}

Open questions:

Should we return a Retained delegate or follow viewModel and return the instance itself?

The new function would be provided by a new artifact: retained-compose.

We have a prototype working, but there is need to refactor and improve the core module. Lifecycle-2.5.0 has a more friendly API that will allow us to simplify a lot our internal code.


Mesh disappears when you press play

Hi, when I hit play the mesh for the TerainMesh program just disappears, but in the editing mode(When play hasn't been hit) everything looks great. Most of the time in play mode it doesn't display anything. If anyone has an idea please let me know how to fix this, I have tried a lot of things, but I'm new to game development and not too familiar with coding... especially to c#

.

You use this in a new scene with a new game object. The mesh generating is really involved and complicated, I believe each body has a BodyPlaceholder script and then SolarSystemSpawner manager goes through each object and creates mesh for it. Only then you have a usable mesh in the play mode.

For those who use the default Planet scene, see #50


Add a command to show metadata from the DB

Users may wish to view the current state of the local DB. Add a blog-replay ls command for this.
With no arguments, it should list the title and key of each blog in the DB.
With -l, also show the number of archived entries remaining in the DB.
With a blog key, list entry titles of the specified blog.

Added in v0.1.4


hook up background

almost all the changes are indentation. http://editorconfig.org/ is your friend 😃


feat: upgrade to mermaid-to-excalidraw v1 🚀

Release notes - https://github.com/excalidraw/mermaid-to-excalidraw/blob/e378ae217d90a26ad2a8c3296629e3a65ce2a977/CHANGELOG.md

seems to work well! 🚀


LESS & Antdesign Template: Hot Module Replacement error

Hot module replacement fails when using the react-static start command with the LESS & Antdesign template. Using react-static build && react-static serve works fine, which leads me to believe that it's very specifically an HMR problem.
Running react-static start also doesn't cause any errors in the terminal, it's just a browser javascript console error, accompanied by a blank white page.
The full error is Error: [HMR] Hot Module Replacement is disabled.
I'm running macOS High Sierra and get the error on Safari and Firefox (haven't checked Chrome since I don't have it installed)

I just tried this yesterday. Basically react-static@5 has broken the less-antdesign example. This error seems to happen with<EMAIL_ADDRESS>but does not happen with<EMAIL_ADDRESS>To repro:
react-static create  # Choose less-antdesign
cd my-static-site  # or the directory you just created
npm run start  # or `react-static start` as described above

The CLI does not report errors but when the browser navigates to http://localhost:3000, the page is blank (instead of showing the example content) and the browser console shows the  Error: [HMR] Hot Module Replacement is disabled. error.
The full stack trace hasn't helped me much in understanding what went wrong.
webpack-internal:///../../../webpack/hot/only-dev-server.js:70

Uncaught Error: [HMR] Hot Module Replacement is disabled.
    at eval (webpack-internal:///../../../webpack/hot/only-dev-server.js:70)
    at Object.../../../webpack/hot/only-dev-server.js (app.83a4ffc2.js:1840)
    at __webpack_require__ (app.83a4ffc2.js:50)
    at Object.0 (app.83a4ffc2.js:1873)
    at __webpack_require__ (app.83a4ffc2.js:50)
    at app.83a4ffc2.js:145
    at app.83a4ffc2.js:148

Interesting. I'm not familiar with this example very much. I'll find the creator and tag him here.


The gitter link that is rendered from the README.md returns 404

Clicking the project glitter link in the README.md that is rendered by Github at  https://github.com/ioam/holoviews returns a 404.

Strange, seems to work for me.

Also seems to work for me. Odd.

Oops; the icon link worked, but there was a bogus link in the text.  Fixed in 65eeb8c.


fix(broker): do not premature kill threads when not exiting

Missing most probably a busted test (testing this with Test::Nginx is difficult)

What is the difference between this PR and #47?

What is the difference between this PR and #47?

This fixes the broker and other fixes the worker.

@bungle @chronolaw can we merge this?


Error OSX command line

Trying to create a deb package. After using electron-packager tried this:
electron-installer-debian --src ./Marinamilitare-linux-x64 --dest ./ --arch amd64

Creating package (this may take a while)

[Error: Error executing command (2):
fakeroot dpkg-deb --build /var/folders/03/krb5m9s50479glzzw7d_h31w0000gn/T/electron-116110-5657-93rkda/marinamilitare_1.0.0_amd64
dpkg-deb: error: parsing file '/var/folders/03/krb5m9s50479glzzw7d_h31w0000gn/T/electron-116110-5657-93rkda/marinamilitare_1.0.0_amd64/DEBIAN/control' near line 9 package 'marinamilitare:amd64':
 blank line in value of field 'Installed-Size'
] 'Error: Error executing command (2): \nfakeroot dpkg-deb --build /var/folders/03/krb5m9s50479glzzw7d_h31w0000gn/T/electron-116110-5657-93rkda/marinamilitare_1.0.0_amd64\ndpkg-deb: error: parsing file \'/var/folders/03/krb5m9s50479glzzw7d_h31w0000gn/T/electron-116110-5657-93rkda/marinamilitare_1.0.0_amd64/DEBIAN/control\' near line 9 package \'marinamilitare:amd64\':\n blank line in value of field \'Installed-Size\'\n\n    at ChildProcess.<anonymous> (/usr/local/lib/node_modules/electron-installer-debian/src/installer.js:55:23)\n    at emitTwo (events.js:100:13)\n    at ChildProcess.emit (events.js:185:7)\n    at maybeClose (internal/child_process.js:821:16)\n    at Socket.<anonymous> (internal/child_process.js:319:11)\n    at emitOne (events.js:90:13)\n    at Socket.emit (events.js:182:7)\n    at Pipe._onclose (net.js:469:12)'

And directory /var/folders/03/krb5m9s50479glzzw7d_h31w0000gn/T/electron-116110-5657-93rkda doesn't exists

Problem was
description: ''
in package.json


Is there some way to insert videos or gif images?

Hi everyone. Could you please tell me how to insert a gif image or another video into my math video in manim?

You can't do that i manim yet

ffmpeg -i xxxx.mp4 xxx.gif

LaTeX insert animate？
\usepackage{animate}
\begin{document}
\begin{frame}
\animategraphics[
autoplay,loop,controls,
width=.7\textwidth,
height=.7\textheight
]{4}{your-gif-to-png-file-prefix}{0}{3}
\end{frame}
\end{document}

Use premiere instead.

You can add video to manim by adding and removing a series of images. Manim knows to take images as numpy arrays. I used opencv to slice the video to frames. This code generates a video at approximately 25 Hz. You can change the wait time to your requirement.
cap = cv2.VideoCapture("path to your video file") flag = True while flag : flag, frame = cap.read() if flag: frame = cv2.cvtColor(frame, cv2.COLOR_BGR2RGB) frame_img = ImageMobject(frame) self.add(frame_img) self.wait(0.04) self.remove(frame_img)
cap.release()

Where do any of the indents in this code go?

Hi @georgeoshardo what is the library to import to get that running please ?

thx a lot. I'm afraid there is no version for MacOs intel. Still looking here and there if any option. :-(

appreciated !

@Turgibot Just FYI this does not work for 3b1b/manim. You can only insert arrays as ImageMobjects in the community version.


[5.6] adjust FileSystem implementations to TSC API changes (#3962)

5.6 cherry pick of #3962

https://github.com/apple/swift-tools-support-core/pull/275
@swift-ci please test


Sublime linter is not working

startup time: 1.42292
first paint time: 1.43781
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/app has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/app inode: 107666033
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/appfs has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/appfs inode: 107666037
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/arq has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/arq inode: 107666051
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/blog has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/blog inode: 107666060
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/cmd has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/cmd inode: 107666067
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/crypt has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/crypt inode: 107666074
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/devweb has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/devweb inode: 107666076
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/gf256 has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/gf256 inode: 107666094
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/fuse has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/fuse inode: 107666080
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/gtfs has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/gtfs inode: 107666112
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/google has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/google inode: 107666099
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/keychain has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/keychain inode: 107666128
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/imap has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/imap inode: 107666114
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/mbta has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/mbta inode: 107666131
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/plist has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/plist inode: 107666136
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/regexp has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/regexp inode: 107666157
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/qr has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/qr inode: 107666139
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/s3get has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/s3get inode: 107666173
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/rosetta has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/rosetta inode: 107666166
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/smugmug has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/smugmug inode: 107666175
scan: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/tmp/xmpp has been seen before, skipping (using inode) previous path: /Users/asaadmahmood/Git/projects/go/src/github.com/mattermost/platform/vendor/github.com/mattermost/rsc/xmpp inode: 107666203
launching: /Applications/Sublime Text.app/Contents/MacOS/plugin_host
reloading plugin Default.block
reloading plugin Default.comment
reloading plugin Default.copy_path
reloading plugin Default.delete_word
reloading plugin Default.detect_indentation
reloading plugin Default.duplicate_line
reloading plugin Default.echo
reloading plugin Default.exec
reloading plugin Default.fold
reloading plugin Default.font
reloading plugin Default.goto_line
reloading plugin Default.history_list
reloading plugin Default.indentation
reloading plugin Default.kill_ring
reloading plugin Default.mark
reloading plugin Default.new_templates
reloading plugin Default.open_file_settings
reloading plugin Default.open_in_browser
reloading plugin Default.pane
reloading plugin Default.paragraph
reloading plugin Default.paste_from_history
reloading plugin Default.save_on_focus_lost
reloading plugin Default.scroll
reloading plugin Default.set_unsaved_view_name
reloading plugin Default.side_bar
reloading plugin Default.sort
reloading plugin Default.swap_line
reloading plugin Default.switch_file
reloading plugin Default.symbol
reloading plugin Default.transform
reloading plugin Default.transpose
reloading plugin Default.trim_trailing_white_space
reloading plugin CSS.css_completions
reloading plugin Diff.diff
reloading plugin HTML.encode_html_entities
reloading plugin HTML.html_completions
reloading plugin 0_package_control_loader.00-package_control
reloading plugin 0_package_control_loader.02-bz2
reloading plugin All Autocomplete.all_views_completions
reloading plugin AutoFileName.autofilename
reloading plugin AutoFileName.getimageinfo
reloading plugin Emmet.emmet-plugin
reloading plugin Package Control.1_reloader
reloading plugin Package Control.2_bootstrap
reloading plugin Package Control.Package Control
loaded 1318 snippets
reloading plugin SCSS.scss_completions
reloading plugin SideBarEnhancements.SideBar
reloading plugin SideBarEnhancements.SideBarAPI
reloading plugin SideBarEnhancements.SideBarDefaultDisable
reloading plugin CSScomb.CSScomb
reloading plugin SublimeLinter.commands
reloading plugin SublimeLinter.sublimelinter
reloading plugin SublimeLinter-contrib-eslint.linter
SublimeLinter: eslint linter loaded
plugins loaded
SublimeLinter: debug mode: on
SublimeLinter: temp directory: /var/folders/f6/mrrfsmgs5fn4k088ndr5v4g40000gn/T/SublimeLinter3-asaadmahmood
found 7 files for base name Main.sublime-menu
Emmet: No need to update PyV8
Package Control: Skipping automatic upgrade, last run at 2016-07-01 18:40:37, next run at 2016-07-01 19:40:37 or after

I've already added the location for eslint to my paths, and as it can be seen I've installed linter for eslint aswell, but it just doesn't work.

Just restart sublime


Update FCM message must be converted to an FCM notification

Instead of sending a "message" via FCM, send an actual notification via FCM

I think I decided that we should just handle it in the client because we can be flexible


Exception running macOS Demo app

Hi. I get a crash during authentication using the SwifterDemoMac app (I tried with your consumer keys and mine, same thing). It happens after returning from the web browser auth step.
Thread 1 Queue : com.apple.main-thread (serial)
Thread 5Thread 7 Queue : com.apple.network.connections (serial)
#0	0x00007fff54af41f6 in __cxa_throw ()
#1	0x00007fff3ad43a6f in Security::UnixError::throwMeNoLogging(int) ()
#2	0x00007fff3aaf13d8 in Security::safeCopyFile(char const*, unsigned int, char const*, unsigned short) ()
#3	0x00007fff3aaf01a1 in Security::MDSSession::updateDataBases() ()
#4	0x00007fff3ab424d5 in Security::MDSSession::DbOpen(char const*, cssm_net_address const*, unsigned int, Security::AccessCredentials const*, void const*, long&) ()
#5	0x00007fff3ab42354 in mds_DbOpen(long, char const*, cssm_net_address const*, unsigned int, cssm_access_credentials const*, void const*, long*) ()
#6	0x00007fff3aaef6e3 in Security::MDSClient::Directory::cdsa() const ()
#7	0x00007fff3ac02eb0 in Security::MDSClient::Directory::dlGetFirst(cssm_query const&, cssm_db_record_attribute_data&, cssm_data*, cssm_db_unique_record*&) ()
#8	0x00007fff3aaef1ac in Security::CssmClient::Table<Security::MDSClient::Common>::startQuery(Security::CssmQuery const&, bool) ()
#9	0x00007fff3aaeeea1 in Security::CssmClient::Table<Security::MDSClient::Common>::fetch(Security::CssmClient::Query const&, int) ()
#10	0x00007fff3aaee67c in MdsComponent::MdsComponent(Security::Guid const&) ()
#11	0x00007fff3aaee2b8 in CssmManager::loadModule(Security::Guid const&, unsigned int, Security::ModuleCallback const&) ()
#12	0x00007fff3aaee057 in CSSM_ModuleLoad ()
#13	0x00007fff3aaedb6b in Security::CssmClient::ModuleImpl::activate() ()
#14	0x00007fff3aaed91e in Security::CssmClient::AttachmentImpl::activate() ()
#15	0x00007fff3aaed822 in Security::KeychainCore::Certificate::clHandle() ()
#16	0x00007fff3ab12c2a in Security::KeychainCore::Certificate::copyFirstFieldValue(cssm_data const&) ()
#17	0x00007fff3ab12ac4 in Security::KeychainCore::Certificate::publicKey() ()
#18	0x00007fff3ab12992 in SecCertificateCopyPublicKey ()
#19	0x00007fff549bd2dd in boringssl_context_extract_certificates_from_x509_store ()
#20	0x00007fff549b51a7 in _boringssl_context_SSL_ctx_verify_callback ()
#21	0x00007fff54a2b301 in ssl_verify_cert_chain ()
#22	0x00007fff549cb521 in ssl3_connect ()
#23	0x00007fff54a2db4e in SSL_do_handshake ()
#24	0x00007fff549b3f5c in boringssl_session_handshake_continue ()
#25	0x00007fff549b3b0d in boringssl_context_handshake_negotiate ()
#26	0x00007fff549bed51 in nw_protocol_boringssl_handshake_negotiate ()
#27	0x00007fff549bf394 in nw_boringssl_read ()
#28	0x00007fff549c0344 in nw_protocol_boringssl_input_available ()
#29	0x00007fff55d9d32b in nw_socket_update_input_source ()
#30	0x0000000100f05d8f in _dispatch_client_callout ()
#31	0x0000000100f1a3d2 in _dispatch_continuation_pop ()
#32	0x0000000100f0848f in _dispatch_source_invoke ()
#33	0x0000000100f1b8d3 in _dispatch_queue_serial_drain ()
#34	0x0000000100f0d801 in _dispatch_queue_invoke ()
#35	0x0000000100f1cfca in _dispatch_root_queue_drain_deferred_wlh ()
#36	0x0000000100f21ea3 in _dispatch_workloop_worker_thread ()
#37	0x0000000100f81fd6 in _pthread_wqthread ()
#38	0x0000000100f81bed in start_wqthread ()

Thanks for raising this issue, I've fixed this in 7ff58d6a0fd11ac004b624938117b09c1122ecdb


1.2.12 버전

Summary by CodeRabbit

New Features

Added Korean localization for Lock Screen Widget terms.
Introduced a new Lock Screen Widget with options for app links and definitions.

Enhancements

Improved project configuration for better scheme visibility and order in Xcode.
Enhanced user experience by displaying a white tinted ProgressView during data loading in the Update List Note view.

Refactor

Updated references from MakeWidgetViewModel to CreateWidgetViewModel in Icon Grid View.
Optimized asynchronous data handling in the Update List ViewModel for a smoother user experience.

Chores

Updated project settings, including deployment target and version.
Enhanced code quality with a new shell script build phase for Swift linting.

Style

Cleaned up code by removing unnecessary print statements and redundant empty lines.

GPT TEST


Correction for raw affiliation Cerema, Research Team ENDSUM, 23 Avenue de l'Amiral Chauvin, 49136, Les Ponts de Cé, France

Correction needed for raw affiliation Cerema, Research Team ENDSUM, 23 Avenue de l'Amiral Chauvin, 49136, Les Ponts de Cé, France
raw_affiliation_name: Cerema, Research Team ENDSUM, 23 Avenue de l'Amiral Chauvin, 49136, Les Ponts de Cé, France
new_rors: 05k0nvr25
previous_rors: 05k0nvr25;02nwvxz07
works_examples: W4386052717
contact: 27001b5a9f8d58835db3abccec3b29c3:5462d28ccf5b58133b187dd05831a8 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


Browser freeze  when using the dragging to upload folder

Hi,
I have problem when trying to using the dragging to upload folder. When I drag folder to the upload area, it works perfectly. But when I click "cacel", my browser freeze immediately. Here's my video about that: https://youtu.be/Q7ttkuNccPc
I hope you can help me about this issue.
Thanks

Did you try to debug ?
What you have like issue on the console ?

what is this project?
On Sun, May 29, 2016 at 3:14 AM, Soufiane Fadil<EMAIL_ADDRESS>wrote:

Did you try to debug ?
What you have like issue on the console ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flowjs/ng-flow/issues/272#issuecomment-222327660, or mute
the thread
https://github.com/notifications/unsubscribe/AFyZc7esx99VLIj2voZWtRN_5OxAHW8zks5qGKI2gaJpZM4IIO8V
.


Fix tests

This would be a temporary fix to solve the name collusion. Eventually the run_tests command will be adjust to accommodating whatever file names as input.
Won't wait for approve but merge quickly to fix build break.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2444   +/-   ##
=======================================
  Coverage   72.16%   72.16%
=======================================
  Files         334      334
  Lines       18741    18741
  Branches     2772     2772
=======================================
  Hits        13524    13524
  Misses       4348     4348
  Partials      869      869

Continue to review full report at Codecov.

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


command line cheat sheet

I wonder if you have a list of command line parameters that can be used, I noticed you are using some like --no-temp but I'm not sure what exactly those do. I there one for a dry run, just to check that the files supplied are correct? Some of them may be inherited from snakemake, but would be good to summarize the most useful ones in a 'cheatsheet'

The minute run command just forwards all supplied arguments to snakemake, so you can use everything that snakemake accepts.
In our Snakefile, some output files are marked as temporary, that is, they are deleted as soon as they are not needed anymore (most of the files under the tmp/ folder). With --notemp, they are not deleted, which is nice for debugging.


Enable price feeds for PEN and ARS on batching server

Context
Since in the future, the PEN (Peruvian Peso) and ARS (Argentinian Sol) asset prices are needed on-chain, we need to add them to the configuration of the batching server.
TODO

[ ] Add them to the default list of supported currencies here
[ ] Redeploy the batching server. The batching server is configured to always use the default parameter of the supported currencies, so this should be enough to apply the setting. The redeployment should be automatically triggered when merging a PR to main.

Hey team! Please add your planning poker estimate with Zenhub @b-yap @bogdanS98 @ebma @gianfra-t @TorstenStueber

Could FIAT:PEN-USD ever lead to confusion with PEN (Pendulum)? The FIAT: prefix describes/implies that the asset between the : and - is a fiat currency and therefore can never be PEN (Pendulum), correct?

Yes exactly. Since we distinguish between forex and crypto prices with the blockchain field we can disambiguate the two.


Probleman CPU usage

100% CPU usage dbeaver 3.8.5 (120%)
S.O: Ubuntu 16.04
java version "1.8.0_121"
Java(TM) SE Runtime Environment (build 1.8.0_121-b13)
Java HotSpot(TM) 64-Bit Server VM (build 25.121-b13, mixed mode)

Version 3.8.4 works fine with this configuration.

That's odd (because version 3.8.5 had CPU load issues which were fixed in 3.8.5).
Check this: #1080
If turning GTK 3 off doesn't help then try this:

Run DBeaver (so CPU load will become near 100%).
Make a thread dump (https://github.com/serge-rider/dbeaver/wiki/Making-a-thread-dump)
Post thread dump here

When running Hive query DBeaver CE 3.8.4 takes around 40% CPU on my MBA.
If I open a new script tab with empty text the CPU drops down to 10%.
JProfiler shows that ScriptPositionColumn.paint takes the most time when in the tab of the running SQL script:
org.jkiss.dbeaver.ui.editors.text.ScriptPositionColumn.paint(org.eclipse.swt.graphics.GC, org.eclipse.jface.text.source.ILineRange)	271663	65	4179	3400	2367	10904	1590	2.2070588235294117

When switched to empty tab this are the top two:
com.cloudera.hive.hivecommon.api.HS2ClientWrapper.GetOperationStatus(org.apache.hive.service.cli.thrift.TGetOperationStatusReq)	5004760	1	5004760	5004760	5004760	5004760	15	0.0
com.cloudera.hive.hivecommon.api.HS2ClientWrapper.recv_GetOperationStatus()	5004534	1	5004534	5004534	5004534	5004534	9	0.0

Thanks for details. It is interesting, although I can't reproduce it on Windows, Ubuntu.
I will try to improve script position rendering function (redraw less often).
Could you check how this works in EA version (http://dbeaver.jkiss.org/files/ea/) ?

The EA version looks similar - I am getting ~8% CPU usage on empty tab vs ~38% on the same script while running the query.

Thanks for info.
8% CPU usage seems to be produced by hive driver (reading something from server? just idle mode?)
I've fixed script paint function so it shouldn't consume so much CPU (unfortunately I can't test this myself). Fresh EA version uploaded.


Port forwarding is breaking my ability to access my network remotely

I have 2 sonoff cameras and i've port forwarded 50000 to 50019, and i've set the same range in the WebRTC integration. For some reason, when I do this, it breaks my ability to access my entire home assistant instance remotely. I have to disable the port forwarding to get back the ability to access my home assistant instance, but then I cannot see the cameras when accessing remotely.

This could be an issue on my ISP side and just co-incidence that when i removed the port forwarding, access was restored to home assistant. I did a few restarts of devices and everything seems fine now. I will keep an eye on this, and report back in a few days if everything is still fine or not.

Port forward will work only if you have public IP address. Port forward with private IP address may break external access


Kinematic

#1

M_PI is non standard, I advise against using it.
We need a Math class with public static attributes and methods for those things.

After discussion with @Hindi, we decided to restore the default operator== and to introduce a method isSimilarinstead.

compute_new_orientation should be computeNewOrientation


feat: Initial layout of CEEI dashboard

Thank you for submitting a pull request!
Description
Creating this PR to coordinate with @mikedstr for the CEEI Dashboard.
This PR will be marked as a draft until it is ready and the Electricity Dashboard is merged.
Type of Pull Request

[x] Adding a feature
[ ] Fixing a bug
[ ] Maintaining documents
[ ] Adding tests
[ ] Others ()

Verify the followings

[ ] Code is up-to-date with the develop branch
[ ] No build errors after pnpm build
[ ] No lint errors after pnpm lint
[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check
[ ] Make sure all the existing features working well

Changesets

[ ] If your PR makes a change under packages folder that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

Refer to CONTRIBUTING.MD for more information.
┆Issue is synchronized with this Wrike task by Unito

Hi @JinIgarashi, I created a new PR that branches off of develop now instead of develop-ibm. Sorry for the mix-up!

I deployed this feature branch as a deployment slot in Azure App Service. It can be accessed from the below URL.
https://geohub-prod-ceei.azurewebsites.net/dashboards/ceei

Hi Jin, I'm currently merging Mike's changes with my own atm, I'll be sure to let you know when it's pushed.

@lrt-sibat I found a bug of attribute popup.
steps to reproduce it:

duplicate a layer
customise duplicated layer to simulate
click anywhere on the map to show attribute popup
compare two layers' values, and discovered duplicated layers' values are not changed after simulating.

@lrt-sibat also, color map is not duplicated to new layer.

change colormap in base layer
duplicate base layer,
make base layer invisible
you can realize color of duplicated layer on map is not copied

also, please solve confict of pnpm-lock.yaml by following command
rm pnpm-lock.yaml
pnpm i

@lrt-sibat I think you can remove All from the dropdown menu.

show all countries if a country is not selected
show only selected country if a country is selected
show all countries after clicking x button to reset

furthermore, I think you can remove select box element. Instead showing text box to filter always. If user types any keyword, show dropdown menu as it does currently.

@lrt-sibat there are two small requests
Rename Base layer to CEEI

current

proposed

make height of popup smaller
The height of popup is too large, I think we need to set max-height to make popup smaller, and set overflow-y to enable scrolling. this might be better.
Also, maybe accordion can be introduced for Potential, Urgent and Means and Resources categories level.

current


[update] Marketplace: update appwrite logos

Description 📝
Update Appwrite to include new logo and colors
Changes  🔄
upload appwrites new logo
Target release date 🗓
as soon as possible (no rush, but if we can fit it in next release that would make the partner happy :))

Before:

After:

After:

The way the wider aspect ratio causes the logo text to become illegible is not great -- I wonder if we could include just the circular icon and leave out the text?

Updated screenshot with new logo


Remove RGi labeling from Verifier and Swagd programs

No branding at all, RGi, ERDC, or otherwise

@jr12 please verify this is fixed and close if so


React: Pass textMaskConfig to update method

This PR does the same as #418, but for the react component.
Instead of passing the textMaskConfig to the createTextMaskInputElement method, its now being passed into the update method.
componentDidMount() {
  this.textMaskInputElement = createTextMaskInputElement()
  this.update()
},

update(value) {
  const textMaskConfig = Object.assign({inputElement: this.inputElement}, this.props)
  this.textMaskInputElement.update(value || this.props.value, textMaskConfig)
},

However, I am unable to test this properly… I don’t know how to rig-up any form controls to change config settings (my knowledge of React is too little).. But I can’t see why it wouldn’t work.
@msafi could you take a look at this PR, and perhaps test it

closing this in favour of #439


All terrain at EKVG removed with autoortho installed.

EKVG is a very mountainous and dangerous airport but with the ortho installed it becomes flat and has zero mountains around it.

Yeah, confirming this.  This is almost certainly some kind of issue with Ortho4XP.  https://github.com/oscarpilote/Ortho4XP

Ah ok I'll make a issue there.

Thanks! I'll try to recreate just those areas and see if any warnings
standout
On Sat, Jan 21, 2023, 12:03 PM Awemeter @.***> wrote:

Ah ok I'll make a issue there.
—
Reply to this email directly, view it on GitHub
https://github.com/kubilus1/autoortho/issues/38#issuecomment-1399289459,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABVEDLGCRTHOIBBIYZQU35LWTQJE3ANCNFSM6AAAAAAT737GYM
.
You are receiving this because you commented.Message ID:
@.***>

As of May the 25th this issue still persists


Map reference as contract address

Hello. Is it possible to map a reference from 1 contract call to be passed as a value to another contract call?
So I would like to get a PAIR of a token-BnB and then use that pair address to call getReserves
[
    {
      reference: 'factory',
      contractAddress: factoryAddress,
      abi: factoryABI,
      calls: [
        { reference: 'tokenPair', methodName: 'getPair', methodParameters: [tokenAddress, bnbAddress] },
      ]
    },
    {
      reference: 'liquidity',
      contractAddress: tokenPair,   <-----------------------------------------------------------
      abi: [
        {
          'constant': true,
          'inputs': [],
          'name': 'getReserves',
          'outputs': [
            { 'internalType': 'uint112', 'name': '_reserve0', 'type': 'uint112' },
            { 'internalType': 'uint112', 'name': '_reserve1', 'type': 'uint112' },
            { 'internalType': 'uint32', 'name': '_blockTimestampLast', 'type': 'uint32' }
          ],
          'payable': false,
          'stateMutability': 'view',
          'type': 'function'
        }
      ],
      calls: [
        { methodName: 'getReserves' },
      ]
    }
  ]

Hey this is a great thing to suggest. Right now this does not support this logic. This would be contract changes for now out of scope to put in this library. Will keep this open and when I get some time investigate further. For now that will have to be 2 separate calls.


several pickle files

hmm for some reason this contribution appears under the github account @eladrap, even though I made the PR via this account (erap129). Did you somehow add @eladrap manually? If so I would be glad if you can swap it so this work will appear under this account (erap129). Thanks.

Hi, are we missing anything for this to continue? Thanks :)

Really hate to nag but do you think we will follow through with this? Thanks.

Hey, all good. Good of you to nag.

I didn't change anything manually. I wouldn't even know how to try something like altering the username this appears under. Let me check if I can fix this, however.

Yeah it was me of course, I made a mess by confusing my private and work accounts. I amended the commit to change the author and made the PR again. It's my first open-source contribution so I guess I'm just checking off beginner mistakes :)


WIP: support TR features

🚧 DO NOT LAND ⚠
Fixes: #68

Coverage decreased (-0.3%) to 90.746% when pulling 359205f3702db26de752dc8322d434f5599f9bb3 on srl295:ht-api-2 into 9873ab8a9e7a0e463c645d293e4cbeed0fbc8c7f on IBM-Bluemix:master.

Coverage decreased (-0.2%) to 90.843% when pulling 42861bea7fe767062025ac7b1e440c59ca21929e on srl295:ht-api-2 into 9873ab8a9e7a0e463c645d293e4cbeed0fbc8c7f on IBM-Bluemix:master.

rebased on master since #69 landed

Coverage increased (+0.1%) to 90.843% when pulling 7c385d62a8ed670bb7724042d243892f42e84ab8 on srl295:ht-api-2 into 32f2eb4986266a5f9661e5165478f3613d729b75 on IBM-Bluemix:master.

Coverage increased (+0.4%) to 91.183% when pulling d7a6c85e5ae406e6544b41ac1594a6aa11488ec2 on srl295:ht-api-2 into 32f2eb4986266a5f9661e5165478f3613d729b75 on IBM-Bluemix:master.

Coverage increased (+1.08%) to 91.828% when pulling fdbdb52e99731663d0357258dadb9212ed589580 on srl295:ht-api-2 into 32f2eb4986266a5f9661e5165478f3613d729b75 on IBM-Bluemix:master.


#6217 Gradle-like %classpath mvn

sorry for the confusion but this should be an additional optional syntax, please keep all our documentation as it is, except add one cell that shows you can use the : if you want.  the magic should work either way.

👍
please bring up to date with master


Pydantic error when listing tailnet devices: "Input should be a valid dictionary or instance of ClientConnectivity"

Problem/Motivation
This morning I am getting an error from python-tailscale when listing tailnet devices:
Traceback (most recent call last):
  File "script.py", line 20, in ts_devices
    devices = await tailscale.devices()
  File "/home/heretic098/anaconda3/lib/python3.9/site-packages/tailscale/tailscale.py", line 116, in devices
    return Devices.parse_obj(data).devices
  File "/home/heretic098/anaconda3/lib/python3.9/site-packages/typing_extensions.py", line 2360, in wrapper
    return arg(*args, **kwargs)
  File "/home/heretic098/anaconda3/lib/python3.9/site-packages/pydantic/main.py", line 945, in parse_obj
    return cls.model_validate(obj)
  File "/home/heretic098/anaconda3/lib/python3.9/site-packages/pydantic/main.py", line 480, in model_validate
    return cls.__pydantic_validator__.validate_python(
pydantic_core._pydantic_core.ValidationError: 1 validation error for Devices
devices.4935547714179711.clientConnectivity
  Input should be a valid dictionary or instance of ClientConnectivity [type=model_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.0.3/v/model_type

Checking the response from the tailscale API shows that they have stopped including the ClientConnectivity details in the output of the request.
Expected behavior
To get a list of devices
Actual behavior
I get the error message detailed above
Steps to reproduce
I suspect most scripts relying on this module are busted like this this morning.
Proposed changes
As a quick fix, I tried substitiuting line 56 in src/tailscale/models.py with this:
    client_connectivity: Optional[ClientConnectivity] = Field(alias="clientConnectivity")

which works for me on my local system and allows the module to list devices again.
I recall seeing something in the tailscale API repo saying that the intention was to remove the clientconnectivity details from the response but I cannot find it now. If you want, I'm happy to submit a PR with the above change.

This does not appear to be an issue this morning although the clientconnectivity information is still missing from the API response. Clearly I don't understand what is going on.

I'm sorry, but this library doesn't use Pydantic anymore. Please make sure you are using the latest version.
../Frenck


[MAJOR] Move immutable/* re-exports to immutable/index.js

[x] Move immutable/* re-exports to immutable/index.js
[x] Scour and update docs
[x] Scour and update examples

Released in https://github.com/FormidableLabs/redux-little-router/releases/tag/v15.0.0


Import & whitespace fixes

I found these changes necessary to compile my project with PureScript 0.13 and the halogen-5 branch.

Thanks! I hadn't verified this with 0.13 yet.

Hi @OndrejSlamecka -- for some reason it looks like no CI ran and I mistakenly merged a change that causes the current master branch to stop building. I've reverted your commits there.
The problem is that the imports and whitespace changes you made are necessary for the 0.13 compiler to work; they were also changes that had to be made in other libraries, like profunctor-lenses and typelevel-prelude. Unfortunately it doesn't look like Halogen v4.0.0 is updated to library versions late enough to work properly and I wasn't able to resolve the dependencies in order to get the 0.13 compiler working and the project compiling.
Please feel free to re-open these changes, but against the halogen-5 branch instead, and I'll investigate what to do about the 0.13 compiler on this project's main branch.

Hi @thomashoneyman , thank you, makes sense. I will open a new PR.


Reorganize info on quickstart

Move script links to the top, prune duplicative script stuff.
┆Issue is synchronized with this Jira Task by Unito

I think this has been covered. Closing


Doctest tutorial files

Try to make sure that our ReST tutorial stays current and does not give misleading examples. To this end, extend refguide_check to also check the contents of the doc/source/tutorial folder. This is partly an experiment to see how much one needs to bend over backwards to allow pseudocode where desirable and that there is no drive towards doctestablity for the sake of it (https://github.com/scipy/scipy/issues/4889#issuecomment-103282654). So far it seems reasonable, only minor whitelisting seems enough.
Travis Ci check is going to fail as there are several issues left.
Am posting this PR early to (i) gather opinions on the concept, and (ii) to see how much small details vary between my machine and Travis box.

OK, this seems to work: all tests pass for me locally, the only last failure left on Travis is somewhere in the plumbing of matplotlib and TNH does not seem related in any obvious way.
The vast majority of changes to the rst files are minor syntax errors. There are several places where I tweaked the tutorials in a slightly less trivial way, so some review would be great. Esp tutorials for spatial, stats, optimize, interpolate and sparse.csgraph could use  a look-over.

@ev-br do you want to merge this without enabling the TravisCI check, or work around the weird MPL issue?

Some further fixup in https://github.com/ev-br/scipy/pull/4

And it did fix the matplotlib failure!
https://travis-ci.org/scipy/scipy/jobs/91233454

LGMT, merged


ORC-1741: Respect decimal reader isRepeating flag

This is a clone of

#1960

This is resolved via https://github.com/apache/orc/commit/ded9d769a9f3f290d5b2ef80268487e562e64069


docs: added AI slug examples for demo to components mentioned in #4072

Closes #4072
Added AI slug examples and fixed storybook ID mismatch for FluidTextInput and FluidPasswordInput
Changelog
New

Added AI slug examples as new variants to the listed components

Changed

updated correct storybook path ID for FluidTextInput and FluidPasswordInput

@Gururajj77 this is good to merge once the merge conflict is resolved


Perform lit review for commuting patterns

Goal: Perform lit review for studying commuting patterns (especially with Call Detail Records)

Helpful paper: google.com #5

Mobile Phones in a Traffic Flow: A Geographical Perspective to Evening Rush Hour Traffic Analysis Using Call Detail Records
Summary: Uses CDR to assess composition of traffic flow during rush hour
How is data presented?

validate CDR data with traffic counter data (show line graph): Figure 3
show line/bar graph where red line is Mon-Thur traffic and blue line is Friday traffic. Bars show percent difference between (mon-thur) and (friday): Figure 4
show overall distribution against (mon-thur evening rush hour) and (fri evening rush hour): Figure 5

Notes:

This CDR data has geolocation of cell towers
Uses 2/7 anchor points of the anchor point model to classify each call activity

home anchor point: probable location of the respondent’s home
work-time location: the respondent’s probable work-time location

Interesting that they studied a single road section and they make a case for a month of behavior being sufficient for estimating normal monthly activity.
Also includes references that may be helpful for supporting the use of CDRs to estimate home and work locations.
Nice find.

Relevant paper: http://documents.worldbank.org/curated/en/122541487082260120/pdf/WPS7969.pdf

They used call data records in Guatemala to track consumption, mobility, and social patterns

Powerpoint summary:
http://helper.ipam.ucla.edu/publications/caws3/caws3_13211.pdf


fix(quote): add additional properties (#61)

Change
Fixes #61

Add those property with optional to quote.ts.

  trailingThreeMonthReturns?: number; // 16.98
  trailingThreeMonthNavReturns?: number; // 17.08
  ipoExpectedDate?: Date; // "2020-08-13",     <---- added
  newListingDate?: Date; // "2021-02-16",      <---- added

Add queto-BEKE.json
Add queto-BFLY.json

Thanks, @short-term, this is great!  Welcome aboard!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[821] - Trainee Slugs

Context

Implement a slug field to be used as the trainee id instead of the current integer-based id
Builds on the work done in this spike https://github.com/DFE-Digital/register-trainee-teachers/pull/358

Changes proposed in this pull request

Fixes broken tests
Make the slug unique and required
A couple of migrations to create the field, set the slug on current records and make the field not nullable

Guidance to review

Fire up the app locally and run through the app as normal. Assert the trainee's slug is used instead of the id in URLs.

 [M] Add slugs to trainee

 [M] Add slugs to trainee


Concurrent call to receive() is not allowed for 2 different websockets

Long story short
I write a websockets proxy. For each connection to a client exists a connection to a server. I try to use asyncio.wait for waiting for the first message either from the client or from the server. But aiohttp raises an error Concurrent call to receive() is not allowed.
Expected behaviour
Concurrent call to receive() from different websockets should be allowed.
Actual behaviour
Exception Concurrent call to receive() is not allowed
Steps to reproduce
Handler for websockets looks like this:
async def ws_handler(request):
    ws_to_client = WebSocketResponse()
    session = aiohttp.ClientSession(loop=request.app.loop)
    ws_to_server = await session.ws_connect(url, autoclose=False, autoping=False)
    ...
    while True:
        done, pending = await asyncio.wait(
            [ws_to_client.receive(), ws_to_server.receive()],
            return_when=asyncio.FIRST_COMPLETED
        )
    ...

Your environment
Python 3.5.2, aiohttp 0.22.2

This is intended behavior.
Please create a separate task for serving client web socket in your example.
See also 8d9ee8b

Thanks a lot, @asvetlov. It works!

Hi @asvetlov. I still curious, how I can handle more than 2 incoming sources? More than one task will fail with the same exception as above. Are there any other tricks for this case?

Could you elaborate?
It's really hard to answer without looking on problematic code.

Hi @asvetlov
same problem, here's my code:
`async def websocket_handler(request):
ws = web.WebSocketResponse()
await ws.prepare(request)
name = ""
while True:
    msg = await ws.receive()
    if msg.type == web.MsgType.text:
        try:
            data=eval(msg.data)
            if msg.data==1 :
                ws.send_str(2)
            elif data.get('type')=='1':
                name=data.get('uid')
                if request.app['sockets'].get(name) is not None:
                    request.app['sockets'][name].close()
                request.app['sockets'][name] = ws
                for ws in request.app['sockets'].values():
                    ws.send_str(json.dumps({'type':1,'online':online},ensure_ascii=False,default=utils.json_default))

        except Exception as e:
            print(e.args)
    elif msg.tp == web.MsgType.close:
        print('websocket connection closed')
        break
    elif msg.tp == web.MsgType.error:
        print('ws connection closed with exception %s' % ws.exception())
        break
    else:
        break

del request.app['sockets'][name]
for ws in request.app['sockets'].values():
    ws.send_str(json.dumps({'action': 'disconnect',
                            'name': name}))
return ws`

You cannot call request.app['sockets'][name].close() from foreign task.
But your traceback points on a different problem: web request should be followed by exactly one response. I believe the rule is obvious.