I001 conflict with autopep8 / pycodestyle's E302

See this minimal reproduction:
from typing import TypedDict, type_check_only

@type_check_only
class GCInputs(TypedDict):
  pass

Ref: https://peps.python.org/pep-0008/#blank-lines and https://pycodestyle.pycqa.org/en/latest/intro.html?highlight=E302#error-codes
I001 wants 1 line, E302 wants 2.
I use the default lines-after-imports (so it should be -1)
Is there a way to ignore just lines-after-imports without ignoring the entirety of I001 ?
I'd like to either let the non-ruff formatter take care of it. Or have Ruff implement E302 (https://github.com/charliermarsh/ruff/issues/2402) so it's aware of it when linting imports.
Temporary workaround, add a dangling #. May as well link to this issue while I'm at it:
from typing import TypedDict, type_check_only

# https://github.com/charliermarsh/ruff/issues/3720

@type_check_only
class GCInputs(TypedDict):
  pass

Just trying to make sure I understand -- if I run Ruff over that file, it leaves the import as it is, and pycodestyle doesn't complain about it either. Is that not two lines, between the import and the function definition? What am I overlooking? :)

For me, ruff (0.0.259) formats
from typing import TypedDict, type_check_only  # Import block is un-sorted or un-formatted Ruff(I001)

@type_check_only
class GCInputs(TypedDict):
  pass

to
from typing import TypedDict, type_check_only

@type_check_only
class GCInputs(TypedDict):
  pass

Oh, you know what, it's a .pyi file, indeed it doesn't seem to happen in a .py file.
Also, here's my complete ruff config pyproject.toml:
# https://beta.ruff.rs/docs/configuration
[tool.ruff]
line-length = 120
select = ["ALL"]
target-version = "py38"
# https://beta.ruff.rs/docs/rules
ignore = [
  ###
  # Not needed or wanted
  ###
  "D1", # pydocstyle Missing doctring
  "D401", # pydocstyle: non-imperative-mood
  "EM", # flake8-errmsg
  "FBT", # flake8-boolean-trap
  "INP", # flake8-no-pep420
  "ISC003", # flake8-implicit-str-concat: explicit-string-concatenation
  "TCH002", # flake8-type-checking: typing-only-third-party-import
  # Short messages are still considered "long" messages
  "TRY003", # tryceratops : raise-vanilla-args
  # Don't remove commented code, also too inconsistant
  "ERA001", # eradicate: commented-out-code
  # contextlib.suppress is roughly 3x slower than try/except
  "SIM105", # flake8-simplify: use-contextlib-suppress
  # Checked by type-checker (pyright)
  "ANN", # flake-annotations
  "TCH003", # flake8-type-checking: typing-only-standard-library-import
  # We want D213: multi-line-summary-second-line and D211: no-blank-line-before-class
  "D203", # pydocstyle: one-blank-line-before-class
  "D212", # pydocstyle: multi-line-summary-first-line
]

# https://beta.ruff.rs/docs/settings/#flake8-implicit-str-concat
[tool.ruff.flake8-implicit-str-concat]
allow-multiline = false

# https://beta.ruff.rs/docs/settings/#isort
[tool.ruff.isort]
combine-as-imports = true
required-imports = ["from __future__ import annotations"] # Safer with Python 3.8 and 3.9
split-on-trailing-comma = false

# https://beta.ruff.rs/docs/settings/#mccabe
[tool.ruff.mccabe]
# Hard limit, arbitrary to 4 bytes
max-complexity = 31
# Arbitrary to 2 bytes, same as SonarLint
# max-complexity = 15

[tool.ruff.pylint]
# Arbitrary to 1 byte, same as SonarLint
max-args = 7
# At least same as max-complexity
max-branches = 15

# https://github.com/hhatto/autopep8#usage
# https://github.com/hhatto/autopep8#more-advanced-usage
[tool.autopep8]
aggressive = 3
ignore = [
  # TODO Suggest "multi-line method invocation style" to Ruff.
  # https://github.com/charliermarsh/ruff/issues/3713
  "E124", # Closing bracket may not match multi-line method invocation style (enforced by add-trailing-comma)
  "E402", # Allow imports at the bottom of file
  "E70", # Allow ... on same line as def
  "W503", # Linebreak before binary operator
]
max_line_length = 120
recursive = true

Ahhh yes the rules are slightly different for .pyi files -- but I think we changed that to match Black's behavior...

Not sure either (other than allowing to disable lines-after-imports, since the non-Ruff formatter already takes care of it anyway, which you may or may not want to do). This was already an issue with AutoPep8 and isort anyway, so nothing new for me, but I figured since Ruff has full control over its own rules and checks, maybe something could be done.

That is the standard for .pyi files, that is what typeshed use.

Yeah unfortunately I think this has to be marked as wontfix. I understand the inconvenience but it's tough to retain compatibility when the tools conflict.

I understand, thanks for considering!
Black is great for stub files, because they don't have any logic. But I don't like it for regular python code because it too often makes some code, especially method-chainign style, less consistent and less readable, without any control. (personal preference y'know, that's why there's less opinionated options 😛)
Granted I also don't particularly care about E302 either, disabling it could lead to inconsistent 1 vs 2 lines above classes, but it still removes extra lines (3+, 2+ above non-classes).
Since this conflict only affects .pyi files starting with both imports and classes, and that adding stubs to a non-stub, non-library project is pretty rare (either I'm patching a stub until it's fixed/published upstream, or I'm vendoring something really niche). The dangling comment after imports is a perfectly acceptable solution for me 😄

Thanks for your understanding and for bearing with me as we talked through this :)


[move-function-value] Improve support for uses that are due to binding a moved value to a different value.

Most of the tests that I wrote used full on non-consuming/consuming uses rather
than initialization of a variable. This commit fixes that hole in the model and
adds some tests that exercise this behavior. Address moves did not have this
problem due to different codegen.

@swift-ci test


Don't seem to be receiving logs in our Splunk Cloud instance

What happened:
Our logs don't show up in our Splunk Cloud instance.
What you expected to happen:
Get our logs indexed and searchable in our Splunk Cloud instance.
How to reproduce it (as minimally and precisely as possible):
This command works
curl -vvv https://http-inputs.<stack-name>.splunkcloud.com:443/services/collector/raw -H "Authorization: Splunk <token>" -d 'Hello, world!' -v

However, our Splunk Connect for Kubernetes deployment using
splunk.hec.host = http-inputs.<stackname>.splunkcloud.com
splunk.hec.port = 443
splunk.hec.insecureSSL =  true
splunk.hec.token = <token>

doesn't seem to work.
We've been told we are running on one of the newest Splunk Cloud stacks. Are there any known interoperability issues, or are we simply misconfiguring something?
Environment:

Splunk version: 8.1.2011.1
Splunk Connect for Kubernetes helm chart version: 1.4.3

It was due to a misconfiguration. We are using annotations to route logs to specific indexes and we forgot to update that part of the system. It's working now.


Design Task: "How To Buy" controller for newly joined users

provide design for the controller @TalgatNurdinov

This issues was dropped from the list due to lack of team consensus on the importance/need of the described feature


How does C++ call the model

I want to call the model process prediction of training through C++, but I do not know how to do it, I hope to get your help, thank you!

@WalterMa Hi, I tried it. It works，thank you for your help.


Manage vault via argocd, update preview.sh

Deploy vault on physical cluster using argocd
Integrate setting of spi into preview.sh in similar way as in pre-kcp branch

@skabashnyuk should be fixed now, I forgot to define kubeconfig on getting apps, I use cluster config by default


ggshakeR Gallery New Issue

This is the new issue for the ggshakeR gallery. I'm creating a checklist for all the stuff I think would be nice to have + things to be added.

[x] Create website
[ ] Solve issue with the colour of code blocks in markdown files (used for articles)

Article ideas :-

[ ] k-means for clustering passes
[ ] Comet charts + GMM clustering
[ ] Bumpy chart

Let me know what other articles you'd like to see.
Also, with this having a separate website, I don't think this necessarily has to run concurrent to the updates made to the package. We can just release articles as they are published. Let me know your thoughts.

Let me know which design looks better + is more practical and can be used in the tutorial for k-means clustering.
Shake already has code for the plot_comet() function which can just be changed up a bit to make it a for for an article.
For a bumpy chart, have to explore {ggbump}


add support for input and password gestures in VS Code notebooks

Any ETA for this feature? Not being able to use input in VSCode .NET notebooks makes harder some experiences where we want to collect input from the user. Without this in VSCode, we still to use a classic jupyter server with the C# kernel (which already supports input).

This is implemented and we've done a poor job of documenting it.
The method is called GetInputAsync rather than the less .NET-ish input.
Give this a try:
var value = await GetInputAsync("Put a prompt here", isPassword: true);

value

@jonsequitur Thank you. It works like a charm.


[Feature Request] Implement the PCG Prng

Implement the Prng abstract class with the PCG pseudo random number generator. This should become the primary random source for echo in the future.


Updating the angular version?

Bower keeps asking me which version to use of angular because the version in the bower file is lower than my current version of angular. Could I post a PR?

@rlweb did you encounter any issues with this using angular >=1.4?

I tried using v1.4.3 or angular and put a resolutions in bower.json for angular 1.4.3 or ~1.4.3. I keep getting the error due to angular version conflict (as ng-websocket specifies v1.3.2 and I don't have that version of angular in intranet artifactory):
Additional error details:
No version found for ~1.3.2. Available tags: [1.4.7, 1.4.3, 1.4.5], Available branches: []

This project is no longer mainteined. Sorry.


Exit if no data because of sensor not in range

Hello, I use ver 0.4 with miflora-mqtt-daemon from ThomDietrich.
If a miFlora sensor is out of range I get a error in the miflora-mqtt-daemon.py -> miflora_poller.py -> gatttool.py
I don't can find out where exact is the problem, but I suppose it's in the backend support.
Because python is not my primary language maybe someone can see the problem !?

Xiaomi Mi Flora Plant Sensor MQTT Client/Daemon
Source: https://github.com/ThomDietrich/miflora-mqtt-daemon

Adding sensor to device list and testing connection ...
Name:          "FloraCare01"
Traceback (most recent call last):
  File "/opt/miflora-mqtt-daemon/miflora-mqtt-daemon.py", line 218, in <module>
    flora_poller.fill_cache()
  File "/usr/local/lib/python3.5/dist-packages/miflora/miflora_poller.py", line 61, in fill_cache
    firmware_version = self.firmware_version()
  File "/usr/local/lib/python3.5/dist-packages/miflora/miflora_poller.py", line 104, in firmware_version
    res = connection.read_handle(_HANDLE_READ_VERSION_BATTERY)  # pylint: disable=no-member
  File "/usr/local/lib/python3.5/dist-packages/btlewrap/gatttool.py", line 23, in _func_wrapper
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.5/dist-packages/btlewrap/gatttool.py", line 253, in read_handle
    raise BluetoothBackendException("Exit read_ble, no data ({})".format(current_thread()))
btlewrap.base.BluetoothBackendException: Exit read_ble, no data (<_MainThread(MainThread, started -1225092352)>)

An exception is thrown, hence I'd say this issue is not a problem with the library and we should solve it in https://github.com/ThomDietrich/miflora-mqtt-daemon/issues/39
As I said, PRs welcome!


Incorrect way to create SQLite client for Astro project

Path: /sdk/ts/guides/astro
The document uses the following to create a client:
import { createClient } from "@libsql/client/web";

export const turso = createClient({
  url: process.env.TURSO_DATABASE_URL!,
  authToken: process.env.TURSO_AUTH_TOKEN,
});

Which results in the following error:
Error [LibsqlError]: URL_INVALID: The URL is not in a valid format

The reason is environment variables are obtained incorrect. Check the official doc in Astro:
https://docs.astro.build/en/guides/environment-variables/#getting-environment-variables
The correct way should be:
import { createClient } from "@libsql/client/web";

export const turso = createClient({
  url: import.meta.env.TURSO_DATABASE_URL!,
  authToken: import.meta.env.TURSO_AUTH_TOKEN,
});

Hey @webberpuma
I’ll get this updated, thanks so much. However, there’s an even better approach using the new :env package. See an example here: https://youtu.be/jU3dbZOaS9g?si=fG5Mxz-l6im5Z4lZ

Hey @webberpuma
I’ll get this updated, thanks so much. However, there’s an even better approach using the new :env package. See an example here: https://youtu.be/jU3dbZOaS9g?si=fG5Mxz-l6im5Z4lZ

@notrab Thank you for the new Info!


Add an Expected Maximum Breeding Value Matrix class in the bvmat submodule.

This would help with code reuse for users who would like to export EMBVs to a file and for EMBV Problem and EMBV Selection classes.

It might be good to do this for Optimal Haploid Breeding Values too.


495 histogram accuracy

Closes #495

~Delete the file dp_histogram_spec.py (and any related imports)~
* from stand-up ^, perhaps use as a base class to factor out common functionality

Accuracy description for histograms

In dp_histogram_integer.py and dp_histogram_categorical.py:

Add theget_accuracy_text function to set_accuracy

See dp_count_spec.py to see where set_accuracy is used
Create a new template file for histogram accuracy to be sent to set_accuracy

Accuracy description within the general description

(Possible solution...free free to discuss)

In stat_spec.py, for get_release_dict add arguments: description_template_html and description_template_text with defaults of None

If the value(s) is/are None, default to the current values in the function
Create new templates to reflect a histogram description -- with if/then statements if there are too many values to display

@raprasad This already exists (see server/opendp_apps/analysis/templates/analysis/dp_histogram_accuracy_default.html)

@ecowan

I see the accuracy description for the histograms. Looks good. (I missed it.)
The general description has an outdated accuracy description within it (2nd bullet). That one needs updating--it can use the accuracy description you already have. (see below)

Also, the general description for a histogram is close to unreadable with the categories and values. Can it be written in a more general way with the assumption that the web page and PDF will have the chart for viewing (as well as a future table)

accuracy.message is different from the last sentence in description.html/description.text

"accuracy": {
                "value": 5.075173853046817,
                "message": "There is a probability of 95.0% that a count in the  DP Histogram will\ndiffer from the count in the true Histogram by at most 5.075173853046817 units. Here the\nunits are the same units the variable Trial has in the dataset."
            },
"description": {
                "html": "A differentially private Histogram for variable Trial was calculated with the result {'categories': [0, 1, 2, 3, 4, 5, 6, 7, 'uncategorized'], 'values': [61, 0, 0, 4, 112, 8, 1, 3, 0], 'category_value_pairs': [(0, 61), (1, 0), (2, 0), (3, 4), (4, 112), (5, 8), (6, 1), (7, 3), ('uncategorized', 0)]}.  There is a probability of 95.0% that the DP Histogram will differ from\nthe true Histogram by at most 5.075173853046817 units. Here the units are the same units\nthe variable Trial has in the dataset.",
                "text": "A differentially private Histogram for variable \"Trial\" was calculated with the result {'categories': [0, 1, 2, 3, 4, 5, 6, 7, 'uncategorized'], 'values': [61, 0, 0, 4, 112, 8, 1, 3, 0], 'category_value_pairs': [(0, 61), (1, 0), (2, 0), (3, 4), (4, 112), (5, 8), (6, 1), (7, 3), ('uncategorized', 0)]}. There is a probability of 95.0% that the DP Histogram will differ from the true Histogram by at most 5.075173853046817 units. Here the units are the same units the variable Trial has in the dataset."
            }
        }

Note from standup: we also need to round values in sentences (accuracy, etc.) to 2 significant digits.


task error

Under high load by siege I have next errors:
ERROR:tokio_proto::server: task error: Connection reset by peer (os error 104)
I suppose we should try read socket again.
PS my current performance results really bad. sapper on hyper 2x faster

Ok, this has been updated with recent tokio-core/tokio-proto changes and I think this is taken care of now. If you still see this though please let me know!


[DOC] Remove mentoring and meetups pages

Resolves #228
We currently do not have a mentoring program, unless someone is willing to update this page and maintain such a system.
The meetups page does not really add anything of value.

Well we can keep the page, but it should probably be reworked to offer specific programs from specific mentors.
I can replace the current content with a placeholder or just leave it as it is if you want @aiwalter?
Some of the phrasing like "We explicitly encourage students from underrepresented groups or who have experienced discrimination in the past to apply!" makes me feel like im recruiting for a cult 🙂

yeah that part should go. I can update that page next days. You can make it empty now if you want, e.g. write "tbd"


[Fix/UI] Context Switcher Ping Not working

Description
CU-2h1g0yf

Kubernetes ping operation from context-switcher not responding
Confirmation Modal ping operation not working
Remove the faulty function signature, so that it can't be used in future

This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Task linked: CU-2h1g0yf k8s Context Switcher: No toaster notification on ad hoc connectivity test on K8s chip in K8s context switcher

OOh looking at it, Probably forgot to remove all references.


Added image object detector files

Fixes
#296

Description
Added a python script used to detect an object from an image by providing it a template image of that object.
Type of change

[x] New project (non-breaking change which adds functionality)

How Has This Been Tested?
https://user-images.githubusercontent.com/64689436/162295049-e650f607-a56b-4d0a-95d4-2a2110e74c9f.mp4
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have included a README.md file
[x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x] I have added demonstration in the form of GIF/video file
[x] I am a GSSoC'22 Participant

closes #296
Test Images

@heet-kumar Done.

@heet-kumar any updates about the review?


Is there a trained model for Kazakh available for download somewhere?

No, we do not distribute models, we provide the tools to train your own.


feat: saliency map

👋 Thanks for submitting a Pull Request to EVA DB!
🙌 We want to make contributing to EVA DB as easy and transparent as possible. Here are a few tips to get you started:

🔍 Search existing EVA DB PRs to see if a similar PR already exists.
🔗 Link this PR to a EVA DB issue to help us understand what bug fix or feature is being implemented.
📈 Provide before and after profiling results to help us quantify the improvement your PR provides (if applicable).

👉 Please see our ✅ Contributing Guide for more details.

@vivek-mandal Can you rewrite the SaliencyUDF to look like this UDF -- https://github.com/georgia-tech-db/eva/blob/master/eva/udfs/sift_feature_extractor.py? We have updated the UDF design.

I have moved around the files to the appropriate locations in the repo.

@vivek-mandal Can you rewrite the SaliencyUDF to look like this UDF -- https://github.com/georgia-tech-db/eva/blob/master/eva/udfs/sift_feature_extractor.py? We have updated the UDF design.

Sure Professor, will do that!!


Use the new fs method for doing the file copying

Fixes #13.

Awesome! Thanks for doing this @kevva :)


Cannot delete some instances of model 'ContentType' because they are referenced through protected foreign keys: 'ObjectChange.changed_object_type'

Peering Manager version
v1.8.0-dev@3f8b9662
Python version
3.10
Steps to Reproduce
Follow the regular upgrade procedure from installation using tags running from commit 4ae81525 to commit 3f8b9662
Expected Behavior
A successful migration.
Observed Behavior
🔄 Removing stale content types
Traceback (most recent call last):
File "/opt/peering-manager/manage.py", line 11, in
execute_from_command_line(sys.argv)
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/core/management/init.py", line 446, in execute_from_command_line
utility.execute()
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/core/management/init.py", line 440, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/core/management/base.py", line 402, in run_from_argv
self.execute(*args, **cmd_options)
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/core/management/base.py", line 448, in execute
output = self.handle(*args, **options)
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/contrib/contenttypes/management/commands/remove_stale_contenttypes.py", line 99, in handle
ct.delete()
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/db/models/base.py", line 1117, in delete
collector.collect([self], keep_parents=keep_parents)
File "/opt/peering-manager/venv/lib/python3.10/site-packages/django/db/models/deletion.py", line 350, in collect
raise ProtectedError(
django.db.models.deletion.ProtectedError: ("Cannot delete some instances of model 'ContentType' because they are referenced through protected foreign keys: 'ObjectChange.changed_object_type'.", {<ObjectChange: tag BGP: created by adminuser>, <ObjectChange: tag Passive created by adminuser>, <ObjectChange: tag Session created by adminuser>, <ObjectChange: tag BGP-Session-Passive created by adminuser>, <ObjectChange: tag BGP: deleted by adminuser>, <ObjectChange: tag Passive deleted by adminuser>, <ObjectChange: tag Session deleted by adminuser>, <ObjectChange: tag BGP-Session-Passive updated by adminuser>, <ObjectChange: tag BGP-Session-Passive updated by adminuser>, <ObjectChange: tag Default-Policy updated by adminuser>, <ObjectChange: tag depref-50 updated by adminuser>, <ObjectChange: tag Fallback-Policy updated by adminuser>, <ObjectChange: tag no-advertise updated by adminuser>, <ObjectChange: tag Prefix-Preference updated by adminuser>, <ObjectChange: tag Default-Contact created by adminuser>})

I assume if I update the references. The protection will disappear and the protected tags will be cleaned up during the next run of the migration script... :thinking:

Yes some pieces are missing to cleanup old ObjectChanges referencing Tags. I need to address that.


Question: how to improve new connections per second rate

Environment

VerneMQ Version: 1.10.2
OS: Linux
Erlang/OTP version (if building from source): Built from source
Cluster size/standalone: standalone
VerneMQ configuration override (vernemq.conf):
accept_eula = yes
allow_anonymous = off
max_inflight_messages = 20
max_online_messages = 1000000
max_offline_messages = 1000
listener.ssl.ssl-listener.buffer_sizes = 2048,4096,8192
listener.ssl.ssl-listener.max_connections = infinity
listener.ssl.ssl-listener.nr_of_acceptors = 1000
listener.ssl.ssl-listener = <IP_ADDRESS>:1884
listener.ssl.ssl-listener.use_identity_as_username = on
listener.ssl.cafile = <path_to_file>
listener.ssl.certfile = <path_to_file>
listener.ssl.keyfile = <path_to_file>
listener.ssl.ssl-listener.require_certificate = on
plugins.vmq_passwd = on
plugins.vmq_diversity = on
vmq_acl.acl_file = <path_to_file>
vmq_passwd.password_file = <path_to_file>
vmq_diversity.script_dir = <path_to_file>
erlang.process_limit = 10000000

I run a bunch of load tests using a custom load testing tool, and experimenting with different VerneMQ configurations and test cases.
Different VerneMQ configurations include testing with and without SSL, playing with 'erlang' and 'messages' parameters from the provided vernemq.conf file above.
Different test cases mostly include playing with different numbers of connections and ramp-up times.
I have used Grafana for monitoring both VerneMQ as a service and hardware resources.
I managed to make a large number of concurrent connections (500K) but for a longer period (1 hour).
One of the goals of performance tests was what is the maximum number of connections per second. The maximum number I have reached is about 600 connections per second. I was not able to find what causes this limitation - everything looks normal (not at maximum) on hardware and vernemq metrics.
I know this may be very specific from case to case, but are there any tips and/or configuration tweaks that make improve the current rate of 600 new connections per second?

@mladen-djordjevic Registers need to be coordinated so that there is ever only 1 unique ClientID on the whole cluster.
You can turn this off with coordinate_registrations = off
You can also test whether metadata_plugin = vmq_swc gives you better register performance.
Also, note that if you use different ClientIDs for every test run, you'll end up loading the cluster with a lot of unused ClientID keys. This could end up slowing down registers too.
On accept_eula=yes, please read EULA, you likely need a subscription.

https://github.com/sponsors/vernemq

@ioolkos thank you for the suggestions, I will try it and keep you updated.
As per VerneMQ EULA, building it from a source doesn't require a subscription. Am I correct?

As per VerneMQ EULA, building it from a source doesn't require a subscription. Am I correct?

Yes.

https://github.com/sponsors/vernemq


Fix #1 (bad credential input loops)

Prevents endlessly looping through WiFi connect sequence if the given credentials are incorrect.

@jasonbrackman is this what you were thinking?


Json in Api

Hi @Adrian,
Iam usin the api test of opentest and i want to send Json but i donot know what is the right way:
body: |
$json(
{
CallerSipUri<EMAIL_ADDRESS>CalleeSipUri<EMAIL_ADDRESS>}

$json(
{
"CallerSipUri"<EMAIL_ADDRESS>"CalleeSipUri"<EMAIL_ADDRESS>}
)
It will be work.

ok thanks


Icon in actions doesnt consider iconSize property

The requirement changed after this was implemented so the current version doesn't match the actual spec
https://github.com/Microsoft/AdaptiveCards/issues/389

since we'd added iconUrl before iconSize was added in UWP, Android, and iOS, should we create different issues for each platform?

PRs for the shared model is merged. Feel free to start with the iOS renderer @nesalang


Work around Eigen build bug

Fixes #527
Note that this PR is a rebase of #530 onto the latest so we can go for green CI.  Assuming we get that, I'm going to go ahead and merge based on the approval in #530.  @rotu FYI.
CI:

Linux
Linux-aarch64
macOS
Windows

macOS warning is one of the known warnings, so merging.


Prevent fail bundle when have other name than napari

Description

Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

References

How has this been tested?

[ ] example: the test suite for my feature covers cases x, y, and z
[x] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas

I just updated the release notes to include this PR

I still use pyinstaller. Without this change I got something like this  (this log is from napari 0.3.7 but similar one is from 0.3.8rc0)
  File "napari/plugins/__init__.py", line 10, in <module>
  File "napari/utils/misc.py", line 27, in running_as_bundled_app
  File "importlib_metadata/__init__.py", line 575, in metadata
  File "importlib_metadata/__init__.py", line 215, in from_name
importlib_metadata.PackageNotFoundError: No package metadata was found for napari
[930886] Failed to execute script launcher_main

I have plan to switch to briefcase but not have enough time for this now.
Current briefcase release does not allow to do this. I create first PR to fix briefcase: https://github.com/beeware/briefcase/pull/475.
But it may need more PR. (Current release is older than this PR merge)

I still use pyinstaller. Without this change I got something like this (this log is from napari 0.3.7 but similar one is from 0.3.8rc0)

Ok thanks, good to know!


Add my first file

I have created a new branch, created a file and made a commit!

Merge completed


Added myself as a developer for Fitnesse plugin

Description
Add @jaruzafa as developer for Fitnesse plugin: https://github.com/jenkinsci/fitnesse-plugin
As said per @lessonz at https://github.com/jenkinsci/fitnesse-plugin/pull/30#issuecomment-424039164 this plugin is not maintained by anyone
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

When adding new uploaders (this includes newly created permissions files)

[x] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

Plugin seems abandoned indeed, so it looks like the plugin is orphaned and we can move forward here.

https://github.com/jenkinsci/fitnesse-plugin/pull/30#issuecomment-424039164 is by a current maintainer (they have write access, just not team maintainership) and points to an email saying

I'm happy to give it over, but I do not see where I have privileges to do so. I originally took over for similar reasons, but I no longer use Fitnesse. I wish you the best of luck.

So @lessonz's approval is indeed all that is needed here.


How can i adjust the overall size of the calendar

i want to resize from

to

You do not want to show the last row?

You do not want to show the last row?

@kizitonwose no i want just to adjust overall calendar size
I thought i could adjust the size of the framelayout surrounding the textview with the day on it, ut the overall size of the calendar hasn't changed. plz help me.

@kizitonwose no i want just to adjust overall calendar size
I thought i could adjust the size of the framelayout surrounding the textview with the day on it, ut the overall size of the calendar hasn't changed. plz help me.

You need to set the daySize property.

You need to set the daySize property.

@kizitonwose  thank you so much!!  I'll try

@kizitonwose  thank you so much!!  I'll try

In Addition to above doubt, how to handle date selection? Like if we select another date, previous one should be deselected. Is there any in-build way to achieve this?

In Addition to above doubt, how to handle date selection? Like if we select another date, previous one should be deselected. Is there any in-build way to achieve this?

@asthagarg2428 what kind of handle?? like add task to date?

@asthagarg2428 what kind of handle?? like add task to date?

I want to implement single selection type of CalendarView, how to do that?
Like if I go for adding click listener on each textView, then I will have to do many other handling. And as per the readMe doc of this library, there is an option to enable single date selection.
Hope I'm able to clearly explain my question

I want to implement single selection type of CalendarView, how to do that?
Like if I go for adding click listener on each textView, then I will have to do many other handling. And as per the readMe doc of this library, there is an option to enable single date selection.
Hope I'm able to clearly explain my question

@asthagarg2428 I replied you in #258 so I can close this.

@asthagarg2428 I replied you in #258 so I can close this.


Keygen questions

The current spec defines the keygen salt as BBS-SIG-KEYGEN-SALT-, while this library uses the value BLS-SIG-KEYGEN-SALT-. The second is consistent with the BLS signatures spec draft 2, and the method seems to follow the keygen defined there, but it has been updated incompatibly for draft 4. So, thoughts on which salt and method should be used?

@mikelodder7 can you review?

Here's a proposal I'm making for BBS+ that should help with this


fix: Obsolete SyncObject Flush method

Adds back Flush method with Obsolete
1 tests failed

Failed test SyncVarsSyncList in Mirror.Weaver.Tests.WeaverSyncVarTests
SetUp : Expected: False
But was: True
SetUp : Unhandled log message: '[Error] Assets/Mirror/Tests/Editor/Weaver/WeaverSyncVarTests~/SyncVarsSyncList.cs:9 -- Assets/Mirror/Tests/Editor/Weaver/WeaverSyncVarTests~/SyncVarsSyncList.cs(9,43): error CS0535: 'SyncVarsSyncList.SyncObjImplementer' does not implement interface member 'SyncObject.Flush()''. Use UnityEngine.TestTools.LogAssert.Expect

Flush is only used by Mirror internally, not by anyone else.
the test was already fixed a while ago:

we could add obsolete, but there's not 1 person who uses Flush other than Mirror internally.
do we really need this?

Where the heck is SyncObjImplementor - I searched the entire solution and came up empty.

Where the heck is SyncObjImplementor - I searched the entire solution and came up empty.

it's in Tests/Weaver/hidden folders

:


Fixes deprecation warnings

The only remaining warning at this time is for readTerm.
Signed-off-by: Johnneylee Jack Rollins<EMAIL_ADDRESS>

Looks good to me!


[Suggestion] Fabric support

Well. as it says: I beg for fabric support. It would be a dream to see it in fabric. this mod is the best logic mod ive ever seen! And we all need it in fabric.

The mod is open-source so anyone who wants to try porting it to fabric is welcome to

Okay thanks.  Sept. 2023, 14:45:

The mod is open-source so anyone who wants to try porting it to fabric is
welcome to
—
Reply to this email directly, view it on GitHub
https://github.com/Commoble/morered/issues/32#issuecomment-1722222777,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5H7J5RRDQXWHCAMRCZCVP3X2WNOXANCNFSM6AAAAAA42ZQSSA
.
You are receiving this because you authored the thread.Message ID:
@.***>


feat: add link to the space from the proposal page

Design TBD

@bonustrack can I use the UI from this issue for this?
https://github.com/snapshot-labs/sx-ui/issues/724

Yes


Can I use MSS for simulating free falling objects under water?

Dear Prof. Fossen,
The free falling of an object underwater should be much easier, no power and no target. Isn't it?
Can I make the simulation by some 'simple' modification to the code?
Regards
Huisky

If you put all control signals to zero in SIMremus100.m the AUV will sink if and only if the weight is larger than the buoyancy force. You can increase the mass in remus100.m, which contains the AUV model. Note that the AUV is cylinder-shaped, but it will sink like a rock.


auth caching doesnt fully work with multiple ldap

I have a setup with two auth servers:
auth_ldap_servers ldap1;
auth_ldap_servers ldap2;

Auth works fine whichever the server contains the user i'm sending credentials for, but for caching it's not the same:
If the user is found in ldap1, the info is properly cached in nginx, ldap2 is not searched for at all, and ldap1 isnt queried for the time the info is cached.
If the user is not found in ldap1 but only in ldap2, ldap1 will be queried at every subsequent user hit (even though nginx has cached the fact that 'user has been found in ldap2'), thus the log is filled with error messages from the query on ldap1 not finding user in there:
2018/03/20 15:25:44 [error] 1568#0: *75989 http_auth_ldap: Could not find user DN,
client:
2018/03/20 15:25:44 [error] 1568#0: *75991 http_auth_ldap: Could not find user DN, client:
2018/03/20 15:25:45 [error] 1568#0: *75993 http_auth_ldap: Could not find user DN, client:

User still doesnt have to resend creds as he's been authentified in ldap2, but the logs are filing up...
Is there a way to search first for the cache (being global to all servers) before trying to auth/search in each server ?

I am using 2 ldap server for 2 domain accounts and will be interested in the solution to reduce the log noise as well.
@landryb not to hijack your thread but are you using require group in your config and if so is nested groups working for you ?

was fixed by 81f7e24 / #219


Adding a column with not null and default value

Describe the bug
@Property({ default: '' })
produces:
alter table "Table" add column "column" varchar(255) not null default ;
Expected behavior
alter table "Table" add column "column" varchar(255) not null default "";
Additional context
I'm trying to add a column with schema:update... just to get the column into an existing database of rows, I need to default it to an empty string '' to start with and then I can remove that by hand.
Maybe I'm doing this wrong.
Versions

Dependency
Version

mikro-orm
3.6.12

your-driver
pg 8.0.3

Try to wrap it in quotes

That works!
But of course, the real solution is for mikro-orm to do it for me. ;-)

Well the default is used as is to support things like “now()”, i am not planning to change this.

I don't get it... internally you see an empty string. Why can't mikro generate an empty string?

Ok agreed, empty string is doable, my point is in generall, 1 is not “1” :)

Nor would I ask it to be. =)

So today I faced this problem myself, but with enum value, which was even less obvious:
@Enum({ default: Role.USER })
role = Role.USER;

And wrapping this in quotes looks quite ugly. Will change this in v4, so default will be strictly typed to string/number/boolean/null and strings will really mean strings there, plus there will be defaultRaw: string that will be used as is. Breaking, but I think its needed, recently I saw a project that was also misusing the default option (with new Date()), so having strictly defined type there should help.

Yea, no bueno. =)
I'm using mikro in two projects now... love it man. Thank you so much for all your hard work!


Support libgpiod and deprecate sysfs

Ref:
https://git.kernel.org/pub/scm/libs/libgpiod/libgpiod.git/tree/
https://github.com/brgl/libgpiod
https://fosdem.org/2018/schedule/event/new_gpio_interface_for_linux/

I haven't used this package in awhile, but @georgefst just informed me that recent Linux kernels don't have sysfs GPIO support anymore, so I guess I'd better get around to this soon.


add aggregation / grouping

https://datagrid.tzezar.pl/experimental/grouping-aggregating-preview

Will be implemented with #27


InvalidArgumentError (see above for traceback): CPU BiasOp only supports NHWC

Got an InvalidArgumentError after 26 minutes of training. I upgraded to the most recent TensorFlow as suggested and did $ pip install -U 'gym[all]' tqdm scipy. I ran this on a TitanX and Ubuntu 16.10.
$ time python main.py --network_header_type=nips --env_name=Breakout-v0 --use_gpu=True --display=True
[2017-01-26 23:58:40,289] DEPRECATION WARNING: env.spec.timestep_limit has been deprecated. Replace any calls to `register(timestep_limit=200)` with `register(tags={'wrapper_config.TimeLimit.max_episode_steps': 200)}`, . This change was made 12/28/2016 and is included in gym version 0.7.0. If you are getting many of these warnings, you may need to update universe past version 0.21.1
[2017-01-26 23:58:40,289] DEPRECATION WARNING: env.spec.timestep_limit has been deprecated. Replace any calls to `register(timestep_limit=200)` with `register(tags={'wrapper_config.TimeLimit.max_episode_steps': 200)}`, . This change was made 12/28/2016 and is included in gym version 0.7.0. If you are getting many of these warnings, you may need to update universe past version 0.21.1
[2017-01-26 23:58:40,289] DEPRECATION WARNING: env.spec.timestep_limit has been deprecated. Replace any calls to `register(timestep_limit=200)` with `register(tags={'wrapper_config.TimeLimit.max_episode_steps': 200)}`, . This change was made 12/28/2016 and is included in gym version 0.7.0. If you are getting many of these warnings, you may need to update universe past version 0.21.1
[2017-01-26 23:58:40,289] DEPRECATION WARNING: env.spec.timestep_limit has been deprecated. Replace any calls to `register(timestep_limit=200)` with `register(tags={'wrapper_config.TimeLimit.max_episode_steps': 200)}`, . This change was made 12/28/2016 and is included in gym version 0.7.0. If you are getting many of these warnings, you may need to update universe past version 0.21.1
{'agent_type': 'DQN',
 'batch_size': 32,
 'beta': 0.01,
 'data_format': 'NCHW',
 'decay': 0.99,
 'discount_r': 0.99,
 'display': True,
 'double_q': False,
 'env_name': 'Breakout-v0',
 'ep_end': 0.01,
 'ep_start': 1.0,
 'gamma': 0.99,
 'gpu_fraction': '1/1',
 'history_length': 4,
 'is_train': True,
 'learning_rate': 0.00025,
 'learning_rate_decay': 0.96,
 'learning_rate_decay_step': 50000,
 'learning_rate_minimum': 0.00025,
 'log_level': 'INFO',
 'max_delta': None,
 'max_grad_norm': None,
 'max_r': 1,
 'max_random_start': 30,
 'memory_size': 1000000,
 'min_delta': None,
 'min_r': -1,
 'momentum': 0.0,
 'n_action_repeat': 4,
 'network_header_type': 'nips',
 'network_output_type': 'normal',
 'observation_dims': [80, 80],
 'random_seed': 123,
 'random_start': True,
 'scale': 10000,
 't_ep_end': 1000000,
 't_learn_start': 50000,
 't_target_q_update_freq': 10000,
 't_test': 10000,
 't_train_freq': 4,
 't_train_max': 50000000,
 'tag': '',
 'unrolled_lstm': False,
 'use_cumulated_reward': False,
 'use_gpu': True}
 [*] GPU : 1.0000
[2017-01-26 23:58:40,330] Making new env: Breakout-v0
[2017-01-26 23:58:40,352] Using 6 actions : NOOP, FIRE, RIGHT, LEFT, RIGHTFIRE, LEFTFIRE
INFO:tensorflow:Summary name episode/max reward is illegal; using episode/max_reward instead.
[2017-01-26 23:58:40,938] Summary name episode/max reward is illegal; using episode/max_reward instead.
INFO:tensorflow:Summary name episode/min reward is illegal; using episode/min_reward instead.
[2017-01-26 23:58:40,940] Summary name episode/min reward is illegal; using episode/min_reward instead.
INFO:tensorflow:Summary name episode/avg reward is illegal; using episode/avg_reward instead.
[2017-01-26 23:58:40,941] Summary name episode/avg reward is illegal; using episode/avg_reward instead.
INFO:tensorflow:Summary name episode/num of game is illegal; using episode/num_of_game instead.
[2017-01-26 23:58:40,943] Summary name episode/num of game is illegal; using episode/num_of_game instead.
WARNING:tensorflow:From /media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/agents/agent.py:61 in train.: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use `tf.global_variables_initializer` instead.
[2017-01-26 23:58:41,273] From /media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/agents/agent.py:61 in train.: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use `tf.global_variables_initializer` instead.
 [!] Load FAILED: checkpoints/Breakout-v0/env_name=Breakout-v0/agent_type=DQN/batch_size=32/beta=0.01/data_format=NCHW/decay=0.99/discount_r=0.99/double_q=False/ep_end=0.01/ep_start=1.0/gamma=0.99/history_length=4/learning_rate=0.00025/learning_rate_decay=0.96/learning_rate_decay_step=50000/learning_rate_minimum=0.00025/max_delta=None/max_grad_norm=None/max_r=1/min_delta=None/min_r=-1/momentum=0.0/n_action_repeat=4/network_header_type=nips/network_output_type=normal/observation_dims=80,80/random_start=True/t_ep_end=1000000/t_learn_start=50000/t_target_q_update_freq=10000/t_test=10000/t_train_freq=4/t_train_max=50000000/unrolled_lstm=False/use_cumulated_reward=False/
  0%|                                    | 0/50000000 [00:00<?, ?it/s]/usr/local/lib/python2.7/dist-packages/numpy/core/fromnumeric.py:2889: RuntimeWarning: Mean of empty slice.
  out=out, **kwargs)
/usr/local/lib/python2.7/dist-packages/numpy/core/_methods.py:80: RuntimeWarning: invalid value encountered in double_scalars
  ret = ret.dtype.type(ret / rcount)
  0%|                    | 50002/50000000 [25:59<436:00:27, 31.82it/s]E tensorflow/core/common_runtime/executor.cc:390] Executor failed to create kernel. Invalid argument: CPU BiasOp only supports NHWC.
	 [[Node: target_network/l1_conv/BiasAdd = BiasAdd[T=DT_FLOAT, data_format="NCHW", _device="/job:localhost/replica:0/task:0/cpu:0"](target_network/l1_conv/Conv2D, target_network/l1_conv/b/read)]]
Traceback (most recent call last):
  File "main.py", line 168, in <module>
    tf.app.run()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 43, in run
    sys.exit(main(sys.argv[:1] + flags_passthrough))
  File "main.py", line 163, in main
    agent.train(conf.t_train_max)
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/agents/agent.py", line 82, in train
    q, loss, is_update = self.observe(observation, reward, action, terminal)
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/agents/deep_q.py", line 61, in observe
    result = self.q_learning_minibatch()
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/agents/deep_q.py", line 84, in q_learning_minibatch
    max_q_t_plus_1 = self.target_network.calc_max_outputs(s_t_plus_1)
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/networks/network.py", line 87, in calc_max_outputs
    return self.max_outputs.eval({self.inputs: observation}, session=self.sess)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 575, in eval
    return _eval_using_default_session(self, feed_dict, self.graph, session)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 3633, in _eval_using_default_session
    return session.run(tensors, feed_dict)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 766, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 964, in _run
    feed_dict_string, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1014, in _do_run
    target_list, options, run_metadata)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1034, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InvalidArgumentError: CPU BiasOp only supports NHWC.
	 [[Node: target_network/l1_conv/BiasAdd = BiasAdd[T=DT_FLOAT, data_format="NCHW", _device="/job:localhost/replica:0/task:0/cpu:0"](target_network/l1_conv/Conv2D, target_network/l1_conv/b/read)]]

Caused by op u'target_network/l1_conv/BiasAdd', defined at:
  File "main.py", line 168, in <module>
    tf.app.run()
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 43, in run
    sys.exit(main(sys.argv[:1] + flags_passthrough))
  File "main.py", line 140, in main
    name='target_network', trainable=False)
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/networks/cnn.py", line 55, in __init__
    hidden_activation_fn, data_format, name='l1_conv')
  File "/media/ch3njus/Seagate4TB/research/ab/deep-rl-tensorflow/networks/layers.py", line 30, in conv2d
    out = tf.nn.bias_add(conv, b, data_format)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/nn_ops.py", line 1172, in bias_add
    return gen_nn_ops._bias_add(value, bias, data_format=data_format, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/ops/gen_nn_ops.py", line 281, in _bias_add
    data_format=data_format, name=name)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/op_def_library.py", line 759, in apply_op
    op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 2240, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/ops.py", line 1128, in __init__
    self._traceback = _extract_stack()

InvalidArgumentError (see above for traceback): CPU BiasOp only supports NHWC.
	 [[Node: target_network/l1_conv/BiasAdd = BiasAdd[T=DT_FLOAT, data_format="NCHW", _device="/job:localhost/replica:0/task:0/cpu:0"](target_network/l1_conv/Conv2D, target_network/l1_conv/b/read)]]

real	26m4.536s
user	10m27.848s
sys	7m41.136s

Hi @ch3njust1n ,
I came across the same error when running DQN for vizdoom.
Have you solved the problem?

Your tensorflow doesn't support GPU. Upgrade your TF to
pip install --upgrade tensorflow-gpu


[Feature Request]: Add a ChatBot Assistant ⭐

Is there an existing feature request for this?

[X] I have searched the existing feature requests

Describe the feature you would like to request
I want to add a ChatBot Assistant..
PA please assign me this task.
What problem does this feature solve?
.
Alternative solutions
No response
Confirmation

[X] I have read the Bitbox Contributing Guidelines
[X] I am a GSSOC-EXT contributor
[X] I am a HACKTOBERFEST contributor
[X] I have starred the Bitbox repository
[X] I would like to help implement this feature

@Anuj3553


Zuul and oauth config

Hi, I'm having following situation in my config file on frontend which is also zuul proxy:
security.oauth2:
  client:
    accessTokenUri: http://localhost:9090/auth-server/oauth/token
    userAuthorizationUri: http://localhost:9090/auth-server/oauth/authorize
    clientId: foo
  resource:
    jwt:
      keyValue: |
        -----BEGIN PUBLIC KEY-----
        SOME_KEY
        -----END PUBLIC KEY-----

zuul:
  routes:
    auth-server:
      serviceId: auth-server
      stripPrefix: false
    resource-server1:
      serviceId: resource-server1
    resource-server2:
      serviceId: resource-server2

What bothers me is that address to oauth2 endpoints is hardcoded, I would like to use serviceId (auth-server in particular) there also, since I'm using eureka as discovery service. How can I achieve that?

We don't support discovery for oauth endpoints at the moment. If we did it would be a spring-cloud-security feature, so you might want to start a discussion there?

Duplicates: https://github.com/spring-cloud/spring-cloud-security/issues/61


Fix diagram links to software systems without system context view

Currently, links in diagrams to software systems without a system context view lead to a page with no content and no active tab in the tab bar:

This PR fixes this by linking to the info page in these cases.

I'm closing this PR, as a better solution has been implemented in #101.


release: prepare release 3.2.0

Codecov Report

Merging #14610 (2c26c3b) into dev (1d0d85b) will decrease coverage by 0.02%.
The diff coverage is 7.44%.

:exclamation: Current head 2c26c3b differs from pull request most recent head 04c8220. Consider uploading reports for the commit 04c8220 to get more accurate results

@@             Coverage Diff              @@
##                dev   #14610      +/-   ##
============================================
- Coverage     38.51%   38.49%   -0.02%
+ Complexity     4563     4562       -1
============================================
  Files          1260     1260
  Lines         43791    43782       -9
  Branches       4829     4804      -25
============================================
- Hits          16865    16856       -9
- Misses        25054    25056       +2
+ Partials       1872     1870       -2

Impacted Files
Coverage Δ

...duler/plugin/task/api/AbstractCommandExecutor.java
0.00% <0.00%> (ø)

...ugin/task/api/am/KubernetesApplicationManager.java
0.00% <0.00%> (ø)

...scheduler/plugin/task/api/k8s/AbstractK8sTask.java
0.00% <0.00%> (ø)

...r/plugin/task/api/k8s/AbstractK8sTaskExecutor.java
77.77% <ø> (+24.44%)
:arrow_up:

...nscheduler/plugin/task/api/utils/ProcessUtils.java
0.00% <0.00%> (ø)

...uler/plugin/task/api/k8s/impl/K8sTaskExecutor.java
36.27% <13.20%> (-2.02%)
:arrow_down:

... and 31 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


HTTP2 Passthrough : RST_STREAM sent in upstream results in downstream being halfclosed.

Description:
When there is a HTTP2 passthrough service is running, there are two streams involved; upstream and downstream. And for upstream and downstream, there could two different idle timeouts. Assume the following condition,
UpstreamTimeout < DownstreamTimeout
Assume that this service is used for bidi streaming. For a while the stream will work as it is expected. Hence the response HTTP Headers also would have 200 http2 status code. But after a while Upstream Stream Idle timeout triggers.
[2021-10-28 17:33:43,849] TRACE {http.tracelog.upstream} - [id: 0x6c565b92, L:/<IP_ADDRESS>:58480 - R:store-data-connector.retailsvc.com/<IP_ADDRESS>:443] OUTBOUND RST_STREAM: streamId=3 errorCode=5

Then the downstream stream would simply send a data frame with end_stream flag set to true. This represents a half closed state. But the ideal solution needs to be sending another RST_STREAM frame in downstream.
[2021-10-28 17:33:43,853] TRACE {http.tracelog.downstream} - [id: 0xa26040db, L:/<IP_ADDRESS>:9095 - R:/<IP_ADDRESS>:46996] OUTBOUND DATA: streamId=1 padding=0 endStream=true length=0 data= 0B

The happens in other direction as well. (UpstreamTimeout > DownstreamTimeout)
Steps to reproduce:
HTTP2 passthrough service is required.
HTTP2 streaming client and a streaming server. (Use a grpc streaming client and grpc streaming server)
Set ListenerTimeout and ClientTimeout such that client timeout is greater than the listener timeout.
HTTP2 stream server should send a bunch of frames first, and then introduce a pause such that stream is open but no data frames transmitted.
Tracelogs needs to be enabled for ballerina passthrough service. Run the server and invoke the streaming client. Finally wait till the timeout expires.
Affected Versions:
ballerina 1.2.20
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This can be the root cause for https://github.com/ballerina-platform/ballerina-standard-library/issues/3274

When this happen if the listener handles it and send 500 internal server error to the front-end client, that is good enough.


Console Output

There is so much output on console. Where it is coming from and how to turn it off?

I fixed this in my Fork
https://github.com/kabrau/keras-yolo3

Hey, can you tell me how you solved that?


Fix 1.7.10 and 1.19-1.19.3 flakyness

1.7.10: https://github.com/3arthqu4ke/mc-runtime-test/actions/runs/12110751056/job/33782760771#step:6:3710
1.19.3: https://github.com/3arthqu4ke/mc-runtime-test/actions/runs/12110751056/job/33783691997
It has to be possible to fix this on the mod side, right?
See @3arthqu4ke in https://github.com/3arthqu4ke/mc-runtime-test/issues/22#issuecomment-2511328752

[ ] 1.7.10
[x] 1.19

We dont know about 1.7.10 yet, that issue is much more rare


Made admin/base.html get its own language variables.

Refs #24117
This is the minimum change needed for the current context_processors proposal.
https://code.djangoproject.com/ticket/24117

Under what circumstances will this be backwards incompatible?

If someone's sticking in their own custom context processor and using different values for those variables. I'm fine documenting it however it should be documented, if it should be documented at all.

Hm, I'm not sure that's a use case, but if it is, is there any way they can update their code to achieve that now? Besides overriding base.html seems like no...

Yeah, they'd need override base.html (not fun).

I added two gis templates. Do you think backwards compatibility is an issue? It seems like it would actually be a pretty rare case. (actually, if they didn't have the context processor installed, the the behavior would change)

Merged in 58833f519797e15ae3cb39b1613e5f7e09c96952.


Populating OpenTAXII instance with data

Hey all,
First of all, thanks for writing a solid software project for TAXII users! I really appreciate it!
Warning before I get started: I am a total newb to TAXII, and there's still a lot I don't know. I recognize that isn't the best place to put such a question as this, but it appears to be the only really active TAXII community on the internet. So, here goes.
I was able to follow the setup instructions and get an instance of the TAXII server up and running. I was also able to get cabby to talk to it without any hitches. However, my question is this: how do I populate the server with data? Currently there isn't any data in it. The TAXII spec suggests that this is possible but doesn't explain how to do it.
Thanks,
Joseph

That's what I was looking for. Thanks!


Add nominal role annotation for TaggedUnion

Before this PR, you could coerce tagged unions even though they have different tags:
{-# LANGUAGE DataKinds, ExplicitNamespaces #-}
module Main (main) where

import Data.Coerce (coerce)
import Data.Int (Int32)
import Data.Text (Text)
import Lib (someFunc)
import Shrubbery (TaggedUnion)
import Shrubbery.TypeList (type (@=))

main :: IO ()
main = someFunc

type Foo = TaggedUnion ["int" @= Int32, "text" @= Text]

fromFoo :: Foo -> TaggedUnion '[]
fromFoo = coerce

toFoo :: TaggedUnion '[] -> Foo
toFoo = coerce

After this PR, it results in this error message:
app/Main.hs:22:11: error: [GHC-18872]
    • Couldn't match type: ["int" @= Int32, "text" @= Text]
                     with: '[]
        arising from a use of ‘coerce’
    • In the expression: coerce
      In an equation for ‘fromFoo’: fromFoo = coerce
   |
22 | fromFoo = coerce
   |           ^^^^^^

app/Main.hs:25:9: error: [GHC-18872]
    • Couldn't match type: '[]
                     with: ["int" @= Int32, "text" @= Text]
        arising from a use of ‘coerce’
    • In the expression: coerce
      In an equation for ‘toFoo’: toFoo = coerce
   |
25 | toFoo = coerce
   |         ^^^^^^

Could you do the same for Union? I bet it has the same issue

Added and tested for Union with:
{-# LANGUAGE DataKinds #-}
module Main (main) where

import Data.Coerce (coerce)
import Lib (someFunc)
import Shrubbery (Union)
import Data.Int (Int32)
import Data.Text (Text)

main :: IO ()
main = someFunc

type Foo = Union [Int32, Text]

fromFoo :: Foo -> Union '[]
fromFoo = coerce

toFoo :: Union '[] -> Foo
toFoo = coerce


Adds optional joint to connect the panda in panda_gazebo.xacro, similar to panda_arm.xacro

Hi,
We're currently working on integrating the panda arm on a mobile base, and I noticed there is a difference between panda_gazebo.xacro and panda_arm.xacro. The parameter connected_to is missing which is required to connect the panda in gazebo to another link.
edit: after looking at the issues I saw #168 is exactly this

Hi @c-salmi sorry I noticed your PR only when we already merged this feature internally. Is now done on latest develop (bd2bfd8). Thanks for your contribution


Hydration issue when the user reload the cart page

When you reload the page on the cart using the reload button or "cmd + r"  the page experience some weird behaviours :

the checkout disappear and is replaced by the empty state
Impossible to navigate between pages

Two errors visible in the console :
Uncaught (in promise) DOMException: Failed to execute 'appendChild' on 'Node': This node type does not support this method.
    at Object.appendChild (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:40599)
    at h (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:53331)
    at f (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:52962)
    at d (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:53417)
    at A (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:56755)
    at A (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:56690)
    at a.__patch__ (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:57155)
    at a.t._update (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:34391)
    at a.r (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:64471)
    at fn.get (https://shop-2.vercel.app/assets/js/app.1e8cb306.js:7:26651)

app.1e8cb306.js:7 TypeError: Cannot read property '_isDestroyed' of undefined
    at destroy (app.1e8cb306.js:7)
    at _ (app.1e8cb306.js:7)
    at _ (app.1e8cb306.js:7)
    at _ (app.1e8cb306.js:7)
    at _ (app.1e8cb306.js:7)
    at a.__patch__ (app.1e8cb306.js:7)
    at a.t.$destroy (app.1e8cb306.js:7)
    at destroy (app.1e8cb306.js:7)
    at _ (app.1e8cb306.js:7)
    at a.__patch__ (app.1e8cb306.js:7)```

@Simon-Renault Yes, I have noticed that - I believe it is to do with the Vuex persisted plugin I am using: try removing that, and everything should work as expected. Not quite sure why it happens though, that needs more investigating.
In the most recent storefront-ui branch I have actually replaced it...

I have a fix for that, simple but not super pretty. The issue does not seems to happen if you load the state after the initial render. So just triggering the store update directly after the first render should be fine. Using nextTick to retrieve and commit the saved state.
 store.subscribe((mutation, state) => {

        if(mutation.type === 'updateCart'){
            localStorage.setItem("cart",JSON.stringify(state.cart))
        }

    })

and in the cart page/component
 mounted(){
    this.$nextTick(() => {

        const cart = JSON.parse(localStorage.getItem("cart"))
        this.$store.commit("updateCart",cart)

    })
  }

Any update on this?


message history

Can I check message history, like sms, maybe it will be more convenient when send several texts.

You should be able to open the SMS app, share from there


Auto-close any open BMC/BDC when ending text (bug 1898053)

/botio test

Moreover, it looks like the FreeText Editor FreeText accessibility must check that the parent structTree id is correct integration test fails on this PR since it fails on both Linux and Windows and I haven't seen this one fail before (so while it could still be an intermittent failure it seems unlikely). We should check if that is indeed caused by this change or if it's a new intermittent.

Having looked at a few real-world PDF documents recently it seems to be fairly common (e.g. with /ActualText entries) for BDC/EMC-blocks to be nested within each other, hence I unfortunately cannot imagine that just closing any open ones on end-text is actually correct in the general case.

Yep, I think I've a fix for it, I'll try to do that this week.


Email Fix: Throw Error when DNS is set but not a Domain

This change adjusts the conditional checks for the AWS Email construct to match the error message:

The "dns" property is only valid when "sender" is a domain

This came up while I was attempting to verify a domain:
export const emailDomain = new sst.aws.Email('Domain', {
    sender: 'my.sub.domain',
    dns: sst.aws.dns({ zone: rootZone.id }),
})

Thanks @san4d! Nice catch 👍


Fix mixed up dimensions in shape comments

I printed the shapes when running it. I noticed these comments disagreed with the actual shapes.

Thanks, good catch! Will also update the blog post.


Using client.close() results in Unhandled Exception: ws not connected

[VERBOSE-2:dart_vm_initializer.cc(41)] Unhandled Exception: ws not connected

Example function:
`void deleteUser() async {
final client = SurrealDB('ws://localhost:8000/rpc');
client.connect();
await client.wait();
await client.use('ns', 'db');
await client.signin('root', 'root');
final delete = await client.query('DELETE user:5');
print(delete);
client.close(); <------ line in question causing issue.
}`


Webpack Resolve Plugins (DirectoryNamedWebpackPlugin)

Version
v1.4.0
Reproduction link
https://github.com/jamesAtcodeninja/nuxt-directory-named
Steps to reproduce
I'm including a resolver plugin in the build.extend property of my nuxt.config.js like so -
build: {
    extend(config) {
       config.resolve.plugins = [
        new DirectoryNamedPlugin({ honorIndex: true }),
      ]
   }
}

but receive this error

  TypeError: resolver.ensureHook is not a function

  - index.js:23 module.exports.doApply
    [sample-project]/[directory-named-webpack-plugin]/index.js:23:25

  - Tapable.js:375 Resolver.apply
    [sample-project]/[tapable]/lib/Tapable.js:375:16

  - ResolverFactory.js:249
    [sample-project]/[nuxt]/[enhanced-resolve]/lib/ResolverFactory.js:249:12

What is expected ?
Should compile normally.
What is actually happening?
TypeError: resolver.ensureHook is not a function

This bug report is available on Nuxt.js community (#c7144)

Seemed to be resolved by https://github.com/shaketbaby/directory-named-webpack-plugin/issues/28


WIP - new BaseLibrary, mixins for Salesforce, plus tests

This change includes a proof of concept for the following features:

Support for keyword mixins in order to organize all of the Salesforce keywords
A new BaseLibrary class to serve as the base class for keyword libraries
Bug fixes to our selenium_retry machinery to support mixins, and to support firefox
A  robot test which tests some of the above

Note: I haven't actually moved or refactored any of the keywords in Salesforce.py. This change makes the changes necessary to support that. The main goal was to be able to use the mixins, which was difficult given the current implementation of the selenium_retry decorator for classes.
TL;DR: Going forward, keyword libraries should inherit from BaseLibrary rather than use the @selenium_retry decorator on a class, and Salesforce.py keywords should be organized into a series of mixin classes for maintainability.
Critical Changes
Changes
Issues Closed

This is old code that is no longer relevant.


resource alignment with CARML & AVM standards

Quoting module guidance

Resource modules: resource modules .... MUST mapped 1:1 to RPs (resource providers) and top level resources

for the Microsoft.Network/virtualNetworks resource provider, this means supporting the virtual network resource and the following child resources:

Microsoft.Network/virtualNetworks/subnets
Microsoft.Network/virtualNetworks/virtualNetworkPeerings

The ddos network protection plan doesn't fit in this hierarchy, should it be in this module?   (I'm aware it is conditionally created and you can just supply the DDOS plan id).
All said, however separating it out is likely to break the following guidance:

Resource modules: resource modules have bring extra value to the end user (can’t just be simple wrappers)

.. so I"m interested to hear AVM core team feedback on how best to approach that.
Currently, Bicep in CARML aligns to the resource provider mapping and treats the ddos protection plan as a separate resource, and alignment to Bicep is desirable,
ref: https://azure.github.io/Azure-Verified-Modules/specs/shared/#id-snfr21---category-publishing---cross-language-collaboration

happy to align with what the CARML implementation.
@Azure/avm-core-team would like to hear your thoughts on this feedback?

Regarding subsets and peering yes, that would be ideal (including the optional reverse peering). DDOS protection I can bring up with the team. Strongly correlated services are usually also ok (like A NIC for a VM, or public IP for a firewall).

got it. Thanks! will remove the DDOS capability then.

...accepting it as an optional input
Please don't remove but let us have the option to enable/disable it.

This has been fixed in PR #20


[WIP] Performance improvements for Fileformat and Presto

Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/316/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/492/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8562/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/351/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8598/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/528/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8650/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/580/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/402/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8651/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/581/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8652/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/403/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/582/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/404/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8653/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/583/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/405/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8654/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/584/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/411/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8661/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/591/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/638/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8708/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8773/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/706/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/665/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/860/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8928/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/667/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8930/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/862/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/668/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8931/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/863/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/882/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8950/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/687/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8951/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/883/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/898/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8966/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/711/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/907/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8975/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/910/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/715/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8979/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/911/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/723/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8986/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/763/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9029/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/961/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/974/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/782/

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/9047/

Build Failed with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/979/

@ravipesala
can you describe the modification?

@ravipesala ...Please add code comments wherever applicable...especially for new classes and to explain major flow or logic...it will make the code more readable and easy to understand

@ravipesala why close this pr?


undefined functions: isImmutable, isCollection, isKeyed, isIndexed, isAssociative, isOrdered, isValueObject

What happened
Hi @leebyron , It seems so bizzare but all is fuctions are undefined.
I checked isImmutable, isCollection, isKeyed, isIndexed, isAssociative, isOrdered and isValueObject, they are all undefined.
I used node v7.5.0. Updated to latest version but still got error.
There is another issue opened (#1165) but reports only isImmutable undefined. Sorry for the direct mention but because of the severity of the situation, I thought It would be better this way.
Regards

What version of immutable.js?

4.0.0-rc.2

Could you show me what code you're writing to get this issue? I can't reproduce. What I'm running:
mkdir test-project
cd test-project
npm init -y
npm install -SE immutable@rc
node
> isImmutable = require('immutable').isImmutable // namespace export
[Function: isImmutable]
> isImmutable = require('immutable').default.isImmutable // default export
[Function: isImmutable]

Note that exporting these functions from the top level is new as of v4.0, so they'll only exist in the RC. Perhaps double check which version you're using?

Sorry, I didn't use rc flag. It was not mentioned in docs on website.
I am really sorry, in a rush I mixed up version numbers. My installed version was ^3.8.1 but mistakenly reported 4.0.0-rc.2.
Please can you update documentation about rc flag so you don't have to deal with similar problems.
Thank you for your time and effort.


Access using access key and secret key

I was wondering if it was possible to use this exporter using aws_access_key_id and aws_secret_access_key rather than using a roleArn? I had this working with the prom/cloudwatch exporter but get the following when trying this for yace with the running locally command in the README.md, using .aws/credentials
"level":"info","msg":"Couldn't describe resources for region eu-west-1: NoCredentialProviders: no valid providers in chain. Deprecated.\n\tFor verbose messaging see aws.Config.CredentialsChainVerboseErrors\n"

I had this problem on my first try : this is likely a lack of permissions :

Make sure if you use MFA for AWS to copy your temp-credentials
Make sure your credentials files is not a symlink (copy the content to a new file to make sure) and has enough permission chmod 664 credentials

Hope it helps

I can confirm it works using the environment variables AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY.

Thank you - I also have got this working now :)


ci: Enable dependabot for GitHub Actions

Once a month update GitHub Actions to latest versions to ensure that CI workflows using latest available versions.

Coverage remained the same at 97.442% when pulling ba58ed509e2ad39b78953903044539977b54724e on ci/dependabot-for-github-actions into d9da3ef7c3146390ec7c827923b69e9a50c992d4 on master.


Image location change

Motivation and Context
Image name is not consistent across travis, pom files etc...
What has changed
build & push image as "eu.gcr.io/census-rm-ci/rm/census-rm-case-service"
How to test?
1. check out branch
2. create a temporary commit by adding the following to .travis.yml file (before "- bash <(curl -s https://codecov.io/bash)" line) in "after_success:"  -
- if [ "$TRAVIS_BRANCH" == "update-image-tag" ] ; then
  docker login -u "${DOCKER_GCP_USERNAME}" -p "${DOCKER_GCP_PASSWORD}" "${DOCKER_GCP_REGISTRY}";
  docker tag "eu.gcr.io/census-rm-ci/rm/census-rm-casesvc" "eu.gcr.io/census-rm-ci/rm/census-rm-casesvc:test";
  docker push "eu.gcr.io/census-rm-ci/rm/census-rm-casesvc:test";
  fi

and replace "branches" section with -
branches:
  only:
  - master
  - update-image-tag

3. push commit to trigger a travis build
4. check "census-rm-ci" registry for tagged image
5. delete tagged image
6. rollback temporary commit changes
Links
Trello

 fixed image location name

 Image location change

Codecov Report

Merging #8 into master will decrease coverage by 1.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
- Coverage   90.74%   89.12%   -1.62%
==========================================
  Files          33       33
  Lines         929      929
  Branches       77       77
==========================================
- Hits          843      828      -15
- Misses         66       84      +18
+ Partials       20       17       -3

Impacted Files
Coverage Δ

...tp/response/casesvc/service/CaseReportService.java
25% <0%> (-75%)
:arrow_down:

...tp/response/casesvc/scheduled/ReportScheduler.java
43.33% <0%> (-30.01%)
:arrow_down:

Continue to review full report at Codecov.

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


Remove outdated requirement: bundle-react

React is now included in bundle-lib, so shouldn't be required here

Oh—@chillu you beat me to it :grinning:


This browser or app may not be secure

After signing (or developing) when trying to run tv.youtube.com I get
This browser or app may not be secure. Learn more

Try using a different browser. If you’re already using a supported browser, you can refresh your screen and try again to sign in.

Needed to add user agent


Neuron starts Jupyter kernel, but can't connect to it

When clicking on "Open Output Pane", I see that in the Task manager as well as via "jupyter notebook list" in the command line, that servers are created on "http://localhost:8888/", "http://localhost:8889/", ... but I still get a Notification in Visual Code saying "Could not start a notebook automatically: Jupyter could not be executed automatically".
I'm running neuron 1.0.4, Visual Code 1.30.2 on Windows 10.

me too


Broken metasync.find

We have totally broken metasync.find, I tested it writing chaining test

@tshemsedinov what is the problem? Does https://github.com/metarhia/MetaSync/pull/75 fix it?

See test retults: https://github.com/metarhia/MetaSync/commit/a9836b92b9479b3a3506c49f1b79f8189b0a393a

https://travis-ci.org/metarhia/MetaSync/jobs/222055726


This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Speed' was slow to respond! See https://git.io/JtMGR for more info.

I'm seeing these errors on the homebridge console.
https://git.io/JtMGR
[3/28/2021, 10:27:58 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Speed' was slow to respond! See https://git.io/JtMGR for more info.
[3/28/2021, 10:27:58 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Direction' was slow to respond! See https://git.io/JtMGR for more info.
[3/28/2021, 10:27:58 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'On' was slow to respond! See https://git.io/JtMGR for more info.
[3/28/2021, 10:27:58 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Brightness' was slow to respond! See https://git.io/JtMGR for more info.
[3/28/2021, 10:28:04 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'On' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
[3/28/2021, 10:28:04 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Speed' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
[3/28/2021, 10:28:04 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Rotation Direction' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
[3/28/2021, 10:28:04 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'On' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.
[3/28/2021, 10:28:04 PM] [homebridge-modern-forms] This plugin slows down Homebridge. The read handler for the characteristic 'Brightness' didn't respond at all!. Please check that you properly call the callback! See https://git.io/JtMGR for more info.

I too am.

Same here. Did anyone try to move the plugin to a child bridge?


Beta

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3c27093). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #107   +/-   ##
=========================================
  Coverage          ?   78.37%
=========================================
  Files             ?       16
  Lines             ?      504
  Branches          ?      118
=========================================
  Hits              ?      395
  Misses            ?       50
  Partials          ?       59

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 3c27093...e2120c6. Read the comment docs.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Setup Deployment ENV

[ ] Prepare deployment server
[ ] Prepare Database (With Devops)
[ ] Update github actions
[ ] Prepare Redis Server

@mhmdksh Note, it must be able to communicate with authentication server

@aminlatifi  please assignee to me this issue. I couldn't since GM Digital Ocean account has MFA. Once is disable this will be completed by tomorrow <3

@Rolazo  @mhmdksh Please add secrets needed for the test pipeline, like what we have in the impact-graph.

Closing this, This has been done!


Contribution rules

Closes #95
I've already referenced the develop branch in Contribution rules. What do you think about that?

Please add task number to your commit message.
wdyt? :)

5. Please add task number to your commit message.
   wdyt? :)

Good point, added.


argparse: Add use_argument_group=True

What does this PR do?
Fixes #6086
Closes #5851
Fixes #3076
Fixes #5875
Before submitting

[/] Was this discussed/approved via a GitHub issue? Not really, but simple 'nuff?
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
(N/A) Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Mayyyyybe. :hand_over_mouth:

Hey @EricCousineau-TRI,
You have some failing tests. Would you mind addressing them,  so we can merge the PR ?
Best,
T.C

You have some failing tests. Would you mind addressing them, so we can merge the PR ?

I think I fixed 'em! :crossed_fingers:

Passed (finally!)

Confused by codecov failures:

https://github.com/PyTorchLightning/pytorch-lightning/pull/6088/checks?check_run_id=2015464436
https://github.com/PyTorchLightning/pytorch-lightning/pull/6088/checks?check_run_id=2015464216

Will try re-merging master.

Sadly, codecov hasn't been working properly lately, feel free to ignore it

Thanks! Also, seems like there're some other failures on master regarding the tpu accelerator - also seem to be caused by something outside of this PR?

Ẇewt! Thanks for y'all's patience!


Support websockets

Right now we do not support websockets.
We should!

This will be interesting. Proxying websockets on arbitrary URLs may be tricky, but we should definitely support this.
BTW, if we figure this out, then my last reason to have used CHP instead of writing a simple proxy with tornado will go away.


Conventions redo

This is a big renaming / reorganization of many project files. This is useful because the project has grown such that each package is completely different from all the others and there is a mess of dependencies and launch files strewn everywhere.
Some of the key things it does:

merge some similar packages together
rename packages with a consistent scheme
enforce conventions within packages

Future things:

enforce standardized readme
continue cleaning up launch files
remove heavily depreciated code

Work that must be done before merging:

ensure there are no large breaking changes (small ones are OK, we can fix them as we go)
ensure the merging / renaming / standardization is OK with the code owners

Timeline

this should be merged before we undertake any new big changes this semester (ideally in the next few days)

Everything builds properly and the simulator, visualizer, and trainer all run:
catkin clean
catkin build
roslaunch rktl_autonomy rocket_league.launch

I think the rktl_perception package needs to be looked at. There is just a lot of C code and it is hard to tell what is responsible for what, and which parts are even still in use.

I think the rktl_perception package needs to be looked at. There is just a lot of C code and it is hard to tell what is responsible for what, and which parts are even still in use.

I just removed the unused files, code still needs refactor but that's a different conversation


feat(compiler-sfc): support transforming asset urls with full base url.

This feature allows for fully qualified URLs (including protocol, host, and port) to be provided as options.base when transforming template asset urls. Previously, passing full URLs was unsupported, since path.join() does not work well with full URLs.
This is my first contribution to vue 3 and I'm not sure if the lack of support for this was intentional or not. I am looking forward to your feedback.

It's common to omit protocol in assets. Can you add a test case for //localhost?

@znck good catch - adding that test revealed a bug. I have pushed a new commit with the test and the corresponding fix.


SAK-50059 - Move sakai-service-worker.js out of the ROOT webapp

I am really putting this up for review, consideration, and discussion.

This is moved to Matt to do a better solution


Complete Design Testing

Live session to test flow and interfaces while making minor design tweaks

Expiration Date is too far to the left:

Would be nice to tighten up this formatting but it works for now.

@skuhlmann if we are going to break these onto separate rows Voting Tokens Requested should be below Tribute Amount

[ ] Add symbol to Non-Voting Tokens Requested

https://bison-narwhal.vercel.app/#/molochV3/0x64/0x8577c1a3caedd9a6bfa431176a9b94474cdffd9c/proposal/28


[docs] fix broken links

Thank you for your commit! I added it internally and it should make it to master sometime soon.
I'm working with our IT on syncing public master branch more frequently.

Fixed in https://github.com/ROCm/rdc/commit/a3ac4bac219dc06339a55e24def9115f63408578 and then in https://github.com/ROCm/rdc/commit/f6ace9fa14fbd4792736d41ef252e74e032cbc99 after another update.
Note - we have a https://github.com/ROCm/rdc/tree/develop branch now!

Thanks @dmitrii-galantsev !


[WIP] sync values types

@howardjohn the reason is that we are still exposing these options(k8s settings, enablement) in values.yaml, although they should only reside in IstioControlPlane API later. Also once we remove those settings from values_types.proto, we should add corresponding error messages in validation to let user know that these should be set under IstioControlPlane API section instead of values.yaml pass through

ok, is it possible to mark as deprecated now?

@howardjohn Yep I think we need to do that but gradually. Existing users are already familiar with helm installation so they may tend to put all overlay into values part, they may have to apply force flag every time if we remove all overlapped IstioControlPlane fields now. Instead, I think we can add some warning message in the validation logic now to let user know that things they are overlaying would be deprecated soon and what would be the correct path to overlay. Then in 1.5 maybe? we can deprecate them. WDYT?

By deprecate I don't mean remove, I mean mark as deprecated. For example in the proto:
 bool java_generate_equals_and_hash = 20 [deprecated=true];

@howardjohn I push new commit to mark k8s related settings as deprecated, but that servers mainly the documentation purpose for developer. We may still need to add some message in the validation function for user to know about it.

@howardjohn @ostromart PTAL again

In response to a cherrypick label: #618 failed to apply on top of branch "release-1.4":
Using index info to reconstruct a base tree...
M	pkg/apis/istio/v1alpha1/values_types.proto
Falling back to patching base and 3-way merge...
Auto-merging pkg/apis/istio/v1alpha1/values_types.proto
Applying: add more missing fields
error: Failed to merge in the changes.
Using index info to reconstruct a base tree...
M	pkg/apis/istio/v1alpha1/v1alpha1.pb.html
M	pkg/apis/istio/v1alpha1/values_types.pb.go
M	pkg/apis/istio/v1alpha1/values_types.proto
M	python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
Falling back to patching base and 3-way merge...
Auto-merging python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
CONFLICT (content): Merge conflict in python/istio_api/pkg/apis/istio/v1alpha1/values_types_pb2.py
Auto-merging pkg/apis/istio/v1alpha1/values_types.proto
CONFLICT (content): Merge conflict in pkg/apis/istio/v1alpha1/values_types.proto
Auto-merging pkg/apis/istio/v1alpha1/values_types.pb.go
CONFLICT (content): Merge conflict in pkg/apis/istio/v1alpha1/values_types.pb.go
Auto-merging pkg/apis/istio/v1alpha1/v1alpha1.pb.html
Patch failed at 0002 add more missing fields


Question: Documentation?

Hey!
Does that package have any more documentation than just the README file?
Cheers,
Damian

Hey Damian! Thanks for the question :) Is there specific documentation type that you are asking about? Like a website, perhaps? Or, more docstring within Hera's code?

Website with more than a comparison example, some use cases, potentially a tutorial, documentation of the API in form of collected docstrings.
I wanted to evaluate usage of hera in my team but without a centralized documentation it will be quite hard to convince anyone to use it.

Thank you for the feedback @heavelock :) I chatted with @Trollgeir about this and we'll look into spinning up a ReadTheDocs page for Hera!

That's great to hear! Looking forward for that!
Cheers,
Damian

The first docs series is available at https://hera-workflows.readthedocs.io/en/latest/!


Pendrive don't install cobalt ... only format itself ...

I tried to install the Cobalt 1.2 via USB with a Pendrive, but to the run, the script formated the pendrive ... the pendrive formated itself!!! Just it did that procedure and clean the same local when was the Cobalt Installer. So, how do I must to do the installation on other pendrive, in other port USB???? I saw the file SETUP.BAT, but how do a lot time I don't work with DOS, I don't know how to do now. Do you can help me? P.S.: I installed the FreeDOS  and it work perfectly, using two pendrives: one with installer and other for the final system ... in that case, I used the file IMG, from the site official FreeDOS 1.2, ok! I'll await your explanation on how to install Cobalt 1.2 ...

Sorry, I'm not working on Cobalt anymore.


Firefox for Android support

Any possibility to add support for Firefox for Android?

I am definitely open to it! I will have to see if the current version can be used on Android without issues.

+1!

I'm definitely open to this too!


pull request for e2e test at 11/24/2016 11:26:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Please update to nodejs8.10

The current version of node in the template is no longer supported.  So just update to nodejs8.10.  Tested this and it works well.
Thanks!!

@r2ad Thank you!
I'm not sure how soon I'll get around to taking care of this, but if somebody wants to submit a PR, I'll review.
Having a second person test and say it worked would also speed things up.

Here's a PR @ehammond, @r2ad: https://github.com/alestic/lambdash/pull/8

Thanks. Merged!