move drop reflector config to relabel_configs

Towards: https://github.com/giantswarm/giantswarm/issues/5520
Attempt at fixing Prometheus OOM issues.
There is a previousPR which attempted to fix this by not ignoring reflector* metrics on scrape.
But it seems that metric_relabel_configs is applied after scrape happened and before storing into database, whereas relabel_config is applied before scrape. So this PR moves the "drop reflector*" to relabel_config.
After testing, reflector* metrics are still present in the database. So this is not correct and need to be adjusted.

After testing locally with a very simple prometheus config, it turned out metric_relabel_configs is the right place to drop metrics.
I ran prometheus using the following command
docker run --rm -it -p <IP_ADDRESS>:9090:9090 -v $PWD/prometheus.yml:/etc/prometheus/prometheus.yml prom/prometheus

and two different configs
global:
  scrape_interval:     15s
  evaluation_interval: 15s
scrape_configs:
  - job_name: 'prometheus'
    static_configs:
    - targets: ['localhost:9090']
8c8
<     metric_relabel_configs:
---
>     relabel_configs:
    - source_labels: [ __name__ ]
      regex: 'prometheus_http_request_duration_seconds_bucket'
      action: drop

using metric_relabel_configs returns 0 results
$ curl -Ss 'http://localhost:9090/api/v1/query?query=prometheus_http_request_duration_seconds_bucket' |jq .
{
  "status": "success",
  "data": {
    "resultType": "vector",
    "result": []
  }
}

using relabel_configs returns 90 results
$ curl -Ss 'http://localhost:9090/api/v1/query?query=prometheus_http_request_duration_seconds_bucket' |jq .
{
  "status": "success",
  "data": {
    "resultType": "vector",
    "result": [
      {
        "metric": {
          "__name__": "prometheus_http_request_duration_seconds_bucket",
          "handler": "/api/v1/query",
          "instance": "localhost:9090",
          "job": "prometheus",
          "le": "+Inf"
        },
        "value": [
<PHONE_NUMBER>.319,
          "5"
        ]
      },
      < 89 entries removed >
    ]
  }
}


Header should be in caps in all translations

Maybe just Russian, Finnish and German shouldn't be capitalized? @alexandrashelenkova thoughts please?

@sasha-abramovich why these languages?) And we have other header in caps too, so it should be consistent everywhere.

@alexandrashelenkova just because their words are very long? remember what the Finnish translator suggested? that was the trigger for my rationale

Let's keep it all caps please.

checked


[CWS] make sure we only deserialize and propagate profiles once

What does this PR do?
This PR ensures profiles are propagated back to the manager if they match the current selectors. Currently if multiple selectors matches we end up deserializing and propagating multiple times the same profile. This PR ensures we do it only once.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

@paulcacheux is this PR still required ?


Add 'now' flag to job entries

Proposed by @ryansanford as a preemptive production support feature.
Add a boolean now flag to job objects.
Toggling this flag will cause the job to run as soon as possible.
Change the job modification (post/put) route to only allow superusers to manipulate the field.
Change the next route to make two queries - first, check for any jobs with a now flag, second, the normal query that exists today. Critically, the now query should sort by the modified timestamp in the opposite order the normal query uses - that is, return a job that has most recently been modified and has a now flag. This contrasts with the normal query which is FIFO.

Unassigning this since I don't anticipate getting to it anytime soon.


Make achievements activity more discoverable

Achievements activity  is currently accessed by clicking the icon.  And that icon might not seem clickable  to some general user
Why should we consider this?

Presently there is no mention of achievements activity or any leaderboard in the app's tutorial
Currently the icon is displayed over another image in the navigation drawer.
To most general users the icon might not be percieved as a clickable element or it may be considered as a UI element.

What can we do?

Since the learderboard is an activity. We can add it like another option in the navigation drawer
Or if this icon looks significantly impressive to users then we can add the same icon to the toolbar

More suggestions on this opinion are highly appreciated :)
Would you like to work on the issue?
Yes

Hi i would like to work on this issue.

Can't we add the icon in the appbar left of the notiification icon.
Please consider else i will start working on the level number idea.
Thanks

Hey, If no-one is working on this Issue or shall I take this up?

@Tanvi-Goyal That's an idea we can keep in mind, but I feel that near the username makes the user really feel that their achievements are associated to them. Did you get a chance to start caching the level or anything towards completing this issue? If yes please commit and push, even if it is not complete, thanks :-)

@nicolas-raoul I would like to work on this issue, could you please tell me what are we finally gonna implement for this issue?

Actually due to a UI revamp, the activity is now much more discoverable.
I suggest either:

closing this issue,
or changing the menu item from TheUser to TheUser's achievements.

@nicolas-raoul I tried to implement this and successfully stored the user level and displayed it in the menu item next to the username. But this works only after the user visits the achievements fragment once, since the data has to be fetched first; only then can I store it. So, when a user freshly installs and opens the app, the menu will display only their username but will show their level after visiting the achievements fragment once. Either we can go with this approach or close the issue since fetching data beforehand, just for the sake of showing level in the menu, doesn't seem feasible.

Yes your screenshot shows the right place.
Tester051005 (Level 1) sounds good!
If the level is unknown yet, how about displaying Tester051005 (see your achievements)?

If the level is unknown yet, how about displaying Tester051005 (see your achievements)?

Sounds great! It will make the user curious and tempt them to it, nice. I'll do as required


DispatcherHelper constructor throws expected exception, creates developer confusion

Describe the bug
When used in a Win32 (non-UWP) app, DispatcherHelper's constructor raises and immediately catches an exception. This breaks into the debugger as a first-chance exception, causing developer confusion.
Details
The DispatcherHelper constructor uses a try/catch to anticipate the exception thrown by CoreApplication::GetCurrentView() in the case where the host app is not a UWP app. For example, the host app could be a Win32 app using XAML islands.
Although the exception is handled, the fact that an exception is thrown at all causes Visual Studio to break into the debugger, and the developer must Continue past it. The developer could avoid the problem by configuring Visual Studio to ignore first-chance exceptions, but this forces the developer out of their preferred workflow to accommodate a platform issue.
In particular, the developer may not even realize that they are using XAML islands. As an example, a plain Win32 app that calls CredUIPromptForWindowsCredentials encounters this unexpected exception because CredUI uses XAML to present its credentials dialog. The Win32 developer is unaware of this implementation detail. They see an exception and fear that they did something wrong.
In this specific case, we sent the developer on a wild goose chase trying to debug the exception, unaware that the exception is not something to be concerned about. This wastes the developer's time.
Even in the case where the developer is consciously using XAML islands, the spurious exception will lead the developer down the wrong path when trying to debug why their XAML islands are not working.
Suggest removing the try/catch and switching to
winrt::CoreApplicationView TryGetCoreApplicationCurrentView()
{
    winrt::CoreApplicationView view{ nullptr };
    winrt::get_activation_factory<winrt::CoreApplication, ABI::ICoreApplication>()
        ->GetCurrentView(put_abi(view));
    return view;
}

        if (!dispatcherQueue)
        {
            if (dependencyObject)
            {
                coreDispatcher = dependencyObject.Dispatcher();
            }
            else if (auto currentView = TryGetCoreApplicationCurrentView())
            {
                coreDispatcher = currentView.Dispatcher();
            }
        }

Steps to reproduce the bug

Write a Win32 app.
From the Win32 app, call CredUIPromptForWindowsCredentials().

Expected behavior
3. No spurious exception.
Version Info
This problem has existed since the DispatcherHelper constructor was introduced in 1809.

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)
Yes

October 2018 Update (17763)
Yes

April 2018 Update (17134)
No

Fall Creators Update (16299)
No

Creators Update (15063)
No

Device form factor
Saw the problem?

Desktop
Yes

Mobile
N/A

Xbox
N/A

Surface Hub
N/A

IoT
N/A

(N/A because issue requires an OS that supports XAML islands.)
Additional context

This has been merged to master. Currently working on porting it to the OS.

Checked in to the OS now.

Hi,
thank you for fixing this bug. It actually took me half an hour to realize the exception is not causing my crash.
But I am having a crash succeeding this exception which occured in code that worked for 2 years after upgrading from 1803 to 1903 (problem persists in 1909) in our Visual C++ Application (compiled with vc12).
It almost always crashes after entering (valid or invalid) credentials in CredUIPromptForWindowsCredentialsW. At least if you try it three times.
The first of numerours exceptions in SHCore.dll before the application finally crashes is
Exception thrown at 0x00007FFE32D3A839 in SpecsLabProdigy.exe: Microsoft C++ exception: Platform::InvalidCastException ^ at memory location 0x00007FF423A6BF70. occurred
Final crash message from windows event log:
Faulting module name: KERNELBASE.dll, version: 10.0.18362.449, time stamp: 0x2bdaa583
Exception code: 0xe06d7363
Any insight would be higly appreciated.
Best Stefan
CallStack_InvalidCastException.txt

@StefanJKern This sounds like a different issue, could you open a new issue to track it? Also please attach a simplified repro project or a crash dump at the point of the call stack (you can save one from visual studio's Debug menu) when you do. Thanks!

@jevansaks Thanks for your feedback. I opened #1662 for this issue.


w2prompt set selection in input

Hi,
Exist possibility to do selection of value in w2prompt ?
I'm using w2prompt from grid 'menuClick' event. Here I'm putting value from grid cell - user can change or submit needful value . It wil be very usefull if value not empty in w2prompt appears as selected text ...
Can this be done ?
Thanks in adavnce ...

Found how to that with jQuery ...


resolveProxy immediately on app ready event causes app to segfault on OS X

This code causes a segmentation fault:
app.once('ready', function() {
   app.resolveProxy('https://google.com', function(proxyURL) {
     console.log('proxyURL is', proxyURL);
   });
});

This code does not:
app.once('ready', function() {
   setTimeout(function() {
     app.resolveProxy('https://google.com', function(proxyURL) {
       console.log('proxyURL is', proxyURL);
     });
   }, 0);
});

This code is also fine:
app.once('ready', function() {
   setImmediate(function() {
     app.resolveProxy('https://google.com', function(proxyURL) {
       console.log('proxyURL is', proxyURL);
     });
   });
});

Here is the barf from the crash dialog: https://gist.github.com/davidbanham/7e4ab9e54f2706554f01
I have confirmed this on both v0.28.3 and v0.28.1

On Windows 7 32 bit it freezes even using a large setTimeout (one second). Is this consistent with your fix @zcbenz ?


DPL-844 Support manual aliquoting before scRNA banking pipeline

User story
As a SeqOps user of the scRNA banking pipeline, I would like there to be an intermediate tube between the initial vacutainer tube and the first plate. This is because I will be manually aliquoting out the SeqOps portion of blood before returning the vacutainer tube to HumGen (they will then use it for banking).
Who are the primary contacts for this story
Lesley, Abby, Katy
Who is the nominated tester for UAT
Lesley, Abby
Acceptance criteria
To be considered successful the solution must allow:

[ ] There is a new tube ("purpose" = "LRC Blood Aliquot"?) that follows on from the "LRC Blood Vac" tube in the Limber pipeline
[ ] "Manual transfer" will be pressed as there is no robot use here (standard Limber behaviour)
[ ] I can print a barcode for the new tube (standard Limber behaviour)
[ ] I can scan these new tubes into the sample arraying screen to make the first plate ("LRC Blood Bank")

Dependencies
This story is blocked by the following dependencies:

https://github.com/sanger/limber/issues/1310

Agreed in working group meeting today that this aliquoting task will sit with SeqOps.


Cocoapods - integer literal overflows when stored into 'Int’

. I'm just googling....

I just add a new tag and it passes the lint!


Improve results will give me Mask not image

Hi,
I tried your example Improve results
but the result is The mask not the image after remove background
how can I get the image itself without background ?

you can refer to this function used in the library functions to mask the image from a mask, you can use it as an extension for your input image and mask it with the output mask
`
 /// mask the input image with our mask to get the final result

func maskImage(withMask maskImage: UIImage) -> UIImage {

    let maskRef = maskImage.cgImage

    let mask = CGImage(
        maskWidth: maskRef!.width,
        height: maskRef!.height,
        bitsPerComponent: maskRef!.bitsPerComponent,
        bitsPerPixel: maskRef!.bitsPerPixel,
        bytesPerRow: maskRef!.bytesPerRow,
        provider: maskRef!.dataProvider!,
        decode: nil,
        shouldInterpolate: false)

    let masked = self.cgImage!.masking(mask!)
    let maskedImage = UIImage(cgImage: masked!)

    // No need to release. Core Foundation objects are automatically memory managed.
    return maskedImage

}`


WIP: extensible checkout FR2

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


Add in the use of iterable datasets when fine tuning

🚀 The feature, motivation and pitch
Some datasets may be too large to fit into memory directly, would be nice to provide the option to create an iterable dataset to be passed into the data loader instead of the current hard coded dataset class.
Alternatives
No response
Additional context
No response

thanks @BaiqingL its a good call, will plan it, at the same time pls let me know if you are interested in having a PR.

@BaiqingL closing the issue for now-but please let us know if you would still be interested in sending a PR! Would love to help and collaborate.
Thanks!


Icon request: Huawei

URL for the brand / site / app
https://www.huawei.com/
Logo resources for the icon
https://upload.wikimedia.org/wikipedia/en/0/04/Huawei_Standard_logo.svg (outdated)
Proof of the Aegis-compatible 2FA

Huawei account is universal across its products, including mobile devices and cloud computing.
Soft token is used by Huawei Cloud: https://support.huaweicloud.com/intl/en-us/usermanual-iam/iam_10_0002.html
https://2fa.directory/int/

Anything else?
Link above is the main brand svg logo. Here is the svg for Huawei Cloud: https://res-static.hc-cdn.cn/cloudbu-site/china/zh-cn/wangxue/header/logo.svg
Confirmation

[X] I have read and followed the issue guidelines & issue form.

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/icons/1_Primary/Huawei.svg

:question: How to add icon(s) to Aegis?
:star: Support this project by giving a star! Just adding like to this comment is also appreciated.


Add other exit command

As a Linux user, I can exit the program by typing quit so that it is similar to most of the exit command

Added quit, bye, /q to the exit command


Selecting training dataset

Did anyone happen to learn the model only on VOC2012 dataset? If yes, how is the model performance learned with the default parameters?
As for me, it scores only 65% mAP for 55k iterations, which I think relatively poor.
Looking forward to discussing this!

I have trained on VOC2007 dataset, and it scores 67%mAP for 15k iterations, the same with Fast R-CNN


Define a logger and log database access

data service
views
n1ql / query service
each time, add execution time

see postgresql logs in rails which are displayed only when rails logger is in debug level

done in https://github.com/Mapotempo/couchbase-orm/pull/19/commits/53b6d153c4100deea58f1ef34e7766da887846ba


Preliminary introduction of RowSet with supporting changes

Background
Initially, I aimed to just close https://github.com/celestiaorg/lazyledger-core/issues/423 with this branch, but due to the high coupling of the code, I added multiple supporting changes and improvements that are now separated from the block propagation logic itself.
Changes

Updates to p2p/ipld API

Decouple providing from saving into new ProvideData func
Move in DAHeader and NamespacedShare with supporting changes
PutBlock -> PutData(more naming improvements should be discussed)
Rely more on ExtendedDataSqaure type and extract DAHeader formation into separate func MakeDataHeader
p2p/ipld is now treated as library

Dependency link change: p2p/ipld -> types ==> types -> p2p/ipld

Rework testing according to changes and make the code there consistent
New public test utilities

Updates to types package

Calculate extended square only once, finally!
Basic implementation of RowSet
Rely on p2p/ipld

Start relying on RowSet updates p2p/ipld API in consensus/state
Contribute to overall code cohesion

TODO
Some of those should be fixed within the PR, for some, we should file issues and discuss.

[ ] Upate docs and commentaries
[ ] Find a place for ABCI test(put overwrite bug) and move it there
[ ] Link all the issues the PR closes
[ ] Remove DAH from Block and its proto
[ ] Extract DAH proto from global Protos place
[ ] Rename p2p/ipld to da or other - TBD
[ ] Name consistency in p2p/ipld. Share, Sample, Leaf, NamespacedShare, Data and other alias are all about the same, but it is really confusing for any newcomer
[ ] Make Block struct self-sufficient to be hashable without RowSet call
[ ] Extract NMT wrapper to p2p/ipld
[ ] Rely on constants in one place

Codecov Report

Merging #427 (9174d8b) into master (8da1644) will decrease coverage by 0.09%.
The diff coverage is 60.57%.

@@            Coverage Diff             @@
##           master     #427      +/-   ##
==========================================
- Coverage   61.76%   61.67%   -0.10%
==========================================
  Files         263      266       +3
  Lines       23003    23074      +71
==========================================
+ Hits        14207    14230      +23
- Misses       7292     7336      +44
- Partials     1504     1508       +4

Impacted Files
Coverage Δ

consensus/types/round_state.go
0.00% <ø> (ø)

ipfs/plugin/nmt.go
30.58% <ø> (ø)

light/client.go
64.34% <0.00%> (ø)

light/provider/http/http.go
40.86% <ø> (ø)

p2p/ipld/sample.go
84.21% <ø> (ø)

p2p/ipld/wrapper/nmt_wrapper.go
74.07% <ø> (ø)

rpc/core/blocks.go
27.90% <0.00%> (ø)

rpc/core/types/responses.go
33.33% <ø> (ø)

types/light.go
72.52% <ø> (ø)

types/shares.go
100.00% <ø> (ø)

... and 34 more

Rebased on master

Here is a general suggestion how to proceed with this: we have good evidence that propagating the data in fixed sized chunks works well and is fast. Why don't we stick to that paradigm and still split the rows into parts of equal size?
That way we are not be changing a network parameter (size of the gossiped chunks) without doing any experimentation / simulations up front. Instead we switch to gossip the (orig) data serialized and arranged into rows but chunk that up into equally sized parts. With that we have exactly the same confidence as before but can still get rid of the partset header (because the 64kb chunks of rows can still be authenticated against the DAHeader instead of the partset header, you'd just need to send all proof nodes that you can't recompute locally). The only change necessary would be that all fields besides block.Data need to be part of the proposal (or gossiped separately too).

Let's close this for now. We won't continue this PR. At a later stage we might revisit this idea.


Multi-cloud multi account support

As a user of VMClarity I would like to be able to install VMClarity control plane once, and use that instance of VMClarity to discover and scan assets from multiple cloud accounts, kubernetes clusters or docker daemons.
Subtasks:

[x] https://github.com/openclarity/vmclarity/discussions/581
[x] https://github.com/openclarity/vmclarity/issues/637
[ ] https://github.com/openclarity/vmclarity/issues/780
[ ] https://github.com/openclarity/vmclarity/issues/640
[ ] https://github.com/openclarity/vmclarity/issues/641
[ ] https://github.com/openclarity/vmclarity/issues/642
[ ] https://github.com/openclarity/vmclarity/issues/643
[ ] https://github.com/openclarity/vmclarity/issues/644
[ ] https://github.com/openclarity/vmclarity/issues/645

@akpsgit @Tehsmash please provide the detailed description and architecture plan for this work piece.

To clarify: decoupling of compnents (just running discovery + scanner in environment, and have a central VMClarity instance without the discovery and scanner that collects results from different envs).


k9mail for Chromebooks: Enhance Tablet Mode

Introduction
Love k9mail on my phone. Thought it would suck on my shiny new Pixebook because it's engineered for smaller displays, but it doesn't and I believe it is the best email app for my Pixelbook.
It does what it says on the box:

Easy to setup and integrates well with contacts and file storage.
No 3rd-party data collection.
It is the only email app I have found that can delete emails with the Pixelbook's "backspace" key. The "refresh" and "back" keys also work (which is handy because some of the settings pages do not have a back button to touch)
it also works in windowed "phone"/portrait mode although the window is quite small. Almost like a widget!

To make it even better Tablet/big-screen mode could be enhanced.
Expected behavior
Would be great to capitalise on extra screen real-estate/space on tablets and Chromebooks.

allow 3 columns? Folders/Accounts, messages and preview (multi-column view for settings would also be handy)
include icons for all tasks (notably: reply, reply all, forward)
a "back" button or "close" button (X) for settings.
WISHLIST: support split-screen (re-sizeable window). A larger portrait view would be handy for split-screen. eg. 3 columns in full-screen landscape mode, 2 columns in split-screen, 1 column in phone/widget mode.

To be honest, that would be enough!!
Actual behavior

lots of wasted space and tedious going back and forth from folders to messages. Less of a problem with the Unified inbox

have to click on the send icon and then select the action every time. Twice as many clicks as needed.
Once you have selected one of the menus have to rely on the "back" button (<-) or "esc" on the keyboard. In tablet mode you need to swipe up from the bottom and use the little "back" arrow that appears bottom left.
You can currently only use k9 in full-screen or mini-phone views.

Thanks
Thanks for your time. I appreciate all the effort you put into this app. I hope none of the above comes off as pure criticism, it is my intention for it to be wholly constructive. This is the first time I have been inspired to post a feature request on GitHub and it's because I think you have an excellent App.
I would even be prepared to pay for a tablet version of K9. It would make that much of an improvement to my Pixelbook.
Environment
K-9 Mail Version: 5.403
Android version: Android for Pixelbook

PS: I don't know how to label this post (as per my message, this is my first).

With the recent release of the Google Pixel Fold and the Pixel Tablet, I would really appreciate this feature.


chore: Add stories for resource permission  select

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


[stable/prometheus-operator] walCompression of wrong type

What this PR does / why we need it:
Fix walCompression type.
Which issue this PR fixes

fixes #17043

Checklist

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


Setting wheelPropagation to false not working on mobile devices

Hi,
I want to disable the wheelPropagation the following way...
<PerfectScrollbar option={{ suppressScrollX: true, wheelPropagation: false }}>
This is working ok in computers but working wrong in mobile devices.
Version used<EMAIL_ADDRESS>

Closed for no updates


Survival function constant

Hi! Thank you very much for this amazing package!
Is it normal that I get constant survival function with this hazard function?

This is my code: `
cfTransplant = CoxPHFitter(normalize = False)
cfTransplant.fit(df = pdTransp.iloc[2:len(pdTransp)-1], duration_col = 'Survival_Time', event_col='NotCensored', show_progress=True, include_likelihood=True)
cfTransplant.print_summary()
fig, axes = plt.subplots(nrows=1, ncols=2, squeeze=False, sharex=True)

cfTransplant.baseline_cumulative_hazard_.plot(ax=axes[0,0], legend=False
                ,title='Baseline cumulative hazard rate')
cfTransplant.baseline_survival_.plot(ax=axes[0,1], legend=False
                ,title='Baseline survival rate')

Hey there,
hm, strange. Are you able to provide the data set just prior to the call to .fit? That would really help debugging.

Sorry, unfortunately I  cannot share the data.
But the weird thing is that if I do  normalize, thus I write:
cfTransplant = CoxPHFitter(normalize = False)
I get the following:

Likely fixed in https://github.com/CamDavidsonPilon/lifelines/pull/293


Reconstruct Framework for Practical Deployment Contents

The current architecture of the musecoco project seems not targeted for deployment. I'll begin by restructuring the framework to align with deployment objectives, ensuring that the architecture supports scalable and maintainable deployment.

The main for text2attribute is kinda easy, simply go through the project looking for do_predict flag

prepare data input

real inference

Complete


[Language request] Lithuanian language

Hello fellow enthusiasts,
I love using Excalidraw with my team and I would love to contribute and translate the application into Lithuanian 🇱🇹  language.
My crowdin username is Zigmas12. I assume it would be the best if you could also grant me proofreader permission. In that way I will be able to approve translations asap 💨
And thanks a lot for the awesome work you do!

Here you go https://crowdin.com/translate/excalidraw/10/en-lt?filter=basic&value=0 @Zigmas12 🚀 , how do you say "Lithuanian" in "Lithuanian"?

Here you go https://crowdin.com/translate/excalidraw/10/en-lt?filter=basic&value=0 @Zigmas12 🚀 , how do you say "Lithuanian" in "Lithuanian"?

Thank you!
"Lithuanian" translates to "Lietuvių"

#4322 added


When request sending fails at the NetworkClient, notify response handler only on network errors

If the request send fails due to connection unavailability (and not due
to node unreachable or other network related errors), do not notify the
response handler as that can lead to the node getting marked down
unnecessarily.

Updated.


Whether typedoc can generate Markdown files according to the directory structure

I'm writing Vue 3's Component Library, which has some hooks (the TS function) . The comments within the component are parsed by the AST and extracted into a Markdown file that is rendered by VitePress, which is generated according to the directory structure. However, when Typedoc is generated, it prints everything to a file. This is very difficult to process and display in a component library.
Whether typedoc can generate Markdown files according to the directory structure?

Version 4 has some additional output options which should hopefully fit most usecases. https://www.typedoc-plugin-markdown.org/docs/options#output-options .


Cleanup code formatting & Style

A lot of the source files are quite "dirty" and not formatted nicely. I think someone (like myself) should go through with something like JetBrain's Riders code formatter to fix all these issues.

I agree that the whole solution should be formatted, but please let me be the one who does it, because it involves changing a huge number of files, and I would like to know what exactly goes into the repo, which is hard if there are hundreds of files changed.

I agree that the whole solution should be formatted, but please let me be the one who does it, because it involves changing a huge number of files, and I would like to know what exactly goes into the repo, which is hard if there are hundreds of files changed.

Agreed!


Add Lidl HG06463A

Device information

https://zigbee.blakadder.com/Lidl_HG06463A.html
Home Assistant Device information

Livarno Lux E27 ST64 filament bulb (HG06463A) by Lidl
Checklist

[x] I have created a single PR per device. When you have multiple submissions please create separate PR's.
[x] For lights - I have only included the gzipped files (*.gz), not the raw CSV files.
[x] For lights - I have provided a CSV file per supported color mode. Look that up in Developer Tools -> States

Additional info

Measured with dummy load. Standby power was 0.03 so I manually edited it to 0.05

Thanks your contribution!
Measurements look good.
Standby power however is too low. Typically Zigbee lights have 0.2-0.4W, Wifi bulbs 0.5-0.8W.
Looking at all the other Lidl lights 0.3W would be a good estimation / average. So I would suggest to set it to 0.3W

Yeah this behavior also seems kinda odd to me. First measurements I did was without a dummy load and the standby power was exactly 0W. Then I added a Nest Audio Speaker, which was idling around 1.8W and the standby power of the bulb was recognized as 0.03W. I even repeated the measurements once more.
Didn't have any of those problems with the other Lidl Bulb (https://github.com/bramstroker/homeassistant-powercalc/pull/2024). Here the standby power was correctly measured even without dummy load.
Is this low standby power for this bulb even possible? If not what might have caused this unusual behavior?

This low power for the bulb is impossible. Never seen that on any of the hundreds of lights before, and that would definitely not be the case for the cheap Lidl bulbs.
You are unable to measure because it's to low for the Tapo to be picked up. This is common for most smart plugs. They are pretty accurate > 0.5W, but will have trouble or not able to pickup any power below that.
You will be able to measure using a manual powermeter which have higher resolution for example Zhurui PR10.
What did you use as dummy load? To make that work correctly you'll have to use a fully resistive load (for example inscadecent light bulb), a nest audio is not that, and should never be used as a dummy load.
When you are not able to measure just set it to 0.3W and call it a day. Which would be very close to reality anyway.

Okay thanks for the explanation. Gonna change it to 0.3W
Kinda odd that even lidl states the standby power to be 0W (https://www.lidl.de/assets/73c0ac02f4ec368b2d809e5fe59989dc.jpeg)


update KV error handler

This PR address error when region is splitting.

This change is

@ilovesoup PR is ready for reviewing. Origin logic is https://github.com/pingcap/tidb/blob/master/store/tikv/region_cache.go#L459
drop old region cache and insert new regions.
Origin logic is correct. I just improve log a little bit.


PCB PMOD Request: Quadrature Encoder For Educational Use

Now that we have SimpleIO-NextGen card with a cursor-type joystick, we'd also like to open this discussion thread for a Quadrature Encoder type of an input device.
This principle is used in many apps, from ordinary mouse and iPod music players, to industrial and even military. It is therefore important to have a good, open demo hardware for teaching the students this important topic.

https://en.wikipedia.org/wiki/Incremental_encoder
https://www.adafruit.com/product/5001
https://digilent.com/reference/pmod/pmodenc/start

If it is for students then best approach might be to go with simple one like this one on digilent PMOD - but it is also available as PMOD and I do not like to copy paste same boards
Adafruit looks super nice, but it is more expensive it will not fit in PMOD standard as it has more legs - but we might be able to fit it if we combine it with GPIO expander IC #6

You can also look at things like this:

Slide switch
Rotary switch

Combining it into GPIO Expander is a good idea!

Closing this issue, as it is now merged into GPIO Expander.


Version 0.3

Let's finish with triggers and new errors system, and then release 0.3?

Yes !

Until we reach 1.0.0: Lets check every month or so if something new is worth releasing. Think of all the people who are downloading alasql every day still having allllllllllllll the bugs we have been fixing the last 4 months.

What do you think about me releasing 0.2.2 tonight with what we have and then we take 0.3 when triggers and errors are OK?

Yes!
2016-01-15 17:45 GMT+03:00 Mathias Rangel Wulff<EMAIL_ADDRESS>
What do you think about me releasing 0.2.2 tonight with what we have and
then we take 0.3 when triggers and errors are OK?
—
Reply to this email directly or view it on GitHub
https://github.com/agershun/alasql/issues/533#issuecomment-171978725.

0.2.2 is out
https://www.npmjs.com/package/alasql
If you could pull the master branch and push to athmosphere (meteor) that would be awesome.

@agershun I have updated lodash to 4.0 - please do a npm install next time you pull develop

Meteor has changed some update things. I fixed one (replace addFiles() to addAssets(), but still in fighting with another error:
Error: Path contains forbidden segment '..'

Will work on it.

@mathiasrw BTW It is you turn to name 0.2.3 :)

Hah - no no - its your turn... :)
I chose "Mitilini" for the 0.2.2 to remember the many people arriving in tiny boats in search for at better life
But more interesting - what shall be the country of the 0.3.0 ?

Ok. Let is be... Spetses.
This is the Greek island from The Magus by John Fowles - the my favorite book.

cras amet qui nunquam amavit
quique amavit cras amet

For the next version - what do you think about Polinesia?

Polinesia is nice :)

We still have a problem with updating Meteor (because they changed API). To be solved.

Meteor updated: https://github.com/agershun/alasql/issues/545#issuecomment-222956897


Make context config option optional and default to any

[ ] Update in JSON schema
[ ] Implement functionality

This was implemented by @timsuchanek in f1bec7195128031876206cda443dfe3f463dd6db


Update access parameters shall display current values

When I display the access parameters modal i expect to see:

The bank login
the bank password as a password (not sure for this one, on a user's point of view)
The customFields (right website selected...)

To be able to know what to update

Thanks for opening the issue. To be able to do that, we need to include the accesses in the data we send to the client. Of course, the password shall not be sent (otherwise one would just have to type a few things in the developer's console to find out what the password is) nor shall it be shown even as stars in the password input (just changing the input type as "text" in the developers' console would be enough to see the password in plain text).

You're right


README suggests a deprecated brew command.

Description
README suggested to use brew update && brew cask upgrade to upgrade. It seems brew cask upgrade is deprecated.
Operating system: MacOs
Steps to Reproduce

run brew update && brew cask upgrade

Expected behavior:
Should not get a warning.
Actual behavior:
Get a warning:
Warning: Calling brew cask upgrade is deprecated! Use brew upgrade --cask instead.
Reproduces how often: 100%

Hi @pierre-vr and thanks for report :)
Would you like to make a fix?


Upgrade dependencies in pom.xml

Update pom.xml and libraries; set minimum Java version to 11

Updated pom.xml to the latest version.
Upgraded libraries to their latest versions.
Changed the minimum required Java version to 11 to support the updated libraries.
Removed aws-kinesis-analytics-java-apps.iml due to the pom.xml update.

@nicusX This has been open for almost a month. If you can view it and close or merge it, that would be great!

Thanks for the PR @toco8
To really upgrade this example to 1.19, the new JDBC Connector should be used, rather than the obsolete Flink JDBC from Flink 1.10.
I am even surprised it still works.
The API has changed completely, and unfortunately it requires a substantial code change.
Also, the Maven plugins (maven-compiler-plugin and maven-shade-plugin) should be updated to a more recent version.

Thank you for your review of the PR, @nicusX.
I will conduct an investigation over the weekend regarding the upgrade to 1.19, focusing on implementing the new JDBC Connector and updating the Maven plugins as you suggested. I aim to respond with my findings and proposed changes by August 6th.


Daikin fixes

Description:
Related issue: #12000, pydaikin==0.6 does not expect f_dir and f_rate to be set.
Checklist:

[x] The code change is tested and works locally.
[x] There is no commented out code in this PR.

Related to #18413

@fabaff you are free to merge this now, I've verified and it is correct now.


Specify scopes needed for GITHUB_TOKEN

https://github.com/bufbuild/buf-setup-action/blob/142056acbddd5ea00e4b7d2834868cd40ed12aca/README.md?plain=1#LL36
Can you please clarify what scopes the token needs?

hi @miparnisari taking a quick look at the code, we use the github token in octokit to call for the latest buf to download:
https://github.com/bufbuild/buf-setup-action/blob/ffdd84c92237b134e53ff61162380d729eac279f/src/buf.ts#L136-L137
Which uses these endpoints:
GET /repos/{owner}/{repo}/releases
GET /repos/{owner}/{repo}/releases/tags/{tag}

so the token would need contents:read permission. If that sounds sensible then I can put a PR up to clarify this in the README.

Yes please :)


Is it a long-only library?

There are rules for entry and exit, but no rules for going short and exiting?

It is not a long-only lib. It is up to you to build your own strategy with your rules or using the existing ones. "Enter" does not mean "going long".


Swift 4.x (x < 2) names are obsoleted in 4.2, not 5.

In APINotes, when an unversioned name overrides a previously declared name with a Swift version, if the
previously declared name is guarded under version 4 the obsoleted version should be 4.2, not 5.  In all other cases, the obsoleted version is the next major Swift version.
Fixes rdar://problem/39950937

Build failed
Swift Test OS X Platform
Git Sha - cc3024d7c5e490fb9e55f14e3bde28ef86c8a018

I think the else-block should preserve the minor version number iff it's 4.2. Another hardcoding, but it doesn't really make the problem worse.

@swift-ci test

@jrose-apple Can you confirm my logic for the else block is what you wanted?

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test


Initial start application

I'm create a hello worldapplication but when start it show me this message:
Please enter your CouchDB _admin credentials:
I'm write my username and password but this command-line show me again the same message.

it looks like you use the old Hoodie version. Can you try this please?
First, uninstall the hoodie CLI, it’s deprecated
npm uninstall --global hoodie

After that create a new folder, create a package.json and install hoodie
mkdir hoodie-test-app
cd hoodie-test-app
npm install --save hoodie

then create public/index.html with any content you  want. And then
npm start

Does that work for you?

I don't need require hoodie package ? Just create a file index.html ?

Great, I understand. tks 😃

Just create a file index.html ?

In the public/index.html file you can load the Hoodie client using <script src="/hoodie/client.js"></script>


"Game does not seem to be a unity project"

When i try to install vrctools using the mod manager it says "Game does not seem to be a unity project. Could no find the libraries to patch."
[Press any key to continue]
help

Hey
The VRChat mods are currently broken since VRChat v2020.1.1 (Udon & Unity 2018 update).
We are currently working on an universal mod loader for every Unity games, called MelonLoader.

If you want more infos, please join the VRCTools discord: https://discord.gg/rCqKSvR

Okay thank you!
On Sat, 11 Apr 2020 at 20:25, Slaynash<EMAIL_ADDRESS>wrote:

Hey
The VRChat mods are currently broken since VRChat v2020.1.1 (Udon & Unity
2018 update).
We are currently working on an universal mod loader for every Unity games,
called MelonLoader.

If you want more infos, please join the VRCTools discord:
https://discord.gg/rCqKSvR
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Slaynash/VRCTools/issues/15#issuecomment-612542639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMSR5BVFJ2OHGYAG2P5HGUDRMEDAPANCNFSM4MGGCXIQ
.


store options with which is DataStore created

It is because I need to read loadParams, from it to call other api with same params

Alternatively, you can assign them externally, and only those parts that are needed:
function createStore2(options) {
    var store = AspNet.createStore(options);
    store._loadParams = options.loadParams;
    return store;
}

It will be some problem to have it there all? just for case?

Appending them to the resulting Store object will mean duplication of data. Options are already there in an intermediate form:
https://github.com/DevExpress/DevExtreme.AspNet.Data/blob/9de39ac3b2cb332c4f98ebc8d141ed9e51030d68/js/dx.aspnet.data.js#L36-L43
Since your use case is sharing loadParams between different APIs, a more clean solution is to introduce a sort of API context object like this:
var apis = {
  loadParams: ...,
  store: ...,
  // any extras
}

and use it across the app code.

I understood, but this approach is not possible in our case. If it is problem to store whole options, it will be enough to store just a store._loadParams = options.loadParams;  as you recommend.

I have to admit, I did not fully understood, how you meant it with the wrapper function. I do not want to create own unique DataStore . Inline javascript code is generated on server side by C#. And is not so easy to modify it. Thats why I want to add possibility to read options from that datastore on user action. In our application we are going to used it for export large datagrid to PDF. To do it, I need to get from datagrid its actual state and its datastorage loading params. To generate PDF, I need to send the params to another API.
So if your "wrapper function" is meant to modify file js/dx.aspnet.data.js like this:
     function createStore(options) {
         var store = new CustomStore(createStoreConfig(options));
         store._useDefaultSearch = true;
         store._loadParams = options.loadParams;
         return store;
     }

It will be OK for my current needs too.

Inline javascript code is generated on server side by C#

@miroslav-peterka Do we understand it correctly that you are using the DevExtreme MVC Controls?

if your "wrapper function" is meant to modify file js/dx.aspnet.data.js

No, on the contrary, my point is that to meet your requirements, no changes are necessary in DevExtreme.AspNet.Data.
In a certain place of your app you definitely have a JavaScript call like this:
DevExtreme.data.AspNet.createStore(...)

(or something similar, depending on how dx.aspnet.data.js is loaded and whether or not you are using ES modules)
I suggest a very straightforward thing. Declare your own JavaScript function similar to the one from https://github.com/DevExpress/DevExtreme.AspNet.Data/pull/245#issuecomment-389456790 and call it instead of the original ...createStore.
To reiterate, from my undetstanding, the changes are needed in the app code, not in the library.

@San4es yes
server side code looks like
@(Html.DevExtreme().DataGrid()
...
   .DataSource(d => d.WebApi().ApiController($"{GridKeyList}").Key("Id")
   .LoadParams(new { status = Model.MyData.Status }))

@AlekseyMartynov It is possible to keep current server side code (it is used on many pages), together with using own datastorage?

Now I understand that you use DevExtreme MVC data sources.
Here are possible solutions.

In Layout, right after <script src="~/lib/devextreme-aspnet-data/js/dx.aspnet.data.js"></script> add the following script:

<script>
    (function () {
        var originaCreateStore = DevExpress.data.AspNet.createStore;
        DevExpress.data.AspNet.createStore = function (options) {
            var store = originaCreateStore(options);
            store.myLoadParams = options.loadParams;
            return store;
        };
    })();
</script>

Serialize the object used in LoadParams as a page-global variable and use it for calling other APIs:

<script>
    var gridDataSourceStatus = @Newtonsoft.Json.JsonConvert.SerializeObject(Model.MyData.Status);
</script>

Take full manual control over data source instantiation with the new JS() helper:

<script>
    function createMyStore() {
        var store = DevExpress.data.AspNet.createStore({
            key: "Id",
            loadUrl: "@Url.Action("...")"
        });
        store.myLoadParams = @Newtonsoft.Json.JsonConvert.SerializeObject(null/*Model.MyData.Status*/);
        return store;
    }
</script>

@(Html.DevExtreme().DataGrid<Order>()
    .DataSource(new JS("createMyStore()"))

I'd suggest to go with the item 2 because it's concise and doesn't introduce unnecessary coupling. Item 1 implements exactly what you do in the PR. Item 3 is the most powerful though less convenient way which you can use in similar cases.


Correction comment for Str::orderedUuid() - https://github.com/larave…

Pull request for https://github.com/laravel/framework/issues/50231: "Incorrect information in the comment block for the function Str::orderedUuid()"

Submitted again with edit permission as https://github.com/laravel/framework/pull/50268


kernel-modules-headers: Revert kernel 5.0.3 fixdep patch

The patch will be merged in meta-balena so let's remove it
( https://github.com/balena-os/meta-balena/pull/1567 )
Changelog-entry: Remove kernel-modules-headers fixdep patch for kernel 5.0.3 as it will be applied in meta-balena
Signed-off-by: Florin Sarbu<EMAIL_ADDRESS>

@balena-ci retest


Consider publishing an npm module from the wasm compilation

In this project, there is already a Blazor application that is using Kiota in wasm:
https://app.kiota.dev/generate
This is arguably awesome 🙂 but is hard to re-use, I noticed that recent improvements are making the compilation with WASM independent:
https://devblogs.microsoft.com/dotnet/use-net-7-from-any-javascript-app-in-net-7/
I assume that it would be possible to publish a JS package that can be reused in frontend-only applications with a process similar to the one used here:
https://github.com/maraf/dotnet-wasm-react/tree/main/qrlibrary/dotnet

The use-case is running the code generator in the browser from a React application.
I haven't noticed the NuGet package before, but, using it should be pretty doable to integrate a thin wrapper layer and expose the functionality.
Thanks for the feedback!


macOS 12.7.1 compatibility

macOS 12.7.1 compatibility implemented - xcode version 14.2 (14C18)

I'm still running a mac trash can (MacPro6,1) and macos 12.7.1. Was so excited to find this project. Works fine in 12.7.1 after adding all the required "@available(...)" decorators.


Mines Market Menu

I still have my Sodexo API key, which would allow us to have the bot post updates to the server about the menu at Mines Market.
Feature Requests:

Command that assigns the bot a channel to post menu updates
Command that defines when and how often the bot updates the menu
Error handling for missing API key that disables the module (I won't be distributing my key, though I can design examples and unit tests that verify the code without the Sodexo API so people can still develop for it)
Schedule bot to periodically fetch from the Sodexo API and post to assigned channel (either by editting existing messages or post new ones)

Extra:

Command to have a version of the menu DMed to you

I am working with a couple students to develop a project that works with the Sodexo Bite API over at https://github.com/ColoradoSchoolOfMines/sodexo-api. When we get far enough in that project I might have them bring those skills over to work on this issue.


Deployvm improvements

Auto-select zone if only one present
Allow users to search zones, pods, clusters and hosts

@blueorangutan package

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

@davidjumani a Jenkins job has been kicked to build primate packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark:centos :heavy_check_mark:debian :heavy_check_mark:archive.
QA: http://primate-qa.cloudstack.cloud:8080/client/pr/666 (JID-3432)

@utchoang can you review  and test this? Thanks


add upgrade tests for helm charts

We are testing upgrades from the kubernetes templates not for the helm charts. As we don't have any upgrade testing for the helm charts it is difficult to identify the upgrade problems. This issue is to track the upgrades tests for the helm charts for both cephfs and rbd.

one of the upgrade bug https://github.com/ceph/ceph-csi/issues/2858


Dev

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

21 of 23 (91.3%)  changed or added relevant lines in 7 files are covered.
18 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-4.5%) to 79.752%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/vba_linter/rule_loader.py
1
2
50.0%

src/vba_linter/rules/rule_base.py
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

src/vba_linter/rule_loader.py
4
64.71%

src/vba_linter/linter.py
14
57.58%

Totals

Change from base Build<PHONE_NUMBER>:
-4.5%

Covered Lines:
193

Relevant Lines:
242

💛  - Coveralls


Fix wrong typing in pybricks.parameters.Stop

David, Laurens,
I've found some minor wrong typing info.

Thanks! Can you also add a changelog entry for this?

Yes, David, I've added a line about this in the change log. :)

I just pushed a commit to master that should fix the CI failures. Can you please rebase both pull requests so we can try the CI again?

great, it seems all sorted out now David :)

Merged, thanks!

Labor Day achievement! :D
Thanks David.


Blaze Server rejects valid Content-Types with option information

Firefox <44 sends an additional charset= option with the content-type header, even when it's not required (see the bug) -- this, or any other option, makes the content-type unrecognizable by Blaze server.

Note this should be fixed by #1490 :smile:


Remove Code climate from the build GH action

Secrets are not available for outside collaborators, this
will make the build fail for every external collaborator

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
630

Relevant Lines:
659

💛  - Coveralls

But having code climate is good right?

It is a good solution to have code climate, but we cannot rely on the api key for this, we need to change the way it interacts with builds.
Let's remove it for now just to unblock the PRs.


[MXNET-1179] Enforce deterministic algorithms in convolution layers

Description
Some of the CUDNN convolution algorithms are non-deterministic (see issue https://github.com/apache/incubator-mxnet/issues/11341). This PR adds an env variable to enforce determinism in the convolution operators. If set to true, only deterministic CUDNN algorithms will be used. If no deterministic algorithm is available, MXNet will error out.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

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

Changes

[x] new environment variable added: MXNET_ENFORCE_DETERMINISM (default false)
[x] CUDNN covolution, deconvolution, max pooling operators

Comments

still need to add a test to verify determinism.

@eric-haibin-lin @DickJC123 @wenyangchu  please review

@apeforest Thanks for the awesome contribution. Could you please take a look at the comments given above
@mxnet-label-bot [pr-awaiting-review, operator]

Is there a reason why a unit test was not added for this change?


External USB Socket

Include an external USB Socket (with weather cap).
Something like the following (but less costly!!):-
https://au.element14.com/multicomp-pro/mp002448/usb-type-c-r-a-ip68-sealed-connector/dp/3369657?gclid=EAIaIQobChMIw7v1jdOY-gIVzgRyCh01vA2CEAQYDCABEgIAJPD_BwE&mckv=_dc|pcrid||pkw||pmt||slid||product|3369657|pgrid||ptaid||&CMP=KNC-GAU-GEN-SHOPPING-PLA-PMAX

cant afford this at this time 😞


Respect robots.txt

If a site to be tested explicitly disallows crawlers, we should remove it from the test URL list.
WPT also allows appending to the User-Agent and we might want to consider setting that to "HTTP Archive" so it's obvious in the logs who initiated the crawl on a site.
I'd say this would also make it easier for sites to explicitly disallow HTTP Archive as opposed to all crawlers or even all WPT agents (via PTST/1 UA string) but the browser version changes the literal UA string frequently enough that it's not really going to matter. A site can still have a UA wildcard, applying to all bots/crawlers, so we should still respect that and blacklist the site.
Related to https://github.com/HTTPArchive/httparchive/issues/115

If this is implemented I think that there should be an option to turn it off as well. However, I think it might be worth considering that if a site disallows all bots except the wayback machine then it might be okay to crawl it.

Could you elaborate? Not sure I follow re: option to turn it off.
It seems iffy to assume that sites that allow archive.org implicitly want to allow httparchive.org. If we go this route I'd prefer to take the site's config literally to be sure.

Like allow someone to run it without listening to robots.txt. We could potentially avoid people getting mad by changing the useragent when that option is on.
Since HTTP Archive is a smaller project than the Wayback Machine, people are less likely to put HTTP Archive into their robots.txt file. For example, github.com/robots.txt excludes all directories except for certain bots. This is the bottom of their robots.txt file:
User-agent: * Allow: /humans.txt Disallow: /
It's likely that many websites have a similar configuration. So respecting robots.txt would have the side effect of many websites being excluded.

Well if anyone else is running their own HTTP Archive crawl, they're free to change any configs in their fork :)
Before we make a change like this, we would definitely run a test to evaluate its impact. If it turns out that we're losing a significant portion of sites, we'd rethink it.

FWIW, I expect that this would be done in a pre-crawl filtering pass of some kind when we are determining the URLs to crawl (same with removing dead domains, etc).

FWIW if/when we switch over from Alexa to Chrome UX Report, non-indexable sites are filtered out by default.

Obsoleted by CrUX corpus


[Sorcerer] new sheet

Changes / Comments
New sheet for the Sorcerer role-playing game. Includes sorcerer and demon sheets. No legacy support required.

[Y] Does the pull request title have the sheet name(s)? Include each sheet name.
[N] Is this a bug fix?
[N] Does this add functional enhancements (new features or extending existing features) ?
[N] Does this add or change functional aesthetics (such as layout or color scheme) ?
[N/A] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[Y] If this is a new sheet, did you follow [Building Character Sheets standards]

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


fix: support evaluate join that has one side condition

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Clear Hand doesn't reset "played this round" check

"Number of cards in hand" also doesn't reset to 0/8

It wasn't meant to, but I guess it makes sense to expect it to.

The number of cards in hand is just a visual thing, but I guess I can fix that.


Decouple from active support?

can we implement time_ago_in_words (with ago) and drop requiring active support?
kaminari depends on this so Outboxer::Messages.list would need to be decoupled as well

related: add_optional_dependency


How to open the lightbulb via shortcut?

I can't find a shortcut for opening the lightbulb / codeActions at the current cursor position.
How can I create such a shortcut?
I tried to create a shortcut for vscode.executeCodeActionProvider by creating a custom keybinding like this:
[{ "key": "alt+enter", "command": "vscode.executeCodeActionProvider"}]

But everytime I hit the shortcut I receive the warning

Running the contributed command:'vscode.executeCodeActionProvider' failed.

In mac it's "CMD+." (Command plus period) and I guess it's "Ctrl+." in windows and linux.

In mac it's "CMD+." (Command plus period) by default and I guess it should be "Ctrl+." in windows and linux. if you want to change it find this:
{ "key": "cmd+.", "command": "editor.action.quickFix", "when": "editorTextFocus" }

Correct, adding @aeschli for further questions regarding quick fix


Support repository creation

This one will break falling through to hub since hub create and lab create would conflict and there isn't a straightforward way ( that I can think of ) do detect which the user wants. That said, I'll implement soon and hub users can call hub explicitly for repo creation. It shouldn't be that big a deal since creating repos is a less frequent operation.

@bentinata I coded up some initial support for this as lab repo create (alias lab project create). I still need to add some tests and address some known edge cases, but should be able to get this out in the 0.10 release targeted for this week.

@zaquestion Thanks for your effort into this. Really appreciated.

@bentinata v0.10 is released
lab project create [path]
# or
lab repo create [path]


Update action-pre-commit to v2

👍 applied changes in all repos so missed this one only updated docs. one more there and we should be good I think: https://github.com/open-turo/actions-argocd/pull/11


Update README.md

Add instructions for migration from OAuth1
Includes instructions for use with devise which closes #2

Thank you very much!


named route

How can ı add named route to pages(for example instagram website has side menu too but when you go to messages url goes to instagram.com/direct/messages ı want to do it like that in my project. I looked around internet but couldn't find it. )

To create named routes with Flutter for navigation similar to Instagram's side menu behavior, you can use the Flutter package called "flutter_bloc" in combination with "flutter_bloc" or "Provider" for state management. Here's a general outline of how to do it:

Add Dependencies: In your pubspec.yaml file, make sure to include the necessary packages.

dependencies:
  flutter:
    sdk: flutter
  flutter_bloc: ^7.0.0
  # Other dependencies as needed

Create Named Routes: Define named routes in your app's MaterialApp by using the routes property:

MaterialApp(
  routes: {
    '/home': (context) => HomeScreen(),
    '/profile': (context) => ProfileScreen(),
    '/messages': (context) => MessagesScreen(),
    // Add other routes as needed
  },
  // Your other MaterialApp configuration
)

Implement Navigation: To navigate between these routes and mimic Instagram's side menu behavior, you can use a package like flutter_bloc or Provider for state management.

Create a Bloc (if using flutter_bloc) or a Provider to manage the current route state.
Update the state when the user selects a different menu item.

For example, if using flutter_bloc, you could create a simple Bloc to manage the selected route and use a BlocBuilder to rebuild the UI when the route changes:
class RouteBloc extends Bloc<String, String> {
  RouteBloc() : super('/home');

  @override
  Stream<String> mapEventToState(String event) async* {
    yield event;
  }
}

// In your app's main function:
void main() {
  runApp(
    MaterialApp(
      home: BlocProvider(
        create: (context) => RouteBloc(),
        child: App(),
      ),
    ),
  );
}

// In your UI, use BlocBuilder to conditionally display the selected route:
BlocBuilder<RouteBloc, String>(
  builder: (context, route) {
    switch (route) {
      case '/home':
        return HomeScreen();
      case '/profile':
        return ProfileScreen();
      case '/messages':
        return MessagesScreen();
      // Add cases for other routes
      default:
        return HomeScreen(); // Default route
    }
  },
)

Trigger Navigation: To navigate to different routes, you can use the Navigator or use widgets like InkWell or GestureDetector for handling tap events.

For example, to navigate to the "Messages" route:
GestureDetector(
  onTap: () {
    context.read<RouteBloc>().add('/messages'); // Use context.read with Provider
  },
  child: Text('Messages'),
)

By following this approach, you can set up named routes and dynamic navigation in your Flutter app, similar to Instagram's side menu behavior. Make sure to adapt it to your specific project structure and state management preferences.

Thank you for this explanation 😊


Support keyboard shortcuts in the TextEditor

Specifically

[ ] CMD-b should be the same as clicking the "bold" button
[ ] CMD-i should be the same as clicking the "italic" button
[ ] CMD-k should be the same as clicking the "link" button (pilfered from Github, but so useful)

Using the CTRL versions should work in Windows, and it looks like mousetrap supports a mod helper to do that, or we can just use "ctrl-b", "cmd-b".

I'll work on this in tandem (as separate branches, of course) so I can better understand and make sure the work done on #2723 lines up with the other upcoming improvements.


[error] Can't run this task on @nektos/act

Hello!
I was checking your repo for a workflow I'm actually writing. For testing locally, I'm using @nektos/act to properly addresses easy-fixes before committing to main. On many actions I've had no inconveniences, but in yours, particularly, it drives my attention the use of a yarn.lock without a proper package-lock.json, which is the reason i think the act tool couldn't handle dependency resolution properly, ultimately outputting an error like this:

Is there any possibility you could deliver both files (package-lock.json and yarn.lock) to check if this fixes the observed behaviour on act?
Thanks!

Sorry for the inconvenience!
It seems I was testing my workflow with a somehow outdated version. Using latest works as expected (Obviously not showing any output data due to depending on GHA totally on that side)

Thanks!


platypus variant calling - region list

I'm getting an error when trying to run platypus joint calling in bcbio.
[2016-02-11T13:22Z] platypus variant calling
[2016-02-11T13:22Z] 2016-02-11 08:22:01,002 - ERROR - Platypus found no regions to search. Check that you are using the correct reference FASTA file or have specified the 'regions' argument correctly

command used:
[2016-02-11T13:19Z] platypus callVariants --regions=/media/sf_BigShare/16014_wes_pilot/work/platypus/3/HapMap-3_173996632_189507659-raw-platypusregion.list --bamFiles=/media/sf_BigShare/16014_wes_pilot/work/bamprep/HapMap/3/13_2016-02-01_project-merged-sort-3_173996632_189507659-prep.bam --refFile=/usr/local/share/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa --output=- --logFileName /dev/null --verbosity=1 --assemble=1 --hapScoreThreshold 10 --scThreshold 0.99 --filteredReadsFrac 0.9 --rmsmqThreshold 20 --qdThreshold 0 --abThreshold 0.0001 --minVarFreq 0.0 | awk -F$'\t' -v OFS='\t' '{if ($0 !~ /^#/) gsub(/[KMRYSWBVHDX]/, "N", $4) } {print}' | vcfallelicprimitives --keep-geno | vcffixup - | vcfstreamsort | bgzip -c > /home/rsicko/tmp/bcbiotx/7e85cbd5-c246-4ba1-b924-1803b0da2eee/tmpE0kpki/HapMap-3_173996632_189507659-raw.vcf.gz

List file and reference file are both present and accessible. This same error appears to be occurring for all regions. I'm not sure if it is something I specified incorrectly, a bcbio error or a platypus error.
Additional info - I'm running ensemble variant calling and it appears gatk, freebayes and samtools are calling variants.
portion of project.yaml file.
details:
- algorithm:
    aligner: bwa
    effects: vep
    ensemble:
      classifier-params:
        type: svm
      classifiers:
        balance:
        - AD
        - FS
        - Entropy
        calling:
        - ReadPosEndDist
        - PL
        - PLratio
        - Entropy
        - NBQ
      format-filters:
      - DP < 4
      trusted-pct: 0.5
    jointcaller:
    - gatk-haplotype-joint
    - freebayes-joint
    - platypus-joint
    - samtools-joint
    mark_duplicates: true
    mixup_check: qsignature
    phasing: gatk
    realign: false
    recalibrate: gatk
    remove_lcr: true
    variant_regions: /media/sf_BigShare/Project_Tsai_Project_025/agilent_files/S04380219_Covered_noChr.bed
    variantcaller:
    - gatk-haplotype
    - freebayes
    - platypus
    - samtools
  analysis: variant2
  description: A06551
  files:
  - /media/sf_BigShare/16014_wes_pilot/work/fastq/merged/A06551_R1.fastq.gz
  - /media/sf_BigShare/16014_wes_pilot/work/fastq/merged/A06551_R2.fastq.gz
  genome_build: GRCh37
  metadata:
    batch: batch1
    phenotype: affected
    sex: male

Thanks for the heads up. Apologies, this is a bug in bcbio triggered by recently updating to the latest 0.8.1 release. The new release no longer reads input files unless they end with .txt or .bed and we were passing in a file suffixed with .list. Since the new version uses BED files directly, I adjusted to use these instead of the custom file Platypus previously required.
Apologies about the problem. If you update to the latest development and remove the current platypus directory you can rerun and things should hopefully proceed cleanly for you:
rm -rf platypus
bcbio_nextgen.py upgrade -u development

Thanks again for reporting the problem.

do I need to change the yaml file to pick up where I left off? when I restart, bcbio is creating a folder called align_prep and is copying the first fast

You shouldn't need to change anything in the YAML. The align_prep bgzipping work is likely due to upgrading from an older version. We didn't do this by default previously, but sped it up a lot so do now to make it easier to parallelize and have a uniform input process for aligners. Sorry for the extra work and I hope the updated run works cleanly for you.


Add User/Team tabs to Job templates

trying to add teams using the toolbar filters in the modal

trying to add teams using the table row item in the modal

QA'ed this locally,
I was able to

create a JT
Select the team(s) and give access to the Job Template I created ✅
Select the roles(s) to apply to the Job Template I created  ✅
Select the user(s) and give access to the Job Template I created  ✅
Select the roles(s) to apply to the Job Template I created  ✅


yii -> Yii - Update TbEditableSaver.php

Uppercase first letter required for Yii class.

Very old request- no longer expecting this to be implemented.


Podman support and installation is not simple, easy, or predictable

Please consider Podman, should be a drop in, supports all the docker cli. However this alone is not enough to get going at least for me.
The Nix install failed for me and I think the docs here understate the complexity of this install, really unfortunate The irony of a tool to have easy predictable shells and yet installation is not predictable itself is not lost on me.

@AddictArts thanks for raising these issues.
On Podman, that's a good suggestion. I'll look into it.
We also similarly feel that the nix install requirement is burdensome. We're actively exploring ways to mitigate this - stay tuned!

@AddictArts What sorts of errors are you seeing in the nix install? While we don't maintain that project, if we understand the errors we might be able to mitigate them or improve them in the context of a devbox install.

Let's focus this issue on just the nix installation difficulties, and continue discussion on podman support on #18 .
Note that you can run devbox build --engine=podman, as mentioned in:
https://github.com/jetpack-io/devbox/issues/18#issuecomment-1236120304
But please give it a show and do let us know if you encounter any issues!

Thanks @loreto and @ipince
I did stumble on that, so if I can ever get passed the nix install then I'll give it a go. Although it looks like the nix install is a bust. MacOS is just not viable for nix and Windows requires WSL2, which basically means devbox will be a linux land utility only. While some may have it working on MacOS or Windows it is not first class like linux and other alternatives should be used if possible. Sorry nix fans, but this is the reality.
The error is mounting the volume, it fails to mount, filed an issue with nix.

Closing this issue out, as we have deprecated devbox build. Developers can now generate a Dockerfile for their project using devbox generate dockerfile, and build the container with the tool of their choice
For nix installation -- if the default Nix installer does not work, we recommend either allowing Devbox to install Nix for you (via devbox setup nix), or using the Determinate Systems Nix Installer (https://determinate.systems/posts/determinate-nix-installer)


Restore Makefile and remove generation of redirects

resolves #3578
What are you changing in this pull request and why?
Two things:

Restore ability for users to use make run and make build to preview/build the docs site while developing locally.
Align with the goal in https://github.com/dbt-labs/docs.getdbt.com/pull/3284 to remove generation of redirects from the Makefile.

Outstanding questions
Question 1
The pull request template says the following:
- [ ] [Ran link testing](https://github.com/dbt-labs/docs.getdbt.com#running-the-cypress-tests-locally) to update the links that point to the deleted page

I'm assuming this is the most up-to-date approach for link testing? I haven't used it personally yet, so I don't know 🤷
Question 2
After the changes within this PR, there are no longer any references to any make commands within our README. Do we want to add them anywhere as an alternative method to running the npm commands directly?
Checklist

[x] Review the Content style guide and About versioning so my content adheres to these guidelines.
[x] Remove unnecessary parts of Makefile
[x] Add @JKarlavige as a reviewer ("needs technical review")

@JKarlavige I tried guessing at what the Makefile and README should look like -- I didn't do any manual testing.
Could you give this PR a look and determine it we need to make any changes?


Make plugin manager to load plugins based on environment variables

Make plugin manager to take 2 env variables:
plugins.root.directory and plugins.loading .
So user could specify the plugins to load based on their own needs.

Codecov Report

Merging #4956 into master will decrease coverage by <.01%.
The diff coverage is 29.16%.

@@             Coverage Diff              @@
##             master    #4956      +/-   ##
============================================
- Coverage     57.13%   57.12%   -0.01%
  Complexity       12       12
============================================
  Files          1159     1159
  Lines         61778    61802      +24
  Branches       9112     9116       +4
============================================
+ Hits          35294    35307      +13
- Misses        23890    23902      +12
+ Partials       2594     2593       -1

Impacted Files
Coverage Δ
Complexity Δ

...ava/org/apache/pinot/spi/plugin/PluginManager.java
36.76% <29.16%> (-6.42%)
0 <0> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
58.06% <0%> (-12.91%)
0% <0%> (ø)

.../impl/dictionary/FloatOnHeapMutableDictionary.java
46.34% <0%> (-8.54%)
0% <0%> (ø)

...impl/dictionary/FloatOffHeapMutableDictionary.java
60.21% <0%> (-7.53%)
0% <0%> (ø)

...pache/pinot/core/util/SortedRangeIntersection.java
83.82% <0%> (-7.36%)
0% <0%> (ø)

...mpl/dictionary/DoubleOffHeapMutableDictionary.java
43.01% <0%> (-6.46%)
0% <0%> (ø)

...ore/startree/v2/builder/BaseSingleTreeBuilder.java
90.52% <0%> (-4.74%)
0% <0%> (ø)

...troller/helix/core/retention/RetentionManager.java
75% <0%> (-4.17%)
0% <0%> (ø)

...e/operator/dociditerators/MVScanDocIdIterator.java
46.03% <0%> (-3.18%)
0% <0%> (ø)

...ller/validation/OfflineSegmentIntervalChecker.java
29.48% <0%> (-2.57%)
0% <0%> (ø)

... and 16 more

Continue to review full report at Codecov.

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


and keyword is not highlighted

When I have something like the following, the and keyword is not highlighted in VS Code.
let foo = (ls | where type == file and name =~ '.*\.nu')

agreed. i believe it was a recent addition to the language. i bet or doesn't highlight right either.

Should be fixed by #73


Error handling

Introduce proper error handling and propagate errors to top level.
This changes the api. Allmost all methods return a Result<T, ApiClientError> now where T mostly is of type Option<T>

added FromHexTrait
added api.signer_account(&self) -> Option<AccountId32>

this looks nice and clean and also covers #105. examples and tutorials covered.
There seems to be a CI issue
@electronix can you have a look why CI fails?

I will


Add dimensions

@somexpert @jason-ellis  this adds height and width of images to aubrey's resource object where applicable.

@jason-ellis @somexpert Had to modify this so it would work with both local and web requested JSON files, but there is less code to look at now :grin:

Looks good to me, and it seems to be working in alpha. :thumbsup:


Exception: torchsearchsorted on CUDA device is asked, but it seems that it is not available.

Everything works fine on CPU. But when I tried to run the test.py on GPU, exception is raised:
" torchsearchsorted on CUDA device is asked, but it seems that it is not available. Please install it"
Please see below:

Looking for 50000x1000 values in 50000x300 entries
NUMPY:  searchsorted in 3538.361ms
CPU:  searchsorted in 2558.010ms
difference between CPU and NUMPY: 0.000
Traceback (most recent call last):
File "test.py", line 58, in
test_GPU = searchsorted(a, v, test_GPU, side)
File "/home/yin/anaconda3/lib/python3.7/site-packages/torchsearchsorted/searchsorted.py", line 41, in searchsorted
raise Exception('torchsearchsorted on CUDA device is asked, but it seems '
Exception: torchsearchsorted on CUDA device is asked, but it seems that it is not available. Please install it
“

Sorry ,similar issue has already been resolved. Hereby I close it

hello, i meet the same issue, could you tell me how you solve it?


Lost datapoint

I collected a datapoint, named it, saved it.  Then it was gone.
I see a mystery point on the Data sync status screen.  But I don't know how to check to see if it's missing point or delete it.

I collected a datapoint, named it, saved it. Then it was gone.
I see a mystery point on the Data sync status screen. But I don't know how to check to see if it's missing point or delete it.

@n-clinton What was the sync status on that point? "Edit/delete" wasn't a priority for the project sponsors so it wasn't implemented.

I see a mystery point on the Data sync status screen. But I don't know how to check to see if it's missing point or delete it.

Also, does the name you gave the point appear on the sync status screen? Could you please share a screenshot?

@n-clinton I pinged you. I suspect #2740 fixes this, but @rfontanarosa still needs to cut a release. In the meantime, would you able to try the workaround I sent on chat?

@n-clinton we just made a release, could you let us know if it's still reproducible? v 0.1.10

@n-clinton We discussed - turned out that the actual GPS location wasn't the main geometry for that site. I've proposed https://github.com/google/ground-platform/issues/2075 to make this scenario less likely, closing this for now.


the -v flag for tar is unsupported on older and bsd versions of tar, …

…which is an issue on older platforms

:+1:

Have we run the integration tests against Ubuntu and CentOS?

@schisamo - I didn't think it was necessary since it's a minor non-additive change. I'll run them now.

@schisamo ran all the tests locally - turns out there's no way they could have worked in the past, not sure what the deal was there. I've updated them and they all work now.

@scotthain Awesome! :+1:
I'll work on wiring all integration tests to run in Travis (like we do for the omnibus and languages cookbooks).


Added meta preview image generation using our defund12-image-api

this changes the meta tag to the url from the image generation api: ie. https://defund12-image-api.now.sh/api/preview?city=Sacramento&state=CA
we already hosted the api on vercel serverless, so the api shouldn't have any quotas, and hence, should be quite reliable
replaces #1629

LGTM, but would like to have @Ceshion take a look as well if possible.

Gonna merge to unblock.


The output pane does not receive touch events on mobile / tablet

See these issues:
• https://github.com/sveltejs/svelte/issues/2499
• https://github.com/sveltejs/svelte/issues/2550
I'm going to apply the fix described in https://github.com/sveltejs/svelte/issues/2550#issuecomment-487301991 while waiting to fix it here and publish it.

closing since the site has been rewritten. please file at https://github.com/sveltejs/svelte.dev/issues/new if the issue persists


Downloading says "finished" but percent is "0" and file does not appear.

I just did a fresh reinstall of youtube-dl-gui on Windows using the Windows installer. When I try to download a youtube video, the status says "Finished" while Percent says "0%" and the file doesn't appear. I'm guessing that youtube-dl is throwing some kind of error, which is supressed by youtube-dl-gui. I recommend that you don't supress errors like this, so that the user has some chance to fix them.

Me too has a problem with this one: https://youtu.be/D2KRO0qRDhU

Yup. Me too. Says "Installation Windows 	run setup.exe." Nope. Installed vcredist_x86.exe, ffmpeg.exe. Still same. Tried portable. Still same. Does this work for anyone else?

wierd i didn't get this issue?
what OS was it?
did you install the latest updates of what this software need, i think you need Visual basic or python if you used the none windows installer

As I said, Windows (10) with Visual C++ Redistributable Package vcredist_x86. I added the installed directory to the path. Just running the installer is supposed to work.

Did you click the little cog wheel and then "Update"?
I feel that most of the "issues" here are just that. Outdated youtube-dl. I tried: https://youtu.be/D2KRO0qRDhU and it worked fine.

Did you click the little cog wheel and then "Update"?
I feel that most of the "issues" here are just that. Outdated youtube-dl. I tried: https://youtu.be/D2KRO0qRDhU and it worked fine.

That's unnecessarily hostile and inaccurate. I was very clear about the issue: "I recommend that you don't supress errors like this, so that the user has some chance to fix them." Clicking the little cog wheel is not going to stop youtube-dl-gui from supressing warnings and errors.

Guys guys. youtube-dl-gui hasn't been updated for years. There is a fork here, maybe you can try that.

OK, it works but Youtube-dl-gui is dated Jul 20 '18. Usually if there is a newer fork, it is linked to so people like us don't have to spend hours trying to get the old fork to work. Please update that.

I don't think that's possible after the fact.

Did you click the little cog wheel and then "Update"?
I tried that with but still didn't work.

"I don't think that's possible after the fact."
What do you mean "after the fact"?

Yeah, this has been happening on multiple Linux-based systems too, which all (to my knowledge) are using v0.4. I cannot download anything, no matter what the options are.  This used to work well. Sad.

If anyone is looking for an alternative, I decided to create one-click-youtube-dl after being frustrated with the GUI.

I haven't been able to use youtube-dl on the Mac. It simply says "This video is unavailable", although with specific options it seems to work fine.

I just downloaded the default quality video from a random video youtube-dl https://www.youtube.com/watch?v=jjdWckwmhB0.
OS X 10.8.5
youtube-dl version 2019.11.05
Python version 2.7.14 (CPython) - Darwin-12.6.0-x86_64-i386-64bit
exe versions: ffmpeg 3.4.1, ffprobe 3.4.1, phantomjs 2.1.1, rtmpdump 2.4

What do you mean "after the fact"?
How can Tartube be marked as a fork of youtube-dl-gui, so that you would have noticed it?

Usually there is a message that the tool is no longer updated with a link to a fork of it.

Usually there is a message that the tool is no longer updated with a link to a fork of it.

Well, that would be great, but I don't have access to the youtube-dl-gui pages and I can't add any such messages.

I see @MrS0m30n3 has had no activity since July 2018. Is there a way to contact him?

Did you click the little cog wheel and then "Update"?
I feel that most of the "issues" here are just that. Outdated youtube-dl. I tried: https://youtu.be/D2KRO0qRDhU and it worked fine.

That's unnecessarily hostile and inaccurate. I was very clear about the issue: "I recommend that you don't supress errors like this, so that the user has some chance to fix them." In other words, I reported an issue that youtube-dl-gui is supressing errors instead of showing them to the user. Clicking the little cog wheel is not going to stop youtube-dl-gui from supressing warnings and errors.

I don't see how this was hostile. It read my reply again and I really don't see it. It wasn't meant as a hostile thing either. You're right regarding that the errors should be shown though but that wasn't what I commented on and that didn't seem to be the main problem here. It doesn't work. The title focuses on it not working and so does most of your original post.
The reason why I said this is because I think the "Update" portion of the GUI is badly implemented and I thought a lot of people would overlook that. So I thought I would mention that. I do see however that I worded it badly but it doesn't come across to me as hostile at all. I wanted to help =/.
Either way, I do think errors should be shown yeah but it's unfortunately doubtful that that will happen because the author of this GUI hasn't updated it in quite awhile. Which is a shame. But yeah, sorry if I seemed hostile to you. Wasn't meant that way. Giving the user (useful) feedback if errors occur is a good thing.

Hey, no worries, and thanks for offering your help.

I found a free app that replaces this and the recommended app–WinX YouTube Downloader. It is very professional looking and works with ease-WinX YouTube Downloader Step-by-step User Guide. I was able to download 2.63 GB 1080p PBS shows instantly. It supports 300+ sites (a subset of the 1100+ youtube-dl Supported sites).
I was looking for a Windows app that made it easy for a non-technical person, my brother, to download videos. This is it. Highly recommended.
I tried the MacX version but it didn't work with PBS.

@PacOh I probably should've gone into more detail.
Although that appears to function in the expected manner, have you tried using the --format=mp3 flag with that command? That has failed numerous times. As far as I can tell, that's the right way to do it. Currently, I'm downloading them as M4A files and converting them to MP3 files, so any assistance would be most appreciated.
Many thanks,
Resynth.

In https://ytdl-org.github.io/youtube-dl/download.html ,

Windows exe requires Microsoft Visual C++ 2010 Redistributable Package (x86) and does not require Python that is already embedded into the binary.

So you should install  Microsoft Visual C++ 2010 Redistributable Package (x86) from https://www.microsoft.com/en-US/download/details.aspx?id=5555 .


Show total number of critter/construction on the card

What's your idea / feature request?
There is a tiny detail (mark on the trunk) on the physical card that is super useful especially when playing some rare critter/construction. It will be awesome to have this information in digital version.

Describe the solution you'd like
Not a good designer but something like this might work well

Thanks for the suggestion, Added this: http://everdell.herokuapp.com/test/ui

Thanks for the suggestion, Added this: http://everdell.herokuapp.com/test/ui

Thank you for adding this. So efficient, I am so impressed! 🎉

Thank you for adding this. So efficient, I am so impressed! 🎉


chore(docs): rsdoctor add the note for CssExtractRspackPlugin

Summary
[Docs] Rsdoctor add the note for CssExtractRspackPlugin

Checklist

[ ] Tests updated (or not required).
[x] Documentation updated (or not required).

Should we move this note to the Rsdoctor documentation?

Did you mean moving "Use Rsdoctor" to Rspack's website or to Rsdoctor's website?
I'll also add it to Rsdoctor's website. If you were referring to the Rspack website, I think this note should be placed near the loader profile documentation for Rsdoctor.

I mean move "this note" to the Rsdoctor website.
If Rsdoctor already provides this note, I don't recommend maintaining two copies of the same content. You can just link to the Rsdoctor documentation.
Considering that the Rsbuild documentation also has a section on Use Rsdoctor, it is unreasonable to maintain multiple copies of the same content.

CI broken:

CI broken:

@chenjiahan  this is be ok~


Adding Colour/texture to the Main Enironment

With the Main Environment grey box done, We are now adding more details to it, that being plants, bushes structures and their textures.
Oscar and Nate are drafting New models including the added Assets

The main map in the project was grey and me (Sam), Oscar and Nate worked on coloring the map.


Tabnine integration

What are you trying to do?
My name is Amir, I'm from Tabnine (www.tabnine.com)  engineering team, the maintainer of jupyterlab-tabnine.
Tabnine is a GPT-2 based auto completer tool which supports most major IDEs.
I got few requests from users who want to work with jupyterlab-lsp & jupyterlab-tabnine together.
jupyterlab-tabnine mainly implements CompletionHandler.ICompletionItemsConnector (https://github.com/codota/tabnine-jupyterlab/blob/master/src/connectors/TabnineConnector.ts)
What would be the best way to integrate both extensions?
Appreciate it
How is it done today, and what are the limits of current practice?

What is new in your approach and why do you think it will be successful?
Who cares? If you are successful, what difference will it make?
What are the risks?
How much will it cost?

How long will it take?
What are the mid-term and final “exams” to check for success?

Hi @bilucodota thanks for getting in touch! I would love to enable all extensions to combine their completions, and users to provide a way to specify in which order they want to see the completions. I was drafting a solution in #600 and @hbcarlos started upstreaming that work in #10523. Long story short we will prepare a new, saner API that should allow multiple extensions to hook into completer without interfering with one another and aim for JupyterLab 4.0 which should happen by the end of the year. How does that sound?

It would probably be good if you could take a look at the discussion in #600 and at the trimmed draft in https://github.com/jupyterlab/jupyterlab/pull/10523 to provide a feedback from your perspective.

@krassowski thanks for the warm welcome! sounds wonderful. I've commented on the thread. Guess we can continue there.

@bilucodota
Is the integration in tabnine extension complete w.r.t. the change in lsp api? Can users start using both lsp and tabnine together within JupyterLab 4.x?

Just for historical record, jupyterlab-lsp 5.0 targeting JupyterLab 4.0 implements ICompletionProvider API , which enables multiple completion provideres to co-exists. This is also supported in Jupyter Notebook v7+. We now just register on ICompletionProviderManager token, as should other extensions like tabnine going forward.
https://github.com/jupyter-lsp/jupyterlab-lsp/blob/3fb68dbab1ef52d1c3a65991755c2f24bf297bf7/packages/jupyterlab-lsp/src/features/completion/index.ts#L62-L63
Happy to finally close this as answered.


Add the godot_init convenience macro

This allows quicker initialization for common cases of NativeScript libraries.

bors r+


"The development server has disconnected"

Hi,
Thanks so much for the library, it's exactly what I was looking for :) However, I am having an issue with hot reloading.
Following the recommended instructions and adding re_path(r'^sockjs-node/(?P<path>.*)$', proxy_cra_requests), is giving me an error in the console followed by "The development server has disconnected:
WebSocket connection to 'ws://localhost:8000/sockjs-node' failed: Error during WebSocket handshake: Unexpected response code: 301

I've thought this may be due to the slash, so I varied it by making the slash optional in the regex, and now I get:
webpackHotDevClient.js:60 WebSocket connection to 'ws://localhost:8000/sockjs-node' failed: Error during WebSocket handshake: 'Upgrade' header is missing

Am wondering if this is a me issue, a docs issue, or a version incompatibility issue, if anyone's using this library and come across this?
Thanks

I'm using react-scripts 3.4.0 (this is a new project).
Screenshot attrached:

Thanks for linking that, I think your second "'Upgrade' header is missing" error response is the actual problem (which you already knew because you made the trailing slash optional to fix the 301 😛)
I poked around Create-React-App and its packages and I have an idea of what might be happening: it looks like in CRA v3.3 they update the hot reload dev server to use native Websockets instead of the sockjs-client library they were using in v3.2 and earlier. I suspect this is part of what's causing this breakage.
I also noticed that the django-proxy view I'm using to proxy this connection is stripping the Upgrade header when it proxies, which might explain the ultimate error message. This library says it's a "simple HTTP proxy" so perhaps it's no longer suitable for use in proxying ws:// URLs...
Work is nuts so I won't be able to work on this right away, but I'll definitely find time to test it out with CRA v3.4 and find a fix 😄

Thanks for looking into it :)

I think I've figured out how to fix this:

Create an .env file in the root of your Create-React-App folder with the following environment variable:

WDS_SOCKET_PORT=3000

Re-run npm start, then reload your Django view

The CRA liveserver should now connect to ws://localhost:3000 and re-enable live-reloading even from within Django:
https://giant.gfycat.com/CourteousEdibleHyrax.webm
(A screen recording showing live reloading working in Django using<EMAIL_ADDRESS>This means the following entry in urls.py is no longer required for<EMAIL_ADDRESS>and up:
re_path(r'^sockjs-node/?(?P<path>.*)$', proxy_cra_requests),

@cnorthwood If you can confirm this fixes the problem on your end then I'll just need to update the README accordingly!

This works excellently! Thanks so much for doing the digging and getting a straightforward answer :)

I've addressed this issue with some updates to README.md (see PR #8) highlighting the ability to use the above .env trick with current versions of create-react-app!


add go 1.23 image and license image to builder-base Dockerfile

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm

Shouldn't we define GOLANG_VERSION_123 in https://github.com/aws/eks-distro-build-tooling/blob/main/builder-base/versions.yaml?

Shouldn't we define GOLANG_VERSION_123 in https://github.com/aws/eks-distro-build-tooling/blob/main/builder-base/versions.yaml?

Thats included in the previous PR #1501.


Add workflow actions and update Makefile

Signed-off-by: Siyu Wang<EMAIL_ADDRESS>Add workflow actions and update Makefile.

/approve


Rate limiting?

Starting last weekend I was getting 503 responses from the online.net api when I used c14. Especially when creating new safes. I created a ticket at online.net because I thought I was a problem with api backend. I was then told that this was causes because of rate limiting and that I was polling the api to often. Since I haven't changed anything in c14 with regard to that I'm not sure what caused this.
Is there something we can do in the client to prevent this?
The internal ticket id is #510273 if that helps.
Traces essentially looked like this:
./c14 --debug create --name=test6
 DEBUG[0000] [GET]: https://api.online.net/api/v1/user/key/ssh
 DEBUG[0000] [Response]: [200] [{"uuid_ref":"226e8a7f-b267-421c-9b96-d71bcb595135","description":"gitlab","fingerprint":"11:93:ff:c6:6c:aa:ad:f7:50:eb:89:90:d0:22:95:b3"}]
 DEBUG[0000] [POST]: https://api.online.net/api/v1/storage/c14/safe
 DEBUG[0000] [Response]: [201] {"uuid_ref":"bec8e55b-cd5f-46b4-a512-dfa51c1a3b3a"}
 DEBUG[0000] [GET]: https://api.online.net/api/v1/storage/c14/safe/bec8e55b-cd5f-46b4-a512-dfa51c1a3b3a
 DEBUG[0001] [Response]: [200] {"$ref":"\/api\/v1\/storage\/c14\/safe\/bec8e55b-cd5f-46b4-a512-dfa51c1a3b3a","uuid_ref":"bec8e55b-cd5f-46b4-a512-dfa51c1a3b3a","name":"test6_safe","status":"active","description":""}
 DEBUG[0001] [POST]: https://api.online.net/api/v1/storage/c14/safe/bec8e55b-cd5f-46b4-a512-dfa51c1a3b3a/archive
 DEBUG[0001] [Response]: [503]
<title>503 Service Temporarily Unavailable</title>

503 Service Temporarily Unavailable

nginx

 FATAL[0001] /Users/kdummann/go/src/github.com/online-net/c14-cli/pkg/api/api.go:179: [503]
<title>503 Service Temporarily Unavailable</title>

503 Service Temporarily Unavailable

nginx

/Users/kdummann/go/src/github.com/online-net/c14-cli/pkg/api/methods.go:186: CreateArchive
/Users/kdummann/go/src/github.com/online-net/c14-cli/pkg/api/helpers.go:56: CreateSSHBucketFromScratch:CreateArchive
/Users/kdummann/go/src/github.com/online-net/c14-cli/pkg/commands/create.go:97: Run:CreateSSHBucketFromScratch

Hi @coolya,
I'm sorry you're facing this issue.
We've been having a lot of requests on the C14 API and had no other solutions than massively rate limiting requests to the API to preserve all the other services.
I'm personally handling with my team scaling issues.
We've been able to improve the rate limiting based on the Authorization token.
We're adding API servers and should restore a full service early next week.

with the last commit and new servers in the API pool, you should no longer get rate limited.

Thanks a lot for you work on that! The issues seems to be gone.

One suggestion for improvement: If the 503 would include a more verbose error message that the request was rejected because of rate limiting and/or that this is temporary because of the high load would be very useful.

We've changed the HTTP return code to 429 and added a message about the rate limiting. Also added a few headers giving the number of request available, left and till when.

👍


Python 3 support

I got a start on this but stopped short of making oauth work since I don't currently need it. Mostly making relative imports more explicit and dealing with modules being rearranged (eg parse_qsl is now in urllib.parse). The work's all in a81422e9.

Hey, thanks a lot. This is something we'll be working on soon, so we really appreciate it.

I've sent a PR (#13) which deals with Python3 compatibility: ImportAPI seems to work ok, but there is still an issue related to OAuth.

Already merged