Question about Reinterpret Cast Conversion Functions

Hi  everyone,
As far as I know about reinterpret Intrinsics, they can reinterpret vector between different type under the same SEW/LMUL, or between different SEW under the same LMUL.
1. I don't see the reinterpret between float type to different SEW float type, such as vfloat16m1_t to vfloat32m1_t
2. If  I want to reinterpret vint8m1_t to vuint16m1_t, I can be achieved by using  __riscv_vreinterpret_v_i16m1_u16m1(__riscv_vreinterpret_v_i8m1_i16m1(a)). If this is right, How do I reinterpret vfloat16m1_t to vint8m1_t?
Thanks!

f16→i8 can be __riscv_vreinterpret_v_i16m1_i8m1(__riscv_vreinterpret_v_f16m1_i16m1(x)).
f16→f32 is a more unwieldy __riscv_vreinterpret_v_u32m1_f32m1(__riscv_vreinterpret_v_u16m1_u32m1(__riscv_vreinterpret_v_f16m1_u16m1(x))), but given that there's not much use for it, I think it's fine.

Correct: the current API doesn't form the full Cartesian product of conversions, but there should be a path between any pair of types with same EMUL.

Thanks for your reply, my problem is solved！


Update poetry lock and reformat

Update Poetry Lock\nRan poetry lock, install, then ni-python-styleguide fix to reformat any changes.\n---\nCreated by Github action

Thank you for contributing! :wave:


Any way to follow the upload's progress ?

Hi everybody,
I'm currently using your FileProvider to write files in a non-secure FTP server and I actually got a working code
let filename = "test.txt"             print("Test started:.")             let data = "Hello world".data(using: .ascii)             self.documentsProvider?.writeContents(path: self.ftpPathPhotos + "/" + filename, contents: data, overwrite: true) { (error) in                  if let theError = error {                      print("error = \(theError)")                  } else {                     print("alllllllright")                 }             }
Question is :
I was wondering if there is any way to follow the upload's progression, cause I saw in your source repo that you've got something close but I can't figure it out ?
In advance cheers and have a great day
Xavier

Hi,
You need the provider's delegate.
Example:
import FilesProvider

class WhatEver: UIViewController or Whatever, FileProviderDelegate {
// Here you have your stuff
// The important thing is to implement 'FileProviderDelegate'
// Then in code you will create the object using something like:
documentsProvider = FTPFileProvider(baseURL: your_url, mode: .your_mode, credential: your_credentials, cache: .your_cache_manage)
// After that, do:
documentsProvider.delegate = self // This will delegate some FilesProvider function to this class

// You will need to add the following stubs from the 'FileProviderDelegate'
func fileproviderSucceed(_ fileProvider: FileProviderOperations, operation: FileOperationType) {
    // Called when transaction finish successfully
  }

  func fileproviderFailed(_ fileProvider: FileProviderOperations, operation: FileOperationType, error: Error) {
    // Called when there's some kind of error in the transaction
  }

  func fileproviderProgress(_ fileProvider: FileProviderOperations, operation: FileOperationType, progress: Float) {
    // This is the one you want. Called with the progress. It is a float that goes from 0.0 to 1.0
  }

}

With that delegate you will be able to manage the progress of the transactions. Upload and download.

Hi,
You need the provider's delegate.
Example:
import FilesProvider

class WhatEver: UIViewController or Whatever, FileProviderDelegate {
  // Here you have your stuff
  // The important thing is to implement 'FileProviderDelegate'
  // Then in code you will create the object using something like:
  documentsProvider = FTPFileProvider(baseURL: your_url, mode: .your_mode, credential: your_credentials, cache: .your_cache_manage)
  // After that, do:
  documentsProvider.delegate = self // This will delegate some FilesProvider function to this class

  // You will need to add the following stubs from the 'FileProviderDelegate'
  func fileproviderSucceed(_ fileProvider: FileProviderOperations, operation: FileOperationType) {
    // Called when transaction finish successfully
  }

  func fileproviderFailed(_ fileProvider: FileProviderOperations, operation: FileOperationType, error: Error) {
    // Called when there's some kind of error in the transaction
  }

  func fileproviderProgress(_ fileProvider: FileProviderOperations, operation: FileOperationType, progress: Float) {
    // This is the one you want. Called with the progress. It is a float that goes from 0.0 to 1.0
  }

}

With that delegate you will be able to manage the progress of the transactions. Upload and download.

Hi,
You need the provider's delegate. Example:
import FilesProvider

class WhatEver: UIViewController or Whatever, FileProviderDelegate {
  // Here you have your stuff
  // The important thing is to implement 'FileProviderDelegate'
  // Then in code you will create the object using something like:
  documentsProvider = FTPFileProvider(baseURL: your_url, mode: .your_mode, credential: your_credentials, cache: .your_cache_manage)
  // After that, do:
  documentsProvider.delegate = self // This will delegate some FilesProvider function to this class

  // You will need to add the following stubs from the 'FileProviderDelegate'
  func fileproviderSucceed(_ fileProvider: FileProviderOperations, operation: FileOperationType) {
    // Called when transaction finish successfully
  }

  func fileproviderFailed(_ fileProvider: FileProviderOperations, operation: FileOperationType, error: Error) {
    // Called when there's some kind of error in the transaction
  }

  func fileproviderProgress(_ fileProvider: FileProviderOperations, operation: FileOperationType, progress: Float) {
    // This is the one you want. Called with the progress. It is a float that goes from 0.0 to 1.0
  }

}

With that delegate you will be able to manage the progress of the transactions. Upload and download.


The Framework 'Microsoft.WindowsDesktop.App', version '3.1.0' was not found .net core 3.1 ubuntu

I have a very basic hello world type winforms app I built in visual studio 2019.
I installed .net core 3.1 runtime and sdk on ubuntu 20.04 using directions supplied from docs.ms website.
When I try to run my application I get the error:
The Framework 'Microsoft.WindowsDesktop.App', version '3.1.0' was not found
what do I need to do to get this to work

You can't make it work, Winforms are only supported on Windows.

@svick is correct, the WinForms and WPF are both Windows-only. You might check out Xamarin.Forms (or, the upcoming .NET MAUI) which has some GTK# support.

For cross-platform .NET UI, have a look at the Avalonia framework as an alternative to the Windows-only winforms.


Deactivated Grafana reporting in monitoring example yaml.

Change description
Added two configuration parameters to Grafana for disabling most of it's reporting capabilities. Feedback links are let on.
Testing
Tested for 12h on dev kube cluster and no report requests were noticed, while the monitoring example kept working as expected.
References
Fixes: GHSA-mxv3-27mh-wcfj security advisory

Changes lgtm, I'll approve following fixing the commit signature

I am sorry, did not know about that.
The new commit should be fine now.

Took me a while to make a verified sign f1a8bac422caa1eddaff2891d41fd9e40bdff5a6

/test

Hi @mvtab , there's no need to merge the main branch into the PR each day; Cilium is a fast-moving project and you'll almost always be out-of-date with respect to the main development branch. Perhaps after a week or two it may be useful to do a rebase (not merge) but otherwise I would suggest to hold off on making further changes.
If you can rebase the PR once more against the main branch to drop the merge commit + force push into this branch, then I can go and retrigger the tests again.

Hi @joestringer, noted. Thanks.
The whole two lines of code should be finally ready now 😅. ceed3923d1c69a61182ee8f0cfbd432d9ef976a6

/test


docs: spiderpool supports calico in bgp mode

name: Pull request
about: Tell us about your contribution
labels: ["pr/release/none-required"]

Thanks for contributing !
Before submitting a pull request, make sure you read about our Contribution notice here: https://spidernet-io.github.io/spiderpool/contributing/development/pullrequest/#need-review
What this PR does / why we need it:
docs: spiderpool supports calico in bgp mode

/cc @weizhoublue  PTAL :)


Ultrawide monitor: set window maximum size

I don't know if what I'm after is possible, but I figured I'd try to ask anyways.
For users with ultrawide monitors, when there is only one window open, it fills the entire display, which is annoying because it is huge. Usually I will open two other "placeholder" windows in three-column layout that, while I'm not actively using, are just there to resize the main window to something more manageable.
Would it be possible to set the maximum window size (and possibly center that window) when only one window is on the desktop?

when only one window is on the desktop

You can make the window  float.

Indeed, I was hoping to automate that process, perhaps through window rules. Seems like windows don't respect the maximum size rule; I'm assuming krohbkite is overriding it? Otherwise, is there a way to float the window using rules?

I'm assuming krohbkite is overriding it?

The kwin script resizes a window and can assign and remove window props.

Otherwise, is there a way to float the window using rules?

 If you mean global window rules - no, the script will override it.

Sounds good, looks like there's no way to achieve what I'm thinking about without adding support in krohnkite. Please feel free to close this issue if that's not something you with to support.


Fix GMM example for matplotlib 3

On the GMM (Gaussian Mixture Model) example, fix the following.

Use scipy.stats to compute bivariate normal.  matplotlab.mlab.bivariate_normal is deprecated since matplotlib 2.2 and is removed since 3.0
Surpress the warning message:

'c' argument looks like a single numeric RGB or RGBA sequence, which should be avoided as value-mapping will have precedence in case its length matches with 'x' & 'y'.  Please use a 2-D array with a single row if you really want to specify the same RGB or RGBA value for all points.

% python gmm.py --output-image=tmp.png
Running CPU...
train_accuracy : 92.143100
test_accuracy : 92.000000
 CPU :  1.087882 sec
Running GPU...
train_accuracy : 92.030900
test_accuracy : 92.000000
 GPU :  0.651976 sec
% imgcat tmp.png

pfnCI, test this please.

Jenkins CI test (for commit b00f952b9f829b573e98b0babeaae7fb01f9f2f9, target branch master) failed with status FAILURE.

pfnCI, test this please.

Jenkins CI test (for commit b00f952b9f829b573e98b0babeaae7fb01f9f2f9, target branch master) succeeded!

I locally confirmed that the example now works with matplotlib 3.1.0.


Use of cache

We've done testing with various "Use of Cache" settings and get nearly the exact same runtime.  I've read elsewhere that this setting was made obsolete to be managed by the MRP engine dating back to 2014?  Is this not the case?  Would be nice to note this in the documentation or just deprecate the setting.

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

ID: f4d94c07-68fa-b36b-9a2b-e79dbd0b474f
Version Independent ID: 74c49920-d228-2d81-8530-0f5a557e1aad
Content: Improve master planning performance - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/supply-chain/master-planning/master-planning-performance.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @t-benebo
Microsoft Alias: benebotg

The "Use of cache" setting is applicable (it has not been deprecated). Its influence on your master planning runtime will depend on your data composition. In some cases it may not affect while in others it can make a huge difference.

Thanks Beatriz, this is very helpful.
On Thu, Aug 29, 2019 at 2:05 AM Beatriz Nebot Gracia <
<EMAIL_ADDRESS>wrote:

The "Use of cache" setting is applicable (it has not been deprecated). Its
influence on your master planning runtime will depend on your data
composition. In some cases it may not affect while in others it can make a
huge difference.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/dynamics-365-unified-operations-public/issues/1623?email_source=notifications&email_token=ABTGUVTJQIZDLNOZ5FNJMPLQG6GNFA5CNFSM4IRTUA3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5NZRKQ#issuecomment-526096554,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABTGUVRWI76J6PHMCWWLPQTQG6GNFANCNFSM4IRTUA3A
.


Replace git clone with go get in README

This is the standard way to fetch a Go repo and what most people are used
to. The -u flag updates the repo if it's already downloaded. The -d flag
prevents building the binary, because we want to use our Makefile so that
dep is run first.

I believe it should be possible to actually use go get to install the binary once your PR to remove the bindata build step is in. Possibly would require vendoring the dependecies.
Would that be worth looking into?

True, true.
I don't really mind. I guess the benefit is still negated because you'd want to use make install unless you ran cf install-plugin yourself?


Update v0.2 release notes

Hey @rachx @INCENDE @fanwgwg, could you update the v0.2 release with notes about what features our app supports right now? Thanks!

Done!


Add Environment.Clear

Add new method Environment.Clear which removes all values.
New method should have documentation comment with code example and be covered by tests.

I'll give this one a go @gavv.

Cool, thank you. Nooe, sorry for misunderstanding, I meant just to remove everything, keys and values. To allow the user to reset state.


Ability to add a link to ministry name in Microsite Header

Add ability to add ministry name to microsite header link so that teams can show the parent ministry, and not have to include all of that information in the service name.

@ArakTaiRoth Reframe story to allow for a property that can be changed and is completely open (not HTML, but all strings)


Fix zip command not found on Windows

Issue #, if available:
Issue #135
Description of changes:
Changes are made to the graphql-elasticsearch-transformer package.

Added npm package bestzip
Using bestzip instead of zip in packages.json

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@kavengagne, shouldn't the bestzip package be a dev dependency?

@kaustavghosh06 My bad. I updated it.


CORS Header issue: Duplicate Access-Control headers in REST API "OPTIONS" method

One possible bug found in REST API:
OIDplusPagePublicRestApi::handle404() calls originHeaders().
If REST is called with the request method "OPTIONS", then restApiCall_OPTIONS() will additionally send some of these headers.
(Note: At least for IIS, the software seems to swallow the "OPTIONS" method and not forward it to PHP. I wonder if any software forwards "OPTIONS" to PHP?!)
Here is the comparison of the contents:

Header
originHeaders
restApiCall_OPTIONS

Access-Control-Allow-Credentials
true
true

Access-Control-Allow-Origin
<HTTP_ORIGIN> or * otherwise
*

Access-Control-Allow-Headers
If-None-Match, X-Requested-With, Origin, X-Frdlweb-Bugs, Etag, X-Forgery-Protection-Token, X-CSRF-Token
Keep-Alive,User-Agent,Authorization

X-Frame-Options
ALLOW-FROM <HTTP_ORIGIN> or absent if <HTTP_ORIGIN> is not set

Access-Control-Expose-Headers
Etag + X-CSRF-Token + All <headers_list()>

Vary
Origin

Access-Control-Allow-Methods

GET, PUT, POST, DELETE, PATCH, OPTIONS

We should remove all headers from OIDplusPagePublicRestApi::handle404() (except Access-Control-Allow-Methods) and instead merge them into originHeaders().
@wehowski Since there are some differences in the values and I don't have much knowledge about CORS, I ask you to take a look and help me with the fix (after all, you are the author of originHeaders()). Thank you very much.

Hello Daniel,
there are 2-3 possible solutions I used before in other projects for this kind of problem:

.htaccess: This is dirty and not configurable well    Header onsuccess unset Access-Control-Allow-Origin  Header always set Access-Control-Allow-Origin "*"
ob_start() and emit output once after removing doublicate headers header_remove...
Use spatie once:  You can wrapp everything that happens in the originHeaders() function into https://github.com/spatie/once. Here is an example how I use it in the webfat file:  \frdl\booting\once(function() use(&$StubVM) {
  	if(!empty($StubVM->getFileAttachment(null, null, false))){
  		\spl_autoload_register([$StubVM,'Autoload'], true, true);
  	}
    });

       $this->autoloadRemoteCodebase(true);

       \frdl\booting\once(function() use(&$StubRunner) {
  	 $StubRunner->getStubVM()->_run_php_1( $StubRunner->getStubVM()->get_file($StubRunner->getStub(), '$STUB/bootstrap.php', 'stub bootstrap.php'));
  	 $StubRunner->getStubVM()->_run_php_1( $StubRunner->getStubVM()->get_file($StubRunner->getStub(), '$HOME/detect.php', 'stub detect.php'));
        });

About to pass OPTION requests to PHP: I am not sure, I must re-look about it, maybe via server-configs or autoprepend file or php-servers/listeners (do not use the last one in production they say).
I must read, learn and test it...

Your solutions are rather complex. We don't need such a complexity.
We just need one set of good parameters, and put everything in originHeaders().
The real question is just what are the "good parameters"? I cannot tell.

Maybe this works?
	header_remove("Access-Control-Allow-Origin");
	header("Access-Control-Allow-Origin: ".strip_tags(((isset($_SERVER['HTTP_ORIGIN'])) ? $_SERVER['HTTP_ORIGIN'] : "*")));

Unfortunately ist does not help to remove the header first.

Does it make sence to only use the access headers if POST/PUT/DELETE request?
I believe the the Access-Control headers are used if POST with X-Requested-With-Header e.g. ?

I have no experience with CORS, so I leave the decision to you.
I would like that the whole CORS stuff is located in one single method and that this method is only called once.
I need to have a set of good values (see my table above), and these set of values should be in that method.
And then, I will make sure that this method is only called once (I will take care of it)
I also need your reproduction in re #21 so that we can find out why your have duplicate headers and I not. Please quickly add a test debug output to the CORS output methods to check if they are indeed called twice.

Ok, sorry!
I am preparing a page with tests for the use case with a reproduction of the error and demos.
The js-console printed out that the header is dublicate, although if the function might work if called only once, my settings don't look correct!
I will notify you in the evening if my tests and reports are complete.

Hallo Daniel,
Please excuse the inconvenience, again the Proxy is the culprit!
This is the page to reproduce the error: https://frdlweb.de/api/rdap/whois-test
Access to XMLHttpRequest at 'https://webfan.de/apps/registry/plugins/viathinksoft/publicPages/100_whois/whois/webwhois_original.php?query=weid%3AEXAMPLE-3$format=json' from origin 'https://frdlweb.de' has been blocked by CORS policy: The 'Access-Control-Allow-Origin' header contains multiple values 'https://frdlweb.de, https://frdlweb.de', but only one is allowed.
OK 200: https://registry.frdl.de/plugins/viathinksoft/publicPages/100_whois/whois/webwhois_original.php?query=weid%3AEXAMPLE-3$format=json


I never joined the space "Hexafox .. " I can not open that space from within the spaces area in the app... I have this bug for several weeks now, and already deleted cached memory...

reported by @stany:matrix.org
https://github.com/matrix-org/riot-android-rageshakes/issues/954

Server issue


Square Image/Video capture

Hello, I am using this library, but I need to get a square image. Is there a setting for this? (if there is not,  could it be possible to crop it after its taken?

There is no current setting for this. You would need to crop the image after it is taken.
I recommend adding overlay views overtop the camera to give the appearance of a square view finder, then cropping the centre of the image into a square once it is taken.
Good luck!


Hydrogen 0.7.0 slow startup

First of all this is a great software! In the newest release I noticed a small problem:
The first startup of Hydrogen is slow.
It takes a few seconds to connect to the kernel and execute the command. In previous versions this was almost instant.
If the kernel is running everything is as fast as usual.

May be you can have a look at this.

I write this code on my SSD and it shines, but on my extra slow HDD it works the same as yours. Besides, for me previous version works slower than new.
Just to clear situation, please, before hydrogen:run launch couple of times this:
time ipython kernelspec list --json
And after that run hydrogen. If this will speedup this command and interface freezing will be gone - notify me here. Thanks

Thanks for the response.
I think I know where the Problem is coming from:
I have the latest version of Jupyter installed.
If I run time ipython kernelspec list --json I get the following warning:
[TerminalIPythonApp] WARNING | Subcommand `ipython kernelspec` is deprecated and will be removed in future versions.
[TerminalIPythonApp] WARNING | You likely want to use `jupyter kernelspec`... continue in 5 sec. Press Ctrl-C to quit now.

The time is real 0m5.345s. Notice the 5 second delay.
If I run time jupyter kernelspec list --json everythin is fine. The time here is real 0m0.331s.


Smart gc{motion}

Hi, I'm no sure to call this bug or not, but coming from tcomment I miss this feature the most.
Let's say I have the following situation
local str = 'This is a test'
-- local str = 'This is not a test'

with tcomment I can do gc{motion} let's say j or k and the line will switch comment and become
-- local str = 'This is a test'
local str = 'This is not a test'

with Comment.nvim resulting code is
-- local str = 'This is a test'
-- -- local str = 'This is not a test'

so to get the result like tcomment I have to press gcc twice, one on each line.
It's not much but I think this behavior is quality of life improvement.
Thank you for this amazing plugin 👍🏽

We've already discussed this feature in #17
Since many users would benefit from this feature I think it should be included in extra (or experimental) mappings

Closing in favour of #17


Pin Distributed dev mode installed in CI

We are currently still pinned to Dask/Distributed 2024.1.1 and discussions on if and how to unpin for RAPIDS 24.04/UCXX 0.37 are still ongoing. Meanwhile, we must install a version of Distributed that is compatible with the Dask version installed.

/merge


Extracting genotypes from Methylation data

Thanks for having such a clear package and workflow.
I was wondering is it also possible to just extract genotypes from the methylation data without having to input genotypes derived from another source?

I'm not sure what you mean by "input genotypes derived from another source"?   The microarrays have probes that measure signal for just over 60 SNPs. These signals can be retreived from the QC objects using the meffil.snp.betas() function:
snp.betas <- meffil.snp.betas(qc.objects)
You could then use our internal function to estimate genotypes from those signals:
genotypes <- meffil:::calculate.beta.genotypes(snp.betas)
Is this what you are looking for?

Yes!!! this is awesome thanks!
samplesheet <- meffil.read.samplesheet("/path/to/folder/[read.450k.sheet] Found the following CSV files:
[1] "/path/to/folder/sheet_meffil.csv"
Warning message:
In FUN(X[[i]], ...) :
  Could not infer array name for file: /path/to/folder/METH/sheet_meffil.csv

Any idea what Im doing wrong here? I have EPIC data, so perhaps I need to specify this?
Header of sheet_meffil.csv looks like this:
Sample_Name,Sex,Slide,sentrix_col,Basename
SAMPLE1,M,203952880001,01,/path/to/folder/203952880001//203952880001_R01C01
SAMPLE2,M,203952880001,01,/path/to/folder/203952880001//203952880001_R02C01

Note that this is a warning, not an error.  The function uses the 'array name' to construct sample names.  However, because you have already defined samples names (see Sample_Name column), sample names don't need to be generated so there is no error. You can proceed without making any changes.
In case you are curious, the function was looking for a column named ArrayID or Sentrix_Position to provide the position of the sample on the slide, e.g. "R04C02" indicating that the sample is on row 4 column 2. The function generates a unique sample name for each sample by combining the slide identifier and this position, e.g.  "203952880001_R04C02".


to do: capture lto id

Need to capture lto id of A drive and post the value to the corresponding RS record using the update_field API function.

done


add theme Help Center

Simple and responsive Jekyll theme for help center.
Demo: https://github.com/gustavoquinalha/jekyll-help-center-theme

@gustavoquinalha  Thanks for adding the Help Center theme. Great theme. Keep it up. Cheers. Prost.


Add Date+Convertible to support using Fluent 1.2 with the new .custom…

…(_:type) API with DATETIME and TIMESTAMP as ‘type’.

Current coverage is 95.94% (diff: 0.00%)

Merging #27 into master will decrease coverage by 1.26%

@@             master        #27   diff @@
==========================================
  Files            28         29     +1
  Lines          1437       1456    +19
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1397       1397
- Misses           40         59    +19
  Partials          0          0

Powered by Codecov. Last update 4cf8c76...2541944

I seem to remember some conversation in Slack about doing the dates in Int instead of String, giving more freedom to the final application. This would require more updates on the database driver's side though.

Code looks good. Let's decide on supporting unix timestamps then get this merged! :)

I'm actually no longer supportive of PR as written. In order to support sub-second time report, I had to change the string format to include "HH:mm:ss.sssssss". But this then breaks compliance with standard JSON timestamps, since the same conversion will be used from JSON to node and node to MySQL.
I think it makes sense to support Double (since it can represent all time formats) but right now the MySQL code cannot use that since it does not have enough hints when preparing a statement to choose to convert the Double into a MYSQL-appropriate format.


~solid-pages import not working on Solid JS

When I tried to import this plugin into a Solid JS project, I ran into a problem - the path ~solid-pages was not visible and gave an error. However, according to one guide for version 0.18.2, I found a solution to this problem. Instead of ~solid-pages when importing, it was necessary to write virtual:generated-pages. This solved my problem.
In this regard, please tell me, is it just that the imports have not been changed since the last version or is there an error in the documentation?

What version are you using? Solid JS support is only available in beta channel for now.

I'm using latest - 0.20.2.

Even with a directive /// <reference types="vite-plugin-pages/client-solid" /> the working path is still the same - virtual:generated-pages

That is not beta version, use npm install -D vite-plugin-pages@beta to install beta version.

Ok, thanks. Now both methods work.


startOf confusion

The docs are saying:
moment().startOf('year');    // set to January 1st, 12:00 am this year

The console is saying:
moment().startOf('year').toDate;    // Wed Jan 01 2014 00:00:00 GMT+0200 (EET)

It's not 12:00 am, it 00:00.
And the second option is good for me. This is what I want and what seem logical. But then docs should be updated.

12:00 am is 00:00/midnight -- 12:00 pm is noon

That is actually incorrect, because pm stands for Post Meridiem, meaning "after noon", so 12pm technically means 12 hours after noon, which is midnight.
The fact that moment parses midday/noon as 12pm is pretty confusing. But then again, the whole 12 hour clock is pretty ridiculous and confusing, so 🤷🏼♂


[ISV-1907] Operator release pipeline no longer pushes images into OSD registry

@mantomas I'll be on PTO tomorrow, so feel free to either merge this if it looks good, or push your changes first if they're ready, and I'll just deal with the conflicts.


[Enum Request] - Table Reservation Entry - Field 11 "Source Subtype"

Please change the field into Enum so it can be extensible

Sorry, this field cannot be transformed to enum, it does not make any sense. We only use Integer value of this field.

Sorry but I don't understand.
I want to create a new document type extending the enum Sales Document Type with my values 50000,50001.
But when I use the reservation entry it's not possible to use these values because it's an option and not an integer
field(11; "Source Subtype"; Option)
{
Caption = 'Source Subtype';
OptionCaption = '0,1,2,3,4,5,6,7,8,9,10';
OptionMembers = "0","1","2","3","4","5","6","7","8","9","10";
}


NodeMCU rebooting after a few seconds - continuous loop

I have just flashed this to my nodemcu but when it starts running i get the 'ESP8266 IR controller' output in console then the device reboots and just keeps doing this over and over.
I have all default settings at the moment just to test.
NodeMCU 0.9 (esp-12 module), 80mhz, 115200, 4m (3m spiffs)
` ets Jan  8 2013,rst cause:4, boot mode:(3,2)
wdt reset
load 0x4010f000, len 1384, room 16
tail 8
chksum 0x2d
csum 0x2d
v3de0c112
~ld
)
`

wdt reset means its crashing
Why exactly is usually tough to pinpoint, but here's some things to start with
Make sure you're using the latest version of the code
If you have an extra nodemcu board, try that and see if its reproducible, these do experience hardware failures from time to time
Disconnect everything from the nodemcu (remove from breadboard) and try running it isolated and see if it boots
If those steps don't work let me know

I had the exact same problem using a Adafruit Huzzah. Found out it did not like the wifi reset on GPIO 10. Make line 35 of the code = to 12. Even  though 12 is already defined as IR send pin 3 it should boot into AP mode. Also u must change line 46 to the static IP u want and change line 47 to your gateway address. Even though these can be changed in setup it won't take them.

While I agree with the first statement @billcramer07 made, there's no reason overriding the WiFiManager settings should have any impact on booting

Everytime I would run WiFiManager and enter my gateway and static IP, WiFiManager would override it with the defaults at lines 46 and 47.

BTW, your video doesn't show the 3 additional lines in Setup to enter the IP, gateway and subnet mask. Took a while to figure it all out. EXCELLENT SKILL FOR ALEXA!!!

Those are just placeholder values that are overwritten before they are used line 334-336 from the values stored in the config.json file from WiFiManager
The values are assigned to the staticIPConfig on line 360
You have to have a placeholder value set so that when WiFiManager enters config mode it will see that it’s a static configuration and prompt for the values for the IP/GW/SN
From: billcramer07
Sent: Thursday, December 28, 2017 10:59 PM
To: mdhiggins/ESP8266-HTTP-IR-Blaster
Cc: Michael Higgins; Comment
Subject: Re: [mdhiggins/ESP8266-HTTP-IR-Blaster] NodeMCU rebooting after a fewseconds - continuous loop (#108)
Everytime I would run WiFiManager and enter my gateway and static IP, WiFiManager would override it with the defaults at lines 46 and 47.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Yeah the video was made when everything was dynamic, I added the static stuff after the fact to make port forwarding more stable
From: billcramer07
Sent: Thursday, December 28, 2017 11:05 PM
To: mdhiggins/ESP8266-HTTP-IR-Blaster
Cc: Michael Higgins; Comment
Subject: Re: [mdhiggins/ESP8266-HTTP-IR-Blaster] NodeMCU rebooting after a fewseconds - continuous loop (#108)
BTW, your video doesn't show the 3 additional lines in Setup to enter the IP, gateway and subnet mask. Took a while to figure it all out. EXCELLENT SKILL FOR ALEXA!!!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I managed to get it working on an ESP-M3. I will have a look.
I can't remember if I tried changing the configpin on these other devices, I'll give that a go later though.
So far I've captured some codes for my TV and they appear to be working with Alexa. It's actually really simple once you get it all up and running which is brilliant.


libnma: bug fix removing path from eap scheme

Fixes known bug in libnma https://gitlab.gnome.org/GNOME/libnma/-/issues/18 By applying patch based on https://gitlab.gnome.org/GNOME/libnma/-/merge_requests/44
Description of changes
Fixes error when trying to create new VPN connection.
Which used to fail with error msg:
settings object created with schema 'org.gnome.nm-applet.eap'
and path '/org/gnome/nm-applet/eap/<uuid>/',
but path '/org/gnome/nm-applet/eap/' is specified by schema

Fixes: https://bugs.archlinux.org/task/76490
Fixes: https://gitlab.gnome.org/GNOME/libnma/-/issues/18

Built on platform(s)

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

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

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

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

This patch also solves my similar issue connecting to Wi-Fi (on aarch64-linux). I have not tested VPN functionality.

Reproduced the issue, this PR WFM. Same patch is also applied on Debian unstable and openSUSE TW, Fedora 37 is still on 1.10.2. nixpkgs-review happy, x86_64-linux.
Thanks.


Abil wind walk def fix #207

https://github.com/wurstscript/WurstStdlib2/issues/207

why closed?!

cause i failed commiting xD
see ..https://github.com/wurstscript/WurstStdlib2/pull/215


TimePicker panel auto close

What problem does this feature solve?
The panel of a timepicker covers elements of the page and having to interact with a click to close them  form fill feel tidious in our application.
What does the proposed API look like?
Could it be considerer an option to close the panel when every select has been clicked ?
for every clickable select in the panel (hour, minutes, seconds) if present, when it get's selected, you consider that the user made the concious choice of every part of the time definition, therefore close the panel.

This should be enhanced after #5770 merges, then you can change the input value and press enter to close panel.


Fix element spacing handling

I'm opening this pull request early to get your opinion on how the plugin should be handling element spacing.
I've added some additional tests with my current opinions:

[x] Collapse whitespace between inline mustache elements

Looks like this is already working

[x] Do not add whitespace between inline mustache elements if none already exists

Looks like this is also already working

[x] Do not add line breaks between sibling inline nodes when an element sibling is present (fixes #14)
[x] Collapse line breaks in a single text node into just a space, since that's the behavior of HTML anyway
[x] Treat elements with only empty whitespace as if they have no child nodes at all

I've added a fix for this

[x] Do not add additional blank lines for script-only templates

This appears to be caused by how the Fragment node is currently handled
I've added a fix for this

[x] Collapse multiple blank lines between HTML elements into a single

This is how vanilla HTML appears to be handled in Prettier core

[x] Do not add additional blank lines between HTML elements if none already exists

This is how vanilla HTML appears to be handled in Prettier core
Currently, this is already working but needs to be kept in mind when fixing the previous issue

looks good so far, happy to follow prettier wrt to the last points

I've added some additional tests and added a couple items to the todo list regarding whitespace and inline (Text/MustacheTag) nodes.

thanks for working out all the tricky spacing details! 👍


Oracle fix empty string as null handling in the test

Fixes #580.

@mloskot Is failure with installing Oracle transient, i.e. is it worth retrying? Or should I wait until you have time to look at this? Or just merge this nevertheless (I'm pretty sure it should fix the problem for Oracle)?

Copying note from https://github.com/SOCI/soci/pull/579#issue-257550117

The Oracle downloader () requires two secret environment variables.
The CI builds for PRs do not work anymore:

Pull requests sent from forked repositories do not have access to encrypted variables or data.
https://docs.travis-ci.com/user/pull-requests/#Pull-Requests-and-Security-Restrictions

We will have to apply some workaround (eg. run Oracle build job only for regular pushes).
Meanwhile, I'm merging this PR, so we see if the Oracle build is properly triggered.

Or just merge this nevertheless

Yes, that is what we have to do until I find workaround for the issue, hopefully later tonight.


AWS: Support for passing CpuOptions

Add support for setting the EC2 instance CpuOptions in the AWS provider.
We can set ami, instanceType, etc, but some workloads need cpu features turned on in the instance that are only enabled by setting values in CpuOptions;  specifically, AmdSevSnp.

I mostly use terraform, and they expose three options documented here:  https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/instance#cpu-options, with links to the underlying AWS docs

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale


Write Auton Fail safe (9 pts)

Write pseudocode or comments for the different actions of the auton. Multiple versions of this failsafe should be written depending on the starting position of the robot

Instructional Material: Exercise: Using the REV IMU

AutonDriveByGyro (adapted from RobotAutoDriveByGyro example) on the dev branch works as proof of concept. Currently (10/1/22) its only front wheel drive, but gyro and distances are working as expected.
Considering a separate branch (dev-auton) for autonomous development

I'll write some example comments later
// Place robot at starting position
// Drive forward

Tested Auton Blue 1 on 10/10/22

Liam is developing auton files based on this and others


"Error: The system cannot find the file specified() under Status column

I'm on Windows10, v0.25.0. I'm seeing the error in the title above. Already restarted the app, and disabled my VPN. no luck.

ah ok it was the 'Ransomware Protection' in Win10 blocking PicoTorrent from accessing my download's folder. Allow it and it works. Thanks!


gui: filter unused events

It may removes the annoying logs that occurs after a while.
[1669885771][iced_futures::subscription::tracker][WARN] Error sending event to subscription: TrySendError { kind: Full }
[1669885771][iced_futures::subscription::tracker][WARN] Error sending event to subscription: TrySendError { kind: Full }
[1669885771][iced_futures::subscription::tracker][WARN] Error sending event to subscription: TrySendError { kind: Full }

By filtering the events to use only the events listened by the application, we may not reach some buffer size limit

Doesn't this risk hiding potential bugs in the future? Or making them harder to debug?

I do not think so, the native events were listened only to get the CloseRequestedEvent, so they were ignored anyway.


Add support for XCode 14.3.1 and XCode 15

Describe the solution you'd like
Add more recent xamarin iOS versions and support newer XCode versions
Describe alternatives you've considered
There is no alternative
Additional context
My project uses libraries which do not work under XCode 14.2 anymore and the application crashes if built on appcenter using these versions. As there arent any newer versions available Im forced to manually release new versions using my Mac which defeats the purpose of paying for AppCenter...

This is preventing us from being able to deploy to iOS at all now.

+1 for this. I have a project that needs a newer version of Xcode to build/deploy.

Support told me they would upgrade this the first week of december but that is both too late and with no guarantee that it will happen like that.. Seriously we are paying for a service which doesnt care at all anymore

It has updated to 15.0.1 for me.

It has updated to 15.0.1 for me.

Native right?
Mine is Xamarin iOS, so unfortunaltely not for us yet


Add an OpenAI page to integrations section in docs

Provides a code sample with typical integration patterns for OpenAI and Statsig.

🚀 Deployed on https://6524bc301774250264d0d0ff--cozy-fox-0defec.netlify.app


Added norm specialisaitons.

This adds all the norm specialisations. It requires PR #1635 in QuTiP to be merged first.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

30 of 30 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 95.679%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
310

Relevant Lines:
324

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

30 of 30 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 95.679%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
310

Relevant Lines:
324

💛  - Coveralls


md-editable property for md-chip

This PR adds a 'md-editable' property  and 'edit' event on md-chip.
They are then used by md-chips to support editing the chips. When a chip is clicked it is removed from the list, and it's content is added to the input where the user can modify it then add it again.
I think it is quite straightforward and integrates nicely with the existing features. I hope you will agree.
On a side note: I wrote my name in the new chip example. I did it because of my lack of imagination, nothing else, feel free to write whatever you want :)

Man. Nice solution. I liked. And merged!
Your name will stay there! :D


fix(Global Tagging): re-gen service with latest API definition

PR summary
This PR contains:

a re-generation of the Global Tagging service and unit test code
a renovation of the integration test starting with a freshly generated source file
a new "examples" class that can be used to provide request examples to the service's online apidocs

PR Checklist
Please make sure that your PR fulfills the following requirements:

[x] The commit message follows the Angular Commit Message Guidelines.
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[ ] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

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


Fix deleting entire array when keyv.get

Please check if the PR fulfills these requirements

[x] Followed the Contributing guidelines.
[x] Tests for the changes have been added (for bug fixes/features) with 100% code coverage.
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Fix bug when in keyv.get put array of keys and one of keys expires. When this case happens deleted all keys from this array but need to delete only one that has expired

@vpoliatychko thanks so much for this big fix.


Private polls backend

Fixes #54
Implements a new functioning api method to backend called "api/polls/privatePoll". The api method returns all private information about a poll that the owner of the poll should see. Most importantly, it returns the answers that have been given to the poll. No new tests were added. Only one new method was added.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have added thorough tests

Codecov Report

Merging #68 (04922ce) into dev (7333f41) will decrease coverage by 0.81%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev      #68      +/-   ##
==========================================
- Coverage   62.37%   61.55%   -0.82%
==========================================
  Files          31       31
  Lines        1063     1077      +14
  Branches       99      100       +1
==========================================
  Hits          663      663
- Misses        387      400      +13
- Partials       13       14       +1

Flag
Coverage Δ

backend
65.85% <0.00%> (-0.96%)
:arrow_down:

dev
61.55% <0.00%> (?)

frontend
16.12% <ø> (ø)

staging
?

unit
61.55% <0.00%> (-0.82%)
:arrow_down:

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

Impacted Files
Coverage Δ

backend/src/controllers/pollController.ts
0.00% <0.00%> (ø)

backend/src/models/Poll.ts
62.17% <ø> (ø)

backend/src/routes/v1/poll.ts
0.00% <0.00%> (ø)

backend/src/services/VotingService.ts
75.34% <0.00%> (-9.28%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Support for Linux [Mint] System-wide media shortcuts

Checklist:

[X] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[X] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[X] I have double checked and can reproduce the issue

OS:
Linux Mint 19.1
GPMDP Version:
4.6.1
Issue Descriptions:
Linux Mint system-wide media hotkeys do not work with GPMDP. I have hotkeys set up that work with other media players (Rhythmbox and/or Spotify) but they don't seem to work with here.

Hitting Ctrl + Alt + X doesn't do anything in GPMDP. If I open Spotify and hit the same key playback resumes or pauses.

Steps to Reproduce:
Hit the hotkey, nothing happens.

I have since restarted the computer and the hotkeys seem to work fine now.
Can probably be closed.


The balance in wallet is cleared to 0.010631004 XDAG when my PC is powered off accidentally and powered it back

My question: How can I repair my wallet? Thanks.

Hi, did the address of your wallet was changed?

@huangfaqian probably invalid chain due to de-sync?

No updates for over two months. Closed.


Heartbeat blocking consumer on lapin-futures 0.15.0

I am hitting an issue where spawning the heartbeat on another task appears to block a basic_consume stream from handling messages ~%80 of the time.
If I skip spawning the heartbeat on another task, the consume stream gets messages every time.
Likewise, if I stop the heartbeat, the consume stream gets messages every time.

Which executor are you using?

I am using the default executor in tokio: https://docs.rs/tokio-threadpool/0.1/tokio_threadpool/struct.ThreadPool.html

@jgrund Hi,
I am encountering a similar issue, where not all of my messages are being consumed appropriately. I've tried disabling the heartbeat, the issue still persists.

Could you try with 0.19.0?
I'm releasing a few alphas until it's properly ready but things should already be better


Juniper SRX Parser update for perf improvement

Before submitting this PR please ensure that you have read the following sections and then completed the template below:
Thank you for your contribution to the Microsoft Sentinel Github repo.

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures, there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

After the submission has been made, please look at the Validation Checks:

Check that the validations are passing and address any issues that are present. Let us know if you have tried fixing and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

PR Template

Description for the PR:
(Enter the description below)
Testing Completed:
Yes/ No :

Don't merge this PR for now. This is shared with customer for review. Waiting for an update from them.

Closing this PR as we haven't heard back from customer for a long time.


BAAS-23701: reduce allocations in objectproto_toString by avoiding Sprintf usage

goja (BAAS-23701)$ go test -bench=^BenchmarkObjectproto -run=^$ -benchmem
goos: darwin
goarch: arm64
pkg: github.com/dop251/goja
BenchmarkObjectproto_toString-10                19064584                62.89 ns/op           40 B/op          2 allocs/op
BenchmarkObjectprotoImproved_toString-10        81820740                14.77 ns/op            0 B/op          0 allocs/op
PASS

func BenchmarkObjectproto_toString(b *testing.B) {
	clsName := "someclass"
	for i := 0; i < b.N; i++ {
		_ = func(s string) string {
			return fmt.Sprintf("[object %s]", clsName)
		}(clsName)
	}
}

func BenchmarkObjectprotoImproved_toString(b *testing.B) {
	clsName := "someclass"
	for i := 0; i < b.N; i++ {
		_ = func(s string) string {
			return "[object " + clsName + "]"
		}(clsName)
	}
}

(pprof) peek Sprintf 10
Showing nodes accounting for<PHONE_NUMBER>50, 100% of<PHONE_NUMBER>50 total
----------------------------------------------------------+-------------
      flat  flat%   sum%        cum   cum%   calls calls% + context
----------------------------------------------------------+-------------
                                          40759875 16.63% |   github.com/10gen/baas/app/utils/pool.GetAppDependencies
                                          38905710 15.88% |   github.com/dop251/goja.(*Runtime).objectproto_toString
                                          38864897 15.86% |   github.com/10gen/baas/function/execution/vm.buildExports

38905710 /<PHONE_NUMBER>50 = 0.00010573294
Hmm this accounts for a very small sliver of object allocations.. This probably wasn't worth the time, luckily it was super quick.

@arahmanan Thanks for that note on compiler optimization. I knew this was faster but did think it was strange we were seeing 0 allocs.
var result string

func BenchmarkObjectproto_toString(b *testing.B) {
	clsName := "someclass"
	var innerResult string
	for i := 0; i < b.N; i++ {
		innerResult = func(s string) string {
			return fmt.Sprintf("[object %s]", clsName)
		}(clsName)
	}
	result = innerResult
}

func BenchmarkObjectprotoImproved_toString(b *testing.B) {
	clsName := "someclass"
	var innerResult string
	for i := 0; i < b.N; i++ {
		innerResult = func(s string) string {
			return "[object " + clsName + "]"
		}(clsName)
	}
	result = innerResult
}

goja (BAAS-23701)$ go test -bench=^BenchmarkObjectproto -run=^$ -benchmem
goos: darwin
goarch: arm64
pkg: github.com/dop251/goja
BenchmarkObjectproto_toString-10                19023538                63.46 ns/op           40 B/op          2 allocs/op
BenchmarkObjectprotoImproved_toString-10        48382054                24.69 ns/op           24 B/op          1 allocs/op
PASS
ok      github.com/dop251/goja  3.857s

@nickpoindexter this should be related to the 0 allocs you were seeing that we talked briefly about.


Replace calendar nested PageViewController with single level PageView…

…Controller. Removed week view, and llist days in months view.

:+1:


[python] Pre-emptively avoid SciPy 1.15, pending a bugfix

Issue and/or context:
Pre-emptively avoids https://github.com/single-cell-data/TileDB-SOMA/pull/new/kerl/pin-scipy-less-than-1.15. Tracking: https://github.com/single-cell-data/TileDB-SOMA/issues/3444.
Note that scipy 1.15.0 is not released yet, but, its 1.15.0rc1 is out, and our next tiledbsoma release (coincidentally also named 1.15.0) is coming out soon, and we want to pre-emptively protect against any time window in which we could suffer from this bug. Once https://github.com/scverse/anndata/issues/1802 is addressed (be it by a bugfix in SciPy or AnnData), we can remove this pin.
Changes:
Notes for Reviewer:

@bkmartinjr ready for round two!


Yun-Hsia : Workflows:  2Week

Learning Objectives

[ ]  Practice git cmd: https://github.com/mametur/GitWorkShop

Week 2
I Need Help With:
Maybe the time management to review the courses and resources.
What went well?
I try to explain what I've learnt with my teammates, and it allowed me to understand it more clear.
What went less well?
Team work. Everyone in the group has different acknowledge, attitude or hardware, it's a challenge to maintain a good teamwork. But practice makes perfect.
Lessons Learned
I learn how to create issues and project boards. And how are the different labels can be used.

@yunhsiaho
1- why didn't you post the learning objective list at the top of your issue ?
2- add check-in for  week1


Another attempt at upgrading to Babel 7

Upgraded all 'babel-' dependencies and various other dependencies that were still using babel 6.  Then worked through the issues that came up:
Updated eslint to latest version that uses babel 7 and converted .eslintrc(s) from yaml to json.
Use strict now uses double quotes instead of single quotes; see this.
Ava was updated to a newer version that uses babel 7, it’s actually using rc1 since even version 0.25 was still using babel 6.  For the tests this caused a change where the process.cwd() is now wherever the package.json is located when running tests; see this thread. This lead to changing all the tests config paths, removal of the ‘basic require with default webpack name’ test (for now, open to suggestions on how to re-add) and changing a lot of the expected fixture paths.
Expected fixtures are now double spaced to match generated output and paths that are replaced become double quoted, not sure why couldn’t find documentation on that one.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cb486b7). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master      #59   +/-   ##
=========================================
  Coverage          ?   94.38%
=========================================
  Files             ?        1
  Lines             ?       89
  Branches          ?        0
=========================================
  Hits              ?       84
  Misses            ?        5
  Partials          ?        0

Impacted Files
Coverage Δ

src/index.js
94.38% <100%> (ø)

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 cb486b7...ac67bd6. Read the comment docs.

Node.js 4 build is failing, according to this babel 7 has dropped support for Node.js < 6

Closing this pull request because it's not getting any traction, I've also begun work on @babel/plugin-webpack-alias as a babel7 replacement over at https://github.com/shortminds/babel-plugin-webpack-alias if anyone is interested.


Multiple WireGuard connections through the same wstunnel client<>server

Describe the bug
This issue is a bit tricky. It is when multiple WireGuard clients connect through the same tunnel to the WireGuard server.
What is working:
2* WireGuard clients -> 2* wstunnel clients -> wstunnel server -> WireGuard server
Basically, each WG client connects to its own wstunnel client
What is not working:
2* WireGuard clients -> 1* wstunnel clients -> wstunnel server -> WireGuard server
When the 2 WG clients connect to the same wstunnel client, initially both WG clients will handshake. Then within seconds, both WG connections will have very high packet dropping rates, almost unusable connections. Ping packet loss is about 75%+
There is hardly any useful information in the trace (with RUST_LOG=trace).
No error is reported at wstunnel client or server.

I understand this is a tricky issue that might indicate some thread/connection interferences related to UDP connections. If other information can be helpful, I am happy to provide.
Also note as per 'What is working' section, there is a known workaround.

Thanks for reporting it :)
Would you mind trying release https://github.com/erebe/wstunnel/releases/tag/v7.9.0, it should fix your issue !
Let me know if it is the case

Tested and confirmed working. Also tested throughput and can confirm there is no side-effect.
Man I can't imagine how you are able to pinpoint and fix the issue with so limited information! Big thank you!

Sorry have to re-open this issue.
I have tested the original scenario:
2* WireGuard clients -> 1* wstunnel clients -> wstunnel server -> WireGuard server
It worked for a while. Then after about 20 mins and total of 300MB data transferred in both connection, the wstunnel client is frozen.
Frozen as there is no message of 'Opening TCP connection to "server name"' when WG is manually reactivated.
wstunnel server seems to be intact, as I then immediately launched a new instance of wstunnel client and again, both WG connections worked at the beginning then froze after a while.
Checked memory footprint of the executable, dead processes are at 4M mark, and the active at 6M mark. Again, no error messages.

Ok, this one is weird, I don't see any obvious reason why it behaves like this.
On the client side are you correctly disabling the timeout  -L udp://...?timeout_sec=0 ?

Ok, this one is weird, I don't see any obvious reason why it behaves like this. On the client side are you correctly disabling the timeout -L udp://...?timeout_sec=0 ?

Thanks for the reply. Let me try this.
Also after many tries, it seems to me that it is more likely to happen when both clients open a good number of connections, like opening and loading several webpages at the same time. This doesn't make sense to me though. As far as I am aware, WG doesn't pass on multi-threading into many UDP connections. Plus it is a bit random, sometimes takes long time and still not frozen. Sometimes very quickly.
Anyway, will do a bit of trying with your recommendation. Let's see.
Is there any past issue for me to read any background of 'disabling the timeout -L udp://...?timeout_sec=0'?
And as usual, thank you very much!

Not really any past issue.
It is just that as UDP is connection less, there is no way to know if the udp peer has stopped sending data or not.
So to avoid leaking connection, wstunnel has a default timeout of 30sec, after which it forces close the connection/tunnel.
In the case of VPN traffic, the peer is never going to stop sending traffic, so disabling this force timeout is a must-do, because else you spend destroying and re-creating a tunnel for the same UDP flow, which will cause some havoc server side.
(I have added the mention in the Wireguard with wstunnel section)

Thanks for the feedback. Again after many tries, it  works slightly better with "-L udp://...?timeout_sec=0". The symptom has changed and I am now able to consistently reproduce it.
The environment:
System A: WG server (Docker in Linux)
System B: wstunnel server (Docker in CloudFoundry)
System C: wstunnel client + WG ClientA  (Windows 11)
System D: WG ClientB connects to wstunnel client on System C (Android WG client)
Put both WG clients to stream video (at about 1MB/s). WG ClientA works flawlessly whereas WG ClientB will soon get frozen, in about a minute). However it can be recovered by reconnecting WB but, still will freeze in a minute. It is always ClientB, regardless which client connects first.
I have fallen back to version 6 to test and can confirm there is not a same issue in the old version.
As I am not able to capture any error message again, at wstunnel server or wstunnel client, nor WG client, I would imagine it will be a bit difficult to trace. I can provide you a testing instance including System A&B if helpful? In such case, let me know and let's try to find a secure way to drop you the config files.

I haven't re-done the whole setup, but I tested with iperf3, and found that the client can miss some packet because of the host dropping them due to the receive buffer being too low.
Would you mind trying the new release https://github.com/erebe/wstunnel/releases/tag/v7.9.1 and let me know if it helps ?

Will do the testing and feedback later. Again thanks a lot again.

Well... This might sound weird but after I updated both server and client to 7.9.1, WG client stopped working completely. I can see 'sent' number growing but 'received' always 0.
Reverted the client back to 7.9.0 and WG connection is restored. I am trying everything else now. Just mentioning this in case it rings any bell.

Can you try only client with 7.9.1 and server with 7.9.0, I will look at it later.

Ok sorry, I left some debug code in the release.
Doing a new 7.9.1 right now, you can re-test in ~10min

Should be good, sorry about that.

No worries!
Good news is the connection is now back. Will do the throughput testing later. Just a thinking in regards to your earlier comment around increased buffer vs. independent buffer, does it mean the former might have a limit on the number of UDP streams supported?
P.S. Looks like you need some coffees. Just sending across a couple more... ;-) Thank you again.

It is not much a limitation around the number of clients, but more a limitation of throughput.
With UDP, the server is only having a single socket to receive the data of all clients. At the difference of TCP where you  have a socket (and associated buffers) per client.
So the rule of the game, is to dequeue messages fast enough before your UDP kernel buffers get full, and it starts dropping packets. Kernel can do that without warning, because there is no guarantee with UDP.
The first implementation (and also the Haskell one) had internal buffer per UDP client. It makes it fast to dequeue the UDP packets from the kernel buffer, but is more CPU involving and make a ceiling to the throughput because it has to copy the data twice.
With the current implementation, there is no more internal buffer, each client is driving the server socket, but it makes it slower to dequeue packet because pushing data into the tunnel take comparatively more wait time, than copying data. So there can be some starving if there are too many clients and one is slow to push data. After that if the kernel buffer gets full, it starts dropping some packets.
There are some ways around that, mainly the trick described in this article https://blog.cloudflare.com/everything-you-ever-wanted-to-know-about-udp-sockets-but-were-afraid-to-ask-part-1/, but it is linux only sadly :x
If you confirm me that your issue is solved, I will try to see if I can't rework the code to alleviate the issue

Hey thanks for fixing it, and taking time to explain what's under the hood! It makes much more sense after I read paragraph 5.
I ran a stress test at the top of my currently available bandwidth for 30mins: 3x 4K video streaming over 2 WG clients, or 10MB/s. It was perfect! Thank you!
I am leaving the issue open so you can see it in the list. But please feel free to close.

FYI only. There are some of this error message, about 1 per hour:
ERROR tunnel{id="018b9390-2883-735d-a728-2ce2698e5810" remote="remote server name"}: wstunnel::tunnel::io: error while reading from websocket rx Reserved bits are not zero
Nothing seems to be broken though.

Awesome :) I will close the issue then
Thank you for having taking the time to report those issues, I would have missed them otherwise !
Also, thank you again for all those coffees !
P.s: for while reading from websocket rx Reserved bits are not zero it should be ok, and re-create the tunnel


Backport GH actions to 5.4

@beikov FYI I rebased your PR, since the 5.4 branch was broken (my bad) and I just pushed a fix.


MacOS Sequoia crash

Just to be sure you known the issue, in Sequoia it starts but then it crashes as soon as you click on Tasks or other menu items.
Sequoia beta 4, MacBook Air M1

Thx for letting me know :-) I have not yet installed Sequoia but will do so in some weeks.

Installing Sequoia on a virtual machine now, will test compiling and running RsyncUI soon on Sequoia.

Wow, that’s a quick reaction! Let me known if I can help anyway
Marco
Il 24 lug 2024, 10:05:37, Thomas Evensen @.***> ha
scritto:

Installing Sequoia on a virtual machine now, will test compiling and
running RsyncUI soon on Sequoia.
—
Reply to this email directly, view it on GitHub
https://github.com/rsyncOSX/RsyncUI/issues/23#issuecomment-2247062944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEYVNUVPM257G3QG4M4XZK3ZN5G4DAVCNFSM6AAAAABLLUKDU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBXGA3DEOJUGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

New dmg is compiled on the latest Xcode 16 beta4, the dmg is signed and the app is notarized. The download link is: https://www.dropbox.com/scl/fi/u5t8441w1ymoc9by4pxfu/RsyncUI.2.0.0.dmg?rlkey=aejby4bv14edma1uytar9eejo&st=k7lyjn59&dl=0
and shasum -a 256 = 26ecd6fde998fec0169e2a86a48ec0cc0e1a80f9d648d331e05a4a5a38638e52
Please let me know when downloaded, I will delete the dmg. There are new features export/import from File menu.. Not yet completed (aka work in progress)...

Will try later, now we are at the beach. I will keep you posted Thks!!  I find your program really usefulMarcoInviato da iPhoneIl giorno 24 lug 2024, alle ore 11:41, Thomas Evensen @.***> ha scritto:
New dmg is compiled on the latest Xcode 16 beta4, the dmg is signed and the app is notarized. The download link is: https://www.dropbox.com/scl/fi/u5t8441w1ymoc9by4pxfu/RsyncUI.2.0.0.dmg?rlkey=aejby4bv14edma1uytar9eejo&st=k7lyjn59&dl=0
and shasum -a 256 = 26ecd6fde998fec0169e2a86a48ec0cc0e1a80f9d648d331e05a4a5a38638e52
Please let me know when downloaded, I will delete the dmg. There are new features export/import from File menu.. Not yet completed (aka work in progress)...
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Excellent, I am closing the issue. You can of course still use this version, there will be some development for export and import only. I am using this version myself on a daily basis.

Thks, I find your program really useful, on a daily basis
M.
Il 24 lug 2024, 23:33:47, Thomas Evensen @.***> ha
scritto:

Excellent, I am closing the issue. You can of course still use this
version, there will be some development for export and import only. I am
using this version myself on a daily basis.
—
Reply to this email directly, view it on GitHub
https://github.com/rsyncOSX/RsyncUI/issues/23#issuecomment-2248850829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEYVNUWMY4NTJMB4I7YLBSTZOAFSXAVCNFSM6AAAAABLLUKDU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBYHA2TAOBSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


test is not exiting unless 'done' parameter is passed to afterAll() without being explicitly called

🐛 Bug Report
I've been trying for the last 2 days to figure out why the test is not existing despite following all the steps with no luck until by accident I left the 'done' parameter in afterAll() without actually explicitly calling it from inside. like this: afterAll((done)=>{ mongoose.disconnect() }). This made it exit and work properly but when I explicitly call done from the function using promises (async/await or then) or as a callback to mongoose.disconnect() such as afterAll(async (done)=> { await mongoose.disconnect(); done()}) it does not exit and keeps hanging.  when using --detectOpenHandles it tells me that app.listen(3001) is the potential open handler which I explicitly closed. I can't figure if this is a problem in my code or a bug.  The way I was able to fix the problem above by passing done param without using it seemed more of a hack (or bug) than a real solution. is done supposed to do anything if it is passed only(without being called)?
To Reproduce
`const request = require('supertest');
 const mongoose = require('mongoose');
 mongoose.connect('mongodb://localhost/debugging');
 const app = require('../app');
 let server = '';

describe('GET /products/:name', ()=>{

 beforeEach(()=> { server = app.listen(3001)})
 afterEach(()=> { server.close() })
 // afterEach((done)=> { server.close(done) })  <-- this does not work also
 afterAll((done) => {
    mongoose.disconnect();
 //  done();                    <--- does not work even when using await on mongoose
 // mongoose.disconnect(done);  <--- does not work also
 })

 it('should return a 404 status', async ()=>{
     const res = await request(server).get('/name');
     expect(res.status).toBe(404);
 })

})

when I explicitly call done() in afterAll I get this:
Jest has detected the following 1 open handle potentially keeping Jest from exiting:

  ●  TCPSERVERWRAP

       8 |
       9 |
    > 10 |     beforeEach(()=> { server = app.listen(3001)})
         |                                    ^
      11 |     afterEach(()=> { server.close() })
      12 |     // afterEach((done)=> { server.close(done) })  <-- this does not work also
      13 |     afterAll((done) => {

      at Function.listen (node_modules/express/lib/application.js:618:24)
      at Object.listen (tests/routes.test.js:10:36)

Expected behavior
To be able to exit process when calling done after disconnecting from server and db.
Link to repl or repo (highly encouraged)
to reproduce behaivor here is a sample rep here. note that the sample I provide is a working. Just try to call done using any method and the test will hang. also try removing the done param from afterAll() and the test will also hang.

Hey @tito300 this is happening because mongoose.disconnect() is a promise and you need to wait until the disconnect promise is resolved. It looks like you tried this a few ways but none of them were correct.
What did work (providing the done arguments without calling) only works because it's basically forcing Jest to wait for a done that is never called, so the open handle detection never runs
Here's why a few of your attempts were wrong:
This will call done before the promise resolves (i.e. before it's done)
afterAll((done) => {
    mongoose.disconnect(); // 1 - start the disconnect
    done();  // 2 - call done
    // 3 - sometime later the disconnect actually happens
})

This will pass done to disconnect, which would work if disconnect accepted a callback (it does not)
afterAll((done) => {
    mongoose.disconnect(done);
})

Two ways that should work are:
// pass done to .then
afterAll(done => {
    mongoose.disconnect().then(done);
})

// return the promise
afterAll(() => {
    return mongoose.disconnect();
})

Give these a try and let me know if they work!

Same issue here. The async handlings proposed by @rickhanlonii did not work :(


Fix text alignment (e.g. in annotations)

The text alignment setting for text with linebreaks (e.g. in annotations) was not set correctly.
An simple example would be:
ax.annotate('line1\n line2', xy=(1, 2), horizontalalignment='right')
The default horizontal alignment is 'left'.

Current coverage is 80.26%

Merging #92 into master will not affect coverage as of fed1736

@@            master     #92   diff @@
======================================
  Files           10      10
  Stmts          897     897
  Branches         0       0
  Methods          0       0
======================================
  Hit            720     720
  Partial          0       0
  Missed         177     177

Review entire Coverage Diff as of fed1736

Powered by Codecov. Updated on successful CI builds.


Functions to list providers and other things

I've been experimenting with deno_kv_oauth to provide a sign in for my personal site, I've created a handful of functions to discover available providers, and to dynamically configure providers based on the environment vars that have been set...
https://github.com/jollytoad/home/blob/main/lib/oauth2_clients.ts

getOAuth2ClientNames - Get a list of all supported OAuth2 client names
hasOAuth2ClientEnvVars - Detect whether environment vars have been set for a given OAuth2 client
getOAuth2ClientFn - Get the create client function for an OAuth2 client given it's name
getOAuth2ClientScope - Get the OAuth2 scope for a provider from environment vars

Would you be interested in having some or all of these fns within deno_kv_auth itself?
I'd be to raise a PR for this if so, and include some tests.
For an example of usage, see... https://github.com/jollytoad/home/blob/main/components/UserWidget.tsx
This JSX component lists sign in links for providers where the env vars have been set, using getOAuth2ClientNames & hasOAuth2ClientEnvVars, the sign in link includes the provider name in the path, and https://github.com/jollytoad/home/blob/main/routes/auth/_lib/oauth2_client.ts has a function used by the signin/callback handlers to extract the provider name and use getOAuth2ClientFn & getOAuth2ClientScope. (All of this stuff is specific to my site though, so I wouldn't suggest incorporating these, I'm referencing them simply as an example of usage)

Here is an example of the UserWidget output btw...

The only changes I have to make to switch providers is to set the appropriate env vars, which is quite handy when working with Deno Deploy (my live site has no providers configured atm, it may do soon though).

I've always wanted to ensure that this module doesn't fall into the trap of doing too much and that its API surface is kept as small as possible. I'd prefer not to have these functions in the module for these reasons. However, you can still create your own module, which could be linked to in the README. I can see these functions having value in some edge cases.

Yeah, I totally understand. I'll let you know if I break them out into a dedicated module. I thought I should discuss the possibility first.


feat: project initial setup

What/Why
Added initial setup to the project and connectors between Vercel and BigCommerce.

Verified the latest works for the core products and search functionality:

The add to cart functionality is error'ing out because of cartId and variantId missing. That can be resolved in the next PR.

Merged as we need this prototype to be handed over to Vercel. @bc-alexsaiannyi will work on comments/suggestions in next PR.


Make array attribute space-separated for better stlyling specificity

I went through the CSS-Tricks page about attributes again and I found a little tweak.
This now allows the use of [data-value~="foo"] rather than [data-value*="foo"], so you don't get partial matches. For example, [data-value*="foo"] also accepts the value "food" because it contains "foo". [data-value~="foo"] would not accept "food".
This is backwards compatible; it still allows the use of [data-value*="foo"], but the user can use [data-value~="foo"] instead as it is more specific.

thanks @Jayby18


[Feature] Add ability to run test scenarios in the Jenkins runner environment

Is your feature request related to a problem? Please describe
I would like to have the ability to run a single test, test suite or gradle test task in the Jenkins runner multiple times to help reproduce and minimize flaky tests. When looking at various flaky test issues, I run the test locally multiple times and the tests pass every time making it harder to simulate the conditions that led to failure on a CI check.
Describe the solution you'd like
When looking at flaky test issues, it could be helpful to run the tests on the Jenkins runner (or any Github runner) to simulate similar conditions that led to failure. I'd like the ability to not run the full gradle check, but rather a single test, test suite or gradle test task (like :server:test) without running the full gradle check.
Related component
Build
Describe alternatives you've considered
Continue to try to reproduce locally or on remote instances of different os types.
Additional context
The security plugin added a RepeatRule to help with local testing for diagnosing flaky tests. It would be helpful to have something similar and have it run on the Jenkins instances or a Github Runner.

@opensearch-project/admin Can you transfer this to opensearch-build?  This seems like a feature of how the jenkins job can be run or scheduled


Add linkouts to CellGuide for curated/validated defs

For all terms in https://docs.google.com/spreadsheets/d/13NJsDH0fBZEFYuGZYkJ2QUah9WV9731zliiW-9uOTuA/edit#gid=2059655613
xrefs → CL for all reviewed definitions
Bioregistry needs a cell Guide entry - base https://cellxgene.cziscience.com/cellguide/
or
https://cellxgene.cziscience.com/cellguide/CL_
CL terms get xrefs: cellguide:CL_0002623  OR cellguide:0002623
Once released this should => links to CellGuide on OLS4

@anitacaron  - just checking this is on agenda for tech team.  Thanks!

@anitacaron - Given discussion on bioregistry and linked OLS tickets, I think we should change plan and add URLs directly to term using seeAlso as predicate.  I would also like to annotate this axiom with some link text so that consumers of the ontology don't have to display raw URLs.  Not sure of best predicate for that. Maybe rdfs:label?

@hkir-dev  - can you add CellGuide URLs as part of the process for adding subset tags for CxG Census?  We will also need CellGuide URLs for ancestor classes in CL for all terms in census.  These should be rdfs:SeeAlso AP + typed IRI.  Could you also investigate whether there is a standard way add hyperlink text to links in seeAlso?

Closing ticket as we are now we are adding xrefs with https - as part of this template https://github.com/obophenotype/cell-ontology/blob/master/src/patterns/dosdp-patterns/ExtendedDescription.yaml#L35
(Maybe better to switch to seeAlso? )


fix: updated rack middleware position to zero

Description
TL;DR;
The current insertion point is after some important middlewares
like the ActionDispatch::Static.
That is making us missing some traces,
so I changed the insert to the first position
Deep down
We are starting to test the OTel in production to remove Datadog. After the deployment, we saw a big difference in requests:
Datadog version:

OTel version:

After some research, I found the problem in the current position of the  TracerMiddleware. The stack looks like that:
[0] = {Class} Datadog::Contrib::Rack::TraceMiddleware
[1] = {Class} Rack::Cors
[2] = {Class} SecureHeaders::Middleware
[3] = {Class} Rack::Sendfile
[4] = {Class} ActionDispatch::Static
[5] = {Class} ActionDispatch::Executor
[6] = {ActiveSupport::Cache::Strategy::LocalCache::Middleware} #<ActiveSupport::Cache::Strategy::LocalCache::Middleware:0x0000564147e12f60>
[7] = {Class} Rack::Runtime
[8] = {Class} Rack::MethodOverride
[9] = {Class} ActionDispatch::RequestId
[10] = {Class} OpenTelemetry::Instrumentation::Rack::Middlewares::TracerMiddleware
[11] = {Class} RequestStore::Middleware
[12] = {Class} ActionDispatch::RemoteIp
[13] = {Class} Sprockets::Rails::QuietAssets
[14] = {Class} Rails::Rack::Logger
[15] = {Class} ActionDispatch::ShowExceptions
[16] = {Class} Datadog::Contrib::Rails::ExceptionMiddleware
[17] = {Class} ActionDispatch::DebugExceptions
[18] = {Class} Rollbar::Middleware::Rails::RollbarMiddleware
[19] = {Class} ActionDispatch::Reloader
[20] = {Class} ActionDispatch::Callbacks
[21] = {Class} ActionDispatch::Cookies
[22] = {Class} RedisSessionStore
[23] = {Class} ActionDispatch::Flash
[24] = {Class} ActionDispatch::ContentSecurityPolicy::Middleware
[25] = {Class} Rack::Head
[26] = {Class} Rack::ConditionalGet
[27] = {Class} Rack::ETag
[28] = {Class} Rack::TempfileReaper
[29] = {Class} OmniAuth::Builder

The ActionDispatch::Static[position 4] comes before the TracerMiddleware [position 10], so every time a static resource is required, we lost the tracing for that.
That is the same approach used in ddtrace library and based on that comment believe to change the order is not a big problem and could bring more benefits

I also think this is fine. If we think it is useful, we can add another middleware to add the rails.request_id attribute to the Rack span, or we can add that functionality to the ActionController::Metal patch.


autogluon.mxnet is missing dependency on autogluon.extra

autogluon.mxnet uses autogluon.extra:
https://github.com/awslabs/autogluon/blob/b403079bcc17171d8b5366096f7d7f9e7eb1b99a/mxnet/src/autogluon/mxnet/task/nets.py#L8
but it doesn't list it as a dependency:
https://github.com/awslabs/autogluon/blob/b403079bcc17171d8b5366096f7d7f9e7eb1b99a/mxnet/setup.py#L19-L28
This breaks when doing a python3 -m pip install autogluon.mxnet and then importing autogluon.mxnet with the following error message:
ModuleNotFoundError: No module named 'autogluon.extra'

This should probably be listed either as a required or optional dependency setup.py.

Thanks for spotting this! #1032 fixes the issue and will be available in our next release.


Feat/markdown link check

Description
Adds new feature that checks all links in the markdown files
Will only check the files original English files on 'src/pages/en' (not the translations).
Will run on all pull request and pushes to master branch.

Detected broken links detected here are fixed on the PR 1341


Server not starting in Win10 Kali

Would love a fix to that; ncat is installed, nothing works

Can you please describe the issue in more detail, maybe attaching logs or console output? Currently, I don't have access to Win10, so troubleshooting without logs would be hard.

I do chmod +x * and run ./http.sh but ncat doesnt run and therefore post 80 and 443 is not listening, you could make a virtualmachine and test it yourself.

I'm on a metered internet connection, so downloading a Win10 image, and then running Kali under WSL would literally cost me money, not to mention the time.
I can't help you if you don't tell me exactly what happens. Why doesn't Ncat run? Do you have correct permissions on all files? Can you run ncat from your current user, and if so, what happens when you run ncat -v -l -p 1337 -k -c ./src/server.sh?

I will test more later, I'll keep you updated

Maybe I forgot to use sudo, would this trigger the bug?

I confirm this is just because I wasn't using sudo


[python] Improve guidance on non-ingestible NaN data in categorical-int obs/var columns

We cannot ingest these, as a stack trace makes quite clear.
However, we can offer improved guidance on which columns are problematic.

Codecov Report
Patch coverage: 100.00% and project coverage change: +26.58% :tada:

Comparison is base (3d4a5ad) 64.82% compared to head (d3feba5) 91.41%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1568       +/-   ##
===========================================
+ Coverage   64.82%   91.41%   +26.58%
===========================================
  Files         105       31       -74
  Lines        8780     2668     -6112
===========================================
- Hits         5692     2439     -3253
+ Misses       3088      229     -2859

Flag
Coverage Δ

python
91.41% <100.00%> (+0.52%)
:arrow_up:

r
?

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

Files Changed
Coverage Δ

apis/python/src/tiledbsoma/io/ingest.py
84.72% <ø> (+1.09%)
:arrow_up:

apis/python/src/tiledbsoma/io/conversions.py
88.88% <100.00%> (+2.77%)
:arrow_up:

... and 84 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement ToValue for Value again

Please revert https://github.com/gtk-rs/glib/commit/390ca3a2. Conversion traits in Rust are usually expected to be reflexive. Things can get weird if you don't: For example, the GtkListStoreExtManual::set method accepts a &dyn ToValue. This makes sense because it allows the user to provide multiple values of different types in an array without requiring conversion, which is convenient.
But if I have some Values that I got from somewhere and ToValue is not reflexive, then I suddenly cannot use that method anymore. As a workaround there is set_value, but it can only deal with one column at a time.
CC @sophie-h

The correct way to fix this is for GtkListStoreExtManual  to provide a set_value variant that would take a glib::Value instead of a &dyn ToValue.


Contact link not working (404 Not Found)

The envelope image link is broken at the bottom of the main page (just after the Github image). The link points to http://www.qgis.org/en/site/getinvolved/about.html

Mmm, thanks for noting, will have a look at it

Duplicate of #377 and fixed in #378


Bug Report: initialRows (or buffersize) doesn't get updated on windows size changes

Environment
Windows build number: Microsoft Windows [Versión 10.0.18362.295]
Windows Terminal version: 0.4.2382.0

Steps to reproduce

Set the initialRows value to something like 30 rows (a value that makes the openned window has less rows than the maximized window should have).
Maximize the window.
Run some command with a lot of output (like ps, lsor dir).

Expected behavior

The buffersize setted with initialRows gets updated after window size changes.
Behavior on PowerShell.

Actual behavior

After maximizing the window and running ps the buffer size setted with the initialRows value doesn't get updated after the window size changes.

The problem gets solved if I open a new tab and run the command again.

@Vreyesm Out of curiosity, does this repro if you execute Remove-Module psreadline before maximizing? Does it repro if you switch the default profile to cmd?

@zadjii-msft Executing Remove-Module PSReadline doesn't make any changes on the behavior, but if I change the default profile to cmd (or  wsl running ubuntu 18.04) as you suggest, the problem gets solved (following the steps described above).

Hey so while burning down bugs, it looks like this is the same bug as #2815. This one was filed first, but the other one's got a bit more details on the specific root cause of the problem, so I'm going to mark that one as the parent bug. Regardless, I've got a fix ready for these that should be in PR later today. Thanks!
/dup #2815

Any idea when this buffer resize issue will have a fix? If I run a normal cmd the screen adjusts, if I run in Windows Terminal I get [process exited with code<PHONE_NUMBER>] {See Image-1 }. If I manually expand the window, and rerun, everything works fine {See Image-2 }.
Image-1

Image-2

Oh, that's probably #4062 or #5094, or some combo of the two

@zadjii-msft - I believe it is thrown due to SetCursorPosition. Please See Below:
Unhandled Exception: System.AggregateException: One or more errors occurred. ---> System.AggregateException: One or more errors occurred. ---> System.ArgumentOutOfRangeException: The value must be greater than or equal to zero and less than the console's buffer size in that dimension.
Parameter name: top
Actual value was 30.
at System.Console.SetCursorPosition(Int32 left, Int32 top)
Any thoughts? thx

@zadjii-msft - I think I found the issue. The old school conhost.exe apparently auto resizes buffer rows without any explicit setting. If I set the console buffer height explicitly. No error. It works as intended. perfectly. Please let anyone know that had similar issue.
Thanks again msft.


Scope of application of the BARTScore

I wonder if BARTScore can evaluate text generated by BART, such as summarization and machine translation.

Yes, it can.


Post request for logout

Logout API is weird: a function supposed to return a string which will be queried with GET. In authorization key flow with PKCE revocation usually is a POST request. Here is an example:
https://docs.pingidentity.com/bundle/pingfederate-93/page/wmo1564003025254.html
https://tools.ietf.org/html/rfc7009#section-2.1

The intent was to support two operations: to revoke the access token and to clear the browser cookies. Revoking the access token has to be done by the Logout method itself using Web.Contents, and this can of course be a POST if necessary. The string that's returned (if any) was supposed to be passed to a browser instance in the Desktop apps, because that would be the only way to clear cookies. Both of these obviously require support from the underlying service in order to be used.


GitHub user scraper

Description
Python script to fetch trending users from github
Fixes #947
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] My documentation follows Template for README.md
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

@kaustubhgupta Do I need to make tables for all possible unique combinations?

@Ayushjain2205 no, just make tables for different languages. The weekly and monthly data can be stored in that specific language table.

OK got it...working on the changes will be done by tomorrow

@kaustubhgupta  I have solved the Bug without adding tables for individual languages, because adding variable table name was giving me some issues.
I have made the unique field non unique and while fetching I am only fetching distinct fields.
I have made the Date range and language as a table column and it gets printed with the records in the UI so the user can easily see what they scraped.