docs(data exchange): adapt and restructure the data exchange

Description
This PR adds some improvements, restructuring within the data exchange documentation. It also adds requests and expected responses to get a more in-depth understanding of the flow. The currently documented flow was tested multiple times.
@dvg36 would be cool to test it again, just to be sure!!! I think after this, we are well-prepared for the Tractus-X Community Days. ;)
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review:

[X] DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
[X] Copyright and license header are present on all affected files

@stephanbcbauer I will test the tutorial tomorrow morning again.

I think it's a very good idea to add Alice and Bob to this great story. Again, very nice job rewriting the documentation, in this case of the data exchange.
I have followed the “tutorial” locally and everything works correctly. I have only found small details in the writing, that with all that you have written, I am surprised there are only those (that I have encountered).
For my part, everything works very well and the PR is great, and all the work performed is going to be valued.

Thank you very much @mgarciaLKS for your effort and detailed testing, :) I applied your suggestions. Hopefully, it worked.

LGTM! Could you please also fix this dead link?

done thx


cigien: Watch groupxit.com

cigien requests the watch of the watch_keyword groupxit\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

groupxit\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


firecracker: Add support for default VM configuration

firecracker: Add support for default VM configuration
Kata support specifing the default VM configuration via configuration.toml. This allows the system or cluster admin  to choose the default (i.e minimum) size of the VM.
Add support in kata to respect the VM configuration for firecracker.
Fixes: #1594
Signed-off-by: Manohar Castelino<EMAIL_ADDRESS>

holding off a \ t e s t and merge until we hear back from @mcastelino

/test

Fedora job failed due to: https://github.com/kata-containers/tests/issues/1449
• Failure [22.547 seconds]
state
/tmp/jenkins/workspace/kata-containers-runtime-fedora-PR/go/src/github.com/kata-containers/tests/functional/state_test.go:26
  container
  /tmp/jenkins/workspace/kata-containers-runtime-fedora-PR/go/src/github.com/kata-containers/tests/vendor/github.com/onsi/ginkgo/extensions/table/table.go:92
    with workload [true], timeWait 5 [It]
    /tmp/jenkins/workspace/kata-containers-runtime-fedora-PR/go/src/github.com/kata-containers/tests/vendor/github.com/onsi/ginkgo/extensions/table/table_entry.go:46

    Expected
        <int>: 1
    to equal
        <int>: 0

    /tmp/jenkins/workspace/kata-containers-runtime-fedora-PR/go/src/github.com/kata-containers/tests/functional/state_test.go:45
------------------------------
SSSSSSS

Summarizing 1 Failure:

[Fail] state container [It] with workload [true], timeWait 5
/tmp/jenkins/workspace/kata-containers-runtime-fedora-PR/go/src/github.com/kata-containers/tests/functional/state_test.go:45

Ran 1 of 8 Specs in 22.548 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 7 Skipped --- FAIL: TestFunctional (22.55s)


ConfigLimitManager

ConfigLimitManager handles config limits; called from validate
Adds validate as an option in build
Streamlines tx/receipt getters to grab the raw tx/receipt frist and pass it into the getters to reduce the number of RPC calls

ready for review @jonathanpwang (tests are passing locally but automation is timing out)

@jonathanpwang ready for re-review


TypeError: can't pickle _thread.lock objects when cleaning

This is most likely an issue with airflow itself (we use 1.10.9), but we didn't have any similar issue with it before - on clearing GE operator, whole system goes kabum with this log:
Node: airflow-airflow-web-6f4fbbfc58-cnx4j
-------------------------------------------------------------------------------
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 2446, in wsgi_app
    response = self.full_dispatch_request()
  File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1951, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1820, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/usr/local/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1949, in full_dispatch_request
    rv = self.dispatch_request()
  File "/usr/local/lib/python3.7/site-packages/flask/app.py", line 1935, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/usr/local/lib/python3.7/site-packages/flask_admin/base.py", line 69, in inner
    return self._run_view(f, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/flask_admin/base.py", line 368, in _run_view
    return fn(self, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/flask_login/utils.py", line 261, in decorated_view
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/airflow/www/utils.py", line 290, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/airflow/www/utils.py", line 337, in wrapper
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/airflow/www/views.py", line 1296, in clear
    include_upstream=upstream)
  File "/usr/local/lib/python3.7/site-packages/airflow/models/dag.py", line 1211, in sub_dag
    for t in regex_match + also_include}
  File "/usr/local/lib/python3.7/site-packages/airflow/models/dag.py", line 1211, in <dictcomp>
    for t in regex_match + also_include}
  File "/usr/local/lib/python3.7/copy.py", line 161, in deepcopy
    y = copier(memo)
  File "/usr/local/lib/python3.7/site-packages/airflow/models/baseoperator.py", line 681, in __deepcopy__
    setattr(result, k, copy.deepcopy(v, memo))
  File "/usr/local/lib/python3.7/copy.py", line 180, in deepcopy
    y = _reconstruct(x, memo, *rv)
  File "/usr/local/lib/python3.7/copy.py", line 280, in _reconstruct
    state = deepcopy(state, memo)
  File "/usr/local/lib/python3.7/copy.py", line 150, in deepcopy
    y = copier(x, memo)
  File "/usr/local/lib/python3.7/copy.py", line 240, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "/usr/local/lib/python3.7/copy.py", line 180, in deepcopy
    y = _reconstruct(x, memo, *rv)
  File "/usr/local/lib/python3.7/copy.py", line 280, in _reconstruct
    state = deepcopy(state, memo)
  File "/usr/local/lib/python3.7/copy.py", line 150, in deepcopy
    y = copier(x, memo)
  File "/usr/local/lib/python3.7/copy.py", line 240, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "/usr/local/lib/python3.7/copy.py", line 180, in deepcopy
    y = _reconstruct(x, memo, *rv)
  File "/usr/local/lib/python3.7/copy.py", line 280, in _reconstruct
    state = deepcopy(state, memo)
  File "/usr/local/lib/python3.7/copy.py", line 150, in deepcopy
    y = copier(x, memo)
  File "/usr/local/lib/python3.7/copy.py", line 240, in _deepcopy_dict
    y[deepcopy(key, memo)] = deepcopy(value, memo)
  File "/usr/local/lib/python3.7/copy.py", line 169, in deepcopy
    rv = reductor(4)
TypeError: can't pickle _thread.lock objects

We're on Airflow 1.10.13 and running into the same issue when trying to clear a GreatExpectationsOperator task.

@Casyfill Oh I think I just found a solution :sweat_smile:. Threads seem to be used to send usage statistics so I disabled them in great_expectations.yml:
anonymous_usage_statistics:
  enabled: false

Thanks @edgarrmondragon , that makes a lot of sense.
Perhaps still makes sense to turn this off automatically (or flash a warning) by default as part of the operator

Perhaps still makes sense to turn this off automatically (or flash a warning) by default as part of the operator

That would be reasonable as the true solution.

2 cents - it is obviously dependent on your deployment, but apparently, if you remove data_context_id (which I assumed to be unused given enabled:false) from the expectations yaml, it will attempt to override this YAML upon initialization. To avoid that (which does not make sense in airflow, and is forbidden by our settings), just get this line back, while keeping enabled:false

Thanks for helping triage this! Obviously this isn't an ideal solution form our perspective since we'd like to keep usage stats turned on if possible, but I'm glad there's at least a workaround. I'll tag this for review and maybe we can come up with a permanent fix.

On airflow-provider-great-expectations==0.0.5 and great-expectations==0.13.9 I too get error can't pickle _thread.lock objects when attempting to clear the GE Task using the below.
(I am using Astronomer with Airflow 1.10.7
anonymous_usage_statistics:
  enabled: false

If I downgrade to airflow-provider-great-expectations==0.0.4 and great-expectations==0.13.7 I do not see this issue (except when attempting to use a Postgres MetricStore


Possibly wrong implementation of "format_units"

Description
I was looking to use the format-units function, part of ethers-core::utils. I was expecting the same behaviour as in the canonical ethers.js, but the format_units function behaves differently in ethers-rs.
Actual Behaviour
Take the following Rust code:
use ethers::{prelude::*, utils::format_units};
let eth_in_wei = U256::from_dec_str("1000000000000000000").unwrap();
let eth_in_gwei = format_units(eth_in_wei, 9);
println!("eth_in_gwei: {:?}", eth_in_gwei);

This logs the following:
eth_in_gwei:<PHONE_NUMBER>11111111

Expected Behaviour
Take the following JavaScript code:
const { ethers } = require("ethers");
let eth_in_wei = "1000000000000000000";
const eth_in_gwei = ethers.utils.formatUnits(eth_in_wei, 9);
console.log({ eth_in_gwei });

This logs the following:
{ eth_in_gwei: '1000000000.0' }

Which is correct, as per the identity 1 eth = 1e9 gwei = 1e18 wei.
Possible Solution
Rewrite the format_units function to divide by 10^(decimals) instead of just decimals.
Environment
Using<EMAIL_ADDRESS>

This should be fixed in #170 @paulrberg!

This should be fixed in #170 @paulrberg!

hey just doing my first steps with rust and I think there might still be an issue:
// js
ethers.utils.formatUnits("409030851937682650000",18) ->  "409.03085193768265"

// ethers-rs
format_units(&U256::from_dec_str("409030851937682650000"), 18).to_string(); -> "409"

@gakonst can you reopen this issue since it's still not correct?

i think this was addressed in https://github.com/gakonst/ethers-rs/pull/463?

I have modified the parse_unit test to show that if you have 18 decimal places some are lost since f64 doesn't have enough bits to hold such a high precision number.
https://github.com/x3ccd4828/ethers-rs/commit/617997a41d736ab93a18e6bb461656eeb61769dc

I can submit a pull request with a fix but it will have to use the rug crate (https://crates.io/crates/rug) to create a float larger than 64bit.

@gakonst have you had a chance to look at the modified test?

+1 on the test and using rug with default-features = false


Add a flag for the current "Site"

If the user switches between sites when editing an entry, the body class should adjust accordingly.

Also for the entry index page.

This is done. As of 2.1.0, there is now an entriessite-* class which exists when you are editing a particular entry.
Unfortunately, the same class is not possible on the entry index page. On the index page, the URL does not change when switching between sites. It could potentially be done via some complex JS, but it's not something I was able to add at this time.

Thanks for adding this! If you're able to return to the more complex problem (of flagging pages based on site, even when the URL doesn't reflect it) in the future, I'd be very excited.

Current workaround:
var header = document.getElementById('header')
var site1Handle = 'Sustainable Agriculture and Food Systems Minor'
var site2Handle = 'EFFS Staging'
var site1Class = 'entriessite-safs'
var site2Class = 'entriessite-effs'

if ( header.innerHTML.toString().indexOf(site1Handle + '
') > -1 ) {
  document.body.classList.add(site1Class)
  document.body.classList.remove(site2Class)
}

if ( header.innerHTML.toString().indexOf(site2Handle + '
') > -1 ) {
  document.body.classList.add(site2Class)
  document.body.classList.remove(site1Class)
}


Templates

Added Templates support for Facebook,
example:
$botman->reply(
    Template::create('generic')
        ->addElement(
            Element::create($trackName)
                ->image(@$artist['image_large'])
                ->subtitle("By: " . @$artist["name"])
                ->url("https://www.youtube.com/watch?v=" . $id)
            ->addButton(
                Button::create($trackName)
                    ->url("https://www.youtube.com/watch?v=" . $id)
                    ->fallback_url("https://www.youtube.com/watch?v=" . $id)
            )
        )
);

Current coverage is 77.64% (diff: 16.17%)

Merging #148 into master will decrease coverage by 7.26%

@@             master       #148   diff @@
==========================================
  Files            35         37     +2
  Lines          1246       1360   +114
  Methods         251        263    +12
  Messages          0          0
  Branches          0          0
==========================================
- Hits           1058       1056     -2
- Misses          188        304   +116
  Partials          0          0

Powered by Codecov. Last update a1cc63c...b31fd1e

Thank you! I modified the code a bit and moved the Facebook-specific classes into it's own namespace.

Very nice. Thanks!


value for count and status are inverted

Here an example of the sraped vlaues:
HELP connection_status_count number of socket with same parameter.
TYPE connection_status_count gauge
connection_status_count{dsthost="<IP_ADDRESS>",dstport="22",name="ssh-from-localhost",process="",protocol="tcp",srchost="<IP_ADDRESS>",srcport="*",status="established"} 1
HELP connection_status_up Connection status of the socket (0 down - 1 up).
TYPE connection_status_up gauge
connection_status_up{dsthost="<IP_ADDRESS>",dstport="22",name="ssh-from-localhost",process="",protocol="tcp",srchost="<IP_ADDRESS>",srcport="*",status="established"} 5

fixed in 0.1.3


Wrong call disposal logic in UnknownMethodHandler.

Grpc.Grpc.Core.Tests.Grpc.Core.Tests.ClientServerTest.UnknownMethodHandler ...
Caught exception in a native handler: System.InvalidOperationException: Call has already been disposed.
at Grpc.Core.Internal.AsyncCall2[System.Byte[],System.Byte[]].CheckNotDisposed () [0x00018] in /usr/local/google/home/jtattermusch/github/grpc/src/csharp/Grpc.Core/Internal/AsyncCall.cs:382    at Grpc.Core.Internal.AsyncCall2[System.Byte[],System.Byte[]].ReceiveMessageAsync () [0x00014] in /usr/local/google/home/jtattermusch/github/grpc/src/csharp/Grpc.Core/Internal/AsyncCall.cs:311
at Grpc.Core.Internal.AsyncCall`2[System.Byte[],System.Byte[]].HandleReadFinished (GRPCOpError error, IntPtr batchContextPtr) [0x000bc] in /usr/local/google/home/jtattermusch/github/grpc/src/csharp/Grpc.Core/Internal/AsyncCall.cs:567

This has been resolved.


Static/dynamic library products

Resolves #159
This workaround Xcode Cloud builds (due to longstanding Xcode Cloud bug that can't build and link dynamic dependency properly)

who's in power to merge it?

@SplittyDev let me know if this solution looks sensible to you.

@davdroman Generally looks good to me. I'm not sure whether the static entry is needed, though. Afaik, static linkage is the default in Swift, so adding an entry for dynamic linkage should be enough to provide both. Providing both Introspect and Introspect-Static shouldn't be necessary, unless I'm missing something.

@SplittyDev static is the whole point of this change because automatically it happen to go dynamic and Xcode build fail

Xcode Cloud bug that can't build and link dynamic dependency

Great stuff. I'll merge this as soon as the CircleCI situation is sorted.

@SplittyDev static is the whole point of this change because automatically it happen to go dynamic and Xcode build fail

Xcode Cloud bug that can't build and link dynamic dependency

You're absolutely right, I got this wrong at first. Great change, thanks for the PR!

@krzyzanowskim #179 should be fixed now. Can you push an empty commit to retrigger the CI please?

This is no longer blocked by CircleCI FYI.

Hi @krzyzanowskim. 0.2.0 is now out and includes this change. Please let us know at some point whether this fixed your Xcode Cloud issues. Thanks!


create new repo

Type of repository request
new repository
Requested name for the repository
infra.leapp
Name of the repository to create or location of existing repository to migrate.
Repository Description
Collection of Ansible roles for automating RHEL in-place upgrades using Leapp.
Associated CoP
The CoP this repository is associated with.
Admin Access
validated_content_curators
Write Access
automation-cop-members
Read Access
none
Additional details
Any additional information or context to provide for this request.

Review request:
cc: @redhat-cop/container-cop-core

this has been created


Redirect type not set when adding from Suggestions

When adding a redirect from Suggestions, the redirect type is set to 0.
I think it would be better to either set a default value of temporary or permanent. An alternative could be to add the possibility to set the type when adding the redirect from Suggestions.

I second this request - the fact that redirectType is set to 0 is very frustrating and makes it useless to add a redirect from the Suggestions list.

@marisks @ellinge any chance you'll look at this issue?


curv: init at 0.5

Description of changes

curv: init at 0.5

Curv is a programming language for creating art using mathematics. It's a 2D and 3D geometric modelling tool that supports full colour, animation and 3D printing.
https://github.com/curv3d/curv
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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

@ofborg build curv

diff LGTM, you may wish to order the attributes according to https://github.com/jtojnar/nixpkgs-hammering/blob/main/explanations/attribute-ordering.md. And the build is failing on darwin, looks like some missing dependency?

Force-pushed a fix with documented darwin deps, but the build still seems to fail. Without a  mac to test with i'll mark it as broken on darwin.
@ofborg build curv

Cannot nix-instantiate `curv' because:
error: Function called without required argument "openmp" at /var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/ofborg-evaluator-5/pkgs/applications/graphics/curv/default.nix:14, did you mean "openlp", "openmm" or "openmpi"?

Rebased


[beta][UI][UX] Seeing multiple challenges with scrollbars for each parent div

Parent divs have scrollbars

Please remove me from the thread
Sent from my iPhone

On Sep 1, 2017, at 9:16 PM, Quincy Larson<EMAIL_ADDRESS>wrote:
Closed #13121.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@jburtiv
looks like you may have accidentally subscribed to all notifications for the FreeCodeCamp repository. To stop receiving notifications please visit the subscriptions page and select "Not watching". Additionally, you can visit the GitHub Help page on managing notification delivery methods for more help.
We are sorry for your inconvenience!


[FEATURE] Check MLModelTool agent before deleting MLModel

Is your feature request related to a problem?
MLModelTool uses MLModel. When deleting the MLModel, we need to first check whether any of the agent has a MLModelTool using this MLModel.
What solution would you like?
Just like connector, it will return an error if the connector is being used by any model:
1 models are still using this connector, please delete or update the models first: [L05NKJIBVJ7VbbiuxOUx]
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

Thanks @yuye-aws, Do you have bandwidth to help on this ?

Assign to you first, let me know if you don't have enough bandwidth.

I'm working on the opensearch-ml-quickstart project in these weeks. @ylwu-amzn You can mark this issue as 2.19.

Plus one use case: Check ConnectorTool agent before deleting the connector

Similar issue: https://github.com/opensearch-project/ml-commons/issues/3088

@dhrubo-os The issue mentions that scan ingest pipelines, search pipelines, any other places where the models may be used in existing resources. Can you at least provide a list of possible places where ml model is being used.

@ylwu-amzn Hi Yaliang, I can take this issue and come up with a RFC ASAP since yuye doesn't have enough bandwidth.

@ylwu-amzn Hi Yaliang, I will take this issue and come up with a RFC ASAP since yuye doesn't have enough bandwidth.

Thanks Xinyuan! This issue can be assigned to you, right?

@xinyual Found that RAG tool also has two fields: embedding_model_id and inference_model_id  https://opensearch.org/docs/latest/ml-commons-plugin/agents-tools/tools/rag-tool/

Also the conversational agent needs llm.model_id https://opensearch.org/docs/latest/ml-commons-plugin/api/agent-apis/register-agent/

The model controller uses model_id https://opensearch.org/docs/latest/ml-commons-plugin/api/controller-apis/create-controller/

There is a search response processor named retrieval_augmented_generation using model_id : https://opensearch.org/docs/latest/search-plugins/search-pipelines/rag-processor/

Raise a RFC for this issue. @yuye-aws @ylwu-amzn @dhrubo-os Please have a look.


mat-select is empty if selected option has value 0

I have this problem:

The mat-select's selected option is "Active", and its value is 0 (NOT "0").
The placeholder (status) doesn't float. I used Augury to check the component's state and I noticed this:

I checked why the FormlyWrapperFormField is empty and found the following line of code:
https://github.com/formly-js/ngx-formly/blob/b56c8aa9381215103b194c042e008134da617bd6/src/material/form-field/src/field.type.ts#L63
this.formControl.value in this case is 0 so the expression returns true. Should you check against null or undefined?

I'm now using you default implementation and it doesn't happen anymore. I was using a custom select component (more or less copied from an older version of your implementation).

I have reproduced this issue in the demo app:

in demo/src/app/ui/common/input/app.component.ts add type: 'number' to the input field.
open the demo app and go to http://localhost:4100/ui/material and type 0 in the first input, now of type number.
the label of the field won't float upon losing the focus.
I've made a fix, will submit a PR in a few minutes.

fixed in 5.0.4


Raise exception if an upload issue occurs.

Overview
Brief description of what this PR does, and why it is needed (use case)?
When S3Sync is used as a module, errors in upload are a print message to the console. There's no way to programatically determine if something went wrong in the most basic of sense.

LGTM


Fix some document errors

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

The implementation of those functions might now always return success, but in the future may return an error. It should not be documented as always returning success.


Use getHost instead of getContainerIpAddress()

getContainerIpAddress is deprecated.

@eddumelendez nice! 💯 Could you please change examples project too?


Create 0000-01-02-immanuella793.md

Previously I was confused about how I used the learning lab, but the steps to start using the learning lab I could understand after I watched the tutorial on using the learning lab on youtube

To be honest, these eight steps had a hard time for me.

To be honest, these eight steps had a hard time for me.


Cleanup profile_list default handling

Based primarily on https://github.com/batpad/kubespawner/pull/1

Have a single function that makes sure slugs are set and defaults are set. This
allows all other code to assume these are always set, simplifying it. Centralizing
all this autogeneration code in one place (+ adding a test for it) helps. We don't
need to specialcase any code for defaults in our HTML template anymore
Remove self._profile_list instance variable, so it is easier to reason about
profile_list. Everyone who wants it gets it from self.profile_list and then passes
through the defaults setting function.
Split the large _load_profile function into smaller pieces so it is easier to reason
about.

I've merged this by hand into https://github.com/jupyterhub/kubespawner/pull/735 as I didn't have rights to merge this!


Does not work with ChatGPT4

I have an account and pay for ChatGPT4 but if i put in the model as "gpt-4" as suggested in the API documents i get an invalid response

your $20/mo subscription to ChatGPT+ gives you access to GPT-4 when using the OpenAI chat web interface, but it does not give you access to the gpt-4 API.  You have to pay separately for that.  Very confusing, I know.  See this page for more info: https://help.openai.com/en/articles/7102672-how-can-i-access-gpt-4


suns.py tool

With pysunspec there was a nice CLI utility, suns.py. With pysunspec2 I didn't find it.
Can I adapt the suns.py making it work with pysunspec2?
Thanks a lot for this library.

We will look at adding it in pysunspec2 in the next release.

We will look at adding it in pysunspec2 in the next release.

Thanks a lot Bob.


Change all existing dependencies in toml file concurrently

Hi @StrajnarFilip,
the following feature does not work in v0.2.0. Please check it out and implement this feature which has a very high priority.
toml-adapt -path pyproject.toml -a change -dep ALL -ver X
ALL represents all dependencies, while X means that we assign default version (in case of poetry, we assign *).

Should be done since the last commit.
Best regards!

Proposed in #13.


Command does not exist

@SaladiHarini
I removed the SharePoint Online Management Shell from my computer and installed the current version from the link on this page (16.0.21610.12000)
I was then able to follow the provided steps until this command should be used:
Add-SPOListDesign
Powershell throws an error that this CmdLet does not exist. Digging through the available CmdLets I couldn't find anything that looks like that.
Where could we download a SPO Management Shell that contains this CmdLet?

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

ID: 6665386e-1ad4-e9d3-565b-4b3cb3c3ea99
Version Independent ID: 870237f7-66ec-270e-1372-c993e8dedbe9
Content: Custom list templates - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/lists-custom-template.md
Service: sharepoint-online
GitHub Login: @kaarins
Microsoft Alias: kaarins

@cwdata: Add-SPOListDesign is a new Lists specific cmdlet that should be available to you are running this against a tenant that is in TR. [We are in the process of rolling out to PROD over the next few weeks]. We are also working on updating the documentation here to include details of the cmdlet.

@SaladiHarini
thanks for the quick response. Yes, the tenant I'm connecting to is set to Targeted Release for everyone.
And I agree with you that the cmdlet Add-SPOListDesign 'should' be available, but unfortunately it is not.
Entering the command
Get-Command ListDesign
in Powershell yields no results. Hence that cmdlet is missing in the currently available SharePoint Online Management Shell.
So I'm wondering if there is a beta version or something like that of the Management Shell which we could download and which contains the command. We would like to try out this functionality.

Hey @cwdata! I've tested with a TR tenant and I do see all the list related cmdlets available in the version of Microsoft.Online.SharePoint.PowerShell that I'm running (16.0.21513.12000). Let's try to troubleshoot this! Thanks for your patience :)
Here are some steps I recommend:

Run PS as administrator. Use the following command to check your version of Microsoft.Online.SharePoint.PowerShell
Get-Module -Name Microsoft.Online.SharePoint.PowerShell -ListAvailable | Select Name,Version
If it doesn't match the version I'm on (which is the latest), please uninstall and let's try to reinstall via Install-Module
Here is how you can install and update from PS instead of trying to get the latest version by downloading every single time:
Install-Module -Name Microsoft.Online.SharePoint.PowerShell
You can then run the following command to make sure you are always running the latest bits!
Update-Module -Name Microsoft.Online.SharePoint.PowerShell

I'm hoping at the end of this you'll be able to run through all the steps in the article, but if not, please reach out and we can connect on a call to troubleshoot live!

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@SaladiHarini,
thanks to your help I found out that even though I uninstalled the old version of the SPO Management Shell another old version was still loaded from somewhere else. After removing that one with Uninstall-Module the command Add-SPOListDesign is now available to me.
Another quick question: Is it planned to have this kind of templates also for document libraries?
@scanum,
the availability of commands in a PowerShell session is in no way related to the tenant. It's a matter of what the installed modules on the local system have built in. It might be a good idea to add a paragraph on how to identify and fully uninstall all old versions of the SPO Management Shell, which would be:
Get-Module -Name Microsoft.Online.SharePoint.PowerShell -ListAvailable | Select Name,Version
and if any were found:
Uninstall-Module -Name Microsoft.Online.SharePoint.PowerShell -AllVersions
This might need to be done in addition to uninstalling the Shell from 'Installed Apps'.


Tab navigation issues: Bad visibility and illogical order

Bad visibility

On the sceenshot the second icon is active, you see a slight blue line around the icon and the icon is a little bigger than the left one. The contrast and visibility of an active tabed element is too weak. We should use double colored borders (that work on light and dark background) and / or a color that is clearly distinguishable from the button color!
Illogical order
The tab order on the player needs a rework: At the moment the upper icons (in den silde and video) are the first one (1) then the player UI icons (2) are navigatable. So you have to tab to all icons above to go to the essentials UI elements like play / pause / forward / backward:

We should change this: After a fresh loaded player page the fist tabbing should go to area 2!
Also in the area tab order is Illogical

and should be re-orderd so that the visual order should match the tab order in general.
Same in the main navigation: Currently you jump from play to settings and then back to backward / forward /

In addition: The volume slider is never accessible by tabbing. You can only mute the video but the volume slider does never show up by using the tab key only:

Other players (with fading in / out volume slider) do have this functionality, e.g. YouTube:

@cgruber-uos We updated the player. Can you test it works as expected?
Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar-multiresolution-remote

Thanks, much better order now! Still some issues:

the volume slider is still not accessible via keyboard only, by focusing the loudspeaker I can press enter and mute or unmute but I have no opportunity to adjust the volume because the slider is not apperearing.
the blue border for a focused element is good visible on the grey background but nearly not visible on the video / slides "inline" buttons. They are also blue (same color!!) so the icons appears just slight bigger than the other - in case you know it! :-)

vs.

Are the colors set or can we change the border color or - perhaps a better solution - can we add an additional slight light border (on the video / slide inline buttons only) inside the focus border to make focused more prominent

I changed de border color of element selected.

I changed for all buttons (slide inline and playbar).
@oas777 What do you think? maybe lisa has a better option
Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar-multiresolution-remote

Well, before we ask Lisa, we probably should ask Clemens whether this is the best option for accessibility. Because if it is, the design is less important...

Yes. Clemens suggested to change the border of the buttons of the slides because accessibility.
Clemens say the blue border button in the play bar is correct.
But If we change the border in slide buttons, should we change also in playbar buttons? or we are going to have two color border (one for playbar buttons and another for slide buttons)?
Also we need to choose the border color. I selected a "pink" border to test. But I'm not sure if its the best color. In this Clemens can help us.

I think we should find a good solution for a11y and desing. So the pink color would not be my favourite (with the desing hat :-) And I would stay with the former color in the navigation area and an other in the video, in case different imlementaion it is not too costly. I would try a dark blue for the inline buttons.
Miguel / @miesgre can you set - for testing only - a dark blue for tab focus, just to see how it woks. In case it is ok we can implement different colors for navigation and video / slides.
This would be not totally consistant, because we would use two different colors for the same effect. But I think it works because in the navigation are we have a mor "dark theme" (because of the background color) and in the video a "light theme", but let's try!

I changed the border to a dark blue color (slides and playbar)

No, bad idea from my side, too less contrast! Pink?! Olaf / Lisa any idea for a color that fits both?

@oas777 Seems tab navigation issues are resolved. What about the focus border color (with keyboard)? Pink is good? Lisa prefers another color?
Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls
Playbar buttons:

Video buttons:

A bit ot:

Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls

Is the demo URL working on your side? I got only a never ending loading icon after clicking on the preview image:

There is a problem with Safari and CORS. I will try to fix it. Meanwhile you can use chrome (maybe edge too)

A bit ot:

Demo: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls

Is the demo URL working on your side? I got only a never ending loading icon after clicking on the preview image:

Clemens, good to see you are still here...
As for the color of the border: The designer would ask for blue, the accessibility expert calls for pink: Can we meet in purple? #800080

Changed to purple, but maybe it does not have enough contrast?

Clemens?

PS: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls works on Edge and Chrome, but not on Firefox (all Windows). Will test on OS X later.

PS: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls works on Edge and Chrome, but not on Firefox (all Windows). Will test on OS X later.

Earlier version of the test page did work on FF, I had no issue with all test via https://polimediaupv.github.io/paella-ethz/... till now. so it must be a newly introduced but or a change in FF?

the accessibility expert calls for pink:

I fear pink was introduced by chance. :-) No reason beside contrast.

Changed to purple, but maybe it does not have enough contrast?

And this is the problem with purple: Less contrast. Not only with the dark blue in the navigation bar, also with the light blue buttons in the video / slide area. Miguel / @miesgre can you play a bit with the color, perpaps a trial could be a light purple to get a higher contrast on both elements inline buttons above and navigation buttons below.

Seems correct to point the finger: OS X works with Safari (!) and Chrome - but not on Firefox.

PS: https://polimediaupv.github.io/paella-ethz/?id=belmar_16_9_hls works on Edge and Chrome, but not on Firefox (all Windows). Will test on OS X later.

Earlier version of the test page did work on FF, I had no issue with all tests via https://polimediaupv.github.io/paella-ethz/... till now. so it must be a newly introduced bug or a change in FF.

It should now work on Firefox. It was a server misconfiguration.

Yes (for Windows), thanks.

Miguel / @miesgre can you play a bit with the color, perpaps a trial could be a light purple to get a higher contrast on both elements inline buttons above and navigation buttons below.

I updated to use 20% lighter purple.
Other colors we can use:
#800080 purple
#cc00cc 20% lighten
#f200f2 30% lighten
#ff19ff 40% lighten
#ff40ff 50% lighten
@oas777, @cgruber-uos Opinions?

I let Clemens decide.

In the video the background color of the buttons is #3a8bdd
In the navigation bar the background color is #1f2937
Results WCAG Contrast checker
#cc00cc (20% lighten)

background #3a8bdd vs. #cc00cc (20% lighten)
-> contrast ratio: 1.33, did neither fit AA nor AAA level requirements
background #1f2937 vs. #cc00cc (20% lighten)
-> contrast ratio: 3.1, fits AA for big elements but nut AAA

#f200f2 (30% lighten)

background #3a8bdd vs. #f200f2 (30% lighten)
-> contrast ratio: 1.02, did neither fit AA nor AAA level requirements
background #1f2937 vs. #f200f2 (30% lighten)
-> contrast ratio: 4.23, fits AA for big elements but nut AAA

And so on. That means we will never get a matching color for both, the contrast with the dark background and the contrast with the light backgrund.
I wold try #f200f2 (30% lighten) so the contrast in the main navigation is high and would fit good with a darker video background but not so good with a light video.

Changed to #f200f2

@oas777 @cgruber-uos ?
I think the only thing left to do is to choose the color of the border.
I changed the border color to #f200f2.

Is it ok? Can we close this issue?

Let's wait a week for Clemens to answer (he's not working for the Opencast project anymore), otherwise we'll stick with the color and can close this.

Not the optimum but ok I would say!

That's all we wanted to hear...


set ULIMIT in nginx.sysconfig.erb

This PR fixes "Too many open files" error.

This is a critical issue. Without this fix nginx can't be used under heavy load.
http://serverfault.com/questions/641899/ulimit-file-descriptor-limits-not-being-applied-for-particular-process

Thanks


Azure Web PubSub service: MapWebPubSubHub extension with custom hub name

Contributing to the Azure SDK
Adding extension method for specifying custom hub name:
public void Configure(IApplicationBuilder app)
{
    app.UseEndpoints(endpoint =>
    {
        endpoint.MapWebPubSubHub<SampleHub>("/eventhandler", "customHub");
    });
}

Updated documentation to make it clear that by default class name is used for hub name.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Microsoft.Azure.WebPubSub.AspNetCore


[SPARKR-178] Integrate with SparkR with spark-submit

This patch integrates SparkR with spark-submit and allows SparkR to be used in yarn-client and yarn-cluster mode. This is done by adding a new SparkRRunner class which is invoked as the main class by spark-submit.  As the JVM is launched using spark-submit we automatically pick up Spark config variables in this case.
For the non-yarn use cases, this patch also handles reading in the default Spark config variables (in spark-defaults.conf or passed through cmdline) by launching the SparkRBackend through spark-submit.
Tested on standalone mode on Mac and on a YARN cluster in Cloudera VM. I'll try to test this on a EC2 cluster with YARN soon.
cc @felixcheung -- It'll be great if you can help test this version of the patch on your YARN cluster !

I've updated everything to your PR and it gets an error:
$ export YARN_CONF_DIR=/etc/hadoop/conf
$ export SPARK_HOME=/opt/cloudera/parcels/CDH-5.3.0-1.cdh5.3.0.p0.30/lib/spark
$ export JAVA_HOME=/usr/java/jdk1.7.0_67-cloudera
$ /usr/lib64/R/library/SparkR/sparkR-submit --master yarn-client ./pi.R yarn-client 4
Running /opt/cloudera/parcels/CDH-5.3.0-1.cdh5.3.0.p0.30/lib/spark/bin/spark-submit --class edu.berkeley.cs.amplab.sparkr.SparkRRunner --files ./pi.R --master yarn-client /usr/lib64/R/library/SparkR/sparkr-assembly-0.1.jar ./pi.R yarn-client 4
Loading required package: methods
[SparkR] Initializing with classpath /usr/lib64/R/library/SparkR/sparkr-assembly-0.1.jar

Error in socketConnection(host = hostname, port = port, server = FALSE,  :
  cannot open the connection
Calls: sparkR.init -> connectBackend -> socketConnection
In addition: Warning message:
In socketConnection(host = hostname, port = port, server = FALSE,  :
  localhost:12345 cannot be opened
Execution halted

And it works when I run R

Thanks @felixcheung  for testing -- can you check if the SparkRBackend comes up successfully in the case of running pi.R ? You can do this with jps or something like that. I am wondering if the problem is that it takes longer than the time we wait (2s) for the process to come up or if the process dies for some other reason.

It's possible. I just rerun sparkR-submit again after successfully running the pi.R code in R, and sparkR-submit works.
AFAIK, we are not caching/keeping the JVM process here, and so any idea why it works the 2nd time but not the 1st?

I am not sure there is a deterministic reason. I think it just happens that sometimes it takes longer to open ports etc. There is a open PR to retry these connection requests instead of failing after trying once https://sparkr.atlassian.net/browse/SPARKR-193

Specifically in the SparkRRunner case we have two threads which are kind of racing -- The first thread starts the backend and the second thread starts the R process. Actually I think I can actually add a synchronization here. Let me do that and ping this PR again

Great!
oh and perhaps it should use a random port to avoid conflict (eg. multiple users or jobs), like pyspark/PythonRunner does.

Thats a good point too - Just pushed out a change with the synchronization and random port assignment.

Retested this, a few times, everything is working well!
One side issue is I've noticed a hardcoded port when running sparkR.init in R directly:
sc <- sparkR.init(Sys.getenv("MASTER", unset = "local"))
Launching java with command  /usr/java/jdk1.7.0_67-cloudera/bin/java   -Xmx512m -cp /usr/lib64/R/library/SparkR/sparkr-assembly-0.1.jar::/etc/hadoop/conf edu.berkeley.cs.amplab.sparkr.SparkRBackend 12345
SparkR Backend server started on port :12345

The reason for this is mostly because we exposed sparkRBackendPort as an option to sparkR.init with a default value of 12345.
Thinking more about this I am worried that we will have collisions with other Spark or Hadoop services (for example at 7077 or 8080 or 50030 etc). I think the right thing to do is to start at a well known port and then incrementally check for available ports.  However I am worried this adds too much complexity to this PR and I'm thinking it'll be better to use a fixed port for now and track the random port with a new PR.

Note that we will have a config variable SPARKR_BACKEND_PORT that users can set in case they want to run multiple backends on the same machine etc.

That's fair. I think Py4J (in pyspark) starts with a default port and increments if there is a conflict.
http://py4j.sourceforge.net/getting_started.html

@concretevitamin - Could you take a look at this ? I think this has been tested by me and @felixcheung but it'll be good to get another pair of eyes

Thanks again @felixcheung -- I've also merged this up with current master branch

LGTM. Thanks!

Alright, I'm merging this change now -- Lets continue testing sparkR-submit and see if any other issues come up.


change task background to red

with the pull request we're going to change the Task.js background to red

modified


Responsive offset prop

I need to make the offset prop responsive in any way. I want the stickiness behaviour to be based on the height of 2 elements (header and subheader).
At the moment I'm trying to find a fixed value for the offset that works well across our supported devices.
The other quick solution, I have in mind is to have 2 separate sticky elements (one for desktop and one for tablet). I would really like to avoid doing this.
Thanks again for this awesome package. There are so few packages that deliver this in a small and convenient way. 👍

Two approaches I could offer:

make the surrounding container responsive. e.g.
<StickyBox/>

Use something like react-media:

     <Media query="(max-width: 599px)">
        {matches =>
          <StickyBox offset={matches ? 20 : 40}/>
        }
      </Media>

The second approach seems the better, because it doesn’t rely on the viewport height.
I just realised I have another question. Is it possible to disable this on a certain breakpoint?

The vw in the first example was just one example of responsiveness. You could also used a
 and define media queries for the class.
By disabling do you mean not showing the box at all? Or just disabling the stickyness effect? For the latter you could do something like this:
     <Media query="(max-width: 599px)">
        {matches => matches ? (
           <StickyBox>{content}</StickyBox>
         ) : (
           {content}
         )
        }
      </Media>

I was hoping that, that feature was already in the package, but I guess it’s a nice way to do it.
Thanks for your quick replies and help. 👍


[HOLD for payment 2023-10-02] [$500] Assign Task - Workspace avatar shows 'U'

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch app
Tap on Workspace chat in LHN
Tap plus icon
Tap assign task
Enter title
Tap next
Select assignee and assign to myself
Tap confirm task
Note header "U" Icon and tap on it to note the details

Expected Result:
Workspace is available, so it must not show unavailable icon and it must show workspace profile icon after assigning a task
Actual Result:
Workspace is available, but it shows unavailable icon instead of workspace profile icon in header after assigning a task
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.66-3
Reproducible in staging?: No
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/6d5b2378-9d32-4d69-90e2-b9d45e2d749a
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a3f658f7ff3bedd3
Upwork Job ID:<PHONE_NUMBER>388905472
Last Price Increase: 2023-09-11
Automatic offers:

dukenv0307 | Contributor | 26727093

reproduced

@sobitneupane thoughts on the proposal above?

Thanks for the proposal everyone.
Proposal from @dukenv0307 looks good to me.
🎀 👀 🎀 C+ reviewed

I agree that @dukenv0307's proposal is better and more detailed.

Just waiting on payment

Payouts due:
Contributor: $500 @dukenv0307 (Upwork)
Contributor+: $500 @sobitneupane (NewDot)
Eligible for 50% #urgency bonus? Y
Upwork job is here.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  [@sobitneupane] The PR that introduced the bug has been identified. Link to the PR:

https://github.com/Expensify/App/pull/17992

[ ]  [@sobitneupane] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

https://github.com/Expensify/App/pull/17992#issuecomment-1752468125

[ ]  [@sobitneupane] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

This is a case missed while adding a new feature.

[ ]  [@sobitneupane] Determine if we should create a regression test for this bug.

Yes.

[ ]  [@sobitneupane] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Login with any account
Create a workspace
Go to the workspace chat
Create a task in this chat report
Verify the header of the task report displays the correct workspace instead of the unavailable workspace

Do we agree 👍 or 👎

https://github.com/Expensify/App/issues/27092#issuecomment-1746809730

Eligible for 50% #urgency bonus? Y

Since it is eligible for urgency bonus. Requested $750 for payment in newDot.

$750 payment approved for @sobitneupane based on BZ summary.

Also re-opening this one for the regression test.

pending regression test feedback

I don't think we need a specific regression test for this since it's an UI issue that's easily noticeable. I think we're good to close.


Mod data stored in the savegame (no more xml and cloud support)

I've moved the mod data into the savegame.
The old xml data is imported, but the mod will not use it anymore after the first save.
On the "old" mod, if your savegame is broken, you have to delete che traffic manager xml files. Now you can use the "reset data" flag from the options panel.
Please test and approve this changes :smile_cat:

Why you reverted this PR?
I don't think that bug reported from steam are related to how the data is loaded.

It broke the savegames of all people that used it, irreversibly. Missing textures and overlaying roads might be a null reference error, or wrong set/get of data. It might be a coincidence if one player is saying it, but it is apparent if 15 people post the same issue in 2 hours, when usually I get one comment per day.

Last I tested it, it worked fine, but as with most things, a certain common case was not tested, and it breaks the savegames badly.

If they had the issue on first load, there are no difference from the actual data loading and there are no difference between the MemoryStream and a FileStream in a .net class serializator (also Traffic++ and other mods uses this pattern).
There are some bugs on data loading that could affect a different data loading timing?
I see a strange timer on save function that avoids some null reference exception and MemoryStream from the savegame is faster than load data from the file. This could be a test case.
I hope you will investigate more, because I'm unable to recreate issue they reported with my 3 saves with +100k people.


fix: tests-pytest/action.yml: inputs: add skip-install

This PR proposes add a missing definition of the skip-install input in tests-pytest/action.yml:
  skip-install:
    description: >
      Whether to skip install.
      Should be false with poetry as build-backend.
      The default value is ``false``.
    default: false
    required: false
    type: boolean

Noticed with https://github.com/ansys-internal/simba-core/actions/runs/8327410143/job/22785207488
Run if grep -q 'build-backend = "poetry\.core\.masonry\.api"' "pyproject.toml"; then
/home/runner/work/_temp/508acb6f-5ce8-452e-84cd-fd2b2305b3de.sh: line 2: poetry: command not found

This is because inputs.skip-install does not exist, then this step that installs poetry is skipped in tests-pytest/action.yml
    - name: "Install project (if required)"
      if: inputs.skip-install == 'false'
      shell: bash
      run: |
        if grep -q 'build-backend = "poetry\.core\.masonry\.api"' "pyproject.toml"; then
          python -m pip install poetry
          poetry install
        else
          python -m pip install .
        fi

Then, with skip-install: false added:
https://github.com/ansys-internal/simba-core/pull/6/commits/191f76402b87d06257706c40c0129dbebc86011e
.github/workflows/ci_cd.yml
      - uses: ansys/actions/tests-pytest@d34d2c5cc77e0bfe0afcb6d913f38e0ad8c5f7bf
        with:
          pytest-extra-args: "--cov=ansys --cov-report=term --cov-report=html:.cov/html"
          skip-install: false

Then in https://github.com/ansys-internal/simba-core/actions/runs/8327598885/job/22785773885  poetry install is restored
...
Run if grep -q 'build-backend = "poetry\.core\.masonry\.api"' "pyproject.toml"; then
  if grep -q 'build-backend = "poetry\.core\.masonry\.api"' "pyproject.toml"; then
    python -m pip install poetry
    poetry install
  else
    python -m pip install .
  fi
...

https://github.com/ansys/actions/actions/runs/8331799321/job/22799487255?pr=427
Run ansys/actions/branch-name-style@main
Run branch_name=cfrisson/tests-pytests/inputs/add-skip-install
\033[[1](https://github.com/ansys/actions/actions/runs/8331799321/job/22799487255?pr=427#step:3:1);91m[ERROR]: Branch name cfrisson prefix is not valid.
Error: Process completed with exit code 1.


Segmentation fault when using a theme

When trying to use theme from a file using "-t" flag the program experiences segmentation fault and crashes.
(Without any C experience, but I tried my best to debug it)
How to reproduce (Void Linux):

Clone repository from latest commit.
Compile with gcc (only tested with it).
Pass "-t" flag to executable with path to theme.
Segmentation fault.

I dug a bit deeper and compiled the program with gcc "-g" flag and run with gdb:
(gdb) run -t ~/.config/hellwal/themes/catppuccin.hellwal
Starting program: /home/main/hellwal/hellwal -t ~/.config/hellwal/themes/catppuccin.hellwal
[INFO]: Loading static theme: /home/main/.config/hellwal/themes/catppuccin.hellwal

Program received signal SIGSEGV, Segmentation fault.
unmap_chunk (self=<optimized out>) at src/malloc/malloc.c:518
warning: 518    src/malloc/malloc.c: No such file or directory

(gdb) backtrace
#0  unmap_chunk (self=<optimized out>) at src/malloc/malloc.c:518
#1  free (p=<optimized out>) at src/malloc/malloc.c:529
#2  0x00005555555778ed in process_theme (
    t=0x55555557ed00 "%% color0 = #1e1e2e %%\n%% color1 = #f38ba8 %%\n%% color2 = #a6e3a1 %%\n%% color3 = #f9e2af %%\n%% color4 = #89b4fa %%\n%% color5 = #cba6f7 %%\n%% color6 = #94e2d5 %%\n%% color7 = #bac2de %%\n%% color8 = #6c7"...,
    pal=0x7fffffffe390) at hellwal.c:1938
#3  0x0000555555577952 in process_themeing (theme=0x7fffffffe833 "/home/main/.config/hellwal/themes/catppuccin.hellwal") at hellwal.c:1950
#4  0x0000555555574510 in get_color_palette (p=...) at hellwal.c:1113
#5  0x0000555555577a69 in main (argc=3, argv=0x7fffffffe538) at hellwal.c:1969

Theme file contents:
%% color0  = #1e1e2e %%
%% color1  = #f38ba8 %%
%% color2  = #a6e3a1 %%
%% color3  = #f9e2af %%
%% color4  = #89b4fa %%
%% color5  = #cba6f7 %%
%% color6  = #94e2d5 %%
%% color7  = #bac2de %%
%% color8  = #6c7086 %%
%% color9  = #f38ba8 %%
%% color10 = #a6e3a1 %%
%% color11 = #f9e2af %%
%% color12 = #89b4fa %%
%% color13 = #cba6f7 %%
%% color14 = #94e2d5 %%
%% color15 = #a6adc8 %%

I forgot to mention I use musl libc on my computer but for all my time using it I haven't experienced any issues with compiling or running programs. I might need a bit of help to provide more detailed information if that is specific to my system.

Hello! Thanks for pointing this out, I tried latest release on alpine also with musl, in container and its working for me, can you check if its working for you now?

Thanks for responding, I have cloned the repository with the new commits and installed the program but I am still experiencing the issue. Not sure if I should close this issue as it seems to be the problem with my system or should I provide more information?

It should work without issues, problem is with my code, so don't worry.
I probably found a thing that could cause this, can you check it once again?
I changed it here: b75e671581aaf004650716b3f2666463e635bed6

I cloned the repository right now from main and I am still getting the issue, here's the commands I use:

git clone https://github.com/danihek/hellwal
cd hellwal
sudo make clean install
hellwal --theme ~/.config/hellwal/themes/catppuccin.hellwal

Unsure if I do something wrong but I get the same error.

okay... I can't replicate error, but let's try some other things, I see you've copied themes folder to ~/.config/hellwal/
First one should work without full path, second one is path to relative theme from repo:

hellwal -t catppuccin.hellwal
hellwal --theme ./themes/catppuccin.hellwal

Check if this is working, and give me tree of your ~/.config/hellwal please :)

This is a bit weird because I now get this error for whatever reason when using themes from the repository:
$ hellwal --theme ./themes/catppuccin.hellwal
[INFO]: Loading static theme: ./themes/catppuccin.hellwal
[ERROR]: Not enough colors were specified in color palette: ./themes/catppuccin.hellwal

And here is the tree of my configuration:
$ tree
.
├── templates
│   ├── fish.fish
│   ├── nvim.lua
│   └── xresources
└── themes
    └── catppuccin.hellwal

3 directories, 4 files

Okay, I managed to fix the error by remove spaces after definitions in ./themes/catppuccin.hellwal file on the repository and now getting the segmentation fault like before.


Is NGT::Index::search thread-safe?

First off, thank you for NGT! It's quite impressive.
I'm trying to parallelize a large number of queries with OpenMP. My parallelized loop calls search() on a read-only index, then further processes the results. However, I seem to get nondeterministic results from this, and I'm wondering if the problem is that this method is not thread-safe.
I should also note that I've tried both with and without the shared memory allocator - I did not observe a difference in behavior.

NGT::Index::search is thread-safe. There are cases where search returns different results for the same query, because search uses randomly select nodes from the leaf of the tree as seeds to explore the graph. However, the possibility is quite low. Also the possibility depends on the indexed dataset.
I would like to confirm that you don't find the issue when you don't use OpenMP. If so, could you provide your simplified search source code and your NGT index? I will check them.

Thanks! I have not been able to reproduce the issue without OpenMP, but it's possible that I'm missing something.
My simplified search code looks like:
struct Edge {
  std::size_t id;
  std::size_t neighborId;
  float distance;
};

#pragma omp declare reduction (merge: std::vector<Edge> : omp_out.insert(omp_out.end(), std::make_move_iterator(omp_in.begin()), std::make_move_iterator(omp_in.end())))

#pragma omp parallel default(none)                                             \
    shared(edges, points, numPoints, radius, indexPath)
{
  NGT::Index localIndex(indexPath, true);
#pragma omp for nowait reduction(merge : edges)
  for (std::size_t i = 0; i < points.size(); ++i) {
    const auto &point = points[i];
    NGT::SearchQuery q(point);
    NGT::ObjectDistances neighbors;
    q.setResults(&neighbors);
    // To ensure that we get everything within range
    q.setSize(numPoints);
    q.setRadius(radius);
// NOTE: Including this critical section removes the bug behavior in my testing - it just also removes the parallelism
// #pragma omp critical
// {
    localIndex.search(q);
// }
    for (const auto &neighbor : neighbors) {
      // NGT indices start at 1; ours start at 0
      const auto neighborId = neighbor.id - 1;
      if (idx != neighborId) {
        edges.emplace_back(Edge{idx, neighborId, neighbor.distance});
      }
    }
  }
}

points is the entire set of nodes added to the NGT index; numPoints = points.size(), and radius is any number in [0.0, 1.0) - 0.33 typically works for exhibiting this behavior.
I can also provide generation code to produce points and the index, if this would be helpful.
The index: ngt_index.tar.gz

Thank you for providing your code and index. However, I could not reproduce differences between parallelism and non-parallelism even by using points extracted from your index. It might depend on the environment. In addition, I am wondering if emplace_back is thread-safe, even though there is no problem on my environment. Apart from this issue, the opened NGT index placed outside of the parallel section can be shared.

Interesting! Thank you for trying to reproduce the issue. The emplace_back should be thread-safe since edges is a reduction variable (meaning that each thread should have a private copy), but perhaps something strange is happening there.


@conda-forge-admin, please rerender

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/orange3-feedstock#113.


Master test failing

@javelinjs @pluskid @tqchen @thirdwing Scala Julia and R tests are failing in master. Could you guys look into it?

The last failure on master seems only due to a 404 not found error while attempting to download inception from R test? https://travis-ci.org/dmlc/mxnet/builds/163794009

@mli I thought we fixed data.dmlc.ml yesterday?

should be fixed now
On Thu, Sep 29, 2016 at 4:59 PM, Eric Junyuan Xie<EMAIL_ADDRESS>wrote:

@mli https://github.com/mli I thought we fixed data.dmlc.ml yesterday?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3419#issuecomment-250625264, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZv4WR6ZUZRb9_f9AtuGEgR_Y1KvoH8ks5qvFDMgaJpZM4KKluJ
.

I think http://data.dmlc.ml/ is still not available.
I can disable that test for now

it works well on my laptop. try it again or try data.mxnet.io?
On Fri, Sep 30, 2016 at 9:28 AM, Qiang Kou (KK<EMAIL_ADDRESS>wrote:

I think http://data.dmlc.ml/ is still not available.
I can disable that test for now
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3419#issuecomment-250789968, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZv4XOVCg9k2xmtbrFBrcxHDkTd9gN-ks5qvTi0gaJpZM4KKluJ
.

404 error https://travis-ci.org/dmlc/mxnet/jobs/163868765#L2574

copied all data in the original data/ directory.
On Fri, Sep 30, 2016 at 2:20 PM, Qiang Kou (KK<EMAIL_ADDRESS>wrote:

404 error https://travis-ci.org/dmlc/mxnet/jobs/163868765#L2574
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3419#issuecomment-250855110, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAZv4cFS-oWcAtSJoBpZAYypBMoJg6NOks5qvX0mgaJpZM4KKluJ
.

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


singleheader.py fixes for Windows and newer Python

Windows assumes cp1252 resulting in:
  ...
  File "...\STC\src\singleheader.py", line 47, in process_file
    process_file(
  File "...\STC\src\singleheader.py", line 28, in process_file
    for line in f:
  File "C:\Program Files\Python312\Lib\encodings\cp1252.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
	   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
UnicodeDecodeError: 'charmap' codec can't decode byte 0x81 in position 426: character maps to <undefined>

Additionally, \* seems to have changed in recent Py3 versions (?), so they need an extra backslash:
...\STC\src\singleheader.py:30: SyntaxWarning: invalid escape sequence '\*'
  if re.search('/\*.*?\*/', line):

Thanks. I have enabled "Beta:Use Unicode UTF-8 for worldwide language support" in the system locale settings in windows, so I don't notice this kind of issues.


Circulation on/off

Hi! Is there any way to switch hot water circulation on/off? I can't find such device.

Hello,
There is no such functionality at API level. Basically, if you cannot do it through the mobile app, you won't be able to do it through the integration.
You can have a look here for some background: https://github.com/thomasgermain/vaillant-component/issues/94 and https://github.com/thomasgermain/vaillant-component/issues/107


Radar system needs transponder support

The radar system needs to detect all entities that are both in range (have a position component) and have a radar_transponder component. The radar_transponder component will contain metadata that can differentiate discovered entities on the UI (name, icon, type, etc)

The transponders need to convey the following information:

The display-friendly name of the object
The type of object that this transponder is representing

Display names for players should be the player's login name (we don't yet have logins, but we should probably rig up Auth0 at some point to get that). The following is a list of object types:

Starbase
Ship
Asteroid

The JSON for the radar_transponder component then might look something like this:
{
    "object_type": "[starbase|ship|asteroid]",
    "display_name": "Asteroid"
}

we should be able to use the CoreUI Pro icons to render a starbase, a ship, or an asteroid. If not, we can find small images suitable for the radar screen display.

In the future, if we want to have different icons for the same object type (e.g. different type of asteroid icons), we can add that later. For now, let's try and keep the scope for radar rendering as small as possible.


NIFI-12898 Allow uploading asset and referencing via Parameter

Summary
This pull request adds the ability to upload assets to parameter contexts and then reference them from parameters in the context. This feature is described in NIFI-12898 and the feature proposal.
A new framework extension - AssetManager -  is responsible for storing/retrieving assets on the file system.
The following REST end-points are added:
Create/upload an asset:
POST /parameter-contexts/{id}/assets
List assets:
GET  /parameter-contexts/{id}/assets
Get asset content:
GET  /parameter-contexts/{id}/assets/{assetId}
The last two end-points are primarily intended to be used when a node joins the cluster for it to compare it's current assets with the coordinator and synchronize it's assets to match.
To reference an asset it is done through a normal parameter context update, where a parameter will send no value and instead provide an asset reference:
referencedAssets: [
  {
     "id": "assetId1"
  },
 {
     "id": "assetId2"
  },
]

The back-end will automatically make the value of the parameter become the concatenated paths of the referenced assets.
The following CLI commands were added:
nifi create-asset -pcid <param-context-id> -af <full-path-to-a-file>

nifi list-assets -pcid <param-context-id>

nifi add-asset-reference -pcid <param-context-id> -pn <param-name> -aid <asset-id>

nifi remove-asset-reference -pcid <param-context-id> -pn <param-name> -aid <asset-id>

When a parameter context is updated, any asset that was previously referenced by a parameter and no longer referenced after the update will be deleted.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks @exceptionfactory ! I noticed the CodeQL issues as well...
I did sanitize the Filename header value before passing it to any code that would use it to write a File.
If that isn't enough then I suppose a fall back option would be to write the file names with the generated UUID and store some kind of mapping file along side the asset that contains the original filename.
The problem with this is that the value of the parameter that references the asset is going to be the concatenated paths of the assets and it's going to contain file names that the user has no idea what they are. Example, upload PostgresDriver.jar and it generates id 123, then reference this asset in a parameter named db_driver, and the value of the parameter will be '/path/to/assets/123'. So some looking at the parameter has no idea what file that is.

Thanks for the reply @bbende. Several classes mitigate the path traversal problem using Java Path objects with normalization and checking that the resolved child path starts with the selected parent path.
https://github.com/apache/nifi/blob/83d2078a4f878ba742f8adba2da382fb117ad6c0/nifi-system-tests/nifi-system-test-extensions-bundle/nifi-system-test-extensions/src/main/java/org/apache/nifi/flow/registry/FileSystemFlowRegistryClient.java#L434
It appears that the parent Parameter Context ID also needs to be checked. Instead of using the user-provided value, recommend looking up the Parameter Context and using the ID returned.

@exceptionfactory thanks for the pointers! I made a couple of improvements and seems to have resolved the CodeQL issues. I think most of them were actually somewhat false positives because the sanitize method was returning the input variable if null or blank, but that was causing CodeQL to think the unmodified user input was being potentially used later on.

@exceptionfactory I pushed a commit that addresses the comments you had, and also another commit that adds a delete end-point to be able to clean up assets that were uploaded and maybe never referenced, let me know of anything else

Is this new feature accessible only via the API and not in the GUI?

@markobean at the moment yes, if someone picks up a UI JIRA to add it then it may be in the UI later on


rules to include tasks in the reports

Currently, no reports include contributions made in form of tasks towards a goal.

[ ] derive a rule
[ ] code login and UI

Its included on mutually exclusive basis to activities in https://github.com/sundeep-anand/makegoalsdaily/pull/19


feat: introduce user-defined physical replication slots synchronization

This commit includes a new feature that synchronizes user-defined physical replication slots across replicas, enhancing self-healing and high availability.
The feature is enabled by default but can be disabled or include exclusion patterns by configuring the newly introduced stanza: replicationSlots.synchronizeReplicas.
Closes #2385

/test limit=local

/test limit=local

I don't think we intend to backport this feature, but I will let you decide @mnencia @gbartolini @mnencia

/test limit=local

I have a change proposal on the documentation side. Will work on that:

Rename the "Replication slots for High Availability" section into a broader "Replication slots" section
Leave the introductory part, then:

place here the "Replication slots for High Availability" section
add another section "User managed replication slots" (containing more or less the new content)

Move at the end of this new section the "Capping the WAL size retained for replication slots" part
Review the existing links

What do you think? @armru

I think we could address a more broad documentation refactor with a new PR

I am happy to give it a try

enhance the slot deleting and now have the following assumption

If both HA and user-defined slot are disabled:

User-defined slots will be deleted by reconcling loop
HA slots will be deleted from both primary/standby in reconciling loop too.

if only HA slots is disabled

HA slot will be deleted from primary/standby by reconcile loop
User-defined slots will NOT be deleted

if only user defined slot is been disabled

user defined slot will be deleted by runnable

/test limit=local

E2E tests in EDB fork https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/7799644093

/ok-to-merge all failures in E2E are unrelated


Unable to access boot2docker from Kinematic CLI shell

New user: Just installed newest Kinematic with Docker and boot2docker for the first time. When I open the shell by selecting "Docker CLI" the shell recognises the "docker" command but not the "boot2docker" command. It is used in several tutorials so I would expect it to be there.

@mmc41 thanks for the issue
Kitematic only currently includes docker-machine, similar to the boot2docker command. More info on that here.
Lookout for some better docs around this soon! Closing for now.


fix: remove mutually exclusive env variable, for #5950

The Issue

#5950

docker-compose up didn't respect COMPOSE_REMOVE_ORPHANS until v2.24.7:

https://github.com/docker/compose/pull/11462

But now it does, and we cannot use two env variables at the same time.
How This PR Solves The Issue
COMPOSE_IGNORE_ORPHANS and COMPOSE_REMOVE_ORPHANS are mutually exclusive:
https://github.com/docker/compose/blob/4efb89709ccb9f11ce0b6571a1c4674be37a42b7/cmd/compose/up.go#L92-L95
This did not appear in the tests because we use v2.24.5.
Manual Testing Instructions
$ ddev config global --required-docker-compose-version=v2.24.7
$ ddev start
Starting d10...
cannot combine COMPOSE_IGNORE_ORPHANS and --remove-orphans
Failed to start d10: composeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-d10 docker-compose -f /home/stas/code/ddev/d10/.ddev/.ddev-docker-compose-full.yaml up -d', action='[]', err='exit status 1', stdout='', stderr='cannot combine COMPOSE_IGNORE_ORPHANS and --remove-orphans'

Automated Testing Overview

Related Issue Link(s)
Release/Deployment Notes

Thanks for catching that! No idea why they'd make them mutually exclusive.


Needs Heroku-specific instructions

Doesn't work with Heroku. The accepted Stack Overflow answers don't seem to work.
Some Heroku-specific installation instructions would be beneficial.

Doesn't work with Heroku.

Yes, it does. I have five applications ranging from Rails 3.2 to Rails 4.2 all deployed to Heroku, and they all use font-awesome-rails.
Please read the contribution guidelines for typical debugging instructions before opening an issue here.

Thanks I am reading through the contribution guide.
I experienced the same failure with the 'foundation-icons-sass-rails' gem. So the issue (whatever it is) isn't specific to this gem.
The fonts are compiled into the public/assets folder, but the references that are rendered into the html produce 404 errors (and they are missing fingerprints).
I'll keep digging and open a more helpful issue. I'm using Rails 4.1.4 so it might be a new issue, or I might just be missing something basic about the asset pipeline.


feat(curriculum): responsive web design quiz

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56192

Hi @uditbaliyan. Congrats on your first pull request (PR)! 🎉
Thank you for your contribution to the page! 👍
We are happy to accept these changes and look forward to future contributions. 📝


Issue with dependencies

helm dependency update is failing with Sonarqube chart (which makes the helm chart fail):
[name@host helm-chart-sonarqube]$ helm dependency update --debug
Error: Chart.yaml file is missing
helm.go:88: [debug] Chart.yaml file is missing
helm.sh/helm/v3/pkg/chart/loader.LoadFiles
        helm.sh/helm/v3/pkg/chart/loader/load.go:158
helm.sh/helm/v3/pkg/chart/loader.LoadDir
        helm.sh/helm/v3/pkg/chart/loader/directory.go:119
helm.sh/helm/v3/pkg/downloader.(*Manager).loadChartDir
        helm.sh/helm/v3/pkg/downloader/manager.go:228
helm.sh/helm/v3/pkg/downloader.(*Manager).Update
        helm.sh/helm/v3/pkg/downloader/manager.go:154
main.newDependencyUpdateCmd.func1
        helm.sh/helm/v3/cmd/helm/dependency_update.go:74
github.com/spf13/cobra.(*Command).execute
        github.com/spf13/cobra@v1.1.3/command.go:852
github.com/spf13/cobra.(*Command).ExecuteC
        github.com/spf13/cobra@v1.1.3/command.go:960
github.com/spf13/cobra.(*Command).Execute
        github.com/spf13/cobra@v1.1.3/command.go:897
main.main
        helm.sh/helm/v3/cmd/helm/helm.go:87
runtime.main
        runtime/proc.go:225
runtime.goexit
        runtime/asm_amd64.s:1371

A recommendation is to package the dependencies and this repo into release with helm package and include in repo releases.
Then change the instructions on the this page here to use the packages that already exist rather than encouraging users to build off master branch:
https://docs.sonarqube.org/latest/setup/sonarqube-on-kubernetes/

While I agree that a packaged version of this chart is desirable and is in the works based on the comments by tobias in #11, however, your Helm output should have already told you whats wrong.
It cannot find the Chart.yaml cause you seem to brunning the helm command from the root of this repo, when the Chart and its templates are within the subfolder charts/sonarqube.
If you run the helm dependency update within that folder it should work without any problem.

I’ll try that later today, but I thought I tried both and the instructions seem off if that is true:
Per the instructions on the page linked above you have helm dependency update in the helm-chart-Sonarqube folder:
…
git clone https://github.com/SonarSource/helm-chart-sonarqube.git
cd helm-chart-sonarqube
helm dependency update
kubectl create namespace sonarqube
helm upgrade --install -f values.yaml -n sonarqube sonarqube ./
…

I’ll try that later today, but I thought I tried both and the instructions seem off if that is true:
Per the instructions on the page linked above you have helm dependency update in the helm-chart-Sonarqube folder:
…
git clone https://github.com/SonarSource/helm-chart-sonarqube.git
cd helm-chart-sonarqube
helm dependency update
kubectl create namespace sonarqube
helm upgrade --install -f values.yaml -n sonarqube sonarqube ./
…

Thanks it worked. You should change the instructions on https://docs.sonarqube.org/latest/setup/sonarqube-on-kubernetes to the correct cd command.

Hi bordenit,
Glad it worked :tada:!
I may be a contributor, however I am in no way associated with SonarSource, so all I can do is ping @tobias-trabelsi-sonarsource.
@ tobias: Sorry to ping you but there indeed seems to be an error within the SonarQube Docs in regards to how precisely this chart needs to be 'built'. Out of curiosity, what would be the appropriate way to report such an error?

Hi @k3rnelpan1c-dev and thank you very much for handling this issue :heart:
usually we try to consolidate all input in the community forum to have more in depth discussions and also to reach more people.
i have created a documentation ticket to track this in our jira SONAR-15199


The animation doesn't work on IE11

Below animation doesn't work on IE11.
grommet-ferret/src/index.html

IE11 does not support SVG animations with CSS. Currently we're animating the paths using CSS & stroke-dashoffset, a possible solution would be to use JavaScript to animate the SVG paths.

I believe we will live with the lack of animation in the loading page for IE11. Thanks for your issue


datePicker error on prod mode

Hi I use angular cli for the build with SystemJS.
I ran my app with angular rc5 and I'm using the feat-rc5 branch for the ng2-bootstrap.
My App uses
[DatePickerModule, ModalModule, DropdownModule] from ng2-bootstrap.
I have things working fine when I ran the application on dev mode. which use JIT compilation.
However I get the below error when I run the app on --prod mode. which use AOT compilation.
`Can't bind to 'activeDate' since it isn't a known property of 'datepicker-inner'.

If 'datepicker-inner' is an Angular component and it has 'activeDate' input, then verify that it is part of this module.
If 'datepicker-inner' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
("
<datepicker-inner [ERROR ->][activeDate]="activeDate"
(update)="onUpdate($event)"
[da"): a@1:22
Can't bind to 'datepickerMode' since it isn't a known property of 'datepicker-inner'.
If 'datepicker-inner' is an Angular component and it has 'datepickerMode' input, then verify that it is part of this module.
If 'datepicker-inner' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
("r [activeDate]="activeDate"
(update)="onUpdate($event)"
[ERROR ->][datepickerMode]="datepickerMode"
[initDate]="initDate"
["): a@3:22
Can't bind to 'initDate' since it isn't a known property of 'datepicker-inner'.
If 'datepicker-inner' is an Angular component and it has 'initDate' input, then verify that it is part of this module.
If 'datepicker-inner' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message.
("e)="onUpdate($event)"
[datepickerMode]="datepickerMode"
[ERROR ->][initDate]="initDate"
[minDate]="minDate"
[maxDate]="maxD"): a@4:22
Can't bind to 'minDate' since it isn't a known property of 'datepicker-inner'.``

This happened with the SystemJS build of angular cli, switching to WebPack resolved these.

@a5hik I have the same issue with ng2-bootstrap 1.1 and with webpack (i'm using angular2-webpack-starter).. can you please describe how did you solve it?


init: ported Get_Length_{String,Unicode} primitive functions

Since a number of level 1 Hestia libraries use string functions, we have to port its primitive ones into HestiaKERNEL library package. Hence, let's do this.
This patch ports Get_Length_{String,Unicode} primitive function into HestiaKERNEL library in init/ directory.

Merge first since production is unaffected.


[BUG] - Clicking on subspace's proposals redirect to home page

Briefly describe the bug.
Proposals links of a subspace doesn't work
How can we reproduce the bug?

Go to https://snapshot.org/#/2.snapspace.eth/
Click on any proposal of a subspace
It will redirect to snapshot.org

Also, in space settings page, on sub spaces section, right now we display this:

But the Actual condition is: You can add some space you like to track as subspace, but subspace won't show your space if its not mutual.
What is the expected behaviour?

[ ] Should redirect to the subspace's proposal
[ ] Change subspace message in space settings page

Can you attach screenshots?
No response
Can you provide your device specific details below?
No response

For me the flow still doesn't make sense. Adding a sub-space should always be mutual, i.e. trigger a notification for the selected space (i.e. popup in the settings) to confirm that the spaces belong to the same organization.
Otherwise why? Tracking another space shouldn't be under sub-space functionality 🤔  @ChaituVR wdyt?

@zzuziak Yeah, the user experience kinda sucks and doesn't make much sense. It feels like it was just thrown together without much care. I've created an issue and added it to the Settings milestone to-do list.


adding distributed-map-csv-iterator-tf workflow

Issue #, if available:
*Description of changes: Adding distributed-map-csv-iterator-tf workflow using Terraform
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

please at a valid example-workflow.json

Hey Ben, workflow is under statemachines/statemachine.asl.json
This is the same structure I normally use, let me know if you need a modified structure

the example-workflow.json is the metadata file about the workflow. Use this as a template: https://github.com/aws-samples/step-functions-workflows-collection/blob/main/_workflow_model/example-workflow.json

Done added example-workflow.json


Improve bar charts

The bar charts could be a little easier to read if 2023 would have to separate bars: one containing only the gas price and the subsidization (just like 2022), and another one containing same data with the own savings incorporated. It would better illustrate the effect of the savings, and especially how it would affect the own costs and the subsidization.
WDYT @mrsimpson ?

Sounds like an idea worth trying out

wdyt, @jschirrmacher – is that what you thought?

Very close. I would recommend to add "mit Einsparung" to the right most bar like the other for 2023 to make that more clear.

Since the saving is initially zero now, we could also hide the fourth bar in this case and only show it if the user has entered some savings amount.
One thing seems strange to me: if I enter a saving of e.g. 2000kWh, the subsidization is exactly the same in both 2023's bars. Shouldn't it be different due to the limitation of the price that normally would have been to pay? If I use less gas, there should be less money to pay and the subsidization should cover only the amount, that exceeds the 12 cent per kWh I think.

Wer hiding the col and renaming it: sounds good, feel free to – I have to sleep tonight.

WRT the calculation: the payment you receive in 2023 is a one time payment which is calculated "the difference in price of 0.12 and market price, applied to 80% of the consumption => if you reduce not more than 20%, the subsidization is always the same. Only if you reduce it more, it will be limited to what you effectively consume (less than 80%).
I do hope I got that right.

WRT the calculation: you're right, it is more difficult to calculate than I expected. But in fact, it sounds reasonable to have an incentive to reduce the consumption to more than 20%.
Maybe we could think about explaining that a little more in the app - at least for the interested.


Run VsTest task with console logger instead of trx

I'm trying to debug a test that is hanging on our build server.
I'd like to run the tests using the /logger:Console flag:

However, when the test is executed, it still has /logger:trx appended tothe end of the command line, which supercedes the Console option.
It would be great if the "Other console options" field was applied with logic to avoid duplicate flags like this.

All console output for failing tests is now uploaded to the server as test attachments. Does that help with your scenario? Thanks!

Yes, I think that would help. We are no longer having the issue so I can't confirm for sure.
Also, what do you mean by, "now"? As of which TFS version? Or, is it something we can patch manually?

Its live on VSTS. It will be in the next version of TFS.
Thanks!


retrieval of data from hashtable

Retrieve data from the hash table

the Data type Complex is returned. The contains() inbuilt method doesn't work as to complex objects are not compared with the instance variable values but some ids. Hence fixed it by comparing the instance real and imaginary vallues.


Error with treesitter plugin

Hi,
I'm getting the following error:
Line   65:
Parser dir ' /nix/store/vhlasav5vp7c83ck32gq6csf205i06gg-vim-pack-dir/pack/myNeovimPackages/start/nvim-treesitter/parser ' should be read/write (see README on how to configure an alternative install location) /nix/store/vhlasav5vp7c83ck32gq6csf205i06gg-vim-pack-dir/pack/myNeovimPackages/start/nvim-treesitter/parser '

My config is the following:
programs.nixvim = {
        enable = true;
        colorschemes.base16 = {
          enable = true;
          colorscheme = "gruvbox-dark-medium";
        };
        plugins.gitgutter = {
          enable = true;
        };
        plugins.lsp = {
          enable = true;
          servers.rnix-lsp.enable = true;
          servers.pyright.enable = true;
        };

        plugins.nix.enable = true;
        plugins.airline.enable = true;
        plugins.airline.powerline = true;
        plugins.treesitter.enable = true;
        plugins.nvim-autopairs.enable = true;
        plugins.undotree.enable = true;
        plugins.startify = {
          enable = true;
          useUnicode = true;
        };
        plugins.treesitter.ensureInstalled = [
          "bash"
          "cpp"
          "css"
          "go"
          "html"
          "java"
          "json"
          "latex"
          "lua"
          "make"
          "nix"
          "python"
          "regex"
          "rust"
          "yaml"
        ];
        options = {
          number = true; # Show line numbers
          shiftwidth = 2; # Tab width should be 2
          mouse = "a";
          tabstop = 2;
          expandtab = true;
          smarttab = true;
          autoindent = true;
          cindent = true;
          linebreak = true;
          hidden = true;
        };
        extraPlugins = with pkgs.vimPlugins; [
          vim-autoformat
        ];
        maps = {
          normal."<F3>" = ":Autoformat<CR>";
        };
      };

This happens on 88fb867da5d3b4785c8c52ea9d0ca2185b0dd51a (current master)

From your config, you don't seem to have enabled treesitter (but I've experienced this issue with it enabled as well!)
Could you try manually setting plugins.treesitter.nixGrammars to both true and false? One of them might work.
Don't get me wrong, this is 100% a bug and I'm trying to figure it out, but I think I've had success with that workaround and manually doing :TSUninstall all followed by :TSInstall all

This seems to be triggered by a change in treesitter. Upstream issue is here: https://github.com/NixOS/nixpkgs/issues/189838. For now, I'll see if I can implement the suggested workaround, but this seems like an upstream problem unfortunately

ah, yes, I see. Thanks for investigating further!

Temporarily fixed by implementing the suggested workaround if nixGrammars = true. This bypasses nvim-treesitter's parser management and installs all of the parsers directly from nix. Should fix a few issues regarding mismatched libc versions, too.


Add a real time character counter

Describe the project you want to add with tech stack
In this project when a paragraph or some text is entered, it shows the number of characters added and the number of characters that can be added so as not to exceed the maximum character limit.
Techstack:HTML,CSS,JS
(As a part of GSSOC'23)

Hey
I am interested in working on this project.
Why?
I have already made some projects using HTML, CSS, JS, Nodejs like weather app, calculator, group chatting app. I can use my previous knowledge to build this
How?

Create a basic HTML page
Add logic using JS
Will use CSS for beautifying view

Next Steps?
Please let me know could i work on this and guide me how can i start on this?
Thanks

@pranjay-poddar I'm a GSSOC'23 contributor,could you please assign this to me as a part of GSSOC'23

@Uma-95 , sorry I didn't realise you were contributor to project. Sorry, the project is yours:)

I would like to add this project, I have participated in open source program for the first time.
Please assign this task to me.

@pranjay-poddar  I'm a GSSOC'23 contributor, could you please assign this to me as a part of GSSOC'23


AVD scaling plan maximum number of session hosts

Not sure if I'm getting something wrong. Please clarify if I do.
The page says:

Off-peak hours: the time with the lowest possible usage. You can define the maximum number of VMs that can be active during this time.

After reading the initially quoted text, I would look for a Maximum number of hosts setting on the off-peak hours tab that would allow to configure "during off-peak hours, scale up according to capacity threshold % but not past  VMs".
I don't see an actual option in the portal to "define the maximum number of VMs that can be active during this time".  The feature appears to turn on or off additional VMs depending on Minimum percentage of hosts (%) and Capacity threshold (%). The only maximum seems to be the total number of hosts in the host pool and if load keeps above the configured capacity threshold percentage (1-100% of available sessions of currently running session hosts), VMs will be turned on until 100% of host pool VMs are running. For example, if there's a host pool with 10 session hosts, there no direct setting allowing to configure that during-off peak hours there should only ever be a maximum of 4 VMs running even if the capacity threshold is exceeded.
The off-peak hours tab only allows configuring Start time and Load Balancing Algorithm with Capacity threshold (%) being taken over from the ramp-down tab and greyed out.
The FAQ at https://docs.microsoft.com/en-us/azure/virtual-desktop/autoscale-faq#what-happens-if-the-host-pool-capacity-is-equal-to-the-capacity-threshold- states the following:

The autoscale feature only reacts when the host pool capacity is greater than or less than the capacity threshold. The feature won't do anything when the host pool capacity is the same as the capacity threshold.

In the above example, I would assume that setting the ramp-down minimum percentage of hosts to 40% and capacity threshold % to 100%, because of what the FAQ says, would result in no scaling operation taking place in off-peak hours as capacity cannot be greater than 100% and 4 VMs would effectively be a maximum. An actual setting would be better.

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

ID: e485eb1b-63d7-6fb4-a84a-367be0b7ae09
Version Independent ID: 6b38c7f4-b264-853c-ac69-9a7d9abc557e
Content: Azure Virtual Desktop session host autoscale preview
Content Source: articles/virtual-desktop/autoscale-scaling-plan.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

Initial quote in original post is now at https://docs.microsoft.com/en-us/azure/virtual-desktop/autoscale-scenarios#how-a-scaling-plan-works - portal unchanged; no setting for maximum number of VMs during off-peak hours.

Thanks for the feedback - I have made a change to how off-peak is defined and the change should be visible later today. Sorry for the confusion. We have also taken your feedback about wanting a max limit you can specify, but that is now with the product group.
#please-close