Set the version of the HTTP Server based in the project version

Set the version of the HTTP Server based in the project version in the Info.plist

I reported the Linux issue with CircleCI support as the issue is trying to run swift test once the Docker image is loaded.


Re-style coronavirus index page

Pair programming effort between @danacotoran and @hannako
What
Frontend: Implement the design of the new index page for the coronavirus publishing tool.
Backend: Wire up the index page to read from database objects passed through from the controller.
Before

After

https://trello.com/c/u16kFlyh

@DilwoarH could you have a look at the FE changes in this PR please


Optimize compound interest calculation

Is your feature request related to a problem? Please describe.
The compound interest formula uses saturating_pow, which can iterate 32 times in the worst case. Lending interest is accrued by external pallets such as the vault-registry or oracle, so it's very important that the implementation is efficient.
Describe the solution you'd like
Aave uses a Taylor series to approximate the exponential and only computes the first three terms. We could use a similar approach, to reduce the function weight. See https://github.com/interlay/interbtc/issues/758.

If compound is using simple interest, why don't we do the same?


Enhance directions around the Hipster store

Need to enhance directions on how to use the Hipster Store sample.
Documentation updates go in the ortelius/ortelius-docs repo.  See this documentation section for the directions on using the sample Hipster Store.

Hipster store has been removed as the SaaS is no longer supported. More has been done around the Ortelius CI/CD CLI for getting started.


Add support for os.path.expanduser for ~/ and ~username/

This is a small change as described in #8 that expands things like ~/somepath or ~user/somepath to the appropriate value in the same way the shell would. This uses os.path.expanduser and is very simple
I tested it with the existing support for the environment variables and didn't see any conflicts or breakage
I didn't put too much thought into the regex for this:
    _uservar_tag_matcher = re.compile(r"^~(\w*?)/")

I figure expansion should only happen for values that begin with ~ followed by alphanumeric or underscore. I wonder if the trailing slash should be optional instead of required?

As a future development, I think this can be extended easily after resolving envvar values.
Opened an issue with this: #16

As a future development, I think this can be extended easily after resolving envvar values.
Opened an issue with this: #16

Great, thanks for accepting this and thanks again for this project


Ability to add arbitrary tags and comments into

Currently useHead() implements a whitelist of tags that can be added into the <head> element. I need to add an ESI tag or an HTML comment to load assets originating from another service. These needs to be added conditionally, so even if Nuxt 3 would allow overriding its app.html a solution within useHead() would be best.
To illustrate, I am currently using this hack:
<template>
    <Style v-if="isServer" :children="myCustomTag" />
    <NuxtPage/>
</template>

<script setup>
const isServer = process.server
const myCustomTag = ' </style> <!--esi <esi:include src="/foo/bar" />--> <style> '
</script>

This inserts the comment at the cost of two empty <style> tags. Obviously this can only be a temporary workaround.
I am unsure what the best API would be, maybe just teleport all child elements of <Head> into <head>. E.g:
<Head>
  <!--esi <esi:include src="/foo/bar" />-->
</Head>

I guess the skys won't tumbling down, if you use plain JS to add a comment node
https://developer.mozilla.org/en-US/docs/Web/API/Document/createComment

This requires access to "document" which was not available in the server side nodejs environment.

ok, wasn't aware of the server side

Sorry but I don't think this will be implemented, at least anytime soon. It adds too much complexity to the code and scope for issues.
You should use Nuxt's render html hook if you need something like this https://v3.nuxtjs.org/api/advanced/hooks#nitro-app-hooks-runtime-server-side


Removing some features (for now)

Hey all, the WashU team has deemed some of the following features as unnecessary at this point in time, especially for the first release of the extension. We thought it would be better to potentially add these back in as needed, but please feel free to discuss it here. Thank you!

Property Name
Type
Description

ml:creation_method
string
REQUIRED. how the label was created e.g. field walk, image-annotated, ML generated

ml:creation_method_detail
string
additional details about the creation method, including potentially a link to the algorithm

ml:boundary_type
string
REQUIRED. whether the boundary is for a field, ownership parcel, hedge, or other boundary

ml:creation_date
datetime
REQUIRED.[Auto-added] Date collection was created or published

ml:verified_until
datetime
REQUIRED. the last date for which the boundary is verified to be present on ground (e.g., through ground-truth or remote sensing image verification)

ml:quality
string
REQUIRED. the level of quality control applied to this polygon

ml:confidence
float
a value between zero and one that represents the confidence in a generated polygon (higher value = higher confidence)

ml:machine_generated
boolean
REQUIRED. Indication of machine-generated boundaries or not

ml:admin0
string
REQUIRED. [Auto-added] a unique admin0 (country) name for the administrative region that contains the field - can auto-generate from GADM boundaries

ml:crop_type
string
the type of crop e.g. maize

ml:crop_category
string
REQUIRED. high-level crop category e.g. cereal (e.g. FAO)

I think ml:admin0 is duplicate of ml:country_code, so can be removed.
ml:verified_until can be removed in favor of determination_datetime.
ml:creation_method and ml:machine_generated can be removed in favor of determination_method.
ml:creation_date is now in the collection properties of this extension.
ml:boundary_type seems to be a good field for a separate extension.
ml:crop_type and ml:crop_category seem to be a good fit for another extension.
ml:quality and ml:confidence could also be a good extension, but may also fit in here.
The only one that's really left then is ml:creation_method_detail.

Some outcomes from yesterdays AI ecosystem discussion:

Extension is not used in FTW yet
We likely mmove many or all fields of the extension into core or other extensions (e.g. timestamps) We then make AI one or two (train, output) meta-extensions of the other new extensions.

Chris to explore the auxiliary data and how it can be integrated with fiboa datasets.


S3 Checkpoint Saving By URI

What does this PR do?
This PR enables checkpoint saving to S3 using a URI by doing the following:

[x] parses save_folder and if it's URI with s3:// prefix, then creates an RemoteUploaderDownloader object
[x] checks to make sure a RemoteUploaderDownloader object with an S3 bucket_uri is not already in loggers (if so then doesn't use the URI and throws a warning)
[x] unit tests
[x] manual tests

What issue(s) does this change relate to?
fixes CO-1143
Before submitting

[x] Have you read the contributor guidelines?
[ ] Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
[x] Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
[ ] Did you update any related docs and document your change?
[x] Did you update any related tests and add any new tests related to your change? (see testing)
[x] Did you run the tests locally to make sure they pass?
[x] Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

Oh I see, the remote_file_name and latest_remote_file_name are left in to support wandb/libcloud/sftp? Do you think we can get rid of those args by using wandb://path, libcloud://path, and sftp://path in this pr, and just require that the correct logger was created by the user in those cases (deferring automatic creation of those, but still removing the extra arguments)?

Ok I thought about it some more and I'm ok leaving remote_file_name in for now to simplify things, but it looks like if I pass s3://bucket/path/to/checkpoints/ as the save folder, the path/to/checkpoints will only be used for the local path not the remote path. I think that path should be passed to s3 as the prefix?
Also, can you include abhi's request to add the PT file extension to the remote files?

ok @dakinggg, I added support for empty path strings and I removed save_remote_file_name and save_latest_remote_file_name. PTAL


vscode snapshot tests not failing when snapshots don't match

Description
My snapshot test isn't failing even when the snapshot stored and the actual value are not equal.
https://github.com/microsoft/vscode/assets/16353531/4400c161-49fc-4f18-8488-dc741bbabfd0
Repro

Create a new snapshot test
Run the new test, which will create a snapshot file
Change the output of your test to something else
Run the test again - it doesn't fail

Version
Version: 1.83.0-insider
Commit: 4268e464763087044d0c1b5bdd37ebbe683cadfa
Date: 2023-09-29T20:44:05.310Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.6.0

You're missing an await on assertSnapshot. I admit that I forgot to do that a few times myself. Maybe we should have a lint rule for "must use/await result".


Remove test dependency on Java default exception message

Backport of fix in #12724

Backport change already approved in https://github.com/apache/lucene/pull/12724. Merging.


azcopy download fails md5 check, tool broken for 100GB+ data

I am trying to download large files from public blob to a local drive with azcopy version 10.26.0 on PowerShell7.
 azcopy copy "https://comp21storage.z5.web.core.windows.net/wiki-cohere-35M/wikipedia_base.bin" cohere/ --from-to BlobLocal

The download proceeds to 100% and then fails MD5 checksum test.
024/10/17 07:43:28 ERR: [P#0-T#0] DOWNLOADFAILED: https://comp21storage.z5.web.core.windows.net/wiki-cohere-35M/wikipedia_base.bin : 000 : the MD5 hash of the data, as we received it, did not match the expected value, as found in the Blob/File Service. This means that either there is a data integrity error OR another tool has failed to keep the stored hash up to date. (NOTE In the specific case of downloading a Page Blob that has been used as a VM disk, the VM has probably changed the content since the hash was set. That's normal, and in that specific case you can simply disable the MD5 check. See the documentation for the check-md5 parameter.). When Checking MD5 hash. X-Ms-Request-Id:

This problem is repeatable on windows cmd prompt, WSL on multiple machines and with multiple origin files on different storage accounts. For example, with this URL as well `'https://msmarco.z22.web.core.windows.net/msmarcowebsearch/vectors/SimANS/passage_vectors/vectors.bin'
Other tools like wget and python downloader packages work just fine for the same blob (we have other ways of validating data integrity).

Hi @harsha-simhadri ,
Could you please get the md5 sum after you download the file from another tool and validate if it matches. This is to see if the file in storage is corrupted.
Alternatively, you could disable the md5 check entirely if you just want to download the file.
azcopy copy <source> <destination> --check-md5 None

After several experiments by my team across machines, we are convinced that the tool is downloading corrupt data (its not md5 error). We manually inspected and we have other sanity checks for our data.
Specifically, it works fine upto 100GB and after that it injects some spurious data in the stream at 100GB mark. So large file download is broken.

same issue here -

Hello,
The issue you're encountering is likely due to the fact that you're trying to download a file from a static website endpoint, rather than directly from the Blob Storage endpoint. When you use the azcopy command with --from-to as BlobLocal, it expects a standard Blob Storage response (which includes proper block ranges), the mismatch leads to the download being treated as incomplete or corrupt, with the checksum validation failing after download.
To resolve this issue, we suggest you to use the Blob Storage endpoint instead of the static website URL.

Closing this issue as this is not a supported use case.
As suggested above, please use the blob endpoint (https://.blob.core.windows.net/%24web/) instead of the static website endpoint

Thanks for the suggestion. These blobs are configured for access via static website since that was Microsoft Research's policy for creating public-facing assets without requiring authentication.
Is there a way to access blobs without requiring Azure Authentication?

Enabling public access to the associated resources would be the way to go here if you want to use Azcopy
https://learn.microsoft.com/en-us/azure/storage/blobs/anonymous-read-access-configure?tabs=portal

Anonymous access has been disabled as a matter of org policy.

Another option would be to use SAS tokens, but those have an expiration, so may not be ideal based on your scenario.


cannot import name 'Color' from 'wafw00f.lib.asciiarts'

Error occurs while running main.py
发生异常: ImportError
cannot import name 'Color' from 'wafw00f.lib.asciiarts' (C:\Users\***\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\wafw00f\lib\asciiarts.py)
  File "D:\***\toolkit\wafw00f\wafw00f\main.py", line 23, in <module>
    from wafw00f.lib.asciiarts import Color, randomArt
ImportError: cannot import name 'Color' from 'wafw00f.lib.asciiarts' (C:\Users\***\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\wafw00f\lib\asciiarts.py)

pip install wafw00f
Requirement already satisfied: wafw00f in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (2.2.0)
Requirement already satisfied: requests in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from wafw00f) (2.28.1)
Requirement already satisfied: pluginbase in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from wafw00f) (1.0.1)
Requirement already satisfied: charset-normalizer<3,>=2 in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from requests->wafw00f) (2.1.1)
Requirement already satisfied: idna<4,>=2.5 in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from requests->wafw00f) (3.3)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from requests->wafw00f) (1.26.9)
Requirement already satisfied: certifi>=2017.4.17 in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from requests->wafw00f) (2022.6.15)
Requirement already satisfied: PySocks!=1.5.7,>=1.5.6 in c:\users\***\appdata\local\packages\pythonsoftwarefoundation.python.3.11_qbz5n2kfra8p0\localcache\local-packages\python311\site-packages (from requests[socks]->wafw00f) (1.7.1)

thanks for raising this issue, I can see you're using Python 3.11 and wafw00f has been rigorously tested against the version, can you try a clean install? I suspect this might be due to a leftover files from a previous installation.

closing this issue due to inactivity! feel free to reopen if you are able to reproduce this issue.


Adding new services to the list of supported AWS services in prow CI/CD

Issue #, if available:
Description of changes:
Adding new services to the list of supported AWS services in prow CI/CD:

route53
cloudtrail
cloudwatch
cloudwatchlogs
acmpca

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Give fused return types the correct exception value

Draft for now because it depends on #5710
Fixes https://github.com/cython/cython/issues/5586
Note that this changes many fused functions from returning 0 to returning -1. I was initially worried about this, but it actually an incompatibility since Cython only assumes except -1 for non-pxd functions
# Nodes.py, line 733ish
                if (return_type.exception_value is not None and (visibility != 'extern' and not in_pxd)):

So no assumptions are made about the return type of functions where we can't see the definition. Unfortunately that reduces the scope to fix any performance regressions when using extern fused functions and checking the exception.

I'm probably missing a lot of historic trial and error here, but at first sight, as it stands now, this feels hackish to me. Like you're passing the types through a messaging channel in order to use them later. Could you explain why is is not enough to specialise the exception values at the same time as specialising the return types of the different signatures?

I'm probably missing a lot of historic trial and error here, but at first sight, as it stands now, this feels hackish to me. Like you're passing the types through a messaging channel in order to use them later. Could you explain why it is not enough to specialise the exception values at the same time as specialising the return types of the different signatures?

I think you're right and it is possible to simplify things a lot by delaying it until FusedNode does its work. The main things you need to defer are coerce_to the return type, and a check that the exception value is coercible to the return type.


DialogContext.EndDialogAsync - remove code duplication.

In DialogContext.EndDialogAsync just call EndActiveDialogAsync to pop the active dialog off the stack.

@cleemullins this looks good to me pulled locally and tested

Merging Per @johnataylor testing & signoff.


[Godot4] Update API to beta 16 breaks compat change

… from https://github.com/godotengine/godot/pull/71907
Explanation: the commit "Make AnimatedSprite's playback API consistent with AnimationPlayer #71907" renamed AnimatedSprite2D.frames to sprite_frames

This should complement https://github.com/poohcom1/godot-animated-sprite-2-player/issues/2

Thanks!


Add array options and make link format configurable

Hi!
First, nice library, really helpful and simple, thanks!
Nevertheless, I have an issue with link's format, like in the PR #14, but this PR seems to be abandoned.
I'm agree with your idea to use an array for optional parameters.
So, I worked the problem, and I hope this can help!

Hi,
Thank you for your pull request.
I worry this is pulling the library into the wrong direction - a direction where every single thing is configurable and the library is no longer simple or easy to use. One person might want different link text, another might want different headings or <hr>s output...
If we were going to go in this way, I would rewrite it so that it accepts a Listener that can then be extended if desired by third parties. But that would probably be better in a separate project, since it would be so targeted. Something like
class MyHtml2TextOutputter < DefaultHtml2TextOutputter
  function visitLink($href, $text) {
    return "$text ($href)";
  }
end

$text = new Html2Text().convert($html, new MyHtml2TextOutputter());

Regardless your PR needs some supporting tests and they are failing CI. :)

Thank for your feedback, you are right, use a listener should be a better solution.
I will waiting for this improvements.
Thanks.


Fix typo <IP_ADDRESS>

This change simply fixes what I believe to be a typo.
Signed-off-by: John Losito<EMAIL_ADDRESS>

hi @jlosito
Thank you for raising the PR unfortunately the pull has not been GPG signed. I have therefore copied and used your PR in reference for #228.
Thanks
uk-bolly

Thank you. Next time I will make sure to sign it with my gpg key.


Opening a link causes friends to freeze until the browser window is closed

Fedora 20
default browser: Mozilla Firefox 39.0a2 (developer edition)
Edit: It seems to only happen if I don't already have an instance of firefox running.

this is probably an electron bug. would be worth looking in atom/electron/issues to see if anyones reported


Custom external media player

Adding support for selecting a custom external media player, defaulting to VLC.
Much of it is renaming of VLC specific code to more generic sounding 'external player'.

Added preference External Media Player for specifying path to any program.
Refactored ipc.js and vlc.js to external-player.js

Related to issues #368 and #531.

LGTM apart from comments

I think one of the biggest things we could do to improve WebTorrent Desktop is to give it codec support comparable to VLC. If we did that, I wonder if we'd still need this. I always hoped "Open in VLC" would be a temporary patch until we could support all the common file types.

A good external player not only supports more codecs, it also offers higher quality playback and more customisation.
I think this needs a "set as default" checkbox.

Merged! I rebased the PR so that it would merge cleanly, and also committed this: https://github.com/feross/webtorrent-desktop/commit/2d3673ea33c41eb244c1df3831a96c793c1ba951

Also, sorry for the delay in getting this in, @mathiasvr!
Btw, @dcposch and I are trying to get as much done as possible in the next few days to help get 1.0 out the door!

Nice work guys! 🎉

I'm guessing this will be in v1.0 right? [since I'm not seeing it in v0.11.0] - really looking forward to it, thanks guys!

@JaKXz Yeah, it will definitely be in the next release. We'll probably do a v0.12 before 1.0.


Support torch patch base images

Do not strip patch version from torch
If the user specifies a torch version without a patch, resolve the highest patch version from our compatibility matrix
Change the version struct to use a pointer for the patch version in order to handle cases where the pointer has not been supplied (and how that manifests when comparing versions)
Standardise on StripModifier for the version rendering
No longer generate a torch minor compatibility matrix, unnecessary now we explicitly define patch version compatibilities

We should also stop stripping out torch and torchvision from the generated requirements.txt in the build process, but that's probably a different PR?

Yeh the next PR after this one handles torchvision and torchaudio


add missing data-group tests

needs data-group tests added as discussed in slack

this issue can be closed, the code has been merged.

code has been merged, this issue can be closed.


Update backend_test.py

I don't see a file with the 2 in the name on develop at the moment?

For me this works locally, I will not wait for the QA test, any objections? @cqc-alec

For me this works locally, I will not wait for the QA test, any objections? @cqc-alec

Sure we can merge this.


[incubator/sparkoperator] Upgrade to use the v1beta2-1.0.0-2.4.4 version

Signed-off-by: Yinan Li<EMAIL_ADDRESS>
Is this a new chart
No.
What this PR does / why we need it:
Upgrade to use the v1beta2-1.0.0-2.4.4 version of the Spark Operator.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
N/A
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

@yuchaoran2011 this replaces #17471.

/lgtm

@yuchaoran2011 I have fixed the Chart and the e2e test passed. Need your lgtm again. Thanks.

/lgtm


Testing API are rendering side-by-side with non-testing docs

please check your design https://material.angular.io/components/autocomplete/api

This should be fixed now


Fix typo in component name

Is the only difference here that the changelog is updated? Then I don’t think we should make a new release just because of that.

Decided to not create a new release with this typo fix.


Mount the host file system in the engine

... so that the engine container can use the host iptables.
Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

... so that the engine container can use the host iptables.

Just curious: With strongswan we did not have this requirement. Any idea what kind of iptable rules we are programming with Libreswan?
PS: Please ignore the failure in globanet job, its fixed via - https://github.com/submariner-io/shipyard/pull/201

Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

... so that the engine container can use the host iptables.

Just curious: With strongswan we did not have this requirement. Any idea what kind of iptable rules we are programming with Libreswan?

I don’t think we’re programming anything, but Libreswan checks for iptables when it starts, which produces a warning if the configured iptables don’t match the in-kernel tables. (Libreswan can use iptables for nflog devices, I think, but that’s it in terms of writing anything AFAICT; it uses it a lot in its diagnostic output, e.g. ipsec barf.)

... so that the engine container can use the host iptables.

Just curious: With strongswan we did not have this requirement. Any idea what kind of iptable rules we are programming with Libreswan?

I don’t think we’re programming anything, but Libreswan checks for iptables when it starts, which produces a warning if the configured iptables don’t match the in-kernel tables. (Libreswan can use iptables for nflog devices, I think, but that’s it in terms of writing anything AFAICT; it uses it a lot in its diagnostic output, e.g. ipsec barf.)

Okay, thanks.


Pass arguments in kwargs to view functions (same behavior as flask)

I recently bumped into an issue trying to convert an existing flask project to use apiflask. One of my view functions has a decorator which relies on arguments being passed as keyword arguments (kwargs) to work correctly. It appears apiflask does not pass arguments the same way that flask does when calling view functions. This seems to be by design for some reason, but it can break existing code.
At any rate, there are cases (as above) where the view function or its decorators needs the arguments to be in kwargs.
I found this piece of code in apiflask which will send kwargs (as flask does) if an attribute is set:
        if rule.endpoint == 'static' or hasattr(view_function, '_only_kwargs'):
            # app static route only accepts keyword arguments, see flask#3762
            # view classes created by Flask only accept keyword arguments
            return view_function(**req.view_args)  # type: ignore
        else:
            return view_function(*req.view_args.values())  # type: ignore

I'm currently using a decorator to help me mark my view function as needing kwargs:
def require_kwargs(f):
    """
    Tag a view function as needing kwargs. This is for apiflask which only passes positional
    arguments unless this attribute exists on the view function.
    """
    f._only_kwargs = True
    return f

Usage:
@bp.route("/some/path/<my_variable>/", methods=("GET",))
@require_kwargs
@another_decorator_that_needs_kwargs
def my_view_function(my_variable):
    pass

A quick search shows I'm not the only one hitting this issue https://github.com/apiflask/apiflask/issues/422 .
Please consider allowing flask-behavior when passing args in kwargs, either globally or on a route-by-route basis, otherwise adopting apiflask in existing projects will be difficult. I suggest making this a global setting to adhere to existing flask behavior.

Hi, thanks for the idea. In APIFlask 2.0, I'm thinking of turning all the arguments into keyword arguments. The input data will be passed with the location value as the keyword (json, query, etc.). For example:
@app.post('/pets/<pet_id>')
@app.input(PetQueryIn, location='json')
@app.input(PetBodyIn, location='query')
@app.output(PetOut, status_code=201)
def create_pet(pet_id, json, query):
    id = json['id']
    query_number = query['number']

I already working on this. Since this is a breaking change, I will release this with APIFlask 2.0.
What do you think about this?

Thanks for the quick reply. That would be a great solution. My main goal is to have it compatible with existing code that uses flask, so if you pass args in keyword arguments similar to flask this would solve the issue.

One slight issue with your example code is if a view function needs to parse or serialize json itself, forcing the argument name to be "json" might be problematic if the function needs to make use of the core json module itself. One solution is to use an alias ( import json as another_name ), but you may want to avoid forcing this workaround for such a commonly used module name to avoid confusion,
import json

@app.post('/pets/<pet_id>')
@app.input(PetQueryIn, location='json')
@app.input(PetBodyIn, location='query')
@app.output(PetOut, status_code=201)
def create_pet(pet_id, json, query):
    id = json['id']
    parsed = json.loads('{"example": 5}') # error

Good point. I'm thinking of setting the default arg name to the location value, then adding an arg_name to support changing it:
import json

@app.post('/pets/<pet_id>')
@app.input(PetBodyIn, location='json', arg_name='data')  # <--
@app.input(PetQueryIn, location='query')
@app.output(PetOut, status_code=201)
def create_pet(pet_id, data, query):
    id = data['id']
    parsed = json.loads('{"example": 5}')

Or how about we set the default arg name to data instead of json? That could solve the conflict with the json module, but there will be a mismatch between the location value (json) and the arg name (data). Personally, I prefer to use json for consistency, then add arg_name for customization needs.
What do you think? @chahu @luong-komorebi

Personally, I prefer to use data as the default, then add arg_name for any customization needs.

webargs will use <location>_args as the default arg name because their decorator is named use_args (https://github.com/marshmallow-code/webargs/pull/833). So I think <location>_data as the default name rule may be the best choice for us.

Another thing I would love to have is a small notice or documentation for APIFlask version < 2.0 in the migrating section apiflask.com/migrating to note the current behaviour for users migrating from Flask.

Good point, I will add a corresponding section to the migrating docs later.

Hey, I ran into a similar problem with an existing codebase, but with a parameter that is specified in the URL string. Will your solution cover that? Example:
@app.post('/pets/<int:pet_id>')
@my_decorator_which_looks_for_a_pet_id_kwarg
def create_pet(pet_id: int):
    ...

Will pet_id also appear in kwargs somehow with your intended changes outlined above?

However, the thing that changes here is when you manipulate the arguments (or keyword arguments) inside @my_decorator_which_looks_for_a_pet_id_kwarg, it will be successful and the behaviour would be the same as flask

Thanks @luong-komorebi for the quick response, but I'm not sure if I completely understand. Let me flesh this out a bit more: in my application I have a decorator that looks like this (changed to fit the above example):
def my_decorator_which_looks_for_a_pet_id_kwarg(function):
    @wraps(function)
    def decorated_function(*args, **kwargs):
        pet_id = kwargs['pet_id']
        # <insert business logic which does something with pet id>
        ...

This doesn't currently work with APIFlask (pet_id would appear, but only as the first element of args). Are you saying that with APIFlask 2.0 this will now work? Or will it not?

Hi @ddominguezb! From APIFlask 2.0, the request input data will always be passed as keyword arguments (default to {location name}_data). See the docs for more details:
https://apiflask.com/migration_guide/#use-keyword-argument-for-the-input-data
You just need to change the params to files_data.

@greyli, it worked like a charm! you saved my day
Thank you so much.


Split dmmf snapshots in multiple files

Right now the snapshots of packages/sdk/src/__tests__/engine-commands/getDmmf.test.ts are all in one big file.
https://github.com/prisma/prisma/pull/8562/files#diff-5557eb8b251a9d2d7ec5a5f262bcf765dcb352b7cee8d85052727521d8a381f7
As the snapshots are massive, any change leads to many changes in that file, very often leading Github to try to match changes in the diff that are not really correct and making it very hard to figure out what is going on:
7b921a9 (#8562)
Having a snapshot file per test for example could improve that significantly and make it easier to follow what exactly changed and why.

Closing, as this was already solved several months ago.


Added optional attribute to pass in a callback which overrides the default action

This adds more flexibility to this plugin and enables a larger audience. Let me know if there is anything you would like tweaked before merging.

Interesting, never thought it would be a feature to have but it does make sense. Looks good to me :+1:

Alright, that still works fine. It should be ready to go.


Support child components

Would be nice to support child components since using CSS absolute method doesn't trigger mouse hover on particles when you have another component blocking it. adding pointer-events: none; will remove scroll effect.
<Particles>

Hello World

</Particles>

To move a div over a canvas element, position: absolute is required. Adding child components would result in a div with absolute position containing all the children.
The effect would be the same.

Div with position: absolute; with user-select & pointer-events set to none does the trick.


ProjectMergeRequestsQueryOptions

Hello, I'd like to report a following bug:
What happened:
Calling MergeRequestsClient.GetAsync method with ProjectMergeRequestsQueryOptions and specifying MergeRequestsIds does not filter it.
Repro steps:
Example code:
client.MergeRequests.GetAsync(projectId, o => { o.State = GitLabApiClient.Models.MergeRequests.Requests.QueryMergeRequestState.Merged; o.MergeRequestsIds.Add(mergeRequestId); });
Such call returns all merged requests for given project, not just the one specified in the options property.
Desired output:
Only merge request with given id is returned.
I have also looked into the code and I think I know where the problem lays:
The two query builders (generic & project) are incorrectly switched in GetAsync methods in MergeRequestsClient.cs
https://github.com/nmklotas/GitLabApiClient/blob/bef459a658c3a7d9a84ec6a0be66880cc23b4c2d/src/GitLabApiClient/MergeRequestsClient.cs#L63 this method is for generic options but uses _projectMergeRequestsQueryBuilder
https://github.com/nmklotas/GitLabApiClient/blob/bef459a658c3a7d9a84ec6a0be66880cc23b4c2d/src/GitLabApiClient/MergeRequestsClient.cs#L46 this method is for project options but uses generic _mergeRequestsQueryBuilder

Thanks for bringing this up!

Thanks, I've merged your fix into 1.0.3.


[WIP] Initial implementation of implicit broadcasting for eltwise ops

Good stuff! In the comments are a couple of cosmetic gripes. A few additional things I think we should do (ideally before merging, but I could be persuaded to take a raincheck):

Expand support to all binops.
Add a few execution tests for binops with implicit broadcast. (Okay to disable in non-CPU/INTERPRETER BEs for now.)
(Likely consequence of 2) Update other backends to invoke.

Will remove WIP once i expand to all binops. We can run execution tests on other backends as well since they will invoke the ImplicitBroadcastElimination pass

Will remove WIP once i expand to all binops. We can run execution tests on other backends as well since they will invoke the ImplicitBroadcastElimination pass

SGTM. I'd actually meant to backspace the comment about updating other BEs, since it occurred to me as I was typing it that the manifest exists. But it might be worth doing in this PR as well.


Url's in the application helper should not be hardcoded

Url's in the application helper should not be hardcoded to "http". 24pullrequest is hosted on https but furthermore, we should use the request object to generate the url parts to protect the method should 24pullrequest ever change domains, or use multiple domain names.

example: https://github.com/24pullrequests/24pullrequests/blob/7d44eb505bcb197753629b0541e79b372d2eeb1c/app/helpers/application_helper.rb#L52

Agreed, we should definitely remove any hardcoding of domains 👍

As those helpers are used in mailers there will be no request to get protocol and domain. I propose to externalize those URLs into configuration setting (https://github.com/24pullrequests/24pullrequests/pull/2124) or try to rely on root_url.


Fix unchecked memory access for files

This PR takes care of two problems:

It uses Memory::(read|write)_bytes to guarantee that memory accesses are checked (Fixes: https://github.com/rust-lang/miri/issues/1007)
It removes the (get|remove)_handle_and methods which were a little bit cumbersome to use. In particular remove_handle_and, because we were using it to avoid borrowing issues before the Evaluator::memory field was public.

@RalfJung @oli-obk

When opening a PR that fixes an issue, please put Fixes: #xxx into the PR description so that the issue gets closed automatically when the PR gets merged.

:umbrella: The latest upstream changes (presumably #1044) made this pull request unmergeable. Please resolve the merge conflicts.

I'm capping count at the beginning of both functions and try_from(count).unwrap()ing everything.

Thanks!
@bors r+

:pushpin: Commit 4baef7120a934b79f87dbe5d09591d781c1c81f7 has been approved by RalfJung

:hourglass: Testing commit 4baef7120a934b79f87dbe5d09591d781c1c81f7 with merge 09b0a8a813d178233aff88d521ff0aa672ff6dbe...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 09b0a8a813d178233aff88d521ff0aa672ff6dbe to master...


Delta Table Diff - MVP design

Closes #4832

@nopcoder your concern is a very valid one.
I think that this design still answers it.
Plugins can be developed and used in a variety of ways with different services (Merge, Diff, Rebase...) by setting the correct configuration file fields:
diff: // service/functionality that uses the plugins
  delta:
    plugin: A
  csv:
    plugin: A
  iceberg:
    plugin: A
merge: // service/functionality that uses the plugins
  delta:
    plugin: A
rebase: // service/functionality that uses the plugins
  parquet:
    plugin: B
plugins:
  A: ~/.lakefs/plugins/A
  B: ~/.lakefs/plugins/B


2.1.0

Long running tests passed locally, moving ahead. There is one sync extensive vector test that fails on ci/cd, it appears to be releated to data that was not cleaned up during the aio tests. We may see that here. I'll address that issue as part of vec-373


Supplemental Policy Rules page is confusing

Create a new supplemental policy, proceed to the Policy Rules page of the wizard.  Is it the Base that was chosen? Or is it the supplemental template that the wizard uses?
Audit mode should not be configurable, but it allows you to toggle it
The options that are editable don't match the ones that are documented as configurable in a Supplemental. In fairness, other than audit mode, the ones you allow to be set are the only ones that are useful, but due to #1 and #2, I'm left confused overall https://learn.microsoft.com/en-us/windows/security/threat-protection/windows-defender-application-control/select-types-of-rules-to-create#table-1-windows-defender-application-control-policy---policy-rule-options

The state is inherited from the base. When you hover over the label of an unsupported supp rule the Wizard shows this message currently

This is a bug. will fix
I don't support modifying rules 5 or 7. At the time of development, EV signers was supported but it might be worth removing now too


Find All References, Got To, Peek Definition dont work in all projects

Find All References, Got To Definition, Peek Definition work not in all projects.
In some opened workspaces 'Find All References' works only for $variables, but not for functions, strings, etc.
I see only "Loading..." text and nothing more.
And 'Got To Definition', 'Peek Definition' dont work for 'variable_name' or 'function_name' in quotation marks or I see spinning wheel effect.
But in most projects it is OK.
Example where it doest work: https://downloads.wordpress.org/plugin/wordpress-seo.5.5.1.zip
I have no errors in Console and Output.
Tried on Windows and MAC OS, latest VS Code 1.16.1 and 1.17 insides
What additions info I need to provide?
And do I need to install php-language-server, or only PHP IntelliSense extension is enough for VS Code?

Same problem, and Workspace symbol search not working too.
And can not go to the include file.

Might link to the issue I keep encountering which I raised on VS Code Github: https://github.com/Microsoft/vscode/issues/24506

It doesn't work when html/javascript file mixed to php file. If it is a pure php file, it works well.

Same error here.
Now for some reason I can see the "Go To Definition" option when right click the function call.
But it always shows "No definition found for functionName"


Expand supported rouge versions

Describe the change
This PR increases the supported versions of rouge to be ">= 3.14", "< 4.2" as there are new 4.x versions of rouge.
Why are we doing this?
The changes allows projects to install this gem as well as versions 4.x of the rouge gem.
Benefits
Same as above.
Drawbacks
https://github.com/rouge-ruby/rouge/releases/tag/v4.0.0 dropped support for versions of Ruby < 2.7, so versions 2.7+ are only supported with rouge 4.x.
Requirements

[ ] Tests written & passing locally?
[x] Code style checked?
[x] Rebased with master branch?
[ ] Documentation updated?
[ ] Changelog updated? - I haven't updated this as I'm not sure if you want a new minor or patch version.

For additional context, we use https://docs.renovatebot.com/ruby/ to keep our Ruby gems up-to-date and it's automatically created an MR at https://gitlab.com/gitlab-org/gitlab/-/merge_requests/111543 which highlights the problem I'm hoping to fix here 🙂🙏
WDYT @piotrmurach?

Hi Ash 👋
Thanks for this PR. Agreed, let's relax the rouge constraint. This is a backwards-compatible change so we should be good to release as 0.7.2. In the changelog, please use this version but don't specify a release date. Instead, put unreleased as I'm not sure when I'll get around to releasing it.

I'm not sure what's happening with CI @piotrmurach, seems to be stuck?

Looking forward to 0.7.2 release with this change. :bow:  This will allow us to update rouge gem to the latest version to fix issue with git-diff rendering in GitLab.

@piotrmurach, is there anything we can assist with in getting this PR merged please? 🙏🙂

Thank you ❤

Released version 0.7.2.

Thanks @piotrmurach! 💜


Debug mode doesn't appear to be working

It appears that debug mode might not be working.  The easiest way to reproduce is load the imagenet example and run dl.ENV.set('DEBUG',true) in the console.  The debug timings that print out will all be less than a 1 ms (usually 0.1ms on chrome for me, 0ms in firefox).

Actually I'm seeing similar behavior on the benchmarks demo.  The CPU benchmarks are giving sane timings, but the GPU benchmarks are all fraction of a ms or faster (usually 0ms).

My platform has GL_EXT_timer_query disabled (Chrome on Linux).  It looks like before the timers were refactored the timing code worked without this extension, and now it's not timing the actual computation when the extension is unavailable.

I will take a look at this (I am preparing for a talk next Friday so unfortunately it will take me until the first week of April).
We just did a refactor of timing / debug mode, and I think that broke the timing when that flag is disabled as you noticed.

This issue was moved to tensorflow/tfjs#144


virtc: Fix virtc to make it properly usable

With recent changes brought to the Clear Containers runtime, virtc
was not really in sync with what we were currently testing. This
patch replaces the systemd target file to be run by the guest kernel
and it also enhances the handling of the hypervisor machine type.
By default, we want to rely on "pc" machine type (with the proper
uncompressed kernel associated), and through the CLI, we can choose
"pc-lite" which will be automatically associated to the compressed
kernel.

Coverage remained the same at 64.851% when pulling f4a2ad19324b1340d90395c5e44996b07243a50f on sboeuf/fix_virtc into aff9031e0f7827c18fb50a747d963783cb11c650 on master.

LGTM

lgtm


Using own dataset in microspeech project doesn't work

@tensorflow/micro
System information

Host OS Platform and Distribution (e.g., Linux Ubuntu 16.04): currently working with Google Colab
TensorFlow installed from (source or binary):
Tensorflow version (commit SHA if source): Tensorflow 1.x
Target platform (e.g. Arm Mbed OS, Arduino Nano 33 etc.): Arduino Nano 33 BLE Sense

Describe the problem
Hello everyone,
I just started to get into the world of machine learning and deep learning and I still need to learn a lot, but I hope that there might be someone who faced similar problems to the ones that I try to solve at the moment.
A few weeks ago, I started working with the microspeech example for microcontrollers. My goal was (and still is) to get the project working with an own wake word. To save time, I decided to expand the speech commands dataset with a folder containing my own data. My wanted wake word is a german one (I'm not sure if this is an important information but I try to tell as much information as possible at the moment).
At first, I only used samples of my own voice. To get more samples in a short time, I combined the original recordings with ones that were slightly manipulated.
After that, I uploaded my data to Google Drive so that I could import it to Colab. I changed the 'wanted word' section and the 'data_dir' section according to my wanted word.
Training seemed to work well and I changed the code in the microspeech project in the Arduino IDE and uploaded it to the Arduino. I expected the green LED to flash when I say the wake word and the blue one to do so if I say some other word of the dataset (as it is intended in the original project). But no matter what I said, only the blue one LED flashed.
In a second step, I changed my dataset because I thought that it might be a problem that my dataset only consists of recordings of my own voice. So I collected new data with recordings from different people and tried everything that I did before with my new data.
This time the arduino behaves a bit different as the green LED flashes randomly when nothing is said at all. And when I say my wake word, the blue LED is still flashing as it did before.
As I said at the beginning, I'm new to this topic and so I only can imagine some possible causes for my issues (please feel free to correct me if some things don't make sense):

I didn't collect enough samples of my wake word (at the moment there are 820 samples in the folder)
there might be a problem that I only want to detect one word instead of two or more
I forgot to change sth. in the Arduino project code or did sth. wrong
it could be a problem that my dataset is a mix of English and German words

Please provide the exact sequence of commands/steps when you ran into the problem
After training, I copy the code and paste it into the micro_features_tiny_conv_micro_features_model_data.cpp and I also correct the data length parameter at the end of the source file.
Then, I change micro_features_micro_model_settings.cpp and micro_features_micro_model_settings.h according to my wake word and the number of labels that I've got.
Finally I go to the arduino_command_responder.cpp and change the first if condition to if(found_command[0] == 'h'), as the wake word begins with an h. I erase the second if condition because I don't have a second wake word.
Then I upload the project to the arduino.
If there are any information missing please tell me and I will provide them.
Kind regards,
eeesi

Hi eeesi, the issue is the Tensorflow Lite library for Arduino is not compatible with the models produced by newly updated Google Collab code. A "fix" is to download the Tensorflow repository and run test_arduino.sh to create an up to date zip library to import into the Arduino IDE. I put fix in parentheses because while it does detect audio the inference runs too slow with the example micro speech code which causes issues. As of right now this seems to be an ongoing issue the Tensorflow Lite team has not addressed so I am glad more people are bringing it to attention here.

Anyone found a solution to this issue? I tried the same as @eeesi, but cannot get the Arduino code to work with an updated model trained using the Colab code


Documents showing locked by current user

Link to reproduction
No response
Environment Info
Binaries:
Node: 22.6.0
npm: 10.8.2
Yarn: 1.22.22
pnpm: 9.7.1
Relevant Packages:
payload: 3.0.0-beta.111
next: 15.0.0-canary.173
@payloadcms/db-mongodb: 3.0.0-beta.111
@payloadcms/email-nodemailer: 3.0.0-beta.111
@payloadcms/graphql: 3.0.0-beta.111
@payloadcms/live-preview: 3.0.0-beta.111
@payloadcms/live-preview-react: 3.0.0-beta.111
@payloadcms/next/utilities: 3.0.0-beta.111
@payloadcms/plugin-cloud: 3.0.0-beta.111
@payloadcms/plugin-cloud-storage: 3.0.0-beta.111
@payloadcms/plugin-form-builder: 3.0.0-beta.111
@payloadcms/plugin-nested-docs: 3.0.0-beta.111
@payloadcms/plugin-redirects: 3.0.0-beta.111
@payloadcms/plugin-seo: 3.0.0-beta.111
@payloadcms/richtext-lexical: 3.0.0-beta.111
@payloadcms/storage-vercel-blob: 3.0.0-beta.111
@payloadcms/translations: 3.0.0-beta.111
@payloadcms/ui/shared: 3.0.0-beta.111
react: 19.0.0-rc-3edc000d-20240926
react-dom: 19.0.0-rc-3edc000d-20240926
Operating System:
Platform: darwin
Arch: arm64
Version: Darwin Kernel Version 23.5.0: Wed May  1 20:14:38 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6020
Available memory (MB): 16384
Available CPU cores: 12
Describe the Bug
When opening documents, locked document warning message is showing, saying that the document is being edited by the current user. This is happening for all documents, regardless of actual locked status, and even for a brand new user who has never edited documents before.
Video of issue:
https://github.com/user-attachments/assets/63fbdd4d-aeab-488c-812f-39eeda231512
Reproduction Steps

Log into payloadcms.com/admin.
Navigate to document's edit view.
See modal that document is locked by you.

Adapters and Plugins
db-mongodb, storage-vercel-blob, richtext-lexical, plugin-seo, plugin-redirects, plugin-form-builder, email-nodemailer

I'm getting this as well, with postgres.

With SQLite (same environment) the lock documents from another user will stay locked indefinitely, until you 'take over'. Last test it stayed locked for 5 hours with no one logging in.

Hey @drewlonious and @mikecebul,
Thank you both for reporting this issue.
After further investigation, it appears that the problem with document locking may stem from a change introduced in v3.0.0-beta.108, where relationships are now stored as ObjectIDs rather than strings. Unfortunately there was a bug we missed for versions that made it so only documents saved on a couple of payload beta versions for about a week were having stale docs show twice in the list view. This issue has been addressed in the latest version (v3.0.0-beta.112).
To resolve this, I recommend saving a new version of each document that appears as a duplicate. This will clear out the problematic records. Additionally, you may need to manually delete any entries in the payload-locked-documents collection from your database to remove any incorrect locks on these duplicated documents.
We encountered the same issue in our website CMS and found that these steps resolved it. If you run into any challenges with this process or need further clarification, please feel free to reach out, and I’d be happy to assist. If the problem persists, we can open a new issue to investigate further.
@Mikecebul – Regarding the behavior you're experiencing: Currently, this is by design. The document edit view relies on an unmount function to remove the lock when a user navigates away. However, if a page refresh occurs or the user steps away without leaving the document, the unmount function doesn’t get triggered, and the lock remains.
We’re actively exploring alternative solutions to address this scenario, but for now, the Take Over option is provided to ensure that other users aren't blocked by stale locks. Additionally, the overrideLock option in the Local API can help manage updates on documents that remain locked. I hope this explanation helps clarify the situation.

Hey @drewlonious and @mikecebul,
Thank you both for reporting this issue.
After reviewing the issue, it seems that the initial document locking problem may be related to a recent change made in the MongoDB adapter, where relationships are now stored as ObjectIDs instead of strings. This caused a bug in certain beta versions (v3.0.0-beta.108 to around v3.0.0-beta.110), leading to stale documents appearing twice in the list view. This bug has since been resolved in the release (v3.0.0-beta.111).
However @drewlonious , since you mentioned that this issue also appeared in your PostgreSQL project, could you please provide additional reproduction steps or describe how you encountered the issue?
In the meantime, for those who encountered this with MongoDB, we recommend saving a new version of each document that appears as a duplicate. Additionally, clearing out any entries from the payload-locked-documents collection in your database should remove any stale locks caused by these duplicates.
We encountered the same issue in our website CMS and found that these steps resolved it. If you run into any issues with this process or need further clarification, please feel free to reach out, and I’d be happy to assist.
@Mikecebul – Regarding the behavior you're experiencing: Currently, this is by design. The document edit view relies on an unmount function to remove the lock when a user navigates away. However, if a page refresh occurs or the user steps away without leaving the document, the unmount function doesn’t get triggered, and the lock remains.
We’re actively exploring alternative solutions to address this scenario, but for now, the Take Over option is provided to ensure that other users aren't blocked by stale locks. Additionally, the overrideLock option in the Local API can help manage updates on documents that remain locked. I hope this explanation helps clarify the situation.

That makes sense, thanks for the explanation.

Fixed with #8744

After reviewing the issue, it seems that the initial document locking problem may be related to a recent change made in the MongoDB adapter, where relationships are now stored as ObjectIDs instead of strings. This caused a bug in certain beta versions (v3.0.0-beta.108 to around v3.0.0-beta.110), leading to stale documents appearing twice in the list view. This bug has since been resolved in the release (v3.0.0-beta.111).
However @drewlonious, since you mentioned that this issue also appeared in your Postgres project, could you please provide additional reproduction steps or describe how you encountered the issue?
In the meantime, for those who encountered this with MongoDB, we recommend saving a new version of each document that appears as a duplicate. Additionally, clearing out any entries from the payload-locked-documents collection in your database should remove any stale locks caused by these duplicates.
We encountered the same issue in our website CMS and found that these steps resolved it. If you run into any issues with this process or need further clarification, please feel free to reach out, and I’d be happy to assist.

I am using release v3.0.0-beta.116 with MongoDB (local). Document locks are added immediately after changing a value in the UI (i.e., on blur/change). The screenshot below appears and a document is added to payload-locked-documents.
Rolling back to 3.0.0-beta.107 resolves the issue.

After reviewing the issue, it seems that the initial document locking problem may be related to a recent change made in the MongoDB adapter, where relationships are now stored as ObjectIDs instead of strings. This caused a bug in certain beta versions (v3.0.0-beta.108 to around v3.0.0-beta.110), leading to stale documents appearing twice in the list view. This bug has since been resolved in the release (v3.0.0-beta.111).
However @drewlonious, since you mentioned that this issue also appeared in your Postgres project, could you please provide additional reproduction steps or describe how you encountered the issue?
In the meantime, for those who encountered this with MongoDB, we recommend saving a new version of each document that appears as a duplicate. Additionally, clearing out any entries from the payload-locked-documents collection in your database should remove any stale locks caused by these duplicates.
We encountered the same issue in our website CMS and found that these steps resolved it. If you run into any issues with this process or need further clarification, please feel free to reach out, and I’d be happy to assist.

@PatrikKozak I am using release v3.0.0-beta.116 with MongoDB (local). Document locks are added immediately after changing a value in the UI (i.e., on blur/change). The screenshot below appears and a document is added to payload-locked-documents.
Rolling back to 3.0.0-beta.107 resolves the issue.


Update default.py

Updated with f string.

Checklist

[x] No CHANGELOG update needed

[ ] No new tests needed

[ ] No documentation update needed

has been addressed in #1353


update deps

Summary

update several dependencies

looks weird the codeql job it get stuck and run forever, i will open a fresh pr to check if we can reproduce


Proposal: rename the nodejs/iojs-fr WG to nodejs/nodejs-fr

Dear members of the io.js French WG iojs-fr
It is proposed to rename this repo to:
nodejs/nodejs-fr
As part of the io.js and node.js convergence, node.js releases are now based on the codebase and user base of the converged node.js and io.js communities. A number of the resources and repositories have already been renamed.
In the absence of objections or other discussion, this rename will occur on or after Oct 28, 2015
However, active language working groups should take ownership of this issue and discuss/dispose of it according to their own processes.
For more information or to discuss the overall issue, please see and comment on https://github.com/nodejs/node/issues/2525
This statement was authorized by the Node.js TSC
Cc: @nodejs/iojs-fr

Sounds good !

Yep, I'm all for it also. :+1:

it is good for me.

@srl295 done.


Adding references to Gateway metrics

wg = WIS2-to-GTS Gateway
gw = GTS-to-WIS2 Gateway
See Issue #18

Now included the CSV files with the relevant metrics.


IS THERE A WAY TO REMOVE CURRENCY FROM OUTPUT?

When used with textfield ,the output I get is something like this:
USD20,000.
Is there a way to get a formatted currency without the currency, like this:
20,000.
?

Hello @Devlonoah !
You can use built-in method.
Check this out.
https://github.com/gtgalone/currency_text_input_formatter#with-built-in-methods
// Built-in Methods
print(formatter.getFormattedValue()); // $ 2,000
print(formatter.getUnformattedValue()); // 2000.00
print(formatter.format('2000')); // $ 2,000

thanks.


Changed self.prior to self.logprior and transformed into PEP8

I was trying to run the 3 examples given on the tutorial page; but they would process because self.prior does not exist.
So I edited the code to change self.prior to self.logprior, which does exist. I then realized that UniformPrior and GaussianPrior are actually "log priors" themselves.
While editing the code, to make it readable on my Sublime, I had to adjust for PEP8 compliance -- I use AutoPEP8 with Sublime, so a lot of it was automated.
I hope that this helps. The example now runs; but the results are not the same as on the docs pages. I think that the boundaries are mismatched.

hey @exowanderer, thanks! however, could you provide the exact error message that you get? on my end the examples run just fine.

Oh, I see now. The bug is actually in the __repr__ method where self.prior is referenced. So, it looks to me that the best course of action would be to just change self.prior to self.logprior in each __repr__. Thanks for the PEP8 adjustments, but could you do that in a separate PR so that this PR is more concise and easier to review?

Hi Mirca,
I agree that the self.prior -> self.logprior in the __repr__ methods was the global issue. It's actually been so long since I fixed this, that I will have to re-download your version and re-run it. No problem; I'll do it.
As for separating the PEP8 form the __repr__ method call:
I wish that I could say "yes" to separating the PEP8 vs __repr__ method fixes; but my Sublime Text does 90% the PEP8 for me. I could submit it with only the basic formatting. I edited the .format strings to be f-strings, in an effort to read the code after Sublime Text auto-PEP8 formatted it.


SVE Implementation for Level-1 BLAS Routines

We have optimized Level-1 BLAS routines (scal, swap, and rot) utilizing ARM SVE, resulting in significant performance enhancements in OpenBLAS on two variants of the A64FX—FUJITSU PRIMEHPC FX700 and the FUGAKU supercomputer. These optimizations achieved performance improvements ranging from 1.80x to 4x through effective code vectorization. This research has been accepted as a full paper and presented at the 28th Annual IEEE High Performance Extreme Computing (HPEC) Conference in September 2024, under the title "Optimization Strategies to Accelerate BLAS Operations with ARM SVE."

Hiya, have you tested the impact on Graviton 3/4?

Thank you very much for this revised PR, I'm looking forward to the HPEC2024 proceedings becoming available.
The CI results so far suggest that
(1)Apple Clang is once again being silly about ambiguous SVE intrinsics, probably requiring a few type casts for the arguments like in #4140
and
(2) the new SCAL kernels may need to handle the dummy2 argument that has recently been (ab)used to signal whether to propagate INF and NAN (not wanted for internal uses of SCAL, but now expected when SCAL gets called from user code - this is probably the cause of the failures in openblas_utest and openblas_utest_ext)

Thank You @martin-frbg  for suggestions. we will do the required modifications.

Thank you for the review. We have reviewed and implemented the changes as per your suggestions. The updated files, which include the modifications for the swap and rotate routines using SVE (Scalable Vector Extension), have been uploaded. please verify and revert.

Thank you very much. There is one SVE_WHILE in swap_kernel_sve.c that is missing the silly uint64_t casts for AppleClang which is currently killing all the Mac CI jobs (I know there is no Mac currently that does non-streaming SVE, but there may be a reason to support it in the future - the SVE kernels get pulled in through DYNAMIC_ARCH builds).
Could you please fix that ?

Have updated the swap_kernel_sve.c with proper cast of uint64_t.
Thank You.

Great, thank you. I assume it makes sense to merge this without the SCAL kernel you had originally planned (?), then we can assess its performance impact on other SVE targets (like the Neoverse-based cpus @Mousius  mentioned above) in more detail


Cannot find module 'better-sqlite3'

Hi,
I try to run npm start and I get every time Cannot find module 'better-sqlite3' error
I use node-v 16.17.0
any idea how I can fix this error?

Same question ?


Outdated Word databases

Ever since the New York Times updated their answer keys and possible input lists, the program may not work correctly with some words on certain days. this can be solved by digging through the source code on the website and extracting the new lists.

Dug through the NYT website source to find and update the word database lists


Merge

https://github.com/trustwallet/assets/actions/runs/3473299795/jobs/5805162657#step:2:1

@djrann13 We'd be happy to look into this for you
kindly file a support ticket below
https://forms.gle/7KbQyuY5NoxyrhNz5


[BUG] stream:error code='515' after entering pairing code on phone

Describe the bug
When entering pairing code on phone, I am getting a 515 code error and the connection closes.
To Reproduce
Baileys 6.4.0. Full reproduction code (just edit BOT_NUMBER):
#!/bin/env node
// Just a single file to help explore/debug the WA protocol and Baileys library
import Baileys, {
  makeWASocket,
  fetchLatestBaileysVersion,
  makeCacheableSignalKeyStore,
  makeInMemoryStore,
  useMultiFileAuthState,
  WAMessageContent,
  WAMessageKey,
  WAVersion,
  AuthenticationState,
  delay,
  isJidBroadcast, // proto,
} from "@whiskeysockets/baileys";
import { mkdir } from "fs/promises";
import path from "path";
import P from "pino";

// enter your phone here
const BOT_NUMBER = "14150001234";
const DATA_PATH = "./data/explore";

export function dbg(val: any) {
  console.dir(val, { depth: null });
}

const logger = P({
  level: "trace",
  transport: {
    target: "pino-pretty",
    options: {
      colorize: true,
    },
  },
});

type Store = ReturnType<typeof makeInMemoryStore>;

interface StartSockConfig {
  store: Store;
  version: WAVersion;
  auth: AuthenticationState;
  saveCreds?: () => Promise<void>;
}

const asyncNoop = async () => {};

// type Sock = ReturnType<typeof makeWASocket>;
async function startSock({
  store,
  version,
  auth,
  saveCreds = asyncNoop,
}: StartSockConfig) {
  const sock = makeWASocket({
    version,
    logger,
    printQRInTerminal: false,
    // using MD api instead
    mobile: false,
    // syncFullHistory: true,
    auth: {
      ...auth,
      /** caching makes the store faster to send/recv messages */
      keys: makeCacheableSignalKeyStore(auth.keys, logger),
    },
    // msgRetryCounterCache,
    generateHighQualityLinkPreview: true,
    shouldIgnoreJid: (jid) => isJidBroadcast(jid),
  });

  // store?.bind(sock.ev);

  // Pairing code for Web clients
  if (!sock.authState.creds.registered) {
    console.log(
      "Waiting 5s for socket to be ready, otherwise we get 428 Precondition Required error and the socket disconnects..."
    );
    await delay(5000);
    const phoneNumber = BOT_NUMBER;
    console.log({ phoneNumber });
    console.log("Requesting pairing code...");
    const code = await sock.requestPairingCode(phoneNumber);
    console.log(`Pairing code: ${code}`);
  }

  // the process function lets you process all events that just occurred
  // efficiently in a batch
  sock.ev.process(async (event) => {
    dbg({ event });
    // credentials updated -- save them
    if (event["creds.update"]) {
      await saveCreds();
    }
  });
}

export async function createBot() {
  // fetch latest version of WA Web
  const { version, isLatest } = await fetchLatestBaileysVersion();
  console.log(`using WA v${version.join(".")}, isLatest: ${isLatest}`);

  // the store maintains the data of the WA connection in memory
  // can be written out to a file & read from it
  const store = makeInMemoryStore({ logger });
  const storePath = path.join(DATA_PATH, "store", "store_multi.json");
  // creates path if it doesn't exist
  await mkdir(path.dirname(storePath), { recursive: true });
  store.readFromFile(storePath);
  // save every 10s
  setInterval(() => {
    store.writeToFile(storePath);
  }, 10_000);

  // Load auth state if previously saved
  const { state: auth, saveCreds } = await useMultiFileAuthState(
    path.join(DATA_PATH, "auth")
  );
  // console.log("Loaded auth:");
  // dbg({ auth });

  // start a connection

  return await startSock({ store, version, auth, saveCreds });
}

await createBot();

Expected behavior
There should be no error.
Environment (please complete the following information):

Is this on a server? On my macbook air.
What do your connectOptions look like? See reproduction code.
Do you have multiple clients on the same IP? Just my WhatsApp for Mac but connected to a different number.
Are you using a proxy? No.

Additional context
See reproduction code.

retry with
// Pairing code for Web clients
  if (!sock.authState.creds.registered) {
    const phoneNumber = BOT_NUMBER;
    console.log({ phoneNumber });

    await sock.waitForConnectionUpdate((update) => !!update.qr)

    console.log("Requesting pairing code...");
    const code = await sock.requestPairingCode(phoneNumber);
    console.log(`Pairing code: ${code}`);
  }

Oops, never mind. I just deleted my auth state, "logged out" the device form my phone and tried again. Same error as before.

Oops, never mind. I just deleted my auth state, "logged out" the device form my phone and tried again. Same error as before.
edit: It does avoid the 5s delay hack I had before though, but doesn't solve the issue.

it is possible that you share the error that you are presenting

You need to implement reconnect logic because when the connection is successfully to log in the first time, you need to restart the connection


Help discovering methods and properties.

First some thanks
Micro is the best thing since XTreeGold (sorry showing my age) and be assured this is intended as high praise. Many thanks for sharing all your hard work.
I appreciate the simple UI - just "keybindings + commands + shell access",  combined with the strength of a proper programming language for development.
Then some problems and suggestions
I am exploring what can be done using Lua functions, by reading the Golang source and trying to decipher the available methods and properties of the go objects exposed in Lua. Any pointers to additional current documentation would be welcome.
Q1. Is there any way to make these objects list their methods and properties?
Q2. Would you consider a micro.RunCommand([string]) function?

The inbuilt "commands" used for key bindings would be very useful in Lua.
Especially with the parsing of multiple commands as described in keybindings.md.
Perhaps returning the name of the last command executed?

Q3. How can I set focus in a specific pane in a tab?

I can generate a table of tabs and panes, with id numbers and descriptions (thanks: https://github.com/zyedidia/micro/issues/1993#issuecomment-760271909)
Using bp.TabSwitchCmd(bp,[num]) I can focus in a specific tab
But how to select a specific pane?

Q4. How can I access the text within a buffer pane from Lua?

I want to process the whole contents of the current buffer (not the disc copy).
I don't want to mess with the current cursor selection(s).

Kind Regards Gavin Holt
Specifications
OS: Windows 10
Terminal: Console - (not Legacy, not Terminal App)
Version: 2.0.8
Commit hash: cfcb2e45
Compiled on October 06, 2020

Hi,
TL;DR?
I will separate these out and re-post.
Kind Regards Gavin Holt

Something like Q2 would probably be nice for a REPL even if it wasn't a Lua REPL.
(xlink https://github.com/zyedidia/micro/issues/1992 )


MediaCodecAudioRenderer error

File won't load because of this error.
It seems to mostly affect my movies in AAC
The only solution i found is to disable tunnel mode, but the image Quality is reduced and there is a slight zoom issue.
Using Xiaomi Mi TV Q1 75.
Thanks.

Unfortunately these are reportedly device specific bugs as stated by ExoPlayer devs and there's nothing to do with that except not to enable tunneling: https://github.com/google/ExoPlayer/issues/9133


week view

provide a week view for ikhal or khal
see https://github.com/geier/khal/issues/65#issuecomment-132464366 and https://github.com/geier/khal/issues/148#issuecomment-133052856
pull requests are very welcome

Might I suggest a "help-wanted" label for issues like this?

Thanks for that suggestion.

How would you imagine a week view for khal? Similar to khal agenda/caledar --days 7 but for monday to sunday or something different?

@libre-man There is a screenshot here. I imagine tiling the screenshot's daily pane across horizontally seven times. The key for me is to have proportional times (unlike the agenda view), as per the other thread. Monday to Sunday makes most sense to me, but I think some people consider a "week" to be Sunday to Saturday, so perhaps a config is necessary.

@protist Ok I got a clear picture of it now. Proportional times would be nice to
have in more functions than just a weekview. The agenda function would benefit
from it too. So it could be an idea to make a function that parses events lists
and it could be used in the get_agenda function in controllers.py.
An other idea is to make a new function all together that implements the
proportional time and make --week a flag. However this would mean duplicate code
or a bit of reshuffling of code. But I think this wouldn't be such a bad thing
as the get_agenda function has gotten quite long.
I'll try somethings soon to find out what is the easiest way.
On Wed, Oct 14, 2015 at 03:21:35PM -0700, protist wrote:

@libre-man There is a screenshot here. I imagine tiling the screenshot's daily pane across horizontally seven times. The key for me is to have proportional times (unlike the agenda view), as per the other thread. Monday to Sunday makes most sense to me, but I think some people consider a "week" to be Sunday to Saturday, so perhaps a config is necessary.

Reply to this email directly or view it on GitHub:
https://github.com/geier/khal/issues/251#issuecomment-148220016

@libre-man Great! Sounds good. Looking at it again, the screenshot was quite wide. i.e. on a 1920 x 1080 pixel screen, you could probably only fit ~4 days across. My Android calendar (Business Calendar 2) has the additional option of only displaying a user-specified number of days across, instead of a full seven. I'm unsure if this is something that is useful, and/or would fit in better with your second suggestion.
To fit in a full week, each day would have to be narrower, but then you might run out of room for event text. Hence, you could have multiple rows for each hour, and display only a fixed set of hours. In fact, looking at the wyrd screenshot again, I've just noticed that it is arranged as a continuum rather than a whole day, i.e. 23:59 on Monday immediately leads into 0:00 on Tuesday. This is not good IMO. Instead, I think it's best to do what Thunderbird Lightning and Korganiser do, i.e. show a user-specified fixed number of hours (e.g. 16), defaulting to a user-specified range (e.g. 8:00–24:00), but also allow vertical scrolling, up to the limits of 0:00 and 23:59. The presence of any events outside the displayed range could be indicated with a small icon on that edge.
FWIW on my monitor (1920 x 1080), I have 50–60 rows of text, so something like 3 rows per hour would work.

@libre-man I'd love to see a PR on this, do not hesitate to restructure the code, also if in doubt, send a PR early, so we can discuss the details

Ok I will work on this. I think it would be nice to have a way to specify if the days should be vertically aligned or horizontally, maybe using user input or dynamically based on the screen and the amount of events. I have busy week coming up, so I think I should have a working example in about two or three weeks.

@libre-man Huh… the horizontal alignment sounds really interesting. I haven't seen that in a calendar before, but it may work really well for a text/console-based calendar. Good luck and thanks for working on this!

I've been working on this a bit and I don't know what format is preferred the wyrd style or more Google Calendar type of style. With wyrd being a large block for the first event with chunks out of it for later events. And Google style with a separate bar for each event and this bar shifts to the right (or the left) if there is an event that start earlier and is still running.
I think the Google style is more logical because each event has an own bar because it is its own entity and has nothing to logically with the other events. However it is not space efficient and with more than three events at the same time wrapping in those small columns could become a real problem. What is your opinion?

I like wyrd's interface more; certainly for multi-user calendars, but also for single-user calendars. There's no reason a "vacation" or "out of office" entry on my work calendar should halve the space available in my personal calendar for the details of the time slot (an indentation as wyrd uses is much better).
And even on the web, the squeezing of events makes them unreadable (and with enough shown, effectively unclickable too), especially when showing calendars of 3+ people who overlap significantly. It'd be nice if they merged the same event which multiple people were all attending into one block, but Google has probably blacklisted the "crazy" ideas I keep sending them (like an NNTP gateway for Groups).

I'm good with whatever you think is best if you want to implement it...
Quoting libre-man (2015-10-18 20:40:48)

I've been working on this a bit and I don't know what format is preferred the wyrd style or more Google Calendar type of style. With wyrd being a large block for the first event with chunks out of it for later events. And Google style with a separate bar for each event and this bar shifts to the right (or the left) if there is an event that start earlier and is still running.
I think the Google style is more logical because each event has an own bar because it is its own entity and has nothing to logically with the other events. However it is not space efficient and with more than three events at the same time wrapping in those small columns could become a real problem. What is your opinion?

Reply to this email directly or view it on GitHub:
https://github.com/geier/khal/issues/251#issuecomment-149037598

I agree with all you say, @libre-man. From screenshots, Google calendar seems to partially overlap events, often hiding text behind other events, whereas Thunderbird Lightning totally separates events horizontally. I like the cleanliness of the latter, but I suspect that in a console setting space would be more important, and perhaps the wyrd version would be more appropriate, as @mathstuf suggests.

I added a pull request that at least provides a week view for calendar/agenda (like --days 7, but always starting on monday).

see #266 for the ikhal part


Page Layout Alias in mapping file

Category
[ ] Bug
[X] Enhancement
Problem Area
Page Layout Mapping
Expected or Desired Behavior
I would like the option to have a mapping, with an "alias" that if there are other page layouts that are almost identical, the mapping can apply to those as well, I didn't have to maintain two or more near exact mappings.
The mapping attribute could be called "AlsoAppliesTo" - then the names, comma delimited of the page layouts to apply the mapping to.
Observed Behavior
Each mapping is separate. It did lead to some maintenance issues were I had to ensure all the mappings were updated when a change was made.

That makes sense. Will look into how we best can do this.

@pkbullock : implemented via the AlsoAppliesTo optional attribute. You can specify a ; delimited list. See https://github.com/SharePoint/sp-dev-modernization/blob/dev/Tools/SharePoint.Modernization/SharePointPnP.Modernization.Framework.Tests/Transform/Publishing/custompagelayoutmapping.xml for an example. Docs will follow as part of the release process


Unicode in Zotero-imported strings is doubly-encoded utf-8

I am returning to an issue I have raised earlier and could not find a solution for.
It concerns Unicode in items that are retrieved from Zotero. Characters above #127 are not encoded correctly as utf-8, but as an utf-8 encoding of the individual byte components of the correct utf-8 characters. In this way, a character that would normally be encoded with 2 bits has 4 bits (see example below).
In most but not all cases, this can be reverted by converting the strings from utf-8, taking the result as consisting of 8-bit characters once more forming utf-8. However, even this is bound to fail in certain cases, making the regular Zotero import basically unusable for working with literature other than English (and Latin).
The issue is easily reproduced using citeproc-java.bat to generate a bibliography from Zotero, e.g.:
.\citeproc-java.bat --output "bib.txt" zotero bibliography
Any accented vowel or German Umlaut is concerned, as are entries containing Greek, Arabic etc.
For instance, one of my entries reads in notpad++, which correctly interprets the file as utf-8:

[19]E. PÃ¶hlmann, DenkmÃ¤ler altgriechischer Musik: Sammlung, Ãœbertragung und ErlÃ¤uterung aller Fragmente und FÃ¤lschungen. NÃ¼rnberg: Carl, 1970.

while it should read:
[19]E. Pöhlmann, Denkmäler altgriechischer Musik: Sammlung, Übertragung und Erläuterung aller Fragmente und Fälschungen. Nürnberg: Carl, 1970.
On closer inspection (e.g. changing the display mode from utf-8 to Ansi, or using a hex editor), the entry actually reads like this:

[19]E. PÃƒÂ¶hlmann, DenkmÃƒÂ¤ler altgriechischer Musik: Sammlung, ÃƒÅ“bertragung und ErlÃƒÂ¤uterung aller Fragmente und FÃƒÂ¤lschungen. NÃƒÂ¼rnberg: Carl, 1970.

So far I have no idea where this double-encoding may happen...
All best,
Stefan
PS: I should add that this does not concern Unicode characters that are part of the csl style - only text imported from Zotero.

I suppose this is fixed with #52. If not, just let me know.

Yes, I hope this fixed it. Thanks for merging!


Delete setlocal textwidth and setlocal colorcolumn

As we already have setlocal formatoption +=l to prevent longline break, it is not necessary to setlocal textwidth=0 as it doesn't matter what value tw is.
Even though preset textwidth is not necessary, setlocal tw=0 here may create conflict with setting up a default value of textwidth in .vimrc, especially when when we autoload LongLines with autocmd. For example, people may have a function to toggle ColorColumn at +1, which is not possible without a textwidth value.
A similar rationale is also applied for deleting setlocal colorcolum=.

This was an issue that was brought up when I first submitted this plugin for review in /r/vim, and I'm not averse to merging this change.  I also agree that one must be very conservative while changing options, but could you describe a situation where you'd want to use both long soft-wrapped lines and use an option such as colorcolumn?

For example, I may want to have a fullscreen window, but all the text still wrapped at textwidth = 80. I just want to use the LongLines plugin as an easy setup for navigation wrapped long lines.
On Oct 18, 2019, 16:27 +0700, manu-mannattil<EMAIL_ADDRESS>wrote:

This was an issue that was brought up when I first submitted this plugin for review in /r/vim, and I'm not averse to merging this change. I also agree that one must be very conservative while changing options, but could you describe a situation where you'd want to use both long soft-wrapped lines and use an option such as colorcolumn?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Oh, it’s my bad. It’s not possible to do that.
On Oct 18, 2019, 16:34 +0700, Hieu Cao<EMAIL_ADDRESS>wrote:

For example, I may want to have a fullscreen window, but all the text still wrapped at textwidth = 80. I just want to use the LongLines plugin as an easy setup for navigation wrapped long lines.
On Oct 18, 2019, 16:27 +0700, manu-mannattil<EMAIL_ADDRESS>wrote:

This was an issue that was brought up when I first submitted this plugin for review in /r/vim, and I'm not averse to merging this change. I also agree that one must be very conservative while changing options, but could you describe a situation where you'd want to use both long soft-wrapped lines and use an option such as colorcolumn?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

From what I understand, you want to use this plugin for navigation alone, and want to hardwrap text to 80 columns by default.  But the longline mode disables hardwrapping text by default, and any new text inserted would only be softwrapped.  The textwidth option (even if it's set) would essentially be ignored in that case.  Yes, setting textwidth is the only way to use the relative /+/- form of colorcolumn, but I fail to see how colorcolumn would be useful if you aren't hardwrapping text.  Also, any softwrapped text is overlayed over the color column, so it would look broken.

Yes, my intention is to use the plugin to navigate only. It’s handy whenever I resize my tmux panel to be too narrow horizontally.
I disabled my colorcolumn and turn on LongLines by default, but sometimes, when Syntastic check pep8 for python and show long columns, I would sometimes want to quickly toggle on the colorcolumn.
On Oct 18, 2019, 16:56 +0700, manu-mannattil<EMAIL_ADDRESS>wrote:

From what I understand, you want to use this plugin for navigation alone, and want to hardwrap text to 80 columns by default. But the longline mode disables hardwrapping text by default, and any new text inserted would only be softwrapped. The textwidth option (even if it's set) would essentially be ignored in that case. Yes, setting textwidth is the only way to use the relative /+/- form of colorcolumn, but I fail to see how colorcolumn would be useful if you aren't hardwrapping text. Also, any softwrapped text is overlayed over the color column, so it would look broken.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Ah, now I understand your use case.  If you want to use longlines for navigation alone, I would suggest commenting out the lines that change the options that affect how text is wrapped in the s:longlines_on() and the s:longlines_off() functions.  Just removing textwidth and colorcolumn isn't going to be of much use.
It is also possible to refactor the code and create a new command, say :LongLinesKeys, that would change the keybindings without interfering with the existing behavior of the :LongLines command.  If you think that such a command would be useful, feel free to refactor the code.

Thank you for your understanding.
I'm not confident with vim script yet. For now, I would just disable the options of set textwidth and colorcolumn of longlines. https://github.com/manu-mannattil/vim-longlines/commit/96f97e73ecd035a97597b13a1ac10f3f00ee539f
If you find the changes make the plugin more flexible for users, please accept pull.
I would learn more about the vim script and may refactor the codes later.

I prefer not to alter the current behavior of longlines.  However, I've defined a new command :LongLinesKeys (see the branch keys) that does exactly what you want—change the keybindings without changing any other option.  If you find it useful, I can merge it into the main branch, and close this pull request.

It sounds wonderful! Thank you so much. I really appreciate your work.
On Oct 19, 2019, 08:47 +0700, manu-mannattil<EMAIL_ADDRESS>wrote:

I prefer not to alter the current behavior of longlines. However, I've defined a new command :LongLinesKeys (see the branch keys) that does exactly what you want—change the keybindings without changing any other option. If you find it useful, I can merge it into the main branch, and close this pull request.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Change Ratio Token Name

Hi, Team!
I changed Ratio Token Name.
From 'Ratio Protocol' to 'Ratio Governance Token'
PR => https://github.com/solana-labs/token-list/pull/22857
Thanks in advance!

Merged


Correct jobDslTest task name

Task name doesn't have a s suffix:
Task 'jobDslTests' not found in root project 'jenkins-dsl'. Some candidates are: 'jobDslTest'

Thank you :)


Silent option

1st: Great work. Looks awesome 👍
I was thinking about including this in a pre-push hook, and using the return code to prevent the hook if the coverage percentage is below the given threshold.
In this use case it'd be handy to have a silent switch, which only prints the covered %.

Is there an alternative way, @nakiostudio ?


Can you update the rust nightly every night?

We have a project that follows along rust nightly in a very rolling fashion and sometimes we depend on rust nightly being a certain minimum version. As such, it would be nice if docs.rs could also always have the respective newest rust nightly.

The failing build in question is this but the point is that this will reoccur often.

I am planning periodic nightly upgrades (need to update cargo first) but you should really consider using clippy as a optional dependency.

I believe this issue is fixed. Periodic rustc updates maybe a thing in future.


Could not resolve dependency with React 18

I tried to install mui-datatables. And I caught error:
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!   peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!   node_modules/@emotion/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!     peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR!     node_modules/mui-datatables
npm ERR!       mui-datatables@"" from the root project
npm ERR!     2 more (@mui/material, @emotion/styled)
npm ERR!   4 more (@mui/icons-material, @mui/material, @emotion/styled, react-dom)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^17.0.2" from<EMAIL_ADDRESS>npm ERR! node_modules/mui-datatables
npm ERR!   mui-datatables@"" from the root project

Tech
Version

Material-UI
5.5.2

MUI-datatables
4.2.2

React
18.0.0

same problem

same problem too

same problem too

same problem too

it workerd for me using npm install mui-datatables --save --legacy-peer-deps

same problem

npm install mui-datatables --save --legacy-peer-deps

Same problem for me, unfortunately.
@dayron9408 --legacy-peer-deps can potentially lead to inconsistent dependency tree:
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

Are there plans to upgrade to React 18?

I have the same problem. I'll use --legacy-peer-deps to start a project but I hope this issue will be solved in the future release.
https://ibb.co/SxGdzsq

Same issue, on deployment...

same problem

Same issue, are you planning a react 18 upgrade?

I've changed my version to 17.0.2 and that error has resolved.

Facing the same issue

Me too

same here

Facing same issue with React 18.
Any plan to upgrade the dependency tree of this package ?

same!

Same, please upgrade! <3

Bump ^^

running into the same issue

same issue. please upgrade

npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.

i need some solution

Hey! Facing same issue while using React 18, with mui-datatables: 4.2.2
using   npm install --legacy-peer-deps for now. Any plans to upgrade?

if anyone find this,  upgrade mui-datatables to 4.3.0 will be ok

if anyone find this, upgrade mui-datatables to 4.3.0 will be ok

nope i have mui-datatables: 4.3.0 with mui: 5.11.13
Same problem

yeah I have the same problem.. only noticed this when I tried to sandbox my app to figure out some of its problems.
does this only work with React 17.0.0 ~ 17.0.2?

Same problem

Still a problem, seems all MUI library not using React 18 not just the datatable. The table still work though, haven't see any issue beside installation so far.


standard tests layout

create a standard layout that formalizes how the test codes should be.

This task was done by @fabiojdf


Add cryptocurrency

This PR adds the possibility to convert a cryptocurrency to a regular currency almost similar to the currency script (which I used for all the boilerplate).
For now I added the top 10 cryptocurrencies as of today. I think I updated all necessary files to adhere to your repo structure.
If you have any remarks, feel free to comment.
Cheers, Jonas

I love the idea!
A few things:

It might make more sense to integrate this into currency rather than a standalone tool
The user should be able to convert from standard currencies to cryptocurrencies (two way conversion)
pcregrep needs to be replaced with grep -Eo as pcregrep is not standard on all systems (including mine)
By using the existing currency converter you should be able to convert cryptocurrencies to more standard currencies. Process would be convert cryptocurrency to USD and then use currency to convert USD to whatever the true target currency is. This would also work for converting to cryptocurrencies by first converting the base currency to USD and then converting that to the target cryptocurrency.

@alexanderepstein Thanks you for your remarks. Of course I was thinking about integrating this functionality into currency but decided against it.

The primary reason to create an own script is that I'm planning on extending the module to also list i.e. the top10 currencies, see relative change in rank and stuff like this which is exclusive to and of particular interest only for cryptocurrencies.

I'm happy to replace pcregrep with grep -Eo but I haven't found a possibility to use only the second matching group with regular grep.

I'll get rid of story.bash, sparrow.json and sparrow.md

That makes total sense, if that is the plan this can stay as a standalone tool.
You should be able to use grep -Eo and by setting the variable as an array and the line after set the variable to the second index of that array ex. If you have a text file that looks like this:
hello
hel

then you could use grep like this
variable=($(cat text.txt | grep -Eo hel[A-z]*)) # notice the extra parenthesis this says create an array
variable=${variable[1]} ## grabs the second result
echo $variable # this should print out hel

I also changed the readme so that all of the tools are in alphabetical order (this is one of the reason you see conflicts)
I also changed the installer so that this could be added to homebrew core. If you copy paste what you added to the readme here then merge with the master branch to get rid of the conflicts (don't use any of your changes to these two files) and i will add back what you changed on the readme after it has been merged and i will add cryptocurrencies to the install

I never got your feedback on two way conversion, what do you think?


adds option to automatically revert backticks (again) and fixes some bugs related to it

related issues: #47 #43

In case this gets merged, then https://github.com/meganrogge/template-string-converter/issues/44 can be reopened too

In reviewing whether this should be merged or not, some things to check are:

the multiple issues reported in https://github.com/meganrogge/template-string-converter/issues/43
the critical issue in https://github.com/meganrogge/template-string-converter/issues/47

Thanks for resolving the issues @xNocken and for the bug reporting @karlhorky.

So this has been released in 0.4.9 right?
Just taking a look at the extension page, it seems like this option is not documented.
@xNocken could you add this to the README.md, maybe with a gif illustrating the feature?

Done @karlhorky

Thanks!! I'll have to re-enable this and give it a try extensively.

Ok, so it seems this feature is still somewhat buggy:
It erroneously makes the following into a double-quoted string, when hitting backspace inside of it.
Notice there are no interpolations. It's important that it stay as a tagged template string.
export async function down(sql: Sql<{}>) {
  await sql`DROP TABLE reset_password_requests`;
}

I'm guessing the logic for this feature is:

If the user is inside a template string AND hits backspace:

switch to double quotes

This is maybe almost ok, but should probably do some better checks:

If the user is inside a template string AND there is an interpolation (${ and } present in the string) AND there is NO interpolation AFTER hitting backspace:

switch to double quotes

That seems like it would be a much more robust start of an implementation (probably also has weird edge cases, but at least it's much better than what's there now).


Update azure-monitor-agent-network-configuration.md

corrected a typo - much to must

Looks good. Thanks @Tungsten66


Support PEP 448 syntax

PEP 448 generalizes unpacking, for example allowing for code such as z = {**x, **y} to merge dictionaries.
Currently the parser gets all confused by this syntax.

We already have #337 tracking this, but thanks for the reminder. It wouldn't surprise me if {**x, **y} is a case we've missed since it looks like a set.

Ah, sorry for the noise, I messed up the search it seems.