feat(rpc): update the type of epoch

remove block_reward, last_block_hash_in_previous_epoch, remainder_reward fields, add epoch_reward
field
BREAKING CHANGE: update the type of epoch, remove block_reward, last_block_hash_in_previous_epoch,
remainder_reward fields, add epoch_reward field.

Codecov Report

Merging #270 into develop will decrease coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #270      +/-   ##
===========================================
- Coverage    95.92%   95.91%   -0.02%
===========================================
  Files            9        9
  Lines          319      318       -1
  Branches       102      102
===========================================
- Hits           306      305       -1
  Misses           8        8
  Partials         5        5

Impacted Files
Coverage Δ

packages/ckb-sdk-rpc/src/resultFormatter.ts
91.66% <100%> (-0.1%)
: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 d4756c7...509a79b. Read the comment docs.


AP-250 A list of legal aid application with links to current positions

Jira AP-250
What

Records the current controller on each legal aid application view in the providers journey.
Adds a list of legal aid application to  provider/legal_aid_applications#index view.
Adds a link to the current journey position for each application in the list.

Notes

The goal is for the list to only be of applications belonging to the logged in provider. As authentication is not ready, the list is of the first ten legal aid applications.
I've refactored Providers::Steps so that I could use that functionality to provide the link to the current step, and to being it closer to Julien's pattern used in the Citizen journey (more work needed here, but I hope this refactor will make it easier to bring the two together - that is, use the same techniques for both journeys.)
No designs were included on the ticket - so at the moment I've used a simple design that I expect to have to change when a design becomes available:

AP-238 contains the styling of the application list - so I'll incorporate that into this PR


docs(README): add some metadata badges

Including license, Python version, package version, build status and coverange.
CodeKitchen: https://codekitchen.community/t/topic/1134/3

Merged, thanks!


Layout 3 UI Teams and Achievements Improvements

Description
A summary of the change
This changes were inspired by http://vis.pku.edu.cn/people/canliu/
Changed the styling of the layout 3's Achievements and Teams Page. This changes make the UI simpler and more appropriate for academic sites. Main changes are the alignment (now left-aligned) and removal of cards. Fonts, title for both pages along with the achievement's image were also changed.
Type of change

[ ] Bug fix
[X] New feature
[ ] Refactoring
[ ] Documentation
[X] Improvement

Problem
The layout 3 UI was nice, but could be more appropriate for academic sites.
Solution description
The changes described is the 'Description' section were based on reviewing academic sites like http://vis.pku.edu.cn/people/canliu/.
Tests
All unit tests pass? Yes
Builds successfully? Yes
Attach screenshot of the expected result (e.g. UI of the website/ postman api call result/ database changes)
Achievements Page:

Teams Page:

For reference (OLD DESIGN):

Risk
Low - Changes were made to UI only. Functionality itself was not altered.

Some work might need to be done on the styling of the dark theme later because some of the headings are unreadable:

Please take this on as a separate ticket on your board.
LGTM!


Trying to add more settings (content_css, external_plugins, etc)

I've been trying to add more settings to the TinyMCE editor, but without success. The Extension is not recognizing my parameters.
Here is my code:
`var p = tweak(api.parameters.instance.plugins);
var tb = tweak(api.parameters.instance.toolbar);
var mb = tweak(api.parameters.instance.menubar);
var sf = tweak(api.parameters.instance.style_formats);
var ep = tweak(api.parameters.instance.external_plugins);
var cc = tweak(api.parameters.instance.content_css);
var vb = tweak(api.parameters.instance.visualblocks_default_state);
var ec = tweak(api.parameters.instance.end_container_on_empty_block);
api.window.startAutoResizer();

tinymce.init({
  selector: "#editor",
  plugins: p,
  toolbar: tb,
  menubar: mb,
  style_formats: sf,
  external_plugins: ep,
  content_css: cc,
  visualblocks_default_state: vb,
  end_container_on_empty_block: ec,`

I've tried to add the external plugin (and content_css) in my forked repo and then pointing directly to that file, but TinyMCE isn't even trying to load that URL on init.
Here are my settings:
   "toolbar": "undo redo | removeformat | tiny_mce_wiris_formulaEditor | help",     "menubar": "edit insert",     "external_plugins": {         "tiny_mce_wiris": "URL to plugin"     }

Hey @orjanb-ne
I don't think using the tweak function there would work for external plugins since you will return an string and the configuration takes an object. You would probably need to parse that as a JSON string instead.


Keep GPU branch up to date

Merges all recent changes from main into gpu.

So, now we got the tests from test/kernelabstractions.jl working and it seems we directly found an error from one of the adapt_structures
@katharinamaetschke can you look at that? The CI report shows an error for SurfaceVariables

The error seems to come from calling SurfaceVariables with Vectors instead of Grids
 KernelAbstractions tests: Error During Test at /home/runner/work/SpeedyWeather.jl/SpeedyWeather.jl/test/kernelabstractions.jl:3
  Got exception outside of a @test
  MethodError: no method matching SpeedyWeather.SurfaceVariables(::Vector{Float64}, ::LowerTriangularMatrix{ComplexF64}, ::Vector{Float64}, ::Vector{Float64}, ::Int64)
  Closest candidates are:
    SpeedyWeather.SurfaceVariables(::Grid, ::LowerTriangularMatrix{Complex{NF}}, ::Grid, ::Grid, ::Int64) where {NF<:AbstractFloat, Grid<:AbstractGrid{NF}} at ~/work/SpeedyWeather.jl/SpeedyWeather.jl/src/diagnostic_variables.jl:255

This PR is really only meant to keep the branch updated, so I'll merge it now as there no merge conflicts. The error in the test, needs to be fixed in another PR.

@katharinamaetschke If you have any questions about the changes with the Grid type, don't hesitate to ask @milankl or me.


Customize the web app manifest file

See here for info on the purpose of the app manifest file.
Customize the existing manifest file to reflect the specific needs of the navi app
for a tutorial on this look to section 3 of this Udacity course

@tanyagupta where would you like me to place the additional app icon files ? The same folder as before?

@alicepalazzolo same folder is good also can you load the logo on a transparent background at the same time please 😃 and Thank you!

@tanyagupta what size would you like the logo?

@alicepalazzolo we are currently just using it for the splash/ home screen which will only display logo, maybe a tagline and two large buttons leading to map & profile.

This Pull I just added should have everything you need @tanyagupta I also added  fav icon package which also has a manifest file and set ups for safari, apple and microsoft app icons, if you want to use this let me know and i can give you instructions for html5, nod CLI or Google Web STarter Kit

Thanks a lot! Bryan hold off on merge please. I will add these and update the pr
Sent from my iPhone

On Mar 2, 2018, at 7:12 PM, Alice Palazzolo<EMAIL_ADDRESS>wrote:
This Pull I just added should have everything you need @tanyagupta I also added fav icon package which also has a manifest file and set ups for safari, apple and microsoft app icons, if you want to use this let me know and i can give you instructions for html5, nod CLI or Google Web STarter Kit
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Extension host crashed with OOM

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T17:16:27.618Z)
OS Version: Linux x64 4.12.8-2-ARCH
Extensions:

Extension
Author (truncated)
Version

stylus
Ala
0.0.4

toml
be5
0.0.3

python
don
0.7.0

vscode-rust
kal
0.4.2

Go
luk
0.6.63

cpptools
ms-
0.12.3

java
red
0.9.0

rust
rus
0.2.2

code-spell-checker
str
1.3.9

gitblame
wad
2.1.0

Steps to Reproduce:
Happens randomly, low reproduce rate.

Reproduces without extensions: Unknown
log:
/opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356 [1] - Grammar is in an endless loop - Grammar pushed & popped a rule without advancing
scanNext @ /opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29 [Extension Host] (node:2302) DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
t.logExtensionHostMessage @ /opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29
/opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356 [1] - Grammar is in an endless loop - Grammar pushed & popped a rule without advancing
scanNext @ /opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356
/opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356 [1] - Grammar is in an endless loop - Grammar pushed & popped a rule without advancing
scanNext @ /opt/visual-studio-code/resources/app/node_modules/vscode-textmate/release/main.js:2356
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29 Extension Host
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29 FATAL ERROR: CALL_AND_RETRY_LAST Allocation failed - JavaScript heap out of memory
 1: node::Abort() [/opt/visual-studio-code/libnode.so]
 2: 0x7f9394daf1b7 [/opt/visual-studio-code/libnode.so]
 3: 0x7f9394b0aa87 [/opt/visual-studio-code/libnode.so]
 4: 0x7f9394b0aa05 [/opt/visual-studio-code/libnode.so]
 5: v8::internal::Factory::NewTransitionArray(int) [/opt/visual-studio-code/libnode.so]
 6: 0x7f93945b049a [/opt/visual-studio-code/libnode.so]
 7: 0x7f939471f4a4 [/opt/visual-studio-code/libnode.so]
 8: 0x7f939471c36e [/opt/visual-studio-code/libnode.so]
 9: 0x7f939471c138 [/opt/visual-studio-code/libnode.so]
10: 0x7f939473541f [/opt/visual-studio-code/libnode.so]
11: v8::internal::LookupIterator::PrepareTransitionToDataProperty(v8::internal::Handle<v8::internal::JSObject>, v8::internal::Handle<v8::internal::Object>, v8::internal::PropertyAttributes, v8::internal::Object::StoreFromKeyed) [/opt/visual-studio-code/libnode.so]
12: 0x7f9394724834 [/opt/visual-studio-code/libnode.so]
13: 0x7f9394727eb8 [/opt/visual-studio-code/libnode.so]
14: 0x7f939472c504 [/opt/visual-studio-code/libnode.so]
15: 0x7f939472bc21 [/opt/visual-studio-code/libnode.so]
16: 0x7f939472abc7 [/opt/visual-studio-code/libnode.so]
17: 0x7f939468a9e5 [/opt/visual-studio-code/libnode.so]
18: 0x312286f04167

<--- Last few GCs --->

[2302:0x12a12aac8000]  2305088 ms: Mark-sweep 1404.8 (1480.6) -> 1404.8 (1479.6) MB, 1688.7 / 0.0 ms  (+ 0.0 ms in 0 steps since start of marking, biggest step 0.0 ms, walltime since start of marking 1689 ms) last resort
[2302:0x12a12aac8000]  2306771 ms: Mark-sweep 1404.8 (1479.6) -> 1404.8 (1479.6) MB, 1682.3 / 0.0 ms  last resort

<--- JS stacktrace --->

==== JS stack trace =========================================

Security context: 0x29dc103af729 <JS Object>
    0: builtin exit frame: defineProperty(aka defineProperty)(this=0x3c29fca82311 <undefined>,0x42bd7645119 <an Object with map 0x38cfdc391c99>,0x3c29fca83019 <String[4]: hour>,0x42bd76450e1 <an Object with map 0x38cfdc383179>)

    1: toDateTimeOptions(aka toDateTimeOptions) [native i18n.js:~912] [pc=0x312287668859](this=0x3c29fca82311 <undefined>,S=0x3c29fca82311 <undefined>,bN=0x29dc103c8741...

index.js:5 [uncaught exception]: Error: read ECONNRESET
onError @ index.js:5
index.js:5 Error: read ECONNRESET
    at exports._errnoException (util.js:1022:11)
    at Pipe.onread (net.js:572:26)
onError @ index.js:5
events.js:160 Uncaught Error: read ECONNRESET
    at exports._errnoException (util.js:1022:11)
    at Pipe.onread (net.js:572:26)
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:28 Extension host terminated unexpectedly. Please reload the window to recover.
e.doShow @ /opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:28
/opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29 Extension host terminated unexpectedly. Code:  null  Signal:  SIGABRT
t.onExit @ /opt/visual-studio-code/resources/app/out/vs/workbench/workbench.main.js:29

@pftbest This appears to be a crash caused by the extension host process running out of memory. The most likely cause for this would be an extension with memory leaks. What extensions are you using, have you found any reproducible steps? (perhaps working with large files, etc.) ?

I was working with rust extension at the time of crash.
But I've not seen this crash anymore, so maybe they fixed it.

I will close this issue for now, as there is no code change I can do at this time to avoid OOM crashes... If you spot it again, and can give some steps, we can try to reproduce and investigate the root cause.


Add Tcl, libffi, Yosys, Project Trellis, Boost, Eigen and Nextpnr

This should be all external dependencies I will add for now.
Project Trellis uses C++17 so I added a .bazelrc that configures the whole project to use that.

I found that Project Trellis in this PR doesn't work on my workstation, I'll close the PR and try to fix


ir channel windows

Hi,
I have realsense camera d415 with latest drivers on windows 10. I'm using python wrapper.
I installed latest pyrealsense2, but it doesn't provide an ir channel. Also viewer from the sdk doesn't show ir channel.
However I can acquire ir channel in ubuntu 18 with the same pyrealsense2.
Can I acquire ir channel in windows?

Hi @dodler   Do you mean that you have selected an IR channel but it does not display, showing an error in the infrared panel such as "No frames received" please?

sorry, i didn't clarified this,
i'm using code similar to this one
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/opencv_viewer_example.py
but I also add rs.stream.infrared
but it fails on
pipeline.start(config)
with runtime error - cannot resolve request

Cannot resolve request indicates that the SDK has been asked to do something that either it does not understand (e.g because of an error in the code) or because the requested mode is not available (e.g because the camera is currently being detected - or misdetected- as being on a USB 2 connection).
Does your IR stream definition statement look something like this:
config.enable_stream(rs.stream.infrared, 1, 640, 480, rs.format.y8, 30)
'1' is put after rs.stream.infrared to inform the SDK that you are requesting the left IR stream.  If you were requesting both the left and right IR streams, the code would look something like this:
config.enable_stream(rs.stream.infrared, 1, 640, 480, rs.format.y8, 30)
config.enable_stream(rs.stream.infrared, 2, 640, 480, rs.format.y8, 30)

actually i'm not specifying exact frame, i'm just using like this
config.enable_stream(rs.stream.infrared, 640,480,rs.format.y8,30)
I also tried your options, but still i'm getting the same error

I looked again at opencv_viewer_example.py and wondered how closely your project was based on that script, because I noticed that it seems to wait for a pair of depth and color frames before it is allowed to proceed.  So i wonder if adding in an infrared stream would disrupt that checking process just after the pipeline starts.
https://github.com/IntelRealSense/librealsense/blob/master/wrappers/python/examples/opencv_viewer_example.py#L28
It may be worth testing the script in the link below that uses all three streams (depth, color, IR) and see whether the same problem with  cannot resolve request occurs:
https://github.com/IntelRealSense/librealsense/issues/6334

Hi @dodler   Do you still require assistance with this case, please?  Thanks!

Hi, problem is not still solved, I will test on another machine and report later
thanks

Hi, problem is not still solved, I will test on another machine and report later
thanks

Thanks very much for the update - I look forward to your report later about testing on another machine.

Hi @dodler  Do you have an update please regarding testing your project on another machine?   Thanks!

Hi, no updates yet, i'll close the issue

Thanks very much for the update @dodler


Incompatible with Protocolize

[00:51:01 ERROR] [VelocityScoreboardAPI]: --------------------------------------------------------------------------------
[00:51:01 ERROR] [VelocityScoreboardAPI]: Failed to register scoreboard packets. This most likely means you are using another plugin that already registered scoreboard packets. Error message:
[00:51:01 ERROR] [VelocityScoreboardAPI]: java.lang.IllegalArgumentException: Can not register class ScoreSetPacket with id 104 for 1.21.2 because another packet is already registered
[00:51:01 ERROR] [VelocityScoreboardAPI]: --------------------------------------------------------------------------------

full log: https://mclo.gs/RPSsE2F
After remove Protocolize, VelocityScoreboardAPI works.

https://mclo.gs/CxIQm2M
clean log with only VelocityScoreboardAPI, TAB and Protocolize.

What kind of solution do you want me to implement on my side? The plugin cannot work without registering the packets, which don't exist on velocity by default.

https://github.com/Exceptionflug/protocolize-data/blob/4262c710e2d5a60eb627fa50ef8b5f4a18c881a5/protocolize-data-bundle/src/main/java/dev/simplix/protocolize/data/packets/SoundEffect.java#L58C1-L58C90
- AbstractProtocolMapping.rangedIdMapping(MINECRAFT_1_20_5, MINECRAFT_LATEST, 0x68)
+ AbstractProtocolMapping.rangedIdMapping(MINECRAFT_1_20_5, MINECRAFT_1_20_5, 0x68)

After limiting Protocolize's protocol version, I made VelocityScoreboardAPI work.
Perhaps Protocolize used the wrong ID.
Okay, it's not the VelocityScoreboard API's wrong.

It really looks like the ID is wrong. The packet is called sound, but since it caused a conflict, it isn't supposed to be. I guess they didn't update the ID properly.


"add-response-headers" property does not seem to be working

The ratelimit headers do get added to the response by default, but when i try to remove them using the "add-response-headers" property they are still in the response.

If you are using Spring Boot version 1.5.x you MUST use Spring Cloud Zuul RateLimit version 1.7.x.

I picked up on this, so i am using ratelimit version 1.7.3.RELEASE
my current config looks like this.
ratelimit:
    enabled: true
    add-response-headers: false
    repository: JPA
    policy-list:
      service-x:
        - limit: 60
          refresh-interval: 10

Hi @devc0n thanks for your finding!
I just checked it out and it indeed is a bug, actually, the feature is only present in the 2.x series.
I'll sync it to the 1.7.x as well.

Hey @marcosbarbero, I have just updated the library to 1.7.4 and build my application again. but i am afraid the headers are still in there with the provided config.

Hey @devc0n it will be available on the 1.7.5.RELEASE, it was already published to the maven central but it takes a few hours to be available.

Just an update for you @devc0n, there seems to be an issue to deploy to the maven central, I'll try to fix it and deploy maybe later today.

Thanx for the update! I'll be checking later today for the newer version. thanx in advance!

@devc0n it took longer than I expected but I finally managed to fix and deploy to maven central the version 1.7.5.RELEASE.


refactor(whiskers): title

A few notes

Move to standard names or hardcode thm_... (https://github.com/catppuccin/community/blob/main/rfc/CAT-0003-Whiskers.md#variables)

What does thm stand for?

Change theme format so we can just source it instead of parsing?
Move themes to a separate folder

Currently only 3 themes (the main flavours) but can be more in the future (accents)

https://github.com/catppuccin/community/discussions/15

See #207


Store ArrayMesh path in RenderingServer for use in error messages

Fixes: https://github.com/godotengine/godot/issues/84875
This only works for ArrayMeshes and not for custom meshes created through the RenderingServer API (unless the uses sets the path manually).
Its the same approach we have used for textures and shaders, so it is nice to keep the API consistent.

Thanks!

I wonder that if no file path is available, we could also check if the user has set a resource name. I know its not common for users to set that, but in case they do, it'll atleast give them a hint?

That makes sense to me, but I assume it also needs to be done to textures and shaders, so I think we can do this as a follow-up.

As of 4.2.2 RC 1, the issue of "Attempting to use a shader that requires tangents with a mesh that doesn't contain tangents." being  spammed is still presently HEAVILY. Literally 1000s of these warning messages.

That's expected, this PR was merged two weeks ago in the master branch (4.3.dev) and hasn't been cherrypicked for the 4.2 branch yet.

Cherry-picked for 4.2.2.


Release 2.1.145

On the docket:

[ ] pex does not use .pip/pip.conf to resolve packages #336

Broken off of #2212 to make way for #2220.


Spamming "Invalid Url for Button1."

Issue Summary:
Current UI assumes all projects are Git repositories, causing annoyance for non-git projects.
Request:
Add a "Don't show again" option or similar feature.

Hey @Walkaisa,
There is a specific button config that is detected for git repos and it uses that. You should be able to configure non git repos separately / disable buttons on them if you wish.
Relevant settings
https://gyazo.com/5560170aad2b9529f6ea97f3b69ca9b7


Remove experimental_json_serialization from tests.

The parameter was removed.  Just purging it from our tests.

The parameter was removed.  Just purging it from our tests.


Fix conversion to DataFrame for dict input in custom inference Model

The Model.predict() method supports pd.DataFrame, list and dict inputs. However, there was a tiny mistake in handling dict inputs, which need to be converted to list of dicts before they can be used to construct a pd.DataFrame.
Happy to provide a unit test for that, but since it is part of the homework to come up with additional unit tests, I haven't done it yet. Please do let me know if you want me to include it in the PR.

This is a very good contribution. Thanks! Do you mind adding a unit test for this?

@svpino Modified the existing unit test to cover all supported input formats. This test should now fail in main branch and pass in the fix branch.
Also updated the type hints in Model.predict() method to make explicit what input formats are supported.
Let me know if you'd like to see anything else added, thanks!


ballerina help pull gives the help text of the dist pull command

Description:
ballerina help pull or ballerina pull --help
Steps to reproduce:
type the above commands and see
Affected Versions:
ballerina-1.0.0-beta

Duplicate #18464


Gulp watch

It was impossible for me to change main.css file, every time it would produce an error in console, and couldn't load main.css again.
Also gulp would displayed a message saying that I have changed my constants .config file, although I didn't changed it.

Same here. Any solution?

I will need a lot more information on your problem and especially the error that you run in to, as well as a repository that I can clone that will produce this error, with a detailed description on how to reproduce this.

I just downloaded this repo, put some class into main.css file to test it, and saved file. On reload main main.css could not be loaded, only ionic.sccs files where included.

You'll need to be a bit more collaborative here so I can help you. There's many things that could go wrong.

did you run yo m-ionic or did you clone the demo repository?
which options did you choose during setup?
which os and version are you on?
which version of node/npm/yo/gulp are you running?
what's the error in the console?

Hello! I have the same error here, I checked console permission (I'm on windows) but happends every time I save any scss file.
First, it runs the build several times, and then crash:
[13:04:22] Starting 'styles'...
events.js:141
      throw er; // Unhandled 'error' event
      ^

TypeError: must start with number, buffer, array or string
    at fromObject (buffer.js:167:9)
    at new Buffer (buffer.js:58:10)
    at DestroyableTransform._transform (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\wiredep.js:136:23)
    at DestroyableTransform.Transform._read (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\node_modules\readable-stream\lib\_stream_transform.js:159:10)
    at DestroyableTransform.Transform._write (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\node_modules\readable-stream\lib\_stream_transform.js:147:83)
    at doWrite (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\node_modules\readable-stream\lib\_stream_writable.js:313:64)
    at writeOrBuffer (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\node_modules\readable-stream\lib\_stream_writable.js:302:5)
    at DestroyableTransform.Writable.write (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\wiredep\node_modules\readable-stream\lib\_stream_writable.js:241:11)
    at write (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:623:24)
    at flow (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:632:7)
    at DestroyableTransform.pipeOnReadable (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:664:5)
    at emitNone (events.js:67:13)
    at DestroyableTransform.emit (events.js:166:7)
    at emitReadable_ (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:448:10)
    at emitReadable (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:444:5)
    at readableAddChunk (D:\DevTools\xampp\htdocs\Puertas_Mag\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:187:9)

Thanks!

Hi @diegoalmesp, I'm not able to test on windows so there's not much I can do. The problem probably lies with one of the node modules in combination with windows.
I'd google for the error messages and see if you find something similar also I'd clone the demo repository and see if you run into the same problem.
Please report back with any findings, so other can benefit as well :)

Hey, same problem here. It disappers when gulp watch is started again.
Let me know what else I can do for you. Would really like to see this solved. I will continue to look for problem.

Hey, just reporting in that I have the same issue.  Created a clean project, ran yo m-ionic.  Picked sidebar, included everything, picked Sass options.  As soon as I modify the main.scss, it deletes the main.css from .tmp\main\styles and doesn't recreate it.
This happens on my existing project, and a clean project just built.
Is there any information i can provide to help you guys pinpoint the issue?

Hi everyone, are you all working on windows?
For everything else please refer to our Issue Guidelines and I'm especially interested in your system setup (OS & version, node, npm, gulp, ...). Find the full list and information on how to obtain them in the guide.

Hello,
I will try be more helpful this time :D This is the steps I took and the versions I am using
Before I did any of these steps, I ran npm cache clean and bower cache clean.
I created a brand new app running yo m-ionic in an empty directory.

Picked Ionic SASS
Included all bower packages
Included IOS and Android
Selected all Cordova plugins
Side Menu Starter Template
Added Ionic Platform ecosystem only

I run gulp watch
Whilst gulp watch is running I then I modified the main.scss (just changed a colour) and the CSS all disappears.  The main.css is deleted from the .tmp\main\styles and not recreated.    This only happens if main.scss is modified when gulp watch is running.
Versions
OS : Win 7 Professional - Service Pack 1
Node: v5.5.0
NPM: v3.9.0
M-Ionic : 1.8.0
Cordova: 6.1.1
Is there anything else I can provide?
Thanks

Exacly same steps, enviroment and problem expect for OS. Running Windows 10 Pro, all latest updates.

Thank you @durasj and @cherylhanlon!
The error output is the same as for @diegoalmesp?
Windows issues are hard for me to work on since I don't have a windows machine available.
You can try to fix it yourself by playing around with the clean task: https://github.com/mwaylabs/generator-m-ionic-demo/blob/master/gulp/building.js#L24-L28
Maybe this has to do with this commit? https://github.com/mwaylabs/generator-m-ionic/commit/c7551ab8a40a3da9e4573249f094b43ee291576c What happens if you revert the changes?
What happens when you remove the clean tasks as a dependency to the styles task? https://github.com/mwaylabs/generator-m-ionic-demo/blob/master/gulp/injecting.js#L36

If I remove the clean task  dependency from styles, it works fine.  Thanks for that.
I assume the only negative impact of this would be if I deleted a scss file, it wouldn't be removed from the tmp folder or from injected includes lines (see below) until gulp watch is ran again?
<!-- inject:css -->
<link rel="stylesheet" href="main/styles/main.css">
<link rel="stylesheet" href="main/styles/mydeleted.css">
<!-- endinject -->

@gruppjo sorry I didn't answer before! if I re-start gulp watch everything works fine, the problem occurs every time I save any scss file. My problem seems to be the same as @cherylhanlon . For now I'm not using watch while doing scss.

@diegoalmesp, thanks for the information
@cherylhanlon, have you tried reverting this commit? https://github.com/mwaylabs/generator-m-ionic/commit/c7551ab8a40a3da9e4573249f094b43ee291576c thus replacing the current clean task with this one?
gulp.task('clean', function () {
  return gulp.src(['.tmp', paths.dist + '/*'])
     .pipe(vinylPaths(del));
});

If I leave the clean task dependency in and revert the commit, when I modify the scss file i get the following error
[BS] Serving files from: app
[BS] Serving files from: .tmp
[14:15:59] Starting 'clean'...
events.js:154
      throw er; // Unhandled 'error' event
      ^

Error: watch null EPERM
    at exports._errnoException (util.js:856:11)
    at FSEvent.FSWatcher._handle.onchange (fs.js:1296:21)```

I'm using a Mac and I have this error too. Previous version works good.

This might be a solution. Can anyone test? https://github.com/mwaylabs/generator-m-ionic/issues/317#issuecomment-222931884


New release

Hi,
I was wondering when the next release is planned.
One of the features that I created a pr for is merged to the existing codebase.
When the new release is availible we can update our build pipeline with the new swashbuckle tool.
Timmy

Indeed, it would be great to have a estimated release date - currently we are holding back the upgrade to 3.1 until 5.0 is released 👍

The only remaining issue before 5.0 is this one.
Can we hope to be very close to a next exit?
I really need for a .Net Core 3.x version!

Finally ... 5.0.0 has just been shipped


Fix json issue context_rules

Added to_json() to context_rule.py
Added test_to_json() to test_context_rule.py
Fixed error in to_dict() to return a list instead of a set

@abchapman93

@abchapman93

Thanks for making this pull request @hannapethani ! Before I run the tests, could you change the pull request so it's merging into the develop branch? Our general workflow is to merge various branches into develop and then to done one big PR to master once we're ready for a release..

Oh actually I was able to change it to mergei into develop.
Looks good to me - @turbosheep not sure of the current procedure for merging pull requests, but Hanna and I worked on this together and everything looks good and ready to be merged.


Check WebService Hosting

Where to host our created service Servlets/WS

We can host our webservices, html pages on Google platform drive.
Check the following link http://workflowcnam2017.appspot.com/
Built by using https://cloud.google.com/appengine/docs/standard/java/quickstart


Download link for pyrosetta is dead

Currently, the download link for pyrosetta is hard-corded as
https://graylab.jhu.edu/download/PyRosetta4/archive/release/PyRosetta4.MinSizeRel.python37.linux/PyRosetta4.MinSizeRel.python37.linux.release-316.tar.bz2
but this link is now dead.
So we need to replace this to the one available today such as
https://graylab.jhu.edu/download/PyRosetta4/archive/release/PyRosetta4.MinSizeRel.python37.linux/PyRosetta4.MinSizeRel.python37.linux.release-328.tar.bz2

Thank you very much. I have corrected it as per your advice.


Membership (and presence) tests

What?
Contains all of the membership tests and presence tests (which can be considered reviewed and 👍-ed).
Why?
Tests are good

CC @hamchapman

          1 Warning

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there. \nYou can find it at CHANGELOG.md.

SwiftLint found issues
Warnings

File
Line
Reason

ViewController.swift
108
Line should be 120 characters or less: currently 123 characters

ViewController.swift
127
Line should be 120 characters or less: currently 143 characters

ViewController.swift
132
Line should be 120 characters or less: currently 123 characters

ViewController.swift
137
Line should be 120 characters or less: currently 123 characters

ViewController.swift
150
Line should be 120 characters or less: currently 124 characters

ViewController.swift
155
Line should be 120 characters or less: currently 124 characters

ViewController.swift
5
Delegates should be weak to avoid reference cycles.

ViewController.swift
7
Delegates should be weak to avoid reference cycles.

ChatManager.swift
72
TODOs should be avoided (When can percent encoding fail...).

ChatManager.swift
90
TODOs should be avoided (This could be nicer).

ChatManager.swift
91
TODOs should be avoided (We don't need to wait for init...).

ChatManager.swift
92
TODOs should be avoided (Do we need to nil out subscrip...).

ChatManager.swift
105
TODOs should be avoided (This is madness).

ChatManager.swift
199
TODOs should be avoided (This being here at the end see...).

ChatManager.swift
203
TODOs should be avoided (Maybe we need some sort of Cha...).

ChatManager.swift
251
TODOs should be avoided (This could be a lot more effic...).

ChatManager.swift
76
Function body should span 40 lines or less excluding comments and whitespace: currently spans 76 lines

ChatManager.swift
94
Variable name should be between 3 and 40 characters long: 'cu'

ChatManager.swift
170
Line should be 120 characters or less: currently 130 characters

ChatManager.swift
180
Line should be 120 characters or less: currently 129 characters

ChatManager.swift
184
Line should be 120 characters or less: currently 126 characters

ChatManager.swift
249
Line should be 120 characters or less: currently 139 characters

ChatManager.swift
253
Line should be 120 characters or less: currently 123 characters

ChatManager.swift
264
Line should be 120 characters or less: currently 122 characters

ChatManager.swift
272
Line should be 120 characters or less: currently 128 characters

ChatManager.swift
285
Line should be 120 characters or less: currently 125 characters

PCBasicCurrentUser.swift
78
TODOs should be avoided (Decide what to do with onEnd).

PCBasicCurrentUser.swift
157
TODOs should be avoided (Should the connection coordina...).

PCBasicCurrentUser.swift
5
Variable name should be between 3 and 40 characters long: 'id'

PCBasicCurrentUser.swift
24
Variable name should be between 3 and 40 characters long: 'id'

PCChatManagerDelegate.swift
18
TODOs should be avoided (Is this the best way of commun...).

PCConnectionCoordinator.swift
96
TODOs should be avoided (Sourcery should be used for al...).

PCConnectionCoordinator.swift
162
Operators should be surrounded by a single whitespace when defining them.

PCConnectionCoordinator.swift
205
Operators should be surrounded by a single whitespace when defining them.

PCConnectionCoordinator.swift
194
Arguments can be omitted when matching enums with associated types if they are not used.

PCConnectionCoordinator.swift
196
Arguments can be omitted when matching enums with associated types if they are not used.

PCConnectionCoordinator.swift
198
Arguments can be omitted when matching enums with associated types if they are not used.

PCConnectionCoordinator.swift
200
Arguments can be omitted when matching enums with associated types if they are not used.

PCConnectionCoordinator.swift
20
Line should be 120 characters or less: currently 137 characters

PCConnectionCoordinator.swift
40
Line should be 120 characters or less: currently 128 characters

PCConnectionCoordinator.swift
111
Line should be 120 characters or less: currently 143 characters

PCConnectionCoordinator.swift
115
Line should be 120 characters or less: currently 135 characters

PCConnectionCoordinator.swift
123
Line should be 120 characters or less: currently 122 characters

PCConnectionCoordinator.swift
127
Line should be 120 characters or less: currently 139 characters

PCConnectionCoordinator.swift
131
Line should be 120 characters or less: currently 133 characters

PCCurrentUser.swift
16
TODOs should be avoided (This should probably be [PCUse...).

PCCurrentUser.swift
368
TODOs should be avoided (Use the soon-to-be-created new...).

PCCurrentUser.swift
456
TODOs should be avoided (Do we need to fetch users in t...).

PCCurrentUser.swift
725
TODOs should be avoided (Do I need to add a Last-Event-...).

PCCurrentUser.swift
766
TODOs should be avoided (What happens if you provide bo...).

PCCurrentUser.swift
787
TODOs should be avoided (This should probably be a room...).

PCCurrentUser.swift
799
TODOs should be avoided (Should we be handling onError ...).

PCCurrentUser.swift
825
TODOs should be avoided (Only consider the room subscri...).

PCCurrentUser.swift
836
TODOs should be avoided (Should we be handling onError ...).

PCCurrentUser.swift
824
Unused parameter "err" in a closure should be replaced with _.

PCCurrentUser.swift
976
Unused parameter "data" in a closure should be replaced with _.

PCCurrentUser.swift
842
Function body should span 40 lines or less excluding comments and whitespace: currently spans 52 lines

PCCurrentUser.swift
658
Lines should not have trailing whitespace.

PCCurrentUser.swift
187
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
246
MARK comment should be in valid format. e.g. '// MARK: ...' or '// MARK: - ...'

PCCurrentUser.swift
660
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
660
Arguments can be omitted when matching enums with associated types if they are not used.

PCCurrentUser.swift
5
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
56
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
157
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
174
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
208
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
263
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
323
Variable name should be between 3 and 40 characters long: 'id'

PCCurrentUser.swift
1005
Enum element name should be between 3 and 40 characters long: 'messageIdKeyMissingInMessageCreationResponse'

PCCurrentUser.swift
104
Collection literals should not have trailing commas.

PCCurrentUser.swift
772
Collection literals should not have trailing commas.

PCCurrentUser.swift
710
Opening braces should be preceded by a single space and on the same line as the declaration.

PCCurrentUser.swift
178
Line should be 120 characters or less: currently 122 characters

PCCurrentUser.swift
196
Line should be 120 characters or less: currently 144 characters

PCCurrentUser.swift
208
Line should be 120 characters or less: currently 137 characters

PCCurrentUser.swift
212
Line should be 120 characters or less: currently 134 characters

PCCurrentUser.swift
510
Line should be 120 characters or less: currently 127 characters

PCCurrentUser.swift
569
Line should be 120 characters or less: currently 121 characters

PCCurrentUser.swift
594
Line should be 120 characters or less: currently 139 characters

PCCurrentUser.swift
598
Line should be 120 characters or less: currently 139 characters

PCCurrentUser.swift
620
Line should be 120 characters or less: currently 133 characters

PCCurrentUser.swift
631
Line should be 120 characters or less: currently 132 characters

PCCurrentUser.swift
695
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
880
Line should be 120 characters or less: currently 123 characters

PCCurrentUser.swift
981
Line should be 120 characters or less: currently 136 characters

PCPresenceSubscription.swift
6
TODOs should be avoided (Do we need to be careful of re...).

PCPresenceSubscription.swift
105
TODOs should be avoided (Do we need [weak self] here?).

PCPresenceSubscription.swift
156
TODOs should be avoided (Could check if any room is act...).

PCPresenceSubscription.swift
176
TODOs should be avoided (So much duplication).

PCPresenceSubscription.swift
205
TODOs should be avoided (So much duplication).

PCPresenceSubscription.swift
43
Line should be 120 characters or less: currently 121 characters

PCPresenceSubscription.swift
79
Line should be 120 characters or less: currently 132 characters

PCPresenceSubscription.swift
101
Line should be 120 characters or less: currently 122 characters

PCPresenceSubscription.swift
116
Line should be 120 characters or less: currently 134 characters

PCPresenceSubscription.swift
120
Line should be 120 characters or less: currently 124 characters

PCPresenceSubscription.swift
132
Line should be 120 characters or less: currently 131 characters

PCPresenceSubscription.swift
177
Line should be 120 characters or less: currently 142 characters

PCPresenceSubscription.swift
210
Line should be 120 characters or less: currently 141 characters

PCPresenceSubscription.swift
13
Delegates should be weak to avoid reference cycles.

PCRoomSubscription.swift
22
Lines should not have trailing whitespace.

PCRoomSubscription.swift
7
Delegates should be weak to avoid reference cycles.

PCUserSubscription.swift
6
TODOs should be avoided (Do we need to be careful of re...).

PCUserSubscription.swift
59
TODOs should be avoided (Decide if we even need this in...).

PCUserSubscription.swift
169
TODOs should be avoided (Use the soon-to-be-created new...).

PCUserSubscription.swift
322
TODOs should be avoided (Why not weak self here?).

PCUserSubscription.swift
387
TODOs should be avoided (Why not weak self here?).

PCUserSubscription.swift
446
TODOs should be avoided (Why not weak self here?).

PCUserSubscription.swift
491
TODOs should be avoided (Why not weak self here?).

PCUserSubscription.swift
48
Function should have complexity 10 or less: currently complexity equals 15

PCUserSubscription.swift
358
Function body should span 40 lines or less excluding comments and whitespace: currently spans 42 lines

PCUserSubscription.swift
83
Line should be 120 characters or less: currently 122 characters

PCUserSubscription.swift
121
Line should be 120 characters or less: currently 127 characters

PCUserSubscription.swift
223
Line should be 120 characters or less: currently 162 characters

PCUserSubscription.swift
284
Line should be 120 characters or less: currently 137 characters

PCUserSubscription.swift
326
Line should be 120 characters or less: currently 169 characters

PCUserSubscription.swift
341
Line should be 120 characters or less: currently 173 characters

PCUserSubscription.swift
391
Line should be 120 characters or less: currently 167 characters

PCUserSubscription.swift
406
Line should be 120 characters or less: currently 171 characters

PCUserSubscription.swift
468
Line should be 120 characters or less: currently 125 characters

PCUserSubscription.swift
513
Line should be 120 characters or less: currently 125 characters

PCUserSubscription.swift
14
Delegates should be weak to avoid reference cycles.

PCUserSubscription.swift
590
File should contain 400 lines or less: currently contains 590

CurrentUserTests.swift
19
Function body should span 40 lines or less excluding comments and whitespace: currently spans 80 lines

CurrentUserTests.swift
113
Variable name should be between 3 and 40 characters long: 'ex'

CursorTests.swift
50
TODOs should be avoided (the following should really wa...).

CursorTests.swift
97
TODOs should be avoided (hook for setting own read curs...).

CursorTests.swift
130
TODOs should be avoided (remove once we can wait on the...).

CursorTests.swift
12
Function body should span 40 lines or less excluding comments and whitespace: currently spans 41 lines

CursorTests.swift
29
When using trailing closures, empty parentheses should be avoided after the method call.

CursorTests.swift
23
When using trailing closures, empty parentheses should be avoided after the method call.

CursorTests.swift
100
Variable name should be between 3 and 40 characters long: 'ex'

CursorTests.swift
116
Variable name should be between 3 and 40 characters long: 'ex'

CursorTests.swift
140
Variable name should be between 3 and 40 characters long: 'ex'

CursorTests.swift
161
Variable name should be between 3 and 40 characters long: 'ex'

CursorTests.swift
146
Prefer _ = foo() over let _ = foo() when discarding a result from a function.

Helpers.swift
52
Unused parameter "data" in a closure should be replaced with _.

Helpers.swift
52
Unused parameter "response" in a closure should be replaced with _.

Helpers.swift
68
Unused parameter "data" in a closure should be replaced with _.

Helpers.swift
68
Unused parameter "response" in a closure should be replaced with _.

Helpers.swift
141
Unused parameter "data" in a closure should be replaced with _.

Helpers.swift
141
Unused parameter "response" in a closure should be replaced with _.

Helpers.swift
175
Unused parameter "data" in a closure should be replaced with _.

Helpers.swift
175
Unused parameter "response" in a closure should be replaced with _.

Helpers.swift
17
Variable name should be between 3 and 40 characters long: 'id'

Helpers.swift
194
Line should be 120 characters or less: currently 133 characters

TokenGenerator.swift
358
Colons should be next to the identifier when specifying a type and next to the key in dictionary literals.

TokenGenerator.swift
216
Line should be 120 characters or less: currently 189 characters

TokenGenerator.swift
220
Line should be 120 characters or less: currently 179 characters

TokenGenerator.swift
224
Line should be 120 characters or less: currently 176 characters

TokenGenerator.swift
296
Line should be 120 characters or less: currently 159 characters

TokenGenerator.swift
445
Line should be 120 characters or less: currently 141 characters

TokenGenerator.swift
472
File should contain 400 lines or less: currently contains 472

TypingIndicatorTests.swift
49
TODOs should be avoided (the following should really wa...).

TypingIndicatorTests.swift
156
TODOs should be avoided (remove once we can wait on the...).

TypingIndicatorTests.swift
110
Unused parameter "_alice" in a closure should be replaced with _.

TypingIndicatorTests.swift
28
When using trailing closures, empty parentheses should be avoided after the method call.

TypingIndicatorTests.swift
22
When using trailing closures, empty parentheses should be avoided after the method call.

UserSubscriptionTests.swift
21
When using trailing closures, empty parentheses should be avoided after the method call.

UserSubscriptionTests.swift
15
When using trailing closures, empty parentheses should be avoided after the method call.

UserSubscriptionTests.swift
54
Variable name should be between 3 and 40 characters long: 'ex'

TestingDelegates.swift
37
Returning Void in a function declaration is redundant.

TestingDelegates.swift
41
Returning Void in a function declaration is redundant.

TestingDelegates.swift
45
Returning Void in a function declaration is redundant.

TestingDelegates.swift
49
Returning Void in a function declaration is redundant.

TestingDelegates.swift
101
Returning Void in a function declaration is redundant.

TestingDelegates.swift
105
Returning Void in a function declaration is redundant.

TestingDelegates.swift
109
Returning Void in a function declaration is redundant.

TestingDelegates.swift
113
Returning Void in a function declaration is redundant.

TestingDelegates.swift
117
Returning Void in a function declaration is redundant.

PresenceTests.swift
49
TODOs should be avoided (the following should really wa...).

PresenceTests.swift
64
TODOs should be avoided (this shouldn't be necessary.).

PresenceTests.swift
82
FIXMEs should be avoided (this is a disgrace).

PresenceTests.swift
91
TODOs should be avoided (this shouldn't be necessary.).

PresenceTests.swift
117
FIXMEs should be avoided (this is a disgrace).

PresenceTests.swift
126
TODOs should be avoided (this shouldn't be necessary.).

PresenceTests.swift
147
TODOs should be avoided (remove once we can wait on the...).

PresenceTests.swift
28
When using trailing closures, empty parentheses should be avoided after the method call.

PresenceTests.swift
22
When using trailing closures, empty parentheses should be avoided after the method call.

RoomMembershipTests.swift
409
TODOs should be avoided (remove once we can wait on the...).

RoomMembershipTests.swift
456
TODOs should be avoided (remove once we can wait on the...).

RoomMembershipTests.swift
90
Unused parameter "room" in a closure should be replaced with _.

RoomMembershipTests.swift
160
Unused parameter "room" in a closure should be replaced with _.

RoomMembershipTests.swift
192
Unused parameter "room" in a closure should be replaced with _.

RoomMembershipTests.swift
182
Unused parameter "alice" in a closure should be replaced with _.

RoomMembershipTests.swift
218
Unused parameter "alice" in a closure should be replaced with _.

RoomMembershipTests.swift
411
Unused parameter "room" in a closure should be replaced with _.

RoomMembershipTests.swift
27
When using trailing closures, empty parentheses should be avoided after the method call.

RoomMembershipTests.swift
21
When using trailing closures, empty parentheses should be avoided after the method call.

RoomMembershipTests.swift
5
Type body should span 200 lines or less excluding comments and whitespace: currently spans 287 lines

RoomMembershipTests.swift
146
Line should be 120 characters or less: currently 121 characters

RoomMembershipTests.swift
172
Line should be 120 characters or less: currently 121 characters

RoomMembershipTests.swift
471
File should contain 400 lines or less: currently contains 471

Errors

File
Line
Reason

ViewController.swift
30
Force casts should be avoided.

PCCurrentUser.swift
4
Type body should span 200 lines or less excluding comments and whitespace: currently spans 593 lines

PCCurrentUser.swift
1024
File should contain 400 lines or less: currently contains 1024

PCPresenceSubscription.swift
219
Enum element name should only contain alphanumeric characters: 'initial_state'

PCPresenceSubscription.swift
220
Enum element name should only contain alphanumeric characters: 'presence_update'

PCPresenceSubscription.swift
221
Enum element name should only contain alphanumeric characters: 'join_room_presence_update'

PCUserSubscription.swift
573
Enum element name should only contain alphanumeric characters: 'initial_state'

PCUserSubscription.swift
574
Enum element name should only contain alphanumeric characters: 'added_to_room'

PCUserSubscription.swift
575
Enum element name should only contain alphanumeric characters: 'removed_from_room'

PCUserSubscription.swift
576
Enum element name should only contain alphanumeric characters: 'room_updated'

PCUserSubscription.swift
577
Enum element name should only contain alphanumeric characters: 'room_deleted'

PCUserSubscription.swift
578
Enum element name should only contain alphanumeric characters: 'user_joined'

PCUserSubscription.swift
579
Enum element name should only contain alphanumeric characters: 'user_left'

PCUserSubscription.swift
580
Enum element name should only contain alphanumeric characters: 'typing_start'

PCUserSubscription.swift
581
Enum element name should only contain alphanumeric characters: 'typing_stop'

CursorTests.swift
93
Force tries should be avoided.

CursorTests.swift
106
Force tries should be avoided.

CursorTests.swift
175
Force tries should be avoided.

Helpers.swift
238
Variable name should be between 3 and 40 characters long: 'r'

Helpers.swift
238
Variable name should be between 3 and 40 characters long: 'e'

Helpers.swift
246
Variable name should be between 3 and 40 characters long: 'e'

TokenGenerator.swift
428
Force tries should be avoided.

TypingIndicatorTests.swift
110
Variable name should only contain alphanumeric characters: '_alice'

  Generated by :no_entry_sign: Danger

Added a test for the users property and replaced all of the guards with XCTAssertNils
Going to merge now


SAC Bugs

In SAC.py, SAC_BipedalWalker-v2.py, the codes:
class NormalizedActions(gym.ActionWrapper):
    def _action(self, action):
        low = self.action_space.low
        high = self.action_space.high

        action = low + (action + 1.0) * 0.5 * (high - low)
        action = np.clip(action, low, high)

        return action

    def _reverse_action(self, action):
        low = self.action_space.low
        high = self.action_space.high

        action = 2 * (action - low) / (high - low) - 1
        action = np.clip(action, low, high)

        return action

now should be changed as follows:
class NormalizedActions(gym.ActionWrapper):
    def action(self, action):
        low = self.action_space.low
        high = self.action_space.high

        action = low + (action + 1.0) * 0.5 * (high - low)
        action = np.clip(action, low, high)

        return action

    def reverse_action(self, action):
        low = self.action_space.low
        high = self.action_space.high

        action = 2 * (action - low) / (high - low) - 1
        action = np.clip(action, low, high)

        return action

in order to adapt to the latest OpenAI Gym core.py

otherwise there will be a overloaded error that "
Traceback (most recent call last):
File "SAC.py", line 308, in
main()
File "SAC.py", line 288, in main
next_state, reward, done, info = env.step(np.float64(action))
File "/Users/Shared/anaconda3/envs/Pytorch/lib/python3.8/site-packages/gym/core.py", line 285, in step
return self.env.step(self.action(action))
File "/Users/Shared/anaconda3/envs/Pytorch/lib/python3.8/site-packages/gym/core.py", line 288, in action
raise NotImplementedError
NotImplementedError
"

otherwise there will be a overloaded error that "
Traceback (most recent call last):
File "SAC.py", line 308, in
main()
File "SAC.py", line 288, in main
next_state, reward, done, info = env.step(np.float64(action))
File "/Users/Shared/anaconda3/envs/Pytorch/lib/python3.8/site-packages/gym/core.py", line 285, in step
return self.env.step(self.action(action))
File "/Users/Shared/anaconda3/envs/Pytorch/lib/python3.8/site-packages/gym/core.py", line 288, in action
raise NotImplementedError
NotImplementedError
"

RuntimeError: Found dtype Double but expected Float 请问该如何解决呢

RuntimeError: Found dtype Double but expected Float 请问该如何解决呢

请问你解决了吗，我也遇到了相同的问题


Wrong $HOME for non-root builds

When running a build using a non-root user, the $HOME variable is still set to /root. This variable should not be overwritten by drone. In my concret case, I use the ekidd/rust-musl-builder image to build a Rust project. Using the following .drone.yml produces the following output:
pipeline:
  build:
    image: ekidd/rust-musl-builder:stable
    commands:
      - id && echo $HOME

uid=1000(rust) gid=1000(rust) groups=1000(rust),27(sudo)
/root

For comparison the output of running the same command directly inside the container:
$ docker run -it --rm ekidd/rust-musl-builder sh -c 'id && echo $HOME'
uid=1000(rust) gid=1000(rust) groups=1000(rust),27(sudo)
/home/rust

Without knowing the codebase, I think the problematic lines in the code are https://github.com/drone/drone/blob/master/vendor/github.com/cncd/pipeline/pipeline/frontend/yaml/compiler/convert.go#L84 and https://github.com/drone/drone/blob/master/vendor/github.com/cncd/pipeline/pipeline/frontend/yaml/compiler/convert.go#L90. These should not set the home directory to a hardcoded string.

FYI drone does not currently support non-root images when running custom commands due to volume ownership issues. The hard-coded values reflect this limitation. There are plans to resolve this, which are being discussed on our mailing list. https://www.reddit.com/r/droneci/comments/8opfxw/build_using_docker_image_with_user_directive/


[WIP] - Error and Warning Handling

@bhough do we want to tackle this? It's not urgent. Could be done in a hacking session when you visit Vienna or Max and I travel to SF :)

Definitely do! I think I'm finally happy with the API now that I've found a way to suss out the file/module info so it does not have to be passed in every call. I also have it properly pulling out the error handling in the production build.  The next thing I'm trying to work through is how we should design the different message types. The four types of message I think we will need to support:

Warning - Something incorrect the user is doing that doesn't impact how this modules works (i.e. passing extra parameters).
Error that would cause a CSS Issue - Something the user is doing that will not cause the module to fail on the JS side but would produce invalid CSS.
Error that would cause a JS issue - Something the user is doing that will cause the module to fail on the JS side.
Deprecation warning - Likely something that would be added in a separate ticket, but the ability to warn a user when a module they are leverage will be deprecated in a later release.

As part of this, also need to determine an order of priority for checking for common errors across modules. Such as number of arguments > argument order > individual argument type > individual argument value.
Always open for a hacking sessions. I am actually in Basel, Switzerland until the end of the year, so much closer than SF. :)


sharp limits in phase plots

I suggest the hard range 0:P_p in phase plots as default.
Note also the units on y-axis are in round brackets instead of squared.

Units were fixed.
About the hard range, I would leave it as it is. Do you want to export plot that has a hard range on the model? I can imagine that it might be difficult to do that by holding the right mouse button.
You can try when you need to export try the following three options:

right-click on the plot --> Xaxis --> Manual select 0: P_p
or somewhat less convenient, right-click on the plot --> Mouse Mode --> 1, and then simply select the range you want to zoom by holding the left mouse button.
finally:  right-click on the plot --> export --> Matplotlib Window --> do whatever you want with the Matplotlib GUI.

Indeed these manual actions are less convenient (looking up the period, manually setting the period while it is already there in the table).
With "hard", I meant sharp. I.e. the plot should be initialised/resetted to the sharp limits.
Vise versa zooming a bit out zoom to give extra margin is easy.

This must be fixed now.

Works when loading a new phase plot. Ok.
I would also expect it, when clicking the resetting 'A'.

This is more complicated than I thought since I might have to make a custom padding for each plot. The padding is global when you press 'A'. I definitely want some padding on the RV, transit, and other plots. Let's keep this issue open, till I find a permanent solution for the Phase plots (e.g. custom user settings controls so the user can control the Auto behavior of each plot). To reset the plot with sharp limits, for now, you can simply select the plot back from the comboBox menu (e.g. phase pl. 1, etc.).

Works when loading a new phase plot. Ok.
I would also expect it, when clicking the resetting 'A'.


fix: Added Deny for CreateLogGroup action in EKS cluster role

Signed-off-by: Christopher Haar<EMAIL_ADDRESS>PR o'clock
Description
cloudwatch log group now gets auto destroyed with new iam policy deny for create log group in eks cluster role
Fixes: #920
Checklist

[x] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

@haarchri  please correct tittle of PR to fix: added deny log group in eks cluster role as CI will fail.
Also seems you using different version of terraform-docs which made some changes in lines 137-147, if possible revert those lines to original.

@daroga0002  rebased and changed readme

@antonbabenko  lets merge this (I will let you know when we can make release as I still looking into few other PRs)

This PR is included in version 17.24.0 :tada:

To anyone landing on this confused like I was about whether this breaks compatibility with Fluent Bit: the IAM role you want to attach the CloudWatch policy to is the node role, not the cluster role.


Make train.py up to date with pylint standard

Also removed some unneeded code and comments. Doing each of these as there own file to not overwhelm.

Had to do some removal of comments due to merge conflicts. Hope that nothing important went away! Keeping the branch.


Uncaught ArgumentCountError: Too few arguments to function OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest::getSuccessUrl()

Release version 1.7.0
Steps to reproduce:

Install and configure extension on Magento Open Source <IP_ADDRESS>
Add an item to the shopping cart
Go to the shopping cart and click the Amazon Pay button
During the final step of one step checkout, click the Place Order button
An HTTP 500 error is thrown on the server

Error Logs / Stack Traces:
<18-Jul-2019 20:39:11 UTC> PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest::getSuccessUrl(), 0 passed in lib/OffAmazonPaymentsService/Client.php on line 1392 and exactly 1 expected in lib/OffAmazonPaymentsService/Model/ConfirmOrderReferenceRequest.php:82
Stack trace:
#0 lib/OffAmazonPaymentsService/Client.php(1392): OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest->getSuccessUrl()
#1 lib/OffAmazonPaymentsService/Client.php(470): OffAmazonPaymentsService_Client->_convertConfirmOrderReference(Object(OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest))
#2 app/code/community/Amazon/Payments/Model/Api.php(98): OffAmazonPaymentsService_Client->confirmOrderReference(Object(OffAmazonPaymentsService_Model_ConfirmOrderRe in lib/OffAmazonPaymentsService/Model/ConfirmOrderReferenceRequest.php on line 82

[18-Jul-2019 20:54:57 UTC] PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest::getFailureUrl(), 0 passed in lib/OffAmazonPaymentsService/Client.php on line 1395 and exactly 1 expected in lib/OffAmazonPaymentsService/Model/ConfirmOrderReferenceRequest.php:115
Stack trace:
#0 lib/OffAmazonPaymentsService/Client.php(1395): OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest->getFailureUrl()
#1 lib/OffAmazonPaymentsService/Client.php(470): OffAmazonPaymentsService_Client->_convertConfirmOrderReference(Object(OffAmazonPaymentsService_Model_ConfirmOrderReferenceRequest))
#2 app/code/community/Amazon/Payments/Model/Api.php(98): OffAmazonPaymentsService_Client->confirmOrderReference(Object(OffAmazonPaymentsService_Model_ConfirmOrderR in lib/OffAmazonPaymentsService/Model/ConfirmOrderReferenceRequest.php on line 115

@jasonkcarter Thanks for reaching out to us.
We are looking in to this and provide an update soon.
Tarini

I had the same error in version 1.6.0, applying the changes above worked for me. Hope that helps. I'm using the Magento Core OnePage Checkout

@SpottedPaint Thanks for confirming!
closing this issue.


Absolute paths are not considered as SNI hostnames

Motivation:
I have a use case where many workloads, communicating through UNIX Domain Sockets, use mutual TLS for authentication.
Before this suggested change, Netty considers absolute paths as valid SNI names. (For example: /run/service/service.S would be considered as a valid SNI hostname by Netty.) After doing this validation, Netty ends up calling the SNIHostName constructor which in turn calls IDN.toASCII which in turns throws an exception because foward slash / should not be present in a hostname.
Modification:
Netty does not consider hostnames starting with forward slash / as valid SNI hostnames.
Result:
TLS over UDS works.

@normanmaurer for viz

Can you add a unit test as well ?

@normanmaurer — I wanted to but I couldn't find existing tests for that function. I will add tests in here.

@marctrem any update ?

@marctrem also can you please sign our ICLA: https://netty.io/s/icla ?

@marctrem How is it going with this?

@marctrem ping...

@marctrem ping


Issue running client API

Hi guys , Ive been testing H2ogpt for while and notice that from one or 2 weeks ago the client api stopped after some git pull.
Running api client on Windows
Successfully installed gradio-client-0.2.10
Source code :
from gradio_client import Client
import ast
HOST_URL = "https://xxx.xx.xx.cloud"
client = Client(HOST_URL)
kwargs = dict(instruction_nochat='Who are you?')
res = client.predict(str(dict(kwargs)), api_name='/submit_nochat_api')
response = ast.literal_eval(res)['response']
print(response)
port used is 443 and is functional
result:
C:\CMD\LLM\lab\h2ogpt\tests>python api.py
Loaded as API: https://xxx.xx.xx.cloud/ ✔
Traceback (most recent call last):
File "C:\Users\CMD\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\gradio_client\client.py", line 595, in _get_config
config = json.loads(result.group(1))  # type: ignore
AttributeError: 'NoneType' object has no attribute 'group'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\CMD\LLM\lab\h2ogpt\tests\api.py", line 5, in
client = Client(HOST_URL)
File "C:\Users\CMD\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\gradio_client\client.py", line 127, in init
self.config = self._get_config()
File "C:\Users\CMD\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\gradio_client\client.py", line 597, in _get_config
raise ValueError(
ValueError: Could not get Gradio config from: https://xxx.xx.xx.cloud/
C:\CMD\LLM\lab\h2ogpt\tests>

Hi.  We have extensive tests for the gradio client.  I cannot tell what is wrong in above, except that "result" is None, but I don't know what that is about.
You'd probably have to share code api.py.

stale


Personal Account vs. Business Account

I'm looking to determine the trade-off between submitting a Teams app from a personal account vs. a business account. And if it is already up and running as a personal account, what the trade-offs would be converting it to a business account. Based on the documentation, this seems like the right spot for that information to be included.
Can you help me better understand a few questions:

What would happen to the customers that are currently on this version of the Teams integration?
Would we be able to turn the other version on first so we would know it is up and functional before shutting down the original version?
Do current customers need to switch then? If so, how do they do this?

Thanks,
Patrick Hanley
Director of Strategic Alliances
m:<PHONE_NUMBER>
e<EMAIL_ADDRESS>Dynamic Signal

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

ID: 143108bb-d6e6-48a8-a7cb-d11b89ebeaab
Version Independent ID: 145773e2-149e-1c1c-5797-433c13f13b22
Content: Admin settings for apps in Microsoft Teams - Microsoft Teams
Content Source: Teams/admin-settings.md
Service: msteams
GitHub Login: @LolaJacobsen
Microsoft Alias: lolaj

@phanley10 Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


[Event Request] codeunit 1339 "Cancel Posted Sales Cr. Memo" OnRunAfterCreateCrMemoCancelling

Can we please get the following event?
trigger OnRun()
var
    SalesHeader: Record "Sales Header";
begin
    UnapplyEntries(Rec);
    CreateCopyDocument(Rec, SalesHeader);
    //+EVENT
    OnRunAfterCreateCrMemoCancelling(Rec, SalesHeader);
    //-EVENT

    CODEUNIT.Run(CODEUNIT::"Sales-Post", SalesHeader);
    SetTrackInfoForCancellation(Rec);

    Commit();
end;

[IntegrationEvent(false, false)]
local procedure OnRunAfterCreateCrMemoCancelling(SalesCrMemoHeader: Record "Sales Cr.Memo Header"; var InvoiceSalesHeader: Record "Sales Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 52831.


Publish geoutils on conda-forge

I think all dependencies are on conda-forge, so this should be relatively straightforward. I've done this before so I can try it again. Otherwise, if someone is keen, please feel free.

It's going forward! Passing all tests in https://github.com/conda-forge/staged-recipes/pull/14997

It is done!


[BUG] incorrect codegen for operators on struct pointers

Describe the bug
For the operators +=/-=/*=//=/%=/^=/&=/|= invalid code is generated when using them on pointers to structs/classes. Currently the following code:
from cython.operator cimport dereference

ctypedef struct Test:
    int a

def test():
    cdef Test* b = NULL
    dereference(b).a += 1

generates the following C/C++ code:
__pyx_t_4test_Test __pyx_t_1;
__pyx_t_1 = (*__pyx_v_b);
__pyx_t_1.a = (__pyx_t_1.a + 1);

this means the assignment is only performed on the temporary and therefor has no effect.
Expected behavior
There are a couple of ways to improve this:

these operators are already forbidden for direct usage with dereference:

def test():
    cdef int* a = NULL;
    dereference(a) += 1;

currently leads to the following error Illegal operand for inplace operation.. It would be certainly better to prevent the usage than generating broken code.
2) I do not see any reason why this temporary even has to be generated here, so the code gen e.g. for += could be
(*__pyx_v_b).a = ((*__pyx_v_b) + 1);

the code gen from 2) is strictly speaking still not correct in C++, since the operators + and += can have different behaviors. Even though this is rare, this means it would be best to avoid the transformation from += to +:

(*__pyx_v_b).a += 1;

Environment (please complete the following information):

OS: any
Python version any
Cython version: 3.0.0a10

Yes probably is a bug but you shouldn't need the dereference at all:
def test():
    cdef Test* b = NULL
    b.a += 1

should work I think

This will not work since b is a pointer so the c code has to be:
Test* b = NULL;
b->a += 1

or
Test* b = NULL;
(*b).a += 1

which is easier to generate. Current workaround is:
dereference(b).a = dereference(b).a  + 1

Cython automatically generates arrow operator on pointers. It works for me at least (just tested it)

Cython automatically generates arrow operator on pointers. It works for me at least (just tested it)

Oh I was only thinking about writing it the way I would in C/C++. That allows for some cleanup in my code base :partying_face:


Not compatible with Firefox on Android

I just got a new phone (yay!) and it is a big upgrade compared to mine from 7years ago. While installing Firefox I kept searching for "open with binder" as an extension. Turns out we apparently declare to not be compatible with Firefox running on Android.
Is this on purpose? Does someone know how we could fix that?
It seems the small bits of JS we use should also run in browsers on android. Opening things on mybinder.org from a phone is super useful.

No particular reason. Compatibility with FF mobile is likely opt-in.

Compatibility with FF mobile is likely opt-in.

And by that I mean the metadata that tell the store wether we are compatible.  I'll see if I find time to set my phone in debug mode and test this on it.


Move codeql.yml one dir up

TODO:

[ ] Fix the permission error

Hi. Apologies for the permissions error. We believe it should be fixed now if you re-run the Code Scanning Action. :slightly_smiling_face:

Thanks, now it works!

After we merge this, I will backport this action to v4-dev too.


--copt=-mavx2 causes build error

Compiles fine with
bazel build -copt=-march=native --local_resources 2048,.5,1.0 -c opt //tensorflow/tools/pip_package:build_pip_package
but then when running tensorflow it warns that my cpu has avx2 but it isn't enabled so I run
bazel build --copt=-mavx2 --copt=-march=native --local_resources 2048,.5,1.0 -c opt //tensorflow/tools/pip_package:build_pip_package
And I get the following error:
ERROR: /home/ben/tensorflow-opencl/tensorflow/core/kernels/BUILD:828:1: C++ compilation of rule '//tensorflow/core/kernels:gather_functor' failed: gcc failed: error executing command /usr/bin/gcc -U_FORTIFY_SOURCE '-D_FORTIFY_SOURCE=1' -fstack-protector -Wall -Wl,-z,-relro,-z,now -B/usr/bin -B/usr/bin -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-canonical-system-headers ... (remaining 51 argument(s) skipped): com.google.devtools.build.lib.shell.BadExitStatusException: Process exited with status 1.
In file included from ./tensorflow/core/framework/numeric_types.h:25:0,
                 from ./tensorflow/core/framework/type_traits.h:22,
                 from ./tensorflow/core/kernels/gather_functor.h:22,
                 from tensorflow/core/kernels/gather_functor.cc:50:
./third_party/eigen3/unsupported/Eigen/CXX11/FixedPoint:42:52: fatal error: src/Tensor/TensorContractionThreadPool.h: No such file or directory
compilation terminated.
Target //tensorflow/tools/pip_package:build_pip_package failed to build

FYI, I was able to resolve this with --copt=-bdver3

This should be fixed now.


Attribute Mapping delete option for SCIM and OIDC mappins is not working in the new React console, but possible in the old mgt console

Describe the issue:

Custom Attribute mappings can be successfully deleted from the new console, but SCIM and OIDC mappings cannot be deleted.
However, in the old management console, these mappings cannot be deleted.
https://github.com/wso2/product-is/assets/41533942/1f0cf74c-9086-4bc8-ba9f-dac0dd69184e
How to reproduce:

Sign into console.
Navigate to attributes page.
Locate attribute mapping section and click on either OIDC or SCIM mapping.
In the edit page, click delete option.
Even though the backend sends a success response, the mapping is not deleted.
However, custom mappings created from the console can successfully be deleted.

Expected behavior:

Attribute mapping delete UX should be consistent with the old mgt console
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 7 Beta2 snapshot
OS: [e.g., Windows, Linux, Mac]
Database: [e.g., MySQL, H2]
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

N/A
Suggested labels:

N/A

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


On README, mention plugin dev access

You need ChatGPT's plugin dev access to set this up.
Mention this and maybe add a link to their waitlist too.

I can take this issue if ok. I guess message/link should be placed at end of opening h1 paragraph?

It should be part of the getting started guide or something similar to that.

'Getting Started Guide' is to be inserted into 'README'? who is assigned to 'GSG'? Another option is to mention it at the end of the first paragraph in the 'README' where it discusses GPT right at top of doc until 'GSG' is ready...

What do you think is the best approach here?

README already seems to be the place to plugin all high-level info for someone new looking at Kaguya so Getting Started Guide section should be included there at top of doc. Roughly: 1) Gain dev access (or other option if unable) 2) Install Docker if not already installed 3) Execute docker.sh script 4) setup localhost port 3000 5) Interact with Kaguya 6) Open-source. There are sub-steps within each step for setup that might be difficult for a beginner so those could be included too. Thoughts?

Sounds good! Feel free to send a draft PR if you want to go step by step, or a full PR if you prefer it that way

ok thx - back at this today will keep updated


Reducing memory usage

What are you trying to achieve?
I'm processing batches of 200-400MB TIFF files on an instance with 1GB of memory. It is occasionally not enough. I'm trying to understand what I can do better to manage memory.
Have you searched for similar questions?
Yes, I've landed on #138 and related issues regarding streaming. But I'm not sure if this is the best approach.
My use case requires me to clone and downsample the entire image to be analyzed. I then use the analysis results for processing the full-size image. If I am to change the file to a stream, I may not be able to split it into two parts (one, downsized, for analysis and one for processing).
I currently start the instance on node:14-alpine container with --optimize_for_size --max_old_space_size=320 --gc_interval=50. Other than streaming, I'm also considering adding an artificial delay between jobs to ensure that the memory has time to empty out.
I would really appreciate some guidance here. Thank you so much!
Are you able to provide a minimal, standalone code sample that demonstrates this question?

// create a reduced-size image sample
const data = await sharp(image).clone().resize(500).toBuffer();

// this object will store values that will be used for full-size image manipulation
const prescriptions = {};

// determine values for full-size image manipulation
const channelNames = ["red", "green", "blue"];
const prescriptionsCalculators = channelNames.map(async (channel) => {
  const forAnalysis = await sharp(data)
  .extractChannel(channel)
  .toColorspace("b-w")
  .blur(1)
  .toBuffer();

  const stats = await sharp(forAnalysis).stats();
  const { channels } = stats;

  const range = [channels[0].min, channels[0].max];
  const linear = levels(range[0], range[1]);

  prescriptions[channel] = { linear, range };
  return await linear;
});
await Promise.all(prescriptionsCalculators);

// use the above data to process full-size image
const channels = {};
const channelEffects = channelNames.map(async (channel) => {
  const result = sharp(reversed)
    .extractChannel(channel)
    .toColorspace("b-w")
    .linear(...prescriptions[channel].linear)
    .toBuffer();
  channels[channel] = result;
  return result;
});
await Promise.all(channelEffects);

// reassemble channels into RGB
const positive = sharp(await channels["red"]).joinChannel([
  await channels["green"],
  await channels["blue"],
]);

return positive;

The quickest way to reduce memory usage would be to switch from RAM to filesystem. Have you tried replacing any of the toBuffer calls with toFile?

Thank you for the tip, @lovell! I tried replacing a few calls and memory usage seems to be a little more constrained.


webkitgtk: set -DENABLE_JOURNALD_LOG=OFF if !systemdSupport

Description of changes
When webkitgtk is built with systemdSupport=false the following build failure occurs:
-- Could NOT find Journald (missing: Journald_LIBRARY Journald_INCLUDE_DIR)
CMake Error at Source/cmake/OptionsGTK.cmake:486 (message):
  libsystemd or libelogind are needed for ENABLE_JOURNALD_LOG
Call Stack (most recent call first):
  Source/cmake/WebKitCommon.cmake:220 (include)
  CMakeLists.txt:20 (include)

-- Configuring incomplete, errors occurred!
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeOutput.log".
See also "/build/webkitgtk-2.36.0/build/CMakeFiles/CMakeError.log".

Let's add -DENABLE_JOURNALD_LOG=OFF if !systemdSupport in order to fix this.
Things done

Built on platform(s)

[X] powerpc64le-linux
[X] x86_64-linux
[X] aarch64-linux

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Ping...

Did not test, can we remove -DUSE_SYSTEMD=OFF?

Did not test, can we also remove -DUSE_SYSTEMD=OFF?

Indeed, we can.


Starting freshclam with defs failing

WIthin the last week I have noticed the update failing with the following error
Can't add daily.mdb to new daily.cld - please check if there is enough disk space available it looks like we have exceeded the Lambda 500mb limit here. I hadn't updated the package in a year or so so I updated and now see the following errors:
Starting freshclam with defs in /tmp/clamav_defs.
freshclam output:
ClamAV update process started at Tue Oct 29 13:25:29 2019
main.cvd is up to date (version: 58, sigs: 4566249, f-level: 60, builder: sigmgr)
Downloading daily-25614.cdiff [100%]
ERROR: cdiff_cmd_close: Can't write to ./clamav-f9e93c076af9719bbedd7d8f184007e4.tmp
ERROR: cdiff_apply: Can't execute command CLOSE
ERROR: cdiff_apply: Error executing command at line 1578
ERROR: getpatch: Can't apply patch
WARNING: Incremental update failed, trying to download daily.cvd
Downloading daily.cvd [100%]tabase load killed by signal 9
ERROR: Failed to load new database
Unexpected exit code from freshclam: 55.
File does not exist: main.cld

Hey @chrisgilmerproj - Is someone looking into this?
Or is there an ETA for a fix for this? We are no longer getting our daily updates either. Thanks!!!

Hi folks, sorry I missed this. Can you tell me if you updated your IAM policies for your lambdas? You now need this in order for them to work:
      {
         "Sid":"s3HeadObject",
         "Effect":"Allow",
         "Action":"s3:ListBucket",
         "Resource":[
             "arn:aws:s3:::<av-definition-s3-bucket>/*",
             "arn:aws:s3:::<av-definition-s3-bucket>"
         ]
      }

In #93 and #96 we made updates that would fix issues around downloading as well as fixing the policies.  I have a feeling this will fix things so I'm going to close the ticket. If it doesn't please let me know and I'll re-open and do more investigation.  As always, I appreciate that you are bringing these issues to us so we can make the tool better.

Hey @chrisgilmerproj. Thanks for getting back to me. So this has been working fine for months, but only recently started failing, and we didn't change anything. However I did go and verify our IAM roles and can confirm that they do indeed have these permissions:
myProjectAntiVirusUpdateIAMRole:     Type: AWS::IAM::Role     Properties:       RoleName: myProjectAntiVirusUpdateRole       AssumeRolePolicyDocument:         Version: "2012-10-17"         Statement:           - Action:               - sts:AssumeRole             Effect: Allow             Principal:               Service:                 - lambda.amazonaws.com       Policies:         - PolicyName: !Sub myProject-av-update-logging-${AppEnv}-policy           PolicyDocument:             Version: "2012-10-17"             Statement:               - Action:                   - logs:CreateLogGroup                   - logs:CreateLogStream                   - logs:PutLogEvents                 Resource: "*"                 Effect: Allow         - PolicyName: !Sub myProject-av-update-s3-${AppEnv}-policy           PolicyDocument:             Version: "2012-10-17"             Statement:               - Action:                   - s3:GetObject                   - s3:GetObjectTagging                   - s3:ListBucket                   - s3:PutObject                   - s3:PutObjectTagging                   - s3:PutObjectVersionTagging                 Effect: Allow                 Resource:                   - !Sub arn:aws:s3:::${myProjectAvDefinitionBucket}                   - !Sub arn:aws:s3:::${myProjectAvDefinitionBucket}/*
I've replaced a few variable names above with "myProject" and we are using params in our CFT but i've confirmed in the AWS Console that all permissions/roles are as expected and that the lambda does have the correct S3 permissions.

That is really odd. And you didn't update the code? Perhaps I'd suggest seeing if there's a traceback in the cloudwatch logs that we can dig into. Let's look at that and decide if we need to reopen.

Hey @chrisgilmerproj no we didn't update the code and we started getting the Can't add daily.mdb to new daily.cld - please check if there is enough disk space available error. We then updated to the latest version. We are now getting the following warning
LibClamAV Warning: ***  The virus database is older than 7 days!  ***
Here's one of our recent logs:
...
Attempting to create directory /tmp/clamav_defs.

Not downloading older file in series: bytecode.cvd
Downloading definition file /tmp/clamav_defs/main.cvd from s3://clamav_defs/main.cvd
Downloading definition file /tmp/clamav_defs/main.cvd complete!
Downloading definition file /tmp/clamav_defs/bytecode.cld from s3://clamav_defs/bytecode.cld
Downloading definition file /tmp/clamav_defs/bytecode.cld complete!
Downloading definition file /tmp/clamav_defs/daily.cvd from s3://clamav_defs/daily.cvd
Downloading definition file /tmp/clamav_defs/daily.cvd complete!
Starting freshclam with defs in /tmp/clamav_defs.
freshclam output:
ClamAV update process started at Thu Nov  7 03:12:09 2019
main.cvd is up to date (version: 58, sigs: 4566249, f-level: 60, builder: sigmgr)
Downloading daily-25614.cdiff [100%]
Downloading daily-25615.cdiff [100%]
Downloading daily-25616.cdiff [100%]
Downloading daily-25617.cdiff [100%]
Downloading daily-25618.cdiff [100%]
Downloading daily-25619.cdiff [100%]
Downloading daily-25620.cdiff [100%]
Downloading daily-25621.cdiff [100%]
Downloading daily-25622.cdiff [100%]
Downloading daily-25623.cdiff [100%]
Downloading daily-25624.cdiff [100%]
Downloading daily-25625.cdiff [100%]

Unexpected exit code from freshclam: -9.
File does not exist: main.cld
Not uploading main.cvd because md5 on remote matches local.
File does not exist: daily.cld
Not uploading daily.cvd because md5 on remote matches local.
Not uploading bytecode.cld because md5 on remote matches local.
File does not exist: bytecode.cvd
Script finished at 2019/11/07 03:14:30 UTC
...

Maybe something to do with Unexpected exit code from freshclam: -9.

Maybe something to do with Unexpected exit code from freshclam: -9.

I think you're right. I'd suggest boosting your memory on the lambda and see if that helps.

Thanks @chrisgilmerproj - I have mine set to the default 512. What are you currently running yours at? Thanks!

Also here is an excerpt from our logging:
Downloading definition file /tmp/clamav_defs/main.cvd complete!
Downloading definition file /tmp/clamav_defs/bytecode.cld from s3://clamav_defs/bytecode.cld
Downloading definition file /tmp/clamav_defs/bytecode.cld complete!
Downloading definition file /tmp/clamav_defs/daily.cvd from s3://clamav_defs/daily.cvd
Downloading definition file /tmp/clamav_defs/daily.cvd complete!
Starting freshclam with defs in /tmp/clamav_defs.
END RequestId: 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e
REPORT RequestId: 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e	Duration: 300100.08 ms	Billed Duration: 300000 ms	Memory Size: 512 MB	Max Memory Used: 512 MB	Init Duration: 447.93 ms
2019-11-08T12:17:03.022Z 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e Task timed out after 300.10 seconds

START RequestId: 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e Version: $LATEST
Script starting at 2019/11/08 12:17:57 UTC

Not downloading main.cvd because local md5 matches s3.
Not downloading daily.cvd because local md5 matches s3.
Not downloading bytecode.cld because local md5 matches s3.
Not downloading older file in series: bytecode.cvd
Downloading definition file /tmp/clamav_defs/daily.cld from s3://clamav_defs/daily.cld
Downloading definition file /tmp/clamav_defs/daily.cld complete!
Starting freshclam with defs in /tmp/clamav_defs.
freshclam output:
ClamAV update process started at Fri Nov  8 12:18:02 2019
main.cvd is up to date (version: 58, sigs: 4566249, f-level: 60, builder: sigmgr)
Downloading daily-25614.cdiff [100%]
Downloading daily-25615.cdiff [100%]
Downloading daily-25616.cdiff [100%]
Downloading daily-25617.cdiff [100%]
Downloading daily-25618.cdiff [100%]
Downloading daily-25619.cdiff [100%]
Downloading daily-25620.cdiff [100%]
Downloading daily-25621.cdiff [100%]
Downloading daily-25622.cdiff [100%]
Downloading daily-25623.cdiff [100%]
Downloading daily-25624.cdiff [100%]
ERROR: cdiff_cmd_close: Can't write to ./clamav-1b9ea5511d3e407d7c111ec492af4b84.tmp
ERROR: cdiff_apply: Can't execute command CLOSE
ERROR: cdiff_apply: Error executing command at line 1159
ERROR: getpatch: Can't apply patch
WARNING: Incremental update failed, trying to download daily.cvd
Downloading daily.cvd [100%]
WARNING: Mirror <IP_ADDRESS> is not synchronized.
bytecode.cld is up to date (version: 331, sigs: 94, f-level: 63, builder: anvilleg)

File does not exist: main.cld
Not uploading main.cvd because md5 on remote matches local.
Not uploading daily.cld because md5 on remote matches local.
Not uploading daily.cvd because md5 on remote matches local.
Not uploading bytecode.cld because md5 on remote matches local.
File does not exist: bytecode.cvd
Script finished at 2019/11/08 12:18:21 UTC
END RequestId: 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e
REPORT RequestId: 8ec89e1a-4aa6-46b4-a722-ae768a1e8c6e	Duration: 24134.02 ms	Billed Duration: 24200 ms	Memory Size: 512 MB	Max Memory Used: 512 MB

I'm re-opening this while I do more investigation. Also going to pull in #98 into this.

I have a PR #100 to test out issues locally. It's a WIP but you might try it out and see if it helps while I'm working on this.

I've done some research here and the best possible answer I have is that you've got to boost your memory up to 1024MB from 512MB. I can't seem to reproduce this exact error even when I do severely memory constrained lambdas or when I download new files from scratch but I haven't given up yet.
Can you boost your memory allocation and report if you see an issue still?

Thanks @chrisgilmerproj. I’ve boosted this up to 2048 (from 512). I’ll check back in once it’s had a chance to run the daily update and let you know what my results look like. Might be Monday morning before I get a chance to check.
Thanks for the help!

@chrisgilmerproj So it "appears" like bumping this from 512mb to 2048mb may indeed have resolved the issue. I think you can go ahead and close this ticket, but I can re-open if this appears again. I'm assuming maybe the ever-increasing size of the daily definitions maybe just peaked over the size able to be processed by a 512mb lambda?

@markmckim - I'm so glad this worked for you! I'll keep this open and see if we get the same feedback from other folks. I'll also probably update the documentation.

Sounds good. Thanks @chrisgilmerproj !

Bumping to 2048mb works for me. Thanks!

Glad to hear it folks!


[cxx-interop][benchmark] Conform vector to CxxRandomAccessCollection

This is needed to benchmark Collection APIs for std::vector.

@swift-ci please benchmark

@swift-ci please smoke test

This patch is superseded by https://github.com/apple/swift/pull/62880.

I am sorry I didn't understand the comment


Can have a way to enable/disable comments

Currently the comments are always displayed. There should be a way to disable the display of comments.

Closing this as Gauge will not be implementing this feature.


[Question]Can I build and use gpu_cache independently?

Or I have to build the whole project?

Thanks for your question. You can get the gpu_cache library by building the whole project, or modify the  CMakeLists.txt, such as keeping only the source and dependencies of gpu cache to get the library independently.


Update README.md

@bill1gates r+

:pushpin: Commit ca3b0fa has been approved by kteza1

:hourglass: Testing commit ca3b0fa with merge ca4e66f...

@bill1gates r+


[warning] stb_image_resize2.h

In file included from rtextures.c:216:
external/stb_image_resize2.h:2866:14: warning: ‘stbir__support_zero’ defined but not used [-Wunused-function]
2866 | static float stbir__support_zero(float s, void * user_data)
|              ^~~~~~~~~~~~~~~~~~~
external/stb_image_resize2.h:2503:13: warning: ‘stbir_simd_memcpy’ defined but not used [-Wunused-function]
2503 | static void stbir_simd_memcpy( void * dest, void const * src, size_t bytes )
|             ^~~~~~~~~~~~~~~~~
ar rcs ../src/libraylib.a rcore.o rshapes.o rtextures.o rtext.o utils.o rglfw.o rmodels.o raudio.o
raylib static library generated (libraylib.a) in ../src!

I use
-W -Wall -Werror -pedantic -pedantic-errors
and yse it harmless ;D

@ubkp Thanks for the review! Note that I'm merging it for consistency with previous solutions but I'm not fully convinved if this is the best approach to deal with those warnings. I'm not a fan of pragmas, I got the feeling that they belong to build side and not code side, despite they are supported by most compilers...
At the moment they are just a few lines in a couple of files and don't seem to generate any issue (beside an aesthetic detriment), so, keeping them for now...