[Bug]: Remix Dev Server Process not terminated once Remix Application is set up with Prisma

Which Remix packages are impacted?

[ ] remix (Remix core)
[ ] create-remix
[ ] @remix-run/architect
[ ] @remix-run/cloudflare-workers
[ ] @remix-run/dev
[ ] @remix-run/express
[ ] @remix-run/netlify
[ ] @remix-run/node
[ ] @remix-run/react
[ ] @remix-run/serve
[ ] @remix-run/server-runtime
[ ] @remix-run/vercel

What version of Remix are you using?
1.0.6
Steps to Reproduce
Steps to introduce the issue
The following steps were done on two separate windows devices. I have not attempted this on a Mac device (yet).

Create a new Remix project using npx create-remix@latest
Follow the steps in the Jokes tutorial to Set up Prisma and Connect to the database.
Load some data in one of the existing project pages so that when you run npm run dev we establish a connection with the Prisma DB

# inside app\routes\demos\about.tsx
...
export let links: LinksFunction = () => {
  return [{ rel: "stylesheet", href: stylesUrl }];
};

+export let loader: LoaderFunction = async (): Promise<Joke[]> => {
+  return await db.joke.findMany();
+}

export default function Index() {
+  const jokes = useLoaderData<Joke[]>();

  return (

About Us

+        {
+          jokes.map(j =>
{j.name}
)
+        }

          Ok, so this page isn't really about us, but we did want to
          show you a few more things Remix can do.

...

Run npm run dev and navigate to http://localhost:3000/demos/about so we load the data from Prisma and see our Joke names.
Execute Ctrl + C to terminate the dev server (scenario: say you needed to install an NPM package or something)
Attempt to run npm run dev again and you receive the following error:

PS D:\tmp\remix-repro> npm run dev

> dev
> remix dev

Watching Remix app in development mode...
events.js:292
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE: address already in use :::8002
    at Server.setupListenHandle [as _listen2] (net.js:1318:16)
    at listenInCluster (net.js:1366:12)
    at Server.listen (net.js:1452:7)
    at new WebSocketServer (D:\tmp\remix-repro\node_modules\ws\lib\websocket-server.js:90:20)
    at watch (D:\tmp\remix-repro\node_modules\@remix-run\dev\cli\commands.js:89:13)
    at Object.dev (D:\tmp\remix-repro\node_modules\@remix-run\dev\cli\commands.js:182:11)
Emitted 'error' event on WebSocketServer instance at:
    at Server.emit (events.js:315:20)
    at emitErrorNT (net.js:1345:8)
    at processTicksAndRejections (internal/process/task_queues.js:80:21) {
  code: 'EADDRINUSE',
  errno: -4091,
  syscall: 'listen',
  address: '::',
  port: 8002
}

Steps the "resolve" the issue
I'm putting this here because it's helped me to narrow it down to be something related to initiating the DB connection.
7. (Continuing where we left off) The Remix Dev Server is still running on port 8002 so we need to terminate it. Since I'm on Windows, I do this with Get-Process -Id (Get-NetTCPConnection -LocalPort 8002).OwningProcess | Stop-Process
8. Comment out the code we added to app\routes\demos\about.tsx so that none of the files in our Remix project establish a connection with the DB.
9. Run npm run dev to re-start the dev server. Run Ctrl + C to terminate the dev server.
10. Run npm run dev one more time. This time, you won't get the  Error: listen EADDRINUSE: address already in use :::8002 error from above.
Expected Behavior
Once I've reached the part of the Jokes App Tutorial where we've set up Prisma and have started interacting with the Data for the tutorial, I expect to be able to run npm run dev, terminate that process with Ctrl + C when I need to do something like install an NPM package, and then get back to work with npm run dev.
Actual Behavior
As described in the Steps to Reproduce, once I've set up the starter Remix project with Prisma and added a simple loader to display data from the DB in the UI, the Remix Dev Server does not terminate upon Ctrl + C. As a result, when I try to run npm run dev in the same terminal/PowerShell process, the Dev Server is unable to start since the old process is still running and am hit with the following error (pasting again for convenience):
PS D:\tmp\remix-repro> npm run dev

> dev
> remix dev

Watching Remix app in development mode...
events.js:292
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE: address already in use :::8002
    at Server.setupListenHandle [as _listen2] (net.js:1318:16)
    at listenInCluster (net.js:1366:12)
    at Server.listen (net.js:1452:7)
    at new WebSocketServer (D:\tmp\remix-repro\node_modules\ws\lib\websocket-server.js:90:20)
    at watch (D:\tmp\remix-repro\node_modules\@remix-run\dev\cli\commands.js:89:13)
    at Object.dev (D:\tmp\remix-repro\node_modules\@remix-run\dev\cli\commands.js:182:11)
Emitted 'error' event on WebSocketServer instance at:
    at Server.emit (events.js:315:20)
    at emitErrorNT (net.js:1345:8)
    at processTicksAndRejections (internal/process/task_queues.js:80:21) {
  code: 'EADDRINUSE',
  errno: -4091,
  syscall: 'listen',
  address: '::',
  port: 8002
}

I've been getting around this by mapping Get-Process -Id (Get-NetTCPConnection -LocalPort 8002).OwningProcess | Stop-Process to a PowerShell Alias. Before I was having to start a new instance of Windows Terminal every time, which was getting inconvenient.
One user in the Remix Discord has mentioned running into this issue on their Macbook: "just got the same error as you, I am on a MB Pro M1 with macOS 12.0.1, within VS Code terminal integration"

Can confirm, got the same issue on MBP (Intel), as a quick fix it's possible on linux based OS to use ps to list process and manually kill the remix process still running.

Fixed in #752


Trim test output

When running tests, we should trim the output so that:

mocha only shows failed tests
c8 only shows coverage < 100%

The latter can be done by adding "skip-full": true to .c8rc.json. We need to find how to achieve a similar result for mocha.

Can I work on this?

Please do!


Wow, I love your step delay! Could you break out some CV controls for it pls!!

Hi, the subject says it all. IMHO your  Step Delay is an overlooked gem.
What would make it even more awesome would obviously cv inputs for controlling Delay, Level, Pan, Feedback and even (if poss) some way to toggle  between Staight/Dotted/Triple delays with trigs or gates.
Would that be feasible? Atm I'm stuck with fading between multiple instances with different settings if I want to automate changes in the delay type.
Cheers & thanks!

I keep coming back to this and wondering how would be a good way of doing this...  Is the new CV Mapping capability in v1 help?


valid_url does not work correctly

Hello!
Url validation doesn't work, if I pass a string like this asdf it passes the validation which should be wrong
my validation is as follows:
'rules' => 'permit_empty|valid_url|max_length[512]'

Hugs!

Tell us what is the expected result bro.
From my point of view the rule works correctly
asdf can be a valid url if you use localhost with some proxy or what not.
Just like localhost is a valid url, asdf is the same.
If you want to prepend the HTTP or HTTPS just use prep_url on the input.

Not a bug, sorry - asdf is a valid hostname.
Please take a look at RFC 2396 - these are the rules behind FILTER_VALIDATE_URL.
But we're open to adding a new rule that will be more "restrictive".

I'm reopening this issue because I think there are some things that need to be addressed.

RFC 2396 is obsolete. The new doc is RFC 3986
The spec differentiates between "URI" (Identifier) and "URL" (Location). A URL:

The  term "Uniform Resource Locator" (URL) refers to the subset of URIs that, in addition to identifying a resource, provide a means of locating the resource by describing its primary access mechanism (e.g., its network "location").

The section "Syntax Components" describes the parts of the URI. Of note, it requires a scheme (e.g. "https") but not a path:

The scheme and path components are required, though the path may be empty (no characters)

Our current validation rule very clearly allows the opposite in order to validate "common" URLs like "CodeIgniter.com". See also the validation rules tests for some wonky passes (https://github.com/codeigniter4/CodeIgniter4/blob/d9864e33758aca9aca9dc44c121e402e2d51136f/tests/system/Validation/FormatRulesTest.php#L88)

Here's my opinion:
a) We need valid_url_strict or something that has a lower tolerance for actual URLs.
b) I think the current validator should not pass simple strings like "asdf", but in order to prevent a breaking change we should still accept "CodeIgniter.com". Probably could check for a scheme and pass to a) if found, use a simple regex otherwise.
Just to note, Laravel and Symfony use a super convoluted regex to check validity (https://github.com/laravel/framework/blob/cb398188729f51e63d6fd0dc1ed9009fcf76b404/src/Illuminate/Validation/Concerns/ValidatesAttributes.php#L1732).

One more comparison, the list of Symfony's invalid URLs test:
            ['example.com'],
            ['://example.com'],
            ['http ://example.com'],
            ['http:/example.com'],
            ['http://example.com::aa'],
            ['http://example.com:aa'],
            ['ftp://example.fr'],
            ['faked://example.fr'],
            ['http://<IP_ADDRESS>:aa/'],
            ['ftp://[::1]/'],
            ['http://[::1'],
            ['http://hello.☎/'],
<EMAIL_ADDRESS>            ['http://:password@@symfony.com'],
            ['http://username:passwordsymfony.com'],
            ['http://usern<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>            ['http://example.com/exploit.html?<script>alert(1);</script>'],
            ['http://example.com/exploit.html?hel lo'],
            ['http://example.com/exploit.html?not_a%hex'],
            ['http://'],

I only disagree with point b) - resolving addresses to http is default behavior. If we use a web browser or wget it will work. The same situation is here. If we agree that codeigniter.com is a valid address then asdf has to be valid as well.

I get that it is a slippery slope, but I think we are dealing with expectations here. I ended up on this issue from a very user-based perspective: I could not get my tests to work because “foobar” was passing as a URL. Having a validation rule that passes almost any string of consecutive characters weakens it. In order to get my tests working I had to track down the actual rule code and determine that a space would invalidate it (“foo bar”) which we should be expecting of devs.
I could see explicit exceptions for common strings one might use (like “localhost”) but I think the current rule is neither RFC-accurate nor helpful.

Valid point about user expectations, but IMO these expectations should not lead to the inconsistencies that are impossible to defend in the technical discussion. And I'm referring just to omitting the protocol.

Understood. I’m not very invested in it either way - personally I think valid_url is rather useless and (when it is written) valid_url_strict will cover my use-cases. I think as long as the docs are clear about what the code actually does - “Fails if field does not contain a valid URL” is both wrong and misleading.
On the topic, I’ve been wondering if validation should be added to the URI class, since it appears to be more complicated than just a regex or PHP command. valid_url_strict could then leverage URI::validate().

inconsistencies that are impossible to defend in the technical discussion

How about this as a rubric: valid_url passes anything you can type into a browser location bar (not including search results). This pretty much encapsulates the desire to support "shorthand" URLs (like CodeIgniter.com) while helping mitigate failed developer expectations (like if a user types their username in the wrong box, suddenly "michalsn" is a valid URL).

I think I would only mention that if the protocol part in the address is omitted it will be added automatically before doing a check. This is just more accurate and nobody will have doubts about how things are working.
About URI::validate() - it's a good idea. If I remember it correctly we're using parse_url() in URI class.

On seriously malformed URLs, parse_url() may return FALSE.

I wonder how many false-positive results we can get.

From my perspective first when i've installed this codeigniter 4 i used this rule, and put asdf myself and found out that was a valid url pissed me off a little, after complaining on this issue here on github, someone closed the topic said it was valid. It made good sens for a while and used parse_url to fix the problem even tough the url wasn`t valid we can make it valid with parse_url function.
After testing with some browsers, i found out that if you type asdf in the url, it will go to the default search automatically type this asdf inside the search text box.
Weird how the web evolved... even if we type mistakes it will get automatically fixed.
However they must be valid urls but i think you have to put minimum at least 2 characters to be valid url no ?

In my opinion. valid_url should only allow URL to pass inspection. Such asdf , localhost, codeigniter.com string, it not a URL. It 's domain.

it is always a good practice to use prep_url to be sure it will be a valid url, even if you type asdf the prep_url will prepend http:// to it.

Hello!
Url validation doesn't work, if I pass a string like this asdf it passes the validation which should be wrong
my validation is as follows:
'rules' => 'permit_empty|valid_url|max_length[512]'

Hugs!

asdf not an url

I sent another PR: #5268


⚠ Samsung has degraded performance

In 387d5b9, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2977 ms

Resolved: Samsung performance has improved in 06b4c12.


Make notification message global

As part of this PR, a custom toast hook and global component are included for displaying notification messages based on toast

 2.0 Make Loading & Error Notification Global


include data absent reasons in checks for must support elements

https://github.com/onc-healthit/onc-certification-g10-test-kit/issues/182

I'm still testing this to see if it resolves the issue

This got the tests passing again for our server.

I can add test coverage for this if yall agree this is the right thing to do

See the linked ticket for more detailed discussion on this. We've decided to make an update to our fhir server instead of merging this


API call to list HIL active extensions

Addressing #529:
API call returns all active extensions.
CLI call accepts a string to narrow the results to a certain extension or group of extensions.
For example, hil list_active_extensions mock will return only the mock extensions currently loaded.
hil list_active_extensions all will return all active extensions.

Addressed @naved001's issues and added some tests.

I know it sounds redundant but we should directly test the API too in tests/unit/api/main/py.

Ack, I put my comments on a single commit diff, not the final thing, so they're already showing up as outdated. The meat of those comments is still relevant to the current version though; please address them.

Everything should be addressed now.

@SahilTikale, I think we're just waiting on you to approve the changes; I think your comments have been addressed.

@SahilTikale ping

+1 merging.


refactor: use-template-literals: reuse isStringLiteral

📚 Description

Reused isStringLiteral
🔗 Linked Issue

🧪 Test Plan

📄 Documentation to Update

 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.

@coderaiser can you please sign the CLA so that merging is unblocked?

a true codemod champ! @coderaiser ! THANK YOU.

@mirai2k linting ci/cd is failing here. Can you please check. If it's a false positive related to env vars blocking building let me know so I can merge.

@mohab-sameh this is looks good, im pretty sure that you can merge it


$ionicPopup  click an item in popup template

template code:

controller code:       $scope.shop.add= function() {	 alert(123);	 }
the method can not work, "ng-bind=shop.des" binds normal,however the ng-click not work,  is this a bug ?or ionic does not support method in $ionicPopup .

Greetings @Jaware!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

You should use a button element instead. I'd also recommend asking on the forum.

thanks


⚠ Bofrost.be has degraded performance

In 593f0bd, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1073 ms

Resolved: Bofrost.be performance has improved in c706037 after 6 minutes.


⚠ Bofrost.lu has degraded performance

In 4774a71, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1336 ms

Resolved: Bofrost.lu performance has improved in 8f9f023.


NA OFI: support vector rma operations

Is your feature request related to a problem? Please describe.
The na_ofi.c already uses libfabric API calls that support vectors for local and remote buffers (fi_writemsg() and fi_readmsg()), but it hard codes the vector lengths to 1 (https://github.com/mercury-hpc/mercury/blob/06bf87e4403dd1e487b53204569e05296e6a5061/src/na/na_ofi.c#L4171).
Describe the solution you'd like
It would be better if it translated vectors from the bulk handle into native vectors for libfabric, as is done in the na_sm.c code here: https://github.com/mercury-hpc/mercury/blob/06bf87e4403dd1e487b53204569e05296e6a5061/src/na/na_sm.c#L3733
We need to confirm if the relevant libfabric providers actually support this capability; if not we will need to retain a fallback path for providers that don't (and add that to the flags checked in the NA_OFI_PROV_TYPES so we can quickly toggle if offending providers gain support later).
Additional context
From a quick scan of libfabric it looks like gni and psm2 likely support this.  Verbs may not but we need to confirm.  See numerous FI_E2BIG return codes in the verbs provider, but I'm not sure what ocde paths are important here when using rxm.

For some more context, here is code in MPICH that is issuing vector libfabric operations when needed:
https://github.com/pmodels/mpich/blob/master/src/mpid/ch4/netmod/ofi/ofi_rma.h#L514
I don't see guards around the provider type there, though it's possible that I'm missing something.

Yes I had looked at that in the past but it was not supported. See https://github.com/ofiwg/libfabric/issues/3062 We should have a look at it again and maybe have a query call to fall back for providers that do not support it.

I didn't think to check the memory registration path; I just looked at what some of the codes are doing on the actual read/write RMA path.  I guess we'll be testing both with this change :)

MPICH appears to be setting msg.desc to NULL:
https://github.com/pmodels/mpich/blob/master/src/mpid/ch4/netmod/ofi/ofi_rma.h#L480
I don't know the implications, just pointing that out.

I ever considered it long time ago. The vector rma should be supported, but OFI providers with limit setting for the local iov_count and remote rma_iov_count (fi_info::tx_attr::iov_limit/::rma_iov_limit), and different provider can with different limit number can be queried by fi_getinfo(). Just FYI.

@carns All providers set mr_iov_limit and rma_iov_limit to 1, I am still doing some more testing and we'll support it internally now but I would keep hopes very low at this point :\

@carns I was looking in more details at the mpich code that you had sent. I think it's still limited though since they also always set iov_count and rma_iov_count to 1.

Ah, Ok.  Thanks for the updates.

Supporting code was added as part of https://github.com/mercury-hpc/mercury/commit/a6bbce8514011e21e5ef1b37283a17606a0d39cb but there are issues remaining with libfabric that will need to be further investigated.

What are the "issues remaning with libfabric" ?   Phil pointed me to this post https://lists.openfabrics.org/pipermail/libfabric-users/2021-June/000861.html documenting the FI_VERBS_TX_IOV_LIMIT and FI_VERBS_RX_IOV_LIMIT environment variables, and the ibv_devinfo -v utility.  on summit, it looks like we should set those environment variables to 30
But no impact on the vectored transfer benchmark:
Before:
<op>    <warmup_seconds>        <concurrency>   <threads>       <xfer_size>     <vector_count>  <total_bytes>   <seconds>       <MiB/s> <xfer_memory>   <align_buffer>
PULL    1       1       0       524288  1<PHONE_NUMBER>88    20.000072       9976.864320     0
PULL    1       1       0       524288  2<PHONE_NUMBER>28    20.000095       9983.227515     0
PULL    1       1       0       524288  4<PHONE_NUMBER>60    20.000066       10028.592124    0
PULL    1       1       0       524288  8<PHONE_NUMBER>00    20.000078       10029.335904    0
PULL    1       1       0       524288  16<PHONE_NUMBER>08    20.000072       9540.365654     0
PULL    1       1       0       524288  32<PHONE_NUMBER>44    20.000097       9196.530270     0
PULL    1       1       0       524288  64<PHONE_NUMBER>04    20.000236       5137.639306     0
PULL    1       1       0       524288<PHONE_NUMBER>7120     20.000313       2562.209957     0
PULL    1       1       0       524288<PHONE_NUMBER>5568     20.000662       1295.232153     0

After
PULL    1       1       0       524288  2<PHONE_NUMBER>36    20.000096       9691.128674     0
PULL    1       1       0       524288  4<PHONE_NUMBER>24    20.000068       9669.291918     0
PULL    1       1       0       524288  8<PHONE_NUMBER>60    20.000085       9661.708882     0
PULL    1       1       0       524288  16<PHONE_NUMBER>88    20.000131       9234.339565     0
PULL    1       1       0       524288  32<PHONE_NUMBER>60    20.000126       8857.944351     0
PULL    1       1       0       524288  64<PHONE_NUMBER>00    20.000202       5377.445768     0
PULL    1       1       0       524288<PHONE_NUMBER>5760     20.000352       2703.577398     0
PULL    1       1       0       524288<PHONE_NUMBER>1168     20.000367       1340.875384     0

I would need to look at it again but I had added support for it, you can turn it on by uncommenting that line: https://github.com/mercury-hpc/mercury/blob/master/src/na/na_ofi.c#L196
I think if I recall correctly there were some issues with OFI not transferring things correctly. I don't think I have tried those environment variables though.

No change uncommenting that define:
PULL    1       1       0       524288  1<PHONE_NUMBER>04    20.000071       8985.793186     0
PULL    1       1       0       524288  2<PHONE_NUMBER>04    20.000109       9052.650574     0
PULL    1       1       0       524288  4<PHONE_NUMBER>92    20.000094       9156.432103     0
PULL    1       1       0       524288  8<PHONE_NUMBER>96    20.000107       9163.500843     0
PULL    1       1       0       524288  16<PHONE_NUMBER>48    20.000067       8754.245780     0
PULL    1       1       0       524288  32<PHONE_NUMBER>24    20.000155       8503.634210     0
PULL    1       1       0       524288  64<PHONE_NUMBER>24    20.000248       5150.511207     0
PULL    1       1       0       524288<PHONE_NUMBER>9808     20.000225       2607.245722     0
PULL    1       1       0       524288<PHONE_NUMBER>1104     20.000391       1277.675006     0

Here's a graph of the three cases, just to make it more clear:

a-ha!  I stuck some debugging into hg_bulk_create:  it tells me
==== bulk:  count: 64 max_segments: 1

fi_info -v gives me this:
        domain: 0x0
        name: mlx5_1
        threading: FI_THREAD_SAFE
        control_progress: FI_PROGRESS_AUTO
        data_progress: FI_PROGRESS_AUTO
        resource_mgmt: FI_RM_ENABLED
        av_type: FI_AV_UNSPEC
        mr_mode: [ FI_MR_LOCAL, FI_MR_VIRT_ADDR, FI_MR_ALLOCATED, FI_MR_PROV_KEY ]
        mr_key_size: 4
        cq_data_size: 0
        cq_cnt: 16777216
        ep_cnt: 262144
        tx_ctx_cnt: 1024
        rx_ctx_cnt: 1024
        max_ep_tx_ctx: 1024
        max_ep_rx_ctx: 1024
        max_ep_stx_ctx: 0
        max_ep_srx_ctx: 8388608
        cntr_cnt: 0
        mr_iov_limit: 1

that mr_iov_limit is what i need to adjust: mercury consults that field :
priv->iov_max = priv->domain->fi_prov->domain_attr->mr_iov_limit;

setting the env changes the iov_limit but the mr_iov_limit is hard coded to 1  (see fi_verbs.h #define VERBS_MR_IOV_LIMIT )
Dang, I really thought I had something by changing that define in libfabric, recompiling, and trying again, however mercury still tells me max_segments is 1.    fi_info tells me mr_iov_limit is now 30... so why is mercury insisting it is 1?

Ah ok from what I see it looks like it's because of the rxm provider. When you're running fi_info, are you specifying verbs;ofi_rxm ? or only verbs?

I'm not calling the fi_info() routine.  I'm calling the fi_info utility, which I think by default dumps all the providers it can find?
The rxm angle is interesting, but I don't see "1" in there.  I do see
prov/rxm/src/rxm.h:#define RXM_IOV_LIMIT 4

so I'd expect a "sticky" 4, but I don't know where the 1 comes from.

I figured "what the heck" and hard-coded 30 in mercury's na_ofi_initialize:
diff --git a/src/na/na_ofi.c b/src/na/na_ofi.c
index f3e6e47d..1152f26b 100644
--- a/src/na/na_ofi.c
+++ b/src/na/na_ofi.c
@@ -3928,7 +3928,9 @@ na_ofi_initialize(na_class_t *na_class, const struct na_info *na_info,
 #endif

     /* Cache IOV max */
-    priv->iov_max = priv->domain->fi_prov->domain_attr->mr_iov_limit;
+    //XXX: giant hack.  set to 30 and see what happens
+    //priv->iov_max = priv->domain->fi_prov->domain_attr->mr_iov_limit;
+    priv->iov_max = 30;

     /* Create endpoint */
     ret = na_ofi_endpoint_open(priv->domain, node_ptr, src_addr, src_addrlen,

Now vector lengths of 1 are ok, and 32 and higer are ok, but vector lengths of 2-16 give me HG_INVALID_ARG

@roblatham00 I don't think I was implying that you were calling the fi_info() routine :) there are options that you can pass to the fi_info utility and one of them is -p which lets you specify which provider you're querying. I was suggesting that you pass verbs\;ofi_rxm since otherwise yes it would just dump everything. You're also confusing the MR iov limit and the regular iov limit I think. The MR IOV limit is the maximum number of buffers that we can pass to memory registration calls at once, i.e. the maximum number of buffers that may be associated with a single memory region. This is basically what happens when you make a call to HG_Bulk_create() with N buffers, we register these N buffers if possible so that they appear as a single virtual region. The regular IOV limit on the other hand defines the maximum number of buffers at once that are passed to fi_writev() or fi_readv() operations. We currently do not have a mode in mercury that allows for passing multiple buffers at once when doing a bulk transfer if they are not part of the same virtual region. I believe I would need to add another query call then / possibly additional NA callbacks (current NA put/get take only a single MR region) and pass that array of buffers and an array of memory regions. Let me look into this.

Thanks for the libfabric information.  You are 100% correct that I am confused by libfabric domains, providers, etc.
Let me be more concrete:
I'm using Phil's benchmark to measure this:   https://github.com/mochi-hpc-experiments/mochi-tests/blob/main/perf-regression/margo-p2p-vector.c#L243
margo_bulk_create immediately calls HG_Bulk_create():     This is a single virtual region, right?  We are just pointing to different parts of it?

Right, that code is creating a single registration to represent a vector of memory regions.  That entire registration (as a whole, including all vectors) is then transmitted with a bulk transfer operation.
Going by @soumagne ;s explanation, that means that this benchmark is reliant on the mr_iov_limit rather than the iov_limit (thanks for the explanation on that- I was wondering what the exact difference was).
Mercury doesn't have an API to pick out pieces with a contiguous memory registration, or for that matter to construct a vector from different registrations.  That could be a neat capability, though.  MPICH for example uses this to stitch pre-registered header structs onto data payloads.  You could also imagine it being used for 2-phase style aggregation, where one party doesn't know anything except the aggregate size, while the other parties fill in specific portions using vector operations.  Right now we couldn't use vectors for that.


Configuration for rule "@typescript-eslint/strict-boolean-expressions" is invalid

What version of this package are you using?
18.0.2
What operating system, Node.js, and npm version?

MacOS Catalina 10.15.5
Node v10.16.0
NPM v6.9.0

What happened?
Eslint exists with an error
Oops! Something went wrong! :(

ESLint: 7.2.0

Error: package.json » eslint-config-standard-with-typescript#overrides[0]:
        Configuration for rule "@typescript-eslint/strict-boolean-expressions" is invalid:
        Value {"allowString":false,"allowNumber":false,"allowNullableObject":false,"allowNullableBoolean":false,"allowNullableString":false,"allowNullableNumber":false,"allowAny":false} should NOT have additional properties.

What did you expect to happen?
ESLint to work as expected without throwing errors.
Steps to reproduce

create-react-app --use-npm --typescript eslint-ts-test
npm install --save-dev eslint@7 eslint-plugin-standard@4 eslint-plugin-promise@4 eslint-plugin-import@2 eslint-plugin-node@11 @typescript-eslint/eslint-plugin@2 eslint-config-standard-with-typescript
Update eslintConfig section of package.json

"eslintConfig": {
  "extends": "standard-with-typescript",
  "parserOptions": {
    "project": "./tsconfig.json"
  }
}

Are you willing to submit a pull request to fix this bug?
I'd love to, but this is totally new to me.
Also, thanks a lot for the project!

Thre is a similar issue which is closed by submitter https://github.com/standard/eslint-config-standard-with-typescript/issues/348
I rechecked my dependencies, here are my versions of packages
"@typescript-eslint/eslint-plugin": "^2.34.0",
"eslint": "^7.2.0",
"eslint-config-standard-with-typescript": "^18.0.2",
"eslint-plugin-import": "^2.21.1",
"eslint-plugin-node": "^11.1.0",
"eslint-plugin-promise": "^4.2.1",
"eslint-plugin-standard": "^4.0.1"

For me it was fixed by setting the newer @typescript-eslint/eslint-plugin in package.json
"@typescript-eslint/eslint-plugin": "^3.1.0",

$ npm ls should warn you that the peer dependency @typescript-eslint/eslint-plugin is not satisfied. Upgrade to v3 of that.


Publish src folder

This adds "src" to the set of published files, which allows source mapped URLs to resolve for consumers of this package. Resolves #265.
If the download size of the package is what we want to optimize for, an alternative approach would be to remove source map URL comments from the production build of this package.

This is a pretty huge issue honestly. It makes it really hard to see real errors in the build using React scripts 5. Could this be merged?

Any updates on this? Really hoping to get a fix to this since it clutters the whole screen with bunch of warnings.

I would also be really happy to have it merged or otherwise solved! Can't see much else on the console with these 28 warnings.

+1 for getting this merged, it's throwing a lot of warnings on react-scripts^5

same here, getting  a lot of errors on after migrating to react-script-5,
would love a solid merge :)

This was blocking enough for me that I decided to publish myself in the meantime
npm -i @kokarn/react-zoom-pan-pinch is pretty much exactly this PR, only removed a few things that wasn't needed to in the package to reduce bundle size

The weird thing is, i edited package.json locally butit did not seem to solve the problem

The weird thing is, i edited package.json locally but it did not seem to solve the problem

If you only edit the package.json nothing will change in regards to using my temporary package.

oh i guess i also have to build the project in order of the package.json change to take effect

oh i guess i also have to build the project in order of the package.json change to take effect,
btw, is this project still maintained? I dont see any late commits...

You have to run install and build, yes

i tried to build but got rolllup related errors

I can't seem to find the repo @kokarn , what did you remove?

I can't seem to find the repo @kokarn , what did you remove?

I set package.json files to this and did a build
  "files": [
    "dist",
    "src/components",
    "src/constants",
    "src/core",
    "src/hooks",
    "src/utils"
  ],

i still have the 30 warning like this one
WARNING in ./node_modules/react-zoom-pan-pinch/dist/index.esm.js Module Warning (from ./node_modules/source-map-loader/dist/cjs.js): Failed to parse source map from '/.../node_modules/react-zoom-pan-pinch/src/utils/styles.utils.ts' file: Error: ENOENT: no such file or directory, open '/.../node_modules/react-zoom-pan-pinch/src/utils/styles.utils.ts'
even after installing @kokarn/react-zoom-pan-pinch
any fix is comming ? thanks

even after installing @kokarn/react-zoom-pan-pinch

what version?

even after installing @kokarn/react-zoom-pan-pinch

what version?

@kokarn/react-zoom-pan-pinch @ 2.1.15
it work now thanks :)
I removed yarn cache completly and run yarn install a second time. first time i just remove react-zoom-pan-pinch and installed @kokarn/react-zoom-pan-pinch

@prc5 Can we please merge this?

The src folder is included in the fork @pronestor/react-zoom-pan-pinch that I created.

Can we merge this?

When this will be released?

@ViktorSoroka07 this weekend 🙏

Thank you @prc5 🙏

I'm seeing this issue with ^3.0.2, looks like it's broken again?

@robertwilkinson-dev Yes, someone already has created a new ticket for this: https://github.com/prc5/react-zoom-pan-pinch/issues/356


Re-indent Python Code Blocks in Integration tests

The latest toil branch is no longer compatible with the integration tests. I think this is because it now applies a fixed deindent to code blocks.
Ex:
        python <<CODE
        python code here
        CODE

becomes
python <<CODE
python code here
CODE

But since it forces the deindent, if the code block is already deindented/not indented enough:
        python <<CODE
python code here
CODE

This will cause an indentation error when parsing the python.
For example, this:
        python <<CODE
def func():
    for loop:
        print hello

will become:
        python <<CODE
def func():
    for loop:
print hello

I'm not sure if this is part of the WDL spec, but both MiniWDL and Cromwell can deal with the case when the indentation level is less than the level at the code block declaration.
This also updates some of the images to be compatible with singularity v4 and higher.

I think Toil might be wrong here according to the spec. But it doesn't hurt to make the tests for other things robust.


WIP: NETOBSERV-1625: Add ebpf altering for flows drop

Description
enable ebpf agent metrics by default so it can be used to generate alerts with flow table is full
Dependencies
n/a
Checklist
If you are not familiar with our processes or don't know what to answer in the list below, let us know in a comment: the maintainers will take care of that.

[ ] Is this PR backed with a JIRA ticket? If so, make sure it is written as a title prefix (in general, PRs affecting the NetObserv/Network Observability product should be backed with a JIRA ticket - especially if they bring user facing changes).
[ ] Does this PR require product documentation?

[ ] If so, make sure the JIRA epic is labelled with "documentation" and provides a description relevant for doc writers, such as use cases or scenarios. Any required step to activate or configure the feature should be documented there, such as new CRD knobs.

[ ] Does this PR require a product release notes entry?

[ ] If so, fill in "Release Note Text" in the JIRA.

[ ] Is there anything else the QE team should know before testing? E.g: configuration changes, environment setup, etc.

[ ] If so, make sure it is described in the JIRA ticket.

QE requirements (check 1 from the list):

[ ] Standard QE validation, with pre-merge tests unless stated otherwise.
[ ] Regression tests only (e.g. refactoring with no user-facing change).
[ ] No QE (e.g. trivial change with high reviewer's confidence, or per agreement with the QE team).

$ oc get prometheusrules -n netobserv-privileged ebpf-agent-prom-alert -o yaml
apiVersion: monitoring.coreos.com/v1
kind: PrometheusRule
metadata:
  creationTimestamp: "2024-04-29T23:53:40Z"
  generation: 1
  labels:
    app: netobserv-ebpf-agent
  name: ebpf-agent-prom-alert
  namespace: netobserv-privileged
  ownerReferences:
  - apiVersion: flows.netobserv.io/v1beta2
    blockOwnerDeletion: true
    controller: true
    kind: FlowCollector
    name: cluster
    uid: 7376e577-a0d4-4ffd-a83d-cacc0199e14e
  resourceVersion: "11150"
  uid: 51e833bc-aef1-4f9c-8210-4eea4500acd9
spec:
  groups:
  - name: NetobservEBPFAgentAlerts
    rules:
    - alert: NetObservNoFlows
      annotations:
        description: NetObserv eBPF agent hashmap table is full, it means that the
          eBPF agent is not able to process new flows. Please consider to increase
          the hashmap table size.
        summary: NetObserv eBPF is not able to process any new flows
      expr: sum(rate(netobserv_agent_dropped_flows_total[1m])) == 0
      for: 10m
      labels:
        app: netobserv
        severity: warning

/ok-to-test

/retest

/retest

/ok-to-test

emulate hashmap table full alert

set sampling to 1
reduce hashmap table size to 100
enable ebpf agent metric
use hey-ho tool to generate traffic ./hey-ho.sh -r 5 -d 3 -z 10m -n 4 -q 2 -p -b

Codecov Report
Attention: Patch coverage is 76.78571% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 66.42%. Comparing base (216e3e6) to head (cf75685).

Files
Patch %
Lines

controllers/ebpf/agent-metrics.go
88.63%
3 Missing and 2 partials :warning:

...pis/flowcollector/v1beta1/zz_generated.deepcopy.go
0.00%
4 Missing :warning:

...pis/flowcollector/v1beta2/zz_generated.deepcopy.go
0.00%
3 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #632      +/-   ##
==========================================
+ Coverage   66.34%   66.42%   +0.07%
==========================================
  Files          67       67
  Lines        7527     7583      +56
==========================================
+ Hits         4994     5037      +43
- Misses       2163     2173      +10
- Partials      370      373       +3

Flag
Coverage Δ

unittests
66.42% <76.78%> (+0.07%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

@msherif1234 I deployed the operator bundle included in catalog image , however I don't see the prometheus rule being configured in netobserv-privileged NS, am I missing something?
$ oc get prometheusrules -A  | egrep netobserv
netobserv                                    flowlogs-pipeline-alert                        78s

oh wait, I had ebpf metrics disabled, enabling it now.


Answered all assignment_1.ipynb questions

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
added the code to answer the assignment_1.ipynb
What did you learn from the changes you have made?
creating conditions, changing to lower-case, sorting
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
thinking of another way to make the code shorter
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
trying to remember all the proper syntax of different commands
How were these changes tested?
I used the checks given and I tried different scenarios myself too
A reference to a related issue in your repository (if applicable)
N/A
Checklist

[x] I can confirm that my changes are working as intended

Please replace all of your print() statement with return

Hi Emma @dwz92, I've changed it now. Thank you very much.


added Issuer to Okta settings in appsettings.json

When running this locally I received a runtime error during validation of OpenID Connect settings, and resolved the problem by replacing "Authority" with "Issuer" in the Okta settings in appsettings.json.

Thanks @paulapaulTW! 👍  Sorry for the delay.


The edit icon on the new bottom par has been replaced by a search icon.

The edit icon on the new bottom par has been replaced by a search icon.
The crayon is much more explicit.

Working on it


⚠ ClamAV has degraded performance

In 031b047, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 199 ms

Resolved: ClamAV performance has improved in 55903f3.


TemplateElement

JSXText and Literal are already supported but TemplateElement is not yet supported. Can that be added?
Template literals (Template strings) - JavaScript | MDN

I believe node.value.cooked is how the text in the template can be queried to be checked… there might be a better way. 🤔


patch files added

Added contributing.md and issue templates for PyDeceive.

@smaranjitghose  I am unable to make a PR into bonus branch here.

When making a PR, just compare it with the bonus branch instead of master


grpcproxy: forward v3lock and v3election requests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@47f5b7c). Click here to learn what that means.
The diff coverage is 77.02%.

@@           Coverage Diff            @@
##             master   #7908   +/-   ##
========================================
  Coverage          ?   75.8%
========================================
  Files             ?     336
  Lines             ?   26390
  Branches          ?       0
========================================
  Hits              ?   20004
  Misses            ?    4952
  Partials          ?    1434

Impacted Files
Coverage Δ

integration/cluster.go
85.54% <ø> (ø)

etcdmain/grpc_proxy.go
20% <0%> (ø)

proxy/grpcproxy/lock.go
100% <100%> (ø)

proxy/grpcproxy/adapter/lock_client_adapter.go
100% <100%> (ø)

integration/cluster_direct.go
100% <100%> (ø)

integration/cluster_proxy.go
100% <100%> (ø)

proxy/grpcproxy/adapter/election_client_adapter.go
68.96% <68.96%> (ø)

proxy/grpcproxy/election.go
82.6% <82.6%> (ø)

Continue to review full report at Codecov.

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

lgtm


Kafka Connect Task Restart created many hung sessions in the database

Hi,
There are sometimes happened when Kafka Connect did not emit any DML events (even though there is a DML events happened in the database). The connection seemed to be hanged up, but I see no error-level log in the connect.log. This problem can be resolved by restarting the Task through the REST API:
http://<host>:8083/connectors/<connector name>/tasks/0/restart

I even create a Restart Automation(simple java application) to restart the task by invoking DML events and restart the task if no topic is received within periods of time. The Restart Automation can restart the task for about as many as 20 times a day, but the logmining seems to working fine after restart.
The problems start to occured when there are reports of hundreds of sessions created from the Kafka Connect Host:

I'm not really sure about this, but is there any possibility that this session creation happened when I'm triggering Kafka Connect Task restart? Thanks.

Hi,
Sorry for late answer. You said connector does not emit DML events even though there is a DML. After restarting connector are uncaptured dml events be captured ?
Lots of sessions shows connector did not stop successfully , previous logminer sessions are still active . You  can select these sessions from v$logmnr_session.
Thanks.


Dependabot keeps posting PRs in forked repo with Dependabot disabled in settings

I have a public repo: https://github.com/Karlson2k/Flexget
The repo is forked from another public repo.
As I got from documentation, Dependabot shouldn't scan this repo because it's fork, however I have constant flow of new PRs from Dependabot.
I disabled all Dependabot alerts in repo settings:

but I still have new PRs.
I also tried to disable Dependabot globally for my account, but it didn't help either.
I think Dependabot should not post new PRs on this repo and it looks like a bug.

Duplicate of https://github.com/dependabot/dependabot-core/issues/2804, which we recently fixed.
However, since this fork was opted in before we shipped the change, you'll need to manually disable it. See https://github.com/dependabot/dependabot-core/issues/2804#issuecomment-1326137957 for the gory details.


Evaluate alternative static file embedder

The tool we use to embed assets into the binary has issues around timestamping. That's the reason why we don't always trigger it on build (commit pollution), which often leads to it being forgotten on actual changes.
https://github.com/rakyll/statik might be an alternative.

Hi,
One way to fix this timestamp issue with go-bindata is to set a fixed timestamp. That's how we solved this issue in our codedbase;
go-bindata -ignore=\\.go -ignore=\\.DS_Store -mode=0644 -modtime=1453795200 -o pkg/templates/bindata.go -pkg templates pkg/templates/...

Interesting. But I'm assuming that this timestamp is used in HTTP header fields, which in return are used to do cache invalidation in the browser so users actually see changes happening.

I see. In our usecase w don't serve them from via HTTP. So, it was not an issue.

I tried statik and can confirm it also uses modification-date from filesystem so it has the same "issue". touch any file and generated file will change. In contrast same http cache invalidation will happen on plain files sitting in filesystem served with nginx for example.
go-bintool fixed timestamp parameter generated from the last commit timestamp on web/ui/static might be viable idea. But i'm not sure if that's possible in build environment.

A related improvement was proposed to bindata but seems dismissed without any progress. https://github.com/jteeuwen/go-bindata/issues/120

Also, go-bindata was abandoned in Februry 2018 and there are now lots of forks with no designated successor  (AFAICS). The (incomplete) history can be found here. Lots of projects are evaluating the situation and are looking for an alternative or started to vendor the go-bindata (e.g. Kubernetes).
BTW: Currently, go-bindata is not vendored in Prometheus. Instead the Makefile go gets it on demand. This could be dangerous if the upstream project really gets backdoored.
FWIW: Yesterday, I was searching for a designated successor to go-bindata (for a private project) and found this interesting article:  Choosing A Library to Embed Static Assets in Go. It evaluates alternatives with some addtional features:

Compression
Optional Decompression
Loading from the local File System
Reproducible Builds
Config File
http.FileSystem interface

Thanks for the information, that bumps the priority. If someone could help with this that'd be great, we've struggled with getting this right a fair bit over the years.

Head-up: I'm looking into it right now and https://github.com/shurcooL/vfsgen looks promising.


OSError: dlopen() failed to load a library: cairo / cairo-2 / cairo-gobject-2'`

I used python3.
I use pip install  pycairo
 F:\3Blue1Brown\manim-master>python extract_scene.py example_scenes.py SquareToCi rcle -pl Traceback (most recent call last):   File "extract_scene.py", line 15, in <module>     from scene.scene import Scene   File "F:\3Blue1Brown\manim-master\scene\scene.py", line 17, in <module>     from camera.camera import Camera   File "F:\3Blue1Brown\manim-master\camera\camera.py", line 10, in <module>     import cairocffi   File "C:\Users\Tony\AppData\Local\Programs\Python\Python36-32\lib\site-package s\cairocffi\__init__.py", line 41, in <module>     cairo = dlopen(ffi, 'cairo', 'cairo-2', 'cairo-gobject-2')   File "C:\Users\Tony\AppData\Local\Programs\Python\Python36-32\lib\site-package s\cairocffi\__init__.py", line 38, in dlopen     raise OSError("dlopen() failed to load a library: %s" % ' / '.join(names)) OSError: dlopen() failed to load a library: cairo / cairo-2 / cairo-gobject-2'

Same for me; have you found a fix yet?

I am on mac10.13/python3.7. My error message is slightly different but must be related to the same issue.

ImportError: dlopen(/Users/ishandutta2007/.pyenv/versions/3.7.0/lib/python3.7/site-packages/cairo/_cairo.cpython-37m-darwin.so, 2): Symbol not found: _cairo_xcb_surface_set_size

This worked for me(mac10.13/python3.7.), not sure if the first line is required though, now I have pycairo 1.18.0 instead of 1.17.1 as in requirements.txt
pip install cairocffi
pip uninstall pycairo
pip install pycairo

Closing, as this issue seems to be resolved.

pip install cairocffi
pip uninstall pycairo
pip install pycairo
worked for me..
thanks :)


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v2.6.28 to before v4.4.273",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=05736a427f7e16be948ccbf39782bd3a6ae16b14",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=0bc62e398bbd9e600959e610def5109957437b28"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "05736a427f7e16be948ccbf39782bd3a6ae16b14",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "0bc62e398bbd9e600959e610def5109957437b28",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "ftrace: Do not blindly read the ip address in ftrace_bug()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.4.273 by commit 0bc62e398bbd9e600959e610def5109957437b28, it was introduced in version v2.6.28 by commit 05736a427f7e16be948ccbf39782bd3a6ae16b14. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000830


[PROTON] Remove cupti/cuda header files in CuptiProfiler.h

CuptiProfiler.h needs to be included in Session.h, which will include RocTracerProfiler.h, mixing cupti.h and roctracer.h will cause problems because some variables have the same name.
-o third_party/proton/CMakeFiles/proton.dir/csrc/lib/Session/Session.cpp.o -c /data/triton/third_party/proton/csrc/lib/Session/Session.cpp
    In file included from /opt/rocm/include/hip/hip_runtime_api.h:566,
                     from /opt/rocm/include/hip/hip_runtime.h:70,
                     from /opt/rocm/include/roctracer/roctracer_hip.h:26,
                     from /data/triton/third_party/proton/csrc/include/Driver/GPU/Roctracer.h:6,
                     from /data/triton/third_party/proton/csrc/include/Profiler/RoctracerProfiler.h:5,
                     from /data/triton/third_party/proton/csrc/lib/Session/Session.cpp:6:
    /opt/rocm/include/hip/amd_detail/host_defines.h:166: error: "__host__" redefined [-Werror]
      166 | #define __host__
          |
    In file included from /data/triton/third_party/nvidia/backend/include/device_types.h:59,
                     from /data/triton/third_party/nvidia/backend/include/builtin_types.h:56,
                     from /data/triton/third_party/nvidia/backend/include/cupti_callbacks.h:54,
                     from /data/triton/third_party/nvidia/backend/include/cupti_activity.h:61,
                     from /data/triton/third_party/nvidia/backend/include/cupti.h:71,
                     from /data/triton/third_party/proton/csrc/include/Driver/GPU/Cupti.h:4,
                     from /data/triton/third_party/proton/csrc/include/Profiler/CuptiProfiler.h:5,
                     from /data/triton/third_party/proton/csrc/lib/Session/Session.cpp:5:
    /data/triton/third_party/nvidia/backend/include/crt/host_defines.h:222: note: this is the location of the previous definition
      222 | #define __host__ \

CuptiProfiler.h needs to be included in Session.h, which will include RocTracerProfiler.h

If CuptiProfiler is not specific to nvidia cupti, should the header be renamed?

If CuptiProfiler is not specific to nvidia cupti, should the header be renamed?

Oh, I see.  This file is indeed specific to cupti, but it's included by the generic session handler.


TEST

Example URL(s)
asdf
Actual Behavior
asdf
Expected Behavior
asdf
Additional information
No response

Test removing more info needed

Howdy!

test


ENH: Add check for doc or workflow label

Checking if the doc or workflow label is added to the PR. If it is then the CI should pass

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.00%. Comparing base (f72383b) to head (ca691ba).

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #2      +/-   ##
==========================================
+ Coverage   98.99%   99.00%   +0.01%
==========================================
  Files           6        6
  Lines         199      202       +3
==========================================
+ Hits          197      200       +3
  Misses          2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Merging Develop into Master 2.1.19 - Twilio updates, other small changes

Hello @apigirl,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Add automatic bias computation in LinearMachine

Adding Bias calculation to train_machine.
Replacingcblas_daxpy call by a linalg call or by using Eigen3.

@karlnapf I have a question in the train_machine() method.
what is the point of using a matrix (kernel_matrix)to hold the weights instead of using a vector?

@karlnapf I have one question in train_machine().
I don't understand the use of kernel_matrix instead of vector to hold weights.

What is the point of opening this exactly? :)

The point you asked why the kernel matrix is stored is simply: because the solution depends on a linear system involving the kernel matrix

@karlnapf yeah the question about the matrix was silly so I erased it.
I opened it because I intend to make the desired changes so just to separate it from the main issue.

Don't worry about such questions ;)
Yes definitely this is a separate task. So I will leave this open until it is done

Renamed it, this should actually be done in the base class -- every linear algorithm should the option of pre-computing the data mean manually, and do it automatically by default

@karlnapf but as I understand from the docs,the train_machine() method in CLinearMachine is just a virtual function re implemented in the subclasses.

@karlnapf I edited the train_machine() method in LinearRidgeRegression class. check here.
I also made a notebook with results using the same datasets.
I got rid of lapack calls and replaced them with eigen.
For the bias calculation, I set to the mean of error between the true target and the predication with bias equals zero.
e.g
true target is vector y
y = bias + w*feats
bias = mean( y - w*feats)

that's what I understood when you mentioned the data mean because I don't think you meant the mean of the labels because if I have a dataset with labels having a real intercept zero but only positive values the calculated intercept will be wrong ( e.g first example in notebook). Am I getting it right ?

sure it is a virtual function, but computing the data mean still should not happen in specialised classes, but in the base class. We can always add an additional method that calls a virtual method that is overloaded in subclasses...

BTW please don't post code with gists, send a PR. Much easier to review

In the notebook, why are the results not exactly the same? Can you check that the ridge has the same value?

What about larger datasets and dimensions? Timing might change then?

Ok had a look at the code. The bias addition should happen in the base class as said, and it should be possible to set the bias manually and disable that it is computed from data.
Can you send a PR with the changes? I have a few more comments, but don't want to put them in your branch

@karlnapf The results are exactly the same for the second and the third datasets because they are fixed but the first is randomly generated each time so the results will change ( sklearn results also changes for the first dataset).
For the bias calculation,so I will make a function that computes the bias in the base class but it's not clear  to me how it will be called without modifying the train_machine in subclasses.

The sub-classes will not compute the bias as it currently has to be set manually.
Therefore, one way to do this would be to rename the train_machine methods (that dont compute bias) and add an additiona lmethod compute_bias() in the base class. Train_machine in the base class then (optionally) computes the bias and then calls the train_machine_no_bias of the specialsed classes.
Something like this, standard polymorhpic OOP

@karlnapf I got it but I have a suggestion. Instead of renaming the train_machine in all subclasses can I override the train() method inherited from the CMachine class and add the optional compute_bias call to it.

Thats Indeed much better and actually the reason why we have this structure in the base class

I think this issue can be closed :) , but I have one question for my understanding.
Why train_machine is implemented in a closed form solution instead of using gradient descent or stochastic gradient descent. From my basic knowledge I think computing X transpose and multiplying X by X transpose may be very expensive operations specially for large number of features like 100,000 feature or something like that.

Yes, the matrix is DxD where D is number of features.
We could implement a stochastic version of this, but even better would be to use an existing library to do so.....Vowpal wabbit for example


Rsync test fails (parsing error)

Here is why. This is the output of my rsync command:
...

Number of files: 47 (reg: 27, dir: 20)
Number of created files: 47 (reg: 27, dir: 20)
Number of deleted files: 0
Number of regular files transferred: 27
Total file size: 25.003 bytes
Total transferred file size: 25.003 bytes
Literal data: 0 bytes
Matched data: 0 bytes
File list size: 1.440
File list generation time: 0,001 seconds
File list transfer time: 0,000 seconds
Total bytes sent: 449
Total bytes received: 1.612

sent 449 bytes  received 1.612 bytes  588,86 bytes/sec
total size is 25.003  speedup is 12,13 (DRY RUN)

The interesting line is the following:
Total transferred file size: 25.003 bytes
Notice that the thousand separator is a dot. This line is parsed here and is expected to use commas, as thousand separator. The function fails and t2b does not perform the backup.
On my system, the LANG env variable is set to fr_FR.utf-8. Setting LANG=C fixes the problem. Other locales are probably affected by this issue.
I would submit a PR but I'm not sure how you would like to fix this.

Override the LANG env variable to get consistent rsync output across all systems
Fix the sed command to handle all possible thousand separators
...

Thanks for report this.
It seems there are only 2 separators that exists in the world: dots and commas.
So I would recommend to handle this inside the sed syntax.
Feel free to open a PR. Thanks a lot!

Sorry, but I will have to let you handle the fix.
I didn't use github for a while and I just found out that I can't use my password for pushing commits to a repo anymore. I would be happy to help, but I don't have time nor patience for microsoft bs.
Anyway, thank you for this great util :+1:

Bug fixed in commit c9136b34542ddd78e3758d88436c55af5a4527d1 and published in 1.9.1 that was released tonight


Use slice::iter instead of into_iter to avoid future breakage

an_array.into_iter() currently just works because of the autoref
feature, which then calls <[T] as IntoIterator>::into_iter. But
in the future, arrays will implement IntoIterator, too. In order
to avoid problems in the future, the call is replaced by iter()
which is shorter and more explicit.
A crater run showed that your crate is affected by a potential future
change. See https://github.com/rust-lang/rust/pull/65819 for more information.

Thank you!

@pyfisch Not to rush anyone, but could this change be released as 0.10.3? Lots of crates depend on this one, so getting this fix out soon would be great :)

Done.
But I doubt that this will help much as many crates still depend on v0.9 and some even on v0.8 and earlier releases.

@pyfisch Thanks a bunch!
Unfortunately, I was not particularly smart regarding this PR. The same happened for itertools. This PR only fixes tests and when users depend on this crate, these tests are not compiled. So indeed not a single user crate regressed because of this crate. This is good in a sense, but also bad since I caused you unnecessary work and a useless release. I am truly sorry for that!
At least we are completely done here now.


FlaUIInspect does not recognize controls under ItemControl at times

I have few controls grouped under ItemsControl and FlaUI is not recognizing them and showing up in the tree. This is happening on and off and my scripts are failing. Am unable to recognize the pattern. If  i close and open the application and try 2 to 3 times it is working. Am i missing something. I have ample amount of tests getting blocked because of this.
 Any help would be appreciated. Do i need to write anything like refresh?

Are the items asynchronously loaded? How does the code in FlaUI look?


Home screen / Basic Workflow Dashboard Dev.

Home screen / basic workflows dashboard development for Litmus Portal with no scheduled workflows.

closing in favor of https://github.com/litmuschaos/litmus/pull/1592


Merge tag v2.10.1

What this PR does / why we need it:
How does this change affect the cardinality of KSM: (increases, decreases or does not change cardinality)
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/cc @rexagod

/lgtm
Only the following files differ, which is expected.
.ci-operator.yaml
.dockerignore
.gitignore
Dockerfile.ocp
OWNERS

FYI I can see a rhel-9 update on https://pkgs.devel.redhat.com/cgit/containers/kube-state-metrics/. Should we consider upgrading, in a future PR?

I tried pulling the updated manifests in CMO by revendoring kube-prometheus but for some reasons, make update + generate didn't fetch the 2.10.1 ksm manifests. If any of you have some time to look into it, that would very appreciated

@rexagod are you talking about https://github.com/openshift/kube-state-metrics/pull/105? It seems unrelated to this PR.
@dgrisonnet CMO is pinned to the main branch of kube-prometheus which is itself pinned to the main branch of kube-state-metrics. Why do you need the updated jsonnet from ksm?

/retitle MON-3548: Merge tag v2.10.1
/jira refresh

/jira refresh

/jira refresh


PIPELINE_VERSION = False doesn't work

I prefer to leave filename-based versioning off, because we need to have our output files in version control and removing old compressed files is tedious.
When PIPELINE_VERSION is set False, the compressed files are not overwritten, but new files are added named with  _1, _2, _3  extensions. The templatetags do not read these files, but read the original compressed file, so changes are not displayed.
I'm using YUICompressor for compilation of both css and js.

That's look like a storage related problem, I'll look into it.
You can use versionning and let pipeline remove old version for your, there is a settings for that, check the doc.

Thanks so much!


Error code 'sapHanaConnectionFailed' when setting up a HANA provider

Hi team,
Creation of a HANA provider fails with the following error:
{"code":"ProviderInstanceValidationOperationFailed","message":"
The provider settings validation operation has failed with code 'sapHanaConnectionFailed'.
Possible Causes: The operation failed with error: 'Connection to SAP HANA Failed.'.
Recommended Action: 'Please Update connection settings.'.
","details":[]}

This is not the same root cause as for Issue #12. The NetWeaver and Linux providers for the same VM are created successfully:

I have tried three different setups against the SYSTEMDB, none of which appear to connect:

dedicated AZUREMONITOR user with the MONITORING role and CATALOG READ system privilege with IP address
the SYSTEM user with IP address
the AZUREMONITOR user with FQDN

The FQDN (labs4hhdb2021.sap-internal.softwareone.cloud) is in a public zone and returns the internal IP of the host.
HANA Studio is able to connect with the dedicated user, so the password works.
It might be relevant that the SSL certificate in the HANA system is self-signed, and the connection from HANA Studio only succeeds if 'Validate the SSL certificate' is unticked.

There are no firewalls active on the host:

This is an MDC system with instance number 21. Port 32115 is reachable from an external machine:

As per #13 it would be really useful to see detailed error messages to troubleshoot this.
Sascha

Hi,
Usually, the systemdb port is 3xx13. What we noticed here is port number given is 3xx15. Can you please try creating a HANA provider by giving port number 313? If the issue still persists, then we can have a quick call.
Thank you.
Neenu

Hi Neenu,
Thank you for the tip - that worked. I was a bit thrown off by the comment in the portal that port 3xx15 should be used for MDC systems. This is also the port that HANA Studio appeared to be connecting to.
Might be worthwhile updating the text of the tooltip?


Package fails to install on Windows

The .tgz file published to npm contains several files with * in their file names:

./src/grammars/generated/js/--*.js
./src/grammars/generated/json/--*.json
./src/grammars/generated/nearly/--*.ne

As * is an illegal file name character on Windows, both yarn and npm fail to fully extract this package, rendering it unusable.
npm output:
C:\projects\test> npm i css-property-parser
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
npm WARN tar ENOENT: no such file or directory, open 'C:\projects\test\node_modules\.staging\css-property-parser-a58395e0\src\grammars\generated\js\--*.js'
npm WARN tar ENOENT: no such file or directory, open 'C:\projects\test\node_modules\.staging\css-property-parser-a58395e0\src\grammars\generated\json\--*.json'
npm WARN tar ENOENT: no such file or directory, open 'C:\projects\test\node_modules\.staging\css-property-parser-a58395e0\src\grammars\generated\nearley\--*.ne'
npm WARN saveError ENOENT: no such file or directory, open 'C:\projects\test\package.json'
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN enoent ENOENT: no such file or directory, open 'C:\projects\test\package.json'
npm WARN test No description
npm WARN test No repository field.
npm WARN test No README data
npm WARN test No license field.

+<EMAIL_ADDRESS>added 15 packages in 2.607s

Would it be possible to re-publish the last version with cross-platform compatible file-names? :)

Hi, thanks for the great diagnosis. v1.0.6 is released and those files have been renamed. Can you please verify if this release works for you?

Verified as installing correctly with both npm and yarn.
Thanks for the prompt response and bugfix release! 👍


setTimeout with infinity delay will execute function immediately

It's interesting that passing Infinity as delay parameter while setTimeout() called, the function will executed immediately instead of infinity delay.
setTimeout(() => console.log('called'), Infinity)

@VdustR That's really interesting !! I've ran a few more tests and the results are the same:

do you recon the setTimeout implementation automatically sets timeout to zero when something is immeasurable ?

According to MDN:

Maximum delay value
Browsers including Internet Explorer, Chrome, Safari, and Firefox store the delay as a 32-bit signed integer internally. This causes an integer overflow when using delays larger than<PHONE_NUMBER> (about 24.8 days), resulting in the timeout being executed immediately.

If browsers store it as a 32-bit signed integer then it's not Infinity anymore.
It's explained that setTimeout receive msec as an Interger in MSDN, too:

msec [in]
Type: Integer
Integer that specifies the number of milliseconds.

According to Node.js Timer API:

Note: When delay is larger than<PHONE_NUMBER> or less than 1, the delay will be set to 1.

I also found this in w3.org:

6.3 Timers
When the above methods are to get the timeout, they must run the following steps:

Let timeout be the second argument to the method, or zero if the argument was omitted.
Apply the ToString() abstract operation to timeout, and let timeout be the result. [ECMA262]
Apply the ToNumber() abstract operation to timeout, and let timeout be the result. [ECMA262]
If timeout is an Infinity value, a Not-a-Number (NaN) value, or negative, let timeout be zero.
Round timeout down to the nearest integer, and let timeout be the result.
Return timeout.

As the 4rd in mentions, If timeout is an Infinity value, let timeout be zero.
It's seems like there is some difference between these references (minimum is 1 in node.js and 0 in w3.org).
I have no idea if there is a standard or how the browsers or node.js handle this exactly.
I also found this article about the difference between the browsers in this case:
Environment:

OS: Windows 7 Ultimate build 7600
Browsers:

IE6
IE7
IE8
Firefox 3.6.9
Chrome 7.0.517.5 dev
Safari 5.0.2
Opera 10.62

Results:
Firefox, Chrome and Safari: executed immediately.
IE and Opera: never executed
I also tested in some cases:

Ubuntu 17.04 Firefox 55.0.2 (64-bit)
Ubuntu 17.04 Google Chrome 60.0.3112.113 (64-bit)
Ubuntu 17.04 Node.js v8.4.0
Windows 7 Google Chrome
Windows 7 IE11

They are all executed immediately.


Fix applicants table styling and sorting

Change width of interview box so that the entire time is visible
Sorting on time is not working as intended

Sorting works local but not in production

It seems to be a problem when dates are close to month change


Initial bootstrap mode

chart-operator is one of the first things to be installed in a management cluster, alongside app-operator.
Back in the day, we used to install cluster-critical components (such as the CNI and kube-proxy) as barebone kubernetes manifests, because the chicken-egg problem prevented us to install apps providing such core components using the app platform.
One day we added the chartOperator.cni.install flag that allows to run chart operator even on a cluster lacking core components (specifically the CNI) and thus being able to install the CNI as part of a giant swarm Chart.
In this scenario, kube-proxy still needed to be installed as part of the master node bootstrap process which is unfortunate.
This PR adds a new flag initialBootstrapMode that relaxes most of chart-operator needs in terms of cluster resources in order to make it run on clusters lacking core components.
This is a backward compatible flag disabled by default, meant to be set to false once the cluster bootstrap is finished.
This is the same behaviour as app-operator
Checklist

[x] Update changelog in CHANGELOG.md.

LGTM


Allow splitting of single element arrays

I have the following code that I needed to add an extra element to the array:
$prov_opts = array('method' => 'speed_only');

I wanted to end up with:
$prov_opts = array(
    'method' => 'speed_only',
);

But it appears that splitjoin only works on arrays with more than one element? Can this be changed? Maybe make it an optional configuration directive?

I guess I probably thought there would be no point in splitting an array with a single item. But you're right that it would be convenient when adding items. I've pushed a commit that should fix this, could you try it out?

Latest update looks like it fixes this. Minor issue is the trailing comma issue that's present in #80.
I'll close this for now, and we can discuss trailing commas in the other issue.


Move kill-on-failure responsibility to caller of weave script

Fixes #2284; also ends up rather simpler.  Undoes much of #2264.
The change also removes two flags from the interface of attach(), at the cost of a duplicate Inspect in case of error.  I prefer it that way.

Added some error handling and squashed commits.


Update guidance: test "job" is now CI "workflow"

As described in the documentation change, we no longer define
test jobs, and now define CI workflows, which work slightly
differently insofar as they're made up of several parallel jobs.
This commit makes a few other updates, including:

Remove reference to small steps within a CI workflow:
We've actually been heading in the opposite direction: lots of
small steps defined within the CI workflow. This is partly because
we're now making use of a lot of shared workflows, so what we lose
in ease of local development, we gain in cross-GOV.UK consistency.
Remove old example code
Looking at the Content Publisher GitHub Actions, we no longer have
the complexity referred to in these docs. I've deleted the docs
that are no longer relevant.
I've kept the instructions for testing Ruby gems, as that is still
being applied.
Move "previous recommendation" section further up:
Moving it closer to where we're justifying our choices for "when
the CI workflow should run".
Also creates a new subsection for running CI on demand.
Softening the language around omitting names for jobs and steps.
In practice, a great many of our repos needlessly define job and
step names that override the job/step key, but there is no harm
in doing so.

Thanks @AgaDufrat - I like the suggestion, but also like the top-down approach of "this is the file", "this is how it's structured", and then eventually "this is where we define our reusable workflows"...
But I think a future restructure would be good, making it easier to dive in and find what you need (and can probably be done in a far less wordy way than the current iteration). 👍


fix issue #258

All whitespace (except for the required indentation) should be ignored in flow context.
Fixes #258.

Indentation currently isn't properly respected (intentionally???) in flow contexts. But that's another thing to fix at another time.


MWPW-138927: merch-twp WIP

This is a PR for the TWP feature branch
preview: https://mwpw-138927--milo--yesil.hlx.page/drafts/ilyas/twp/illustrator#twpmodal
also contains latest stage changes.
Resolves: MWPW-138927

FYI @yesil I removed the gigantic list of reviewers since this PR is for now internal (our WIP)

also contains latest stage changes.
next time we should probably reset twp branch on stage instead of merging from pr


Add rspec-puppet tests for owner/group/mode of ODL dir and .service file

#51 added the odl user and group, #58 set them as the owner:group of ODL's directory. That behavior needs rspec-puppet tests.

Making this more general (see name change logged above). We needs tests that cover the owner, group and mode of both the ODL dir and the opendaylight.service systemd file.

a914c93b12dcd6351a0416ef2b7b4d4a4e254599 and 5a0c176bc04c812365764c3777a150886f3b1703 implemented this.


docs: fix html nits

Description
Really minor fixes like closing the <br /> tag in preparation for Docusaurus2, which doesn't like it when they're not closed.
This PR has:

[x] been self-reviewed.

Hey @317brian I have https://github.com/apache/druid/pull/13747 failing on a static CI check for something that looks appropriate for this PR and I'm guessing it will fail for you as well?
> spellcheck
> mdspell --en-us --ignore-numbers --report '../docs/**/*.md' || (./script/notify-spellcheck-issues && false)

    ../docs/tutorials/tutorial-jupyter-index.md
       41 | er both try to use port `8888,` so start Jupyter on a different port

>> 1 spelling error found in 227 files

Although to be fair I have no idea what its upset about?  Seems to be complaining about this line

Yeah, someone else was running into this earlier. It passes spellcheck locally, but for some reason the GHA doesn't like the position of the comma. It wants it outside of the `, which is technically correct but looks odd in the rendered output.

@317brian @jwitko, yeah, I ran into the same spellcheck failure. spellcheck assumed there was a huge codeblock with an incorrect syntax because of an additional backtick that got accidentally introduced. It's now fixed in master; if you pull the top of master, that should have the fix https://github.com/apache/druid/pull/13836.

The step in CI that's failing is the license check
Error: found 1 missing licenses. These licenses are reported, but missing in the registry
druid_module: avro-extensions, groupId: com.sun.activation, artifactId: jakarta.activation, version: 1.2.1, license: Eclipse Distribution License 1.0

It's unlikely to be caused by this PR since this PR only touches files in docs and not the avro-extension. The PR also doesn't add any new files that would need a license.


fix for compile problems on Ubuntu 21.04 and other new distributions

Feathercoin-qt doesn't compile on ubuntu 21.04 and other newer distributions due to a change in the includes for qt, leading to a missing QPainterPath include
This path fixes that.

Is this version of QT included vulnerable to this bug in this security bulletin - https://www.cvedetails.com/cve/CVE-2020-0570/ ?
It looks like version 5.9.8 is included according to the QT version file - https://github.com/FeatherCoin/Feathercoin/blob/master/depends/packages/qt.mk
The bug report says:  "Uncontrolled search path in the QT Library before 5.14.0, 5.12.7 and 5.9.10 may allow an authenticated user to potentially enable elevation of privilege via local access."
I just thought I would add a note in case it needed to be updated I had noticed a really old one in another coin recently and most aren't updated.


Prevent start exception if encryption section missing

The encryption service is not usable if no passphrase set, but that service is not in use at all - currently.

Summary of OSS sync: this was for MFA feature (which is already removed); hence these fields are currently unused. But this PR keeps them in case there is future usage.

sonar :
https://sonarcloud.io/summary/new_code?id=cloudfoundry-identity-parent&pullRequest=2767


Refactor channel ENV variables to be configured via JSON

With the number of channels we're adding, we will eventually run into constraints with the # of bytes we're trying to store in ENV variables on Heroku.
We eventually will need to move to using a different configuration system for channels, for which I would suggest using JSON.
This could be as simple as something like channels.json that looks like:
{
  "CHANNEL_FIDENZA_SALES":<PHONE_NUMBER>75192320
  "CHANNEL_FIDENZA_GENERAL":<PHONE_NUMBER>97001293
  // ... etc
}

Or we could (this is preferable but more nuanced and this work could be done in two parts as such) configure the entire project bots (each ProjectBot) in JSON which would make it easier for to add support for new artists/projects.
E.g., something like:
{
  "fidenza": {"channel":<PHONE_NUMBER>75192320, projectNumber: 78, name: "Fidenza", mintCount: 999}
  // ... etc
}

CC @CosineXArt I thought you may find this interesting, no pressure to take this on by any means, just wanted to put this idea on your radar and see if you had any thoughts.

working on this! :)


False alarm TRIO107 with try/finally block

async def to_queue(iter_func, queue):
    async with iter_func() as it:
        async for x in it:
            queue.put(x)

async def to_queue2(iter_func, queue):
    try:
        async with iter_func() as it:
            async for x in it:
                queue.put(x)
    finally:
        queue.put(None)

$ flake8 example.py
example.py:7:1: TRIO107: exit from async function with no guaranteed checkpoint
                         or exception since function definition on line 7

but if the first case is fine, the second should be fine too.

async def f():
    with ...:
        try:
            await trio.sleep(0)
        finally:
            ...

Seems like a simpler repro, probably the same underlying issue?

(the with is also irrelevant)
Took me a while to figure out wth is going on here, and ultimately the error was due to me wanting to catch
try:
  [code that might error]
  await trio.sleep(0)
finally:
  return # or yield

So I always entered finally with a worst-case jump from try - which is correct for checking the content of the finalbody, but in the case of no errors it's incorrect to exit finally with that worst-case.

Fixed with #49


Unexpected "[" while reading a shortcut option statement

What is the current behavior?
I have options in my Yarn writeup that follow the expected escape pattern for special characters, e.g.
"->[Wisdom] Say something wise."
The graph in VS Code renders correctly, and the Yarn file runs correctly through my parsers and runners, but VS Code shows the following parse errors file in my Problems view:
Unexpected "[" while reading a shortcut option statement
Please provide the steps to reproduce, and if possible a minimal demo of the problem:
This is the minimum yarn file I need in order to reproduce:
title: Node
position: 113,134
---
->\[Wisdom\] Say something wise.
===

What is the expected behavior?
As this is documented as the correct way to escape special characters, I would expect no Problems to be reported.
Please tell us about your environment:
Yarn Spinner prerelease v2.2.135
VS Code Insiders 1.84.0
MacOS Sonoma (14.0)
Other information
N/A

Reproduced this issue locally. The issue doesn't appear when any other non-blank character begins the line. The issue appears in both options and in regular lines.

Closing this as the core issue has been fixed, next update to VSCode should include this


Rename current --verbose into --debug, and remove FRACTAL_LOGGING_LEVEL variable

The fractal command has a -v,--verbose option, that we only use to set the logging level to DEBUG. We also have a FRACTAL_LOGGING_LEVEL, that provides a default value (in case --verbose is not set).
This feature is essentially useless, since most of the logging instances are at the ERROR level and therefore they are always shown:
$ git grep "logging\.critical" fractal_client/

$ git grep "logging\.error" fractal_client/
fractal_client/cmd/_job.py:        logging.error(f"Server returned {res.status_code}")
fractal_client/cmd/_job.py:        logging.error(
fractal_client/cmd/_job.py:        logging.error(
fractal_client/cmd/_job.py:        logging.error("Terminating.\n")
fractal_client/cmd/_job.py:        logging.error(
fractal_client/cmd/_job.py:        logging.error(
fractal_client/cmd/_job.py:        logging.error(
fractal_client/cmd/_job.py:        logging.error("Terminating.\n")
fractal_client/cmd/_task.py:                logging.error(
fractal_client/cmd/_task.py:            logging.error(
fractal_client/cmd/_task.py:            logging.error(
fractal_client/cmd/_workflow.py:            logging.error(
fractal_client/response.py:        logging.error(f"Server returned {res.status_code}")
fractal_client/response.py:        logging.error(
fractal_client/response.py:        logging.error(
fractal_client/response.py:        logging.error(f"Server error message: {error_msg}\n")
fractal_client/response.py:        logging.error("Terminating.\n")

$ git grep "logging\.warning" fractal_client/
fractal_client/cmd/_workflow.py:        logging.warning(
fractal_client/cmd/_workflow.py:        logging.warning(

$ git grep "logging\.info" fractal_client/

$ git grep "logging\.debug" fractal_client/
fractal_client/client.py:    logging.debug(f"Arguments: {args}")
fractal_client/response.py:    logging.debug(res.status_code)

Moreover, we will need the --verbose option for a more useful feature, namely setting the verbosity of the command output (see upcoming issue).
I propose that:

We remove the FRACTAL_LOGGING_LEVEL, and always use INFO by default.
We introduce a --debug option, which sets this to default level to DEBUG (the same as the current --verbose).
We add logic to the client/response modules so that the using --debug actually shows debugging info (what API call I'm going to make, the response status, ...).

From commit ce730a21a22a464ba77c24f0b22a7c93135ee86c
# without --debug
$ poetry run fractal --password=secret project new test_project

{
  "name": "test_project",
  "read_only": false,
  "id": 79,
  "timestamp_created": "2024-01-26T11:55:09.276251+00:00"
}

# using --debug
$ poetry run fractal --debug --password=secret project new test_project2

DEBUG:root:
Arguments:
    user: None
    password: ***
    debug: True
    batch: False
    cmd: project
    subcmd: new
    name: test_project2
DEBUG:root:
Fractal Client sending HTTP request to:
    POST http://localhost:8000/api/v1/project/
Request body:
    name: test_project2
DEBUG:root:
Response status code:
    201
{
  "name": "test_project2",
  "read_only": false,
  "id": 80,
  "timestamp_created": "2024-01-26T11:56:08.127493+00:00"
}


Sync CTS Roles to Core Roles

This PR;

Adds the ability to get mentors from CTS based on either their RTS affiliation or on a "search" for callsign they have the rights to mentor on (e.g. EGKK_APP or EGKK)
Adds the ability to get ATC or Pilot examiners from CTS
Adds the ability to get students within an RTS from CTS
Adds a command to sync the roles on CTS with a role on Core. This role is then used for Discord access (and will be automated via events with PR #2122)

The command lacks test coverage and has a lot of code duplication that could likely be tidied up, but figured I get this PR open as a starting point to discuss.

:tada: This PR is included in version 6.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[cuda] Build error on x64-windows

Package<EMAIL_ADDRESS>Host Environment

Host: x64-windows
Compiler: MSVC 19.42.34433.0
vcpkg-tool version: 2024-11-12-eb492805e92a2c14a230f5c3deb3e89f6771c321
vcpkg-scripts version: 834977918 2024-11-20 (2 days ago)

To Reproduce
vcpkg install darknet[full]:x64-windows
Failure logs
-- Executing NVCC-NOTFOUND --version resulted in error: 1
CMake Error at ports/cuda/vcpkg_find_cuda.cmake:69 (message):
  Could not find CUDA.  Before continuing, please download and install CUDA
  (v10.1.0 or higher) from:

      https://developer.nvidia.com/cuda-downloads

Call Stack (most recent call first):
  ports/cuda/portfile.cmake:9 (vcpkg_find_cuda)
  scripts/ports.cmake:196 (include)

-- Executing NVCC-NOTFOUND --version resulted in error: 1
CMake Error at ports/cuda/vcpkg_find_cuda.cmake:69 (message):
Could not find CUDA.  Before continuing, please download and install CUDA
(v10.1.0 or higher) from:
 https://developer.nvidia.com/cuda-downloads

You need download CUDA and install it.

Duplicate of #41834.


Age picker shows 18 by default but no age gets selected if we don't click on it on choose page

Duplicate of Issue #687


pngstream stops sending new images after sometime

Hey!
I noticed that, if the node program is running for some 15-30 min, after that time it start sending the same image again and it does not refresh the new image.
Is this a known image?

Can you please provide more details? What are you using to display the image stream?

I used the sample program where it streams to local host 8080.
Thanks for your support
Jeff

On 23-Mar-2017, at 12:05, Haskman<EMAIL_ADDRESS>wrote:
Can you please provide more details? What are you using to display the image stream?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I'm getting the same issue. Running on node on MacOS Sierra. The stream stops after a few minutes. No errors are thrown.


Re-enable stack trace tests

Depends on

[ ] #245

Definition of Done
Port Hardhat stack trace JSON files to the new EDR format, resulting in passing tests. Also, see README.
WIP
list the JSON files that you're currently working on or have already completed porting

This is spread over the issues in https://github.com/NomicFoundation/edr/issues/246, as part of the way we are going to implement those (gradually migrating the relevant tests).


handle reserved type fields for forward compatibility

handle reserved type fields for forward compatibility

Type has 64bit, why there could be only 16 types?

@sgaragagghu
	union {
		struct {
			union {
				struct {
					uint64_t transport_header;
					uint64_t type;
					xdag_time_t time;
				};
				xdag_hashlow_t hash;
			};
			union {
				xdag_amount_t amount;
				xdag_time_t end_time;
			};
		};
		xdag_hash_t data;
	};
};

the filed type in xdag_field has 64bits, and only the header xdag field represents the types of the fields in one block. Each 4 bits represent the type of one xdag field in block, each block has 16 xdag fields, so the type field in header have to represent 16 xdag field.

The first f means the type of the first xdag field. The second f means the type of the second xdag field.
And 4bits can only represents 16 types. That's why just handle 16 types in code.

Good.
But you can also do:
case XDAG_FIELD_RESERVE0:
case XDAG_FIELD_RESERVE1:
case XDAG_FIELD_RESERVE2:
case XDAG_FIELD_RESERVE3:
case XDAG_FIELD_RESERVE4:
case XDAG_FIELD_RESERVE5:
case XDAG_FIELD_RESERVE6:
break;

it will let everyone to do a network hard fork, my bad that i didn't think about that first time i saw this PR


jcarousel have support dynamic add or remove an item?

Currently, my implementation is using empty after append jcarousel.
jcarousel have support dynamic add or remove an item?

Yes, you can add/remove items. Be sure to call reload afterwards.


Assertion failed in Windows

I have encountered the panic described in #231 (but different test case). The cause is that IAudioCaptureClient::GetBuffer method may return AUDCLNT_S_BUFFER_EMPTY, which is success HRESULT, and make the buffer null.
ref: IAudioCaptureClient::GetBuffer | Microsoft Docs

Why is it not merged yet?

Thanks @azyobuzin and sorry about the delay! This looks fine to me, I'm happy to merge.


Automatically deploy from master

We might be able to use deploy-to-deta (github repo)
Recommended by deta themselves: https://github.com/deta/awesome-deta#github-action

This is done automatically now that we've changed to Vercel for hosting


Issue #SB-1376 and Issue #SB-1370 fixed in Release 1.5

@harishgilimi
Issue #SB-1370 and Issue #SB-1376 fixed

@vbrajath  fix the review comments


Info - API limit per User

Hi,
with this package can I set a limit requests for each user or for each type of user?
Example:
Free account : 1.000requests per day
Paid account:  10.000 requests per day
Exclusive account: 100.000 requests
and so on
Thanks!

Hi @ChristianGiupponi
You could probably do this yourself within your controller method (or make a middleware). I don't see a way of doing this within the $apiMethods variable. You could, however, use the api_logs table to check for the number of requests the API key has made.


Fix Linux compiler errors

Most issues were due to case-sensitive naming on Unix-y platforms like Linux.
I also fixed a typo while I was there. It does not currently link, it throws pages of complaints about undefined references to TransformMatrix, DrawBatchData, etc.

I have been wanting someone to try running this on other operating systems. I knew there were probably errors like this but haven't been able to check for them. Also, very good timing on the linking fixes because I think they will solve a problem I am currently facing with my CI system. Thank you!


Implement Add to Home in checkout confirmation page

See #160 for wireframes
PWA add to home component implemented already
Let's add it to checkout confirmation page

Hi @marlass, can you please add some smoke test instructions on how to validate the happy path for this feature?
Also, it looks like a mobile platform centric feature.  Please specify if we can test this on a pc somehow or if it's just better to use a mobile device.
Thanks!

About the device
Mobile devices have the best support for add to home screen, however desktop browsers start to catch up. On Windows 10 in Chrome this feature also works. It will move the tab to new window that is styled according to our branding. Additionally, it will be added to start menu and as a icon on desktop. I tested it mostly on this setup and it works 👌
How to test it

run yarn build:core:lib
run yarn build
run yarn start:pwa
go through the checkout and after placing an order you should be able to see the banner (if it isn't there and you don't have app installed you can manually invoke the browser event for it (Devtools -> Application tab -> Manifest -> Add to homescreen ~ top right corner)
after confirming "Add to homescreen" modal banner should dissappear (link in mobile menu also) and message about successfull install should pop below header

thanks Marcin! someone from montreal will review, might not necessarily be Patrick.

New PR: https://github.com/SAP/cloud-commerce-spartacus-storefront/pull/346


Fixes the link to the monitored hashtag

The link in the leaderboard header was wrong and hardcoded, moved to a dynamic link using ng-href.
@Particular/hashbus-committers please review

:ok_hand: @mauroservienti


[DOC-7932][DOC-7933] Document cluster-edit, horizontal and vertical scaling on Azure

[DOC-7933] Document horizontal scaling support on Azure
[DOC-7932] Document vertical scaling on Azure
Previews
releases/cloud.md
src/current/cockroachcloud/cluster-management.md
src/current/cockroachcloud/cockroachdb-dedicated-on-azure.md
Tests

[x] Local build
[x] Linkcheck

Double LGTM for merging this once this week's extraordinary release is out.

FYI, I'm in the process of prepping the extraordinary release, and we're on track to deploy tomorrow morning or afternoon, EST. Here's the tracking card - i haven't made a slack channel yet.


Font and vertical align

Bug, feature request, or proposal:
bug / feature request
What is the expected behavior?
From controls should have the same font size, font weight and have a matching baseline by default when placed side by side.
What is the current behavior?

What are the steps to reproduce?
http://plnkr.co/edit/I1cx3e?p=preview
What is the use-case or motivation for changing an existing behavior?
Need a consistent look and feel without much custom css overrides as no customer would except such an inconsistant ui.
I think most devs should need the typography part quite fast as lots of people are already building apps with the beta. When using a beta bugs and breaking changes are common. But such a basic consitency is an important must have in the very early stages of a frontend.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
ng 4.1.x, md beta 5
Is there anything else we should know?
I'm totally impressed that almost everything I used by now works in IE without the need of extra hacks. 👍

This is tracked via #2124


Does it support Android?

This is a very cool text-to-speech project!
I got it working on Windows, but I am struggling with Android. I suppose it is not supported, can you please confirm?
Thanks!

Well, I think you can get it working if you build it for android, the problem is that the original lib doesn't have dll build target in the CMake and I had to add it, with it I guess you can to build it for arm v7 with android NDK and drop the library to the plugins

hello, i am facing the same issue, i need to have the plugin for android (building for oculus) but i after sevral days i still can't use cmake to create it, have u done it?

@adem-rguez I am also building for Oculus, I ended up using wit.ai from Meta. It has text-to-speech with different voices, and it also supports language understanding. It's pretty cool, but it requires internet access.

lol, i was using wit as well, but ended up canceling that due to plugin issues and also the unability to stream audio, but for piper i managed to implement a streaming solution in unity, that's why


Resuming a HLS live stream on IOS

Hi,
I'm using react-native-video to play a HLS live stream served by  wowza streaming cloud.
It works great and the lag from live is about 12 seconds which is reasonable given that the video has to be encoded and pushed to CDN.
When the app goes to background and returns to the foreground however, the player tries to resume playback from the time the app went to background.
This causes the lag to increase and the playback never catches up. If the app was in the background for 15 seconds then the lag from live jumps from 12seconds to about 35seconds and never catches up.
Is there a hook to reset playback of the live stream so the player resumes from the latest available chunk and does not try to request older chunks of the HLS stream?
Thanks

Also having this issue, let me know if you can figure it out. I've been trying to setup amlst which will allow adaptive bitrate streaming but I'm getting an 'error: %0', { error: { domain: 'AVFoundationErrorDomain', code: -11800 },
target: 486 } error

Also facing similar issue. How do we set the paused prop when AppState was inactive or background? Appreciate it.


Quix-46319: Update quix-samples code segments to 0.5.6

Update dotnet project samples to use version 0.5.6 of SDK.
Update dotnet projects to latest net7.0.
Update python projects to use version 0.5.6 of SDK.

we need to update this to 0.5.6

Pending testing

Update: should use 0.5.7, as 0.5.6 has a python flag incorrectly set and by default doesn't start consuming without setting it explicitly to true.