Rework the template

The issue templates include checkboxes to guide the user to provide important information for us in understanding and evaluating the ticket. Such information is the impact that it has on the user's workflow, the likelihood of it occuring, and some more info about whether Visit crashed or the user got the wrong results (for a bug). Some info, such as if Visit crashed, is important for us to know from the user, whereas for other information we usually have a pretty good idea ourselves.
While the checkboxes may be a nice way to get important information, they do not correlate with the default labels. For example, if a submitter clicks "High Impact", the default label will still be "Medium Impact". This can be confusing. I believe we added the default labels to make our job a little easier: Because most labels will be "medium", it's faster to automatically add them.
I see two elements to discuss here:
(1) What information do we want to get from the user and how should we get it?
(2) Do the default labels confuse the ticket creation?

Still want "wrong results" and "visit crashed". Don't use checkboxes. Just have text that explicitly asks the user to indicate if it crashed or if it got wrong results. Can put these down at the bottom near the os and version, etc.

So, I started re-thinking my remarks in yesterday's project meeting about this. I took a look at the bug template and started thinking there is another way to go...more checkboxes of a slightly different sort. Some of which we might need to maintain over time. So, another option is to remove the labeling questions and just use GitHub labels (so only developers can make those assertions and that kinda sorta makes sense anyways since only developers are likely to be able to have the knowledge required to decide impact, likelihood, etc.) and leave the reproducability section asking for either a) steps to reproduce with VisIt test data or b) attaching session file and data and then some check boxes...

[ ] Did VisIt crash?
[ ] Did VisIt produce incorrect results?
[ ] Is this a performance issue?
[ ] Is this a useability issue?
[ ] Were you running client/server?
[ ] Were you running over X windows or VNC?
What version of VisIt is running on the desktop/client?

[ ] Earlier than 2.10
[ ] 2.10
[ ] 2.11
[ ] 2.12
[ ] 2.13
[ ] 3.0
[ ] 3.1

What platform type is this?

[ ] Linux/Unix

[ ] Redhat
[ ] Ubuntu
[ ] Centos
[ ] Fedora

[ ] OSX/macOS

[ ] 10.10 (Yosemite)
[ ] 10.11 (El Capitan)
[ ] 10.12 (Sierra)
[ ] 10.13 (High Sierra)
[ ] 10.14 (Mojave)

[ ] Windows

[ ] 7
[ ] 8
[ ] 10

What programs are you a member of?

[ ] ASC
[ ] ECP
[ ] AWE
[ ] CEA


Compiling on Jetson Xavier with NVIDIA pytorch and cuda 10.2 fails; missing libcudart

I am trying to install TRTorch on the Jetson Xavier with Jetpack 4.6.
Both the torch and cuda installations work fine on their own.
I have commented out the "http_archive" lines and uncommented the "new_local_repository" lines in the WORKSPACE file, and I have changed the path to the cuda installation.
However, when trying to compile with sudo bazel build //:libtrtorch --platforms //toolchains:jetpack_4.6 I get the following error.
INFO: Build options --cxxopt, --define, and --linkopt have changed, discarding analysis cache. INFO: Analyzed target //:libtrtorch (0 packages loaded, 2813 targets configured). INFO: Found 1 target... INFO: From Compiling cpp/bin/trtorchc/main.cpp: cpp/bin/trtorchc/main.cpp: In function 'int main(int, char**)': cpp/bin/trtorchc/main.cpp:340:18: warning: catching polymorphic type 'class args::Help' by value [-Wcatch-value=]    } catch (args::Help) {                   ^~~~ cpp/bin/trtorchc/main.cpp:343:29: warning: catching polymorphic type 'class args::ParseError' by value [-Wcatch-value=]    } catch (args::ParseError e) {                              ^ ERROR: /home/quinn29/.cache/bazel/_bazel_root/9161e410ccf95eb99b9534360aa0ca3e/external/cuda/BUILD.bazel:10:11: SolibSymlink<EMAIL_ADDRESS>failed: missing input file 'external/cuda/targets/x86_64-linux/lib/libcudart.so', owner: '@cuda//:targets/x86_64-linux/lib/libcudart.so' Target //:libtrtorch failed to build    Use --verbose_failures to see the command lines of failed build steps. ERROR: /home/quinn29/.cache/bazel/_bazel_root/9161e410ccf95eb99b9534360aa0ca3e/external/cuda/BUILD.bazel:10:11 SolibSymlink<EMAIL_ADDRESS>failed: 1 input file(s) do not exist INFO: Elapsed time: 224.547s, Critical Path: 42.39s INFO: 19 processes: 4 internal, 15 processwrapper-sandbox. FAILED: Build did NOT complete successfully
I have tried all other combinations of parameters I have seen, but I keep getting the same error.

Have you tried clearing the bazel cache?


Go to File command should only open files from current workspace

VSCode Version: 1.20.1
OS Version: macOS High Sierra 10.13.2

Steps to Reproduce:

Setup two workspaces preferably clones of same git repo. Say there is a file called "foo.js" in both workspaces
open -a '''Visual Studio Code''' $HOME/first.code-workspace
Go To File and search for foo and it will autocomplete to foo.js in first workspace
Close Visual studio code
open -a '''Visual Studio Code''' $HOME/second.code-workspace
Notice that when you search for foo it offers foo.js from another workspace.

Ideally it should only offer files from the current workspace.
Does this issue occur when all extensions are disabled?: Yes

@ptrajkumar weird, can you show a video of this issue in action? Unless foo.js is not in both workspaces, I would not expect foo.js to show up in second workspace.

foo.js is in both workspaces. The problem is once you open foo.js in first workspace and close that workspace and open the second workspace.  Doing quickopen and typing foo.js completes it to foo.js in first workspace and not the second workspace.

@ptrajkumar and to be clear foo.js is in 2 different folders and you still get the folder from the other workspace? Can you please attach screenshots or a recording what you are doing.


add WUSTL and MGI branding to site

from #151

@malachig, @ahwagner, was there anything I needed to do to make sure this branding stuff wouldn't piss any admin off?

Can you post a screenshot for reference?

This looks fine to me.


[W5.8b][T11-A4]Eka Buyung Lienadi

@yungyung04
Some comments added. Please close the PR after reading comments.


Release mouse button presses on window focus change.

Prevent mouse button from been stuck when Alt-Tabbing.

Superseded by https://github.com/godotengine/godot/pull/92424


LLaVa support

Feature request
Hello! It would be awesome to have LLaVa support (upload an image to the API and have it embed it via CLIP etc)
https://github.com/haotian-liu/LLaVA
text-generation-webui already has this built in, but we need to make use of the true model parallelization for faster inference of this repository.
Thank you very much! :)
Motivation
Current LLaVa inference code does not allow for true model parallelization. We (as many others) need very fast inference with LLaVa models.
Your contribution
I am not experienced enough to do any of the foundational work but I would love to help when the foundation is there, porting code from the LLaVa repository to here.

Hi,
The first step would likely be implementing it like here: https://github.com/huggingface/transformers/issues/22848 within transformers itself.
Then, this library has custom modeling code to speed things up, but it's very much hands-on work.
If you want to tackle it, you could look at llama for instance since the models should be relatively similar.
The 2 main things to consider are:

Sharding: TP parallelism is important, that means replacing some layers with their sharded counterpart.
Flash attention: This means changing everything around from transformers to remove padding and use specifically the flash attention kernels.

Also if I read correctly, this is a multimodal model. This is probably quite different from the current models supported by this repo which are highly targeted towards pure text generation. I am not sure if this repo will be a good fit (there are many assumption this repo relies on which might not be true for llava)


weird spacing in multi line access to nested tuples

i know that due to #1063 one.0.1 is formatted as one.0 .1, so that the 0.1 doesn't get interpreted as a float literal.
but if the next lines are long enough that the 0.1 gets put onto the next line, then that should not be affected by the bug in #1063, and could be formatted like expected.
this gets fixed by adding version = "Two", but as far as i can tell could be fixed with version = "One" as well.
struct One(Two);

impl One {
	fn new() -> Self {
		One(Two((), Three))
	}
}

struct Two((), Three);

struct Three;

impl Three {
	fn test(&self, _: &str) -> bool {
		true
	}
}

fn main() {
	let one = One::new();
	let _ = one.0 .1; // this is due to https://github.com/rust-lang/rustfmt/issues/1063
	let _ = one
		.0
		 .1 // this shouldn't be affected by that and just looks very bad
		.test("really really long string so that it wraps to the next line");
}

Thanks for the report and for double checking that this is resolved when setting version=Two. I've also confirmed that setting version=Two resolves the issue locally so there's nothing left to do. rustfmt's strong stability guarantee means we can't fix this for version=One.


Add firewall rule name help message

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[ ] I adhere to the Command Guidelines.

@haroldrandom  please help with the PR.


Not working since yesterday

Hello, @soimort!
Since yesterday, I'm getting the following response:
$ trans -brief pt:en 'Olá'
[ERROR] Null response.
[ERROR] Oops! Something went wrong and I can't translate it for you :(

I thought I should update my translate-shell, but I get the same error with gawk -f <(curl -Ls git.io/translate) -shell.
Thank you and best regards!

Maybe this is similar to a recent problem in the Google Speech API, for which a solution is available at https://stackoverflow.com/questions/9893175/google-text-to-speech-api/31791632#31791632
Basically, a "Referer: http://translate.google.com/" header is now mandatory in the Speech API request.

This is not "a recent problem" anymore, everything related to Speech API should be fixed in <IP_ADDRESS> (so as closing #73). If it still doesn't work for you, then it could be your IP address being blocked by Google. See another discussion #81 on this.
A general tip: please don't abuse Google's API.

Well, I don't think I'm abusing the service, since I just use it once a day to check some pieces of text before posting, but what is considered as "abuse" is up for Google to decide, so I'll have to check it with another IP to see if it works, thanks!

I have just checked, and my IP seems to have been blocked indeed. Thank you for your help!


Error reported when using with Modeler 4.x

Hello 👋🏻
Starting with Camunda Modeler 4.0.0 an error raised a couple of times (more than 6k error events) inside our error tracking tools which we identified from this plugin (cf. https://github.com/camunda/camunda-modeler/issues/2001)
TypeError: Cannot read property 'extensionElements' of undefined
  at addStyle (~/build/client-bundle.js:124:71)
  at None (~/build/client-bundle.js:44:13)
  at None (~/build/client-bundle.js:2153:43)

I reproduced it when I tried to trigger the overlays for a participant with no processRef. Hard to reproduce though, maybe there is an easier way to reproduce.

Hi @pinussilvestrus
This issue is fixed in the latest release.
Thank you and kind regards


IP Protection:  Is every request anonymized, or just HTTP/S requests? For example, what about DNS requests?

HTTP/S and DNS requests to eligible third-party domains are anonymized via the privacy proxies. We will provide additional details in an upcoming explainer on how we will determine which domains will be included. The rest of the traffic (e.g. the rest of the DNS requests or other HTTP/S traffic) is unaffected.

How this will affect localhost access e.g. test.domain.com pointed to <IP_ADDRESS> ?

Only third-party loads to domains on our list will be proxied. The top-level origin DNS will be resolved on the client device so such DNS redirects to localhost will still work.

Only third-party loads to domains on our list will be proxied. The top-level origin DNS will be resolved on the client device so such DNS redirects to localhost will still work.

@DavidSchinazi could you please also comment on #13?


Removing variable docker version and graphDriver data from image tests

Docker version can and GraphDriver seems to varies between different installations of docker engine causing image tests to fail, hence removing it from test config file.

LGTM.


Container name clash when deploying a mixed protocol load balancer service

Environmental Info:
K3s Version:
k3s version v1.21.5+k3s2 (724ef700)
go version go1.16.8

Node(s) CPU architecture, OS, and Version: k3os v0.21.5-k3s2r1
Linux katana 5.4.0-88-generic #99 SMP Tue Oct 5 16:53:38 UTC 2021 x86_64 GNU/Linux

Cluster Configuration:
1 server, 0 agents
Describe the bug:
The service load balancer uses only the port number when creating a load balancer service container. If the MixedProtocolLBService feature-gate is enabled and an ingress is created with the same port in multiple protocols (e.g.: enabling access to port 53 on both UDP & TCP for DNS) this causes a name clash and the deployment fails.
A solution might be to include the protocol in the name (fmt.Sprintf("lb-port-%d-%s", port.Port, port.Protocol)) to avoid this clash as it is available to the function.
Steps To Reproduce:

Installed K3s with arguments  --disable traefik --kube-apiserver-arg feature-gates=MixedProtocolLBService=true
Install ingress-nginx and set TCP values 53: "${namespace}/${tcp-service}:53" and UDP values 53: "${namespace}/${udp-service}:53"

Expected behavior:
Load balancers should be created for each ingress route
Actual behavior:
The deployment fails due to a container name clash
Additional context / logs:
E1207 20:17:46.712564    3195 fieldmanager.go:191] "[SHOULD NOT HAPPEN] failed to update managedFields" err="failed to convert new object (apps/v1, Kind=DaemonSet) to smd typed: .spec.template.spec.containers: duplicate entries for key [name=\"lb-port-53\"]" VersionKind="/, Kind="
E1207 20:17:46.713208    3195 controller.go:135] error syncing 'ingress-nginx/ingress-controller-ingress-nginx-controller': handler svccontroller: failed to create ingress-nginx/svclb-ingress-controller-ingress-nginx-controller apps/v1, Kind=DaemonSet for svccontroller ingress-nginx/ingress-controller-ingress-nginx-controller: DaemonSet.apps "svclb-ingress-controller-ingress-nginx-controller" is invalid: spec.template.spec.containers[3].name: Duplicate value: "lb-port-53", requeuing

yeah, since MixedProtocolLBService is still Alpha we haven't updated the servicelb controller to support it yet. We will definitely need to do so before it goes to Beta.

This FeatureGate will go to Beta in 1.24, so we'll need to fix it by then.
https://github.com/kubernetes/enhancements/issues/1435

The next milestone is going to be v1.23.4 so as we approach v1.24.0, be on the lookout to get item closeout asap.

Validated using v1.23.5-rc1+k3s1

I recreated this issue using v1.21.5+k3s2 but with simpler steps:

# Install k3s with the feature gate enabled
$ curl -sfL https://get.k3s.io | INSTALL_K3S_VERSION=v1.21.5+k3s2 INSTALL_K3S_EXEC="server --cluster-init --write-kubeconfig-mode 644 --kube-apiserver-arg=feature-gates=MixedProtocolLBService=true" sh -

# Apply mixed-service manifest
$ k apply -f mixed.yaml

$ cat mixed.yaml
apiVersion: v1
kind: Service
metadata:
  name: mixed-protocol
spec:
  type: LoadBalancer
  ports:
    - name: dns-udp
      port: 53
      protocol: UDP
    - name: dns-tcp
      port: 53
      protocol: TCP
  selector:
    app: my-dns-server

Doing this on v1.21.5+k3s2 failed -- the service was stuck with ExternalIPs  and there were no pods created by servicelb. Additionally, the same failure logs were seen as in this issue.
Doing these same steps but using v1.23.5-rc1+k3s1 succeeded: ExternalIPs are set and the servicelb pods are created. The logs do not have the same errors

https://github.com/k3s-io/k3s/issues/5327 is seen, but that is not a blocking issue. We may fix that as part of this release, otherwise it will likely be fixed as part of next month's patch release.


Added Numbered Navigation Bar

Fixes Issue🛠

Closes #178
Description👨💻

Added a numbered Navigation bar under navigation bars component
Type of change📄

[x] New feature (non-breaking change which adds functionality)

This has been tested✅

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷
https://github.com/Rakesh9100/Beautiify/assets/125905089/535a3a35-2004-4f47-a7a5-5a0c4e7320de

Resolve the conflicts pls @saurav2208

Resolve the conflicts pls @saurav2208

Done


Remove note that the dotnet-uninstall tool does not support .NET 8+.

Type of issue
Outdated article
Description
The tracking issue indicates that the latest update of this tool does support .NET 8: https://github.com/dotnet/cli-lab/issues/279#issuecomment-2070904214
Page URL
https://learn.microsoft.com/en-us/dotnet/core/install/remove-runtime-sdk-versions
Content source URL
https://github.com/dotnet/docs/blob/main/docs/core/install/remove-runtime-sdk-versions.md
Document Version Independent Id
090b8fbd-39e6-b754-e1da-890b1123f2c4
Article author
@adegeo
Metadata

ID: 4efe9b33-ff9f-3b37-e8bd-a34c0a77bd1e
Service: dotnet-fundamentals
Sub-service: install

cc @baronfel

Thanks for catching this! Yeah, we've added support for 8. There are some mixed-architecture bigs we have to run down but for the most common user needs we're good to go.


[Bug] Sometimes it will check type not variable name

This is because LLVM clang treats this Decl as an invalid decl, it may reference to unknown header files.
I added a new option in config file, BypassInvalidDecl. I can be bypassed or indicated a include folder with -include command line option. This PR(https://github.com/dougpuob/cpp-namelint/pull/47) fixed the problem.


Support Cabal v3 in Setup.hs

This is the same issue as in the cuda package so I copied the same
fix over.
I've only tested this on cabal >= 3.

Thanks! (:


fix-credential-process-without-stdin

in the ProcessProvider class, the _retrieve_credentials_using method uses the self._popen to open a subprocess. If there is no stdin attached to currently running program, then this call can result in an "handle is invalid" error on Windows. Adding the parameter stdin=subprocess.PIPE fixes the issue for this case.

here is an example traceback to show how this call can fail:
  File "boto3\session.py", line 258, in client
  File "botocore\session.py", line 839, in create_client
  File "botocore\session.py", line 441, in get_credentials
  File "botocore\credentials.py", line 1975, in load_credentials
  File "botocore\credentials.py", line 961, in load
  File "botocore\credentials.py", line 980, in _retrieve_credentials_using
  File "subprocess.py", line 804, in __init__
  File "subprocess.py", line 1142, in _get_handles
OSError: [WinError 6] The handle is invalid```

@nateprewitt do you know what are the next steps to get this looked at?

Checking in as there has not been any activity here in a while. I could not find any other reports of this issue, is it still something you're encountering on new versions of Boto3? If so please create a issue for further review and investigation.


[Go Functions] Support schema logic in Go Functions

Currently, Go client already supports schema function, we should do the same support in Go Function

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. Please create a new issue if it's still relevant to the maintained versions.


Fix typo PassthrougBackends -> PassthroughBackends

This change is

/lgtm

Not sure if we have to keep the typo for backcompat - in case someone is using it in a custom template?

Not sure if we have to keep the typo for backcompat - in case someone is using it in a custom template?

No because there's no release from this repo yet.

Thanks!

Coverage increased (+0.04%) to 39.815% when pulling fb8208cf8bdbb985d1bc5a1065b3e0212f8b5836 on justinsb:fix_typo into 8e90fc029025f5af82121c9b16fdbab222eef1e0 on kubernetes:master.


EE-ES Multidisc support.

Allows the auto creation of m3u files that can contain multiple discs. In order for the game option to show up the initial disc has to have "disc 1" in it (case-insensitive and can be in between brackets). Then as long as the other discs have the same name and are sequential then it will generate a new m3u file and should show up in the gamelist. Launching the m3u should be the same as any other file, but in retroarch it will recognise the file has multiple discs.
I believe dolphin also recognises m3u files.
This depends on the Emuelec function addition "createMultidisc" in emuelec-utils.

Gotta re-test this with the collections line un-commented.

Ready for commit.


Provide fast variants of operations

See https://github.com/arxanas/git-branchless/blob/a4bc3c0c31b7be5cdd95ed8cfe4385e384482a99/src/git/repo.rs#L914-L924

Please feel free to copy any amount of code directly from git-branchless. The executable as a whole is licensed under GPL-3 because it uses a GPL library, but I'm happy to license all code under MIT+Apache. I'd also recommend you copy the appropriate benches: https://github.com/arxanas/git-branchless/blob/85692c507a672f7c0df9f65436ed80f441bf308c/benches/benches.rs#L83-L118

under what condition should the parent be dehydrated as well? I see this is done for everything but the target commit. I assume this is just a matter of "when it will need to diff between two things, ensure both of those diffed items are dehydrated" but wanted to double check.

Yes, that's correct. The patch's parent commit should be dehydrated so that the diff is correct, i.e. it doesn't report a million deletion operations. The target commit's parent doesn't need to be touched because it's not diffing the target commit.
A better API might be to have the user pass the parent commit explicitly. That would also be helpful for implementing squashes, etc.

have you looked into heuristics for when to use the normal operations?

I haven't looked explicitly, but I think it's pretty much always faster. The index data structure always requires a linear scan to write all the objects and build the trees, which requires a bunch of potentially-spurious memory allocations and disk writes, whereas the cherry_pick_fast code can directly reuse any common trees. (I think Git, but not libgit2, supports some optimizations to skip building existing trees for the index, but the cherry_pick_fast path should still be the same or better.) You can also try the benchmarking code linked above to confirm.

Huh, I realized this won't quite work for the current algorithm that git2-ext uses.  I based squash and cherry-pick off of rebase to reduce the chance of conflicts since rebase will automatically go through and check for when things apply cleanly.


Why is it necessary to ask a user to edit files in order to know the user's ApplicationID and JavascriptKey?

Now to make any of the samples work, a user is required to edit db.js (for the Grid sample) or data.js (for the ToDo one) and replace fake values with actual ones.
Instead, it would be great to ask for them in modal dialog after the sample starts.

From now samples are available online.
Grid, ToDo.


Error window.NEXT_LOADED_CHUNKS is undefined on production

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Hi, I'm trying to make my app in production on Google App Engine but this error occur :
Uncaught TypeError: Cannot read property 'forEach' of undefined
When I check where the problem is I find :
window.__NEXT_LOADED_CHUNKS__.forEach
Everything is ok on dev.
You can check the live app here : https://www.chefing.fr
Here is my Dockerfile:
FROM node:6.5

RUN mkdir -p /app
WORKDIR /app

ADD package.json /app/

COPY . /app/

RUN npm i
RUN npm i -g nodemon
RUN npm i -g next
RUN npm i -g react
RUN npm i -g react-dom

RUN next build
RUN echo $GAE_VERSION > .next/BUILD_ID

ENV NODE_ENV=production

EXPOSE 3000
EXPOSE 8080

CMD [ "npm", "run", "start" ]

Everything is ok with the deployement, push / pull, deployement, start etc... No error.
And this is my next.config.js file (In case of problem).
module.exports = {
  webpack: function (c) {
    if (c.resolve.alias) {
      delete c.resolve.alias['react']
      delete c.resolve.alias['react-dom']
    }
    return c
  }
}

I hope you can help us :)
Thanks, Steve.
|---------|---------|
| next    |   2.4.7 |
| node    |     6.5 |
| mobX    |     3.2 |

Are you using a custom document?

Hi @jonaswindey, yes.
./pages/_documents.js
import Document, { Head, Main, NextScript } from 'next/document';
import { DOMProperty } from 'react-dom/lib/ReactInjection';
import { properties as DOMProperties } from 'react-dom/lib/DOMProperty';

import { reset } from '../public/styles/reset';
import { layout } from '../public/styles/pages/layout';
import { grid } from '../public/styles/grid';

export default class MyDocument extends Document {
  render () {
    const { html } = this.props
    return (
      <html>
        <Head>
          <meta charset='utf-8' />
          <link rel='canonical' href='/' />
          <meta name='viewport' content='width=device-width,minimum-scale=1' />
          <link rel="stylesheet" type="text/css" href="https://cdnjs.cloudflare.com/ajax/libs/slick-carousel/1.6.0/slick.min.css" />
          <link rel="stylesheet" type="text/css" href="https://cdnjs.cloudflare.com/ajax/libs/slick-carousel/1.6.0/slick-theme.min.css" />
          <script src="https://maps.googleapis.com/maps/api/js?key=xxxxxx&libraries=places"></script>
          <link href="https://fonts.googleapis.com/css?family=Amiri:400,400i|Roboto:100,100i,300,300i,400,400i,500,500i,700,700i,900,900i" rel="stylesheet" />
          <style>{reset}</style>
          <style>{layout}</style>
          <style>{grid}</style>
        </Head>
        <body>
          <Main />
          <NextScript />
        </body>
      </html>
    )
  }
}


Feature Request: allow scripts to be ignored from list

If you prefer to install npm modules locally, and run them via npm run, you can install ntl in this way:
npm install --save-dev ntl

Then you can configure in your package.json:
"scripts": {
  "start": "ntl",
  ...
}

This means you don't need to install ntl globally, and instead you can run npm start to bring up the ntl options.
The issue is, ntl will also display start as one of the items in the list. It would be nice to ignore this script via a command line flag. For example:
"scripts": {
  "start": "ntl --ignore start",
  ...
}

👍 for the blacklist option.
Excluding npm start isn’t possible because it’s just a personal use case.
I just wonder if this should be another key in the package.json or if it would make more sense to have an option for that like:
-b, --blacklist       Exclude scripts from the list      [list]

or:
-e, --exclude       Exclude scripts from the list      [list]

Example:
ntl --exclude=precommit,ntl

Hej @ruyadorno,
Would you accept a PR for that? :octocat:

sure thing! my thoughts:

naming-wise I think either --exclude or --ignore are much better options
ideally we should support both package.json field AND cli options (but if you want to start with just one that's fine 👍 )
another improvement I think could be useful is to be able to accept * in order to ignore a given prefix/suffix e.g: --ignore=pre* to ignore all pre-scripts

@paquan and me are currently working on this.
We have decided to name the option -e --> --exclude to prevent an alias name clash with -i -> --info.
We skip support of defining the excludes via an exclude key in the package.json because this should be done with #23.

PR is created :octocat:

Landed on v3.2.0
thank you so much @mischah @Paquan 🎉

It was our pleasure and we hat lots of fun working together on this 😄


Support dots in model names: Make FQN model selectors work with them

resolves #3246
Description
Support dots in model names as namespace separators. In our dbt projects we are managing multiple schemas. We are namespacing the models with the standard SQL syntax <schema>.<relation>.sql and following this approach. It works quite well for the most part, however when specifying model selectors by FQN (dbt run --model myschema.mytable) the matcher is unable to locate the desired models
Somehow also related to #1269
Checklist

[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have updated the CHANGELOG.md and added information about my change to the "dbt next" section.

Thanks a lot for reviewing @jtcohen6 !
Regarding your remarks, I think this is indeed mostly interesting for models to avoid collisions, as other objects such as sources are inherently anchored to their relevant namespaces . As you mentioned, I think it'd be more convenient to open new separate issues for other resources.
I've simplified the FQN matching logic and extended the following integration tests:

024_custom_schema_test: I've included a replica of our current setup. Schema and relation are parsed from the model name using custom macros, and all models are created accordingly. The test also exports the seed files to their corresponding schemas.
007_graph_selection_tests and 005_simple_seed_test: Included support for dots in model names without schema/relation parsing. Only relevant for postgres because snowflake and bigquery will complain if there are dots in the relation names

Personally I think this feature is useful only when combined with custom parsing of db, schema and alias, therefore the latter integration tests might not be of relevance. Let me know what you think


Is the project still alive?

Is the project still alive?
will this project integrate the go-ipfs?

still alive?

Yes, sort of. We haven't had much time to work on this following other projects. Once I fix my email notifications I should start to react faster to incoming PRs.

will this project integrate the go-ipfs?

That's not planned. In the meantime you should probably be using go-ipfs directly.


Selection menu takes a while to pop up.

I'm testing your grail app. It's amazing.
When I had a very very long article with lots of formatting, and I selected all, the "copy, paste, look up" menu took about 10 sec to pop up, and the app was basically frozen.
Not sure if it's a known issue, maybe I'm not a typical user who wouldn't use the app this way. But I really like the app, so I'll put the issue here.

Thanks for reporting this. I will see if I can repro. About how long of an article did you create?

I just tried in iPhone's native Notes app, it seems they have the same freezing issue. I think I'm not using it as supposed to xD.
The string (not actually an article) I used contained 8,600-ish characters, 2,000 of which are white space. I just wanted to find out how long an article I can load into the notes. I guess 8000 characters are too much for iPhone?
here is the string.

Thanks! I'm at my day job now but will look at this when I have free time in an evening / weekend.


API rate limit exceeded for *

When visiting the page for the first time, it seems that the API rate limit is set to 1 or the client makes unvoluntary requests, as it triggers a 403 error:
Heads up! Error: API rate limit exceeded for [IP] (But here's the good news: Authenticated requests get a higher rate limit. Check out the documentation for more details.)
I'm pretty sure it has to do with Astro. Maybe some misconfiguration

Looks good to me on incognito. I think you just hit GitHub's API limit. Have you tried using a different public IP?
A shared public IP with more people is often the cause. ISP's NAT, VPN's (Apple's private browsing is turn on by default if you are subscribed to iCloud+), ppl browsing GitHub services without logging in, just to name a few.

I haven't tried using another IP, but I'd guess that the API limit is also for the main GitHub website and it works fine. I'll test some more when I get home

Glad to know it resolved on its own! Thanks for stopping by😊


Model and apply Metacontrol to the system

First task is to decide which adaptations we are going to apply. After that, we need to model it with metacontrol and actually apply it.

system modes config file for the pilot example: https://github.com/MROS-RobMoSys-ITP/Pilot-URJC/blob/master/metacontroller_pilot/modes_conf/pilot_modes.yaml

lifecyclenodes: https://github.com/ros2/demos/blob/foxy/lifecycle/README.rst

More information on the system_modes: https://github.com/micro-ROS/system_modes/blob/master/system_modes/README.md#how-to-apply

https://github.com/MROS-RobMoSys-ITP/mros_navigation2/tree/master/nav2_bt_navigator/behavior_trees

Mode Change server is declared here: https://github.com/MROS-RobMoSys-ITP/Pilot-URJC/blob/master/metacontroller_pilot/metacontroller_pilot/simple_mode_manager.py

The correct system_modes file is this one: https://github.com/MROS-RobMoSys-ITP/Pilot-URJC/blob/master/pilot_urjc_bringup/params/pilot_modes.yaml

Modeling is done, we just need to round up the details to apply it.

I am closing this, as we are now able to apply the search adaptation, and switch between tasks. It is still missing to add thruster adaptation, but there are other issues for this point.


Error when installing CircuitPython on a Feather STM32F405 Express using TinyUF2

Operating System
MacOS
Board
Feather STM32F405 Express
Firmware/Software/Tool Version
TinyUF2 (0.5.0 release): https://github.com/adafruit/tinyuf2/releases/download/0.5.0/tinyuf2-feather_stm32f405_express-0.5.0.zip
CircuitPython (main branch): adafruit/circuitpython@b9fa06cf6bb3c45eaf57724d351a04564d562b6a
What happened ?
I am trying to install CircuitPython on a Feather STM32F405 Express using the TinyUF2 bootloader.  When I try to install the CircuitPython firmware by copying the firmware.uf2 image onto the FTHR405BOOT mass storage device, it starts to copy the firmware:

However, about 5-10s later, I get the following error:

The CircuitPython image does not run (the board is unresponsive) and pressing the reset button bricks the board (must be recovered with DFU).
How to reproduce ?

Install UF2 bootloader on the Feather STM32F405 Express:

dfu-util -a 0 --dfuse-address 0x08000000 -D ~/Downloads/tinyuf2-feather_stm32f405_express-0.5.0/tinyuf2-feather_stm32f405_express-0.5.0.bin

Build circuitpython image with UF2 bootloader support

cd circuitpython
git checkout b9fa06cf6bb3c45eaf57724d351a04564d562b6a
make -C ports/stm/ BOARD=feather_stm32f405_express UF2_BOOTLOADER=1

Copy build-feather_stm32f405_express/firmware.uf2 to /Volumes/FTHR405BOOT/ disk
Wait for error message to occur

Debug Log
No response
Screenshots
No response

FYI, if I program the UF2 bootloader and the CircuitPython firmware.bin firmware images manually via DFU, the board boots the CircuitPython image without issues:
dfu-util -a 0 --dfuse-address 0x08000000 -D ~/Downloads/tinyuf2-feather_stm32f405_express-0.5.0/tinyuf2-feather_stm32f405_express-0.5.0.bin

dfu-util -a 0 --dfuse-address 0x8010000 -D ports/stm/build-feather_stm32f405_express/firmware.bin

hmm, how did you build the circuitpython for f4, You will need to run make and specify UF2_BOOTLAODER=1 to select the linker with bootloader in mind. default linker is no bootloader. The main reason for this, is tinyuf2 project started much later than the circuittpython port of stm. Let me know if you still have issue with it.
make BOARD=feather_stm32f405_express UF2_BOOTLOADER=1

I'm not sure if this will help but here is a copy/paste of my recent message on adafuit/tinyuf2 discord channel, the behavior might be related.

I would like some advice for using the tinyuf2 bootloader on a STM32F401CCU6 Blackpill module.
I flashed the tinyuf2-stm32f401_blackpill-0.5.0 firmware on the module, once plugged in I have a "BlackPill" drive which contains "current.uf2", "index.html" and "info_uf2.txt", so everything looks good.
I tried to convert a bin file to UF2 format (the orginal bin works fine when it is flashed directly to address 0x08000000 without the bootlader), for that I used the following command (uf2conv is a Rust crate):
uf2conv firmware.bin -b 0x08010000 -f 0x57755a57
I also tried with the uf2conv.py script provided by Microsoft on the microsoft/uf2 repo:
uf2conv.py firmware.bin -c -b 0x08010000 -f 0x57755a57
I got the chip family id and base address directly from the tinyuf2 port code to be (a bit more) sure of what values to use.
When I copy the generated uf2 file into the bootloader drive, the drive disconnects (looks normal) but instead of booting on my program the bootloader restarts.
I have tried several base address values and the only difference is that sometimes the bootloader cannot be restarted (not surprisingly in case I overwrite the bootloader).
For information I use the same UF2 conversion method without problem on Adafruit SAMD21/51 modules.

I am able to reproduce the issue, looking into it now.

should be fixed by #137, please try again

Tried again with #137 and everything appears to be working correctly.  Thanks @hathach!

Thanks for confirmation, It is released as 0.5.1


support typeparam instantinate

Codecov Report
Base: 94.86% // Head: 94.78% // Decreases project coverage by -0.08% :warning:

Coverage data is based on head (89753d7) compared to base (8f25f1c).
Patch coverage: 16.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
- Coverage   94.86%   94.78%   -0.09%
==========================================
  Files          12       13       +1
  Lines        5880     5885       +5
==========================================
  Hits         5578     5578
- Misses        234      238       +4
- Partials       68       69       +1

Impacted Files
Coverage Δ

types_go117.go
0.00% <0.00%> (ø)

codebuild.go
95.73% <25.00%> (-0.16%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Error creating app offering

Hello,
I tried to install a sample application in our QA account but this ends up with error
Unable to create app offering: "Unable to query Create app offering"
Failed to upsert app offering: "Unable to query Create app offering"
After an additional console log output the error seems to be
Errors from GraphQL call: [{"message":"Parse error on \".\" (error) at [3, 8]","locations":[{"line":3,"column":8}]}]
All other GraphQL-Calls were successful (adding configuration items, service instances...).
Any hint on this how to fix?
Regards
Patrick
Edit:
Original error message is:
Unable to create app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Failed to upsert app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Translated message is like: "Reference has already been used"

@pfleckenstein  Thanks for reporting this.
To be able to add a new app offering you have to provide a reference. These references have to be unique across all 4me accounts and it seems like the value you are sending is already in use. So to be able to install the app you will have to select another value to use as reference for your app offering and use that instead of the value you are currently sending.
It seems like the error handling in this project doesn't seem to provide you the error message in a good/clear way. Can you maybe share the full response you get from 4me (which contains the 'Original error message' you refer to)?
I'm not quite sure what you mean by "After an additional console log output the error seems to be". Is that something you already see in the log, or did you add an additional log statement yourself, or ...?

Thanks for your reply.
Neither in our QA nor in our production environment any app offer exist.
Is the reference predefined in the examples (e.g. lansweeper, note-dispatcher...)?
What I understand is that if no app offer exist it will be created.
Otherwise it will be updated. In my scenario both actions failed.
However, I renamed the reference in the json-config file and it seems to work now.
Thank you.
With "additional console log output" I meant I tried to debug the code with console.log().
Please find the complete default output below after running npm run deploy-lansweeper.
><EMAIL_ADDRESS>deploy-lansweeper
> node lansweeper/deploy_integration.js

asset ./.aws-sam/build/IntegrationFunction/app.js 630 KiB [compared for emit] [minimized] (name: IntegrationFunction) 2 related assets
orphan modules 674 KiB [orphan] 538 modules
runtime modules 937 bytes 4 modules
built modules 1.34 MiB [built]
  modules by path ../../node_modules/ 1.23 MiB
    cacheable modules 1.11 MiB 175 modules
    + 3 modules
  modules by path ./integration-lambda/ 80.5 KiB
    modules by path ./integration-lambda/*.js 80 KiB 12 modules
    modules by path ./integration-lambda/errors/*.js 490 bytes 2 modules
  modules by path ../../library/helpers/ 28.3 KiB
    modules by path ../../library/helpers/*.js 27.3 KiB 9 modules
    modules by path ../../library/helpers/errors/*.js 1010 bytes 3 modules
  + 14 modules
webpack 5.89.0 compiled successfully in 24572 ms

                Managed S3 bucket: aws-sam-cli-managed-default-samclisourcebucket-usqrxyrder8e
                A different default S3 bucket can be set in samconfig.toml
                Or by specifying --s3-bucket explicitly.
File with same data already exists at app-builder-lansweeper/123 skipping upload

        Deploying with following values
        ===============================
        Stack name                   : app-builder-lansweeper
        Region                       : us-east-1
        Confirm changeset            : False
        Disable rollback             : False
        Deployment s3 bucket         : aws-sam-cli-managed-default-samclisourcebucket-usqrxyrder8e
        Capabilities                 : ["CAPABILITY_IAM"]
        Parameter overrides          : {"4MeDomainParameter": "4me.qa", "BootstrapSecretApplicationParameter": "4me-app-builder", "BootstrapSecretAccountParameter": "myCompany", "OfferingReferenceParameter": "lansweeper"}
        Signing Profiles             : {}

Initiating deployment
=====================

File with same data already exists at app-builder-lansweeper/123.template, skipping upload

Waiting for changeset to be created..

CloudFormation stack changeset
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Operation                                    LogicalResourceId                            ResourceType                                 Replacement
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
* Modify                                     IntegrationFunctionCWSchedulePermission      AWS::Lambda::Permission                      Conditional
* Modify                                     IntegrationFunctionCWSchedule                AWS::Events::Rule                            False
* Modify                                     IntegrationFunctionSecretsEventListenerPer   AWS::Lambda::Permission                      Conditional
                                             mission
* Modify                                     IntegrationFunctionSecretsEventListener      AWS::Events::Rule                            False
* Modify                                     IntegrationFunction                          AWS::Lambda::Function                        False
* Modify                                     ServerlessRestApi                            AWS::ApiGateway::RestApi                     False
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Changeset created successfully.

2024-01-23 10:38:53 - Waiting for stack create/update to complete

CloudFormation events from stack operations (refresh every 5.0 seconds)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
ResourceStatus                               ResourceType                                 LogicalResourceId                            ResourceStatusReason
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
UPDATE_IN_PROGRESS                           AWS::CloudFormation::Stack                   app-builder-lansweeper                       User Initiated
UPDATE_IN_PROGRESS                           AWS::Lambda::Function                        IntegrationFunction                          -
UPDATE_COMPLETE                              AWS::Lambda::Function                        IntegrationFunction                          -
UPDATE_COMPLETE_CLEANUP_IN_PROGRESS          AWS::CloudFormation::Stack                   app-builder-lansweeper                       -
UPDATE_COMPLETE                              AWS::CloudFormation::Stack                   app-builder-lansweeper                       -
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

CloudFormation outputs from deployed stack
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Outputs
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Key                 DeadLetterQueueUrl
Description         SQS dead-letter queue URL
Value               https://sqs.us-east-1.amazonaws.com/123

Key                 IntegrationFunctionIamRole
Description         Implicit IAM Role created for Integration Function
Value               arn:aws:iam::123:role/123

Key                 RefreshQueueUrl
Description         SQS queue for refresh URL
Value               https://sqs.us-east-1.amazonaws.com/123/123

Key                 RefreshQueueArn
Description         SQS queue for refresh ARN
Value               arn:aws:sqs:us-east-1:123:lansweeper-refresh-trigger

Key                 DeadLetterQueueArn
Description         SQS dead-letter queue ARN
Value               arn:aws:sqs:us-east-1:123:lansweeper-dead-letter

Key                 IntegrationApi
Description         API Gateway endpoint URL for Prod stage for Integration Function
Value               https://123.execute-api.us-east-1.amazonaws.com/Prod/integration/

Key                 IntegrationFunction
Description         Integration Lambda Function ARN
Value               arn:aws:lambda:us-east-1:123:function:app-builder-lansweeper-IntegrationFunction-123
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Successfully created/updated stack - app-builder-lansweeper in us-east-1

Created lambda "arn:aws:lambda:us-east-1:123:function:app-builder-lansweeper-IntegrationFunction-123" at: "https://123.execute-api.us-east-1.amazonaws.com/Prod/integration/"
Used S3 bucket aws-sam-cli-managed-default-samclisourcebucket-123for storage
Unable to create app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Failed to upsert app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]

@pfleckenstein the app offering reference must be unique across all accounts, this means you will never be able to create app offerings using the default values in this project (for instance in lansweeper/config/4me/app_offering_input.json) as we already used those values for an app offering in the '4me' account. (In your example the Lansweeper app we offer.)
I believe this issue can be closed now, as changing the reference value worked for you.


Fix sorting indicator

Resolves #1444
Arrows got shrunken by 1/3:

Codecov Report

Merging #1458 into demo will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             demo    #1458   +/-   ##
=======================================
  Coverage   82.83%   82.83%
=======================================
  Files         128      128
  Lines        5780     5780
  Branches      663      663
=======================================
  Hits         4788     4788
  Misses        836      836
  Partials      156      156

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 1e2f9ed...0de9ee7. Read the comment docs.


Coin flip UI v4 (branch from Keiths PR404 branch)

Added Coin logic, icon and coin GUI to Seedsigner.

After a quick review of the code... Nice and tight, clean commits, made it look simple.
As for the choice to build off an unmerged PR... I see that you're using functionality that didnt previously exist.  I wonder if you couldn't implement enough logic within a helper function within your view, maybe in a parallel PR, so that this can work with or without PR-#404 (in similar fashion as is done in the BruteForceAddressVerificationThread methods: derive_singlesig/derive_multisig.  Those are sort of helper funcs inside a class inside a view.... with a TODO note to use the helper functions from derive_mnemonic as soon as they're merged.)
Well done on your first PR.
And now I'll reiterate once again, It would be much appreciated to have your eyes on other PRs and Issues in the forum too.

This PR is pending revision, review and merge of PR #404. Once PR#404 is merged, conflicts can be resolved and review and testing can take place.

@thedon702 apologies for the long hiatus. PR #404 will hopefully be moving along soon. In anticipation of that, can you update this PR?
I guess the steps would be something like:
# Get local copy of latest in #404
git fetch pull/404/head:pr_404

# rebase your branch to latest in #404
git checkout coin_flip_UI_v4
git rebase pr_404

# interactively resolve any conflicts...

# force push the resulting changes
git push --force

@kdmukai It should be up to date now. I noticed that you removed DICE__NUM_ROLLS__12WORD variables, so that took me a little bit to figure out.
Can you guys test the code since I don't have my ss setup for debugging atm. I might fire up my emulator as well to do a quick test. The original code was tested last summer and it seems to be basically the same.
Gracias.

@thedon702 Did you mean to close this PR? Also not sure how this PR is now showing 0 changes but your branch in your repo still has your changes.
The DICE_NUM_ROLLS__12WORD is still in my PR:
https://github.com/SeedSigner/seedsigner/blob/c4aafd0bf32bf245d636f18efdebaeafd1fce5ce/src/seedsigner/helpers/mnemonic_generation.py#L17

@kdmukai Oh my bad, I was working off the main dev branch not your branch. Can I submit a new PR after your branch is merged, that would make my life easier. So we can close this PR and I create a new one after the merge.

@kdmukai
Btw, when I try to do git fetch pull/404/head:pr_404
I get this error: fatal: couldn't find remote ref pull/404/head
It's weird because when I created this PR last summer it was working fine from your branch with the exact command. But yeah it's probably easiest to wait until you merge so I can redo this PR on the main dev channel.

@thedon702 yep, getting the same error with git fetch origin pull/404/head:pr_404. Maybe it's an issue with codespaces, but I don't know why because it was working last summer.

@kdmukai worst case you can just put the code in yourself, since it's just a few edits. codespaces just ins't cooporating with your branch for some reason.

@kdmukai I managed to figure out what I was doing wrong, I needed to use
git fetch upstream pull/404/head:pr_404
git rebase pr_404
I created a new draft PR, you can check it out and see if it looks ok?


data-order doesn't work (probably because of dynamically added row?)

Hi,
I'm trying to sort the column using data-order but the row doesn't sorted as expected.
The row is created dynamically using datatable.row.add method.
// Get search table
var table = $('#table-search').DataTable();

// Add row to table (add html as row using jquery object)
table.row.add($('#table-search-row')).draw();

Here is the screenshot of the html and it's data-order value when i sort asc by clicking the column header.

Any help?
Thank you

Can you link to a test case showing the issue so it can be debugged please.

Here is the fiddle.
http://jsfiddle.net/4pry7og5/3/
If you see, the number column doesn't sort as expected even tough there is data-order declared in td.
Thank you

I'm not seeing a problem here - both columns are sorting fine in your fiddle from what I see.  You are also generating random strings as well... perhaps defined strings instead of random would make your fiddle more clear and show exactly what row(s) you see are out of place.

@dexcell I know that this isn't directly related to issue, but I saw Your code and noticed two places that You can easily improve.

Don't call draw inside loop, call in after You add all rows, table object also has draw() method.

Here is Your code with those fixes:
http://jsfiddle.net/4pry7og5/6/
I added small timing information, so You will see difference with and without my fixes.

@dexcell I also fixed Your issue by moving DataTable initialization after for loop:
http://jsfiddle.net/4pry7og5/7/
This solves Your issue, but as You wrote, when using datatable.row.add DataTables is "skipping" data-* attributes.
@DataTables is this by design or a bug?

is this by design or a bug?

Neither really! The table is empty when it is initialised so DataTables can't automatically detect any attributes for sorting, filtering, etc. Then new rows are added, which is fine, but DataTables doesn't automatically look for attributes in newly added rows - only when the table is initialised.
So I guess you could say it is by design - it is working as I would expect.
What you can do is tell DataTables to expect attributes using the columns.render option:
        columnDefs: [
            {
                targets: -1,
                data: {
                    _: "1.display",
                    sort: "1.@data-order",
                    type: "1.@data-order"
                }
            }
        ]

Updated fiddle.
Inspecting rows on add for automatically detecting attributes is not something I currently want to add into DataTables, so this manual option is currently the only way.

Thank you so much for your help and tips

@DataTables I would like to discuss the solution you gave by using collumnDefs as a solution to use data-order on dynamically loaded content. I do not know if this is the correct place, but here goes..
I updated the Fiddle you gave, which you can view here, to having three columns, but this gives an error. Can you help me out?
My actual problem is the same as @dexcell described above...using data-order on dynamically loaded content.  My actual code can be viewed below, but the updated Fiddle should suffice.
    // Initialize DataTable
    $(mpcTableSelector)

        // Show processing message and hide table until DataTables is loaded
        .on('processing.dt', function(e, settings, processing) {
                if (processing) {
                    $(mpcTableSelector).css('display', 'none');
                } else {
                    $(mpcTableSelector).removeAttr('style');
                }
            })

        // When DataTables is loaded, hide processing message and show table
        .dataTable(
        {
            // Localization
            language    : {
                emptyTable    : mpc.translations.datatables.event.emptyTable,
                info          : mpc.translations.datatables.event.info,
                infoEmpty     : mpc.translations.datatables.event.infoEmpty,
                infoFiltered  : mpc.translations.datatables.event.infoFiltered,
                infoPostFix   : '',
                thousands     : '.',
                lengthMenu    : mpc.translations.datatables.event.lengthMenu,
                loadingRecords: mpc.translations.datatables.event.loadingRecords,
                processing    : mpc.translations.datatables.event.processing,
                search        : mpc.translations.datatables.event.search,
                zeroRecords   : mpc.translations.datatables.event.zeroRecords,
                paginate      : {
                    first   : mpc.translations.datatables.event.first,
                    last    : mpc.translations.datatables.event.last,
                    next    : mpc.translations.datatables.event.next,
                    previous: mpc.translations.datatables.event.previous
                },
                aria          : {
                    sortAscending : mpc.translations.datatables.event.sortAscending,
                    sortDescending: mpc.translations.datatables.event.sortDescending
                }
            },

            // Features
            ordering    : true,
            paging      : true,
            searching   : true,
            columnDefs  : [
                {
                    targets   : 'nosearch',
                    searchable: false
                },
                {
                    targets : 'nosort',
                    sortable: false
                },
                {
                    targets      : 'radio',
                    orderDataType: 'dom-radio'
                }
            ],

            // Callbacks
            initComplete: function() {

                // Remove pagination if there is only one page
                var wrapper = $(mpcTableSelector).parent();
                if ($('.dataTables_paginate > span a', wrapper).length < 2) {
                    $('.dataTables_info', wrapper).remove();
                    $('.dataTables_paginate', wrapper).remove();
                }
            },

            // Options
            dom         : '<"top"lr>t<"bottom"ip>',
            order       : [[$('th.sort').index(), 'asc']],
            pageLength  : 10,
            processing  : true
        }
    );

If you could post support questions in the DataTables forums please. Thanks!

DataTables post can be found here.

To answer the question:
The 1 (in the example below) is the key where the data-order attribute listens to.
        columnDefs: [
            {
                targets: [1],
                data: {
                    _: "1.display",
                    sort: "1.@data-order",
                    type: "1.@data-order"
                }
            }

So when you change your target to ThisIsMyNewTarget for example, then you should also change your data to the following:
        columnDefs: [
            {
                targets: ThisIsMyNewTarget,
                data: {
                    _: "ThisIsMyNewTarget.display",
                    sort: "ThisIsMyNewTarget.@data-order",
                    type: "ThisIsMyNewTarget.@data-order"
                }
            }

This worked for me:
table.rows().invalidate().draw();

@iuyiuy This should be somewhere in huge bold red letters... After hours of searching - leaving it- coming back to it- routine, finally this worked for me

@Tomino2112 I completely agree. Do we need to open a new bug report or can we reopen this one?

@iuyiuy Dude! Thats works! But why didn't the author hinted this. But worked fine in my case..

Is there a way to do that on multiple keys at once? without defining a new target for each

No - sorry. You have to do it one at a time. The only other option would be to have a function do the bulk of the work for you for each target.

Ok I thought so but the data function is used for all kinds of operation
and I don't know what to return for each occasion. Is there a documentation
for that or at least a solid example? The "price" one is not very helpful
On Thu, Jul 6, 2017 at 15:40 Allan Jardine<EMAIL_ADDRESS>wrote:

No - sorry. You have to do it one at a time. The only other option would
be to have a function do the bulk of the work for you for each target.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/DataTables/DataTables/issues/496#issuecomment-313384465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABrVvIkxZkK-eNbKGUiOFYMdigPB1SR0ks5sLNW5gaJpZM4DgiBq
.


reach matching and daily agg

This PR is adds two main functionalities. Both main parts were discussed in the inland-salinity repo so this should (hopefully) go pretty quick.

site-to-reach matching (closes #16) - see https://github.com/USGS-R/drb-inland-salinity-ml/pull/27 for relevant discussion
temporal aggregation of instantaneous to daily summaries (closes #34) - see https://github.com/USGS-R/drb-inland-salinity-ml/pull/26 for relevant discussion.

Hey @jsadler2 do you mind pulling changes from main into 16-34-reach-matching-daily-agg before I review?

Just did a rebase onto main, @lekoenig, so it should be good now.

Thanks for the feedback, @lekoenig . I think I addressed all of your comments.

I think there's just one lingering target name thing to address before we merge. See my comment above.

oh boy. i guess that's what i get for not actually building the thing. thanks for your patience on this.
I've now actually successfully built it and I think it's good to go

oh boy. i guess that's what i get for not actually building the thing. thanks for your patience on this.
I've now actually successfully built it and I think it's good to go

No worries, that's what code reviews are for! It looks good to me and I was able to successfully build the pipeline. I approve to go ahead and merge.


Searching based on synonyms

At this moment, added synonyms to a class aren't working when searching in the bSDD Revit plugin. It would be very helpful if this could be implemented in the plugin, as this would make searching a lot more approachable.
I noticed that in the API documentation, the search function /api/Class/Search/v1 does make use of synonyms.
Regards,
René Sondermeijer
(https://github.com/buildingsmart-community/bSDD-Revit-plugin/issues/98)

Yes, it should

The bSDD search pages does search synonyms:

Fixed in https://github.com/buildingsmart-community/bSDD-filter-UI/commit/62571f59e0c76010a9cac05b3e3ee84a8e5369f1


[IOS] Bug with tabs page and non-tab page

Bug Report
Ionic Info
   ionic (Ionic CLI)          : 4.0.3 (/usr/local/lib/node_modules/ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.0
   @angular-devkit/core       : 0.7.0-rc.3
   @angular-devkit/schematics : 0.7.0-rc.3
   @angular/cli               : 6.0.8
   @ionic/ng-toolkit          : 1.0.0
   @ionic/schematics-angular  : 1.0.1

Cordova:

   cordova (Cordova CLI) : 7.1.0
   Cordova Platforms     : ios 4.5.5

System:

   ios-deploy : 1.9.2
   ios-sim    : 6.1.1
   NodeJS     : v10.6.0 (/usr/local/bin/node)
   npm        : 6.3.0
   OS         : macOS High Sierra
   Xcode      : Xcode 9.4.1 Build version 9F2000

Describe the Bug
I have an application with a tabs ( 5 pages home, search, map , info and tarif).
From the search page i can go on a non-tab page like this this.router.navigate(['station'], { queryParams: station});
On the station page i have a back button to return to the search page ```  ````
When i click on the tab menu on an other page, i have a white screen without log on xcode.
On web all work fine, i reproduce the bug only on IOS.
Here are some additional informations:
tabs.router.module.ts
const routes: Routes = [
  {
    path: 'tabs',
    component: TabsPage,
    children: [
      {
        path: 'home',
        outlet: 'home',
        component: HomePage
      },
      {
        path: 'search',
        outlet: 'search',
        component: SearchPage
      },
      {
        path: 'map',
        outlet: 'map',
        component: MapPage
      },
      {
        path: 'info',
        outlet: 'info',
        component: InfoPage
      },
      {
        path: 'tarif',
        outlet: 'tarif',
        component: TarifPage
      }
    ]
  },
  {
    path: '',
    redirectTo: '/tabs/(home:home)',
    pathMatch: 'full'
  },
  {
    path: 'search',
    redirectTo: '/tabs/(search:search)',
    pathMatch: 'full'
  }
];

app-routing.module.ts
const routes: Routes = [
  { path: '', loadChildren: './tabs/tabs.module#TabsPageModule' },
  { path: 'map', loadChildren: './map/map.module#MapPageModule' },
  { path: 'search', loadChildren: './search/search.module#SearchPageModule' },
  { path: 'station', loadChildren: './station/station.module#StationPageModule' },
  { path: 'info', loadChildren: './info/info.module#InfoPageModule' },
  { path: 'tarif', loadChildren: './tarif/tarif.module#TarifPageModule' }
];

i don't know if it's my bad or a bug ?
Steps to Reproduce
Steps to reproduce the behavior:

Click on tab menu to go to search page
From search page go to station page ( a non-tab page)
Go back to search page
Click on tab menu to go on an other page

Expected Behavior
Fix nav when i from a non-tab page
Additional Context
Only on IOS, works on web

There’s already two open issues related to this, I will link them after.

@omaxlive thx for your response. Do you link them when the bug will be fix ?

#14885
#14888
#15045
#15050

hello @manucorporat thx to fix the bug.
If i want to try i have to update ionic with npm i -g ionic to pull the last ionic core ?


promql: Fix issue where summation of +/- infinity returns NaN instead of infinity

This fixes an issue with kahanSumInc which would result in NaN being used as the final result when +Inf or -Inf was one of the values added.
I've expanded the test coverage of kahanSumInc to cover these cases and others involving NaN.
This does now mean that kahanSumInc does an extra check for Inf at each step. An alternative to this would be to check for +Inf or -Inf when computing the final result (ie. after all the calls to kahanSumInc), but I opted for this as the simplest, least invasive change given how widely kahanSumInc is used.
This also revealed what I believe was an incorrect test case in promql/promqltest/testdata/native_histograms.test, where the stddev and stdvar were incorrectly calculated when a native histogram contained a sample at +Inf.

Just 1 nit: Can we put the test inside functions_test.go? It is a small file and we can add a comment to the test saying why we are testing it separately.

That was my original intention, but functions_test.go is in the promql_test package, and so can't access kahanSumInc.
promql_test was introduced in https://github.com/prometheus/prometheus/pull/13999. I can move the test to functions_test.go, but then I'll need to change its package back to promql.


Wire up preload scripts for the WebView to get status text updates. r…

…=mossop

LGTM

Coverage remained the same at 87.398% when pulling 1c9d7ea5e034c25541f306bc45beca6ed3572b4d on jsantell:enable-preload into a8de8b71f741a3a9f2d6b813dca70c3eb193771a on mozilla:master.


7.4.2 Connected Dapps Pop up

1. User Story
As a user I want to disconnect dapps from my Kukuza wallet that i no longer use.
2. Acceptance Criteria
Technical implementation can be done later.

Clicking "disconnect" disconnects the dapp from my wallet.

3. Related Issues
Technical feature tracked here, park for now:
https://app.dework.xyz/kukuza/product-management

[ ] Create all related issue for this issue, label them, assign them, and move them to the right projects.
[ ] tbd
[ ] tbd

4. Wire Frame
None
5. Design
Add:

Screenshot
Figma link to specific element

@Chegemartinx please add design


[HUDI-389] Fixing Index look up to return right partitions for a given key along with fileId with Global Bloom

What is the purpose of the pull request
Fixing Index look up to return partitions for a given key along with fileId with Global Bloom
Use-case:
If a record is updated with a different partition than where it exists, with Global bloom, an exception is thrown as given here. This patch fixes the same.
Essentially in HoodieGlobaIndex#tagLocationBacktoRecords, each record is tagged with the right fileId and Partition and not just fileId (which was the case before this patch). In order to achieve this, I had to change the interface for IndexFileFilter from getMatchingFiles(String partitionPath, String recordKey) returning Sets to returning Set<Pair<ParitionPath, fileId>>s.
Brief change log

Change interface of IndexfileFilter to return Set<Pair<PartitionPath, fileId>>
Fix Global Bloom to tag records with correct partitonPath from index look up and not from the passed in records(HoodieKey).

Tests:
Added test in TestHoodieClientOnCopyOnWriteStorage#testUpsertToDiffPartitionGlobaIndex.

Added a workaround for collectAsMap. Will continue review today.

@vinothchandar : have addressed all feedback so far. waiting for few clarifications from you.


generate docker command is not working

Describe the bug
generate docker command is not working as described in release notes
Environment
Provide accurate information about the environment to help us reproduce the issue.

Harvest version: harvest version 21.05.3-1 (commit 63f0b11) (build date 2021-06-23T22:11:00+0530) linux/amd64
Command line arguments used: started via systemd
OS: Red Hat Enterprise Linux release 8.3 (Ootpa)
Install method: rpm-package
ONTAP Version: NetApp Release 9.5 upwards until 9.8

To Reproduce
[root@vihlv0230 harvest]# pwd
/opt/harvest
[root@vihlv0230 harvest]# bin/harvest generate docker
panic: open docker/onePollerPerContainer/docker-compose.tmpl: no such file or directory

goroutine 1 [running]:
goharvest2/cmd/tools/generate.generateDockerCompose(0x905636, 0xd)
        goharvest2/cmd/tools/generate/generate.go:72 +0x6e5
goharvest2/cmd/tools/generate.doDockerCompose(0xbf4e60, 0xc2a4f8, 0x0, 0x0)
        goharvest2/cmd/tools/generate/generate.go:47 +0xcb
github.com/spf13/cobra.(*Command).execute(0xbf4e60, 0xc2a4f8, 0x0, 0x0, 0xbf4e60, 0xc2a4f8)
        github.com/spf13/cobra@v1.1.3/command.go:856 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0xbf5fe0, 0x0, 0xbb5768, 0xc000096058)
        github.com/spf13/cobra@v1.1.3/command.go:960 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
        github.com/spf13/cobra@v1.1.3/command.go:897
main.main()
        command-line-arguments/harvest.go:683 +0x34

Expected behavior
should create docker-compose file for each poller configured in harvest.yml
Actual behavior
generates error messages and does not create docker-compose files
Additional context
seems that docker-template/docker-folder is missing in RPM. Can not find the docker-compose.tmpl file in current working directory.

You're right @florianmulatz some files were missed in the rpm/deb build. If you want to give it a try sooner you can you grab the tar.gz and add the directories to your rpm install or alternatively just try from the exploded tar.gz?
Thanks for reporting

Hi @cgrinds,
I can confirm that grabbing the docker directory from the harvest-21.05.3.tar and putting into the harvest-directory leads to working creation of docker-compose stdout. Maybe you should mention in documentation that output to a docker-compose.yml should be piped via ">".
Altough I suggest to not use "restart: always" option for the docker but "restart: unless-stopped" so the administrator does have more control over the container start/stop behaviour if wanted.
Just my two cents
Cheers

Agree on the redirect - this is what bin/harvest generate systemd says
Save the following to /etc/systemd/system/harvest.target or | sudo tee /etc/systemd/system/harvest.target

I'll add similar to docker.
Agreed, restart policies are tricky, but you're right unless-stopped sounds better. Will update

Similar to always, except that when the container is stopped (manually or otherwise), it is not restarted even after Docker daemon restarts.

verified on release/21.05.4 running on RHEL8.
bin/harvest generate docker
Save the following to docker-compose.yml or > docker-compose.yml
and then run docker-compose -f docker-compose.yml up -d --remove-orphans
version: "3.5"
services:

  foo2:
    image: harvest:latest
    container_name: harvest_21.05.4_foo2
    restart: unless-stopped

    stdin_open: true # docker run -i
    tty: true        # docker run -t
    command: ' --poller foo2 --loglevel 2  --config /opt/harvest.yml'
    #entrypoint: ["sh", "-c", "sleep 2073600"]
    volumes:
      - harvest:/opt/harvest/
      - type: bind
        source: /opt/harvest/harvest.yml
        target: /opt/harvest.yml

volumes:
  harvest:⏎


get_view_state gets triggered twice on v0.5.0

On the latest release, a get_view_state in an observer gets triggered twice (one time with the old value, and again with the right one). You can see it when grabbing the zoom in a shinyapp:
shinyApp(
  ui = fillPage(
    rdeckOutput("map", height = "100%")
  ),

  server = function(input, output) {

    output$map <- renderRdeck({
      rdeck(initial_view_state = view_state(center =  c(-73.58, 45.53), zoom = 15)) |>
        add_mvt_layer(id = "test",
                      data = tile_json("mapbox.mapbox-streets-v8", "mapbox"))
    })

    observe(print(get_view_state("map")$zoom))

  }
)

The bug does not happen in the release v0.4.0.

Fixed in dev/latest


[pulsar-broker] Change dispatcher blocked/unblocked log level to debug

Motivation
There are so many dispatcher blocked/unblocked log(more than 30K within 15 min), I supposed change it to debug.

Verifying this change

[x] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Documentation
Check the box below or label this PR directly.
Need to update docs?

[x] no-need-doc
Just change log level

@wolfstudy @merlimat @eolivelli PTAL

I don't see a way to retrigger tests for some reason. Closing and reopening.

@Shawyeok - looks like there are merge conflicts now. Are you able to resolve those? Thanks.

@Shawyeok - looks like there are merge conflicts now. Are you able to resolve those? Thanks.

@michaeljmarshall It's done, PTAL


onyo commands in --help should be sorted

If there is an intentional order to the way onyo commands and options are listed in help, it is unclear to me. I find it to be hard to find a given command.
I propose alphabetical.
❱ onyo --help
usage: onyo [-h] [-d] [-C ONYOPATH] {init,mv,new,edit,cat,tree,history,git,config,mkdir,rm,fsck,set} ...

A text-based inventory system backed by git.

options:
  -h, --help            show this help message and exit
  -d, --debug           Enable debug logging
  -C ONYOPATH, --onyopath ONYOPATH
                        Run onyo commands from -C <dir>

onyo commands:
  {init,mv,new,edit,cat,tree,history,git,config,mkdir,rm,fsck,set}
    init                Initialize an onyo repository
    mv                  Move an asset
    new                 Create a new asset
    edit                Edit an existing asset
    cat                 Print the contents of an asset
    tree                Print a folder and all descendants as a tree structure
    history             Show the history of an asset or folder
    git                 Run git command in onyo
    config              Configure onyo options
    mkdir               Create folder(s) in onyo; will automatically git anchor
    rm                  Delete asset(s)
    fsck                Check the consistency and validity of the onyo repository and its contents
    set                 Set values in assets

Great point! I think it is just the order from the parse.args() function in onyo/main.py. If that is the case, it can easily be solved via copy/paste.
At the same time, we should also do that for the subcommands (e.g. onyo mv --help) and also in the readme (or read the docs), because these also list the commands and subcommands.


Making this work in nixpkgs

I've noticed grpc-haskell and loads of downstream deps are broken in nixpkgs master.
However since it is working here, it should be possible to make it work in nixpkgs as well.
First was proto3-wire which had an older version, fixed by overriding with a custom derivation pointing to the 1.2.0.
Second thing was parameterized. It was fixed by disabling tests: https://github.com/louispan/parameterized/issues/2
Third thing was proto3-suite with 1 test failure due to a missing test file. Fixed by disabling tests.
But finally there's some C compilation issues. We can see alot of changes in release.nix but, not sure what is causing the C issues here.
Why not push up the specific overlays here to nixpkgs upstream so that grpc-haskell can work cleanly?

I tried importing the release.nix from this repo directly in my nix files.
But because we have a different nixpkgs pin, haskell package conflict occurs.
My shell.nix
{ pkgs ? import ./pkgs.nix }:

with pkgs;
let
  haskellPackages = haskell.packages.ghc865;
  grpc-haskell-src = builtins.fetchGit {
    url = https://github.com/awakesecurity/gRPC-haskell;
    rev = "d8f6e0b476797938f5c759146a2bbdc829ba1a84";
  };
  grpc-haskell = (import "${grpc-haskell-src}/release.nix").grpc-haskell;
  drv = (haskellPackages.callPackage ./default.nix { inherit grpc-haskell; }).env;
in
  drv.overrideAttrs (attrs: {
    src = null;
    nativeBuildInputs =...
    shellHook = ...
  })

Error message is:
Resolving dependencies...
cabal: Could not resolve dependencies:
[__0] trying: matrix-ark-<IP_ADDRESS> (user goal)
[__1] trying: grpc-haskell-<IP_ADDRESS>/installed-IZB... (dependency of matrix-ark)
[__2] next goal: conduit-extra (dependency of matrix-ark)
[__2] rejecting: conduit-extra-1.3.5/installed-GDE... (conflict: grpc-haskell
=> async==2.2.2/installed-6rO..., conduit-extra =>
async==2.2.2/installed-Axg...)
[__2] fail (backjumping, conflict set: conduit-extra, grpc-haskell,
matrix-ark)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: grpc-haskell, matrix-ark,
conduit-extra

This is still broken on Nixpkgs 21.05:
nix-repl> :b pkgs.haskellPackages.grpc-haskell
builder for '/nix/store/6h7m7i87gk1mrj6iv7hg65nlx0c0jz2c-grpc-haskell-core-<IP_ADDRESS>.drv' failed with exit code 1; last 10 log lines:
  Using tar found on system at:
  /nix/store/hlmzk7zf3xp54dgf6kmk0qg13wc9ma8j-gnutar-1.34/bin/tar
  No uhc found
  building
  Preprocessing library for grpc-haskell-core-<IP_ADDRESS>..
  c2hs: Errors during expansion of binding hooks:

  src/Network/GRPC/Unsafe.chs:222: (column 7) [ERROR]  >>> Unknown identifier!
    Cannot find a definition for `grpc_channel_ping' in the header file.
cannot build derivation '/nix/store/6v5mc2mcmyqqwhpqy9j9mkzs4adjp32r-grpc-haskell-0.1.0.drv': 1 dependencies couldn't be built
[0 built (1 failed)]
error: build of '/nix/store/6v5mc2mcmyqqwhpqy9j9mkzs4adjp32r-grpc-haskell-0.1.0.drv' failed

Tasty is resolved but preprocessing is still a problem. Seems like the default.nix needs to be fixed as well.


Update master shrinkwrap file

Coverage increased (+0.05%) to 90.984% when pulling 46d9165e09064c058d6958b4a4bea3d97ed36689 on SAP:shrink into 800faab967a4d61ab7f7cd21017a8fb4bf72f08e on cloudfoundry-incubator:master.

Coverage increased (+0.05%) to 90.984% when pulling 46d9165e09064c058d6958b4a4bea3d97ed36689 on SAP:shrink into 800faab967a4d61ab7f7cd21017a8fb4bf72f08e on cloudfoundry-incubator:master.

Coverage increased (+0.05%) to 90.984% when pulling 46d9165e09064c058d6958b4a4bea3d97ed36689 on SAP:shrink into 800faab967a4d61ab7f7cd21017a8fb4bf72f08e on cloudfoundry-incubator:master.


How to ExcludeFromCodeCoverageAttribute in a test framework scenario?

Description
If I run dotnet test --collect="Code Coverage", then it counts xunit classes towards code coverage statistics.
Coverlet addresses this problem by providing an ExcludeByAttribute where the user can define arbitrary attributes to use for exclusion purposes.
Steps to reproduce

Create a new .NET library project that has a class Foo that calculates A + B
Create a new xunit test project that tests class Foo
Run dotnet test --collect="Code Coverage"

Expected behavior
xunit assemblies not included in code coverage report.
Actual behavior
xunit assemblies included in code coverage report.
Diagnostic logs
n/a - please specify what you would want for this, and I can provide
Environment
Test SDK 17.5

Oops. Hit close by accident while holding my newborn.
Just wanted to say that you should also think about Coverlet approach ExcludeByAttribute. For example, what about assertion libraries like Shoudly.

In latest version https://dev.azure.com/dnceng/public/_artifacts/feed/test-tools/NuGet/Microsoft.CodeCoverage/overview/17.7.0-preview.23203.7 we added default exclusion of few libraries. We are working on some more generic approach like Coverlet has so not closing this.

We are working on some more generic approach like Coverlet has so not closing this.

Hi @jakubch1 , what does this approach consist of? Can you share some information about it? It would be nice to exclude from code coverage report  all about third-party libraries.

We added some default exclusions for xunit and moq libraries. Generic solution is still not implemented.

I think this is related to #72

In latest version https://dev.azure.com/dnceng/public/_artifacts/feed/test-tools/NuGet/Microsoft.CodeCoverage/overview/17.10.0-preview-24073-02 we ported from Coverlet feature to optionally exclude modules based on existence of sources.
To enable this feature in command line you can use:
dotnet test --collect "Code Coverage;ExcludeAssembliesWithoutSources=MissingAll"

If you use config/runsettings you can add:
<ExcludeAssembliesWithoutSources>MissingAll</ExcludeAssembliesWithoutSources>

under <Configuration> tag.
Remember that this is adding additional processing to check existence of sources. You will get always better performance by configuring correctly <ModulePaths> regexes.

Shouldn't this be the default?
When would I ever care about coverage of an assembly for which I don't have any sources?

I don't want to make it default because of performance. To check if sources exist you need to start with dll path => read dll metadata => read PDB data and get all sources => in case of deterministic build calculate original paths => check each file existence. This can take time for big solutions
For <ModulePaths> regexes you take dll path and check regex => no disk operations.
To calculate coverage using our tooling, sources are not needed so you can imagine scenarios that you build test projects on 1 machine then move binaries to several test nodes and execute them there. We are able to collect coverage in such scenarios.

We added some default exclusions for xunit and moq libraries. Generic solution is still not implemented.

What about NUnit?

We added some default exclusions for xunit and moq libraries. Generic solution is still not implemented.

What about NUnit?

We added default exclusions for nunit libraries. It will be available in the next release.


(Question) Can it be a conversion problem from .cdex to .dex?

I was able to complete a deodex with Android 9, but I had a strange problem, which I'm trying to understand, basically to get the device up and running the following files:
framework/arm
framework/arm64
framework/oat
framework/*.vdex
I can not figure out if this problem is due to the conversion from .cdex to .dex or maybe it's the new .rel files?
As a premise the script I use works perfectly on android 8.x so I exclude a script problem

I don't understand what exactly is the question? If you have a specific file that you suspect is not decompiled or converted to StandardDex properly, I can have a look if you upload it somewhere. Otherwise I can't really help.
I've used decompiled & converted files in other projects I'm working on and so far I haven't noticed any issues.

I close it is probably an error of my tool thank you for answering


fix the bug of the return value of conn.WriteTo

Change-Id: I04c0e0b0d4ef9502195b38a545f04c9f1bc305af

name: Pull request
about: Bug fix
title: 'fix the bug of the return value of conn.WriteTo'
labels: ''
assignees: ''

1. Are you opening this pull request for bug-fixes, optimizations or new feature?
bug-fixs
2. Please describe how these code changes achieve your intention.

conn.WriteTo should return the number of bytes actually written
3. Please link to the relevant issues (if any).

https://github.com/panjf2000/gnet/issues/343
4. Which documentation changes (if any) need to be made/updated because of this PR?

connection.go
4. Checklist

[x] I have squashed all insignificant commits.
[x] I have commented my code for explaining package types, values, functions, and non-obvious lines.
[ ] I have written unit tests and verified that all tests passes (if needed).
[ ] I have documented feature info on the README (only when this PR is adding a new feature).
[x] (optional) I am willing to help maintain this change if there are issues with it later.

Thanks~


Unemployment rate template

Create 2 templates(multiple choice and scalar) (the number input field needs to be capped to 1 decimal place...so 4.4 is good, 4.48 is not)
Both scalar and Multiple choice will use the same Resolution Rules Below
Scalar have Min Range 0.0 Max Range 30
Resolution Begins date needs to be the following month of what was named in the market question.  So if March 2020 was named in the market question, Event Expiration would start April 30th, 2020.

Multiple Choice:
Will the seasonally adjusted national Unemployment Rate for [month/year picker] be [enter Number]% or higher according to the US Bureau Labor of Statistics?
Outcome A) [Number]% or Higher
Outcome B) Below [Number]%
Scalar
What will the seasonally adjusted national Unemployment Rate be for [month/year picker] according to the US Bureau Labor of Statistics?
Added Resolution Rules (to be used for both the Multiple Choice and Scalar Market)
Settlement is found using the US Bureau Labor of Statistics website (https://www.bls.gov/).  You can go directly to https://data.bls.gov/timeseries/LNS14000000 to find the market settlement number, go to the Year/Month stated in the market question in the chart.

Added -closed


Further specializations for setindex!!

Ref: https://github.com/JuliaFolds2/BangBang.jl/issues/21#issuecomment-2067788205
This PR adds further specializations to possible(::typeof(__setindex!), ...) to improve selection of mutating version.
For example:
xs = Vector{Vector{Float64}}(undef, 2)
setindex!!(xs, [1.0], 1)

is not mutating on #master but is on this branch.

@MasonProtter Would you be able to have a look at this?:)

@torfjelde can you re-trigger CI? I hope the new version of IRTools fixes the failure

Alternatively you can just remove the nightly tests

Uhmm am I blind? :eyes: I can't find the re-run checks button :grimacing: Or is it maybe reserved for org supers?

Thank you @MasonProtter !
Would it be possible to also make a release?


Update installation instructions to include scipy in list of required packages

Encountered bug when using subcube_from_regions method where scipy could not be installed. Scipy was not included in required packages.

Both scipy and regions are listed as optional requirements in setup.cfg.  Regions is listed as an optional dependency here:
https://spectral-cube.readthedocs.io/en/latest/installing.html#requirements
but scipy isn't and should be.
The relevant lines that require scipy are:
https://github.com/radio-astro-tools/spectral-cube/blob/606b1eff79c8e3ecfbb8cb9b5af3a4fa8d6efbcd/spectral_cube/spectral_cube.py#L1884-L1885

Closed by #724


ENH: make it easier to manage volume properties

We turn off auto-creation of volume property nodes
so that the user is required to make them by hand
before running the animation.
This now works for me:

load data
turn on volume rendering
create two extra volume properties and initialize them from presets and then tweak the params as needed
go to Animator
add the effect and pick the volume properties via the gui

Does it work the same way for ROI action as well (i.e., no ROI creation)?

Does it work the same way for ROI action as well (i.e., no ROI creation)?

No, but if you like this behavior we could make that behave the same.


Unable to work on android 10

I tested the app on my Nokia 6.1 Plus (android one device) running on stock ROM android 10 and I noticed that it's only the launch an app feature that works.
The snapshot feature is not working, when I swipe left in home screen, nothing is shown, it behaves as if the "show google app" was disabled. Below are the logs:

discoverkiller-overlay-logs-2022-03-26T17_39_08.834.txt
--------- beginning of system
--------- beginning of crash
--------- beginning of main
03-26 17:38:35.164 20723 20723 I OverlaySController: Calling with an invalid UID, the interface will not work 10113 vs 10156
03-26 17:38:35.223 20723 20723 E OverlaySController: Error creating overlay window
03-26 17:38:35.223 20723 20723 E OverlaySController: java.lang.NoSuchMethodError: No virtual method handleStartActivity(Landroid/os/IBinder;Landroid/app/servertransaction/PendingTransactionActions;)V in class Landroid/app/ActivityThread; or its super classes (declaration of 'android.app.ActivityThread' appears in /system/framework/framework.jar)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.app.LocalActivityManagerCompat$ActivityThreadCompat.handleStartActivity(LocalActivityManagerCompat.java:784)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.app.LocalActivityManagerCompat$ActivityThreadCompat.access$000(LocalActivityManagerCompat.java:774)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.app.LocalActivityManagerCompat.moveToState(LocalActivityManagerCompat.java:203)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.app.LocalActivityManagerCompat.startActivity(LocalActivityManagerCompat.java:477)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at com.kieronquinn.app.discoverkiller.ui.screens.overlay.snapshot.SnapshotOverlay.onCreate(SnapshotOverlay.kt:130)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at com.google.android.libraries.gsa.overlay.callbacks.OverlayControllerCallback.handleMessage(OverlayControllerCallback.kt:101)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at com.google.android.libraries.gsa.overlay.callbacks.MinusOneOverlayCallback.handleMessage(MinusOneOverlayCallback.kt:28)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.os.Handler.dispatchMessage(Handler.java:103)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.os.Looper.loop(Looper.java:214)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at android.app.ActivityThread.main(ActivityThread.java:7397)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at java.lang.reflect.Method.invoke(Native Method)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492)
03-26 17:38:35.223 20723 20723 E OverlaySController: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:935)

There's probably not much point of me fixing this, as Snapshot is being killed off anyway

Kindly fix mate. Recently your module was marked as one of the best Xposed module by an article from XDA and there may be others in my category who would like to test this feature before it gets nuked. It would be said we didn't get a chance


Does kafkajs support broker names with domains?

While I am rummaging through the src to see how the broker string is parsed I figure I'd ask if I'm going down the wrong hole.
When I deploy kafka in k8s in the same namespace I can point kafkajs at kafka-cluster:9092 and it connects and does its thing.
When I deploy kafka in k8s in a different namespace, I can point kafkajs at kafka-cluster.dev:9092 and it does not connect and errors out.
I have verified kafka is up because I can deploy the same service into that same namespace and it will work.
In the logs though when I am in a different namespace, I see the domain dropped from the error output:
kafka:producer creating producer uploader kafka-cluster.dev:9092 +0ms
... other init stuffs, try to publish a message and...
KafkaJSNonRetriableError
  Caused by: KafkaJSConnectionError: Connection error: getaddrinfo ENOTFOUND kafka-cluster
    at Socket.onError (/headless/wrk/fetcher/node_modules/kafkajs/src/network/connection.js:152:23)
    at Socket.emit (events.js:314:20)
    at emitErrorNT (internal/streams/destroy.js:92:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:60:3)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  name: 'KafkaJSNumberOfRetriesExceeded',
  retriable: false,
  helpUrl: undefined,
  originalError: KafkaJSConnectionError: Connection error: getaddrinfo ENOTFOUND kafka-cluster
      at Socket.onError (/headless/wrk/fetcher/node_modules/kafkajs/src/network/connection.js:152:23)
      at Socket.emit (events.js:314:20)
      at emitErrorNT (internal/streams/destroy.js:92:8)
      at emitErrorAndCloseNT (internal/streams/destroy.js:60:3)
      at processTicksAndRejections (internal/process/task_queues.js:84:21) {
    retriable: true,
    helpUrl: undefined,
    broker: 'kafka-cluster:9092',
    code: 'ENOTFOUND'
  },
  retryCount: 3,
  retryTime: 2400
} +1ms

Turned on debug logging to see if there was more information.
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.949Z","logger":"kafkajs","message":"[Connection] Request Metadata(key: 3, version: 6)","broker":"kafka-cluster.dev:9092","clientId":"uploader","correlationId":1,"expectResponse":true,"size":40}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.952Z","logger":"kafkajs","message":"[Connection] Response Metadata(key: 3, version: 6)","broker":"kafka-cluster.dev:9092","clientId":"uploader","correlationId":1,"size":119,"data":{"throttleTime":0,"brokers":[{"nodeId":1,"host":"kafka-cluster","port":9092,"rack":null}],"clusterId":"wcOdsadtTPyOz7Z7mJsJbw","controllerId":1,"topicMetadata":[{"topicErrorCode":0,"topic":"coordinator","isInternal":false,"partitionMetadata":[{"partitionErrorCode":0,"partitionId":0,"leader":1,"replicas":[1],"isr":[1],"offlineReplicas":[]}]}],"clientSideThrottleTime":0}}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.955Z","logger":"kafkajs","message":"[Connection] Connecting","broker":"kafka-cluster:9092","clientId":"uploader","ssl":false,"sasl":false}
{"level":"ERROR","timestamp":"2021-05-06T01:52:11.968Z","logger":"kafkajs","message":"[Connection] Connection error: getaddrinfo ENOTFOUND kafka-cluster","broker":"kafka-cluster:9092","clientId":"uploader","stack":"Error: getaddrinfo ENOTFOUND kafka-cluster\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:66:26)"}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.969Z","logger":"kafkajs","message":"[Connection] disconnecting...","broker":"kafka-cluster:9092","clientId":"uploader"}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.969Z","logger":"kafkajs","message":"[Connection] disconnected","broker":"kafka-cluster:9092","clientId":"uploader"}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.970Z","logger":"kafkajs","message":"[Connection] disconnecting...","broker":"kafka-cluster:9092","clientId":"uploader"}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.970Z","logger":"kafkajs","message":"[Connection] disconnected","broker":"kafka-cluster:9092","clientId":"uploader"}
{"level":"DEBUG","timestamp":"2021-05-06T01:52:11.970Z","logger":"kafkajs","message":"[Connection] Connecting","broker":"kafka-cluster:9092","clientId":"uploader","ssl":false,"sasl":false}
{"level":"ERROR","timestamp":"2021-05-06T01:52:11.977Z","logger":"kafkajs","message":"[Connection] Connection error: getaddrinfo ENOTFOUND kafka-cluster","broker":"kafka-cluster:9092","clientId":"uploader","stack":"Error: getaddrinfo ENOTFOUND kafka-cluster\n    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:66:26)"}

alright, for anyone else that runs into this, what is happening is the kafkajs code IS using the full dns, reaching out to kafka, and in this case the kafka prop KAFKA_ADVERTIZED_ADDRESS  was set to kafka-cluster instead of kafka-cluster.dev as it should. So when kafkajs reached out, got told to connect to just kafka-cluster, obviously that wouldnt work.
Thank you all for entertaining my digging and frustration, and thanks again for an awesome kafka implementation!