Clean devenv does not support process-compose

Describe the bug
process compose does not work in a clean.enabled = true devenv.
$ devenv  up
• Building processes ...
• Using Cachix: devenv
✔ Building processes in 7.1s.
• Starting processes ...
• Building shell ...
✔ Building shell in 1.5s.
Running tasks     devenv:enterShell
Succeeded         devenv:enterShell 2ms
1 Succeeded                         2.49ms

{"level":"warn","error":"open /etc/xdg/process-compose/settings.yaml: no such file or directory","time":"2024-10-02T13:21:59+02:00","message":"Error reading settings file /etc/xdg/process-compose/settings.yaml"}
panic: exec: "infocmp": executable file not found in $PATH

goroutine 9 [running]:
github.com/f1bonacc1/process-compose/src/tui.SetupTui({0xe9e960?, 0xc000127ea0?}, {0xc000051f78, 0x3, 0xc000140640?})
	github.com/f1bonacc1/process-compose/src/tui/view.go:505 +0x156
github.com/f1bonacc1/process-compose/src/cmd.startTui({0xe9e960, 0xc000127ea0})
	github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:89 +0x1d5
created by github.com/f1bonacc1/process-compose/src/cmd.runTui in goroutine 1
	github.com/f1bonacc1/process-compose/src/cmd/project_runner.go:73 +0x65

To reproduce
Run devenv up with the following configuration:

devenv.yaml

# yaml-language-server: $schema=https://devenv.sh/devenv.schema.json
clean:
  enabled: true

inputs:
  nixpkgs:
    url: github:cachix/devenv-nixpkgs/rolling

devenv.nix

{ pkgs, lib, config, inputs, ... }:

{
  packages = with pkgs; [
    coreutils
  ];

  processes.uname.exec = "uname";
}

devenv.lock

{
  "nodes": {
    "devenv": {
      "locked": {
        "dir": "src/modules",
        "lastModified":<PHONE_NUMBER>,
        "owner": "cachix",
        "repo": "devenv",
        "rev": "eb86c60fc04a8be6a544b2647cc0ca8dc611533e",
        "type": "github"
      },
      "original": {
        "dir": "src/modules",
        "owner": "cachix",
        "repo": "devenv",
        "type": "github"
      }
    },
    "flake-compat": {
      "flake": false,
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "owner": "edolstra",
        "repo": "flake-compat",
        "rev": "0f9255e01c2351cc7d116c072cb317785dd33b33",
        "type": "github"
      },
      "original": {
        "owner": "edolstra",
        "repo": "flake-compat",
        "type": "github"
      }
    },
    "gitignore": {
      "inputs": {
        "nixpkgs": [
          "pre-commit-hooks",
          "nixpkgs"
        ]
      },
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "owner": "hercules-ci",
        "repo": "gitignore.nix",
        "rev": "637db329424fd7e46cf4185293b9cc8c88c95394",
        "type": "github"
      },
      "original": {
        "owner": "hercules-ci",
        "repo": "gitignore.nix",
        "type": "github"
      }
    },
    "nixpkgs": {
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "owner": "cachix",
        "repo": "devenv-nixpkgs",
        "rev": "4267e705586473d3e5c8d50299e71503f16a6fb6",
        "type": "github"
      },
      "original": {
        "owner": "cachix",
        "ref": "rolling",
        "repo": "devenv-nixpkgs",
        "type": "github"
      }
    },
    "nixpkgs-stable": {
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "owner": "NixOS",
        "repo": "nixpkgs",
        "rev": "1719f27dd95fd4206afb9cec9f415b539978827e",
        "type": "github"
      },
      "original": {
        "owner": "NixOS",
        "ref": "nixos-24.05",
        "repo": "nixpkgs",
        "type": "github"
      }
    },
    "pre-commit-hooks": {
      "inputs": {
        "flake-compat": "flake-compat",
        "gitignore": "gitignore",
        "nixpkgs": [
          "nixpkgs"
        ],
        "nixpkgs-stable": "nixpkgs-stable"
      },
      "locked": {
        "lastModified":<PHONE_NUMBER>,
        "owner": "cachix",
        "repo": "pre-commit-hooks.nix",
        "rev": "5f58871c9657b5fc0a7f65670fe2ba99c26c1d79",
        "type": "github"
      },
      "original": {
        "owner": "cachix",
        "repo": "pre-commit-hooks.nix",
        "type": "github"
      }
    },
    "root": {
      "inputs": {
        "devenv": "devenv",
        "nixpkgs": "nixpkgs",
        "pre-commit-hooks": "pre-commit-hooks"
      }
    }
  },
  "root": "root",
  "version": 7
}

Version
devenv 1.2.0 (x86_64-linux)

I keep forgetting to add guidance about this to the docs 🤦 clean completely nukes the shell environment. You absolutely must use keep to bring it back to a working state. It's best not to use this feature at all.
Closing as duplicate of https://github.com/cachix/devenv/issues/1163.

@sandydoo  That's quite sad. I do like the idea of the clean feature.. I am trying to launch something inside a devenv - and it does work on all of my machines but on some other machines it does not. Except if they use devenv -c.. That's why I wanted to add clean: true.
but if clean is not meant to be used it should maybe be removed?

but on some other machines it does not.

Do you have an idea what it is? That is generally something we'd be interested in hearing about and fixing.

but if clean is not meant to be used it should maybe be removed?

Yeah, so it's an option that's passed down to nix develop. Some people didn't like the "noise" that nix develop adds to the shell, so this option was exposed. We ended up inheriting its issues. There's will to fix it, just not while we continue to use nix develop.

Do you have an idea what it is? That is generally something we'd be interested in hearing about and fixing.

If I'd know this i would probably have fixed it already ;) I am bubbelwrapping chromium to fake a /etc/hosts.. And in the process of that chromium fails to with:
[2:2:0930/170447.244416:ERROR:process_singleton_posix.cc(1057)] Failed to create socket directory.
[2:2:0930/170447.244549:ERROR:chrome_main_delegate.cc(597)] Failed to create a ProcessSingleton for your profile directory. This means that running multiple instances would start multiple browser processes rather than opening a new window in the existing process. Aborting now to avoid profile corruption.

But I have not yet found out why this does not work on any system..

Yeah, so it's an option that's passed down to nix develop. Some people didn't like the "noise" that nix develop adds to the shell, so this option was exposed. We ended up inheriting its issues. There's will to fix it, just not while we continue to use nix develop.

ack.


Where are the rest of the instructions?

This doesn't have any info on how to connect the App to the Source - seems to be a lot of people saying that Microsoft have taken this documentation down at some point and not replaced it?

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

ID: 879fd196-a46f-9f5d-f33b-b5ae9db14b58
Version Independent ID: 67642cb3-92aa-9340-0dff-3786d0672779
Content: Set up the Help Desk sample app - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/help-desk-install.md
Service: powerapps
Sub-service: canvas-maker
GitHub Login: @norliu
Microsoft Alias: norliu

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


bottle: don't keep old cellar

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

This PR is a proposal, to primarily make linuxbrew-core maintainers life easier.
There are cases when we merge homebrew-core to linuxbrew-core and attempt to build a bottle, but then it fails like this for example:
Error: --keep-old was passed but there are changes in:
cellar: old: "/home/linuxbrew/.linuxbrew/Cellar", new: :any_skip_relocation

If the above happens, we need to change the cellar manually accordingly as the error says.
This is of course doable, but why do it manually if we can have it done by brew?
Don't know if this change is really an ideal solution to the problem, I'm open for suggestions.

Ideally each sha256 should have it’s own cellar definition. Because it may be different on mac and on Linux. I think we had a similar patch in linuxbrew a few years ago, but it was not migrated to Homebrew.
You are changing the bottling code for mac too here, maybe this change would be more acceptable if it was Linux only.
An alternative solution would also to look at why there are differences and why the bottling code detects that a bottle is relocatable on one platform and not the other. The we could add some more rules and if/else blocks in the bottling code.

Ideally each sha256 should have it’s own cellar definition. Because it may be different on mac and on Linux. I think we had a similar patch in linuxbrew a few years ago, but it was not migrated to Homebrew.

Yes, this is probably the right approach (for Linux). Something like
sha256 "abcdabcd" => :linux_x86_64, :cellar => :any_no_relocation

would be sensible/acceptable and with the :cellar => missing it'll default to the block definition (cellar :any etc.).

An alternative solution would also to look at why there are differences and why the bottling code detects that a bottle is relocatable on one platform and not the other. The we could add some more rules and if/else blocks in the bottling code.

I'm not sure this would work. The part that differs is the rewriting of the binary format on macOS and Linux and I can't see us fixing that. The only thing that might help is if checking we do similar things with rpath.

Few cases for reference:
https://github.com/Homebrew/linuxbrew-core/pull/16409
https://github.com/Homebrew/linuxbrew-core/pull/16410
https://github.com/Homebrew/linuxbrew-core/pull/16411

Thanks again @dawidd6!


Update and use RP

433 PV 100% CSAT
Moved from #6807

The instructions are a bit incomplete.
The sample isn't a Razor Pages app.
The sample requires both Redis and SQL Server config to work, and that's a bad bet for an effective sample. Devs probably won't be using both simultaneously. Having to configure both when the dev just wants to see the sample use the one they plan to implement is a major drag on learning from the sample. I'd like to solve this either with two samples or one sample that makes it easy to switch between the two technologies.

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

ID: 937e7280-ac1c-936c-6650-20557da31a8c
Version Independent ID: d81e3546-e3db-2851-2f61-71f01404057f
Content: Distributed Cache Tag Helper in ASP.NET Core
Content Source: aspnetcore/mvc/views/tag-helpers/built-in/distributed-cache-tag-helper.md
Product: aspnet-core
GitHub Login: @pkellner
Microsoft Alias: riande

Opened from wrong topic. Moved to #11236.


pretags laod

Annotations that I export in Cityscapes 1.0 format using cavat.ai can't be imported using the Upload annotations option.
I do the task of image semantic segmentation. I have tried to import annotations in other formats, such as CVAT for images 1.1 and Segmentation mask 1.1.
In addition, I have a problem that I am doing semantic Segmentation tasks, but I find that the exported annotations have instance segmentation tags, such as Segmentation mask 1.1 and PASCAL VOC 1.1.But I don't need these labels. But when I remove SegmentationObject from Segmentation mask 1.1 and PASCAL VOC 1.1, the load annotations are not available, but I don't need the SegmentationObject folder. Error: Request failed with status code 400. "Image 000001: Error: Request failed with Status code 400. "Image 000001: can't import annotation #14 (polygon): Label 'background' is not registered for this task\n".. May I ask how to solve this? I don't see the solution in the document.
Has anyone encountered this problem? Thank you very much for your answer.My requirement is to import pretags (semantically segmented labels inferred from other models) into cvat.ai.

This is the  Cityscapes 1.0 format wrong warning:
Could not upload annotation for the task 193710Error: Request failed with status code 400. "The annotations that were uploaded are not correct"
And if I do not change dataset formats. In fact, there is a lot of information that I don’t need. For example, I don’t need instance segmentation labels for semantic segmentation tasks, but for Segmentation mask 1.1 and PASCAL VOC 1.1, if I don’t have instance segmentation labels, I can’t import them into cvat. This is It seems a bit useless.

In fact, there is a lot of information that I don’t need

But CVAT needs.

In fact, there is a lot of information that I don’t need

But CVAT needs.

In this case, how should I pre-label, I want to use some models for global pre-labeling and then fine-tune to improve labeling efficiency.

You can use any format CVAT supports (there are more formats supporting masks), but please, do not modify them.


Add support for parsing dates with microseconds

Unfortunately, the badges API is funny and returns dates using
microseconds, which is not our usual ISO8601 date handling.
This traps date parsing so that it can handle both cases.

@bguertin, @BenjiLee  please review

@@             master       #687   diff @@
==========================================
  Files           463        464     +1
  Lines         19801      19827    +26
  Methods           0          0
  Messages          0          0
  Branches       2867       2870     +3
==========================================
+ Hits           5919       5941    +22
- Misses        13882      13886     +4
  Partials          0          0

Review [#687][cc-pull] on Codecov

Powered by Codecov. Last updated by [5139b2f...9fefa49][cc-compare]
[cc-base-branch]: https://codecov.io/gh/edx/edx-app-android/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/edx/edx-app-android/compare/5139b2f9912c4817365198551a60910c83923e1c...9fefa490eccb565a1e54a502138fa18275524a6d
[cc-pull]: https://codecov.io/gh/edx/edx-app-android/pull/687?src=pr

:+1:

👍

👍


[Issue]: WebBrowserClient does not contain a definition for 'LogWarning'

What platform are you experiencing this issue on?
Windows
What architecture is your platform?
64-Bit
What version of UWB are you using?
Stable (1.6.3) GitLab Package
What Unity version are you running?
2021.2.12f1
Describe what the issue you are experiencing is.

Provide reproducible steps for this issue.

Open existing unity project
Add packages as instructed on repo home page
Error shows after downloading packages

Any additional info you like to provide?
No response

Someone on Discord had this issue as well, it seems to be that Unity sometimes doesn't download git packages correctly, so just re-install the packages, and that should fix it.
Also side note, that version you are installing is 2.0-preview, not 1.6.3 stable. But I would recommend staying with 2.0-preview.


Python xDS interop client traffic splitting / path matching additions

This PR is analogous to https://github.com/grpc/grpc/pull/23464, adding features to the Python xDS interop client to support testing traffic splitting and path matching.

CC @menghanl

Can you also update the shell script to enable the extra tests, as in https://github.com/grpc/grpc/pull/23641

Unrelated ruby MacOS build timeout.


CarouselView crashes in method DestroyItem on Android

Application using Xamarin.Forms <IP_ADDRESS>635 with Image as ItemTemplate crashes on Android when DestroyItem method is invoked. There is invalid cast exception - it's not possible to cast ImageView to ViewGroup.

Fixed. Next release.


possibility to hide controls and top-nav-bar

Hi @CDrummond,
Just a quick question: Is there a way (or a parameter) to hide the top navigation bar and player controls (except for the timeline)  in fullscreen mode? Or could that be an enhancement feature?
I'd like to use the FS mode to show what is plaing in a room via a screen (no touch).
Best regards and stay safe in these times, goeste

No, but you can do this with Material's custom.css  - see https://github.com/CDrummond/lms-material/wiki/Detailed-Information
Try this:
Create material-skin/css/np.css within your LMS's prefs folder, with the follownig contents:
.lms-toolbar{display:none!important;}
:root{--main-toolbar-height:0px!important;}
.no-control-adjust,.np-controls-wide{display:none!important;}

Launch material with http://SERVER:9000/material/now-playing?css=np&player=01:02:03:04:05:06 (replace 01:02:03:04:05:06 with actual player's ID, or its name (need to URL encode name (space -> %20, etc)).

No, but you can do this with Material's custom.css  - see https://github.com/CDrummond/lms-material/wiki/Detailed-Information
Try this:
Create material-skin/css/np.css within your LMS's prefs folder, with the follownig contents:
.lms-toolbar{display:none!important;}
:root{--main-toolbar-height:0px!important;}
.no-control-adjust,.np-controls-wide{display:none!important;}

Launch material with http://SERVER:9000/material/now-playing?css=np&player=01:02:03:04:05:06 (replace 01:02:03:04:05:06 with actual player's ID, or its name (need to URL encode name (space -> %20, etc)).


XL Deploy by XebiaLabs for "CI/CD" section of Cloud Native Landscape

Please only report on one project or product per issue
Guidelines for getting added to the CNCF Cloud Native Landscape:

We generally will only list a company in one box, to represent its major or best known offering. We occasionally make exceptions for large companies.
Projects with at least 250 GitHub stars that clearly fit in an existing category are generally included.
We are unlikely to create a new category for products/projects as we'd rather find the best home with the current options.
We are generally not including commercial versions of open source software, when that version just includes minor tweaks.
Closed source products need to link to a clear description of your product; no stealth mode companies.
Your project or company needs a logo.

Project or Product name: XL Deploy
Is it open source? (if you are tempted to say partially, please answer no): No
What are the best URL(s) for it? Repo and homepage (if different) for open source, product homepage for closed source?:
Homepage: https://xebialabs.com/products/xl-deploy/
Repo: https://github.com/xebialabs-community/xlr-xldeploy-plugin
What one category does it best belong in? CI/CD
What's the URL for the best logo (and it needs to include the name)?
https://xebialabs-clients-iglusjax.stackpathdns.com/assets/files/media-kit/xl-deploy.png
Anything else we need to know?: Thanks for the consideration! if you have any questions or need more information, you can contact me at<EMAIL_ADDRESS>

Hi @bclevesy, we are including XL Deploy in the upcoming version, releasing soon! Thanks

Closing as this update was included in the most recent version https://github.com/cncf/landscape#current-version


Email Automation Script

Aim
The objective of the Email Automation Script is to send automated emails using Python's smtplib library.
Details
The Email Automation Script in Python aims to automate the process of sending personalized email messages or notifications to one or multiple recipients. By utilizing the smtplib library for handling email connections and the MIMEText module to create the email message, the script allows users to specify the sender's email address, recipient's email address, SMTP server details (such as server address and port), and the sender's login credentials (username and password). After composing the email, the script establishes a secure connection to the SMTP server using TLS encryption and authenticates the sender's account. Once authenticated, the email is sent to the recipient(s) using the sendmail() method. Proper error handling is implemented to handle exceptions during the email sending process. This versatile script can be tailored to various use cases, enabling automated email communication as part of applications or scheduled tasks.
Do I want to work on this:

[x]  Yes
[ ] No

/assign

i wanna work on this too please


px-data-table select all indicates all records selected rather than filtered records when table filtered.

Expected Behavior
When data table is filtered and the user clicks the "Select All" checkbox it should select only the records that are visible / available while filtered.
Actual Behavior
When select all is checked it indicates that 22 records are selected when only 7 match my filter. If you clear the filter all the records are selected.
Steps to reproduce the problem:

Install Predix UI Seed
Navigate to Parent Asset
Select Data Table View
Filter CurrentReading to 0.88
Click Select All

Environment:

Version: v1.14.22
Browser: Chrome version 54.0.2840.99
Operating System: Windows 10 Professional

related to this: select all with pagination selects everything, rather than just the current page of items. Items that are not on the current page should not be selected.

Fixed in v2.1.1. Thanks for the PR @caseyliss !


Invoke and Query Api Endpoint

These endpoint not working properly or not responding.. it keep requesting for hours please elaborate on the proper procedure to invoke and query chaincode

Hello,
could you post additional information?
It would be great to have:

your fablo-config.json or description how did you set up the network.
http requests that where made. Preferably in curl format

I'm using the the default fabric-config-talk.json, launched perfect however trying to invoke the chain code but just keeps requesting

Hi,
sorry for delay. I've prepared scenario that you can try yourself to verify everything:

Check that you don;t have any other active containers (sometimes exposed ports may overlap ):
docker ps - result should be empty
Create new fablo network with sample chaincode:
fablo init rest node - sholud create basic fablo-config and sample chaincode in current directory
Run the network
fablo up - should run the network and end with "Done! Enjoy your fresh network"

Next we can try fablo-rest:
4.  It might be an issue with the config, when Fablo REST cannot connect discoverers

{   "$schema": "https://github.com/softwaremill/fablo/releases/download/0.3.0/schema.json",   "networkSettings": {     "fabricVersion": "2.3.2",     "tls": true   },   "orgs": [     {       "organization": {         "name": "Orderer",         "domain": "root.com"       },       "orderers": [         {           "groupName": "group1",           "type": "raft",           "instances": 1         }       ]     },     {       "organization": {         "name": "Org1",         "domain": "org1.com"       },       "peer": {         "instances": 2,         "anchorPeerInstances": 2       }     },     {       "organization": {         "name": "Org2",         "domain": "org2.com"       },       "peer": {         "instances": 2,         "anchorPeerInstances": 2       }     }   ],   "channels": [     {       "name": "my-channel1",       "orgs": [         {           "name": "Org1",           "peers": [             "peer0",             "peer1"           ]         },         {           "name": "Org2",           "peers": [             "peer0",             "peer1"           ]         }       ]     },     {       "name": "my-channel2",       "orgs": [         {           "name": "Org1",           "peers": [             "peer1"           ]         },         {           "name": "Org2",           "peers": [             "peer1"           ]         }       ]     }   ],   "chaincodes": [     {       "name": "chaincode1",       "version": "0.0.1",       "lang": "node",       "channel": "my-channel1",       "endorsement": "OR('Org1MSP.member', 'Org2MSP.member')",       "directory": "./chaincode-kv-node",       "privateData": [         {           "name": "private-collection",           "orgNames": [             "Org1",             "Org2"           ]         }       ]     }   ] }

I'm using postman instead of curl

Hi @KG-DI-KXNG, I have some ideas that might help you.
First, it seems that you try to run Fablo REST standalone, since you don't have it in your fablo-config.json file. The easiest way to run it Fablo REST to provide it in tools for organization, so you may try to update the Org1 in fablo-config.json as follows:  { "organization": {...}, "peer": {...}, "tools": { "fabloRest": true }. Then Fablo REST for this organization will be available at 8801 port. I just found out that we should have more information about it in the docs, sorry for that -- we are going to update it.
If you want to run it standalone, try to start Fablo REST with the following command, after your Fablo network started:
#!/usr/bin/env bash

# provide value from fablo-target/fabric-docker/.env
# from variable COMPOSE_PROJECT_NAME
network=fablo_network_202201031559

docker run \
  -e MSP_ID="Org1MSP" \
  -e FABRIC_CA_URL="http://ca.org1.com:7054" \
  -e FABRIC_CA_NAME="ca.org1.com" \
  -e AS_LOCALHOST=false \
  -e DISCOVERY_URLS=grpcs://peer0.org1.com:7041,grpcs://peer1.org1.com:7042,grpcs://peer0.org2.com:7061,grpcs://peer1.org2.com:7062 \
  -e DISCOVERY_TLS_CA_CERT_FILES=/crypto/org1.com/peers/peer0.org1.com/tls/ca.crt,/crypto/org1.com/peers/peer1.org1.com/tls/ca.crt,/crypto/org2.com/peers/peer0.org2.com/tls/ca.crt,/crypto/org2.com/peers/peer1.org2.com/tls/ca.crt \
  -p "8000:8000" \
  -v /your/absolute/path/to/fablo-target/fabric-config/crypto-config/peerOrganizations:/crypto:ro \
  --network="${network}_basic" \
  -d \
  --rm \
  softwaremill/fablo-rest:0.1.0

(in this case Fablo REST will be available on 8000 port)
If none of this works, please provide the command(s) you use to start Fablo REST and logs of Fablo REST container. And we will dig deeper to find the cause of this issue.
If some of proposed solutions work, let us know and feel free to close this issue.
Our TODOs are:

improve documentation on Fablo REST - https://github.com/hyperledger-labs/fablo/issues/271
return meaningful status codes when discovery service cannot connect - #37

Got it working but invoking and querying on port 8800 doesn't work but port 8801 works perfectly

Great to hear that :-)
By the way, we are going to release the version 1.0.0 today or tomorrow, with a bunch of new features. Feel free to check it if you want.

Got it working but invoking and querying on port 8800 doesn't work but port 8801 works perfectly

Hi I am also facing the same problem

I have also added "tools": { "fabloRest": true }
Enroll admin is working but login is invocation is not happening with no response for a long time.

These endpoint not working properly or not responding.. it keep requesting for hours please elaborate on the proper procedure to invoke and query chaincode

Same problem here


Change password

As a customer,
I need the function to change my password,
so that I can protect my account and data


Fails to report error when stacktrace includes core JavaScript API

Description
When an exception occurs, and the stacktrace includes a core JavaScript API, the exception-reporting package fails to report the exception to bugsnag. For example, the stacktrace below involves an exception in the fuzzy-finder package, and the stacktrace includes a call to Array.forEach:
     ~/github/fuzzy-finder/lib/main.js:81 Uncaught Error
         at forEach (/Users/j/github/fuzzy-finder/lib/main.js:81:31)
===>     at Array.forEach (native)     <===
         at Object.createProjectView (/Users/j/github/fuzzy-finder/lib/main.js:81:9)
         at HTMLElement.fuzzy-finder:toggle-file-finder (/Users/j/github/fuzzy-finder/lib/main.js:12:14)
         at CommandRegistry.handleCommandEvent (/Applications/Atom.app/Contents/Resources/app/src/command-registry.js:384:49)
         at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent      (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:621:22)
         at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent      (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:412:28)
         at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:110:40)

When the exception-reporting package attempts to normalize the paths in the stacktrace, the package errors when it encounters Array.forEach in the stacktrace. Instead of reporting the exception to bugsnag, the package logs an error to the console:
Error reporting uncaught exception TypeError: Cannot read property 'replace' of null
    at Reporter.normalizePath (~/github/exception-reporting/lib/reporter.js:71:27)
    at Reporter.scrubPath (~/github/exception-reporting/lib/reporter.js:77:31)
    at ~/github/exception-reporting/lib/reporter.js:61:20
    at Array.map (native)
    at Reporter.buildStackTraceJSON (~/github/exception-reporting/lib/reporter.js:59:40)
    at Reporter.buildExceptionJSON (~/github/exception-reporting/lib/reporter.js:54:24)
    at Reporter.buildNotificationJSON (~/github/exception-reporting/lib/reporter.js:33:27)
    at Reporter.reportUncaughtException (~/github/exception-reporting/lib/reporter.js:229:30)
    at ~/github/atom/out/app/node_modules/exception-reporting/lib/main.js:25:23
    at Function.module.exports.Emitter.simpleDispatch (/Applications/Atom.app/Contents/Resources/app/node_modules/event-kit/lib/emitter.js:25:20)
    at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app/node_modules/event-kit/lib/emitter.js:141:34)
    at window.onerror (/Applications/Atom.app/Contents/Resources/app/src/atom-environment.js:949:26)
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:621:22)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/node_modules/atom-keymap/lib/keymap-manager.js:412:28)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:110:40)

Steps to Reproduce

Clone the repository for another bundled package and apm link it. We'll use fuzzy-finder for this example:

git clone atom/fuzzy-finder
cd fuzzy-finder
git checkout v1.8.1
apm install && apm link

In the fuzzy-finder code, edit main.js to trigger an error that has a core JavaScript API in the stacktrace:--- a/lib/main.js
+++ b/lib/main.js
@@ -78,6 +78,8 @@ module.exports = {
   },

   createProjectView () {
+    [0].forEach(() => { throw new Error('whoops') })
+
     this.stopLoadPathsTask()

     if (this.projectView == null) {

Clone the atom/exception-reporting repository and apm link the package:

git clone atom/exception-reporting
cd exception-reporting
git checkout v0.43.1
apm install && apm link

In the exception-reporting code, edit lib/reporter.js so that it will treat our local copy of fuzzy-finder as a bundled package for error reporting purposes:--- a/lib/reporter.js
+++ b/lib/reporter.js
@@ -253,7 +253,7 @@ export default class Reporter {
   }

   isBundledFile (fileName) {
-    return this.normalizePath(fileName).indexOf(this.resourcePath) === 0
+    return true
   }

   isTeletypeFile (fileName) {

Open Atom
Trigger the Fuzzy Finder: Toggle File Finder command to force the error above to be thrown

Expected behavior: Error should be successfully posted to bugsnag
Actual behavior: Error is not reported to bugsnag
Reproduces how often: 100%
Versions
exception-reporting 0.43.1
$ atom --version
Atom    : 1.27.0-dev-6dd878d
Electron: 1.7.11
Chrome  : 58.0.3029.110
Node    : 7.9.0

/cc @as-cii (since we encountered this issue while pairing today)

This issue was moved to atom/atom#18141


device: Add GPU device support

Enable GPU device support in kata runtime, including GVT-g and GVT-d.
GVT-g: graphic virtualization technology with mediated pass through
GVT-d: graphic virtualization technology with direct pass through
BDF of device eg "0000:00:1c.0" is used to distinguish GPU device in GVT-d,
while sysfsdev of device eg "f79944e4-5a3d-11e8-99ce-479cbab002e4"is used
in GVT-g.
Signed-off-by: Zhao Xinda<EMAIL_ADDRESS>

Please can you also update your commit. Looking at the Travis log:
Running checkcommits version 0.0.1 (commit<PHONE_NUMBER>285c710c10e1063d5a6f75310a58a3)
Detected TravisCI Environment
Found 1 commit between commit ef8a0428467a2211d719422589b274c891adbd61 and branch master
ERROR: No "Fixes" found
ERROR: checkcommits failed. See the document below for help on formatting
commits for the project.

https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#patch-format

I've just raised #542 so you can specify a Fixes #542 in your commit to keep checkcommits happy :)

sorry for the late response,  Since my test environment is totally broken, it takes a lot of time to repair it.
I have submitted the new commit and thanks for having a review.

lgtm
Adding the DNM merge till the govmm PR is merged and vendored.

@xindazhao I have opened this issue : https://github.com/kata-containers/runtime/issues/594
The issue documents the limitation with  hotplugging Intel GPUs on a pci bridge.

@xindazhao The govmm PR https://github.com/intel/govmm/pull/30 is merged, can you split this PR into two commits, one is for vendoring new govmm and the other one is your modification?

ok, I will do it. thanks.

LGTM  thanks for this pr~

Split the PR into two commits according to WeiZhang555's suggestion.

I think first PR should use dep tool to vendor govmm changes.

Yes, please use dep to do vendoring as @caoruidong suggested.

@xindazhao You can check https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#re-vendor-prs and https://github.com/kata-containers/community/blob/master/VENDORING.md

Hi @xindazhao - do you think you will be able to update this PR soon?
If those documents are not clear, please join us on slack/irc so we can help you! :smile:

hi @jodh-intel, thanks a lot, I am just came back from a long vacation, I am preparing another patch for govmm.  @amshinde has submitted another patch to force hotplug VFIO device on root bus (#594) and I need to add qmp.ExecuteVFIOMediatedDeviceAdd() to follow this change.

Now no need to update vendor, it is updated by other PRs. So you can just put your own code.

Now no need to update vendor, it is updated by other PRs. So you can just put your own code.

If vendor code is not included in the same commit, then the build of kata runtime will be failed, right?

Hi @xindazhao - Since https://github.com/intel/govmm/pull/30 has now landed, you can use dep ensure to update the vendoring for that project on your first commit. That is fine and will not cause any problems because the code is not using the new ExecutePCIVFIOMediatedDeviceAdd() function.
Your second commit can add the code that calls ExecutePCIVFIOMediatedDeviceAdd().

Thanks, so your suggestion is that I split commit into two, the first commit includes ExecutePCIVFIOMediatedDeviceAdd function, together with change in Gopkg.toml and Gopkg.lock, the second commit includes ExecuteVFIOMediatedDeviceAdd function, together with vendor code. I'm not sure I understand it clearly.

Note quite.

The first commit will be just the result of dep ensure and will change:

Gopkg.*
vendor/github.com/intel/govmm/

The second commit will simply use the new ExecutePCIVFIOMediatedDeviceAdd(), since that will then be available in vendor/github.com/intel/govmm/qemu.go (from the first commit).

Doing this allows us to detect vendor issues - the first commit should be a "NOP" and have no effect (since nothing is using the new function), but if it fails, we know the problem is with the vendoring, not with your changes that make use of ExecutePCIVFIOMediatedDeviceAdd().

This new commit includes two major modifications, one is qmp.ExecutePCIVFIOMediatedDeviceAdd() which has been reviewed in intel/govmm#30 and merged, the other is qmp.ExecuteVFIOMediatedDeviceAdd(() which is under review in intel/govmm#47.

Yes. If you introduce another update in govmm, you should still include 2 commits. This first commit is to vendor govmm.

@xindazhao Yes, this should still be split into 2 commits, and need a rebase. Code looks good.

@xindazhao please rework the govmm PR as suggested, and then you'll be able to revendor it from this PR. On top of this, you'll add your current commit.

thanks a lot. I have updated the comments in intel/govmm#47 and it encounters some problems if using qmp.ExecutePCIVFIOMediatedDeviceAdd() on root bus.

LGTM. @amshinde ptal

lgtm

thanks @xindazhao
lgtm


Added BooleanFilter

I added a BooleanFilter that accepts a value and adds a 'WHERE' or 'WHERE NOT' depending on whether the value is True or False.

Coverage increased (+0.06%) to 93.363% when pulling 94fb5a026739ddab834632433a94e85b462590c2 on ENG-175331-create-boolean-dimension-type into 40b4a6590134d1bc590cf57eb4ca5c6344c7b6d0 on master.

Coverage increased (+0.05%) to 93.358% when pulling 472428336e9c6b5185c7e1f1b149671f6e4d4565 on ENG-175331-create-boolean-dimension-type into 40b4a6590134d1bc590cf57eb4ca5c6344c7b6d0 on master.


⭐ 11 - Budget Tracker (Hacktoberfest Demo)

🤩 Thank you for participating in the CopilotKit Hacktoberfest event.
Follow This Link to Get Started!🎉
NOTE: CopilotKit will allow multi-assign issues
Please show your support and give a ⭐ to the CopilotKit repository.
Demo:
Budget Tracker
Description
Build a financial budgeting tool that helps users track expenses and savings goals with text-based inputs
Technology Requirements
Using CopilotKit

Required: Your project must utilize CopilotKit as a core component.
Installation: Include CopilotKit by following the installation guide at CopilotKit Quickstart.
Documentation: Familiarize yourself with CopilotKit's features through the Official Documentation.

Styling with Shadcn-UI

Required: All UI components in your project should be styled using Shadcn-UI.
Installation: Install Shadcn-UI documentation

Hi @NathanTarbert I am currently working on this project using shadcn-ui and copilot-kit for the quira quest. Can you assign it to me! :)

Hey @NathanTarbert I want to work on this, can you kindly assign me this issue?

I would like to contribute in this as well !
Can you assign me this ?

🍀Hey @NathanTarbert can you assign this to me as well !

/assign

Please assign this to me I already have it in the pipeline


Google Guava depedency resolution issue

While trying to add latest version of dependency in project, build is failing with reason that Google Guava is not able to choose between android and java version of library.

I tried to exclude dependency and add java version manually but it is not working.
if someone has found solution to this Can you please help me with that?
Thank you so much.

Hi,
Good point, we actually don't need guava anymore. It's gone in https://github.com/seatsio/seatsio-java/releases/tag/v82.16.0

... actually it looks like releasing failed. We'll have a look at that.

Should be ok now 👍

Thanks @mroloux  for your help


Update README.md

On default options, rename amqp.Config{} to amqp091.Config{}

All the examples in the examples folder use this alias: amqp "github.com/rabbitmq/amqp091-go" I'm not sure it will be less confusing this way

Understood but it show inconsistency with others line


width Decimal is not supported?

width Decimal is not supported?

Same issue here ..... 1 is too small to be printed with enough details, and 2 is too large, 1.5 would be perfect for code 128 printed ...
Will test it here and let you know if readers are ok with the decimal values.

@120011676
Just in case even if we are way later ...
https://github.com/lindell/JsBarcode/pull/431


Add aria2 on windows

Tool name
aria2
Tool license
GPL 2
Add or update?

[X] Add
[ ] Update

Desired version
1.35.0
Approximate size
2MB
Brief description of tool
aria2 is already available on linux and on macOS
aria2 - The ultra fast download utility
aria2 is a utility for downloading files. The supported protocols are HTTP(S), FTP, SFTP, BitTorrent, and Metalink. aria2 can download a file from multiple sources/protocols and tries to utilize your maximum download bandwidth. It supports downloading a file from HTTP(S)/FTP/SFTP and BitTorrent at the same time, while the data downloaded from HTTP(S)/FTP/SFTP is uploaded to the BitTorrent swarm. Using Metalink's chunk checksums, aria2 automatically validates chunks of data while downloading a file like BitTorrent.
URL for tool's homepage
https://github.com/aria2/aria2
Provide a basic test case to validate the tool's functionality.
aria2c --version

Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11
[X] Windows Server 2016
[X] Windows Server 2019

Can this tool be installed during the build?
choco install

Tool installation time in runtime
1s
Are you willing to submit a PR?
Yes

Closing this one https://github.com/actions/virtual-environments/pull/3671#issuecomment-885529702

Can you reopen this issue please? I was just out on vacation. The PR only needed to have main merged in...

Thanks @AlenaSviridenko

Re-opened issue, thanks for confirming the status of the tool 🙂


🙏 This package looks awesome! Thanks!

Just wanted to say thanks for this package. 🙏 Can't wait to try it out. 🎉

Thanks for the kind words, Max. Would like to hear your thoughts after you've tried it - good or bad.


Torrent peers are showing docker bridge IP address

Title. Running the container on a Synology NAS DSM7 with the latest docker/container.
Quite a lot of torrent peers are showing up with the bridge IP <IP_ADDRESS>. Is this normal or is there a way to fix this?
Not sure if this is the right place but I don't know where to ask. A bit of a newbie in docker/container

I'm seeing the same thing

Can't tell without log, config, etc... Please create a proper bug report.


Automator: update istio/client-go@master dependency in istio/istio@master

Generated by Automator - 2024-11-07T07:23:37+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
2f0b4de977a562c0afffecf7a352133b8c3ad4dd
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-ambient_istio
2f0b4de977a562c0afffecf7a352133b8c3ad4dd
link
true
/test integ-ambient

unit-tests_istio
1d92cf2ade542cb86b4af4ec7e16c14c8a59ab19
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
1d92cf2ade542cb86b4af4ec7e16c14c8a59ab19
link
true
/test unit-tests

integ-security-multicluster_istio
1d92cf2ade542cb86b4af4ec7e16c14c8a59ab19
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
1d92cf2ade542cb86b4af4ec7e16c14c8a59ab19
link
true
/test integ-security-multicluster

unit-tests-arm64_istio
81c2dbd7ca9c24b38f6a6cde494591a0c44e2da4
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.


Block advertisements in player iframe

I'm not convinced that I want to do this, but it would be nice to explore.
Currently, any time the user switches streams, a new iframe is made and connects to the new stream. This means that an advertisement will play after a few seconds. Again, this happens anytime we change streams.
It would be great if we could either

prevent advertisements from playing on stream switch somehow

(maybe by changing Twitch component to listen for new props and manually set existing iframe's url without re-rendering? See #15 )

or block advertisements altogether

Looks like Twitch now seems to inject advertisements straight into the stream instead of overlaying them, so it seems like this is not very feasible anymore. Closing this for complexity and lack of return.


Update dependencies

Update dependencies of almost all packages.
Remove one ESLint rule since there is a breaking change in typescript-eslint.class-name-casing - replaced by naming-convention

@ktmud There's specific instructions on how to update dependencies: https://github.com/airbnb/nimbus/blob/master/.github/CONTRIBUTING.md#updating-dependencies

@milesj Didn't know we had this CONTRIBUTING file. I think normally it is under the root folder of a project?
I tried to update by package by package but there were some interdependencies and I just figured it was easier to update them all. Will try again to split and make separate PRs.


loading error when filtering by certain providers

see attached that results from this search: https://ccsearch.creativecommons.org/search?q=natural history&provider=museumsvictoria&li=&lt=&searchBy

This is a duplicate of #280


SetOnResponse doesn't seem to work

Hi
I have the feeling that SetOnResponse setting doesn't work.
Here is the docker-composer.yml file that spawns Traefik and their whoami service:
version: "3.3"

services:
  traefik:
    image: "traefik"
    container_name: "traefik"
    command:
      - "--providers.docker=true"
      - "--experimental.plugins.htransformation.modulename=github.com/tomMoulard/htransformation"
      - "--experimental.plugins.htransformation.version=v0.2.7"
    volumes:
      - "/var/run/docker.sock:/var/run/docker.sock:ro"

  whoami:
    image: "traefik/whoami"
    container_name: "whoami"
    labels:
      - "traefik.enable=true"
      - "traefik.http.routers.whoami.tls=true"
      - "traefik.http.routers.whoami.tls.domains[0].main=[HIDDEN]"
      - "traefik.http.routers.whoami.rule=Host(`whoami.[HIDDEN]`)"
      - "traefik.http.routers.whoami.entrypoints=websecure"
      - "traefik.http.routers.whoami.middlewares=htransformation"
      - "traefik.http.middlewares.htransformation.plugin.htransformation.Rules[0].Name=Set Foo"
      - "traefik.http.middlewares.htransformation.plugin.htransformation.Rules[0].Type=Set"
      - "traefik.http.middlewares.htransformation.plugin.htransformation.Rules[0].Header=Foo"
      - "traefik.http.middlewares.htransformation.plugin.htransformation.Rules[0].Value=bar"
      - "traefik.http.middlewares.htransformation.plugin.htransformation.Rules[0].SetOnResponse=true"

When I curl the whoami service, I can see the Foo header in the request...but not in the response:
curl -v https://whoami.[HIDDEN]
< HTTP/2 200
< content-type: text/plain; charset=utf-8
< date: Sun, 16 Apr 2023 11:22:03 GMT
< content-length: 404
<
RemoteAddr: [HIDDEN]
GET / HTTP/1.1
Host: whoami.[HIDDEN]
User-Agent: curl/7.58.0
Accept: */*
Accept-Encoding: gzip
Foo: bar
X-Forwarded-Host: whoami.[HIDDEN]
X-Forwarded-Port: 443
X-Forwarded-Proto: https

Am I doing something wrong here or is there an issue with the SetOnResponse field?

By the way, traefik version is 2.9.10, in case it matters.


Locale Timezone: Europe/Kiev typo

Describe the bug
Kyiv is a correct city name.

Hello @AM1988, got it. I will address this with the corresponding team, and we will review it. Thank you very much for your feedback!


Cannot run Cesium and Leaflet side by side

When Cesium is loaded, you cannot run Leaflet in emp.
To duplicate Open Leaflet map.   Then open Cesium map.   Leaflet appears to load fine.   Make sure to open leaflet first, then cesium.   Cesium overwrites the leaflet library variables.   The viewerCesiumNavigationMixin is what is overriding Leaflet. The leaflet EMP uses is 1.0.3, the navigation mix in uses Leaflet 0.7.7.  This overwrites Leaflet's variables, and screws up anything to do with adding features.   This is an extemely important issue to me as when Leaflet and Cesium are running side by side, Leaflet cannot load data.

The dependency on Leaflet is not need it inside the Cesium navigation plugin. I'm removing the dependency from t he plugin.

Test case:
Open leaflet, and then Cesium so the 2 maps are open simultaneously. Plot a polygon in Leaflet. The test  is a  success if the [polygon renders on the leaflet map.

Reintegrate

merged into the dev branch. closing this issue.


Incompatible with with Comlink Workers

Describe the bug
I was testing my code that uses a Comlink web worker. I followed the Testing with Web Workers instructions.
Error prior to @vitest/web-workerconfig
ReferenceError: Worker is not defined

Error after putting @vitest/web-worker into setupFiles config
[vitest] `importScripts` is not supported in Vite workers. Please, consider using `import` instead

My service worker is used as such, this is the only please a worker is used across/referenced the entire application.
const instance = new ComlinkWorker<typeof import("../service")>(new URL("../service", import.meta.url));
instance.doWork()

My understanding of service workers is still quite limited.
Reproduction
TODO ( about to 💤 on it and get back to in the AM)
System Info
System:
    OS: macOS 14.3.1
    CPU: (8) arm64 Apple M2
    Memory: 77.75 MB / 8.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 20.9.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.5.0 - /usr/local/bin/npm
    pnpm: 8.14.0 - /usr/local/bin/pnpm
  Browsers:
    Chrome: 123.0.6312.124
    Safari: 17.3.1
  npmPackages:
    @vitejs/plugin-react: ^4.0.4 => 4.2.1
    @vitest/coverage-v8: ^1.4.0 => 1.4.0
    @vitest/web-worker: ^1.5.0 => 1.5.0
    vite: ^5.1.3 => 5.1.3
    vitest: ^1.4.0 => 1.4.0

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

### Tasks

We cannot support importScripts because importing is asynchronous, but this API is synchronous. I would recommend using the browser mode instead.

@sheremet-va  Thank you. Is this something potentially worth working on within the Comlink library? Or is browser mode a good workaround to avoid this additional work?

I am still curious about this issue. As my last comment is coming up on the 3 day mark, I am adding an additional one to prevent any automatic action from taking place, and also to add that I will be writing the tests for this module at the beginning of next week, so I hope to be adding a reproducible sample here within the next 3 days.

I don't think the comlink library can do anything to support @vitest/web-worker - it is meant to be used with the default Vite's worker handling.
Supporting importScripts is out of scope for Vitest, I would recommend testing your code in a real browser.


Clean up the usage module

[ ] documentation
[ ] type hints
[ ] misc clean up

Closes #544.

I'm putting this on the 2020.8 project board, but no rush @andrewsanchez - if it doesn't make the release, no big deal.

Including changes proposed in #549 for Travis.  I can re-write the history if you prefer to keep these change sets discreet.  Still a WIP, but it would be nice to get a quick review, @thermokarst, just to make sure I'm on the right track.  This has been incredibly useful for understanding the Usage API and the framework a little better. 😄

@thermokarst, we still have issues, even with moving usage to plugin, and moving to core won't help either.
Are you familiar with this PEP (563) that solves the problem of circular imports in the context of typing?  Seems like it is the solution to my problems, but only in 3.6+.  😕
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/__init__.py", line 9, in <module>
    from qiime2.sdk import Artifact, Visualization
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/sdk/__init__.py", line 11, in <module>
    from .plugin_manager import PluginManager
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/sdk/plugin_manager.py", line 16, in <module>
    from qiime2.plugin.model import SingleFileDirectoryFormatBase
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/plugin/__init__.py", line 17, in <module>
    from .usage import UsageAction, UsageInputs, UsageOutputNames
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/plugin/usage.py", line 19, in <module>
    class ScopeRecord:
  File "/Users/aas/Projects/qiime2/qiime2/qiime2/plugin/usage.py", line 70, in ScopeRecord
    ) -> typing.Union[sdk.Artifact, sdk.Visualization, metadata.Metadata]:
AttributeError: module 'qiime2.sdk' has no attribute 'Artifact'

So if I'm understanding all of this correctly, when our type hints depend on accessing module attributes which are unavailable at "definition time," those hints will need to be string literals for now.  Does that sound right?

Since you no longer need to worry about circular imports, can you please revert your changes that moved usage.py out of sdk and into plugin?

I'm surprised to learn that string literal type hints confused flake8:  ./qiime2/sdk/usage.py:31:29: F821 undefined name 'Artifact'.  We will need to import these objects to satisfy flake8 or decide on another solution.

We will need to import these objects to satisfy flake8 or decide on another solution.

You have already imported most of these objects, you'll need to use the namespace in the annotations:
Instead of 'Artifact', 'Visualization', 'Metadata' you'll need to use 'sdk.Artifact', 'sdk.Visualization', 'sdk.Metadata'.
I'll leave the import and namespace for the PipelineSignature objects as an exercise to you, but as a hint, it can be done (I just verified locally).

A minor quibble - the diff here was pretty painful to review because the order of the code was shuffled in addition to the content. In the future it would be helpful to stage the commits in a logical order to assist with review.

I raised this concern (it feels like ages ago at this point!) during a call when I realized this changeset would require shuffling the order. I thought I remember you telling me that I shouldn't worry about it because you thought GitHub's UI would display the diff intelligently.  But maybe I misinterpreted your response.  Sorry about that!
The latest changes address your suggestions as well as what was reported by python -m numpydoc.

Not sure why I couldn't reply directly to your comment above, but after reviewing numpydoc Sections again, what do you think about dropping the Raises section here entirely?  Or keeping just the TypeError?

This section should be used judiciously, i.e., only for errors that are non-obvious or have a large chance of getting raised.

Not sure why I couldn't reply directly to your comment above

Because the comment is on a previous review - scroll up and unhide the collapse comments to find it. Alternatively you can reply in the "Files changed" tab.

what do you think about dropping the Raises section here entirely? Or keeping just the TypeError?

Why do you think the TypeError should be kept over the KeyError?  I think following the guidance you quoted above, both types of errors "have a large chance of getting raised."


Unable to find the "Notational Velocity" folder including "template.html" and "custom.css"

On nvALT's mainpage, Brett introduced that the users can customize the preview style free:

Customization
After the first time you run the Preview window, look in ~/Library/Application Support/Notational Velocity and you’ll find two files: template.html and custom.css.

My problem is, I failed to find the path.
I did run the Preview window, and checked the "Application Support" folder, even the hidden files and folders under this folder, but there was no folder named "Notational Velocity".
I've tried to install Notational Velocity, still didn't work.
What should I do?

Just confirmed that the user can modify the preview style by putting their own custom.css into  ~/Library/Application Support/nvALT, seems that the Customization Guide on is out-dated ☹

Brett's blog about this issue, check here


Auto refresh alignment heat map on registration

Decided that auto refresh might be a pain, so I added a refresh button instead


NCrunch support?

Hi team,
have you got an idea why NCrunch is ignoring code lines with .Should()...?
Thanks, Ingo

What do you mean with that? We use NCrunch quite heavily with XUnit

Hi Dennis,
thanks for your fast answer! In lines where the Should extensions are used, e. g.
firstName.Should().Be('Ingo');
there are no black/red/green NCrunch dots on the left while all other lines are "dotted" like usual. But now, on my PC at home, everything is working fine, that's weird. OK - I'll have a look at it tomorrow in the office again and let you know then.
Thanks!
Ingo

Hi Dennis,
my NCrunch was out of sync, that caused the problem, sorry! Everything is working fine.
Cheers
Ingo


Fixup margins on the Android Box binding

https://github.com/cashapp/redwood/assets/133019/d67574ec-19af-4d9c-9a00-5ffece237d4f


Lock editing page

Hi. How can i lock the editing document ? It is necessary because multiple users can't edit page at the same time. And application doesn't show notification for this case

Hi @Mracobes9,
There is no current way to force a lock of a single page.
BookStack will attempt to show a notification to a second editor if an edit is in progress by a previous user. Further functionality has already been requested in #395 so I'll therefore close this off as a duplicate.


Deadly Snowball

The snowball is marked as deadly but it only slows the player down.
Look for a transition with “sick” which marks the item as non-lethal. Also change how the death transitions look. Consider adding a decay transition of 40 seconds into death on the non-sick objects.
See this issue for more details.

Snowball - Just Thrown appears to still be deadly

Hmm, I'll reopen this to check transitionsToward for sick object as well.


date --set is failing in freebsd

---- test_date::test_date_set_valid_2 stdout ----
current_directory_resolved:
run: /tmp/cirrus-ci-build/target/debug/coreutils date --set Sat 20 Mar 2021 14:53:01 AWST
thread 'test_date::test_date_set_valid_2' panicked at 'assertion failed: self.success', tests/common/util.rs:86:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- test_date::test_date_set_valid_3 stdout ----
current_directory_resolved:
run: /tmp/cirrus-ci-build/target/debug/coreutils date --set Sat 20 Mar 2021 14:53:01
thread 'test_date::test_date_set_valid_3' panicked at 'assertion failed: self.success', tests/common/util.rs:86:9

---- test_date::test_date_set_valid_4 stdout ----
current_directory_resolved:
run: /tmp/cirrus-ci-build/target/debug/coreutils date --set 2020-03-11 21:45:00
thread 'test_date::test_date_set_valid_4' panicked at 'assertion failed: self.success', tests/common/util.rs:86:9

https://github.com/uutils/coreutils/pull/1852/checks?check_run_id=2161834207
Caused by https://github.com/uutils/coreutils/pull/1798
@marcosatti  could you please have a look?

@sylvestre Those are expected to fail at the moment - that was what PR #1846 added in. Do you want me to change it to assert a failure for now instead?
GNU handles those dates correctly under my system.

yeah, for these platforms too :)
thanks

These tests are still set to expect failure. So still needs to be fixed.


Remove docusaurus website version

If for any reason this needs to be brought back/used for reference. Branch archive/docusaurus contains the code before this PR

No changes needing a change description found.

You can try these changes here

🛝 Playground
🌐 Website
📚 Next docs

try it looks good


Cache avatars

only queries the GitHub API on site build, instead of on page load. Let's see if CircleCI will allow the API call 🤞🏻

The error persists, I'd rerun with SSH so you can interactively debug remotely


Fix #2982

This null check fixes bug I described in ticket #2982

Closing as per the discussion on https://github.com/querydsl/querydsl/pull/2577 .
The required use of isNull() is very much intentional.


Should the user get a message when someone signs?

They get an email when everyone has signed, but should they get one for each signature.
Also need to check if people can undo signatures and whether they should be able to. If so, should the user get a message when a co-signer has undone their signature?

Yes. User should get a message when each codefendant signs.  This Is not a MVP feature. If there are compelling reasons not to have it that is fine too. Ask lead UX person in the future.  Now that person is David or Caroline.

Also need to add an email to go to user if CoD refuses to sign ala issue #33 Text of that email in #33

Decision made. Implementation discussions in other issues. (#33, #111, etc.)


Elvis has too many domains and none of them are good

Currently elvis has a number of different domains, some of which are even hosting different content:

https://github-issue-search-client-silverstripe.vercel.app/

This is what vercel auto-deploys to when stuff is merged in.

https://silverstripe-github-issues.vercel.app/

Seems to be using an outdated codebase, so must be hosted separately somewhere

https://silverstripe-github-issues.now.sh/

This is what the docs (indirectly, via redirector pages on .org) point to
Redirects to 2 above

http://silverstripe-issue-tracker.silverstripe.org/

A half decent subdomain which redirects to 1 above.
No ssl cert, so https links make the browser grumpy.

Epic

https://github.com/silverstripe/.github/issues/141

Acceptance Criteria

Only one version of the vercel app exists
We have a good subdomain that we can easily refer to in docs
The subdomain doesn't redirect anywhere, it's the direct URL for the application
The subdomain has a valid SSL cert, and accessing http redirects to https
Links in docs are updated to use the new subdomain instead of redirector page URLs
Issue and PR templates link to elvis wherever they encourage people to look for existing issues
The new subdomain is called out in the triage and peer review page

Closed as a duplicate of https://github.com/silverstripe/github-issue-search-client/issues/139 which will handle this along with a few other things


[hi] Localize  Tightly Coupled Architectures in hindi.

Signed-off-by : Aarav238<EMAIL_ADDRESS>Describe your changes
I have localized the Tightly Coupled Architectures in hindi.
resolves #2640
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

Netlify build has been failed!

@kumarankit999 How I can resolve it ?

I have resolved them @kumarankit999

Please approve this @kumarankit999 .

@bishal7679 check this one PR!

I have replaced all the words with exact words. @Garima-Negi  @abhay-raj19 @kumarankit999 @bishal7679 Please review.

Thanks @Garima-Negi .

Thanks for contributing @Aarav238


Bump jansi to 2.4.0

Almond uses jansi's HtmlAnsiOutputStream to convert the ammonite's output (ansi) to Jupyter output (HTML)
However, jansi dropped HtmlAnsiOutputStream support (at least from 2.0.0), and the API of AnsiOutputStream has quite changed. see: https://github.com/fusesource/jansi/issues/210
This commit adopt the jansi v2 compatible HtmlAnsiOutputStream implementation from https://github.com/fusesource/jansi/pull/212 Even though the PR isn't merged because the maintainer doesn't believe there's no demand on HtmlAnsiOutputStream, but the code seems legit, and I believe we can safe to use that implementation.

This is used when updating vars values in previous cells' output in particular

it works fine :+1:

Does it work fine locally?

yes, but I found the declaration of var's color is a bit different from the previous version (the first cell)
previous

now

maybe it should be the same color with the previous one?

maybe it should be the same color with the previous one?

It might be a bug. It seems both the : and the = should be black (they should be after an ANSI reset I think). Could there be an issue with resets?

It might be a bug. It seems both the : and the = should be black (they should be after an ANSI reset I think).

Ah, right, I'll take a look

before

    x
  :

    Int
   =

      3

after

  x:
    Int =

        3

you're right, it seems a bug

Looks like the current implementation ignores ansi's set color default
incoming ansi escaped code x: Int = 1 ([39m should set the foreground color to default)

https://github.com/almond-sh/almond/pull/993/commits/9bf3dbfc9361bf4bf4f6ac8de5888f49c9387145 fixed HtmlAnsiOutputStream to take set to default color into account (now, : and = is black (default color))
https://github.com/almond-sh/almond/pull/993/commits/e3150b947491a6e4563a562c74663e8446b9b0f7 restore the RGB_COLOR_MAP

now the color is the same as the previous one.

I misunderstood HtmlAnsiOutputStream is used for all the cells, but it turns out (as you mentioned), it's used only for var stuff.
Just out of curiosity, how are other things like val x = 1 colored?


Doesn't work with select2 v.4.0.6

Same here, I switched to version 4.0.3 in cdn url, as it is the version used on this jsfiddle : https://jsfiddle.net/wasikuss/7ak9skbb/

I have a patch for 4.0.6-rc.1 here:
https://github.com/prij/select2-multi-checkboxes/commit/3ffe044da83a0c1e8daef82568797811c47412e3#diff-b198f2bcf63152df67a84c5056b556f5

fixed in #25


[bug] The "editions" field in the minting page is croll sensitive

Describe the bug
The "Editions" field on the minting page is an integer field and thus "scrollable" - due to the minting page layout, if you fill out the editions last and scroll down to "upload objkt" it will reduce the edition size
we had this issue on other fields and it got fixed, it should get fixed here too
additional: it would be great to include "edition size" on the preview page so creators can check the edition size whe4n minting

Reproduction

go to https://teia.art/mint
2 fill out the form including a higher edition size
while the edition size field is still activated, scroll down to find the "upload objkt" butt

Expected behavior
the edition field to not be sensitive to scrolling
Platform and versions
windows, firefox explorer
(but i am pretty sure it goes for all browsers)

Console output
No response
Additional context
No response

Is the solution to change focus on the field after a timeout, or can the scroll behaviour be changed?

I think we will just disable scroll affecting fields all together, I think it can make sense for interactive controls with feedback but for fields on a form not so much


Last update removed access to configuration

It might be something I did incorrectly, but the access to the configuration has disappeared after the last update.
I have tried removing and reinstalling the extension but nothing changed.
Is there a different approach to use it now?

Thank you!


Expose group_info() for Regex::Captures

Hi there!
I'm writing a wrapper for regex crate's Regex::Captures struct, and for this I need to get the group_info member of regex_automata::util::captures::Captures.
But I found that the Regex::Captures type currently does not have a way to export its caps attribute or the group_info() method of the caps attribute.
Would regex willing to accept a patch that expose group_info()? Or Maybe I should head to use regex_automata?
Thanks!

You can't just expose GroupInfo. That would make regex-automata a public dependency of regex, which it is very intentionally and very carefully not. So to do that, we'd need to make a wrapper type for it.
In general, I do not expand the API of regex without good reason, and I don't see any motivating examples here to argue in favor of that direction.
Yes, you should be using regex-automata.


Feat: - Add color picker

Is your feature request related to a problem? Please describe.
The current website allows users to input colors using HEX codes, RGB, and HSL values, but it lacks a visual color picker. This can be challenging for users who prefer selecting colors visually. I noticed that the design you shared (design/V01_design.png) includes a color picker, but it’s missing from the current website, making the process less intuitive for some users.
Describe the Solution You'd Like
I would like to add the color picker from design/V01_design.png to the website, allowing users to visually select colors. This would make it easier for users to choose their desired shades without needing to manually input codes, providing a smoother experience.
Describe Alternatives You've Considered
An alternative could be to continue with the current input methods, but this does not address the need for a more visual and user-friendly option.
Additional Context
Adding the color picker would align the website with the provided design (design/V01_design.png) and improve the user experience, making the color selection process more intuitive and accessible.

please assign this to me under Hacktoberfest

This is a good feature request, but actually I took the decision of removing the colour picker because it kind of makes the RGB, HSL, and HEX Input duplicated (since Color Picker already has these 3 categories). But I understand the concern of visual colour picking.
We probably need something like the picker here - https://rgbcolorpicker.com/

Thoughts?

This would be a great idea. I can implement this, but how do you want the color canvas in the design? My suggestion is that when the color bar is clicked or dragged, the canvas will open, allowing the user to pick a color. If the user clicks anywhere else outside the color bar while the canvas is open, it will close automatically. What do you think?


Feedback for guides

User Guide

Too lengthy
Clash between conversational and technical tone
Merge some commands on TOC (e.g., done and undone; today and tomorrow)
No need to be too creative.
Actual link in brackets

Introduction

alone is extra
Not sure if affinity for typing is appropriate

Getting started

Where is the release tab? (Perhaps we don't even need the screenshot if we have link)

Visual Walkthrough

Superimpose on top of actual screenshot

Functional Walkthrough

Not sure if it's necessary
Can use in the software demo

Command summary

Rephrase multiple words.
Use examples to show multiple words instead of using dots.
KEYWORD(S) might be clearer.

Commands

Repeat case-insensitivity & order not important in command summary.
Change // to something else, e.g., bullet point.

Add

Can move description of tags to other command.

Remind

Fun fact is not fun.
Ever wondered -> If you wonder
Remove behind the scene
Show screenshot of multiple search results when there's ambiguity
Update example -> note is now compulsory

Edit

Cut it down, not completely
"Any new tags you add will replace all exising tags."

Editr

Refactor "this is a variant of edit command but for reminders"

Rename

Refactor "all tasks with that tag to the new tag"

Done

Remove would you?
Remove dot at the end of example.

Delete
Freetime

Show screenshot perhaps.

Today

"All events that occur today."

Tomorrow

"All events that occur tomorrow."

Undo

"the following command(S)"

Help

"we will tell you all that you need to know" -> what you actually show
Perhaps move to the end, show screenshot and double as command summary.

Exit

Too much, sounds like someone is dying.
Not necessary to say that it's equivalent to clicking X.

Conclusion

Issue tracker -> show link

Developer Guide
Introduciton

"being an open-source projects, we"
"who wants"

Setting Up

Link

Coding Standards

Link

1 command -> one command

At most one command

CI

following things -> what things

dependecies -> dependencies

Is the architecture acceptable?

Sequence diagram.

Dev ops is like text book rather than instructional.

Submit by Nov 7. Email if need feedback.

Mention that we need to add command to CommandManager when adding a new command.

Presentation

Introduction

Good morning & address the audience (how, since audience is not the user)
The problem statement
How our app solve the problem

The software
Functionalities of the system

Add
Today & tomorrow
Done
Remind
Path

Special functionalities and how they work

Freetime
Alias
Shortcuts
Command suggestion
Undo

Conclusion

Made most of the textual changes in user guide. Still need to add screenshots.


Update cargo.lock and add vscode to gitignore

Relevant Issue (if applicable)
If there are Issues related to this PullRequest, please list it.
Details

Update cargo.lock to latest
Add vscode to gitignore for all contrib subdir

Types of changes
What types of changes does your PullRequest introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Go over all the following points, and put an x in all the boxes that apply.

[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Need squash.

/retest


Add Lea Verou TAG F2F January 2024 travel request

Meeting info: https://github.com/w3ctag/meetings/tree/gh-pages/2024/01-london
Note that while it says 23-25, 22 is agenda planning day, it's just technically optional, so I plan to arrive on the 22nd.
Optimistically requesting $2000 broken down by:

Flight: $800
Hotel for 4 nights: $800 (London is expensive)
Ground transportation (including to/from airports): $200
Meals: $200

@LeaVerou This request just happened to occur right when we were revamping the travel fund :)  I spoke with @joesepi on this one and:

The request is approved, we do not want to go back and have you follow the new process since the event already happened!
Please note for future reference that meals are not currently covered by the travel policy (either old or new)

Go ahead and file expenses using the standard process.


Sharp not working on Some Windows 7

I am really tired what is not working in my code. I install venom-bot which has sharp as a dependency. Sharp is installed correctly no errors. But I am getting errors on some windows 7 pc as following -

It says sharp.node is not found while I checked its right where its checking it to be.  I uninstalled reinstalled and rebuild. But all failed. Does anyone here has any idea?
@lovell Please, I am in a tough situation if you can help.

Did you see the documentation relating to installation?
Have you ensured the architecture and platform of Node.js used for npm install is the same as the architecture and platform of Node.js used at runtime?
Are you using the latest version? Is the version currently in use as reported by npm ls sharp the same as the latest version as reported by npm view sharp dist-tags.latest?
If you are using npm v6 or earlier and installing as a root or sudo user, have you tried with the npm install --unsafe-perm flag?
If you are using npm v7, does the user running npm install own the directory it is run in?
If you are using the ignore-scripts feature of npm, have you tried with the npm install --ignore-scripts=false flag?
What is the complete output of running npm install --verbose sharp? Have you checked this output for useful error messages?
What is the output of running npx envinfo --binaries --system?

@lovell As I said, there is no install error in sharp. Sharp is being installed successfully, I am on node v13 and tried everything other than npx envinfo. Will give it a try too.

@lovell does \?\ before the path have any relation to the problem?

@bhavya32 Were you able to make any progress with this? If you're still having problems, please can you provide the requested information.

@lovell Didn't find any solution. Tried many things, but none worked. I had to remove sharp to run on those systems.
Those were not my systems. I fear I won't be able to provide the requested output as none dev machine faced that. I can just assure that sharp installed correctly without any error.  Don't have access to those machines now. You can close the issue.


namespace hardcoded to ceph

++ kubectl get pods --namespace=ceph -l daemon=mon -o template '--template={{range .items}}{{if .status.podIP}}--add {{.metadata.name}} {{.status.podIP}}:6789 {{end}} {{end}}'
No resources found.
+ MONMAP_ADD=
+ ((  timeout--  ))
+ sleep 1
+ [[ -z '' ]]
+ [[ 7 -gt 0 ]]
+ [[ 0 -eq 0 ]]

Image tested: docker.io/ceph/daemon:tag-build-master-luminous-centos-7

What is this command supposed to accomplish?

Sorry for not being clear, this is a copy/paste from the log output.
I've installed with the helm charts to a different namespace, but in the logs I still see commands are run with ceph namespace.
It might be coming from the variable CLUSTER=ceph (aka ceph.cluster=ceph in values.yml) , and ignoring the namespace being set at helm install.
It's a minor thing, but the limitation was not obvious at the beginning.
I'm using ceph namespace to work around it.

No worries, I recently pushed a change here: https://github.com/ceph/ceph-docker/blob/master/ceph-releases/jewel/ubuntu/14.04/daemon/config.k8s.sh#L19
I might forgotten to build a new image, doing this now.


Update shopware6.php

Microoptimization

Q
A

Bug fix?
No

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

@peterjaap I can remove the commas and spaces again if you prefer. I prefer the commas at the end of the array. They make, that with the next change, you only have one changed line on adding a line <3

Perfect :+1:


Feature/passenger field

I added filtering on passenger field in the new drive form (using v-select)
This PR closes #233

Does anyone know why there are old commits showing up?

Tested locally right after latest commit (fixes local storage caught by @magul) and it works fine.
Green light from my side.


Update tlds.txt

add edu.vn

@haxuanman I am sorry, there is a solid reason for not doing so. Thus, I have to close this request without merging. Thank you.


Overrall input system optimization

changes:

uses Set instead of Map
fixed the render-blocking behavior of tappedColumns
optimized the lookup of keybinds
simplified anyKeyTapped

Looks good, thanks for another PR 🙏


Local Abandoned Accounts metric change - f'ing James.

====================================================
Local Abandoned / Leftover Accounts
----------------------------------------------------
Total Detected: 8 / 1378
====================================================

Press ENTER to continue...

Change 2nd value to be ungrouped total of accounts.

Also, update label for clarity of metrics displayed.


Enhancement: Siacoin Wallet Sync Times

Hello,
I have been testing out Siacoin system for a few days.
Simply put it took me 3 days to sync a siacoin wallet so I could use it to confirm a transaction. Let's put this issue into perspective:
Amazon Cloud = Takes less than 1 hour to setup.
Google Cloud = Again takes less than 1 hour to setup.

Siacoin = Takes 3 days to setup (depending on type of hard drive).

In the modern world we live in, it's all about speed. Bitcoin is very slow for example taking 1 hour for some transactions and other cyrpto's like Ripple for example take 3-5 seconds. The world is moving across to Mobile and this is why Google is changing it's whole search engine from Desktop over to Mobile with what they call "Mobile First Indexing". Let's try to imagine trying to setup a Siacoin on a Mobile Smart Phone would it take 3 weeks to setup the wallet syncing?!
Solution and some common sense:
There are two types of users:

1. People that want to use Siacoin and don't care about hard drive hosting. These people should be able to open a Siacoin wallet and use it in a few seconds and NOT have to worry about syncing at all.

2. People that want to use Siacoin and host or use the hosting features. These people should be the ones that wait for the syncing.

You really need to update your wallet to work like this! With Mobile payments in the coming years being the biggest form of payments over Desktop payments, Siacoin needs to address these issues and change it's UI-Siacoin Wallet.
Lastly, sorry if this topic has been covered before, but I have only been testing out your system for around 7-10 days now. I feel it's a good project but the syncing is crazy and pure madness enough to make this project totally fail if not addresses asap.
Who is going to say let's use Siacoin over Amazon cloud to host their website files and have to wait 3 days to set it up. Right now the block chain says around 123,000 blocks so it's still very small, for example burst coins are around 300,000. Does this mean next year people are going to wait 6-7 days to sync their wallet?
I look forward to the replies and hope this issue is going to get addressed properly.

We never expected the average user to sync the full blockchain -- you are absolutely correct that it is impractical for them to do so, at least with today's blockchain tech.
Sia is a backend, not a user-facing app. Our current user interface exists only to bootstrap the network. Long-term, the average user of Sia will not even know they are using Sia. They'll use a service like Dropbox, and Dropbox will transparently store their data on Sia. In this scenario, only a few thousand large entities (and a few crypto enthusiasts) need to deal with the blockchain directly. That's better anyway, since blockchains run into significant scaling issues past that point.
Hope this clears things up a little. And thanks for being an early adopter :)

@lukechampine Thanks for your reply, I see this project having a 'possible' big future, when I read that you guys want to add Video Streaming features later on. I can almost compare Sia to Megaupload doing the same thing. But my downside to the project is just thinking when a user opens a bitcoin wallet it takes them less than an hour, whereas Siacoin takes 72 hours (3 days). So it's really NOT energy saving as your video promo advert let's you believe coz users have to use 71 hours more computing power just to open a wallet.
I think this project is good, however because it's 'pegged' to a currency called Siacoin, steps must be made to make Siacoin strong in order for the project hosting to succeed. If Siacoin value was to drop to 0.0000001 BTC would you really want to continue building it?
So I hope the developers will try to fix this issue as it's a big one I feel and this is the only thing from stopping Sia becoming as big or bigger than Google and Amazon for cloud hosting.
I would like to know where I can keep in the loop with this project and hope the developers try to fix this issue with the syncing. As for the wallet design, actually the design is quite good, it's the syncing which is the only problem and has the power to kill off this project and other projects like Burst coin, File coin etc.

I think this project is good, however because it's 'pegged' to a currency called Siacoin, steps must be made to make Siacoin strong in order for the project hosting to succeed. If Siacoin value was to drop to 0.0000001 BTC would you really want to continue building it?

If the value drops, then it will become very cheap to store data on the Sia network. The increase in demand then allows hosts to raise their prices. In the long term, the supply/demand relationship between renters and hosts causes the value of the Siacoin to stabilize around the value of storage on the network.
The best way to stay in the loop is to join our Discord server. We also have a newsletter (bottom of the page) if you just want infrequent updates.


[DOC] Running the Thrift JDBC/ODBC server with CarbonExtensions

Why is this PR needed?
since version 2.0,  carbon supports starting spark ThriftServer with CarbonExtensions.
What changes were proposed in this PR?
add the document to start spark ThriftServer with CarbonExtensions.
Does this PR introduce any user interface change?

No

Is any new testcase added?

No

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3552/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbon_PR_Builder_2.4.5/3552/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5312/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12444/job/ApacheCarbonPRBuilder2.3/5312/

LGTM

LGTM


Parallel configurations of the same project can result in corrupted cpm-cache

When configuring the same project twice in parallel and with cpm cache enabled, it can happen, that a dependency is created with an empty / corrupted git repository. The configuration will then fail for both.

Parallel configurations should now be supported in v0.37.0.


Ability to remove the added images

Hello there, while it is good to be able to add as much stuff as we want to the tier list, sometimes it's possible to add duplicates or the wrong content. For people who add to the list piece by piece and rate in order to get a baseline this can be hard as you have no choice but to start over. It would be pretty epic to have the ability to delete specific images.

Good idea, I added it in d5ed8670d461ffe86b1baa663d964fc187871537


[CogVLM-chat-v1.1] LM weights are different with vicuna-7b-v1.5

While CogVLM is trained, LM weights are fronzen.
From my observation however, the LM weights of cogvlm are different with Vicuna
Vicuna: https://huggingface.co/lmsys/vicuna-7b-v1.5/tree/main
CogVLM: cogvlm-chat-v1.1 (both from HF or SAT)
Can I ask why or the proper source of the language model?

CogVLM-Chat-v1.1 (SAT)

Vicuna-7B-v1.5

I have migrated the CogVLM-chat-hf to mindspore and found the modle works well when the input including image and text, but if there is only text query without image, the performance is not so good, it may relate to this issue I guess.

I have migrated the CogVLM-chat-hf to mindspore and found the modle works well when the input including image and text, but if there is only text query without image, the performance is not so good, it may relate to this issue I guess.

@antigone660 In text-only mode, the prompt template is different. Did you use the following prompt for text-only query?
https://github.com/THUDM/CogVLM/blob/b37f36bf5df8c0356920b70f21338c1968c20e47/basic_demo/cli_demo_hf.py#L52

@minostauros Thanks for your reply I did not use the template before and it works now : )

So, the LM had also been tuned? I also found that the LM weights of cogvlm2 are different with llama-3-8B-Instruct.


Next and previous not working as expected

Steps to reproduce :

Open the pdf, press next/previous in the toolbar, the pages are loaded in steps of 1. (works fine on first try)
Close the pdf and open again.
Press next/previous button. The page is now loaded in steps of 2. (ie page 1,3,5..)
For subsequent opening and closing of the pdf, the step size is increasing with count one.

Please suggest on how to tackle this.

Did you find a workaround ? I am facing the same problem

hello all
We are also facing same issue because event listeners are getting added again and again whenever pdfjs-viewer directive gets laoded. It is making to fire all the events again and again please help to fix this issue. I have two templates which are using same directive but controller is same. Whenever i navigate between these two viewes events are getting added again and again


Add dunst service

Motivation for this change
This is a fresh attempt at implementing #19183
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Good candidate for NixOS/rfcs#42

Good candidate for NixOS/rfcs#42

Yes, the rules part is probably too verbose. I wanted to internalize it more, in order to check for naming conflicts with official section names. But I agree, there is no need to go that deep into individual options for that.

Other changes:

Use writeText instead of toFile to allow e.g. rules to reference storepaths properly.
Don't add the wrapper to systemPackages by default

I still have some problem with running scripts from the systemd services context though.

I pushed some changes to use the upstream systemd unit.

ping (triage)

Reading this code again after some time, I wonder if that is the right way to create the wrapper.
The problem was/is I think, that there is no sane way to reuse the upstream service file without overriding the original dunst derivation. If someone has a suggestion how to do this properly, I would very much like to hear it.

Ok, I have an alternative implementation here using symlinkJoin (similar to how the sway and way-cooler modules do things). But at the cost of not being able to reuse the upstream systemd service file.

@xaverdh Well it looks like you have satisfied every request in the review of this PR so far... so let us know when you make up your mind because from skimming the comments in this PR I think a merge is in order :smile:

Ok, lets merge this as is then. The implementation can still be switched later on.

(triage) @dotlambda From the comments it looks like final word is yours. Please merge if you are satisfied.

Thanks to Infinisil for the extensive review!
I pushed some changes to apply most of your suggestions.

Status? Can we get this ready in time for 19.09?

Are there any news regarding this PR's status? i'm really interested in this service 😀

It would be good if somebody could try to see if it works how they expect.
And check whether we should add some default config or at least more examples.
Upstream has a big default config file with lots of explanations: https://github.com/dunst-project/dunst/blob/master/dunstrc

I'd really like to get this in, so I picked added some defaults, examples, and fixed merge conflict here: #112100

closed in favor of https://github.com/NixOS/nixpkgs/pull/112100