Uptime included microseconds

Greetings,
The uptime display currently includes the microseconds to remove the decimal part use this:
uptime = datetime.now().replace(microsecond=0) - datetime.fromtimestamp(psutil.boot_time())
Solution given on the Elite Dangerous Community Developers discord.

That's great, thanks for your tips.


Completed Jobs' Task Status are showing as "Running"

In v0.12.0, we introduced the new Task Status Column for users to indicate status of each tasks used by job.
The problem is: some tasks in the completed (succeeded, failed, stopped) jobs are showing as still running.

@sterowang @fanyangCS @qfyin -
From UX perspective, tasks in Succeed and Failed jobs should not be "Running", we need to fix this.
Per @yqwang-ms  - From technical perspective, PS task role are not able to exit by themselves, they are killed on job completion which made launcher always get the latest PS task status as Running.
We need a technical decision about how to fix it. Please suggest.


assert HTTP status code from goose request?

Are we not supposed to assert HTTP status codes from a Goose request and just view the status_codes in the metrics from the config?

You can see validate_and_load_static_assets() for an example of how to work with this:
https://github.com/tag1consulting/goose-eggs/blob/main/src/lib.rs#L780

Should we be retrieving static elements which would  be hosted on a CDN? Wouldn't we just want to validate our system? I was thinking if we made a POST and received 200 that would be seen as a pass.
And also I'd be reticent to have a function that contains 'and' as a function should just do one thing.
We could add an error message on Err() couldn't we and I was thinking of the HTML status code and not  the HTML response as a string?  But my thinking may be incorrect and confused.

There are many ways and things to load test. One of the purposes of load tests is to test assumptions: launches can go very badly if in fact the CDN isn't properly configured or if things are flushed too frequently etc. We often leverage Goose to validate our assumptions. That said, Goose is a framework so it's up to you if it just validates the response codes or more.
The linked function is part of a helper library called Goose Eggs, and solves a direct need we have which is to load an HTML page AND then also all contained "static" elements (many of which may actually be dynamically generated). The AND function joins several smaller functions from the Goose Eggs library that can instead be called individually if desired, but this can result in a lot of boilerplate in your load test.
The HTML status code is one of MANY things that Goose can (and does) validate. In some cases you can blindly trust the response code, but it's not uncommon to receive say a 200 OK that is actually a Load Balancer error message, etc. So it's critical that Goose can also validate Headers, Cookies, HTML Titles, and arbitrary text (including JSON) returned by the request. See goose_eggs::ValidateBuilder for a list of things we frequently validate in load tests to be sure things are working as we believe.

That was a good answer.


AppBar content has no default left padding after v0.0.15

Steps to Reproduce

Create new app with wizard on IntelliJ
Run app with no changes on Android emulator

0.0.14
0.0.15

Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-US, channel alpha)
    • Flutter at /Users/germannodomingues/development/workspace/tools/flutter
    • Framework revision 701d534ba9 (8 hours ago), 2017-09-12 14:01:51 -0700
    • Engine revision 31d03de019
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.1)
    • Android SDK at /Users/germannodomingues/Library/Android/sdk/
    • Platform android-26, build-tools 26.0.1
    • ANDROID_HOME = /Users/germannodomingues/Library/Android/sdk/
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.3, Build version 8E3004b
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Community Edition (version 2017.2.3)
    • Flutter plugin version 17.0
    • Dart plugin version 172.3968.27

[✓] Connected devices
    • Android SDK built for x86 64 • emulator-5554                        • android-x64 • Android 5.1.1 (API 22) (emulator)
    • iPhone SE                    • 3EF9C14A-6D08-4B7A-A1EA-C72AAA2A9419 • ios         • iOS 10.3 (simulator)

Adding padding manually makes the title aligned as before (though probably it's not the solution):
    AppBar appBar = new AppBar(
      title: const Padding(
        padding: const EdgeInsets.only(left: 16.0),
        child: const Text("Flutter Demo Home Page"),
      ),
    );

This was a deliberate decision to fix: https://github.com/flutter/flutter/issues/11963 . There are scenarios where app wants to take over the entire app bar. This was previously not possible due to automatic addition of margin even if there was no leading widget in the app bar.
Material Design isn't clear on what to do if there is no menu or back button in the toolbar. Therefore, we decided to leave it to the app to decide (e.g. some apps prefer 32dp padding).

We may have to put the padding back. This affects the default template app in an ugly way.

Synced up with @yjbanov, I am working on a fix for this.

Thanks for the explanation @mehmetf .
A solution I could think of is adding a "padding" parameter to AppBar with a default value of 16dp for example, which could be overwritten by those who don't want to use it.

Thanks for the suggestion. Unfortunately this is not padding in the traditional sense. However, we are indeed going to allow the AppBar to override this value and default it to 16dp. Stay tuned.

Hi, was this fixed? If yes whats is the way to go now? Adding a padding to title property or something else?

This was reverted to the old behaviour of having a margin. I'm not sure if we've rolled the fix to alpha yet, if this is affecting you try switching to the master channel (flutter channel master).


Siikomik change URL

Source information
Siikomik
Source language
Indonesia
Source new URL
https://siikomik.cc/
Other details
No response
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

Add a :+1: reaction to issues you find important.

The old domain is still working. Did they announce the change somewhere?


Introduce a timeout (possibly via config) for large queries

Related to https://github.com/frectonz/sql-studio/issues/14, but it may be worth making it possible to cancel queries that are taking too long to run - in the case it's a poorly optimised query/database

Maybe covered by https://github.com/frectonz/sql-studio/commit/82ab1f8ddf1e73dde960f35ee6021fd2549b8382


[charts/csi-powerflex] Added env for kube node name

Is this a new chart?
No
What this PR does / why we need it:
This PR includes addition of env X_CSI_POWERFLEX_KUBE_NODE_NAME that is required in the PowerFlex driver to fetch the node labels.
Which issue(s) is this PR associated with:

https://github.com/dell/csm/issues/1086

Special notes for your reviewer:
Checklist:
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[ ] Variables are documented in the chart README.md
[x] Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

@suryagupta4 : Do installation wizard needs changes for this variable?

@suryagupta4 : Do installation wizard needs changes for this variable?

@shanmydell : No, installation wizard does not require this change.

@suryagupta4 : Please update the templates in Installation wizard not in UI, approving for the changes in helm charts


use of undeclared crate or module extra

I just finished up the Chumsky tutorial (great stuff btw), but I'd like to play around with the errors a bit more. I see that for nano_rs and json, Rich is used. However, I can't seem to import it (or find the declaration in the source). My Cargo.toml has chumsky = "0.9.2" as dependency

The examples on the repository are for version 1.0.0-alpha.1 and above - they've been updated in step with the next planned release.
If you look at the 0.9 release tag, it should have a link to the tagged commit, and if you look at the repository for that commit the examples should match the version you're using.


Firebase deploy does not complete from Travis CI

See the build logs here and here.  Looking at the Firebase deployment history, it looks like there have not been any Firebase deployments sent from Travis-CI, but have been deployed manually.
When running the deploy script locally, the console output is as follows:
donejs - deploying to 'firebase'

donejs - deploying to 'bitballs-donejs'. Please be patient.
donejs - successfully deployed to 'bitballs-donejs'.

In the build logs, the success message is missing. It seems like the Travis shell is exiting before the script completes.

Looking into this, the deploy doesn't actually log if there are errors:
deploy(packageJSON, packageName).then(function() {
    process.exit(0);
}, function(e){
	process.exit(0);
});

when I put a console.log there, I see an error.  We should do something similar to at least let people know what's wrong.

The logging error has been fixed in v0.4.1 and the issues was that we needed to provide a Firebase CI token which you can get via:
node_modules/.bin/firebase login:ci

And then running
travis-encrypt --add -r <your-username>/<your-repository> "FIREBASE_TOKEN=\"<token>\""

I updated the guide accordingly in https://github.com/donejs/donejs/pull/565.


replay: fall back to qlog

requires: https://github.com/commaai/openpilot/pull/22361
@adeebshihadeh do you want to do this silently or require confirmation/command line flag like plotjuggler?

I like the juggle script's confirmation. juggle.py doesn't do this, but let's make the command line flag use rlogs when possible and fill in the gaps with qlogs.


Add cluster connection settings user input in cluster modal

Implement the option to customize the ansible ssh connection settings, specifically the user that is used during the ssh connections.
This is done through the cluster settings modal.
It follows the next logic:

User stored data goes to: trento/v0/clusters/mycluster/user_settings/connection/ as map
If the data is not stored, the code tries to use the Azure metadata information to get the OsProfile admin user name
Ansible inventory is changed based on this logic, adding the ansible_user value

Demo:

Follow up issues:
https://github.com/trento-project/trento/issues/218
https://github.com/trento-project/trento/issues/219

@fabriziosestito @rtorrero @aleksei-burlakov @dottorblaster @stefanotorresi
I need a review here to complete all the changes for the 0.4.0 release

@arbulu89 I tried to run the PR on gollum, the default user is an empty column in the modal.
Is this desired or we want to show "root" in the case we are not running on Azure?

P.S: Please don't laugh at my cluster petname

This value is coming from the cloud metadata, so in libvirt we don't have it. Behind the scenes, if it is not being shown there root will be used.
The thing is that there is not any way of getting this default value, so using root would be a hardcoded value on the code.
We could put it there to have at least some value in the web ui

@arbulu89 I tried to run the PR on gollum, the default user is an empty column in the modal.
Is this desired or we want to show "root" in the case we are not running on Azure?

P.S: Please don't laugh at my cluster petname

This value is coming from the cloud metadata, so in libvirt we don't have it. Behind the scenes, if it is not being shown there root will be used.
The thing is that there is not any way of getting this default value, so using root would be a hardcoded value on the code.
We could put it there to have at least some value in the web ui

Oh ok, yes I would go with the hardcoded one in case we are not on azure, just to be consistent with the UI

@arbulu89 I tried to run the PR on gollum, the default user is an empty column in the modal.
Is this desired or we want to show "root" in the case we are not running on Azure?

P.S: Please don't laugh at my cluster petname

This value is coming from the cloud metadata, so in libvirt we don't have it. Behind the scenes, if it is not being shown there root will be used.
The thing is that there is not any way of getting this default value, so using root would be a hardcoded value on the code.
We could put it there to have at least some value in the web ui

Oh ok, yes I would go with the hardcoded one in case we are not on azure, just to be consistent with the UI

"root" default user now is showed in the modal if there is not any cloud metadata found:


Furnace & smelting parameters

I would like to be able to adjust such vanilla parameters as the time of burning/smelting/cooking items in the furnace, as well as the amount of fuel obtained from each burning item required to maintain the heat

You can do this very simply with datapacks :
https://crafting.thedestruc7i0n.ca

What @craftish37 said ^


Workaround for Remove-WmiObject on PowerShell Core

Fix Test-xDscSchema failing to call Remove-WmiObject on PowerShell Core.
As a workaround Remove-WmiObject is now called through Windows PowerShell on
PowerShell Core (issue #67).

Fixes #67

This change is

@kwirkykat could you please review this one? This fixes the last problem with Test-xDscSchema when running the command through PowerShell Core. This resolves a problem in DscResource.Tests when running tests on PowerShell Core.

@kwirkykat Closing this issue to see if we can use deleteclass pragma instead, as per suggestion from @SteveL-MSFT.


Build Error: API 'variant.getPackageLibrary()' is obsolete and has been replaced with 'variant.getPackageLibraryProvider()'.

just imported into Android Studio & receive this build error.
Is UAMP an abandoned project? Is there a current maintained example project for AA media player?

This is not a build error, but a warning. Following through with the suggestion in the warning (namely "use -Pandroid.debug.obsoleteApi=true on the command line to display more information") results in this:
WARNING: API 'variant.getPackageLibrary()' is obsolete and has been replaced with 'variant.getPackageLibraryProvider()'.
It will be removed at the end of 2019.
For more information, see https://d.android.com/r/tools/task-configuration-avoidance.
REASON: The Kotlin plugin is currently calling this API. We are working to solve this.

I have the utmost confidence that JetBrains will update the plugin in time for the removal.
Since this is not anything related to the UAMP project itself, I'm going to close this issue. Thank you.


Long numbers

I'm having Bad request using this number:
+345906566798696
Is it too long?

Hello!
According to the information available on Wikipedia, the maximum length of a telephone number in Spain, including the country code, is 11 digits. The number you've provided, +345906566798696, exceeds this limit and therefore appears to be incorrect.
If you are confident that the number is correct, could you please provide any additional information about this phone number's format? Sometimes, phone numbers may include extra digits for various reasons, such as extensions or calling through certain carriers, but these are typically not part of the standard phone number.
Please double-check the number and ensure that it adheres to the standard format.
Thanks!

The number is correct.
It's from a m2m simcard from this provider.
It's working in Spain, if you send a message to that number it receives.

Hello!
I've been thinking about the situation with the non-standard phone numbers you're dealing with. What I can do is add an option to the API that allows you to disable phone number validation. This way you can bypass the check. But just a heads up, with validation turned off, you'll need to make sure the numbers are clean, starting with a + and followed only by digits, in line with the E.164 format.
It'll take a little bit of time for me to update both the client and server parts of the app to make this happen. I'll definitely let you know as soon as the updates are up and ready for you to use.
Thanks!

Hello Aleksander,
Thank you very much, if I can help with a PR please let me know.

Hello!
I've implemented the feature to disable phone number validation that we discussed. A test version is now available for you to try out. You can download it from the following link: https://github.com/capcom6/android-sms-gateway/releases/tag/v1.6.1
To disable phone number validation, please append the query parameter skipPhoneValidation=true to your API requests. For example, POST /3rdparty/v1/message?skipPhoneValidation=true.
I would appreciate it if you could test this new feature and let me know if it works as you expected.
Thanks!


Should be able to bootstrap broker from some given raft index

If we have a server that starts up by itself with nothing but the shards and metastore in the data directory, it should be able to start up a new raft cluster and start accepting writes and assign the first one the max raft index from those shards.

Fixed by #2053


chore: my first contribution ❤

Description ✏
Added Github username to CONTRIBUTORS.yml.
Type of Change 🐞

[ ] Feature - A non-breaking change which adds functionality.
[ ] Fix - A non-breaking change which fixes an issue.
[ ] Refactor - A change that neither fixes a bug nor adds a feature.
[ ] Documentation - A change only to in-code or markdown documentation.
[ ] Tests - A change that adds missing unit/integration tests.
[x] Chore - A change that is likely none of the above.

Checklist ✅

[x] I have done a self-review of my code.
[ ] I have manually tested my code (if applicable).
[ ] I have added/updated any relevant documentation (if applicable).

@ramiAbdou Just a thought: you could add "linked Github to Colorstack profile" as part of the checklist since you have to be a member to contribute.

Great suggestion! We actually have a PR out to automatically do something similar, see #201!


Fix/ race condition in MQTT unittests for log handlers

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Fix/ race condition in MQTT unittests for log handlers
Tests performed by the developer:
That is my purpose in life
Tips for QA testing:
Green flowers

Can someone give that test a tickle please?


Add MarketCap for $Punks

Can you please check? The format is correct as i verified with another accepted pr.
Thanks

Run the Prettier style check on the code, you can see the error under Details


SendNatIntroduceRequest() request fails because GetLocalIp() in NetUtils.cs returns a private local ip of range 169.254.xxx.xxx in case of multiple NICs

Hi
Bug Description:
When there are multiple Network interfaces (WIFI and vmware network interfaces ), the SendNatIntroduceRequest() sends the incorrect Local IP to the server resulting in SendNatIntroduceRequest() never succeeding.
Scenario detail:
The problem is only produced when:

You separate out the Server component and client component into separate console apps from the provided sample in **HolePunchServerTest.cs
HolePunchServerTest.txt
**
You have multiple interfaces (when you have Vmware installed along with a WIFI adapter connected to the internet) on your client machine A and B
You run the client console app (based on the code from HolePunchServerTest.cs) on Client machine A and client Machine B
Your client machines A and B are both located on the same network and both are connected to the internet via their wireless network adapters.
Both have multiple virtual network adapters (in my case they exist because of VMWARE) but the situation can exist because of any 3rd party VPN software.
You run the server component on a machine on a (potentially) different network with a public IP address and an open/forwarded port (preferably on a different network than the client machines' network so that HOLD punching can be properly tested).

What really happens?

The GetLocalIp() method in NetUtils.cs class will return a private local ip address i.e 169.254.xxx.xxx while the correct IP on your WIFI card would be something like 192.168.xxx.xxx
The call _c1.NatPunchModule.SendNatIntroduceRequest(new NetEndPoint(IpAddress, ServerPort), "token1"); will send that incorrect IP 169.254.xxx.xxx as localEndpoint to the server.
As a result both client machines A and B will report their incorrect local IP Address as 169.254.xxx.xxx , to the server, while their original IP address on their WIFI cards will probably be <IP_ADDRESS> and <IP_ADDRESS> (ip address for example case only, actual IPs may be of different range or values depending upon the router/DHCP server they get their IP addresses from)
The server will try to introduce machines with private IP address of range 169.254.xxx.xxx while the clients are necessary not listening on their corresponding local ip addresses.
As a result introduction will fail.

FIX
I fixed the problem by getting all local Ip address using the following piece of code i found somewhere on the internet.
Put this method in Netutils.cs
public static List<string> GetAllLocalIPAddresses(bool preferIPv4 = false)
        {
            return NetworkInterface.GetAllNetworkInterfaces()
                .SelectMany(adapter => adapter.GetIPProperties().UnicastAddresses)
                .Where(adr => adr.Address.AddressFamily == AddressFamily.InterNetwork && adr.IsDnsEligible)
                .Select(adr => adr.Address.ToString()).ToList();
        }

Next step: In NatPunchModule.cs locate the following method:
public void SendNatIntroduceRequest(NetEndPoint masterServerEndPoint, string additionalInfo)
        {
			if (!_netBase.IsRunning)
                return;

            //prepare outgoing data
            NetDataWriter dw = new NetDataWriter();
            var listOfIps = NetUtils.GetAllLocalIPAddresses();
            string networkIp = NetUtils.GetLocalIp(true);
            int networkPort = _netBase.LocalEndPoint.Port;

			NetEndPoint localEndPoint = new NetEndPoint(networkIp, networkPort);
			dw.Put(localEndPoint);
			dw.Put(additionalInfo, MaxTokenLength);

			//prepare packet
			var packet = _netBase.PacketPool.GetWithData(PacketProperty.NatIntroductionRequest, dw);
			_netBase.SendRawAndRecycle(packet, masterServerEndPoint);
	}

and replace the code with this method.
New Code:
public void SendNatIntroduceRequest(NetEndPoint masterServerEndPoint, string additionalInfo)
        {
            if (!_netBase.IsRunning)
                return;

            //prepare outgoing data
            NetDataWriter dw = new NetDataWriter();
            var listOfIps = NetUtils.GetAllLocalIPAddresses();
            string networkIp = NetUtils.GetLocalIp(true);
            int networkPort = _netBase.LocalEndPoint.Port;

            foreach (var address in listOfIps)
            {
                //NetEndPoint localEndPoint = new NetEndPoint(networkIp, networkPort);
                NetEndPoint localEndPoint = new NetEndPoint(address, networkPort);
                dw.Put(localEndPoint);
                dw.Put(additionalInfo, MaxTokenLength);

                //prepare packet
                var packet = _netBase.PacketPool.GetWithData(PacketProperty.NatIntroductionRequest, dw);
                _netBase.SendRawAndRecycle(packet, masterServerEndPoint);
            }

        }

Now I know that this is not an ideal solution. However, it gets the job done. The only cost is that it sends all the local ip addresses to the server and the server sends introduction packets to all these local ip addresses which might result in a little bit more traffic.
Maybe the code can be improved and fix is implemented by  @RevenantX  :)
Looking forward to hearing comments on this.
Regards
TM
P.S I have also attached updated files with my modifications.
NatPunchModule.txt
NetUtils.txt

We should look at https://github.com/lidgren/lidgren-network-gen3/tree/a7468f4f1c0b337b814cc2c0f713a4459dc84299/Lidgren.Network/Platform
because its not the same on all platforms

I know about this bug. And i know how to fix it. I just don't have enough time for this.

@boonkerz
The lidgren.network also suffers from the same issue.
As a matter of fact, I had a working app based on Lidgren.Network but the problem appeared when VMWARE or any 3rd party ap with virtual interfaces is installed then even the Lidgren's code get's the incorrect adapter (virtual adapter of vmware in this case).
I tested this again by copying code from PlatformWin32.cs/ I called the GetMyAddress() from the Lidgren's code as linked. Here is the result.

Now this result is incorrect because the following logic returns vmware adapter instead of the correct WIFI or ethernet adapter. Here is the relevant code which results in this behavior:
/ make sure this adapter has any ipv4 addresses                 IPInterfaceProperties properties = adapter.GetIPProperties();                 foreach (UnicastIPAddressInformation unicastAddress in properties.UnicastAddresses)                 {                     if (unicastAddress != null && unicastAddress.Address != null && unicastAddress.Address.AddressFamily == AddressFamily.InterNetwork)                     {                         // Yes it does, return this network interface.                         **return adapter;** //<----This is faulty logic. and this yields the wrong interface.                     }                 }
So essentially, we might take some inspiration from Lidgren's code but ultimately we need to test scenarios with additional virtual network intefaces which are always connected (like vmware etc) and ignore such adapters based on some intelligent logic.
Finally, @RevenantX  might have some interesting idead and I hope he finds some free time to look into this implement such a logic :)

Based on this answer on stackoverflow I just found out that vmware network interfaces donot have a gateway address.
So I came up with this code which seems to return the correct One) IP address:
public static List<string> GetAllLocalIPAddresses(bool preferIPv4 = false)
        {
            var ipAddresses = NetworkInterface.GetAllNetworkInterfaces()
                .Where(ni => ni.NetworkInterfaceType != NetworkInterfaceType.Loopback
&& ni.NetworkInterfaceType != NetworkInterfaceType.Unknown
                && ni.IsReceiveOnly == false && ni.OperationalStatus == OperationalStatus.Up
&& ni.SupportsMulticast == true
                && ni.GetIPProperties().GatewayAddresses.Count > 0)
                .SelectMany(adapter => adapter.GetIPProperties().UnicastAddresses)
                .Where(adr => adr.Address.AddressFamily == AddressFamily.InterNetwork
&& adr.IsDnsEligible)
                .Select(adr => adr.Address.ToString())
                .ToList();
            return ipAddresses;
        }

Of course there is a change that a person may have set gateway address to a vmware virtual network adapter, lets say for experimentation purposes, then we can even explicitly rule out vmware network interfaces by excluding their names along with the gateway address exclusion logic:
public static List<string> GetAllLocalIPAddresses(bool preferIPv4 = false)
        {
            var ipAddresses = NetworkInterface.GetAllNetworkInterfaces()
                .Where(ni => ni.NetworkInterfaceType != NetworkInterfaceType.Loopback && ni.NetworkInterfaceType != NetworkInterfaceType.Unknown
                && ni.IsReceiveOnly == false && ni.OperationalStatus == OperationalStatus.Up && ni.SupportsMulticast == true
                && ni.GetIPProperties().GatewayAddresses.Count > 0)
                .SelectMany(adapter => adapter.GetIPProperties().UnicastAddresses)
                .Where(adr => adr.Address.AddressFamily == AddressFamily.InterNetwork && adr.IsDnsEligible
                && adr.Address.ToString().ToLower().Contains("vmware") == false)
                .Select(adr => adr.Address.ToString())
                .ToList();
            return ipAddresses;
        }

There are two other points which still need to be taken care of:

What if the code is being run inside a VM? What if vmware's network adapter is the only network adapter in the machine.?
What if there are other virtual network adapters in the system besides vmware which match all the conditions?

Answer to these could be the following:

We can add additional checks to see how many network adapters are in the system that are up, physical, and whether or not a virtual adapter is the only adapter in the system. Using a configuration value, we can control whether the app should dynamically use a different exclusion criteria when ruling out adapters and ip addresses.
We can make a list of all known vendors which provide and ship virtual network adapters use that list to exclude the network adapters. The list can be supplied via a config file instead of hard coding in the code so that exlcusion criteria can be dynamically adjusted.

@RevenantX When you get some time, please share your thoughts on this.
Cheers! :)

I wrote a very long reply before I had to erase it and say.
This commit really seems to fix the issue! 🥇
I still need to do a little more test with some scenarios and will report accordingly !
Great job! 👍

i just added gateway checking...

@timematcher i can close issue?

@RevenantX  Yes I think we are good to go. Thanks for your great work! 👍


[security] Remove sensitive msg from consumer/producer stats log

Motivation
Currently, we are print password field to consumer/producer stats log
Modification

add missed @JsonIgnore on field and getMethod
delete unused withoutAttribute call

@michaeljmarshall The @Secret is not recognized by jackson writer.

@michaeljmarshall The @Secret is not recognized by jackson writer.

@Shoothzj - I don't believe that is correct. When I remove the @JsonIgnore annotations that you added, and run your test while logging the output, here is what I get:
{"serviceUrl":null,"authPluginClassName":null,"authParams":null,"authParamMap":null,"operationTimeoutMs":30000,"lookupTimeoutMs":30000,"statsIntervalSeconds":60,"numIoThreads":1,"numListenerThreads":1,"connectionsPerBroker":1,"useTcpNoDelay":true,"useTls":false,"tlsTrustCertsFilePath":"","tlsAllowInsecureConnection":false,"tlsHostnameVerificationEnable":true,"concurrentLookupRequest":5000,"maxLookupRequest":50000,"maxLookupRedirects":20,"maxNumberOfRejectedRequestPerConnection":50,"keepAliveIntervalSeconds":30,"connectionTimeoutMs":10000,"requestTimeoutMs":60000,"initialBackoffIntervalNanos":100000000,"maxBackoffIntervalNanos":60000000000,"enableBusyWait":false,"listenerName":null,"useKeyStoreTls":false,"sslProvider":null,"tlsTrustStoreType":"JKS","tlsTrustStorePath":null,"tlsTrustStorePassword":"*****","tlsCiphers":[],"tlsProtocols":[],"memoryLimitBytes":67108864,"proxyServiceUrl":null,"proxyProtocol":null,"enableTransaction":false,"dnsLookupBindAddress":null,"dnsLookupBindPort":0,"socks5ProxyAddress":null,"socks5ProxyUsername":null,"socks5ProxyPassword":"*****"}

Note that the @Secret fields correctly have **** in the names.
The @Secret annotation was added here, https://github.com/apache/pulsar/pull/8910, and the annotation's interface definitely shows an integration with Jackson. That being said, perhaps you found a case where the Jackson serializer wasn't interpreting the annotation correctly? I'm not familiar enough with custom annotations to know, but it seems like the old behavior was working in unit tests.

@michaeljmarshall
There mightbe sometime the @Secret ability is broken. If you checkout to 8bf6785c0803d314465b2d9156df6ca5bbb3c644 commitId, the test will fail.

@Shoothzj - The test fails because a configuration name including Password is printed in a key. The sensitive field's value is ****, so there is no leak. When I remove Assert.assertFalse(s.contains("Password"));, the test passes.

@michaeljmarshall I retest the code. You are right, shall we revert this PR? I can work on it.

@Shoothzj - thanks for testing it out and confirming! I think it'd be good to revert it. I'm happy to help review and cherry-pick it, if that helps.


Remove fullNamespace

Context
https://github.com/celestiaorg/celestia-app/blob/5bcf4ad51756e5f31dd7d9ea140844063b6ce391/pkg/square/builder.go#L361-L364
Prerequisites

Merge https://github.com/celestiaorg/celestia-core/pull/1030
Cut a new release of celestia-core

Proposal

Bump to the new release of celestia-core
Replace usage of fullNamespace in celestia-app with celestia-core's exported Namespace method

I'm not aware of the codebase but I assume that the feat: add Namespace method to blob will work as intended(in the merge)
As for the Proposal 2, I'm assuming that this function will do the trick
func fullNamespace(blob core.Blob) []byte {
return blob.Namespace()
}
Let me know if you have any bug.


Location

This is a Feature Request

What would you like to be added

Why is this needed
Comments

I want is the location friend every morning

@kingaooto this seems to be off topic.
If re-opening, please clarify how the Kubernetes SIG Docs can help.
/close


Calling pause does not pause animation.

Not sure whats going on but when I call pause:
let r = new Rune;
r.play();
r.on('draw', fc => {
    // Do some wicked animation stuff.
    r.pause();
});

Does not actually stop the 'draw' event. However if I do:
let r = new Rune;
r.play();
r.on('draw', fc => {
  // More wicked...
});
r.pause();

This seems to work.

Thanks for reporting this. It was a small problem related to requestAnimationframe being renewed before cancelAnimationframe was called, thus cancelling cancelling a non-existing frame. I just fixed it, and I'll publish a 0.2.4 release later today. :+1:

0.2.4 has been released!

Grazie!


Fix URL error for projects with underscore in name

Description
Fixes #735!
This is an urgent issue on the site.
Once the fix is merged to master, I'll update production to match master.

Good enough – need to get this fixed, I'll merge :)


feat: improvements for startingpoint

Don't merge yet. There will be more. I'll alert you on Discord when it's ready.
It's ready!
Edit: I just realized that some of these things had tickets, so I'm tagging them below.
Closes: #53 #54

Don't merge yet.

Tip: Set PR as draft

I had planned the config to look like this once there is a need to make a breaking change to the rpms: section.
rpm:
  repos:
    - https://blahblah.com/abc.repo
  include:
    - micro
  exclude:
    - firefox
    - firefox-langpacks

Okay I can make that change too. That structure looks even better.

@EinoHR I just noticed that many older pull requests were merged via squash. Please be sure to not do that here, so tht we preserve individual commits to help out with future history inspection, git blame, looking up commit messages etc. :)

Whoa, that's a lot! I've read all the commits and can't seem to find anything more substantial (the script running functionality feels complicated, but I can stand that) to complain about rn, so I'll merge (Rebase and merge, not squash). And if there's bugs, oh well, we'll fix it.
Future notice for @Arcitec, when using Conventional/Semantic Commits, postfix the commit indentifiers with ! to show that it's a breaking change. https://www.conventionalcommits.org/en/v1.0.0/#commit-message-with--to-draw-attention-to-breaking-change
Since this is a template repo under active development, non-breaking versioning is not that required, but I'm hoping to do a 1.0 release after the ISO building functionality is added and after that try to do versioned updates.

@EinoHR Thank you for everything. It's been a pleasant process.
I've done thorough testing locally, and the scripts are really robust (decades of experience with bash scripting). But I have only done local testing, no cloud builds. Still, it should all work the same in the cloud too.
Thank you for telling me about the ! breaking change flag for semantic commits. It's a really interesting system that I've been seeing more and more. After seeing that it can even be used to auto-generate changelogs, I'll be using it for my own projects from now on. :)
Regarding the upcoming fix for the firstboot launcher, there's an ongoing discussions at https://github.com/ublue-os/startingpoint/issues/69 to determine where ublue should store its system-wide component files. When that's settled, I'll contribute that fix too. It's looking like that's going to be the location, though, so I'll probably start coding that fix locally in the meantime.

@EinoHR I forgot to comment about "the script running functionality feels complicated". Complicated is another term for flexible. :)
It needs to be this way for the reasons mentioned in a separate ticket:
https://github.com/ublue-os/startingpoint/issues/53#issuecomment-1541296379
This new system allows the user to choose whether they want autorun, manual, or both combined. :+1:


Windows test not working

The following issue has been opened to the Upstream repo.
https://github.com/nanomsg/nng/issues/1504

Garrett has responded, any feedback for him?


Reduce foreign node requests to work better with hosted nodes

Closes #576
This should reduce the foreign node requests from daily ~150k to ~40k.
This comes with the trade of of losing one safty check, which I think is not necessary, and
a doubled poll interval of now 10s. Because 10s is still under the 15s block time of Görli and Mainnet, this should not be a problem.

Codecov Report

Merging #585 into master will decrease coverage by 0.05%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #585      +/-   ##
==========================================
- Coverage      76%   75.94%   -0.06%
==========================================
  Files          18       18
  Lines         925      923       -2
==========================================
- Hits          703      701       -2
  Misses        222      222

Flag
Coverage Δ

#bridge_end2end
?

#bridge_unit
75.94% <87.5%> (-0.06%)
:arrow_down:

Impacted Files
Coverage Δ

tools/bridge/bridge/config.py
86.74% <100%> (+0.32%)
:arrow_up:

tools/bridge/bridge/event_fetcher.py
100% <100%> (ø)
:arrow_up:

tools/bridge/bridge/node_status.py
50.98% <75%> (+0.98%)
:arrow_up:

Continue to review full report at Codecov.

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


[Form Provider Refactor] RoomNameInput

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/28877
Tests
IMPORTANT: User needs to have the permission to use policy rooms enabled.
Workspace new room:

Log in
Press + button > Press Start chat > Switch to # Room
Check if errors appears when incorrect data is typed in
Type correct data and create a new room

Room name:

Log in
If there are no rooms, create a new one (steps above)
In the chat room press its name > Press Settings > Press Room name
Check if errors appears when incorrect data is typed in
Type correct data and update the name of the room

[x] Verify that no errors appear in the JS console

Offline tests

Same as above.
QA Steps

Same as above.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/91068263/ec670ab0-2a43-437c-ae4c-9b9a195a0b23

Android: mWeb Chrome
https://github.com/Expensify/App/assets/91068263/8202c5dd-9daf-4c27-8d13-31017ff624c9

iOS: Native
https://github.com/Expensify/App/assets/91068263/02c8d0fd-1f64-4e83-a7a3-ba453d0bdc13

iOS: mWeb Safari
https://github.com/Expensify/App/assets/91068263/8d93e18b-5af0-440d-aa4b-ff801f2df6e5

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/91068263/a0fbf646-8b2d-4965-b4e2-cf2963c18610

MacOS: Desktop
https://github.com/Expensify/App/assets/91068263/692044b8-6f5f-4914-9376-f03753e38c7b

@luacmartins the PR is ready for the review.

Hey @allroundexperts will you manage to do a review today?

@allroundexperts any updates?

@allroundexperts did you get a chance to review this one?

Able to submit the form empty.
Checking it right now

It seems the first issue was also connected with the second one.

@allroundexperts feel free to check it.

BTW I would love to close this issue tomorrow at the latest, because I'm OOO for a week since this Friday.

@allroundexperts The one I've pushed right now should fix it.

Nice work here! Let's get this one wrapped up today!

I will handle this in a couple minutes.

Done

This one is fixed here and I would like not to overwrite this prop here as well, to avoid conflicts.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.3.92-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@cdOut @kowczarz we had to revert this PR since it caused the 3 blockers below:

https://github.com/Expensify/App/issues/30500
https://github.com/Expensify/App/issues/30504
https://github.com/Expensify/App/issues/30507

Could you please work on a new PR and fix the above issues?

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.92-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/luacmartins in version: 1.3.93-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.93-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


use extra index rathe than find links for link patching

Fixes #55.

This actually fixes the failures seen in #58 :tada:


bug: failed to find Lua file if Apache APISIX install with Lua 5.3

https://github.com/apache/incubator-apisix/pull/1157#issuecomment-601973561

it should be fixed https://github.com/apache/incubator-apisix/pull/1351


More details would be welcome.

@shentao have you tried using

Like @shentao said more details would be good. I've used :taggable=true and works fine with asynchronous so I guess it's an implementation issue. If you could share your code example and we can then advise on a fix. :)

@rickbolton you have also used


Get Installation object in listeners and custom http routes

Description
I tried to find it, but I didn't find anything related so I'm asking here.
Is there any way to retrieve the Installation object without using middleware?
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version: 3.11.1
node version: 14.18.2
OS version(s): Linux Ubuntu 20.04

@WendellAdriel - Hm, so I know that there's a way to get installations stored in your installationStore via the installProvider#authorize() method. Here is the documentation on that method from the underlying node-slack-sdk module that Bolt uses.
From Bolt, I think this could look like:
const result = app.receiver.installer.authorize({teamId: 'TEAMIDHERE'});
/*
result = {
  botToken: '',
  userToken: '',
  botId: '',
  botUserId: '',
  teamId: '';
  enterpriseId: '';
}

As for best practice if you're looking for this information from within a listener for example, I've asked around with the wider team on what you can do here.

Hey @srajiang, thanks for the info! That helps a lot!
Just as a suggestion, it would be GREAT if the full installation object that we get from the fetchInstallation method would be added by default in the context object, so we can have access to it in all the listeners in the app.
If you think that this feature would be useful, I can work on a PR for that.

@WendellAdriel Thanks for the suggestion! but we are not planning to change the minimum level data structure of context objects at least in the short term due to backward compatibility consideration.
When you need all the properties of installation data in listener functions, (I know that this is not the most efficient way from the performance perspective but) I would suggest having your own global middleware to attach the rest of installation data to a context object.

@seratch got it!
I created a global middleware to inject this info for my project, I just asked to see if you wanted to include this as default and if so I could help with a PR, but thanks for all the info @srajiang and @seratch!!!


[MXNET-613] Upgrade jetpack to version 3.2.1

Description
Jetson builds are currently failing to install due to some missing dependencies.  We can fix the dependency issue and upgrade Jetpack to 3.2.1.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

LGTM

Seems like you opened to Box of Pandora :P http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/incubator-mxnet/detail/PR-11494/11/pipeline/796
+ apt-get install -y openjdk-8-jre

Reading package lists...

Building dependency tree...

Reading state information...

openjdk-8-jre is already the newest version (8u171-b11-0ubuntu<IP_ADDRESS>).

openjdk-8-jre set to manually installed.

0 upgraded, 0 newly installed, 0 to remove and 52 not upgraded.

+ echo 'deb https://dl.bintray.com/sbt/debian /'

+ tee -a /etc/apt/sources.list.d/sbt.list

deb https://dl.bintray.com/sbt/debian /

+ apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv 2EE0EA64E40A89B84B2DF73499E82A75642AC823

Executing: /tmp/tmp.BQK2RhtT4W/gpg.1.sh --keyserver

hkp://keyserver.ubuntu.com:80

--recv

2EE0EA64E40A89B84B2DF73499E82A75642AC823

gpg: requesting key 642AC823 from hkp server keyserver.ubuntu.com

gpgkeys: key 2EE0EA64E40A89B84B2DF73499E82A75642AC823 can't be retrieved

gpg: no valid OpenPGP data found.

gpg: Total number processed: 0

gpg: keyserver communications error: keyserver helper general error

gpg: keyserver communications error: unknown pubkey algorithm

gpg: keyserver receive failed: unknown pubkey algorithm

@marcoabreu I know right!  Tracking this one as well.  We're thinking of work arounds.  Other key servers unfortunately don't have this key.

@marcoabreu This one should be ready if you're ok with the changes.


Documentation on  how to use  Chakra with PyTorch

A simple 101 example to setup, run and visualize/analyze  a PyTorch run would  be very useful.

Please see if https://github.com/mlcommons/chakra/wiki/Chakra-Execution-Trace-Collection-‐-A-Comprehensive-Guide-on-Merging-PyTorch-and-Kineto-Traces works for you.


Log request/response  not showing up

Hi! :)
According to the README, I wrote dependency in pom.xml:
<dependency>
    <groupId>org.zalando</groupId>
    <artifactId>logbook-core</artifactId>
    <version>1.11.2</version>
</dependency>

Also wrote in application.properties:
logging.file=log/rest-sample.log
logging.level.org.springframework.web=DEBUG
logging.level.org.zalando.logbook: TRACE

But in my log-file request/response  not showing up.
My demo project (tree "filter"):
https://github.com/IBlasterus/rest-sample/tree/filter
Best regards, Alexandr.

You need the logbook-spring-boot-starter for your example.


If you select all blogs to be added to the queue and the first blog is offline, then enqueue selection is greyed out

Describe the bug
When you select all blogs to be added to the queue while the first of these blogs is offline, then the context menu option 'enqueue selection' is greyed out. Any other blog can be offline but only when the first blog is offline, then the 'enqueue selection' option is unavailable.
To Reproduce
Steps to reproduce the behavior:

Select all blogs while the first blog is offline
Right click on the selected blogs
Notice that the 'enqueue selection' option of the context menu is greyed out

Expected behavior
When selecting all blogs and some of these (but not the first blog) is offline, then you can still use the context menu option 'enqueue selection' to queue the blogs for download. The offline blogs are correctly ignored and all online blogs are added to the queue. But this does not work when the first blog is offline. I assume that this is a bug and that the 'enqueue selection' context menu option should also work in this case.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

TumblThree version: <IP_ADDRESS>7
OS: Windows 10 pro 64
Browser NA
Version NA

Additional context
Add any other context about the problem here.

Thanks for reporting a bug. We'll take a look at it.


Untitled - apply language mode from copied text upon paste

When pasting text from a VSCode editor into an untitled editor, it would be great if we inherit the mode we copied from.
Probably only unless a language mode is already set.

@bpasero You forgot to verify the second part. Try setting "plaintext" manually and then paste from an existing editor. The pasted language is still adopted.

Probably only unless a language mode is already set.


Nuclide run-through #2

Since the first one went really well, let's do another round :)
The issues are getting more scoped and less serious :)

[x] Should put the arrow function body on the next line instead of breaking it in weird ways

[ ] I think I like the arrow function better for type

[ ] Would be nice to make empty body have the } on their own line

[x] Space before would be nice

[ ] Wrong code generated because of comment placement

[x] Invalid code generated because of trailing comma

[ ] Would be nice to keep the comment where it was

[ ] If one && is broken, would be nice to break all of them into their own line

[ ] Missing type annotation for argument spread

[ ] Missing ; for export default

[ ] Missing parenthesis for ++

[ ] Missing generic

[ ] Import doesn't break

Should put the arrow function body on the next line instead of breaking it in weird ways

This was because of the last arg grouping. It's an anon function so it lets it break itself. However I think it makes sense to only allow that if it's an arrow function with braces. I fixed this in https://github.com/jlongster/jscodefmt/commit/892d07031314b0e353af35601491db659328f802

I think I like the arrow function better for type

I'd like more feedback from Flow users... I think I got this logic from recast. I believe it's easy to tweak if we do want this.

For things like like this I would like more feedback from the community, but I also want to prioritize helping make this work for you. I think getting it work on nuclide is a strong use case and I'm fine tweaking the defaults to make that work.

Invalid code generated because of trailing comma

Looks like this was already fixed by another commit

I like the empty block :)

Cool, that's good. There aren't many empty blocks in real world anyway :p

Cool, that's good. There aren't many empty blocks in real world anyway :p

I may have just convinced myself that maybe it's better to split them across 2 lines because then it's easier to add code to them... but this can easily be changed later.

Cool, that's good. There aren't many empty blocks in real world anyway :p

I may have just convinced myself that maybe it's better to split them across 2 lines because then it's easier to add code to them... but this can easily be changed later.

} catch (err) {}
should format to
} catch (and_ignore_err) {}
only half joking...

Invalid code generated because of trailing comma
Looks like this was already fixed by another commit

Do you have a link to the commit that fixes this? I'm still seeing it on version 0.0.4. This code:
const LongVariableName = longMethodName(longParamName, longSecondParamName, () => {});

becomes:
const LongVariableName = longMethodName(longParamName, longSecondParamName, (
  ,
) =>
  {});

Which is invalid as the comma is inside the empty parameter list.
https://vjeux.github.io/prettier-browser/#{"content"%3A"const LongVariableName %3D longMethodName(longParamName%2C longSecondParamName%2C () %3D> {})%3B%5Cn%22%2C%22options%22%3A%7B%22printWidth%22%3A80%2C%22tabWidth%22%3A2%2C%22useFlowParser%22%3Afalse%2C%22singleQuote%22%3Afalse%2C%22trailingComma%22%3Atrue%2C%22bracketSpacing%22%3Afalse%7D%7D

Yeah, there's another bug about that too. I'll look into it.

Just sent #133 and #134 for the types that didn't print on babylon parser.

Closing this one, most of those are either done or already have open issues!


Assets optimization + misc.

Lossless compression using the Leanify tool, also added homepage tag to the fabric.mod.json file, updated Gradle version to support Java 17 (MC 1.18).

You should probably forward these changes to https://github.com/Direwolf20-MC/BuildingGadgets/ too

Ohh I forgot, thanks for reminding me.

lgtm


Inf in a netcdf file crashes Ferret

Reported by Ansley B Manke on 3 Nov 2005 18:49 UTC
On operations that would do a fomratted list the value, we get an error (in
TM_FMT, where Ferret tries to write the INF).
yes? use inf_data.nc  ! file attached to this report
yes? stat ha
startint: error in format
apparent state: internal I/O
last format: (F**.**)
lately writing sequential formatted internal IO
Abort
Plot commands work on the data, with the Inf being treated as missing.
(this is a non-standard cdf file, but this is at least worth noting.)

ncdump inf_data.ha

netcdf inf_data {
dimensions:
LON62_70 = 9 ;
LAT79_81 = 3 ;
TIME = UNLIMITED ; // (1 currently)
variables:
double LON62_70(LON62_70) ;
LON62_70:units = "degrees_east" ;
LON62_70:modulo = 360. ;
LON62_70:point_spacing = "even" ;
LON62_70:axis = "X" ;
double LAT79_81(LAT79_81) ;
LAT79_81:units = "degrees_north" ;
LAT79_81:point_spacing = "even" ;
LAT79_81:axis = "Y" ;
double TIME(TIME) ;
TIME:units = "SECONDS" ;
TIME:axis = "T" ;
float HA(TIME, LAT79_81, LON62_70) ;
HA:missing_value = -1.e+34f ;
HA:_FillValue = -1.e+34f ;
HA:long_name = "Wave Amplitude" ;
HA:history = "From hilo_runup_ha" ;
HA:units = "CENTIMETERS" ;
// global attributes:
:history = "FERRET V5.81    3-Nov-05" ;
data:
LON62_70 =<PHONE_NUMBER>42001,<PHONE_NUMBER>36001,<PHONE_NUMBER>30001,
204.931709824001,<PHONE_NUMBER>18001,<PHONE_NUMBER>12001,<PHONE_NUMBER>06001,
204.933739400001,<PHONE_NUMBER>94001 ;
LAT79_81 = 19.7397807139993, 19.7402881079993, 19.7407955019993 ;
TIME = 18595.0000000026 ;
HA =
-1098.219, -1039.61, -1021.432, -736.558, -549.6904, 1.138712e+21, Inf,
Inf, Inf,
-564.2064, -560.843, -909.522, -858.5319, -650.9611, 1.800843e+18, Inf,
Inf, Inf,
-719.9532, -427.5533, -410.7674, 252.4728, -7.528062, 1.389645e+09,
2.026626e+32, Inf, 5.408152e+27 ;
}
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1355

Comment by<EMAIL_ADDRESS>on 3 Nov 2005 21:06 UTC
Note that tm_fmt.F and tm_fmt_digits.F already call TM_CHECK_NAN, which invokes
the C routine isnan.  There is also an isinf routine, so we could easily do the
same to check for Inf.

Comment by<EMAIL_ADDRESS>on 4 Nov 2005 22:08 UTC
Fix with new routine tm_check_inf, called by tm_fmt.F and tm_fmt_digits.F which
looks for a value of Inf before trying to format the value.

Attachment from<EMAIL_ADDRESS>on 3 Nov 2005 18:50 UTC
short cdf file contining values of inf
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1355_GIT_637/inf_data.nc


Issue with k8s.io/docs/concepts/services-networking/network-policies/

This is a Bug Report

Problem:
In section 'Default deny all ingress and all egress traffic' the associated example is the same of 'Default deny all egress traffic' that is

apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
name: default-deny-egress
spec:
podSelector: {}
policyTypes:

Egress

Proposed Solution:
I think is missing
policyTypes:

Ingress

based on what I've read before but I could be wrong since I'm just learning these days
Page to Update:
https://kubernetes.io/docs/concepts/services-networking/network-policies/

I would like to take this up. Thanks.

/assign

It's possible that PR #18091 introduced a regression here.

https://kubernetes.io/docs/concepts/services-networking/network-policies/#default-deny-all-egress-traffic looks correct to me though.

https://kubernetes.io/docs/concepts/services-networking/network-policies/#default-deny-all-ingress-and-all-egress-traffic
---
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: default-deny-egress
spec:
  podSelector: {}
  policyTypes:
  - Egress

which should be
---
apiVersion: networking.k8s.io/v1
kind: NetworkPolicy
metadata:
  name: default-deny-all-ingress-egress
spec:
  podSelector: {}
  policyTypes:
  - Ingress
  - Egress

Also I would like to update the name to default-deny-all-ingress-egress
Let me know if that is appropriate.

@sftim Submitted PR - 19052 for this.


Added supported of 'AUTH LOGIN'

i've been missing a 'r' in previous, lol~

this is for https://github.com/gogits/gogs/issues/1514

Oops, forgot to mention that would you mind to re-PR to develop branch?

never mind, it's up to you :-)

Cool! So can you close this and start a new to develop?


Add additional roles to role dropdown in groups

Allow us to have more customization within our group related to roles. I spoke with Psi about the following hierarchy:
Leader (have currently) > Staff/Mod/Co-Leader Title Here > Member (have currently) > Alt
They would appear by default in this order while loading the group. This will allow all of your leaders and co-leaders to be at top and any alt accounts of members will be on the bottom.

You could go with all cc ranks, and just have it set to 'member' by default if one is not set. Would be great if the plugin could automatically update the group from the clan setup screen.
Otherwise just Leader/Staff/Member would probably be sufficient for most clans
Additional options could be Alt, as mentioned above, and possibly Founder.

We should probably wait to hear more about the clan update before considering this as it seems to be bringing quite a few interesting changes to CC ranks.

Implemented n #830


Red blood splitting itself in containers

Description
So as usual some dumbo atmos blinded themselves and I needed to make some Ocu, handed them syringe to come back later with blood but got some of it off SSD myself in the end, when combined in chemmaster I had multiple stacks of same reagent
Screenshots

its not the same, they have different dna

its not the same, they have different dna

Only original things used to have DNA, did that change? But even then, shouldn't they simply get merged?

they do not or the information would be lost


Removes proxy.config.http.server_tcp_init_cwnd

This was only ever supported on Solaris, a platform that we no
longer supports. Cleaning this up avoids confusion.
Closes #5476

[approve ci freebsd]


Improve username behavior

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Updated Python script to define that username is not a number

error

error


Allow non-browser websocket clients to access the /logs endpoint

 websocket.Handler.ServeHTTP serves to browser requests and was also causing logspout to crash when accessed by a client like wscat as they don't typically set the origin header (See https://github.com/gliderlabs/logspout/issues/412). This change bypasses the validation for origin header.

Thanks for the contribution. Thoughts on a unit test or two for this and the package in general?

Hei ; the error is sill happening and non-browser client seem to have a problem using the ws endpoint


net: pkt: Check the minimum buf and pkt counts

For networking code, it does not make sense to have zero net_buf
or net_pkt instances. Make sure this is enforced by code.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #13307 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13307      +/-   ##
==========================================
+ Coverage   48.76%   48.76%   +<.01%
==========================================
  Files         319      319
  Lines       46864    46864
  Branches    10832    10832
==========================================
+ Hits        22853    22854       +1
  Misses      19381    19381
+ Partials     4630     4629       -1

Impacted Files
Coverage Δ

subsys/net/ip/net_pkt.c
65.4% <ø> (ø)
:arrow_up:

arch/posix/core/posix_core.c
91.91% <0%> (+1.01%)
:arrow_up:

Continue to review full report at Codecov.

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

As those tests does not depends on other value, can we enforce them directly in Kconfig using the range option? I think you need to specify a minimum and maximum value, we can probably just set and arbitrary big value for the maximum.

I was thinking that but as you said the range option needs the upper limit and we cannot really know it. I did not want to put max int to Kconfig file as the user might get the impression that it would be a valid value.

As those tests does not depends on other value, can we enforce them directly in Kconfig using the range option? I think you need to specify a minimum and maximum value, we can probably just set and arbitrary big value for the maximum.

@ulfalizer I think it's not the first time that Kconfig misses a way to constraint a value with only a minimum or a maximum value (as opposed to a range). Do you think this option could be added in the future?
@jukkar Thanks for the details, until Kconfig is improved, I think your approach is perfectly acceptable.

Speaking of that, I wonder if anyone really tried low values for those options. I did some time ago, and IIRC, with CONFIG_NET_PKT_TX_COUNT=1 there was just silent lockup on boot, etc. Really in my queue to retest and look deeper into that one day.


Add cursor to web api ListDeployments

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/cc @cakecatz


HDDS-9934. Display 'Last Heartbeat' in SCM UI in the Node Status section

What changes were proposed in this pull request?
It's useful for the cluster admin to have each datanode's 'Last Heartbeat' available in the SCM UI. This patch is adding that change. The time is presented in a relative value e.g. 2s ago, 3m 1s ago etc.
The method for making the calculations, could be in a Util class and not in SCMNodeManager. Feel free to provide any feedback on that.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-9934
How was this patch tested?
A new unit test was added for this patch. It was also tested manually. Check the attached screenshots from SCM UI.

@ArafatKhan2198 I've updated the patch. Please take another look.


ledmatrix: gamma correction for smoother brightness ramps

The brightness values sent to the LED controller actually control a PWM
duty cycle. LEDs emit light roughly in proportion to the PWM duty cycle,
but human vision perceives brightness on an exponential curve --
generally it takes 2x the physical luminous flux for the eye to perceive
something as one step brighter.
As a result, brightness ramps (like the one generated by
--all-brightnesses) were rapidly brightening to what appeared to be max,
and then flattening.
This change adds gamma correction, which maps the linear input
brightness values to a non-linear function of PWM duty cycles, causing
the ramp to look far more ramp-y.
The gamma value I've chosen here is arguably a preference, I messed with
output on a PVT panel until I found something that looked approximately
right.
Photo of a panel with the firmware from main (left) vs this change (right):

FWIW, I'm not at all sure that I hooked the gamma correction at the right points in the code --- definitely open to guidance if you'd like it moved!

It's also worth calling out that gamma correction makes all mid-range brightnesses look dimmer (by design!). So it might be good to increase the default brightness multiplier (currently 51 on PVT looks like?) to something higher. I'm happy to find a value that matches the approximate intensity of the original firmware if you'd like.

This change reminds me of what FastLED does. Great idea and work. Can I use this code in my fork of the repo?

Would appreciate progress on this change! I also noticed this issue.


Add i18n and l10n for driver download

Add i18n and l10n for driver download. (eclipse.nl.feature has been removed)
Please merge this at first then I'll add fragements and properties files into your specific feature and fragment.

It is hard to review all changes.
Yuanyuan, could you please split this on 2-3 different pull requests based on fresh devel branch?
I can't understand the nature of changes in pom.xml and feature.xml.

Serge， I'll do that later.


Table cells not responding to key presses and exhibiting wrong behaviour

Preflight Checklist

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
There are two related issues concerning Tables.

When I select a table cell and start typing, Draw IO seems to be assuming that I'm not typing anything. The cursor does not appear and whatever I type does not get rendered in the cell. I have tried double-clicking into the cells (which should be unnecessary) and the cursor still does not appear.
For those cells which do have text in them I have to double click just to get into the cell but double-clicking does not guarantee that the cell will get focus and show a cursor (sometime I have to click many times). The entire text gets highlighted (which is the wrong behaviour and should only position the cursor next to (or between) characters - not select everything). I should be able to press the delete key or just start typing to overwrite the text but draw IO ignores my key presses until I select un-select the highlighted text.

To Reproduce
Steps to reproduce the behavior:

Create a "table with Title"
Click on one cell of that table
Start typing (nothing happens)
Double click into the same cell and start typing (again, nothing happens)
Frantically click many times into the same cell just to see the cursor ... again, nothing happens.

Expected behavior
I expect tables to work just as simply and cleanly as a spreadsheet following these rules as observed in Excel:

If a cell is empty and I single-click that cell and start typing then I expect to see the cursor and any text I type.
If the cell already has content and I single-click that cell then I expect the text to be overwritten with new text.
If the cell already has content and I double-click that cell I expect the cursor position to be between (or next to) the characters where the cursor is.

Screenshots
I have single and double-clicked a cell but the cell is unresponsive to my key-presses (no cursor, no response from keyboard)

I have double-clicked into the cell, the entire text is selected (wrong behaviour). I start typing and the text is not overwritten (nothing happens). Only if I click on the text one more time will I see the cursor after which I can then press CTRL-A and continue typing or press del.

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 17.1.3

Desktop (please complete the following information):

OS: Windows 10 (Version 10.0.19044 Build 19044)
Browser: Google Chrome (Version 99.0.4844.74 (Official Build) (64-bit))

Smartphone (please complete the following information):

n/a

Additional context
Add any other context about the problem here.

Could you test this in other browsers and Incognito mode? Also hard reload it and you'll see 17.1.5 version (re-test it please)
I tested in Chrome Version 99.0.4844.74 (Official Build) (64-bit), digrams.net 17.1.5 and I couldn't reproduce the issues.
"double-clicked a cell but the cell is unresponsive to my key-presses (no cursor, no response from keyboard)" - double click shows cursor at my place
"I have double-clicked into the cell, the entire text is selected (wrong behaviour). I start typing and the text is not overwritten (nothing happens)"  - text is overwritten
If nothing helps, please attach the file

Thanks for the reply. I retried a couple of days ago (sorry for not replying same day).
I believe I now know the cause of the problem which you might be able to replicate as follows:

Add the 'props' plugin
Create any object that holds text - I've tested on Tables and Boxes.
Try to edit the cell or box

I understand that plugins are not part of the core functionality and as such are unsupported but are the plugins produced by the DrawIO developers or are they 3rd party?


breaks archwiki enumeration

everything is changed to 1
works as expected when not using chatgptbox
url: https://wiki.archlinux.org/title/Arch_Linux

The reason for this problem is that the counter-reset style of the page body is disabled when the addon activated, but I have not found an effective solution yet. Even if I try to disable all styles of chatgptbox, the problem still exists.
I think it may be related to the implementation of counter-reset in Firefox. I believe counter-reset is still unstable. I found a discussion that may be relevant: https://github.com/w3c/csswg-drafts/issues/5477 and some other thing: https://stackoverflow.com/questions/70877408/css-counter-continues-counting-in-firefox

can this be fixed? the only thing thats keeping me from using this extension

@kushagra-xo Have found a solution, I think I will fix it in the next version.

thank you very much :)

fixed in v2.5.1

many thanks.


d3LChoropleth data object

Hi, thanks for this great package! I cannot get the d3LChoropleth show a map with the data object that I'm passing. I have the map initialised like this:
<d3-l-choropleth :data="mapData"></d3-l-choropleth>
Currently I'm passing a GeoJson object as mapData but getting the following three errors on the console:
[Vue warn]: Error in nextTick: "TypeError: Cannot read property 'lat' of undefined"
TypeError: Cannot read property 'lat' of undefined
Uncaught TypeError: Cannot read property 'x' of undefined

What is the right structure of the data object to be passed for the d3LChoropleth component? There is unfortunately no information about this on the docs. Thanks for your help.

Hello @asilcetin, to be able to use d3LChoropleth, we need to prepare data and maybe options if you want to customlize, data should be a GeoJson FeatureCollection.
Here is what it looks like:
"data": { "type": "FeatureCollection", "features": [ { "type": "Feature", "properties": { "id": 61333, "name": "tech", "tags": [ "frigo" ], "rgb_color": null, "level": 0, "type": "map_zone", "value": 0.8 }, "geometry": { "type": "Polygon", "coordinates": [ [ [ 2.37260662004924, 48.8438652396112 ], [ 2.3726817580992, 48.8439130473407 ], [ 2.37271626348932, 48.8438892006837 ], [ 2.37266441507589, 48.8438575857975 ], [ 2.37261599913583, 48.8438285001021 ], [ 2.37258113243273, 48.8438516241332 ], [ 2.37260662004924, 48.8438652396112 ] ] ] } }, { "type": "Feature", "properties": { "id": 61332, "name": "sales", "tags": [ "café" ], "rgb_color": null, "level": 0, "type": "map_zone", "value": 0.5 }, "geometry": { "type": "Polygon", "coordinates": [ [ [ 2.37249370033574, 48.8437972465289 ], [ 2.37252585719145, 48.8437755677497 ], [ 2.37260859786514, 48.8438261515677 ], [ 2.37257680232242, 48.8438474690339 ], [ 2.37249370033574, 48.8437972465289 ] ] ] } }, { "type": "Feature", "properties": { "id": 61357, "name": "coffee", "tags": [ "café" ], "rgb_color": null, "level": 0, "type": "map_zone", "value": 0 }, "geometry": { "type": "Polygon", "coordinates": [ [ [ 2.37254728065663, 48.843755876192 ], [ 2.37273786465381, 48.8438700510955 ], [ 2.37271909501773, 48.8438873941188 ], [ 2.37253356438411, 48.8437732192153 ], [ 2.37254728065663, 48.843755876192 ] ] ] } }

Sorry for the late reply~


Update changelog

Updates the changelog with newest changes and edits some of the previous wording.
If preferred, we can leave this open until 0.2.0 is ready, and I'll add to this / rebase accordingly.
Rendered

I think we should merge this now / try to keep it as up to date as possible. That way people can have a "master branch change log".
When 0.2.0 drops we can move the changelog items to a 0.2.0 section

That way people can have a "master branch change log".

Good point. Sound good!


E2E/Playwright: Initial setup for Boards

Summary

Added MM-T4274 (functional) test for Boards
Added visual tests for Boards
Reorganized folder structure to separate functional and visual testing per product. Note that visual testing is recommended to be separated so it could run in separate run group.

Others:

Added Playwright test-slomo npm script to easily follow along test flow in slow motion

Ticket Link
none
Related Pull Requests

Waiting for https://github.com/mattermost/mattermost-webapp/pull/12213 to merge

Release Note
NONE

/e2e-test

Successfully triggered E2E testing!
GitLab pipeline | Test dashboard

@nevyangelova @sbishel @mvitale1989 For your reviews.

@saturninoabril I am not familiar with playwright but do we need to commit the screenshots, or do we wanna store them as artifacts?

Yes, it's going to be committed in a repo as golden screenshots. There are two ways for our visual testing: one is via committed screenshots where any committer/contributor would be able to validate their changes, and the other is via percy.io where it would be most useful during release testing. Both will be used in our testing process.

Thanks and good point! I've removed the banner by default in the recent push - https://github.com/mattermost/mattermost-webapp/pull/12237/commits/c25a61ba50c3fbc59568353d4d2688d00cfbe19f


Automatically create shims after running pip install or others

quickenv knows which entries have been added to PATH, it should be able to figure out which binaries to add to ~/.quickenv/bin/. Possibly this behavior should be behind a flag like quickenv reload --shim

Another idea here is to suggest creating shims after commands like pip install. However, that is hard to execute on, because it requires spawning each shimmed command as subprocess instead of just execing it, because you want to run code after the command returns. So, more overhead and annoyances when trying to attach a debugger to the shimmed command.

Reopening this, i saw that volta actually didn't even exec its shims, so it might be okay to just spawn a command, wait for its return and check for new shims to create.

quickenv master now also adds a warning. What we can additionally do is to consider whether a pip install (where pip is shimmed), should auto-create shims for the installed binaries.


Improve invariant testing

This PR improves the echidna invariant tests for MagicSpend paymaster implementation and better document them.

Closing is fine I guess, I would like to finish it though but maybe with a fresh start using foundry only, what do you think @wilsoncusack?


name conflict in cppship packages

We need a central package repository, like brew, or two  cppship packages' names may conflict.

use conan as portable way to manage depencies
if you replies on cppship packages, make sure it works by yourself


Other
Relevant log output
No response
Code of Conduct

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

Open("Provider=MSOLEDBSQL.1;Data Source=<IP_ADDRESS>;Initial Catalog=master;User Id=babelfish_user;Password=12345678;","","",adModeUnknown);

select * from student where STU_NUM='6' and STU_NAME='6';

Provider=MSOLEDBSQL.1; and Provider=SQLOLEDB.1 Login is normal, write is normal, read failed.
Driver={SQL Server};Login failed, but it is normal in MSSQL
ADO operation under VC 6.0

isqlw. exe login prompt: master_ Dbo.syscharsets does not exist！

Babelfish for PostgreSQL with SQL Server Compatibility - 12.0.2000.8
PostgreSQL 13.7 on x86_64-pc-linux-gnu

VC program running environment is wine 7.0。

How to create master.dbo.syscharsets?

Which Babelfish version is this? Please provide the output of SELECT CAST(SERVERPROPERTY('BabelfishVersion') AS VARCHAR)

serverproperty
1.3.2

In Babelfish, you can run this:
create view dbo.syscharsets as select * from sys.syscharsets

SQL Server Query Analyzer can log in normally.

but run "SELECT @@VERSION"
out=
version
[Microsoft][ODBC SQL Server Driver]收到来自 SQL Server 的未知令牌

VC programming ADO accesses MSSQL and runs relevant TCP messages of "select * from student where STU_NUM='0 'and STU_NAME='0'"
22:55:12 TRN -->> <IP_ADDRESS>:57192 <166> ................&.....p......p.s.e.l.e.c.t. .*. .f.r.o.m. .s.t.u.d.e.n.t. .w.h.e.r.e. .S.T.U..N.U.M.=.'.0.'.
.a.n.d. .S.T.U..N.A.M.E.=.'.0.'...&........&........&..
[03 01 00 A6 00 00 01 00 FF FF 02 00 00 00 00 01 26 04 00 00 00 E7 70 00 09 04 00 01 C6 70 00 73 00 65 00 6C 00 65 00 63 00 74 00 20 00 2A 00 20 00 66 00
72 00 6F 00 6D 00 20 00 73 00 74 00 75 00 64 00 65 00 6E 00 74 00 20 00 77 00 68 00 65 00 72 00 65 00 20 00 53 00 54 00 55 00 5F 00 4E 00 55 00 4D 00 3D 0
27 00 30 00 27 00 20 00 61 00 6E 00 64 00 20 00 53 00 54 00 55 00 5F 00 4E 00 41 00 4D 00 45 00 3D 00 27 00 30 00 27 00 00 01 26 04 04 02 80 03 00 00 01
26 04 04 04 80 04 00 00 01 26 04 00 ]
22:55:12 TRN <<-- <IP_ADDRESS>:57192 <396> .....4..................S.T.U..N.U.M..............S.T.U..N.A.M.E..............S.T.U..C.H.I.N.E.S.E........
.....0. 0. 0. 0. 0. 0.

How do I configure or modify Babelfish to be compatible with MSSQL?

Babelfish is doing exactly that. But as you can see we don't have exact 100% compatibility. For the sysXXX catalogs that are not existing in dbo but only in sys, you can create a view as I showed in my earlier response. Other than that, we are continuously working to extend the coverage of Babelfish. You may want to go to the recently-released v.2.2.0 (on PG 14.5) which supports significantly more T-SQL features than your 1.3.x version (though I think the issues you mentioned above will exist there too)

I have installed 1.3.2.How do I upgrade to 2.2.0?
---Original---
From: "Rob @.>
Date: Thu, Nov 17, 2022 16:28 PM
To: @.>;
Cc<EMAIL_ADDRESS>Subject: Re: [babelfish-for-postgresql/babelfish_extensions] [Bug]:MSOLEDBSQL.1 Write is normal, read fails, and TCP communication message isrecorded (Issue #853)
How do I configure or modify Babelfish to be compatible with MSSQL?
Babelfish is doing exactly that. But as you can see we don't have exact 100% compatibility. For the sysXXX catalogs that are not existing in dbo but only in sys, you can create a view as I showed in my earlier response. Other than that, we are continuously working to extend the coverage of Babelfish. You may want to go to the recently-released v.2.2.0 (on PG 14.5) which supports significantly more T-SQL features than your 1.3.x version (though I think the issues you mentioned above will exist there too)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

The log mentioned above includes the request packet. As per the log, the driver is trying to open a DYNAMIC and OPTIMISTIC cursor for executing the select query. As of now, we don't support OPTIMISTIC, SCROLL-LOCKS and UPDATABLE cursors (link).
I'll assume that you're calling a driver API to execute the SELECT statement that's internally sending this cursor open request. Could you please share the code snippet for executing this SELECT query?

Update babelfishpg_tsql.so file?
------------------ 原始邮件 ------------------
发件人:                                                                                "babelfish-for-postgresql/babelfish_extensions"                                                                                @.>;
发送时间: 2022年11月17日(星期四) 下午4:55
@.>;
<EMAIL_ADDRESS>主题: Re: [babelfish-for-postgresql/babelfish_extensions] [Bug]: MSOLEDBSQL.1 Write is normal, read fails, and TCP communication message is recorded (Issue #853)
The log mentioned above includes the request packet. As per the log, the driver is trying to open a DYNAMIC and OPTIMISTIC cursor for executing the select query. As of now, we don't support OPTIMISTIC, SCROLL-LOCKS and UPDATABLE cursors (link).
I'll assume that you're calling a driver API to execute the SELECT statement that's internally sending this cursor open request. Could you please share the code snippet for executing this SELECT query?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

We need the application code that's causing this failure. We would like to reproduce the scenario using the same driver at our end.


[APM-CI][All] Make the trigger by comment case-insensitive

If I say: JenkinS run the tests pleasE


[Intel MKL] Quantized (int8) Pooling Support

This PR adds support for quantized pooling op with MKL-DNN. It also includes many clang-format changes.

Thank you @penpornk for your review. I have addressed your comments. Please let me know if anything else is needed.

@penpornk Thank you for the comments. I have run the newer version of clang-format and I added that as a new commit to the PR as you suggested.


Creation of community page

Docs Build status updates of commit efaec96:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/community.md
:white_check_mark:Succeeded
View

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.


Finished all API functions

Handles GET request for reviews and meta review
For GET reviews, handles different sort, page and count request
Handles POST request for new review
Handles PUT request to vote for or report a review
Trello: https://trello.com/c/5CKMJnDF

 MVP Server for Ratings & Reviews


chore: vendor wait-port to unblock CI

🎉 Thanks for submitting a pull request! 🎉
Summary
Fixes #<replace_with_issue_number>

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[ ] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

I guess we probably don't need to do this, because the pr is already merged, so hopefully there will be a release soon.


Struggling to get SQLite3 3.35+ installed

I need* 3.35+ in order to retrieve the ID of the newly created row (as suggested here), like:
db.run('INSERT INTO table VALUES (?, ?) returning table_id', null, 'value', (err, row) => {
    console.log(row.table_id)
})

For Ubuntu 20.04 I compiled 3.36 but that only takes care of the OS version (which I wouldn't even need upgraded), not the version used by node-sqlite3. Apparently node-sqlite3 ships with its own binary (3.34) and while there are instructions to build from source my issue is that I'm using Yarn, not NPM, and --build-from-source is not a thing in Yarnworld. I've tried running npm_config_build_from_source=true yarn add sqlite3 but I keep running into various errors from node-gyp (from Python not being found to bcrypt issues..).
Should I continue struggling with the above, or:

Is there another way to get the primary key of the last insert (this.lastID as suggested in some places returns undefined)?
Should I give up Yarn and try installing from source with NPM as per official instructions?
Is there something else I could do?

*) unless someone has a better suggestion

Solved! 🙌

Is there another way to get the primary key of the last insert (this.lastID as suggested in some places returns undefined)?

Yes! Or no? But it is actually possible to get the lastID right now. I'm getting a value with this.lastID if I change the arrow function into a regular one.
I.e. from this:
db.run('INSERT INTO table VALUES (?, ?) returning table_id', null, 'value', (err, row) => {
    console.log(row.table_id)
})

To this:
db.run('INSERT INTO table VALUES (?, ?) returning table_id', null, 'value', function (err, row) {
    console.log(this.lastID)
})

Hopefully that helps someone.


Failed to set 'pids.max' attribute on '/system.slice/softether-vpnserver.service'

Hi
i use ubuntu 22 and i get this error in systemd :
● softether-vpnserver.service - SoftEther VPN Server
     Loaded: loaded (/lib/systemd/system/softether-vpnserver.service; enabled; vendor preset: enabled)
     Active: active (running) since Mon 2023-12-18 04:13:47 CET; 20h ago
   Main PID: 190700 (vpnserver)
      Tasks: 46 (limit: 16777216)
     Memory: 23.1M
        CPU: 7h 46min 23.139s
     CGroup: /system.slice/softether-vpnserver.service
             ├─190700 /usr/libexec/softether/vpnserver/vpnserver execsvc
             └─190701 /usr/libexec/softether/vpnserver/vpnserver execsvc

Dec 18 04:13:47 softether systemd[1]: Starting SoftEther VPN Server...
Dec 18 04:13:47 softether vpnserver[190693]: The SoftEther VPN Server service has been started.
Dec 18 04:13:47 softether systemd[1]: Started SoftEther VPN Server.
Dec 18 04:14:19 softether systemd[1]: softether-vpnserver.service: Failed to set 'pids.max' attribute on '/system.slice/softether-vpnserver.service' to '16777216': Invalid argument
Dec 18 04:14:19 softether systemd[1]: softether-vpnserver.service: Failed to set 'pids.max' attribute on '/system.slice/softether-vpnserver.service' to '16777216': Invalid argument
Dec 18 04:14:19 softether systemd[1]: softether-vpnserver.service: Failed to set 'pids.max' attribute on '/system.slice/softether-vpnserver.service' to '16777216': Invalid argument

i use ppa to install SE

https://code.launchpad.net/~paskal-07/+archive/ubuntu/softethervpn

# Adding SoftEtherVPN PPA to your system
sudo add-apt-repository ppa:paskal-07/softethervpn
sudo apt-get update

# Upgrade System
sudo apt-get update && sudo apt-get upgrade

# Install SoftEtherVPN
sudo apt-get install softether-vpnserver

# Start service
systemctl restart softether-vpnserver
systemctl enable softether-vpnserver
systemctl status softether-vpnserver

uname -a output
Linux softether 6.1.68-x64v3-xanmod1 #0~20231213.gead1733 SMP PREEMPT_DYNAMIC Wed Dec 13 22:13:53 UTC x86_64 x86_64 x86_64 GNU/Linux

any help ?

Hello, we are not proficient in 3rd party packaging, sorry.
please ask people who maintain that PPA


Data files in central place

As of now all files are randomly over the system. Some in /srv some in /etc some in /var.
Why can't we have 1 location on the system where everything lives.
Config in /etc/ is fine btw

This approach was taken as it is how many applications work.
Nginx is a good example of a program that does this, as well as PHP.
Configs are stored in /etc, any runtime data is stored in /var.
http://www.thegeekstuff.com/2010/09/linux-file-system-structure/?utm_source=tuicool
While it's old, it is still accurate, and the approach taken by us by default.
You can easily change where files are located by changing the config and telling it a new location.

Thanks for the info about that.


Adding aws s3 honeybucket dataset

Dataset:

Discovery of open public AWS s3 buckets.
access attempts to list and write objects to s3 buckets

@Cyb3rWard0g Not entirely sure about how should i generate the Id for dataset.
I just added randomly. let me know if any changes are required.

Looks good brother! ID looks good.

Thank you very much!