make Latest Screenshot Proxy Object image-type configurable

I love the new Latest Screenshot Proxy Object. However, I generate a mix of .png, .jpg, and .gif files in my screenshots folder.
Is it possible to expand the image types the Proxy Object looks for? The Proxy Object seems to only find .png files at this point.

The Latest Screenshot proxy uses Screenshot.app's settings to determine what type of screenshot it pics up.
The default of Screenshot.app is to create PNGs, but you can change this at any time by following the instructions here: https://www.laptopmag.com/uk/articles/change-default-screenshot-file-format-macos

Thank you — I switched the system default via the command line and it does correctly find the last screenshot JPG I took using the system command. However, that screenshot is years old because I use a screenshot utility.
This might be too niche to be useful to anyone else — but I guess my request is a way to configure the proxy object just search the screenshots folder for the latest image of any time. However it seems proxy objects aren't user-configurable in that way.
I'm also interested in attempting to "roll my own" — but not sure where to start with that.

I've created another issue here: to make this a feature request, but currently it's not quite possible.
What you could do is use the QS Command line tool (install the plugin) to open the latest file:
ls -At /path/to/my/file/*{*,.*} | head -n1 | qs


Feature/#57 fullscreenComponent, UserSettings Modal, Data Aggregator

what has been done:

Fullscreen Component is callable for images and Poster images. state hook callback
Settings Modal Card is now a component and callabel from two components
DataRandomizer helper class for Bio, MemberSince-Date, HomeTown.
Profile Edit button only visible if current user
ImageComponent changed in PizzaComponent 1.54

Würde die Fullscreen Komponente eher in ImageModal umbenennen, dann ist der Name etwas näher am Verwendungszweck.


CORGI-662 create variants for et relations

Associate streams using BREW_TAG relations with ProductVariant where there are erratum that link the two. For example if there is a ProductStream eg. openshift-4.13.z, which uses brew_tags to associate builds with that ProductStream, when one of the builds gets released with the 8Base-RHOSE-4.13 ProductVariant the stream will then be associated with that ProductVariant.
This should allow us to associate builds and their components with ProductVariants where that wasn't possible before with stream using BREW_TAG relations. It also allows us to build a list of CPEs for streams which don't have errata_info, composes, or yum_repository relations.

@RedHatProductSecurity/corgi-devs please review the design of this. It's not functionally complete, but remaining changes should be small.

I ran this on all the existing relations and it only uncovered new variants for 4 streams. So I don't think it's worth the added complexity and maintenance burden to do this type of linking.


--wait help for layout installations

Could the support article be updated to explicitly state that the --wait parameter must be used immediately after the executable. Eg. vs_Professional.exe --wait --norestart. Just to make it a little less ambiguous.
Thanks!

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

ID: ff556dcf-4de1-8483-702a-73699062be1f
Version Independent ID: cba846ae-6af4-af49-b194-28b546169426
Content: Create a network-based installation of Visual Studio - Visual Studio
Content Source: docs/install/create-a-network-installation-of-visual-studio.md
Product: visual-studio-dev15
GitHub Login: @TerryGLee
Microsoft Alias: tglee

@taylorbindon Thanks for your feedback; we appreciate it! Your request is on record and in the queue. We'll report back soon.

How can this important change take so long time to fix? Without this little parameter --wait it just dont work. So please update for all developers out there that missing to add this... like for me.

@taylorbindon I'm going to close this issue now; if you do get a chance to see if you can reproduce the error and want to report back, we'll reopen it then.


@Optional Issue

Hey so am trying to use @Optional from the example in the README.md and for some reason it doesn't work for me.
Am using the latest version. (1.2.10).
This is the code of the command:
    @Command({"balance", "bal"})
    public static void balance(@Sender Player sender,
                               @Optional Player target) {
        if (target == null) {
            sender.sendMessage("Your balance is: $100");
        } else {
            sender.sendMessage(target.getName() + "'s balance is: $100");
        }
    }

This is in the Main class
        Blade.of()
                .fallbackPrefix("fallbackPrefix")
                .containerCreator(BukkitCommandContainer.CREATOR)
                .binding(new BukkitBindings())
                .build()
                .register(BalanceCommand.class);

Issue:

I just ran into this issue as well, I'll look into it tomorrow.

Fixed in 2.0.0.


Seemingly empty alt text due to special whitespace chars

Describe the bug
I recently saw a post where the author had added the blank space character (char code: 32) as the only alt text, resulting in the alt banner appearing over the image, but then there's no actual alt text.
so it's currently possible to include only whitespace as the alt text, with no other characters present.
To Reproduce
Steps to reproduce the behavior:

make a new post containing an image
add the alt text " " <- copy paste this or use String.fromCharCode(32), or any other special whitespace char you wanna test.

Expected behavior
if the alt text contains ONLY special whitespace characters, strip them out.
Screenshots

Details

Platforms: all
Platform version: n/a; latest
App version: latest

Additional context
seems like there's a few ways this could be solved
but the right way is probably just adding a guard to the model or sanitizing for special whitespace chars when the alt text is set: setAltText in the image model

@ansh handing to you


chore: disable CodeCov comments

During Thursday sync, we agreed to try disabling the CodeCov comments on PRs b/c they can be noisy. Note: the code coverage on PRs still gets uploaded to CodeCov via make test-coverage
Ref: https://docs.codecov.com/docs/pull-request-comments#disable-comment

Summary by CodeRabbit

Chores

Updated configuration to disable automated comments from the code coverage tool.

IMO not really. I don't find it helpful so I originally proposed removing it entirely


Assignments in & functions

As discussed in Discord, it's nice to be able to assign to properties in a pipeline, especially fat, as an alternative to .. cascade notation.  The DOM manipulation example at the top of reference.md is particularly compelling.

Happy Thanksgiving! And thanks for the reviews 😊


Improvement request:    get.state for Player  & Houshold

Hi there,
I'm using your node for a while with great sucess, all works seamlessly .. thx a lot
You currently have a node Group.get.state ... is it possible to have the same for Player & Houshold ??
e.g.  Player.get.state  ... Household.get.state
this would simplify the collection of state information dramatically , avoiding several get calls
thanks for considering
goog wook .. keep going

Hi. For what do you need it?
There is no single SONOS command to get the state. group.get.state sends 6 requests to the SONOS-Player, extracts the relevant information and output them. Most other group.* commands take 2 requests.
For a better understanding:
player.get.state: All data are group related (playbackstate, group volume, group mutestated, ....) so a player.get.state for any player in a group would provide the same data as the the group.get.state.
If you are interested in the player volume and player mute state you can already now get these data for all player in a group with group.create.snap
householde.get.state: I could implement it by sending group.get.state to all group coordinator. It will take some time as everything is serial and each command needs 6 request. With nodeRED split/join it is parallel, therefore much faster and already now available with node-red split/join. See this example: example 9


Extract THINFS to it's own project?

It's not LaTeX specific, perhaps it's useful for other similar compiled tools?

Related to this, we should maybe use -js-library instead of just placing tTHINFS in pre.js...

I've raised the question on https://github.com/kripken/emscripten/issues/5132, will action based on response.


Update auto-value-annotations to 1.10.2

About this PR
📦 Updates com.google.auto.value:auto-value-annotations from 1.10.1 to 1.10.2
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.auto.value", artifactId = "auto-value-annotations" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.auto.value", artifactId = "auto-value-annotations" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #362.

Superseded by #365.


Rinsing: Remove Field

Remove the Instrument Wattage field

ready for review and merge on pull request https://github.com/rosalindfranklininstitute/empiar-spw/pull/46


Add tag to botbuilder-ai-orchestrator so shows up in Composer

Versions
What package version of the SDK are you using: 4.13.1
What nodejs version are you using: 10, 12, 14
What os are you using: Windows
Describe the bug
Please add appropriate tags to the Orchestrator recognizer package to enable it to show up in Composer.
https://www.npmjs.com/package/botbuilder-ai-orchestrator
There is a workaround to add a custom feed.
To Reproduce
Steps to reproduce the behavior:

Go to 'Package Manager'
Select 'npm'
Scroll down - no orchestrator package.

Expected behavior
Orchestrator package should show up in npm feed.

NOTE: Workaround:
Add a new feed without the filter..

We'll likely take this in a patch release

Shiproom - meta data changes.  low risk for a core scenario


Improve parsing/handling of command line options

The current approach, using Apache CLI is verbose from a code standpoint and rigid from a usage standpoint (e.g. expects options in a specific order).  Consider replacing Apache CLI with another library such as JCommander or Args4J for improved handling of command line options.

JCommander++

Related to #331  - Ensure that CLI errors are shown in a prettier format.


ci: fix workflow quotes

For some reason it seems the paths-ignore rule is not being considered upon releases, which causes the release process to abruptly end by a new workflow instance that cancels the running one. All of the GH docs show to use single quotes, so it may help, although I'm not positive.

Codecov Report

Merging #56 (15f6171) into develop (846a044) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop     #56   +/-   ##
=======================================
  Coverage     0.09%   0.09%
=======================================
  Files            6       6
  Lines         1010    1010
=======================================
  Hits             1       1
  Misses        1009    1009

Flag
Coverage Δ

unittests
0.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

:tada: This PR is included in version 0.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 0.6.0-develop.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Upgrade Subscription on Android doesn't cancel current subscription

When I use:
const transaction = await Glassfy.purchaseSku({ sku: newSku, , skuToUpgrade: currentSkuToUpgrade });
sku: a new subscription plan
skuToUpgrage: the active subscription
Glassfy creates a new subscription without cancelling the current one. Since I can't use base plans in the play console because I need a product id for each SKU I don't know what to do.

Hello, the Android SDK should indeed automatically cancel the previous subscription.
Your code looks fine, I'll be investigating this and keep you posted!

Sorry it was our fault. 0ur project was created with an old capacitor version. So npm installed the plugin version 1.x
Upgrading to<EMAIL_ADDRESS>fixed the issue.

Ok, thank you for taking the time and let me know!


K8S Setup

Hello,
I am trying to setup zeebe-simple-monitor in k8s.
The first problem I hab was CORS. Because the origin header is set to the Ingress FQDN, the ws upgarde is rejected. I already build the service with the ingress as allowed origin, which makes the upgarde working.
Now I am running into the next Problem. When I try to upload a new deployment, I am getting the success message. After a refresh nothing is there. Not sure where the problem is here, I will investigate this further next week, but it is quiet wierd that i am getting a response code 200 evene if the deploy was not successfuel.
I think it could also be some cors stuff because if i use kubectl port-forword and access the pod directly, everything works fine.
Also for future references it would be greate to haven an example yaml file for k8s with:

deployment
cluster ip
ingress

After further investigantion, the deployment problem only occures, if you have multiple gateways (see camunda helm chart) in front of the zeebe engine.

PR #477 merged.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v4.14.240",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=5b06ca113bf197aab2ab61288f42506e0049fbab"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "5b06ca113bf197aab2ab61288f42506e0049fbab",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "misc/libmasm/module: Fix two use after free in ibmasm_init_one\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v4.14.240 by commit 5b06ca113bf197aab2ab61288f42506e0049fbab. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001421


MetricsClient opens new socket on every request, starving ephemeral ports when doing lots of request to the API

Hi all,
I use MetricsClient to fetch metrics about my cloud sevices (VMs and web/worker roles). When I'm doing a lot of calls, I've found that there are a lot of sockets open to management.core.windows.net. I've debugged this to the point where I found that every call to the API opens a new ConnectionGroup in the ServicePoint for management.core.windows.net, this causes the connections not being reused but still staying open for (a default of) 100 seconds before they are closed. This can become a problem   fast if you're doing lots of calls in a resource-constrained environment (such as WAWS).
My current workaround is setting ServicePointManager.MaxServicePointIdleTime to a lower value than 100 seconds, but I don't really like that since it's a global setting and 100 seconds is probably a sane default for other use cases.
Any advice?
Cheers,
-Freek

@FreekPaans Thank you for the feedback.  Assigning to the service team to investigate.

Thank you. I should add that this doesn't happen if you reuse the MetricsClient instance, only if you create a new one for every request. Either way, the connections are not cleaned up if you dispose the MetricsClient.

Apologies for the delayed response.  We are planning a broad set of changes in the metrics area to greatly improve the consumption of metrics data.  These changes will go live in the next few months

@FreekPaans please open a new issue if the problem still exists/repros.


feature: decrementing Nonce or gen_mid_nonce()

For two way communication ever i.e. secretbox it's crucial to avoid nonce collision when incrementing nonce scheme is used. Current solution for second-party would be i.e. to feed pre-made slice (with half of slice set to 0xff and second half to 0x00) into Nonce::from_slice (we cannot just set MSB in the slice because we don't know whether from_slice is BE or LE).
In my opinion lack of built-in solution unnecessarily leaves users' apps prone to implementation errors.
It would be nice to have either:
decrement_le()
or
gen_zero_nonce() and gen_mid_nonce()

That would be very clunky.
Don't reinvent the wheel. The key exchange function (crypto_kx) returns a pair of shared keys. Use one in one direction, the other one in the other direction.
And unless ordering is not important, just use crypto_secretstream, so you don't have to worry about nonces or key rotation.

@jedisct1 I'm new to rust, read wrong docs https://docs.rs/exonum_sodiumoxide/0.0.20/exonum_sodiumoxide/ (no crypto_kx there), partially because docs link from crates.io does not exist and so do https://sodiumoxide.github.io/.
crypto_secretstream will suite me best (from libsodium docs I see that I can calculate ciphertext length from plaintext length, am I right?)  but I don't see it in sodiumoxide yet, thankfully there's a PR with it.

@kac- yeah there is some issue with cargo doc atm. you can check the 0.1.0 docs here


Question

if iam correct it only sends the request from your IP?, how would you even call it a ddos if it would get affected as the modern systems are way more secured ((iam not intrested in using any ddos applications just saying))

It's not sending directly from your IP; instead, it's sending from multiple different random IPs while also spoofing your actual IP to hide your location.
But when it comes to HTTP-based attacks (such as HTTP GET floods), the script cannot spoof IP addresses because the TCP handshake (SYN, SYN-ACK, ACK) is needed to establish a connection, and spoofed IPs would never receive the necessary response for completing the handshake. The server would simply ignore them.
You can only use proxies in HTTP flood.

It's not sending directly from your IP; instead, it's sending from multiple different random IPs while also spoofing your actual IP to hide your location.
But when it comes to HTTP-based attacks (such as HTTP GET floods), the script cannot spoof IP addresses because the TCP handshake (SYN, SYN-ACK, ACK) is needed to establish a connection, and spoofed IPs would never receive the necessary response for completing the handshake. The server would simply ignore them.
You can only use proxies in HTTP flood.

Hmm, interesting thanks for responding :)


Report from Liberia

[so unfortunately that this service is not available in Liberia with all the knowledge I have gained and encouragement I have made so far to bring others on board thanks for your understanding and hope to see Liberia on the list]

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

ID: 8da07311-409a-3de1-d501-93f6a798f6ff
Version Independent ID: 1878ec1a-a173-8234-65db-e7835f1c4438
Content: Frequently asked questions about Azure for Students and Azure Dev Tools for Teaching
Content Source: articles/education-hub/azure-dev-tools-teaching/program-faq.yml
Service: azure-education
Sub-service: education-hub
GitHub Login: @v-dihans
Microsoft Alias: rymend

@Gargar17 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@rymend
@tfosmark
I no longer maintain the topic. Please contact the other author mentioned in the issue post.
Thanks


0.13.1 was remarkably faster than 0.14.0

Output of restic version
restic 0.14.0 compiled with go1.19 on linux/amd64
(precompiled from github release page)
How did you run restic exactly?
$ restic_0.14.0_linux_amd64 --cacert certfile backup --exclude-caches --exclude /foo/bar/baz /foo/bar
(while RESTIC_PASSWORD_FILE and RESTIC_REPOSITORY are defined)

What backend/server/service did you use to store the repository?
the repository is in a local folder on the same disk array
(there are about 700 snapshots for the last 24 months)
Expected behavior
be (more or less) as fast as 0.13.1 was

Actual behavior
0.14.0 is remarkably slower than 0.13.1
I backup about 45,000,000 files (about 35TB) where most of the files aren't modified between backup runs. 0.13.1 needs about 4 hours 30 minutes (±15 minutes) while 0.14.0 needs about 5 hours 30 minutes (±15 minutes). I measured about 40 backup runs, 20 for each.
I also tested (10x each) on a subset where runtime is increased from 6 minutes 30 seconds to about 9 minutes.

Steps to reproduce the behavior
none

Do you have any idea what may have caused this?
no
Do you have an idea how to solve the issue?
no
Did restic help you today? Did it make you happy in any way?

I'm using restic for a quite a while now, on several machines. It was useful in many situations from accidentally deleted files to a broken harddisk. Thanks a lot for creating/maintaining it!

Did you enable compression perhaps ?
Did you migrate repo version from v1 to v2 ?

Pretty sure it's a duplicate of https://github.com/restic/restic/issues/3916.
@leapfog try a dev version containing https://github.com/restic/restic/pull/3927 on https://beta.restic.net/ (a binary that is more recent than Sept 12)

@Alwaysin seems you're right. Latest beta finishes as fast as 0.13.1. When running 10 times, latest beta even beats 0.13.1 (66min vs. 69min). Is it safe to use that beta or should I wait until 0.14.1?

Heard some people saying they run dev versions as production (not necessarily enterprise production, idk).
Maintainers seem to only allow quality code to be merged, but you never know, there could always be a bug. Better safe than sorry I'd say.
Anyway I think it's quite an important issue actually and it should IMHO trigger a release rather sooner than later (I am not involved in the project, it's just an opinion).

I'd run it.

@leapfog As there is nothing actionable in this issue, might you close it?

Except for #3916, a performance difference is mostly likely due to either #3878 or #3854. Actually, I'd be rather surprised/shocked if #3916 would make a 1 hour difference for a 45million files backup.

Is it safe to use that beta or should I wait until 0.14.1?

In general the beta builds are safe to use, although there could be some rare exceptions from time to time. That said, I'm not aware of a serious problem in the latest builds (as of this moment). Some smaller bugs like #3954 can show up from time to time, although these usually only cause temporary problems, which can be fixed by up/downgrading restic.

I did not mention that I do not backup all files in one run but instead have 13 folders that, in sum, hold these 35TB/45M files. Doing all at once, I'd have to keep (or forget) all the files for/at the same time. While doing individual backups allows applying individual policies. So in fact I do something like for d in folder*/; do restic backup "$d"; done.
Having that in mind, it's not that 0.14.0 needs an extra hour, but about extra 5 minutes per invocation. So at first glance #3916 and #3878 really look promising. (But isn't #3854 about restoring sparse files?)

The daily backup is done and yesterday's beta does indeed beat 0.13.1: Todays backup was done in exactly 4 hours while 0.13.1 needed something between 4:08 and 4:41 and 0.14.0 needed something between 5:14 and 5:44 (with each 16 backup runs).
I think I'll stick to the beta build and look forward to the next release. Thanks @MichaelEischer and others!

Having that in mind, it's not that 0.14.0 needs an extra hour, but about extra 5 minutes per invocation

That sounds indeed like an explanation. 5 minutes for such a large repository are approximately to what I'd expect.

(But isn't #3854 about restoring sparse files?)

Yes, but there's also an optimization to speed up chunking for files containing large sections which only consist of zero bytes.


Extracting Constants and Utility Functions to One File

Description:
Currently, our project has scattered constants and utility functions across multiple files. This makes it difficult to maintain and locate these essential pieces of code. We should extract all constants and utility functions into a single file to improve code organization and maintainability.
Benefits:

Improved code organization: Consolidating constants and utility functions into one file will make it easier for developers to locate and reference them.
Simplified maintenance: With a centralized location for constants and utility functions, future modifications and updates can be made more efficiently.
Enhanced code readability: Having a dedicated file for constants and utility functions will improve the overall readability and comprehensibility of the codebase.

Tasks:

Create a new file named constants.js and utils.js in the project's root directory.
Identify all constants and utility functions scattered across existing files.
Move the identified constants and utility functions to the new files.
Update the existing files to import the constants and utility functions from the new file.
Test the codebase to ensure the changes haven't introduced any errors or bugs.
Document the changes made and provide instructions on how to use the consolidated constants and utility functions.

I'll work on this, please assign this to me.


Purity calculation error

Currently purity calculation fails (at least for the Taylor 1.5 integrator), since vec_soln is a list of numpy.array and the purity calculation tries to square the list. Should probably refactor everything so vec_soln is a 2d array.

Wrote test that caught offending behavior and fixed by using numpy.vstack in place of  list.append.


item cabinet rework + stuff

About the PR

rework all ItemCabinet's to use Openable and store a slot id not a slot itself
related refactor

Why / Balance

code duplication bad
interaction with cabinets was previously very annoying now its consistent
im making thing which will benefit from it soon

Technical details
LockSystem:

added IsLocked helper similar to IsOpened

OpenableSystem:

made the verbs alt verbs so you can alt click to open/close things, if they arent drinks
this means you can now just alt click twice to close and lock a filled cabinet instead of fumbling with right click menu
added OpenOnActivate bool to do what ItemCabinet used to do itself
clicking/using openables will now let you close it instead of having to use verb
this means you can click instead of alt click an empty cabinet to close it
added OpenableOpenAttemptEvent that i will use later

ItemCabinetSystem:

rewrote so now it just handles controlling the slot's locking and visuals.
IT NOW USES GENERIC VISUALIZER INSTEAD OF MODIFYING SPRITECOMPONENT

Media
theyre all still real

damage visuals work

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

ItemCabinet now has the slot defined in ItemSlots, default slot name is "ItemCabinet" and can be changed with the slot field
all cabinets now use Openable instead of duplicating behaviour on ItemCabinet
instead of ItemCabinet having open/close states and setting them, it now uses generic visualiser. inherit either BaseItemCabinet or BaseItemCabinetGlass depending on if they use open/closed states or glass-up/glass states. of course you can change the visualizer as well
ItemCabinetVisualLayers renamed to ItemCabinetVisuals and is now has just ContainsItem and Layer. use OpenableVisuals instead of door.

Changelog
no cl no fun

I tried to resolve conflicts in glass_box.yml but I have given up so that still needs doing.

does ItemCabinet even need to exist anymore, can they not just use an ItemSlot and have ItemSlotSystem correctly account for openable/lockable

doing that would make it less flexible since you can then only have 1 thing, you cant for example have a cabinet that also has a powercell which can always be swapped open or closed

merg

multiple entities in item slots

would this not just be storage


cformat.rs: fix remaining test_format.test_common_format failures

With one exception, which I'll comment on inline. Otherwise, commits are atomic.

Thanks for the patient review, @DimitrisJim!


Preprocessing h36m

Hi!
I think there are two minor fixes necessary for the preprocessing of h36m.
1.) h5py module is missing in requirements.txt
2.) File path in step 8 is missing "processed" at the end in order to be consistent.
Cheers,
Urs

Thanks a lot!


Configure codecov

          2 Errors

:no_entry_sign:
Please provide a summary in the Pull Request description

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md

  Generated by :no_entry_sign: danger


During flatten, expected instance of PDFDict or PDFStream, but got instance of undefined

Hello!
I'm encountering the following crash when trying to flatten a W4 from the IRS' website.
/home/ubuntu/src/x/node_modules/pdf-lib/cjs/core/errors.js:38
this = _super.call(this, msg) || this;
^
Error: Expected instance of PDFDict or PDFStream, but got instance of undefined
at new UnexpectedObjectTypeError (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/core/errors.js:38:24)
at PDFContext.lookup (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/core/PDFContext.js:93:15)
at PDFDict.lookup (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/core/objects/PDFDict.js:64:48)
at PDFWidgetAnnotation.PDFAnnotation.getAppearances (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/core/annotation/PDFAnnotation.js:89:20)
at PDFTextField.needsAppearancesUpdate (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/api/form/PDFTextField.js:670:48)
at PDFForm.updateFieldAppearances (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/api/form/PDFForm.js:534:23)
at PDFForm.flatten (/home/ubuntu/src/x/node_modules/pdf-lib/cjs/api/form/PDFForm.js:450:18)
at file:///home/ubuntu/src/x/fillOut.js:293:6

Using v1.16.0, any suggestions to workaround this issue?
My code adds a bunch of text (it doesn't use the Form API, just writes at given coordinates), and then flatten the PDF using PDFForm.flatten().

The crash happens here:
const AP = this.ensureAP();
const N = AP.get(PDFName.of('N'));

Looks like one of the widget's AP dictionary doesn't have a normal appearance (N). According to PDF1.7 spec, that dictionary however is required. I wonder what's going on here.

From PDF1.7, appears that the AP dictionary itself is optional, so maybe the widget annotations corresponding to the PDF fields don't have an appearance?

The following is a workaround that works for me, but it probably leaves in the widget annotations which may result in odd behavior from some PDF viewers. I tested Preview on macOS Big Sur and it looks good so far.
function workaroundFlatten(form) {
  const fields = form.getFields();
  for (let i = 0, lenFields = fields.length; i < lenFields; i++) {
    const field = fields[i];
    form.acroForm.removeField(field.acroField);
    form.doc.context.delete(field.ref);
  }
}

It also worked for me to "touch" all field via setting an empty text as value inside:
const pdfDoc = await PDFDocument.load(pdfDocBytes);
const form = pdfDoc.getForm();
const fields = form.getFields();

fields.forEach(field => form.getTextField(field.getName()).setText(''));

// Form flatten works now as expected.
form.flatten();

Please fill out the new bug report form if you're still having trouble with this.

It also worked for me to "touch" all field via setting an empty text as value inside:
const pdfDoc = await PDFDocument.load(pdfDocBytes);
const form = pdfDoc.getForm();
const fields = form.getFields();

fields.forEach(field => form.getTextField(field.getName()).setText(''));

// Form flatten works now as expected.
form.flatten();

This will work iff field is instance of PDFTextField!!
updated code:
fields.forEach((field) => {
      if (field instanceof PDFTextField) {
        pdfForm.getTextField(field.getName()).setText('');
      }
});

It also worked for me to "touch" all field via setting an empty text as value inside:
const pdfDoc = await PDFDocument.load(pdfDocBytes);
const form = pdfDoc.getForm();
const fields = form.getFields();

fields.forEach(field => form.getTextField(field.getName()).setText(''));

// Form flatten works now as expected.
form.flatten();

This will work iff field is instance of PDFTextField!!
updated code:
fields.forEach((field) => {
      if (field instanceof PDFTextField) {
        pdfForm.getTextField(field.getName()).setText('');
      }
});

This did it for me, thx. Tt would be cool if the package itself could just insert empty strings on save to avoid this issue. Although for the eyes of some, I understand why that would not be desirable.


Strip extra whitespace from header so script can use resulting columns

Strip extra whitespace from header so script can use resulting columns using map
ie; if 'stop_lat' is '-41.03933487162742   ' in stops.txt script won't find it in list:
File "../gtfs2geojson.py", line 32, in gtfs_stops     lat_col = header.index('stop_lat') ValueError: 'stop_lat' is not in list
However '-41.03933487162742' works :)

Heya, thanks for the PR.
However, the file you've submitted uses 2 spaces for indentation rather than hard tab characters like the rest of the script.
As a result, the code you uploaded won't actually work.
I suspect that this would be a good fit for one of my other projects, shittypack.  It deals with badly formed and bloated GTFS feeds to make them reasonable.  I'm pretty sure that this particular error is totally against the spec, and would fail in feedvalidator (from Google's GTFS tools repository).

Hey micolous,
Thankyou for the comprehensive reply, really nice work on the tools :)  - wish I'd found https://github.com/micolous/shittypack earlier!


fix error when AdvertisingProvider is undefined

When the AdvertisingProvider config is undefined, the teardown function should not execute.

Pull Request Test Coverage Report for Build 71

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 94.382%

Totals

Change from base Build 68:
0.06%

Covered Lines:
190

Relevant Lines:
196

💛  - Coveralls


Build failure on rustc 1.7.0 (Ubuntu 16.04 on Windows)

Hope this is the correct way to report this kind of issue?
michaeld@Ayumi:~/amalie$ cargo build
   Compiling gcc v0.3.40
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359:17: 359:41 error: the trait `core::cmp::PartialEq<std::ffi::os_str::OsStr>` is not implemented for the type `std::ffi::os_str::OsString` [E0277]
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359                 var == OsStr::new("LIB")
                                                                                ^~~~~~~~~~~~~~~~~~~~~~~~
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359:17: 359:41 help: run `rustc --explain E0277` to see a detailed explanation
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359:17: 359:41 help: the following implementations were found:
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359:17: 359:41 help:   <std::ffi::os_str::OsString as core::cmp::PartialEq>
/home/michaeld/.cargo/registry/src/github.com-88ac128001ac3a9a/gcc-0.3.40/src/lib.rs:359:17: 359:41 help:   <std::ffi::os_str::OsString as core::cmp::PartialEq<str>>
error: aborting due to previous error
Could not compile `gcc`.

To learn more, run the command again with --verbose.

Oh this was fixed on master actually, but I just haven't made a release yet. I will do so now!


TestWapWorkflow cleanup

remove unsued vars
remove commented code
fix comments

thanks @rdsr for the cleanup.

Thanks! Always great to have commits that keep tests maintained!


Composing schemas

Thank you so much for this library!
I'm not sure if I just missed this, or if it's a bad idea, or if there's a better way to achieve it - but does Norm include an "and" for specs/schemas, corresponding to the "or" provided by one_of?
For example, say I have two schemas like:
quick_checks = schema(%{
  id: spec(is_integer()),
  name: spec(is_string())
})

expensive_checks = schema(%{
  id: spec(some_complex_calculation()),
  age: spec(another_complex_calculation())
})

Perhaps most of the time I only want to do the quick_checks, but on some occasions I want to do ALL the checks. Is there (/should there be) a way to compose the two schemas so that I end up with something like the following?
all_checks = schema(%{
  id: spec(is_integer() and some_complex_calculation()),
  name: spec(is_string()),
  age: spec(another_complex_calculation())
})

I guess if you're using conform!/2 it doesn't really matter, because you can just do
data
|> conform!(quick_checks)
|> conform!(expensive_checks)

and you'll get an exception if something goes wrong.
But with conform/2 it's a bit trickier if you want to catch all the validation errors in one output. I wrote a helper function called conform_all, but it doesn't seem like a very elegant approach.
def conform_all(input, specs_and_schemas) do
  Enum.reduce(specs_and_schemas, {:ok, input}, fn spec_or_schema, previous_result ->
    case previous_result do
      {:ok, input} ->
        case conform(input, spec_or_schema) do
          {:ok, input} -> {:ok, input}
          {:error, new_errors} -> {:error, new_errors}
        end

      {:error, existing_errors} ->
        case conform(input, spec_or_schema) do
          {:ok, _input} -> {:error, existing_errors}
          {:error, new_errors} -> {:error, existing_errors ++ new_errors}
        end
    end
  end)
end

Apologies if I'm missed something obvious, and thanks so much again for Norm!

There are a couple of ways around this in Norm's current form. The most straightforward is to compose the maps before passing them to schema. Something like:
quick_checks = %{
  id: spec(is_integer()),
  name: spec(is_string())
}

expensive_checks = %{
  id: spec(some_complex_calculation()),
  age: spec(another_complex_calculation())
}

checks = schema(Map.merge(quick_checks, expensive_checks))

This obviously only works for schemas and is still pretty limited. We're working on a more general way to compose arbitrary specs. This might be something like an all_of function.

Thanks Chris, I hadn't thought of Map.merge - that's helpful.
If you do go down this road, I think all_of seems like exactly the right name for such a function - that's actually what I looked for in the docs, once I had seen one_of and coll_of.
Thanks again!

Speaking of all_of and one_of, perhaps worth deprecating one_of in favour of any_of? The rationale is twofold:

it would match Enum.any?/2 and Enum.all?/2
"one of" could be interpreted as "exactly one of". To be fair whenever I used one_of, the predicates were mutually exclusive, but just mentioning this anyway.


Allow fractional dashed line patterns

Follow up for https://github.com/tangrams/tangram/pull/404.

This is being used in walkabout-style now: https://github.com/tangrams/walkabout-style/blob/gh-pages/walkabout-style.yaml#L875-L890 so these styles are currently not rendering in tangram-es :\


Create sickrage.py

I use sickrage rather than sonarr and this is what i use.

to further explain. Sickrage uses sickbeard's api. It has the same type of argparse that plexpy/tautulli does. Check the payload to clean up your url :)


Auto npm publish

So here is auto NPM publish workflow. Already tested on https://github.com/sbcgua/mockup-compiler-js

requires NPM_TOKEN secret ( @larshp could you pls add ? )
triggers on new tags starting with v
minor improvements in autotagging

split into 2 steps for clarity
better curl error handling (appeared that status 4** does not return error to console, so additional analysis needed of -f flag. however -f flag does not the body with error message then ... well ... now both work)

@larshp In fact I'd make one more change. In this or in a separate PR. Issue: currently autotag is running on change of package.json. which makes sense. But ! What if the package.json changed together with other files and maybe test does not pass ! It should not be a new version until mistakes are fixed ! So in fact autotagging should trigger only after successful test pass. Thus in main workflow (like in mockup-compiler above). This is actually important imho. The question is just if to make this change in this PR or in another.

hmm

why ? :)


Create a VoronoiDiagram struct that wraps the DCEL for Voronoi specific behaviour?

I think it would be nice to create a VoronoiDiagram struct which provides a place to put Voronoi-specific features (for example: make_polys) while keeping DCEL generic. We could also implement logic for linking cells to the points list and excluding the border cell here too.
Idea for API:
(note: I used the impl syntax for brevity, actual implementation would use concrete types so it works on stable Rust)
struct VoronoiDiagram {
    /// DCEL is public allowing low-level access
    pub dcel: DCEL
}

impl VoronoiDiagram {
    /// Returns an iterator of the cells in the diagram
    pub fn cells(&self) -> impl Iterator<Item=VoronoiCell>;

    /// Returns an iterator over line segments in the diagram
    pub fn segments(&self) -> impl Iterator<Item=(Point, Point)>;

    /// Returns an iterator over Voronoi points in the diagram (excluding border points?)
    pub fn points(&self) -> impl Iterator<Item=Point>;
}

struct VoronoiCell {
    /// No public attributes
}

impl VoronoiCell {
    /// Returns the index of the point in the original points array that this cell was generated from
    pub fn index(&self) -> usize;

    /// Computes the centroid of the cell
    pub fn centroid(&self) -> Point;

    /// Returns an iterator of the points that border the cell
    pub fn points(&self) -> impl Iterator<Item=Point>;
}

The voronoi function would be changed to return a VoronoiDiagram instead of the DCEL.
Let me know if you have any thoughts.

This looks reasonable to me. I expect that a lot of applications would want to use the underlying DCEL to get adjacency information, but this seems very useful for simple applications and plotting.
I think there is probably an efficient way to compute the Voronoi-cell-to-original-point mapping during construction of the DCEL, but I haven't thought it through.
Somewhat related: probably the boundary cropping shouldn't be happening in the voronoi method, and should instead be a method on VoronoiDiagram. This would mean that the DCEL needs to be extended to represent half-infinite edges. The main difficulty I see with this is that then you can't just remove points outside the boundaries before computing the Voronoi diagram, so the boundary cropping algorithm gets more complicated.


Doc(eos_designs): Two small changes to documentation for eos_designs role

Change Summary
Fix two typos in the eos_designs documentation:

interface_descriptions variable is spelled incorrectly in node_types.md
endpoint_ports is required to be defined in connected_endpoints.md

Related Issue(s)
Component(s) name
arista.avd.eos_designs
Checklist
Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

Does the bot not like that I included an optional scope?

Goodness. Sorry about the title edit spam on the Title

So interestingly, while the switch and switch port are conceptually required, I couldn't find any syntactic requirement for them. The lack of the endpoint_ports caused a build failure, but you can define a batch of servers and not allocate them to a switch right away. I'll make your requested changes and fix both commit messages now that I better understand the title semantics.
Where I tripped on this is because I'm trying to use AVD to build an L2 EVPN fabric for an Internet Exchange Point. The endpoints are Internet edge peering and outside our autonomous system, so I have no idea what port on their device they're using. So ideally I think we should make the endpoint ports optional, but I'm trying to go one step at a time here.

I took another look. If you don't have switches and switch_ports documentation is generated correctly. If switch_ports is missing you will face an error, so it is validated:
TASK [arista.avd.eos_designs : Generate device configuration in structured format] ***********************************************************************************************
Friday 10 December 2021  17:20:03 +0100 (0:00:00.768)       0:00:07.330 *******
Friday 10 December 2021  17:20:03 +0100 (0:00:00.768)       0:00:07.329 *******
ok: [POD1_SPINE01 -> localhost]
ok: [POD1_SPINE02 -> localhost]
fatal: [POD1_LEAF01A -> localhost]: FAILED! => {"msg": "'dict object' has no attribute 'switch_ports'"}

For your specific case, I might suggest they use as endpoint_ports whatever value helps them to identify the connection on the other peer, as this value is only used in the description field:
my_cool_server:
  cool_1:
    rack: RACK_TEST
    adapters:
      - speed: "10g auto"
        endpoint_ports: [ blah ]
        switch_ports: [ eth7 ]
        switches: [ POD1_LEAF01A ]
        mode: access
        vlans: "3"

Generated documentation:
#### L2

| Interface | Description | Mode | VLANs | Native VLAN | Trunk Group | Channel-Group |
| --------- | ----------- | ---- | ----- | ----------- | ----------- | ------------- |
| eth7 |  cool_1_blah | access | 3 | - | - | - |

Generated config:
interface eth7
   description cool_1_blah
   no shutdown
   speed 10g auto
   switchport
   switchport access vlan 3
   switchport mode access
!

Good point. Once the switch is assigned and the endpoint gets pulling into the switches state, the lack of the port assignment is a problem.
I've force pushed over my previous branch with your requested changes.

To add up to your discussion. I just found that server_ports or endpoint_ports is required. Because if we don't define endpoint_ports then the default value is taken from server_ports. But this looks good to me to set endpoint_ports to required.

Also, change your PR title to Fix documentation rather than saying two or three suggestions.
I found one more typo in the same connected-endpoints.

I don't think we need:
< port_profile_2 >:
  mode: < access | dot1q-tunnel | trunk >
  vlans: < vlans as string >

This is a port_profile_2 which should be defined under port_profiles knob but accidentally sitting under adapters settings.

I've changed the pull request title.
port_profile_2 is outside the scope of what I'm working on.


Preload action runtimes

First time you execute an action taks time and sometimes it times out.
The problem is related to pulling the images for action.
Execute a kind load docker-image of openwhisk/action-nodejs-v14:nightly to preload them in the docker image to speed up the deployment of the first action.

Resolved in a different way in this issue


The hiddenGroupRef is erroneously supplying a carriage return.

@rthomas320, will you provide a screenshot of what the problem looks like in the issue description?  Thanks.


Possible Memory leak

Hi,
When ever the script runs my cpu usage hovers around less than 10% but my memory runs at 90%-100%. And then conversion takes a very long time.

This is probably more related to FFMPEG than anything else, the script doesn't use much memory, but copying/remuxing streams will use lots of memory with minimal CPU usage

Yeah it was an ffmpeg issue. I downgraded a few versions and it works great now.


Supporting library 27.1.0

Hello!
I want to use your lib with latest version of picasso (it requires 27.1.0 lib's version).
I have the warning in my build.gradle:

Please update lib :)

Sorry for my inattention, there is conflict with another lib.
It's all good with CircleImageView.
Thanks for convenient tool for android!


ECMAScript 6 migration aids.

Hey @Romick2005 ! Thanks for the PR. Unfortunately, some tests are still failing. Could you look into it again? :relaxed:

Thank you for the contributions.
Because these codes are templates, so we didn't apply eslint on it, we're considering about the refactoring, so I'll close this pull request for now.


dotnet interactive notebook cannot create sql connection

Type: Bug
I run this code block
#r "nuget:Microsoft.DotNet.Interactive.SqlServer,*-*"
which returns back this message
Installed Packages
Microsoft.DotNet.Interactive.SqlServer, 1.0.0-beta.22606.2
Loading extensions from `/Users/Chris.Lumnah/.nuget/packages/microsoft.dotnet.interactive.sqlserver/1.0.0-beta.22606.2/interactive-extensions/dotnet/Microsoft.DotNet.Interactive.SqlServer.dll`
Query Microsoft SQL Server databases.
This extension adds support for connecting to Microsoft SQL Server databases using the #!connect mssql magic command. For more information, run a cell using the #!sql magic command.

This appears to work ok, so the next step would be to create a connection via this line
#!connect mssql --kernel-name TPCCDemo "Persist Security Info=False; Integrated Security=true; Initial Catalog=tpcc_2000gb; Server=sql1.domain.com;"

This returns back an error
Error: System.ComponentModel.Win32Exception (2): An error occurred trying to start process 'MicrosoftSqlToolsServiceLayer' with working directory '/Users/Chris.Lumnah/Downloads/Rubrik Demos/Object Level Recovery'. No such file or directory
   at System.Diagnostics.Process.ForkAndExecProcess(ProcessStartInfo startInfo, String resolvedFilename, String[] argv, String[] envp, String cwd, Boolean setCredentials, UInt32 userId, UInt32 groupId, UInt32[] groups, Int32& stdinFd, Int32& stdoutFd, Int32& stderrFd, Boolean usesTerminal, Boolean throwOnNoExec)
   at System.Diagnostics.Process.StartCore(ProcessStartInfo startInfo)
   at Microsoft.DotNet.Interactive.SqlServer.ToolsServiceClient.StartSqlToolsService()
   at Microsoft.DotNet.Interactive.SqlServer.ToolsServiceClient.Initialize()
   at Microsoft.DotNet.Interactive.SqlServer.ToolsServiceKernel..ctor(String name, ToolsServiceClient client, String languageName)
   at Microsoft.DotNet.Interactive.SqlServer.MsSqlKernel..ctor(String name, String connectionString, ToolsServiceClient client)
   at Microsoft.DotNet.Interactive.SqlServer.MsSqlKernelConnector.CreateKernelAsync(String kernelName)
   at Microsoft.DotNet.Interactive.SqlServer.ConnectMsSqlCommand.ConnectKernelAsync(KernelInvocationContext context, InvocationContext commandLineContext)
   at Microsoft.DotNet.Interactive.CompositeKernel.<>c__DisplayClass26_0.<<AddKernelConnector>b__0>d.MoveNext() in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\CompositeKernel.cs:line 329
--- End of stack trace from previous location ---
   at System.CommandLine.NamingConventionBinder.CommandHandler.GetExitCodeAsync(Object returnValue, InvocationContext context)
   at System.CommandLine.NamingConventionBinder.ModelBindingCommandHandler.InvokeAsync(InvocationContext context)
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass15_0.<<AddMiddleware>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass12_0.<<UseHelp>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass19_0.<<UseTypoCorrections>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Invocation.InvocationPipeline.<InvokeAsync>g__FullInvocationChainAsync|2_0(InvocationContext context)
   at System.CommandLine.Parsing.ParseResultExtensions.InvokeAsync(ParseResult parseResult, IConsole console)
   at Microsoft.DotNet.Interactive.Commands.DirectiveCommand.InvokeAsync(KernelInvocationContext context) in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\Commands\DirectiveCommand.cs:line 34
   at Microsoft.DotNet.Interactive.Kernel.HandleAsync(KernelCommand command, KernelInvocationContext context) in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\Kernel.cs:line 325
   at Microsoft.DotNet.Interactive.KernelCommandPipeline.<BuildPipeline>b__6_0(KernelCommand command, KernelInvocationContext context, KernelPipelineContinuation _) in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\KernelCommandPipeline.cs:line 60
   at Microsoft.DotNet.Interactive.KernelCommandPipeline.<>c__DisplayClass6_1.<<BuildPipeline>b__3>d.MoveNext() in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\KernelCommandPipeline.cs:line 75
--- End of stack trace from previous location ---
   at Microsoft.DotNet.Interactive.CompositeKernel.LoadExtensions(KernelCommand command, KernelInvocationContext context, KernelPipelineContinuation next) in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\CompositeKernel.cs:line 130
   at Microsoft.DotNet.Interactive.KernelCommandPipeline.<>c__DisplayClass6_0.<<BuildPipeline>g__Combine|2>d.MoveNext() in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\KernelCommandPipeline.cs:line 73
--- End of stack trace from previous location ---
   at Microsoft.DotNet.Interactive.KernelCommandPipeline.SendAsync(KernelCommand command, KernelInvocationContext context) in D:\a\_work\1\s\src\Microsoft.DotNet.Interactive\KernelCommandPipeline.cs:line 41

VS Code version: Code 1.74.1 (Universal) (1ad8d514439d5077d2b0b7ee64d2ce82a9308e5a, 2022-12-14T10:33:40.793Z)
OS version: Darwin x64 22.1.0
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-1038NG7 CPU @ 2.00GHz (8 x 2000)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
4, 4, 5

Memory (System)
16.00GB (4.31GB free)

Process Argv
--crash-reporter-id 0b19a0f4-14a7-40b9-8900-1b55f5d0febc

Screen Reader
no

VM
0%

Extensions (15)

Extension
Author (truncated)
Version

vscode-pull-request-github
Git
0.56.0

vscode-graphql
Gra
0.8.4

vscode-graphql-syntax
Gra
1.0.5

vscode-edit-csv
jan
0.7.2

dotnet-interactive-vscode
ms-
1.0.3611020

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.12

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

remote-containers
ms-
0.266.1

remote-ssh
ms-
0.94.0

remote-ssh-edit
ms-
0.84.0

powershell
ms-
2022.12.1

remote-explorer
ms-
0.0.3

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931:30280409
vshan820:30294714
vstes263cf:30335440
vscoreces:30445986
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848:30577860
nodejswelcome1:30587005
3biah626:30602489
gswce1:30612156
iaj6b796:30613358
89544117:30613380
fim-prod:30623723

Please file issue against the .NET extension.

@clumnah did you ever get this resolved?
I'm trying out the new Polyglot Notebooks and want to be connecting to a database. This has me dead in my tracks before I can even get started.
Thank you

I have not tried in the last couple of weeks, but the last time I did try, it was still broken.

I had .net SDK version 7.0.1x installed on my machine.
When I upgraded to 7.0.202 it started working.
https://dotnet.microsoft.com/en-us/download

Oh good to know. I’ll try that tomorrow. Thank youOn Mar 15, 2023, at 9:39 PM, Rabbi Moshe Plotkin @.***> wrote:
I had .net SDK version 7.0.1x installed on my machine.
When I upgraded to 7.0.202 it started working.
https://dotnet.microsoft.com/en-us/download
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

Interesting find. I upgraded and now this issue is resolved. Thanks for the tip.

On Mar 15, 2023, at 9:39 PM, Rabbi Moshe Plotkin @.***> wrote:
I had .net SDK version 7.0.1x installed on my machine.
When I upgraded to 7.0.202 it started working.
https://dotnet.microsoft.com/en-us/download
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/vscode/issues/169449#issuecomment-1471148035, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFKMBSAHXFI53G73Y45BBADW4JVMNANCNFSM6AAAAAATBSH2OU.
You are receiving this because you were mentioned.


How do I append a texture to the GLTF file?

As far as I can see, the library supports a wide variety of converting resources, but I wonder if I can also add new resources (in this case a texture) to an already existing GLTF file.
So far I thought of two ways, both won't work for now:
Adding a resource, when exporting nothing is added.
  resource = Base64Resource(image_bytes, mime_type='image/jpeg')
  gltf.resources.append(resource)

Adding an image, on export it tries to find a resource with the same uri and fails.
  image = Image(extensions=None, extras=None, name="newTexture", uri=base64_uri)
  gltf.model.images.append(image)

There must be a simple way since there is a complete model, could someone help me with this one?

Ok.. doing both solved the issue..


Tag Version (this repo) fails due to branch protection rules (OSOE-912)

The last step named Commit and Push Merge Target Changes fails, because the dev branch is protected.
remote: error: GH006: Protected branch update failed for refs/heads/dev.
remote:
remote: - Changes must be made through the merge queue
remote:
remote: - Required status check "validate-gha-refs" is expected.
To https://github.com/Lombiq/GitHub-Actions
 ! [remote rejected] dev -> dev (protected branch hook declined)
error: failed to push some refs to 'https://github.com/Lombiq/GitHub-Actions'
Error: Invalid status code: 1

Make the last two steps in the tag-version workflow skippable by configuration.
Add instructions to the readme on how to resolve it:

As an elevated user (with push access to dev):

Start a merge from the release branch to dev, but don't create a commit.
Revert all the file changes.
Merge the empty commit.

Otherwise:

Create a commit on the release branch that reverts all the file the changes (compared to dev).
Create a PR for the release branch and merge it.

Jira issue

We can change the branch protection rule or something to allow just these. I think I have a solution, will follow up.


M_PI is deprecated in Swift 3.1, switch to Double.pi

Pretty much the title, should be a quick fix.

ok


NOT WORKING

If I try to send 1 euro or less than one euro the system does not check the accreditation in fact is constantly pending .... also there is a problem in the generation of qr code ... in fact on the NAME STORE if a user does a space generates a link with the space and therefore does not read the qr code.

Thank you for letting me know. I will look at it as soon as possible.

It should be fixed now. It was consulting an obsolete service.
I couldn't replicate the NAME STORE issue. I added a space (or more) before the name, even only spaces as the name but it seemed to work. If you still have it please provide me with an exact string that causes it.
Let me know if the above issues were fixed (for new issues please add a new issue on github).
To use on testnet (assuming you use the codebase) go to BitcoinUtils.java and change isMainnet() to return false.

I will update the app in play store some time soon.

very thanks for your fix i try and send you confirm... for the problem the
name store... the problem is if i set for exsample NAMESTORE then generate
a qr code correct because not space but if i set a name whit space NAME
STORE confirm all but when another user scan qr code this code present a
link whit space generating error from another wallet... you understand?
sorry for my bad english
thanks
Il gio 28 giu 2018, 19:10 Kostas Karasavvas<EMAIL_ADDRESS>ha
scritto:

I will update the app in play store some time soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/karask/bitcoinpos/issues/2#issuecomment-401107441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeiATij6bk10nfttqjsIuzKhoDPuThRAks5uBQ4dgaJpZM4U6CDW
.

I have encountered other problems:

the payment process works and goes immediately to PENDING at the first
confirmation ... how can you decrease the waiting time and directly show
the payment CONFIRMED already from the first confirmation? (I know it's not
safe but the customer can not wait 15 20 minutes to wait for the credit)

when a transition is confirmed in the HISTORY returns incorrect
characters there is also no way to eliminate the transitions made

this problem present also in file BitcoinPos-X.csv not correct show
caracter exsample date, time and amount.

when the name of a store has the space then it generates the payment
request address also with the space generating errors

bitcoin:1HVfvnszfN4EzKHyRzGVnktuUfJjG9JpqX?amount=0.01705195&label=Merchant

name

attach you all pictures of error.
For the rest is perfect.
Il giorno gio 28 giu 2018 alle ore 20:53 Staff<EMAIL_ADDRESS>ha scritto:

very thanks for your fix i try and send you confirm... for the problem the
name store... the problem is if i set for exsample NAMESTORE then generate
a qr code correct because not space but if i set a name whit space NAME
STORE confirm all but when another user scan qr code this code present a
link whit space generating error from another wallet... you understand?
sorry for my bad english
thanks
Il gio 28 giu 2018, 19:10 Kostas Karasavvas<EMAIL_ADDRESS>ha
scritto:

I will update the app in play store some time soon.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/karask/bitcoinpos/issues/2#issuecomment-401107441,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeiATij6bk10nfttqjsIuzKhoDPuThRAks5uBQ4dgaJpZM4U6CDW
.

--
AVVISO IMPORTANTE:
Le informazioni contenute in questa mail e negli allegati sono riservate
al destinatario indicato e possono essere confidenziali e/o tutelate dal
segreto professionale; qualsiasi uso, copia, distribuzione, riproduzione o
divulgazione, anche parziale, di quanto in essi contenuto è severamente
vietata ai sensi dell’art. 616 c.p., nonché a norma della legge 675/96 e
del D. lgs. 196/2003 in materia di tutela dei dati personali. Se questo
documento è pervenuto per errore a persona diversa dal suo destinatario o
da un suo incaricato, si prega di informare immediatamente il mittente. Il
contenuto della presente mail non va letto ma distrutto.

I couldn't find any pictures (attachments) with the errors.
Where do you get the issue with the space on the bip21 string (bitcoin:1HV...). I tried reading the qr code from mycelium and it reads the label containing "Merchant name" fine.
I will look into the CSV report file and test it again soon.
What characters are shown incorrectly on History? Can you give an example string? Any UTF-8 characters should be printed fine.
Yes, I know what you mean with the immediate payment confirmation. Right now it works as follows:
The transaction status has 3 states, Pending (red color), Ongoing (yellow color) and Confirmed (green color).
When a payment request is made it is "Pending" and it checks for updates from the blockchain. If it sees the transaction it changes the state to "Ongoing" and most of the times this would be safe from double spends (given the amounts are small and not many would bother). The app keeps on checking periodically and if it sees 1 confirmation for the transactions it updates the history of that transaction to "Confirmed".
Note that while "Pending" there is a "Cancel" button. When "Ongoing" the button changes to "OK". You could just press OK and consider the transaction done (as you suggested)... and later on you can check the History tab to see that it was eventually confirmed.
I do not want to explicitly say "done" or "confirmed" when the tx is visible to the network since in the rare occasion that something goes wrong (double spend) people could accuse me of not warning them!!
You are right about removing transactions from history.. I didn't have time to look into it yet.

The new commit fixes most of the issues.
Deleting transactions is not available yet.


Add support for GraphQL

[x] GraphQL
[x] Import GraphQL schema
[x] Populate GraphQL schema from JSON response
[x] GraphQL editor with completion based on schema
[x] parentheses background highlighting
[x] I would try the highlighting colors from GitHub
[x] Expanders Primary+B should spawn the entity to the right for now. We will do the expanders afterwards.
[x] Completion even when there is no character (right now we need to type one)
[x] Add the snippet result to a playground variable
[x] Make the context Dropdown accept self when the object is a graphql result
[x] Variables rename
[x] Show errors inline
[ ] Primary+Enter should spawn another graphql snippet with the same context
[ ] When creating a graphql snippet:

[ ] When the context is automatically chosen, the focus should go to the editor
[ ] When the context is not automatically populated, the focus should go to the context dropdown

[ ] When moving up/down arrows, the cursor should move between editors, context dropdown, variable editor and in/out of snippet

More completion locations:

https://user-images.githubusercontent.com/249595/202982449-aea8064d-8f1b-4003-84d3-79bf9faf8df5.mp4


[models] leak in GenMesh*

looks like there could be some leaks in or around the GenMesh functions, I can take a look but it will be in a few days...

closing this as in the end it turned out to be an issue between keyboard and seat


Does LWJGL move any data off heap

Question
If you use a non direct buffer does it move it off heap I have a program showing 500 so mb and gc spikes

No: you can't use non-direct buffers in any LWJGL functions. LWJGL doesn't move any of your byte buffers.

No: you can't use non-direct buffers in any LWJGL functions. LWJGL doesn't move any of your byte buffers.

Ok


Can't access the URL links of trained model

The model weights you've published appear to be hosted in a private repository.

Sorry, I didn't notice it's private. I have made it public and you can try to download the model weights now. Thanks for reminding me.

Thank you very much.


Run Vault in unprivileged SCC group

We are unable start Vault in unprivileged group.
If we  start Vault with disable_mlock and SKIP_SETCAP=true, we will face to the necessarity of  running Vault under SCC with "allowedCapabilities: - IPC_LOCK" becaue it capability is included into securitycontext of statefulset.
I found that it is hardcoded into code of opearotor and it will be added in any case:
https://github.com/banzaicloud/bank-vaults/blob/3908ad73b09670131f921cc580b8fe743299ecf4/operator/pkg/controller/vault/vault_controller.go#L1218-L1221
Could you please add the opportunity to create vault statesfulset without IPC_LOCK in cases if we disable it?

I totally support this proposal. there is no reason to start Vault with IPC_LOCK if "disable_mlock: true". Vault recommend to not disable mlock unless swap is disabled:
https://www.vaultproject.io/docs/configuration/
In Openshift clusters swap is disabled by default, so we should be able to choose this option.


Added getServiceInstanceInfo API

Also, split TranslationConfigData into two, one for creating a new configuration, and another for read-only access including some system generated fields (updatedBy, updatedAt).

LGTM!


Lara: button "icon only" do not work with loading state

Repository URL
https://tailwind.primevue.org/button/#icons
https://tailwind.primevue.org/button/#loading
Icon only button, no loader
<Button  icon="pi pi-trash" size="small" :loading="true" />

Loading with label:

Loading without label:

As of v4.1.1, the new Tailwind styles have been rewritten using the preprocessor approach with @apply instead of pass-through. As a result this issue is not relevant any more.
New Website is live.


Inconsistent source views

Compare move.rs shown in docs.rs source view and rustdoc source view.

the rustdoc search bar is only shown in the rustdoc view
the rustdoc search bar is pushed away from a nonexistent sidebar, and there's way too much margin between it and the source listing
line numbers are only shown in the rustdoc view
the file tree is only shown in the docs.rs view
the docs.rs view uses the Source tab, while the rustdoc view uses the Documentation tab
the syntax coloring schemes differ

Thanks for reporting. Some people suggested replacing [src] links in documentation to docs.rs source view. But IMO rustdoc source view is much better.
docs.rs source view is actually exist for crates that don't have any documentation. It is also useful to view Cargo.toml and see all features of a crate.
I am not sure how can we solve this issue.

Add line numbers and line anchors to the docs.rs view? Those are the big missing features.

Closing in favor of #31


Introduce oneConfigurationSettings

this is initial commit for issue(#58)
configuration settings panel is for serving GUI environment to build ONE .cfg file
ONE-vscode-DCO-1.0-Signed-off-by: TaeKyuRyu<EMAIL_ADDRESS>

@OTKRyu , this is the second PR that you closed just after creating it.
Can your share your problem ?

i just forget to write license things so i closed and make another pull request.

i just forget to write license things so i closed and make another pull request.

There are lots of solutions for this

You can add PR/NO MERGE label
edit the file -> add -> commit -> push (will add more commits)
edit the file -> add -> commit with --amend  -> force push (will overrite last commit
after you think is finished, remove PR/NO MERGE label and add PR/Ready for review label


config specific LS

i was using nvim-lspinstall. with it, i used to use following config for LUA

require'lspconfig'.lua.setup{ -- for Lua
  on_attach = on_attach,
  settings = {
    Lua = {
      diagnostics = {
        -- Get the language server to recognize the 'vim', 'use' global
        globals = {'vim', 'use'},
      },
      workspace = {
        -- Make the server aware of Neovim runtime files
        library = vim.api.nvim_get_runtime_file("", true),
      },
      -- Do not send telemetry data containing a randomized but unique identifier
      telemetry = {
        enable = false,
      },
    },
  }
}

how to do this with nvim-lsp-installer?

sorry for wasting your time but yes, i did read the README and :help nvim-lsp-installer both. But i didn't get it worked.
here is my config
local lsp_installer = require("nvim-lsp-installer")

-- Provide settings first!
lsp_installer.settings {
    ui = {
        icons = {
            server_installed = "✓",
            server_pending = "➜",
            server_uninstalled = "✗",
        },
    },

    -- Limit for the maximum amount of servers to be installed at the same time. Once this limit is reached, any further
    -- servers that are requested to be installed will be put in a queue.
    max_concurrent_installers = 4,
}

lsp_installer.on_server_ready(function(server)
    local opts = {}

    -- (optional) Customize the options passed to the server
    -- if server.name == "tsserver" then
    --     opts.root_dir = function() ... end
    -- end

    -- This setup() function is exactly the same as lspconfig's setup function (:help lspconfig-quickstart)
    server:setup(opts)
    vim.cmd [[ do User LspAttachBuffers ]]
end)

local function setup_server(server)
  local lsp_installer_servers = require'nvim-lsp-installer.servers'
  local ok, server_analyzer = lsp_installer_servers.get_server(server)
  if ok then
    if not server_analyzer:is_installed() then
      server_analyzer:install(server)   -- will install in background
      -- lsp_installer.install(server)     -- install window will popup

    end
  end
end

local servers = {
  "sumneko_lua",        -- for Lua
  "rust_analyzer",      -- for Rust
  "pyright",            -- for Python
  "clangd",             -- for C/C++
  "bashls",             -- for Bash
}

for _, server in ipairs(servers) do
  setup_server(server)
end

I have run into a similar issue as @shaeinst. I looked at his config and it seems pretty similar to mine, but my custom settings don't seem to get applied... I am wondering if I can get some eyes on it.
local on_attach = function(client, bufnr)
  local function buf_set_keymap(...) vim.api.nvim_buf_set_keymap(bufnr, ...) end
  local function buf_set_option(...) vim.api.nvim_buf_set_option(bufnr, ...) end

  -- Enable completion triggered by <c-x><c-o>
  buf_set_option('omnifunc', 'v:lua.vim.lsp.omnifunc')

  -- Mappings.
  local opts = { noremap=true, silent=true }

  -- See `:help vim.lsp.*` for documentation on any of the below functions
  buf_set_keymap('n', 'gD', '<cmd>lua vim.lsp.buf.declaration()<CR>', opts)
  buf_set_keymap('n', 'K', '<cmd>lua vim.lsp.buf.hover()<CR>', opts)
  buf_set_keymap('n', 'gi', '<cmd>lua vim.lsp.buf.implementation()<CR>', opts)
  buf_set_keymap('n', 'gd', '<cmd>lua vim.lsp.buf.definition()<CR>', opts)
  buf_set_keymap('n', 'gr', '<cmd>lua vim.lsp.buf.references()<CR>', opts)
  buf_set_keymap('n', '<C-k>', '<cmd>lua vim.lsp.buf.signature_help()<CR>', opts)
  buf_set_keymap('n', '<Leader>wa', '<cmd>lua vim.lsp.buf.add_workspace_folder()<CR>', opts)
  buf_set_keymap('n', '<Leader>wr', '<cmd>lua vim.lsp.buf.remove_workspace_folder()<CR>', opts)
  buf_set_keymap('n', '<Leader>wl', '<cmd>lua print(vim.inspect(vim.lsp.buf.list_workspace_folders()))<CR>', opts)
  buf_set_keymap('n', '<Leader>D', '<cmd>lua vim.lsp.buf.type_definition()<CR>', opts)
  buf_set_keymap('n', '<Leader>rn', '<cmd>lua vim.lsp.buf.rename()<CR>', opts)
  buf_set_keymap('n', '<Leader>ca', '<cmd>lua vim.lsp.buf.code_action()<CR>', opts)
  buf_set_keymap('n', '<Leader>d', '<cmd>lua vim.diagnostic.show_line_diagnostics()<CR>', opts)
  buf_set_keymap('n', '[d', '<cmd>lua vim.diagnostic.goto_prev()<CR>', opts)
  buf_set_keymap('n', ']d', '<cmd>lua vim.diagnostic.goto_next()<CR>', opts)
  buf_set_keymap('n', '<Leader>q', '<cmd>lua vim.diagnostic.set_loclist()<CR>', opts)
  buf_set_keymap('n', '<Leader>f', '<cmd>lua vim.lsp.buf.formatting()<CR>', opts)
end

-- ref: https://github.com/williamboman/nvim-lsp-installer/wiki/Advanced-Configuration#overriding-the-default-lsp-server-options
local enhance_server_opts = {
  ["texlab"] = function(opts)
    opts.settings = {
      texlab = {
        build = {
          executable = "zathura",
          onSave = true,
        },
        chktex = {
          onOpenAndSave = true,
          onEdit = true,
        }
      },
    }
  end,
  ["pyright"] = function(opts)
    opts.settings = {
      python = {
        venvPath = "."
      }
    }
  end,
}

-- LSPINSTALLER
-- Register a handler that will be called for all installed servers.
-- Alternatively, you may also register handlers on specific server instances instead (see example below).
local capabilities = require('cmp_nvim_lsp').update_capabilities(vim.lsp.protocol.make_client_capabilities())
lsp_installer.on_server_ready(function(server)
  local opts = {
    on_attach = on_attach,
  }
  capabilities = capabilities
  -- ref: https://github.com/williamboman/nvim-lsp-installer/wiki/Advanced-Configuration#overriding-the-default-lsp-server-options
  if enhance_server_opts[server.name] then
    enhance_server_opts[server.name](opts)
  end
    -- This setup() function is exactly the same as lspconfig's setup function.
    -- Refer to https://github.com/neovim/nvim-lspconfig/blob/master/doc/server_configurations.md
  server:setup(opts)
end)

I am not sure what else I need to do. Is there some way to check the settings of a server that has been loaded?
I have read over: https://github.com/williamboman/nvim-lsp-installer/wiki/Advanced-Configuration#overriding-the-default-lsp-server-options
https://github.com/neovim/nvim-lspconfig/blob/master/doc/server_configurations.md#texlab
and https://github.com/latex-lsp/texlab/blob/master/docs/options.md
And I must be missing something...
Any help would be greatly appreciated.

checkout my lsp-installer config here
lspconfig
lsp-installer

@saltsucker I can't see anything wrong with your config at first glance. Are you sure the settings you're providing are valid for those servers?

@williamboman I am pretty sure they are:
i.e: texlab.build.executable is equivalent to:
settings = {
  texlab = {
    build = {
      executable = 'something'
    }
  }
}

Right?
Dot notation = nesting of properties right?

@shaeinst Mine seems pretty similar, not sure how yours works but mine doesn't... :/

Nevermind. I think it is working. When I set the onEdit and onOpenAndSave the linting doesn't occur when I save the file. But does when I set them to boolean. The issue I ran into was that it wasn't opening zathura after I compiled it. I instead just set my default PDF viewer to zathura and that works instead.

after the new changes in this plugin I have tried setting the same options for sumneko_lua as op did, but they are not working. since now the setup is demanded to lspconfig, are the options recognised? in my case I cannot solve the error undefined global vim with this config:
local servers = { 'pylsp', 'clangd', 'lemminx', 'sumneko_lua', 'cmake', 'bashls' }

require('nvim-lsp-installer').setup({
	ensure_installed = servers,
	automatic_installation = true,

for _, lsp in pairs(servers) do
	if lsp == 'sumneko_lua' then

		lspconfig[lsp].setup{
			on_attach = on_attach,
			-- capabilities = capabilities,

			settings = {
				Lua = {
					runtime = {
						version = 'LuaJIT',
						path = runtime_path,
					},
					diagnostics = {
						globals = {'vim'},
					},
					workspace = {
						library = vim.api.nvim_get_runtime_file("", true),
					},
				},
			},
		}
	end

which is taken directly from lspconfig readme, but doesn't work. any idea about the role of nvim-lsp-installer in this?
})

Hey @DavidePatria. Exactly, nvim-lsp-installer is no longer explicitly involved in the setting up of servers. Apart from your snippet being syntactically incorrect, I'd probably do something like the following instead. Set up sumneko lua separately because we have custom settings for it, and only set up the remaining servers in the loop (and apply the same default settings to them).
local servers = { 'pylsp', 'clangd', 'lemminx',  'cmake', 'bashls' }

require('nvim-lsp-installer').setup({
  automatic_installation = true,
})

lspconfig.sumneko_lua.setup {
    on_attach = on_attach,
    settings = {
      Lua = {
        runtime = {
          version = 'LuaJIT',
          path = runtime_path,
        },
        diagnostics = {
          globals = {'vim'},
        },
        workspace = {
          library = vim.api.nvim_get_runtime_file("", true),
        },
      },
    },
}

for _, lsp in pairs(servers) do
  lspconfig[lsp].setup {
    on_attach = on_attach,
  }
end

the ifs are ugly, but it is the first way I could do it to only keep one table for the list of servers to use in both lspconfig and nvim-lsp-installer. turns out the error in the config was my fault. sorry


Improvements to Opening Times page

Type

✨ Feature

Modified hours
Current design prioritises regular hours over modified hours. When we have modified opening hours (e.g. for bank holidays, Christmas), we need a better way to inform users what has changed.
This could be as simple as messaging at the top of the page, or could be a full list of changed hours.
Note that it's important to include the non-changed days in a timespan, e.g. here are the hours for Easter weekend, rather than 'here are the two days where hours are different'. The messaging should cover the whole timespan.

Subsumed by #3439


Timeless candle chart

Hi, I need to build a timeless graph... In actual examples, when I put two equals data, one override another.. How I can plot candles considering list order and without override?

Anyone? How can one put hours and minutes in dates?
Seconds?
No example there...

Unix datestamps in place of dates are not working.
How can I put data in chart with time not just date?

I mean, what date format should I put here:
{
"Date": "28-Apr-2019",
"Open": "0.291600",
"High": "0.291700",
"Low": "0.291400",
"Close": "0.291700",
"Volume": "23"
},
To specify hours and minutes?
Thanks

Huh, found the solution.
Located line:
var parseDate = d3.timeParse("%d-%b-%y");
Changed it to:
var parseDate = d3.timeParse("%d-%b-%y %H:%M");
Who needs dates with days only?
Very confusing examples you have here...
Anyways. now dates like "28-Apr-2019 04:30" works.
Hope it helps someone.

From the samples:
Date,Open,High,Low,Close,Volume
9-Jun-14,62.40,63.34,61.79,62.88,37617413
6-Jun-14,63.37,63.48,62.15,62.50,42442096

var parseDate = d3.timeParse("%d-%b-%y");
{
  date: parseDate(d.Date),
  volume: +d.Volume,
  open: +d.Open,
  high: +d.High,
  low: +d.Low,
  close: +d.Close
}

Data with unix timestamp:
{
  date: new Date(d.time * 1000),
  volume: +d.Volume,
  open: +d.Open,
  high: +d.High,
  low: +d.Low,
  close: +d.Close
}


feat: new module recommendationsBySymbol

This PR adds a new module recommendationsBySymbol that retrieves "Similar instruments" to the one's you pass.
How it's used on Yahoo Finance
When you view a specific symbol on Yahoo Finance they will show a widget on the right hand side; "People also watch" - which the user might be interested in.

About the endpoint
https://query2.finance.yahoo.com/v6/finance/recommendationsbysymbol/${symbols}

Accepts multiple symbols, comma separated
No other query parameters as far as I know

The response
The response back from Yahoo looks like this:
{
  "finance": {
    "result": [
      {
        "symbol": "AAPL",
        "recommendedSymbols": [
          { "symbol": "AMZN", "score": 0.292276 },
          { "symbol": "FB", "score": 0.274045 },
          { "symbol": "GOOG", "score": 0.272778 },
          { "symbol": "TSLA", "score": 0.270931 },
          { "symbol": "NFLX", "score": 0.209186 }
        ]
      },
      {
        "symbol": "BMW.DE",
        "recommendedSymbols": [
          { "symbol": "DAI.DE", "score": 0.1927 },
          { "symbol": "VOW.DE", "score": 0.105787 },
          { "symbol": "SIE.DE", "score": 0.102734 },
          { "symbol": "VOW3.DE", "score": 0.098733 },
          { "symbol": "BAS.DE", "score": 0.098715 }
        ]
      }
    ],
    "error": null
  }
}

Changes

Add new module recommendationsBySymbol
Add basic tests for recommendationsBySymbol
Add documentation for the module

Example usage
// Get similar symbols to "AAPL" and "BMW.DE"
const recommendationsBySymbol = await yahooFinance2.recommendationsBySymbol(
  ['AAPL', 'BMW.DE'],
  {},
  { validateResult: true },
);
recommendationsBySymbol.finance.result.map((result) => {
  console.log(
    `If you like ${result.symbol}, you might also like`,
    result.recommendedSymbols.map((item) => item.symbol),
  );
});

// will output:
// If you like AAPL, you might also like [ 'AMZN', 'FB', 'GOOG', 'TSLA', 'NFLX' ]
// If you like BMW.DE, you might also like [ 'DAI.DE', 'VOW.DE', 'SIE.DE', 'VOW3.DE', 'BAS.DE' ]

Tests
yarn test --verbose --t recommendationsBySymbol

 PASS  src/modules/recommendationsBySymbol.spec.ts (6.685 s)
  recommendationsBySymbol
    ✓ passes validation for symbol: AAPL (14 ms)
    ✓ passes validation for symbol: OCDO.L (1 ms)
    ✓ passes validation for symbol: BABA (2 ms)
    ✓ passes validation for symbol: QQQ (1 ms)
    ✓ passes validation for symbol: 0P000071W8.TO (1 ms)
    ✓ passes validation for multiple symbols ("AAPL" and "BMW.DE") (2 ms)

@gadicc feel free to have a look when you have time! :)

@pudgereyem!  This is phenomenal work!!  You really checked all the boxes on this one... styling, approach, docs, tests... it really is such a pleasure for a project maintainer when contributors fit in so well.  Thanks also for getting up to date on all the latest conventions and noticing the new quote module... I know there have been no few changes in the last week :sweat_smile:   I only have two minor comments:

As per https://github.com/gadicc/node-yahoo-finance2/commit/1806e61940f30ceaf53f6af50dbd7230df511759, could you switch the order of the overloading functions so that it defaults to returning the interface and not any?  This was my bad, sorry about that :sweat_smile:

I decided early on (but need to document somewhere, maybe in the contributing doc?) that I like the lib to clean up the result as much as possible before passing off to the user.  Do you think you could adapt the code to just return an array of results like we do in quote?  It's basically just the result transformWith (but result.finance instead of result.quoteResponse).  And then for convenience we can return just a single result if that's all they asked for (and types e.g. lines 11, 115/118, 129/132).  The quote API is obviously just a day or two old but I think it's comfortable to work with like that?

Anyway, thanks again for this amazing work!  Not sure how long you spent on this but it's super appreciated :grinning:  Keep up the awesome work :raised_hands:

@gadicc ah, thanks! It was super simple to create the new module. Thanks to the great architecture 💯
Both your comments make total sense, I'll update the PR and request a review once done.

@gadicc please have a look now, I've addressed your two comments above.

Legend!  This is really great, @pudgereyem!  Pleasure working with you :grin:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


bullet 2.87 update

update to bullet 2.87

Hm, not compiling for me. It's complaining about "m_multiSapParentProxy" being undeclared in btMultiSapBroadphase.h and .cpp
Are we just not supposed to include those files?
The errors relating to that var are the only ones I got though, so if we get that figured, the rest is likely fine.

I tested again cleaning everything, it work for me.


76 expand phase ramp and do minor fixes

The phase_ramp functions is extend to work for row column position not located on a grid. This is need to reramp the secondary image after resampling.
The get_burst_mid_times() function is also changed to return all the mid times for all burst in the original image. It makes it easier to get mid time and state for a specific burst.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@cf93f8c). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main      #77   +/-   ##
=======================================
  Coverage        ?   80.40%
=======================================
  Files           ?       26
  Lines           ?      796
  Branches        ?        0
=======================================
  Hits            ?      640
  Misses          ?      156
  Partials        ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat: add missing deps used in QML offscreen rendering

This is an attempt to fix this issue reported on Zulip

It looks like Scene3D isn't needed after all


Replace localhost with <IP_ADDRESS> for healthcheck

Additionally change the commands in the pod probes from a single string to a list of strings.
Fixes #48
I'm not sure how to handle the chart and appVersion in Chart.yaml, because they don't match what is published in the chart repo anyway, so I kept it at v1.0.0.

I'm not sure how to handle the chart and appVersion in Chart.yaml, because they don't match what is published in the chart repo anyway, so I kept it at v1.0.0.

That's fine, thank you for the PR. The versions get automatically updated from the tag when a new release is created.


Update 5-4-augmented-deliberation.md

I did some more checking on changes I had submitted...and the paragraph I added about the deliberative processes didn't get added.  (I think I did 3 different small edits so you could see them separately but only one was actually added)
I just went in and added it again along with key references to the field of processes and tools.

This is conflicted...I think that was why the other didn't get added as well.  Please make sure you have the write name for the chapter you added.

I am really confused Glen.
I literally started at the front door and went to the page and did the edits.
I didn't change the name of the chapter.
These are important edits.

Kaliya,
I have no intention of disregarding your edits or pushing things to one side.  If I have a problem with your edits, I will tell you directly; I have not shied away from this in the past.  You can verify the conflict on GitHub.
Glen

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural, Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital Minister Audrey Tang, myself and dozens around the world to tell the story of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Sunday, March 17, 2024 7:59 PM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update 5-4-augmented-deliberation.md (PR #643)
I am really confused Glen.
I literally started at the front door and went to the page and did the edits.
I didn't change the name of the chapter.
These are important edits.
—
Reply to this email directly, view it on GitHubhttps://github.com/pluralitybook/plurality/pull/643#issuecomment-2002787665, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABIRKSFU2X2STBFMOO4W7RLYYZKCZAVCNFSM6AAAAABE2MNUJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSG44DONRWGU.
You are receiving this because you modified the open/close state.Message ID: @.***>

I just re-did the edits.
If it doesn't work again it would be helpful to understand what is wrong
with the edits or how to address whatever is blocking them. .
On Sun, Mar 17, 2024 at 8:01 PM Glen Weyl @.***> wrote:

Kaliya,
I have no intention of disregarding your edits or pushing things to one
side. If I have a problem with your edits, I will tell you directly; I have
not shied away from this in the past. You can verify the conflict on
GitHub.
Glen

Research Lead, Plural Technology Collaboratoryhttps://aka.ms/plural,
Research Special Projects

Collaboratehttps://github.com/pluralitybook/plurality with Digital
Minister Audrey Tang, myself and dozens around the world to tell the story
of Pluralityhttps://www.plurality.net/

From: Kaliya - Identity Woman @.>
Sent: Sunday, March 17, 2024 7:59 PM
To: pluralitybook/plurality @.>
Cc: Glen Weyl @.>; State change @.>
Subject: Re: [pluralitybook/plurality] Update
5-4-augmented-deliberation.md (PR #643)
I am really confused Glen.
I literally started at the front door and went to the page and did the
edits.
I didn't change the name of the chapter.
These are important edits.
—
Reply to this email directly, view it on GitHub<
https://github.com/pluralitybook/plurality/pull/643#issuecomment-2002787665>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABIRKSFU2X2STBFMOO4W7RLYYZKCZAVCNFSM6AAAAABE2MNUJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSG44DONRWGU>.
You are receiving this because you modified the open/close state.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/pluralitybook/plurality/pull/643#issuecomment-2002792205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW3CC4PMQESC6I6IUXFQTLYYZKIPAVCNFSM6AAAAABE2MNUJOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSG44TEMRQGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


updates to nbextensions/usability/collapsible_headings:

Make the heading toggle buttons optional. This also necessitates registering existing cells only after config has loaded
add collapsible_headings_collapsed css class to the cell, rather than the toggle button
remove cell.metadata.heading_collapsed value from non-heading cells
update readme

Josh
Really nice extension, thanks. I use collapsed headings daily now and its one of the most useful additions to a notebook.
With regards to the toggle buttons, I too found them overly large. As an alternative to removing them, perhaps use a small, grey triangular toggle. This is pretty much the UI standard for collapsible headings.
Here is an example of it being used
http://codepen.io/echozone/details/LwJCD
and  some idea in CSS
http://stackoverflow.com/questions/19913491/toggle-triangle-on-dropdown-jquery
If you wanted to be fancier, look at how it done in a Mathematica notebook (which was the original notebook UI) look at these two.
https://reference.wolfram.com/language/tutorial/UsingANotebookInterface.html
https://reference.wolfram.com/language/guide/NotebookBasics.html
Note the group brackets to the right shows the scope of what will be collapsed. You double click anywhere on the bracket to open/close.

Thanks Kevin, glad you like it :)
Small grey carets (like the old hierarchical collapse, but preferably using fa icons) would certainly be less obtrusive, I'll add an option.
I did originally try the mathematica brackets, but couldn't think of a neat way to get them done purely in CSS, so abandoned the idea as too complicated. I didn't want to use an overlay div, as it gets complicated trying to keep track of resizing windows and such. I'd thought of using a border for the cell element, but that doesn't really work for nested brackets. The only way I can think of at the moment to add nested brackets would be to add extra elements to the right of each cell, but that'd need one in each of the input area, widget area and output area, would require fiddly handling for the tops and bottoms of the brackets depending on which elements were visible, and would break given any padding between cells sub-areas or between cells. If you can think of a neat way to do it, by all means let me know and I can give it a shot, though!
I also thought of adding a bottom border to collapsed heading cells, but didn't really want to mess with the stock css too much. Anyway, now that the collapsible_heading_collapsed css class is applied to the cell rather than the toggle control, a user could add a rule in their custom css to add a bottom border, if they wanted.

Ok, I've added a bunch more options, but no mathematica-style brackets yet, I'm afraid.

Thanks for the updates, the extension is already very useful and this makes it nicer. Would perhaps a small (gray?) bracketed ellipsis at the end of each collapsed heading be a good way to indicate that it is collapsed? Like this:
Heading
And some text.
Heading 2 [...]

@OndrejMarsalek mhm, I see what you mean about the bracketed ellipsis, it's a nice idea.
I managed to get a quick-and-dirty mockup done in pure css by tagging it onto the end of the heading as a pseudo-element, but it's far from ideal as it fights for space with the anchor link. In addition, it sort of looks like it should be clickable, but as a pseudo-element, can't be.
.collapsible_headings_collapsed .rendered_html h1:after,
.collapsible_headings_collapsed .rendered_html h2:after,
.collapsible_headings_collapsed .rendered_html h3:after,
.collapsible_headings_collapsed .rendered_html h4:after,
.collapsible_headings_collapsed .rendered_html h5:after,
.collapsible_headings_collapsed .rendered_html h6:after {
  content: "[\002026]";
  color: #aaa;
}

The real way to go about it would be to insert the ellipsis after rendering, but I think that might be slightly complicated to implement correctly because of the way the markdown is rendered, which is asynchronously, so that a function bound to the textcell execute event may (will? certainly is at least sometimes) be executed before all the markdown is in place. And that still doesn't solve the problem of conflicting with the anchor link...

So, with a little relative/absolute positioning tinkering, I've added ellipses that I think are passable, and even Mathematica-style brackets.
I think there's now plenty of options for discreetly showing (un)collapsed heading cell status, with the mini caret icons (turn icon size-matching option off), ellipses (optional), and right-hand brackets (again, optional).
FYI, Since the Mathematica-style brackets can be affected by headings below them opening/closing, it means that if they're enabled, the brackets for the whole notebook are regenerated every time a heading is toggled. This isn't the smartest scheme ever, and in really long notebooks might turn out to be a bit sluggish, but seemed ok in my testing, and, after all, it's optional anyway. Trying to make the updates smarter (skipping unaffected cells) is of questionable value in shorter notebooks, and is partly what got hierarchical collapse into buggy territory in the past, so I've avoided it for now.

Thank you for the update, @jcb91, this is great. Personally I like the ellipses the best (especially aligned to the right), but I think it is best to have these various configuration options for different people. :+1:


Relative account position sizing

Allow relative account size for trade.

Merged here.