fromEnvironment should also pick up the python path from jupyter

I am not sure if this is a bug or not.
Say, you create a new folder and work with some Jupyter notebooks. Before the first time you open a python script in this workspace, isort will still pick up the default python interpreter, even if a kernel for Jupyter is already selected.
For example, here I created a folder called isort.  The fromEnvironment option actually picks up the default python shipped with the OS. This will only be changed if I create a dummy python script, open it up, and select an interpreter there. In my view, the moment you select Jupyter kernel, isort extension should be aware as well.
2024-02-20 11:50:04.827 [info] Name: isort
2024-02-20 11:50:04.827 [info] Module: isort
2024-02-20 11:50:04.827 [info] Python extension loading
2024-02-20 11:50:04.827 [info] Waiting for interpreter from python extension.
2024-02-20 11:50:04.827 [error] Python version 3.6 is not supported.
2024-02-20 11:50:04.827 [error] Selected python path: /usr/bin/python3
2024-02-20 11:50:04.827 [error] Supported versions are 3.7 and above.
2024-02-20 11:50:04.827 [info] No interpreter found from setting isort.interpreter
2024-02-20 11:50:04.827 [info] Getting interpreter from ms-python.python extension for workspace /vast/home/user/codes/isort
2024-02-20 11:50:04.827 [info] Python extension loaded
2024-02-20 11:50:04.827 [error] Python version 3.6 is not supported.
2024-02-20 11:50:04.827 [error] Selected python path: /usr/bin/python3
2024-02-20 11:50:04.827 [error] Supported versions are 3.7 and above.
2024-02-20 11:50:04.827 [info] No interpreter found for /vast/home/user/codes/isort in settings or from ms-python.python extension
2024-02-20 11:50:04.827 [error] Python interpreter missing:
[Option 1] Select python interpreter using the ms-python.python.
[Option 2] Set an interpreter using "isort.interpreter" setting.
 Please use Python 3.7 or greater.

@tautomer The python selection is coming from python extension. isort uses whatever it gets from either the isort.interpreter setting or from the python extension. It does not do its own selection.


[BUG] Field Issues in Organization's General Tab

Description
Tried updating the form with some information:

Unable to change the email to empty anymore.
Accepts any random email.
Website field and Workspace filed accept spaces as values
Deleting a value doesn't update but shows last stored value (I remove info's in the Website field before starting the screencast)

Steps to reproduce the behaviour:
Errors Screencast: https://recordit.co/DyIcHUHKrq
Important Details

Version: Cloud
OS: Windows 10
Browser: Microsoft Edge Version 85.0.564.63 (Official build) (64-bit)

@devrk96 please add validation for these fields


Spike: Watt-slider

ngx-slider which is used for the current implementation is unmaintained and now causes issues because it's no longer up-to-date.
We need to figure out if we can find a new slider library to use instead, build our own or maintain our own version of ngx-slider.
Currently we have disabled our tests for the slider to make it possible to update our dependencies.
AC

[ ] Figure out and document possible solutions
[ ] Align on the best solution in the team
[ ] Create issues to implement chosen solution

Simple solution which might can be used for inspiration: https://stackoverflow.com/a/31083391


[REF-2626] reflex init generates incorrect reflex version in requirements.txt

Describe the bug
When generating a brand new reflex project with init the incorrect reflex version is added to the generated requirements.txt file
To Reproduce
python -m venv .venv
source .venv/bin/activate
pip install reflex
reflex init
cat requirements.txt # reflex==0.4.5
reflex -v # 0.4.8.post1

Expected behavior
The output of reflex -v should match the version specified in requirements.txt
Specifics (please complete the following information):

Python Version: 3.12.2 (via pyenv)
PIP Version: 24.0
Reflex Version: 0.4.8.post1
OS: Mac 14.4.1
Browser (Optional): N/A

REF-2626

@camstuart can you provide more info on how to repro this? I just tried your repro steps and the reflex version was the same as it was in the requirements. Also looking at the code, we seem to get the reflex version from the current interpreter(which should be the activated venv). The only scenario this differs for me is if you run reflex init on an existing project which doesn't rewrite the requirements file. Is this case you're describing?


parse operation shouldn't save core data context

I believe theres a mistake that the parse operation shouldn't save core data context because it might lead to the store being in an inconsistent state if the user has made changes to the context before the parse operation completes and then decides to throw away the changes rather than save.

Parse operation is for demo purpose, it is fine because context is pased as a private context from main queue which need to be saved to be albe to see parsed data in main queue.


Create DES obs properties

2 new vocabulary files to create the observed properties for DES as used in the QESD Catalogue.

TERN properties - a version of the properties used by TERN; uses the same URIs. Likely that after review a number of the properties will be removed for our purposes, but sticking with everything as a start.
Qld properties - a set of additional properties used by DES; currently all water related items, that will fit under the TERN top concept of 'water properties'.

Qld properties is going to import and use the TERN properties. But unsure if I actually have that setup correctly, require a review from @KellyVance and @nicholascar  to validate that this is correct.

I should also note that the files have passed validation with the VocPub validator and the SKOS testing tool. A couple of issues with the SKOS testing tool in the Qld properties file with labels, etc related to the TopConcepts used from the TERN vocabulary.

@nicholascar I have made the updates based on your review.
In terms of the URIs, where there is a TERN uri I am using it and new terms have a new URI that we are creating in a concept scheme. Both of the files. The intention is that both sets of concepts (TERN and DES) would be combined into a single vocabulary list. TERN doesn't include all of the observed properties that we will need.
I was trying to keep a degree of separation between the TERN and DES concepts to allow for easier management and updating. For the time being we can use the top level concepts from TERN without having to create additional concepts.
Does using owl:imports for the TERN vocab in the DES one bring them together?

where there is a TERN uri I am using it and new terms have a new URI

Great, the best approach! I just couldn't be sure of that.

Does using owl:imports for the TERN vocab in the DES one bring them together? Or do I need to combine this into a single set of terms?

In theory it does but owl:imports is tool-dependent, i.e. some copy ' paste mechanism must be implemented in a tool for it to work, and VocPRez can't do this. If you owl:imports an external vocab to one in VP, VP would have to resolve that vocab from source, bring it into that vocab's graph and serve it's content up. It could be done but it would require a fair few now functions not yet present.
I suggest you just manually copy 'n paste all the Concepts you wish to appear in your vocab unless they are just "hand off" links like your vocab's top concepts point to other Concepts elsewhere but you don't wish those concepts to appear in your vocab directly. If this is the case, use skos:broadMatch [1] rather than skos:broader. The mapping properties are there for these sorts of hand-offs but often you want to incorporate other vocab's Concepts directly in your vocab, as you've done for the majority of them here.
[1] https://www.w3.org/TR/skos-reference/#mapping

I suggest you just manually copy 'n paste all the Concepts you wish to appear in your vocab

Right - I will combine the 2 individual vocabularies into a single file. Please hold off on any further reviews while I do the consolidation.

@nicholascar @UbuntuScott @KellyVance - I have consolidated the concepts into a single vocabulary and file. Which is now ready for review.
@KellyVance  - I have left the file that I used from TERN. Probably don't need it loaded VocPrez, but will be a good reference as we move forward. What is your workflow for those types of items?

Should position x (and y) coordinate have broader term of skos:broader http://linked.data.gov.au/def/tern-cv/9e03a73d-8e89-4187-9f96-0306aad93458 ;
= physical parameter?

@UbuntuScott - I have left as is given this is directly from TERN.

should water count and water prevalence be given a new broader term under "water parameter" or moved to "land surface"?

Left this as is since directly from TERN.

All the water parameter 'headings' also need to be skos concepts

Updated - good pick up.

Additional validation reviews and validation, including visual exploration of the graph identified some deficiencies which have since been corrected.


Update regex to catch dependencies

Overview
Updates the packing/repacking regex to update the dependencies lines as well.

/azp run

/azp run

/azp run


refactor(ast_tools): remove well_known! macro

Remove well_known! macro by expanding code manually.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#6957  👈
#6956
#6954
#6953
#6952
#6951
#6950
#6949
#6948
#6947
#6946
#6945
#6944
#6943
#6942
#6941
#6940
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite

Merge activity

Oct 27, 11:31 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


update documentation to reflect resolution of bugs 422068 and 453386

Q
A

content fix?
yes

New article?
no

What's in this Pull Request?
These changes update the documentation to the following changes:

The Get-SPOTheme cmdlet now returns all uploaded themes if no theme name parameter is specified

The Add-SPOTheme cmdlet now accepts color palette settings as either a hash table or a dictionary. (Previously a dictionary was required.)

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/site-theming/sharepoint-site-theming-powershell.md
:white_check_mark:Succeeded

docs/images/Get-SPOTheme-example.png
:white_check_mark:Succeeded

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


Reset layer style when original GeoTIFF source is ready

Fixes https://github.com/openlayers/openlayers/issues/16486
If a renderer is created before the original GeoTIFF source finishes loading it is necessary to wait until the source is ready before parsing the style with the correct bands. This fix simply re-parses the style after the load finishes.
Inspired by https://github.com/openlayers/openlayers/pull/16067

Currently failing test https://github.com/openlayers/openlayers/pull/16494 should be merged after this patch.


What's the right way to stop data_phase runs when enough data has been generated?

Hey Dian,
It's me again - had a question about how to correctly stop the data_phase methods once enough data has been collected. The workflow I've been going with is: spin up CARLA servers with launch_carla.sh, and run python -m data_phase1 for example. I check the target directory that data is written to and call ray stop once enough has been generated. But, I'm not sure if this is okay.
I'm asking because I collected a data_phase1 dataset and ran data_phase2, but got the following:
(wor) [aaronhua@trinity-0-11 WorldOnRails]$ python -m rails.data_phase2 --num-workers=4
Traceback (most recent call last):
  File "/home/aaronhua/anaconda3/envs/wor/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/aaronhua/anaconda3/envs/wor/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/aaronhua/WorldOnRails/rails/data_phase2.py", line 59, in <module>
    main(args)
  File "/home/aaronhua/WorldOnRails/rails/data_phase2.py", line 13, in main
    total_frames = ray.get(dataset.num_frames.remote())
  File "/home/aaronhua/anaconda3/envs/wor/lib/python3.7/site-packages/ray/worker.py", line 1379, in get
    raise value.as_instanceof_cause()
ray.exceptions.RayTaskError(TypeError): ray::RemoteMainDataset.num_frames() (pid=13194, ip=<IP_ADDRESS>)
  File "python/ray/_raylet.pyx", line 422, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 456, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 459, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 463, in ray._raylet.execute_task
  File "python/ray/_raylet.pyx", line 415, in ray._raylet.execute_task.function_executor
  File "/home/aaronhua/WorldOnRails/rails/datasets/main_dataset.py", line 216, in __init__
    super().__init__(*args, **kwargs)
  File "/home/aaronhua/WorldOnRails/rails/datasets/main_dataset.py", line 123, in __init__
    n = int(txn.get('len'.encode()))
TypeError: int() argument must be a string, a bytes-like object or a number, not 'NoneType'

I've double checked that the main_data_dir specified in config.yaml and the data-dir argument in rails/data_phase2 point to the correct directory, and each of the runs within the data directory are at least a couple tens of megabytes. Strangely, I was previously just Ctrl-C'ing the processes which seemed to work fine (running train_phase2 on a different set of data currently with no issue). I was under the impression that ray stop would be the "correct" way to stop data processes.

This can happen if you terminate while the workers are right saving data to disk -- the lmdb files become corrupted. You can just print the file in the glob for loop and simply delete trajectories that are corrupted.

Makes sense, the suggested fix works. Closing the issue


Maximum size/length of BannedIpList

What's the maximum size/length of the BannedIpList?

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

ID: 81e37706-af12-9dc3-4004-bdab6f8039e9
Version Independent ID: c9f498a9-457b-b22f-2071-5f5dab609981
Content: Configure AD FS Banned IP Addesses
Content Source: WindowsServerDocs/identity/ad-fs/operations/Configure-AD-FS-Banned-IP.md
Product: windows-server-threshold
GitHub Login: @billmath
Microsoft Alias: billmath

300 entries, at least per the command itself (I had a list MUCH longer....).    This REALLY needs to be longer.  Like 5 digits longer.

Nick42 were you using a script to read from a file containing your 300+ banned IP list?

No.  I made a ridiculously long powershell script that added each one with a separate command (in an attempt to see if it was a batch thing, or ...)

Thanks for responding Nick. I managed to get this figured out. We have a list of banned IPs in a CSV coming from our PaloAlto and I created a basic script to read the contents and add to the banned IPs list.
Looks like it’s working great!!!
If this email is in regards to an IT request, please redirect it to ITSupport<EMAIL_ADDRESS>or by calling<PHONE_NUMBER> so it may be dealt with in a timely manner.
LEO HO | Systems Administrator
D<PHONE_NUMBER>
E Leo.Ho<EMAIL_ADDRESS>1500, 525 8th Avenue SW | Calgary, AB T2P 1G1
bonavistaenergy.comhttp://bonavistaenergy.com/
<EMAIL_ADDRESS>From: Nick42-<EMAIL_ADDRESS>Sent: Monday, October 22, 2018 3:50 PM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Leo Ho<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: EXTERNAL: Re: [MicrosoftDocs/windowsserverdocs] Maximum size/length of BannedIpList (#1257)
No. I made a ridiculously long powershell script that added each one with a separate command (in an attempt to see if it was a batch thing, or ...)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1257%23issuecomment-432002093&data=02|01|leo.ho%40bonavistaenergy.com|2debb9471a944c6f611108d6386840f8|83a2a5a8559046bc9c980ee53013fd49|0|0|636758417758024776&sdata=oGjGLbNxDzQqBAPzpnx8xZuJhAuhL6PBf3vnuAoGgkc%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAqUnG1t_ALibNZ4IIqgDFl9avJDxktH4ks5unj1tgaJpZM4Vmb1C&data=02|01|leo.ho%40bonavistaenergy.com|2debb9471a944c6f611108d6386840f8|83a2a5a8559046bc9c980ee53013fd49|0|0|636758417758024776&sdata=ykRmvEuhAQlDEuB1ISjPeCZO11uBRhRcoKKfortXy1Q%3D&reserved=0.
This e-mail message and any attachments are confidential. Any dissemination or use of this information by a person other than the intended recipient is unauthorized. If you are not the intended recipient, please notify me by return e-mail, do not open any attachment and delete this communication and any copy. Thank you!

More than 300?!?!?!

No, our IP list gets updated daily and maybe has a few hundred
If this email is in regards to an IT request, please redirect it to ITSupport<EMAIL_ADDRESS>or by calling<PHONE_NUMBER> so it may be dealt with in a timely manner.
LEO HO | Systems Administrator
D<PHONE_NUMBER>
E Leo.Ho<EMAIL_ADDRESS>1500, 525 8th Avenue SW | Calgary, AB T2P 1G1
bonavistaenergy.comhttp://bonavistaenergy.com/
<EMAIL_ADDRESS>From: Nick42-<EMAIL_ADDRESS>Sent: Monday, October 22, 2018 3:56 PM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Leo Ho<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: EXTERNAL: Re: [MicrosoftDocs/windowsserverdocs] Maximum size/length of BannedIpList (#1257)
More than 300?!?!?!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fwindowsserverdocs%2Fissues%2F1257%23issuecomment-432003980&data=02|01|leo.ho%40bonavistaenergy.com|437e50bc264a4755054308d638691bf3|83a2a5a8559046bc9c980ee53013fd49|0|0|636758421440879619&sdata=YEC9UCmllEaKFE9etU9xF2TUNaxOfA3X0E3Bd8oWQwM%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAqUnG7W8yTZv0xNZOmL3IUc8CaRbk1dwks5unj7cgaJpZM4Vmb1C&data=02|01|leo.ho%40bonavistaenergy.com|437e50bc264a4755054308d638691bf3|83a2a5a8559046bc9c980ee53013fd49|0|0|636758421440879619&sdata=qC4PbwSx%2FOF%2BN2NHWXKFexJQhJ8ftnuBOYGnO5qiJsw%3D&reserved=0.
This e-mail message and any attachments are confidential. Any dissemination or use of this information by a person other than the intended recipient is unauthorized. If you are not the intended recipient, please notify me by return e-mail, do not open any attachment and delete this communication and any copy. Thank you!

FYI, this is how I went through automating adding banned IPs to ADFS.
We have a list of banned IP addresses that gets updated from our PaloAlto to a CSV file.
I created the following powershell script to pull the IPs from the CSV and then add those IPs to the ADFS bannedip list:
$BNPBannedIPs = Get-Content C:\Scripts\ADFS\Test.csv
Set-AdfsProperties -AddBannedIps $BNPBannedIPs
Since the CSV file that contains the banned IPs gets updated daily from our PaloAlto,
I created a scheduled task to have the powershell script run daily.
Hope this helps

So I'm curious what happens once you hit 301 entries...

I will let you know if and once I get there :)

I hit the 300 limit fairly quickly. I created a powershell script to automatically ban IPs found in brute force password spray attacks in ADFS and I reached the 300 limit. It simply will not increment higher than 300.

Since I'm still fairly new with this, when I run get-adfsproperties I see 4 banned ips followed with an ellipses. Does anyone know how to view all of the banned ips that have been set?

get-adfsproperties | select bann* -expandproperty bann*
I didn't confirm this, but I think it's correct.
On Thu, Oct 25, 2018 at 9:56 AM leohoknows<EMAIL_ADDRESS>wrote:

Since I'm still fairly new with this, when I run get-adfsproperties I see
4 banned ips followed with an ellipses. Does anyone know how to view all of
the banned ips that have been set?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/windowsserverdocs/issues/1257#issuecomment-433060823,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgpleY4ih3kiQrO5MzcskxVdNTzcIUiks5uocL2gaJpZM4Vmb1C
.

Hey Bob, do you happen to know how to remove an ip from a user's list of familiarips?

Arrgh... just came across the same issue.
I was in the process of implementing a rotating block list (of known recent brute force attacks), and reached the 300 limit sigh
To assume there are only 300 “bad” IP’s out there in the big wide world is delusional.
I totally agree, and hope the limit is increased!  :)
Cheers, Kurt.

Leeharrison,
You can view the entire list this way:
get-adfsproperties | select bann* -expandproperty bann*
On Tue, Apr 9, 2019 at 9:29 AM leeharrison<EMAIL_ADDRESS>wrote:

Hi all, I have been adding banned IP's, but when I run Get-AdfsProperties
it only returns the first 3 entries. How can I see the full list, I have
entered 100+ CIDR blocks in there.
Cheers,
Lee
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/windowsserverdocs/issues/1257#issuecomment-481251962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFgplZEY8PksyjDjO-s1cdEn_pY8eIDfks5vfJXLgaJpZM4Vmb1C
.

The field accepts ranges and CIDR notation. So try getting more spread by blocking large blocks of IPS that are your largest offenders.
You can use tools below to identify appropriate CIDR notation and what IP spaces are owned by who to help identify appropriately what you want to block.
https://www.ipaddressguide.com/cidr  and
https://wq.apnic.net/static/search.html

Still need more entries than the system allows for, since the address spaces are not congruent.
-----Original Message-----
From: reevesjeremy<EMAIL_ADDRESS>Sent: Tuesday, June 18, 2019 9:02 AM
To: MicrosoftDocs/windowsserverdocs<EMAIL_ADDRESS>Cc: Antone, Nick<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windowsserverdocs] Maximum size/length of BannedIpList (#1257)
The field accepts ranges and CIDR notation. So try getting more spread by blocking large blocks of IPS that are your largest offenders.
You can use tools below to identify appropriate CIDR notation and what IP spaces are owned by who to help identify appropriately what you want to block.
https://www.ipaddressguide.com/cidr and
https://wq.apnic.net/static/search.html
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/windowsserverdocs/issues/1257?email_source=notifications&email_token=ACF3K4KUZVBC4O652QQFX6DP3EBJFA5CNFSM4FMZXVBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX7D6UY#issuecomment-503201619 , or mute the thread https://github.com/notifications/unsubscribe-auth/ACF3K4JBAQC3GYQVQ6ABWDTP3EBJFANCNFSM4FMZXVBA . https://github.com/notifications/beacon/ACF3K4MLQVZOMWDZQJLLQETP3EBJFA5CNFSM4FMZXVBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX7D6UY.gif

Still need more entries than the system allows for, since the address spaces are not congruent.

@Nick42- I use Splunk to create a report with all Class B subnets generating excessive EventCode=516, filtered by foreign IPs using Geo location. Sorting by highest count, I check the Class b subnet to ensure the entire net is "owned/operated" by the same foreign entity. Assuming I know that our org does not have affiliates regularly in that geographical location (or require them to VPN if they are), I'll block that entire Subnet range using CIDR. We have 164 ranges covering about 1200 Class Bs. I havn't done a count, but I venture to say that we're blocking millions of IP hosts this way. This reduced our EventsCode=516 from a daily range of 37,000 to 70,000, down to an average of 4,000. Coincidently, most of them are internal now, due to password changes!
If your people are geographically located in places you know, it could be beneficial to block all the other offending geo locations.
Hopefully that helps. If not, sorry! Maybe it could improve but not completely eliminate...  I review the splunk alert periodically and tweak a little here and there.

#please-close


Add SFAController model

Add SFAController and interop.
Signed-off-by: Joe Grund<EMAIL_ADDRESS>

This change is

Rebuild passed in https://build.whamcloud.com/view/IML/job/manager-for-lustre/688/


Somehow this mods v2.0.1 breaks advanced company

I have a modpack with over 100 mods i spent the last 8 hours removing and reenabling them some how removing the mod fixed the issue i have to revert back to 2.0.0 for now until fixed this is the screenshot 10+ have sent me today.

apologies for all the trouble! — let me know if this is still an issue in v2.0.2!
(if so, please do share a mod code / mod list so that i can identify the incompatibility!)


Problem when using Material Widgets like Cards with ListWheelScrollview

Hi i have a Problem when using any Material widgets which usually supports the elevation parameter
like the Card  or even the Material Widget. And the problem is that when using these kind of widgets with the ListWhellScrollView , it prevents it from scrolling and lays out the cards (as an example) on top of each other . Here is my Code :
import 'package:flutter/material.dart';
import 'package:superellipse_shape/superellipse_shape.dart';

class HalfCircleExample extends StatelessWidget {
  Widget _buildItem() {
    return Center(
      child: Container(
        width: 50,
        height: 50,
        child: Card(
          shape: SuperellipseShape(
            borderRadius: BorderRadius.circular(40),
          ),
        ),
      ),
    );
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('WheelExample'),
      ),
      body: SafeArea(
        child: Center(
          child: ListWheelScrollView(
            itemExtent: 60,
            children: List.generate(
              10,
              (int i) => _buildItem(),
            ),
          ),
        ),
      ),
    );
  }
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: HalfCircleExample(),
    );
  }
}

void main() => runApp(MyApp());

and here is an image of the output  :

and here is my flutter doctor :
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.12.13+hotfix.5, on Microsoft Windows [Version 10.0.17763.914], locale en-US)

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.0-rc1)
[√] Android Studio (version 3.3)
[√] VS Code (version 1.41.1)
[√] Connected device (1 available)

• No issues found!

Fix is already in stable (Tried with 1.17.1). Closing


Documentation: Mapbox guide

Jocelyn will start document, Ryan may need to review

I've made some changes and updates, but have questions and the document will need review by Ryan.
https://docs.google.com/document/d/1p2xvMFHBCOctnsGiJgsGc_iWZiIe0InqXq7K_jV_DqI/edit?usp=sharing


Confusing usage of assertThat

In step 6 org.junit.Assert.assertThat is used while in step 7 com.google.common.truth.Truth.assertThat is used. I feel like this is confusing and should be called out?

Need to do a consistency pass


Updated default recipe to work on all windows and remove dependency o…

…n windows binaries.
Was having issues with wrong version of powershell and dependency on xcopy binary.
Used the windows cookbook zip function to better handle zip extraction and remote_file resource to copy the file instead

This also fixes issue #9

the test is broken since i changed the install nssm code and i'm not familiar enough with tests to fix it. feel free to let me know what to do so the tests work. ill see if i can figure them out in the meantime

OK well ive managed to fix all the other issues except the actual tests now. will try the tests tomorrow.


Settings screen does not allow you to select default session type

Summary of the new feature/enhancement
Allow the default session type to be set, so I can start in WSL/bash immediately.
Proposed technical implementation details (optional)
Add option in settings panel. Use it at startup.

Oh, copy-paste UUDIs? Please make this a real dialog.


Quaternions instability

Hello I'm trying to connect my game engine in WebGL to cannon.js, but I have a trouble to convert your quaternions to euler angles. (I'm using euler angles, I never had the problem until I faced quaternions)
Function on Quaternion.js toEuler provides very unstable angles. They are ok in x axis. But y and z are behaving weird. I captured it on video. It is very noticeable on the end of video. http://youtu.be/ljd4avOFPs4 If you know where the problem could be please let me know. Thank you :)

Maybe the euler order is wrong? What order do you use in your rendering engine?
In any case, it is often more convenient to not use euler. Use the quaternion directly or convert it to a rotation matrix.

I tried all 6 permutations of xyz order, none of them works stable.
I woke up today and try different aproach... searched for formulas on internet:
These works perfectly:
function quatToVec3(x,y,z,w){
var ax = Math.atan2(-2yz+2xw,1-2xx-2yy);
var ay = Math.asin(2xz+2yw);
var az = Math.atan2(-2xy+2zw,1-2yy-2zz);
return [ax,ay,az];
}
It's for XYZ order. Someone will be happy if you add this order to cannon.js. Anyway thank you for awesome physics library! :)


Add something that blocks web access after too many failed logins like fail2ban

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
I would like to have a middleware that can track failed logins and can block the IP for some time after a configurable number of failures.

Hello,
you can look at the plugins catalog.
You will find a plugin that handles fail2ban: https://pilot.traefik.io/plugins/605afbaaa5f67ab9a1b0e51c/fail2-ban.

@ldez yes ok. This is a 3rd party plugin that might cover this functionality.
But 3rd party plugins might have huge issues in the future.
There might be version updates in Traefik and suddenly the plugin is incompatible.
Or there might be security issues and the contributor does not fix them.
Anyway. I would be happy if this functionality is implemented as a real part of Traefik.

The plugin has been created by a Traefik maintainer.

@ldez ok - so you suggest to just use that plugin?
Closing this issue then.


Only create volumes with instance.az if cinder.cross_az_attach is False (back port from upstream open change)

Commit 6060888b58db42eea826939852838f9e1c204d2c changed the boot from
volume flow where source != 'volume' (where Nova creates the volume and
then attaches it to the instance during the server create operation)
such that the instance.availability_zone is always passed to the volume
create API.  That was done because if the cinder.cross_az_attach config
option is False then Nova requires the instance and volume AZs to be the
same when attaching the volume.
However, telling Cinder to create a volume based on the instance AZ can
fail if that AZ doesn't exist in Cinder.
Cinder commit b85d2812a8256ff82934d150dbc4909e041d8b31 works around this
by allowing Cinder to be configured to basically not fail if the
requested AZ does not exist. This is not a great solution though if you
have cinder.cross_az_attach=False in Nova because Cinder ignored your
request.  It's also terrible UX for the end user because they get a
NoValidHost when boot from volume fails.  There is a TODO in this change
to address that later (because exactly how cross-service AZs should work
is still being discussed).
This change addresses the gap introduced by always telling Cinder to
create the volume with the instance AZ.  We should only do that when
cinder.cross_az_attach is False, because that's the strict case where
Nova expects them to match.  If cross_az_attach is True, Nova doesn't
care what AZ the volume is in so we shouldn't try and force
Cinder to accommodate the Nova AZ when creating the volume.
Closes-Bug: #1496235
Change-Id: I3d60c084fd71162a0d4835394f8e819723c1b8a7
(cherry picked from commit f9a51b970f688b90baf0ae3ef31d79b3fec02ed1)

LGTM


Explain how items get posted to Message Center and how to report issues with them

Please add a section explaining how items created and posted to Message Center and on how to report issues with individual messages.  It's not clear if an issue needs to be raised with the product team (e.g. drop-down list when creating a Support ticket) or with the product team that is the Admin portal / Health portal / Message Center team.  If the later - how does one do that since there is no obvious selection on the Support ticket Product drop-down for admin items.
[Enter feedback here]

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

ID: 746a06ce-1e81-1e99-2132-28ae2e156cb0
Version Independent ID: 117921fb-38fc-9a43-2cd9-4fb69d206439
Content: Online service activation for Open program FAQ
Content Source: microsoft-365/commerce/licenses/online-service-activation-faq.yml
Service: microsoft-365-business
GitHub Login: @cmcatee-MSFT
Microsoft Alias: cmcatee

@cmcatee-MSFT Pleas help us in resolving this issue. Thanks

@yogkumgit I've forwarded this to the volume licensing PM for a response.

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

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

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


Publish ingest results to a post-ingest SNS topic

Related Issue(s):

#45

Proposed Changes:

Add post-ingest SNS topic provisioning to serverless.example.yml
Publish ingest results to the post-ingest SNS topic
Add post-ingest documentation to the README

I could not have completed the changes in this PR as quickly I did without the guiding example of @jkeifer 's #131. Much of the code in this PR was adapted from that previous PR. Notably the format and attributes included with messages published to the post-ingest SNS topic are the same as those in #131. The changes in this PR should be compatible with deployment using #131 as a foundation.
Thanks to @stark525 for helping to exercise the changes in serverless.example.yml. He was able to run that serverless provisioning in an AWS development environment and verify that the post-ingest SNS topic was created and that the ingest lambda had the ARN of that topic available in the POST_INGEST_TOPIC_ARN environment variable.
PR Checklist:

[x] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.

The only thing I'd like to see somewhere is the URI for the Collection or Item that was ingested. This isn't present in the object that's written to the DB, and only gets added as a self link by the API. Otherwise, and subscriber to the SNS topic just has to know what the STAC API URL is and how they should construct the Collection/Item URI from the collection or collection+item.
Everything else looks great, only one other substantive comment and a few typo fixes.

Thanks for the thoughtful review, @philvarner.
I have pushed up commits that address all of your comments including making use of STAC_API_URL and the existing functions in lib/api.js to include item and collection URIs in the published SNS messages.
This is ready for another look.

I'm sorry for the typo noise. Will work on adjusting my development tooling to catch these.
I will fix these remaining typos and rebase to squash all the fixups.

Updates merged and fixup commits squashed and rebased. Thanks again.


Modify pod_startup_latency not to fail in case of missing schedule event.

Events are best effort and it can happen that some of them are missing, especially when we schedule a lot of events together.
/assign @wojtek-t

/lgtm
/approve


Use Cmdliner

In particular, move https://github.com/usubalang/usuba/blob/cbd2f9338cf1d0f6d030add19dc425ff39505ac9/src/usuba_AST.mli#L74 config somewhere else (it is not AST related).

Config has been moved to its own file in #51 with the creation of src/config.mli and configure.ml generating a src/config.ml file with the definition of the record
Cmdliner is clearly not a priority, closing this issue for now


Implement regex routing for Console

Regex routing is powerful, not limited to HTTP.  This is a placeholder ticket to request the equivalent of Zend\Mvc\Router\Http\Regex for Console.

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Using a basic retainerReferenceFilter causes Node OOM exception

Followed the example from here: https://facebook.github.io/memlab/docs/api/interfaces/core_src.ILeakFilter/#-optional-retainerreferencefilter-referencefiltercallback, but getting Node out of memory exception.
Tried Node 16, 18 and 21 with 8GB memory. The snapshots are approximately 35 mb each -- can be reproduced using Memlab examples.

Seems to be happening during/after this function call: https://github.com/facebook/memlab/blob/3b05576df932b30b491106ffe522bf82d6f62a8f/packages/core/src/lib/Utils.ts#L1818

@singh-sp The reason your leak filter OOMed is probably because your retainerReferenceFilter callback returned true for edges that shouldn't be used. For example, self-referencing edges cause to infinite loop when traversing from a node to the GC root. MemLab's default edge filter excludes these kind of edges, but MemLab's path finder didn't consider the case where external leak filter may bypass MemLab's internal edge filter.
I will make a patch to MemLab so that its path finder will consider the case where external edge filter includes edges that should be bypassed. You can also use the following code to unblock for now:
  retainerReferenceFilter(edge, _snapshot, isReferenceUsedByDefault) {
    // memlab by default removes self-referencing edges
    // and other V8 internal and hidden edges
    if (!isReferenceUsedByDefault) {
      return false;
    }
    // your logic to filter other edges
    // return true or false
  }

Let me know if this fixes the OOM issue.

@singh-sp retainerReferenceFilter is exposed as an API so that you can override the edge filter used by the retainer trace generator. Edges returned as false by retainerReferenceFilter won't be used for calculate the retainer trace.
If you meant to use the default edge filter, there is no need to provide the retainerReferenceFilter callback in the leak filter.

By providing a custom retainerReferenceFilter, are users overriding any default behaviors? (other than what you mentioned)

It will also affect the retainer size calculation and dominator calculation:
https://developer.chrome.com/docs/devtools/memory-problems/get-started

Thanks for the explanation! That's helpful. I plan to use this filter to exclude certain edges (false positive leaks).
I ran some tests, and it looks promising.
I couldn't find anything in docs, but is there a utility function that can parse the returned object ISerializedInfo[] from the findLeaks() function? I plan to display detected leaks in a certain way, and it would be nice to access the leaked objects' names and other metadata. Currently, this object has dynamic keys, making it hard to access nested objects.

I plan to use this filter to exclude certain edges (false positive leaks).

If the edges your filter excluded were the edge that must go through from the GC root to the leaked object, then it will remove the object (i.e., the false positive) from the final result.
A bit more context: MemLab first uses the leakFilter callback to identify memory leak objects (if no leakFilter callback is provided then MemLab uses its default object filter); then MemLab uses retainerReferenceFilter to calculate the retainer trace (i.e., path to GC root) for those identified leaks.

is there a utility function that can parse the returned object ISerializedInfo[] from the findLeaks() function?

Unfortunately, there is no utility function for parsing ISerializedInfo, which is has the following form:
{
 "0: edge0Name object0Name": {object0}
 "1: edge1Name object1Name": {object1}
 "2: edge2Name object2Name": {object2}
 ...
}

If you would like to dive deeper, here the code that generates ISerializedInfo:
https://github.com/facebook/memlab/blob/0f11b29f81d558bb81e5050e68c4a475f6d0978c/packages/core/src/lib/Serializer.ts#L486

Closing this issue as the patch has been released in<EMAIL_ADDRESS>


Mbah Victor Somtochukwu added

i added my name to the list

Please I am sorry for all the trouble we have caused you but count it all joy. Your PR is about to be merged and you will be able to see your name on this url.
To enable me to do this, READ the README again carefully, update your code with the latest and push the update. I am here for you for the next 6 hours before I close the PR so you can initiate a fresh PR which will be easier to merge.

Sir Mbah Victor!
Please open a new PR! Remember to make a branch as you work on your changes!
We look forward to merging your next PR! :partying_face:


Fix title cleaning

Fixes  #194

@vetal4444 good catch thanks. It is fixed now.

I think this pr still bad for cleaning title. View #196. Your method will return 'GHG Protocol for Cities Greenhouse Gas Protocol'.


Breaks vim plugin

All vim keys don't work, and strangely even with vim extension uninstalled pressing enter doesn't insert a newline! Possibly related to https://github.com/monovaldes/vscode-linter-codeclimate/issues/1

hi @suan, i cannot reproduce your issue, CodeClimate analyze by itself requires a load of resources, maybe it ended up blocking your main input in a moment... even so, I've made some improvements in the plugin's performance, feel free to test them out.


Hi, all. Dose the producer cannot reconnect? if I config only one broker

Versions
Please specify real version numbers or git SHAs, not just "Latest" since that changes fairly regularly.
Sarama Version: V0_10_2_0
Kafka Version: 2.11
Go Version: 1.11
Configuration
What configuration values are you using for Sarama and Kafka?
{
"kafka_host":        "<IP_ADDRESS>:9092",
"kafka_topic":       "test",
"kafka_client_id":   "localhost",
"kafka_retry_max":   "3",
"kafka_compression": "lz4",
"kafka_timeout":     "30s",
"kafka_keep_alive":  "0",
"max_message_bytes": "4194304",
}
Logs
When filing an issue please provide logs from Sarama and Kafka if at all
possible. You can set sarama.Logger to a log.Logger to capture Sarama debug
output.
[Sarama] 2019/08/19 18:33:00 client/metadata no available broker to send metadata request to
[Sarama] 2019/08/19 18:33:00 client/brokers resurrecting 0 dead seed brokers
[Sarama] 2019/08/19 18:33:00 client/metadata retrying after 750ms... (3 attempts remaining)
[Sarama] 2019/08/19 18:33:01 client/metadata no available broker to send metadata request to
[Sarama] 2019/08/19 18:33:01 client/brokers resurrecting 0 dead seed brokers
[Sarama] 2019/08/19 18:33:01 client/metadata retrying after 750ms... (2 attempts remaining)
[Sarama] 2019/08/19 18:33:02 client/metadata no available broker to send metadata request to
[Sarama] 2019/08/19 18:33:02 client/brokers resurrecting 0 dead seed brokers
[Sarama] 2019/08/19 18:33:02 client/metadata retrying after 750ms... (1 attempts remaining)
[Sarama] 2019/08/19 18:33:02 client/metadata no available broker to send metadata request to
[Sarama] 2019/08/19 18:33:02 client/brokers resurrecting 0 dead seed brokers
Problem Description

are you sure you can talk to kafka?
try using kafkacat to see if this is not an environment issue.
Example to get cluster metadata:
kafkacat -L -b <IP_ADDRESS>:9092

are you sure you can talk to kafka?
try using kafkacat to see if this is not an environment issue.
Example to get cluster metadata:
kafkacat -L -b <IP_ADDRESS>:9092

Thanks for your reply. The producer could connect to kafka broker at first. After I kill the broker and restart the broker, the producer could not reconnect to the broker. The saram log is printed after I kill the broker.


[Question/Suggestion] SelectInterceptorsWith should support injection

Neither of current implementation of SelectInterceptorsWith supports injection. Any particular reason? Or I just did not find the right one?
I think a SelectInterceptorsWith<T> would be nice.

Could you describe how you'd use this overload, I don't understand the question/request about injection.

container.
...
.Interceptors<Interceptor1>()
.SelectInterceptorWith<InterceptorSelector1>()
;

If the constructor of InterceptorSelector1 has dependencies, it would allow the kernel to inject the dependencies.

Doesn't the second overload allow you to do that.
public ComponentRegistration<TService> SelectInterceptorsWith(Action<ItemRegistration<IInterceptorSelector>> selector)

Container.Register(Component.For<IMyService>()
    .ImplementedBy<MyComponent>()
    .SelectInterceptorsWith(s => s.Service<MyInterceptorSelector>()));

Yes, it does. Thanks :)


Implement server permissions management

Need to figure out how discord permissions work and make the bot have the proper permissions for everything it needs to do.

I mis-understoond how permissions worked and this issue isn't really valid anymore. Default perms are set using the oauth join link.


Allow for forward declaring nlohmann::json

To avoid including json.hpp in header files, I would like to forward declare nlohmann::json as follows:
namespace nlohmann { struct json; }

struct my_stuff
{
    int some_member;
    nlohmann::json encode() const;
    void decode(nlohmann::json const&);
};

However, this doesn't work, since nlohmann::json is a typedef. If you just derived it from basic_json<>, it would work:
struct json : basic_json<>
{
    using basic_json<>::basic_json;
};

What do you think?

See discussions in https://github.com/nlohmann/json/issues/314

@marton78 Please add this idea to #314.

@nlohmann @marton78 That idea was already mentioned in #314.


Batch Update of Campaigns

As a user of the MediaMath SDKs, I'd like to be able to efficiently queue updates to campaigns, so that I can update thousands of campaigns quickly.

Fetch all strategies to be uploaded in one call, gathering version info
setup a queue to efficiently track the update status of each campaign.

-- Original Ticket --
[redacted] is using MediaMath java SDK to upload changes to [redacted] campaigns/strategies.
Currently, it takes 15 mins to update bids for a campaign that has 1000 strategies.
Each update requires the correct "version" marker for the strategy, which can slow down the API calls to fetch.

Fixed in v1.1.0 for Ebay Usecase.


Slow performance of concat()

What is your issue?
In attempting to concatenate many datasets along a large dimension (total size ~100,000,000) I'm finding very slow performance, e.g., tens of seconds just to concatenate two datasets.
With some profiling, I find all the time is being spend in this list comprehension:
https://github.com/pydata/xarray/blob/51554f2638bc9e4a527492136fe6f54584ffa75d/xarray/core/concat.py#L584
I don't know exactly what's going on here, but it doesn't look right - e.g., if the size of the dimension to be concatenated is large, this list comprehension can run millions of loops, which doesn't seem related to the intended behaviour.
Sorry I don't have an MRE for this yet but please let me know if I can help further.

I've noticed this as well, see #7824.

@alimanfoo The slow code stems from my changes in #7400. Obviously the performance drop did not manifest in the tests/ benchmarks.
In #7824 @Illviljan is tackling concat performance.

Awesome, thanks @kmuehlbauer and @Illviljan 🙏🏻


i18n(ko-KR): update routing.mdx

Description (required)

update routing.mdx
#9796

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
guides/routing.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


OpenSimplexNoise not declared in the current scope

Godot version
v4.0.alpha [e8520044e]
System information
ArchLinux (godot-git)
Issue description
OpenSimplexNoise doesn't appear to be available in GDScript, it just throws the error in the title in editor and when running.

Has it been removed or something?
Steps to reproduce

Build latest Godot
Create new project
Create script
@onready var noise = OpenSimplexNoise.new()
Observe error

Minimal reproduction project
OSN_Test.zip

Yes. OpenSimplexNoise has been removed in favour of FastNoiseLite. See more in the PR here https://github.com/godotengine/godot/pull/56718
The online docs still need to be updated I guess.


13.20 update

I'd like to know why it is not necessary an update for this version? #832

Absolutely, how do we fix DLL error without updating?

The offsets have not updated.


Wellspec duplicates removed

Codecov Report

Merging #536 (923625a) into master (bf4799e) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #536      +/-   ##
==========================================
- Coverage   79.31%   79.31%   -0.01%
==========================================
  Files         176      176
  Lines       29250    29254       +4
==========================================
+ Hits        23201    23204       +3
- Misses       6049     6050       +1

Impacted Files
Coverage Δ

resqpy/olio/wellspec_keywords.py
97.67% <100.00%> (+0.04%)
:arrow_up:

resqpy/model/_xml.py
91.13% <0.00%> (-0.28%)
:arrow_down:

Continue to review full report at Codecov.

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


Keyboard Shortcut in Safari conflicts with Responsive Design mode

Describe the bug
I use Safari for development, so the Keyboard Shortcut for  "Enter Responsive Design Mode" (ctrl-cmd-R) is essential for me. Recently (Sonoma?) this now opens the raindrop modal, and there is no way to change this other than disabling the extension?
To Reproduce
Steps to reproduce the behavior:

Make sure  "Show features for web developers" is checked in Settings -> Advanced in Safari
Press ctrl-cmd-R
The raindrop extension modal is shown

Expected behavior
The Responsive design mode feature in Safari should be enabled.
Screenshots

Desktop (please complete the following information):

OS: Mac OS 15.0 (24A335)
Browser: Safari
Version 18.0 (206<IP_ADDRESS>.6)

Hello!
Apologies for the inconvenience.
A fix is on the way, and you can expect it soon (depending on the App Store review speed).
I aimed to unify shortcuts across all browsers but overlooked some edge cases.
The new shortcut for saving a bookmark or highlight is Cmd+Shift+S. Will be no additional shortcuts in the Safari extension.


KYLIN-3597 Fix sonar issues

Refactor methods to reduce the Cognitive Complexity
Reuse "Random" objects
Use SQL binding mechanisms

Can one of the admins verify this patch?


It should be dynamic based on the hardware, or at least arm64.

Hi
Thanks for your feedback.
The PLATFORM variable is intended to be configured for the target architecture as per the build instructions, prior to running build.sh.
Both arm7 and arm64 are supported by multiple Mikrotik router models.. changing it to arm64 is fine.
Regards
Frank

So build.sh is not used in step 6.a (Container registry)?
For people who aren't super familiar like me with Docker, I don't know what files "/container add remote-image=fluent-networks/tailscale-mikrotik:latest" uses from this repo.
If it doesn't use build.sh that should be clarified in the README, and if it uses it I don't see how you can change the PLATFORM when you pull the file directly.

Hi,
That's right, build.sh is not required in Step 6a.
The ghcr.io repo contains linux/arm/v7,  linux/arm64 and  linux/amd64 images. When you issue the pull /container add command, the router requests its required architecture.
You can see these images at:
https://github.com/orgs/Fluent-networks/packages/container/package/tailscale-mikrotik
... click on OS/Arch tab.
The same process as build.sh is used to create & upload these images, just using a multi-architecture build script. I guess I could include that, I just thought it might confuse people who want to create a local image.
Regards
Frank

I've updated README to clarify the build step is only required if Configuration Step 6b is being used.


Install rvm 2.8.0 with ruby 1.8.7

Hi,
I want to install an old rvm version (2.8.0) but I have this error :
gem install capistrano -v 2.8.0
ERROR:  Error installing capistrano:
net-ssh requires Ruby version >= 2.0.
It use to work before ... any idea?

Capistrano has a liberal requirement on net-ssh (i.e we require any version) their latest relea requires Ruby 2.2.
The fix is simple, name the last version of Net-ssh that you can find that works on your Ruby version in your Gemfile as well, and all should work.

No, I did install net-ssh 2.1.4 which works fine with ruby 1.8.7 but capistrano still look for a more recent version.
I fixed my issue by :
wget https://github.com/capistrano/capistrano/archive/v2.15.7.tar.gz
tar -xf v2.15.7.tar.gz
cd capistrano-2.15.7
gem build capistrano.gemspec
gem install --local capistrano-2.15.7.gem
Now I still have an issue with capistrano-ext which require net-ssh too (with ruby >= 2.0.) :(

Also, I don't have a Gemfile in this (very) old project

Just added a Gemfile, everything works fine now :+1:


No proxy

Yesterday was normal but from morning until now.....
Error during fetching the points: Request failed with status code 400
🔍 Processing<EMAIL_ADDRESS>Proxy: No Proxy, Points: 0
⚠ Error during keep-alive request for<EMAIL_ADDRESS>Request failed with status code 400
❌ Keep-Alive Failed for<EMAIL_ADDRESS>

try to get new bearer token from your account


[TACHYON-1248] Undeprecated ASYNC_THROUGH write type and added examples

https://tachyon.atlassian.net/browse/TACHYON-1248

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7843/Failed Tests: 1org.tachyonproject:tachyon-integration-tests: 1tachyon.master.ServiceSocketBindIntegrationTest.connectDifferentAddressTest
Test FAILed.

Jenkins, test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/7845/
Test PASSed.

Do we have any integration tests for ASYNC_THROUGH?

Yes, the test is in FileOutStreamAsyncWriteIntegrationTest

LGTM


Unable to find GDB in default search path and /usr/local/bin/lldb-9

When I use CMake Tools's Debug It says "Unable to find GDB in default search path and /usr/local/bin/lldb-9."
I don't know where to set gdb path for the CMake Tools Plugin. I think this is an issue.
My Env:
macos: 10.15.3
Xcode: 11.3.1
/usr/local/opt/llvm/bin/lldb -v: 9.0.0
cmake: 3.15.3
make:4.3
gdb: 8.3
g++: 9.2.0_3
vs code: 1.42.0
cmake tools: 1.3.0
When I add in settings.json:
"cmake.debugConfig": {
"MIMode": "gdb",
"miDebuggerPath": "/usr/local/bin/gdb",
}
it does not take effect.

When I add in setting.json:
"cmake.debugConfig": {
"MIMode": "gdb",
"miDebuggerPath": "/usr/local/bin/gdb",
}
it does not take effect.

On Windows and Ubuntu I am not able to get any "cmake.debugConfig" settings to be ignored. I will try on MAC shortly, although I don't think something may reproduce only because of this.
I wonder what else is specific to your situation.
You tried CMake Tools 1.3.0, right?

On Windows and Ubuntu I am not able to get any "cmake.debugConfig" settings to be ignored. I will try on MAC shortly, although I don't think something may reproduce only because of this.
I wonder what else is specific to your situation.
You tried CMake Tools 1.3.0, right?

Reply:

The Output When I push CMake Tool's Debug located on the middle bottom of Vs Code.

[build] Starting build
[proc] Executing command: /usr/local/bin/cmake --build /Users/jzwang/Documents/localrepos/codes/CppCmakeGettingStarted/build --config Debug --target CppCmakeGettingStarted -- -j 6
[build] make[1]: Entering directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] make[2]: Entering directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] make[3]: Entering directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] make[3]: Leaving directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] [100%] Built target CppCmakeGettingStarted
[build] make[2]: Leaving directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] make[1]: Leaving directory '/Users/Documents/localrepos/codes/CppCmakeGettingStarted/build'
[build] Build finished with exit code 0
[proc] Executing command: /usr/local/bin/lldb-9 --version
I don't know what the last message means.
2. I tried CMake Tools 1.3.0, 1.1.3.

Finally, I find the lldb path in macOS: /usr/local/opt/llvm/bin.

Make CMake Tools Debug breakstop Success on Xcode 11.3.1!!!
Question:
We have lldb under /Applications/Xcode.app/Contents/Developer/usr/bin/, but we don't have lldb-mi.
Solution:
Compile lldb-mi for Xcode 11.3.1
Steps:
(1) sudo vim /etc/paths
/usr/local/Cellar/llvm/9.0.0/bin
/usr/bin  (this dir must before /usr/local/bin so that compile lldb-mi will use /usr/bin/clang tools)
/usr/local/bin
/bin
/usr/sbin
/sbin
(2) git clone  https://github.com/lldb-tools/lldb-mi.git
(3) cd lldb-mi
cmake .
cmake --build .
(4) sudo cp ./src/lldb-mi /Applications/Xcode.app/Contents/Developer/usr/bin/

Unfortunately Apple doesn't distribute lldb-mi anymore.  The cpptools extension is almost ready to start distributing a copy of it with the extension.  At that point we can start pointing to that one by default.

Unfortunately Apple doesn't distribute lldb-mi anymore. The cpptools extension is almost ready to start distributing a copy of it with the extension. At that point we can start pointing to that one by default.
https://github.com/microsoft/vscode-cmake-tools/issues/1060#issuecomment-585751508

I believe I have a fix for this in PR #1102

CMake Tools 1.4.0 Beta is available on GitHub.
https://github.com/microsoft/vscode-cmake-tools/releases/tag/1.4.0-beta
Please try it out and let us know if you encounter any issues.

This fix is available in CMake Tools #1.4.0 which has been released.

How to configure if I need to compile the program using GCC and debug with GDB.
"cmake.debugConfig": {
"MIMode": "gdb",
"miDebuggerPath": "/usr/local/bin/gdb"
}
using the configuration above the debugger doesn't go to the breakpoint.

How to configure if I need to compile the program using GCC and debug with GDB.
"cmake.debugConfig": {
"MIMode": "gdb",
"miDebuggerPath": "/usr/local/bin/gdb"
}
using the configuration above the debugger doesn't go to the breakpoint.

That should work. Double check if the source code file where you want to hit the breakpoint is compiled with debug information (-g flag). As an experiment, also add "stopAtEntry" inside cmake.debugConfig and set it to true, see if that generates any breakpoint hit at startup.

That should work. Double check if the source code file where you want to hit the breakpoint is compiled with debug information (-g flag). As an experiment, also add "stopAtEntry" inside cmake.debugConfig and set it to true, see if that generates any breakpoint hit at startup.

Make CMake Tools Debug BREAKSTOP Success on Xcode 11.3.1!!!
Question:
We have lldb under /Applications/Xcode.app/Contents/Developer/usr/bin/, but we don't have lldb-mi.
Solution:
Compile lldb-mi for Xcode 11.3.1
Steps:
(1) sudo vim /etc/paths /usr/local/Cellar/llvm/9.0.0/bin /usr/bin /usr/local/bin /bin /usr/sbin /sbin notes: /usr/bin before /usr/local/bin so when step(3) compile lldb-mi will use /usr/bin/clang tools) (2) git clone https://github.com/lldb-tools/lldb-mi.git (3) cd lldb-mi cmake . cmake --build . (4) sudo cp ./src/lldb-mi /Applications/Xcode.app/Contents/Developer/usr/bin/
Set soft link(no need to set cmake.debugConfig anymore)
cd /usr/local/bin ln -s /Applications/Xcode.app/Contents/Developer/usr/bin/lldb-mi lldb-9
Others:
Because I Use g++ under /usr/local/bin/, so after compiling, I modified /etc/paths as: /usr/local/bin /usr/bin /bin /usr/sbin /sbin
refs:
https://forums.developer.apple.com/thread/125482 https://github.com/lldb-tools/lldb-mi

works for me , Great thanks~


$getLeaderboardValue

Type

[ ] Functions: <$getLeaderboardValue>

Description
gets varible values by leaderboard position
Usage:
$getLeaderboardValue[variable name;variable type (user/globalUser/server);asc/desc;position;{servername} - {value} - {server id}]

$getLeaderboardInfo exists

it is not same thing

Type

[ ] Functions: <$getLeaderboardValue>

Description
gets varible values by leaderboard position
Usage:
$getLeaderboardValue[variable name;variable type (user/globalUser/server);asc/desc;position;{servername} - {value} - {server id}]

is that server leaderboard ?

example:
i have "money" varible its value "10" for user1
and its value "5" for user2
when i use this command
$getLeaderboardValue[money;user;asc;1;{servername} - {value} - {server id}]
i will get this message :
username - 10 - userID
// example: it will takes richest persons information

example:
i have "money" varible its value "10" for user1
and its value "5" for user2
when i use this command
$getLeaderboardValue[money;user;asc;1;{servername} - {value} - {server id}]
i will get this message :
username - 10 - userID
// example: it will takes richest persons information

u can do that bye $userLeaderboard[money;asc;{username}-{value}-{id};1;1]
this will return the top user
and for other u can use textsplit
$splitText[position here]
$textSplit[$userLeaderboard[money;asc;{username}-{value}-{id};20;1];\n]


CCITT Group 4 TIFF Images / Pillow and AWS Lambda

I was looking into incorporating this amazing project into something I am working on. Out of the gate, everything worked pretty smoothly for a lot of my use cases but I DID notice that for some reason I get 502 from the lambda function whenever I try to do anything with a TIFF images that has Group 4 compression on it.
For TIFF images with other compressions (like LZW) things seemed to be fine, but the CCITT Fax 4 compression.
To validate that PIL (well, Pillow) could even do this, I took one of the images that had troubles in the lambda function and ran it through the following:
from PIL import Image;
image = Image.open("some_file.tiff")
image.convert("RGB")
image.save("some_file.jpg", "JPEG")

And everything checked out. After that, I had two 🤔 👍

it might've been because I have Pillow 5.2.0 that it worked and the project has an older version
something with perhaps libtiff when it runs in lambda?

I wasn't sure what to look into next, so I thought I would check here to see what I'm doing wrong or how to even go about validating my assumptions.
Also, and pardon my ignorance, but is this the repo that actually builds the lambda function? Because I ran the shell script and it didn't produce that as an output. Either that or there's an assumption that you're building it on some lambda equivalent version of Linux before you do so? This is the first time I've messed with it so I wasn't sure.
Thanks in advance.

Closing this issue as I've made some amount of progress trying to build the lambda function.


x/peggy: increase msg server test coverage

Summary
Increase message server test coverage of x/peggy by testing all the (currently untested) message handlers.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Closing as we are not using Peggy anymore


psh: Add network utilities

Commit misses JIRA ID

Kinda too much line breaking in this code, makes is less readable. I think that 120 chars wide line is ok limit, we have now huge 16:9 or 21:9 LCDs, so we can afford that.

What  about, for example, such line ( 207 chars wide ):
plo/armv7m7-imxrt106x/syspage.c

Rebased onto #34.

CI found some warnings, not triggered by my build system. I'll fix this in a moment.


Swap the order of directories in --dir

This commit changes the --dir argument on the wasmtime CLI to be HOST::GUEST rather than GUEST::HOST. This matches Docker for example and is a little more consistent with only --dir path where the first argument is always treated as a host directory.
In terms of breaking-ness the movement from --mapdir to --dir hasn't been released with Wasmtime 14 yet so my hope is that this can land on both main and Wasmtime 14.0.0 before it's released to avoid any breakage other than existing scripts migrating from --mapdir to --dir.

Matching docker makes sense to me too. As a historical note, the original syntax here wasn't something anyone thought about deeply, it was just something simple to get basic use cases working, that ended up sticking around.

Is there a way this arg could be extended in future (in a backward-compatible way) to support an optional verbose form? E.g. --dir host=a;guest=b, or even --dir host=a guest=b, where you can put host and guest in whatever order you want?
I'm trying to think through how the delimiters/escaping would work, and I realised I don't understand how it works today — e.g. what happens if your path contains ::. Is the answer "don't have silly paths"? 😅
I ask this because these kinds of order-dependent syntaxes (Docker volume mounts, SSH port forwarding, etc.) have caused me seemingly endless confusion and suffering, and I'm not sure if I'm uniquely vulnerable to them or if this is something that lots of other people would like to be able to avoid as well.

I personally agree with @jeffparsons, although it's a bit too late for Wasmtime 14/15 to change the syntax here that much. This at least makes things "no worse" on the understandability front hopefully.
I've opened https://github.com/bytecodealliance/wasmtime/issues/7309 to continue discussion here now that this is merged, since I think there are still improvements to be had!


Yelp

Ref: Conversation #4801 / Nathan

Ref: Conversation #5800 / Thomas K

Ref: Conversation #7335 / John


docs: FreeBSD Linux Binary Compatibility

Closes #1877
@Thediscordian do you think this is clear enough?

lgtm :+1:


Holdings data not yet added

Yahoo! is finally giving holder data which can be pretty valuable to measure what I call the "manipulability" of a stock.  See for example: https://finance.yahoo.com/quote/MSFT/holders?p=MSFT
This is a feature request, any chance we will see it in the next release, as I saw many nice new data was added in 0.1.50?

Added in 0.1.51
stock.major_holders
stock.institutional_holders


fix: another mishandling of commit ids. cannot use shallow clone for that case

ref: #1642
issue submitted through slack thread
note: cannot modify test case for commit id at the moment ... all the example apps look to be at a head commit needed to actually work for epinio v1.
that said, it was locally tested with the example app and commits given on slack, and worked for that.

Codecov Report

Merging #1661 (1ef9b8f) into main (5d25c71) will decrease coverage by 4.73%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1661      +/-   ##
==========================================
- Coverage   62.93%   58.20%   -4.74%
==========================================
  Files         168      168
  Lines       12497    12501       +4
==========================================
- Hits         7865     7276     -589
- Misses       3635     4225     +590
- Partials      997     1000       +3

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
38.06% <0.00%> (-0.47%)
:arrow_down:

acceptance-cli
57.15% <100.00%> (-0.94%)
:arrow_down:

unittests
8.69% <0.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/api/v1/application/importgit.go
68.75% <100.00%> (+2.08%)
:arrow_up:

pkg/api/core/v1/models/namespace.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/api/v1/application/match.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/api/v1/configuration/match.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/api/v1/appchart/match.go
0.00% <0.00%> (-80.65%)
:arrow_down:

internal/api/v1/namespace/match.go
0.00% <0.00%> (-80.65%)
:arrow_down:

internal/api/v1/service/match.go
0.00% <0.00%> (-75.68%)
:arrow_down:

internal/api/v1/service/catalogmatch.go
0.00% <0.00%> (-74.29%)
:arrow_down:

internal/api/v1/application/portforward.go
0.00% <0.00%> (-70.97%)
:arrow_down:

internal/api/v1/configuration/replace.go
0.00% <0.00%> (-60.38%)
:arrow_down:

... and 39 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 5d25c71...1ef9b8f. Read the comment docs.


Route get remounted

Related: #4578
Version
4.3.1
Test Case
With render: https://codesandbox.io/s/7jz5ozwky0?expanddevtools=1&hidenavigation=1&view=preview
With component: https://codesandbox.io/s/l5wok42x49?expanddevtools=1&hidenavigation=1&view=preview
Steps to reproduce

Click /a. Notice console Component A DidMount.
Click /b. Notice console Component B DidMount.
Click /a. Notice console Component A DidMount.

Expected Behavior
Component should not be re-mounted. As per docs (https://reacttraining.com/react-router/web/guides/basic-components):

You should not use the component prop with an inline function to pass in-scope variables because you will get undesired component unmounts/remounts.

Actual Behavior
Component is being remounted on every route change, regardless of using component or `render.

This is expected behavior. You have two separate Route's and two separate components, so each one is coming into the tree with each route change. This will cause it to not display the message more than once:
          <Switch>
            <Route path="/a" component={ComponentA} />
            <Route path="/b" component={ComponentA} />
          </Switch>


Unspecified code_sign_arg

Hi,
I try to build my app project with iOS-Build-Kit.
I have one issue for signing an IPA.
Checking the code at
https://github.com/adamwaite/iOS-Build-Kit/blob/master/lib/ios_build_kit/tasks/create_ipa.rb#L69
code_sign configuration is set to code_sign_arg.
However, code_sign_args does not being use at
https://github.com/adamwaite/iOS-Build-Kit/blob/master/lib/ios_build_kit/tasks/create_ipa.rb#L71
In my environment, without specified the sign, the generated IPA will have old provision information.
I guess my Xcode's provisions are outdated, but I want to update my provisions handy without Xcode.
Are there any reasons for not specifying the sign in xcrun?

Hey @Kudo. This should be sorted now. Only took me 7 months. I'll get on to your PR soon, thanks!

Thanks @adamwaite.
After a little testing, I found there are still some code sign inconsistency between xcrun and xcodebuild.  For example, I use my developer provision in xcode, then during xcodebuild step, it will sign with the developer provision implicitly.  If later for xcrun code sign step, I give another different provision, when upload to app store, they will complain for inconsistency.
My workaround is to do code sign in xcodebuild step but not xcrun step.  One limitation for xcodebuild is that it only accept provision included in Xcode, i.e. Before build, we should refresh Xcode's provisions handy.
BTW, there are still some other changes from me, but the design is totally different from your, so I did not send PR for you.  Here are they and you may able to have some reference and I am likely to see if the changes can be merge in mainstream.

690da06 for update Decorated-* in project plist during build time, so we don't need to modify project plist in VCS(git).
33d0baf for Asset Catalogs support
ba50505 876a275 as what I mentioned to do code sign in xcodebuild.

Thank you.


Vkotha master

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Ping....
Are you waiting on me? is there something blocked from my side?

Hi @vivekkothakota - sorry about the late review - I requested just a couple of minor changes and we're good to go :)

@vivekkothakota - This PR is missing ARM sign-off. Since many days have passed, please let me know if you prefer to:

follow up with ravbhatnagar and wait, or
I can merge the PR without ARM’s team sign-off. Please note that even after merge, you should follow up with ARM team regarding ARM issues and fixes may impact the specs and/or generated SDKs.

ping @ravbhatnagar
Thanks!

Thank you @annatisch. I did send an email to @ravbhatnagar, lets wait for a day for his response before merging.

Thanks @ravbhatnagar - looks like we're good to go? :)

Looks good!


fixed spelling error

Hi @tspascoal, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

@noamben Please review this contribution. Thanks.

@noamben Thanks for the review and approval. I'll merge this pull request. For subsequent pull requests, please sign off so that automation adds the "ready-to-merge" label. Thanks.


Fix missing dependencies building with make

Fixing issue #6 and switching to gvt!

@atosatto Thanks!

@atosatto anyway this doesn't seem to have fixed the submodule problem:
If you look at
https://github.com/fntlnz/doenter/tree/master/vendor/github.com/goburrow
serial  is considered as a submodule.
while go-homedir is not
https://github.com/fntlnz/doenter/tree/master/vendor/github.com/mitchellh

That's why I've implemented this workaround:
https://github.com/fntlnz/doenter/blob/master/Makefile#L19-L21
:stuck_out_tongue_closed_eyes:
I don't know how to fix this. It seems to me something related to Github...
Is it possible?!

Yes I think that is related to the fact that gvt removes *_test.go files from the source repository so the mitchellh's go-homedir, which have tests is ok while goburrow's serial is not because github recognises it as the same repository and treat it as a submodule.


Send input/output mapping information to Python engine

This PR is part of the effort to align the Python engine implementation with the Java engine implementation, after many refactorings in the Java engine are merged to master.
In this PR, the Python operator initialization receives two new parameters: input_ordinal_mapping, the mapping from an input link to an input port ordinal, and output_ordinal_mapping, the mapping from an output link to an output port ordinal. Additionally, the id of the operator is also passed to Python.
Before this PR, the Python engine does not know which link corresponds to which port, so it assumes the first link that receives data messages is the first port. After this PR, this hacky implementation is fixed.

Update Apr. 17 2023: @zuozhiw will merge this.

maybe we can merge this PR after #1891 to include broadcast as one more option in the frontend.


Using latest version of discovery

I wonder how we should do this. This will make the guzzle adapter incompatible with the mailgun SDK for example.

I think we should merge this and release a new patch version. I'll send a PR to most other libraries depending on discovery to bump them to requiring 0.9 and make them release a patch version as well.

On 25 Jun 2016, at 22:17, Márk Sági-Kazár<EMAIL_ADDRESS>wrote:
I wonder how we should do this. This will make the guzzle adapter incompatible with the mailgun SDK for example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

That will still easily end up in version conflict. What if we use version >0.8,<1.0?

That's probably problematic as well, as we can't make sure that BC will not break in MJZ versions.

yes. Sure it might be a version conflict. But that means we have to bump all discovery 0.8 to 0.9.
I am currently waiting for a new patch release of guzzle5 adapter because I do already have version conflicts. https://github.com/Happyr/LinkedIn-API-client/pull/106

Alright then, let's get this merged.

v0.6.0 is tagged

Thank you


Integration seems to be broken

Getting a 500 Internal Server Error when trying to configure the integration for the first time.

2024-03-31 02:09:20.360 ERROR (SyncWorker_9) [homeassistant.util.package] Unable to install package pysnoo2==0.3: ERROR: Could not find a version that satisfies the requirement pysnoo2==0.3 (from versions: 0.3.3, 0.3.4)
ERROR: No matching distribution found for pysnoo2==0.3


Add missing initialization for tableKeys field

13d4cad refactored Configure(), but in the process failed to initialize the tableKeys field, which ends up panic()ing due to an uninitialized map when scanning the table primary keys.
This commit fixes this. :)
Description
Please include a summary of the change and what type of change it is (new feature, bug fix, refactoring, documentation).
Please also include relevant motivation and context.
List any dependencies that are required for this change.
Fixes # (issue)
Quick checks:

[x] I have followed the Code Guidelines.
[x] There is no other pull request for the same update/change.
[x] I have written unit tests.
[x] I have made sure that the PR is of reasonable size and can be easily reviewed.

@pgguru Thank you very much for this contribution!
It would be good to have this test verified in a test (the lack of which made us miss this). An integration test like we have in destination_integration_test.go that just opens the source connector should be fine. If you have some time for writing the test, that would be great. But if not, I can get to it too.

Hi, pushed a new file source_integration_test.go (which seemed more sensible to me than including in destination_integration_test.go).  Unfortunately was not able to run it locally:
$ make test
# run required docker containers, execute integration tests, stop containers after tests
docker compose -f test/docker-compose.yml up --quiet-pull -d --wait
[+] Running 1/1
 ⠿ pg-0 Pulled                                                                                14.9s
[+] Running 3/3
 ⠿ Network test_default     Created                                                                                0.1s
 ⠿ Volume "test_pg_0_data"  Created                                                                                0.0s
 ⠿ Container test-pg-0-1    Healthy                                                                                11.9s
go test  -race ./...; ret=$?; \
                docker compose -f test/docker-compose.yml down --volumes; \
                exit $ret
go: downloading go1.22 (darwin/amd64)
go: download go1.22 for darwin/amd64: toolchain not available
[+] Running 3/3
 ⠿ Container test-pg-0-1  Removed                                                                                0.2s
 ⠿ Volume test_pg_0_data  Removed                                                                                0.1s
 ⠿ Network test_default   Removed

At this point, pushing so CI can validate, or if you have the ability to validate/fix locally you should be able to push to this branch as well.
Thanks!

@pgguru Thanks! I pushed a small change to the integration test, passes locally.

@pgguru thanks for your contribution!! We really appreciate it. This will go live in https://github.com/ConduitIO/conduit-connector-postgres/releases/tag/v0.7.1


Rename parameter_values to parameter_vectors in primitives

What should we add?
I have been bothered lately by the fact the we call parameter_values to a Sequence[Sequence[float]] in the primitives, and I think it is leading to confusion.
I keep coming across with loops like:
for parameter_value in parameter_values:
    ...

which seems intuitive but it is semantically wrong, because the type of parameter_value here is Sequence[float].
I wanted to propose renaming ~paramenter_values~ to parameter_bindings. This will make everything more clear imho:
for binding in parameter_bindings:
    for value in binding:
        ...

Since we now also support just passing the parameter_values (i.e. Sequence[float] instead of a doubly-wrapped) with #8704, is this still an issue? 🙂 Personally, bindingsto me sounds like it would be a mapping/dict of parameter: value pairs, and values sounds more like a list 🤔

it would be a mapping/dict of parameter

I feel like same.

I see your point, but I would not feel very comfortable using the supported exception (single entry) to name the general case (multiple entries), unless we did the same for the other args as well (i.e. circuit and observable  instead of ~circuits~ and ~observables~). As a matter of fact I would be perfectly fine with this approach as well.
Consistency is key for clarity.

I thought that $\theta$ itself is a parameter value even if it's a vector, so the list $[\theta_{0}, \theta_{1}...]$ is parameter values.
hmm... difficult. how about parameter_vectors?

Interesting @ikkoham! I kind of like it tbh, although I probably need to get used to it. It is a good middle ground 🙂

It's a good point with observable(s) and circuit(s)! Reading that actually it to me it would be clearest to use
circuit observable parameter_values

but I'm not sure it's an option to rename the all arguments based on this 😅

transpile's argument is circuits. We used it for consistency.

The more I think about it, the more I like @ikkoham 's idea of calling it parameter_vectors!

I have discovered a ParameterVector class under qiskir.circuit which may be interesting to analyze in this context. I would probably refrain form using it.

The ParameterVector is just a indexed vector of parameters used e.g. in variational ansatz circuits (or even mostly there?). By "refrain from using it", do you mean refrain from using parameter_vectors as replacement of parameter_values? Or refrain from using a ParameterVector? 🤔

I meant refrain from using the ParameterVector class. But no strong opinions if you think otherwise, especially after your nice explanation! 😊

The ParameterVector nicely solves some problems we had with parameterized ansätze, like sorting by index if the name is the same (to sort a b[0] b[1] b[2] ... z instead of a b[0] b[1] b[10] .... z) and adjusting the number of parameters if the ansatz is resized (which e.g. TwoLocal circuits allow) 😄 I see how parameter_vectors is confusing as name, maybe we need to choose something else in that case 🤔

I honestly like parameter_vectors, I just don't want it to be confused with the class 😊 and I agree that parameter_bindings may be more confusing
There are only so many synonyms haha but happy to consider alternatives, we have time until the next release

I close this because the discussion is obsolete


Why Gumbel-Softmax relaxation applied in sample generation?

Hi, thank you for sharing the code!
But I have a question for sample generation... In my opinion, Gumbel-Softmax relaxation is used for directly passing gradients back to generator. However, why it is still applied when generating samples?
Thanks ahead!

Because we replace the sampling with the Gumbel trick. We keep using the Gumbel trick instead of sampling directly during sample generation just in order for maintaining the same pipeline between training and testing. Otherwise, we may loss randomness.

Thank you for your reply!
I am still a little confused about "randomness".
Now I trained a model on coco data with a high temperature, but if I use it for generation, should I still keep a high temperature? It seems to generate some bigrams that are illogical.
Maybe I should express in a another way, I can understand the sample process from softmax which still matches the inner distribution of texts, but I cannot understand the randomness from Gumbel trick, especially when I use a high temperature.

Sorry for having not made it clear. When I said “we keep using the Gumbel trick instead of sampling directly during sample generation”, I didn’t mention we still use the softmax part for sample generation. You can see the code, where next_token for the sampling.

Okay, I understand now. Thank you so much for your time and your patient explanation!
I will close this issue then. :)


IsarError: Could not initialize IsarCore library for processor architecture "android_arm". If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies. Invalid argument(s): Failed to load dynamic library 'libisar.so': dlopen failed: library "libisar.so" not found

IsarError: Could not initialize IsarCore library for processor architecture "android_arm". If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies. Invalid argument(s): Failed to load dynamic library 'libisar.so': dlopen failed: library "libisar.so" not found
in
Samsung J2 Android Version : 6

Duplicate of https://github.com/isar/isar/issues/1095
Please check other issues before opening new one.

Already added dependencies still getting error

This is fixed in v3.1.0 which will be available soon on pub.dev.

v3.1.0 is available but its not solved

v3.1.0 is available but its not solved

Same here, got this one from Crashlytics:
IsarError: Could not initialize IsarCore library for processor architecture "android_arm". If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies. Invalid argument(s): Failed to load dynamic library 'libisar.so': dlopen failed: library "libisar.so" not found

v3.1.0 is available but its not solved

Same here, got this one from Crashlytics: IsarError: Could not initialize IsarCore library for processor architecture "android_arm". If you create a Flutter app, make sure to add isar_flutter_libs to your dependencies. Invalid argument(s): Failed to load dynamic library 'libisar.so': dlopen failed: library "libisar.so" not found

Any update on this issue? Getting same error @simc

For me this was being caused by not being able to get the version number.  I resolved it by going into XCode and making this setting change:


DBeaver Community Edition Cannot Connect to DuckDB

Description
error: Serialization Error: Failed to deserialize: field id mismatch, expected: 201, got: 103
DBeaver Version
Community  <IP_ADDRESS>405191503
Operating System
Win 11
Database and driver
duckdb v0.10.3 70fd6a8a24
duckdb_jdbc-0.9.2.jar
dbeaver-diagnostic-info-1717206723769.zip
Steps to reproduce
Steps to reproduce
Launch the "Connect to a database" dialogue
Select DuckDB
Select DuckDB db file
Press "Test Connection..."
Additional context
No response

Hello @pavlokurochka,
It seems that you have quite an outdated driver (the latest version is 0.10.3), try upgrading it.

Please let me know if it helps.

Upgrading the diver this way worked. Тhanks a lot!


get muliple pages from oai harvest

This uses the DataSource interface to harvest multiple pages from an OAI feed.

The process of collecting HTTP responses from the OAI feed cannot be parallelized due the nature of the OAI feed.  To reduce memory load prior to parallelization, the responses are treated as Strings rather than XML objects.  They are parsed into XML object after parallelization has occurred.
The OaiResponseProcessor has been changed from a class to a static object in order to make it easily serializable.
At this time, the only two methods of OaiResponseProcessor that are being used are getRecordsAsTuples and getResumptionToken.

This has been tested locally with the following options:
--executor-memory "1g"
--driver-memory "2g"
endpoint: http://oai.artstor.org/oaicatmuseum/OAIHandler
metadataPrefix: oai_dc
verb: ListRecords

This addresses ticket DT-1279.

Great work!


How to REMOVE / UPGRADE Best Practice Rules in Visual Studio?

I just tested to add my own Best Practice Rules to Visual Studio using these instructions:
https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/dev-tools/author-best-practice-rules
How do I REMOVE (not only uncheck) the rules once installed?
(There is no UnInstall-button - only Install.)
 :-)
B, If I upgrade the rules, I can´t install the new version since "it already exists". It does not help to delete old DLL. I can rename the new DLL (and the classes within, to avoid duplicates), but the system vill get messier and messier for each new version. :-(
Best Regards,
/ Daniel Risman

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

ID: 68a08e21-dcff-f7be-1173-59496cef4038
Version Independent ID: 4b1dc605-701d-151f-11f5-d8eb9cc8ae19
Content: Write best practice rules - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/author-best-practice-rules.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @pvillads
Microsoft Alias: pvillads

@pvillads, please review.

Bug #470084 logged to track this issue.

Hello,
Just wanted you to know that I’ve found a working ”work-around”. I just have to close down Visual Studio and manually delete the old DLL from:
K:\AosService\PackagesLocalDirectory\bin\BPExtensions
(Still it would have been nice to both install AND uninstall Best Practice-DLL:s from inside Visual Studio.)
Med vänliga hälsningar / Best regards
Daniel Risman
+46 73-629 09 01<EMAIL_ADDRESS>Hornsgatan 174  SE-117 34 Stockholm
[cid:image005.png@01D65A21.6B718270]http://www.orango.se/
Från: Tonya Fehr<EMAIL_ADDRESS>Skickat: den 14 juli 2020 16:03
Till: MicrosoftDocs/dynamics-365-unified-operations-public<EMAIL_ADDRESS>Kopia: Daniel Risman<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Ämne: Re: [MicrosoftDocs/dynamics-365-unified-operations-public] How to REMOVE / UPGRADE Best Practice Rules in Visual Studio? (#2562)
Bug #470084 logged to track this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics-365-unified-operations-public/issues/2562#issuecomment-658198675, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQGBAU4XXFBKGNZMJ7BFNBLR3RQRHANCNFSM4ORNLWZA.

Thanks for the update.


Index Out of Range

New EOW account for 2 water meters.    After installing the integration I get 2 leak detection entities and 2 water consumption entities but nothing has updated.     After checking the logs I see:
Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 239, in _async_refresh     self.data = await self._async_update_data()   File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 195, in _async_update_data     return await self.update_method()   File "/config/custom_components/eyeonwater/__init__.py", line 50, in async_update_data     await eye_on_water_data.read_meters()   File "/config/custom_components/eyeonwater/__init__.py", line 102, in read_meters     await meter.read_meter(self.client)   File "/config/custom_components/eyeonwater/eow.py", line 102, in read_meter     return self._parse_reading_data(data)   File "/config/custom_components/eyeonwater/eow.py", line 120, in _parse_reading_data     self.reading_data = meters_read[METER_READ_FIELD][meter_index][NEW_LATEST_READ_FIELD] IndexError: list index out of range

Guess this integration isn't going to be supported.

I've not really tried to investigate.    I'd be willing to collaborate on the issue in a couple weeks--have some vacation coming up.

Hi @kdeyev I also have two meters in my account (culinary and secondary/irrigation) and would love to collaborate to get this integration working.
I'm happy to help however you'd like including working with you to login and parse the output from my account.

Hi @kdeyev , I'm not a developer at all, so hopefully I'm doing this right.   I cloned the latest code from Git, updated my userID/password and ran test.py using PyCharm CE.
Here's the output:
/usr/bin/python3 /Users/mw/PycharmProjects/eyeonwater/custom_components/eyeonwater/test.py
2 meters found
meter 190240692 shows 354908.13
meter leaks: False
Traceback (most recent call last):
File "/Users/mw/PycharmProjects/eyeonwater/custom_components/eyeonwater/test.py", line 25, in
loop.run_until_complete(main())
File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
return future.result()
File "/Users/mw/PycharmProjects/eyeonwater/custom_components/eyeonwater/test.py", line 17, in main
await meter.read_meter(client=client)
File "/Users/mw/PycharmProjects/eyeonwater/custom_components/eyeonwater/eow.py", line 102, in read_meter
return self._parse_reading_data(data)
File "/Users/mw/PycharmProjects/eyeonwater/custom_components/eyeonwater/eow.py", line 120, in _parse_reading_data
self.reading_data = meters_read[METER_READ_FIELD][meter_index][NEW_LATEST_READ_FIELD]
IndexError: list index out of range
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x105e2a130>
Unclosed connector
connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x105e76040>, 1.385297083), (<aiohttp.client_proto.ResponseHandler object at 0x105e7eca0>, 2.419574375)]']
connector: <aiohttp.connector.TCPConnector object at 0x105e2a4f0>
Process finished with exit code 1

@whitema Thank you.
It seems that the first meter worked as expected and the problem with the second meter:
2 meters found
meter 190240692 shows 354908.13

I still have no idea how to fix it, since this integration is a result of reverse engineering.
I'm going to be partly unavailable for the next month, but will do my best for fixing the issue with 2 meters.
It would be super helpful if you could temporarily share access to your account. If you are ok with it - please reach me bt kn.deev at gmail.
Have a good one.

@kdeyev Happy to share it.

The fix is published in v0.0.2-beta.1

v0.0.2 has been released


cleanup docker containers after graceful termination

When terminating the runner by pressing Ctrl+C (SIGINT), it prints the following in console:
CTRL+C received, stopping accepting new jobs
Canceled stopping

So, it seems it terminates gracefully, but it leaves docker container running after itself:
github@xu4r1:~/gha-act-runner$ docker ps
CONTAINER ID   IMAGE                   COMMAND                  CREATED         STATUS         PORTS     NAMES
fd2a6f7dd779   arm32v7/debian:buster   "/usr/bin/tail -f /d…"   9 minutes ago   Up 9 minutes             act-ci

I think it should stop and remove all the docker containers it spawns during a runtime.

still happens with latest snapshot:
^CCTRL+C received, stopping accepting new jobs
Canceled stopping
github@xu4r1:~/github-act-runner$ ls
agent.json  auth.json  cred.pkcs1  github-actions-act-runner-linux-arm
github@xu4r1:~/github-act-runner$ docker ps
CONTAINER ID   IMAGE                    COMMAND                  CREATED         STATUS         PORTS     NAMES
f43912b86532   igagis/raspbian:buster   "/usr/bin/tail -f /d…"   6 minutes ago   Up 6 minutes             act-ci

I have the feeling, act ignores the config to always remove the container after job exit. (added in the first snaphot of today)
In one issue of act I read the container is kept for debugging purposes.
In any case the container should be removed if you start a job with the same name as the last one. (This has caused container id collisions in act for me)
My github actions wrapper don't know the containerid, act have to be patched.