QUESTION frontend React  & Evolution

Hello, I use your copybot which works very well thank you, for my part I transform the interface into React in the form of a dashboard, I wonder if you would be interested that once my work is finished to share with you in order to contribute to your project, the dashboard will be able to add new slaves with a mongoDB database, be able to add an allocation option in the slaves to dedicate an allocated sum in percentage, encryption of API secret keys, exchange choice, jwt connection to protect the table on board, recording each order of each slave, start shutting down the bot, and then see the stats of the bot

Hello. I glad to read that its tool was useful to you.
I would appreciate any help on the project. Your plan sounds good.
If you have any question or help with written code let me know.
Also do not be afraid to improve or radically change already written code.

Hello. I glad to read that its tool was useful to you.
I would appreciate any help on the project. Your plan sounds good.
If you have any question or help with written code let me know.
Also do not be afraid to improve or radically change already written code.

Hello. I glad to read that this tool is useful to you.
I would appreciate any help on the project. Your plan sounds good.
If you have any question or help with written code let me know.
Also do not be afraid to improve or radically change already written code.

Ok very well, I will let you know when the front end is finished for my part, I only touch the front-end, it is more my domain

Hello. I glad to read that this tool is useful to you.
I would appreciate any help on the project. Your plan sounds good.
If you have any question or help with written code let me know.
Also do not be afraid to improve or radically change already written code.

Ok very well, I will let you know when the front end is finished for my part, I only touch the front-end, it is more my domain


Rename Selector textFilter to text

This comment is just for notify @kirovboris and @arubtsov about this BC.
Do you use this in your tests?


make device memory tracker global

This is a fix to the device max memory watermark metric to track memory allocs globally rather than per task (but still take snapshots of global mem at the task level).
Relates to https://github.com/NVIDIA/spark-rapids/pull/11725/files

Please upmerge first to ensure license header check check your latest changes only. Thanks!

build


[Linux]Lack of 'Load More...' link at the bottom of blobs list though there is more than 100 blobs

Repro Steps:

Choose a blob container which contains 100 blobs at least.
Check whether 'Load More...' link displays at the bottom of blobs list or not.

Expect:
Display 'Load More' link and it works well.
Actual:
Lack of 'Load More...' link at the bottom of blobs list though there is more than 100 blobs.
More Info:
This issue doesn't reproduce on Windows/Mac.

I'm not seeing this issue.
Ubuntu Server 16.04 LTS

@StephenWeatherford
I feel embarrassed,,ԾㅂԾ,,.
Today, I tracked this issue, surprised found 'Load More' that displays and works well(●'◡'●).


Allow join to be called more than once on a thread.

We use this because iree_thread_delete on posix calls
pthread_join but we also have reasons to want to join it
manually at other times.
This also reverts the previous attempt to remove the joins from deferred_work_queue.
Signed-off-by: Andrew Woloszyn<EMAIL_ADDRESS>

Since we don't want to carry this any longer than we have to can you add // TODO(#issue) comments to the relevant code and make sure we delete it when fixing the deferred action queue?

(or ideally, move this into the deferred action queue instead?)


bugfix(k8s): do not log false-positive error when pause image is running

Follow up for #308 which successfully prevented a panic, but was still logging errors (about the pause image running) when it shouldn't have. It seems the test case added in #308 did not cover the pause image running, so this adds a couple of tests for that to ensure that that does NOT fail.
We have to use the canonical (parsed) pause image or it won't match at runtime:
https://github.com/go-vela/worker/blob/c25339835ea70c10e00d947ab3c08e2ab31b2135/runtime/kubernetes/container.go#L138
I also check for the simple non-canonical image name as well to simplify any unrelated tests that might incidentally use this.
Also add a helpful note/comment about ContainerState objects.

Oops. I missed that you approved. In any case, I dropped the comment, I can add it later if needed.


new meshstack setting

Pending https://github.com/meshcloud/tap-meshstack/pull/3 that adds this new setting

@pnadolny13 is this ready to merge?

@tayloramurphy nope not yet, the tap PR is still pending.


Analog modulation for NI RFSG driver

Add functionality to enable analog (amplitude / frequency / phase) modulation for NI-RFSG based devices. Includes example code in the example notebook. Tested amplitude modulation with a PXIe-5654.

Codecov Report

Merging #130 (60758d6) into master (2fd9c39) will decrease coverage by 0.81%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
- Coverage   19.91%   19.10%   -0.82%
==========================================
  Files         121      123       +2
  Lines       14708    14589     -119
==========================================
- Hits         2929     2787     -142
- Misses      11779    11802      +23

Impacted Files
Coverage Δ

...ontrib_drivers/drivers/NationalInstruments/RFSG.py
0.00% <0.00%> (ø)

...b_drivers/tests/QDevil/test_real_qdac2_ieee_std.py
33.33% <0.00%> (-9.53%)
:arrow_down:

...ontrib_drivers/tests/QDevil/real_qdac2_fixtures.py
30.90% <0.00%> (-3.58%)
:arrow_down:

...contrib_drivers/tests/QDevil/sim_qdac2_fixtures.py
78.78% <0.00%> (-1.22%)
:arrow_down:

...trib_drivers/drivers/ZurichInstruments/ZIHDAWG8.py
55.33% <0.00%> (-0.88%)
:arrow_down:

...odes_contrib_drivers/tests/test_Keysight_M3201A.py
15.22% <0.00%> (-0.76%)
:arrow_down:

...ivers/drivers/Keysight/SD_common/memory_manager.py
79.64% <0.00%> (-0.36%)
:arrow_down:

qcodes_contrib_drivers/__init__.py
100.00% <0.00%> (ø)

...es_contrib_drivers/drivers/CopperMountain/M5180.py
0.00% <0.00%> (ø)

...contrib_drivers/drivers/ERAInstruments/erasynth.py
0.00% <0.00%> (ø)

... and 16 more

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


Add "appimage-help" to display all possible arguments for a particular AppImage

Quite a few times I've had to go to the documentation on the web to check for the proper argument to use. I think it would be useful if we had an argument appimage-help that shows you all the possible commands you can use for that particular AppImage.
What do you think?

Sorry, I made a mistake in the repo in which I opened this. It should be in AppImageKit:
https://github.com/AppImage/AppImageKit/issues/524


[Bug Report] Not able to custom style el-select tag

Element UI version
2.13.2
OS/Browsers version
Mac OS Catalina / Latest Chrome Version 84.0.4147.105
Vue version
3.0.0-rc.5
Reproduction Link
https://codepen.io/darrylsteve/pen/WNwNLyB
Steps to reproduce
Just wanna know how to custom style an el-select tag
What is Expected?
Custom styling el-select and removing border
What is actually happening?
el-select does not respond to custom styling when trying to remove the borders

https://codepen.io/lemonapplemo/pen/JjXjzpP
Because you need to set the border of the input tag.

Thanks. That worked!


Update README.md

Fixing few typos in the README.md for improving the documentation of the project.

[ Y] Have you followed the guidelines in our Contributing document?
[ Y] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ Y] Have you added an explanation of what your changes do and why you'd like us to include them?
[ NA] Have you written new tests for your changes? Here's an example.
[ NA] Have you successfully run brew style with your changes locally?
[ NA] Have you successfully run brew typecheck with your changes locally?
[ NA] Have you successfully run brew tests with your changes locally?

These are correct in Commonwealth English. The proposed changes are American English. Our guidelines state that we prefer British/Commonwealth English.


Make font and text support optional.

Add some feature macros to disable font and text support at compile time.

Conditional compilation is not really available for configuring options with the new package distribution system.  The linker will remove unused functions though, and that is probably as far as I'm going to take this idea.


Manually Load Data and Steam User Saves

Hi Richard,
I am using your FileBasedPlayerPrefs for a Steam game of mine and backing up the saved data to the cloud. So far, it works excellent, but I've run into an issue when trying to backup the data by Steam User. I have basically modified the saved data location in your script to navigate to " Application.persistentDataPath + "/" SteamManager.gameObject.GetComponent().SteamID...etc. + "/" " and it works, however, the Steam Manager must initialize before I get the save file, since otherwise the "SteamID" will return 0. As a result, the FileBasedPlayerPrefs will write the file to the correct location, but not load from the correct location.
Do you know of any simple way to get around this? Is it possible to not load any FileBasedPlayerPrefs data until I decide to (i.e., after Steam intializes? Maybe via some code on the title screen instead of the splash screen?). The documentation explains how I can manually save the data, but not manually load it.
Thanks!

Hello
I just released V2 that includes a new Start method that loads the data.


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Missing NSPhotoLibraryUsageDescription permissions from image upload

Blocked on TestFlight atm, cc @Sherlouk

Why does Xcode never remind me to do this 🙄

Fixed in #672


How can i add Logo ?

I need to add my Logo as a Drawable in the searchview Edit Text like Google Playstore.
How can i acheive this ?

Hi @AruLNadhaN , this isn't supported out of the box. But, feel free to import the library into your project and add the drawable to the xml. Or, just do something like  ((TextView)mSearchView.findViewById(R.id.search_bar_text))                 .setCompoundDrawablesWithIntrinsicBounds( R.drawable.your_drawable, 0, 0, 0);
Regards,
Ari


Rotala - Implement source abstraction

Data sources are currently a bit of a grey area because the HTTP layer isn't particularly well-defined. One path here was to try and go for LOB immediately: this is probably necessary for use cases but is going to mean doing more stuff with exchanges (which is tedious).
I think it would make more sense to define the required functionality of source/inputs so that we have a foundation to come back to LOB later.
Key functions are:

At the start of a backtest, load data from file (it doesn't make sense to support network in any way)
The data on file has to be CSV
Data is immutable over the runtime of the server
Only way to change data is stop server and reload

This is less flexible than the current situation but we need a defined way of dealing with data, and this creates limitations to build on later.

An optimization here is that the Exchange currently requires a copy of the dataset, this is not needed because the datasets are immutable. All the exchange needs to store is the name of the dataset that it is using and it's position in this.
I am not sure why this was done but I think this is going to require changes across repo, as this mistake was copied multiple times.

This is mostly implemented already but should also add data sources. There are no real abstractions here so should be able to load from CSV.
Penelope/Uist is currently working on L1 data. So should also implement source/exchange for l2. May need to split off Orderbook abstraction from exchange too.

Implemented L2 Exchange
Tidied up some of the sharp edges around exchanges coming from transition to rotala
Clarified gap between source/input
Created binary for L2 source


Response size always seems to be 0B

I'd like to work on this, @helloanoop can you please assign this to me?


Introduce replacement method for avoiding the use of org.gradle.util.Clock

Context
The return type of the existing method cannot be easily changed as the build scan plugin depends on it.
Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation including proper use of @since and @Incubating annotations for all public APIs
[ ] Recognize contributor in release notes

Rather than replace the existing internal method with another one that will be used by the build scan plugin, we should consider exposing this in a public API instead.
We wouldn't need to expose all the methods on Timer, only provide a public way to get the ms elapsed since the build started.

OK, that makes sense. I am going to close the PR. Here's the follow up card for designing the public API: https://github.com/gradle/gradle/issues/1846


Update the Error Reporting documentation to note the Logging API is used by default.

The change also shows how to use the Error Reporting API directly and how to enable the Error Reporting API.
This adds the needed documentation required in issue #704
cc @steren

After thinking about this more I (updated) and moved the documentation into the code where the options are.  Most users probably will not need to use this option or need to change anything.  I think adding the docs where I did would just be more confusing.
PTAL

Thanks Mike!


Fix double-generation of scratch token (#6832)

Removed scratch token regeneration in NewTwoFactor, as it's generated anyway before calling this function.
This fixes bug (#6832) when scratch token was generated twice during 2FA setup and first-generated scratch token was not working (due to instantly being overwritten).

Codecov Report

Merging #6833 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #6833      +/-   ##
==========================================
+ Coverage   41.22%   41.22%   +<.01%
==========================================
  Files         421      421
  Lines       58124    58120       -4
==========================================
- Hits        23960    23959       -1
+ Misses      30992    30988       -4
- Partials     3172     3173       +1

Impacted Files
Coverage Δ

models/twofactor.go
5.31% <0%> (+0.21%)
:arrow_up:

modules/log/event.go
64.46% <0%> (-1.53%)
:arrow_down:

models/unit.go
67.56% <0%> (+5.4%)
:arrow_up:

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 ade88a8...eb848e9. Read the comment docs.

Please send backport to release/v1.8 branch


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

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


Violation line content

I also had to do this to be able to get the line content.

Thank you :rabbit:


Added slot entity resolution

I've added slot entity resolution classes to deserialize the json and have access to the fields.

it is possible somehow to use your version without waiting for it to be released?

sure, just checkout my branch and run "mvn clean deploy" to deploy it to your local repository.

I downloaded your branch, navigated to the root of the directory structure, and ran "mvn clean deploy". I got the following failure. Any suggestions?
[INFO] --- maven-install-plugin:2.4:install (default-install) @ alexa-skills-kit ---
[INFO] Installing /Users/jeromegarcia/alexa-skills-kit-java-masterSebastianBranch/target/alexa-skills-kit-1.5.1-SNAPSHOT.jar to /Users/jeromegarcia/.m2/repository/com/amazon/alexa/alexa-skills-kit/1.5.1-SNAPSHOT/alexa-skills-kit-1.5.1-SNAPSHOT.jar
[INFO] Installing /Users/jeromegarcia/alexa-skills-kit-java-masterSebastianBranch/pom.xml to /Users/jeromegarcia/.m2/repository/com/amazon/alexa/alexa-skills-kit/1.5.1-SNAPSHOT/alexa-skills-kit-1.5.1-SNAPSHOT.pom
[INFO]
[INFO] --- maven-deploy-plugin:2.7:deploy (default-deploy) @ alexa-skills-kit ---
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2.303 s
[INFO] Finished at: 2017-10-02T14:30:37-07:00
[INFO] Final Memory: 19M/331M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-deploy-plugin:2.7:deploy (default-deploy) on project alexa-skills-kit: Deployment failed: repository element was not specified in the POM inside distributionManagement element or in -DaltDeploymentRepository=id::layout::url parameter -> [Help 1]
[ERROR]

Hey @sebastiande & @Saandji . We've added entity resolution to 1.6.0 of alexa-skills-kit package.
If you modify your POM to target 1.6.0 then you can make use of these features.

Hey @jeromeernestgarcia Sorry i meant mvn clean install. But now you simply can use the official release. Thank you @brendanclement


Implement DataFrame.explode

This PR proposes DataFrame.explode.
>>> df = ks.DataFrame({'A': [[1, 2, 3], [], [3, 4]], 'B': 1})
>>> df
           A  B
0  [1, 2, 3]  1
1         []  1
2     [3, 4]  1

>>> df.explode('A')
     A  B
0  1.0  1
0  2.0  1
0  3.0  1
1  NaN  1
2  3.0  1
2  4.0  1

This method was not in our missing list, I found this at stackoverflow question.
I think we'd better to update the missing list to match with pandas.
I'll check and add if there is additional missing list.

Codecov Report

Merging #1507 into master will decrease coverage by 1.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1507      +/-   ##
==========================================
- Coverage   93.93%   92.66%   -1.27%
==========================================
  Files          36       36
  Lines        8445     8457      +12
==========================================
- Hits         7933     7837      -96
- Misses        512      620     +108

Impacted Files
Coverage Δ

databricks/koalas/frame.py
95.31% <100.00%> (-0.27%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
26.12% <0.00%> (-68.47%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/koalas/__init__.py
84.61% <0.00%> (-7.70%)
:arrow_down:

databricks/koalas/utils.py
94.25% <0.00%> (-2.40%)
:arrow_down:

databricks/conftest.py
94.33% <0.00%> (-1.89%)
:arrow_down:

databricks/koalas/generic.py
96.65% <0.00%> (-0.38%)
:arrow_down:

databricks/koalas/plot.py
94.04% <0.00%> (-0.24%)
:arrow_down:

databricks/koalas/indexes.py
96.88% <0.00%> (-0.19%)
:arrow_down:

... and 1 more

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 6fbe6d0...90b95d7. Read the comment docs.

Codecov Report

Merging #1507 into master will decrease coverage by 2.13%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1507      +/-   ##
==========================================
- Coverage   93.97%   91.84%   -2.14%
==========================================
  Files          36       36
  Lines        8488     8499      +11
==========================================
- Hits         7977     7806     -171
- Misses        511      693     +182

Impacted Files
Coverage Δ

databricks/koalas/frame.py
92.47% <100.00%> (-2.97%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.32% <0.00%> (-70.28%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/koalas/__init__.py
73.07% <0.00%> (-19.24%)
:arrow_down:

databricks/conftest.py
88.67% <0.00%> (-7.55%)
:arrow_down:

databricks/koalas/utils.py
94.03% <0.00%> (-2.30%)
:arrow_down:

databricks/koalas/namespace.py
84.49% <0.00%> (-1.61%)
:arrow_down:

databricks/koalas/plot.py
93.33% <0.00%> (-0.96%)
:arrow_down:

databricks/koalas/generic.py
96.30% <0.00%> (-0.37%)
:arrow_down:

Continue to review full report at Codecov.

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

Looks good otherwise.


Aborted test run is in progress on zafira

Please check the logic for displaying test run statuses. Sometimes, after run is abprted it's still in progress (spinner is rolling) on zafira

it should be fixed in 2.1 qps-pipeline logic where we moved abortZafiraRun on CI level


Version 1.14 installed on ubuntu-latest

It seems that after running this in my workflow (for linux, mac and window) that the version installed on ubuntu-latest (ubuntu-22.04) is a very old version 1.14, without a new option (--erase-functions) I would like to use.
https://github.com/andrewdavidmackenzie/pigg/actions/runs/9522864197/job/26253268146?pr=159
while on macos it's 2.1-1 and much newer versions are available on ubuntu (https://launchpad.net/ubuntu/+source/lcov)
Does that sound correct?
Any way to get a newer version or lcov on ubuntu-latest?

@andrewdavidmackenzie Noted.
I'll work on it this week.

I just checked. That is the latest version available on ubutu-latest/22.04, so it's really up to the Ubuntu/love maintainers to fix it.
You would have to build and install your own, newer, love package.


Secrets with Hashicorp Vault

Add a separate section for ops showing how secrets can be securely stored e.g. in Hashicorp Vault and provided as K8s secrets to a dev

Refactoring with #15

Fixed by https://github.com/arun-gupta/kubernetes-aws-workshop/commit/bfb86d03e65bbf0b6615a8b9c61bf137b76b9f69


Rename repo to devices-exporter

Prefer to rename this repo to devices-exporter, and add a directory/package for CPU.

We already have a "Devices" repo for GPU device plugin. I am not sure whether it is a bit confusing to use "device-exporter".
How about "resource-exportor". It is a general name. We can export more kinds of resource in the future.

resource-exportor is ok to me :) just make sure we do not bind a repo to specific resources.

Thanks for the suggestion. I changed name to "resource-exportor".


If in a frame or iframe issues

in iframe window obj,don't have any rtc api...............

Anybody alse knows this issue, help me please...

There is code in openwebrtc-ios-sdk that injects some simple JavaScript into the WebView which in turn makes an XHR request to a local http server that serves some more JavaScript that contains the WebRTC API implementation.
I would guess that an iframe has a separate JavaScript context and so this mechanism does not support iframes. I'm not sure how the WKWebView APIs work as to whether we could also support iframes.
@stefanalund @pererikb @Rugvip - any ideas?

Is there any news about this?


 tag is not working for "Content-Type: image/jpg"

There has been a report from Korean local community - http://cafe.naver.com/fusefactory/142 .
Problem: The following source returns "ImageSource error : 'HttpImageSource-failed-converson'" error on problems tab on Fuse Studio.
<App>
	<Panel>
		<Image Url="http://tong.visitkorea.or.kr/cms/resource/23/1989223_image2_1.jpg"/>
	</Panel>
</App>

Strangely, almost image URLs work well: e.g., [2] and even downloading the image and uploading it into another web server works well: e.g., [3] .
[1] http://tong.visitkorea.or.kr/cms/resource/23/1989223_image2_1.jpg
[2] http://www.dasprinzip.com/wp/wp-content/uploads/2014/12/fuse_screen04.jpg
[3] http://demo.riv-tech.co.kr:9898/1989223_image2_1.jpg
From my investigation, [1] returns HTTP response with "Content-Type: image/jpg" but [2] and [3] return HTTP response with "Content-Type: image/jpeg".
So I think the problem would be from such content type. In Safari, [2] and [3] URL displays images on Safari, but [1] downloads the image file. On the other hand, Chrome on Windows displays [1], [2], and [3] on the web browser. I cannot find further differences.
I am not too much sure whether the content type should be "image/jpeg" or not, but from my basic search, some web servers use "image/jpg".
If the problem is related with such content type,
Is it the problem from web servers, or would it be easy to also support "Content-Type: image/jpg" ?

The forum thread Fuse Image tag  Can not be done ImageSource error: 'HttpImageSource-failed-converson' posted by CHUL SEUNG LEE was linked to this issue.

@ianychoi: Thanks for digging. image/jpg is not an IANA-registered content-type, while image/jpeg is. So I think the server is mis-configured in this case.
That being said, we should probably be as fault resistant here as we can reasonably be. So perhaps we can fix this to tolerate this, even if the server is mis-configured.

This is a fabulous error report btw. :)

This was fixed.


Add LICENSE

Currently, there is no LICENSE which shows how you can use this repo.
Add a MIT License.

Can I work on this?


Import Tables and Large Datasets (XS Advanced)

Tutorial URL: https://developers.sap.com/tutorials/xsa-import-shine-data.html
Please specify the step you are referring to Step 3.
Hello,
I get the following error if I BUILD the DB:
--> to here, everything is fine!
Deploying "src/data/load.hdbtabledata$0.expand"...
Expanded from "src/data/load.hdbtabledata"
--> first error
Error: com.sap.hana.di.tabledata: The "include_filter" definitions in the table import files "src/data/load.hdbtabledata" and "src/data/loads/PurchaseDemo.hdbtabledata" use key values that are not disjunct; "src/data/load.hdbtabledata" has an empty filter set [8251511]       at "src/data/load.hdbtabledata" (54:22)
-->second error
Error: Worker 4 has encountered an error; all remaining jobs will be canceled [8214600]
Error: com.sap.hana.di.tabledata: The "include_filter" definitions in the table import files "src/data/load.hdbtabledata" and "src/data/loads/PurchaseDemo.hdbtabledata" use key values that are not disjunct; "src/data/load.hdbtabledata" has an empty filter set [8251511]       at "src/data/load.hdbtabledata" (14:22)
-->Build failed...
Inserted 2 records of the data source file "header.csv" into the target table "PurchaseOrder.Header"    Deploying "src/data/loads/PurchaseDemo.hdbtabledata$0.expand"... ok
Inserted 20 records of the data source file "item.csv" into the target table "PurchaseOrder.Item"    Deploying "src/data/loads/PurchaseDemo.hdbtabledata$1.expand"... ok
Error: Processing work list... failed [8212102]
Make failed (4 errors, 67 warnings): tried to deploy 61 files, undeploy 2 files, redeploy 2 dependent files  Error: Making... failed [8211605]
Error: Starting make in the container "DB_1" with 61 files to deploy, 2 files to undeploy... failed [8214168] Command failedDeployment to container DB_1 failed - error: HDI make failed [Deployment ID: none].
Error: HDI make failed(50s 658ms)npm ERR! code ELIFECYCLEnpm ERR! errno 1npm ERR! deploy@ start: node<EMAIL_ADDRESS>ERR! Exit status 1npm ERR! npm ERR! Failed at the deploy@ start script.npm ERR! This is probably not a problem with npm. There is likely additional logging output above.npm ERR! A complete log of this run can be found in:npm ERR!     /hana/shared/HXE/xs/controller_data/executionagent/executionroot/768bc893-0bc8-4969-a833-9a396ee5f1d1/app/META-INF/cache/node8.14/.npm/_logs/2019-05-28T11_55_12_177Z-debug.log
13:55:10 (Builder) Build of /HANA_APP/db failed.

Hi, What is the content of the files causing the error? What version of HXE is this?

Thank you for your feedback. We haven't heard from you in a while, so we are closing the issue.
If you still have questions, feel free to reopen the issue.

I just had the same issue as the one posted here.

Can you open a new ticket -- and provide the information on the files?

Done #3772 raised.


Shorten command line when computing ulib extraction dependencies

When calling fstar.exe --dep to extract ulib, listing all .fst and .fsti files makes the command line longer than the 8192-byte limit imposed by Windows, so building F* fails with an error message of the following form:
(Error 129) File experimental/FStar.N could not be found
gmake[5]: *** [Makefile.extract:38: .depend.extract] Error 1

indicating that the fstar.exe --dep command was actually truncated.
This PR solves this issue by listing all files in a response file, .depend.extract.rsp, and calling fstar.exe --dep with @.depend.extract.rsp, following the same pattern as #2442 (which was dealing with the same issue for ulib verification.)

Thanks Tahina!


Use Microsoft.PowerShell.MarkdownRender package from nuget.org

PR Summary
The code for Microsoft.PowerShell.MarkdownRender has been moved to a separate repository https://github.com/powershell/markdownrender. This PR removes the code from this repository and uses the package from nuget.org instead
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

I see the initial commit https://github.com/PowerShell/MarkdownRender/commit/345e2f35f2cc51b25a6dd86e12031c353c429f8a was on 2020-10-02. We should ensure changes made after this date are reflected in powershell/markdownrender.

@TravisEz13 - please re-review

@rjmholt This is ready for review now.

@xtqqczze - Filed issue in MarkdownRender repo to make sure the improvements mentioned above are taken there: https://github.com/PowerShell/MarkdownRender/issues/11

@rjmholt This is ready to merge.

@iSazonov We should avoid merging any more changes to Microsoft.PowerShell.MarkdownRender, like those in #14078.

@iSazonov We should avoid merging any more changes to Microsoft.PowerShell.MarkdownRender, like those in #14078.

These files will simply be deleted - this does not affect anything.

@PoshChan please remind me in 1 hour

@rjmholt, this is the reminder you requested 1 hour ago


1.8.1 mongoid adapter has a reference to moped. Is this right?

We have 3 projects that currently use the database cleaner mongoid adapter for cleaning our MongoDb 3.6 database when testing.
The release of version 1.8.1 kicked of Dependabot to raise a PR in each to upgrade Database Cleaner. However, every one of them broke

https://github.com/DEFRA/waste-carriers-front-office/pull/388
https://github.com/DEFRA/waste-carriers-back-office/pull/632
https://github.com/DEFRA/waste-carriers-engine/pull/654

The error we get is
An error occurred in a `before(:suite)` hook.
Failure/Error: DatabaseCleaner[:mongoid].strategy = :truncation

LoadError:
  cannot load such file -- database_cleaner/mongo/truncation_mixin
# ./spec/support/database_cleaner.rb:9:in `block (2 levels) in <top (required)>'

From our understanding of what 1.8 is doing to get ready for version 2.0, this is not unexpected. We understand 1.8 now needs us to explicitly reference the adapter gem required in our Gemfile. So we did that but still got an error.
LoadError:
  cannot load such file -- database_cleaner/moped/truncation_base
# ./spec/dummy/config/application.rb:12:in `<top (required)>'
# ./spec/dummy/config/environment.rb:2:in `require'
# ./spec/dummy/config/environment.rb:2:in `<top (required)>'
# ./spec/rails_helper.rb:6:in `require'
# ./spec/rails_helper.rb:6:in `<top (required)>'
# ./spec/controllers/waste_carriers_engine/registrations_spec.rb:3:in `require'
# ./spec/controllers/waste_carriers_engine/registrations_spec.rb:3:in `<top (required)>'

When we checked the database_cleaner-mongoid.gemspec it didn't seem to include any reference to database_cleaner-moped. But we did find a require call to it in DatabaseCleaner::Mongoid::Truncation
require 'database_cleaner/mongoid/base'
require 'database_cleaner/generic/truncation'
require 'database_cleaner/mongo/truncation_mixin'
require 'database_cleaner/mongo2/truncation_mixin'
require 'database_cleaner/moped/truncation_base' # <- Here
require 'mongoid/version'

module DatabaseCleaner
  module Mongoid
    class Truncation

We were able to get our tests running again by also including a reference to gem "database_cleaner-moped" in our Gemfile.
If you check the commits and their comments in https://github.com/DEFRA/waste-carriers-engine/pull/654 you should see what we attempted, and then what worked for us.
Based on what we found, we're just wondering whether the require 'database_cleaner/moped/truncation_base' is correct. If so, does the database_cleaner-moped gem need to be a dependency of database_cleaner-mongoid?
I hope that's clear. I may have added too much detail 😳.

It goes deeper! In addition to DC::Mongoid relying on those three separate implementations, DC::Ohm depends on DC::Redis, and DC::MongoMapper depends on DC::Mongo! Honestly, I'm embarassed that I didn't notice this when extracting the adapters. Bugfix releases forthcoming!

@Cruikshanks would you do me a favor and test your app with the #617 branch to verify that it has resolved this issue for you? To be clear, please note that at the moment, its just the base database_cleaner gem, not the database_cleaner-mongoid adapter, so your Gemfile would look something like:
gem "database_cleaner", git: "https://github.com/DatabaseCleaner/database_cleaner", branch: "inline_mongoid_dependencies"

If this resolves this issue for you, I'll release database_cleaner-mongoid v1.8.1 immediately, and likely follow that up with a v1.8.2 of database_cleaner once I resolve a few more other known issues.
Thank you for your help!

@botandrose it does work for me (1.8.1 doesn't work for me as well)

@vitalinfo Awesome! Thank you for testing it. I'll merge and release database_cleaner-mongoid v1.8.1 presently.

https://rubygems.org/gems/database_cleaner-mongoid/versions/1.8.1 :rocket:
However, note this issue won't be resolved in the base database_cleaner gem until we release v1.8.2, which will contain a few additional bugfixes.

Thank you, thank you, thank you!! 😁 🎉 🥇

And just to confirm, Dependabot closed those original PR's, but the new ones it created for 1.8.2 all passed CI and were merged successfully this AM without needing to add additional dependencies or make changes to our Gemfiles.
So many thanks again 👏

That's wonderful news! Thanks for letting us know. Hopefully all future releases will be that non-eventful 🤞


update package.json

updates package.json with dependencies (so npm published distrubution is accurate) for PTNFLY-1626
add .npmignore to allow dist files in npm distribution
add index.js to allow Browserify or CommonJS support. Noted here

@dgutride @dlabrecq @jeff-phillips-18 please take a look

Agreed, the strategy was to not require lodash. Hopefully nothing was added to make it a requirement.

i've updated index.js and added a simple example of using angular-patternfly with Browserify here

any more thoughts here ?

I'm OK with this then.

@jeff-phillips-18 @dtaylor113  I'm happy to make these updates for lodash in a future PR. Just didn't want to conflate it.

@jeff-phillips-18 @dtaylor113 ok to merge?

I'm ok with this as well.  The lodash requirement would only be for the npm only install, correct?  If so, then this won't break any existing customers ('cause angular-pf doesn't have an npm yet :-)


Troubles with uploading binary files via arc

Hello. I use latest versions of ex_aws and arc for uploading files to S3. I try to upload files from a binary, like this:
# photos is a list of binaries
photos = photos |> Enum.map(&(%{filename: "photo.png", binary: &1 |> Base.decode64!}))

for photo <- photos do
  changeset = Photo.Validator.changeset(%Photo{}, %{property_id: property.id, photo: photo})
  if changeset.valid?, do: changeset |> Repo.insert
end

When I use arc Locale storage it works as well, but when I try to upload to the S3 bucket I got an error:
[error] Task #PID<0.673.0> started from #PID<0.663.0> terminating
** (MatchError) no match of right hand side value: %{body: "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n<InitiateMultipartUploadResult xmlns=\"http:
//s3.amazonaws.com/doc/2006-03-01/\"><Bucket>ave88-test</Bucket><Key>uploads/properties/665/47e081b4-6af9-4b90-b2ca-dbe5652786af/large.png</Key><Upl
oadId>...--</UploadId></InitiateMultipartUploadResult>", headers:
 [{"x-amz-id-2", "..."}, {"x-amz-request-id", "..."}, {"Date", "Thu, 20 Oc
t 2016 09:35:06 GMT"}, {"Transfer-Encoding", "chunked"}, {"Server", "AmazonS3"}], status_code: 200}
    (ex_aws) lib/ex_aws/s3/upload.ex:39: ExAws.S3.Upload.initialize!/2
    (ex_aws) lib/ex_aws/s3/upload.ex:80: ExAws.Operation.ExAws.S3.Upload.perform/2
    (arc) lib/arc/storage/s3.ex:53: Arc.Storage.S3.do_put/3
    (elixir) lib/task/supervised.ex:94: Task.Supervised.do_apply/2
    (elixir) lib/task/supervised.ex:45: Task.Supervised.reply/5

    (stdlib) proc_lib.erl:247: :proc_lib.init_p_do_apply/3
Function: #Function<0.113666932/0 in Arc.Actions.Store.async_put_version/3>
    Args: []

Original photo was saved, but another versions (medium.png etc) aren't uploaded.
It is a ex_aws trouble or arc? (I created issue about this on arc repo too).
Thank you!

Please don't create an issue in both locations, it makes keeping track of the discussion very hard. Discussion should happen on Arc until we have a clear ExAws error.


Release v0.1.12

This is a quick release to immediately supersede v0.1.11, which was thwarted by faulty automation.

The flurry of releases today is an artifact of attempts to work around the 2023-03-16 npm incident which temporarily broke package publishing.


auto-format core markdown files (e.g. readme)

clean up formatting on existing markdown files to align with prettier
this creates a few diffs but it means that we have consistently formatted files for future changes to these root markdown files

The changes LGTM. Will the next step be to lint .md with prettier in pre-commit? https://github.com/wagtail/wagtail/blob/main/.pre-commit-config.yaml#L28

Not at the moment. My understanding is that we want to avoid too much friction for small markdown changes to be submitted via GitHub UI online.
However, I think at some point we will find it beneficial to adopt prettier for markdown also.
@thibaudcolas may have more input here.

Thanks @zerolab


Error loading the exercise in android studio

I am facing the error of plugins of android support when I open the file of exercise in android studio. So, can anyone help me out how to load the exercise in android studio?

Hello vyomshah06, Can you explain a little more about what type of error you are facing.

Go to File--> Settings--> Plugins --> Enable Android Support

Closing this issue as it is not related to the repo's code.


[WIP] Fix update-codegen.sh for update-cert-manager.sh update

As per title, we're getting unwanted updates on the update deps automation PRs

/retest-required

/cc @Cali0707 @creydr


Creating associate learning

This creates associate learning for link modules. So, if the admin creates a course and adds a face-to-face module. And then adds a link module to this course and sets it as associate learning. Then the user should not be able to start the link module unless the admin confirms the booking and the payment of the user.

Replaced by https://github.com/Civil-Service-Human-Resources/lpg-management/pull/224


Plugin compiled with Groovy's @CompileStatic breaks with Gradle 4.9

A bug in the Groovy compiler 2.4.10 and before ((GROOVY-8142)[https://issues.apache.org/jira/browse/GROOVY-8142]) can cause a plugin to refer to Gradle internal APIs even though its source only refers to public ones. Once Gradle changes or removes the unintentionally referred-to internal APIs, the published plugin stops working.
This is what happened at least in one case with the Gradle Packer plugin: it refers to the public Project.file() API, but ended up with the bytecode referring to the internal FileOperations.file() API. In Gradle 4.9 DefaultProject stopped implementing FileOperations, which was a change to only internal APIs, yet it caused the Packer plugin to break with the following error:
java.lang.IncompatibleClassChangeError: Class org.gradle.api.internal.project.DefaultProject_Decorated does not implement the requested interface org.gradle.api.internal.file.FileOperations

The problem to occur requires the following conditions to all be present:

the plugin must be compiled with Gradle 3.5 or earlier (Gradle 4.0 shipped a year ago with Groovy 2.4.11 that has the compiler bug fixed)
the plugin must be calling Project.file(), Project.files() etc. from inside a type that extends DefaultTask (there might be other cases, but the methods on FileOperations are the ones that we know about)
the task in question must be written in Groovy, and must be using @CompileStatic
GROOVY-8142 must trigger while building the plugin – this doesn't happen every time, so even if all the above conditions are present, you might still end up with the correct bytecode

One way to fix a broken plugin is to re-build it using Gradle 4.0 or later. If the plugin must be compiled using an earlier Gradle version for backwards compatibility reason, the solution could be to recompile with a newer Groovy version.
Adding the following snippet should achieve exactly this:
// Use a newer Groovy compiler to work around https://issues.apache.org/jira/browse/GROOVY-8142
// Only needed when running with Gradle versions earlier than 4.0
if (GradleVersion.current() >= GradleVersion.version("4.0")) {
    throw new RuntimeException("${GradleVersion.current()} already ships with a Groovy compiler that has GROOVY-8142 fixed")
} else {
    configurations {
        fixedGroovyCompiler
    }

    dependencies {
        fixedGroovyCompiler "org.codehaus.groovy:groovy-all:2.4.12"
    }

    tasks.withType(GroovyCompile) {
        groovyClasspath = configurations.fixedGroovyCompiler
    }
}

Submitted a merge-request for the Packer plugin to fix it: https://gitlab.com/ysb33rOrg/packer-gradle-plugin/merge_requests/3

Closing this issue since there is a workaround to fix it for existing plugins.

It seems this is still an issue. At least with Schalk's packer plugin.
https://gradle.com/s/y6jzkrnafxffw

Seems like the asciidoctor plugin also ran into this problem: https://github.com/asciidoctor/asciidoctor-gradle-plugin/commit/7e0c9ffb8902cd4a5210fcc007d33f1afe63c63c
See https://github.com/asciidoctor/asciidoctor-gradle-plugin/issues/253#issuecomment-410207568

OK, so it seems Groovy 2.4.11 has fixed the reproducibility issue with which type a certain method is referred to when there are multiple candidates... But it now always uses that of the private field. This code for example:
    /** The template file to use for the images.
     *
     * @return Resolved template file
     */
    @InputFile
    File getTemplate() {
        project.file(templateFile)
    }

https://gitlab.com/ysb33rOrg/packer-gradle-plugin/blob/master/src/main/groovy/org/ysb33r/gradle/packer/tasks/PackerBuilder.groovy#L154-161
Compiles into this:
  public java.io.File getTemplate();
    descriptor: ()Ljava/io/File;
    flags: ACC_PUBLIC
    Code:
      stack=2, locals=1, args_size=1
         0: aload_0
         1: checkcast     #2                  // class org/ysb33r/gradle/packer/tasks/PackerBuilder
         4: invokevirtual #42                 // Method getProject:()Lorg/gradle/api/Project;
         7: checkcast     #44                 // class org/gradle/api/internal/project/ProjectInternal
        10: aload_0
        11: getfield      #167                // Field templateFile:Ljava/lang/Object;
        14: invokeinterface #173,  2          // InterfaceMethod org/gradle/api/internal/file/FileOperations.file:(Ljava/lang/Object;)Ljava/io/File;
        19: areturn
        20: nop
        21: athrow
      LocalVariableTable:
        Start  Length  Slot  Name   Signature
            0      20     0  this   Lorg/ysb33r/gradle/packer/tasks/PackerBuilder;
      LineNumberTable:
        line 160: 0
      StackMapTable: number_of_entries = 1
        frame_type = 255 /* full_frame */
          offset_delta = 20
          locals = []
          stack = [ class java/lang/Throwable ]
    RuntimeVisibleAnnotations:
      0: #165()

Notice in particular:
        14: invokeinterface #173,  2          // InterfaceMethod org/gradle/api/internal/file/FileOperations.file:

The same behavior is observed with Groovy 2.4.12 and 2.4.15.

Filed https://issues.apache.org/jira/browse/GROOVY-8732 for Groovy. It appears to be a problem on every release branch (2.4, 2.5, 2.6 and 3.0) alike.
To provide a workaround for affected plugins, we'll revert to implementing FileOperations in DefaultProject again in 4.10. Users who bump into the problem using Gradle 4.9 should upgrade to Gradle 4.10 as soon as it's out, or downgrade to Gradle 4.8.1.
We can introduce a better workaround in Gradle 5.0 if we stop using the same name for private fields and public getters where the private field has a more specific type. One example is the AbstractTask.project field.

Thanks @lptr for looking into this again - it had me scratching my head.

Can we update TaskSubclassingBinaryCompatibilityCrossVersionSpec to cover this scenario?

@big-guy yes.

PR has been merged.

Looks like the original issue is going to be resolved in Groovy 2.5.2


Add check if includeXYZ method is available

Adds an additional check to fix the problem illustrated in #402 ...
This just checks, if respective method can be called (e.g., if it is available). If so, it is called..
Cheers

The only downside of this is it will break includes for anyone using __call.  I think this behavior is preferable to supporting magic methods though.

and what do you suggest?

I think solving the issue this way is fine.  Just noting that it will be a BC break for anyone relying on __call.

so then lets wait what the others say, i guess, right?!

I can't merge with a failing build, but @yuloh is correct, this is a BC break. I'm not sure that it's a large enough issue to warrant the break.
Has this caused you issue in the past?

Hey @greydnls ..
The Travis CI build is failing for HHVM with

HHVM is no longer supported on Ubuntu Precise. Please consider using Trusty with dist: trusty.

Calling "unavailable" includes is definately a problem for me / my application. I think, solving the issue right here in fractal is the better way of dealing with it.. As described in #403 , calling an unavailable include causes an Exception that is not handled in fractal..
Can i do something to get this merged? What do you suggest?
Cheers

any update on this? how can i help to get this merged?

Sorry about that and thanks for pointing it out (re: Trusty). I just got that fixed.
However, after thinking this over a bit more I think I'm against merging it as it is a BC break without a lot of benefit. I like the ability to use __call when necessary

why "without a lot of benefit"? Calling an ?include that does not exist results in an Exception that needs to be handled in another place. So why not directly handle it in this package?
Am i missing something, but you can still __call('include') right? or where would it break?
That means, a dev would need to __call the TransformerAbstract::callIncludeMethod() right? But that would still be possible?! The PR just checks, if the respective include-method is available...

sorry to bother you again @greydnls .. How shall we proceed with this issue and PR?
Shall I adapt the PR in order to comply with your requirements? Or is this closed forever?
I don't think that this PR messes with "old implementations" as it just adds a check if the method is callable.. Or am I missing something out?!
Cheers

My issue with it is that currently there is the ability to use __call to handle includes, this would completely remove that.
  public function __call($name, $arguments)
    {
        if (substr($name, 0, 7) !== "include") {
            return;
        }

        $relationshipName = lcfirst(substr($name, 7, strlen($name) - 1));

        if (!in_array($relationshipName, array_map([$this, 'normalizeIncludeName'], $this->availableIncludes))) {
            return $this->null();
        }

        //dynamic relationshipStuffHere
    }

Unless we can find a way to maintain that functionality this wouldn't be approved.

But the includeXYZ method must exist in the respective Transformer..
Let's go for an example..
You have a BookTransformer and an AuthorTransformer (assume, that the Models & Relationships are correctly set).
Basically, you would need to have a includeAuthor method in your BookTransformer. Assume, that this method is not available (e.g., it does not exist).
Trying to call GET api/books?include=author currently results in an Exception. Trying to __call('includeAuthor', ...) should also fail, because there is no such method..
😕

ok, now i get it, at least i think so..
__call(method, args) is used when the method is not defined..
But how can we fix this issue nontheless?

I need to think on it a little bit, I think. At this point the solution coming to mind is to allow some kind of configuration to be used in the manager that would include  strict check for this?
$manager->setStrictIncludes(true);

Hmmmmm, could be a good solution..
And regarding my PR, just add it to the if statement...
if($manager->getStrictInclude && is_callable(....)) { .... }

This could do the trick..

Yes, that would work. I use would a has/is-er, rather than a getter, though.
if ($manager->hasStrictIncludes() && is_callable()){ ... }

if you like, i can make a new PR during this week (with the hasStrictIncludes) or would like to wait and re-think it?


Failed because one or more lines were written to the STDERR stream

Note
Issues in this repo are for tracking bugs, feature requests and questions for the tasks in this repo
For a list:
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
If you have an issue or request for the Azure Pipelines service, use developer community instead:
https://developercommunity.visualstudio.com/spaces/21/index.html )
Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Azure CLI Inline script
list here (V# not needed):  v1
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
From about yesterday we are unable to deploy with azure cli script, for some reason we are getting "2020-02-20T21:47:46.4285528Z ##[error]Script failed with error: Error: The process 'D:\a_temp\azureclitaskscript1582235228379.bat' failed because one or more lines were written to the STDERR stream" even though there is no errors in the logs (even in debug).
Command this Inline is "az storage container create --account-name *** --name ***"
Task logs
[Enable debug logging and please provide the zip file containing all the logs for a speedy resolution]
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
[Insert error from the logs here for a quick overview]
2020-02-20T21:47:46.4285528Z ##[error]Script failed with error: Error: The process 'D:\a_temp\azureclitaskscript1582235228379.bat' failed because one or more lines were written to the STDERR stream

I'm also having this error.

@goriunov @goatandsheep Thanks for reporting the issue. To confirm, you are able to see the errors written to stderr in the logs correct? Could you share them as well?

@issacnitin from the logs which i have (debug enabled) only 3 lines contain error and nothing else:
2020-02-23T21:49:20.9367440Z ##[error]Script failed with error: Error: The process 'D:\a\_temp\azureclitaskscript1582494522128.bat' failed because one or more lines were written to the STDERR stream
2020-02-23T21:49:20.9375412Z ##[debug]Processed: ##vso[task.issue type=error;]Script failed with error: Error: The process 'D:\a\_temp\azureclitaskscript1582494522128.bat' failed because one or more lines were written to the STDERR stream
2020-02-23T21:49:20.9376402Z ##[debug]Processed: ##vso[task.complete result=Failed;]Script failed with error: Error: The process 'D:\a\_temp\azureclitaskscript1582494522128.bat' failed because one or more lines were written to the STDERR stream

@goriunov I'm able to reproduce the issue. We had a bug in the Azure CLI task, where if you have "Fail on Standard Error" checked, the script will always fail. The issue has been fixed and is in due rollout. For now, as a workaround, please uncheck the option to unblock your pipeline.
Related issue: https://github.com/microsoft/azure-pipelines-tasks/issues/12143

Closing the issue, as issue is already fixed and is in rollout.

I got this issue today. Have you rolled out the fix already?

This is happening to me as well

we too are getting this....

@goriunov I'm able to reproduce the issue. We had a bug in the Azure CLI task, where if you have "Fail on Standard Error" checked, the script will always fail. The issue has been fixed and is in due rollout. For now, as a workaround, please uncheck the option to unblock your pipeline.

Should we still be doing this work around in May?

This issue could still be happening if your script is writing to stderr.
In v2 of Azure CLI task we have made improvements to print the stderr logs. You can upgrade to v2 and see what logs are being written to stderr by your script.

Azure CLI task v2 causes the pipeline to fail although everything was successfully deployed. Checking the logs written to stderr by my script showed errors that were supposed to be informational logs from running the command az eventgrid event-subscription create. When will this be fixed?
Task
Task: AzureCLI@2
...
scriptType: bash
failOnStandardError: true

Command in script
      az eventgrid event-subscription create --name "es-$app_name" \         --source-resource-id $eventgrid_id \         --endpoint $app_id/functions/eventGrid_format --endpoint-type azurefunction
Logs written to stderr

2020-06-09T00:34:02.5424246Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5434374Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5436270Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5437644Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5438645Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5441016Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5442431Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5443524Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5445044Z ##[error]If the provided endpoint does not support subscription validation handshake, navigate to the validation URL that you receive in the subscription validation event, in order to complete the event subscription creation or update. For more details, please visit http://aka.ms/esvalidation
2020-06-09T00:34:02.5446488Z ##[error]The behavior of this command has been altered by the following extension: eventgrid
2020-06-09T00:34:02.5447512Z ##[error]Script has output to stderr. Failing as failOnStdErr is set to true.

@Sarah-Aly Please take it up with https://github.com/Azure/azure-cli
They'll be able to help you regarding why the informational text is written to stderr

I installed the Web Deploy  3.6 on agent deploy server, and my problem was solved.

Getting a similar issue right now as well. Within the pipeline, I am logging into my ACR to push a helm image, then I go on to upload an artifact. I checked my login credentials locally and within Pipeline Variables and they are working, as for the artifact, I am following the guideline that Microsoft provides. I had failOnStderr set to true, Disabled it and the pipeline got unblocked, and deployments went through smoothly.

We are also getting the same error.
What is the resolution? Does anybody know?
##[error]Bash wrote one or more lines to the standard error stream.
##[error]Traceback (most recent call last):
File "/o***t/az/lib/***ython3.11/site-***ackages/azure/cli/core/cloud.***y", line 544, in get_active_cloud_name
##[error]    return cli_ctx.config.get('cloud', 'name')
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/o***t/az/lib/***ython3.11/site-***ackages/knack/config.y", line 99, in get
raise last_ex  # ylint:disable=raising-bad-tye
^^^^^^^^^^^^^
File "/ot/az/lib/***ython3.11/site-ackages/knack/config.y", line 94, in get
return config.get(section, otion)
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ot/az/lib/***ython3.11/site-***ackages/knack/config.y", line 208, in get
return self.config_arser.get(section, otion)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/ot/az/lib/ython3.11/configarser.***y", line 797, in get
##[error]    d = self._unify_values(section, vars)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/o***t/az/lib/ython3.11/configarser.***y", line 1168, in _unify_values
##[error]    raise NoSectionError(section) from None


revert removal of support for dev/preview versioning

https://github.com/Azure/azure-sdk-for-js/pull/12606 caused failures in master because there are still a few packages using the old versioning scheme. This PR adds support back for dev and preview versions in the linter. The PR is a quick patch and I will follow up with a better handling for this soon.

Closing this in favor of https://github.com/Azure/azure-sdk-for-js/pull/12634.


Add YAML as a console output diagnostic notation option

an option named --human or similar would be used to visually check transcoding from sources by writing YAML to the console.
no_md_entries:
- md_entry_px:
    exponent: -9
    mantissa:<PHONE_NUMBER>000
  md_entry_size: 4
  md_entry_type: Offer
  md_price_level: 1
  md_update_action: New
  number_of_orders: 1
  rpt_seq: 2328011
  security_id: 123890
  tradeable_size: 0
no_order_id_entries:
- md_display_qty: 4
  md_order_priority:<PHONE_NUMBER>0
  order_id:<PHONE_NUMBER>543
  order_update_action: New
  reference_id: 1
transact_time:<PHONE_NUMBER>199755749

After some discussion with @mservidio, we will try using the absence of an output_type being specified to infer diagnostic output to the console and tthus ranscode ephemerally and human readably. Operator still has option to save console output via the host shell facilities,


The extension contains or enables malware/PUA.

这个插件已经发布了很多版本了也审核通过了,但是就是最近更新一个bug之后,
插件审核被拒绝 , 只是做了一些bug修复, 一直因为这个提示
The extension contains or enables malware/PUA.
被拒绝, 为什么, 然后联系不到审核人员, 在支持社区也发了消息 也没人回复 没人解决, 这个应该怎么去解决

Hi @zrl2088,
Can you please provide more details regarding your extension? like the Seller ID, extension name, etc.

I have the same confusion. This is my extension. Can you help me to see what malicious software there is?

Microsoft Store ID: 0RDCKH2RC045
Microsoft Store ID: 0RDCKGVNFPVR

@bhuvanapriyap Excuse me, please help me.

I have the same confusion. This is my extension. Can you help me to see what malicious software there is?

Microsoft Store ID: 0RDCKH2RC045
Microsoft Store ID: 0RDCKGVNFPVR

Hi @Xdy1579883916 ,
Thank you for writing to us.
We are looking into this and will get back to you as soon as possible.

Hi @Xdy1579883916 ,
Thank you for writing to us.
We are looking into this and will get back to you as soon as possible.

Thank you, I will keep an eye on this progress to help you troubleshoot and solve the problem.
Please feel free to contact me if you need any help. I can provide a compressed extension or related source code to assist in troubleshooting.


How to achieve corresponding 2D points for each 3D points?

I want to map each corresponding points in the point cloud to its 2d counterpart.

Done

@HardikJain02 Can you please share your code?


Fragile E2E test: 34-expenses.comment.test.js

Running:  34-expenses.comment.test.js

  1) Expense Comments loads more comments:
     CypressError: Timed out retrying: Not enough elements found. Found '23', expected '25'.

Sample CI run: https://github.com/opencollective/opencollective-frontend/pull/3241/checks?check_run_id=374461613

Probably related to https://github.com/opencollective/opencollective-frontend/pull/3246. I'll investigate tomorrow


changed Classes.js to classes.js

changed Classes.js to classes.js to follow other .js file naming convention

new PR to be pushed into master instead of develop.


⚠ Slack has degraded performance

In 99cb96f, Slack (https://status.slack.com/api/v2.0.0/current) experienced degraded performance:

HTTP code: 200
Response time: 183 ms

Resolved: Slack performance has improved in 7e4dc11.


Corrected round-robin config for component renames made by 1774288

Checklist

[ ] Type annotations added to new functions
[ ] Docs added to functions touched in main classes
[ ] Dry-run produced the expected results
[ ] The to-be-announced tag added if this is worth announcing

Based on the names on Bugzilla, the names in the round-robin config should look as follows:
{
    "Core::Graphics": "default",
    "Core::Graphics: Canvas2D": "default",
    "Core::Graphics: CanvasWebGL": "default",
    "Core::Graphics: Color Management": "default",
    "Core::Graphics: Image Blocking": "default",
    "Core::Graphics: ImageLib": "default"
    "Core::Graphics: Layers": "default",
    "Core::Graphics: Text": "default",
    "Core::Graphics: WebRender": "default",
}

I noticed that Core::Graphics: WebGPU is not in the list. On the other side, Core::Graphics: Layers is in the list even though it is marked as inactive on Bugzilla.

(it was likely wrong before the change on bugzilla)

Looks likely that it was.  I have to wonder if it was even working.

On the other side, Core::Graphics: Layers is in the list even though it is marked as inactive on Bugzilla.

Yes, Core::Graphics: Layers was disabled by 1774288.  I'll remove it from the list.


Gem fails to install on SmartOS

The gem fails to install on SmartOS due to been unable to find the library or the header files for libxml2.
#107 fixes this.

Commit merged - thanks!


Guidance on creating composite parameters for supporting chaining on group of conditions.

Question
How do we define custom composite search parameters, and can it solve the following use case?
Description
We are currently seeing an issue trying to find a way to query on groups of joined parameters in search. Normally this is handled by the _filter parameter in FHIR services, but per https://docs.microsoft.com/en-us/azure/healthcare-apis/fhir/overview-of-search#common-search-parameters this is not currently supported.
Expectation:
There is a way to group address-use=home and address-postalcode=66226, such that returned results have an address that is BOTH a home address and in postalcode 66226.
Reality:
The following query string {{baseUrl}}/ServiceRequest?_include=ServiceRequest:patient&subject:Patient.address-use=home&subject:Patient.address-postalcode=66208 results in all service requests that have an address that has EITHER address-use=home OR address-postalcode=66226.
While this is working as intended, we are looking for a way to achieve our expected behavior without _filter.
per https://hl7.org/FHIR/search.html#chaining - the other method for handling this is to define a composite search parameter - in this case one that involves narrowing on the address field for Patient.
Is there an example of how we would define a composite search parameter for address on Patient that can be used in search chaining?

This looks possible using a custom search parameter. Following the same format for composite such as below:
{
    "resourceType" : "SearchParameter",
    "id" : "Patient-address-use-postcode",
    "extension" : [{
      "url" : "http://hl7.org/fhir/StructureDefinition/structuredefinition-standards-status",
      "valueCode" : "trial-use"
    }],
    "url" : "http://hl7.org/fhir/SearchParameter/Patient-address-use-postcode",
    "version" : "4.0.1",
    "name" : "address-use-postcode",
    "status" : "draft",
    "experimental" : false,
    "date" : "2019-11-01T09:29:23+11:00",
    "publisher" : "Health Level Seven International (Orders and Observations)",
    "contact" : [{
      "telecom" : [{
        "system" : "url",
        "value" : "http://hl7.org/fhir"
      }]
    },
    {
      "telecom" : [{
        "system" : "url",
        "value" : "http://www.hl7.org/Special/committees/orders/index.cfm"
      }]
    }],
    "description" : "Code and quantity value parameter pair",
    "code" : "address-use-postcode",
    "base" : ["Patient"],
    "type" : "composite",
    "expression" : "Patient",
    "xpathUsage" : "normal",
    "multipleOr" : false,
    "component" : [{
      "definition" : "http://hl7.org/fhir/SearchParameter/individual-address-use",
      "expression" : "address.use"
    },
    {
      "definition" : "http://hl7.org/fhir/SearchParameter/individual-address-postalcode",
      "expression" : "address.postalCode"
    }]
}

The caveat at the moment is that I see SearchParameterOperations isn't resolving the composite components in Custom Search Parameters (so it will fail validation on create).
It worked when I tested with the following, I think we can get this change in to support this scenario.
var searchParameterWrapper = new SearchParameterWrapper(searchParam);
var searchParameterInfo = new SearchParameterInfo(searchParameterWrapper);

if (searchParameterInfo.Component?.Any() == true)
{
    foreach (SearchParameterComponentInfo c in searchParameterInfo.Component)
    {
        c.ResolvedSearchParameter = _searchParameterDefinitionManager.GetSearchParameter(c.DefinitionUrl.OriginalString);
    }
}

(bool Supported, bool IsPartiallySupported) supportedResult = _searchParameterSupportResolver.IsSearchParameterSupported(searchParameterInfo);

Search queries would look like:
GET https://{{hostname}}/Patient?address-use-postcode=home$74047

Awesome! Following question is about Azure Healthcare APIs - if this is the wrong place for those questions can direct this to Azure support.
This looks like the validation part of this would be blocking. Looks like versioned releases to the Azure Healthcare APIs service are rather infrequent (at least in visible versions in the portal). Would we need to move to an OSS deployment to accept this change? This, and a couple of other composite parameters are blockers for us.

You could test in OSS first. We package every 2 weeks, managed deployments usually follow 2-3 weeks after

Thank you so much! Will test in OSS when that merges!

AB#89147


'Taskbar Segments' option does not show tray icon, system icons or anything

yes i just saw the Taskbar Segment Feature and loved it so much but sadly i use my Tray far too often to use that feature without my tray 😢

Came here with the same issue. I hope that in a next update they'd add a feature for that

Just chiming in- I expected Segment to create two bar entities, or at least would allow me to configure them separately. 'Segment' should probably be changed to 'Hide tray area' or something, because that's all it seems to do on mine.


Fixed compile error build with VisualStudio2013 on windows.

REG_NONE defined in ml.hpp conflicts with defined in Winnt.h

Does this work for you?
#include <Windows header>
#undef REG_NONE // clear identifier for OpenCV usage
#include <OpenCV headers>

If it's my own project, of course I'll do that.
But, that conflicts error occurred in opencv when building itself.
It seems difficult to do that.

I agree with the solution. :+1: Logistic regression is quite recent addition, we can rename the enumeration constant.


Add e2e Workflow

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Hi @Revolyssup, may you help us take a look the CI issue. https://github.com/meshery/meshery-linkerd/runs/4647688938?check_suite_focus=true

Hi @Revolyssup, may you help us take a look the CI issue. https://github.com/meshery/meshery-linkerd/runs/4647688938?check_suite_focus=true

Yes I had seen this issue, service mesh pods are failing to start in workflow. I will debug in depth as I get time.

Hi @Revolyssup, may you help us take a look the CI issue. https://github.com/meshery/meshery-linkerd/runs/4647688938?check_suite_focus=true

Yes I had seen this issue, service mesh pods are failing to start in workflow. I will debug in depth as I get time.

Thanks.


Add support for Django 1.9

Ideally we'd be supporting the long term release and the current release of Django

Added with #420


APPCRASH after app exit if NewBigInteger().ToString() function used.

I'm submitting a bug report I'm submitting a feature request
Library Version:<IP_ADDRESS>

Please tell us about your environment:
Operating System: Windows 10 & 11

Current behavior:
Public Function HEX_To_BigNumber_2(ByRef decimalDataArr() As Byte) As String
Dim decArrPrefix00() As Byte, ls_NumVal As String
On Error GoTo error_trap
ls_NumVal = NewBigInteger(decimalDataArr).ToString()
HEX_To_BigNumber_2= ls_NumVal
Exit Function
error_trap:
HEX_To_BigNumber_2= "0"
End Function
During start program calls NewBigInteger(decimalDataArr).ToString() and everything seems fine.
But when the app is terminated some time later it crashes instead of closing gracefully. If the line:
ls_NumVal = NewBigInteger(decimalDataArr).ToString()
is replaced by:
ls_NumVal = "0"
app closes quietly (no error, no crash).

Windows Error Reporting file attached.
Report.wer.txt


Error:(313, 53) 'hrtime' is available starting with 7.3 PHP version

Guzzle version(s) affected: 7.4
PHP version:
PHP 7.2.34-28+ubuntu20.04.1+deb.sury.org+1 (cli) (built: Nov 19 2021 06:36:58) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
with Zend OPcache v7.2.34-28+ubuntu20.04.1+deb.sury.org+1, Copyright (c) 1999-2018, by Zend Technologies
with Xdebug v3.1.2, Copyright (c) 2002-2021, by Derick Rethans
cURL version:
cURL support => enabled
cURL Information => 7.68.0
Description
The hrtime function only works in php7.3. Wich version of guzzle can i use in this case ?

Guzzle works with PHP 7.2.5 or higher. We don't require the hrtime function to exist.


{gotosocial,castopod,swapspace,psi-plus,kmon}: remove misuzu from maintainers

Description of changes
Remove myself from maintainers for packages I no longer use.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3832


PHP Notice:  Use of undefined constant T_DOC_COMMENT_STRING - assumed 'T_DOC_COMMENT_STRING'

Use of undefined constant T_DOC_XXXXXXXXXX - assumed 'T_DOC_XXXXXX'
I am getting a lot of these errors when running the command.  They come from different files located under /vendor/pragmarx/laravelcs/Standards/Laravel/Sniffs/
I am unable to install this globally using composer, because I need to deploy it at build time on a continuous integration server.  So, I have added it to the composer.json file for the laravel project.
I call it like this
phpcs --standard=./vendor/pragmarx/laravelcs/Standards/Laravel /path/to/laravel
I am running
PHP 5.5.22-1+deb.sury.org~precise+1 (cli) (built: Feb 20 2015 11:25:27)
Thanks!

I have the same problem as you. Any news?

+1


[IMPROVEMENT] Show empty view on pinned messages screen

A view needs to be displayed when there are no pinned messages available, right @leonardoaramaki ? Is there anything else that needs to be done?

I'd rather have an image only or along with a text line. I'll see if I can have the image here.

@leonardoaramaki @aniketsingh03 maybe we can do something like this.

@divyanshub024 Looks nice! 👍

@rafaelks Thanks a lot! I start working on it. Can you please assign it to me?

@divyanshub024 Go ahead. I cannot assigned now because you didn't create the issue and you're not from the organisation, but go for it!

@rafaelks sure :+1:


rowDetail async

[x ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

**Current behavior**
toggleExpandRow does not work! If I load row detail asynchronously.

**Expected behavior**

**Reproduction of the problem**
toggleExpandRow(row) {
    this.myService.getSingle(row.id).subscribe(
      res => {
        if (res["result"]["code"] == 200) {
         this.tableExp.rowDetail.toggleExpandRow(res["result"]["data"]);
        }
      },
      error => {
      .......
      }
    );
}
* **Table version:**
 "@swimlane/ngx-datatable": "12.0.0",

* **Angular version:**
angular version : 6.0.0

* **Browser:** [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

* **Language:** [all | TypeScript X.X | ES6/7 | ES5]
Typescript 2.7

I am having the same problem, did you found solution to the problem.

I found a workaround by having additional component for row detail, that does the loading:
    <ngx-datatable-row-detail>
      <ng-template let-row="row" let-expanded="expanded" ngx-datatable-row-detail-template>
        <app-row-details [rowId]="row"></app-row-details>
      </ng-template>
    </ngx-datatable-row-detail>

And then inside the RowDetailsComponent do all the async fetching.

I am facing the same problem, Did someone found solution to this problem?


TipDialog and loading_view doesn't support null safety

I installed the latest version as explained  in the readme.
When i run my app, it shows
Error: Cannot run with sound null safety, because the following dependencies
don't support null safety:

package:tip_dialog
package:loading_view

any solution to this

please upgrade tipDialog version to 4.0.0 or run dart pub upgrade --null-safety


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/59750

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/60859

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/84165

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92679

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/93179

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/102951

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/106558

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/110145

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/111163

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/111357

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/112122

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/112794


Refactor listen_addresses to be set in the sample file, and make the sample file easier to overwrite

Closes #120
Fixes #105
Closes #54
(needs documentation updates too)

This also updates the entrypoint to allow for docker run postgres --flag --flag --flag.

This makes it possible for folks who want to embed a custom config in a new image (or to modify the existing config) to just replace /usr/share/postgresql/postgresql.conf.sample instead of thinking about which version of PostgreSQL they're using (or reference PG_MAJOR, which isn't even available at the time of -v when starting a container).

LGTM

@tianon
Now, can you give me an example on how to replace postgresql.conf and pg_hba.conf configuration files,
I do something like:
docker run \
  -v /somewhere/postgres/data:/var/lib/postgresql/data \
  -v /somewhere/postgres/postgresql.conf:/etc/postgres/postgresql.conf:ro \
  -v /somewhere/postgres/pg_hba.conf:/etc/postgres/pg_hba.conf:ro  postgres:9.4

But postgres ignores them.

@omidraha see https://github.com/docker-library/postgres/issues/134#issuecomment-196541551:

We have not yet added docs on overriding the config, :disappointed:
If you are replacing the .example with something like  -v /my/local/postgres.conf:/usr/share/postgresql/postgresql.conf.sample then you would need to make sure that your conf file has a listen_addresses='*'. (where as a previously undocumented feature, you could've mounted to /usr/share/postgresql/9.5/postgresql.conf.sample and the startup script would've rewritten your listen address line for you.)

Found this thread and the related thread that @bw-matthew started, but with much effort.
@tianon - I feel that this is a very common use case. Should we include this in the README or docs?

@omidraha How did you resolve it?


proposal: log: add flag to display ISO 8601 date and time format

go version go1.12.9 linux/amd64
package main

import "log"

func main() {
	log.Println("foobar")
}

Output:  2019/09/19 22:49:58 foobar
There is no option to specify the output format with the timezone included.

The LUTC flag will log the date/time in UTC. This should be enough to then convert to a local timezone. Does this satisfy your requirement?

I think it would be better to add a new flag to output the ISO 8601 format directly. Like in the Unix 'date' command:
$ date --iso-8601=seconds
2019-10-17T22:13:31+02:00

No need to convert between timezones, just print the time with timezone information.

Marking this as proposal so it goes through the appropriate channels.
/cc @robpike

We can add an infinite number of flags to package log to format the time.
If you need a specific formatting of the time, it seems like it makes more sense to call log.SetOutput with a custom io.Writer that inserts the time itself.

Something like this could suffice: https://play.golang.org/p/Ky4tu3FSHhV

Thanks @smasher164.
It seems like this is a likely decline - it's not common enough to be worth adding to the core API and it can be easily implemented outside.
Leaving open for a week for final comments.

No change in consensus, so declining.


Alexa ChannelController interface discovery

Description:
HA does not currently support the Alexa ChannelController capability as described here (https://developer.amazon.com/docs/device-apis/alexa-channelcontroller.html)
this is the first step for implementation of the feature.
Currently if you have a television and you say to alexa "Next channel" Alexa replies that the device does not support this function (local HA is not contacted). With this simple modify Alexa will contact HA asking to change channel and will reply that the device did not reply correctly.
This is my first pull in the project (to be honest the first pull request in open source projects at all), I choosed to make it simple (I choosed not to wait for the full implementation because I'd like to discuss with someone of the media player component in order to discuss my ideas... and I've no clue on how to find that someone)
For the complete implementation, my idea is to add the following 3 services to the media player component:

next_channel
previous_channel
channel_set
similar to the approach that has been taken to implement the volume management.  Feel free to contact me for any need and discussion.... I'm here to learn.

I will then implement those services in the samsungtv component.
Regards,
Paolo Tuninetto
PS: Speaking about tests... I'll need help in defining them and write them down.
Checklist:

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

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

[ ] 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.
[ ] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[ ] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Hello @tulindo,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

c472f394389b13fb29932a3eb64cfb6c5d28c9f1 - This commit has something that looks like an email address (tuni@tulindo.no-ip.biz). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

Sorry... the commit with wrong author information is the first of the two.
You said:
You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
I'm new to git... how can I do it?

Hello @tulindo,
When attempting to inspect the commits of your pull request for CLA signature status among all authors we encountered commit(s) which were not linked to a GitHub account, thus not allowing us to determine their status(es).
The commits that are missing a linked GitHub account are the following:

c472f394389b13fb29932a3eb64cfb6c5d28c9f1 - This commit has something that looks like an email address (tuni@tulindo.no-ip.biz). Maybe try linking that to GitHub?.

Unfortunately, we are unable to accept this pull request until this situation is corrected.
Here are your options:

If you had an email address set for the commit that simply wasn't linked to your GitHub account you can link that email now and it will retroactively apply to your commits. The simplest way to do this is to click the link to one of the above commits and look for a blue question mark in a blue circle in the top left. Hovering over that bubble will show you what email address you used. Clicking on that button will take you to your email address settings on GitHub. Just add the email address on that page and you're all set. GitHub has more information about this option in their help center.

If you didn't use an email address at all, it was an invalid email, or it's one you can't link to your GitHub, you will need to change the authorship information of the commit and your global Git settings so this doesn't happen again going forward. GitHub provides some great instructions on how to change your authorship information in their help center.

If you only made a single commit you should be able to rungit commit --amend --author="Author Name<EMAIL_ADDRESS>
(substituting Author Name and<EMAIL_ADDRESS>for your actual information) to set the authorship information.
If you made more than one commit and the commit with the missing authorship information is not the most recent one you have two options:

You can re-create all commits missing authorship information. This is going to be the easiest solution for developers that aren't extremely confident in their Git and command line skills.
You can use this script that GitHub provides to rewrite history. Please note: this should be used only if you are very confident in your abilities and understand its impacts.

Whichever method you choose, I will come by to re-check the pull request once you push the fixes to this branch.

We apologize for this inconvenience, especially since it usually bites new contributors to Home Assistant. We hope you understand the need for us to protect ourselves and the great community we all have built legally. The best thing to come out of this is that you only need to fix this once and it benefits the entire Home Assistant and GitHub community.
Thanks, I look forward to checking this PR again soon! :heart:

The PR branch contains unrelated commits. Probably easiest to start over in a fresh branch and a new PR.

Thanks Martin... I don't know what happened (new to git).
I'm planning to start from scratch (a new fork) so I'll have a single commit (and everything will be clean).
What about this PR? Do I have to close it?


net/http: serveContent() via FS() fails when the fs.File does not implement io.Seeker and content-type cannot be determined from filename

What version of Go are you using (go version)?

$ go version
go version go1.16 linux/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
...
GOARCH="amd64"
GOOS="linux"
...

What did you do?
Using http.FS with an fs.FS that does not return io.Seeker files will fail to serve if the content-type cannot be guessed from the filename.
Here is a testcase bug_test.go:
package main

import (
	"io"
	"io/fs"
	"net/http"
	"net/http/httptest"
	"strings"
	"testing"
	"time"
)

const FilenameWithoutExtension = "my-filename-without-extension"

type myFs struct{ fs fs.FS }

type myFile struct{ r *strings.Reader }

func (my *myFs) Open(name string) (fs.File, error) {
	if name != FilenameWithoutExtension {
		return nil, &fs.PathError{"Open", name, fs.ErrNotExist}
	}

	var content = `Go!`

	return &myFile{strings.NewReader(content)}, nil
}

func (my *myFile) Close() error               { return nil }
func (my *myFile) Read(p []byte) (int, error) { return my.r.Read(p) }
func (my *myFile) Stat() (fs.FileInfo, error) { return &fileInfo{my.r}, nil }

type fileInfo struct{ r *strings.Reader }

func (fi *fileInfo) Name() string       { return FilenameWithoutExtension }
func (fi *fileInfo) Size() int64        { return int64(fi.r.Len()) }
func (fi *fileInfo) Mode() fs.FileMode  { return 0444 }
func (fi *fileInfo) ModTime() time.Time { var t time.Time; return t }
func (fi *fileInfo) IsDir() bool        { return false }
func (fi *fileInfo) Sys() interface{}   { return nil }

func TestDefaultContentTypeForUnseekableContent(t *testing.T) {
	var (
		fs     = &myFs{}
		httpfs = http.FS(fs)
		fsrv   = http.FileServer(httpfs)
		req, _ = http.NewRequest("GET", FilenameWithoutExtension, nil)
		rr     = httptest.NewRecorder()
	)

	f, _ := fs.Open(FilenameWithoutExtension)
	if _, ok := f.(io.Seeker); ok {
		t.Fatalf("%T must not implement `io.Seeker`", f)
	}

	fsrv.ServeHTTP(rr, req)

	t.Logf("rr %+v", rr)

	if rr.Body.String() == "seeker can't seek\n" {
		t.Logf("An `fs.File` not implementing `io.Seeker` fails when Content-Type cannot be guessed from filename")
	}

	// PDM: Suggested fix
	if rr.HeaderMap.Get("Content-Type") != "application/octet-stream" {
		t.Fatalf("BUG: Should return `application/octet-stream` when file is not seekable")
	}
}

(play.golang.org is currently running Go 1.14, but this testcase relies on io/fs from 1.16)
What did you expect to see?
When the content-type cannot be guessed from the filename, and when the content itself is not seekable, I would suggest that returning application/octet-stream would be a reasonable default.
What did you see instead?
HTTP response code 500 with body seeker can't seek (coming from https://github.com/golang/go/blob/master/src/net/http/fs.go#L243 ).
Suggested fix
The following patch to src/net/http/fs.go works around the case where *http.ioFile is being used to wrap an fs.File (via http.FS) and add a Seek method, even when a given fs.File doesn't implement it (see https://github.com/golang/go/blob/master/src/net/http/fs.go#L774-L780):
diff --git a/src/net/http/fs.go b/src/net/http/fs.go
index a28ae85..e6df3ce 100644
--- a/src/net/http/fs.go
+++ b/src/net/http/fs.go
@@ -229,10 +229,18 @@ func serveContent(w ResponseWriter, r *Request, name string, modtime time.Time,

        // If Content-Type isn't set, use the file's extension to find it, but
        // if the Content-Type is unset explicitly, do not sniff the type.
+       // Finally, if the underlying content doesn't support seeking, use `application/octet-stream`.
        ctypes, haveType := w.Header()["Content-Type"]
        var ctype string
        if !haveType {
                ctype = mime.TypeByExtension(filepath.Ext(name))
+
+               if ctype == "" {
+                       if f, ok := content.(*ioFile); ok {
+                               if _, ok := f.file.(io.Seeker); !ok {
+                                       ctype = "application/octet-stream"
+                               }
+                       }
+               }
+
                if ctype == "" {
                        // read a chunk to decide between utf-8 text and binary
                        var buf [sniffLen]byte

Thank you for filing this issue @pdmccormick and welcome to the Go project! Thank you @seankhliao for the triage!
@pdmccormick the contract for http.ServeContent firmly requests that an io.Seeker be passed in, as has been for the past 9 years

Breaking this contract might be forward compatible, but it'll introduce unknown behavior and it renders http.ServeContent useful defeating its make up. The seek method is also later used to determine the size. For common usage, we recommend using http.Dir(DIRECTORY_ROOT") https://golang.org/pkg/net/http/#Dir
In what cases in the wild would it be possible to have a non-artificial fs.FS that doesn't implement io.Seeker?

Thanks @odeke-em for the review and feedback. I suppose my problem is not so much with ServeContent itself but with the fact that serveFile and ServeContent both use serveContent, and that adapting a fs.FS to a http.FileSystem quietly imposes the extra requirement that the returned fs.Files must implement io.Seeker iff the content type cannot be guessed by the filename extension.
I came across this issue when creating a gzip decompressing filesystem wrapper: when asked to open path/to/filename, first check if path/to/filename.gz exists in the underlying filesytem, and a return a compress/gzip reader for it. My first attempt at this didn't implement any kind of seeking, and my filename happen to not have an extension. This has come in handy since embed currently embeds file data as-is.

Edited title to clarify that it is serveContent (not ServeContent) invoked via (*fileHandler).ServeHTTP to serveFile to serveContent.

I think I've run into this exact issue by trying to open https://golang.org/src/mime/type_windows.go.