Add better logging system

As the title says, sushi needs to help users understand what is going on by adding more logs.
This is just a future idea.

@ksawery29 Did you have anything in mind? sushi could be more verbose when configuring, saving & executing functions. Perhaps we can include the function names and arguments passed when sushi is running. We could also add a VERBOSE setting in sushi.conf to turn off/on verbose printing/logging.

That's what I wanted to add but that pytest error slowed me. I want to add something like verbose mode as you said and display everything what it is doing.


Iterate over cursor inside "with server_side_cursors():" block to avoid exiting block before we are done with the cursor.

We call the server_side_cursor helper with:
with server_side_cursors():
    return queryset.iterator()

but this means the server_side_cursor has already exited
We should rather do:
with server_side_cursors():
    for item in queryset:
        yield item

Django's querysets iterator() method first yields once it has the first object, which is what is making me think that this change is worth a try for fixing the odd "returns only one row" CSV export issue.

@hodgestar Ready for review

:+1:


Gripper stuck when grasping object

Type of Issue
Select the type of issue:

[x] Bug report (to report a bug)
[x] Feature request (to request an additional feature)
[ ] Tracker (I am just using this as a tracker)
[ ] Refactor request
[ ] Documentation Ask

Description
When using the gripper goto function (regardless of blocking), the gripper gets stuck when asked to fully close when there is an object between its fingers. As a result, it does not respond to all future commands because it continuously attempts to reach the fully closed position.
Expected Behavior
With blocking off, the gripper should respond to a new command regardless of whether the old one was achieved.

I'm experiencing this issue as well. Specifically, I'm experiencing two issues:

The gripper pauses unnecessarily when going to a new width (ex from fully open to fully closed)
When the gripper tries to grasp an object, if it isn't able to fully able to go to the desired location, it gets stuck - even after I update the desired state to be fully open. It only starts to move after I remove the object blocking the gripper fingers from closing.

Here’s some videos of the issues discussed:
https://user-images.githubusercontent.com/30299984/201435469-aa006e6f-0fdd-457d-9e85-42ada36fbd30.MOV

https://user-images.githubusercontent.com/30299984/201436229-9497d19c-6e17-46a0-bcf7-3f93b7e2ba3f.mov

Wanted to bump this issue :)

Hello sorry for the slow response. I cannot change the behavior of goto for the Franka Hand as we are simply wrapping around franka::Gripper::move.
What I can do is to expose franka::Gripper::grasp, which is designed to avoid this issue when grasping. I've done that in the PR mentioned above and will close the issue since there is nothing I can do more than that.
So basically, the thing with the Franka Hand is that you can either move to a specific width (and get blocked until it errors out if the gripper is unable to reach the width), or close the gripper all the way until the fingers hit something.
Thanks as always for the detailed description of the issue, I hope your fingers are fine :)


No while loop!

Found this while clearing out some files :smile:
Feel free to chop up or ignore. You can probably get rid of the ternaries using ramda ifElse, when and unless.

Coverage decreased (-0.01%) to 99.412% when pulling 63709ba8ecbf63124bb111f3a52f53a5360b9ca9 on ronanyeah:master into b1ec3a23de770c070771d10f6a040398ffb38b55 on richardscarrott:master.

@ronanyeah nice one :)


feat: 200 estimate fee rpc

Implementation of RPC endpoint starknet_estimateFee
Pull Request type
Please add the labels corresponding to the type of changes your PR introduces:

Bugfix
Feature [x]
Code style update (formatting, renaming)
Refactoring (no functional changes, no API changes)
Build-related changes
Documentation content changes
Testing
Other (please describe):

What is the current behavior?
No estimate fee interface
Resolves: #200
What is the new behavior?
Implementation of RPC endpoint starknet_estimateFee
Does this introduce a breaking change?
No

use the labels

Merging anyway, feel free to address comments in a follow up PR if needed


Unable to setup SASL enabled memcached instance on mac

Hi team,
Does anyone of issue, connecting SASL enabled memcache.
Here are the few steps I followed to install memcached with SASL on my mac but when I connect using java client (attached) it’s fails with password verification failed error? Any one can help! Thanks!
Nishants-MacBook-Pro-2:~ nishant_jain$ brew install memcached --enable-sasl-pwdb
Nishants-MacBook-Pro-2:~ nishant_jain$ echo "mech_list: plain" > memcached.conf
Nishants-MacBook-Pro-2:~ nishant_jain$ echo "nishant:pass" > /tmp/memcached-sasl-db
Nishants-MacBook-Pro-2:~ nishant_jain$ export MEMCACHED_SASL_PWDB=/tmp/memcached-sasl-db
Nishants-MacBook-Pro-2:~ nishant_jain$ export SASL_CONF_PATH=pwd/memcached.conf
Nishants-MacBook-Pro-2:~ nishant_jain$ /usr/local/opt/memcached/bin/memcached -S -vv -p 8010
Initialized SASL.
slab class   1: chunk size        96 perslab   10922
slab class   2: chunk size       120 perslab    8738
Memcache terminal dump:
<27 new binary client connection.
<27 Read binary protocol data:
<27    0x80 0x21 0x00 0x05
<27    0x00 0x00 0x00 0x00
<27    0x00 0x00 0x00 0x12
<27    0x00 0x00 0x00 0x11
<27    0x00 0x00 0x00 0x00
<27    0x00 0x00 0x00 0x00
authenticated() in cmd 0x21 is true
mech:  ``PLAIN'' with 13 bytes of data
INFO: User<EMAIL_ADDRESS>failed to authenticate
SASL (severity 2): Password verification failed
sasl result code:  -20
Unknown sasl response:  -20

27 Writing an error: Auth failure.

Intellij Exception:
2017-02-23 16:42:36.510 INFO net.spy.memcached.MemcachedConnection:  Added {QA sa=localhost/<IP_ADDRESS>:11211, #Rops=0, #Wops=0, #iq=0, topRop=null, topWop=null, toWrite=0, interested=0} to connect queue
2017-02-23 16:42:36.607 ERROR net.spy.memcached.protocol.binary.SASLAuthOperationImpl:  Error:  Auth failure.
2017-02-23 16:42:36.608 WARN net.spy.memcached.protocol.binary.BinaryMemcachedNodeImpl:  Discarding partially completed op: SASL auth operation
2017-02-23 16:42:36.609 INFO net.spy.memcached.MemcachedConnection:  Reconnection due to exception handling a memcached operation on {QA sa=localhost/<IP_ADDRESS>:11211, #Rops=0, #Wops=0, #iq=0, topRop=null, topWop=null, toWrite=0, interested=1}. This may be due to an authentication failure.
OperationException: SERVER: Auth failure.
at net.spy.memcached.protocol.BaseOperationImpl.handleError(BaseOperationImpl.java:192)
at net.spy.memcached.protocol.binary.OperationImpl.finishedPayload(OperationImpl.java:204)
at net.spy.memcached.protocol.binary.SASLBaseOperationImpl.finishedPayload(SASLBaseOperationImpl.java:98)
at net.spy.memcached.protocol.binary.OperationImpl.readPayloadFromBuffer(OperationImpl.java:196)
at net.spy.memcached.protocol.binary.OperationImpl.readFromBuffer(OperationImpl.java:139)
at net.spy.memcached.MemcachedConnection.readBufferAndLogMetrics(MemcachedConnection.java:861)
at net.spy.memcached.MemcachedConnection.handleReads(MemcachedConnection.java:840)
at net.spy.memcached.MemcachedConnection.handleReadsAndWrites(MemcachedConnection.java:720)
at net.spy.memcached.MemcachedConnection.handleIO(MemcachedConnection.java:683)
at net.spy.memcached.MemcachedConnection.handleIO(MemcachedConnection.java:436)
at net.spy.memcached.MemcachedConnection.run(MemcachedConnection.java:1446
memcache-java.txt

Are there any step by step documentation on how to enable SASL on mac. I had hard time figuring it out on web, as there are pretty less documentation on it. Not sure if there is bug in client or their is some set up problem I'm having.

Yay! I got the issue. Couch base client automatically adds hostname at the end of the username. In my case when I set the username as myuser:mypassword but when I pass myuser in my java code, memcacheclient pass that information as<EMAIL_ADDRESS>and that's where the mismatch was happening. To solve that I added the complete username with local host when creating username and password.
echo "nishant:pass" > /tmp/memcached-sasl-db
got replaced with
echo "nishant@Nishants-MacBook-Pro-2.local:pass" > /tmp/memcached-sasl-db


Please release a new version.

I've been using the HEAD parameter in my Podfile for getting the fix in e887eea for months to avoid flashing in setProgress:animated.
The last version was released on Jul 19. Please release a new version to include these fixes after Jul 19. Thanks.

Done. version 0.1.3 released

Thanks!


vmauth: allow override default config

In a case described here user utilises vault sidecar for configuration generation, which is not currently supported by operator.
Added ability to disable config reloader if auth.config is defined in extraArgs for vmauth spec

I think, it's better to add some kind of config source field at vmauth.spec.
Which should  contain enum of possible sources for unmanaged version of vmauth.
Because, current change creates some kind of undocumented hack and this behavior could be non-intuitive for users.
WDYT of  kind.spec.ExternalConfigSource, which current has the following values:

undefined ( nil value) - configuration managed by operator. Default behavior.
secretRef - (key,name) for secret with configuration. Replaces deprecated SecretConfig.
localPath - string for locally mounted secrets.

Goal of this change - provide clear example, when configuration managed by operator and when it's not.
cc @Haleygo

It also, would be great to create an issue with propose change. To keep track of motivation and discussion of proposed changes (slack keeps history only for 3 months atm).

Replaced configSecret with externalConfig, which includes secretRef and localPath

Thanks for contribution!


Add option to record forced experiment

This PR re-implements @christophervalles's work in https://github.com/seatgeek/sixpack/pull/177 to add the ability to record forced experiments.
While I understand the initial intent of force was mainly for the ability to preview alternatives, adding this option is very useful for us for a number of reasons:

There are some 'experiments' where we want to control the segmentation ourselves - enable an alternative for staff, or users from a certain geographic area.
Be able to use segmentation on the server when using a cache/CDN like Akamai, which can perform segmentation for us in a cache-friendly manner and still use sixpack to report conversions and results.

Hmm, not sure why CI is failing. The tests themselves aren't running, pip is failing saying it doesn't recognise the --use-mirrors option, which appears it was removed from pip recently (I don't normally do Python development)

@joshhunt mind re-basing onto master? If not, thats fine, I can re-issue the pull request. I just removed the --use-mirrors flag usage in master.

Just letting you know I haven't forgotten about this! I'll fix it up this weekend


Need a lil help

https://pastebin.com/up97Wyf5
i would like to know if this is some sort of bug or if there is a fix for this?
Thanks!

Duplicate of #1419. Use the latest development build at https://ci.codemc.io/job/PikaMug/job/Quests/153/


Error handling on contract calls

I would be nice to have a possibility to handle contract calls failures, for CompletableFutures.
Currently the exception propagation stops at 'EthereumProxy.processTransactions()'

done


Bug on lectures/micrograd/micrograd_lecture_second_half_roughly.ipynb

First of all, thank you @karpathy for this amazing repo.
I think the exp function in the second half of the first lecture of micrograd has a bug:
def exp(self):
    x = self.data
    out = Value(math.exp(x), (self, ), 'exp')

    def _backward():
      self.grad += out.data * out.grad # NOTE: in the video I incorrectly used = instead of +=. Fixed here.
    out._backward = _backward

    return out

The gradient should be calculated like this: self.grad += self.data * out.grad
What do you think?

local grad is exp(x), so we use out.data, not self.data @younes-io


Undo polution of MethodCallParameter

Think it would be best to remove these parameters from MethodCallParameter
        /// <summary>
        /// Parameter can combine multiple LogEvents into a single parameter value
        /// </summary>
        public bool EnableGroupLayout { get; set; }

        /// <summary>
        /// Group Header when combining multiple LogEvents into a single parameter value (<see cref="EnableGroupLayout"/>)
        /// </summary>
        public Layout GroupHeaderLayout { get; set; }

        /// <summary>
        /// Group Item Separator when combining multiple LogEvents into a single parameter value (<see cref="EnableGroupLayout"/>)
        /// </summary>
        public Layout GroupItemSeparatorLayout { get; set; }

        /// <summary>
        /// Group Footer when combining multiple LogEvents into a single parameter value (<see cref="EnableGroupLayout"/>)
        /// </summary>
        public Layout GroupFooterLayout { get; set; }

They were introduced with #2223, but I'm not sure I like this change after reading my code again. Prefer to keep MethodCallParameter simple without being involved in this batching/group-logic.

Alternative solution is adding the following properties to WebServiceTarget:
GroupCreator-Layout - Used to perform BucketSort
GroupHeader-Layout
GroupItemSeperator-Layout
GroupFooter-Layout

So the layout will become:
  GroupHeader
     Parameters for LogEvent1
     GroupItemSeperator
     Parameters for LogEvent2
  GroupFooter

Each group will become a separate request.

Would prefer to have this fixed before NLog 4.5 RTM, but I will not have time before 21th February

Good idea


feat(i18n): add cs-CZ translation

On CrowdIn, it looks like Czech translation hasn't been finished so I would avoid merging this PR until then

@SebastienGllmt just finished translation

@SebastienGllmt just finished translation

mhm what do you mean? I just checked CrowdIn and Czech is at 0%. Would you like to join and add the translations yourself? 😄

@SebastienGllmt just finished translation

mhm what do you mean? I just checked CrowdIn and Czech is at 0%. Would you like to join and add the translations yourself? 😄

yes, thats what I did (or thought I did :D ),
is this correct link, right?
https://crowdin.com/project/yoroi-app/cs

@SebastienGllmt just finished translation

mhm what do you mean? I just checked CrowdIn and Czech is at 0%. Would you like to join and add the translations yourself? 😄

yes, thats what I did (or thought I did :D ),
is this correct link, right?
https://crowdin.com/project/yoroi-app/cs

oh that's the Yoroi extension. Yoroi mobile has a separate Crowdin project https://crowdin.com/project/yoroi-mobile

@SebastienGllmt just finished translation

mhm what do you mean? I just checked CrowdIn and Czech is at 0%. Would you like to join and add the translations yourself? 😄

yes, thats what I did (or thought I did :D ),
is this correct link, right?
https://crowdin.com/project/yoroi-app/cs

oh that's the Yoroi extension. Yoroi mobile has a separate Crowdin project https://crowdin.com/project/yoroi-mobile

OK, don't have permissions to see this project, could you grant me somehow?

@mychalvlcek sure just send me an email address, or join our discord and I'll send you the invite there https://discord.gg/pstXdg5R

Finally translation should be finished (except of terms of use)

thank you @mychalvlcek ! note that if you want,  you can put your name in the components.common.languagepicker.acknowledgement key in crowdin, and your name/nickname will be in the initial screen.


Reenable Continuous Deployment of the dApp to github pages

Description
Reenable hosting of the dApp to github pages
Acceptance criteria

Privacy policy is there #174
We need a domain name (Proposal: demo, light-client, light, demo-light-client)
It has got a CNAME

Tasks

[ ] Comms for the domain name @franzihei

As long as we haven't properly branded (found a name for) the dApp, I'd go for a domain like:
dapp.raiden.network or lightclient.raiden.network.
Generally, the shorter and the more self-explanatory, the better.


Added package for missing fonts, otherwise updated pandoc 2 with pand…

If you are missing this package and try and run this version of the pandoc command:
https://github.com/strongdm/comply/blob/master/internal/render/pandoc.go#L93 it will throw the following error:
mktextfm ecrm1000
I know that the comply command can run the correct docker image that is hosted in dockerhub and is using pandoc <IP_ADDRESS>.  In this case I want it part of my build system so I can just run comply from within the same container as what is being built here and set  pandoc: "pandoc" in my comply.yml file so it uses the local version instead of trying to do docker in docker.

@jmccarthy Awesome, thanks.


Migrated Processor code into cache package. Added test file, formatted project, golint fixing.

Coverage increased (+0.2%) to 77.269% when pulling 5b8b84364db26a8989d4ab2e321c2524110fe5dd on omegaman123:contiv-crd into 721d5949347f0afb2150062f6d0485c42ac7473c on contiv:contiv-crd.


Feature Request: Disable Logs

Thank you to the author of the package. I've used this package to extend the UI's Image component. However, I encountered a small issue where when I call it in OnDrawGizmosSelected, if there are any vertex errors, it continuously outputs logs. I only want to draw the wireframe in red to alert the user. Is it possible to provide an option to disable log output?

I have a few more minor questions:

If subdivision is not used, can it be guaranteed that the output position list will be exactly the same as the input vertex array? If so, it would eliminate the need for caching the result in an array. I also suggest providing an option to avoid allocating an extra output list during the calculation process.

For small tasks, Is synchronous immediate execution possible? In my UI extensions, there is usually a small amount of data, and synchronous immediate execution might be more efficient.

Every time I modify vertex data, an error log appears, but the input data is correct, and the output result is also correct. I can't find the reason; can you help me take a look?

MeshImage.zip

Hi @yuyang9119
Thanks for the contribution!

Is it possible to provide an option to disable log output?

Currently, BurstTriangulator outputs logs only when ValidateInput = true. If I understand correctly, you're using Triangulator on OnDrawGizmosSelected, resulting in multiple runs for the same result during gizmo drawing. You can additionally verify the triangulation status in Output.Status for any errors, including those from validation. You could implement a simple check to disable/enable Validation for future draws if the status is OK. However, caching the Triangulator result might be a better approach to avoid multiple runs and logs.

If refinement is not enabled, can it be guaranteed that the output position list will be exactly the same as the input vertex array? If so, it would eliminate the need for caching the result in an array. I also suggest providing an option to avoid allocating an extra output list during the calculation process.

Could you clarify what you mean by "exactly the same"? The input/output positions array should match up to a certain floating-point precision. You can always rely on your input data for 100% precision. Regarding caching/allocation, we're planning to add this feature soon. Currently, the package is undergoing significant development and many exciting features are coming :)

For small tasks, Is synchronous immediate execution possible? In my UI extensions, there is usually a small amount of data, and synchronous immediate execution might be more efficient.

Do you mean running on the main thread or synchronous compilation? The API includes Triangulator.Run() method, which currently calls IJob.Schedule().Complete(). With recent changes, it can be simplified to Triangulator.Run() -> IJob.Run(). I'll include this in the upcoming release!

Every time I modify vertex data, an error log appears, but the input data is correct, and the output result is also correct. I can't find the reason; can you help me take a look?

Certainly! I'll investigate this issue sometime this week and get back to you.

Best,
Andrzej

I appreciate your response and glad to see that you are continuing to update this package. Here are some additional details to clarify my previous questions.
Yes, I've already cached the results in the code. However, users may continue to input incorrect data (such as dragging coordinates with the mouse), resulting in a large number of error logs being output. From a UX/UI design perspective, the best way to handle this is to retrieve error information and display it in the inspector, rather than outputting it to the console. However, this requires support for custom error handling.

Could you clarify what you mean by "exactly the same"?

In my UI extension, there is no support for digging holes or refinement, so I guess that the output position list is exactly the same as the input vertex array—same coordinates, same quantity, and same order. In the previous comment, I provided the source code. In the code, _vertices represent the input, and _positions represent the result. If it can be confirmed that these two are identical, I can remove _positions.
Regarding "synchronous immediate execution," what I mean is to run directly on the calling thread without using multithreading. I recall some articles mentioning that if the amount of data to process is small, the overhead of using multithreading itself becomes non-negligible, and ultimately the time consumed may be greater than processing directly on a single thread. Whether to execute immediately on the calling thread or to use multithreading, I think there are two ways to handle it: one is to let the programmer decide, and the other is to automatically decide based on the amount of data.

Certainly! I'll investigate this issue sometime this week and get back to you.

Thank you very much, I await your response patiently.

I've checked your example, but I'm afraid I'm not quite sure what the issue is; it seems to be working as expected to me. There are some errors in the console related to bad input, but for valid input, the triangulation performs correctly. Could you please provide more details about the issue?
https://github.com/andywiecko/BurstTriangulator/assets/15237586/23ff8d44-b28a-45e2-b225-6e31590649b8

For small tasks, Is synchronous immediate execution possible? In my UI extensions, there is usually a small amount of data, and synchronous immediate execution might be more efficient.

BTW, I've just merged aa55beca9d70b86de6ff8c1e0ed45559f2b1f129 on a main branch and triangulator.Run() now works on the main thread . This will certainly be included in the future release. I will also push the requested Verbose option, probably today or tomorrow.

Could you please provide more details about the issue?

I just discovered that this issue is related to how Unity Prefabs work, and the error is only outputted when the Prefab synchronizes data to the instance. This problem does not belong to this package. Thank you for your assistance!

 If you have any other questions, feel free to let me know!
I've just pushed b9b6dbba5bc888342fe5e94f32106fa0b8e461bb to the main branch. This commit includes the requested feature and it will be included in the future release. To disable logging in the Console, set Verbose to false:
triangulator.Settings.Verbose = false;

By the way, in your previous example, I noticed the #if UNITY_EDITOR directive around ValidateInput. Since v2 validation is supported in builds, playmode, etc, so you can safely skip this :)


Notification policy with multiple email ids not possible neither when importing nor when applying.

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.2.9
on windows_386

provider registry.terraform.io/cloudflare/cloudflare v3.23.0

Affected resource(s)
cloudflare_notification_policy
Terraform configuration files
resource "cloudflare_notification_policy" "example_name" {
  account_id = "<account_id>"
  alert_type = "health_check_status_notification"
  enabled    = true
  name       = "example_name"

  description = ""
  email_integration {
    id =<EMAIL_ADDRESS><EMAIL_ADDRESS>### not possible! ###
  }
  filters {
    health_check_id = ["<healthcheck_id>"]
    status          = ["Healthy", "Unhealthy"]
  }
}

Link to debug output
https://gist.github.com/Done203/54c96e80afc31dd2b5b431682e0093d4
Panic output
│ Error: Incorrect attribute value type
│
│   on main_notifications.tf line 63, in resource "cloudflare_notification_policy" "example_name":
│   63:     id =<EMAIL_ADDRESS>│   64<EMAIL_ADDRESS>│   65<EMAIL_ADDRESS>│
│ Inappropriate value for attribute "id": string required.
Expected output
Plan: 1 to add, 0 to change, 0 to destroy.
Actual output
Inappropriate value for attribute "id": string required.
Steps to reproduce

create a cloudflare_notification_policy ressource with multiple values for the id field in the email_integration block.
try to apply / validate

Additional factoids
This is my first time working with Terraform so apologies if its an error on my part.
Creating a notification with multiple emails is impossible with the cloudflare ressource due to the id field of the email_integration block requiring just a string, though its possible on the cloudflare website.
Out of curiosity i imported an existing notification policy with multiple emails and i've found out that terraform takes only the last email of the policy and saves it in the state file.
I'd like the id field of the email_integration block to be a set of strings instead of a string to make multiple emails for a single notification policy possible. Maybe this might also fix the incorrect (?) import.
References
none

Fortunately just an error on my part.
It appears that each email-id needs its own email_integration block... wasn't clear to me when i read the documentation.


Placeholder for supressing logging

There's at least one use use-case for suppressed log output - particularly, some users cannot avoid calling newTracker twice. While it doesn't harm - it could be undesirable for end-user to see this message (if for some reason end-user open console). One more way to solve this particular problem is #514.
All other console output in tracker related to deprecated methods such as setCookieNamePrefix.

Does it belong to https://github.com/snowplow/snowplow-javascript-tracker/milestone/33?

Sure, added!


Beam Search Baseline

I have been going through generate/synthesis_exp.py script. I noticed that default num_beams is set to 1 for the generation. Does this mean you're doing greedy decoding for "Beam-Search" entry of the Table 1 metrics in the paper? Thanks

Thanks for your question! We actually used a beam size of 5 as described in the paper. So there's a missing argument in the script (I reorganized the scripts before release and it looks like I missed this argument). I just updated the script below. Sorry for the confusion.
https://github.com/shunzh/Code-AI-Tree-Search/blob/d075c6892ef3bd3ce2f747420acac05d406b0ce4/generate/scripts/beam/run.sh#L5


[🐞] PandaCSS Plugin Doesn't Allow CSS Object To Be Passed As Props

Which component is affected?
Qwik Rollup / Vite plugin
Describe the bug
I would like to pass some CSS to a component that's using PandaCSS css() function. Everything works correctly unless I try to pass CSS to the component as props, as shown on the PandaCSS docs here. The CSS does get passed to the component, as I'm able to log it for example. However attempting to use in the the css() function in that component is what brings up this error. The issue seems to be related to the vite-plugin-macro package somehow, although I'm not sure exactly what is causing the issue. The error:
[vite] Internal server error: cssProp is not defined
  Plugin: vite-plugin-macro
  File: /home/justin/Projects/test/test-app/src/components/test/test.tsx
      at eval (eval at transform (file:///home/justin/Projects/test/test-app/node_modules/.pnpm/@builder.io+vite-plugin-macro@0.0.7_@types+node@20.5.9/node_modules/@builder.io/vite-plugin-macro/dist/index.js:472:34), <anonymous>:3:12)
      at TransformContext.transform (file:///home/justin/Projects/test/test-app/node_modules/.pnpm/@builder.io+vite-plugin-macro@0.0.7_@types+node@20.5.9/node_modules/@builder.io/vite-plugin-macro/dist/index.js:476:28)
      at async Object.transform (file:///home/justin/Projects/test/test-app/node_modules/.pnpm/vite@4.4.7_@types+node@20.5.9/node_modules/vite/dist/node/chunks/dep-3b8eb186.js:44240:30)
      at async loadAndTransform (file:///home/justin/Projects/test/test-app/node_modules/.pnpm/vite@4.4.7_@types+node@20.5.9/node_modules/vite/dist/node/chunks/dep-3b8eb186.js:54896:29) (x2)

And this is how I'm attempting to use the css in props:
// components/test.tsx
import { component$ } from<EMAIL_ADDRESS>import { css } from "~/styled-system/css";
import type { SystemStyleObject } from "~/styled-system/types";

interface Props {
  css: SystemStyleObject;
}

export default component$<Props>(({ css: cssProp }) => {
  return
;
});

// routes/index.tsx
import { component$ } from<EMAIL_ADDRESS>import type { DocumentHead } from<EMAIL_ADDRESS>import Test from "~/components/test/test";

export default component$(() => {
  return (
    <>

Hi 👋

        Can't wait to see what you build with qwik!

        Happy coding.

      <Test css={{ bg: "blue" }} />
    </>
  );
});

Reproduction
https://github.com/Kesmek/qwik-pandacss-css-props
Steps to reproduce

pnpm create qwik@latest
pnpm qwik add pandacss
pnpm upgrade @pandacss/dev -L (The version that gets added by Qwik is quite outdated)
Create a component that accepts a style object as a prop.
Use that style object in PandaCSS css(style) function.

System Info
System:
    OS: Linux 6.4 Arch Linux
    CPU: (16) x64 AMD Ryzen 7 7700 8-Core Processor
    Memory: 20.87 GB / 30.48 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.5.1 - /usr/bin/node
    Yarn: 1.22.19 - /usr/bin/yarn
    npm: 9.8.1 - /usr/bin/npm
    pnpm: 8.7.1 - /usr/bin/pnpm
  npmPackages:
    @builder.io/qwik: ^1.2.10 => 1.2.10
    @builder.io/qwik-city: ^1.2.10 => 1.2.10
    @builder.io/vite-plugin-macro: ~0.0.7 => 0.0.7
    undici: 5.22.1 => 5.22.1
    vite: 4.4.7 => 4.4.7

Additional Information
I'm not certain that this is actually an error based in Qwik/vite-plugin-macro so if I'm mistaken, please let me know and I'll gladly bring up this issue to the appropriate project.

This is strange indeed.  I'm not sure what's different between my repo (private project) and yours, but I'm using this technique and not receiving any errors.

So this works as intended....
PandaCSS is compile time (not runtime) which means that css() calls get computed at build time. There is no access to the prop at build time; hence, this can't work.
To solve your issue, create a CSS variable and pass it to PandaCSS through styles. Here is an example:
import { component$, useSignal } from<EMAIL_ADDRESS>import { css } from "~/styled-system/css";

export default component$(() => {
  const opacity = useSignal(0.5);
  return (

      Opacity:{" "}
      <input
        type="range"
        min="0"
        max="1"
        step="0.05"
        value={opacity.value}
        onInput$={(e) =>
          (opacity.value = (e.target as HTMLInputElement).valueAsNumber)
        }
      />{" "}
      {opacity.value}
      <Child opacity={opacity.value} />

  );
});

export const Child = component$<{ opacity: number }>(({ opacity }) => {
  return (

      Hello Qwik!

  );
});

I am going to close this as working as intended... But feel free to continue the discussion or create new issues.

Oh I see, thanks for the clarification!


Not clear if I should use this or not

First of all thank  you very much for this work.  A huge step in the right direction.  On the other hand, it is a bit dated, and maybe no longer such a good starting point.   So it is not clear what I should do.
What is most interesting in the README, is your mention of using signle port RAM.  My understanding is that the J1 family requires dual port RAM, so I need to poke around in the code to figure out what you did.

So my Upduino and pico-ice boards arrived.  I published a flashing echo server gateware.  Now I need to get mecrisp running.   There are lots of different pieces here, so I am not quite sure what to do next.  I am just poking around and trying to figure it all out.


Check partial answers for ongoing requests

For long-running requests, it makes sense to show partial results in the UI (for example when searching for all references in a workspace, but even for completion). Streaming them from the server is not a very good idea, but the client could send progress requests whenever it wants and receive the current results, updating the UI. Kind of like cancelRequest but without cancelling.

I have made a proposal for progress in #70

This is more than progress report. I'd like to be able to get a list of partial results for an ongoing request. This should be something the client requests.

Streaming them from the server is not a very good idea,

Why not? Personally I very much like the idea of having a fully async and incremental mechanic for sending completions (for example).
Having this allows for deliver any results as soon it has them rather having to make an awkward decision around how much time is 'reasonable' to try and compute as many results as possible before returning an incomplete list. This often means you end up returning 0 completions even if the user was willing to wait a bit (or just stopped typing for a moment to reflect).

I think also that it would be neat with a streaming API, fully async. It has the downside is that it offers no flow control. Also, I believe that clients might have trouble handling it properly, thus making it less likely to get implemented. I started with a less ambitious suggestion, hoping to get it accepted easier.
I have absolutely no objection against a fully async protocol, when it is supported by both the client and the server. There will have to be a fallback solution for participants that need flow control or can't handle the async streams of events.

With the new capability flag support in 3.0 I can imagine a flag 'supportsResultStreaming' on the completion capability to indicate that the client accept streamed results. Then the server can stream results.

With the new capability flag support in 3.0 I can imagine a flag 'supportsResultStreaming' on the completion capability to indicate that the client accept streamed results. Then the server can stream results.

How exactly would that work? Streaming depends entirely on the client parsing the response as a JSON stream, for example if it is an array emitting each array item instantly and not waiting for the whole response to parse it completely.

Not necessarily, I think. The server could also send multiple answers to a request, with a flag saying if it's done or not. There should be a merging policy for the non-array values. This way, it works fine even if a response would contain two arrays that are computed in parallel (there isn't such a response now, but there could be).

Agree with @vladdu. This can be achieved using a paging support. That would even allow the client to request all results if a user picks a already sent one in the example of code complete.

Partial results are possible with our streaming implementation: #182

Marking as dup of #110


Meteo "Maps" section doesn't work; TLS/SSL support not available; install glib-networking

Describe the bug
Gives this error: TLS/SSL support not available; install glib-networking, yet installing glib-networking doesn't fix the problem.
Steps To Reproduce
Steps to reproduce the behavior:

Launch Meteo
Click Maps

Expected behavior
That the Maps section works.
Notify maintainers

@bobby285271

Add a :+1: reaction to issues you find important.

I have this exact issue. Hoping for some clarity.

glib-networking should be added here:
https://github.com/NixOS/nixpkgs/blob/c4e29bc0c6737bb7284b27d47efdda07fc939495/pkgs/applications/networking/weather/meteo/default.nix#L43


Using options vs individual attributes

I am failing to see the differences between these two examples, yet they have different behaviors:

https://codepen.io/toonewbie/pen/zYJvXVN
https://codepen.io/toonewbie/pen/xxaweBd

I'm not quite sure that <ma-data> supports individual attributes for constructor options. What benefits the user would get if we had this feature, what do you think?

I am failing to see the differences between these two examples, yet they have different behaviors

Mind the the structure the returned data has: in the first pen (with the individual attributes) it's an array of arrays (the default behavior), in the second one we have an array of objects (thanks to the provided headerRow and keys options).
Since not every spreadsheet has headings, MaData should support those cases too. And it does. In those cases, it returns an array of arrays where each nested array corresponds to an individual row of data. So in your case, you could have something like this:
<v-app data='{"cats": []}'>
	<ma-data v-model="cats"
           src="https://docs.google.com/spreadsheets/d/..."></ma-data>

	<article v-for="cat in cats.slice(1)">
		{{ cat[0] }} is {{ cat[1] }} years old
	</article>
</v-app>

If your spreadsheet hasn't headings but you'd like to get an array of objects where each object corresponds to an individual row of data, you can provide the needed object keys with the key option (as in the second pen).
And if you'd like to have an array of objects instead of an array of arrays but don't know what object keys to use, you can ask MaData to use the default ones—zero-based indices—by providing the keys: [] option (mind the empty array). As a result, you'll get something like this: [ { "0": "name", "1": "age" }, { "0": "tom", "1": 1 }, { "0": "kit", "1": 4 }, { "0": "catty", "1": 2 }, { "0": "sam", "1": 9 } ].
By the way, in your second pen, there is no need to use the keys: ['name', 'age'] option—your data has headings that already can be used as proper object keys. There is no need to duplicate them. OTOH, in the expenses app, the headings we have are human-readable however are not very convenient to use as object keys. So we had to tweak them a bit with the keys option.
Hope that makes sense!

I'm not quite sure that  supports individual attributes for constructor options. What benefits the user would get if we had this feature, what do you think?
Oh, I am sorry, this is a misunderstanding. I noticed the list in this link, so my first thought was that each of these are individual attributes, rather than single options. I think there should be an example showing the correct way to do it. I'll try to contribute to the documentation once I'm free. Thanks for the explanation too!

Though, maybe also supporting individual attributes is a nice thing, since all Vue plug-ins I have used before works that way.

I'll try to contribute to the documentation once I'm free.

That would be awesome! Thank you.

Though, maybe also supporting individual attributes is a nice thing, since all Vue plug-ins I have used before works that way.

Do you mind filing a feature request as an individual issue so we can discuss it?


Remove root package.json

Please squash this when merging, whoever you will be. Messed up some stuff.
Removing root's package.json file and adapting dependabot config to take the one in components directory. Dockerfiles use that one, and I could docker-compose build without issue without the root's file.

Just type reset upstream/master and commit :-) (Assuming you have the dd repo added as remote called upstream)

Just type reset upstream/master and commit :-) (Assuming you have the dd repo added as remote called upstream)

Just learned something new ;-) I was always trying to fiddle with rebase, etc... I guess my commit message could have been a bit more to the point of this PR then!


Add .finite() for numbers? Or require numbers to be finite by default?

Usually, when I want a number, I want it to be finite.
I'd like to be able to say something like z.number().finite() to represent numbers that can't be Infinity or -Infinity.
Or, even better in my opinion (but a breaking change), I'd prefer that the default number implementation disallow non-finite values, so that z.number().parse(Infinity) throws an error. I think it'd be consistent with rejecting Number.NaN, since Infinity is a number-like thing in javascript that's not really a number, and NaN is already disallowed.
Would you accept a PR to make either of these changes?

In my opinion NaN should be also accepted for z.number() because the value belongs to the Number type (typeof NaN === 'number') and also there should be a built-in refinement called z.number().rational() that excludes NaN, -Infinity and Infinity. This way Zod would stay true to the actual type definitions while maintaining practical approach.

Strongly disagree that NaN should be accepted on a simple z.number(). What's more likely? That you actually want to accept NaN or that you forgot to explicitly exclude it? I can't remember the last time I wanted to get a NaN passed anywhere. It's usually a math mistake
Something like z.union([z.number(), z.nan()]) is more explicit and will lead to less mistakes.
Also, going by "keep true to types" we might as well accept null, functions, and arrays as z.object(). The point of a validation/parsing library is to make things explicit. Allows NaN for z.number() is as implicit as allowing array/null in z.object() and IMO misses the point of a validation util
I guarantee you most junior developers wouldn't even dream about NaN/Infinity being accepted by z.number(). Not everyone knows every small intricacy/insanity of the old parts of JavaScript

I understand your point and do agree that excluding NaN from z.number() and requiring it to be specified explicitly by union can actually benefit most real-life use-cases and also help spot math errors.
But then, by the same rationale, why include Infinity which can be a result of division by zero? I can't remember the last time I wanted to get Infinity passed anywhere either. Perhaps it should be removed then.
I also think that it would be good idea to put those values in the docs so that it would be made clear which of the possible values for number type are accepted and which are not.

I would still like this.

@stale bump

@stale don't


zstandard-compare-stream: Ignore error.DictionaryIdFlagUnsupported

It is - there were several cases of this in the last fuzzing set. ZstandardStream.read calls ZstandardStream.frameInit(), and passes on this error which can be generated here.

Likely from an older version. The error would need to be part of the error set here:
https://github.com/dweiller/zig/blob/e4fcce269867964d1bc0a9421e7d9796b460d5b9/lib/std/compress/zstandard.zig#L32-L37
to be possible here.

My guess is that the format of the output is being misleading. The real error that got returned is only printed in the
zstd error: error.NoError, zig error: error.MalformedFrame

line, but it gets printed after the error return trace, so it's a bit weird. The top of the error return trace may not be the final error that reaches this catch, as it prints starting from the place the original error got returned (in this case DictionaryIdFlagUnsupported that gets eventually gets translated to MalformedFrame I'd assume).

Could be something weird going on with the error return trace (or it's being misinterpreted?), as the relevant ZstandardStream error set has never included DictionaryIdFlagUnsupported. The error would need to be part of the error set here:

I just found this too.
I tracked down the issue - the problem is that ZstandardStream.read() is masking the error, it shouldn't be doing that. So I guess the next commit in the Zstandard PR is going to require this one be implemented as well.

Thanks! The switch was also missing error.MalformedBlock and error.StreamTooLong, so I added those in.


Change /scan publisher frequency

Hi! Can somebody tell me what part of the src code must be changed in order to get a different scan publisher frequency? (Currently 35Hz)
Thank you!

Hi, you could simply set the skip value in the launch file, this will then skip the amount of messages specified. Reducing the publishing frequency.

Thank you so much!!

Currently I'm getting frequency of 35 hz . Is there any way to increase it ?


[FEATURE] Add Internationalization

There is a request to internationalize the text and select the language in the settings of the locally running application

open to add i18n - https://react.i18next.com/

If this is still on the to-do list, I’d love to implement this feature.


Merge pull request #1 from Microsoft/master

update

@dreadlord1984 , thank you very much for the contribution. However, it seems that this PR has no changes.


[Designer] Capitalization of property values inconsistent

Setting property values with ELEMENT PROPERTIES will (usually?) capitalize the value. For some properties in certain contexts, the capitalized value is marked with a warning in the CARD PAYLOAD EDITOR.
I've not been able to establish consistent circumstances but a simple example is below.
New Card => Flight Itinerary
Microsoft Teams - Light
v1.2
Changing the value in the EDITOR to lowercase satisfies the warning temporarily but it will revert at some point after making changes.

@planetpearce This is a problem with the schema file used by the designer, which isn't a recent version. Enum values are case insensitive in the AC schema, these warnings shouldn't exist and can be ignored. We will eventually make it so that they don't appear anymore.

Awesome, thank you


Feature Request: Tables

Hi,
I'd like to see the use of tables in this repo.
Pls lemme know any thoughts.

What does that even mean? Please actually describe what you want.


unfair: NPCs can launch their simple melee attack during iframes knock back.

NPCs shouldn't be able to harm the player when they're in knock-back, iframes.

I haven't noticed this lately


[5.6] Use consistentCarbon versioning across 5.5, 5.6 and 5.7

Rather than upgrading from only 1.25 to 1.26 (https://github.com/laravel/framework/commit/b12feabda70f136cb2d7081af41441a0bf22edc7), version should be aligned on 5.5, 5.6 and 5.7 with ^1.26.
1.26.* is no longer maintained, far from last version 1 and even more from 2 (current state of documentation).
And version 1 only receive security/language fixes. All new features are added only in version 2.
This means you can safely use the current 1 (1.36.2) as ^1 release frequency is now very slow, and will now have no method added, and no test changed.

Laravel 5.6 is also no longer maintained, and this could also be considered a breaking change.

Not sure to understand. Upgrading from 1.25 to 1.26 was not a BC but 1.26 to 1.36 would be?
Next version of 5.6 will just to a earlier minor version of Carbon, that seemed to me a good occasion to take the earliest (minor).
Unit tests reveal a change I should inspect, but it's more likely due to the behavior of DateTime::createFromFormat. That could be a precision lack in tests (hidden before) rather than a BC due to version upgrade. I will try to provide an example showing it with 1.26.

5.6 isn't maintained anymore. Please upgrade to 5.7. 5.8 will be out at the end of the month and maintenance for 5.7 will stop next month.

We don't make changes to 5.6 anymore except for extraordinary circumstances.


Tile function is not working.

Engine Version 1.52.6.
I used "Tile" because I wanted to express the same pattern repeating.
Placed it under 2D Screen with Element→Type "Image".
It looked fine.
However, when Engine became Version 1.53.2
I started getting the following error.
Failed to compile fragment shader:

ERROR: 0:210: 'texture' : no matching overloaded function found
ERROR: 0:210: 'a' : field selection requires structure, vector, or interface block on left hand side

205: #ifdef MAPFLOAT
206: dAlpha *= material_opacity;
207: #endif
208:
209: #ifdef MAPTEXTURE
210: dAlpha *= texture2D(texture_opacityMap, nineSlicedUv, -1000.0, textureBias).a;
211: #endif
212:
213: #ifdef MAPVERTEX
214: dAlpha *= clamp(vVertexColor.VC, 0.0, 1.0);
215: #endif

[object Object]

Error in material "defaultScreenSpaceImage9TiledMaterial" with flags 65797

I have created a project to verify the issue.
https://playcanvas.com/project/919074/overview/tiletexture
I have tried Image, Sprite, Texture, and I tried Repeat and Clamp ......

Thanks, we'll be fixing this today.

Fixed by #4214

Thank you!


Fix a System.FormatException caused by invalid format string

If compilation fails with an error such as this:
testproj.sharpmake.cs(59,2) : error CS1513: } expected
Shaprmake throws System.FormatException because the error format string contains a character which is considered as a special formatting character. This can be very confusing and prevents users to find the actual problem which is in their script code.

Thanks for the fix!


docs: add supported Git URL patterns

I've added documentation on the supported Git URL patterns, mainly URL prefix/postfix compatibility tables in which case Copier will recognize a URL as a Git URL. This documentation is based on the copier.vcs.get_repo(url) function:
https://github.com/copier-org/copier/blob/47f3d59c27094e6f8ef0bc71acc74d88cbf6bb78/copier/vcs.py#L67-L95
Resolves #1043.
Would this additional documentation have helped you, @sandeeps311?

@sisp It is very helpful, Thank you

Well, @yajo finds the tables confusing. So we should find an alternative description of the rules. I was struggling a bit to come up with the tables because Markdown tables are quite limiting and I didn't want to use HTML tables because the markup wouldn't look good. Alternatively, I could add a bullet point list with rules that describe the relationship between the prefixes and postfixes for the different protocols. Or ang other ideas?

Simple but effective, @yajo. :+1: :smile_cat: I've changed the docs as you suggested.

Thanks for the review and suggestion! 🙏


Dagger module error message with a mistyped --sdk flag

When I initialize a new module with dagger init and I mistyped the --sdk, I'm getting the following error.
$ dagger init --name=my-module --sdk=typescrip
Error: failed to generate code: input: resolve: moduleSource: withName: withSDK: withSourceSubpath: resolveFromCaller: failed to collect local module source deps: failed to collect local sdk: missing config file /Users/slumbering/workdir/dagger-zenith/trivy/typescrip/dagger.json

I'm wondering if we could improve this message. Do you think there's anything we could do to make it clearer or more effective?
e.g:
The typescrip SDK does not exist.

Available SDKs:
- go
- typescript
- python
- ...

@helderco @jedevc @vito Is it still a relevant issue?

Yes, the confusing error still appears.

I like the way AWS CDK makes the error clear:
$ cdk init app --language g
Invalid values:
  Argument: language, Given: "g", Choices: "csharp", "fsharp", "go", "java", "javascript", "python", "typescript"

@helderco I'm pretty sure it's doable from the CLI, but the sdk field could also be a URL so we would need to be generic on the helpers.
Like
The typescrip SDK does not exist.

Available SDKs:
- go
- typescript
- python

You can also use a no-bundled SDK from its github URL (e.g., https://github.com/dagger/dagger/sdk/rust@main)

I am taking a look at it today. I am planning to add:

a local var in cmd/dagger/module.go with list of sdk supported.
a validation function that verifies that if the sdk is provided, it is either one of the configured list or an https url.
add a PreRunE step to cobra commands init and develop.

I think we should catch this error even before our call reaches engine, and therefore i am adding validation in PreRunE step, but also open to suggestions here.
If this approach looks good, I can open a PR for this.

add a PreRunE step to cobra commands init and develop.

Hm, this is a bit problematic. If we do this, then it means if the engine has a new SDK that the CLI doesn't know about it might not work. We have some tolerance for different versions of CLI and engines to connect, so we need to take that into account.

https url.

The defined format is a module ref - atm, they don't have schemes at all, but #7708 will change that.

I think this might be relevant: https://github.com/dagger/dagger/blob/82ff692e30025f60bfb7ab1f2c126ccaf05ae460/core/schema/sdk.go#L85-L97
Potentially, this could be reworked into a global map - builtinSDK then just looks up the SDK using a map, and then we can easily derive the list of SDKs from that.

Hi @jedevc , thank you for the additional info/context.
when we add and release a new built-in SDK, the engine may support the new SDK, but the CLI may not know about the new SDK yet? if that is the case, I am thinking we may need to add support to query supported sdk's from the engine, and use that to validate. because if we use global map, the CLI may still be built using old values of map?
Kindly pardon my understanding of the flow here, and please correct me if I am missing something here.

Could be an error returned by the engine when the request is made, instead of a pre-validation from the CLI before the request, which would have to happen even when that value is correct. Makes for a more informative error when used from the API or a custom client too (I mean Module.withSDK, etc).
The only downside is in the formatting (newlines, etc), but maybe that's not necessary. We already have a similar error message when using an enum with an invalid value. The --sdk is basically an enum too.
Even if it is worth the "rich" formatting, you could have a custom error[^1], to avoid the need for parsing the error message in order to extract the necessary information.
[^1]: No need to implement in SDKs... the custom API errors will one day be a part of codegen in an automated way.

I am looking at this issue, and have few questions:

should we add this validation to only topLevel sdk or the dependencies as well? I think we should add the validation irrespective of toplevel here.

do we support running dagger init with local-path-sdk as well? e.g. in collectCallerLocalDeps, specifically here after we confirmed that the sdk is NOT the in-built SDK, we verify if it is a local ref or a git ref.

depending on whether local-sdk-path is supported or not in dagger init, we have two options:

if providing local-path-sdk to dagger init is valid, then we need to add a check inside case core.ModuleSourceKindLocal: to check if the directory exists or not on host (possibly using buildkit.StatCallerHostPath() function), and error out accordingly. IMO, this is what we might have to do.

if we don't support local-path-sdk, then we can return error if it is not a built-in sdk AND not a git ref.

Even if it is worth the "rich" formatting, you could have a custom error1, to avoid the need for parsing the error message in order to extract the necessary information.

I tried adding it as a custom error on engine side, but by the time error is returned to init command, it no longer has its type retained and is of type gqlerror.List, so there are two options IIUC:

we send the final error msg from the engine side, and let the CLI print the error msg as is. (it could be either of the multiline formatted error or one line msg)
we send a "invalid sdk" error msg, and based on that string print a more formatted error on CLI side.

IMO, we should send the msg from engine, and print it as is on the CLI side.


chore(gh action): fix to pull before pushing...

What does this PR do?
chore(gh action): fix to pull before pushing to avoid collisions RHIDP-2842, second attempt
Signed-off-by: Nick Boldt<EMAIL_ADDRESS>Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
N/A (or see commit message above for issue number)
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:

[x] The Eclipse Contributor Agreement is valid
[x] Code produced is complete
[x] Code builds without errors
[ ] Tests are covering the bugfix
[x] The repository devfile is up to date and works, if one exists
[x] Sections What issues does this PR fix or reference and How to test this PR completed
[x] Relevant user documentation updated
[x] Relevant contributing documentation updated
[x] CI/CD changes implemented, documented and communicated

Reviewers
Reviewers, please comment how you tested the PR when approving it.

RHDH docs triage: @themr0c Can you review this?

@nickboldt Can you update the branch please?

@briandooley your whole team should be able to push the Update branch:

That way, the PR can be rebased, and a few mins later you have an updated PR preview too!

@briandooley your whole team should be able to push the Update branch:

That way, the PR can be rebased, and a few mins later you have an updated PR preview too!
Also since a PR like this doesn't affect anything content wise, rebasing it won't actually change the PR preview.
Plus when PRs are merged they're automatically rebased... so for the above 4 reasons, you shouldn't need me to push the button for you :D

Indeed @nickboldt. Just prefer owners of PRs to do things like that. :D

Indeed @nickboldt. Just prefer owners of PRs to do things like that. :D

It takes less time to mash the button than to ask someone else to mash the button. Why MAKE work when you can REDUCE workload for others?

@hmanwani-rh Can you review this?


Generate random password when master_password is empty

what

If master_password is not provided or empty random value will be used

why

AWS API requires to provide password
Secrets should not be kept in repository

can someone let me know what's wrong with bridgecrew analysis? I don't have access there

/test all

/test all

@dkossako Thanks for the contribution! Released as https://github.com/cloudposse/terraform-aws-documentdb-cluster/releases/tag/0.15.0


Put together a game plan for leveraging Cockroach U. with new employees

Meeting w/ Chelsea 1/7
DoD

We decide what the expectations are for new employees (which ones will take the course, what the standards are)
Loop in managers on the plans based on Chelsea's advice
Draft a list of expectations for Training (what info will go to managers)

Closing since this has been moved to "Done" in the sprint board.


Accept Twitter posts

I can use Zapier to push my Twitter content over to the API.
This can then be formatted, and added for webmentions.
I'll need slightly different designs for tweets, replies, retweets and likes.

I'm cheating a bit on this one.
I have a Zapier zap that publishes from a Notes feed with a category of "notes".
This means anything I deem a short form note gets auto POSSE to Twitter.


Integrate the new Gnosis chain testnet

At the time Gnosis chain was integrated, it didn't have a dedicated testnet so I used POA Sokol. Now there is a Gnosis chain testnet and we're still referring to POA Sokol as gnosis-testnet. We should do a Gnosis chain testnet integration and replace gnosis-testnet contract addresses with that, assuming no one wants to use POA Sokol anyway.

Looking at the actual Gnosis Chain testnet (formerly xDai testnet) the last block seems to be 2 months ago https://blockscout.com/xdai/testnet
Gnosis chain docs recommend Sokol to be used https://developers.gnosischain.com/for-developers/developer-resources/smart-contract-deployment which we have already integrated under the name gnosis-testnet.
It should be clear to the user from the chain ID that we're referring to POA Sokol when we say gnosis-testnet, so I'm closing this issue


Crash while loading (solo mode) world + modules

What you were trying to do
After first launch of the game, for my first game I chose the solo game, and there the Sample something world. To customize this world I took several modules, as nothing was warning this could cause a trouble...
What actually happened
When the game I selected started to load, it suddenly crashed.
How to reproduce
The problem is I don't remind exactly which modules I selected to customize the world...

Step 1 : take the first world from the list in Solo mode of the game, named "something Sample something"
Step 2 : add modules, not all of them but several, and all the ones which were shown in yellow as I supposed they were needed by the modules I took (total around 20 certainly, maybe a little bit more)
Step 3 : click on "play", the game showed the loading screen a short moment then crashed.

Log details and game version
Terasology - Version
Jenkins:
URL: http://jenkins.terasology.org/job/TerasologyStable/71/
Build number: 71
GIT branch: origin/master
Created at: 2017-03-20T05:04:33Z
http://terasology.org
https://github.com/MovingBlocks/Terasology
Computer details
Processor Intel Core i7 - 6700 CPU @ 3.40 GHz (x8)
Graphic Card NVIDIA GeForce GTX 1060 6 GB
RAM 8 G
Network : Eth, WiFi
OS : Windows 10
Log details : https://pastebin.com/cgTFuZvw

That's an OutOfMemoryError so the Java Virtual Machine running the game ran out of memory while perfotming the task.
You could try to allow the JVM to use more of your machine's physical memory.
How do you run the game? If you use the launcher, go in its settings (by clicking on the gear icon) and tweak with the Max and Initial memory settings; since you have 8GB total I would set them to 6GB, provided you run the game without other programs in background.
Also you should ensure that you are using a 64-bit JVM (using a 32-bit JVM on a 64-bit CPU on Windows works due to WOW64, but with performance limits). Since you are on Windows, you can do it by going in Programs and Features in the Control Panel and searching for "Java". Look if you see something like Java 8 Update 121 (64-bit). If you only have a Java 8 Update 121 it's the 32-bit JRE; in this case, to get optimal performance you should download and install jre-8u121-windows-x64.exe from here.

First, thank you very much for your very quick answer, I'm really impressed (there is games I paid for whose devs made me wait more than several months before they "try" to get interest in my issues... And I send reports because I think that can help them to fix troubles, not to complaint !). Second, excuse me if this was not really a game or java bug, I must say I am not at all good with Windows OS (I am a Linux user forced by hardware troubles to use a friend's computer in place of mine, and I know nothing about Microbrain's Windows). I checked the JVM version and it was effectively a 32 bit one, then I installed the good one in place (thanks again, for the link). I tried the game since then and it runs nicely now, so I can tell you the case is solved. Have a nice day, and thanks a lot again !

Glad you solved!
Your report is important, because it points out the game should warn the user in some way when it's running on a 32-bit JVM on a 64-bit system, to avoid people not taking advantage of their powerful hardware. The installer I'm developing for Windows shows a warning during installation if only a 32-bit JRE is found; however an in-game warning should probably be added for people which get the game through other distribution formats (e.g. zip, packages on Linux, etc).

Ok, happy I did not totally make you loose your time ! I installed the game from the zip package for Windows (I did not want to use the installer because it's not my computer, and I must be able to take off everything I added, without having to crawl through the register to erase traces...), then no warnings appear during "installation" (unzip + cut/paste folder...). Thanks for this game, and for the fact I will be able to play it under my Mageia Linux as soon as my own computer will be repaired ! By the way, if you ever need an english to french translator for any program, I did already several translations for Linux (gettext, KDE and Qt, and even text parts of scripts) and I would help with pleasure (my way to pay back the apps I use...). Thanks again and have a nice day !

Closing as resolved :-)
Hi @magali-demorangies, thanks for the report, and I'm glad you were able to resolve it :-)


Multiday Program

Taking over a feature request from the original repository:

I am hosting a conference with a 3 day program. If someone has used this template for such events before, I'd be very happy to get suggestions for

adjusting the datastructure and includes for the program,
adding social events or other non-talks to the program

From the point of view of this project, please interpret this issue as a long-term feature request. :)

For more details see the original issue under https://github.com/DigitaleGesellschaft/jekyll-conference/issues/10

Added in edd7656bc3366483c962ff5eb3144f5bb217d280


Crash when clicking on Google Docs link

In the Kottke.org feed, the article at https://kottke.org/20/10/how-to-protect-yourself-from-the-aerosol-spread-of-covid-19, there is a link at the bottom to a Google Docs file, at https://docs.google.com/document/d/1fB5pysccOHvxphpTmCG_TGdytavMmc1cUumn8m0pwzo/preview?pru=AAABdQ5s5mA*YrLy9epGlHWhKPSl5JMT3w#
Clicking on this link crashes NetNewsWire 5.1 (3007).
Crash log is attached.
NetNewsWire Crash.txt

Duplicate of #2445.


Pylint, docstrings, formatting

Apologies for the large PR, none of these changes affect functional code, just comments/formatting.
Here is what I did:

Ran everything through pylint and made edits that made sense. I ignored some things if they weren't reasonable
Slightly updated the docstring format and unified them across all modules
Added class docstrings for the fields.py and expansions.py scripts (which include a "how to use" section for each) in case people (such as myself) like to use the help() function instead of the Wiki
Polished up the examples/
Reran the tests.py --> everything passed.

Closes #80

It's nice to have things cleaned up!


Use setImmediate(fn) Instead Of setTimeout(fn, 0)

IE10+ and Edge provide native non-blocking setImmediate and, for non-supporting browsers, the messageChannel API can provide a non-blocking temporary polyfill. Non-blocking means that DOM updates, HTTP requests, and everything else is able to happen in between the time setImmediate is called, and its call back is executed. Thus, a non-blocking setImmediate would provide the exact same functionality as setTimeout(fn, 0) INCLUDING the below improvements.

setImmediate could potentially make CodeMirror more responsive in some cases because although 4ms is less than most monitors framerates, things built on top of CodeMirror or that extend CodeMirror would be given 4ms more to process before there is a "lag-frame" which fails to update because the CPU is swamped.
setImmediate put into CodeMirror at this critical juncture will make CodeMirror several times more power efficient because instead of the browser having to swamp the CPU with operations to get 4ms resolution timing, the function will be executed right away, leaving the CPU able to "idle-out" and conserve lots of power.
Lastly, I would like to explain my reasoning for only including the setTimeout backup in this change without also including postMessage. Indeed, Microsoft has taken windows Vista off extended support, clearing the crowds of people quite a lot. Yes, it is true that you could elaborate on this polyfill and extend support to IE9 by adding a postMessage fallback, however you would increase the size of CodeMirror by an appreciable degree, run the risk of conflicting with other javascript that uses postMessage, and break IE8 compatibility because IE8 does synchronous postMessage. For all these reasons, I would say that the postMessage implementation presented in this change is good as is, and should not be modified to add in extra compatibility.

Therefore, because this setImmediate change will make CodeMirror more responsive and more power efficient, all without harming the IE-crowd, I submit that this major change should be accepted right away and not discarded.

Could you remove the big fallback and just use setTimeout on any platform that doesn't have setImmediate? It seems more code than is justified by having a slightly better polyfill on a few old platforms. Also, are you sure that not passing a second argument to setTimeout is supported and behaves as passing 0 on all platforms?

(P.S. I am the same guy as AnonyCo, but on a different P.C.)
Marinjh, let me first answer your question about the fall back. As seen here, direct native setImmediate has unfortunately been removed from the standard. and very few browsers support it. Thus, a polyfill/alternative is required. However, I can assure you that this alternative setImmediate I have put in is just as good as the real thing. To prove this, open up a new tab in your browser at example.com. Then, run the below code in the console.
(() => {
  "use strict";
    const setImmediateOrFallBack = (() => {
      if (typeof setImmediate !== "undefined") return setImmediate // for IE10+
      if (typeof MessageChannel === "undefined") return setTimeout // for IE9 and below
      const msgChannel = new MessageChannel
      const port1 = msgChannel.port1
      const port2 = msgChannel.port2
      const postMessage = port2.postMessage.bind(port2)
      var waitingImmediates = null
      var immediatesLen = 0
      port1.onmessage = () => {
        var curImmediates = waitingImmediates
        waitingImmediates = null
        for (var i = 0; i !== immediatesLen; i++) {
          curImmediates[i]( /*no arguments*/ )
        }
      }
      return f => {
        if (waitingImmediates === null) {
          waitingImmediates = [ f ]
          postMessage( undefined )
          immediatesLen = 1
        } else {
          waitingImmediates.push( f )
          immediatesLen += 1
        }
      }
    })()
    var i = 1000000
    console.log("starting!");
    setImmediateOrFallBack(function doNext(){
    if (i--) {
      setImmediateOrFallBack(doNext)
        } else {
      console.log("All Done!")
        }
    })
})()

After running the above code in the console, try interacting with the page (selecting text and what-not). Then, open up a task manager and observe that the page is using up 100% of the CPU power for its thread (100/#virtual_threads% of the total CPU power). Now, try closing the page and observe that even page close events are not blocked. As you can see, this polyfill for setImmediate is non-blocking which is incredibly good for the UI while at the same tie saving power because there is only a minimal delay. As for browsers that natively support a setImmediate like this, which are IE10+ and Edge, I think that it would be a shame if we did not take advantage of native support in those browsers.
As for your second question about the second argument to setTimeout, I am pretty certain that the second argument to setTimeout is coerced to an integer. Thus, when the second argument is undefined, that undefined is coerced to an integer, resulting in 0 like so.
undefined | 0 === 0 // is true

To prove this for absolute certain, I did tests in the two browsers that will use the setTimeout fallback: IE8 and IE9. I used IETester and the result (shown below in images) conclusively prove that when there is no second argument to setTimeout, setTimeout behaves as if the second argument is 0. Please open the images in a new tab for a better view of them.
Internet Explorer 8:

Internet Explorer 9:

I really do not want that big polyfill in the code for the obsure, rare case where this setImmediate implementation is actually being used. Is there a concrete issue this patch is solving, or did you grep for 0-ms timeouts?

marijnh, for the issue of the polyfill size, I have run my code through the closure compiler and, as seen below, there will only be a 367 byte increase (before gzip is applied).
var constsetImmediateOrFallBack=function(){if("undefined"!==typeof setImmediate)return setImmediate;if("undefined"===typeof MessageChannel)return setTimeout;var e=new MessageChannel,f=e.port2,g=f.postMessage.bind(f),a=null,c=0;e.port1.onmessage=function(){var b=a;a=null;for(var d=0;d!==c;d++)b[d]()};return function(b){null===a?(a=[b],g(void 0),c=1):(a.push(b),c+=1)}}();

If you want to sacrifice a little performance for a smaller size, then we can even make that 240 bytes before gzipped.
Source Code (564 bytes):
const setImmediateOrFallBack = self.setImmediate /*IE10+*/ || (MessageChannel => { // for all others
  if (!MessageChannel) return setTimeout; // for IE9 and below
  const msgChannel = new MessageChannel
  const waitingImmediates = []
  var immediatesLen = 0;
  msgChannel.port1.onmessage = () => {
    while (immediatesLen) {
      waitingImmediates.shift()( /*no arguments*/ )
      immediatesLen--
    }
  }
  return f => {
    if (!immediatesLen) msgChannel.port2.postMessage( 0 )
    immediatesLen = waitingImmediates.push( f )
  }
})(self.MessageChannel)

Compressed (240 bytes without gzip):
window.setImmediateOrFallBack=self.setImmediate||function(a){if(!a)return setTimeout;var c=new a,d=[],b=0;c.port1.onmessage=function(){for(;b;)d.shift()(),b--};return function(a){b||c.port2.postMessage(0);b=d.push(a)}}(self.MessageChannel);

Next, for the issue of how this could be practical, please see my code. This setImmediate function will be used in every single signalLater call. signalLater, as I have seen frequently in the CodeMirror source code, is used quite frequently. With this change that I am proposing, every single place in the entire CodeMirror code that uses signalLater will receive a major power-saving performance benefit that will make CodeMirror a lot more appealing to many more people. For me personally, I would consider 240 un-gzipped bytes of code to be just a small, but very worthy, sacrifice for tremendous power saving that I would gladly implement.
Lastly, there is no issue. This proposal does not fix any code since the code was already working before. Rather, this proposal simply makes the existing code better.

This setImmediate function will be used in every single signalLater call.

That's not the case, though—almost all signalLater calls are used under an operation, in which case the setTimeout doesn't run at all.
As such, I'm going to close this as adding more complexity than warranted.

Marijn, thank you for clarifying that. I now understand. I am very glad that you have reached the best decision for the future of CodeMirror. Thank you for your time.


[Bug]: YouTube links in Repo Guide not working

Is there an existing issue for this?

[X] I have searched the existing issues

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Current Behaviour
All of YouTube links in the Enarx repo guide are not working (video marked as not available).
Expected Behaviour
The videos are available.
Environment Information
Not relevant
Steps To Reproduce
No response

Unfortunately, when Profian closed the account and videos were deleted. I believe @nickvidal has some of the videos. Can we get any videos restored?


Use String.toLowerCase(Locale.US)

@bot-gradle test and merge

OK, I've already triggered a build for you.


chore: Fetch pages in editor with READ permissions

Fixes #18371
With GAC, it is possible now to take away edit permissions for a given page even though the user might have edit on the entire application (except a particular page). This change ensures that as long as the user has view permission on a page, they would still be able to open the said page in the editor.

/ok-to-test sha=5a9318f


how to update Direct image

Could you please give more details about what you specifically need?
For how to update an image you can check the package doc here Update an image and Official doc here
Just a side note: if what you want is to do an image update from a flutter app, you should take into account image update requires a cloudflare signed request I mean a request using accountId and token, so this should not be handled at app side but at backend side.

Could you please give more details about what you specifically need? For how to update an image you can check the package doc here Update an image and Official doc here
Just a side note: if what you want is to do an image update from a flutter app, you should take into account image update requires a cloudflare signed request I mean a request using accountId and token, so this should not be handled at app side but at backend side.

The problem is what data is needed in MetaData ?

Did you read the Official doc? What you ask is explained there.
Metadata is a Map, no data is needed there, it is a map you can use to set any key:value pair of info you want to store with your image.

User modifiable key-value store. Can be used for keeping references to another system of record for managing images. No change if not specified.
"{"meta": "metaID"}"


Drop Fedora 39 from ponyup-init.sh

Once we have our first Fedora 41 release, we need to drop Fedora 39 support from the init script.

This will be closed by #309


FAQ Shows Black for Some Users

Opening the FAQ apparently shows black on the iPhone 8. I have not been able to repro this issue.

This should be fixed with PR #73.


Will this work be published on any conference or transactions?

CLIP4clip is a great work to promote text-video matching, I am curious about whether this work will be submitted to any conference or transactions? Thanks!

Yes, it is under review for a long time. Thanks.

Yes, it is under review for a long time. Thanks.

Thanks for satisfying my curiosity, good luck to you ~


Update configure-s-mime-settings-for-outlook-web-app.md

Added  AAD joined devices to the considerations

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/2411

@JohanFreelancer9  Please copy review

@chrisda Please merge this PR. Thanks


docs: Add installation instructions for AstroNvim users

Hi this seems to be the case for any lazy enabled neovim setup and doesn't have much to do with AstroVim itself.
Plug is a separate package manager. This seeems out of scope for the project.

However, maybe adding just the {waycrate... line is better to keep the info public.

This looks great @HamzaMateen. Thanks!


Profile settings keys page

Integrates profile-settings-keys into Gitness, adding support for public keys.
Adds navigation from avatar on bottom left to settings/profile.
Adds table component for displaying SSH keys in playgroun/components.
Attached screenshot is from Gitness.

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


Update TRSS api

fixes: https://github.com/AdoptOpenJDK/openjdk-test-tools/issues/243
together with this PR https://github.com/AdoptOpenJDK/openjdk-test-tools/pull/244
[ci skip]
Signed-off-by: Yixin Qian<EMAIL_ADDRESS>

Since this can only be tested locally, could you paste the local test output? Thanks

Attempting to get test duration data from TRSS:
curl --silent --max-time 120 localhost:3001/api/getTestAvgDuration?limit=10&jdkVersion=11&impl=j9&platform=x86-64_mac&group=functional&level=sanity

TEST DURATION
====================================================================================
Total number of tests searched: 1598
No test duration data found.
(Default duration assigned, executed tests: 40s; skipped/disabled tests: 0s.)
====================================================================================

Test target is split into 3 lists.
-------------------------------------testList_0-------------------------------------
Number of tests: 534
********

No error is thrown here and json is successfully parsed

Did you have data by using localhost:3001/api/getTestAvgDuration?limit=10&jdkVersion=11&impl=j9&platform=x86-64_mac&group=functional&level=sanity?

Sorry I didn't use openjdk group , here is the updated test output
Modes data parsed from resources/modes.xml and resources/ottawa.csv.

Attempting to get test duration data from TRSS:
curl --silent --max-time 120 localhost:3001/api/getTestAvgDuration?limit=10&jdkVersion=11&impl=j9&platform=x86-64_mac&group=openjdk&level=sanity

TEST DURATION
====================================================================================
Total number of tests searched: 7
Number of test durations found: 7
Top slowest tests:
        41m38s jdk_lang_j9_0
        32m58s jdk_util_j9_0
        09m23s jdk11_tier1_pack200_0
====================================================================================

Test target is split into 3 lists.
Reducing estimated test running time from 92m48s to 41m38s.

-------------------------------------testList_0-------------------------------------
Number of tests: 1
Estimated running time: 41m38s
TESTLIST=jdk_lang_j9_0
******

TRSS server is updated: https://trss.adoptopenjdk.net/api/getTestAvgDuration?platform=x86-64_linux&level=sanity&group=openjdk&jdkVersion=8&impl=j9&numMachines=3


Use of Meteor.Collection causes crash with<EMAIL_ADDRESS>

I have problems using<EMAIL_ADDRESS>through<EMAIL_ADDRESS>together with for example<EMAIL_ADDRESS>The issue seems to be cased by the use of Meteor.Collection in the cfs packages, instead of the now preferred Mongo.Collection.
The error message is:
W20150302-16:13:27.006(1)? (STDERR) 						throw(ex);
W20150302-16:13:27.006(1)? (STDERR) 						      ^
W20150302-16:13:27.050(1)? (STDERR) Error: use "new" to construct a Mongo.Collection
W20150302-16:13:27.050(1)? (STDERR)     at new Mongo.Collection (packages/mongo/collection.js:29:1)
W20150302-16:13:27.050(1)? (STDERR)     at [object Object].ns.Collection (packages/matb33:collection-hooks/collection-hooks.js:190:1)
W20150302-16:13:27.050(1)? (STDERR)     at Package (packages/cfs:tempstore/tempStore.js:31:1)
W20150302-16:13:27.050(1)? (STDERR)     at /Users/johan/dev/collection-hooks-test/.meteor/local/build/programs/server/packages/cfs_tempstore.js:403:4
W20150302-16:13:27.051(1)? (STDERR)     at /Users/johan/dev/collection-hooks-test/.meteor/local/build/programs/server/packages/cfs_tempstore.js:410:3
W20150302-16:13:27.051(1)? (STDERR)     at /Users/johan/dev/collection-hooks-test/.meteor/local/build/programs/server/boot.js:205:10
W20150302-16:13:27.051(1)? (STDERR)     at Array.forEach (native)
W20150302-16:13:27.051(1)? (STDERR)     at Function._.each._.forEach (/Users/johan/.meteor/packages/meteor-tool/.<IP_ADDRESS>ocyxen++os.osx.x86_64+web.browser+web.cordova/meteor-tool-os.osx.x86_64/dev_bundle/server-lib/node_modules/underscore/underscore.js:79:11)
W20150302-16:13:27.051(1)? (STDERR)     at /Users/johan/dev/collection-hooks-test/.meteor/local/build/programs/server/boot.js:116:5

The error can be reproduced like this:
meteor create collection-hooks-test
cd collection-hooks-test/
meteor add cfs:standard-packages
meteor add sewdn:collection-behaviours
meteor

It seems to be something sewdn:collection-behaviours is doing that matters too. This since if I do not include that package - but instead only add matb33:collection-hooks directly - I do not get the error. So perhaps this issue should be reported to that repo, but the thing is that I could avoid the issue by modifying the matb33:collection-hooks code only.
I have managed to avoid the error with two different methods,

I made a local copy of the cfs packages that used Meteor.Collection and changed them to use Mongo.Collection and everything was fine.
I made a local copy of the<EMAIL_ADDRESS>and edited the code in CollectionHooks.wrapCollection() to handle Meteor.Collections by replacing

 var ret = constructor.apply(this, arguments);

with
 var ret;
    if (ns === Meteor) {
      ret = new Mongo.Collection(arguments['0']);
    } else {
      ret = constructor.apply(this, arguments);
    }

Is this change wrong in any way? Or should I make a PR with that change to allow the use of collection-hooks together with collection-behaviours and packages that use Meteor.Colelctions?

It looks like<EMAIL_ADDRESS>is deprecated. I hesitate to attempt any fixes if this is the only package causing issues...


emptyArray() does not match an empty array

From<EMAIL_ADDRESS>on January 18, 2014 01:18:09
What steps will reproduce the problem? I have a class RESTful Controller that returns a body with just
[]
Then I run these two lines of code on it.
get("collections").then().body("", empty());
get("collections").then().body("", emptyArray()); What is the expected output? What do you see instead? I expect the first one to fail, and the second one to pass. Or POSSIBLY both to pass. But the second one fails. What version of the product are you using? On what operating system? Newest version of rest-assured Please provide any additional information below. This is the error it gives
java.lang.AssertionError: JSON path  doesn't match.
Expected: an empty array
Actual: []
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=300

From<EMAIL_ADDRESS>on January 18, 2014 02:42:48
That's probably because it's represented as a List and not array. If there's such a thing as emptyList() then try that instead. I usually use "hasSize" instead:
get("collections").then().body("", hasSize(0));

From<EMAIL_ADDRESS>on January 18, 2014 06:45:59
Thanks! I was able to get it working with the below code.
    get("collections").then().body("", emptyCollectionOf(List.class));

From<EMAIL_ADDRESS>on January 18, 2014 08:08:54
Great!
Status: Invalid


Grammatical errors in readme

There are some grammatical errors in README.md file.
I'd like to eliminate that.

I would like to work on this issue please assign to me under LCM Soc21

Go ahead
Deadline 13 June,2021

I would like to work on this issue. Please assign me under LCM Soc21.

@kshitija-2001 assigned to you please check if any other errors are still left
Deadline 17 june

Ok Thank you


Add TDD view to dashboard

This PR -

Adds TDD View to the dashboard
Consolidates API queries into central stores

I've started taking some UXQA notes in this Notion doc

Can we remove the tdd-virtualized-inactive directory? Is that all old, unused code?

(Nit) There's a handful of places missing types which makes the code less approachable. Some I've run into:

Functions in the data-graphics directory: createSparkline, bisectData, createTimeFormat
Functions in features/pivot: mergeBlocks, merge2DBlocks
Props in TDDTable.svelte


Tweak attachments throughput driver to support Poisson distribution

Tweak the attachments throughput test driver by adding the ability to issue requests to create and delete NetworkAttachments following a Poisson distribution.
Before this PR requests can only be issued with a constant period.

The force push to 2fb4226 is a rewording of a README paragraph.

The force push to  b13cfb4 tweaks printing of attachments ops timing making it more informative in case of failures.
b13cfb4 piggybacks another change. Before, users could only add node label selectors to role.kos.example.com/workload=true, b13cfb4 makes role.kos.example.com/workload=true the default but gives users complete control over the label selector to use.


Simulation Time Resolution Notification

Addressing #69, his PR adds a notification when small resolution values are selected. I think we should not forbid these values, in case a user needs them, but instead give a warning, so that the user is aware of that risk.

An open problem is that all watchers fire, so that the tooltip is shown for fractions of seconds. This seems to be a bit more complicated to solve.

LGTM.


Predict protocol V2 proposal

What this PR does / why we need it:
Predict Protocol V2 proposal.
Release note:

NONE

This change is

/assign @ellis-bigelow

/ok-to-test

Love to see this land. Will wait for some lgtms but you have my /approve

Will follow up!
On Fri, Mar 6, 2020, 8:28 PM Animesh Singh<EMAIL_ADDRESS>wrote:

@animeshsingh commented on this pull request.
thanks @deadeyegoodwin https://github.com/deadeyegoodwin - excited
about this! Will do another pass.
In docs/predict-api/v2/required_api.md
https://github.com/kubeflow/kfserving/pull/709#discussion_r389226752:

@@ -0,0 +1,829 @@

+# Predict Protocol - Version 2

+This document proposes an predict/inference API independent of any
⬇ Suggested change
-This document proposes an predict/inference API independent of any
+This document proposes a predict/inference API independent of any

In docs/predict-api/v2/required_api.md
https://github.com/kubeflow/kfserving/pull/709#discussion_r389226793:

@@ -0,0 +1,829 @@

+# Predict Protocol - Version 2

+This document proposes an predict/inference API independent of any
+specific ML/DL framework and capable of supporting both easy-to-use
⬇ Suggested change
-specific ML/DL framework and capable of supporting both easy-to-use
+specific ML/DL framework and model servers. Its capable of supporting both easy-to-use

In docs/predict-api/v2/required_api.md
https://github.com/kubeflow/kfserving/pull/709#discussion_r389226817:

@@ -0,0 +1,829 @@

+# Predict Protocol - Version 2

+This document proposes an predict/inference API independent of any
+specific ML/DL framework and capable of supporting both easy-to-use
+and high-performance use cases. By implementing this protocol both
+inference clients and servers will increase their utility and
+portability by being able to operate seamlessly on platforms that have
+standardized around this API. This protocol is endorsed by NVIDIA
+TensorRT Inference Server, TensorFlow Serving, and ONNX Runtime
@rakelkar https://github.com/rakelkar can we get someone to approve
ONNX compliance?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/709?email_source=notifications&email_token=AA2CTBIVI65L5VOUITKNR4LRGHEQJA5CNFSM4K77ZXHKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCYMMFZY#pullrequestreview-370721511,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2CTBJHLMABCQTVL2R5FTDRGHEQJANCNFSM4K77ZXHA
.

@rakelkar @cliveseldon @animeshsingh, @yuzisun , would love to get this merged. Any complaints?

Given Nathan Yan (assuming from ONNX runtime team) has agreed to include ONNX there, I am fine - cc @rakelkar
/lgtm

That is correct!
/lgtm
On Mon, Mar 9, 2020 at 10:52 AM Animesh Singh<EMAIL_ADDRESS>wrote:

Given Nathan Yan (assuming from ONNX runtime team) has agreed to include
ONNX there, I am fine - cc @rakelkar https://github.com/rakelkar
/lgtm
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/709?email_source=notifications&email_token=AA2CTBMOB6CZSHAM5WGCENDRGU3HBA5CNFSM4K77ZXHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIKLWI#issuecomment-596682201,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2CTBLMOFM6MN7BHZNG773RGU3HBANCNFSM4K77ZXHA
.

--
Rakesh Kelkar

/approve!

@ellis-bigelow
github won't pick the excitement with the exclamation :-)
/approve


Jar unzip issues on MacOS Arm

Please provide a brief summary of the bug
We are getting occasional mangled classes on macos aarch64 .
An example: java.lang.ClassFormatError: Unknown constant tag 101 in class file com/semmle/cli2/generate/LogSummaryCommand but we get various different symptoms (mangled dependency names, verification issues, etc). The classes on disk are fine and work fine for future runs.
We think this is JDK-8282954.
The fix was supposedly backported here: JDK-8297637 but the adoptium builds are not bundling zlib on macos arm and I don't see the backport in the jdk repository.
It seems like the fix just changes default build arguments so it should be possible to fix this in the build scripts instead.
Please provide steps to reproduce where possible
I suspect the reproduction in JDK-8282954 would work?
Expected Results
Not crash.
Actual Results
It crashes on class loading.
What Java Version are you using?
openjdk version "17.0.7" 2023-04-18 OpenJDK Runtime Environment Temurin-17.0.7+7 (build 17.0.7+7) OpenJDK 64-Bit Server VM Temurin-17.0.7+7 (build 17.0.7+7, mixed mode)
What is your operating system and platform?
Mac OS arm.
How did you install Java?
Downloaded via the api from the website. Repacked with
jlink \
                      --output "new-jdk" \
                      --module-path jmods/ \
                      --compress=2 \
                      --strip-debug \
                      --add-modules java.management,java.instrument,java.naming,jdk.management.agent,jdk.zipfs,jdk.compiler,java.xml,java.sql,java.rmi,java.desktop,jdk.unsupported,jdk.crypto.ec

Did it work before?
It has never worked on macos aarch64.

Did you test with the latest update version?
We tested with the latest version of 17 (17.0.7)

Did you test with other Java versions?
We have had the same issue with all previous versions of 17

Relevant log output
No response

https://bugs.openjdk.org/browse/JDK-8297637 is an Oracle JDK only backport, we'll need to backport to OpenjDK

Backport to JDK17u-dev here: https://github.com/openjdk/jdk17u-dev/pull/1609


SCF property validation

Description

[X] Fixes #17

Todos
Notable points that this PR has either accomplished or will accomplish.

[x] Fixes typos in the factories and datasets
[x] Adds validators for scf properties
[x] Adds submission tests that check scf properties are valid.
[ ] Add a test that trys to extract one of the requested scf properties.

Questions

[ ] Is there a better way than having a predefined list of scf properties that will have to be updated for validation?

Status

[ ] Ready to go

Codecov Report

Merging #18 into master will increase coverage by 4.57%.
The diff coverage is n/a.


[Podman] Older version of kernel not available which used for mac workaround

We used old version of kernel as workaround https://github.com/crc-org/vfkit/issues/11 , b13c0e6 one but looks the older version of kernel is now not available which is used for amd64 ( removed from koji) and failing to create the bundles for amd64.
${SSH} core@${vm_ip} "curl -L -O https://kojipkgs.fedoraproject.org/packages/kernel/5.19.17/300.fc37/x86_64/kernel-5.19.17-300.fc37.x86_64.rpm -L -O https://kojipkgs.fedoraproject.org/packages/kernel/5.19.17/300.fc37/x86_64/kernel-core-5.19.17-300.fc37.x86_64.rpm -L -O https://kojipkgs.fedoraproject.org/packages/kernel/5.19.17/300.fc37/x86_64/kernel-modules-5.19.17-300.fc37.x86_64.rpm"
We might need to using https://koji.fedoraproject.org/koji/buildinfo?buildID=2079160 one.

This is also fixed by using older kernel from koji.


Cody Gray: Watch lootahperfumes.ae

Cody Gray requests the watch of the watch_keyword lootahperfumes\.ae. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

lootahperfumes\.ae has been seen in 1 true positive, 1 false positive, and 0 NAAs.

Approved by tripleee in Charcoal HQ