"Layer" is not the right term

Given the discussion on #37 and on the mailing list, non-chain topologies of "layers" are possible and parents are in fact optional. Given this, I think the term "layer" is not ideal. I think it comes directly from Docker's implementation and particularly from the Dockerfile build mechanism. For this spec, I think these should be considered historical, and not have a strong bearing on how images are defined, and what terminology is used.
As a use case, it should be possible to define an image as a collection of disjoint archives. Say I have a statically linked server binary, and a tree of files for it to serve. I would like to specify an image as combining these two, with neither the parent of the other. The ideal here would be to allow

changing the binary without needing to redistribute the data
changing the data without needing to redistribute the binary

In this setup, I'd find the term "layer" confusing.

On Wed, Apr 20, 2016 at 07:51:49PM -0700, Kamal Marhubi wrote:

Given the discussion on #37 and on the mailing
list,
non-chain topologies of "layers" are possible and parents are in
fact optional.

While I don't think baking parent information into a layer is
necessary 1, all Docker-like serialization formats I've seen use a
single, ordered array of tarballs (with whitelist/blacklists?).  I'm
personally a fan of the increased granularity (sub-file!) of IPFS's
unixfs objects (to avoid issues like 2), but as long as the
serialization is based on an ordered array of tarballs I think the
“layer” term is fine.
 Subject: Re: image-spec: Can layer parents be omited? Are non-chains possible?
 Date: Wed, 20 Apr 2016 14:20:09 -0700
 Message-ID<EMAIL_ADDRESS>
 Subject: host-mounted volumes in Dockerfile's VOLUME

I agree this is something we should discuss and address. I am making it a post v1.0.0 item though.

Appc/spec allows this. In fact the folks at BlaBlaCar are doing this in https://github.com/blablacar/dgr
It's something I feel ought to be allowable, but not immediately if it's going to break compatibility in this initial v1 release.

@vbatts

but not immediately if it's going to break compatibility in this initial v1 release.

I find this notion of breaking compatibility in a v1 to be odd. I opened #42 to make sure I'm understanding correctly that v1 is meant to be compatible with Docker's current spec.

@kamalmarhubi It seems odd, granted, and I would like it to not be the case either. The intention of the spec is to carry forward lessons learned, and build on the pervasive industry standards. So some changes that would break much infrastructure need to be planned and coordinated. It's not an entirely clear line.

@kamalmarhubi would you say that #102 is more articulately the same as your issue?

Yes I think so! Closing this because the important part is that feature and less so the name.


Very minor nil-or-empty cleanup

Just randomly exploring, found with
❯ sg run --pattern '$X == nil || len($X) == 0'
./common/dynamicconfig/configstore/config_store_client.go
413│	if dcValues == nil || len(dcValues) == 0 {

This is clearly a no-op.

Pull Request Test Coverage Report for Build 018e1634-9a58-4ad0-96af-c22a5fa5dab1
Details

11 of 13 (84.62%)  changed or added relevant lines in 1 file are covered.
100 unchanged lines in 11 files lost coverage.
Overall coverage remained the same at 63.82%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/dynamicconfig/configstore/config_store_client.go
11
13
84.62%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
1
88.06%

tools/cli/admin_db_decode_thrift.go
1
71.79%

common/persistence/historyManager.go
2
66.67%

common/util.go
2
91.69%

service/history/execution/mutable_state_util.go
2
78.48%

common/persistence/nosql/nosqlplugin/cassandra/workflow.go
6
63.51%

common/task/fifo_task_scheduler.go
7
80.41%

common/persistence/data_manager_interfaces.go
13
91.64%

service/history/execution/mutable_state_task_refresher.go
14
62.34%

common/persistence/sql/workflowStateMaps.go
19
83.84%

Totals

Change from base Build 018e1631-7df5-41c4-aa3c-783fa88e3719:
0.0%

Covered Lines:
93560

Relevant Lines:
146599

💛  - Coveralls


Make some improvements to the Python script.

Don't allow CTRL+C while a Notecard transaction is ongoing. Previously, if the user interrupted the script with CTRL+C during the middle of a transaction and immediately ran the script again, it was likely for the old transaction to still be coming over the I2C bus. This wrecks communication between the script and the Notecard. Instead, note that CTRL+C was issued, finish the current transaction, and then interrupt the script.
Retry web.get requests if the status code isn't 206 or 200. I saw a 503 error at one point during testing. This change makes the script resilient to these kinds of errors.
Format everything with YAPF.

@paigen11 Note that the problem with a slow web.get request timing out is still unresolved. I will fix that once we know how we want to approach it.


Inquiry about the Hybrid design

Hello, thank you for sharing the code of the paper.
Could you please release the code of the hybrid design?
Also, I have a question please about the hybrid design in Table. 6. Did you replace the whole pooling stage with attention or SpatialFC stage ? or just replace the last block of each stage?
Thank you.

Hi @AbdelrahmanShakerYousef ,
Thanks for your attention.
Sorry that currently, I need to prepare for my final exams. I will release the code and trained models of the hybrid design before 1 May. We replace the whole pooling stage with the attention or SpatialFC stage. Remember to add position embedding before the first attention stage. For hybrid design, the norm is set as Layer Norm and the batch size as 1024.

Hi @AbdelrahmanShakerYousef ,
I have completed the release of code and trained models of the hybrid design before 1 May (Anywhere on Earth Time Zone haha).
Thanks for your attention :)


Dataset-JSON 1.1

Dataset-JSON 1.1 major version

Overhauled schema based on community feedback
Switched from TS to LinkML for more varied modelling support
Added tooling for conversion from 1.0 to 1.1
Added Jupyter Notebook for easier interaction
Updated all examples

Should the dataset-json version be updated to 1.1.0 in these files?

Need to set datasetJSONVersion to "1.1.0"

I think it would be good to add the SEND examples from https://github.com/cdisc-org/DataExchange-DatasetJson/pull/21 as well. We want to make sure SEND is represented.

The examples need some refinement.
Example adam\adlbcpv.json:
            "dataType": "integer",             "displayFormat": "DATE9.",             "itemOID": "IT.ADLBCPV.TRTSDT",             "keySequence": 0,             "label": "Date of First Exposure to Treatment",             "length": 8,             "name": "TRTSDT",             "targetDataType": "integer"         },
In this case dataType should be "date".
Also. the keySequence should not be 0 when the item is not part of key variable in the dataset structure. I would prefer not to have the keySequence key then.
Also, not wrong, but:

if targetDataType is the same as dataType, do we need to specify it?
if displayFormat is null, do we even need to have the key.
I would prefer the JSON files not be pretty printed, so that people get used to this. You really don't want pretty-printing in a submission. Common tools can format with indentaton on-the-fly.

I think it would be good to add the SEND examples from #21 as well. We want to make sure SEND is represented.

Good point. Can we do it after this PR has been corrected and approved, then update the SEND examples the same way upon merging #21 to the version 1.1?

Looks good!
One more use case to illustrate targetDataType. In the SDTM datasets there are decimal data types.
They also need targetDataType.
Example:
        {
            "dataType": "decimal",
            "itemOID": "IT.VS.VSSTRESN",
            "label": "Numeric Result/Finding in Standard Units",
            "length": 8,
            "name": "VSSTRESN"
        },

I think this would be:
        {
            "dataType": "string",
            "itemOID": "IT.VS.VSSTRESN",
            "label": "Numeric Result/Finding in Standard Units",
            "length": 8,
            "name": "VSSTRESN"
            "targetDataType": "decimal",
        },

Great that you added the order!
I think you missed a few:
    "metaDataRef": "https://metadata.location.org/CDISCPILOT01/define.xml",
    "metaDataVersionOID": "MDV.MSGv2.0.SDTMIG.3.3.SDTM.1.7",
    "studyOID": "cdisc.com/CDISCPILOT01",
    "itemGroupOID": "IG.VS"

Great that you added the order! I think you missed a few:
    "metaDataRef": "https://metadata.location.org/CDISCPILOT01/define.xml",
    "metaDataVersionOID": "MDV.MSGv2.0.SDTMIG.3.3.SDTM.1.7",
    "studyOID": "cdisc.com/CDISCPILOT01",
    "itemGroupOID": "IG.VS"

Hi Lex, this has been fixed now. I used vs.json from 1.0 as the reference for ordering elements
Thanks for the thorough review!

@swhume It is:
        {
            "itemOID": "IT.VS.VSSTRESN",
            "name": "VSSTRESN",
            "label": "Numeric Result/Finding in Standard Units",
            "dataType": "string",
            "targetDataType": "decimal",
            "length": 8
        },


Plan to support raspberry pi Pico?

Hi,
Could zephyr support support raspberry pi Pico?
Looks rtt and freeRTOS already support it.
FYI.
https://www.seeedstudio.com/blog/2021/02/05/rust-rt-thread-os-freertos-come-to-the-raspberry-pi-pico/

https://www.youtube.com/channel/UCA4j-J-TFW_9Dt4iTwIjtYQ
Once visit this channel for raspberry pi pico projects.

https://www.youtube.com/channel/UCA4j-J-TFW_9Dt4iTwIjtYQ
Once visit this channel for Raspberry Pi Pico Projects


Use ReCaptcha to validate form input for donations

The credentials for validating with the ReCaptcha servers are in env_vars.js (I'll send it to you).
Making this work has a front-end and back-end component. The documentation for setting it up is here:
(Stas has admin access too through his gmail account).
https://www.google.com/recaptcha/admin#site/319715017?setup
Let's discuss this. It doesn't have to be done until after the Form submission thing is working otherwise.

Done and working! Thanks


Add read_with_hint and scale_general

In one of my projects, I need to read tons of different JPEG files from the internet, some of which make leptonica complain and give up, so I added pixReadWithHint
I also wanted to experiment with different leptonica scaling functions and sharpness and their performance when applied before OCR, and ended up using pixScaleGeneral so I can downscale without sharpening, which sped up my usecase slightly.
Just wanted to upstream these changes, if anything is out of place let me know.

Thank you. I've merged and published
Good luck with ocrlocate


error on npm install

I'm trying to install on
node -v v10.16.0
npm -v 6.9.0
On the terminal I get error:
In file included from ../../nan/nan.h:192:0,                  from ../src/binding.cpp:1: ../../nan/nan_maybe_43_inl.h: In function ‘Nan::Maybe<bool> Nan::ForceSet(v8::Local<v8::Object>, v8::Local<v8::Value>, v8::Local<v8::Value>, v8::PropertyAttribute)’: ../../nan/nan_maybe_43_inl.h:112:15: error: ‘class v8::Object’ has no member named ‘ForceSet’    return obj->ForceSet(isolate->GetCurrentContext(), key, value, attribs);                ^~~~~~~~
On the log this is the end:
18778 verbose stack Error<EMAIL_ADDRESS>postinstall: node scripts/build.js18778 verbose stack Exit status 1 18778 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:301:16) 18778 verbose stack     at EventEmitter.emit (events.js:198:13) 18778 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14) 18778 verbose stack     at ChildProcess.emit (events.js:198:13) 18778 verbose stack     at maybeClose (internal/child_process.js:982:16) 18778 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5) 18779 verbose pkgid<EMAIL_ADDRESS>18780 verbose cwd /home/kyubi/itemsense-viztool-js 18781 verbose Linux 4.18.0-22-generic 18782 verbose argv "/usr/bin/node" "/usr/bin/npm" "install" 18783 verbose node v10.16.0 18784 verbose npm  v6.9.0 18785 error code ELIFECYCLE 18786 error errno 1 18787 error<EMAIL_ADDRESS>postinstall:node scripts/build.js18787 error Exit status 1 18788 error Failed at the<EMAIL_ADDRESS>postinstall script. 18788 error This is probably not a problem with npm. There is likely additional logging output above. 18789 verbose exit [ 1, true ]
Any idea?

Possibly related to this? https://github.com/nodejs/nan/issues/763
I'm guessing that npm is too current, and needs to be downgraded for this rev. We are currently planning an upgrade of this project, but we're not there yet.

Thanks, I just Downgrade to nodejs 8 and now is working.


Home assistant core

Hi! I also have the Peimar inverter but the installation of my HA is core and does not support the addon. Is it possible to convert the addon into a custom_component?
Thank you

Hi, I'm sorry, I don't know how to help you. The integration "https://github.com/cislow970/raspberry-solar-mon/tree/main/home_assistant integration" was made with Home Assistant installed in a virtual machine.

Close issue


Question: Azure DevOps Extension

https://marketplace.visualstudio.com/items?itemName=nukeeper.nukeeper
If I install and use the extension, how do I know which version of NuKeeper is being used?
My current understanding is that these extensions do not need to be "updated" (when installed) in any manner.
So is the assumption correct that a version of this extension package comes with a specified/packaged version of NuKeeper and (once installed) if new versions of the extension are pushed to the marketplace (containing an updated version of NuKeeper) I am automatically using these new versions?

Unfortunately this is still a manual process. So everytime there is a new version of NuKeeper we have to update the DevOps extension as well and include that version. This is definitely something we need to improve and if we can, automate. Right now i believe the extension uses version 0.10.2.
I've tried using the dotnet tool install command but that didn't work for the Task back then. I'll have a look at this next week in how to automate it and also display same version information in the extension. Is there something you are missing in this version?

I've tried using the dotnet tool install command but that didn't work for the Task back then.

So a few days I wanted to integrate NuKeeper in my Azure DevOps Build pipeline and have it run daily. I managed to install NuKeeper but stumbled upon following issue: https://github.com/NuKeeperDotNet/NuKeeper/issues/613
I did not even know about an Azure DevOps extension at that time. Installing with dotnet tool install could potentially select the highest minor version right? That would atleast automate it somewhat.

I was already using it via dotnet tool install but for me it would be somewhat easier to use the extension.
I was just curious about the update handling of the extension.
Definitely appreciate the idea of displaying the same version information in the extension.
And @MarcBruins: Nope, I am not missing something specific, so no stress on that.
I just wanted to clarify which version is inside the extension and that I receive the "latest version" of the extension when you perform updates on it which is obviously the case.
Thanks!

What i was referring too, is make the extension use the dotnet tool install to make sure that the extension is up to date. Now it just packages up the DLL as you can here: extension repo

Indeed, that would be a better solution imho.

@MarcBruins Just realized that the version within the extension package is actually pretty old:
2018-12-20T07:57:42.9146973Z ##[section]Starting: NuKeeper
2018-12-20T07:57:42.9735452Z ==============================================================================
2018-12-20T07:57:42.9736111Z Task         : NuKeeper
2018-12-20T07:57:42.9736633Z Description  : Automagically update nuget packages in .NET projects
2018-12-20T07:57:42.9737121Z Version      : 0.1.42
2018-12-20T07:57:42.9737583Z Author       : NuKeeper
2018-12-20T07:57:42.9738002Z Help         :
2018-12-20T07:57:42.9738486Z ==============================================================================

That is the Task version not the NuKeeper version ;)

Ah, sorry. 2 coffees are not enough in the morning... ;)


EVP_PKEY_keygen_init: fix return check

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged to master and 3.0 branches. Thank you for your contribution.


re-enabling UI testing

Signed-off-by: eifrach<EMAIL_ADDRESS>Description
enabling UI failure on build

lgtm, but we need generation to be fixed before moving forward :-)

The make ui-image-ci is passing locally. When executed within CI, we are facing

random SIGSEGV nodejs issue when running eslint

just flaky, re-execution helps. We can dig deeper but I can expect a lot of effort with limited outcome here. Or we can hope in a fix in future versions of either nodejs or eslint (which I will update in a follow-up)

external registry issue - requests on packages in the yarnpkg registry return HTTP 500 randomly (varies among attempts)

A hope to fix the SIGSEGV issue: #430

ATM, the CI is passing with and without #430 . So it LGTM.
We will need to monitor flakiness.

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


[v6] useHistory use case

I have an application where I want to go back several entries in history, but I need to know what entries exist to know how far back to go. Is there a way to read from history entries in v6?

Never mind, I realize this is a limitation of the browser's history API for security reason. Sorry for the spam, thanks for all your great work! 🦆


Errors

I tried to use the class FinancialsDownloader() class and didnt work :(.
Code example below:
fd = gm.FinancialsDownloader()
test = fd._download('APPL', 'is')

and gives the error:
with urllib2.urlopen(url) as response:
NameError: name 'urllib2' is not defined ...
In order to fix the problem I changed the line 300
FROM with urllib2.urlopen(url) as response: TO with urllib.request.urlopen(url) as response:
As soon as I overcomed the problem above I am getting the following error that is caused by line 302 json_data = json.loads(json_text) :
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
I am on a dead-end situation now :(

@NkdmsL I think you need to try the example exactly as specified
import good_morning as gm
kr = gm.KeyRatiosDownloader()
kr_frames = kr.download('AAPL')

Regardign urllib2, yes, I think that made it in there from my commit somehow.  I'm fixing and will send a pull request to @petercerno .  I made changes this morn and had it pull back Apple stock in an environment with Python 3.5.


Add switches (on/off) for geniushub outlets

Description:
The geniushub integration shows outlets as binary sensors, it can report when they are on or off, but does not allow the user to change the status. This PR adds them as SwitchDevices, and implements on / off.
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass - no tests for geniushub
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

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

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

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

[ ] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest. - nothing changed
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all. - no new ones
[x] Untested files have been added to .coveragerc.

Hi @castaway,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Thanks for your PR - we'll definitely get this useful PR merged.
Please add:
    @property
    def device_class(self):
        """Return the class of this device, from component DEVICE_CLASSES."""
        return DEVICE_CLASS_OUTLET

The geniushub integration shows outlets as binary sensors, it can report when they are on or off, but does not allow the user to change the status. This PR adds them as SwitchDevices ...

Please be aware of the distinction between a GH Device (that is a switch), and a GH on/off Zone.
For example, an on/off Zone can have (say) three Devices (smart plugs). All 3 will turn on when the Zone is on, and all three off otherwise.

The geniushub integration shows outlets as binary sensors, it can report when they are on or off, but does not allow the user to change the status. This PR adds them as SwitchDevices ...

Please be aware of the distinction between a GH Device (that is a binary_sensor)), and a GH on/off Zone (a switch).
For example, an on/off Zone can have (say) three Devices (smart plugs). All 3 will turn on when the Zone is on, and all three off otherwise.

I am aware of these however, (see above), I can only see ways to turn on/off whole zones, not individual devices.. At least in the v1 API.

Please add:
    @property
    def device_class(self):
        """Return the class of this device, from component DEVICE_CLASSES."""
        return DEVICE_CLASS_OUTLET

Aha ok, I took the switch parts from the smartthings/switch.py, which doesn't contain this..

I am aware of these however, (see above), I can only see ways to turn on/off whole zones, not individual devices.. At least in the v1 API.

Correct.

The problem is, and this is why I haven't been rushing to implement this functionality, that HA switches have no concept of operating mode / profile. That is, they expect to be turned off, and turned on - they have no concept of following a schedule (Timer mode)...

Latest push: https://github.com/home-assistant/home-assistant/pull/28182/commits/857dce085c456c15f275fc7d4edbeb96dfb593ab - should contain all of the suggestions, apart from the discussion on "!= "off" .

FYI: just identified an issue with the geniushub-client, where is can't change the override on a on/off zone.

I think the set override time (without temp) on the client is needed, as otherwise the on/off only blinks it on for a second or two.
Anything else?

Anything else?

Sorry, got side tracked with another PR (now done).
The changes are here: https://github.com/zxdavb/geniushub-client/compare/v0.6.28...toggle_on_off_zones
It previous had a duration of 0 seconds for override mode - it is now 3600 seconds.

Ah awesome, much better!
On 29 October 2019 21:49:42 GMT+00:00, David Bonnes<EMAIL_ADDRESS>wrote:

Anything else?

Sorry, got side tracked with another PR (now done).
The changes are here:
https://github.com/zxdavb/geniushub-client/compare/v0.6.28...toggle_on_off_zones
It previous had a duration of 0 seconds for override mode - it is
now 3600 seconds.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/home-assistant/home-assistant/pull/28182#issuecomment-547644937

--
Sent from my Android phone with K-9 Mail. Please excuse my brevity.

Please edit manifest.json to bump the client (NB: 0.6.29):
{
  "domain": "geniushub",
  "name": "Genius Hub",
  "documentation": "https://www.home-assistant.io/integrations/geniushub",
  "requirements": [
    "geniushub-client==0.6.29"
  ],
  "dependencies": [],
  "codeowners": ["@zxdavb"]
}

... then run: python3 -m script.gen_requirements_all

Please edit manifest.json to bump the client (NB: 0.6.29):
{
  "domain": "geniushub",
  "name": "Genius Hub",
  "documentation": "https://www.home-assistant.io/integrations/geniushub",
  "requirements": [
    "geniushub-client==0.6.29"
  ],
  "dependencies": [],
  "codeowners": ["@zxdavb"]
}

... then run: python3 -m script.gen_requirements_all

I was literally about to look how to do that, well mind read..

Please edit manifest.json to bump the client (NB: 0.6.29):
{
  "domain": "geniushub",
  "name": "Genius Hub",
  "documentation": "https://www.home-assistant.io/integrations/geniushub",
  "requirements": [
    "geniushub-client==0.6.29"
  ],
  "dependencies": [],
  "codeowners": ["@zxdavb"]
}

... then run: python3 -m script.gen_requirements_all

I was literally about to look how to do that, well mind read..

Hrm, except that updated the reqs.. if I tell it then to install from requirements_all.txt (or just run it), it doesnt update.. I did pip3 install geniushub-client==0.6.29 instead

Hrm. Didnt work - that is, still doesnt set override to an hour.. retested with just latest client from repo to make sure.. doesnt with ghclient either.. time to debug (over on the client repo)

Hrm, except that updated the reqs.. if I tell it then to install from requirements_all.txt (or just run it), it doesnt update.. I did pip3 install geniushub-client==0.6.29 instead

HA installs the updated client library from PyPI next time you start HA.
Confirm this by removing geniushub-client, and restarting HA:
pip uninstall -y geniushub-client
pip list | grep genius
hass

...

pip list | grep genius

Hrm. Didnt work - that is, still doesnt set override to an hour.. retested with just latest client from repo to make sure.. doesnt with ghclient either.. time to debug (over on the client repo)

The script in this issue works for me - have a go at it: https://github.com/zxdavb/geniushub-client/issues/25
You can push your latest PR, and I'll have a look.

Hrm, except that updated the reqs.. if I tell it then to install from requirements_all.txt (or just run it), it doesnt update.. I did pip3 install geniushub-client==0.6.29 instead

HA installs the updated client library from PyPI next time you start HA.
Confirm this by removing geniushub-client, and restarting HA:
pip uninstall -y geniushub-client
pip list | grep genius
hass

...

pip list | grep genius

Aha, the "remove existing version" wasnt obvious, I assumed it would upgrade if given a higher version.. anyhoo manually got there in the end.

Hrm. Didnt work - that is, still doesnt set override to an hour.. retested with just latest client from repo to make sure.. doesnt with ghclient either.. time to debug (over on the client repo)

The script in this issue works for me - have a go at it: zxdavb/geniushub-client#25
You can push your latest PR, and I'll have a look.

I haven't changed it..
Hm, the ghclient.py one? Aha! I'm just calling --zone=X --mode=override .. which still doesn't send a duration (and I think ought to).

I haven't changed it..
Hm, the ghclient.py one? Aha! I'm just calling --zone=X --mode=override .. which still doesn't send a duration (and I think ought to).

To make life more interesting / weird .. now I've tried --zone=9 -t0 -s3600.. and now the light is "Override: Off, override duration: 01:00" in the genius UI

Aha, the "remove existing version" wasnt obvious, I assumed it would upgrade if given a higher version.. anyhoo manually got there in the end.

You shouldn't need to - I just find that it is often necessary in a dev environment.  Can you confirm which version you're running?

I haven't changed it..

If you've updated manifest.json, then you have changed it.

To make life more interesting / weird .. now I've tried --zone=9 -t0 -s3600 .. and now the light is "Override: Off, override duration: 01:00" in the genius UI

To be clear, for an on/off zone, -t0 is off, -t1 is on, and the absence of -s is the same as -s3600.

python ghclient.py ${HUB_ADDRESS} -u ${USERNAME} -p ${PASSWORD} -z11 -v
{"id": 11, "name": "WiFi Repeater", "type": "on / off", "mode": "override", "setpoint": false, "override": {"duration": 865, "setpoint": false}}

python ghclient.py ${HUB_ADDRESS} -u ${USERNAME} -p ${PASSWORD} -z11 -t1

python ghclient.py ${HUB_ADDRESS} -u ${USERNAME} -p ${PASSWORD} -z11 -v
{"id": 11, "name": "WiFi Repeater", "type": "on / off", "mode": "override", "setpoint": true, "override": {"duration": 3589, "setpoint": true}}

Please do : https://github.com/home-assistant/home-assistant/pull/28182#issuecomment-548006864
and add the two files to this PR.

This works! ;)

This works! ;)

We cannot merge until you update the manifest file as per: https://github.com/home-assistant/home-assistant/pull/28182#issuecomment-548006864

done!

I will probably bump the client again before merging.

Woops - my fault: sorted now.

Well done! We can merge as soon as the docs PR is sorted.
You're not interested in adding services are you?
https://developers.home-assistant.io/docs/en/dev_101_services.html

https://github.com/home-assistant/home-assistant.io/pull/11095


pull request for e2e test at 12/14/2016 04:26:09

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


Updated package.json for 1.5.0 release.

Coverage remained the same at 98.595% when pulling 08d14fafe6bd66f0d434528f6f0ac6c983b94e0d on 1.5.0-bump into a40a38684bf8d788782803927d70fb8186db716e on master.

Coverage remained the same at 98.595% when pulling 728953f1297258ac675d99a21f66c40553aaffd8 on 1.5.0-bump into a40a38684bf8d788782803927d70fb8186db716e on master.


docs(pagination): update a11y tab bugs for pagination nav

Closes #4351
This PR updates a few image bugs and content sections to reflect accurate accessibility information in regards to the pagination navs interactions.

Changed

Changed images with correct annotations.
Changed some sections of content to reflect correct annotations.

@thyhmdo Made the changes!


Float#% does not work

eg. 3.0 % 2 results in

Error: Error { msg: "method MethodFirstname("%") not found on ClassFullname("Float")", backtrace:    0: backtrace::backtrace::libunwind::trace

I think this is resolved? And I saw the % method is implemented in float.sk.
$ cargo run -- run a.sk
    Finished dev [unoptimized + debuginfo] target(s) in 0.06s
     Running `target/debug/shiika run a.sk`
clang: warning: argument unused during compilation: '-nopie' [-Wunused-command-line-argument]
$

Looks like it works. I've updated the test.


Bundler sometimes fails to detect that it is running on jruby

Hi there,
I have a bit of a strange issues that happens for years now on different computers in our company:
We're using buildr as build tool and jRuby as ruby implementation. Buildr is available in two versions:

one jRuby (-java) version which uses the native Java-Support
one MRI version which depends on RJB as java bridge

RJB comes with a native extension and fails to install on jRuby.
It sometimes happens that bundler "thinks" it is on MRI when it really is on jRuby and tries to install the normal MRI-Version.
Sometimes means, that it is hard to reproduce and most times, we where able to solve it through installing a fixed (older) version of buildr?
Is this some caching issue?
I'm not sure if it is on the bundler or the rubygems side.
Best,
Tobias

Thanks, for the insight...and some laughter. i.e. ... "just stop...problem solved".


https://ticker.openbazaar.org/api behind cloudflare: tor

I tried to setup the latest release of openbazaar over tor (tor only, NOT dual stack). Everything works fine, except the error
invalid character '<' looking for beginning of value
In the console output of openbazaar-go.
In the ob.log, i found following line:
09:20:42.861 [ERROR] [exchangeRates/fetch] Failed to decode JSON from https://ticker.openbazaar.org/api invalid character '<' looking for beginning of value
After manually calling the api with curl:
curl -x socks5h://<IP_ADDRESS>:9050/ https://ticker.openbazaar.org/api
I realised  noticed following output:
~ snip
Attention Required! | Cloudflare
~ snip
In other words, the ticker api is behind cloudflare, and tor-only openbazaar nodes cannot reach it, thus are unable to parse the output, thus show errors in their log and console output

Disabled tor proxy please try again @mocacinno

Sorry for the slow response, was AFK during the weekend.
I did test without proxy and it worked fine... At the moment it's also working with proxy after i changed my circuit and got a new exit node. I have no idear if the cloudflare settings for the api were changed or if i just got lucky and have a new exit node cloudflare hasn't blocked yet

Yes, we disabled the tor proxy. If the problem continues, please reopen this issue.


How many epochs are required to converge?

Hi. Thanks for your contribution.
These days I  am training your network. The total loss fluctuates between 1.5 and 3 after 10 epochs while the edge_loss is fixed at 0.00244~. Does this means I have achieved a convergent result? How many epochs are required in your expriment? Or can you share your trained model with us? Your work will work for us very much. Eagerly awaiting your reply..

also concerned.

I also faced such problems. I followed the guides from the author, but the result in KITTI eigen split is not very good. I trained the network in KITTI for 20w steps and the performance is terrible, abs_rel is 0.2322.

The best performance I achieved is in the 15th epoch with abs_rel 0.1924.

May be there are some details for training and test have not been published.
At least I can't reproduce the performance in the paper.


How to set HTTP read timeout?

I have a request and this response  block more than 2 min，then browser throw a error，as below.

so,how should I set unlimited response timeout？
THX！

Hi,
Thank you for your question.
Please use StackOverflow. I think that would be a better place to troubleshoot your setup.
Thanks.

This is not 3d party troubleshoot. In my case i set timeout in express response object
response.setTimeout(seconds * 1000, () => {
  console.error('Request has timed out.');
  res.writeHead(408, 'Request Timeout');
  res.end('Timeout!')
});

When i send direct request (without http-proxy-middleware) all works fine. But when i use proxy, request canceled on default 2 minutes.
app.use(proxy('/api/', {
  target: 'http://localhost:4000/',
  ws: true,
  pathRewrite: {
    "^/api/": "/"
  }
}));

Did you try the timeout option?
https://github.com/chimurai/http-proxy-middleware#http-proxy-options


[WIP] Replace grad norm. try to fix TPU dataloader issue

Still playing around with TPU. I can now train one epoch fast, but when I start the second epoch it just ends with no error. Trying to figure out why that could be:
It seems to be something about this:
https://github.com/pytorch/xla/issues/1191
Not sure if TPU dataloaders need to be specifically restarted?

Hello @srush! Thanks for opening this PR.

In the file pytorch_lightning/trainer/data_loading.py:

Line 179:13: W503 line break before binary operator
Line 193:13: W503 line break before binary operator
Line 194:13: E128 continuation line under-indented for visual indent
Line 194:13: W503 line break before binary operator

Do see the Hitchhiker's guide to code style

@dlibenzi @mruberry any ideas?

@dlibenzi @mruberry any ideas?

I've got no error, how can I have an idea? :smile:
You are re-creating the ParallelLoader at every EPOCH, right?

@dlibenzi nope! is that the issue?
https://github.com/PyTorchLightning/pytorch-lightning/blob/be244560b24b68b0236a4694707fb9bb63c2e6d0/pytorch_lightning/trainer/training_loop.py#L451-L455

@dlibenzi nope! is that the issue?
Currently, we create it right before all the epochs.
https://github.com/PyTorchLightning/pytorch-lightning/blob/be244560b24b68b0236a4694707fb9bb63c2e6d0/pytorch_lightning/trainer/training_loop.py#L451-L455

Yep, that's the issue :wink:

lol. nice. @srush want to fix in this PR?

@srush i'll take care of the TPU thing on a different PR. Let's make this about the grad norm

Yes... fixed!

Check out the video of it working :)
https://twitter.com/PyTorchLightnin/status/1232813118507692033?s=20

Awesome! Closing this guy. Switching to https://github.com/PyTorchLightning/pytorch-lightning/pull/959


Waypoint doesn't trigger on list made with .map

A fellow dev and I have tried for hours now.
There seems to be no way to get Waypoint to work in components that create lists. We also tried putting the waypoint on the main page but it seems to think it's inside even after the list populates. I am now trying to force it with position absolute and same.
Here's the component for reference:
const EventsList: React.FC<IEventsList> = ({ events, handleEnd, lang, distanceMetric, loading, t, ad }) => {
  return (

              Events

            {events.map(event => (
              <EventCard upcomingEvents key={Math.random()} event={event} t={t} distanceMetric={distanceMetric} lang={lang} />
            ))}
            <ScrollUpButton />

            <Waypoint onEnter={({ previousPosition, currentPosition }) => handleEnd({ previousPosition, currentPosition })} />

            {ad}

{loading ? {t('Loading')} : {t('noevent')}}

  );
};

Im using 10.1.0 is this a known issue or am I doing something wrong.

bump

It's hard to know what's going on without also knowing the css styles for the elements. The most likely thing is that the waypoint is consistently "visible" in the scrollable parent container. If you inspect the DOM and look for the react-waypoint div, where is it located?


Only list profiles in selected category

Fixes #1069
In this PR I've:

Fixed listing profiles when a category is selected (e.g. https://speakerinnen.org/de/categories/11). It was displaying all profiles, now it displays only those that are tagged with tags in selected category.
Fixed the counting of those profiles in selected category.
Deleted the custom_params() method in profiles controller, because it's not called from anywhere.
Deleted the categories controller, which isn't used anymore. There are no routes leading there.

In this PR I made two choices, which are very much open for debate:

I removed the "ORDER BY random" when searching for profiles in a category. The random order is not persisted when paginating through pages, which means that on each page in the results, it'll reorder all profiles, which means that duplicate profiles may appear on multiple pages. It also means that one may end up not finding all profiles in the category, unless they refresh over and over again.
In my opinion, the user experience is damaged this way.
I think there may be a case to make for choosing a different order for each session.
One idea might be introducing some kind of ranking algorithm, based on the number of views for each profile. This, of course, holds other complications.
Another idea might be to use "ORDER BY random", but somehow persist the order for the user's session, so that they are able to browse all profiles, and won't get a different set of profiles on each page refresh.
I've removed the main_topic_translated_in  filter when listing category profiles. The reason is that this filter isn't applied on the homepage when we list the number of profiles in each category. So if someone sees that there are 500 profiles in "Career & Education", then clicks it and sees that there are 270 (because they are in locale EN), I think it can be confusing. Also, if they switch to DE, they will see 230, which can confuse even further.

Hey @alonpeer and @tyranja
ad 2) didn't we use this filter/logic also when displaying search results?
The thinking behind that was: if people look on our site in :en locale they expect english language content. They probably can't even read the things speakers with only german speaking content write in their profile. How can they now select those with english language content?

@zaziemo I understand what you're saying, and yes, I agree that one should be able to filter by language. However, I think there should be a separation between the interface language and filtering results. I think one should be able to browse the website in language A and look at profiles that have language B, one shouldn't affect the other.
I think that we should come up with a way to allow one to filter by language as well once they enter a page such as https://speakerinnen.org/en/categories/9. Maybe we can add that filter at the top or something? I don't know, I'm not a designer 😅

I just noticed a bug on one of those pages, I reported it here: https://github.com/rubymonsters/speakerinnen_liste/issues/1073
Perhaps it's related to this PR.


Add message option for exit command

Would be nice to be able to pass a message to the exit command that people will see when they get disconnected.
IE: /exit "Restarting server. Rejoin in 5 minutes!"

Good idea. It sounds like a plugin idea. but would be good in the base as well. Might also want an integer argument for time.

@skylord123 I've implemented the message for the time being as the old tdsm used to send a going down message, so i figure it wasn't hard to allow the option for it too., @D1G1T4L3CH0 the time may be nice, but it could be for a plugin too; up for discussion anyway.


OpenCL head: "Intrinsic has incorrect return type!"

Ubuntu 17.10, Vega 64.
With freshly built opencl, I get this on clGetPlatformIDs() invocation, e.g.:
$ LD_LIBRARY_PATH=/home/preda/rocm/opencl/build/lib clinfo
Intrinsic has incorrect return type!
i8 addrspace(2)* ()* @llvm.amdgcn.dispatch.ptr
Intrinsic has incorrect return type!
i8 addrspace(2)* ()* @llvm.amdgcn.implicitarg.ptr
Intrinsic has incorrect return type!
i8 addrspace(2)* ()* @llvm.amdgcn.implicitarg.ptr
Intrinsic has incorrect return type!
i8 addrspace(2)* ()* @llvm.amdgcn.implicitarg.ptr
Intrinsic has incorrect return type!
i8 addrspace(2)* ()* @llvm.amdgcn.dispatch.ptr
Number of platforms                               0

fixed in https://github.com/RadeonOpenCompute/ROCm-Device-Libs/commit/0f4a9922bd3b0ce15821257645b026d1b1fc6d4d

I have the same problem on 18.04, I think that we'll have to revert to the older driver.

I also have the same problem on 18.04, with the padoka PPA.

If you are having this issue in ubuntu 18.04
you likely need to install Rocm
https://rocm.github.io/ROCmInstall.html#ubuntu-support---installing-from-a-debian-repository


[WFLY-19982] Upgrade WildFly Core to 27.0.0.Beta3

Jira issue: https://issues.redhat.com/browse/WFLY-19982

Tag: https://github.com/wildfly/wildfly-core/releases/tag/27.0.0.Beta3
Diff: https://github.com/wildfly/wildfly-core/compare/27.0.0.Beta2...27.0.0.Beta3

        Feature Request

[WFCORE-6935] -         Improve systemd service units

        Bug

[WFCORE-6981] -         Spurious data in upload-deployment-url reply description

[WFCORE-7031] -         Unstable annotations package should be provisioned only for preview stability level

[WFCORE-7043] -         Subsystem-level testing of core-managment is incomplete

[WFCORE-7047] -         Installation manager integration tests are not executed in the CI

[WFCORE-7058] -         CachingRealm should start lazily in admin only mode

        Task

[WFCORE-7044] -         Split the launcher into its own repository

        Component Upgrade

[WFCORE-6997] -         Upgrade snakeyaml to 2.3

[WFCORE-7046] -         Upgrade Galleon to 6.0.4.Final

[WFCORE-7048] -         Upgrade licenses-plugin to 2.4.2.Final

[WFCORE-7057] -         Upgrade JBoss Parent to 47

[WFCORE-7060] -         Upgrade Jboss Logging Tools to 3.0.3.Final

        Enhancement

[WFCORE-7039] -         Add ServiceInstaller.Builder.onRemove(...) to accomodate ValueRegistry removal

[WFCORE-7049] -         Remove MaxMetaspaceSize settings

Thanks @yersan


[Mega menu & dropdowns]: overlay vs no overlay

User story
The Cloud team reached out to ask: why there's no overlay being shown for regular dropdown and only for mega-menu?
Answer from Olivia in slack:

The mega menu takes up so much of the screen that we wanted to ensure it was really differentiated from the page.
Where as the dropdown is a more condensed experience it doesn’t need the overlay. I’m not sure if I’ve seen an example of another site/experience that uses an overlay for a dropdown
When I first designed the mega menu we only anticipated it being used for the global masthead, and that all of the links would have a mega menu.

Now that the Cloud masthead has a mix of mega menus and dropdowns we might want to look at if we think having an overlay for mega menu and no overlay for dropdown is the best solution.
Initial ideas to explore

Turn off overlay for when there are both mega menu and dropdown
Inset mega menu and turn off overlay
Add overlay when there are both mega menu and dropdowns
Leave as is (note: Olivia doesn't think this is a huge deal because the user has to click on each dropdown/mega menu to open it, so the experience isn't as noticeable) But we should check against an example that might like a mega menu, but it would have less content aka the mega menu wouldn't take up the entire screen.

Additional information
Cloud example: https://www.ibm.com/cloud/watson-assistant

Acceptance criteria

[ ] {{State acceptance criteria}}
[ ] {{State another}}
[ ] {{And another}}

@oliviaflory Thanks for creating this issue. Please keep me posted on the decision that will be coming out from the design research.

We will be OK with mixing the mega menu and dropdown and allowing for the mix of overlay treatment.  Closing.


Github actions: server build

This pull requests closes https://github.com/deep-foundation/deeplinks/issues/324.


Galaxy deployment via Tools Hub on the EOSC EU Node

Hello,
@kotsu21, @bgruening, and I are testing the TOSCA templates for Galaxy in the EOSC EU node. We would like to showcase this in the upcoming EOSC Winter School as part of the Opportunity Area 4: https://eosc.eu/eosc-focus-project/winter-school-2025/
A deployment of the below template does not seem to work:
https://github.com/grycap/tosca/blob/eosc_lot1/templates/galaxy.yaml
Via the Tools Hub on the EOSC EU Node:
https://open-science-cloud.ec.europa.eu/dashboard/tools-hub/tool-view?eo=MjEuMTExNjcvQnVZYk53
It creates this endpoint that seems inaccessible: https://<IP_ADDRESS>/galaxy
Has this TOSCA template been tested successfully in the past? or does it need an update?
Best regards,
Sebastian

Hi @sebastian-luna-valero,
The problem with this recipe is that it takes some time to be up, after the infrastructure has been configured.
Could you try again and wait some minutes to ensure that all the services have completed the initialization steps?

As you say, the deployment took some time but generated the output endpoint https://<IP_ADDRESS>/galaxy (it may have been expired already as it was created for testing with the minimum credits).
By the way, should my OS be Linux to access this endpoint?

As you say, the deployment took some time but generated the output endpoint https://<IP_ADDRESS>/galaxy (it may have been expired already as it was created for testing with the minimum credits). By the way, should my OS be Linux to access this endpoint?

Yo only need a browser, it can be used by any OS,
In this case it seems that there is a misconfiguration.
I will test it again.

I think we tried 5min later or so and we still just saw the nginx error page.
Which version of the container is used here? The new version 24.1 should have a nicer nginx error page.
Thanks @micafer for looking at this.

I updated the version of the helm chart and now it deploys last Galaxy version.
But there is no nice error page ...
The problem is that the amount of resources set in the Galaxy EU Node template are to low.
You could try creating a copy of the Galaxy template in your tools, and increase the number or resources.
You only need to update the input values.
The recommended option is to set 3 WNs with 2 CPUs and 8 GBs or RAM.
Please test it and let me know.

Thanks for the update.  I changed the input values (see below) and it took 20 minutes to complete deploying.  The endpoint is still not accessible via https://<IP_ADDRESS>/galaxy.  What interface are we expecting to see?
wn_num:
type: integer
description: Number of WNs in the cluster
default: 3
required: yes
wn_cpus:
type: integer
description: Number of CPUs for the WNs
default: 2
required: yes
wn_mem:
type: scalar-unit.size
description: Amount of Memory for the WNs
default: 8 GB
required: yes
When I clicked the deployed VM to view externally from the EU Node platform, api.cloud.psnc.pl took too long to respond.


As a logged-in user (Expert) I want to choose Title, Tags, Text for Material card on the Article/Note page so that I can create a correct material.

Each user with the Expert role should be able to choose Title/s for his Article/Note
List of topics:

AC:
Experts should be able:

select 1-3 topics from the list
change the choice of topic

Clicking on the topic name adds it to the list of selected topics
Сlicking on a topic deselects the topic
Each user with the Expert role should be able to choose tag/s for his Article/Note
List of tags:

AC:
Experts should be able:

select a tag
add a new tag
deselect the tag
edit the tag

Clicking on the + button displays a tag input field and a list of existing tags
Clicking on tag displays a text input field for editing the tag
Deleting tag text deselects the tag
Each user with the Expert role should be able to create Text for Material card for his Article/Note
Window with a text editor for material card text:

AC:

By default, the text of the card is taken from the beginning of the text of the Article/Note
Expert should be able to edit the text of the card
Change text should be displayed on card preview
Text for Material card length is from ???

Each user with the Expert role should be able to see text for Material card preview

The material card should contain:
Author's picture, first name, last name, primary place of work
Material type (article, note, video etc.), topic and title. For articles, notes: preview of material (4 lines). For video: player preview
Date of adding
Use https://github.com/ita-social-projects/dokazovi-be/issues/22
Preview button/Publish button

Сlicking on the Preview button redirects to the Preview page
Clicking on the Publish button publishes content

BA check 18/07/21
Not accepted.
The list of available topics on site is smaller than the list of topics represented on design.
Tags are not implemented on the site.
The text of the card is not taken from the beginning of the text of the Article/Note.
Material card does not contain primary place of work.

BA check 20/07/21
Not accepted.
The list of available topics on site is smaller than the list of topics represented on design(outdated design).
Tags are out of scope for first release(https://github.com/ita-social-projects/dokazovi-be/issues/280).
Material card was redesigned(https://github.com/ita-social-projects/dokazovi-be/issues/229).
Text for Material card length is from?
Expert can not pubish content.


Beautify welcome page

Made the front page all fancy. Still not perfect but certainly getting there.

Only issue I found was a couple warnings in ./src/templates/mdTemplate.js:
i ｢wdm｣: Compiling...
WARNING  Compiled with 3 warnings                                                                                3:24:50 PM
warning  in ./src/templates/mdTemplate.js
"export 'Portal' (imported as 'reactstrap') was not found in 'reactstrap'
warning  in ./src/templates/mdTemplate.js
"export 'index' (imported as 'reactstrap') was not found in 'reactstrap'
warning  in ./src/templates/mdTemplate.js
"export 'setupTests' (imported as 'reactstrap') was not found in 'reactstrap'
After talking with Ryan, he said he used a python script to generate the names of all the reactstrap components for the md files. It's likely they don't exist (like index).
Once this is fixed Ryan can go ahead and merge it in by my watch!

Once those warnings mentioned above are fixed, I think we can merge this in. Will add my touches to the index page after the merge.

Warnings are fixed.
@ethanWallace was the addition of wide margins on the sides of the page intentional? It's inconsistent with the mockup, just wanted to be sure before merging.

@ryanlacroix the margins came from using Bootstrap's columns. If we want to increase the size of the container, we can do it at another time.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/32 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 84e8a7f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Autorest Regen Preview to 3.0.0-alpha.20230928.3 by Dapeng Zhang from refs/pull/3792/merge

Triggered from https://github.com/Azure/autorest.csharp/pull/3792

API change check
API changes are not detected in this pull request.


python3.pkgs.keyring: 21.5.0 -> 23.0.0

depends on https://github.com/NixOS/nixpkgs/pull/113028
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

included in https://github.com/NixOS/nixpkgs/pull/117452

included in #117452

The updates in that PR are of lower quality because they often miss other changes except the update and to get it to build. Individual PRs are always preferred.


Tile doesn't appear in quick settings.

Hey dev,
I gave your app a trial today, as the "Caffeine Tile" in F-Droid that I used formerly hasn't been maintained in 3 years.
Unfortunately the Tile doesn't appear in the quick settings. Is there any troubleshooting I can do on my end? Do permissions have to be set?
Thank you,
Jay

My bad, I missed the tile in the settings.


limit validation errors or spit out only first one

Hi,
first of all thanks for this fabulous library. Much nicer to use than struct tags!
Unfortunately I encountered a memory and CPU problem when using ozzo to validate large structs or slices.
The problem stems from the fact that ozzo spits out all validation errors, which can be a lot in a large map or slice.
In particular we ran into an issue in production, where we were using ozzo to validate json payloads of a REST API. The expected payload in this API is a nested struct which contains slices typically containing more than 100,000 items. If the caller of this API sends a payload where each array item has a validation error (even if it's practically the same error for each item), ozzo will create 100,000 validation errors. Since each validation error in ozzo is a map, it means ozzo creates 100,000 maps which use quite a lot of memory and also spit out rather noisy and unreadable, super long validation error messages. Since validation errors are typically of very ephemeral nature, the just-created-errors will become very soon eligible for garbage collection.
In our case this meant suddenly our app's CPU usage pegged to 100% and most of it was spent on creating validation errors and garbage collecting them afterwards.
This actually caused some semi outage in one of our production apps after introducing ozzo.
In summary, it would be nice if ozzo has a way to limit the creation of validation errors (i.e. max 10) or some sort of short circuiting once the first validation error in a structure was encountered.
My PR #93 introduces a new rule EachWithFirstErrorOnly which introduces this capability for large maps and slices, but I think having the capability to limit validation errors a first class option in ozzo (instead of just a rule variation) would be better.
Thanks.

Hey, is there a good solution to this problem now?


IPython javascript widgets in colaboratory?

Hi
I have a short IPython notebook with a simple widget that runs fine under IPython notebook 2.1.0, but which I can't get to work with colaboratory. I'm running the "classic web app" on my own computer i.e. python -m colaboratory, I launch chrome with --disable-web-security, and I visit http://<IP_ADDRESS>:8844 to use colaboratory.
The notebook contains the following three cells (the .ipynb file is at https://drive.google.com/file/d/0B2GEeTv36tvTb0FCRWU3eHFqWWM/view?usp=sharing):
from __future__ import print_function
from IPython.html import widgets
from IPython.utils.traitlets import Unicode

class DateWidget(widgets.DOMWidget):
    _view_name = Unicode('DatePickerView', sync=True)

from IPython.display import Javascript
Javascript("$.getScript('test.js');")

DateWidget()

And test.js is just:
require(["widgets/js/widget"], function(WidgetManager){

    var DatePickerView = IPython.DOMWidgetView.extend({
        render: function(){ this.$el.text('Hello World!'); },
    });

    WidgetManager.register_widget_view('DatePickerView', DatePickerView);
});

This works in IPython notebook. For testing in colaboratory, I put test.js in colaboratory/resources/colab/ and I changed the second cell's Javascript() call to Javascript("$.getScript('/colab/test.js');"). When I call DateWidget() I don't see any widget, and the browser console log says:
No such target registered:  WidgetModel comm.js:80
Available targets are:  Object {}

I'm not familiar with IPython notebook's widgets, so I'm trying to puzzle my way backwards from this error message at the moment. I'd be grateful for any pointers!
Thanks
Chris

This version of colaboratory is no longer supported, please switch to https://colab.research.google.com.


Make the named frontend routes customisable

In core/server/routes/frontend.js there are a number of named routes which use English words including tag, author and page. These need to be abstracted out so that they can be customised for two reasons:

translations: when we support i18n properly, we'll want to allow users to translate those words
flexibility: if you're running a music blog, you might want the route to be /genre/ rather than /tag/, and perhaps you'd rather the route be /by/ rather than /author/.

At the moment, these routes are hardcoded strings repeated multiple times through the codebase, and depended upon for creating logic elsewhere in Ghost.
This needs to be abstracted out into information that can be used to dynamically build the correct routes and assign the right context.
Notes on refactoring
At present there are effectively 3 main contexts which have a top-level route, and a set of subroutes.

index - lives at /
tag - lives at /tag/:tag_slug/
author - lives at /author/:author_slug/

All three of these have the following subroutes:

/ - the first page of results
/page/:page_num/ - subsequent pages of results
/rss/ - the first page of the RSS feed
/rss/:page_num/ - the subsequent pages of the RSS feed

Also note that at present /feed/ is an alias of the base/index /rss/ route only, when in fact feed should probably be an alias anywhere where rss is used, and should always 301. I can imagine Apps reasonably wanting to specify other aliases for this.
Fundamentally, the main thing we need to achieve is making it possible to replace the words tag, author and page with other words without things breaking, the other information here is intended to paint a bigger picture of what this refactor needs to move us towards.

@mattiascibien @ErisDS I'm interested in taking a fresh look at this if there has been no recent developments.

@katiefenn did not had time to work on this. :( you can have a look.

Are hard-coded URLs in themes within the scope of this refactor?
For example, take tag RSS subscription links in Casper:
https://github.com/TryGhost/Casper/blob/master/tag.hbs#L8
The "tag" keyword is hardcoded, and would break if a tag keyword setting were changed to "genre".
It seems a change to the template is, strictly speaking, outside of the scope of Ghost itself. But should I consider how best to expose a keyword setting to theme templates?

For themes to be able to route to customized front-end urls, ghost is going to need to add a new {{url-for}} helper that looks up these customized routes.
I'd say that's outside the scope of any PR against this issue, but thanks for bringing it up. We'll have to make a new issue for it, and add all the necessary docs to go along with that :)

@katiefenn agreed, that's definitely not in the scope of the refactor - also the Casper theme can be updated to use the {{url}} helper to achieve the same thing without hardcoding. I'll update this by way of an example to other themes.

@ErisDS @novaugust Thank you for the feedback. I'm getting ready to create another PR (compare), and wanted to first check that the refactor matches what was asked for:

I've added the route keywords tag, author, and page to the config manager.
Refactored hard-coded instances of the keywords to build strings using configured keywords
Added keywords to stub config in unit tests.

There are things which I could use some feedback on:

Naming of config properties (eg. config.routeKeywords.tag) - could perhaps be simpler.
Placing keywords in config. I've concentrated on refactoring the hard-coded strings and put the keywords in config for convenience. If there is a better place, I can move them.

That compare looks sensible to me. It'll probably be easier to get feedback on the PR if you just open it up :)


Make deliver aware of review_user_needed

The review_user_needed checkbox strikes again: Deliver isn't aware of it.
Added the same logic as before. Did not test by actually submitting something to review.
#4241, #4255

I believe that the review_user_needed attribute was added to spaceship very recently, so please update the deliver.gemspec to specify a new minimum spaceship version of 0.26.1

Any news with this PR?

danger is failing, let's see if that fixes it https://github.com/fastlane/fastlane/pull/4297

Merged with https://github.com/fastlane/fastlane/pull/4297


Bump Newtonsoft.Json version

Upgrade Newtonsoft.Json to latest version
Changelog
Add ability to use old elasticsearch and client version in new projects

Hi, @stevejgordon thank you for answer
I understand that 5.x version is no longer supported, but applications that use old Elasticsearch version and client is still exists
I can't reference a newer Newtonsoft.Json version in my project, because receive nuget error about NEST 5.6.6 require Newtonsoft.Json version in range 12.0.1 .. 13.0.0, but I try to install 13.0.1 (it's only one existing version with 13 major)
I think bump Newtonsoft.Json version in package helps many people who can't right now update Elasticsearch but want to use newer packages

@oldmine Ok I understand the problem. Unfortunately that is the trade-off of remaining on an out of support version. These versions will receive no new releases.


adds a "version" JMX attribute to the agent mbean

We will need this in the future so we can collect it as a resource property later.

Seems fine. Waiting to go green...


Update to WebView2 SDK 1.0.902.49

I've tried to update the SDK, as Tauri needs a newer SDK version for this PR: https://github.com/tauri-apps/wry/pull/280
I downloaded the SDK from nuget and removed files not present in the current SDK folder. However, I was not able to regenerate the bindings using idl2rs
I figured I'd open this draft anyways, as a starting point. Some help getting this working would be much appreciated.
cargo run < ../Microsoft.Web.WebView2.*/WebView2.idl > ../src/lib.rs && rustfmt ../src/lib.rs
    Finished dev [unoptimized + debuginfo] target(s) in 0.12s
     Running `target/debug/idl2rs`
Parsing error:     --> 4357:7
     |
4357 |       [ in, size_is(originsCount) ] LPCWSTR * origins);␊
     |       ^---
     |
     = expected identifier

I'm working on an update in #18.

Getting/setting user_agent is supported now. See https://github.com/sopium/webview2/blob/d30fbb413b8e87f2f5e7f4601cfa4789b1f84f46/examples/user_agent.rs#L195


call sqld instead

This PR calls sqld instead of trying to implement hrana in go.
It puts the burden on the user to install it, but as soon as we start publishing binaries we can download it ourselves.

Will close #431 if you all think this is a better approach (@CodingDoug @honzasp )

I like pushing as much responsibility down to the actual software rather than a simulation when possible, without adding much dev overhead.
I'm wondering though, at this point, what value is the CLI adding over just running sqld yourself? I'm sure there is room to improve going forward, but I want to be clear what exactly we intend to offer in the near term.

At this point the main thing is that we make sure the addresses are set up correctly, and provide better visibility into the sqlite file. The user can use a visible file in the local directory of their choosing, that I then link into the data/ directory of sqld, that is temporary.
A good next step is to download sqld once we have builds, and follow ups, as you suggested, are auto-downloading the schema and matching versions of databases, installing extensions, etc.

btw, one thing we can do very soon - if someone helps me with that, is add certificates so we can keep using libsql:// URLs.

I'm not sure how we could do that, we have no control over the list of trusted CAs in all clients, and no CA will give us a certificate for localhost:8080. But I'm planning to add support for ?tls=0 query string to libsql client (after the new HTTP API is implemented), then we could use libsql://localhost:8080?tls=0.

@CodingDoug I'd advocate for merging this, since it already provides one QoL feature, which is the ability of passing a local sqlite file (and then we'll handle links internally), and we can soon install sqld automatically. But would love to hear from you

@glommer Sure, let's do whatever we can do make things easier in the near term.

do you want to test drive first?
@haaawk this should also fix the build on windows, as it removes moderc.org/sqlite

do you want to test drive first?

Yes, I'll do this shortly.

Command output
I'd suggest changing this output:
For environments where a filesystem is available:
→   use file:somefile.db as a URL. This server is not needed. (Ctrl-C)
For all others environments:
→   connect to ws://<IP_ADDRESS>:8080.

Without context, it's confusing how I should act on this messaging. If my intent in running turso dev was to start a local server, I'm not clear why it would be trying to talk me out of that.
What I suggest is changing the help text for this command to the following:

$ turso dev --help
Starts a sqld emulator for local development with Turso.
If you're using a libSQL client SDK that supports SQLite database files on the local filesystem, then you might not need this server at all.  Instead, you can use a file: URL with the path to the file you want the SDK to read and write.

The change the normal output:

$ turso dev
sqld emulator listening on port 8080. Use the this URL to configure your libSQL client SDK for local development:
ws://<IP_ADDRESS>:8080

[persistence-message]

[persistence-message] can be either:

This server is using an in-memory database. Changes will be lost when this server stops. If you want to persist changes, use --db-file to specify a SQLite database file instead.

or:

Using database file [db-file]

Command line args
Suggest using --db-file with short flag -f instead of --local-file.  Files are implicitly local for the developer.
Suggest adding -p as a short flag for --port.
--verbose doesn't appear to do anything.  Minimally I'd like to see a record of each incoming request so I know it's working, maybe even each JSON payload.
turso db shell
The shell does not work with <IP_ADDRESS> or localhost URLs.  This is unexpected. It errors with:

Error: hostname 'localhost' not found

I thought this used to work for running against other local sqld. Maybe something bad happened in #419?

I'll also remove verbose for now. We won't have the necessary level of control over the sqld verbosity level without patching it, so that's for later, best case.  All we can do at the moment is increase the rust log level from error to info, debug or trace, but that will print. a lot of unrelated stuff.

I don't think the issues with localhost are really solvable from our side.
It seems the client is always trying the ipv6 address on the mac, even though the hosts file lists both.
the shell issue was fixed.

Ok, to support both localhost and <IP_ADDRESS> on mac, we'd have to patch sqld to accept more than one listen-addr for the server, and explicitly listen on both <IP_ADDRESS>:8080 and [::1]:8080.
If I listen on the latter, then localhost works (but <IP_ADDRESS> doesn't)

<IP_ADDRESS> works fine for me. We can assume the user will copy the provided URL, and the server of course do whatever it's coded to do (listen on <IP_ADDRESS>).


ci: add codeQL analisys to quality workflow

Description
Add codeQL analisys to quality workflow.
Checklist
Check or add the boxes that apply to this pull request:

[x] All tests pass.
[x] Code follows style guidelines.
[x] All merge conflicts have been resolved.
[x] The pull request is targeted to the correct branch.

Proposed Changes
In order to improve code quality on CI, this PR add codeQL analisys to quality check.

[x] Add codeQL analisys

Tests Performed
Execute PR workflow.

The CodeQL analysis requires permissions that I'm not going to enable yet, so the PR will be closed.


Cleanup Imports

[] Make sure tick symbol ` is used only when parameters are implicitly obtained
[] Reorg entire repository
[] Fill in the missing rules in ProcessorCore


Horizontal scrolling in viewlets

Tracking issue for how to handle horizontal scrolling in viewlets. Currently, VS Code doesn't support horizontal scrolling in viewlets, and relies on hover over or dragging the view window. Using this issue to get feedback and track related work around horizontal scrolling.

I discovered this is actually covered by a user preference "workbench tree horizontal scrolling". We should respec this settings.

Design pattern considerations: our team has seen both scroll and truncation behavior, with scroll for VS and truncation for VSCode. The ability to resize the viewlet width combined with tooltip on hover provides a minimal workaround, but respecting a user setting/preference to toggle horizontal scrolling should provide the flexibility needed.

This is no longer needed since we're matching VS Code UX pattern.  Reopen if this needs reconsideration.

reopening the issue. I checked the setting @anthonydresser mentioned above and found 2 issues:

the servers viewlet does not respect this setting, i couldn't make it show the horizontal scrollbar
the Azure/CMS viewlets do respect the setting, but as shown in the screenshot below, the width calculation is off by maybe the width of the vertical scrollbar, the horizontal scrollbar is shown but the text is still cut by a couple letters

You should just open those up as separate issues. It helps with issue management and triage.


Type piracy?

It looks like there is type piracy somewhere related to this package:
(@v1.6) pkg> activate --temp
  Activating new environment at `/tmp/jl_FXOiIk/Project.toml`

(jl_FXOiIk) pkg> st
Status `/tmp/jl_FXOiIk/Project.toml` (empty project)

(jl_FXOiIk) pkg> add HCIDatasets
    Updating registry at `~/.julia/registries/General`
    Updating git-repo `https://github.com/JuliaRegistries/General.git`
   Resolving package versions...
  Unless there is a bad interaction between them, I suspect the type piracy is in this package.

It also happens only in Julia v1.6, but not v1.5

Uh oh, that's not good.
To start, I do a kind of nasty trick to lazy-load the data, and that's overloading Base.keys and Base.getindex for a type, like so-
abstract type Dataset end
struct BetaPictoris <: Dataset end

Base.getindex(::Type{BetaPictoris}, key) = # ... parse key and load data from file

the other possibility might be this method, which reaches into Datadeps.HTTP
https://github.com/JuliaHCI/HCIDatasets.jl/blob/f7bf4f0bac61c96161fef9759407a51ed5b0f24d/src/BetaPictoris.jl#L20-L27
any ideas?

any ideas?

Binary search in the code of HCIDatasets would be my strategy to track down the culprit

I skimmed through the code, couldn't see anything blatantly pirat-y

https://github.com/JuliaHCI/HCIDatasets.jl/blob/d52ceed9585a665c1cba0b58bbb3eaa200d68211/src/HCIDatasets.jl#L14-L15
looks very piraty to me since it also applies to Union.

@KristofferC thanks for the tip. Is the piracy removed if I avoid this-
Base.getindex(::Type{<:Dataset}, # ...

and instead write this?
Base.getindex(::Type{BetaPictoris}, # ...

I think it would be best to have
Base.getindex(::Dataset, ...)

and use singletons for the instances (like Nothing and nothing). But yes, using the concrete types like you wrote would be an improvement.

as of da2c8121a9317de6a2351edad984cda3cabc00b4 I switched to using singletons, which worked just like I needed it to. I've since bumped to 0.2.1 and started a release. I'll try recreating the previous bug and seeing if this fixes it.

I am not able to reproduce the bug on my build of julia master
Julia Version 1.6.0-DEV.1567
Commit bdcaee033e* (2020-11-24 03:27 UTC)

using the exact same command order as @giordano
pkg> activate --temp
pkg> st
pkg> add HCIDatasets
julia> using HCIDatasets
pkg> st

output
(@v1.6) pkg> activate --temp
  Activating new environment at `/var/folders/g2/n_lg14f56k776c2p35gktr000000gn/T/jl_J9SAHb/Project.toml`

(jl_J9SAHb) pkg> st
Status `/private/var/folders/g2/n_lg14f56k776c2p35gktr000000gn/T/jl_J9SAHb/Project.toml` (empty project)

(jl_J9SAHb) pkg> add HCIDatasets
    Updating registry at `~/.julia/registries/General`
   Resolving package versions...
Updating `/private/var/folders/g2/n_lg14f56k776c2p35gktr000000gn/T/jl_J9SAHb/Project.toml`
  [1893dc6a] + HCIDatasets v0.2.0
Updating `/private/var/folders/g2/n_lg14f56k776c2p35gktr000000gn/T/jl_J9SAHb/Manifest.toml`
  [3b1b4be9] + CFITSIO v1.0.0
  [124859b0] + DataDeps v0.7.4
  [525bcba6] + FITSIO v0.16.3
  [1893dc6a] + HCIDatasets v0.2.0
  [cd3eb016] + HTTP v0.9.0
  [83e8ac13] + IniFile v0.5.0
  [692b3bcd] + JLLWrappers v1.1.3
  [739be429] + MbedTLS v1.0.3
  [189a3867] + Reexport v0.2.0
  [5c2747f8] + URIs v1.0.1
  [b3e40c51] + CFITSIO_jll v3.48.0+0
  [c8ffd9c3] + MbedTLS_jll v2.24.0+1
  [0dad84c5] + ArgTools
  [56f22d72] + Artifacts
  [2a0f44e3] + Base64
  [ade2ca70] + Dates
  [f43a241f] + Downloads
  [b77e0a4c] + InteractiveUtils
  [b27032c2] + LibCURL
  [76f85450] + LibGit2
  [8f399da3] + Libdl
  [56ddb016] + Logging
  [d6f4376e] + Markdown
  [ca575930] + NetworkOptions
  [44cfe95a] + Pkg
  [de0858da] + Printf
  [3fa0cd96] + REPL
  [9a3f8284] + Random
  [ea8e919c] + SHA
  [9e88b42a] + Serialization
  [6462fe0b] + Sockets
  [fa267f1f] + TOML
  [a4e569a6] + Tar
  [8dfed614] + Test
  [cf7118a7] + UUIDs
  [4ec0a83e] + Unicode
  [deac9b47] + LibCURL_jll
  [14a3606d] + MozillaCACerts_jll

julia> using HCIDatasets

(jl_J9SAHb) pkg> st
Status `/private/var/folders/g2/n_lg14f56k776c2p35gktr000000gn/T/jl_J9SAHb/Project.toml`
  [1893dc6a] HCIDatasets v0.2.0

I made the code less susceptible to this issue: https://github.com/JuliaLang/julia/pull/38511.

I can confirm this is fixed now, thanks everybody!

It is not fixed though. It still type pirates it and any code using the same syntax I did to create such an array will get the same error here.


Do you have any plans to provide a light theme?

Thanks for providing a useful API.
I wonder if you also have plans to offer light-themed badges.

Thank you for your interest.
Today I added a light theme. :)

Awesome!
I also applied the light theme directly to my GitHub profile.


feat(longitudinal controller): add params explanation

Signed-off-by: Takayuki Murooka<EMAIL_ADDRESS>Related Issue(required)
https://github.com/autowarefoundation/autoware.universe/issues/380
Description(required)
Added the explanation of parameters of ongitudinal controller
Review Procedure(required)

Related PR(optional)

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[ ] Read commit-guidelines
[ ] Assign PR to reviewer

If you are adding new package following items are required:

[ ] Documentation with description of the package is available
[ ] A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets

CI Checks

Build and test for PR / build-and-test-pr: Required to pass before the merge.
Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

@tkimura4 Thanks for the comment! Fixed them in https://github.com/autowarefoundation/autoware.universe/pull/381/commits/cec511a366f995e886bbadea2b882b0ec5fd3bc8 :pray:

@takayuki5168 Thank you!


Examine use of UnsignedLong.longValue() in datastructures.main

create tickets for any use of UnsignedLong.longValue() that is inappropriate
eg SECONDS_PER_ETH1_BLOCK should probably NOT be an UnsignedLong

this is more complicated than it looks so has gone to the betting table.

This one is now a separate ticket: SECONDS_PER_ETH1_BLOCK should probably NOT be an UnsignedLong


New version: Alibaba.AlibabaCloudCLI version 3.0.151

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

Microsoft Reviewers: Open in CodeFlow
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Installation-Error


Allow only encoded paths to be mentioned in the path segments

Purpose

$subject

Fixes #19666
Approach

Allow only encoded paths to be mentioned in the path segments except for path params which is covered with {}

Samples

N/A

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #19864   +/-   ##
=======================================
  Coverage   15.21%   15.21%
=======================================
  Files          50       50
  Lines        1308     1308
  Branches      199      199
=======================================
  Hits          199      199
  Misses       1096     1096
  Partials       13       13

Continue to review full report at Codecov.

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


Redesign event waits/publishing paradigm

We would like to remove the complexity and confusion surrounding the "Recurring" checkbox for events, and have all events behave in the expected manner in which:

Activities registering for an event would be notified if event has already arrived
An event can be published multiple times
Activities can "re-register" for an event that has already arrived, to be notified the next time it arrives
Any number of activities can register to listen for the same event

Something important to note with this change is that when an activity that received/processed an event is put on hold (to launch a correction handler, exception handler, etc. for example), any events that the activity would have normally waited for, which happen to be received while the activity is on hold, will NOT be processed once the activity resumes from being on hold.  Instead, when the activity resumes back from on hold, it will re-register to listen for the events once again, but only events published from that moment forward will trigger, not events published while activity was on hold.


Add cloudwatch logging options for runCommand patch manager tasks

Based on AWS documentation [1] runCommand executions support directly logging to cloudwatch instead of an S3 bucket.
To that end we introduce 3 extra module variables, the first 2 (cloudwatch_log_output_enabled & cloudwatch_log_group_name) to configure whether cloudwatch logging is enabled and whether a custom cloudwatch log group name should be provided (if not a default is created, see [2]). The 3rd variable (s3_log_output_enabled) is introduced to optionally disable suppluing an S3 bucket for logging, since cloudwatch login should be enough for some use cases.
[1] https://docs.aws.amazon.com/systems-manager/latest/userguide/sysman-rc-setting-up-cwlogs.html
[2] https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/ssm_maintenance_window_task#cloudwatch_log_group_name

/terratest

/terratest

I was not able to update the readme as the make command failed for me on MacOS. If you would be so kind as to generate the README files correctly yourselves and update the PR I would appreciate it

/terratest

@gpapakyriakopoulos could you please run again the following commands locally?
  make init
  make readme

I believe this should work now 👍

@gpapakyriakopoulos could you please run again the following commands locally?
  make init
  make readme

I believe this should work now 👍

Seemed to worked correctly this time! I've added the README update in the PR, thanks!

/terratest

@gpapakyriakopoulos mind syncing your fork with the base branch?

Just rebased using the GitHub button @gberenice. Seems like you need to run terratest again + add your review again.

/terratest

A new version v1.0.1 was released. Thanks @gpapakyriakopoulos !


Support Stop() during the runtime of a SeedLang script

Example case
The sorting example of SeedLang runs SeedPython scripts such as bubble_sort.py. It usually costs a rather long time (say, a few minutes) to run the code and play animations for each operation.
There will be two buttons on the UI of the sorting example: "Run" and "Stop". The user uses the "Run" button to start tunning a program. When the program is running and the visualization animations are being played, the user shall be able to click the "Stop" button to cancel the demonstration. - For doing this, the SeedLang engine may need to provide a Stop() interface.
Possible solutions

SeedLang.Engine provides an interface Stop() for client code to call. Typically the client code will prefer calling this interface in the SingleStep event's On() callback.

Enable the SingleStep event to receive a return value from the client code, to indicate if the rest execution of the program should be continued or not.

Another example case
There are cases where the visualizer figures out that some semantic aspects of the current program cannot be visualized. For example, the sorting example app might limit the maximum length of the data array (under sorting) to 10, and limit the maximum number of each data item to 10. When the visualizer finds that the program defines a data array that exceeds the limit, the visualizer will report the issue, stops the program and wait for the user to update the code.
A Stop() interface is needed in this case too.


Clown can't fall when in crit or dead

Description
Title.
Reproduction
Be in crit or dead as a clown.
Screenshots

Additional context

i am pretty sure it's a dupe issue

Dupe of #26977

Dupe of #26977

Oh, someone needs to set labels to it...


Requirement of Repo2Docker Version Leads to JSONDecodeErrors

The current version of hubploy requires jupyter-repo2docker>=0.8.
With the most recent version, 0.10.0, there comes an issue when trying to build / push the Docker image for your hub. Pushing the image with streaming output eventually returns more than one json object and the repo2docker code errors with JSONDecodeError.
Ariel started an issue that got figured out here. So, in theory, using the master version of repo2docker avoids this error. However, the master version does not have a version number. It is listed as version 0+unknown. When installing the master version with
pip install --upgrade https://github.com/jupyter/repo2docker/archive/master.zip
I get the notice:
hubploy 0.1.0 has requirement jupyter-repo2docker>=0.8, but you'll have jupyter-repo2docker 0+unknown which is incompatible.
Attempting to run the hubploy build command complains similarly.
For now, I'm commenting out the jupyter-repo2docker requirement in setup.py of my hubploy image. I also have to go to /hubploy-template/lib/python3.7/site-package/hubploy-0.1.0-py3.7.egg-info and delete the repo2docker requirement out of requires.txt.

Fixed now!


create-cpcode not showing API response link

Using the create-cpcode parameter to create a CP code should return the response from the API (cpcodeLink followed with the endpoint to use to get details about the new CP code), however, the CLI does not return any output.
I can see the output using the --verbose parameter, so it seems a very easy fix to just print the API response body to standard output.
Here is the API documentation for your convenience: https://developer.akamai.com/api/core_features/property_manager/v1.html#postcpcodes
Here is the command I am using:
akamai property-manager create-cpcode --contractId $CONTRACT_ID --groupId $GROUP_ID --productId $PRODUCT_ID --cpcodeName test-cp1
You can run the same command using HTTPie to see the desired output:
http --auth-type=edgegrid --auth default: --body POST :/papi/v1/cpcodes contractId==$CONTRACT_ID groupId==$GROUP_ID productId=$PRODUCT_ID cpcodeName=test-cp2

i don't know that we need to see the full api response, but could the output just return cp code id that was created?

As a user of the property manager cli I only care about the cpcode id, and I really look forward to this being fixed.


Encoding .json files

Hi,
I'm using the useStaticFilesLoader to load my language files. In the de_DE.json are some special characters which are not displayed correctly.
I have set <meta charset="utf-8"> in the header and used Sublime to save the json file as UTF-8. Still the error occurs.
Whats wrong?
EDIT:
In the normal text its fine, just in the placeholder bit in input fields its shown incorrectly

Could you put up a short example with plunker?
But I think it's due to $sce / sanitation - please see http://angular-translate.github.io/docs/#/guide/19_security

Thx for the answer. The strange thing is, that the Umlaute work everywhere else perfectly fine. Its just the inputs placeholder don't seem to work:
<input type="text" name="position" id="position" placeholder="{{ 'SIGNUPFORM.INPUTS.POSITION.PLACEHOLDER' | translate }}" ng-model="position" />''
A "normal" paragraph for example works without a problem:

<translation translate="ANALYTICS.INFOSTATE.THIRDPARAGRAPH"></translation>

As @tspaeth already mentioned: Unless your issue is covered by $sce (see #1101), you should at least provide a demo.


Modal Animation bug in safari

https://media.giphy.com/media/1wq8UXgTzB8D6dL8K7/giphy.gif

Having the exact same issue on
macOS 10.13
Safari 11.0

So I have found the solution.
I limited the issue to the animation keyframes for the "comingIn" animation. Safari appears to have problems (or at least a weird behaviour) when rendering rotateX.
I then googled and came across this Stack post. So basically the solution is to add
-webkit-transform:translateZ(-1000px);
to .iziModal-overlay.
On (Mac) Safari and Chrome this solution seems to solve the issue (for Safari) and not cause any new ones for Chrome.


Fixing _TypeError when switching between

Using covariant in this way disallows us to switch between image providers (useful when for example you want to check if a firestore document has a field set with an image and if not display a placeholder). This is because DecorationImage and the like use an equals check on their image provider.
As shown in error below, which was thrown when switching between an asset image and the firebase image:
======== Exception caught by widgets library ======================================================= The following _TypeError was thrown building Builder(dirty, dependencies: [Directionality, MediaQuery, _LocalizationsScope-[GlobalKey#61773]]): type 'AssetImage' is not a subtype of type 'FirebaseImageProvider' of 'other'

The relevant error-causing widget was:
  Ink Ink:file:///anon/lib/components/widget.dart:100:28
When the exception was thrown, this was the stack:
#0      FirebaseImageProvider.== (package:firebase_cached_image/src/firebase_image_provider.dart)
#1      DecorationImage.== (package:flutter/src/painting/decoration_image.dart:196:24)
#2      BoxDecoration.== (package:flutter/src/painting/box_decoration.dart:329:24)
#3      InkDecoration.decoration= (package:flutter/src/material/ink_decoration.dart:355:15)
#4      _InkState._build (package:flutter/src/material/ink_decoration.dart:292:13)
#5      Builder.build (package:flutter/src/widgets/basic.dart:7448:48)
#6      StatelessElement.build (package:flutter/src/widgets/framework.dart:5038:49)
#7      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:4968:15)

It should be reproducible with something like this:
import 'package:firebase_cached_image/firebase_cached_image.dart';
import 'package:flutter/material.dart';
import 'package:provider/provider.dart';

class TestWidget extends StatefulWidget {
  const PlantCard({Key? key}) : super(key: key);

  @override
  State<PlantCard> createState() => _PlantCardState();
}

class _PlantCardState extends State<PlantCard> {
  late ImageProvider<Object> _image;

  @override
  void initState() {
    _setImage(const AssetImage('some-asset.jpeg')); // CHANGE ME

    super.initState();
  }

  void _setImage(ImageProvider<Object> imageProvider) {
    setState(() {
      _image = imageProvider;
    });
  }

  @override
  Widget build(BuildContext context) {
    print('${widget.plant.name}: ${widget.plant.imagePath}');
    return InkWell(
      onTap: () {
        _setImage(FirebaseImageProvider(
          FirebaseUrl.fromReference(firebaseRef), // CHANGE ME
        ));
      },
      child: Ink.image(
        image: _image,
        fit: BoxFit.cover,
      ),
    );
  }
}

Ohh!! I completely forgot about that covariant keyword in the == override. Some vscode extension generated it. Anyways Thank you for the pull request. Merging it...🔗


Access key

How to setup json key? directly paste json or file path?
and is it compatible with ce 2.2.x ?

@pppwu  Put the json value. no need put file path


[Failing Test]: Beam ML unit tests fail with Keras 3.3.3 and TF 2.16.1

What happened?

E               ValueError: Cannot convert '3' to a shape.

test-suites/tox/py39/build/srcs/sdks/python/target/.tox-py39-ml/py39-ml/lib/python3.9/site-packages/keras/src/backend/common/variables.py:530: ValueError

Issue Failure
Failure: Test is continually failing
Issue Priority
Priority: 2 (backlog / disabled test but we think the product is healthy)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

cc: @damccorm - we might want to prioritize https://github.com/apache/beam/issues/30908.

to repro:
gradlew :sdks:python:test-suites:tox:py39:testPy39ML -Pposargs=apache_beam/ml/


⚠ LPDM Handbook (hosted on Divio) has degraded performance

In 9f7468c, LPDM Handbook (hosted on Divio) (https://repohubdev.tk) experienced degraded performance:

HTTP code: 500
Response time: 9394 ms

Resolved: LPDM Handbook (hosted on Divio) performance has improved in 8b0fe82.


fixes captive portal bug where long ipfw rules fail to apply bug#4150

When captive portal creates ipfw rules, too many interface addresses can cause ipfw to return "rule too long"
See iussue #4150: https://redmine.pfsense.org/issues/4150
This is a problem when running a captive portal zone on > ~120 interfaces. This pull request breaks up the rules into chunks of 100 interface addresses.
$rulenum will overlap 65532 when ~20,000 interfaces are added to CP zone. This patch assumes you will not approach 20K interfaces on a CP zone, a fairly reasonable limitation.

Fixed in 2.2.5 and 2.3 with the use of tables.
Thanks!


Golang Debugger in VS Code

Debugging Go code using VS Code
https://github.com/Microsoft/vscode-go/wiki/Debugging-Go-code-using-VS-Code

setting completely


Please create VK icon

Hi,
we love your icons and regularly use them on our projects.
Please create VK icon for us.
Thanks a lot :)

@dimasaenko Do you have an SVG icon in mind?

This one is a good starting point:

Source: Icon finder

what about a vk logo without the square?

Will be too in the next release.
Sent from my HTC
----- Reply message -----
From: "fitodac"<EMAIL_ADDRESS>To: "thesabbir/simple-line-icons"<EMAIL_ADDRESS>Subject: [thesabbir/simple-line-icons] Please create VK icon (#56)
Date: Wed, Sep 7, 2016 09:48
what about a vk logo without the square?
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/thesabbir/simple-line-icons","title":"thesabbir/simple-line-icons","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/thesabbir/simple-line-icons"}},"updates":{"snippets":[{"icon":"PERSON","message":"@fitodac in #56: what about a vk logo without the square?"}],"action":{"name":"View Issue","url":"https://github.com/thesabbir/simple-line-icons/issues/56#issuecomment-245216174"}}}


style(chrome-extension): improve look and responsiveness

I noticed that the chrome extension could use some love.
Desktop View

Mobile View

Wow, this is amazing! Thanks a lot! 🥇


Uninstall-PSResource should take Version from ValueFromPipelineByPropertyName

Summary of the new feature / enhancement
As a user I want to be able to pipe Get-InstalledPSResource SomeModule to Uninstall-PSResource.
Currently if you try to pipe a single version to Unisntall-PSResource it will try to uninstall all of them:

Using Uninstall-PSResource -Name SomeModule -Version 2.0.0 works, but should be pipeable ideally.
Proposed technical implementation details (optional)
No response

Thanks @SeeminglyScience looks like this is a bug we thought we addressed in preview 11 but is still an issue

Resolved in https://github.com/PowerShell/PowerShellGet/pull/593


default config of terraform_docs fails

Hi,
terraform-docs has removed the --aggregate-type-defaults flag with the latest release.
https://github.com/terraform-docs/terraform-docs/blob/38316ec246f3eea4ed737f7d8bf54f2faabd6ff2/CHANGELOG.md#v080---2020-01-17
terraform_docs_without_aggregate_type_defaults hook still works. But terraform_hooks doesn't anymore because of this.

Thanks for opening this issue!
What do you think if have both terraform_docs_without_aggregate_type_defaults and terraform_docs acting identically (for backward compatibility reasons)?
/cc @robinbowes @rgevaert

I am fine with that.
Also maybe #57 is linked to this? In any case it can be closed if this one is closed I think. As I "tested" it.

v1.41.0 has been just released.

Awesome, thank you!


question regarding station names

ASDFDataSet.WaveformAccessor uses a station/network/channel naming convention, but not all data sets include this type of information.   For example, obspy.read leaves these attributes empty when reading from certain formats, such as SeismicUnix.  Any suggested workarounds?  Would you suggest manually setting the 'station' attribute for each trace with, say, the trace number index?
Any  suggestions would be greatly appreciatated.  Thanks, Ryan

Yes - I would manually set it.
Soonish (hopefully) the FDSN will expand the semantics of the network/station/location/channel identifers quite a bit to for example make it more suitable for active source and large-N deployments. At that points ASDF naturally just gains these. I actually don't see another option.
Note that ASDF  currently limits the maximum length of the identifiers to 2 (network), 5 (station), 2 (location), and 3 (channel). But pyasdf ignores these limits for practical reasons - they are only enforced in the validator and the specification.

Thanks very much!