Snap Install not working - hangs/no-output

Description
Hi, We are using snap to install node - sudo snap install node --classic --channel=${{ inputs.node_version }}
during the github workflow run, The command just stops and operation is cancelled after 15 mins of timeout. After rerun it sometime works.
Operating System
Ubuntu - 22.04.5 LTS
Runner Image
Image: ubuntu-22.04
Version: 20241201.1.0
Platforms affected

[ ] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Image version and build link
20241201.1.0

Is it regression?
no idea
Expected behavior
It should quickly install node and move on to next step
Actual behavior
stuck on this command, node install
Repro steps
sudo snap install node --classic --channel=${{ inputs.node_version }}
this command is being used in a GHA build step.

Hi @nayan-shrivastava-dazn , Thankyou for raising the issue with us , we will look into the issue and update you on this Asap.

Hi @nayan-shrivastava-dazn, We appreciate your patience. Please attempt the workaround provided below, Hope it will resolve the issue. Thank you.
name: Install Node.js using Snap

on: [push, pull_request]

jobs:
  install-node:
    runs-on: ubuntu-22.04
    steps:
      - name: Checkout repository
        uses: actions/checkout@v4

      - name: Install Node.js
        run: sudo snap install node --classic --channel=${{ inputs.node_version }}

      - name: Verify Node.js version
        run: node -v


Low accuracy

Why the accuracy is low even for a very clear image? Note: i use default code and training data

It's improved doing manual training on the font we used.
Thanks;


Remove ISummaryConfiguration from IClientConfiguration

As continuation from #9990 , we need to remove ISummaryConfiguration  from IClientConfiguration and contact Gary so it gets removed from the server.
export interface IClientConfiguration {
// Max message size the server will accept before requiring chunking
maxMessageSize: number;
// Server defined ideal block size for storing snapshots
blockSize: number;

// Summary algorithm configuration. This is sent to clients when they connect
summary: ISummaryConfiguration;

}

Will track it on https://dev.azure.com/fluidframework/internal/_workitems/edit/412


Malformed [?] Streams kill engine

Streams have been popping up like this one 07fc9479782a3a3be5fd18d91611542986a2fdd0 that are killing (hanging)the engine and as the player does not reload the engine or even close it when exited it is causing many a frustrated user that just thinks the player doesn't work.
Should be a simple fix.

@Hueristic Livestreamer has been abandoned try out Streamlink
it's an active developed fork of livestreamer, comes with many improvements and bug fixes.

@Hueristic what stream is that?  Is it an acestream?


[BACKLOG-39677] Added code changes for local VFS  connection.

 Note:

Frogbot also supports Contextual Analysis, Secret Detection, IaC and SAST Vulnerabilities Scanning. This features are included as part of the JFrog Advanced Security package, which isn't enabled on your system.

🐸 JFrog Frogbot

:white_check_mark: Build finished in 20m 9s
Build command:
mvn clean verify -B -e -Daudit -Djs.no.sandbox -pl core

:ok_hand: All tests passed!
Tests run: 1277, Failures: 0, Skipped: 1    Test Results

:information_source:  This is an automatic message


[feature] Support for Terraform Cloud - Enterprise

Already supported scenario

[X]  1. Create the TFE/TFC environment (organization, variables, workspaces)
[x]  1. Use TFC as remote backend (local execution, backend stored in TFC) - https://github.com/aztfmod/rover/pull/101

Future scenarios:

[ ]  Use TFC with Terraform Cloud Agents (https://www.terraform.io/docs/cloud/workspaces/agent.html)
[ ]  Deploy Terraform Enterprise Server and remote agents
[ ]  Use TFE in online mode (execution in TFE with remote agents)

Is it going to be possible to use terraform cloud as an alternative for Azure Storage Accounts and Keyvault in the level 0 landing zone?


Protocol documentation?

Hi! I'm looking for docs to implement this in other languages. My understanding is that it's DHT + MDNS, but I'm not sure how straightforward it is. Are there any resources y'all could recommend for making Hyperswarm work outside of Node.js? Thanks! :star:

I’m in the middle of documenting it right now :) We’ll be putting up a website as well. Which languages are you porting it to?

Great! I'm not actively working on porting to any language, but after investing heaps of time learning the SSB stack I'm hesitant to adopt a reference implementation before I can understand how hard it'd be to implement from scratch. Hyperswarm is lovely to work with because it Just Works, but I don't actually know what's Just Working and so I figured I'd reach out and start a chat.
Thanks for all of your work on this project!

Makes tons of sense. The DHT protocol is not too complicated (at least compared to the rest of the Hyper stack).
Since there’s growing interest I’ll try to just get the content up and style it later to be all nice and stuff :)

Sounds great, looking forward to checking it at your convenience.

Hi, I'm checking back in on this -- are there any resources I should check out, or are the protocol docs still WIP?

Yes, just wanted to chime in that this would be utterly fantastic (having the python implementation in mind...) :sun_with_face:

I'm considering to port this specific implementation to python :D Keep up the great work!

Hey @amoallim15 if you get started and want to chat, I've been working on porting the hyper stack to python over at https://hyperpy.decentral1.se so would happily try to help out with your effort! Would be cool to see more python stuff in this space.

@mafintosh
I was working on a Go implementation last year and have just come back to working on the project.  The changes from v2.x.x to v3.x.x eliminate the use of protobuf and so the above referenced documentation is more important.  What's the status of the protocol documentation?  Would you want help with it?  Or proof-reading it?  (I'm a long-time protocol engineer and worked on the DOCSIS 3.0 and 3.1 specifications so I've got an eye tuned towards separating the protocol specifics from the implementation.


INTENG-5493 -- Refactor Install Referrer Wrapper

Reference
INTENG-5493 -- Hidden-price Branch Android SDK proguard error.
Description
While investigating proguard issues around the Branch Install Referrer Wrapper, I was thinking that maybe the issue was around a static inner class that looked suspicious.
It turns out that this was not the root problem, however I did refactor the static inner class out into its own class such that we have one less static to worry about.
Testing Instructions

Sample app should continue to function.
Unit tests should all pass

Risk Assessment [LOW]
This is just a refactor of an inner class to a class in its own right.

[X] I, the PR creator, have tested — integration, unit, or otherwise — this code.

Reviewer Checklist (To be checked off by the reviewer only)

[ ] JIRA Ticket is referenced in MR title
Correctness & Style

[ ] Conforms to Style Guides
[ ] Mission critical pieces are documented in code and out of code as needed

[ ] Unit Tests reviewed and test issue sufficiently
[ ] Functionality was reviewed in QA independently by another engineer on the team (clone for each required reviewer)

I had the wrong base -- I would like approval to merge this to Staging

👍


large_icon now showing when application is in Background or Closed

I am sending this:
custom_notification: {  title: 'a', body: 'b', show_in_foreground: true, large_icon:'https://image.freepik.com/free-icon/small-boy-cartoon_318-38077.jpg' }
It displays the image when the application is in foreground, but not when the application is in Background or closed.

do you mean there is no notification at all or just the image is not showing?
does regular notification work in background?

Hi, thanks for the quick response, really appreciated, and congrats for the lib.. it's awesome.
Yes the notification is showing, but is not considering the image I am sending, it displaying the one I was displaying before I add the large_icon property.
BTW I am using your sample app, testing in Android, device Samsung Galaxy 5.

ok the issue is it is trying to make network request on main thread on download the image. finding solution now

Thanks!!

fixed in v9.3.0

Thank you very much.
When will be available?

it is already. also sub_text is in example as well


[#2552] feat(catalog-jdbc-doris): add Doris container implement for integration-test

What changes were proposed in this pull request?
add Doris container implement for integration-test
Why are the changes needed?
Fix: #2552
Does this PR introduce any user-facing change?
N/A
How was this patch tested?
UT

@yuqi1129 @mchades @qqqttt123   I had fix integration test, and update this PR.
Please help to review this PR again when you have time.

@mchades do you have any further advice?

Thanks for your hard work.


Model request: internet router geo-location based on hostname

Hi,
I am the author of Trippy (https://github.com/fujiapple852/trippy) which is a OSS traceroute/mtr like tool (which also uses Ratatui!).
I would like to propose a model which is trained on a combination of DNS and GeoIp data and can be used for geo-locating IPs based on hostnames (and perhaps AS names).
Trippy currently supports GeoIp lookup (i.e. IP address -> geo location) using mmdb database files from MaxMind and IPinfo. These databases are useful but are both incomplete and inaccurate for many IPs.
Another technique often used to geo-locate IPs is to lookup the reverse DNS hostname (and sometimes the AS name) as these often contain clues as to the location. For example, the hostname xe-11-1-0.edge1.NewYork1.Level3.net is likely to be located in New York.
Typically these are interpreted by humans eyeballing the hostnames, and sometimes these are fuzzy matched by tools against sets of known country/city codes and/or hostname formats. This approach is high maintenance and has limited utility. See slide 11-16 of this presentation for examples of the types of codes used in hostnames for internet routers.
I believe it may be possible to train a model to do this using the large quality of DNS and GoeIp data available.
A large data set could be created which contains the following:

Field
Example

IP
<IP_ADDRESS>

Hostname
CS.stanford.edu

AS Name
AS32 STANFORD, US

Geo Location
Los Altos, California, United States, North America

This could then be used to train a model with Burn which is able to take a hostname (or hostname + AS name) and predict the geo-location to the country/city level.

Don't see the model you want? Don't hesitate to open an issue, and we may prioritize it.

I don't know if this is of interest to anyone, but I though i'd try my luck and ask! If such a model were to exist I would be keen to integrate the functionality into Trippy.

Hey 👋
Thanks for your interest and for sharing Trippy - I had not heard of the project before. Looks really cool!
This looks like a really interesting application, and it definitely could be built with Burn 😄
However, this request goes a little beyond the intended scope 😅 Model requests are intended for existing architectures or pre-trained models that could be added relatively easily.
Building a custom model from scratch — especially without an existing dataset — would require a lot more effort for development and training.
I'm going to keep this open (who knows, a community-driven effort might arise around the idea), but just wanted to let you know that we are not actively going to work on this. If someone decides to lead the effort, this can serve as a discussion board 🙂 And don't hesitate to ask questions on discord! We'll be happy to help and follow the progress.

Thanks a lot for responding and for leaving this open, appreciate the clarity!
For anyone who sees this (and is motivated to try), I can certainly help preparing the dataset and with the integration.


1.3 beta three step compass calibration wrong data.

I've tried several times to calibrate the compass with the three step process in the 1.3 beta, every time I get the heading off by about 30deg.
I've calibrated the compass using the stable firmware and the six point process and it works every time.

Calibration method (looking good):
https://youtu.be/OOKthp_VciE
Good compass cal params:
https://files.gitter.im/PX4/Firmware/jj8F/px4_stable.params
Bad compass cal params:
https://files.gitter.im/PX4/Firmware/lDoq/px4_13_beta.params

If my latest changes on master fix it for Greg we can stick to 3 positions, if not we'll return to 6 positions.

I've done some tests and this method gives a good result
1.Hold the craft horizontaly. turn round the yaw axis.
2.Hold the craft nose down. turn round the roll axis.
3.Hold the craft nose down. Turn round the pitch axis.
This was suggested by Lorenz, I'm testing it to find out how reliable it is, so far it works very well.


Keda scaled job pods are prematurely terminated by Karpenter

Description
Observed Behavior:
We are running the Generative AI workloads using Keda-scaled jobs. But noticed that Keda scaled job pods are prematurely terminated by Karpenter after 14 mins.
For example, I placed a message in an SQS queue, which triggered a Keda job to start a pod. This pod ran for 14 minutes before being terminated.
We have set the following annotation for Karpenter not to disrupt GPU nodes on both the Scaled job and its pod as well.
annotations:
    karpenter.sh/do-not-disrupt: "true"

Karpenter logs:
{"level":"INFO","time":"2024-06-07T14:04:33.106Z","logger":"controller.provisioner","message":"created nodeclaim","commit":"3ec00e6","nodepool":"gpu","nodeclaim":"gpu-86bv5","requests":{"cpu":"190m","memory"
:"248Mi","[nvidia.com/gpu](http://nvidia.com/gpu)":"1","pods":"9"},"instance-types":"g4dn.xlarge"}
{"level":"INFO","time":"2024-06-07T14:04:34.857Z","logger":"controller.nodeclaim.lifecycle","message":"launched nodeclaim","commit":"3ec00e6","nodeclaim":"gpu-86bv5","provider-id":"aws:///us-east-2c/i-02a685
532f5eba4e2","instance-type":"g4dn.xlarge","zone":"us-east-2c","capacity-type":"on-demand","allocatable":{"cpu":"3920m","ephemeral-storage":"359Gi","memory":"14481Mi","[nvidia.com/gpu](http://nvidia.com/gpu)":"1","pods":"29","[vpc.am](http://vpc.am/)
[azonaws.com/pod-eni](http://azonaws.com/pod-eni)":"39"}}
{"level":"INFO","time":"2024-06-07T14:05:26.458Z","logger":"controller.nodeclaim.lifecycle","message":"initialized nodeclaim","commit":"3ec00e6","nodeclaim":"gpu-86bv5","provider-id":"aws:///us-east-2c/i-02a
685532f5eba4e2","node":"ip-10-25-66-161.us-east-2.compute.internal"}
{"level":"INFO","time":"2024-06-07T14:08:59.234Z","logger":"controller.disruption","message":"disrupting via consolidation delete, terminating 1 candidates ip-10-25-66-161.us-east-2.compute.internal/g4dn.xla
rge/on-demand","commit":"3ec00e6","command-id":"517ba768-1cac-41bd-9870-867f3249f34c"}
{"level":"INFO","time":"2024-06-07T14:08:59.601Z","logger":"controller.disruption.queue","message":"command succeeded","commit":"3ec00e6","command-id":"517ba768-1cac-41bd-9870-867f3249f34c"}
{"level":"INFO","time":"2024-06-07T14:08:59.664Z","logger":"controller.node.termination","message":"tainted node","commit":"3ec00e6","node":"ip-10-25-66-161.us-east-2.compute.internal"}
{"level":"INFO","time":"2024-06-07T14:09:02.603Z","logger":"controller.node.termination","message":"deleted node","commit":"3ec00e6","node":"ip-10-25-66-161.us-east-2.compute.internal"}
{"level":"INFO","time":"2024-06-07T14:09:03.025Z","logger":"controller.nodeclaim.termination","message":"deleted nodeclaim","commit":"3ec00e6","nodeclaim":"gpu-86bv5","node":"ip-10-25-66-161.us-east-2.comput
e.internal","provider-id":"aws:///us-east-2c/i-02a685532f5eba4e2"}

Expected Behavior:
Keda-scaled jobs should run successfully without terminating the Keda-scaled job pods.
Reproduction Steps (Please include YAML):
Keda scaled job manifest file details.
apiVersion: v1
kind: ServiceAccount
metadata:
  name: video-summarization
  namespace: gen-ai
  annotations:
    eks.amazonaws.com/role-arn: arn:aws:iam::1234:role/gen-ai-video-summarization-pod
---
apiVersion: keda.sh/v1alpha1
kind: ScaledJob
metadata:
  name: video-summarization-job
  namespace: gen-ai
  annotations:
    karpenter.sh/do-not-disrupt: "true"
spec:
  jobTargetRef:
    parallelism: 1
    template:
      metadata:
	annotations:
	    karpenter.sh/do-not-disrupt: "true"
      spec:
        serviceAccountName: video-summarization
        tolerations:
          - key: "accrete.ai/compute-type"
            operator: "Equal"
            value: "gpu"
            effect: "NoSchedule"
        nodeSelector:
          accrete.ai/compute-type: gpu
        hostPID: true
        containers:
        - name: video-summarization-job
          image: <image-details>
          imagePullPolicy: Always
          command: ["python", "main.py"]
          env:
            - name: REQUEST_QUEUE_URL
              value: "https://sqs.us-east-2.amazonaws.com/1234/video-summarization-batch-job-input-queue"
            - name: RESPONSE_QUEUE_URL
              value: "https://sqs.us-east-2.amazonaws.com/1234/video-summarization-batch-job-output-queue"
            - name: AWS_REGION
              value: "us-east-2"
          resources:
            requests:
              nvidia.com/gpu: 1
            limits:
              nvidia.com/gpu: 1
        restartPolicy: Never
  pollingInterval: 60
  maxReplicaCount: 10
  triggers:
    - type: aws-sqs-queue
      metadata:
        queueURL: "https://sqs.us-east-2.amazonaws.com/1234/video-summarization-batch-job-input-queue"
        queueLength: "1"
        awsRegion: us-east-2
      authenticationRef:
        name: video-summarization-keda-job-triggerauth
---
apiVersion: keda.sh/v1alpha1
kind: TriggerAuthentication
metadata:
  name: video-summarization-keda-job-triggerauth
  namespace: gen-ai
spec:
  podIdentity:
    provider: aws
    identityOwner: workload

Versions:

Kubernetes Version (kubectl version):
Client Version: v1.29.0
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.4-eks-036c24b

Karpenter Version: 0.34.5

Keda Version: ghcr.io/kedacore/keda:2.13.0

This is https://github.com/kubernetes-sigs/karpenter/issues/1167 and should be resolved in v0.36.2 or v0.37.0. Can you upgrade and confirm that the issue no longer appears?

/triage needs-information

Yeah, this issue is related to Keda but not the Karpenter.


upload fails on iOS 10.1.1

Upload was working on iOS 9, but now upload button shows preview of image, but data is never uploaded.

Same here


Chmod mockPrinter

I was going through the wiki and clicked in the link to download the executable for the mock printer, I then ran into a missing permission:
 ./MvRxMockPrinter
 zsh: permission denied: ./MvRxMockPrinter
 miken@Mikes-Mac Downloads % sudo ./MvRxMockPrinter
 sudo: ./MvRxMockPrinter: command not found

Chmod did the trick

Fixed in beta1


Only create Athenz resource group once for each appliation name

FYI @bratseth

It's annoying that the Athenz API are not idempotent

👍


Where could find the souce code of the function[cv2.projectPoinst()]?

I want debug the function cv2.projectPoints and get some process result. But couldn't the the python vesion source code of it.
Thx

Python bindings for OpenCV are generated during build procedure in C++. There is no Python code for the function.

OK,got it,I have found the C++ souce code of this function. Thanks.
Additional, I have a new question.
For the C++ source code:Python bindings for OpenCV could be generated during build procedure in C++.
For some python source code: C++ bindings for it could be generated during build procedure in python？
Thanks again

Package setup.py builds full library from C++. Python code distributed with the package has C++ "back-end" and just adapt some C++ abstractions for Python users.

Yeah，I got it, But I want to put some python code into c++ code, so the python version could be built into c++ lib directly or not?Thanks!

That's not how it works. If you would like to add a new feature in OpenCV, you need to go and code it in C++ to the upstream repository.
Of course, you can always extend opencv-python by creating your own open-source package that wraps around opencv-python. But embedding Python into C++ is not feasible.

OK，got it，Thank you very much for the quickly and detail response


feat: 🎸add useKeyPress hook

First of all, thanks for the amazing project 🚀
This PR basically adds a hook for detecting when the user is pressing a specific key on their keyboard – pretty much like this sample here does. The key difference it also allows the so called key combos to be set – e.g. firing something specifically when a very weird combination of keys is pressed.
Turns out that handling multiple key presses at once can grow a lot of code – so I used a very popular alternative for that.
Then comes the tradeoff of adding an extra dependency and growing your bundle. I handle this by dynamically importing KeyboardJS only if a useKeyboardJS option is set true when the hook is called – so, no extra KBs on the bundle for simple use cases.
Screen capture on docs + working sample available here.

:tada: This PR is included in version 5.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Thanks 😄


Is this Open GL Mapper still necessary without the Map window?

Wondering if this is now invalid with the choice to remove the map window.
https://github.com/HydrologicEngineeringCenter/HEC-FDA/blob/b23441b56fe7b5da9950cfb467925fb176ebf06f/ViewModel/Watershed/TerrainElement.cs#L17

Nice catch. This led to several map related events and lines of code in base classes that I was able to delete.


Fix the location of where to find the CA key.

Running sign.sh $NAME now successfully completes, instead of throwing an error.

Thanks!


Display the version of the IBM Operator Collection SDK that is installed

Feature Request
Describe the problem you need a feature to resolve.
Display the version of the IBM Operator Collection SDK that is installed
Describe the solution you'd like.
Display the version somewhere in the IBM OPERATOR COLLECTION SDK box at the top left of the screen, and/or on the hover help for the operator collection SDK symbol on the left.

Fixed in PR https://github.com/IBM/operator-collection-sdk-vscode-extension/pull/68


Updated validations.py python script

Fixed the behaviour of validate_user function in validations.py
Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


bump deps; cull redundant peer deps

bump devDeps for ESLint 2.0, etc (fixes #8)

reduce peerDeps down to just eslint-config-standard (bare essentials)

no need to specify peerDeps that are already specified as peerDeps by eslint-config-standard

+1 this makes a lot of sense, especially trimming down the peer deps

@Flet ? any thoughts?

I was kinda waiting for standard v7 to release before bumping semistandard...
https://github.com/feross/standard/issues/404

Ah, okay. Yeah, that makes sense.

I would be fine with merging this if the standard versions are kept the same for now.
Also, does it make sense to switch the standard deps to just dependencies?

I made a separate PR: #9
I don't believe it is necessary or beneficial to move the standard deps from devDeps to deps. That would cause every downstream project that uses eslint-config-semistandard to also download standard, for example, which is unnecessary.

I notice that you lint your code with https://www.npmjs.com/package/standard .
Would you accept a PR that uses ESLint instead? That would mean we could drop 1 depDep.

Just rebased this on master, and made the commit match the scope of the changes here

feross/standard is since 6.0.5 compatible with eslint 2.0.0:
https://github.com/feross/standard/blob/master/CHANGELOG.md#605---2016-02-12
So there is no need to wait for v7.


Standby transaction legacy

Description

Move current stand-by transaction logic to a "legacy" code.
Display new stand-by transactions in the feed.
Add a stand-by transaction in sending transactions.

Tested
Manually in a emulator + unit tests
How others should test
Anyone should be able to see the new stand-by transaction after doing a transaction. Make sure you have the ff on for the new flow.
Related issues

Fixes some bullet points on #1573

Backwards compatibility
Compatible

Hey @MuckT, Need clarification for standby transaction,We are assuming that when we buy CELO than at first Confirming activity appears, before getting completed. Can we call it as stand by transaction ?
If not can you please let us know what is stand by transaction.
thanks..!!


proposal: add static cache

This is WIP since it's a code proposal.
This intends to get rid of the arguments "do not use Node for static file serving", as software like Nginx does cached reading itself, in order to not call the FS. I suggest having this because it's a very simple yet powerful addition.
This needs some more checking in the code and obviously tests.
da4b549 adds benchmarks
0d34b27 is the implementation
cc @dougwilson
this is also a prelude to my comment here https://github.com/expressjs/serve-static/issues/62#issuecomment-325443953
See a simple, yet not averaged out benchmark here.

Failing tests because of missing lang features on old Node versions. Easily fixable.

What are the arguments against using Node.js for file serving? Thr only thing I am aware of in that regard is that Node.js doesn't have async file I/O; instead all file operations are sync on a background thread pool that only has 4 threads and is shared by crypto and dns operations, so they can starve each other potentially, but I'm not sure if that actually happens in practice.

Anyway, there are a lot of issues with the actual implementation, though it is a WIP, so that's not an issue. Let me know what you're looking for out of this PR :)

The argument against Node comes from many guides that are perceived as best practice. E.g. the "use nginx for static files" topic. I would discourage this in teams, because it's another thing that might just break or can be misconfigured.
The biggest reason for using Node like this here are:

a fs read not made is the best fs read.
fs reads of files that will likely not change (everything related to websites) or are compiled (e.g. webpack toolchains) are redundant.
(without ref) servers written in system programming languages can and do make use of fs features like page caches, that Node doesn't.
there is natural JS and C overhead to fs calls, which amounts to the performance degradation seen in the picture.
Node's event loop and thread pool can be easily overwhelmed by too many file reads, eventually blocking it.

Some of the guides are even near to the repo, as in express or send
@dougwilson I am happy with any implementation and/or guidance in order to change the implementation above. It would be just very important to me to complement the ecosystem with such a thing, to ease the "use another technology" argument. If you think that is worth it I will work on it the coming week.

I think it is worth it, though as a separate module.

It probably needs some kind plugin architecture here to call the two functions for storing and reading out the cache.
Woud you take it into docs?

The plugin architecture here would imo be charming, because it would also cater for #62 (comment)

If you would like to pursue landing into this module, I can take a better look at your PR here in the coming weeks. As for the plugin architecture, I'm not sure I understand; if it was separate, it would simply act as a replacement middleware to drop in instead of serve-serve static.

And yes, I'm happy to add anything to the readme, including alternatives and why someone would want to use it, etc.

A drop in replacement doesn't seem to valuable too me, because I don't want to promote modules, but rather enhance and help maintain an existing ecosystem. Imo, here it would okay having it, because it's opt-in, but you call the shots.
Re: plugin architecture: I was thinking of something like express.
serveStatic.use(require('serve-static-cache))

Where serve-static-cache could expose two functions that will be called on hooks like beforeRead and beforeSend. Those would correspond to the two code insertions above.
Let's do it like that: I work on the implementation plus tests, because it's a good stab anyhow. I keep in mind that factoring into a module if scopes don't align and we will see afterwards.
If you have some time: Do you have any guidance on API and implementation that would make it more fitting to the surrounding code?

Yea, I'm still trying to visualize your idea, so need some time to think on it. If it helps, the main issue with the implementation here is that after the first response, all future responses are missing the headers like Content-Type and no longer honor If-Modified-Since etc. Also if the first request for a filr is a ranged request, the stream you'll save the data from will only be the requested slice of the file so all future requests will never get more than that first slice. Hopefully some of that will help in thinking through how the hooks will work.

Sorry for inactivity


Lab1 - Fibonacci function

Note: I understand storing a map of all the calculated fibonacci numbers is inefficient given the criteria is only to print them, so I have over-engineered here.
Fixes #193
Review of colleague's PR #
Changes proposed in this PR:

Solution to lab1 - Fibonacci sequence
Tests for the fib function

-

Hi Jake, can I do the review your PR once it's passed CI? For now, can you please remove 'Ready for review' label?

Codecov Report

Merging #195 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #195   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      3    +1
  Lines          18     36   +18
=====================================
+ Hits           18     36   +18

Impacted Files
Coverage Δ

01_fib/Jake88/main.go
100% <100%> (ø)

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 9605cdc...cc9a4d2. Read the comment docs.

Hi Jake, can I do the review your PR once it's passed CI? For now, can you please remove 'Ready for review' label?

More than happy for you to review my changes! :) I've fixed up my typo now, so go for gold.

Hi Jake,
All looks good. Few things to consider:

Map would occupy more computing resources compare to array or slice, so computing a larger fib number might have efficiency differences?
If you compute large fib number, you might get integer overflow from int, so change it to a larger native type.
A good programming practices is seperation of concerns. You could have a function that generate the map and another function does the print. So that these two functions can be resued if the program gets larger.

Hi Jake,
All looks good. Few things to consider:

Map would occupy more computing resources compare to array or slice, so computing a larger fib number might have efficiency differences?
If you compute large fib number, you might get integer overflow from int, so change it to a larger native type.
A good programming practices is seperation of concerns. You could have a function that generate the map and another function does the print. So that these two functions can be resued if the program gets larger.

Thanks!

You're right here. In hindsight I should have stuck to the criteria and just stored the last two numbers in the sequence, since we don't need to retain or return the full list!
I didn't really think about error handling or scaling too high! I should have tried to break it first and then I would have seen these gaps. ;) Good spot.
Sure. I think my fib function is definitely more complex than it needs to be. I guess if the program needed to scale I would look at taking reusable pieces and putting them into their own function.

Please refrain from merging master into your branch until you have the merge me label added by an instructor.


Getter function API instability

Getter functions aren't working correctly across all platforms using the current (implot ~v0.8) binaries (see #18), due to a bug with the wrapper function prototype in cimplot, which has already been fixed upstream. This means getter functions currently work only on Linux and segfault on other platforms. It should be resolved with the upgrade to the binaries in cimguipack_jll which uses the fixed wrappers.

By the way, I've tried it on Ubuntu 20.04.1 LTS and had the same problem.
signal (11): Segmentation fault
in expression starting at none:0
unsafe_store! at ./pointer.jl:118 [inlined]
unsafe_store! at ./pointer.jl:118 [inlined]
SineWave at /home/gvg/.julia/dev/ImPlot/demo/implot_demo.jl:44
unknown function (ip: 0x7f6df168db41)
_Z7WrapperPvi at /home/gvg/.julia/artifacts/5b2b61e1b0721d0f69fa855d20f464a98f4df73f/lib/libcimplot.so (unknown line)
_ZN6ImPlot9PlotLineGEPKcPF11ImPlotPointPviES3_ii at /home/gvg/.julia/artifacts/5b2b61e1b0721d0f69fa855d20f464a98f4df73f/lib/libcimplot.so (unknown line)
PlotLineG at /home/gvg/.julia/dev/ImPlot/src/libcimplot.jl:1590
PlotLineG at /home/gvg/.julia/dev/ImPlot/src/libcimplot.jl:1590
unknown function (ip: 0x7f6df169f1c5)
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2237 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2419
ShowDemoWindow at /home/gvg/.julia/dev/ImPlot/demo/implot_demo.jl:1203
#3 at /home/gvg/.julia/dev/ImPlot/demo/implot_demo.jl:1952 [inlined]
renderloop at /home/gvg/.julia/dev/ImPlot/demo/Renderer.jl:70
#4 at ./task.jl:411
unknown function (ip: 0x7f6df169d4bc)
_jl_invoke at /buildworker/worker/package_linux64/build/src/gf.c:2237 [inlined]
jl_apply_generic at /buildworker/worker/package_linux64/build/src/gf.c:2419
jl_apply at /buildworker/worker/package_linux64/build/src/julia.h:1703 [inlined]
start_task at /buildworker/worker/package_linux64/build/src/task.c:839
unknown function (ip: (nil))
Allocations: 20839517 (Pool: 20828136; Big: 11381); GC: 27
Segmentation fault (core dumped)

That's because in the last commit before the merge I revised it to how it "should be", based on what we found out (which does indeed segfault on Linux with that error) If you're curious, try checking out v0.20 from the registry and see if it still segfaults (there the demo should be the original and was working for me)


Selection Summary Enhancement

I love the selection summary feature and when I need quick stats for my results it's such a time saver.  It gives me the functionality that I have with SSMS and SSMS Boost.  But there are a couple of statistics missing that I feel would be valuable.
I realize UI space in the bottom bar is limited so I propose this may be better as a tooltip pop-up window.  But the addition of:
Distinct Count
Min
Max
Can be very useful and save time.

@clcridgeview FYI, this will be in our next stable release.


Fix only apply dismissTimeout if required

The docs say that if dismissAfter is false, the timeout won't be applied. This fixes things to actually do that :)

Thanks @joeybaker! I'm sorry you're having trouble. 😦
I will defer to @joeyfigaro because he implemented the new dismissTimeout functionality in #90. Since the new tests from #90 passed on that PR and are also passing on this PR, it looks like the tests might be faulty. If that is the case, you/we should also patch the tests in this PR.

@joeyfigaro, any comments here?
@joeybaker (or @joeyfigaro), would you mind checking the tests for dismissAfter and verifying they are correct? We should follow a red/green/refactor process for adding tests.

@joeybaker @pburtchaell Apologies for not seeing this earlier. I did very little in the way of writing anything for the changes I introduced - I really just brought the changes in from a previous contributor who closed a PR before it could be merged. I'll do some digging this week.

Thanks, @joeyfigaro.

@joeyfigaro @pburtchaell I'll see what I can do, but this week is packed for me.

There is no rush! Thanks all.

@pburtchaell the tests are likely passing because somethings wrong with make.
make: Nothing to be done for `test'.

@pburtchaell FYI: Looks like the tests are broken :(

Unless I'm completely mistaken, all tests are broken.

@joeybaker the tests aren't broken, but rather, it looks like Make is not working. I updated npm test to run Mocha directly instead of running the make script.
Unless I'm mistaken, this feature works as expected and I'm closing the PR. I've verified the test and added a permanent notification to the example.
If this is not the case, can you open a bug report issue?


Find strategy to make Jekyll theme repositories useful templates without forking

I think that creating a cookiecutter (or maybe there is a Ruby equivalent — a question for Henry probably) makes more sense for achieving what the design goal seems to be here, as otherwise all the projects are tied to each other forever (unless you email GitHub support for each individual repo and ask them to sever the "fork" tie between things). This isn't necessarily "bad", but once projects start to deviate then it can start to become a bit strange to have large deviations in the commit history from the parent project and the fork if there isn't any plan for reunification in the future.

Originally posted by @matthewfeickert in https://github.com/UW-Madison-DSI/ospo.wisc.edu/issues/11#issuecomment-2269953934

@cranmer While I'd still like to find a better strategy, for the time being I've enabled the "Template repository" setting in https://github.com/UW-Madison-DSI/uw-jekyll-theme/settings

so that people can create a new repository with the same structure by clicking the "Use this template" button now added, without the problems of having the "fork" connection to the parent project.

GitHub docs: https://docs.github.com/repositories/creating-and-managing-repositories/creating-a-repository-from-a-template

Nice! That’s close to the cookie cutter approach. Thanks!
Message ID: @.***>

unless you email GitHub support for each individual repo and ask them to sever the "fork" tie between things

I have a ticket open with GitHub to detach this repo (https://github.com/UW-Madison-DSI/uw-jekyll-theme/) from its fork parent https://github.com/cranmer/cranmer-lab as the commit history divergence is going to become large without any plan to reconcile them.

My ticket has been resolved and so https://github.com/UW-Madison-DSI/uw-jekyll-theme has now been "extracted to a new network" and so is no longer connected to https://github.com/cranmer/cranmer-lab as a fork.


[Juicer 2.0] Exception in thread "main" java.lang.RuntimeException: Unknown command: statistics

Hi,
I'm using Juicer 2.0 on Ubuntu 18.04. I've used the same command to analyze identical datasets with Juicer 1.6 and Juicer 2.0
Juicer 1.6 finished without issues, but I got those results from Juicer 2.0:
The command (with paths reducted):
/opt/juicer/scripts/juicer.sh -g hg38 -s none -p [path]/reference/GRCh38_EBV.chrom.sizes.tsv -z [path]/reference/GCA_000001405.15_GRCh38_no_alt_analysis_set.fna -D /opt/juicer/ -t 92 -d [sample]
[main] Real time: 6784.211 sec; CPU: 610804.131 sec
(-:  Align of [sample]/splits/sample_pooled.fastq.gz.sam done successfully
[bam_sort_core] merging from 460 files and 92 in-memory blocks...
(-: Finished sorting all sorted files into a single merge.
(-:  Mark duplicates done successfully
awk: fatal: can't open source file `/opt/juicer//scripts/stats_sub.awk' for reading (No such file or directory)
Picked up _JAVA_OPTIONS: -Xmx1024m -Xms1024m
Exception in thread "main" java.lang.RuntimeException: Unknown command: statistics
        at juicebox.tools.HiCTools.main(HiCTools.java:98)
Picked up _JAVA_OPTIONS: -Xmx1024m -Xms1024m
Exception in thread "main" java.lang.RuntimeException: Unknown command: statistics
        at juicebox.tools.HiCTools.main(HiCTools.java:98)
Picked up _JAVA_OPTIONS: -Xmx150000m -Xms150000m
Not including fragment map
Error while reading graphs file: java.io.FileNotFoundException: [sample]/aligned/inter_hists.m (No such file or directory)
Start preprocess
Writing header
Writing body
..............................................................................................................................................................................................................................................
.......................................................................................
Writing footer

Finished preprocess
Done creating .hic file. Normalization not calculated due to -n flag.
To run normalization, run: juicebox addNorm <hicfile>

real    36m57.120s
user    38m25.987s
sys     7m10.443s
Picked up _JAVA_OPTIONS: -Xmx150000m -Xms150000m

Calculating norms for zoom BP_2500000
Calculating norms for zoom BP_1000000
Calculating norms for zoom BP_500000
Calculating norms for zoom BP_250000
Calculating norms for zoom BP_100000
Calculating norms for zoom BP_50000
Calculating norms for zoom BP_25000
Calculating norms for zoom BP_10000
Calculating norms for zoom BP_5000
Calculating norms for zoom BP_2000
Calculating norms for zoom BP_1000
Calculating norms for zoom BP_500
Calculating norms for zoom BP_200
Calculating norms for zoom BP_100java.lang.ArrayIndexOutOfBoundsException: Index 167 out of bounds for length 167
        at juicebox.tools.utils.original.NormalizationCalculations.computeVC(NormalizationCalculations.java:319)
        at juicebox.tools.utils.original.NormalizationVectorUpdater.updateHicFile(NormalizationVectorUpdater.java:161)
        at juicebox.tools.clt.old.AddNorm.run(AddNorm.java:84)
        at juicebox.tools.HiCTools.main(HiCTools.java:96)

I suspect that I might be missing some libraries. Could you help?

Which folder are you running from? (CPU/LSF/SLURM/etc)

I have the same problem as you and hope it can be solved!!!

I think I may have found a solution to this problem.Once I put the latest version of juicer_tools into the CPU/common folder, the problem was solved.Newer juicer_tools releases are posted under Github Releases in the Juicebox repo.
https://github.com/aidenlab/Juicebox/releases
You can click "v2.13.07" to download latest juicer_tools.

@sa501428 I'm using the CPU version.
@Mrhuangc good idea! I was using the one from the wiki.

Juicer2 is still under development, so we do appreciate all the reports from you all. Note that the last stable release for 1.6 is available under Github Releases. I'll add a clearer explanation of this to the main README.
Regarding the bugs:
1 - yes, the Exception in thread "main" java.lang.RuntimeException: Unknown command: statistics at juicebox.tools.HiCTools.main(HiCTools.java:98) error comes from using an old Juicer Tools jar
2 - awk: fatal: can't open source file /opt/juicer//scripts/stats_sub.awk' for reading (No such file or directory)` suggests that the stats_sub.awk file is missing; I will fix this under the CPU repo.

Actually the file does seem to be in the repo; can you check that the repo was cloned correctly?

Hi, the file is present. The issue is that Juicer is looking in /opt/juicer//scripts/ and not /opt/juicer//scripts/common
I will download the new version and let you know.

Ah yes, you are correct. I just pushed a fix. Thanks!!

@Zojka are all the bugs in this issue resolved / can we close it?


Support export of Google Workspace documents

Summary
My use case needs support for downloading Google Sheets to a csv file. So I want to add support for export.
Reference: https://developers.google.com/drive/api/guides/manage-downloads
Closes: #1

Hey @mitrydoug , thanks for your support. I will take a look at this on weekend but IMHO it would be great to add small info about this feature in README.md, but if you can't do it, i'll add it later, no worries.


Error in lya.AttrDict

Thanks for sharing your great works.
When i'm calculating the detection mAP in Pascal VOC 2007 (test set) using python3 code/tasks/test.py --cfg configs/baselines/vgg16_voc2007.yaml  --dataset voc2007test \ --model oicr_lambda_log_distillation \ --load_ckpt snapshots/deepvision2020/oicr_lambda_log_distillation/final.pth，the system prompts me that lya is not installed.
I pip3 install lya，and try again.I encountered the error Import Error: cannot import name AttrDict
https://github.com/luiszeni/Boosted-OICR/blob/cc4f3e00b6e1ed1d0ab17df5d13d9a67d2734bf6/code/tasks/config.py#L1-L2
Is there a problem with the way I installed lya? Could you provide some suggestions

I solved this problem by pip install layered-yaml-attrdict-config
Thank you again~


cargo-about: add darwin dependency

Description of changes
ZHF: https://github.com/NixOS/nixpkgs/issues/309482
Fixes cargo-about build for darwin systems by providing a missing dependency.
Also runs nixfmt-rfc-style and updates lib usage in meta.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 311359 run on aarch64-darwin 1

1 package built:

cargo-about


Retry for all KafkaConnectorTests to stabilize mainBluewhisk

In our tests we encounter every now and then failing tests that fail because of timeouts or other reasons.
Description
This code change tries to recover from these intermittent failures by retrying the failed test. Objective is to stabilize the mainBluewhisk builds.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[X] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[X] I signed an Apache CLA.
[X] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

./gradlew :bluemix:tests:testArmada --tests services.KafkaConnectorTests

> Configure project :bluemix:services:billingService
appArgs are not defined

> Task :bluemix:tests:addWhiskCertificate
Certificate was added to keystore

> Task :bluemix:tests:addStageCertificate
Certificate was added to keystore

> Task :bluemix:tests:addProdLondonCertificate
Certificate was added to keystore

> Task :bluemix:tests:addProdCertificate
Certificate was added to keystore

> Task :bluemix:tests:addOpenWhiskCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixUsSouthMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixUsEastMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixStageMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixStageCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixStage0Certificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixStage01Certificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixOpenWhiskStage1NgBluemixNetCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixOpenWhiskNgBluemixNetCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixOpenWhiskEuGbBluemixNetCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixOpenWhiskEuDeBluemixNetCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixJpTokMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixEuGbtMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addBluemixEuDetMultiDomainCertificate
Certificate was added to keystore

> Task :bluemix:tests:addArmadaUSEastCertificate
Certificate was added to keystore

> Task :bluemix:tests:addArmadaSelfSigned
Certificate was added to keystore

> Task :bluemix:tests:addArmadaLetsEncryptCACertificate
Certificate was added to keystore

> Task :open:tests:buildArtifacts
Test zip artifacts already built, skipping

> Task :bluemix:tests:testArmada

services.KafkaConnectorTests STANDARD_OUT
    Create test topic 'KafkaConnectorTestTopic' with replicationFactor=1

services.KafkaConnectorTests > Kafka connector should send and receive a kafka message which sets up the topic STANDARD_OUT
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@20
    Receiving message from topic.
    Received 1. Took 3418 msec: List(Sun Mar 22 08:03:22 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@21
    Receiving message from topic.
    Received 1. Took 132 msec: List(Sun Mar 22 08:03:26 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@22
    Receiving message from topic.
    Received 1. Took 145 msec: List(Sun Mar 22 08:03:26 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@23
    Receiving message from topic.
    Received 1. Took 132 msec: List(Sun Mar 22 08:03:26 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@24
    Receiving message from topic.
    Received 1. Took 76 msec: List(Sun Mar 22 08:03:26 EDT 2020)

services.KafkaConnectorTests > Kafka connector should send and receive a kafka message which sets up the topic PASSED

services.KafkaConnectorTests > Kafka connector should send and receive a kafka message even after session timeout STANDARD_OUT
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@25
    Receiving message from topic.
    Received 1. Took 150 msec: List(Sun Mar 22 08:03:27 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@26
    Receiving message from topic.
    Received 1. Took 135 msec: List(Sun Mar 22 08:03:27 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@27
    Receiving message from topic.
    Received 2. Took 445 msec: List(Sun Mar 22 08:03:27 EDT 2020, Sun Mar 22 08:03:38 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@28
    Receiving message from topic.
    Received 3. Took 397 msec: List(Sun Mar 22 08:03:27 EDT 2020, Sun Mar 22 08:03:38 EDT 2020, Sun Mar 22 08:03:49 EDT 2020)
    Send message to topic.
    Successfully sent message to topic: KafkaConnectorTestTopic-0@29
    Receiving message from topic.
    Received 1. Took 136 msec: List(Sun Mar 22 08:03:50 EDT 2020)

services.KafkaConnectorTests > Kafka connector should send and receive a kafka message even after session timeout PASSED

BUILD SUCCESSFUL in 1m 47s
58 actionable tasks: 25 executed, 33 up-to-date

Codecov Report

Merging #11 into master will increase coverage by 0.12%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
+ Coverage   78.28%   78.40%   +0.12%
==========================================
  Files         197      197
  Lines        8879     8879
  Branches      603      603
==========================================
+ Hits         6951     6962      +11
+ Misses       1928     1917      -11

Impacted Files
Coverage Δ

...whisk/connector/kafka/KafkaProducerConnector.scala
60.00% <0.00%> (-12.50%)
:arrow_down:

...rg/apache/openwhisk/common/ForcibleSemaphore.scala
84.61% <0.00%> (-3.85%)
:arrow_down:

.../scala/org/apache/openwhisk/utils/Exceptions.scala
40.00% <0.00%> (+20.00%)
:arrow_up:

...whisk/connector/kafka/KafkaConsumerConnector.scala
81.69% <0.00%> (+22.53%)
:arrow_up:

Continue to review full report at Codecov.

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


Incorrect attribution in brace initialization?

(just above the 4th code snippet)
If the default constructor is explicitly declared but marked as deleted, default brace initialization cannot be used:
This looks incorrect. According to cppreference, you can use brace initialization in the presence of deleted/defaulted constructors. What is wrong is the use of brace initialization when user defined constructors are present.
In the code snippet that follows the above statement, error C2280 is correctly marked but attributed incorrectly for aforementioned reasons. If you were to comment out the single parameter constructor, the program would compile correctly.
class class_f {
public:
    class_f() = delete;
    class_f(string x): m_string { x } {}  // commenting this will remove C2280 error
    string m_string;
};
int main()
{
    class_f cf{ "hello" };
    class_f cf1{}; // compiler error C2280: attempting to reference a deleted function
}

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

ID: 441b3ef5-7a22-0496-ddec-6baee7262991
Version Independent ID: 711a8364-a43c-22fc-217a-30c95c5c946b
Content: Brace initialization for classes, structs, and unions
Content Source: docs/cpp/initializing-classes-and-structs-without-constructors-cpp.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

It would also be nice to expand upon the fact that brace initialization can also refer to constructors when appropriate. I don't have a complete grasp of the rules to suggest where/how it can be added.

@AvidDrawer
Thanks for commenting here. I think there's some confusion about "default brace initialization", which here means initialization with an empty pair of braces. The statement above the fourth example isn't saying you're unable to use brace initialization with other user-defined constructors in this situation. It's just saying default brace initialization isn't going to work if the default constructor is explicitly deleted. I still see a C2280 compiler error in your example if I comment out the string-argument constructor in class_f (and the declaration of cf in main).
I'm closing this as not repro, but feel free to argue your case if I've misunderstood your objection; we can reopen it.

@corob-msft  You're right, C++20 still throws compiler errors as you pointed out. I was testing on c++17 and referenced cppreference based on the change logs.
Apologies for the misunderstanding!


Bump JobSet to v0.3.1 after testing integration

JobSet minor version v0.3.0 was released with greatly improved scheduling throughput for large scale training jobs.
JobSet v0.3.1 is currently the latest release, a patch release with a bug fix. We need to update XPK to use the latest JobSet release after performing some testing that everything works as expected and there are no regressions.

/assign


l10n: #save and #save_label undefined

looks like the translations for save and save_label are missing
willing to support if you give me a general direction
looks like the variables were renamed from
download_* to save_*

This will be fixed with future l10n updates, please see https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md

Note that the translations for PDF.js in the l10n folder are imported from the Nightly channel of Mozilla Firefox, such that we don't have to maintain them ourselves. This means that we will not accept pull requests that add new languages and/or modify existing translations, unless the corresponding changes have been made in Mozilla Firefox first.


Issue using yarn_start

I get this error whenever I try to run yarn start.
``
PS D:\startupwebsite> yarn start
yarn run v1.22.17
$ react-scripts start
node:internal/modules/cjs/loader:488
throw e;
^
Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './lib/tokenize' is not defined by "exports" in D:\startupwebsite\node_modules\postcss-safe-parser\node_modules\postcss\package.json
at new NodeError (node:internal/errors:371:5)
at throwExportsNotFound (node:internal/modules/esm/resolve:416:9)
at packageExportsResolve (node:internal/modules/esm/resolve:669:3)
at resolveExports (node:internal/modules/cjs/loader:482:36)
at Function.Module._findPath (node:internal/modules/cjs/loader:522:31)
at Function.Module._resolveFilename (node:internal/modules/cjs/loader:919:27)
at Function.Module._load (node:internal/modules/cjs/loader:778:27)
at Module.require (node:internal/modules/cjs/loader:999:19)
at require (node:internal/modules/cjs/helpers:102:18)
at Object. (D:\startupwebsite\node_modules\postcss-safe-parser\lib\safe-parser.js:1:17) {
code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}
Node.js v17.0.1
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
``

Hey @rkalahasty ,
I can't reproduce this error, just installed the template using npx without running with any issues.


Fix localize_path with custom protocol identifiers

This is a follow-up PR to https://github.com/godotengine/godot/pull/67251. simplify_path is now fixed, but the same issue was caused by ProjectSettings::localize_path when it was called from ResourceLoader.load("test://").

Looks good to me :) cc @KoBeWi

What issue does this fix exactly? Paths like uid://stuff remain unchanged, even though they are not specifically handled.
I think res:// and user:// are checked here to avoid unnecessary processing, because they are already localized. But this seems irrelevant for generic protocol paths.

I had a second check today and it seems you are right. The problem was only visible in some weird use cases, for mine, I was using w4res:// which does have two issues:

It has a number in the protocol identifier. I was not sure it was my fault but it seems numbers should be accepted, like in the s3:// protocol. Thus I decided to update the PR relaxing the ascii char check to include numbers too.
It has res:// in its name, which seems wrongly handled somewhere later on. I suspect this line to cause the issue but haven't investigated more: return cwd.replace_first(res_path, "res://");

Thank you!


How to Deploy Deno to Digital Ocean missing step

I think your otherwise very useful Digital Ocean tutorial page is missing a step:
$ doctl registry login

The push command didn’t work until I had done this.

PR welcome!


App assumes Wi-Fi is not commissioned because wpa_supplicant DBUS proxy setup takes a while

https://github.com/project-chip/connectedhomeip/blob/417b2477674fc34be555f47e30beb1e396a2240b/examples/platform/linux/AppMain.cpp#L116
StartWiFiManagement called in AppMain asynchronously initializes wpa_supplicant DBUS proxy which takes a while. In the meantime app proceeds to query Wi-Fi commissioning status and assumes it is not provisioned. This causes app to reset fabric configuration and continue in BLE pairing mode when restarted.
Section of log with the scenario playing out is reproduced below
[1628247641.706501][8928:8928] CHIP:DL: wpa_supplicant: _IsWiFiStationProvisioned: interface not connected (state=0)
[1628247641.706802][8928:8928] CHIP:SC: Assigned local session key ID 0
[1628247641.706902][8928:8928] CHIP:SC: Waiting for PBKDF param request
[1628247641.706996][8928:8928] CHIP:DIS: Start dns-sd server
[1628247641.716802][8928:8928] CHIP:DIS: CHIP minimal mDNS started advertising.
[1628247641.730670][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.730812][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.730894][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.731204][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.731321][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.731401][8928:8928] CHIP:DIS: Replying to DNS-SD service listing request
[1628247641.731502][8928:8928] CHIP:DL: wpa_supplicant: _IsWiFiStationProvisioned: interface not connected
[1628247641.731591][8928:8928] CHIP:DL: wpa_supplicant: _IsWiFiStationProvisioned: interface not connected
[1628247641.732533][8928:8928] CHIP:DIS: Using wifi MAC for hostname
[1628247641.732723][8928:8928] CHIP:DL: rotatingDeviceId: 00007C5F6E176CD40F68685D100A1CF8A98B
[1628247641.732817][8928:8928] CHIP:DIS: Advertise commission parameter vendorID=9050 productID=65279 discriminator=3840/00
[1628247641.733053][8928:8928] CHIP:DIS: CHIP minimal mDNS configured as 'Commissionable node device'.
[1628247641.733145][8928:8928] CHIP:IN: CASE Server enabling CASE session setups
[1628247641.733228][8928:8928] CHIP:SVR: Server Listening...
[1628247641.734846][8928:8931] CHIP:DL: wpa_supplicant: connected to wpa_supplicant proxy

Closing this issue as PR https://github.com/project-chip/connectedhomeip/pull/9322 has been merged.


Correct JSON of "Properties within each card"

Changed the JSON to be valid for parsing.
Could you take a look and change it at this page? Link below.
https://github.com/Heitor4u/Temp/blob/master/ThemeProperties.json

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

ID: 8d9a8511-6ff9-78c6-5a8c-3f5ac68224d0
Version Independent ID: f4fd28c7-71fa-a776-7154-e76eb9bd3649
Content: Use report themes in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-report-themes.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @Heitor4u . This issue has been assigned to the doc author for review.

Thanks for your comment - the JSON and other content on that page has been significantly updated for better clarity.


Windows OS versions support

Attempting to run on Windows Server 2016 Datacenter with Edge Beta installed. I have not logged in/out and restarted the server after installing the Edge browser. Does that matter? Is it suppose to work on Windows Server 2016?
What OS versions has been tested and verified to work?
System.DllNotFoundException: Unable to load DLL 'WebWindow.Native' or one of its dependencies: The specified module could not be found.

Have attempted to build (publish) and run it on Windows Server 2012, Windows Server 2016, Windows 10, mostly with Edge Beta, all fails same way. What is the correct way of building a published release that works?
.NET Core 3.1
Release
Self-contained/Framework dependent (tried both)
win-x64
Will attempt a downgrade to .NET Core 3.0 and see if that helps. It runs just fine when launching published output from the dev machine. Dev machine have Edge Canary, which I have also tried on one of the other machines.

This is same as issue #1

I've verified on Windows 10 and both .NET Core 3.0 and 3.1, with Edge Dev installed. Maybe it isn't compatible with Edge Beta?

Works when C++ build tools is installed. So leaving the issue open until a packaged version of WebWindow is available that does not require those tools.

Versions 0.1.0-20200214.9 and 0.1.0-20200214.10 do not work on Windows 8.1 with C++ build tools installed.
I get the same exception:
System.DllNotFoundException: Unable to load DLL 'WebWindow.Native' or one of its dependencies: The specified module could not be found.


Add option to wait for logs.

Reduce test time by waiting for specific log message rather than suspending execution. Cf #52.

@tillahoffmann and @SergeyPirogov what is the state of this PR?
Are you planning to merge soon?
It would be very useful to complete the #51


Component emitted event "edit" but it is neither declared in the emits option nor as an "onEdit" prop.

This is  a bug.

Prerequisites

[x] Are you running the latest version?
[x] Are you reporting to the correct repository?
(enso is made of many specialized packages: https://github.com/laravel-enso)
[x] Did you check the documentation?
[x] Did you perform a cursory search?

Description

After click on edit button e.g. in userGroups table I get the following warning after update to Vue 3:

I tried to add emits: ['edit'], to @enso-ui/user-groups/src/bulma/pages/userGroups/Index.vue but it did not remove the warning.
Steps to Reproduce

Log in
Open developer console
Go to Administration -> User Groups
Click on edit on one row

Expected behavior

No warning
Actual behavior

Warning

The warning is caused by vue3, which recommends documenting all events in the component that emits them.
Unfortunately, dynamic event names is not supported yet, but don't worry, you won't have any problems with it.
Meanwhile you can follow https://github.com/vuejs/rfcs/issues/204

Thx - in that case I will close this.


Edit query wiql command not added to Queries section

Describe the bug
Install from Marketplace. Edit query wiql command is missing. Switching to new boards experience shows the command. Switching back to prior boards experience removes it again.
Environment:

Browser [e.g. chrome, edge]  Edge
onpremises (Azure Devops Server/tfs) or hosted (Azure Devops) Hosted
If using a local installation what version

Screenshots
If applicable, add

screenshots to help explain your problem.
error in the devconsole (f12)

To Reproduce
Steps to reproduce the behavior:

Navigate to Queries section
The command is missing.

Expected behavior
A visible Edge query wiql command button

Thanks for reporting, we'll dig into this.


Hazrards are unwalkable in rivers and bridges map

This tells pathy to avoid hazards all together in the fall league map Rivers and Bridges.

.deploy


Watching changes in the library throws an error when the application is served (Consuming your library during development).

Context :

Consuming the library in development mode npm link
Changes in the dist directory of the library are watched npm run build:watch
An application that consumes the library is served ng serve

Make some changes in a file of the src directory of the library.
As the clean:dist task deletes all files in the dist directory, the index.js file is missing,so webpack can't load it to compile :
ERROR in .-lib/dist/index.js
Module build failed: Error: ENOENT: no such file or directory, open 'LIBRARY-DIRECTORY/dist/index.js'
 @ ./src/app/app.module.ts 10:0-44
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://localhost:4200 ./src/main.ts
 webpack: Failed to compile.

Fast fix : editing the library gulpfile:
/**
* 1. Delete /dist folder
*/
gulp.task('clean:dist', function () {

 // Delete contents but not dist folder to avoid broken npm links
 // when dist directory is removed while npm link references it.
 return deleteFolders([distFolder + '/**', '!' + distFolder, '!' + distFolder + "/index.js"]);
});

Watch changes an serve the application again.
Make another changes in a file of the src directory of the library.
webpack will successfully compile.

@raphael-volt : Yes, same happened with me! That's why I have created the separate issue for this as 'testing as a standalone application' and waiting on the same. I know its pain while development but still a workaround is restart the application server or if you are still seeing the errors, just publish your package and then consume it as a dependency and it should work fine. For now, I am following the same approach for development.
Hope it helps!

@raphael-volt — Thank you for sharing your workaround.
Unfortunately, we can't implement this behaviour by default, because we don't want any old  files lingering in the dist directory. If the build would fail, the dist directory would still contain the index.js file generated by the previous run.
Let's leave this issue open for a couple of weeks to see if someone else comes up with a possible solution. Thanks again! 👍

@raphael-volt Thanks for this solution, it was very useful.

I am having same issue, my consuming app fails in compilation. Any workarounds for assets folder?
We should write script to look the changes only. And leave whole dist folder as is. While developing in build:watch task not in build task

We hope to merge the playground soon. See #146 👍
I will update this thread once the playground is available.

You can now preview library code changes live in the browser. The playground has been merged and released as v11.3.0 👍


Direct copy problem from file manager to Vs Code

Steps to Reproduce:
1. Copy file needed from file manager
2. Open Vs Code and Right-Click in folder project in Vs Code file explorer!
3. Oh! Paste in inactive!

Hi, I can copy and paste from the vs code file explorer to vs code file explorer , but I can't do it directly from the file manager to vs code file explorer! The paste button is inactive.
I did a search for the resolved the problem , but I didn't find a solution.
I need to open the project folder in the file manager to add the files needed from the file manager to the project.
Thanks .

VSCode Version: 1.31.1
OS Version: Linux x64 4.15.0-43-generic(Ubuntu 18.04)

Does this issue occur when all extensions are disabled?: Yes

Dupe of issue #164

Duplicate of issue #164
Hi, Thanks
Therefore , the solution is only drag and drop the files , according to the issue #164.

/duplicate of #164


Fix for issue #131

Fixes issue where field is specified by ID
Add new test layout (btw does not render in source but still transforms - see below)
Add new property in mapping schema for FieldId, this is because I translate the field ID (from ListItem) into the Static Name, I retain the original field id in mapping to allow SME to visually see mapping vs page layout file if manually inspected. The Field ID is not used by Transformation.

Related to #131

Nice work, thanks a lot @pkbullock !


wasmtime-fiber dep causing downstream conflicts

Hello, I'm reaching out on behalf of https://github.com/containerd/runwasi
In that project we a couple of crates which depend on wasmtime-fiber as a transitive dependency.
Since wasmtime-fiber is linking native code this means it can only show up in the dependency graph once.
We unfortunately have two.
One is from wasmedge and the other from wasmtime which both depend on this crate for async support.
For now I've just disabled async support since we weren't using it anyway but do foresee wanting to add this later.
I noticed this in wasmtime-fiber's Cargo.toml and was curious if this is feasible?:
https://github.com/bytecodealliance/wasmtime/blob/13be5618a7fce47ae1069be011979538fe7809db/crates/fiber/Cargo.toml#L10-L14
Other alternatives we are looking at is just splitting these two crates out of the workspace and deal with managing the deps separately.

Thanks for the report, but the wasmtime-* family of crates that the wasmtime crate depends on are not intended for general purpose reuse, so we've not committed to providing any degree of support for them. You're always welcome to fork the crates and publish under a different name (with license-appropriate attribution naturally). Additionally if you'd like to send a PR to the crate we can review that and merge that as well.

The reason I'm posting an issue is to discuss if the change proposed in the Cargo.toml is a valid/acceptable solution.

Hey I think @cpuguy83 is referring to the comment in Cargo.toml from wasmtime-fiber, specifically
 # If this is an issue in the future we should tweak the build script to set `#define`
 # directives or similar to embed a version number of this crate in symbols.

We are running into an issue where wasmtime-fiber show up in dependency graph twice as a transitive dep.

I understand. That approach is technically possible for wasmtime-fiber to take, but it puts a significant burden on our release cycle to do so, and this isn't a problem that the wasmtime project created. Wasmedge took a dependency on wasmtime internals which makes it impossible for their code to co-exist with wasmtime, and so we believe the burden should be on wasmedge to correct this problem by forking and publishing the wasmtime-fiber crate under a different name.

Right, but the reason it's showing up twice is because wasmedge-sys directly depends on wasmtime-fiber, which is not a usage we support. Specifically, they're depending on the crate from the Wasmtime 2.0 release, while you're using Wasmtime 7.0.
We're recommending that wasmedge should fork wasmtime-fiber and maintain their own copy with a different name, which will resolve the issue you're seeing.

Thanks for the quick responses.
Seems like this is not a desirable change for wasmtime so I'll go ahead and close this.
Thanks again! 👼


How to show lookup without caching.

I have some other service which import data in database and i need to show that data in lookup.
Due to caching, lookup is not able to show the new data imported in database.
So i want to use lookup without caching. I tried to remove TwoLevelCaching attribute but that also not work.
Do i need to do similar to following thread?
https://github.com/volkanceylan/Serenity/issues/375
In this thread you mentioned that there may be a new type of ServiceLookupEditor in serenity for ajax based lookup.
Please confirm if i need to follow above link to get this functionality or there is anything in serenity for this?

Create a custom lookup script with Timeout negative value

I am sorry.. but could not get.
How to create that?

@volkanceylan
Please help me on this... What should i do..

See multi-tenant guide for custom lookup scripts

Ok, Thanks for point me.
I have created Custom Lookup script by inhering from RowLookupScript and Set Expiration = TimeSpan.FromDays(-1); in constructor.
Now i want to create a custom editor inheriting from LookupEditorBase so that i can override CascadeItems to show all Item by defualt.  But i am not able to create CustomEditor becasue there is no class generated for Custom Lookup script.
I also tried to transform and build many times.
_ Just to inform you that still i am using saltarelle for my old code._
Here is screen shot to illustrate the problem -

Your editor wont change, just its lookup key change.

Thanks for your help and guidance..
Its working..

Maybe it could help somebody. In current version it's possible to set expiration period for lookup cache directly in LookupScript attribute of row class:
[LookupScript ("YourLookupScript", Expiration = -1)]// in seconds


Update wiEnrich.c

Updated stale comment.  The move from [RED] to [FRED] made DIM the next smallest.

This is too complicated right now. I'm trying to extract the libraries to their own repo, when that is done I'll gladly accept this PR


What is the purpose of the Metadata element indicated in the TechnicalProfile?

The documentation indicates a Metadata element can be included within the TechnicalProfile:
The collection of Item of key/value pairs utilized by the protocol for communicating with the endpoint in the course of a transaction to configure interaction between the relying party and other community participants.
For this particular TP, what are the uses of the Metadata element? And why is Metadata included in the documentation, but several other unused standard TP elements (like InputClaims) are not included?

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

ID: 1e3ce49f-746e-9326-9846-32d569558b65
Version Independent ID: 4978d164-555d-b3cc-2989-5246b0f44313
Content: RelyingParty - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/relyingparty.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

From IEF schema perspective the rallying party technical profile MAY have metadata. Although in most cases the metadata is not required, there are some scenarios that you may set a metadata. For example, the previous version of SAML relying RP (service provider) had some metadata.

Thanks @yoelhor !
Closing this out.


Support parsing pom.xml / maven / java

"it's just XML" - I think we should be able to have at least partial support.
In the sample I looked it, it seems you can do a sort of variable substitution which I think we shouldn't try supporting initially, but could actually land at least basic support for e.g. properties as a follow up

+1

So I've almost finished an initial implementation for this, but realised that pom.xml actually isn't a lockfile so it's far less useful at this point because you only have top-level dependencies (and anything else you specify).
The issue I face is that Maven / Java is one of the few ecosystems I actually work in, so I don't even have the tools installed locally let alone know how the ecosystem feels about this to be able to know what a good path forward is here (or even if I'm missing something about how Maven works).
Overall, I think there are at least two possible paths:

something to do with mvn dependency:tree

I don't think I'd want the detector to support parsing this natively, but if we landed #13, people could use a script (which could live in/around this project) to transform the output from Maven into a CSV for the detector

something to do with https://github.com/vandmo/dependency-lock-maven-plugin

again, not sure how much support I'd want to be in the detector itself but it looks like it's output as a pom.xml is literally a standard pom just with all the dependencies explicitly computed, meaning the work I've done so far should mean anyone using that will benefit - I just don't know how popular that plugin is
likewise after #13 is landed, it could be good to explore writing a script to transform its JSON format into a CSV.

@cniweb is this something you'd be able to help with?

This has been released in v0.5.0


Correct minor typo in substr docs example

This fixes a very minor typo.
Python strings should be started and ended with same character.
Shows up at: https://www.tensorflow.org/api_docs/python/tf/substr

This was raised at https://github.com/malmaud/TensorFlow.jl/issues/318
by @shencebebetterme

Can one of the admins verify this patch?


Windows platform support for string streams

Is there any chance to get it running on Windows, i.e. avoid error '''Error
in MessagePack.Utilities.Stream.Stream: String streams are not implemented for this platform'''? See e.g. http://stackoverflow.com/questions/10305095/can-i-replace-a-call-to-open-memstream-with-a-malloc-and-an-implicit-cast for replacing open_memstream by CreateFileMapping/MapViewOfFile.

This is fixed with #14, correct?

Yes, fixed. Thanks.


Update to current Electron version (1.6.8)

I use it for my own project but I thought you might like to merge the changes.

Thanks!


Does not display workflows for me

Installed the extension. It stays loading for a minute and so but it does not populate  Current Branch, and Workflow section, even though i am sucessfully connected to the branch. It recognizes Secrets though.

@e-simo I suspect there might be an exception happening in loading the repo workflow information (since you said secrets are shown).  I just published a small update to log the widest exception to the output window.  If you can take that update and see if it shows anything in output windows, that would be great.
I'm happy to debug your repo as well if it is public (or you are comfortable adding me as a member for a small bit)

@timheuer Thanks for the feedback. The repository is not public, it's a private repo of a client I am working with. If I can help you in some other way, just let me know. We can even have a quick online call if you wish.

@timheuer Thanks for the feedback. The repository is not public, it's a private repo of a client I am working with. If I can help you in some other way, just let me know. We can even have a quick online call if you wish.

Got it -- if you can try the latest version update (0.1.70) and see if an exception shows up, that'd be good.

@timheuer reinstalled the new version and  I got this exceptions:
System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 199
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 250
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__3.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__64.MoveNext() in //Octokit/Http/Connection.cs:line 704
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__631.MoveNext() in /_/Octokit/Http/Connection.cs:line 695 --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiConnection.<Get>d__81.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHubActionsVS.GHActionsToolWindow.d__13.MoveNext() in D:\a\GitHubActionsVS\GitHubActionsVS\src\ToolWindows\GHActionsToolWindow.xaml.cs:line 258
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHubActionsVS.GHActionsToolWindow.d__12.MoveNext() in D:\a\GitHubActionsVS\GitHubActionsVS\src\ToolWindows\GHActionsToolWindow.xaml.cs:line 153
Hope it helps.

It gives me a pointer! This is trying to retrieve Environments...I wonder if there is a permission thing here...looking

Ok, I think 0.1.71 may resolve your issue...let me know. Also added a few more output logging.

hmm still not. Pasting the output log and the exceptions again:
Extension settings retrieved and applied
Loading Secrets...
Number of Repository Secrets found: 1
Loading Environments...
Loading Workflows...
Loading Workflow Runs...
System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 199
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 250
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__3.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__64.MoveNext() in //Octokit/Http/Connection.cs:line 704
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__631.MoveNext() in /_/Octokit/Http/Connection.cs:line 695 --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiConnection.<Get>d__81.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHubActionsVS.GHActionsToolWindow.d__14.MoveNext() in D:\a\GitHubActionsVS\GitHubActionsVS\src\ToolWindows\GHActionsToolWindow.xaml.cs:line 274
System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 199
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__3.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__64.MoveNext() in //Octokit/Http/Connection.cs:line 704
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__631.MoveNext() in /_/Octokit/Http/Connection.cs:line 695 --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiConnection.<GetPage>d__461.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ApiConnection.<>c__DisplayClass19_01.<<GetAll>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiPagination.<GetAllPages>d__01.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ActionsWorkflowsClient.d__12.MoveNext() in //Octokit/Clients/ActionsWorkflowsClient.cs:line 246
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHubActionsVS.GHActionsToolWindow.d__15.MoveNext() in D:\a\GitHubActionsVS\GitHubActionsVS\src\ToolWindows\GHActionsToolWindow.xaml.cs:line 309
System.Threading.Tasks.TaskCanceledException: A task was canceled.
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 199
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__10.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 250
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Internal.HttpClientAdapter.d__3.MoveNext() in //Octokit/Http/HttpClientAdapter.cs:line 50
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__64.MoveNext() in //Octokit/Http/Connection.cs:line 704
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.Connection.d__631.MoveNext() in /_/Octokit/Http/Connection.cs:line 695 --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiConnection.<GetPage>d__461.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ApiConnection.<>c__DisplayClass19_01.<<GetAll>b__0>d.MoveNext() --- End of stack trace from previous location where exception was thrown ---    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)    at Octokit.ApiPagination.<GetAllPages>d__01.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Octokit.ActionsWorkflowRunsClient.d__3.MoveNext() in /_/Octokit/Clients/ActionsWorkflowRunsClient.cs:line 70
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at GitHubActionsVS.GHActionsToolWindow.d__13.MoveNext() in D:\a\GitHubActionsVS\GitHubActionsVS\src\ToolWindows\GHActionsToolWindow.xaml.cs:line 169
Note that under Environments it shows now: Unable to retrieve Environments, please check logs. Current Branch and Workflows still not populated.

Thanks @e-simo I have to think it is a bit of a permission thing. The exception details not enough here, but added some more Api logging to see if the message will come through.  Pushing a new release now with this logging.


Added type to button

Closes #136

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
2
❗

❗ storybook/utility-responsive-image--responsive-image__[w1440px]:

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1920px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
2
❗

❗ storybook/utility-responsive-image--responsive-image__[w1440px]:

Before
After

❗ storybook/utility-responsive-image--responsive-image__[w1920px]:

Before
After

Shot Type
Count
Status

Added
0
🟡

Deleted
0
❌

Regression
3
❗

❗ custom/widgets-collapsiblenavbar-opened__[w899px]_statistics_submenu:

Before
After

❗ custom/widgets-meal-nutrients--hint-opened__[w900px]:

Before
After

❗ custom/widgets-user-profile--is-opened__[w1920px]:

Before
After


Improve how properties to pass to jclouds are selected

including b.l.j.jclouds.xxx properties (which jclouds understands) and known multi-word

triggering rebuild with some test fixes merged

Looks good, merging.


Hello World - Chengzhy's Blog

https://chengzhy.github.io/2022/03/25/hello-world/
Welcome to Hexo! This is your very first post. Check documentation for more info. If you get any problems when using Hexo, you can find the answer in troubleshooting or you can ask me on GitHub. Quick

Hello World


一把梭增加 slot-scope 支持

支持 slot-scope，只兼容2.5+写法，即 使用 slot-scope 属性
支持多字段多形式绑定
<template>

         <slot v-bind='withBind' v-bind:normal='viaNormal' :normal1='varNormal2' :more='compData[i]'></slot>

</template>

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
存在一些问题我处理不了：

更改了 wxml 模板生成，添加了变量向后传递，实现逻辑是一把梭，优化空间较大。
内有一个隐藏约定，slot-scope指定的变量名转译为wxml后都替换为了 $scopedata ，此举是为保持 slot-scope 与小程序中取变量层级一致。这导致用户不能自己向下绑定 key 为 $scopedata 的数据
转译 slot-scope 时需使用上下节点信息，处理时，options 变量未能满足使用，添加新的测试案例，未能按照预期输出。

TODO:

[ ]  :varName 提交有bug需要修复
[ ] 不支持 slot name='notdefault'