Formatters extension / discussion

I've been working, experimenting with and thinking about mocking (raml) API data. I've come to the conclusion that, although faker is nice, a lot nicer is chance.js. One of the reasons is the functions are a lot more sophisticated (but normalized, through option map arguments, normally they would be unary fns taking 0 or 1 argument).
The problem with the current architecture we have, mainly formats.js contains a literal mapping from our fns to fakers. Perhaps back then (I'm talking the ~v1 faker here) it was manageable due to the small amount of functions present but at the current time it is cumbersome to write them all for ourselves (in this library) as well as expect the user to do so.
I was inspired mostly by work done here: https://github.com/json-schema-faker/json-schema-faker/blob/master/lib/util/container.js
What I do not really like about their solution is that they use the schema keywords for each lib, so e.g. calling a faker function would take something like:
"faker" : "somefn"

and chance would take:
"chance" : "somefn"

I am not really sure how they pass arguments though. Personally, I'd think the following might be a (more, but I'm biased of course) elegant solution:
"mock" : {"func" : "faker.somefn.we.map.by.name",
          "args" : {"min": 40, "max": 120}}

Question is how one would want to expose this. I am not sure if raw (faker["somefn"]["we"]["map"]["by"]["name"]) access is a great idea but on the other hand, few harmful functions in those libraries. Pretty much all is geared towards, well duh, generating random data. Most are functions at about 1 level deep from the root of the main library exported object or function. Further limiting allowed selection (through JSON Schemas) to functions (as you already do the normal/lower-case check for functions) and allowing calls to root or first level nested functions could be mapped dynamically or, called directly via string reference (I guess the functional JS way would be to just split on the dots and reduce the properties from the object) as I demonstrated, using square bracket notation.
Another option, is to 'map' the object property structure a bit more:
"mock" : { "faker": {"company" : {"companyName": {"min-words": 3, "max-words": 6}}}}

More typing, twice the amount of brackets/nesting we need but more closely resembles the actual object structure (so in this case, we might take faker/chance/any-other-lib). One might remove the "mock" even but I like it because mostly, it is unambiguous and can be easily be spotted by people less familiar with JSON Schema (and what belongs there, what doesn't).
I've been hacking in Clojure the past few years btw, and recently active with Node again but I use wisp since my brain really loves polish notation so much better now... reading JS is hard, writing it (good) is a pain which is and why (time constraints as well) I haven't come up with a working prototype yet - I also realized format.js has faker baked in hard and one might not want to break existing functionality (functions expecting (faker, schema) to be passed. Another option would be (perhaps also why mock as a schema key is convenient) to have a second one on the side for backwards compatibility.

There is a initiative to build a better Stub REST API server out of the box. You can check the status here:
https://github.com/json-schema-faker/json-schema-faker/issues/145
Thanks and sorry about that delay ! I didn't have the time to give you a proper reply.

@repocho Awesome, thanks for the feedback... no worries about the delays, I know everything about them (as this is about half a year later ;) I'll close this, again, much appreciated


Build error on cmake:   "unknown component LLVMBitWriter"

Failed on
$ cmake ..
-- The C compiler identification is GNU 13.2.1
-- The CXX compiler identification is GNU 13.2.1
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found Curses: /usr/lib/libcurses.so
CMake Deprecation Warning at build/_deps/bcc-src/CMakeLists.txt:3 (cmake_minimum_required):
  Compatibility with CMake < 3.5 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

-- Latest recognized Git tag is -128-NOTFOUND
-- Git HEAD is 9df2abf47280d99f4802f7eb15ca0f7858435ef9
-- Revision is 128-NOTFOUND-9df2abf4
-- Performing Test HAVE_NO_PIE_FLAG
-- Performing Test HAVE_NO_PIE_FLAG - Success
-- Performing Test HAVE_REALLOCARRAY_SUPPORT
-- Performing Test HAVE_REALLOCARRAY_SUPPORT - Success
-- Performing Test HAVE_FFI_CALL
-- Performing Test HAVE_FFI_CALL - Success
-- Found FFI: /usr/lib/libffi.so
-- Looking for histedit.h
CMake Warning (dev) at /usr/share/cmake/Modules/CheckIncludeFile.cmake:77 (message):
  Policy CMP0075 is not set: Include file check macros honor
  CMAKE_REQUIRED_LIBRARIES.  Run "cmake --help-policy CMP0075" for policy
  details.  Use the cmake_policy command to set the policy and suppress this
  warning.

  CMAKE_REQUIRED_LIBRARIES is set to:

    /usr/lib/libedit.so

  For compatibility with CMake 3.11 and below this check is ignoring it.
Call Stack (most recent call first):
  /usr/lib64/cmake/llvm/FindLibEdit.cmake:28 (check_include_file)
  /usr/lib64/cmake/llvm/LLVMConfig.cmake:262 (find_package)
  build/_deps/bcc-src/CMakeLists.txt:67 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Looking for histedit.h - found
-- Found LibEdit: /usr/include (found version "2.11")
-- Performing Test Terminfo_LINKABLE
-- Performing Test Terminfo_LINKABLE - Success
-- Found Terminfo: /usr/lib/libtinfo.so
-- Found ZLIB: /usr/lib/libz.so (found version "1.3.1")
-- Found zstd: /usr/lib/libzstd.so
-- Found LibXml2: /usr/lib/libxml2.so (found version "2.12.6")
-- Found LLVM: /usr/include 17.0.6
-- Found BISON: /usr/bin/bison (found version "3.8.2")
-- Found FLEX: /usr/bin/flex (found version "2.6.4")
-- Found LibElf: /usr/lib/libelf.so
-- Performing Test ELF_GETSHDRSTRNDX
-- Performing Test ELF_GETSHDRSTRNDX - Success
CMake Error at /usr/lib/cmake/llvm/LLVM-Config.cmake:306 (message):
  unknown component LLVMBitWriter
Call Stack (most recent call first):
  /usr/lib/cmake/llvm/LLVM-Config.cmake:328 (expand_topologically)
  build/_deps/bcc-src/cmake/clang_libs.cmake:26 (llvm_expand_dependencies)
  build/_deps/bcc-src/src/cc/CMakeLists.txt:110 (include)

OS: non-systemd Arch (Artix openRC)
Help out please,  already updated clang & LLVM packages, how to solve such?

solved as on
https://github.com/iovisor/bcc/issues/4519#issuecomment-1669100182
cmake .. -DENABLE_LLVM_SHARED=1


Add support for Python Solutions

Background
Currently the project does not support users supplementing a solution written in Python despite being a popular choice for beginners. This forces the learner to dive deep into learning Kotlin, which is not a bad thing, but giving choice to a familiar language can allow a subset of users to complete all challenges in an adequate time and allow them present something they can share with their school colleagues without having to explain a new language semantics.
Requirements
It's possible to refactor the project to introduce a ScriptEngine which allows you to run different language specifications within the JVM.
Given all we do is assert on STDOUT, it would be fairly easy for us to capture their solutions output and run the same tests on them with appropriate modifications.
This is a ticket to apply such changes.

It's also possible to refactor the project to introduce Chaquopy, which is a solution to allow you to run Python scripts within Java/Kotlin code.

Notes & Ideas
Jython
Jython (https://www.jython.org) is one available solutions based on the JSR-223 Scripting Engine. The problem is I cannot find an 3.x implementation.
Process Builder
We can also use the ProcessBuilder to just call Python from the container:
@Test
public void givenPythonScript_whenPythonProcessInvoked_thenSuccess() throws Exception {
    ProcessBuilder processBuilder = new ProcessBuilder("python", resolvePythonScriptPath("hello.py"));
    processBuilder.redirectErrorStream(true);

    Process process = processBuilder.start();
    List<String> results = readProcessOutput(process.getInputStream());

    assertThat("Results should not be empty", results, is(not(empty())));
    assertThat("Results should contain output of script: ", results, hasItem(
      containsString("Hello Baeldung Readers!!")));

    int exitCode = process.waitFor();
    assertEquals("No errors should be detected", 0, exitCode);
}

We can also use Apache Commons Exec library for a better API.
You can read more here: How to Call Python From Java) for further details.


/t setcolour rgb problem

Apparently a well known problem, but seems like no one ever made a ticket about it. The cmd /t setcolor which uses rgb values, makes the colours yellow, pink, purple, black, white, grey, cyan and some others unobtainable. We deduced that, any green value below 85 doesn't exist, any blue below 86 doesn't exist, and any red value that's not dark_red doesn't exist, and refuses to influence the rest of the colours. For example, the values for pink is 255 192 203, whoever in game it gives:

Going to close this, because we have an older feature request for this on the github.


DCNV4 training speed

Hello author, thank you for your work. I am using a 4090 graphics card and an Intel(R) Xeon(R) Gold 6133 CPU @ 2.50GHz. I found that the memory consumption of DCNV4 is indeed smaller than DCNV3, but the GPU utilization has decreased, and its training speed has become slower. What could be the reason for this?

Can you share more details about your training config (model size, input shape, etc.)? We may need to update the default hyperparameters for the backward pass as DCNv4 requires additional proper hyperparameters to perform well.

The configuration I am using belongs to image segmentation and involves DCNv4_op and DCNv3_op. I have performed a compilation installation with all groups set to 8, depth set to 1, channels set to 128, drop_path set to 0.2, layer_scale set to 1.0, dw_kernel_size set to 3, and other parameters left at their default values. Can you provide information on what this situation represents?

it would be great if you could provide a minimal case (i.e. with all the DCNv4 hyperparameters and its input shape so we could define a DCNv4 and DCNv3 op and create a dummy input based on your information and run forward / backward) for us to reproduce your issue

input shape= (512, 256),channels = 128, depth=1, groups=8, downsample=Fasle, downsample_layer=DownsampleLayer, mlp_ratio=3.,  drop=0., drop_path=0.2, act_layer='GELU', norm_layer='LN', post_norm=False, offset_scale=0.5, layer_scale=1, with_cp=False, dcn_output_bias=False, mlp_fc2_bias=False, dw_kernel_size=3, post_norm_block_ids=None, res_post_norm=False,
center_feature_scale=False


Add ability to modify chart in MongoDB and MysqlDB structs

Change Overview

added interface HelmApp that helps us to extend current App chart configuration;
HelmApp implemented in MysqlDB and MongoDB.

Pull request type
Please check the type of change your PR introduces:

[x] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[x] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[ ] :world_map: Documentation
[ ] :robot: Test

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

/ok-to-test


Add CVE identifier to Vulnerability class

This adds the CVE information to --json output

I don't think the failing macOS check is caused by this PR, but by some AppVeyor issue.

@jheyens  I'd love to get this merged.  Looking to integrate with gitlab CI. Here's an example payload that gitlab would accept. Is this a standard schema? or there own?
{
  "version": "2.0",
  "vulnerabilities": [
    {
      "tool": "npm_audit",
      "category": "dependency_scanning",
      "name": "axios",
      "namespace": "axios",
      "message": "Server-Side Request Forgery",
      "cve": "package.jsonaxios:cve_id:CVE-2020-28168",
      "description": "The `axios` NPM package before 0.21.1 contains a Server-Side Request Forgery (SSRF) vulnerability where an attacker is able to bypass a proxy by providing a URL that responds with a redirect to a restricted host or IP address.",
      "severity": "High",
      "fixedby": "Anonymous",
      "confidence": "High",
      "scanner": {
        "id": "npm_audit_advisories",
        "name": "NPM Audit"
      },
      "location": {
        "file": "package.json",
        "dependency": {
          "package": {
            "name": "axios"
          },
          "version": "<0.21.1"
        }
      },
      "identifiers": [
        {
          "type": "cve_id",
          "name": "CVE-2020-28168",
          "value": "CVE-2020-28168",
          "url": "https://nvd.nist.gov/vuln/detail/CVE-2020-28168"
        },
        {
          "type": "cwe_id",
          "name": "CWE-918",
          "value": "CWE-918",
          "url": "https://cwe.mitre.org/data/definitions/918.html"
        }
      ],
      "solution": "Upgrade to 0.21.1 or later.",
      "instances": [
        {
          "method": "axios"
        }
      ],
      "links": [
        {
          "url": "https://npmjs.com/advisories/1594"
        }
      ]
    }
  ],
  "remediations": []
}

@jheyens  I'd love to get this merged.  Looking to integrate with gitlab CI. Here's an example payload that gitlab would accept. Is this a standard schema? or there own?
{
  "version": "2.0",
  "vulnerabilities": [
    {
      "tool": "npm_audit",
      "category": "dependency_scanning",
      "name": "axios",
      "namespace": "axios",
      "message": "Server-Side Request Forgery",
      "cve": "package.jsonaxios:cve_id:CVE-2020-28168",
      "description": "The `axios` NPM package before 0.21.1 contains a Server-Side Request Forgery (SSRF) vulnerability where an attacker is able to bypass a proxy by providing a URL that responds with a redirect to a restricted host or IP address.",
      "severity": "High",
      "fixedby": "Anonymous",
      "confidence": "High",
      "scanner": {
        "id": "npm_audit_advisories",
        "name": "NPM Audit"
      },
      "location": {
        "file": "package.json",
        "dependency": {
          "package": {
            "name": "axios"
          },
          "version": "<0.21.1"
        }
      },
      "identifiers": [
        {
          "type": "cve_id",
          "name": "CVE-2020-28168",
          "value": "CVE-2020-28168",
          "url": "https://nvd.nist.gov/vuln/detail/CVE-2020-28168"
        },
        {
          "type": "cwe_id",
          "name": "CWE-918",
          "value": "CWE-918",
          "url": "https://cwe.mitre.org/data/definitions/918.html"
        }
      ],
      "solution": "Upgrade to 0.21.1 or later.",
      "instances": [
        {
          "method": "axios"
        }
      ],
      "links": [
        {
          "url": "https://npmjs.com/advisories/1594"
        }
      ]
    }
  ],
  "remediations": []
}


/wb fill crashes in 1.14

Server: Spigot 1.14
Plugin: Worldborder 1.9.0
This plugin works otherwise well in 1.14 but when I try to fill un-generated areas, it fills about 1000-9000 chunks before crashing by a ticking world.
Latest.log: https://pastebin.com/mU8K9xEs
Crash report: https://pastebin.com/XL1syrx9
How to reproduce:
Run spigot server 1.14
Have Worldborder 1.9.0 installed
set worldborder to eg. 2000 blocks away from spawn
do command /wb world fill
confirm
wait about a minute
Crash

https://bugs.mojang.com/browse/MC-148830

Thanks for pointing out that this is a vanilla 1.14 bug. I guess that we have to wait until 1.14.1 or 1.14.2 then...


Dynamic header does not change after a client has been instantiated

Bug report

[ x] I confirm this is a bug with Supabase, not with my own application.
[ x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
I need to send headers with my client requests. To do that I have configured the following:
export function createClient(orgContactId: string | null) {
    const supabase = createBrowserClient<Database>(env.NEXT_PUBLIC_SUPABASE_PROJECT_URL, env.NEXT_PUBLIC_SUPABASE_ANON_KEY, {
        cookies: {},
        cookieOptions: {
            domain: env.NEXT_PUBLIC_APP_DOMAIN,
        },
        global: {
            headers: {
                "org-contact-id": `${orgContactId ?? ""}`,
            },
        },
        isSingleton: false,
    })

    return supabase
}

However the issue is that I can't update this header after the client is created so if the orgContactId changes, I need to create a completely new client
I do that by taking the "singleton" in to my own hands and when I detect an ID change, I create a new client that has the new header
export let supabaseBrowserClient: SupabaseClient<Database> | null = null
let orgContactId: string | null = null

export function getSupabaseBrowserClient(activeOrgContactId: string | null) {
    if (supabaseBrowserClient && activeOrgContactId === orgContactId) {
        return supabaseBrowserClient
    }

    orgContactId = activeOrgContactId
    supabaseBrowserClient = createClient(activeOrgContactId)

    return supabaseBrowserClient
}

function useSupabaseBrowser() {
    const {
        userPreference: { activeOrgContactId },
    } = useAppStore((store) => store)

    return useMemo(() => getSupabaseBrowserClient(activeOrgContactId), [activeOrgContactId])
}

The downside to this is that I get warnings in my console stating there are now multiple clients created and that I may deal with unintended behavior.
I think I'm running in to that unintended behavior now. I use NextJS and just tried running my dev server with next dev --turbo. For some reason, my supabase client will only ever use my first instantiated version of the client (which doesnt contain a org-contact-id) even if a few seconds later a new auth client is instantiated with an org-contact-id. It simply just isn't used for any of my requests (the header is empty since it was the first client instantiated it seems...)
I would love to be able to just simply update the headers in the client after it's created...if possible.
Hope you see the use case for it, let me know if you have any other questions :)
Meant to send this months ago but the recommendation came from this auth/ssr discussion: https://github.com/orgs/supabase/discussions/27037
System information

OS: macOS
Version of supabase-js: 2.43.5
Version of Node.js: 20.12.0

Hi! Can you try using .setHeader() for this? E.g. https://github.com/supabase/postgrest-js/blob/e45a2c79231ca6e1469dba80a981501c60d3c8f7/test/basic.ts#L130

Closing this for now - feel free to reopen or create a new issue if the proposed solution doesn't work.


Doesn't work on mobile

I see the text that I'm supposed to press on to write on. However, a single press opens the plugin in the "community browse" section, which is frustrating because it keeps happening accidentally. But I figured out that when I hold and press on the text it shows me the caret, but not the keyboard; and I'm just there watching my Caret trying to figure out how to get my keyboard to show.
Similar to the previous issue #1 I think there needs to be a button,  not for the same reason (though I like his reason) but to make it at least work on mobile, I think? That way you'd be fixing two issues at once?
Suggestion: I think it would be too good if you made it to have its own menu of plugins, similar to "Plugin manager." I think it would be cleaner, if it has its own options, like: "filtering, arranging, hiding, favouriting"
Edit: Partially fixed. As i took the screenshot with the caret, I closed obsidian, and went on GitHub. When I went back to obsidian, and out of nowhere, the keyboard showed up; So basically, the temporary fix is: hold on the text, for the caret to show. Close obsidian and open again.

I am sorry to hear that you have to:

So basically, the temporary fix is: hold on the text, for the caret to show. Close obsidian and open again.

I did not see this bug before. Now, I have realized that if you had already annotations, perhaps because you made them using a desktop version of Obsidian, it is very easy to edit them from a mobile, but if there is no annotation, it does not show the keyboard.
My suggestion for now is to use a desktop version to enter your annotations. You can then easily modify them from your mobile.
This bug will be fixed in the next weeks, but I cannot promise in the next days.
And yes, having a setting page can help to manage other options. However, I wanted to give the user the possibility to effortlessly edit annotations directly from where the plugins are officially maintained. I wanted to avoid that the setting page of my plugin becomes a replacement for the Obsidian pane where now community plugins are maintained...
But let's collect ideas and then try to make an implementation of a meaningful setting page.

I did not see this bug before. Now, I have realized that if you had already annotations, perhaps because you made them using a desktop version of Obsidian, it is very easy to edit them from a mobile, but if there is no annotation, it does not show the keyboard.

You are totally correct. But it's not a bug that is frustrating for the user, it works perfectly. It's not a terrible bug. It just needs to be clarified. sorry if there is misunderstanding didn't use desktop to annotate. I used mobile, using the method I stated, in the previous comment and this one.

My suggestion for now is to use a desktop version to enter your annotations. You can then easily modify them from your mobile.

I found a good way for mobile. The whole problem is that the keyboard doesn't show up the first time editing an annotation, only the caret does, which means that once it is shown, I can simply scroll while the keyboard is shown and write.
For clarification:

I don't need to reopen obsidian, every time I edit an annotation, only the first time when the caret shows up; I close obsidian and reopen it again; Once the keyboard is shown, I just keep adding annotations while the keyboard is shown.

When I said that, I close and reopen obsidian, for the keyboard to show up. I didn't mean to literally close the whole app; I meant simply pressing the home button and opening it again. Similar to desktop, assuming you only have a virtual keyboard to write on, when you click to add an annotation, the virtual keyboard doesn't show up (assuming, also, that you have no way of opening the virtual keyboard except it being automatic, like mobile), but the caret does, so you simply minimize the app and reopen it, then it shows up, and then you simply continue using that virtual keyboard to edit other annotations; however, once the keyboard is not visible, you have to repeat the same method.

This bug will be fixed in the next weeks, but I cannot promise in the next days.

I think the best thing for you to do for now, at least, is to clarify how it should be used on mobile until it is fixed, to avoid uninstallations, because I did uninstall it at first, but I liked the plugin a lot, so I doubted myself, so I opened the issue while having Obsidian in the background. When  I opened it, voila, the keyboard showed up.

And yes, having a setting page can help to manage other options. However, I wanted to give the user the possibility to effortlessly edit annotations directly from where the plugins are officially maintained. I wanted to avoid that the setting page of my plugin becomes a replacement for the Obsidian pane where now community plugins are maintained...

Understandable. Some people may like it, but adding a feature that only serves certain people (who want that feature) would only create enemies against it. Keeping it simple, ig, is best for all; also less work for you.

But let's collect ideas and then try to make an implementation of a meaningful setting page.

An easy feature would be, adding custom colors. Helping to differentiate between the plugin description and the annotation—but a better one—is adding custom text formats, like bold or something.
Thanks for your response.

Bug fixed!
I released a new version 1.0.12 that should have the bug fixed. Check the change log:
Changelog
[1.0.12] - 2024-06-23
Bug fixes

Mobile Optimization:

Resolved the issue where the caret and keyboard were not shown when clicking on placeholder text in mobile devices.
Clicking on a placeholder text now selects the entire text, allowing users to easily start typing.

Bug Fixes

Annotation Deletion:

Fixed an issue where annotations were not correctly deleted when the text was removed.

Thanks a lot for the many ideas:

Great idea I had, which I think I am easy feature, would be: adding custom colors. Helping to differentiate between the plugin description and the annotation—but a better one—is adding custom text formats, like bold or something.

Styles are easy to add. You can already now change your style if you use custom .css. I could provide more hints. The other possibility to write styled text is a bit more demanding because it requires an HTML editor. However, I could imagine allowing the user to enter HTML in the field. But non experienced users might find it too tricky. So this could be an advanced opion which could be enabled in the setting pane (which does not exist yet).

Bug fixed!
I released a new version 1.0.12 that should have the bug fixed. I would appreciate if you can give feedback. Check the change log:
Changelog
[1.0.12] - 2024-06-23
Bug fixes

Mobile Optimization:

Resolved the issue where the caret and keyboard were not shown when clicking on placeholder text in mobile devices.
Clicking on a placeholder text now selects the entire text, allowing users to easily start typing.

Bug Fixes

Annotation Deletion:

Fixed an issue where annotations were not correctly deleted when the text was removed.

Brilliant. Nice idea to get around the issue. Now it selects the words, even when there is no annotation. Thanks a lot, it works perfectly.


Error during rename in vscode

This file is causing issue during rename in vscode:
https://github.com/json-iterator/go/blob/ff2b70c1dbffdd98567bd8c2f9449d97c0d04c88/skip_tests/string/skip_test.go#L1
Rename failed: While scanning Go workspace: Package "playground/vendor/github.com/json-iterator/go/skip_tests/array": E:\workspace\golang\src\playground\vendor\github.com\json-iterator\go\skip_tests\array\skip_test.go:1:1: expected 'package', found 'ILLEGAL'. Package "github.com/gin-gonic/gin/vendor/github.com/json-iterator/go/skip_tests/object": E:\workspace\golang\src\github.com\gin-gonic\gin\vendor\github.com\json-iterator\go\skip_tests\object\skip_test.go:1:1: expected 'package', found 'ILLE

Thank you @taowen!


draft: debug multiple goals, more fixes

Randomly coming here: this PR can be closed, right?

Yes, I think that this was a tracking PR for all adjustments that were "in-progress" and got out-of-sync.


[Feature] Event record for failed Pod creation

Search before asking

[x] I had searched in the issues and found no similar feature requirement.

Description
Recently, while using RayCluster, a user configured an invalid label in the pod template. I could only discover this issue through the logs of RayOperator. Perhaps, we could use the following methods to help us troubleshoot or avoid such issues more quickly:

Record relevant failure information using EventRecorder when pod creation fails.
Add validation logic for the pod template in the validating webhook.

Use case
RayCluster troubleshooting
Related issues

#2210

Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

So far in KubeRay we've tried to avoid validating webhooks for two reasons:

Many users have company-wide policies disallowing webhooks
Webhooks potentially introduce additioanl complexity and reliability issues

We do have a separate effort to improve RayCluster observability via status conditoins: https://github.com/ray-project/enhancements/pull/54
Maybe we can incorporate invalid Pod templates as part of this @kevin85421 @rueian

Also just worth clarifying that we do have config for validating webhooks but it's optional and you need to manually deploy it like this: https://github.com/ray-project/kuberay/blob/master/ray-operator/Makefile#L132-L134
The validation logic is here: https://github.com/ray-project/kuberay/blob/master/ray-operator/apis/ray/v1/raycluster_webhook.go#L51-L68

Ah, I see you changed the issue to ask for events instead of validating webhooks. Related issue: https://github.com/ray-project/kuberay/issues/2189

@andrewsykim. Haha. In fact, we didn't enable webhooks, so the ErrorEvent is sufficient for us to troubleshoot issues. Of course, using validating webhooks to verify the RayCluster would be even better if allowed.
A failure to create a pod is not always due to an invalid YAML. Using Events can help us quickly diagnose the issue.

Closed by #2286


Delay in calculating FRF

@romero-patrick noticed that the execution time when using the ROSS FRF response is high. In one example the time spent was ~5min. To speed up the calculations, here is the code example:
import numpy as np
import ross as rs
import time

samples= 100
speed_range = np.linspace(2*np.pi*0,2*np.pi*30,samples)
inp = 3*4+1
out = 4*4+1

L=np.array([    0,   30,   50,    60,   65,  100,  120,  150,  180,  210,  244,  270,  300,  335,  345,  355,  375,  400,  430,  460,  490,  520,  550, 578,
                       616,  626,  636,   676,  700,  730,  765,  791,  820,  850,  880,  915,  950,  985, 1000])

odl = 0.017
L = L*0.001
L = [L[i] - L[i - 1] for i in range(1, len(L))]
list_odl = [odl]*len(L)
for i in [2,3,13,14,24,25]:
    list_odl[i] += 0.020
Nnos = len(L)
Nele = Nnos-1
Steel = rs.Material("Steel", 7850, E=205e9, G_s=None, Poisson=0.29, color='#525252')
Steel2= rs.Material(name="Steel2", rho=7810, E=211e9, G_s=80e9)
shaft = []
for i in range(len(L)):
    shaft.append(
        rs.ShaftElement(
            material=Steel,
            L=L[i],
            idl=0,
            odl=list_odl[i],
            rotary_inertia=False,
            shear_effects=False
    ))

disks = [rs.DiskElement.from_geometry(n = 14, material = Steel2, width = 0.01, i_d = 0, o_d = 0.200),
    rs.DiskElement(Id=0.007377186232163936, Ip=0.003688593116081968,
    m=2.66029, color='Firebrick', n=25),
    rs.DiskElement(Id=0.007377186232163936, Ip=0.003688593116081968,
    m=2.66029, color='Firebrick', n=3)
    ]
bearing2 = rs.BearingElement(6, kxx=1e6, kyy = 8.5e5, cxx=50, cyy = 20)
bearing3 = rs.BearingElement(35, kxx=5e5, kyy = 7e5, cxx=15, cyy = 35)

rotor_data = rs.Rotor(shaft, disks, [bearing2, bearing3])

FRFrstart = time.time()
results = rotor_data.run_freq_response(speed_range)
FRFrend = time.time()
fr = results.freq_resp[inp, out]

print(f'Time FRF of ROSS {FRFrend-FRFrstart} s')

Together with @romero-patrick, we are developing a new way of calculating the FRF, where the time drops considerably ~1s for the same example.
This is the idea of a function:
def FRF(rotor, speed_range, inp, out, num_modes = 10):
    M_mod = rotor.M()
    K_mod = np.zeros(M_mod.shape)
    K0 = rotor.K(0) + K_mod
    FRF_resp = np.zeros(len(speed_range))
    Lambda, Phi = sp.linalg.eigh(K0, M_mod)
    Phi = Phi[:,:num_modes]
    M_red = Phi.T@M_mod@Phi
    G_red = Phi.T@rotor.G()@Phi
    for i in range(len(speed_range)):
        w=speed_range[i]
        K_red = Phi.T @ (rotor.K(w) + K_mod) @ Phi
        C_red = Phi.T @ rotor.C(w) @ Phi
        H_red = -w**2*M_red + (1j*w)*(C_red+w*G_red) + K_red
        Hinv_red = sp.linalg.inv(H_red)
        Hinv = Phi<EMAIL_ADDRESS>        #FRF_resp[i] = np.abs(Hinv[int(inp)][int(out)])
        FRF_resp[i] = (Hinv[int(inp)][int(out)])

    return FRF_resp

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.


Add hooks for additional loading criteria to network page

Allows plugins to delay the rendering of the Network page by returning a Promise in a filter. Page will render when the Promise has resolved.
By default the page will not be delayed.

I like this and would cheerfully ship it!
I am a little apprehensive though. This PR introduces a small change to the way one specific page renders. I am reminded uncomfortably of #119, which I'm sure you'll remember introduced a degree of tension and discord quite disproportionate to its effect. Let's make sure that we consult as widely as possible so that we don't end up in that situation again.

Thanks @philipnrmn. I understand your concern and appreciate your understanding that we really only need this functionality on this page. If the need arises to have this elsewhere, let's reengineer and rollout globally.


Add ability to take photos on phone

Currently (on my phone at least), our uppy component will only let me take selfies, not front-facing photos

The camera also doesn't seem to work on Mac laptops at all (the buttons are obscured)


Do not use the dynamic :action segment on Omniauth routes.

This was deprecated on rails/rails#23980.
We now generate scope and provider specific routes, like user_facebook_omniauth_callback
or user_github_omniauth_callback.
We could deprecate the omniauth_authorize_path in favor of the generated routes, but
the shared/links.html.erb depends on it to generate all omniauth links at once.
Closes #3983.

:shipit:


Update: set scroll event window variable to browserWindow, changed tmpScrollTop variable to use vanilla js.

closing for now, late night coding can be dangerous!


The recovered patient should be show in the list district ways.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Seems to duplicate #675


Donator skins

Test with "ace" or "android"
Dont add this until I have talked you through with it on skype :)
(v2)

Go on skype

Closed for now as we changed our minds


Is this package maintained?

Hey guys,
thanks for your help with these package!
I would like to know if this package is maintained , and if not, what are the other options for using elasticsearch and asyncio
Thank you guys in advance.

Hi,
this package is indeed maintained and is the recommended way to interact with elasticsearch through python's asyncio module.


Add infrastructure to support backwards compatibility of KRKG versions

We want to always support older KRKG versions. If we decide to start verifying sync with extra data points, we should still be able to run older KRKGs with fewer data points through Kinoko.
Our immediate idea is to do the following:

For every new datapoint to be captured in sync verification, add a new member to TestData struct
In TestDirector::findNextEntry(), use a switch against m_versionMinor to determine which fields to initialize
In TestDirector::test, use a switch against m_versionMinor to determine which if-statements to execute

Resolved in a7aee778519f4b4e4150cb44109150000bb46011.


#237 Patch

Just a minor change of querying style and ordering on edgedb

Updated Query to view egress summary based on listed files in manifest


feat: pagination for get all users

Description
Pagination while getting all users
Checklist:

[x] This pull request follow our contribution guidelines
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (postman and wiki)
[ ] I have written tests for the code
[ ] I have updated .env.example file for new .env variables

What is current behavior?

Closes #157

@is-it-ayush I am merging this for now, you can review it and if you want any changes than you can open new issue for that.


Add swc-loader for TypeScript files

I added @swc/core and swc-loader as dev dependencies and changed ts-loader to swc-loader in webpack config.

That's strange, but should we continue it here or create another issue for it? I may help sniffing around the issue if I have time.


Track/work around actions/setup-python#682

Due to actions/setup-python#682, we see failures of the "pytest" CI workflow in ixmp (e.g. here) and message_ix (e.g. here).

These occur only on macos-latest for Python 3.7, specifically 3.7.17 as noted upstream.
The cause is that the Python build is missing _bz2, which is imported by pandas.

This issue is to track the upstream one.

As mitigation, we could temporarily modify the job matrix in the CI workflow to add:exclude:
# Work around #XXX / actions/setup-python#682
- {os: macos-latest, python-version: "3.7"}

…and then later remove it.
We can also simply tolerate the failure and merge PRs despite it, referencing this issue.
The solution will/should be delivered upstream.

A fix is apparently made upstream and is being deployed gradually across the network of GHA runners. One of the devs made a suggestion https://github.com/actions/setup-python/issues/682#issuecomment-1604319803 on how to force the refresh, so we could add that to the affected workflows if needed.

Yes, I've seen that, too. We would only need these steps for one case of our matrix workflows, so it might be simpler to wait until the change reaches us naturally. How would we see that, though? The macos-py3.7 tests should pass again, but the fix was apparently published three days ago, so shouldn't the fix have arrived at our tests in the meantime?

How would we see that, though? The macos-py3.7 tests should pass again

Yes exactly.

the fix was apparently published three days ago, so shouldn't the fix have arrived at our tests in the meantime?

Not necessarily. I guess the runners in their various data centres, and the software on them, updates or replicates on some periodic schedule, maybe there is caching involved, and perhaps this one fix didn't go through a fast/urgent channel or involve a forced cache refresh. It's a big system!

So do you think it's worth it to update the workflows and remove the fix in a week or two (if that is a more appropriate time frame)?

@glatterf42 noted that the fix made it to ixmp as of 2023-06-29, and we don't see the issue on message_ix any more, either, as of today. Thus, I'll close this tracking issue.


Refactor render method

Thank you for sharing this project 👍
This Pull Request addresses a bug I noticed with the renderChildren logic.
If we had the below code, it would be wrapped in a
 tag.
<Online>
  Hello
</Online>

// ...becomes

Hello

Is this intentional? Shouldn't all inline tags be wrapped in a  tag? And we might also want to document the wrapperType prop.
To me, the logic is starting to get more complicated than it has to.
I think we should leave it to the user of react-detect-offline to decide how they want to render their children.

awesome work. I used most of this in 2.0.0. i kept the ability to have multiple children though. I didn't know if we wanted to remove possibility.


[Bug]: Docker Build Fails only with Testcontainers

Testcontainers version
3.2.0
Using the latest Testcontainers version?
Yes
Host OS
Windows
Host arch
x64
.NET version
6.0.404
Docker version
Client:
 Cloud integration: v1.0.31
 Version:           20.10.24
 API version:       1.41
 Go version:        go1.19.7
 Git commit:        297e128
 Built:             Tue Apr  4 18:28:08 2023
 OS/Arch:           windows/amd64
 Context:           default
 Experimental:      true

Server: Docker Desktop 4.18.0 (104112)
 Engine:
  Version:          20.10.24
  API version:      1.41 (minimum version 1.12)
  Go version:       go1.19.7
  Git commit:       5d6db84
  Built:            Tue Apr  4 18:18:42 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          1.6.18
  GitCommit:        2456e983eb9e37e47538f59ea18f2043c9a73640
 runc:
  Version:          1.1.4
  GitCommit:        v1.1.4-0-g5fd4c4d
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

Docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc., v0.10.4)
  compose: Docker Compose (Docker Inc., v2.17.2)
  dev: Docker Dev Environments (Docker Inc., v0.1.0)
  extension: Manages Docker extensions (Docker Inc., v0.2.19)
  init: Creates Docker-related starter files for your project (Docker Inc., v0.1.0-beta.2)
  sbom: View the packaged-based Software Bill Of Materials (SBOM) for an image (Anchore Inc., 0.6.0)
  scan: Docker Scan (Docker Inc., v0.25.0)
  scout: Command line tool for Docker Scout (Docker Inc., v0.9.0)

Server:
 Containers: 100
  Running: 0
  Paused: 0
  Stopped: 100
 Images: 183
 Server Version: 20.10.24
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 2456e983eb9e37e47538f59ea18f2043c9a73640
 runc version: v1.1.4-0-g5fd4c4d
 init version: de40ad0
 Security Options:
  seccomp
   Profile: default
 Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
 Operating System: Docker Desktop
 OSType: linux
 Architecture: x86_64
 CPUs: 16
 Total Memory: 49.97GiB
 Name: docker-desktop
 ID: XNNW:ZAMV:SXVL:PJJ5:FSVK:MU7H:B3QR:FKLA:BU5B:L4DF:B5B4:FUI2
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 HTTP Proxy: http.docker.internal:3128
 HTTPS Proxy: http.docker.internal:3128
 No Proxy: hubproxy.docker.internal
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  hubproxy.docker.internal:5555
  <IP_ADDRESS>/8
 Live Restore Enabled: false

WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support

What happened?
Following Dockerfile buils with docker but not with Testcontainers:
FROM smallstep/step-ca AS certs
ARG RESOURCE_REAPER_SESSION_ID="00000000-0000-0000-0000-000000000000"
LABEL "org.testcontainers.resource-reaper-session"=$RESOURCE_REAPER_SESSION_ID
RUN mkdir -p exampleCA
RUN step certificate create "Smallstep Root CA" "/home/step/exampleCA/cacert.pem" "/home/step/exampleCA/cakey.pem" \
    --no-password --insecure \
    --profile root-ca \
    --not-before "2021-01-01T00:00:00+00:00" \
    --not-after "2031-01-01T00:00:00+00:00" \
    --san "example.com" \
    --san "mail.example.com" \
    --kty RSA --size 2048

via
        TestcontainersSettings.Logger = logger;
        var image = new ImageFromDockerfileBuilder()
            .WithDockerfileDirectory(CommonDirectoryPath.GetSolutionDirectory(), Path.Combine("services", "mail"))
            .WithDockerfile("Dockerfile")
            .WithBuildArgument("RESOURCE_REAPER_SESSION_ID", ResourceReaper.DefaultSessionId.ToString("D"))
            // .WithCleanUp(true)
            .Build();
        await image.CreateAsync().ConfigureAwait(false);

Relevant log output
Starting test execution, please wait...
A total of 1 test files matched the specified pattern.
[xUnit.net 00:00:06.56]     AclTool.MailClient.Tests.MailClientTest.SendRetrieve [FAIL]
  Failed AclTool.MailClient.Tests.MailClientTest.SendRetrieve [5 s]
  Error Message:
   System.InvalidOperationException : Docker image localhost/testcontainers/42c1e266-ace0-4ed9-b169-84363ddc3c07:latest has not been created.
  Stack Trace:
     at DotNet.Testcontainers.Clients.DockerImageOperations.BuildAsync(IImageFromDockerfileConfiguration configuration, CancellationToken ct)
   at DotNet.Testcontainers.Clients.TestcontainersClient.BuildAsync(IImageFromDockerfileConfiguration configuration, CancellationToken ct)
   at DotNet.Testcontainers.Images.FutureDockerImage.UnsafeCreateAsync(CancellationToken ct)
   at DotNet.Testcontainers.Images.FutureDockerImage.CreateAsync(CancellationToken ct)
   at AclTool.MailClient.Tests.MailClientTest.SendRetrieve() in C:\Develop\TFS\SVN.AclTool\src\AclTool.MailClient.Tests\MailClientTest.cs:line 31
--- End of stack trace from previous location ---
  Standard Output Messages:
 info [] Connected to Docker:
   Host: npipe://./pipe/docker_engine
   Server Version: 20.10.24
   Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
   API Version: 1.41
   Operating System: Docker Desktop
   Total Memory: 49.97 GB
 info [] Pattern ^(.+)\/\.idea added to the regex cache
 info [] Pattern ^([\\\/]?(\.idea\b|$)) added to the regex cache
 info [] Pattern ^([\\\/]?((.+)\/\.vs\b|$)) added to the regex cache
 info [] Pattern ^([\\\/]?(\.vs\b|$)) added to the regex cache
 info [] Pattern ^([\\\/]?(\.dockerignore\b|$)) added to the regex cache
 info [] Pattern ^([\\\/]?(Dockerfile\b|$)) added to the regex cache
 trce [] Step 1/13 : FROM smallstep/step-ca AS certs
 trce []  ---><PHONE_NUMBER>81

 trce [] Step 2/13 : ARG RESOURCE_REAPER_SESSION_ID="00000000-0000-0000-0000-000000000000"
 trce []  ---> Using cache

 trce []  ---> 2d37b22a6e0e

 trce [] Step 3/13 : LABEL "org.testcontainers.resource-reaper-session"=$RESOURCE_REAPER_SESSION_ID
 trce []  ---> Running in ce5cc26dce8b

 trce [] Removing intermediate container ce5cc26dce8b

 trce []  ---> a9e364896075

 trce [] Step 4/13 : RUN pwd
 trce []  ---> Running in 277b0851559b

 trce [] /home/step

 trce [] Removing intermediate container 277b0851559b

 trce []  ---> 8aad84034619

 trce [] Step 5/13 : RUN mkdir -p exampleCA
 trce []  ---> Running in 548e0b6db0ab

 trce [] Removing intermediate container 548e0b6db0ab

 trce []  ---> 0248862a6a65

 trce [] Step 6/13 : RUN ls /home/step
 trce []  ---> Running in 77e61f318fe4

 trce [] Removing intermediate container 77e61f318fe4

 trce []  ---> 6652a344d349

 trce [] Step 7/13 : RUN touch test
 trce []  ---> Running in 457e448e0b04

 trce [] Removing intermediate container 457e448e0b04

 trce []  ---> 8955d3a50664

 trce [] Step 8/13 : RUN step certificate create "Smallstep Root CA" "/home/step/exampleCA/cacert.pem" "/home/step/exampleCA/cakey.pem"     --no-password --insecure     --profile root-ca     --not-before "2021-01-01T00:00:00+00:00"     --not-after "2031-01-01T00:00:00+00:00"     --san "example.com"     --san "mail.example.com"     --kty RSA --size 2048
 trce []  ---> Running in 3486f349862a

 trce [] \x1b[91mopen /home/step/exampleCA/cakey.pem: no such file or directory
\x1b[0m
 fail [] The command '/bin/sh -c step certificate create "Smallstep Root CA" "/home/step/exampleCA/cacert.pem" "/home/step/exampleCA/cakey.pem"     --no-password --insecure     --profile root-ca     --not-before "2021-01-01T00:00:00+00:00"     --not-after "2031-01-01T00:00:00+00:00"     --san "example.com"     --san "mail.example.com"     --kty RSA --size 2048' returned a non-zero code: 1

Failed!  - Failed:     1, Passed:     0, Skipped:     0, Total:     1, Duration: < 1 ms - AclTool.MailClient.Tests.dll (net6.0)

Additional information
Works if executed with docker build .
Seems like testcontainers is still using the old docker build pipeline and not buildkit?

Seems like its not a Testcontainers issue by itself, it only occurs with BuilKit disabled (e.g. $env:DOCKER_BUILDKIT = 0). Is there a way to build with buildkit enabled?

The legacy docker build engine seems to recreate the folder that is created in the step before, thus erroring in the subsequent step as it is empty.

Removing intermediate container 548e0b6db0ab

Interessting observation, I think due to Docker removing the intermediate layer, the previous steps are not present in the final step. For example, if you look into RUN mkdir -p exampleCA, you will notice that the exampleCA directory is missing. That's why you receive the following error message:

/home/step/exampleCA/cakey.pem: no such file or directory

If you chain the commands and run them in a single command, it works fine.

Seems like its not a Testcontainers issue by itself, it only occurs with BuilKit disabled (e.g. $env:DOCKER_BUILDKIT = 0). Is there a way to build with buildkit enabled?

We encountered a similar request in Node, and it appears that BuildKit is not available through the Docker Engine API: https://github.com/testcontainers/testcontainers-node/issues/571#issuecomment-1576465904.

We encountered a similar request in Node, and it appears that BuildKit is not available through the Docker Engine API: https://github.com/testcontainers/testcontainers-node/issues/571#issuecomment-1576465904.

Hopefully that will change as they deprecate it and add a EoL

If you like, we can create it here. I can pin the issue and forward it to the other Testcontainers maintainers.

Almost a year later I've run into it again :D (but this time via #1247)
Are there any updates regarding a central way of supporting buildkit?

No, we cannot support it unless it is available via the Docker Engine API. For more details, please see: https://github.com/testcontainers/testcontainers-dotnet/discussions/1193#discussioncomment-10315903. That is out of our control — sorry.


Fruits Classification

My aim here is to build a model using various approaches that will be able to classify the image of the fruit/ vegetable out of the 131 classes on which it will be trained.
Dataset: https://www.kaggle.com/datasets/moltean/fruits
My approach:

Fork this repo.
Create and train DL models.
Compare the models.
Synch the forked repo.

Kindly allow me to work on this issue as part of Hacktoberfest2022. Thank you!

Can I be assigned to this issue?


sabre/dav got stylized differently.

Since a year or two we started stylizing SabreDAV as sabre/dav. I hope this change can also be reflected on the PHP-FIG website.

Light speed. Thank you!

Indeed. I thought I was going to be super on top of this one tonight and BAM. Seconds later... :)

Insomnia. :-/

:+1:

Sorry to hear that @pmjones =(


Fix [Storage Staking] Section Bug

Bug Fixing
In the section of Use a binary serialization format, rather than JSON https://docs.near.org/docs/concepts/storage#use-a-binary-serialization-format-rather-than-json
When using a binary serialization format with borsh towards the array [0, 1, 2, 3],  the last 4 bytes should be \u0000\u0001\u0002\u0003 instead of \u0000\u0001\u0002\u0004

@thisisjoshford  thanks for merge, Josh.


[release-1.12] Bump docker + helm, plus other updates

Pull Request Motivation
SImilar to #7206  but also bumps docker to fix a vuln (CVE-2024-41110) reported by trivy (but not by govulncheck).
Also aims to address reported Helm vulns (CVE-2024-25620 and CVE-2024-26147) in the ctl image.
Kind
/kind bug
Release Note
Fix CVE-2024-25620 and CVE-2024-26147 by upgrading Helm
Fix CVE-2024-41110 by upgrading Docker

/test pull-cert-manager-release-1.12-make-test

Supplanted by https://github.com/cert-manager/cert-manager/pull/7214

https://github.com/cert-manager/cert-manager/pull/7214


MethodCallHandlerImpl - Build failed

I got this error after update from 8.0.3 to 8.0.5.
e: /home/maril/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-8.0.4/android/src/main/kotlin/io/github/ponnamkarthik/toast/fluttertoast/MethodCallHandlerImpl.kt: (17, 16): Redeclaration: MethodCallHandlerImpl e: /home/maril/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-8.0.5/android/src/main/kotlin/io/github/ponnamkarthik/toast/fluttertoast/MethodCallHandlerImpl.kt: (21, 16): Redeclaration: MethodCallHandlerImpl e: /home/maril/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-8.0.5/android/src/main/kotlin/io/github/ponnamkarthik/toast/fluttertoast/MethodCallHandlerImpl.kt: (129, 28): Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type View?                                                                  FAILURE: Build failed with an exception.                                                                                What went wrong:                                                       Execution failed for task ':fluttertoast:compileReleaseKotlin'.           Compilation error. See log for more details                            Try:                                                                   Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.  Get more help at https://help.gradle.org                               BUILD FAILED in 2m 14s
8.0.3 is OK.

I have same error with 8.0.5

Same error as well on upgrading ; )

Same:
Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type View?

Same here

same error.....

same error

After I updated the flitter to 2.0.5, an error occurred when I started the Android project in MacOS. IOS runs smoothly (Windows runs Android smoothly). The error is as follows:
e: /usr/local/Caskroom/flutter/2.0.2/flutter/.pub-cache/hosted/  pub.dartlang.org/fluttertoast  -8.0.5/android/src/main/kotlin/io/github/ponnamkarthik/toast/fluttertoast/M  ethodCallHandlerImpl.kt  : (129, 28): Only safe (?.)  or non-null asserted (!!.)  calls are allowed on a nullable receiver of type View?

Change
fluttertoast:            git:               url: https://github.com/ponnamkarthik/FlutterToast.git               ref: master

@nguyenxdat,
if you fixed this issue, please create a PR.
thanks

Fixed in 8.0.6

Still have.
e: /home/maril/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/fluttertoast-8.0.6/android/src/main/kotlin/io/github/ponnamkarthik/toast/fluttertoast/MethodCallHandlerImpl.kt: (21, 16): Redeclaration: MethodCallHandlerImpl                                                                       FAILURE: Build failed with an exception .

@Mariluk  try a flutter clean. I had that same thing in 8.0.6. But a flutter clean fixed it for me.

@Mariluk same. A flutter clean fixed it.

Yea sry .. flutter clean fixed it. :)

Fixed here too. Thanks!


General Feature Discussion

In the absence of a slack channel, let's use this issue as a general discussion area for features etc.

Phase one
MVP

public app
add Todos
remove Todos
mark Todos as completed
mark Todos as cancelled

Phase two
Make the app into a PWA (connectivity independent, progressive, responsive)

create the app shell
set up service worker
cache files in IndexedDB

Phase three
Introduce authentication, allow users to log in

OAuth (Google, Facebook, Twitter)

Phase four
Allow multiple users to share/track todos within a group

set up Firebase

Discussion on potential improvements welcome. Please comment below.

Updated your list with checkboxes on items :) Also moved IndexedDB into MVP since it will be required to store the todo list at all.

And what do you mean by app shell?

App shell or skeleton, the minimal html/css/js that is needed to be displayed on the screen, before the rest of the content is fetched and displayed.
https://developers.google.com/web/fundamentals/architecture/app-shell

Hi guys, I would like to implement Hot loading to this app which probably will help a bit on our development environment


feat(avoidance): make it selectable output debug marker from yaml

Description
See https://github.com/autowarefoundation/autoware.universe/pull/7097.

Tests performed

Psim
Effects on system behavior

Nothing.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

LGTM


swagger-maven-plugin resolves wrong OpenAPI spec within maven multi module project

Hi!
I have a maven multi module project with two maven module containing JAX-RS resources. My swagger-maven-plugin config for each maven module looks as follows:
<plugin>
    <groupId>io.swagger.core.v3</groupId>
    <artifactId>swagger-maven-plugin</artifactId>
    <version>2.0.5</version>
    <executions>
        <execution>
            <phase>compile</phase>
            <goals>
                <goal>resolve</goal>
            </goals>
            <configuration>
                <openapiFilePath>${project.basedir}/src/main/swagger/openapi.json</openapiFilePath>
                <outputFileName>openapi</outputFileName>
                <outputPath>${project.build.directory}/generated-resources/swagger</outputPath>
                <outputFormat>JSON</outputFormat>
                <prettyPrint>true</prettyPrint>
                <resourcePackages>
                    <resourcePackage>[java-resource-root-package-name]</resourcePackage>
                </resourcePackages>
            </configuration>
        </execution>
    </executions>
</plugin>

When I do a mvn clean install on each maven module everything is fine and two different openapi.json files are resolved by the maven plugin. When I perform the same build on the whole maven project the Swagger maven plugin resolves the JAX-RS resources from the first module (regarding the build order) correctly. For the second JAX-RS resource module I actually get the same openapi.json file resolved as for the first JAX-RS resource module.
Looks like the Swagger maven plugin doesn't scan for the JAX-RS resources in the second module and just resolves the second openapi.json spec based on the JAX-RS resources from the first module.
I saw that you manipulate the classpath within the Swagger maven plugin. Looks like this interferes with the maven class loader scopes/handling.
I am using the following versions:
Oracle Java 1.8.0 Update 192
Apache Maven 3.6.0
maven-compiler-plugin 3.8.0
swagger-maven-plugin 2.0.5
Thanks,
Martin

This seems to be caused by same issue of #3013, which is fixed in latest 2.0.6-SNAPSHOT available on sonatype, please give it a go and close ticket if it solves your issue

I just cloned the master branch (51bfc35facc819b26a9d8604be6354c1dbf60c15, Version 2.0.6-SNAPSHOT) and built the whole swagger-core project and it worked as expected.


configurable backend with env variables

This makes the backend configurable by setting up environment variables to avoid the hardcoding of values such as ports, keys e.t.c

Oh I see. This actually makes a lot of sense. Thank you.
I'll work on it

I've modified the PR. Kindly let me know what you think.

Resolved!

Thank you @Falence

@ivange94 I'm looking forward to more tasks and more code reviews


Create create-footer-blue-3.html

Create a create-footer-blue-3

Good job @kingwisdom
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.

Star this repository. ⭐
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.


[Bug]: Jest cannot run tests in Sapling repos with invalid JSON backups

Version
41bf23008 (main)
Steps to reproduce
sl clone git@github.com:facebook/jest.git # or git clone git@github.com:facebook/jest.git
cd jest
yarn
cd examples/getting-started
yarn
mkdir -pv .sl/origbackups/
echo "aoeu}{" > .sl/origbackups/package.json
node ../../packages/jest-cli/bin/jest.js

Expected behavior
expected: tests pass successfully
Actual behavior
actual:
❯ node ../../packages/jest-cli/bin/jest.js                                                                           ─╯
Error: Cannot parse /Users/m0c0j7y/workspace/github.com/facebook/jest/examples/getting-started/.sl/origbackups/package.json as JSON: Unexpected token 'a', "aoeu}{
" is not valid JSON
    at Object.worker (/Users/m0c0j7y/workspace/github.com/facebook/jest/packages/jest-haste-map/build/worker.js:128:13)
    at execFunction (/Users/m0c0j7y/workspace/github.com/facebook/jest/packages/jest-worker/build/workers/processChild.js:130:17)
    at execHelper (/Users/m0c0j7y/workspace/github.com/facebook/jest/packages/jest-worker/build/workers/processChild.js:
118:5)
    at execMethod (/Users/m0c0j7y/workspace/github.com/facebook/jest/packages/jest-worker/build/workers/processChild.js:121:5)
    at process.messageListener (/Users/m0c0j7y/workspace/github.com/facebook/jest/packages/jest-worker/build/workers/processChild.js:44:7)
    at process.emit (node:events:513:28)
    at emit (node:internal/child_process:944:14)
    at process.processTicksAndRejections (node:internal/process/task_queues:83:21)

Additional context
I have tested this on multiple versions of jest
Environment
❯ npx envinfo --preset jest                                                                                ─╯

  System:
    OS: macOS 12.6.1
    CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
  Binaries:
    Node: 19.2.0 - /usr/local/bin/node
    Yarn: 3.3.0 - /usr/local/bin/yarn
    npm: 9.2.0 - /usr/local/bin/npm

FYI this bug would probably also happen if you did
mkdir .aoeu
echo yourmom > .aoeu/package.json

but I encountered this with Sapling

workaround: deleting the corrupted package.json

@SimenB Do you think it's a bug that an invalid JSON file name package.json anywhere in your project will cause any Jest test to fail?
If I'm working on a JSON validator and I have a few test files that are invalid I could not use Jest
It's worth thinking about better workarounds than deleting those files

adding the invalid files to an ignore pattern seems fine if you want them?

https://github.com/facebook/jest/releases/tag/v29.4.0


[RESOLVED] S100-Secundario has been restarted (uptime < 10m)

Problem started at 07:48:30 on 2023.05.18
Problem name: S100-Secundario has been restarted (uptime < 10m)
Host: S100-Secundario
Severity: Warning
Operational data: 00:06:08
Original problem ID: 20821
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23497&eventid=20821

Problem has been resolved in 4m 0s at 07:52:30 on 2023.05.18
Problem name: S100-Secundario has been restarted (uptime < 10m)
Host: S100-Secundario
Severity: Warning
Original problem ID: 20821


Make mobile as QR signer device

Same as https://github.com/Koniverse/SubWallet-Mobile/issues/1167


Searching Algorithms in C++

Searching Algorithms in C++
Implementations of:

Binary Search
Fibonacci Search
Linear Search
Ternary Search
Median Search

resolved #120
Checklist:

[x] I have named my files and folder, according to this project's guidelines.
[x] My code follows the style guidelines of this project.
[x] My Pull Request has a descriptive title. (not a vague title like Update index.md)
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] I have created a helpful and easy to understand README.md, with problem description and my name.
[ ] I have included a requirements.txt file (if external libraries are required.)
[x] My changes do not produce any warnings.
[ ] I have starred this repository.
[ ] I have added a working sample/screenshot of the script.
[ ] I have checked for trailing spaces in file names and none have them.

@pratik-choudhari kindly review

@dhrubajyoti89 This will be your last contribution to this repo (Max 2). Thanks for stopping by✌

Kindly assign the hacktoberfest-accepted tag @pratik-choudhari

Kindly assign the hacktoberfest-accepted label @pratik-choudhari

@dhrubajyoti89 that label is not required as this PR is merged. It is required when PR is not merged but maintainer wants it to be counted for hacktoberfest.


Sporadically, Steam Audio Return does not work

In most cases, Steam Audio Return does not work. In the editor, it is solved by reassignment. v4.0.3 Release. In version v2.0-beta.20 Release this was not observed

Can you provide more details on what steps need to be taken in order to reproduce the issue? Alternately, could you provide a minimal scene/project that reproduces the issue? Thanks!

this problem occurs only in the editor, steamAudioReturn does not work at all in the assembly (223)

We've just released Steam Audio 4.1.2, which contains a fix for this issue: https://github.com/ValveSoftware/steam-audio/releases/tag/v4.1.2. Try it out, and let us know if the issue persists.


luci-mod-admin-full: improve connection page lookup loop

Currently the lookup loop is very inefficient. This improve the lookup loop by moving the entities to a separate array and adds them only one time. Every entities is checked 3 times, after that it will be moved to dns cache as a raw addres (not resolved). This also introduce the Type column to identify common packet using the source port.
Signed-off-by: Ansuel Smith<EMAIL_ADDRESS>

@jow- any chance for this?

@jow- can you check this?

@jow- news?

Please rebase against the master branch

@feckert done but i think  this won't be part of default code as it's too shitty ahahhaha

please fix also the intention and new line at end of file

@feckert intention? if you mean indentation where is broken?

Sorry for the confusion I meant of course indentation See for example the following lines.
https://github.com/openwrt/luci/blob/1db213e5d7ce89de319f020a6bc5509da42f9157/modules/luci-mod-status/luasrc/view/admin_status/connections.htm#L183-L185
https://github.com/openwrt/luci/blob/1db213e5d7ce89de319f020a6bc5509da42f9157/modules/luci-mod-status/luasrc/view/admin_status/connections.htm#L191-L193
https://github.com/openwrt/luci/blob/1db213e5d7ce89de319f020a6bc5509da42f9157/modules/luci-mod-status/luasrc/view/admin_status/connections.htm#L200-L205

@feckert should be good now

news on this? @feckert

Please fix the last indentation.
And it works as expected so it is LGTM
@jow- any hint from your side if no then i will merge this

@feckert good now?

Thank you for your contribution and for the long waiting time.


Y24 111 - Add a required constraint to the flowcell type field for NovaSeq requests.

Looks good I think. A few questions:

Have you manually tested it in the UI?
Did you consider adding a unit test for RequestOptionsValidator, especially illumina_htp_novaseq_request?? (It doesn't look like this is covered in your asset_submission_spec test as you use an instance_double)

Thank you for having a look

Yes, I performed some manual test
I am using instance_double for RequestOptionsValidator class that is receiving a request of type novaseq, which I think it should cover this specific use case, having said that, I am not quite sure now, as I am still quite new to the language


[9.x] Add new ConvertStringBooleansToBoolean middleware

What
This PR adds a new ConvertStringBooleansToBoolean middleware to the frameworks. This middleware converts string-like booleans "true" or "false" into their true boolean type.
Upon acceptance, and merge, of this PR I would want to add a follow up PR to laravel/laravel to include this new middleware as part of the default stack - like ConvertEmptyStringsToNull.
Why
In my recent Laravel adventures I was building a filtering system to a GET endpoint and sending these filters via a query string. I found that when validating these inputs that booleans were treated as strings, this prevented me from using the 'boolean' validation rule.
e.g; /invoices?approved=true
// ...
public function rules()
{
    return [
        'approved' => ['boolean'],
    ];
}
// ...

The above pseudo-code  would results in a validation error being thrown as within the request object 'approved' => 'true'.
This new middleware resolves what I believe to be a developer inconvenience and normalises expectations when passing booleans in a query string.

Hi, I've fixed the tests on 9.x so please rebase your PR and mark your PR as ready when tests pass. Thanks

Thanks @driesvints 🤙

Converting 'true' | 'false' in bool by default can be a very bad solution for some projects. For example, if a user leaves a comment with the text 'true', his comment will be added to the database as '1'

I think this is edge case when you want to cast true and false strings and this is risky to add this by default.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


EncounterGen

Before #3, but after #2, there should be an EncounterGen.  This can take encounter generation out of the dungeon generator entirely, it can build on CharacterGen, and the dungeon generator just handles building and filling a dungeon.  Also, then users can generate their own encounters sans dungeon

The stub page has been created.


Github API integration

https://api.github.com/users/lramage94

http://ghchart.rshah.org/lramage94


How do you get the train.txt.npy data?

I have get the vocab.pkl, but I can't understand the numbers in train.txt.npy's meaning. Please help me, thank you!

From your description I am not sure what exactly is that you are asking for. Each document is encoded as bag-of-words, i.e. how many times each of the words (from the vocabulary) occur in a document.

Thank you very much. I get it. Thanks for your sharing of these codes, from which I have learnt a lot.


update(cmake): bump libs to master.

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
This PR bumps libs to master, fixing c-ares build error and thus fixing bundled-deps CI.
Which issue(s) this PR fixes:
Fixes #3248
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @incertum

/milestone 0.39.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, incertum
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Create Subscription from futures::stream::Stream

I have a situation where I have a futures::stream::Stream which I would like to drive events in iced. I am using iced to receive events from a simulation (which can run on a cluster of computers) and display them.
I am a bit new to async/await, so I don't know the best way to solve this problem, but it seems the only way I can have a streaming async subscription right now is to:

On initialize, run a Command::perform that retrieves an item from the stream
On each item retrieved from the stream, run another Command::perform that gets the next item

This works, but is really strange. Subscription says that it it for listening to external events, so it makes the most sense to support this in Subscription.

The idea is that you subscribe in your Application to subscriptions you want to listen to which then kick of messages you can deal with in your update function. The download_progress example subscribes to a stream of download events like this:
match self {
  Example::Downloading { .. } => {
    download::file("https://speed.hetzner.de/100MB.bin")
             .map(Message::DownloadProgressed)
    }
    _ => Subscription::none(),
}

The magic happens in download.rs where we create a Recipe which creates the stream.
Because you already have Stream just wrapping it into a Recipe and mapping it to an Message should be all you need. Much fun!

Yes, as @Songtronix pointed out you should create the stream in Recipe::stream.

Is it not possible to impl Recipe for S where S is a stream of Message? I checked the examples and it seems a bit excessive just to forward a stream into a message. I just want to do something approximately like:
run().map(Message::FromSim)

Is it possible that this can impl Recipe?

I also have to initiate the connection and store an outgoing sink inside the state of the application as well, and I am not sure how I can get both the tx and rx side to the subscription and new() at the same time.

Check out #336.
You will need to create the stream and the channels in Recipe::stream.

Interesting, so I should send sink over a message back to the main thread. That may work, but it might make the UI elements unresponsive temporarily. Additionally, the transmitter would then need to be added as an Option rather than the real thing. Here is where it is happening:
https://github.com/evomata/evonomics/blob/a57b62baf97c1c8a8fb77977a0db03d47210e380/src/main.rs#L91
As you can see, I use a command that indefinitely recreates itself to simulate the Subscription. This allows me to put the tx and rx side in the new(). It would be nice if there was some way for me to create subscriptions using the Flags, as that would eliminate the issue. Alternatively, if I can make a Command that generates a Subscription, that would also work.
I can probably store the transmitter as an Option, but I would rather the UI be initialized in full before it starts, otherwise I might end up having to use the transmitter through an Option, which will make the code more verbose.

That may work, but it might make the UI elements unresponsive temporarily.

I don't think it should as long as you use async/await with Subscription and Command properly.

Additionally, the transmitter would then need to be added as an Option rather than the real thing.

It's very common in Elm applications to model the state using an enum with Loading and Ready variants.

It would be nice if there was some way for me to create subscriptions using the Flags, as that would eliminate the issue. Alternatively, if I can make a Command that generates a Subscription, that would also work.

Subscriptions themselves are not streams, but just a request to keep a certain stream alive (or create it if it does not exist). This is consistent and plays well with the rest of the immediate architecture. The same way you can stop showing a widget by not returning it in view, dropping a stream is just a matter of not returning its Subscription.

I can probably store the transmitter as an Option, but I would rather the UI be initialized in full before it starts, otherwise I might end up having to use the transmitter through an Option, which will make the code more verbose.

I believe this is the way to go. I'd argue the Option or explicit enum variants are necessary because you are dealing with a concurrent model, not because of the current architecture.
As an alternative, you could choose to store your Sim in your application state and use a Command to compute the next tick just like the game_of_life example. And if you are concerned with cloning the whole world before each tick (I measured this for the game_of_life and it was not even close to being the bottleneck), you could wrap it in an Option and use Option::take.

@hecrj My plan is to run the simulation on multiple machines in the future, so the state will live on other computers. For the time being, I will keep things as they are because it seems to work well and it keeps the concerns in the code roughly where I want them. I do agree that it might be good to have two states for the app (sim running and no sim running), and that the transition between them can just have some lag associated with it or a spinner or something. I will look into spinner widgets or how I can deal with the lag between when the simulation is triggered and the actual UI update.
I will also look into the possibility of creating an API that impls Recipe that sends you back both a stream of messages and the sender over it, as you mentioned. I think that concept is common enough to warrant its own API. If I make that, I will open an issue to see if you would like to include it upstream.
Thanks.

My plan is to run the simulation on multiple machines in the future, so the state will live on other computers.

In this case, you will definitely need Loading and Disconnected variants in your application state.

the transition between them can just have some lag associated with it or a spinner or something

I am not exactly sure what you mean by this, nor how your current approach solves this "lag". There is no need to drop the subscription when the simulation isn't running.

@hecrj Right now the UI is not created until the simulation is ready. However, it likely shouldn't be like that. It should probably start up and there should be a loading indicator visually showing that it is not ready yet. It is also possible that you may want to connect to new simulation sources in the UI later. I will have to investigate ways of making this work with the subscription model. For the time being the simulation is just running locally, so there is no issue.


Laracasts content for Object-Oriented Programming

https://laracasts.com/series/object-oriented-bootcamp-in-php is listed as free, but from lesson 2 on it's not. Also, as noted on Laracasts, this series is archived and recommends https://laracasts.com/series/object-oriented-principles-in-php
Maybe in addition from the Laracasts PHP section: https://laracasts.com/series/design-patterns-in-php and https://laracasts.com/series/solid-principles-in-php

Ahhh--I didn't realize that. Thanks @michapietsch! I've updated the URL. I think (it's been a while) our current seeder doesn't even allow for differentiating paid vs free, but we're going to move all the data into Nova and a database very soon.


KAFKA-15169: Added TestCase in RemoteIndexCache

Test Cases Covered

Index Files already exist on disk but not in Cache i.e. RemoteIndexCache should not call  remoteStorageManager to fetch it instead cache it from the local index file present.
RSM returns CorruptedIndex File  i.e. RemoteIndexCache should throw CorruptedIndexException instead of successfull execution.
Deleted Suffix Indexes file already present on disk  i.e. If cleaner thread is slow , then there is a chance of deleted index files present on the disk while in parallel same index Entry is invalidated. To understand more refer https://issues.apache.org/jira/browse/KAFKA-15169

Committer Checklist (excluded from commit message)

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

@showuon @iit2009060  please don't wait for my review on this one. I might not get to it until next week.

@showuon @iit2009060 please don't wait for my review on this one. I might not get to it until next week.

@showuon  Can you review and address @divijvaidya  comments. I have taken care of all  comments.

@iit2009060 , overall LGTM, left some refactor comments.

@showuon  done as mentioned.

Since we are adding new tests, can we create a new RemoteIndexCacheTest under storage module and start writing the tests in java?
The source code is in Java but the test is in scala. We can move the existing tests in a separate PR.

@kamalcph  Can i create a ticket for same and work on it after PR merge.  This has been pending from last week and already approved.

Since we are adding new tests, can we create a new RemoteIndexCacheTest under storage module and start writing the tests in java?
The source code is in Java but the test is in scala. We can move the existing tests in a separate PR.

@showuon  @kamalcph  @divijvaidya  I have rebased changes and resolved conflict on the changes merged by
the PR https://github.com/apache/kafka/pull/14381.
Can you please review and merge it.

@iit2009060 , there are compilation failure, could you fix it?
https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/10/pipeline/

@iit2009060 , there are compilation failure, could you fix it? https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/10/pipeline/

on it.

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning.
If possible, can I fix this issue in your branch? @iit2009060

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning. If possible, can I fix this issue in your branch? @iit2009060

Yes sure @hudeqi

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning. If possible, can I fix this issue in your branch? @iit2009060

Yes sure @hudeqi . This seems to be a case of flaky behaviour i.e. race condition ?

I think so, please cherry-pick this commit, thanks. @iit2009060

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning. If possible, can I fix this issue in your branch? @iit2009060

Yes sure @hudeqi . This seems to be a case of flaky behaviour i.e. race condition ?

I think so, please cherry-pick this commit: https://github.com/hudeqi/kafka/commit/854c773e708a8352d018986873743b325e97c3f5, thanks. @iit2009060

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning. If possible, can I fix this issue in your branch? @iit2009060

Yes sure @hudeqi . This seems to be a case of flaky behaviour i.e. race condition ?

I think so, please cherry-pick this commit: hudeqi@854c773, thanks. @iit2009060

@hudeqi  This is given me bad object/revision . Can you have the command handy to do cherry pick from another fork ?
git cherry-pick 854c773e708a8352d018986873743b325e97c3f5
fatal: bad object 854c773e708a8352d018986873743b325e97c3f5
This seems to be existing in your fork repository

@hudeqi @showuon There is one test case failing related to the above change #14381, can you help me resolving it , in the local it is not failing ? Build / JDK 8 and Scala 2.12 / testClearCacheAndIndexFilesWhenResizeCache() – kafka.log.remote.RemoteIndexCacheTest java.io.UncheckedIOException: java.nio.file.NoSuchFileException: /tmp/kafka-RemoteIndexCacheTest5502978390891743756/mwbAzcjOTtKRIDxlQrgTTw:foo-0/remote-log-index-cache/0_xg9JTaPnS4C47HfCgv2_gA.txnindex.deleted https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/12/tests/

hi, I think this may be the reason: when executing getIndexFileFromRemoteCacheDir, the .deleted file will be listed in the filter(Files.isRegularFile(_)) stage, but when executing path.getFileName.toString, the .deleted file has already been deleted because cleanStarted is already true to start cleaning. If possible, can I fix this issue in your branch? @iit2009060

Yes sure @hudeqi . This seems to be a case of flaky behaviour i.e. race condition ?

I think so, please cherry-pick this commit: hudeqi@854c773, thanks. @iit2009060

@hudeqi This is given me bad object/revision . Can you have the command handy to do cherry pick from another fork ? git cherry-pick 854c773 fatal: bad object 854c773 This seems to be existing in your fork repository I figured out through this link. Will update soon the PR with changes.

@hudeqi  Please review it

Unrelated test failures
https://ci-builds.apache.org/blue/organizations/jenkins/Kafka%2Fkafka-pr/detail/PR-14482/14/tests
@showuon  @divijvaidya  can we merge this changes ?

@showuon  Can we merge this changes ?

Failed tests are unrelated and also failed in trunk build.


Bug report: m365 aad o365group user remove does not remove a user if it is part of the owners and members group

Description
I can be part of the members and owners in a Microsoft 365 Group at the same time. The m365 aad o365group user remove only removes the owner if I am a member and owner at the same time.
Steps to reproduce

Create a Microsoft 365 Group
Add a user to the members and owners of that Group
Execute the m365 aad o365group user remove for that user
The user is only removed from the owners but not from the members

Expected result
The user is removed from both, owners and members
Actual result
The user is removed from the owners only
Environment
Latest CLI version

Closing the issue since this was a pure timing problem. After 1-2 minutes the membership was updated correctly.

After investigating a little bit I found the issue lies in this logic:
https://github.com/pnp/cli-microsoft365/blob/ead66ba17873688fbd5b8aaea53faedad2b8b6db/src/m365/aad/commands/o365group/o365group-user-remove.ts#L59-L84
It queries the owners list and assumes that a user is either in the owner or members list but not both.
This logic will fail for Microsoft 365 Groups and Teams. For example, when you are adding an owner to a Microsoft 365 Team, this person will be added by Teams in the members and owners Group. So removing the user with this command once will only "degrade" the user as member first. On the second run it will remove the user completely.
Ideally the logic is changed to check the members and owners and remove the user from both lists.

Hi @pnp/cli-for-microsoft-365-maintainers,
what do you think about this? I would keep the command parameters as they are but try to delete the membership of the user in both members and owners at the same time to have a clean state.
Do you have any concerns? I do not see breaking changes since this is a bug.
If you do not have any concerns, I am going to pick this up.
br,
Patrick

Correct, it's a bug that we should fix.


Implement security checks on the workflow that built the artifact (list of actions, has permissions etc)

Overview:
I want the workflow that built my artifact:

to use only a set of actions that I have allowed to be used
to have their permissions explicitly set and fit within my boundary permissions

@puerco is interested in this, but I'm not sure if we have a feature lined up to support it.

Rado wants to keep this open.


Batch add documents and add UI preview of process

In this PR:
fixes: #88
fixes: #60
Added information on the UI

In MeiliSearch -> does the collection has an index of the same name in MeiliSearch
Indexing -> are documents being indexed in MeiliSearch
Documents -> number of collection documents already indexed in MeiliSearch
Hooks -> automatic updates in MeiliSearch

Batching addition
Previously only 100 documents were fetched from Strapi and added to MeiliSearch.
Now they are fetched 1000 at a time and added to MeiliSearch in the same batch size.
If there are less than 1000 documents, it still works.
Watching for updates
Updates are watched 2 at a time. In order to track addition more rapidly in the front end.

bors merge


Is there any reason that v0.5x.0 releases (after v0.50.0 ) are not listed in bosh.io anymore?

The latest release of this series is v0.54.0 in GH but bosh.io shows v0.50.0 as the latest one.

Thanks @abg ,
We have two release series. one has already moved to Jammy and using PXC v1.x but the other one has to stay with Xenial, for backward compatibility.
We just switched to github-release but before that we missed few upgrades because we were checking the bosh.io for the most recent versions.


⚠ Bofrost.de has degraded performance

In b29ad4b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1508 ms

Resolved: Bofrost.de performance has improved in 288f086 after 21 minutes.


Ful 21112 v2

This fixes the following bugs:

If the unit is before the number in Canada, it is not accepted
States LA, KY, and MT are seen as street types and not states

This also adds tests for Canada since there were none

I'd encourage you to have better PR description, not just ticket number


How to avoid the endless scroll after the on Mapview

@initiuminc1 if u solve this problem plz notif me

@hunterand1 sure. i am trying fix this

@hunterand1
on the MapView.java add the below code in onTouchEvent(MotionEvent event)
if (!withFloorPlan(event.getX(), event.getY())) {
            Log.d(TAG, "overflow");
}

now i have to find some logic to disable scroll at this point

hey @initiuminc1 ,
sorry, i don't understand you. what means endless scroll ? And u can tell me your detailed questions. thank u!

@onlylemi in the mapview scroll doesn't stop after the map image. it is showing a white background

@initiuminc1 oh, i think i understand u. u can set it by yourself. when the map is disable, the onTouchEvent will not handler scroll. or u can also let the map center.

@onlylemi but i need the scroll and zoom properties and need to block the scroll soon after the it reach the map boundary

@initiuminc1 u can convert  the points coordinate of map to screen by the function named convertMapXYToScreenXY(x, y). And determine whether it is in screen.

@onlylemi is there any way to change the background color from white ?

@onlylemi I just need a work around for this infinite scrolling, in my scenario If I go on zooming I keep on sticking with this infinite scroll, So help me around for this infinite scroll view. Just help me how to overcome with that.

@vivektamilarasan
do you find any work around for this issue ?
I tried a lot with no solution :(


HTTPCPOST: "text/plain"

Environment

Development Kit:      ESP32-S2-Saola-1
V1.2
Module or chip used:  [ESP32S2-WROOM-32]
IDF version (run git describe --tags to find it):
v4.2-351-gce2c615bb
Build System:         [CMake]
Compiler version (run xtensa-esp32-elf-gcc --version to find it):
xtensa-esp32-elf-gcc (crosstool-NG esp-2020r3) 8.4.0
Operating System:     [Windows]
Power Supply:         [USB]

Problem Description
//Detailed problem description goes here.
Expected Behavior
Header request "Content-Type: text/plain" taken into account wiyh HTTPCPOST AT command
Actual Behavior
Header request "Content-Type: text/plain" is not takink into account. Only Content-Type: text/xml
Steps to repropduce

send At command to chip to post on httpbin.org with HTTPCPOST
AT+HTTPCPOST="http://httpbin.org/post",4,1,"Content-Type: text/plain"
chip response:
AT+HTTPCPOST="http://httpbin.org/post",4,1,"Content-Type: text/plain"

OK

Send data to send:
test
chip response
+HTTPCPOST:282,{
"args": {},
"data": "test",
"files": {},
"form": {},
"headers": {
"Content-Length": "4",
"Content-Type": "text/xml",
"Host": "httpbin.org",
"User-Agent": "ESP32 HTTP Client/1.0",
"X-Amzn-Trace-Id": "Root=1-609a741f-3ff4b2c73937431438351862"

+HTTPCPOST:90, },
"json": null,
"origin": "<IP_ADDRESS>",
"url": "http://httpbin.org/post"
}
SEND OK

@xdjbobo could you please capture the packets by wireshark.
It should get some useful information from the packets, which include the HTTP POST request.

same issue with "Content-Type: application/json" -> the server receives: Content-Type: text/xml

@ymichon @xdjbobo it got fixed at: https://github.com/espressif/esp-at/commit/3464b2631a7cd0a46f2a96eef1ef37c18d750493
please check.

It looks like this was rolled into <IP_ADDRESS> for ESP8266 - is the ESP32 release coming soon?

@robertpoll The bugfix has been synchronized to branch https://github.com/espressif/esp-at/tree/release/v<IP_ADDRESS>_esp32,
user can self-compile ESP32-AT firmware on this branch.
The next version of ESP-AT firmware for ESP32 is in preparatory stage, but no an estimated time of release.
Please know.


api: add proxyProtocol in BackendTrafficPolicy

Relates to https://github.com/envoyproxy/gateway/issues/1328

@Xunzhuo for api request, please kindly leave it for more than 24h before merging.


Customize the size of the slide thumb

Please describe the feature you would like to request.
The size of the thumb should be cuztomizable to be able to present more information. Right now when displaying 6 digit numbers it glitches out. Even shortening it as 26.5k i.e. looks bad without proper padding. Please refer to the image below.

I have tried playing with the styles to fix the problem, but the animation and the placement of the thumb is broken when styles are changed:

What is the use-case or motivation for this proposal?
To display the exact value and to add customizability.

Closing this since I don't see us adding support for the feature unless it's added to the material design spec.


Additional: Fix multiple mime-type validation

Fixes #1775

Rebased and merged. thx @Arkni , great job!


Able to create related (belongsto) record when In view page

Package
filament/forms
Package Version
2.12.3
Laravel Version
9.12.2
Livewire Version
2.10.5
PHP Version
8.1.3
Bug description
Pretty simple, you are able to open the create belongsto record when viewing (not editing) a record
Steps to reproduce

Create a resource with a view and edit page
Use the following field:
                  BelongsToSelect::make('brand_id')                                  ->relationship('brand', 'name')                                  ->createOptionForm([                                      Forms\Components\TextInput::make('name')                                          ->label(__('forms.labels.name'))                                          ->required()                                  ]),
Open record
You are able to open the model on the view page

Relevant log output
No response

Nor sure if I understand correctly? but do you mean like this? https://twitter.com/danjharrin/status/1524052133057908737