[🐛 Bug]:  Type error: Conversion of type 'ProcessEnv'

next-on-pages environment related information
No response
Description
As specified here: https://developers.cloudflare.com/pages/framework-guides/deploy-a-nextjs-site/#use-bindings-in-your-nextjs-application
The build fails with:
17:28:19.027 | ▲  ./src/app/api/hello/route.ts:11:24
-- | --
17:28:19.027 | ▲  Type error: Conversion of type 'ProcessEnv' to type '{ MY_KV: KVNamespace; }' may be a mistake because neither type sufficiently overlaps with the other. If this was intentional, convert the expression to 'unknown' first.
17:28:19.027 | ▲  Property 'MY_KV' is missing in type 'ProcessEnv' but required in type '{ MY_KV: KVNamespace; }'.
17:28:19.028 | ▲
17:28:19.028 | ▲     9 \|
17:28:19.028 | ▲    10 \| 	// the type `KVNamespace` comes from the @cloudflare/workers-types package
17:28:19.028 | ▲  > 11 \| 	const { MY_KV } = (process.env as { MY_KV: KVNamespace });
17:28:19.028 | ▲       \| 	                      ^
17:28:19.028 | ▲    12 \|
17:28:19.029 | ▲    13 \| 	console.log(MY_KV);
17:28:19.029 | ▲    14 \|
17:28:19.108 | ▲  Error: Command "npm run build" exited with 1

Reproduction
No response
Pages Deployment Method
Pages CI (GitHub/GitLab integration)
Pages Deployment ID
No response
Additional Information
No response
Would you like to help?

[ ] Would you like to help fixing this bug?

@Mecanik Thanks so much for the issue! 😄
I'm fixing this in our Cloudflare docs repo: https://github.com/cloudflare/cloudflare-docs/pull/11971
Please have a look at my PR and let me know if you're happy with the changes 🙂
Besides that I'm closing this issue as I believe this to be a documentation mistake and not an actually issue within the next-on-pages repository, please feel free to reopen it if you disagree

@dario-piotrowicz Thanks, to be honest this was quite time wasting until I found the actual solution. Luckily I found this: https://socket.dev/npm/package/@cloudflare/next-on-pages-next-dev meanwhile and I figured it out, but the docs confusing users is very bad.
You know how it is... "people don't read the docs"; well, now if people read the docs they get confused and nothing works. There is so much more wrong with the docs, for example the fact that if you follow the instructions here: https://developers.cloudflare.com/pages/framework-guides/deploy-a-nextjs-site/#deploy-via-the-create-cloudflare-cli-c3 the project will be built on your side and uploaded as static files - no Edge runtime. However if you connect the project via Github and configure the build normally on the dashboard, the project builds as it should. And so on...

Also, the comment make sure to include the package in your tsconfig.json's "types" field is not really valid; I have no such thing in my tsconfig.json, I just installed the npm package @cloudflare/workers-types.

Also, the comment make sure to include the package in your tsconfig.json's "types" field is not really valid; I have no such thing in my tsconfig.json, I just installed the npm package @cloudflare/workers-types.

The env.d.ts file uses the workers-types types, and in order to do that those need to be declared in the tsconfig.json file like so:

The comment there was meant to remind people of the above without going too much into details and getting too verbose, but it looks that it is not really clear (and it does for sure assume a certain familiarity with this sort of setup), I'll update that, thanks so much for the very valuable feedback 🙏

@dario-piotrowicz Thanks, to be honest this was quite time wasting until I found the actual solution. Luckily I found this: https://socket.dev/npm/package/@cloudflare/next-on-pages-next-dev meanwhile and I figured it out, but the docs confusing users is very bad.
You know how it is... "people don't read the docs"; well, now if people read the docs they get confused and nothing works. There is so much more wrong with the docs

Yes, you're right, sorry about that, I'm sorry about the time wasting, I'll look into improve our docs 😓

for example the fact that if you follow the instructions here: https://developers.cloudflare.com/pages/framework-guides/deploy-a-nextjs-site/#deploy-via-the-create-cloudflare-cli-c3 the project will be built on your side and uploaded as static files - no Edge runtime. However if you connect the project via Github and configure the build normally on the dashboard, the project builds as it should.

Sorry could you clarify what you mean by saying that with C3 the project will be build and uploaded as static files - no Edge runtime? That's definitely not the case 😕

Sorry could you clarify what you mean by saying that with C3 the project will be build and uploaded as static files - no Edge runtime? As far as I can tell that's definitely not the case 😕

Just try to follow the documentation and deploy the project from wrangler. In my case, it uploaded as static (no edge functions) + I had to add the node_compat variable to the project myself on the dashboard and re-deploy again.
The only way I got the project working was to create the project as in the docs (npm create cloudflare@latest my-next-app -- --framework=next) and then create a repo, upload, create the project via dashboard, connect the repo and let it build.

Also, the comment make sure to include the package in your tsconfig.json's "types" field is not really valid; I have no such thing in my tsconfig.json, I just installed the npm package @cloudflare/workers-types.

The env.d.ts file uses the workers-types types, and in order to do that those need to be declared in the tsconfig.json file like so:
The comment there was meant to remind people of the above without going too much into details and getting too verbose, but it looks that it is not really clear (and it does for sure assume a certain familiarity with this sort of setup), I'll update that, thanks so much for the very valuable feedback 🙏

Thanks, I haven't done this; I just installed the npm package @cloudflare/workers-types lol.

Thanks, I haven't done this; I just installed the npm package @cloudflare/workers-types lol.

No problem 😄, as I said it's my fault for having that unclear comment there (again thanks for pointing that out!)
Do you find this clearer? https://github.com/cloudflare/cloudflare-docs/pull/11971/commits/f0e7a6740a057714689f3be6b938b0dceac89699

Just try to follow the documentation and deploy the project from wrangler. In my case, it uploaded as static (no edge functions) + I had to add the node_compat variable to the project myself on the dashboard and re-deploy again.

This is really not right, when you create a next application with npm create cloudflare@latest it will:

add edge runtime logic (that you can see in the api/hello route, like here)
add the nodejs_compat flag to the project for you (we do have specific logic for that, here we declare the flag needed for Next.js and here we create the project with such flag)

Could you please have another try, document everything you're doing and if you still see the same issues open a new issue in the workers-sdk repository (where the create cloudflare code lives)? 🙏
(If you do please tag me there and I'll take it from there 🙂)


feat: added status of calculating data.

The dropzone should also allow gzipped or bzipped files .xml.gz and .xml.bz2.


feat(templates/react-ecs): use a namespace 🏎

Closed in favor of a single PR with all the templates.


Typos in MSL documentation

Reported by leo.gall on 17 Jul 2012 16:54 UTC
At the last design meeting in Lund we discussed quality of MSL.
I started to work on a topic which doesn't have any impact on model results but still is important for the "quality impression" of model users.
Please find attached a first SVN patch.
For the contained files, I ran through info layer, comments and parameter descriptions.
I think that especially parameter descriptions are important to be free of typos because they appear in various screenshots etc.
If library officers don't object, I would commit these changes to trunk and branches/maintenance/3.2.1
Alternatively, library officer could apply the appended patch.

Migrated-From: https://trac.modelica.org/Modelica/ticket/777

Modified by dietmarw on 18 Jul 2012 06:51 UTC

Comment by anonymous on 20 Jul 2012 07:22 UTC
A first set of corrections has been committed to
trunk in 786a6b64f26183f6b89d24962e5e76922a51b153 and
branches/maintenance/3.2.1 in r5307
Thus, the attached file [attachment:trunk.patch] "patch fixing typos in trunk r5298" is obsolete.

Comment by leo.gall on 24 Jul 2012 21:07 UTC
I appended the next patch [attachment:Typos_3.2.1dev_r5316.patch] for branches/maintenance/3.2.1.
It contains typo fixes for
Modelica.Blocks
Modelica.Electical
Modelica.Mechanics.Rotational
Modelica.Mechanics.Translational
(34 files modified)
If nobody objects, I'll commit the changes on Friday, 2012-07-27.

Comment by leo.gall on 27 Jul 2012 08:11 UTC
The next set of corrections has been committed to
branches/maintenance/3.2.1 in r5324 and merged to
trunk in 5208de1bbb9e13b5f4625d976d6a8c042713bf26.
Thus, the attached file [attachment:Typos_3.2.1dev_r5316.patch] is obsolete.

Modified by dietmarw on 20 Dec 2012 13:44 UTC

Changelog removed by dietmarw on 20 Dec 2012 13:44 UTC

Comment by otter on 26 Mar 2013 15:58 UTC
Seems it was forgotten to close this ticket.


Create Service Instance for Document Information Extraction

Tutorial URL: https://developers.sap.com/tutorials/cp-aibus-dox-service-instance.html
Please specify the step you are referring to
In Step 3:
In the entitlements, it says "Standard plan for DOX application" although DOX is not an official name approved by one voice and should not be used externally. Change to an one voice approved name.

Hi Daniel,
I'm not able to find "Standard plan for DOX application". Can you please send me a screenshot?
Thank you,
Juliana

It is in Step 2, "Select Document Information Extraction, and choose the default service plan. Click Add 1 Service Plan."

Hi Daniel,
Now I see it. The service entitlements metadata will need to be updated so we have "Default standard plan" instead of  "Standard plan for DOX application". I'll follow up with Sebastian on that.
Best regards,
Juliana


Comments which are posted once, and appear on the public site once, are visible several times in the Admin Panel Comments area, and in Community Comment Moderation

Observed on internetofgoodthings.org when commenting on https://www.internetofgoodthings.org/en/sandbox/djibouti-youth-ambassadors-against-fgm/youth-against-fgm-community-forum-article/introduce-yourself-section/introduce-yourself/
On the public site:

In Admin Panel comments:

In Community Comment Moderation:

It is not clear from the admin panel that two of the three duplicated comments are marked as 'is removed' - this can be seen from the edit page for each comment. The public-facing site does take this into account and filters out the two comments that are marked as removed and that is why we only see one instance of it there.
The admin panel comments area shows comments regardless of whether 'is removed' is set. The filter option "By flagged for removal" is linked to flags on comments, which are separate from the 'is removed' field. Likewise, the community comment moderation feature is not taking 'is removed' into account when showing comments.
The comments features of IoGT are composed of several comment modules (django_comments, django_comments_xtd, IoGT comments) which are presenting a confusing mix of somewhat overlapping mechanisms to show/hide comments. The situation might be improved by making the following changes:

treat the 'is removed' field as an indication of whether a comment is deleted; don't show deleted comments anywhere, by default
add a filter to the admin panel comments area to allow deleted comments to be displayed, if required
stop showing deleted comments in Community Comment Moderation


Enable auto code detector

hello. how can i enable auto code detection then past fromVisual studio code ?

I don't understand what you mean.
Component can't detect the code in the content.

example past code from vsc to milkdown editor
https://user-images.githubusercontent.com/61819835/205444800-5010434b-6e6f-42c3-8434-2ad158cf76c4.mp4

Got it. Thanks for your video. I'm trying to achieve this feature.

Sorry, I've tried, but no way to eliminate differences between code and text. And same with other editors. This feature will be discarded

others have been able to do it and you haven't... ((((

others have been able to do it and you haven't... ((((

Yes, But they have serious problems. Although they could detect the code, but if we copy simple words from vscode and patse to editor, editor still treats it as code. Even for component used in my company.
It's easy to design in API e.clipboardData.getData('vscose-editor-data'). But we should consider the availability.

All right, I'll add a state to editor, developer Developers adjust this state to control detecting code or not. Way to adjust state is controlled by the developers

This feature has been added in >2.6.0, need to set autoDetectCode to true.


TypeError: Cannot read property 'token' of null

Hi there! I found your project on twitter and was very interested to try it out. So I tried it out locally. This is my (very basic) editor (index.html)
<script src="https://unpkg.com/handel-pl"></script>
<textarea id=music style="width:100%;height:50%">
start
	chunk example using somePlayable
		play somePlayable
		rest for 1b
	endchunk
	save myPlayable = Eb3 for 1b
	run example using myPlayable with sound piano, loop for 5
finish
</textarea>
<script>
function play(){
	console.log(music.value)
	Handel.RunHandel(music.value);
}
</script>
<button onclick="play()">Play</button>

Now I get similar errors as on the online editor, so there's something wrong here? would be nice if the error reporting was bit more specific what is wrong.

After few other tries I noticed its the examples that result in errors.
This one (from website) doesnt work: handel-pl:2 Uncaught (in promise) Error: error parsing input at line 1
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration

This one (from examples on github) also doesnt work: TypeError: Cannot read property 'token' of null
start
	chunk main
    	play D3, F3, G3, B3 for 1b
       	play F3, A3, C3, E3 for 1b
       	play C3, E3, G3, B3 for 1b
       	play D4, F3, G4, B3 for 1b
       	play F4, G3, B4, D3 for 1b
       	play D4, F3, G4, B3 for 1b
       	play F4, G3, B4, D3 for 1b
       	play D4, F3, G4, B3 for 1b
    endchunk

    chunk kickdrums
    	play E1 for 1b
        rest for 3b
    endchunk

    chunk hats
    	play E4 for 1b
    endchunk

    run main with sound piano, loop for 10
    run kickdrums with sound kick, loop for 20
    run hats with bpm 280, sound hihat, loop for 200

finish

After few other tries I noticed its the examples that result in errors.
This one (from website) doesnt work: handel-pl:2 Uncaught (in promise) Error: error parsing input at line 1
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration

This one (from examples on github) also doesnt work: TypeError: Cannot read property 'token' of null
start
	chunk main
    	play D3, F3, G3, B3 for 1b
       	play F3, A3, C3, E3 for 1b
       	play C3, E3, G3, B3 for 1b
       	play D4, F3, G4, B3 for 1b
       	play F4, G3, B4, D3 for 1b
       	play D4, F3, G4, B3 for 1b
       	play F4, G3, B4, D3 for 1b
       	play D4, F3, G4, B3 for 1b
    endchunk

    chunk kickdrums
    	play E1 for 1b
        rest for 3b
    endchunk

    chunk hats
    	play E4 for 1b
    endchunk

    run main with sound piano, loop for 10
    run kickdrums with sound kick, loop for 20
    run hats with bpm 280, sound hihat, loop for 200

finish

@markknol thank you for submitting this issue! It seems to be a bug in the latest version of Handel v0.5.0 (which added new types). I think the issue is specific to rests. I'll look into more into this bug asap!

@markknol thank you for submitting this issue! It seems to be a bug in the latest version of Handel v0.5.0 (which added new types). I think the issue is specific to rests. I'll look into more into this bug asap!

@markknol The bug is now addressed in commit d51586c. You should not come across the issue in the latest version of Handel which is now published (version 0.5.1).
Regarding what the bug was. Rests are treated similar to plays by the Handel symbol table builder. But with rests some nodes to visit in the relevant part of the syntax tree will be null or undefined. Checks for that case were missing.

@markknol The bug is now addressed in commit d51586c. You should not come across the issue in the latest version of Handel which is now published (version 0.5.1).
Regarding what the bug was. Rests are treated similar to plays by the Handel symbol table builder. But with rests some nodes to visit in the relevant part of the syntax tree will be null or undefined. Checks for that case were missing.

Additionally (looking at your follow up), some examples in the Handel documentation are full programs (all the ones that can be ran in the webpage). But some are just code snippets rather than full programs.
For example
This is not a complete Handel program (it does not have the start and finish keywords):
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration

But this is:
start
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration
finish

Additionally (looking at your follow up), some examples in the Handel documentation are full programs (all the ones that can be ran in the webpage). But some are just code snippets rather than full programs.
For example
This is not a complete Handel program (it does not have the start and finish keywords):
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration

But this is:
start
save myplayable = E4, F4, G3 for 3b
save myduration = for 1b

play myplayable
rest myduration
finish

Lastly, you mentioned more verbose errors. The errors you mentioned are Lexer errors or errors involving invalid characters --   version (0.5.2) now adds clearer messaging for those errors (they were missing line numbers as well). So an invalid program like this:
start
*
finish

Would now produce an error like this:
error analyzing input at line: 2; invalid character: *

Lastly, you mentioned more verbose errors. The errors you mentioned are Lexer errors or errors involving invalid characters --   version (0.5.2) now adds clearer messaging for those errors (they were missing line numbers as well). So an invalid program like this:
start
*
finish

Would now produce an error like this:
error analyzing input at line: 2; invalid character: *

Thanks for looking into this! 🎉

Thanks for looking into this! 🎉


workaround for date matching on differentially tokenized queries

Coverage decreased (-0.05%) to 81.741% when pulling 4dca07ef68cf51bc0525ec28d2d4e257ff2d1b33 on bugfix into 4c36394ba936337e24f86c6b5ed1cf1d62b568d9 on develop.

@eawoods @mhl10 Ok, I think that I may have it figured out. Pushed a few more commits to this branch. Will continue to test.

Added fixes for #183, #135, #145

ARXIVNG-1012 - Inconsistent highlighting with wildcard search
ARXIVNG-1013 - Weird higlighting for "theory" when searching all fields
ARXIVNG-1014 - "More" toggle is still shown for abstracts whose entire contents are already shown


Display registered events in account profile.

blocked by #47

This will be unblocked once #42 is merged.

Also #71

Handled in https://github.com/djangonaut-space/wagtail-indymeet/pull/77


Actually detect the existing lint failures

Currently, make lint doesn't detect any problems, apparently because
golint internal/activity.go doesn't report any issues.
However, if you golint internal you get multiple results, and you get
the same if you golint internal/activity.go internal/other_files.go.
Why golint fails like this, I have no idea.
The fix though is to call golint with a whitelisted set of go files,
per directory (on the assumption that every directory is a package).
Make doesn't make this particularly easy, but it's at least possible.
$ make lint
./internal/activity.go:186:9: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:48:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:66:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:85:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:103:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:121:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:136:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:149:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:179:8: should not use basic type untyped string as key in context.WithValue
./internal/internal_task_pollers.go:352:9: should not use basic type untyped string as key in context.WithValue
Found 10 lint suggestions; failing.
make: *** [lint] Error 1

It even retains the same "run everything, fail if any fail" behavior:
$ git diff
diff --git a/client/client.go b/client/client.go
index d2d4183..8240de8 100644
--- a/client/client.go
+++ b/client/client.go
@@ -53,8 +53,6 @@ type (
        // WorkflowIDReusePolicy defines workflow ID reuse behavior.
        WorkflowIDReusePolicy = internal.WorkflowIDReusePolicy

-       // Client is the client for starting and getting information about a workflow executions as well as
-       // completing activities asynchronously.
        Client interface {
                // StartWorkflow starts a workflow execution
                // The user can use this to start using a function or workflow type name.

results in
$ make lint
./client/client.go:56:2: exported type Client should have comment or be unexported
Found 1 lint suggestions; failing.
./internal/activity.go:186:9: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:48:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:66:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:85:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:103:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:121:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:136:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:149:8: should not use basic type untyped string as key in context.WithValue
./internal/activity_test.go:179:8: should not use basic type untyped string as key in context.WithValue
./internal/internal_task_pollers.go:352:9: should not use basic type untyped string as key in context.WithValue
Found 10 lint suggestions; failing.
make: *** [lint] Error 1

 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.

I'm not sure at the moment what should be done for the lint failures, but figured this is an effective way to display them :)


Don't merge yet ... [ZEPPELIN-4796] Better representation of string-like data in Cassandra interpreter

What is this PR for?
Refactoring of the Cassandra interpreter to a new driver changes the results representation to be strict CQL format, that is not always desirable.
This PR improves the situation by allow to control the output format (CQL vs human-readable), and also control such things as locale, timezone, format of timestamp/date/time columns, and precision for float & double types.
All settings could be set either on interpreter, or cell level.
What type of PR is it?
Improvement
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4796

How should this be tested?

https://travis-ci.org/github/alexott/zeppelin/builds/695625373
Tested manually (see screenshot)

Screenshots (if appropriate)

Good question is what better - should we use local properties of interpreter instead of the properties specified on the cell level & handled by parser? like,
%cassandra(locale=...., timezone=...)

I think local properties is better, so that we can do it in zeppelin-server side and make this feature available to all the interpreters, otherwise each interpreter need to implement this feature separately.

@zjffdu it's ready for review... It still needs to implement code interpreter-side, as we need to "know" data type information, but at least we can have some common properties, etc. I'm thinking about creating followup ticket for refactoring it for supporting most interpreters.


Copy and Paste Disabled by Default

Hello. Copy and paste doesn't seem to work. I've read that you need to enable it with Electron, but I was thinking most people will probably want it on by default with your project.
Thank you!

please explain copy paste where from where 🤔🤔🤔???

When you test the app it works fine, but when you make a built version copy and paste functionality doesn't work. See: https://pracucci.com/atom-electron-enable-copy-and-paste.html

I am still figuring out, i tried fixing this issue. But no menu was showing neither on testing nor on build.
If somebody can fix this and send me the code then i would add it.

@ryanpamplin  I fixed it by following code and updated in master. So please let me know is it working as you feel it to be or not. :)
///////////////////////////////

app.on('ready', () => {
  createWindow()
  if (process.platform === 'darwin') {
    var template = [{
      label: 'FromScratch',
      submenu: [{
        label: 'Quit',
        accelerator: 'CmdOrCtrl+Q',
        click: function() { app.quit(); }
      }]
    }, {
      label: 'Edit',
      submenu: [{
        label: 'Undo',
        accelerator: 'CmdOrCtrl+Z',
        selector: 'undo:'
      }, {
        label: 'Redo',
        accelerator: 'Shift+CmdOrCtrl+Z',
        selector: 'redo:'
      }, {
        type: 'separator'
      }, {
        label: 'Cut',
        accelerator: 'CmdOrCtrl+X',
        selector: 'cut:'
      }, {
        label: 'Copy',
        accelerator: 'CmdOrCtrl+C',
        selector: 'copy:'
      }, {
        label: 'Paste',
        accelerator: 'CmdOrCtrl+V',
        selector: 'paste:'
      }, {
        label: 'Select All',
        accelerator: 'CmdOrCtrl+A',
        selector: 'selectAll:'
      }]
    }];
    var osxMenu = Menu.buildFromTemplate(template);
    Menu.setApplicationMenu(osxMenu);
  }
})


fix set gateway in advanced networking

When a gateway was set in advanced networking, it did not happen in vCloud.

@frapposelli any reason this hasn't been merged?


Switch to fxhash

The profile still shows a lot of time spent in is_on_list, but now it's time spent calculating hashes. The default hasher in the standard library uses a hash function that is cryptographically secure (sip hash). Since this is not needed for this use case we can use an alternative hasher with better performance. Using fxhash yields about a 30% speed up on the benchmark for me.

Nice! I'm also seeing a 30% decrease in benchmark time with this change.

The profile still shows a lot of time spent in is_on_list

I've recently removed that function in favor of the built-in contains (I was able to make that switch in part because of your HashSet changes from earlier). Assume those contain calls are still computationally expensive, but I'm not sure how to improve them.

Assume those contain calls are still computationally expensive, but I'm not sure how to improve them.

They're not super expensive, especially when using this alternate hash function but it's called a lot so it adds up.


DEPLOY SCRIPT TESTS

DEPLOY SCRIPT TESTS

[x] #98
[x] #99
[x] #100

Originally posted by @anna-carroll in https://github.com/nomad-xyz/monorepo/issues/96#issuecomment-1084660697

above three checks were accomplished


Intermittent test failures

Fascinatingly the changes I suggested in #2 highlighted intermittent errors using php7.2.2 on Windows 10 64-bit
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

.............................                                     29 / 29 (100%)

Time: 591 ms, Memory: 4.00MB

OK (29 tests, 68 assertions)

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

.............................                                     29 / 29 (100%)

Time: 552 ms, Memory: 4.00MB

OK (29 tests, 68 assertions)

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

.............................                                     29 / 29 (100%)

Time: 571 ms, Memory: 4.00MB

OK (29 tests, 68 assertions)

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

....................F........                                     29 / 29 (100%)

Time: 544 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\HelpersTest::testWaitCap
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\HelpersTest.php:66

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

....................F........                                     29 / 29 (100%)

Time: 612 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\HelpersTest::testWaitCap
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\HelpersTest.php:66

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

..........F..................                                     29 / 29 (100%)

Time: 559 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\BackoffTest::testWait
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\BackoffTest.php:151

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

....................F........                                     29 / 29 (100%)

Time: 553 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\HelpersTest::testWaitCap
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\HelpersTest.php:66

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

..........F..................                                     29 / 29 (100%)

Time: 558 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\BackoffTest::testWait
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\BackoffTest.php:151

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

.............................                                     29 / 29 (100%)

Time: 573 ms, Memory: 4.00MB

OK (29 tests, 68 assertions)

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

..........F..................                                     29 / 29 (100%)

Time: 554 ms, Memory: 4.00MB

There was 1 failure:

1) STS\Backoff\BackoffTest::testWait
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\BackoffTest.php:151

FAILURES!
Tests: 29, Assertions: 68, Failures: 1.

lewis@ASUS-LAPTOP MINGW64 ~/Desktop/backoff (master)
$ vendor/bin/phpunit
PHPUnit 5.5.7 by Sebastian Bergmann and contributors.

..........F.........F........                                     29 / 29 (100%)

Time: 544 ms, Memory: 4.00MB

There were 2 failures:

1) STS\Backoff\BackoffTest::testWait
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\BackoffTest.php:151

2) STS\Backoff\HelpersTest::testWaitCap
Failed asserting that false is true.

C:\Users\lewis\Desktop\backoff\tests\HelpersTest.php:66

FAILURES!
Tests: 29, Assertions: 68, Failures: 2.

I committed my changes to a separate branch, then ran git checkout master && git reset --hard. Looks like there is a small problem with the wait tests

I believe I have a fix, PR upstream in a few moments


Clarify claiming a namespace

I'm new to the process of creating orbs. While going through the docs, I came across the circleci namespace create command and found the parameters confusing. After a while of digging, I found that I could run circleci namespace create --help to understand what the params meant. It would have been easier if there was something right there in the documentation that gave a short explanation.

thanks ! @trotzig i filed a new PR w/ a slightly modified version of this https://github.com/circleci/circleci-docs/pull/3005


Only attempt to shut down a Store if we opened one

Only attempt to Shutdown() the default storage.Store if we actually opened it to begin with.  This avoids some unnecessary churn with storage in cases where argument parsing flags an invocation error.

@rh-atomic-bot r+ 5152a04

 [Buildah] build-dockerfile


Improve coverage

Hi @tsamaya,
thanks for this! I really like it to have an improved test coverage. I added some minor line comments. would you mind addressing them? After that I think this is good to go.

Hi @chrismayer,
I pushed a new commit with API Docs and a test for getLanguage

Thanks for your ongoing effort on this, @tsamaya! :+1:  This is good to go now. I will merge this.


Why I would do this, I'm not exactly sure.


broken link in CONTRIBUTING.md

https://github.com/c4gnv/business-web/projects/1 results in a 404. This link is in Project Board. I don't know where to point the link otherwise I'd fix it myself.

Oh that's strange, I wonder if github deprecated that feature. I'll ask around to figure out what happened. Maybe @gotoplanb knows?

@Bjorn248 can you let me know if this issue is still relevant or whether we can close it?

@Bjorn248 can you let me know if this issue is still relevant or whether we can close it?

@jamessmell This repo seems like it has been abandoned. I think we can close it.

@jamessmell This repo seems like it has been abandoned. I think we can close it.

Closing

Closing


Updated the Create a Temporary Access Pass heading

Updated the required roles section.

Learn Build status updates of commit bb32b9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/authentication/howto-authentication-temporary-access-pass.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@tilarso
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off
Only minor updates being suggested for correct grammar.


check recommended up/downstream MR.P modules in resources of all READMEs

formulate one descriptive concise sentence per module and then distribute accrodingly across all READMEs.

or always refer to fitting recipes, so to only have one place to maintain the recommended MR.P modules?
but less crosslinking...

Have both.
Up- and downstream recommended/compatible modules
And
Recipes that use/include this module

Maximise cross linking


Make sure that uniform sampling respects broadcasting

Fixes #1644
When any of low or high are multidimensional and size is None, then the final shape is computed before sampling the values

/test

/test mini

I think the CI failures are related to NCCL and not to the diff changes

/test jenkins

@andfoy LGTM! Thank you for your PR!


chore: bdd-spring-1572874831 to 0.0.1

chore: Promote bdd-spring-1572874831 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Support JupyterLab 3

JupyterLab 3 is out. It's great if this plugin can be updated to be compatible with jupyterlab 3.

A lot of people +1'd this issue. Is there anyone here that can volunteer to maintain this extension, or at least submit a PR for updating this?

I will be submitting a PR to migrate this extension to JL3.


Limit time range dynamically

Heyhey, let me first say I love your plugin!
I'd really like to generate a graph that only shows the last n entries, is there any chance you could add something like
...
startDate: -n

?
Cheers

Great idea.
It would be very helpful if one wants to track, for example, the data in last month.
With this, no need to change the startDate every time one opens the tracker file.

This might do what I want. More specifically, I'd like to include natural language dates so that dynamically I can have Tracker show me data that meets these criteria. (Any form of date math for date ranges would work.) Possible?
searchType: frontmatter
searchTarget: Sleep
folder: Journal/
startDate: one month ago
endDate:
line:
    title: "Sleep"
    xAxisLabel: Date
    yAxisLabel: Value

Hi @eatondpe,
Do you use the plugin nldates-obsidian?
I am considering using the API it provides to help parsing natural language dates, but that will require users to install the plugin to use it.

Yes. I do use the nldates plug-in. I need it for a few other plug-ins, too.
On Sun, May 30, 2021 at 2:13 AM pyrochlore @.***> wrote:

Hi @eatondpe https://github.com/eatondpe,
Do you use the plugin nldates-obsidian
https://github.com/argenos/nldates-obsidian?
I am considering using the API it provides to help parsing natural
language dates, but that will require users to install the plugin to use it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pyrochlore/obsidian-tracker/issues/18#issuecomment-850948396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQCNZR7AIQIVAKWR44YOIV3TQHJRDANCNFSM4474SAXA
.

Thanks @eatondpe .
I will do more research and implement the feature.

Hi @eatondpe,
I have to apologize for the promise to you.
Regarding to the frequency of writing date fields in tracker code block, and the complexity of using/mixing natural language dates (NLD) with momentjs dates,  I decided not to use NLD for the date input.
To input a relative date to today,
I might use "-1d", "-3w", "-5m", "-1y" where d/w/m/y stands for day/week/moth/year.

No worries. Your suggestion would work just as well. Maybe even better!
On Tue, Jun 29, 2021 at 4:05 AM pyrochlore @.***> wrote:

Hi @eatondpe https://github.com/eatondpe,
I have to apologize for the promise to you.
Regarding to the frequency of writing date fields in tracker code block,
and the complexity of using/mixing natural language dates (NLD) with
momentjs dates, I decided not to use NLD for the date input.
To input a relative date to today,
I might use "-1d", "-3w", "-5m", "-1y" where d/w/m/y stands for
day/week/moth/year.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pyrochlore/obsidian-tracker/issues/18#issuecomment-870373892,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQCNZRZB4V3ZCPUTUMO2XATTVF5ELANCNFSM4474SAXA
.

The feature was just released with version 1.8.0.
Here is an example

Works beautifully! Thanks!
On Sun, Jul 4, 2021 at 6:27 AM pyrochlore @.***> wrote:

The feature was just released with version 1.8.0.
Here
https://github.com/pyrochlore/obsidian-tracker/blob/master/examples/TestDateFormats.md#relative-date-input-for-startdate-and-enddate
is an example
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pyrochlore/obsidian-tracker/issues/18#issuecomment-873561417,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQCNZR33XS5P2Z4HJECE553TWAZRLANCNFSM4474SAXA
.

Will close the issue.
Welcome to report more bugs or make suggestions.


send 'node.chosenAs' msg when a node is chosen as the collector or the witness.

Send the 'swarm.init' or 'node.init' after config the w3.swarm to sync the swarm between different clients. The data should be nodes' brief info in the swarm in order.
{ 'node.init', swarm.nodes.map(node => node.briefObj) }

Also need publicKeyString, to determine in which round the node is a witness.

@weihaopeng message added, checkout and close this issue if it's fine :)

@weihaopeng message added, checkout and close this issue if it's fine :)

I need to get swarm.config in swarm:init message. Because in swarm graph, a departure node is highlight and after LOCAL_UPPER_BOUND will auto downplay.

done


Enabled ESLint's "eqeqeq" rule.

https://eslint.org/docs/rules/eqeqeq

@jdufresne Thanks :+1:

buildbot, test on selenium.


Disable prefix / postfix for css className

Hi, thank you for MUI! I want to add styles to already existing classes inside the component. If I use the function makeStyles, prefixes / postfixes appear in the class names. For example
const useStyles = makeStyles(theme => ({
  root: {
    color: "red"
  }
}));

I get this class name at the output makeStyles-root-11
How can I remove the prefix / suffix in the name of the css class?

Tip: https://material-ui.com/styles/advanced/#jss-plugin-global


#17: Update react and fix PropTypes

[x] Update React to v15.5
[x] Update Radium to v0.19
[x] Replace React.PropTypes with package prop-types
[x] Use React.PureComponent instead of package react-pure-render
[x] Bump version

@ondrejbartas pls review


iPhone SE: Cannot add email address in contacts

Description
Adjust content inset when keyboard is visible
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-5575
Screenshots

When checking out this branch and testing on an simulator via xCode (iPhone SE, iOS 14.4), every time I try to enter something into the e-mail field the save button jumps above it.
Is this something with my environment/simulator?
Please forgive me if this is normal on an emulator, I'm quite new with working with Xcode, etc.

Screen recording
https://user-images.githubusercontent.com/67682506/110853591-169c5380-82b4-11eb-807a-156fb59687ce.mov

Fixes https://github.com/corona-warn-app/cwa-app-ios/issues/2185

When checking out this branch and testing on an simulator via Xcode (iPhone SE, iOS 14.4), every time I try to enter something into the e-mail field the save button jumps above it.
Is this something with my environment/simulator?
Please forgive me if this is normal on an emulator, I'm quite new with working with Xcode, etc.
Screen recording

@Ein-Tim Please re-test. I changed the VC to a UITableViewController which should now handle keyboard and scrolling a lot better without implementing hacky code.

Works as intended now @freshking. Thanks!


Improve rem_scalar/div_scalar  performance

Refer to:
Blog: https://lemire.me/blog/2019/02/08/faster-remainders-when-the-divisor-is-a-constant-beating-compilers-and-libdivide/
Paper:  https://arxiv.org/abs/1902.01961
Go: https://github.com/bmkessler/fastdiv
Rust: https://docs.rs/strength_reduce/0.2.3/strength_reduce/

Fuse processing number% 3 is slow. The main reason is that type conversion is made every time. I try to remove type conversion. The performance can be 2G -- > 5G / s

@lideen999
#252 shows the main reason cast.
I did some perfs in datafuse, seems the rem is the hot path.

@jorgecarleitao
By using strength_reduce, it shows great improvement in datafuse.

This is a valid request: valid use-case, documented benefits, crate with implementation available. 👍
Would you like to work on it, or would you like me to take it?
Also, I went through the crate and is unsafe free, so even easier sell.

The same optimization can be used for division as well right?

The same optimization can be used for division as well right?

Yes! Of course.

Would you like to work on it, or would you like me to take it?

Sorry, currently I may have no time to work on it, I just provide this idea to make arrow2 work better.

Done in #275


Allow the MQTT endpoint to serve different dialects

Currently the MQTT endpoint serves a Drogue Cloud specific MQTT dialect.
However, we also agree that we want to serve other dialects too:

https://github.com/drogue-iot/drogue-cloud/issues/90
https://github.com/drogue-iot/drogue-cloud/issues/98
https://github.com/drogue-iot/drogue-cloud/issues/222

As with MQTT 3.1 there is no indication a client could send to the server which dialect it want to "speak", we need another way to distinguish what the client wants. A simple way would be to simply create additional endpoints which serve one dialect each.
The downside is, that we already have 3 variants (MQTT with client certs, MQTT over WS with client certs, MQTT over WS without client certs). Each of them has a dedicated deployment. So having 4 different dialects, we would end up with 12 different services/deployments.
So, we do need a different way to solve this.

We do have that mechanism now. We still lack support for #90 and #98 though. Still, let's close this issue, as the capability itself is available now.


Configuration guide neede

Hello contributors,
I'm really interested in testing this vue-scaffold, I'm having some problems in understanding well the "custom-environment-variables.json" because I cannot find all the places where the variables in the files are used.
Just for example  I cannot find where "the frontend is mounted on the /app path".
Is there any example to help me?
thanks

The custom-environment-variables.json file provides a complete mapping of ENV variables to the config that the node application will see, and the default.json provides generic defaults for all non-sensitive config values. If you do a search for config.get(...) on the repository, you'll get a sense of how the configuration variables are utilized.
If you are running this on a local machine, you'll need to create a local.json file the values you want set in the same directory as custom-environment-variables.json. For more information on how the config library loads and searches for environment variables, take a look at this article: https://github.com/lorenwest/node-config/wiki/Configuration-Files.
At an absolute bare minimum, you will want your local.json to at least have the following values defined:
{
  "frontend": {
    "keycloak": {
      "clientId": "REDACTED",
      "realm": "REDACTED"
    }
  },
  "server": {
    "keycloak": {
      "clientId": "REDACTED",
      "publicKey": "REDACTED",
      "realm": "REDACTED",
      "clientSecret": "REDACTED"
    },
    "logLevel": "debug",
    "morganFormat": "dev"
  }
}

You may need more configured depending on your own personal environments. Hope that helps.

Thanks Jeremy.
Don't you think that the additional variables below are also necessary at a bare minimum?

keycloak.serverUrl
server.port
server.keycloak.serverUrl

Best Regards

Thanks Jeremy.
Don't you think that the additional variables below are also necessary at a bare minimum?
* keycloak.serverUrl

* server.port

* server.keycloak.serverUrl

Best Regards

Assuming you are using your own Keycloak provider, you will want to define your own keycloak.serverUrl and server.keycloak.serverUrl values instead of using one pre-defined for BCGov. The value server.port does not exist as that would be embedded as part of the serverUrls directly if necessary.


[hyperdrivepy] add close long, close short, calc present value, and update hyperdrive to 0.4.0

pypechain has some problems with linting, so I made issues & am ignoring the generated file for now.
https://github.com/delvtech/pypechain/issues/88
https://github.com/delvtech/pypechain/issues/89


Empty data for some kitties

It seems quite random, it fails with sometimes, but not always. I haven't found a reason or a general rule to make it fail.
In the value box it says: value: Name your price
In the traits: [object Object]

The devs updated the site with new info. Working on fixing.

Yes, I've just updated and it works :)
Now I cannot see the value, but that would be another issue.
Thank you 🙀


Feat/support report

Adds flow for supporting a report

@thebeyondr I pushed a commit 4a4dd3d612a3833c170f58c6429959e7e6e640de to fix hard-coded localhost:3000


fix: tmux allow-passthrough handling in image.nvim

Fixes #243

Update allow-passthrough check to include all value.

Modify lua/image/utils/tmux.lua to consider both on and all as valid values for allow-passthrough.

Thank you!


ActionHeader next-previous button-group buttons

Closes #1507

Don't render the next-previous button group if neither the onNext nor onPrevious prop is passed in.
Render the next-previous button group with both buttons enabled if both onNext and onPrevious are passed in.
If only one of onNext or onPrevious prop is passed in, render the next-previous button group with the appropriate button enabled and the missing action as a disabled button.

Verification Link:

https://terra-core-deployed-pr-1735.herokuapp.com/#/tests/terra-action-header/action-header/next-enabled-previous-disabled-action-header
https://terra-core-deployed-pr-1735.herokuapp.com/#/tests/terra-action-header/action-header/previous-enabled-next-disabled-action-header

Functional verification completed. The next/previous button group examples display and function correctly.


customizal xp drop and predicted hit and xp drop are not working for me while turned on

i have the plugins turn on and i attach to play and also do hide icons or icon on also , still not showing xp drops as i hit a npc or a player?????

Are your xp drops in-game hidden? Try toggling the xp orb next to the minimap


Error with rgbmatrix library

Hi,
When I try to run the python file via terminal, I always get this error:
pi@Malani:~/Desktop/rpi-matrix-gif-master $ sudo python displayGIF.py
Traceback (most recent call last):
File "displayGIF.py", line 18, in
from rgbmatrix import Adafruit_RGBmatrix
Is there something wrong with the rgbmatrix import?
Earlier, I had the same issue with the code "import Image" but was able to fix it by changing it to "from PIL import Image"
How can I fix this?
Thanks.

Hey there! I have the same problem, did u find a solution?

Thanks for your comments! This project is a few years old. Adafruit may have changed their API. I'll look into it! And thank you for checking out my repo.

I looked into the issue over the weekend and figured out what broke my script.
I originally wrote the script in 2016. Since then there have been some changes to the dependencies. Notably, Adafruit's fork of the rpi-rgb-led-matrix library is now deprecated. Adafruit's installer script now uses the rpi-rgb-led-matrix repo instead.
I'm working on updating my script to work with the new version of the rgbmatrix library. (I also had to troubleshoot some issues getting the library to build.) I'll post another update when I've fixed my script. Thanks!

Hey @Forianz, what @poemusica wrote is what I figured. This script is old and the Adafruit RGB repo has changed since.
Nice to hear that @poemusica will be updating this!
In the meantime, @Forianz, I ended up and recommend using this instead: https://github.com/DIYCharles/RGBGIFSlideshow
I faced a few issues but it worked for me at the end. Few things you should do to avoid those issues:

install graphicsmagick-libmagick-dev-compat
add --led-no-hardware-pulse to the shell script file if you get an error about audio

Good luck!

I updated the script today 🙂
It now works with Raspberry Pi OS and Adafruit's installer script.
(I'm still working on updating my README to reflect the changes.)

✅ README update


Board test function not working

Firmware 2.2.051 (Pi & ESP), BusRaider board revision 2.01.

BusRaider is the only board on the RC2014 Pro backplane.
NMI/Page/Clk jumpers all installed.
Pi-ESP1 and Z80-ESP2 jumpers installed.
Z80-Serial1 jumpers installed.

Board boots OK, Wifi connects, HDMI and keyboard OK. Web UI works. F2 brings up Immediate Mode. Memory test (m) works (and - correctly - fails as no memory board is installed). Bit test (b) appears to work, pressing space cycles through the tests and eventually returns to Normal mode.
Board test (t) does not work. Entering "t" displays "t" in the right-hand status area, pressing return then does nothing, no further output at all. Board appears to be hung, refresh rate spinner stops. Web server still serves the config page, but no web UI functions work. Power cycle or Pi reset button is required to regain control. After entering "t"/return, while board is apparently in hung state, Pi green LED is flashing rapidly at about 5Hz. This stops when the Pi is reset.
Board test worked with the same hardware and f/w 2.0.39. Not got time to revert and retest right now.
In all other respects the board appears to be working fine. Adding Z80 CPU and Spencer's 64K RAM board I can load Spectrum and TRS80 ROM images OK. Switching to Rob's 1MB RAM card (in RAM+RAM configuration) I can load RomWBW OK.

Hi
Apologies for the delay in responding. Did you make progress with this?
The rapid flashing of the Pi LED means the Pi has crashed unfortunately. So that’s a definite bug.
I’m going to take a look at this now.
Regards
Rob
Rob Dobson FIET
<EMAIL_ADDRESS>+44 7770 866452
http://LinkedIn.com/in/robdobsn
@robdobsn www.robdobson.com
From: The Liverpool Ringing Simulator<EMAIL_ADDRESS>Sent: 08 May 2020 14:25
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [robdobsn/PiBusRaider] Board test function not working (#3)
Firmware 2.2.051 (Pi & ESP), BusRaider board revision 2.01.

BusRaider is the only board on the RC2014 Pro backplane.
NMI/Page/Clk jumpers all installed.
Pi-ESP1 and Z80-ESP2 jumpers installed.
Z80-Serial1 jumpers installed.

Board boots OK, Wifi connects, HDMI and keyboard OK. Web UI works. F2 brings up Immediate Mode. Memory test (m) works (and - correctly - fails as no memory board is installed). Bit test (b) appears to work, pressing space cycles through the tests and eventually returns to Normal mode.
Board test (t) does not work. Entering "t" displays "t" in the right-hand status area, pressing return then does nothing, no further output at all. Board appears to be hung, refresh rate spinner stops. Web server still serves the config page, but no web UI functions work. Power cycle or Pi reset button is required to regain control. After entering "t"/return, while board is apparently in hung state, Pi green LED is flashing rapidly at about 5Hz. This stops when the Pi is reset.
Board test worked with the same hardware and f/w 2.0.39. Not got time to revert and retest right now.
In all other respects the board appears to be working fine. Adding Z80 CPU and Spencer's 64K RAM board I can load Spectrum and TRS80 ROM images OK. Switching to Rob's 1MB RAM card (in RAM+RAM configuration) I can load RomWBW OK.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/robdobsn/PiBusRaider/issues/3, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABAYSN5VWQ4JXSSMJGA6ZJTRQQB5VANCNFSM4M4FN3HA.

Hi Rob,
No, not had time to do anything much else with it at the moment. Is there
any additional debugging I can collect for you on this?
Regards
Andrew
On Sun, 31 May 2020 at 11:23, Rob Dobson<EMAIL_ADDRESS>wrote:

Hi
Apologies for the delay in responding. Did you make progress with this?
The rapid flashing of the Pi LED means the Pi has crashed unfortunately.
So that’s a definite bug.
I’m going to take a look at this now.
Regards
Rob
Rob Dobson FIET
<EMAIL_ADDRESS>+44 7770 866452
http://LinkedIn.com/in/robdobsn
@robdobsn www.robdobson.com
From: The Liverpool Ringing Simulator Project<mailto:
<EMAIL_ADDRESS>Sent: 08 May 2020 14:25
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS>Subject: [robdobsn/PiBusRaider] Board test function not working (#3)
Firmware 2.2.051 (Pi & ESP), BusRaider board revision 2.01.

BusRaider is the only board on the RC2014 Pro backplane.
NMI/Page/Clk jumpers all installed.
Pi-ESP1 and Z80-ESP2 jumpers installed.
Z80-Serial1 jumpers installed.

Board boots OK, Wifi connects, HDMI and keyboard OK. Web UI works. F2
brings up Immediate Mode. Memory test (m) works (and - correctly - fails as
no memory board is installed). Bit test (b) appears to work, pressing space
cycles through the tests and eventually returns to Normal mode.
Board test (t) does not work. Entering "t" displays "t" in the right-hand
status area, pressing return then does nothing, no further output at all.
Board appears to be hung, refresh rate spinner stops. Web server still
serves the config page, but no web UI functions work. Power cycle or Pi
reset button is required to regain control. After entering "t"/return,
while board is apparently in hung state, Pi green LED is flashing rapidly
at about 5Hz. This stops when the Pi is reset.
Board test worked with the same hardware and f/w 2.0.39. Not got time to
revert and retest right now.
In all other respects the board appears to be working fine. Adding Z80 CPU
and Spencer's 64K RAM board I can load Spectrum and TRS80 ROM images OK.
Switching to Rob's 1MB RAM card (in RAM+RAM configuration) I can load
RomWBW OK.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/robdobsn/PiBusRaider/issues/3>, or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABAYSN5VWQ4JXSSMJGA6ZJTRQQB5VANCNFSM4M4FN3HA

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/robdobsn/PiBusRaider/issues/3#issuecomment-636451427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACOJ5DUKXXMCQ4DZ2D4JVU3RUIV3BANCNFSM4M4FN3HA
.

Finally found a solution which seems to be toolchain related. The code for board test works when compiled with an earlier ARM cross compiler (2019 linux I think) but fails on 2020 compiler. Commenting out code from the test function gets it working again and the solution I've implemented is simply to shorten the test function by moving code to a sub-function and calling that instead of inline.


Feat: Batch buffer adding method

Also did tiny tweak on the code.
@chep PTAL Thanks

feedback addressed, and README updated.


server: add compatibilty with aiohttp 3.8

The version string parsing was recommended by the maintainer of aiohttp
Andrew Svetlov
https://aio-libs.discourse.group/t/aiohttp-web-run-app-incompatibility-between-aiohttp-3-8-and-aiohttp-3-8/259/4
Signed-off-by: Florian Scherf<EMAIL_ADDRESS>

FTR: @jluebbe, @SmithChart and i decided to pin the aiohttp version to 1.8.x, because we won't maintain the complexity of a version range of supported aiohttp versions.
@jluebbe, @SmithChart: I updated my patch accordingly

Tested on my setup. LGTM.


unable to retrieve key when using WSL

I have setup WSL ubuntu environment to work in VSCodium, however I cannot seem to get the extension to find the key.
Orignially VSCodium has only version 6.2 and I got the following error and log in that version.
After prompting for a password, it shows the error Failed to unlock: No key for current folder with log output of:
[INFO] Active GPG Indicator extension ...
[INFO] Setting: sync status interval: 30
[INFO] Create key status manager
[INFO] Update folder information
[INFO] Change folder to \\wsl.localhost\Ubuntu\home\Ubuntu
[INFO] Got one update function
[WARN] Can not find key information for folder: \\wsl.localhost\Ubuntu\home\Ubuntu

As far as I know all settings are set properly,

gpg --list-keys --keyid-format long shows my key.
directory shown above has .gitconfig file with commit.gpgSign = true and signingkey = mykeyhere

I downloaded installed the latest .vsix file for version 7.0.
With this version I get a popup error of Unable to retrieve any key in current folder. Not much more information is given.
Log output from extension is:
[INFO] Active GPG Indicator extension ...
[INFO] Setting: sync status interval: 30
[INFO] Create key status manager
[INFO] Got one update function
[INFO] Update folder information

I guess the extension cannot access \\wsl.localhost\Ubuntu\home\Ubuntu as that is outside of linux, or not the right name.
Is there something I should change in my setup to use WSL Ubuntu on windows?

Never mind - I didn't realize I needed to use the WSL extension to have VSCodium use all WSL files and paths.
Sorry!


Fix #45 support STM32 set I2C pins

Tested successfully with my arduino project (STM32G4) and HW.
Nicely done! I didn't know about ARDUINO_ARCH_{...}

@BRTFTW
FYI,
I am refactoring the API of begin as setting wire parameters in the library is not portable in the end.
The change is merely you need to set the Wire parameters yourself before calling AS5600.begin().
I've added an example in the PR / develop branch.


Unable to get current languages.

The current languages can no longer be scraped from https://www.churchofjesuschrist.org/languages?lang=eng.
Getting languages
Found 0 languages

Creating metadata-languages.json
Creating metadata-scriptures.json
Creating metadata-uri-to-name.json

Traceback (most recent call last):
  File "d:\...\python-scripture-scraper\scrape.py", line 1265, in <module>
    main()
  File "d:\...\python-scripture-scraper\scrape.py", line 132, in main
    output_full_content(config.DEFAULT_LANG)
  File "d:\...\python-scripture-scraper\scrape.py", line 727, in output_full_content
    if metadata_scriptures['languages'][bcp47_lang]['churchAvailability'][publication_slug]:
       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
KeyError: 'en'

The best work around I found so far is it use an older version of the page using the Wayback Machine.

Thanks for your report! Glad you found a workaround. I'll see if I can find a longer-term fix later after work today.

Fixed in this commit: https://github.com/samuelbradshaw/python-scripture-scraper/commit/ea2fb14f8522adf4f15e044c125ce4698b77915f


Deprecate support for custom Service Fabric AppInsights package

The Application Insights SDK has become rich enough where a custom SF experience is no longer required.  We are recommending customers to use what is available to them from the AI team directly.

Hi @mattrowmsft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (mattrow). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

wrong repo


After deployment vis toolkit getting cannot find restify module

I'm trying to deploy and publish the app to teams using teams toolkit, even I have locally tested the echo bot and its totally working fine, but after deploying to Azure its throwing cannot find module restify.
And also the app isn't getting published to Teams.
Sometimes I'm getting dotnev can't find similar error:
Tried running npm i, but it still producing the same error

Using JS samples

Is this still causing issues @Sandeep-FED

We have done major revamps to all the samples (both C# and JS) you should try provisioning those samples. Closing issue for now.


Add T[] Value.toJavaArray(T[])

(See #1578)
These are our options:
// unsafe
T[] toJavaArray(Class<? super T> componentType) // one size fits all

vs

// safe
T[] toJavaArray(Class<T> componentType) // variant for non-generic types
T[] toJavaArray(T[] array)

I think the first, unsafe variant is no viable solution.
Example:
// before
Value<CharSequence> v = Option.of("");
CharSequence[] s = v.toJavaArray(CharSequence.class);

// after
Value<String> v = Option.of("");
String[] s = v.toJavaArray(CharSequence.class); // will eventually throw at runtime

The existing method toJavaArray(Class<T>) is still useful for non-generic types,
we do not need to deprecate it.

T[] toJavaArray(T[] array)

Because of erasure I don't think it's possible to make this completely type-safe.

I think the first, unsafe variant is no viable solution.

Doing it the Java way would result in the same unsafety
@SuppressWarnings("unchecked")
Option<Integer>[] options = singletonList(Option.some(1)).toArray(new Option[0]);
System.out.println(options[0].get());

i.e you cannot call toArray with new Option<Integer>[0] either ...

@paplorinc you are right - it makes no sense to have toArray(T[]). That is the reason I introduced the simpler toArray(Class<T>).
If an application of toArray(Class<T>) compiles, it is correct. Therefore this is the best solution we can come up with.
For now, we will not implement the unsafe case, I don't see the urge to do it. We have many other things to do.

If an application of toArray(Class) compiles, it is correct.

Unfortunately that's not true either:
Array.of(1).toJavaArray(int.class);

errs at run time with:
java.lang.ClassCastException: [I cannot be cast to [Ljava.lang.Object;

I still think the <? super T> variant was the best compromise.

I'm not sure. Is it possible to fix our variant, e.g. by first checking type.isPrimitive() and then decide further how to proceed?
I will play around a bit.
Safety first.

checking type.isPrimitive() and then decide further how to proceed?

and throw a different kind of RunTimeException?

Safety first.

I agree, but I din't think it's possible, because of type erasure, it's inherently unsafe :/

See #1585


Hierarchical multiple select - parent items are completely removed from drop down, cannot add their children

Hey,
in earlier versions it was possible to select a parent element in multiselect, and its children elemen where still selectable. The parent element was disabled and could not be selected anymore but its children were not removed from the drop down.
Now the behavior has changed and items are removed completely upon selection, so if you select a parent item you can't select any of its children anymore.
See http://plnkr.co/edit/UQ3ur38TC6uVxLv8xvUr?p=preview
compared to http://plnkr.co/edit/a6rrYHYbLDtEB27K3j77?p=preview (version 3.2, has another issue though).
Maybe the behavior should be configurable?
Best,
Nils

+1

As of v4: https://select2.org/options#hierarchical-options

Only a single level of nesting is allowed per the HTML specification. If you nest an <optgroup> within another <optgroup>, Select2 will not be able to detect the extra level of nesting and errors may be triggered as a result.

Furthermore, <optgroup> elements cannot be made selectable. This is a limitation of the HTML specification and is not a limitation that Select2 can overcome.

If you wish to create a true hierarchy of selectable options, use an <option> instead of an <optgroup> and change the style with CSS. Please note that this approach may be considered "less accessible" as it relies on CSS styling, rather than the semantic meaning of <optgroup>, to generate the effect.


Tables - How to address custom cell renderers?

While working on a different approach to build new features into the table, I realized that users may want to provide their own custom cell renderers to add buttons or links into the table. I already had an idea for what the rendering function signature should look like, but haven't figured out if I'll need to adopt a functional component approach to build them. I'll need to consider allowing users to specify a property name and a custom cell renderer to go along with it, defaulting to the current minimal approach. The table API will almost definitely stay the same, but converting the templates to JSX / function style rendering may simplify the custom cell renderers and not make any of the existing features more challenging.
customCellRenderer(cellProperty, rowData){
    //return html cell content;
}

column data object
id: String
displayText: String
cellRenderer: Function(createElement, rowData, columnObj, vueInstance){
    return createElement(...);
}

I believe I'm going to commit to the column data object listed above, and also accept just a string or the data object for the columns array, that way the implementation can be very simple or more complex for customization.

The rest of this issue should be dedicated to changing the table column API to accept an array of [string | object] so cell renderers can make it into the 1.4.0 release.


Use docker image with correct imagemagick version on .travis.yml

It's taking too long, to setup the build every time we need to run it on CI. It tooks like 5 to 6 minutes just for building spec env. Multiply this to four ruby versions and we have something like 25 minutes to have build jobs completely done. If we use the docker image we've got up on performance better than the actual

Hi Vinicius, Im interested in working on this issue :)
Would you mind detailing more about this issue (acceptance criteria)? Is it just to add a docker image for each ruby version?

Hello Ricardo,
If you could add a base image with the correct ImageMagick version, it would be enough for now. If you check the root path, you'll see a file named befor_install_linux.sh, which basically destroys all ImageMagick previous installed, and reinstall it downloading from IM site. Travis, by default, has an IM available but is kinda older (6.7 I guess).
The main purpose here is to stop downloading and reinstalling IM every time the build runs.
So, answering your question, a single image with IM installed would be great, everything else can run as it runs now.

Instead of the docker image, we configured the ccache directory to avoid reinstall it every run. The build is running around 4 minutes now.
Thanks @rhiroyuki.


Build on MacOS

Does puffinn support MacOS? I've tried building on several versions of it on Travis, without success so far. With default settings, OpenMP support seems to be missing:
clang: error: unsupported option '-fopenmp'
I changed build options to use -mmacosx-version-min=10.14, in case the old settings had forced some old llvm version without openmp support. This seems to work in principle, but throws another error.
I've also tried with homebrew gcc9, which throws some other erros, see Travis log.

Thanks for the report. We never tried it! :-) I'll give it a look.
Martin

I see :) I thought I may have just overlooked how to set one build parameter correctly, or something along the line. Mac support would be nice to have, but it's not a top priority for me.

I fixed some issues with compiling using clang and a c++17 feature that snuck in. I think that fixes the errors you were seeing, but I don't have the means to test it. Could you give it another try?

I removed the dependency on openmp in #3, @mvesterli. It compiles fine on mac os x, except from
± % ./Test                                                                                !10141

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Test is a Catch v2.1.0 host application.
Run with -? for options

-------------------------------------------------------------------------------
filtering equal/opposite vector
-------------------------------------------------------------------------------
/Users/maau/Documents/Projects/puffinn/test/include/filterer_test.hpp:12
...............................................................................

/Users/maau/Documents/Projects/puffinn/test/include/filterer_test.hpp:39: FAILED:
  REQUIRE( sketches.passes_filter(filterer.get_sketch(1, i), i) )
 1 setup.py                                                                                X
with expansion:
  false

===============================================================================
test cases:    48 |    47 passed | 1 failed
assertions: 87282 | 87281 passed | 1 failed

for the unit test.

Thanks! Building, installing and importing works now on several mac os x versions on travis.
I'll do some additional functional tests, and report back, if I encounter any issues.

I'm getting strange results, where the same tests succeed on Linux, but fail on MacOS. It appears as if puffinn was less precise on Mac. Is that possible? I'll try to investigate further, but unfortunately, I don't have a local Mac for testing/debugging (only using Travis).

Was it compiled without AVX2 on Mac by any chance? The fallback dot product function was incorrect, but that should not have caused different results on Mac and Linux. I also fixed a couple other minor issues, but I doubt that they caused this. With the fixes the tests all pass on the Mac I tested on.

I think it was compiled without AVX2. setup.py calls gcc with -march=native, and the machine is Intel(R) Xeon(R) CPU E5-2697 v2, if I'm not mistaken. The specs tell me that this CPU supports AVX, but not AVX2. (Though the v3 version actually supports AVX2).


Building production

Help
Building
I am knowing dioxus now and when compiling is it normal to appear a prompt when I run the generated binary or is there any parameter I should use during compilation?
From now on, i grade

If I understand the question, you want to not show the terminal when you launch the application?
It depends on the platform, but if you are on windows adding #![windows_subsystem = "windows"] to the top of your main.rs file will remove the terminal and just show the app.

Yes, that was exactly it. The prompt no longer appears.
Thank you very much.


Eval output improvements

Closes #2492
What
As well as correcting the logic for how many lines are displayed in the eval output (see #2492), this PR improves output truncation as follows:

You can now know an individual line was shortened (but not completely removed) by the inline ... at the end.
If there are any completely removed lines, this is indicated by a ... on a new line after the output.

Examples:
NB: There have been some minor changes to the truncation logic since these screenshots to ensure the final output is <= max_chars, but the actual elipses and truncation messages are the same.
One line that is too long
Before (not clear what was edited -- whether the line was shortened, or a line completely removed (or both)):

After (explicitly shows that no line was removed, and that the displayed line was shortened):

One line that is too long with a truncated line
Before (shows that something changed, but it's not clear what, and it's not clear that a line was completely removed):

After (explicitly shows that a line was removed and the displayed line was shortened):

Too many lines (the actual bugfix for #2492)
Before (displays max_lines + 1 lines instead of max_lines):

After (displays max_lines lines):

Too many lines and too long
Before (again not clear what was truncated):

After (explicitly shows that a line was removed, and the final displayed line was shortened):

Multiple lines
Before (shows line number):

After (still shows line number, so didn't break this):

Single line
Before (doesn't show line number, since there's only one line):

After (still doesn't show line number, so didn't break this):

NB: I'm not sure how much (if any) of the docker-compose changes should actually be part of this PR. Can someone please instruct on that.

This code seems quite complex, given this is balancing quite a lot of different aims/constraints I think we'd benefit from having some tests for this.
I'm also not sure the complexity is really worth it, a simple solution that replaces the current logic and fixes the linked issue would be something like this:
def truncate(text, max_lines, max_chars):
    # Make the truncation message
    lines = text.split("\n")
    reason = []
    if len(text) > max_chars:
        reason.append("too long")
    if len(lines) > max_lines:
        reason.append("too many lines")
    end = f"\n... (truncated - {', '.join(reason)})" if reason else ""

    # Truncate text, leaving space for truncation message if necessary
    text = "\n".join(lines[:max_lines - (1 if end else 0)])
    text = text[:max_chars - len(end)]

    return text + end

Which meets the following aims:

Never have more than max_chars characters in the final output
Never have more than max_lines lines in the final output
Have as many lines/chars as possible in the final output

@wookie184

Never have more than max_chars characters in the final output
Never have more than max_lines lines in the final output
Have as many lines/chars as possible in the final output

For the record, none of these are fulfilled in the old code (that's currently in main). All three of these are fulfilled by my new code, so it does meet the necessary criteria.

This code seems quite complex

I've added some images showing before vs after for different scenarios to the PR description to make the resulting differences (aside from the aforementioned criteria) clearer). I personally think the extra "complexity" is worthwhile. I've also made the code a bit cleaner since your comment.

given this is balancing quite a lot of different aims/constraints I think we'd benefit from having some tests for this.

Tests can be added if we decide we want them.

I'm not sure what issue is being solved by having the ... in different places. Is this solving something that has been an issue in reality? If you want to know what was truncated you can just look at the full output in the pastebin.
Requiring two lines (... and then (truncated ...)) takes up more space to indicate the same thing, and I think it's also a bit confusing not to have the truncation indicated in a consistent way.
Also, I think the truncated text should be included in the line limit. Consider if the maximum line length was 3, print(*"abc", sep="\n") would give
a
b
c

Currently (from what I can tell from your screenshots), print(*"abcd", sep="\n") would output
a
b
c
...
(truncated - too many lines)

which doesn't seem great, because the full output could have been displayed with one less line of text in total.

I'm not sure what issue is being solved by having the ... in different places.

It just makes it clear what was truncated without having to leave Discord and go to the paste site. It didn't feel like much extra work, so I did it 🤷.

Requiring two lines (... and then (truncated ...)) takes up more space to indicate the same thing

It doesn't "indicate the same thing" though. It's possible for the truncation reason to be "too long", and to still have lines removed (e.g. for _ in range(2): print("a" * (max_length // 2)). Just from "too long" you wouldn't know that lines were removed, which is why I added the ... line.

I think it's also a bit confusing not to have the truncation indicated in a consistent way.

It is in a consistent way. The ... tells you something was truncated. That's consistent across all scenarios.

Also, I think the truncated text should be included in the line limit.

Yeah, I agree. Have updated the code accordingly.

Having ellipses in multiple places seems confusing to me and I'm not sure if we have to find a way to address this. In the end the information we had before is that this isn't the full output and that you should open the paste link for the full output, which I think is sufficient.

@mbaruh Having ellipses in multiple places seems confusing to me

I really don't understand where this confusion comes from. ... has long been used in the world of UI/UX to indicate that the displayed text has been shortened (and we already had this in the eval output before this PR).
What I've changed is that if the line is shortened, the ... shows after the shortened line, showing it was simply shortened, and if the line is completely removed then the ... shows in-place of the line, showing it was completely removed.
The ... tells you where it was shortened, as is the job of ... in the world of UI/UX. Where is the confusion here?

... has long been used in UI yes, however you're taking that symbol and expanding it to potentially appearing several times at the same point, each time for a different reason.
I understand that this may seem obvious to the person who wrote the code, but I purposefully didn't look at the code when trying to understand the examples, and it wasn't obvious what's going on until I did. Your explanation in the latest comment is clear (although again I'm already "tainted" having already read the code), but you're not going to post it for every end-user trying to use snekbox right? This should be something that is immediately obvious without any prior knowledge.
Aside from having two ellipses in the same spot not looking great (in my opinion), I also think it's confusing. While I appreciate you going the extra mile I'd rather focus on addressing what's originally described in the issue.

Closing in favor of recreating within a new PR.


Vim abbreviations - :ab/:iab

Hi,
Running into an interesting issue with this,
:ab/:iab doesn't seem to be working in vscode:1.47.0 even though it works in nvim just fine..
I can confirm my neovim path is set correctly in vscode, other vim features are working as expected.
Doing :ab in nvim launched from the shell works as expected as well..
Could this be a configuration issue or is :ab not supported?
Appreciate the feedback.

I see, thanks for clarifying. Looking thru the src & docs the design choice makes sense now.
Sounds like any nvim insert-mode plugins are also affected similarly I'm guessing.

Correct, anything involving the insert mode on neovim side won't work


It's tired to maintain packages/option-t/tools/pkg_files.json

We maintain https://github.com/option-t/option-t/blob/50b76a2/packages/option-t/tools/pkg_files.json to test whether expect files is contained before publish a new version. This file contains all file that we expect to publish. But it's too tired to maintain it by hand.
Originally, this file was introduced to check a expect file exist to there, especially for d.ts files. It does not ensure all d.ts files has proper relationships for each other. But it was a convenient adhoc solution.
We would like to make it a snapshot test but it's a bit complex.

Even if we move to snapshot testing, it will not still check that all d.ts files has can correct module relationships. This does not ensure any correctness about types.
It is a just design issue of build system. We create temp __dist/ dir under packages/option-t during build process to allow to test by this easily. But outer inner-packages under packages/cannot seek it. So we cannot implement a clean snapshot testing as packages/api_tests.

Even if we move to snapshot testing, it will not still check that all d.ts files has can correct module relationships. This does not ensure any correctness about types.

I think we have some other approach to this.

Write ts-expect and compile it to test each of exported packages. It's hard way including large refactoring. But this approach ensures a type relationship. Probably this approach might not test whether expect files is contained before publish a new version. And this would require to "double-check" all of our code for us.
Get a type information and write a snapshot by calling typescript API. But this might be flaky by changing a TypeScript version. Furthermore, Language Service API might be low level for testing this kind of problems.
Give up to publish commonjs. It's simplify our build process at all and we don't have to check a complex relationship if we drop commonjs. But it causes a complex compatibility problems.
Drop .d.cts and we should use .d.ts only. But I'm not sure about  typescript compiler can work correctly for that case in commonjs.

I try to add a test to import exposed types and relationship by #1668.
It will help to resolve this.


credhub metrics are cached indefinitly

using prometheus bosh release 25 / credhub exporter 0.1.6, we have applied a soon to expire cert rotation, which should have discarded the alert.
The credhub_exporter keeps giving to prometheus the old metrics (metrics seemed cached with no time limit).
credhub_certificate_expires_at{deployment="fe-prod",environment="fe-prod",id="08a91662-0ad0-40a5-a4c4-86a01e83cdd4",index="1",name="director_ssl-ca",path="/bosh-master/bosh-coab/director_ssl"} 1.563376859e+09
17/7/2019 à 15:20:59
credhub_certificate_expires_at{deployment="fe-prod",environment="fe-prod",id="11117042-0ddf-47e9-ad41-416dc3e5522b",index="1",name="director_ssl-ca",path="/bosh-master/bosh-coab/director_ssl"} 1.877630181e+09
1/7/2029 à 19:56:21

Restarting the credhub_exporter jobs solved the issue.
cc @ogrand

Fixed in version 0.1.7 through  #13, thanks a lot @Freakin !
@psycofdj is mentionning privately to me that he is currently unable to prioritize further testing this PR and contributing the bump upstream similar to https://github.com/bosh-prometheus/prometheus-boshrelease/pull/251
@Freaking please feel free to go ahead and submit such a bump PRupstream if this can help speed up the resolution and its deployment in your prometheus deployments.
/CC @poblin-orange @ogrand

v0.2.0 now resets the metrics on each analyze loop


Fixed per torrent quality property name inconsistency

The common per quality torrent includes seeds and peers but there are an inconsistency with movies, for movies are seed and peer thats we have to extend the torrent per quality object and override the getter for both properties.

I will look into updating the database structure so this can be done once this PR goes into production.

I'm closing this PR, because I would like to have all the structural changes you purposed in a separate branch. Hopefully I'll be able to make it later today, else I'll do it tomorrow.


LFS-107/LFS-108: Wrapper classes for ontology repository jsons.

@zm2he There's no code that actually uses these. Did you forget to add some new files to this branch, or that code isn't written yet?


[tool_crash] FileSystemException: Deletion failed, OS Error: The process cannot access the file because it is being used by another process., errno = 32

Command
flutter run

Steps to Reproduce

flutter run.
Delete a file assets file while flutter is running on a device.
Tried to hot reload.

Logs
FileSystemException: Deletion failed, OS Error: The process cannot access the file because it is being used by another process.
, errno = 32
#0      _Directory._deleteSync (dart:io/directory_impl.dart:201:7)
#1      FileSystemEntity.deleteSync (dart:io/file_system_entity.dart:458:47)
#2      ForwardingFileSystemEntity.deleteSync (package:file/src/forwarding/forwarding_file_system_entity.dart:72:16)
#3      ResidentRunner.preExit (package:flutter_tools/src/resident_runner.dart:1264:25)
#4      HotRunner.preExit (package:flutter_tools/src/run_hot.dart:1152:17)
<asynchronous suspension>
#5      ResidentRunner.exit (package:flutter_tools/src/resident_runner.dart:947:11)
<asynchronous suspension>
#6      TerminalHandler._commonTerminalInputHandler (package:flutter_tools/src/resident_runner.dart:1500:30)
#7      TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1584:13)
#8      _rootRunUnary (dart:async/zone.dart:1198:47)
#9      _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#10     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1005:7)
#11     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:357:11)
#12     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:285:7)
#13     _SyncBroadcastStreamController._sendData (dart:async/broadcast_stream_controller.dart:385:25)
#14     _BroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:250:5)
#15     _AsBroadcastStreamController.add (dart:async/broadcast_stream_controller.dart:484:11)
#16     _rootRunUnary (dart:async/zone.dart:1198:47)
#17     _CustomZone.runUnary (dart:async/zone.dart:1100:19)
#18     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1005:7)
#19     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:357:11)
#20     _BufferingStreamSubscription._add (dart:async/stream_impl.dart:285:7)
#21     _SinkTransformerStreamSubscription._add (dart:async/stream_transformers.dart:69:11)
#22     _EventSinkWrapper.add (dart:async/stream_transformers.dart:15:11)

[✓] Flutter (Channel master, 1.20.0-8.0.pre.31, on Microsoft Windows [Version 10.0.18363.900], locale de-DE)
    • Flutter version 1.20.0-8.0.pre.31 at C:\Users\Jesse\OneDrive\Dokumente\flutter
    • Framework revision 0268096bcd (24 hours ago), 2020-07-09 17:11:01 -0400
    • Engine revision a48f05f833
    • Dart version 2.9.0 (build 2.9.0-21.0.dev 06cb010247)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at F:\Files\TMFiles\android-sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = F:\Files\TMFiles\android-sdk
    • ANDROID_SDK_ROOT = F:\Files\TMFiles\android-sdk\tools
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.6.3)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.6.30225.117
    • Windows 10 SDK version 10.0.19041.0

[✓] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[!] IntelliJ IDEA Community Edition (version 2019.3)
    • IntelliJ at C:\Program Files\IntelliJ IDEA Community Edition 2019.3.4
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] VS Code (version 1.47.0)
    • VS Code at C:\Users\Jesse\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.12.2

[✓] Connected device (2 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 5.0.2 (API 21) (emulator)
    • Windows (desktop)                  • windows       • windows-x64 • Microsoft Windows [Version 10.0.18363.900]

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: master
Creation framework version: 462b0ea76eca028a7776f60a20a05ebce30dfdbd
Plugins
cloud_firestore-0.13.7
cloud_firestore_web-0.1.1+2
firebase_core-0.4.5
firebase_core_web-0.1.1+2
firebase_storage-3.1.6

@SolomonRosemite  please fill out the steps to reproduce with more details on the commands run and devices targeted.

Without additional information we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


Specific minimal subset of JSKOS API

Which endpoints and query parameters must be supported to

provide a vocabulary
provide mappings

In addition to wikidata-jskos there will be an implementation of subset of JSKOS API for Konstanz Systematics so definition of what to expect would be helpful.

The subset for vocabularies and concepts ist most relevant. To do so add API endpoints:

alias /narrower as /voc/narrower and make the second the primary, the first deprecated
alias /ancestors as/voc/ancestors and make the second the primary, the first deprecated

And then specify a subset of API endpoints /voc* as JSKOS Vocabulary API. API perameters should be reduced too, e.g. make support of properties optional.

I just noticed that /voc/narrower//voc/ancestors are not consistent with other endpoints, considering your suggestions in #189. From my point of view, it should be /voc/concepts/narrower//voc/concepts/ancestors, right? Maybe it would be better to have vocabulary data under /voc*, but concept data under /concepts*? We also have /search and /suggest which are concept endpoints, so we would have /concepts/search for concept search and /voc/search for vocabulary search.

With #193, the endpoints were made more consistent. See this comment for more details. These changes will be released in an upcoming version 2.0.


Please provide an example for BeforeUnloadEvent

Hi @glennsl, I am wondering how I could use https://github.com/reasonml-community/bs-webapi-incubator/blob/bc0badfc40420e271f4b1703dd9c43bed7d68b13/src/Webapi/Webapi__Dom/Webapi__Dom__BeforeUnloadEvent.re to stop user from reloading/exiting the current page, it's appreciated if you could provide an example, cheers

Something like this should work I think:
open Webapi.Dom;
Window.addEventListener("beforeunload", Event.preventDefault);

@glennsl thanks a lot, :)

@glennsl this is what I've come up with:
let onBeforeUnload = (e: Dom.event) => {
   Webapi.Dom.Event.preventDefault(e);
  Js.log(e);
};

Webapi.Dom.(
    React.useEffect0(() => {
      Window.addEventListener("beforeunload", onBeforeUnload, window);

      Some(
        () =>
          Window.removeEventListener("beforeunload", onBeforeUnload, window),
      );
    })
  );

the above code is supposed to print the shape of e once the user is leaving the site, but it's not ..., I am wondering what I did wrong with above code


[MUI System][@mui/icons-material] why @emotion/react and @emotion/styled are optional dependencies?

Steps to reproduce

use @mui/icons-material in a package without @emotion/react @emotion/styled
fail to compile

Current behavior
Why are @emotion/react and @emotion/styled listed as optional in package.json? They are direct dependency of StyledEngineProvider, which is exported from MUI System
Expected behavior
No response
Context
No response
Your environment
No response
Search keywords: StyledEngineProvider, @emotion/react, @emotion/styled

Because they include a React context, duplication of the npm package would break behavior.
A bit related to https://github.com/mui/material-ui/issues/34489 on making the icons smaller standalone.

@mondaychen It's marked as optional because emotion can be swapped for styled-components.

Thanks for the reply! I think that makes sense. I'm wondering if there's any possible way to detect at least one of the required dependencies is installed.


CPU Backend Crashes JVM on MLPMnistSingleLayerExample (Windows)

Using the current master, the cpu backend crashes the JVM when running MLPMnistSingleLayerExample.
hs_err_pid1312.log.txt

It looks like this problem is caused by this commit https://github.com/deeplearning4j/nd4j/commit/3a3b9166b93af1768d3a3831dbc2b0e7f955caea

package com.example;
import org.nd4j.linalg.api.ndarray.INDArray;
import org.nd4j.linalg.factory.Nd4j;

public class App
{
    public static void main( String[] args ){
        INDArray indArray = Nd4j.create(1);
        indArray.assign(Nd4j.ones(1));
    }
}

This minimal example crashes for me.

For what it's worth: no issue for me on Windows 8.1, on either MLPMnistSingleLayerExample or the above minimal example...

I should also add that it works on my linux vm, but on windows 10 it simply crashes.

Looks like it was something weird with intellij, restarting it resolved the issue.


Removes whitespace introduced by commit b2c4093

Commit b2c4093 added unnecessary whitespace to the dev guide. It appears that make docs does not clean-up whitespace. Ran make docs which added the "release-cadence" section to the dev guide.

/assign @bowei

This includes the text from the other commit?

/lgtm
/approve

@bowei I updated the PR so only the whitespace removal is included. Previously, I ran make docs after removing the whitespace which added the release cadence content to the dev guide.


Make it easier to swap nwaku for go-waku in tests

Blocked by https://github.com/status-im/go-waku/issues/247

protons is not performant so we may want to switch to protobufjs (fixed by https://github.com/ipfs/protons/pull/58 ?)
This was pending https://github.com/ipfs/protons/pull/48


HTML Validation

I ran the contributing page HTML through W3C Markup Validation Service and it returned a few warnings and errors. This PR resolves those:

Updated the IE meta tag to the modern syntax (got this from HTML5 boilerplate)
Removed cellspacing and cellpadding attributes from the table (it recommends to do this via CSS now)
Removed the extraneous label that was attempting to be used for the radio and check input examples

:panda_face:


Enable Codecov integration

We want to track the code coverage for this project. In order to do so, we integrate with codecov.io.
In scope of this issue, it is needed to include Codecov-related action into our CI workflows, which build our code on PR basis.
In particular, the following step must be added to the workflow:
- name: Upload coverage reports to Codecov
  uses: codecov/codecov-action@v3

@Artem-Semenov-dev please process this issue in a separate PR.