Excessive Token Wastage

Describe the bug
Lets start that I LOVE the platform, but Im disappointed with the amount of token wastage in regenerating the same code over and over again. in the last 24 hours ive used over 10 million tokens? Whilst I know im developing something complex, most of this has been used on issue like:

The ai starts randomly generating code in the chat and not the files leaving files half completed and forcing a regeneration.
When the ai is told about such issues, it doesnt regenerate the files it messed up, it starts from the start again, causing token wastage.
The platform seems to break quite a bit when building and deploying. This again leads to regenerated tokens for no reason.

As i said at the start of this. I really LOVE this platform. Its awesome. Truly! I just cant believe ive been able to use up the code tokens of your highest plan in less than 24 hours.
Please can we discuss this further as I would happily pay more and would like more tokens. Can you assist there?
Link to the Bolt URL that caused the error
https://merry-chebakia-634b1c.netlify.app/login
Steps to reproduce
Its in the code generation platform, just look at my chat history and you will see the frustration of chats that go wrong and when they do the excessive token regeneration on files that didnt go wrong.
Expected behavior
Token usage should not be counted when there are issues with the platform its not fair on a paid account.
Or, a way to request tokens back for poor generations.
Or, a subscription package that accomodates the excessive token useage so that a developer can use this for more than 1 day
Screen Recording / Screenshot
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

I am agree with you also need to add option to get additional token , now I am stack with my code.

I am agree with you also need to add option to get additional token , now I am stack with my code.

I just spent 800k tokens on a npm build issue. Yay!

I also agree. I'm quite happy with the platform, but from my experience, it sometimes regenerates an entire long file (a few hundred lines), just to add tailwind styles for example.
Does the pricing work on updated parts of the file only, or it considers the entire file regeneration? For example if in 100 lines of code it needs to update only one, is the rewriting of the other 99 lines paid for?
I also agree it'll be nice to be able to buy token packages. Right now it can feel you're stuck and all you can do is wait for the next day. Token usage could be better adjusted to the packages you offer.
Good luck!

Hey all- apologies on the issues here, just FYI we've got fixes in flight for this and will be landing within the next 24-48 hours, more details & timeline here: https://x.com/ericsimons40/status/1843345406576787496
For the time being, for those of y'all that paid, shoot me an email at<EMAIL_ADDRESS>with your stackblitz username and/or email and I'll reload your tokens back to 10m until we get this landed 🙇

just a scam 9usd do you really need to take my 9usd and time ?

just a scam 9usd do you really need to take my 9usd and time ?

Its no scam. I created an entire working UI for a complex CRM in less than a day. The platforms awesome already, it just needs some fine tuning which Eric and the team are doing already i can see and As the originator of this thread, Eric has already been in touch to address my concerns. Great product, Fast support. Cant ask for more.  This is the issue!

Update: we have now shipped an update that will make token usage more efficient going forward! As @EricSimons stated above, if you were affected by inefficient token usage and want your tokens reset, please reach out via email to him directly or to me directly and we will get you reset back to full health!

@YDX64 my purchased tokens already gone also and i cant continue with my project! This is the issue!

@YDX64 on my end I can see you used 1,161,407 tokens today and you still have additional tokens remaining. I want to make sure you have a great experience with Bolt.new so I have gone ahead and reset you back to full 10M token health so you should have even more room to experiment this month!

I though an additional 10mil tokens was going to be $10. I just checked and it's double. Twice the price of the subscription is not cool. 10 million is already not enough for anything serious but this is very frustrating to see. You want twice the price of the subscription to refill tokens?


[DO_NOT_MERGE] [CPF-3723] LogLevelTest

Add Logging level TEST, only output during test runs.
A replacement for System.out.
npm test presently failing with:
`Failures:

ConsoleLogger should construct in default context
Message:
Expected function not to throw, but it threw [ false, 'Missing required import: testing in foam.log.ConsoleLogger' ].
Stack:
Error: Expected function not to throw, but it threw [ false, 'Missing required import: testing in foam.log.ConsoleLogger' ].
at UserContext. (/Users/jhughes/workspace/repos/NANOPAY-logging/foam2/test/src/log/ConsoleLogger.js:82:12)
Pending:`

Originally 'testing' was named 'test', but changing to 'testing' and then to anything else results in the same failure.
I've run out of ideas on how to fix the issue.

Not needed


resize listener produces error if component is unmounted/remounted

Issue:
I'm getting an error when I resize (if the component is unmounted/remounted):
Uncaught Error: Unable to find node on an unmounted component.     at invariant (invariant.js:42)     at findCurrentFiberUsingSlowPath (react-dom.development.js:3552)     at findCurrentHostFiber (react-dom.development.js:3621)     at Object.findHostInstance (react-dom.development.js:12611)     at Object.findDOMNode (react-dom.development.js:16395)     at RadiumEnhancer.updateDimensions (CoverFlow.js:124)
Reproduce:

use CoverFlow inside a component.
remove container component (confirm componentWillUnmount occurs)
curiously, notice CoverFlow DOES execute: window.removeEventListener('resize', this.updateDimensions.bind(this));
4 resize the browser window now that the CoverFlow is gone.
see error above.

I could avoid the issue by never adding the resize to begin with--that is, comment out:
window.addEventListener('resize', this.updateDimensions.bind(this));
Please tell me if there's something I can do to help.

Hey Phil, I believe this is related to #37 - TLDR is that componentWillUnmount is triggered before componentWillMount (this is inherent within React https://github.com/facebook/react/issues/11106). I'm going to delve into the repo this week to see if there's a way of refactoring Coverflow so that it is independent of the window and these particular lifecycle events. I'll let you know when it's all ready to pull

Most recent version temporarily removes reference to resizing (https://github.com/andyyou/react-coverflow/commit/62085c9e6b7dcc00594ce99b34be5e45465cc9c6)


List non-discoverable taps/targets on extractors/loaders page

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/62
Originally created by @tayloramurphy on 2021-05-25 17:30:52

We should list all of the taps on this page https://hub.meltano.com/extractors/ but have them sectioned out into "discoverable" and "not discoverable".
@DouweM do you want to take a look at this? I don't have a sense for how difficult this would be to implement.

View 2 previous comments from the original issue on GitLab


SCA: Resolve recurisve call and overhead in SCA Eclair integration

Is your enhancement proposal related to a problem? Please describe.
Due to the experimental integration of the ECLAIR SCA tool it is known that if https://github.com/zephyrproject-rtos/zephyr/pull/68324 is resolved there will be a recursion and overhead using ECLAIR as SCA tool.
Describe the solution you'd like
Resolve the recursive call and the overhead by a mechanism where the tool might be called within the actual build process and not call again the build with the configurations the same as it was called the first time.
Additional context
Status and documentation of the initial integration:
https://github.com/zephyrproject-rtos/zephyr/pull/68324
Look into a solution like the sparse SCA integration
https://docs.zephyrproject.org/latest/develop/sca/sparse.html

This issue was resolved in the PR: https://github.com/zephyrproject-rtos/zephyr/pull/68324 itself and uses the same concept as the sparse integration


Status indicator for builds/services/workloads

Is your feature request related to a problem? Please describe.
It is confusing to see spinning wheel before clicking the Deploy change button
Describe the solution you'd like A clear and concise description of what you want to happen.
- add extra Grey icon to warn that you have to deploy first
- Tooltips for the status icons to show what the icons mean
Describe alternatives you've considered
Get rid of Deploy changes button.
Additional context Add any other context or screenshots about the feature request here.

Hey team! Please add your planning poker estimate with Zenhub @Ani1357 @dennisvankekem @ElderMatt @ferruhcihan @merll


microcodeIntel: 20230214 → 20230512

Description of changes
Security updates for 8th-generation Core and later.
https://github.com/intel/Intel-Linux-Processor-Microcode-Data-Files/releases/tag/microcode-20230512
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Please add the changelog to the commit message.

@mweinelt There isn’t much public information available, but how’s this?

There's usually an advisory linked. I'd just assume that it will be linked later, so this is the best solution for now.

@ofborg build microcodeIntel


Fixed issue #7 (partially) and #8

This PR should fix issue #8 and a part if issue #7.
The plugin now also works on new elements, but modified elements aren't correctly updated yet.

Awesome. Do you have an Amazon wish list or Bitcoin address or any other kind of tip jar I can promote with the Chrome extension / README / elsewhere?

Not at the moment, but I'm planing on setting up something like that soon.


[Bug]: Cannot access x-forwarded-host in getServerSideProps

Summary
Locally, including when using netlify dev I can use x-forwarded-host header.
But once deployed, it's not populated. In fact, I can't find a way to access the hostname within getServerSideProps at all.
I'm looking to trigger a function from a page route, and just can't see how to achieve this.
A link to a reproduction repository
https://github.com/on-associates/on-nextjs-template
Expected Result
x-forwarded-host returns site hostname
Actual Result
x-forwarded-host is not populated
Steps to reproduce

Attempt to access context?.req?.headers['x-forwarded-host'] within getServerSideProps

Next Runtime version
4.39.1
Is your issue related to the app directory?

[ ] Yes, I am using the app directory

More information about your build

[X] I am building using the CLI
[X] I am building using file-based configuration (netlify.toml)

What OS are you using?
Mac OS
Your netlify.toml file
[build]
command = "yarn build:prod"
publish = ".next"
[dev]
functions = "functions"
[build.environment]
YARN_FLAGS = "--network-timeout 1000000"
[[plugins]]
package = "@netlify/plugin-nextjs"
[[plugins]]
package = "@netlify/plugin-functions-install-core"
[build]
command = "yarn vendors:storybook:build"
publish = "storybook-static/"
Your public/_redirects file

`_redirects`
# Paste content of your `_redirects` file here

Your next.config.js file

`next.config.js`
# Paste content of your `next.config.js` file here. Check there is no private info in there.

Builds logs (or link to your logs)

Build logs
# Paste logs here

Function logs

Function logs
# Paste logs here

.next JSON files

generated .next JSON files
# Paste file contents here. Please check there isn't any private info in them
# You can either build locally, or download the deploy from Netlify by clicking the arrow next to the deploy time.

Thanks for reporting the issue @brightonmike. I've added it to our backlog for the team to investigate.

Hi @brightonmike
The team at Netlify have now released the new v5 Next.js runtime which has support for both pages and app router, on-demand and time-based revalidation, automatic fine-grained cache control, and automatic image optimization using Netlify's image CDN.
You can find the documentation and additional information on the new runtime here: Next.js on Netlify
The v4 runtime is now in the maintenance support phase with no new features being added. Occasional bug fixes and security patches will be applied when needed.
Thanks!


Internal routing using "azda-route-as"?

In samples and channel9 videos I have seen "azda-route-as" HTTP header used in order to handle routing to child dev spaces, but I can't find documentation for this.
While there is documentation for public Uris as they relate to parent/child dev spaces, documentation for internal routing seems lacking. For example, considering the bike rental example where there is a bike service and a reservation service, how is the reservation service configured to call the right endpoint of the bike service. Is only the service name used and "magically" the "azda-route-as" header is read, and traffic is then re-routed to the appropriate service in a child namespace?
Did I miss documentation on this somewhere?

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

ID: 9c0f176f-ffed-ce36-150d-c6ed6d47d6f7
Version Independent ID: 9a642e32-7081-dbdb-c4be-69f44fdc8e8c
Content: Team development on Kubernetes - Azure Dev Spaces
Content Source: articles/dev-spaces/quickstart-team-development.md
Service: azure-dev-spaces
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@dahovey thank you for the feedback. Routing and the azds-route-as header are documented here:
https://docs.microsoft.com/en-us/azure/dev-spaces/how-dev-spaces-works#how-routing-works
Does this cover what you were looking for?

Can't believe I didn't see that. Yes that it is. Thanks.
I do see it now but would also like to recommend some re-organization so information is easier to locate. On the right panel "In this article", the "Run your code" doesn't seem to match with the contents of technical aspects of file synchronization and routing. While I understand these are aspects of "running your code", such features, I could see, deserve their own pages.


If you can set the width would be good, if too wide feel very strange, such as similar to the proportion of the phone?

Or is it a split column approach?

@zhl111 Hi. Just so I understand your issue. You want to be able to set a custom width for the chat bubble?
No, it is not a split column approach.

The Chat Bubbles are set to be dynamically sized based on the size of your text. And the design of Obsidian's plugin development kit means I'd have to set several classes for different sizes of the chat bubbles.
But, I have addressed your underlying issue of the chat bubbles looking too crammed in the mobile UI. The Chat View bubbles will now have a width of 90% on mobile as compared to 75% on desktop.
I might add customization for bubble width sometime in the future. But, for now, I'm closing the issue as resolved.


add support for esc and Fx hotkeys, rename from wasdcamera

Added support to the wasd camera plugin to redistribute the top Fx keys and Esc shortcuts to the whole keyboard. Since it has more options, I renamed it to "Key Redistribution".
A message for the future: This is my first commit and @melchor629 guided me through the whole process. This is my thank you note. IOU a meal!

But there is already #4991 I made earlier. I dont see how also making to what the fkeys map to is useful.

I saw #4991 after the pr :s
I would love to be able to map the camera to WASD and the most used fkeys to QERF123. It would add a more wheel-like style to the tab switch and would require less space. It would also let anyone configure the tabs emulating other control schemes they're comfortable with.

Fixed!


Hack typeface pairing

Thanks for recommending Hack with your theme!

Thank you for the awesome typeface, @chrissimpkins ! :)

My pleasure!


Updates for clean CI build

TestData.remoteCOGMultiband was on an S3 bucket that has been removed; added new public multiband COG to own bucket
Some dependency in python has changed causing a conflict between jupyter_client and nbclient
pkg_resources.ContextualVersionConflict: (jupyter-client 5.3.5 (/home/circleci/repo/pyrasterframes/target/python/.eggs/jupyter_client-5.3.5-py3.7.egg), Requirement.parse('jupyter-client>=6.1.5'), {'nbclient'})

close in favour of #509 which should trigger the docs build in CI


Create should-you-do-this.md

2nd attempt on this. Lets see if the new directory name comes through....

@erogray can you re-review? Contents should be identical to the previous attempt - but please do sanity check me on directory name and file extension.

Looks good to me, anyway...


help dialogue for 2048 Expert

A help dialogue which explains how the game works would be very useful for new users.
Related to #18

Approved

Approved


Removal of external site logic

Fixes #638
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

Only issue I saw was noted in the comment above (modifying the json schema to prevent clients from sending the site when modifying permissions lists). Unfortunately that makes this a breaking change to the API.

+135 −368

:tada: :tada: :tada:


Nextflow v21.02.0-edge breaks igenome integration

I have kept this bug report concise since I have already fixed the issue and will submit a pull request momentarily (just waiting for the test pipeline to run on my cluster...).
Check Documentation
This behaviour only appears on the latest edge release of nextflow, and so it won't be fixable using this pipeline's current docs.
Description of the bug
From nextflow v21.02.0-edge, interaction with s3 filesystems will no longer be supported by default. Instead, the nf-amazon plugin will need to be loaded.  Since igenome is hosted on s3, the rnaseq pipeline using --genome to pull data from igenome will not work until this plugin is loaded.
Steps to reproduce
Run any nf-core RNAseq pipeline with Nextflow v21.02.0-edge pulling a genome from igenomes with --genome .
Expected behaviour
The pipeline to complete without errors.
System

Hardware: HPC
Executor: SGE
OS: RHEL

Nextflow Installation
Nextflow v21.02.0-edge
Container engine

Engine: singularity
version: 3.5.3-1.1.el7
Image tag:  nfcore/rnaseq:3.0.0]

Additional context
I will momentarily submit a pull request to fix this bug before the next stable version of nextflow is released.

Pull request submitted: #572.


[question] Changing security without node removal (keeping location/name and ID)

Is there an easier way of changing security on existing device? I kinda understand that the device has to be excluded and then included in order for S2 to work but this means all node ids will change and that will messup with my hass.io and I will need to organise all switches again?

Thre is a step by step guide written by @blhoward2 but I cannot find the link to it. Maybe colud be worth adding it to docs? @blhoward2

https://github.com/zwave-js/node-zwave-js/issues/3227
Others have also said that you can use a second stick to exclude it, if you have one, in lieu of these.

@blhoward2 i assume that means reset the zwave end point device not zwave js right?

Correct.


Docker hub logs contain ANSI character codes during RUN commands

Hello,
My docker hub logs look something like this:
+ find /out -name tvheadend_*.deb

+
tvh_deb=/out/tvheadend_3.9-2883~g4019f69~trusty_amd64.deb

+
find /out -name tvheadend_*.changes

+ tvh_changes=/out/tvheadend_3.9-2883~g4019f69~trusty_amd64.changes

+ find /out -name tvheadend-dbg_*.deb

+ tvh_dbg_deb=/out/tvheadend-dbg_3.9-2883~g4019f69~trusty_amd64.deb

+ echo /out/tvheadend_3.9-2883~g4019f69~trusty_amd64.deb

+

Another example: http://stackoverflow.com/questions/28517090/docker-hub-automated-build-fails-but-locally-not
How to switch off / disable this please? It makes reading the logs difficult.

Hi!
If you are reporting a new issue, PLEASE make sure that does not have any duplicates already open.
We would like to take this time to remind you of the information we need to debug the problem you are seeing. This is an automated response so if this ticket is not about a bug, do not fret.
If you fail to provide this information within 7 days, we will close this because we cannot debug your issue. We can reopen whenever the information is provided.
Thank you.
Please see:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues
Description of problem:

`docker version`:

`docker info`:

`uname -a`:

Environment details (AWS, VirtualBox, physical, etc.):

How reproducible:

Steps to Reproduce:
1.
2.
3.

Actual Results:

Expected Results:

Additional info:

#ENEEDMOREINFO

@dreamcat4 Can you please send to<EMAIL_ADDRESS>Thanks!

Thank you. Email is sent to support.


Indentation of an included template does not respect the reference implementation.

I'll commit a fix on my fork and open a PR for this, below the description.
When a template include another template like in:
main(data) ::= <<

        <form>
            $data.inputFields : inputField(); separator="\n"$
        </form>

>>

inputField(f) ::= <<

    $f.name$
    <input type="text" id="$f.name$"></input>

>>

in the result the
 are correctly indented, but not $f.name$ and <input..., so you get:

        <form>

            Field 1
            <input type="text" id="Field 1"></input>

        </form>

instead of

        <form>

                Field 1
                <input type="text" id="Field 1"></input>

        </form>

Hi @jsnyders
I am very interested on having the pending PRs added to the main line and maintaining this awesome library live. Eager to help dedicating time as a maintainer. Maybe @mar9000 is also interested in helping too.
How we should proceed?


Add sbt-crashlytics to community plugins list

Hi @seroperson,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Typesafe Contributors License Agreement:
http://www.lightbend.com/contribute/cla

Thanks!


Complete shell selection

This allows users to pick what terminal shell is used if the reverse or bind shell they chose executes a terminal shell directly. This could be useful if /bin/bash isn't available. I made /bin/sh the default shell for portability.
This is just a starting point, I realize there may be some bind or reverse payloads that will fail when /bin/bash is not used, or only work on certain shells. I have tested some, but wanted to make a PR so we could discuss changes.
If there are many payloads that require executing a certain terminal shell/certain set of terminal shells to run successfully, maybe I can add a variable to the Shell class that contains the shells the payload is know to work against? Just an idea.

The more I've thought about it, the more I think making /bin/bash the default shell is the easy to go. Systems almost always have bash, at least in my experience, so users might end up using the --shell option regularly to change it to bash.
I think due to the project's goals, making popping shells quick and easy, /bin/bash should be the default terminal shell, and if users need to change it to anything else, which will probably not be often, they can use this option.
What do you guys think?

Hi Caps, I have read your code and need to test some before pushing it to branch.
About moving the default shell to /bin/bash, now that all shells enforce the use of --shell argument string to be generated in shellpop payload code, I do agree that /bin/bash is the best choice. Mostly that /bin/bash acts differently about file descriptor binding than /bin/sh, and that if we stick to /bin/sh this will probably break some reverse/bind shells that enforce the use of /bin/bash. (like the bash shell)
I was using /bin/sh as the default shell mostly because it only would change only the shell used in decoding string from encoding process ( --base64 or --xor). Because /bin/sh is present is most unix machines.
Thanks for the contribution, I am very busy this weekend but I will do my best to test this new code as soon as possible.

Oh, I didn't know about that bash treats file descriptors differently... that makes sense. Just a side note also, I didn't change the bash reverse shell to allow it to take different terminal shells for obvious reasons, but I get what you mean.
Setting /bin/sh as the default for decoding stubs makes sense, but if /bin/bash is the default on all the payloads, I think it makes sense for the decoding stubs to have it as the default as well. Because if /bin/bash isn't found when decoding the payload, the payload won't work if it uses /bin/bash anyway.
I will push another commit later today that changes the default shell to /bin/bash. No problem, just let me know if there's anything you want me to change after you test some!


Vanya blog post

coverage: 94.317%. remained the same when pulling 61bb3513a07e6d0e923efd6865dd64b5ca346636 on cody-vanya-blog-post into c132b0c465c1630a00fc64461186b9d3e98b10e0 on main.


Add a new config option for cookie name

Overview/summary
This library uses fixed/static cookie names (defined as STATE_COOKIE_NAME) to validate state (nonce).
The same goes session cookie name - but let's stick to the state (nonce) cookie for now.
When deploying to Firebase, cookie state validation fails.
The reason is that Firebase strips every cookie name, but only the __session cookie is allowed to pass through. (Reference)
Proposed solution
Add a config option named stateCookieName, which will be used instead of the default state cookie name
Note
The suggested solution will work for embedded apps only as it requires only one cookie (state cookie) for validating nonce.
Sessions are handled via JWT.
For non-embedded apps, it requires a separate cookie for the session as well. So not sure if it will work in the Firebase context.
While we're at it, we can just expose both cookie names (SESSION_COOKIE_NAME and STATE_COOKIE_NAME) to be configurable.

Some more findings:
The previously proposed solution will not work as is - because it uses signed cookies.
Signing creates an additional cookie {cookieName}.sig. And in the Firebase context, __session.sig will still be stripped out. (I did a test run..)

We can either simply expose another option to disable cookie signing, just like cookie-session package.
That'll solve it for embedded apps.

The complex solution would be to use a unified cookie (one cookie to rule them all!).
We store all the data (session id, nonce state, etc) in one cookie.
Like this (hypothetical code):

app.get('/auth/login', (req, res) => {
  const sessionData = {
    // sessionId: '123456789',
    nonce: 'abcdefg',
    // other properties...
  };

  const serializedData = JSON.stringify(sessionData);
  const mac = generateMAC(serializedData);

  // Set the cookie with the MAC
  res.cookie('__session', serializedData + ':' + mac);
})

And verify like this (hypothetical code):
app.get('/auth/callback', (req, res) => {
  const cookieValue = req.cookies.__session;

  // Extract the serialized data and MAC
  const separatorIndex = cookieValue.lastIndexOf(':');
  const serializedData = cookieValue.slice(0, separatorIndex);
  const storedMAC = cookieValue.slice(separatorIndex + 1);

  // Verify the MAC
  const computedMAC = generateMAC(serializedData);

  if (computedMAC === storedMAC) {
    // MAC is valid
    const sessionData = JSON.parse(serializedData);

    // Access the values
    const sessionId = sessionData.sessionId;
    const nonce = sessionData.nonce;
    // other properties...

    // Perform necessary operations with the session data

    res.send('Example endpoint');
  } else {
    // MAC is invalid
    res.status(401).send('Invalid MAC');
  }
});

More context in next-firebase-auth package:
https://github.com/gladly-team/next-firebase-auth/issues/190
I'm not using this package. Only added it for reference since it's a problem many packages have.


Add UniQ partitioning algorithm

Firstly, we need to fix unit test, it seems that current implementation is not compatible with python3.8.
Then you may refer to my comments. Most are just coding style issues.


Changes in extension does not appear on Mozilla Firefox Browser

I would like to work on this and try to add a separate index.html file to firefox directory.

Go ahead.

Staling weakly described issue, closing.


Error deploying Aria Automation using VMware.PlatformTools

Code of Conduct

[X] I have read and agree to the Code of Conduct.
[X] Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
[X] Do not leave "+1" or other comments that do not add relevant information or questions.
[X] If you are interested in working on this issue or have submitted a pull request, please leave a comment.

VMware Cloud Foundation
4.5.2
Module Version
<IP_ADDRESS>6
PowerShell Version
7.3.6
PowerCLI Version
<IP_ADDRESS>24340
PowerVCF Version
<IP_ADDRESS>0
Guest Operating System
windows server 2019
Environment Details
No response
Description
30, 93, 5 | Management Domain | vRealize Suite Lifecycle Manager: Deploy vRealize Automation by Using vRealize Suite Lifecycle Manager
Facing issue with PCA option 5: Deploy vRealize Automation by Using vRealize Suite Lifecycle Manager
Error or Debug Output
Error at Script Line 17892
Relevant Command: $jsonObject += [pscustomobject]@{
Error Message: You cannot call a method on a null-valued expression.
Expected Behavior
Vra should get deployed without any issues
Actual Behavior
Facing issues while deploying vra.
Steps to Reproduce
30, 93, 5 | Management Domain | vRealize Suite Lifecycle Manager: Deploy vRealize Automation by Using vRealize Suite Lifecycle Manager
Facing issue with PCA option 5: Deploy vRealize Automation by Using vRealize Suite Lifecycle Manager
Log Fragments and Files
No response
Screenshots
No response
References
No response

Is this error still be observed in the CI, @Ashwini-Ravi?

Is this error still be observed in the CI, @Ashwini-Ravi?

No i am not seeing this issue now and it can be closed


Notifications in JIRA/Confluence

I know this is a GitHub issue, but seeing as some people are not using JIRA yet, I'll put this here.
If you like the notification system on GitHub to see comments posted on issues, there is a way of doing this in JIRA/Confluence. Log into your account here, once logged in, hit the Hamburger icon and select 'Confluence' from the drop-down menu. You will now see the notification tray similar to GitHub's appear next to your user icon. By clicking one of the modified issues displayed, you will see recent comments and changes to that issue, like GitHub.
List of modified issues:

Recent changes to an issue:

Please excuse the full-page screen shots, I've not got around to installing basic image manipulating software yet. :P

Which OS is that? Is that Windows 2000 or a style for Windows 7?

Nah it's Windows XP. :P I use this old PC with 1.24GB of RAM and an Pentium 4 I clocked at 3.04 Ghz. Apart from the RAM, it's actually faster than my Windows 7 laptop because it doesn't have so much rubbish installed and I'm not using a fancy theme (and the clock speed is faster). :P This is actually turning into my main PC, for OpenSprites, anyhow; I have a HDD dedicated to daily backups. :P

Ah, OK. I use Windows XP for my old laptop that I'm using right now because it seems to work a lot better than Ubuntu, Windows Vista/7 and Linux Mint. It was either XP or 2000, which also worked well too, soo... :P

:P

Test comment

Testing 123

Why are we test commenting?

test lel

To test the Gitter integration :P

I just watched the tutorial video today, (I had been busy and I just came back from my mini weekend computer-less vacation) and I feel this is a silly question but how do you contact Zeusking19 to request an account? Here? https://opensprites.atlassian.net/admin/users/sign-up ( I just did this )

I'll take a look for you. :)

Well, I think it's correct. I got the email, logged in, and see the developer agreement (which only logged in users can see.. right?)

Yep! Click the hamburger icon in the corner to navigate to JIRA, our new issues platform.


Support for COMMENT ON TABLE, COMMENT ON COLUMN...

Please add support for the following constructs ( used by Postgres, Oracle, Redshift ) :
COMMENT ON TABLE texto_fichero is "This is a sample comment";
COMMENT ON COLUMN texto_fichero.fichero_id is 'This is a comment';
MySql uses:
ALTER TABLE texto_fichero COMMENT "This is a sample comment";
ALTER TABLE texto_fichero MODIFY id COMMENT 'some comment';

So first of all JSqlParser does not allow this double quoted string literals. For JSqlParser those are always object names.
So using single quotes these statements are parsed successfully using JSqlParser 3.2-SNAPSHOT:
COMMENT ON TABLE texto_fichero is 'This is a sample comment';
COMMENT ON COLUMN texto_fichero.fichero_id is 'This is a comment';

These alter statements are not yet supported.

The normal comment statement was introduced late 2018 for issue #685.

Right, thank you, I found it.
One small issue on Comment:
it has two methods comment.getTable() and comment.getColumn()
When using 'COMMENT ON COLUMN 'myTable.myColumn' is 'Some comment'
it returns comment.getColumn().getColumnName() = "myTable.myColumn"
and comment.getColumn().getTable() = null
Wouldn't be correct that the column has also the Table set ( and not the table in the column name ) ?
Please make sure the that it works also  COMMENT ON COLUMN 'mySchema.myTable.myColumn' is 'Some comment'
Best regards,
Dragos Pruteanu
--
DbSchema - Wise Coders GbmH
82256 Fürstenfeldbruck bei München
Email<EMAIL_ADDRESS>- Web: [1] http://dbschema.com/
--
[1] http://www.example.com/

You are right. I will change this.
I will look into the alter statement to change the comments. Lets see how far I get with my somewhat limited time :).

OK. I looked into it. The Comment class differs between commenting a table or a column exactly by setting the table or the column. I tested your example and could not reproduce your error report. Here is my test:
    Comment comment = (Comment) CCJSqlParserUtil.parse("COMMENT ON COLUMN myTable.myColumn is 'Some comment'");
    assertThat(comment.getTable()).isNull();
    assertThat(comment.getColumn().getColumnName()).isEqualTo("myColumn");
    assertThat(comment.getColumn().getTable().getFullyQualifiedName()).isEqualTo("myTable");

It produces not this combined name myTable.myColumn like you described.
The second example including a schema name works as expected too:
    Comment comment = (Comment) CCJSqlParserUtil.parse("COMMENT ON COLUMN mySchema.myTable.myColumn is 'Some comment'");
    assertThat(comment.getTable()).isNull();
    assertThat(comment.getColumn().getColumnName()).isEqualTo("myColumn");
    assertThat(comment.getColumn().getTable().getFullyQualifiedName()).isEqualTo("mySchema.myTable");
    assertThat(comment.getColumn().getTable().getName()).isEqualTo("myTable");
    assertThat(comment.getColumn().getTable().getSchemaName()).isEqualTo("mySchema");

So do you want go have this mysql variant of commenting using alter as well?

Yes, it would be nice to implement it.
Best regards,
Dragos Pruteanu
--
DbSchema - Wise Coders GbmH
82256 Fürstenfeldbruck bei München
Email<EMAIL_ADDRESS>- Web: [1] http://dbschema.com/
--
[1] http://www.example.com/

So implementing those new constructs will take its time. So I will see, if I could get on it in the next days. OK?

Great, thank you !
Best regards,
Dragos Pruteanu
--
DbSchema - Wise Coders GbmH
82256 Fürstenfeldbruck bei München
Email<EMAIL_ADDRESS>- Web: [1] http://dbschema.com/
--
[1] http://www.example.com/


 component How to set height?

I want  to be short

this is my code

If you take a look at the source code, you'll see that <Pagination /> receives a default style of { paddingVertical: 30 }.
Using prop containerStyle, you can either pass a different paddingVertical (my recommendation) or set it to 0 and define a custom height.

@bd-arc Thanks very much


Same url different method, should be listed in UI.

Currently, we are not logging twice if the same URL is encountered more than once due to a issue in sinon which returns a 404 if the same URL is intercepted twice. Refer #48. Need to add a condition such that same URL with different method must be listed in view

Closing as the issue got addressed in #69.


[Integration][PagerDuty] Fix Rate limit from querying Pagerduty service analytics

Description
What - Implemented rate limiter on PagerDuty
Why - Lots of requests are failing as 429.
How - Used a single sephamore since concurrent requests are not advised
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)

 All tests should be run against the port production environment(using a testing org).
Core testing checklist

[ ] Integration able to create all default resources from scratch
[ ] Resync finishes successfully
[ ] Resync able to create entities
[ ] Resync able to update entities
[ ] Resync able to detect and delete entities
[ ] Scheduled resync able to abort existing resync and start a new one
[ ] Tested with at least 2 integrations from scratch
[ ] Tested with Kafka and Polling event listeners
[ ] Tested deletion of entities that don't pass the selector

Integration testing checklist

[ ] Integration able to create all default resources from scratch
[ ] Resync able to create entities
[ ] Resync able to update entities
[ ] Resync able to detect and delete entities
[ ] Resync finishes successfully
[ ] If new resource kind is added or updated in the integration, add example raw data, mapping and expected result to the examples folder in the integration directory.
[ ] If resource kind is updated, run the integration with the example data and check if the expected result is achieved
[ ] If new resource kind is added or updated, validate that live-events for that resource are working as expected
[ ] Docs PR link here

Preflight checklist

[ ] Handled rate limiting
[ ] Handled pagination
[ ] Implemented the code in async
[ ] Support Multi account

Screenshots
Include screenshots from your environment showing how the resources of the integration will look.
API Documentation
Provide links to the API documentation used for this integration.

Missing pyproject.toml


load html document into an iframe object

I've got a web service that scrapes and parses an html document, and then scrapes and parses its contained iframes' html document. At the end, I return the root node's outerHTML.
After parsing the iframes' html document, I'm trying to load it under the iframe object (mimicing what a web browser does). However, when the returned outerHTML is rendered, the iframes are empty.
Is what I'm trying to do can be done?
code sample:
using System;
using System.Linq;
using HtmlAgilityPack;
using ScrapySharp.Extensions;

public class MyClass
{
    public static string GetContent(string url)
    {
        var web = new HtmlWeb();
        var document = web.Load(url);
        var iframes = document.DocumentNode.CssSelect("iframe").ToList();
        foreach (var iframe in iframes)
        {
            var iframeUrl = iframe.Attributes["src"].Value;
            string noSrcOuterHTML = iframe.OuterHtml.Replace("src=" + "\"" + iframeUrl + "\"", string.Empty);
            var clonedIframe = HtmlNode.CreateNode(noSrcOuterHTML);
            iframe.ParentNode.ReplaceChild(clonedIframe, iframe);
            var iframeDocument = web.Load(iframeUrl);
            iframe.InnerHtml = iframeDocument.DocumentNode.InnerHtml;
        }
        return document.outerHTML;
    }
}

Hello @HeyJude1 ,
Could you send us the URL for which you are testing this code?
Best Regards,
Jonathan

@JonathanMagnan, the url is here. The Github Gists found there are the iframes I try to get.

Hello @HeyJude1 ,
I believe there is for sure a way to do it.
var web = new HtmlAgilityPack.HtmlWeb();
var document = web.Load("https://medium.freecodecamp.org/i-dont-understand-graph-theory-1c96572a1401");
var iframes = document.DocumentNode.CssSelect("iframe").ToList();
foreach (var iframe in iframes)
{
    var iframeUrl = iframe.Attributes["src"].Value;

    if (iframeUrl.StartsWith("/"))
    {
        iframeUrl = "https://medium.freecodecamp.org" + iframeUrl;
    }

    //string noSrcOuterHTML = iframe.OuterHtml.Replace("src=" + "\"" + iframeUrl + "\"", string.Empty);

    var iframeContent = web.Load(iframeUrl);
    var clonedIframe = HtmlNode.CreateNode(iframeContent.DocumentNode.InnerHtml);

    var parent = iframe.ParentNode;
    iframe.ParentNode.ReplaceChild(clonedIframe, iframe);

    //iframe.InnerHtml = iframeDocument.DocumentNode.InnerHtml;
}
var outerHtml = document.DocumentNode.OuterHtml;

Here is a code sample that actually replaces the iFrame by the content of the page.
I hope that will help you to get started.
Best Regards,
Jonathan

@JonathanMagnan, thanks, but it doesn't work - it actually removes the iframes from the DOM (Because you replace the iframe itself (iframe) with its content (clonedIframe)).

Oh, this is what I thought you wanted.
I'm still not sure when looking at your first post, so let me know exactly what it should do and I may try it.
Best Regards,
Jonathan

I want the iframes (and their content) to be displayed, just as in the original url.

Hello @HeyJude1 ,
Something like this?
var web = new HtmlAgilityPack.HtmlWeb();
var document = web.Load("https://medium.freecodecamp.org/i-dont-understand-graph-theory-1c96572a1401");
var iframes = document.DocumentNode.CssSelect("iframe").ToList();
foreach (var iframe in iframes)
{
    var iframeUrl = iframe.Attributes["src"].Value;

    if (iframeUrl.StartsWith("/"))
    {
        iframeUrl = "https://medium.freecodecamp.org" + iframeUrl;
    }

    var iframeContent = web.Load(iframeUrl);
    iframe.InnerHtml = iframeContent.DocumentNode.InnerHtml;
}

var outerHtml = document.DocumentNode.OuterHtml;

@JonathanMagnan, that's pretty much what I did (See the code I posted when opening the issue; I also removed the src attribute from the iframe element), and it doesn't work. Any other idea?

No idea,
it's working fine on my side ;(

Hello, @HeyJude1
Code from issue doesn't work beacause of two mistakes: first of all, you try to load a page without domain name right here:
var iframeDocument = web.Load(iframeUrl);

This mistake leads to runtime exception.
Second mistake is that you assign content of iframe document to wrong object. Just replace
iframe.InnerHtml = iframeDocument.DocumentNode.InnerHtml;

with
clonedIframe.InnerHtml = iframeDocument.DocumentNode.InnerHtml;

and everything will work perfectly.
Also consider to use this code. It looks more efficient and works as it should.

@SMishanya, you're right about my two mistakes. After I fixed them, it still doesn't work - see this screenshot: It demonstrates the DOM I get, which is rendered as a blank iframe.
BTW, I don't see how it's possible that the code you linked works if it doesn't remove the src attribute, which gives a 404 error - see this screenshot.

@HeyJude1, oh,  I see where is the problem.  Try this: https://stackoverflow.com/questions/6102636/html-code-as-iframe-source-rather-than-a-url

Hello @HeyJude1 ,
This issue will be closed since it has been resolved by @SMishanya I believe.
Feel free to reopen it if you feel otherwise. However, you will probably get a better answer by asking directly on Stack Overflow.
Best Regards,
Jonathan


Clarify resource percentage calculation in README

Fixes https://github.com/kubernetes-sigs/descheduler/issues/514
Clarifies what we mean by "number of pods in terms of percentage" with a link to upstream docs

/lgtm


add  David Doston as maintainer [skip ci]

@dotsdl Are you OK if I add you as an official maintainer?

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/core are they build fails because this package still needs an updated conda-smithy?
Since I've just added a maintainer name there shouldn't be big issues.

you probably need to rerender

@scopatz how do I do the rerender? So far that always happened automatically for other recipes.

you need to install conda-smithy and then run "conda smithy rerender" in the repo.

ref: https://github.com/conda-forge/conda-smithy/blob/master/README.md#re-rendering-an-existing-feedstock

@dotsdl ping

@kain88-de I'm good to do that. Please add me.


go-ethereum: 1.10.11 -> 1.10.13

Motivation for this change
go-ethereum 1.10.12 or higher is needed to sync with the network after the hardfork recently
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

#152712

#152712


[cmake] Fix capitalization in find_path argument.

/cc @compnerd

Since I removed the duplicated script in #1746, I suppose this is the only change needed. But if you want the duplicated script, I can duplicate this change easily.

@swift-ci please test and merge

The failure seems unrelated to this changes. I think I have seen the same test failing in another completely unrelated PR. Can we restart the tests? Thanks.

@swift-ci please test and merge


File Upload Issue

I am a novice Java Developer. Apologies for any mistakes.
Just wanted to check with you guys if this code can be enhanced in the package to run file uploads
Java File
public static String runFileUpload(String base64Img, String fileName, String fileType, String selector) {
        return "window.fetch(\""+base64Img+"\").then(result => result.blob()).then( res => {const f = new File([res],'"+fileName+"', { type:'"+fileType+"' });const dt = new window.DataTransfer();dt.items.add(f);var element = document.querySelector('"+selector+"');element.files = dt.files; element.dispatchEvent(new Event('change', { bubbles: true }));});";
    }

Feature File
 * def scriptString =<JavaFile>.runFileUpload(base64Img, fileName, fileType, "#file-upload")
 * script(scriptString)

I have tried to run with chromedriver, it works perfectly fine. May have to consider and test cross browser compatibility while running java script. But i guess, it should work.

@dulloor I think this question belongs on stack overflow: https://stackoverflow.com/questions/tagged/karate
to be honest I don't understand your question at all, so when you re-ask on SO, please try to be more clear.

@ptrthomas this was in reference to solution for file upload driver.inputFile() function which only works with chrome driver and it’s inability to expand it to grid. My above solution is debatable but it was helping my cause in grid automation and I  was able to extend and test it to chrome, Firefox browser.


Set Price for specific type/colour of blocks.

If I wanted to set a specific color of stained glass for example, to have a different price how would I set this in the config file.
From what I can tell we can't set for example
241:4 35 # Yellow Stained Glass $35
241:1 120 # Orange Stained Glass $120
It seems to only accept the first number of the ID.  It'd be nice to have the flexibility to set different prices for blocks of the same type but different colors. This is not limited to the stained glass example and can apply to a variety of blocks that use a secondary number to identify the block.
The other advantage would be when someone went to use /sell all  while they had the Yellow Stained Glass in hand.. rather then it selling all stained glass it'd sell just the stacks of that specific color.

sorry for the wait, but i have now implemented this.

Thanks!


CDK/CloudFormation: ignore mutable tag "basti:in-use"

Feature Description
When using CDK deploy or diff, "no operation" is expected if there is nothing to change
Use Case
Running cdk diff on a Basti CloudFormation stack reports systematically a change on EC2 tag basti:in-use
Resources
[~] AWS::EC2::Instance basti-instance/Ec2InstanceBastion bastiinstanceEc2InstanceBastionC84AB388
 └─ [~] Tags
     └─ @@ -9,7 +9,7 @@
        [ ] },
        [ ] {
        [ ]   "Key": "basti:in-use",
        [-]   "Value": "2024-06-28T12:42:12.448Z"
        [+]   "Value": "2024-08-09T08:19:57.890Z"
        [ ] },
        [ ] {
        [ ]   "Key": "Environment",

Proposed Solution
Exclude this tag from resources under lifecycle management in CF stack

@ymartin59, thanks for reporting this! I'll take a look at the problem soon. Meanwhile, would you consider contributing to Basti?


[feat] add a warning when not using trackBy with ngFor

Even though its considered best practise, I always forget to add a trackBy when using ngFor. It would be awesome to have a rule to enforce this in templates. Since invalid usage of trackBy is currently swallowed silently by angular, as a stretch it would be cool if the rule validates the trackBy expression as well. If you're in favour of this I would love to take a go at a PR for this feature. Thanks! 😄

Sounds like a good suggestion. There's issue related to the errors reporting in ngFor. Once this is fixed it'll be relatively easy to implement.

I came here to post that but @mattlewis92 did it 2 days ago :smile:  !
Indeed, I think it would be a really nice feature! :beers:

Will move it to the next release in order to not bloat this one.

This would be an amazing feature!
... and it even looks like @maxime1992 offered some beer to get it fixed 😉

I keep track of that ;)


Change VERSION to  0.1.1

Codecov Report

Merging #76 (c972bae) into main (53636b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   78.01%   78.01%
=======================================
  Files          49       49
  Lines        4475     4475
=======================================
  Hits         3491     3491
  Misses        984      984

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 53636b2...c972bae. Read the comment docs.

Codecov Report

Merging #76 (c972bae) into main (53636b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #76   +/-   ##
=======================================
  Coverage   78.01%   78.01%
=======================================
  Files          49       49
  Lines        4475     4475
=======================================
  Hits         3491     3491
  Misses        984      984

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 53636b2...c972bae. Read the comment docs.


Converted all bash scripts' shebangs

As the shebang of postgres/docker-entrypoint-initdb.d/extensions.sh was broken, I fixed it. While I was going at it, I also converted all the other bash scripts' shebangs to the /usr/bin/env style.
In the ruby scripts the shebang was pointing to /bin/env, which is not available in the ubuntu:16.04 docker image. Changed that to the real location at /usr/bin/env, too.

Thanks.


Spelling and grammatical updates for readability

Simple, non-code changes 😄

thanks for catching a few more of those pesky 'vica versas'.


availability of xfms and other HCP Pipeline intermediates

Hi, these pipelines and the already preprocessed dataset are amazing resources, thank you all so much for putting this out there! I'm still new to the collection, but it looks like the subject-level HCP-Pipeline-style data available through NDA are limited to:

anatomical midthickness surfaces (native+MNI @ 32k+164k)
anatomical CIFTI outputs (curvature, thickness, sulcal depth, myelin maps)
the ACPC-aligned native-space anatomical segmentation from FreeSurfer's Desikan-Killiany atlas (aparc+aseg)
fMRI dense timeseries CIFTIs and associated movement parameters
fMRI parcellated timeseries CIFTIs created using the Gordon/HCP/Markov/Power/Yeo parcellations

Is it possible to get any of the other intermediate files generated by the HCP pipelines? In particular, the warpfields, jacobians, and affine matrices stored in T1w/xfms and MNINonLinear/xfms would be very helpful, since for many applications, I want to define ROIs in native space and then transform them into MNI/CIFTI space or vice/versa. These would also enable useful things like quickly regenerating the ACPC-aligned, distortion corrected anatomical images (T1w_acpc_dc/T2w_acpc_dc) from the original T1w/T2w data to check FreeSurfer's surface extraction and segmentation.
Thanks again! Looking forward to digging into the collection.
-Ely

@elyb01 Terribly sorry for the long wait for you on this issue.  First, many thanks for asking for data exactly as directed in the documentation.  Second, let's try to answer your questions in the last paragraph there:

I believe we intentionally deleted many of the transforms as they were numerous small files taking up inordinate amounts of space and file handles on our servers (as you can imagine).  But I will check of your list what intermediary transform files, etc, we have.  Are there any that are more important to you than others?

I believe we mapped in the T1 and T2 volume files in our most recent upload.  Read here and see if that might be what you want.

derivatives.anat.(T1w|T2w)
Anatomical imaging masked brain or full head in MNI space in a volume.
sub-#/ses-#/anat/sub-#_ses-#_(T1w|T2w)_space-MNI_(brain|head).nii.gz

Let us know what else you need for your analyses and we will prioritize it for addition to the collection.

@elyb01 Hope you're doing well, we are closing this comment but let us know if you need anything else from us!


Oval tool filling pixels incorrectly for oblong shapes

The oval shape is filling incorrect pixels along the X axis
X=2 Y=1

X=5, Y=1

X=6, Y=3

Y=3. X=6

@Cody811 Might be an issue in algorithm, we will fix it, Thanks.
https://github.com/rgab1508/PixelCraft/blob/cf69d934483dbd06e8b4e7b8dd7190823fb8f9f1/lib/Shapes.js#L134

@Cody811 Might be an issue in algorithm, we will fix it, Thanks.
https://github.com/rgab1508/PixelCraft/blob/cf69d934483dbd06e8b4e7b8dd7190823fb8f9f1/lib/Shapes.js#L134


Remove Definitions from References

Remove Definition from Reference list, since there are different.
Defined functions or methods should'n appear in reference list, because they'r not references and available from separate menu Go/Peek Definition or within shortcut.

Version 1.0 will respect the value of ReferenceContext.includeDeclaration which is sent with the textDocument/references request. In my tests vscode sends true for this value when finding references so definitions/declarations will be included. Please file an issue against the vscode repo if you think this should be changed.

Vscode #68071


Add dock static-only command

Fixes #39

@yannbertrand Do I need to have mac os to get screenshots and test this command?

@yannbertrand Do I need to have mac os to get screenshots and test this command?

@thisisommore Thanks for your PR, it's ok without it 🙂

@yannbertrand I have marked it as ready for review

LGTM, thanks @thisisommore :)


Raspberry Pi Sense-Hat LED Matrix stays lit to rainbow pattern, and inaccessible to code.

I got the Raspberry Pi Sense-Hat to work on an earlier build of Raspian, and was even able to program some simple programs using it. Now, the LED Matrix remains on, permanently showing the rainbow test pattern.
I try to clear it in Python 3, and I get the following when I try to instantiate the object:
Traceback (most recent call last):
File "", line 1, in
File "/usr/lib/python3/dist-packages/sense_hat/sense_hat.py", line 39, in init
raise OSError('Cannot detect %s device' % self.SENSE_HAT_FB_NAME)
OSError: Cannot detect RPi-Sense FB device
I've checked the connections to the RPi and Sense-Hat, and they seem secure. This is in fact been an ongoing problem, as I ended up exchanging both a perfectly good RPi3 and Sense-Hat for new ones, only to have the same issue.
I have also tried a fresh install with a microSD with only Raspian installed, to avoid any possibility of a misconfiguration on the part of the OS. The issue did not resolve.
Any input would be appreciated.

What has happened between the time it was working and not?
What's the output of i2cdetect -y 1?

Thank you for the reply. Between the time where the system worked properly and when it started to give me trouble was a system update. Basically,
sudo apt-get update
sudo apt-get upgrade
sudo apt-get dist-upgrade
I'm afraid that entering the output of that command gets messed up in the text due to formatting isssues, so I've attached the textfile containing output.
out.txt

Could you also post the output of raspi-gpio get?

Sure thing.
Again in the text file to skip formatting.
gpiooutput.txt

Is there anything else connected to GPIO? It looks like none of the devices are acknowledging that they're there. This could happen if something else was connected and was pulling the pin low.
It might be worth trying the earlier build again. If that doesn't work anymore, then you'll know it's a hardware problem.

Just a few buttons. I followed the instructions provided at https://www.raspberrypi.org/learning/astro-pi-flight-case/worksheet2/, and after testing, the buttons and camera are all functional.
How do I revert to an older build of Raspian?

An archive of all the images can be found here:
http://downloads.raspberrypi.org/raspbian/images/
Another possibility is that the HAT itself was damaged or the pin on the pi was damaged. That could've happened if there was a short, for example. If you have any other I2C devices you could test, that would be useful information, but I suspect you might not have anything suitable lying around.

Unfortunately, I don't have any other I2C devices. I've tried this on two SenseHats, on two separate RPi's, and I doubt the possibility that it's a hardware issue.
Nevertheless, I tried loading the library, and the issue is still not resolved.
I have a multi-meter. Is there any guide for testing the pins with a regular multimeter?

Just saw your edit. Here's the output of the two commands you wanted to see. This is from the
uname.txt
vcgencmd.txt

If we've ruled out software (by returning to your previous setup), and you're saying it couldn't possibly be hardware, what does that leave?
If you start with a fresh, untouched card with only power and the HAT connected, you should see the rainbow pattern go on and then turn off when the kernel loads the relevant module. If that doesn't happen, that certainly points to a hardware issue.
Using the multimeter, you could arm yourself with the datasheet of one of the i2c devices and the sense hat schematic (https://www.raspberrypi.org/wp-content/uploads/2015/08/Sense-HAT-V1_0.pdf), you could do some basic troubleshooting. While it's running, carefully check the main voltage rails. When it's disconnected, check the connections and that there are no shorts to ground on the sda/scl lines. To check what's really happening, you'd want to use a scope and/or a logic analyzer.

You were right, it was a hardware issue. Except not with the boards.
It was with the extended 23-way pin header I was using. Either it was a dud or it was poorly made, because I swapped it out with a more sturdy extended header from Adafruit, and now it's working just fine. Thanks for your input, and I'm closing the issue.


Relation delete button error

"relation delete button" is not working in v1.5.0.
but it works in v1.4.x

#2612


GraphQL+- supported?

GraphQL+- is an alternative GraphQL used for dgraph.  Have any plan to support this QL?

No, not in the foreseeable future. It looks like all of the additional features of GraphQL+- are available in EdgeQL and in a more expressible way.


clean-css border error result

my code before clean-css:
border: 2rpx dashed #ddd
Output:
border:dashed #ddd;

Do you suggest it should be scrapped altogether? As far as I know rpx is not a valid unit, is it?

Please reopen if needed.


Clang fails to compile KeyDB with a linker error

Describe the bug
Clang fails to compile with the error is shown below:
ld.lld: error: undefined symbol: __atomic_fetch_add_2
>>> referenced by sds.c:185
>>>               lto.tmp:(sdsdupshared)
clang-10: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [Makefile:276: keydb-server] Error 1

To Reproduce
export CC="clang"
export CXX="clang++"

CFLAGS="-flto=thin -fno-plt -march=native -mtune=native -O3 -pipe"
CXXFLAGS="-flto=thin -fno-plt -march=native -mtune=native -O3 -pipe"
LDFLAGS="-fuse-ld=lld -Wl,--as-needed,-O1,--sort-common,-z,now,-z,relro"

git clone https://github.com/JohnSully/KeyDB.git
make BUILD_TLS=yes -C "KeyDB"

Thanks!
I need to do a few more tests before merging. This atomic lib has caused no end of grief. So platforms need. Some don’t care and others can’t have it all.

@JohnSully I only skimmed over the code, but the file src/sds.c doesn't seem to provide an alternative to not using libatomic.

In some cases its using compiler builtins and therefore not dependent on an external library. Initially I was unconditionally linking libatomic but that caused build failures with GCC on ARM (didn’t test clang).
Your change appears fine, just needs to be validated on ARM. I’ll have it merged soon.

@JohnSully I'm a bit busy ATM for this one, but a hint on memkind that it fails with Clang too.


Create deps.yml when --only-changed is set

This allows you to always run with --track-dependencies --only-changed,
and if deps.yml is missing, it will write a new one.
It also fixes a small issue where other types of failure in loading
deps.yml were always treated as a general corruption, because the errors
inherited from StandardError through RuntimeError, so they were
re-raised as the less-specific InvalidDepsYAML.
Closes https://github.com/middleman/middleman/issues/2262.

@tdreyno could you take a look, please?


Aggregation as an OpenSearch Module

TBD
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hello @anasalkouz ! Can you please fill this out with more information about the enhancement you want to make? We need more information to understand the problem and the enhancement.
Thank you!

I have added the high level idea, we will add more details soon.

Thanks @anasalkouz ....more details would be very useful in this case!  More details about the problem you're trying to fix in particular would be good :)
Thanks again,
/C


I'm guessing an empty home page of YT, I can totally get why @s97712 wants this lol, the recommendations can sometimes be annoying

Closing as this is a filterlist for shorts only.


feat(answer): knowledge answer controller

Summary
This increment adds the Controller that will map the state of the answerApi Slice to the Generated Answer State used by the front end components.
Why this is needed
Since we must for a while support the current working flow with the API stream, while adding the new flow from the AnswerAPI, it was way simpler to make sure both implementation rely on the same state. Cuz having to rework all the components to align them with the AnswerApi slice directly would be breaking and would not cover all the actual functionalities, like rephrasing.
How does it work
The front end components will use the same controller in both the flows, the only thing changing, at a DX level, is to pass the configuration Id in the props of the controller.
By doing so, the first level controller will then build and extend the right sub controller to handle the proper GenAI flow.
Both the legacy controller working with the searchUi and the new controller working with the Answer API share the same state.
The configuration Id will be passed to the client seamlessly.
In most cases, the answer generation will be identical in both the flows. The main difference is that the flow using the answerConfigurationId unlocks the answer management capabilities.

The changes look good 👍

Thanks !

I have just a question about why the new way of streaming answers using the AnswerAPI is not implemented as well in the Insight Generated Answer controller?

Not yet !

Also, after this PR, the next one to review is the following: https://github.com/coveo/ui-kit/pull/4148/files#diff-40f9b6094183b67c164540da56e5747ca8dec42711e580891dd77cd7a682a008 right?

Yes !


No .NET SDKs were found

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/create
Operating System: windows
After installing the dotnet SDK as specified in the "Download and Install" section in the tutoial, I tried running the dotnet new console -o MyApp -f net7.0 command, which produced the following message:
The command could not be loaded, possibly because:
  * You intended to execute a .NET application:
      The application 'new' does not exist.
  * You intended to execute a .NET SDK command:
      No .NET SDKs were found.

Download a .NET SDK:
https://aka.ms/dotnet-download

Learn about SDK resolution:
https://aka.ms/dotnet/sdk-not-found

running dotnet --info yields the following:
global.json file:
  Not found

Host:
  Version:      6.0.10
  Architecture: x86
  Commit:       5a400c212a

.NET SDKs installed:
  No SDKs were found.

.NET runtimes installed:
  Microsoft.NETCore.App 5.0.10 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 6.0.10 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.WindowsDesktop.App 5.0.10 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
  Microsoft.WindowsDesktop.App 6.0.10 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]

Download .NET:
  https://aka.ms/dotnet-download

Learn about .NET Runtimes and SDKs:
  https://aka.ms/dotnet/runtimes-sdk-info

I tried restarting my compute after installing the SDK, and reinstalling it, to no avail.

@mfalagan did you download and install the SDK itself and not, say, a runtime? e.g. SDK 7.0.202 on this page -https://dotnet.microsoft.com/en-us/download/dotnet/7.0.

Yes, I had installed the SDK.
Anyway, I tried again with the SDK you provided, and still had no luck.
I see you added the needs more info label, I'm not well versed in github issues, so I'd greatly appreciate if you could specify which more useful information I could provide, I would be happy to do so.
I am not sure either whether this issue of mine belongs here, as seemingly it is an issue with my installation athe than with the SDK. Please, if I should seek help somewhere else, do tell me.
Thank you for your assistance in any case.

It looks like you might have both x86 and x64 versions of .NET installed, which can cause what you're seeing. I'd suggest taking a look at where dotnet and adjusting PATH if necessary. Check out these comments for more info:
https://github.com/dotnet/core/issues/8275#issuecomment-1466867133
https://github.com/dotnet/core/issues/5962#issuecomment-780084535

I have the same problem, but the where dotnet doesn't give an output.

On Windows you have to run where.exe dotnet

For everyone still stuck, I had the Runtime and the SDK in different folders, you have to have them in the same folder.
+---host
|	\---fxr
|		\---7.0.11
+---metadata
|	\---workloads
|		\---7.0.400
|			\---installertype
+---packs
|	+---Microsoft.AspNetCore.App.Ref
|	|
|	+---Microsoft.NETCore.App.Host.win-arm
|	|
|	+---Microsoft.NETCore.App.Host.win-arm64
|	|
|	+---Microsoft.NETCore.App.Host.win-x64
|	|
|	+---Microsoft.NETCore.App.Host.win-x86
|	|
|	+---Microsoft.NETCore.App.Ref
|	|
|	+---Microsoft.WindowsDesktop.App.Ref
|	|
|	\---NETStandard.Library.Ref
|
+---sdk
|	\---7.0.401
|
+---sdk-manifests
|
+---dotnet.exe
|
+---shared
|	+---Microsoft.AspNetCore.App
|	|	\---7.0.11
|	\---Microsoft.NETCore.App
|		\---7.0.11
+---swidtag
\---templates
	\---7.0.11

I was able to temporarily fix this, it was an issue with the path.
After running 'where.exe dotnet', I copied the path to my dotnet.exe and use it for running any command like so..
"C:\Program Files\dotnet\dotnet.exe" new webapp -o MyWebApp --no-https -f net7.0

Hi, I also encountered this problem. In my case what helped was installing the SDK again, because the version that the site downloaded automatically was x64 and I needed x86.

Olá, também encontrei esse problema. No meu caso o que ajudou foi instalar o SDK novamente, pois a versão que o site baixou automaticamente era x64 e eu precisava de x86.

Cara, eu amo você. Escrevi no terminal dotnet --info para ver os sdks e não aparecia nada, depois que li seu comentário percebi algo:
Host:
Version:      8.0.1
Architecture: x86 <-------- ESSA PORRA
Commit:       bf5e279d92
Estava baixando o x64 tbm kakakaka

The site should actually be referring you to the proper arch based on your machine information. To be certain, you should go to the about your PC and confirm the processor type of your machine.
However, many people encounter issues with their PATH. Check out these instructions:
https://github.com/dotnet/core/issues/5962#issuecomment-780084535
If you really have an x86 machine and the website is showing x64, can you please log an issue at https://github.com/dotnet/website-feedback?
@isaqueMarinh0 para você também. Por favor confirme se a sua máquina é realmente x86 ou se o dotnet está invocando x86 por cause de um problema no PATH.
Por favor confirme nas configurações do Windows se sua máquina é realmente x86. Aquela informação de arquitetura apenas indica qual versão do dotnet está sendo invocada.

It looks like you might have both x86 and x64 versions of .NET installed, which can cause what you're seeing. I'd suggest taking a look at where dotnet and adjusting PATH if necessary. Check out these comments for more info: #5962 (comment) #5962 (comment)

Thanks mate, this worked for me.
I was trying to fix the issue for past eight hours 😭


Validator view filters on Node ID

The validators page has a filter - the placeholder specifies address whereas node id seems more appropriate

Thanks. Turns out the search always worked with node id as opposed to address. That was a typo.
https://github.com/ava-labs/ava-explorer/pull/4 should reflect your intended change along with a slew of other performance issues. Will close this PR.


[New game]: Add a tilting maze game (3D)

🎮 Game Request
Game logic and basic description. A tilting maze game is an interactive game where the player navigates a ball through a maze by tilting the game board. The goal is to guide the ball from the starting point to the destination while avoiding obstacles and holes.
Point down the features

Maze Generation
Tilt Controls
Ball Movement
Scoring
Levels

Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @abhinav-m22 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @abhinav-m22 👋!
@kunjgit here,
We have this annoucement to make 📢!!
We have updated our README 🚀
Make sure you follow proper guideline to raise your next issues !!
Make sure you join our Discord🕹 if you haven't 😇
Thank you 💗!

Hello @abhinav-m22, Time's Uppp!⏰
Sorry for closing your issue!
But it's more than a week since we haven't received anything from your side 😢 .
Come up with new ideas, create a new issue and make sure you finish it within a week! 🔥
All the best! 🚀
Happy Hacking! 💗

Hey @abhinav-m22 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Msbemba patch 1

added verbatim to step5 and changed the screenshot to reflect the new steps

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@LolaJacobsen
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@msbemba I think these changes have already been made in the article. Check it out and let me know if you disagree.
https://docs.microsoft.com/microsoftteams/guest-access-checklist
Thank you for all the good work you do on our documentation!


HttpRequest: move http cache to RequestRepeater

Would be nice to start writing tests for this kind of stuff. These small UI PRs take a lot of effort to review to the point where it's almost not worth it for these incremental refactors.

ok


Update oakpal version to allow guava variation vis. OAK-7182

@justinedelson oakpal 1.1.12 released with oak 1.8.9 dependency

The confirmation build technically passed, it was that email health check that borked on one of the sub-builds.  I restarted it so we have a clean sign-off.


iguana support in build.zig

NOTE: this does not currently work because iguana does not compile on latest zig (inline functions got changed to calling convention inline). We should wait to merge this until it does.

My PR fixes that here: https://github.com/alexnask/iguanaTLS/pull/9
However, even with that fix, the tests aren't passing with iguana yet.  But that's fine since we're not breaking anything yet.


fix possible security issues, avoid warnings from other tools

fix possible security issues, avoid warnings from other tools although it is not necessary here

also can replace regex pattern with ^lib/([^/\.]+)/([^/]+\.so)$ to avoid matching lib/../xx.so


Require-match does not implement confirm and confirm-after-completion

REQUIRE-MATCH can take the following values:
- t means that the user is not allowed to exit unless the input is (or
  completes to) an element of COLLECTION or is null.
- nil means that the user can exit with any input.
- confirm means that the user can exit with any input, but she needs
  to confirm her choice if the input is not an element of COLLECTION.
- confirm-after-completion means that the user can exit with any
  input, but she needs to confirm her choice if she called
  minibuffer-complete right before minibuffer-complete-and-exit
  and the input is not an element of COLLECTION.
- anything else behaves like t except that typing RET does not exit if it
  does non-null completion.

I looked into this and the other values seem to make most sense for the default UI. We already don't allow exit with the prompt when the value is t and in the other cases you either have to intentionally select the prompt or it is visually indicated that is selected so the RET can be seen as the confirmation already. With default completion that is not the case as you don't have a selection there. Maybe we should keep things like they are in this case?

I didn't really consider it makes sense to do anything here. I just thought about reporting this when I saw the completing-read documentation.
After reading the documentation again I think the Selectrum behavior is insufficient. If I am using require-match=t, I cannot return the input when it does not match a candidate. But with 'confirm option this should be possible? Now if I am using require-match=nil I can return an arbitrary value, but having only selected the prompt is not sufficient confirmation. There should be a second step.

If one uses 'confirm, Selectrum behaves like nil, right? Then I guess it is okay, but I am missing a bit the second confirmation step. Maybe it is unnecessary. Hmm.

Yes, we only block the t all other ones are treated like nil.

With default completion, when doing (consult--read '(a b c) :require-match 'confirm) I have to press RET twice. This is different from Selectrum, there I only have to press RET once.

With default completion you have no idea if RET will submit of complete so it makes more sense to have a separate step to avoid accidental submission. With Selectrum RET will always submit the selected candidate so accidental submission can't happen, I think an additional safe guard would probably more annoying than it helps.

But reading the docs again we should also treat any non-nil value but confirm and confirm-after-completion as t.

With default completion, when doing (consult--read '(a b c) :require-match 'confirm) I have to press RET twice. This is different from Selectrum, there I only have to press RET once.

Agree.

With Selectrum RET will always submit the selected candidate so this kind of accidental submission can't happen, I think an additional safe guard would probably be more annoying than it helps.

I am not sure. I wonder which commands actually use this feature. But let's assume that it is used in a german air traffic control system of the 90s, then I would probably appreciate the annoying double RET ;)

On Mon, Mar 22 2021, Daniel Mendler wrote:

After reading the documentation again I think the Selectrum behavior
is insufficient. If I am using require-match=t, I cannot return the
input when it does not match a candidate. But with 'confirm option
this should be possible? Now if I am using require-match=nil I can
return an arbitrary value, but having only selected the prompt is not
sufficient confirmation. There should be a second step.

[Apologies in advance if this is not related with what you're discussing
here, but it sounds as it might]
Today, i was playing with the new quick-keys in the context of a
consult-lines invocation, with orderless in my completion-styles.  With
that configuration, the content of a line might not match itself (e.g.,
the line has a portion that looks like a regexp that doesn't match the
line); so, when you select the line you want with a quick key and
therefore move its contents to selectrum's prompt, you cannot select it
(i.e., jump to it by pressing RET) because require-match is t.  A quick
workaround is to add orderess-literal to orderless-matching-styles, but
i am not sure i want that everywhere (and it's not a default macthing
style in orderless).

@jaor I don't see this behavior. If a line matches a candidate exactly it is selected as far as I know. Could you give a more precise example?

@jaor On another note why do you insert the line first and not directly select it using M-m?

On Mon, Mar 22 2021, Daniel Mendler wrote:

@jaor I don't see this behavior. If a line matches a candidate exactly
it is selected as far as I know. Could you give a more precise
example?

Go to any buffer, e.g. scratch

Insert these lines:
line a [1 xd]
line b [2 bc]
line x [3 bd]

M-x consult-line

Narrow the candidates by typing, say line

Now M-i will assign to line 3 the character d

My expectation is that pressing d i'll jump to the third
line. Instead, line x [3 bd] is inserted at my prompt

That value doesn't match any line in the buffer (because orderless is
matching using either a regexp or initialisms), so pressing RET or
C-j won't jump to the desired line (one gets an error, Match required,
instead).

Isn't that what you see?

 I just assumed blindly that you had problems when using the quick select (and not the quick insert). I think for the quick insert the issue you are observing is expected and should be solved at the level of the completion style (orderless?), if it should even be solved at all.

On Mon, Mar 22 2021, Daniel Mendler wrote:

@jaor I see what you see. I just assumed blindly that you had problems
when using the quick select (and not the quick insert).

Right.  I was just oblivious of the existence of quick select, and
thought i had to go through insert.

I think for the quick insert the issue you are observing is expected
and should be solved at the level of the completion style
(orderless?), if it should even be solved at all.

Agreed.  Maybe @oantolin could consider adding orderless-literal to the
default value of orderless-matching-styles, to minimize surprises for
newcomers.  I am trying to imagine a case where i would want an exact
duplicate of what i'm completing for not to match it, and finding only
rather esoteric use cases.

I thought the consult-async commands meant we were all comfortable with the idea that completion candidates can't be relied on to match themselves! 😛

Agreed.  Maybe @oantolin could consider adding orderless-literal to the
default value of orderless-matching-styles, to minimize surprises for
newcomers.  I am trying to imagine a case where i would want an exact
duplicate of what i'm completing for not to match it, and finding only
rather esoteric use cases.

@jaor I am not sure. Even if you add literal to the defaults, Orderless still splits the input string at spaces and possibly matches more than the exact candidate you are interested in. So I think it is justified to just keep things as is.
If you want to force the exact matching you can combine orderless with substring:
(setq completion-styles '(substring orderless))

But this has other bad effects, because substring is used first and then you don't get initialism matching directly or whatever you have configured in orderless.

I thought the consult-async commands meant we were all comfortable with the idea that completion candidates can't be relied on to match themselves! stuck_out_tongue

@oantolin I agree with this :)

On Mon, Mar 22 2021, Daniel Mendler wrote:

@jaor I am not sure. Even if you add literal to the defaults,
Orderless still splits the input string at spaces and possibly matches
more than the exact candidate you are interested in. So I think it is
justified to just keep things as is.

I won't insist, but, at first blush, seeing more than one candidate is to
me less awkward than inserting the exact line i'm looking for and being
told that it doesn't exist!

@jaor I agree that it you it would be good to fix this, though it is an edge case which isn't that important but it would be good to have open a separate issue for it to keep track of it.

This issue got fixed by #510.

I agree that it you it would be good to fix this, though it is an edge case which isn't that important but it would be good to have open a separate issue for it to keep track of it.

I tried to "fix this" by setting (setq orderless-matching-styles '(orderless-regexp orderless-literal orderless-initialism)) but this does not work. However (setq orderless-matching-styles '(orderless-literal orderless-initialism)) gives the desired result at the great cost of not allowing regexps by default anymore. @oantolin Is literal somehow incompatible with regexp?

That's weird, adding orderless-literal anywhere in the orderless-matching-styles list should definitely work (unless, of course you have dispatchers that do something else). It also does seem to work here. Do you have a test case where it didn't work, @minad?

Maybe my problem was due to dispatchers. I have to investigate this better.

Just for the record, the mystery in the last couple of comments was elucidated over at oantolin/orderless#42.

On Tue, Mar 23 2021, Omar Antolín Camarena wrote:

That's weird, adding orderless-literal anywhere in the
orderless-matching-styles list should definitely work

it does indeed work for me.

@clemera
This commit also seems to have changed how find-file works. Now I have to confirm before making a new-file using the find-file interface? I'm not sure whether there's an option to disable this although I'd really appreciate one.
Reproduction instructions:

find-file
Enter and select the name of a file that doesn't exist.
selectrum asks to confirm before accepting it.

@clemera @mohkale I cannot reproduce this. I don't have to confirm for new files?!

The behavior of find-file depends on the Emacs option confirm-nonexistent-file-or-buffer. Generally there is not equivalent for confirm-after-completion in Selectrum, I decided to treat it the same as confirm but looking at this it seems it might be better to not do that and treat it like nil instead?

@clemera Thank you for letting me know about this option. I think treating confirm-after-completion like confirm is correct in the context of Selectrum. Confirm means confirm, so it should happen at some point (Remember the air traffic control?).
And just recently - maybe they had their steering system running on Emacs?

Maybe I should also respect the setting in consult-buffer?

@clemera What is weird. On my Emacs confirm-nonexistent-file-or-buffer is set to after-completion, but the documentation of completing-read mentions confirm-after-completion. Do both symbols mean the same? Do we treat only one as confirmation?

:laughing: Yes, who knows what else runs on Emacs. The symbol after-completion is treated like confirm-after-completion in this context, see the function confirm-nonexistent-file-or-buffer.

I think treating confirm-after-completion like confirm is correct in the context of Selectrum. Confirm means confirm, so it should happen at some point (Remember the air traffic control?).

I thought the same but when using find-file with confirm-after-completion/after-completion in emacs -Q you can type a non existent file name and don't need to confirm, the behavior for Selectrum is different in this case.

@clemera Great. I think I was running on and old version of Selectrum. Now it works and asks me to confirm. Regarding the treatment of after-completion, I don't really understand what this is supposed to do. I think in Selectrum it is not bad like this, asking for an additional confirmation (and one can always turn this off). But on both default completion and Icomplete setting after-completion for new files does nothing.

The option also does not make sense to me, I don't see why pressing TAB before makes confirmation more desirable. As there is no equivalent to trigger completion help with TAB and to match the default behavior it is probably okay to ignore it as you say you can adjust confirm-nonexistent-file-or-buffer to always get the confirmation.

@clemera
Thanks unsetting confirm-nonexistent-file-or-buffer fixed the issue for me.

@mohkale Note you don't need to unset if after a7130ca. Only need to set it to confirm if you want the confirmation.


[BUG] Add-PnPPageTextPart remove CR/LF in multiline text variables

Notice
Many bugs reported are actually related to the PnP Framework which is used behind the scenes. Consider carefully where to report an issue:

Are you using Invoke-PnPSiteTemplate or Get-PnPSiteTemplate? The issue is most likely related to the Provisioning Engine. The Provisioning engine is not located in the PowerShell repo. Please report the issue here: https://github.com/pnp/pnpframework/issues.
Is the issue related to the cmdlet itself, its parameters, the syntax, or do you suspect it is the code of the cmdlet that is causing the issue? Then please continue reporting the issue in this repo.
If you think that the functionality might be related to the underlying libraries that the cmdlet is calling (We realize that might be difficult to determine), please first double check the code of the cmdlet, which can be found here: https://github.com/pnp/powershell/tree/master/src/Commands. If related to the cmdlet, continue reporting the issue here, otherwise report the issue at https://github.com/pnp/pnpframework/issues

Reporting an Issue or Missing Feature
When using in powershell Add-PnPPageTextPart -Page $myPage -Text $mymultilinevariable, the webpart is correctly created but the multiline variable becomes a single line variable
Expected behavior
That a multiline variable remains multiline in the text part.
Actual behavior
the text part remove all CR/LF, so the multiline variable becomes a single line variable
Steps to reproduce behavior
see above
What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)
ModuleType Version    Name                                ExportedCommands

Manifest   1.11.126   PnP.PowerShell                      {Add-PnPAdaptiveScopeProperty, Add-PnPPropertyBagValue, Add-PnPSiteClassifica...
Manifest   1.10.55    PnP.PowerShell                      {Add-PnPAdaptiveScopeProperty, Add-PnPPropertyBagValue, Copy-PnPFolder, Expor...
Manifest   1.10.47    PnP.PowerShell                      {Add-PnPAdaptiveScopeProperty, Add-PnPPropertyBagValue, Copy-PnPFolder, Expor...
Manifest   1.7.15     PnP.PowerShell                      {Add-PnPClientSidePage, Add-PnPClientSidePageSection, Add-PnPClientSideText, ...
Directory: C:\Program Files\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands

Manifest   1.3.15     PnP.PowerShell                      {Add-PnPClientSidePage, Add-PnPClientSidePageSection, Add-PnPClientSideText, ...
Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

What do you mean by multiline variable ?? Not following the issue here, can you please elaborate.
The input that is required is a string for the text webpart.

Hi
When you have a powershell string, it can be in 1 line or in multiple lines. It is still a string.
For example, create a string :
$test = “myfirst line ()

mysecond line”

The result of $test will be :
Myfirstline
Mysecondline
But when I used it into “Add-PnPPageTextWebPart”, the result is :
Myfirstline Mysecondline
It should be possible that a text in a text webpart spread on multiple lines, doesn’t it ?
CS
From: Gautam Sheth @.>
Sent: 15 November 2022 21:05
To: pnp/powershell @.>
Cc: Christophe Sarteel-EXT @.>; Author @.>
Subject: Re: [pnp/powershell] [BUG] Add-PnPPageTextPart remove CR/LF in multiline text variables (Issue #2544)
EXTERNAL SENDER: Do not click links or open attachments unless you trust the sender and know the content is safe.
What do you mean by multiline variable ?? Not following the issue here, can you please elaborate.
The input that is required is a string for the text webpart.
—
Reply to this email directly, view it on GitHubhttps://github.com/pnp/powershell/issues/2544#issuecomment-1315805018, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4DS5ZSWZCFZTYQJ76OWD33WIPUHTANCNFSM6AAAAAAR4OYFXE.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>"The information contained in this message is intended for the addressee only and may contain confidential and/or privileged information and/or information protected by intellectual property rights.
If you are not the addressee, please delete this message and notify the sender; you should not use, alter, copy or distribute this message or disclose its contents to anyone.
Email transmission cannot be guaranteed to be secure or error free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses.
No responsibility is accepted by S.A.B.C.A. for any loss or damage arising in any way from its use.
Any views or opinions expressed in this message are those of the author and do not necessarily represent those of S.A.B.C.A. or any of its affiliates.
Therefore this email does not constitute a commitment by S.A.B.C.A. unless it contains an express statement to the contrary from an authorised representative.
S.A.B.C.A. - RPM Bruxelles TVA BE 0405.770.992 -"

hi @CSARTSABCA , the server-side API I think trims out the blank empty spaces.
You need to specify the new line like you would do that in HTML.
Usually, we can use

 or
 to specify text in new lines. You can use that.

Thanks, it is working with a  tag.
From: Gautam Sheth @.>
Sent: 27 December 2022 21:27
To: pnp/powershell @.>
Cc: Christophe Sarteel-EXT @.>; Mention @.>
Subject: Re: [pnp/powershell] [BUG] Add-PnPPageTextPart remove CR/LF in multiline text variables (Issue #2544)
EXTERNAL SENDER: Do not click links or open attachments unless you trust the sender and know the content is safe.
hi @CSARTSABCAhttps://github.com/CSARTSABCA , the server-side API I think trims out the blank empty spaces.
You need to specify the new line like you would do that in HTML.
Usually, we can use  or  to specify text in new lines. You can use that.
—
Reply to this email directly, view it on GitHubhttps://github.com/pnp/powershell/issues/2544#issuecomment-1366158896, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4DS5ZT4H7PKR7TFL45S6ZDWPNGJDANCNFSM6AAAAAAR4OYFXE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>"The information contained in this message is intended for the addressee only and may contain confidential and/or privileged information and/or information protected by intellectual property rights.
If you are not the addressee, please delete this message and notify the sender; you should not use, alter, copy or distribute this message or disclose its contents to anyone.
Email transmission cannot be guaranteed to be secure or error free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or contain viruses.
No responsibility is accepted by S.A.B.C.A. for any loss or damage arising in any way from its use.
Any views or opinions expressed in this message are those of the author and do not necessarily represent those of S.A.B.C.A. or any of its affiliates.
Therefore this email does not constitute a commitment by S.A.B.C.A. unless it contains an express statement to the contrary from an authorised representative.
S.A.B.C.A. - RPM Bruxelles TVA BE 0405.770.992 -"


Allow using ssh-key to push version while using token to publish to hvcs

Scenario
In GH Actions I want to use a ssh Deploy Key to push changes to a protected branch because not every user's GITHUB_TOKEN is allowed to push to protected branches but Deploy Keys are.
Problematic
If I specify a GH_TOKEN such as GITHUB_TOKEN in a GitHub Action with enough permission to publish a GitHub Release, python-semantic-releases will also use that user's token to push version and tag changes to the repository.
The user running the action though might not have enough permission to push to a protected branch even if grating some degree of permission to its GITHUB_TOKEN. Only Deploy Keys or Admin user's GITHUB_TOKEN(or Personal Access Tokens) can.
Solution
Add an option that will allow to use the token to submit GitHub Releases while will avoid using that token for pushing changes to the repository, relying on the underlying git authentication mechanism. In the case of GitHub Actions it can be combined with ssh-agent action to allow using Deploy Keys for this purpose.

Anyone ?

Looks like some tests need to be fixed.

I'll look into it, thanks @danth

Sorry for the late fix @danth. I've figured out the problem which had to do with the GITHUB_ACTOR env var set by the GH runners altering the test outcome(thus why locally the tests were ok)

One of the tests is still broken 🤔 Perhaps there is still some interference from the test environment.

Hope this one works. I explicitly added the GITHUB_ACTOR env var to my test execution to emulate the runner behaviour


Add supported version to fish completion

I added the supported kubectl version to the fish completion page. Spent some time trying to figure out why it wouldn't work when I copied it directly from the documentation. Turns out Docker Desktop had installed an older version of kubectl. Trying to save others some time.
Reference: https://github.com/kubernetes/kubectl/issues/1189

@themarquee 👋🏻 Please update this PR to use the feature state shortcode. Follow the link for an example. Thanks in advance!


Docs/update readme

I've updated the README and index page of the Snowflake documentation.
Notable mentions:

I removed mentions of SnowflakePlots. Once, it has moved to its own repository (#148) we should definitely reference it.
I've replaced the four typical steps of a quantum application with the three typical steps of submitting a circuit to a quantum service. The four steps did not seem typical at all. (Shor's algorithm, QAQO, VQE are examples which do not follow these steps)
The centring of the CNOT circuit image does not render on GitHub. I'll still try to fix this.

@plerouxanyon There is no info on transpilation on the readme file. I think we should add one example.