Update watchOS deployment target to 3.0

New Pull Request Checklist

[x] I have read and understood the CONTRIBUTING guide

[x] I have read the Documentation

[x] I have searched for a similar pull request in the project and found none

[x] I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

[x] I have added the required tests to prove the fix/feature I am adding

[x] I have updated the documentation (if necessary)

[x] I have run the tests and they pass

[x] I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: #954
Pull Request Description
I updated watchOS deployment target to 3.0 and added DDOSLoggerTests that is almost the same as( a little different from) DDBasingLoggingTests.
And I also added a few tests to check fileName property in DDLogMessge.

You have copy-pasted tests... For what reason?
It is a bad style.

It is just a test for check logging(DDLogMesage) by DDOSLogger.
A better test or style is welcome:)

I think we should add (legacy) tests for DDTTYLogger and DDASLLogger and instead change the logger used in DDBasicLoggingTests to DDOSLogger.

@ffried,
DDBasicLoggingTests could cover whole set of default loggers (TTY, ASL, OS).

          2 Warnings

:warning:
Any changes to library code should be reflected in the CHANGELOG. Please consider adding a note there about your change.

:warning:
The CocoaPods podspec was modified but the Carthage project wasn’t. Did you forget to update the xcodeproj?

  Generated by :no_entry_sign: Danger

Codecov Report

Merging #981 into master will increase coverage by 2.4%.
The diff coverage is 98.61%.

@@            Coverage Diff            @@
##           master     #981     +/-   ##
=========================================
+ Coverage   25.26%   27.67%   +2.4%
=========================================
  Files          21       22      +1
  Lines        3451     3523     +72
=========================================
+ Hits          872      975    +103
+ Misses       2579     2548     -31

Impacted Files
Coverage Δ

Classes/DDASLLogger.m
0% <ø> (ø)
:arrow_up:

Classes/DDASLLogCapture.m
0% <ø> (ø)
:arrow_up:

Tests/Tests/DDBasicLoggingTests.m
98.59% <100%> (+0.02%)
:arrow_up:

Tests/Tests/DDOSLoggerTests.m
98.59% <98.59%> (ø)

Classes/DDOSLogger.m
91.42% <0%> (+91.42%)
: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 8280cbe...f191894. Read the comment docs.

Yes, but I think DDBasicLoggingTests almost covers DDTTYLogger, DDASLogger, and DDOSLogger.
Then this time I added test for using DDOSLogger.

DDBasicLoggingTests could cover whole set of default loggers (TTY, ASL, OS).

Yes. I think so too.

So if we add tests for DDTTYLogger and DDASLogger, I think it is enough to add diffs for each Logger
(for example, by creating another PR).

I agree with @ffried : forDDBasicLoggingTests just replace DDTTYLogger with DDOSLogger. Since we are deprecating DDASLLogger, I don't really care about testing it.
Should we also reprecate DDTTYLogger?

@bpoplauschi
In which version will you obsolete all loggers except DDOSLogger?

I agree with @ffried : forDDBasicLoggingTests just replace DDTTYLogger with DDOSLogger. Since we are deprecating DDASLLogger, I don't really care about testing it.
Should we also reprecate DDTTYLogger?

Now I'm a little confused...😅 Do you say it is enough to replace DDTYLogger with DDOSLogger?

@lolgear :) man, I was asking a question, what do you all think about marking as deprecated DDASLLogger and DDTTYLogger.
In my opinion, we can mark them for the next version (3.5.0) as this is not breaking backwards compatibility.
@sushichop yes, I mean for the DDBasicLoggingTests we can just replace with DDOSLogger. The point of the DDBasicLoggingTests is to test the overall mechanism from log macro (i.e. DDLogDebug) to any logger, not necessarily one of them.
If we want to add specific logger tests, we can do so, but I would not spend time in adding tests for classes we want to deprecate.

I updated commit.
Sorry guys, it's time over for me today(I'll take a bath and go to bed). Good night!
Anyway I'm glad to discuss various things recently:)
Thanks in advance.

Hi guys! Finally I updated all commits. I summarize my opinion at the moment.
See each commit for details:)

I left DDBasicLoggingTests.m using DDTTYLogger. Because I think it is necessary to test all OS version(e.g. iOS 8.0 to 12.0) and overall mechanism from log macro.
I added DDOSLoggingTests.m using DDOSLogger and DDASLogger.
(e.g. DDOSLogger for iOS 10.0+ and DDASLogger for iOS 10.0-)
I added iOS 9.3 simulator for testing DDASLogger using travis-ci.

If you think tests for DDASLogger is unnecessary, I'll gladly remove it.
That's all. Thanks!


AKSamplePlayer looping behaviour after update to 4.0.3

Hi Aurelius, I am experiencing change in how AKSamplePlayer handles looping after updating to xCode 9.1 and AudioKit 4.3.

AKSamplePlayer seems to "forget / reset" the loop start and end points after first loop. If I extract the issue to audioKit playgrounds:

import AudioKitPlaygrounds
import AudioKit

let file = try AKAudioFile(readFileName: "alphabet.mp3")
let samplePlayer = AKSamplePlayer(file: file)
AudioKit.output = samplePlayer
AudioKit.start()

samplePlayer.loopEnabled = true
samplePlayer.play(from: Sample(44100), length: Sample(44100))

import PlaygroundSupport
PlaygroundPage.current.needsIndefiniteExecution = true

Output of this is "b" on the first loop and whole alphabet on consecutive ones.

If loop end point is set on a separate line for the first loop it seems that the calculation is done twice.

import AudioKitPlaygrounds
import AudioKit

let file = try AKAudioFile(readFileName: "alphabet.mp3")
let samplePlayer = AKSamplePlayer(file: file)
AudioKit.output = samplePlayer
AudioKit.start()

samplePlayer.loopEnabled = true
samplePlayer.loopEndPoint = Sample(Int(13 * 44100))
samplePlayer.play()

import PlaygroundSupport
PlaygroundPage.current.needsIndefiniteExecution = true

This playground plays first 6.5 letters on a first loop and first 13 letters on consecutive ones.

Will hopefully have a chance to look at this on Wednesday.

Thanks for all the efforts! The current workaround (kind of) that I am using is to initialise AKSamplePlayer with a "dummy sample" which is just 5 minutes of silence and then use the second approach. This makes the first loop really short and almost inaudible.

In the develop branch, we changed the way AKSamplePlayer works, so this should be fixed now. Would appreciate any feedback.
Thanks


Include CPU arch in host info

To distinguish x64 and ARM machines.
Change in gopsutil: https://github.com/DataDog/gopsutil/pull/26

The change to DataDog/gopsutil has no effect here, since we use shirou/gopsutil to get host metadata (see: https://github.com/DataDog/datadog-agent/blob/9d50ebe02207dbccaa92e43211cb10ad22d01611/pkg/metadata/host/host_notwin.go#L15-L16 for instance).
Example on a datadog-agent status from your branch's deb package:
  Host Info
  =========
    bootTime: 2019-08-05 17:17:55.000000 UTC
    kernelVersion: 4.18.0-1024-azure
    os: linux
    platform: ubuntu
    platformFamily: debian
    platformVersion: 18.04
    procs: 198
    uptime: 2758h34m21s
    virtualizationRole: host
    virtualizationSystem: kvm

It's in the host metadata payload.


feat: chart: add labels to QuarksStatefulSets

Description
Add the standard labels to the generated QuarksStatefulSets, so that we can see the version when examining a live deployment easier.  The selector are unaffected, as we want to be able to use the previous version for a bit during an upgrade.
Motivation and Context
When examining an upgrade, it is difficult to figure out what version of KubeCF corresponds to each pod.
How Has This Been Tested?
Deployed during an upgrade (HA). I have confirmed that:

The StatefulSets are updated, not recreated (their creation timestamp is from before the upgrade took place).
The existing pods are rolling as expected — api-1 is upgraded and ready before api-0 is upgraded.

(Actually, the upgrade never completes because api-1 is not counted for bootstrap, so DB migration never takes place, and therefore api-1 can never get ready.  But that's an existing issue outside this PR.)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code has security implications.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

@mook-as Can this be an annotation rather than a label?

@f0rmiga Sure, but the actual labels are all pretty standard.  It's not really a problem to move them, but it seems odd to put them under annotations unlike every other resource.

@f0rmiga Sure, but the actual labels are all pretty standard. It's not really a problem to move them, but it seems odd to put them under annotations unlike every other resource.

I see. Indeed we have the rest as labels.


[BUG] target_temperature has unit None which is unsupported for device_class temperature

Describe the bug
Getting this in my log for all temperature entities, the unit of measurement seems to be missing ?
Since install of 2022.7.x
EDIT: I added the UOM in customize and all is working fine again now, no log warnings
Logger: homeassistant.components.sensor.recorder
Source: components/sensor/recorder.py:259
Integration: Sensor ([documentation](https://www.home-assistant.io/integrations/sensor), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+sensor%22))
First occurred: 10:20:10 am (30 occurrences)
Last logged: 10:20:10 am

sensor.living_room1_target_temperature has unit None which is unsupported for device_class temperature
sensor.living_room2_target_temperature has unit None which is unsupported for device_class temperature
sensor.toilet_target_temperature has unit None which is unsupported for device_class temperature
sensor.upstairs_target_temperature has unit None which is unsupported for device_class temperature
sensor.utility_room_target_temperature has unit None which is unsupported for device_class temperature

Expected behaviour
A unit of measurement in the entities

Yes, 2022.7.3 fixes it for me, many thanks


Remove dependency on favicons-webpack-plugin

The favicons-webpack-plugin plugin causes some difficulties behind corporate proxies due to the dependency on phantomjs.
For now we can just remove this plugin and use the standard favicon support in webpack.

We have been surviving pretty well with this dep and its baggage - closing, can re-open if it becomes a bigger problem in another environment.


Masthead: James and Edward

Resolves #3
Make sure your branch name conforms to: <feature|staging|bugfix|...>/<username>/<3-4 word description separated by dashes>. Otherwise, please rename your branch and create a new PR.
Changes

Created the layout for the masthead including these features:

Adaptability to screen resizing
Background image slideshow
Arrow Animation
General layout of buttons and text

Testing

Ran the development server and observed the resulting webpage in many different screen resolutions.

Confirmation of Change
https://user-images.githubusercontent.com/73041370/218377297-b7e2c694-06a1-4e9b-9043-a3b2fb565709.mov

Great work! I just have a few comments on the masthead's appearance:

I'm not sure if it's just my computer, but it seems like the page overflows horizontally and makes a scrollbar appear on certain screen sizes (this is 1265 by 745 px)

The icons on the right, logo in the center, and text on the left both look a little small on mobile and tablet screens:


Command "wt -w 0 new-tab" open two tabs on new window

Windows Terminal version
1.19.2682.0
Windows build number
10.0.19045.2846
Other Software
No response
Steps to reproduce
I want to have a .cmd script which opens a new Terminal Preview windows containing N tabs (each tabs have same profile but different directory).
Example of my script :
wt -w 0 new-tab -p git-bash -d C:\path0 ;
wt -w 0 new-tab -p git-bash -d C:\path1 ;
wt -w 0 new-tab -p git-bash -d C:\path2 ;
wt -w 0 new-tab -p git-bash -d C:\path3 ;
wt -w 0 new-tab -p git-bash -d C:\path4

Expected Behavior
I was expecting to get only my five tabs in my new windows (or at least, my five tabs + a sixth tab with default profile).
Plus, I was expecting my five tabs to be organized in the same way I organized my calls (first path0, then path1 and so on).
Actual Behavior
Instead, I always get 10 tabs : the five I was expecting (with git-bash profile) plus five with default profile.
Example, let us say that my default profile is "Windows powershell". I got the following :

Plus, the tabs with the profile I want are always organized randomly.

Yes, that definitely work a lot better !
Thank you very much for you amazingly fast answer ! 💯
I even ended up using your command line to create a new profile, which was what I was trying to accomplish in the first place.
Thanks again !!


[DOCS] Fix some of the docs errors

This PR reduces the doc warnings introduced by Python file cleaning from 8x to 53. Most of them are duplicated targets and reference not found. Are you going to fix it in this PR or I could file another one based on this to fix them?

@comaniac  updated, also cc @ZihengJiang @icemelon9


Deprecated StatsD Exporter (fixes #82)

Issue #, if available: #82
Description of changes: Previously the envoy prometheus exporter did not emit metrics around request latencies, and it was necessary to include a sidecar for exporting statsd metrics to obtain these. This is no longer the case, so the statsd exporter sidecar is not required any more.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Is there something up with the TravisCI job? It seems to have run on my initial PR, but the new commits I've added don't seem to have triggered it.
I have manually tested the new deprecation warning commit by building on my machine (via the Dockerfile), and testing that the deprecation warning shows successfully if the flag is used.


Sound in html5 only coming through one speaker

On lime 2.6.1 and openfl 3.3.2, when playing html5 sounds they only come through one of my speakers. This is reproducible via PiratePig. The html5 version exhibits the single channel behaviour, where the neko and native versions plays the sound just fine.

Seems it is only a chrome issue. Opening up the same html5 app in safari plays the sound correctly. If no-one else has this issue with chrome, feel free to close, it could just be my computer. Interestingly though, youtube and other sound players are not affected in Chrome.

see https://github.com/openfl/lime/issues/556

Fixed :smile:


publish go-mod/cache as release artifact

What would you like to be added: I would like to see the go-mod/cache added as release artifact next to the source and the compiled binaries. Please extend the Release action to also publish a compressed tarball of all used dependencies.
Why is this needed: To have an available and consistent dependency package for distributions that want to distribute syft.
Additional context:
For source packages, some (many/all?) distributions require all artifacts to build the package (including their dependencies) to be present if not relying on distribution specific library/include dependencies (which would be rather unusual for a go package). These dependencies also need to be available for offline usage and may not be fetched at compile time through the language-specific module resolver (i.e. go mod download or npm i). Usually this results in someone creating a tarball of he deps. But this tarball could be much easier created and made available by the source package (here) rather than by each distribution package maintainer. Also, there would be higher consistency across distributions when made available from an authoritative party.
If size is an issue (> 500MB per version!), it could be limited to still supported or last x versions.
Example: Gentoo's syft package has a tarball of all dependencies at https://dev.gentoo.org/~williamh/dist/${VERSION}-deps.tar.xz which makes the whole package update process unnecessarily complicated.

Yes, you are perfectly right. Having the vendor-dir should be sufficient.


How should one setup container volumes with approach?

I'm looking for the equivalent of
VOLUME /app/output

in the csproj
If this is already supported today, it should be documented here

Volumes are not currently supported. This is actually the first request we've had for them. Would you mind describing what you'd do with them, and how you'd use them on the container versus a compose file or something?

Well, as a public container producer (mcr.microsoft.com/openapi/kiota) I added the volumes to the dockerfile so they are "advertised" to consumers when the inspect the image.
This way they "know": "oh this container will look at those volumes if I map them"
I'd be expecting to be able to do the same with this approach of building my container image.

In https://github.com/dotnet/sdk-container-builds/discussions/521 I discovered a new use case for Volume support that bumps the priority up a bit for me - Volumes are how you get proper permissions configured for any volume mounts that users may do.
This is especially important for rootless execution - a user can mount a volume via the Docker CLI but those mounts won't have the same ownership as the users home directory, so the rootless app won't be able to access them. To correct this the app would need to declare Volumes for the expected mount locations, which effectively works like creating a directory with the property ownership at that location.

I did some investigation and this may not be as simple as I had hoped. Some reference materials:

https://github.com/moby/moby/issues/45919
https://www.redhat.com/sysadmin/rootless-podman-user-namespace-modes

When you create a volume in a Dockerfile, a new entry is added to the Volumes property of the configuration:
"Volumes": {
    "/home/app/data": {}
},

The Docker daemon still mounts this as root:root, however - regardless of what configuration the image author or the container runner may pass. Podman has some controls to help support this (see the linked article) but there is not currently any standard here. I worry that complex rootless execution modes will be difficult for .NET users using the SDK container tooling to adopt as a result.

Just for my understanding: is rootless such a new and uncommon scenario? It surprises me that there's no established solution for such scenarios yet

Is there any proper workaround for this yet? I was using dotnet publish then mounting the resultant image with a volume via the docker command line but now with the rootless images I can't write to the volume.
I can't find it in the docs but is there an option to at least revert back to a root image as before? What I'm running isn't important security-wise... I just need it to work as before


FT6336U::FT6336U(int8_t sda, int8_t scl, uint8_t rst_n, uint8_t int_n)

I use ESP32S3 and the code configuration is as follows.
#define I2C_SDA 2
#define I2C_SCL 1
#define RST_N_PIN -1
#define INT_N_PIN -1
FT6336U ft6336u(I2C_SDA, I2C_SCL, RST_N_PIN, INT_N_PIN);
When I looked at the serial port, after each reset, the serial port printed the following, of course, ft6336 is working normally.
E (16) gpio: gpio_set_level(226): GPIO output gpio_num error
E (26) gpio: gpio_set_level(226): GPIO output gpio_num error
I checked the FT6336U.cpp file. The problem is with the function digitalWrite(rst n, X).
It is located on lines 44 and 46.
I printed the values for rst_n and int_n and found that they were both 255, not -1.
So. I submit feedback to you.

at STM32f103 needs only : FT6336U ft6336u(RST_N_PIN, INT_N_PIN);

This library needs to set the REAL gpio num for RST_N and INT_N. It means I assume the circuit will connect RST_N and INT_N to FT6336U.
But I did not write error handling. So, I do not guarantee that it will work properly if you don't set RST_N pin and INT_N pin.

I use ESP32S3 and the code configuration is as follows.
#define I2C_SDA 2
#define I2C_SCL 1
#define RST_N_PIN -1
#define INT_N_PIN -1
FT6336U ft6336u(I2C_SDA, I2C_SCL, RST_N_PIN, INT_N_PIN);
When I looked at the serial port, after each reset, the serial port printed the following, of course, ft6336 is working normally.
E (16) gpio: gpio_set_level(226): GPIO output gpio_num error
E (26) gpio: gpio_set_level(226): GPIO output gpio_num error
I checked the FT6336U.cpp file. The problem is with the function digitalWrite(rst n, X).
It is located on lines 44 and 46.
I printed the values for rst_n and int_n and found that they were both 255, not -1.
So. I submit feedback to you.

It work only on SDA and SCL
Pins?


Added factorial program using loops in C language.

What type of PR is this? (check all applicable)

[x] 🚀 Added Name
[x] ✨ Feature
[x] 🌟 stared the repo
[x] 🐛 Grammatical Error
[x] 📝 Documentation Update
[x] 🚩 Other

Description
Add Link of GitHub Profile

Please give "hacktoberfest accepted" label to it.

Please give "hacktoberfest accepted" label to it so it will accepted as pull request on hacktoberfest platform


Fixed the issue of ServiceBusReceiverClient recreating the link after closing the client

Fixes https://github.com/Azure/azure-sdk-for-java/issues/34664

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


⚠ Bofrost.fr has degraded performance

In 866baed, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1518 ms

Resolved: Bofrost.fr performance has improved in 5d434cb after 8 minutes.


IMPALA-2729: Support default values for DECIMAL in Avro.

Merged necessary patch from Jansson.
Support '\0' in the middle of string:
https://github.com/akheron/jansson/commit/9c259c07aa53381df5819ef61627342c932d626f

Merged useful fixes from Jansson.
Fix NUL byte check for object keys:
https://github.com/akheron/jansson/commit/4d5aead31cbc783915a1e0fe2e240316e5d71631
json_string_set* functions should set the length too:
https://github.com/akheron/jansson/commit/a76dc45512acf8182f4fad5d3a9d5db6e08bbe9e

Avro C library parsses string with length info. Old api is not changed.

Have run test on http://sandbox.jenkins.cloudera.com/user/hxu/my-views/view/Impala/job/impala-master-64bit-PRIVATE-custom-toolchain/. with IMPALA-3045 change (in code review)


Dump in ZCL_EXCEL_READER_2007->load_worksheet  CONVT_NO_NUMBER

Hi ,
another issue with my very complex excel file in the reader class
It seems sometime excel have an span property of row with two values.
And then it dumps with message
"240 5454:5" cannot be interpreted as a number
Workaround , but not the best one because you lose the second value. But it works with my excel without any issues.
Line : 2427 of ZCL_EXCEL_READER_2007
    FIND ALL occurrences OF ':' IN ls_row-spans MATCH COUNT lv_counter.
    if lv_counter = 1.
        SPLIT ls_row-spans AT ':' INTO lv_min_col_s lv_max_col_s.
    else.
*    BugFix Sometimes excel have two values
* Work Around use the first one
      data lv_row_span_1 type string.
      data lv_row_span_2 type string.
      SPLIT ls_row-spans AT ' ' INTO lv_row_span_1 lv_row_span_2.
      SPLIT lv_row_span_1  AT ':' INTO lv_min_col_s lv_max_col_s.
    ENDIF.

Sorry for the late answer. I don't think I can reproduce your issue. Could you provide an Excel file to reproduce it please?
Thank you.

I think I have just reproduced the issue and I proposed a temporary fix in this other same issue: #407. If you agree, we may close your issue and continue in the other one. Thank you.

No answer. Closing.


Rework live data cache to use caffine

Rework live data cache to use caffine, make use of built-in expiry, stats etc

Done, build 482


Anti ND module is not working correctly

Description of issue
The Anti ND module is not working correctly, no part of it currently works regardless of settings chosen.

I know what the issues are and am working on fixing it now.

PR raised: https://github.com/Global-Conflicts-ArmA/Olsen-Framework-Arma-3/pull/50


Tensorflow 2.1.0 2.1.0-rc0 2.1.0-rc2 causes SIGSEGV when is used with textgenrnn

Hi All,
I have a problem when I build TensorFlow 2.1.0 2.1.0-rc0 2.1.0-rc2 from sources on CentOS 7
build is successfully compiled without any problems but when trying to learn from a text file with textgenrnn package
it causes SIGSEGV error, you can see the log from gdb same on mentioned versions
Missing separate debuginfos, use: debuginfo-install python3-3.6.8-10.el7.x86_64 (gdb) run ./train-sources.py -k půjčky-limited Starting program: /usr/bin/python3 ./train-sources.py -k půjčky-limited [Thread debugging using libthread_db enabled] Using host libthread_db library "/lib64/libthread_db.so.1". [New Thread 0x7fffecf3f700 (LWP 27545)] [New Thread 0x7fffec73e700 (LWP 27546)] [New Thread 0x7fffe9f3d700 (LWP 27547)] warning: File "/usr/lib64/libstdc++.so.6.0.27-gdb.py" auto-loading has been declined by your auto-load safe-path' set to "$debugdir:$datadir/auto-load:/usr/bin/mono-gdb.py".
To enable execution of this file add
add-auto-load-safe-path /usr/lib64/libstdc++.so.6.0.27-gdb.py
line to your configuration file "/home/jpuchky/.gdbinit".
To completely disable this security protection add
set auto-load safe-path /
line to your configuration file "/home/jpuchky/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
info "(gdb)Auto-loading safe path"
[Thread 0x7fffec73e700 (LWP 27546) exited]
[Thread 0x7fffe9f3d700 (LWP 27547) exited]
[Thread 0x7fffecf3f700 (LWP 27545) exited]
Detaching after fork from child process 27777.
Detaching after fork from child process 27779.
[New Thread 0x7fffe9f3d700 (LWP 27893)]
[New Thread 0x7fffec73e700 (LWP 27894)]
[New Thread 0x7fffecf3f700 (LWP 27895)]
[New Thread 0x7fffb95aa700 (LWP 27896)]
[New Thread 0x7fffb8da9700 (LWP 27897)]
[New Thread 0x7fffa3fff700 (LWP 27898)]
[New Thread 0x7fffa37fe700 (LWP 27899)]
[New Thread 0x7fffa2ffd700 (LWP 27900)]
2020-02-20 19:06:27.583250: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
[Thread 0x7fffa37fe700 (LWP 27899) exited]
[New Thread 0x7fffa37fe700 (LWP 27901)]
[New Thread 0x7fffa27fc700 (LWP 27902)]
[New Thread 0x7fffa1ffb700 (LWP 27903)]
[New Thread 0x7fffa17fa700 (LWP 27904)]
2020-02-20 19:06:27.588578: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x34fe630 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-02-20 19:06:27.588620: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
[New Thread 0x7fffa0ff9700 (LWP 27905)]
[New Thread 0x7fff83fff700 (LWP 27906)]
[New Thread 0x7fff837fe700 (LWP 27907)]
[New Thread 0x7fff82ffd700 (LWP 27908)]
[New Thread 0x7fff827fc700 (LWP 27909)]
Program received signal SIGSEGV, Segmentation fault.
---Type  to continue, or q  to quit---
[Switching to Thread 0x7fffb8da9700 (LWP 27897)]
0x00007fffd8967b20 in Eigen::internal::TensorBlockIOV2<unsigned int, long, 2, 1>::Copy(Eigen::internal::TensorBlockIOV2<unsigned int, long, 2, 1>::Dst const&, Eigen::internal::TensorBlockIOV2<unsigned int, long, 2, 1>::Src const&, Eigen::DSizes<int, 2> const&) ()
from /usr/local/lib64/python3.6/site-packages/tensorflow_core/python/_pywrap_tensorflow_internal.so
(gdb)
`

@sjurajpuchky,
In order to expedite the trouble-shooting process, could you please provide the complete code to reproduce the issue reported here. Thanks!

@amahendrakar
`#!/usr/bin/python3
from textgenrnn import textgenrnn
import sys
import getopt
import os
def usage():
print('train-sources.py -k ')
noinspection SpellCheckingInspection
def main(argv):
keyword = ''
epochs = 10
try:
opts, args = getopt.getopt(argv, "hk:e:", ["keyword=", "epochs="])
except getopt.GetoptError:
usage()
sys.exit(2)
for opt, arg in opts:
if opt == '-h':
usage()
sys.exit()
elif opt in ("-k", "--keyword"):
keyword = arg
elif opt in ("-e", "--epochs"):
epochs = arg
srcfile = os.path.dirname(os.path.realpath(__file__)) + '/sources/' + keyword + '.txt'
wfile = os.path.dirname(os.path.realpath(__file__)) + '/weights/' + keyword + '.hdf5'
textgen = textgenrnn(weights_path=wfile)
textgen.train_from_file(srcfile, new_model=True, num_epochs=int(epochs))
textgen.save()

if name == "main":
main(sys.argv[1:])
`
srcfile is plain text file

absl-py (0.9.0) astor (0.8.1) cached-property (1.5.1) cachetools (4.0.0) certifi (2019.11.28) chardet (3.0.4) cycler (0.10.0) docker (2.6.1) docker-pycreds (0.2.1) dockerpty (0.4.1) docopt (0.6.2) future (0.18.2) gast (0.2.2) google-auth (1.11.2) google-auth-oauthlib (0.4.1) google-pasta (0.1.8) grpcio (1.27.2) h5py (2.10.0) idna (2.9) joblib (0.14.1) jsonschema (2.5.1) Keras-Applications (1.0.8) Keras-Preprocessing (1.1.0) kiwisolver (1.1.0) Markdown (3.2.1) matplotlib (3.1.3) mock (4.0.1) numpy (1.18.1) oauthlib (3.1.0) opt-einsum (3.1.0) pip (9.0.3) protobuf (3.11.3) pyasn1 (0.4.8) pyasn1-modules (0.2.8) pyparsing (2.4.6) pypi-install (0.0.5) PySocks (1.6.8) python-dateutil (2.8.1) PyYAML (3.11) requests (2.23.0) requests-oauthlib (1.3.0) rsa (4.0) scikit-learn (0.22.1) scipy (1.4.1) setuptools (45.2.0) six (1.14.0) tensorboard (2.0.2) tensorflow (2.1.0rc0) tensorflow-estimator (2.0.1) termcolor (1.1.0) textgenrnn (2.0.0) texttable (1.4.0) tqdm (4.42.1) urllib3 (1.25.8) websocket-client (0.47.0) Werkzeug (1.0.0) wheel (0.34.2) wrapt (1.12.0) youtube-dl (2019.9.28)
Thanks a lot

System
Linux localhost.localdomain 3.10.0-957.21.3.el7.x86_64 #1 SMP Tue Jun 18 16:35:19 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
CentOS 7

@sjurajpuchky,
I was unable to reproduce the issue due to the missing .hdf5 file. You can take a look at the gist here. Could you please provide all the supporting files needed to run your code.
Also, please add ``` before and after your code to preserve the indentation of the code. Thanks!

You have to install which contains all required files
I do not have hdf5 file I train network from text file
also have to mention that my version of tensorflow is 2.1.0-rc2 builded from source on centos 7.1
what i suggest is common problem
see attached WHL file which is my build

`#!/usr/bin/python3
from textgenrnn import textgenrnn
import sys
import getopt
import os
def usage():
print('train-sources.py -k ')
noinspection SpellCheckingInspection
def main(argv):
keyword = ''
epochs = 10
try:
opts, args = getopt.getopt(argv, "hk:e:", ["keyword=", "epochs="])
except getopt.GetoptError:
usage()
sys.exit(2)
for opt, arg in opts:
if opt == '-h':
usage()
sys.exit()
elif opt in ("-k", "--keyword"):
keyword = arg
elif opt in ("-e", "--epochs"):
epochs = arg
srcfile = os.path.dirname(os.path.realpath(__file__)) + '/sources/' + keyword + '.txt'
wfile = os.path.dirname(os.path.realpath(__file__)) + '/weights/' + keyword + '.hdf5'
textgen = textgenrnn(weights_path=wfile)
textgen.train_from_file(srcfile, new_model=True, num_epochs=int(epochs))
textgen.save()

if name == "main":
main(sys.argv[1:])

Name: textgenrnn
Version: 2.0.0
Summary: Easily train your own text-generating neural network of any size and complexity
Home-page: https://github.com/minimaxir/textgenrnn
Author: Max Woolf
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.6/site-packages/textgenrnn-2.0.0-py3.6.egg
Requires: h5py, scikit-learn, tqdm, tensorflow
Metadata-Version: 2.1
Installer:
Classifiers:
Entry-points:
`

@sjurajpuchky Please add ``` before and after your code blocks. Otherwise, the things you post are unreadable and if you don't invest some time into the issue to make it easier to understand other contributors will just move over the issue and look at other ones.

@mihaimaruseac I have fixed it but it's not working properly

It's 3 backticks, not two

Can you check if the two packages (TF and textgenrnn) are built using the same toolchains?

@sjurajpuchky,
I was able to run the above code without any issues using TF-nightly. Please find the output attached to this message.

I had used the sample hacker_news.hdf5 and hacker_news_2000.txt files given in the documentation of textgenrnn as the input. Thanks!
36937 full_output.txt

Thanks a lot i will try tommorow

Its working with version r2.2 build on CentOS 7 Python 3.6
Thanks a lot

Happy to help @sjurajpuchky. Closing this issue as resolved.


Make some parameters configurable for DataNodeDiskMetrics

JIRA: https://issues.apache.org/jira/browse/HDFS-15814
For ease of use, especially for small clusters, we can change some parameters(MIN_OUTLIER_DETECTION_DISKS, SLOW_DISK_LOW_THRESHOLD_MS) configurable.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 18s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 49s

trunk passed

+1 :green_heart:
compile
1m 21s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 13s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

+1 :green_heart:
checkstyle
1m 12s

trunk passed

+1 :green_heart:
mvnsite
1m 19s

trunk passed

+1 :green_heart:
shadedclient
17m 34s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 55s

trunk passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 28s

trunk passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

+0 :ok:
spotbugs
3m 13s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 11s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 18s

the patch passed

+1 :green_heart:
compile
1m 17s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 17s

the patch passed

+1 :green_heart:
compile
1m  8s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

+1 :green_heart:
javac
1m  8s

the patch passed

-0 :warning:
checkstyle
1m  4s
/diff-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 5 new + 611 unchanged - 2 fixed = 616 total (was 613)

+1 :green_heart:
mvnsite
1m 15s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  2s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
15m 27s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 49s

the patch passed with JDK Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 23s

the patch passed with JDK Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

+1 :green_heart:
findbugs
3m 17s

the patch passed

_ Other Tests _

-1 :x:
unit
215m 33s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 37s

The patch does not generate ASF License warnings.

308m 54s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.datanode.TestNNHandlesBlockReportPerStorage

hadoop.hdfs.server.balancer.TestBalancerWithHANameNodes

hadoop.hdfs.TestReconstructStripedFile

hadoop.hdfs.server.namenode.TestNamenodeCapacityReport

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2676

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle xml

uname
Linux 16c46c9927cc 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f37bf651993

Default Java
Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-<IP_ADDRESS>+1-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_275-8u275-b01-0ubuntu1~20.04-b01

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/1/testReport/

Max. process+thread count
2252 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/1/console

versions
git=2.25.1 maven=3.6.3 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Failed junit tests:
hadoop.hdfs.server.datanode.TestNNHandlesBlockReportPerStorage
hadoop.hdfs.server.balancer.TestBalancerWithHANameNodes
hadoop.hdfs.TestReconstructStripedFile
hadoop.hdfs.server.namenode.TestNamenodeCapacityReport
Sorry. I didn't update those four unit tests.

@hanishakoneru
Hi Hanisha Koneru, could you please review the code for me? Thanks!

The change looks pretty good to me. Please file an Apache jira.
The failed tests look unrelated to your change.

The change looks pretty good to me. Please file an Apache jira.
The failed tests look unrelated to your change.

Thank you for your review!
This is the Apache jira: https://issues.apache.org/jira/browse/HDFS-15814

Can you look at the 5 checkstyle warnings flagged above whether they were introduced by the patch?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
24m 52s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
35m  2s

trunk passed

+1 :green_heart:
compile
1m 20s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
compile
1m 11s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
checkstyle
1m  9s

trunk passed

+1 :green_heart:
mvnsite
1m 22s

trunk passed

+1 :green_heart:
shadedclient
18m 23s

branch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 53s

trunk passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 23s

trunk passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+0 :ok:
spotbugs
3m 15s

Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
3m 13s

trunk passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 14s

the patch passed

+1 :green_heart:
compile
1m 16s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javac
1m 16s

the patch passed

+1 :green_heart:
compile
1m  9s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
javac
1m  9s

the patch passed

+1 :green_heart:
checkstyle
1m  5s

hadoop-hdfs-project/hadoop-hdfs: The patch generated 0 new + 612 unchanged - 2 fixed = 612 total (was 614)

+1 :green_heart:
mvnsite
1m 13s

the patch passed

+1 :green_heart:
whitespace
0m  0s

The patch has no whitespace issues.

+1 :green_heart:
xml
0m  1s

The patch has no ill-formed XML file.

+1 :green_heart:
shadedclient
14m 54s

patch has no errors when building and testing our client artifacts.

+1 :green_heart:
javadoc
0m 48s

the patch passed with JDK Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04

+1 :green_heart:
javadoc
1m 19s

the patch passed with JDK Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

+1 :green_heart:
findbugs
3m 12s

the patch passed

_ Other Tests _

-1 :x:
unit
213m 49s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

330m 25s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestRollingUpgrade

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2676

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle xml

uname
Linux df5455cfa77e 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / f9a073c6c18

Default Java
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/2/testReport/

Max. process+thread count
2456 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2676/2/console

versions
git=2.25.1 maven=3.6.3 findbugs=4.0.6

Powered by
Apache Yetus 0.13.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Can you look at the 5 checkstyle warnings flagged above whether they were introduced by the patch?

Hi, Arpit. Thank you for your advice. I fixed the checkstyle warnings. And the failed test (hadoop.hdfs.TestRollingUpgrade ) is unrelated to the change.

Thanks @arp7 and @dineshchitlangia .


Set up password-less access from internally

I just noticed that OpenSSH_jll defines the following:

which has me thinking - could we not add a passwordless connection from within this package? I realized I also had to use ssh-copy-id<EMAIL_ADDRESS>which is not defined by OpenSSH_jll. But perhaps the defined ssh_agent could do this? Alternatively, it is also possible to run arbitrary commands via run, so this should be doable either way.

We might be able to automate some setup and I welcome PRs to make this easier. Or just to document the process.
However, we can't entirely automate getting the user's public key to the remote machine in a secure way. To do this, the user needs a way to get the public key data onto that machine securely. Some examples of reasonable ways to do this:

Physically walking to that machine with a USB stick and copying the key
Copy the key data using SSH username and password login for the remote machine
Sending your public key to the admin of the remote machine via an email
Using a remote desktop protocol with username and password
etc etc

I'll never make RemoteREPL insecure by default, but any other type of PR which helps the user to set things up is great.
In certain situations it might be possible to automate some more things by using HTTPS as a transport instead of SSH and relying on the public key infrastructure which comes with that to identify the server to the client. This would also secure the link against MITM attacks, similar to what we get from SSH. However, it still wouldn't solve the problem of identifying clients to the server as trusted clients who are allowed to start a session. Requiring users to set up their SSH login solves both problems of secure transport and identifying trusted clients. See https://github.com/c42f/RemoteREPL.jl/issues/25 for more about TLS


Proposal: Add Diagonal type to mat

I think we should add a Diagonal type to mat. If we also add TriBanded, then I think that will be a complete set of types for real-valued matrices in mat.
Diagonal matrices are a pretty core concept, and I think it'll be something users look for. Actually having it there will make it easy. We've also had a bunch of requests for extracting the diagonal of a matrix (and related), and having a Diagonal type is a clear means to do this.
What is special about Diagonal is that it implements both Triangular and Symmetric, which cannot be done by either TriBanded or SymBanded. This makes it difficult when the task is to extract and then use for a specific purpose, as we can't guess in advance which the user wants. A Diagonal type would implement both, and so there is no need to know.
There is one problem, which is that a Diagonal is both upper and lower triangular, and that is impossible to represent in a boolean. We could redefine the type to be Upper/Lower/Both, but this would have a lot of ramifications on the current codebase, and it would be generally uglier and harder to deal with. I think there is a clean solution though. NewDiagDense  (and associated constructions) return the Diagonal as an upper triangular matrix, just like the default position for Triangular (such as in Cholesky) is to use an upper triangular matrix. The Transpose method on Diagonal would then swap Upper to Lower (and vis-versa), but otherwise be a no-op. This would make it easy to use a Diagonal as a Triangular matrix that requires a specific orientation, but does not change any of our existing code.

(adding Stability label because this affects NewDiagonal)


Read configuration from pyproject.toml

Thanks for your quick "fix" of #26
I have another proposal: It would be great if it wasn't required to add another configuration file but to include a liccheck section in the new pyproject.toml file ( https://www.python.org/dev/peps/pep-0518/ ), similar to how black is doing it:

https://black.readthedocs.io/en/stable/pyproject_toml.html
https://github.com/python/black/blob/1fbf7251ccdb58ba93301622388615633ecc348a/black.py#L210

A sample configuration might look like this:
[tool.liccheck]
authorized_licenses = [
  "bsd",
  "new bsd",
  "bsd license"
]
unauthorized_licenses = [
  "gpl v3"
]
authorized_packages = [
  "uuid: 1.30"
]

What do you think about this? I'll be happy to open a PR for this.

This sounds like a great idea to improve developer experience! (Actually I did not know about PEP-0518 😄.)
Please feel free to submit a PR.

So is this issue still open ?

Releasing version 0.4.1 with PR #32 "as is". Feel free to experiment with it. Thanks.


[Table] Action buttons visual defects when parent is expanded

Describe the bug
The new fix we did for actions buttons has a small defect when a row is expanded. Seems like action buttons are always show when expanded but the background color is only on hover. So you can see the rows behind it.
To Reproduce
Steps to reproduce the behavior:

Go to https://carbon-addons-iot-react.com/?path=/story/watson-iot-table--stateful-example-with-expansion-and-column-resize
Click on the first row toggle
Remove hover of expanded row
See error

Expected behavior
We should not see action buttons when not being hovered.
Additional context
If applicable, add screenshots to help explain your problem.
Add any other context about the problem here.

Related to #1085

:tada: This issue has been resolved in version 2.71.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Statestore not available in third-party ui nodes

Hi guys,
I want to store my dynamic properties (from msg.ui_update) into the State store. However that fails, because only the Data store seems to be available:

Had a quick look and I think it should be added here in the ui_base:
        node.stores = {
            data: datastore
        }

I will implement this change myself during the weekend and test it.
But then at least it has been officially registered, in case somebody else meanwhile has the same problem.
Bart

That was indeed what I needed.  Seems to be better now after a quick fix:

Will create a pull request later on...

Closed by #1075


Wrong date on latest release date.  s/b January 22, 2021 not 2020

Wrong date on latest release date.  s/b January 22, 2021 not 2020
So much for version control....

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

ID: 811ed1f4-1c78-aa1b-e752-307250ca8eb7
Version Independent ID: f36629bf-b2e6-4993-7c77-32fa311ea942
Content: Download and install Azure Data Studio - Azure Data Studio
Content Source: docs/azure-data-studio/download-azure-data-studio.md
Product: azure-data-studio
Technology: azure-data-studio
GitHub Login: @yualan
Microsoft Alias: alayu

@gregsoc -- Curtis, thank you for your feedback.  This has been fixed and is now live.


nixos/borgmatic: automatically add binaries required for database dumping

Fixes https://github.com/NixOS/nixpkgs/issues/329602 .
Coreutils does seem to be a reasonable addition to the systemd unit closure; the others are added only conditional on the relevant configuration parameter being set, and stay override-able.
I also included sudo -u ${username} when applicable by default because postgresql defaults to peer authentication; it still stays override-able for anyone who'd have a non-default setup.
Things done
Basically nothing, only added the lines suggested in #329602 (with the sudo) on my machine, and checked that postgresql backups now work fine.

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/)
25.05 Release Notes (or backporting 24.11 and 25.05 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

[ ] Fits CONTRIBUTING.md.

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

@ofborg build borgmatic.tests

Do not land yet: the sudo mechanism is not yet ready due to systemd privilege restrictions. I'm in the process of fixing that up by relaxing the restrictions by default iff sudo is required by default, but that'll likely take me a bit more time, while I figure out exactly which hardening parameters I exactly need to disable.

With the latest push, this is ready to land, I tried a similar thing with my postgresql database and it works fine for me :)

@ofbord build borgmatic.tests

@ofborg build borgmatic.tests

@ofborg build borgmatic.tests

https://github.com/ofborg build borgmatic.tests

@ofborg build borgmatic.tests


Configure Azure Pipelines

This configuration builds the application and produces builds for both Debug and Release configurations.

 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: Igor1306 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


chore: temporarily disable swaps feature

Try out Leather build 886bb40 — Extension build, Test report, Storybook, Chromatic

This pr adds swapsEnabled parameter in config, and disables swaps button, showing tooltip on hover, video:
https://github.com/leather-wallet/extension/assets/46521087/47b85904-64f1-44b7-9e16-9a795aa448f7

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Added the ability to control the swaps feature in the wallet configuration with a new swapsEnabled setting.
Introduced a tooltip for the Swap button to indicate when swaps are temporarily disabled.

Updates

Updated the IconButton component to support a disabled state, including visual feedback with reduced opacity.

Enhancements

Improved user experience by dynamically enabling or disabling the Swap button based on the configuration.

LGTM
Here is a stub guide you can link to.
https://leather.gitbook.io/guides/troubleshooting/integrated-swap-disabled

Looking good — Since we're actually disabling the button here, how about moving the learn more into the tooltip itself? This way it'll also be more explicit / clearer.

We could just hide the button like we do for testnet if not enabled?


Change field to final

Fixes #.
Motivation
Explain the content here.
Explain why you want to make the changes and what problem you're trying to solve.
Modifications
Describe the modifications you've done.
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@garimellasrisudha please submit the issue first and associate the pr with the issue, like other pull requests.

Add the serial number of the issue you are fixing here.


dh_make in stubbs:archive can be invoked non-interactively

I'm debugging a build issue and for whatever reason in the environment I'm building, this line creates a problem:
<EMAIL_ADDRESS>\
    yes | dh_make --indep --copyright apache --email<EMAIL_ADDRESS>
dh_make should support a -y option to invoke it non-interactively
so this issue is to track the possibility of changing to that invocation style

Silly request, would you mind changing -y to --yes?

Silly request, would you mind changing -y to --yes?

Done :)

Closed by #246


Issue] STS client is not using regional endpoints

Our use case is to start the SageMaker training job from the SageMaker Studio notebook and the studio is attached to a private vpc.
When I try to create SSHEstimatorWrapper using below from my studio notebook
ssh_wrapper = SSHEstimatorWrapper.create(pytorch_estimator, connection_wait_time_seconds=0)
I'm getting ConnectTimeoutError: Connect timeout on endpoint URL: "https://sts.amazonaws.com/" exception.
This is because since we have regional vpc endpoint and we can access only regional endpoints like https://sts.us-east-1.amazonaws.com/ . From here  I see that this calls only global endpoint
We would need to pass region parameter during the initalization of sts boto3 client so that it uses sts regional endpoints based on teh region

We can set env variables to use the regional endpoint.
os.environ["AWS_STS_REGIONAL_ENDPOINTS"] = "regional"

Released in v2.1.0:
https://github.com/aws-samples/sagemaker-ssh-helper/releases/tag/v2.1.0


Hot Reloading when manually triggering multiple builds with compiler.run

Feature request

I'm putting together a tool that uses Webpack as part of a build system. I've got the static builds to work and am working on dev builds with hot module reloading now.
Because the Webpack build relies on previous build steps, I cannot simple uses compiler.watch to watch for changed files. Instead, I'm relying on sequential calls to compiler.run.
This seems to work and the builds feel incremental (rebuilds are speedy). The HMR plugin seems to even pick up that a rebuild is taking place and logs this in the browser console.
There's one issue however: HotModuleReplacementPlugin is not emitting hot-update.json files because it thinks every build is the "first" build.
Is there a way to make HotModuleReplacementPlugin to realise that subsequent calls to compiler.run should be emitting hot-update.json files?
What is motivation or use case for adding/changing the behavior?
I'm building a tool that needs webpack --watch-like behaviour, but also needs to coordinate the Webpack build with other build steps.
How should this be implemented in your opinion?
Not entirely sure what the right way to do this is. AFAICT, HotModuleReplacementPlugin uses some heuristics to determine wether or not to output the hot-update.json files. An extra option to override that heuristic might be nice.
OTOH, I'm not even sure if sequentially calling compiler.run is the right way to go with this. Any other tips on how to coordinate compiler.watch builds with build steps that happen outside of Webpack are welcome too!
Are you willing to work on this yourself?
yes

Any news on this?

Sounds like a bug in the HMR plugin. I think it should better check if there is info about the last build in records to determine if a HMR update should be emitted.


.htaccess possible ?

Hi,
have you tried using .htaccess  to restrict access to the cirro server ? At present okta or google are a little too complicated, but initial tests seem to  lead to many, many password queries when trying .htaccess. Is cirro inherently unsuitable ?
Thanks for this nice tool.
Colin

Are you using Apache as a reverse proxy? This should work to require a user and password to access the server, however all datasets will be available to all users.


Fixing update torpedo yaml for image pull secrets

What this PR does / why we need it:
Fixing docker pull issue in torpedo
Which issue(s) this PR fixes (optional)
Closes #PTX-23068
Special notes for your reviewer:
Test job : https://jenkins.pwx.dev.purestorage.com/view/Leela-monitoring-jobs/job/Torpedo/job/tp-nextpx-btrfs-eks/73/console
kubectl -n vdbench-sharedv4-pooladddisk-0-03-27-18h51m43s get po
NAME                                READY   STATUS    RESTARTS   AGE
vdbench-sharedv4-7b566bb875-5v4vd   1/1     Running   0          4m44s
vdbench-sharedv4-7b566bb875-9rn64   1/1     Running   0          4m44s
vdbench-sharedv4-7b566bb875-rlbq8   1/1     Running   0          4m44s

kubectl -n nginx-sv4-svc-pooladddisk-0-03-27-18h51m43s get po
NAME                     READY   STATUS    RESTARTS   AGE
nginx-6bd8749764-d5vdb   1/1     Running   0          4m49s
nginx-6bd8749764-gqfql   1/1     Running   0          4m49s
nginx-6bd8749764-wt65k   1/1     Running   0          4m49s

kubectl -n fio-writes-pooladddisk-0-03-27-18h51m43s get po
NAME    READY   STATUS    RESTARTS        AGE
fio-0   2/2     Running   0               4m53s
fio-1   2/2     Running   0               3m56s
fio-2   2/2     Running   1 (2m44s ago)   3m27s
fio-3   2/2     Running   0               2m42s
fio-4   2/2     Running   0               2m5s
fio-5   2/2     Running   0               73s

kubectl -n elasticsearch-pooladddisk-0-03-27-18h51m43s get po
NAME                       READY   STATUS    RESTARTS   AGE
es-load-555449867f-r9xrt   1/1     Running   0          5m2s
esnode-0                   1/1     Running   0          5m2s
esnode-1                   1/1     Running   0          3m13s
esnode-2                   1/1     Running   0          103s

kubectl -n kube-system get po -l name=debug
NAME          READY   STATUS    RESTARTS   AGE
debug-2glc4   1/1     Running   0          6m
debug-6wbtx   1/1     Running   0          6m
debug-77vlg   1/1     Running   0          6m
debug-7b2dw   1/1     Running   0          6m1s
debug-8hbwt   1/1     Running   0          6m
debug-bt2d7   1/1     Running   0          6m
debug-cp2qw   1/1     Running   0          6m
debug-nx4tg   1/1     Running   0          6m1s
debug-pck2m   1/1     Running   0          6m1s
debug-s6m7k   1/1     Running   0          6m

kubectl get po
NAME      READY   STATUS    RESTARTS   AGE
torpedo   1/1     Running   0          6m25s

Add more verbose messages for success and failure, like which secret name and which namespace, this will help to debug later on, if and when there are issues

Fixed


Add documentation site

Pull Request
📖 Description

This PR adds the scaffolding for a documentation site. It is created using a mixture of webpack and a build script which converts markdown files.
There are only a few test pages which will be filled out.
The intent of this PR is to establish a baseline for future documentation that will be published to github pages.
The style is similar to the styling of the FAST repositories docusaurus site.
✅ Checklist
General

[ ] I have added tests for my changes.
[x] I have tested my changes.
[x] I have updated the project documentation to reflect my changes.

⏭ Next Steps

Fill out documentation pages.

After reading some documentation on serving static pages, closing this PR in favor of creating a branch gh-pages.

What was the plan for this? Was it an FHL project?

@EisenbergEffect yes, it was an FHL project, there are now PRs out against the gh-pages branch which is hosting most of this logic. I think when it is finalized, the logic will move to the main branch and the gh-pages branch will eventually be only for generating the static docs. This will take some time to get to though, as the documentation pages still need content.


Create "min" versions of css and js files

Hi @alexandernst ,
I did once, and it was a bit troublesome maintaining the min versions.
I don't really have experience on automatic builders / minimizers.


Remove Fedora 22 as it's EOL

Describe the proposed changes
Fedora 22 is EOL, so will be removed as a supported platform for Docker 1.13;
https://fedoramagazine.org/fedora-22-end-of-life-2016-july/

With the recent release of Fedora 24, Fedora 22 will officially enter End Of
Life (EOL) status on July 19th, 2016. After July 19th, all packages in the
Fedora 22 repositories will no longer receive security, bugfix, or enhancement
updates, and no new packages will be added to the Fedora 22 collection.
Upgrading to Fedora 23 or Fedora 24 before July 19th 2016 is highly recommended
for all users still running Fedora 22.

Project version

Engine 1.13
Related issue or PR in another project

https://github.com/docker/docker/pull/27432

lgtm

LGTM

Thanks, @thaJeztah :)


Incoming avoidance

I believe everything is been taken care of incoming avoidance so I'm creating another poor quest to find the close
and this will close issue 22
close #22
:ballot_box_with_check: Definition of Done checklist

[ ] Meaningful title and valuable description in pull request
[ ] Code change tested manually if not covered by automated tests
[ ] Pull request linked to one or more issues/stories
[ ] Acceptance criteria of the linked stories satisfied
[ ] Code change comprehended by those who approved it
[ ] State of target branch maintained or improved after the code change

Velma problem going to fix it on my local branch and then push it and then another pull request


Using Chartboost via SonarCocosHelper

Hi!
I successfully added Chartboost with your Cocos Helper! Great!
Now I called SonarCocosHelper::Chartboost::showFullscreenAd();. After this call I am using the cocos2d-x director to replace the current Scene. Sadly the scene is replaced and after that the fullscreen ad is shown. The game itself isn't paused. How can i achieve that a) the ad is shown before the replace and b) the app is paused like it's when using admob.
Thanks!

Same Issue with showVideoAd();

For pausing we recommend you actually halt using Cocos2d-x code, sort of like a state machine.
For the other task try using a timer.


Trim whitespace on color input

 #fff should be valid, and we should trim trailing and leading whitespace. better user experience.

Go for it @JoshuaRotimi !


Guide: How to upload a file of any size?

Is your feature request related to a problem? Please describe.
Hi, I couldn't find anything about file handling in the docs. How can we upload a file with the connect protocol? I can use bytes as a type in the proto, and it works, but I think it has limits.
More info:
I used Client Streaming to upload a file from Node.js to Node.js but how about Web? According to https://github.com/bufbuild/connect-es/issues/669 Client Streaming can't be supported yet.
Thank you in advance.

Going to close this in favor of https://github.com/bufbuild/connect-es/issues/669 since both are interrelated.


Implemented _removeItem on Windows

Alrighty, take 3. This time, it just changes FileManager+Win32.swift.

@swift-ci please test and merge


Make migrate wait until valid ECS host

We were hitting a race condition where an EC2 instance was "Healthy"
but had not registered with ECS yet

The script would then continue onwards and set another host as
"Draining" in ECS before the new instance was a registered host

ECS then couldn't drain that host with the healthy percentage window
so it would get stuck in "Draining"

When the new EC2 instance did register, ECS did a pretty bad job of
realising this and rescheduling tasks onto it

This commit refactors the EC2 checks into a method that also checks
the ECS status if the command is being run against a cluster

This should ensure there is always somewhere else to place tasks
before we start another host draining

@bandesz Thanks for the approve - I can't merge into this repo though so you'll have to hit the merge button if you are happy


Example config file

Hello!
I'm trying to set up ipmi_exporter on a host to do remote monitoring on two HP servers.
The problem I'm running into is that I don't quite know how to set up the config file.
(Unless I'm missing something) In the readme it only says the syntax is similar to blackbox. However I don't quite know what modules exist nor do I know what the exact syntax is.
Having an example config.file would be very helpful. If not then just a list of modules and a general overview.

Hi there!
Please not that config.file only refers to the command line parameter used to specify the config file to be used, i.e. with the latest version you would run a command like this: ./ipmi_exporter --config.file=ipmi.yml
As stated in the README, there are two example configuration files available, ipmi_local.yml for scraping the IPMI interface on the local host, and ipmi_remote.yml for scraping remote hosts.
With the local version, you would run the exporter on each server you want to monitor. With the remote version, you could run the exporter on some host (e.g. the Prometheus server) and use the same exporter to monitor all hosts. The latter will make the Prometheus config a bit more complex, but it is also documented in the README.
The "modules" in the IPMI exporter are only sets of configuration items that you define yourself. There are no predefined modules. If your servers are configured the same way (including usernames and passwords), you can use a single module to scrape them both. If not, you'll need a separate module for each server.
I am pretty sure the example configs will answer most of your questions. Let me know if anything remains unclear. If you have any improvement suggestions for the README, I am happy to consider those as well, however the example configs are clearly already in there, so I will go ahead and close this issue :wink:
(feel free to use it for follow-up questions anyways)

Thanks, I guess I missed the files, sorry!
I do have a bug report (of sorts), not used to GitHub, so not sure if should be written here or as a separate issue.
On the example prometheus config for setting up remote ipmi hosts there's an error at line 3
- job_name: ipmi
  params:
    module: default
  scrape_interval: 1m
[...]

Just leaving it default errored prometheus out, with the log saying

Apr 29configuration (--config.file="/etc/prometheus/prometheus.yml"): parsing YAML file /etc/prometheus/prometheus.yml: yaml: unmarshal errors:\n  line 51: cannot unmarshal !!str default into []

So I looked back at how I set up the blackbox exporter and put default into brackets
Like so: ,
- job_name: ipmi
  params:
    module: [default]
  scrape_interval: 1m
[...]

Doing that fixed the issue.


fixes for cmake when integration with other project

Only CMake changes and small documentation update - no reviewer added.


Missing permissions

I have luckperms installed on a sponge server for 1.12.2 and just installed nucleus to give more commands that I can use for myself and others like /tpa, /spawn, /home etc. After installing nucleus, the permissions for pixelmon disappeared from luckperms completely. Is there a way to re-display them?

In what context do you mean they have disappeared? Re-display them where, how?
Need more info.

So, for example, when you open the editor and use the search bar at the bottom, it gives you a full list of permissions right? Before, the permissions for pixelmon and the other mods I have installed were "pixelmon.command.[]" or "wondertrade.command.[]" but now all that is there are nucleus commands.

Right, so that list isn't actually a "full list of permissions", it's merely a list of suggestions based on what LuckPerms knows. There are certain reasons why permissions will be shown there:

The permissions have been defined correctly within the plugin/mod (I actually have no idea about mods so can't say whether there is some sort of interface for defining permissions which is readable from a plugin like LP)
The permissions have been checked since the server was last restarted - LP will cache checked permissions and send these to the web editor - it's highly possible that the permissions you usually see just haven't been checked since the last restart, so as a result they are not shown this time.

You could attempt to perform some permission checks for the missing nodes and see if that helps. Usually running something like /help will generate a ton of permission checks and might populate the list (again, no idea if this works for mods).

The issue also persists in-game as well. If I were to use tab to auto-complete, pixelmon and other mods don't show up as an option to set permissions for.

Did you read what Turbotailz sent?

Yea, I'm doing what he suggested to do with the /help command but it doesn't seem to be working. I'll restart the server and my own client and see if that fixes the issue

The /help command seems to have worked. Thanks!


Device address format issue

The device address format is confusing and not the consistent. It has been a hex representative value in the past, which is easier to read. But with commonizing the input fields it has become a number value.

Fixed


Remove files with contributions from the people where no approval of the licensing terms was received

A while back, my RFC to relicense this repo under a dual Apache2/MIT license has been merged. Since then I've opened five issues to track approval of the new licensing terms by the various contributors to this repository and gotten the approval of almost every contributor! Only for the contributions of 7 people I was unable to obtain approval, either because they didn't respond to my contact attempts, or because they were unable to give me approval.
The relevant commits without approval are:
Barosl Lee (3):
      568ed6a75653b75dcd4f36a102afa59cd73f018f
      5b9a48d4ba1e314ce7d4e97d06b91c4a9d6e7865
      be5eb6f88e3e89059917f168123ba10dd2900f43

Benjamin San Souci (1):
      938ff6dac5c19dd6830e3400f6799bfc174b7950

Carl Lerche (3):
      1e5c6030cf48a3021b59dc898c7d0c8f7ec12bcf
      50ff468f9b1d7bafb912374e7df2c51329112d6e
      a17aa3563a07fdedd301faa783a92c9212831eb2

Clark Gaebel (3):
      a1d6e9efc5e264de49ea25b8d5d161972f5105b9
      ab6265e31f7748a8bfd4b7f9451935f677f011be
      d54bd9b55af927b7b9f27c4a5a1a9cb44c91aa53

Sean Patrick Santos (12):
      2d2d81aecdc862f5e2120f03f2f9f0d38534e7d3
      7a3df4d89fd4150aefaea0f23d176baada27c472
      c4024520b3725f1457f311cd6f9984f3863dc9e1
      734e123838907397e689084360012b36905398ed
      e58a632345654f8041fb390be062d4e080d46d34
      2c5876ffe70d5a1395218b275eb7eb90cf7afa78
      d4eec1f72cb36eaf7de651d37665d40e22dda1c1
      8f975bdb96bbcce362c7171feeebca82909a8f6b
      3b5abbfd05049b38dce501f862bcf37b71ad587e
      4ba33ee94a14cb40411c283f9ad4bff7bf8ec0e1
      1ad497df2877d46a85fa256c63797e3d00c24491
      992550b1ac1869c3ef0591ac64c55ca961656f2c

Tom Jakubowski (1):
      8d322ada15e05e0b5640c2dd95a4d3df3b8542c3

Jakub Bukaj (1):
      c539b10a99c59bb96ba4657ff9263e2074aaa637

These commits have touched the following files:
Barosl Lee:
text/0000-rename-connect-to-join.md
text/0000-panic-safe-slicing.md

Benjamin San Souci:
text/0114-closures.md

Carl Lerche:
text/0000-move-thread-local-to-std-thread.md
text/0517-io-os-reform.md

Clark Gaebel:
active/0000-smaller-refcounts.md
text/0000-collections-reform-part-2.md

Sean Patrick Santos:
text/0000-new-array-repeat-syntax.md
0601-replace-be-with-become.md
text/0601-replace-be-with-become.md
text/0195-associated-items.md

Tom Jakubowski:
active/0031-better-temporary-lifetimes.md

Jakub Bukaj:
text/0000-feature-gate-box-patterns.md

Additionally, Clark Gaebel has touched active/0000-smaller-refcounts.md, but it was moved to a rejected directory in commit 7443c30a25b7ddc10a36ee3c3344596cb60ed549 and removed from the repository in commit abd6134d150c8b2523600c1f76f473fc5a4438ac.
This PR removes the current incarnations of those files, ordered by contributor, except of text/0114-closures.md which only received typo fixes by the relevant commit. Here I've chosen a git revert instead.
If in the future approval can be obtained from these contributors, I hope that their changes can be applied again.

cc @aturon @joshtriplett

Removing accepted RFCs from the repo seems very very wrong to me. Much worse than having a small subset of files with an unclear license.

I would just add a licence: All rights reserved Author1, Author2, ... to the files as well.

My PR doesn't remove the RFCs from the repo, only from the master branch. If you go back in history, the files are still available. You can still link to them, inspect them, etc. A full clone of the git repo will still contain the files as blobs. You can do deletion from the history as well with git (requiring force pushes and breaking sha1 hashes), but it's not what I'm proposing in my PR.
Removing them from the master branch will mean that we prevent accidental infringement like e.g. if we opt to host a rendered version of the RFCs on doc.rust-lang.org. IANAL, but from my guess it seems that hosting them through Github Pages (like #2192 proposes it) would be okay as it's covered by the license the contributors granted through the TOS to Github, while hosting them on EC2 where doc.rust-lang.org is might get us into trouble already.

Sure they’re in the git history, but IMO that’s not enough. These RFCs still describe aspects of Rust that the community has discussed on found consensus on, and they still shape it. They should be kept available together with the rest of RFCs, and existing links to github’s rendering of the master branch should not be broken.

@SimonSapin what about adding stubs like the ones I've just pushed? This will allow hosting in different places and make best use of the limited possibilities we have by pointing to Github.

I think we'd need to talk to Mozilla's legal. I'd hope that something like what @nagisa proposes with an explicit statement that certain people possess rights to particular files would be possible. I am against removing merged RFCs from the repository, even if they remain in git history. RFCs are intended to be "permanent" so to speak...

Removing accepted RFCs is not OK, especially after this little time. A relicensing process is hard, and we shouldn't skip steps. First of all, for the people who simply haven't responded, we should both try other contact methods and give more time. And for the people who are "unable" to give approval, we should work with them to figure out the issue there. We've gone years with the RFC repository not having a clear license; we can go a few more months while we do this the right way.

@joshtriplett we can chat about this in private. Are you using IRC?

Talking to @joshtriplett has convinced me to give this more time and more contact attempts before issuing further steps.

Thanks for your work on this, @est31; please let us know if you need further help.


update replacement logic for repos cloned via https

This PR updates the logic to clean urls in repos that are cloned via https. Fixes #2

Thank you!


Fixing Documentation Formatting & Added styling

Hello,
This PR aims to implement & fix #29
Changes to the documentation's formatting have been made where ever necessary :)
Thanks :)

@bari12 @bziemons Can you please review this PR? ... just wanted to know about the same :)

Hello @bziemons, I also fixed the sidebar.json for having all the release-notes on the documentation. It was not working while building the documentation. 😄

Hello @bziemons, I also fixed the sidebar.json for having all the release-notes on the documentation. It was not working while building the documentation. smile

we opted against having to list every new release file anywhere. There's currently only a generated page here: https://rucio.cern.ch/documentation/release-notes – what's wrong?

If we build the documentation locally, it throws out a lot of warnings. 🤔 It doesn't recognise the release-notes folder as a valid entry in the documentation

If we build the documentation locally, it throws out a lot of warnings. thinking It doesn't recognise the release-notes folder as a valid entry in the documentation. It doesn't even build it locally if we don't remove the "release-notes" entry from the sidebar.json

Did you run ./rucio-generate-doc.py ? I admit, the documentation on building the docs is a bit scarce. The current steps are in the workflow file: https://github.com/rucio/documentation/blob/main/.github/workflows/update_documentation.yml#L23-L32 (there is no need to use the docker login/logout commands, since those are just for the GH Actions cache usage)

Did you run ./rucio-generate-doc.py ? I admit, the documentation on building the docs is a bit scarce. The current steps are in the workflow file: https://github.com/rucio/documentation/blob/main/.github/workflows/update_documentation.yml#L23-L32 (there is no need to use the docker login/logout commands, since those are just for the GH Actions cache usage)

Yes, I did use those commands for building the Website, used the yarn run build and then yarn run start. Still gave the same errors 😢


Suggestion: Remove Instability

Would it be possible to completely remove instability from the infusion altar?Thanks.

Would be possible.  Will probably add it as a config option in future versions.

OK very cool, Thanks :)


Incompatible with prettier-plugin-twig-melody

Hello,
Regarding this part of the README.

This means this plugin is incompatible with other Prettier plugins that are using the same APIs.
...
If you discover any other incompatibilities, open an issue and hopefully we can figure out a way to make it work.

This plugin does not work with *.twig files, with prettier-plugin-twig-melody
prettier version: 2.5.1
prettier-plugin-twig-melody version: 0.4.6
tailwindcss version: 2.2.19
My .prettierrc.js
module.exports = {
  bracketSpacing: true,
  printWidth: 100,
  semi: true,
  singleQuote: true,
  tabWidth: 2,
  trailingComma: 'es5',
  useTabs: false,
  vueIndentScriptAndStyle: true,
  htmlWhitespaceSensitivity: 'ignore',
  twigPrintWidth: 100,
  twigMultiTags: ['html_element,end_html_element', 'with,endwith'],
  tailwindConfig: './tailwind.config.js'
};

Only the settings from the twig plugin are applied when running prettier, class names stay in the same order.

The source from the plugin does not have a parser for Twig:
https://github.com/tailwindlabs/prettier-plugin-tailwindcss/blob/main/src/index.js#L349
If a different approach can be taken to use this tailwind prettier plugin with twig files, i'm all ears
Anyway this plugin is 🔥

Same request for me, how to make this plugin work with twig files? 🤔 #PrettierBeginner

@florianbouvot
Hm, .html it works fine  but on .html.twig not.
I'm using PHPStorm. Did you something special? :D

Hey! Thanks for reporting this. I've created a new issue (#31) dedicated to keeping track of Prettier plugins that are not compatible with this plugin. Closing this issue in favor of that 👍
Please feel free to post this incompatibility with prettier-plugin-twig-melody in that issue, it would be great to have it there for quick reference.

I posted the same question in the discord but worried it's too specific to get an answer there.
Just wondering if there is a way in VSC to enjoy class sorting via prettier-plugin-tailwindcss inside of .twig files, ...but without losing basic twig syntax highlighting...which would be lost if I simply asked VSC to treat .twig as .html.

Is it possible to set this plugin as the parser of twig files without having to use the HTML parser too?

@kerns did you find a solution?


Calculating Ascent Profiles

I've created this so @gruneisen and I can continue a conversation started on the Mechjeb Github.

I can say that the manner in which I go about calculating the ascent profile is pretty simple and straightforward. I definitely think that it could be easily implemented into KOS. The hard part is determining the optimum profile, which is what is done in Simulink.
To calculate the ascent profile, I wrote a function for altitude vs. range:
alt(range) = alt_final*(1-e^(-gamma*range)
Of course, it makes a little bit more sense to cast equation in such that range is a function of altitude, since I found it was difficult to perturb the range of the rocket and get into the 'meat' of the above function. This makes it:
range(alt) = 1/gamma*ln(1-alt/f_alt)
Where alt_final is the final altitude you want and gamma is the 'shape' parameter. If you plug this into EXCEL and run it down, you can see how gamma effects the shape of the ascent. This formula is pretty simple and asymptotically converges to the final altitude you choose. For metric units, I found that gamma ~1.0e-06 is a good starting point.
In Simulink, I run the simulation by using PID controllers to keep the gravity vector on the ascent profile vector. Then,  I optimize gamma and the start of the gravity turn in order to maximize the velocity at burnout and keep the vertical velocity greater than zero. (I think there are couple of other things I optimize / constrain, but I haven't had a chance to go back into the simulation in about a week and a half).
All this being said, I think programming in the ascent profile function and the control loop to keep the gravity vector inline with the ascent vector in KOS wouldn't be too difficult. But then again, I'm more comfortable with the math side of things than the coding.

Well I know what I'm doing this weekend then!
I don't quite see how the velocity works in just yet, but I'll have play in Excel and maybe ask some questions, and then start coding. I'm flat out with RL stuff until tomorrow afternoon though.
Actually I have a question now - I had been thinking of the problem in terms of altitude, pitch and velocity. Not range. (I assume that we're talking horizontal distance from the launchpad. (How does curvature affect this?)). Now though I see that you're modelling the shape that the ascent profile takes on. I assume that this corresponds to the shape that an opitmal gravity turn would take? Can we map your function in some way to give a target pitch (Not the direction the rocket is pointing, but the angle between the velocity vector and the horizon), for a given altitude and velocity? Or am I approaching this wrong?
I'll have to leave it at that and check back in tomorrow.

I too thought about problem in terms of altitude, pitch, and velocity however pitch and velocity are really dependent on altitude and range (and yes range is horizontal distance - I take no account of curvature. That's a good point! I'll have to think about that!). Therefore, once you have altitude and range, you can calculate everything else!
The function does not necessarily yield the optimum shape - its just an arbitrary function I chose because I thought it would do well - if you (or anyone else) knows of a a better shape function, then I'd love to use it.
Mapping the function to give a target pitch is done inside my simulation right now by feeding the difference between the angle of the ascent function and the angle of the gravity vector (the angle of the gravity vector being as you described it above - between the vector and the horizon), and then feeding the difference between the two (the error signal) into a PID controller to generate the pitch drive signal. This way the PID controller serves to minimize the error between the ascent profile and the velocity vector by modulating the pitch. My intention had been to use the PID output (gimbal as a function of time or altitude or whatever) as an input to the KSP autopilot. KOS, however, should be able to run the control loop internally and therefore use a dynamic solution rather than a static one.
If KOS can do this difference and PID work internally, then there is no need for the external simulation other than to optimize the shape of the ascent! But that could be done just with trial and error inside KSP...just time consuming, I guess, but not impossible.

Yeah I've been working on a general purpose ascent script by trial and error up to this point, and got some very good results and smooth efficient ascents. But I'm trying to get the ascent profile something other than based on magic numbers I've arrived at by trial and error.
If you look at the the files in this repository, you'll see that one of them is a PID library. The 'cooked' control that you can read about in the KOS documentation is basically a hard-coded PID of some kind as well, so that side is definitely covered by KOS, it's just a matter of calculating the appropriate setpoints as the ascent goes on.
We're definitely talking about the same thing. I hope you're online this weekend to talk to while I work on it. I'm in New Zealand (GMT+1200), what timezone are you in?

Yes, we're definitely taking about the same thing. I'm in California, so GMT-8:00. I'll be online this weekend, though I may be sporadic - my wife and I just had a child (a major part if the reason I haven't been able to work on my simulations). I should be able to read and chat during naps ;-).
Just so we can link up in the same level, what's your background? I'm a mechanical engineer and do simulations and analysis.
In order to make it more useful, I could go back to excel since it now has a pretty powerful optimization capability built in, but I'd be curious what your interest would be.


Numberwang fix

Fixed the Numberwang algorithm to correctly recognise recently discovered Numberwang values. Fixes #204
The following Numberwang formats are now recognised:
3i
-2275
2.4
√14
885%
F0rty-s1x
Shinty-six
Filth-hundred and neeb
http://numberwang.wikia.com/wiki/Numberwang_Wikia:All_numbers

agreed. this bring us one step closer to having all 37 volumes of the rulebook in code!


Update readme

Update readme:

Change examples to ES6 (Node v4 was recently set to end-of-life)
Add "About" and "Table of Contents" headings
Remove bower installation guide (not modern / well supported anymore)

@blewisio Thank you so much for the contributions. But I am not sure about the removal of browser installation part, can you please explain why do you think it should be removed?

Hey @mubaidr, thanks for the response. To be clear, it's the bower part, not the browser part. Bower used to be more popular, but it's not anymore. Many consider it irrelevant now-a-days. This is directly from the top of the bower website:

...psst! While Bower is maintained, we recommend using Yarn and Webpack or Parcel for front-end projects read how to migrate!

If you want to leave it in, that'd be okay with me, but I don't think it's useful anymore as almost everybody is using Yarn/Webpack/Parcel.

@blewisio I am sorry, I don't know why I read it as browser. :(
Thanks once again! 👍


skipping network configuration during configure-bosh option

I can an only configure iaas and director options.
The other options result in the following error, and are ignored:
skipping network configuration: detected deployed director - cannot modify network
it appears that lines 95-100 are absolute, when it should only be for a specific option, and result in the rest of the options not being test for.

@DonBower - if you already have a bosh director deployed we are intentionally stopping you from reconfiguring the network due to the fact that the Ops Manager throws an error when you attempt to do this.
If you want to be able to reconfigure the bosh director network you have to uninstall the director tile.

@zachgersh I believe this is a self imposed restriction.
There are lots of changes I can make via the GUI, that OM has blocked.
Just one example, Add a new Network.

Hi @DonBower,
We don't currently use the Ops Manager API to configure the director, so I think this check was put in place to attempt to protect the user. We are going to investigate switching to an API endpoint for this soon so we will revisit the need for this fast-fail at that time.
Regards,
Dave


[Feature] Redesign workload_resource_test function

For most of the tests we use "workload_resource_test" function to iterate through resources and test required functionality.
Problem is that this function iterates for each container in each resource and sometimes it is not necessary. In some tests, "cnf_workload_resources" function is used for better efficiency, but i don't see that as a good final solution:

Resource formats provided by these two functions is different, which brings complications and extra adaptation code.
We don't have a "central point" for resource-related tests, which could bring confusion and instabilities for some changes.

I propose:

To make "workload_resource_test" an only function for iterations through resources in tests.
To optimize "workload_resource_test" functionality and usage, let it iterate only required amount of times depending on the test.

It might be better to check what the actual tests are intended to do and what resources they should iterate over instead. Some tests might suffice with access to containers through functions like ClusterTools.all_containers_by_resource?. The library tooling should be revised to give more targeted functions (with sensible names), i.e. get_all_resources, get_all_pods, get_all_containers and they should not be split across two different libraries.

Possible solution would be having option to choose which resources are needed for "workload_resource_test" function, and all the logic behind getting these resources would be hidden from the tests


Testset

Hi, In one of the issues, you said that the test data does not need data processing. Is MiniGPT-v2 or TextBind applied to this data? What about the results reported in the article? Because many of the images in the test set have not very good captions, which I think are more similar to the story-level narration that you mentioned in the paper.

The dataset we released contains narration and descriptions generated by TextBind, which can be used directly. We also tried MiniGPT-v2. To be honest, it generates better descriptions, but it does not perform as well as TextBind in subsequent generation tasks, so we still use TextBind by default.


Fix leftover from refactor to git-ruby

git-ruby doesn't require that anymore. Fixes #140

Ah yeah, this makes sense!
Weird though, our CI should fail, cause we were running danger local on CI

1 Error

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md.

2 Warnings

:warning:
Author @jeroenvisser101 is not a contributor

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger

Can this be considered trivial?

def

@orta weird indeed, not sure why...

$ bundle exec danger --verbose is actually what we run on Travis, so that's not local, right?

Both: https://github.com/danger/danger/blob/master/.travis.yml#L21-L22

@orta but one is init, and the other is not local, so does it internally run 'local'?

Oh yeah! Maybe we can add that, and/or afterr #132 is in, we can add tests at that level

@orta totally! I'm going to be working on #134 now :)


Lint on build

See max errors flag. Defaults to minus.
https://eslint.org/docs/user-guide/command-line-interface
Update cookbook. There is also an item there which could be useful
Note that fix flag might not fix everything when run locally but checking for errors without fix flag maybe can give error output

Done


With a large number of torrents (> 500) WebTorrent blocks, making the entire machine unresponsive

Upon re-adding 500 torrents to WebTorrent (on node, under Linux) the machine basically dies by becoming unresponsive and a hard reboot is required to continue operation.
This is the last output to the console I managed to capture:

From what I've read online the problem seems to be with I/O going synchronous when overloaded.
I think the solution might be to set a static, configurable limit on all the I/O done by WebTorrent instances.
Equally, IP connections should have a static, global limit, as in default Ubuntu configuration (without changes to sysctl) I'm experiencing global connections (unrelated to WebTorrent) being dropped very soon (even with only 100 torrents in the instance).
I realize running 500, or even 100 torrents at the same time is not something most people want to do, but there should be some basic safeguards from the app crashing the system.

related https://github.com/feross/webtorrent/issues/587 https://github.com/feross/webtorrent/issues/599

WebTorrent needs resource management - this is a known issue.


Auto-fill information from previous variant page when adding evidence items

It would save quite a bit of effort to pre-populate the add evidence form with information from the previous page. This could be Gene, Entrez ID, and Variant

I've moved this button from the Evidence tabs to the subheader, so this issue is somewhat moot.
I can still capture the current page's gene/variant IDs if they're on an event page and pre-populate the form based on that, but I'm not totally sure that's what users would expect from clicking a button in the header. Please reopen if you want to discuss further.


Introducing RBS [Ruby Static Typing]

Tasks -

[x] Install rbs_rails & steep gem and configure it #3807
[x] Add steep check to CI
[ ] Write rbs annotation files for the codebase

[ ] models
[ ] decorators
[ ] helpers
[ ] jobs
[ ] channels
[ ] project components
[ ] mailer
[ ] notification
[ ] policy
[ ] strategies
[ ] uploader

[ ] Update documentation [if required]

Are you working on this? (Yes/No)
Yes

@tachyons can you reopen this issue, as I am using this issue to like the PRs of RBS

@vedant-jain03 can you reopen this issue, it has been closed due to PR


Add Date Picker Field to a Contact Form Module

Is this currently possible with new API?
Where to start?)
Any insights about the native Divi support of this feature in the nearest future?

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

Any news on this? It's now April 2019 and @lots0logs last reply said 'later this year' in June 2018. I agree with the original poster, a date picker is sorely needed. In addition, there should be an option to add additional labels / titles above the fields.


Update intellij-idea-next-* (2016.3 EAP) to build 163.6110.12

If there’s a checkbox you can’t complete for any reason, that's okay, just explain in detail why you weren’t able to do so.
After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} reports no offenses.
[x] The commit message includes the cask’s name and version.

Additionally, if adding a new cask:

[ ] Named the cask according to the token reference.
[ ] brew cask install {{cask_file}} worked successfully.
[ ] brew cask uninstall {{cask_file}} worked successfully.
[ ] Checked there are no open pull requests for the same cask.
[ ] Checked that the cask was not already refused in closed issues.

Included in #2753


Adds distributed tracing policies

Added policies for opencensus and datadog
#4976

Can one of the admins verify this patch?

cc: @chlowell @bryevdv

@SuyogSoti This is moving in the right direction, I've left some comments above


chore Fix segment cache ut's compile error due to miss cherry-pick

Proposed changes
Issue Number: close #xxx

run buildall