Fix warnings when compiling with >=2.5.0

Pull Request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

Do you think you can do the full migration to v2.5.0?

Sure, it already works on 2.5.0, I assume you mean upgrade to use the new features? I'd merge this first so other projects can start using right away though

Started the migration in other PRs


the builds were broken on the website

because the package names weren't correct, versions mainly

I guess this needs cherry-picking to a branch-0.8 and a new release?


blink pedal LED slowly to avoid confusion with fast blinking errors

we wasted a lot of time thinking our blinking pedal board was working when in fact it wasn't (stuck in bootstub).
this change makes the pedal blink slowly when firmware is properly running, to differentiate from the fast blinking that is done in bootstub during flashing or to indicate errors.

This PR should be made in the panda repo, which will then get added to the openpilot repo with next release.


[Don't review] Testing Azure pipelines

Verifying compile fix. I will not check in this PR.


Fix typos and shift examples around in UG for Add Command

Edited the UG portion for Add Command as there were some typos and flaws as pointed out in the PE dry run.

Codecov Report

Merging #125 (f2450d0) into master (b44bf21) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #125   +/-   ##
=========================================
  Coverage     30.41%   30.41%
  Complexity      499      499
=========================================
  Files           105      105
  Lines          4140     4140
  Branches        637      637
=========================================
  Hits           1259     1259
  Misses         2738     2738
  Partials        143      143

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 b44bf21...f2450d0. Read the comment docs.


[Documentation Bug]: Add Project Description and explicit Contribution Guidelines

Describe the bug ✍
Description of bug:
Missing Project Description: The README is missing a clear and concise description of the project. This description should explain:
Project Goal: What problem does the project aim to solve?
Key Features: What are the main functionalities or functionalities offered by the project?
Provide step by step information reproduce the bug 📄
solution to bug: Adding a Project Description: I've included a concise description of the project, outlining its goals and key features. This provides users with a clear understanding of what the project is about.
Providing Contribution Guidelines: I have written more clear instructions on how to contribute to the project.  This makes it easier for newcomers to get involved and contribute code.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

@Durgesh4993  please assign this issue to me under gssoc level 1 . I have already created a pull request for this issue, please review it.


Drop unittest library support

In #553 #554 #611 we have added support for the pytest. This caused that we are supporting two testing libraries in parallel, which adds code and conceptual complexity to coreblocks. Particularly I have hit in that problem when adding the hypothesis in #573, where I had to integrate it with both unittest and pytest at the same time.
This PR want to drop unittest support and left only pytest. The most important changes are in TestCaseWithSimulator class where much of code from run_simulation was moved to separate fixtures. Whats more it was possible to drop the dirty trick with wrapping functions in DependencyContext.
There is a little drawback. If we want to use DependencyContext in setUp function we have to add one more line:
    @pytest.fixture(autouse=True)
    def setup(self, configure_dependency_context):

You can observe that in test/fu/functional_common.py. It is because we have to say that configure_dependency_context has to run before setup function.
ToDo:

[x] Support for test execution
[x] Support for profiles
[x] Support for traces
[x] Support for logging
[x] Migrate some example tests
[x] Migrate all tests to the new syntax

Please say what is your opinion about these changes? Are they ok for you and can I migrate all the tests?

Some unit tests still use unitest.TestCase, is that intentional? unittest is also used in scripts.

Good catch. I will prepare a separate review for that to doesn't delay this one.


Fix Rubocop offenses

Summary
The rake command currently fails due to rubocop offenses. This should fix the problem.
Details
Though, I don't like the idea of disabling Metrics/ClassLength for the Pretty class.
We should spend time making the code more Rubocop friendly, but another time.
How Has This Been Tested?
By the CI hopefully.
Types of changes

[x] Refactor (code change that does not change external functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] I've added tests for my code
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

RuboCop shows the following warning:
Warning: no department given for UnneededInterpolation. Run `rubocop -a --only Migration/DepartmentName` to fix.

https://circleci.com/gh/cucumber/cucumber-ruby/2755
Can you apply bundle exec rubocop -a --only Migration/DepartmentName?

Can you apply bundle exec rubocop -a --only Migration/DepartmentName?

Thanks for pointing that out, that's fixed now :)


Match.Start is incorrect

The value given for Match.Start is sometimes wrong. I suspect that the copyright logic is to blame.
Example:
ISC License

Copyright (c) 2013-2017 The btcsuite developers
Copyright (c) 2015-2018 The Decred developers
Copyright (c) 2017 The Lightning Network Developers

Permission to use, copy, modify, and distribute this software for any
purpose with or without fee is hereby granted, provided that the above
copyright notice and this permission notice appear in all copies.

 IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.

Calling licensecheck.Cover with this and the default options returns the Match
{Name:ISC Type:Other Percent:93.75 Start:197 End:852 IsURL:false}

Byte offset 197 is the beginning of the word "distribute". I would have expected the start to be at "Permission", or maybe one of the Copyright lines.

Sorry for the bad example. Here's a better one:
Copyright (c) 2010 The Walk Authors. All rights reserved.

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions
are met:
1. Redistributions of source code must retain the above copyright
   notice, this list of conditions and the following disclaimer.
2. Redistributions in binary form must reproduce the above copyright
   notice, this list of conditions and the following disclaimer in the
   documentation and/or other materials provided with the distribution.
3. The names of the authors may not be used to endorse or promote products
   derived from this software without specific prior written permission.

When I run this I get Start = 1, in the middle of the word "Copyright".

I did think of the bug, but only fixed half of it!


Option to ignore empty messages composed entirely of non-printable characters - ie: line breaks, tabs and spaces

Is your feature request related to a problem? Please describe.
I'm managing a Kubernetes cluster with a large number of Java applications, most of which tend to use empty line breaks between log messages. Fluentd sends logs to a Graylog instance, and the GELF protocol refuses empty records.
java.lang.IllegalArgumentException: GELF message <e0281040-79cd-11ea-8753-0000ac19acf0> (received from <<IP_ADDRESS>:50695>) has empty mandatory "short_message" field.

While this problem is particularly related to the GELF protocol, refusing to store apparently empty messages in a search index is a reasonable requirement.
Because of the nature of our applications, Graylog is flooded with the same error messages, which takes its toll on performance at large scale.
Under the hood, in docker logs, these messages appear with the following format:
{"log":"\n","stream":"stderr","time":"2020-03-25T18:08:47.256011014Z"}

Fluentd captures that record and sends \n as the message content.
I've been trying to configure fluentd to ignore messages that contain a single new line character \n, but I can't seem to get that filter to work. While I work this out in the help forums, I decided to write this feature request.

Describe the solution you'd like
Add a global flag somewhere in the native fluentd configuration that allows these empty messages to be ignored, mainly messages composed entirely of space, line break and tab characters -- the usual \s+ in a regular expression.

Describe alternatives you've considered
I'm trying to configure fluentd filters to ignore these log messages, but I'm having a hard time getting it to work. The following filter has no effect:
<filter **>
  @type grep
  <exclude>
    key message
    pattern ^\n$
  </exclude>
</filter>
<filter **>
  @type grep
  <exclude>
    key short_message
    pattern ^\n$
  </exclude>
</filter>

The following grep test works. This might help you https://docs.fluentd.org/configuration/config-file#note-on-match-order
<source>
  @type dummy
  tag dummy
  dummy {"message": "\n"}
</source>

<filter **>
  @type grep
  <exclude>
    key message
    pattern ^\n$
  </exclude>
</filter>

<match>
  @type stdout

  <buffer>
    flush_interval 1s
  </buffer>
</match>

I was not aware of the dummy source. That is so helpful!
I managed to figure out my problem. I was matching against key message, when in fact I sould be matchin key log
{"log":"\n","stream":"stderr","time":"2020-03-25T18:08:47.256011014Z"}

So, the following works as expected:
<filter **>
  @type grep
  <exclude>
    key log
    pattern ^\n$
  </exclude>
</filter>

Thank you!


Test User Signin

What does this PR do?
This PR ensures that users are able to sign in successfully
Description of Task to be completed?
Test should be written to confirm users email
Test should be written to verify users token
How should this be manually tested?
npm test
What are the relevant pivotal tracker stories?
157801291

Pull Request Test Coverage Report for Build 138

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.251%

Totals

Change from base Build 129:
0.0%

Covered Lines:
139

Relevant Lines:
257

💛  - Coveralls


[FEATURE] move the grid generator input into the input section

Description
copy this -> https://github.com/Dun-sin/Code-Magic/blob/main/index.html#L1419-L1505
paste it after this -> https://github.com/Dun-sin/Code-Magic/blob/main/index.html#L1414
and fix whatever issues occur
Screenshots
No response
Additional information
No response
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
None

Hello @Dun-sin , I want to work on this issue. Please assign this issue to me.

Hello @Dun-sin , I want to work on this issue. Please assign this issue to me.

Sure, thanks for wanting to contribute. Make sure to read the issue description carefully and ask if you have questions on the discord server. Follow the rules here, or your PR won't be accepted and will be closed. Good luck!

I would like to work on this issue

can you assign me this issue @Dun-sin

can you assign me this issue @Dun-sin

Sure, thanks for wanting to contribute. Make sure to read the issue description carefully and ask if you have questions on the discord server. Follow the rules here, or your PR won't be accepted and will be closed. Good luck!

:tada: This issue has been resolved in version 2.79.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Utilise any 433MHz devices others than shades?

Hi,
I'm wondering if I could also also use the ESPSomfy for detecting/controlling other devices for complete different purposes, such as an ELRO AB440 remote or an ELRO plug etc. (of course using a second device working on 433.92)? I'm not aware of the protocol they're using, but it seems to be a quite basic and common static protocol (with fixed house codes defined by dip switches and without rolling codes) ...
I still have a few of these remotes lying around and find them pretty handy, so I would like to continue using them with home assistant... (I saw an ESPHome adoption for the Sonoff RF Bridge, but I would prefer your device if possible as it is so great and moreover I'm already used to it ...😇)

Many other utilisations are already available, see screenshot.

If you have tuned the radio to 433.92 and have not seen frames in the log (any frames valid or not) then this is probably not possible.  There are a lot of solutions out there that try to be a swiss army knife for every possible device but they make you use the spring loaded scissors to cut leather.  They work... but not very well.
If it is an RTx protocol as indicated by frames in the log then the dry contact functions will likely work.  If there are no frames logged then decoding it is probably a chore for someone else since I do not have any of these devices.

didn't try that so far (as my currently only available device is in production for moving my somfy shades VERY WELL!! 😇👍)
But yea - you're absolutely right...


Reviewed code for Style tag

Here is my new PR, following your remarks here #24. I've also answered to them directly. Thanks for having already merged the Image tag.

Thanks for your work, again. And please excuse the fact that I haven't responded yet. I'm currently busy with another project that has a very strict deadline, but it will be done soon, and I will get back to you ASAP.

No problem at all. No need for excuses, I totally understand. No rush at all, really, take your time. Thanks again for reviewing my changes/propositions. Have a nice day.

Finally got around to merging this. Thanks for the great contribution :)

I'll change a few more things, then push out a new release.


Usability: identify Q2 components to test and approach

Evaluate expectancy of basic components early in build, provide recommendations to inform design iterations
1 - Define components remaining to test (test at least 4 components in Q2)

Component
React
Figma

AG Grid Theme
Released in v1.0.0
N/A

Avatar
Released in v1.3.0
Ready

Border Layout
Released in v1.0.0
N/A

Button
Released in v1.0.0
Ready

Card
Released in v1.1.0
Ready

Data Grid
Released in v1.0.0
Ready

Flex Layout
Released in v1.0.0
N/A

Flow Layout
Released in v1.0.0
N/A

Grid Layout
Released in v1.0.0
Ready

Icon
Released in v1.0.0
Ready

Link
Released in v1.0.0
Ready

Panel
Released in v1.1.0
Ready

Salt Provider
Released in v1.0.0
N/A

Spinner
Released in v1.2.0
Ready

Split Layout
Released in v1.2.0
N/A

Stack Layout
Released in v1.0.0
N/A

Status Indicator
Released in v1.0.0
Ready

Text
Released in v1.0.0
Ready

Theme
Released in v1.0.0
Ready

Tooltip
Released in v1.2.0
In progress

Checkbox
In progress
Ready

Country Symbol
In progress
Ready

Form Field
In progress
In progress

List
In progress
In progress

Radio Button
In progress
Ready

AG Grid Theme
Released in v1.0.0
N/A
Avatar
Released in v1.3.0
Ready
Border Layout
Released in v1.0.0
N/A
Button
Released in v1.0.0
Ready
Card
Released in v1.1.0
Ready
Data Grid
Released in v1.0.0
Ready
Flex Layout
Released in v1.0.0
N/A
Flow Layout
Released in v1.0.0
N/A
Grid Layout
Released in v1.0.0
Ready
Icon
Released in v1.0.0
Ready
Link
Released in v1.0.0
Ready
Panel
Released in v1.1.0
Ready
Salt Provider
Released in v1.0.0
N/A
Spinner
Released in v1.2.0
Ready
Split Layout
Released in v1.2.0
N/A
Stack Layout
Released in v1.0.0
N/A
Status Indicator
Released in v1.0.0
Ready
Text
Released in v1.0.0
Ready
Theme
Released in v1.0.0
Ready
Tooltip
Released in v1.2.0
In progress
Checkbox
In progress
Ready
Country Symbol
In progress
Ready
Form Field
In progress
In progress
List
In progress
In progress
Radio Button
In progress
Ready

Meeting scheduled next week

Item submitted for testing: https://github.com/jpmorganchase/salt-ds/issues/1570

components identified, results reporting to align with all team DEMOs


Refactor the query completion checks.

Provide a new API in Foreman to check the query completion, instead of the return values of several methods.

Hey @zuyu,
I will soon start working on implementing the scheduler changes in this branch. Those changes should provide better checks for tracking query progress. If this PR is not urgently needed, we can postpone adding this feature until I implement my changes.
Let me know if that sounds reasonable.

I have two follow-up PRs based on this PR: foreman-msg-proto and work-order-proto. Therefore, I would prefer to merge this PR first.
And I'd like to know your steps to add the scheduler, particularly about Foreman as a long-running service.

LGTM, merging.


[android] SDK >= 31 - Add more feature request permission for MANAGE_EXTERNAL_STORAGE

Why it is needed?
on android SDK >=31 , when i need readDir file, need allow permission "All management of all files", but in package is not have the function.
Please, update new functions
I hope it have early!
thanks for reading.
Possible implementation
No response
Code sample
No response

Please respect the issue template.


⚠ Turnos 24x7 has degraded performance

In 1e4d188, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1028 ms

Resolved: Turnos 24x7 performance has improved in a73c37f.


Gitter IM

Created a gitter room for React Bangalore, can it be integrated to this repo.
https://gitter.im/React-Bangalore/Lobby

Nice! What's the popular form of chat in the group?
Is it slack, gitter, hipchat, etc?

Actually we already have a discord channel.
https://discord.gg/dhD8vyQ
I am not promoting it recently because it is not very active.
We can git it one more try

@siddharthkp slack is good for teams within organisations. For open source slack is not great.
@akiran Haven't used discord earlier, we can try it.
gitter works great because it gives the feel of a chat communication, though it takes time to get into that kind of mode. Maybe it is too early for that.

Btw, gitter has some cool github integrations, you can see all the activity happening on github on the side panel, who starred the repo, who commented and what not :)

Reactiflux, biggest react group is using discordapp.
https://www.reactiflux.com/
So we went discord way back them

slack is good for teams within organisations. For open source slack is not great.

Not so sure about that.
I'm biased towards slack because a lot of people* use it at work, and a lot of great OSS communities are moving to it now.
Let the community decide 😄 @akiran Poll on the react bangalore twitter?

citation required 😛

@siddharthkp Slack free account has some limitations like 10k messages etc.
But as you said, I think it is the easy for many users to use slack.
Lets start with slack to reduce the initial barrier.
When we hit the limits of free account, we might think of other solutions

@siddharthkp @Jaikant I created this auto invite link
https://reactbangalore.herokuapp.com/


Problem with GDScript line breaking

Godot version:
Godot v3.1.1
OS/device including version:
Windows 10
Issue description:
Godot detects a misplaced linebreak. I can't seem to detect it so I cannot solve it.

if Input.is_action_pressed("hum"):
		get_node("AudioStreamPlayer").play()
		counter++
	if counter == 2:
		counter = 0
		get_node("AudioStreamPlayer").stop()

The error is occurring because Godot doesn't support the ++ operator on line 35. Use counter += 1 instead.

Thank you so much!

Would it be feasible to detect usage of ++/--  and display a more user-friendly error message in this case?

That error message is a bit bogus, but since it's related to #19128 I mentioned it there.


Skip to markup triple slash comments as markdown

Hi @qinezh, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;


Game.images and Flame.images are different instances

I noticed Game.images and Flame.images are different instances and
Game.images is not be able to change because marked as final.
Is this behavior right?
Library(flame) always use Flame.images in 7 times.
Library user maybe preferred gameRef.images instead because of less file import.
abstract class Game {
  final images = Images();
  final assets = AssetsCache();
}

class Flame {
  static AssetsCache assets = AssetsCache();

  static Images images = Images();
}

We explain the diferente between the two instances on our docs, check it here for a better understanding: https://flame-engine.org/docs/#/images?id=flameimages

I understood.
Thanks your patient to explain it.


latest macOS_Updater from 5.12.2018 Error URL not found

Hello Gengik84
You last Version have a Error when will update in a HFS+Partition for Mojave 10.14.2 Bet4 to Final.
"URL not found" is the message.

Hi.. I don't know your problem but this script is ok.
I used it yesterday and today, I updated several records.
I tried again now, the result is ok.

Yes you have option „Normal“ i have Option „beta“  selectet for download i will Developer beta download nit Normal.
Greeting Fritz
Am 06.12.2018 um 18:58 schrieb Gengik84 <notifications<EMAIL_ADDRESS>Hi.. I don't know your problem but this script is ok.
I used it yesterday and today, I updated several records.
I tried again now, the result is ok.
[schermata 2018-12-06 alle 18 57 43]https://user-images.githubusercontent.com/14853542/49602472-d806e880-f988-11e8-9a11-b9a3f322fc51.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Gengik84/MacOS_Updater/issues/1#issuecomment-444967476, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AP2Lk34grgJ8BLNmWGZw64q3j3DGkZQaks5u2VrBgaJpZM4ZG6kc.

The script is ready to be used in the future 10.14.3 beta 1.
It is still not present.
So it is normal that this channel does not work for the moment.
Please see this line
https://github.com/Gengik84/MacOS_Updater/blob/master/MacOS_Updater#L29
if you want to download the old beta, you must change that line for your personal use.

Not a script issue.


Fixed event action (create) for metric events

Fixes https://github.com/sensu/sensu/issues/977

:+1:


Not building on Hirsute

This package is currently not available on 21.04, which means Tensorman can't use an NVIDIA GPU. I'm getting errors attempting to build locally:
jacob@pop-os:~/Work/nvidia-container-runtime$ debuild -b -uc -us
 dpkg-buildpackage -us -uc -ui -b
dpkg-buildpackage: info: source package nvidia-container-runtime
dpkg-buildpackage: info: source version 3.4.0-1pop1
dpkg-buildpackage: info: source distribution focal
dpkg-buildpackage: info: source changed by Ian Douglas Scott<EMAIL_ADDRESS> dpkg-source --before-build .
dpkg-buildpackage: info: host architecture amd64
 fakeroot debian/rules clean
dh clean
dh: warning: Compatibility levels before 10 are deprecated (level 9 in use)
   dh_auto_clean
dh_auto_clean: warning: Compatibility levels before 10 are deprecated (level 9 in use)
	make -j1 distclean
make[1]: Entering directory '/home/jacob/Work/nvidia-container-runtime'
rm -rf /home/jacob/Work/nvidia-container-runtime/dist
make[1]: Leaving directory '/home/jacob/Work/nvidia-container-runtime'
   dh_clean
dh_clean: warning: Compatibility levels before 10 are deprecated (level 9 in use)
 debian/rules build
dh build
dh: warning: Compatibility levels before 10 are deprecated (level 9 in use)
   dh_update_autotools_config
   dh_auto_configure
dh_auto_configure: warning: Compatibility levels before 10 are deprecated (level 9 in use)
   debian/rules override_dh_auto_build
make[1]: Entering directory '/home/jacob/Work/nvidia-container-runtime'
ln -sTf . src/vendor/src
GOCACHE="$(pwd)/gocache" GOPATH="$(pwd)/src/vendor" make -C src build
make[2]: Entering directory '/home/jacob/Work/nvidia-container-runtime/src'
go: cannot find main module, but found Gopkg.lock in /home/jacob/Work/nvidia-container-runtime/src
	to create a module there, run:
	go mod init
make[2]: *** [Makefile:8: build] Error 1
make[2]: Leaving directory '/home/jacob/Work/nvidia-container-runtime/src'
make[1]: *** [debian/rules:11: override_dh_auto_build] Error 2
make[1]: Leaving directory '/home/jacob/Work/nvidia-container-runtime'
make: *** [debian/rules:7: build] Error 2
dpkg-buildpackage: error: debian/rules build subprocess returned exit status 2
debuild: fatal error at line 1182:
dpkg-buildpackage -us -uc -ui -b failed

Not sure if this local error is the same one Jenkins had or not, since the Jenkins logs have long rotated out.

what is the reason this was removed? it makes pop 21.04 unusable for docker.io with nvidia-container-runtime, unless you use nvidia repos for ubuntu

+1
I use nvidia-container-runtime for GUI on LXD containers. Especially for ROS.
I only see the toolkit in apt-cache search, but sadly unable to install it since I get this error.
The following packages have unmet dependencies:
 nvidia-container-toolkit : Depends: libnvidia-container-tools (>= 1.3.0) but it is not installable
                            Depends: libnvidia-container-tools (< 2.0.0) but it is not installable
E: Unable to correct problems, you have held broken packages.

I'd appreciate all help.

+1
I use nvidia-container-runtime for GUI on LXD containers. Especially for ROS.
I only see the toolkit in apt-cache search, but sadly unable to install it since I get this error.
The following packages have unmet dependencies:
 nvidia-container-toolkit : Depends: libnvidia-container-tools (>= 1.3.0) but it is not installable
                            Depends: libnvidia-container-tools (< 2.0.0) but it is not installable
E: Unable to correct problems, you have held broken packages.

I'd appreciate all help.

Workaround is to add official nvidia repository and install it from here .
https://github.com/pop-os/pop/issues/1708#issuecomment-877830843


Saving edits

Is there a way to have edits saved if you click to edit another item? or if you press enter within the edit box?

Well, currently there is no such a functionality out-of-box.
But you can achieve it redefining editItem method, something like the following:
var baseEditItem = jsGrid.prototype.editItem;

jsGrid.prototype.editItem = function() {
    if(this._editingRow) {
	this.updateItem();
    }
    baseEditItem.apply(this, arguments);
};

This just seems to return 'Unable to get property 'editItem' of undefined or null reference', no previous experience of overriding methods so not sure exactly how to use this code. Any help appreciated.

Sorry, made a mistake in code. Corrected the sample above.
See working fiddle http://jsfiddle.net/tabalinas/rkxbgzrx/

That makes much more sense, thanks!


Test macOS & Windows platforms and Beta & Nightly channels

supercedes  / closes #621
r? @jyn514

I'm going to close this, as getting postgresql to actually work as a service on macOS and Windows is too much for me to investigate and work on currently.

Ok, thanks for taking a look!


'onSelect' return undefined or incorrect element when using search module

Hi,
I am using search module for tag suggestion, here's html code

      <input class="prompt" type="text" placeholder="search tags...">

and this is js code
$.fn.api.settings.api = {
        'tag search': '/tag/search?keywords={query}'
    };
$('.ui.tag.search').search({
        apiSettings: {
            action: 'tag search'
        },
        type: 'category',
        onSelect: function(result, response) {
            console.log(result)
            return false;
        }
    })

This is json returned from server.
{
  "success": true,
  "results": {
    "category1": {
      "name": "Category 1",
      "results": [
        {
          "title": "S1",
          "description": "Optional Description1"
        },
        {
          "title": "S2",
          "description": "Optional Description2"
        },
        {
          "title": "S3",
          "description": "Optional Description3"
        }
      ]
    },
    "category2": {
      "name": "Category 2",
      "results": [
        {
          "title": "S10",
          "description": "Optional Description10"
        }
      ]
    }
  }
}

All the results displayed correctly, I want do some js hack when clicking the element, so I use onSelect callback.
'S10' is logged correctly, but 'S2' and 'S3' returned 'undefined' in console after click

More weird is when I click 'S1', console shows 'S10' element

Please take a look, thanks!

There were fixed related to this in one of the last patch releases. I'm on my phone but check closed issues.
Are you using latest?

@jlukic Yes. I am using 1.11.6

Thanks for the clarification. I'll take a look

It looks like search results are removed from dom before click bubbles to dimmer causing this check to fail.

Investigating this now, thanks for your patience.

I've reworked the offending method, it was a pretty poor implementation previously, and some of the variable names were a bit confusing.
I've verified against your example using the new $.api({ mockResponse: 'foo'}); syntax which I've also added as a new feature.

Available with 2.0

Cheers!


slug is missing from og:url

Describe the bug
post slug is missing from og:url, resulting all embed linked to main page instead of post.
To Reproduce

Open demo blog post like this
og:url is set to https://nobelium.vercel.app (instead of https://nobelium.vercel.app/what-make-vs-code)

Expected behavior
og:url should be post link, not blog main page link
Screenshots

Platform:

OS: Windows
Browser: Chrome
Version: 106.0.5249.91

Additional context
I fixed by setting Layout tag slug to frontMatter.slug - See this line

This is not stale issue.

Closed - Looks like repo is not actively on development.

Closed - Looks like repo is not actively on development.

Hello, it's easy to fix this problem.
In Container.js first line add:
import { useRouter } from 'next/router'

In line 9 below add:
const router = useRouter()

Edit line 37 code:
      <meta
        property='og:url'
        content={meta.slug ? `${url}/${meta.slug}` : `${url}${router.asPath}`}
      />

Finally, if you are interested, you can refer to my fork project: Notionic


Private-DNS swagger

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
https://github.com/Azure/azure-rest-api-specs-pr/pull/156
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[x] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Swagger to SDK encountered a Subprocess error: (Azure/azure-sdk-for-go)
Command: profileBuilder -s preview -name preview
Finished with return code 127
and output:
/bin/sh: 1: profileBuilder: not found

Swagger to SDK encountered a Subprocess error: (Azure/azure-sdk-for-go)
Command: profileBuilder -s preview -name preview
Finished with return code 127
and output:
/bin/sh: 1: profileBuilder: not found

@owolabi2 you know this is the public repository right? You feature is then public and can be seen by everybody?

@owolabi2 seems this is a duplicate PR, the other PR is here.
Also whenever we introduce a new api-version for which there is a previous api version exists, it is recommended to have PR shows us the diff between API versions thus both ARM and SDK reviewers can provide feedback quickly. Could you follow below steps?

Close this PR
Copy over all the files from 2017-09-01 to 2018-03-01-preview directory.
Commit this change as `git commit -n “Initial commit (Copying over previous api version 2017-09-01)”
Now make changes that are relevant for new api-version 2018-03-01-preview. In your case, you will have to simply overwrite the files with the changes that you made (since you would have them saved.)
Now commit the changes “Changes for new API version 2018-03-01-preview”
Push the changes to your fork (git push origin <branch-name>)
Send a PR to master from your fork

This way reviewers can go to second commit and see the update specific to new api-version. This will also helps customers to see the new features.
This step is documented in our contribution guidelines (you can find this link is pasted in the checklist of every PR), see https://github.com/Azure/azure-rest-api-specs/blob/master/.github/CONTRIBUTING.md#filenames-and-folder-structure

Swagger to SDK encountered a Subprocess error: (Azure/azure-sdk-for-go)
Command: profileBuilder -s preview -name preview
Finished with return code 127
and output:
/bin/sh: 1: profileBuilder: not found


allow includeLevel to accept an array of levels

It would be nice to have this functionality to allow more detailed and customized tocs.
Example:
md.use(require('markdown-it-table-of-contents'), {
   includeLevel: [2, 3]
})

Expected output:
- h2
   - h3
   - h3
- h2
   - h3
   - h3

This would be really awesome. IMHO it would be the difference between this plugin being almost useful and fantastic! I hope the PR is merged soon!

Now available in https://github.com/Oktavilla/markdown-it-table-of-contents/releases/tag/v0.2.0

@martinlissmyr would you mind publishing 0.2 to npm when you get the chance? This plugin is working great for me btw, thanks for making it!

@nicolasartman Oh, forgot to do that. Done now!

Thank you!


split algorithm to multiple files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

177 of 177 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 96.543%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
363

Relevant Lines:
376

💛  - Coveralls


Alternative to image_tensor_mapped in Tensoreflow-Lite

Hi there, For my iOS app using Tensorflow the pre-processing was done by using image_tensor_mapped. i.e.

uint32_t *pixels = (uint32_t *) calloc(width * height, sizeof(uint32_t));

uint8_t *bufptr = (uint8_t *)pixels;
for (i = 0; i < height; i++){
   for (j = 0; j < width; j++){
        blue = bufptr[1] / 255.0;
        green = bufptr[2] / 255.0;
        red = bufptr[3] / 255.0;

        image_tensor_mapped(0, i, j, 0) = blue;
        image_tensor_mapped(0, i, j, 1) = green;
        image_tensor_mapped(0, i, j, 2) = red;

        bufptr += 4;
    }
}

Have I written custom code - No
OS Platform and Distribution - MacOS
TensorFlowLite installed from - Pod
TensorFlowLite version - 0.1.7
What will be the alternative for image_tensor_mapped in Tensorflow-lite(tflite)

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Have I written custom code - No
OS Platform and Distribution - MacOS
TensorFlow installed from Pod
TensorFlow version - r1.8
TensorFlowLite version - 0.1.7
Bazel version - 0.11.0
CUDA/cuDNN version - N/A
GPU model and memory - N/A
Exact command to reproduce - N/A


Is Access to the Labeled-3D Dataset for Human Preference Evaluation Possible?

Thank you for sharing your fascinating research!
I am very interested in the academic use of this dataset.
Are there any plans to release the Labeled-3D dataset mentioned in your paper?
If it has already been made available, could you please inform me where I can access it?
Thank you!

Thank you for your interest!
The Labeled-3D dataset will be publicly available in a few weeks.

Are the few weeks over, does this work or just hot air ?


First Pass at Demo App

Podfile is pretty typical for a Berbix integration (using the same custom specs repo)
Heavy lifting happens in the SessionViewController
Navigation and view layout are mostly in storyboard with a few hand crafted AutoLayout examples
Pod is committed and pre-installed so developers can download this repo and run in Xcode without CocoaPods
Pod contains 2 frameworks (1 device & 1 simulator). The simulator framework contains both arm64 (for M1) and x86_64 (for Intel Mac)

Remaining

[ ] Custom README for this repo

Looking through this now, but one thing is that I think the .gitignore needs to be updated. The Pods/ is checked in currently

This is intentional. Referenced in the description.

This is intentional. Referenced in the description.

ah 🤦♂  my bad


Fix up a bit of whitespace to the side of the dropdown menus.

Hey @bakert,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 449890f0-f449-11e9-9e15-dd9742ab6bc9


Add OnlyAvailable and PopularFirst options for featured products

Questions
Answers

Description?
Adds an ability to change order of the featured products and filter out products that are not available for ordering. Right now if a product is unavailable it can still be visible inside featured products which can be kinda pointless. This PR will also introduce a way to change the order of displayed products to display products which have been ordered the most.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
no

How to test?
Change module settings, Verify homepage

Good point, I mostly left it here for any1 who would want to have a feature like this. Might rework this according to the tip, thanks @Hlavtox

Since we have had no news from you for more than 20 days, I'm closing this PR. Feel free to re-open it if you find the motivation to finish it. Don't hesitate to ping me if you have any questions or doubts.


Missing board definition for CUAV X7 (non-Pro)

For general questions please use PX4 Discuss or Slack.
Describe problem solved by the proposed feature
Missing board definition for the device mentioned.
Describe your preferred solution
Being able to build PX4 for the device mentioned.
Describe possible alternatives
Is the CUAV Nora definition working with the CUAV X7, too?
Additional context

If it is PX4 standard (ie. Pixhawk compatible) then it won't have a specific board definition and would be under https://github.com/PX4/Firmware/tree/master/boards/px4.
If it is supported then it will be under https://github.com/PX4/Firmware/tree/master/boards/cuav/x7pro

The X7 is basically a Nora, but with damped sensors in the cube.


Rebase to master

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 7 committers have signed the CLA.:white_check_mark: RakhithaRR:white_check_mark: Oshanath:white_check_mark: PasanT9:white_check_mark: Avishka-Shamendra:x: Tharika Madurapperuma:x: tgtshanika:x: tharikaGitHubTharika Madurapperuma seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


withSort not sorting properly

When I have two posts with posted as part of the metadata, one posted 28/3 2016, the other posted 14/2 2016, it puts the 28/3 one at the very bottom.
The answer was to replace return a > b with
if(a > b){
          return 1;
    } else if (a < b) {
          return -1;
    }
    return 0;

on line https://github.com/assemble/handlebars-helpers/blob/master/lib/array.js#L604
I saw suggestions that someone else had gotten a similar pull request merged in, but that does not seem to there in version ^0.7.5 which is what I got when I installed it via NPM.
An oversight somewhere?

@doowb @jonschlinkert this would be fixed by bumping the version to 0.8.1 (this commit fixes it 0ad46f8dcb12ffb65c66795f5632aefa3c80c3e2)

fixed and published in 0.8.1


Improve variable naming in legacy code

The variable naming in the legacy code needs to be improved.
I'll update this issue with a guide on naming conventions

Will be completed by #178


Broken button for delete temporal coverage button

I am editing a resource and trying to delete the temporal coverage. The delete icon has no effect.

@Maurier - could you take a look at this?  Looks like the delete ui button is broken.

@sblack-usu Yeah, I'll check it out this week.

@Maurier are you working on this issue?

@martinseul I think this was fixed a while ago by the same fix to another issue. I can't reproduce this anymore.

I can confirm this is now functional. I'm closing the issue.


buildtorrent: init at 0.8

Motivation for this change
Adding buildtorrent, which is a simple tool to create .torrent files. Even if mktorrent exists, I personally prefer this one.
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

waw gg


Sagas for adding notes to a transaction

Task Description
Sagas for adding notes to a transaction
ETA
Estimated time
Dependencies
Any dependencies for the task

We've implemented this.


Not able to find lotus installation through env variable

Issue Description
try to install lotus-farcaster by running the install.sh. It displayed the following error:
ERROR: user filecoin doesn't seems to be a lotus user /home/filecoin/.lotus/config.toml doesn't exist
Expected Results
Seems the farcaster tries to find the lotus on a default installation path. Lotus-farcaster should use the LOTUS_MINER_PATH or command line parameters like --miner-repo to find the correct lotus installation.

I ran the miner and daemon nodes on different machines. So I made some modifications but was able to eventually run the dashboard. Here are some changes I made:

on the miner machine, I created a ln link to point the .lotusminer to the real directory. However, since the daemon was a different machine, so I just created a directory for .lotus and copied the api files in order to run the install and pass check. The downside of this in the future is that if I change the daemon url, I will have to remember to come back here to update it.

I edited the variables in the first part of lotus-exporter-farcaster to set the MINER_URL, MINER_TOKEN, DAEMON_URL and TOKEN. seems working fine.

After that, on the grafana machine (which is also different from the miner or daemon), I use http://localhost:9090 to set the prometheus to read data from the miner node (which has the prometheus node exporter).

Import the grafana-dashbard to see the lotus dashboard.

I ran the miner and daemon nodes on different machines. So I made some modifications but was able to eventually run the dashboard. Here are some changes I made:

on the miner machine, I created a ln link to point the .lotusminer to the real directory. However, since the daemon was a different machine, so I just created a directory for .lotus and copied the api files in order to run the install and pass check. The downside of this in the future is that if I change the daemon url, I will have to remember to come back here to update it.

I edited the variables in the first part of lotus-exporter-farcaster to set the MINER_URL, MINER_TOKEN, DAEMON_URL and TOKEN. seems working fine.

After that, on the grafana machine (which is also different from the miner or daemon), I use http://localhost:9090 to set the prometheus to read data from the miner node (which has the prometheus node exporter).

Import the grafana-dashbard to see the lotus dashboard.


Fix 22: touch fails when running in strict mode

As discussed in #22 the scripts have been updated to ensure strict mode is off when running. Set-StrictMode only applies to the current and child scopes, and so is automatically reverted on return.

Thank you!


UX Polish: Modal overlaps with the system bar

The modals for Disclaimer, About Arrival Advisor, are too close to the system bar on iPhone XS and similar iPhones (with notch)
On Staging 1.62.0

Recommendation: Either make sure the modal is under the system bar and doesn't overlap, or replace the modals with pages - eg. page used for Learn More -> Services listed in Arrival Advisor (see below) - and ensure that the page does not overlap the system bar

Acceptance Criteria

[ ]  Modal is made less tall so that it does not overlap the system bar on iPhone XS and similar iPhones

Steps for QA:

No longer touching notch.

Conflicts with #1490. About us/Disclaimer will no longer be a modal

Reopening because #1490 will still be a modal.

@cherrielam could you decide whether this current modal height is good?

@cherrielam this is a notchless screenshot on ipod touch 7th generation

I think it looks OK

@cherrielam I think this can close?

Looks good!


Trying Github actions

This is a basic action added through the Github interface to see if goreleaser works out of the box.

Codecov Report

Merging #1728 (bf22722) into master (a1f5496) will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1728      +/-   ##
==========================================
+ Coverage   58.06%   58.22%   +0.16%
==========================================
  Files         189      189
  Lines        9080     9172      +92
==========================================
+ Hits         5272     5340      +68
- Misses       3210     3230      +20
- Partials      598      602       +4

Impacted Files
Coverage Δ

internal/store/leaf/crypto.go
32.86% <0.00%> (-4.85%)
:arrow_down:

internal/action/commands.go
100.00% <0.00%> (ø)

internal/backend/crypto.go
71.43% <0.00%> (+0.60%)
:arrow_up:

internal/action/init.go
56.30% <0.00%> (+2.28%)
:arrow_up:

internal/action/clone.go
59.72% <0.00%> (+2.37%)
: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 a1f5496...bf22722. Read the comment docs.

Codecov Report

Merging #1728 (bf22722) into master (a1f5496) will increase coverage by 0.16%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1728      +/-   ##
==========================================
+ Coverage   58.06%   58.22%   +0.16%
==========================================
  Files         189      189
  Lines        9080     9172      +92
==========================================
+ Hits         5272     5340      +68
- Misses       3210     3230      +20
- Partials      598      602       +4

Impacted Files
Coverage Δ

internal/store/leaf/crypto.go
32.86% <0.00%> (-4.85%)
:arrow_down:

internal/action/commands.go
100.00% <0.00%> (ø)

internal/backend/crypto.go
71.43% <0.00%> (+0.60%)
:arrow_up:

internal/action/init.go
56.30% <0.00%> (+2.28%)
:arrow_up:

internal/action/clone.go
59.72% <0.00%> (+2.37%)
: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 a1f5496...bf22722. Read the comment docs.

This should trigger when we push to a tag v*, I'll try that next to see how it works: I didn't understand what's happening with the releases once they are produced yet 😆
Signing should be super easy to add once we've figured out how we want to handle the GPG key part:
https://github.com/goreleaser/goreleaser-action#signing

This should trigger when we push to a tag v*, I'll try that next to see how it works: I didn't understand what's happening with the releases once they are produced yet 😆
Signing should be super easy to add once we've figured out how we want to handle the GPG key part:
https://github.com/goreleaser/goreleaser-action#signing


Adding functionality to include URLs as arguments, as well as files.

Details
Added functionality to detect if the file passed as an argument starts with 'http://' or 'https://'. If it does, request the file from the server.

Hey @SecGus! Thank you for your contribution, I know many people have been asking for this feature :)


Can't get properties from functions (Object.keys)

This snippet crashes:
let object = JSObjectRef.global.Object.object!

as global Object is a function. This function has properties though, one of them being Object.keys. There is currently no way to access such property in JavaScriptKit, as this snippet crashes too:
let keys = JSObjectRef.global.Object.object!.keys!(anotherObject)

JSFunctionRef is a JSObjectRef, so please access global.Object as a function.
e.g.
JSObjectRef.global.Object.function!.keys.function!(jsObject)

Huh, I didn't know that JSFunctionRef supports dynamic members. Thanks for clarifying!


 The bot was refusing to catch anything because it only wanted to use ultraballs on VIPs. If we have no other balls, we should be willing to use ultra balls.

Then eventually you will end up with 0 Ultraballs as well. If you came to 0 Pokeballs that's mean your pokestop/pokemon ratio is bad and need improve.

I think this will use ultraball even if there are other pokeballs when Pokemon catch rate is very low, but I'm not 100% sure about it, maybe extend condition?

The lowest available pokeball is chosen with this logic:
current_ball = ITEM_POKEBALL		             current_ball = ITEM_POKEBALL
             while items_stock[current_ball] == 0 and current_ball < maximum_ball:		             while items_stock[current_ball] == 0 and current_ball < maximum_ball:
                 current_ball += 1

Ok

Personally I would like this behaviour to be configurable through configuration.
I would like to save my ultraballs for my defined VIPs.

I run my bot with this change since the catch was reworked. Yes it will use more ultraballs the best way to deal with this is to be able to set both max_ball_non_vip and ideal_catch_rate_before_throw_non_vip

The challenge for me is that I don't get very many low level pokeballs from pokestops anymore, so I'm completely out of pokeballs that it is willing to use, but it still tries to catch pokemon and complains that there are no available balls. While it is definitely important to save some to be able to catch VIPs, if it isn't willing to use ultraballs for regular pokemon, the bot essentially stops doing anything.

Agreed wit @TheSavior
PokeBalls: 78 | GreatBalls: 0 | UltraBalls: 200

@TheSavior
current_ball = ITEM_POKEBALL
while items_stock[current_ball] == 0 and current_ball < maximum_ball:
  current_ball += 1

will this also be a fix for  #3313 ?

    maximum_ball = ITEM_ULTRABALL
     ideal_catch_rate_before_throw = 0.9 if is_vip else 0.5

I'm running it like this right now after lvl 20 you get alot more great and ultra balls with this config I'm forcing it to use more great/ultra balls and berries and also it is way more effective in catching

I think we should be able to specify minimum catch rate for every ball via config. And for berries too. Also berry must be thrown every catch attempt, not only first one, because it boost only one attempt.

I think the problems you guys are bringing up with configuring pokemon catching is valuable, but I'm not sure it is relevant to this PR. This PR simply allows us to catch pokemon when we still have balls left. Changing the logic of how we pick balls is something we should tackle separately.

@TheSavior If you fix this can you please also investigate this two issues https://github.com/PokemonGoF/PokemonGo-Bot/issues/3271 & https://github.com/PokemonGoF/PokemonGo-Bot/issues/3287 they appear after the last changes as there been so many pokeball changes and it's damaging the bot. There are many infinity loops and so on. Thank you!

@douglascamata Please review

there should be an option to reserve (lets say 10 for e.g.) ultraballs for v.i.p pokemon only

It sounds like there is more logic that needs to go into this to satisfactorily cover everyone's use cases than I am willing to work on for this right now. In the short term I can modify my config to keep more low level pokeballs.


[Elastic] Ensure base url path isn't removed when admin_forwarder is used

What does this PR do?
Creates a new function to handle joining the base url with a path to ensure that the base url path isn't removed when using the admin_forwarder function
Motivation
After enabling admin_forwarder, the check was still removing the base url path
Testing Guidelines
An overview on testing
is available in our contribution guidelines.
Versioning

[x] Bumped the check version in manifest.json
[x] Bumped the check version in datadog_checks/{integration}/__init__.py
[x] Updated CHANGELOG.md. Please use Unreleased as the date in the title
for the new section.
[ ] If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo

Additional Notes
Addresses an issue with using the admin_forwarder parameter, introduced in this PR

Hey @dixonscottr could you rebase this when you get a chance? Also the tests were ported to pytest, would you be able to create a test for this new feature? Let me know if you have any questions and thanks again!


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/1054 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit 2d8db0f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: RepositoryUnauthorizedWithSourceCommit] Git data access to 'https://github.com/opstest2/docs-archive-test-source' (branch: Release_Archive_main_2022-02-18-11-53-40) is not authorized.. It looks like this might be the first time user: v-alji submitted a pull request to this repo. Before user can run builds in a pull request, he/she can enable 'v-alji' GitHub account in Docs by visiting Docs Portal (https://ops.microsoft.com). After enabling his/her account (v-alji), close and then reopen this pull request to start another build.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Request failed with status code 503

i use your CORS from API GET and it success, but when i use for API POST it can't working. Can you solve this Rob ? Thank you before :)

But when i cek in your website it success, i little confuse now

In your second example, you have to put the URL in the first box and the POST data in the second box to get a comparable result.
Do you still experience the same problem?

Yes Rob i do nothing and my project is normal again and your jquery function is working again. Why sometimes is worked and not ?

On Heroku, 503 means that either the upstream request (i.e. the proxy destination) timed out (>=30 seconds), or that the Node app has crashed. If this happens again, just let me know, then I'll set up a monitor for errors and debug further.

The problem persists. AJAX requests return a 503 error, and https://cors-anywhere.herokuapp.com shows the following error message:

I've started collecting a log since 2015-02-13T14:48:56.902700+00:00 (8 minutes since your report), and saw only a few 503s (I've removed the request URL, host and client IP from the next result).  Thanks anyway! :)

Just got this 503 now. Will update if it fixes itself

Got the 503 error, so it's safe to say it will fix itself whenever the Node app reboots?

I'm receiving the 503 error as well

@ssavardr @jtung23
When you get the 503 error, did that happen for multiple minutes (even if you visit https://cors-anywhere.herokuapp.com/ ) ?

If yes, then there is still a bug that I need to look into.
If not: that can happen if the other server does not reply (the actual error is "Request Timeout").

Hey,
I get a 404 error when i visit the herokuapp page, "Failed to load
resource: the server responded with a status of 404 (Invalid host)"
I just tried again today and I'm not receiving the 503 error anymore. I
received the 503 error all day yesterday, 12/13/17 though.
On Thu, Dec 14, 2017 at 1:10 PM, Rob Wu<EMAIL_ADDRESS>wrote:

@ssavardr https://github.com/ssavardr @jtung23
https://github.com/jtung23
When you get the 503 error, did that happen for multiple minutes (even if
you visit https://cors-anywhere.herokuapp.com/ ) ?

If yes, then there is still a bug that I need to look into.
If not: that can happen if the other server does not reply (the
actual error is "Request Timeout").

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Rob--W/cors-anywhere/issues/20#issuecomment-351836703,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPnzDTbTpY_R2_mpK0IDwwaG3_R54rpks5tAY6xgaJpZM4DUgUr
.

I get a 404 error when i visit the herokuapp page, "Failed to load resource: the server responded with a status of 404 (Invalid host)"

What URL are you visiting? I am not getting 404 at all. Did you add some extra characters after the URL that I provided?

It was the same link but now I'm not receiving the error anymore. So the
503 error I was receiving yesterday was due to a request timeout?
On Thu, Dec 14, 2017 at 1:18 PM, Rob Wu<EMAIL_ADDRESS>wrote:

I get a 404 error when i visit the herokuapp page, "Failed to load
resource: the server responded with a status of 404 (Invalid host)"
What URL are you visiting? I am not getting 404 at all. Did you add some
extra characters after the URL that I provided?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Rob--W/cors-anywhere/issues/20#issuecomment-351838796,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPnzG5DZGkFP3BYSyyoWtoJNDAQQnOuks5tAZC5gaJpZM4DUgUr
.

It was the same link but now I'm not receiving the error anymore. So the 503 error I was receiving yesterday was due to a request timeout?

Likely, but I don't know for sure since I don't keep persistent logs. I can only debug in real-time.

any recommendations for the front end to avoid the errors in the future?
On Thu, Dec 14, 2017 at 2:01 PM, Rob Wu<EMAIL_ADDRESS>wrote:

It was the same link but now I'm not receiving the error anymore. So the
503 error I was receiving yesterday was due to a request timeout?
Likely, but I don't know for sure since I don't keep persistent logs. I
can only debug in real-time.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Rob--W/cors-anywhere/issues/20#issuecomment-351849464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPnzMWUo_2rz6KKFpd57zVSoj0DA0hLks5tAZrQgaJpZM4DUgUr
.

any recommendations for the front end to avoid the errors in the future?

Be prepared to accept errors.
Or simply don't try to connect to servers which are known to time out responses.
Or, if you believe that it is an issue with CORS Anywhere, check whether you get 503 when you visit https://cors-anywhere.herokuapp.com/ and if so notify me so I can look into it.

Sounds good, thanks Rob! I'll be on the look out.
Cheers
On Thu, Dec 14, 2017 at 2:11 PM, Rob Wu<EMAIL_ADDRESS>wrote:

any recommendations for the front end to avoid the errors in the future?
Be prepared to accept errors.
Or simply don't try to connect to servers which are known to time out
responses.
Or, if you believe that it is an issue with CORS Anywhere, check whether
you get 503 when you visit https://cors-anywhere.herokuapp.com/ and if so
notify me so I can look into it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Rob--W/cors-anywhere/issues/20#issuecomment-351851815,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AcPnzEm0c1yhzlaKe3uiOMeVfxgWLuBdks5tAZ0hgaJpZM4DUgUr
.

Getting 503 on https://cors-anywhere.herokuapp.com/

I am getting 503 on https://cors-anywhere.herokuapp.com/

Restarted.

Yeah I can visit the site fine, but using in the app yields a 503. Thank you Rob

Ah she's working now. Thank you!!

Thank you! working on my end as well

Hi Rob,
The cors anywhere proxy is showing a 503 error at the moment. Can you please fix it?  Thanks

I getting this error POST https://first-react-app1.herokuapp.com/login/server 503 (Service Unavailable)
Error: Request failed with status code 503
plz solve this error as soon as possible thanks

I don't see any actionable task here, so I'm closing this issue.


[Bug]: Can't configure an SMTP provider with no authentication

Preflight Checklist

[X] I could not find a solution in the documentation, the existing issues or discussions
[ ] I have joined the ZITADEL chat

Environment
Self-hosted
Version
Tested on 2.55.1, 2.56.0 and development
Database
CockroachDB
Database Version
No response
Describe the problem caused by this bug
Generic SMTP without authentication is not working anymore since I updated Zitadel to 2.55.1 version.
It seems that this feature is still available and  handled by smtpAuth function when username and password are empty: https://github.com/zitadel/zitadel/blob/d90db6693c2922b909938e087d0a14ad10998fcd/internal/notification/channels/smtp/channel.go#L151C2-L153C3
But there is 2 issues here (It seems to be introduced by this PR):

Web interface does not allow me to create a configuration with empty user and password (I don't know if it worked before)

From API, It creates an SMTP config with a null password instead of an empty password which crash zitadel:

 payload = json.dumps(
        {
            "senderAddress"<EMAIL_ADDRESS>            "senderName": "ZITADEL",
            "tls": True,
            "host": "relay.example.com:25",
            "user": "",
            "password": "",
            "replyToAddress": "",
            "description": "provider description",
        }
    )

          id         |         creation_date         |          change_date          | sequence |   resource_owner   |    instance_id     | tls |   sender_address    | sender_name | reply_to_address |         host         | username |                                             password                                             | state |     description
---------------------+-------------------------------+-------------------------------+----------+--------------------+--------------------+-----+---------------------+-------------+------------------+----------------------+----------+--------------------------------------------------------------------------------------------------+-------+-----------------------
<PHONE_NUMBER>49954883 | 2024-07-19 15:39:06.58709+00  | 2024-07-19 15:39:06.58709+00  |      118 |<PHONE_NUMBER>99156547 |<PHONE_NUMBER>99156547 |  t  |<EMAIL_ADDRESS>  | ZITADEL     |                  | relay.example.com:25 |          | NULL                                                                                |     3 | provider description

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0xb9a0cf]

goroutine 5132 [running]:
github.com/zitadel/zitadel/internal/crypto.checkEncryptionAlgorithm(0x0, {0x56f7e10, 0xc002296990})
	/home/runner/work/zitadel/zitadel/internal/crypto/crypto.go:83 +0x2f
github.com/zitadel/zitadel/internal/crypto.DecryptString(0x0, {0x56f7e10, 0xc002296990})
	/home/runner/work/zitadel/zitadel/internal/crypto/crypto.go:76 +0x25
github.com/zitadel/zitadel/internal/command.(*Commands).TestSMTPConfigById(0xc00216b400, {0x56edbb8?, 0xc003e60ed0?}, {0xc003e2a7f8?, 0x2fc9d80?}, {0xc003e2a600?, 0xc003e6e900?}, {0xc003e12f90, 0x23})
	/home/runner/work/zitadel/zitadel/internal/command/smtp.go:345 +0x13b
github.com/zitadel/zitadel/internal/api/grpc/admin.(*Server).TestSMTPConfigById(0xc002a33e80, {0x56edbb8, 0xc003e60ed0}, 0xc003ed0ff0)
	/home/runner/work/zitadel/zitadel/internal/api/grpc/admin/smtp.go:134 +0xa6
github.com/zitadel/zitadel/pkg/grpc/admin._AdminService_TestSMTPConfigById_Handler.func1({0x56edbb8?, 0xc003e60ed0?}, {0x2e2bb60?, 0xc003ed0ff0?})
[...]

I've just pushed a commit on my Zitadel repo to fix the null password issue. It is working well for my needs but web interface still need to be fixed. And SMTP configuration updates should also handle empty password, and  I'm not sure how you want to handle the difference between the password is updated for an empty password and the password is not updated.
I'd be glad to continue working on this issue and create a PR if it seems relevant!
To reproduce
    url = "https://MYINSTANCE/admin/v1/smtp"

    payload = json.dumps(
        {
            "senderAddress"<EMAIL_ADDRESS>            "senderName": "ZITADEL",
            "tls": True,
            "host": "relay.example.com:25",
            "user": "",
            "password": "",
            "replyToAddress": "",
            "description": "provider description",
        }
    )
    headers = {
        "Content-Type": "application/json",
        "Accept": "application/json",
        "Authorization": "Bearer FIXME",
    }

    response = requests.request(
        "POST", url, headers=headers, data=payload, verify=False
    )

    print(response.text)

Screenshots
No response
Expected behavior
No response
Operating System
No response
Relevant Configuration
No response
Additional Context
No response

Encountering this issue as well for an authless SMTP relay

Same issue with newest version v2.63.1. If the TLS flag is set to false the UI but also API still requires a username/password. If you set a random username and no password it works to save the configuration and doesn't use TLS.

The required validator for user and password will be removed. Moved this to bugs/small issues, estimated about an hour


Reduce Dependencies

With the recent aligner addition, new dependencies (rust-bio) can burden the user to install/compile biobear. Obviously, we'd like to avoid that if possible. See https://github.com/wheretrue/biobear/issues/106.

I think this is resolved at e.g. previously this took ~11 mins on ubuntu and is down to ~20 secs to install from pip:

Prior: https://github.com/wheretrue/biobear/actions/runs/8366079345/job/22905443202
Today: https://github.com/wheretrue/biobear/actions/runs/8380346075/job/22949360752

@abearab, not sure if you might be able to see if that helps on your end?

@tshauck I could also install the new version much faster now! Great job 💪

@abearab Thanks for checking! Closing this issue.


Bug | Profile > About | Responsiveness

The text in the upload picture section goes outside the screen width. It should be responsive.
Screenshot

Proposed Solution
CSS Adjustments:

Use media queries to adjust the font size and layout for smaller screens.
Ensure the container of the text has appropriate padding and margins.
Implement word wrapping to ensure long text does not overflow.

Responsive Design Implementation:

Modify the CSS to use flexible units (like percentages or viewport units) instead of fixed units (like pixels).
Adjust the flexbox/grid properties to ensure elements resize and rearrange appropriately.

Testing:

Test the changes on various screen sizes using browser developer tools.
Ensure that the text is readable and the layout is visually appealing on both mobile and desktop devices.

Sounds good.

Sure then, will start working on the issue!

@parthzile - Please share an update on the issue every 24 - 48 hours. Otherwise, the issue will be unassigned.

@aadeshkulkarni kindly assign this issue to me

Sure, assigned it to you.

@aadeshkulkarni Please Review.


Translated to Slovenian

Although it is quite hard to translate to Slovenian, because the language is quite old, I still gave it my best, but if I'll think of better translations I'll of course fix them!
This is my first time doing this so if I'm doing anything wrong please tell me, thanks.

Thanks! RimWorld itself isn't translated to Slovenian so I don't know whether it will be useful for anybody though.


Ability to see the previous notices, and read the entire notice on the terminal itself.

@somyaup, start by reading and understanding the code, make it more readable wherever you find need, after that, this issue seems the easiest to start with

Not required.


3-to-4 project converter adds extra line for move_and_slide

Godot version
4.0-beta1
System information
Linux (Pop!_OS 20.04)
Issue description
When the 3-to-4 project converter converts a call to KinematicBody2D.move_and_slide(), it adds a line containing only "velocity", resulting in a warning message: "Standalone expression (the line has no effect)."
This does not affect the proper execution of the code.
Steps to reproduce
Convert and run the minimal reproduction project attached below. View the script main.gd and the Errors tab.
Minimal reproduction project
project_converter_3_to_4-bug1.zip

This should look like
set_velocity(direction.normalized() * speed)
move_and_slide() only without the "velocity" line, am I right?

Hi, I made PR#66233 with the solution. Could you look at it?


PROPOSAL: Reorganize docs

Background
For #1513 - Change "Advanced" to "Developer Guide"
Change List

document index and some pages.

Our top-level categories are still confusing. My suggestions:

Fold custom layer guide into a subsection of developer guide, and also make it clear that any future deep-dive articles must go into developer guide instead of a top-level category.
Split deckgl-test-utils into articles and api references. Articles go into a subsection of developer guide called testing. API reference go into a top-level deckgl-test-utils API
Rename the current API-reference to deck.gl API and redirect old links here

Updated per comments


Many Angular carbon UI components do not export properly into CodeSandBox

https://www.loom.com/share/c85bf4e3851e404abf7b665f2df6b355
Issue happening with items:
Accordion
Bread Crumb
Button
Checkbox
Code Snippet
Content Switcher
Dropdown
Grid
Inline Loading
Link
Loading
Progress Indicator
Search
Tag
Toggle

Thanks for reporting this, Dirk!
It's because fragments have the same names as existing components so the builder creates a clashing module name 🤔
@MoizMasud can you look into this?


Add Christmas Logo

Summary
Added the christmas logo
Test Plan
Open staging and look at it
Issues
N/A

Yeah thats a good idea, will do


includes card codes in download/export formats

Unless I'm missing something, neither of the currently available export/download formats for decks (plain text/OCTGN) allow to reference cards using their thronesdb-json-data, e.g., 04081.
What do you think of adding the code to either of the currently available download formats?
Alternatively, one could think of adding a new download format just for this, e.g., JSON, where all available card info in JSON format are inlined.
With many thanks for ThronesDB!

I think the best way to do that is to use the API: http://thronesdb.local/app_dev.php/api/doc#get--api-public-decklist-{decklist_id}.{_format}


Add details about Stub priority handling and default level to fix 1574

Addresses https://github.com/wiremock/wiremock/issues/1574 by adding a paragraph about how priorities are handled.

Thanks!


use mount_nix_store to avoid running out of disk on Semaphore CI

Use mount_nix_store to avoid running out of space on Semaphore CI.
Here are a couple of example failing builds :x: with the disk issue:

https://channable.semaphoreci.com/jobs/46cb6181-b263-45fc-b9df-4d1a06e5d495#L1213-L1217
https://channable.semaphoreci.com/workflows/9a9ebc9e-0e13-446a-8189-910fc48dd4ed

After adding the changes on this PR, the build goes okay :heavy_check_mark::

https://channable.semaphoreci.com/workflows/1b962226-27c3-409e-9e87-d85e1c15bf63?pipeline_id=abbab2e8-7471-49bd-b608-c5f30a0c05bc

Thanks to Bert Peters (@bertptrs) for suggesting the fix!

@bertptrs Thanks for the review.
I have amended the PR in light of your suggestions.  Can you please take a look again?

@bertptrs @fatho Thanks for the reviews.
@OpsBotPrime merge

Pull request approved for merge by @rudymatela, rebasing now.

Rebased as 37bc6c49ea6463cb210a1f4d61e2d6cd634a9914, waiting for CI …


OSD-4362 add aws permission step to quickstart

readme: add aws permission step to quickstart

/assign @jharrington22

@jharrington22 - is it worth to make this part of the moactl init command, or another sub-command within the tool?

@vkareh yes I've got a card to check for this specific permission automatically and inform the user if its missing

/retest

/cc @sfortner-RH

@RiRa12621 could you address @sfortner-RH comments and we can merge? I think we had some trouble with tide last time around so as soon as its done I'll try to add the labels. Thanks.

will do
not for @sfortner-RH : github supports suggestions now that the PR creator can directly merge into their branch, making this even quicker that copy pasting the comments :-)

Also since the initial review the name changed. So that will need to be updated if it hasn't already :) Victor already made these changes in the code.
Amazon Red Hat OpenShift --> Red Hat OpenShift Service on AWS
moactl --> rosa

/retest

incorporated feedback
changed name from moactl to rosa
rebase

/close
seems obsolete by now, sorry for letting that slip for so long

Thanks @RiRa12621


python: fix 'nproc not found' on darwin

Motivation for this change
Things done

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

[ ] NixOS
[x] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Fixes: 58b862b75 ('darwin purity: pythonPackages.pandas')
Cc: @pikajude

as I am trying to unify the expressions of the Python interpreters, please check whether this is also needed for the other interpreters.

I've grep'ed python 3.5.2 for hw.ncpu, sysctl and nproc and see nothing relevant

Yes, new implementation uses sysctl call instead: https://hg.python.org/cpython/file/tip/Modules/posixmodule.c#l10944

Great, thanks for checking.

should this go into staging first? @copumpkin

I think this case can go directly.


Implicit solver

Add an implicit solver as an option for integrators.

Wrote the integrator. Some next steps:

Clean up the memory allocations into one pointer (e.g. WORK) [a la LAPACK routines]. Should be a separate function as the intermediate work steps are useful for testing.
Run some tests with the LLG - this is what really needs to work.
Try dropping this into the simulation object
Document the mathematics for the test

Addressed in #19


♻ Move from Optional[X] to Union[X, None] for internal utils

♻ Move from Optional[X] to Union[X, None] for internal utils

Simple answer: tiangolo prefers Union

@josegonzalez ref: https://mobile.twitter.com/tiangolo/status/1531583406651363328


feat: add support for specifying the contract name

This PR brings two main changes:

We derive the contract file name from the .tree file name.
We derive the contract name from the identifier at the tree's root node.

Closes #3
Also, this PR adds integration tests.

Codecov Report

Merging #27 (471da34) into main (5904c07) will increase coverage by 0.3%.
The diff coverage is 58.6%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files

Files Changed
Coverage Δ

src/cli.rs
0.0% <0.0%> (ø)

src/main.rs
0.0% <0.0%> (ø)

src/scaffold/ast.rs
92.3% <ø> (ø)

src/scaffold/error.rs
81.6% <ø> (ø)

src/scaffold/span.rs
41.8% <ø> (ø)

src/scaffold/utils.rs
93.1% <ø> (ø)

src/scaffold/parser.rs
79.1% <90.9%> (ø)

src/scaffold/tokenizer.rs
93.4% <95.0%> (ø)

src/scaffold/emitter.rs
99.1% <100.0%> (ø)

src/scaffold/mod.rs
100.0% <100.0%> (ø)

... and 2 more


For a 0.20.9 release

Codecov Report

Merging #801 (8aa256f) into master (7ed607a) will not change coverage.
The diff coverage is 90.90%.

:exclamation: Current head 8aa256f differs from pull request most recent head f6c706b. Consider uploading reports for the commit f6c706b to get more accurate results

@@           Coverage Diff           @@
##           master     #801   +/-   ##
=======================================
  Coverage   85.93%   85.93%
=======================================
  Files          36       36
  Lines        3470     3470
=======================================
  Hits         2982     2982
  Misses        488      488

Impacted Files
Coverage Δ

src/MLJBase.jl
60.00% <ø> (ø)

src/composition/learning_networks/nodes.jl
71.24% <ø> (ø)

src/sources.jl
86.95% <ø> (ø)

src/machines.jl
87.38% <87.50%> (ø)

src/composition/learning_networks/machines.jl
91.83% <100.00%> (ø)

src/composition/models/methods.jl
100.00% <100.00%> (ø)

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 7ed607a...f6c706b. Read the comment docs.


Building the code with Xcode.

I have cloned Klaus and tried to build it with my xcode( xcode 6 beta-5).
It somehow fails due to file "arm_neon.h".
error message says that "NEON support not enabled".
Any clue how to solve this prob?
On which Xcode does it succeed to build?

MUCH faster! :zap:


Port away from requests -> urllib3 for thread-safety. Resolves #144.

This is not yet ready to merge as I running some integration level tests with our software. The problems I reported in #144 were hard to reproduce, so I need to do a few runs before there is a reasonable chance that I can conclude all is well.
In the meantime, I would appreciate feedback.
It is worth noticing that the fix involves porting to urllib3>=1.25; that is because it is this version which introduces certificate checking as an option.
I also note in passing that there is a Python3.7 compatibility issue which make it hard for me to get a clean run:
/usr/lib/python3.7/ast.py:35: in parse
    return compile(source, filename, mode, PyCF_ONLY_AST)
E     File "/main/srhaque/kdedev/python-consul/tests/test_aio.py", line 77
E       fut = asyncio.async(put(), loop=loop)
E                         ^
E   SyntaxError: invalid syntax

so I have ignored that for now (these also show up as reserved word warning in 3.6).

After quite a bit of testing, the code seems to be behaving as well as the previous code, with no obvious regressions. The Travis failures seem to be caused by an issue running flake8 with what seems to be a malformed command, though I cannot work out what the problem is. So, plus or minus some help getting that sorted out, I believe the code is ready to merge.
[Sadly, this does not resolve my original problem, but I'll open a new issue for that]

The Travis issue turned out to be a W504 against setup.py which is now disabled. I added Python3.7 because that is what I need (including to test with), and fixed the syntax changes that required too.


Configuration documentation suggestion

This is obviously a complicated Lovelace card with lots and lots of options, so making it simple for a first-time user is a very difficult task.
I have been using HA for over 5 years and have installed dozens of HACS customisations. They mostly follow a convention in documenting their config, which is to provide a minimal example straight-up and to describe the top-level elements first. Then for each top-level element that has its own options, there is a section further down, that covers each of these. These can all be accessed from the first "screen" of the README by their clickable links to the sections. Apexcharts cards is a good example.
The config docs for this repo go straight into detailed explanation of (just) the cameras top-level setting. Sprinkled further down appears a few other top-level settings, like menu but you have to scroll and seek to find these. There is no clickable table of contents for the sections, so scrolling is the only way to get there.
My suggestion is to add a table of contents at the top the README and to document all the top-level config vars first and then go into details for each one below that.
Lastly, is it possible to add a title to appear at the top of the card, independent of the cameras?

Thanks @nickdos. A few notes:

Github already offers table of contents feature  that can be used to move around vs scrolling:

There's no special treatment for cameras really, all options are listed in order. There is just a lot of them, and cameras is the only required argument.

Overall, the main README is just much too long now and I think a multi-page setup might need to considered. For now, perhaps this issue could be to add:

A small (not exactly minimal, but basic) example config.
A table that contains the main sections (e.g. cameras, menu, dimensions, etc) and links to each of their detailed sections.

Sound right?

Lastly, is it possible to add a title to appear at the top of the card, independent of the cameras?

i.e. a static title that is always shown above the card? No, this is not currently possible, although you could presumably do it with a markup card outside of the frigate card.

Thanks for the thoughtful reply.
Dang, I never noticed that builtin table of contents (been too long on GH so don't notice new features) - thanks for pointing out. That was my main frustrating as I was scrolling up and down and up and down the page, which got old quickly.
I'm probably not an exception in not noticing the builtin ToC so adding a small inline ToC for the main sections, as you suggested, is a good idea.
And yes, a minimal config example would've helped me too. I didn't push that in my original comment, as you provide a very good UI mode for config and I mostly used that to good effect.
I did get stumped when trying to configure WebRTC and/or go2trc (via Frigate) and gave up. The examples were numerous and complete but had the warning "don't copy them verbatim as not all options are required" (or words to that effect). But I just couldn't work out which options I did need vs I didn't need. I also couldn't work out the values I needed in some cases, so a real example would've helped.
I think many people will be coming from a few common setups, so having a few preferred/opinionated config examples for those groups would be useful. In my case, give me an example for Frigate using the webrtc card and associated webrtc settings and indicate which options I need to have set in Frigate (go2rtc with webrtc, etc) to match that.
Lastly, I found the "overridable" green ticks very confusing - is the user overriding a default value with "this" option or is there another option further down that might override "this" setting? E.g. the very first option "camera-entity" is overridable and not marked as required, so I'd assume I could leave that empty (i.e. I can choose not to override with a value of my own?). But it errors, so this assumption appears to be wrong. So if it is the other interpretation, then what other options might be able to override the current setting? Without knowing this, marking it as overridable is kind-of pointless (my opinion). There might be many other options that do override it but I feel you should link to all of them - or not bother to have the overridable flag at all.
I did a quick scan and every option for the first 14 screen lengths (hitting page down) had overridable set as true. I got to thinking that it should be assumed any setting might be overridable and maybe it should be the other way around - only indicate when a setting overrides another setting and link back to that "higher" setting in those cases. That would seem to be a more useful way of doing it, assuming I have guessed correctly on how to interpret "overridable" (I'm still not sure).
An example:

Why is one overridable and the other not? How does that flag change the way I interpret these settings? What action or check do I need to follow-up with, when it set vs not set?
Apologies if this comes across as over critical, it's not my intention. I think this is a fantastic card and was hoping my observations might help improve access for other users. As a software developer I struggled with some aspects, so I'm guessing people from a non-technical background would struggle even more.

I'm probably not an exception in not noticing the builtin ToC so adding a small inline ToC for the main sections, as you suggested, is a good idea.

Sounds good.

And yes, a minimal config example would've helped me too.

Sounds good.

I did get stumped when trying to configure WebRTC and/or go2trc (via Frigate) and gave up. The examples were numerous and complete but had the warning "don't copy them verbatim as not all options are required" (or words to that effect). But I just couldn't work out which options I did need vs I didn't need. I also couldn't work out the values I needed in some cases, so a real example would've helped.

I think many people will be coming from a few common setups, so having a few preferred/opinionated config examples for those groups would be useful. In my case, give me an example for Frigate using the webrtc card and associated webrtc settings and indicate which options I need to have set in Frigate (go2rtc with webrtc, etc) to match that.

I interpret this as: Please add a simple example that uses the go2rtc live provider, and separately the webrtc_card live provider. The latter is already provided here, but I think really the minimal config (mentioned above) could feature early in the documentation, and could provide a few "common scenarios".

Why is one overridable and the other not? How does that flag change the way I interpret these settings? What action or check do I need to follow-up with, when it set vs not set?

I struggle a little with this one. Draggable is a property of the entire carousel, so cannot reasonably change depending on a "slide" within the carousel, whereas Zoomable is a property of an individual slide. I don't expect the user to understand the "why" behind these distinctions, but without some sort of indicator it could be quite confusing why the override is not working. Realistically though, most people are not going to be using overrides, so stripping out the checks/xs entirely is always a possibility if we think it improves clarity for the majority (vs occasional head-scratching for the minority).

Apologies if this comes across as over critical, it's not my intention. I think this is a fantastic card and was hoping my observations might help improve access for other users. As a software developer I struggled with some aspects, so I'm guessing people from a non-technical background would struggle even more.

Not at all, I understand and appreciate very much. When one is in the trenches, it's very hard to see what is readable vs not. I'll go one step further, if you fancied putting together a PR with some of the ideas here, that would be fantastic -- if not the contribution of the ideas is still very much appreciated and I can work on a PR at some point in the future.

I'm going to consider this closed. I've completely overhauled the documentation into something approaching usable:
https://dermotduffy.github.io/frigate-hass-card
I hope you'll find the new documentation significantly improved, with a bunch of your tips/advice incorporated. Thanks for the quality issue report -- knowing that a HA-experienced developer was struggling to use it was a significant motivator.


add icons for abaplint.json and .abapgit.xml

https://code.visualstudio.com/api/extension-guides/icon-theme
perhaps as part of the abap highlighting extension instead?

smaller scale for preview

in fact there is regular abap file too ... so may be not that easy
P.S. is there "official" abaplint image ?

hmm images are re-sampled on upload for some unknown reason ... attaching as zip
code-icons.zip

personally like 3.2 with probably different colors for abap file and .abapgit.xml. The rest would be cluttered in so small res.
The best is 2.1 but proportions are unfortunate. And 5.3 is cool imho but also in a larger res

Also nice option with mirroring ... mind the hidden arrows in the high res -the're cool ;)

End of creativity. :)

for .abapgit.xml, I guess this one will work

for *.abap, one of:

for abaplint.json see https://github.com/abaplint/img, it might need a different colour to be visible

attached.
svg-icons.zip

hmm, looks like it should go into https://github.com/jesseweed/seti-ui to go into the default theme

I dont think this is possible as of now, closing


Change ROW representation on the client to avoid fake row field names

Fixes https://github.com/prestosql/presto/issues/4587

This changes CLI, but also impacts ResultSet#getObject. It's no longer a Map.
cc @electrum

The one of remianing failures is
java.lang.AssertionError:
Expecting:
  <{tag=0, field0=180, field1=null, field2=null}>
to be an instance of:
  <io.prestosql.client.Row>
but was instance of:
  <io.prestosql.jdbc.$internal.client.Row>

before i go into fixing this, let's agree on the direction.
@electrum @martint @kokosing please take a look.

Discussed offline with @electrum -- seems fine as a general direction

PTAL

(no longer "WIP")

@losipiuk thanks for the review. I split this into 3 commits now. Accidentally rebased on current master, sorry.


Setting up a basic Firewall on the Cilium side in K8s Cluster CAPI

/kind bug
What steps did you take and what happened:
I did not find a basic Firewall setup using Cilium in this repository.
What did you expect to happen:
I expect that in the end I will be able to close the K8s Cluster created using CAPI using Cilium under the basic Firewall.
Anything else you would like to add:
I would like to get the basic Firewall configuration for Cilium in your version from CAPI, or better yet, a configuration file, because by default we do not close the Firewall when installing K8s Cluster using the CAPI solution.
Environment:

cluster-api-provider-hetzner version: v1.0.0-beta.33
Kubernetes version: (use kubectl version): v1.28.7
OS (e.g. from /etc/os-release): ubuntu-22.04

@BigBrather The reason for this is that the firewall depends heavily on your setup. So these kinds of things are out of scope, see also: https://github.com/syself/cluster-api-provider-hetzner?tab=readme-ov-file#-clarifying-scope

I need a basic setup for the Cilium Firewall, not for my installation.

@batistein I need a basic setup for the Cilium Firewall, not for my installation. But, if CAPI does not have a basic configuration for the CAPI firewall. I'll close this issue.

@BigBrather Configuring a Cilium Firewall is explained in the docs of Cilium: https://docs.cilium.io/en/latest/security/host-firewall/
If you have question about that, please ask at their preferred community channel (afaik Slack).
I close this issue, since I think it is not related to caph.
Feel free to re-open it if you disagree, or open a new issue, if caph is not working for you (or something is not easy to understand).

@guettli I disagree that I didn't get an answer like this (does CAPH do basic setup for Cilium when first deployed):

This is not the case; when deployed, we receive an open K8s Cluster firewall, where FIreWall is configured separately according to the Cilium documentation.

Thanks for your answer.


Update docs to explain tags can not process white spaces

Fixes #82. @AY2324S2-CS2103T-W12-2/developers can someone help check whitespaces don't create unrecoverable states in the app.

Fixes #82. @AY2324S2-CS2103T-W12-2/developers can someone help check whitespaces don't create unrecoverable states in the app.

what does unrecoverable states mean

As of now, tag names only support alphanumeric characters, so underscore doesn't work

As of now, tag names only support alphanumeric characters, so underscore doesn't work

@yadunut would it be right to change this to just say whitespaces are not accepted then?

I think if you change it to "You can use capitalization such as camelCasing" to distinguish different words in a tag and then write a line in the glossary about camelCasing or sth


Remove sloth_window from alert expression

Below is a graph of the alert expression for an ongoing alert we had

it appears like the sloth_window label is part of the generated alert, and that the label changes value depending on whether the quick or slow alerting conditions are triggered. This can cause a lot of noise with resolving and re-firing alerts. So in the example above, essentially the same alert fired about 6 times over an hour.
I would propose to change the alerting expression a bit to actively drop the sloth_window label, to avoid excessive firings of the same alert. A proposal for a new alerting expression would be something like
(
  max(
    slo:sli_error:ratio_rate5m{sloth_id="a",sloth_service="b",sloth_slo="c"} > (13.44 * 0.01)
  ) without (sloth_window) and
  max(
    slo:sli_error:ratio_rate1h{sloth_id="a",sloth_service="b",sloth_slo="c"} > (13.44 * 0.01)
  ) without (sloth_window)
) or
(
  max(
     slo:sli_error:ratio_rate30m{sloth_id="a",sloth_service="b",sloth_slo="c"} > (3.5 * 0.01)
  ) without (sloth_window) and
  max(
     slo:sli_error:ratio_rate6h{sloth_id="a",sloth_service="b",sloth_slo="c"} > (3.5 * 0.01)
  ) without (sloth_window)
)

In the expression, max should never aggregate multiple metrics. But even if it did it would take the highest error ratio, which seems like a good pessimistic approach.
Would be happy to try to contribute a PR if you think the change makes sense

Hi @slok,
Any interest in this if I do a PR, or is maybe the issue here not well enough explained for you?

Wish to have this fix, too.


Tunneling

Hi! I am wondering if I can use this library to model the ssh command: ssh -N -p 2222 -L6116:<target-host>:5432 tunnel@localhost

forwarding is not implemented (yet)

@Miyoshi-Ryota is this something you'd consider for this crate?


Workflow series

I want to do the following with one button press:
Take your song (vocals, instrumentals, all of it, the original).
Run it through UVR5 with the (MDX) Kim Vocals 2 model.
Take the vocal output and run THAT through the (VR) de-reverb model.
Take the output from that and run it through the (VR) aggressive de-echo model.
Take the output from that and run it through a (VR) karaoke model.
The way I imagine I could would be to have a sort of processing order workflow and then be able to re-use it over and over.
Also, this would allow multiple inputs to be lined up. I can get ten songs and have UVR5 go through them one by on until they're all finished with a single button press.


refactor: replace deprecated String.prototype.substr()

String.prototype.substr() is deprecated so we replace it with String.prototype.slice() which works similarily but isn't deprecated.
.substr() probably isn't going away anytime soon but the change is trivial so it doesn't hurt to do it.

Why not just use String.prototype.substring ? https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/substring

substring() and slice() are very similar but differ in some regards.
I personally prefer slice() over substring() as it's imo stricter. There is no "re-sorting" of arguments. It's also shorter and has an array counterpart which works the same way which makes it more intuative. slice() is also slightly faster but that probably doesn't matter that much.