InaccessibleObjectException is not thrown by gradle

gradle test passes successfully in CI but fails in production with java.lang.reflect.InaccessibleObjectException, i.e. tests pass when executed by gradle by fails when executed with java -cp ....
I can see in logs that multiple --add-open are used in gradle daemon:
2021-10-16T12:54:17.238+0800 [DEBUG] [org.gradle.launcher.daemon.server.exec.ExecuteBuild] Executing build with daemon context: DefaultDaemonContext[uid=17080b27-1ebe-47be-b046-fd5766bfbfab,javaHome=/home/gtx/.jdks/semeru-jdk-17+35,daemonRegistryDir=/home/gtx/.gradle/daemon,pid=21961,idleTimeout=10800000,priority=NORMAL,
daemonOpts=-XX:MaxMetaspaceSize=256m,-XX:+HeapDumpOnOutOfMemoryError,--add-opens,java.base/java.util=ALL-UNNAMED,--add-opens,java.base/java.lang=ALL-UNNAMED,--add-opens,java.base/java.lang.invoke=ALL-UNNAMED,--add-opens,java.base/java.util=ALL-UNNAMED,--add-opens,java.prefs/java.util.prefs=ALL-UNNAMED,--add-opens,java.prefs/java.util.prefs=ALL-UNNAMED,--add-opens,java.base/java.nio.charset=ALL-UNNAMED,--add-opens,java.base/java.net=ALL-UNNAMED,--add-opens,java.base/java.util.concurrent.atomic=ALL-UNNAMED,-Xms256m,-Xmx512m,-Dfile.encoding=UTF-8,-Duser.country=AU,-Duser.language=en,-Duser.variant]

Likely by following class: https://github.com/gradle/gradle/blob/4a482a4c36576fafe8398e4717bd779f7f76a3e5/subprojects/base-services/src/main/java/org/gradle/internal/jvm/JpmsConfiguration.java
Expected Behavior
Expecting tests to fail with same error as production code.
Current Behavior
Test passes successfully.
Context
I would like to rely on tests and CI to catch runtime exceptions. I was also able to reproduce same issue with java 16, but not as reliable (some toolchains pass, some fail without clear difference). Is there a way to isolate from gradle daemon environment?
Steps to Reproduce
Sample project: https://github.com/sukolenvo/sample-gradle-java-17
./gradlew test
Your Environment
I don't think there is something specific to my environment, nevertheless build scan attached.
Build scan URL: https://scans.gradle.com/s/hst2fqczytf4u

I think this should be considered as a bug and should be re-opened. We also stumbled on the same behaviour:

Running tests via Gradle passes on CI
Code fails at runtime in production

We usually put trust in our test suites, and especially during upgrades from Java 8 to Java 17 we hope for our tests to catch as many issues as possible - even if it's our invalid code which should be fixed. Running the tests without Gradle makes them rightly fail. Gradle's defaults (JpmsConfiguration) hide such potential issues.
It seems like the issue originates in #6725 to work around some warnings and has probably evolved over time. I'm not entirely sure on which levels the --add-opens JVM args of the JpmsConfiguration class are used, but I'd expect that Gradle should not apply such JVM args when compiling and testing a user's project. With Java's changed defaults in Java 16 and with the removal of --illegal-access in Java 17 (see JEP 403) more Gradle users will stumble upon this and I suggest to at least document more visibly how to disable the JpmsConfiguration, or, even better, reconsider its use in the Groovy compiler and test runner.
An annoying warning log on the CI has less impact than broken code in production.
Breaking builds due to broken code are a good thing.


[OwnershipUtils] NFC: Two small cleanups.

Factored getLocalScopeEndingInstructions through visitLocalScopeEndingUses and deleted a dead declaration.

@swift-ci please test and merge


random_seed seems to be ignored (or at least inconsistent) for inflight_batcher_llm

System Info
I've converted Llama 3 using TensorRT-LLM's convert_checkpoint script, and am serving it with the inflight_batcher_llm template. I'm trying to get diverse samples for a fixed input, but I've found that if I make several requests concurrently, several will have identical outputs.
I'm setting top_p=1, top_k=1024, temperature=1.0, beam_width=1, and generating a unique random seed for each request. The requests are being made over the gRPC API, and I'm using v0.9.0 of TensorRT-LLM and tensorrtllm_backend.
Who can help?
@byshiue
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[X] My own task or dataset (give details below)

Reproduction

Serve a model (essentially following this guide, with some settings changes: https://developer.nvidia.com/blog/turbocharging-meta-llama-3-performance-with-nvidia-tensorrt-llm-and-nvidia-triton-inference-server/)
Make 5 gRPC requests concurrently

Expected behavior
I expect each request with a different seed to yield a different response
actual behavior
Several of the 5 responses are consistently identical
additional notes
I changed the script I'm using for testing to wait for a response before sending another request, and this results in all 5 outputs being distinct, so it seems like the concurrency/inflight batching really is the problem.

Another detail which is interesting is that the identical sequences I observe in the concurrent case are the same run to run, even though I'm sampling the random seed from 1-1,000,000.
For example, with the input of <|begin_of_text|>Hello, my name is, I saw a continuation of of "Ahmed, and I am an experienced Software Engineer with proficiency..."  in 3/5 responses, and then 2/5 responses on the next run. I did not observe this prefix at all when making requests serially.

@byshiue I incorrectly typed your name when opening this issue originally. Can you comment on whether there's a workaround for this? It's currently making batch inference effectively useless.

@byshiue I incorrectly typed your name when opening this issue originally. Can you comment on whether there's a workaround for this? It's currently making batch inference effectively useless.

@dyoshida-continua I applied the solution described in this pull request: NVIDIA/TensorRT-LLM#1742, and it resolved the issue for me.

Thank you for the help replying, @chiendb97 . Since the https://github.com/NVIDIA/TensorRT-LLM/pull/1742 is related to fix of random seed setting, it might be related to your issue, @dyoshida-continua . Could you take a try?


updated info on farmerbot, no farming rewards on qa net, and need to …

updated info on farmerbot:

no farming rewards on qa net
need to wipe disks when changing network

@xmonader
Thanks!


Can only see the line you are editing

When editing a specific file, you cannot see any other lines besides the one you are currently in. Original link.

Quick fix:
.mtk1 {
    color: #6D7178;
}


Lark Provider can't work, it show "unexpected end of JSON input"

I used the AIO Docker image for testing.
Lark authenticates successfully, but returns to the web site with the following error.
我使用AIO Docker镜像做测试。
飞书验证成功通过（能收到登录成功通知），但返回到网站时却报以下错误。

@Raven-L please paste the Go backend console output (with the crash) here

@Raven-L first, use latest version (v1.315.1 or master)

I use AIO v1.315.0 .  and it work.  The reason should be that I pulled a very old mirror（2021）, sorry

我用了AIO最新版本还是出现了一样的问题


Drones spawn in white box for race...

Hello I can't seem to overcome 'the white box' when a drone race starts. Finicking around with basic commands I realized the multirotor would sometimes spawn in a white box when on the soccer field map. I had to backspace to reset the drones position and I was able to play with it. Whenever I replay the sample race script, baseline_racer.py, the environment resets and the level reloads putting me back in the white box. How do I deal with this white box? Is there any way I can delete it? :)

Anyone struggling here getting your vehicle out of the start box (baseline_racer.py) . I simply listed the vehicles start position (drone_1) and used moveToPositionAsync to go up, out of the box (I should have probably tried this combination initially). takeoffAsync() got me nowhere so this solution was a little confusing.
def placement(self):

    print("main vehicle's posistion (drone_1): %s" % self.airsim_client.simGetVehiclePose())

x.placement()
##output=start location

def move(self):                                                                                #X   Y   Z

    self.airsim_client.moveToPositionAsync(.75, -2, -5, 5).join() #start location manually entered (.75,-2,-5) with displacement against the Z-axis

So, displace upwards with the help of the above commands before you move along the spline if this box annoys you...


Improve Docker for Web, update Readme.

@nardev @ihti please review and delete the source branch after PR is merged. Thanks


doc: net: Fix VLAN documentation in doc and in sample

The VLAN documentation had double wording and proper links to files
in github was missing, which did not look good in the rendered HTML
page.
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

Codecov Report

Merging #14164 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14164   +/-   ##
=======================================
  Coverage   52.68%   52.68%
=======================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
=======================================
  Hits        23956    23956
  Misses      16640    16640
  Partials     4877     4877

Continue to review full report at Codecov.

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


kpng winuserspace backend

Windows userspace mode - integrating backend with services and endpoints. OnEndpointsUpdate is not required, this list is kept with add/delete only.
Fixes https://github.com/kubernetes-sigs/kpng/issues/65

Ah yes, finally ! ok , thanks amim now we can deprecate userspace out of intree for windowsssss
/lgtm
/approve

/approve


The image in the 6.6 section is wrong

I think there should have been another image there... my edit should be rejected but I didn't know how file "issues". Not a github guru :)

@omartin2010 : Thanks for your contribution to the Azure documentation! The author, @UnifyCloud, has been notified to review your proposed change.

Not sure what the issue is here. Can you provide more details? Thanks!

There was an image of an ExpressRoute circuit in the section of Web Application Firewall, which I think is not the image that should show up in section 6.6 AND in section 6.4. Does that make sense? Or is it intended?
From: Tom Shinder<EMAIL_ADDRESS>Sent: Tuesday, May 9, 2017 16:10
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Olivier Martin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] The image in the 6.6 section is wrong (#1620)
Not sure what the issue is here. Can you provide more details? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/azure-docs/pull/1620#issuecomment-300286627, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK2FXuQcZmcECI_8RDXksfCDc0X99gbzks5r4MgvgaJpZM4NQp3i.

The images provided in both the sections 6.4 and 6.6 are fine as per the following MS references.
Reference link for Express Route Image
Reference link for Web Application Firewall Image

#sign-off

Well the links to the images you are including here are the right ones. However on the page that I am looking at on github (I can’t find the page back on Azure), sections 6.4 and 6.6 have the same image that appear on my screen…
And by the way I didnt know how to do a PR without changing something (or creating an issue – Im no git wizard) so I’ve added a !wrong image comment before the actual image. Might need to revert the change…
From: unifycloud<EMAIL_ADDRESS>Sent: Wednesday, May 10, 2017 5:01
To: Microsoft/azure-docs<EMAIL_ADDRESS>Cc: Olivier Martin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/azure-docs] The image in the 6.6 section is wrong (#1620)
The images provided in both the sections 6.4 and 6.6 are fine as per the following MS references.
Reference link for Express Route Imagehttps://docs.microsoft.com/en-us/azure/expressroute/expressroute-introduction
Reference link for Web Application Firewall Imagehttps://azure.microsoft.com/en-in/blog/azure-web-application-firewall-waf-generally-available/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/azure-docs/pull/1620#issuecomment-300420285, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AK2FXrRhjFzfMGO91o9UFagdtf47-dYxks5r4XywgaJpZM4NQp3i.

oh! Now we got the mistake and updated the images on git hub.

#sign-off


Can not use spaces in class names when using List/Vec

Problem
When using a List (a tuple within the html! macro) or a Vec of classes and one item contains spaces e.g.
html! {

{ "Example" }

}

and using yew from master the resulting class string is "hello hello world".
and using yew version 0.11.0 the application crashes (not due to a panic)

this is very similar to #121, which was fixed by #289
#927 fixes the crash, but the results are non-unique class names and i guess that vtag.classes.contains("world") will return false.

Steps To Reproduce

Include the above code within a components view function.
Inspect the class string or read the console of the used web browser.

Expected behavior
I expected unique class names: "hello world" or the support of non-unique class names when specifying the classes as string.
Screenshots
master (as of commit 1ede16795a8f08975708e1808c0d528f31d5c946)

Version 0.11.0

(Note that Example is not rendered)

Environment:

Yew version v0.11.0, master as of commit 1ede16795a8f08975708e1808c0d528f31d5c946
Rust version 1.41.0 (5e1a79984 2020-01-27)
stdweb / web-sys version [e.g. web-sys v0.3.33]
OS: Windows 10, Ubuntu 16.04.6 LTS running on WSL
Browser Firefox
Browser version 72.0.2 (64-bit)

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[x] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Suggestions
I have a solution in mind for both of my expected behaviors, but i am not sure if either of them is a desired one.
Should i open another issue to suggest some kind of feature to support non-unique class names?
Non-unique class names can be desired if e.g. Semantic UI is used, which is also one reason why the ordering of classes is supported in yew: https://github.com/yewstack/yew/issues/393.
I suggest to also add documentation for the expected behavior to

https://yew.rs/docs/concepts/html/elements#classes
https://github.com/yewstack/yew/blob/1ede16795a8f08975708e1808c0d528f31d5c946/src/virtual_dom/mod.rs#L59
https://github.com/yewstack/yew/blob/1ede16795a8f08975708e1808c0d528f31d5c946/src/virtual_dom/vtag.rs#L125

@liquidblock by non-unique class names do you mean that class names can be repeated?

@liquidblock by non-unique class names do you mean that class names can be repeated?

Yes

Should i open another issue to suggest some kind of feature to support non-unique class names?

I'm in favour of non-unique class names if it's needed for using Semantic-UI. So in that case, we could probably just change this issue to reflect supporting that feature. Thoughts?

Semantic-UI has an closed issue, where they discussed the usage of repeating class names: https://github.com/Semantic-Org/Semantic-UI/issues/1526
An example class string would be:

"sixteen wide mobile eight wide tablet four wide computer column"

There are some arguments about the HTML5 spec:

The classes that an HTML element has assigned to it consists of all the classes returned when the value of the class attribute is split on spaces. (Duplicates are ignored.)

As far as i am aware of, this is the reason, that https://github.com/meteor/blaze does not support repeating class names, but there is a library for meteor supporting Semantic-UI: https://github.com/Semantic-Org/Semantic-UI-Meteor
The above quote from the spec is outdated, as far as i know.
Below are some current quotes from the spec:
https://html.spec.whatwg.org/multipage/dom.html#classes

When specified on HTML elements, the class attribute must have a value that is a set of space-separated tokens representing the various classes that the element belongs to.

Which itself links to https://html.spec.whatwg.org/multipage/common-microsyntaxes.html#set-of-space-separated-tokens:

A set of space-separated tokens is a string containing zero or more words (known as tokens) separated by one or more ASCII whitespace, where words consist of any string of one or more characters, none of which are ASCII whitespace.
[...]
An unordered set of unique space-separated tokens is a set of space-separated tokens where none of the tokens are duplicated.
An ordered set of unique space-separated tokens is a set of space-separated tokens where none of the tokens are duplicated but where the order of the tokens is meaningful.
[...]

Some things i noticed:

The HTML spec says that the class attribute is a set of space-separated tokens, but it does not specify neither an unordered set of unique space-separated tokens nor an ordered set of unique space-separated tokens

The classList attribute is defined as a set, which has a different definition:

Some lists are designated as ordered sets. An ordered set is a list with the additional semantic that it must not contain the same item twice.

https://dom.spec.whatwg.org/#dom-element-classlist.

But the className attribute is defined as must reflect the "class" content attribute.

The classList does not support for repeated class names, but i am not so sure about className or the class HTML attribute.
Maybe someone can elaborate on the definition of the spec regarding unique class names?
Regardless of the spec requiring unique names or not, i would like to suggest at least the optional usage of repeated class names.
E.g. in the case of the spec requiring unique names the cargo features feature could be used for conditional compilation options.

@ADustyOldMuffin you could try this issue as well. It requires less knowledge of how Yew works internally

I'll be honest, I did not understand the outcome of the comments about the using multiple class names. The implementation proposed in #1084 continues using the IndexSet in the current Classes struct which enforces unique class names.

@bryanjswift that's fine, this issue was about spaces in class names. We can create a new issue for non-unique classnames if it's really needed


[WIP] Generate cloud images from config file

This commit switches the generation workflow from a script to a config
file method.
Signed-off-by: Costin Galan<EMAIL_ADDRESS>

closing this PR, as the PR will be made to the experimental branch


LDAP login with spaces in CN

Hello,
I have a question is it possible with ldap to login with CN=  Geoffry Van den Eede
I get the following error when I try that.
Or is there another solution so I can login with the sAMAccountName attribute? I can't get that to work :(
2016/10/20 15:36:26 [T] LDAP will bind directly via UserDN template: CN=%s,OU=DVIT Users,DC=DVIT,DC=LOCAL
2016/10/20 15:36:26 [D] 'Geoffry Van den Eede' contains invalid DN characters. Aborting.
2016/10/20 15:36:26 [W] Failed to login 'Geoffry Van den Eede' via 'login': user does not exist [uid: 0, name: Geoffry Van den Eede]
Thanks already!

This is claimed to be fixed by merging #3906, please help test on latest release or master branch.


Ubiquiti AC Lite AP Only Allows Manual Config of 5GHz Channels 36 - 48

It is not possible to manually configure the Ubiquiti AC Lite AP for channels beyond channel 48 from the CT dashboard.  When the channel selection is set to "auto", the the AP may select channels beyond channel 48 itself, but this needs to be available as a manual operation too.
Actual Behavior: Not possible to select channels above ch 48 on the 5GHz band

What is the issue? * It is not possible to perform manual 5GHz channel planning beyond using channels 36, 40, 44 & 48 which is too limited for general usage
What is the expected behavior? Manual configuration of channels 52, 56, 60, 64, 100, 104, 108, 112, 116, 120, 124, 128, 132, 140 should also be available for manual selection in the UK

Additional Information:

Browser Type: * Chrome
Browser Version: * Version 56.0.2924.87
OS: * Windows
Page URL: *https://my.ctapp.io/#/locations/home-lyndhurst-grove-jpiqs9iuj_ptuwohpovuda/devices/5hxipfdwmw8habz7hxpbwq/edit

Do not modify the titles or questions. Simply add your responses to the ends of the questions.
Add more lines if needed.

Add screenshots, gifs, illustrations as required to demonstrate your issue.

No duplicates, please use the search tool.

Think this is a duplicate of #252
https://github.com/cucumber-tony/cucumber-frontend/issues/252

Closing but still need to address some country specific validations.


Task cleanup fixes for decommission and delete

Requests in deleting can get stuck if a task was already bouncing, fix that by allowing the deleting cleanup to overwrite a previous one
If a slave is decommissioned at the same time as a deploy is happening, a new task from the new deploy can be left in a cleaning state. The pending request was always being made against the active deploy. It will now be made against the deploy for the particular task being cleaned

🚢


🐛 [Bug]: 复制静态文件时接口报错

Environment
QQ浏览器11.1 (5155) Chromium94.0.4606.71  IE11.3636.19041.0
Version
v16.20.0
Version
tiny-engine@ D:\其他项目\tiny-engine  empty
Link to minimal reproduction
https://github.com/opentiny/tiny-engine/
Step to reproduce

What is expected
No response
What is actually happening
No response
Any additional comments (optional)
No response

由于复制的静态文件的schema中包含了id及_id两个数据，因此在提交时由于数据库中已经存在所以报错导致提交失败。


It doen't compile

node-fpgrowth/dist/fpgrowth.d.ts(1,23): error TS2688: Cannot find type definition file for 'node'.
node-fpgrowth/dist/fpgrowth.d.ts(15,22): error TS2420: Class 'FPGrowth' incorrectly implements interface 'IFPGrowthEvents'.
Property 'on' is missing in type 'FPGrowth'.

Added Building from source README as well as a runnable example


Firefox: Some icons work, some don't

Hi,
I'm finding some icons work, but others don't in Firefox. Chrome and safari behave as expected. Any ideas?
The CSS and the font files are all on the same domain (primary domain).

Never mind,  v1.4 was being as well as v2. WHY and HOW?!! :facepalm:


Trouble installing

OSX 10.10.4. Could there be an undocumented gdcp prerequisite?
cmckay$ which python
/opt/local/bin/python
sudo pip install --upgrade pydrive
[snip]
sudo pip install --upgrade backoff
[snip]
cmckay$ gdcp
Traceback (most recent call last):
  File "/usr/local/bin/gdcp", line 3, in <module>
    import apiclient.errors
ImportError: No module named apiclient.errors

No, installing pydrive should bring in the Google api library that contains apiclient. I actually had exactly this problem yesterday. It turned out that the pip that was being run in the context of sudo was not tied to the Python installation that my normal user uses. So I pip installed to one Python location and then gdcp ran with a different Python which didn't have any prereqs installed.

Ug, I was sure that I closed this bug yesterday after I discovered a similar resolution! I resolved mine by installing the mac ports version of pip, then installing pydrive and backoff again. Anyway, thanks for looking into it.


Progress bar of waves is getting stuck in safari browser

Bug description:
At times in safari progress bar is not getting seek with the song, while song is played perfectly but there is no progress seen in the waves. It works fine in chrome.
Environment:

Browser: Safari

Minimal code snippet:
        wavesurferInstance.on("audioprocess", () => {           if (currentTrackRef.current.id !== currentTrack.id) {             wavesurferInstance.pause();             return;           }           const per =             wavesurferInstance.getCurrentTime() /             wavesurferInstance.getDuration();           if (per === 1) {             currentTrack?.song_wavesurfer?.seekTo(0);           }           const array = allSongsRef?.current?.findLast((item) => {             return item?.song_url?.id === currentTrack.id;           });           if (per > 0) {             array?.song_wavesurfer?.seekTo(per);             currentTrack?.song_wavesurfer?.seekTo(per);           }         });       }
Expected result
Waves must show the progress as songs plays
Obtained result
Screenshots

Hello @lucastheis @katspaugh
I am also facing the same issue. Please share your suggestion for the same.
Thanks
Ankit


Graphql filter on one type doesn't work

Hi,
Question:
I use the queries below in the gatsby-graphql playground.
When I use the current query:
query MyQuery {
  allStrapiJob(filter: {data: {elemMatch: {id: {eq: 2}}}}) {
    nodes {
      data {
        attributes {
          slug
          title
        }
        id
      }
      id
    }
  }
}

I get this as a result
{
  "data": {
    "allStrapiJob": {
      "nodes": [
        {
          "data": [
            {
              "attributes": {
                "slug": "helpdesk-engineer-uk",
                "title": "Helpdesk Engineer"
              },
              "id": 1
            },
            {
              "attributes": {
                "slug": "frontend-developer",
                "title": "Frontend developer"
              },
              "id": 2
            }
          ],
          "id": "Job_undefined"
        }
      ]
    }
  },
  "extensions": {}
}

Always 2 results (there are 2 entries in strapi at the moment)
If I change the id to filter, to either 1 or 2, I always get 2 results.
When changing it to anything else then 1 or 2, I get nothing as a result.
What am I doing wrong here?
Also, I got "strapiJob" to use.
query MyQuery {
  strapiJob(data: {elemMatch: {id: {eq: 1}}}) {
    data {
      attributes {
        slug
        title
        description
      }
      id
    }
  }
}

Same result, always 2 entries.

Seems that this is due to the fact that the nodes are wrapped in a data object. So the node that includes the slug is returned, which is in fact everything. But.... Correct me if I'm wrong.

@jan-wagenaar These queries are made in the graphql playground from gatsby.
So that's how the queries need to be build...I guess?
There is no other way to get those slug/title/description...

Hi,
Question: I use the queries below in the gatsby-graphql playground. When I use the current query:
query MyQuery {
  allStrapiJob(filter: {data: {elemMatch: {id: {eq: 2}}}}) {
    nodes {
      data {
        attributes {
          slug
          title
        }
        id
      }
      id
    }
  }
}

I get this as a result
{
  "data": {
    "allStrapiJob": {
      "nodes": [
        {
          "data": [
            {
              "attributes": {
                "slug": "helpdesk-engineer-uk",
                "title": "Helpdesk Engineer"
              },
              "id": 1
            },
            {
              "attributes": {
                "slug": "frontend-developer",
                "title": "Frontend developer"
              },
              "id": 2
            }
          ],
          "id": "Job_undefined"
        }
      ]
    }
  },
  "extensions": {}
}

Always 2 results (there are 2 entries in strapi at the moment) If I change the id to filter, to either 1 or 2, I always get 2 results. When changing it to anything else then 1 or 2, I get nothing as a result.
What am I doing wrong here?
Also, I got "strapiJob" to use.
query MyQuery {
  strapiJob(data: {elemMatch: {id: {eq: 1}}}) {
    data {
      attributes {
        slug
        title
        description
      }
      id
    }
  }
}

Same result, always 2 entries.

I also encounter this problem and don't know how to solve it

@jan-wagenaar These queries are made in the graphql playground from gatsby. So that's how the queries need to be build...I guess? There is no other way to get those slug/title/description...

I also encounter this problem and don't know how to solve it😭

@gzhanghui actually happy to see I'm not the only one :)

@jan-wagenaar These queries are made in the graphql playground from gatsby. So that's how the queries need to be build...I guess? There is no other way to get those slug/title/description...

Yeah I do not know an workaround either. RIght now I'm retrieving everything and use an array.filter() to get the right item. But definitely feels too hacky.

@jan-wagenaar if you are dealing with a large dataset, that is indeed not a solid solution... This needs to be addressed to the core-team, imo...

Is your query for a page?
In that case, one thing you should try if it's possible to use the Gatsby Node API to filter the data on build time. Then pass a custom object to your screen.

It's for a page, off course, but gatsby node api needs the same query as far as I know...

That right, but in the gatsby-node.js code you may be able to implement custom logic.

I will try it for myself. Ofcourse, this is a temporary workaround....

That could work yes...
create a clone of the nodes[0].data array
and the custom logic would be to find the current node again in the cloned array, use that data to pas it to the template an remove that from the cloned array again; so that on the next cycle of the foreach, it will find it's data faster and faster.
but not a solid solution yet no :)

Nice I was able to use this as an workaround. My code:
  const { data } = await graphql(
query {
allStrapiWorks {
nodes {
data {
id
attributes {
slug
name
description
}
}
}
}
}
`)
data.allStrapiWorks.nodes[0].data.forEach(node => {
const slug = node.attributes.slug
actions.createPage({
path: /work/${slug},
component: require.resolve(./src/templates/work-detail.js),
context: {
workItem: node.attributes
},
})
})
}`
In your component use pageContext to get the passed object:
function Product({ pageContext }) {
  const { workItem} = pageContext
  return (
    <div>
      Name: {workItem.name}
      Description: {workItem.description}
    </div>
  )
}

Thank you so much! I was stuck on this issue for the last two weeks... This solved all my query issues!

I'm currently faced this issue, and figured out a way to get single data in the gatsby-graphql with new node like this:
graphQL
query Articles {
  strapiArticle(data: {id: {eq: 3}}) {
    data {
      id
      attributes {
        publishDate
      }
    }
    id
  }
  strapiArticles {
    data {
      attributes {
        publishDate
      }
      id
    }
    id
  }
}

data
{
  "data": {
    "strapiArticle": {
      "data": {
        "id": 3,
        "attributes": {
          "publishDate": "2022-02-10"
        }
      },
      "id": "21e77627-27c5-5f52-b0c4-62aa23ccfcf9"
    },
    "strapiArticles": {
      "data": [
        {
          "attributes": {
            "publishDate": "2022-01-30"
          },
          "id": 1
        },
        {
          "attributes": {
            "publishDate": "2022-02-01"
          },
          "id": 2
        },
        {
          "attributes": {
            "publishDate": "2022-02-10"
          },
          "id": 3
        }
      ],
      "id": "Articles_undefined"
    }
  },
  "extensions": {}
}

strapiArticles is the original node which the gatsby-source-strapi made for us.
strapiArticle is the one we created, and we can get single data with $id .
The workaround here is to get out the data (inside the array), and then generate a new node with Gatsby's API createNode .
gatsby-node.js
module.exports.onCreateNode = async ({ node, actions, createNodeId, createContentDigest }) => {
  const { createNode } = actions;

  if(node.internal.type === "StrapiArticles") {
    node.data.forEach(d => {
      const newNode = {
        data: d,
        id: createNodeId(`StrapiArticle-${d.id}`),
        parent: node.id,
        children: [],
        internal: {
          type: "StrapiArticle",
          contentDigest: createContentDigest(d)
        },
      };

      createNode(newNode);
    })
  }
};

References
Gatsby Node APIs
onCreateNode
Gatsby Actions
createNode
Node Interface
contentDigest
Node API Helper
createContentDigest
createNodeId

This is very annoying to be honest. Could you please let me know when we can expect a fix.
Actually if it matches on two elements it is returning two sets of results.


Compiling for iOS 10.0, but module 'Alamofire' has a minimum deployment target of iOS 11.0

Hi, our project depends on SuperAwesome pod and recently the builds started failing because Moya is targeting iOS 10, but the Alamofire dependency have bumped their iOS minimum target to 11 so we get this error:
Compiling for iOS 10.0, but module 'Alamofire' has a minimum deployment target of iOS 11.0: /Users/[...]/Alamofire/Alamofire.framework/Modules/Alamofire.swiftmodule/arm64-apple-ios.swiftmodule

This is a Moya 14.0 dependency issue and needs to either target Alamofire 5.6.X, or update the minimum iOS version.

5.7.1 should be out now (depending on CocoaPods' deployment speed), so this should be resolved.

I can confirm this issue has been resolved.

Working for me as well. Thanks @jshier.


Invalid characters showing in vim-airline.

I believe this plugin is responsible for breaking some vim-airline interactions. When I have it enabled, and I use the ctrl-p plugin, I see many invalid ^ characters in my status bar.

Should all be fixed now, let me know if this is not the case
Here are some relevant links if anyone is interested in this issue:

https://github.com/vim-airline/vim-airline/issues/258
https://stackoverflow.com/questions/7223309/vim-statusline-uses-carret-symbols-for-spaces-how-to-switch-to-spaces

Thanks Connor. verified its working!


Revert the Revert of #34545: [Dependency Scanner] Add missing clang overlay dependencies for placeholder modules

With the added test fixed to be more generic to platforms.

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 4260808eb6dc441975db76f5feb18c9fd773c657

@swift-ci please test


Improve cross platform story

Update to the latest PCL and .NET Core stuff once that stabilizes on Windows and Linux.

Libraries and tools are now fully .NET Core. There is still only a UWP host. #25 tracks a Linux host.


Unclear complete scenario

Two things are unclear for me:

The complete scenario publishing to specific runtime. If i follow the installation example for linux- what do i need to then run the exe on linux? Will it just work by double click?         This page https://learn.microsoft.com/en-us/dotnet/core/deploying/deploy-with-cli#self-contained-deployment suggests that the native libraries will need to be there somehow and install specific runtime. Does that mean that if i use the -r linux option it would just run from linux or do i need to install dotnet as well?
The page states that RuntimeIdentifier will set SelfContained to true by default but the CLI example sets it to false - does that mean that it sets it to true but you can force it to be false ?

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

ID: e1932751-7c2b-ce92-936e-e97f8581d5fa
Version Independent ID: de677d1f-386a-e4c3-fdd0-6806b3ef2c0e
Content: Create a single file for application deployment - .NET
Content Source: docs/core/deploying/single-file/overview.md
Product: dotnet-fundamentals
GitHub Login: @LakshanF
Microsoft Alias: lakshanf

Associated WorkItem - 147248

Also the example uses both
net6.0
and
true
How is it possible these work together ? isnt the second one means its not for a specific framework?

Thank you for this issue. I'll look into this this month.

Sorry for the delay in getting back to you about your questions.

The complete scenario publishing to specific runtime. If i follow the installation example for linux- what do i need to then run the exe on linux? Will it just work by double click?

There isn't an equivalent concept of .exe on linux. When you have a binary on linux, you have to have to set it with executable permissions, which would let the desktop double click it or let a terminal run it.

This page suggests that the native libraries will need to be there somehow and install specific runtime. Does that mean that if i use the -r linux option it would just run from linux or do i need to install dotnet as well?

Self-contained apps include the dotnet runtime with the app, so it doesn't require that the runtime be preinstalled. However, dotnet itself has dependencies which need to be installed on the host machine. Depending on which linux distro you're using, the dependencies can be different. For example, here are the dependencies required for Debian.
While publishing an app puts all the binaries of the app and dotnet together for you to distribute, it doesn't "package" it up into an installer or anything like that. You'll still have to handle that on your own if you want a nice end-to-end experience for your users. This is why package managers are so popular, you can publish your app as a package for Ubuntu, then list the libraries it's dependent upon, and they'll be installed automatically.
This is kind of the case for Windows too, though not so much because Windows includes a lot of the dependencies .NET needs by default. But, take Windows 7 for example. It's an old OS and was supported with .NET 6 (but not .NET 7). The dependencies of .NET 6 for that OS include the Microsoft Visual C++ 2015-2019 Redistributable which may or may not be installed already. But the latest redistributable comes with Windows 10/11 and on these OSs you don't have to worry about it because it's already there.

The page states that RuntimeIdentifier will set SelfContained to true by default but the CLI example sets it to false - does that mean that it sets it to true but you can force it to be false ?

Correct. Any parameters on the command line will override the parameters set in the project file.

Also the example uses both
net6.0
and
true
How is it possible these work together ? isnt the second one means its not for a specific framework?

(assuming you mean true for SelfContained)
You're always targeting a specific framework (6.0 in this case), but what you may not be targeting is a platform (windows vs linux for example). When you publish self-contained, you need to designate a platform (it defaults to your current platform) so that the platform-specific binary can be built, the .net runtime for that platform is included, and any app-specific dependencies you have are also included (not .net native dependencies mind you). But if self-contained is false, you still generate a platform-specific executable, and it includes any app-specific dependencies you have are included, but the .net runtime isn't included.
I think this answers your questions, so I'll close this issue. Please feel free to continue to comment though.


sched/timer: get the previous reload value correctly

Summary
sched/timer: get the previous reload value correctly
Impact
Testing

@Donny9 is this correct?
 *   ovalue - A location in which to return the time remaining from the
 *     previous timer setting.	 *     previous timer setting.

This says "time remaining" not total time.
time remaining make sense to me. total delay does not

https://man7.org/linux/man-pages/man2/timer_settime.2.html.
If old_value is not NULL, then it points to a buffer that is used to return the previous interval of the timer (in old_value->it_interval) and the amount of time until  the  timer  would  previously have next expired (in old_value->it_value).
so, we can get time remaining by old_value->it_value, get  the previous interval of the timer by old_value->it_interval. And the pt_delay records the interval time.

@Donny9 is this correct?
 *   ovalue - A location in which to return the time remaining from the
 *     previous timer setting.	 *     previous timer setting.

This says "time remaining" not total time.
time remaining make sense to me. total delay does not

https://man7.org/linux/man-pages/man2/timer_settime.2.html.
If old_value is not NULL, then it points to a buffer that is used to return the previous interval of the timer (in old_value->it_interval) and the amount of time until  the  timer  would  previously have next expired (in old_value->it_value).
so, we can get time remaining by old_value->it_value, get  the previous interval of the timer by old_value->it_interval. And the pt_delay records the interval time.

BTW, timer_gettime use pt_delay(https://github.com/apache/incubator-nuttx/blob/master/sched/timer/timer_gettime.c#L71-L91):
int timer_gettime(timer_t timerid, FAR struct itimerspec *value)
{
  FAR struct posix_timer_s *timer = (FAR struct posix_timer_s *)timerid;
  sclock_t ticks;

  if (!timer || !value)
    {
      set_errno(EINVAL);
      return ERROR;
    }

  /* Get the number of ticks before the underlying watchdog expires */

  ticks = wd_gettime(&timer->pt_wdog);

  /* Convert that to a struct timespec and return it */

  clock_ticks2time(ticks, &value->it_value);
  clock_ticks2time(timer->pt_delay, &value->it_interval);
  return OK;
}

BTW, timer_gettime use pt_delay(https://github.com/apache/incubator-nuttx/blob/master/sched/timer/timer_gettime.c#L71-L91):
int timer_gettime(timer_t timerid, FAR struct itimerspec *value)
{
  FAR struct posix_timer_s *timer = (FAR struct posix_timer_s *)timerid;
  sclock_t ticks;

  if (!timer || !value)
    {
      set_errno(EINVAL);
      return ERROR;
    }

  /* Get the number of ticks before the underlying watchdog expires */

  ticks = wd_gettime(&timer->pt_wdog);

  /* Convert that to a struct timespec and return it */

  clock_ticks2time(ticks, &value->it_value);
  clock_ticks2time(timer->pt_delay, &value->it_interval);
  return OK;
}

Hmm maybe I do not understand this?
struct posix_timer_s
{
  FAR struct posix_timer_s *flink;

  uint8_t         pt_flags;        /* See PT_FLAGS_* definitions */
  uint8_t         pt_crefs;        /* Reference count */
  pid_t           pt_owner;        /* Creator of timer */
  int             pt_delay;        /* If non-zero, used to reset repetitive timers */
  int             pt_last;         /* Last value used to set watchdog */
  WDOG_ID         pt_wdog;         /* The watchdog that provides the timing */
  struct sigevent pt_event;        /* Notification information */
};

But this is my interpretation based on the comments.
pt_last should be == pt_delay if the timer was repeating. If the timer is not repeating the pt_delay will be 0 and only pt_last will be valid until the timout.

Hmm maybe I do not understand this?
struct posix_timer_s
{
  FAR struct posix_timer_s *flink;

  uint8_t         pt_flags;        /* See PT_FLAGS_* definitions */
  uint8_t         pt_crefs;        /* Reference count */
  pid_t           pt_owner;        /* Creator of timer */
  int             pt_delay;        /* If non-zero, used to reset repetitive timers */
  int             pt_last;         /* Last value used to set watchdog */
  WDOG_ID         pt_wdog;         /* The watchdog that provides the timing */
  struct sigevent pt_event;        /* Notification information */
};

But this is my interpretation based on the comments.
pt_last should be == pt_delay if the timer was repeating. If the timer is not repeating the pt_delay will be 0 and only pt_last will be valid until the timout.

Hmm maybe I do not understand this?
struct posix_timer_s
{
  FAR struct posix_timer_s *flink;

  uint8_t         pt_flags;        /* See PT_FLAGS_* definitions */
  uint8_t         pt_crefs;        /* Reference count */
  pid_t           pt_owner;        /* Creator of timer */
  int             pt_delay;        /* If non-zero, used to reset repetitive timers */
  int             pt_last;         /* Last value used to set watchdog */
  WDOG_ID         pt_wdog;         /* The watchdog that provides the timing */
  struct sigevent pt_event;        /* Notification information */
};

But this is my interpretation based on the comments.
pt_last should be == pt_delay if the timer was repeating. If the timer is not repeating the pt_delay will be 0 and only pt_last will be valid until the timout.

Yes, but pt_last and pt_delay are not equal at the first expired if timer was repeating, so we should select pt_delay as old_value->it_interval. And if the timer is not repeating, the old_value->it_interval must be 0,

Hmm maybe I do not understand this?
struct posix_timer_s
{
  FAR struct posix_timer_s *flink;

  uint8_t         pt_flags;        /* See PT_FLAGS_* definitions */
  uint8_t         pt_crefs;        /* Reference count */
  pid_t           pt_owner;        /* Creator of timer */
  int             pt_delay;        /* If non-zero, used to reset repetitive timers */
  int             pt_last;         /* Last value used to set watchdog */
  WDOG_ID         pt_wdog;         /* The watchdog that provides the timing */
  struct sigevent pt_event;        /* Notification information */
};

But this is my interpretation based on the comments.
pt_last should be == pt_delay if the timer was repeating. If the timer is not repeating the pt_delay will be 0 and only pt_last will be valid until the timout.

Yes, but pt_last and pt_delay are not equal at the first expired if timer was repeating, so we should select pt_delay as old_value->it_interval. And if the timer is not repeating, the old_value->it_interval must be 0,


feat: Deref coercion

The mechanism is similar to Rust's, but the scope is a lot smaller: trivially allow access to a member of T holding a type U.
https://doc.rust-lang.org/book/ch15-02-deref.html#treating-a-type-like-a-reference-by-implementing-the-deref-trait
Should go in 11. Advanced Features
Demonstration here: compile and read the casm to see what's happening
#[derive(Copy, Drop)]
struct MyStruct {
    x: felt252,
    y: felt252,
    z: felt252,
}

impl DerefType1ToType2 of Deref<Box<MyStruct>> {
    type Target = MyStruct;
    fn deref(self: Box<MyStruct>) -> Self::Target {
        let unboxed = self.unbox();
        Self::Target { x: unboxed.x, y: unboxed.y, z: unboxed.z }
    }
}

fn main() {
    let my_struct = MyStruct { x: 1, y: 2, z: 3 };
    let boxed = BoxTrait::new(my_struct);

    foo(boxed);
    bar(boxed);
}

#[inline(never)]
fn foo(boxed: Box<MyStruct>) -> felt252 {
    boxed.x
}

#[inline(never)]
fn bar(boxed: Box<MyStruct>) -> felt252 {
    boxed.unbox().x
}

Hi @enitrat I would like to work on this

@enitrat , Could you look at the PR and suggest possible changes as the build is failing.


Wrong PGDATA directory for official TimescaleDB image: allow customizing PGDATA?

I'm trying to get the "stock" TimescaleDB docker image working with cnpg. This is the Cluster definition I'm using:
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: my-timeseries-db
spec:
  instances: 1
  # cnpg needs to know the pg major version. because the timescale image tags don't follow an auto-detectable pattern,
  # we specify the major pg version manually using an image catalog.
  # See https://github.com/cloudnative-pg/cloudnative-pg/blob/main/docs/src/container_images.md#image-tag-requirements
  imageCatalogRef:
    apiGroup: postgresql.cnpg.io
    kind: ClusterImageCatalog
    name: timescale
    major: 16
  storage:
    size: 10Gi
  bootstrap:
    initdb:
      database: my_timeseries
      owner: my_timeseries
      secret:
        name: my-timeseries-credentials

with this cluster catalog:
apiVersion: postgresql.cnpg.io/v1
kind: ClusterImageCatalog
metadata:
  name: timescale
spec:
  images:
    - major: 16
      image: timescale/timescaledb-ha:pg16

Unfortunately, the initdb container runs into the following issue:
{"level":"error","ts":"2024-10-07T09:10:41Z","msg":"Error while checking for an existing PGData","logging_pod":"wams-timeseries-db-1-initdb","error":"stat /var/lib/postgresql/data/pgdata: permission denied","stacktrace":"github.com/cloudnative-pg/cloudnative-pg/pkg/management/log.(*logger).Error\n\tpkg/management/log/log.go:125\ngithub.com/cloudnative-pg/cloudnative-pg/pkg/management/log.Error\n\tpkg/management/log/log.go:163\ngithub.com/cloudnative-pg/cloudnative-pg/pkg/management/postgres.InitInfo.CheckTargetDataDirectory\n\tpkg/management/postgres/initdb.go:141\ngithub.com/cloudnative-pg/cloudnative-pg/internal/cmd/manager/instance/initdb.initSubCommand\n\tinternal/cmd/manager/instance/initdb/cmd.go:151\ngithub.com/cloudnative-pg/cloudnative-pg/internal/cmd/manager/instance/initdb.NewCmd.func2\n\tinternal/cmd/manager/instance/initdb/cmd.go:108\ngithub.com/spf13/cobra.(*Command).execute\n\tpkg/mod/github.com/spf13/cobra@v1.8.1/command.go:985\ngithub.com/spf13/cobra.(*Command).ExecuteC\n\tpkg/mod/github.com/spf13/cobra@v1.8.1/command.go:1117\ngithub.com/spf13/cobra.(*Command).Execute\n\tpkg/mod/github.com/spf13/cobra@v1.8.1/command.go:1041\nmain.main\n\tcmd/manager/main.go:66\nruntime.main\n\t/opt/hostedtoolcache/go/1.22.6/x64/src/runtime/proc.go:271"}
Error: while verifying is PGDATA exists: stat /var/lib/postgresql/data/pgdata: permission denied

My suspicion is that this is due to timescaledb diverting from the defaults quite a bit, e.g. customizing PGDATA: https://github.com/timescale/timescaledb-docker-ha/blob/v1.9.5/Dockerfile#L432
Sadly this cannot be worked around by setting the PGDATA environment variable manually, because this...
  env:
  - name: PGDATA
    value: /home/postgres/pgdata/data

results in this error:
admission webhook "vcluster.cnpg.io" denied the request: Cluster.cluster.cnpg.io "wams-timeseries-db" is invalid: spec.postgresql.env[0].name: Invalid value: "PGDATA": the usage of this environment variable is reserved for the operator

Do you think it makes sense working towards getting this image to work? Or are non-cnpg-images strictly out of scope and I'd need to always use custom images like https://github.com/clevyr/docker-cloudnativepg-timescale ?

@Felk The problem here is likely not with customizing the PGDATA directory but with permissions. The Official Timescale uses non-default UID and GID.
There are a bunch of small things you need to to to get the official Timescale image to work. I would recommend trying the cluster chart with type: timescaledb which will configure everything.
Alternatively override the UID and GID to 1000.

The cluster chart has a test suite, which you can look at for more advanced configuration.
Example:
https://github.com/cloudnative-pg/charts/blob/main/charts/cluster/test/timescale-minio-backup-restore/07-recovery_backup_pitr_cluster.yaml

That looks very promising, thank you!
I've tried using the Cluster helm chart using your linked examples plus the chart's Values.yaml for reference. I just can't quite get it to work, as it keeps telling me:

Helm install failed for release timeseries-db/cloudnative-postgres with chart<EMAIL_ADDRESS>execution error at (cluster/templates/cluster.yaml:16:16): You need to provide your own cluster.imageName as an official timescaledb image doesn't exist yet.

I've tried the following yaml:
apiVersion: helm.toolkit.fluxcd.io/v2
kind: HelmRelease
metadata:
  name: timeseries-db-hr
spec:
  releaseName: cloudnative-postgres
  chart:
    spec:
      chart: cluster
      sourceRef:
        kind: HelmRepository
        name: cnpg
        namespace: flux-system
      version: "0.0.11"
  interval: 1h0m0s
  values:
    type: timescaledb
    mode: standalone
#Error is the same, whether these versions are specified or not
#    version:
#      postgresql: "15.7"
#      timescaledb: "2.15"
    cluster:
      instances: 1
      storage:
        size: 100Gi
#Error is the same, even if I explicitly reference the TimescaleDB docker image 'timescale/timescaledb-h2:pg15' via catalog
#      imageCatalogRef:
#        apiGroup: postgresql.cnpg.io
#        kind: ClusterImageCatalog
#        name: timescale
#        major: 15
    backups:
      enabled: false

If I try to specify cluster.imageName directly, I get this error:

spec.imageName: Invalid value: "timescale/timescaledb-ha:pg15": invalid version tag

Given you linked actual test suite configs, I surely must have misunderstood some core concept and am doing something wrong, apologies for that. It would be really appreciated if you could point me in the right direction.

I think this is on main, but haven't been released yet. You can try using the chart from git or you can give me some time to release all the changes. I'll try to do that today.

No problem, thanks for taking the time! In other news, setting the GID and UID seems to have worked :) Though maybe I'll encounter more issues down the line due to "bunch of small things you need to get for the official Timescale image to work" you mentioned that I'm ignoring right now

I think the most important bit is that we pre-install the extension by default in the app database in the initdb step. You can achieve that by setting the following in your Cluster:
bootstrap:
  initdb:
    postInitApplicationSQL:
      - CREATE EXTENSION IF NOT EXISTS timescaledb;


[release/1.4] vendor: update cri plugin with 3b02bec1603179debe2cde54509b2bfc45fc27d3

Diff: https://github.com/containerd/cri/compare/1360416eca4fef15c763444914e60fe1eaedbc3d...3b02bec1603179debe2cde54509b2bfc45fc27d3
Signed-off-by: Wei Fu<EMAIL_ADDRESS>

LGTM


Docusaurus 3.2.0 - Breaking Changes

In the latest Docusaurus upgrade, 3.2.0, a few things broke the plugin.

addLeadingSlash and addTrailingSlash were removed from @docusaurus/utils due to being a duplicate in @docusaurus/utils-common. PR can be found here

PluginOptions passing undefined values for showLastUpdateTime and showLastUpdateAuthor in the options property.

src/index.ts(87,13): error TS2739: Type 'PluginOptions' is missing the following properties from type 'Omit<MetadataOptions, "numberPrefixParser" | "editCurrentVersion" | "editLocalizedFiles" | "breadcrumbs">': showLastUpdateTime, showLastUpdateAuthor

cypress test fails due to sidebar collapsible menus now having the role of button instead of link. Items in the menu are still considered links however.

(Results)
│ Tests:        4
│ Passing:      3
│ Failing:      1
│ Pending:      0
│ Skipped:      0
│ Screenshots:  1
│ Video:        false
│ Duration:     6 seconds
│ Spec Ran:     test.spec.ts
AssertionError: Timed out retrying after 4000ms: Unable to find an accessible element with the role "link" and name `/^pet$/i`

Actions I took to get this working on Docusaurus 3.2.0:

addLeadingSlash and addTrailingSlash imported now from @docusaurus/utils-common
Omit showLastUpdateAuthor & showLastUpdateTime from PluginOptions
Edit to cypress test tonav() function due to certain links having the role of button instead of link.
Updated plugin to use Docusaurus 3.2.0 packages
Updated demo to run on docusaurus 3.2.0

Thanks for filing such a thorough issue :) Would you mind opening a pull request with these fixes?


How to get call history back?

How to get call history back?

You may utilize messages in output from task_manager which contains the history and save it to retrieve it.
https://github.com/bolna-ai/bolna/blob/68beb27de1355894a166e13fd0b17b6cef913e6a/bolna/agent_manager/task_manager.py#L2044-L2049


[semver:major] Rewrite to odk spec.

Marking as major due to some of the script changes.

It doesnt appear integration tests have ran

@KyleTryon There were no integration tests to speak of initially - the tests that did exist where just Bets tests to check that things were inlined correctly? Regardless, I'm not too familiar with Cloudforge myself so might take some time to get "real" tests going.


Replace ToDo with link to Writing documentation in Inline Documentation tutorial

I found a dangling todo in the inline documentation section which should (I guess?) link to the writing documentation section. Ping me if it should have linked to the topical guide on writing documentation instead (they both have the same name).
You might want to check if the link works correctly. I have some ruby build issues locally and already spend too much time on it for a single line docs fix on a Sunday :) So i could not check it.
Anyways, thanks for building this resource!

@danielballan can comment, I think this might have been referring to https://github.com/scientific-python/cookie/issues/157?

I thought so too, at first, but on closer look I think we left this here mid-refactor when the Guide on docs had not yet been put into place.
I think this can be merged.


Fixed some missing language tags

And enabled an automatic check that complains if there were any missing language tags requested at the end of each unit test.  This caused 39 new test failures, of which apparently only a couple were actually honest-to-goodness bugs.  Still good to catch a few.
I thought that we had been doing this all along, but it was only doing it for integration tests.  Now it does it for all tests. It helped, for example, reveal a bug in an error message associated with validating the classification string.  It was attempting to translate the rank the user typed in, regardless of whether it was a recognized rank.  So, this is a good way to catch auto-generated language tags which language_test.rb doesn't find (by searching for ":blah.l" throughout the code).

Coverage decreased (-0.01%) to 91.397% when pulling 2f2d80d588d492d7b30606e2353602d800c34dd5 on pellaea:search-help into def26ae83bbcc1b8a79338d32c24a939bc570b44 on MushroomObserver:master.


when trying to download any album they all turn into text.url files

Ripme version:1.7.13
Java version:
Operating system:
Exact URL you were trying to rip when the problem occurred:
Please include any additional information about how to reproduce the problem:

Expected Behavior
Detail the expected behavior here.
Actual Behavior
Detail the actual (incorrect) behavior here. You can post log snippets or attach log files to your issue report.

@cyian-1756 no response from author.  Recommend closing.


Newline after CMSGroups classAd

My bad I did not catch this bug in testbed, the problem is that my test workflows don't have a group and so they never matched that regexp. Pushing it to the newest production agents right now.

tested in production :stuck_out_tongue:

Thank you very much Alan, since this is only for agent side change, merging it.

Thanks Seangchan!


(frontend,api): add landing page basic statistics

pull requests per day
runs per day (failed,completed,aborted, run type)
efficiency (Total time / # runners / run time = efficiency %)


Pulse Oximetry issue for FHIR API

For pulse oximetry the response generated with my user data is the same as the response generated by the inforno g(10) reference server. But the error is shown for only my data.
Please check the attached screen shots and provide with a solution.

https://github.com/onc-healthit/onc-certification-g10-test-kit/wiki/FAQ#q-how-does-inferno-test-mustsupport-flag-on-an-element
If you look at the other resources returned by the reference server, at least one of them does contain a L/min component.

We do not support "Inhaled oxygen flow rate" in our EHR. What can be done about this?
We support "Inhaled oxygen concentration" and "SPO2"

Ok. Thank you


Feature request:  "string" + XXX_strerror(...) should be detected as an error

Hello!
Here's an idea for a new rule, that I'm 100% positive will find real bugs.
We had some networking code that was returning a  "0" for an error message.  It took while to track down.  The source the problem was the + instead of a .
$errmsg = "networking error: " + socket_strerror(...);

This type of bug is especially bad since it's often very difficult or impossible to create a test case for, so you find this bug in the worst possible way:  when you are having a real problem, and you get "0" for a log message :-(
There are a large number of XXX_strerror() functions that always return a string, not a number.

http://php.net/manual/en/function.socket-strerror.php
http://php.net/manual/en/function.posix-strerror.php
http://php.net/manual/en/function.curl-strerror.php
http://php.net/manual/en/function.pcntl-strerror.php
http://php.net/manual/en/function.curl-multi-strerror.php
http://php.net/manual/en/function.radius-strerror.php
http://php.net/manual/en/function.mqseries-strerror.php

error_get_last is different but might also make sense to include.  http://php.net/manual/en/function.error-get-last.php
Doing grep in our code base found a mix of the correct use of . and incorrect use of +.  I have a hard time seeing why you would ever use + with these functions, so it seems like it might be a good rule.
I would contribute a PR but as you can probably tell, I'm not a regular PHP user!
Thanks for your time.
Regards,
nickg

@client9 thanks for your idea.
I see your problem but creating a rule for covering such a narrow case would be an overkill.
Or better said not many will ever benefit from it.
The real problem here is that two strings are getting concatenated with the wrong operator.
My IDE, PhpStorm, for example, shows me this clearly.

Also, I can use PhpStorm's built-in code inspection over my whole project, which would uncover me such mistakes.
I don't say PHPMD is the wrong tool for that or that we shouldn't create a rule for the problem at hand, but there are existing tools, which are in the better position as PHPMD is.
Because an IDE does a full index of your code and has also an internal index of the PHP function signatures.

Hi guys,
thanks for your answer.
Ideally what I'm looking for is "PhpStorm analysis", but done via command line so it can go into a automated CI system.
That's probably NOT the goal of PHPMD, so it's ok to close this if you wish.
thanks for listening, and thanks for educating me!
regards,
n


Add config to use custom phpunit.humbug.xml (e.g for symfony project)

{
    "source": {
        "directories": [
            "src"
        ],
        "excludes": [
            "vendor"
        ]
    },
    "chdir": "app",
    "configure": "app/humbug/phpunit.humbug.xml",
    "timeout": 30,
    "logs": {
        "text": "humbuglog.txt",
        "json": "humbuglog.json"
    }
}

Also needs to be documented. It's relatively simple, so I might look at it later over the weekend.

@tabbabi could you add the missing doc? After that I think it will be ready to be merged :)


Feature/add format tools in docker container

Add cmake-format in the docker dev container
Add clang-format in the docker dev container

Codecov Report

Merging #8 (80dbf0c) into main (cb6413a) will decrease coverage by 3.70%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   77.77%   74.07%   -3.71%
==========================================
  Files           3        3
  Lines          36       27       -9
  Branches       19        0      -19
==========================================
- Hits           28       20       -8
  Misses          7        7
+ Partials        1        0       -1

Flag
Coverage Δ

Linux
?

Windows
74.07% <ø> (-5.93%)
:arrow_down:

macOS
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/main.cpp
22.22% <0.00%> (-20.64%)
:arrow_down:

test/tests.cpp
100.00% <0.00%> (ø)

test/constexpr_tests.cpp
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Thank you @ddalcino for the review! I will need more approvals for this. @lefticus, @aminya please if you find some time for this, so I can continue with the "root user security" issue in the docker container. Thank you.


types"' has no exported member 'html2canvas'

Bug reports:
I'm using the library in an Angular project with Typescript.
I get "/node_modules/html2canvas/dist/types"' has no exported member 'html2canvas'.ts(2305)"
on my import { html2canvas } from '../../../../../node_modules/html2canvas';

Hi !
Same issue ! Did you find a solution ?

@maxcl10 @aws-ynov
This seemed to work for me :)
import * as html2canvas from 'html2canvas';

import { html2canvas } from 'html2canvas';  --> KO
import * as html2canvas from 'html2canvas'; --> OK


True list of observed dependencies rather than list of a priori and approximate dependencies

Hi,
The IBM DBB dependency detection mode used to date is not satisfactory:

it is done a priori by analysis of the source code
it does not support conditional compilation (at least for COBOL)
it does not know how to analyze cascading inclusions (at least in COBOL), in particular if the scanner does not have access to COPYBOOKS libraries

However, it is essential to know the real dependencies to be able to trigger constructions on impacts: in COBOL, trigger the reconstruction of programs following an (impacting) modification of a COPYBOOK.
To have a real list of dependencies, we have to analyze a posteriori the result of the construction or monitor calls to external elements (calls to COPYBOOKs when compiling a COBOL).
In COBOL, to monitor / capture events on the inclusion of COPYBOOKs it is possible to use exits, by means of the LIBEXIT compilation option.
https://www.ibm.com/docs/en/cobol-zos/6.4?topic=option-processing-libexit
The LIBEXIT exit could create a file of dependencies to then integrate into the IBM DBB database to drive next constructions of the impact type.
The same should be done for other languages, for pre-compilers, and for linkedit.
For information, we are currently replacing the CA Broadcom Endevor product with IBM DBB (for the construction part).
Endevor has a mechanism to capture real dependencies by monitoring I/O at file level, (MONITOR option on ddname).
To see if IBM DBB could have an equivalent mechanism which would make the dependency management solution universal.
So to see if it is the groovy type zAppbuild scripts that must manage the list of true dependencies, or if it is the IBM DBB API that must manage it.
My opinion is that it is rather the IBM DBB API which should provide this service, but failing that it can be implemented in the groovy zAppBuild scripts.

Hi Denis,
thanks for bringing up this topic. Trying to get back to the different topics you are bringing up here.
Recursive dependency resolution
DBB is capable to perform a recursive dependency and impact analysis. The MortgageApp sample actually requires a recursive dependency resolution - see epsmtcom.cpy which includes two nested copybooks.
During dependency resolution DBB will find the dependencies in the build workspace and resolve them to upload them to the target build datasets.
The same is for impact analysis and impact resolution. This is just the other direction - from the nested copybook to the programs or even link cards which are on the dependency chain.
You need to configure the search paths (search rules) correctly, because this is what DBB is using to actually find the matches. With the new APIs (see #201), which got added with 1.1.2 and 1.1.3, they also support wildcards.
Additionally, there is also a LinkEdit scanner to scan and capture the dependency on the object level (the included CSECTs) to understand the dependencies pulled in from the linkage editor.
** conditional compilation **
We need to understand that DBB is not a compiler :-) So, the conditional compilation is depending on the compilation options and is evaluated during compilation time, isn't? DBB is making sure, that all dependencies are present in the build datasets and then invoke the compiler, so it cannot "predict" the compilation options in place.
With an after-the-build dependency capturing would we not loose some of the dependency information? I assume the information to be present in the sysprint.
Unfortunately, the IBM sample for conditional compilation is pretty simple. I would be interested to understand more about how you use conditional compilation.

Hi @dennis-behm
Thank you for your answers.
For the analysis of COPYBOOKs in a framework, there may be a problem with the settings at the moment in our DBB configuration: the COPYBOOKs are not stored in the same Repositories as the COBOL programs, and at the compilation stage we uses PDS and not the Git repositories containing the COPYBOOKs.
I will look at this point.
Concerning conditional compilation, here are some test programs that we use in our work to set up IBM DBB with many cases of conditional compilation and the use of COPYBOOK in cascade based on conditional compilation.
CC samples.zip


Rotate -90deg

Hello
Is it possible to rotate the text?
Thanks.

You should be able to just use CSS Transform.
By default, the label classes on these are .ct-label-above and .ct-label-below depending on whether you've crossed the threshold.
Just try:
.ct-label-above, .ct-label-below {
   transform: rotate(-90deg);
}

thank you. But the origin is not correct.
.ct-label-above, .ct-label-below {
transform: rotate(-10deg);
}
/*use transform-origin ? */

Yeah, looks like they all have a default transform-origin of 0px 0px so rotation is blowing them them around that point.  Unfortunately, to get it to rotate, we'd have to put in a transform-origin on each point so you could rotate, and that would be weird too, because there are needed adjustments for alignment with fonts.  Not undoable, I suppose it would be pretty straight forward to at least provide the existing location of each label as the transform-origin, but then it would be up to the user to figure out they're adjustment needs from a translation standpoint.

Thanks !


Add support for keyboard navigation

First of all, thank you for all the work on the Devtools, they're looking great and really help with day-to-day work with Svelte.
I just miss something when compared to the regular DOM Devtools, keyboard navigation! Currently, it seems that arrows up/down performs page scrolling, if the content overflows. I think it would make more sense for the arrow keys to behave as when navigating the DOM in the Elements view.

Hmm, yes, good idea. I'll see if I can find some time to work on this soon.

@RedHatter major thanks for addressing this 😄 will report back on any bugs I find

@RedHatter major thanks for addressing this 😄 will report back on any bugs I find


Phone number authentication via REST API using firebase

Phone number authentication via REST API.I
I have an app(android/ios) which calls backend api. This backend api will integrate the firebase or using googleapis for otp authentication and then send the otp to the number.
I am using below api at first to get the verification code:
https://identitytoolkit.googleapis.com:443/v1/accounts:sendVerificationCode?key=AIzaSyC0XViRhLH9sF6An3XXVHJHrnFNhIN-VAA"
RequestBody:
{
"phoneNumber": "8879373947",
}
I am getting MISSING_CLIENT_IDENTIFIER
Guide me about what i am doing wrong as there is little to no documentation on how to accomplish phone number authentication via REST API.

You need to attack either a safetyNet token or Recaptcha token from Android for Phone Auth - https://firebase.google.com/docs/auth/android/phone-auth#enable-app-verification. Similarly you need a suitable client identifier on iOS as described in https://firebase.google.com/docs/auth/ios/phone-auth#enable-app-verification.

The client identifier tokens are obtained via the client SDKs, so this is not really supported via rest APIs.


feat: allow shape and geotiff upload

This adds a modal for uploading shape- and geotiff files to the specified GeoServer.

Per default the upload is available for admin users only. If you want to allow upload for other user roles, note the given configuration authorizedRoles and ensure the rest interfaces in GeoServer are accessible for the users. An example rest.properties allowing the upload for users with the ROLE_USER could look like the following:
/rest/workspaces/shogun/datastores/*/file.shp;PUT=ROLE_USER,ROLE_ADMINISTRATOR
/rest/workspaces/shogun/datastores/*/featuretypes;POST=ROLE_USER,ROLE_ADMINISTRATOR
/rest/workspaces/shogun/coveragestores/*/file.geotiff;PUT=ROLE_USER,ROLE_ADMINISTRATOR
/rest/workspaces/shogun/coveragestores/*/coverages;POST=ROLE_USER,ROLE_ADMINISTRATOR
/**;GET=ROLE_ADMINISTRATOR
/**;POST,DELETE,PUT=ROLE_ADMINISTRATOR

Please review @terrestris/devs.

Looks good! Thanks for this

Thanks for the reviews!

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


doc: add npm download count in README

I added the npm download count in our README :)

maybe also add the ci passing badge?


Refactor resource picker

These commits clarify the display logic of the file by eliminating several confusing if / unless resource.present?
And a bug was found where the file would be picked but the name would not show up above "Remove this file", so it was fixed by forming the link even if the resource isn't present, leaving the information that requires a file blank.

Yes that or a decorator (probably over the top). The point of this commit is to fix the display bug and point out the fuzziness of the code
Joe Sak
517 944 5230
On Jan 19, 2012, at 5:42 PM, Philip<EMAIL_ADDRESS>wrote:

Would helpers be useful here?

Reply to this email directly or view it on GitHub:
https://github.com/resolve/refinerycms/pull/1236#issuecomment-3576588

seems like a good thing


Create Edit Page

Should be created an Edit page to edit the CV informations.
TODO

[x] Check if it's logged or not. If not, redirect to login page
[ ] Form with Formik
[x] Load existing data when open the page
[x] Save Changes

#35


Cranelift: shrink the size of SigData further

This could maybe give us some small perf gains due to a smaller working set that better fits in cache.
SigData is defined here: https://cs.github.com/bytecodealliance/wasmtime/blob/348f962d23df0a598ea80629ca6d8e4a158fe153/cranelift/codegen/src/machinst/abi.rs?q=SigData#L601-L627
The two changes we could make to shrink its size are:

We have implementation limits on the number of arguments and returns from a function, so we don't need a fulle Range<u32> to represent the subslice in SigSet::abi_args for this signature's arguments and returns. Instead we could have a u32 start and a u16 length. Doing this for both args and returns would save a total of 4 bytes.

SigData::stack_reg_arg is currently an Option<usize> but, again because of implementation limits on the number of returns, could be Option<u16>. This would save 12 bytes for this field, but I think we might end up actually saving only 8 on the struct because of alignment.

For anyone who picks this up, we can measure the impact this change has via

Building the bench API on main:$ git checkout main
$ cargo build --release -p wasmtime-bench-api
$ cp target/release/libwasmtime_bench_api.so /tmp/main.so

Building the bench API on the feature branch for this patch:$ git checkout my-feature-branch
$ cargo build --release -p wasmtime-bench-api
$ cp target/release/libwasmtime_bench_api.so /tmp/smaller-sig-data.so

Running the sightglass benchmarks:$ cd ~/path/to/sightglass
$ cargo run --release -- benchmark -e /tmp/main.so -e /tmp/smaller-sig-data.so --stop-after compilation -- benchmarks/default.suite

(You can also try passing --measure perf-counters to see the effects on cache accesses/misses if you're on linux.)

It might also be possible to entirely remove the end-points of the argument and return ranges. I haven't checked carefully but I think the end-point in one SigData is always equal to the start-point in the next SigData, and the last SigData's end-point is the length of the corresponding backing array.
If we move all the SigData methods that take a SigSet to be on SigSet instead and take a Sig index—which I think is a good idea for API design reasons anyway—then it's easy to access the Sig+1 index.
Are there any constraints on where the stack-ret arg can appear in the list? For example, if it's always the first argument, we could replace the Option<usize> with a bool. If there are no constraints, we could still cut the field's size in half by getting rid of the Option, if we can pick a reserved value like u16::MAX to indicate that there's no stack-ret arg.
Also, do the sized_stack_*_space fields need to be i64? For one thing, what does it mean to reserve a negative amount of stack space? For another, surely 32 bits is enough for the size of a stack frame...?
All together, I think we can shrink SigData from 7 u64 to 5 u32, taking padding and alignment into account, for a savings of 64%.

It might also be possible to entirely remove the end-points of the argument and return ranges. I haven't checked carefully but I think the end-point in one SigData is always equal to the start-point in the next SigData, and the last SigData's end-point is the length of the corresponding backing array.
If we move all the SigData methods that take a SigSet to be on SigSet instead and take a Sig index—which I think is a good idea for API design reasons anyway—then it's easy to access the Sig+1 index.

Yeah, this is possible but a larger change and probably not a good first issue anymore.

Are there any constraints on where the stack-ret arg can appear in the list? For example, if it's always the first argument, we could replace the Option<usize> with a bool. If there are no constraints, we could still cut the field's size in half by getting rid of the Option, if we can pick a reserved value like u16::MAX to indicate that there's no stack-ret arg.

There aren't, but in practice I think it is always the first. Maybe we could just start enforcing that? Again, probably not in the realm of a good first issue though.

Also, do the sized_stack_*_space fields need to be i64? For one thing, what does it mean to reserve a negative amount of stack space? For another, surely 32 bits is enough for the size of a stack frame...?

Yeah I was kinda wondering that too. Haven't investigated it, so didn't want to include it in a good first issue.

I'll try out changing the SigData methods as suggested with Sigset

Let us know if you have any questions! I'm excited that you're digging into this.

May I ask what's still missing for this issue?

Good question! I don't remember for sure.
I think the only remaining possibility for shrinking this struct further is the Option<u16> storing stack_ret_arg, which is four bytes with one byte of padding in the middle. I think the isa::CallConv type is one byte, so with all the other fields being four bytes, the call_conv field has three bytes of padding. Therefore if we can pack the calling convention in with the return argument index, we can save four bytes.
However, I don't think that's a "good first issue". I'm not even sure it's worth doing.
So I'm going to go ahead and close this because I think it's basically done. Thanks for checking!


Imaging 266

This pull request adds the capability of reading numeric data from a TIFF file using the two-byte integer sample format. This format is commonly used in GeoTIFF data sources that provide elevation data, most notably the Shuttle Radar Topography Mission (SRTM) data set.
The change reorganizes the TiffRasterData class into a set of three classes:

TiffRasterData  (abstract base class)
TiffRasterDataFloat  (derives from TiffRasterData, supports floating-point formats)
TiffRasterDataInt (dertives from TiffRasterData, supports integer formats)

Methods in TiffDirectory and TiffParser that were formerly called getFloatingPointRasterData() are now simply getRasterData().
I have supplied new unit tests for the reorganized classes.
Additionally, I added unit tests for an integer-data round-trip test.  This writes temporary files in the 16-bit integer sample style and then reads them back.   Earlier I had discussed adding new sample images to the Commons Imaging test file collection, but I chose to use the round-trip test instead (since it is more flexible and will not clutter the distribution with more special-purpose TIFF sample files).
Thank you for your attention in this matter.
Gary

Submitted now with changes from review.

Submitted now with changes from review.

@gwlucastrig I'm afraid there are a few more items...

Clicking on the "Load more" should display more items :grimacing:

Probably going to need a few more days to address the remaining comments.

Resolved most changes you fixed @gwlucastrig , leaving only the ones I'll need when reviewing again the next time. Thanks!

As far as I know, I've hit everything except the assertFails change for
TiffRasterDataIntTest.java.
I haven't requested a re-review yet, but if you are willing to let that one
go, then I'd say the code should be ready.
Hope you don't have too much "review fatigue" just yet...
On Mon, Aug 9, 2021 at 7:29 PM Bruno P. Kinoshita @.***>
wrote:

Resolved most changes you fixed @gwlucastrig
https://github.com/gwlucastrig , leaving only the ones I'll need when
reviewing again the next time. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kinow/commons-imaging/pull/23#issuecomment-895619202,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEWJDYMAOEKETLLQXDEKM3DT4BQFNANCNFSM5BXIAR6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

Had a quick look and everything looks in order! Found one extra blank lines but don't worry @gwlucastrig , I can fix that when merging. I left a few items open as I'll re-read the comments when reviewing again.
Amazing work, and super quick. Will try to get it reviewed again before the weekend :+1:

@gwlucastrig this PR also has conflicts. Although I can try to rebase/fix conflicts on my side, I may accidentally add/remove something during the rebase, so if you are able to rebase/fix conflicts that'd the best option. Once you push the rebased changes (you might need to git push -f to force the re-write) GitHub UI here should remove the conflicts, showing the PR as ready to be merged.

Withdrawn due to issues related to the incorrect use of git/github.


NYU Handoff

All of these items should be completed before our demo to NYU
Work completed:

[ ] Binder link works
[ ] Dev docs up to date https://github.com/jupyterlab/jupyterlab-commenting/pull/46
[x] User facing docs include user stories and instructions
[x] GitHub issues and milestones for current and future work are up to date

[x] Issue and PR templates.
[x] Appropriate labels exist and are applied to issues/PRs.
[x] Milestone for current and next release exists.
[x] Issues and PRs are targeted to appropriate Milestone.

[ ] Test Suite up to date https://github.com/jupyterlab/jupyterlab-commenting/pull/46

[ ] Setup testing infrastructure for local and CI based tests.
[ ] Documented on how to run test suite locally.

Work completed:

[ ] Binder link works
[ ] Dev docs up to date https://github.com/jupyterlab/jupyterlab-commenting/pull/46
[x] User facing docs include user stories and instructions
[x] GitHub issues and milestones for current and future work are up to date

[x] Issue and PR templates.
[x] Appropriate labels exist and are applied to issues/PRs.
[x] Milestone for current and next release exists.
[x] Issues and PRs are targeted to appropriate Milestone.

[ ] Test Suite up to date https://github.com/jupyterlab/jupyterlab-commenting/pull/46

[ ] Setup testing infrastructure for local and CI based tests.
[ ] Documented on how to run test suite locally.


res.cork(() => res.upgrade()) in Upgrade handler with ws.send() in Open handler = close connection

Doing as described in title can result in websocket dirty close connection, to prevent this can do setTimeout(() => ws.send('msg'), 0) if need to send message in Open handler after async Upgrade. I think calling res.upgrade() while corked calls the Open handler still corked and ws.send inside the upgrade cork can cause some tcp or protocol error. Removing res.cork() fixes connection error but displays warnings that cork is missing

With res.cork(() => res.upgrade()) and ws.send in open:

No cork res.upgrade() and ws.send in open:

Do you have a full reproducer?

I am making progress narrowing down what is causing issue, a simple example does not cause issue, trying to find what exactly does

Here is example it causes issue like 50% of the time, look for immediate socket close after open
import { App } from 'uWebSockets.js';

const port = 80;

App()

.get('/', r => r.end(`<script>
   ws = new WebSocket('ws'+location.protocol.slice(4,-1)+'://'+location.host+'/ws');
   ws.onopen = () => console.log('open');
   ws.onmessage = e => console.log('message', e.data);
   ws.onclose = () => console.log('close');
</script>`))

.ws('/ws', {
   upgrade: async (res, req, context) => {
      const wsHeaders = [ req.getHeader('sec-websocket-key'), req.getHeader('sec-websocket-protocol'), req.getHeader('sec-websocket-extensions') ];
      res.onAborted(() => res.aborted = true);
      await new Promise(r => setTimeout(r, 100));
      res.cork(() => res.upgrade({}, ...wsHeaders, context));
   },
   open: ws => console.log('open') || ws.send('w'.repeat(56)),
   message: ws => console.log('message'),
   close: ws => console.log('close'),
})

.listen(port, s => s && console.log(`http://localhost:${port}`));


Pls add arabic

Can you add the arabic language?i can translate the whole game(cuz my language is arabic so...)

Hi, Arabic is one of the languages PewPew Live will probably not support for a while.
The language has many complex curves and requires us to rework the whole text rendering system to allow ligatures.
At some point we might implement the support, but currently, the game has other higher priority features to add.
Sorry for the inconvenience.

Thx for replying
BUT if you need a help about arabic things like translationjust talk to me bro

Sure, we'll be sure to ask you for help when we eventually decide to implement Arabic.
For now, I'm closing this issue. We will reopen once help is needed.


Split Array Largest Sum

Description
Given an integer array nums and an integer k, split nums into k non-empty subarrays such that the largest sum of any subarray is minimized.
Return the minimized largest sum of the split.
A subarray is a contiguous part of the array.
DSA Problem
https://leetcode.com/problems/split-array-largest-sum/description/

Hey @Dhandeep10  interested to work on this issue if possible can you assign it to me ?

I have already completed this one in a PR

@Dhandeep10 I've complete this but i'm not able to push to the branch.


Godot 4 RC6 Animated textures crash the game and the editor.

Godot version
4.0 RC6
System information
Windows 10
Issue description
I'm working on porting a medium size game from 3.5 to 4.0. In there, I have a menu with a small 32x32 animated icon that uses an animated texture with 6 frames.
Whenever I close the menu with that icon showing, the game crashes, if I hide the icon it works perfectly.
It seems to have to do with a shader I use to change the icon color, however I use that same shader in many places and this is the only instance of the problem.
Forgot to mention, I have another animated texture in a minigame and when the minigame ends, the game also crashes, said texture is part of a progress bar that uses a red tint, but it also crashes when resetting the tint.
IMPORTANT:
Turns out even trying to close a scene that has the animated textures on screen crashes the whole editor.
Steps to reproduce
Use queue_free() on something that has an animated texture with a shader/part of a progress bar showing on screen.
Minimal reproduction project
N/A

@strellydev Please upload a minimal reproduction project to make this easier to troubleshoot.

AnimatedTexture was supposed to be removed in 4.0, but for some reason that didn't happen. Maybe later we will deprecate it.

Wait, really? And there won't be any substitute?

I think that we will not remove AnimatedTexture, it will just remain deprecated. In this case, the bug will be fixed over time. But due to the AnimatedTexture status, you should consider alternatives:

Sprite2D, Sprite3D or TextureRect. You can use AnimationPlayer or Tween for animation.
AnimatedSprite2D or AnimatedSprite3D.
TileMap supports animation.
CanvasItem.draw_animation_slice() (and CanvasItem.draw_end_animation()). It can be useful for custom controls.
Animation via shader (you mentioned that you are using a shader).

I've been trying to create a small project that reproduces the bug with no success, so I guess it must be something else causing it, sorry. I'll close this.


ci: use cargo-tarpaulin for coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@b29e8fa). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage        ?   84.66%
=======================================
  Files           ?      148
  Lines           ?     7146
  Branches        ?        0
=======================================
  Hits            ?     6050
  Misses          ?     1096
  Partials        ?        0

Continue to review full report at Codecov.

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

sample report


Spring Cloud Config (Server) Monitor does not send events to clients via RabbitMQ

Consider the following usecase: https://stackoverflow.com/questions/53426608/spring-cloud-config-server-monitor-does-not-send-events-to-clients-via-rabbitm
So my configurationserver fetches the properties for the configurationclient via a Git repository (namely https://github.com/dnijssen/configuration), when a change is made I'll trigger the /monitor endpoint on my configurationserver manually (just for testing). Which is supposed to send an event via RabbitMQ.
I've noticed this issue does not occur when using Spring Boot 2.0.6.RELEASE and Spring Cloud Dependencies Finchley.SR2. However since the combination of Spring Boot 2.1.0.RELEASE and Finchley.SR2 are not possible, I tried upgrading to Greenwich.M3 which does not throw exceptions anymore, however its not broadcasting the refresh events either...
Seems like a regression thing to me. Maybe related, maybe not, on startup with 2.1.0 RELEASE / Greenwich.M3 I'll also receive the following warnings:
2018-11-22 15:07:19.353  WARN 2980 --- [           main] o.s.boot.actuate.endpoint.EndpointId     : Endpoint ID 'bus-env' contains invalid characters, please migrate to a valid format.
2018-11-22 15:07:20.782  WARN 2980 --- [           main] o.s.boot.actuate.endpoint.EndpointId     : Endpoint ID 'bus-refresh' contains invalid characters, please migrate to a valid format.

Fixed in http://github.com/spring-cloud/spring-cloud-bus/issues/139


Constructor lambda tear-off not typed correctly

I made a class:
class Client {
  final String id;
  const Client({required this.id});

  factory Client.fromJSON(Map<String, dynamic> json) {
    return Client(id: json["cid"] as String);
  }
}

and when I tried to do
dynamic someJsonData;
List<dynamic>.from(someJsonData).map((item) => Client.fromJSON(item)).toList();

I got no errors, but there was a suggestion: Don't create a lambda when a tear-off will do.
However, when I tried to use a lambda tear-off:
List<dynamic>.from(someJsonData).map(Client.fromJSON).toList();

the application failed to compile:
lib/services/server.dart:414:42: Error: The argument type 'Client Function(Map<String, dynamic>)' can't be assigned to the parameter type 'dynamic Function(dynamic)'.
 - 'Client' is from 'package:instaboard/services/server.dart' ('lib/services/server.dart').
 - 'Map' is from 'dart:core'.
    List<dynamic>.from(someJsonData).map(Client.fromJSON).toList();
                                         ^
Try again after fixing the above error(s).

It seems to me that there's a bug in the type checker for constructor lambda tear-offs.

Flutter doctor -v
```
$ flutter doctor -v                                                                                7.91 s
[✓] Flutter (Channel stable, 3.0.4, on macOS 12.4 21F79 darwin-arm, locale en-US)
    • Flutter version 3.0.4 at /Users/david/prog/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision 85684f9300 (12 days ago), 2022-06-30 13:22:47 -0700
    • Engine revision 6ba2af10bb
    • Dart version 2.17.5
    • DevTools version 2.12.2
[✓] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at /Users/david/Library/Android/sdk
• Platform android-32, build-tools 32.1.0-rc1
• ANDROID_HOME = /Users/david/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• CocoaPods version 1.11.3
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2021.1)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
[✓] IntelliJ IDEA Ultimate Edition (version 2021.3.2)
• IntelliJ at /Applications/IntelliJ IDEA.app
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
[✓] VS Code (version 1.67.2)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.42.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-arm64   • macOS 12.4 21F79 darwin-arm
• Chrome (web)    • chrome • web-javascript • Google Chrome 103.0.5060.114
[✓] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
</details>

Hello @theniceboy. I think this issue is more suited to be raised in https://github.com/dart-lang/sdk. This is an issue in the analyzer and not in Flutter itself. Closing this issue for this reason.


Uncaught Error: Cannot call function 'getPosition' on missing remote object 3Error: Cannot call ...

[Enter steps to reproduce:]

Open Atom in a folder

Atom: 1.25.0 x64
Electron: 1.7.11
OS: Windows 10 Home
Thrown From: Atom Core
Stack Trace
Uncaught Error: Cannot call function 'getPosition' on missing remote object 3
Error: Cannot call function 'getPosition' on missing remote object 3
at throwRPCError (C:\Users\edtho\AppData\Local\atom\app-1.25.0\resources\electron.asar\browser\rpc-server.js:143:17)
at EventEmitter. (C:\Users\edtho\AppData\Local\atom\app-1.25.0\resources\electron.asar\browser\rpc-server.js:354:7)
at emitMany (events.js:127:13)
at EventEmitter.emit (events.js:204:7)
at WebContents. (C:\Users\edtho\AppData\Local\atom\app-1.25.0\resources\electron.asar\browser\api\web-contents.js:256:13)
at emitTwo (events.js:106:13)
at WebContents.emit (events.js:194:7)
At C:\Users\edtho\AppData\Local\atom\app-1.25.0\resources\electron.asar\renderer\api\remote.js:234

Error: Cannot call function 'getPosition' on missing remote object 3
Error: Cannot call function 'getPosition' on missing remote object 3
    at throwRPCError (~/AppData/Local/atom/app-1.25.0/resources/electron.asar/browser/rpc-server.js:143:17)
    at EventEmitter.<anonymous> (~/AppData/Local/atom/app-1.25.0/resources/electron.asar/browser/rpc-server.js:354:7)
    at emitMany (events.js:127:13)
    at EventEmitter.emit (events.js:204:7)
    at WebContents.<anonymous> (~/AppData/Local/atom/app-1.25.0/resources/electron.asar/browser/api/web-contents.js:256:13)
    at emitTwo (events.js:106:13)
    at WebContents.emit (events.js:194:7)
    at metaToValue (~/AppData/Local/atom/app-1.25.0/resources/electron.asar/renderer/api/remote.js:234:13)
    at Object.remoteMemberFunction (~/AppData/Local/atom/app-1.25.0/resources/electron.asar/renderer/api/remote.js:118:18)
    at AtomEnvironment.getWindowDimensions (~/AppData/Local/atom/app-1.25.0/resources/app/src/atom-environment.js:691:40)
    at AtomEnvironment.storeWindowDimensions (~/AppData/Local/atom/app-1.25.0/resources/app/src/atom-environment.js:728:40)
    at WindowEventHandler.handleWindowBlur (~/AppData/Local/atom/app-1.25.0/resources/app/src/window-event-handler.js:191:32)

Commands

Non-Core Packages
altgr 0.1.0
ariake-dark-syntax 0.1.3
atom-eclipse-syntax 0.0.5
atom-ternjs 0.18.3
autumn-highlighter-syntax 1.13.0
bleen-syntax 0.3.1
busy-signal 1.4.3
city-lights-syntax 1.1.3
city-lights-ui 1.2.0
emmet 2.4.3
firewatch-syntax 1.1.1
ice-cold-syntax 0.1.0
intentions 1.1.5
language-babel 2.80.1
language-graphql 0.9.0
language-mjml 1.10.0
language-pgsql 0.2.3
linter 2.2.0
linter-mjml 1.2.0
linter-ui-default 1.6.10
lorem 1.3.0
mac-os-ui 0.3.0
minimap-plus 4.29.2
mjml-preview 1.4.0
neon-syntax 0.3.7
prettier-atom 0.41.0
seti-ui 1.8.0
seti-ui-red 0.13.0
slim-dark-ui 0.12.0
styri-syntax 0.1.1

Thanks for the report - this looks like a duplicate of #16630 so you can subscribe there for updates if you'd like.  Also, is the error reproducible for you by just opening a folder in Atom or it just happened to you once?

Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


refactor streaming module

streaming module is the only one that has not been refactored since the early days of this project. re-write it if necessary.
ps: I started to refactor this, but do not know how long will it take.

Closed with PR - https://github.com/serengil/deepface/pull/1110


changed 'perspires' to 'persists' in chapter 1 quiz

Description
Changed 'perspires' to 'persists' in Chapter 1 quiz
Fixes #585
I'm new to open source - please let me know if anything needs to be changed or if my process is wrong. Figured I'd start with something simple. Thanks all!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Remove requirements_test.txt

Application
Currently, the demos require a requirements_test.txt file that calls for coverage to be installed. This is too onerous on the person submitting a demo; they shouldn't need to make a requirements_test.txt as it is unexpected and should not be handled by them.
Proposed Solution
Automatically install coverage for every virtual environment created to run a demo's unit tests.

What if some demo requires special testing, and additional packages?