These labels still exist in annotations wrong label count bug

Describe the bug
Wrong label count of drafts and annotation displayed when trying to change label name in label config.
To Reproduce
Steps to reproduce the behavior:

Create a project and add 2 sample tasks and create an object detection project
Create one label on one of the image and wait for draft to be saved
Create 2 or 3 labels on the same image and wait for draft to be saved, don't submit the task
Now delete all the labels one by one and let draft save
Now go to labelling config and change the annotated label name, you will see 4 or 5 label count for the label in drafts ideally i should be able to change the name of label as there are no labels in the project

Expected behavior
Projects correct label count should be displayed when user tries to change the label in labelling config
Screenshots
(https://github.com/HumanSignal/label-studio/assets/63253704/418dd067-caf7-4035-b9b8-95909e07b03b)
Environment (please complete the following information):

ubuntu, chrome
Label Studio Version 1.11.0

Hi @AjayVarmaK,
Thanks for reporting this issue! I just wanted to let you know that a feature that would unblock you in this situation is currently in the works: https://github.com/HumanSignal/label-studio/pull/5432. Unfortunately there are deeper issues with how we are calculating project summary data, which leads to what you're describing, and while we have those issues tracked, they're not on our immediate roadmap. In the case of a miscalculation though, resetting your project summary data would fix the issue.
Cheers, and thanks for using Label Studio,
Jo


refactor(cleanup): remove force-fetch-alert-labels-from-metaphysics ff

The type of this PR is: TYPE

This PR solves FX-4671
Description
remove force-fetch-alert-labels-from-metaphysics ff

converted to Draft, will address later (switched focus to artwork lists for now)


Update ES lint and other JS tools

Current coverage is 91.74% (diff: 100%)

No coverage report found for master at 6b24756.

Powered by Codecov. Last update 6b24756...2ad1448


Llama has wrong abilility

Describe your idea, include images if necessary
the Llama has the wrong ability when the ability should be Llama spit not horse jump

same with trader llamas

fixed as of bf4eb2d2247004ffc1785c6a23094cfa9a61ec4a


Only wallet

https://getdebrief.atlassian.net/browse/SLAS-358

Great first pass! I think we need to remove the useSlashAuth as this is an external hook that we should try not to use in the components.


Dynamic Property

Sometimes it's useful that fixture value from computed value.
For this, enable not value initialized  to use getProperty with instance value accessible.

It may  be good to implement using Proxy.
https://stackoverflow.com/questions/7891937/is-it-possible-to-implement-dynamic-getters-setters-in-javascript

Current, this is supported by getter property, but it can be refactored better.


Add heading slash to ja content link

This PR adds heading slash "/" on ja content link.
The link is 404 without heading slash, that's why I create this commit.
the link is expanded as followings if we use [something](ja/docs/something) style:
https://kubernetes.io/ja/docs/reference/ja/docs/reference/setup-tools/kubeadm/ (<-Double "ja" in URL.)

/assign @t-inu

/assign @nasa9084
Could you review this?

This could be titled “Add leading slash to ja content links”
However:
/retitle [ja] Fix broken links to localized pages

Thank you for retitling of this PR :smile:


Add Grav version to PHPdebugbar

Adds "Grav vx.y.z" as first line info to Messages tab on debug bar

Sheesh, sorry about those old commits, that was me trying to sync my fork with upstream. Guess I did that wrong.

Oh man, failed integration test, this is when Github just gets in the way. What do you recommend, a rebase? just delete my old fork and fork again? would not be an issue if it would allow me to create a second fork.
Or can you somehow work with this?
What I did to get to this point is:

cloned my fork repo
added upstream remote
fetched and merged my clone with upstream
pushed to origin (my fork)
committed changes, pushed to my fork

I think it failed due to a 'blip' on travis.. try another dummy commit to kick off another test.

Starting to really dislike that Travis guy ..
Forgot about GH inline edits, so abandoning this and refer https://github.com/getgrav/grav/pull/2106


Motor Joints Omitted from Constraints Deep Dive

No information or examples regarding setting motors on 6dof or spring joints in physicsV2 deep dive.
I have a nice clean example showing how to use motors here Motor Joint Example. Happy to PR this if it's suitable!

I love it, it would be great, thanks :-)

@CedricGuillemet can you check the example ?

This is a great example @Rex-tr909 can't wait to see the PR! :)

#1218


WEEK6CHALLENGE sergei.haller.u-niq

https://people.sap.com/sergei.haller.u-niq

Thanks a lot for your contribution. The solution looks great and I'll close this PR now!
You should receive your badge soon.


Add action executor

Fixes #36

Codecov Report

Merging #38 into feature/generic_process_interpreter_service will increase coverage by 3.39%.
The diff coverage is 58.97%.

@@                               Coverage Diff                               @@
##           feature/generic_process_interpreter_service      #38      +/-   ##
===============================================================================
+ Coverage                                        36.20%   39.59%   +3.39%
===============================================================================
  Files                                               24       31       +7
  Lines                                              732      846     +114
===============================================================================
+ Hits                                               265      335      +70
- Misses                                             467      511      +44

Impacted Files
Coverage Δ

action-executor/src/iface_impl/trigger_reader.rs
0.00% <0.00%> (ø)

process/src/process/config/executor.rs
0.00% <0.00%> (ø)

process/src/process/config/mod.rs
75.00% <0.00%> (-6.82%)
:arrow_down:

trigger-interpreter/src/manager.rs
0.00% <0.00%> (ø)

action-executor/src/exec.rs
42.85% <42.85%> (ø)

action-executor/src/system.rs
65.00% <65.00%> (ø)

action-executor/src/manager.rs
81.48% <81.48%> (ø)

action-executor/src/tests/mock.rs
87.50% <87.50%> (ø)

action-executor/src/tests/executor.rs
100.00% <100.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


Add caching to datanode endpoints required for transaction receipt enrichment

Chore
Add caching to datanode endpoints required for transaction receipt enrichment
Tasks

[ ] Define caching strategy for each endpoint
[ ] Build it
[ ] Priority order

[ ]  Assets
[ ] Markets
[ ] Other?

Notes

Dupe, done


[FEATURE] Upgrade TinkerPop version to 3.5.2.

Upgrade TinkerPop version to 3.5.2.
Currently it is 3.4.8 and this has issues with OpenJDK17 which some users are using.

Superseded by #230

TinkerPop has been upgraded to 3.6.4 with #230.


[BOW-54] Properly highlight when an asset failed to load

Codecov Report
Attention: 21 lines in your changes are missing coverage. Please review.

Comparison is base (5063ff6) 24.60% compared to head (46f1297) 24.47%.

Files
Patch %
Lines

...s/[show_id]/rundown/[rundown_id]/RundownAssets.tsx
0.00%
15 Missing :warning:

jobrunner/src/jobs/LoadAssetJob.ts
0.00%
6 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #192      +/-   ##
==========================================
- Coverage   24.60%   24.47%   -0.13%
==========================================
  Files          83       83
  Lines        3243     3260      +17
  Branches      459      464       +5
==========================================
  Hits          798      798
- Misses       2445     2462      +17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Remove BuildResult - CLOUDBLD-8975

Detailed description is in the individual commit message.
Maintainers will complete the following section

[x] Commit messages are descriptive enough
[ ] Code coverage from testing does not decrease and new code is covered
[ ] Python type annotations added to new code
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request has a link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

Note: This pr is related to #1843. If it gets merged firstly, rebase this pr. For the group_manifests.

Rebased.

Mostly LGTM
Should we also remove build result handling from the ContextDir?

@chmeliik Updated. PTAL.


Update memory cgroup kernel cmdline option for newer kernels, remove …

…errant +, fixes #40

Thanks @ecliptik, that's indeed a great improvement!


Handle dropping capabilties correctly when running as non root user

Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

@lsm5 @giuseppe @mheon @vrothberg @TomSweeneyRedHat @baude PTAL

Can we get a test for this?

@mheon I did add a test for this?

@rhatdan Didn't see one when I commented, sorry

/lgtm

/hold

Tests are red

/lgtm

Fixes: https://github.com/containers/libpod/issues/6645


Add watchium tool

We have our own ETL :)
If you need an invite code, ping me with the chat box on vapolia.eu or watch the video https://youtu.be/vU6WrWYHOzM there is a code inside.

We have our own ETL :)

love this.


added multiavatar api to existing random avatar generators

Issue
Closes #46
Description
Added Multiavatar API to the existing random avatar generator options.

Thank you for your contribution, @aybeedee! You can take a look at the other issues if you are interested in contributing further.

@TheRustyPickle I'll definitely go through them if I get more time. Also, I'm very new to Rust. What's up with the emphasis on snake case? Even the compiler suggests me to use it! I'm very used to camel and pascal case myself. Is there a particular reason for the preference? Or am I just too much of a newbie to this ecosystem.


The destroy method is not being called

I have a feeling this library slows down my application after showing a lot of animations. Now I am seeing that the lottie-web library has a destroy method: https://github.com/airbnb/lottie-web#destroy but I don't see this method being called here on destroy of the component. I am not sure if that is needed?
Simply adding
    	return () => {
	 		animation.destroy()
		}

to the onMount function would fix it.
See: https://github.com/MattDavis00/lottie-svelte/pull/22

Fixed and merged in https://github.com/MattDavis00/lottie-svelte/pull/22

Fix is included in the lastest release:
https://www.npmjs.com/package/lottie-svelte/v/1.3.7


Remove AD from cattle

https://github.com/rancher/rancher/issues/6378
https://github.com/rancher/rancher-auth-service/pull/11

You need to add the ldap_user and ldap_group to the property "resources/content/cattle-global.properties:auth.service.external.id.types"

Also remove the "ldapconfig" fields from admin-auth.json


Using time() in unit tests leads to inconsistent results depending on for

Using time() in an alert leads to surprising unit-test results. Although I have a full writeup of my usecase, I've narrowed the problem down to a minimal failing test case here.
What did you do?
Ran promtool test rules on a file with this test.yml:
rule_files:
  - alert.rules

evaluation_interval: 1s

tests:
  - name: simple
    interval: 1s
    promql_expr_test:
      - expr: time() > bool 5
        eval_time: 1s
        exp_samples:
          - value: 0
      - expr: time() > bool 5
        eval_time: 10s
        exp_samples:
          - value: 1
    alert_rule_test:
      - eval_time: 1s
        alertname: SimpleAlert
        exp_alerts: []
      - eval_time: 10s
        alertname: SimpleAlert
        exp_alerts:
          - exp_labels:
              severity: critical

The contents of alert.rules are:
groups:
  - name: simple
    rules:
      - alert: SimpleAlert
        expr: time() > bool 5
        labels:
          severity: critical

What did you expect to see?
Either all tests should succeed, or - if the alert_rule_test with eval_time: 1s fails, so should the promql_expr_test at the same time, as they would appear to be testing the same condition.
Furthermore, modifying the alert rule to have for: 1s should not change the outcome.
What did you see instead? Under which circumstances?
As-is, I get the following failure (note that the promql_expr_test does not fail):
Unit Testing:  /etc/prometheus/test2.yml
  FAILED:
    name: simple,
    alertname:SimpleAlert, time:1s,
        exp:"[]",
        got:"[Labels:{alertname=\"SimpleAlert\", severity=\"critical\"} Annotations:{}]"

Furthermore, if I add the line for: 1s to the alert rule, the failure changes to this:
Unit Testing:  /etc/prometheus/test2.yml
  FAILED:
    name: simple,
    alertname:SimpleAlert, time:10s,
        exp:"[Labels:{alertname=\"SimpleAlert\", severity=\"critical\"} Annotations:{}]",
        got:"[]"

Environment

System information:
Linux 5.4.0-74-generic x86_64

Prometheus version:
prometheus, version 2.27.1 (branch: HEAD, revision: db7f0bcec27bd8aeebad6b08ac849516efa9ae02)
  build user:       root@fd804fbd4f25
  build date:       20210518-14:17:54
  go version:       go1.16.4
  platform:         linux/amd64

cc @dgl can you please have a look at this? Thanks.

It looks like part of this is answered on https://stackoverflow.com/questions/67835221/prometheus-unit-test-failing-when-adding-for-to-alert
I don't see a failure at 10s, if I add a for: 1s though.

Thank you for your response.
With the StackOverflow answer, the issue was apparently of data staleness - this shouldn't occur with a 1s evaluation interval, should it?
Also, could you clarify why "using the > bool operator in an alert will make the alert fire [...] regardless of the value"?
Thank you for the tip, I'll give it a whirl.

Ahh, here's an important note: An alert with expr: 0 will always fire (!), and because, for example, 1 > bool 2 returns 0, it will also always fire if set as expr. Changing the alert to have expr: time() > vector(5) (...or expr: vector(time) > 5, in fact) makes the whole thing work correctly, as the result of the false evaluation is an empty vector, which does not trigger an alert. This is more realistic, too, as it more closely matches my stackoverflow question (the input would be a vector of which-backup-succeeded-when).


docstyle.convention = "google" should ignore D401

This is tested on ruff 0.0.245
e.g. this example from the google style guide
test.py:
def fetch_smalltable_rows(table_handle: smalltable.Table,
                          keys: Sequence[Union[bytes, str]],
                          require_all_keys: bool = False,
) -> Mapping[bytes, tuple[str, ...]]:
    """Fetches rows from a Smalltable.

    Retrieves rows pertaining to the given keys from the Table instance
    represented by table_handle.  String keys will be UTF-8 encoded.

    Args:
        table_handle: An open smalltable.Table instance.
        keys: A sequence of strings representing the key of each table
          row to fetch.  String keys will be UTF-8 encoded.
        require_all_keys: If True only rows with values set for all keys will be
          returned.

    Returns:
        A dict mapping keys to the corresponding table row data
        fetched. Each row is represented as a tuple of strings. For
        example:

        {b'Serak': ('Rigel VII', 'Preparer'),
         b'Zim': ('Irk', 'Invader'),
         b'Lrrr': ('Omicron Persei 8', 'Emperor')}

        Returned keys are always bytes.  If a key from the keys argument is
        missing from the dictionary, then that row was not found in the
        table (and require_all_keys must have been False).

    Raises:
        IOError: An error occurred accessing the smalltable.
    """

pyproject.toml
[tool.ruff]
select = ["D"]
pydocstyle.convention = "google"

then ruff test.py gives
test.py:8:5: D401 First line of docstring should be in imperative mood: "Fetches rows from a Smalltable."
Found 1 error.

I'll note that pydocstyle agrees D401 is ignored for google style: http://www.pydocstyle.org/en/stable/error_codes.html#default-conventions
Thanks for all of your hard work developing ruff! This tool has made a big improvement in my workflow.

Oh this might be an oversight... Will confirm. You're right that it should be ignored.

Thanks, I think we added this rule after we ported the conventions, so it didn't exist at the time!


Test Import/Export

Test the IPortable interface

There is a bug in the DNN Import routine that ignores the specific settings of a module, see https://github.com/dnnsoftware/Dnn.Platform/issues/2636


chore(deps): update dependency hypothesis to v6.70.1

This PR contains the following updates:

Package
Type
Update
Change

hypothesis (source, changelog)
dev
patch
6.70.0 -> 6.70.1

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will now ignore this update (^6.61.0). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Too much latency compared to oh-my-posh2

Prerequisites

[x] I have read and understand the CONTRIBUTING guide
[x] I looked for duplicate issues before submitting this one

Description
There is too much latency compared to oh-my-posh2, especcialy when
typing the clear command.
Environment

Oh my Posh version: 3.11.0
Operating System: Windows 10 20H2
Shell: Powershell Core 7.0.3
Terminal: Windows Terminal

Steps to Reproduce

Type clear command in the terminal.
You will see some latency until the shell bar pops up.

Expected behavior:
Little to no latency.
Actual behavior:
Some latency.

What theme?

What theme?

I was using the Star theme.

it seems pretty responsive to me.  But this might be causing an issue on Windows?
{
          "type": "command",
          "style": "plain",
          "foreground": "#C49EE9",
          "properties": {
            "shell": "bash",
            "command": "find . -name package.json -depth 1 -exec echo \" <#ffffff>via node</>\" \\;"
          }
        },

The command segment seemed like a good idea but I'm starting to resent it. I'm going to look for a different solution. I haven't noticed latency, but I'm mostly on zsh which might explain that. The only segment that has a noticeable impact is spotify. And as @nwykes states, in theory command is the only one that can have an impact this negative.

For star specifically I will replace the command with the default node segment functionality as that one makes no sense anymore. I think command is neat when you know what you're dealing with, but the out-of-the-box themes shouldn't be using this.

@aykxt can you check to see if the latest version offers a different experience?

@aykxt can you check to see if the latest version offers a different experience?

Seems like it is a little bit faster now. But there is still some delay when I'm in a git repository.

I'll see if we can speed up things a bit in that regard.

@aykxt looking forward to seeing if the latest update makes a difference. If not, can you check if you're on the latest git version?

@aykxt looking forward to seeing if the latest update makes a difference. If not, can you check if you're on the latest git version?

Just tried it, it's much faster now!

Do we close this one for now? I'm not sure there's much I can do to make it even faster, the layout should also keep it that way even when extending to offer support for revert/bisect.

@aykxt can you check to see if the latest version offers a different experience?

I think we can close it, it's much faster now.


Custom markdown styles not working?

Refs: https://github.com/Microsoft/vscode/issues/7945
I tried to configure a custom CSS with 3 different approaches:

relative file
absolute file path
URL

I do not seem to get it to work:

It does not seem to work to me neither

I improved the documentation in the user settings. You can use URLs like file:///your file path or http://.... I also made a couple changes to ensure we look workspace relative if there is an open workspace.

Works for me when using a local file path or absolute path, but not when using a URL like https://gist.githubusercontent.com/bpasero/5574e2cb0cd47aaf9870f956adcc7ff5/raw/bf2a775e5c71eda058b534ae37fa03b930026a60/gistfile1.css or https://raw.githubusercontent.com/sindresorhus/github-markdown-css/gh-pages/github-markdown.css

What I'm seeing is that the stylesheet is correctly added to the html page that is being displayed. However, when I open the dev tools inside the webview, the stylesheets don't seem to have any content.

The issue is that GitHub responses with X-Frame-Options: deny.

Ah ok, sorry about that...

@kieferrm @bpasero @sandy081  I figured out a way. If you paste your gist url(i.e https://gist.githubusercontent.com/wghglory/9b6e502a0c81d135e83fce87be83a18c/raw/7a9bdd06e9f2fe4a53d84900d5ac77912f939873/github-markdown.css) here: https://rawgit.com, it will generate a new url for you. You can paste that into your settings.json.
"markdown.styles": [
    // your gist url, not working
    // "https://gist.githubusercontent.com/wghglory/9b6e502a0c81d135e83fce87be83a18c/raw/7a9bdd06e9f2fe4a53d84900d5ac77912f939873/github-markdown.css"

    // use this generated new url
    "https://cdn.rawgit.com/wghglory/9b6e502a0c81d135e83fce87be83a18c/raw/7a9bdd06e9f2fe4a53d84900d5ac77912f939873/github-markdown.css"
],

In macOS, the file:///your file path seems not working....

@Molunerfinn use file:/your path

@wghglory still not working....

@Molunerfinn I use the following in my settings:
"markdown.styles": [
        "/Users/kieferrm/Development/markdown.css"
    ]


disable change price from explorer

Thank you for your contribution to the Koda - Generative Art Marketplace.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[ ] Refactoring

Context

[x] Closes #7879
[ ] Requires deployment <worker/stick/speck>

Before submitting pull request, please make sure:

[ ] My contribution builds clean without any errors or warnings
[ ] I've merged recent default branch -- main and I've no conflicts
[ ] I've tried to respect high code quality standards
[ ] I've didn't break any original functionality

Optional

[x] I've tested it at </ahp/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at Koda Ecosystem Telegram?

Screenshot 📸

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

can't capture cursor on screenshot :laughing: it's on the left Christmas City card

hey, it's still enabled on RMRK2/1, can you also remove it there? thank you


Allow withState accept initialState as function of props

Also I think the same must be done for defaultProps, if you agree with this
@defaultProps(props => ({
...blablabla
}))

Thanks!


updates from May Case analysis

@srinathvasireddy : Thanks for your contribution! The author, @genlin, has been notified to review your proposed change.

@srinathvasireddy, Thanks for submitting the pull request. I does not see the change in this pull request (except changing a message's format to bold). Could you share mode detail?


feat(docs): RFC on Searching for Focus Component on UI

What kind of change does this PR introduce?
An RFC in attempt to solve https://github.com/OpenAdaptAI/OpenAdapt/issues/421 and to provide a direction for https://github.com/OpenAdaptAI/OpenAdapt/issues/442

Summary

When working on https://github.com/OpenAdaptAI/OpenAdapt/issues/421, I realize that there are a few missing pieces of the puzzle in an attempt to evaluate a prediction of the models. Mainly, we do not have yet the definition of "how good and useful" a prediction is. Without this, it is hard to evaluate and see if a model provides correct prediction or not.
Somehow, this problem is entangled with the fact that we need to come up with a prompt strategy and a strategy to summarize a window data into useful hints for the models.
This PR provides a baseline for discussion on how we can decompose the windows into useful elements which later can serve not only model evaluation process, but also potentially prompt building as well as to some extend, RFLHF process.
Checklist

[ ] My code follows the style guidelines of OpenAdapt
[ ] I have performed a self-review of my code
[ ] If applicable, I have added tests to prove my fix is functional/effective
[ ] I have linted my code locally prior to submission
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (e.g. README.md, requirements.txt)
[ ] New and existing unit tests pass locally with my changes

How can your code be run and tested?

The core RFC file should be reviewed/discussed. No code is written yet at the moment
Other information

Not sure where to put the RFC file, hence I created it in openadapt.docs.
It is more convenient to just read the RFC file

Thank you @LaPetiteSouris ! As per our conversation last week, this sounds directionally correct. Before implementing, can you please coordinate with @FFFiend to parallelize the work? 🙏
The first step of the problem formulation is:

At a single time step, given an action and a tree, find the target in the tree

@FFFiend can you please suggest some next steps once this has been implemented?

formulation

Thanks
With pleasure.  This week I'll come up with a kind of skeleton code proposals to make sure the this is compatible with future fine-tuning, completion provider. If @FFFiend  has time, please be one of the reviewers for that coming code/skeleton proposal for the next steps.
@FFFiend how far are you off to finish FineTuning. Can I wrap up another round of review ?
I would love to review CompletionProvider as well when it is available.


As per security, we need to configure readiness-probe in Istio-Operator deployment but readiness probe is getting fail

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
As per security, we need to configure readiness-probe in Istio-Operator deployment but readiness probe is getting fail, Please suggest readiness config for istio-operator
Version
1.20.2

Additional Information
No response

The IstioOperator has now been removed https://istio.io/latest/blog/2024/in-cluster-operator-deprecation-announcement/


When window resizing, coverflow effect will fall into chaos

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.2.2.

Platform/Target and Browser Versions: Chrome in Windows.

Live Link or JSFiddle/Codepen or website with isssue: https://jsfiddle.net/93gpfyxj.

What you did
When I change the mobile's orientation, or even just change the window's size, the cover flow will lose control.
Expected Behavior
No matter how I change the window's size, or rotate the mobile, swiper should work correctly.
Actual Behavior
Totally lose control.
P.S. Remember, an issue is not the place to ask questions. You can use Stack Overflow
for that.
Before you open an issue, please check if a similar issue already exists or has been closed before.

Hi you should use vendor prefix in transform style in this method. eg: webkitTransform  In old version ios phone and android phone, update method in coverflowEffect mode has bug。


Correct collectd python exception

Collectd python plugin was throwing exception from time to time:
Unhandled python exception in read callback: ValueError: invalid literal for int() with base 10: '0.0000'
This disables plugin for 120 seconds.
Changing int to float at line 275 corrects the problem

Also fixed by #8.


fix(docs): update helm instructions

Signed-off-by: kmova<EMAIL_ADDRESS>As a pre-step towards 3.0 and enabling CSI drivers for jiva and
cstor, made the necessary documentation and values.yaml updates.
The premise is that openebs chart will continue its current templates
of ndm and local provisioner, while jiva and cstor will be deprecated
in favor of the corresponding csi charts.
Once 3.0 is released, the recommended way to install cstor, jiva and other
engines would be to enable via this umbrella chart. If there is a strong
use-case for using only one type of engine in a cluster - the user can opt to
install the sub-charts directly.
Trimmed down the configuration options in the README to the commonly used
options.
Also, added Shovan as a maintainer.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/openebs])

As NDM and localpv dependency chart are removed we need to fix

https://github.com/openebs/charts/blob/d803266702c555bcc16b60f6527378cfa4a0eb24/charts/openebs/templates/ndm/cm-node-disk-manager.yaml#L2-L3
https://github.com/openebs/charts/blob/d803266702c555bcc16b60f6527378cfa4a0eb24/charts/openebs/templates/ndm/deployment-ndm-operator.yaml#L2-L3
https://github.com/openebs/charts/blob/d803266702c555bcc16b60f6527378cfa4a0eb24/charts/openebs/templates/ndm/daemonset-ndm.yaml#L2-L3
https://github.com/openebs/charts/blob/d803266702c555bcc16b60f6527378cfa4a0eb24/charts/openebs/templates/deployment-local-provisioner.yaml#L2-L3

We need to remove some checks else LGTM.

Changes merged by:

https://github.com/openebs/charts/pull/253
https://github.com/openebs/charts/pull/278


[MOB-14763] Test app for Core (Lifecycle / Assurance support for now)

A simple test app with support for Assurance use and Lifecycle Edge in order to perform manual testing. Skeleton code has been setup for multiple views support for Core extensions as well as per this issue

@cdhoffmann can you please include some screenshots in the PR description?


[Query][Runtime][Critical] Is it possible to convert a string to an int/long/float/double

In docs its not possible to find a way of doing $subject.

Added in PR: #1076


Rename PointLight2D to SpriteLight2D

See https://github.com/godotengine/godot/issues/16863#issuecomment-869587241
And https://github.com/godotengine/godot/pull/50066

Looking at this comment, I'm not sure this isn't a misspelling of "SpotLight".

But doesn't this miss the point of the "2D/3D" suffixes? There isn't any SpriteLight3D.

@YeldhamDev There isn't (currently) a PointLight3D either, but it's still PointLight2D. There isn't a CircleShape3D, but there is a CircleShape2D. I think it's better to explicit, especially since there is already a Sprite3D.

Looking at this comment, I'm not sure this isn't a misspelling of "SpotLight".

It's not. It's a light based off of a sprite, a texture. But SpotLight would work in this case as well.

@YeldhamDev Maybe we can keep the suffix to refer to the kind of light our light extends, Light2D or Light3D, even if for the current object itself it doesn't make much difference.

Rebased.

There was no consensus for this in today's PR meeting; if we're going to rename this, we need to do this via proposal with a clear consensus for a new name.


python310Packages.pyialarm: 2.0.0 -> 2.2.0

Description of changes
https://github.com/RyuzakiKK/pyialarm/releases/tag/v2.2.0

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 181590 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.ialarm
python310Packages.dicttoxml2
python310Packages.pyialarm
python39Packages.dicttoxml2
python39Packages.pyialarm


STRING_CTE incompatible with C+++

Find a way to make STRING_CTE compatible with C++.
The difficulty is generating a constant with its char[] array field aligned to an integer.

It seems that the code at https://ideone.com/IDEdY0 do the job. It seems complicated though.


File "av/video/frame.pyx", line 336, in av.video.frame.VideoFrame.from_ndarray AssertionError

Hi, I'm not able to resolve the following error when I'm using VideoFrame.from_ndarray(img). Could anyone help me understand why this is happening and how to fix it??

https://github.com/PyAV-Org/PyAV/blob/9ac05d9ac902d71ecb2fe80f04dcae454008378c/av/video/frame.pyx#L336
Your array has the wrong shape for rgb24 or bgr24.
array = numpy.random.randint(0, 256, size=(480, 640, 3), dtype=numpy.uint8)


[Feature] Limit the overall proxySink message and job number to prevent oom

Description
[Feature] Limit the overall proxySink message and job number to prevent oom
Use case
[Feature] Limit the overall proxySink message and job number to prevent oom
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

no need for this feature anymore


fix backslash in f-string

Fix https://github.com/nedbat/coveragepy/issues/1836

@the-13th-letter Thanks for your careful tests in https://github.com/nedbat/coveragepy/pull/1828#issuecomment-2310266578 !
But I don't understand very much because of I am not a native English speacker.
In https://github.com/nedbat/coveragepy/pull/1828#issuecomment-2310266578, do you mean the same as the table below ?

Python Version
coverage version
use PR1838
Still has bug

3.11
7.6.1
No
No

3.11
7.6.1
Yes
No

3.12
7.6.1
No
Yes

3.12
7.6.1
Yes
No

I tested these four scenarios below and It seems like this PR(#1838) can fix your problem in both Python3.11 and Python3.12.

So can I understand you like this:

This PR(#1838) can fix your problem now.
You think coveragepy need more test to include your bug report about f-string.

Please let me know if I'm wrong.

@the-13th-letter Thanks for your careful tests in #1828 (comment) !
But I don't understand very much because I am not a native English speaker.
In #1828 (comment), do you mean the same as the table below ?
[...]

@LiuYinCarl, no, that's not what I mean. In #1828, I can trigger the opposite effect: backslash continuation sometimes disappears in the HTML report, so a line break is missing, and coverage info gets "shifted upwards".
Specifically, in the following pytest-style file test_bug.py, the HTML report contains synthetic joined lines (e.g. the first two lines of the x variable are one line in the HTML report) by removing a backslash continuation from the input.
test_bug.py
def test_bug():
    prog_name = 'bug.py'
    err_msg = f"""\
{prog_name}: ERROR: This is the first line of the error.
{prog_name}: ERROR: This is the second line of the error.
\
{prog_name}: ERROR: This is the third line of the error.
"""
    if (lambda: None)():  # just some non-constant always-false predicate
        'This line intentionally is not covered.'
    'This line is covered again.'

    # test cases from https://github.com/nedbat/coveragepy/pull/1828
    def f(a, b):
        pass
    a = ["aaa", \
         "bbb \
         ccc"]
    b = ("a",\
         "b \
         c")
    c = "xxx"\
        "yyy \
        zzz"
    f(100,\
      "hello \
      world")
    d = """\
    hello
    """

    # work-alikes
    x = ["""aaa""", \
         """bbb \
         ccc"""]
    y = ("""a""",\
         """b \
         c""")
    z = """xxx"""\
        """yyy \
        zzz"""
    f(100,\
      """hello \
      world""")
    A = [f"""aaa""", \
         f"""bbb \
         ccc"""]
    B = (f"""a""",\
         f"""b \
         c""")
    C = f"""xxx"""\
        f"""yyy \
        zzz"""
    f(100,\
      f"""hello \
      world""")
    D = f"""\
    hello
    """
    AA = [f"""\
          aaa""", \
          f"""\
          bbb \
          ccc"""]
    BA = [f"""\
          {x}""", \
          f"""\
          {y} \
          {z}"""]

    if (lambda: None)():
        'This is not covered.'
    'This is covered.'

if __name__ == '__main__':  # if not running under pytest
    test_bug()

(See the attached ZIP file for full coverage output: Py3.11 unpatched, Py3.11 patched with 01779db1fea8d98acb0a15ccf229768c5851b64a, Py3.12 unpatched, and Py3.12 patched with 01779db1fea8d98acb0a15ccf229768c5851b64a. I do not know why the same behavior does not also happen with your cov.py file contents.)
All of the above has nothing to do with this PR, which is why I commented on #1828 instead of here. The only connection is that while reviewing this PR, I was reading up on #1828 to understand the surrounding code and comments. And then I realized that the tests added in dc819ff4d2333dfbf01c37406bb73ff1aceb8b3a looked weird, so I commented on that.

So can I understand you like this:

This PR(fix backslash in f-string #1838) can fix your problem now.

Yes.

You think coveragepy need more test to include your bug report about f-string.

I was trying to make a different point: I think that the tests in dc819ff4d2333dfbf01c37406bb73ff1aceb8b3a are wrong. They seem to expect HTML-rendered output or tokenizer output that is clearly different from the input.

The attached patch contains a pytest test function to check for #1836.
issue_1836_tests.diff.txt

@LiuYinCarl @the-13th-letter Does one of you want to apply that new test to this pull request?  Then we'd be able to merge it.
@the-13th-letter Why use {{prog_name}} in the test?  It seems like you intended it to be substituted, but it's not, and now you have literal {prog_name} in the results.   You can make a pull request into this pull request to apply the test to it.

Sorry, I went ahead and made a pull request with the test: https://github.com/LiuYinCarl/coveragepy/pull/1

@nedbat, your PR was merged and I add a annotation in this line to make it pass the tox checks.  https://github.com/nedbat/coveragepy/pull/1838/files#diff-29c93c7c1984a5716eb3a992a5e15cdcf5b7471539f0cc2831f141cc780a0885R1210

I cleaned up the commits and merged this as commit 515d99c6 and a77bba7b.  Thanks!

Just to follow up:

@the-13th-letter Why use {{prog_name}} in the test? It seems like you intended it to be substituted, but it's not, and now you have literal {prog_name} in the results.

It's a transcription of my original bug.py sample file in the original post of #1836, which included a "true" f-string. Since we're generating that as an output, and using an f-string to format the "f"/"fr"/whatever string leader, interpolations in the inner f-string like {prog_name} need to be double-escaped, as {{prog_name}}. It's very much on purpose, though admittedly born of the perhaps pointless desire of reusing the original demonstration file for this bug.

@the-13th-letter also, maybe we can do without {quote} by using triple-single-quotes? The escaping is a lot to puzzle through.

Sure, go ahead. It'd be a style violation in my corner of the world to not use double quotes, but of course that doesn't necessarily apply here. (It does mean that this is the first solution that comes to mind, though.)

Looking at the coverage reports for phystokens.py, there are some partial branches in the new checks:

Is that because it's impossible for those conditions to be false, or because we are missing some test cases in the test suite?  Either way, it would be great to resolve so that we don't have partial branches in the coverage report.

@nedbat It's my opinion below.
It is impossible for the condition on line 74 to be false because if last_line and last_line.endswith("\\\n"):
prevent other FSTRING_MIDDLE which is not end with \\\n into the condition.
I'm not sure about line 69, but I have tested the cases in https://github.com/nedbat/coveragepy/pull/1828 and https://github.com/nedbat/coveragepy/pull/1838, and it is impossible for any tests on line 69 to fail.
So, I think we can just replace the two conditions with inject_backslash = False

@nedbat I too am mostly convinced that the condition on line 74 must always be true because of the if last_line and last_line.endswith("\\\n") guard on line 40. I would probably settle on leaving the condition inside the code, as an assertion.
Like @LiuYinCarl, I'm not so sure about line 69 either. I did however try to design (failing) test cases for that condition, without success. Thus, pragmatically, I would convert that condition into an assertion as well, and leave a note à la "We failed to design a test case that doesn't fulfill this condition.".
attempted test inputs that don't improve branch coverage
space = " "
string = f"""\
line{space}1\
 and still line{space}1
line 2 \
and still line 2\
 and yet still line 2
line{space}3\
{space}and still line 3
"""

space = " "
weird_formatting = {"empty_string": """\
""", "nonempty_string": """ \
\

""",\
"chunked_string": "hello"\
" "\
"world"}
weird_formatting_with_fstrings = {f"empty_string": f"""\
""", f"nonempty_string": f""" \
\

""",\
f"chunked_string": f"hello"\
f"{space}"\
f"world"}

Thanks to both of you for continuing to work on this.  I really appreciate it! <3

Updated in commit 595ea22bbebcc84884c1401628a6353d2f10e222, coverage changed from 95.256% to 95.266% :)

This is now released as part of coverage 7.6.2.


Use local http server in integration tests

closes #274

This change is

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

tests/integration/image_test.go, line 49 at r1 (raw file):

}

func (it *imageTester) pullImage(url string) {

Maybe it's better to have another method like PullSampleImage that does ...PullImage(imageCirrosUrl) ? Because PullImage("") calls below are slightly confusing

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions.

tests/integration/go.test, line 28 at r1 (raw file):

fi

kill $HTTP_SERVER_PID

this should be done in a "trap" handler so the server gets killed even if the scripts stops before it completes

Comments from Reviewable

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

tests/integration/go.test, line 28 at r1 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
this should be done in a "trap" handler so the server gets killed even if the scripts stops before it completes

+1

tests/integration/image_test.go, line 49 at r1 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
Maybe it's better to have another method like PullSampleImage that does ...PullImage(imageCirrosUrl) ? Because PullImage("") calls below are slightly confusing

+1, or maybe to pass kubeapi.ImageSpec{} as arg

Comments from Reviewable

Review status: 2 of 4 files reviewed at latest revision, 7 unresolved discussions, some commit checks failed.

tests/integration/go.test, line 2 at r2 (raw file):

#!/bin/bash

perhaps Mirantis copyright section is needed here?

tests/integration/go.test, line 9 at r2 (raw file):

    exit $retval
  fi
}

why is this needed? Perhaps using the standard set of bash directives is better
set -o errexit
set -o nounset
set -o pipefail
set -o errtrace

tests/integration/go.test, line 12 at r2 (raw file):

cleanup() {
  if [ ! -z "${HTTP_SERVER_PID}" ] ; then

if [[ ! ${HTTP_SERVER_PID} ]]; then is a bit more idiomatic bash

tests/integration/go.test, line 17 at r2 (raw file):

}

trap cleanup SIGHUP SIGINT SIGTERM EXIT

just use trap cleanup EXIT, it'll be called on other signals

tests/integration/go.test, line 30 at r2 (raw file):

if [ $# -ne 0 ] ; then
  go test -test.v -run $*
  retval=$?

no need to use retval if you use set -o errexit

Comments from Reviewable

Review status: 2 of 4 files reviewed at latest revision, 7 unresolved discussions, some commit checks broke.

tests/integration/go.test, line 2 at r2 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
perhaps Mirantis copyright section is needed here?

We don't have it in any of scripts...

tests/integration/go.test, line 9 at r2 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
why is this needed? Perhaps using the standard set of bash directives is better
set -o errexit
set -o nounset
set -o pipefail
set -o errtrace

Yeah, it's better, will use this in the minute

tests/integration/go.test, line 12 at r2 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
if [[ ! ${HTTP_SERVER_PID} ]]; then is a bit more idiomatic bash

Right.

tests/integration/go.test, line 17 at r2 (raw file):
Previously, ivan4th (Ivan Shvedunov) wrote…
just use trap cleanup EXIT, it'll be called on other signals

Ack.

Comments from Reviewable

Reviewed 1 of 2 files at r2, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, 7 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 7 unresolved discussions, some commit checks failed.

Comments from Reviewable


New gaps for staked statistics

https://github.com/bro-n-bro/spacebox-api/issues/59#issuecomment-1639981902

Check returned value, too much result for 500000+ staked tokens

values rechecked , values returned correct


Tiny text improvements

Tiny things I came across when reading the cheatsheet. I hope they make it easier to read and understand.

Thank you!


Osram Smart+ Plug

Added support for Osram Smart+ Plug.
Who wants to test?

Just to know: Do I need a hub to connect this? Or will it connect through an app, and then be controllable with Homekit? (like webosTVs or Broadling RMs?)
Thanks!!

Yes, some kind of a hub/gateway. In this case homebridge with zigbee. I have a Raspberry Pi running hombebridge, where the smart plugs are registered. The iOS Home app can access homebrdige.
But you can use an App

Added support for Osram Smart+ Plug.

Can you add some other devices?

@Olepir what kind? I would need the zigbee.db and some one to test...

@Olepir what kind? I would need the zigbee.db and some one to test...

I can send you zigbee.db and test it. Where to send?

@Olepir open issue, I will try and create a branch which can be tested


Update Podfile.lock to Cocoapods 1.0.1

:shipit:

👎  botched it, needed a second ship it and remembered right after I merged.

next time.


Question: How to throttle bandwidth of a given REST call

I'm transitively using Requests, by using locustio.
Basically, we're loadtesting a service that accepts file-uploads (pretty much just a vanilla POST request). We want to slow down the upload to resemble what a real user's bandwidth would look like. (For clarification, the machine that runs the test is high-powered to replicate a large number of users. This, however, has the trade off that a large file upload that a user normally performs takes seconds instead of minutes; which falsely tests the application we're hitting).
We're looking at systemically slowing the bandwidth of the machine itself to replicate what a user would see, but we want to know: is it possible to do this programmatically?

There is nothing built into requests to do this. Depending on how you're trying to upload files you can either implement a generator that sleeps the appropriate amount of time or a file-like object that sleeps the appropriate amount of time.
In the future, please ask questions on StackOverflow.

@sigmavirus24 Thank you, and will do


Experience replay with frame stacking?

Hi all! Thanks for open-sourcing this, I feel like the world has been waiting for a flexible experience replay framework.. I definitely have myself.
I was reading through the README and I was wondering if you might have an example of a replay buffer with frame stacking. How would I implement that?
Thanks!

Hey Kristian! Great to hear that you are excited about Reverb!
There are a few different ways to implement frame stacking and depending on the use case the "optimal" approach may vary. I've created a colab to give an example of how you could do this and since I suspect that more than you might be interested in doing exactly this I'll add it to the example folder. I'll send it out for review now and I'll update this issue once it has been submitted.
Cheers!

Example is now submitted. You can find it here.

I'm going to close down this issue now but feel free to reopen if what I shared wasn't what you had in mind.

Thanks for the example notebook @acassirer
From the two examples, I prefer the second, in which you don't multiply the memory footprint by stack_size.
I was actually hoping for a selector class similar to reverb.selectors.Uniform. Maybe something like
reverb.selectors.UniformWithStacking, which would do the stacking on the server side.
There are 2 things/requests I'd like to mention.

In your second example, the sampled stacks will always come from the same trajectory/episode. I'm looking for a way to sample stacks coming from multiple trajectories.
You might want to initialize the frame stack deque by filling it with the first frame (repeated).

Either way, thanks a bunch for the notebook, much appreciated!


Build Routinator RPM packages

This is a draft PR at present because I want to add to this PR a GitHub Actions workflow that uses Docker and LXC like the Debian packaging does to actually build and test the RPMs.

I will mark this PR as ready for review to see if that causes GH to run the new RPM packaging workflow. If not I might have to push a copy of the workflow file to main as well.

Progress: the https://github.com/NLnetLabs/routinator/pull/566/checks?check_run_id=2679469756 build passed.
The CentOS 7 package needs to be built with the minimal systemd unit, as we do for older Debian versions. It isn't currently failing but the logs on the client on which Routinator is installed contain a lot of warnings about unrecognized configuration keys in the service unit file.

The CentOS 7 package needs to be built with the minimal systemd unit, as we do for older Debian versions. It isn't currently failing but the logs on the client on which Routinator is installed contain a lot of warnings about unrecognized configuration keys in the service unit file.

This is now handled by the GitHub Actions workflow.


runt launch

Launch a new kernel, using one of the existing kernelspecs.

Set up the necessary zeromq ports (see jupyter client implementation
write a connection file in the proper place
construct the argv &[OsStr] vector
Launch a subprocess either with std::command or subprocess
Handle stdin, stdout, stderr (connect to dev null, or perhaps connect to siblings of the connection file)
Open question: should this be exposed only through runtimed?
Open question: detachment. If exposed through runtimed, then there's a chance to hold onto the process struct. It's unclear if that's necessary or desirable, if the kernels can be stopped through other means. If not in runtimed, then runtimelib could host the appropriate data structure for managing child process lifetime too.

Starting has been resolved, but we don't yet have a way of removing runtimes from the system after closing.


Gui is not working for remove bloatware custom

When I use Windows10DebloaterGUI and check mark options for the custom list then select remove bloatware with customize black list. It shows everything in the list that its going to remove but nothing happens. This use to work before but not anymore. All the other options in the GUI work except this one.
I have to manually run the Debloat Windows in individual scripts and it works.
Is anyone else able to get this working?

I may have figured it out. It looks like when you check mark Nvidia it prevents it from working. I will do more testing and get back.


Creating a new Subscription on an empty Event Store and restarting the application causes a null-reference Exception.

I took me a while to narrow this one down, I thought it was a mistake in my implementation.
Reproduction:

Create a new, empty Event Store database (Postgres in this implementation)
Do not (yet) push any events to the Store
Start an application that registers an (All Stream) Subscription to this Event Store. In this example, I am using the Postgres Event Store Checkpoints table itself (instead of a separate read-model database).

The application creates an 'open' real-time stream and writes a Checkpoint record to the store, but since there are no events yet the position field is left as null:

Now stop the subscribing application and restart it. This will cause a null-reference Exception since it attempts to retrieve the Checkpoint record, but there is no value for position:

I am assuming there is a non-nullable field in the backend that throws this Exception. This field cannot be coalesced with a fallback value (e.g. 0) since that would mean (if someone pushed events to the Event Store in the meantime, while the subscription application is down) that the subscription would skip the first event.
The reason I ran into this issue repeatedly is that I am in the proof-of-concept phase of designing a historical system and so frequently delete the database and re-create it (in order to re-try my test cases). I can imagine this issue is easy to miss if one created a development Event Store once and never deleted it since.

Hmm, on reflection: perhaps -1 as a fallback value for an empty position field? So it will search for event 0 on startup?

Stranger even. My assumption that there was a nullable field in the backed is incorrect:

My new guess is that this cast to Nullable is too late. Shouldn't this be a TryRead?

Yeah, the (wrong) assumption was that casting to a nullable value type would not crash.
I think we need a test for it.

eah, the (wrong) assumption was that casting to a nullable value type would not crash.

Just to be sure we are on the same page: I don't think the cast to nullable value type fails (ulong to ulong? is definitely possible, it will just never result in a null value).
The operation reader.GetInt64() executed on a null value (which comes from the database) causes a Null-Reference Exception. Hence my suggestion to use a TryGetInt64 instead.

Or check if its DbNull

yes, I came across DbNull (when googling) check too when I looked into this. It was a while ago, I don't have the link anymore unfortunately.

@ReinderReinders have a look at the PR, it should fix the issue.


Missed SequenceAckMessage definition inside UpstreamMessage oneof

https://github.com/MicrosoftDocs/azure-docs/blob/e711e11b290ee72bccf2907b39f41db57e919906/articles/azure-web-pubsub/reference-protobuf-reliable-webpubsub-subprotocol.md?plain=1#L45
looks like need to add
SequenceAckMessage sequence_ack_message = 8;

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

ID: b05f311f-f88c-e54c-7b40-ecd7a1056656
Version Independent ID: f5a5a217-9681-8732-2212-e740dfc02092
Content: Supported features in external tenants - Microsoft Entra External ID
Content Source: docs/external-id/customers/concept-supported-features-customers.md
Service: entra-external-id
Sub-service: customers
GitHub Login: @msmimart
Microsoft Alias: mimart

@vaskravchuk
Thanks for your feedback! We will investigate and update as appropriate.

@vaskravchuk
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@PesalaPavan @vaskravchuk Thanks for taking the time to provide feedback, however this code is not found in the referenced document

#please-close

@msmimart what kind of code is not found?
"SequenceAckMessage sequence_ack_message = 8;"  is missed in document,
we tested and without it reliable protnbuf subprotocol doesn't work


Addressing features/bugs with the terminal

Description

Draft PR for now, address bugs and features of the built in terminal
Updated SwiftTerm to point to main
Currently added changing the cursor style
Fixed bleeding edge issue
Related Issue

closes #789
closes #615
closes #841

Checklist

[x] I read and understood the contributing guide as well as the code of conduct
[x] My changes generate no new warnings
[x] My code builds and runs on my machine
[x] I documented my code
[x] Review requested

Screenshots

Nice job in finding this, just want to comment that it's called caret not carrot, this could cause confusion with future contributors

And this whole time I thought we were picking carrots! 😄🥕

Well, I guess we were… 🐰

Thanks for looking, lets just say I was a little tired when I did this lol!

One additional thing I'd like to discuss:
Instead of having a list of all the curser styles like you can see below, just have the main 3 styles (block, underline, bar) and have a separate boolean toggle the blinking behavior instead. This would make selection a lot cleaner IMO.

Any thoughts on this? @austincondiff @Wouter01 @0xWDG @KaiTheRedNinja

I don’t like the duplication. Blink will eventually be much more than a simple toggle. See the discussion here.

@austincondiff so for now would you just recommend leaving it as the picker?

If you can, go ahead and add the blink toggle if you can. We can change it to a blink style picker later on.

@austincondiff So the picker will be the style(block, underline, bar), and then a toggle for if you want it to blink or not?

@bombardier200 JFYI SwiftTerm now has a new release. Please change the dependency to use exact: 1.2.0.
https://github.com/migueldeicaza/SwiftTerm/releases/tag/1.2.0

@bombardier200 Yes, that works great.

New preferences view, this PR should be ready to go now!

@Wouter01 Should we wait for the splitview branch to be merged before merging this PR into main?

@Wouter01 Should we wait for the splitview branch to be merged before merging this PR into main?

Nope, all fine!

Will the same happen with: https://github.com/CodeEditApp/CodeEdit/pull/1113 @Wouter01? Or should we wait?


[Bug]: _typings produces no files

What happened?
When I try to follow the tutorial for formatting tests for typescript, I get issues because my _typings target doesn't seem to produce files with the Provider that rules_lint is using:
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_YAML_with_yamlfmt: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_Jsonnet_with_jsonnetfmt: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_Jsonnet_with_jsonnetfmt' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_Go_with_gofmt: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_Go_with_gofmt' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_Shell_with_shfmt: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_Shell_with_shfmt' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_YAML_with_yamlfmt' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_JavaScript_with_prettier: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_JavaScript_with_prettier' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_Terraform_with_terraform-fmt: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_Terraform_with_terraform-fmt' failed
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: in args attribute of sh_test rule //ts/factorio/react:react_lint_formatting_Starlark_with_buildifier: label '//ts/factorio/react:react_typings' in $(location) expression expands to no files. Since this rule was created by the macro 'ts_project', the error might have been caused by the macro implementation
ERROR: /Users/tshadwell/code/monorepo/ts/factorio/react/BUILD.bazel:4:11: Analysis of target '//ts/factorio/react:react_lint_formatting_Starlark_with_buildifier' failed

Version
Development (host) and target OS/architectures:
Output of bazel --version:
bazel 7.1.2
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file:
bazel_dep(name = "aspect_rules_lint", version = "0.19.0")
bazel_dep(name = "aspect_rules_lint", version = "0.19.0")

Language(s) and/or frameworks involved:
typescript
How to reproduce
You can pull this commit: https://github.com/zemn-me/monorepo/pull/5472/commits/a117b1a5614ddc908855186a852a45302bb4d951

and run `bazel test --keep_going //ts/factorio/react/...`

Any other information?
No response

You're trying to check the formatting of some build outputs, which is wrong. You should format-test the original srcs of the ts_project:
diff --git a/ts/rules.bzl b/ts/rules.bzl
index 42538157f..fdf978f21 100644
--- a/ts/rules.bzl
+++ b/ts/rules.bzl
@@ -49,11 +49,6 @@ def ts_lint(name, srcs = None, **kwargs):
         **kwargs
     )

-    format_test(
-        name = name + "_formatting",
-        srcs = srcs,
-    )
-
 def ts_project(name, visibility = None, lint = True, deps = [], data = [], resolve_json_module = True, srcs = None, tsconfig = "//:tsconfig", preserve_jsx = None, tags = [], **kwargs):
     """
     Compile a set of typescript files, dependencies, runtime data and other source files into typescript types and source maps.
@@ -109,3 +104,7 @@ def ts_project(name, visibility = None, lint = True, deps = [], data = [], resol

     if lint:
         ts_lint(name = name + "_lint", srcs = [name + "_typings"], tags = tags)
+        format_test(
+            name = name + "_formatting",
+            srcs = srcs,
+        )


Rqt dotgraph

Please Add This Package to be indexed in the rosdistro.

rolling
humble
iron

The source is here:
https://github.com/niwcpac/rqt_dotgraph
Checks

[x] All packages have a declared license in the package.xml
[x] This repository has a LICENSE file
[x] This package is expected to build on the submitted rosdistro

New package review checklist

[ ] At least one of the following must be present

[X] Top level license file:
[ ] Per package license files:

[ ] License is OSI-approved:
[X] License correctly listed in package.xmls
[X] Public source repo:
[X] Source repository contains ROS packages
[ ] Each package meets REP-144 naming conventions

@tdenewiler The CC0 license is not OSI approved, please check the list and consider changing the license for the package to be indexed.

We developed the code as part of official duties working for the US Government. We are not allowed to copyright that work. The best we can do is dedicate the work to the public domain. That is why we use the CC0 license.

@clalancette please advise!

So I took a look at the package in detail.
First of all, we can accept a CC0 license.  However, it appears that not all of the code in the package is CC0.  In particular, there is this line in the README: To visualize the DOT graphs, this module includes a version of [xdot_qt.py](https://github.com/jrfonseca/xdot.py) forked from [ROSPlan](https://github.com/KCL-Planning/ROSPlan/blob/master/rosplan_rqt/src/rosplan_rqt/xdot_qt.py) and released under [LGPLv3](https://www.gnu.org/licenses/lgpl-3.0.html). The forked version in this package has been modified and the changes to it are released under the LGPLv3..
The best way to handle this would be to submit the modifications to upstream rosplan_rqt, and then take a dependency on rosplan_rqt.  With that in place, the question of license in this repository would be restricted to CC0, which we can accept.
If that is not possible, then at the very least the package.xml here needs to add in a second <license> tag for LGPLv3, reflecting the fact that there are multiple licenses for this repository.  It would also be best to add a comment next to the license tag saying which files are covered by which license (for CC0 you can just say "all other files" or something like that).

I went with the second license approach with comments. PRs in progress for rqt_dotgraph repo.
I really like your idea of adding a dependency on rosplan_rqt, but that package is not released. A request was made in 2019 but no action was taken: https://github.com/KCL-Planning/ROSPlan/issues/201.

I see there is still a "changes requested" label. I have made requested changes to the license in package.xml upstream. Are there any other desired changes?


chore: add stage rabbit secrets

NO-TICKET

In reval services we did not separate the application.yml file for different envs. It is  one common file used in different envs with specific variable values.


Development stack deployment process

We need an easy way to deploy a full development stack for local testing.

[x] Docker-compose environment
[x] Documentation
[ ] Train developers

@ThomasThelen We think this is ready to go.  Could you give it a try on your VM and let me know how it goes?
https://github.com/whole-tale/deploy-dev/blob/master/README.md

I was able to go through the readme and deploy the stack with ease. The only note is that I manually ran setup_girder after seeing the error below.
docker stack deploy --compose-file=docker-stack.yml wt
Creating network wt_mongo
Creating network wt_celery
Creating network wt_traefik-net
Creating service wt_dashboard
Creating service wt_registry
Creating service wt_traefik
Creating service wt_mongo
Creating service wt_girder
Creating service wt_redis
./run_worker.sh
e352f86e892cbf716f4fb03d77726a91efc30ea15dcc2851515d5291da31a865
./setup_girder.py
Waiting for Girder to start
Traceback (most recent call last):
  File "./setup_girder.py", line 25, in <module>
    r = requests.get(api_url)
  File "/home/ubuntu/.local/lib/python2.7/site-packages/requests/api.py", line 72, in get
    return request('get', url, params=params, **kwargs)
  File "/home/ubuntu/.local/lib/python2.7/site-packages/requests/api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "/home/ubuntu/.local/lib/python2.7/site-packages/requests/sessions.py", line 512, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/ubuntu/.local/lib/python2.7/site-packages/requests/sessions.py", line 622, in send
    r = adapter.send(request, **kwargs)
  File "/home/ubuntu/.local/lib/python2.7/site-packages/requests/adapters.py", line 513, in send
    raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPConnectionPool(host='girder.vcap.me', port=80): Max retries exceeded with url: /api/v1 (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7fd32c559210>: Failed to establish a new connection: [Errno 111] Connection refused',))
Makefile:19: recipe for target 'dev' failed
make: *** [dev] Error 1

It would also be nice to throw a dependencies install script inside.

This issue can most likely be closed now that there are complete instructions and automated scripts to do the heavy lifting. I was able to re-deploy the development stack with ease on a new Ubuntu VM.


PH/MIN-361 Parachain and Standalone runtimes

Description:
Split runtime into two almost identical implementations: Parachain and Standalone.
Comparing to Parachain runtime, Standalone has additional grandpa pallet, benchmarks and tests.
In cli and service modules was added logic to start node in parachain mode and in standalone mode.
In chain_spec was split into two parts: "staging" parachain spec and dev/local standalone spec.
Changes in storage or API:
CLI changes:

Added relaychain args
New build-in chain-specs:

default - Local Testnet parachain
dev or standalone-dev - Standalone Dev
standalone-local - Standalone Local Testnet

Checklist

[ ] Code is covered with tests
[ ] Code is covered with benchmarks
[ ] Fresh benchmark results included

Please, fix all warnings in compile time.


Fix domain of GHG staging hub

Also a trivial change in common to retrigger deployment
Ref https://github.com/2i2c-org/infrastructure/issues/3029

Follow-up to unblock https://github.com/2i2c-org/infrastructure/actions/runs/5977100479


Feature/76 read data from confluence into vector database

All functionalities have been implemented except for the embeddings.

I think your test is failing because it installs a hashlib version that is not compatible with python3. But I think hashlib is part of the standard library anyway so you should not need it in the requirements afaik

Great improvments

the test is still failing because hashlib is still in requirements.txt btw

but looks good otherwise

The PDF now can be stored in Supabase as chunks but it takes little more time than usual.

Forgot to comment here there was a minor issue in token.env (already updated in our secure channel)
The URL for confluence have to end with /wiki/
So https://testqaware.atlassian.net/wiki/ instead of https://testqaware.atlassian.net/.
Otherwise the PDF download link wont work in the code.


Minor issue: no newline after last line in portfolio CSV export

This is really a small problem/question.
Is it intended that there is no \n after the portfolio content here?

Probably not. Is the \n important to you? I guess you can open a PR that adds it then... 😄


nixos/mysql: fix permission error during first startup

When mysql starts up for the first time, the binary mysql_install_db will run and try to set up the correct folder structure and permissions.
The very first step is to change the owner and group of the data directory. This can fail in some cases, for example if cfg.dataDir is something like /mnt/mysql:
Jul 31 15:24:35 junction systemd[1]: Starting MySQL Server...
Jul 31 15:24:36 junction mysql-pre-start[1346]: chown: changing ownership of '/mnt/mysql': Operation not permitted
Jul 31 15:24:36 junction mysql-pre-start[1309]: Cannot change ownership of the database directories to the 'mysql'
Jul 31 15:24:36 junction mysql-pre-start[1309]: user.  Check that you have the necessary permissions and try again.
Jul 31 15:24:36 junction systemd[1]: mysql.service: Control process exited, code=exited, status=1/FAILURE
Jul 31 15:24:36 junction systemd[1]: mysql.service: Failed with result 'exit-code'.
Jul 31 15:24:36 junction systemd[1]: Failed to start MySQL Server.

This is because /mnt usually is owned by root.
To work around this issue, use systemd.tmpfiles to set the owner, group and permissions. This will always work, and if the permissions are already set as expected, mysql will not try to alter them again.
Description of changes

The mysql module ensures the permissions of the dataDir are set up as expected

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

@ofborg test mysql

Yes, it's fine

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


[Bug]: $CompilationError$ is given as variable type with create variable code action

Description
Consider the following scenario where we're trying to apply the create variable code action without importing the required modules,
https://user-images.githubusercontent.com/61020198/195032527-b4771425-b979-4387-8708-8ab03313de8e.mov
Steps to Reproduce
No response
Affected Version(s)
No response
OS, DB, other environment details and versions
No response
Related area
-> Editor
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Tried two codes snippets to reproduce the issue

We do not get the diagnostic at the getStatus() method call when the import statement is commented out

import ballerina/http;

function getStatus() returns http:MutualSslStatus {
    return http:PASSED;
}

function name() {
    getStatus();
}

Created a project(called testComp) and added two modules inside the project (newLib and oldLib)

inside the newLib in the newLib.bal I have added following code snippet,

public type TestRecord record {
    int recField1 = 0;
    string recField2 = "";
};

and have using that inside oldLib in the oldLib.bal

import testComp.newLib;

function getFunction() returns newLib:TestRecord2 {
    return {rec2Field1: 10, rec3Field2: "rec3Field2"};
}

function testFunctionFunction() {
    getFunction();
}

here also we do not get the diagnostic for getFunction()

As per the offline discussion with @mindula this is reproducible only when there is a union return type for the function. Hence adding another type-descriptor worked.
ex:
import ballerina/http;

function getStatus() returns http:MutualSslStatus|string {
    return http:PASSED;
}

function name() {
    getStatus();
}
`1``


Error creating new field

New install of SuperTable, gives this error when hitting "new field":
Property "Craft\WebApp.superTable_matrix" is not defined.
Appears to be coming from /supertable/fieldtypes/SuperTableFieldType.php(22): Craft\SuperTableFieldType->_getFieldTypeInfoForConfigurator()

installing release 1.0.1 "fixes" this. I guess the top tip is don't install dev versions. ;oP

Not sure what that error was about. Were you installing the latest version from the master, or from a release?

I was installing from master, iirc there was a file deleted between versions and presumed you were working on it. Installing latest release works fine for me.


Blank page and Colophon

Dear @aso2101!
I want to add two issues regarding xml to latex transformation:

There are still a few blank pages in the chapter shift of the edition. Can it be controlled ? To give you a context, please see my screen shot, taken from the Śikṣā Guru edition (pp. 212–125 in pdf).

<ab xml:id="siksaguru_14.01" type="colophon"> in the Śikṣā Guru and
 <ab xml:id="smg_47.01" type="colophon">in the Sasana Mahaguru should not generate a new page in LaTex, but be treated as the last
 in the
 which it follows

Best,
Aditia

@aditiagunawan
I have this when I am running the code:

Have you pushed all the recent modification of your text so that we can take a look with the same files as you?
Best,
Axelle

@ajaniak
I pushed now the necessary files, please regenerate. I make some adjustment manually. I will sent a list of my adjustment in LaTex very soon.
Best,
Aditia

@aditiagunawan : can this issue be closed?


Long responses while high load !

I'm experiencing some issues as mentioned in title !
It takes longer than usual forexample a sendMessage takes about 1 second to respond back.
I'm using curl_multi_init in PHP to send message to multiple users at the same time. for a game & chat purpose and I can't change because users are happy with it.
I'm not getting 429 error ! but also I can't understand why it should be slow on high loads!
It's not a resource problem as CPU in average 2% and only 10% of server's memory is in use.
I have no clue about the networking of the server but it's hetzer finland location.
here is some logs:
( I must mention that the api= is actually a different bot token. )
It clearly shows this problem happens on high load because texts are similar.
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=125405****&text=لـئو ستاره ۲ رای، مهتا ۱ رای، \nمحدثه به مهتا رای داد.&parse_mode=html
[2022-04-11 23:32:02] Took 1.383043 to finish api=0
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=180390****&text=لـئو ستاره ۲ رای، مهتا ۱ رای، \nمحدثه به مهتا رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 2.044794 to finish api=5
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=1254056****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.380002 to finish api=5
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=126607****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.370661 to finish api=1
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=137855****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.356331 to finish api=1
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=140552****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.345066 to finish api=1
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=148424****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.378432 to finish api=1
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=157359****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.230961 to finish api=17
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=171450****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.919404 to finish api=0
URL=> http://<IP_ADDRESS>:8081/botXXXXX/sendMessage?chat_id=180390****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.304997 to finish api=22
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=206386****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.466971 to finish api=23
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=518169****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:03] Took 1.385182 to finish api=0
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=89304****&text=لـئو ستاره ۳ رای، مهتا ۱ رای، \nتابان به لـئو ستاره رای داد.&parse_mode=html
[2022-04-11 23:32:04] Took 1.021891 to finish api=5
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=125405****&text=لـئو ستاره ۳ رای، کت وومن ۱ رای، \nمحدثه رای خود را پس گرفت و به کت وومن رای داد.&parse_mode=html
[2022-04-11 23:32:05] Took 1.182429 to finish api=3
URL=> http://<IP_ADDRESS>:8081/botXXXX/sendMessage?chat_id=106844****&text=سوگندی ۳ رای، \nمهی به سوگندی رای داد.&parse_mode=html
[2022-04-11 23:32:06] Took 1.586595 to finish api=6

If CPU usage remains low, then the problem isn't in the server. Increase server log verbosity level to 3 with the parameter -v3 and try to find the clues in the log.

what should I look for in log in particular?
Are you sure telegram doesn't limit my requests by volume ?

what should I look for in log in particular?

Track, how the request was completed.

Are you sure telegram doesn't limit my requests by volume ?

Absolutely.

@levlam some messages took 35 seconds to complete
what the hell is wrong!
my log is almost 1.5gb how can I find error in that and what kind of error I must look for ?

@mnkblog please avoid such harsh words 😐 it is not the right way to approach our engineer  @levlam

@iamDonkey hey donkey i'm sorry , i definetly need help... can't keep my users happy :(

Did you try sendMessage api request via curl cli command? Is it the same speed as php curl? Did you also test php file_get_contents function? What is your php version?

@mnkblog If you need help with finding the exact cause, you can send the log in an archive to https://t.me/tdlib_bot,

@levlam does telegram slow me down if i'm trying to send more than 1 message per second to the same private chat?

hey @mnkblog , did you solved this? can you please share with us what's wrong? and how to solve it? Thanks.


Added new methods to call PUT POST and DELETE by REST. Added new hawkular methods to create/edit/delete resource. Added new test cases for hawkular. Added mock REST API calls.

Lint report for commit 7c72d4a5e115a8c654d19a7196ed472330dc2daf:
Everything seems all right :smile: .
CFME QE Bot


bump esprima-harmony-jscs to 1.1.0-tolerate-import

Coverage remained the same at 98.64% when pulling 7fbf9826823fe0d547bbddc57b2307337c5863d4 on cvrebert:patch-1 into ca77989defdee0320f92fb029ca61fa062ed0f35 on jscs-dev:master.

I'd need to conduct further testing to be sure, but combined with the existing tolerant: true change, this appears to be sufficient to fix #952.

Ok, let me know when you've finished testing.

@mikesherov Yup, have confirmed that this finishes fixing #952.

Any estimate of when the next JSCS release will happen?

@cvrebert, as soon as we land autoconfiguration, which is almost completed.


xq replaces " with " when it can cause issues:

Original text:
  <SEQUENCE mandatory="FALSE" precondition="GetParam("HASH_VALUE_SHARING_MODE") != 1">

After running xq:
    <SEQUENCE mandatory="FALSE" precondition="GetParam("HASH_VALUE_SHARING_MODE") == 1">

The original place of " is to ensure there is no closure of the " opened before Get Param.
The conversion of " to " causes issues

Similar with < that we use instead of < that is confused as part of XML format.
and & that we use instead of &

@nworms thank you for your report. I think it's a little bit controversial. The main intention was to have a utility that shows XML content that is easy to read by humans (e.g. with proper formatting, syntax highlighting and etc.)

A brief check shows that it's a behavior of XML standard library in Go that is not easy to override or influence somehow: https://github.com/golang/go/blob/master/src/encoding/xml/xml.go#L1086
It looks like a good candidate to be reported as an issue to the Golang project.

Go has support to add the escaped version of the problematic characters using:
xml.EscapeText
I understand now your motivation to use xq to have a more human readable version of XML.
May be (in the future) add a command line option to have it be both be human and machine readable and it that case you will use xml.EscapeText for the attribute values.

@nworms Thank you for your suggestion. I'll apply the corresponding fix.


Thoughts on swagger configuration

There is a lot of info that is not available in Eve and yet is needed for Swagger. The current approach is that the settings can be expanded with a SWAGGER document which includes all the extra data. I am not sure this is the best approach.
Maybe we could just add support new keywords in the settings file, eventually prepended by the SWAGGER title, very much like the MONGO prefix( e.g. API_TITLE or SWAGGER_API_TITLE). This would also come in handy at a later stage, when we're working on resources ("paths" in swagger terms). Iterating through the  DOMAIN would allow to retrieve both eve standard settings and swagger's if available, instead of having to look at two different places during the whole process. Would probably be easier on the user too, as all resource configuration is held at one place.

+1 for the SWAGGER_ prefixed approach in the already existing domain dictionaries.


feat req: provide badge sevice

write it in Readme

There is already an open issue for this #4 . Closing this, feel free to add any points you have in the original issue.


[BUG] Ignore extra whitespace in loader

currently, if there are extra spaces between tag values in the data file, it causes errors in the loading. This simply strips out any unnecessary white space.
Also removed a dataset list which is not correct with data on tsc.com

Thanks Tony, for your work.
I think it was me who added whole_set. I couldn't find a way to collate the 142 datasets used in the bakeoff redux. As you well know, they are the 112 univariate datasets without missing and unequal + 30 multivariate datasets of the 33. Maybe we can create a new list with the 30 datasets that I suppose are equal-length and non-missing. Am I correct?
Thank you Tony!

whole_set contains datasets that are not yet explicitly on TSC such as AsphaltPavementTypeUniEq so it fails the loading. I think it better to keep lists like this in your own branch, and only include lists where we can download the data directly from one of the websites

Thanks Tony, for your work.
I think it was me who added whole_set. I couldn't find a way to collate the 142 datasets used in the bakeoff redux. As you well know, they are the 112 univariate datasets without missing and unequal + 30 multivariate datasets of the 33. Maybe we can create a new list with the 30 datasets that I suppose are equal-length and non-missing. Am I correct?
Thank you Tony!

whole_set contains datasets that are not yet explicitly on TSC such as AsphaltPavementTypeUniEq so it fails the loading. I think it better to keep lists like this in your own branch, and only include lists where we can download the data directly from one of the websites

Great, I see what you mean. Thanks Tony!


Alternative to zeus ?

It's seams that unfortunately, zeus is not maintained  by community and contributors for more than 1 year.
Is there an alternative that you know guys ?

Starting from Rails versions: 4.2, 5.0, 5.1, spring is installed by default. It can do some preload similar to zeus but I always prefer zeus as it has a UI to see what's happening and can custom plans for your specific need.
zeus is still working fine for me

Check out the helpful "Related gems" section in Zeus's README


chore: Align conversion QS names

No jira.
This sets the correct names for the pre-conversion, conversion and leapp (8to9) quickstarts (referenced in https://github.com/RedHatInsights/quickstarts/tree/main/docs/quickstarts).
How to test
Go to /tasks/available in stage, find any available tasks that have quickstarts enabled and click "Help me get started"; you should see a quickstart displayed. The conversion QS content PR is not yet merged (https://github.com/RedHatInsights/quickstarts/pull/185), so this most probably won't work which is expected.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.73%.  Consider uploading reports for the commit 1328af1 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #188   +/-   ##
=======================================
  Coverage   81.73%   81.73%
=======================================
  Files          75       75
  Lines        1621     1621
  Branches      521      521
=======================================
  Hits         1325     1325
  Misses        259      259
  Partials       37       37

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


I want to know all the 'params'

I want to know all the params'

Please check README again. You can find all the params info there.


Fixed anchorX, anchorY

I had some issues when I changed the sprites anchor point, if the anchor point is different to 0 the collision is not exact
Idk if is a bug, but now works well on any anchor position
PD: I cant build the project on this machine, I just made changes on src/bump.js

Thanks so much!!


Broken Link in Markdown File

In the markdown file setup-vagrant.md the link to the "Zulip development community server" under the Troubleshooting and Common Errors sub-section is broken.
The "Zulip development community server" links to ../contributing/chat-zulip-org.html, which gives a 404 error, as chat-zulip-org.html does not exist in the contributing folder. I suspect the link is supposed to be linking to the chat-zulip-org.md file, which is stored in the contributing directory, or to the actual community server.
@zulipbot label "area: markdown" "good first issue"
@zulipbot claim

@zulipbot claim --force

@eschadt next time just add help wanted or good first issue with @zulipbot label

@lonerz Thank you for manually assigning me the issue! And yes of course, next time I will use those labels with the @zulipbot!

Thanks for working on this @eschadt!
The link isn't broken; this markdown file just isn't intended to be read on GitHub; it's a ReadTheDocs markdown file.  You can see it renders correctly at https://zulip.readthedocs.io/en/latest/development/setup-vagrant.html.  Ideally, there'd be something we can do on the GitHub side to redirect people to ReadTheDocs (I think this would probably involve some syntax that would be hidden in ReadTheDocs but not in GitHub; some research required).  @rosatamsen might have some ideas for how to do that...

@timabbott ah okay, I see what you're saying!
I think this could be a simple fix then.
If we change the relative link of ../contributing/chat-zulip-org.html on "Zulip development community server" to an absolute link of "https://zulip.readthedocs.io/en/latest/contributing/chat-zulip-org.html"  people reading the setup-vagrant.md file on GitHub (even though that is not an intended case) that click the link "Zulip development community server" will be directed to the correct readTheDocs page, and when someone is reading the readTheDocs version of setup-vagrant they will be directed to the chat-zulip-org.html as usual.
That would take care of directing a reader from the GitHub side to the ReadTheDocs side, as you mentioned above.

So, the problem with absolute links is that if you're developing locally, you'll end up getting directed off the local deployment to the production readthedocs; this can be annoying for testing whether e.g. a refactor broke links.

@timabbott That makes sense! Thank you for explaining your reasoning throughout this thread, I appreciate it!

I looked into this issue, but unfortunately it seems like Sphinx does not support this sort of .md to .html extension replacement for links when generating docs. And, like Tim said, the docs are not intended to be read on GitHub. Might want to consider adding a README.md to https://github.com/zulip/zulip/tree/master/docs saying so and direct users to ReadTheDocs instead.

@timabbott , @rosatamsen , I would be happy to add a README.md file to https://github.com/zulip/zulip/tree/master/docs stating that docs are not intended to be read on GitHub and directing users to the ReadTheDocs, if you would like to add it!

sure, go for it.


Capture templates

Would it be possible to also capture the templates
GET /_template

👍
Yes, they are in the cluster state, but so are other things that we capture. When you have a large cluster state, it can be a pain to extract selected items, and working with peoples' templates is something we have to do quite frequently.

Don't disagree.  It will be in the next version, out in the next couple of days.


coverage: try using codecov

Signed-off-by: Ruslan Kuprieiev<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff            @@
##             devel     #588   +/-   ##
========================================
  Coverage         ?   82.16%
========================================
  Files            ?       35
  Lines            ?     1901
  Branches         ?        0
========================================
  Hits             ?     1562
  Misses           ?      339
  Partials         ?        0

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 13e0e9d...72da7c9. Read the comment docs.


setup-ide.sh error with GVRAudioSDK (= 1.120.0)

I ran setup-ide.sh on a freshly initialized project and got the following error
Unable to find a specification for GVRAudioSDK (= 1.120.0) depended upon by ViroKit

Hi @mypark,
Looks like your Cocoapods repo hasnt been updated recently.
Try pod repo update before performing a pod install.
Let us know if that works!

yup that did it!


Support for asp.net core targeting full framework

I'm working on an asp.net project that is targeting the full .net framework. I get the following error when attempting to use the asp.net core XRay library:
System.MissingMethodException: Method not found: 'Boolean Amazon.XRay.Recorder.Core.AWSXRayRecorder.IsLambda()'.
   at Amazon.XRay.Recorder.Handlers.AspNetCore.Internal.AWSXRayMiddleware.ProcessHTTPRequest(HttpContext context)
   at Amazon.XRay.Recorder.Handlers.AspNetCore.Internal.AWSXRayMiddleware.<Invoke>d__13.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.<Invoke>d__7.MoveNext()

I haven't dug in to try and find where the issue is but I have created a simple example that illustrates the issue here: https://github.com/blebaron/AWSXRayExample
I created the example by using the VS 2017 asp.net core api new project template and then modifying it to target the net461

Hi @blebaron ,
The reason for your exceptions is, you are using ASP.NETCore 2.0 X-Ray package for net461 framework.
IsLambda() is present only for netcore2.0 framework. As a result, X-Ray SDK doesn't find this method on net461 framework.
Reference: https://github.com/aws/aws-xray-sdk-dotnet/blob/master/sdk/src/Handlers/AspNetCore/Internal/AWSXRayMiddleware.cs#L225
I suggest to use ASP.NET X-Ray handler for net461 framework : https://github.com/aws/aws-xray-sdk-dotnet/tree/master#aspnet-framework-net--nuget
Is there any specific reason for a use case where you change the framework from netcore2.0 to net461?
Thanks,
Yogi

Sorry for the delayed response. I got this working by manually pulling out the .net core assemblies from the nuget packages and referencing them directly and then forgot that I had logged an issue.
I'm using the asp.net core MVC constructs (startup, kestrel, etc...) inside of a full framework project. That's why I can't use the net461 package. The example I linked to illustrates exactly how I have things set up.

Hi @blebaron ,
We appreciate your response.
Closing this issue.
Best,
Yogi

I'm not sure why this was closed nor how @blebaron worked around the issue.  Targeting kestrel and asp.net core is recommended for .net framework developers.  How can xray instrument asp.net core on this platform?


feat(Flowie Timeline)

[x] Dashboard setup
[x] Redux setup (got stuck at fetching multiple URLs at once)
[x] Horizontal Component
[x] Data Fetching from GitHub API
[x] PR Component
[x] Comments for each PR is now fetched concurrently
[x] Issue Component
[x] Filters for Issues, PRs
[x] Commits
[ ] More to come

@SiddharthSham @Diaga Merging!


improve api

@paulofaria So awesome!!!

Wow, awesome! I like it 😄

@paulofaria I’m rebasing #7 now, maybe we should merge it first.

@dreymonde before this one? LGTM! :shipit:

@paulofaria ready to merge?

Rebased on master; ready to merge after a LGTM from another pair of eyes.