can't load package: package github.com/castawaylabs/cachet-monitor

Ok total GO Noob here.
I'm on upstream debian jessie and install golang.
Issuing go install github.com/castawaylabs/cachet-monitor returns:
can't load package: package github.com/castawaylabs/cachet-monitor: cannot find package "github.com/castawaylabs/cachet-monitor" in any of:
        /usr/lib/go/src/pkg/github.com/castawaylabs/cachet-monitor (from $GOROOT)
        ($GOPATH not set)

I've cloned the repo:

run it inside it, same effect.
run it outside it, pointing to it's path, same effect.
moved the repo to /usr/lib/go/src/pkg/github.com/castawaylabs/cachet-monitor, different error but yet error

What's the gist about this?

The issue as the log posted says, $GOPATH not set.
Seems like you havent done anything with go yet, so:
mkdir /go
export GOPATH=/go
go install github.com/castawaylabs/cachet-monitor
# now installed in /go/bin/cachet-monitor

Thanks now I get
# github.com/castawaylabs/cachet-monitor
/usr/lib/go/src/github.com/castawaylabs/cachet-monitor/main.go:24: syntax error: unexpected range, expecting {
/usr/lib/go/src/github.com/castawaylabs/cachet-monitor/main.go:28: syntax error: unexpected }

At least a step in the right direction haha

Replacing line 24 of main.go with for _ = range ticker.C { like in https://github.com/CastawayLabs/cachet-monitor/commit/d421b35e9be1900399026fe1f16e3e482d085615#diff-7ddfb3e035b42cd70649cc33393fe32c solved it.
Is this a bug? Is my Go outdated?

It might be a bug. Which go are you running?

Sorry for late answer, the mail got under a flood of other GH issues hehe
Here my version:
# go version
go version go1.3.3 linux/amd64

Right I think you need to either compile the binary on a system where you can have go1.4 or 1.5 or install the relevant version on your machine.
Cross-compiling is very easy (at least on 1.5) and it could be possibly provided in the Releases section in the future.
go1.3 might not support the shorthand syntax which you had to fix manually (amongst other things!)

Fair enough :)
Not sure how Go's release cycles are, but perhaps put that in the readme if I'm not on a terribly outdated version (debian is slow ;)

yeah its debian :P. But the thing is, you dont even need Go installed for this to work on your linux...

Oh thats decent, I'll get a newer version on my gentoo laptop and push it up to the servers and see if it goes any better :)
Anyways thanks a lot for the chat, I'll close this invalid issue ;)

no probs


chore: implement Clone & Copy common traits

Implement Clone & Copy common traits for most structs in src.
Only implement Copy for structs that are simple and trivial to copy.
Reorder the derive fields to be more consistent:
Debug, Default, Clone, Copy, Eq, PartialEq, Ord, PartialOrd, Hash

see: https://github.com/ratatui-org/ratatui/issues/307

Thanks again for the great effort on this!


Ionic 2: Slide menu slide in gesture doesn't disable page scrolling underneath

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
Slide in the menu from the edge, scrolling in page underneath still possible before you lift your finger off the screen

Expected behavior:
Slide menu opening should immediately disable scrolling on page underneath.
Steps to reproduce:

Make a long enough page to require scrolling.
Add a side menu.
Slide the menu in partially and try to scroll up or down at the same time.
Observe page underneath scrolling when it shouldn't.

Other information:

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Davids-MacBook-Air:apphazard-app daveshirman$ ionic info

Your system information:

Cordova CLI: 6.2.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.46
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: OS X El Capitan
Node Version: v4.3.0
Xcode version: Xcode 8.1 Build version 8B62

Hi @daveshirman
sorry for out of topic, can you please tell me what do you use to make gif

Download AndroidTool for Mac. Don't know if it's available for Windows.
Super useful.
On Dec 14, 2016 12:58, "Tushar"<EMAIL_ADDRESS>wrote:

Hi @daveshirman https://github.com/daveshirman
sorry for out of topic, can you please tell me what do you use to make gif
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/driftyco/ionic/issues/9633#issuecomment-267027373,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANnlXkk9h2meFQlyupe9TRk_7E-U4CKvks5rH-f6gaJpZM4LM16b
.

@daveshirman Could you try using the nightly version of the framework and let me know if you still have this issue? I couldn't reproduce it using nightly. To install nightly:
npm install --save ionic-angular@nightly

This will also require an update to Angular 2.2.1, see: https://github.com/driftyco/ionic-conference-app/blob/master/package.json#L13
Thanks :)

@daveshirman Hey! We released that latest nightly brandy mentions above as RC4 today so if you update to RC4 you will get the same fixes. I also cannot reproduce this issue with RC4.

Hey, same problem!
I clone the "ionic-conference-app" project today, and everything works fine, but when i start a "blank" or a "sidemenu" project, the problem remains.
Here is some of my packages info, did i got anything wrong?
 "dependencies": {
    "@angular/common": "2.2.1",
    "@angular/compiler": "2.2.1",
    "@angular/compiler-cli": "2.2.1",
    "@angular/core": "2.2.1",
    "@angular/forms": "2.2.1",
    "@angular/http": "2.2.1",
    "@angular/platform-browser": "2.2.1",
    "@angular/platform-browser-dynamic": "2.2.1",
    "@angular/platform-server": "2.2.1",
    "@ionic/storage": "1.1.6",
    "ionic-angular": "2.0.0-rc.4",
    "ionic-native": "2.2.11",
    "ionicons": "3.0.0",
    "rxjs": "5.0.0-beta.12",
    "zone.js": "0.6.26"
  },
  "devDependencies": {
    "@ionic/app-scripts": "0.0.47",
    "typescript": "2.0.9"
  }

Your system information:

Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.2 Build version 8C38

@jgw96 This is still an issue on 2.0.0 final. Can we get an update please?

Hi, everyone, i think the problem has been located. Comparing to your "ionic-conference-app" which works fine,  project created by "ionic start 'projectName' --v2" use default webView engine which i think is outdated for IOS 9+(may be),  while "ionic-conference-app" use new webView engine "cordova-plugin-wkwebview-engine" (which i'm not sure is ready for public use)
So, use new webView engine, lots of bugs in iOS could be solved.
@daveshirman please compare "config.xml" file in your project file to that in "ionic-conference-app" project for details. Change your "config.xml", then Re-Add iOS platform could make your app work.
By the way, @brandyscarney @jgw96 is "cordova-plugin-wkwebview-engine" ready for commercial use?

@adamdbradley The team has been super silent with many of these outstanding issues and they need addressing. Any updates please?

Hello all! Sorry for the delay on this one. So their is kind of a long story behind this issue. So UiWebView which is the default webview for cordova-ios at the moment has broken scroll events. It also has some interesting flickering issues when changing certain css properties. Because of these two issues we are unable to disable scrolling on UiWebView while opening or closing a sidemenu without causing a massive flicker and we made the decision that being able to scroll while swiping on UIWebView was better than having a huge flicker. Apple has since released a new webview (WkWebView) and while it still has the same flickering issue (although scroll events work great now) we were able to do a fix on the cordova side of things in our fork of the wkwebview plugin that fixes this issue without causing a flicker. For a while we were not ready to push the wkwebview plugin because it had some weird issues but we are not at a state with it that we are ready to start pushing it for production use and in fact we are planning on making it the default iOS webview in the next Ionic release. Because of all this I am going to close this issue now and recommend that https://github.com/driftyco/cordova-plugin-wkwebview-engine be used as the fix. Thanks everyone and again, very sorry for the delay on this.

Hey. Just an update. I installed the wkwebview plugin, and while the functionality is improved, it still allows scrolling while initially opening the menu. Once scroll has occurred, it locks as expected, but this is still a problem.

@lincolnthree The link in Justin's comment was outdated. I updated it, could you try the new instructions here and let me know if you are still having the issue? https://ionicframework.com/docs/wkwebview/#upgrading-to-wkwebview-uiwebview-users-only
Thanks :)

Hey @brandyscarney - I've done all that, and these are the issues I'm seeing:
https://www.youtube.com/watch?v=K_vBD6IJdpE
List of issues visible in this video, with time references:
0:03 -  If the menu is opened while a page is loading (by setting the rootPage attribute in app.component.xml), then opening the menu (via this.menuCtrl.open()), the menu appears first, then the page slides out overtop of it. I can probably work around this last issue via some creative timing (or possibly using NavController.setRoot(), but it seems a bit odd.
0:05 -  While the menu is open, the content view can be scrolled, then once the scroll has completed (0:11), it locks down and can no longer be scrolled.
0:12 - Some odd behavior with swipe gesture. The menu doesn't initially follow the swipe (presumably because the user was swiping up, first). It then, however, recognizes the swipe after the user has released the screen, and snaps closed.
0:20 - The content view can be scrolled, again. Once scrolled, it stops being scrollable (as it should be from the start (0:22))
WKWebView is definitely an improvement for this behavior, since prior to upgrading, the content view was always scrollable, and now it's just sometimes scrollable, but still not quite right.
Any thoughts? Thanks.


MP: formation of translation initiation complex

The way this is defined
Joining of the large subunit, with release of IF2/eIF2 and IF3/eIF3. This leaves the functional ribosome at the AUG, with the methionyl/formyl-methionyl-tRNA positioned at the P site.
it could be housed under GO:0002183 cytoplasmic translational initiation
(also some of the siblings)
Reported by: ValWood
Original Ticket: geneontology/ontology-requests/11121

Following  up on this,  so you don't need to go through the thread, all this requires is to make
GO:0001731 part_of  GO:0002183
v

GO:0001731 part_of GO:0002183

OK.  Have done this.
Still feels like it should have something in the comment at least to make it clear why it belongs here and not on general term.  Closing anyway, but will add if you suggest something.
Ta,
david

Well the def is describing the cytoplasmic TIC assembly
so why not just make:
014-09-24	Added	SYNONYM	cytoplasmic translation initiation complex assembly (exact)
the primary term name. No confusion then....


Inline snapshot drift to the right

💥 Regression Report
Inline snapshots are re-indented whenever another inline snapshot in the same file is saved/updated.
Last working version
Worked up to version: 24.5
Stopped working in version: 24.6
To Reproduce
Steps to reproduce the behavior:
Given this test file and we're running in watch mode:
describe('broken', () => {
  test('1', () => {
    expect(11111).toMatchInlineSnapshot();
  });

  test('2', () => {
    expect([[1]]).toMatchInlineSnapshot();
  });
});

Upon save, both snapshots are written:
describe('broken', () => {
  test('1', () => {
    expect(11111).toMatchInlineSnapshot(`11111`);
  });

  test('2', () => {
    expect([[1]]).toMatchInlineSnapshot(`
      Array [
        Array [
          1,
        ],
      ]
    `);
  });
});

Now I update the first test (expect(10000).toMatchInlineSnapshot()) and save. The snapshots get rewritten:
describe('broken', () => {
  test('1', () => {
    expect(10000).toMatchInlineSnapshot(`10000`);
  });

  test('2', () => {
    expect([[1]]).toMatchInlineSnapshot(`
            Array [
              Array [
                1,
              ],
            ]
        `);
  });
});

Now I add another test (expect(3).toMatchInlineSnapshot()) and save:
describe('broken', () => {
  test('1', () => {
    expect(10000).toMatchInlineSnapshot(`10000`);
  });

  test('2', () => {
    expect([[1]]).toMatchInlineSnapshot(`
                  Array [
                    Array [
                      1,
                    ],
                  ]
            `);
  });

  test('3', () => {
    expect(3).toMatchInlineSnapshot(`3`);
  });
});

Notice how the snapshot for test 2 slowly drifts to the right. This can get super annoying after many test-update-save cycles.
Expected behavior
Inline snapshots should stay put.
Link to repl or repo (highly encouraged)
Repl.it only has Jest 22.1.2. I didn't find a way to upgrade it.
Run npx envinfo --preset jest
Paste the results here:
  System:
    OS: macOS Mojave 10.14.4
    CPU: (4) x64 Intel(R) Core(TM) i7-7567U CPU @ 3.50GHz
  Binaries:
    Node: 10.15.0 - ~/.nvm/versions/node/v10.15.0/bin/node
    Yarn: 1.15.2 - /usr/local/bin/yarn
    npm: 6.4.1 - ~/.nvm/versions/node/v10.15.0/bin/npm
  npmPackages:
    jest: ^24.6.0 => 24.7.1

Thanks for the report. I'll take a look at it very soon. Do you know if it happens outside of watch mode?

Yep, just tested and confirmed same thing happens with jest -u.

I recommend removing the snapshot and regenerating it - that should stop the diff. Still a bug, of course 🙂

@SimenB that's what I ended up doing.

I have a video

Is there a way to tell yarn generate everything from scratch?

FYI I see this happen with<EMAIL_ADDRESS>as well<EMAIL_ADDRESS>also has the same bug. I'm not sure that it's a recent regression.

Not related to VSCode, I don't use it but still see this happen frequently using latest Jest. Didn't have time to investigate yet though :/

May be related to this? https://github.com/facebook/jest/pull/8198

Yeah definitely

@petternordhold,
Until your pr get merged, you can downgrade jest-snapshots using yarn resolutions to 24.5

@Bnaya jest-snapshots only got version
'1.0.0',
  '1.0.1',
  '1.0.2',
  '1.0.3',
  '1.1.0',
  '1.1.1',
  '1.1.2'

Any chance to see it released before jest 25?

@Bnaya yes, we haven't landed breaking changes yet, there will we another minor before 25

(Seems this fix was released in jest 24.0.18?)

24.9.0 has it 🙂


WebAuthn and multi factor support

Add two factor support to Pow with at least OTP and FIDO U2F support.
Good primer for ensuring secure and useful two factor setup: https://medium.com/@stuartschechter/before-you-turn-on-two-factor-authentication-27148cc5b9a1
Edit: Here's some security considerations: https://shahmeeramir.com/4-methods-to-bypass-two-factor-authentication-2b0075d9eb5f

MFA auth

WebAuthn
OTP/TOTP
Recovery codes
E-mail/magic link
Device authorization (in app)

Process
In general the MFA authentication process will go like this:
User ID & password --> MFA authorization --> Signed in

OR as passwordless:
User ID --> MFA authorization --> Signed in

This latter option could also be all on the same page for some of the above methods. The MFA methods can also be used for the registration process.
There should be a controller action for handling 2FA auth, and probably one for handling registration with 2FA. Maybe one for managing 2FA methods.
WebAuthn links
https://webauthn.guide/
https://auth0.com/blog/web-authentication-webauthn-overview-demo-tool/

Suggested Ecto schema
There're two ways to go about this. Embedded or separate table. Separate table may make it easier to not require user id, but it could be that the user id should just be set to generated_id if developers wish to login without user id with a randomly generated id that will be stored with the authenticator.
field :type, :string
field :nickname, :string
field :settings, :map

WebAuthn use case
WebAuthn allows for no user id and can be used for single authentication.
The WebAuthn will store at minimum the following data:
{
  credential_id: "credential_id",
  cose_key: "cose_key"
}

For no user id setup, the user id used for the authenticator could be a randomly generated id that's stored in the authenticator schema.
TOTP use case
TOTP can only be used for 2FA. The data stored would be the following:
{
  issuer: "My Service",
  otp_secret: "secret",
  last_otp_at: "492039"
}

A WebAuthn library that could be tested with: https://github.com/tanguilp/wax

WebAuthn standard has been finalized by W3C!
We should consider a pure single factor Webauthn registration/authentication process where user id and password is not required (or user id is the the webauthn credential id(s). On top of this there will also be two factor.
I do think it’s the future of web authentication, far more than provider login.

From the elixirforum, there's a question on multiple user id field authentication:

I’m just a bit curious. ^^
Is it possible (or could it be) to pass a list of fields to :user_id_field , so that user can authenticate through the field of their choice?
defmodule MyApp.Users.User do
  use Ecto.Schema
  use Pow.Ecto.Schema,
    user_id_field: [:email, :pseudo, :phone]

  # ...
end

This may fit with the user id less WebAuthn setup.

What's the feeling on using external libraries to make this go faster/smoother?
Specifically, elixir2fa and passwordless_auth?

I don't mind using a library to do the heavy lifting at all, but I don't plan to add any more dependencies to Pow so it would have to be an adapter setup, or an extension. No matter how it will be, I believe the biggest hurdle is to refactor the code so it's flexible enough to handle a custom auth flow (independent on what auth logic is used).

An extension is probably sanest, then. Refactoring is certainly quite the task.

Yeah. I've been too busy, but now with API integration guide and out-of-the-box distributed Mnesia released soon, I believe this is probably the next thing I'll take a dig at 😄

Sounds awesome! Looking forward to removing my custom-rolled 2FA stuff in favor of the official implementation. (Fewer things I can do wrong if I'm not the one doing them. :joy: )

For sure. If you can share the code, I would be happy to see how you solved it (maybe gist or repo)!

Still implementing, but an extension that hooks into before_respond/4 is my current plan.

Any updates on how this is going? Would love to see MFA in Pow!


#1680 ValidateHash method Alg is null when token is an jwe token

#1680 ValidateHash method Alg is null when token is an jwe token

@jxnkwlp it's odd that the token is a JWE.
Where is this being called from?

/Azp run

/azp run

/AzurePipelines run

/azp run Wilson-6x-PR

/azp run

/Azp run

@jxnkwlp this fell off the radar, would like to pull it in but we need to have it rebased against dev.

Okay, working on it

@jxnkwlp thanks.


Fix Spot Price: Quote Asset and Base Asset

Closes: #XXX
What is the purpose of the change
This PR fixes the spot price API to return correct spot price results, whilst previously it was returning reversed spot prices for the base asset and quote assets.
This API fix has impact on the tx fees: this PR also fixes the logic within the txfees module to use the updated Spot Price API
Alot of the changes within this PR are test case fixes: reversing the base asset and the quote assets used in the testing params.
Brief Changelog

Fix Spot Price APIs so that they return correct spot price

Testing and Verifying
Fixed existing spot price related test cases.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? yes
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? yes
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Replacing this PR with #3253


Error: KeyError: 'graph_encoder.layers.0.encoder.lin_src.weight'

Dear authors,
Thank you for your great work on Fairseq. When I tried to run the below command, I got this error. Hopefully I would received some advices from you.
🐛 Bug

I got the bug KeyError: 'graph_encoder.layers.0.encoder.lin_src.weight' when trying to load checkpoint_utils.load_model_ensemble from fairseq.
To Reproduce

from fairseq import options, tasks, checkpoint_utils, progress_bar, utils
parser = options.get_generation_parser()
args = options.parse_args_and_arch(parser)
task = tasks.setup_task(args)
models, _ = checkpoint_utils.load_model_ensemble(["models/generation_models/tplgen_template_generation/model.pt"], task=task)

where the models folder it is from https://drive.google.com/drive/folders/1EpfamTiOosKvy_s9Bm_tW1NdtAzKpkY7 and I alreadly used the model so I think it works fine.
Expected behavior
models, _ = checkpoint_utils.load_model_ensemble([args.path], task=task)
  File "/content/gdrive/.shortcut-targets-by-id/1euwk1Zyt7Sa5sxm4_AVBZ9m4ECrtcIo7/A*/open-ended_question_ontology/fairseq/fairseq/checkpoint_utils.py", line 368, in load_model_ensemble
    state,
  File "/content/gdrive/.shortcut-targets-by-id/1euwk1Zyt7Sa5sxm4_AVBZ9m4ECrtcIo7/A*/open-ended_question_ontology/fairseq/fairseq/checkpoint_utils.py", line 471, in load_model_ensemble_and_task
    state["model"], strict=strict, model_cfg=cfg.model
  File "/content/gdrive/.shortcut-targets-by-id/1euwk1Zyt7Sa5sxm4_AVBZ9m4ECrtcIo7/A*/open-ended_question_ontology/fairseq/fairseq/models/fairseq_model.py", line 128, in load_state_dict
    return super().load_state_dict(new_state_dict, strict)
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1468, in load_state_dict
    load(self)
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1466, in load
    load(child, prefix + name + '.')
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1466, in load
    load(child, prefix + name + '.')
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1466, in load
    load(child, prefix + name + '.')
  [Previous line repeated 2 more times]
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1463, in load
    state_dict, prefix, local_metadata, True, missing_keys, unexpected_keys, error_msgs)
  File "/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py", line 1372, in _load_from_state_dict
    hook(state_dict, prefix, local_metadata, strict, missing_keys, unexpected_keys, error_msgs)
  File "/usr/local/lib/python3.7/dist-packages/torch_geometric/nn/dense/linear.py", line 131, in _lazy_load_hook
    weight = state_dict[prefix + 'weight']
KeyError: 'graph_encoder.layers.0.encoder.lin_src.weight'

Environment

PyG version (torch_geometric.__version__): 2.0.3
PyTorch version: (torch.__version__): 1.10.0
OS (e.g., Linux): Colab, Yes
Python version (e.g., 3.9): 3.7.12
CUDA/cuDNN version: 11.1
How you installed PyTorch and PyG (conda, pip, source): I install by pip from the instruction here: https://pytorch.org/
How do you install fairseq?

git clone https://github.com/pytorch/fairseq.git
cd fairseq
python setup.py build develop
pip install --editable ./
python setup.py build_ext --inplace

Any other relevant information (e.g., version of torch-scatter): I install torch_geometric:

!pip install torch-scatter -f https://data.pyg.org/whl/torch-1.10.0+cu111.html
!pip install torch-sparse -f https://data.pyg.org/whl/torch-1.10.0+cu111.html
!pip install torch-geometric

Additional context

just encountered a similar issue, did you figure this out perhaps?


[Suggestion] Steam Spike/Grinder

I'm thinking like a rotary drill that turns and grinds up mobs that come into its hitbox. It can be a 1 wide x 2 long x 1 high block (placed in any orientation, and accepts steam from the box half, the other half is the drill). It can be configured to accept steam at different rates, and depending on the rate of steam input, it can spin faster and deal more damage.

The multiblock API we have isn't really designed for multiblocks in that style, its meant for freeform ones like the boiler. I could hijack it to do something like that, but it wouldn't be ideal.
As for the concept itself, there's the steam vent already implemented which does something similar.


Add jupyter logging so that users can see what jupyter commands we run

For #9086
We decided in triage we needed this before the release. This pipes stdout and stderr from the server to an output window for VS code.

Don did the same thing. HA! I like his better. He also added time out tracking

Codecov Report

Merging #9092 into master will decrease coverage by 0.34%.
The diff coverage is 58.62%.

@@            Coverage Diff             @@
##           master    #9092      +/-   ##
==========================================
- Coverage   57.12%   56.78%   -0.35%
==========================================
  Files         551      552       +1
  Lines       29100    29119      +19
  Branches     4395     4390       -5
==========================================
- Hits        16623    16534      -89
- Misses      11570    11677     +107
- Partials      907      908       +1

Impacted Files
Coverage Δ

src/datascience-ui/react-common/postOffice.ts
3.63% <ø> (ø)
:arrow_up:

src/client/datascience/serviceRegistry.ts
0% <0%> (ø)
:arrow_up:

src/client/common/utils/localize.ts
94.64% <100%> (+0.01%)
:arrow_up:

src/client/datascience/types.ts
100% <100%> (ø)
:arrow_up:

...lient/datascience/jupyter/jupyterOutputListener.ts
40% <40%> (ø)

...rc/client/datascience/jupyter/jupyterConnection.ts
72.17% <80%> (+0.62%)
:arrow_up:

src/client/datascience/jupyter/notebookStarter.ts
72.13% <83.33%> (+0.46%)
:arrow_up:

src/client/testing/serviceRegistry.ts
49.09% <0%> (-47.28%)
:arrow_down:

src/client/testing/codeLenses/main.ts
40% <0%> (-30%)
:arrow_down:

src/datascience-ui/interactive-common/tokenizer.ts
35% <0%> (-30%)
:arrow_down:

... and 22 more

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 56e2488...99d9c2f. Read the comment docs.


feat(linter): implement eslint rule no-return-await

Eslint test cases are more than 100, and clippy warning this, @Boshen can we change this config to allow more test cases code?

Eslint test cases are more than 100, and clippy warning this, @Boshen shall we change this config to allow more test cases code?

Nice work! Thank you!


Remove active empty ENV parameters and add details to comment

Signed-off-by: Cristian Prundeanu<EMAIL_ADDRESS>

I'm confused here. According to the docs, --env overrides ENV, but this comment makes it seem like it's the other way around.

It is the other way around from empirical observation. Having ENV entries makes the corresponding --env parameters have no effect.

I'm confused here. According to the docs, --env overrides ENV, but this comment makes it seem like it's the other way around.

It is the other way around from empirical observation. Having ENV entries makes the corresponding --env parameters have no effect.

ok.  I feel we need to get to the bottom of this before merging this change, given this is explicitly contradicted by the docker docs.    Or maybe create an issue in the docker repo and link it here?  Maybe there's already an open issue.

It is the other way around from empirical observation. Having ENV entries makes the corresponding --env parameters have no effect.

Hrmmm, I haven't seen this.

ok. I feel we need to get to the bottom of this before merging this change, given this is explicitly contradicted by the docker docs.

Agreed

Don't know what's happening. Now when I check environment inside the container, it is correctly taken from docker's --env parameters, overriding ENV in the Dockerfile.

This is still an issue in some configurations, commenting the ENV lines correctly passes the command line variables. Reopening PR.


Update helper node tag

Update helper node tag to get the code for setting proper permissions to named zone  files

Also undo https://github.com/ocp-power-automation/ocp4-upi-powervm/pull/93

/assign @Prajyot-Parab

/assign @Prajyot-Parab

This is already fixed.


[docs] Review Templates page for technical accuracy

Review the content at https://lit-element.polymer-project.org/guide/templates for technical accuracy only.
Markdown source: https://github.com/Polymer/lit-element/blob/master/docs/_guide/templates.md

In Compose a template from other templates section, the code snippet doesn't seem to work. I think you want the getters without static keyword.

Example code under Use named slots don't look right

"Use name, not id, to select slots."
That's seems more like a warning than actual documentation, since id on <slot> doesn't do anything special. Maybe use .alert-warning for gotcha things?


No example of using hardware path

From the current examples, there is no detailed guide and code on how to use the hardware path.

Hello @optimistyzy, thank you for raising an issue. It seems for me that Job API examples are completely unusable in the current state, they definitely should be reworked.
As for High Level API examples, the only change that is required to run hardware path is change the line:
using execution_path = dml::software;

to:
using execution_path = dml::hardware;

In any case, I added the issue to bug tracker.

Hello @optimistyzy, thank you for raising an issue. It seems for me that Job API examples are completely unusable in the current state, they definitely should be reworked.
As for High Level API examples, the only change that is required to run hardware path is change the line:
using execution_path = dml::software;

to:
using execution_path = dml::hardware;

In any case, I added the issue to bug tracker.
Hi @EgorKupaev

Thanks. From my usage experience,
(1)It still needs some make file or cmake change if we use the hardware path.
(2) Configuring wqs by idxd-config.
So better to add this to make it clear.

There is a part of REAMDE.md which describes how to build the library with hardware path enabled. That extra step is likely to be removed in future releases.
Regarding configuration, the library relies on idxd-config documentation, which is linked in reference manual.

There is a part of REAMDE.md which describes how to build the library with hardware path enabled. That extra step is likely to be removed in future releases.
Regarding configuration, the library relies on idxd-config documentation, which is linked in reference manual.

@EgorKupaev I know this. What  I means that we at least need a example to use hw path with a very simple description.

Hi @gaowayne, just wanted to let you know that we're working on improving this area for DML.
We've extended Quick Start to provide some instruction on how to change current examples, so that they could be offloaded to accelerator, and additionally would be providing an option to specify execution path as input parameter for examples executables.
Hope this would make in clearer.


Non-Checkbox DockableMenuItem

DockableMenuItem extends JCheckBoxMenuItem and setting an icon on this menu item doesn't work properly. Modern Docking should support another option to display a normal JMenuItem that can have an icon applied to it.

Looks like this is all look and feel based. I was assuming the weird highlighting was a bug, but it's actually a configurable option for FlatLaf to tell when a JCheckBoxMenuItem is selected.


Fix sync tests to what is expected

What does this Pull Request accomplish?
Some of the sync system tests have been failing for a while. Specifically the ones that check for errors. This is probably because they require real hardware and are not ran as part of the CI.
I updated the tests to work as expected with the current implementation. It appears that #692 did some changes where the error and message is reported through the grpc status instead of the response status. Updated the tests work with those changes.
Why should this Pull Request be merged?
Fixes sync tests so we can rely on their output to identify issues with any changes.
What testing has been done?
built and ran tests locally.

xnet test ConvertFramesToFromSignalsFromExample_CompareFrames_ValuesMatch fails on Ubuntu repeatedly. Does not appear to be caused by this change. @reckenro @astarche

xnet test ConvertFramesToFromSignalsFromExample_CompareFrames_ValuesMatch fails on Ubuntu repeatedly

Yeah, I can't see your change affecting this. Didn't see a Bug/Issue either so created one: https://github.com/ni/grpc-device/issues/1058


Not Working

Installed Python and Executed the code with correct ip address and live device id. It failed. (I tried over LAN and it failed)
Enter the IP address: 192.168.1.XX
Enter the Live ID: <16 digit alpha-numeric id>
Sending power on packets to 192.168.1.XX...
Xbox should turn on now, pinging to make sure...
Failed to ping Xbox :(
Do you wish to keep trying? (y/n): y
Sending power packets and pinging until Xbox is on...
Failed to ping Xbox :(
Failed to ping Xbox :(
Failed to ping Xbox :(
Failed to ping Xbox :(
Failed to ping Xbox :(
Failed to ping Xbox :(
Background Settings of my Xbox One S:
Number 1> my console is on Energy saving mode (it has to be, else i won't be searching on internet and run this script if i set to instant on mode)
Number 2> my console is connected to router/internet via LAN cable. Its a wired connection And not on Wi-Fi. (I need to do this to good stream over LAN)
Number 3> my console is allocated static ip in my LAN. So ip address is not an issue.
Number 4> my console NAT type is open.
So, I kindly request you to please tell me what to do to get this script work. Also tell me the limitations of this script and also at the same time add the limitations to the project description on this github website as well so that no one will waste time like me do.

SayHaiprashant - this used to work; years ago...seems like this year it broke...possibly due to a microsoft update of xbox?  Mine was working for years, but this year seems to have stopped working.

Any sign of a fix?


Fix deprecation warnings on node 10.12

node commit: https://github.com/nodejs/node/commit/46c7d0d21ff13db9af731fc77ee56ead0336ab43
Fixes https://github.com/nodejs/nan/issues/810

I started out trying to make this based on V8_*_VERSION but only the embedder string changed and the previous node 10 builds didn't have the APIs needed to support the post-deprecation code.

@addaleax Is there a better way to detect support for the new APIs?

I started out trying to make this based on V8_*_VERSION but only the embedder string changed and the previous node 10 builds didn't have the APIs needed to support the post-deprecation code.

That seems a bit fishy. Was there some #define changed in Node which caused existing deprecations to become visible?

Afaict it was introduced in the node commit linked above. It's a patch on top of V8 floated by node if I'm reading it correctly.

Ah, that explains it. I must have missed the commit link.

the others should all work on V8 6.9 already?

Totally possible. I got lazy once I finally found a working guard for the StringObject case and just blindly reused it.

If I understand this change correct it turns compilation warnings into incompatibility.
E.g. a native add-on using NAN compiled with node 10.12.0 would not work with 10.11.0 as the v8 APIs used are not available there. The other way around should work.
If an add-on is using v8 directly it doesn't face this issue.
I know the target of NAN is not to avoid such problems but I think the price to get rid of warnings is high here.
To my understanding the node change was done to give add-on owners more time to adapt for node 11 and NAN is already prepared for this.
Maybe just silence the warnings within the NAN files?

E.g. a native add-on using NAN compiled with node 10.12.0 would not work with 10.11.0 as the v8 APIs used are not available there. The other way around should work.

That’s correct, yes; but only for the StringObject construction case. I doubt there is more than a handful of people who have ever used it in addons.

But then the ifdefs should be adapted to select the new APIs for all node 10 versions.

For I'll go with something like this:
      "conditions": [
        [
          '"<!(echo $V)" != "1"',
          {
            "cflags": [
              "-Wno-deprecated-declarations",
            ],
            "xcode_settings": {
              "OTHER_CFLAGS": [
                "-Wno-deprecated-declarations",
              ],
            },
          },
        ],
      ],

Not really windows compatible but allows seeing deprecations in verbose mode while not leaking them into npm install of downstream consumers. 👍

Unless there's a way to fix this without breaking compatibility - should this PR be closed?

For windows comapt add
            'msvs_disabled_warnings': [4996],

like:
      "conditions": [
        [
          '"<!(echo $V)" != "1"',
          {
            "cflags": [
              "-Wno-deprecated-declarations",
            ],
            "xcode_settings": {
              "OTHER_CFLAGS": [
                "-Wno-deprecated-declarations",
              ],
            },
            'msvs_disabled_warnings': [4996],
          },
        ],
      ],

Any update on this? It seems like a lot of modules are fixing this themselves by adding -Wno-deprecated-declarations to their binding.gyp but a fix in NAN itself would obviously be a lot better!


Outputted videos are "very slightly" shorter than input videos

This issue is similar to #23 but I believe a different bug is causing this.
Outputted videos are shorter by usually only a few seconds. If I rip the audio from the source video and give it to the new video, the beginning of the video is synced up with the audio but the video slowly gets more de-synced with the audio until it's end. This issue is more apparent with longer videos. This is due to frames being dropped very rarely.

This is purely speculation but I wonder how RIFE is handling scene changes. If RIFE were to not generate an interpolated frame when scenes change, but also not add an additional frame between scene changes (as is needed to keep time synced), then RIFE would effectively drop one frame per every scene change.

I guess this is related to the previous memory leak, scene changes should be handled well.

This issue has existed since before the #26 memory leak was introduced. I will update and test again soon.
Can you think of a way to identify and view the frames which were dropped? That might help us diagnose what is causing this issue.

If you can upload your video to google drive, I'll find the missing frame.

I think I find it. It's caused by the wrong frame rate. The frame rate should be a float number. I shouldn't round it.

My latest run using an input video with 23.976 FPS shows significant improvement. I don't notice the audio getting more de-synced over time. However the video does seem to be offset by a small amount of time (less than a 10th of a second) with the audio. That offset, while initially difficult to perceive, does make things feel a little weird when watching people speak.
Also, assuming that our calculation for how many frames should be in the output video is ( (startingFrames * frameMultiplier) - 1), my output video is missing 2 (or 0.00093%) of frames.
frameMultiplier being 2 to the power of the value given to  --exp.
I will do more testing soon, and I will share a test video if I am able.

Which code did you run? inference_video.py?

Yes python3 inference_video.py --exp=2 --video=video.mp4 .

Now the input and output are in separate threads, and both support the format of PNG folder. Is it possible to reconfirm this issue and investigate the cause?

@hzwer The issue is still not fixed. You're welcome to test videos yourself by opening two video players, one containing the original and the other the interpolated video. Start them at the same time, and pause and play either one until the audio is in sync. Then see if one of the videos has scene changes happen before the other.
For some of the interpolated videos that don't have sound added, videoplayers will also state that the interpolated videos have one second less of video playtime.

Yes python3 inference_video.py --exp=2 --video=video.mp4
edit:
and I am checking the frame count of videos using ffmpeg -i video.mp4 -map 0:v:0 -c copy -f null - and looking at the frame= property at the bottom of what is logged.

can confirm that I have the similar issue with a video. However, my issue is the opposite of yours. My video has an unrounded fps of 29.97, but RIFE reads it as 29.970028956549715 FPS, causing the output to have EXTRA frames. This offsets the audio and makes the audio out of sync in the opposite way and became quite noticeable after only 20 seconds of video. I do not understand why it seems to read the FPS as more than it actually is, but this causes it to have more frames and the output video is 1 second longer than the initial file. The issue is not fixed.


there is trojan

XD

That is a false positive. The skin changer as well as the injector uses some Win32 API's that aren't commonly used in normal programs. The project is open-source, so you are welcome to compile the skin changer on your own, otherwise disable your antivirus and add the folder with the binaries to the exclusion list.
I'm tired of closing issues like this one is like a 10 issue relating to this issue this week. Please check closed issues before posting an another one.

That is a false positive. The skin changer as well as the injector uses some Win32 API's that aren't commonly used in normal programs. The project is open-source, so you are welcome to compile the skin changer on your own, otherwise disable your antivirus and add the folder with the binaries to the exclusion list.
I'm tired of closing issues like this one is like a 10 issue relating to this issue this week. Please check closed issues before posting an another one.

hi i just wanna say this. i tried downloading  skin changer  last week but the windows defender did not keep the files and tried to disable defender but it still deletes it bc it detectes it i tried many solution and totorials to disable windows defender and bc it deletes the files i was downloading it again and again. but one time when i redownload the defender did not delete the files but it still detecte torjan,i tried deleting it from defender but it cant delete ,after that my pc strated slowing down .so i had to do a full clean up to my pc ,do u have a reason for this?


Haste-0.6: Documentation/clarification required: Module top-level items.

I lost half a night of sleep over the following problem, until I gave up and ran my exam (I know I am daring, but Haste is just too good!) with less functionality than hoped for: In my Main module, I have a big record that contains, among other things, a number of remote $ static (import_ $ ... calls, either directly as fields, or slightly deeper down. I refactored one of these into top-level functions, as I had done previously in other cases successfully, one for the part outside the remote, and one for the remote, and then got run-time errors: With some arrangements:
CalcCheckWeb: too few arguments to remote function

, but mostly
CalcCheckWeb: user error (Pattern match failure in do expression at src/Haste/App/Remote.hs:51:5-12)

I played with this a bit more today: I am only refactoring, and not changing anything, and still get that effect: It works with everything inside the record field, but not after refactoring.
As I have previously observed mess-up when I had #ifdef-ed top-level items present only for GHC, but not for HASTE, I now conjecture that GHC is inlining away my factored-out functions, and producing that same kind of mess-up.
I have not investigated enough to be certain that my explanation is right, but I am seeing  “superfluous” top-level definitions all over the Haste/Haste-App source, so there must be something in this direction.
In my opinion, this deserves being documented prominently...

This does indeed sound like GHC is inlining away the functions, but that shouldn't happen with functions that have a static pointer pointing to them. I think this should be fixed rather than documented.
Does putting NOINLINE pragmas on the affected functions make the problem go away? Do you have a minimal example which triggers this that you're able to share?


Update Dart sdk and packages versions

Upgrade Flutter version
Update the Dart SDK and the versions of outdated packages.
Motivation
With the current version of Dart 2.13.0 we aren't able to update the version of Flutter to the latest: 2.10.3. To know more about the improvements in this version, see: Flutter 2.10.0 improvements.

https://github.com/ardriveapp/ardrive-web/pull/351

@jdaev , thanks, I didn't see this PR. Closing this one.

@thiagocarvalhodev
Nah. Let's leave it open as a draft. If you wanna do this please create a ticket for this task and we can proceed with this and PR. What's with the changes to the application name?

What's with the changes to the application name?

Please see Upgrading pre 1.12 Android projects.

Closing as we already have a PR doing it #626 .


Evaluate and choose a JSON parsing tool

Need to choose a tool that can:

create a fresh JSON-LD object
read JSON-LD
validate JSON-LD
query JSON-LD (including querying on values)
insert into existing JSON-LD object
be implemented within Galaxy

There's been a change of approach from when we wrote this original story - we're expecting a lot more of the data type now and not explicitly including the JQ JSON parser in workflows like the dummy POC.
To discuss - what functionality is inherent in the OpenDS data type?

Create a fresh JSON-LD object
Don't expect the data type to be able to do this

Read JSON-LD
The data type should be able to do this?

Validate JSON-LD
Should this be functionality of the data type? What level of validation - certainly if we don't have valid JSON the data type will fail

Query JSON-LD (including querying on values)
The data type should be able to do this?

Insert into existing JSON-LD object
The data type should be able to do this?

A lot of these requirements have been debunked - not using JSON-LD in the SDO, so closing this as existing json libraries appear to do the part


Respect user autostart settings.

Read QUnit.config.autostart after test-helper and friends have had a chance
to set the value.  Load the tests in either case, but only start them
automatically if QUnit.config.autostart has not been set to false.
This is particularly useful for users who want to debug tests in PhantomJS.

LGTM

I'll release shortly..

awesome thanks @rwjblue

Published as 0.3.11


UML-947 - Create Certificates

Purpose
Make it safer to create account-level resources in dev
Secure communications between users and our service.
Fixes UML-947
Approach
Only show the plan of account-level resources in dev during the pr-build workflow, and apply on path-to-live only.
Create a new set of certificates for the public-facing URLs.
Learning
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage remained the same at 81.604% when pulling 24dc74f8f092fe26caed88cfd380e0626695b686 on UML-947 into 099f064345c3a169ada465afae323adcdade1437 on master.


Cross domain uploads - json response is empty

Hello, I implemented this plugin with Ruby on Rails 3.1, nginx, nginx upload module and everything works great! :) But when I want upload by cross domain, uploading is done, but json response is empty.
Of course in my nginx server I set in config:
if ($request_method = 'OPTIONS') {
    more_set_headers 'Access-Control-Allow-Origin: http://mydomain.com';
    more_set_headers 'Access-Control-Allow-Methods: POST, OPTIONS';
    more_set_headers 'Access-Control-Allow-Credentials: true';

    return 200;
}

And set:
xhrFields: {
    withCredentials: true
}

I use browser Firefox 8.0
Thanks!

You need to return the "Access-Control-Allow"-Headers for both OPTIONS and POST requests.

Thank you for quick reply! Now I have in nginx configuration:
if ($request_method = 'OPTIONS') {
more_set_headers 'Access-Control-Allow-Origin: http://mydomain.com';
more_set_headers 'Access-Control-Allow-Methods: POST, OPTIONS';
more_set_headers 'Access-Control-Allow-Credentials: true';
return 200;

}
if ($request_method = 'POST') {
more_set_headers 'Access-Control-Allow-Origin: http://mydomain.com';
more_set_headers 'Access-Control-Allow-Methods: POST, OPTIONS';
more_set_headers 'Access-Control-Allow-Credentials: true';
return 200;

}
and it's always the same. In firebug is response still empty, but plugin now response "Empty file upload result". Can somehow find out where I have a bug?
OPTIONS REQUEST:
Host	myuploadserver.com
User-Agent	Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0.1) Gecko/20100101 Firefox/9.0.1
Accept	text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
Accept-Language	cs,en-us;q=0.7,en;q=0.3
Accept-Encoding	gzip, deflate
Accept-Charset	windows-1250,utf-8;q=0.7,*;q=0.7
Connection	keep-alive
Origin	http://mywebserver.com
Access-Control-Request-Me...	POST
Pragma	no-cache
Cache-Control	no-cache
OPTIONS RESPONSE:
Server	nginx/1.0.11
Date	Fri, 06 Jan 2012 18:50:24 GMT
Content-Type	application/octet-stream
Content-Length	0
Connection	keep-alive
Access-Control-Allow-Orig...	http://mywebserver.com
Access-Control-Allow-Cred...	true
Access-Control-Allow-Meth...	POST, OPTIONS
POST REQUEST:
Host	uploadserver.com
User-Agent	Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0.1) Gecko/20100101 Firefox/9.0.1
Accept	application/json, text/javascript, /; q=0.01
Accept-Language	cs,en-us;q=0.7,en;q=0.3
Accept-Encoding	gzip, deflate
Accept-Charset	windows-1250,utf-8;q=0.7,*;q=0.7
Connection	keep-alive
Referer	http://mywebserver/uploads/new
Content-Length	15451
Content-Type	multipart/form-data; boundary=---------------------------41184676334
Origin	http://mywebserver.com
Pragma	no-cache
Cache-Control	no-cache
POST RESPONSE:
Server	nginx/1.0.11
Date	Fri, 06 Jan 2012 18:50:24 GMT
Content-Type	application/octet-stream
Content-Length	0
Connection	keep-alive
Access-Control-Allow-Orig...	http://mywebserver.com
Access-Control-Allow-Cred...	true
Access-Control-Allow-Meth...	POST, OPTIONS

My goal is to have a webserver and upload a few servers with nginx upload module. Will it work? My webserver configuration is classic. And my upload server have this configuration:

if ($request_method = 'OPTIONS') {
    more_set_headers 'Access-Control-Allow-Origin: http://mydomain.com';
    more_set_headers 'Access-Control-Allow-Methods: POST, OPTIONS';
    more_set_headers 'Access-Control-Allow-Credentials: true';

    return 200;
}

if ($request_method = 'POST') {
    more_set_headers 'Access-Control-Allow-Origin: http://mydomain.com';
    more_set_headers 'Access-Control-Allow-Methods: POST, OPTIONS';
    more_set_headers 'Access-Control-Allow-Credentials: true';

}

Thanks for reply.

Hello, problem is solved. I move

        more_set_headers 'Access-Control-Allow-Origin: http://mywebserver.com';
        more_set_headers 'Access-Control-Allow-Credentials: true';

from

 if ($request_method = 'POST') {

to

  location @fast_upload_endpoint
    more_set_headers 'Access-Control-Allow-Origin: http://mywebserver.com';
    more_set_headers 'Access-Control-Allow-Credentials: true';
    proxy_pass http://<IP_ADDRESS>;
  }

and now everything is Ok. Probably nginx upload module always delete this headers.


Move code being used by kafka plugin from client

Changes

:gift: Moved shared code for kafka plugin from client

/kind enhancement

Fixes partly https://github.com/knative/client/issues/1591

Release Note

Docs

/test all

/approve
/lgtm


Syntax Error when executing command

Very cool tool! No rush to fix this ever, only creating this issue in case you wanted to.
Try: wallace https://carbon.now.sh

Thanks for reporting! It seems that there is a CSS Syntax Error in your CSS, causing the analyzer to break:
.container.jsx-1841802010 {
	position: relative;
	undefined;
}

The error reporting could be clearer though. Leaving this issue open to allocate some time for that. Error is reported by @projectwalace/css-analyzer and only displayed here.

Thank you!!


Unable to install Azure Databases into VS code

I have VS code 2019 installed on my machine, however when I click install from the VS marketplace it does not seem to do anything. I also have searched the extension within VS code and was unable to find it. Any help would be greatly appreciated!

@glennstratton you can download the extension via this link: https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-cosmosdb

@wwlorey I appreciate the quick response, however when I try and click install nothing happens. What should I do?

@glennstratton I would try reinstalling VS Code. Unfortunately there's not much we can do here since we don't control the install experience.
If you're still having problems after reinstalling, I would contact general VS Code support: https://visualstudio.microsoft.com/support/


Removed unused attributes.

Removed default values for "flowEvictionThreshold" and "systemSyncWindow".

Hi @martin-leclerc,
Is it a common practice to add Eclipse specific files to java projects? I'm not doing any Java so I'm not sure, but it seems a little bit weird.

You are right, the .project file should not be included in the merge.
Martin.
From: Corentin Henry<EMAIL_ADDRESS>Sent: August 31, 2016 4:56 PM
To: nuagenetworks/vspkgenerator
Cc: martin-leclerc; Mention
Subject: Re: [nuagenetworks/vspkgenerator] Removed unused attributes. (#6)
Hi @martin-leclerc https://github.com/martin-leclerc ,
Is it a common practice to add Eclipse specific files to java projects? I'm not doing any Java so I'm not sure, but it seems a little bit weird.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/nuagenetworks/vspkgenerator/pull/6#issuecomment-243898720 , or mute the thread https://github.com/notifications/unsubscribe-auth/APGZ0kVBwtf50AZ1MqMdo_9eBfCxGonPks5qleptgaJpZM4Jx_wO .  https://github.com/notifications/beacon/APGZ0rS5lBdD9qjh9W3rOKfvLydam17kks5qleptgaJpZM4Jx_wO.gif

PR should now be ok to merge. Thank you for your help!

thanks!


Attempted import error: 'UserManager' is not exported from 'oidc-client-ts'.

When using the react-oidc-context library in a project:
./node_modules/react-oidc-context/dist/react-oidc-context.mjs
Attempted import error: 'UserManager' is not exported from 'oidc-client-ts'.

UserManager is exported:
// package.json
var version = "2.0.0-beta.3";

// src/Version.ts
var Version = version;
export {
  AccessTokenEvents,
  CheckSessionIFrame,
  InMemoryWebStorage,
  Log,
  MetadataService,
  OidcClient,
  SessionMonitor,
  TokenRevocationClient,
  User,
  UserManager,
  UserManagerEvents,
  Version,
  WebStorageStateStore
};

when using "moduleResolution": "node"
oidc-client-ts.cjs

When using the react-oidc-context library in a project:
./node_modules/react-oidc-context/dist/react-oidc-context.mjs Attempted import error: 'UserManager' is not exported from 'oidc-client-ts'.

Have you installed oidc-client-ts ? (Which is a peerDependency)

Yes.
I am confused why it takes react-oidc-context.mjs, which is an esm module, where my project is using node resolution, should it not take react-oidc-context.cjs? ...

strange wise webpack (create react app) is mixing module(esm) and main(commonjs), why?

react-oidc-context (esm)
oidc-client-ts (commonjs)

The problem seems to be the the .mjs extension.
When i rename react-oidc-context.mjs to react-oidc-context.esm.js and adapt package.json accordingly it magically works.
https://webpack.js.org/guides/ecma-script-modules/
@kherock
I will change the esm name in this package + oidc-client-ts unless you have a better idea? This way webpack will autotetect the correct format...

I can't reproduce this, and the .mjs extension is important for native ESM support from Node. I was actually going to suggest moving to the package exports field ahead of the 2.0 release.
What version of Node are you running, and what does your webpack config look like?

We

We can change this back + make a fix. I was unable to find another solution yet.
We are using react create app (https://www.npmjs.com/package/react-scripts v4.0.3), which is using under the hood webpack v4.
Docu:
https://create-react-app.dev/docs/getting-started/

Ok, Webpack v4 is really the problem here. I think migrating to exports should solve this. Webpack 4 doesn't implement resolution for those and will just use the CJS bundle. Can you let me know if a package.json like the following fixes it?
  "main": "dist/oidc-client-ts.cjs",
  "types": "dist/oidc-client-ts.d.ts"
  "exports": {
    ".": {
      "import": "./dist/oidc-client-ts.mjs",
      "require": "./dist/oidc-client-ts.cjs"
    },
    "./package.json": "./package.json"
  }
  // ...

I think this is relevant:
https://github.com/reactioncommerce/reaction-component-library/issues/399#issuecomment-467860022

"exports": {
    ".": {
      "import": "./dist/react-oidc-context.mjs",
      "require": "./dist/react-oidc-context.cjs"
    },
    "./package.json": "./package.json"
  }

still leads to this error:
./node_modules/react-oidc-context/dist/react-oidc-context.mjs
Attempted import error: 'UserManager' is not exported from 'oidc-client-ts'.

  "main": "dist/react-oidc-context.cjs",
  "types": "dist/react-oidc-context.d.ts",
  "exports": {
    ".": {
      "import": "./dist/react-oidc-context.mjs",
      "require": "./dist/react-oidc-context.cjs"
    },
    "./package.json": "./package.json"
  }

leads to:
./src/index.tsx
Attempted import error: 'AuthProvider' is not exported from 'react-oidc-context'.

I am wondering if it would make sense to wait for react-scripts v5, where they have switched to webpack v5. Within their milestone there is only one issue left. On the other hand we could go now with webapck v4 support and later drop that by using mjs extenstion...


ArchUnit refactore code, fix tests and test imports

Situation
The ArchUnit tests are flaky, because they wok on the build level. Further some tests seem not to work properly.
Wanted
Working ArchUnit tests wihtout side effects.
Solution

We need to fix the resideInPackage (no working properly)
We need to fix the execution of the unit tests -> test classes are only available when tests were executed before executing the archunit tests:
write ArchUnitExtension annotation for Junit test,
test are only executed when sechub.archunit.buildfolder is set,
GH action: ./gradlew :sechub-archunit-test:test after build
add shell command to sdc.sh
We need to document if side effects still exists (e.g. with .because, when to use "sechub.archunit.ignoreFolders")
Ideas: archunit gradle plugin?

Here a good example how to test this:
An eclipse build does always compile all classes (also test classes etc.) and when we start the arch unit tests with eclipse we have (at least one) failing test(s) - e.g. CodingRulesTest complains about:
java.lang.AssertionError: Architecture Violation [Priority: MEDIUM] - Rule 'no classes should access @Deprecated members, because there should be a better alternative' was violated (3 times):
Method <com.mercedesbenz.sechub.domain.scan.product.sereco.SerecoReportProductExecutor.$SWITCH_TABLE$com$mercedesbenz$sechub$sharedkernel$ProductIdentifier()> gets field <com.mercedesbenz.sechub.sharedkernel.ProductIdentifier.NESSUS> in (SerecoReportProductExecutor.java:35)
Method <com.mercedesbenz.sechub.domain.scan.product.sereco.SerecoReportProductExecutor.$SWITCH_TABLE$com$mercedesbenz$sechub$sharedkernel$ProductIdentifier()> gets field <com.mercedesbenz.sechub.sharedkernel.ProductIdentifier.NETSPARKER> in (SerecoReportProductExecutor.java:35)
Method <com.mercedesbenz.sechub.storage.s3.AbstractAwsS3Storage$LogS3ProgressListener.$SWITCH_TABLE$com$amazonaws$event$ProgressEventType()> gets field <com.amazonaws.event.ProgressEventType.BYTE_TRANSFER_EVENT> in (AbstractAwsS3Storage.java:277)
  at com.tngtech.archunit.lang.ArchRule$Assertions.assertNoViolation(ArchRule.java:94)
  at com.tngtech.archunit.lang.ArchRule$Assertions.check(ArchRule.java:86)
  at com.tngtech.archunit.lang.ArchRule$Factory$SimpleArchRule.check(ArchRule.java:165)

Which is correct, because we marked the product identifiers "NESSUS" and "NETSPARKER" as deprecated and did not ignore those.
But the CI/CD build does NOT fail. Reason is - as mentioned already before in issue description - that at gradle builds not all classes are compiled before they are tested by arch unit.
We could use this test as a canary that the changes, which will be done inside this issue, work. When it works the test will fail in gradle as well. Aftwards we should simply remove the deprecation parts from the product identifiers (because they are still in use by Sereco) or we add a ignore for this situation.


Arnej/rework ann filter bm

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.
@geirst please review

@geirst PTAL

Overall looks very good, but you forgot to add "hnsw-like.h":
/source/eval/src/tests/ann/xp-hnswlike-nns.cpp:3:10: fatal error: hnsw-like.h: No such file or directory  #include "hnsw-like.h"           ^~~~~~~~~~~~~ compilation terminated.


Different outputs when running coverlet.console on windows powershell/macOS terminal

I'm running coverlet.console on macOS and no coverage is generated for my project which actually does cover the dll it aims to test.
IT011972:heartbeat dpaz$ coverlet test/Application.Tests/bin/Debug/netcoreapp2.2/Application.Tests.dll --target "dotnet" --targetargs "test test/Application.Tests/Application.Tests.csproj --no-build -l trx -r ./TestResults" -o "./TestResults/coverage/coverage" --format json
Test run for /Users/dpaz/Documents/projects/heartbeat/test/Application.Tests/bin/Debug/netcoreapp2.2/Application.Tests.dll(.NETCoreApp,Version=v2.2)
Microsoft (R) Test Execution Command Line Tool Version 15.9.0
Copyright (c) Microsoft Corporation.  All rights reserved.
Starting test execution, please wait...
Results File: /Users/dpaz/Documents/projects/heartbeat/test/Application.Tests/TestResults/_IT011972_2018-12-12_10_41_00.trx
Total tests: 1. Passed: 1. Failed: 0. Skipped: 0.
Test Run Successful.
Test execution time: 1.6534 Seconds
Calculating coverage result...
  Generating report './TestResults/coverage/coverage.json'
+--------+--------+--------+--------+
| Module | Line   | Branch | Method |
+--------+--------+--------+--------+
Total Line: 100%
Total Branch: 100%
Total Method: 100%

I tried to see if I got the same on windows powershell, but surprisingly, it does show coverage there for the exact same file and command
PS C:\Users\dpaz\Documents\projects\heartbeat> coverlet test/Application.Tests/bin/Debug/netcoreapp2.2/Application.Tests.dll --target "dotnet" --targetargs "test test/Application.Tests/Application.Tests.csproj --no-build -l trx -r ./TestResults" -o "./TestResults/coverage/coverage" --format json
Test run for C:\Users\dpaz\Documents\projects\heartbeat\test\Application.Tests\bin\Debug\netcoreapp2.2\Application.Tests.dll(.NETCoreApp,Version=v2.2)
Microsoft (R) Test Execution Command Line Tool Version 15.9.0
Copyright (c) Microsoft Corporation.  All rights reserved.

Starting test execution, please wait...
Results File: C:\Users\dpaz\Documents\projects\heartbeat\test\Application.Tests\TestResults\dpaz_DESKTOP-MQE7N20_2018-12-12_10_27_48.trx

Total tests: 1. Passed: 1. Failed: 0. Skipped: 0.
Test Run Successful.
Test execution time: 2.1279 Seconds

Calculating coverage result...
  Generating report '.\TestResults\coverage\coverage.json'

+---------------------------+--------+--------+--------+
| Module                    | Line   | Branch | Method |
+---------------------------+--------+--------+--------+
| Application               | 100%   | 0%     | 100%   |
+---------------------------+--------+--------+--------+

Total Line: 100%
Total Branch: 0%
Total Method: 100%

The content of the test and the application proj itself it's super simple:
test:
public class HeartbeatIntervalProviderTests
    {
        [Fact]
        public void Provide01()
        {
            var options = Options.Create(new HeartbeatIntervalConfig
            {
                IntervalInSeconds = 10
            });

            var provider = new HeartbeatIntervalProvider(options);

            Assert.Equal(10, provider.Provide());
        }
    }

project has 2 classes:
public class HeartbeatIntervalProvider : IHeartbeatIntervalProvider
    {
        private readonly HeartbeatIntervalConfig _config;

        public HeartbeatIntervalProvider(IOptions<HeartbeatIntervalConfig> config)
        {
            _config = config.Value;
        }

        public int Provide()
        {
            return _config.IntervalInSeconds;
        }
    }

public class HeartbeatIntervalConfig
    {
        [Range(1, 1000000)]
        public int IntervalInSeconds { get; set; }
    }

I have the same problems.
.NET Core SDK (reflecting any global.json):
Version:   2.2.101
Commit:    236713b0b7
Runtime Environment:
OS Name:     ubuntu
OS Version:  18.04
OS Platform: Linux
RID:         ubuntu.18.04-x64
Base Path:   /usr/share/dotnet/sdk/2.2.101/

Guys is this issue solved?

Close for stale, feel free to re-open if needed.
FYI this kind of issue could be related to know issue https://github.com/tonerdo/coverlet/blob/master/Documentation/KnowIssues.md#1-vstest-stops-process-execution-earlydotnet-test


Question on user config

Hi, I watched your youtube videos and have been checking out your dotfiles to try to understand better how nix works.
I was wondering how you activate your user configuration, In your tutorials you had a separate script for updating user/home-manager and system-wide configuration. Is that the same here? I only see an apply script in the repo.
thanks for your help.

Hi @jacksonludwig, yeah I do need to clean my dotfiles up a bit and make them more usable for other people.
I actually have a script called sys that I bundle all of my system maintenance stuff up into.
It is located here:
https://github.com/wiltaylor/dotfiles/blob/master/roles/core/scripts.nix

thanks for the info, would you maybe also be able to clarify what the shells section (here) does in your config?
I see that it uses a helper function to make a nix shell, but I'm not sure how you would run something like that without a shell.nix file.
thanks again!

@jacksonludwig Yeah that is something I am going to rework. I am trying to break all of my system config up to be workflow based so i can keep as little in the core as possible. Other things have come up in the last month and a bit though so haven't had a chance to look at it.
As for shells and the confusion of not using a shell.nix file. This is because the way flakes change things. In regular old nix you would run nix-shell and it would create a shell from the derivation in shell.nix. Where as with flakes you can do something similar with nix develop which will by default create a shell from the derivation found in devShell.${system} or you can pass in which output you want it to run as a shell. in my case nix develop .#shells.video for example.
I also put a helper function in the sys script i mentioned above to launch these easier.
I am however thinking of breaking these separate shells out into their own flakes (kinda like what i did with my neovim config) so it  is a little easier to follow. I am going to make some more videos on a lot of this too :).

sounds good, thanks for the help


Cannot Locate the methods findByUserIsCurrentUser() and findAllWithToOneRelationships()

I am using jhipster 8.1.0 and following the tutorial to this repo at https://www.youtube.com/watch?v=IfyjKCt6YHE&t=236s.
I did create manually the findByUserIsCurrentUser() as it's straight forward. However the findAllWithEagerRelationships() isn't using at all the findAllWithToOneRelationships().
Below is the generated code from BlogRepository.java
@Repository
public interface BlogRepository extends ReactiveCrudRepository<Blog, Long>, BlogRepositoryInternal {
    @Override
    Mono<Blog> findOneWithEagerRelationships(Long id);

    @Override
    Flux<Blog> findAllWithEagerRelationships();

    @Override
    Flux<Blog> findAllWithEagerRelationships(Pageable page);

    @Query("SELECT * FROM blog entity WHERE entity.user_id = :id")
    Flux<Blog> findByUser(Long id);

    //This was added by following the tutorial
    @Query("select blog from Blog blog where blog.user.login = ?#{authentication.name}")
    Flux<Blog> findByUserIsCurrentUser();
   // ...
}

Where to add the "#{authentication.name}" condition for the eager loading?

It looks like you're using Spring WebFlux instead of Spring MVC. The tutorial and video uses Spring MVC.

Oops thank you, I will switch to the spring MVC then.


Can I create multiple action groups in the same template?

I need two or 3 action groups and can I create them in a single template. If yes can you post something which is helpfull.
Thanks

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

ID: 3ef46ed0-e53c-c99e-ccfc-80f26f19820d
Version Independent ID: e0ee52b9-7f31-02e4-d10e-c5f71ee801fe
Content: Create action groups with Resource Manager templates
Content Source: articles/monitoring-and-diagnostics/monitoring-create-action-group-with-resource-manager-template.md
Service: azure-monitor
GitHub Login: @dkamstra
Microsoft Alias: dukek

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

Is there any update on this.
Also please post the parameter file example. And creating multiple action groups . I did try to follow the documentation but I saw the above examples which can be used to create a action group.

@akramwaseem Apologies on a delay. I am checking internally on this one and hope to have an update by end of week. Unfortunately, the primary owner of the action group feature is OOF so trying to find a backup to help with your question.

@akramwaseem - below is one sample for creating multiple action groups based on parameters. You can expand this to have more properties in the parameters.
agtemplate.json.txt

@vramanx thanks for stepping in and helping with this.
@akramwaseem  please review and let us know if you have any followup questions.

@vramanx -- thanks for the file. I will take a look and get back to you.

I fixed a small bug in the output section where I did not use the correct namespace for the resource
agarraytemplate.json.txt


Package: zlib:x64-windows-static

Host Environment

OS: Win10

To Reproduce
Steps to reproduce the behavior:
.\vcpkg install zlib:x64-windows-static
Failure logs
-Cut and past the appropriate build messages from the console output.
Downloading https://mirror.bit.edu.cn/msys2/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz... Failed
-Please attach any additional failure logs mentioned in the console output.
I visit the https://mirror.bit.edu.cn/msys2/mingw/i686/mingw-w64-i686-pkg-config-0.29.2-1-any.pkg.tar.xz via IE browser,  the file is not existed.
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.

The pkg-config in this mirror was update to 0.29.2-2:
mingw-w64-i686-pkg-config-0.29.2-2-any.pkg.tar.zst

yeah, it's update to 0.29.2-2, but when I process the command .\vcpkg install zlib:x64-windows-static, it  download the 0.29.2-1, how can I replace 0.29.2-1 to 0.29.2-2?

The pkg-config in this mirror was update to 0.29.2-2:
mingw-w64-i686-pkg-config-0.29.2-2-any.pkg.tar.zst

yeah, it's update to 0.29.2-2, but when I process the command .\vcpkg install zlib:x64-windows-static, it  download the 0.29.2-1, how can I replace 0.29.2-1 to 0.29.2-2?

@wangweiwei1188  Will fix later.

@wangweiwei1188  Will fix later.

It's ok now.

It's ok now.


Option to preserve comments in the output?

Hi there, my project will be JS going forward so I'm looking to do a one-off transpile and I'd rather not have to manually copy the comments across as part of the clean up. Is there a way to keep them in the output?
Thanks!

comments in docstrings will be preserved as doc attributes in the generated code if you use the -d flag.

Wow, that's clever...


Image import is not tested

We need to write CI for image import/export for the supported cloud providers

Hi, image import may not be available on all Ostacks. I get:
TASK [upload image kubenow-v020a1 if not present ( url= https://httpd.pharmb.io/kubenow-img/kubenow-v020a1.qcow2 )] ***
changed: [localhost] => {"changed": true, "cmd": "glance --os-image-api-version 1 image-create --copy-from \"https://httpd.pharmb.io/kubenow-img/kubenow-v020a1.qcow2\" --disk-format qcow2 --min-disk 20 --container-format bare --name kubenow-v020a1 --progress | grep -w id | awk '{print $4}'", "delta": "0:00:01.598477", "end": "2017-04-29 20:51:12.423866", "rc": 0, "start": "2017-04-29 20:51:10.825389", "stderr": "400 Bad Request: External sources are not supported: 'https://httpd.pharmb.io/kubenow-img/kubenow-v020a1.qcow2' (HTTP 400)", "stdout": "", "stdout_lines": [], "warnings": []}

Hi @sneumann. Do you mean that you would like the CI to import the image automatically in your OpenStack installation?

Fixed in #158

Hi, sorry for the late response, my issue is that on an OStack without
support for remote image locations there is the response 400 Bad Request: External sources are not supported but the ansible snippet will happily retry until it bails out.
It should be smart and give a reasonable error message for thus 400 Bad request.
Yours, Steffen

I see! I just merged a major refactory #158 that also fixed some problems with the OpenStack import. Would you mind to test it?

Is there a point if the os does not support pointing to an http location
And insists on a real file upload?
Yours Steffen

I blame Android for the brevity and typos

@sneumann Yes, there is a point, the new playbook downloads locally first and then uploads to OS. In the new glance API they removed the http/https image location when uploading images. I don't understand why they've done this. I was doing some research on Google, but I didn't really find any explanation for that. We'll do more research and come back to you.


Projects: Sorting not working properly

Sorting not working properly for startdate and endDate

Will pick up once UI is a bit ready

 Project Read related bugs


board information missing on OAK-FFC

I am trying to get board information from OAK-FFC with following code:
auto dev = std::make_unique<dai::Device>();
auto eeprom = dev->readCalibration().getEepromData();
auto boardName = eeprom.boardName;
auto boardRev= eeprom.boardRev;

Actual behavior:
boardName and boardRev are empty strings on OAK-FFC
on OAK-D it correctly returns OAK-D and R1M0E1
Expected behavior:
For OAK-FFC it should return correct boardName and boardRev and possibly other info.
Reason:
I need this info because you have introduces breaking change on OAK-FFC-3P (Different pinout on camera connector). And I need to give warning to user of our system that some cameras cannot be connected to this board.
Erik from your company pointed out that the OAK-FFC does not have camera so it is not needed to calibrate it. So nothing is flashed into eeprom. Maybe you can start flashing eeprom also on OAK-FFC.
Thanks

@oto313 you can use the script inside the zip below, you'll just have to run it and wait a few minutes.
Please be careful with this, only use the script for the OAK-FFC-4P device, make sure only one OAK-FFC-4P is connected to the PC, and do not disconnect the device while the script is running.
write_ffc_4p.zip

Ok thanks for information. But I have OAK-FFC-3P and OAK-FFC-3P-OG. I have all revisions as in this picture https://github.com/luxonis/depthai-hardware/blob/master/DM1090FFC_DepthAI_USB3/Images/Flowchart_FFC.png

ah, ok, let me check

Hi @daxoft any progress?

flash_FFC_3P.zip
Hi @oto313
Yes, sorry for delay, here are the files for flashing the EEPROM of the 3 versions, I could only test DM1090FFC version since the other two are not in production anymore, but it should work, let me know if you have any issues.


silent notification & perform background operation once received

Hey @rdelrosario , thank you so much for this wonderful plugin!
I was inspired by your "ChatUIXForms" and had implemented the same in my App. Amazing 3 blogs I must say!
Um, I need your help here:

I want to send silent notification to both Andorid and iOS. To do so, I am using below payload but still it's getting displayed as Notification:

{
	"to" : "<device_token>",
	"data" : {
		"title" : "title",
		"message" : "message",
		"type" : "order"
	},
	"notification" : {
		"body" : "Hello world",
		"title" : "FCM Message",
		"content_available": true
	}
}

I want it to be slient, not showing any alert at all! How can we achieve this?

Once we receive this notification, I want to perform some background operation, like initiating socket connection (without user opening the notification).
So how do I achieve this? In which method should I write my socket code into?

As per the Doc, once we receive the notification we have 30 Sec. to download of new content, which in my case is to initiate the socket connection!
In advance, thank so much for your help.

You send a message with a "notification" tag. AFAIK this is never a silent notification, since the system handles the notification part.


Configurating sign-in component in a2t-ui

Hello. Is it any way to config links in sign-in component? In my project only administrator can register anybody, so i don't want to display 'sign up' link on the first screen.

Hello @er-ant, I don't think this is an issue with the library, whan you can do is do not add a signup button on the first screen, then on the Route for the signup validate if the used is loged and if he is an administrator, you can do this in both backend and frontend.

@Verurteilt I am about configuring default a2t-ui module.

Thanks @Veruteilt.
The Module is supposed to be for an easy start only, it is therefore not configurable. You can roll your own "quick start" by cloning the example repo.


Implement "multiline" prompt

Motivation looks similar to #1, the idea here is to allow the user to input a multi-line text without having to open a text editor.
Not sure how useful this would be, as I consider #1 more important.
Edit: this is now activelly not planned to be implemented by the maintainer (me...). I'll be leaving this open in case anyone is interested in implementing it.

It won't be easy if you are going to keep the package crossplatform.
Maybe this one can be a source of inspiration: https://github.com/Avarel/mini-me

It won't be easy if you are going to keep the package crossplatform.

Can't say I have much experience on that front, what kind of challenges are you expecting?
I thought it would be pretty standard stuff, just a lot of work.
Anyway, probably another reason to prefer editor support over this :)

When I was writing my comment, I did not know yet that you had already used crossterm package. It is already crossplatform and it makes my comment unrelated.
I agree, editor support is good to have as well. I'm fine with calling an editor to type long text - I do it to enter multi-line git comments.  :)

would love to see this feature ... I have a use case where i need to write few bullet points , but opening an editor to do this feels really unnatural

@gopherine unfortunately it isn't in our plans due to how complex a multi-line editor would be
I'd gladly review any PRs but it should be a lot of work to implement it right and I unfortunately don't have the time atm

Unfortunately there isn't really much of a point in keeping this open. Even Github's CLI uses a text editor for larger inputs.
The point about accepting it if someone implements stays :)


Incorrect description for Challenge.

Challenge Name

https://www.freecodecamp.com/challenges/target-html-elements-with-selectors-using-jquery
Issue Description

In the challenge description, it says not to use single-quote selectors.
"single-quote selectors will not pass our tests."
But they pass the tests correctly.
So, either the tests should not pass or the description should be changed.
Browser Information

Browser Name, Version: Google chrome Version 57.0.2987.133
Operating System: Windows
Mobile, Desktop, or Tablet: desktop

Your Code

<script>
  $(document).ready(function() {
    $('button').addClass('animated bounce');
  });
</script>

Screenshot

Solution with single quotes is also passing on my machine with Chrome/Firefox. But it might be that it's not passing on some machines so I think the solution is to just make the wording a bit "softer".

@prateekgoel thanks for the issue and thanks @techstonia for writing up a PR to try and fix this. As I mentioned in the PR, this still doesn't change the test which requires no single quotes in your code.
This single quote requirement came from this issue https://github.com/freeCodeCamp/freeCodeCamp/issues/9303, which essentially states that jQuery prefers double quotes. However, I'm not aware of any limitation in the challenge tests to detect single quotes. I think that requirement was just so that it is easier to detect a specific type of quote, rather than build in searching for both double and single quotes. Plus, I think stylistically, double quotes are preferred.
cc/ @freeCodeCamp/moderators I'm not sure if I'm missing anything, but I think both single and double quotes should be acceptable for this challenge? Correct me if I'm wrong. Otherwise, I vote for just modifying the tests and description to accept both single and double quotes.

I think if we try to enforce style guides then it should be done for all the jQuery challenges.
I thought that this double quote requirement came from some browser specific quirk and since I didn't find any evidence I just made the requirement a bit softer.

I agree, the description should be changed to remove the last paragraph. The tests works fine though :)

I am willing to take this up.

Thanks @Manish-Giri for the PR, but I've yet to find where jQuery suggests double-quotes vs. single and while it'd be more tedious, I think modifying all the tests to accept either (where necessary) may be a better route. cc @freeCodeCamp/moderators

@dhcodes if we're not going to enforce the use of double quotes, we should remove all mention of it from the challenge. Otherwise we're overloading campers with useless style opinions. They can choose whether to use single or double quotes for themselves, then when contributing to open source can use whatever that project's linting requires.
@Manish-Giri would you be interested in removing this line from the instructions and its corresponding test?

@QuincyLarson Yes, I'll do that.

@Manish-Giri Awesome - thanks for your help with this!


Horizontal scrollbar appears for numbered equations

If I add numbered equations to a div with the CSS property overflow: auto; I'm getting a scrollbar at the bottom of it even though there is enough space to show the equations in their entirety. If I suppress numbering that scrollbar disappears. The below screenshot shows the result.

The code to create the numbered equations looks as follows;
const numberedEquations = String.raw`
  \begin{align}
  \textcolor{ff00ff}{x_i} &= \textcolor{1280da}{x_s} + (\textcolor{0ed170}{x_e} - \textcolor{1280da}{x_s}) \cdot t \\
  \textcolor{ff00ff}{y_i} &= \textcolor{1280da}{y_s} + (\textcolor{0ed170}{y_e} - \textcolor{1280da}{y_s}) \cdot t
  \end{align}
`;

Suppressing automatic numbering by adding a * to both \begin{align} and \end{align} prevents the scrollbar from appearing. The code for the corresponding equations shown in the above screenshot looks as follows:
const equationsWithoutNumbering = String.raw`
  \begin{align*}
  \textcolor{ff00ff}{x_i} &= \textcolor{1280da}{x_s} + \textcolor{0ed170}{x_e} \cdot t - \textcolor{1280da}{x_s} \cdot t \\
  \textcolor{ff00ff}{y_i} &= \textcolor{1280da}{y_s} + \textcolor{0ed170}{y_e} \cdot t - \textcolor{1280da}{y_s} \cdot t
  \end{align*}
`;

The reason for using overflow: auto; is to be able to scroll equations if they are too large on mobile devices. I've come across this approach in #327.

I'm not sure this is a KaTeX bug as it arises in non-KaTeX situations as well because the browser is expecting content wider than what it's being given.
Two possible solutions (both somewhat hacky, I guess) which would need testing across a range of content and browser widths:
(1) Apply the following width to the container: width: calc(100% - 1px);
For example, on the KaTeX demo page, https://katex.org/, if I use your {align} example and apply the above width to the span with class "katex-display", the horizontal scroll bar disappears:

(2) Another solution I've used for this situation is to wait for KaTeX to do its thing, get the width of the processed span.katex and only apply overflow-x:auto if that width is greater than the available container width.
Hope it helps.
Regards
Murray

In my experience, this can happen for all sorts of formulas, not just numbered equations.  I haven't been able to figure out the cause.  I've seen it in non-KaTeX settings too.
The documentation gives a workaround for this specific issue: overflow: auto hidden.  I'm not aware of any problems with this CSS specification (I use it myself on my sites), though I'd be interested to know if you run into any problems.  Hopefully this solves your problem?

From what I observed so far, it only affects my numbered equations. Having numbered equations is useful to be able to refer to those equations in text and so on. However, I figured that the web technologies provide better ways to refer to objects on a page anyway which is why I stopped using numbered equations altogether.
Instead, I now highlight important equations by applying a CSS border to them using the following CSS rule:
.highlighted-equation .katex .base {
  max-width: 100%;
  overflow: auto;
  padding: 10px 10px 0 10px;
  border: 5px solid #ff0000;
}

Adding class="equation-highlighted" to the div into which the equation is rendered neatly highlights it. Adding an identifier to the same div makes it referable using the standard a HTML element with a href attribute starting with a # and continuing with the identifier.
So, while this does not solve the problem described above, it is a workaround that suits me well. The original issue might come back to bite me someday though. For the moment I'd keep that issue open.

From what I observed so far, it only affects my numbered equations. Having numbered equations is useful to be able to refer to those equations in text and so on. However, I figured that the web technologies provide better ways to refer to objects on a page anyway which is why I stopped using numbered equations altogether.
Instead, I now highlight important equations by applying a CSS border to them using the following CSS rule:
.highlighted-equation .katex .base {
  max-width: 100%;
  overflow: auto;
  padding: 10px 10px 0 10px;
  border: 5px solid #ff0000;
}

Adding class="equation-highlighted" to the div into which the equation is rendered neatly highlights it. Adding an identifier to the same div makes it referable using the standard a HTML element with a href attribute starting with a # and continuing with the identifier.
So, while this does not solve the problem described above, it is a workaround that suits me well. The original issue might come back to bite me someday though. For the moment I'd keep that issue open.

Hi.
I have tried searching the source of the problem and it seems to be this -2 margin right of the .vlist-t2 element
removing this -2 to 0 (or removing the css property) corrects the issue.
.katex .vlist-t2 {
margin-right:-2px
}
Initially

After removing

I don't know if it affects any other elements display.

For what it's worth, this only affects numbered equations in math environments. Things like
\tag{1}
x = 1

are fine. Only if you wrap it in any environment, e.g.,
\begin{equation}
x = 1
\end{equation}

it overflows.


Chrome Extension

I am not sure if there is a hastebin extension that supports custom urls if so can i get the link of it? If not i think you should implement it

Hi, @Oscaramr !
Thanks for such a nice idea!
I've checked Google Chrome Web Store and there is a hastebin extension there - https://chrome.google.com/webstore/detail/hastebin/jnjbjegfekjhdodgjblogcemdmdajocl
However it's not under our control.
You can try it out and check if it would do the job for you!
We don't have plans to create an extension ourselves in the near future. We are now focusing on improving the existing project.

Hi, @Oscaramr ! Thanks for such a nice idea!
I've checked Google Chrome Web Store and there is a hastebin extension there - https://chrome.google.com/webstore/detail/hastebin/jnjbjegfekjhdodgjblogcemdmdajocl However it's not under our control.
You can try it out and check if it would do the job for you!
We don't have plans to create an extension ourselves in the near future. We are now focusing on improving the existing project.

Go ahead for your project, you do what you want to do but i saw that extenstion and it doesnt allow for custom urls, which basically breaks the purpose of hosting your own haste bin. Maybe i can contact him and he could add it


Profile page Active Drafts sections need improved filters for drafts that have become RFCs

Describe the issue
Hi,
We received a ticket that on this person record<EMAIL_ADDRESS>it lists a nonactive wg (draft-clemm-nmrg-dist-intent) as Active.
Ticket: https://rt5.ietf.org/Ticket/Display.html?id=14903
Jenny
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

The problem is here:
https://github.com/ietf-tools/datatracker/blob/main/ietf/person/models.py#L188-L190
There is more than one state machine with "active" as a state. In the case reported above, the RG state machine is still in the "active" state.
The query above should be modified to match only the 'draft' state machine.
Similar adjustments should be made to the queries around it.


Add Sonar testing of repo

SonarCloud is a cloud-based version of SonarQube, and it is designed to automatically analyze and continuously inspect code for quality issues. Similar to SonarQube, SonarCloud helps identify and report on various aspects of code quality, including code smells, bugs, security vulnerabilities, and duplications. It is integrated with popular version control systems, including GitHub, making it a convenient solution for projects hosted on GitHub.
Here's how SonarCloud works with GitHub:

Integration with GitHub: SonarCloud integrates seamlessly with GitHub repositories. Users can connect their GitHub repositories to SonarCloud.
Configuration in GitHub Actions or Other CI/CD Systems: Developers configure GitHub Actions or other CI/CD systems to trigger SonarCloud analysis during the build and test process.
Automatic Code Analysis: Whenever changes are pushed to the GitHub repository, the CI/CD pipeline triggers SonarCloud to automatically analyze the code.
Detailed Reports and Feedback: SonarCloud generates detailed reports that include code quality metrics, issues, and recommendations for improvement. Developers receive feedback directly within the GitHub interface, making it easy to identify and address code quality issues.
Pull Request Decoration: SonarCloud can decorate GitHub Pull Requests with inline comments, providing developers with insights into code quality issues right in the pull request view.
Continuous Monitoring: SonarCloud provides continuous monitoring of the codebase, helping teams maintain a high level of code quality over time.
Security Vulnerability Detection: SonarCloud includes security vulnerability detection capabilities, helping teams identify and address potential security issues in the code.
Integration with GitHub Checks: SonarCloud can integrate with GitHub Checks, allowing developers to view the status of SonarCloud analysis directly in the GitHub Checks interface.

SonarCloud offers a user-friendly and cloud-hosted solution for continuous code quality management. It automates many aspects of static code analysis, making it easier for development teams to maintain and improve the overall quality of their codebases.
When referring to "SonarCloud GitHub tests," it likely involves the automated code analysis and quality checks that SonarCloud performs on code hosted on GitHub repositories, especially in the context of GitHub Actions or other CI/CD systems.


Unable to access hub.fastgit.org

For whatever reason today, when I tried to add a new plugin it seems vim-plug started having trouble accessing/downloading plugins from fastgit. Not sure what may have changed in my end or vim-plugs end for this to happen, so please let me know if you have any ideas!
:PlugInstall

:PlugUpdate

NVIM v0.5.0-dev+1353-g0a653f7ab
Build type: RelWithDebInfo
LuaJIT 2.0.5
Compilation: /usr/sbin/cc -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -fexceptions     -Wp,-D_FORTIFY_SOURCE=2,-D_GLIBCXX_ASSERTIONS     -Wformat -Werror=
format-security     -fstack-clash-protection -fcf-protection -Wp,-U_FORTIFY_SOURCE -Wp,-D_FORTIFY_SOURCE=1 -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-p
arameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnos
tics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/home/main-builder/p
kgwork/src/build/config -I/home/main-builder/pkgwork/src/neovim-nightly-git/src -I/usr/include -I/home/main-builder/pkgwork/src/build/src/nvim/auto -I/home/ma
in-builder/pkgwork/src/build/include
Compiled by main-builder

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

Type:

[x] Bug
[ ] Enhancement
[ ] Feature Request
[x] Question

OS:

[ ] All/Other
[x] Linux
[ ] OS X
[ ] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

Turns out I had, which is no longer valid.
let g:plug_url_format = 'https://hub.fastgit.org/%s.git'


could have a option to update / replace the same _id row?

I wish to trasnfer data from mongodb to ES, and I don't want to set 'query' or 'project'.
could have a option to update / replace the same _id row?

Hi @agrozyme ,
The tool doesn't provide an update/replace option. It only transfers the data itself between two platforms.

OK, thanks


fix: propagate GH headers into cloud event

Fixes #120
Proposed Changes
Encode X-GitHub-Event and X-GitHub-Delivery in Cloud Event extensions, encoded according to:

https://github.com/cloudevents/spec/blob/v0.1/spec.md#extensions
https://github.com/cloudevents/spec/blob/v0.1/http-transport-binding.md#3131-http-header-names

/ok-to-test

@mattmoor PR sent https://github.com/knative/pkg/pull/172, I'll update here if it gets merged.

@matzew @srvaroa I just approved the pkg PR

Updated with the latest pkg (which includes the now public constant)

@mattmoor the failures after upgrading pkg seem to relate to https://github.com/knative/pkg/commit/4b704fa7948ad9ae8ec90d1cd5b4a34516b252ea, do you have any pointers on the right way to fix them? Thx in advance.

I found a commit that seems to deal with this same update (https://github.com/knative/eventing/commit/7dc7624e34a4da510648d5d58235638502dd06a9). I've pushed the bump on knative/pkg separately at https://github.com/knative/eventing-sources/pull/130, applying the same fix to the tests to let them run. I'll wait until that is merged and update this PR.

@mattmoor this one was just pending the bump on pkg which seems to be working fine, any changes you'd like to see before merging?

/lgtm
/approve

I am now super excited for 0.3!