Optimize Node, remove duplicate Settings

Description
I find some code might be optimized.
Issues Resolved
No related issue, since this is a small optimize
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thanks for making the code concise! 😄

You are welcome, I am interested in OpenSearch, hope to make this project better.


Create License

Hi @DoonOnthon 👋,
I would recommend you create a LICENSE file. If you need help, just let me know.
Best regards
NH

Furthermore, I would recommend that you create a Security.md and issue templates (see "Insights > Comunity Standards")

A useful tip for improving your security measures is to utilize code scanning alerts @DoonOnthon. To access this feature, navigate to the Security tab and select Code scanning alerts. From there, click on the button and proceed to the Code Scanning tab where you can set up Code QL analysis by clicking on the corresponding button. This will ensure that your code is thoroughly scanned for any potential vulnerabilities.


Merge telegram into main repository

Telegram connector needs some changes before merging:

[x] Remove dff/core/runner.py, rewrite addon using dff.core.pipeline
[x] Examples in tests should be executed, not compiled
[x] Example tests should be skipped if BOT_TOKEN or TEST_CHAT are not set or if pytelegrambotapi package is not installed
[x] I already set BOT_TOKEN in github secrets. Need to decide what to set TEST_CHAT to
[x] Update tests and examples once #34 is merged

I've got 404 on https://sun9-49.userapi.com/s/v1/if2/gpquN.jpg in examples/generics/media.py:62 and 65

Shouldn't we add __init__.py file to dff/messengers/telegram?

I don't understand. It's there, no?
https://github.com/deeppavlov/dialog_flow_framework/blob/merge/telegram/dff/messengers/telegram/__init__.py

add usecases with several values of payload

I'm not sure what that was supposed to mean, but I think I completed it in example 3: https://github.com/deeppavlov/dialog_flow_framework/pull/13/commits/b0d4e588d6e40bf88a23667cc8352afc111ffdae

8_conditions_extras.py

This one is difficult.  It uses conditions, that are currently not achievable during testing (e.g., ChatJoinRequest). Maybe we should use PollAnswer or ShippingQuery as extra conditions instead.

2_responses.py (with attachments)
2_responses_bached_attachments.py (with batched attachments)

Feels unnecessary. The concept is simple (so it doesn't need a separate example), and is already shown in conditions_with_media.
Also, sqlite is failing. Maybe need to restrict version of sqlalchemy.


Filter Inventory by Location report by Location

I would like to see if I can help address this issue

@BrianHC are you still working on this one?

@tangollama sorry for long absence, I am still working on this feature! I will try to get this feature implemented sometime this week.

Awesome. Thx.
Typed or dictated imperfectly on my iPhone<PHONE_NUMBER>)

On Apr 26, 2016, at 3:43 PM, BrianHC<EMAIL_ADDRESS>wrote:
@tangollama sorry for long absence, I am still working on this feature! I will try to get this feature implemented sometime this week.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@BrianHC consider this a friendly #bump on this issue. Are you still working it?

Hi Joel! Thank for friendly reminder. I've been looking into it, on the
screen it looks like there is a drop-down to filter the inventory report by
location already. Is there any more requirements specified for this request?
On May 14, 2016 9:14 AM, "Joel Worrall"<EMAIL_ADDRESS>wrote:

@BrianHC https://github.com/BrianHC consider this a friendly #bump on
this issue. Are you still working it?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/HospitalRun/hospitalrun-frontend/issues/126#issuecomment-219219792

@BrianHC you are correct that there is a filter that appears to meet this requirement
@tangollama it looks like we already do this or are you looking for something else:

It's a open source miracle! Seriously, sorry I didn't follow up on this before I reached out @BrianHC. I'll close this. You up for tackling something else?

No problem! It was still fun taking a look into how our report system works. Absolutely, I'll take a look at what issues are still open and put my name on one. Unless you have any that you recommend?


target temperature of control_dhw

Hi!
I'm new to a your integration (and a vaillant heat pump), but already very happy to have a solution to get the data.
I'm curious about the control_dhw entity because I expect there my target temperature according to my time table for hot water. Actually it shows me during day (with target temperature in sensoApp of 55°C) no temperature at all and mode time_controlled:

During night time (eco mode) the temperature is shown as 35°C (which is in your note the default):

I would like to get (& change) this temperature during day-time (depending on my pv-power)
Additionally, how can I re-enable the "time_controlled" mode when I switched to on/off/auto before?
Second, prio 2 topic: my "dhw_circulation" is always "off" even it is also time controlled in the SensoApp. Do you know why?
Thanks for your feedback!

Hello,
You can change the temperature using water_heater.set_temperature service and change the operation mode with water_heater.set_operation_mode service (for time controlled, the value should be AUTO).
@xaviergriffon or @sylvainvh the integration doesn't display the target temp when time controlled is 'ON'. Maybe there is a bug in Function#active_mode ?
Also, circulation seems to always be off. I see we use
a_mode = self.active_mode
return (
    a_mode.current in (OperatingModes.ON, QuickModes.HOTWATER_BOOST)
    or a_mode.sub == SettingModes.ON
)

I think a_mode.sub is DAY (and not ON) in senso. I will provide a fix for this

Version https://github.com/thomasgermain/vaillant-component/releases/tag/1.14.2 should fix the circulation sensor issue

@thomasgermain fix for circulation is working perfect. Thanks!

Hello,
You can change the temperature using water_heater.set_temperature service and change the operation mode with water_heater.set_operation_mode service (for time controlled, the value should be AUTO).
@xaviergriffon or @sylvainvh the integration doesn't display the target temp when time controlled is 'ON'. Maybe there is a bug in Function#active_mode ? Also, circulation seems to always be off. I see we use
a_mode = self.active_mode
return (
    a_mode.current in (OperatingModes.ON, QuickModes.HOTWATER_BOOST)
    or a_mode.sub == SettingModes.ON
)

I think a_mode.sub is DAY (and not ON) in senso. I will provide a fix for this

A bit of a side question to this. What I want would love to do is the following:
I want to check if the sun will shine the next day (HA has a prediction for this). If it will shine, I want to make hot water during the day at peak sun hours, if not,  I want to make the hot water before 6am as we have cheaper prices then.
Is it possible to set the time of water heating in this integration?

Hello,
You can change the temperature using water_heater.set_temperature service and change the operation mode with water_heater.set_operation_mode service (for time controlled, the value should be AUTO).
@xaviergriffon or @sylvainvh the integration doesn't display the target temp when time controlled is 'ON'. Maybe there is a bug in Function#active_mode ? Also, circulation seems to always be off. I see we use
a_mode = self.active_mode
return (
    a_mode.current in (OperatingModes.ON, QuickModes.HOTWATER_BOOST)
    or a_mode.sub == SettingModes.ON
)

I think a_mode.sub is DAY (and not ON) in senso. I will provide a fix for this

Hello,
I've looked at the information returned by the API and the modes are indeed wrong (except for OFF).
The equivalent of ON is MANUAL and AUTO is TIME_CONTROLLED.
Extracts from the jsons :

 "hotwater": {
        "configuration": {
            "operation_mode": "MANUAL",
            "hotwater_temperature_setpoint": 53.0
        },

 "hotwater": {
        "configuration": {
            "operation_mode": "OFF",
            "hotwater_temperature_setpoint": 53.0
        },

 "hotwater": {
        "configuration": {
            "operation_mode": "TIME_CONTROLLED",
            "hotwater_temperature_setpoint": 53.0
        },

Release https://github.com/thomasgermain/vaillant-component/releases/tag/1.16.2 should diplsay correctly the target high temperature when the DHW is in `TIMED_CONTROLLED'


allow "value" as an alias for "name" in survey sheet

see discussion in odk-developers group

Why not add an extra column?
On Friday, December 11, 2015, dorey<EMAIL_ADDRESS>wrote:

—
Reply to this email directly or view it on GitHub
https://github.com/XLSForm/pyxform/issues/54.

For a little while, value was accepted as an alias of name which is clearer for the end user. But in this commit in 2013 we started checking against a more specific set of columns.
It's doing this to ensure that the sheets are not missing the top row with the column names, and so the user gets a clear error when they omit it, but this goes against the simplicity of the worksheet dict and has caused some minor bugs during testing. I'm looking into an alternative that avoids using "%s_header" items in the worksheet_dict.

closed by #55

I will edit this test to specifically test the use of value on the choices sheet , which is what the original discussion referred to.


missing or wrong info in JSON kflow represenation

Below is the JSON format of Kflows that are received from Kentik.

Field device_name is empty
There are no fields for the following enrichment information:

Port name (port = device interface)
Port description
Port connectivity type
Port network boundary

Some integer value is shown instead of route prefix:

Src_route_prefix
Dst_route_prefix

{
"timestamp":<PHONE_NUMBER>,
"dst_as": 65500,
"dst_geo": "DE",
"header_len": 0,
"in_bytes": 44,
"in_pkts": 1,
"input_port": 551,
"ip_size": 0,
"dst_addr": "<IP_ADDRESS>",
"src_addr": "<IP_ADDRESS>",
"l4_dst_port": 0,
"l4_src_port": 2048,
"output_port": 549,
"protocol": "ICMP",
"sampled_packet_size": 44,
"src_as": 8075,
"src_geo": "US",
"tcp_flags": 0,
"tos": 0,
"vlan_in": 0,
"vlan_out": 0,
"next_hop": "",
"mpls_type": 0,
"out_bytes": 0,
"out_pkts": 0,
"tcp_rx": 0,
"src_flow_tags": "",
"dst_flow_tags": "",
"sample_rate": 32,
"device_id": 1171,
"device_name": "",
"company_id": 1013,
"dst_bgp_as_path": "",
"dst_bgp_comm": "",
"src_bpg_as_path": "",
"src_bgp_comm": "",
"src_nexthop_as": 174,
"dst_nexthop_as": 65500,
"src_geo_region": "Iowa",
"dst_geo_region": "-",
"src_geo_city": "Des Moines",
"dst_geo_city": "-",
"dst_nexthop": "<IP_ADDRESS>",
"src_nexthop": "<IP_ADDRESS>",
"src_route_prefix":<PHONE_NUMBER>,
"dst_route_prefix":<PHONE_NUMBER>,
"src_second_asn": 8075,
"dst_second_asn": 0,
"src_third_asn": 0,
"dst_third_asn": 0,
"src_eth_mac": "00:00:00:00:00:00:00:00",
"dst_eth_mac": "00:00:00:00:00:00:00:00",
"input_int_desc": "",
"output_int_desc": "",
"input_int_alias": "",
"output_int_alias": "",
"input_int_capacity": 0,
"output_int_capacity": 0,
"input_int_ip": "",
"output_int_ip": "",
"custom_str": {
"1036": "",
"1037": "AzureCloud.centralus",
"src_as_name": "MICROSOFT-CORP-MSN-AS-BLOCK",
"dst_as_name": "-Private",
"1435": "inbound",
"395": "ICMP",
"1035": "azure",
"1038": ""
},

custom_bigint": {
"129": 0,
"130": 0
},
"eventType": "KFlow",
"provider": "kentik-router"
},

Good call out on Route Prefixes. Should be fixed in https://github.com/kentik/ktranslate/pull/52
Re device_name, Port name (port = device interface), Port description, Port connectivity type, Port network boundary these all get matched at query time not ingest time so aren't available in the firehose.

route prefixes do not have length, however I think this should be included:
        "src_route_prefix": "<IP_ADDRESS>",
        "dst_route_prefix": "<IP_ADDRESS>",

I posted the question on the slack channel about connectivity type and network boundary as from what I can see in portal UI, it seems like these are written in DB. I see that @kagawaa confirmed this is the case, so I guess these info can be included.
It might be worth if you can also include "device IP" address as I think that should be more useful than "device_id".
I understand that there is a technical limitation and that you probably try to make this efficient, so that you don't enrich everything, but if I put customer's hat, I would expect to have these info as well


Changing import extension version to 1.0 for GA

This PR fixes #

Pull Request Test Coverage Report for Build 244947053

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.046%

Totals

Change from base Build 244924658:
0.0%

Covered Lines:
19010

Relevant Lines:
44457

💛  - Coveralls


fix(overlay): re-use last preferred position when re-applying to open connected overlay

Currently when updating the position of an open connected overlay (e.g. when the user is scrolling) we go through the same process for determining the preferred position as when the overlay was attached. This means that the preferred position could change, causing the overlay to jump. With these changes the overlay will determine its preferred position when it's attached and the re-use the same position until it's detached. This is a prerequisite to moving the scroll clipping logic out of the ConnectedPositionStrategy and into the scroll strategies.
This PR is a resubmit of #5471.
Note: The positioning test failure seems to be coming from master and should be fixed by #7807.

I'm not sure about this- I've always considered it a feature that overlays can flip positions to stay on-screen when scrolling
(I even showed it off during my AngularMix talk)
What's your reasoning for wanting to change it?

My reasoning was that it could be distracting having the overlay "follow you" after you've decided to scroll away. Alternatively we can make this functionality opt-in and only enable it on the components where we think it makes sense (e.g. the select doesn't handle the repositioning very well because it also has to change the offsetY and the scroll position).

I'd be okay with it being a setting. Not sure on the name, though.

@jelbourn I've reworked it to make the functionality opt-in.


[srt_fix] post-processed 'xxx.mp4' with {}

Hi, I'm using the latest version of yt-dlp, and my options are (through .conf file):
-P "%DOWNLOADS%" -o "%(title)s.%(ext)s" -S ext --write-auto-sub --sub-lang en --convert-subs=srt --use-postprocessor srt_fix
But no fixed.srt file generated after run.
Could you help find the problem? Thanks.

Does the project even work?
It works for me, I would ask you to consider that I am not an experience developer.
If you could please provide me with a link to a video that causes the issues, that would be great.

Also please redownload the latest merge does include a fix for the -o option, maybe that is all you need.
the gui py indeed had a silly error, that should also be fixed now


Add an Adopters file

We should add an ADOPTERS.md file that lists all the adopters of kpack.
For an example see the CNB ADOPTERS.md file: https://github.com/buildpacks/community/blob/main/ADOPTERS.md

@tomkennedy513  are we fine with this PR? I think we need it merge before the donation


Merge conflict resolution strategy

What happens?

First guess:

Throws a proper exception if a merge conflict is detected, instead of just falling over on its face.
The exception should show a page telling the user a merge conflict was found.
It should show what the differences are in the files that are different.
It should let the user pick which one of the files they want to keep (whole files, not partials/single lines etc.).
Jekit can preview either version, to help them make a decision.

This should do for first parse merge conflict handling.


Fix must_contain() and must_not_contain()

The two functions, must_contain() and must_not_contain(), from the testing framework didn't work when searching a substring located at the beginning of the file (at offset 0).
Contributor Checklist:

[x] I have created a new test or updated the unit tests to cover the new/changed functionality.
[x] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[ ] I have updated the appropriate documentation

Coverage increased (+2.1%) to 76.033% when pulling ec2f065ab7ab954c08ab6cae2d1e8b0a210be401 on ptomulik:fix-must_contain into 8f2701fbd6a9d3894dd9918e32317e7be972912d on SCons:master.

Coverage increased (+2.1%) to 76.033% when pulling ec2f065ab7ab954c08ab6cae2d1e8b0a210be401 on ptomulik:fix-must_contain into 8f2701fbd6a9d3894dd9918e32317e7be972912d on SCons:master.

Ugh handling (None,-1) is ugly.
Do any of the tests actually pass in a find function?

There is a bunch of test_find methods in testing/framework/TestCommonTests.py. Also, some external tools can possibly use find argument.

I like the consolidation of the find logic! Nice improvement.


Add lambdacube-ir, lambdacube-gl, lambdacube-compiler

Travis says:
async-2.1.0 is out of bounds for:

[ ] lambdacube-compiler-<IP_ADDRESS> (==2.0.*). Csaba Hruska<EMAIL_ADDRESS>@csabahruska. @lambdacube3d

Can you release a version of lambdacube-compiler that is compatible with async-2.1?

done: lambdacube-compiler-<IP_ADDRESS>

Cool, thanks!


Update org.yaml - remove bobgy, james-jwu, k8s-ci-robot, rmgogogo from org members

Remove bobgy, james-jwu, k8s-ci-robot, rmgogogo from org members, as they were added to org admins
To fix github sync error: "Configuration failed: failed to configure kubeflow members: users in both roles: bobgy, james-jwu, k8s-ci-robot, rmgogogo"
/assign @Bobgy @james-jwu @rmgogogo

/lgtm
/approve


How come OidcClientSettings doesn't accept 'client_secret'?

#499 Why does class OidcClientSettings takes 'client_secret' but OidcClientSettings in typescript doesn't accept 'client_secret'? Don't I need client_secret?

Prolly because no one submitted a PR for the typings with it?

I'm bit confused...
Is 'client_secret' not supported on this library?


[Bug] Unable to access Headscale http server after installation

Is this a support request?

[X] This is not a support request

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
I've completed the installation of Headscale on Ubuntu 24.04 LTS. Headscale is being run as a systemd service and the server address is the default one provided in config: <IP_ADDRESS>:8080. I've ensured no other program is using that port. The logs also indicate there is no error. Trying to access the http server by curling on the local machine doesn't seem to be working.
2024-11-16T06:46:41Z WRN
WARN: The "dns.use_username_in_magic_dns" configuration key is deprecated and has been removed. Please see the changelog for more details.

2024-11-16T06:46:41Z INF Opening database database=sqlite3 path=/var/lib/headscale/db.sqlite
2024-11-16T06:46:41Z INF Setting up a DERPMap update worker frequency=86400000
2024-11-16T06:46:41Z INF listening and serving HTTP on: <IP_ADDRESS>:8080
2024-11-16T06:46:41Z INF listening and serving debug and metrics on: <IP_ADDRESS>:9090

Expected Behavior
Should be able to access Headscale http server of machine
Steps To Reproduce

In Ubunutu 24.04 LTS
Install Headscale server using official guide using .deb package
Curl <IP_ADDRESS>:800 on the machine.

Environment
- OS: Ubuntu 24.04
- Headscale version: v0.23.0
- Tailscale version:

Runtime environment

[ ] Headscale is behind a (reverse) proxy
[ ] Headscale runs in a container

Anything else?
No response

Can you give the output of these commands (run those on the same machine where headscale installed):

curl http://localhost:8080
sudo ss -tlpn | grep headscale
sudo ss -tlpn | grep 8080

If you want to reach headscale from the outside, you'll need to update the configuration to set at least:
server_url: http://your-headscale.exaple.com:8080
listen_addr: <IP_ADDRESS>:8080

I've also completely turned off my firewall btw to check if that was somehow the problem.

This is actually fine. Headscale does not provide a web interface by default. You can visit http://localhost:8080/windows which shows how to setup a windows client. Same with  http://localhost:8080/apple

Yeah that works lol. Wish someone would add this in the documentation though. It was not very clear at the end.


Formatting Bar renders when it shouldn't

Hi there.
I've created a block that sets formattable: false, and the results are not quite 100%.
When this is set, I lose the ability to use the keyboard to format, as expected, but the FormatBar still shows up and I am able to format the text.
This doesn't seem like the expect result?
Thanks, this is a great project!

Is this still an issue with 0.5.0?

:-(

Please :cry:

Why closed?


Use locale-aware uppercase functions

This PR is an attempt to solve the incorrect uppercase operations in EmulationStation for 2+ byte-wide characters, such as "ü" and "ç" in "Türkçe".
Current strToUpper helper functions in Util.cpp source file are very optimistic about the
locale. They assume that each character in a string is 1-byte wide, thus fails to make 2+ byte-wide characters uppercase.
Please see the images below:
This one is from the current state of EmulationStation on Turkish locale. See the lowercase letters. Even though the translation is correct, strToUpper is failed for specific characters.

This is from the applied patch with the correct behaviour:

Not a random game you choose there... Thans for the PR :)
I see with @nadenislamarre if it's ok with its current work and i merge that quicly. Thanks

thanks a lot, this was my next todo entry while we get the same issue in french ;-)
i'll check it today to confirm it's ok.

I merge, we will see in the nightly build if all is ok.


Add customImagesYaml docstring

@aronatkins Unfortunately, we have to bump the chart version anytime we modify the values file

I'm OK either way -- there's a slim chance that the release including these changes is not 2023.04.0, but we can reconcile if that ends up being the case. Your call.


One particular  resource always times out on GET

Here is a problem which has not gone away — I can never raed https://solidos.solidcommunity.net/Team/SolidOs team chat/2020/11/06/chat.ttl
as it times out
with browser or with curl

Error: Fetcher: Request to https://solidos.solidcommunity.net/Team/SolidOs%20team%20chat/2020/11/06/chat.ttl timed out

hmmm maybe others time out   like https://solidos.solid.community/Team/docs/solidos.html  or maybe the server crashes and goes down for a while

Should be closed. It was an ACL problem due to migration.


Add pipelines notebook

Adds a notebook example to show the use of ML pipelines for node classification. At the moment the corresponding doc entry is not added.

Nice work! The style of the notebook is a bit different than the others in that it hasn't cleaned up results for example. Would be nice to have a holistic take on this at some point

Thanks @adamnsch and @FlorentinD, I am going through your suggestions and comments now. As a general comment on the style, in this version I'm keeping some of the output that might play nicely as example output in the docs, along with some assertions that may assist in testing, so we can try both and see how they play out.

I am on vacation until next year, so I hope @adamnsch or @Mats-SX can approve the PR


[opt] I don't think it's necessary to copy the static resources by manually, right?

[v1.1.1]
fixed


[BUG] E2E mortgage notebook breaks on Pascal and Rapids 0.6 & 0.7

Running the mortgage notebook:
https://github.com/rapidsai/notebooks-extended/blob/master/advanced/E2E/mortgage/mortgage_e2e.ipynb
On NGC containers:
0.7-cuda10.0-devel-ubuntu18.04-gcc7-py3.6
and
0.6-cuda10.0-devel-ubuntu18.04-gcc7-py3.6
On dual P6000 cards, it can only do 2 parts of the data. However, on Rapids 0.5.1 I can run up to 6 parts of the data.
I have not noticed this issue when I ran it on dual RTX8000.
Note, that I also have a P400 with the dual P6000. On Rapids 0.5.1, it will only detect the P6000s, but on Rapids 0.6 and Rapids 0.7 it would detect all three cards (This also result in workers dying and restarting during ETL).

If you're using a container you disable the P400 card. This is causing the issue. Please use GPU isolation https://github.com/NVIDIA/nvidia-docker/wiki/GPU-isolation-(version-1.0) and let us know if it solves the problem.

@datametrician Thank you. I've verified that this works for Rapids 0.7.
Even though it works, it does however give the following error in the terminal:
/data/acq/Acquisition_2015Q1.txt
/data/acq/Acquisition_2015Q2.txt
/data/perf/Performance_2015Q1.txt
/data/perf/Performance_2015Q2.txt
/data/acq/Acquisition_2015Q4.txt
/data/acq/Acquisition_2015Q3.txt
/data/perf/Performance_2015Q3.txt
/data/perf/Performance_2015Q4.txt
tornado.application - ERROR - Exception in callback <bound method SystemMonitor.update of <SystemMonitor: cpu: 104 memory: 4523 MB fds: 49>>
Traceback (most recent call last):
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_common.py", line 342, in wrapper
    ret = self._cache[fun]
KeyError: <function Process._parse_stat_file at 0x7fe18eea69d8>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_pslinux.py", line 1554, in _parse_stat_file
    data = f.read()
OSError: [Errno 9] Bad file descriptor

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/conda/envs/rapids/lib/python3.6/site-packages/tornado/ioloop.py", line 907, in _run
    return self.callback()
  File "/conda/envs/rapids/lib/python3.6/site-packages/distributed/system_monitor.py", line 48, in update
    cpu = self.proc.cpu_percent()
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/__init__.py", line 1111, in cpu_percent
    pt2 = self._proc.cpu_times()
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_pslinux.py", line 1507, in wrapper
    return fun(self, *args, **kwargs)
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_pslinux.py", line 1699, in cpu_times
    values = self._parse_stat_file()
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_pslinux.py", line 1507, in wrapper
    return fun(self, *args, **kwargs)
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_common.py", line 349, in wrapper
    ret = self._cache[fun] = fun(self)
  File "/conda/envs/rapids/lib/python3.6/site-packages/psutil/_pslinux.py", line 1554, in _parse_stat_file
    data = f.read()
OSError: [Errno 9] Bad file descriptor
/data/acq/Acquisition_2016Q2.txt
/data/perf/Performance_2016Q2.txt
/data/acq/Acquisition_2016Q1.txt
/data/perf/Performance_2016Q1.txt


time property should not trigger /import endpoint

In a server-side implementation of the Mixpanel API, it's reasonable to want to use the client's timestamp (rather than an implicit "now") for the time property. But in the current implementation, that causes the /import endpoint to be used, with the consequence that the event doesn't appear in the Mixpanel Live View.
One alternative: add an endpoint argument to track() (so that import() can use it to force the /import endpoint) and change the time-property test to look for a timestamp older than Mixpanel's 5-day limit for /track.

I think this line should just be changed:
https://github.com/mixpanel/mixpanel-node/blob/master/lib/mixpanel-node.js#L120
So that it automatically defaults to /track if the time is less than or equal to 5 days. If someone is sending the track event, they inherently do not want it to be classified as an import.


Update README.md

I have tried to make the directions on opening the docs a little more succinct, based on the discussion with @garrison and @caleb-johnson.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Follow up to https://github.com/Qiskit-Extensions/circuit-knitting-toolbox/commit/33371bcfd2ea7ec777a68a3d96a3a7ccabfab35a#r122869186.


BUG: fixed a dtype bug in linalg.solve.

main change: Bugfix: take care of the mismatch between assume_a='her' and a real a matrix in linalg.solve
other changes:

a small pep8 fix in linalg/basic.py
updated+added unit tests for linalg/basic.py

Reference issue
Closes #14055
What does this implement/fix?
before, using assume_a='her' with a being a real symmetric matrix would cause scipy to look up for a lapack function that doesn't exist, leading to an obscure error.
now, assume_a is silently converted to 'sym' (equivalent assumption) when that happens.
misc. notes:

relevant tests passed locally
pep8 checked
QUESTION: does this bugfix need to be mentioned in solve's documentation?

Thanks for the quick PR @liam-o-marsh. Looks good. The remaining part is checking whether RHS is complex and casting a possibly to complex or complex64. I'll try to come back to this later today

I'm not completely sure what the goal of this conversion is, so I need to ask if this code would correspond to what you have in mind.
(I made the assumption that the solver for hermitian matrices is better in some way, so that explains parts of the code)
    # if `a` is given as a real matrix, there is no difference
    # between "symmetric" and "hermitian". Also, knowing that:
    # - [check before commit] the hermitian algorithm is generally better
    # - lapack doesn't have the right functions for "real hermitian matrices"
    # we can choose what the better assumption is by looking at b
    if not np.iscomplexobj(a1) and assume_a in ('sym','her'):
        if np.iscomplexobj(b1):
            assume_a = 'her'
            new_dtype =	{
                'f': np.complex64, 'd':np.complex128, 'g':np.complex256
            }[b1.dtype.char]
            a1_pre = a1
            a1 = np.asarray(a1, dtype=newdtype)
            # [TODO before commit] choose one of the following two lines
            overwrite_a = True
            overwrite_a =  overwrite_a or _datacopied(a1, a1_pre)
        else:
            assume_a = 'sym'

If that's correct, I'll run tests and commit

Actually I spoke too soon and we are lucky in this case since csysv and zsysv is already wrapped. So the complex b part is covered in the current code. These routines handle symmetric but not hermitian complex matrices hence in the assume_a=sym branch it chooses the correct flavor.

…OK?
but, IIRC, in #12824, the hermitian solver is "preferred" to the symmetric solver. Is there a reason to this?
If so, isn't it be better to use the assume_a='her' instead of assume_a='sym' if a needs to be converted into a complex matrix at some point anyway?

Can't say I follow but there is no preference. Sym and Her solvers are the same just the data type is different. So actually this fix is sufficient. So we are all good and waiting for the CI to pass.

oh OK.
thanks for helping me do this PR!

Pleasure is mine.  Thanks again.


I am installing rasa nlu but showing error during building wheel for spacy

Thanks for the question, our Github issue board is for bugs and feature requests only, though 😕
Please ask this question in the forum✨
If you were describing a bug, please create a report and fill out the template


[Question] How to use the writer-opts?

I'm trying out using conventional-changelog with the  --writer-opts however I can't get the host url to change for my commits. I'm running like so
conventional-changelog -p angular -r 1  --writer-opts ./src/conf/conventional-changelog-writer.js
Where conventional-changelog-writer.js is
module.exports = {

    host : "https://github.com"

};

The output i'm getting is

## [0.2.51](/http://1022/test/compare/v0.2.51...v0.2.51) (2015-11-28)

### Bug Fixes

* boo ([5981e03](http://1022/test/commits/5981e03))

### Features

* **package.json:** Hey look at me ([14b8548](http://1022/test/commits/14b8548))

as you can see the links to the commits are going to 'http://1022/'

host option should be under context https://github.com/stevemao/conventional-changelog-writer#host

So doing
module.exports = {
	context: {
		host: "https://github.com"
	}
};

still yields the same results

I will change the cli flags. for now, use https://github.com/ajoslin/conventional-changelog/blob/master/cli.js#L28

@stevemao
Cool that kind of works I did
conventional-changelog -p angular -r 1 --context ./src/conf/conventional-changelog-writer.js

With the conventional-changelog-writer.js as
module.exports = {

    host : "https://github.com"

};

And it added the url however it didn't replace the "http://1022" part of it

## [0.2.51](https://github.com/http://1022/test/compare/v0.2.51...v0.2.51) (2015-11-28)

### Bug Fixes

* boo ([5981e03](https://github.com/http://1022/test/commits/5981e03))

### Features

* **package.json:** Hey look at me ([14b8548](https://github.com/http://1022/test/commits/14b8548))

where is 'http://1022/' from?

from the package.json ( removed our internal host we have it on)  it looks like
 "repository": {
    "type": "git",
    "url": "ssh://<internal host>:10022/test.git"
  },

So it looks like adding repository will overwrite the whole thing correctly however ideally I would like it scrap the repository from the package.json as I was going to have other packages pull this custom grunt task so that it works out of the box.
module.exports = {

		host: 'https://github.com',
        repository: 'angelblade27'
};

## [0.2.51](https://github.com/angelblade27/compare/v0.2.51...v0.2.51) (2015-11-28)

### Bug Fixes

* boo ([5981e03](https://github.com/angelblade27/commits/5981e03))

### Features

* **package.json:** Hey look at me ([14b8548](https://github.com/angelblade27/commits/14b8548))

ideally I would like it scrap the repository from the package.json

repository defaults to the parsed result from the repo url in package.json. You can overwrite the default. Does that answer your question?

Kind of.
So if the url is
ssh://localhost:1022/test.git why does it still append http://1022 when I only have 'host' set?
Shouldn't it just automatically pull 'test' as the repository and not have 'http' + port number along with the ?
Instead of hardcoding the repository name can I override it with a regex to parse the repository url correctly?

Shouldn't it just automatically pull 'test' as the repository and not have 'http' + port number along with the ?

Can you check if getPkgRepo module can parse your url correctly and the context object (https://github.com/ajoslin/conventional-changelog/blob/master/index.js#L126) contains the right data
'

Instead of hardcoding the repository name can I override it with a regex to parse the repository url correctly?

Not really.

@angelblade27 any news?

Sorry was sick and then forgot to get to this.
Is that what you mean I dumped some of those parameters by doing
           context.owner = context.owner || repo.user || '';
            context.repository = context.repository || repo.project || browse;
            console.log('repository:' + context.repository);
            console.log('repo.project:' + repo.project);
            console.log('browse:' + browse);

Output was
repository:<our project>/test
repo.project:undefined
browse:http://10022/test

Can you use the same settings that you can produce https://github.com/http://1022/test/commits/5981e03?

Ok removed the repository.  Just to clarify the command is
node node_modules/conventional-changelog/cli.js -p angular -r 1 --context ./src/conf/conventional-changelog-writer.js. I wrote a grunt target to execute this.
The conventional-changelog-writer.js has
module.exports = {
	//Override the host url that gets scraped from the package.json to link to our git lab
	host: 'https://gitlab-app.eng.qops.net',
	//When overriding the host we need to override the repository as well.
	//repository: 'testcore'
};

package.json url is
  "repository": {
    "type": "git",
    "url": "ssh://<internal host>:10022/testcore.git"
  },

output is
repository:http://10022/testcore
repo.project:undefined
browse:http://10022/testcore

It looks like getPkgRepo module cannot understand your repo url.

Is it because we have a port number there?
Sent from my iPhone
~S

On Dec 13, 2015, at 2:46 PM, Steve Mao<EMAIL_ADDRESS>wrote:
It looks like getPkgRepo module cannot understand your repo url.
—
Reply to this email directly or view it on GitHub.

Maybe. Also because your host is a customised host so the module doesn't understand it. Can you print the results parsed by getPkgRepo so the problem is clearer? Thanks.

output of getPkgRepo is
{ protocol: '<internal host>:',
  slashes: null,
  auth: null,
  host: '10022',
  port: null,
  hostname: '10022',
  hash: null,
  search: null,
  query: null,
  pathname: '/test.git',
  path: '/test.git',
  href: '<internal host>:10022/test.git',
  browse: [Function] }

I noticed you mention https://github.com/ajoslin/conventional-changelog#transform however I don't see what arg to use to pass this option via commandline also not sure exactly how to use it. It mentions it takes package.json as arg and transforms it but how do you get it transform it.  If there is a code example I can look at that.

https://github.com/ajoslin/conventional-changelog/blob/master/test/test.js#L171-L174

I tried parsing 'ssh://abc:10022/testcore.git' (I replaced you <internal host> with 'abc') and it returns me
Url {
  protocol: 'ssh:',
  slashes: true,
  auth: null,
  host: 'abc:10022',
  port: '10022',
  hostname: 'abc',
  hash: null,
  search: null,
  query: null,
  pathname: '/testcore.git',
  path: '/testcore.git',
  href: 'ssh://abc:10022/testcore.git' }

That is weird which version are you using or is it directly from the code just wondering if that might be?
Sent from my iPhone
~S

On Dec 13, 2015, at 10:40 PM, Steve Mao<EMAIL_ADDRESS>wrote:
I tried parsing 'ssh://abc:10022/testcore.git' (I replaced you  with 'abc') and it returns me
Url {
protocol: 'ssh:',
slashes: true,
auth: null,
host: 'abc:10022',
port: '10022',
hostname: 'abc',
hash: null,
search: null,
query: null,
pathname: '/testcore.git',
path: '/testcore.git',
href: 'ssh://abc:10022/testcore.git' }
—
Reply to this email directly or view it on GitHub.

I use node4. I briefly tested with getPkgRepo directly. Same as url.parse.
Sent from my iPhone

On 14 Dec 2015, at 6:05 PM, angelblade27<EMAIL_ADDRESS>wrote:
That is weird which version are you using or is it directly from the code just wondering if that might be?
Sent from my iPhone
~S

On Dec 13, 2015, at 10:40 PM, Steve Mao<EMAIL_ADDRESS>wrote:
I tried parsing 'ssh://abc:10022/testcore.git' (I replaced you  with 'abc') and it returns me
Url {
protocol: 'ssh:',
slashes: true,
auth: null,
host: 'abc:10022',
port: '10022',
hostname: 'abc',
hash: null,
search: null,
query: null,
pathname: '/testcore.git',
path: '/testcore.git',
href: 'ssh://abc:10022/testcore.git' }
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub.

Is your problem solved? @angelblade27

I think it is the node version we are on an very old version


[WIP] investgate github action for PR integration test

WIP. DO NOT MERGE

Codecov Report

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

@@             Coverage Diff              @@
##             master    #5164      +/-   ##
============================================
- Coverage     64.19%   64.17%   -0.02%
+ Complexity      463      461       -2
============================================
  Files           769      769
  Lines         33255    33255
  Branches       5239     5239
============================================
- Hits          21347    21342       -5
+ Misses         9485     9484       -1
- Partials       2423     2429       +6

Impacted Files
Coverage Δ
Complexity Δ

...ache/dubbo/remoting/transport/AbstractChannel.java
75% <0%> (-12.5%)
0% <0%> (ø)

.../apache/dubbo/rpc/protocol/AsyncToSyncInvoker.java
79.16% <0%> (-8.34%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

...he/dubbo/remoting/transport/netty/NettyServer.java
69.64% <0%> (-3.58%)
8% <0%> (-1%)

...g/apache/dubbo/rpc/protocol/rest/RestProtocol.java
68.21% <0%> (-3.11%)
0% <0%> (ø)

...a/org/apache/dubbo/monitor/dubbo/DubboMonitor.java
87.85% <0%> (-2.81%)
16% <0%> (-1%)

.../apache/dubbo/remoting/transport/AbstractPeer.java
63.04% <0%> (-2.18%)
0% <0%> (ø)

...pache/dubbo/remoting/transport/AbstractServer.java
47.91% <0%> (-1.05%)
0% <0%> (ø)

...rg/apache/dubbo/common/timer/HashedWheelTimer.java
62.41% <0%> (-0.35%)
0% <0%> (ø)

... and 3 more

Continue to review full report at Codecov.

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

Is this PR still need to be opened?


Switch from Typheous to Faraday

Description
Hi! I see this gem uses Typheous as an HTTP library. Unfortunately, Typheous is poorly-maintained (no commits in two years!) and often segfaults on Apple Silicon when combined with forking processes (a very common practice in Ruby). As this is a library, my recommendation is to switch to Faraday which allows users to switch the underlying HTTP library to something which doesn't segfault.
Noticed because I started having this happen more locally again (I cannot consistently reproduce, unfortunately):
/Users/emma/.asdf/installs/ruby/3.2.5/lib/ruby/gems/3.2.0/gems/ethon-0.16.0/lib/ethon/curl.rb:61: [BUG] Segmentation fault at 0x0000000000000110
ruby 3.2.5 (2024-07-26 revision 31d0f1a2e7) [arm64-darwin23]

-- Crash Report log information --------------------------------------------
   See Crash Report log file in one of the following locations:
     * ~/Library/Logs/DiagnosticReports
     * /Library/Logs/DiagnosticReports
   for more details.
Don't forget to include the above Crash Report log file in bug reports.

Thank you for bringing this up. We'll be working on switching to Faraday shortly.

@NuckChorris We've now switched to Faraday in v3.0.0.rc1. Could you give it a shot and let me know how it goes?


KeyError: 'l1EthDefaultBridge'

Hello?
File "main.py", line 31, in
deposit()
File "main.py", line 20, in deposit
eth_provider = EthereumProvider.build_ethereum_provider(zksync=zksync_web3,
File "/usr/local/lib/python3.8/dist-packages/zksync2/provider/eth_provider.py", line 36, in build_ethereum_provider
bridge_contracts: BridgeAddresses = zksync.zksync.zks_get_bridge_contracts()
File "/usr/local/lib/python3.8/dist-packages/zksync2/module/zksync_module.py", line 271, in zks_get_bridge_contracts
return self._zks_get_bridge_contracts()
File "/usr/local/lib/python3.8/dist-packages/web3/module.py", line 61, in caller
return apply_result_formatters(result_formatters, result)
File "cytoolz/functoolz.pyx", line 249, in cytoolz.functoolz.curry.call
File "/usr/local/lib/python3.8/dist-packages/web3/module.py", line 38, in apply_result_formatters
formatted_result = pipe(result, result_formatters)
File "cytoolz/functoolz.pyx", line 666, in cytoolz.functoolz.pipe
File "cytoolz/functoolz.pyx", line 641, in cytoolz.functoolz.c_pipe
File "cytoolz/functoolz.pyx", line 502, in cytoolz.functoolz.Compose.call
File "/usr/local/lib/python3.8/dist-packages/zksync2/module/zksync_module.py", line 105, in to_bridge_address
return BridgeAddresses(l1_eth_default_bridge=HexStr(to_checksum_address(t["l1EthDefaultBridge"])),
KeyError: 'l1EthDefaultBridge'
When will you fix it?

When will you fix it?

I have the same error.
Python3.9
web3               5.30.0
zksync2            0.2.0

same error https://github.com/zksync-sdk/zksync2-python/issues/6

Hello. There are breaking changes. Please use new release

Hello. There are breaking changes. Please use new release

It doesn't install on Windows!
The error is caused by the pysha3 library

It has been tested under Linux based OS. I have to investigate the Windows NT

This is the latest checks under Windows 10 OS:
https://github.com/zksync-sdk/zksync2-python/issues/3#issuecomment-1499090601

I  encountered pysha3 library error too, my solution is downgrad Python from 3.11 to 3.10 and it works, hope this was helpful for you.


optimize: Only global transaction with the status of begin is read for the timeout check task.

Ⅰ. Describe what this PR did
优化全局事务超时处理功能。只读取状态为Begin的全局事务数据。
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #2792 into develop will decrease coverage by 0.14%.
The diff coverage is 40.00%.

@@              Coverage Diff              @@
##             develop    #2792      +/-   ##
=============================================
- Coverage      50.47%   50.32%   -0.15%
- Complexity      2919     2923       +4
=============================================
  Files            572      572
  Lines          18616    18614       -2
  Branches        2245     2211      -34
=============================================
- Hits            9396     9368      -28
+ Misses          8288     8286       -2
- Partials         932      960      +28

Impacted Files
Coverage Δ
Complexity Δ

...java/io/seata/server/session/SessionCondition.java
50.00% <ø> (+3.33%)
7.00 <0.00> (ø)

...ver/storage/db/session/DataBaseSessionManager.java
35.00% <0.00%> (-1.21%)
12.00 <0.00> (ø)

...ver/storage/redis/session/RedisSessionManager.java
28.33% <0.00%> (-0.98%)
8.00 <0.00> (ø)

...o/seata/server/coordinator/DefaultCoordinator.java
52.28% <14.28%> (-2.36%)
26.00 <4.00> (-2.00)

...in/java/io/seata/server/session/SessionHolder.java
48.97% <33.33%> (-2.14%)
19.00 <1.00> (+1.00)
:arrow_down:

...erver/storage/file/session/FileSessionManager.java
63.35% <80.00%> (+1.69%)
32.00 <2.00> (+5.00)

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
3.00% <0.00%> (ø%)

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
3.00% <0.00%> (ø%)

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
3.00% <0.00%> (ø%)

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
3.00% <0.00%> (ø%)

... and 28 more

这个PR我先关闭掉，感觉这样改反而增加了SessionManager的代码复杂度，我后面再考虑整改方案。


save_hyperparameters does not save the selected learning_rate when auto_lr_find is used

Bug description
I am running my trainer with the auto_lr_find option, but the model's saved learning_rate reflects the original learning rate and not the final one selected by trainer.tune(). This is because save_hyperparameters() needs to be called at model construction, however it then cannot save the final selected learning rate. This feels like a shortcoming - is there a recommended workaround here? I didn't see one in the docs. Thanks!
How to reproduce the bug
No response
Error messages and logs
# Error messages and logs here please

Environment

Current environment
#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0):
#- Lightning App Version (e.g., 0.5.2):
#- PyTorch Version (e.g., 1.10):
#- Python version (e.g., 3.9):
#- OS (e.g., Linux):
#- CUDA/cuDNN version:
#- GPU models and configuration:
#- How you installed Lightning(`conda`, `pip`, source):
#- Running environment of LightningApp (e.g. local, cloud):

More info
No response
cc @awaelchli @borda @Blaizzy

Was there a solution for this?

Was there a solution for this?

Hi @Palzer I am using wandb (but I think you could apply similar code for other loggers). Here is the solution I came up with.
`def run_lr_finder(cfg, trainer, model, data_module, wandb_logger):
# Implement auto lr finder
tuner = Tuner(trainer)
model.hparams['lr'] = cfg['lr']  # Ensure model has lr hyperparameter so it can be updated
lr_finder = tuner.lr_find(model, datamodule=data_module)
# Only log new lr on main device (0)
if torch.cuda.current_device() == 0:
new_lr = lr_finder.suggestion()
print(f"LR finder identified:\n{new_lr}\nas candidate learning rate")
fig = lr_finder.plot(suggest=True)
plot_fpath = "temp/lr_finder_plot.jpg"
plt.savefig(plot_fpath)
fig.show()
    # Save plot as a artifact in wandb
    if wandb_logger is False:
        pass
    else:
        wandb_logger.log_image(key="lr_finder_plot", images=[plot_fpath])

        # Set new lr hp in wandb config
        wandb.config.update({"lr": new_lr}, allow_val_change=True)`

This still doesn't update automatically when using comet logger. I have learning_rate parameter in the Module definition, it gets updated in the object, but comet still shows the initial learning rate that was passed when creating that object.
# Class definition:
class RGB_Main_Module(L.LightningModule):
    def __init__(self, model, learning_rate):
        ...
        self.save_hyperparameters(ignore=['model'])
    ....

# creating an object
m = RGB_Main_Module(model=model, learning_rate = 0.0001)
tuner = Tuner(trainer=trainer)
lr_finder = tuner.lr_find(model=m, datamodule=data_module, num_training=1000)

Should I use save_hyperparmeters somewhere else or can i use some other command to update the learning_rate value on comet? Currently I'm just logging the new learning_rate explicitly under another name.


SignInAsync verifyHashpassword

I try to login with Admin user from seeduser information but i get this error :
Curl request details:
curl -X POST "https://localhost:6001/api/Auth/Login" -H "accept: application/json" -H "Content-Type: application/json" -d "{ \"userName\": \"Admin\", \"password\": \"123@456#Pass\"}"

at this line

In development environment, password is 123456.


Add device specific settings on all tabs

Problem Description
I'm converting an old plugin to a VB extension and decided to add an additional field on the content tab that would control the width of a line. I want to be able to set this width for different devices. My code for the field was as follows:
'horizontal_line_width' => array(
                'label'             => esc_html__('Horizontal Line Width', 'diti-divi-timeline'),
                'type'              => 'range',
                'option_category'   => 'layout',
                'description'       => esc_html__('Here you can set a width for the Horizontal Line.', 'diti-divi-timeline'),
                'toggle_slug'       => 'main_settings',
                'mobile_options'    => true,
                'responsive'        => true,
                'default'           => '3px',
                'default_unit'      => 'px',
                'range_settings'    => array(
                    'min'   => '1',
                    'max'   => '20',
                    'step'  => '1',
                ),
            ),

I had defined the 'main_settings' toggle on the 'general' tab and all options list nicely. However, upon changing the value, no class is added to pop up the other device options tabs. After a bit of struggle I changed the took out the 'toggle_slug' and added in 'tab_slug' => 'advanced', Now it works fine, it is just on a different tab than the rest of the controls. It would be nice if it was possible to add mobile responsive fields to bith the general and advanced tabs.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.


...

Hello I am trying to include a statistically using dangle in a dynamically popup someone could tell me how I can do this myself I added <fs: date-histo .... content in my popup the statistics only appear when you click to display the popup and only appear with the previously added elements present directly in html ...

I even found the solution using the $ compile as follows for those who need it.
var popup = L.popup()
.setContent($compile(template)($scope)[0]);
marker.bindPopup(popup);


Balls move on scroll

Maybe also have a gradient change onScroll

We have to remember to keep the performance in mind for when working on this issue: https://developer.chrome.com/docs/lighthouse/performance/non-composited-animations/?utm_source=lighthouse&utm_medium=lr


feat(auth): add idToken on signIn w/ Google response

should allow to propagate the token to the web view layer and sign-in within both tiers (native and web)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation  changes
[ ] Other... Please describe:

PR Checklist
Please check your PR fulfills the following requirements:
Bugfixes:

[x] Regression testing has been carried out using the example project to ensure the intended bug is fixed and no regression bugs have been inadvertently introduced.

New features/enhancements:

[x] Exhaustive testing has been carried out for the new functionality
[x] Regression testing has been carried out to ensure no existing functionality is adversely affected
[x] Documentation has been added / updated
[ ] The example project has been update to validate/demonstrate the new functionality.

What is the purpose of this PR?

It's a common case to authenticate on native layer and propagate idToken to the web view layer. After the sign-in process completes, the user will be signed in within both tiers, and we can use web specific sdks, as AngularFire for example, to handle Firebase account.
It's widely used on Cordova/Capacitor communities, as exposed by Dayana Jabif (ionicThemes) on ionic forum

I used baumblatt/capacitor-firebase-auth to perform native login with Google, Facebook and Twitter and it works like a charm!
The plugin handles the authentication on Native layer and propagate the token to the web view layer. After the sign-in process completes, the user will be signed in within both tiers.

On medium: Implement Google login in Ionic 5 apps using Firebase, the author uses EddyVerbruggen/cordova-plugin-googleplus, this plugin returns idToken on signIn response
Similar approach has been shared on Simon Grimm (ionicacademy): How to use Sign In with Apple inside Ionic for Firebase Authentication, using rlfrahm/capacitor-apple-login plugin, native signInWithApple returns the idToken and is used on JS layer to signInWithCredential.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

What testing has been done on the changes in the PR?

I've updated own corporate project with this PR on both platforms, iOS and Android, works fine!
What testing has been done on existing functionality?

Other information
If this approach is ok, I can I fix https://github.com/dpa99c/cordova-plugin-firebasex/issues/551 related to sign-in w/ Apple, following same logic.

@vdias38, your merge was perfect, and I use your implementation on my app. Since most of us (I guess) publish apps on both the Apple Store and the Play Store, having the same feature for sign-in w/ Apple would be wonderful (and as you said already, it would fix this issue https://github.com/dpa99c/cordova-plugin-firebasex/issues/551)

Hi @TheNotorius0, I don't think the it's big deal to add it for iOS. But at the moment I've no time to implement it. I'm surprised nobody added it yet. I'll track it and if I can find a day free on next weeks I'll try to do it.


TypeError: 'NoneType' object is not iterable

Apart from the warning-message, there is no obvious problems... According to this bugreport, this shouldn't be a problem though...
martin@felicitas:~/Desktop/EMVConsole$ python main.py
Failed to load symbol for: SCardCancelTransaction, /lib/x86_64-linux-gnu/libpcsclite.so.1: undefined symbol: SCardCancelTransaction!
[ ] Waiting for card...
[+] Found card:  3B 6E 00 00 80 31 80 66 B0 84 0C 01 6E 01 83 00 90 00
6F 1E 84 0E 31 50 41 59 2E 53 59 53 2E 44 44 46 30 31 A5 0C 88 01 01 5F 2D 02 66 72 9F 11 01 01
[!] First PSE Record at:  01
SFI:  [12]
[ ] 1. INTERAC (A0 00 00 02 77 10 10)
[ ] 2.  (43 43 41 54 32)
[?] Which application? 1
 Pin tries left: 3
Traceback (most recent call last):
  File "main.py", line 56, in <module>
    afl = func.getAFL([0x83, 0x04] + pdol_enc)
  File "/home/martin/Desktop/EMVConsole/emvfunctions.py", line 147, in getAFL
    response, sw1, sw2 = self.getProcessingOptions(data)
TypeError: 'NoneType' object is not iterable

This problem occurs with multiple cards I tried - Interac, Master, Visa, Maestro, Vpay. The error persists, even when there is only one application on the card and the "[?] Which application?"-prompt is not displayed.
pyscard version: <IP_ADDRESS>

Yeah, I have confirmed that the undefined symbol error seemingly doesn't cause any problems here.
The issue is likely to be in the getAFL response parsing code. I noticed a similar issue there a few weeks ago too. All the code

Good job, i had same issue too, looking forward for the patch =)

@kelvintx3 Don't expect any patch from me. I don't spend any time on this project anymore, and haven't really for years. I'd happily accept a pull request though.


Add button to copy  the link from field "repository url"

Hello, it is hard to copy the link  (like CTRL+C) from field "repository url" in a repository.
Then can you add a button to easily do it ?
A good js lib to do it is : https://clipboardjs.com/

Hello, it is hard to copy the link (like CTRL+C) from field "repository url" in a repository. Then can you add a button to easily do it ? A good js lib to do it is : https://clipboardjs.com/

Thanks for the idea of clipboard.js.
I have implemented it using nginx.
# nginx.conf
location / {
	proxy_pass	mygitblit;
	# add clipboard.min.js to html and init class cpbtn
	sub_filter   ''   '   ';
	# replace CTRL+C btn with new btn
	subs_filter  ""    '<button class="cpbtn" data-clipboard-text="$1"></button>' r;
}

This works for my gitblit GO 1.9.3 and nginx 1.18 configured with ngx_http_substitutions_filter_module.
I don't think it's difficult to add it to the web app via java.

Yes, it is very annoying. This is because it based on a very old technique using SWF.
There is even an issue to replace this with normal JS: #1241
I don't know much about frontend, so it will take a little while until I get this done. If anyone knows this better than me, PRs are welcome. Thank you, tammst, for your comment, it will probably be very helpful.
In the meantime, if you are not running a nginx reverse proxy in front of Gitblit or simply do not want to resort to filtering the HTML, you can switch Gitblit back to use an old system, where it opens an extra dialog. This is somewhat ancient, but at least it is not as annoying as the default behaviour.
To do so, set the following in your gitblit.properties file:
web.allowFlashCopyToClipboard = false

Duplicate of #1241

I would like to propose a patch, for the copy in the cliboard in automatic on the click of the text.
How can I do it ? I indicate where to modify the sources (here), or ...
I can propose something more elaborate with a copy icon on the right of the text, but it affects more files than the first solution

Closing as duplicate after issue 1241 has an implementation.


Using codesearch for my own queries. (How to instanciate model.py Class)

Hello I managed to finetune codebert for the codesearch task, and I was wondering how I could use the model.bin I just created to perform codesearch on my own natural langage queries.
So the model.bin is a state dictionnary containing all the weight but I don't get how I am supposed to go from that to a working model.
I tried to instanciate it with
model = RobertaModel.from_pretrained("microsoft/codebert-base")
but it returns the following error
RuntimeError: Error(s) in loading state_dict for RobertaModel:
Missing key(s) in state_dict: "embeddings.position_ids", "embeddings.word_embeddings.weight", "embeddings.position_embeddings.weight", "

etc,etc
And I noticed that there is a Model class in model.py so it is probably the one I mused use but I don't get how am I supposed to instanciate it in order for the model to work ? Can you show me an example or explain me a little bit ?
Thank you very much

And if the instanciate is too complicated, how can I use the inference.py file to get my predictions in a file? Like I give the model a string of the query and its output me the code predicted

As discussed in e-mail, you can refer to this issue

As discussed in e-mail, you can refer to this issue

Thank you very much for the fast answer.


pull request for e2e test at 01/30/2017 17:15:13

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


fixing issue#572

Description
fixing issue #572 with js-ng current workflow, the problem was setup-python github action sets a new python env for the current user, while using 'sudo' with pip makes poetry get installed for the global, system-wide Python installation, which in this case, was 3.6.
verifying that the issue solved from the actions job log:
Run poetry install
  poetry install
  shell: /bin/bash -e {0}
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.8.7/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.8.7/x64/lib
Creating virtualenv js-ng-Aq1zHSlJ-py3.8 in /home/runner/.cache/pypoetry/virtualenvs

Changes
in this modified workflow:

now poetry install will pick the correct Python environment sets by the GitHub action matrix
disabling the python 3.9 build for the current incompatibility issues.
making sure we have updated pip before installing packages.

Related Issues
closes #572

Codecov Report

Merging #573 (50aa48e) into development (e3046bd) will decrease coverage by 3.00%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #573      +/-   ##
===============================================
- Coverage        70.70%   67.70%   -3.01%
===============================================
  Files               84       91       +7
  Lines             3892     4483     +591
===============================================
+ Hits              2752     3035     +283
- Misses            1140     1448     +308

Impacted Files
Coverage Δ

jumpscale/servers/gedis/server.py
58.20% <0.00%> (-8.12%)
:arrow_down:

jumpscale/core/executors/remote.py
73.68% <0.00%> (-3.59%)
:arrow_down:

jumpscale/clients/redis/redis.py
85.29% <0.00%> (-3.28%)
:arrow_down:

jumpscale/servers/openresty/server.py
69.66% <0.00%> (-2.51%)
:arrow_down:

jumpscale/core/base/store/__init__.py
64.60% <0.00%> (-2.07%)
:arrow_down:

jumpscale/core/executors/command_builder.py
90.00% <0.00%> (-0.91%)
:arrow_down:

jumpscale/core/executors/local.py
86.66% <0.00%> (-0.84%)
:arrow_down:

jumpscale/core/logging/logging.py
93.56% <0.00%> (-0.79%)
:arrow_down:

jumpscale/core/events/__init__.py
88.37% <0.00%> (-0.76%)
:arrow_down:

jumpscale/servers/openresty/location.py
91.17% <0.00%> (-0.72%)
:arrow_down:

... and 23 more

Continue to review full report at Codecov.

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


Get rid of superfluous uncycle param

Figured this out over at https://github.com/corysimmons/lost and felt like Jeet could benefit from it.
This effectively gets rid of the uncycle param from cycle() and span(). It's pretty handy since uncycle was always kind of a pain to keep up with throughout media queries.
It's an API breaking change since column(ratios = 1, offset = 0, cycle = 0, uncycle = 0, gutter = jeet.gutter) will now be missing a param, so gutter would become the 4th param instead of uncycle.
column(1/2, 1/2, 2, **3**, 30px) becomes column(1/2, 1/2, 2, 30px).
I'm pretty sure this will need bumped to 7.0.0 to adhere to semver, but I think @jakecleary is already working on a ton of stuff for 7.0.0.
Feel free to let this lay dormant.
Also feel free to remove me from maintainers if you want.

@samccone :icecream: :racehorse: :deciduous_tree:

Don't merge. Should be nth-of-type

This looks great man; thanks!
Will definitely be a v7 release.
I think we were talking about maybe moving back to nth-type (#405), but keep it consistent with the current version for now :smile:

Can you explain how the new cycle system works? I'm too tired to grok it

Anytime cycle is triggered, it creates &:nth-of-type(n) { clear: none; } on the element in question. :nth-of-type(n) carries the same weight as :nth-of-type(Xn) so it will "erase" the previous cycle (e.g. &:nth-of-type(3n) { float: right; } &:nth-of-type(3n + 1) { clear: both; } before starting a fresh cycle.
In fact, now that I'm reading it, that float: right needs to be reset as well. 2 secs...

Fixed. Should be good to merge whenever. I haven't done any visual tests on it as I'm just a super confident kinda guy, so you should probably do that.
I went ahead and bumped to 7 so you can just roll this into your 7.

@jakecleary The code is still here. Just deleted it from my repo collection since it might be a while before it gets merged and I'm a neat freak.

@corysimmons Okay mate no worries. Will be in touch this week. Back in the UK 9-5 grind now.


Suggestion: Traditional Characters to Pinyin and Bopomofo

I am uncertain if there would be a different result but as someone trying to learn the traditional characters it would increase the confidence to have translation directly from traditional characters to the phonetic systems.

Unfortunately, I don't have the pronunciation data for Taiwan and Hong Kong, and the data in the project are all processed according to the Chinese mainland standard (Putonghua).
If someone can provide the corresponding pronunciation data, it will be easy to produce the content you need.


brush facets; target root element

Thanks. I’m going to try a slightly different approach, but this is the functionality that I want.

Implemented in 46c496e179a5cc38cb9c2cbb1ba9352d70bc05ab in the parent branch. Thanks for the assistance!


chore: added new crypto sponsor way.

ref: https://github.com/zhyd1997/sponsor/

Thanks for raising this. We are going to stick with the existing options for now.


Update use of NEURON_GROUP_SIZES in inferentia backend

There are two changes in this PR:

Update inferentia/qa/setup_test_enviroment_and_test.sh to build on the latest build.py.
Change NEURONCORE_GROUP_SIZES to NEURON_RT_NUM_CORES NEURON_RT_VISIBLE_CORES

Related PR: https://github.com/triton-inference-server/server/pull/4794

It is updated to use NEURON_RT_VISIBLE_CORES, and both pytorch and tensorflow tests have passed.

The testing of inferentia is very premature. I don't think it captures which triton model instance got mapped to which neuron rt cores.

can you add "protobuf<4" to setup.sh while you are here? The updated docs https://awsdocs-neuron.readthedocs-hosted.com/en/latest/neuron-intro/pytorch-setup/pytorch-update.html and https://awsdocs-neuron.readthedocs-hosted.com/en/latest/neuron-intro/tensorflow-setup/tensorflow-update.html use it

Yes. Updated. The result remains the same.


Tarball distributions contain useless files

The tarballs contain files that are not necessary to the end-user. I stumbled on this when I noticed that they also make it into Debian packages. For examples:

Images that VisTrails loads from resource files (*_rc.py)
Unbuilt documentation
Internal documents (PEPs, coding_conventions.txt, ...)
Development scripts (update_copyright_year, release_notes, ...)

272dacf9 removes the outdated developer documents; the wiki is probably the right place for this
1cebb74b makes it so that doc/dist/ is no longer included


Two inputs (WASD, Mouse button + Mouse Movement) that trigger an action, mouse movement will ignore the chord

Version
0.11.2
Operating system & version
macOS 14.1.2
What you did
I have an InputMap that goes into a resource:
 InputMap::new([
            (UserInput::VirtualDPad(VirtualDPad::wasd()), Self::Move),
            (
                UserInput::Chord(vec![
                    InputKind::Mouse(MouseButton::Middle),
                    InputKind::DualAxis(mouse_motion()),
                ]),
                Self::Move,
            ),
            // other inputs...
        ])

Usage in the system:
fn camera_movement(
    time: Res<Time<Real>>,
    mut action_state: ResMut<ActionState<PlayerAction>>,
    mut camera: Query<(&mut Transform, &OrthographicProjection), With<Camera>>,
) {
    if !action_state.pressed(PlayerAction::Move) {
        return;
    }

    let Some(axis) = action_state.axis_pair(PlayerAction::Move) else {
        return;
    };

    let (mut camera_transform, projection) = camera.single_mut();

    let velocity = axis.xy() * projection.scale;
    camera_transform.translation += velocity.extend(0.);
}

These two work as expected separately:

WASD will move the camera.
Holding down middle mouse button while moving the mouse will move the camera.

But when I hold down one of the WASD keys and move the mouse...
What you expected to happen
Moving the mouse while pressing any WASD keys should ignore the mouse.
What actually happened
When I:

Hold down one of the WASD keys and
Move the mouse and
Don't hold down the middle mouse button

... it activates the mouse movement.

Another situation which I was going to report as a separate bug, but realised it's probably the same bug:
            (
                UserInput::Chord(vec![
                    InputKind::Mouse(MouseButton::Middle),
                    InputKind::DualAxis(mouse_motion()),
                ]),
                Self::Move,
            ),
            (
                UserInput::Chord(vec![
                    InputKind::Keyboard(Space),
                    InputKind::DualAxis(mouse_motion()),
                ]),
                Self::Move,
            ),

Having these two will make the axis_pair value go twice as fast, as if the other chord modifier was ignored. So both mouse_motion()'s "activate" even though I might only be holding down space and not middle mouse.

I'm having the same problem but it seems to be easy to fix?

I'd be happy to see a PR for this.


[BUG] Search input not cleared when exiting combo

When i enter search query in combo, the search field is not cleared if i exit combo without selecting a value.

@LorenzoReply Thank you for the report. We have added this item to our platform backlog. If you are interested in expediting the resolution, please contact<EMAIL_ADDRESS>for pricing information.

We're currently addressing a backlog of GitHub issues, and as part of this effort, some inactive issues may be marked as closed. This isn't a dismissal, but a step toward more efficient tracking.
If you feel the issue is still relevant, please re-open and we'll ensure it gets the attention it deserves. Your understanding is appreciated as we work to enhance our open-source responsiveness.


Add Selenium tests for Web UI

https://softwarefactory-project.io/r/#/c/7687/

initial set of tests added and merged.


Example type definition is incorrect - raises  System.NotImplementedException

See https://github.com/dotnet/docs/issues/28828

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

ID: 114a6d66-f77b-19a9-4e0d-ff46ced10228
Version Independent ID: 60721783-7fa5-aed0-d823-3935ebc90b7b
Content: Get started with Azure Table Storage using F#
Content Source: docs/fsharp/using-fsharp-on-azure/table-storage.md
Product: dotnet-fsharp
GitHub Login: @sylvanc
Microsoft Alias: dotnetcontent

This is probably fixed by #28878.


docs(prefix change): announce prefix change

Describe pull-request

Adds Announcements section.
Annoucements use our component and style - no MDX
Redid the landing page to use our components.
Excluded stories files from build

Solving issue
Fixes: https://tegel.atlassian.net/browse/DTS-1698
How to test

Go to storybook
Check in Tegel Design System and Announcements

I feel like the announcement is tucked away and not easily discovered.
What would you think about linking it on the start page in some way?
Maybe even before the title with bold and with a 📢 or something. 🤔

I feel like the announcement is tucked away and not easily discovered. What would you think about linking it on the start page in some way? Maybe even before the title with bold and with a 📢 or something. 🤔

Do you mean the prefix change or the tegel announcement? I think this is more in line with how we would want it in the future. The start page is just an introduction and then the announcements are under "Announcements".

I feel like the announcement is tucked away and not easily discovered. What would you think about linking it on the start page in some way? Maybe even before the title with bold and with a 📢 or something. 🤔

Pushed an update now with a slimed down version of the announcement on the start page.

I meant just linking to it so it's discovered. Thought it was easily overlooked being tucked away in the announcements folder.
Like:
📣 Were moving away from the sdds prefix, read more here!
Tegel Design System
Bla bla bladida blablabla lorem ipsum dolor sit amet...

I meant just linking to it so it's discovered. Thought it was easily overlooked being tucked away in the announcements folder.
Like:
📣 Were moving away from the sdds prefix, read more here!
Tegel Design System
Bla bla bladida blablabla lorem ipsum dolor sit amet...

Great idea! I like it

I wonder why this one shows up now.
Is it possible to remove it?

Could you try it now?

Kind of want to make the other pages the same design now as well, what do you  think? Separate PR?

Kind of want to make the other pages the same design now as well, what do you  think? Separate PR?
@mJarsater @timrombergjakobsson

Yeah I had the same thought. I'd say separate PR. We don't want to block the prefix change epic by not merging this.

Kind of want to make the other pages the same design now as well, what do you  think? Separate PR?
@mJarsater @timrombergjakobsson

Yeah I had the same thought. I'd say separate PR. We don't want to block the prefix change epic by not merging this.

Sounds great!

@mJarsater @timrombergjakobsson Approved from my end, but will give you a change to look before merge.


Fix: Source -> Model Navigation

This PR addresses a layout issue when navigating from an unsaved source to a model resource. While the issue seems small, the cause required a substantial cleanup to bring sources more in line with the recently update models pages.

I didn't realize the scope on this, but nice to see the cleanup! Some UXQA findings:

I'm seeing that sometimes I don't land on the intended model page, but instead get redirected to the root page. JAM: https://jam.dev/c/fa21242d-76cc-4bac-8dff-a7d640b3c2c3
The "Refresh" button doesn't work for Local Files sources. JAM: https://jam.dev/c/0a24f64a-16f8-4187-bbc4-7c2310c6c94e
Switching between sources does not change the source name in the Workspace Header. This header should read "nyc311_reduced":

The dialog now says you're leaving a "model" not a "source"

When the left sidebar is closed, the "open sidebar" button overlaps with the Source name

Otherwise looks good, so I'll approve for you to merge at your discretion.

@ericpgreen2 Would love it if you could take another look. Also tagging @AdityaHegde since he seems to have a PR open that touches these files.
Essentially, I had to just nip this thing in the bud rather than trying to do patch fixes. This PR is now a total rework of the workspace routing and combines the model and source workspaces into a single +page.svelte file. It solves a ton of issues, removes a lot of code and will make the new data loading model a lot more straightforward (not included in this PR). That said, it's obviously a huge change and needs other eyes.
The +page file is currently pretty long, but I think relatively straightforward It'll get a lot simpler once we move the data loading out and can unify the Editor, error handling and inspectors. I can actually do the error handling in this PR if we can decide on which version (model or source) is preferred.


Removed azure websites constraint

Proved outbound rules for Access-Control-Allow-Origin worked on azure websites

Hi @dabutvin, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (dabutvin). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@dragon119

@CamSoper do you handle the CDN docs?
Can you offer any input on this? //cc @atoakley +

I help with CDN, but I'm not the primary author at the moment, at least for the time being (due to being assigned other priority projects).  Please loop in CDN program manager @Lichard .

Looks like now it works with Web site.

sign-off

#sign-off

Only the author of the article can use the sign off comment to add the ready-to-merge label.

#sign-off


fix npm: watch problem matcher stalling default debug task

This PR fixes the default debug task hanging because of a non-existing problem matcher:

I used vs code's api reference as a reference to update the build script as well.
The esbuild Problem Matchers is added as a recommended workspace extension.

Thank you 🙏🏼


Treat port 0 as meaning 'use the default'

Summary
Resolves #16.  If collector_port set to 0, it will be treated as meaning "use the default."  This can be used in conjunction with and respects the collector_encryption option.
FYI @djspoons.

Thanks!!


added UIActivityExtensions

UIActivty.ActivityType has various constants for different services (see https://developer.apple.com/documentation/uikit/uiactivity/activitytype). Unfortunately, some important services are missing (WhatsApp, LinkedIn, etc.). This adds some of them with the possibility to add more in the future. Let me known what you think about this 😀👍
Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 4.
[x] New extensions support iOS 8.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 526 tests, with 0 failures (0 unexpected) in 8.753 (12.746) seconds

:book:
tvOS: Executed 495 tests, with 0 failures (0 unexpected) in 3.581 (4.561) seconds

:book:
macOS: Executed 360 tests, with 0 failures (0 unexpected) in 1.108 (1.404) seconds

  Generated by :no_entry_sign: Danger

If you have a link to where you're getting these identifiers please add that in somewhere.

hmm, the most complete list I could find is this: http://johnszumski.com/blog/excluding-third-party-apps-from-ios-8-share-sheet (I don't know if these identifiers are all still valid)
I found the ones I listed by actually selecting and testing them in the UIActivityController. If you want, I could test some more if they are still valid!

If you want you can test all of them, but honestly for something like this I'd say just add the ones that you're using for your project. If somebody needs more they can add at a later stage. Probably best to put that link in a comment in the code so that it can be referenced in the future.

@staffler-xyz please resolve the conflicts and fix the (very) minor change requests


Running square.out causes entire system to hang and screen to flicker a few times

As the title stated, I am trying to get ROCm to work on my Laptop with AMD Ryzen 7 4800U with integrated Radeon graphics.
I am using Ubuntu 20.04, with kernel 5.8.0-59-generic.
I've also tried reinstalling rocm multiple times with:
sudo apt autoremove rocm-opencl rocm-dkms rocm-dev rocm-utils
sudo apt install rocm-dkms

Without rebooting my laptop, running square.out works perfectly fine.
However, when I try rebooting ubuntu and running the same program again, I get the following output
info: running on device Renoir
info: allocate host mem ( 7.63mb)
info: allocate device mem ( 7.63mb)
info: copy Host2Device

and my ubuntu hangs immediately.
I can still move my cursor around, but nothing responds to any clicks or whatsoever and I have to force shutdown the laptop.
can anyone please suggest how I may be able to figure out the root cause and how i could fix it? Thank you.

Thanks @caprinux for reaching out.
ROCm does not support integrated GPUs and so things might not work.
I can not comment much on this as its not a supported configuration.
Please check https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support for supported hardware with ROCm
Thank you.


NB Disclaimer and Acks update - IO Batch

@bmerino95 please review these updates, feedback is appreciated.
Updated Disclaimers and Attributions Sections along with version strings.
Notes:

The following files fail to execute (underlying SIA service for the GOGREEN files needs an update). Acks cell and version string updated, but not executed:

./03_ScienceExamples/GOGREEN_GalaxiesInRichEnvironments/2_GOGREENDr1SIA.ipynb
./03_ScienceExamples/GOGREEN_GalaxiesInRichEnvironments/3_GOGREENDr1ClusterMembership.ipynb
./03_ScienceExamples/GOGREEN_GalaxiesInRichEnvironments/4_GOGREENDr1Spec1DRedshift.ipynb

./03_ScienceExamples/GOGREEN_GalaxiesInRichEnvironments/1_GOGREENDr1DataAccessAtDataLab.ipynb did not have a Disclaimers and Attributions section. A new cell was added with the updated template.

The Large Scale Structure NB is only executed to before the "Large scale structure in 3D" section. Output data volume past this point is very large.

Hi Isabella! Everything looks good here. Nice work!

Hi @rnikutta
I fixed those mismatched anchor names in the GOGREEN NBs and made sure the other NBs did not have the same issue.
Thank you!

Hi @i-olin many thanks, looks good! Merging the PR.


Features/flask project

Flask implementation of the project.

Also please check the secure and lint fails.

Can be considered as ready to merge.
@vahidoqtayoglu please prepare the test suite for the flask in the next PR.


Add support for tvOS

I would like to be able to use APNGKit on tvOS.

It should not be hard to support tvOS since both using UIKit. However, currently I've no time for it now. Any pull request on this would be warmly welcomed.

tvOS is now officially supported in APNGKit v2.


"Traceback (most recent call last):"

I'm not sure what's the problem as all the fields in confing.ini are ok. So I'll leave some of the output here (the remaining output it's related to my FB profile. groups, friends...).
xx = my e-mail
Output:
Logging in xx...
Login of xx successful.
Pyrogram v0.7.4, Copyright (C) 2017-2018 Dan Tès https://github.com/delivrance
Licensed under the terms of the GNU Lesser General Public License v3 or later (LGPLv3+)
Registered commands
Traceback (most recent call last):
File "main.py", line 4, in
bridge.startBridge()
File "/home/ubuntu/facegram/facegram/facegrambridge.py", line 16, in startBridge
CommandsHandler(self.networkClient, self.bridge, self.config)
File "/home/ubuntu/facegram/facegram/commands_handler.py", line 10, in init
self.threads = self.networkClient.fbClient.fetchThreadList()
File "/home/ubuntu/.local/lib/python3.6/site-packages/fbchat/client.py", line 808, in fetchThreadList
return [graphql_to_thread(node) for node in j['viewer']['message_threads']['nodes']]
File "/home/ubuntu/.local/lib/python3.6/site-packages/fbchat/client.py", line 808, in
return [graphql_to_thread(node) for node in j['viewer']['message_threads']['nodes']]
File "/home/ubuntu/.local/lib/python3.6/site-packages/fbchat/graphql.py", line 206, in graphql_to_thread
raise FBchatException('Unknown thread type: {}, with data: {}'.format(thread.get('thread_type'), thread))
...

Looks like a problem with library I'm using (fbchat)[https://github.com/carpedm20/fbchat/issues]. Do you have anything other than normal facebook groups / friends in your recent facebook messages?

@feelfreelinux I have one group and 2 private chats. The account that I use to create the Bridge on FB is the same that I use to bridge Discord and FB.  Obviously, I've deleted the project on my hosting service (FB-Discord) so that it didn't interfere with this bot.
I can give it a try by creating a fresh profile, although I don't think that could be the cause.
I will come back and after that and if the problem persists I will pass the full log.
Thank you.

@feelfreelinux
After I've made a new account and installed tgcrypto (as the log suggests me), I manage to make it "run" using these commands:
pip3 install --upgrade tgcrypto
pip install --upgrade pip
Also, the /help command works now.
However, as soon as somebody texts me in private or in a group, the following error shows up:
(in private)
Registered commands Listening... Messages seen by<PHONE_NUMBER>31508 in<PHONE_NUMBER>651550 (GROUP) at<PHONE_NUMBER>.588s Exception in parsing of {'ofd_ts':<PHONE_NUMBER>423, 'queue':<PHONE_NUMBER>01820, 'delta': {'class': 'NewMessage', 'attachments': [], 'irisSeqId': '116', 'messageMetadata': {'skipSnippetUpdate': False, 'messageId': 'mid.$cAAAADOfIQQhqMFAxq1j_wtAVchBk', 'offlineThreadingId': '6413058593367658596', 'threadKey': {'otherUserFbId': '100011199331508'}, 'tags': ['source:chat:web'], 'actorFbId': '100011199331508', 'timestamp': '1528992295339'}, 'body': 'Hmmmm'}, 'type': 'delta', 'iseq': 116} Traceback (most recent call last):   File "/home/olime/.local/lib/python3.5/site-packages/fbchat/client.py", line 1521, in _parseMessage     thread_id=thread_id, thread_type=thread_type, ts=ts, metadata=metadata, msg=m)   File "/home/olime/facegram/facegram/fbclienthandler.py", line 9, in onMessage     self.bridge.facebookMessageHandler(author_id, message_object, thread_id, thread_type)   File "/home/olime/facegram/facegram/bridge.py", line 29, in facebookMessageHandler     thread = self.networkClient.createConversation(thread_id)   File "/home/olime/facegram/facegram/networkclient.py", line 49, in createConversation     title=thread.name   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/client/client.py", line 805, in send     r = self.session.send(data)   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/session/session.py", line 417, in send     return self._send(data)   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/session/session.py", line 404, in _send     Error.raise_it(result, type(data))   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/api/errors/error.py", line 71, in raise_it     )(x=x) pyrogram.api.errors.exceptions.bad_request_400.UsersTooFew: [400 USERS_TOO_FEW]: Not enough users (to create a chat, for example) Inbox event: 1, 1, 1
(and the one below it's from a group)
Registered commands Listening... Inbox event: 1, 1, 1 Marked messages as seen in threads [('100011199331508', 'USER')] at<PHONE_NUMBER>.995s Marked messages as seen in threads [('1844064765651550', 'GROUP')] at<PHONE_NUMBER>.766s Inbox event: 0, 0, 0 Exception in parsing of {'queue':<PHONE_NUMBER>01820, 'ofd_ts':<PHONE_NUMBER>846, 'iseq': 121, 'type': 'delta', 'delta': {'class': 'NewMessage', 'irisSeqId': '121', 'body': '¿Este?', 'messageMetadata': {'timestamp': '1528992638770', 'tags': ['source:chat:web'], 'offlineThreadingId': '6413060033959593412', 'messageId': 'mid.$gAAaNKslMql5qMFVvMlj_xB9_aoXE', 'threadKey': {'threadFbId': '1844064765651550'}, 'actorFbId': '100011199331508'}, 'attachments': [], 'ttl': 'TTL_OFF'}} Traceback (most recent call last):   File "/home/olime/.local/lib/python3.5/site-packages/fbchat/client.py", line 1521, in _parseMessage     thread_id=thread_id, thread_type=thread_type, ts=ts, metadata=metadata, msg=m)   File "/home/olime/facegram/facegram/fbclienthandler.py", line 9, in onMessage     self.bridge.facebookMessageHandler(author_id, message_object, thread_id, thread_type)   File "/home/olime/facegram/facegram/bridge.py", line 29, in facebookMessageHandler     thread = self.networkClient.createConversation(thread_id)   File "/home/olime/facegram/facegram/networkclient.py", line 49, in createConversation     title=thread.name   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/client/client.py", line 805, in send     r = self.session.send(data)   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/session/session.py", line 417, in send     return self._send(data)   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/session/session.py", line 404, in _send     Error.raise_it(result, type(data))   File "/home/olime/.local/lib/python3.5/site-packages/pyrogram/api/errors/error.py", line 71, in raise_it     )(x=x) pyrogram.api.errors.exceptions.bad_request_400.UsersTooFew: [400 USERS_TOO_FEW]: Not enough users (to create a chat, for example) Inbox event: 1, 1, 1 Marked messages as seen in threads [('1844064765651550', 'GROUP')] at<PHONE_NUMBER>.698s Inbox event: 0, 0, 0
I don't know why it says "not enough users"...
Also, when I use "/import 1" it says nothing. On the other hand I'm confused as I don't know if I should use "/import 1" or "/import #1".

@RingCell You need two separate telegram accounts to use this app:

First account its your private telegram account that you use every day, configure its telegram username / handle in config.ini
Second account needs to act as bridge, configure its api_hash and api_id in config.ini.

I should make it more clear in readme tho

@feelfreelinux It worked!
I thought that those tree fields were completely related (username, api_hash_ and api_id). You should definitely put a comment on the first line specifying that it doesn't have to be the "bot" username.
So... Even if everything is working right now there is a behavior of the bot that I don't understand.
When I use "/import 1" (which is the FB group id) in my Telegram group, it creates a new group instead of linking the current one with the FB group.
Maybe that's the intended behavior or I'm missing something. Sorry, if this is no related to the main issue but I feel like it's better than create a new "issue" at the moment.

Closed as it's related to Python version.


Sht40

sht40 test

Hi @TheGodMotherG! Thanks for adding a definition for this sensor.
Are you also planning to add the required I2C Component Driver (https://learn.adafruit.com/how-to-add-a-new-component-to-adafruit-io-wippersnapper/adding-an-i2c-component-driver) to WipperSnapper Arduino? It should be similar to the AHT20 driver which already exists in the firmware.

Lmao I’m brand new to this but I need the chip to work I will give it my
best shoot. Because you guys said later in the summer but I need it asap
so. Yes I will be attempting it
On Tue, May 31, 2022 at 9:27 AM Brent Rubell @.***>
wrote:

Hi @TheGodMotherG https://github.com/TheGodMotherG! Thanks for adding a
definition for this sensor.
Are you also planning to add the required I2C Component Driver (
https://learn.adafruit.com/how-to-add-a-new-component-to-adafruit-io-wippersnapper/adding-an-i2c-component-driver)
to WipperSnapper Arduino? It should be similar to the AHT20 driver which
already exists in the firmware.
—
Reply to this email directly, view it on GitHub
https://github.com/adafruit/Wippersnapper_Components/pull/49#issuecomment-1142135781,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPQAUPENZVYYBBJUJTA353VMYHS5ANCNFSM5XHR4A4A
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok i did the driver hope it is what you are looking for to add it to the IO platform

Going to merge this so it's usable for testing in Adafruit IO (you should see the SHT40 component on IO).
Let's continue the convo over on https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/pull/266/ so we can test!


ERROR  [h3] [unhandled] Cannot read properties of undefined (reading 'indexOf')

Environment

Operating System: Darwin
Node Version:     v18.7.0
Nuxt Version:     2.16.0-27720022.54e852f
Nitro Version:    0.6.1
Package Manager<EMAIL_ADDRESS>Builder:          webpack
User Config:      buildDir, srcDir, bridge, components, ssr, dev, alias, image, generate, content, head, typescript, publicRuntimeConfig, privateRuntimeConfig, css, loading, loadingIndicator, plugins, buildModules, modules, proxy, vuetify, build, serverHandlers, devServerHandlers
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>~/modules/apolloSSR<EMAIL_ADDRESS>

Reproduction
Latest nuxt-bridge-edge 3.0.0-27778503.de9e5c2 causes this error after upgrading not present on previous versions
Describe the bug

Additional context
No response
Logs
No response

Hi @stephenjason89 I'm having the exact same error, have you manage to solve this somehow?

Same error, looking for any solution

also having this same error and would appreciate any solution. Note that our module is using @nuxt/proxy but the error still occurs even when that is uninstalled

Im not using nuxt proxy but just axios proxy 🤔  same issue, probably not related to axios proxy

https://github.com/nuxt/bridge/issues/141#issuecomment-1097972563

Please help repair it, thank you

https://github.com/nuxt/bridge/issues/141#issuecomment-1097972563

https://github.com/nuxt/bridge/issues/616#issuecomment-1314171413

Are you using @nuxtjs/proxy, nuxi, nitro: true?
If so, you need to use devProxy with nitro.
export default defineNuxtConfig({
-  modules: ['@nuxtjs/proxy']
-  proxy: {
-    '/api': 'http://example.com',
-  }
+  nitro: {
+    devProxy: {
+      '/api': 'http://example.com',
+    }
+  }
})

reproduction
https://stackblitz.com/edit/github-3fgb91-mo4eua


"API" is not visible in Environment column for Test runs in Zebruner

Env.
jenkins-master:5.1, sonarqube:2.2 and stf:3.2, qps-infra:develop ,pipeline: 5.1
Steps:

Run Api tests via Jenkins
Open test runs in Zafira
Verify that API test runs show "API" in Environment column
Expected:API test runs show "API" in Environment column
Actual:API test runs show "empty" in Environment column

Tested, Fixed.

fix is available in develop branch


API-based login with proper status codes (fixes #596)

Story / Bug id: https://github.com/CodeForPoznan/volontulo/issues/596
Reference person: @magul
Description:
Proper API response status codes for login enpdoint.
New imports / dependencies:

N/A

Unit Tests:

N/A

Codecov Report

Merging #612 into master will increase coverage by <.01%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master     #612      +/-   ##
==========================================
+ Coverage   92.64%   92.64%   +<.01%
==========================================
  Files          80       80
  Lines        2624     2625       +1
  Branches        2        2
==========================================
+ Hits         2431     2432       +1
  Misses        191      191
  Partials        2        2

Impacted Files
Coverage Δ

backend/apps/volontulo/views/api.py
78.37% <33.33%> (+0.6%)
:arrow_up:

Continue to review full report at Codecov.

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