Urgent fix: Use filterChangeWatchedFiles hook to avoid infinite change loop

tl;dr The changes in this PR are required for this package to be usable on my machine.
Currently, if you enable logging on atom-languageclient via atom.config.set('core.debugLSP', true) and reload, this is what you see on startup.

It seems that we're getting into an endless loop. We detect the language server's own changes writing to a subdirectory of the project's target folder, then send those changes to the language server, which triggers another build and more writes, etc...
This PR makes use of the filterChangeWatchedFiles hook to avoid reporting changes inside of this target folder or the .git directory. It also takes care of updating the rls.toml while we're at it to avoid the need to subscribe to changes on package activation.
I also took some care to optimize these path comparison code paths by pre-computing the paths of interest and using a for...in loop rather than Object.keys.
/cc @damieng @daviwil because I think Rust is a super important target audience.

Thanks for doing this @nathansobo.

Actually if (filePath.startsWith(project.projectTargetPath)) return false is wrong as the rls is interested in when the target dir is deleted. That's why I implemented #15 the filter as
// #15
filterChangeWatchedFiles(filePath) {
  return !filePath.includes('/.git/') && !filePath.includes('/target/rls/')
}

This was fast, simple and didn't misuse the filter hook as a rls.toml watching mechanism. And as noted in that PR I think we eventually want to remove this anyway with being directed by the rls client/registerCapability requests telling us exactly what rls wants watching.

@alexheretic Sorry, didn't see your PR. Should have looked. I considered a similar fix with the includes check, but it's not 100% precise in filtering out only the paths in question. Sounds like I should follow up to scope it just to target/rls.
As for abusing the filter hook, it just seems unfortunate to look at all of these events twice when we're already invoking that hook for everything. I get it that it isn't strictly "filtering", but it avoids an extra call on every single change event. It seems simpler to me.

@mehcode Maybe @alexheretic's solution is better given the unlikelihood of those path fragments actually appearing anywhere else in practice. He is right that it's faster and simpler. If you want to revert this and merge #15 that's fine with me.
Is the language server just not responding at all after a while for anyone else. I thought this PR fixed that for me, hence my perception of urgency, but it seems it just delayed it upon using it a bit more.

@nathansobo Yes I suppose you could have a rust rls module inside a target module. But rls doesn't listen to the module files this way anyway, so it couldn't currently be an issue. Later this wouldn't be an issue either as we'd have swtiched to the client/registerCapability stuff.
The filter hook side effect stuff being bad is a matter of opinion, tbf probably not a big deal in this small code base.

@alexheretic Sounds like you know way more than I do about all of this and on further reflection O think you're right about all of it. I prefer your solution. Sorry for not scanning PRs before opening my own. Up way too late and got sloppy.

Perhaps I should have waited a bit before jumping the gun here with merging. I'll revert and merge #15.

@nathansobo @mehcode No problem at all, I'm just glad I can help a little with this. Thanks for replying and dealing with this so fast!


[flutter_gallery] text_form_field_demo problems

_autovalidate = true; should be wrapped inside setstate().
It is possible to get snackbar information "Please fix the errors in red before submitting." when there are no error messages (fill the name and phone fields correctly, then press Submit button).

I assume you mean https://github.com/flutter/flutter/blob/8e9780767110b8ec420f336572dbb8545190e52d/examples/flutter_gallery/lib/demo/material/text_form_field_demo.dart#L100-L101

Yes

Hi @hubertdej
Please try new https://github.com/flutter/gallery
Closing as this issue flutter_gallery issn't available anymore and removed
Thank you


[edn/dv] Randomly enable regwen in as many tests as possible

Currently only the edn_regwen test (which is edn_smoke_test + edn_regwen_vseq, thus very simple) enables regwen. We should ideally enable regwen with a certain probability in as many of the different test cases as possible, in order to verify that it works properly. (Note that there are also automated CSR tests, which AFAIK take regwen into account, so we should only add tests for what is not covered by the automated tests.)
This will also need support from the scoreboard (in the part that predicts the effects of TL accesses).

Moving to M7 as this is more like a V3 item.


Additions to the community showcase

Description
new additions as of 3rd aug to community showcase

@aryankhanna475 : can you resolve the conflicts please?


New Array methods

Are you doing any testing in Unity of the changes being made to the embedding API implementation? It may be worth running locally if nothing else, to verify any large changes don't cause failures to CoreCLR runtime tests.

Am I placing too much faith in the tests that exist in this repo?

I think they provide reasonable coverage. My recommendation is to try Unity itself at reasonable intervals, otherwise you may rewrite the entire embedding API and then have a large set of changes to debug if you have issues integrating.


[APIM 3.0.0 M33 Store] You can access APIs which is in created state via Store

Environment
WSO2 API Manager 3.0.0 M33
Reproduce Steps
Go to the overview section of a created api in publisher(API is in created state)
Go to the permission tab and click on the link "view in store"
You will redirect to the store and you can access that API in store even it's in created state
Actual
Even If the api is in created state, you can access it in store
Expected
APIs which is in created state should not be able to access via store

This is fixed in APIM 3.0.0 Alpha


hex::FromHex::Error does not implement Eq

Is there some reason why FromHexError does not derive Eq? It seems common in the Rust std lib to do so, here are a bunch of example errors that do so: https://doc.rust-lang.org/std/cmp/trait.Eq.html
Happy to do a PR but its so trivial I thought there might be some reason I can't work out.
Thanks

Are there plans to work on this?


Sync with Sketch and Sketch Beta

Preference option to have plugins sync to Sketch, Sketch Beta, or both.

+1


NUTCH-3044 Generator: NPE when extracting the host part of a URL fails

we could provide a TestGenerator#testNullHostInReducer test case

Good idea! Done, see 4729786.

... also fixed the Javadoc error.

Excellent @sebastian-nagel +1


Update documentation in anticipation of repo going public

README

Make it a lot smaller and link to separate documents (e.g. tutorial, releases page, FAQ, etc.) instead of having so much info in the README itself.
Suggest nightly releases as the recommended way to get the compiler, and include the note about the macOS security warning. Mention that in practice they'll probably need Rust, Zig, or C installed to use any platforms. Note that this will no longer be necessary once we have a package repo, but work won't likely start on a package repo until sometime in 2023.
Set clear expectations that Roc is a work in progress that is pre-0.1, there are lots of known bugs and unimplemented features, and that the documentation is incomplete.

Builtin docs

Make sure the documentation is actually in sync with the implementations.
Document any builtins that don't currently have documentation.

Tutorial

main.roc default
Use examples/cli to teach some basics - should resolve https://github.com/rtfeldman/roc/issues/506
Abilities
Opaque types
Modules
Tuples (if they've landed already)

roc-for-elm-programmers

Make sure it's actually accurate

This was all done long ago.


Relocate single node to directly after depot

Describe the bug
In Exchange<1,0>, when prev(V) == depot, the move (U,V) should move node U directly after the depot (before V) if this gives an improvement: [0, V, U, ...] => [0, U, V, ...]. The commit https://github.com/N-Wouda/PyVRP/commit/c8ac485c75f27e11b2cfecbbc865d867f9bc6cad defines a test case that fails in current main since it does not find this improved solution even though it exists and the only move in the neighbourhood is this move.
Code sample
See https://github.com/N-Wouda/PyVRP/commit/c8ac485c75f27e11b2cfecbbc865d867f9bc6cad.

INSTALLED VERSIONS
------------------
     pyvrp: 0.0.1
     numpy: 1.24.2
matplotlib: 3.7.0
    vrplib: 1.0.1
      tqdm: 4.64.1
     tomli: 2.0.1
    Python: 3.10.6

Expected Output
The node should be relocated to directly after the depot if this is a strict improvement, so the test case in https://github.com/N-Wouda/PyVRP/commit/c8ac485c75f27e11b2cfecbbc865d867f9bc6cad should succeed.

Normal relocate works fine (the first half of the test case), after fixing the test case to return the improved individual. So this:
    # First test normal relocate (325 after 533)
    neighbours = [[] for _ in range(data.num_clients + 1)]
    neighbours[325].append(533)
    ls = LocalSearch(data, pm, rng, neighbours)

    op = Exchange10(data, pm)
    ls.add_node_operator(op)

    indiv = Individual(data, pm, sol_before)
    assert_(indiv_expected.cost() < indiv.cost())
    improved = ls.search(indiv)

    assert_equal(improved, indiv_expected)

is OK.

https://github.com/N-Wouda/PyVRP/blob/44ebcbb9c7e0a03439a2b05ab094261110000010/pyvrp/cpp/educate/Exchange.h#L56-L63
This has a subtle bug: when V is the depot (V->position == 0) and we are in a relocate operator (where M == 0), the right-hand side of the second condition states 0 + 0 - 1. In unsigned arithmetic, that wraps around to some large number. This causes the entire function to return true, and thus moves wanting to relocate directly after the depot never get evaluated.
I have a fix and will commit that on #143 in a minute.


Implementing Flores 200 translation evaluation benchmark across 200 languages

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


appleIncRootCertificateNotFound

Hi, I get this issue if install via cocoapods as described
2017-02-03 13:52:56.618413 test-validate[2488:578098] appStoreReceiptURL: file:///private/var/mobile/Containers/Data/Application/505D79D0-6DA5-4D8E-8473-1B04AF9D0E31/StoreKit/sandboxReceipt
appleIncRootCertificateNotFound

It goes away, though, if I manually copy AppleIncRootCertificate.cer to my project and include it to my target
Thanks

Hi, thanks for letting me know. I will take a look shortly.

@truemetal Hey, I couldn't reproduce the issue. I've made a fresh project, installed the pod and I can see that certificate inside the framework's bundle.
Cocoapod 1.1.1
What's your version of cocoapod?

@tikhop the described error happens in runtime, when I try to execute this line:
let receipt = try InAppReceiptManager.shared.receipt()
yes, my cocoapods version is 1.1.1

@truemetal Yeah, I'm also testing in runtime. Moreover, this error can't happen when you execute that line. It seems that you have an old version of the library, because I throw this error only if you try to validate the receipt.

@tikhop you're right, I've had outdated pod version. This podfile:
# Uncomment the next line to define a global platform for your project
# platform :ios, '9.0'

target 'test-pod' do
  # Comment the next line if you're not using Swift and don't want to use dynamic frameworks
  use_frameworks!

  # Pods for test-pod
  pod 'TPInAppReceipt'
end

Produced this Podfile.lock:
PODS:
  - TPInAppReceipt (<IP_ADDRESS>)

DEPENDENCIES:
  - TPInAppReceipt

SPEC CHECKSUMS:
  TPInAppReceipt: 256bd4659cd93f9d16644db3d2b6a89c9089c14e

PODFILE CHECKSUM: c945987080b71e044ef877cb438dd756ef39e539

COCOAPODS: 1.1.1

After pod update everything is fine. Thank you.

Cool.
Make sure that verification process used to be a part of initialization process. However, since version 1.0 you have to execute this line try receipt.verify() in order to validate the receipt.


Users can view other users profile page.

I will describe this issue more soon. This final feature is yet to be finalized.


Update Tech Spec LC Smart Contract v0.3

[x] Explain input content and prerequisites (who signs, sign message)

[ ] Explain acknowledgement message content
[ ] Explain order of documents binaries (size)

[ ] Explain verification for documents
[ ] Explain input for documents binaries

@noobmdev @dmanh1204 update lại results


migrate_home.sh not setting login information correctly

@msneubauer is finding that after running
bash migrate_home.sh

that upon logout and login they aren't being dumped into /raid/projects/neubauer but are still being placed in /home/neubauer. This indicates that
https://github.com/Neubauer-Group/dgx-setup/blob/main/migrate_home.sh#L29-L30
isn't getting properly set or sourced in raid/projects/neubauer/.bash_profile.

This was due to not having.
So at minimum need a .bashrc_user like
#!/usr/bin/env bash

export HOME="/raid/projects/${USER}"

and a .bash_profile of
# .bash_profile
export HOME="/raid/projects/${USER}"

# Get the aliases and functions
if [ -f ~/.bashrc ]; then
        . ~/.bashrc
fi

# .bashrc_user sets $HOME to another location than default
cd "${HOME}"

# User specific environment and startup programs

PATH="${PATH}:${HOME}/.local/bin:${HOME}/bin"

export PATH


Colors/rgba/rgbaf: support tuple-shaped color lists

usage example:
https://github.com/mhaberler/jumpvis/blob/93b3b723d27aab7f3d4319cc91d06432022ddc6d/meteo.py#L124 and https://github.com/mhaberler/jumpvis/blob/93b3b723d27aab7f3d4319cc91d06432022ddc6d/meteo.py#L52-L53
sorry, hat a missing rebase

Codecov Report

Merging #79 (c02724b) into master (8185ff4) will decrease coverage by 0.26%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
- Coverage   99.19%   98.93%   -0.27%
==========================================
  Files          12       12
  Lines         743      749       +6
==========================================
+ Hits          737      741       +4
- Misses          6        8       +2

Impacted Files
Coverage Δ

src/czml3/properties.py
99.49% <66.66%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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


.menu not working

when i type .menu no answer, but another function was working correctly

Check #53

belum bisa,


fix: docker image diff command usage

LIIKUNTA-634

Sports-Helsinki branch is deployed to platta: https://liikunta-pr657.dev.hel.ninja :rocket::rocket::rocket:

EVENTS GRAPHQL PROXY is deployed to platta: https://events-graphql-proxy-pr657.dev.hel.ninja :rocket::rocket::rocket:


Add business logic execution time measure

We should also set a threshold that will log a warning message if execution time exceeds threshold.

Implemented as of 7c37c2d


Fix default container location

The default value was missing a v character in the container image tag. The correct location is ghcr.io/digitalis-io/vals-operator:v0.7.2.
Without this change, a default helm install ... ends up in the ImagePullBackOff state.

Thank you @irasnyd


AttributeError: 'Node' object has no attribute 'output_masks'

Hi @ZFTurbo ,
Thanks for sharing this amazing work. I am able to import the pretrained efficient_3D but later while setting the batchnorm layer to the inference mode by below code. Facing issue:

Any suggestion or help would be appreciated!
Best,
Dilip

I googled error:
https://stackoverflow.com/questions/51821537/attributeerror-node-object-has-no-attribute-output-masks
You're likely importing tf.keras.layers or tf.keras.applications or other keras modules from tensorflow.keras, and mixing these objects with objects from the "pure" keras package, which is not compatible, based upon version, etc.
Check how you import keras. import keras from tf or import keras and try to change it.

This can also happen if you got Keras installed in the same env you are using with tf.keras, due to the way the code in model.py is written (use keras unless exception is thrown).

The question is answered, so I think this issue can be closed.


block-timestamp

Add support for block timestamps to BlockInfo. Many database entries use the now() command to save timestamps and the state can't be re-created without access to the block timestamp.

Need to branch off of develop properly and re-submit PR


Chatterino account switching

A way to add multiple accounts
Suggestion: be able to sign in with multiple accounts and easily switch between them. I tried to do this, and when adding an account, you can only log in with one. When clicking 'Log In (Opens in browser)' it gives you the auth link to the same one. If you try the advanced tab I am unaware of a way to get the user ID, client ID, and OAuth token for another account.
It is a useful way of switching between accounts quickly.

This already does work, but if you are still logged into Twitch with the same account in the default browser, it's going to present you with a login token for the same account.
Either log out of Twitch before logging in via Chatterino, or open https://chatterino.com/client_login in incognito mode.

ok thanks


feat: add pre-commit for conventional commit

This PR adds a pre-commit hook that ensures commits adhere to Conventional Commit. The installation of this hook is optional since I haven't found a good solution that doesn't require tools such as Python or Node be available on the developer's device. Because of this (and because commits can be created with the --no-verify flag), a GitHub action is added that also validates all commits in PRs follow Conventional Commit.

To link it up, this pr is related to automated release: #4025

I'm guessing you mean the enforcing of Conventional Commit messaging. It does create some friction which I also don't really like. However, having proper commit messages is definitely a best practice and you get used to it quite quickly (there's even a CLI tool to automatically create proper commit messages for you). It's also the only way to have proper versioning and change logs. It should also help ensure new features aren't accidentally introduced into maintenance branches that should only be receiving fixes or breaking changes getting into branches they shouldn't. Strictly speaking not every commit would need to adhere to the Conventional Commit standard, but those commits are ignored for deciding if a release should be cut and what the version of the release needs to be and could cause rouge commits in branches they shouldn't be.

What metrics would we use to determine if it affects contribution?
It doesn't stop you raising a pr or stop the tests running, it's not much different to us asking for a squash or that commits are signed off. Signing off commits seems like a similar hurdle too - rewrite the commit message
If we notice ill effects we can always pick a new process.

I'm guessing you mean the enforcing of Conventional Commit messaging. It does create some friction which I also don't really like.

Yes, that's what I meant. There are already so many other barriers to get over (the ease of running tests, etc, etc.) that I feel like adding another obstruction could (would?) just make things even harder

However, having proper commit messages is definitely a best practice

I'd wager this is relative 😄 but don't get me wrong, I do get the benefits of having them. Mind you the mere requirement of the hassle of having to install some tool can be a put off

What metrics would we use to determine if it affects contribution?

This is a good question. I think right now the only guiding light we have is the number of PRs opened within a certain time period 🤔

Signing off commits seems like a similar hurdle too - rewrite the commit message

Yeah I dont like that either and it's been a major pain for me personally getting people signing off their commits
I'm looking at https://github.com/goharbor/harbor/pulls and it seems it's a bit of a mixed bag - some conventional commit messages, some "random"
Curious what other @distribution/maintainers think.
I want to reiterate: I am a fan of this, but as I said my worry is a new point of friction (on top of git itself, DCO, etc.)


chore: update samples go.mod

Fixes Dependabot issues when upgrading dependencies using Generics or any.
Ref: #903

@duglin

@lionelvillard for opinion

@lionelvillard any thoughts on upgrading golang? Or anyone else from the community? Does anyone know what might break by moving to 1.18 from 1.17?

Sorry, I probably wasn't too clear on the change/ask here. The change is in the samples, bumping go.mod version to 1.18 to keep up with dependencies. The v2 go.mod remains at 1.17. However, I had to upgrade the CI because it runs across the various go.mod files and broke with the sample 1.18 (since an older version was used to compile a breaking release using any).
The README of the SDK states we support 1.17. My concern was that we're only testing with 1.18 now, i.e., we don't see regressions with 1.17 (unless we add fancy foo to CI which I'm not keen to touch tbh).
We could update the README as well and keep v2 go.mod at 1.17 to not cause downstream impact (on SDK users).

rebase needed

@duglin going to close this draft PR until we see issues with Dependabot again.


What is the algorithm of accelation ?

I see the acceleration seems being achieved by origin_by_velocity_and_sample and psuedo_velocity_wrt_noisy_and_timestep funtions, could you please tell us the algorithm behind it ?

maybe lcm，after model distillation，the steps of denoising will be significantly reduced


About the Pre-trained weight

Hello, thanks for your great work!  Currently the publicly available pre-training weights are trained based on CT data with input channel 1. However, the downstream task of Brats21 is a 4-modal data with 4 channel inputs. Using the “Load Pre-trained weight” code you provided will result in a parameter mismatch error. How should the CT pre-training weights be applied to the MRI data? Thank you!

Hi, many thanks for your kind attention to our work!
For MRI training, please refer to these two closed issues:
https://github.com/Luffy03/VoCo/issues/2
https://github.com/Luffy03/VoCo/issues/3
Feel free to raise your concerns here!

Hi, many thanks for your kind attention to our work! For MRI training, please refer to these two closed issues: #2 #3 Feel free to raise your concerns here!
Thank you for your answer. I would like to test the segmentation effect (without fine-tuning) using the VoCo_10k.pt you provided directly on 4 modalities of MRI data. The code to load the weights is as follows:
model_dict = torch.load('./pretrained_models/VoCo_10k.pt', map_location=torch.device('cpu'))
state_dict = model_dict
state_dict = delete_patch_embed(state_dict)
if "module." in list(state_dict.keys())[0]:
print("Tag 'module.' found in state dict - fixing!")
for key in list(state_dict.keys()):
state_dict[key.replace("module.", "")] = state_dict.pop(key)
if "swin_vit" in list(state_dict.keys())[0]:
print("Tag 'swin_vit' found in state dict - fixing!")
for key in list(state_dict.keys()):
state_dict[key.replace("swin_vit", "swinViT")] = state_dict.pop(key)
model.load_state_dict(state_dict, strict=False)
print("Using pretrained voco ema self-supervised Swin UNETR backbone weights !")

After using “delete_patch_embed”, I still get the following error:

RuntimeError: Error(s) in loading state_dict for SwinUNETR:
size mismatch for encoder1.layer.conv1.conv.weight: copying a param with shape torch.Size([48, 1, 3, 3, 3]) from checkpoint, the shape in current model is torch.Size([48, 4, 3, 3, 3]).
size mismatch for encoder1.layer.conv3.conv.weight: copying a param with shape torch.Size([48, 1, 1, 1, 1]) from checkpoint, the shape in current model is torch.Size([48, 4, 1, 1, 1]).

Many thanks for pointing out our mistake! The previous version of function delete is used for SwinunetrV1 and our 1.6k pre-training. I have updated it as https://github.com/Luffy03/VoCo/blob/5cb1297095f0fc55dc81b4e40640e4ee80e2c4e0/Finetune/nnUNet/nnunetv2/training/nnUNetTrainer/nnUNetTrainer_swin.py#L153.

Many thanks for pointing out our mistake! The previous version of function delete is used for SwinunetrV1 and our 1.6k pre-training. I have updated it as
https://github.com/Luffy03/VoCo/blob/5cb1297095f0fc55dc81b4e40640e4ee80e2c4e0/Finetune/nnUNet/nnunetv2/training/nnUNetTrainer/nnUNetTrainer_swin.py#L153
.

Thanks for the answer, the code has run through, much appreciated!

Hello, thanks for your great work! I tried to train voco on the multimodal Brats dataset with the following train_transforms, but it seems that only one modality (one channel) can be loaded. May I know what processing should be done if I use multimodal MRI for pre-training?

Hello, thanks for your great work! I tried to train voco on the multimodal Brats dataset with the following train_transforms, but it seems that only one modality (one channel) can be loaded. May I know what processing should be done if I use multimodal MRI for pre-training?

Sorry for my late reply.
To pre-train on MRI data, you need to change 'in_channels' to 4 if you have 4 sequences or (modalities) as in https://github.com/Luffy03/VoCo/blob/94ed426bec328b7b9b5ddcf25b43fc14f27672ab/voco_train.py#L131


fetching volume timeout

A note for the community

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
If you are interested in working on this issue or have submitted a pull request, please let us know in a comment

Problem
I see timeout errors with feching volumes sometimes.
error="failed to fetch data: error making request StatusCode: 200, Error: context deadline exceeded (Client.Timeout or context cancellation while reading body), API: /api/private/cli/volume?fields=snaplock_type%2Csize_available_for_snapshots%2Csnapshot_reserve_size%2Ccompression_space_saved%2Cexpected_available%2Cdedupe_space_saved%2Csnapshot_count%2Cnodes%2Clogical_used%2Coverwrite_reserve_used%2Csis_space_saved%2Csnapshot_reserve_available%2Cused%2Cfilesystem_size%2Cphysical_used%2Cavailable%2Cstate%2Clogical_used_by_afs%2Coverwrite_reserve%2Csize%2Csize_used_by_snapshots%2Cis-space-enforcement-logical%2Cdedupe_space_saved_percent%2Cfiles_used%2Ctype%2Csis_space_saved_percent%2Cvserver%2Cclone_parent_vserver%2Cclone_volume%2Cis_encrypted%2Cfiles%2Clogical_used_percent%2Cpercent_snapshot_space%2Csnapshot_space_used%2Ctotal%2Cis_sis_volume%2Cautosize_grow_threshold_percent%2Cpercent_used%2Cvolume%2Cclone_parent_name%2Csnapshot_policy%2Ccomment%2Clogical_used_by_snapshots%2Caggr_list%2Ccompression_space_saved_percent%2Cmax_autosize%2Cis_space_reporting_logical%2Cuuid%2Cvsroot%2Cphysical_used_percent%2Cjunction_path%2Cvolume_style_extended&is_constituent=%2A&return_records=true" Poller=stnpca1-st090 collector=Rest:Volume task=data

what is the default timeout time? is it configurable? Can I configure it to fetch in pages?
Configuration
No response
Poller
Volume
Version
harvest version 23.11.0-1 (commit 166b0263) (build date 2023-11-13T14:57:09+0000) linux/amd64
Poller logs
No response
OS and platform
Debian GNU/Linux 11 (bullseye)
ONTAP or StorageGRID version
NetApp Release 9.14.1P2: Tue Mar 19 23:28:08 UTC 2024
Additional Context
No response
References
No response

Thank you, I will try it out

Closing. Feel free to reopen if you still see this issue


Improved code of search page and scanner modal to view product and scanner in same view(#25k8p8j)

Related Issues

Closes #
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules
[ ] I have changed the target to hacktoberfest branch

https://user-images.githubusercontent.com/82817616/162445678-325ced74-851f-4b11-9648-4e761727d06a.mp4

Not using this UI anymore, closing PR


disables generation of pyramids for output formats which do not support pyramids in bfconvert tool

FOR CONTRIBUTOR:

[x] - I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
[x] - License permits unrestricted use (educational + commercial)
[ ] - This PR adds a new tool or tool collection
[x] - This PR updates an existing tool or tool collection
[ ] - This PR does something else (explain below)

@alliecreason Kind reminder to please check whether the changes of this PR are fine from your side :)


[app] Add "mongodb" driver

Add a new "mongodb" driver to use MongoDB as storage backend instead of
Bolt. The MongoDB storage backend can be used to scale kobs, when the
Bolt backend reaches it's limits.
The MongoDB storage backend can be used via the "hub.store.driver" flag.
To get the best results with the MongoDB storage backend we recommend to
use two deployments for the hub: One deployment can be used to serve the
API and React UI ("--hub.mode=server") and the other one can be used to
sync the data from the configured satellites and write the data into
MongoDB ("--hub.mode=watcher"). This deployment architecture is also
supported by the Helm chart, via the "watcher.enabled" value.
To add a readiness and liveness probe for the watcher Pods, the metrics
server has now a "/health" endpoint which is used in the Helm chart. We
also had to update the Go dependencies, to use the new MongoDB storage
backend.

[x] I added a CHANGELOG entry for this change.
[x] I adjusted the corresponding documentation for this change.
[ ] I adjusted the values.yaml file and the documentation for all values.

Codecov Report

Merging #402 (30e6432) into main (6b11697) will increase coverage by 1.58%.
The diff coverage is 70.05%.

@@            Coverage Diff             @@
##             main     #402      +/-   ##
==========================================
+ Coverage   45.66%   47.24%   +1.58%
==========================================
  Files         137      138       +1
  Lines       10672    11439     +767
==========================================
+ Hits         4873     5404     +531
- Misses       5706     5897     +191
- Partials       93      138      +45

Impacted Files
Coverage Δ

cmd/kobs/hub/hub.go
0.00% <0.00%> (ø)

pkg/hub/store/shared/cluster.go
100.00% <ø> (ø)

pkg/hub/store/store.go
75.00% <0.00%> (-25.00%)
:arrow_down:

pkg/kube/clusters/cluster/cluster.go
29.90% <ø> (ø)

pkg/metrics/metrics.go
0.00% <0.00%> (ø)

pkg/hub/store/mongodb/mongodb.go
70.60% <70.60%> (ø)

pkg/hub/watcher/watcher.go
8.73% <0.00%> (-3.40%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us.


Add RxLint

http://www.littlerobots.nl/blog/RxLint-a-lint-rule-for-RxJava/

Current coverage is 81.58%

Merging #151 into master will not affect coverage as of 8dd61e9

@@            master    #151   diff @@
======================================
  Files           38      38
  Stmts          554     554
  Branches        47      47
  Methods          0       0
======================================
  Hit            452     452
  Partial         29      29
  Missed          73      73

Review entire Coverage Diff as of 8dd61e9

Powered by Codecov. Updated on successful CI builds.

Was on my list :) :+1:
Strange to see 0 warnings, in NY Times I got ~15 false-positives (already reported)

👍 great tool, will add to my project as well.

Me too

@artem-zinnatullin anything against merging it?

@vanniktech only my lack of time this week (and at least few next weeks too…)
sorry


Some conda environments missing from selection dropdown

Environment data
Version: 1.43.2 (system setup)
Commit: 0ba0ca52957102ca3527cf479571617f0de6ed50
Date: 2020-03-24T07:38:38.248Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.18362

VS Code version: 1.43.2 (system setup)
Extension version (available under the Extensions sidebar): 2020.3.69010
OS and version: Windows_NT x64 10.0.18362
Python version (& distribution if applicable, e.g. Anaconda): Anaconda
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: Python
Jedi or Language Server? (i.e. what is "python.jediEnabled" set to; more info #3977): Jedi
Value of the python.languageServer setting: Microsoft

Expected behaviour
All conda environments available are shown in dropdown that appears when clicking Python environment name in bottom left corner of VSCode window.
Actual behaviour
Only some conda environments are shown (see image). In the Anaconda Prompt window (left), the output conda env list includes environments guild, herokuflask and radian. These do not appear in the VSCode environment dropdown. The other environments appear in both Anaconda Prompt and VSCode.
I THINK the envs not being shown are the ones I created most recently. All of them were created at least one month ago.

The only VSCode extension installed is Python.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]
Not sure.

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
None (it is blank)
Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
Log attached. I wasn't sure where to run Enable source map support for extension debugging.
log.log
Thank you.

Hello @andycraig,
Thank you for reaching out. Let's see if we can narrow down your issue, can you try out the following:

If you create a new conda environment now, will it appear in the dropdown the next time you open a Python file?
Is the interpreter recognized if you set python.pythonPath (in your settings.json file) to the path to the python executable of one of the missing environments?

Can you also provide the following:

The content of your settings.json file
The version of conda/Anaconda you're using
The content of your environments.txt file (should be at C:\Users\AC\.conda\environments.txt)

Thanks!

Hi @kimadeline, thank you for your help.

The version of conda/Anaconda you're using

conda 4.2.9

The content of your settings.json file

settings.zip

The content of your environments.txt file (should be at C:\Users\AC.conda\environments.txt)

environments.txt

Is the interpreter recognized if you set python.pythonPath (in your settings.json file) to the path to the python executable of one of the missing environments?

In trying this I discovered what the difference between the environments shown/not shown in VSCode. The environments shown in VSCode contain an executable python.exe. The environments not shown in VSCode do not contain an executable python.exe.
I now know how to reproduce the issue.
To reproduce
In Anaconda Prompt:
conda new --name wontshow

y at prompt.
conda create --name willshow python=3.6

y at prompt.
C:\Anaconda3\envs\wontshow contains no python.exe, and does not show in VSCode.
C:\Anaconda3\envs\willshow contains python.exe, and shows in VSCode.
Note that Python can still be run from wontshow. In Anaconda Prompt:
activate wontshow
python
# Output:
# Python 3.5.2 |Anaconda 4.2.0 (64-bit)| (default, Jul  5 2016, 11:41:13) [MSC v.1900 64 bit (AMD64)] on win32
# Type "help", "copyright", "credits" or "license" for more information.
# >>>

In this case my guess is that it is using the executable C:\Anaconda3>python.exe, which is also Python 3.5.2.

Thank you for investigating it :mag_right:

C:\Anaconda3\envs\wontshow contains no python.exe, and does not show in VSCode.

This is as expected, since the extension will only list environments with an interpreter. You confirmed it when you created C:\Anaconda3\envs\willshow and it contained a python executable.

my guess is that it is using the executable C:\Anaconda3>python.exe

You can verify it by running which python, it will show you the path to the executable being used:

👉 On a side note, I tried conda new using conda 4.8.1 and it doesn't exist anymore, it doesn't even appear in the command reference documentation. I would suggest updating to a more recent version of conda, and continuing to use conda create [--name|-n] envname python=3.X to create your conda environments.
Closing since the extension is working as expected.

@kimadeline Thank you for your help identifying the source of my problem! I can avoid it now since I know the cause. Good to discover it wasn't a bug after all.


Does anybody know the Default Wifi Password for EO Mini pro 2 - Broken QR Code

Hi all. I know this is probably the wrong place, but a friend of mine has the EO mini pro2. Their home wifi got disconnected for a hour or so and the app got disconnected.
They went to try connecting again via the qr code, but the code had faded in the sun and is not scanning
Ive tried Enhancing the QR Code in photoshop, but the qr code still wont scan via the app.
I am able to scan it with my iphone camera and it returns a code (some letters changed)
1 RU0yMTc4UEg= PA9IVUItMTc4MDg= M1AwVTRIPlRTNkFX 811baa67aeb6040ad40d27e95e036e200a3b9509844ac249a91fac0f4e48f971
My understanding was, all this qr did was allow you to connect to the wifi signal that the charger was putting out. EG eoHUB-17101
Does anybody know what the default wifi password so i can try that?

Have you reached out to EO support?
They could likely advise, or failing that provide the QR code. I was under the belief it was unique for each unit.

They have, but they dont provide any support really. It with their support team , but no update in the last 2 weeks despite numerous calls
from playing around it seems like charger is offline, because it cant connect to the internet. The led is blue and goes to green when you plug it in, but nothing happens, the car doesnt charge
Do you know if you scan your qr code with your normal camera, does it come out like that string ?

Sorry about your trouble, but I don't see how this home assistant integration can help with this - the integration relies on the cloud service hosted by EO, rather than directly connecting to the charger's built-in wifi AP. I hope you find a solution.


Wrong number of parameters

I'm trying to use the command as simple as this: jiraf PROJECT-123 and it doesn't work.
I have this error: https://github.com/ivaaaan/jiraf/blob/496632856d58d78f40dad274f0227695aa9aabb8/main.go#L56-L57
Usually, the executable filename is included in the list of arguments in other languages, maybe it's the sane with go. And maybe you have never tested your CLI while being installed? Idk

Fixed as well. Thanks for your feedback


[Feature]: Dynamically generating form structures

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Currently, the function "Generate from data table" already exists, which reads values from MongoDB and creates a nice editor. However, all components are generated statically and are therefore not customizable if multiple data types are present in the MongoDB collection. For example, a field "value" could accept values of type Object, as well as of type String or Integer and then differentiate. Also, when creating new elements, it should be possible to create Objects without a predefined structure.
Why should this be worked on?
I am currently trying to add some sort of stripped down version of MongoDB Compass to Appsmith for end users to view and or edit specific Collections in Appsmith. All items in these collections have approximately the following structure:
{
name: "string",
value: >any<
}
Here, value can actually have any data type supported by MongoDB.
If you create a table from MongoDB data for the first time, all table components and forms are generated based on the structure of the already existing elements.
Now the problem: If the collection has a not very static structure, it is problematic that the table elements are relatively static. So it may be that the form for creating new elements now has a field "value" which only supports Objects, although you want to enter a String or a Date, or instead of a String you want to enter an Object.
Another case is when you want to insert an object with custom columns, but the form just doesn't support it, because it only supports the previously defined columns.
Would it be possible in general to provide a dynamic generation as an option for e.g. some fields?

@Schniemax I'd like to learn a bit more about your use case. As I understand it, in your data, the value type of the key is dynamically changing but in the case of the JSON form, it should automatically update based on the type of data

Im sorry for the late response. Our use case is the following:
We have created a MongoDB Collection, which we are using as a storage for configuration settings for several projects. We want to use appsmith for editing the configuration within the browser (basically some user-friendly simplified sort of replacement of MongoDB Compass).
Therefore a table for each collection of configurations is created.
Each setting entry has this structure:
{
name: "string",
value: >any<,
release: "boolean",
delete: "boolean"
}
Basically all entries just differ in the type of the value. It is possible that one entry contains an int, the other one a set and the third one an object inside.
The problem is, that the table just creates the edit form for one datatype, normally the first element. This can be changed, so that it adapts to all types of data, but still there are several problems. For example:

you can not change the structure of a given object, it should be possible to enter new entries
you can only delete array entries, but when adding them, you only have only the default color options (yellow, ...), it should be possible to enter your own new entries

Also the creation of a new element can only support the first datatype in the collection. Therefore, when e.g. a String is loaded first, you can not create new objects or other types of data, because they will all be stored as string.
And thats the point. It would be great if the edit and new element form would support any datatype without the need of creating big workarounds that reduce usability.


Fixes

Some fixes:

Add the ip and control labels to the plotfingerprint.
Correctly assign ch_picardcollectmultiplemetrics_multiqc channel
Correct pvalue and qvalue arguments for macs2 callpeak
Fix skip_peak_annotation and skip_peak_qc options
Run preseq on the bam coming from markduplicates and not after filtering orphans

All comments have been addressed in 277bd11


#2714 the models array can be empty

Description
Fixes issue #2714
Checklist

[ X] The base branch of this PR is dev, rather than main
[ X] The relevant docs, if any, have been updated or created

Screenshots
[ For visual changes, include screenshots. ]
Testing
[ For new or modified features, provide step-by-step testing instructions to validate the intended behavior of the change. ]
[ For new UI features, ensure that the changes look good across viewport widths, light/dark theme, etc ]

oops completely forgot to read the issue 😅 yeah that makes sense. thx for the PR!


Replace depracted container by getContainer

Replace depracted container by getContainer

I was running into the same problem before. The problem is that getContainer is not available in Symfony 5.4.
The solution could be a BC Layer (do not know how to implement it yet) or stay with the deprecated version. (i guess the problem will arise again with symfony 6 support)
5.4 https://github.com/symfony/symfony/blob/5.4/src/Symfony/Bundle/FrameworkBundle/Test/KernelTestCase.php => static::getContainer() available
4.4 https://github.com/symfony/symfony/blob/4.4/src/Symfony/Bundle/FrameworkBundle/Test/KernelTestCase.php => only static::$container available

Yes, that could be great! We can do this for now and dropping it when dropping Symfony 4.4 support

I think we can just keep $container for the moment (it's Ok to have deprecated code in code base) and we replace by getContainer when we drop the symfony 4.4 support.
My 2 cents

That is an valid approach and totally understandable.
End of Support for 4.4 is November 2022/23. https://symfony.com/releases
I hate to code for 4.4 in bundles. Because it is hard to use latest features, support symfony 6 and handle 4.4.
It woul be interesting to know if $container is working in symfony 6. (and removed only in symfony 7)

I'm still thinking of dropping Symfony 4.4 in the first major version of that bundle.
But for the moment, let's try to fix it as it's only two classes to add (see #67) :slightly_smiling_face:

Superseded by https://github.com/mtarld/api-platform-ms-bundle/pull/67, I'm gonna close it
Many thanks, @ismail1432!


hello world, rebuilding unclear

Where in the docs did you come across this?
https://rustwasm.github.io/book/game-of-life/hello-world.html
Describe what about it does not make sense
Exercises says  "rebuild the .wasm binary,", but no text before that explains how you actually do that. The only command that was mentioned before is wasm-pack init,
Why does it not make sense?
according to https://github.com/rustwasm/wasm-pack/issues/185 the correct command is
wasm-pack init --mode no-install

but that is non obvious
How could we improve it?
mention wasm-pack init --mode no-install

Hey there. I was just completing the hello-world tutorial. A note in the book stating that one can use wasm-pack init --mode no-install(or wasm-pack build --mode no-install eventually since init is DEPRECATED) to skip installing wasm-bindgen would be helpful...


Prepare for melpa

Hi,
I tried to prepare the package for inclusion in the MELPA package archive. (I tested the changes locally by cloning melpa/melpa and doing make recipes/moonscript there until everything seemed ok.)
The package will include only the MoonScript editing mode at this time - I left out the REPL mode because it's so experimental. We could add it to the package later or make it a separate package.
I had to rename the main file to moonscript.el (dropping -mode from the name) because all the symbols there start with moonscript (whereas not all of them start with moonscript-mode). If we wanted to keep the filename moonscript-mode.el then we'd have to rename all the symbols to start with moonscript-mode which is more work.
In order to get the package into MELPA we need to send them a pull request that adds a very short recipe file. Please see https://github.com/lassik/melpa/commit/fab4ee9c5613fa5dd136031426d6ba7b699b527b for the contents of the file. Do you want to send them the request, or should I?
BR,
Lassi

@lassik thanks so much for this! feel free to send the PR to melpa


Add metrics for measuring query load

query-metrics-max-live-samples - Maximum amount of data points in memory
at any time for a query.
query-sample-latency - Effectively the latency of 1 sample read. This
makes for a latency measurement that is comparable across queries. This
can be used to show if any query loaded the system more than what would
have been expected from just the number of samples.

Codecov Report

Merging #284 into master will decrease coverage by 0.01%.
The diff coverage is 23.8%.

@@             Coverage Diff              @@
##             master     #284      +/-   ##
============================================
- Coverage      47.6%   47.58%   -0.02%
- Complexity     2557     2558       +1
============================================
  Files           610      610
  Lines         16130    16143      +13
  Branches       1200     1200
============================================
+ Hits           7679     7682       +3
- Misses         8000     8010      +10
  Partials        451      451

Impacted Files
Coverage Δ
Complexity Δ

...istics/semantic/SemanticMetricBackendReporter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...oic/statistics/semantic/SemanticQueryReporter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...tify/heroic/statistics/noop/NoopQueryReporter.java
75% <100%> (+3.57%)
6 <1> (+1)
:arrow_up:

...main/java/com/spotify/heroic/CoreQueryManager.java
75.64% <100%> (+0.25%)
9 <0> (ø)
:arrow_down:

...om/spotify/heroic/metric/memory/MemoryBackend.java
80.59% <0%> (-2.99%)
13% <0%> (-1%)

...m/spotify/heroic/aggregation/simple/MinBucket.java
52.38% <0%> (+9.52%)
5% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

Hold off on this one a bit. I'm adding a third metric.

@udoprog, @zalenski: This is now ready for code review, if you'd like. Thanks!


Propagate object flags related to object and array literals onto module.exports types

fixes https://github.com/microsoft/TypeScript/issues/57460

@typescript-bot test it

IIRC those flags affect isPerformingExcessPropertyChecks
the in-repo change looks like being related to type inference for object literals. It looks like a correct change to me - it now benefits from this since we are correctly propagating those literal flags. As to the webpack's change from this comment - I'm not sure yet. I just synced with main, could we rerun those tests to get a confirmation that nothing wonky is happening with that report (I'm not sure how likely that is with the current infra)? If it comes back with this error again then I'll happily investigate this (cc @jakebailey )

According to Jake, the extended tests use the latest main, so the not-synced PR shouldn't be a problem, but I can try running again.

@typescript-bot test it

My impression at the moment is that the CFA change introduced some non-determinism; that same error went away in that change and is now back again.

Something weird is happening with those tests but I think it's fair to say that it's not related to this PR 😉

This PR was out of date and broke main.

This PR was out of date and broke main.

I'll fix it now


Add sentry to updater

we only have error reporting to one grafana view, sentry would help with this.
After adding sentry, add failures notifications for oodikone notifications channel in Slack.

Sentry and slack notifications added to Updater, will monitor in case slack notification rules need some adjustments.


ENH - More efficient B.dot and B.T.dot in Cox datafit

In Cox datafit, It is possible to perform the operations evolving the matrix B in linear time and without having to form explicitly the matrix.
Up to a sorting permutation of tm, the matrix B has a block triangular structure involving matrices with coordinates equal to 1.
Hence applying B to a vector (slightly) resembles performing a cumulative sum on the coordinates of v.
Advantage

reduce the memory footprint of Cox datafit from O(n²) --> O(n) as we no longer store B
Evaluate gradient and Hessian in O(n) instead of O(n²)

Benchmarks
A link to benchmark results to showcase the timing improvements compared to the main branch.

Riding "la trottinette :kick_scooter: à la @agramfort"

Need to merge #167 beforehand

The test fails because the gradient and Hessian disagree with scipy check_grad at atol=1e-3.
I'm not that familiar with truncation and roundoff errors to blame numerical errors for that.
However, I pushed a debug_script that checks independently the matrix-vector operations. It reveals that both operations are the same (difference of 1e-11).
@mathurinm, any clues?


routing guards/filters

Any future plans on implementing route guards/route filters for @page components something similar to
@attribute [Authorize]
or
https://docs.microsoft.com/en-us/aspnet/core/mvc/controllers/filters?view=aspnetcore-3.1
or
https://angular.io/guide/router#milestone-5-route-guards
just to control user from navigating. looks like the best way right now is create a base class which implements the initialize or on navigate to check if user has authorization/access to specified page then redirect or display error.
It would be cool to have something like
from the authorize attribute
@attribute [MyCustomPageComponentAuthorize]
then redirect or trigger an error response. Also please do document how to implement multiple custom attributes, is this possible?
@attribute [MyFilter, MyFilter2]
the above features are corollary to my previous question
https://github.com/dotnet/AspNetCore.Docs/issues/16111

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

ID: 79da574a-570f-04eb-86bd-c8065a016f44
Version Independent ID: ac771374-6102-b838-57d2-e99bcf3f7d3e
Content: ASP.NET Core Blazor routing
Content Source: aspnetcore/blazor/routing.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

Sorry about that @hoksource ... we didn't actually address your ask on #16111.
Yes, the [Authorize] attribute is covered in the Blazor security Overview ...
https://docs.microsoft.com/aspnet/core/security/blazor/
The approach for controlling the content that's served to a non-authorized user is to merely customize the content of the AuthorizeRouteView component ...
https://docs.microsoft.com/aspnet/core/security/blazor/#customize-unauthorized-content-with-the-router-component
... and that's obviously not very fine-grained to just place text/markup there. However, you could further tailor that by sticking your own component(s) in there that contain special content and logic.
WRT to further routing options or features that they're working on, I'll leave this up for triage for Artak to take a look at. These issues are usually looked at on Fridays.

... and one more tidbit WRT to general doc improvements. We've been so busy getting basic features covered on high priority issues, that a BIG sweep on topics for improved guidance is needed. There should be a quiet period right after 3.2 lands in May. The entire Blazor doc set will receive attention during that time. For example, your issue here implies that better cross-linking between the Routing and Security topics is called for. Those types up updates will occur with the UE work shortly.
Artak, unless you have something specific to call out, I recommend that we close this. Soon, I'll set up a tracking issue for discussion prior to the UE sweep, and better cross-linking will be a bullet point on that task list for UE improvements.

the exact problem i'm trying to solve is:
a page/X created by user A.
if user B tries to access page/X,
then a filter/event/guard should check the database if B has access to page/X.
If not force a NotFound response. then it will trigger the router results
or redirect to another page etc.
Im not sure if you can solve this with component life cycle OnInitialize or is there a BeforeInitialize event?

Hasn't this been asked and answered? ...
https://github.com/dotnet/aspnetcore/issues/18425

... try it the way that Javier suggested. Create your own MyCustomWrappingComponent to use based on AuthorizeRouteView, which you can find in the ref source. Customize it however you like.

https://github.com/dotnet/aspnetcore/blob/master/src/Components/Authorization/src/AuthorizeRouteView.cs

Artak, perhaps we should say in passing (one line quick remark) that readers can provide their own custom AuthorizeRouteView component with custom logic. I don't intend to link to the component ref source 🙈, but we can say that the AuthorizeRouteView component can be found in ref source and link to the engineering repo. They only need to search for it at that point.

My questions are more of feature requests, Javier suggestion would involve the page does not exists. and strongly integrated with identity framework
my current problem is blogger A created a post with id 1 '/edit/1'. Since @page "/edit/{postid?}" should be restricted to the creator. from the implementation of policy based authentication.
https://docs.microsoft.com/en-gb/aspnet/core/security/authorization/policies?view=aspnetcore-3.1
the IAuthorizationHandler does not seem to have access to request context nor request parameters.
So if so there should be a way to force the page to redirect to NotFound or return a NotFound result. when blogger B accessed 'edit/1'.

His suggestion applies to every user component navigation, not just a missing page request ... you can do anything you want with that approach, including any kind of authn/z logic.
Feature requests have to go to the product unit, so you'd need to open an issue there ...
https://github.com/dotnet/aspnetcore/issues
... but I don't recommend it because I think Javier is going to give you the same answer.
Your request for coverage on custom code is outside of the current doc scope. We recommend chatting with devs on the usual support channels for things like this.

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter: Blazor

Leave this open tho because it might be a good idea to call out the idea of a custom AuthorizeRouteView component to surface it to readers. After that tho, it's up the dev to build it on their own.

I can't seem to fit the case. example:

<Router AppAssembly="@typeof(Program).Assembly">
    <Found Context="routeData">
        <MyCustomWrappingComponent RouteData="@routeData">
            <AuthorizeView>
                <Authorized>
                    @if(CanEditBlogPost(context.User.Identity.Name, routeData["id"]) {

                    <MyCustomRedirectComponent />
                    } else {
                     <RouteView RouteData="@routeData" DefaultLayout="@typeof(MainLayout)" />
                    }
                </Authorized>
                <NotAuthorized>
                    <MyCustomRedirectComponent />
                </NotAuthorized>
            </AuthorizeView>
        </MyCustomWrappingComponent>
    </Found>
</Router>

where
`
protected override void OnInitialized()
{
NavigationManager.NavigateTo("MyNotFoundPage");
}
`
is the above practive good? calling NavigationManager.NavigateTo when OnInitialized is called?
because it might have loaded some of the data before redirecting. I'm not really sure. That is why i need a onbeforeinitialized or a filter attribute that calls navigateto before data is loaded.

Also users can just type 'mywebsite.com/edit/1' if the user is logged in then the client should do an ajax request for the data. if the data is not found, the user is redirected to MyNotFoundPage.
im not really sure how this is implemented with the blazor framework.

from razor pages you can create non-auth filters.
from angular they have client side guards, loads the data then checks the data, if so they can redirect or display an error message.

That's not anything like what the recommended approach would be via just using the provided options laid out in the Blazor Security topic. That's all done for you using the built-in bits. You would just establish a role claim and set up a policy and then check the policy with the [Authorize] attribute as the topic shows.
You'll have to inquire further on a support channel. I'm up to my 👂👂 in work 🏃😅. We just don't have the staffing to provide product support. Try those support channels that I listed.

When I get to a doc sweep right after 3.2 releases, I'll have more time to assess the overall presentation of concepts, and I hope to work further clarifications and examples into the topics, including perhaps a security sample app. Right now tho, I recommend sticking with the approaches in the topics ... authenticate the user, they have a role claim (or a few if needed), the components use policies and procedural logic to check their role claim(s) to permit certain actions. In Blazor WASM, checks need to be made client and server-side because anything client-side in a WASM app can be hacked.

problem with role claim is that the IAuthorizationHandler does not have access to the route parameters id=1 from /edit/1. So i can't check if the user is the owner of post 1.
from
https://docs.microsoft.com/en-us/aspnet/core/blazor/call-web-api?view=aspnetcore-3.1
and
https://github.com/dotnet/AspNetCore.Docs/blob/master/aspnetcore/blazor/common/samples/3.x/BlazorWebAssemblySample/Components/HTTPRequestTester.razor
Since authentication can't access route parameters, and non-auth filters are not supported, will below be a good alternative practice?

create a base class BlogPostBase with ajax inside oninitializedasync
redirecting when error is received.
all page that follows the the same authorization, lets say /edit/1 or /review/1 or /publish/1 should inherit by @inherit BlogPostBase

BlogPostBase.cs

public class BlogPostBase : ComponentBase {

   ...
protected override async Task OnInitializedAsync() {
    try{

        var res = CallHttpRequest("CheckUserAccessTo/GetPostContent", request["postId"]);
        //api should never return the content if not authorized to view
        ....
    }catch(Exception ex) // returned 401 or error
    {
        NavigationManager.NavigateTo("MyErrorPage");
    }

}

}

Edit.razor

@page "/edit/{id}"
@inherit PostBase

....

Publish.razor

@page "/publish/{id}"
@inherit PostBase

....

will the above alternative be a good practice? im not sure.

I'm going to close this because it was asked and answered by engineering and is a general support request. We don't have the staffing to provide general support. Converse with the community further based on Javier's response in the engineering issue ...

Stack Overflow
ASP.NET Core Slack
ASP.NET Gitter


net: LookupPort translates negative numbers into valid port numbers on Windows without error.

OS: Windows 7 Professional, SP1.
Go: go1.5.1 windows/amd64
The following program prints 65527 on Windows, but panics on Linux.
package main

import (
	"fmt"
	"net"
)

func main() {
	port, err := net.LookupPort("tcp", "-9")
	if err != nil {
		panic(err)
	}
	fmt.Println(port)
}

Also "-2000000000" returns 27648 and "-4294967295" or numerically smaller returns 1.

/cc @alexbrainman
I guess this is the newLookupPort path?
func newLookupPort(network, service string) (int, error) {
        acquireThread()
        defer releaseThread()
        var stype int32
        switch network {
        case "tcp4", "tcp6":
                stype = syscall.SOCK_STREAM
        case "udp4", "udp6":
                stype = syscall.SOCK_DGRAM
        }
        hints := syscall.AddrinfoW{
                Family:   syscall.AF_UNSPEC,
                Socktype: stype,
                Protocol: syscall.IPPROTO_IP,
        }
        var result *syscall.AddrinfoW
        e := syscall.GetAddrInfoW(nil, syscall.StringToUTF16Ptr(service), &hints, &result)
        if e != nil {
                return 0, &DNSError{Err: os.NewSyscallError("getaddrinfow", e).Error(), Name: network + "/" + service}
        }
        defer syscall.FreeAddrInfoW(result)
        if result == nil {
                return 0, &DNSError{Err: syscall.EINVAL.Error(), Name: network + "/" + service}
        }
        addr := unsafe.Pointer(result.Addr)
        switch result.Family {
        case syscall.AF_INET:
                a := (*syscall.RawSockaddrInet4)(addr)
                return int(syscall.Ntohs(a.Port)), nil
        case syscall.AF_INET6:
                a := (*syscall.RawSockaddrInet6)(addr)
                return int(syscall.Ntohs(a.Port)), nil
        }
        return 0, &DNSError{Err: syscall.EINVAL.Error(), Name: network + "/" + service}
}

Let me play with this a little bit.
  We also have TestLookupPort in net package that checks that net.LookupPort("tcp", "-1") fails. Can you run that test to see if it PASSes? You can add your values to the test to see what is going on. You can also debug this yourself (since it is broken for you). Just add fmt.Printf or similar everywhere in net as you descend into net.LookupPort call. Please let us know how you went. Thank you.
Alex

@alexbrainman This issue was fixed by https://github.com/golang/go/commit/b50b21d3e130cf19de99c2736d038b636dde75c3.


root in config

Hello.
Specifing a path with nesting in root -> twirp.json cause error.
Error: Cannot find module 'path\to\protos\node_modules\twirpscript\dist\compiler.js'
It's trying to find compiler.js in nested path, not in project's nodes_modules .

{
   "root": "my/protos",
   "dest": "my/api"
}

And it's windows case

Thanks for reporting @Blancduman I’ll take a look

@Blancduman Can you let me know if you still encounter this issue on v0.0.63 https://github.com/tatethurston/TwirpScript/releases/tag/v0.0.63


remove shade plugin as it does not allow overriding dependencies

shade plugin should not be used to compile libraries used by other maven projects ,
not shading the library allows excluding the library dependencies if required , for example if the library is using Jackson version <IP_ADDRESS>
another project using the library may exclude the library's Jackson dependency and use its own.

just saw that there is already an open ticket for this , IMHO the version deployed to maven central should be unshaded.


webpack does not include node_modules dependency!!

Do you want to request a feature or report a bug?
bug
What is the current behaviour?
When there is no node_modules folder in the current directory
when I build my typescript node project and run node dist/index.js
it throws all shorts of error for eg:
cannot find module 'graphql-import'
cannot find module 'express'
If the current behaviour is a bug, please provide the steps to reproduce.
my webpack.prod.js file:
const path = require("path")
const nodeExternals = require("webpack-node-externals")
const Dotenv = require("dotenv-webpack")

module.exports = {
  entry: "./src/server.ts",
  externals: [nodeExternals()],
  mode: "production",
  devtool: "inline-source-map",
  target: "node",
  resolve: {
    extensions: [".ts", ".js"],
    modules: [path.resolve("./node_modules"), path.resolve("./src")]
  },
  node: {
    __dirname: false
  },
  output: {
    filename: "index.js",
    path: path.resolve(__dirname, "dist")
  },
  module: {
    rules: [
      {
        test: /\.ts$/,
        use: "ts-loader",
        exclude: /node_modules/
      }
    ]
  },
  plugins: [
    new Dotenv({
      path: "./.env"
    })
  ]
}

my tsconfig.json file:
{
  "compileOnSave": true,
  "compilerOptions": {
    "baseUrl": "src",
    "outDir": "./dist/",
    "skipLibCheck": true,
    "sourceMap": true,
    "noImplicitAny": true,
    "module": "es6",
    "target": "es5",
    "allowJs": true,
    "moduleResolution": "node",
    "noUnusedLocals": false,
    "allowSyntheticDefaultImports": true,
    "resolveJsonModule": true,
    "esModuleInterop": true,
    "lib": ["es2016", "esnext.asynciterable"]
  },
  "exclude": ["node_modules"]
}

What is the expected behaviour?
expect to run my nodejs server smoothly and printing
🚀 Server is running on port: 4000
If this is a feature request, what is motivation or use case for changing the behaviour?
No its not a feature request.
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
Node version: 10.13.0 (i tried with 12.x.x too)
webpack: 4.39.2
webpack-cli: 3.3.7
typescript: 3.5.3

I encourage you to ask your question on StackOverflow instead. We try to use the issue tracker for bug reports and feature requests, as well as some discussions, but this appears to be a very localized question.
I’m closing for these reasons.

Please help I just stuck for more than 2 days googling and stack overflowing

@suhailbinnisar please create reproducible test repo

Reproducible test repo
https://github.com/suhailbinnisar/test
steps:

clone repo & yarn install
yarn build:dev (should run server on 4000)
yarn build:prod && node dist/index.js (should run server on 4000)
rm -rf node_modules && node dist/index.js
(cannot find module 'graphql-import'')

same goes when I deploy my code (/dist only) to cloud hosting.
to get it working i have to deploy the whole repo including /src folder otherwise it throws,
cannot find module root.graphql

@suhailbinnisar Can't find yarn build:dev command

sorry for that its actually yarn start

It is expected, you still need node_modules when you use target: node

so what can I do so that webpack will bundle all the dependencies to index.js file

@suhailbinnisar you need change target to web, but you don't need it, don't bundle server code it is unnecessary

so what's the solution to this problem ??

Do not bundle code for node and bundle code only for browser

Still not working, what actually I want to achieve is, I just want to push/upload dist folder to my cloud server, and then it should work on need to manually yarn install each time and upload all env files, graphql files etc. is there any way to do so ?? if not feel free to close this issue otherwise please help

Just do not bundle server code and update server code and node_modules to cloud

if it does not impact the performance of my code you can close this issue. thanks for your help

It is not impact the performance

@evilebottnawi  So what about when using my code in a monorepo, I need to upload my entire repo just to execute my nodejs-api code, any workaround for this case?


Fix slider scripts

Changing the artifactId from gaffer-slider to slider has broken the 'quick start' scripts

Merged into develop.


voyager unit test

Unit test for voyager integration. It tests all voyager artifacts (generated yaml files and kubernetes objects that are created at runtime by the voyager).

Voyager unit test code has been merged into branch issue_293. It can be submitted with voyager change. See pull request https://github.com/oracle/weblogic-kubernetes-operator/pull/339


Browser compatiblity

Is it compatible with low level browsers too?

If you mean with old versions of browsers, then no,
Any SW library requires ServiceWorker. You can look at a compatibility-table here (in the end of the page) https://developer.mozilla.org/ru/docs/Web/API/Service_Worker_API to find which versions of browsers support ServiceWorker.

@NekR is right. Another take on service worker support is https://jakearchibald.github.io/isserviceworkerready/ which also includes general browser vendor opinions of SW.
Worth noting that service workers can be treated as a progressive enhancement (i.e. your page will work without service workers, it just won't work offline or make use of the cache.


how to get chr20.PEI.Enhancer.data.test.txt

Hi,how to get the chr20.PEI.Enhancer.data.test.txt,given the loop bedPE file,
since i just want to extract the EPC info from the loop file

Hi，this article (https://www.ncbi.nlm.nih.gov/pmc/articles/PMC8397779/) may be help you.


Don't Merge! PR real time test for webhook case

OPS Build status updates of commit 9c8c889:
:clock10: Full build: the average full build time is 6 min(s) 8 sec(s), this is based on the last 216 build(s) for this repository.
:warning: This is a FULL build due to the 'pr-webhookbackup' branch being a new branch in Open Publishing service and is still building. No action is required on your part. For more detailed info, please click here.

OPS Build status updates of commit 9c8c889:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Sync words array changes

Sync changes with Heroku version

Issue has been fixed in https://github.com/SecondThundeR/secondthunder-js-bot/commit/3d4f9349df757c3c15fa84583048833bca569a08


Feature: add Signal K JSON over serial support in admin UI

Add the ability to add providers that receive line oriented Signal K delta JSON over serial.

I have not been able to test this, as I don't have a handy test setup.

@joabakk do you a setup that you could use to test this?

Not currently, but it’s not very complicated code?

Not complicated, but nicer to verify that it works than merge blindly.

I'm trying to build the admin-ui to test this out, but I get an error not related to this PR when I run npm run prepublishOnly:
Version: webpack 3.12.0
Time: 2125ms
                           Asset       Size  Chunks             Chunk Names
                 index.bundle.js    7.97 kB       0  [emitted]  index
             index.bundle.js.map    69.4 kB       0  [emitted]  index
                      index.html  520 bytes          [emitted]
             img/logo-symbol.png    7.18 kB          [emitted]
                 img/favicon.ico    15.1 kB          [emitted]
img/signal-k-logo-image-text.svg    10.8 kB          [emitted]
[./src/index.js] ./src/index.js 260 bytes {0} [built] [failed] [1 error]
   [0] multi ./src/index.js 28 bytes {0} [built]

ERROR in ./src/index.js
Module parse failed: Unexpected token (45:8)
You may need an appropriate loader to handle this file type.
|     if (action.type === 'RECEIVE_PLUGIN_LIST') {
|       return {
|         ...state,
|         plugins: action.data
|       }
 @ multi ./src/index.js
Child html-webpack-plugin for "index.html":
     1 asset
    [./node_modules/html-webpack-plugin/lib/loader.js!./public_src/index.html] ./node_modules/html-webpack-plugin/lib/loader.js!./public_src/index.html 488 bytes {0} [built]```

What version of Node?
WOMM with a fresh clone, Node v8.12.0 npm install && npm run prepublishOnly.


Cuda Linking Problem - Cmake generates invalid linker commands

System information

Operating system: Ubuntu 14.04
Compiler version: GCC 4.8.4
CMake version: 3.11.2
CMake arguments: No args
Relevant libraries/versions : CUDA 8.0, CuDNN v6.0.21

At first cmake detected cuda like this:
-- CUDA root directory : /usr/local/cuda-8.0
-- CUDA library : /usr/lib/x86_64-linux-gnu/libcuda.so
-- CUDA NVRTC library : /usr/local/cuda-8.0/lib64/libnvrtc.so
-- CUDA runtime library: /usr/local/cuda-8.0/lib64/libcudart.so

As it was pointed it out by @pjh5 , the cuda library libcuda.so was in a different directory than that of other cuda libraries. This was due to the fact that I installed cuda toolkit from a .deb file. To resolve this you will need to re-install cuda again only from a .run file. which will install libcuda.so in its proper location that is /usr/local/cuda-8.0/lib64/stubs/libcuda.so. yet to let cmake detect the library in the stubs folder, I had to rename/hide the one in the x86_64-linux-gnu directory.
The problem is simply that libcaffe2.so can't link against cuda libraries because cmake auto generated the linker command (found in link.txt for each target) and it didn't specify the full paths of the cuda libraries. Please note that this is not an exporting issue regarding those libraries.
Finally, My current cmake summary is as follows:
CMake summary output
-- The CXX compiler identification is GNU 4.8.4
-- The C compiler identification is GNU 4.8.4
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Performing Test CAFFE2_LONG_IS_INT32_OR_64
-- Performing Test CAFFE2_LONG_IS_INT32_OR_64 - Success
-- Does not need to define long separately.
-- Performing Test CAFFE2_EXCEPTION_PTR_SUPPORTED
-- Performing Test CAFFE2_EXCEPTION_PTR_SUPPORTED - Success
-- std::exception_ptr is supported.
-- Performing Test CAFFE2_IS_NUMA_AVAILABLE
-- Performing Test CAFFE2_IS_NUMA_AVAILABLE - Failed
-- NUMA is not available
-- Performing Test CAFFE2_NEED_TO_TURN_OFF_DEPRECATION_WARNING
-- Performing Test CAFFE2_NEED_TO_TURN_OFF_DEPRECATION_WARNING - Failed
-- Turning off deprecation warning due to glog.
-- Performing Test CAFFE2_COMPILER_SUPPORTS_AVX2_EXTENSIONS
-- Performing Test CAFFE2_COMPILER_SUPPORTS_AVX2_EXTENSIONS - Success
-- Current compiler supports avx2 extention. Will build perfkernels.
-- Build type not set - defaulting to Release
-- Looking for pthread.h
-- Looking for pthread.h - found
-- Looking for pthread_create
-- Looking for pthread_create - not found
-- Looking for pthread_create in pthreads
-- Looking for pthread_create in pthreads - not found
-- Looking for pthread_create in pthread
-- Looking for pthread_create in pthread - found
-- Found Threads: TRUE
-- Caffe2: Found protobuf with new-style protobuf targets.
-- Caffe2 protobuf include directory: /usr/local/include
-- Found Git: /usr/bin/git (found version "1.9.1")
-- The BLAS backend of choice:Eigen
-- Could NOT find NNPACK (missing: NNPACK_INCLUDE_DIR NNPACK_LIBRARY PTHREADPOOL_LIBRARY CPUINFO_LIBRARY)
-- Brace yourself, we are building NNPACK
-- The ASM compiler identification is GNU
-- Found assembler: /usr/bin/cc
-- Found PythonInterp: /usr/bin/python (found version "2.7.6")
-- Check if compiler accepts -pthread
-- Check if compiler accepts -pthread - yes
-- Caffe2: Cannot find gflags automatically. Using legacy find.
-- Found gflags: /usr/include
-- Caffe2: Found gflags  (include: /usr/include, library: /usr/lib/x86_64-linux-gnu/libgflags.so)
-- Caffe2: Cannot find glog automatically. Using legacy find.
-- Found glog: /usr/include
-- Caffe2: Found glog (include: /usr/include, library: /usr/lib/x86_64-linux-gnu/libglog.so)
-- git Version: v0.0.0
-- Version: 0.0.0
-- Performing Test HAVE_CXX_FLAG_STD_CXX11
-- Performing Test HAVE_CXX_FLAG_STD_CXX11 - Success
-- Performing Test HAVE_CXX_FLAG_WALL
-- Performing Test HAVE_CXX_FLAG_WALL - Success
-- Performing Test HAVE_CXX_FLAG_WEXTRA
-- Performing Test HAVE_CXX_FLAG_WEXTRA - Success
-- Performing Test HAVE_CXX_FLAG_WSHADOW
-- Performing Test HAVE_CXX_FLAG_WSHADOW - Success
-- Performing Test HAVE_CXX_FLAG_WERROR
-- Performing Test HAVE_CXX_FLAG_WERROR - Success
-- Performing Test HAVE_CXX_FLAG_PEDANTIC
-- Performing Test HAVE_CXX_FLAG_PEDANTIC - Success
-- Performing Test HAVE_CXX_FLAG_PEDANTIC_ERRORS
-- Performing Test HAVE_CXX_FLAG_PEDANTIC_ERRORS - Success
-- Performing Test HAVE_CXX_FLAG_WSHORTEN_64_TO_32
-- Performing Test HAVE_CXX_FLAG_WSHORTEN_64_TO_32 - Failed
-- Performing Test HAVE_CXX_FLAG_WFLOAT_EQUAL
-- Performing Test HAVE_CXX_FLAG_WFLOAT_EQUAL - Success
-- Performing Test HAVE_CXX_FLAG_FSTRICT_ALIASING
-- Performing Test HAVE_CXX_FLAG_FSTRICT_ALIASING - Success
-- Performing Test HAVE_CXX_FLAG_WZERO_AS_NULL_POINTER_CONSTANT
-- Performing Test HAVE_CXX_FLAG_WZERO_AS_NULL_POINTER_CONSTANT - Success
-- Performing Test HAVE_CXX_FLAG_WSTRICT_ALIASING
-- Performing Test HAVE_CXX_FLAG_WSTRICT_ALIASING - Success
-- Performing Test HAVE_CXX_FLAG_WD654
-- Performing Test HAVE_CXX_FLAG_WD654 - Failed
-- Performing Test HAVE_CXX_FLAG_WTHREAD_SAFETY
-- Performing Test HAVE_CXX_FLAG_WTHREAD_SAFETY - Failed
-- Performing Test HAVE_CXX_FLAG_COVERAGE
-- Performing Test HAVE_CXX_FLAG_COVERAGE - Success
-- Performing Test HAVE_STD_REGEX
-- Performing Test HAVE_STD_REGEX
-- Performing Test HAVE_STD_REGEX -- compiled but failed to run
-- Performing Test HAVE_GNU_POSIX_REGEX
-- Performing Test HAVE_GNU_POSIX_REGEX
-- Performing Test HAVE_GNU_POSIX_REGEX -- failed to compile
-- Performing Test HAVE_POSIX_REGEX
-- Performing Test HAVE_POSIX_REGEX
-- Performing Test HAVE_POSIX_REGEX -- success
-- Performing Test HAVE_STEADY_CLOCK
-- Performing Test HAVE_STEADY_CLOCK
-- Performing Test HAVE_STEADY_CLOCK -- success
-- Found LMDB: /usr/include
-- Found lmdb    (include: /usr/include, library: /usr/lib/x86_64-linux-gnu/liblmdb.so)
-- Found LevelDB: /usr/include
-- Found LevelDB (include: /usr/include, library: /usr/lib/x86_64-linux-gnu/libleveldb.so)
-- Found Snappy: /usr/include
-- Found Snappy  (include: /usr/include, library: /usr/lib/libsnappy.so)
-- Could NOT find Numa (missing: Numa_INCLUDE_DIR Numa_LIBRARIES)
CMake Warning at cmake/Dependencies.cmake:205 (message):
  Not compiling with NUMA.  Suppress this warning with -DUSE_NUMA=OFF
Call Stack (most recent call first):
  CMakeLists.txt:101 (include)

-- Found CUDA: /usr/local/cuda-8.0 (found suitable exact version "8.0")
-- OpenCV found (/usr/local/share/OpenCV)
-- Found system Eigen at /usr/local/include/eigen3
-- Found PythonInterp: /usr/bin/python (found suitable version "2.7.6", minimum required is "2.7")
-- Found PythonLibs: /usr/lib/x86_64-linux-gnu/libpython2.7.so (found suitable version "2.7.6", minimum required is "2.7")
-- Found NumPy: /usr/local/lib/python2.7/dist-packages/numpy/core/include (found version "1.14.0")
-- NumPy ver. 1.14.0 found (include: /usr/local/lib/python2.7/dist-packages/numpy/core/include)
-- Could NOT find pybind11 (missing: pybind11_INCLUDE_DIR)
-- Found MPI_C: /usr/lib/libmpi.so (found version "2.1")
-- Found MPI_CXX: /usr/lib/libmpi_cxx.so (found version "2.1")
-- Found MPI: TRUE (found version "2.1")
-- MPI support found
-- MPI compile flags: -pthread
-- MPI include path: /usr/lib/openmpi/include/openmpi/usr/lib/openmpi/include
-- MPI LINK flags path: -L/usr/lib/openmpi/lib -pthread
-- MPI libraries: /usr/lib/libmpi_cxx.so/usr/lib/libmpi.so/usr/lib/x86_64-linux-gnu/libdl.so/usr/lib/x86_64-linux-gnu/libhwloc.so
CMake Warning at cmake/Dependencies.cmake:324 (message):
  OpenMPI found, but it is not built with CUDA support.
Call Stack (most recent call first):
  CMakeLists.txt:101 (include)

-- Found CUDA: /usr/local/cuda-8.0 (found suitable version "8.0", minimum required is "7.0")
-- Found CUDNN: /usr/local/cuda-8.0/include
-- Caffe2: CUDA detected: 8.0
-- Found cuDNN: v6.0.21  (include: /usr/local/cuda-8.0/include, library: /usr/local/cuda-8.0/lib64/libcudnn.so)
-- Automatic GPU detection returned 6.1.
-- Added CUDA NVCC flags for: sm_61
-- Could NOT find NCCL (missing: NCCL_INCLUDE_DIRS NCCL_LIBRARIES)
-- Could NOT find CUB (missing: CUB_INCLUDE_DIR)
-- Could NOT find Gloo (missing: Gloo_INCLUDE_DIR Gloo_LIBRARY)
-- MPI include path: /usr/lib/openmpi/include/openmpi/usr/lib/openmpi/include
-- MPI libraries: /usr/lib/libmpi_cxx.so/usr/lib/libmpi.so/usr/lib/x86_64-linux-gnu/libdl.so/usr/lib/x86_64-linux-gnu/libhwloc.so
-- CUDA detected: 8.0
-- Found libcuda: /usr/local/cuda-8.0/lib64/stubs/libcuda.so
-- Found libnvrtc: /usr/local/cuda-8.0/lib64/libnvrtc.so
-- Found nccl: /home/eslam/caffe2/third_party/nccl/build/include
CMake Warning at cmake/Dependencies.cmake:457 (message):
  mobile opengl is only used in android or ios builds.
Call Stack (most recent call first):
  CMakeLists.txt:101 (include)

CMake Warning at cmake/Dependencies.cmake:533 (message):
  Metal is only used in ios builds.
Call Stack (most recent call first):
  CMakeLists.txt:101 (include)

-- Could NOT find pybind11 (missing: pybind11_INCLUDE_DIR)
-- GCC 4.8.4: Adding gcc and gcc_s libs to link line
-- Include NCCL operators
-- Including image processing operators
-- Excluding video processing operators due to no opencv
-- Excluding mkl operators as we are not using mkl
-- Include Observer library
-- Using lib/python2.7/dist-packages as python relative installation path
-- Automatically generating missing __init__.py files.
 Kindly, Please advise!

@BerserkerTiger thanks for the nice write-up. I will look into this and see if I can make our cmake more robust here.

I don't know if this is somehow related but if not I'll open a new issue.
I followed the instructions on caffe2 webpage and created a clean anaconda python2 environment where I did  conda install -c caffe2 caffe2-cuda9.0-cudnn7.
Everything went smoothly, the packages & deps are installed. I have CUDA9 and CUDNN7 in my LD_LIBRARY_PATH already. But when importing caffe2, I get a segfault because it can not find libcudart.so.8 . Going into the anaconda package and doing ldd reveals that some shared libraries are weirdly linked to libcudart.so.8 instead of .so.9. If you download the tarball that gets pulled when you run the above conda install command and extract it, you can see that apart cudart everything are linked to cuda9 SO's.

@ozancaglayan sorry I didn't see this earlier. The packages right now look linked correctly to me, but maybe we had a temporary error a while back. Do you still have this issue?

I had to build from source :)


read api and ssh endpoints from .lagoon.yml file

When working on a project that uses other-than default endpoints, it would be good for the CLI to read these from the project's own .lagoon.yml

Possible to do this, but would involve a lot of extra work to make it happen.
Is it possible to utilise the following that is in lagoon-cli v0.8.+ ?
export LAGOONCONFIG=/path/to/mycluster.yaml

where /path/to/mycluster.yaml could be
current: mycluster
default: mycluster
lagoons:
  mycluster:
    graphql: https://api.lagoon.amazeeio.cloud/graphql
    hostname: ssh.lagoon.amazeeio.cloud
    kibana: https://logs-db-ui-lagoon-master.ch.amazee.io/
    port: 32222
    ui: https://ui-lagoon-master.ch.amazee.io

Having that envvar defined will mean any commands run will execute against the cluster defined in the config that is being given.

we're looking to place less reliance on the api/ssh documented in the .lagoon.yml - closing.


Persistent Forms in a Rich Text Field

Hi @cortexrd
Data is not being preserved in a rich text field when persistent forms is turned on. The paragraph text is saved so I know persistent forms is working.
Craig

Fixed in 0.17.2


Getting 503 error code in fluentd for retryable_response_codes

Describe the bug
We are sending data from fluentd to logstash with http.We are using fluentd version fluentd-1.16.2. Previously it is working fine but after restart the fluentd pod we start getting the below errors
Status code 503 is going to be removed from default `retryable_response_codes` from fluentd v2. Please add it by yourself if you wish
/fluentd/vendor/bundle/ruby/3.1.0/gems/fluentd-1.16.2/lib/fluent/plugin/out_http.rb:189:in `initialize': no implicit conversion of nil into String (TypeError)

Could someone please help to understand the issue and how to resolve that ?
To Reproduce
We are using the below configuration provided
Expected behavior
There should not be any error as logstash pod is up and running.
Your Environment
- Fluentd version:1.16.2
- TD Agent version:
- Operating system: Debian GNU/Linux
- Kernel version: 11

Your Configuration
<source>
      @type monitor_agent
      bind <IP_ADDRESS>
      port 24220
    </source>

    <system>
      workers "#{ENV['FLUENTD_WORKER']}"
    </system>

    <worker 0>
      <source>
        @type tail
        #@id in_tail_fluentd_logs
        # @label @fluentdlogs
        path /var/log/containers/*.log
        pos_file /var/log/fluentd-containers.log.pos
        tag kubernetes.*
        read_from_head true
      <parse>
        #@type json
        #time_format %Y-%m-%dT%H:%M:%S.%NZ
        @type multi_format
        <pattern>
          format json
          time_key time
          keep_time_key true
        #time_format %Y-%m-%dT%H:%M:%S.%L %Z
          time_format %Y-%m-%dT%H:%M:%S.%NZ
        </pattern>
        <pattern>
          format none
          time_key time
          keep_time_key true
        #time_format %Y-%m-%dT%H:%M:%S.%L %Z
          time_format %Y-%m-%dT%H:%M:%S.%NZ
        </pattern>
      </parse>
      </source>
    </worker>

    <filter kubernetes.**>
      @type kubernetes_metadata
      @id filter_kube_metadata
      cache_size 20000
      cache_ttl 7200
      watch false
    </filter>

    <match **>
      @type http
      time_as_integer true
      http_method post
      tls_client_cert_path "/fluentd/etc/ssl/client.crt"
      tls_private_key_path "/fluentd/etc/ssl/client.key"
      tls_ca_cert_path "/fluentd/etc/ssl/ca.crt"
      tls_verify_mode peer
      tls_version TLSv1_2
      <buffer>
        @type file
        path /var/log/fluentd_application/kubernetes.internal.application.buffer
        chunk_limit_size 512k
        flush_interval 5s
        total_limit_size 4GB
        flush_thread_count "#{ENV['FLUENTD_THREADS']}"
        overflow_action drop_oldest_chunk
        retry_forever true
        retry_wait 5s
        retry_max_interval 5m
        retry_exponential_backoff_base 5
      </buffer>
      endpoint "https://#{ENV['LOGSTASH_HOST']}:#{ENV['LOGSTASH_PORT']}"
    </match>

Your Error Log
Status code 503 is going to be removed from default `retryable_response_codes` from fluentd v2. Please add it by yourself if you wish
/fluentd/vendor/bundle/ruby/3.1.0/gems/fluentd-1.16.2/lib/fluent/plugin/out_http.rb:189:in `initialize': no implicit conversion of nil into String (TypeError)

Additional context
No response

Status code 503 is going to be removed from default retryable_response_codes from fluentd v2. Please add it by yourself if you wish

You need to add retryable_response_codes [503] into your <match> section to suppress this warning.

/fluentd/vendor/bundle/ruby/3.1.0/gems/fluentd-1.16.2/lib/fluent/plugin/out_http.rb:189:in `initialize': no implicit conversion of nil into String (TypeError)

Your TLS client certification file seems empty or broken.
Please check it.
https://github.com/fluent/fluentd/blob/d5685ada81ac89a35a79965f1e94bbe5952a5d3a/lib/fluent/plugin/out_http.rb#L184-L189


lomo fails to build on Darwin

Describe the bug
lomo fails to build on Darwin.
Steps To Reproduce

nix-build -A lomo

Expected behavior
Build completes.
Additional context
Failure log:
DEPRECATED: Libs must be qualified, change fipp => fipp/fipp (/private/var/folders/1s/gpt7p1x559sb7qjb_4bhq7zc75yv8y/T/nix-build-lumo-1.10.1.drv-0/lumo/deps.edn)
DEPRECATED: Libs must be qualified, change fipp => fipp/fipp (deps.edn)
WARNING: Implicit use of clojure.main with options is deprecated, use -M
java.io.FileInputStream
#'user/write-transit-json
#'user/process-caches
Building production bundle...
Circular dependency: src/js/cljs.js -> src/js/repl.js -> src/js/cljs.js
Circular dependency: src/js/cli.js -> src/js/cljs.js -> src/js/repl.js -> src/js/socketRepl.js -> src/js/cli.js
Circular dependency: src/js/repl.js -> src/js/socketRepl.js -> src/js/repl.js
Circular dependency: src/js/cljs.js -> src/js/repl.js -> src/js/socketRepl.js -> src/js/cljs.js
----> framework => node
----> using provided bundle target/bundle.min.js since noBundle is true
----> monkey patch /private/var/folders/1s/gpt7p1x559sb7qjb_4bhq7zc75yv8y/T/nix-build-lumo-1.10.1.drv-0/lumo/tmp/node/11.13.0/node.gyp
----> monkey patch /private/var/folders/1s/gpt7p1x559sb7qjb_4bhq7zc75yv8y/T/nix-build-lumo-1.10.1.drv-0/lumo/tmp/node/11.13.0/src/node.cc
----> monkey patching configure file
----> monkey patch /private/var/folders/1s/gpt7p1x559sb7qjb_4bhq7zc75yv8y/T/nix-build-lumo-1.10.1.drv-0/lumo/tmp/node/11.13.0/configure.py
----> monkey patch /private/var/folders/1s/gpt7p1x559sb7qjb_4bhq7zc75yv8y/T/nix-build-lumo-1.10.1.drv-0/lumo/tmp/node/11.13.0/lib/internal/bootstrap/node.js
----> using the after 0.12.5 method of ignoring flags.
----> using the after 5.5.0 method of ignoring flags.
....> ERROR: Failed to find a way to patch node.cc to ignoreFlags
----> startLine = -1 | endLine = -2
----> patched node.cc
----> make
ERROR: Did not find a new enough assembler, install one or build with
       --openssl-no-asm.
       Please refer to BUILDING.md
Missing or stale config.gypi, please run ./configure
make: *** [Makefile:134: config.gypi] Error 1
....> ERROR: The release executable has not been generated. This indicates a failure in the build process. There is likely additional information above.
builder for '/nix/store/9v3zzl8asnbp11pp0yl3a5n1z4zq522a-lumo-1.10.1.drv' failed with exit code 1
error: build of '/nix/store/9v3zzl8asnbp11pp0yl3a5n1z4zq522a-lumo-1.10.1.drv' failed

Notify maintainers
@hlolli
Metadata
 - system: `"x86_64-darwin"`
 - host os: `Darwin 21.4.0, macOS 10.16`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.6`
 - channels(abutler): `"home-manager, nixpkgs-22.05pre360843.3eb07eeafb5"`
 - nixpkgs: `/Users/abutler/.nix-defexpr/channels/nixpkgs`
 - ```

lumo has been removed from nixpkgs in #170278.


Cater for "stitch" bank in 160-5370/160-5371 ROMs

It turns out that the PB2 line in PORT1 is actually used. When it's set, the U2250 PAL stitches together the 3 "lost" 8k sections of the two ROMs to create a 5th 24k bank.
The loader should be able to cope with this by looking for a pair of 5370/5371s and stitching up another overlay memory block from the right pieces.

Looks like backing the memory blocks with FileBytes is the right approach. This allows accessing the full contents of the file after the initial memory blocks are created.

This should be covered now.


Enqueue Job from Adonis Command

I'm trying to enqueue a Kue Job from a Command, but command doesn't terminate and it keeps running.
Do I have to do something inside the command to close the process from the job?

The job command should automatically terminate once the handle function returns.
Can you share the code that is stuck?

I'm closing this for now since I haven't heard back. But please re-open the issue you still have trouble.

Late to the party, but i got the same behaviour:
My command handle looks like this:
    const data = {
      test: "something"
    };
    const priority = "medium";
    const attempts = 5;
    const remove = true;
    await kue.dispatch(Job.key, data, priority, attempts, remove).result;
    this.info("Job enqueue");
  }```

It gets all the way to the "Job enqueue" info, but never exits.

When i remove the kue part then the command exits fine.


Faulty TcpServer and PortMap lifecycle

Describe the bug
The TcpServer is a singleton instance for the entire application, while it could have a "per-mdoule" lifecycle and the PortMap never removes a port registration, even if the listener shuts down or restarts on a different port. We must however validate if this still allows to register the same protocol on the same port from different modules and if this is a realistic requirement.
To Reproduce
Steps to reproduce the behavior:

Configure one protocol on PortA and another on PortB
Start listener connection
Switch ports between protocols
Restart module/connection/...

Expected behavior
Since each connection is no longer active on the port, it should be available for other protocols. Instead this will cause an exception telling me about the original port assignment.
Additional context
Unlike #89 this problem is more theoretical than practical and thereby solution is not urgent.

Problem is still theoretical and never occured in production. If Ports change, just restart MORYX.


Fix Search Bar

User should be able to search by neighborhood (to center map and find jobs) or by role.

Search bar works. No auto-correct for spelling errors, though.


UI to display ingestion status

After https://github.com/refstudio/refstudio/issues/113 we can implement a new UI to show ingestion status.
The proposal is to use a footer like VS Code with a spinner while the ingestion is running.

This was already implemented by https://github.com/refstudio/refstudio/pull/192


Select cells in subsections is not working properly

After opening any section in the tableview, selecting a cell in the subsection is not responsive at first time. Then, all cells are mixed up and they all point at wrong indexpath.row
To reproduce:
Add the didDeselectRowAtIndexPath delegate function in the firstviewcontroller.m file in the example provided

(void)tableView:(UITableView *)tableView didDeselectRowAtIndexPath:(NSIndexPath *)indexPath {
NSLog(@"%d", indexPath.row);
}

Any solution to that issue?!

Hi there,
So I tested the Example just now and didSelect and didDeselect work as expected.
 So this explains the "not responsive at first time" - you had no cells that were selected.
Let me know.