Sample code for setting content type and encoding

Can you please provide a sample code demonstrating how to set the message content type and encoding using python sdk?
I am trying to achieve "IoT Hub message routing with routing on message body "and I am looking for a simple python code for constructing the message body and content type in the correct format similar to what is described in this link. Thank you.

@apcsaba
Here is the sample code:
setContentResult = message.set_content_encoding_system_property("utf-8")
if (setContentResult != 0):
print ( "set_content_encoding_system_property failed" )
break
setContentTypeResult = message.set_content_type_system_property("application/json")
if (setContentTypeResult != 0):
print ( "set_content_type_system_property failed" )
break
Best Regards,
Zoltan

I modified SimulatedDevice.py to set the content type and content encoding as described above:
message = IoTHubMessage(msg_txt_formatted)
set_content_result = message.set_content_encoding_system_property("utf-8")
set_content_type_result = message.set_content_type_system_property("application/json")

if set_content_result != 0:
    print("set_content_encoding_system_property FAILED")

if set_content_type_result != 0:
    print("set_content_type_system_property FAILED")

I have also set the sleep timer to 1000 seconds because I noticed that I was not receiving a response from the IoT hub. After some time has passed I get the following response:
Sending message: {"temperature": 20.52,"humidity": 67.88}
IoT Hub responded to message with status: MESSAGE_TIMEOUT

I am not seeing any incoming messages to the event hub using the device explorer tool. However if I don't set the content type and content encoding the messages send.
Not sure if this is relevant but I also receive these two error messages when running the script with or without the content settings:
Error: Time:Tue Jul 10 16:00:40 2018 File:/usr/sdk/src/c/iothub_client/src/iothub_client_core_ll.c Func:IoTHubClientCore_LL_CreateFromConnectionString Line:892 iotHubName is not found
Error: Time:Tue Jul 10 16:00:40 2018 File:/usr/sdk/src/c/iothub_client/src/iothub_client_core.c Func:create_iothub_instance Line:928 Failure creating iothub handle

I have been ignoring it because the messages were still sending and being acknowledged by the IoT hub.

@janickat24 it's bizarre because the error you're seeing indicates it might not find the IoT hub (which means no matter how much you change message properties, it shouldn't work).
One thing that might be related: there's a bug in the C SDK that requires an additional call to make sure the content-type property is picked up correctly if you are using MQTT: see https://github.com/Azure/azure-iot-sdk-python/issues/153
Hope that helps - and if it doesn't, feel free to open a new issue, showing us in detail your code, the console logs, what protocol you're using, and what you are observing. thanks!

Added client.set_option("auto_url_encode_decode", True) to the iothub_client_init() function and everything seems to be working correctly. Thanks!

@janickat24 thanks for confirming! glad to hear that!


[DCOS-55016]: node action dropdown, drain active nodes

Closes DCOS-55016
Testing

add this key to the "uiConfiguration" key in Config.dev.ts

features: {
  maintenance: true
},

enable useUIConfigFixtures
Visit the nodes page and find the new actions dropdown on the far right (for active nodes)
Ensure actions don't appear when feature is disabled
Try draining a node (this deactivates the node as well)
In order to reactivate the node, issue the following curl command, but replace the Cookie header VALUE and NODE_ID tokens with your own values

curl -v -XPOST 'http://localhost:4200/mesos/api/v1' -H 'Content-Type: application/json' -H 'Accept: application/json' --data-binary '{"type":"REACTIVATE_AGENT", "reactivate_agent": {"agent_id": {"value": "NODE_ID"}}}' -H 'Cookie: dcos-acs-auth-cookie=VALUE; dcos-acs-info-cookie=VALUE' --compressed

Notes
I'm told the drained node will eventually be garbage collected by mesos but I'm not sure how long it takes.
Screenshots
Sorry I was unable to make a screenshot before my cluster expired. I'll make a new one tomorrow.

@DanielMSchmidt thank you for removing that! I didn't realize I committed it

Should the tooltip be on the icon, not on the node name?

:tada: This PR is included in version 2.131.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


It required some modifications to work with blazor server

Thank you for the informative instructions.
i would like to mention that It required some modifications to work with blazor server ... some instructions with build. in the start should be removed ... some instructions should be in the starup.cs not in program.cs

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

ID: 85af7a99-672c-0438-dcc2-e8edfaf70c54
Version Independent ID: 85af7a99-672c-0438-dcc2-e8edfaf70c54
Content: Use ASP.NET Core SignalR with Blazor
Content Source: aspnetcore/blazor/tutorials/signalr-blazor.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @guardrex
Microsoft Alias: riande

Duplicate of #27939.


Nodes not appearing on map

Node info has been uploaded but they are not appearing on map

how long ago was it updated? The current backend process will update the bpjson and nodes info each 24h because we are not displaying real-time info from the blockchain, we have a databas table that works as a cache with the producer info and the bpJson that it's updated each 24 hours

This was updated several days ago. It appears it is not updating data from the nodes table


img to img?

Any way to add img2img to this node?

img2img node will be released in the future.

img2img node will be released in the future.

This issue has been merged into https://github.com/Limitex/ComfyUI-Diffusers/issues/7


[Feature][Engine] Engine Server and Client Instance support custom JVM parameters.

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
We should support user custom JVM parameters like -Xmx=5G -Xms=5G. User can set it in seatunnel-env.sh from default. Also can set it in shell : seatunnel.sh and seatunnel-cluster.sh.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please assign it to me.Thx.

Close by #3307


Design the coupling/forcing interface

NextSIM_DG will need some way of ingesting data from external sources, either climatological data or by coupling to other models.
A common interface needs to be designed that will be implemented by classes providing this data via OASIS or similar coupling software.

What fields are needed?
How are the different implementations specified (ModuleLoader?)
How closely do we tie the design to OASIS? What is the trade off between ease-of-use with OASIS and future generality?

How does it deal with the shorter timestep of most atmospheric models?

The ExternalData class currently provides:

air temperature
dew point and/or water vapour mixing ratio
sea level pressure
snowfall
inbound shortwave flux
inbound longwave flux
mixed layer depth.

The PrognosticData class provides:

sea surface temperature
sea surface salinity,
but probably shouldn't. These values are prognostic in the NextSIM slab ocean.

An obvious absentee is:

surface winds.

Old NextSIM defines the following fields as external_data:

surface wind
2 m temperature
water vapour mixing ratio
atmospheric pressure
incoming short-wave radiation
incoming long-wave radiation (twice)
total precipitation
snowfall rate
precipitation snow fraction
2 m dew point
specific humidity of the air
"geostrophic" ocean currents
sea surface height
sea surface temperature
sea surface salinity
mixed layer depth
fraction of short-wave radiation absorbed by the mixed layer
bathymetry.

More general design thoughts:

The interface should be flexible enough to couple using OASIS or the American (CESM) coupling software.
The code should be able to be built even if the OASIS headers are not present.
How the partition logic of OASIS ends up accessing the ModelArray data may inform (me) how that data should be partitioned.
A preprocessing, post restart file read step will probably be necessary. Can this be done within the nextsimdg binary, or will an independent process also need to read the restart file for the array sizes and shapes, do some processing and them start the nextsimdg and OASIS binaries?

If preprocessing is necessary, the same script could be the place to set the DG grid parameters and the MPI array partitioning.

~One possible way of implementing the independence of the code to OASIS might be to compile the OASIS code to a dynamic library. This entirely separates the OASIS code from the core code base. A second, stub version of the library could be compiled for when OASIS is absent.~
Since the model will have to be compiled at least on a per-machine basis (if not a per-user, per-machine) basis, then the selection between OASIS implementation and stub implementation can be done at compile time. The Module code could even be altered to include or exclude the OASIS implementation of the coupling modules. And similarly for any other libraries that might or might not be present.

Design by doing:
Write simple atmosphere and ocean "models" on the same grid as the 30x30 nextsim test. They provide the same data as used in that test, except the ocean responds to the simple presence/absence of ice.
Write sufficient interface to couple these together without hard-coding anything.
(Optional) throw it all out when you realize it doesn't work as hoped. Redo from Start.

Rename the AtmosphericState and OceanState update() functions so that one class can implement both.
Move the calls to these update*() functions from deep in the IceGrowth call tree to be the first thing that the PrognosticData update() function does.
Add a data export Component which can be implemented by the coupling class or by a dummy class.
What OASIS initialization needs to go where? The coupling class needs a configure() function or something like it.

A possible design:
If OASIS is selected in the configuration, then there is an OASIS class that performs all the necessary configuration. It also selects the OASIS implementations of IDataExport, IAtmosphereState and IOceanState and assigns them over any other configured classes for these modules.
Since the key functions are the get and put functions, which require very little state from the central OASIS class, there is little point in a single coupler class that does everything.
The OASIS implementations will include almost all their code within a conditional preprocessor directive. If OASIS is not compiled, the classes will be almost entirely non-functional but still selectable as implementations of IDataExport, IAtmosphereState and IOceanState. I really wouldn't recommend using them though.
Other future couplers would then consist their own implementations of the relevant functions along with any classes needed to set up and communicate with the external coupler program.
The Coupler class is the only one that CMake sees. The code within is included or not, based on preprocessor directives, allowing the inclusion of exclusion of the OASIS and any other coupler code. The actual code can be stored in individual coupler specific source files that are included as requested by the pre-processor.
In Model::configure() call Coupler::configure(). This (after the grid is set up from the restart file) uses the MPI information to configure Oasis and its grids. This information is then available for the OASIS implementation classes.
The OASIS implementations themselves will require selection in the configuration files as the implementations of the relevant interfaces. This allows coupling to only one of an atmosphere or ocean model. The OASIS configuration module should select the OASIS implementation of the DataExport class if at least one of the model coupling implementations are selected.

Resolving issue #154 will impact this issue.


init event, html support for dictDefaultMessage

Hi again)
I've made some changes you may find useful.

With  the current version of the template Angular will escape dictDefaultMessage, so it is impossible to create custom html layout. [innerHTML] fixes that.

According to documentation, .dz-text in template is not necessary.

In some cases it is very useful to access dropzone instance from parent component. init event exposes it.

I think it's also worth thinking about refactoring code repeat in directive and component as they are much the same.

Yes I plan to refactor the code, but atm angular does not support host directive which I think would make the code much nicer instead of making component for the directive. So I decided to wait a bit and see if host directives will be supported in near future.
What is the benefit of having the init event to expose the dropzone vs getting it via ViewChild?
Also what is the benefit of having the separate html for dictDefaultMessage? Why just not add the innerHTML for the div?
Don't get me wrong I am happy to make the changes, but I always want to keep the code to minimum and as clean as possible so I want to understand if there are some benefits with my take on those changes would be bit different.

You're right, init event is unnecessary, I forgot about @ViewChild.
What is the proper way to provide custom markup without refactoring the component template? All markup is escaped when I pass something like
test
 to dictDefaultMessage or message. I've found that it's possible to pass unescaped markup via [innerHTML] : http://stackoverflow.com/questions/34585453/how-to-bind-raw-html-in-angular2
Thank you!

Yes I can change the way the message / dictDefaultMessage is used to the innerHTML I just meant that I dont think there needs to be alternate template since it can be done in one div. I will release new version today which allows using html in the message. Thanks for bringing that into my attention since I did not think at all that naturally someone might want to use html in the message.

I close this PR now, todays release will allow usage of html in the message and dictDefaultMessage.

Great, thank you!

Will have to delay new release by day or two since I am bringing support for latest angular-cli also and most likely for AOT at the same time.

I just release 2.0.0 which will allow html in dictDefaultMessage. It also has new build system which should produce aot compatible library to be used with latest angular / angular-cli. Also now there is a minified umd bundle version included. I tried to test this setup with as many different build systems I could, but if you run into problems then let me know.

This is great, thank you! I will test it next week and of course let you know if any bugs will appear.


Uncaught Error: EPERM: operation not permitted, lstat 'C:/Users\admin'

[Enter steps to reproduce:]

...
...

Atom: 1.26.0 x64
Electron: 1.7.11
OS: Microsoft Windows 10 Pro
Thrown From: Atom Core
Stack Trace
Uncaught Error: EPERM: operation not permitted, lstat 'C:\Users\admin'
At fs.js:961

Error: EPERM: operation not permitted, lstat 'C:\Users\admin'
    at fs.lstatSync (fs.js:961:11)
    at Object.fs.lstatSync (ELECTRON_ASAR.js:244:16)
    at Proxy.realpathSync (fs.js:1641:21)
    at Proxy.fs.realpathSync (ELECTRON_ASAR.js:321:29)
    at atom.project.getPaths.map (~/AppData/Local/atom/app-1.26.0/resources/app/node_modules/fuzzy-finder/lib/path-loader.js:16:73)
    at Array.map (native)
    at Object.startTask (~/AppData/Local/atom/app-1.26.0/resources/app/node_modules/fuzzy-finder/lib/path-loader.js:16:56)
    at Object.startLoadPathsTask (~/AppData/Local/atom/app-1.26.0/resources/app/node_modules/fuzzy-finder/lib/main.js:129:43)
    at get_process.nextTick (~/AppData/Local/atom/app-1.26.0/resources/app/node_modules/fuzzy-finder/lib/main.js:31:45)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)

Commands

Non-Core Packages
Hydrogen 2.4.1

Thanks for contributing!
We noticed that this looks like a duplicate of https://github.com/atom/fuzzy-finder/issues/289 so you can subscribe there if you'd like.

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Update Causes Unknown function: unique alongside tsserver

I just updated lsp_signature and am now running into issues in buffers to which I attach tsserver (no such problem with other language servers).
When I enter insert mode, I get the following error:
Error executing vim.schedule lua callback: Vim:E117: Unknown function: unique
stack traceback:
        [C]: in function 'unique'
        ...er/start/lsp_signature.nvim/lua/lsp_signature/helper.lua:513: in function 'chec
k_lsp_cap'
        ...cker/start/lsp_signature.nvim/lua/lsp_signature/init.lua:440: in function 'sign
ature'
        ...cker/start/lsp_signature.nvim/lua/lsp_signature/init.lua:556: in function 'cb'
        vim.lua:285: in function <vim.lua:285>

Any thoughts / suggestions would be greatly appreciated. Thank you!

Me too.
:version
NVIM v0.7.0-dev+751-g67bb01ae2                                                                                Build type: RelWithDebInfo                                                                                LuaJIT 2.1.0-beta3                                                                                Compilation: /usr/bin/gcc-11 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/runner/work/neovim/neovim/build/config -I/home/runner/work/neovim/neovim/src -I/home/runner/work/neovim/neovim/.deps/usr/include -I/usr/include -I/home/runner/work/neovim/neovim/build/src/nvim/auto -I/home/runner/work/neovim/neovim/build/include                      Compiled by runner@fv-az65-852                                                                                Features: +acl +iconv +tui                                                                                See ":help feature-compile"                                                                                system vimrc file: "$VIM/sysinit.vim"                                                                                fall-back for $VIM: "/share/nvim"

I can confirm the problem. Seems to have been introduced in the last commit, since reverting to this commit resolves the problem for me:
    use({
      "ray-x/lsp_signature.nvim",
      config = function()
        require("plugin-configs.lsp_signature")
      end,
      commit = "be39dacc17d51531f9e3a50f88de0a45683c6634"
    })

That commit works fine for me too.

thanks for submitting the issue. Should be fixed.

Confirmed, it's fixed! Thank you @ray-x! & might I say, AWESOME PLUGIN!


Feature Request: Adjust implicit build_for in schemes

Basically, I want it so this logic-flow works:

Need to launch it? Add a launch_action. It will implicitly get a profile_action as well. If that isn't desired, set None for profile_action.
Need to have tests run? Add a test_action and include it in the list of targets.
Need it to build while testing, but you don't want it in the test lists? Add a build_action and include it in the list of targets. If the target isn't a test target, use build_target instead of a string and set build_for to build_for(testing = "enabled") as well.
Need it to build normally (not when testing)? Add a build_action if it doesn't exist and include it in the list of targets.
Want to adjust which phases a target builds for? Add a build_action if it doesn't exist and add a build_target with build_for set to the list of build_action.targets.

Which it will be a (small) breaking change for this to go in after 1.0, I don't think it's important enough to delay the release.

The new xcshemes API which is used with incremental generation mode obsoletes the need for this change.


Update smetrics to 1.0.6

Updates com.evolutiongaming:smetrics from 1.0.5 to 1.0.6.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.evolutiongaming", artifactId = "smetrics" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "com.evolutiongaming", artifactId = "smetrics" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #170.


Not upload any file and how mount as fs?

From rclone:
2023/12/23 18:13:36 Failed to move: HTTP error 413 (413 Request Entity Too Large) returned body: "<html>\r\n<head><title>413 Request Entity Too Large</title>... etc
From UI

I use one docker-compose with DB.

UPD
I just succeeded in uploading a 100kb photo via rclone, but a 9mb text file fails, error above

By the way, does TelDrive cache files somehow? I have 20gb free on my server, I wouldn't want to run out of space

Why when i try watch 4G video via rclone mount?
2023/12/23 19:52:10 ERROR : IO error: HTTP error 520 (520 ) returned body: "error code: 520"

And how i can mount via fstab or systemd? Thanks and sorry for many messages

The update fixed the problems, thanks


Merge release -> master

Build in pipeline check aborted.

The code coverage is the same after your change 🤟
This is a good thing if you are not sure ;)
Report (compared to build #297)
Covered lines: 21.97%, change: 0%
Covered branches: 34.25%, change: 0%
Coverage report
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/301/artifact/ci/results/coverage/lcov-report/index.html

Build succeeded (gate pipeline).

https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-merge-compliance-check/119/ : SUCCESS in 44s
https://checklist.intgdc.com/job/client-libs/job/gooddata-create-gooddata-react-app-unit-tests-zuul-docker-el7/301/ : SUCCESS in 1m 45s
https://checklist.intgdc.com/job/compliance/job/gooddata-create-gooddata-react-app-fossa-scanning-zuul-el7/84/ : FAILURE in 4m 06s (non-voting)


TERM/INT signals do not propagate to children

Hi,
I think I found a severe issue in Angel:
Steps to reproduce:

run angel with a single arbitrary process
send a parent angel either TERM or INT signal (^C won't work because it sends INT to the whole process group)
watch the parent orphan its children

% pstree -p $(pgrep angel | head -1)
-+= 00001 root /sbin/launchd
 \-+= 00341 vladki /Applications/iTerm.app/Contents/MacOS/iTerm -psn_0_40970
   \-+= 00453 root login -fp vladki
     \-+= 00456 vladki -zsh
       \-+= 03508 vladki dist/build/angel/angel -v2 stubborn.conf
         \-+- 03509 vladki dist/build/angel/angel -v2 stubborn.conf
           \--- 03510 vladki test/test_jobs/StubbornJob
% kill -TERM 3508
% pstree -p $(pgrep angel | head -1)
-+= 00001 root /sbin/launchd
 \-+- 03509 vladki dist/build/angel/angel -v2 stubborn.conf
   \--- 03510 vladki test/test_jobs/StubbornJob

Extended logs:
[2015/05/21 14:48:43] {main} INT | TERM received; initiating shutdown...
[2015/05/21 14:48:43] {main}   1. Clearing config
[2015/05/21 14:48:43] {main}   2.

Anticipated cause:
Angel tries to modify a TVar inside forkProcess that obviously doesn't modify a parent process' TVar.
A current workaround is to simply kill the whole process group of Angel, but this is definitely not the cleanest solution.

The same issue applies when you modify the config and remove processes from it (they never get stopped)

I have not had much time to look into it, but I have managed to replicate the issues that you have decribed.  I will try to give some initial feedback this weekend.

So there are three things that appear to be contributing to this issue.

onValidHandleAction is trying to register the process handle(s) from the supervisee, this call is failing to register in the main configuration because it is called via a new process it should be the process id of the forked process that is actually registered as this is the controlling process for the supervisee
the forked process does not correctly terminate the supervisee (and the log process is there is one) it should take responsibility rather just trying to store them in the current configuration
the forked process does not respond to TERM and INT.  The new forked process needs to have handlers like in Main.hs, so that it can handle the proper termination of the supervisee as in point 2.

All these points should addess the HUP configuration re-read problem as well.
@proger I did not find this whilst testing because I was relying on Crtl-C and did not realise that it appeared to be working because of the termination of the Process group.
@MichaelXavier what is the best way to proceed how about a new issue branch so that I can send a pull request and then get feedback from @proger
It will take me a few days due to other work commitments, but I have a good handle on a fix.
@proger Thanks for taking the time to report this issue.

@ignesco Thanks for the writeup. Sounds like you have a plan. I'd definitely accept this as a PR. That's how I've been doing it as well, letting the person who files the issue test it out and giving it a week or so for feedback before merge. The only thing I'll add is that if you can demonstrate this in the test suite, having a test for this would be great. Let me know if you have any further questions, and thanks again for taking on the work!

@MichaelXavier I booked the day off today to finialise the work for this issue and I coded up a solution in about an hour or two, and it worked for the simple test case, but when I stress tested it, it has hilighted a much more complicated and tricky side effects.  I have spend most of the day trying to get to the bottom of what the problem is but to  no avail.  When count is greater than 1 for particular configuration, it has the very bad effect that the forkProcess function does not actually fire for any more that a couple of instances in my test of count = 10, at the moment I am tempted to say it is GHC and forkProcess not playing with multiple processors (as per the warning in its hoogle page) or just plain old lazy evaluation but as to which one, I'm not sure.
So the short of it is that I do not think my fix will work for anything but the simplest case.
At this stage for the sake of stability, I would recommend backing out the "run as user" change until I can come up with a more robust solution to forkProcess not working, what do you think?  I don't mind do the backing out too, having proposed and implemented it in the first place I feel that I should clean up too.
Can you let me know what you would like me to do?
Thanks,
Craig

@ignesco thanks for putting your effort into it!
As a workaround, you could also wrap the command into sudo -u or something like that while you're looking for the nicer fix.

@ignesco Sorry about the delay. You caught me right at the beginning of a vacation. I've given you commit access to the repo, since it would seem you use it in production more than I do. I'd say in general to use your best judgement with the following values in mind:

Stability is important. We should avoid breaking changes.
We should be aiming for the most robust production behavior we can. I'd rather avoid pushing a feature that we're iffy on. By all means, if there's a question as to what angel should do in a particular instance, we should be guided by the principle of least surprise and if possible, copy the behavior of the other venerable process monitors.

For this particular instance, sounds like backing out the change is the right thing to do until you're really confident in it. I'm sure you'll use good judgment here. Thanks!

@MichaelXavier I have pushed an issue branch for the initial fix for this issue (40-term-int-propogation-01).  I opted for the the interim suggestion of @proger of "sudo -u" do you think this would this be OK?  Also the test for "forcefully kills stubborn processes" seems to fail on my machine, but it was failing in master SHA1-ID: d5e3e199223 would you still be OK with this test failing?  Also if there anything I need to do with Travis to get it to run tests for this branch?
@proger I got side tracked with trying to get to the bottom of the issues that I saw whilst trying to fix this issue.  To get things moving again, I have coded the removal of forProcess for now, with your suggestion of "sudo -u".  The changes are in the branch (40-term-int-propogation-01).  If you get a chance can you try these latest changes to confirm that it fixes the original issue that you were experiencing?
On a side-note with the issue that I saw whilst trying to fix this issue, I think I have narrowed it down to something to do with the child processing seein stdout as being locked so it just waits ad infinitum and cannot respond to any of the parent's signals.  I have some other things to try around this, so I'll continue with trying to get it to work.

@ignesco Seems like the tests actually ran and passed for everything except for 7.10.1: https://travis-ci.org/MichaelXavier/Angel
Its probably a travis file issue. I'm not too worried about it. I say that between you and @proger you have a pretty good idea of the issue at hand so once you get it solved to your satisfaction, I'll be happy to merge it. Using sudo still feels weird to me. The possible issues are either env issues where Angel won't be able to find sudo or possibly systems where sudo doesn't exist. Does Mac have sudo?

@MichaelXavier by suggesting sudo I mean it should not be a feature of the app per se, a user can just use sudo whenever one feels comfortable (we're chpst from runit, for example).

@MichaelXavier and @proger , I have tried lots of different options for fixing this issue, non of which have actually resulted in anthing that I could have confidence in.  After considering other options to get this issue resolved, the two steps that I think are now most viable are:

Remove the forkProcess function as I consider this to be the root cause
And either...
a) Choose a more global approach and use setuid to set the user id across the whole process on initial start up, reading the user from the command line options or from the configutation file.
b) Just remove the "run as a different user" functionality

So what does everyone think?  @proger you said your system deployment uses chpst, would it be beneficial to you to be able to do this directly using just Angel or would you just stick with chpst anyway?  Personally I would like this functionality to be embeded in the Angel config file because backing the config file up would be backing up more of the overall system deployment configuation.
Thanks for waiting on this issue, new baby took presidence.
Craig

@ignesco I think you've got the right idea here. Having the config file be the one source of truth seems to be a good goal. As for which of a/b to choose, I'll defer to you guys. I'm not currently using this project in production and value the opinion of people using it in earnest most. I don't have too much time to devote to this project now so I'm happy to let you guys decide what is best and support where I can.

You may be interested in using https://github.com/haskell/process/pull/45 instead of forkProcess

Hi All,
Sorry I have been off grid for a while. @proger the https://github.com/haskell/process/pull/45 still has all the issues with the locking stdout problem.  My plan is to finish off the implementation leaving in the command line argument option and implementing the suggested configuration file option. This way anyone using the current version does not have a nasty surprise. Hopefully, if there are no objections we should be able to close this issue very soon.
Thanks for all your patience,
Craig

Hi All,
I have pushed a potential fix for the original issue, whilst keeping the flexability for the user to change the executing user, with both -u and user= options. the can be found here
https://github.com/MichaelXavier/Angel/tree/40-term-int-propogation-02
@proger would it be possible to retest this issue with this branch?  @MichaelXavier and @proger please let me know your thoughts and hopefully we can get closer to fixing this issue.

Hi All,
Is there someone that could review this fix before I push it to the master branch.  Would be nice to get this in as it has been an issue for some time now.  I was hoping to push my this coming Sunday 2016-09-04  Many thanks.

@ignesco I don't really have much time for this project and I'm not actively using it right now. The build failure looks trivial though so I wouldn't worry about. This project could probably be set up to use stack so different GHC versions can be used easily.
BTW if any of you are actively relying on Angel and would like commit bit let me know and I'll make it so.

Thanks for everyone's patience with this one - this issue is fixed with commit SHA1-ID<PHONE_NUMBER>3ca4f34310c1b9e74f26bf565dac0f


[CP] [skwasm] Fix platform view placement.

Issue Link
https://github.com/flutter/flutter/issues/146539
Target
stable
Cherry pick PR Link
https://github.com/flutter/engine/pull/54870
Changelog Description
Fixes platform view placement when using the WASM renderer
Impacted Users
Web apps compiled to WASM using platform views
Impact Description
Apps using platforms views are broken when compiled to WASM
Workaround
Supposedly delaying the rendering of platform views fixes this, but I have been unable to get that to work in any of my own projects
https://github.com/gskinnerTeam/flutter-wonderous-app/commit/575304b849a230f367176a28297aff642aae6b4b
Risk
low
Test Coverage
yes
Validation Steps
Run the camera plugin example on web compiled to WASM

Approved by @eyebrowsoffire in https://github.com/flutter/engine/pull/54870

@itsjustkevin Was this one included in 3.24.3? I don't see it in the changelog.

If this is included in 3.24.3 it's not working

@eyebrowsoffire any thoughts on what could cause this fix to not work?

@Rexios80 this was included in the hotfix release.
If there is still an issue, it is no longer related to this cherry-pick.  Please open another issue if it still persists.

I confirmed this still works on master. There must be an issue with the cherry pick. How do we proceed from here?


Default values

ResourcesApi.register_setting(name, options) and ModuleSettings.static register() should have empty as default value

@plaasmaandraax I don't think I understand what you're trying to say. If you're referring to the default values that get passed into game.settings.register() from the methods you mentioned, then I believe these to be valid as is?

Closing this until further notice.

I get some bugs without these defaut values, in my specific case. I can't remember which ones today, but I define them explicitly


Update Eclipse Installer to Mars.1

Thank you for the contribution. It was merged directly as 27e5880e1ac2e53f96cae59ba74950345a023d2d to keep commit history cleaner. Your contribution is still credited to you.

Hi @vitorgalvao, have you really merged this? ;)

Looks like something went wrong, thanks for the notice. Will fix shortly.

Thanks for merging!
To keep history: the new commit id is 8fc41f265c9983325cda7cc1d5db65dec61d141d.


Replace request and then-sleep dependencies

This changes request to node-fetch.
And also then-sleep to a one-line function sleep.
Resolves out of date dependencies brought up by @bnb upon npm install.

@leo if the one liner was published as an npm module, would that be acceptable? Currently then-sleep adds 37 dependencies for something that can be published as a single line 😬

if the one liner was published as an npm module, would that be acceptable?

@bnb Absolutely! That's what I thought we'd get. 🙏

@leo I replaced my sleep function for delay, which is a slightly larger version of what I wrote but it has zero dependencies.
https://packagephobia.now.sh/result?p=delay


When you try to click, but scroll a little by mistake, your click doesn't register

I am experiencing weird behaviour with my mac mouse, because its too easy to scroll by mistake few pixels while trying to click.
It can be observed here in the demo too:
https://bvaughn.github.io/react-virtualized/#/components/ArrowKeyStepper
I guess when you scroll by mistake even 3 pixels, things rerender and your mouse isn't over anything until it moves again.

I believe this is inherent browser behavior. If you click the link you posted and drag it 3px "accidentally scroll' it won't register your click
If this is a QA raised issue RIP


[ci:component:github.com/gardener/machine-controller-manager:v0.50.1->v0.51.0]

Release Notes:
Phase transition diagram for a machine object is added to FAQs

machine controller won't reconcile machine on non-spec update events

fixed IT for seed with k8s >= 1.27 as control cluster

MCM restart happens properly in integration tests now. This fix will get activated, once this version is vendored in your mcm-provider

Fix for edge case of Node object deletion missed during machine termination.

The default `machine-safety-orphan-vms-period` has been reduced from 30m to 15m.

Removes `node.machine.sapcloud.io/not-managed-by-mcm` annotation from nodes managed by the MCM.

Change OCI Image Registry from GCR (`eu.gcr.io/gardener-project`) to Artifact-Registry (`europe-docker.pkg.dev/gardener-project/releases`). Users should update their references.

Bump `k8s.io/*` deps to `v0.28.2`

/kind enhancement
/cc @elankath @rishabh-11


Avoid duplicating maven-javadoc-plugin configuration and CSS files

Check List:

Fixes N/A
Unit tests : NA
Javadoc with a code example (on API only) : NA
PR meets the contributing guidelines

Following the contributing guidelines will make it easier for us to review and accept your PR.
Summary
The assertj-core and assertj-guava modules contain identical Javadoc CSS files and almost identical  maven-javadoc-plugin configuration. This PR removes the duplication from poms and moves CSS files to another directory to allow them to be referenced in sub-modules.

Thanks, @armandino!


Use protocol or NSNumber or generic gymnastics to allow generic numerical type

There's a few ways to go about this.  Leaning toward NSNumber for now.


Pod perceiver: unable to update annotations/labels for pod -- privileged

time="2018-03-13T15:44:29Z" level=error msg="unable to update annotations/labels for pod default:sti-ruby-1-build: pods "sti-ruby-1-build" is forbidden: unable to validate against any security context constraint: [provider restricted: .spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed provider restricted: .spec.containers[0].securityContext.volumes[0]: Invalid value: "hostPath": hostPath volumes are not allowed to be used]"

If I understand correctly, this isn't a perceiver issue.  It looks like incorrect permissions on the service account the perceiver is using and not something fixable in the perceiver code base.  Have a reproducer?

.... Whats the hostPath error coming from ?

on phone w/ matt, assuming this is a non issue now.

[10:31 AM] Joyce Lin: "time="2018-10-12T14:30:24Z" level=error msg="unable to update annotations/labels for pod cover-all-deployment-types-script:django-ex-1-build: pods \"django-ex-1-build\" is forbidden: unable to validate against any security context constraint: [spec.volumes[1]: Invalid value: \"hostPath\": hostPath volumes are not allowed to be used spec.volumes[2]: Invalid value: \"hostPath\": hostPath volumes are not allowed to be used spec.containers[0].securityContext.privileged: Invalid value: true: Privileged containers are not allowed]""
[10:32 AM] Joyce Lin: "time="2018-10-12T14:30:31Z" level=error msg="unable to update annotations/labels for pod cover-all-deployment-types-script:ibmjava: pods \"ibmjava\" is forbidden: unable to validate against any security context constraint: []""

some other examples here: https://jira.dc1.lan/browse/OPSSIGHT-449


New Project Added https://github.com/swapnilsparsh/30DaysOfJavaScript…

…/issues/4

@yash06112002 Add your project to the official website also.

@yash06112002 Thank you for your contribution. Looking forward to more such amazing contributions 🥂


Support isort >= 5

Hi. I hope this will be usefull.

ping @paradoxxxzero

Any word on reviewing and merging this PR?

+1

+1

Would be great to get this in

@paradoxxxzero, are you waiting for something?

So sorry about the delay, I've been quite busy recently.
If someone want to take maintenership just ask :)

@paradoxxxzero Thanks for merging! Any idea if this could be absorbed into https://github.com/palantir/python-language-server? It seems like they already host plenty of plugins there, both isort and mypy frankly seem like pretty odd exclusions.


Language Pack/Font?

Hi there, I'm struggling to get the folder and filenames in the zip file to display properly. What language pack and/or fonts are being used? I have the language pack installed properly and other files on my system display the Chinese characters fine, but the zip file's contents are garbled for me.

I think the first thing you can try is to use third-party software (like 7-zip, bandizip) to unzip the file.

Just tried this with 7zip, still no luck.

Just tried this with 7zip, still no luck.

I guess the problem was caused by the default encoding setting on Windows, which is GBK on my windows computer and I think this is not the same on yours.
I am not very sure about the solutions, but there is something you can try:

Use the new zip files. The new files are generated on macOS, and encoding problem should be fixed.

link for the first dataset (character as top level): https://drive.google.com/file/d/1k849yUZhkUfbupZT0kRR2ZzZj5g89yLw/view?usp=sharing
link for the second dataset (calligrapher as top level): https://drive.google.com/file/d/10QJrw0Qdk4O1bIrehCLmdiCkwpLbGVe8/view?usp=share_link

Changing encoding in the archiver (only for bandizip because 7-zip does not support it). You can change it by clicking this button in the software and choose Simplified Chinese:

Looks like re-encoding it via macOS did the trick, the new zip file from Google drive works! Thank you very much @zhuojg!


Port donuts and Donut boxes

About the PR

Ports the donuts and donut boxes from CM-13
Why / Balance

Parity
Technical details

just YML
Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no cl just sprit

done


Wrong url in driver

Sometimes I receive different URL addresses in response and driver.
self.logger.warning("Request response url: %s" % response.url)
self.logger.warning("Request driver url: %s" % response.meta['driver'].current_url)

And more than that, driver url sometimes duplicates urls from other responces (at the same time response urls are unique).

Oh, I understand. It's a bad idea to share a single instance of WebDriver because parse happens parallel with new requests.
response.meta['driver'].find_element_by_css_selector('button.btn.btn-primary').click()
response = response.replace(body=response.meta['driver'].page_source)

It's how I tried to use driver. It leads to a situation when response will be replaced with body from other url (which is in the driver at the moment)

Hi @Tamplier,
Thanks for opening this!
What you experiencing looks similar to #21, and you are right about the single instance of the webdriver.
Exposing the driver in the response meta is not working as expected since many requests are happening at the same time. Unless I find a workaround (creating one webdriver for each parallel request?) I think I will have to stop exposing the driver from the meta.
Another problem from another issue is that since only one instance of webdriver is created, it can slow down the entire Scrapy request / response processing...
I don't have any solution right now for theses problems, but I am currently working on another project who is at least solving the second one: https://github.com/clemfromspace/scrapy-puppeteer (Fully asynchronous webdriver using puppeteer instead of Selenium).

I'm facing the same problem. Any updates?

I am facing the same issue. I created an issue before I found this one, so I'm going to link it in order to focus the discussion about the topic: https://github.com/clemfromspace/scrapy-selenium/issues/111


Move to es6

Summary

Upgrades all code from ES5 to ES6, where possible.
Non-breaking.
Linked issue(s)

#54 and #5
Involved parts of the project

Every js file, including tests, plus .eslintrc and package.json
Added tests?

Added npm scripts lint and lint:fix which can now be used in ci as well
OAuth2 standard

No changes in behavior
Reproduction

$ git checkout move-to-es6
$ npm run lint
$ npm run test

@HappyZombies Can you approve this?

Approved!


Socket connection failed "/tmp/CO_command_socket": No such file or directory

Dear All,
I have received following error, when I tried to run cocomm program,
$ ./cocomm "set node 4"
Socket connection failed "/tmp/CO_command_socket": No such file or directory

I have tried to create a /tmp/CO_command_socket directory and received same error message again,
Do you have any idea?
I have used peak_USB device and I used socketCAN with following driver,
$ sudo modprobe peak_usb
$ ifconfig
can0: flags=193<UP,RUNNING,NOARP>  mtu 16
        unspec 00-00-00-00-00-00-00-00-00-00-00-00-00-00-00-00  txqueuelen 10  (UNSPEC)
        RX packets 2  bytes 9 (9.0 B)
        RX errors 0  dropped 2  overruns 0  frame 0
        TX packets 0  bytes 0 (0.0 B)
        TX errors 0  dropped 0 overruns 0  carrier 0  collisions 0

Could you guide me?
Thanks,
Best Regards

I have tried below command, and received following messages:
$ nc -U /tmp/CO_command_socket
nc: unix connect failed: No such file or directory
nc: /tmp/CO_command_socket: No such file or directory

I have cloned the CANopenLinux repository  and cd CANopenLinux and make and make install,
When I try cocomm "help" it gives the title of this issue, but "cocomm --help" command works,
is it about virtual CAN interface ? Do I need virtual CAN interface I do not think so but I think I received socket error!

Thank you for your reply,
It works in local ubuntu PC,
What about remote system?
I have planed to run canopen can0 -i 1 in the remote linux system, I have used peak_USB devices for CAN network,
How can I send data, messages from local ubuntu PC to remote linux system?
Will "cocomm" be available then?
Thank you again,
Best Regards


Support Proxy Information for Docker

I've run across an issue where, when behind a proxy, the Docker deployment does not function as expected. This is because there is currently no way in these playbooks to set the HTTP/HTTPS proxy information in /etc/sysconfig/docker. When I do this manually, the docker host is able to pull from the hub, quay.io, etc. as expected. I will create a pull request to add this functionality.

Hi @rsmitty, indeed we never faced the issue as we've never been behind a proxy.
We'll think about a solution. I'll try to get back to you soon


invert b-spline transformation

Hi, I have registered some images with the following parameter map vector :
elsx = sitk.ElastixImageFilter()
pvect = sitk.VectorOfParameterMap()
pvect.append(sitk.GetDefaultParameterMap('affine'))
pvect.append(sitk.GetDefaultParameterMap('bspline'))
elsx.SetParameterMap(pvect)

I managed to invert the affine transformation manually (by reading the transform parameter file and inverting the matrix given by elsx.GetTransformParameterMap()[0]["Transform Parameters"]).
Yet, I can't manage to invert the bspline transform.
Any idea how I could do it ? thx

I have tried out two other methods but neither of them lead to decent results :
Image A is fixed. I want to register image B on image A with a b-spline transform and inverse this b-spline to apply it to image A's segmentation mask.

method 1

set image A as fixed image
set image B as moving image
register image B on image A with a b-spline : this gives image C
save the parameters of this transform with sitk.WriteParameterFile(elsx.GetTransformParameterMap()[0], bspline_save_path)

So as to inverse this b-spline I do the following :

set image B as the fixed image
set image C as the moving image
register image C on image B with another b-spline transform : this gives image D
save the parameters of this transform with sitk.WriteParameterFile(elsx.GetTransformParameterMap()[0], inverse_save_path)

when comparing image D and image B, they do not look similar enough to consider that the last b-spline transform is the inverse of the first one...

method 2

I tried to do it manually as it is (very quickly) explained in the elastix manual (section 6.1.6) :

set image B as fixed image and as moving image
create a new b-spline parameter map p = sitk.GetDefaultParameterMap('bspline') which will later contain the inverse of the b-spline function used to register image B on image A
set the original b-spline that we aim to inverse as the initial transform with p['InitialTransformParametersFileName'] = [bspline_save_path]
set p['Metric'] = ['DisplacementMagnitudePenalty'] and p['HowToCombineTransforms'] = ['Compose']
selx.Execute()
get the result parameter map with inverse_bspline = selx.GetTransformParameterMap()[0]
set the initial transform in this last parameter file to "NoInitialTransform" : inverse_bspline["InitialTransformParametersFileName"] = ["NoInitialTransform"]
apply this transform to image C : it gives image D

here again, images B and D are too different to consider we found the inverse...


Missing user guide

Cannot locate full user guide. Can you please provide a path. Thanks

Hi @sumwar, the user guide has been migrated into the wiki. Click on the Wiki tab above or here's a direct link: https://github.com/t-wolfeadam/AnyLogic-Pypeline/wiki
From there, you can navigate the pages on the right.
In retrospect I should've added a comment to the main readme about this; I'll do that now 😅

@t-wolfeadam, thanks for posting that update. This is a great addition to AnyLogic.


Trouble with getting Synapse to Start

Hi,
I've been playing around with ZooKeeper + Nerve + Synapse to try and get a hello_world type setup working. I got Nerve to register a hello_service to register to ZooKeeper with the path /nerve/services/userslist/services. I'm having some trouble with setting up Synapse and HAProxy to abstract the connection to that service for my hello_world app though and that's probably from me not understanding how to configure everything properly.
The setup:
hello_world:
listening: localhost:8000
Originally directly hits http://localhost:8200/users to get some dummy data
Trying to set it up to hit http://localhost:3213/<endpoint> instead

hello_service:
listening: localhost:8200
Serves up a generic json response at /users

ZooKeeper:
listening: localhost:2181
path with the service registered: /nerve/services/userslist/services

I start off by launching ZooKeeper, hello_service, hello_world, and Nerve. I can do a get /nerve/services/userslist/services in the ZooKeeper CLI and see that Nerve has hello_service in there.
When I try to start Synapse with the configuration below, I get this response and it stops at the last line:
synapse -c /root/src/synapse/synapse.conf.json
I, [2015-06-14T18:17:59.255979 #2046]  INFO -- Synapse::Synapse: synapse: starting...
I, [2015-06-14T18:17:59.256148 #2046]  INFO -- Synapse::ZookeeperWatcher: synapse: starting ZK watcher users @ hosts: localhost:2181, path: /nerve/services/userslist/services
I, [2015-06-14T18:17:59.256208 #2046]  INFO -- Synapse::ZookeeperWatcher: synapse: zookeeper watcher connecting to ZK at localhost:2181
I, [2015-06-14T18:17:59.274773 #2046]  INFO -- Synapse::ZookeeperWatcher: synapse: discovering backends for service users
I, [2015-06-14T18:17:59.277248 #2046]  INFO -- Synapse::ZookeeperWatcher: synapse: discovered 1 backends for service users
I, [2015-06-14T18:17:59.277395 #2046]  INFO -- Synapse::Synapse: synapse: regenerating haproxy config

It looks to me like it's trying to generate an haproxy.cfg file in the path specified in "config_file_path" in synapse.conf.json. I created an empty haproxy.cfg in the path specified but Synapse still gets stuck there. I can also see that Synapse is running but HAProxy isn't in the background. I've tried starting HAProxy first before Synapse but I still get the same results. I also noticed that /var/haproxy for stats.sock didn't exist so I created that folder. The .sock file didn't get created though when I ran Synapse. I've also tried hitting localhost:3212 and localhost:3213 and they both return with a connection refused error.
My synapse.conf.json is pasted down below. I'm doing this testing on an Ubuntu 14.04 Docker image. My HAProxy version is 1.4.24 and was installed before Synapse. Any help or suggestions would be greatly appreciated!
{
  "services": {
    "users": {
      "default_servers": [
        {
          "name": "default1",
          "host": "localhost",
          "port": 8200
        }
      ],
      "discovery": {
        "method": "zookeeper",
        "path": "/nerve/services/userslist/services",
        "hosts": [
          "localhost:2181"
        ]
      },
      "haproxy": {
        "port": 3213,
        "server_options": "check inter 2s rise 3 fall 2",
        "listen": [
          "mode http",
        ],
        "frontend" : ["mode tcp"],
        "backend" : ["mode tcp"]
      }
    }
  },
  "haproxy": {
    "reload_command": "service haproxy reload",
    "config_file_path": "/root/src/synapse/haproxy.cfg",
    "socket_file_path": "/var/haproxy/stats.sock",
    "do_writes": false,
    "do_reloads": false,
    "do_socket": false,
    "global": [
      "daemon",
      "user haproxy",
      "group haproxy",
      "maxconn 4096",
      "log     <IP_ADDRESS> local0",
      "log     <IP_ADDRESS> local1 notice",
      "stats   socket /var/haproxy/stats.sock mode 666 level admin"
    ],
    "defaults": [
      "log      global",
      "option   dontlognull",
      "maxconn  2000",
      "retries  3",
      "timeout  connect 5s",
      "timeout  client  1m",
      "timeout  server  1m",
      "option   redispatch",
      "balance  roundrobin"
    ],
    "extra_sections": {
      "listen stats :3212": [
        "mode http",
        "stats enable",
        "stats uri /",
        "stats refresh 5s"
      ]
    }
  }
}

If you want synapse to write the config, or modify the running HAProxy you'll want to set do_writes, do_reloads, do_socket appropriately.
So for a basic proof of concept I'd set all three to true.

Awesome. Thanks @jolynch. That got me past this!


validate duration to be positive

This PR did validate duration to be positive
When using ./bin/swarmctl service create --name=aa --image=redis --update-delay=-100s and other negative flags,
Output will be :
root@ubuntu:~# ./bin/swarmctl service create --name=aa --image=redis --update-delay=-100s
Error: TaskSpec: update-delay cannot be negative
root@ubuntu:~# ./bin/swarmctl service create --name=aa --image=redis --restart-window=-100s
Error: TaskSpec: restart-window cannot be negative
root@ubuntu:~# ./bin/swarmctl service create --name=aa --image=redis --restart-delay=-100s
Error: TaskSpec: restart-delay cannot be negative

This PR did:

validate heartbeatperiod in the client;
2.validate restart-window, restart-delay, update-delay in daemon side;
3.add unit tests for service

Signed-off-by: allencloud<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 54.44%

Merging [#909][cc-pull] into [master][cc-base-branch] will increase coverage by 0.01%

2 files (not in diff) in protobuf/plugin were modified. more

Misses +3
Partials -6
Hits +3

2 files (not in diff) in manager/orchestrator were modified. more

Misses +1
Partials +1
Hits -2

1 files (not in diff) in manager were modified. more

Misses -1
Partials -1
Hits +2

File agent/task.go (not in diff) was modified. more

Misses +1
Partials 0
Hits -1

@@             master       #909   diff @@
==========================================
  Files            75         75
  Lines         11289      11289
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6144       6146     +2
- Misses         4288       4292     +4
+ Partials        857        851     -6

[][cc-pull]

Powered by Codecov. Last updated by [e5a001d...e636b4b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/e5a001d37e8cb9aa844e98e820a41e8e8cc01c0a...e636b4b6ff2275b025ff270fbeb31413b7f59c0c
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/909?src=pr

Thanks @allencloud !
I think we should probably do those checks server-side to make sure they don't depend on the client

@aluzzardi
It is really strange that CI fails and says that the goftm goes wrong. I am always using sublime plugin to make this.
Can anybody check this for me?

PTAL @aluzzardi

@allencloud

It is really strange that CI fails and says that the goftm goes wrong. I am always using sublime plugin to make this.

The most common cause of this is the lack of -s in the sublime plugin or when using goimports.

@stevvooe @aluzzardi
Actually I once used gofmt -s in command line, still error in CI. After that I changed the test a little by a little and then make it.
Can your review this PR, it validate the input parameters to be positive.

@allencloud thanks for identifying this. We definitely had a type inconsistency here. #1020 is going fix that.
We will still the validation, so one #1020 is merged, rebases and we'll get this merged.
Again, thanks for the keen eye here.

@stevvooe
Thanks for your review.
Rebased and updated.

LGTM
@aaronlehmann PTAL

LGTM

@aaronlehmann @stevvooe
Thanks for your advise. Updated the comparison.


Add parse_json/to_json support for Array

Add parse_json/to_json support for Array

JSON makes sense for a Dictionary, but what would be the use case with an Array?

A few people had this problem, where they get an array from the network
instead of a dictionary. The array usually has dictionaries inside of it. A
lot of standard REST apis use this. The workaround is to wrap your data
with something like "{ "_payload": "+your_json_string+" }" and then
access the _payload key, but if this is a standard json feature, I don't
see why not support it.. Maybe in Marshalls, so you don't have to actually
know the type of object you need to instance to call parse_json on it?
On 2 November 2015 at 11:06, Rémi Verschelde<EMAIL_ADDRESS>wrote:

JSON makes sense for a Dictionary, but what would be the use case with an
Array?
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/pull/382#issuecomment-153025096.

Could we have a decision on this old PR? Do we want to add JSON parsing support to Array, and if so, should @sanikoyes or someone else try to rebase this PR?

I'd rather have a method in Marshalls or somewhere like that which returns
the correct structure (array or dictionary), otherwise you'd have to do
some string hacking to determine if you're supposed to feed your string to
an array or a dictionary to get it parsed. Let the parser do that.
On 4 June 2016 at 06:39, Rémi Verschelde<EMAIL_ADDRESS>wrote:

Could we have a decision on this old PR? Do we want to add JSON parsing
support to Array, and if so, should @sanikoyes
https://github.com/sanikoyes or someone else try to rebase this PR?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/382#issuecomment-223746644,
or mute the thread
https://github.com/notifications/unsubscribe/AGVmPRj5KBN4jeYq9sQfc2s_4YhzBaPmks5qIUe9gaJpZM4B5BKB
.

Would you implement it? We miss your commits in the recent git history ;)

This would be great to have for 2.1.

marshalls is for marshalling, this has nothing to do with marshalling
What actually has to be done is to move parse_json() to String, and a
to_json() in Array()
as this would break compatibility, let's do it for 3.0
On Sat, Jun 4, 2016 at 12:12 PM, George Marques<EMAIL_ADDRESS>wrote:

This would be great to have for 2.1.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/382#issuecomment-223760795,
or mute the thread
https://github.com/notifications/unsubscribe/AF-Z2zeoLXRaX1fpXL54aSTR3m--DcEUks5qIZXygaJpZM4B5BKB
.

@reduz We could always keep Dictionary.parse_json() as a deprecated method until 3.0.

Yeah IMO we don't have to wait for 3.0 for every API change that would break compatibility, we just have to keep the deprecate methods around until 3.0, with a proper warning so that users know beforehand about it. Then most users would have a painless migration to 3.0 if they were using 2.1 and 2.2 before.

This feature will be very usefull for keeping row order intact when comminicating with REST API's to query tabular data.

The PR as is won't be merged, but the good points it raised are now tracked in various issues (especially #5696), so let's close and reduce the PR backlog.

Fixed in 62273e5.


Click on button not taking input from text field?

I'm entering information into a text field then trying to click a button to enter the information, using
     const result = await Runtime.evaluate({
        expression: 'new Promise((resolve) => {document.getElementById("name-field").value = "name"})',
        awaitPromise: true
    });
    result = await Runtime.evaluate({
        expression: 'new Promise((resolve) => {document.getElementsByClassName("submitButton")[0].click()})',
        awaitPromise: true
    });

And after I click the button the input fields either haven't loaded or are wiped out...? (I know this works in the UI I'm pointing it out). So my thought was that I needed to do something to wait until the input is done, but I thought that's what the 'new Promise...awaitPromise' was doing?
If I print a screenshot before I click the button the input is there, and if I print a screenshot after I click it it's no longer there (the form entry fails because it's empty).
Also, neither 'result' yields an error.

Component
Version

Operating system
Ubuntu 18.04.3 LTS

Node.js
12.12.0

Chrome/Chromium/...
78.0.3904.108

chrome-remote-interface
0.28.0

Is Chrome running in a container? NO
P.S. Thank you for this wonderful tool! It's been awesome to work with :D

Hm, well that's weird, it could be that the onsubmit of the form wipes the input field or that its action is set to the same page so the field is empty at the end... hard to say without a working example. Those promises should have no effect. Is the target web application up somewhere so I can try?

P.S. Thank you for this wonderful tool! It's been awesome to work with :D

Awesome to hear that! ^_^

Here's a little PoC that works as expected:
const CDP = require('chrome-remote-interface');

CDP(async (client) => {
    try {
        const {Page, Runtime} = client;

        // prepare the dummy page
        await Page.navigate({url: 'about:blank'});
        await Runtime.evaluate({
            expression: `document.write('<form onsubmit="return !!alert(x.value)"><input id="x" /><input type="submit" /></form>')`
        });

        // set the input value and click
        await Runtime.evaluate({
            expression: `x.value = 'name'; document.querySelector('input[type=submit]').click()`
        });
    } catch (err) {
        console.error(err);
    } finally {
        await client.close();
    }
}).on('error', (err) => {
    console.error(err);
});

Is the target web application up somewhere so I can try?

Unfortunately not :(
So I did some more debugging and it looks (this is vue) like using the
     const result = await Runtime.evaluate({
        expression: 'new Promise((resolve) => {document.getElementById("name-field").value = "name"})',
        awaitPromise: true
    });

isn't setting the variable ('name') that's set when you manually type into the 'input' field
<input id="name-field" v-model="name">

Try it in headfull mode.

How do I try that? (Does this just mean not-headless?)

Also, how did you get
 expression: `x.value`

to work? I thought you had to do a
document.getElementById('name').value = ...

It seems like the way I'm entering the information is a likely culprit

isn't setting the variable ('name') that's set when you manually type into the 'input' field

It makes sense, you could try to simulate key presses with Input.dispatchKeyEvent.

How do I try that? (Does this just mean not-headless?)

Yeah, sorry for the pun :), just run Chrome without --headless, it's easier to debug.

Also, how did you get
 expression: `x.value`

to work?

It's maybe nonstandard, but surely works in Chrome that every id="xxx" nodes are mapped to global variables.

<input id="name-field" v-model="name">

BTW if your final intent is this you can try to just use:
document.getElementById("name-field").setAttribute("v-model", "name");

It makes sense, you could try to simulate key presses with Input.dispatchKeyEvent.

Trying that now based on the code https://github.com/cyrus-and/chrome-remote-interface/issues/226 and it doesn't appear to be doing anything?
     const result = await Runtime.evaluate({
        expression:  'document.getElementById("project-name-field").focus()');
    });

    await Input.dispatchKeyEvent({
        type: 'char'
        text: "name",
    });

Am I missing something obvious?

Yeah, sorry for the pun :), just run Chrome without --headless, it's easier to debug.

lol I'm currently running it on localhost on a VM, and I can't access it externally due to network permissions, so I can't really debug it like that, unfortunately.

It's maybe nonstandard, but surely works in Chrome, every id="xxx" node is mapped to a global variable.

Cool. Did not know that. owo

document.getElementById("name-field").setAttribute("v-model", "name");

Also tried this, to no avail. Nothing happens. :(

Am I missing something obvious?

Yes, but it's not obvious... that method sends one char at a time, so:
const CDP = require('chrome-remote-interface');

CDP(async (client) => {
    try {
        const {Input, Page, Runtime} = client;

        // prepare the dummy page
        await Page.navigate({url: 'about:blank'});
        await Runtime.evaluate({
            expression: `document.write('<form onsubmit="return !!alert(x.value)"><input id="x" /><input type="submit" /></form>')`
        });

        // focus the input fiels
        await Runtime.evaluate({
            expression: `x.focus()`
        });

        // simulate typing
        for (const text of 'name') {
            await Input.dispatchKeyEvent({type: 'char', text});
        }

        // click
        await Runtime.evaluate({
            expression: `document.querySelector('input[type=submit]').click()`
        });
    } catch (err) {
        console.error(err);
    } finally {
        await client.close();
    }
}).on('error', (err) => {
    console.error(err);
});

YESSSSS!! That works!! OwO Thank you!! :D

So, I am finding that doing
        // simulate typing
        for (const text of 'name') {
            await Input.dispatchKeyEvent({type: 'char', text});
        }

prints only sometimes (possibly asynchronously?)... Are you seeing that behavior, or am I missing an await or something?

Are you seeing that behavior, or am I missing an await or something?

Hm, I cannot reproduce it here, even with longer strings, anyway you can try the Input.insertText method:
await Input.insertText({text: 'name'});

Also, if you need to interact with Chrome using a higher-level interface you may want to give Puppeteer a try.

I was missing the 'await' on the focus 🤦♂ Sorry about that. Thanks again!!


Request returns 422 Unprocessable Entity

Hi,
there is a strange bug. The request works when I try the request in Postman.
When I execute the request in my application, the rest-client returns an error.
RestClient::UnprocessableEntity: 422 Unprocessable Entity
from /usr/share/rvm/gems/ruby-2.5.1/gems/rest-client-2.0.2/lib/restclient/abstract_response.rb:223:in `exception_with_response'

Request:
response = RestClient.put(
   "#{@rest_url_base}#{url}",
   payload.to_json,
   __request_params.merge(params)
)
p response
=> nil

Payload to_json:
"{\"Id\":\"126782\",
  \"arrival\":\"2018-11-18T22:16:54+01:00\",
  \"departure\":\"2018-11-19T10:00:00+01:00\",
  \"persons\":1,
  \"ServiceIds\":[\"3829\"]}"

Headers:
{:content_type=>"application/json",
 "Authorization"=>"Bearer eyJhbGc...QYWXw"}

what can I do to send a valid request?

Do you have a more minimal reproducer?
What are the params expected to be? The way you've written this code:
response = RestClient.put(
   "#{@rest_url_base}#{url}",
   payload.to_json,
   __request_params.merge(params)
)

The last argument to RestClient.put() is treated as the HTTP headers hash, which maybe isn't what you expected. Setting RestClient.log also might be helpful, or sending the request to an echo service like https://http-params.herokuapp.com/.

I'm having the same issue in the same way. "Authorization"=>"Bearer the_token" and sending payload.to_json.
I get the same error. Same kind of request works with Httparty and postman.


Lower the bar for performance check

Since we run the performance check 8 different times in different CIs, we have to run a 7th interleaved attempt to keep the probability of false failure in the same range. I also increased the threshold to 0.5% while we were at it. False failure should be few and far between with this change, but should still reliably detect reduced performance above a certain noise level.

Feeling good about all this for the moment, I'll start back up on some actual improvements next :)


LevelDB implementation of new storage interface

Once this is done, add support for LevelDB back in to aries-agent-rest (was removed in #2543)

LevelDB support for aries-agent-rest was added back in in #2607


[Discussion] What to do with photometric spectral types?

Roth24 has photometric spectral types. Should we ingest them, or ingest them but not "adopt" them? Related to #482

I would vote no, because they are not spectral types. You have to measure a spectral type from a spectrum with sufficient resolution in an appropriate wavelength range.  Broadband Colors or Absolute magnitudes are correlated with spectral type but not the same thing.  I'd say these are predictions of spectral types not measurements of spectral types.
If the magnitudes are in the database then one can use whatever relations one likes to estimate the things that haven't been measured, but to me they need to be kept distinct.
(Similarly, I wouldn't place 1/(photometric distance estimate in parsecs) into a table of measured trigonometric parallaxes although I'd be happy to have it in a "best distance estimate" table, and I wouldn't put a predicted i-z color in from a J-K measurement.)
Just an opinion of course, it all depends on what the database is supposed to represent.

There is also a good discussion happening in the #simple-archive Slack channel.  One suggesstion from there which I really like is the idea to add a Photometric boolean column. If there is only one spectral type and its based on photometry, it can still be adopted, but it will have a Photometric = True flag.


Deserialize and serialize implements

Objective

Add serde Deserialize and Serialize for structs that doesn't implement it, even if they could benefit from it

Solution

Derive these traits for the structs Style, BackgroundColor, BorderColor and Outline.

A bit annoying that serde isn't feature-flagged in bevy_ui, but that can wait for a different PR.


Unabe to install Tomcat8 - error in script

Trying to install on Ubuntu 14.04 server -
Installing Tomcat 8
Reading package lists... Done
Building dependency tree
Reading state information... Done
E: Unable to locate package tomcat8

saw that it is only tested with 16.04... works there.  Thanks

Yes for 14.04 you have to use tomcat7, this branch installs Lucee 4.5 on Ubuntu 14.04 if interested: https://github.com/foundeo/ubuntu-nginx-lucee/tree/lucee45-ubuntu14


Primitive type icon is not shown if the metadata is generated by XVIZMetadataBuilder

Hi, I try to run website-demo, and it works well if I just use the KITTI dataset. But when I use XVIZMetadataBuilder to generate a metadata message myself, the primitive type icon will not display. I console.log the meta message and compare it with the meta part of 1-frame.glb, found that the scalar_type is in upper case in the generated one and lower case in the .glb file. Also found in the custom-style.js the icon object PRIMITIVE_TYPE_TO_ICON has all lower case keys. I think this may be the reason why type icons are missing. How to generate a meta message with lower case scalar_type?

@lgmcode This seems to be a problem in the XVIZ Builders.  Let me take a look.

@lgmcode Actually, we did change enums to UPPERCASE.  So the builder is correct, the website-demo is simply not up to date.  I will change the demo code to apply .toUpperCase() to the type.

This fix was addressed in https://github.com/uber/streetscape.gl/pull/414


Setting SERVICE_HOST breaks etcd v3.2.

Make --listen-client-urls use $HOST_IP
See 3.2 changelog: https://github.com/etcd-io/etcd/blob/master/Documentation/upgrades/upgrade_3_2.md#change-in---listen-peer-urls-and---listen-client-urls

Thank you for contributing to openstack-dev/devstack!
openstack-dev/devstack uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


[BUG] "Show in Table Mode" for Resistance and Immunities in Core Tab makes them disappear.

Clicking the "Show in Table Mode" button for Resistance and Immunities in Core Tab causes the fields to disappear entirely. This persists between opening and closing the actor sheet. The child divs within first-row are deleted and nothing replaces them.

None clicked

Resistances clicked

Both clicked

Those do not disappear. Resistance and Immunities tables are bellow the skill tables.

That they are! I did not see that since I didn't scroll down!
Lovely option, thank you sir!


Bug Fix: Plugins Fail To Load on Python 3.8+

Problem

Python 3.8+ multiprocessing library has switched to spawn where before it was fork
This is an issue since spawn does not share any memory between the parent and child processes.
The plugins/filters are stored as singleton global variable in the Settings class

Solution

Default the multiprocessing to fork when creating a pool of processes
Python does have a shared memory library now however I find that it is only useful when sharing simple data types like strings, ints, list but not custom classes

Hi @jpdakran, I also saw #482, as a possible alternative solution.
@domanchi  can you review this one? I think you have the most context.

I think it may be better to use the approach with #482 -- essentially, initializing a new settings object per process, rather than sharing memory. It's been a while since I looked at it, but the Settings object was designed for easy serialization/deserialization, so you could pass in a serialized object to the spawned process, and hydrate it when starting it up.
This way, we'd be less dependent on OS level details since this library is used on multiple platforms.


Electron Config Renamed to Electron Store

Fix the Metadata for the community page
https://github.com/electron/electron.atom.io/blob/afe62610cd8cc156f9edb4e5b702d549272a57dc/_data/awesome_electron.json#L1372-L1378
 {
    "name": "electron-store",
    "href": "https://github.com/sindresorhus/electron-store",
    "description": "Simple data persistence for your Electron app or module.",
    "category": "tools",
    "subcategory": "for_electron"
  },

sindresorhus/electron-store#4

Thanks. This will get fixed the next time we pull in an update to our awesome-electron data.


v0.0.1 Code Preview

This answers the question of is it possible? to use a very simplified closures systems to build a simple MVC system


Change MarkNode::clone node arg type to this

Subclasses of MarkNode can't override clone without adding a Flow type error ignore annotation without this change.

I signed the CLA, can you rerun that action please?

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

Ignore my last message about re-running the CLA signature action :)

Seems like we should be doing this for other nodes though, if it's good for one


Uploading release asset route & browser compatibility

the POST /repos/:owner/:repo/releases/:id/assets route is the only one that currently requires a file read stream
https://github.com/octokit/node-github/blob/72bff1e434ad3bbe0218558208f6ce402e109f2e/lib/index.js#L841
The route expects raw file in its binary form rather than JSON, see https://developer.github.com/v3/repos/releases/#upload-a-release-asset
Browser support binary file uploads with fetch today: https://stackoverflow.com/questions/36067767/how-do-i-upload-a-file-with-the-html5-js-fetch-api
The node-fetch polyfill for Node.js also supports a read stream as body, so that should all probably work pretty easily. But I have not yet tried it out. If someone wants to give it a go, it'd be cool to create a minimal standalone library using node-fetch which supports the same APIs for both browser & node.js that does nothing but uploads assets to a GitHub release :)

Somewhat related, the current implementation of github.repos.uploadAsset does not strictly follow the specification described at https://developer.github.com/v3/repos/releases/#upload-a-release-asset. The current implementation has the host hardcoded to uploads.github.com while the specification says

This endpoint makes use of a Hypermedia relation to determine which URL to access. This endpoint is provided by a URI template in the release's API response

And there is a workaround built in only for that case
https://github.com/octokit/node-github/blob/72bff1e434ad3bbe0218558208f6ce402e109f2e/lib/index.js#L647-L654
This is me just thinking out load :) One way would be to remove the method altogether from node-github and provide an easy workaround. And if the host for asset uploads can change on GitHubs site, then the current implementation is wrong. I'll find out

this is all done.


GET /ob/profile moderatorInfo to include list of acceptedCurrencies

To match https://github.com/OpenBazaar/openbazaar-desktop/issues/1491

moderatorInfo should accept an acceptedCurrencies field as a list of currency codes.

It should be validated to ensure the moderator can actually support all of the saved codes.
The acceptedCurrencies list should be populated automatically by the server, since we're not exposing it in the client.


centos 6.5 install modsecV3 error

when i run build.sh, report this err msg:
`
bash build.sh
libtoolize: putting auxiliary files in .'. libtoolize: copying file./ltmain.sh'
libtoolize: putting macros in AC_CONFIG_MACRO_DIR, build'. libtoolize: copying filebuild/libtool.m4'
libtoolize: copying file build/ltoptions.m4' libtoolize: copying filebuild/ltsugar.m4'
libtoolize: copying file build/ltversion.m4' libtoolize: copying filebuild/lt~obsolete.m4'
configure.ac:44: warning: macro AM_PROG_AR' not found in library configure.ac:44: warning: macroAM_PROG_AR' not found in library
configure.ac:44: error: possibly undefined macro: AM_PROG_AR
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
configure:19307: error: possibly undefined macro: AM_DEFAULT_VERBOSITY
autoreconf: /usr/bin/autoconf failed with exit status: 1
configure.ac:44: error: possibly undefined macro: AM_PROG_AR
If this token and others are legitimate, please use m4_pattern_allow.
See the Autoconf documentation.
configure:19307: error: possibly undefined macro: AM_DEFAULT_VERBOSITY
`
releaseinfo
autoconf : 2.63
automake: 1.11.1
libtool: 2.4
Did anyone else meet?
adn anyone who can help?

Hi,
Which version you're trying to setup? RC1 of libModSecurity?
You might want to follow the compilation recipes here: https://github.com/SpiderLabs/ModSecurity/wiki/Compilation-recipes

In my experience, it's not possible to build modsecv3 with stock CentOS/RHEL 6 toolchain. For one, v3 is written in C++11 which is not supported by GCC 4.4 that ships with EL6.
CentOS 7 userland works for me (GCC 4.8, Automake 1.13, Autoconf 2.69, Libtool 2.4, flex 2.5, bison 2.7)

I wrote the recipe for 6.5 it is possible just a royal PITA

Hi @365moods, Please upgrade your distro and or follow the receipts to have all the basic dependencies met.

Hi @365moods, Please upgrade your distro and or follow the receipts to have all the basic dependencies met.

Maybe you should update dependency  toolchain version into the Compilation-recipes.


WRN-16290 [Agate] Sampler: Merge migrated storybook with controls to develop branch

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[x] Documentation was verified or is not changed
[ ] UI test was passed or is not needed
[ ] Screenshot test was verified or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Merge migrated storybook with controls to develop branch
Resolution
Additional Considerations
Links
WRN-16290
Comments

Please change placehold.it image service in default/virtualGridList.js as https://github.com/enactjs/sandstone/pull/1185

Also, please resolve conflicts.


Template update for nf-core/tools v2.13.1

Resolved conflicts in automated PR.

I noticed the local lint failure on the images too. I just ran nf-core lint --fix and it did update the images (minor text kerning change). I've committed that change here and we'll see if it passes.

I can't speak for the details of the groovy code, slowly getting into it, but in general I really like the subworkflows that check input and format output. It is much more intuitive, at least for me :). Great work @jmuhlich

I think the EditorConfig and Prettier tests were previously required but have been phased out with the new tools update, so they'll never be run here. I'll ask some nf-core folks how to proceed.

I can't speak for the details of the groovy code, slowly getting into it, but in general I really like the subworkflows that check input and format output. It is much more intuitive, at least for me :). Great work @jmuhlich

Thanks Jose! Those new subworkflows are all from the new nf-core tools release and template updates, so all credit to the tools team.

OK! Maxime adjusted our required checks to remove the stale ones, so we're good to go.


Move public/js/queries to workspace/templates

[query] templates are for the host and should not be stored as js/public. There is no reasons for [query] templates to be accessible  by the client.
The folder has been renamed as templates sitting with the assignTemplates module.
Unused templates have been removed:
count_locations
filter_aggregate
get_last_location
labels
nnearest_locations.sql

@RobAndrewHurst I added the get_last_location template


refactor: port to TypeScript

When I started working on Astro-Shield, I did not expect it to grow so much. Because of that, I wanted to avoid having to introduce build systems and all the hassle they imply...
But we are already at a point where the code is becoming difficult to maintain, with the following problems:

Ugly and bulky JSDoc annotations everywhere, TS type annotations are much more ergonomic.
JS is more difficult to refactor and organise than TS.
It is too easy to introduce strange type-related bugs (even with JSDoc), this is not acceptable in a security focused library.
The served code is too bulky, we already need a build system to package something a little bit slimmer, whether we use TS or not.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

Done in #92 !


Eia epa crosswalk

So far I've created a replica of Greg's transformation notebook that takes the EIA crosswalk file and fills in some of the gaps. I'm looking for feedback on next steps regarding how to modularize this. Should it live as a function in the transform directory in the eia.py file? Should it be called in the final transform step?
Waiting on the (hopefully) imminent release of the "official" EPA version of the crosswalk.

Next steps would then be to load the crosswalk table from the database and add it to the appropriate output tables in the output modules?

The notebook comments will probably only really make sense here: https://app.reviewnb.com/catalyst-cooperative/pudl/pull/822


 Any ideas why?

The mouse pointer is not in tandem with where the signature is getting drawn.

Make sure that you canvas.width and canvas.height match the actual size of the canvas. That's what this part of the demo code is doing: https://github.com/szimek/signature_pad/blob/master/docs/js/app.js#L24-L27.


Remove English tab from translations UI

Description
All English translations are contained in other parts of the UI and are duplicative and confusing. Additionally, maintaining multiple updaters of this text requires duplicating any validation logic as well, which is particularly problematic for program statuses (since there is no internal/external distinction between the status display name).
Now, if one navigates to the English translations UI (or any unsupported locale), they're redirected to the program/question index page with an error toast message. Note that previously we rendered the translation UI for ANY locale that was parseable from the URL.
There are a few views that generate a link to the translation UI. Previously, they were forming a link to the "English" translation form. Instead of adding logic to each of these for choosing the first locale, I introduced a /translations route that performs the "choose the first non-English locale" logic and had calling views link to that.
Finally, left links back to the program/question editing views so that it was easier to edit the English text.
Screenshots
Program translations:

Question translations:

Release notes:
Removing the English translations tab since the information is already contained in the program / question builders.
Checklist

[X] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[X] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Issue(s)
#2754

Codecov Report

Merging #3069 (0cbd8f0) into main (d7aa69c) will decrease coverage by 0.08%.
The diff coverage is 73.07%.

@@             Coverage Diff              @@
##               main    #3069      +/-   ##
============================================
- Coverage     77.88%   77.80%   -0.09%
- Complexity     2151     2160       +9
============================================
  Files           274      275       +1
  Lines          8575     8632      +57
  Branches        582      586       +4
============================================
+ Hits           6679     6716      +37
- Misses         1678     1697      +19
- Partials        218      219       +1

Impacted Files
Coverage Δ

...ers/admin/AdminQuestionTranslationsController.java
56.52% <60.71%> (-7.48%)
:arrow_down:

...lers/admin/AdminProgramTranslationsController.java
72.00% <71.42%> (-21.75%)
:arrow_down:

...app/controllers/admin/AdminQuestionController.java
72.39% <100.00%> (-0.17%)
:arrow_down:

server/app/services/TranslationHelper.java
100.00% <100.00%> (ø)

...trollers/admin/AdminProgramStatusesController.java
90.32% <0.00%> (-0.16%)
:arrow_down:

.../controllers/admin/AdminApplicationController.java
16.90% <0.00%> (-0.12%)
:arrow_down:

server/app/repository/ProgramRepository.java
82.48% <0.00%> (ø)

server/app/forms/admin/ProgramStatusesForm.java
100.00% <0.00%> (ø)

... and 5 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


1.0.0-dev.8: presence failling?

using a local supabase. (started this morning). under macos.
(postgres realtime is working with this supabase client, so I suspect the code managing channels is somehow working)
class PresenceRepository {
  late final String uid;

  void init(String uid) {
    this.uid = uid;
  }

  void registerPresence(String id) {
    final client = Supabase.instance.client;

    final chan = client.channel(id);
    chan.subscribe((status, [err]) async {
      print('Presence status: ${id} : ${status} - ${err}');
      if (status == 'SUBSCRIBED') {
        final status2 = await chan.track({uid: uid});
        print('presence: ${id} : ${status2}');
      }
    });
  }

  Stream<List<String>> subscribeToPresence(String id) {
    final controller = StreamController<List<String>>();
    final client = Supabase.instance.client;

    final chan = client.channel(id);
    chan.on(RealtimeListenTypes.presence, ChannelFilter(event: '*'), (a, [b]) {
      print(
          "currently online users: ${id} : ${a} - ${b} - ${chan.presenceState()}");
      // controller.add(somethingtofigureout);
    });
    return controller.stream;
  }

  void unregisterPresence(String id) {
    final client = Supabase.instance.client;
    print("Presence: unregister: ${id}");
    final chan = client.channel(id);
    chan.unsubscribe();
  }
}

(I register then subscribe once)
I get the following output:
flutter: Presence status: 42 : SUBSCRIBED - null
flutter: Presence status: 42 : CHANNEL_ERROR - {}
flutter: presence: 42 : ChannelResponse.timedOut
flutter: Presence status: 42 : SUBSCRIBED - null
flutter: Presence status: 42 : CHANNEL_ERROR - {}
flutter: presence: 42 : ChannelResponse.timedOut
... (repeats every seconds or so)

Is there something I am missing? (can it be some missing headers also? like it was for rpc in https://github.com/supabase-community/supabase-flutter/issues/221?)

I can confirm this happens on local version of Supabase, but does not seem to happen with the cloud version of Supabase, which makes me think it might be a backend issue. If you could open an issue on realtime repo to confirm with them, it would be awesome!

Closing this is with the comments from our realtime team.
https://github.com/supabase/realtime/issues/284#issuecomment-1224908325

supabase/realtime docker image is not yet compatible with @supabase/realtime-js rc versions. We've been developing the new Broadcast and Presence in a private repo and will be moving over all code to this repo in the upcoming weeks


Implement new register_vault

Objective
Rename the register_vault to register_vault_with_did and implement a new register_vault function to deploy a DID Contract, and register the new DID in a vault.
Checklist

[x] Rename register_vault to register_vault_with_did.
[x] Implement register_vault function to deploy DID and register it in a vault:

[x] Create DID.
[x] Add the DID to the vault.

[x] Update the documentation.

Released in v0.15.0


Support .NET Core SDK Portable Mode

I would like to be able to use the .NET Core SDK from a USB drive, this is already somewhat possible but there are still default folders created on the host system (.nuget, .dotnet, .templateengine, etc.). There is already some issues around this:
7569 - Request, able to config the location of ~/.dotnet and ~/.templateengine
4003 - Change default .nuget/package folder .NET Core
It would be nice to have an option, maybe in the dotnet-install script, to enable a -Portable option that would change the path for all directories used by the SDK to point to a single location (i.e. a USB .\data\ folder). This way full usage of the SDK would never touch the host file system.
The Dream Setup:
Use VS Code Portable with .NET Core SDK Portable living solely on a USB drive, and being able to move from computer to computer with a full dev environment on your key chain.

Issue moved to dotnet/cli #9853 via ZenHub


Method cops not found when changing the method

We've got a simple method:
  def update
    user = User.authorized.find(params[:id])
    check_auth :edit_user, user
    update_from_params! user
    respond_with_ok
  end

After changing or adding some lines:
  def update
    user = User.authorized.find(params[:id])
    check_auth :edit_user, user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    update_from_params! user
    respond_with_ok
  end

There is a rubocop warning like "AbcSize" or "MethodLength":
app/controllers/users_controller.rb:37:3: C: Metrics/AbcSize: Assignment Branch Condition size for update is too high. [15.03/15]
  def update ...
  ^^^^^^^^^^
app/controllers/users_controller.rb:37:3: C: Metrics/MethodLength: Method has too many lines. [12/10]
  def update ...
  ^^^^^^^^^^

Because the issues are in the line where the method is defined and this line wasn't changed, the issue is not reported by pronto.

That's a bit tricky, as it's not part of the diff. Not sure there's an easy way to detect this without parsing the file itself, though 🤔

That's a bit tricky, as it's not part of the diff. Not sure there's an easy way to detect this without parsing the file itself, though 🤔


[Perf] Add ref to azure-sdk-tools repo

Needed to fix pipeline "js - aggregate-reports"

/azp run js - aggregate-reports

Fixes #22502


admin dashboard login issue

Sry, but I encountered a problem.
When I entered the default credentials:
·<EMAIL_ADDRESS>· admin
I still couldn't complete the login request. What's wrong about that?

And in DB, the webvirtcloud.account_user is empty, is this correct?

Hello,
You have to try the next command:
$ docker compose exec backend python manage.py loaddata initial_data

Hello,
You have to try the next command:
$ docker compose exec backend python manage.py loaddata initial_data

Gotcha, it works. Thx!


Test Swede_2024 on Swede bot

Deploy Swede_2024 repo code to Swede bot and test.  Fix values for bot size (aka transforms/offsets of pods from bot center)
Start here:  https://github.com/Team3176/Swede_2024

The continues work from #30

test the defense branch of Code_2024 that adds the ROBOT_DEFENSE robot type within Constants.
This lets us continue drivetrain development in one unified codebase.

Tested in Defense branch and resolved by PR#52


[ISSUE #8982] Dynamically install latest Go version for e2e pipeline

Which Issue(s) This PR Fixes
Fixes #8982
Brief Description
This PR updates the pipeline script to dynamically install the latest stable version of Go, ensuring compatibility with dependencies.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 47.62%. Comparing base (b638d4c) to head (8e6ba9d).

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #8985      +/-   ##
=============================================
- Coverage      47.69%   47.62%   -0.07%
+ Complexity     11774    11756      -18
=============================================
  Files           1304     1304
  Lines          91192    91192
  Branches       11715    11715
=============================================
- Hits           43491    43434      -57
- Misses         42327    42367      +40
- Partials        5374     5391      +17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


React Folder Structure

Have to be able to deploy app locally using npm i, npm run start

@Solrovivrus
I fixed this. Please be sure to add documentation on how to run app.


Is it available for python3?

ATT，as pyAudioAnalysis is a python2 project?

Only python2 sorry...
On Oct 6, 2017 8:25 AM, "Chenyang Zhou"<EMAIL_ADDRESS>wrote:

ATT，as pyAudioAnalysis is a python2 project?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MikeMpapa/CNNs-Speech-Music-Discrimination/issues/3,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMB79v94Dofw1Oc9jUujVNFzyOjAM-rHks5spipSgaJpZM4Pweh_
.