64 bit support and machine additions, #7

Fixes #7
Note already reviewed and approved by community CCB on babelfish.  Submitting pull request to follow GitHub process for merging to master.
Work performed by @the-other-james

Updated commit message style to match proposed standard.

Changed commit message only to match as-of-yet unpublished standard.

Note this was approved for merge to master as part of CCB 2019-09-04 review (since it was already tested/merged on babelfish).


Adding support for CommonJS and AMD registration.

Allows for registration as a CommonJS module or registration with an AMD, such as RequireJS.
Falls back to the browser global jQuery object.

I fetched this from your branch, and it looks like you removed the commonjs definition.  Is this intended?

Yeah, mostly it was from the discussion here: https://gist.github.com/1344389
From one of @jrburke comments:
"I just added a version that does not do optional CommonJS registration, since jQuery is unlikely to run in those environments"
It made the addition a little smaller so I decided to remove it.


adjust func2hcp behaviour to match current hcp pipelines..

From Tim Coalson

The func2hcp.py script uses the -volume-parcel-resampling-generic command that we took out of our pipelines, in order to instead do cubic spline resampling, resolving an issue of it creating a grid pattern in the noise variance (since this command uses a form of linear resampling).  We now use -cifti-resample after making a subject-specific subcortical cifti file at the acquired resolution.  See here:
https://github.com/Washington-University/Pipelines/blob/master/fMRISurface/scripts/SubcorticalProcessing.sh

done


CMS_decrypt*(): fix misconceptions and memory leak

It turns out that my recent implementation of CMS_EnvelopedData_decrypt() in #18301 was somewhat mislead

by the set1_ part of the function name CMS_decrypt_set1_password() and
mostly by the fact that all the CMS_decrypt*() functions were insufficiently documented.

The reviewers did not notice presumably for the same reason(s).
Both the function and the docs are fixed by this PR, pointing out the potential pitfall.
The catch is that all CMS_decrypt*() functions do not only set/use decryption credentials
but actually attempt decryption (unless CMS_decrypt() is called with a NULL pkey argument)
and this will fail if not just the right type of decryption key (symmetric or asymmetric) is provided.
BTW, this CMS API is (needlessly) limited in case the recipient does not know which decryption credential to use, which may be asymmetric and/or symmetric. So it could be helpful to provide both, in case it is unclear which type of encryption the peer used.
In the original implementation of CMS_EnvelopedData_decrypt() I had assumed that both types of decryption keys can be provided at the same time, but this not only lead to the error(s) quoted here:
...:CMS routines():crypto/cms/cms_smime.c:776: ERROR: no matching recipient
...:OSSL_CRMF_ENCRYPTEDKEY_get1_pkey():crypto/crmf/crmf_lib.c:717: ERROR: error decrypting encryptedkey
...:ossl_cmp_certresponse_get1_cert_key():crypto/cmp/cmp_msg.c:1102: ERROR: failed extracting central gen key
...:get1_cert_key_status():crypto/cmp/cmp_client.c:457: ERROR: certificate not found:; cannot extract certificate from response
...

but also to a tricky memory leak:
=================================================================
==1366017==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 48 byte(s) in 1 object(s) allocated from:
    #0 0x7f03d5997e8f in __interceptor_malloc ../../../../src/libsanitizer/asan/asan_malloc_linux.cpp:145
    #1 0x7f03d54312a4 in CRYPTO_malloc crypto/mem.c:196
    #2 0x7f03d5330f59 in ossl_cms_RecipientInfo_pwri_crypt crypto/cms/cms_pwri.c:378
    #3 0x7f03d532b40d in CMS_RecipientInfo_decrypt crypto/cms/cms_env.c:1016
    #4 0x7f03d5336c1a in CMS_decrypt_set1_password crypto/cms/cms_smime.c:831
    #5 0x7f03d5801295 in CMS_EnvelopedData_decrypt crypto/crmf/crmf_lib.c:983
    #6 0x7f03d57ff750 in OSSL_CRMF_ENCRYPTEDKEY_get1_pkey crypto/crmf/crmf_lib.c:714
    #7 0x7f03d5819d2a in ossl_cmp_certresponse_get1_cert_key crypto/cmp/cmp_msg.c:1096
    #8 0x7f03d5821709 in get1_cert_key_status crypto/cmp/cmp_client.c:455
    #9 0x7f03d5822325 in cert_response crypto/cmp/cmp_client.c:578
    #10 0x7f03d5823241 in OSSL_CMP_exec_certreq crypto/cmp/cmp_client.c:726
    ...

SUMMARY: AddressSanitizer: 48 byte(s) leaked in 1 allocation(s).

which is fixed - hopefully in a sufficient way - by this PR, too.

I do not think this will cherry-pick cleanly to 3.0 branch.

Thanks Tomáš.
BTW, as I understand, no need to backport this to 1.1.1 because this is not a security fix.

Ping @openssl/committers  for 2nd review

Asking @t8m to re-approve and @slontis to provide the 2nd review+approval.

Thinking a little more about the tricky mem leak, I realized that the same trouble can happen also with
the roles of  CMS_decrypt_set1_password and CMS_decrypt_set1_pkey{,_and_peer} reversed.
So I've added the same fix also in this symmetric case, namely to CMS_decrypt_set1_password().
@t8m, please reapprove.

Today I came across a further issue with CMS decryption, namely a misleading error:
whenever anything goes wrong with decryption, e.g., due to a wrong decryption key or password,
CMS_decrypt_set1_key(), CMS_decrypt_set1_pkey_and_peer(), and CMS_decrypt_set1_password() unconditionally and wrongly report CMS_R_NO_MATCHING_RECIPIENT, e.g.:
CMS routines():crypto/cms/cms_pwri.c:387: ERROR: unwrap failure
CMS routines():crypto/cms/cms_smime.c:847: ERROR: no matching recipient

Added a commit fixing this, such that CMS_R_NO_MATCHING_RECIPIENT is reported only if no matching recipient was found.

Thanks @t8m for the re-approval.
@paulidale, fine now?

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.

Merged - thanks @t8m and @paulidale

I do not think this will cherry-pick cleanly to 3.0 branch.

Ah right, CMS_EnvelopedData_decrypt() is not present there,
and there are also some conflicts in the doc files.
So removed the 3.0 tag and will have to provide a follow-up PR with the changes relevant for 3.0.

The exchange in https://github.com/openssl/openssl/pull/19981#issuecomment-1414230784 reminded me of this.
Done now in #20209, also including the fixup done in #19981.


Add libbitcoinconsensus to the project as resource

The library should be added to the project in one way or another.
Ideally there were some safe guards, to ensure the file was not tempered with.
This may include:

comparing file hashes against known references (how? as test?)
a lot more tests

The library should be build via Gitian, or extracted from the official release.

I see this PR as being about two things, but we can create a separate PR if you want to split it up.
#1 - Automating the process of loading in the library binaries
#2- Adding the binaries to the distribution
I'm actually more interested in #1 at the present because I want to use this project to test various binaries. I was thinking the next step would be to replace the download and verify code you have in the .travis.yml file with roughly equivalent code in build.gradle. That will allow people to easily configure the test to use different builds of libbitcoinconsensus.  Does that sound like a reasonable approach?

It would also be nice to be able to download the test JSON files that match the version of the lib we're testing.

Automating the process of loading in the library binaries

Sorry, can you please clarify? If the library is locally installed, then it should be available without providing an explicit path. But I assume you were referring to something else?

Sorry, can you please clarify?

My idea is to add the rough equivalent of what your .travis.yml file does to download the libraries to the Gradle file. That way a user can change some variables (properties file) to run the test against different versions of bitcoin. We may also want (need) to pull different versions of the JSON test files.

the current test set fails with 0.13 libbitcoinconsensus

I'm wondering if we should update the default (checked-in) tests to match the latest (released?) version of Bitcoin Core...

My idea is to add the rough equivalent of what your .travis.yml file does to download the libraries to the Gradle file.

Ah gotcha, so you want to download the library files via Gradle. Sounds good to me as a start. We might provide different versions (e.g. libs from 0.10-0.13).

I'm wondering if we should update the default (checked-in) tests to match the latest (released?) version of Bitcoin Core...

Generally I'd say yes. However, I'm not sure how to deal with changes in the interface. There is a new bitcoinconsensus_verify_script_with_amount since 0.13, which isn't available in earlier versions. If we are going to provide support for the older libs, this could be an issue?


Codeunit 80 Sales-Post - Hide Dialog

Hi,
many thanks for implementing HideProgressWindow for suppressing dialogs!
In the code unit 80 there are some Window.Close statements which are not prefixed with "IF NOT HideProcessWindow". According to the help a dialog.lose without dialog.open leads to an error. Has the behavior of the dialog.close changed here?
Here is the link to the corresponding help:
https://docs.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/methods-auto/dialog/dialog-close-method
Originally posted by @AJasp in https://github.com/Microsoft/ALAppExtensions/issues/963#issuecomment-479523538

As I can see this is fixed long time ago but request is still open. Closing.


Update rubocop configuration and add rubocop_todo.

Before we had a file with all the rubocop rules defined, this is the incorrect way to use rubocop. This change adds a rubocop_todo.yml to overwrite the base configuration.

Coverage remained the same at 99.9% when pulling 30cf06b609b5d24f3720ea27168b44c3f0213b1c on pedelman:rubocop_updates into a79019168ada0c622a4a5759952c16885e122625 on bigcommerce:master.

LGTM :+1:


Microsoft.Azure.DocumentDB.1.6.1 in cosmos db integration tests example

Example on github does not compile because it uses 1.6.1 version of Microsoft.Azure.DocumentDB which is too old

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

ID: 617f002c-f40e-28a0-2ab7-e3227e08e37c
Version Independent ID: a06ba438-55c8-e77c-ec89-095d44fbf4f3
Content: Set up CI/CD pipeline with Azure Cosmos DB Emulator build task
Content Source: articles/cosmos-db/tutorial-setup-ci-cd.md
Service: cosmos-db
GitHub Login: @deborahc
Microsoft Alias: dech

@fjod Thank you for the feedback. We are actively investigating and will get back to you soon.

@fjod This is being assigned to the content author to evaluate and update as appropriate.

@milismsft can you please look into this issue?

Apologies for the very slow reply to this.
I have reviewed this and have updated the link to point to a newer version of our To Do app sample. This is in a PR now and should publish sometime today.
thanks for reporting. Will close this issue now.
#please-close

#assign:markjbrown


Added check for Mac OS X version.

If High Sierra, use different permissions for '/usr/local' that when older version of Mac OS X.
This should fix #76 . Not sure if what I did is the best way to test for a Mac OS X version, but I reckon it's probably good enough.

I need this 👍

Sorry about that—I thought the permissions adjustment would apply to all versions correctly, but now I see that was not the case. Merged, and released in 2.3.1!

Shouln't this use version_compare? I'm assuming it wont go back to the old behavior in upcoming macos releases.

@oxyc yeah, that may be better. I wanted to at least get this fix in as-is but that would be a good follow-up, since it should be 10.13+ vs < 10.13.

Ah, cool, didn't know about the version_compare filter. That would indeed be better, slimmer chance for it to fail in the next release. I reckon we've got a bit of time to prepare though :-)

Is there some reason that we can't do these in macOS 10.13?
    group: admin
    mode: 0775

Those look harmless to me, and would allow collapsing the 2 tasks back into 1.

According to #75 , brew fails on High Sierra with those permissions. I don't know what the exact error is, as I don't have access to a High Sierra box.


[On dev branch] Tuple rewrite

Contents in this PR:

Remove support for tuple y0 and tuple-based sde for sdeint.

Rationale: This feature is somewhat niche. Supporting this would likely require adding an alternative tuple-based ForwardSDE that has the correct versions of f, g_prod, gdg_prod, and dg_ga_jvp (per our email discussion).

Make AdjointSDE take in ForwardSDE and use the latter's g_prod.

This simplifies g_prod of the former.

Make AdjointSDE take in (and output) flattened tensors and perform flattening/unflattening inside the registered functions.

One advantage of this is that we only need to unflatten 2 tensors now.

Refactor solvers to not be based on tuples.
Explicit flattening/unflattening in adjoint.py

This differs slightly from the current implementation in torchdiffeq, where the _TupleFunc construction is in _check_inputs.
Rationale: We assume that sdeint.integrate only supports tensor-based y0 and sde, and we should enforce this usage for now.

Filter out the parameters that don't require gradients in sdeint_adjoint.
Fix plot folder names for files in diagnostics.
Temporarily deprecating logqp support.

This should either be added back or the example latent_sde rewritten with state augmentation when we get the general noise adjoint working with the log-ODE scheme. (I view getting the log-ODE scheme-based adjoint working for general noise Strat SDE as a milestone. )

Fix scalar noise shape check and it's functionality in sdeint inconsistency. Hence, resolving #27.

The rate diagnostics files run, but I haven't been able to obtain rates due to #36. I need to run these when this is fixed. All tests pass.
Overall, this PR should also make adding dg_ga_jvp for the adjoint easier.

Also,  tests/__init__.pyc shouldn't be included in the commit.

Overall, things are looking so much cleaner now. These changes make me very happy.

Just spotted, both sdeint and its adjoint have the docstring for names still specifying the possibility of prior drift.

Just spotted, both sdeint and its adjoint have the docstring for names still specifying the possibility of prior drift.

Done.

This PR looks good to me. I've not hit the merge button as I assume the plan is to merge #40 first, so that the diagnostics can be run.

Thanks for the reviews @patrick-kidger! All rates preserved after running the diagnostics. Merging now.


[BUG] Support for ASGI startup events

Investigative information
Please provide the following:

Timestamp: 2021-11-10T10:58:00
Function App name: (debugging locally)
Function name(s) (as appropriate):(debugging locally)
Core Tools version: 3.0.3785 Commit hash: db6fe71b2f05d09757179d5618a07bba4b28826f  (64-bit)

Repro steps
Provide the steps required to reproduce the problem:

Setup a new Function App, with an HTTP trigger and the following code in func/__init__.py:

import logging

import azure.functions as func
from azure.functions import AsgiMiddleware
from api_fastapi import app

IS_INITED = False

def run_setup(app, loop):
    """Workaround to run Starlette startup events on Azure Function Workers."""
    global IS_INITED
    if not IS_INITED:
        loop.run_until_complete(app.router.startup())
        IS_INITED = True

def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    asgi_middleware = AsgiMiddleware(app)
    run_setup(app, asgi_middleware._loop)
    return asgi_middleware.handle(req, context)

After the 1st request, you will get the following exception:

[2021-11-10T09:58:11.178Z] Executed 'Functions.api_az_function' (Failed, Id=50cbe047-48bd-42be-9314-0194c2e17ec9, Duration=65ms)
[2021-11-10T09:58:11.181Z] System.Private.CoreLib: Exception while executing function: Functions.api_az_function. System.Private.CoreLib: Result: Failure
Exception: RuntimeError: Task <Task pending name='Task-39' coro=<AsgiResponse.from_app() running at C:\Program Files\Microsoft\Azure Functions Core Tools\workers\python\3.8/WINDOWS/X64\azure\functions\_http_asgi.py:65> cb=[_run_until_complete_cb() at C:\Users\User\AppData\Local\Programs\Python\Python38\lib\asyncio\base_events.py:184]> got Future <Future pending> attached to a different loop

Expected behavior
Provide a description of the expected behavior.

The Function should be able to fulfil any number of requests.

Actual behavior
Provide a description of the actual behavior observed.

Because the AsgiMiddleware is instantiated for each call, the event loop is not reused and thus we get the exception.

Known workarounds
Provide a description of any known workarounds.

Have a global AsgiMiddleware object as shown here:

import logging

import azure.functions as func
from azure.functions import AsgiMiddleware
from api_fastapi import app

IS_INITED = False
ASGI_MIDDLEWARE = AsgiMiddleware(app)

def run_setup(app, loop):
   """Workaround to run Starlette startup events on Azure Function Workers."""
   global IS_INITED
   if not IS_INITED:
       loop.run_until_complete(app.router.startup())
       IS_INITED = True

def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
   run_setup(app, ASGI_MIDDLEWARE._loop)
   return ASGI_MIDDLEWARE.handle(req, context)

Contents of the requirements.txt file:
Provide the requirements.txt file to help us find out module related issues.
azure-functions==1.7.2
fastapi==0.70.0

After more digging in azure.functions._http_asgi, I managed to reuse the worker's event loop like this:
import logging

import azure.functions as func
from azure.functions._http_asgi import AsgiResponse, AsgiRequest
from api_fastapi import app

IS_INITED = False

async def run_setup(app):
    """Workaround to run Starlette startup events on Azure Function Workers."""
    global IS_INITED
    if not IS_INITED:
        await app.router.startup()
        IS_INITED = True

async def handle_asgi_request(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    asgi_request = AsgiRequest(req, context)
    scope = asgi_request.to_asgi_http_scope()
    asgi_response = await AsgiResponse.from_app(app, scope, req.get_body())
    return asgi_response.to_func_response()

async def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    await run_setup(app)
    return await handle_asgi_request(req, context)

In Improve throughput performance of Python apps in Azure Functions, it is recommended to define the main function as asynchronous.
Shouldn't this then be the preferred way to run ASGI applications?

I'm facing a similar problem. Based on https://github.com/ecerami/fastapi_azure I have create an Azure function with FastAPI inside. Everything works very smooth so far but my startup event handler is not run
from fastapi import FastAPI
app = FastAPI()

@app.on_event("startup")
def startup_event():
    raise Exception()  # is not raised

After more digging in azure.functions._http_asgi, I managed to reuse the worker's event loop like this:
import logging

import azure.functions as func
from azure.functions._http_asgi import AsgiResponse, AsgiRequest
from api_fastapi import app

IS_INITED = False

async def run_setup(app):
    """Workaround to run Starlette startup events on Azure Function Workers."""
    global IS_INITED
    if not IS_INITED:
        await app.router.startup()
        IS_INITED = True

async def handle_asgi_request(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    asgi_request = AsgiRequest(req, context)
    scope = asgi_request.to_asgi_http_scope()
    asgi_response = await AsgiResponse.from_app(app, scope, req.get_body())
    return asgi_response.to_func_response()

async def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    await run_setup(app)
    return await handle_asgi_request(req, context)

In Improve throughput performance of Python apps in Azure Functions, it is recommended to define the main function as asynchronous.
Shouldn't this then be the preferred way to run ASGI applications?

This works for me too, but there is no way to perform the app global teardown operations (FastAPI shutdown event), since you have no hook on the running threadpool context of function worker.
This is dirty if you have some channel open (like an open AIOHttp session) :(

I've run across a different issue with the AsgiMiddleware and arrived at basically the same solution as posted above. I noticed that simple async functions do not behave as I would expect an async application should. For example, I tested the following:
from __app__.app import app
import azure.functions as func
import asyncio
import nest_asyncio
nest_asyncio.apply() # as suggested in docs

@app.get("/api/test")
async def test(code: Optional[str] = None):
    await asyncio.sleep(5)
    return "OK"

async def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    return await AsgiMiddleware(app).handle(req, context)

If you hit this endpoint above with a few concurrent requests, you'll notice that they execute serially instead of concurrently (I assume because the AsgiMiddleware blocks with a call to run_until_complete here). E.g., if I hit this endpoint twice in rapid succession, the second request will take ~10 seconds to complete instead of ~5 seconds.
Using a plain azure function without the middleware works as expected (e.g., the above scenario completes in ~5 seconds with two concurrent requests):
async def main(req: func.HttpRequest, context: func.Context) -> func.HttpResponse:
    await asyncio.sleep(5)
    return "OK"

As mentioned, the posted solution resolves this, but I'm concerned about the implications of this as mentioned above. It would be great to see a robust solution for this in the near future if possible.

@rbudnar Yeah I'm struggling with this too https://github.com/Azure-Samples/fastapi-on-azure-functions/issues/4

@rbudnar This is fixed now here https://github.com/Azure/azure-functions-python-library/pull/143 which also removes the need for nest_asyncio

I am having a similar issue, the startup event of FastAPI is not working but works when included in the main loop. This leads to it running every time a request comes in

@ApurvaMisra maybe you would be able to put it inside an extension using  AppExtensionBase: https://learn.microsoft.com/en-us/azure/azure-functions/develop-python-worker-extensions?tabs=windows%2Cpypi . If you are using Premium you can use warmup :-)

Submitted a draft implementation of lifespan events that would work with this example (in the V2 programming model)
https://github.com/Azure/azure-functions-python-library/pull/187

Same issue here, any updates on that?


Character Editor Problem

Describe your bug here.
So I Compiled PE 0.7.3 Today And The Character Edititor Is Just A Black Screen. All The Ui Is There But Nothing Works. Please Provide Help :(
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
no

same here

any way how to fix?


[Feature request] Package binary also for arm darwin M1/M2 chips

I can see in packages three different packages for darwin, linux and windows, all x86_64
It would be great to be able to use this CLI natively also on arm chips like the M1/M2 ones.

Yeah, a universal build for macOS supporting x86_64 and arm64 would be awesome. The x86_64 build works thanks to Rosetta 2 but who knows how long Apple will support it.


Update README-pt.md

Add translation of Building the API reference pages section
Fix typos
Remove duplicate whitespaces
Remove trailing spaces

/language pt

/lgtm
/approve

/assign @savitharaghunathan

This lgtm I understand pt-pt but just to be fair I'd like someone from @kubernetes/sig-docs-pt-owners to take a look into this PR

/assign @kubernetes/sig-docs-pt-owners


add power flash to the mod

add power flash to the mod

Not exactly sure what that is, do you think you could give more context and maybe provide examples?

ohhh i see

don't really see how these would be realistically implemented. Power flashes are cause by Tornadoes going over powerlines and other electrical equipment. Minecraft doesn't have powerlines. maybe a red flash if it passed over active redstone? but other than that, this doesn't really make any sense to add.

I agree with AlexTheSuprWolf on this one, this doesn't really fit with the mod in general.


Artificer's flamethrower bug

When you use the flamethrower while having this mod on, all skills get disabled. Everything works fine if you use all skills but that one. but the moment you use it, you lost all skills.

Yeah, generalfixes needs to be deprecated. Many of the features I was planning to add in the future were added by the devs to the base game.

Ah, i see. It's good to know tho 👍🏼

Will likely fix after I finish my current massive project.


Various updates to the blackmagicprobe runner

Hi, quick one for the BMP runner to catch up with https://github.com/blackmagic-debug/blackmagic/pull/1056/commits/240099dde9fae2be6f50a0913f065736500d0727, they changed the command, updating the runner to send both the old and new ones, and take both version of the flags.
EDIT: added two patches to also improve the serial port guessing.
--
Recently the blackmagicprobe command for "connect under reset" has been updated from "connect_srst" to "connect_rst". Update the flag name in the west blackmagicprobe binding, keep the old one as well for compatibility with out of tree boards, and send both the old and new commands to GDB so that it works with both the old and new firmware. GDB is going to get a "command not supported by this target" for one of the two and proceed with the other one.

Hey, since this had not been reviewed yet I added a couple patches to address https://github.com/zephyrproject-rtos/zephyr/issues/53859.
So patch 1 fixes the "connect under reset" bit
Patch 2 makes the runner port selection OS aware
Patch 3 drops the current explicit flags from board files
Note that I only tested this on Linux.
@nordicjm what do you think?

@fabiobaltieri here is the windows output for the same segger serial port, in case it's useful to you:
>>> [vars(info) for info in serial.tools.list_ports.comports(include_links=True)]
[{'device': 'COM3', 'name': 'COM3', 'description': 'Intel(R) Active Management Technology - SOL (COM3)', 'hwid': 'PCI\\VEN_8086&DEV_9D3D&SUBSYS_081C1028&REV_21\\3&11583659&0&B3', 'vid': None, 'pid': None, 'serial_number': None, 'location': None, 'manufacturer': 'Intel', 'product': None, 'interface': None}, {'device': 'COM4', 'name': 'COM4', 'description': 'JLink CDC UART Port (COM4)', 'hwid': 'USB VID:PID=1366:0105 SER=000683441799 LOCATION=1-9:x.0', 'vid': 4966, 'pid': 261, 'serial_number': '000683441799', 'location': '1-9:x.0', 'manufacturer': 'SEGGER', 'product': None, 'interface': None}]

@fabiobaltieri here is the windows output for the same segger serial port, in case it's useful to you:
>>> [vars(info) for info in serial.tools.list_ports.comports(include_links=True)]
[{'device': 'COM3', 'name': 'COM3', 'description': 'Intel(R) Active Management Technology - SOL (COM3)', 'hwid': 'PCI\\VEN_8086&DEV_9D3D&SUBSYS_081C1028&REV_21\\3&11583659&0&B3', 'vid': None, 'pid': None, 'serial_number': None, 'location': None, 'manufacturer': 'Intel', 'product': None, 'interface': None},

{'device': 'COM4', 'name': 'COM4', 'description': 'JLink CDC UART Port (COM4)', 'hwid': 'USB VID:PID=1366:0105 SER=000683441799 LOCATION=1-9:x.0', 'vid': 4966, 'pid': 261, 'serial_number': '000683441799', 'location': '1-9:x.0', 'manufacturer': 'SEGGER', 'product': None, 'interface': None}]

edit: separated entries in the results list above

Thanks, cool looks like vid and pid are indeed there for the USB device so this should work with the current version of this PR.

It's "hilarious" that you had to go with 3 different ways to match this device. Sad that the interface descriptor isn't universally available.

It's "hilarious" that you had to go with 3 different ways to match this device. Sad that the interface descriptor isn't universally available.

That would have been ideal. This reminded me of sigrok splitting libserialport out to its own library (https://github.com/sigrokproject/libserialport), similar story I guess.

I know this has been closed. However I'm getting
COM15: No such file or directory.
"monitor" command not supported by this target.
Don't know how to attach.  Try "help target".
You can't do that when your target is `None'
The program is not being run.

May I propose that blackmagicprobe_gdb_serial_win32() return a COMXX port prefixed with \\.\ so that GDB is instructed to open \\.\COM15 for example. As this works on Windows 10.
I'm using Zephyr SDK v0.15.2, and Zephyr v3.3.0-rc1

May I propose that blackmagicprobe_gdb_serial_win32() return a COMXX port prefixed with \\.\ so that GDB is instructed to open \\.\COM15 for example. As this works on Windows 10.

Sure thing! Can you propose it in the form of a pull request? :-)

I know this has been closed. However I'm getting
COM15: No such file or directory.
"monitor" command not supported by this target.
Don't know how to attach.  Try "help target".
You can't do that when your target is `None'
The program is not being run.

May I propose that blackmagicprobe_gdb_serial_win32() return a COMXX port prefixed with \\.\ so that GDB is instructed to open \\.\COM15 for example. As this works on Windows 10.
I'm using Zephyr SDK v0.15.2, and Zephyr v3.3.0-rc1

@TequinDragon opened https://github.com/zephyrproject-rtos/zephyr/pull/54678 for that, would you be able to verify it? (can't add you as reviewer directly)


disable send option while loading

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


Update simple_memo.rst

Description
Please include a description of the change, and why it was needed.
Linked issues
Please link any issues this pull request related to. Using the word "closes" before the
link will mean the issue is automatically closed by this Pull Request.
Testing
Please describe how your changes have been tested.
Checklist

[ ] I have provided a good description of the change above
[ ] I have added any necessary tests
[ ] I have added all necessary type hints
[ ] I have checked my linting (docker-compose run --rm lint)
[ ] I have added/updated all necessary documentation
[ ] I have updated CHANGELOG.md, following the format from
Keep a Changelog.

Giving up.

@bwagner No worries, git/github can take a while to get used to. It is a really cool think to learn though, so I'd recommend looking at the documentation and some tutorials. GitHub has some really good ones: https://docs.github.com/en/github/getting-started-with-github
I have just created #121 to fix the issue. Many thanks for pointing out the problem, and I hope this didn't cause you too much frustration!

Thank you for your suggestions, @jstockwin! I'm not surprised I come across as a newbie, even though I've been on github since 2010.
I hadn't understood (and still don't) whether you had actually made changes to my PR or whether you had simply suggested changing the links along with the prose amendments in my PR.
Had you done changes to my PR?
How would I have accepted them into my PR?
Thank you

@bwagner Apologies for assuming you were a newbie!
Let me answer your questions in a slightly different order to what you asked them in:

Also, after I gave up on my original PR, I made a new one with your suggested improvements. Why wasn't that PR used?

I saw your "Giving up" comment (here https://github.com/jstockwin/py-pdf-parser/pull/120#issuecomment-696336813) and figured something had gone wrong. I didn't even look at the changes, but I now agree the changes were correct.
Given your comment and closing the issue, I assumed your second attempt was just starting from scratch entirely and had still gone wrong somehow (which is why I assumed you were a newbie). I guess you didn't actually mean to comment and close that new PR (i.e. this PR)?

Had you done changes to my PR?

 (See point 6 of https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/commenting-on-a-pull-request#adding-line-comments-to-a-pull-request), which is what I did.
In fairness to you, the suggested changes thing is a fairly new feature.

How would I have accepted them into my PR?

There should have been a "Commit Suggestion" option which would have walked you through incorporating my change into your PR (as an additional commit), see https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/incorporating-feedback-in-your-pull-request#applying-suggested-changes.
I think what you did was "Mark as resolved", which kind of closes the conversation but does not actually do any changes. See https://docs.github.com/en/github/collaborating-with-issues-and-pull-requests/about-pull-request-reviews#resolving-conversations. It might be that once it's marked as resolved it is no longer possible to apply the suggestion, I am not sure...
Thanks again for submitting the issue, and sorry for the confusion and not ending up getting your PR merged.

Awesome, @jstockwin, thanks a lot for taking the time to explain. I appreciate it.


Forbid merge commits

We don't allow those in Hibernate Search, in particular.
See https://github.com/hibernate/hibernate-search/blob/main/CONTRIBUTING.md#commit

Not necessary; there is already a branch protection option for that in GitHub. See "Require linear history".


setec: adding Yanjun

He's a trusted Envoy developer, has a fix to an embargo-reported bug, and is happy to help with setec work going forward

LGTM


Form fields auto stacking on mobile width, chubby theme dropdown overflow

AFAIK, unstackable feature is not implemented in 2.1.8. I'm using less version of sem-ui in meteor. Is there any css solution to keep the form layout as is on mobile. I.e. not automatically stacking on mobile screen. Also there is a weird bug with dropdown menu width in chubby theme.

Any workaround for this. Thanks guys.

I think it happens not only in mobile env, but also in desktop browser, exactly looks the same. But I didn't find this in official website demo which is in version 2.2.x, so I guess it has been fixed in the latest version.


ECONNREFUSED on NodeJS 18

When using node-fetch on NodeJS 18, got ECONNREFUSED
// test.mjs
import { Server } from '@hapi/hapi'
import fetch from 'node-fetch'

const start = async () => {
  const server = new Server({
    port: 3344,
  })

  server.route([
    {
      method: 'GET',
      path: '/',
      handler: async () => {
        return JSON.stringify({ value: 'hello' })
      }
    }
  ])
  await server.start()

  const response = await fetch('http://localhost:3344')
  console.log('response', await response.text())
}

start()

Steps to reproduce the behavior:

Run the above in Node 18.7.0. It fails with:

FetchError: request to http://localhost:3344/ failed, reason: connect ECONNREFUSED ::1:3344
    at ClientRequest.<anonymous> (file:///D:/code/mocktomata/node_modules/node-fetch/src/index.js:108:11)
    at ClientRequest.emit (node:events:513:28)
    at Socket.socketErrorListener (node:_http_client:481:9)
    at Socket.emit (node:events:513:28)
    at emitErrorNT (node:internal/streams/destroy:151:8)
    at emitErrorCloseNT (node:internal/streams/destroy:116:3)
    at process.processTicksAndRejections (node:internal/process/task_queues:82:21) {
  type: 'system',
  errno: 'ECONNREFUSED',
  code: 'ECONNREFUSED',
  erroredSysCall: 'connect'
}

Expected behavior
The same script will work in NodeJS 16.
Video repro: https://youtu.be/PgyYPw4RCoI
Your Environment

software
version

node-fetch
2.6.7, 3.2.10

node
18.7.0

@hapi/hapi
20.2.2

Operating System
Windows

Calling the server using curl is working, showing it is not a problem of @hapi/hapi:
curl http://localhost:3344 -v
*   Trying <IP_ADDRESS>:3344...
* Connected to localhost (<IP_ADDRESS>) port 3344 (#0)
> GET / HTTP/1.1
> Host: localhost:3344
> User-Agent: curl/7.83.1
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< content-type: text/html; charset=utf-8
< cache-control: no-cache
< content-length: 17
< accept-ranges: bytes
< Date: Fri, 12 Aug 2022 07:14:03 GMT
< Connection: keep-alive
< Keep-Alive: timeout=5
<
{"value":"hello"}* Connection #0 to host localhost left intact

I started seeing this and the only solution I found was to use the IP <IP_ADDRESS> instead of localhost. I'm not sure why

I started seeing this and the only solution I found was to use the IP <IP_ADDRESS> instead of localhost. I'm not sure why

This would be since localhost resolves to ::1 which is the "equivalent" of <IP_ADDRESS> but in IPv6 instead of IPv4. Since the server seems to be listening only on IPv4 this means that the connection will be refused when trying ::1.
I believe that this error exists when using the builtin http module as well, and probably has to do with how Node.js does dns resolving...
If you want to move forward with this I would suggest first seeing if you can reproduce the same issue using the builtin http module, and if you can, file an issue with Node.js directly. If this cannot be reproduced in the builtin http module, than it is most likely something in this module that messes this up.

Can confirm this issue on Node version v18.8.0. I am getting the very same error on different port 8080. Calling with curl works, calling from node results in ECONNREFUSED as described above.
I can also confirm that the very same code worked on node 16.X.X. I can also confirm that changing localhost to <IP_ADDRESS> fixes the issue.

Indeed it is issue with nodejs. https://github.com/nodejs/node/issues/40702

One thing to check for people seeing this on Macs, specifically with port 5000, is Airplay Receiver. I learned that it actually uses localhost:5000 and it was the reason, at least in my case, that the IP worked, but localhost didn't. I think an update from our IT must've enabled it without me realizing it

I experience the same issue,
I use nvm for Windows and same code works properly for 16.17.0

I started seeing this and the only solution I found was to use the IP <IP_ADDRESS> instead of localhost. I'm not sure why

That worked for me!

I started seeing this and the only solution I found was to use the IP <IP_ADDRESS> instead of localhost. I'm not sure why

works on node 16 but above solution doesn't work even if the IP were changed to localhost

there was a breaking change in node v17 that changed the default IP resolving. ip6 is preferred by default. You can add dns.setDefaultResultOrder('ipv4first') at the beginning of you app and that should fix the problem (https://nodejs.org/api/dns.html#dnssetdefaultresultorderorder)

there was a breaking change in node v17 that changed the default IP resolving. ip6 is preferred by default. You can add dns.setDefaultResultOrder('ipv4first') at the beginning of you app and that should fix the problem (https://nodejs.org/api/dns.html#dnssetdefaultresultorderorder)

This fixed the issue, didn't realize the cause was from Node and going by the error message I got, thought it was something else.
Thanks anyway.

At this point it seems like the underlying problem is identified, and that there is nothing that should be changed in node-fetch. Happy to re-open if someone disagrees 👍

Changing 'localhost' to '<IP_ADDRESS>' has fixed the problem for me.

I started seeing this and the only solution I found was to use the IP <IP_ADDRESS> instead of localhost. I'm not sure why

I changed localhost to <IP_ADDRESS>, I don't know what is the reason but work successfully

Node.js 18+ has built-in fetch. You just need to remove this package.

Is there anyone having this issue in Ubuntu 20.04.6 with node version 18.19.0?

I have the issue with node v20.11.1. Neither setting ip to <IP_ADDRESS> or dns.setDefaultResultOrder('ipv4first'); fixes the problem.

I confirmed this fixed the issue
require('node:dns').setDefaultResultOrder('ipv4first');

On a mac, commenting out the ipv6 entry forces the ipv4 resolution.
comment out this line as alternative to using <IP_ADDRESS>
e.g. ##::1             localhost

I confirmed this fixed the issue
require('node:dns').setDefaultResultOrder('ipv4first');

this works


Add more User-Agents

We need to add more user-agents on the project. My network IP address have been blocked many times at few minutes, then i need reboot my modem/router always to generate a new IP. Maybe trying it, we'll fix it.
1 - create a random function that provide a new user-agent. Take a look at Python example:
import random
# set some user-agents inside of the list
user_agent_list = ["user-agent 01",  "user-agent 02",\
                   "user-agent 03", "user-agent 04", "user-agent 05"]

# get the random agent for not to seems obvious.
# python provide a random function seems with 'srand()' function on awk language
#                                         |
#                                         v
Option["user-agent"] = random.choice(user_agent_list)

Code snippet that could be affected:
# include/Main.awk L46~L47
# Networking
Option["proxy"] = ENVIRON["HTTP_PROXY"] ? ENVIRON["HTTP_PROXY"] : ENVIRON["http_proxy"]
Option["user-agent"] = ENVIRON["USER_AGENT"] ? ENVIRON["USER_AGENT"] : "Mozilla/5.0 (X11; Linux x86_64; rv:40.0) Gecko/20100101 Firefox/40.0"

latest output of trans --debug hi:
33 bytes > HTTP/1.1 503 Service Unavailable
  36 bytes > Date: Sat, 03 Oct 2015 06:36:25 GMT
  17 bytes > Pragma: no-cache
  39 bytes > Expires: Fri, 01 Jan 1990 00:00:00 GMT
  78 bytes > Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0
  24 bytes > Content-Type: text/html
  30 bytes > Server: HTTP server (unknown)
  21 bytes > Content-Length: 1773
  32 bytes > X-XSS-Protection: 1; mode=block
  28 bytes > X-Frame-Options: SAMEORIGIN
  18 bytes > Connection: close
   1 bytes >
  63 bytes > <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
   6 bytes >
 165 bytes > <meta http-equiv="content-type" content="text/html; charset=utf-8"><meta name="viewport" content="initial-scale=1"><title>http://www.google.com/</title>
 177 bytes >
  30 bytes >

  69 bytes >

   1 bytes >
   1 bytes >
  31 bytes >

 225 bytes >    Our systems have detected unusual traffic from your computer network.  Please try your request again later.  Why did this happen?

 119 bytes >

 734 bytes >     This page appears when Google automatically detects requests coming from your computer network which appear to be in violation of the Terms of Service. The block will expire shortly after those requests stop.

This traffic may have been sent by malicious software, a browser plug-in, or a script that sends automated requests.  If you share your network connection, ask your administrator for help — a different computer using the same IP address may be responsible.  Learn more

Sometimes you may see this page if you are using advanced terms that robots are known to use, or sending requests very quickly.
  13 bytes >

   1 bytes >
  92 bytes >  IP address: <IP_ADDRESS>
Time: 2015-10-03T06:36:25Z
URL: http://www.google.com/

   7 bytes >

   6 bytes >

   7 bytes >
   7 bytes >
[ERROR] Null response.
content = '''

'''
tokens =
ast =
[ERROR] Oops! Something went wrong and I can't translate it for you :(

latest output of trans -V:
Translate Shell       <IP_ADDRESS>-git:0c02934

platform              Linux
gawk (GNU Awk)        4.1.1
fribidi (GNU FriBidi) [NOT INSTALLED]
audio player          mpg123
terminal pager        less
terminal type         xterm-256color
user locale           pt_BR:pt:en (Portuguese)
home language         pt
source language       auto
target language       pt
proxy                 [NONE]
user-agent            Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/39.0.2171.95 Safari/537.36
theme                 default
init file             [NONE]

Report bugs to:       https://github.com/soimort/translate-shell/issues

Please, i don't understand AWK language, then i cannot help with a PR.

My network IP address have been blocked many times at few minutes.

If your IP address has been blocked by Google, then faking random User-Agent will NOT help. Have you abused the service, or use it in an abnormal way?
If you (or someone from the same network of yours) hit Google so hard that they regard you as a robot misusing their service, please kindly notice that they have a TOS preventing us from doing so.

It make sense. I really have used a lot the service because we keep communication with many American persons.
thank you for the information and sorry for any inconvenience! :blush:


docs: fix typo

depending on what index is, you can do both, e.g. nothing wrong with da[1:3] = DocumentArray.empty(2)

depending on what index is, you can do both, e.g. nothing wrong with da[1:3] = DocumentArray.empty(2)

Sure, but I think the basic assumption is that index is an int. Indexing by ranges would be a slightly "advanced" topic


How to use with const values

I want to use this package to create constant values. For example I want to create a const with 100 into. I have tried both :
pub const HUNDRED: BigInt = match BigInt::parse_bytes("100".as_bytes(), 10) {
    Some(x) => x,
    None => BigInt::default()
};

and
pub const HUNDRED: BigInt = BigInt::parse_bytes("100".as_bytes(), 10).unwrap();

and
pub const HUNDRED: BigInt = BigInt::from(1000000000i32);

For the easy use of my package I don't want to keep them as Option and want to unwrap them as I'm sure what is the content because it's const. How can I create it from a decimal form to have a constant ?

This is discussed in #181, but we can't do this in general because Vec is only constant for its empty Vec::new().
I do suggest that From is a better choice when a known value is small enough to fit in a primitive integer, but we still can't create that as a const. One possibility is to use once_cell to create a static Lazy value instead.

Ok I close in favor of #181 then.


Split Generated Source Into 26 files for Windows

[NOTE: this PR includes the changes from #615, so it would be easier to review once that one is merged]
The Problem
Windows is failing to build on the CI machine.
It appears to be due to the humungous size of the combined generated file, which I suspect causes some sort of exponential explosion in the compiler; the upshot of which is that it just hangs.
This PR attempts to fix that.
The Solution
Instead of generating a single enormous file, we generate 26 files SwiftGodotA.swift, SwiftGodotB.swift etc.
Each file contains all the types that start with that letter (or more specifically, all the types output by Printer instances whose names start with that letter).
This keeps the number of input files down to a manageable amount, but also the size of each file from getting too large.
If necessary we could use a different hash to produce more, smaller, files, but this seems to be enough to get the build to actually finish.

I still takes ~20 mins to build on Windows, but it does at least build now, and the minimal tests that don't need libgodot do actually link & run.

Could we undo the whitespace changes on the files that are unrelated to this?   Like the EntryPoint changes have a lot of churn.
I am trying to merge a large set of internal changes for object ownership, and the whitespace changes every time become very time consuming.

Question, the generated split only applies to Windows, but on Unix we still keep all of our files the way we have them now?

Question, the generated split only applies to Windows, but on Unix we still keep all of our files the way we have them now?

That's the intention, yes.

Does Linux have a similar problem with path lengths & the linker?
I was actually thinking that I might try to add a Linux runner to the workflow (later, not as part of the PR) - but I wasn't sure what state the linux build was in.

I am not aware of issues with Linux, I think that would be fine.

The original comment states that I should merge firs the #615 change, but it seems like that one has the build broken, which this one fixes.
Should I just merge this one instead?

Btw, it would be really nice if we did not include whitespace changes on these PRs.


Added p8 to Tools section of README

p8 is a tool I wrote a couple years ago that I used for my own personal development flow. I really enjoyed coding dependency management and build tools in the JavaScript ecosystem. This was my attempt to bring a little bit of that magic to the PICO-8 world in hopes that it helps folks share code more easily, see their code run faster, and even use MoonScript if they really wanted do something crazy.

That's great! Thanks!

Thank you!


There is no example in repo

Missing example video/audio from example command line
python merge_testing.py "src/penguins.mp4" "src/wolves.mp3" -o "out/merged.mp4" --threshold 2

This example only illustrates how to populate the command placeholders with something meaningful. Uploading example input files would unnecessarily increase the repo size. You can find a set of public test videos here https://gist.github.com/jsturgis/3b19447b304616f18657 and sample audio files here https://github.com/rafaelreis-hotmart/Audio-Sample-files


[timeline-chart][tsp-typescript-client] Switch to solid versions

Now that we can easily publish solid (latest) revisions of these dependencies, let's switch to requesting a "solid" version range, for them, instead of using "next". From now-on, these dependencies should be managed like any other production dependencies.
Note: waiting for a release of timeline-chart that contains the size improvements and will then update this PR to pick-up that version.
Done - PR updated

Thanks for the review! Merging.


Ability to force changes irrespectively of the state in registry

The same way there is a --log-only option for deploy and revert commands, a --no-log or --force option could exist to deploy and revert changes by taking advantage of the sqitch plans but without managing the changes in a registry.
Context behind the proposal
I created a first sqitch project for a PostgreSQL server in order to create a set of 2 roles and 1 database which aims to set the base to later deploy another, more consequent, sqitch project.
The scripts make use of psql variables to avoid hard-coding the roles and database names and also to be able to create multiple sets of roles and databases. However, once the deployment is done for one set, I cannot deploy with different variables to create a different set because the sqitch registry consider that the deployment is already done:
$ sqitch deploy --set account_ownerrole=blue --set account_approle=blue_app --set account_db=blue mytarget
Deploying changes to mytarget
  + role/account ...... ok
  + role/account_app .. ok
  + database/account .. ok
$ sqitch deploy --set account_ownerrole=acme --set account_approle=acme_app --set account_db=acme mytarget
Nothing to deploy (up-to-date)

I suppose that I could use a dedicated registry for each set. However, in that case, I do not feel that I need to manage the changes in a registry and therefore I would like to be able to force the changes.

Forcing changes in this way would pretty much defeat one of the main purposes of Sqitch: ensuring that the database is in a consistent, expected state. I think you might be better off managing such changes outside of Sqitch.


Contain CVE vulnerabilities of deleted images

We have customer feedback that to contain not only deleted image list but also the vulnerabilities of the deleted images in logs.

Hello Team,
It would be great to see this feature enabled.

Currently eraser uses Trivy to scan vulnerabilities on running / stale images on aks cluster. However, we want to leverage this feature to surface vulnerability details in the logs and use log analytics / container insights to develop custom query alerts to detect image vulnerabilities on AKS clusters.

This feature will be useful for us who are using Private AKS clusters in air gapped environment.

Thank you

@asifkd012020 do you mind elaborating more on why are you looking for vuln details for deleted images?
or are you asking for #356 (vuln details on running images)?

Hello @sozercan,
yes, we are looking for vuln details on running images for security and compliance requirements.
We use Jfrog Artifactory to build images and its hard to get vul details(as in ACR).
Thank you.

@asifkd012020 thanks! that is out of scope for eraser at this time, but we will consider it. we would recommend using https://aquasecurity.github.io/trivy-operator for this feature

@sozercan @asifkd012020 could you please elaborate how the integration with Eraser will look like ?

@chen-keinan if this is re: #356, please comment in that issue


Modules not loading.

I have added a custom module under the modules_v4 folder, but it is not showing up anywhere (even under preferences). I am using the latest version (2.1.1) of Webtrees in docker and I am using the descendants chart, which was updated to support the latest release. Here's my docker-compose file:
version: '3.0'

services:
  webtrees:
    image: dtjs48jkt/webtrees
    restart: unless-stopped
    container_name: webtrees
    ports:
      - 8079:8079
    environment:
      - DISABLE_SSL=TRUE
      - PORT=8079
    volumes:
      - /mnt/docker/containers/webtrees/data:/var/www/html/data
      - /mnt/docker/containers/webtrees/modules_v4:/var/www/html/modules_v4
      - /mnt/docker/containers/webtrees/media:/var/www/html/media
    networks:
      - my-network

  mysql:
    image: mysql:5.5
    restart: unless-stopped
    container_name: webtrees_mysql
    environment:
      - MYSQL_ROOT_PASSWORD=PASSWORD
      - MYSQL_DATABASE=webtrees
      - MYSQL_USER=webtrees
      - MYSQL_PASSWORD=PASSWORD
    networks:
      - my-network
    volumes:
      - /mnt/docker/containers/webtrees_mysql:/var/lib/mysql
networks:
  my-network:

Everything is working except for the custom modules. It does not even show up in modules:

Sorry....looks like it was a permission issue. changed the permissions to 777 and now it is working :)


No data found for this date range, symbol may be delisted

I got the following error. I am not sure what is wrong. Does anybody know what is the problem?
>>> import yfinance as yf
>>> ticker = 'MSFT'
>>> data = yf.Ticker(ticker).history(interval='1m', start='2021-11-06', end='2021-11-07')
- MSFT: No data found for this date range, symbol may be delisted

That date is Saturday ...


bigint with max and min produces incorrect schema

Bug description
We have a field that has a type of bigint with min/max. But the output doesn't produce a correct schema.
Input
  /**
   * @minimum -999999999 scale must be greater than or equal to -999999999
   * @maximum 999999999 scale must be less than or equal to 999999999
   */
type Foo = {
  scale: bigint
}

Expected output
const fooSchema = z.object({
  scale:  z.bigint()
    .min(BigInt(-999999999), 'scale must be greater than or equal to -999999999')
    .max(BigInt(999999999), 'scale must be less than or equal to 999999999'),
})

Actual output
const fooSchema = z.object({
  scale: z.bigint()
    .min(-999999999, 'scale must be greater than or equal to -999999999')
    .max(999999999, 'scale must be less than or equal to 999999999'),
})

This is also a Typescript error

Versions

Typescript: ^5.5.4
Zod: ^3.23.8
ts-to-zod: '^3.13.0'

I did try this but it did not work
  /**
   * @minimum BigInt(-999999999) scale must be greater than or equal to -999999999
   * @maximum BigInt999999999) scale must be less than or equal to 999999999
   */

Hello,
Indeed, the bigInt type it not detected.
Not an easy fix given the way JSDoc tags are handled (they're unaware of the underlying property)
A workaround can be implemented using the @schema annotation:
type Foo = {
  /**
   * @schema .min(BigInt(-999999999), "scale must be greater than or equal to -999999999").max(BigInt(999999999), "scale must be less than or equal to 999999999")
   */
  scale: bigint;
};

This will append whatever is behind the @schema at generation time:
// Generated by ts-to-zod
import { z } from "zod";

const fooSchema = z.object({
  scale: z
    .bigint()
    .min(
      BigInt(-999999999),
      "scale must be greater than or equal to -999999999"
    )
    .max(BigInt(999999999), "scale must be less than or equal to 999999999"),
});

This worked great. Thank you. Looks like @schema is where we'll be spending a lot of our time.


Store dates in Firestore using Timestamp type instead of String

Observation :
Actually RevenueCat store dates as ISO 8601 "string" inside Firestore. This is an issue.
Feature request:
To let us availability to create some Firestores rule based on dates, dates should be of type "Timestamp". Timestamp object can be easily converted to Date() on any Frontend/Backend. So this is not an issue.
Example of current limitation:
Bellow an exemple explaining why we need that :
service cloud.firestore {
  match /databases/{database}/documents {

    function hasPaidSubs() {
      let query = get(/databases/$(database)/documents/users/$(request.auth.uid)/purchaser-info/$(request.auth.uid));
      let subscription = query.data;

      return subscription.entitlements.pro != null && subscription.entitlements.pro.expires_date > request.time;
    }

     match /cars/{carID} {
      allow read: if true;
      allow create: if request.auth != null && request.auth.uid == request.resource.data.creator && hasPaidSubs()
      allow update: if request.auth != null && request.auth.uid == resource.data.creator
      allow delete: if request.auth != null && request.auth.uid == resource.data.creator;
    }
  }
}

In this example, "expires_date" is of type "string" and equal "2024-08-26T07:43:35Z", the comparaison generated the following exception because it can't be compare with request.time which is of type Timestamp :
Unsupported operation error. Received: string > timestamp. Expected: string > string, bytes > bytes, int > int, float > float, duration > duration, timestamp > timestamp, constraint > any, any > constraint. for 'create' @ L32, cause=null}
If you agree with this suggestion, it will generate a Breaking Change.
Alternative:
Inside Firebase Firestore, the Monitoring Rule test space allow the comparaison between String and Timestamp. But this is a bug and that didn't work on PROD and on Firebase cli Emulator.
I started a conversation with the Firebase support to check this issue and to know if it's possible to allow String to Timestamp conversion or comparaison inside Firestore Rules. If they allow that, i will close this issue.

Hello @vicfergar, can you check this ? This is really blocker :/


Not work on windows

Only works on Mac， but not work on windows
It works if I change
const filter = /react-virtualized\/dist\/es\/WindowScroller\/utils\/onScroll.js$/;
to
const filter = /react-virtualized\\dist\\es\\WindowScroller\\utils\\onScroll.js$/;

Thanks for raising this. It's a fairly simple fix so I'll sort it ASAP.

Hi @trend-alan-bie
I've just released a fix. Can you test with version 1.0.3 and see if your problem is resolved?

@abemedia , I got this error:
[esbuild-plugin-react-virtualized] "onLoad" filter is not a valid Go regular expression: "react-virtualized\\dist\\es\\WindowScroller\\utils\\onScroll.js$"

It works now, thank you so much


Uses a file payload as response

File Response Strategy. Uses a file payload as response. Recommended for large responses.
You can define the transformer that is used for the result of the route using the @responseFile tag
@responseFile /apidoc_responses/search.json
The file must be in storage/apidoc_responses/search.json

Codecov Report

Merging #410 into master will decrease coverage by 0.68%.
The diff coverage is 72.72%.

@@             Coverage Diff              @@
##             master     #410      +/-   ##
============================================
- Coverage     91.36%   90.67%   -0.69%
+ Complexity      205      170      -35
============================================
  Files            10       11       +1
  Lines           521      515       -6
============================================
- Hits            476      467       -9
- Misses           45       48       +3

Impacted Files
Coverage Δ
Complexity Δ

src/Tools/ResponseResolver.php
100% <ø> (ø)
7 <0> (ø)
:arrow_down:

.../Tools/ResponseStrategies/ResponseFileStrategy.php
72.72% <72.72%> (ø)
6 <6> (?)

src/Postman/CollectionWriter.php
87.5% <0%> (-0.97%)
6% <0%> (-6%)

src/Tools/Generator.php
93.28% <0%> (-0.51%)
38% <0%> (-25%)

...ols/ResponseStrategies/TransformerTagsStrategy.php
85.71% <0%> (-0.34%)
21% <0%> (-2%)

src/Commands/GenerateDocumentation.php
94.54% <0%> (-0.15%)
27% <0%> (-8%)

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 d52d217...545ae27. Read the comment docs.

Should I update the README.md?

Should I update the README.md?

You can leave the README. I'll handle it.

Some tests are failing when I try to use the Storage::fake, I believe it's because it's running on an earlier version of Laravel. Any idea how I should proceed?

Yes, the failed tests are due to a certain version of flysystem.
Okay, can you revert back to storage_path? The test too. Seems the safest course of action. And it's not like we'll be breaking anything 🙂
Sorry for the back and forth. This is good work. Make these changes and we're good to go 👍

No problem man, thank you for the tips. 😃


Make parse_cred safe

Improve CBOR decoding and make sure all accesses are within bounds.

Solved in #169


Removed duplicated address storage.

Unnecessary duplicated storage of the address in NetworkClient.
Arguably, addresses could be removed entirely from Mirror and left to the transports. They aren't actually used except for 1. logging, and 2. exposing to users. Look at NetworkConnection.address.

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


Broken links in downloaded SVG images

When using SVG Crowbar to download models, only links to PNG images are shown.  These need to embedded directly into the SVG models.

Fix largely works, but HTML table labels in risk models seem to be a bit problematic.


IndexSizeError on IE11 when rendering SVG icons as map points

Describe the bug
Trying to render icons as points on the map based on an SVG URI (base64 encoded). Works fine on Chrome, Safari, Firefox but not IE11. An IndexSizeError is thrown:

after digging into this a bit more it looks like x and/or y params are negative values in the points generated, and IE11 seems to be the only browser that has an issue with this
I've looked through all the other issues related to this and have tried suggested fixes, however these have not worked. Hopefully I am missing something but happy to contribute towards a bug fix if required
To Reproduce
Minimal Repo to reproduce
https://github.com/ben-cloud9/openlayers-ie-11

clone this repo
install dependancies npm install
run npm start and go to localhost:1234 in IE11
an IndexSizeError will be thrown on map render
no issues opening in any other browser

Expected behavior
Icons to render on map points as in other browsers
Thanks in advanced and let me know if you need any further info :)

Hi,
IE does not work very well with svg + canvas, I avoid using them.
I use canvg to "convert" the svg to png and return it in a style function
Like this:
var styleIcon;
function StyleFunction(f, res) {
          if (styleIcon)
           return styleIcon;
          let src = "<svg width='51' height='29' version='1.1' xmlns='http://www.w3.org/2000/svg'><rect x='0' y='5' width='24' height='24' stroke='#000' fill='#e8f442' stroke-width='1' /><line x1='7' y1='8' x2='7' y2='26' stroke='#000' stroke-width='3' /><line x1='17' y1='8' x2='17' y2='26' stroke='#000' stroke-width='3' /><line x1='7' y1='17' x2='18' y2='17' stroke='#000' stroke-width='3' /></svg>";
          let temp_svg_to_canvas = document.getElementById("temp_svg_to_canvas");
          temp_svg_to_canvas.width = 20;
          temp_svg_to_canvas.height = 20;
          canvg(temp_svg_to_canvas, src, {
            scaleWidth: 20,
            ignoreDimensions: true,
            ignoreAnimation: true,
          });
          let img = temp_svg_to_canvas.toDataURL("image/png");
          temp_svg_to_canvas.svg.stop();

          styleIcon = new ol.style.Style({
            image: new ol.style.Icon({
              src: img,
              rotateWithView: true,
            })
          });
         return styleIcon;
}

Thanks I'll give that a try :)

Hi, I have tried using a standard .png file (made more sense with our use case) and am still getting the same IndexSizeError on IE11
I've updated the repo to replicate the issue with png this time- https://github.com/ben-cloud9/openlayers-ie-11

@ben-cloud9 you need to set the imgSize: [width, height] property for ol.style.Icon. This fixed SVG icons for me in IE11.

This is set - see the demo repo i set up above - anyone have any other ideas?
const iconImage = new Icon({
    src: icon,
    imgSize: [32, 32],
    scale: 1,
    anchor: [12, 24],
    anchorXUnits: 'pixels',
    anchorYUnits: 'pixels',
    rotation: 0,
    opacity: 1
});

Update - after switching to a .png file  instead of svg. I finally got it working in IE11 by removing the imgSize property from the icon. Having imgSize set, even with png, still caused the IndexSizeError
I'm now having to control the icon size using scale (0.5 scale unselected, 1 scale selected in our use case)

@ben-cloud9 scale is the only way to control the icon size. imgSize is supposed to be the actual dimensions of the image.

ok thanks
So in summary for anyone having a similar issue - Using an SVG URI as the src for the icon will not work in IE11. My solution for this was to use a .png icon and NOT set imgSize. Setting imgSize causes
an IndexSizeError on IE11

@ben-cloud9 An SVG URI does work on IE11 if you have the correct imgSize set.

svg marker not show even if I set the imgSize on ie11.

@vipcxj try adding width and height attributes in the SVG file that match your imgSize.

@icholy
new ol.style.Icon({
      color: "#cc3388",
      imgSize: [68.673, 68.674],
      src: "data:image/svg+xml,%3c%3fxml version='1.0' encoding='iso-8859-1'%3f%3e %3csvg version='1.1' id='Capa_1' fill='white' xmlns='http://www.w3.org/2000/svg' xmlns:xlink='http://www.w3.org/1999/xlink' x='0px' y='0px' width='68.673px' height='68.674px' viewBox='0 0 68.673 68.674' style='enable-background:new 0 0 68.673 68.674%3b' xml:space='preserve'%3e %3cg%3e %3cpath d='M67.682%2c30.151L53.394%2c19.149V7.684c0-1.404-1.141-2.544-2.544-2.544c-1.402%2c0-2.543%2c1.14-2.543%2c2.544v7.548L35.889%2c5.668 c-0.915-0.703-2.188-0.703-3.104%2c0L0.992%2c30.151c-1.113%2c0.856-1.32%2c2.453-0.463%2c3.566c0.855%2c1.112%2c2.453%2c1.319%2c3.566%2c0.463 l2.793-2.15v28.961c0%2c1.403%2c1.14%2c2.543%2c2.544%2c2.543h49.493c1.403%2c0%2c2.543-1.14%2c2.543-2.543V31.784l3.11%2c2.396 c0.463%2c0.356%2c1.008%2c0.529%2c1.551%2c0.529c0.763%2c0%2c1.517-0.341%2c2.018-0.992C69.001%2c32.604%2c68.794%2c31.007%2c67.682%2c30.151z M29.103%2c30.664 c1.717%2c0%2c3.107%2c1.393%2c3.107%2c3.108c0%2c1.718-1.391%2c3.109-3.107%2c3.109s-3.109-1.394-3.109-3.109 C25.994%2c32.057%2c27.386%2c30.664%2c29.103%2c30.664z M21.189%2c36.565c1.717%2c0%2c3.108%2c1.393%2c3.108%2c3.107s-1.392%2c3.107-3.108%2c3.107 c-1.718%2c0-3.108-1.394-3.108-3.107C18.081%2c37.954%2c19.472%2c36.565%2c21.189%2c36.565z M24.026%2c49.432c-0.001%2c0-0.002%2c0-0.004%2c0h-0.365 l0.404%2c1.619c0.031%2c0.127%2c0.003%2c0.261-0.077%2c0.362c-0.081%2c0.104-0.204%2c0.164-0.335%2c0.164h-0.642l-0.22%2c2.734 c-0.018%2c0.223-0.201%2c0.391-0.423%2c0.391h-2.353c-0.221%2c0-0.404-0.168-0.422-0.391l-0.219-2.734h-0.643 c-0.131%2c0-0.254-0.062-0.334-0.164s-0.108-0.235-0.076-0.362l0.404-1.619h-0.365c-0.119%2c0-0.232-0.049-0.312-0.136 c-0.08-0.09-0.12-0.204-0.111-0.321l0.424-5.272c0.019-0.222%2c0.203-0.392%2c0.424-0.392H23.6c0.221%2c0%2c0.404%2c0.17%2c0.422%2c0.392 l0.418%2c5.188c0.01%2c0.038%2c0.016%2c0.077%2c0.016%2c0.118C24.455%2c49.243%2c24.277%2c49.427%2c24.026%2c49.432z M32.8%2c46.349 c-0.08%2c0.085-0.191%2c0.135-0.309%2c0.135h-0.725l0.568%2c2.346c0.025%2c0.057%2c0.039%2c0.115%2c0.039%2c0.182c0%2c0.234-0.189%2c0.419-0.428%2c0.424 c-0.002%2c0-0.003%2c0-0.005%2c0h-0.693l-0.298%2c4.874c-0.014%2c0.226-0.199%2c0.396-0.424%2c0.396h-2.786c-0.225%2c0-0.409-0.174-0.423-0.396 l-0.299-4.874h-0.693c-0.13%2c0-0.252-0.061-0.333-0.162c-0.081-0.101-0.109-0.234-0.079-0.359l0.587-2.428h-0.725 c-0.116%2c0-0.229-0.049-0.309-0.135c-0.08-0.085-0.122-0.197-0.114-0.314l0.502-8.225c0.014-0.223%2c0.199-0.396%2c0.424-0.396h5.709 c0.225%2c0%2c0.409%2c0.177%2c0.423%2c0.396l0.503%2c8.225C32.921%2c46.149%2c32.879%2c46.264%2c32.8%2c46.349z M47.483%2c36.353 c1.719%2c0%2c3.109%2c1.393%2c3.109%2c3.108s-1.394%2c3.106-3.109%2c3.106c-1.715%2c0-3.107-1.391-3.107-3.106S45.766%2c36.353%2c47.483%2c36.353z M39.571%2c30.664c1.717%2c0%2c3.107%2c1.393%2c3.107%2c3.108c0%2c1.718-1.392%2c3.109-3.107%2c3.109c-1.717%2c0-3.107-1.394-3.107-3.109 C36.46%2c32.057%2c37.853%2c30.664%2c39.571%2c30.664z M43.235%2c46.349c-0.078%2c0.085-0.189%2c0.135-0.309%2c0.135h-1.062l-0.479%2c7.822 c-0.013%2c0.225-0.197%2c0.396-0.423%2c0.396h-2.786c-0.226%2c0-0.41-0.174-0.425-0.396l-0.479-7.822h-1.062 c-0.116%2c0-0.229-0.05-0.311-0.135c-0.08-0.085-0.121-0.199-0.113-0.314l0.502-8.225c0.015-0.224%2c0.199-0.396%2c0.424-0.396h5.71 c0.225%2c0%2c0.409%2c0.176%2c0.424%2c0.396l0.502%2c8.225C43.357%2c46.149%2c43.316%2c46.264%2c43.235%2c46.349z M50.628%2c49.296 c-0.08%2c0.087-0.193%2c0.136-0.312%2c0.136h-0.842l-0.393%2c4.884c-0.021%2c0.221-0.201%2c0.391-0.424%2c0.391h-2.354 c-0.223%2c0-0.404-0.17-0.422-0.391l-0.396-4.884h-0.843c-0.118%2c0-0.23-0.049-0.312-0.136c-0.08-0.09-0.121-0.204-0.111-0.321 l0.424-5.272c0.019-0.222%2c0.203-0.392%2c0.424-0.392h4.818c0.221%2c0%2c0.404%2c0.17%2c0.422%2c0.392l0.424%2c5.272 C50.748%2c49.092%2c50.707%2c49.206%2c50.628%2c49.296z'/%3e %3c/g%3e %3c/svg%3e"
 })

width and height has been set.

@vipcxj I'm pretty sure those refer to pixels, so they can't have decimal values.

try integer value, too, still not work

any news?


[pwrmgr_cdc.sv] slow_main_pd_no should reset to 1'b1

It resets to 1'b0, which mismatches the programmed value in the fast clock domain.
https://github.com/lowRISC/opentitan/blob/d150a3c65c4607bad763e9f0214a74f5f7215bdb/hw/ip/pwrmgr/rtl/pwrmgr_cdc.sv#L152

@tjaychen, any thoughts on this one?

ah good catch. I will fix.


Kubernetes manager uninstall shows error while running django

I installed django-kubernetes-manager in my django app but unsupporting with Mysql I uninstalled it. Now when I try to run my django app it shows error me like,
Watching for file changes with StatReloader
Exception in thread django-main-thread:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/usr/local/lib/python3.8/threading.py", line 870, in run
self._target(*self._args, **self._kwargs)
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/utils/autoreload.py", line 53, in wrapper
fn(*args, **kwargs)
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/core/management/commands/runserver.py", line 110, in inner_run
autoreload.raise_last_exception()
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/utils/autoreload.py", line 76, in raise_last_exception
raise _exception[1]
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/core/management/init.py", line 357, in execute
autoreload.check_errors(django.setup)()
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/utils/autoreload.py", line 53, in wrapper
fn(*args, **kwargs)
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/init.py", line 24, in setup
apps.populate(settings.INSTALLED_APPS)
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/apps/registry.py", line 91, in populate
app_config = AppConfig.create(entry)
File "/opt/ashwin/app/pyenv/lib/python3.8/site-packages/django/apps/config.py", line 90, in create
module = import_module(entry)
File "/usr/local/lib/python3.8/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1014, in _gcd_import
File "", line 991, in _find_and_load
File "", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'kubernetes_manager'

sorry I forget to remove it from the installed_app in settings.py file


CI: propose a github action to check dead links

Type of change

[ ] Add an entry to an existing category.
[X] Remove an existing entry.  Dead or misleading link
[X] Modify an existing entry. Moving url
[X] Add/modify/remove a category or subcategory. Remove Coap category since empty
[X] Other fixes or amendments.

Rationale
I saw the current repo relies on Travis, i'm proposing an alternative to use github action, the key benefits i can see:

Easy access the result to everyone
Github action Configuration in the repo through the file:  https://github.com/mfornos/awesome-microservices/blob/dd8f2779c26b1590777571e10439aabe3f6519bc/.404-links.yml
Contributor can support on the github action config

Checklist

[X] The contribution follows the CONTRIBUTING.md document guidelines.

Regarding the suggested entry:

[X] It is sorted alphabetically in its section.
[X] It does not exist already in the list.
[X] It is not for marketing purposes.

Before my changes:

Full detail:

Before: https://github.com/olivierodo/awesome-microservices/actions/runs/3834291786
After: https://github.com/olivierodo/awesome-microservices/actions/runs/3834447334/jobs/6526859533

Thank you very much for this contribution @olivierodo
I will review the changes as early as possible :)


JointGroupPositionController should respect velocity limits

Is your feature request related to a problem? Please describe.
As described here I'm trying to port old urdf tutorials.
In ROS 1, you could limit the speed that the position controller moved at by specifying a <limit> tag within the <joint> tag. See this part of the tutorial
I'm trying a similar thing in ROS 2, but the <limit> tag has no effect, still snapping immediately. Check out this branch and this section of the partially rewritten tutorial
Describe the solution you'd like
The functionality that existed in ROS 1
Describe alternatives you've considered
Switching back to ROS 1 forever.

I agree it should.
Due to early insecurities about param handling with ROS2 we plowed ahead
without the controllers having access to the URDF.
Now there's a PR that fixes that so it opens up the possibility of
respecting joint limits once again automagically (some controllers already
do by having duplicated limits info in their configs).
On Mon, 14 Aug 2023, 15:12 David V. Lu!!, @.***> wrote:

Is your feature request related to a problem? Please describe.
As described here https://robotics.stackexchange.com/q/25100/35046 I'm
trying to port old urdf tutorials.
In ROS 1, you could limit the speed that the position controller moved at
by specifying a  tag within the  tag. See this part of the
tutorial
http://wiki.ros.org/urdf/Tutorials/Using a URDF in Gazebo#Joint_Control
I'm trying a similar thing in ROS 2, but the  tag has no effect,
still snapping immediately. Check out this branch and this section of the
partially rewritten tutorial
https://github.com/ros/urdf_sim_tutorial/tree/ros2_devel#joint-control
Describe the solution you'd like
The functionality that existed in ROS 1
Describe alternatives you've considered
Switching back to ROS 1 forever.
—
Reply to this email directly, view it on GitHub
https://github.com/ros-controls/ros2_controllers/issues/744, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AA24PYKH43IDSEHNQRTWIGLXVIW5NANCNFSM6AAAAAA3PYEXJA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Please rate your experience 1-10 with our customer success team professionals today!!


Bimap should be an instance of Ord

The instance is free and allows things like using Bimaps as Map keys.

Sorry it's taken me so long to respond. This looks reasonable. I'll try to get it merged on the weekend.

http://hackage.haskell.org/package/bimap-0.3.3


Initial config for pre-commit

Add an initial configuration for pre-commit.com checks to run on this repository
Related to #2530

Some of these pre-commit tests fail on the current contents of the repo. Some pre-work exists to resolve those failures.


Autoware keeps acceleration less than 0 and the ego vehicle does not drive at all from the starting point.

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
Hi team,
I'm currently running Autoware with Carla using bridge implementation, and I found a situation that Autoware keeps acceleration less than 0 and the ego vehicle does not drive at all from the starting point.
Here's the rviz video of the situation: [video] and corresponding launch.log file: [launch.log]
When I checked the control command by subscribing the topic /control/command/control_cmd and plotted acceleration and speed as below, it showed that the acceleration is below 0 and  the speed does not increase (the values oscillates though..).

Expected behavior
I hope the ego vehicle to drive well to the goal point.
Actual behavior
But the ego vehicle does not start to drive.
Steps to reproduce
Here's the ros2bag file for the reproduction: [ros2bag]
Versions

autoware.universe: 7877192cb5700c1db15109dcd959785fc381951d
Carla: 0.9.15

Possible causes
No response
Additional context
I thought this issue is not duplicated with https://github.com/autowarefoundation/autoware.universe/issues/8032, as no logs in launch.log says process died.

Your logs basically show 3 errors:
1720950077.5001881 [component_container_mt-64] [ERROR] [1720950077.499698771] [compressed_depth_image_transport]: Compressed Depth Image Transport - Compression requires single-channel 32bit-floating point or 16bit raw depth images (input format is: rgb8).
1720950077.5126553 [component_container_mt-6] [WARN] [1720950077.511458876] [sensing.lidar.concatenate_data]: transformed_raw_points[/sensing/lidar/left/pointcloud] is nullptr, skipping pointcloud publish.
1720950077.5150239 [component_container_mt-6] [WARN] [1720950077.511521889] [sensing.lidar.concatenate_data]: transformed_raw_points[/sensing/lidar/right/pointcloud] is nullptr, skipping pointcloud publish.

Concatenated lidar data is required by the perception pipeline, so the concatenation issue could be blocking in your case.
Did you check Autoware diagnostics with RQT for other potential issues?

Thank you for the response @VRichardJP!
Actually, I did not check the whole perception pipeline, but I checked that the car drives well in most of the cases even with the messages above, and it is not that frequent to stuck at the starting point.
To my knowledge, those error messages are from the bridge implementation as the bridge is not handling the topics /sensing/lidar/left/pointcloud and /sensing/lidar/right/pointcloud.
I'll check if those errors are relevant to this issue soon.
Thanks!

Hi @VRichardJP!
When running simulation on Carla with the bridge, there's the only LiDAR for the ego vehicle due to performance issue, so it seems we need not to process those topics for now. (Thank you for checking @maxime-clem!)


[Bug] Typings startWith

It seems like there is a bug in the typing for startWith.
It seems like it is introduced with #5376
TS throws following error:
Type `...` is missing the following properties from type `SchedulerLike`...

This occurs when ts is falling back to the following type signature of startWith:
export function startWith<T>(scheduler: SchedulerLike): MonoTypeOperatorFunction<T>;

This could occur for example by doing the following:
of(true).pipe(
    startWith<boolean>(undefined)
);

In general, this problem occurred recently in G3 so it is at least some kind of breaking behavior

Not a bug, IMO.

It also doesn't work if you set the explicit type parameter to boolean. And for some reason this was just popping up at google, so I don't know, but it seems like a regression or something.

The answer is to not use an explicit type parameter - this is Google's own advice. Or, if one is to be specified, specify one that's compatible with the argument being passed:
of(true).pipe(startWith<boolean | undefined>(undefined));

Not a bug, IMO.

Even of(true).pipe(startWith<boolean | undefined>(undefined)); is not working.  It gives the same error Jan-Niklas mentioned above. The only way I could get it to work is with
startWith(undefined as boolean|undefined)

This is particularly annoying on void observables, such as Subject<void>. I really just want startWith() which doesn't work, and because of this issue startWith(undefined) works but shows as a deprecated signature. The only way to truly have it work is then something like startWith(undefined as void) which is slightly insane. :-)

Addressed in https://github.com/ReactiveX/rxjs/pull/6114


SRX-63BV32 If config.json file is invalid, give a fatal error during repository initialization

https://revolution.dev/app/-JqFGExX46gs9mH7vxR5/-M37cZx7XYoOvQ5Mmcir/FILESYSTEM/FOLDER/ROOT/STORY/-Mp6i5lzSJREYmTNHy6p
When the config.json file in any of the environments is an invalid json file (missing file is ok) then do not start kuberpult cd-service, throw a valid error message and quit, do the same in /health check as well.

@mnishamk-freiheit please add tests


Provide an arm64 build for torchvision on pytorch-nightly channel

🚀 The feature
an arm64 build for torchvision on pytorch-nightly channel
Motivation, pitch
PyTorch has already provided arm64-build on pytorch-nightly channel and support M1 GPU acceleration, however, M1 users like me still cannot benefit from this, because of the lack of an arm64 torchvision.
Pls provide with an arm64 build for torchvision on pytorch-nightly channel.
Thanks.
Alternatives
No response
Additional context
No response
cc @seemethere

Linked to mps issue in https://github.com/pytorch/pytorch/issues/77748#issuecomment-1130554011

@ivaquero Thanks for raising this.
You are right this is an important missing features. Your request is already recorded at #5171, so I will close this issue to avoid duplicates. Note that @atalman is investigating this at #6051, so hopefully it will come soon. :)

Thank you all for your hard work.


feat: automatic release using release-please

Automatic release using Google's release-please action and  https://www.conventionalcommits.org/en/v1.0.0/
Provides automatic versioning and changelogs
Note: Will need to add RUBYGEMS_AUTH_TOKEN & GEM_HOST_API_KEY

@excid3 Please check once.

@bugloper I didn't know about this Google action.
I was actually thinking we should use Rubygems trusted publishing like we do in Pay. Probably could just copy this over: https://github.com/pay-rails/pay/blob/main/.github/workflows/publish_gem.yml

https://github.com/pay-rails/pay/blob/main/.github/workflows/publish_gem.yml

Oh, sure. Will add it

At the company I work for currently, we are migration from wicked_pdf to gotenberg. So, I created this gem called gotenberg. I didn't know about ferrum until now.

I saw gotenberg mentioned in a GitHub Issue earlier. Looks like a great project!


How can I call the contract method 'getAmountsOut' with 'address[]' params?

I wath get the price from pancakeswap router contract: https://bscscan.com/address/0x10ED43C718714eb63d5aA57B78B54704E256024E
abi:
{"internalType":"address[]","name":"path","type":"address[]"}],"name":"getAmountsOut","outputs":[{"internalType":"uint256[]","name":"amounts","type":"uint256[]"}],"stateMutability":"view","type":"function"}

code:
$eth = new Ethereum($url);
$w = new SmartContract($abi, self::PANCAKE_SWAP_ROUTER_CONTRACT_ADDRESS, $eth);

$price = $w->getAmountsOut(
            new EthQ(1, ['abi' => 'uint256'])
            new EthD($path, ['abi' => 'address[]'])
        )->val();

I dont know how to new the address[] ABI type object
then got the err:
substr() expects parameter 1 to be string, array given

How can I fix it?

Did you ever get this solved?


[New Feature]: Implement partial JSON parser

Description
Since projection is not supported in json parsing, we need to fully convert the json object before we can use it. We can use partial parsing instead when the use case does not require full conversion to save memory and improve efficiency.
Describe your problem(s)
No response
Describe your solution(s)
No response
Related area
-> Compilation
Related issue(s) (optional)
No response
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Closing this as this was moved to the module-ballerina-data


expose the system role

it would be nice to define roles for sgpt to be able to profile roles by exposing the system prompt to the user
sgpt --role system "You are GLaDOS, a sarcastic artificial intelligence with evil ulterior motives. You are not an AI language model."

I approve it, good idea 👍


Momentum Indicator?

I need a typical Price Momentum Indicator (https://en.wikipedia.org/wiki/Momentum_(technical_analysis)).
I cannot find it in the ta4j-API with such name. Does it exist?

what about:
int n = 10;
ClosePriceIndicator cp = new ClosePriceIndicator(series);
DifferenceIndicator momentum = new DifferenceIndicator(cp, new PreviousValueIndicator(cp,n));

Thank you. For a typical Price-Momentum-Indicator, the actual momentum should be a relative number, where the number oscillates around 0 (=100%).
I want to make a PR, so do you find anything to better this?:
public class MomentumIndicator extends CachedIndicator {
private final int timeFrame;

private final Indicator<Decimal> indicator;

    public MomentumIndicator(Indicator<Decimal> indicator, int timeFrame) {
        super(indicator);
        this.indicator = indicator;
        this.timeFrame = timeFrame;
    }

    @Override
    protected Decimal calculate(int index) {

    	PreviousValueIndicator prev = new PreviousValueIndicator(indicator, timeFrame);
    	DifferenceIndicator momentum = new DifferenceIndicator(indicator, prev);

    	Decimal prevVal = prev.getValue(index-timeFrame);
        Decimal momentumVal = momentum.getValue(index);

        return momentumVal.dividedBy(prevVal)
                .multipliedBy(Decimal.HUNDRED);
    }
}

Looks good. But regarding #192 PR will not be merged.

Okay, no need for that. Because ta4j already has such a momentum-indicator called "ROCIndicator".


Fix ResourceWarning appearing when running run_tests.py.

Can appear as:
run_tests.py:98: ResourceWarning: unclosed file <_io.TextIOWrapper name='testsuite/nocodings.py' mode='r' encoding='UTF-8'>
  stdin_get_value.return_value = open('testsuite/nocodings.py').read()

Coverage remained the same at 100.0% when pulling bfbce0f81fd83edbbadf699672a339fb6623d08e on jdufresne:close-file into 2b9566814e085023a9b5ef0f8e5b15dcd932f6c0 on tk0miya:master.


PerfMon detectors showing up in external calls

Looks like Perf Counters are made public and showing up in external calls:
exceptioncount
threadcount
Need to make them internal only detectors

deployment completed for this bug. You can quickly verify and close this one.


Add canvas cleanup for useColumnSizer()

#711 noticed that some of the canvas elements created by the grid were not being cleaned up. I looked into it and discovered that while most of them are being cleaned up (e.g. DataGrid's bufferA and bufferB)
https://github.com/glideapps/glide-data-grid/blob/c9e1f3df5d545171ccca2226c0c55e253fb6cc83/packages/core/src/data-grid/data-grid.tsx#L637-L647
the same was not being done for the offscreen canvas in useColumnSizer()
https://github.com/glideapps/glide-data-grid/blob/c9e1f3df5d545171ccca2226c0c55e253fb6cc83/packages/core/src/data-editor/use-column-sizer.ts#L81-L89
I adapted the useEffectLayout() and cleanup logic from the former to use in the latter.

Before: Canvases being left behind with each Show/Hide
https://github.com/glideapps/glide-data-grid/assets/94077014/57e74d4a-0b98-4b4c-b918-1e0ccc45f633
After: All canvases are cleaned up after each Show/Hide
https://github.com/glideapps/glide-data-grid/assets/94077014/78c5e04f-a07e-4bae-a411-faa728ae3317

Side note: I noticed there are three places where canvases are getting created with essentially style="display: none; opacity: 0; position: fixed;. Maybe this should be consolidated in a helper function?
Not sure how much of a performance concern it is, but there appear to be a number of places where document.createElement("canvas") is called such as any time normalSizeColumn() gets called or its dependencies (such as columns, mergedTheme or rows among others) change, any time LinkCell's onSelect or onClick get called. And canvases are retained for each sprite of each fg/bg color and dpr, and for each bg color, border style, dpr and height of each drill down border.

Looks good!


Remove Doesn't Seem to work all of the time

Hello,
I have a map with several layers that contain markers in each. When I remove a layer I'm trying to remove the edge marker for the markers that are no longer there.  When I call .destroy, the markers don't go a way. The edge markers continue to point to makers that are no longer in the map.  It works fine when I toggle all on and off but not for individual layers.

Hi,
can you please provide an example page where the error happens?

It's not so much an error as it doesn't seem to remove the edge marker
So in one case I monitor a change on the map like listenerService.listen('USER_SETTINGS_CHANGE'), function(settings){
if (settings.edgeMarkers){
edgeMarker.destroy();
}
});
but in the one where it's not working,  Is below. No error is thrown it just doesnt do the removal. I have some similar code like
listenerService.listen('LAYER_REMOVED), function(){
edgeMarker.destroy();
});
edge markers is created above either of the listeners like this:
var edgeMarker;
listenerService.listen('RESULTS_PRESENT), function(results){
var resultsLayers = results.getLayers();
var myLayerGroup = L.layerGroup(resultsLayers );
edgeMarker = new L.EdgeMarker({
icon: myIcon,
layerGroup: myLayerGroup ,
title:'clickThest',
rotateIcons:true
}):
edgeMarker.addTo(map);
});

OH this is on an Intranet so I have no way to point you to a live example

Sorry, I need a working example to debug your issue.

Your demo doesn't work either @ubergesundheit that makes it hard for us to help ourselves.

@SLCPS I fixed the demos, but I haven't done anything recently with the project and don't have time beyond fixing the demo.


Filters Removal causes "type" error

Package
filament/tables
Package Version
v3.2.92
Laravel Version
v10.48.13
Livewire Version
v3.5.1
PHP Version
PHP 8.0.2
Problem description
So I am using only filament's table builder in my project and I am using both queryBuilder and simple Filter in one of my tables. The scenerio I have in my table is that I want to preselect some filters when the table loads like that:
          ->filters([
            Filter::make('status')
                ->form([
                    TextInput::make('status_name')->debounce(500)->default('active'),
                ])->query(function (Builder $query, array $data) {
                    return $query->whereHas('status_type', function ($query) use ($data) {
                        $query->where('name', 'like', '%' . $data['status_name'] . '%');
                    });
                })->indicateUsing(function (array $data): array {
                    $indicators = [];
                    $name = $data['status_name'] ?? '';

                    if (!empty($name)) {
                        $indicators[] = Tables\Filters\Indicator::make($name)->removeField('status_name');
                    }

                    return $indicators;
                }),
            Filter::make('assignee')
                ->form([
                    TextInput::make('assignee_name')->debounce(500)->default(Auth::user()->full_name),
                ])->query(function (Builder $query, array $data) {
                    return $query->whereHas('administrative_officer', function ($query) use ($data) {
                        $query->orWhere('name', 'like', '%' . $data['assignee_name'] . '%');
                        $query->orWhere('surname', 'like', '%' . $data['assignee_name'] . '%');
                    });
                })->indicateUsing(function (array $data): array {
                    $indicators = [];
                    $assignee = $data['assignee_name'] ?? '';

                    if (!empty($assignee)) {
                        $indicators[] = Tables\Filters\Indicator::make($assignee)->removeField('assignee_name');
                    }

                    return $indicators;
                }),
            Filter::make('admin')
                ->form([
                    TextInput::make('admin_name')->debounce(500)->default(Auth::user()->full_name),
                ])->query(function (Builder $query, array $data) {
                    return $query->whereHas('authorized_officer', function ($query) use ($data) {
                        $query->orWhere('name', 'like', '%' . $data['admin_name'] . '%');
                        $query->orWhere('surname', 'like', '%' . $data['admin_name'] . '%');
                    });
                })->indicateUsing(function (array $data): array {
                    $indicators = [];
                    $assignee = $data['admin_name'] ?? '';

                    if (!empty($assignee)) {
                        $indicators[] = Tables\Filters\Indicator::make($assignee)->removeField('admin_name');
                    }

                    return $indicators;
                }),
            QueryBuilder::make()
                ->constraints([
                    QueryBuilder\Constraints\TextConstraint::make('case_number'),
                    QueryBuilder\Constraints\TextConstraint::make('case_date'),
                    QueryBuilder\Constraints\TextConstraint::make('case_cause.name'),
                    QueryBuilder\Constraints\TextConstraint::make('case_forum.name'),
                    QueryBuilder\Constraints\TextConstraint::make('case_reference_number'),
                    QueryBuilder\Constraints\TextConstraint::make('department.name'),
                    QueryBuilder\Constraints\TextConstraint::make('district.name'),
                    QueryBuilder\Constraints\TextConstraint::make('member_category.name'),
                    QueryBuilder\Constraints\TextConstraint::make('relates_to.case_number'),
                    QueryBuilder\Constraints\TextConstraint::make('award'),
                ])->columnSpanFull(),
        ], Tables\Enums\FiltersLayout::AboveContentCollapsible)

But the problem is when I add a rule from query builder like that:

And when I remove all the filters by clicking the remove all button on the bottom right I get this error: "Undefined array key "type" which is happening only if I use both the filters (simple filter and queryBuilder)
Expected behavior
Expected behavior would be not getting any error if I clear all the filters.
Steps to reproduce

Use both filters and click the clear all button in bottom right
Reproduction repository (issue will be closed if this is not valid)
https://github.com/PropaySystems/solidarity
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

I solved this issue for now by overriding the "removeTableFilters" function in my Livewire component and unsetting queryBuilder from the filters like that:
    $filters = $this->getTable()->getFilters();

    if(isset($filters['queryBuilder'])) {
        unset($filters['queryBuilder']);
    }

Invalid reproduction repository.


Switch to Open Telemetry for logging

Resolves #3761
Switch to Open Telemetry logging and add variable to set logging level. Also updated the docs.
This makes troubleshooting issues  much easier.
Custom attributes for operation_id:

Application map:

Fast API requests:

/test-extended

/test-extended

/test-extended

/test-destroy-env


EndpointFeature.Invoker - RequestDelegate or Func<RequestDelegate, RequestDelegate>?

Investigate whether Invoker should be RequestDelegate or Func<RequestDelegate, RequestDelegate>

Done https://github.com/aspnet/Routing/pull/712


ApplePay 422 Error

Hi, we're getting this error in the latest 4.1.6.
The endpoint did not accept the request. (Code: 422)
Note that in 4.1.0 ApplePay works fine.
I checked channels settings and app webhooks are selected.
Update: looks like an SDK error: https://github.com/checkout/checkout-sdk-php/issues/44

Please note we do not offer any shipping, perhaps that's the cause? missing shipping data?

After some debugging seem that main issue is token_invalid in the gateway response.

We found out issue is in class-wc-checkoutcom-api-request.php - removing sanatize from $apple_token = sanitize_text_field($_POST['token']); fixed the issue.


⚠ Bofrost.at has degraded performance

In 488590c, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1528 ms

Resolved: Bofrost.at performance has improved in 85310b3.