Mockito fails for reified kotlin methods in Java tests

Actual behavior (the bug)
Running simple unit tests on Java with Mockito fails when mocking the .header() method of class Context.kt. Apparently Mockito can not choose the correct method to mock. This behavior is not 100% reproducible, it fails for 90% of the time but runs fine in roughly 10% of the time. A user on Stackoverflow was able to reproduce the issue as well.
Exception:
org.mockito.exceptions.misusing.WrongTypeOfReturnValue:

String cannot be returned by header()
header() should return Validator
***
If you're unsure why you're getting above error read on.
Due to the nature of the syntax above problem might occur because:
1. This exception *might* occur in wrongly written multi-threaded tests.
   Please refer to Mockito FAQ on limitations of concurrency testing.
2. A spy is stubbed using when(spy.foo()).then() syntax. It is safer to stub spies -
   - with doReturn|Throw() family of methods. More in javadocs for Mockito.spy() method.

    at my.package.stupidTest1(JavalinTest.java:28)

More info: https://stackoverflow.com/questions/61792392/final-kotlin-class-can-not-be-mocked-because-method-should-return-validator
Expected behavior
Java test runs fine and the header() method can be mocked
To Reproduce

set up Mockito in a java project

<dependency>
  <groupId>org.mockito</groupId>
  <artifactId>mockito-core</artifactId>
  <version>3.2.0</version>
  <scope>test</scope>
</dependency>

create the file resources/mockito-extensions/org.mockito.plugins.MockMaker with the content mock-maker-inline so that final Kotlin classes can be mocked
create a test class with a single test method:

 @Test
  void stupidTest1() {
    Context context = mock(Context.class);

    String test1 = "hello123";
    when(context.header("Authorization")).thenReturn(test1);
  }

run mvn tests, it should fail almost every time with the exception shown above

Thanks @juv ! Would you like to submit a PR to add the sugested annotations?

@tipsy I've never written code in Kotlin, I'm unsure if all of those reified methods can be safely annotated or if that breaks something for users: https://github.com/tipsy/javalin/search?q=reified&unscoped_q=reified
IMHO, this should be done by someone that knows what he's doing

Fair enough @juv, I'll mark it as HELP WANTED and fix it myself if no one grabs it!

@juv can you try depending on the current master using https://jitpack.io/#tipsy/javalin to see if it fixed your issues?

tl;dr:
My unit test still fails with the version master-javalin-3.8.0-gbe8910b-36. My IntelliJ decompiler does not see the annotation @JvmSynthetic for the header() method of class Context.kt -- I'm not sure if it should. One thing that is weird is that I can see other annotations such as @kotlin.jvm.JvmOverloads and @field:kotlin.jvm.JvmField in the decompiled class that IntelliJ shows me.

More detailed information:
I went to https://jitpack.io/#tipsy/javalin and took the latest version master-javalin-3.8.0-gbe8910b-36 in tab "Builds". The Jitpack log says Start: Wed May 27 07:11:37 UTC 2020 26818da26f7b [...] [interop] Add JvmSynthetic annotations to reified methods (#981) and thus it should reference the pull request tipsy/javalin#981 in the log file.
My environment is presumably using the correct dependency. IntelliJ links me to this class when I CTRL+Click on the Context class reference in my code:

However, I can not see the @JvmSynthetic annotation in the Context.kt class that I downloaded. I am not sure if I am supposed to see the annotation. https://github.com/tipsy/javalin/blob/be8910b3e867e1662663fbedb4e914c4262aff64/javalin/src/main/java/io/javalin/http/Context.kt#L252-L254
The unit test still fails with the exception shown in the initial post. It looks like Mockito is still chosing the incorrect method or something is missing to make Mockito be aware of that @JvmSynthetic annotation.

I assume that Mockito is ignoring the synthetic flag while mocking the Context object. It seems like Mockito gets confused by the two methods that have the same signature. This might actually be a bug in Mockito.
An alternative approach: Mock the HttpServletRequest and HttpServletResponse and create a normal instance (Link to the constructor) of the context object.

Just to clarify, mockito thinks it's mocking hte inline reified function header, when you're actually attempting to mock the standard header function? That seems very strange to me, almost certainly a mockito bug.
I'm not familiar with how mockito works, but an inline function doesn't exist at run time, so I'm not sure how it would even be getting confused, especially since there's no way for java to even reference an inline function.
I just threw together a new test on my local javalin for this and I'm running into the same issue. The only thing I could see is that maybe the mock maker inline configuration causes Mockito to run at compile time and is looking at the class badly?
Might be better to report this to Mockito as they're more likely to understand what is going on.

@MrThreepwood yes you describe exactly what is happening.
Thanks for the suggestion, I've now created https://github.com/mockito/mockito/issues/1943

A suggestion (Mockito.<String>when(context.header("Authorization")).thenReturn(test1);) was recently posted in mockito/mockito#1943, might be worth a shot.

@tipsy it looks like this is not helping and it sounds like a potential fix in Mockito itself would require a lot of work :/

I guess we can't really do anything about this issue, so I'll just close it. We can consider different naming in Javalin 4.

I'd definitely go with a different name pretty please...

The best hack around this is to create a real context with mock response and request objects.  Would like something easier...
val context = mock<Context> {
    on { header(AUTHORIZATION) } doReturn "Basic anNtaXRoOnBhc3N3b3Jk" // jsmith/password
}

with
val context = Context(
        mock { on { getHeader(AUTHORIZATION) } doReturn "Basic anNtaXRoOnBhc3N3b3Jk" }, // jsmith/password
        mock {}
)

Works well for getHeader() but for body() for instance it means creating the input stream and whatnot. It’s a rabbit hole, don’t go there.

I guess we can’t really do anything about this issue, so I’ll just close it. We can consider different naming in Javalin 4.

Additional convenience methods wouldn’t break compatibility and might be worth the effort as it’s currently breaking unit testing of controller methods in many cases.

Also, it does likely apply to MockK as well: mockk/mockk#502

Yes - I tried Mockk and seemed to have problems too.  Another alternative is using something like the Spring test's MockHttpServletRequest & Response but it's still a lot of work.

Additional convenience methods wouldn’t break compatibility and might be worth the effort as it’s currently breaking unit testing of controller methods in many cases.

Do you have a suggestion for naming?

Can someone explain the purpose of that method? When would I use it in Kotlin?
    /** Reified version of [header] (Kotlin only) */
    inline fun <reified T : Any> header(header: String) = header(header, T::class.java)

@juv:
// Headers (Kotlin)
val example = ctx.header("Example");
val version = ctx.header<Int>("Version").get();
val version = ctx.header<Int>("Version").check({ it > 4 }).get();
val myValue = ctx.header<MyValue>("X-My-Header").get();

// Headers (Java)
String example = ctx.header("Example");
int version = ctx.header("Version", Integer.class).get();
int version = ctx.header("Version", Integer.class).check(i -> i > 4).get();
MyValue myValue = ctx.header("X-My-Header", MyValue.class).get();

Maybe jHeader() and kHeader() for the inline so it doesn’t get in the way when auto-completing. Since this is quite technical unlike the rest of the API which is rather colloquial.
For the topic in general: I have spent quite some time now debugging MockK and Javalin trying different trimmed down reproductions and I can’t seem to find a deterministic behavior in this. The lesson learnt for me is that inline functions should not be part of a public API—unless for Kotlin object declarations maybe. Since they cannot be mocked the calling method is not unit testable.
Of course the syntax is nice, but my suggestion is to remove those inline functions in Javalin 4. At least I personally wouldn’t recommend using it for said reasons. It’s kinda frustrating to run into this problem. I personally in the beginning didn’t even notice that there was that conflict in method signatures between the inline function and the other function. Although that statement isn’t entirely true as the return types are in fact different.
One method returns String, the other Validator. In Mockito I saw this causing ClassCastException when Mockito mocked the wrong of the two methods/functions and it’s kind of a path to figure out that the inline function versus method with same parameter is causing this behavior.
I ended up working around this using Delegation Pattern. Here’s an example diff: https://github.com/Zemke/javalin-chess/commit/707cfaaa85e9eba3277c1ec50d4f86147f30380f

Of course the syntax is nice, but my suggestion is to remove those inline functions in Javalin 4.

@Zemke Yeah, I've been trying to work around this for a couple of hours now, it definitely has to be fixed in Javalin 4.
Could you summarize the problem and create a new issue?

This will be fixed soon: https://github.com/tipsy/javalin/pull/1162


feat: exposes defaultSort property for collection list view

Description
#2377
Adds defaultSort property to collection admin options.

[X] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

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

Checklist:

[X] Existing test suite passes locally with my changes
[X] I have made corresponding changes to the documentation

The defaultSort property should also be taken into account in the collection find operation as well on the backend. Which means that it should actually not be a property on admin and instead directly on the collection itself.

If this is done on the backend, then the FE doesn't even need to know about it. With sort left undefined in the search params, it will just work as intended.


Catch errno=ENOMEM on heap ops in target

Maybe impl at the same time as https://github.com/Arinerron/heaptrace/issues/37 ?

gonna update #37 with my research on this

solved #37 a different way, but my research on errno is still in #37


Enter the Prize Draw now

If you participated, then please enter your details in this form now so that you don't miss out on the chance to win in our prize draw.

Today is your final notice and last chance to enter your details to claim a prize. We still have a few people who are eligible but haven't submitted their details.

Hi Alex,
You should have had mine? Please let me know if not.
Thanks
On Wed, Nov 10, 2021 at 10:24 AM Alex Ellis @.***>
wrote:

Today is your final notice and last chance to enter your details to claim
a prize. We still have a few people who are eligible but haven't submitted
their details.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexellis/growlab/issues/78#issuecomment-964992163,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHYJPVBVLPP23HUAB6JLK23ULJB6HANCNFSM5FTWWINQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


How to use webkit2gtk with glade?

In glade, I added a WebKitWebView, then I want to get this object in my code:
        let builder = gtk::Builder::new_from_resource("/im/idlesong/handybox/window.ui");
        let widget: gtk::ApplicationWindow = builder.get_object("window").expect("Failed to find the window object");
        let webview: WebView = builder.get_object("webview").expect("Failed to find label");

It builded success, but when run:
failed to add UI: .:136:1 Invalid object type 'WebKitWebView'

I searched the web,  it seems there are only two related answers,
https://stackoverflow.com/questions/51010820/glade-invalid-object-type-webkitwebview-when-loading-from-file-works-on-gla
https://prognotes.net/2019/12/make-a-gtk-web-browser-with-glade/
But one is C, one is Vala. It's too hard to convert to rust for me. Is there any samples or  documents about it? Thanks.

Not sure.
Try to call get_type() before you use it.

Heyo! I'm using Glade on my GTK-rs web browser, and I've got this working.
Firstly, the get_object() function gets an object by its ID/unique instance name. Have you given your WebView widget a name in Glade?
Additionally, if that doesn't work for whatever reason (though, it should), you can create your own WebView instance in Rust, like this:
let web_view_builder = webkit2gtk::WebViewBuilder::new()
        .is_ephemeral(false)
        .automation_presentation_type(webkit2gtk::AutomationBrowsingContextPresentation::Tab);
let web_view = web_view_builder.build();

The webkit2gtk-rs and gtk3-rs documentation books are really useful, so take a look at those on the regular.


Add automatic dependency updating workflow to github actions

Adds a workflow triggered by remote branches that will:

Run an appropriate update script
Push and create a PR into next
Use an appropriate commit message to trigger the full CI run

This will require some upstream changes before it starts triggering, but has been extensively tested in the forked repo.

Closed in favour of https://github.com/bugsnag/bugsnag-js/pull/2178


edit contents of scss output

Hello, thanks for this package, very useful! I have a question/suggestion. At the moment, when I generate an scss file, it outputs a file with a font-face declaration. I would like the output to be just the variable map. Is this possible?
current output:
$trinity-iconfont-font: "trinity-iconfont";

@font-face {
    font-family: $trinity-iconfont-font;
    src: url("./trinity-iconfont.woff?55321795d3354290fd22d5fc0f063eae")
            format("woff"),
        url("./trinity-iconfont.woff2?55321795d3354290fd22d5fc0f063eae")
            format("woff2");
}

i[class^="icon-"]:before,
i[class*=" icon-"]:before {
    font-family: trinity-iconfont !important;
    font-style: normal;
    font-weight: normal !important;
    font-variant: normal;
    text-transform: none;
    line-height: 1;
    -webkit-font-smoothing: antialiased;
    -moz-osx-font-smoothing: grayscale;
}

$trinity-iconfont-map: (
    "externalLinkIcon": "\f117",
    "checkIcon": "\f103",
    "lineCrossIcon": "\f10a",
    "alertIcon": "\f101",
    "arrowForwardIcon": "\f102",
    "arrowReplyIcon": "\f104",
    "bellIcon": "\f105",
    "boxIcon": "\f106",
    {...etc}

my fantasticonrc:
module.exports = {
    inputDir: "style-dictionary/assets/fonts/iconfont/svg", // (required)
    outputDir: "style-dictionary/assets/fonts/iconfont/", // (required)
    fontTypes: ["woff", "woff2"],
    assetTypes: ["scss", "css", "json"],
    codepoints: {
        externalLinkIcon: 61719,
        checkIcon: 61699,
        lineCrossIcon: 61706,
    },
    pathOptions: {
        scss: "packages/styles/src/variables/_iconfont-tokens.scss",
        woff: "packages/styles/src/variables/trinity-iconfont.woff",
    },
};

I would also like to know the answer to this. The reason being I would like to use a base64 version font-face of the generated font file instead.


[Question] How to use query editor?

Can anyone tell me how to use query editor? I have entered a query in it, but nothing happens and I am confused about what to do next.

@MuhammedZakir you need to capture your query/node (node) @foo, then putting your cursor over @foo will highlight all matches in your code buffer.

Thanks @stsewd! :-)

Seems not to work as described now.
health#nvim_treesitter#check
========================================================================
## Installation
  - OK: `tree-sitter` found  0.19.3 (parser generator, only needed for :TSInstallFromGrammar)
  - OK: `node` found v15.11.0 (only needed for :TSInstallFromGrammar)
  - OK: `git` executable found.
  - OK: `cc` executable found.
  - OK: Neovim was compiled with tree-sitter runtime ABI version 13 (required >=13). Parsers must be compatible with runtime ABI.

## Parser/Features H L F I
  - php            ✓ ✓ ✓ ✓
  - query          ✓ ✓ ✓ ✓
  - c              ✓ ✓ ✓ ✓

In query editor I put cursor over foo and nothing is highlighted.
(php_tag) @foo

nvim-treesitter/playground@d96cef521d22afd1a409449a890f20f50b436ee1
NVIM v0.5.0-dev+1221-g9b2d4ff62
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3

Tested with -u NORC

I spent a bunch of time scratching my head about this too. It would be nice to add an example about how to do this!

Addressing this in https://github.com/nvim-treesitter/playground/pull/44


Allow usernames with plus signs in unblock view

This fixes bug #76 where an exception like
Reverse for 'defender_unblock_username_view' with arguments '(u'user+test@domain.tld',)' and keyword arguments '{}' not found. 1 pattern(s) tried: [u'admin/defender/blocks/username/(?P[A-Za-z0-9-._@]+)/unblock$']

was raised when trying to access the /admin/defender/blocks/ URL when a user with a plus sign had been locked out.

I've added the main urls to the test urls file. I don't know if this is right, actually I don't know why a separate urls.py for testing is needed.

Coverage increased (+1.5%) to 96.042% when pulling c0534d83f8f514a4930490ed5989d9e789e35110 on dukebody:unblock-view-username-with-plus into c4f3a610365282d9fa0746413da4dc7736a8f4ce on kencochrane:master.

We keep the urls in separate files because this is an app people include in their overall projects and if they decide to use our URL, we want to make sure our testing urls aren't accidentally exposed in their main app. Probably a little paranoid, but better safe then sorry :)


Added deepgram-topic-detection function

closes appwrite/appwrite#4457

Thanks for the amazing PR 😇 I have added small changes to fix env variables and readme commands.
Merging the PR now 🥳

@Meldiron Thanks if you like you can assign bugs/tasks which you think i can do it'll be a great help.


Add instructions how to configure existing OpenMetadata catalog

Fybrik allows to use existing, maybe externally install OpenMetadata catalog, we have to provide instructions how to use it.

Resolved by https://github.com/fybrik/fybrik/pull/1811


Update default branch name

Since the recent change on the default branch name on Github, CodePipeline is failing as master doesn't exist.
https://github.com/github/renaming
This is just a temporary fix, best course of action will be to update the Cloudformation template.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closed in favor of #963


Fix pagespeed_libraries_generator.sh

to fetch pagespeed_libraries.conf from github rather than svn.

LGTM, this should get cherry-picked to master too.

err... actually, this should be fetching from trunk-tracking, not master

On Wed, Jul 8, 2015 at 2:29 PM, Jeffrey Crowell<EMAIL_ADDRESS>wrote:

err... actually, this should be fetching from trunk-tracking, not master

There's not trunk-tracking branch on mod_pagespeed.  Are you sure?  I'm
pretty master for mod_pagespeed is latest and greatest, and releases are
down branches.  It should be OK to pull this data from head.

—
Reply to this email directly or view it on GitHub
https://github.com/pagespeed/ngx_pagespeed/pull/985#issuecomment-119690061
.

you're right, it should be good for both, my mistake :+1:

LGTM

merged and added the -S to curl


Give KKR and Tara ability to add users to communications github

they are currently part of communications team, need to make a communicatins repository with admin permissions

Both have admin permissions, need to show them how to use them.

Seems like Tara has this one well in hand.  Will train Ken when it comes up.   Since public, sharing git link should also work for many use cases.


connect to https server

I have arangodb running on ssl protected server. Admin interface is running ok.
When I try to connect using:
client = ArangoClient(protocol="https", host="www.test.com", port=12345,
    username="root", password="password", enable_logging=True, verify=True
)

(python 3.5 on windows 7)
I got an SSL error:
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:645)
complete traceback:
Traceback (most recent call last):
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\connectionpool.py", line 595, in urlopen
    chunked=chunked)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\connectionpool.py", line 352, in _make_request
    self._validate_conn(conn)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\connectionpool.py", line 831, in _validate_conn
    conn.connect()
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\connection.py", line 289, in connect
    ssl_version=resolved_ssl_version)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\util\ssl_.py", line 308, in ssl_wrap_socket
    return context.wrap_socket(sock, server_hostname=server_hostname)
  File "C:\Users\crecheverria\Apps\WinPython\64bit-<IP_ADDRESS>\python-3.5.1.amd64\lib\ssl.py", line 376, in wrap_socket
    _context=self)
  File "C:\Users\crecheverria\Apps\WinPython\64bit-<IP_ADDRESS>\python-3.5.1.amd64\lib\ssl.py", line 747, in __init__
    self.do_handshake()
  File "C:\Users\crecheverria\Apps\WinPython\64bit-<IP_ADDRESS>\python-3.5.1.amd64\lib\ssl.py", line 983, in do_handshake
    self._sslobj.do_handshake()
  File "C:\Users\crecheverria\Apps\WinPython\64bit-<IP_ADDRESS>\python-3.5.1.amd64\lib\ssl.py", line 628, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:645)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\adapters.py", line 423, in send
    timeout=timeout
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\packages\urllib3\connectionpool.py", line 621, in urlopen
    raise SSLError(e)
requests.packages.urllib3.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:645)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "c:\Users\crecheverria\Devel\SGOT\fallas\v_python\arangodb3\test_02.py", line 16, in <module>
    username="root", password="calipso01", enable_logging=True, verify=True
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\arango\client.py", line 65, in __init__
    self.verify()
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\arango\client.py", line 78, in verify
    res = self._conn.head('/_api/version')
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\arango\connection.py", line 162, in head
    auth=(self._username, self._password)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\arango\http_clients\default.py", line 40, in head
    auth=auth
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\sessions.py", line 510, in head
    return self.request('HEAD', url, **kwargs)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\sessions.py", line 475, in request
    resp = self.send(prep, **send_kwargs)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\sessions.py", line 596, in send
    r = adapter.send(request, **kwargs)
  File "C:\Users\crecheverria\Devel\z_venvs\py64-3.5.1-gen\lib\site-packages\requests\adapters.py", line 497, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:645)

Hi cer1969,
It looks like your server does not have a valid SSL certificate.
I release python-arango 3.3.0, which now has a new argument you can use to disable to SSL verification:
client = ArangoClient(
protocol="https", host="www.test.com", port=12345,
    username="root", password="password", enable_logging=True, verify=True
)

I'am using the web interface and the browser says the certificate is valid. I also checked here  https://www.digicert.com/help/.

Hi cer1969,
It could be that the requests library (which python-arango uses under the hood) cannot locate the certificate bundle to verify the the certificate of your server.
Could you try some of the solutions mentioned in http://stackoverflow.com/questions/10667960/python-requests-throwing-up-sslerror? Please note that the new parameter check_cert simply passes whatever value you specify to request library's verify parameters.
For example, it is possible that you don't have the certificate bundle installed at all which you could solve by something like: sudo apt-get install ca-certificates (ubuntu)
Let me know how that goes.
Thanks.

I tried with check_cert="certificate.file" but I got the same error.
Probably a request,py problem.
When I do check_cert=False I understand the certificate is used but not
verified, so if I know the certificate is ok I can trust the connection is
secure?
2016-09-08 12:03 GMT-03:00 Joohwan Oh<EMAIL_ADDRESS>
Hi cer1969,
It could be that the requests library (which python-arango uses under the
hood) cannot locate the certificate bundle to verify the the certificate of
your server.
Could you try some of the solutions mentioned in http://stackoverflow.com/
questions/10667960/python-requests-throwing-up-sslerror? Please note that
the new parameter check_cert simply passes whatever value you specify to
request library's verify parameters.
For example, it is possible that you don't have the certificate bundle
installed at all which you could solve by something like: sudo apt-get
install ca-certificates (ubuntu)
Let me know how that goes.
Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/joowani/python-arango/issues/19#issuecomment-245627826,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANFidZkZ02d1jvGhR26ZMXJjBGI5JoL5ks5qoCPTgaJpZM4J27qm
.

If verify if set to False, the certificate is not checked at all, which leaves you open to man-in-the-middle attack. You would probably be fine going without this check, but ideally you want to fix the core problem. Have you tried this?
pip install certifi

As this is not a problem with python-arango itself I will close this issue now.


IndexError: list index out of range

Hey when I try to run salsa:
python ~/SALSA-master/run_pipeline.py -a BW_racon3.pilon2_shasta.fasta -l BW_racon3.pilon2_shasta.fasta.fai -b BW_shasta.bed -e CATG -o scaffolds_shasta -m yes -g contigs_graph.gfa
I get this error:
File "/home/SALSA-master/layout_unitigs.py", line 827, in update_bed
first = prev_attrs[3].split('/')[1]
IndexError: list index out of range
Can you help me understand why I am getting this error?

I got the same error, you can change the sam to bam and then bam to bed formate.

But I already did that through this code
#Map raw read to reference
bwa index BW_PoreC_all.fastq
bwa mem -t 10 "BW" BW_racon2_pilon2_miniasm.fasta BW_PoreC_all.fastq > BW_PoreC_shasta_mapped_reads.sam; samtools view -b -@ 1 BW_PoreC_shasta_mapped_reads.sam | samtools sort -@ 1 -o BW_shasta.sorted.bam ; samtools index BW_shasta.sorted.bam
#Turn bam into bed
bamToBed -i BW_shasta.sorted.bam > BW_shasta.bed
#Sort the bam file
sort -k 4 BW_shasta.bed > tmp && mv tmp BW_shasta.bed

OK， I change sam to bed and got the same errors.
At 2020-11-09 05:52:47, "Hannah1746"<EMAIL_ADDRESS>wrote:
But I already did that through this code
#Map raw read to reference
bwa index BW_PoreC_all.fastq
bwa mem -t 10 "BW" BW_racon2_pilon2_miniasm.fasta BW_PoreC_all.fastq > BW_PoreC_shasta_mapped_reads.sam; samtools view -b -@ 1 BW_PoreC_shasta_mapped_reads.sam | samtools sort -@ 1 -o BW_shasta.sorted.bam ; samtools index BW_shasta.sorted.bam
#Turn bam into bed
bamToBed -i BW_shasta.sorted.bam > BW_shasta.bed
#Sort the bam file
sort -k 4 BW_shasta.bed > tmp && mv tmp BW_shasta.bed
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Did you ever figure out how to get around this error?

I tried this pipline (https://github.com/ArimaGenomics/mapping_pipeline), that the SALSA paper recommend.
At 2020-11-10 02:13:05, "Hannah1746"<EMAIL_ADDRESS>wrote:
Did you ever figure out how to get around this error?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.


P4C: Fix URL map

Currently a few of the college costs app user-entered values are not being preserved in the URL. This PR updates the variable names in the URL map so the values will be correctly added to the query string.
Changes

Updates to URL map

How to test this PR

Check out branch
Run gulp scripts:apps
Navigate to college costs app
Enter values in the following fields

Costs page: Additional financial obligations
Grants & scholarships: Other federal grants, Military tuition assistance, GI Bill, Other (Yellow Ribbon, etc)
Other sources: Other funding

Copy the URL, open it in another tab, and verify that the values entered in the previous step persist in the new tab

Perfect! Thank you!


Guardians of the rift alias

Closing this, aliases are now defined in the API code, the app/bot no longer need these duplicated.
https://github.com/wise-old-man/wise-old-man/pull/1095


Best way to use the scrollbar as body-scrollbar?

When using this plugin as a body-scrollbar in combination with parallax plugins the problem occurs that the $(window).scroll()-event doesn't get fired. Of course this is correct, because the only thing that fires the scroll-event is the scrollbar-div. However, this makes it impossible to use such plugins. Is there any workaround?

There are 2 ways to do this:

Use position:fixed for your scrollable div and update body height/width/scroll with your scrollable dimensions
Patch your parallax plugin to handle scroll of your div instead of body

I recommend to use second way as it's much easier. Unfortunately you have to make this patch every time you update your plugin to newer version.

Thank your for the reply. Could you please explain the first point? Do you mean I should set the body-height to the scrollable-height?

The main idea is that your body content is invisible, but scrollable - it's covered by your fixed element. And you update your body content's dimensions with your fixed element's content ones. It's really not good idea to do it that way, but it's possible as one of the ways to accomplish your goal.
I recommend you to look at  your parallax source code and if it uses jQuery-based scroll event, you can trigger scroll event for body from your custom div element. Something like
$('.my-div').on('scroll', function(event){ $('body').trigger(event); })


New Networking is broken by alpha6

@levitanong reported:
Below is the minimal case, based off of the shadow-cljs template.
https://github.com/levitanong/load-ok-repro
Relevant bits are in client.cljs, specifically line 22 and below.
I have verified that Inspect is causing the breakage.

Fixed in 6fc89bf6c8da8f53224c8f250299d1a4d501ad3b


Introducing Docker to Grape

Hi,
I've been using Docker for a while now and I thought it would be a great addition to include a Dockerfile to Grape. Also, docker-compose always been my go to for running containers even though its for multiple containers. It's just easier to configure than running docker commands with arguments.
Dockerfile

As an ARG called RUBY_VERSION for building an image on different ruby-version
Setting BUNDLE_PATH to /usr/local/bundle/gems
Setting LIB_PATH to /var/grape. Code from the host will be copied in that folder
Add basic packages for development
Install latest version of rubygems and bundle
Starts with an entrypoint

Entrypoint

echoes version of ruby, rubygems and bundle.
check gems sanity with bundle
update gems to latest version
execute bundle exec :your_command

Docker-compose

Default RUBY_VERSION to 3.
Add gems volume

Basic CLI

Basic command  docker-compose run --rm --build grape <your_command>. You may replace <your_command> by anything that runs under bundle exec
--rm will remove automatically the container
--build will automatically build or rebuild the image (useful when changing RUBY_VERSION)

CLI

Running all specs docker-compose run --rm --build grape rspec
Running rspec on a specific file docker-compose run --rm --build grape rspec spec/:file_path
Running task docker-compose run --rm --build grape rake <task_name>
Running rubocop docker-compose run --rm --build grape rubocop
Running all specs on a specific ruby version (e.g 2.7.7) RUBY_VERSION=2.7.7 docker-compose run --rm --build grape rspec
Running specs on a specific gemfile (e.g rails_7.gemfile) docker-compose run -e GEMFILE=rails_7 --rm --build grape rspec

Running the container will automatically install the missings gems and run your command.

Let's add something to the README about using this and a CHANGELOG entry?
Should there be a workflow that brings up the container in CI?

This is nice to show people how to use Docker with grape but if it goes in the repo, feels like it belongs in something like an “examples” directory.

This is nice to show people how to use Docker with grape but if it goes in the repo, feels like it belongs in something like an “examples” directory.

@duffn I've added examples in the CONTRIBUTING.md

Can I have other maintainers say whether they would like this merged or not?
Should we require a CI to ensure the Dockerfile works?

rubocop offenses fixed in #2319

Thanks. Rebase?


Roll Engine from e8c13aa012c9 to 14c78ff3aa6c (41 revisions)

Roll Engine from e8c13aa012c9 to 14c78ff3aa6c (41 revisions)
https://github.com/flutter/engine/compare/e8c13aa012c9...14c78ff3aa6c
2020-06-10<EMAIL_ADDRESS>Make sure the native callback is registered before running the test. (flutter/engine#18981)
2020-06-10<EMAIL_ADDRESS>Fix right-to-left selection not working. (flutter/engine#18951)
2020-06-10<EMAIL_ADDRESS>Revert "Add RAII wrapper for EGLSurface (#18849)" (flutter/engine#18971)
2020-06-10<EMAIL_ADDRESS>Roll Skia from f7005498d416 to 9c401e7e1ace (7 revisions) (flutter/engine#18965)
2020-06-10<EMAIL_ADDRESS>Update firefox.dart (flutter/engine#18939)
2020-06-10<EMAIL_ADDRESS>Roll Dart SDK from f40875bf1b68 to 2b917f5b6a0e (10 revisions) (flutter/engine#18964)
2020-06-10<EMAIL_ADDRESS>Fixes UI freezes when multiple Flutter VC shared one engine (flutter/engine#18816)
2020-06-10<EMAIL_ADDRESS>Make rasterizer screenshot work with gl_context_switch (flutter/engine#18850)
2020-06-10<EMAIL_ADDRESS>[web] running ios unit tests from felt … (flutter/engine#18856)
2020-06-10<EMAIL_ADDRESS>Roll ANGLE to ToT (flutter/engine#15868)
2020-06-10<EMAIL_ADDRESS>Roll Skia from 4442bfa50b21 to f7005498d416 (6 revisions) (flutter/engine#18961)
2020-06-10<EMAIL_ADDRESS>Add support for horizontalDoubleArrow and verticalDoubleArrow cursors (flutter/engine#18916)
2020-06-10<EMAIL_ADDRESS>Roll Skia from 5d18c24f7f3a to 4442bfa50b21 (2 revisions) (flutter/engine#18959)
2020-06-10<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from P46X2... to oWhyp... (flutter/engine#18958)
2020-06-10<EMAIL_ADDRESS>Roll Dart SDK from dd47387430a0 to f40875bf1b68 (8 revisions) (flutter/engine#18957)
2020-06-10<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from Lwsff... to 8AiUM... (flutter/engine#18956)
2020-06-10<EMAIL_ADDRESS>Roll Skia from 437c78593cb5 to 5d18c24f7f3a (9 revisions) (flutter/engine#18954)
2020-06-10<EMAIL_ADDRESS>fix build (flutter/engine#18950)
2020-06-10<EMAIL_ADDRESS>Add RAII wrapper for EGLSurface (flutter/engine#18849)
2020-06-10<EMAIL_ADDRESS>Add FlMouseCursorPlugin (flutter/engine#18888)
2020-06-10<EMAIL_ADDRESS>Roll Skia from 1ce11e676e6d to 437c78593cb5 (5 revisions) (flutter/engine#18947)
2020-06-10<EMAIL_ADDRESS>Roll Dart SDK from 0f15f8722e29 to dd47387430a0 (12 revisions) (flutter/engine#18943)
2020-06-10<EMAIL_ADDRESS>remove failing tests on safari (flutter/engine#18944)
2020-06-10<EMAIL_ADDRESS>Roll Skia from 64688229fd4b to 1ce11e676e6d (10 revisions) (flutter/engine#18942)
2020-06-10<EMAIL_ADDRESS>Add onDisplayOverlaySurface JNI (flutter/engine#18859)
2020-06-10<EMAIL_ADDRESS>Add FlPlatformPlugin (flutter/engine#18316)
2020-06-09<EMAIL_ADDRESS>Move Surface and friends to flow/ (flutter/engine#18938)
2020-06-09<EMAIL_ADDRESS>Roll Skia from 45f36b5b5547 to 64688229fd4b (8 revisions) (flutter/engine#18935)
2020-06-09<EMAIL_ADDRESS>Remove the input type and action from TextInputModel. (flutter/engine#18919)
2020-06-09<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from FYAm7... to P46X2... (flutter/engine#18937)
2020-06-09<EMAIL_ADDRESS>Run the rasterizer on the platform thread  (flutter/engine#18841)
2020-06-09<EMAIL_ADDRESS>SkMatrix::MakeFoo is deprecated, use SkMatrix::Foo instead (flutter/engine#18934)
2020-06-09<EMAIL_ADDRESS>[fuchsia] add robust scheduling for flutter_runner (flutter/engine#18233)
2020-06-09<EMAIL_ADDRESS>Fix CanvasKit text colors by not passing a Malloc'ed array. (flutter/engine#18936)
2020-06-09<EMAIL_ADDRESS>[web] Line break algorithm using unicode properties (flutter/engine#18795)
2020-06-09<EMAIL_ADDRESS>Add user agent for Chromium based Edge Browser to browser detection (flutter/engine#18910)
2020-06-09<EMAIL_ADDRESS>Remove pipeline in favor of layer tree holder (flutter/engine#18901)
2020-06-09<EMAIL_ADDRESS>Roll Dart SDK from 5c1376615ec6 to 0f15f8722e29 (6 revisions) (flutter/engine#18932)
2020-06-09<EMAIL_ADDRESS>Roll Skia from 9d79ac6b81ba to 45f36b5b5547 (6 revisions) (flutter/engine#18931)
2020-06-09<EMAIL_ADDRESS>Log EGL errors (flutter/engine#18917)
2020-06-09<EMAIL_ADDRESS>Use constant for error code (flutter/engine#18913)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from LwsffthReD15 to 8AiUMhG_j62z
fuchsia/sdk/core/mac-amd64 from FYAm7VABULN0 to oWhypFySb_c0
...

Commit queue failed; closing this roll.


[BCEL-317] Pluggable cache for ConstantUtf8

https://issues.apache.org/jira/browse/BCEL-317

I'm not sure if this PR is a good thing. I'd like feedback from the community.
TL;DR
If you want caching, use the existing getCachedInstance() API, it's an existing feature.
Details
[Please do not hesitate to correct details I may have wrong]
Right now, before this PR, we have two clear and simple APIs:

getInstance() always returns a new instance.
getCachedInstance() returns a cached instance unless the cache is full.

Some thoughts:

The PR adds 2 global variables as system properties, this is definitely not a good thing, even if the current code already uses a sys prop.
The PR removes an existing sys prop, therefore breaking compatibility.
The PR changes the behavior of getInstance() which can now return a cached instance, which could (or not) break things but definitely falls into the unexpected category IMO.

More Details
The PR should not write to the console.
The existing code might be better re-implemented using a ConcurrentHashMap for simpler synchronization.

@garydgregory Thank you for the feedback. My approach in this PR tries to avoid modifying other classes than ConstantUtf8.
While waiting for other maintainers' comments, I will think about another approach that does not change the behavior of getInstance and getCachedInstance.
Removed System.out.println from this PR.

Hi @suztomo
I can appreciate the desire to provide a PR with a narrow footprint, but we must take semantics into account here. My basic issue is still that we have two public APIs, one that says "give me an instance" and the other that says "give me a cached instance". This to me implies that the former never returns a cached instance and it currently does not indeed. The latter always returns a cached instance, but in fact only tries to do so by enforcing a cache size limit.
It feels like this PR is trying to force caching on clients that use getInstance() instead of properly calling getCachedInstance(). If the client is BCEL itself, then we need to address using BCEL in a caching vs. not mode.

I had an (wrong) impression that getInstance can return a cached instance. It's because until the performance degradation problem (BCEL-186), getInstance used to return a cached result depending on the system property (code diff to revert that 4 years ago). This PR is on top of the reverted design but not caching by default.

Closing this in favor of #27


build: updated proto annotations

This PR was generated using Autosynth. :rainbow:
Commits in this repo since last synth:
0bb2037f1a8a97b61062d88daba63f85df259144 chore: skip img.shields.io in docs test
6f74808cb8ee612cdc4804097bf6f646b29b94e7 chore(deps): update dependency @types/mocha to v7
Log from Synthtool
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Cloning googleapis.
synthtool > Pulling Docker image: gapic-generator-typescript:latest
latest: Pulling from gapic-images/gapic-generator-typescript
Digest: sha256:f942dfb2c5054f69e6d4d190fb9b4a099f9bd38ec6ca16de6afedc777a4f5a80
Status: Image is up to date for gcr.io/gapic-images/gapic-generator-typescript:latest
synthtool > Generating code for: google/cloud/redis/v1beta1.
synthtool > Generated code into /tmpfs/tmp/tmp39pq4xok.
synthtool > Pulling Docker image: gapic-generator-typescript:latest
latest: Pulling from gapic-images/gapic-generator-typescript
Digest: sha256:f942dfb2c5054f69e6d4d190fb9b4a099f9bd38ec6ca16de6afedc777a4f5a80
Status: Image is up to date for gcr.io/gapic-images/gapic-generator-typescript:latest
synthtool > Generating code for: google/cloud/redis/v1.
synthtool > Generated code into /tmpfs/tmp/tmpw3bq699v.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node12/common.cfg
.kokoro/continuous/node12/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node12/common.cfg
.kokoro/presubmit/node12/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
samples/README.md
synthtool > Replaced '/compute/docs/' in src/v1beta1/cloud_redis_client.ts.
synthtool > Replaced '/compute/docs/' in src/v1/cloud_redis_client.ts.
npm WARN npm npm does not support Node.js v12.14.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/
npm WARN deprecated<EMAIL_ADDRESS>core-js@<3 is no longer maintained and not recommended for usage due to the number of issues. Please, upgrade your dependencies to the actual version of core-js@3.

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node -e "try{require('./postinstall')}catch(e){}"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

><EMAIL_ADDRESS>prepare /tmpfs/src/git/autosynth/working_repo
> npm run compile

npm WARN npm npm does not support Node.js v12.14.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>compile /tmpfs/src/git/autosynth/working_repo
> tsc -p . && cp -r protos build/

npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/watchpack/node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules/fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"linux","arch":"x64"})

added 1036 packages from 1044 contributors and audited 7181 packages in 26.669s
found 0 vulnerabilities

npm WARN npm npm does not support Node.js v12.14.1
npm WARN npm You should probably upgrade to a newer version of node as we
npm WARN npm can't make any promises that npm will work with this version.
npm WARN npm Supported releases of Node.js are the latest release of 6, 8, 9, 10, 11.
npm WARN npm You can find the latest version at https://nodejs.org/

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> gts fix && eslint --fix samples/*.js samples/**/*.js

/tmpfs/src/git/autosynth/working_repo/samples/quickstart.js
  24:38  error  "@google-cloud/redis" is not found  node/no-missing-require

✖ 1 problem (1 error, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>fix: `gts fix && eslint --fix samples/*.js samples/**/*.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!


HMR broken for markdown files

Steps To Reproduce

standard install from template
edit some markdown in index.md

Environment

Node version (node -v):  18.12.1
npm version (npm -v): 9.6.4
OS: Mac
Browser: Firefox, Simple Browser
Database: DuckDB

Expected Behavior

HMR meaning the page does not reload.
Instant update of page content

Actual Behaviour

Serveral seconds to display update
hard page reload

2:55:49 PM [vite] page reload /Users/archie/Projects/template/pages/index.md
2:55:49 PM [vite] hmr update /src/pages/+page.md, /<EMAIL_ADDRESS>/src/app.css
2:55:50 PM [vite-plugin-svelte] ssr compile done.

2:55:49 PM [vite] hmr update /src/pages/+page.md, /<EMAIL_ADDRESS>/src/app.css
The fact that the css is bring hmr updated puzzles me.

Breaking commit was 200a5eda01c681ae9ee6e05d938cd18391f8db60, for some reason just switching content: ["./src/**/*.{html,js,svelte,ts,md}"] -> content: ["./src/**/*.{html,js,svelte,ts,md}", "../../pages/**/*.md"] breaks HMR.


Incompatible with datasette-leaflet-geojson

With both plugins installed, cluster map stops working.

Solution is to upgrade Leaflet in datasette-cluster-map to 1.5.1, the latest release.

Upgraded in https://github.com/simonw/datasette-cluster-map/commit/a1ec67b5b3bf5cf315b14cefe2238ced82c909b9

Released as 0.7

Confirmed fixed.


Don't call ffmpeg with -x with video splited

Hi all, well here a little thing, some pages like youtube actually split the videos in videa and audio, so in this case youtube-dl shouldn't call ffmpeg.
(-x option) && (page with splited videos) && ((audio format not specified) || (audio format = to the downloaded format)) -> don't call ffmpeg
a logic explanation :D (&& and, || or)
Thx.

That's what it does by default:
 youtube-dl --ignore-config -v 'http://www.youtube.com/watch?v=BaW_jenozKcj' -x
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--ignore-config', u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj', u'-x']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Git HEAD: 62d231c
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: avconv 11.4, avprobe 11.4, ffmpeg N-42628-g6662ca2, ffprobe N-42628-g6662ca2, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] BaW_jenozKc: Downloading webpage
[youtube] BaW_jenozKc: Downloading video info webpage
[youtube] BaW_jenozKc: Extracting video information
[youtube] BaW_jenozKc: Downloading DASH manifest
[youtube] BaW_jenozKc: Downloading DASH manifest
[debug] Invoking downloader on u'...'
[download] youtube-dl test video ''_ä↭𝕐-BaW_jenozKc.m4a has already been downloaded
[download] 100% of 310.24KiB
[ffmpeg] Correcting container in "youtube-dl test video ''_ä↭𝕐-BaW_jenozKc.m4a"
[debug] ffmpeg command line: avconv -y -i 'file:youtube-dl test video '"'"''"'"'_ä↭𝕐-BaW_jenozKc.m4a' -c copy -f mp4 'file:youtube-dl test video '"'"''"'"'_ä↭𝕐-BaW_jenozKc.temp.m4a'
[debug] avconv command line: avprobe -show_streams 'file:youtube-dl test video '"'"''"'"'_ä↭𝕐-BaW_jenozKc.m4a'
[ffmpeg] Post-process file youtube-dl test video ''_ä↭𝕐-BaW_jenozKc.m4a exists, skipping

If that's not what you want, please elaborate on what you need.

mm, i'll explain a little more.
In windows we don't have always the ffmpeg specially for normal users, so if we want download for example the audio of a youtube video we do 'youtube-dl "some video" -x' and it download the m4a file, but then it try to call ffmpeg, and if only need the audio it feels don't is necesary, bacause when it don't found it send the error messsage, obvs we can use the -i option but we will skip other problems we can be interested, for example an audio don't is downloaded.

Ok, but note that some player doesn't play or have problems playing the m4a audio file generated by YouTube and ffmpeg is required to fix the file.

yes, but if the m4a file is corrupted ffmpeg will can't fix it, but if the player want other format to can reproduce it there call ffmpeg.


Done with all changes for login and signup page #82

Final Preview of Login and Signup Section
https://github.com/user-attachments/assets/72358635-a8e5-4eca-ab51-4c787c69a067
Task Done Successfully ✅

Thank you for your contribution! If you like the project, don't forget to give it a star.

Would like to contribute more in future.

Would like to contribute more in future.


Transparent Proxy Port Forward Rule Failing

Creating a transparent proxy for Tor port 9040 using the interface's network addresses (<IP_ADDRESS>/24) as the permissible source, any source and dst port, and all of it being forwarded to <IP_ADDRESS>:9040. Same setup as the transparent HTTP proxy writeup in the docs, just the dst port is "any" in the original packet.
Enabling this DNAT along with its FW rule generates:
opnsense: /usr/local/etc/rc.filter_configure: New alert found: There were error(s) loading the rules: /tmp/rules.debug:58: syntax error - The line in question reads [58]: nat on em0_vlan3 inet proto tcp from <IP_ADDRESS>/24 to <IP_ADDRESS> port -> (em0_vlan3)

That seems to be missing the dst_port on localhost (9040), and complaining about that fact. Am i doing something wrong or is the UI passing the wrong data to the firewall? This is all on a current version (updated this morning).

As per web proxy template dst port should be "(other)" with 9040 set, also for the target redirect port. "any" seems weird here, it'll fail, maybe just because "port" is in there when it should match all or explicitly state the full range ("any").
Maybe @AdSchellevis knows this better.

So the matcher for the rule is TCP/IP (<IP_ADDRESS>/24:any) -> TCP/IP(any:any), the redirect target is TCP/IP(localhost:9040), at least in the firewall GUI. What it produces doesnt seem to be that. :)
Thanks for looking into this

It would probably work if you did "(other)" from 1 to 65535. We'll fix the "any" to do the same one way or another, but I don't want to decide.

lots of things have changed in the rule generation, this might be fixed in the process


Do not fail when :ok or {:ok, dict} is not returned from callbacks.

Nice fix, thank you very much!


⚠ Samsung has degraded performance

In c0e29e7, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3133 ms

Resolved: Samsung performance has improved in 4252a9c.


user can't clear Amount in withdraw modal

accidently created this in augur-ui, creating here.
Input doesn't allow the user to backspace value from modal.
to repro,
type in 1 in Amount then backspace, the 1 doesn't get removed.

This is still not working.

fixed - closing


Pass a non-const  MerkleTree as a pointer.

To avoid surprises, as recommended by the style guide:
https://google.github.io/styleguide/cppguide.html#Reference_Arguments

For the record, this is done to comply with the style guide.

I'll add a CHECK_NOTNULL before merging.


WebUI - after logging out and refreshing the site you are still logged in

WebUI - after logging out and refreshing the site you are still logged in

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten


Idea: Add a function to get the current mouse position

Use case
Adding a way to retrieve the current mouse position may at first sound like something unneeded. After all, you can already use MouseRegion to get the exact same functionality. But there is a problem here.
What if you want to use a CustomPainter that runs at a consistent frame rate using a Timer. Then it's impossible for you to accurately get the mouse position from the MouseRegion widget because it will be desynchronized with the  current frame. This is a problem because as soon as you want something to follow the mouse or even replace the mouse cursor with a custom one, the object following the mouse is gonna flicker constantly.
For this reason, it would be really useful to have a custom method that enables the user to just retrieve the current mouse position in the window. It's gonna enable people to build more interactive experiences using the CustomPainter that aren't restricted to the default frame rate. For example, in my app, I have a tabletop simulation that runs in multiplayer. Currently, when you drag an object on the table, it doesn't follow the mouse properly.  Through the implementation of the method I suggested, I would then be able to make it follow the mouse better.
Proposal
As said above, I would like a method to retrieve the current mouse position unrelated to the current frame. You should be able to call something like getMousePosition() inside of a CustomPainter widget and just get back the current position of the mouse, no extra widget required. I'm not all to familiar with the Flutter engine which is why I don't know where exactly such a method would fit. In my eyes, it should just be a global function from some class. It would be cool, if someone reading this would suggest a better place for such a function.

Hi @Unbreathable

Then it's impossible for you to accurately get the mouse position from the MouseRegion widget because it will be desynchronized with the current frame.

Could you share an example of this issue? Have you tried Listener?
Also, this looks somewhat similar to https://github.com/flutter/flutter/issues/134652. Please check it to see if it's also your case.

Alright, sorry for the late response, I had some things going on in my life that prevented me from responding sooner. Anyways, I've checked out #134652 and it seems like this is unrelated. I don't want to capture the mouse pointer in a certain area, I just want to be able to get an accurate mouse position for every frame in the CustomPainter.
Here is an example app:
class CustomMouseCursor extends StatefulWidget {
  const CustomMouseCursor({super.key});

  @override
  State<CustomMouseCursor> createState() => _CustomMouseCursorState();
}

class _CustomMouseCursorState extends State<CustomMouseCursor> {
  int method = 0;
  Offset mousePosition = const Offset(0, 0);

  @override
  void initState() {
    super.initState();
  }

  void recalibrate() {
    setState(() {
      mousePosition = const Offset(0, 0);
    });
  }

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        backgroundColor: Colors.black,
        body: SizedBox(
          width: double.infinity,
          height: double.infinity,
          child: Builder(builder: (context) {
            if (method == 0) {
              return MouseRegion(
                onHover: (event) => setState(() {
                  mousePosition = event.localPosition;
                }),
                child: CustomPaint(
                  painter: MousePainter(mousePosition),
                ),
              );
            }

            return Listener(
              onPointerHover: (event) => setState(() {
                mousePosition = event.localPosition;
              }),
              child: CustomPaint(
                painter: MousePainter(mousePosition),
              ),
            );
          }),
        ),
        floatingActionButton: FloatingActionButton(
          child: const Icon(Icons.add),
          onPressed: () => setState(() {
            method = (method + 1) % 2;
          }),
        ),
      ),
    );
  }
}

class MousePainter extends CustomPainter {
  final Offset mousePos;

  MousePainter(this.mousePos);

  @override
  void paint(Canvas canvas, Size size) {
    canvas.drawCircle(mousePos, 10, Paint()..color = Colors.red);
  }

  @override
  bool shouldRepaint(covariant CustomPainter oldDelegate) {
    return true;
  }
}

The problem may not be obvious at first glance, but when looking at the red circle while moving the mouse, one notices that it's always one frame behind. Like in this video:
https://www.youtube.com/watch?v=GQmlhStfhEk (my mkv recording wasn't accepted by GitHub, so I uploaded it to YouTube instead)
Both Listener and MouseRegion have this problem, in the sample app I made, you can switch between the two using the FloatingActionButton.
The only way to fix this in my opinion is to include the mouse position with the Canvas API natively somehow. This isn't even a bug right now, because when receiving the new mouse position from the Listener or MouseRegion, you call setState to update the mouse position. Problem here is though, that this requests a new frame and calls all kinds of other stuff to run which causes this delay between the mouse moving and the red circle actually showing up at the right position. So again, to overcome this limitation of the framework, you would need to retrieve the mouse position before rendering the actual frame (in the paint method itself) with the CustomPainter and then the delay will most likely no longer be there.

Thanks for the update with detailed information. Labeling the issue for other's thoughts.


Provision EC2 with FSXN filesystem

Is your request related to a new offering from AWS?
I would like to know if there is a way to provision EC2 instances with FSxN as it can be done with AWS Console.
Due to the fact that I don't see any reference in the documentation, this feature is not available yet.
Describe the solution you'd like.
I would like to automate EC2 provisioning with FSxN filesystem.

If FSxN requires an EC2 instance, you can provision it using this module.
Unfortunately, this repository is not the best place to ask "How?", since we are focusing efforts on module maintenance in the first place.
Please ask on designated forums like - https://discuss.hashicorp.com/ or https://repost.aws/

FSxN does not require EC2, but there is an option to provision EC2 with the filesystem FSxN. And this option does not exist in terraform provider. Therefore this is a feature request or a request for the right documentation.


example of interpret with multiple arguments and function name

would suggest better name for an example, "invoke"
so that somebody could run it easier

@NikVolf good points, thank you!


Remove numpy legacy random support.

Numpy dropped support for v1.19 on june 21st. This removes the legacy randomness support for Numpy v1.19.

Yeah, you're right, as long as the legacy API is still in numpy, we should continue to support it. But we should keep it in mind and revisit it when numpy removes the API completely.


⚠ Bofrost.de has degraded performance

In 3bef313, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1899 ms

Resolved: Bofrost.de performance has improved in 32f2ea9.


python3Packages.pynanoleaf: 0.0.7 -> 0.1.0

Motivation for this change
Update to latest upstream release 0.1.0
Change log: https://github.com/Oro/pynanoleaf/blob/master/CHANGELOG.md
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118230 run on x86_64-linux 1

2 packages built:

python38Packages.pynanoleaf
python39Packages.pynanoleaf

Result of nixpkgs-review pr 118230 at 419443a7 run on aarch64-linux 1

2 packages built successfully:

python38Packages.pynanoleaf
python39Packages.pynanoleaf


feat: update home page design

This PR updates the style and design of the home page.

Sorry I know this is a little late in the process, but can we had a section like the one below to the website under the first section? "We are a proud CNCF project"
Some people don't realize this and it would be great to get on the website

Just fyi, I updated Docsy and Hugo version, you might need to rebase on top of main (for that PR https://github.com/cilium/tetragon/pull/2497). I checked, and nothing breaks on your homepage, you just need to merge some commits during the rebase but it should be trivial, just accept both changes all the time.


Windows invalid extension Id case

Hi @sttz ,
The structure of manifest as below:
struct NativeMessagingManifest
{
public string name;
public string description;
public string path;
public string type;
public string[] allowed_extensions;
}
But my windows10 com.expressvpn.helper.json as Below:
{
"name": "com.expressvpn.helper",
"description": "ExpressVPN Chrome Helper",
"path": "C:\Program Files (x86)\ExpressVPN\expressvpnd\expressvpn-browser-helper.exe",
"type": "stdio",
"allowed_origins": [
"chrome-extension://fgddmllnllkalaagkghckoinaemmogpe/",
"chrome-extension://olhkmmhcabcohlcodnbkfoekafimgkmb/",
"chrome-extension://gicmddjpmepnbabhkhihlgdfccbppflm/"
]
}
If I  change allowed_origins to allowed_extensions, the program still promt "invalid  chrome extendsion id!"
I have check the extendsion installed in chrome, it is the first one in the list. May I know this program tested under windows? or this program only compatible with specific version ExpressVPN?

In Widnows, the origins is the first parameter. I solved this by Google

In Widnows, the origins is the first parameter. I solved this by Google

Yeah, I've only tested on macOS. If you get it running on Windows, a PR would be welcome!

Yeah, I've only tested on macOS. If you get it running on Windows, a PR would be welcome!

In Widnows, the origins is the first parameter. I solved this by Google

Could you share your solution if possible? I'm looking for a CLI client as well on Windows as I need to reconnect programmatically with ExpressVPN

I've somewhat solved this. You need to change all references to "com.expressvpn.helper" to "com.expressvpn.helper.firefox". On Windows they are two separate files. You also need to add "C:/Program Files (x86)/ExpressVPN/expressvpnd" to the variable manifestBasePaths.


Teams Rooms for Android - Wrong Product

[We are setting Logitech Roommate device and found this page to setup auto-answer.  The documentation says "Microsoft Teams Rooms on Android" which is what this device is.  We also had a meeting with Logitech since it did not work and they confirmed this is a missing feature for video calls on Teams Room devices.  I am wondering if this article is incorrect when it said Room systems.]

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

ID: ffdd3ead-73c3-b335-0d98-b197e06f7fe9
Version Independent ID: ffdd3ead-73c3-b335-0d98-b197e06f7fe9
Content: Set up auto answer for Teams Android devices - Microsoft Teams
Content Source: Teams/set-up-auto-answer-on-teams-android.md
Service: msteams
GitHub Login: @mkbond007
Microsoft Alias: mabond

@kreischl68 were you able to see my comment here? Thanks

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


fix: [migration] fix dashboard json_metadata when native filter is not enabled

SUMMARY
This PR will scan all dashboard json_metada: if DASHBOARD_NATIVE_FILTERS feature is not enabled, remove show_native_filters and native_filter_configuration properties from json_metadata.
See https://github.com/apache/superset/pull/14933#issuecomment-860168672 for more context.
TESTING INSTRUCTIONS
CI and manual test.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[X] Includes DB Migration (follow approval process in SIP-59)

[X] Migration is atomic, supports rollback & is backwards-compatible
[X] Confirm DB migration upgrade and downgrade tested
[X] Runtime estimates and downtime expectations provided: a few seconds up to 1 minute, depends on the number of dashboards to scan.

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I agree with @betodealmeida that conditional migrations based on feature flags may lead to unintended results.
If the DASHBOARD_NATIVE_FILTERS feature flag is disabled, what is the impact of having show_native_filters and native_filter_configuration values set? Shouldn't they just be ignored?

I agree with @betodealmeida that conditional migrations based on feature flags may lead to unintended results.
If the DASHBOARD_NATIVE_FILTERS feature flag is disabled, what is the impact of having show_native_filters and native_filter_configuration values set? Shouldn't they just be ignored?

Please see my reply to Beto. It will impact the decision after we start migrating filter_box to filter component.

The concerns that have been raised are valid in the sense that if someone who has been using NF for a long time happens to turn off the FF during the migration, it might end up wiping out the existing filter configurations. To mitigate this and remove the dependency on the feature flag, I propose the following:
If, and only if, the following properties are set to the following:
"show_native_filters": true,
"native_filter_configuration": [],

the properties are removed. Otherwise we don't touch them.
This won't have any effect on dashboards for envs that have the native filters feature flag enabled, nor will it impact envs that have it disabled.

@betodealmeida @robdiciuccio Do you feel more comfortable with @villebro's suggestion?

i discussed this solution with @villebro, i decided to keep this change inside airbnb.
for open source users, if they happened to have a deployment during bad time, they can contact us to get a solution.


jacdac branding is not consistent

this page  https://microsoft.github.io/jacdac-docs/overview/ looks like this:

but it should look like this:

here is the SVG to use:

these other pages also have a bad font:
https://microsoft.github.io/jacdac-docs/clients/
https://microsoft.github.io/jacdac-docs/tools/
https://microsoft.github.io/jacdac-docs/overview/connectorcable/
the correct font is 'Sen (Regular)'

Removed icon so that it does not look like a logo. We need to do a review of the other font before using it.


Migrate AppContainer to new Context API

Summary
1/4 of https://github.com/facebook/react-native/issues/28103

Changelog

[General] [Fixed] - Migrated AppContainer to new Context API.
Test Plan

No flow errors, built RNTester successfully.

Thanks for working on this!
It looks like it is pretty common to consume this rootTag context downstream. Class components access this.context.rootTag to pass to native modules for example.
I think this would be a fairly substantial breaking change as is, do you have any ideas of how to make this backwards compatible and make it easy to migrate?
For an example, take a look at Modal.js, I think one of the usages of context there is to receive this context type.
I'd love to see your test plan include the RNTester examples for Modal.
Ideally, first without any changes to the JS code and a warning being printed recommending to use the new way to consume this API, and second changing the Modal code to consume the context in the new way.
This will hopefully make sure the necessary pieces are exported for downstream consumers to use the context, and for it to be backwards compatible for now.

I see, thank you for the context @TheSavior!
Not sure if there is a way to make it backwards compatible. With this change downstreams need to access the rootTag from the AppContainer's Consumer. Is a warning sufficient enough to address this breaking change?

To make it backwards compatible could you leave the existing code:
static childContextTypes:
    | any
    | {|rootTag: React$PropType$Primitive<number>|} = {
    rootTag: PropTypes.number,
  };

  getChildContext(): Context {
    return {
      rootTag: this.props.rootTag,
    };
  }

but make rootTag in here a getter?
    return {
      rootTag: this.props.rootTag,
    };

And then also render <Context.Provider value={{rootTag: this.props.rootTag}}>?

Makes sense, since keeping the legacy Context API is not an issue, no need to break usage over it.
Just need to migrate the flow typing to not use PropTypes with this.

I think it is fine to leave the usage of prop-types for now, as long as we know we are making progress towards removing it. I'm not sure if the React API requires the usage of prop-types for the old context API. If we can avoid it, great! Otherwise, I'm not so worried as long as we have a plan.

Great! Sounds like a plan I’ll make the changes & update the Test Plan.
Learned a new mindset here. Should be thinking about backwards capability with any migration.

I made the changes to warn with the getter.
Unfortunately, getChildContext gets called with any prop or state change: https://reactjs.org/docs/legacy-context.html#updating-context
So even using the new Context API, the warning still gets thrown.
cc @TheSavior

Any progress on this one?

I'll finish this up tonight! @TheSavior

@TheSavior made the updates per code review.
With the getter it seems to still fire when the function is called.
The only other usage of AppContainer is in renderApplication.js which receives rootTag from the renderApplication() function which does not access through the Legacy API.
<AppContainer
  rootTag={rootTag}
  fabric={fabric}
  showArchitectureIndicator={showArchitectureIndicator}
  WrapperComponent={WrapperComponent}
  internal_excludeLogBox={isLogBox}>
  <RootComponent {...initialProps} rootTag={rootTag} />
</AppContainer>

Is it warranted to introduce the breaking change given this?

Let’s avoid the breaking change. We have a couple hundred callsites internally at FB that we will need to migrate so I imagine other people are using it too.

sigh. It looks like the new context was already added in July of last year https://github.com/facebook/react-native/commit/f9ecce7440e712782f4fd760a46e54b3102914d8
i'm so sorry for not realizing this earlier. You have helped me learn about the core.

hmm I see, so Modal should be migrated to use the rootTag provider instead it being consumed from AppContainer, then remove the legacy context from AppContainer?

Modal needs to use RootTagContext, but we still can't get rid of the legacy context api in AppContainer yet because we have hundreds of internal callsites still using it that need to get migrated to RootTagContext as well.

Got it, makes sense!

@safaiyeh still interested? :) Recently Modal was migrated in #29002

Alrighty, I’ll check it out! @thymikee


Parameters: { evals_result } might not be used.

environment：
macOS 10.15.4
$ python -V
Python 3.7.7
$ pip freeze
...
xgboost==1.1.0
...
code:
evals = ['auc', 'logloss']
eval_set = [(train_x, train_y), (test_x, test_y)]

model = XGBClassifier(
    learning_rate=0.1,
    n_estimators=30,
    max_depth=5,
    min_child_weight=1,
    gamma=0,
    subsample=0.8,
    colsample_bytree=0.8,
    objective='binary:logistic',
    nthread=8,
    scale_pos_weight=1,
    seed=1024,
    evals_result=evals
)

model.fit(train_x, train_y, eval_metric=evals, eval_set=eval_set)

warning message：
[10:30:44] WARNING: /Users/travis/build/dmlc/xgboost/src/learner.cc:480:
Parameters: { evals_result } might not be used.
This may not be accurate due to some parameters are only used in language bindings but
passed down to XGBoost core.  Or some parameters are not used but slip through this
verification. Please open an issue if you find above cases.

I believe this is a correct message.  Passing evals for eval_metric is sufficient.  There's no evals_result on scikit learn interface.


Improve node.children handling

I'm dealing with very large SVG pictures which I need to shrink. My setup is running SVGO in AWS Lambda through API Gateway, which puts my time limit to 29 seconds, which in several cases is not enough for the optimization.
I've started delving deeper into where the time goes, since in my mind the optimization process should be linear in several of the preset plugins. I have an example SVG with around 46000 lines/polylines/what have you (which eventually shrink to 27 paths) and even the simplest plugins seem to take 500 ms on average. This is caused by the following .includes check in xast.js:
    if (parentNode.children.includes(node)) {
     for (const child of node.children) {
        visit(child, visitor, node);
      }
    }
Although I'm sure this check is there for a reason, removing it had no effect on my sample data result aside from removing 15 seconds out of the 19 it previously took. The way xast.visit function currently works it's not possible to, for example, use a Set to handle the validity of nodes. Handling the deletion of children in plugins is also fairly ineffective, sometimes using splice with indexOf and sometimes xast.detachNodeFromParent.
Introducing an index Set to each node, refactoring xast.js and preset plugins to support the index Set is easy, but this change would possibly break existing self-made plugins. My goal here is to start a small discussion about this, since there are easy performance improvements to be gained.
What does removing the if-clause break? Should the plugins be advised not to modify their siblings (so the check is unneccessary)? One possible solution might be to offer an interface to get .next() and previous() siblings for the plugins, and then tag the nodes deleted and only delete them after the forementioned for-loop has ended.

The rub is that visit is never called with parentNode except from itself (from my quick visitation). In theory, removing the clause should break nothing.

Enforcing the usage of visitSkip would allow the if-clause to be deleted. Just using it and leaving the if-clause in would not benefit in this situation. I believe that if-clause can be avoided with easy restrictions and there's good performance improvements to be gained.
Another issue I have is with detachNodeFromParent. Merging 46000 paths into a handful calls it a lot and the function goes through parentNode.children every time (although shrinking it in every call, but it's still a lot). My current solution is to copy mergePaths.js to my own plugin and modify the way it handles child deletion and path-merging. This more than halved the runtime of svgo alltogether.

Another issue I have is with detachNodeFromParent. Merging 46000 paths into a handful calls it a lot

mergePaths doesn't use detachNodeFromParent at all right now. Are you on an older version?

I opened this issue because of the problematic if-clause. It's O(n²) and there are no ways around the problem with custom plugins. Like you suggested, removing the check and using visitSkip any time the current node is deleted would solve the issue, although it would cause problems with existing custom plugins.
As for mergePaths, I see it doesn't use detachNodeFromParent anymore, but due to the excess updatePreviousPath calls, it's slow. This is a separate issue.

UPDATE: since there is no way to bypass this problem with the custom plugin codes itself, I did the following:

copied svgo.js, xast.js and plugin.js to my project
set my project to import { optimize } from ./custom-svgo/svgo.js
fixed imports in the three files:

in svgo.js, imported invokePlugins from ./plugin.js
in plugin.js, imported visit from ./xast.js
in xast.js, imported visitSkip from the original package
other imports seemed to work when prefixed with svgo/lib/ (as in, import { builtin } from 'svgo/lib/builtin.js')

removed the if-clause from xast.js
only plugin I am using affected by the if-clause seemed was removeEditorNSData, so I copied the source and had it return visitSkip when node was removed.

notes:

my monstrous SVG with 46,000 paths is now ~5 seconds compared to the previous 20
haven't noticed any broken SVG files so far, out of several thousands
this has to be the ugliest solution I've done since beginner courses in University
detaching a node is still done with an ineffective .filter (same with replacing node with it's children, which is done with .indexOf and .splice). I'm very sure there's a faster solution available where you return the node(or nodes) from enter/exit functions and then use that information to recreate the child-list in xast.js. I tried it quickly but couldn't wrap my head around it, and due to the urgency of this, I settled for the forementioned dumb solution.

The thing is that of course there isn't any breakage because it's deleted anyway. I bet this change could slow down some SVGs where the hidden elements have many many children.

Well ideally it should always return visitSkip. I might be misinterpreting your comment, because what I got from it is that you only added a new visitSkip to one plugin.

Ah, I see. I also removed the if-clause from xast.js and manually added all the necessary files to my project up until that.
So TLDR: my fix does not have the if-clause anymore, and the only problem that I got taking it out was the one plugin, which I fixed by adding visitSkip.
I know I'm a bad explainer and having to translate everything to english adds an extra spice to it, but I'll try to be more precise in the future.
That being said, if you decide to fix this, I'll be glad to assist. I'd even love to get the detachNodeFromParent issues fixed too, but that requires a structural change.


Remove Public IP switch from the deployment details in case the deployment doesn't have one

Description
The public IP switch should be removed entirely from the deployment details if the deployment doesn't have one.

The current implementation may give the user a wrong idea that a public ip could be added after the deployment is complete.
If removing the field entirely will not work then it should be replaced with a field that contains - or N/A or Option was not selected but hiding it would be a better option.

Deployed a VM without any ip and still shown as the issue screen not updated.

Verified on devnet.
The toggle button was removed from the deployment details.

Deployment without a public ip

Deployment with a public ip


driver: regulator: npm1300: fix build warning

Initialize prev. uninitialized variable in regulator_npm1300_init().

Variable is initialized, compiler likely not smart enough.

Well, the error is checked as follows:
https://github.com/zephyrproject-rtos/zephyr/blob/b46ca11c4c255a8823705bdfda156edb63ac6158/drivers/regulator/regulator_npm1300.c#L591-L593
but get_enabled_reg() contains:
https://github.com/zephyrproject-rtos/zephyr/blob/b46ca11c4c255a8823705bdfda156edb63ac6158/drivers/regulator/regulator_npm1300.c#L534-L540
so if mfd_npm1300_reg_read() returned a value > 0, enabled would end up uninitialized. For consistency, it would be better to change the latter check to ret < 0.

indeed, great explanation :-)

Created a new PR by request of @maxd-nordic with the suggested fix:
https://github.com/zephyrproject-rtos/zephyr/pull/80276

alt https://github.com/zephyrproject-rtos/zephyr/pull/80276


Meteor 1.6 common.gypi not found

On Meteor 1.6 beta 27, while node-gyp is building node-sass in:
materialize_materialize\0.100.2\npm\node_modules\.temp-3nmk2a.l4ujd\node_modules\node-sass\build'
common.gypi is not found in:
1.6.0-beta.27\mt-os.windows.x86_32*dev_bundle*\common.gypi
However common.gypi is present in:
materialize_materialize\0.100.2\npm\node_modules.temp-3nmk2a.l4ujd\node_modules\node-sass\common.gypi
It seems that, while building node-sass, node-gyp loses track of the current path and reverts to the meteor npm default path.
Using version 0.100.2 on Windows 10 x64.
I realize Meteor 1.6 is in Beta, but just wanted to make mention of this, should anyone else be running into the same problem (took me some time to pinpoint).
Also I am not sure whether this bug should be filed here, or its is a result of some of the dependencies of this package (fourseven:scss or node-sass).

This issue should be reported in the node-sass project.


Update cime-env to 1.1.0, use latest evv4esm (0.2.3)

This merge also

updates hdf5 and libnetcdf to be consistent with conda-forge pins
updates netcdf4 to the latest release
constrains python >= 3.7
removes pinned json_tricks (which is a livvkit dependency, not a CIME or E3SM one).

@mkstratos, version 1.1.0 has been deployed on Cori, Compy, Anvil/Chrysalis and Cooley.  It will load with the load_latest_cime_env.sh script on each machine.  Let me know how it goes so I can merge this PR if all is in order.

@mkstratos, version 1.1.0 has been deployed on Cori, Compy, Anvil/Chrysalis and Cooley. It will load with the load_latest_cime_env.sh script on each machine. Let me know how it goes so I can merge this PR if all is in order.

The new environment is working for me (EVV runs and generates pages correctly) on Cori, which is as far as my access goes.

@mkstratos, okay.  I'll assume that others will let me know in the future of there are issues on other E3SM machines.  Thanks for testing.


fix(challenges): typos and verbiage in previous fixes

Refers review comments on https://github.com/FreeCodeCamp/FreeCodeCamp/pull/10172 and https://github.com/FreeCodeCamp/FreeCodeCamp/pull/10147

Thanks for fixing those @raisedadead :heart:


How to use this with letsencrypt and assign domain name as well

Is it possible to embed letsencrypt certificate creation and also assign domain name like www.example.com?

Yeah, maybe in the future, but not now.

just updating, working on ubuntu 18.04 2020
generate your cert , and Combine both certificate and private key in one file.
certbot certonly --webroot -w /mnt/data -d example.com -d www.example.com
cat /etc/letsencrypt/live/example.com/cert.pem /etc/letsencrypt/live/example.com/privkey.pem > /etc/letsencrypt/live/example.com/web.pem
ran should load the certs, if you have issue, recheck your cert paths .
/usr/bin/ran -cert=/etc/letsencrypt/live/example.com/cert.pem -key=/etc/letsencrypt/live/example.com/web.pem -tls-policy=redirect  -tls-port=443 -root=/mnt/data -port=80 -index=index.html -listdir=true -serve-all=true cors=true -gzip=true


How to use custom built ckeditor?

we have an option to build the ckeditor as we like it. but how would we be able to use that version with this?

Assuming your ckeditor resource package was placed at /yourapp/static/ckeditor, you can use custom_url in load():
<head>
{{ ckeditor.load(custom_url=url_for('static', filename='ckeditor/ckeditor.js')) }}
</head>

i tried it and it worked. thanks.

:)


📚 Review GovAssure doc

Review the following GovAssure Doc
Definition of done

[x] doc reviewed and Simon updated

Reference
How to write good user stories

Information and links to AP-Docs sent over to Simon for GovAssure doc. We need to make sure we are adhering to the Secure by Design tools. Will raise another issue to investigate


feat(Radio): changes color and adds hover behavior

Overview

changes the color of radio buttons to purple
adds a purple border (box shadow) on hover for unchecked buttons
makes the purple border (box shadow) thicker on hover for checked buttons

PR Checklist

[x] Related to designs: https://drive.google.com/file/d/1cZcB9osjWg0RoNRknZxqWkwJ1aUTZHVv/view and https://www.figma.com/file/xaosTDGWkXtTJBLPDV7ME2/Checkout---Clarity-improvments?node-id=389%3A4
[x] Related to JIRA ticket: [GROW-2105(https://codecademy.atlassian.net/browse/GROW-2105)
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

🚀 Styleguide deploy preview ready!https://602d6e2dbf1ccc45ea571e91--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.45-alpha.00eb4e.0@codecademy/gamut-labs@10.0.12-alpha.00eb4e.0@codecademy/gamut@23.2.2-alpha.00eb4e.0<EMAIL_ADDRESS>
wait shoot i didnt realize it automatically adds reviewers, was not ready for review 😬

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.45-alpha.875cc1.0@codecademy/gamut-labs@10.0.12-alpha.875cc1.0@codecademy/gamut@23.2.2-alpha.875cc1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602d8a214a42904df3cc8c66--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.45-alpha.ebdcd9.0@codecademy/gamut-labs@10.0.12-alpha.ebdcd9.0@codecademy/gamut@23.2.2-alpha.ebdcd9.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.45-alpha.084b8d.0@codecademy/gamut-labs@10.0.12-alpha.084b8d.0@codecademy/gamut@23.2.2-alpha.084b8d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602daec6edaee90fe4966006--gamut-preview.netlify.appDeploy Logs

okay actually ready for review now!

LGTM! Always fun to work on CSS animations. 😄
I found it annoying to have to refresh the page to trigger the animation again... if you want ✨ bonus points ✨ you could add another option or two to the story.

i was about to do that but realized we have that in the radio group story

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.45-alpha.4fe8a3.0@codecademy/gamut-labs@10.0.12-alpha.4fe8a3.0@codecademy/gamut@24.0.0-alpha.4fe8a3.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602ea783d2de3516b1cdf7e9--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.46-alpha.4d5a61.0@codecademy/gamut-labs@10.0.13-alpha.4d5a61.0@codecademy/gamut@24.0.1-alpha.4d5a61.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602ea9dc16ddfe5317640615--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.46-alpha.86c3c6.0@codecademy/gamut-labs@10.0.13-alpha.86c3c6.0@codecademy/gamut@24.0.1-alpha.86c3c6.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602eb7e8ddb4bb0da83b1cf8--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.46-alpha.88dcca.0@codecademy/gamut-labs@10.0.13-alpha.88dcca.0@codecademy/gamut@24.0.1-alpha.88dcca.0@codecademy/styleguide@28.0.1-alpha.88dcca.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602eba3dddb4bb145c3b1821--gamut-preview.netlify.appDeploy Logs

Wow this looks so good compared to the current radio buttons! Question: Will there always be a default chosen so we will never have a true error state hence why the error looks the same as a regular option? By true error state I mean something like this:

Wow this looks so good compared to the current radio buttons! Question: Will there always be a default chosen so we will never have a true error state hence why the error looks the same as a regular option? By true error state I mean something like this:

great question, @zippyzow
for checkout there is always a default, not sure about the other instances though, but i will raise with tamar

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.47-alpha.43cc85.0@codecademy/gamut-labs@10.1.1-alpha.43cc85.0@codecademy/gamut@25.0.1-alpha.43cc85.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602fe5eab3b0d60094c9cda2--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-illustrations@0.4.47-alpha.05d127.0@codecademy/gamut-labs@10.1.1-alpha.05d127.0@codecademy/gamut@25.0.1-alpha.05d127.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://602ffda4859d570cc323bd04--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-labs@10.1.3-alpha.fdd314.0@codecademy/gamut@25.1.1-alpha.fdd314.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6033ec7bf7eaa50ae051a1d9--gamut-preview.netlify.appDeploy Logs


Bug: ImportError: DLL load failed

ImportError: DLL load failed
doesnt show the name of the DLL which failed

Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary?
Make sure you also include the exact command if possible to produce the output included in your test case. If you are unclear what to include see the issue template displayed in the Github new issue template.
We ask for this in the issue submission template, because it is really difficult to help without that information. Thanks!

I think it was resolved. Closing due to lack of recent activity. Please open new ticket if you see similar issue. Thanks!


Rate Limit Exceeded - can I just download the parser?  (REMOVE)

Nevermind.
Please remove, I'm just going to avoid making 60 requests per hour.

Hi Jon,
I think that is something happening behind the scenes, not in my code but
in some previously existing code called by mine.  I wasn't even aware that
the code would go to a server to do any of the work. I thought it was
stand-alone code.  I'm disappointed that it accesses a server.  I would
rather that my code not stop working because some server is unhappy or has
a new API or maybe even is no longer online.
Sounds like the server will accept more API calls on an authenticated
connection, but I don't make any such connection in my code.
Sorry for the inconvenience, but I can't help in this case.
Cheers,
Craig
On Sat, May 27, 2017 at 6:26 AM, jonandrewdavis<EMAIL_ADDRESS>wrote:

Hey there, thanks for the tool. Was working great.
I noticed my code blocks weren't formatted well, so I switched to: var
Markdown = require('markdown-to-html').GithubMarkdown;. Otherwise my code
is default.
Great it worked, but after 10 minutes, I got:
{"message":"API rate limit exceeded for <IP_ADDRESS>. (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)","documentation_url":"https://developer.github.com/v3/#rate-limiting"}
Is there a way around this? I don't need the most updated api... I just
want good github flavored markdown... Can I use the original class and add
a flavor?
Thanks for any help. Feel free to delete this if its annoying.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cwjohan/markdown-to-html/issues/18, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AH5q3KXwACoLKChP1yS3AkNVv8qzl9xNks5r-CSAgaJpZM4NoWcn
.


MSID: 93830 Version: 2 DOI: 10.1101/2<IP_ADDRESS>9779

MSID: 93830
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>9779
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=93830
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>9779

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/93830v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>9779
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/93830/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

error: [journal-ref-article-title] This journal reference (id c2) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c3) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c6) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c7) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c8) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c9) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c10) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c11) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c13) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c14) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c15) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c16) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c18) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c20) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c21) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c22) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c26) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c28) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c30) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c31) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c33) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c34) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c36) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c37) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c38) has no article-title element.
error: [journal-ref-article-title] This journal reference (id c39) has no article-title element.
warn: [article-email-corresp-author-count-equivalence] The number of corresponding authors (2: Jeong-Oh Shin; Jong-Bin Lee) is not equal to the number of distinct email addresses (3<EMAIL_ADDRESS><EMAIL_ADDRESS>jwkim84@ewha.ac.kr). Is this correct?


⚠ SceneTime has degraded performance

In 52758e9, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 18564 ms

Resolved: SceneTime performance has improved in 5144ade.