Fix image insertion behavior

Various fixes related to image uploading & insertion:
Broken image uploading behavior if media is set in _config.yml
It appears that when 29b0f35 was merged, it accidentally broke image uploading if you've specified a media folder in your _config.yml.
How to reproduce

Edit a repo with a _config.yml that specifies a media folder as well as at least one recognized image file within config.media.
Attempt to upload a photo
src is [Object object]/{file}

The updateImageInsert function was determining path by prepending assets which is an object of all uploaded files, not the config.media path. This would result in the upload url being [Object object]/path when using the media uploader.
This fix

Resolves this error by reinstating use of the defaultUploadPath function (reused from the default file upload code)

Un-buildable metadata as result of image form use
When an image was uploaded to/selected from the modal in the metadata section, the value was prepended with {{site.baseurl}} which is improper as jekyll will not parse liquid syntax in front matter. A side affect of fixing this behavior is that image urls hosted externally now insert properly (as requested in  #842 & #1019)
How to reproduce

Insert an external image url eg. https://example.com/logo
Inserted markdown is `{{site.baseurl}}/https://example.com/logo``
or
Inset an image (internal or external) via the metadata view
Inserted path is {{site.baseurl}}/asset

This fix

Uses absolute paths in metadata (since Jekyll does not parse front matter)
Checks for external urls when inserting via the toolbar (starts with http[s]://)

Additionally I dd the following maintenance while I was poking around

Expanded the isImage classifier to include modern media formats based on Mozilla's Image format guide for the web
Expanded the isMedia classifier to include formats based on Mozilla's media format guide for the web

Although I tried to spend considerable time getting testing nailed down for this change, the lack of testing infra for toolbar & assetselection made this a no go. The good news is that defaultUploadPath has tests already (which are passing)

Force push to https://github.com/prose/prose/commit/a15b01f0f11f3a113a125125da6b41fefce33e00 due to unforseen issues with es6 conflicts (use of const)


glfw does not respect xkeyboard option caps:swapescape on Wayland

It's reproducible on latest sway (and probably any compositor that uses wlroots).
One needs to add these lines to sway configuration file to enable the option:
input "type:keyboard" {
    xkb_options caps:swapescape
}

After enabling it, pressing Caps Lock should be treated as pressing Escape and vice versa, but in applications that use glfw it does not happen.

This doesn't seem to be specific to wayland I have the same issue on X

The current GLFW keyboard input API is mostly concerned with the physical location of keys and less what they're currently mapped to. We will look at whether to add support for both views in 3.5, which will be focused on keyboard input enhancements.


Support cli.php -c value command -o option

I'd like to be able to add InputOptions with with VALUE_REQUIRED  to the Application::getDefaultInputDefinition because for some things it's just more natural for options to precede the command (eg; cli.php --config ~/.mycnf do_something --another option.
However the code relies on the token being the first argument (being the command) and does not take the InputDefinition into consideration for this.
I'm not entirely sure if this is the right way to solve it, I don't know if the Application::run just takes the 2 arguments for testing purposes or if there are other reasons, when I needed to do $application->getDefinition() in the tests I realised that if someone would feed Application::run a custom Input they'd have to do that too, which might be odd?

never, this PR seems to break the arguments after it

@fabpot any ideas on how I could get this behavory?


[PE-D] exit command accepts random args where there should be non

exit command accepts random args like list command.

Labels: type.FeatureFlaw severity.Medium
original: gweeyc/ped#10

Not a bug, according to UG -> Features notes, the App is programmed to behave as defined below:

Extraneous parameters for commands that do not take in parameters (such as help, list, exit and clear) will be ignored.
e.g. if the command specifies help 123, it will be interpreted as help.


'noUnusedParameters' compiler options warn the specified 'this' type as 'never used' parameter

Environment

TypeScript Version 2.0.0-dev.20160707

Step to Reproduce
compile the following code with tsc test.ts --noUnusedParameters
// test.ts
class A {
    constructor(private a: number) {}

    b(this: A): number {
        return this.a;
    }
}

function bar(this: Window): string {
    return window.location.href;
}

Expected behavior
There is not any compile error.
Actual behavior
tsc reports following compilation errors
test.ts(4,7): error TS6133: 'this' is declared but never used.
test.ts(10,14): error TS6133: 'this' is declared but never used.

// test.ts
class A {
    constructor(private a: number) {}

    b(this: A): number { // error TS6133: 'this' is declared but never used.
        return this.a;
    }
}

function bar(this: Window): string { //  error TS6133: 'this' is declared but never used.
    return window.location.href;
}

@mhegazy
Sorry, I missed that you assigned this to you and I opened the pull request #9557 to fix this bug.
How should I treat #9557?

thanks @saneyuki. waiting on the CI server, then will merge it.

@mhegazy This bug is not fixed in Visual Studio 2015 with TS2.0<EMAIL_ADDRESS>installed by npm works correctly). Can you reopen this issue?

Why is the second example (this: Window but this is not used) not an error? Why do you need to declare the type of this if you never use it?


266-tests: Added client-side timestamp tests

Added C* version checking so when integration tests are run using < C* 2.1, the tests will not be marked as failure.
Should we add the edge cases where the CQL timestamp should override the cass_cluster_set_timestamp_gen and/or cass_statement_set_timestamp settings?


Update contentdefinitions.md

fixes https://github.com/MicrosoftDocs/azure-docs/issues/64895

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

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


[kdstatemachineeditor] new port

[x] Changes comply with the maintainer guide.
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx.
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

@drdanz, Thanks for your PR, Have you tested the usage locally? I want tested the usage on my machine but I get a install error:
CMake Error at scripts/cmake/vcpkg_execute_build_process.cmake:134 (message):
    Command failed: F:/Feature-test/kdstatemachineeditor/downloads/tools/cmake-3.30.1-windows/cmake-3.30.1-windows-i386/bin/cmake.exe --build . --config Debug --target install -- -v -j9
    Working Directory: F:/Feature-test/kdstatemachineeditor/buildtrees/qtdeclarative/x64-windows-dbg
    See logs for more information:
      F:\Feature-test\kdstatemachineeditor\buildtrees\qtdeclarative\install-x64-windows-dbg-out.log

Call Stack (most recent call first):
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_build.cmake:74 (vcpkg_execute_build_process)
  installed/x64-windows/share/vcpkg-cmake/vcpkg_cmake_install.cmake:16 (vcpkg_cmake_build)
  installed/x64-windows/share/qtbase/qt_install_submodule.cmake:352 (vcpkg_cmake_install)
  ports/qtdeclarative/portfile.cmake:30 (qt_install_submodule)
  scripts/ports.cmake:196 (include)

F:\Feature-test\kdstatemachineeditor\buildtrees\qtdeclarative\install-x64-windows-dbg-out.log:
install-x64-windows-dbg-out.log

Tested usage successfully by kdstatemachineeditor;x64-windows


Cannot delete the last track of an MA provider playlist

What version of Music Assistant has the issue?
2.3.0b21
What version of the Home Assistant Integration have you got installed?
None
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Trying to delete the last track of a playlist leads to an error message
How to reproduce

go to playlists
click a playlist. I chose playlist in the MA music provider, containing 2 Qobuz tracks
I selected | net to the second track , and "Remove from playlist"

I got "List assignment index out of range"

Music Providers
MA
Qobuz
Player Providers
N/A
Full log output
No response
Additional information
ma_playlist_deletion.log
What version of Home Assistant Core are your running
2024.9.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Windows

No such problem editing a Qobuz provider playlist.

It actually is deleting the next track in the list which is why it fails when you try and delete the last

It actually is deleting the next track in the list which is why it fails when you try and delete the last

Off by 1 error, then ?

fixed in 2.3.0 rc1

Confirmed fixed


Extend exclude with php and dependencies settings

Q
A

Documentation
yes

New Feature
yes

Description

This introduces the ability to exclude specific jobs for specific PHP and/or composer dependency sets.
Fixes #93

@internalsystemerror is the CI failure related to what you fixed in #190 ?

I believe you mean @gsteel although I see it losing the "before_script" from the matrix output for some reason.
I did some similar changes in https://github.com/laminas/laminas-ci-matrix-action/pull/167 and https://github.com/laminas/laminas-ci-matrix-action/pull/166 but upon reviewing those, I don't think they would affect this either.

It looks like the only issue here is that it's no longer adding the xmllint checks in the matrix output whilst that is expected. Given the only recent change is #190, I suspect the top change (the one I commented on) needs reverting and adding multiline: true is all that's required.

@boesing @internalsystemerror should I rebase and release this to 1.23.x, if green?

It is not properly working yet, please ignore this for any release. I'll handle this once I find some time to work on this. There is no pressure on this topic and no high need (at least for me) as of now 👍🏼

Ok, just beware that it will go into 2.1.x, probably.


Bug in copy button of context popup menu

How to get the bug :-

Select some text
Press right click and press copy from popup menu
You will notice that, the word is not reflected to the clipboard of the context popup menu.

Have done some progress at commit 74b23e66c87218d0813bf3f2448122409d24ff38
However there is a bug in it , ....
If you go to copy same word may number of times it shows that many times on the clipboard... the expected behavior should be one word should come once in the clipboard.

@saanvibehele @AayushGupta69 Please test this feature at your system as well.

@NishantWankhade Yes, can confirm that bug. Have started working on a fix for that.


Upgrade to curlylint 0.13.1

Addresses https://github.com/thibaudcolas/curlylint/issues/132

Thanks @gasman

You’re too fast :)

I was all ready to open a PR that pinned click to the old version until I saw you'd already made a new release, so the feeling's mutual :-)


Test Documentation

Document the purpose of each of the test values in the python and C code in a separate file.  May help with referencing later on.

Note: Pertaining to NoPCM

@smiths, @JacquesCarette, Should we do something with this issue?

If I understand it correctly, yes, we should exactly as described.  But likely re-assign it to whoever is taking over NoPCM.

@elwazana, you have been looking at the noPCM example.  What are your thoughts on this issue.  As far as I can tell, there is no test plan, or test report, for the noPCM example.  There is testing code in the CCode and Python folders.  In your review of this code, was the meaning of the test values clear?  My guess is that the meaning was confusing, but we would like to hear your thoughts, since this example is new to you.

@smiths I did find it more confusing reviewing the noPCM testing code than the SWHS repository testing code, but I wanted some clarification with regards to what you mean by test values, do you mean the inputFiles?
As an example SWHS Python inputFiles:
https://github.com/smiths/swhs/tree/master/Program/Python/Testing/inputFiles

@smiths Dr. Smith, I was wondering if there is anything specific I should do for this issue? I understand that I should document the test values, but is there any specific format that you and @JacquesCarette Dr. Carette would prefer? Should I hold off?

You should document these in a way that they can be used in Drasil itself. Some of these will be "reasonable values" (already implemented), but we might not be able to capture all of this knowledge in Drasil yet.
I guess the first thing is to (informally) document what each test value means, and then we'll see.

@elwazana, please write a document like PCM_VVPlan for the noPCM example.  You don't need to worry about all of the sections.  You should focus on summarizing the test cases themselves.  Each of the input files (which you pointed to above) provide an input case.  What we really want is to know what distinguishes each case from the others.  To do this, I have the following advice:

Give each of the test cases a name.  Names like P01, P02 say nothing.  The order means nothing.  We should give the test cases meaningful names in the documentation and in the associated files.
Not to needlessly repeat values in the documentation.  The easy solution would be to just copy the input files into the test plan, but this would made it a long document, and difficult to maintain.  We should summarize the information that is common between the test cases and then distinguish each test case by its "delta" from the standard test case (or cases).

@elwazana, since this issue is specific to the "manual" version of the tests cases, could you close this issue by making a new issue in the case study repo?  I think it would help manage the projects better if we separate the Drasil issues from the case study issues.

https://github.com/smiths/caseStudies/issues/75


implementation of asyncio.wait  waiting for iterator is not complete

asyncio.wait  has 3 different options to run with
Constant	Description
FIRST_COMPLETED	The function will return when any future finishes or is cancelled.
FIRST_EXCEPTION	The function will return when any future finishes by raising an exception. If no future raises an exception then it is equivalent to ALL_COMPLETED.
ALL_COMPLETED	The function will return when all futures finish or are cancelled.
but there is no way to run it in 4-th mode  FIRST_COMPLETED + FIRST_EXCEPTION
which is "must-have" in the case when it suppose to wait for iterator
it will send totally legit Exception about end of iteration
but asyncio.wait has no way to catch/handle it properly.

Actually, FIRST_COMPLETED returns if any future/coroutine raises an exception. In general, a future is considered completed when it gets its result or when an exception is raised. This example illustrates the use case you describe:
import asyncio

async def task1():
    await asyncio.sleep(3)
    raise RuntimeError

async def task2():
    await asyncio.sleep(10)

async def main():
    tasks = [task1(), task2()]
    done, pending = await asyncio.wait(tasks, return_when='FIRST_COMPLETED')
    exception = done.pop().exception()
    assert isinstance(exception, RuntimeError)

loop = asyncio.get_event_loop()
loop.run_until_complete(main())
loop.close()

@005 I'd be interested in the actual code that caused this suggestion.

it looks its not asyncio, but something esle.
here is the simplest example i could come up with.
notice there are 2 urls that server will handle
/1
and
/2
open each one in different browser tab, and then there will be a difference
when you close /2 - server can see it and will print "websocket connection closed"
but if you close tab /1 it will still be stuck in "enter wait.." and will never get to the line "websocket connection closed"
import asyncio
import aiohttp
from aiohttp import web

page = """<!DOCTYPE html>
        <html lang="en">
            <head>
                <title></title>
            </head>
            <body>
                <script>
                    var url = window.location.href.replace('http','ws');
                    console.log('connect to', url);
                    var ws = new WebSocket(url);
                    ws.onopen = function () {
                        console.log('send to server');
                        ws.send("Hello Web Sockets!");
                    }
                    ws.onmessage = function (msg) {
                        console.log('got msg',msg.data);
                        if (msg.data == 'disconnect') {
                            ws.close();
                        } else {
                            ws.send(msg.data);
                        }
                    }
                    setInterval(function(){
                        console.log('i am here');
                        ws.send('i am here');
                    },1000);
                    //ws.close();
                </script>
                <button >
            </body>
        </html>"""

async def index(request):
    ws = web.WebSocketResponse()
    yes_we_can, protocol = ws.can_prepare(request)
    print('yes_we_can, protocol ', yes_we_can, protocol)
    if yes_we_can:
        await ws.prepare(request)
        ws.send_str('hello:1')
        session_id = await ws.receive()
        print('got session id', session_id)
        while True:
            print('enter wait..')
            done, pending = await asyncio.wait([ws.receive()], return_when=asyncio.FIRST_COMPLETED)
            print('wait done', done, pending)
            msg = done.pop().result()
            print('incoming msg', msg.data)
            if hasattr(msg, 'type') and msg.type == aiohttp.WSMsgType.CLOSE:
                break
            continue
        print('websocket connection closed')
        return ws
    else:
        return web.Response(text=page, content_type='text/html')

async def index2(request):
    ws = web.WebSocketResponse()
    yes_we_can, protocol = ws.can_prepare(request)
    print('yes_we_can, protocol ', yes_we_can, protocol)
    if yes_we_can:
        await ws.prepare(request)
        ws.send_str('hello:1')
        session_id = await ws.receive()
        print('got session id', session_id)
        async for msg in ws:
            print('incoming msg', msg)

        print('websocket connection closed')
        return ws
    else:
        return web.Response(text=page, content_type='text/html')

app = web.Application()
app.router.add_route('GET', '/1', index)
app.router.add_route('GET', '/2', index2)
loop = asyncio.get_event_loop()
loop.set_debug(True)
handler = app.make_handler()
f = loop.create_server(handler, '<IP_ADDRESS>', 8444)
srv = loop.run_until_complete(f)
print('serving on', srv.sockets[0].getsockname())
try:
    loop.run_forever()
except KeyboardInterrupt:
    pass
finally:
    loop.run_until_complete(handler.finish_connections(1.0))
    srv.close()
    loop.run_until_complete(srv.wait_closed())
    loop.run_until_complete(app.cleanup())
loop.close()
print('CLEAN EXIT')


append empty headers matching support - for example, if header is abs…

Append empty headers matching support
Description
In general, I have made it possible to apply predicate to header even if it is absent considering header as empty string. Below u can see my triuble with that
Motivation and Context
II have faced an issue I want to exclude with logbook some requests, that has contain empty Authorization header or does not contain it at all. As u can understand header name here is unimportant, it could be easily some another header. Unfortunately, by default, if logbook see that header is absent it is not apply predicate to it at all. This it quite sadly, because what I really need to tell logbook is something like this : "Hey, if Authorization header is absent, or consists of an empty string, that you should not log request and corresponding response".
Types of changes
The changes I have made is quite simple:

Adjust 2 methods in Conditions - I think changes is self descriptive, I have made possible to compare header even if it is absent (in this case header is considered as empty string) with predicate
I have obviously run unit tests and fix one that have failed because of my change. This test btw describes my idea - even if header is empty it possibly could satisfy predicate
Append apache commons dependency, to just simplify some ordinary stuff, hope we will make use of this latter also

Checklist:
I think my changes should be somewhere documented. If you will procure me place where this change should be documented I will certainly supply you with one
Best regards,
Mikhail

Thanks for your contribution!
Unfortunately, the way how it's implemented can't be merged for two reasons:

It's a breaking change, i.e. it changes behavior
All parameters are non-null by default. See https://github.com/zalando/logbook/blob/main/logbook-core/src/main/java/org/zalando/logbook/package-info.java

Instead of sneaking new behavior into an existing method, why not introducing a new one to test for absent headers?
Logbook.builder()
    .condition(exclude(
        withoutHeader("Authorization").or(header("Authorization", ""))
    ))
   .build()

Uh! I just does not know that I am able to use withoutHeader() Condition, so of course, in this case it is not necessary at all. Maybe I am a bit inattentive, but consider doc this opprotunity :) Because I have not seen this functional during me research before merge request (maybe I am just a bit remiss). But thank you!

That condition doesn't exist yet, but it would be a better addition, imo.
On Fri, 30 Apr 2021, 10:31 Mikhail, @.***> wrote:

Closed #1013 https://github.com/zalando/logbook/pull/1013.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/zalando/logbook/pull/1013#event-4667915850, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADI7HKRCDNH3EWELNVNC7DTLJTFJANCNFSM43Y7JRYA
.

I can take care of this, if I may

Contributions are highly appreciated! 📣


[RFC] Deprecation decorator

This is a Request For Comments on the idea of introducing a decorators that would mark either a class, function methods or a specific kwarg within any of these callables as deprecated. After input I will make a more polished PR. What I need input on is this:

The implementation has both the notion of soft and hard deprecation, where the former issues a warning and the latter raises and exception. Personally, I think that is nice, but I need input on whether you think it is overkill
The periods where a feature is soft or hard deprecated in given as a time interval since last supported release. I.e "the feature is soft deprecated for all releases for 3 months after version '1.2.3' and hard deprecated an additional 3 months after that". Is this the way you prefer to work with these intervals? One might also list those intervals as a number of releases i.e. "the feature is soft deprecated 2 releases after version '1.2.3' and hard deprecated an additional 2 releases after that"
EDIT How clever do we want it to be? In principle we could of course give it a "last supported time stamp" and well defined time deltas and then we could calculate when it would move from soft deprecation to hard, as opposed to the suggested implementation when you will need to flip the hard_deprecation boolean by hand. I didn't do it because it is my view that it will not be worth the extra code complexity and because I don't want peoples code to all of a sudden hard deprecate with them having upgraded.

The feature is used like this:
import sys
from time import sleep

from ixdat.tools import deprecate

@deprecate("1.2.3", "1 month", "1 month", "Please use `MyNewClass` instead")
class MyClass:
    pass

my_obj = MyClass()

# This is just a bit of trickery to make the warnings appear in order with the printed
# separator
sys.stderr.flush()
sleep(0.1)
print("\n\n===\n\n", flush=True)

class MyClass:

    @deprecate("1.2.3", "1 month", "1 month", "Please use `mynewmethod` instead")
    def mymethod(self):
        ...

    @classmethod
    @deprecate(
        "1.2.3",
        "1 month",
        "1 month",
        "Please use `kwargb` instead",
        kwarg_name="kwarga",
        hard_deprecate=True
    )
    def myothermethod(cls, arg, kwarga=None, kwargb=None):
        ...

my_obj = MyClass()
my_obj.mymethod()  # Will warn

sys.stderr.flush()
sleep(0.3)
print("\n\n===\n\n", flush=True)
sleep(0.3)

my_obj.myothermethod(1, kwarga=7)

which outputs something like this:
/home/kenneth/venv/ixdat/ixdat/test.py:12: DeprecationWarning: The class 'MyClass' is deprecated, its last supported version being 1.2.3.
This means that:
* It is soft deprecated (issuing warnings) in all releases for 1 month after the last supported release
* It is hard deprecated (raising exceptions) in all releases for an additional 1 month after that
* After the hard deprecation period ends, it will be removed

See instruction below on how to update your code to avoid this message:
Please use `MyNewClass` instead
  my_obj = MyClass()

===

/home/kenneth/venv/ixdat/ixdat/test.py:40: DeprecationWarning: The method 'MyClass.mymethod' is deprecated, its last supported version being 1.2.3.
This means that:
* It is soft deprecated (issuing warnings) in all releases for 1 month after the last supported release
* It is hard deprecated (raising exceptions) in all releases for an additional 1 month after that
* After the hard deprecation period ends, it will be removed

See instruction below on how to update your code to avoid this message:
Please use `mynewmethod` instead
  my_obj.mymethod()  # Will warn

===

Traceback (most recent call last):
  File "/home/kenneth/venv/ixdat/ixdat/test.py", line 48, in <module>
    my_obj.myothermethod(1, kwarga=7)
  File "/home/kenneth/venv/ixdat/ixdat/src/ixdat/tools.py", line 211, in inner_function
    raise DeprecationError(compound_message)
ixdat.exceptions.DeprecationError: The property named 'kwarga' in method 'MyClass.myothermethod' is deprecated, its last supported version being 1.2.3.
This means that:
* It is soft deprecated (issuing warnings) in all releases for 1 month after the last supported release
* It is hard deprecated (raising exceptions) in all releases for an additional 1 month after that
* After the hard deprecation period ends, it will be removed

See instruction below on how to update your code to avoid this message:
Please use `kwargb` instead

Hi Kenneth,
I think all of this is awesome! The useage syntax is simple and easy enough that I would indeed use it all over the place!
The only thing I think is overkill is the timed moving from soft to hard depreciation. I think it would be better to do that by versions. That gives us more flexibility, important since ixdat is not anyone's full job. If, say, we set up a depreciation in an ixdat release but then no one has time for ixdat for a few months and, in the meanwhile it is revealed that the thing should not have been depreciated, we'd have shot ourselves in the foot with the timer but have caused approx. zero damage with the versions.
So maybe the code could be
class MyClass:
    @deprecate("1.2.3", "1.3.0", "Please use `kwargb` instead", kwarg_name="kwarga", )

to soft depreciate use of kwarga after 1.2.3 and hard depreciate after 1.3.0... something like that?
I also like the only_one_of decorator, and agree that could clear up some complex kwarg interpreting at the start of a number of methods.

@ScottSoren It's a good point about avoiding the timed hard deprecation. I will change it. Only thing is, if you provide a specific version number as opposed to say "3 minor/major releases after" then you are a little locked in with what the next versions will be. It might not be a problem in reality, what do you think?

Ah. I think there is a small terminology difference here. I refer to the second digit as "minor", as per semver. In any case, since you are primary maintainer, if you would prefer a specific version to be listed, I think we should do that.
Making the deprecation thresholds optional is a good idea. I will change the implementation to accommodate this. What this means is that:

Applying the decorator means the feature is immediately (in practice only noticed from next release) soft deprecated
The developer may optionally list at which versions the feature will be hard deprecated
The developer may optionally list at which version the feature will be removed entirely

Since we are to go ahead with version numbers I think I will also change the implementation to automatically hard deprecate by comparing version numbers, if the hard deprecate version number is given. This change will then mean that the hard deprecate argument disappears and that a developer, for a deprecation where the entire schedule is known, will not have to touch the decorator between initial deprecation and deletion.
Thanks for the good suggestions.
Plan now is to split the only_one_of decorator out into a separate branch, implement the changes and remove the draft tag from this.


I wanted to user former version 3.02 but....

I wanted to use former version 3.02 but.... google link redirect here......
so i can't get former version.... fix please....

http://sourceforge.net/projects/tesseract-ocr-alt/files/


UnicodeDecodeError

Hello,
using pip install carbone-sdk under Windows Server 2019 with Python 3.11.0 produces following error:

Any ideas?

I had to manually download the repository, build and install it and change in setup.py the encoding to utf-8.
with open("README.md", "r", encoding="utf-8") as fh:     long_description = fh.read()
Can you tell me what this error means:
Exception: Carbone SDK render error: Missing data key in body
I tried:
with open(sys.path[0] + "\\test.json", 'r', encoding="utf-16") as file:     data = json.load(file)
report_bytes, unique_report_name = csdk.render("template_ID", data)

Same here, Windows 10, Python 3.10

Hello,
using pip install carbone-sdk under Windows Server 2019 with Python 3.11.0 produces following error:

Any ideas?

Thank you for reporting the issue!
I'll test the @qwertguy solution and patch the SDK.
For the error "Carbone SDK render error: Missing data key in body"
When you generate a report, the render options object should look like this:
{
  "data": {} // ALL YOU DATA,
   "convertTo": "pdf" // or other formats
}

If the "data" element is missing, the error is thrown
I keep the feed updated when the patch is deployed.

I just fixed the issue and published a new version: https://github.com/carboneio/carbone-sdk-python/releases/tag/v1.0.7
You can try to install it again:
pip install carbone-sdk

Let me know if everything is good now :)


pull request for e2e test at 01/04/2017 07:15:16

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Possible blocking issue

There's a bit of time here where I am unable to move my cursor, and then all the actions that I did make happen all at once at the end, it's hard to convey but look for the rapid movement all at once at the end of this gif:

It feels like what is happening is on lint the thread is being blocked until codeclimate comes back with an answer which prevents me from navigating my code in the mean time
After disabling codeclimate linter in visual studio code and reloading:

Movement in this gif was unhindered

Hi @slifin, sadly, by the time I created the addon, I couldn't get execAsync/spawn to function properly (ref: code). By then spawning decoupled sub-processes, (in this case codeclimate analyze, which invokes a docker exec underneath) never returned the analysis result. I'll look again into it soon, for now, you are welcome to fork and try fixing this up.
Best regards.

Hi again @slifin, I've managed to call the analyzer via spawn, also made a few performance improvements. Feel free to test again to see if this resolves your issue.


[Feature]: Connector for Walmart Marketplace API

Connector Name
other (None Listed Above)
Suggested feature
Support for. Walmart Marketplace API through a connector
Related issues
I am trying to do [...] but [...]

https://developer.walmart.com/doc/us/mp/us-mp-intro/

@SkNuwanTissera is working on this


[🐛] launchImageLibrary takes too long to return video result on iOS.

Description
I'm using react-native-image-picker version 5.4.0, I tried using assetRepresentationMode prop but it doesn't work well.
Proposed Solution
It would be great if the processing had a progress indicator when selecting a video file.
How can I solve this problem so that the user understands the video selection process?

Have the same problem; any help regarding this?

Same problem

Same issue

I have this issue too!
Some videos can take several minutes, even tough they are about the same length as other videos that returns quicker. But all videos takes seconds to be returned.
I can't see any reason why this is if we're just returning the path to the file. I've disabled both includeBase64 and includeExtra.
My wish is that this is a lot quicker, but also that we can have a callback to RN that a video has been selected and the native side is processing the file so we can present some UI to the user that the app is actually doing something meaningful and not just stopped working.

Same problem here

I'm seem to recall a PR that fixed this, will have to double check if it had been merged or had outstanding changes.

@Johan-dutoit any update ?

@Johan-dutoit any update with this PR that you commented?

vaax :D mec ak movedi with the same problem

any updates


doc: add GCP configuration documentation in README.md

Description
Add documentation on how to configure KFM to be able to deploy data plane clusters located in GCP.
Verification Steps

Checklist (Definition of Done)

[ ] All acceptance criteria specified in JIRA have been completed
[ ] Unit and integration tests added that prove the fix is effective or the feature works (tested against emulated and non-emulated OCM environment)
[ ] Documentation added for the feature
[ ] CI and all relevant tests are passing
[ ] Code Review completed
[ ] Verified independently by reviewer
[ ] All PR comments are resolved either by addressing them or creating follow up tasks
[ ] Required metrics/dashboards/alerts have been added (or PR created).
[ ] Required Standard Operating Procedure (SOP) is added.
[ ] JIRA has been created for changes required on the client side

I added the do not merge label to this PR as we shouldn't merge this as we consider GCP as "enabled" / ready to be used.
However, this PR can be reviewed already

Specifics about how to retrieve our environment specific credentials for development have not been added in this repository/documentatior. For that refer to our team onboarding documentation.

I am also working on adding a makefile target to setup credentials

I added the new Makefile target and updated the documentation.
This should be ready for review

LGTM. Content clear and well linked

I added the do not merge label to this PR as we shouldn't merge this as we consider GCP as "enabled" / ready to be used.

@miguelsorianod the PR still awaits for GCP evaluation?

I added the do not merge label to this PR as we shouldn't merge this as we consider GCP as "enabled" / ready to be used.

@miguelsorianod the PR still awaits for GCP evaluation?

Yes.


Statements in the next line of 'else' cannot be highlighted in atom.

pug-template:
if foo
  a foo
else
  a bar // cannot be highlighted

I have the same bug, it seams to because of the absence operand after else: I guessed it because there is no such issue with else if. I tried to fix it modifying this line. However I’m not sure it can change anything because I haven’t got a proper dev environment for this project. To be continued..

I have the same issue. It's very annoying because I work with express and I have a lot of conditionals in my views. Any workarounds?

+1
the first line after an else is greyed out, whereas consecutive lines are correctly highlighted.
A possible workaround could be making the else statement else if(true) which should mean the same thing as else. However, it pains me to edit code unecessarily like that just because of failures of syntax highlighting.


Where can I find the whole-body model?

I'm trying to go through the Tutorial_Annotation.ipynb notebook. However, I'm unable to find the link to download the pre-trained whole-body model. Can you please point me in the right direction?

Hi @alex-l-kong , here is the link for the model https://drive.google.com/drive/folders/1oWh_-ZRdhtoGQ2Fw24HP41FgLoomVo-y?usp=sharing . BTW, you can access all checkpoint folders here


Implement logging

Here the first question is whether we use the Python logging module or the one from pypath (https://github.com/saezlab/pypath/blob/master/src/pypath/share/log.py). I like the pypath one, and it has the advantage that all log messages will go into the same file.

I started with the Python logging (wanted to learn it), and I had no troubles sending it to one file; maybe it has gotten better since you last looked at it. You can check it out on the latest commit.


Am I missing something important? How does the lemon-squeezy/webhook route work?

Lemon Squeezy for Laravel Version
1.5.3
Laravel Version
10.4.1
PHP Version
8.1.0
Description
How does the /lemon-squeezy/webhook route work? I didn't see any route added to the routes.php file.When I send request to /lemon-squeezy/webhook, it returns 404 not found
Steps To Reproduce
NA

Heya, this is added by the service provider here: https://github.com/lmsqueezy/laravel/blob/main/src/LemonSqueezyServiceProvider.php#L38
How are you sending the request?

Maybe this is related, but when I run php artisan lmsqueezy:listen, I get this error:
Route [lemon-squeezy.webhook] not defined.

  at vendor/laravel/framework/src/Illuminate/Routing/UrlGenerator.php:497
    493▕             ! is_null($url = call_user_func($this->missingNamedRouteResolver, $name, $parameters, $absolute))) {
    494▕             return $url;
    495▕         }
    496▕
  ➜ 497▕         throw new RouteNotFoundException("Route [{$name}] not defined.");
    498▕     }
    499▕
    500▕     /**
    501▕      * Get the URL for a given route instance.

      +17 vendor frames

  18  artisan:13

@frankspin89 will need a lot more info to reproduce that one...

Heya, this is added by the service provider here: https://github.com/lmsqueezy/laravel/blob/main/src/LemonSqueezyServiceProvider.php#L38
How are you sending the request?

Readmd.md updated, https://github.com/lmsqueezy/laravel/pull/89

@aiddroid that can't be it because the service provider is autoloaded. You say you use Laravel 10.4.1. Could you please attempt to upgrade to the latest version to see if that helps?

Going to close this one now as I don't believe there's an issue atm.


Playground loading animation doesn't disappear on scene load if tab is not active

Repro

Bug repro on playground. While the scene is loading, switch tabs repeatedly.
Expected result: When you return to the original tab after the scene has finished loading, the loading screen should be gone.
Current result: The screen is still dim and the loading animation still appears over the scene, at least sometimes.

Screenshots

Desktop (please complete the following information):

OS: Windows
Browser: Edge
Version: 91.0.864.41

Additional context
This may be caused by onDisplayWaitRingObservable not being correctly fired if the tab is not active, but I have not confirmed if this is the case.

I believe you it's there, but I can't seem to be able to reproduce this. What's the best moment to switch tabs in order to reproduce this?

This may be caused by onDisplayWaitRingObservable not being correctly fired if the tab is not active, but I have not confirmed if this is the case.

I initially thought it is something similar, but it seems from your image that the animation has already started. so it already triggered the fade out animation. I need to investigate, but i'll have to reproduce this first.

Yeah, I am also having difficulty repro'ing it now. I know that at least one another person was able to repro the issue, but it seems to only happen in very specific circumstances. I will let you know if I can figure out a consistent way to trigger the bug

Let me close it for house keeping and reopen once we figure the full repro


How to handle a request from client?

Hello,
I have some problems to understand how to handle a request which is send by a client. I understand (because of the examples) how to send a response to a client but not the way in the other direction.
Can you help me?
Furthermore, the next step would be to use the request within QML in "newConnection()" signal.
For the qml example have a look here:
https://github.com/rschroll/qhttpserver/blob/master/examples/qml/app.qml
I think you already use the because of the forking...

the way in the other direction.

http://doc.qt.io/qt-5/qtqml-xmlhttprequest-example.html

Furthermore, the next step would be to use the request within QML in "newConnection()" signal.

I would recommend to you, not to handle such things in QML, but instead do that as a QML-C++ binding, so you can access C++ methods from QML ... but thats a matter of taste i guess.


v1.1.0

Update README.md

What this PR does / why we need it:

Fix autoupdate Type Mismatch
Support for Custom Secrets Template
Support for Custom LogDNA API Hostname
Support for Custom LogDNA Ingestion Hostname

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #13869, fixes #13498, fixes #13496

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

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@scottrigby @unguiculus
Can you quickly approve this Pull Request? It has passed almost all checks! This is just addition!
Thanks in advance!

are you sure that it fixes #13869 ?

@maorfr, doesn't it? This PR contains OWNERS file too which is for specifying maintainers.

right, but it will close the issue, which is not what we want.

@maorfr, I see - I'll remove that part from description

i think you will have to close and reopen

@scottrigby @unguiculus
Can you quickly approve this Pull Request? It has passed all checks! This is just addition! And it is waiting for /ok-to-test
Thanks in advance!

can we rebase and resolve conflicts? i'll approve it.
already added you as collaborators to the repo.

@smusali any chance of getting this rebased and approved? I was about to submit a similar PR for the apihost and loghost variables but didn't want to duplicate this one.


Change to system language keyboard

Added changeKeyboard to tap globe button, and switch to Simulator’s
language keyboard. Works with #3785.

I'm unsure if we want this in the snapshot core, but if we want it as an external file instead. What would you think about building a simple "Plugin" system that allows easy import of helpers and we can put those available plugins in this GitHub repo?

I'm going to close this PR since we don't plan on merging this into snapshot at this point. Thanks!

I have a pretty decent solution in the works. Borrowing ideas from this PR plus some extra.


Flesh out client tests

Description
We had a user asking in Discord about how to consume a server stream, and I was going to point them at the tests when I realized that we don't actually have those tests in this repo. This is more about having a mechanism for documentation than about any concern about correctness.
Changes

Add uuid as an explicit dep for generating test tokens
Add the Conn as a member of the client struct so that callers can do things like teardown logic
Make tests match what's in authzed-dotnet

Testing
Review, see that things are green

@ecordell i'm also a little worried that I have a flake in the BulkImportExportTest. is there anything obviously borked with it?


I sent ton to the wrong address. Can anybody help

@Deputy111 Thanks for posting the issue.
Your query would be best dealt with by the support team.
Please see the link below to our dedicated support line:
Support : Help Center
Note: Click on the live chat icon at the bottom corner of the page to start a conversation


Tables are too wide for container

Laravel Version: 5.5.21
Voyager Version: 1.0.6
PHP Version: 7.0.10
Database Driver & Version:

Description:
Users table, Roles table, Posts table and other table that use DataTable are too wide for the container. I suppose this is due to extra row class added.
Steps To Reproduce:
Go to http://<IP_ADDRESS>:8000/admin/posts

Depends 100% on your screen resolution and how many fields you have in those tables.  Voyager can't force it to be 100% width of the container without scrolling because then you'd potentially have columns that are 10px (or some other ridiculously small size) wide.  Voyager can't possibly know how wide your table is going to be, so it can't provide a CSS solution for it.
Do you have a proposed solution?

In my widescreen 1920x1080 and fresh install without additional columns it shows an horizontal scrollbar.
I see that each table row is inside a div class="row" that add margin. I think removing thos calasses solve the problem.

Can you provide a screenshot of what you're seeing? 1920 should be way more than enough to show the table with default fields.

Ah, that's a CSS box model issue, not a table width issue.  What browser are you using? Does it behave the same with another browser?

Same on firefox and IE

The problem is caused by the negative (-15px) margin introduced by the .row CSS class (Bootstrap) of the div(s) inside the data table.
This can be rectified by the following change in app.scss (line 1004):
.dataTables_wrapper .row {
  margin-bottom: 0.5em; }

To:
.dataTables_wrapper .row {
  margin-bottom: 0.5em;
  margin-left: 0;
  margin-right: 0; }

@fletch3555 Do you want me to make a pull request?

Yes, please do @dfk7677

PR done @fletch3555 . I am not sure if I shouldn't have submitted the compiled app.css though.

Perfect, thanks.  Yes, we do want the compiled app.css, since we don't have a CI pipeline set up to recompile it.


HTMLView with ngIf and  tag renders blank

From @noumaans on May 4, 2018 20:46
Tell us about the problem
If an HTMLView component is used with an *ngIf, it renders blank / empty if the HTML starts with a <br/> tag.
Which platform(s) does your issue occur on?
iOS
Please provide the following version numbers that your issue occurs with:

CLI: 4.1.0-2018-04-18-11407
Cross-platform modules: 4.0.0-2018-05-03-03
Runtime(s): "tns-ios" 4.1.0-2018-04-24-01
Plugin(s): (look for the version number in the package.json file of your
project)

Please tell us how to recreate the issue in as much detail as possible.
Playground Link:
https://play.nativescript.org/?template=play-ng&id=LUVNqq&v=2
NOTE: Remember to make sure the delay is present in ngAfterViewInit - if delay is reduced to 1s, the render starts working again.
Copied from original issue: NativeScript/NativeScript#5783

The issue is still reproducible with @next nativescript-angular.

Hi  @noumaans
In our further investigation, we found that the issue is related to a problem with the correct measuring the component dimensions while loading the HtmlView content.
As a workaround, you can set up the component height, and the content will be appropriately displayed.
<HtmlView  [html]="delayedWithBR.html" height="200"></HtmlView>

@tsonevn that doesn't work when your height requirement is dynamic since one is rendering a large amount of text - in our case inside a scrollview.
GridLayout with auto,* and height=100% workarounds didn't work either...
Any help for providing this component a dynamic height would be much appreciated.

Hi @noumaans,
The current option is to bind the height property and to change its value via code-behind. For example:
<HtmlView backgroundColor="green" *ngIf="delayedWithBR" [height]="hProperty" [html]="delayedWithBR.html"></HtmlView>

Would this be your recommendation?
1 - set an initial value of height say 100
2 - get device screen size and orientation
3 - perform a calculation in TS to compute the size to allocate to HTMLView
4 - set the height value
Thanks so much for your help. Really appreciate it.


Nick: Watch exacthacks.com

Nick requests the watch of the watch_keyword exacthacks\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

exacthacks\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Update country_codes_info Package on pub.dev to Reflect Latest Changes

There were updates to the dependencies in the recent commit here of the country_codes_info library.
However, these changes have not been reflected in the version available on pub.dev, which remains at version 0.0.7.
As a result, attempting to install the package from pub.dev causes an Android build failure with the error message:
 Could not find com.android.tools.build:gradle:4.3.10.
I kindly request that the updates made in the latest commit be pushed to pub.dev to ensure users can access the updated version without encountering build issues.

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':country_codes_info'.

Could not resolve all files for configuration ':country_codes_info:classpath'.
Could not find com.android.tools.build:gradle:4.3.10.
Searched in the following locations:
- https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/4.3.10/gradle-4.3.10.pom
- https://repo.maven.apache.org/maven2/com/android/tools/build/gradle/4.3.10/gradle-4.3.10.pom
Required by:
project :country_codes_info
Failed to notify project evaluation listener.
Could not get unknown property 'android' for project ':country_codes_info' of type org.gradle.api.Project.
Could not get unknown property 'android' for project ':country_codes_info' of type org.gradle.api.Project.

There were updates to the dependencies in the recent commit here of the country_codes_info library.
However, these changes have not been reflected in the version available on pub.dev, which remains at version 0.0.7.
As a result, attempting to install the package from pub.dev causes an Android build failure with the error message:  Could not find com.android.tools.build:gradle:4.3.10.
I kindly request that the updates made in the latest commit be pushed to pub.dev to ensure users can access the updated version without encountering build issues.

https://pub.dev/packages/country_codes


pull request for e2e test at 01/02/2017 18:26:12

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Storing the build information in the binary

fixes: #1472

should be fixed by now

should be fixed by now


type heartbeaters() undefined

I face compilation error type heartbeaters() undefined
I searched Lapin documentation and github issues for Lapin and could not find anything related to heartbeat

Can you please include the complete error you are getting from the compiler?

iex -S mix
Erlang/OTP 27 [erts-15.0] [source] [64-bit] [smp:128:8] [ds:128:8:10] [async-threads:1] [jit:ns]
===> Analyzing applications...
===> Compiling rabbit_common
===> Compiling src/rabbit_heartbeat.erl failed
rabbit_heartbeat.erl:22:39: syntax error before: 'maybe'
rabbit_heartbeat.erl:20:2: type heartbeaters() undefined
rabbit_heartbeat.erl:32:13: type heartbeaters() undefined
rabbit_heartbeat.erl:38:13: type heartbeaters() undefined
rabbit_heartbeat.erl:49:21: type heartbeaters() undefined
rabbit_heartbeat.erl:50:22: type heartbeaters() undefined
** (Mix) Could not compile dependency :rabbit_common, "(mydirectory)/rebar3 bare compile --paths (mydirectory)/_build/dev/lib/*/ebin" command failed. Errors may have been logged above. You can recompile this dependency with "mix deps.compile rabbit_common --force", update it with "mix deps.update rabbit_common" or clean it with "mix deps.clean rabbit_common"


fix gha pre commit autoupdate

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 55.98%. Comparing base (50b020b) to head (bd93d03).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #103      +/-   ##
==========================================
+ Coverage   54.77%   55.98%   +1.21%
==========================================
  Files          24       24
  Lines        1446     1486      +40
  Branches      259      268       +9
==========================================
+ Hits          792      832      +40
  Misses        579      579
  Partials       75       75

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


Implement PartialEq, Eq, PartialOrd and Ord for WString

Fix #90.

This implementation is incorrect. You are simply comparing slices, which doesn't take into consideration any padding nulls. The correct Win32 string comparison is made with lstrcmp.

Oh..I don’t know that..
Thank you!
发自我的iPhone
------------------ Original ------------------
From: Rodrigo @.>
Date: Sun,Aug 27,2023 10:34 PM
To: rodrigocfd/winsafe @.>
Cc: 南宫茜 @.>, Author @.>
Subject: Re: [rodrigocfd/winsafe] Implement PartialEq, Eq, PartialOrd and Ord for WString (PR #91)
This implementation is incorrect. You are simply comparing slices, which doesn't take into consideration any padding nulls. The correct Win32 string comparison is made with lstrcmp.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


backupccl: add revision history backup support for user defined schemas

PR: https://github.com/cockroachdb/cockroach/pull/53241
From release notes:

User-defined schemas now support being backed up with revision history and restored with AS OF SYSTEM TIME. [#53241][#53241] {% comment %}doc{% endcomment %}

Doesn't need to be doc'ed. It's implied


Update copyright headers

Copyright headers need to be updated in all source files.

This issue is resolved by #1686

PR #1686 has been merged.


muparser: add darwin support

Support Darwin
Add setfile dependency

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

[ ] NixOS
[X] 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 nix-review --run "nix-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.

Notify maintainers
cc @

Thanks. I will rebase on master.
On Sat, Nov 2, 2019 at 8:46 AM Daiderd Jordan<EMAIL_ADDRESS>wrote:

@LnL7 commented on this pull request.
Looks good, but please target master instead of the release. Changes are
generally only cherry-picked there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/72499?email_source=notifications&email_token=AAAPQITL4OVOV5CUDV6KWKLQRWOFZA5CNFSM4JIBCV4KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCKDBCIQ#pullrequestreview-310776098,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPQIVOJ6OS6QAE2HP3VCTQRWOFZANCNFSM4JIBCV4A
.


update jackson.core.version

Codecov Report

Merging #1037 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #1037   +/-   ##
==========================================
  Coverage      38.36%   38.36%
  Complexity       624      624
==========================================
  Files            112      112
  Lines           4699     4699
  Branches         729      729
==========================================
  Hits            1803     1803
  Misses          2749     2749
  Partials         147      147

Continue to review full report at Codecov.

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


Possibility of strobe effect?

I am trying to use the app on Android to create a strobe effect.  The problem is that I can't get the light to turn on and off faster than about once per second.  Any faster and the on/off states queue up and keep going at a rate of about 1 per second after I "stop" the process by killing the javascript timeout.
I can't be sure if this is related to the plugin, the hardware, or a javascript limitation.  Any thoughts?

I think it's all the overhead of getting the camera and releasing it each time.  I'm working on a modification that lets you do the overhead before and after, just switching the light on and off between.

Did you find a solution to this?


Do not include border of viewfinder

Adjust snap coordinates to only include space inside the viewfinder borders. This should be data-driven, probably, somehow?

I think you fixed this by flooping the tab inside the viewfinder?  or did you want to exclude the border, too?  could the VF tab have a lil button on it to switch which side of the frame it's on?

Yeah I consider this sorta fixed with the tab floop, with caveats - I should make the opacity of the entire viewfinder lower, or perhaps have it go translucent when your mouse isn't over it, so that you can see a little of what's under the border. It's not much but it could be important. Maybe even have a drawing mode where instead of full borders there's just four corner elements. Gotta try some experiments.

Made it translucent. Feels fine to me for now.


Converting from a bsoncxx::value to a specified type

Good morning!
Sorry for another thing to bother you over, but I was wondering how possible it would be to use the BSON converter to convert between a bsoncxx::value to a struct/class. In our project we are storing data inside of a Mongo database using the mongocxx driver, which depends upon libbson.
When we get a document via the driver we get a bsoncxx::view or bsoncxx::value back, and I noticed when reading through the docs that it's possible to implement a custom from_bson method. I understand that the BSON library here is using the C driver, but I was wondering if it would be possible somehow to convert between the bsoncxx:value we get and the reflect-cpp interface in order to serialize the class into our desired type.
There are somethings on the class that exist that I doubt reflect-cpp has any context over, like bsoncxx::oid _id (which most documents in the db have). I find the mongocxx docs are not the best so was hoping you might have some insight if this is possible or potential steps to try and implement it - I know given its a different library/dependency it cannot be implemented into this library itself, but maybe you have an idea of what needs to be done on our side. I imagine this is not a unique issue ^^

Hi, don’t apologise…issues and comments are often just as helpful as code contributions.
As far as an oid is concerned we could simply treat that as an integer.
I agree that the documentation is not the best…implementing the libbson interface was a bit of a nightmare. Is it somehow possible to get the underlying libbson type from bsoncxx::view or bsoncxx::value?

I think it is possible to return the underlying bytes from bsoncxx::value or view. You could then use that to construct a libbson value without copying the either by using this:
https://mongoc.org/libbson/current/bson_init_static.html
or by constructing a BSON value the way we do it here:
https://github.com/getml/reflect-cpp/blob/main/include/rfl/bson/read.hpp
It should then be fairly easy to write a rfl::bson::read function that takes bsoncxx::value or bsoncxx::view as an input.

Here's a feature branch:
https://github.com/getml/reflect-cpp/tree/f/bsoncxx
I have added a function rfl::bson::read_bsoncxx(document_or_view). Could you confirm that it works?

@Lazrius , I hadn't heard back from you, but I want to draft a new release, so I going for this solution for now:
https://github.com/getml/reflect-cpp/commit/7f2be9b7cbe8e967bad667dbe7fe5f1e94a77ed2
I am rather certain that this will work now:
read<YourStruct>(bsoncxx_value_or_view.data(), bsoncxx_value_or_view.length());

But I haven't tested it myself, so I don't want to document it as the proposed solution for this problem. If you could give it a try and then give me feedback whether it worked then I would explicitly put this in the documentation.

So sorry for not getting back to you. I've been busy with a different project and haven't a chance to test it yet. I will try my best to test it next weekend when I have some more time! Thank you again for your work.

OK, no problem.

Very sorry for the delay. While testing it I found that bsoncxx::oid was not a recognised type. Is this a thing I'll have to implement myself?
I tested with this structure:
struct Account
{
        std::string _id;
        std::vector<bsoncxx::oid> characters;
        bool banned = false;
        int64 scheduledUnbanDate = 0;
        int64 cash = 0;
        std::optional<std::vector<std::string>> gameRoles;
        std::optional<std::vector<std::string>> webRoles;
        std::optional<std::string> hashedToken;
        std::optional<std::vector<byte>> salt;
        std::optional<std::string> username;
        std::optional<std::string> passwordHash;
};

And called it like so:
    auto accountsCollection = db.GetCollection("accounts");
    if (!accountsCollection.has_value())
    {
        return;
    }

    const auto accountBson = accountsCollection->GetItemByIdRaw(id);
    if (!accountBson.has_value())
    {
        // TODO: Create Account
        return;
    }

    auto& accountRaw = accountBson.value();
    auto account = rfl::bson::read<Account>(accountRaw.view().data(), accountRaw.view().length());
    if (account.error().has_value())
    {
        return;
    }

@Lazrius , no problem about the delay.
I think what I could to is to support the bson_oid_t. That wouldn't be hard.
Looking at the code, it seems impossible for me to support the bsoncxx class without explicitly having bsoncxx as a dependency. But we're using the C library and it seems like overkill to be introducing a dependency for just this one use case.
Would that be an acceptable solution to the problem?

That makes sense and sounds like a good solution. Out of curiosity, how hard would it be for us to implement the mongocxx version within our project? I assume its just a template definition somewhere?

@Lazrius for discussions on OID support, please refer to issue #69 . I believe this to be a separate issue from bsoncxx::value and bsoncxx::view.

Understood! Thank you very much for your help. This issue can be closed now I think.


Move non-marketing-sales data sources to Advanced/Unsupported section of docs

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1855
Originally created by @DouweM on 2020-03-13 18:37:30

[ ] Move non-marketing-sales data sources that were removed in https://gitlab.com/meltano/meltano/-/issues/1854 to Advanced/Unsupported section of docs. This may be an evolution of https://meltano.com/plugins/extractors/ (which is not the same thing as https://meltano.com/plugins/)

View 7 previous comments from the original issue on GitLab


Incremental Compilation

I'm not sure if incremental compilation is the right term for this. Basically, whenever I make an edit to a query/mutation, I'd like for Apollo to automatically detect this change and recompile so I don't have to rebuild manually to access my new generated query. SQLDelight does this really well when modifying SQL queries.

There are ~2 things~ 3 things:

Incremental compilation: it is the process of compiling a partial subset of files. If you have query1.sql and query2.sql and you only modify query1.sql, we could optimize and not recompile query2.sql.
Up-to-date checks: if you change query1.sql, the plugin should detect the input files have changed and recompile the generated sources. This happens at compile time, not when editing and should already work.
On the fly generation. This is generating files at editing time and I think this is what you are thinking about. SQL delight did it here: https://github.com/cashapp/sqldelight/commit/a55f80a0056b3eff3affa91c1bd0bb3db0dcaa9b. It is significant work but would be nice indeed.

I'd like for Apollo to automatically detect this change and recompile so I don't have to rebuild manually to access my new generated query

Current versions (@martinbonnin pls correct me if I'm wrong) of gradle plugins (including incubating) must behave as you described. Whenever you touched: schema.json or any graphql file, UP-TO-DATE must be false and models should be re-generated.
If you don't see than this is a bug, could you please confirm?

I'm using the current version (1.2.0) and I don't see this behavior

@AOrobator can you confirm you're talking about up-to-date checks (happens at compile time and ahould be working already, we have tests for that) or on-the-fly generation (happens at editing time, this is more work)

On the fly generation, happens at editing time

I believe this is done thanks to the IntelliJ plugin. This also allows going back and forth between usages and query files using Command + Click. If I'm not mistaken, this concept is called Light Class in IntelliJ. It is a way to create fake classes/references on the fly to make the IDE happy.

The newly released IntelliJ Plugin generates models on the fly. See the blog post for more details. Closing this issue.


Scrape the legislative data and upload it to s3

Scrape the legislative ballots from CEE.
The JSON should have the precent number as they key and an object with the ballot and town name as the value. The expected JSON should look like this:
{
  "082": {
      "townName": "Caguas",
      "ballot": "<link to PDF>",
      "ballotData": "<link to JSON data extractor>"
   }
}

Link to PDF found on site.
Link to JSON extractor will be provided later

This is done


some function is not so reasonable

some functions' paramter is FILE* ,these functions may cause program crash.
for the reasons that not all FILE is same in different c/c++ compilers.
for example,if yara dll bulild in msc++ 6 and the dll is being used in msc++ 10,the program will crashed for different FILE definetion.and in gcc,same case is exists.

I understand what you say, but as soon as you don't access members of the FILE structure in your own code (in YARA's code in this case) you are safe. The only thing YARA does is storing pointers to FILE structures, and passing those pointers to the standard runtime library implementing fopen, fread, fwrite, etc. The FILE structure can change from one version of the runtime to the other, but YARA doesn't mind about that, because the structure is opaque to YARA. A problem could arise if you use the fopen function from one runtime version and the fread function from another version, but that's not case.

the FILE structure is not opaqueb to any but public to all。
when you use FILE in msc 10，your FILE definetion is only yours，not yara used(may compiled in msc 8).
suppose in fread for msc 8，just like this:
int fread(FILE* pFile,...)
{
//maybe, in this version of fread，need operate one field like this:
pRead->nCount=0；
//do some thing others
}
but in your msc10 project，you sure your FILE really has a nCount field?
you put your msc10's FILE structure to yara's msc8's fread function，no bug?
发自我的 iPhone

在 2016年6月20日，18:53，Victor M. Alvarez<EMAIL_ADDRESS>> 写道：
I understand what you say, but as soon as you don't access members of the FILE structure in your own code (in YARA's code in this case) you are safe. The only thing YARA does is storing pointers to FILE structures, and passing those pointers to the standard runtime library implementing fopen, fread, fwrite, etc. The FILE structure can change from one version of the runtime to the other, but YARA doesn't mind about that, because the structure is opaque to YARA. A problem could arise if you use the fopen function from one runtime version and the fread function from another version, but that's not case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

and i'll give a stack to proof this:
1,in your project with msc 10，you call
pFile=fopen()；//a msc 10 FILE structure is created
2,you call
yara_scan_file(pFile,...);
3,in the inside of yara_scan_file，fread called:
fread(pFile,...);
4,the fread in 3rd step is the runtime of msc 8，for example msvcr8.dll，
fread in msvc8.dll can use the FILE come from msvc 10 and work well?
发自我的 iPhone

在 2016年6月20日，18:53，Victor M. Alvarez<EMAIL_ADDRESS>写道：
I understand what you say, but as soon as you don't access members of the FILE structure in your own code (in YARA's code in this case) you are safe. The only thing YARA does is storing pointers to FILE structures, and passing those pointers to the standard runtime library implementing fopen, fread, fwrite, etc. The FILE structure can change from one version of the runtime to the other, but YARA doesn't mind about that, because the structure is opaque to YARA. A problem could arise if you use the fopen function from one runtime version and the fread function from another version, but that's not case.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

When you link YARA against a runtime library, both fopen and fread are implemented by the same library, so they are using the same version of the FILE structure. YARA doesn't knows anything about the internal fields of FILE, the fields from that structure are never used within YARA's code, that's what I mean when I say that FILE is opaque to YARA.

but ，when i call yara in my project，the FILE is created by my fopen, and then my FILE is been put into yara's fread，this is the focus
发自我的 iPhone

在 2016年6月21日，00:04，Victor M. Alvarez<EMAIL_ADDRESS>写道：
When you link YARA against a runtime library, both fopen and fread are implemented by the same library, so they are using the same version of the FILE structure. YARA doesn't knows anything about the internal fields of FILE, the fields from that structure are never used within YARA's code, that's what I mean when I say that FILE is opaque to YARA.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

you said my program linked with msc 10 and my program will use msc 10 runtime set:yes，it's right.
but if yara.dll is compiled with msc 6 and yara.dll will use msc 6 runtime。
is that ok?
i create a FILE USING MSC 10 and put it into yara's MSC 6 fread ，it work well?
发自我的 iPhone

在 2016年6月21日，00:04，Victor M. Alvarez<EMAIL_ADDRESS>写道：
Closed #462.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

So, I think your concern is with the function yr_compiler_add_file. You're right about the problem that could arise from mixing two different runtimes (for example by compiling yara.dll with one version of the compiler and your main program with another version). But keep in mind that doing that is very bad idea for a number of reasons not related to YARA. Take a look at this and this. In the later they even mention the problem with the FILE structure.

so you think who using yara.dll should know which compiler the yara.dll was compiled.
but,may be i only get yara.dll from a ways and i only use it,why should i know how it compiled?
it's not reasonable.
as a public yara.dll or yara.so module,it should have less dependcy and work independently with no these special restricts.
keeping all modules of a program to use one same run-time set is impossible at some times expecially the project is depend on many  3rd party modules.
and in fact ,all program run in at least 2 or more different run-time sets.the linux or windows itself also work in this case,but they keep all work well.
发自我的 iPhone

在 2016年6月21日，01:01，Victor M. Alvarez<EMAIL_ADDRESS>写道：
So, I think your concern is with the function yr_compiler_add_file. You're right about the problem that could arise from mixing two different runtimes (for example by compiling yara.dll with one version of the compiler and your main program with another version). But keep in mind that doing that is very bad idea for a number of reasons not related to YARA. Take a look at this and this. In the later they even mention the problem with the FILE structure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

as a solution，i think adding a function like
FILE* yara_open_file(char* pszFile,...)；
into yara public functions set is more reasonable.
it can give a way to ENSURE all yara use the SAME file structure.
发自我的 iPhone

在 2016年6月21日，01:01，Victor M. Alvarez<EMAIL_ADDRESS>写道：
So, I think your concern is with the function yr_compiler_add_file. You're right about the problem that could arise from mixing two different runtimes (for example by compiling yara.dll with one version of the compiler and your main program with another version). But keep in mind that doing that is very bad idea for a number of reasons not related to YARA. Take a look at this and this. In the later they even mention the problem with the FILE structure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

and also ,yes ,i compile the all project files in one run-time set includibg all 3rd party modules and also i have all 3rd party modules' source code.
but,if the compile options is /MT，the bug is still exist.
the module compiled with /MT means every dll/exe has its own heap handle,so all memory should keep/free in a same module.malloc in a.dll and free in b.exe is a bug,causing memory destroyed.
and get back to yara:
i open file in my a.exe，and put my FILE* into yara.dll's fread.
you're sure no memory free/malloc will be doing in fread in yrara.dll?
and when i call fclose to the FILE* in my a.exe ，you are sure no memory pointer from fread in yara.dll would be free?
and ok,you can say ，i should keep all modules compiled in option /MD，
but why?
as a public library,why yara should have so many restrictions?
public library should keep working well with less dependency and restrictions，no need user to know how it was compiled .
is all right?
发自我的 iPhone

在 2016年6月21日，01:01，Victor M. Alvarez<EMAIL_ADDRESS>写道：
So, I think your concern is with the function yr_compiler_add_file. You're right about the problem that could arise from mixing two different runtimes (for example by compiling yara.dll with one version of the compiler and your main program with another version). But keep in mind that doing that is very bad idea for a number of reasons not related to YARA. Take a look at this and this. In the later they even mention the problem with the FILE structure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

so you think who using yara.dll should know which compiler the yara.dll was compiled.
but,may be i only get yara.dll from a ways and i only use it,why should i know how it compiled?
it's not reasonable.

That's why YARA is open source after all. You can build yara.dll yourself with the same compiler you are building your program.

as a solution，i think adding a function like
FILE* yara_open_file(char* pszFile,...)；
into yara public functions set is more reasonable.

A function that receives a filename sounds reasonable, but you can also use yr_compiler_add_string, which receives a buffer with the text for the rules. You can read the rules from a file in the way you prefer, and pass the buffer to yr_compiler_add_string. This way you handle all fopen/fread/fclose operations in your program and just pass a buffer to YARA.

and ok,you can say ，i should keep all modules compiled in option /MD，
but why?
as a public library,why yara should have so many restrictions?

In fact is the other way around, you should not compile your program with /MD just because some yara.dll is built that way. What you should do is building your own yara.dll in a way which is suitable for your project.

We already have a function yr_rules_scan_fd which takes an integer file descriptor (Unix) or a HANDLE (Windows) -- how about adding a function yr_compiler_add_fd?

yes,yr_rules_scan_fd is ok!
generally,i suggest that no any sepcial data type in public functions but only use basic types:int ,void*,char,and so on.
From: Hilko Bengen
Date: 2016-06-21 19:33
To: VirusTotal/yara
CC: mccoysc; Author
Subject: Re: [VirusTotal/yara] some function is not so reasonable (#462)
We already have a function yr_rules_scan_fd which takes an integer file descriptor (Unix) or a HANDLE (Windows) -- how about adding a function yr_compiler_add_fd?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Vehicle testing

Added Load and Save for Vehicles.
Vehicle interaction options have their own separate settings
Settings are applied to the currently active radio

If you have a backpack LR on, you should set vehicle radio to active or it will use the settings from your backpack to save from, and will save to it.
This should be fixed in the future once I'm able to reliably implement a way to set vehicleLR to the active through the script.

Pull Request Issue:

Testing code still inside, have to re-implement conditions for tree settings.

Do not merge at this time

Good for review now.

I'm Bad and Messed up the loading of SR and the tree for vehicle loading and didn't catch it. Fixed in e150b62 push. Did a full test of everything just to make sure I gots it all.

Get Vehicle LR Specifically.
Better implementation, can do checking for if has vehicle LR

Closed for rework


add sdk for Azure.Communication.Messaging

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.Messaging


Syncing repository URL

Package.json: https://docs.npmjs.com/files/package.json#repository
In Bower: https://github.com/bower/bower.json-spec#repository
Quite similar. Would be great to sync this by default. Thanks for the great plugin. We use it daily.

Great idea :) I'll make that change.

v1.0.1

Sync 'repository' by default


Enhancement of Movie-Recommender-System using python

I want to enhance this project by enriching it with much bigger dataset and want to train it again .
Kindly issue this to me

The issue has been assigned to you , Please follow project guidelines for PR.

I am a GSSoC'24 contributor and a machine learning engineer, could you please assign me this issue


[WIP] for the next v0.39 release

Description

Fixes #432
Fixes #433
Fixes #435
Fixes #280
Fixes #191
Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Run meteor npm run lint
[ ] Added an entry to the CHANGELOG.md file.
[ ] Re-reviewed Files changed in the Github PR explorer.

 Launchpad release

 Launchpad release


Add Prowjobs for 1-29 release branch

Add Prowjobs to test building 1-29 variants of build-tooling projects.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Seed and download file on the local machine sometimes successfully

Hi, I am using the Webtorrent API to seed and download files locally. When there are many files being seeded locally and each file is being seeded by one seed(that's why only 0 in torrentSeeds[0] below), some downloaders could block at 'torrent.on('done', callback). In other words, after those downloader calls addPeer(), torrent will not be done forever and there is not any error message. I am really confused why some wait forever, while some can download successfully. Here is the code:
Seeder:
var torrent = new WebTorrent({ dht: false, tracker: false })
  torrent.seed(filePath , function (value) {
    callback(filemeta)
  })

Downloader:
  var client = new WebTorrent({ dht: false, tracker: false })
  var torrent = client.add(torrentSeeds[0].infoHash)

  torrent.addPeer(torrentSeeds[0].ipAddr + ':' + torrentSeeds[0].torrentPort)

  torrent.on('done', function() {
      torrent.files.forEach(function(file){
        var ws = fs.createWriteStream(downloadedFilePath)
        var rs = file.createReadStream()
        rs.pipe(ws)
        rs.on('end', function(){
          callback()
        })
     })
  })

WebTorrent version:
Node.js version: v6.5.0

Thanks.

It looks like you are creating a separate webtorrent client instance for each torrent. That will probably overwhelm your network with DHT traffic. Try using a single webtorrent client across your whole webtorrent process.


Licence

Thanks for the example.
Any plan on adding licence to the project?

Added Apache 2.0 license https://github.com/TeamTechnologies/security-workshop-sample/commit/876d8b07980eb38ded120623a483e56c58f9bc5f


Unicode emoji doesn't work for Windows 7+

OS X and iOS aren't the only 2 OSs that ship with an Emoji font, Windows 8+ does as well (And it was backported to Windows 7), as it stands the script unnecessarily converts the emoji to images for these platforms when they're capable of displaying them.

the patch for this landed some time ago: https://github.com/iamcal/js-emoji/commit/3ca22c79bb93427cf33cd62982b93384e6a974a9


Meteor 2.13 support

Fixes https://github.com/zodern/meteor-docker/issues/32

@zodern could you take a look at see if this can be merged? Would be nice to be able to deploy Meteor 2.13 using meteor-up!

        You need to run "nvm install v14.21.4" to install it before using it.
        dirname: missing operand
        Try 'dirname --help' for more information.

              ------------------------------------------------------------------------------

Workaround while this is pending:
according to https://forums.meteor.com/t/announcing-extended-support-maintenance-for-node-js-14/59811/9
the officially supported image is:
Meteor NodeJS 23: Nodejs runtime from Meteor and Compatible with MUP

@vachigaggl Are you deploying rocketchat with this image?

While you are waiting for this to get merged, you could use our custom image: https://hub.docker.com/r/duodeka/meteor.

This image has this line commented out
https://github.com/zodern/meteor-docker/blob/076a7ea1a128e419d90b6b4ad74bb3a01be6d4e5/image/scripts/start.sh#L15

For anyone who has built @jdgjsag67251 root image, and it's throwing this error, https://github.com/zodern/meteor-docker/pull/34#issuecomment-1742532173 (@aboire)
Basically, root-image/Dockerfile is extending the current zodern/meteor and you're not really using @jdgjsag67251 fix.

cd image/ and build that base image docker build --no-cache -t jdgjsag67251/meteor .
vim root-image/Dockerfile, replace the first line that uses @zodern image by the one that you just built FROM jdgjsag67251/meteor. If you used another tag name for the first step, use that one then
cd root-image/ and build the root image  docker build --no-cache -t jdgjsag67251/meteor:root .

For everyone facing the problem, till this PR is merged you can my custom image momenysr/meteor:root and your problem will be solved.
in mup.js file:
...
docker: {
    image: 'momenysr/meteor:root',
}
....

For everyone facing the problem, till this PR is merged you can use my custom image momenysr/meteor:root and your problem will be solved.
in mup.js file:
...
docker: {
    image: 'momenysr/meteor:root',
}
....

Thank you! That worked for me.

Thank you for sharing @MomenYasser
`> > docker: {

image: 'momenysr/meteor:root',

}
....
`

The app does start with this image, but the  buildInstructions: [...
are not running. Took me a while to realise it. Not sure why though.

Does anyone find a solution or a working image in root with buildInstructions ?

While you are waiting for this to get merged, you could use our custom image: https://hub.docker.com/r/duodeka/meteor.

it works , thank 🚀

I'm sorry which image should I use for meteor 2.13?

Any of the tags published since this PR was merged supports Meteor 2.13. For example: latest, root, 0.7.1, 0.7.1-root.


Syntax error in SQL statement "CREATE TABLE CM_PII_CATEGORY  in Consent Script against H2 2.1.210

How to reproduce:
Trying to do a migration from IS 5.11.0 GA to IS 5.12.0 Beta2. While getting 5.11.0 GA H2 Consent script executed getting below errors and therefore unable to try out the migration against H2 latest (2.1.210).

Get a external H2 database configured with docker . Ref https://hub.docker.com/r/oscarfonts/h2

I. docker pull oscarfonts/h2

II. docker run -d -p 1521:1521 -p 81:81 -v /path/to/local/data_dir:/opt/h2-data -e H2_OPTIONS=-ifNotExists --name=MyH2Instance oscarfonts/h2

III. docker logs -f MyH2Instance

IV. Access http://<ip>:81

Give a database name in the UI browser console for the H2 (eg:- test2. So a new database will be created as test2 when you click on connect
Get the 5.11.0 GA pack from git
Execute the script dbscripts/consent/H2.sql against the H2 server

Getting below Errors

Syntax error in SQL statement "CREATE TABLE CM_PII_CATEGORY (\000d\000a  ID           INTEGER AUTO_INCREMENT,\000d\000a  NAME         VARCHAR(255) NOT NULL,\000d\000a  DESCRIPTION  VARCHAR(1023),\000d\000a  DISPLAY_NAME VARCHAR(255),\000d\000a  IS_SENSITIVE INTEGER      NOT NULL,\000d\000a  TENANT_ID    INTEGER DEFAULT '-1234',\000d\000a  UNIQUE [*]KEY (NAME, TENANT_ID),\000d\000a  PRIMARY KEY (ID)\000d\000a)"; expected "("; SQL statement:
CREATE TABLE CM_PII_CATEGORY (
  ID           INTEGER AUTO_INCREMENT,
  NAME         VARCHAR(255) NOT NULL,
  DESCRIPTION  VARCHAR(1023),
  DISPLAY_NAME VARCHAR(255),
  IS_SENSITIVE INTEGER      NOT NULL,
  TENANT_ID    INTEGER DEFAULT '-1234',
  UNIQUE KEY (NAME, TENANT_ID),
  PRIMARY KEY (ID)
) [42001-210] 42001/42001 [(Help)](https://h2database.com/javadoc/org/h2/api/ErrorCode.html#c42001)

Expected behavior:

Environment information (Please complete the following information; remove  any unnecessary fields) :
H2 2.1.210
IS 5.11.0 GA

This is happening because H2 1.4.xx was used in 5.11.0 GA, hence, those are not compatible with H2 2.1.210.
These tables are getting created without an issue in their compatible DB versions.
5.11.0 GA - H2 1.4.xx heavy_check_mark
5.11.0 U2 updated - H2 2.1.210 heavy_check_mark


Support the external postgresql database with a postgresql URI

Signed-off-by: myan<EMAIL_ADDRESS>Reference: https://github.com/stolostron/backlog/issues/24506

/reset

/cc @clyang82 @morvencao

As we discussed in slack, we agreed to move DB initialization work to operator.


[3.0.4] Timeout on Android

Environment

System:
    OS: macOS 13.0.1
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 20.41 GB / 64.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 14.20.0 - ~/.nvm/versions/node/v14.20.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v14.20.0/bin/yarn
    npm: 8.18.0 - ~/.nvm/versions/node/v14.20.0/bin/npm
    Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.11.3 - /Users/stathis/.rvm/gems/ruby-3.0.0/bin/pod
  SDKs:
    iOS SDK:
      Platforms: DriverKit 22.1, iOS 16.1, macOS 13.0, tvOS 16.1, watchOS 9.1
    Android SDK: Not Found
  IDEs:
    Android Studio: 2021.3 AI-213.7172.25.2113.9123335
    Xcode: 14.1/14B47b - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.13 - /usr/bin/javac
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^17.0.2 => 17.0.2
    react-native: ^0.68.5 => 0.68.5
    react-native-macos: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Platforms

Android
Versions

Android: 13
iOS:
react-native-geolocation: 3.0.4
react-native: 0.68.5
react: 17.x

Description

After upgrading to 3.0.4 I get location timeouts, sometimes it manages to get location some time it does not. Please note that the device (Samsung S22 Android 13) doesn't have a sim card. Downgrading to 3.0.3 it works as a charm even though I had to patch-package this PR.
edit: the granted permissions are approximate, I've opened google maps, gave approximate permissions too and it could find my location.
These are the configs:
App.tsx:
 useEffect(() => {
    Geolocation.setRNConfiguration({
      skipPermissionRequests: true,
      authorizationLevel: "whenInUse",
      locationProvider: "auto"
    });
  }, []);

and :
export const getPosition = async () => {
  return new Promise<{ lat: string; lng: string }>((ful, rej) => {
    Geolocation.getCurrentPosition(
      (location) => {
        console.log("position fulfilled");
        const { latitude: lat, longitude: lng } = location.coords;
        ful({ lat: String(lat), lng: String(lng) });
      },
      (error: GeolocationError) => {
        console.log("location error: ", error);
        if (error.code === PositionError.PERMISSION_DENIED) {
          console.log("position denied");
          rej(new Error("com.xxx.home.permission_denied"));
        } else if (error.code === PositionError.POSITION_UNAVAILABLE) {
          console.log("position unavailable");
          notify(error.message, "warning", "location position unavailable");
          rej(new Error("com.xxx.home.permission_unavailable"));
        } else if (error.code === PositionError.TIMEOUT) {
          console.log("position timeout");
          notify(error.message, "warning", "location timeout");
          rej(new Error("com.xxx.home.permission_timeout"));
        } else {
          rej(error);
        }
      },
      {
        enableHighAccuracy: true,
        distanceFilter: 250, // 100 meters
        maximumAge: 20000,
        timeout: 120000
      }
    );
  });
};

Reproducible Demo

same

after switching to  locationProvider: "playServices" I do not get timeouts anymore on Samsung S22 Android 13 but on a Xiami Note 8 Android 11 it throws:
FATAL EXCEPTION: main
Process: com.xxxx, PID: 30274
java.lang.NullPointerException: Listener must not be null
at com.google.android.gms.common.internal.Preconditions.checkNotNull(com.google.android.gms:play-services-basement@@18.1.0:2)
at com.google.android.gms.common.api.internal.ListenerHolders.createListenerKey(com.google.android.gms:play-services-base@@18.0.1:1)
at com.google.android.gms.location.FusedLocationProviderClient.removeLocationUpdates(com.google.android.gms:play-services-location@@20.0.0:6)
at com.reactnativecommunity.geolocation.PlayServicesLocationManager$1.onLocationResult(PlayServicesLocationManager.java:70)
at com.google.android.gms.internal.location.zzaw.notifyListener(com.google.android.gms:play-services-location@@20.0.0:2)
at com.google.android.gms.common.api.internal.ListenerHolder.zaa(com.google.android.gms:play-services-base@@18.0.1:2)
at com.google.android.gms.common.api.internal.zacb.run(Unknown Source:4)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:236)
at android.app.ActivityThread.main(ActivityThread.java:8057)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:656)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:967)

After running the example I was able to get the location only if enableHighAccuracy: false.  The test case High Accuracy always fails with timeout. On normal example I've added timeout: 120000 but still I get timeout.
Please note:

I do not have a SIM card on both devices
I'm standing next to a window in the last floor of my building with clear sky, no other buildings around or something that could block the gps.

I've got the exact same issue with Android when enableHighAccuracy : true as @efstathiosntonas with the same circumstances (testing indoors near the window, no SIM card). Works fine on iOS.

Should be addressed in https://github.com/michalchudziak/react-native-geolocation/pull/230
In the meanwhile, you can get around this by just passing in a sensible timeout value

Addressed in 3.0.5

I have 3.0.5 and the problem remains

I have 3.3.0 and the problem remains

Same here

still remains


Entry fs.existsSync check breaks with custom extensions

Current Behavior
With the new config style, Haul now tries to verify that the provided entrypoints exist via node path resolution. This breaks support for custom extensions. In our config, we add support for TypeScript, and now our TypeScript-based entrypoints are failing to pass this check.
Expected Behavior
Haul should rely on Webpack's file resolution instead of node's, enabling custom extensions in entrypoints. When I remove the check locally, building succeeds as expected.
Haul Configuration (webpack.haul.js)
A simplified example of our config:
...

const config = createWebpackConfig({
    entry: "index.ts",
})(env);

config.resolve.extensions.push(`.${platform}.ts`, ".native.ts", ".ts");

...

Your Environment

software
version

Haul
v1.0.0-rc.0

react-native
v0.55.3

node
v9.11.1

npm or yarn
yarn v1.6.0

I agree @jaredru we can remove whole check there https://github.com/callstack/haul/blob/next/src/utils/makeReactNativeConfig.js#L322
I will approve your PR if you make a time for it. 💪

Fixed, thanks @jaredru 🚀


[DO NOT MERGE] Dummy PR to trigger CI on the dev branch

Codecov Report

Merging #118 (a605c69) into dev (a322ccf) will decrease coverage by 0.66%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #118      +/-   ##
==========================================
- Coverage   92.80%   92.13%   -0.67%
==========================================
  Files           7        7
  Lines         264      267       +3
==========================================
+ Hits          245      246       +1
- Misses         19       21       +2

Impacted Files
Coverage Δ

src/common.jl
91.30% <0.00%> (-8.70%)
:arrow_down:

Continue to review full report at Codecov.

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

All green!


Consider disabling compression for tiny responses

Minor performance tweak...
A 1-byte response of type "text/plain" will get intercepted and gzip'ed. The overhead of compressing a tiny response takes up CPU and likely extra bandwidth.
Sample middleware:
app.Run(c =>
{
  c.Response.Headers["Content-Type"] = "text/plain";
  return c.Response.Body.WriteAsync(new byte[] { 65 }, 0, 1);
});

This gets transmitted as 'chunked' transfer encoding (even if I set c.Response.Headers["Content-Length"] = "1";) and it gets compressed using gzip
It doesn't happen if the content-type is "application/octet-stream".
Consider disabling compression for responses under a certain size (200 bytes, 500 bytes, 800 bytes, everyone seems to have a different recommendation)
(I first posted this in the corefx repo - oops!  I hope this is the proper place!)

Are you using the new response compression middleware (https://github.com/aspnet/BasicMiddleware/blob/dev/samples/ResponseCompressionSample/Startup.cs#L34) or just IIS/Express?

IIS/Express - no middleware

@shirhatti ?

For additional clarity, I'm using 1.0.1, Visual studio 2015, brand new asp.net core (.net core) project, and in Startup.cs:
// This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
public void Configure(IApplicationBuilder app, IHostingEnvironment env, ILoggerFactory loggerFactory)
{
    app.Run(c =>
    {
        c.Response.Headers["Content-Type"] = "text/plain";
        return c.Response.Body.WriteAsync(new byte[] { 65 }, 0, 1);
    });
}

And here's the raw response headers (followed by about 120 bytes of gzip data):
HTTP/1.1 200 OK
Transfer-Encoding: chunked
Content-Type: text/plain
Content-Encoding: gzip
Vary: Accept-Encoding
Server: Kestrel
X-SourceFiles: =?UTF-8?B?RDpcUHJvamVjdHNcV2ViQXBwbGljYXRpb24xXHNyY1xXZWJBcHBsaWNhdGlvbjE=?=
X-Powered-By: ASP.NET
Date: Wed, 02 Nov 2016 23:08:36 GMT

Yeah, this is unrelated to ASP.NET Core, it's an IIS/Express feature

Ahh, my bad.


ci: commit helm chart files on release

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