Got an error when equesting textDocument/codeAction

👓 What did you see?
Got an error when equesting textDocument/codeAction.
I wrote And 60 minutes have passed  in my feature file and wrote     @When("{int} {time_unit} have/has passed") in my *.steps.java and I hope the feature file should jump to the step definition, but it got a error.
[Info  - 10:43:53 AM] * Found 37 parameter types in those glue files
[Info  - 10:43:53 AM]   * {time_unit} = (?i)(nanosecond|microsecond|millisecond|second|minute|hour|day)s?

✅ What did you expect to see?
The feature file should jump to the step definition.

📦 Which tool/library version are you using?
Cucumber for Visual Studio Code V1.3.0

And the output is
[Info  - 10:48:41 AM] * Built 449 suggestions for auto complete
[Error - 10:48:41 AM] Request textDocument/codeAction failed.
  Message: Request textDocument/codeAction failed with message: Invalid regular expression: /((?i)(nanosecond|microsecond|millisecond|second|minute|hour|day)s?)/: Invalid group
  Code: -32603

Thanks for the detailed report! I'll look into this.

@DouyuShinyruo can you please share your Java code where you define the parameter type?

The embedded (?i) flag is Java-specific, and isn't understood by JavaScript RegExps. (The VSCode extension runs on Node.js).
If you change your expression from ((?i)(nanosecond|microsecond|millisecond|second|minute|hour|day)s?) to (nanosecond|microsecond|millisecond|second|minute|hour|day)s? it should work.
Is there a particular reason why you want it to be case insensitive?


Can get ignore to work with array and multiple directories

I want to ignore two locations. My config looks like this
"compile-hero.ignore": [
    "**/club/html/vote/**",
    "**/club/NewClub/**"
  ]

But compile-hero still compiles my scss files in those subdirectories.
If I change my config to one directory without an array like this, the ignoring works.
"compile-hero.ignore": "**/club/html/vote/**"
Any help appreciated.

Can you upload your directory to git, show me the specific directory structure and files.


includedResources with UPDATE action

It's exactly like this issue https://github.com/jamesplease/redux-resource/issues/414 but for update action.
I made the same changes that yours, based on this PR: https://github.com/jamesplease/redux-resource/pull/415

Sorry for the extreme delay on this one. I'll get a new release published in the next few days; I want to take one final close look since it's been some time since I've looked at this lib.

@GuillaumeJasmin , would you like to add yourself as a contributor? This will show up on the README and on the website.

This has been published as<EMAIL_ADDRESS>Feel free to give it a try and let me know how it goes, @GuillaumeJasmin ! And sorry again for the long delay here.


ToTensor confuse me with the way it takes input

So the ToTensor class of to_tensor function takes input in the dimension of (H, W) while PIL has it's images dimension be (W, H).
Why is this transpose ?
cc @vfdev-5

PIL.Image.size reports (W, H) and not (H, W), see https://pillow.readthedocs.io/en/stable/handbook/concepts.html?highlight=size#size
As tensor we need to have B, C, H, W format thus it may seem that there is a transpose.

+1 on @vfdev-5 answer, this is the reason.
I'll close the issue to keep things neat but please feel free to reopen if you still have issues.

@vfdev-5
That's exactly what i am asking, PIL grayscale image has its shape to be [W, H] … the F.to_tensor function takes input image with shape [H, W, C] and outputs tensor with shape [C, H, W].
if i entered the image without change it will see the Width as the Height and the Height as the Width, This is what confuses me

I think that what @vfdev-5 is saying is that while PIL.Image.size reports (W, H), PIL images are still represented as H, W arrays or H, W, C. For the shape, we just put the C in the beginning instead of at the end when converting to tensor.
In [1]: import numpy as np

In [2]: from PIL import Image

In [3]: import torchvision.transforms as T

In [4]: a = np.random.randint(0, 10, (2, 4), dtype=np.uint8)

In [5]: pil_img = Image.fromarray(a)

In [6]: t = T.ToTensor()(pil_img)
/Users/nicolashug/dev/vision/torchvision/transforms/functional.py:136: UserWarning: Legacy tensor constructor is deprecated. Use: torch.tensor(...) for creating tensors from tensor-like objects; or torch.empty(...) for creating an uninitialized tensor with specific sizes. (Triggered internally at  /Users/distiller/project/conda/conda-bld/pytorch_1617466503169/work/torch/csrc/utils/tensor_new.cpp:476.)
  img = torch.ByteTensor(torch.ByteStorage.from_buffer(pic.tobytes()))

In [7]: np.asarray(pil_img)
Out[7]:
array([[1, 2, 1, 0],
       [7, 0, 9, 6]], dtype=uint8)

In [8]: t * 255
Out[8]:
tensor([[[1., 2., 1., 0.],
         [7., 0., 9., 6.]]])


Fix splat destructuring type information (retry)

Motivation
This is a re-try of @paracycle 's [sorbet/pr/2863] Fix splat destructuring type information, which had to be reverted.
Test plan

See included automated tests.
Ran against Stripe's codebase

Does this not suffer the same problem as trevor pointed out in the original revert?
https://github.com/sorbet/sorbet/pull/2878#issuecomment-613090075

We have a policy of testing changes to Sorbet against Stripe's codebase before
merging them. I've kicked off a test run for the current PR.  When the build
finishes, I'll share with you whether or how it failed. Thanks!
Stripe employees can see the build results here:
→ https://go/builds/bui_PW21egKKS9ngZj
→ https://go/builds/bui_PW21LhdcoX0yj9
→ https://go/builds/bui_PW21csZvI0GndQ

Does this not suffer the same problem as trevor pointed out in the original revert?
#2878 (comment)

Yes, it does.. I just wanted to bring it up to date to better understand what was going on :)


Files Structuring Done

PR Description 📜
Transferred all the js script to js files.
removed redundant code.
Fixes #652

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for review.

## Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

@ayush-t02 please review the PR


NullableBooleanInput problem in filters

NullableBooleanInput filter value only updates when switching from "Yes" state to "No" or "Null" not when switching from "No" to "Null" or "Null" to "No".
I'm using following versions:

"admin-on-rest": "^1.2.3"
"react": "^15.6.1"

Here is a sample of my code:
import React from "react";
import {
    BooleanField,
    Datagrid,
    Filter,
    List,
    DeleteButton,
    EditButton,
    TextField,
    TextInput,
    DateField,
    NullableBooleanInput,
} from "admin-on-rest";
import {datetimeFormatter}  from "../../helpers/formatter";
import config from "../../helpers/config";

const AdministratorFilter = (props) => (
    <Filter {...props}>
        <TextInput label="Search" source="search" alwaysOn/>
        <NullableBooleanInput label="Active" source="active"/>
    </Filter>
);

export default (props) => (
    <List perPage={config.PER_PAGE} filters={<AdministratorFilter/>} {...props}>
        <Datagrid>
            <TextField source="title" sortable={false} />
            <BooleanField source="active"/>
            <DateField source="created_at" format={datetimeFormatter} showTime locales={config.LOCAL}/>
            <DateField source="updated_at" format={datetimeFormatter} showTime locales={config.LOCAL}/>
            <EditButton/>
            <DeleteButton/>
        </Datagrid>
    </List>
);

And here is sample screenshots:
This is when there is no filter applied:

This is when a filter applied as "Yes" (just take a loot at the Active filter url in browser):

And this is when we apply "No"

And this is when at last we apply "Null":

As you can see active filter is still remaining as False but if we switch to "Yes" everything is returning to normal.

Is it still relevant in 1.3.1 ?

@djhi  I update my project to aor  to 1.3.1. The bug is fixed now. Thanks.


Allow multiple team panels to be opened on team competitions

Currently on team competitions, you can open each team individually to view their individual member gains within the team standing, but you can only open one team's panel at a time.
If I'm viewing Team A, and then want to view Team B, Team A will close when I open Team B.
This doesn't allow me to open certain teams side by side to compare, but most importantly it seems a bit janky/jerky with how they open/close.
I feel it would be better if they stayed open until you closed them for each team.
See discord message for reference
https://discord.com/channels/679454777708380161/774713646382972978/889933106046464041

Added in the new beta website https://beta.wiseoldman.net/
I'll consider it done as it'll eventually be live in production. Closing.


Make dep checker and site gen only run on scheduled and manual builds.

This PR optimizes PR and IndividualCI runs by eliminating the Maven site generation and dep checker steps.

/azp run java - core

/azp run java - core

/azp run java - core

/azp run java - core

/azp run java - core


QTE's - hold instead of mash

How about an option
to just hold the button instead of having to mash it during QTE's
for those of us with sore joints?
Thanks!

AutoHotKey is your best bet here. You can use it to spam keyboard strokes, and it can be used in conjunction with DS4Windows if you are using a controller. There are lots of threads in their forums about how to assign keystrokes to spam keys.

It's not a baaaad idea... buuut...
Auto-fire through AHK via the controller doesn't work very well,
and auto-fire through AHK via the keyboard means I have to move hands from controller to keyboard,
On my GameCube-style controller, I'm only ever mashing the A or B button.
But it can't just be “hold A to mash A” or “hold B to mash B”
because then it'd always activate,
and I tried to make it only mash A or B while I'm holding down the L1/LB button,
but then the game would have to register L1/LB + a keyboard key,
and the game doesn't support simultaneous K+M/controller input.
It really is a pain.

It might be possible by patching KeyTrgCheck calls to use KeyRepCheck instead - AFAIK Trg (Triggered) returns true when button state was changed to pressed from unpressed state, and returns false is button state hasn't changed (eg. if it's being held down) - while Rep (Repeat) returns true while button is being held down.
Looking at some of the QTEs that re4_tweaks has patches for already, it looks like some of them are using KeyTrgCheck, but some are also checking the triggered variable by themselves instead, so unfortunately patching KeyTrgCheck to always call KeyRepCheck probably wouldn't be enough (and would probably break some things too) - so I guess main way to allow this would be to go through code for each QTE & manually patch each one...

Looking at some of the QTEs that re4_tweaks has patches for already, it looks like some of them are using KeyTrgCheck, but some are also checking the triggered variable by themselves instead

Something I've been thinking about doing is revisiting the QTE patches, now that we have your IDB file as a reference. Back when I first made those patches, I had no idea what I even was patching, tbh.
The current approach probably wouldn't scale well with above 60 FPS too, so yet another reason to reimplement this stuff.
I'll probably take a look at it again soon.

Added an option to disable the QTEs, like modern games.
Here's a test build:
dinput8-disableQTE_v1.zip
It makes the QTEs pass automatically and disables the on screen prompts, making QTEs look more like cutscenes.
Apparently everything works fine, but I'd appreciate some testing.

@nipkownix
This is only for occasions where you're asked to mash a button really fast right?
Not for any A+B or L+R or whatever, right?
If so, it's not a bad solution.
If not, it's definitely less than ideal.

Nope, this is for all QTEs, not just the mashing ones.
I didn't think people cared about the other ones, tbh. Wouldn't be hard to add another option to make just the mashing ones automatic, I guess.

@nipkownix
I'm just trying to detract as little as possible from the game's character.
If it's not hard to make a mashing-only one, I'd humbly ask for that.

New build:
dinput8-disableQTE_v2.zip

; Disables most of the QTEs, making them pass automatically.
DisableQTE = false
; Unlike the previous option, this only automates the "mashing" QTEs, making them pass automatically. Prompts are still shown!
AutomaticMashingQTE = false

@nr1995 let me know if that works for you.

You can enable the new settings using the F1 menu

@nipkownix  Oh right.  Dumb.  Sorry.

@nipkownix
Yeah it's a bit janky in the laser corridor, I've just discovered.
Doesn't seem to work great in there.


Reference to commands.ts is invalid.

https://github.com/fellipeutaka/discord-bot-template/blob/main/src/deploy-commands.ts#L3 references 'commands.ts' but there is no source file named commands.
The same issue seems to be in https://dev.to/fellipeutaka/creating-your-first-discord-bot-using-typescript-1eh6
Now, I haven't checked out your repo and built it, so I'm just going on reading the code, but I feel confident this is a valid issue.

Never mind, today I learned about index.ts


[add-resource] New Theme

I would like to add my new theme to the Grav Repository.
Here are the project details: https://codeberg.org/mononym/grav-theme-doric
Demo: https://doric.julienrippinger.eu
Releases: https://codeberg.org/mononym/grav-theme-doric/releases

You need to host on GitHub to be included in GPM as our script uses the GitHub API to handle releases.

OK, I understand. But I'm sorry, I won't do that.
c.f. https://sfconservancy.org/GiveUpGitHub/

OK, I understand. But I'm sorry, I won't do that.
c.f. https://sfconservancy.org/GiveUpGitHub/


Fix worker tool box in visualizer

Description
The worker toolbox in the visualizer shows some buggy information in the phase between adding a worker to the scheduler and actual task processing (so while the dependency tree is built). This looks like:

The reason is that workers is both used for the list of worker objects in the visualizer for rendering and the number of worker processes of a worker. See here (first and last line in the marked area). Initially, there's no 'worker' field in the worker object so Mustache falls back to the list of workers which renders to the example above.
Motivation and Context
A simple renaming of the variable that holds the list of worker objects for rendering fixes the bug (2nd commit). In addition, the worker toolbox is removed now entirely when a worker is disabled (1st commit). Previously, the "disable" button was removed while the "add workers" menu was still visible.

How was this tested by the way? Looks good too me.

Whoups, I wanted to use the rebase merge strategy. Oh well. Not too big deal.

I created a short test task that just did a time.sleep(n) to delay the dependency tree building. Wasn't sure how this could be covered by a test case.


In BaseResultIterators.getParallelScans(...), performa binary search …

…instead of linear search for the first guide post in the first region of the targeted scan ranges. In details:
1. The aglorithm continuously decodes and loads guide posts in batches and perform binary search in each batch, until it finds the start key of the first region in the scan ranges or its insertion position.
   There are two reasons to use moving windows to load guide posts in batches.
   a. Firstly, we don't want to load all guide posts in the memory to increase memory footprint.
   b. Secondly, we don't want to decode and load the guide posts beyond the targed scan ranges to increase the system overhead.
2. Added config parameter STATS_GUIDEPOST_MOVING_WINDOW_SIZE to denote the size of moving window used for loading guid posts.

@karanmehta93  Can you also review this?


Add FilteredRecordsSavingRecordFilter

The component is a decorator for all filter that implement the RecordFilter interface that save the records discarded from the specific filter provided. The real filter is wrapped from the new component that internally use it to perform the job.
The idea of this new component is the records filtered could be useful in next steps of the job. Saving them, user can easily use them without implement other code and saving time.

Hi @benas,
I haven't found which is the problem with the tests in JDBC module. I run locally the build and the test and everything work well. Can help me please?
Obviously, give me a feedback about the new component.

This is a great idea!! Thank you for this PR 👍
It is a common use case to save filtered records for later use.
I've added a couple of notes to review.
I was about to release v5.2 but I will wait for your addition to include it in the release.
Take your time and update the PR. Then it will be ready to merge.
Also, don't hesitate to add your name to the contributors list in the README file.

I haven't found which is the problem with the tests in JDBC module. I run locally the build and the test and everything work well. Can help me please?

Don't worry about that. It looks like I broke the build here: 0dec0b6a92af94d3e854ff54d38c8fb8cb85d854
I just found that I'm building with Java 8 on my machine..
I will fix the build asap.

Hi,
I followed your suggestions, I hope that everything is fine now.
I hope you fix the build because is interesting see how change the project statistics after the commit.
Thanks a lot to admit me as contributor. Do you include other contributions?

Thank you for these updates! I'm merging your PR but I will apply a couple of changes:

No need to limit the size of filtered records to 5 : private List<Record> filteredRecords = new ArrayList<>(5);
No need to add getFilteredRecord(int index) and getFilteredRecordNumber to the api since this can be done by the user after getting all records with  getFilteredRecords

I will fix the build asap.

The build is fixed now.

Thanks a lot to admit me as contributor

You are welcome! Thank you for your contribution 👍

Do you include other contributions?

I'll review and comment on each PR.
Best regards
Mahmoud

Hi, under performance aspect, is better provide initial capacity that by default is 10.
Using the new component, most of the time I expected to save a few records, less of five, maintain the default size is a waste of memory. This kind of optimization help to decrease the footprint in memory of the programs.
The other two method, follow the same optimization idea, because providing them, you prevent two reflection access that are slow operations.
But the boss are you :) , so no problem if you want change something.

Hey,

This kind of optimization help to decrease the footprint in memory of the programs.

This kind of optimization is a premature optimization. The default size of 10 is perfectly fine.

The other two method, follow the same optimization idea, because providing them, you prevent two reflection access that are slow operations.

There is no additional reflection access . From an API design point of view, either:

You give me access to the whole list and let me do what ever I want with it
Or you hide the list and give me access to to only few methods you think useful

If you give access to the whole list (List<Record> getFilteredRecords()) and expose only get and size methods, why don't you also expose isEmpty and contains(Record) etc.. ?
As a client of the class, since you gave me the whole list, that's enough. I can check the size, get elements by index and so on.

But the boss are you :)

Nope, I'm not the boss. I'm just the first contributor 😄
But I am always open to discussions in a constructive way.


cutting pcb design with edb duplicates cells

Will fix this issue and also add small features to improve

@svandenb-dev I've implicitly fixed in #704


automatic generation of api documentation through conf.py

This PR changes conf.py to automatically generate the API reference without the need for calling the script from the Makefile. This notably also works on ReadTheDocs.

Any further work needed for this to be merged?

Apologies. I just haven't gotten around to reviewing it yet :-/
On Apr 13, 2016 8:45 AM, "Andreas Mayer"<EMAIL_ADDRESS>wrote:

Any further work needed for this to be merged?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/uwescience/shablona/pull/28#issuecomment-209518490

I can leave some formatting / approach comments. Don't have time to pull it & test it at the moment.

Thanks bcipolli for your comments. I've pushed a new commit fixing these.

Hi. Again, my apologies that this has taken so long. I am getting the following error, when I run make html:
D-108-179-172-65:doc (andim-apidocgeneration) $make html
sphinx-build -b html -d _build/doctrees   . _build/html
Running Sphinx v1.3.4
making output directory...
Initializing GitHub plugin
loading pickled environment... not yet created
loading intersphinx inventory from http://docs.python.org/objects.inv...

Exception occurred:
  File "/Users/arokem/source/shablona/doc/tools/buildmodref.py", line 44, in writeapi
    source_version = __version__
NameError: name '__version__' is not defined
The full traceback has been saved in /var/folders/jz/4k438hl91v7gvdd592ct4lpc0000gn/T/sphinx-err-173mlzeu.log, if you want to report the issue to the developers.
Please also report this if it was a user error, so that a better error message can be provided next time.
A bug report can be filed in the tracker at <https://github.com/sphinx-doc/sphinx/issues>. Thanks!
make: *** [html] Error 1

It works on my machine, but in order to make the import of the version file more robust I've pushed a new commit building the path relative to buildmodref.py. Let me know whether that works on your machine.

It works on my machine, but in order to make the import of the version file more robust I've pushed a new commit building the path relative to buildmodref.py. Let me know whether that works on your machine.

Seems like a Python2/Python3 issue. This works fine on Python 2, but still
errors as before on Python 3.
On Tue, Apr 19, 2016 at 9:10 AM, Andreas Mayer<EMAIL_ADDRESS>wrote:

It works on my machine, but in order to make the import of the version
file more robust I've pushed a new commit building the path relative to
buildmodref.py. Let me know whether that works on your machine.
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/uwescience/shablona/pull/28#issuecomment-211999440

With a little bit more digging, I am slowly realizing that this might not be due to your changes, but is already there as a bug on master. Still digging.

Wonderful! Thanks so much for the contribution, and for your patience while it took us time to review.

Great to see this merged! Happy to contribute to such as community resource, profits everyone.


More clarity wrt to not charging for VNet peering for traffic passing through a remote gateway

For the https://docs.microsoft.com/en-us/azure/virtual-network/virtual-network-peering-overview#pricing section:

It makes sense to explicitly call out that traffic passing through VNet peering and then remote gateway is not charged for VNet peering only if flows directly to the gateway, not through an NVA (in the hub VNet) before reaching the gateway.
Would be good to mention in the example with VNetA and VNetB that VNet peering is not charged for both traffic from the peered VNet (VNetB) to on-premises and from on-premises to VNetB.

Thank you!

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

ID: cc20a64a-096f-7e72-1fb5-436e3366fbd5
Version Independent ID: d1ba1c1b-5d36-71eb-5442-cf17de18a13b
Content: Azure Virtual Network peering
Content Source: articles/virtual-network/virtual-network-peering-overview.md
Service: virtual-network
GitHub Login: @anavinahar
Microsoft Alias: anavin

Thanks for the feedback! We are currently investigating and will update you shortly.

@mgaufman Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

This traffic is charged.

#please-close


[release/1.3] Update Golang 1.12.14

go1.12.14 (released 2019/12/04) includes a fix to the runtime. See the Go 1.12.14
milestone on our issue tracker for details:
https://github.com/golang/go/issues?q=milestone%3AGo1.12.14+label%3ACherryPickApproved

Codecov Report

Merging #3917 into release/1.3 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/1.3    #3917   +/-   ##
============================================
  Coverage        42.07%   42.07%
============================================
  Files              129      129
  Lines            14322    14322
============================================
  Hits              6026     6026
  Misses            7395     7395
  Partials           901      901

Flag
Coverage Δ

#linux
45.59% <ø> (ø)
:arrow_up:

#windows
37.01% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


gollvm: some built-in libgo tests failed

What version of Go are you using (go version)?

$ go version
go version unknown linux/amd64
llvm-project/llvm/tools/gollvm$ git log --oneline | head -n 1
9bc0905 gollvm: fix libgo cmake rules to include new C source

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?

Ran all built-in libgo tests:

build-release$ for target in $(ninja -t targets | grep check_libgo_ | sed -e 's/\(.*\):.*/\1/') ; do ninja $target 2>&1 | grep "FAILED:" ; done

What did you expect to see?
Nothing, all tests pass
What did you see instead?

FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_internal_fmtsort
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_math_cmplx
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_mime
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_os_signal
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_runtime
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_runtime_debug
FAILED: tools/gollvm/libgo/CMakeFiles/check_libgo_runtime_pprof

internal/fmtsort fails due to the same issue described at #51238. I've got a fix for this one already. I will submit a cl soon.
os/signal requires cmake files to be updated, I'll submit a cl fixing this.
runtime fails due to the same issue described at #51098.
runtime/debug requires the same check added for runtime in gofrontend with https://go-review.googlesource.com/c/gofrontend/+/140517/
I also may have a fix for runtime/pprof but I'm not sure yet.

Mikhail Ablakatov

cc @thanm

Thanks for that investigation.

I think that everything here is fixed now.  Thanks for reporting the problems.


docs: fix hashbang example for linux env command

Turns out there is a behaviour difference in the GNU env tool between MacOS and Linux where the latter does require -S for it to work. I've confirmed this working now with:

env (GNU coreutils) 9.5 on MacOS 14
env (GNU coreutils) 9.5 on Arch Linux
env (BSD) on MacOS 14

I posted this as an answer one https://stackoverflow.com/questions/55777677. Also see https://github.com/TypeStrong/ts-node/issues/639#issuecomment-538984217 for similar solution.

:tada: This issue has been resolved in v4.7.3
If you appreciate this project, please consider supporting this project by sponsoring :heart: :pray:


Correct the variable name for the script class path in an example

Summary
Correct the name of the variable for the script class path in an example.
Additional Details
None
Checklist
Testing
N/A - no code changes
Documentation
N/A - no code changes

Besides fixing the variable name it's probably a good idea to add to the array instead of replacing it?

Hi @lionelve, I don't get what you mean. Could you explain it please? My pull request is quite old. :innocent:


Can't install in windows

Errors throwed when installing rn-webview-rpc from npm.
Windows CMD doesn't support cp -r dist/* .
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>postinstall: `cp -r dist/* .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      C:\Users\Dell\AppData\Roaming\npm-cache\_logs\2020-04-09T15_15_34_677Z-debug.log

But something confuses me is when I check git://github.com/ronhe/comlink.git, you know, "postinstall" doesn't exist in the packages.json. Is this something magic...?
Sorry for poor grammar.

Hi @Asuka109,
First, please note that WebView api is deprecated both in iOS and Android, so I advise you not to use it for new projects.
Seconds, notice #dist at the end of git://github.com/ronhe/comlink.git#dist. It means that the comlinkjs dependency is installed from the dist branch where the postinstall script exists.
Third, to solve your error easily, you can run npm install  rn-webview-rpc inside a console emulator, such as cmder or Git BASH.


barycentric extrapolation should not override interpolation

I have made a mistake when designing the “extrapolation” part of the barycentric interpolator. Adding points to the frame can create huge distortions when the frame doesn't include all data points—for example when we zoom in on a region.
If we think that points that live inside the convex hull should be governed by the normal triangulation (as I do now), we have to rethink how extrapolation works.

I have explored a solution here: https://observablehq.com/@observablehq/barycentric-extrapolation-1700

Looks good to me!


ci: align semantic regex for version update

Current regex is not matching beta releases - this PR fixes that.
Test run: https://github.com/splunk/test-addonfactory-repo/actions/runs/9516876272

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


wit #1788: Adding MultiTenant info to CheServerState

What issues does it solve? #1788
Adding info about che-server type (single-tenant / multi-tenant)
Related che-starter PR https://github.com/redhat-developer/che-starter/pull/247

@ibuziuk snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1789-2

@xcoulon instructions for running wit against prod-preview seems to be changed. There is no public key information in config.yaml anymore, so I was not able to test the PR. Could you please review ?

Codecov Report

Merging #1789 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1789      +/-   ##
==========================================
- Coverage   57.49%   57.48%   -0.02%
==========================================
  Files         129      129
  Lines       15260    15264       +4
==========================================
  Hits         8774     8774
- Misses       5783     5787       +4
  Partials      703      703

Impacted Files
Coverage Δ

codebase/che/client.go
11.24% <ø> (ø)
:arrow_up:

controller/codebase.go
29.28% <0%> (-0.32%)
:arrow_down:

Continue to review full report at Codecov.

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

@ibuziuk what do you need the public key for?
make dev should still work.

@alexeykazakov @xcoulon AFAIK wit will not run against prod-preview by default. Instructions in Readme.md are now oboslete:

https://github.com/fabric8-services/fabric8-wit#2-configuration-file
https://github.com/fabric8-services/fabric8-wit#5-prod-preview-setup

Could you please explain how to run wit locally, so that prod-preview auth would be used ?

@ibuziuk yeah.. Readme has to be updated. But again. Simple "make dev" with all default configuration should work for you. It will start Auth and DBs locally and configure them to work with the local WIT and prod-preview Keycloak.


feat(lang): add .astro support

Overview
Astro is a meta JavaScript framework for building snappy UI. It leverages island architecture and zero-JS by default to achieve faster performance by default. It supports other UI library integrations such as React, Vue, and Svelte.
Reference

Astro Documentation
Compiler Package

This is a challenge:

There are 2 separate paths to be imported @astrojs/compiler for parsing and @astrojs/compiler/utils for generic walk function
@astrojs/compiler doesn't actually provide a walker function for the frontmatter, meaning we have to parse it ourselves.
If we must parse it by ourselves, we don't even know if the frontmatter is written in TypeScript or not. If it's written in TypeScript, then we need to load the TS compiler manually since it doesn't get loaded in astro flow.

Generally, this issue can be completely fixed if we move on from acorn to typescript as JS parser. More details later.


Fixed xhamster getnextpage

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[x] a bug fix (Fix #943)

Description
Fixed the getNextPage function
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[ ] I've verified that this change works as intended.

[ ] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[ ] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Coverage increased (+0.07%) to 41.354% when pulling b04d123fec0dadaebe40df4f27ddce0f4a48dd91 on cyian-1756:xhamsterPageFix into c788753ed3061675e737eb98dad62b505aa52807 on RipMeApp:master.


chore: remove Java 8 from v2 examples

Issue #1466
Description of changes:
Removed Java 8 from all examples.
Deployed and smoke tested all examples using Java 11 updating some README files as I went
Checklist

[ ] Meet tenets criteria
[ ] Update tests
[ ] Update docs
[ ] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

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

:exclamation: No coverage uploaded for pull request base (v2@e7757c7). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff          @@
##             v2    #1531   +/-   ##
=====================================
  Coverage      ?   77.75%
  Complexity    ?      623
=====================================
  Files         ?       83
  Lines         ?     2504
  Branches      ?      270
=====================================
  Hits          ?     1947
  Misses        ?      469
  Partials      ?       88

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


[Not a bug]

AMAZING PLUGIN!!
Really the best one out there! you are so awesome, thank you!!
I was just wondering two things:

) Is the CI Build stable?
) Is the CI Build most recent (ie: If I want the latest features, I should use this one?) - Such as, does the CI Build have any features that do not exist in the market place version?

Thanks!! <3

Yes, the CI build is stable and is always the most recent. It might be the same version as what's on the Marketplace, but it will never be older. I update the Marketplace pretty frequently, but you will find newer features from time to time in the I build


Discord link is expired

Question and Help
Can I get an updated link? I can't get this to work at all in my Next project . . .

At present, the discord server is not active but still here's the invite,
Discord: https://discord.gg/7cKycj94yB
But if you want my discord tag, here it is NO U#1792.


Terminal shows texts one letter per row since 1.16

Expected behavior
Show text properly
Actual behavior
Shows text one letter per row.
For example like this:
D
i
s
p
l
a
y
s

How do you echo this text? can you share the code you are using?

Do you have updated css file?

It seems that cmd::resize is broken in FireFox don't know why it work in Chrome. Maybe that's the issue you have.
But when terminal runs it always invoke cmd::resize with number of characters that's calculated in terminal and that was ok.

Thanks, I will check more. I tested in Chrome BTW. For now I postponed it but I'll get back to it soon.
It started in 1.16. 1.15 Fine.

Can you create a demo with the problem? Because I cannot reproduce the issue.

Hey did you have a chance to test this again? Did you update css file?

Closing because of inactivity if you still will have a problem I can reopen.


Cloud Formation Fails on us-east-1 t2.micro instances

Loading the master.template on CloudFormation with 2 indexers, I get the following error in the middle of the process:
16:20:06 UTC-0400	CREATE_FAILED	AWS::CloudFormation::Stack	BastionHost	Embedded stack arn:aws:cloudformation:us-east-1:926291082449:stack/Splunk-BastionHost-1JDRJ8V6LBJVQ/189bfc50-5d79-11e5-bea4-50fa1dbb2c64 was not successfully created: The following resource(s) failed to create: [ControllerCondition].
Physical ID:arn:aws:cloudformation:us-east-1:926291082449:stack/Splunk-BastionHost-1JDRJ8V6LBJVQ/189bfc50-5d79-11e5-bea4-50fa1dbb2c64
Can someone let me know how to troubleshoot this further?  Thanks!

Note: This operation was run using the Admin account.

This might be related to issue #5

Hey @dkindlund, it appears BastionHost EC2 instance bootstrap either failed or timed out. To troubleshoot this further, here are some steps to follow:

In CloudFormation web console, under status tab, refer to the column Status Reason of the resource with logical ID ControllerCondition which failed to create based on the error message above. It should include a specific error message, e.g. "Failed to run cfn-init".
SSH to EC2 instance (BastionHost), and refer to these logs:

/var/log/cloud-init.log
/var/log/cfn-init.log

Feel free to paste relevant contents from those logs in here.
Please note that default behavior of CloudFormation stack creation is to rollback & delete all underlying resources upon failure, so the EC2 instance may have been deleted. You'll need to disable rollback by clicking the right checkbox while going through the CloudFormation create stack wizard.


Mixed content error http vs https

Hi,
I have a problem when I load my site with https protocol, and the script (gmaps) loads over http.
The message is the following:
was loaded over HTTPS, but requested an insecure script 'http://maps.googleapis.com/maps/api/js?sensor=false'. This request has been blocked; the content must be served over HTTPS.
Thx,

Fixed!

Thanks!
El sábado, 11 de julio de 2015, Bruno Rocha<EMAIL_ADDRESS>escribió:

Fixed!
—
Reply to this email directly or view it on GitHub
https://github.com/rochacbruno/Flask-GoogleMaps/issues/12#issuecomment-120662968
.

Please, when you can, change
DEFAULT_ICON = 'http://maps.google.com/mapfiles/ms/icons/red-dot.png'to
DEFAULT_ICON = '//maps.google.com/mapfiles/ms/icons/red-dot.png'
Thanks!
On Sat, Jul 11, 2015 at 11:39 PM, Bruno Rocha<EMAIL_ADDRESS>wrote:

Fixed!
—
Reply to this email directly or view it on GitHub
https://github.com/rochacbruno/Flask-GoogleMaps/issues/12#issuecomment-120662968
.

@joanbales done!


Small Typo on Installation Page

There is a small typo here: http://vuejs.org/guide/installation.html#CSP-compliant_build
"Secuirty" is written wrong. It's Security. =)

There is another mistake here: http://vuejs.org/guide/#Model "instnace"


Make package.json ignored by git

I think I have to include package-lock.json because I have to save dependencies that is built between I and other devs/users that cloned this repository. Behaviors of software might depends on version of libraries so I won't merge this pull request.

And please explain about overview of your pull request at the title.

I think I have to include package-lock.json because I have to save dependencies that are built between me and other devs/users that cloned this repository. Behaviors of software might depend on the version of libraries so I won't merge this pull request.

I meant package-lock.json sorry, package-lock.json isn't needed for everyone, only package.json, package-lock.json is more of a custom thing compared to package.json which is more of a global thing, it is a shared dependency manager for the project unlike the package-lock.json which is more custom to the dev's profile, its an automatically generated file anyway. so not really applicable to everyone.
and sorry about the commit name, it just consisted of better grammar/wording and removing the unnecessary package-lock.json from the master branch,


vsphere CPI must place the "director_cid" in the agent's env, and not…

… the "clean_cid".  Since encoding metadata in the CID is something specific to this CPI, the director has no clue it must remove the metedata from the CID when it talks to the agent.  The simplest way to keep the director unaware of this, would be to have the agent also use the "director_cid", so the director can talk to the agent using "director_cid" for disk operations.

A further improvement might be using . as the disk ID everywhere...  The ID is generated by vsphere-CPI (UUID) and not by vCenter...  so would be simpler if the CPI, director, agent all used the same ID, which would happen to have metadata suffix for disk pools with datastore.

@dickeyf confirmed that this is a bug, I'll get your change merged shortly. I like the suggestion of using the "cid"."metadata" format everywhere to avoid confusion, but I remember vSphere has a character length limit of disks that wouldn't allow it.
Thanks for the heads up and PR!

@dickeyf Thanks again for the PR, once it runs through CI we'll cut a new CPI release (v29). We also added an integration test here to ensure this feature functions as expected going forward.


The bot does not play music

Sorry, can you be a little more specific?

If you do not know already you have to add a song to the queue by &add <yt link> then type &play which will play songs from the queue.
FYI, restarted Music maybe it'll work now, please let me know.

No reply since past 48 hours, closing issue.


feat(dummy diag publisher): change diag name specification method to YAML

Signed-off-by: asana17<EMAIL_ADDRESS>Description
diag_name parameter of the ROS node dummy_diag_publisher is configured in the node launch file.  dummy_diag_publisher can take only one diag name parameter and the ROS node is launched for each diagnostic. This causes differences in the launch files among projects.
I changed to publish diagnostic messages in one ROS node and multiple diag_name parameters can be configured in the parameter YAML file. status and is_active parameters of each diagnostic can be reconfigured from the command line.
I also changed the launch files to update tier4/autoware_launch. Please see the related links below for more details about the launch files.
(I reopened this PR for https://github.com/autowarefoundation/autoware.universe/pull/2745.)

Changes

Use parameters in the YAML file as the name of diagnostics instead of the parameter in the launch file.
Use new parameters to read from the YAML file and enable parameter reconfiguration for each diagnostic.
Multiple diagnostics messages are published from one ROS node.

Related links

TIER IV INTERNAL LINK TO SLACK
https://github.com/tier4/autoware_launch/pull/762
https://github.com/autowarefoundation/autoware_launch/pull/206
https://github.com/tier4/aip_launcher/pull/101
https://github.com/autowarefoundation/sample_sensor_kit_launch/pull/55
Tests performed
dummy_diag_publisher

Run dummy_diag_pubilsher and rqt_runtime_monitor.
The config YAML file is system/dummy_diag_publisher/config/dummy_diag_publisher.param.yaml

ros2 launch dummy_diag_publisher dummy_diag_publisher.launch.xml

2. Change the parameter from the command line.
ros2 param set /dummy_diag_publisher velodyne_temperature.status "Error"

ros2 param set /dummy_diag_publisher obstacle_crash.is_active false

launch files
I used tier4/pilot-auto for the tests. I replaced universe , launcher, and aip_launcher. I confirmed the planning simulator works successfully in case 1 below.

Launch planning simulation. (x1, xx1)

terminal 1
source ~/pilot-auto/install/setup.zsh
ros2 launch autoware_launch planning_simulator.launch.xml map_path:=/home/asana/autoware_map/sample-map-planning
vehicle_model:=lexus sensor_model:=aip_xx1

terminal 2
ros2 run rqt_runtime_monitor rqt_runtime_monitor

ros2 node list

ros2 topic list

Launch system component (x1, xx1, x2)

terminal 1
ros2 launch autoware_launch tier4_system_component.launch.xml run_mode:=online launch_system_monitor:=true sensor_model:=aip_x2

ros2 launch autoware_launch tier4_system_component.launch.xml run_mode:=planning_simulation launch_system_monitor:=false sensor_model:=aip_x2

terminal 2
ros2 run rqt_runtime_monitor rqt_runtime_monitor

ros2 node list

ros2 topic list

In each case, I compared the rqt_runtime_monitor results, the node list results, and the topic list results and confirmed no difference other than the points below.

service_log_checker is launched in tier4_system_launch, but is not launched in the previous system_launch.
The changes of  dummy_diag_publisher nodes: all dummy diags are published by one dummy_diag_publisher node, and the diag names, Hardware IDs are changed as the screenshots.

Notes for reviewers

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

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

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

[x] The PR follows the pull request guidelines.
[x] The PR has been properly tested.
[x] The PR has been reviewed by the code owners.

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

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

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

@kminoda I closed the previous PR because I signed off some commits by mistake. Could you check this PR instead of the previous one (https://github.com/autowarefoundation/autoware.universe/pull/2745)?

@ito-san @kenji-miyake Could you review this PR?

By the way, it's not a bug fix, I changed the type to feat.


Add a benchmarking script for nvtabular triton models

This adds a simple script to benchmark nvtabular triton models, using
triton's perf_analyzer utility.
Since perf_analyzer requires a fairly long commandline with nvtabular
(we need to specify shape for each column), and we need to pass example
data - this wraps up the execution to automatically set the appropiate
commandline and transform a csv/parquet file with data into the JSON
format perf analyzer expects.
This saves out the results from perf_analyzer for various batch sizes to
a dataframe, and contains some basic matplotlib code to help visualize

The output of this benchmarking script is kinda interesting to compare the cpp_backend and python_backend.
With the movielens example, we don't see much difference between the cpp_backend and python_backend:

With the rossmann example, at smaller batch sizes the cpp backend and python backend are basically identical - but the cpp_backend starts to break away as results get higher.

While the cpp_backend is doing better here with rossmann - the times are still too slow to be acceptable. Looking at the inference POC I put together last week (zoomed into the first 1k rows), shows that we are starting to get to our < 5ms target for latency here.


[NEW]: 3 projects in 1 issue (SwarmNFTify, Watch!, SwarmBootOS

Introduction
I am Chris. I enjoy searching for ways to integrate networks in the web3 ecosystem with each other, with the objective of building network-2-network connections, at the human, client, and protocol levels.
I got into Ethereum because of Livepeer - a project to build video on web3, and also work as part of #TestingTheMerge.
ETHBerlin³ is inevitable for me. Post-pandemic hackathon in the cultural melting pot that is Berlin. Unmissable IMHO.
Idea description
I've got a lot of ideas. Top of my mind are:

SwarmNFTify: select folder, upload contents to Swarm, and mint each file as NFT. This issue offers no implementation suggestions, but can be imagined to be applied to photo / music / video collections sitting on all our endless hard drives.
Skillset: UX Research, Full Stack Dev (golang perhaps useful), Graphic Designer, UI Dev.

Watch!: reimagine the "smartwatch" for a pure web3 context. No AppleID, no Google. Conceptual device design project, with hardware and software symbiosis, leveraging new "cloudchain" back-end superpowers afforded to a user by virtue of their participation in Ethereum.
Skillset: Software Architecture, Industrial Design.

SwarmBootOS: boot a diskless RaspberryPi by loading a Unix Operating System into RAM from storage on Swarm. Documented instructions are available for doing similar using network storage (please ask below in a comment if you would like to audit the instructions).
Skillset: Unix SysAdmin, Software Developer

Skillset
These are listed above, but I am very open to alternative approaches.
Communication
Please leave a comment below if you have any questions.

Hi Chris, these projects sound cool and I'm interested in joining (especially 3). Let's chat: @andrew:freemath.xyz

Hi @awmacpherson thanks for the comment.
Re. 3. here is a starting point for diskless booting using pxe, ramdisk and network storage. Hopefully it can provide a decent starting point: http://panic.fluff.org/stateless/hive/ (the name "hive" is in no way related to "swarm" or "bee" it's coincidence)


Revisit pub types

For a binary crates, most of the types should be local to the crate. Revisit existing crates to see any violations.

I'm surprised its not implied by the fact it is a binary crate. Can you even attempt to link it to a wider project?

Maybe I am not sure. What if a binary crate wants to export some types so an external project can use them too ? Should be possible, but not required in our case.

Anyway the change look good to me.


Fabric and AWS CloudHSM doc updates

Hello,
Are there any detailed docs about how to configure fabric with AWS CloudHSM?
I've set up a CloudHSM instance and can connect to it with the cloudhsm_mgmt_util tool. I created a CU "fabric" user and I configured the fabric-ca-client tool like this:
bccsp:
  default: PKCS11
  pkcs11:
      Library: /opt/cloudhsm/lib/libcloudhsm_pkcs11.so
      Pin: 'fabric:password'
      AltId: FABRIC
      Label: cavium
      hash: SHA2
      security: 256

However when I try to enroll a user with fabric-ca-client I get this error:
Error: Failed to get BCCSP with opts: Could not initialize BCCSP PKCS11: pkcs11: could not find token with label cavium
Here is the libcloudhsm log:
root@ee2506438d16:/etc/hyperledger/fabric-ca-client# cat /opt/cloudhsm/run/cloudhsm-pkcs11.log.2021-10-06
2021-10-06T20:20:03.895+02:00 INFO  [30] ThreadId(5) [cloudhsm_provider::hsm1::connection::connection_pool] Adding HSM connection to connection pool: HsmConnection { hsm_info: HSM { IP: "<IP_ADDRESS>", Port: 2223 } }
2021-10-06T20:20:03.899+02:00 INFO  [30] ThreadId(5) [cloudhsm_provider::hsm1::hsm_connection::server_connection::common] Initializing new connection: HSM { IP: "<IP_ADDRESS>", Port: 2223 }
2021-10-06T20:20:03.944+02:00 INFO  [30] ThreadId(5) [cloudhsm_provider::hsm1::hsm_connection::server_properties] Version handshake with server succeeded. Received version: ComponentVersion { major: 2, minor: 3 }
2021-10-06T20:20:03.945+02:00 INFO  [30] ThreadId(5) [hsm1_marshaling::server_handshake] Reporting sdk version Pkcs11:5.2-1-bionic:CodeBuildBatchProject-uFu5sNXfquqK:ed9a3dbd-7172-439f-8ee3-79408d4e3f39
2021-10-06T20:20:03.992+02:00 WARN  [30] ThreadId(2) [cloudhsm_provider::hsm1::hsm_connection::dispatcher_strategy] UX000: Discarding HSM response because it does not match a pending request.
2021-10-06T20:20:04.043+02:00 INFO  [30] ThreadId(5) [cloudhsm_provider::hsm1::connection::device] Retrieving HSM information.

(I know I'm talking about fabric-ca-client and not fabric-peer or orderer but I guess it's the same code between the 2 projects)
In any case, a detailed setup guide with AWS CloudHSM would be very welcome in the Fabric docs.

After some extensive googling I found that the label should be the HSM cluster name:
This gets me a bit further (I init a brand new fabric ca server here as a simpler example than enroll):
env FABRIC_CA_SERVER_BCCSP_PKCS11_LABEL=cluster-dzetr75hznp fabric-ca-server init -b admin:adminpw

2021/10/06 22:14:40 [FATAL] Initialization failure: Failed generating ECDSA P256 key: P11: keypair generate failed [pkcs11: 0x13: CKR_ATTRIBUTE_VALUE_INVALID]

2021/10/06 22:14:40 [FATAL] Initialization failure: Failed generating ECDSA P256 key: P11: keypair generate failed [pkcs11: 0x13: CKR_ATTRIBUTE_VALUE_INVALID]

Actually that error was because the AltId value was the same between 2 fabric-ca-server init runs... Now I have that same issue with the TLS cert generation when starting the ca server. Maybe it's trying to use the same AltId to generate the TLS cert in the HSM?
Anyway I'll report that issue in the fabric-ca repo...

@denyeart Looks like the label field is dependent on the version of the cloudhsm library used. For v3 it's cavium but for v5 its looks to have changed and is the cluster name (I would hope that the AWS docs would provide this information though). We could add some detail to the docs about this as well as iterate that the AltId value for each certificate in the same HSM must be unique.

@theblop Im facing the same issue, are you planning to write a blog or if you could explain the resolution here would be of great help, thanks.

HI can u guys confirm if  documentation for cloudhsm integration for fabric is updated

The suggestions from this issue are not yet included in Fabric docs. We would welcome anybody with knowledge/experience on this topic to edit this page:
https://github.com/hyperledger/fabric/blob/main/docs/source/hsm.md


Missing changelog for 5.6.x

I just saw there was a 5.6.3 release and went to see what changed. It looks like the changelog fell out of date. There's nothing for the 5.6.x series at all in the changelog/ directory. Fortunately, there weren't many commits between 5.6.2 and 5.6.3, so I could work that out by digging through the code. But, that process becomes a lot harder the more releases there are.

Hi @nirvdrum
Sorry, the original idea was to only add breaking changes to the changelog. This is quite sloppy. When I made this decision, CRACO was still my first OSS project and I did not care much about these concerns.
Learned a few tricks along the road. In the future, i'lll try to be more accurate and list all the changes in the matching Github release.
Thank you,
Patrick

No problem. Thanks for putting this project together in the first place.


Parsing error for generic function named 'of'

Hi guys, thanks for this great tool :) I seem to have found a small issue with generic functions and the name 'of'. Hoping this is the right repository to report this to - if not, please let me know where to put this.
Bug description
Babel-eslint parses a generic function just fine:
function someGenericFunction<R>(args: R): Array<R> {
  return [args];
}

It fails to parse that same function when it is named of, however:
function of<R>(args: R): Array<R> {
  return [args];
}

/path/to/index.js
  5:12  error  Parsing error: Unexpected token, expected "("

  3 | }
  4 |
> 5 | function of<R>(args: R): Array<R> {
    |            ^
  6 | 	return [args];
  7 | }
  8 |

✖ 1 problem (1 error, 0 warnings)

error Command failed with exit code 1.

Flow accepts the function just fine and shows 'No errors!'.
It would seem that of is treated as a keyword - maybe because of the for...of syntax?
Related info

I'm using these dependencies:

    "babel-eslint": "^10.0.3",
    "eslint": "^6.6.0"

I've also tried this with "parser": "eslint" which obviously fails due to the typing syntax, unrelated to this bug.
The same error is produced with eslint-plugin-flowtype added.

Reproduce it
Reproduction repository: https://github.com/alexwalterbos/generic-function-of
Built with yarn - clone the repo, run yarn install and yarn eslint . in the repo to produce the error.
Workaround
Don't name your generic functions of.
Cheers!

Thanks for the report. Since this is a parser error, can you file this in the babel/babel repository? Thanks!

Reported here https://github.com/babel/babel/issues/10675


Create documentation on linear search

please review sir

The code should be formatted with MD syntax and in the README.md file. For the syntax check this


template fails to render page with a broken symlink

Directory Lister Version

PHP Version

7.4
PHP Extensions

https://hatebin.com/qwprurnffn
HTTP Server

nginx/1.18.0 (Ubuntu)
Describe the bug

DirectoryLister fails to render a page if a broken symlink is present in the directory, empty files are not affected
Steps To Reproduce

Create a broken symlink and attempt to view the directory with DirectoryLister
Expected behavior

Mark the file as broken / 0kb in size
Additional context

https://hatebin.com/yqepoctaou

Thanks, I'll see what I can do about this when I have some free time.

I believe I've fixed this with<PHONE_NUMBER>e36a43213dbb2a8bd50a9c09f0a100, will be available in the next release (not sure when that will be at the moment).


Calculator example added

Please describe the changes this PR makes and why it should be merged:
A new example added to kitchen sink, extending Million's use cases
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.013%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1267

Relevant Lines:
5065

💛  - Coveralls

@oliverloops amazing work! make sure to format w/ prettier + wrap the Screen component in a block

Sure!, I'll update the PR soon

I already formatted with prettier and added the block to Screen component. I also I added a react bare bones todo list example to complement

amazing work @oliverloops


'Search' highlight has lower priority than 'Visual'

Describe the bug
When doing visual selection over search results, highlight of search results disappear.
To Reproduce
Detailed steps to reproduce the behavior:

Run vim -g -u NONE +'set hlsearch'
Open some text file and search for something. Results will be highlighted with yellow.
Start selecting the lines that contain the search results.
Search highlight is overruled by Visual.

Expected behavior
Search highlight is not overruled by Visual.
Screenshots

Environment (please complete the following information):

Vim 8.2.2650
OS: Ubuntu 20.04
Terminal: GUI

It is correct that the Visual highlighting has higher priority, since it's temporarily while search highlighting stays around.  It would also be difficult to see the Visual area if there are many search matches (e.g. whole lines).
However, I notice that the highlight is overruled, while it could be combined.  You could make Search highlight bold, then it would still be visible.


Try.withResources() with Consumer

It would be nice that can use Try.withResources() with Consumer.
But now, Try.withResources() only has of(CheckedFunction1<? super T1, ? extends R> f).
What do you think adding ofConsumer(CheckedConsumer<? super T1> consumer)?
File file = new File(absoluteFilePath);
Try.withResources(() -> new FileWriter(file, false))
   .of(writer -> {
       writer.write(contents);
       return true;
   }).getOrElseThrow(t -> new RuntimeException(String.format("File overwrite failed. path: %s", file), t));

// above code can be changed like following
File file = new File(absoluteFilePath);
Try.withResources(() -> new FileWriter(file, false))
   .ofConsumer(writer -> {
       writer.write(contents);
   }).getOrElseThrow(t -> new RuntimeException(String.format("File overwrite failed. path: %s", file), t));

Hi @dlsrb6342, I'm afraid, we will drop the Try.withResources API in Vavr 1.0.
The migration path is to use native try-with-resources blocks within Vavr's Try.
You already can do it! Please write your example like this:
File file = new File(absoluteFilePath);
Try.run(() -> {
  try (FileWriter writer = new FileWriter(file, false)) {
    writer.write(contents);
  }
}).getOrElseThrow(t -> new RuntimeException(String.format("File overwrite failed. path: %s", file), t));

The Try.withResources method overloads introduced

unnecessary runtime overhead (additional objects got instantiated because of the fluent API)
unnecessary big API surface area

The suggested approach outlined above has a smaller footprint and it is still easy to use. No additional vocabulary is needed, it is native Java notion.
I hope you understand our design decision.
Thanks for using Vavr!
- Daniel


The creation of the prepared statements for the postgresql auth

This auth plugin panic'ed due to a slightly slienced bug in the auth/postgresql.go plugin.
When creating the a.authStmt the returned error is disregared: https://github.com/wind-c/comqtt/blob/main/plugin/auth/postgresql/postgresql.go#L97
After adding a check here I saw that sqlx didn't like the syntax of the SQL query. Changed to use $1 instead of '?' fixed the issue.

good job


Hide non mobile optimized menu items

closes https://github.com/covidwatchorg/portal/issues/467

Looks great - and seems easy to roll back once we get the other pages mobile-friendly.
Do we care about the tablet view at all? It seems like most pages work for tablet except the settings page, pictured here:

Not sure how many people will be using tablets, but it's worth considering - especially as the includemedia library can target specifically the tablet view.

Probably eventually we'll want that, but for now let's just focus on getting desktop to play nicely with mobile


capture more network information to improve waterfall

Couple of improvements that we can do to improve the step level waterfall in the UI

[x] Move the overall timing calculation logic in the UI to the agent to improve the performance #168
[x] Use requestSentime and responseReceivedTime in favor of start/end time and use the monotonic clock in start
and end time
[x] Capture the step index info for all the netinfo events to make querying easier
[x] Fix the timestamp to get the value of when the event got generated vs when the event is written to stdout which is after journey end.
~+ [ ] Enhance the additional headers information by listening to requestWillBeSentExtraInfo and responseReceivedExtraInfo event~
~+ [ ] Figure out if an entry is from the cache by using requestServedFromCache event.~

Removing capture web vitals and other RUM metrics - #120 from the list of requirements as we're tracking #120 separately.

Since we don't actually need the last two items for our UI right now I've crossed this out, and am closing the issue.

Since we don't actually need the last two items for our UI right now I've crossed this out, and am closing the issue.


Separate puc docs

Docs for prebid-universal-creative #165
In prebid docs, replace dist/creative.js (and native-render.js) with dist/%%PATTERN:hb_format%%.js.## 🏷 Type of documentation

[ ] new bid adapter
[ ] update bid adapter
[ X ] new feature
[ ] text edit only (wording, typos)
[ ] bugfix (code examples)
[ ] new examples

@musikele musikele
Please see previous comments.

@musikele - hi, the related code PR is still open. Any idea when it will be resolved?

Update: I put a request in the slack channel about the code status

@musikele Please resolve conflicts.

Fixed the conflicts. Is there any reason to mention the old way of doing things?

Updated the main PUC page to specifically address this major change. What's not done with this PR so far is to change the other AdOps docs.

Ok - think this is ready to go now. Also adding @patmmccann as reviewer.


Do not display screen in GLMakie picking test

Description
As suggested by @ffreyer in #2063.
Type of change

[X] Bug fix (non-breaking change which fixes an issue)

Compile Times benchmark
Note, that these numbers may fluctuate on the CI servers, so take them with a grain of salt. All benchmark results are based on the mean time and negative percent mean faster than the base branch. Note, that GLMakie + WGLMakie run on an emulated GPU, so the runtime benchmark is much slower. Results are from running:
using_time = @ctime using Backend
# Compile time
create_time = @ctime fig = scatter(1:4; color=1:4, colormap=:turbo, markersize=20, visible=true)
display_time = @ctime Makie.colorbuffer(display(fig))
# Runtime
create_time = @benchmark fig = scatter(1:4; color=1:4, colormap=:turbo, markersize=20, visible=true)
display_time = @benchmark Makie.colorbuffer(display(fig))

using
create
display
create
display

GLMakie
–
–
–
–
–

master
–
–
–
–
–

evaluation
–
–
–
–
–

CairoMakie
11.20s (11.15, 11.31) 0.06+-
23.39s (23.32, 23.48) 0.05+-
3.46s (3.45, 3.48) 0.01+-
17.34ms (17.21, 17.42) 0.07+-
25.08ms (25.01, 25.30) 0.10+-

master
11.31s (11.27, 11.34) 0.03+-
23.37s (23.34, 23.42) 0.03+-
3.46s (3.45, 3.47) 0.01+-
17.54ms (17.48, 17.62) 0.05+-
25.07ms (25.01, 25.17) 0.05+-

evaluation
-0.91%, -0.1s faster ✓ (-2.28d, 0.00p, 0.04std)
+0.07%, 0.02s invariant (0.39d, 0.49p, 0.04std)
+0.15%, 0.01s invariant (0.50d, 0.37p, 0.01std)
-1.12%, -0.19ms faster ✓ (-3.18d, 0.00p, 0.06std)
+0.04%, 0.01ms invariant (0.11d, 0.83p, 0.08std)

WGLMakie
–
–
–
–
–

master
–
–
–
–
–

evaluation
–
–
–
–
–


[bug] resolve empty object change reject

Describe the bug
This code change reject at iOS
invoke.resolve([:])
invoke.resolve()

Browser error: failed to deserialize response: invalid type: map, expected unit
Reproduction
No response
Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.4.1 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.80.1 (3f5fd8dd4 2024-08-06)
    ✔ cargo: 1.80.1 (376290515 2024-07-16)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (default)
    - node: 22.6.0
    - pnpm: 8.9.0
    - yarn: 1.17.3
    - npm: 10.8.2

[-] Packages
    - tauri [RUST]: 2.0.0-rc.3
    - tauri-build [RUST]: 2.0.0-rc.3
    - wry [RUST]: 0.42.0
    - tao [RUST]: 0.29.0
    - tauri-cli [RUST]: 2.0.0-alpha.7
    - @tauri-apps/api [NPM]: 2.0.0-rc.1
    - @tauri-apps/cli [NPM]: 2.0.0-rc.3

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - bundler: Rollup

[-] iOS
    - Developer Teams: ***

Stack trace
No response
Additional context
No response

Can you share the command call? I think this is just a mismatch between the type you're sending from the plugin and what you are expecting in the frontend.

Sorry! It seems they were replaced during my debugging.
It was resolve() that didn't work. I will investigate more closely and report back.


[REACT] Research Page Responsiveness

Expected should be similar to one hosted in master branch

I wan to work on this under JWOC


Team alias seems like don't works for hill objectives scoreboard.

Video: http://youtu.be/Nm6K--Etijs
Version: 778d5d5
There's no console log.

Same with TDM


Unbearable slowness in Featurizer.get_feature_matrices

Description of the bug
featurizer.get_feature_matrices(train_cands) does not return within a bearable amount of time.
To Reproduce
Steps to reproduce the behavior:

Deploy the fonduer-tutorials using docker
Upgrade fonduer to 2a49291003c2826835b0b900425d7663f0ecc202 (introduced by #407)
Run the hardware tutorial up to 2.2 Multimodal Featurization

Expected behavior
featurizer.get_feature_matrices(train_cands) returns about 1 min or so for hardware tutorial.
Error Logs/Screenshots
When I forcefully cancel the operation, the following stack trace is shown.
KeyboardInterrupt                         Traceback (most recent call last)
<timed exec> in <module>

~/.venv/lib/python3.7/site-packages/fonduer/features/featurizer.py in get_feature_matrices(self, cand_lists)
    303             features.
    304         """
--> 305         return get_sparse_matrix(self.session, FeatureKey, cand_lists)
    306
    307

~/.venv/lib/python3.7/site-packages/fonduer/utils/utils_udf.py in get_sparse_matrix(session, key_table, cand_lists, key)
    157             else:
    158                 annotations.append({"keys": [], "values": []})
--> 159         result.append(_convert_mappings_to_matrix(annotations, key_names))
    160     return result
    161

~/.venv/lib/python3.7/site-packages/fonduer/utils/utils_udf.py in _convert_mappings_to_matrix(mappings, keys)
    186         if mapping:
    187             for key, value in zip(mapping["keys"], mapping["values"]):
--> 188                 if key in keys:
    189                     indices.append(keys_map[key])
    190                     data.append(value)

Environment (please complete the following information)

OS: debian:buster
PostgreSQL Version: 12.1
Poppler Utils Version: N/A
Fonduer Version: 2a49291003c2826835b0b900425d7663f0ecc202

Additional context
I think this is a regression caused by #407.
Also this issue is not noticeable unless the feature vector is big (like 30000 dimensions).

https://github.com/HiromuHota/fonduer-tutorials/runs/883626478 demonstrates the hardware tutorial stalls at featurizer.get_feature_matrices(train_cands) (right after featurizer.apply(split=0, train=True, parallelism=PARALLEL)) and timeouts (360 minutes).

Demonstrated by https://github.com/HiromuHota/fonduer-tutorials/runs/883723629, the proposed fix #484 can reduce the elapsed time to about 1 min.
Fri, 17 Jul 2020 23:21:41 GMT CPU times: user 49.5 s, sys: 3.21 s, total: 52.8 s
Fri, 17 Jul 2020 23:21:41 GMT Wall time: 1min 2s
Fri, 17 Jul 2020 23:21:46 GMT (28935, 27735)


fix issues reported by the linter

just curious if you want to make it on-going check as part of pipeline as well ? or at least to have make target ?

Sure! I can do that in another PR and possibly include more lint fixes, if that's okay :)

Codecov Report

Merging #560 into master will increase coverage by 0.38%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #560      +/-   ##
==========================================
+ Coverage   52.72%   53.11%   +0.38%
==========================================
  Files          63       63
  Lines        4922     4886      -36
==========================================
  Hits         2595     2595
+ Misses       1924     1888      -36
  Partials      403      403

Impacted Files
Coverage Δ

pkg/loadtester/concord.go
0.00% <0.00%> (ø)

pkg/loadtester/runner.go
86.48% <ø> (ø)

pkg/router/smi.go
65.95% <ø> (+18.26%)
:arrow_up:

pkg/controller/scheduler.go
47.21% <66.66%> (ø)

pkg/notifier/teams.go
78.57% <100.00%> (ø)

Continue to review full report at Codecov.

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

@stefanprodan done


realm: dynamic load of gasnetex (stage 1)

This is a tracker issue for:
https://gitlab.com/StanfordLegion/legion/-/merge_requests/1229
So that I can make sure it gets into the June release.
Current status: needs testing on Frontier to make sure everything is working.

Merged via https://gitlab.com/StanfordLegion/legion/-/merge_requests/1427


[curl][c-ares] can't link to curl built with c-ares on Apple platforms

It looks like c-ares uses libresolv, but dependency on it is not propagated in installed CMake config files. As a result, when trying to build app using libcurl it fails to link due to missing -lresolv flag
Host Environment

OS: OSX, but I suspect iOS as well

To Reproduce
install curl with c-ares feature with static linkage, try to use that curl in your app
Failure logs
Undefined symbols for architecture arm64:
  "_res_9_getservers", referenced from:
      _ares_init_options in libcares.a(ares_init.c.o)
  "_res_9_ninit", referenced from:
      _ares_init_options in libcares.a(ares_init.c.o)
  "_res_9_ndestroy", referenced from:
      _ares_init_options in libcares.a(ares_init.c.o)
ld: symbol(s) not found for architecture arm64

Additional context
I am using Ninja generator to build my app on OSX

Can you please provide a sample project to reproduce this?
Thanks.

We don't have enough information to solve this issue, unfortunately; if you have any more information to help us solve this, please reopen!

If anyone ever gets here, this had the answer: https://github.com/c-ares/c-ares/issues/675#issuecomment-1871159204
Just link with resolv e.g. -lresolv on Mac

Just link with resolv e.g. -lresolv on Mac

That's what the orginal post suggests.
But this is already carried in CMake config and in pkg-config.


Extended validation, added CreatedDate

Hi Gatblau,
We have added some validation for user name and also populated created date in the UserData model. VSCode has also updated (annoyingly) the mod file(s) so that the lines are in a different order lol! Hope that is ok?
Thanks for looking at our PR.
Steve.

Not accepted


Only work when app is in background ?

Hey,
I follow all instructions, and seems to work when my app is in background.
But when app is close, the value seems to be null..no idea why..
Do you have this problem too ?
Nelson,

use https://pub.dev/packages/receive_intent/example
but I am not able to use this package like receive_sharing_intent
if you font solution then please tell me
thank you


Service is going up and down

Nodes in both installs are constantly connecting and disconnecting from the service

You can only have one install.


Convert British English to US English before 1.0 release

Lots of functions and APIs within substrate use British English rather than US English. For example on_finalise vs on_finalize.
Should probably migrate to use US English since it is pretty standard for programming, and we should conform.

how is this going @shawntabrizi ?

@joepetrowski wanted to take a first stab and I will do a second pass to make sure nothing gets missed.
I will also create an item on the docs side to do a full scrub.

Is this docs? The APIs are the important things here.

@gavofyork this issue is for the Substrate codebase.
The other issue is for the docs.

https://www.npmjs.com/package/american-british-english-translator
Could be somewhat useful


[FLINK-6436] [table] Bug in CommonCorrelate's generateCollector method when using udtf with udf

Thanks for the fix @godfreyhe
+1 to merge


[WIP] Status Bar Scrolling

Fixes #673

So far, I have fixed the navbar dissapearing issue. I did notice we have some other issues here though.
Some of our content is positioned absolutely for various reasons, and therefore have scrolling state apart from the rest of the app. So it's going to take little more work to get these working:

[ ] Discover
[ ] Devotionals


Any library similar to this in android????

Hello. Currently we don't have similar lib in android, but we'll implement it, when we are able to.


[Notion] Separate page to edit page property

Description
Close #12751

Edit Page Properties action now pushes to separate view.
All database pages in search view show the Edit Page Properties action.

I definitely need someone to help me with testing this. There are a lot of changes, and I want to be sure everything works before we roll it out.
Screencast
https://github.com/user-attachments/assets/ef65b420-2d42-4ec8-97f8-0f39557f412e
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I'll need to revisit how this extension works to properly review it and help you test it. I'll probably do that within the next week.

I like the idea of creating a single PageForm.tsx. I initially wanted to do that, but it would have been super complicated due to how page properties are handled.
As I mentioned in my last PR, the way property components really need to be reworked. I've been dreading it, but it's a constant roadblock.
I'm going to put this PR on hold and get to work on handling the page property components. I'll open a draft soon.

Sounds good, I've marked it as draft. Let me know when it's ready to review again 🙂


Fix MiscFile system creating new project for each file

Looks like https://github.com/OmniSharp/omnisharp-roslyn/pull/1252 had an intent to create one project for misc files per language, but bug caused it to create new project for each file, causing a skyrocket of memory consumption in larger projects. This should fix the issue

Good catch! Thanks @SirIntruder!


Added eslint rules to prevent tailing whitespaces, tabs, and incorrect indentation.

You can run yarn lint --fix to automatically fix most errors (except no-tabs, you can fix these errors by opening the file in vim and using :retab assuming your indentation is set up correctly).
How do y'all feel about adding more lint rules? I added some rules to let us get away with things like "any" and "Function". Unused vars are left as a warning, and this commit made no effort to fix them. If we decide that we like these lint rules, I'll go ahead and fix more of the unused vars warnings.
I mostly did this because I noticed that I've been using tabs instead of spaces because I messed up my vimrc on my home linux machine (oops!). This should help to keep the codebase a little cleaner (automagically).
There's apparently ways to set up your text editor to automatically run yarn lint --fix on save, which could be useful.

Are you using an IDE or are you using vim to program? There is a package in the project (should be installed in your node_modules folder) called "prettier" that you can also run that fix spacing. You can do it through the command line, or if you are using an IDE, you can set it up to automatically run on file save. Linting is great (and fine with me if we want to implement in builds), but prettier will solve the spacing issues!

Are you using an IDE or are you using vim to program? There is a package in the project (should be installed in your node_modules folder) called "prettier" that you can also run that fix spacing. You can do it through the command line, or if you are using an IDE, you can set it up to automatically run on file save. Linting is great (and fine with me if we want to implement in builds), but prettier will solve the spacing issues!

I also want to throw out there that prettier and eslint can sort of compete with each other, so you have to be careful on implementation. It's not super difficult, but you have to take extra steps to be sure the correct rules are setup. I would say I prefer prettier for spacing issues and eslint for more semantic rules.

Didn't realize we were already using prettier 😅 I was looking into that as well but decided it was too strict. Removing PR.

I think some standard style is a great idea! I haphazardly added and eslintrc and prettier extension to keep things clean, then when Hannah started contributing my prettier extension kept muddying up PRs, so I set it to only run on modifications I make to PRs. I'd love it if we all used the same linter and prettier! Just not sure the best way to set it up.

I think some standard style is a great idea! I haphazardly added and eslintrc and prettier extension to keep things clean, then when Hannah started contributing my prettier extension kept muddying up PRs, so I set it to only run on modifications I make to PRs. I'd love it if we all used the same linter and prettier! Just not sure the best way to set it up.

I suspect we are different prettier versions. I think you had globally installed prettier, and I ended up adding a local installation to the project. I know what you mean about it muddying up PR's, but I think enforcing the same version by using a project-specific copy will fix that (at least I hope).
Airbnb has a published set of eslint rules they maintain (though it has changed over time) that I've used before. There are a lot of rules in there, haha. But whatever we want to do, I'm good with! I am used to having a linter bark at me all the time, so I think I just program that way by default now 😅
I don't have a particularly strong opinion on styling, as long as it doesn't take too much away from actual development time (which is why I love the prettier on save workflow).

Agree.  I do like the idea of having built into the lint rather than relying on people running prettier separately. Causing the build to fail due to poorly formatted code would prevent merging PRs with poor formatting. I am dubious of using prettier in the linter due to its sctrictness and potential conflict issues, but am still open to it. I think we could add enough typescript lint rules to get pretty close to what prettier does.

From my understanding, Prettier is not very configurable because the coding style it uses is already opinionated. Adding our own lint rules (possibly based on the rules by something like AirBnB) would let us employ our style in a more opaque manner.

Agree. I do like the idea of having built into the lint rather than relying on people running prettier separately. Causing the build to fail due to poorly formatted code would prevent merging PRs with poor formatting. I am dubious of using prettier in the linter due to its sctrictness and potential conflict issues, but am still open to it. I think we could add enough typescript lint rules to get pretty close to what prettier does.

I actually find it the opposite. Eslint (at least with airbnb rules applied) is far stricter than prettier is, and as you mentioned fails the build. To me, that's an argument for using it on semantic rules vs. spacing rules. And I have not found that eslint is capable of auto-fixing many of the rules that it complains about (it just complains - of course, maybe I'm using it wrong!), so I much prefer the auto-reformatting of prettier. Prettier doesn't/shouldn't do any updates to the code that would cause the code to be changed in significant ways, just indentation and such, which I consider to be less important than semantic issues that eslint would catch.

Causing the build to fail due to poorly formatted code (such as my tabs instead of spaces 😅) would prevent merging PRs with poor formatting

I think either package (or both packages) could be run automatically by setting them up in package.json. We just have to set the "build" script (or possibly even the "dev" script) to use these packages. Then they would just run automatically without worrying about whether or not individual devs ran them or not.

I actually find it the opposite. Eslint (at least with airbnb rules applied) is far stricter than prettier is, and as you mentioned fails the build. To me, that's an argument for using it on semantic rules vs. spacing rules.

I suppose what I really meant "inflexible" rather than strict. Like with our own eslint rules we can make it as strict as we want, whereas the prettier in some cases cannot be configured to match the style that is already being used most places in the the project.
And as far as semantic rules goes, so far the only significant semantic rule that is employed by this minimal eslint config is that it changes var to let in every case (good thing IMO), and changes let to const when the variable doesn't get modified. Those seem pretty fair to me.

I suppose what I really meant is "inflexible" rather than strict. Like with our own eslint rules we can make it as strict as we want, whereas the prettier in some cases cannot be configured to match the style that is already being used most places in the the project.

I hear what you're saying. I think that's also where I meant that "I don't have a strong opinion on style". I think most places in the project are currently using prettier (although, admittedly, I think different versions or rulesets, which I do think we should figure out).

And as far as semantic rules goes, so far the only significant semantic rule that is employed by this minimal eslint config is that it changes var to let in every case (good thing IMO), and changes let to const when the variable doesn't get modified. Those seem pretty fair to me.

Yeah! I'm not opposed to using a linter. I'm used to using one! There are many, many rules you can apply semantically to a linter, and many of them are very useful. I am definitely in favor of linting for semantics.

Looks like most "style" eslint rules are deprecated and replaced by esline-stylistic https://eslint.style/

I'm going to table this for now and just use prettier locally

I really like the idea of adding more code quality checks to the linter. The eslintrc isn't doing much for us as is. What about adding the rules from this PR and maybe adding eslint-config-prettier to avoid conflicting styles and we can all run the same version of prettier locally to get the style fixes on save? Deal with code formatting with prettier and code quality with eslint?

I really like the idea of adding more code quality checks to the linter. The eslintrc isn't doing much for us as is. What about adding the rules from this PR and maybe adding eslint-config-prettier to avoid conflicting styles and we can all run the same version of prettier locally to get the style fixes on save? Deal with code formatting with prettier and code quality with eslint?

This is my vote!

Here's an example of the airbnb rules. There are a ton, and I'm not necessarily saying we go with this package, but it will give you an idea of the kinds of options we have.
https://github.com/airbnb/javascript/tree/master/packages/eslint-config-airbnb-base/rules

This is my vote, minus these rules:
"no-trailing-spaces": "error",
    "@typescript-eslint/indent": ["error", 2],
    "no-tabs": "error"

Those are style-specific and not semantic.
=== EDIT AGAIN
Sorry if that last bit came across a bit harsh. The prettier eslint rule will invalidate any style rules that it overwrites, so we can leave these in if you guys want!

You're not harsh at all! I think this combination of your ideas will clean the code up considerably. What do you think @j-huff? I agree that those lines can stay or go, no biggie.


Integration tests for extract-text-webpack-plugin

This pull requests is based on #14 and #15.
The actual change is https://github.com/jantimon/html-webpack-plugin/commit/04e775da020f586ded526189b666fb2e0e3e0ee1

Rebased down to 5 commits

Rebased

Thanks. Now this is merged I will do a bit of refactoring around the tests, they're getting a little bit unwieldy.


Slow queries on MongoDB using include for relations

Bug description
When you use {include: {relationModel: true}} on prisma with a MongoDB, you can observe several slow queries in the mongo logs.
Prisma generates a aggregate query which cannot use the index on _id. Instead mongo falls back to COLLSCAN. This is of course very slow.
How to reproduce
Prisma seems to do the same on include, like loading 500 documents manually like this
const ids = ["61b86bc0a5eb740008ea98c7", "61b9be7ea5eb740008eb22bc", ....]
(await prisma.production.findMany({where:{id:{in:ids}}})).length

This generates the following in mongo:
"msg":"Slow query","attr":{"type":"command","ns":"nucleus.Production","command":{"aggregate":"Production","pipeline":[{"$match":{"$expr":{"$and":[{"$in":["$_id",[{"$oid":"61b86bc0a5eb740008ea98c7"},{"$oid":"61b9be7ea5eb740008eb22bc"},
...
}]]}]}}}]},"planSummary":"COLLSCAN","cursorid":4032979547306890542,"keysExamined":0,"docsExamined":670117,"numYields":670,"nreturned":101,"queryHash":"75F140EB","planCacheKey":"0C6217C0","reslen":358655,"locks":{"Global":{"acquireCount":{"r":671}},"Mutex":{"acquireCount":{"r":1}}},"readConcern":{"level":"local","provenance":"implicitDefault"},"writeConcern":{"w":"majority","wtimeout":0,"provenance":"implicitDefault"},"storage":{},"remote":"<IP_ADDRESS>:45590","protocol":"op_msg","durationMillis":2009},"truncated":{"command":{"pipeline":{"0":{"$match":{"$expr":{"$and":{"0":{"$in":{"1":{"282":{"type":"objectId","size":12}}}}}}}}}}},"size":{"command":10555}}

Expected behavior
When writing this command to mongo directly, it uses the index and is very fast:
 db.getCollection('Production').find({_id: {$in: ids.map(id => ObjectId(id))}}).explain()

Prisma information

Environment & setup

Database: MongoDB 5.x

and a collection containing > 1 million documents.
Prisma Version
"@prisma/client": "^4.2.1",
"prisma": "^4.2.1",

this is a duplicate of https://github.com/prisma/prisma/issues/14871 and https://github.com/prisma/prisma/issues/12793

The issue stems from the generate prisma query is too complex to use an index. There is a hack instead of "id_in":
prisma.production.findMany({
  where: { OR: ids.map(id => ({ id: id })) }
});

But I have found many other prisma queries to have performance issues, so I have unfortunately had to migrate away from prisma.
Hope this helps

I made similar discoveries here: https://github.com/prisma/prisma/issues/17142#issuecomment-1648791852

The issue stems from the generate prisma query is too complex to use an index. There is a hack instead of "id_in":
prisma.production.findMany({
  where: { OR: ids.map(id => ({ id: id })) }
});

But I have found many other prisma queries to have performance issues, so I have unfortunately had to migrate away from prisma. Hope this helps

What tool have you migrated to recently? Is it Mongoose?

This performance problem is fixed in Prisma 5.14.0


The results are impressive, but problems exist.

Hi, thanks for your excellent work! I have got some impressive results with your method. However, I find that the results are not always satisfactory when using some prompts.  For example, "a camouflage off-road vehicle" (the window is not correctly generated):
https://user-images.githubusercontent.com/20635237/217236165-171484f3-6d23-4714-bb49-3592b29b9d10.mp4
And I failed to reproduce "A desktop Apple iMac" on the project page:
https://user-images.githubusercontent.com/20635237/217237236-19dbb96c-5917-45f1-9c27-23c1a8245c5e.mp4
Compared to the result on the project page, my result shows significant artifacts. Besides, the result of "An iPhone 11 pro max" failed to distinguish the front and back views:
https://user-images.githubusercontent.com/20635237/217238041-4daa9a7f-1ed1-417d-88e1-5bf22e5e5493.mp4
Do you have any suggestions for alleviating the artifacts and improving the multi-view consistency? What parameters can I tune for this goal? I believe this work can facilitate a wide range of real applications.

Generally this is because of UV Mapping problems. Xatlas is decent but it's not perfect. Need to figure out how to AI UV Unwrap, that is a much more tougher problem to solve.


Question: Tab with dynamic data

All of the documentation examples illustrate static content.  In these situations, the resizing works perfectly..
However, I would like the ability to dynamically load data for a tab and then have the resize kickoff for the content.  My scenario is roughly as follows:
 <md-tabs>
     <md-tab id="Inventory" md-label="Inventory">
         <inventory-tab :items="currentInventory"></inventory-tab>

In this example, on render the custom inventory-tab component is instantiated and the property is bound.  Once user selection kicks off filters, an ajax call would populate this.currentInventory, pushing the data to the custom component.
However, on initial render, the resize doesn't kick in.  However, after tabbing away and returning, the tab content properly resizes.
I would like the content to resize when the data is loaded.
Could you please provide an example of how this can be done?

I have also noticed this behavior, I was populating data using apollo-graphql, and you have to come back to the tab before it displays for the first time. I was thinking it was graphql issue

@mygnu thanks for the additional confirmation.  In my case it is a simple $.ajax call, so the retrieval method doesn't seem to be related.
Also, manually triggering the event via: window.dispatchEvent( new Event('resize') ); seems to have no impact.

After a lot of painful and unsuccessful experimentation, the best solution/hack I found is to add a ref to the tab you want to watch.  This allows you to expose the updateTabData method for that tab.
// ajax call.then(() =>
this.$refs.tab1.updateTabData(this.$refs.tab1.$data);

This feels pretty hacky and it would be nice if there was a more formal API for this.  However, it does properly resize the tab area with animations, which is great.
On a somewhat related note, I also had quite a bit of trouble loading data for a tab on tab change, which I suspect would go hand in hand with people needing to resize based on dynamic data.
Thanks to ref, you can also override the click handler to reload your data.  This isn't optimal either, but it does do the trick.  Once again, it would be wonderful to have an API hook that enables actions on click.

Oops! The md-tabs will be improved on the very next minor version. There is some performance issues, some animation problems that will be fixed too. Sorry to make you wait a little bit.

No need to apologize!  You've done fantastic work and I appreciate it, thank you!  I just wanted to make certain I wasn't missing something and if not, bring it to your attention. Looking forward to the next minor version.

No. This is on Vue Material side. The md-tabs is a problematic component. Some concepts that were developed in others components will be used to make the tabs better. :)

Fair enough :)  Looking forward to the improvements. If I can help, please let me know.


Expand Example for Event Hubs

The example just shows passing in a parameter, for example, eventHubAuthorizationRuleId, but does not show or provide detail on what that rule ID actually is.
More comprehensive example would be nice here.

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

ID: b8f43189-3418-fd30-0528-550157d8366d
Version Independent ID: 049e3b0e-6f10-6985-c719-05fe43fc00b8
Content: Create diagnostic setting in Azure using Resource Manager template - Azure Monitor
Content Source: articles/azure-monitor/platform/diagnostic-settings-template.md
Service: azure-monitor
Sub-service: ****
GitHub Login: @bwren
Microsoft Alias: bwren

@CtrlDot Thank you for your query, we will look into it and get back to you at the earliest.

@CtrlDot Apologies for delayed response on this.
eventHubAuthorizationRuleId is the Resource ID of the event hub authorization rule for the Event Hubs namespace in which the event hub should be created or streamed to. Below is an example of
eventHubAuthorizationRuleId: /subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxxx/resourceGroups/Analytics-Resources/providers/Microsoft.EventHub/namespaces/AnalyticsEHNamespace/eventhubs/analytics/authorizationRules/myauthorule11
Diagnostic Settings - Create Or Update which has been linked in the document has more details on this, please have a look.

One of the possible ways you can create and get the EventHub Authorization Rule ID.

Will also share your feedback to our Document Author to further evaluate and see the existing document needs any enhancement.
Hope the above information helps. We will go ahead and close this issue, however if you have any further queries please tag us in your reply and will be happy to reopen the thread and continue the conversation. Thank you


Rename AsNamed to AsTemplateParam

And also its trait methods is_named and as_named into is_template_param
and as_template_param respectively.
These new names better reflect what the trait is about.
r? @emilio

:pushpin: Commit 88ea656 has been approved by emilio

@bors-servo r+

:hourglass: Testing commit 88ea65696b6850460405357836ea6ab4a12a5f3f with merge 521b7d11b0a814b9533a83e981f7a5d7fac63fbf...

:sunny: Test successful - status-travis
Approved by: emilio
Pushing 521b7d11b0a814b9533a83e981f7a5d7fac63fbf to master...


fix/NR-Button-Animation-12704: fixed Button animation and footer text…

**Jira Ticket:**https://issues.newrelic.com/browse/NR-12704
Description: Updated animation for View repo and Build your own buttons and in the footer section changed text hover changes.
Screen shots:
Button animation:

Footer hover text color changes:

as Gatsby build is not successful in this PR, I am closing this PR and have raised another PR: https://github.com/newrelic/instant-observability-website/pull/222