Add support for search_path

Hi @JoshRosen,
Added support to specify the Redshift schema and do not default to PUBLIC for non-schema-qualified table names.
Fixes: https://github.com/databricks/spark-redshift/issues/295

ping! Highly appreciate if you can take a look soon.

Any progress on this?
We are using a different schema per environment but would like to keep the sql scripts not repeating the schema (or have them parameterized). This PR would quite probably solve our problem, so would be neat to get this merge&released :)

@JoshRosen can you please take a look.

We created a work around for this by creating Redshift Users with same name as the schema name, since Redshift defaults '$user' for search_path
I think it would still be valuable to merge this.

👍

have similar use case, anything I do to help this pull request ?

Any progress on this?


ATProto Quote Posts => AP: RE link has reverted to bsky.brid.gy/convert URL

This was working correctly before, but this afternoon I started noticing the links were back to the bsky.brid.gy/convert URL which just leads to some JSON.
Sample on Mastodon: https://darkfriend.social/<EMAIL_ADDRESS>

Argh, thanks for reporting! This may be due to https://github.com/snarfed/bridgy-fed/issues/1047#issuecomment-2190063293. I'll look.

Ok, should be fixed now. Thanks again for filing the issue!

Had to roll this back, will look again tomorrow.

OK, I've updated and re-deployed, this should be fixed. Let me know if you see bad quote links on new posts!


fix: getQuote should work with ZETA

Looks like now it's calculating output correctly, even if one of the tokens is ZETA.

@coderabbitai is @fadeev doing too many commits and should he chill a bit?


Add a Codacy badge to README.md

Button didn't do what I thought it did, manually added button to README


add unready patch

@Segelzwerg
Hi, I am a little confused on why the codecov checks are saying they failed, because when I see the detail, it is showing a warning about certain lines of code, but not displaying any information on why it's a warning. Is there something wrong with my exceptions?
Thanks

@Segelzwerg
Hi, I am a little confused on why the codecov checks are saying they failed, because when I see the detail, it is showing a warning about certain lines of code, but not displaying any information on why it's a warning. Is there something wrong with my exceptions?
Thanks

Codecov is a tool to display coverage of tests. A warning means that no test has covered that line.

What would I need to do to make sure the tests covers that line and the check passes?
Thanks

@Segelzwerg
Would that mean I would have to change the code so that a test can cover that line? If so, how does that work?
Thanks

@Segelzwerg
Would that mean I would have to change the code so that a test can cover that line? If so, how does that work?
Thanks

@abant07 you don't need to change the code in order to fullfil the codecov requirements, but to write tests that cover the changes/additions you made.

@Segelzwerg
Sorry if I am asking too many questions, but where would I need to write those tests and how would I write those tests?
Thanks

@Segelzwerg
Where should I put the test for the codecov requirements? Do I need to put it in the code, or do I put it in the tests folder? Also to write a test, should I look at the tests folder to get ideas on how to write a test for a specific line?
Thanks

@Segelzwerg
Where should I put the test for the codecov requirements? Do I need to put it in the code, or do I put it in the tests folder? Also to write a test, should I look at the tests folder to get ideas on how to write a test for a specific line?
Thanks

@abant07 sorry for getting back so late, but have a little spare time these days.
I would suggest you have a look at the existing tests and see how these works and maybe also do some research on python's unittest.
Kind regards

duplicate of #219


"Dumbrăveni","Alexandru Vlahuță","Cândești","Dragosloveni" are listed in the drop down menu

"Dumbrăveni","Alexandru Vlahuță","Cândești","Dragosloveni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and running. After every step delete the text you typed from the search bar.

Step 1
Write in the search bar "Dumbrăveni"

Expected results
The location appears in the drop-down menu as "Sat Dumbrăveni  , VRANCEA (Dumbrăveni)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Alexandru Vlahuță"

Expected results
The location appears in the drop-down menu as "Sat Alexandru Vlahuță  , VRANCEA (Dumbrăveni)"

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Dragosloveni"

Expected results
The location appears in the drop-down menu as "Sat Dragosloveni  , VRANCEA (Dumbrăveni)"


Update about-keys-secrets-and-certificates.md

Added spaces between words that were missing in the sentence and italicized the "exp" keyword in enabled description under the Certificate Attributes and Tags section.

@lleonard-msft - Would you review the proposed changes? Thank you.

#sign-off

@lleonard-msft: I'm sorry - only the author of this article, @lleonard-msft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@lleonard-msft Sorry for this issue... apparently the file's meta data had a space before your alias which caused this issue. We have fixed the same.

@seshenoy - Thanks so much for looking into this.  I really appreciate the quick turnaround!

@asadmansr : Thanks for your contribution! The author, @lleonard-msft, has been notified to review your proposed change.


NPE is thrown when setting log level lower than FINER

Currently I can't set the log level lower than FINER because this library throws a NPE. The cause of this problem is a missing pair of parentheses in https://github.com/googleapis/gax-java/blob/master/gax/src/main/java/com/google/api/gax/retrying/BasicRetryingFuture.java#L179.
This is the current snippet:
"enclosingMethod: " + callable.getClass().getEnclosingMethod() != null
  ? callable.getClass().getEnclosingMethod().getName()
  : ""

Which is equivalent to:
("enclosingMethod: " + callable.getClass().getEnclosingMethod()) != null
  ? callable.getClass().getEnclosingMethod().getName()
  : ""

This results in calling callable.getClass().getEnclosingMethod().getName() even if callable.getClass().getEnclosingMethod() is null.
For a potential fix, see https://github.com/googleapis/gax-java/pull/1198.

Can be closed as https://github.com/googleapis/gax-java/pull/1198 was merged.


Changes needed to compile in VS2015 w/ latest Xamarin

Addresses my issue in #239 - This is a fresh install of VS2015 with the latest Xamarin bits. Changes the hint path to .... from C:\WORK\REPOS in Tests.Generic, updates the Android framework to 6, re-install PCLStorage in iOS, updated TestFramework version 14 for WinRT

I'm sad that AppVeyor fails - is it because of multiple slns? Doesn't yaml specify the right sln?

Fix for #242 in this PR. After removing the two bad references in OSS, everything is compiling in appveyor but it dies because of failed tests due to
System.DllNotFoundException : Unable to load DLL 'sqlite3': The specified module could not be found. (Exception from HRESULT: 0x8007007E)

Ok, I see the problem here, all the Generic Tests fail because sqlite3 is not present in the test folder. All the Win32 pass. https://ci.appveyor.com/project/oysteinkrog/sqlite-net-pcl/build/1.0.217/tests
Solutions:

remove Generic Tests from OSS
ignore Generic Tests
add sqlite3.dll

Never worked this hard for a PR before. ;P  Ncrunch successfully runs the Generic Tests (minus the ones that don't work like Stress) when sqlite3.dll is added. However, appveyor now complains that
System.BadImageFormatException : An attempt was made to load a program with an incorrect format. (Exception from HRESULT: 0x8007000B)

Maybe Generic wants x64 version?

Swapping in the x64 sqlite3.dll worked - it is now just failing tests which are ignored in Win32. Now just to figure out how to ignore those tests.
SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes
SQLite.Net.Tests.Generic.dll 20 ms
SQLite.Net.Tests.AsyncTests.StressAsync
SQLite.Net.Tests.Win32.dll 8 sec 23 ms
SQLite.Net.Tests.AsyncTests.StressAsync
SQLite.Net.Tests.Generic.dll 9 sec 46 ms
SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes
SQLite.Net.Tests.Win32.dll 63 ms

Not sure why but the Assert.Ignore is not being tripped in the Generic Tests, although it is in the Windows one.
Tests run: 332, Errors: 0, Failures: 4, Inconclusive: 0, Time: 39.9826234 seconds
  Not run: 4, Invalid: 0, Ignored: 4, Skipped: 0

Errors and Failures:
1) Test Failure : SQLite.Net.Tests.AsyncTests.StressAsync
     Expected: 0
  But was:  18

at SQLite.Net.Tests.AsyncTests.<StressAsync>d__32.MoveNext() in c:\projects\sqlite-net-pcl\tests\AsyncTests.cs:line 217
at NUnit.Framework.AsyncInvocationRegion.AsyncTaskInvocationRegion.WaitForPendingOperationsToComplete(Object invocationResult)
at NUnit.Core.NUnitAsyncTestMethod.RunTestMethod()

2) Test Failure : SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes
     Expected: collection containing <System.DateTimeOffset>
  But was:  <empty>

at SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes() in c:\projects\sqlite-net-pcl\tests\BlobSerializationTest.cs:line 307

3) Test Failure : SQLite.Net.Tests.AsyncTests.StressAsync
     Expected: 0
  But was:  17

at SQLite.Net.Tests.AsyncTests.<StressAsync>d__32.MoveNext() in c:\projects\sqlite-net-pcl\tests\AsyncTests.cs:line 217
at NUnit.Framework.AsyncInvocationRegion.AsyncTaskInvocationRegion.WaitForPendingOperationsToComplete(Object invocationResult)
at NUnit.Core.NUnitAsyncTestMethod.RunTestMethod()

4) Test Failure : SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes
     Expected: collection containing <System.DateTimeOffset>
  But was:  <empty>

at SQLite.Net.Tests.BlobDelegateTest.CallsOnUnsupportedTypes() in c:\projects\sqlite-net-pcl\tests\BlobSerializationTest.cs:line 307

Tests Not Run:
1) Ignored : SQLite.Net.Tests.BlobDelegateTest.BlobSerializationTest.VerifyTableCreationSucceedsWithSupportedSerializer
   Serialize does not support this data type
   at NUnit.Framework.Assert.Ignore(String message, Object[] args)
   at SQLite.Net.Tests.BlobSerializationTest.VerifyTableCreationSucceedsWithSupportedSerializer() in c:\projects\sqlite-net-pcl\tests\BlobSerializationTest.cs:line 160

2) Ignored : SQLite.Net.Tests.ConnectionTrackingTest.CreateThem
   This test class/file was not included in the original project and is broken
3) Ignored : SQLite.Net.Tests.BlobDelegateTest.BlobSerializationTest.VerifyTableCreationSucceedsWithSupportedSerializer
   Serialize does not support this data type
   at NUnit.Framework.Assert.Ignore(String message, Object[] args)
   at SQLite.Net.Tests.BlobSerializationTest.VerifyTableCreationSucceedsWithSupportedSerializer() in c:\projects\sqlite-net-pcl\tests\BlobSerializationTest.cs:line 160

4) Ignored : SQLite.Net.Tests.ConnectionTrackingTest.CreateThem
   This test class/file was not included in the original project and is broken

Command exited with code 4

In https://github.com/oysteinkrog/SQLite.Net-PCL/issues/196 the StressAsync and CallsOnUnsupportedTypes is said to be buggy.

oops, hold on that last commit, newtonsoft didn't get picked up

Only the stress test fails now, #243 is resolved

@oysteinkrog  let me know if you need these changes split out into separate PRs, don't think I can resolve #244 now, but appveyor is working properly now

The appveyor build has never actually worked :/
Thank you so much for looking at it.
I will look more closely at your changes and merge later this week (probably on Friday as I will have some free time).

it is in my notes, but the major appveyor problem was that the yml file had
to be lower case - hard to diagnose!
On Wed, Nov 11, 2015 at 2:28 PM, Øystein Krog<EMAIL_ADDRESS>wrote:

The appveyor build has never actually worked :/
Thank you so much for looking at it.
I will look more closely at your changes and merge later this week
(probably on Friday as I will have some free time).
—
Reply to this email directly or view it on GitHub
https://github.com/oysteinkrog/SQLite.Net-PCL/pull/241#issuecomment-155929660
.


LDR Tutorial & Manage LDR pages

Preview:
Setup Guide
Manage LDR Ref
This PR is 1 of 4(?) for LDR docs. It contains:

Setup Guide (tutorial) for setting up a unidirectional/bidirectional LDR stream.
Manage LDR for understanding conflict resolution, working with the DLQ, coordinating schema changes, and job management. Also, a small section on schema validation (this should maybe move into a known limitation section when there is more content).

Fixes DOC-10483, DOC-11073, DOC-10859
Reviewers: You'll see a lot of comments for follow-up PRs to connect pages with links and such.
Content pages to come:
LDR Overview
Monitoring
CREATE/SHOW sql ref
Technical Overview
LDR Metrics Dashboard docs

This lgtm from an structure perspective!

@alicia-l2 I worked through your feedback. PTAL. I have also added the known limitations, and added more detail in places where the support was unlikely to be a known limitation.
@msbutler Would appreciate your review on these two starting pages.
Thanks both!

TFTRs! Merging this, more updates to LDR docs to come shortly.


feat(dep): Update ghastoolkit to fix DepGraph issues

This pull request includes a small change to the Pipfile. The change updates the version of the ghastoolkit package from 0.14.0 to 0.14.1.

Fix for #119 & #140


full coverage is ensured! (this took a lot longer than expected...)

Edits are good and able to be merged without further edits.


Security > Block common exploits (cfr Nginx Proxy Manager regex)

Feature request
Security > "Block common exploits" checkbox
Feature description
When selected (under Security?) a file like block-exploits.conf is added into the config
ref: https://github.com/NginxProxyManager/nginx-proxy-manager/blob/bb0f4bfa626bfa30e3ad2fa31b5d759c9de98559/docker/rootfs/etc/nginx/conf.d/include/block-exploits.conf
How the feature is useful
Seems like a quick win? - more easy to understand than things like modsecurity

Sure thing :)

Closing this out based on the discussion over in #388.


Clarify docs regarding default Client.new behavior

No host given defaults to connecting to localhost.

Thanks @bmurphy1!


Clean up sprint 2: Complete test suite

Implement MockBuilders to simplify mocking objects
Add and match some test cases with current code for ControllerTest
Testing part should be complete after this PR

@paul891112 or @Patertuck could you please create an issue for this under the clean up sprint and link it her and on the client. Thanks! PS dont forget to write it into the merge commit message.

@paul891112 or @Patertuck could you please create an issue for this under the clean up sprint and link it her and on the client. Thanks! PS dont forget to write it into the merge commit message.

On it

194: ...


Making context available in schema parts resolvers

Hi, currently the context is not available when resolving parts of the schema.
This is sometimes needed. Actually in my case quite often, but maybe I'm doing something wrong.
Example:
type Schedule {
    _id: String
    name: String
    achievements: [Achievement]
  }

type Achievement {
 ...
}

In this case, when I resolve schedules achievements I want to resolve them for the currently logged in user. That is, I need userId to obtain achievements for the given schedule, this ID is available in the context.
Is this actually the place to do this? If not, what would be your recommendation?

Yeah the context is passed to every resolver as the third argument - are you able to access it from there?

@stubailo ... not resolvers of queries, but resolvers of types. There I could not access it.

@tomitrescak What do you mean with resolvers of types? Only fields have resolvers.

Explained by @stubailo ... the problem was that I was testing everything in the GraphiQL environment and it is not picking up the req.headers.authorization header, this not initialising the context. I asked the question about it on Slack as well. I can confirm that all resolvers have context as their third parameter.


Update DevKit build "bootstrap" to ensure all tooling comes from bootstrap

All the necessary DevKit tooling (ld, as, ar, ranlib,...) needs explicitly setting to the bootstrap to ensure the final DevKit is bootstrapped correctly.

@sxa for review please

I think this is ok but to be clear what was the reason for changing from the environment variables for CC/CXX to putting them on the make command line? Was that significant? Is there any potential problem if, say, the compiler calls ld internally that might result in it not using the one from the devkit in this situation? It may be nothing, but just a though when I saw the changing in where it is defined.

@sxa  Which do you think is better, export or make variable ? or should we just do both...?

I think this is ok but to be clear what was the reason for changing from the environment variables for CC/CXX to putting them on the make command line? Was that significant? Is there any potential problem if, say, the compiler calls ld internally that might result in it not using the one from the devkit in this situation? It may be nothing, but just a though when I saw the changing in where it is defined.

@sxa Which do you think is better, export or make variable ? or should we just do both...?

Honestly I'm pretty confident that once you've set the PATH you're fine if the tools aren't suffixed (i.e. they don't have ld-2.39 instead of just ld). I've generally used the environment variables rather than parameters to make (which is why I asked the question originally rather than me having deep insight here!) but that's what you had originally (and I guess you saw some issues with it?) From a quick test it seems to manage to find them in the PATH even if CC etc. are set to gibberish values ...

I think this is ok but to be clear what was the reason for changing from the environment variables for CC/CXX to putting them on the make command line? Was that significant? Is there any potential problem if, say, the compiler calls ld internally that might result in it not using the one from the devkit in this situation? It may be nothing, but just a though when I saw the changing in where it is defined.

@sxa Which do you think is better, export or make variable ? or should we just do both...?

Honestly I'm pretty confident that once you've set the PATH you're fine if the tools aren't suffixed (i.e. they don't have ld-2.39 instead of just ld). I've generally used the environment variables rather than parameters to make (which is why I asked the question originally rather than me having deep insight here!) but that's what you had originally (and I guess you saw some issues with it?) From a quick test it seems to manage to find them in the PATH even if CC etc. are set to gibberish values ...

The issues we had were from a 3rd party (in this case Red Hat) independent DevKit build, which was different, and although we've tracked down the exact difference, I want to try and do "belt & braces" on all the variables, i'm thinking maybe changing to PATH and export... what do you think?


Tweak gemspec template when using Ruby 2.x

What was the end-user problem that led to this PR?
There are unnecessary magic comment for currently maintained Ruby 2.2, 2.3, and 2.4.
What was your diagnosis of the problem?
The default script encoding from Ruby 2.0 is UTF-8.
https://bugs.ruby-lang.org/issues/6679
Ruby 1.9 is EOL, so I think that there is not much Gem to start developed using it.
What is your fix for the problem, implemented in this PR?
This PR removes magic comment when starting Gem development (i.e. bundle gem) with Ruby 2.0 or higher version.

@bundlerbot r+


Sufficiently deeply nested code causes stack overflow error

Describe the bug
Rust code (I think you could probably reproduce this without this and by just using the input code):
pub fn parse_swc_ast(file_path: &str, file_text: &str) {
    let handler = Handler::with_emitter(false, false, Box::new(EmptyEmitter {}));
    let session = Session { handler: &handler };

    let file_bytes = file_text.as_bytes().to_vec();
    let source_file = SourceFile::new(
        FileName::Custom(file_path.into()),
        false,
        FileName::Custom(file_path.into()),
        file_text.into(),
        BytePos(0),
    );

    let comments: Comments = Default::default();
    let (module, tokens) = {
        let mut ts_config: swc_ecma_parser::TsConfig = Default::default();
        ts_config.tsx = should_parse_as_jsx(file_path);
        ts_config.dynamic_import = true;
        ts_config.decorators = true;
        let lexer = Lexer::new(
            session,
            Syntax::Typescript(ts_config),
            JscTarget::Es2019,
            SourceFileInput::from(&source_file),
            Some(&comments)
        );
        let lexer = Capturing::new(lexer);
        let mut parser = Parser::new_from(session, lexer);
        let parse_module_result = parser.parse_module();
        let tokens = parser.input().take();

        match parse_module_result {
            Err(mut error) => {
                // mark the diagnostic as being handled (otherwise it will panic in its drop)
                error.cancel();
                // return the formatted diagnostic string
                Err(format_diagnostic(&error, file_text))
            },
            Ok(module) => Ok((module, tokens))
        }
    };
}

pub struct EmptyEmitter {
}

impl Emitter for EmptyEmitter {
    fn emit(&mut self, _: &DiagnosticBuilder<'_>) {
        // for now, we don't care about diagnostics so do nothing
    }

    fn should_show_explain(&self) -> bool {
        false
    }
}

Input code
I'm running this in a test case:
module.exports = function test() {
  if (true) {
    if (true) {
      while (test) {
        if (test) {
          if (test) {
            if (test) {
              if (test) {
                while (test) {
                  if (test) {
                    if (asdf) {
                      if (asdf) {
                        test += ' || ' + (test) + ' === null || ' + (test) + ' === \'\' ';
                      }
                    }
                  }
                }
              }
            }
          }
        }
      }
    }
  }
}

Error is:
thread 'name' has overflowed its stack

Version
swc_ecma_parser: 0.21.3

The test success with increated RUST_MIN_STACK or on a release build.
On linux, the default stack size is 1MB and it's not sufficient in debug mode.
Also, you can set the stack size for a thread except the main thread.
thread::Builder::new()
                  .stack_size(8 * 1024 * 1024)

Can you increase the stack size? (It's required only on debug mode)

@kdy1 ah awesome! I did not know all this. Yup, I was running in debug mode 😅
I'm going to close this.

@kdy1 running the test suite on windows now throws this. What can we do so that when people run the test suite they don't have to change any configuration?


Add some nonlinearities stax.Elu, stax.LeakyRelu

As title.

Your productivity is going to make the rest of us look bad...


Emergent Object Detection/Classification Model.v1

Closes #12
Model to detect / classify emergent objects & pretrained weights for the model.

Good first issue for someone to take up the fixes for this PR

I worked out how to use the model and the output and added some code demonstrating its use. Docstrings and proper typehinting still need to be added, but the functionality should be pretty much all there at this point.

Opened new PR #96 because something was wrong with this branch that caused a split.


Mitigate Restarting the Server for Reading Updated Config File

Currently, if the server needs to be restarted in order to read any changes in an updated forseti_conf.yaml file.  It would be nice if the server does not have to be restarted, i.e. the server will read in the conf file when it does an operation.

Reloading the configuration file live is very dangerous. If someone is editing the file on the system (even though we expect people to edit and push to GCS, this is not always going to be the case) then the editor may save the file in an intermediate state that is not valid, while they are editing. This will immediately cause the system to crash if it attempts to reload the configuration.
That is why most Linux daemons require a signal from the user to restart, either a SIGHUP, or running a command to force the reload.
I recommend that the system supports sighup and we add a grpc method so the cli can force a reload as well.
Normal operation should be something like:

Download a new configuration from GCS.
Validate the configuration.
If validation passes, send HUP to the service to reload.


Fix potential buffer overflow issue

I found this issue while implementing a small resampler based on this project.
Description
AudioRendererand AudioOverlayFilteruse:
val estimatedTargetSampleCount = ceil(sourceSampleCount * samplingRatio).toInt()
 val targetBufferCapacity = estimatedTargetSampleCount * channelCount * BYTES_PER_SAMPLE

to calculate the size of the target buffer.
In case this estimation is too small (happens on big differences in samplerate) this results in 2 issues.
Issue
processAudioFrame in audio-processor.cpp doesn't know about the size, resulting in memory access vialoation.
Fix
Drop audio when target buffer capacity is reached.
Issue
targetFrame.buffer.limit(targetBufferSize) throws an IllegalArgumentException.
Fix
Take min min(targetBufferSize, targetFrame.buffer.capacity())

@izzytwosheds i fixed a small issue you could review again.


Implement CORS

Not sure if you had another idea in mind. I just need to be able to set withCredentials.

Looks good. Could you update JSDoc about cors flag and add JSDoc about [settings.cors]?

Done :smile:

I can add tests later on too.

Merged in #8.


[SITES] patchwork.kernel.org

First please check that it is really an issue with the library, and not some special case of website:
[x] There is no paywall
[x] You do not have to be logged in to see the articles
[x] You tried using a common browser user agent in your configuration / call
[x] The website is not in the list of well known problematic sites
Your report as follows:
Website that does not parse correctly:

https://patchwork.kernel.org/

Some sample urls that I have tried

<EMAIL_ADDRESS>
The exact code i used to test this articles/website

  article = Article(url, fetch_images=False, follow_meta_refresh=True)
  article.download()
  article.parse()

** What parts of the article are missing / not parsed correctly **
[ ] Title
[x] Text Content
[ ] Publication Date
[ ] Authors
[ ] Images
[ ] Movies
[ ] Other, please specify:
Other information, remarks, messages, etc:

Doesn't really look like a newspaper, is it within the project scope to support this kind of websites?

The problem is that all replies are in divs with "comment" class
The module removes all these since in most news sites these are down in the comments section, and thus removed from the article body
These classes are defined here https://github.com/AndyTheFactory/newspaper4k/blob/c5e4170918a6d1e99cb1bab6fd188ee8ed5a2afa/newspaper/cleaners.py#L25-L38


Ryzen Master fails to install

Brief description of your issue

when you try to install ryzen master with winget no error appears and everything seems to be fine but it doesn't appear anywhere. i can't even find a way to run it. I can only uninstall it from the configuration panel.
Steps to reproduce

try to install it winget install --id=AMD.RyzenMaster  -e
Expected behavior
the app should create a shortcut on the desktop, appear in the list of installed apps in the start menu, or have an executable in its installation folder and none of this happens.

Actual behavior
when you try to install ryzen master with winget no error appears and everything seems to be fine but it doesn't appear anywhere. i can't even find a way to run it. I can only uninstall it from the configuration panel.
Environment
[winget --info]
Windows Package Manager v1.0.11692
Windows: Windows.Desktop v10.0.19043.1165
Paquete: Microsoft.DesktopAppInstaller v1.12.11692.0

this seems isn't the installation failed, just no proper switch given. If you have access to the script to build the installer, you may find a proper switch to add shortcut, which you can add to current manifest.

I did a bit of testing.
Looks like the silent installation and the interactive installation behaves differently.
Silent Installation:

Extracts an MSI installer to C:\AMD\RyzenMasterExtract\
Setup files are extracted to C:\Qt_Dependancies\

To actually get AMD Ryzen Master to install properly in silent mode, you will have to run C:\Qt_Dependancies\Setup.exe manually because the installer doesn't do it:

Interactive Installation:

Extracts an MSI installer to C:\AMD\RyzenMasterExtract
Installation happens in C:\Program Files\AMD\RyzenMaster
Launching AMD Ryzen Master from C:\Program Files\AMD\RyzenMaster\bin\AMD Ryzen Master.exe (which is the shortcut created on the Desktop)

ProductCode is different as well:

Silent Installation = {9AD90625-3883-4F0F-9991-DD32B8092371}
Interactive Installation = AMD Ryzen Master

For now, I'll remove the package until this is either fixed on AMD's side or until interactive-only installers are supported in the package repository to make sure it doesn't try to corrupt someone's installation.


PR to pass ARI config path as env var

PR to pass ARI config path as env var, where user has the option to pass ARI config path as ENV. variable. If not ARI config path would be defaulted to path ~/.ari/config.

cc. @hirokuni-kitahara @yuji-watanabe-jp


Make cache key timestamping optional.

Prompted by #117.
For Github repositories with many different actions and concurrent PRs, ccache-action's timestamps can quickly fill up the cache and evict other cached entries. This PR adds an option to disable timestamping.
I am not familiar with either Javascript or Github Actions, so suggestions on how to improve this are very much welcome.

We have the same problem! Having this option might help us a lot @hendrikmuhs

Looks good to me, instead of useTimestamp I suggest appendTimestamp.

Thanks! I can merge and fix the js files myself. The last dependabot updates broke the build anyway.

Nice!! thanks a lot @hendrikmuhs, @fruffy

I was eager to try this as I also don't like having one cache for each of the runs, but it seems that this doesn't actually work, as saving the cache always fails when using the new option and the cache already exists.
According to this comment, an existing cache can't be overwritten, and if this is correct, then the new option is not very useful. Or am I missing something here?

Part of the motivation for this PR was to get some control over the key signature. You can still define a key that is flexible enough to avoid that issue (using PR/commit IDs). This is what we have been doing.
But yes, it is an issue right now. Hope they will fix that bug at some point.

Thanks for the clarification, it wasn't obvious that using this option required making the key unique in some other way. I think it would be nice to mention this in the README.


Filter works only on 1st page of search results

Yes. That is by design. Else, it'll need to do a constant poll on the page.


📝 Update includes in docs/en/docs/tutorial/background-tasks.md

related to https://github.com/fastapi/fastapi/issues/12554
fixed background page syntax

Thanks for your help! @FarhanAliRaza :sparkles:


Problem with bitbucket.org

When cloning public bitbucket repo https://bitbucket.org/larshp/test.git it gives error "branch not found"
Something is wrong when parsing the branch list,

result after doing SPLIT iv_data AT zif_abapgit_definitions=>c_newline INTO TABLE lt_result. in ZCL_ABAPGIT_GIT_BRANCH_LIST
bitbucket:
001d# service=git-upload-pack000000d2b5d5f1f84ebcaeb8a299edd14c959518e9d81bb5 HEAD#multi_ack ...snip
003fb5d5f1f84ebcaeb8a299edd14c959518e9d81bb5 refs/heads/master
0000

github
001e# service=git-upload-pack
000001080e6fe6b311f789ccbac6c5122702d4f48a4f6bda HEAD#multi_ack ...snip
003f0e6fe6b311f789ccbac6c5122702d4f48a4f6bda refs/heads/master

note that bitbucket does not send a newline

https://github.com/git/git/blob/master/Documentation/technical/pack-protocol.txt
"unless otherwise noted the usual pkt-line LF rules apply: the sender SHOULD include a LF, but the receiver MUST NOT complain if it is not present"


Feature request: Integrate Actionlint

Actionlint is a static checker for GitHub Actions workflows and it currently has no VS Code extension. It would make an excellent addition to yours.

It looks like there is an extension now: https://marketplace.visualstudio.com/items?itemName=arahata.linter-actionlint

@laughedelic Yes, but it depends on the Linter extension which I have found to be slow and buggy.

I don't think we can integrate actionlint in the short-term. It has a lot of great features and I hope we can support some of them in the future, but closing this for now.


add gRPC-haskell benchmark

Summary of changes
Bench gRPC-haskell
BTW: This Haskell implement server may have some memory leaks.
Tested on my local machine: AMD 5900X, 64G RAM, with the following options:
GRPC_BENCHMARK_DURATION=120s
GRPC_BENCHMARK_WARMUP=30s
GRPC_CLIENT_CONNECTIONS=50
GRPC_CLIENT_CONCURRENCY=1000
GRPC_CLIENT_QPS=0
GRPC_CLIENT_CPUS=9
GRPC_REQUEST_SCENARIO=complex_proto
GRPC_SERVER_RAM=1g
GRPC_SERVER_CPUS=1

It requires GRPC_SERVER_RAM to be at least 1g. If I run with the default GRPC_SERVER_RAM (which is 512m), the container will be killed because of running out of memory.

@4eUeP thanks for the PR! In general, a memory leak doesn't sound good. There is already a Haskell implementation that was disabled because (as far as I remember) of some dependency issues. Maybe you could look at it and try to re-enable it? I think it didn't have any leaks. What do you think?

@LesnyRumcajs Hi, AFAIK the mu-haskell (which is already in this repo but currently disabled) is a bit slower than grpc-haskell. Anyway, re-enable it sounds useful. I'll try to look at it :)

@4eUeP makes sense. I'm not a Haskell expert, can we somehow get rid of the leak? In general, the implementations here should serve as an example, it'd be bad if someone copies them and gets into trouble with memory leaks.

@LesnyRumcajs Hi, I think the leak is from the library itself, so we can't get rid of it here easily. However, I'll open an issue (or pr) to the upstream grpc-haskell repo and try to fix it if I have time.

@LesnyRumcajs Hi, I think the leak is from the library itself, so we can't get rid of it here easily. However, I'll open an issue (or pr) to the upstream grpc-haskell repo and try to fix it if I have time.

Sounds good to me, thanks! That'd really help the Haskell ecosystem in this field. In general, if a library has leaks on a rather simplistic RPC call then it's not really benchmark nor production ready.

@4eUeP Do you plan to play with this some more, or should I close this PR for now?

@LesnyRumcajs Sorry for the delay. Since the fix PR(https://github.com/awakesecurity/gRPC-haskell/pull/140) hasn't been merged, let's go ahead and close this now.

@4eUeP Thanks. Something seems to be moving in the blocking issue, so hopefully, it will get resolved eventually.


Add FreeBSD support to omnibus packaging

Added a case for FreeBSD detection and fixed up a few things that were assuming Linux.

LGTM


How do I replace using python -m with uv

To download a language for spacy, normally I would run  python -m spacy download, I  can't seem to figure out the uv way of doing this.
Version: uv 0.4.1 (823f23e22 2024-08-30)
as a package
I've tried adding it to my workspace and I get the error when i try to use uv run:
$ uv run --package spacy download es_core_news_md
error: Package `spacy` not found in workspace

from my pyproject.toml:
dependencies = [
    ....,
    "spacy>=3.7.6",
]

[tool.uv.sources]
spacy = { workspace = true }

Running uv add spacy gives me this error:
  Caused by: Failed to parse entry for: `spacy`
  Caused by: Package is not included as workspace package in `tool.uv.workspace`

as a tool
$ uv tool run spacy download es_core_news_md
...\uv\tools\spacy\Scripts\python.exe: No module named pip

Good question. { workspace = true } is intended for other local projects in your workspace (like, if you had a subdirectory that was a standalone Python project, and you wanted to import it from the root of your project).
You have a few different options...

If you're outside of a project, and are just trying to run this as a one-off command, you could uv run --with spacy -- spacy download es_core_news_md. The --with spacy says "Download and install Spacy before running the command", and then the contents after the -- are the actual command (spacy download es_core_news_md).

❯  uv run --with spacy spacy download es_core_news_md
Collecting es-core-news-md==3.7.0
  Downloading https://github.com/explosion/spacy-models/releases/download/es_core_news_md-3.7.0/es_core_news_md-3.7.0-py3-none-any.whl (42.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 42.3/42.3 MB 42.5 MB/s eta 0:00:00
Requirement already satisfied: spacy<3.8.0,>=3.7.0 in /Users/crmarsh/.cache/uv/archive-v0/g5MPE_L8O18cSfqHr7c9v/lib/python3.12/site-packages (from es-core-news-md==3.7.0) (3.7.6)
Requirement already satisfied: spacy-legacy<3.1.0,>=3.0.11 in /Users/crmarsh/.cache/uv/archive-v0/g5MPE_L8O18cSfqHr7c9v/lib/python3.12/site-packages (from spacy<3.8.0,>=3.7.0->es-core-news-md==3.7.0) (3.0.12)
Requirement already satisfied: spacy-loggers<2.0.0,>=1.0.0 in /Users/crmarsh/.cache/uv/archive-v0/g5MPE_L8O18cSfqHr7c9v/lib/python3.12/site-packages (from spacy<3.8.0,>=3.7.0->es-core-news-md==3.7.0) (1.0.5)
Requirement already satisfied: murmurhash<1.1.0,>=0.28.0 in /Users/crmarsh/.cache/uv/archive-v0/g5MPE_L8O18cSfqHr7c9v/lib/python3.12/site-packages (from spacy<3.8.0,>=3.7.0->es-core-news-md==3.7.0) (1.0.10)
Requirement already satisfied: cymem<2.1.0,>=2.0.2 in /Users/crmarsh/.cache/uv/archive-v0/g5MPE_L8O18cSfqHr7c9v/lib/python3.12/site-packages (from spacy<3.8.0,>=3.7.0->es-core-news-md==3.7.0) (2.0.8)
...

Alternatively, you can make a pyproject.toml like:
[project]
name = "foo"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.12"
dependencies = ["spacy>=3.7.6"]

And then run uv run -- spacy download es_core_news_md from that directory. In this case, uv will lock and sync your project (which lists spacy as a dependency), then run spacy download es_core_news_md in your project's virtual environment.

(In both examples, the -- is optional, I'm just including it for clarity.)

Thanks for the quick response @charliermarsh ! I was able to download the model 🙌🏽

Related https://github.com/astral-sh/uv/issues/6638

@lrvdijk probably with uv venv --seed or uv pip install pip, though the specifics depend on more information about the error.
Please open new issues instead of bumping old ones though, see #9452


Filament Shield - Posts, Categories and Exclusions not working

Hi,
Just found another issue, when I create a new role and try to exclude Posts, Categories and Exception it is not working.
For the exception I found that was missing in the app/Providers/AuthServiceProvider the entry  Exception::class => ExceptionPolicy::class in the policies, but still not working.
Even tried generating a new policies for all but still not working.
Can you help me identify what I am missing here?
Thanks

try manual binding model with policy in app/Providers/AuthServiceProvider
use App\Models\Blog\Category;
use App\Models\Blog\Post;
use App\Policies\Blog\CategoryPolicy;
use App\Policies\Blog\PostPolicy;
use App\Policies\ExceptionPolicy;
use BezhanSalleh\FilamentExceptions\Models\Exception;

protected $policies = [
        ...
        Category::class => CategoryPolicy::class,
        Post::class => PostPolicy::class,
        Exception::class => ExceptionPolicy::class,
        ...
];

Hi,
It is that, everything is running now.
Thanks for the help

Glad to help 🥂


Check the OS Licenses that are used in out dependencies match FINOS requirements

Feature Request
Description of Problem:
The libraries that we use in the DataHelix project must match FINOS requirements
Potential Solutions:
use command mvn license:license-list to generate a list of licenses used by our dependencies and check that each will be acceptable for FINOS.
Acceptance Criteria:

[ ] All licenses used by our dependencies are FINOS compliant.

@ColinEberhardt do you know what requirements (if any) we have for our dependencies when it comes to FINOS?
Is there anything else (except #669) that we will need to do.

Looking at the automated licence checker they use on one of their other projects, it looks like the following are fine:
MIT ISC Apache 'Apache 2.0' BSD BSD-2-Clause BSD-3-Clause  MIT/X11 Unlicense
If you have any other dependencies that don't use the above, let me know.
Also, regarding  #669, I doubt FINOS will mandate that. I'm also not a fan of reproducing licence text on ever file. AFAIK, the main purpose of this is to lower the risk of someone accidentally violating your licence terms. I don't see that as a significant risk in this case!

maven has produced a report which says our dependancies use the following licences:
The Apache License, Version 2.0: org.apiguardian:apiguardian-api, org.opentest4j:opentest4j
Eclipse Public License 1.0: JUnit
MIT License: Cucumber Expressions, Cucumber Tag Expressions, Cucumber-HTML, Cucumber-JVM, Cucumber-JVM: Core, Cucumber-JVM: JUnit, Cucumber-JVM: Java, Cucumber-JVM: PicoContainer, DataTable, Gherkin
Eclipse Public License v2.0: JUnit Jupiter API, JUnit Jupiter Engine, JUnit Jupiter Params, JUnit Platform Commons, JUnit Platform Engine API, JUnit Platform Launcher, JUnit Platform Runner, JUnit Platform Suite API, JUnit Vintage Engine
Public Domain: AOP alliance, Gherkin Repackaged Dependencies
New BSD License: Hamcrest Core
BSD Licence 3: Java Hamcrest
BSD: Automaton, PicoContainer Core
BSD License: Cucumber-JVM Repackaged Dependencies
The MIT License: Mockito
Apache License, Version 2.0: Apache Commons CSV
The Apache Software License, Version 2.0: Commons Lang, Cucumber-JVM Repackaged Dependencies, Gherkin Repackaged Dependencies, Google Guice - Core Library, Gson, Guava: Google Core Libraries for Java, JSONassert, Jackson-annotations, Jackson-core, Shaded DataTable Dependencies, Shazamcrest, jackson-databind, javax.inject

Thanks for the info. I think we should postpone any further action on this until we move the project to FINOS.
The only one I am mildly concerned about is AOP alliance, I don't think 'public domain' is a sufficient licence for software.

@ColinEberhardt is there anything we should do re AOP alliance now that we're pushing towards FINOS?

No, I don't think there is anything we should be doing right now. FINOS will do their own licence check very shortly as we now have the contribution underway.


Add client version to CLI to disallow running with old server

looks like we have the concept but not being used yet: https://github.com/uber/cadence/blob/master/tools/cli/factory.go#L120


ENH: Relax minmax to PartialOrd

The rationale is that it's the user's choice to input floating point elements or not. The restriction to Ord is too strict for regular use.

cc @birkenfeld as the original author.
It breaks with libstd conventions; I think they are too strict.

Sure!

Good to hear from you.

Thanks for your great work.


Cannot prompt api key

Hello !
I've seen your repo and wanted to use it in a Ubuntu Multipass vm but I basically cannot prompt my api key so nothing want to start when I try starting it and there's error.
Could you help me ?
Thanks :)

Hi,
Assuming that you have installed curl and jq in your machine, running the install script should work for Ubuntu. You can install curl using sudo apt-get install curl and jq in the same way.
Once you have curl, you can download and run the install script with this command curl -sS https://raw.githubusercontent.com/0xacx/chatGPT-shell-cli/main/install.sh | sudo -E bash, which will ask for your OpenAI API key and set it as an environment variable.
If want to install it manually, all you have to do is:

Download the chatgpt.sh file in a directory you want
Add the path of chatgpt.sh to your $PATH. You do that by adding this to your shell profile.export PATH=$PATH:/path/to/chatgpt.sh. If you use zsh, your shell profile is ~/.zprofile, if you're using bash it's ~/.profile in Ubuntu.
Add the OpenAI API token to your shell profile export OPENAI_TOKEN=your_token_here

I am still having the same problem even though i'm installing it manually. I have an error from jq saying "jq: error (at :8): Cannot iterate over null (null)". I am not familiar with JQ, could you help me to fixe this error ?

Prompt is the question you ask to chatGPT, you don't need to set it anywhere, you just type it when the script is running, the same way you would do when chatting in https://chat.openai.com/chat.
I just added the quotation marks escape, so now you can use quotation marks in the prompt.
In your ~/.profile, it shouldn't matter if you add the export statements with or without quotation marks, it should work both ways.
Try to curl the API from your terminal to see if you get a correct response. Just replace YOUR_TOKEN with your OpenAI token.
curl https://api.openai.com/v1/completions \
                -sS \
                -H 'Content-Type: application/json' \
                -H "Authorization: Bearer YOUR_TOKEN" \
                -d '{
                        "model": "text-davinci-003",
                        "prompt": "How to make an HTTP request in javascript?",
                        "max_tokens": 1000,
                        "temperature": 0.7
        }'

If that works and you have set your OpenAI key correctly in your shell profile, the script should work.

It works like that. I tried to reinstall everything from the script and it seems working now. I'll come back to you if it doesn't work anymore. Thanks !


Non-working tcp443 profile

I have a problem with the tcp443 profile on both split and full tunnel, below logs:
2020-05-14 07:32:48 Oficjalna kompilacja 0.7.16 na samsung SM-G950F (universal8895), Android 9 (PPR1.180610.011) API 28, ABI arm64-v8a, (samsung/dreamltexx/dreamlte:9/PPR1.180610.011/G950FXXS8DTC6:user/release-keys)
2020-05-14 07:32:48 Tworzenie konfiguracji…
2020-05-14 07:32:48 Stan sieci: CONNECTED LTE to MOBILE internet
2020-05-14 07:32:48 Debug state info: CONNECTED LTE to MOBILE internet, pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED
2020-05-14 07:32:49 Debug state info: CONNECTED LTE to MOBILE internet, pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED
2020-05-14 07:32:49 OpenVPN core 3.6_git:master(icsopenvpn/v0.7.16-0-gf4b9a655) android arm64 64-bit
2020-05-14 07:32:49 Copyright (C) 2012-2020 OpenVPN Inc. All rights reserved.
2020-05-14 07:32:49 OpenVPN core 3.6_git:master(icsopenvpn/v0.7.16-0-gf4b9a655) android arm64 64-bit
2020-05-14 07:32:49 Frame=512/2048/512 mssfix-ctrl=1250
2020-05-14 07:32:49 UNUSED OPTIONS
1 [verb] [4]
2 [connect-retry] [2] [300]
3 [resolv-retry] [60]
14 [nobind]
19 [resolv-retry] [infinite]
2020-05-14 07:32:49 Contacting <IP_ADDRESS>:443 via TCPv4
2020-05-14 07:32:49 Connecting to [<IP_ADDRESS>]:443 (<IP_ADDRESS>) via TCPv4
2020-05-14 07:32:49 Tunnel Options:V4,dev-type tun,link-mtu 1523,tun-mtu 1500,proto TCPv4_CLIENT,cipher AES-128-GCM,auth [null-digest],keysize 128,key-method 2,tls-client
2020-05-14 07:32:49 Creds: UsernameEmpty/PasswordEmpty
2020-05-14 07:32:49 Peer Info:
IV_VER=3.6_git:master
IV_PLAT=android
IV_NCP=2
IV_TCPNL=1
IV_PROTO=2
IV_CIPHERS=AES-256-GCM:AES-128-GCM:CHACHA20-POLY1305
IV_NCP=2
IV_TCPNL=1
IV_PROTO=2
IV_AUTO_SESS=1
IV_GUI_VER=de.blinkt.openvpn 0.7.16
IV_SSO=openurl,crtext
2020-05-14 07:32:49 VERIFY OK: depth=1, /CN=ChangeMe
2020-05-14 07:32:49 VERIFY OK: depth=0, /CN=pi-hole_b7d33516-7eec-4ca4-8415-c010b5d11e09
2020-05-14 07:32:50 SSL Handshake: peer certificate: CN=pi-hole_b7d33516-7eec-4ca4-8415-c010b5d11e09, 256 bit EC, curve:prime256v1, cipher: TLS_AES_256_GCM_SHA384  TLSv1.3 Kx=any      Au=any  Enc=AESGCM(256) Mac=AEAD
2020-05-14 07:32:50 Session is ACTIVE
2020-05-14 07:32:50 Sending PUSH_REQUEST to server...
2020-05-14 07:32:50 OPTIONS:
0 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
1 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
2 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
3 [dhcp-option] [DNS] [<IP_ADDRESS>]
4 [dhcp-option] [DNS] [<IP_ADDRESS>]
5 [block-outside-dns]
6 [compress] [lz4-v2]
7 [route-gateway] [<IP_ADDRESS>]
8 [topology] [subnet]
9 [ping] [10]
10 [ping-restart] [120]
11 [ifconfig] [<IP_ADDRESS>] [<IP_ADDRESS>]
12 [peer-id] [0]
13 [cipher] [AES-256-GCM]
2020-05-14 07:32:50 PROTOCOL OPTIONS:
cipher: AES-256-GCM
digest: NONE
compress: LZ4v2
peer ID: 0
2020-05-14 07:32:50 exception parsing IPv4 route: [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]  : tun_prop_error: route is not canonical
2020-05-14 07:32:50 We should call this session<IP_ADDRESS>
2020-05-14 07:32:50 Otwieram interfejs tun:
2020-05-14 07:32:50 Warning Samsung Android 5.0+ devices ignore DNS servers outside the VPN range. To enable DNS resolution a route to your DNS Server (<IP_ADDRESS>) has been added.
2020-05-14 07:32:50 Lokalne IPv4: <IP_ADDRESS>/24 IPv6: (not set) MTU: 1500
2020-05-14 07:32:50 Serwer DNS: <IP_ADDRESS>, <IP_ADDRESS>, Domena: null
2020-05-14 07:32:50 Trasy:
2020-05-14 07:32:50 Wykluczone trasy: <IP_ADDRESS>/12, <IP_ADDRESS>/16
2020-05-14 07:32:50 VpnService dodano trasy: <IP_ADDRESS>/32
2020-05-14 07:32:50 Dozwolone aplikacje VPN: com.chrome.canary, com.brave.browser, com.google.android.webview, com.android.chrome, com.microsoft.emmx
2020-05-14 07:32:50 Connected via tun
2020-05-14 07:32:50 LZ4v2 init asym=1
2020-05-14 07:32:50 COMPRESSION_ENABLED: Asymmetric compression enabled.  Server may send compressed data.  This may be a potential security issue.
2020-05-14 07:32:51 Debug state info: CONNECTED LTE to MOBILE internet, pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED
#help wanted

I have the same problem, while it looks connected it just leads to failed DNS lookups. Connecting with split tunnel UDP works on my phone, and I have only tested UDP on my laptops. Will paste my log as well, in case it can help resolve this. I am using a Samsung Note 10+. However, if UDP is recommended then I suppose it may not matter, just confused.
Log has been edited to remove my vms ip.
2020-05-25 23:19:58 official build 0.7.15 running on samsung SM-N975U (msmnile), Android 10 (QP1A.190711.020) API 29, ABI arm64-v8a, (samsung/d2qsq/d2q:10/QP1A.190711.020/N975USQS3CTD6:user/release-keys)
2020-05-25 23:19:58 Building configuration…
2020-05-25 23:19:58 OpenVPN core 3.6_git:master(icsopenvpn/v0.7.14-0-g5391351a) android arm64 64-bit
2020-05-25 23:19:58 Copyright (C) 2012-2017 OpenVPN Inc. All rights reserved.
2020-05-25 23:19:58 OpenVPN core 3.6_git:master(icsopenvpn/v0.7.14-0-g5391351a) android arm64 64-bit
2020-05-25 23:19:58 Frame=512/2048/512 mssfix-ctrl=1250
2020-05-25 23:19:58 UNUSED OPTIONS
1 [verb] [4]
2 [connect-retry] [2] [300]
3 [resolv-retry] [60]
6 [connect-timeout] [120]
15 [nobind]
20 [resolv-retry] [infinite]
2020-05-25 23:19:58 Network Status: CONNECTED  to WIFI
2020-05-25 23:19:58 Debug state info: CONNECTED  to WIFI , pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED
2020-05-25 23:19:58 Contacting VM-IP:443 via TCPv4
2020-05-25 23:19:58 Debug state info: CONNECTED  to WIFI , pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED
2020-05-25 23:19:58 Connecting to [VM-IP]:443 (VM-IP) via TCPv4
2020-05-25 23:19:58 Tunnel Options:V4,dev-type tun,link-mtu 1523,tun-mtu 1500,proto TCPv4_CLIENT,cipher AES-128-GCM,auth [null-digest],keysize 128,key-method 2,tls-client
2020-05-25 23:19:58 Creds: UsernameEmpty/PasswordEmpty
2020-05-25 23:19:58 Peer Info:
IV_VER=3.6_git:master
IV_PLAT=android
IV_NCP=2
IV_TCPNL=1
IV_PROTO=2
IV_CIPHERS=AES-256-GCM:AES-128-GCM:CHACHA20-POLY1305
IV_AUTO_SESS=1
IV_GUI_VER=de.blinkt.openvpn 0.7.15
IV_SSO=openurl,crtext
2020-05-25 23:19:58 VERIFY OK: depth=1, /CN=ChangeMe
2020-05-25 23:19:58 VERIFY OK: depth=0, /CN=pi-hole_0ac2c0f8-59b0-4957-93ce-c8705f896227
2020-05-25 23:19:58 SSL Handshake: CN=pi-hole_0ac2c0f8-59b0-4957-93ce-c8705f896227, TLSv1.3, cipher TLSv1.3 TLS_AES_256_GCM_SHA384
2020-05-25 23:19:58 Session is ACTIVE
2020-05-25 23:19:58 Sending PUSH_REQUEST to server...
2020-05-25 23:19:59 OPTIONS:
0 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
1 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
2 [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]
3 [dhcp-option] [DNS] [<IP_ADDRESS>]
4 [dhcp-option] [DNS] [<IP_ADDRESS>]
5 [block-outside-dns]
6 [compress] [lz4-v2]
7 [route-gateway] [<IP_ADDRESS>]
8 [topology] [subnet]
9 [ping] [10]
10 [ping-restart] [120]
11 [ifconfig] [<IP_ADDRESS>] [<IP_ADDRESS>]
12 [peer-id] [0]
13 [cipher] [AES-256-GCM]
2020-05-25 23:19:59 PROTOCOL OPTIONS:
cipher: AES-256-GCM
digest: NONE
compress: LZ4v2
peer ID: 0
2020-05-25 23:19:59 exception parsing IPv4 route: [route] [<IP_ADDRESS>] [<IP_ADDRESS>] [net_gateway]  : tun_prop_error: route is not canonical
2020-05-25 23:19:59 exception parsing IPv4 route: [route] [<IP_ADDRESS>/24] [] [net_gateway]  : tun_prop_error: route is not canonical
2020-05-25 23:19:59 We should call this sessionVM-IP
2020-05-25 23:19:59 Opening tun interface:
2020-05-25 23:19:59 Warning Samsung Android 5.0+ devices ignore DNS servers outside the VPN range. To enable DNS resolution a route to your DNS Server (<IP_ADDRESS>) has been added.
2020-05-25 23:19:59 Local IPv4: <IP_ADDRESS>/24 IPv6: (not set) MTU: 1500
2020-05-25 23:19:59 DNS Server: <IP_ADDRESS>, <IP_ADDRESS>, Domain: null
2020-05-25 23:19:59 Routes:
2020-05-25 23:19:59 Routes excluded: <IP_ADDRESS>/12, <IP_ADDRESS>/16
2020-05-25 23:19:59 VpnService routes installed: <IP_ADDRESS>/32
2020-05-25 23:19:59 Allowed VPN apps: org.mozilla.firefox, tv.twitch.android.app, org.mozilla.focus, com.facebook.katana, com.google.android.youtube, com.google.android.webview, com.android.chrome
2020-05-25 23:19:59 Connected via tun
2020-05-25 23:19:59 LZ4v2 init asym=1
2020-05-25 23:19:59 COMPRESSION_ENABLED: Asymmetric compression enabled.  Server may send compressed data.  This may be a potential security issue.
2020-05-25 23:19:59 Debug state info: CONNECTED  to WIFI , pause: userPause, shouldbeconnected: true, network: SHOULDBECONNECTED

https://github.com/rajannpatel/Pi-Hole-on-Google-Compute-Engine-Free-Tier-with-Full-Tunnel-and-Split-Tunnel-Wireguard-VPN-Configs
This guide walks you through running a script which will configure a Split Tunnel IPv6 Wireguard connection for your Android, iOS, Linux, macOS, & Windows devices.
I do not have any plans to support OpenVPN at this time, so I am closing this ticket.


Develop a REST API service to update product quantity

As a User
I need a way to update specific product quantity in shopcart
So that I can update new product quantity for this item
Assumptions

The base url is /shopcarts/{id}/update
The method of this API should be PUT
Url should return whether the operation is successful or not

Acceptance Criteria:
Given the service is available
When I call the URL `PUT /shopcarts/{id}/update?productName=<name_of_product>&productQuantity=<quantity_of_product>`
Then it should return 200 when the product quantity has been updated
Or it should return the corresponding error code if the operation fails

The enhancement associated with this issue should already be assigned in issue #4 and issue #5. Adding some quantity of a product and deleting some quantity of a product are updating.

Should we have a separate issue to just update quantity?


Reusable tests

Closes
General idea here is that a test suite is built that tries to be as agnostic of the content as possible. Instead, based on a set of conditions which need to be met per-renderer, it will test aria patterns.
If we want to test controlled states, we just create a renderer which renders a controlled implementation of the component. This allows us to test things like the onChange/value props indirectly. This leaves as much of the API for each component as possible up to the implementation.
If a renderer is not provided, then a set of the tests are skipped. This is useful because you may not want or need to run the absolute full suite on every implementation. You may also not have implemented a particular aria feature yet, but it's good to have tests on everything else. You can turn on those tests when you implement the feature easily enough.
Still working on API for it.
Spectrum v3 and s2 tests would be restricted to their own suites using Daniel's work to simplify.
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

I've also updated the utils PR with the remainder of the comments, apologies about the conflicts but hopefully they aren't too bad

I've also updated the utils PR with the remainder of the comments, apologies about the conflicts but hopefully they aren't too bad

no worries, it wasn't that bad

I'll see about rebasing my test util docs PR against this one soon, but I think the changes/conflicts will be pretty superficial


Setup Question

Hi all,
I have read through the docs but I am a little confused setting this up.
For my network name I went with my domain name. "domain.com"
Network CIDR I went with the default <IP_ADDRESS>/16.
Yes to auto-filling my public IP from <IP_ADDRESS> which was completed successfully.
The external endpoint was the bit that confused me. What should that be? Obviously not my public IP as that was auto-filled from the previous step. So is the external endpoint the IP address of the machine I am running the server on? That was I did, as a guess. "<IP_ADDRESS>:51820"
Then the listen port default was 51820 which I went with.
The setup finished but I have not yet started the service. Can anyone advise about the external endpoint and if I have that correct? If it's not the machine the server is running on, what should it be?
Thanks,
D

The external endpoint is the one outside the VPN i.e. the one innernet managed wireguard tunnels connect to. This is usually a public IP. The internal endpoint is the one the innernet clients connect to over the wireguard tunnel, usually the very first address in your network CIDR.
Regards,
afics

Right. What is the function of the "auto-fill public IP" step in the setup supposed to do then? I think that's what confused me a bit. Was it supposed to auto-fill the external endpoint address with my public IP? That didn't happen but the little tick on the auto-fill step was green and it said it was completed.
So the external endpoint should be :51820?
Is there a way to edit bits of the server config? Or do I need to delete and re-create the server?
Thanks,
D

Was it supposed to auto-fill the external endpoint address with my public IP?

Yes, interesting that this did not work for you.

So the external endpoint should be "Public IP":51820?

Yes.

Is there a way to edit bits of the server config? Or do I need to delete and re-create the server?

Recreation is a possibility. As far as I know your only other option is to change it directly in the innernet server sqlite database, usually under /var/lib/innernet-server/something.db.
Regards,
Armin

OK. Thanks.
I'm running the server on a MacMini. Or at least trying to. Although there are instructions on how to install with Homebrew etc, the starting/stopping of the server instructions uses the systemd (Linux) syntax.  I have tried using brew services start innernet@server-name to invoke launchctl but no luck. I'm guessing because there is no .plist file?


Fix feature = "cargo-clippy" deprecation

https://blog.rust-lang.org/2024/02/28/Clippy-deprecating-feature-cargo-clippy.html

Thanks for the PR. Would it be possible to drop the feature gate and use #[allow(clippy::lint_name)] throughout?

Let's try again :)

Thanks!


Move some classes out of shared

Some public types that are living in the shared folder and not being used elsewhere. Brought them out of Shared into the normal src folder.

API change check
API changes are not detected in this pull request.


added DP folder

Added DP folder ,which contains few dynamic programming problems.
Please do add nessesory tags and lavels.


Fixing the issue no. 6052

fixes #6052 by making Href return the url path as-is if it starts with http or https

Hi @mkinawy, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, DNFBOT;


Some Issues with Replication

(1) I used mmcv.imread to read the data, because directly using mmcv.load as in your code would result in an error, indicating unsupported color.
(2) Using the configuration hop_bevdet4d-r50-depth.py and reading the data with mmcv.imread shouldn't cause any issues. However, the replicated accuracy is only around 0.36+ mAP.
Can you please provide more details about your configuration and setup? It's possible that there might be some misconfigurations or parameters that need adjustment. Additionally, double-checking the dataset, model settings, and any preprocessing steps could help identify any potential sources of the lower accuracy.

Hi @wuwangzhuanwan , thanks for your feedback.
We fix the bug for loading data by disk. It should work fine now.
Specifically, I find that mmcv.imread is not equal to the original code Image.open from BEVDet. This may probably be the reason for the low accuracy.

(1) I used mmcv.imread to read the data, because directly using mmcv.load as in your code would result in an error, indicating unsupported color. (2) Using the configuration hop_bevdet4d-r50-depth.py and reading the data with mmcv.imread shouldn't cause any issues. However, the replicated accuracy is only around 0.36+ mAP.
Can you please provide more details about your configuration and setup? It's possible that there might be some misconfigurations or parameters that need adjustment. Additionally, double-checking the dataset, model settings, and any preprocessing steps could help identify any potential sources of the lower accuracy.

Have you tried python tools/test.py?  My  training log is 0.36+ mAP  but 0.39+ mAP in testing, which is almost same as the author  released. It doesn't need to change configs or anything, just run python tools/test.py $config $ checkpoint. I guess there may be some TTA diff between val and test.
NDS=0.5066/0.5099,mAP=0.3979/0.3990

Hi, the performance in the log file is based on epoch_24.pth. However, we test the model with epoch_24_ema.pth. The ema checkpoint is empirically better than the original model. This should account for the difference of mAP.


fix: changed swapchain and dependencies to get it working on linux

#14

Looks good but I won't be able to test it for a while, so I trust it'll be correct after the quirks are fixed


Introduce gopkg.in for go module

This change introduces gopkg.in to allow everyone to use multus
code as library. Currently multus-cni uses version 'v3', hence
it hits go modules limitation, as #504 describes.
This change fix #504 with gopkg.in.

Pull Request Test Coverage Report for Build 588ee00af-PR-505

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 73.939%

Totals

Change from base Build ec5fd6c92:
0.0%

Covered Lines:
1098

Relevant Lines:
1485

💛  - Coveralls


ENH: Allow partial table schema in to_gbq() table_schema (#218)

This PR:

Adds a function to pandas_gbq.schema, update_schema which takes and old schema and a new one, updating any existing fields and adding any new ones
Generates the default schema using schema.generate_bq_schema for every DataFrame passed to to_gbq, and if a table_schema is passed, this schema is used to update the generated schema. This way custom provided fields in table_schema are added, and missing fields fall back to the generated defaults.

This is the functionality requested in #218

Great!
Any thoughts @tswast ?

@JohnPaton Want to add a Whatsnew? Feel free to add you GH handle

@max-sixty By Whatsnew do you just mean to update the Changelog?

@max-sixty Done! I didn't see any other handles in the Changelog so if that's not what you meant just let me know 😆

Many thanks @JohnPaton !

Thanks a bunch for the contribution!


fix VFIO_USER_DEVICE_GET_INFO request size check

The spec says "only the message header is needed in the command
message", we were needlessly requiring the device info struct to be provided
too.
Signed-off-by: John Levon<EMAIL_ADDRESS>

I think the plan was to include struct vfio_device_info in the request to mimic VFIO. Maybe there's value in keeping it this way so that the client can set argsz and the server can tell which exact version of the struct to respond with? Or will we be covered by the verson negotiation?

It's how (say) get region info works, certainly. It's a bit weird since vfio of course has to provide a buffer - it's ioctl not socket - but providing an unused struct, (except argsz I guess), is very weird over a socket

After input by JJ, going to fix the spec instead.


Remove X-UA-Compatible meta tags

The X-UA-Compatible meta tag needs to be included before all other tags except for the <title> and the other <meta> tags. This isn't the case for most demos.

Since the X-UA-Compatible HTTP header is sent, the meta tag isn't needed.

See also:

https://github.com/h5bp/html5-boilerplate/blob/41b63975e5bab30f8926148b43d0864c71f8f8a4/dist/doc/html.md#the-order-of-the-title-and-meta-tags

We shouldn't remove that tag.
We are adding it from the live site but this demos can be consumed in another way (we could move the Demos to another domain, run from http-server, etc.) so we shouldn't assume that the server is always going to add it.

We are adding it from the live site but this demos can be consumed in another way (we could move the Demos to another domain, run from http-server, etc.) so we shouldn't assume that the server is always going to add it.

@molant Thanks for the info! Will open another PR to include the tags correctly.

so we shouldn't assume that the server is always going to add it.

@molant Do note that then the X-UA-Compatible meta tag needs to be added to all demos, same as the meta charset.
Will open PR for both or those case.

Keep in mind that the newer demos are for edge features and this tag is for IE only (edge ignores it), so not all demos need it.

Keep in mind that the newer demos are for edge features... so not all demos need it.

@molant The need for it is debatable even for older demos, but I do get your point!

this tag is for IE only (edge ignores it),

@molant IE 11 also ignores it if I remember correctly.


Res:  add "Covered EC2 Instances" table (as per jaws 1)

Analogous to the res uncovered instances table, add back the Covered instances.

Closed:  done.


Support server side encryption using AWS key management

from the SQS homepage

KEEP SENSITIVE DATA SECURE
You can use Amazon SQS to exchange sensitive data between applications using server-side encryption (SSE) to encrypt each message body. Amazon SQS SSE integration with AWS Key Management Service (KMS) allows you to centrally manage the keys that protect SQS messages along with keys that protect your other AWS resources. AWS KMS logs every use of your encryption keys to AWS CloudTrail to help meet your regulatory and compliance needs.

It seems like this was added in https://github.com/Jaymon/morp/commit/157057a2af130aae93b06284a8418040c751d7c2


Implement Binary prefix, Currency prefix (Merge #70 #81)

This branch merges #70 and #81 which were targeting the same set of lines.

hmmm something went wrong in my repo when I tried to fix the credits on a commit 😰

The currency “prefix” feature is misnamed: this is not a prefix, it’s a symbol, or even a suffix based on how it’s formatted. In the case of SI, it’s called a prefix because it’s used at the start of a word: for example, “k” stands for kilo as in “kilogram” or “kilobyte”.  (In retrospect, d3.formatPrefix should probably have been named to d3.formatSi, but that ship has sailed.)
More generally, I’m wary of adding new localization features to this library. The binary prefixes seem reasonable because they correspond closely to the existing SI prefixes and are well-defined, but the currency formatting feature feels… a little ad hoc and I question how useful it is for all locales. In the long run, I’d prefer to use the standard JavaScript APIs for number and date localization and I’m generally opposed to expanding the surface area of this library.

Should we go ahead with #70 since there is no consensus on adding currency formatting?

@Fil @curran Any plan on this? As there are open source data visualization tools that depend on d3format and if this "human consumption filesize" introduce in d3format, then it will be very helpful to the community.

@nimisha1921 I’ve just been rolling my own using Math.log2 and Intl.NumberFormat:
const PREFIXES = ['', 'Ki', 'Mi', 'Gi', 'Ti', 'Pi', 'Ei', 'Zi', 'Yi'];

function binaryFormatParts(num: number) {
  if (!Number.isFinite(num) || Math.abs(num) < 1) {
    return [num, 0];
  }

  const exponent = Math.log2(Math.abs(num));
  const prefix = Math.min(Math.trunc(exponent / 10), PREFIXES.length - 1);

  return [num / 2 ** (prefix * 10), prefix];
}

export function binaryFormat(num: number): string {
  const [ex, prefix] = binaryFormatParts(num);
  const displayNumber = new Intl.NumberFormat('default', {
    maximumFractionDigits: 2,
    style: 'decimal',
  }).format(ex);

  return `${displayNumber} ${PREFIXES[prefix]}B`;
}


Multiple notifications while reading emails on another device

Describe the bug
When reading emails on another device (IMAP IDLE enabled on mobile), the new notification sounds on every new email marked as read (total new mail count is decreased as I am reading emails).
To Reproduce
Steps to reproduce the behavior:

Enable IMAP IDLE on mobile
Wait for some new emails (more than one), notification properly appears on mobile (with sound)
Read some of the new emails on another device (e.g. desktop)
As soon as some of the mail is marked read on server (number of new emails is changed), new notifications pops making sound.

Expected behavior
I think the proper solution is not to make a sound again, update the existing notification instead (which should result to no-sound update). Unless another fresh email arrives - this would raise a new notification with sound.
Environment:

K-9 Mail version: 5.800
Android version: 9
Device: Sony Xperia XZ1 Compact
Account type: IMAP with IDLE (Dovecot IMAP server)

As far as I can determine, the notification happens when I delete something on a PC. The PC is using IMAP and and when deleting, it marks the e-mail read (if it is not already) then moves it to the Deleted/Trash folder. If I do an immediate delete (shift and delete in Thunderbird, my desktop client) I do not appear to get a notification (which is great!).


added FloatProperty to set monochrome background color

Check out rviz ColorProperty.

I think this would provide a bit too much choice.

What if the Global Options Background Color were used?
Is there a use case where the rviz camera would need a different background color than the standard rviz render window?

I personally like to have a black RVIZ background to work with, but it does not look good if the image is integrated into a document (thesis or bugreport).

It seems like the default behavior should be that the rviz camera background is the same as the main rviz window.  If there is a need to have two different background colors at the same time, then there should be a full color property (and a checkbox to enable it, or can switching between them be handled otherwise?).
I like a color background to distinguish it from foreground objects that are frequently monochrome.
For future reference another useful capability would be to optionally have a transparent background in an rgba Image.

Squashed and merged with c20d2bbc78212ee76e24f6174ad75e8173630966

The background color doesn't match desired as revealed by #14 (channels are swapped), it will be corrected separate from this.


Error following 0.1.7 update

Version of the custom_component
0.1.7
Installation method (hacs / manual)
hacs
Installation method of hass (venv, docker, hassio,...)
docker
Configuration
configuration.yaml.txt
Describe the bug
Logger: homeassistant.setup
Source: setup.py:206
First occurred: 12:21:58 (1 occurrences)
Last logged: 12:21:58
Setup failed for custom integration volkswagen_we_connect_id: Requirements for volkswagen_we_connect_id not found: ['weconnect>=0.55.1'].
Debug log

Logger: homeassistant.util.package
Source: util/package.py:107
First occurred: 12:21:53 (3 occurrences)
Last logged: 12:21:58
Unable to install package weconnect>=0.55.1: ERROR: Cannot install weconnect==0.55.1 because these package versions have conflicting dependencies. ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts [notice] A new release of pip is available: 23.1 -> 23.1.2 [notice] To update, run: pip install --upgrade pip
Logger: homeassistant.setup
Source: setup.py:206
First occurred: 12:21:58 (1 occurrences)
Last logged: 12:21:58
Setup failed for custom integration volkswagen_we_connect_id: Requirements for volkswagen_we_connect_id not found: ['weconnect>=0.55.1'].

Tried to downgrade to v0.1.6, received the same errors following restart.
Downgrade to v0.1.5 has got the component working again.
Happy to try upgrading again and assisting with the debug as required.

Further update, resolved by upgrading HASS to 2023.6 as there are conflicting dependancies.


The 2.x Purge!

Ticket: #2323
Issue
Remove the classes we identified in #2323

[x] PostGetter

[x] ... and tests

[x] TermGetter

[x] ... and tests

[x] Any old $CommentClass props and the like

[x] QueryIterator

[x] ... and tests

[x] Timber_WP_CLI_Command.php (or move to its own repo)

[x] Post::determine_id() and possibly other methods

Solution
Delete the classes. Ensure that Factories and other 2.x mechanisms are fully functional without these.
Usage Changes
Everything in 2.x
Considerations
It's been well considered
Testing
Interested to see how this affects coverage score. Looking to bring that back up to come close to current 1.x levels

Coverage increased (+0.5%) to 87.426% when pulling 3a0b530229a1082ecbcff270faab9cb4975dddcb on 2323/purge_again into d9ec0333ca0c81bf67ae3d4a3a0fff503e248bae on 2.x.

QueryIterator ... and tests

@jarednova @gchtr I think we're ready to delete these. Of the remaining two tests in test-timber-query-iterator.php, one is already ported over into test-timber-post-collection.php. The last one is basically just testing Twig internals. I highly doubt there are any code paths through Timber being tested here that aren't already thoroughly tested elsewhere. Can you take a look and let me know if I'm missing anything?

@acobster looks like versions of both had actually been ported over to test-timber-post-collection.php. I went ahead and removed the file in my most recent commit. I think we're on to final review here!

SO SATISFYING. :tada: :hocho:

It's like losing fifty pounds, right? (or 1865 lines of code I guess)


Memory leak: service.js process is running after node heap out of memory crash

Current behavior

Every time node falls with Heap out of memory, the <project>/node_modules/fork-ts-checker-webpack-plugin/lib/service.js is still running, causing memory leak. The more crashes: the more service.js processes causing leak
Expected behavior

Proper shutdown of all running services
Steps to reproduce the issue

run node --max-old-space-size=256 ./node_modules/.bin/webpack-dev-server on any mid-size webpack+typescript+react project
wait node Heap out of memory crash
ps aux | grep fork-ts-checker

Issue reproduction repository

Environment

fork-ts-checker-webpack-plugin: 4.0.4
typescript: 3.7.4
tslint: 5.11.0
webpack: 4.19.1
os: linux mint 19.1

@vlamitin
Please try fork-ts-checker-webpack-plugin@alpha - I've published a new version which should resolve this issue 🚀
I will close this issue to clean-up the backlog. If this release didn't solve the issue, we can re-open this :)

Was the alpha released to the 4.x branch?  It seems to not exist anymore, and I am getting out of memory errors using 4.1.6.  I'm unable to upgrade to version 5 for now, because I am stuck on Node 8.

Unfortunately, the alpha was released on the 5.x branch


request a new release

👋 it has been five years since last release, thus, requesting a new one. Thanks!

The reason I haven't cut a new release is because nothing actually changed since then other than dependency bounds (which I've already published as matching revisions to Hackage).  However, I can still cut a release if you want (especially if there was something I missed)

@Gabriella439 that would be great, I need a new release to get some build patch. Thanks!


Suggestion: Automatically request broader permissions

Hello. Instead of dax asking permission to read every single possible filename in every single directory in path, it would be nice if dax could automatically request read permissions in each path directory.
Before
await $`node -v`;

Long output
✅ Granted env access.
✅ Granted read access to <CWD>.
✅ Granted read access to "C:\pathdir\node.COM".
✅ Granted read access to "C:\pathdir\node.EXE".
✅ Granted read access to "C:\pathdir\node.BAT".
✅ Granted read access to "C:\pathdir\node.CMD".
✅ Granted read access to "C:\pathdir\node.VBS".
✅ Granted read access to "C:\pathdir\node.VBE".
✅ Granted read access to "C:\pathdir\node.JS".
✅ Granted read access to "C:\pathdir\node.JSE".
✅ Granted read access to "C:\pathdir\node.WSF".
✅ Granted read access to "C:\pathdir\node.WSH".
✅ Granted read access to "C:\pathdir\node.MSC".
✅ Granted read access to "C:\pathdir\node.CPL".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.COM".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.EXE".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.BAT".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.CMD".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.VBS".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.VBE".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.JS".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.JSE".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.WSF".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.WSH".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.MSC".
✅ Granted read access to "C:\Program Files\PowerShell\7\node.CPL".
✅ Granted read access to "C:\WINDOWS\system32\node.COM".
✅ Granted read access to "C:\WINDOWS\system32\node.EXE".
✅ Granted read access to "C:\WINDOWS\system32\node.BAT".
✅ Granted read access to "C:\WINDOWS\system32\node.CMD".
✅ Granted read access to "C:\WINDOWS\system32\node.VBS".
✅ Granted read access to "C:\WINDOWS\system32\node.VBE".
✅ Granted read access to "C:\WINDOWS\system32\node.JS".
✅ Granted read access to "C:\WINDOWS\system32\node.JSE".
✅ Granted read access to "C:\WINDOWS\system32\node.WSF".
✅ Granted read access to "C:\WINDOWS\system32\node.WSH".
✅ Granted read access to "C:\WINDOWS\system32\node.MSC".
✅ Granted read access to "C:\WINDOWS\system32\node.CPL".
✅ Granted read access to "C:\WINDOWS\node.COM".
✅ Granted read access to "C:\WINDOWS\node.EXE".
✅ Granted read access to "C:\WINDOWS\node.BAT".
✅ Granted read access to "C:\WINDOWS\node.CMD".
✅ Granted read access to "C:\WINDOWS\node.VBS".
✅ Granted read access to "C:\WINDOWS\node.VBE".
✅ Granted read access to "C:\WINDOWS\node.JS".
✅ Granted read access to "C:\WINDOWS\node.JSE".
✅ Granted read access to "C:\WINDOWS\node.WSF".
✅ Granted read access to "C:\WINDOWS\node.WSH".
✅ Granted read access to "C:\WINDOWS\node.MSC".
✅ Granted read access to "C:\WINDOWS\node.CPL".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.COM".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.EXE".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.BAT".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.CMD".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.VBS".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.VBE".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.JS".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.JSE".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.WSF".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.WSH".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.MSC".
✅ Granted read access to "C:\WINDOWS\System32\Wbem\node.CPL".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.COM".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.EXE".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.BAT".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.CMD".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.VBS".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.VBE".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.JS".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.JSE".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.WSF".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.WSH".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.MSC".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0\node.CPL".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.COM".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.EXE".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.BAT".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.CMD".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.VBS".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.VBE".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.JS".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.JSE".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.WSF".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.WSH".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.MSC".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH\node.CPL".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.COM".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.EXE".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.BAT".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.CMD".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.VBS".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.VBE".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.JS".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.JSE".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.WSF".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.WSH".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.MSC".
✅ Granted read access to "C:\ProgramData\chocolatey\bin\node.CPL".
✅ Granted read access to "C:\Program Files\dotnet\node.COM".
✅ Granted read access to "C:\Program Files\dotnet\node.EXE".
✅ Granted read access to "C:\Program Files\dotnet\node.BAT".
✅ Granted read access to "C:\Program Files\dotnet\node.CMD".
✅ Granted read access to "C:\Program Files\dotnet\node.VBS".
✅ Granted read access to "C:\Program Files\dotnet\node.VBE".
✅ Granted read access to "C:\Program Files\dotnet\node.JS".
✅ Granted read access to "C:\Program Files\dotnet\node.JSE".
✅ Granted read access to "C:\Program Files\dotnet\node.WSF".
✅ Granted read access to "C:\Program Files\dotnet\node.WSH".
✅ Granted read access to "C:\Program Files\dotnet\node.MSC".
✅ Granted read access to "C:\Program Files\dotnet\node.CPL".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.COM".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.EXE".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.BAT".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.CMD".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.VBS".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.VBE".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.JS".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.JSE".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.WSF".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.WSH".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.MSC".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin\node.CPL".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.COM".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.EXE".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.BAT".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.CMD".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.VBS".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.VBE".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.JS".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.JSE".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.WSF".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.WSH".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.MSC".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin\node.CPL".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.COM".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.EXE".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.BAT".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.CMD".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.VBS".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.VBE".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.JS".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.JSE".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.WSF".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.WSH".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.MSC".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin\node.CPL".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\node.COM".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\node.EXE".
✅ Granted run access to "C:\Users\ethan\scoop\apps\nodejs\current\node.EXE".
v20.8.0

After
await $`node -v`;

✅ Granted env access.
✅ Granted read access to <CWD>.
✅ Granted read access to "C:\pathdir".
✅ Granted read access to "C:\Program Files\PowerShell\7".
✅ Granted read access to "C:\WINDOWS\system32".
✅ Granted read access to "C:\WINDOWS".
✅ Granted read access to "C:\WINDOWS\System32\Wbem".
✅ Granted read access to "C:\WINDOWS\System32\WindowsPowerShell\v1.0".
✅ Granted read access to "C:\WINDOWS\System32\OpenSSH".
✅ Granted read access to "C:\ProgramData\chocolatey\bin".
✅ Granted read access to "C:\Program Files\dotnet".
✅ Granted read access to "C:\Program Files (x86)\Gpg4win\..\GnuPG\bin".
✅ Granted read access to "C:\Program Files\Docker\Docker\resources\bin".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current\bin".
✅ Granted read access to "C:\Users\ethan\scoop\apps\nodejs\current".
✅ Granted run access to "C:\Users\ethan\scoop\apps\nodejs\current\node.EXE".

This could also be controllable via a setting (like setBroadReadPerms or something), so that any application that accesses the internet could opt out of this functionality, for the sake of it being clear what the app does,


⚠ Bofrost.nl has degraded performance

In d9b78d8, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1473 ms

Resolved: Bofrost.nl performance has improved in ef51f39.


Standardize formatting

Some options
black
snakefmt
markdownlint
shellcheck
shfmt
How to reformat without screwing up git blame

This is a good one to save until right before we merge into main.


BT-1592

Add a check on any refresh token to renderLoading for rendering. Handling the logic of an invalid token is handled in other place.
Manually tested with an invalid access token.

No, the renderLoading function returns the SplashView in our case. So it will keep the SplashView until refreshing the access token is resolved.


Regression in v3.7.7

What are the steps to reproduce this issue?
Not sure there is any steps to reproduce as it happens during vscode startup when the extension is enabled.
What happens?
Extension seems to throw during initialization. Regression was introduced in v3.7.7 since everything seems to work fine in v3.7.6.
What were you expecting to happen?
Any logs, error output, etc?
(To get extension logs see Terminal > Output > Extension Host and copy output concerning pico-w-go or micropico)
(If it’s long, please paste to https://gist.github.com and insert the link here)
2024-05-30 22:50:40.794 [error] Activating extension paulober.pico-w-go failed due to an error:
2024-05-30 22:50:40.794 [error] Error: spawn Unknown system error -86
  at ChildProcess.spawn (node:internal/child_process:414:11)  at Object.spawn (node:child_process:799:9)
  at new e (/Users/nestor/.vscode/extensions/paulober.pico-w-go-3.7.7-darwin-x64/dist/extension.cjs:1:5181)  at Wh.activate (/Users/nestor/.vscode/extensions/paulober.pico-w-go-3.7.7-darwin-x64/dist/extension.cjs:8:86711)
  at async exports.activate (/Users/nestor/.vscode/extensions/paulober.pico-w-go-3.7.7-darwin-x64/dist/extension.cjs:8:105526)  at async h.n (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:143:6281)
  at async h.m (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:143:6244)  at async h.l (/Applications/Visual Studio Code.app/Contents/Resources/app/out/vs/workbench/api/node/extensionHostProcess.js:143:5701)

Any other comments?
I can provide more details if necessary since it seems I can reproduce it 100% of times just by launching the editor.
Which version of MicroPico are you using?
v3.7.7
Support info
Version: 1.89.1 (Universal)
Commit: dc96b837cf6bb4af9cd736aa3af08cf8279f7685
Date: 2024-05-07T05:14:24.611Z
Electron: 28.2.8
ElectronBuildId: 27744544
Chromium: 120.0.6099.291
Node.js: 18.18.2
V8: 12.0.267.19-electron.0
OS: Darwin x64 23.5.0

Should be fixed with release v3.0.9.

I can confirm it is now fixed. Thank you for such a quick update! 💖


Changed datepicker widget from egui_extras to use DateTime

Chrono crate advises not to use Date type, and, to discourage to use it, that type is non serializable with serde. Since persistence feature of eframe requires serde, is is impossible to make datepicker state persistent without conversions. Switching Date type to Datetime fixes that issue.

Whilst I agree that attaching time to explicit date functionality is not perfect, I'd strongly shy away from any custom Date type.
The cleanest choice might be to extend the widget to include time (and default to zero time as in the pull), and adopt the ecosystem time standard. I found this pull looking for a date time picker widget!

Chrono API documentation recommends using NaiveDate instead of Date, which would make sense here since the datepicker widget is time zone agnostic.


Add support for Maven Polyglot POM formats (internal)

Replaces #85 in order to run private integration tests

Fixes #84

I don't understand any of these changes, but I approved them because I trust you :P


event call on update settings duplicated

in code , there is a watch on settings changes. when this change is made , the destroyandInit function is call, in this function the init is call .
in the init function you have on calling to initiate the event listener.
each time setting change its add one listener to the event .
to prevent this you have to :
1 - name the event with a namespace ,
2 - make a call to off before on
example  :
slickness.on('swipe', function (event, slick, direction) {
$timeout(function () {
                  scope.$apply(function () {
                    options.event.swipe(event, slick, direction);
                  });
                });
              });
becomes :
slickness.off('swipe.slickCarousel').on('swipe.slickCarousel', function (event, slick, direction) {
 $timeout(function () {
 scope.$apply(function () {
 options.event.swipe(event, slick, direction);
 });
 });
 });

Thanks for report.
Will take a look soon!


warns-if-compiled-on-debug-build

Adds buttons to warn that project was compiled in debug mode
renames WARN_COLOR (previously was WARM)
removes duplicated WARN_COLOR
Can be removed in settings tab

FYI: Debug mode overflows in macos (my mac is quite powerful)


Allow authors to go to completely arbitrary urls

Our current recommendation to go to a completely arbitrary url is to use the "I start the interview at" Step. Unfortunately, it checks that the page is a da interview page. Some authors have change their pages to customize them and they don't contain things like .daquestion (#daquestion?) that we use to check that it's a da page. For example, the log in page at https://dev.pleadingpower.com/user/sign-in.
We could look for other docassemble elements, but it feels like it's just pushing off the inevitable. Tests won't be able to do very much on those pages if they don't use some da conventions, like the kinds of input field DOM structures we're expecting, but they can do some things, like check for phrases.

PR merged


Update tutorial-enable-cloud-sync-sspr-writeback.md

Updating so admins know WHERE to run the cmdlets

@JeffBley : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

#sign-off


VSCode trying to update WSL on launch for literally no reason

Type: Bug
Every time I launch VS Code after the latest update, it opens a Terminal.exe that tries to update WSL. I don't have WSL installed at all on my system, and I don't have any exstensions that require or assume WSL to be installed. This is really annoying. Please fix.
Windows Subsystem for Linux must be updated to the latest version to proceed. You can update by running 'wsl.exe --update'.
For more information please visit https://aka.ms/wslinstall

Press any key to install Windows Subsystem for Linux.
Press CTRL-C or close this window to cancel.
This prompt will time out in 60 seconds.

I even disabled the WSL extension and it still does it.

Not sure if this info helps, but I had WSL installed and then uninstalled it.

@thesoftwarephilosopher Same situation with me. Previously had WSL and then uninstalled it, but VS code was working fine with no issues. And after the update, this pop starts showing up out of nowhere.

Yes this is probably related to this code if it's showing somewhere when WSL is not installed:
https://github.com/microsoft/vscode/blob/24b8e3ce99ceccde43b8b19e8f15dc9d548d3c2d/src/vs/platform/terminal/node/terminalProfiles.ts#L351-L359

I think I'm having a similar problem. Just uninstalled WSL and now I get this prompt every time the extension host restarts.

same problem after uninstalling wsl