Untranslated locales list is always empty

Description

When locales are classified into translated vs untranslated, the presence of the JSON file is used as the differentiating criterion.
https://github.com/WordPress/openverse/blob/0c2b85dc1e0c9e1017f7ebc46b7b51f314e1abf5/frontend/src/locales/scripts/get-validated-locales.js#L36-L39
But the translations JSON file is present even if the locale has no translations, it contains {}. See the following examples:

Afrikaans / af.json
Marathi / mr.json
Zulu / zul.json

This means that the untranslated-locales.json is always an empty list [].
Reproduction

Run pnpm i18n.
See untranslated_locales.json is empty	[].

Additional context
More importantly, it seems that this untranslated-locales.json file is not used anywhere so it can be removed entirely instead of fixing the problem.

This is probably for the best though, because if we dropped locales with 0 translations, we'd only have 51 left, which is about a quarter of the total locales in WordPress, and a fifth of the total locales GlotPress supports.


Discuss Orbit Discovery and Management

Metadata
What data define an orbit? Here are some ideas:

Patricia Merkle trie root hash of most current content updated by administrator
Patricia Merkle trie root hash of most current authorization policy log updated by administrator
List of IPv4/IPv6/etc. hosts that are in the orbit
What else?

Discovery
How do you find an orbit? Here are some ideas:
Public Orbits - meant to be discovered and and accessed by anyone

Smart contracts on public blockchains
DHTs that are publicly accessible a la bittorrent

Private Orbits - meant to be selectively disclosed

Smart contracts on private blockchains
Smart contracts on public blockchains with encrypted contents
Permissioned DHTs or distributed databases, possibility implemented using an Orbit
FTP server

Management
How are orbits administered? TBD
cc @chunningham

Re: Discovery, I've been thinking something like the following:

All Kepler instances share a public DHT using a public shared key to isolate traffic from the public IPFS network
In IPFS, the DHT describes providers of content addresses. In the Kepler orbit discovery DHT, nodes can query for providers of an orbit. this query returns one or more http endpoints (or a more generic address type)
on HTTP GET/POST requests, if the Kepler node does not host that orbit, it can query for a provider address and Redirect the request (redirect is preferable to returning the data via the nodes to prevent the non-host from seeing the content, and also because the non-host may not be able to verify the request authorization)
if the orbit is administered via smartcontract, the contract state can be queried instead of/in addition to the public DHT
this implies that to be discoverable, the orbit config must be public (or at least the set of host addresses must be verifiable against the orbit address)
does any of this make sense?

Re: metadata, the auth policy log commitment and admin/host lists are essential to include. A tree hash of contents would allow for explicitly checking consistency between hosts (something IPFS already does but that other storage backends may not (e.g. multiple separate S3 buckets operated by non-cooperative hosts)) so might be a good option, but it would require a much more regular state transition to make that check timely (basically introducing block/confirmation time). given a deletion capability tho it might be essential. A human-readable name might also be useful in the metadata.

@chunningham recording our conversation here for future design sessions.
Potential short-term orbit ID strategy: hash the client PKH concat w/"orbit secret" that the client picks. This could be static for something like tzProfiles, provided by the tzProfile frontend.
Orbit resolution architecture--kind of similar to DIDs. Here's one example of how it could work:

Every change to the Orbit State (latest merkleroot hash, hosts policies (who are the replicators?), content access policy, etc.) is a signed patch/CRDT/delta/etc. The patches can hold some kind of evidence of existence at a point in time, such as a blockhash from Bitcoin, Tezos, Ethereum, etc.
For our orbit type, it starts with a default state that the patches continue upon.
Patch state build with correct patches (signing was authorized correctly and data layer consistent) and the most recent evidence of existence wins.
We can access the list of patches from a KV query with the Orbit ID as the key. This could be public or permissioned depending on use case.

From our discussion, here are the roles for orbits:

Orbit Commander: Can
Read Delegator: Can delegate or revoke scoped capabilities across all Read operations.
Write Delegator: Can delegate or revoke scoped capabilities across all Write operations.
Writer: Can perform all Write operations.
Reader: Can perform all Read operations.

Here are rough metadata structures:

HostMap :: Map PublicKeyHash [IPAddress]
ContentMerkleRoot :: [u8]

Signed object capabilities can contain cryptoscript/ioscript as an unlocking mechanism, not dissimilar from bitcoin's Script. E.g., cryptoscript to allow a download of a JPG for an NFT owner:
FA2Asset -> AssetId -> VPSigner -> Bool
getOwner  -- consumes FA2Asset and AssetId, pushes TezosAddress to stack iff successful
asTezosAddress. -- consumes VPSigner, pushes TezosAddress to stack iff successful
assertEqual  -- consumers TezosAddress and TezosAddress, throws nothing iff successful

This could be embedded in something like a ZCap and the "VPSigner" could be equivalence checked to the NFT asset's owner.

Discovery about orbit discovery:
Here is the orbit string:
tz:${SALT}:${INDEX}:did:...
tz:${SALT}:KT1...

The DID included MUST resolve to a DID document with an authentication method using an Ed25519 public key, secp256k1 public key, or P-256 public key. A BLAKE2B (digest-size = 20) hash of that public key is also acceptable. Examples of conforming DID methods include did-tz and did-pkh(:tz). The orbit config lives at the ${INDEX}th smart contract
deployed which conforms to the Kepler Orbit TZIP (TBD).
If KT1 approach is taken, then the smart contract at that address MUST implement the Kepler Orbit TZIP (TBD).
Open questions

What if the DID's authn method is rotated and it's a new key?


cannot compile in Ubuntu 16.04

Hello,
the project seem to not be able to compile in ubuntu 16.04, using Swift 4.1
libressl cannot be installed.
swift build report several error around CLibreSSL

Hey! Make sure you obtain them from http://apt.orkhanalikhanov.com.
Also you need to update LD_LIBRARY_PATH to include libressl. I use following in travis builds:
env LD_LIBRARY_PATH='/usr/local/lib:/usr/local/opt/libressl/lib:$LD_LIBRARY_PATH' swift test

To install libressl following should work:
echo "deb [trusted=yes] http://apt.orkhanalikhanov.com ./" | sudo tee -a /etc/apt/sources.list
sudo apt-get update
sudo apt-get install libressl

Closing due to inactivity


[Feat] add contributor or team page at footer

Description
add contributor or team page at footer will look more proeffesional
Screenshots
No response
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I'm a GSSoC'24-Extd contributor
[X] I'm a Hacktoberfest'24 contributor
[X] I am willing to work on this issue (optional)

Hey @ananyag309 As of now, we do not need it. We will let you know in the future if there is a need. Please search for another issue and raise it, which I'm happy to assign to you.

please assign this issue

Hey @ananyag309 @BhavikaPachauri, if you see this, please let me know if you can do something like this. I’ll explain below.
I am also mentioning the top contributors on the GSSoC leaderboard as they asked me. Please let me know if there’s a need to contribute and also I'm mentioning who are genuinely working and have already created good pull requests merged on Ezyshop.

@shubhagarwal1
@ShivanshPlays
@inkerton
@kavya410004
@Bhum-ika
@Shariq2003
@IkkiOcean
@Subhajit-2023-44
@Sawan-Kushwah
@priyanshuverma-dev
@mehul-m-prajapati

Actually, we already have a contributor page in the footer, and it’s already implemented. Check out the implemented contributor page:

But I’m thinking of changing the UI and design of the contributor page. I have a design for the updated contributor page; please let me know if you can work on it. Below, I will attach the image of that design: 👇

[!NOTE]
You will receive a level 3 label for the GSSoC-Ext leaderboard points after the pull request for this issue is merged. However, please try to implement this within 1 to 2 days; otherwise, I will assign it to someone else and provide other opportunities according to the guidelines.

[!NOTE] 2
Don't add my profile. I mean, remove my profile from the contributor page. What I mean is, even though my profile will appear as the top contributor based on my commits, don’t include it—remove my profile and don’t fetch it from the GitHub contributors. Hope you understand.

Let me know if anyone is interested in working on this. It is on a first-come, first-served basis.

Thank you!

@mdazfar2 I can work on this. Assign me

@mdazfar2 I am also interested

@mdazfar2 sir, assign this to me

@mdazfar2 i am interested

Alright @IkkiOcean you are going to be assigned.
Make sure to implement the exact design I shared above; otherwise, if there is any issue with the UI or fetching, the PR will be closed.
Read our Contrubuting guidelines and contribute accordingly.
Try to create PR as soon as possible.

okk i start work on it
On Fri, 1 Nov 2024, 15:25 Md Azfar Alam, @.***> wrote:

Hey @ananyag309 https://github.com/ananyag309 @BhavikaPachauri
https://github.com/BhavikaPachauri, if you see this, please let me know
if you can do something like this. I’ll explain below.
I am also mentioning the top contributors on the GSSoC leaderboard as they
asked me. Please let me know if there’s a need to contribute and also I'm
mentioning who are genuinely working and have already created good pull
requests merged on Ezyshop.

@shubhagarwal1 https://github.com/shubhagarwal1
@ShivanshPlays https://github.com/ShivanshPlays
@inkerton https://github.com/inkerton
@Shariq2003 https://github.com/Shariq2003
@IkkiOcean https://github.com/IkkiOcean
@Subhajit-2023-44 https://github.com/Subhajit-2023-44
@Sawan-Kushwah https://github.com/Sawan-Kushwah
@priyanshuverma-dev https://github.com/priyanshuverma-dev
@mehul-m-prajapati https://github.com/mehul-m-prajapati

Actually, we already have a contributor page in the footer, and it’s
already implemented. Check out the implemented contributor page:
CE8E232A-4382-459F-BFCA-F41D32588704.png (view on web)
https://github.com/user-attachments/assets/96036f08-b29d-4334-a885-314db1d59b2e
But I’m thinking of changing the UI and design of the contributor page. I
have a design for the updated contributor page; please let me know if you
can work on it. Below, I will attach the image of that design: 👇
Copy.of.Meet.Our.Contributors.png (view on web)
https://github.com/user-attachments/assets/d44f3ad8-1aca-4e82-8e85-f3690077524e
Note
You will receive a level 3 label for the GSSoC-Ext leaderboard points
after the pull request for this issue is merged. However, please try to
implement this within 1 to 2 days; otherwise, I will assign it to someone
else and provide other opportunities according to the guidelines.
[!NOTE] 2
Don't add my profile. I mean, remove my profile from the contributor page.
What I mean is, even though my profile will appear as the top contributor
based on my commits, don’t include it—remove my profile and don’t fetch it
from the GitHub contributors. Hope you understand.

Let me know if anyone is interested in working on this. It is on a first-come,
first-served basis.

Thank you!
—
Reply to this email directly, view it on GitHub
https://github.com/mdazfar2/Ezyshop/issues/820#issuecomment-2451615421,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBQXYCLP3VP7ZGL3VNNBZ23Z6NFXRAVCNFSM6AAAAABQIPM7SKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJRGYYTKNBSGE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@mdazfar2 : thanks for tagging me.
Everyone can design this page, and then you can pick the best one for a grand total of 45 points!

No @mehul-m-prajapati, it’s not a good approach as it wastes other contributors' time for only one best PR, which isn’t ideal. Hope you understand.

@mdazfar2 Ah! I am late. Let me know if their is any other issue

@priyanshuverma-dev check the issue #978

@mdazfar2 Ah! I am late. Let me know if their is any other issue

@priyanshuverma-dev check the issue #978


Fix Notifications screen header on tablet

<ViewHeader> only had a desktop and mobile layout, so for tablets there was no side borders.
Before:

After:

Works well on iPadOS 16.6 👍

This adds double borders on Moderation. Let's audit existing usages of ViewHeader and fix them consistently?

FYI the Liked By and Reposted By pages also don't have side borders on tablet.


RedirectedStandardOutputHandler &  RedirectedStandardErrorHandler on ToolSettings

As discussed in #2460, we should expose RedirectedStandardOutputHandler and  RedirectedStandardErrorHandler on ToolSettings, which was just recently added to ProcessSettings in pr #2447.

I'd suggest that, instead of exposing the RedirectedStandardOutputHandler and RedirectedStandardErrorHandler from ProcessSettings, we extend ToolSettings by an handler that gets passed the stdout and stderror, but doesnt intercept. We could implement that using a intercepting ProcessSettings.RedirectedStandardOutputHandler from inside Tool. Possibly a ToolSettings.StandardOutputHandler and ToolSettings.StandardErrorHandler ?
That way we can grant access to stdout and stderror of a Tool without breaking the GetStandardOutput() some Tools use.
A downside to this would be that other Tool implementations could not use the ProcessSettings.RedirectedStandardOutputHandler.

Related discussion: https://github.com/cake-build/cake/discussions/3202

This came up a few time lately. Also related discussion: https://github.com/cake-build/cake/discussions/3130


Concat hostName and port for ldap_connect

Instead of using two parameters for ldap_connect (hostName and port), this PR combines them to a single parameter. The docs of ldap_connect describe the host parameter as either a hostName or a LDAP URI. In case the latter is used, the second parameter (port) is ignored. When using a protocol, the parameters are combined, otherwise it stays the same.
It works for me with ldaps now, did not test it without the protocol.
resolves #1220

Thanks @arkraft for offering this pull request.
We did already have a similar pull request open in #1278.
I've just commited c24764018a9684fcf4418b359ddd793bb380063b to update the logic as required & simplify the logic, while fixing this issue. Therefore will close this pull request.
This update will be in the next release (v0.26.0)


Support ClickHouse-JDBC-0.1.x

Please answer these questions before submitting pull request

Why submit this pull request?
New feature provided

Related issues

Bug fix

Bug description.

How to fix?

New feature or improvement

Describe the details and related test reports.

CI fails. :)

Any update? We need your integration test PR.

Is this pr still active? Do you have plan?
This pr is no update in half month

Closing as no update in 20 days.


Map attribution option

Fixes #67

thanks for this @npeihl.  i hate to be nitpicky, but we'll need to include a note in the documentation that explains that altering/omitting the geocoding attribution is only allowed if you are not using the World Geocoding Service.

@jgravois Is it enough to put that into documentation? I suppose we could build a check into the function that ignores custom text in mapAttribution and forces the default if the 'useArcgisWorldGeocoder' option is true.

thats not a bad idea.  the doc can explain that mapAttribution is optional and will override the default ifaof useArcgisWorldGeocoder is set to false.

@jgravois I've updated the PR and tested that it works. Let me know if you need anything else.

thanks for this @npeihl!


MMT didn't use all available TMX files due to different language codes

Hello MMT team,
I am creating an English>Simplified Chinese engine and I found out later that there were different TMX memories with different language codes.
EN>zh_CN vs. en-US>zh-CN
MMT picked the files with the second language codes only, but during the engine creation of my enzh engine, MMT displayed the following:

Engine: enzh
BILINGUAL CORPORA: 18 documents
LANGS: en > zh

So, I  assumed it picked up all 18 documents.
I only found out alter that it used only 5 memories that has this language code: en-US>zh-CN
My question is:
Either MMT should have warned me that there are different/mixed language codes and your engine will use the following 5 files only ......
Or at least, the number of files next to "BILINGUAL CORPORA:" should have been 5 only, not 18 and still it should throw a warning that not all memories will be used.
I think this could be an enhancement request. What do you think?
One last question:
Does this mean that if I put English Arabic files with different language codes, it won't pick all files. Say,
EN-US > AR-SA
EN-UK > AR-EG
etc...
In such scenario, which language combinations will be picked up? Or MMT will pick up all language codes as long as they are ISO-compliant?
Thanks in advance for your support!
Best regards,
Mohamed Zeid

Hi @mzeidhassan,
I apologize for the delayed answer.
As for the first question, MMT will only use segments that are both ISO compliant and that match the engine language direction.
If any of these two conditions are not matched, MMT will just skip the segment.
For parallel files (e.g. a pair of files like corpus.en and corpus.zh), MMT can immediately know how many files it is going to use, with no surprises at runtime.
This is possible because the language direction can be inferred from the files extension itself, and it is the same for all the segments in these files.
For TMX files, however, the situation is a bit different: the same TMX file can contain segments for different language directions, so some segments might be "valid" whereas some others might not be. In the worst case, which is the one you have experienced, a TMX can contain only invalid segments for the current training.
MMT has no way to know this before opening the TMX file and scanning it all.
Performing such a "full check" before starting the training, however, would not be acceptable efficiency-wise. So for the time being I don't think that MMT can tell you that it is using 5 files only instead of 18.
As for the second question:
EN-US and EN-UK are considered as subcategories of EN; likewise, AR-SA and AR-EG are considered as subcategories of AR.
So let's say that you have a TMX file with some EN-US > AR-SA segments and some EN-UK > AR-EG ones.
If you are training an EN->AR engine, both your EN-US > AR-SA and your EN-UK > AR-EG will be used.
Otherwise, if you are training, for instance, an EN-UK -> AR engine, only the segments with EN-UK source language will be employed (EN-US ones will be skipped).
The same goes for the target languages of course :)
I hope this was helpful!
Andrea

No problem, @AndRossi ! I thought you were busy with the new release :-)
Thanks for the info about subcategories of language variances. This is good to know.


Create blank layout on sub-page not working

Describe the bug
I have a plain sub-page with zero layout for print purposes. Now I upgraded to the latest SvelteKit version and followed the SvelteKit named layouts documentation, but I am not able to remove the root layout.
I tried

empty (<slot /> only) __layout-test.svelte and<EMAIL_ADDRESS>empty (<slot /> only)  __layout.svelte
empty (<slot /> only<EMAIL_ADDRESS>

Maybe the documentation is not quit clear on how to just get a blank layout.
Reproduction
Create SvelteKit demo app and try to reset /todos/index.svelte page to blank layout.
Logs
No response
System Info
System:
    OS: macOS 12.2.1
    CPU: (8) arm64 Apple M1
    Memory: 92.53 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 16.15.0 - /opt/homebrew/opt/node@16/bin/node
    npm: 8.5.5 - /opt/homebrew/opt/node@16/bin/npm
  Browsers:
    Chrome: 101.0.4951.64
    Firefox: 99.0.1
    Safari: 15.3
  npmPackages:
    @sveltejs/adapter-auto: next => 1.0.0-next.45
    @sveltejs/kit: next => 1.0.0-next.341
    svelte: ^3.46.0 => 3.48.0

Severity
blocking an upgrade
Additional Information
No response

This is working as intended: layouts inherit the parent folder's default layout if present.
In your example, move your layout resets from src/routes/todos/__layout-test.svelte into src/routes/__layout-test.svelte.

@mrkishi Thanks for your response!
This somehow feels strange to me. So far I could have an independent "module" in a routes sub-folder, but for a layout reset I need to create a file in the parent folder?
Are there plans to enable this within a sub-folder, enable a __layout-root.svelte within the "module" folder itself?

Layouts have always extended parent layouts, except for __layout.reset.svelte. But that old reset style can be replicated nearly identically with named layouts, you just have more control over it.
What you may be missing is the fact that named layouts in the root folder may be reused anywhere in the app. Taking that into account, you can recreate the old layout resets like so:
src/
└── routes/
    ├── __layout.svelte
    ├── __layout-reset.svelte        # an empty layout <slot/>
    ├── foo/
    │   ├── __layout.svelte
    │   └── {a,b,c}.svelte
    └── bar/
        ├──<EMAIL_ADDRESS>  # equivalent to the old __layout.reset.svelte
        └── {a,b,c}.svelte

Note that foo's layout extends the root's main layout, while bar's layout extends the empty reset layout. Each subfolder layout is still "self-contained," apart from being wrapped by their chosen parent.

I have a reproduction of a weird issue related to this topic. If you go to the StackBlitz repro listed bellow, and, once the application bootstraps, you append /todos in the embedded browser you should see the issue. Being that the<EMAIL_ADDRESS>file in the /todos directory uses the __layout-reset.svelte in the parent directory, I would think the todos page would be entirely unstyled, since that layout reset is just <slot />.
Repro<EMAIL_ADDRESS>You can see that the layout reset stripped away all of the components from the default layout, but that import '../app.css'; is hanging around. Is my assumption that everything the default layout imports should not be present when resetting the layout as demonstrated in the StackBlitz app incorrect?


Chore/re write auth tests

Codecov Report

Merging #77 (e909dc4) into main (7556b88) will increase coverage by 0.26%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #77      +/-   ##
==========================================
+ Coverage   72.06%   72.32%   +0.26%
==========================================
  Files          28       28
  Lines         981      983       +2
==========================================
+ Hits          707      711       +4
+ Misses        224      222       -2
  Partials       50       50

Impacted Files
Coverage Δ

api/auth/jwt.go
100.00% <100.00%> (+8.69%)
:arrow_up:

Continue to review full report at Codecov.

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


DBZ-215 Validating configuration to prevent that database history topic and schema change topic collide

https://issues.jboss.org/projects/DBZ/issues/DBZ-215

@rhauch Addressed your remarks, rebased and pushed. Thanks for reviewing!


Limits

This is optional in the spec:

At any time, the user agent MAY "remove excess cookies" from the
cookie store if the number of cookies sharing a domain field exceeds
some implementation-defined upper bound (such as 50 cookies).
At any time, the user agent MAY "remove excess cookies" from the
cookie store if the cookie store exceeds some predetermined upper
bound (such as 3000 cookies).
When the user agent removes excess cookies from the cookie store, the
user agent MUST evict cookies in the following priority order:

Expired cookies.

Cookies that share a domain field with more than a predetermined
number of other cookies.

All cookies.

If two cookies have the same removal priority, the user agent MUST
evict the cookie with the earliest last-access date first.
When "the current session is over" (as defined by the user agent),
the user agent MUST remove from the cookie store all cookies with the
persistent-flag set to false.

It's a good idea to have some limits to guard against DOS attacks. A malicious server could keep creating cookies until the client memory is exhausted.

There's also some guidance on the limit sizes in https://datatracker.ietf.org/doc/html/rfc6265#section-6.1.

This is essentially ready in a branch, but I want to make sure cookies from redirects are picked up with Req and this is blocked on https://github.com/wojtekmach/req/issues/361.


Set up EU/GDPR-compliant cookie warnings

What's your idea for a cool feature that would help you use OHM better.
We should have a per-user warning about Cookies on the site and persist that information somehwere.
Might be something we can snag from OSM.
Current workarounds
None
Additional info
This will require more detail.

I'm surprised we are not getting this from upstream OSM. Does OSM website not comply with GDPR? If so, s that because of a non-commercial exception or something similar? I know California's GDPR-inspired rules do not apply to orgs that never sell personal information, though many orgs are offer cookie selection features anyway, esp if they are unsure about third party services they might use (eg Google Analytics). OSM might truly have no such third party data harvesters.

We were recently discussing GDPR and other privacy laws related to our work at GreenInfo, and I am as surprised now as I was a year plus ago that openstreetmap.org doesn’t have some sort of GDPR compliance banner or alert, but the privacy policy does comply https://osmfoundation.org/wiki/GDPR_Privacy_Statement.
We did implement a similar Privacy Policy, https://www.openhistoricalmap.org/privacy-policy
But I'm still not sure why they don't provide an affordance for blocking cookies. Maybe that's not strictly required by the law?

Possibly relevant thread from the OSM forum involving a member of the Licensing Working Group who investigated GDPR compliance several years ago.


Roadmap: PHP does not support non-LTS releases

Summary
The developer of PHP has decided to not support non-LTS releases. Look at special implementation for Ubuntu 23 Lunar for installing PHP.
Description
Developer of PHP has decided to not support non-LTS releases, which means that the typical steps for installing PHP via apt will not work. PHP is possible to install, but will require an alternative set of instructions in order to be done properly.
Note
Ubuntu 23.10 will ship with PHP 8.2

Update
PHP 8.2 / 8.2-fpm fully tested with Ubuntu Lunar and fully working, added to Proteus Apt Repo along with automatic updating of the packages.


[5.5] Add a validator service provider for registering class based rules

Proposed fix for laravel/internals#717.
Adds a ValidatorServiceProvider to make it easy to register class based rules for the application.
Just add a ValidatorServiceProvider.php file to your app, have it loaded, and look something like this:
<?php

namespace App\Providers;

use Illuminate\Foundation\Support\Providers\ValidatorServiceProvider as ServiceProvider;

class ValidatorServiceProvider extends ServiceProvider
{
    /**
     * A list of custom rules to add to the validator.
     *
     * @var array
     */
    protected $rules = [
        'barcode' => \App\Rules\Barcode::class,
        'country' => \App\Rules\Country::class,
        'country_code' => \App\Rules\CountryCode::class,
    ];
}

If this looks good I'll send a PR over to docs.

This I like!

The rule instances are meant to just be passed directly to validators.

@taylorotwell I am skeptical of this decision.
As it stands, using a rule class made by the make:rule artisan command requires more boilerplate and requires a use of foreign-looking syntax in Request classes when compared to registering a rule by calling Validator::extend with a closure or static class method.
I suppose that it is unclear right now, based on the documentation and the conversation on GitHub, whether using rules that extend the Rule class is supposed to be the modern/preferred way to make custom rules going forward, or whether it's another tool thats useful for specific use cases.
If the former is true, it is strange that we cannot register these rules and use them using the same syntax as the build in rules, especially since the other ways to create custom rules allow you to do this--it is strange to remove functionality when rolling out an upgrade.
If its the latter, and the Rule class is intended to be a co-equal option to the closure/static class method options, then this needs to be more clearly communicated in the docs, both by placing the simpler methods of creating a custom validation rule first, a brief explanation of what use cases might benefit from using the new Rule class option, (e.g. instantiating classes with programmatically generated variables), and explicitly calling out the limitation that these rules are incompatible with Validator::extend and the string/pipe shorthand syntax.
However, even if the Rule class was not intended to supplant the other methods of creating custom validation rules, I think that it's a better for the developer to support registering rule classes because it minimizes the amount of code to be refactored when changing from using the string based syntax and the array/rule object syntax.


Added more keywords, built-in packages, etc

Thanks for your PR, but I was already working to improve the colorization by adding keywords, functions ...
So I compared my list with yours and completed it if necessary.
I hope I have not forgotten anything
Published in version 0.0.9

It seems your version 0.0.9 covers my suggestions (and more!), however I would appreciate that you also include the below match for user-defined packages. I realize there is no "standard" way to name packages, but many people use the _pkg suffix and so it would be useful to have this included in the syntax highlighting.
<dict>
  <key>match</key>
  <string>\w*_pkg\b</string>
  <key>name</key>
  <string>support.class.user-defined.oracle</string>
</dict>

I added a match for user-defined packages like your example. (see version 0.0.13)
For other customizations, I have also added an example, because it's a bit more complicated...


javascript radio buttons ... unresponsive on first click

Once a radio button is checked( clicked on ) ...     	console.log($('input: radio :checked').attr('id'));
doesn't change. But changes after the consecutive clicks.
A reproduction of the issue is @ https://jsfiddle.net/b9gbx069/1/
Check console for the irregular behavior  observed.

Hi @khannavidur!
You appear to have posted a live example (https://fiddle.jshell.net/b9gbx069/1/show/light/), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 53, column 5 thru column 70: Duplicate attribute value.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

Hi ... posted the old fiddle url .... https://jsfiddle.net/b9gbx069/2/ ...

@khannavidur In what browser and OS?

Chrome .... Ubuntu 14.0.4

Google Chrome Exact version - 46.0.2490.80 (64-bit)

X-Ref: #13850
Due to the way the event handling works, your handler needs to be at the document level if it wants the correct post-click checkedness. Example: https://jsfiddle.net/bp094393/7/
This bears documenting.

Thank you ... meantime I found a workaround with the label's ids ... but that was applicable in just my use case.

Bootstrap 3 is no longer being officially developed or supported.
All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!
<3,
@mdo and team


Adjust alevin to use splici index and USA mode

Closes #91. This PR changes some of the parameters in the alevin-fry and cellranger workflows being used for benchmarking.
For alevin-fry, I have done the following:

Combined the previous parameters for index_name and t2g to one index_type parameter to indicate if the workflow should be run using settings for the single-cell (cell) or single-nucleus (splici) samples. The default is set to cell and will then assign the index_name and corresponding tx2gene mapping file.
Added a parameter for the tx2gene_3col.tsv file needed to run Alevin-fry in USA mode. This can only be used if the index_name is set to splici and the resolution is cr-like. Otherwise, the 2 column tx2gene file is still used.
The unfiltered parameter has now been changed to params.filter = unfiltered as the default. If that is changed to knee then the --knee-distance flag is used to filter true barcodes during the generate-permit-list step. Based on our discussion of switching to only using the unfiltered version moving forward, I chose to set this as default and keep knee as an option just in case moving forward.

Originally, I was planning on using the splici index as the default index for all single-nuclei RNA-seq samples, but with the addition of testing the splici index on single-cell samples that became less straight forward. Instead, I included a map of index_names and tx2gene files based on the index_type (currently set as cell or splici) with the anticipation that if we do choose to use an index for single-cell vs. single-nuclei, we can use the seq_unit column of the metadata to set the index and tx2gene (similar to how the barcode list is assigned now).
For Cellranger, I set the default for single-nuclei samples to run with the pre-mRNA index using the --include-introns option, removing it as a parameter.
I also did think about adding in checks to make sure the input parameters were correct, specifically the params.filter and the params.resolution options, but didn't think that was necessary quite yet. Probably something we want to consider adding if we do choose alevin-fry as our permanent workflow.

Thanks for all the helpful comments! I went back through and I believe I have addressed all of your suggestions and comments now. The main edits I did were the following:

I reorganized the workflow in general so all parameters are at the top, followed by any of the maps or lists that we use, then the processes, and lastly the workflow.
I moved the index and tx2gene paths out of paramaters and into the workflow. Additionally, I omitted the use of a ternary operator to build the tx2gene_3col_path and am now using a separate if/else statement (or what is now being called the t2g_quant_path - I'll explain why I still think we need both of these below).
I removed using the ternary operator for the choice in filtering options during generate-permit-list and now use a map within that process to assign the flag and barcode file if applicable.

I think the main thing I am trying to understand is how the index and resolution options interact (or don't). It seems like if we use splici, we will always be using cr-like resolution? Or am I misunderstanding that? We might also want to use cr-like with other tools, and perhaps that would become our new default...

In regards to your confusion between use of cr-like and splici together, the only way that we can use the USA mode with single-cell RNA sequencing (and subsequently use the functions written by alevin-fry developers) is to use those two conditions simultaneously. However, we have been using the splici index with the full resolution and using our own functions to read in spliced + intronic counts and combining them specifically in the context of the single-nuclei samples. It was my understanding that if we are still testing cr-like vs. the full resolution (for both snRNA and scRNA) and the use of splici vs. cDNA for the single-cell samples we would want to have the option of being able to use any combination of those two. Once we have narrowed down a default, then we could combine it into one argument if we feel like that is necessary. I'm not sure I understand what you mean by also using cr-like with other tools?

I'm a little confused by the logic here. If we only use t2g_3col with the cr-like & splici settings, do we need to set t2g_3col_path to the other t2g file in the other cases?
Looking at this again, it seems like we could put all of this in t2g_path as we don't (by my quick look) use that at all.

So this isn't actually true. For the scenario that we are doing cr-like and splici, we need both the 2 column tx2gene file and the 3 column file. The 2 column file is being used for the first alevin process during alignment and no matter what the resolution or index, we will always need to use the 2 column file for the first step. The 3 column file is what is needed only in the context of cr-like and splici and only during the last alevin-fry quant step. Otherwise, that step requires the 2 column tx2gene mapping. To try and clear this up a little in the actual code, I renamed it to be the t2g_quant_path and it is only set to the 3 column file with cr-like and splici, otherwise it is equivalent to the t2g_path used in the alevin step. If you have other suggestions on how to deal with this I am all ears!

So this isn't actually true. For the scenario that we are doing cr-like and splici, we need both the 2 column tx2gene file and the 3 column file. The 2 column file is being used for the first alevin process during alignment and no matter what the resolution or index, we will always need to use the 2 column file for the first step. The 3 column file is what is needed only in the context of cr-like and splici and only during the last alevin-fry quant step. Otherwise, that step requires the 2 column tx2gene mapping. To try and clear this up a little in the actual code, I renamed it to be the t2g_quant_path and it is only set to the 3 column file with cr-like and splici, otherwise it is equivalent to the t2g_path used in the alevin step. If you have other suggestions on how to deal with this I am all ears!

Thanks for this explanation! That cleared it up in my head nicely, and the code changes I think help as well.

I went back to read the example more carefully, and my understanding is that this should probably be conditional on the tx2gene file rather than the resolution mode alone? If the tx2gene file is three columns, that is the trigger for USA mode (I missed that point before), which then requires the mtx output so we can read the matrix (now 3x times longer) "manually" rather than with tximport/tximeta. Since the use of 3col is contingent on cr-like and splici, I would expect that to be condition here.

You are very right about the mtx output being dependent on the use of the 3 column tx2gene mapping, since that is how it designates USA mode or not. I went ahead and defined the use of that parameter within the same if statement for use of the 3 column file to address that. Although it does sound like we can get rid of this in the future hopefully as I would also prefer to use tximport.


Compatibility with Mechanize

I cannot get to work webmock with Mechanize. Mechanize uses net-http-persistent internally, which is based on net-http, so it should be compatible.
Any thoughts or ideas why it's not working? Thanks!

The problem is that net-http-persistent monkey patches Net::HTTP, and WebMock does not monkey patch the monkey patch.
Someone will have to update WebMock to support net-http-persistent.

Thanks! Maybe I'll take a look...


Add a "view all related people" button to people section of subject pages

If there are more than 10 people associated with a subject, add a "view all related people" button that functions like the ones we use in the works section. This is related to issue #294.

@sbagg, Hmm, I noted in #294 that for the people, the querry doesn't seem to be working....
My quick review of the client seems to indicate the query is broken.

though the subject term seems to be extended into the query, the query is not properly responding.
The counts do not include the search term

Investigating further, we see that this searech results in this ES bad query:
{
  "offset": 0,
  "limit": 8,
  "sort": [
    {
      "hasContactInfo.familyName": "asc"
    }
  ],
  "filters": {
    "@type": {
      "type": "keyword",
      "op": "and",
      "value": [
        "experts:person"
      ]
    },
    "hasResearchArea": {
      "type": "keyword",
      "op": "and",
      "value": [
        "ucdrp:concept/FoR/09"
      ]
    }
  },
  "facets": {
    "@type": {
      "type": "facet"
    }
  }
}

Instead we need to use hasResearchArea.@id as in:
{
  "offset": 0,
  "limit": 8,
  "sort": [
    {
      "hasContactInfo.familyName": "asc"
    }
  ],
  "filters": {
    "@type": {
      "type": "keyword",
      "op": "and",
      "value": [
        "experts:person"
      ]
    },
    "hasResearchArea.@id": {
      "type": "keyword",
      "op": "and",
      "value": [
        "ucdrp:concept/FoR/09"
      ]
    }
  },
  "facets": {
    "@type": {
      "type": "facet"
    }
  }
}

Which works.

Looks good to me. Leaving to @wrenaria to sign off.

Functionally it appears to be working correctly.
Visually, I desire two changes:

I'd prefer about 5px more space between the X icon and the subject text so there's better visual separation. Right now it squishes into it almost like it is part of the word.
There's also extra white space above the blue block that shouldn't be there and should be removed (see search result for a correct example to compare to the "view all" subject results with extra white space below)


Reuse OpenstackClientImage instead of dedicated InstanceHaImage

This resolves the warning at "make bundle" time:
WARN[0001] warning: multiple related images with the same image ref, "infra-instance-ha-image-url-default", and different names found.The image will only be listed once with an empty name.It is recmmended to either remove the duplicate or use the exact same name.

while preserving the needed configmap:
[cifmw@haa-09 install_yamls]$ oc get cm infra-instanceha-config -o yaml
apiVersion: v1
data:
  instanceha-image: quay.io/podified-antelope-centos9/openstack-openstackclient@sha256:403c4f660f1b704d9ebbb9838c711ec6d20ca7183c63b2f224ad450f5ec37cf2
kind: ConfigMap
metadata:
  creationTimestamp: "2024-10-22T04:39:21Z"
  name: infra-instanceha-config
  namespace: openstack
  ownerReferences:
  - apiVersion: core.openstack.org/v1beta1
    blockOwnerDeletion: true
    controller: true
    kind: OpenStackControlPlane
    name: openstack-galera-network-isolation
    uid: cee79e70-639c-4974-acc5-5ec8f75ca299
  resourceVersion: "51877"
  uid: 8dd46abb-0cf1-414b-a3d7-6a3f34a9bb56

/cherry-pick 18.0-fr1


Mealz initialisation solution

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#5
#4  👈
#3
#2
#1
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ThibaultMiam and the rest of your teammates on  Graphite


New function sct_merge_images

This function merges multiple volumes into the same space. It is designed for labeled files.
It takes as input:

list_fname_src
fname_dest
fname_warp
fname_out

Maybe we need a list_fname_warp to register all fname_src from list_fname_src into the same space (eg Subject #10 space or Template space).


drop support for python3.6

Problem
Support for python3.6 ends on 23 Dec 2021.
Solution
This PR removes references to python3.6 from the repo.
Acceptance Criteria

Full tox run

Coverage remained the same at 98.073% when pulling ee0686a8dfae357a0e4bcdfeaa00eb776f68c92e on matthewhegarty:drop-python36 into 0d4340f02582e86d4515159becb045cad1d53ee0 on django-import-export:main.

Superseded by #1366


gitAndTools.git-remote-hg: 0.2-e716a9e1a9e460a45663694ba4e9e8894a8452b2 -> 1.0.0

Motivation for this change
The current version no longer works with our packaged Mecurial.
The original repository appears to no longer be maintained, but there is a maintained fork, which is now used for the Debian package.
Things done

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

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

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

Nobody seems to object, so I'm going to go ahead with this.


[examples] Need new camera example

The existing one is not working on VS2017 C++. It compiles, but camera works badly.

@DGMexe which one is not working? they work ok for me...

Not OP, but at least for me, the core_2d_camera example is not working.
The camera doesn't follow the player like in the web example.

The rotation seems to work

I tried with the release version and the current master branch
OS: Linux(Fedora 30)
G++ Version: 9.2.1

Not OP, but at least for me, the core_2d_camera example is not working.
The camera doesn't follow the player like in the web example.

The rotation seems to work

I tried with the release version and the current master branch
OS: Linux(Fedora 30)
G++ Version: 9.2.1
Same thing happens to me.
OS: Windows 8.1

@DGMexe are you using latest raylib version from github? Could you try with latest raylib and example from github? It was modified recently...

@DGMexe are you using latest raylib version from github? Could you try with latest raylib and example from github? It was modified recently...

I'm using raylib 2.5 probably from here https://raysan5.itch.io/raylib so I think it's the recent version. Maybe i got it wrong but I see this example on github and it was modified a month ago. "This example has been created using raylib 1.5 (www.raylib.com)"  Also I'm in C++, and it's in C, so (Vector) gives me an error so I use Vector instead.

@DGMexe latest raylib version is the 2.6-dev, it should be manually compiled from github.

Retested examples, they work ok for me with latest raylib from github.

@raysan5 i can't find the 2.6-dev branch / tag that you mentioned.

@pirey it's the master branch, no release yet.


unable to add documents on (cloud run + cloud storage) meilisearch

Disclaimer
I know this isn't an intended usecase😅, however would love to get it to work!
Describe the bug
I'm trying to deploy meilisearch on GCP Cloud Run and Cloud Storage using gcsfuse. I am able to get an instance url that loads meilisearch UI here and it seems to have successfully created files on the connected storage bucket

To Reproduce
Steps to reproduce the behavior:

deploy melisearch to Cloud Run using this  repo
this step appears to be successful

try to add a document using curl

curl \
  -X POST 'https://serverless-meilisearch-qtcobgiyyq-uc.a.run.app/indexes/movies/documents' \
  -H 'Content-Type: application/json' \
  -H 'Authorization: Bearer superSecret' \
  --data-binary '[
    {
      "id": 287947,
      "title": "Shazam",
      "poster": "https://image.tmdb.org/t/p/w1280/xnopI5Xtky18MPhK40cZAGAOVeV.jpg",
      "overview": "A boy is given the ability to become an adult superhero in times of need with a single magic word.",
      "release_date": "2019-03-23"
    }
  ]'

returns successful response of the pending task
{"taskUid":0,"indexUid":"movies","status":"enqueued","type":"documentAdditionOrUpdate","enqueuedAt":"2022-08-29T19:18:02.100185041Z"}

it also appears to have created the index

checking the task,

curl \
  -X GET 'https://https://serverless-meilisearch-qtcobgiyyq-uc.a.run.app/tasks/0' -H 'Authorization: Bearer superSecret'

it remains as pending for about a minute then starts returning the following error
{"uid":0,"indexUid":"movies","status":"failed","type":"documentAdditionOrUpdate","details":{"receivedDocuments":1,"indexedDocuments":0},"error":{"message":"An internal error has occurred. `failed to fill whole buffer`.","code":"internal","type":"internal","link":"https://docs.meilisearch.com/errors#internal"},"duration":"PT25.230456086S","enqueuedAt":"2022-08-29T19:18:02.100185041Z","startedAt":"2022-08-29T19:18:02.540759336Z","finishedAt":"2022-08-29T19:18:27.771215422Z"}

Expected behavior
expected to have successful document created
Screenshots
If applicable, add screenshots to help explain your problem.
Meilisearch version: v0.28.1
{"commitSha":"22aa349e31bc7662a065f4dc3229a93abd1f1f33","commitDate":"2022-07-21T11:08:37Z","pkgVersion":"0.28.1"}

Additional context
Additional information that may be relevant to the issue.
[e.g. architecture, device, OS, browser]

Hi @shamsmosowi,
As explained on the Slack thread. I am not sure this is a problem with Meilisearch, but we found that it could be related to an issue when reading something from an IO stream (file or network), as discussed in this thread.
Hope it helps. Many thanks 🙂


The code

Hi, thanks for the great work. Could I know when the code will be released?

Hi, there is nothing code about clustering. May be upload wrong code.


Enhance torchvision's draw_keypoint method

🚀 The feature
Follow up of #4216 . The current utility works fine, but we can make it better :)
Motivation, pitch
I think the following can be added.

[ ] Support multiple colors

There were 2 proposals one by @fmassa and one by me. The problem was handling excess colors.
I think that in fmassa's version we can broadcast to bigger dimensions and make it work?

[ ] Support specifying line color.

In my opinion, we shouldn't complicate this and just allow a single string.
Happy to hear thoughts though.

[ ] Plot the circle above the line or vice versa?

We currently plot the line above the circle. So at places I see that the line comes above the circle, somehow it looks like the line cuts / bisects the circle. I feel we should do the opposite.

[ ] Don't use default white for all colors. Use either a random palette / better default one.

I guess we should extend this functionality to all plotting utils, by default both draw_bounding_boxes and draw_keypoints plot with white color. While segmentation masks uses a fixed color palette.
Either we could use random colors / just have a nice fixed default palette.
(Also see #4658 )
cc @fmassa @datumbox @NicolasHug

@datumbox @NicolasHug @YosuaMichael let's take this down. I'm up for a PR. It's been 2 releases since we have the keypoints API which isn't  super good yet.

Thanks for the ping! Apologies for the lack of responses.
I want to be transparent on what's holding back the decisions around this. We are currently a bit swamped with the release, on going projects etc and we haven't had the opportunity to look into the Keypoint side of things. For example, though Transforms V2 is built to support Detection and Segmentation models, there are no current plans to extend it to Keypoints. That unfortunately is a beast of each own and requires us to put some thoughts on how to represent them, transform them and also visualize them. The risk we are running here is agreeing with a proposal without getting the full picture and then locking ourselves in due to the very strict BC-guarantees that TorchVision offers. So the problem is mostly that we lack the bandwidth to review the big picture around keypoints and make the necessary decisions over the direction. Note that some of these topics are tricky, as we discussed previously in the area of Detection, colouring different objects often require additional meta-data that we don't have (such as things vs stuff). I don't have great visibility on Keypoints and it's the things you don't know you don't know that often get you.
So a valid question is, when can we pick this up? I think after releasing Transforms V2 for Image & Video classification, Object Detection, Semantic & Instance Segmentation we would have to look into the Keypoints. There are other areas that might get in front of the queue (for example Depth Perception and Optical Flow). So there is overall lack of clarity on that front.
If @fmassa has an advice here, I would be happy to adopt it. But I also know he is completely swamped and I suspect reviewing this will take time.
Apologies for not being very helpful on this one. 😞

Yeah it's a bit tight. And please don't say sorry 😐
Multiple projects like, transforms, datasets with datapipes, newer models, batteries included phase 1/2/3, multi weight support, newer Operators, adding new tasks like optical flow, stereo.
The torchvision team is really working so hard. Just that this is a bit lower on priority but will get back to it soon!

Thank you for the response. I completely understand. I would happy to help in future if you work on it. Also want to appreciate all the good work you folks are putting in the repository.
Thanks again!!


Makefile is not compatible with newer versions of sigs.k8s.io/controller-tools

https://github.com/fennec-project/snoopy-operator/blob/56a283ba42e90bea9aa12e5f16639645208fb8bc/Makefile#L41
both of the options referenced here are removed in v0.7 of sigs.k8s.io/controller-tools as they are only used in v1beta1 crds,  which are removed as of k8s 1.22
Since the version of sigs.k8s.io/controller-tools isn't pinned to a pre-v0.7 version, new builds in clean environments fail.

seems the line should be changed to
CRD_OPTIONS ?= "crd"

Hi @csfreak , that makes total sense. I swapped this file recently and changed the CRDs from v1beta1 to v1 but I think I've introduced other problems. Another dev told me about that problem last week. Weirdly, it doesn't break on my environment. I'm putting this on the agenda for tomorrow's meeting.

introduced in #38


feat: Allow to enforce ellipsis instead of three dots

Our translation and design guidelines say to use ellipsis instead of three dots, yet some code is using three dots instead of the ellipsis character. This adds a custom fixer and rule to allow auto-fixing those strings.
I did not default enable it, because it is a risky rule that might change behavior (e.g. string comparison).
Feel free to dismiss if you feel like this should not be enforced.

Good idea, especially because it's also enforced for apps then.
Do we still run https://github.com/nextcloud/server/blob/master/build/triple-dot-checker.php?

~30 files in server.

That's a bit weird, but okay.

Or (when it's only about user facing strings) in the translation checker?
https://github.com/nextcloud/server/blob/master/build/translation-checker.php#L65

Or (when it's only about user facing strings) in the translation checker?

The idea was to also check for apps.

Should this be a Psalm rule like build/psalm/OcpSinceChecker.php?

If that makes more sense for you, then sure. I can look into that when I have some more time.

The idea was to also check for apps.

Yeah the above and some future checks should be done for all apps as well


Required permissions unclear

Permissions is a link to a picture and not inline.
In the picture all permissions are grayed out.

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

ID: da7ce00b-f843-f8d1-c949-bf4a15a53fb6
Version Independent ID: abd7e5ff-1cee-2c1f-71ef-7b9c27b023b2
Content: Authorizing the Office 365 Connector - Migrate to SharePoint and OneDrive
Content Source: migration/mover-o365.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Sorry, I do not get it.
Applications require a set of Graph permissions and typically the vendor documents the required Graph permissions.
For mover, these permissions are not documented because in the image link the permissions are grayed out.


Old logo in the docs docs

Contributions

[X] I have read the contribution docs, and understand what's expected of me.

Link to the page on docs.getdbt.com requiring updates
I found an old logo! https://docs.getdbt.com/reference/resource-properties/description#include-an-image-from-your-repo-in-your-descriptions
What part(s) of the page would you like to see updated?
The example image should use the new logo
Additional information
No response

hey @joellabes thanks for opening this up! going to remove the image for now so the old logo doesn't show up


storybook-loader for webpack

I have a project in which, each component directory contains a story:

With current react-storybook I have to require each .story.js file:

Is it possible to load all files ending with .story.js as stories using webpack:
loaders: [
  {
    test: /\.story.js/,
    include: require.resolve('src'),
    loader: 'storybook-loader',
  }
]

storybook-loader is just an idea.

I hope you found this: https://getstorybook.io/docs/basics/writing-stories#loading-stories-dynamically


App crashes in RKObjectRequestOperation.m on line 131 (objectRequestOpertationDidStart:)

When a user logs into my app, it does several request operations, but seems to crash when the object in the notification is 'nil'. This is happening everytime.
RKObjectRequestOperation *objectRequestOperation = [notification object]; objc_setAssociatedObject(objectRequestOperation, RKOperationStartDate, [NSDate date], OBJC_ASSOCIATION_RETAIN_NONATOMIC);
objectRequestOpertation is sometimes nil.
I've updated to the latest development branch that added the "RKManagedObjectStore_Private.h" file.
Update: The notification being sent is from initWithHTTPRequestOperation:responseDescriptors:. WeakSelf is declared and then once it gets into the executeBlock of the stateMachine, weakSelf is nil.
It appears it is when I set the completion block and then just do [operation start] and then leave the method it was called. Looks like my operation is being deallocated. This is happening with the 0.27.0 release as well.
This was working using 0.20.3.

I am seeing this too, in development under iOS10, fwiw.

I am getting the same issue if I revert back to 0.26 I no longer get this crash. But I need to update AFNetworking for other libraries and can't stay on 0.26, am thinking of abandoning RestKit all together

@icy1007 which Xcode / iOS version are you using?

@valeriomazzeo
Xcode: 7.3.1
iOS: 9.3

Are you able to write a test that reproduce this bug?

Hi there!
I've encountered this same issue using Xcode 7.3.1 and RestKit 0.26
Enclosed is a test that can reproduce this.
Thanks! 👍
RKTests.m.zip

@arron-jeffery thank you very much, hope @segiddins, myself or any good samaritan can have a look at it soon :)

I've actually gone all the way back to 0.20.3 and just added the fix for the RKManagedObjectStore warnings related to the @count property. Everything appears to be working as it should on iOS 9 and iOS 10.

Hi guy. Any news about this?

Same crash here.
iOS 10
XCode 8
RestKit 0.27.0

Same issue here, iOS 8, 9 and 10, Xcode 7.3.1, RestKit 0.27.

Replace [[NSNotificationCenter defaultCenter] postNotificationName:RKObjectRequestOperationDidStartNotification object:weakSelf];
To
[[NSNotificationCenter defaultCenter] postNotificationName:RKObjectRequestOperationDidStartNotification object:self];
Worked for me

I am having the same problem and cant find a solution for it

I'm having the same issue with 0.27. Xcode 8, iOS 10.1. Is there any workaround generally accepted without having to change the RestKit internal code?

@lazar-sidor I followed @silvaigor answer and it solved it
Use [[RKObjectManager sharedManager] enqueueObjectRequestOperation:operation];
Insted of
[operation start];

@silvaigor: Thanks for the update. I've found the code where I start the operations, I'vew used enqueue and it seems to work now!

we got the same problem because we use [operation start], seems like it doesn't crash on pre-0.27.0 because of this 'leak' https://github.com/RestKit/RestKit/commit/f4e70b69f2a584fb31814f50f19fcaacfb950fd4

I'm on iOS 11 and still, this issue occurs in 0.27.2 😞 I'm updating a legacy project to latest cocoapods and due to this issue, I'm unable to do so.
Any luck people?
Also [[RKObjectManager sharedManager] enqueueObjectRequestOperation:operation]; doesn't seem to work for me.

@mayooresan You need to set a RKObjectRequestOperation object to instance property.
@interface Requester : NSObject
@property (nonatomic, strong) RKObjectRequestOperation *op;
@end

@implementation Requester

- (void)request1 {
    // GET a single Article from /articles/1234.json and map it into an object
    // JSON looks like {"article": {"title": "My Article", "author": "Blake", "body": "Very cool!!"}}
    RKObjectMapping *mapping = [RKObjectMapping mappingForClass:[Article class]];
    [mapping addAttributeMappingsFromArray:@[@"title", @"author", @"body"]];
    NSIndexSet *statusCodes = RKStatusCodeIndexSetForClass(RKStatusCodeClassSuccessful); // Anything in 2xx
    RKResponseDescriptor *responseDescriptor = [RKResponseDescriptor responseDescriptorWithMapping:mapping method:RKRequestMethodAny pathPattern:@"/articles/:articleID" keyPath:@"article" statusCodes:statusCodes];

    NSURLRequest *request = [NSURLRequest requestWithURL:[NSURL URLWithString:@"http://restkit.org/articles/1234.json"]];
    RKObjectRequestOperation *operation = [[RKObjectRequestOperation alloc] initWithRequest:request responseDescriptors:@[responseDescriptor]];
    [operation setCompletionBlockWithSuccess:^(RKObjectRequestOperation *operation, RKMappingResult *result) {
        Article *article = [result firstObject];
        NSLog(@"Mapped the article: %@", article);
    } failure:^(RKObjectRequestOperation *operation, NSError *error) {
        NSLog(@"Failed with error: %@", [error localizedDescription]);
    }];
    [operation start];
    self.op = operation;
}
@end

I also get the crash on enqueueObjectRequestOperation but on different line number.
RKObjectManager.m line 920
-[RKObjectManager enqueueObjectRequestOperation:]
Additional crash info:
Fatal Exception: NSInternalInconsistencyException<_UILayoutSupportConstraint:0x1fc2b07f0 _UILayoutGuide:0x152a0a290.width == 0 priority:999 (active)>: An -observeValueForKeyPath:ofObject:change:context: message was received but not handled. Key path: operationCount Observed object: <NSOperationQueue: 0x1c0025ca0>{name = 'NSOperationQueue 0x1c0025ca0'} Change: { kind = 1; new = 1; } Context: 0x0 Raw Text


spinkube: bump to 0.2.0

[x] Notified the Maintainer of the application in this pull request so that they are aware of your proposal. (@radu-matei / @saiyam1814)
[x] Outlined the changes you are proposing, and the reasons these are required (e.g. stability, compatibility with new versions of Kubernetes implementations, etc).
[ ] Tested that the application works with the proposed updates applied (including a screenshot).
[x] Updated the version number of the app if that has changed.

I haven't tested this yet (was testing something else on Civo and remembered we cut a release last week), but I can do so later.
Changelog, the main new feature being ensuring that SpinApps have TLS certs for outbound connections: https://github.com/spinkube/spin-operator/pull/207

Done (also attached a screenshot of 0.2.0 running in a new Talos cluster)


Comments on translations

If you have suggestions, complaints, compliments, or any input on the new (professional) translations, please comment here.  If there are significant comments on a particular language, we can open an issue specifically for that language.
This is specifically for the actual translation quality, not layout or other technical issues.  Please open separate issues for anything of that nature (note known issue #532).
If you speak any of these languages, please have a look at the doc in that language and let us know what you think:

de (German)
es (Spanish)
fr (French)
it (Italian)
ja (Japanese)
ko (Korean)
pt-br (Brazilian Portuguese)
ru (Russian)
zh-cn (Simplified Chinese)
zh-tw (Traditional Chinese)

Note that these two languages are community contributions:

uk (Ukranian)
uz (Uzbek)

this documentation part is not translated to spanish http://expressjs.com/es/4x/api.html

@ripper2hl The API docs were not translated, as I noted in the blog:

The professional translations cover the entire doc set, except for the API docs and a few of the non-documentation pages ...

Please note that http://www.expressjs.com.cn/ is not connected in any way to this repo, and is outdated and potentially inaccurate.

The translations are more than 1 year old...
Most of the translated documents seem not to have updated since the "Initial delivery of all translations (46b27bb)" on Dec 18 2015 except for minor updates.
I'm willing to work on Japanese translation but it's said not acceptable in CONTRIBUTING.md.
Do you have any plans to update the documents?
Or isn't it possible to change the policy to accept community translations?
Thanks!

Hi @monmonmon
You are correct: We haven't updated the initial translations donated by IBM with any more professional translation.  However, there have been a few PRs with corrections / improvements to the existing translated material.

I'm willing to work on Japanese translation but CONTRIBUTING.md says it's not acceptable.

Actually, it says "... except for corrections to the existing translations." [my emphasis]
In other words, we welcome your work on the existing Japanese translation!  Perhaps I should clarify the wording to say "corrections, improvements, additions, and updates?  We just didn't want people to start translating "from scratch," but rather start with the existing translated material for a given language.
In any case, please feel free to contribute any updates to the existing Japanese content.  And thanks in advance! :-)

Nice to hear that :D I'm working on it now!
Is it better to make another issue?

Thanks :)

Hi,
I am french, so I checked the french translation, which is not that bad! :)
But I have spotted some mistakes like translating "node" into the french word  "noeud" even if it means "Node.js". So it could be confusing.
Also I have noticed there are some updates on the english version which are not present in the french one.
Should I make a PR to add fix and updates?

Hi @laudeon Yes, PRs for the translated material are welcome, especially if they are obvious mistakes as you note, or information that needs to be updated to align with the English version.
Thanks!

We haven't updated the initial translations donated by IBM with any more professional translation.

@dougwilson @wesleytodd are professional translation services no longer being donated to the project? If not, we should chat about how we can best try and get the word out to the community to help with translations.

are professional translation services no longer being donated to the project?

I believe the answer is "no" on this point. If @crandmck has a chance to chime in he would be the source of truth.

If not, we should chat about how we can best try and get the word out to the community to help with translations.

Yes 👍 Or just maintain the existing ones as well, as there are a lot of things on the English site that ever made their way into the translated sites, even the reorganization of middleware list.

I no longer work at IBM, so I don't have any insight into their intentions.  Given that professional translation can be rather expensive, I wouldn't expect any pro bono contributions along those lines from any company, but you never know!

About the pt-br translation, a lot of it seems to be literal.
Example: callback function to função de retorno de chamada. It is very common for brazilian portuguese speakers to use foreign words as part of the language (neologismos, as we call it), so the translation função callback seems more apropriate, less verbose and widely used by the community.
I've noticed that the usage of função de retorno de chamada seems to be some sort of standard.
Was this a deliberate choice? Could we ever change the translation to just função callback?

@jpricardo I suspect the original translators just took their best guess and probably weren't aware of the "neologismo," so changing it would be fine in this case.   Feel free to open a PR to change it.


Auto-close feedback success modals

When clicking "Yes" or "No" to provide feedback, the modal that displays at the end "Thank you" must be manually exited. Adding an automatic dismissal at the .5 or 1 second mark would improve the UX.
Modal HTML:

Thank you!

Button HTML:
<button title="Close (Esc)" type="button" class="mfp-close">×</button>

Now that we've gotten rid of the modals, closing this.


add test_sshd

ami-val tests that the instance uses systemctl. As cloud-image-val does not support rhel-6 or lower, this is not necessary.

Thanks for helping in this incredibly short but incredibly time consuming test!


Unify all tests into single cloudbuild execution

This PR refactors all of the various integration test scripts so that they can be run as a single cloudbuild execution. This has the following benefits:

less bash!
parallelization of build steps
running in Cloud Container Builder guarantees repeatability and hermeticity for integration test builds
any release candidate built with this cloudbuild configuration will necessarily have passed integration tests

Before merging, we need to do the following:

[ ] update build scripts in internal CI to use scripts/build.sh with the correct args
[ ] make sure that the GCP project used for release builds will tolerate spinning up multiple app engine services on each test run

Thanks! Documentation updates will come soon.
Cleanup is slightly complicated since failed builds will exit immediately by default. One solution I'm considering is to have a separate cloudbuild execution that just tries to clean up known resources.

@meltsufin Updated documentation, added cleanup!


Cannot compile example.tex

When trying to compile example.tex after cloning the repo, I get the following:
(./lib/dndcomment.sty
! Undefined control sequence.
l.2 \DeclareTColorBox
                     {commentbox}{O{} m O{commentboxcolor}}{%

Any thoughts?

Do you have a full TexLive 2017 installation?


Should the "Permission denied" error have its own custom message?

When working on my script, I often had the exp_data.csv opened in Excel to check if the data collected was stored as intended.
But the file gets locked for writing when it's still opened in Excel, so I regularly got a "Permission denied" error, when I forgot to close it:
 Exception during experiment execution.
Traceback (most recent call last):
  File "d:\python\expra\venv\lib\site-packages\alfred3\localserver.py", line 120, in experiment
    script.exp_session.movement_manager.move(direction=move)
  File "d:\python\expra\venv\lib\site-packages\alfred3\ui_controller.py", line 426, in move
    self._move(direction=direction)
  File "d:\python\expra\venv\lib\site-packages\alfred3\ui_controller.py", line 312, in _move
    cpage.section._move(direction=direction, from_page=cpage, to_page=target_page)
  File "d:\python\expra\venv\lib\site-packages\alfred3\section.py", line 652, in _move
    to_page.section._enter()
  File "d:\python\expra\venv\lib\site-packages\alfred3\section.py", line 953, in _enter
    self.experiment.finish()
  File "d:\python\expra\venv\lib\site-packages\alfred3\experiment.py", line 1070, in finish
    self._export_data()
  File "d:\python\expra\venv\lib\site-packages\alfred3\experiment.py", line 1093, in _export_data
    exporter.export(DataManager.EXP_DATA)
  File "d:\python\expra\venv\lib\site-packages\alfred3\export.py", line 89, in export
    self.export_exp_data()
  File "d:\python\expra\venv\lib\site-packages\alfred3\export.py", line 162, in export_exp_data
    self._write(alldata, fieldnames, path)
  File "d:\python\expra\venv\lib\site-packages\alfred3\export.py", line 121, in _write
    self.write(data, fieldnames, path, self.delimiter)
  File "d:\python\expra\venv\lib\site-packages\alfred3\export.py", line 130, in write
    with open(path, "w", encoding="utf-8", newline="") as csvfile:
PermissionError: [Errno 13] Permission denied: 'D:\\Python\\Expra\\data\\exp_data.csv'
2021-11-19 16:48:48,150 - exp.default_id - INFO - experiment id=default_id - session id=sid-8bfdce3b90aa42c08104e3b7b75252a4 -
ExperimentSession.abort() called. Aborting session. Reason: error

I did understand why I got this error, because I got it various times before when working with other frameworks, but beginners are likely to not understand why their experiment threw an error at this exact time. So it might be beneficial to include a custom error message here that gives a hint that the exp_data.csv might still be opened in another program.

I think this is a good idea. We should simply catch the error in export.py and re-raise it with an informative error message. But we should not presume too much, only suggest a possible cause, namely the .csv file still being opened.

We discussed this and decided that the current message is sufficient, since this is a very rare scenario


Question about 'time' crate dependency version

Hi,
unicode_names2_generator crate's time dependency is pinned to be <0.3.21
Any reason why this is done and can it be lifted?
This is giving dep resolution issues when using with crates that depend on recent 'time' versions
Example
error: failed to select a version for `time`.
    ... required by package `gix-date v0.8.1`
    ... which satisfies dependency `gix-date = "^0.8.1"` of package `gix v0.56.0`
    ... which satisfies dependency `gix = "^0.56.0"` of package `activity-git v0.1.0 (/Users/h7kanna/Repos/activity-git)`
versions that meet the requirements `^0.3.23` are: 0.3.30, 0.3.29, 0.3.28, 0.3.27, 0.3.26, 0.3.25, 0.3.24, 0.3.23

all possible versions conflict with previously selected packages.

  previously selected package `time v0.3.0`
    ... which satisfies dependency `time = "^0.3, <0.3.21"` of package `unicode_names2_generator v1.1.0`
    ... which satisfies dependency `unicode_names2_generator = "^1.1.0"` of package `unicode_names2 v1.1.0`
    ... which satisfies dependency `unicode_names2 = "^1.1.0"` of package

Any reason why this is done

time v0.3.21 removed support for the Rust version in Debian stable (1.63.0), which is the only source of binaries on the machine I use to upload crates.

and can it be lifted?

I'd welcome an alternative way to make this crate compile on Rust 1.63.0, but fiddling with cargo update for hours didn't help.

Resolved in v1.2.2 by making the dependency on time optional.


nostr: get nip46 relays from nip05 address

to enable the standard nip46 login flow from just the nip05 address

Sorry, its fails ci. parameter on line 196 needs to be prefixed by an underscore.

No problem, I adjusted it during cherry-picking :)

@DanConwayDev I thought to another solution that merge get_profile and get_nip46. Let me know what do you think. PR #481


Observation Deleted bottom sheet shows on wrong ObsDetail screen

Describe the bug
The observation was deleted bottom sheet pops up for me on an observation that I haven't deleted remotely or locally.
To Reproduce
Steps to reproduce the behavior:

Create some observations locally, upload them, and delete them on web
Go to the observation just below the last observation you deleted remotely in MyObservations
Tap into ObsDetails
See observation deleted screen, even though you haven't deleted this observation

Expected behavior
The deleted bottom sheet should only show on observations which have actually been deleted remotely.
Screenshots

Additional context
Not sure if we need more exact repro steps here.

Cannot replicate in 0.32.6+94 on iPhone , works as expected


July 31st, 2023 Community Meeting

Please add a topic in this thread and add a link to the GitHub issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on GitHub before coming into the meeting
(optional) Paste the image of an animal 😸

Introduce myself as PM of OpenShift Builds at RH.

Secret for projects pipeline, a secret is necessary: https://github.com/shipwright-io/.github/pull/1#discussion_r1278189905

Needs to be created here: https://github.com/organizations/shipwright-io/settings/secrets/actions
But how to get the key ? using a functional ID ? Using some personal access token that we seem to not have onboarded our org (https://github.com/organizations/shipwright-io/settings/personal-access-tokens-onboarding) ?

Meeting minutes:

Per https://github.com/shipwright-io/community/issues/149#issuecomment-1657801088 , it seems we are looking at credentials from a functional user. Options, OpenShift-ci , OpenShift-merge, shp-ci-bot, shp-ci-user and LF-user.  To figure out who are the shp ones. This is currently blocking https://github.com/shipwright-io/.github .
From https://github.com/shipwright-io/community/issues/149#issuecomment-1653819145, we are inclined more on an extension of build tools, instead of a replacement. @apoorvajagtap to open an issue on this.
From https://github.com/shipwright-io/community/issues/149#issuecomment-1651866939, there is another dependency on the release procedure, where we have nothing. @qu1queee to open a new issue for a quick overview on the API in Build for Triggers.
From https://github.com/shipwright-io/community/issues/149#issuecomment-1651866939, to continue on the potential improvements for the website deployment automation. @adambkaplan fyi


issue/31

Fixed a typo in msgs/help for google.
Instead of "g" it was "d" causing
confusion.
ref #31

daamn merge conflict

Is it something I've done?

Don't think so

Good :P
On 17 Feb 2012, at 22:40, "Callum Macrae"<EMAIL_ADDRESS>wrote:

Don't think so

Reply to this email directly or view it on GitHub:
https://github.com/callumacrae/Hex/pull/32#issuecomment-4029200

It is your fault - you shouldn't ever commit to the master branch xD

I did tell you
On 18 Feb 2012, at 11:32, "Callum Macrae"<EMAIL_ADDRESS>wrote:

It is your fault - you shouldn't ever commit to the master branch xD

Reply to this email directly or view it on GitHub:
https://github.com/callumacrae/Hex/pull/32#issuecomment-4033500

Did you? XD

Yes XD
On 18 Feb 2012, at 12:14, "Callum Macrae"<EMAIL_ADDRESS>wrote:

Did you? XD

Reply to this email directly or view it on GitHub:
https://github.com/callumacrae/Hex/pull/32#issuecomment-4033691


perf(module:date-picker): opening animation is not working

官网上随便找个 range picker 的例子，让 date picker 靠近页面底部以让 overlay 向上部展开，动效消失且 overlay 闪烁。

Duplicate of #1429. Please take care of that issue. @wenqi73


[Server-side] Add the most recent event entry in the search results

Ideally, we would like to have the most recent upcoming event and the most recent past event so that the timetable can be displayed correctly. Add this as a part of the current response

@giannisvas02 How about I add 2 query parameters start and end to the events endpoint? This way you can control the time range of what is displayed?
Otherwise which endpoint would you like that I add the most recent ond and the next upcoming event?

@samuelbohl sure, it will do! Its fine by me


wont run in the cloud

No matter what I do I can't get this to run in the cloud tried using render.com, netlify, and vercel.

I'm curious what problems you've encountered?

I get an "Invalid Host header" when I try and deploy it. Could be a really easy fix or something im doing wrong.. im new to all this and just started learning all this not long ago
but I also get this code
Build successful 🎉
==> Deploying...
==> Starting service with 'npm run start'

<EMAIL_ADDRESS>start /opt/render/project/src/lama_cleaner/app
react-scripts start

｢wds｣: Project is running at http://<IP_ADDRESS>/
｢wds｣: webpack output is served from
｢wds｣: Content not from webpack is served from /opt/render/project/src/lama_cleaner/app/public
｢wds｣: 404s will fallback to /
Starting the development server...
Your service is live 🎉
Failed to compile.
src/components/shared/Toast.tsx
Line 77:3:  defaultProp "desc" defined for isRequired propType  react/default-props-match-prop-types

got it all figured out!

figured it out


storage: make ingest and split_region interface compatible with API v2

What is changed and how does it work?

Add a ts field for RawWriteBatch if the API version is v2.
Encode the split key if the API version is v2.

Issue Number: ref #10938
What's Changed:

make ingest and split_region interface compatible with API v2.

Related changes

Need to cherry-pick the release branch

Check List
Tests

Unit test
Integration test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Release note
storage: make ingest and split_region interface compatible with API v2

@BusyJay @pingyu PTAL

@3pointer PTAL

@3pointer

/merge

/merge

/run-all-tests

/run-all-tests

/run-test

/merge

/run-test

/run-all-tests

/run-test

/run-test

/run-test

/run-all-tests


Developer experience enhancements

Description
This PR provides several changes aimed at improving developer experience for this repository:

VS Code configuration: .vscode/settings.json and .vscode/extensions.json for recommended workspace settings and extensions, respectively.
LocalStack workflow shortcuts via Taskfile: New task local: command group for streamlining common tasks
README updates: Incorporates mentions of new Taskfile shortcuts

Testing

Run task clean
Start LocalStack (or restart if already running)
Run task local:from-scratch and ensure that the command runs to completion (it will abort if an error is encountered)
Run awslocal s3api list-buckets and verify that buckets specified in Terraform exist, along with a local-terraform bucket
Run task local:deploy and ensure that the command runs to completion, including the following

Builds new binaries for any modified Lambda handler code in cmd/
Runs terraform apply targeting LocalStack

Run task local:invoke-DownloadGrantsGovDB and wait for the command to run. When finished, it should emit the response from awslocal invoke ... (i.e. JSON with a successful status code and message).

@joshgarza The included .vscode/settings.json file should help resolve the tabs-vs-spaces issue in openapi/openapi.yaml.


PyThaiNLP 1.7.4

Fixed #176
removed conllu from requirements.txt  #175

Coverage decreased (-0.03%) to 53.024% when pulling 44afd2a4c4a708a4164e9220c397cec3c17e8c93 on 1.7 into 8035bf261162df2a8d4ce027638eb3fc49da1a05 on master.

Coverage decreased (-0.03%) to 53.024% when pulling 44afd2a4c4a708a4164e9220c397cec3c17e8c93 on 1.7 into 8035bf261162df2a8d4ce027638eb3fc49da1a05 on master.

Coverage decreased (-0.03%) to 53.024% when pulling 44afd2a4c4a708a4164e9220c397cec3c17e8c93 on 1.7 into 8035bf261162df2a8d4ce027638eb3fc49da1a05 on master.


settings doesn't load Error while getting configuration.

notes are working properly, but setting page doesn't load.
I'm using Nginx with a subdomain setup.
I get this error on the website
Error while getting configuration.
and this in terminal
joplin                         | Application instance <Task pending name='Task-6' coro=<ASGIHandler.__call__() running at /root/.local/lib/python3.9/site-packages/django/core/handlers/asgi.py:155> wait_for=<Future pending cb=[_chain_future.<locals>._call_check_cancel() at /usr/local/lib/python3.9/asyncio/futures.py:384, <TaskWakeupMethWrapper object at 0x7f0e8b489460>()]>> for connection <WebRequest at 0x7f0e8bc12910 method=GET uri=/joplin/config/ clientproto=HTTP/1.1> took too long to shut down and was killed.
thank you so much for this docker image. it is outstanding!

Hello.
I think I had this issue on my raspberry... I think it was working well on me dev computer (which is powerful) but not very good on my raspberry (I'm speaking about the config page only).
I don't remember how I did on my raspberry to get the config page ... maybe I tried several time.
What is your server config ?

I'm running ubuntu server 18.04  with 2GB RAM.
It ended up being the internal docker network was being blocked by firewall. allowing the docker network like 192.168.x.1 made the notes work, but I tried allowing 192.168.x.2 and 192.168.x.3 and now it works.


Remove mandatory evaluation types

Update DOS evaluationType JSON schemas
Removes mandatory options and makes the question optional for all lots.
Add a migration to drop mandatory assessment methods from brief data
Removes work history and written proposal from all briefs, since they're no longer a valid option for evaluationType and are added automatically to all briefs.
Downgrade will add the options to all briefs based on lot. Since we don't know if the brief had the option selected before the upgrade migration we add them to ALL briefs, including draft ones. This means we might end up adding it to briefs that didn't have it before, but that seems better than silently removing the option, as all briefs should have them selected in the end.

The revision numbers need to be bumped 10 each, but after a rebase looks good.  The raw lot IDs in the downgrade make me slightly nervous, but as the lots were all added by migrations they will work fine.
:+1:


[ENHANCEMENT] Read directories

Context
Right now it reads a hardcoded base.star from the passed path. Instead it should read all the files
Description
To make things work, the CLI expects a base.star to be present in the directory, but this shouldn't be the case. it should load and read all files that have a .star extension and then concatenate them into a single string.
Acceptance Criteria

can read all the files in a directory
can extract the module from the read definition, so that it picks a module to load.

/assign
/triage
/label enhancement

/retitle Read all files in directory


Component - Data Table - Development

Detailed description
Describe in detail the issue you're having. Is this a feature request (new component, new icon), a bug, or a general issue?

Is this issue related to a specific component?

What did you expect to happen? What happened instead? What would you like to see changed?

What browser are you working in?

What version of the Carbon Design System are you using?

What offering/product do you work on? Any pressing ship or release dates we should be aware of?

Steps to reproduce the issue

Step one
Step two
Step three
etc.

Additional information

Screenshots or code
Notes

Add labels
Please choose the appropriate label(s) from our existing label list to ensure that your issue is properly categorized. This will help us to better understand and address your issue.

Designs are done, ready for dev: https://github.com/IBM/carbon-components/issues/1246
@carbon-design-system/ui

@jnm2377 - is this the issue that comprises all of the Data Table work (and thus closed by various PRs when complete)? Or is those closable by a specific PR?

@shinytoyrobots this compromises all of Data Table work so it's being closed by several PRs. I've been referencing it in my PRs and was planning on closing it with the last PR.

This issue was closed by stalebot.  Did all of the modifiers get added through the various linked PRs?


Updating upgrade package creation script

PR Overview:

Manually adding the BBB toolchain to PATH so that the dtc executable can be found when this script is run outside of the full build process
Bumping the default U-Boot version to our latest version number
Removing the -k option because it isn't used by anything

Documentation:
Relevant doc changes have been made in kubos/kubos#398

 Review/Update Using Kubos Linux doc


feat(robot-server): Mock up of access control models and router.

overview
Mock up of API to interact with access tokens and take control of a robot.
See : https://coda.io/d/Session-State-Management_dZ8yJSRoO6_/Access-Control_suRGF#_lu_TE for more details
These are the new API
# Token stuff #
# Get all the tokens present in the system
GET /access/tokens
# Create a token. This could potentially be renamed login if we want to support authentication
POST /access/tokens
# Delete a token
DELETE /access/tokens/{token}

# Control #
# Get info about who is in control of some scope of the robot
GET /access/controls/{scope}
# Try to give a token control of some scope of the robot
POST /access/controls/{scope}
# Forcibly take control of some scope of the robot
PUT /access/controls/{scope}/{token}
# Release control of robot scope
DELETE /access/controls/{scope}/{token}

Example of usage for taking control of robot:
request:
POST /access/tokens
response:
{
    'data': {
       'attributes': {
           'token': 'abc_123'
        }
    }
}

request
POST /access/controls/robot
{
    'data': {
       'attributes': {
           'token': 'abc_123'
        }
    }
}

risk assessment
Zero.
closes #5746

What would be an example client flow for calling these endpoints in the given user flows?

I think I like the sound of this. I'm a little more used to flows where the scopes (controls) are properties of the token rather that another resource, though I'm certainly not opposed to separating them because it looks fairly flexible. Do you have any reasoning on splitting them up like this?

I think I like the sound of this. I'm a little more used to flows where the scopes (controls) are properties of the token rather that another resource, though I'm certainly not opposed to separating them because it looks fairly flexible. Do you have any reasoning on splitting them up like this?

I considered both. Somehow I felt like it didn't quite make sense to me that a scope would be a prop of the token, but the other way around. Also, the path got really long the other way around.
I don't feel strongly about it. So would happy to change if you prefer it.

I think I like the sound of this. I'm a little more used to flows where the scopes (controls) are properties of the token rather that another resource, though I'm certainly not opposed to separating them because it looks fairly flexible. Do you have any reasoning on splitting them up like this?

I considered both. Somehow I felt like it didn't quite make sense to me that a scope would be a prop of the token, but the other way around. Also, the path got really long the other way around.
I don't feel strongly about it. So would happy to change if you prefer it.


Coordinates near Hungary/Romania border (on Hungary side) return no timezone

Wrong or unexpected timezone is returned
Coordinates near Hungary/Romania border (on Hungary side) return no timezone.
Which Timeshape version are you using?
2022g.17
Which Java version are you using?
18.0.2
If you upgrade to the latest Java minor version, does it still fail?

Yes

18.0.2

Which coordinates are you using?
46.16799, 20.71524
Which timezone is returned?
None
Which timezone is expected?
Europe/Budapest
Additional information
This is very close to the border with Romania, but still in Hungary:
https://www.google.com/maps/place/46°10'04.8"N+20°42'54.9"E/@46.16799,20.7130513,17z/data=!3m1!4b1!4m4!3m3!8m2!3d46.16799!4d20.71524

Thanks for reporting this, I'll have a look. From the first glance it looks like the problem with the source data, so it might be worth checking this with the https://github.com/evansiroky/timezone-boundary-builder.

Hello. I'm the maintainer of timezone-boundary-builder. This particular coordinate is so close to being exactly on the border of data from OpenStreetMap which is the data source of timezone-boundary-builder. I'm attaching a screenshot of this coordinate as it appears on OpenStreetMap:

I tried this coordinate out with timezonefinder and node-geo-tz and both of them are returning Europe/Budapest.

I think the problem occurs because of the use of the "GeometryEngine.contains" method in the Index class. https://github.com/RomanIakovlev/timeshape/blob/a170d21fffb99b0727aa00a48871f1588afc7df4/core/src/main/java/net/iakovlev/timeshape/Index.java#L51
In the documentation, the "Contains" relation is described as follows: "One geometry contains another geometry if the other geometry is a subset of it and their insides have at least one point in common". http://esri.github.io/geometry-api-java/doc/Contains.html
In this particular case, the point is on the boundary of the zone, so the interior of the zone has no points in common with the point, only the boundary. So the point is not contained within the zone, and the "contains" method returns false if it checks if
the point is contained in the zone. I suppose it would be better to use "intersects" instead of "contains", as this will cover cases where the point belongs to either the boundary or the interior of the zone.

Nice find @lavika11! Contains is most probably the culprit, given the description from the docs. Would you mind taking a stab at making the necessary change, so that Timeshape can support both internal and border point queries?
P.S. it's probably worth noting Timeshape also uses contains in a couple of other places, particularly in Index::queryPolyline method, which should be updated too.

Yes, I an take a look.

This should be fixed in Timeshape version 2023b.19 thanks to contribution from @lavika11. @joaocanaverde please give it a try and feel free to reopen the issue if the problem persists.


crash in onefile.py:  assert not os.path.exists(source_dir)

Hi,
On Windows 10, I've recently upgraded from python 3.7.4 / Nuitka 2.2.1
to python 3.11.9 / Nuitka 2.2.3.
I'm recompiling one of my projects with the new versions,
but I get a compile error right at the end:
...
Nuitka-Onefile: Using compression for onefile payload.
Nuitka-Onefile: Onefile payload compression ratio (26.23%) size 67447222 to 17691695.
Nuitka-Onefile: Removing onefile build directory 'MediaPlayerTexture.onefile-build'.
Traceback (most recent call last):
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 161, in
main()
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka_main_.py", line 147, in main
MainControl.main()
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1124, in main
_main()
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\MainControl.py", line 1035, in _main
packDistFolderToOnefile(dist_dir)
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 52, in packDistFolderToOnefile
packDistFolderToOnefileBootstrap(onefile_output_filename, dist_dir)
File "C:\Users\Orac\AppData\Local\Programs\Python\Python311\Lib\site-packages\nuitka\freezer\Onefile.py", line 261, in packDistFolderToOnefileBootstrap
assert not os.path.exists(source_dir)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AssertionError
Nuitka-Reports: Compilation crash report written to file 'nuitka-crash-report.xml'. Please include it in your bug report.
I use the following command:
python -m nuitka --onefile --remove-output --windows-disable-console --onefile-tempdir-spec="{TEMP}/_MediaPlayer" --windows-icon-from-ico="MediaPlayer.ico" --output-filename="MediaPlayer" MediaPlayerTexture.py
After it crashes, it leaves behind dir 'MediaPlayerTexture.dist',
and interestingly, if I run the .exe inside that dir, the app runs just fine.
The packaging into Onefile seems to be the only problem.
Running python MediaPlayerTexture.py from the console works fine.
The older python 3.7.4 / Nuitka 2.2.1 combo works fine too.

It's asserting that it is able to delete the pre-existing build folders before building them. That not working, my best guess, is that AV tools hold locks while deleting the folder. It is doing
        removeDirectory(path=source_dir, ignore_errors=False) right before, and that it does not work is very strange. This should be the .onefile-build folder. What is left behind there? Usually, the exception should be caught, and a retry be made, and if that still arises, it's passed to the outside.
Our error handler retrying, doesn't necessarily make things continue after succeeding in a local retry though. I don't know if this function changed in Python3.11, but the error handler is called, but the action is not resumed, the error handler should always raise I guess, or not be there at all, it might not be helping. I will therefore switch to a mode, where it now records the function that failed, and retries it for as long as something else fails.
I got a new implementation of it, that I want you to try on the factory branch.

Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback on whether this is working is very welcome. Please do not share plans to do it; only confirm or deny that it is working.

Thanks for the feedback. Before going to the factory branch, I tried
another compile to re-verify the error. The weird thing is that it
now compiles cleanly (with the new python/nuitka versions)!
Not sure why. The only change I made to my environment was for a
different project which required me to install the following packages:
pip install requests
pip install ratelimit
pip install PyJWT
I'm not sure that there is any value in trying the factory branch now,
but if I get a similar error in a different project, I will give it a try.
Thanks for your help.

It would depend on specific race conditions I suppose. I think I might have prevented those from appearing, but it's going to be hard to tell. I am closing this early for lack of general relevance.

Part of the 2.3 release I just made.