actorODP module report from evaluation

Report from FOOPS!

[x] Version IRI could not be resolved
[x] ~Inconsistent ontology IDs (error due to versioned ontology URI, ignore this)~
[x] Minimum metadata missing (license)
[x] ~Prefix not found in prefix.cc nor LOV registries (ontology not yet released, ignore this)~
[x] ~Ontology not found in LOV registries (ontology not yet released, ignore this)~
[x] ~Metadata not accessible in LOV registries (ontology not yet released, ignore this)~
[x] OM2: Missing parts of recommended metadata (creation date)
[ ] OM2: Missing parts of recommended metadata (citation (?))
[ ] OM3: Missing detailed metadata (doi, publisher, logo, status, source, issued date) (not all are applicable)
[x] License is not resolvable
[ ] Missing documentation labels for 4 ontology terms.
[ ] Missing documentation definitions for 4 ontology terms.
[ ] Basic provenance metadata missing (author, creation date)
[x] ~Detailed provenance metadata missing (issued, publisher) (ontology not yet released, ignore this)~

Report from OOPS!

[ ] Unconnected ontology element (Process)
[ ] Missing annotations (see above)
[x] ~Missing disjointness (ignore this)~
[ ] Missing domain and range for properties (fix where applicable)
[x] ~Inverse relationships not explicitly declared (ignore this)~
[ ] Ambiguous namespace (verify that ontology namespace is defined)

SPARQL support for CQs (for v. 0.1)

CVN-Process-3: Supported
CVN-Actor-1: Partial support
CVN-Actor-4: Partial support
CVN-Actor-6: Partial support
CVN-Actor-7: Supported
CVN-Competency-3: Supported
CVN-Information-4: Partial support
C11-1: Partial support
C11-3: Partial support
E1-6: Partial support
E1-7: Partial support
E1-8: Partial support
E1-9: Not covered
E4-10: Partial support

FOOPS

Citation refers to https://schema.org/citation. Do we want to include a link to e.g. the GitHub project?
http://purl.org/ontology/bibo/status could potentially be http://purl.org/ontology/bibo/status/draft(see http://gbol.life/ontology/bibo/DocumentStatus/)
process:Process is missing both label and comment. However, it seems strange to define them here. Should not actorODP import process? The same thing applies to resource:Resource.

OOPS!

process:Process is missing both label and comment. However, it seems strange to define them here. Should not actorODP import process? The same thing applies to resource:Resource.


textInputProps & clearButtonMode

Hello,
This does not work as intended:
      textInputProps={{
          clearButtonMode: 'never'
        }}

Thanks

I submitted a pull request to fix this, https://github.com/FaridSafi/react-native-google-places-autocomplete/pull/153, but if you need a fix for it now, just follow the changes in that commit.

Thanks !


Selfhosted under subdirectory won't work with Onedrive

[x] I agree to follow the Code of Conduct that this project adheres to.
[x] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
We have been running a self-hosted version for some time and are in the process of implementing OneDrive. Our version does not run on the root URL of the host, ie DRAWIO_BASE_URL=https://hostname.servername.com/diagrams/
The OneDriveClient.js file hardcodes the redirect URL's to the root directory:
OneDriveClient.prototype.redirectUri = window.location.protocol + '//' + window.location.host + '/microsoft'; OneDriveClient.prototype.pickerRedirectUri = window.location.protocol + '//' + window.location.host + '/onedrive3.html';
This hardcoding prevents the use of running the software in a path other than the root URL.
The application was locally modified to support subdirectories, authorization was approved by the administrator, but a 500 error was thrown. It appears the Java portion is using the root URL as well. Here is a snippet from the Tomcat server:
08-Aug-2023 14:59:56.141 SEVERE [http-nio-8080-exec-109] com.mxgraph.online.AbsAuth.contactOAuthServer AUTH-SERVLET: [https://login.microsoftonline.com/common/oauth2/v2.0/token] ERROR: Server returned HTTP response code: 400 for URL: https://login.microsoftonline.com/common/oauth2/v2.0/token -> {"error":"invalid_client","error_description":"AADSTS500112: The reply address 'https://hostname.servername.com/microsoft' does not match the reply address 'https://hostname.servername.com/diagrams/microsoft' provided when requesting Authorization code.\r\nTrace ID:........}
Expected behavior
The software should support OneDrive when self-hosting under non-root URL locations
Screenshots
500 Error after trying to access OneDrive:

draw.io version (In the Help->About menu of the draw.io editor):

draw.io version 21.1.4

Additional context
Application has been successfully setup in Azure and authorization approved by the administrator.

This should work in 24.0.6.


Add option to configure disabling automatic search in repos/views

This change:

Adds the DisableAutomaticSearch field to the spec of HumioRepository and HumioView
Adds the Description field to the spec of HumioView
Resolves #693

Notes:
I decided to inverse the Automatic Search option to DisableAutomaticSearch, that way by default the bool has it's default false value, and the repo/view maintains it's true value if this field is not specified.
Requires:

cli PR: https://github.com/humio/cli/pull/163

I created https://github.com/humio/humio-operator/pull/815 which also bumps the dependency, which we can also use to see if tests pass with this change.

Seems like even with your change and updated dependency as I tried in #815 tests seem to fail with:
2024-06-26T08:33:39.9150010Z   [FAILED] in [It] - /var/src/controllers/suite/resources/humioresources_controller_test.go:496 @ 06/26/24 08:33:39.393
2024-06-26T08:33:39.9151130Z   << Timeline
2024-06-26T08:33:39.9151379Z
2024-06-26T08:33:39.9151664Z   [FAILED] Timed out after 300.001s.
2024-06-26T08:33:39.9152147Z   Expected
2024-06-26T08:33:39.9152502Z       <api.View>: {
2024-06-26T08:33:39.9153032Z           Name: "example-view",
2024-06-26T08:33:39.9153549Z           Description: "",
2024-06-26T08:33:39.9154017Z           Connections: [
2024-06-26T08:33:39.9154441Z               {
2024-06-26T08:33:39.9155159Z                   RepoName: "example-repository-view",
2024-06-26T08:33:39.9155826Z                   Filter: "*",
2024-06-26T08:33:39.9156386Z               },
2024-06-26T08:33:39.9156746Z           ],
2024-06-26T08:33:39.9157181Z           AutomaticSearch: true,
2024-06-26T08:33:39.9157629Z       }
2024-06-26T08:33:39.9157920Z   to equal
2024-06-26T08:33:39.9158270Z       <api.View>: {
2024-06-26T08:33:39.9158784Z           Name: "example-view",
2024-06-26T08:33:39.9159293Z           Description: "",
2024-06-26T08:33:39.9159761Z           Connections: [
2024-06-26T08:33:39.9160179Z               {
2024-06-26T08:33:39.9160898Z                   RepoName: "example-repository-view",
2024-06-26T08:33:39.9161564Z                   Filter: "*",
2024-06-26T08:33:39.9162012Z               },
2024-06-26T08:33:39.9162368Z           ],
2024-06-26T08:33:39.9162803Z           AutomaticSearch: false,
2024-06-26T08:33:39.9163248Z       }
2024-06-26T08:33:39.9164196Z   In [It] at: /var/src/controllers/suite/resources/humioresources_controller_test.go:496 @ 06/26/24 08:33:39.393

Thanks @SaaldjorMike. I'll take another look at this later today. It looks like due to the different field names "AutomaticSearch" in the CLI, and "DisableAutomaticSearch" in the CRD.
That got me thinking about the bool value as well, as it will currently default to false, which would enforce change onto any operator managed repo, which may or may not be what some people want.
I might have a look at using a separate type instead to handle the value and allow it to be null/un-managed.

If you want, you could make them pointers, and so you'd know if they are actually explicitly set by the user or not:
From the current:
// DisableAutomaticSearch is used to disable the start search automatically on loading the search page option.
DisableAutomaticSearch bool `json:"disableAutomaticSearch,omitempty"`

To this:
// DisableAutomaticSearch is used to disable the start search automatically on loading the search page option.
DisableAutomaticSearch *bool `json:"disableAutomaticSearch,omitempty"`

then you know if e.g. DisableAutomaticSearch is not not that the user explicitly set it to either true or false.
But if you do, then you might as well just go with:
// AutomaticSearch is used to set the start search automatically on loading the search page option.
AutomaticSearch *bool `json:"automaticSearch,omitempty"`

since you no longer rely on the empty value of that bool.

Thanks @SaaldjorMike, good idea. I've changed them over to pointers now and including a BoolTrue function in the helpers package. Also added a couple line to the tests which should resolve those issues. Cheers.

The diff seems to be showing a lot of unrelated things to this PR, things that was introduced on master. Maybe we could get that cleaned up a bit, then I can kick off a round of tests, and once green, we can look at getting it merged in.

Alright, I fixed that up. You should be good to kick off the tests @SaaldjorMike.
Note to self: watch what you rebase. :)


After installing the app the view page not render on store? app installed successfully everything working fine but facing this issue

After installing the app the view page not render on the store? app installed successfully everything working fine but facing this issue.

Package Version: osiset/laravel-shopify": "^17.1
Laravel Version: 8.65
PHP Version: v7.3

Hi @Msk101
Can you provide steps on how I can reproduce this issue and anything specific you have done to the package which differs from the standard install?

After installing the app the view page not render on the store? app installed successfully everything working fine but facing this issue. how to solve it,

After installing the app the view page not render on the store? app installed successfully everything working fine but facing this issue. how to solve it,

Can you provide steps to reproduce and code snippets. We can't help otherwise.

Can you please share the url and your code in web.php and your home blade view.

I'm also facing the same issue.
below URL used for app setups.
https://tsm.cgprojects.in
https://tsm.cgprojects.in/authenticate
...waiting for the resolution on the same.

Route::get('/', function () {
return view('welcome');
})->middleware(['verify.shopify'])->name('home');

View file code - LSRI\resources\views\welcome.blade.php
@extends('shopify-app::layouts.default')
@section('content')

You are: {{ $shopDomain ?? Auth::user()->name }}
@endsection
@section('scripts')
@parent
<script>
    actions.TitleBar.create(app, { title: 'Welcome' });
</script>

@endsection

@ali-cartgeek I can't replicate this but can you update to 17.2 and try.

@ali-cartgeek this sems to be a code-igniter project.

@usmanpakistan good spot actually, closing this out as it does we don't support codeigniter.


Max number of owners per team

The current page mentions "There can be as many as 10 team owners per team."
However here (https://docs.microsoft.com/en-us/MicrosoftTeams/assign-roles-permissions) it says: "A team can have up to 100 owners."

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

ID: c807f882-de3c-9c30-f92f-43769356f015
Version Independent ID: 7658a338-51a0-196a-ce53-2a213a5f0dc6
Content: Plan for lifecycle management in Teams - Microsoft Teams - Microsoft Teams
Content Source: Teams/plan-teams-lifecycle.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@GaelleLeC Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

https://support.office.com/en-us/article/assign-a-new-owner-to-an-orphaned-group-86bb3db6-8857-45d1-95c8-f6d540e45732    This one says 100 Owners

@GaelleLeC Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


sample downloader: 取得失敗 UTPC 2011

http://judge.u-aizu.ac.jp/onlinejudge/description.jsp?id=2259
https://beta.atcoder.jp/contests/utpc2011/tasks/utpc2011_1
AOJ経由でもAtCoder経由でもだめ そもそもAtCoder側は既に壊れてるし

This contest's HTML seems different from any other contests in AtCoder.
In my work on #573, I re-found this issue.
We can solve this issue by using soup.find_all('pre', 'literal-block').

Resolved by #581


Add custom resource quota example

/milestone 1.15
xref https://github.com/kubernetes/kubernetes/pull/64201

/lgtm

/approve

Hi  @tengqm can you approve these changes?


Diablo2  Unable to enter the game, log error report。

DESKTOP-R8DUG87 Realm startup at 2020-11-14 18:00:27.500
18:00:27.500  ------------------------------------------------------
18:00:27.500  2020-11-14 18:00:27
18:00:27.500  Diablo II running under Windows XP (Version 6.010)
18:00:27.500  Vendor:  Intel
18:00:27.500  Version: Pentium III (no brand) (Type 0, Family 6, Model E, Stepping A, Brand 0)
18:00:27.516  Speed: Approx. 2807 MHz
18:00:27.516  Total Physical Memory: 1024.00MB
18:00:27.516  Application Path: F:\D2Lod
18:00:27.516  Local IP: <IP_ADDRESS>
18:00:27.516  Computer name: DESKTOP-R8DUG87
18:00:27.516  ------------------------------------------------------
18:00:27.516
18:00:27.516  imagehlp.dll loaded for better stack crawls.
18:00:28.086  ........DirectSound progress: 0 buffers available is less than min needed 16 buffers for 3d sound
18:00:28.101  ........DirectSound progress: 0 buffers available is less than min needed 16 buffers for 3d sound
18:00:29.273  ***** DirectDraw error: SetEntries failed! - DDERR_GENERIC (0x80004005)

It's probably not using cnc-ddraw since it wouldn't return DDERR_GENERIC errors on SetEntries.
Some chinese anti-virus programs add ddraw.dll to the KnownDlls list into the registry, maybe that could be the reason?

OK~


Support more platforms with docker images

Hi there
would it be possible to support more platforms as docker images?
Thanks & BR
Matthias

Hi @mad73923
Thank you very much for your question.
Can you tell me more about what you mean by platforms? Do you mean other base images (instead of the official PHP one)? Or where do you want to use the image but can't because of this limitation?
Thank you very much for your help!
Kind regards,
zepich / Matthias :)

Hi @zepich
thank you for your super fast reply!

What I'm asking for are image platforms for docker. eg:

x86-64 (linux/amd64, linux/i386)
ARM architectures (linux/arm/v5, linux/arm/v6, linux/arm/v7, linux/arm64)
PowerPC and IBM Z (linux/ppc64le, linux/s390x)

Actually, I'm building an ARM64 image right now. If successful, I'll provide a PR.
BR
Matthias

Hi @mad73923
Thank you very much. Now I understand what you mean, and you're right: we should offer more platforms.
If you have a PR for arm64, that would be awesome. I will look into this topic. I think it should not be a big issue and probably (more or less) just an adjustment of the deployment process.
Thank you very much for bringing this issue to our attention!
Kind regards,
zepich

Hi @zepich
I successfully got mosparo running in docker under linux/arm64.
The only thing I had to do was to run the docker build process with following command:
docker buildx build --platform linux/arm64 --build-arg="MOSPARO_VERSION=v1.2.1" -t mosparo/mosparo:latest .

I couldn't find any place where you build & deploy your docker images. Ever taught about github actions?
By the way: mosparo looks & feels awesome!! And works without hassle. It's been a while since I used such a nice piece of software which worked out of the box! :clap:
BR
Matthias
P.S.: I got one more idea to improve: make the manual installation process (in docker) obsolete by using environment variables. Maybe a point for another enhancement issue?

Hi @mad73923
Thank you very much for your feedback, testing, and, in general, your help in making mosparo better!

I successfully got mosparo running in docker under linux/arm64.
The only thing I had to do was to run the docker build process with following command:

docker buildx build --platform linux/arm64 --build-arg="MOSPARO_VERSION=v1.2.1" -t mosparo/mosparo:latest .

I found the same (or a similar) command yesterday and tested it to build it. I could not test it since I had no ARM CPU available for testing. But since we're using PHP, we should technically not have any problems with all platforms.
I will test it with an ARM CPU over the weekend. Additionally, I plan to release a new version on the weekend (or early next week) and try to add more platforms with that release (and for all future releases).

I couldn't find any place where you build & deploy your docker images. Ever tought about github actions?

We build all our releases for mosparo itself and for all integrations, libraries, and other stuff like the website or the version/update files on a virtual server. For me, it was the easiest approach to set everything up and the most secure from our perspective because we do not have to store any credentials and certificates on a website.
Now, with the multiple platforms, we have a bigger issue because building takes time, and I would like to be able to have a fast release (right now, with one Docker platform (AMD64), it takes around 15 minutes to do all the stuff we have to do for a new version; testing the ARM64 build on a AMD64 platform tooks 20+ minutes alone). I looked yesterday at Docker Build Cloud, but the problem is that we would be limited in the number of builds we can make per month, or it can get expensive. GitHub Actions could be an alternative for that problem. But yeah, I didn't have time to look into GitHub Actions, and doing it with a simple shell script on a virtual server is way faster (at least when setting it up for the first Docker image)...

By the way: mosparo looks & feels awesome!! And works without hassle. It's been a while since I used such a nice piece of software which worked out of the box! 👏

Thank you very much for your feedback and words! As a developer, this is the best thing to read and help continue working on such a project! Thank you very much!

P.S.: I got one more idea to improve: make the manual installation process (in docker) obsolete by using environment variables. Maybe a point for another enhancement issue?

I once thought about this and that we could optimize the Docker image with it. However, I didn't have time to proceed with this adjustment. It would be great if you could create another issue for that improvement. Otherwise, I will have a look in the future anyway.
I wish you a nice day, and thank you for all your help!
Kind regards,
zepich

Hi @mad73923
We've just released version 1.2.2 with your pull request regarding the PostgreSQL extension. Additionally, as announced, the Docker image for this version (and all future versions) is available for multiple platforms (amd64, 386, arm64/v8, arm/v5, and arm/v7).
For these platforms, we're using Docker Build Cloud, which makes this task very easy and fast, so the whole deployment process will not take more time. For the other platforms that the PHP base image supports (mips64le, ppc64le, and s390x), we do not have a solution right now since the Docker Build Cloud does not support them (as far as I understand).
What's your opinion about these platforms? Do you think these platforms are required? I also have the problem of not having a server on such a platform, which makes it impossible to test the images. Do you know a hoster with such a platform?
Thank you very much for your help!
Kind regards,
zepich

Hi @zepich
you're to fast for my little time I can spend here :see_no_evil:
Here's what I was writing before your answer:
thank you for your detailed answer!
Regarding github actions:
The free plan will include 2000 minutes of building your images (Source)
If this doesn't sound good enough to you I highly recommend running a self hosted github action runner on your VM. I do the same for some gitlab projects. No more manually starting any build scripts!
I'll provide a first draft for the action manifest. The only thing missing is you to add your Dockerhub credentials in your repo.

Here's what I have to say after your 2nd answer:
Thanks for the update! I didn't know Docker Build Cloud so far.

For the other platforms that the PHP base image supports (mips64le, ppc64le, and s390x), we do not have a solution right now since the Docker Build Cloud does not support them (as far as I understand). What's your opinion about these platforms? Do you think these platforms are required? I also have the problem of not having a server on such a platform, which makes it impossible to test the images. Do you know a hoster with such a platform?

These are some very special platforms which I just copied from the docker docs. As long as noone requests them it's not worth the effort to provide them. with the setup you just delivered with 1.2.2 you will cover 99% of the usecases I guess.
Regarding the hoster: I use Hetzner for arm64.
PR for github actions will be here in a sec.
BR
Matthias

Hi @mad73923

you're to fast for my little time I can spend here 🙈
Here's what I was writing before your answer:

My biggest issue in the past two years: Time... :D Don't worry, I can understand that! You answered on Sunday, but I didn't have time to answer you until now... So, I'm not always that fast... :)
Thank you very much for your answer and explanation regarding GitHub Action Runner. I didn't know that it's even possible to host it on our server. I will look into that one.
Also, thank you for your pull request with the actions configuration. Since we're using Docker Build Cloud now, using GitHub Actions to build the images would be a good option. Before, with only one image, I tested the image before I pushed it. That's not possible anymore, so we have to push it anyway, and therefore, it doesn't matter if we build it locally with Docker Build Cloud or with GitHub Actions.

These are some very special platforms which I just copied from the docker docs. As long as noone requests them it's not worth the effort to provide them. with the setup you just delivered with 1.2.2 you will cover 99% of the usecases I guess.

Perfect! I have the same opinion. In the end, the Dockerfile (and mosparo itself) is open source, so if someone needs an image for one of these platforms, they can build it pretty easily (as you were able to make the ARM image without any troubles). And if, at one point, somebody will over virtual servers on these platforms, we're open to discussing it again.
Thank you regarding Hetzer. I've tested the ARM image with a Hetzner ARM VM and a Raspberry Pi, which both worked perfectly... :)
I will close this issue since it's solved.
Thank you very much, and have a great day!
Kind regards,
zepich


show count and total bytes of delayed deleted slices in status subcommand

Close #2957
With this PR, we can find the count and total bytes of delayed deleted slices in the status subcommand:
./juicefs status redis://localhost
2022/11/14 14:12:12.077252 juicefs[84497] <INFO>: Meta address: redis://localhost [interface.go:414]
2022/11/14 14:12:12.078792 juicefs[84497] <INFO>: Ping redis: 58.958µs [redis.go:2950]
Delayed Slices count: 166372                                                        ⠙
Delayed Slices bytes: 48.13 GiB<PHONE_NUMBER>7 Bytes)                                 ⠙
{
  "Setting": {
    "Name": "myjfs",
    "UUID": "4bdf1c96-c1cd-4450-84af-d16162af60c0",
    "Storage": "minio",
    "Bucket": "http://<IP_ADDRESS>:9000/jfs",
    "AccessKey": "minioadmin",
    "SecretKey": "removed",
    "BlockSize": 4096,
    "Compression": "none",
    "EncryptAlgo": "aes256gcm-rsa",
    "KeyEncrypted": true,
    "TrashDays": 1,
    "MetaVersion": 1
  },
  "Sessions": [
    {
      "Sid": 2,
      "Expire": "2022-11-14T14:13:03+08:00",
      "Version": "1.1.0-dev+2022-11-09.4bf5f66b",
      "HostName": "Arch",
      "MountPoint": "/mnt/jfs",
      "ProcessID": 48807
    }
  ],
  "Slices": {
    "Delayed": {
      "Count": 166372,
      "TotalSize":<PHONE_NUMBER>7
    }
  }
}

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

Codecov Report
Base: 58.89% // Head: 58.78% // Decreases project coverage by -0.11% :warning:

Coverage data is based on head (b0928d9) compared to base (34262a7).
Patch coverage: 48.07% of modified lines in pull request are covered.

:exclamation: Current head b0928d9 differs from pull request most recent head cf1e65b. Consider uploading reports for the commit cf1e65b to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2977      +/-   ##
==========================================
- Coverage   58.89%   58.78%   -0.12%
==========================================
  Files         141      141
  Lines       24545    24593      +48
==========================================
- Hits        14457    14456       -1
- Misses       8214     8248      +34
- Partials     1874     1889      +15

Impacted Files
Coverage Δ

pkg/gateway/gateway.go
44.52% <0.00%> (+1.15%)
:arrow_up:

pkg/meta/interface.go
45.45% <ø> (ø)

pkg/meta/sql.go
63.73% <40.00%> (-0.58%)
:arrow_down:

pkg/meta/redis.go
69.69% <45.45%> (-0.11%)
:arrow_down:

pkg/meta/tkv.go
74.42% <54.54%> (-0.13%)
:arrow_down:

cmd/status.go
67.85% <62.50%> (-2.88%)
:arrow_down:

pkg/vfs/reader.go
75.42% <0.00%> (-3.79%)
:arrow_down:

pkg/fs/fs.go
76.61% <0.00%> (+0.03%)
:arrow_up:

... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Added more API directories for web application  enumeration

Hi,
Thanks for the PR 👍
Where the information come from?
Thank you very much in advance 😃

Hey @righettod
I was looking for a bigger wordlist to have API endpoint enumeration. I thought of combining a good API wordlist present in SecList and removing the duplicates.
Have a great day ahead.

Thanks for putting this together @InTruder-Sec


KAFKA-14255: Fetching from follower should be disallowed if fetch from follower is disabled

There are clients out there that have implemented KIP-392 (Fetch From Follower) and thus use FetchRequest >= 11. However, they have not implemented KIP-320 which add the leader epoch to the FetchRequest in version 9. Without KIP-320, it is not safe to fetch from the follower. If a client does it by mistake – e.g. based on stale metadata – that could lead to offset out of range.
This patch proposes to disable fetching from a follower when the cluster does not have a replica selector. If it does not, consumers are not expected to fetch from followers.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This change makes sense. I guess the other interesting case is if fetch from follower is enabled on the broker, but the client is not sending the epoch (KIP-320). What should we do in that case? Is it safe to let it through? We may need another JIRA to discuss that case, which is not as clear, but still potentially problematic.

This change makes sense. I guess the other interesting case is if fetch from follower is enabled on the broker, but the client is not sending the epoch (KIP-320). What should we do in that case? Is it safe to let it through? We may need another JIRA to discuss that case, which is not as clear, but still potentially problematic.

I think that one fundamental issue is that a follower is accessible even if it has not caught-up yet with the leader. The follower might be far behind for instance or even have wrong records if the truncation did not happen yet. It seems to me that we should make it inaccessible until the follower has at least got the updated high watermark from the leader. Once we have the HWM, we could perhaps also use OFFSET_NOT_AVAILABLE if the requested offset are not available yet. I will file a jira and put my thoughts in it.

@hachikuji Yeah, that's right. If a client already specifies the rack id before the cluster sets up a replica selector, the client won't be able to fetch until all the brokers have the selector. An alternative to the current proposal would be to check wether the rack id is present or not. We could for instance disallow fetching from a follower if the rack id is null and the leader epoch is -1. That would limit the fix to the set of clients that implemented FFF without KIP-320. Do you think that would be better?

We could document that fetch from follower must first be enabled on the cluster before it's enabled on the client if KIP-320 is not supported by the client.

@dajac Yeah, this is a tough one. I think one perspective we could take is that consumers which have not implemented the protocol should not be supported. So perhaps we should only check whether leader epoch is provided and reject the request otherwise. After all , even if follower fetching is intended, the user might still encounter spurious OFFSET_OUT_OF_RANGE errors if the client has not implemented the protocol. Alternatively, maybe we should not return that error in the first place for clients which are not providing the leader epoch? We could return OFFSET_NOT_AVAILABLE instead if the leader epoch is -1. Would that work?

I think one perspective we could take is that consumers which have not implemented the protocol should not be supported. So perhaps we should only check whether leader epoch is provided and reject the request otherwise. After all, even if follower fetching is intended, the user might still encounter spurious OFFSET_OUT_OF_RANGE errors if the client has not implemented the protocol.

That would be ideal but that seems quite brutal for existing deployment. There are definitely a few users using those clients with follower fetching. I wonder if we should consider something like bumping the fetch request version and do this as well in order to strengthen the protocol. That would at least give a chance to those clients to rectify their implementation if they want to use newer fetch versions in the future. I am not sure that it is worth it though.

Alternatively, maybe we should not return that error in the first place from followers if the client is not providing the leader epoch? We could return OFFSET_NOT_AVAILABLE instead if the leader epoch is -1. Would that work?

librdkafka does not handle OFFSET_NOT_AVAILABLE well, unfortunately. It only retries the fetch request with a small backoff when it receives it but it does not refresh its metadata so the client would likely not rediscover the correct leader.
It seems that there is no good solution to mitigate the issue for those existing clients. We've discussed the following approaches:

Disallow fetch from follower when there is no replica selector. This seems pretty safe but could be an issue when the cluster is rolled to enable the selector. We could make the config dynamic to mitigate this. Clients using follower fetching would still be subject to the issue though.
Disallow fetch from follower when there is no leader epoch and no rack id in the request. This would ensure that existing clients that does not use follower fetching are safe. However, we have no guarantee that it would not break existing deployment because a selector does not necessary use rack.id. Clients using follower fetching would still be subject to the issue though.
Disallow fetch from follower when there is no leader epoch in the request. This would prevent existing clients from working.

Wasn't the 4th option:

Return OFFSET_NOT_AVAILABLE instead of OFFSET_OUT_OF_RANGE if the leader epoch is not provided?

librdkafka does not handle OFFSET_NOT_AVAILABLE well, unfortunately. It only retries the fetch request with a small backoff when it receives it but it does not refresh its metadata so the client would likely not rediscover the correct leader.

Two things:

We could change librdkafka to refresh its metadata and backport the fix to older versions.
How long does librdkafka retry for before giving up?

Yes, that's the 4th option.

Yeah, that's right but librdkafka is not the only one concerned. It is not clear to me how Sarama would handle this for instance. The compatibility story is quite complicated here.
5 minutes.

librdkafka does not handle OFFSET_NOT_AVAILABLE well, unfortunately. It only retries the fetch request with a small backoff when it receives it but it does not refresh its metadata so the client would likely not rediscover the correct leader.

Hmm, perhaps that handling is good enough? In the event that the out of range error was spurious, the follower would soon catch up to the leader. I assume that is the common case unless unclean election is enabled. It does not seem like a big problem if there is a delay finding a genuine out of range case.

Yeah. The delay may be acceptable in this case. I am actually more concerned by the other clients out there. It is hard to really know if this change would be OK for all of them, knowing that some clients have partially implemented the fetch protocol.

Using OFFSET_NOT_AVAILABLE on followers seems to be the right direction I think. It is the only approach we've discussed which addresses the spurious out of range issue for users who want follower fetching on these clients. This error can already be returned after a normal leader change, so clients must handle it.
Perhaps we could make the behavior more precise instead of blinding returning OFFSET_NOT_AVAILABLE for all fetches above the local end offset. The follower doesn't know if the offset is out of range or not since its log is always behind the leader, so we can argue it is incorrect for it to return the error in the first place. But it can find out by querying the leader. In fact, the Fetch request already returns the leader's end offset. Could we make use of that? Maybe we could hold out-of-range fetches in purgatory until the follower knows whether or not the leader has the requested offset?

Jason and I discussed offline. Returning OFFSET_NOT_AVAILABLE is not really appropriate here because clients may not expect it on the fetch path. The issue is that we have never implemented it on the fetch path so it is not clear what the impact of returning it on non-java clients would be. We discussed another alternative that I prototypes here: https://github.com/apache/kafka/pull/12734.


markdownlint-cli removed?

Why was this linter removed?  IMO, its the best structural markdown analyzer that was available in your list?  If it was removed in earnest, is there a way for me to add this individually to the list of markdown linters?

I can't find any indication that it has ever been included. There's a separate tool called mdl.


Inconsistencies in certificate behavior in Netty.

Expected behavior
N/A
Actual behavior
Steps to reproduce
Hello guys, I'd like to understand whether the behavior shown below is expected and there is some logic behind it, or it's inconsistent (that's the way it looks to me).
To reproduce, we're using 2 certificates, one with private key
- ca.crt
- test.crt, test.key
, where "test.crt" is signed by "ca.crt".
certificates.zip
Let's review 4 scenarios, each presenting a unique behavior:
1.1) Mutual TLS
- Client is using
KeyManagerFactory with { "test.crt", "test.key" }
InsecureTrustManagerFactory.INSTANCE
-Server is using
new SelfSignedCertificate();
TrustManagerFactory with { "test.crt" }
Behavior - TLS handshake error on server "Empty client certificate chain"

1.2) Mutual TLS
- Client is using
KeyManagerFactory with { "test.crt", "test.key" }
InsecureTrustManagerFactory.INSTANCE
-Server is using
new SelfSignedCertificate();
TrustManagerFactory with { "ca.crt" }
Behavior - works normally

2.1) One-way TLS
- Client is using
TrustManagerFactory with { "test.crt" }
-Server is using
KeyManagerFactory with { "test.crt", "test.key" }
Behavior - works normally

2.2) One-way TLS
- Client is using
TrustManagerFactory with { "ca.crt" }
-Server is using
KeyManagerFactory with { "test.crt", "test.key" }
Behavior - TLS handshake error on client "Extended key usage does not permit use for TLS server authentication"

Minimal yet complete reproducer code (or URL to code)
N/A
Netty version
4.1
JVM version (e.g. java -version)
openjdk version "17.0.12" 2024-07-16
OS version (e.g. uname -a)
Linux dev1 6.8.0-41-generic #41-Ubuntu SMP PREEMPT_DYNAMIC Fri Aug  2 20:41:06 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

1.1) Mutual TLS
- Client is using
KeyManagerFactory with { "test.crt", "test.key" }
InsecureTrustManagerFactory.INSTANCE
-Server is using
new SelfSignedCertificate();
TrustManagerFactory with { "test.crt" }

Behavior - TLS handshake error on server "Empty client certificate chain"

Correct. Server requests a cert path from the client, and that request includes a list of CAs that will be accepted. In this case server thinks test.crt is a CA. The client will then be looking for a cert that is issued by test.crt and finds none, hence returns an empty cert path that the server rejects.
1.2) Mutual TLS
- Client is using
KeyManagerFactory with { "test.crt", "test.key" }
InsecureTrustManagerFactory.INSTANCE
-Server is using
new SelfSignedCertificate();
TrustManagerFactory with { "ca.crt" }

Behavior - works normally

Correct. You configured stuff correctly.
2.1) One-way TLS
- Client is using
TrustManagerFactory with { "test.crt" }
-Server is using
KeyManagerFactory with { "test.crt", "test.key" }

Behavior - works normally

Correct. The client considers test.crt to be a Trust Anchor, and can thus immediately trust it without doing any validation. This is effectively certificate pinning.
2.2) One-way TLS
- Client is using
TrustManagerFactory with { "ca.crt" }
-Server is using
KeyManagerFactory with { "test.crt", "test.key" }

Behavior - TLS handshake error on client "Extended key usage does not permit use for TLS server authentication"

Correct. The test.crt only include TLS Web Client Authentication in its extended key usage, so the server is not allowed to use this cert and the client refuses to validate it.

@chrisvest Just to get this straight, according to you,
1.1 In this case server thinks test.crt is a CA. The client will then be looking for a cert that is issued by test.crt and finds none, hence returns an empty cert path that the server rejects.
and
2.1 The client considers test.crt to be a Trust Anchor, and can thus immediately trust it without doing any validation. This is effectively certificate pinning.
is not inconsistent?

@codekrolik2 the protocol is not symmetric between client and server. The server always sends the cert you configure, so the client can check it. The other way, clients only send certs when requested, and the request includes what CAs are accepted, most clients will not include those CAs in the returned list of certs because the CA should already be known to both peers. Then you end up with an empty path, which is an error. Clients and servers are different in important ways, and it shows like this in your tests.

Ok, I was just making sure that it's expected behavior when client does certificate pinning, and server doesn't.
What I read above is that there is no consistency that's expected between the behaviour of similar concepts on server and client.
Therefore, I shouldn't expect server to pin client certificates - it's expected that server won't pin, even though I can pin server certificates on the client.
@chrisvest Apparently to achieve similar pinning logic on server I need to implement a custom TLS handshake handler?
Thanks!


Move location of font files

@ascott1 Try pulling this down. I believe I had the fonts in the wrong dir. :/
Fixes #178.


Show hint in resolution failure on Forbidden (403) or Unauthorized (401)

Summary
Closes https://github.com/astral-sh/uv/issues/8167.

This was really bothering me haha.


Add not[x] type (instance of univalence)?

Might be worthwhile to add this type as an example of a non-trivial higher dimensional type.

You could add "isovalence" instead or in addition, although I am not sure whether there are enough rules implemented to actually prove that things are strict isomorphisms. As a stopgap measure, you could simply add a rule stating that not is a strict isomorphism.

@cangiuli I believe we can already prove that not is a strict isomorphism, but only for the strict booleans. I don't think we can prove it yet for weak bool, because we don't yet have any way to eliminate weak bool into an equality judgment.

That sounds right. I also plan to work on some low-hanging fruit (strict natural numbers, coproducts, etc.) this summer; those will provide much more fodder for strict univalence.

I'm going to close this ticket, since I prefer to just implement strict univalence.


testing workflow

[x] #16
[ ] #20
[ ] test: echo local pl local logs
[ ] test: repo load store emit
[ ] #21

#24 and #23 motor debug and mesh updated with #20 and #16


add support "app.kubernetes.io/component" to find apiserver pod

What type of PR is this?
kind/enhancement
What this PR does / why we need it:
support using component=kube-apiserver, or app.kubernetes.io/component=kube-apiserver to find apiserver pod
Which issue(s) this PR fixes:

Fixes # https://github.com/clusternet/clusternet/issues/223
Special notes for your reviewer:

@fangyuchen86 Please rebase your codes with latest main branch.

Codecov Report

Merging #227 (092b0a6) into main (fd045f2) will increase coverage by 0.21%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #227      +/-   ##
==========================================
+ Coverage   10.26%   10.48%   +0.21%
==========================================
  Files          39       39
  Lines        4237     4254      +17
==========================================
+ Hits          435      446      +11
- Misses       3771     3775       +4
- Partials       31       33       +2

Impacted Files
Coverage Δ

...ernet/clusternet/pkg/hub/apiserver/shadow/utils.go
89.28% <0.00%> (-10.72%)
:arrow_down:

.../pkg/controllers/clusters/clusterstatus/network.go
38.94% <0.00%> (+9.46%)
: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 fd045f2...092b0a6. Read the comment docs.


Fix IST0117 message false positive in 1.20

Please provide a description of this PR:
Based on https://github.com/istio/istio/pull/46295
https://github.com/istio/istio.io/pull/13717

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
7a80adb466bf9c60c3a85640ef43b1b30f3f7ea9
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
7a80adb466bf9c60c3a85640ef43b1b30f3f7ea9
link
true
/test lint

gencheck_istio
7a80adb466bf9c60c3a85640ef43b1b30f3f7ea9
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-pilot_istio
e4128a0d9c4b48baf31eacf57d876d2d40991418
link
true
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Failing to run simulation mode.

I had built Teaclave in simulation mode
docker run --rm -v $(pwd):/teaclave -w /teaclave   -it teaclave/teaclave-build-ubuntu-1804-sgx-2.9.1:latest    bash -c ". /root/.cargo/env && \      mkdir -p build && cd build && \      . /opt/sgxsdk/environment && \      cmake -DTEST_MODE=ON -DSGX_SIM_MODE=ON .. && \      make   --trace"
I want to launch all services after building Teaclave.
But when i run this code
 (cd docker && docker-compose -f docker-compose-ubuntu-1804.yml up --build)
and it show
ERROR: for 1d54dc72d75f_teaclave-storage-service  Cannot start service teaclave-storage-service: linux runtime spec devices: error gathering device information while adding custom device "/dev/isgx": no such file or director
So,why the simulation mode is not work.May be i do wrong step?
My os : Ubuntu 18.04.4 LTS
My CPU:Intel(R) Xeon(R) CPU E5-2620 v4 @ 2.10GHz
There is no sgx tech in the CPU.

Under simulation mode, you also need to change the docker-compose-ubuntu-1804.yml file accordingly to remove unused device/volume mappings.
That is, comment out these lines:
      - type: bind
        source: /var/run/aesmd/aesm.socket
        target: /var/run/aesmd/aesm.socket
    devices:
      - /dev/isgx

It works!
Thank you very much.


[Batch mode] [rdar://40712614 ] Remove “using batch mode” remark.

@swift-ci please smoke test

@swift-ci please smoke test

@jrose-apple At Bob's suggestion, I have pointed this PR in your direction. A simple revert did not work.

@swift-ci please smoke test


Typos and grammar issues on website

I was reading about Picnic on the website at http://picnicss.com/ and I noticed there are a lot of minor grammar issues and no space before "terms and condition" (see screenshot below). It's just little things, but I think presentation matters.

I'd love to help fix these little issues since I'm just starting to try my hand at contributing to open source projects; I think these little things are a good place for me to start, and I'm very interested in this project!

Sure! I think spacing issues come from migrating to Jade and then Pug (did I migrate to Pug?) and my editor.

I see what you mean about the spacing issue.
Also, some fixes to grammar seem to have been made three days ago, but they just don't appear on the website because they haven't been migrated to the index.html. How would we do this (or would you handle it later on)?

For minor fixes (missing an 's) I would just wait until the next small update, but since you batched few of them I just recompiled it and updated it.
If you want to go that extra step, you'd need to run grunt and it will create the proper HTML. Then I still need to pull from here and push to two Github Repositories (two different PR's to each wouldn't work).
Thank you for the fixes!

Ah, I see. Thank you for explaining and no problem!


Database tracker is able to detect changed values

Story
When user tells the database tracker to create a commit (see #41)
Then it detects the changes
and converts them to blocks in a jawn commit
And writes the commit to the feed in jawn
Additional Info
What kind of Database?
For now we will assume sqlite.
How does it figure out what changes have occurred?
The database tracker will need to be able to compare what's in the jawn feed and what's in the database.
Either way, the feed should have info about when data was last pulled from the database.
Option 1: Parse the database log
Option 2: If the database tables keep a "last updated" timestamp then we can just query for things that have been updated since the last time the feed was updated. -- we can either ask the Open Data team whether their data has this info, or we can read up on ArcGiS ourselves.

Standardized SQL functions in ArcGIS Online

Start with one table toy example.
Then move on to several tables. How do you represent multiple tables of data in one feed? Or do we separate tables into multiple feeds?

My current thinking on how to do this: Storing Tabular Data in a DAG.  If you need context, this is from the larger Replication Patterns deck, which outlines how Git structures its DAG.
I do have some new tweaks to the ideas in that deck, based on the way "live" feeds work in hypercore, but the slides get at the essential idea.

@flyingzumwalt @CameronSima
Just wanted to give you an update on what I'm going. Here is my current thinking for how to detect updates, assuming we have a lastUpdate column in each table in the db.

var sqlite3 = require('sqlite3').verbose()
 var fs = require('fs')

  module.exports = DatabaseTracker

  function DatabaseTracker (dbfile) {
    var exists = fs.existsSync(dbfile)
    this.db = new sqlite3.Database(dbfile)
  }

  DatabaseTracker.prototype.getAllTables = function (callback) {
  	var query = "Select name FROM sqlite_master WHERE type = 'table'"
  	var dbTracker  = this
  	this.db.serialize(function () {
  		dbTracker.db.all(query, function (err, tables) {
  			if (err) {
  				throw new Error(err)
  			}

  			callback(tables)
  		})
  	})
  }

  DatabaseTracker.prototype.getChanges = function (date, callback) {
  	var currentDb = this.db

  	function findAllChanges(tables) {
  		var changes = {}
  		var tableNumber = tables.length
  		tables.forEach(function (item, index) {
  			//TODO: might be better to use db.All() instead of statement
  			var statement = currentDb.prepare('SELECT * FROM (?) t WHERE t.lastUpdate > ' + date)
  			statement.all(item.name, function (err, rows) {

  				if (err) {
  					console.log(err)
  				}

  				changes[item.name] = rows
  				statement.finalize()
  				if(changes.length = tableNumber) {
  					callback(changes)
  				}
  			})
  		})
  	}

    this.getAllTables(findAllChanges)
  }

We should put the database tracker code into a separate repository. I set up a github repo here: https://github.com/CfABrigadePhiladelphia/jawn-db-tracker (cc @mels065)


[Event Request] Codeunit 5988 "Serv-Documents Mgt." - OnBeforeGetAndCheckCustomer,OnGetAndCheckCustomerBeforeShipment

Hi, would it be possible to get an event OnBeforeGetAndCheckCustomer and OnGetAndCheckCustomerBeforeShipment in codeunit 5988 (Serv-Documents Mgt.), like this:
local procedure GetAndCheckCustomer()
    var
        Cust: Record Customer;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeGetAndCheckCustomer(ServHeader,IsHandled);
        if IsHandled then
            exit;
        with ServHeader do begin
            Cust.Get("Customer No.");

            IsHandled := false;
            OnGetAndCheckCustomerBeforeShipment(ServHeader,IsHandled);
            if not IsHandled then
              if Ship or ServMgtSetup."Shipment on Invoice" and
                 ("Document Type" = "Document Type"::Invoice)
              then begin
                  ServLine.Reset();
                  ServLine.SetRange("Document Type", "Document Type");
                  ServLine.SetRange("Document No.", "No.");
                  ServLine.SetFilter("Qty. to Ship", '<>0');
                  ServLine.SetRange("Shipment No.", '');
                  if not ServLine.IsEmpty then
                      Cust.CheckBlockedCustOnDocs(Cust, "Document Type", true, true);
              end else
                  Cust.CheckBlockedCustOnDocs(Cust, "Document Type", false, true);

            if "Bill-to Customer No." <> "Customer No." then begin
                Cust.Get("Bill-to Customer No.");
                IsHandled := false;
                OnGetAndCheckCustomerBeforeShipment(ServHeader,IsHandled);
                if not IsHandled then
                    if Ship or ServMgtSetup."Shipment on Invoice" and
                       ("Document Type" = "Document Type"::Invoice)
                    then begin
                        ServLine.Reset();
                        ServLine.SetRange("Document Type", "Document Type");
                        ServLine.SetRange("Document No.", "No.");
                        ServLine.SetFilter("Qty. to Ship", '<>0');
                        if not ServLine.IsEmpty then
                            Cust.CheckBlockedCustOnDocs(Cust, "Document Type", true, true);
                    end else
                        Cust.CheckBlockedCustOnDocs(Cust, "Document Type", false, true);
            end;
            ServLine.Reset();
        end;
    end;

[IntegrationEvent(false, false)]
local procedure OnBeforeGetAndCheckCustomer(var ServiceHeader: Record "Service Header" temporary; var IsHandled: Boolean)
begin
end;

[IntegrationEvent(false, false)]
local procedure OnGetAndCheckCustomerBeforeShipment(var ServiceHeader: Record "Service Header" temporary; var IsHandled: Boolean)
begin
end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Add description of VTCLab Media Analyzer to 'Tools" section

Sorry, wrong file. Will update contents.json soon


Skip kubeadm preflight checks.

A race condition in one of kubeadm's preflight checks has been causing occasional failures of masters/nodes to come up correctly.
Since we provide a completely automated setup, any failed preflight checks don't offer much value since the user isn't interacting with kubeadm to see its error messages and address the issues, so we should
disable them anyway.
CC @mikedanese

LGTM after rebase


Localization: Add french language

Are the month and weekday names always capitalized in French? From quickly searching, that doesn't appear to be the case. For months, on what appears to be the official website (using Google Translate; I don't speak French):

Since they are common nouns, they must not, except at the beginning of the sentence, be written with a capital letter and they take, one and the other, the mark of the plural.

Also, what level experience do you have with French (personally)?

@jhpratt You are right, months and weekdays are not capitalized in french. I did copy paste the english translation and though that everything should be capitalize (I didn't see that spanish ones weren't).
I am French so i'm a native speaker :)
(I will fix the PR tomorrow)

Awesome, thanks! I'll merge it in once that's updated.

I did just push an update.
I ordered the enum alphabetically and use the same translations as C strftime (found here) as the doc states:

Time's formatting behavior is based on strftime in C

Complete nitpick, but can the periods/full stops be removed from the abbreviations?
Side note: I wish GitHub gave me some way to edit the code directly. I could merge the PR with edits locally, but then this wouldn't be indicated as merged.

I just removed those periods. Those are used in french to indacate that it's an avbbreviation in that context.

It's the same situation in English, generally speaking. I just think that it's trivial to add the periods if desired, but it's not as easy to remove them.
Thanks for the changes! Merging.

But for some short month the abbreviation is the same as the none abbreviated form so it might be harder to add correctly the period if needed :)

Hmm, you're right. I'll think about this more for sure, as that's also the case for English. Perhaps it could be another modifier on the specifier.

@homersimpsons After looking into glibc some, it looks like their localization files are quite accurate. As such, I'll actually be pulling in data for plenty of other languages as well, leaving punctuation as-is.
This will be done via a build script, as otherwise it would be an unwieldy file to maintain.
Thanks again for the reference.


panic: assignment to entry in nil map

Incident - panic: assignment to entry in nil map

The atlantic-2 validator node crashed with the following panic message.
If you try to add or modify a key-value pair in a map that has not been initialized or has been set to nil, you will get the "panic: assignment to entry in nil map" error message.

It is assumed that the map was not initialized properly. This can harm the stability of the validator node and should be fixed.

9:22AM INF entering precommit step with invalid args current=5168653/0/RoundStepPrecommit entryLabel=precommit-two-thirds expected=#5168653/0 height=5168653 module=consensus round=0d=0 current=5168653/0/RoundStepPrecommit height=5168653 module=consensus
9:22AM INF commit is for a locked block; set ProposalBlock=LockedBlock block_hash=4E826AAFCEF37FF39561E3914E8B847D1EF9F2890D59C79D9D54D143B39077EB commit_round=0 height=5168653 module=consensus7FF39561E3914E8B847D1EF9F2890D59C79D9D54D143B39077EB height=5168653 module=consensus num_txs=25 root=87A0087F516C9B5C2F41B2BBAEBBAFB1CB0FAC5B28C385C0E6A3BC800CF97CC04268: connect: connection refused
9:22AM INF FinalizeBlock took 103ms
9:22AM INF finalized block block_app_hash=CE9AE13D7F6BADD7DF27AE93255310487842800E537ED99E40795ABC8C22310D height=5168653 latency_ms=103 module=state num_txs_res=25 num_val_updates=07F6BADD7DF27AE93255310487842800E537ED99E40795ABC8C22310D
9:22AM INF committed state block_app_hash=87A0087F516C9B5C2F41B2BBAEBBAFB1CB0FAC5B28C385C0E6A3BC800CF97CC0 height=5168653 module=state num_txs=25INF signed and pushed vote height=5168654 module=consensus round=0 vote={"block_id":{"hash":"E22A9ACC4B92031CC41B2DEE40E023D12EA10661C412FB0F322F1A9D03C27903","parts":{"hash":"7A9564C34873BAC1306B32F4705B2BBDE06481638868FD1A957DCDBF55D57A71","total":1}},"extension":null,"extension_signature":null,"height":"5168654","round":0,"signature":"LKZyDLtDsOCbuTaN8vOkGzX1qJHmQ/4bgJXY7rlOGjUkKuVKtpA/XbShQuZ+7K1d7lhvcGIbTEj3IiXQXwwVCg==","timestamp":"2023-04-08T07:22:49.132903589Z","type":2,"validator_address":"6557E859315448DB2DF93755C7D03F5CBF3BCC28","validator_index":7}
9:22AM INF entering precommit step with invalid args current=5168654/0/RoundStepPrecommit entryLabel=prevote-future expected=#5168654/0 height=5168654 module=consensus round=0
panic: assignment to entry in nil map

goroutine 9557 [running]:
github.com/tendermint/tendermint/internal/p2p.(*PeerManager).Add(0xc000c62280, {{0xc06f02ba48, 0x28}, {0xc06f02ba40, 0x5}, {0xc06f02ba71, 0x1b}, 0x6820, {0x0, 0x0}})
        /usr/local/go/pkg/mod/github.com/sei-protocol/sei-tendermint@v0.2.2/internal/p2p/peermanager.go:435 +0x5a5
github.com/tendermint/tendermint/internal/p2p/pex.(*Reactor).handlePexMessage(0xc0003c5ad0, {0x2998db0, 0xc00055eb00}, 0xc13b5c5280, 0xc00719daa0?)
        /usr/local/go/pkg/mod/github.com/sei-protocol/sei-tendermint@v0.2.2/internal/p2p/pex/reactor.go:303 +0x7ff
github.com/tendermint/tendermint/internal/p2p/pex.(*Reactor).processPexCh(0xc0003c5ad0, {0x2998db0?, 0xc00055eb00}, 0xc00719daa0)
        /usr/local/go/pkg/mod/github.com/sei-protocol/sei-tendermint@v0.2.2/internal/p2p/pex/reactor.go:221 +0x528
created by github.com/tendermint/tendermint/internal/p2p/pex.(*Reactor).OnStart
        /usr/local/go/pkg/mod/github.com/sei-protocol/sei-tendermint@v0.2.2/internal/p2p/pex/reactor.go:152 +0xd0

Thanks for this report! This should be resolved in https://github.com/sei-protocol/sei-tendermint/pull/134


You need to run Homebrew Launcher again in order to Udsploit and Safehax appear

The guide says to remove the SD card with Homebrew Launcher running, place Udsploit and Safehax in the 3ds folder, then put the SD back
However, Udsploit and Safehax won't appear after doing that, you need to close homebrew launcher and then open it again. It would be better if the guide told you to place all the files (Starter Kit, Otherapp, Soundhax, Udsploit, and Safehax) at the same time, so you will not need to run Soundhax one more time.

For example


Bitbucket as publish target

Any chance of adding bitbucket as a publish target?

Can you give more details on the help that you need.  I will see if i can be of any help. I need to see if i have the required environment setup.

The "help wanted" label is on this issue because we would need to have someone file a pull request for the feature. We don't currently have any documentation on how to create publisher plugins for 6.x, you'd have to look at the existing ones in the repository.

@malept Is that the reason why I'm not able to publish it to GitHub? I understood how "helpwanted" works now. Thank you

Publishing to a git repository should be in a separate issue, although I don't know whether we'd want to support that as a first-party plugin.


Question 3

Question 3

We can add graphs of pdfs with the estimated parameters to the histogram for better understanding.

We can add graphs of pdfs with the estimated parameters to the histogram for better understanding.
That would be nice.  We can add it.


Add the functionality for conflict error code generation in case of a Email-ID redundancy

As a Developer
I need a conflict error code to be generated when a customer tries to register with an email id already in use
So that there are no redundant customer accounts with the same email ID.
Assumptions

the base URL would be /customers
API present that allows for customer registration

Acceptance Criteria
Given a request to register a new customer
When an email ID is used that is already in use
Then HTTP 409 Conflict error code should be generated and returned

Perhaps add the enhancement label to this issue?

Should the URL be something other than /customers? This function should be used within the POST /customers to check whether the email address already belongs to a customer. Maybe something like /customers/email which would return a boolean if the address already exits in the persistent memory..?


Can't remove tick marks on X-Axis

I would like to make the X-Axis on a Lien graph invisible. I have managed to remove everything except the tick marks (ignore the blue container div border underneath)

My XAxis is defined as follows:
  <XAxis
    gridLineWidth='0'
    labels='None'
    lineWidth='0'
    minorTickLength='0'
    tickLength='0'
  />

Normally in HighCharts this would result in the whole axis disappearing. Any help appreciated.

The zero values need to be passed as integers, not strings.
  <XAxis
    gridLineWidth={0}
    labels='None'
    lineWidth={0}
    minorTickLength={0}
    tickLength={0}
  />

Thanks again


bugfix(#63): Fix combined event test

Codecov Report

Merging #64 into master will increase coverage by 0.95%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   96.48%   97.43%   +0.95%
==========================================
  Files           5        5
  Lines         199      195       -4
==========================================
- Hits          192      190       -2
+ Misses          7        5       -2

Impacted Files
Coverage Δ

test/unit/NotifierBuilderTests.cpp
97.25% <100%> (+1.01%)
: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 a98fe8b...54e5a7f. Read the comment docs.


Add DB schema version check

Add a Version check and auto-update the db if needed.  The frame work is in place already, thanks to Aric's work to move data from neon to CIAB

I believe this is now resolved.

I believe this is finished.


Compiling a .js file and serving it later

Hey there FranckFreiburger,
First off the bat, I wanted to thank you for the amazing tool. It's great that you managed to connect so many different things into one functional repository.
We wanted to develop sites using primarily this and avoid the build step entirely. However, Google Page Speed insights has a bad score on Largest Contentful Paint due to processing occuring in the library.
Tried running the withCache example as a prototype, but it shoots an error on execution.
Uncaught (in promise) TypeError: null has no properties
    e vue3-sfc-loader.js:113
    e vue3-sfc-loader.js:113
    async*e</he/r[i]< vue3-sfc-loader.js:113
    he vue3-sfc-loader.js:113
    we vue3-sfc-loader.js:113

Is there a way to serialize the compiled modules somehow? I see that loadModule returns a promise with scopeId, render and setup functions - is there a way to serialize this and deserialize it later?
In essence, vue-sfc-compiler can compile a .js file that can be imported into the browser later on. Can a similar .js file be compiled somehow with your loader and then cached?
Any other approach or insight is very much appreciated!
Best regards,
Aleksandar

Hi Saleiv
vue3-scf-loder cannot (an will not) pre-compile assets without they are used.
However, since v0.9.2+ vue3-scf-loder can run server-side to compile and cache assets.
see: https://github.com/FranckFreiburger/vue3-sfc-loader/discussions/178


Wrong self convention for ToValueOptional, ToGlibContainerFromSlice

error: methods with the following characteristics: (`to_*` and `self` type is not `Copy`) usually take `self` by reference
   --> gstreamer/src/miniobject.rs:462:34
    |
462 |               fn to_value_optional(s: Option<&Self>) -> $crate::glib::Value {
    |                                    ^
error: methods with the following characteristics: (`to_*` and `self` type is not `Copy`) usually take `self` by reference
   --> gstreamer/src/miniobject.rs:167:40
    |
167 |               fn to_glib_none_from_slice(t: &'a [$name]) -> (*mut *mut $ffi_name, Self::Storage) {
    |                                          ^

CC @GuillaumeGomez @bilelmoussaoui

It seems to be that it is the lint that is wrong

to_value_optional(s: Option<&Self>) is taking &Self...
to_glib_none_from_slice is taking &[]


Wrong typing of getSeoUrls in api-client

The type here is wrong, it should be the following type
Promise<EntityResult<"seo_url", { apiAlias: string; seoPathInfo: string }[]>>
I'll provide a PR later.

👌 thank you! Closing as resolved!


Bug_Fixed_933

Fixes #1933
Describe the changes you have made in this PR -
If we cancel the new circuit dialog then also it will create new circuit
simple bug just add the default name parameter prompt
Screenshots of the changes (If any) -Before

if we cancel

now

if we cancel

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage decreased (-0.05%) to 84.66% when pulling b5471dfa9b3fbab48f8de8b22493135c64732164 on ZadeAbhishek:New-Circuit-Dialog#1933 into f05c0617683d7a488061380271fae77775c765b6 on CircuitVerse:master.

Thanks!


Enhance the optimization pipeline

We do not perform target specific optimization (e.g., fast-math or contractions) in our generic O3 pipeline.

Extract clang's optimization flags from command line and store in IR to replicate more closely the AOT pipeline.


upcoming_house_floor.py failing because url should be https, not http.

This line:     base_url = 'http://docs.house.gov/floor/Download.aspx?file=/billsthisweek/'
should be https://
thanks.

Fixed, thanks!


want to set showNodeSelectionBox to true in selecting

Expected Behavior

showNodeSelectionBox默认设置为true
文档更新

Possible Solution

Additional Context

@NewByVector 修改默认值时不兼容的更新，不建议做，影响线上业务。


Added link to a bug which prevents PWA update

There is a bug in PWA manifest generator which prevents PWA cache update. Added a link to an issues for explanation and possible workaround.

We prefer not to document short-term framework problems that will be patched quickly unless they're serious and widespread. @mkArtakMSFT, will the next patch release with the fix for this land soon?

Adding more 'keywords' to the issue description may be the alternative for now. I was able to find the issue on github only after going through the target's source.

Update ... Although the patch release date isn't published publicly ahead of time, I do know the date and can say that it won't be long 🕙. We'll let the patch take care of this.


As a User, I want to see CDH Data Fellows listed on the Affiliates and Students pages so I know who received a fellowship.

Is your feature request related to a problem? Please describe.
The new cohort of CDH Data Fellows need to be entered into the system, including their projects and profiles. The projects should work the same as other projects (and as DCGs worked in the past), but getting the profiles onto the appropriate person list pages is more complicated.

Data Fellows who are students should appear on the students list page
Data Fellows who are staff/faculty should appear on the affiliates list page
In both cases, the title displayed should be "Data Fellow"

For more info, see slack conversation.
Describe the solution you'd like
Most of the logic to determine who goes on what page lives in the PersonQuerySet:
https://github.com/Princeton-CDH/cdh-web/blob/27537f7eeed402c0c4d90284a088cdf01b7f09e9/cdhweb/people/models.py#L44-L190
Ideally, the affiliates() and student_affiliates() methods would change to incorporate Data Fellows into the queryset.
To change the actual title that's displayed, we need to edit the display_label() method on the PersonListView subclasses, for example:
https://github.com/Princeton-CDH/cdh-web/blob/27537f7eeed402c0c4d90284a088cdf01b7f09e9/cdhweb/people/views.py#L193-L203
For students this may work without modification, since "Data Fellow" is a title (and is likely their most recent title), but for faculty/staff (above) it almost certainly won't, because we normally only display grant awards on that page, and the Data Fellowship is not a grant.
Describe alternatives you've considered
The strict separation between students and affiliates deserves more thought, and the two categories may continue to become blurrier/be combined in the future.

Let's do the simple version of this, since we still want to display Data Fellow projects and have a grant history for them.

modify display_label to generalize for fellowships — if grant ends in "fellowship", change the label to "fellow"
make sure student list uses the same display label logic

Looks good!


fix badge for Instagram

Pre-flight Checklist
Please ensure you've completed all of the following.

[x] I have read the Contributing Guidelines for this project.
[x] I agree to follow the Code of Conduct that this project adheres to.

Description of Change
I have fixed the badge of notifications for the Instagram and Instagram-direct-messages applications. The query selector now directly retrieves the value within the  that contains the number of notifications.

Thank you for this fix!
To be able to merge this, please update the version in package.json of the instagram recipe as described in https://github.com/ferdium/ferdium-recipes/blob/main/docs/updating.md#4-updating-the-version-number

Thank you for feedback !
I update version and complet the fix

I running the commande pnpm lint:fix && pnpm reformat-files && pnpm package on my local and check pass


[docs] Remove one inline-style

The usage of the paddingInlineStart CSS property is flooding all the other errors, #25669 was hard to spot. This the only occurrence of a direction-aware CSS property we have in the codebase. The change didn't seem to cost much, it's unclear when validator will solve the root issue. My hope is that it will help us catch other issues on the pages.
I have opened https://github.com/validator/validator/issues/1134 upstream, which is probably the most important part.

Edit, this PR now focuses on remove usages of inline-style: https://github.com/mui-org/material-ui/issues/19938#issuecomment-645893048.

What are you actually fixing here?

@eps1lon I have updated the description. I hope it's clearer now.

There is no sign that this will be fixed upstream any time soon. Could we move forward at the end of the week if upstream stays silent, by pragmatism? 1. the JS overhead is negligeable, 2. having to systematically filter messages to look into pages reports is not practical.

by pragmatism

The pragmatic solution is you adjusting your workflow. How is disabling the warning not working for you?

@eps1lon I see three pains on HEAD

a developer has to learn which warning is correct or wrong and remember it. It's mentally draining

a developer has to learn how to hide a warning and do it every single time. It's time-consuming

we have to teach it to everybody else.

are IMHO bad enough but I think that 3. is where we lost everybody

We wanted to move away from inline-style, the latest commit allows to.

We wanted to move away from inline-style, the latest commit is one step in this direction.

I don't believe you. We have multiple issues where you don't think this way and changes where you did apply inline styles.

The primary problem I'm going after is the noise in the w3c validator output. The other advantage are IMHO secondary. I think that it depends on how one evaluates the benefits. For instance, what about browser support https://caniuse.com/mdn-css_properties_padding-inline-start. I'm merging as there are no little to downsides with the latest approach.

The primary problem I'm going after is the noise in the w3c validator output.

This is a you-problem. Nothing more.

For instance, browser support caniuse.com/mdn-css_properties_padding-inline-start.

Do you realize that you're falling into post-hoc rationalizaton? IE11 support is not an issue for you generally. When it is useful to justify a faulty implementation, it is.

I'm merging as there are no little, to no downsides with the latest approach.

Why do you merge something with downsides?


feat(k8s/configuration): rename add ingress controller button and changed information text

Close #4535

/azp run

@pull-dog up

@pull-dog down

@pull-dog up

@pull-dog down


Intellisense doesn't update when a new column is added to a table

Issue Type: Bug
Running:
USE [myDatabase]
GO
ALTER TABLE dbo.myTable ADD newColumn DATETIME NULL;

results in intellisense showing "newColumn" is not valid. I have to quit SQL Op Studio and reopen for it to refresh.
SQL Operations Studio version: sqlops 0.31.4 (e026ab85a72c190cc4ccc834503aa9fe36dd6407, 2018-07-18T02:14:21.826Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6400 CPU @ 2.70GHz (4 x 2712)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
15.91GB (7.68GB free)

Process Argv
C:\Program Files\SQL Operations Studio\sqlops.exe

Screen Reader
no

VM
0%

Extensions: none

@MrSimonC you should be able to run the Refresh IntelliSense Cache  command to manually trigger an update.  I'll leave this open as a suggestion to handle the drift detection automatically.

As suggested, "Refresh Intellisense Cache" should manually trigger an update to intellisense.
Closing as no-op.


update rmag.interactive_verwey_estimate() to not get 'Series.argmin' is deprecated FutureWarning

Get this FutureWarning when first running rmag.interactive_verwey_estimate() on EarthRef JupyterHub. Note that it goes away when running the function again. But need to fix code so that we don't see this.
/opt/tljh/user/lib/python3.6/site-packages/numpy/core/fromnumeric.py:51: FutureWarning:

The current behaviour of 'Series.argmin' is deprecated, use 'idxmin'
instead.
The behavior of 'argmin' will be corrected to return the positional
minimum in the future. For now, use 'series.values.argmin' or
'np.argmin(np.array(values))' to get the position of the minimum
row.

/home/jupyter-polarwander/rockmag/PmagPy-RockmagPy-notebooks-0658e2f/rockmag.py:326: FutureWarning:

The current behaviour of 'Series.argmax' is deprecated, use 'idxmax'
instead.
The behavior of 'argmax' will be corrected to return the positional
maximum in the future. For now, use 'series.values.argmax' or
'np.argmax(np.array(values))' to get the position of the maximum
row.

Commit 1122d29 got rid of the warning associated with rockmag.py. It looks like the continued warning is a numpy issue specific to the JupyterHub environment. I will make a commit to suppress future warnings for now.

Now the warning is suppressed with https://github.com/PmagPy/RockmagPy-notebooks/commit/92ce5001b941c965766ed791dbc4eb17e5b46309 The issue is likely that packages need to be updated with EarthRef JupyterHub. But this will work for the moment.


Check if cluster is offline when the app is not found

Related Issue: https://github.com/open-cluster-management/backlog/issues/12005
Description of Changes

Add check if cluster is offline when the app is not found

Need to deliver this PR first and then patch the e2e cluster so the tests will pass.


Generate code coverage with Coveralls

WIP
Not ready yet—we'll see if this works!

Changes Unknown when pulling 30e8e8bd5b8e174427aabb43ed85f9407754c623 on coverage into ** on master**.

Changes Unknown when pulling 30e8e8bd5b8e174427aabb43ed85f9407754c623 on coverage into ** on master**.

Changes Unknown when pulling 30e8e8bd5b8e174427aabb43ed85f9407754c623 on coverage into ** on master**.

Changes Unknown when pulling f58b52232ec6d5cf3a127dfdbb6a84bcc20d92fb on coverage into ** on master**.

@svenkreiss PTAL!

Note that the .coverage files from individual runs are not summed: you can see this because the import statements are only run once, not 3x.

LGTM

Changes Unknown when pulling 2ed19902afdd4b38ad3724dc0003187661abdeea on coverage into ** on master**.


github_webhook: Support "ping" events from organization webhooks.

In addition to "ping" webhook payloads from repository-specific
webhooks, we now also support "ping" events from organization-wide
webhooks.

Automated message from Dropbox CLA bot
@eeshangarg, it looks like you've already signed the Dropbox CLA. Thanks!

Merged, thanks @eeshangarg!


Data export

Issues
Closes #26

Just saw the filters tab, I like that!

Yeah, the export is a snapshot of the current nests and filters.  We can't think of the filters in the tool the same as the filters in the export.  I think just noting the current filter settings in the second tab should suffice.

Merging for now.  Need to revisit filter columns in export, captured in #44 and #45.


Adding .netcoreapp1.1 to the list of tfms

Was this supposed to go into RC2? If so, you need to merge into that branch.


[eas-cli] Add --channel to update republish

Checklist

[ ] I've added an entry to CHANGELOG.md if necessary. You can comment this pull request with /changelog-entry [breaking-change|new-feature|bug-fix|chore] [message] and CHANGELOG.md will be updated automatically.

Why
Keep the usage of update and update:republish similar, by adding --channel to republish.
How
This only works for channels that are pointing to a single branch. When that branch is fetched, it reuses the code from the --branch flag.
Test Plan

See tests
eas update:republish --channel <channelName>
→ should prompt you to select an update from the branch that is connected to the channel
→ should fail with --non-interactive, --group, or --branch
eas update:republish --branch <branchName>
→ should prompt you to select an update from the branch
→ should fail with --non-interactive, --group, or --channel
eas update:republish --group <updateGroupId>
→ should work with --non-interactive
→ should fail with --branch or --channel

/changelog-entry new-feature Add --channel option to update:republish

This PR is blocked by PR #1581 unfortunately


ArteTV support broken by unicode encoded JSON URLs

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2019.07.02
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.arte.tv/fr/videos/065424-071-A/blow-up-c-est-quoi-claudia-cardinale/']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.07.02
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: ffmpeg N-90169-gf4709f1b7b, ffprobe N-90169-gf4709f1b7b
[debug] Proxy map: {}
[arte.tv:+7] blow-up-c-est-quoi-claudia-cardinale: Downloading webpage
[download] Downloading playlist: None
[arte.tv:+7] playlist None: Collected 0 video ids (downloading 0 of them)
[download] Finished downloading playlist: None

Description
Since yesterday 2019/07/02, URL of videos on the arte.tv website, which were working until then, stopped getting downloaded.
I don't know what changed, but I figured that the extractor, in _extract_from_webpage(), looks for a JSON URL with find_iframe_url() by searching for a pattern starting with <iframe but matches nothing as JSON data are unicode-escaped in the response stored in the webpage variable, and would thus appear as \\u003Ciframe instead.
I found that by unescaping the retrieved webpage as follows, expected eventual behavior (i.e. videos getting downloaded) is restored:
--- arte.old.py
+++ arte.new.py
@@ -215,1 +215,1 @@
-        webpage = self._download_webpage(url, video_id)
+        webpage = self._download_webpage(url, video_id, encoding="unicode_escape")

I don't know whether this is the piece of the extractor that got broken by a recent change in the website response, but I can confirm it restores video downloading functionality for the URLs that recently stopped working.
Please note that most _TESTS URLs are now outdated and it's better to test with fresh URLs from https://www.arte.tv.

Duplicates #21614 I guess.
Sorry I didn't catch it when reporting.


Standardize Python service attributes

Updates the Python service to be serialized with a hash argument, rather
than an array.
Adds a StyleGuide::Base#enqueue_job private method that handles the
Resqueue details. The Python linter overrides this because it has a
non-standard class name, review.PythonReviewJob. As far as I can
tell, this name is required because the Python Resuque library, Pyres,
uses the python import system, which needs a module name. I'd love to
hear more about this from anyone with more knowledge.

:+1:


Support Landba function Urls configuration

Environment information
System:
  OS: Windows 11 10.0.22000
  CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
  Memory: 1.75 GB / 15.73 GB
Binaries:
  Node: 20.10.0 - C:\Program Files\nodejs\node.EXE
  Yarn: undefined - undefined
  npm: 9.8.1 - C:\Program Files\nodejs\npm.CMD
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/backend: 0.12.1
  @aws-amplify/backend-cli: 0.11.1
  aws-amplify: 6.0.19
  aws-cdk: Not Found
  aws-cdk-lib: Not Found
  typescript: 5.4.2
AWS environment variables:
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
  AWS_STS_REGIONAL_ENDPOINTS = regional
No CDK environment variables

Description
Are there any plans to integrate Landba function Urls in fonction configuration? I would like to create a function to handle webhooks and it would be nice to be able to create an HTTP endpoint easily.

Hey @jbp35, thank you for reaching. you can override the function and add the function URL. Refer to the documentation providing an example
for example, I was able to add a function URL using the following
import { myDemoFunction } from "./function/resource";
import { Function, FunctionUrlAuthType } from "aws-cdk-lib/aws-lambda";

const backend = defineBackend({
  myDemoFunction,
});

const myFunc = backend.myDemoFunction.resources.lambda as Function;

myFunc.addFunctionUrl({
  authType: FunctionUrlAuthType.AWS_IAM,
});

Hey @jbp35, thank you for reaching. you can override the function and add the function URL. Refer to the documentation providing an example
for example, I was able to add a function URL using the following
import { myDemoFunction } from "./function/resource";
import { Function, FunctionUrlAuthType } from "aws-cdk-lib/aws-lambda";

const backend = defineBackend({
  myDemoFunction,
});

const myFunc = backend.myDemoFunction.resources.lambda as Function;

myFunc.addFunctionUrl({
  authType: FunctionUrlAuthType.AWS_IAM,
});

Thank you, that's working great!


Webpack Error During Tailwind Configuration

I believe it's related to #36, and I think the issue left unresolved.
I get exactly the same error message as @Ladet02.
I've installed Wagtail with "npm command at frontend directory" setup, enabled live reloading feature and I get the following error trying to configure Tailwind:

I've checked all the steps provided here like a million times.

@FilipWozniak
If possible, could you please create a project and upload your code to Github
I can not figure out the problem from the above screenshot.
Thx.

If possible, could you please create a project and upload your code to Github

Sure, here it is.
https://github.com/FilipWozniak/dj-fw_blog

@FilipWozniak
You can try to put tailwind.config.js at the root directory, jus like this https://github.com/AccordBox/wagtail-tailwind-blog
Thx.

Thank you, it worked! I apologize for unnecessarily bothering you, but the "Python Webpack Boilerplate" website did not specify anywhere that tailwind.config.js must be in the root directory, and it was generated by default in the frontend folder. I should have thought of that 😉


❤❤❤❤❤❤❤❤——This type emoji can not be decided!

String emoijs = "❤❤❤❤❤❤❤❤";
System.out.println(EmojiManager.isOnlyEmojis(emoijs)); // return false。 I think that it should return true!

this emoji ——🥲 ，also can not  be decided

in version 5.1.1 this ❤ gets correctly removed via EmojiParser.removeAllEmojis(s)

in version 5.1.1 the ♥ does not get correctly removed as it leaves an invisible character behind.


Use from javascript.

Hey :-)
I'm currently trying out jss and i'm writing my components using js, hyperscript (instead of JSX) and jss (instead of css), so they look like:
// STACK
var h       = require('virtual-hyperscript-hook')(require('virtual-dom/h'))
var jss     = require('jss')
var css2jss = require('jss-cli/lib/cssToJss')

var SHEET
function initSheet (css) {
  SHEET = jss.create().createStyleSheet(css2jss(css),{ named: true, link: true }).attach()
}
function s (name, modifier) {
  modifier = modifier ? '  ' + sheet.classes[modifier] : ''
  name     = SHEET.classes[name]
  return name + modifier
}

module.exports = ContentComponent

function ContentComponent (state) {
  // STYLING
  if (!SHEET) {
    initSheet(`
    .content {
      background-color  : seagreen;
      display           : flex;
      align-items       : center;
      justify-content   : space-around;
      height            : 300px;
    }
    .box {
      text-align        : center;
      background-color  : yellow;
      padding           : 20px;
      font-size         : 20px;
      height            : 200px;
      width             : 300px;
    }
    `)
  }
  // RENDER TEMPLATE
  return function Content (state) {
    return h('div', { className: s('.content') }, [
      h('div', { className: s('.box') }, [
        h('div', 'Learning Experience'),
        h('div', state.learningexperience)
      ]),
      h('div', { className: s('.box') }, [
        h('div', 'Female Leadership'),
        h('div', state.femaleleadership)
      ]),
      h('div', { className: s('.box') }, [
        h('div', 'Book'),
        h('div', state.book)
      ]),
    ])
  }
}

As you can see, instead of having 3 files component.css, component.html, component.js, it's all readuced to one component.js file, which uses virtual-hyperscript to create the markup and jss to create the styling. The css is authored inside of an es6 template string.
I have little to no experience with responsive grid systems, but I used jeet in the past.
**Is it possible to require('lost-grid') and somehow use it in a way that would be easily combined with jss?
If not - then how does lost work and what would in theory be needed to make it work with jss?

I'm still not familiar with JSS enough (or get enough requests for it really) to do special work to make Lost compatible with it.
I'd love it if you could test this out and update the Wiki with instructions on how to use with JSS though. I imagine a lot of people are using PostCSS packages with JSS.
Closing for now.


AllowCheats set to True by default

Set the xml value of AllowCheats to True by default. This change will solve 80% of the questions about the project. Don't forget to update the documentation.

Fixed by https://github.com/EtienneLamoureux/TQVaultAE/pull/204


[BUG] Email confirmation

Describe the bug
There is no email confirmation after registration
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Stack Trace
If applicable, add in the error stack trace. By default, all the errors are logged to src/Bootsrapper/Logs/logs***.json. You can also find the stack trace details logged here.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Are you using the default email providers? or have you switched to some other SMTPs? By default, registration mails don't go to the actual email ID. They get sent to the ethereal mailbox, whose auth config you can see in our appsettings for mail.

I have the same issue: No email confirmation is sent after register a user.
I have configured mail.json and i checked twice and i can confirm that the config is OK, but no email was sent...
What can i do? Thanks


Improve password handling

The password is now cached in the ~/.irods/.irodsA file, but using method incompatible with password created with iinit.  Update the logic to improve this.

Pull request #40 resolves this issue.


Update HoverTest to pass with newer browser versions

Update HoverTest to pass with newer versions of Chrome and Firefox,
which have the elements placed in slightly different position and/or
have different size.

I hope that it doesn't need much maintenance, if it does (like breaking in each new version) we should drop it.

k


Add option to allow ActiveJob to manage retries on app-level first

First off: a big thank you to all contributors that made this Gem possible! 🚀
Motivation
Our company ran into some unfortunate circumstances regarding retries and failing jobs with an app that relies on Rails' ActiveJob and (depending on the tenant) Sidekiq/Redis or Cloudtasker/CloudTask as the backend:
due to the behavior already described in

https://github.com/keypup-io/cloudtasker/issues/41 ,

some jobs were retried endlessly in the CloudTask-based stacks.
Proposed solution
My suggestion would be to let a config option retry_mechanism control whether ActiveJobs executions counter is overridden with the value of CloudTask's RetryCounter header.
As implemented, ActiveJob should be able to handle retries using retry_on and only after those are used up let the backend handle further retries.
Drawbacks of the proposed solution

config.max_retries will be without effect in the implementation at hand if ActiveJob is chosen, as executions is not "handed back" to Cloudtasker to update once ActiveJob is done retrying and bubbles errors up for the backend to handle.
If it is acceptable to introduce a retry counter variable that does not clash with AJ's executions, e.g. like Sidekiq's retry_count, this could be a possible way out.

The behavior of first handling retries on application level and then on backend/adapter level is probably quite unintuitive and must be documented to prevent people from accidentally shooting themselves in the foot.

a Rails-specific config variable is introduced that has no relevance for projects that do not even use the CloudtaskerAdapter

Alternative
A more elaborate mechanism could provide the means to configure both app-level and backend-level retries at the same time. This would include either a "hand-off" of the execution/retry counters when the responsibility changes or separate means of tracking and applying executions/retries.

It would be great if you could consider including this change and/or sharing your thoughts on how this contribution can be improved.

@lagr thanks a lot for this PR and the detailed description 😉
I quickly scanned through the code and it looks good. I need to spend some time playing with it locally to evaluate the drawbacks as well as try to find potential edge cases (and refresh my memory on the internals of ActiveJob retry mechanisms)
I'll try to spend some time on it this week.

@lagr So I've finally found some time to play with this. I think there is still a bit of tweaking to do to make it work.
When testing locally, I have the following behavior:

I configured a job to retry_on NameError and put an undefined variable inside the perform method
Without options, retry_on will make 5 attempts and wait for 3s between each
The first 5 attempts led to the job being rescheduled by ActiveJob, with no exception trace in the logs
After that, the job keeps retrying with exception traces in the logs. This loops forever (until the execution max is hit on the Cloud Tasks side)

See below:

Config-wise, my view is to be more aggressive and always use ActiveJob's retry mechanism for ActiveJob jobs (= get rid of the retry_mechanism config option). People using ActiveJob should expect it to be working as per the docs after all. Keeping a mixed behavior will lead to confusion.

@alachaum Thank you for looking into this! 🙏  I may need some time to figure out how to properly detect whether to use the ActiveJob mechanism or not, then.


GetBlockOperationHashesAtLevel() not return all operation hashes but only one

alphanet where level=345799, GetBlockOperationHashesAtLevel() only return one opreation hash opAgfDzQbcGogUpXkWxPdjgaHko9oRUS98ZEnbfiXmpEZtuCedH, but from tzscan https://alphanet.tzscan.io/345799 we can got more then one.

Will look into it, thanks for reporting!

I fixed it with this patch:
https://github.com/DefinitelyNotAGoat/go-tezos/commit/8ba781ee0092184e2527ff2c7137e0fadce60d53
Marking as resolved, thanks for reporting.


Add way of specifying nondefault hparams that are shared across sweep

Nora: Another thing for the next PR is probably to add like --config-path foo.yaml or some way of specifying nondefault hparams that are shared across the sweep

This was solved a while ago, I forget which PR honestly


fixes #433, responsive layout

Have to create an extra div in container, so the dynamic size of the actual parent container can be used. For the rest, it is as suggested in https://github.com/alexandersimoes/d3plus/issues/433.

Great work @bryanph, thanks for contributing.
I've pushed 2 additional tweaks that address multiple visualizations resizing on the same page and the user passing a D3 selection to .container( ): #433


WCMGas summary series missing data in daily aggregates

summary series for waste coal mine gas (WCMG) shows no data before june 2021:

but there was WCMG generation prior to this date.
eg:

and:

see #190 merged into there

on dev @simonhac

seeing a very strange emissions intensity:

check the json debug output for emission intensity

@nc9 not ready to close, weird emissions intensity persists on dev and prod:
https://dev.opennem.org.au/energy/nem/?range=all&interval=1M
When you select NEM it shows the strange emissions intensity, when you selected Queensland it looks normal

the QLD and NEM outputs should be identical - since the facilities are only in qld. at the moment its showing the same energy values and same emission outputs but different intensities:
NEM:

QLD:

closer to narrowing down what that is

console shows that intensity output

the one below doesn't look right either - i'll find where that is being output

Resolved by no longer subtracting battery discharge from totalPowerEnergy used in the emissions intensity equation. Deployed to: https://7f8ef5f4.opennem-fe-bl7.pages.dev/

I really like the branch builds - I'll do the same on the BE using railway. and we'll figure out a way to point them to one another

Resolved and deployed to prod


Fix pulse energy unit test

I think that I broke this when I fixed the pulse durations in #67. Probably we should change this to checking that the FWHMs are consistent or something.
def test_pulse():
        """Test that the max of pulses match previously calculated values."""

        parameters = dict(center_wavelength_nm=1550, time_window_ps=20,
                          fwhm_ps=0.5, GDD=0, TOD=0, npts=2**13,
                          frep_MHz=100, power_is_avg=False, epp=50e-12)

        pulse = lf.Pulse(pulse_type='gaussian', **parameters)
        aw = np.abs(pulse.aw)**2
>       assert np.abs(np.max(aw) - 0.075)/0.075 < 0.01
E       AssertionError: assert (0.05812053821250025 / 0.075) < 0.01
E        +  where 0.05812053821250025 = <ufunc 'absolute'>((0.13312053821250025 - 0.075))
E        +    where <ufunc 'absolute'> = np.abs
E        +    and   0.13312053821250025 = <function max at 0x7f5ac99393f0>(array([5.40943[60](https://github.com/laserfun/laserfun/actions/runs/9131655704/job/25111361872?pr=72#step:7:61)5e-70, 1.09213654e-36, 6.54347033e-36, ...,\n       3.83593973e-33, 1.82245944e-35, 8.14318183e-33]))
E        +      where <function max at 0x7f5ac99393f0> = np.max

Fixed in #75


Num to Word converter

This will convert a Number to a word.
for example :-
100 to one hundred
2 to two

I want to work on this issue under JWOC.

@LuckySingh9897 Upto how much upperlimit of numbers you can do it?

@Srijita-Mandal  up to  1 Trillion.

@LuckySingh9897 Okay go for it.


kpt live diff errors out when (only) resource is deleted

When following the example provided in kpt guide Apply, I ran kpt live diff after removing helloworld/deploy.yaml, expecting it to show the Deployment resource to be deleted in the diff output.
rm helloworld/deploy.yaml
kpt live diff helloworld/
kpt live preview helloworld/
kpt live apply helloworld/ --reconcile-timeout=2m

However, the command errors out as follows:
kpt live diff helloworld/
error: You must provide one or more resources by argument or filename.
Example resource specifications include:
   '-f rsrc.yaml'
   '--filename=rsrc.json'
   '<resource> <name>'
   '<resource>'

kpt live preview and kpt live apply worked as expected.

kpt live diff command has been removed -- closing