Automation of deployment of smart contract using the ansible

Description
As a developer i want to automate the deployment of smartcontracts on Besu using the signed transactions and deploy.js code.
Acceptance Criteria

Able to deploy the smartcontract on tls-disabled Besu org node using Ansible playbook

This issue is currently blocked by https://github.com/hyperledger-labs/blockchain-automation-framework/issues/999
Once the dependent issue is fixed, this issue can be picked up


CI: Disable sanitizer builds until we fix them

Added in #40994 but without recent rebase, so they don't pass currently.

Error in CI shows that some commits between this PR branch and current 3.2 branch introduced leak
Direct leak of 72 byte(s) in 1 object(s) allocated from:
    #0 0x7f62d43d4bc8 in malloc (/lib/x86_64-linux-gnu/libasan.so.5+0x10dbc8)
    #1 0xf0103a3 in Memory::alloc_static(unsigned long, bool) core/os/memory.cpp:82
    #2 0xf0102a2 in operator new(unsigned long, char const*) core/os/memory.cpp:42
    #3 0x590910c in MutexPosix::create_func_posix(bool) drivers/unix/mutex_posix.cpp:52
    #4 0xf014a48 in Mutex::create(bool) core/os/mutex.cpp:43
    #5 0x144cc0b in OS_X11::initialize(OS::VideoMode const&, int, int) platform/x11/os_x11.cpp:124
    #6 0x15ba827 in Main::setup2(unsigned long) main/main.cpp:1269
    #7 0x15b54b6 in Main::setup(char const*, int, char**, bool) main/main.cpp:1213
    #8 0x143ec3b in main platform/x11/godot_x11.cpp:49
    #9 0x7f62d2eb20b2 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x270b2)

Which as blame shows:
https://github.com/godotengine/godot/blame/8d36b7206cf11607395e68f060f003223ae62a95/platform/x11/os_x11.cpp#L124
probably was caused by this commit https://github.com/godotengine/godot/pull/42341
Probably deleting in destructor this mutex will fix issue CC @pouleyKetchoupp
https://github.com/godotengine/godot/blob/8d36b7206cf11607395e68f060f003223ae62a95/platform/x11/os_x11.cpp#L124

I'll merge for now to fix CI, but this can be reactivated (remove the if: false) once the regression is fixed.


Fixed some layer issues on the home.html navbar

Issue #160

Solves #160 . Good work noticing this. I was waiting for someone to point this out.


Fix duplicate results in callback

Summary
Fix duplicate results in callback
Details and comments
Fixes #182

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2044

Relevant Lines:
3600

💛  - Coveralls

FYI Integration tests in your fork fail with
qiskit_ibm_runtime.accounts.exceptions.CloudResourceNameResolutionError: 'Failed to resolve CRN value for the provided service name Quantum Services -Experimental-nn.'


wait_for_connection to localhost explicitly specified in the inventory and ansible_python_interpreter set to ansible_playbook_python fails with 'timed out waiting for ping module test: 'int' object has no attribute 'template'"'

Reproduction steps:
$ docker run --rm -it rockylinux:8.9.20231119 bash
$ yum install -y python38
$ pip3 install ansible==6.7.0
$ pip3 install mitogen=0.3.7
$ ANSIBLE_STRATEGY_PLUGINS=/usr/local/lib/python3.8/site-packages/ansible_mitogen/plugins/strategy ANSIBLE_STRATEGY=mitogen_linear ansible -i /tmp/hosts.yml localhost -m ansible.builtin.wait_for_connection -vvv

where
# $ cat /tmp/hosts.yml
all:
        hosts:
                localhost:
                        ansible_connection: local
                        ansible_python_interpreter: "{{ ansible_playbook_python }}"

Result:
localhost | FAILED! => {
    "changed": false,
    "elapsed": 600,
    "msg": "timed out waiting for ping module test: 'int' object has no attribute 'template'"
}

Suprisingly, with implicit inventory it works fine:
$ ANSIBLE_STRATEGY_PLUGINS=/usr/local/lib/python3.8/site-packages/ansible_mitogen/plugins/strategy ANSIBLE_STRATEGY=mitogen_linear ansible localhost -m ansible.builtin.wait_for_connection
[WARNING]: No inventory was parsed, only implicit localhost is available
localhost | SUCCESS => {
    "changed": false,
    "elapsed": 0
}

Which version of Ansible are you running?

$ ansible --version
ansible [core 2.13.13]
(...)
  python version = 3.8.17 (default, Aug 10 2023, 18:03:39) [GCC 8.5.0 20210514 (Red Hat 8.5.0-20)]
  jinja version = 3.1.3
  libyaml = True

Is your version of Ansible patched in any way?
No

Are you running with any custom modules, or module_utils loaded?
No

Have you tried the latest master version from Git?
Yes

Do you have some idea of what the underlying problem may be?
https://mitogen.networkgenomics.com/ansible_detailed.html#common-problems has
instructions to help figure out the likely cause and how to gather relevant
logs.
No

Mention your host and target OS and versions
Rocky 8.8

Mention your host and target Python versions
Python 3.8

If reporting a performance issue, mention the number of targets and a rough
description of your workload (lots of copies, lots of tiny file edits, etc.)

If reporting a crash or hang in Ansible, please rerun with -vvv and include
200 lines of output around the point of the error, along with a full copy of
any traceback or error text in the log. Beware "-vvv" may include secret
data! Edit as necessary before posting.

If reporting any kind of problem with Ansible, please include the Ansible
version along with output of "ansible-config dump --only-changed".
Empty

EDIT: 2024-10-07 @moreati Syntax highlighting

Now I've done a lot of digging around in ansible_mitogen/transport_config.py and ansible_mitogen/connection.py I can probably diagnose this. Prime suspect is
https://github.com/mitogen-hq/mitogen/blob/8e64459bbfbdc6f89cc4f275cb82142bea7bf776/ansible_mitogen/connection.py#L944
and https://github.com/mitogen-hq/mitogen/commit/833e2845e9d25fac94b5aab6fd0b613a3bd97007 probably contains the building blocks to fix it.

Still occurs with master,
➜  mitogen git:(master) ✗ ANSIBLE_STRATEGY=mitogen_linear ansible -i issue1079_inv.yml localhost -m ansible.builtin.wait_for_connection -a "timeout=10"
localhost | FAILED! => {
    "changed": false,
    "elapsed": 10,
    "msg": "timed out waiting for ping module test: 'int' object has no attribute 'template'"
}

same -vvv

➜  mitogen git:(master) ✗ ANSIBLE_STRATEGY=mitogen_linear ansible -i issue1079_inv.yml localhost -m ansible.builtin.wait_for_connection -a "timeout=10" -vvv
ansible [core 2.17.5]
  config file = /Users/alex/src/mitogen/ansible.cfg
  configured module search path = ['/Users/alex/src/mitogen/tests/ansible/lib/modules']
  ansible python module location = /Users/alex/.local/share/uv/tools/ansible-core/lib/python3.13/site-packages/ansible
  ansible collection location = /Users/alex/.ansible/collections:/usr/share/ansible/collections
  executable location = /Users/alex/.local/bin/ansible
  python version = 3.13.0 (main, Oct 16 2024, 08:05:40) [Clang 18.1.8 ] (/Users/alex/.local/share/uv/tools/ansible-core/bin/python)
  jinja version = 3.1.4
  libyaml = True
Using /Users/alex/src/mitogen/ansible.cfg as config file
host_list declined parsing /Users/alex/src/mitogen/issue1079_inv.yml as it did not pass its verify_file() method
script declined parsing /Users/alex/src/mitogen/issue1079_inv.yml as it did not pass its verify_file() method
Parsed /Users/alex/src/mitogen/issue1079_inv.yml inventory source with yaml plugin
Skipping callback 'default', as we already have a stdout callback.
Skipping callback 'minimal', as we already have a stdout callback.
Skipping callback 'oneline', as we already have a stdout callback.
[top  88106] 16:20:03.219759 D ansible_mitogen.process: inherited open file limits: soft=1048575 hard=(infinity)
[top  88106] 16:20:03.219842 D ansible_mitogen.process: max open files already set to hard limit: 524288
[mux  88108] 16:20:03.224877 D mitogen.service: Pool(81a0, size=32, th='MainThread'): initialized
[mux  88108] 16:20:03.225766 D ansible_mitogen.process: Service pool configured: size=32
wait_for_connection: attempting ping module test
...
wait_for_connection: attempting ping module test
[task 88109] 16:20:13.271395 D ansible_mitogen.mixins: _remove_tmp_path(None)
[task 88109] 16:20:13.273145 D ansible_mitogen.mixins: _remove_tmp_path(None)
localhost | FAILED! => {
    "changed": false,
    "elapsed": 10,
    "msg": "timed out waiting for ping module test: 'int' object has no attribute 'template'"
}
[mux  88108] 16:20:13.280671 D mitogen: Waker(fd=36/38): disconnecting
[mux  88108] 16:20:13.282092 D mitogen.service: thread mitogen.Pool.81a0.0 exiting gracefully
[mux  88108] 16:20:13.283423 D mitogen.service: thread mitogen.Pool.81a0.1 exiting gracefully
[mux  88108] 16:20:13.284447 D mitogen.service: thread mitogen.Pool.81a0.5 exiting gracefully
[mux  88108] 16:20:13.285470 D mitogen: Router(Broker(ee40)): stats: 0 module requests in 0 ms, 0 sent (0 ms minify time), 0 negative responses. Sent 0.0 kb total, 0.0 kb avg.
[mux  88108] 16:20:13.286389 D mitogen.service: thread mitogen.Pool.81a0.6 exiting gracefully
...
[mux  88108] 16:20:13.304953 D mitogen.service: thread mitogen.Pool.81a0.30 exiting gracefully
[mux  88108] 16:20:13.305462 D mitogen.service: thread mitogen.Pool.81a0.31 exiting gracefully
[mux  88108] 16:20:13.306406 D mitogen.service: FileService().on_shutdown()
[top  88106] 16:20:13.308218 D ansible_mitogen.process: multiplexer 0 PID 88108 exited with return code 0

doesn't occur with vanilla Ansible
➜  mitogen git:(master) ✗ ANSIBLE_STRATEGY=linear ansible -i issue1079_inv.yml localhost -m ansible.builtin.wait_for_connection -a "timeout=10"
[WARNING]: Reset is not implemented for this connection
localhost | SUCCESS => {
    "changed": false,
    "elapsed": 0
}

all:
  hosts:
    localhost:
      ansible_connection: local
      ansible_python_interpreter: "{{ ansible_playbook_python }}"

Untemplated ansible_interpreter_python makes the problem go away
➜  mitogen git:(master) ✗ ANSIBLE_STRATEGY=mitogen_linear ansible -i issue1079_inv.yml localhost -m ansible.builtin.wait_for_connection -a "timeout=20"
localhost | SUCCESS => {
    "changed": false,
    "elapsed": 0
}

I need to ponder this some more. The templar made available in '_mitogen.smuggled.reset_connection' is only available during a meta: reset_connection task.


does this project supports the OCRs and ERFilter modules?

Hi,
This opencvsharp version is great.
In my case I want to use the OCR and text detections features:
http://docs.opencv.org/3.1.0/d4/d61/group__text.html#gsc.tab=0
is it supported today with opencvsharp?

"opencv_contrib" features are not supported today ☹
I'd like to support them someday...
see https://github.com/shimat/opencvsharp/issues/146

I ask again : OpenCVSharp3-AnyCPU (v<IP_ADDRESS>81129)
does this project supports the OCRs and ERFilter modules
use the OCR and scene text detections features like:
class cv::text::ERFilter
class cv::text::TextDetector
and so on..


Fix zero opacity case

This fix checks a TiledImage for the case that opacity === 0 and where _preload is false and aborts the tile updates if so, which stops tiles from being loaded for a fully transparent layer. Will fix the undesirable behavior described in https://github.com/openseadragon/openseadragon/issues/2461

This does not seem to address the issue in #2461 . Rather, the opacity condition should be in tiledimage.js:294 update function.

@jetic83 Try the latest. It is now working correctly for me in my tests.
I moved the logic for checking for the zero opacity case (in conjunction with the preload option) into getDrawArea which makes it work well. I believe doing it this way will still allow any relevant events to fire about the springs updating, etc., even while the image is zero opacity (which is desirable).

Great, glad you could confirm it works for you. The minor change in the demo page was just due to some tweaks I made to test this (before I realized I didn't need to change it after all).
The most recent commit also lets the new WebGLDrawer skip the entire drawing pipeline for zero-opacity images, which should help performance in the case where potentially hundreds or more TiledImages are loaded but are fully transparent. Finally, the CanvasDrawer and HTMLDrawer had logic related to the _preload status of the TiledImage which isn't needed and would be counterproductive from a performance status. Those have been fixed as well.
Thank you for pointing this out so we can get it fixed!


✨[Feature Request]:  add popup thanks for the feed back when feed back form is submitted

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
popup thanks for the feed back when feed back form is submitted
Use Case
thanking message
Benefits
popup thanks for the feed back when feed back form is submitted
Add ScreenShots
at present
https://github.com/user-attachments/assets/249935db-c428-4131-b5e4-7aaa0b7956d5
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@varshith257 please assign me this issue


More precise timeline for release

Hi!  I really enjoyed your paper, and in particular would really like to play around with your implementation of the contrastive loss for place recognition; would you be able to give any more information on the release schedule for your code than beginning of 2023?
Thanks,
Josh

Hey Josh,
thanks for you interest, most likely soon after the ICRA deadline, so first week of march.

Hi ! I am also interested in this paper. Will it be open source recently in march?
thanks a lot

Currently working on it.


use openshift_context instead of internal function in util package

as requested by @bn222 I create a new PR to always use the ocp context
https://github.com/k8snetworkplumbingwg/sriov-network-operator/pull/365#discussion_r1105800681

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 27 (7.41%)  changed or added relevant lines in 3 files are covered.
15 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 25.12%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

controllers/sriovnetworkpoolconfig_controller.go
1
2
50.0%

pkg/utils/openshift_context.go
0
24
0.0%

Files with Coverage Reduction
New Missed Lines
%

controllers/sriovnetworkpoolconfig_controller.go
1
54.55%

controllers/sriovibnetwork_controller.go
2
67.33%

pkg/utils/cluster.go
12
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
1891

Relevant Lines:
7528

💛  - Coveralls

/lgtm

/lgtm


O(N^2) assignment of points to cells

I noticed that part after computing the polygons the code assigns all points to these polygons.  This results in an O(N^2) implementation which is untenable (memory and speed) when there are a lot of points (I have 200000).  Assigning most points to cells by computing the matrix of distances is unnecessary.  Most points (90-95%) are unambiguously assigned by Voronoi from scipy.spatial.  It would be fantastic if the library (which is definitely filling a much needed niche) could be sped up by using these unambiguous assignments to cells that don't have edges that extend to infinity.
Thanks!

Good point. Haven't looked into the code for a while, but I think your strategy should work. Currently I don't have the time to implement it, but I have it on my list. However, please feel free to suggest an update via PR, this would be much appreciated!

Had the time to actually work on this and I got a version that is much faster and memory efficient on branch issue_6. Will have to do some more testing but I think a new version of geovoronoi with this update can be expected in the next few weeks.

Had the time to actually work on this and I got a version that is much faster and memory efficient on branch issue_6. Will have to do some more testing but I think a new version of geovoronoi with this update can be expected in the next few weeks.

A new, faster algorithm was implemented in version 0.3.0, which is now available on PyPI.


Start/Stop Recording Button modifying Play mode should be optional

What happened?
Frame Recorder starts and stops play mode when you hit the "Start Recording" and "Stop Recording" button. This is a nice feature, but there should be an option to not interfere with play mode. I'd like to play a scene, move to a  location, start and stop recording, move to another location, start and stop recording etc without having the entire scene stopped each time I hit "Stop Recording".
Is it reproduceable?
Yes, all the time.
Released package version / tag?
Master
Unity version, operating system, target platform (standalone windows, mac, iOS, PS4...)?
2018.1.0b6, Windows 10, Unity Editor

the intent is that if already in play mode when start / stop recording are pressed, game mode should NOT be altered.
If that is not the case, then it's a bug. we will investigate...


How to use SetParameters.xml to change connection strings in app.config for the project was created with asp.net core 2.1 (.NET Framework)

I am working on the CI/CD pipeline for a quite big project that was created 3 years ago and it is a asp.net core 2.1 (.NET Framework) project. This project store the connection strings in app.config file for EF 6. I was trying to create Web Deploy Package for this project, but VS cannot detect the connection string in app.config, I have tried to create custom Parameters.xml file
<parameter name="OptagEntities-app.config Connection String" description="doesn't matter" defaultvalue="__OptagEntities__" tags="">
      <parameterentry kind="XmlFile" scope="\\app.config$" match="/configuration/connectionStrings/add[@name='OptagEntities']/@connectionString">
</parameterentry>

I have tried to deploy this project to local IIS and I can see that connection string is not updated from the value in SetParameters.xml file. We have on premise TFS server and same flow works for webform project. The different is Web config and this project is using app.config, because asp.net core .NET framework only create web.config when publish. I know I can use appsetting.json to transform the different the ENV, but it require code changes and worsting thing is that this project is referencing another project via nuget that also using a connection string in app.config. Any work around for this?
Software versions
Check the .NET target framework(s) being used, and include the version number(s).
asp.net core 2.1 (.NET Framework)

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support group like Stack Overflow with the asp.net-core tag.


New screenshots

I don't have time to support the application, so the edits are unfortunately not relevant.


add todo notes and remove abs which is not needed

Pull Request Test Coverage Report for Build 1555
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 89.005%

Files with Coverage Reduction
New Missed Lines
%

int/int.mbt
3
8.33%

Totals

Change from base Build 1554:
-0.1%

Covered Lines:
2558

Relevant Lines:
2874

💛  - Coveralls


Add log entries for the admin panel quick links.

Fixes #348.

@PetterS ping me for a review once the tests pass.

Looks like things have changed between Django versions unfortunately. Let me see if I can fix it.

@clintonb Done.


User input form

We need the user to be able to enter a list of stocks which they own and the quantities held.

User wants to be able to value his portfolio of stocks:
(a) He/she should be able to enter a transaction for a given stock (ticker), which will be either a purchase or a sale
(b) He/she should be able to record the date of the transaction.
(c)  He/she should be able to record the quantity involved in the transaction
(d) The user should be able to store their transactions under a portfolio name.
(e) The user should be able to store multiple portfolios (NB nice to have)
(e) The user should be able to edit the portfolio including renaming it (NB nice to have).
Notes:

Use bootstrap components for UI elements for smoother look and responsiveness out of the box.
Consider using a Calendar Control for the entry of dates as this will avoid any conversion issues.


s,gopkg.in/yaml.v2,sigs.k8s.io/yaml for json-tagged structs

We recently discovered that gopkg.in/yaml.v2 doesn't work properly in
cases where the struct being unmarshalled to has only json tags. An
important example is unmarshalling install-config. The reason can be
found in this blog post [1].
Swap to sigs.k8s.io/yaml in the couple of places where this applies.
(Notably, we're leaving alone places where the target struct has yaml
tags, as the interface between the two libraries is not fully
compatible.)
[1] https://web.archive.org/web/20190603050330/http://ghodss.com/2014/the-right-way-to-handle-yaml-in-golang/
HIVE-2205

/assign @lleshchi

same interface

Clearly untrue.
/hold

/lgtm

Hello tide? 628 is closed since like Tuesday.
/hold

/unhold


Failed to resolve

Failed to resolve: com.afollestad:material-dialogs:0.6.0
I Nneed this version

You're not using the correct dependency format and that version doesn't exist.
https://bintray.com/drummer-aidan/maven/material-dialogs%3Acore


Quickstart: Add Outscale

Hello rancher team,
This PR adds quickstart information for getting started on the Outscale.
This is a part of the PR on rancher/quickstart

The proposed content looks fine to me, but we'll need to wait for the dependent PR(s) to go in first.

Hi @btat,
The other PR is now merged 👍


Unnecessary transmute

Instead of u32::from_be(transmute(array)) you could use u32::from_be_bytes(array).

This has already been fixed on master (commit 8d120a24). Thanks for telling me anyway.


Leather Wallet lost the display domain or subdomain name when they are in punycode

When you want to select an account, a list of your accounts is displayed.  If an account does not have a domain or subdomain associated to it displays Account 6, otherwise displays the domain or subdomain name.
Since some weeks ago, the accounts that use punycodes are not displayed at all. Not even the unicode code.
The same happens to the Dropdown list box of the UI.
It should display the account name.

Thanks for reporting, I am seeing reports that recent domain registrations are not displayed at all too.

@paradigma-cl which version of the wallet are you using?
Would you mind sharing some screenshots? feel free to blurr private details.
Do you see any signs in the console that there are issues, reaching relevant API's?

were the affected domains registered or transferred recently?

@paradigma-cl which version of the wallet are you using?
Would you mind sharing some screenshots? feel free to blurr private details.
Do you see any signs in the console that there are issues, reaching relevant API's?

Above is the Firefox wallet.  It does not display the account name, as it is punycode.
Below is the Chrome wallet

No changes to the names have been made.

I found that something happened with the name, and the stacks address lost its association to the name.  I traced it and could not find a reference anywhere.  Something similar happened jan 2023.  Several accounts lost their associations.  Mainly I think they are subdomain names.
In my case, I saw that the name dissappeared while looking at the Wallet.  There is no problem with the display of the punycode name


WorkItem Link type pull down broken style in Firefox

The work item link drop down has been fixed.


WSGI middleware - Display formatted tracebacks

Hi.
Sentry is very useful to catch production errors. Though at my company, we usually do not use it in the development phase, as we generate a lot of exceptions and prefer just see them on the fly instead of generating few valuable exception info in mass.
We would love raven to be able to format and display tracebacks like cgitb does. Maybe the piece of code that sentry server uses to format tracebacks is reusable ?
What do you think?

It’s not reusable. This is best left up to other libraries.


[SPARK-44916][DOCS][TESTS] Document Spark Driver Live Log UI

What changes were proposed in this pull request?
This PR aims to document Spark Driver Live Log UI. In addition, this PR fixed a test case to clean up the test directory properly.
Why are the changes needed?
To help a user to use this feature easily, especially in K8s environment.
1. Spark Configuration page

2. Running Spark on Kubernetes page

Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this PR, @viirya ?

Thank you, @viirya !

Merged to master for Apache Spark 4.


Render Performance Tests

Startup time
How long until the first frame
Min max frame latency
Average frame time
Uniforms
Buffer changes
draw() calls

Feel free to break these issues out into pieces as that makes sense.  We've got a bunch of different tests in the list.

This might be a duplicate or it may just be too broad and ongoing.


Keyboard mapping seems to be broken for Mac and Linux keyboards

How to reproduce: Start the remote docker container, try to type in '@'. Nothing happens.

We may just have to set the server-layout setting in the Guacamole connection options: https://guacamole.apache.org/doc/gug/configuring-guacamole.html

Setting server-layout was the solution. Will close this issue.


Extracting both summary and description from path function docstring in PEP 257 compliant way

First check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.
[X] After submitting this, I commit to:

Read open issues with questions until I find 2 issues where I can help someone and add a comment to help there.
Or, I already hit the "watch" button in this repository to receive notifications and I commit to help at least 2 people that ask questions in the future.
Implement a Pull Request for a confirmed bug.

Recent library adopter and first-time issue opener here. :) Thanks to the team for the great library!
The library has a helpful feature of extracting OpenAPI path documentation from the docstring of the handler function. However, I find the feature counter-intuitive from PEP 257 point of view. The best practices defined by the PEP defines that a multi-line docstring consists of a summary line and an elaborate description. I believe that, instead of using the whole docstring as description, the summary line and description should be mapped to OpenAPI summary and description, respectively.
I'm willing to contribute the PR, but would like to clear my idea with the maintainers first.
Example
Here's a self-contained minimal, reproducible, example with my use case:
from fastapi import FastAPI

app = FastAPI()

@app.get("/")
def hello():
    """Respond with the friendly greeting

    This is a long long description of the API call.
    """
    return { "greeting": "Hello, world!" }

Description

Open the browser and call the endpoint /openapi.json.
It returns a the following JSON doument

{
  "openapi": "3.0.2",
  "info": {
    "title": "FastAPI",
    "version": "0.1.0"
  },
  "paths": {
    "/": {
      "get": {
        "summary": "Hello",
        "description": "Respond with the friendly greeting\n\nThis is a long long description of the API call.",
        "operationId": "hello__get",
        "responses": {
          "200": {
            "description": "Successful Response",
            "content": {
              "application/json": {
                "schema": {}
              }
            }
          }
        }
      }
    }
  }
}

I would instead expect the summary line and the description in the docstring to be mapped to the summary and the description of the OpenAPI document, respectively:

{
  "openapi": "3.0.2",
  "info": {
    "title": "FastAPI",
    "version": "0.1.0"
  },
  "paths": {
    "/": {
      "get": {
        "summary": "Respond with the friendly greeting",
        "description": "This is a long long description of the API call.",
        "operationId": "hello__get",
        "responses": {
          "200": {
            "description": "Successful Response",
            "content": {
              "application/json": {
                "schema": {}
              }
            }
          }
        }
      }
    }
  }
}

The solution you would like
I realize that this is a backward-incompatible change. Nevertheless, I think it's justified for compliance with docstring best practices and a more natural way to write documentation. I suggest the following to mitigate the impact:

If the route decorator has both summary and description set, ignore the docstring
If the route decorator has summary set but description not set, interpret the whole docstring as a description (compatible with the current behavior)
If the route decorator has neither summary nor description set, populate both summary and description from the docstring as described above

As I said above, I'll be happy to write a PR for this, but I would like to get feedback on the idea first.
Describe alternatives you've considered
What I described can currently be done with decorator arguments, so this feature is just for enhancing productivity and, arguably, the cleanliness of documentation. The alternative is to document the paths like so far.
Environment

OS: Linux
FastAPI Version: 0.58.0
Python version: 3.8.2

I would like to add that it would be even better if Python docstrings for parameters were also extracted from the description, in Sphinx notation:
def foobar(id: int):
    """
    Get a single Item

    Returns the given item from it's id.

    :param id: The item id, e.g. "2019asa3e8938"
    """

I would like to add that it would be even better if Python docstrings for parameters were also extracted from the description, in Sphinx notation:
def foobar(id: int):
    """
    Get a single Item

    Returns the given item from its id.

    :param id: The item id, e.g. "2019asa3e8938"
    """

Nice idea! I submitted a PR for the original issue but this is a good follow-up.

I think the main idea about the summary and description makes sense, but it will break all my documentation, so I'm skeptical about it. About having the parameters as well, I don't see the real usefulness of having it, considering that we already a nice way to do it and it adds an unnecessary complexity.

Does the documentation have to be mutually exclusive? The usefulness would be to be able to auto-generate user documentation using Sphinx that embeds the API documentation (for example using readthedocs.org, which allows cross-referencing and all that), and it's the de-facto standard way to do Python API documentation.

From what it appears to me, the guiding principle of FastAPI design is that it's possible to do things in multiple ways: One is the DWIM way which should be and work most of the time. Example: If I just say that a route handler accepts a parameter X, the library figures out from the context if X is a path, query or body parameter, and is correct most of the time. But there is a way to fine-tune everything by explicit configuration. So I can have a plain number from the request body if I want to.
I don't see why this couldn't be applied to documentation. Sphinx autodoc is widely used in the Python community, and allowing developers write API documentation just as they are used to documenting all their functions certainly qualifies as DWIM.
Breaking existing documentation is a valid concern. As a random first-time-contributor-hopeful I can't really decide if it's a blocker, and to which extent the library guarantees backward compatibility.

Thanks for the interest and discussion everyone!
But I see it would be cumbersome to support, being a breaking change, forcing everyone to change to the new behavior, it wouldn't be easy to opt-out, and it would also conflict with the auto-generated summary based on function names.
I also prefer the keyword argument summary that can be found with autocompletion. :shrug:
So I would prefer to keep it as it is. But thanks for the discussion! :coffee:

About the format, OpenAPI supports Markdown, and the docs (Swagger UI) render Markdown. You are free to use reStructuredText if you prefer that, but it won't be rendered by Swagger UI.
If you want to have both, you can add a \f after the main part to add additional info, it won't be added to OpenAPI, and you can extract it for Sphinx if you want, you can check more details here: https://github.com/tiangolo/fastapi/pull/556

Hi @tiangolo,
Thanks for taking the time to respond. Fair enough, I'll close this issue as won't do.


Add deepcopy to DimCoord

The points and bounds properties of the DimCoord class are intended to be read-only. However, when a DimCoord object is deepcopied, the underlying numpy arrays have their 'writeable' flag set to True by np.ndarray.__deepcopy__(...).
This fix adds an implementation of __deepcopy__(...) to DimCoords which resets these flags to False. It also modifies some unit tests which were failing due to the fix.

Nice addition @djkirkham. :+1:
I'm not the biggest fan of our use of readonly arrays, but I do agree that this change keeps the consistently with what is already implemented.
Would you mind putting an entry in the what's new. Something susinct, but which highlights the fact that (as some of the tests did), it may have been possible to change DimCoord values in-place, but not any more.
Other than that :+1:

This would constitute a major API change that could break some user code, so I guess it should go into 2.0? (I don't know if there will be a 1.10 anyway).

This would constitute a major API change that could break some user code

I certainly agree about the user code breakage possibility. I could fall either side of the fence as to whether this constitutes an API change. Most bug fixes will break some form of API:

https://xkcd.com/1172/

Well, I'm interested in breakage primarily rather than the technicalities of what constitutes an API change.

Agreed. I was trying not to be biased based on the comment I made earlier (I'm not the biggest fan of our use of readonly arrays).
In prevention of in-place modification I believe Iris goes too far into defensive programming. The reason for its existence is to ensure that a DimCoordinate is and remains a monotonic entity, but there is nothing that Iris can do to do this completely, and so all that is left is for a line to be drawn which balances giving users the ability to do what they want and helping users to avoid making mistakes with their metadata.

One alternative is to have the underlying array be a subclass of, or wrapper around, an ndarray and ensuring monotonicity there. That would allow in-place modification while keeping monotonicity. That might be over-complicating things though.

One alternative is to have the underlying array be a subclass of, or wrapper around, an ndarray and ensuring monotonicity there. That would allow in-place modification while keeping monotonicity. That might be over-complicating things though.

I think that probably is going too far. As long as this change is properly documented as potentially incompatible with previous versions it should be fine.

Thanks @djkirkham. And thanks for reviewing @ajdawson.
Congratulations on your first deep code-change in Iris @djkirkham!


feat: Support Python 3.12

Waiting for

~https://github.com/pymssql/pymssql/pull/856 to be shipped~. Shipped in 2.2.10.
~A final release of aiohttp 3.9.0, currently in beta.~. Using AIOHTTP_NO_EXTENSIONS=1. Should be ok since aiohttp is only required by the meltano-cloud CLI in the cloud branch.

I think there's a false negative test for the Azure state backend. Should raise https://github.com/Azure/azure-sdk-for-python/issues/32875.


Use ACE arsenal actions instead of UI button

When merged this pull request will:

Title.
Fixed indentations in stringtable.xml

Something weird happens when you press the text in the arsenal - needs investigating, so don't merge quite yet.

You can go ahead and merge it if you want, it's a bug in ACE.

@mjc4wilton What's the status with this PR? Does something need to be done on my end?


Make internal gRPC client less strict when decoding

SUMMARY
Current, the client gRPC protocol is not versioned (#1861) and for example, adding a new field makes the client fail with something like:
grpc: failed to unmarshal the received message cbor: found unknown field at map element index 11

Since we are only using the client gRPC protocol between trusted parties, we could make the CBOR decoding less strict and ignore unknown fields.
ISSUE TYPE

Feature Idea

Note that this should ONLY apply to the internal gRPC client (e.g. for the oasis-node client gRPC protocol), not to any other uses of gRPC (like storage).

I am against this as it is not possible to rule out unexpected behavior in the future due to default values in new fields that get added, and the same people that will complain about having to keep the client and core build in-sync will be the loudest to complain to us when something bad does happen.


Fix misc http issues

remove null key headers from response.
no post request if the content length is zero.

Closing due to inactivity


Add bookmarks to sidebar

Bookmarks for the current project should be listed in the sidebar, along with files.  This would make this plugin much more valuable for those who want to see all their bookmarks for a project in one place!

+1
Maybe you could have it so that the bookmarks show as a dropdown from the filename?


Fix normalization during install

Escape command line requirements as necessary

Fixes #2998

Just tested and this correctly installs the pdfminer.six dependency within my lockfile 🎉

Holy moly! So much headache about quoting and unquoting, and all because delegator uses shell=True unconditionally and is buggy when you pass in a list of arguments
https://github.com/pypa/pipenv/pull/3007/commits/12e80edede0e61e2ea19c132570862d25e4368fe#diff-ec74dda0078733c5b6755671cacf1de5R1424
https://github.com/kennethreitz/delegator.py/issues/58
This is the culprit IMHO, after all it's the library that should handle command execution. Instead arbitrary argument quoting snippets are found here and there all over pipenv.

Not entirely accurate. We do proper splitting in various other libraries and are still forced to contend with this.

@techalchemy why do you need to do extra splitting? I mean you need to do some splitting to handle requirements.txt lines, but other than that passing commands around as lists of verbatim (without any quoting) arguments should be enough, no?

What if a path has a space in it? What if that happens on windows? What if both of those are true and the user is using nave path separators (that is, \)? If we figure out a target path for example it still needs to be quoted even if it is split. And if we are parsing user input as we do for --python we also need to be able to perform the splitting

The short answer is that normally command line quoting should be handled by standard libraries.

What if a path has a space in it?

subprocess.check_output([mycmd, 'hello world']) will pass hello world as a single string to mycmd command, that is if it were a python script it would have sys.argv == ['echo', 'hello world']. It achieves that by using one of the exec-family of functions.

What if that happens on windows?

On windows, CreateProcess takes a string rather than a list, and subprocess uses subprocess.list2cmdline to internally properly quote a string that is passed to a subprocess.

What if both of those are true and the user is using nave path separators (that is, )?

It depends on what do you want to do with it. If you want to pass it forward, then again subprocess does all the quoting as necessary, so something like
subprocess.check_output([cmd, ..., sys.argv[1], ...])

will just work™.
If you want to handle it within pipenv, w.r.t. native/non-native path separators, it's a bit of a mess, but something along the lines of of path_components = os.path.normpath(user_path).split(os.path.sep) and then os.path.join(path_components) down the line should do the trick.

And if we are parsing user input as we do for --python

Same as above.
But like you say, apparently the problems pipenv is experiencing with quoting are related to delegator.py, because it is using shell=True kwarg for all subprocess invocations which is not only slow, but also leads to problems:

on POSIX with shell=True arguments passed as a list are interpreted differently (see this python bug and this delegator.py bug
on Windows, it is very tricky to quote shell variables reliably (the Internet is full of advice to do smth like cmd /c prog ^%PATH^%, but it is not 100% bulletproof as you can have an env-variable named PATH^ and it seems to get expanded before ^-escaping takes place)


Error Applying Less Grind

Sorry for not giving context, just trying to help out, but don't want to dig into that much.

Thanks for letting me know... which version are you using and do you have any other mods installed?
Those errors seem to indicate that this may be a conflict with another mod.

I'm using the most recent version of all mods and this is the list.

Mod Manager
Add Sewing Kit
Big Backpack - Donovan
Better Collectors - Donovan
Better Dyes - Donovan
Craftable Parts - Donovan
More Books - Donovan
More Perks - Donovan
Stop Fuel Waste
Faster Walk and Run Speed
HUD Plus 1.8.0

My best guess would be a conflict with the stop fuel waste mod.
On Tue, Aug 8, 2023, 6:20 PM Donovan C. Young @.***>
wrote:

Thanks for letting me know... which version are you using and do you have
any other mods installed?
Those errors seem to indicate that this may be a conflict with another mod.
—
Reply to this email directly, view it on GitHub
https://github.com/DonovanMods/donovan-7d2d-modlets/issues/57#issuecomment-1670437955,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKM7ZO5DAQSC2Y3OWTIY42TXULCSRANCNFSM6AAAAAA3EV43FI
.
You are receiving this because you authored the thread.Message ID:
@.***>

I also use Stop Fuel Waste with my modlets, so I don't think that is the conflict.
The error is coming from donovan-lessgrind and I suspect it may be conflicting with donovan-bettercollectors (which loads first). If so, it's safe to ignore that error because donovan-bettercollectors is already doing what you want, but I'll investigate that further.
However, either way, I highly recommend you download and install the latest release of my mods directly from here instead of from 7daystodiemods.com because that site lags a bit behind on releases.
Here's a direct link to the latest releases

Thanks for the information,
I love 7daystodiemods.com as it seems to have better quality on it. I
really wish there was a search feature with filters though to allow you to
search by most liked and game version.
Have a good one,
Garrett.
On Tue, Aug 8, 2023 at 11:29 PM Donovan C. Young @.***>
wrote:

I also use Stop Fuel Waste with my modlets, so I don't think that is the
conflict.
The error is coming from donovan-lessgrind and I suspect it may be
conflicting with donovan-bettercollectors (which loads first). If so,
it's safe to ignore that error because donovan-bettercollectors is
already doing what you want, but I'll investigate that further.
However, either way, I highly recommend you download and install the
latest release of my mods directly from here instead of from
7daystodiemods.com because that site lags a bit behind on releases.
Here's a direct link to the latest releases
https://github.com/DonovanMods/donovan-7d2d-modlets/releases
—
Reply to this email directly, view it on GitHub
https://github.com/DonovanMods/donovan-7d2d-modlets/issues/57#issuecomment-1670633415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKM7ZOZIBJJD7ZW77M4YZHDXUMGZHANCNFSM6AAAAAA3EV43FI
.
You are receiving this because you authored the thread.Message ID:
@.***>


pix2pix

hey all just checking if pix2pix is supposed to be working as I'm getting some odd errors
ValueError: Incorrect configuration settings! The config of pipeline.unet: FrozenDict([('sample_size', 64), ('in_channels', 4), ('out_channels', 4), ('center_input_sample', False), ('flip_sin_to_cos', True), ('freq_shift', 0), ('down_block_types', ['CrossAttnDownBlock2D', 'CrossAttnDownBlock2D', 'CrossAttnDownBlock2D', 'DownBlock2D']), ('mid_block_type', 'UNetMidBlock2DCrossAttn'), ('up_block_types', ['UpBlock2D', 'CrossAttnUpBlock2D', 'CrossAttnUpBlock2D', 'CrossAttnUpBlock2D']), ('only_cross_attention', False), ('block_out_channels', [320, 640, 1280, 1280]), ('layers_per_block', 2), ('downsample_padding', 1), ('mid_block_scale_factor', 1), ('act_fn', 'silu'), ('norm_num_groups', 32), ('norm_eps', 1e-05), ('cross_attention_dim', 1024), ('attention_head_dim', [5, 10, 20, 20]), ('dual_cross_attention', False), ('use_linear_projection', True), ('class_embed_type', None), ('num_class_embeds', None), ('upcast_attention', False), ('resnet_time_scale_shift', 'default'), ('_class_name', 'UNet2DConditionModel'), ('_diffusers_version', '0.8.0'), ('_name_or_path', '/home/jerrick/.cache/huggingface/diffusers/models--stabilityai--stable-diffusion-2-base/snapshots/d28fc8045793886e512c5389771d3b3d560f9575/unet')]) expects 4 but received num_channels_latents: 4 + num_channels_image: 4  = 8. Please verify the config of pipeline.unetor yourimageinput.

figured it out

@J3rr1ck I'm running into the same issue. Any pointers?

The diffusers pix2pix-sd model here is the only model I've gotten working so far

yeah. it needs to be a pix2pix model.

this will also work: https://huggingface.co/timbrooks/instruct-pix2pix

I get the following error with the timbrooks model
> ERROR   Model type not supported, img2img pipeline not created

reopening. will take a look. it should work.

Any luck on poking this? Any way others can help?


Including ESXi 8.0 as a supported Hypervisor in readme.txt

Updating readme.txt doesn't necessarily mean it's working on 8.0+
Did you test this to confirm?
Screenshots or code execution examples should be attached.


The CNCF landscape source is in a proprietary vector image format

The use of proprietary vector image formats (e.g. ".ai") prevents most from making their suggested changes directly and submitting a pull request.  If there's no intent to facilitate collaboration, what's the point of using Github for this?  Please move to an open equivalent of some form (perhaps SVG).  Thx!

Rob, the image is being made by our graphic artists in Illustrator. So,
we're releasing the source file he uses.
I think you will be much happier with the 1.0 interactive version that we
are working on which has a JSON backend, where that data can be edited via
GitHub. But, I don't think publishing a SVG would made the file any more
editable.
The point of using GitHub is just to have the discussions of inclusion out
in the open.
On Wed, Apr 5, 2017 at 2:38 PM, Robert Esker<EMAIL_ADDRESS>wrote:

The use of proprietary vector image formats (e.g. ".ai") prevents most
from making their suggested changes directly and submitting a pull request.
If there's no intent to facilitate collaboration, what's the point of using
Github for this? Please move to an open equivalent of some form (perhaps
SVG). Thx!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cncf/landscape/issues/96, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8MBkfZQkIorGB3aOjegbSQ9FlSIeOSks5rtAnRgaJpZM4M02La
.

--
Dan Kohn<EMAIL_ADDRESS>Executive Director, Cloud Native Computing Foundation https://cncf.io/
tel:+1-415-233-1000

Thanks @resker, stay tuned for the upcoming release of the interactive version


Adding William Angliss Institute

https://www.angliss.edu.au/

@StefanBuchman Please provide us with:

the school official website URL
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course

@StefanBuchman You might miss my previous comment. Please react.

URL: https://www.angliss.edu.au/
Technology (including IT course): https://www.angliss.edu.au/courses/foods/food-science-and-technology/

@StefanBuchman Thank you for the clarification. Pull request merged.

Thanks, apologies needed to add the email domain as well, could you approve:
https://github.com/JetBrains/swot/pull/10184


Adopt properties for StableHLO dialects

Request description
Upstream has now an option to assist in the migration to properties. You probably wanna to try to switch before upstream changes the default and deprecate the current settings.
In general the switch is fairly trivial, here is an example of how it was done in OpenAI Triton:

https://github.com/openai/triton/pull/1684
https://github.com/openai/triton/pull/1696

Some extra cleanup that is worthwhile is to replace all uses of getDictionaryAttr to more targeted APIs, this one is now quite costly.

"Some extra cleanup that is worthwhile is to replace all uses of getDictionaryAttr to more targeted APIs, this one is now quite costly". Yeah, this is exactly why we haven't yet upgraded. StableHLO <=> MHLO and StableHLO <=> VHLO is based on getAttrs (example), and we haven't yet had the time to think what to do instead.
"You probably wanna to try to switch before upstream changes the default and deprecate the current settings". Thank you for thinking about us! What timeline do you expect for deprecating and stopping to support old-style attributes?

I don't have a timeline right now, the first milestone will be switching the default, hopefully before the end of the month. Then it'll likely take a few months at least to hammer down everything.

Nice! I'll take a look at this in the near future. Will post in the discord if I hit any issues along the way!

Adding compatibility label since VHLO will need to be migrated as well


Add tvOS platform support

Added tvOS platform support to the podspec file. Tested and works great!
This is my first contribution to github ever, so if something is missing or incorrect, please let me know!

Thanks, nice to have!

@BennetLinden: According to Podspec Syntax Reference, we should use [platform].deployment_target instead, I've already updated this on new release though. PRs always welcome. 😃

Good to know! Thanks for the update!


Add SSL/TLS settings feature for fluentd output Elasticsearch

Is your feature request related to a problem? Please describe.
My elasticsearch cluster endpoints using HTTPS with self-sign certificate. I try to use config "ssl_verify false" of fluentd but the crd doesn't support it.
Describe the solution you'd like
Add SSL/TLS settings feature for fluentd output Elasticsearch.
Additional context
No response

would love to have "ssl_verify" setting also for development environments.

+1

@benjaminhuo, Can I work on this?
@husnialhamdani
Yes, you're well come to work on this.


test(validator): check for correct inheritance

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Checking function is called twice

(fact "Checking function is called twice"
  1 => (fn [x] (do (println "YES" x) true))
  2 => (fn [x] (do (println "NO" x) false)))

Result:
YES 1
NO 2
NO 2

The checking function is called twice when it doesn't match. That's really confusing when debugging. It made me think my code was running twice by mistake.
Thanks, Klaus

I apparently agreed with you several years ago:
                             ;; TODO: It is very lame that the
                             ;; result-function has to be called again to
                             ;; retrieve information that extended-=
                             ;; knows and threw away. But it's surprisingly
                             ;; difficult to use evaluate-checking-function
                             ;; at the top of the cond

I'm gradually working up the infrastructure behind Midje 2.0. Using Specter for parsing - and generally being a better Clojure programmer these days - should avoid such glitches.

Nice :)
On Mon, Jul 20, 2015 at 11:01 AM, Brian Marick<EMAIL_ADDRESS>wrote:

I apparently agreed with you several years ago:
                         ;; TODO: It is very lame that the
                         ;; result-function has to be called again to
                         ;; retrieve information that extended-=
                         ;; knows and threw away. But it's surprisingly
                         ;; difficult to use evaluate-checking-function
                         ;; at the top of the cond

I'm gradually working up the infrastructure behind Midje 2.0. Using
Specter for parsing - and generally being a better Clojure programmer these
days - should avoid such glitches.
—
Reply to this email directly or view it on GitHub
https://github.com/marick/Midje/issues/317#issuecomment-122894799.

addressed in https://github.com/marick/Midje/issues/395


Can't view custom media type response

When I send a request, the response is blank.
However, when I check the console, I get a response.

It seems like the media type is application/ds-json.
Thank you.

That's a bug, I will fix ASAP.

Fixed in v3.0.6.

Thank you!

Fixed in v3.0.6


cleanup: don't remove portable ruby on periodic cleanups.

Otherwise we end up deleting the running Ruby version.
Fixes #5707.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

Thanks @MikeMcQuaid.  Do you have any suggestions for anyone who used brew before this fix and lost portable ruby?

@Amit-PivotalLabs Why do you need portable Ruby?


WeibullAFTFitter.plot_partial_effects_on_outcome

I've used plot_partial_effects_on_outcome for both numeric and categorical covariates for the CoxPHFitter. I can do this for numeric covariates for WeibullAFTFitter, but if I do something like:
 aft.plot_partial_effects_on_outcome(covariates='some_categorical_feature',
    y='cumulative_hazard',
    values=["foo", "bar"])

I get TypeError: must be real number, not numpy.str_.
Again, this works perfectly for a numeric feature for WeibullAFTFitter. Is this only partly implemented for the Weibull model?
(I'm running version 0.25.4)

hrm, this is a bug I've seen before. It may be fixed in the latest version (0.25.6)
Yes, looks like I discovered and patched it in 0.25.5: https://github.com/CamDavidsonPilon/lifelines/commit/adf00b0a4ce496fc72752379da784d828a9ccb13
Give that a try and let me know!

Works now, thanks! I should have tried the latest version before I opened a ticket.
Thanks for your quick response, and also thanks for building such a useful package.


Studio fails to render CommonMarkings

Describe the bug
CommonMark in demo document does not render in righthand "human friendly"  rendering of document.
How to Reproduce
Open studio at https://studio.asyncapi.com/#introduction
Look for /info/description element in title: Streetlights Kafka API (the demo document)
You find Common Mark in lines 8-12:
Check out its awesome features:
* Turn a specific streetlight on/off 🌃
* Dim a specific streetlight 😎
* Receive real-time information about environmental lighting conditions 📈

To the right side bullets and bold are missing

Screenshots

Link to GitHub repository with project that has issues

Expected behavior
Rendered text following CommonMark instructions

The issue has been addressed by altering templates in studio, but that is not a real solution!
Original description for streetlights-Kafka template reads quite OK as code (studio left hand) but fails to render in html (studio right hand). This triggered the issue
  description: >
    The Smartylighting Streetlights API allows you to remotely manage the city
    lights.
    ### Check out its awesome features:

    * Turn a specific streetlight on/off 🌃
    * Dim a specific streetlight 😎
    * Receive real-time information about environmental lighting conditions 📈

Revised description in current streetlights-Kafka template reads horrible as code (lefthand) but has nice rendering. So do we want to set an example of horrible coding??
  description: "The Smartylighting Streetlights API allows you to remotely manage the city lights.\n\n### Check out its awesome features:\n\n* Turn a specific streetlight on/off \U0001F303\n* Dim a specific streetlight \U0001F60E\n* Receive real-time information about environmental lighting conditions \U0001F4C8\n"

Solution can be found at editor.swagger pet store example
replace  "description: >"  with "description:|-" and the engine renders as expected

@eelcofolkertsma I see the issue here.  can you open a PR with a fix?


[TextField] Underline misalignment when Divider present

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
It seems that when using <TextField select={true} /> and when there is a <Divider /> element somewhere in the tree above the underline is one pixel off

The same is happening to the Filled variant
Expected Behavior 🤔
The elements should be colinear despite presence of Divider anywhere on the tree
Steps to Reproduce 🕹

Your Environment 🌎

Tech
Version

Material-UI
v4.9.11

React
v16.13.1

Browser
Chrome 81.0.4044.122

It looks alright in my env.

Probably it happens only on 4k displays?

It seams a problem of height. If you try to change the zoom of the page you can see the error.
I think that something like this can solve
+++ b/packages/material-ui/src/TextField/TextField.js
@@ -20,6 +20,9 @@ const variantComponent = {
 export const styles = {
   /* Styles applied to the root element. */
   root: {},
+  fullHeight: {
+    height: "100%",
+  },
 };

 /**
@@ -149,6 +152,10 @@ const TextField = React.forwardRef(function TextField(props, ref) {
       inputProps={inputProps}
       {...InputMore}
       {...InputProps}
+       className={clsx(
+         classes.fullHeight,
+         InputProps?.className,
+       )}
     />
   );

@@ -179,6 +186,10 @@ const TextField = React.forwardRef(function TextField(props, ref) {
           value={value}
           input={InputElement}
           {...SelectProps}
+           className={clsx(
+             classes.fullHeight,
+             SelectProps?.className,
+           )}
         >
           {children}
         </Select>

@marcosvega91 Looking at the history, it seems that we can remove an older patch for #13302:
diff --git a/packages/material-ui/src/NativeSelect/NativeSelect.js b/packages/material-ui/src/NativeSelect/NativeSelect.js
index 00464905b..8d86decaf 100644
--- a/packages/material-ui/src/NativeSelect/NativeSelect.js
+++ b/packages/material-ui/src/NativeSelect/NativeSelect.js
@@ -58,7 +58,6 @@ export const styles = (theme) => ({
   },
   /* Styles applied to the select component `selectMenu` class. */
   selectMenu: {
-    height: 'auto', // Reset
     textOverflow: 'ellipsis',
     whiteSpace: 'nowrap',
     overflow: 'hidden',

Would that work?

@marcosvega91 I noticed that simple removal of height: auto helps, but git blame shows that that this is some kind of native-select fix for FF. I am not sure that it is still actual because removing it does not break anything
--- a/packages/material-ui/src/NativeSelect/NativeSelect.js
+++ b/packages/material-ui/src/NativeSelect/NativeSelect.js
@@ -58,7 +58,7 @@ export const styles = (theme) => ({
   },
   /* Styles applied to the select component `selectMenu` class. */
   selectMenu: {
-    height: 'auto', // Reset
+    // height: 'auto', // Reset
     textOverflow: 'ellipsis',
     whiteSpace: 'nowrap',
     overflow: 'hidden',

@esseswann I have just tested it, it seems that we fixed the issue later on by forcing the height.

Seems to be working on the variant=filled aswell

@oliviertassinari So does anybody make a pull request? :)

@esseswann Priority to the author of the issues ;)

I was checking it for NativeSelect and accidentally noticed that NativeSelect from current master no longer works, the dropdown is not opening onClick. It works on the website though.
I did not apply any fixes, might be a separate issue

@esseswann Solved in #20777.


UI: Positioning issues for the illustrations

Issue:
The cards and the illustration are not positioned properly, and the logos in the cards are not having proper positioning.

I am working on these, can you  assign me? @Saurav-Pant

Go ahead, @Sayan-dev


fix(chat): fix chats and chat folders removing if such exist somewhere in 'shared with me' section (Issue #838)

Description:
Fix chats and chat folders removing if such exist somewhere in 'shared with me' section
Issues:

https://github.com/epam/ai-dial-chat/issues/838

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review

/deploy-review

/deploy-review

/deploy-review


Using natural gradient for the distibution of inducing variables in the inner layers

Dear Salimbeni,
In your demo you used the natural gradient to optimize the distribution of the inducing variables at the final layer. I tought that it may be interesting to use the natural gradient also for the distribution of the inducing variables in the inner layers. However, i always obtain an error in the Cholesky decomposition. " Cholesky decomposition was not successful. The input might not be valid." Which i never obtain when using the natural gradient only for the the final layer.
Did you encounter this problem?
Thank you in advance.

An update: I've done some more experiments with this and have found that nat grads can work well if tuned correctly, but can sometimes converge to poor local optima. This is to be expected, I think, since the optimization of the inner layer is potentially highly multimodal, so momentum based optimization might find better a optimum in practice.

I've made some more experiments and the interesting observation is that when it is possible to use nat grads for the inner layers with a relative high gamma (0.1) the results are better than nat grad only in the inner layers, and this in terms of ELBO, better uncertainty estimation and better prediction of the final model. (My experiments were restricteed to the two layers case)
however, when i'm restricted to a small gamma the nat grad for the inner layers is very slow to converge, which is expected.
My guess is if we can keep a good gamma value for the nat grad for all the inner layers, we may have better results.
The issue of the chelosky decomposition comes from the ill conditionned updated nat2 parameter when we try to transform it to the covariance parameter . I was thinking if it was possible that instead of updating the nat2 parameter, we update its square root matrix L_nat2. However, this means that the gradient must be taken with respect to L_nat2 and not the second natural parameter. But it will constrain the updated  nat2 to be positive definitive. Do you think this can help avoiding this problem?

I've made some more experiments and the interesting observation is that when it is possible to use nat grads for the inner layers with a relative high gamma (0.1) the results are better than nat grad only in the last layer, and this in terms of ELBO, better uncertainty estimation and better prediction of the final model. (My experiments were restricteed to the two layers case)
however, when i'm restricted to a small gamma the nat grad for the inner layers is very slow to converge, which is expected.
My guess is if we can keep a good gamma value for the nat grad for all the inner layers, we may have better results.
The issue of the chelosky decomposition comes from the ill conditionned updated nat2 parameter when we try to transform it to the covariance parameter . I was thinking if it was possible that instead of updating the nat2 parameter, we update its square root matrix L_nat2. However, this means that the gradient must be taken with respect to L_nat2 and not the second natural parameter. But it will constrain the updated nat2 to be positive definitive. Do you think this can help avoiding this problem?

The nat grad code allows the nat grad taken with respect to different parameterizations. The default is using the natural parameters as this works best in this paper, but this wasn't assessed for a deep gp. I've never tried different parameterizations for the deep gp but would be interested to know how well it works!


manpage never mentions HTTPS access

From some strange reason the manpage of gitup(1) never mentions that gitup clones only via anonymous HTTPS and nothing else. Please add that important piece of information.

Done.  Thank you!

Done.  Thank you!


Update list of members from Ant Group

These members are no longer with Ant Group.

/lgtm
/approve
Thank you for the update!


[BUG]: URL components are being suggested

Components of URLs that are included in the note are suggested, too, which probably makes little sense. I'd simply tell the plugin to ignore any url it finds?

Hi, @chrisgrieser. Thank you for your bug report.
I couldn't reproduce it. Could you tell me more details?

URL on file
settings

I could reproduce it. I'll fix it 👍

@chrisgrieser
I have released to v3.4.0 🚀
Please confirm whether it works as you expected.

yep, can confirm, doe snot occur anymore. Thanks for the quick fix!


git_info.h debate

git_info.h should probably not be a public header or at least should be grouped under nbl/ as it may conflict with other projects using Nabla.
https://github.com/Devsh-Graphics-Programming/Nabla/blob/de62785b6db71a04154fd6ba4039ce1d87193e19/include/nbl/video/ILogicalDevice.h#L17
See also https://github.com/Devsh-Graphics-Programming/Nabla/pull/775#discussion_r1834065004

git_info.h should probably not be a public header

it should be in public headers, metadata should be accessible for projects linking Nabla

nbl/ as it may conflict with other projects using Nabla

however we could have better include path for each meta header


Customise navigation pane

Hi Team,
Can the navigation pane be customized for all users?
I aim to restrict the visibility of certain features for the 'Attack Payload Author' user once they log into the security portal. Currently, users can navigate to different sites within the portal, which is not ideal.

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

ID: 59c87455-77af-093a-6fb6-286049aad15a
Version Independent ID: b2690e98-781d-4a27-a9bc-bd621964feb0
Content: Get started using Attack simulation training
Content Source: microsoft-365/security/office-365-security/attack-simulation-training-get-started.md
Service: microsoft-365-security
Sub-service: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@bally-singh, if there is a way, it likely involves custom permissions in Entra (Azure AD). But, there's no guarantee. In the Exchange Online world, there isn't always a 1:1 relationship between permissions (roles/cmdlets) and what you have access to in the UI.

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


FR: Allow the proxy to hijack some net.Conn's to be serviced internally

In short, I want to make it possible for the proxy to hand off the connection to some go code to handle, rather than always proxy it out.
I know server.Dial exists, but I think thats a little messy + lots of wrapping+unwrapping layers of abstraction (for this use case).

The client connects to the proxy and issues a connect for a specific endpoint.
The connect request matches some internal state indicating it should be hijacked.
The server proceeds as normal, except instead of opening the remote connection and proxying the data, it passes off the net.Conn to a callback.

In my use case, I pass the net.Conn to a http.Server and serve proxy status pages.
I am happy to code this and send a PR, just wanted to make sure this is okay with you.

I envision the API looking like this:
conf := &socks5.Config{
   Hijacker: myHijackerFunction,
}

// myHijackerFunction returns True if the connection is being hijacked.
func myHijackerFunction(req *Request, conn *net.Conn) bool{
  if req.RemoteAddr.FQDN == 'proxystate' {
    handleProxyStateRequest(conn)
    return True
  }
  return False
}

Implemented. Lemme play around with usage and implementation in prod before merge.
Thoughts? https://github.com/twitchyliquid64/go-socks5/commit/9398063b9e2ea7e0e57d93ab872be448d802c764

@twitchyliquid64 Looks good, I would just simplify from ShouldHijack and HandleHijack to a single Hijack which returns (bool, error) with the boolean indicating if it did hijack.

@twitchyliquid64 Link is not working anymore. Could you upload it again?

I ended up using goproxy instead.

@twitchyliquid64 Could you upload the example?

Cant find it I'm afraid, sorry.

I think you should add the hook somewhere in this function https://github.com/armon/go-socks5/blob/master/request.go#L119


Fix URLs with '@' parsing as emails

Only consider a link an email if it fails to parse as URL.
Also use a proper email validation instead of a simple '@' check.
This uses the fast_chemail crate which parses email links according
to the HTML specification (which is much more practical than checking
for RFC 5322 formatted emails).  It's also worth noting that
fast_chemail is used internally (albeit indirectly) by the
check_if_email_exists crate.  This means that email addresses
not considered valid by fast_chemail wouldn't pass link checks
anyway.
Fixes #161.

Brilliant PR. Thanks so much @pawroman! ⭐


[Event Request] Table 77 "Report Selections" - OnBeforeBLOBImportFromServerFile

Hi, would it be possible to get an OnBeforeBLOBImportFromServerFile event in table 77 (report Selections) ?
We need this to attach documents merged with spécific PDF documents

Please provide definition of required event and there it should be added in the code.

Could be like this :
`procedure SaveAsDocumentAttachment(ReportUsage: Integer; RecordVariant: Variant; DocumentNo: Code[20]; AccountNo: Code[20]; ShowNotificationAction: Boolean)
...
begin
...
with TempAttachReportSelections do
repeat
if CanSaveReportAsPDF(TempAttachReportSelections."Report ID") then begin
IsHandled := false;
OnBeforeBLOBImportFromServerFile(RecordVariant: Variant; TempAttachReportSelections: Record "Report Selections"; var IsHandled: Boolean)
                if not IsHandled then
                    FileManagement.BLOBImportFromServerFile(
                        TempBlob,
                        SaveReportAsPDF("Report ID", RecordVariant, "Custom Report Layout Code", "Report Selection Usage".FromInteger(ReportUsage)));

                SaveDocumentAttachmentFromRecRef(RecRef, TempAttachReportSelections, DocumentNo, AccountNo, TempBlob, NumberOfReportsAttached);
            end;
        until Next() = 0;

`

Is it ok for you to use existent event OnBeforeSaveDocumentAttachmentFromRecRef for SaveDocumentAttachmentFromRecRef function?

Just saw this issue. We asked in the linked issue to not use BLOBImportFromServerFile in this case and use the cloud compliant solution instead where we could completely swap the Blob content. Maybe that would work as well?
https://github.com/microsoft/ALAppExtensions/issues/14356

@Oleg-Romashkov
Maybe - I'm just trying to do this - Only there is one thing that is missing at this moment :
The event
OnBeforeSaveDocumentAttachmentFromRecRef(RecRef, TempAttachReportSelections, DocumentNo, AccountNo, TempBlob, IsHandled);
does not contain the variable NumberOfReportsAttached as a VAR Parameter - and so I get wrong notification
Is it possible to add this parameter ?
@DavidFeldhoff
What do you mean by "use the cloud compliant solution" ?

Not relevant, I just re-read your case and had to notice that the idea I posted is not helpful here in your case, sorry :)

@Oleg-Romashkov
After testing i'm able to subscribe and do my merging job by using the event OnBeforeSaveDocumentAttachmentFromRecRef you proposed.
As I said the only thing missing is variable NumberOfReportsAttached as a VAR Parameter - Could this be done ?

Yes, sure. I will do this then.

Thanks a lot !
What do you think in which version this will be available ?

Usually we do not provide this information. Hopefully it will be in 18.6 and 19.1

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


lemmy-ui: unpin nodejs-14_x

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
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

[x] Fits CONTRIBUTING.md.

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

1 package built:

lemmy-ui

passthru.tests are failing.


Update : Doodle Jump

https://github.com/ChewyChiyu/Doodle-Jump
new Updates: Monsters (MEANIE GREENIES! dey dont do much but fun 2 shoot) shoot dem cute monsters by left clicking! Also pause the game with right click! Hopefully if you download you can play with no thread issues.

Stop showing off.

hue
On Thu, May 18, 2017 at 1:47 PM, Brandon Phan<EMAIL_ADDRESS>wrote:

Closed #8 https://github.com/bphun/FinalProject/issues/8.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bphun/FinalProject/issues/8#event-1088530945, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZmnBdR2g1mc2g4Y-jMRmHgwBGNkEwitks5r7K5XgaJpZM4NYyk8
.

my sprite reader is dope af
On Thu, May 18, 2017 at 4:41 PM, Evan Chen<EMAIL_ADDRESS>wrote:

hue
On Thu, May 18, 2017 at 1:47 PM, Brandon Phan<EMAIL_ADDRESS>wrote:

Closed #8 https://github.com/bphun/FinalProject/issues/8.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bphun/FinalProject/issues/8#event-1088530945, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZmnBdR2g1mc2g4Y-jMRmHgwBGNkEwitks5r7K5XgaJpZM4NYyk8
.

look at the sprite reader it got revamped with a new UI
On Thu, May 18, 2017 at 4:51 PM, Evan Chen<EMAIL_ADDRESS>wrote:

my sprite reader is dope af
On Thu, May 18, 2017 at 4:41 PM, Evan Chen<EMAIL_ADDRESS>wrote:

hue
On Thu, May 18, 2017 at 1:47 PM, Brandon Phan<EMAIL_ADDRESS>wrote:

Closed #8 https://github.com/bphun/FinalProject/issues/8.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bphun/FinalProject/issues/8#event-1088530945, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AZmnBdR2g1mc2g4Y-jMRmHgwBGNkEwitks5r7K5XgaJpZM4NYyk8
.


DMS Explorer v0.7.37 | application does not work when I have internet connection

Expected behavior
The application should be running / working and not problematic.
Actual behavior
Application not working. It said I do not have an internet connection. Though the connection I have is 4G. I prove it in the recording. I play youtube can be normal.
How to reproduce

Download app on playstore
Open the app and turn on your internet connection
it will say you do not have an internet connection.

Device: ONE PLUS 2 OxygenOS Version 3.6.1
Operating System:Android 6.0.1
App Version : 0.7.37
Recording of the bug
https://youtu.be/YYKSMc_vPw0

That is correct behavior as designed. Because this app is a DLNA player.
Local network connection (WiFi) is required to use DLNA. It does not work on 4G.


fix: Fix typo that breaks styling in components list

Coverage remained the same at 80.583% when pulling d24fceb4fe6930bc2165d78966c8eca29d67f06f on fix-components-list-typo into bb71ad2e3f81e32130f5399317bd5beb6ac913d7 on develop.


feat: port SetTheory.Game.Basic

There was no activity here for ~2 weeks, so I assume that you are not actively working on this PR. I'll add help-wanted so that anyone can contribute without an explicit permission from the author.

I got quot_left_distrib and quot_mul_assoc working. Now the only remaining task I see is to resolve some simpNF linter issues.

@Parcly-Taxel the new lemmas you should added, e.g.
@[simp]
theorem neg_mk_mul_moveLeft_inl' {xl xr yl yr} {xL xR yL yR} {i j} :
    -(mk xl xr xL xR * mk yl yr yL yR).moveRight
    (@FunLike.coe _ _ (fun _ => RightMoves (mk xl xr xL xR * mk yl yr yL yR))
      Equiv.instFunLikeEquiv toLeftMovesNeg.symm (Sum.inl (i, j))) =
    -(xL i * mk yl yr yL yR + mk xl xr xL xR * yR j - xL i * yR j) :=
  rfl

seem unlikely to be useful to me. At least for this file, we don't need them. Could you either:

remove them
working out how to reformulate them in some way that is still useful, but doesn't have @FunLike.coe?

@Parcly-Taxel the new lemmas you just added, e.g.
@[simp]
theorem neg_mk_mul_moveLeft_inl' {xl xr yl yr} {xL xR yL yR} {i j} :
    -(mk xl xr xL xR * mk yl yr yL yR).moveRight
    (@FunLike.coe _ _ (fun _ => RightMoves (mk xl xr xL xR * mk yl yr yL yR))
      Equiv.instFunLikeEquiv toLeftMovesNeg.symm (Sum.inl (i, j))) =
    -(xL i * mk yl yr yL yR + mk xl xr xL xR * yR j - xL i * yR j) :=
  rfl

seem unlikely to be useful to me. At least for this file, we don't need them. Could you either:
* remove them

* working out how to reformulate them in some way that is still useful, but doesn't have<EMAIL_ADDRESS>

I don't know how to get the FunLike.coe out –
toRightMovesNeg.symm (Sum.inr (i, j))

as shown in the infoview doesn't work.

Okay, I've created a lean4 issue for the termination_by problem (or at least something closely related?).
@Parcly-Taxel, if you could just fix the porting notes then I think we can merge.

@semorrison OK, done.

Bors merge


I am getting an error after upgrading the react-native to the latest version(Could not find the following native modules)

I done the following to update all npm and react-ative latest version.
For npm, I ran npm update

To update the react-native, I have changed the version in package.json and ran npm install. It updated the react-native version

After this, I have run react-native run-ios it shows all my packages are manually linked and you need to unlink all packages.
Also, can anyone clarify the react-native-config.js file is mandatory for this after updating to the latest react-native.
Always I open the app.Xcodeproj but the below shows I need to open app.xcworkspace. Can anyone clarify this?

warn The following packages use deprecated "rnpm" config that will stop working from next release:
  - react-native-fetch-blob: https://github.com/wkh237/react-native-fetch-blob#readme
  - react-native-orientation: https://github.com/yamill/react-native-orientation#readme
  - rn-fetch-blob: https://github.com/joltup/rn-fetch-blob#readme
Please notify their maintainers about it. You can find more details at https://github.com/react-native-community/cli/blob/master/docs/configuration.md#migration-guide.
error Could not find the following native modules: BugsnagReactNative, react-native-camera, react-native-cookies, react-native-fast-image, react-native-fetch-blob, RNFS, RNGestureHandler, RNImageCropPicker, react-native-image-picker, RNImageRotate, BVLinearGradient, react-native-orientation, RNShare, RNSVG, RNVectorIcons, RNViewShot, react-native-webview, rn-fetch-blob. Did you forget to run "pod install" ?
info Found Xcode workspace "Fashion.xcworkspace"

react-native info
react-native info
warn The following packages use deprecated "rnpm" config that will stop working from next release:
 - react-native-fetch-blob: https://github.com/wkh237/react-native-fetch-blob#readme
 - react-native-orientation: https://github.com/yamill/react-native-orientation#readme
 - rn-fetch-blob: https://github.com/joltup/rn-fetch-blob#readme
Please notify their maintainers about it. You can find more details at https://github.com/react-native-community/cli/blob/master/docs/configuration.md#migration-guide.
info Fetching system and libraries information...
System:
   OS: macOS Mojave 10.14.2
   CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
   Memory: 158.01 MB / 8.00 GB
   Shell: 3.2.57 - /bin/bash
 Binaries:
   Node: 9.4.0 - ~/.nvm/versions/node/v9.4.0/bin/node
   npm: 5.6.0 - ~/.nvm/versions/node/v9.4.0/bin/npm
 SDKs:
   iOS SDK:
     Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
 IDEs:
   Xcode: 10.1/10B61 - /usr/bin/xcodebuild
 npmPackages:
   react: 16.8.1 => 16.8.1
   react-native: ^0.61.4 => 0.61.4
 npmGlobalPackages:
   react-native-cli: 2.0.1
   react-native-git-upgrade: 0.2.7

Hey nihp ,
First of all upgrading like that will only upgrade react-native core files. But your  IOS / Android Files won't be in sync with the latest updates.
You can either do that manually by using : upgrade-helper
or you can also automate most of that using the upgrade command in cli : react-native upgrade  /
Hope this helps !

react-native upgrade not works for me. So I have changed the package.json and make it to upgrade.
error Failed to build iOS project. We ran "xcodebuild" command but it exited with error code 65. To debug build logs further, consider building your app with Xcode.app, by opening Fashion.xcworkspace. Run CLI with --verbose flag for more details.
error Could not find the following native modules: BugsnagReactNative, react-native-cookies, react-native-fast-image, react-native-fetch-blob, react-native-image-picker, react-native-orientation, RNSVG, rn-fetch-blob. Did you forget to run "pod install" ?

Follow this upgrade helper and make sure you add this to your Podfile:
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

....

use_native_modules!

Then run pod install. This solved my issue.

Follow this upgrade helper and make sure you add this to your Podfile:
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

....

use_native_modules!

Then run pod install. This solved my issue.

running pod install create a dependency issue
144 duplicate symbols for architecture x86_64


Expose HTTP Client

From<EMAIL_ADDRESS>on November 05, 2012 06:34:49
Add a detailed config or filter that let's you configure HTTP Client directly
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=203

From<EMAIL_ADDRESS>on July 01, 2013 06:58:16
Issue 247 has been merged into this issue.

From<EMAIL_ADDRESS>on November 12, 2013 06:47:31
Status: Fixed


filetype: djot files not recognized

https://djot.net
just yanked these from https://github.com/jgm/djot/tree/main/editors/vim

You've added the syntax file as a ftplugin, can you please move it to runtime/syntax/djot.vim?

sorry, fixed

@jgm, are you happy for this to be distributed with the Vim runtime?

Well, these are very crude syntax files that I wrote in the early stages of this project, and haven't maintained even as the syntax has changed.  So I don't think it would be appropriate yet to make it part of vim.


Replacement for RunAs in Azure Automation for on-prem scripts

Currently it is possible to execute PowerShell scripts against on-premise Active Directory infrastructure from Azure using Azure Automation with a hybrid worker and RunAs using Active Directory credentials for authentication (see https://practical365.com/how-to-manage-on-premises-infrastructure-using-azure-automation-hybrid-worker/).
Since Managed Identities cannot be used to authenticate to Active Directory, what is the recommended migration path for this use case as RunAs is being deprecated?

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

ID: 3eedb810-487f-bc9c-89f5-d5fdbcc5d796
Version Independent ID: 329e9ec7-d9ea-518b-625b-d39880365e80
Content: Migrate from a Run As account to a managed identity
Content Source: articles/automation/migrate-run-as-accounts-managed-identity.md
Service: automation
Sub-service: process-automation
GitHub Login: @SnehaSudhirG
Microsoft Alias: sudhirsneha

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

@LuckyLeo2008 - Thank you for your patience. The Engg team has confirmed that you can use Managed Identities to authenticate  Azure AD resources. Pl check this article for more information - https://learn.microsoft.com/en-us/azure/automation/automation-security-overview#managed-identities
Also, check the prerequisites mentioned here -https://learn.microsoft.com/en-us/azure/automation/migrate-run-as-accounts-managed-identity?tabs=run-as-account#prerequisites

#please-close

There seems to be a confusion between run as account and hybrid worker credentials. Went through the doc shared in the issue by @LuckyLeo2008 and based on the analysis, there are two parts :

run as credential (now named as hybrid worker credential) - This contains the username and password of the hybrid worker on-prem machine. This feature is not getting deprecated. It will remain the same.

run as account- this is something which is being used by the users in the script with azurerunascertificate and azurerunasconnection. This is replaceable with Managed identity. Details here: https://learn.microsoft.com/en-us/azure/automation/migrate-run-as-accounts-managed-identity?tabs=run-as-account#prerequisites


In causual mode, this expression will lead to a null tensor 'out'

https://github.com/ksw0306/FloWaveNet/blob/c436e128cbb03ccb30e5e05ed1f3c88451aafe75/modules.py#L20-25
I fix this bug by replace this expression with
out = out[:, :, :out.size(2)-self.padding]
and this code work well in causal mode.

Thank you for the bug report! We've reproduced the issue and it's caused form mistakenly cleaning up some of the code for the GitHub release. We'll commit the our original working version, which has the same effect when the self.padding gets 0.


request: The current SDK bundles target framework '8.0' which is not supported by MSBuild.Sdk.SqlProj.

Currently updating from .net 6 to .net 8 and receive error message MSBuild.Sdk.SqlProj is not supported beyond .6? Any fix for this?

Which SDK version do you use? As you can see in the readme,  latest is 2.7.2

Thank you - updated.

From: Erik Ejlskov Jensen @.>
Sent: Monday, May 20, 2024 11:41 AM
To: rr-wfm/MSBuild.Sdk.SqlProj @.>
Cc: Sam Choudry @.>; Author @.>
Subject: [External]Re: [rr-wfm/MSBuild.Sdk.SqlProj] request: The current SDK bundles target framework '8.0' which is not supported by MSBuild.Sdk.SqlProj. (Issue #560)
==> External Email : Use Caution with Links and Attachments <==
Which SDK version do you use? As you can see in the readme, latest is 2.7.2
—
Reply to this email directly, view it on GitHubhttps://github.com/rr-wfm/MSBuild.Sdk.SqlProj/issues/560#issuecomment-2120811816, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BITV2H3I6USSMLPPIE2SE43ZDIRSXAVCNFSM6AAAAABIAAYGJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRQHAYTCOBRGY.
You are receiving this because you authored the thread.

Assume you are unblocked now?


feat: add orange contact everyone plugin

Fixes: AM-3077

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 5.14.0-AM-3077-orange-plugin-SNAPSHOT
download it from Artifactory here

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

5.14.0
GitHub release

Your semantic-release bot :package::rocket:


Separate time polling and time generating for X-Media-Response-From: Generated Remotely responses

Currently the two other headers are 'X-Media-Response-Start' (absolute start time) and 'X-Media-Elapsed-Seconds' (relative end time).
This needs to be changed to:

X-Media-Response-Start (absolute start time): request received
X-Media-Processing-Elapsed-Seconds (time period in seconds): from job queue to job finished
X-Media-Waiting-Elapsed-Seconds (time period in seconds): from job finished to data sent to client
X-Media-Elapsed-Seconds (time period in seconds): from request received to data sent to client

Almost finished. Header keys, values, and descriptions:

X-Media-Response-From (possible values: 'Cache',  'Generated Remotely', 'Generated Locally'): How media response was carried out.
X-Media-Response-Start (absolute date time): Date and time response began processing.
X-Media-Elapsed-Seconds-Total (seconds): total time taken for response.
X-Media-Elapsed-Seconds-Processing (seconds): time taken in generating (local: from or remote).

local: measured from start of media processing to processing completed
remote file polling: from polling started to file available on disk (or time out error)
remote resque polling: from polling started to job complete (will respond with error if job finishes with error or poll times out)

X-Media-Elapsed-Seconds-Waiting (seconds): time spent waiting - from end of processing to response data sent (in case of remote resque poll, waiting time includes time to refresh directory listing to get available files).

Almost finished. Header keys, values, and descriptions:

X-Media-Response-From (possible values: 'Cache',  'Generated Remotely', 'Generated Locally'): How media response was carried out.
X-Media-Response-Start (absolute date time): Date and time response began processing.
X-Media-Elapsed-Seconds-Total (seconds): total time taken for response.
X-Media-Elapsed-Seconds-Processing (seconds): time taken in generating (local: from or remote).

local: measured from start of media processing to processing completed
remote file polling: from polling started to file available on disk (or time out error)
remote resque polling: from polling started to job complete (will respond with error if job finishes with error or poll times out)

X-Media-Elapsed-Seconds-Waiting (seconds): time spent waiting - from end of processing to response data sent (in case of remote resque poll, waiting time includes time to refresh directory listing to get available files).


W7.7

Added user stories, use cases, non-functional requirements and glossary with correct formatting.

Update of Developer Guide is done as a collaborative effort of @aali195 and myself on Google Docs.


Persistent TCP connections to UniMRCP cause memory to grow due to "leaked" messages

Originally reported on Google Code with ID 68
What steps will reproduce the problem?
1. Establish a persistent MRCPv1 connection over TCP
2. Every message received by UniMRCP will use memory allocated from the
pool associated with the connection, so they can not be freed.
3. Once the connection is dropped the pool is released, freeing the
"leaked" memory.

What version of the product are you using? On what operating system?
v0.8.0 on RedHat linux ES 3 update 8

Please provide any additional information below.
Some media servers seem to try to establish a heartbeat connection to
monitor the resource.  This causes a memory issue if the heartbeat
connection remains up for a long enough time.

Reported by dchidelf on 2010-02-10 18:52:46

This issue was actually addressed in UniMRCP 1.5.0 by introducing max lifetime (use count) for a TCP connection.


[QPG6100] Apply size optimizations

Problem
QPG6100 area filling up with latest development.
Solution
Tuning down over-dimensioned/unneeded features.

Lower NVM area reserved - 8kB flash
Disable Detail logging - 4kB flash
Remove TCP/RAW from inet and LWIP - 14 kB flash - 2kB static RAM

Frees up 26kB flash/2kB static RAM

Added cherry-pick label as SDK version update also holds a fix for TE2 - BLE UUID used is updated


Not trigger ActiveRecord loading on requiring gem

When this gem required ActiveRecord::Base triggered to load and make connections to database. This should not be due to (for example) failing assets:precomplie in Heroku (https://devcenter.heroku.com/articles/rails-asset-pipeline#troubleshooting), where build phase and run phase are separated and database is not available on build phase.

Great work, @gsmetal!

@gsmetal I've added you to the gemspec as a contributor and I've published a new version of the gem. Thanks for your contribution!


Adept shield still boosts armor stats

Trowl223 said he wanted to remove adepts armor buff, which if it stays, with the full armor feat for adepts, that is a lvl 2 with 50 dr with 100% coverage, 60 DR if cyborg...

I don't recall discussing that.

Well then, let's discuss it. Isn't that OP at level 2?

It was the one off mission where I had the Adept with the chain who cross trained paladin for shield, @bleehu almost died...

I remember now. Mostly it was the paladin shield in addition to the Adept shield with armour.
As far as the Adept's own abilities go, I don't see how being a level 2 changes anything, the adept's shield abilities aren't level based.
I think we should yank Adept for the time being. They were meant to be what the paladin became, and I'll need to rework them to find a good niche for them. The idea of the class is basically the tankiness of the paladin, minus automatic rifles, long rifles, and spells.
Verdict: Yank for now until such a time when they have a more distinct role.

no, it was lvl 2 = full armor feat

Please clarify. Do you mean that at level 2 you take the full armour feat with the class? In the future please mention a feat by name rather than just saying "at level two this class has this ability." We are rapidly approaching the point (or have surpassed it, really) where just saying that a class has a certain output potential means nothing because of how many possible feat combinations can be made.

Acquired: Metal Jacket
Prerequisites: Soldier, Engineer, Adept, Berserker
Enables the use of Full Armour by the listed classes
Sorry, here is the feat, taken at level 2

plus cyborg and adept shield bonus to dr


Update monix-execution to 3.4.0

Updates io.monix:monix-execution from 3.3.0 to 3.4.0.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.monix", artifactId = "monix-execution" } ]

labels: library-update, semver-minor

Waiting for release notes (https://github.com/monix/monix/releases) before merging to see what changed

The release is binary and source compatible with 3.x series, and was cross-built for the following Scala and ScalaJS versions:

    Scala 2.12, 2.13 and 3.0
    Scala.js 1.5.1

WARN: we're dropping compatibility with Scala 2.11 and ScalaJS 0.6.x. If you still need those you'll have to stay on version 3.3.0.

Changes in this release:

    Dropped support for Scala 2.11 and Scala.js 0.6.x
    Dependency updates:
        Cats 2.6.1
        Cats-Effect 2.5.1
        JCTools 3.3.0
    Adds support for Scala 3 (#1326, #1327, #1328, #1329, #1344, #1323)
    Adds Observable.whileBusyAggregateEvents (#1320)
    Fix tracing in Coeval and Task via a more accurate filter (#1353)
    Adds Observable.throttleLatest (#1396)
    Implement pagination for Observable (#1381)


[status]: Fail generic type

Hello! Currently status is passing Param & Done generics to underlying effect. All my effects have a custom 'Fail' type, so typescript complains about Types of property 'fail' are incompatible.. It would be nice to be able to pass Fail type

@irrvrsl Thank you. It will be released in the next version.


Support text search.

added create|drop index for meta.

Ready to review.

integrate with listener.

Done. ready to review.

Resolve the conflict

Addressed darionyaphet's comments.

Simplified the method loadFulltextClients