Determine newsletter signup tool

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/478
Originally created by @derek-knox on 2019-03-14 17:15:26

The below TODO was extracted from the docs checklist issue #467
Context: https://gitlab.com/meltano/meltano/issues/467#note_150546993

[ ] “Sign Up” or “Get Started” CTA vs solely "Read the Docs"

[ ] Add button
[ ] Add email collection (simple form)

View 4 previous comments from the original issue on GitLab


fix: disable the dollarSyntax linter by default

And move the linter to the Style namespace.

Follow-up to #15909, taking into account #15616.

Thanks!
maintainer merge

This looks spurious
bors r-
bors r+
bors d+


Allows unknown configuration variables when specified on the commandline

It will still generate an error, but will not stop configuration from continuing.
This makes conf_remap more consistent with itself and records.config

Cherry-picked to 8.0.x


arch: arm: add synchronization point after Stack Pointer switch

This commit adds a Context Synchronization Point immediately
after the reset handler switches to use the Process Stack
Pointer, during initialization.
Signed-off-by: Ioannis Glaropoulos<EMAIL_ADDRESS>

This PR implements the ARM programming rule of always adding a synchronization point after switching Stack Pointers.

CC @galak @GAnthony.
Related to #7128 (but not necessarily fixing the issues reported there)

It appears the safe thing to do; but note: the ARM CMSIS macros, used in MCUBoot and Zephyr,  _set_PSP(), _set_MSP(), also don't do an isb (or set the CONTROL register) after the msr instruction.

@agross-linaro the synchronization barrier is not needed with the stack pointer registers. The following text from ARMv8-M specification is descriptive:

Except for writes to the CONTROL register, any change to a special-purpose register by a CPS or MSR instruction is
guaranteed:
• Not to affect that CPS or MSR instruction, or any instruction preceding it in program order.
• To be visible to all instructions that appear in program order after the CPS or MSR.

So, it does not seem we need the barriers on MSR PSP instructions. Actually, this makes sense to me, because if an MSR updates the stack pointer, and the stack pointer is needed in following instructions, then the PE needs to guarantee that the write must complete (i.e. be "visible") before "directly" using the SP.
Writing to the CONTROL register, however, does not affect the SP contents, but, rather affects which SP is used, so the PE does not need to complete the outstanding MSR CONTROL, before pushing to stack using SP. That's why the barrier is necessary here.
Also: the following specification text (regarding the necessity of synch barrier) only exists for the CONTROL register.

The architecture requires a Context synchronization event to guarantee visibility of a change to the CONTROL register.

Strictly speaking, the ISB is only needed when  changing the SP. I.e. for changing PRIV mode, the ISB is not mandatory. The following ARM Note in the spec
[E.g. The Cortex‑M33 Processor > Programmer's model > Core registers > CONTROL Register] is  quite descriptive:

Note:
When changing the stack pointer, software must use an ISB instruction immediately after the MSR instruction. This ensures that instructions after the ISB instruction execute using the new stack pointer.

BTW, that is the only place in the ARM code that the CONTROL register is used to switch from MSP to PSP.


move gsutil copy to condtional to avoid breakages

gsutil is not in the requirements list, hence not installed by default. The copy command is breaking for the containers which don't have it by default. Moved it under a conditonal

Isn't it better to error out when a user requests profiling but the profiling fails? User can set profiler='' (default, no profiling) to avoid profiling instead of this check?
This error points to a real bug in maxtext - we should either ensure gsutil is installed or perhaps can use a different gcs API (upload_blob)

Hi @gobbleturk the profiling works fine as is. This line just copies the generated .nsys-rep file to some directory. Without gsutil this copy fails and the run breaks. If for some reason the profiling breaks, it should break even with this PR. In sum, this PR does not ignore profiling failures, rather it avoids the breakages caused by gsutil file copy.

Hi @gobbleturk the profiling works fine as is. This line just copies the generated .nsys-rep file to some directory. Without gsutil this copy fails and the run breaks. If for some reason the profiling breaks, it should break even with this PR. In sum, this PR does not ignore profiling failures, rather it avoids the breakages caused by gsutil file copy.

Don't we need to move the profile to GCS? How are we able to use the profile otherwise?

@gobbleturk yes, on clusters using gcs, it is needed indeed. Hence, the nightly maxtext container made through this script installs gsutil etc. However, for other clusters it may not be necessary as there could be different file system mounts. Without any checks, it leads to breakages.
Another way could be to enforce the gsutil to be on the requirements list. Even with that, we might need to change the behavior for non-gcs scenarios.

script

Ya this makes sense. For TPUs the profile will be sent to base_output_directory which is configurable (can be GCS or local), I think we should change the GPU path to also be configurable, as it won't always be GCS.

Will approve for now to unblock nsys profiling use cases, as we work on longer term fix

Created https://github.com/AI-Hypercomputer/maxtext/issues/911 to track longer term fix


Build updates

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

The previous build seemed to suffer from a timing problem where the Linux builds succeeded in the PR, but failed after merge due to a change in the $MESON_ARGS variable. While we're at it, there are a few things about the build to update for the future.

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/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-recipe-manager. Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12225958331. Examine the logs at this URL for more detail.

NB, this appears to have the same glib-mkenums issue on Windows as conda-forge/harfbuzz-feedstock#118.

@conda-forge/pango This is ready to go. In case I wasn't clear above, the latest new version never made it to Linux due to some unluckily-timed build failures, and I feel like it's worth getting this all fixed up before the next stable release comes out.


Add pytests for functions that need to connect to Snowflake

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/1611
Originally created by @iroussos on 2020-01-27 14:38:43

Add pytests for functions that need to connect to Snowflake.. { more details to come }

View 7 previous comments from the original issue on GitLab


logical expression logic in sub-expressions.

can.stache has strange rules for scope lookup.  This was inherited from can.Mustache.  Currently, lookup changes behavior if an expression has arguments.  If an argument is present like {{key arg}}, helper methods will be looked within the helperOptions and then the global helpers.  If nothing is found, then a method on the current context will be explored.  If still nothing is found, then the scope will be looked within.
Also, functions are called with default arguments.
I would like to simplify this in subexpressions for 2.3.  This way if anyone wants "logical" expression lookup, they can wrap their expression in parenthesis like:
{{(foo bar)}}

In 3.0, we can make this the default behavior.
Here's what I propose are the lookup behavior for something like:
(key arg)

key lookup looks in "scope" first.  If key is a function, it will be called with arg as a "simple" value (not a compute).  If key maps to an attribute, the attribute value will be returned (arg is effectively ignored).
If no method or property matching key is read-able, then helpers will be looked within.

Cases
#each ( (game.statsForPlayerId player.id).filter type)
#each gameStatsForPlayerId(player.id).filter(type)

#1699 should be taken into account.


Calendar for Diocese of Shrewsbury

Is it possible to add a Diocesan Calendar for the Diocese of Shrewsbury (https://www.dioceseofshrewsbury.org/)?
While I am eager to contribute and assist in any way I can, I'm hampered by my limited understanding of how the liturgical calendar works. I've contacted the Diocesan Liturgy Office for assistance, so I can request any required information from them, but I'm still not sure what information I need, and what I then need to do with it.
I really appreciate that you built this project, it's a fantastic resource.
Any help you could give me in implementing this change would be much appreciated

These pages from the 2023 calendar are useful for reference.

Hello George, thank you for taking interest in this project. I will look into it when I have time. I believe there may be a few differences in the way the English calendar handles festivity coincidences, so I may need to implement logic to account for that.


⚠ Bofrost.be has degraded performance

In 8c23e87, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 819 ms

Resolved: Bofrost.be performance has improved in b75185f after 2 hours, 37 minutes.


rename to storybook-addon-backgrounds

Hey there!
Maintainer of storybook here, I find this addon pretty cool, perhaps you'd like it to become part of the core some day?
I'm building a mono-repo which will allow us to to better releases, and I'd welcome your team and repo!
I was hoping you'd be interested and possibly rename the package to storybook-addon-backgrounds. Perhaps one day we could autoload addons which are installed, and a common naming scheme would help with that 👍 .

Hi do you need help maintaining?


[MM-23412] - Align mention and close icon horizontally

Summary
Aligns the close and mention icon horizontally
Ticket Link
https://mattermost.atlassian.net/browse/MM-23412
Fix 5.22.0

/update-branch

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-5127.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Test server destroyed


Support Basis transcoding on iTwin.js frontend

Implement support for transcoding universal super-compressed Basis textures on the frontend: https://github.com/BinomialLLC/basis_universal
This will be important when Cesium tiles offer Basis textures as an option.
We also propose that connectors for iModels should convert to Basis textures, which will also make this important.
WIP : https://github.com/iTwin/itwinjs-core/compare/basis-support-frontend

BasisLoader.ts does this:
async function _fetchBasisTranscoder(): Promise<{js: string, wasm: Uint8Array}> {
  const { basisTranscoderScript, basisTranscoderWasm } = await import("./BasisTranscoder");
  return { js: basisTranscoderScript, wasm: Base64.toUint8Array(basisTranscoderWasm) };
}

This dynamically loads the large .js file string and the large .wasm binary array.
However, it only loads this dynamically if the following is set in tsconfig.json:
{
  // ...
  "compilerOptions": {
    "module": "es2020",
    "moduleResolution": "node",
  },
}


mouse control for rotation in the z-axis

asked on https://gitter.im/nglviewer/Lobby

I can add a mouse action for that


📝 docs: Add docs about integrating into your NodeJS application

In this PR, I added documentation about how you can utilise babel to integrate React Email into your NodeJS application. I also updated the imports to match the export default props

@bastiaanv Sorry for the delay, but thanks for the PR ❤


Hyprshade toggles on hyprland config reload

I'm` using hyprshade along with it's systemd unit to apply blue-light-filter automatically. It's working fine normally but the shader gets deactivated whenever hyprland config is reloaded such as making changes to it and saving it.
I'm using this line in hyprland.conf:
# Shader Management
exec-once = hyprshade auto

And this is my hyprshade.toml:
[[shades]]
name = "vibrance"
default = true  # shader to use during times when there is no other shader scheduled

[[shades]]
name = "blue-light-filter"
start_time = 18:00:00
end_time = 06:00:00

NVM, i was being dumb. It was as simple as using exec instead of exec-once.


Preserve commutative law in additive expression and multiplicative experssion

Description:
The current implementation does not preserve commutative for additive expression and multiplicative expression. Consider below samples.

int a = int:MAX_VALUE - 1 + 1; //<PHONE_NUMBER>854775807
int b = int:MAX_VALUE + 1 - 1; // error: {ballerina}NumberOverflow {"message":"int range overflow"} at test:main(test.bal:12

float x1 = 92233720e300f;
float x2 = 10;
float y1 = 0.1 * x1 * x2; // 9.223372E307

float y2 = x1 * x2 * 0.1; // Infinity

int x1 =<PHONE_NUMBER>854775807;
int x2 = 2;
float y1 = 0.1 * x1 * x2; // 1.84467440737095526E18

float y2 = x1 * x2 * 0.1; // error: {ballerina}NumberOverflow {"message":"int range overflow"} at test2:main(test2.bal:8)

We need to think about all the possible cases and handle this.

Closing since, (IEEE floating point values are not real numbers and do not follow the laws that apply to real numbers in mathematics). https://github.com/ballerina-platform/ballerina-spec/issues/261#issuecomment-1092387744 Also, according to associativity, in the first sample we should resolve int:MAX_VALUE + 1 first, since it is an overflow its fair to panic at that time.


Implementation of Wallet and Provider

Everything seems to be fine 🙂. A few things to notice:

Ethereum signer should consist of an interface and an implementation.

Done. Separated into protocol EthSigner and implementation class DefaultEthSigner

Everything seems to be fine 🙂. A few things to notice:

Ethereum signer should consist of an interface and an implementation.

Done. Separated into protocol EthSigner and implementation class DefaultEthSigner


Project WOA Deployer: Error Dism exited with code 2

hi, im trying installing windows 10 desktop on my raspberry pi 3b+ and everytime i try to deploy it after the deployment exit this error: There has been a problem during deployment: DISM exited with code 2. im leaving u the log of woa deployer. help me pls i dont know what to do :(
Log-20211107.txt
r

i had same error

cc @SuperJMN


[Packaging] Install on Ubuntu 23.10

Describe the enhancement requested
Hi
I want to upgrade my machine to Ubuntu 23.10, released a month ago. But in the repository is there only support for 23.04?
There will be support for 23.10 in the future? or I will have to wait until 24.04? Is there a way to install the 23.04 version on 23.10? Is it possible?
Component(s)
Release

We haven't updated https://arrow.apache.org/install/ for 14.0.0 yet but the instruction will work on 23.10. :-)

Ah, sorry. It will not work. We don't have packages for 23.10 yet. We only have 23.04 packages.

Wow!!! that was fast!!! Thanks for this. Arrow is essential for a project I am working on, but I want to migrate to the new Ubuntu.
Also, 23.03 support ends in January!!! So for me is not an option to migrate
Thanks


L2 Update: New ELSE Statement in J.3 Section of LP5

The clinicians realized that we need to provide a care recommendation for individuals who's high-grade event occurred > 25 years ago and they have passed the 'initial intensive surveillance period'. Prior to this update, clinicians caring for these women would not have been presented with the following: Continued surveillance at 3-year intervals beyond 25 years is acceptable for as long as the patient's life expectancy and ability to be screened are not significantly compromised by serious health issues.
See the update in BOLD text.
2._ After the initial intensive surveillance period, continued surveillance at 3-year intervals is recommended for at least 25 years after treatment of high-grade histology (histologic HSIL, CIN 2, CIN 3, or AIS) or high-grade cytology (HSIL, AGC, or persistent ASC-H) even if this is beyond the age of 65 years (BII)._
IF MOST RECENT ‘treatment for high-grade histology or cytology’ <=25 years ago
AND ‘initial intensive surveillance period’
DISPLAY In patients treated for histologic or cytologic HSIL, after the initial intensive surveillance period, continued surveillance at 3-year intervals is recommended for at least 25 years after treatment of high-grade histology (histologic HSIL, CIN 2, CIN 3, or AIS) or high-grade cytology (HSIL, AGC, or persistent ASC-H) even if this is beyond the age of 65 years (BII). When patients with a history of treated high-grade histology or cytology reach the age of 65 years, if they have completed the initial 25-year surveillance period, continued surveillance at 3-year intervals is acceptable and may continue as long as the patient is in reasonably good health (BIII). Discontinuation of screening is recommended if a patient has a limited life expectancy. Management according to the highest-grade abnormality found on histology or cytology is recommended.
**ELSE
IF MOST RECENT ‘treatment for high-grade histology or cytology’ > 25 years ago
AND ‘initial intensive surveillance period’
DISPLAY: Continued surveillance with HPV testing or cotesting at 3-year intervals for at least 25 years is recommended after treatment and initial post-treatment management of histologic HSIL, CIN 2, CIN 3, or AIS. Continued surveillance at 3-year intervals beyond 25 years is acceptable for as long as the patient's life expectancy and ability to be screened are not significantly compromised by serious health issues.**

My guess would be that this situation would be rare, since it would involve needing to identify a treatment procedure and the abnormal test results that led to the treatment both from >25 years ago. But definitely makes sense to add in order to be comprehensive!

I made the change in this commit: https://github.com/ccsm-cds-tools/ccsm-cds-with-tests/commit/4654925d36003b37c6d5dc5c5cbef75ac6f12ad3.
In general I will make any changes that arise from the Issues you just submitted in the l2-updates-april-may branch of this repository (https://github.com/ccsm-cds-tools/ccsm-cds-with-tests/commits/l2-updates-april-may). This way we can review them all together as one larger pull request before merging them into our main branch. That link shows all of the commits on that branch, if you are curious for tracking. I will try to align the names with the name of the Issue you submit, including the Issue #, so that the code change and the Issue are automatically linked.
I will close out the Issues after I make the change and it seems like there is no follow-up discussion, however, feel free to re-open any Issues as necessary.

Sounds good. Thanks Mick!


partially broken download of test data

In the latest runs of GH action the first attempt to fetch a data file from GIN fails with the error
datalad.support.exceptions.CommandError: CommandError: 'git -c diff.ignoreSubmodules=none cat-file blob git-annex:remote.log' failed with exitcode 128 under /home/runner/ephy_testing_data [err: 'fatal: path 'remote.log' does not exist in 'git-annex'']

See attempt 2 of https://github.com/NeuralEnsemble/python-neo/runs/6325659861?check_suite_focus=true
The same test run in attempt 1 were successfull 17 days earlier.
The two attempts use the same versions of the datalad, git-annex are the same. git differs only in a minor version (1:2.35.1-0ppa1~ubuntu20.04.1 vs 1:2.36.0-0ppa1~ubuntu20.04.1). I didn't succeed in running the tests with the older git version again, but this seems to unlikely the reason for the failing tests, at it's on the git-annex-gin level. The respective test file on gin did not change in the last year.
@samuelgarcia Do you have any more ideas what to check? Should we removed the cached test files to check if this is the issue?

On spikeinterface side.
Alessio is also fightinh here : https://github.com/SpikeInterface/spikeinterface/pull/621


Add LoopingRiveIcon

Added a new LoopingRiveIcon widget for rendering a standalone, infinitely looping Rive animated icon.

Apologies for working directly on the main branch, which has resulted in the two changes being combined into a single PR. If needed, I can create separate branches for each change.

I apologize for starting directly on the main branch, which has combined the two changes into a single PR. However, I believe this is a minor modification and should not cause significant issues. If needed, I’m happy to make adjustments or create separate branches for each change.

it's fine. No need to create a different branch

I have reviewed your contributions. Thanks again for contributing.
To achieve a looping rive icon, we don't need to create a separate widget. I suggest you wrap the InkWell widget with an AbsorbPointer, then create a new param enableAbsorbPointer which will be false by default. You can then pass this param to AbsorbPointer to enable or disable the AbsorbPointer. The icon already loops by default.

I would suggest having two separate components: RiveAnimatedIcon and RiveAnimatedIconButton.

I think the current name RiveAnimatedIcon is a bit misleading, as it currently feels more like a button than an icon. Distinguishing it, similar to distinguishing between an IconButton and an Icon, improves clarity. With regard to using AbsorbPointer, I think removing unnecessary widgets can make the widget tree more lightweight, although not doing so may not cause serious performance issues.

True. It has some button features that make it different from regular icons. However, the intention behind building the RiveAnimatedIcon like this is to have an animated icon that its animation can be controlled by tap or hover instead of just a regular looping animation. It wasn't intended to be a button.
I also understand your proposal for RiveAnimatedIcon and RiveAnimatedIconButton, it is a great idea. Although, as you previously mentioned it will lead to breaking changes.
The issue you raised earlier was that you needed a looping rive animated icon without the inkwell. We can easily achieve this using the absorbPointer.


unknown flag: --default-service-account in source controller v0.33.0

flag --default-service-account is listed in source controller documentation:
https://fluxcd.io/flux/components/source/options/
but when I deploy source controller v0.33.0 with this flag, it will complain with error:
unknown flag: --default-service-account
so, is this flag available for v0.33.0?

That flag just got removed in https://github.com/fluxcd/website/commit/96d5e962a3f70e2b4632cc2ab802768766a8e2b0 so problem solved? :)


Use actions for buttons and links

[x] Add submitSelf().

Whats up with the tests?

No idea, maybe some internal thing since we use callAction() instead of call() now as the buttons have been converted to actions.


add encapsulate with parentheses (quotes, brackets, braces) function

Encapsulating parentheses and others
I want to be able to highlight code and press the button for some beginning character like the opening parenthesis, so that the whole highlighted text is surrounded by an opening parenthesis on the left and a closing parenthesis on the right, as an example. Many editors support this kind of behavior and I have gotten used to being able to do this.
What steps will reproduce the problem?

highlight some code
press opening parenthesis key

What is the expected output? What do you see instead?

The text should be surrounded by parenthesis.
The text was replaced.

Versions and main components

Spyder Version: 2.3.9 (in an Anaconda env for Python 3.5)
Python Version: Python 3.5
Operating system: Fedora 22

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:
IPython >=1.0      :  5.1.0 (OK)
jedi >=0.8.1;<0.9.0:  0.9.0 (NOK)
matplotlib >=1.0   :  1.5.1 (OK)
pandas >=0.13.1    :  0.18.1 (OK)
pep8 >=0.6         :  1.7.0 (OK)
psutil >=0.3       :  4.3.1 (OK)
pyflakes >=0.6.0   :  1.3.0 (OK)
pygments >=1.6     :  2.1.3 (OK)
pylint >=0.25      :  None (NOK)
qtconsole >=4.0    :  4.2.1 (OK)
rope >=0.9.2       :  0.9.4-1 (OK)
sphinx >=0.6.6     :  1.4.1 (OK)
sympy >=0.7.3      :  None (NOK)
zmq >=2.1.11       :  15.4.0 (OK)

Good idea. It shouldn't be that hard to implement :-)

If I may add something to this request, also enclose selection with brackets, curlies, and quotes, when the corresponding key is pressed.
Cheers.

Would like to implement this if someone points me to the right direction in the code.

I would start by studying https://github.com/spyder-ide/spyder/blob/da167e4e088373bcb3028112027c3eddab34c14f/spyder/widgets/sourcecode/codeeditor.py#L2668
That block automatically inserts the closing parenthesis when you type the opening parenthesis in foo(.
Good luck!

@h4gen Spyder already has quote insertion for selected text in master.  If you select text and press the ' or " key the selection will get quoted.
This functionality is now implemented as the first example of an editor extension:
https://github.com/spyder-ide/spyder/blob/master/spyder/widgets/sourcecode/extensions/closequotes.py
This section does the part you want to do with brackets:
https://github.com/spyder-ide/spyder/blob/da167e4e088373bcb3028112027c3eddab34c14f/spyder/widgets/sourcecode/extensions/closequotes.py#L71-L78
I think you will want to write an editor extension similar to closequotes extension. See this PR for more info.
https://github.com/spyder-ide/spyder/pull/5002
Sounds like a fun project!


Fix compiler warnings.

There are couple of warnings to be seen during the build process and it should be quite easy to get rid of them:
4>------ Rebuild All started: Project: Stratis.Bitcoin, Configuration: Debug Any CPU ------
4>Features\MemoryPool\MempoolValidator.cs(554,4,554,6): warning CS0162: Unreachable code detected
4>Features\MemoryPool\MempoolOrphans.cs(192,11,192,22): warning CS0219: The variable 'nFetchFlags' is assigned but its value is never used
4>Features\Consensus\ConsensusStats.cs(15,31,15,36): warning CS0169: The field 'ConsensusStats.stack' is never used
4>Features\RPC\RPCRouteHandler.cs(20,38,20,54): warning CS0169: The field 'RPCRouteHandler.contollerMapping' is never used
4>Features\Miner\PosMinting.cs(48,25,48,38): warning CS0169: The field 'PosMinting.hashPrevBlock' is never used

5>------ Rebuild All started: Project: Stratis.Bitcoin.IntegrationTests, Configuration: Debug Any CPU ------
5>UtilTests.cs(39,6,39,48): warning CS4014: Because this call is not awaited, execution of the current method continues before the call is completed. Consider applying the 'await' operator to the result of the call.
5>MemoryPoolTests.cs(946,14,946,16): warning CS0649: Field 'TestMemPoolEntryHelper.lp' is never assigned to, and will always have its default value null
5>BlockStoreTests.cs(20,15,20,35): warning xUnit1013: Public method 'BlockRepositoryBench' on test class 'BlockStoreTests' should be marked as a Fact.
5>MinerTests.cs(567,15,567,42): warning xUnit1013: Public method 'MinerCreateNewBlockValidity' on test class 'MinerTests' should be marked as a Fact.

 [General] Fix compiler warnings

I fixed the Mempool ones in https://github.com/stratisproject/StratisBitcoinFullNode/pull/311

great, i wait for that pull

Fix the mempool tests in https://github.com/stratisproject/StratisBitcoinFullNode/pull/316


Issue with mssql docker linux container AD

Hi I was wondering if someone else has had this issue before,
Currently trying to authenticate with LDAP using How to Enable AD Authentication for SQL 2019 Containers in Less than 5 Minutes | Data Exposed - YouTube as a guide with Configure Active Directory authentication with SQL Server on Linux-based containers using adutil - SQL Server | Microsoft Learn
the only thing different i changed is the docker compose file which im not sure if thats the issue,
I was checking the logs and couldn’t find information of the issue
the only issue im having is when i try to create the user
the ad server is <IP_ADDRESS>
create login [domain\administrator] From Windows
Msg 15401, Level 16, State 1, Line 1
Windows NT user or group 'domain\administrator' not found. Check the name again.

version: '3.7'
services:
      mssql:
        image: mcr.microsoft.com/mssql/server:2019-latest
        ports:
          - "1433:1433"
        env_file:
          - sqlserver.env
          - sapassword.env
        volumes:
          - /scsi3/sqlsystem:/var/opt/mssql/
          - /scsi3/sqldata:/var/opt/sqlserver/data
          - /scsi3/sqllog:/var/opt/sqlserver/log
          - /scsi3/sqlbackup:/var/opt/sqlserver/backup
          - /scsi3/krb5.conf:/etc/krb5.conf
        dns:
            - <IP_ADDRESS>
        extra_hosts:
            - 'apolo.domain.local:<IP_ADDRESS>'
            - 'domain.local:<IP_ADDRESS>'

volumes:
  sqlsystem:
  sqldata:
  sqllog:
  sqlbackup:

when i ssh into the container and ping apolo.domain.local it resolves correctly not sure what else i missed?

I have exactly the same issue , is there a way to check if the kerberos config is taken in account ? (/etc/krb5.conf)

@tellierflexus i tried many times cant seem to get it working not sure if you were able to get it working?

Nop, doesn't work for me... Do you have an idea of how to check if the kerberos link works ?

i got somewhere with this but still trying to figure out
root@hercules:/# klist
Ticket cache: FILE:/tmp/krb5cc_0
Default principal<EMAIL_ADDRESS>
Valid starting     Expires            Service principal
12/26/23 03:54:34  12/26/23 13:54:34<EMAIL_ADDRESS>	renew until 12/27/23 03:54:32
12/26/23 03:54:37  12/26/23 13:54:34<EMAIL_ADDRESS>	renew until 12/27/23 03:54:32
root@hercules:/# /opt/mssql/bin/mssql-conf validate-ad-config /var/opt/mssql/secrets/mssql.keytab
Error: Cannot contact default realm 'TEST'
Warning: RDNS could not resolve this host. This is not a fatal error, but should be fixed by updating your RDNS records
Traceback (most recent call last):
  File "/opt/mssql/bin/../lib/mssql-conf/mssql-conf.py", line 598, in <module>
    main()
  File "/opt/mssql/bin/../lib/mssql-conf/mssql-conf.py", line 594, in main
    processCommands()
  File "/opt/mssql/bin/../lib/mssql-conf/mssql-conf.py", line 310, in processCommands
    COMMAND_TABLE[args.which]()
  File "/opt/mssql/bin/../lib/mssql-conf/mssql-conf.py", line 266, in handleValidateADConfig
    config.validate()
  File "/opt/mssql/lib/mssql-conf/mssqlad.py", line 149, in validate
    if self.checkSPNs() != successExitCode:
  File "/opt/mssql/lib/mssql-conf/mssqlad.py", line 376, in checkSPNs
    kvno, ret = self.checkKeytabEntry(spn)
  File "/opt/mssql/lib/mssql-conf/mssqlad.py", line 315, in checkKeytabEntry
    kvno = kvno.split('=')
TypeError: a bytes-like object is required, not 'str'

@tellierflexus i got it working normally installing with https://www.cviorel.com/tag/mssql/
i then retried on the container debug it and got this
01/02/2024 01:41:24.395092415 Error [security.ldap] <0000000266/0x000003a4> Could not look up short domain name due to error: No address associated with hostname.
01/02/2024 01:41:24.395179302 Debug [security.kerberos] <0000000266/0x000003a4> SSPI operation 0x0000000D returned status: File: KerberosStream.cpp:1678 [Status: 0xC0000001 Operation unsuccessful]

and in the container it can resolve perfectly

Any specific hints re: file permissions @saxn ? Have battled a long time to get SQL Server/AD on docker to work but stumped with PAL logs like
07/16/2024 05:40:43.000340731 Error [security.kerberos] <0000000226/0x0000031c> Error in AcceptSecurityContext: Failed to lookup identity of initiator```

Infuriatingly, the sql instance will join the domain once and then after restart or failing to perform a magic rain dance, it stops working.

@neutmute I had the same log entries and got it to work with checking file permissions on the keytab file, the krb5.conf and the mssql.conf files. chown & chmod are your friends, try with all permissions and take them away, unfortunatly I can't tell you what exact permissions to use.
Also we are working with a rootless docker install, so that complicated things a bit (you have to find the correct user id, it gets replaced as defined in /etc/subuid)
We ran into another problem when trying to update the version, we tried anything above 2022-CU13-ubuntu-20.04 without success, but then when we migrated back, it would not work anymore!
We switched back to 2022-CU12-ubuntu-20.04 and updated again, then it once again started to work, but not able to reproduce it.
I am trying to figure out what changes in the update process, we would like to migrate to a newer image version at some point.

@neutmute
Again with an update after a couple of hours of testing.
We identified the upgrade/downgrade of mssql as the source of the problem!
Check out our testing protocol:

We started from the top and migrated to different versions.
Note that we also switched to Ubuntu Version 22.04 - with not much changing.
Red means it won't work, either because "FQDN not returned by RDNS lookup" or "Failed to bind to LDAP server" (Again no pattern)
Green means it was working.
The problem is - we dont have a clue what the difference is. We suspect some kind of upgrade/downgrade script, but if you see any pattern, feel free to let us know :) We don't see one.
Big version changes have a better chance of bricking from what we can tell.
Would love to hear your findings.

We got some consultants in and through their network got multiple advises that SQL Server Docker and AD just doesn't work. They had tried and failed several times to get it work repeatedly and reliably.
For the workload we intend, they suggested we shouldn't even be using docker. So I installed SQL Server 2019 onto Ubuntu 20.04 metal and AD just worked without any issue.
Next problem: The performance compared to Windows on same hardware sucks...

@neutmute
I was finally able to solve the riddle!
Turns out it was an issue with docker and docker compose and the hosts entries.
We defined the entries we needed but docker compose sorts them differently and not always in the same way, and if the order of the entries wasn't correct it would not work. I thought the order isn't important, but it is, obviously.
In the end I solved it by combining the entries in the same line:
extra_hosts:
- "fqdn.xd.dsd dsd:<IP_ADDRESS>"
- "fqdn.xd.dsd dsd:<IP_ADDRESS>"

Note the dsd is in there twice, once after a space.
This way it works reliable and also faster than sql server on windows server - specially the reboots! 5 seconds with docker is simply unbeatable.
So glad we finally have a consistent behaviour now!! And we did learn a lot in the process.
I hope this helps somebody stuck with the same problem.

🤯 oh wow. So much black magic.


Hotkey Toggle?

Would you mind implementing a hotkey for enabling/disabling sync? This is a feature missing from every ambilight program I have found so far and it's driving me nuts. I don't even care if I can set my own as long as there is one as I would use it all the time and it'd also make coding up an OpenHAB toggle a lot easier as I'll be able to use Eventghost to turn sync ON and OFF and Eventghost can receive MQTT messages/commands from my OpenHAB. Thanks in advance as this would make my day!
*Edit
After confirming that the light's not being toggle able while Sync is active is an API issue I'd like to stress how amazing this would be to have! I'm in and out of my office all the time and I wont be able to use the new API at all if I don't have a quick way to turn sync on and off!

folding into duplicate #101


Fullscreen messes up editor size

Upon entering fullscreen I got this effect:

and then exiting fullscreen left me with

Can you try it in safe mode (atom --safe).
If this won't happen anymore, your culprit is a community package that needs fixin'.

Looks like it's minimap. Thanks!


cp2102usb to uart

[Enter feedback here]

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

ID: b98c3de9-19b4-de58-d7c2-4f8529d953e1
Version Independent ID: 63b0b15a-13a2-0951-2e39-0467d5bbf030
Content: Installing Plug and Play Serial Ports and COM Ports - Windows drivers
Content Source: windows-driver-docs-pr/serports/installing-plug-and-play-serial-ports-and-com-ports.md
Product: windows-hardware
Technology: serports
GitHub Login: @DOMARS
Microsoft Alias: domars

Closing out due to empty feedback.  Please reopen with info if still needed


Client and Server at same time

Hi
Note: my problem is't library issue, it's work just fine.
I'm trying to create one loop  as a server( web socket for a web application) and another as a client(get data from a service using by a tcp server). the received data by client will send to web application after some processing things through server (web socket).
When I ran loop (even inside a thread) after that I can't run second one(first one block running procedure).
How can I solve this?

There can only be one event loop. So if you have two one will always block another, instead let both client and server use the same loop

Thanks for help. problem solved.

I'll assume this is resolved and will close this for now, please feel free to report back otherwise :+1:


Support for the modular Firebase 9

The modular Firebase v9 is in the final beta stages and it's quite stable. I am looking forward to adding support for it to the Firebase adapter.

I don't oppose it, but since the adapter is server-side only, it won't really have a big effect in my opinion. Or is there any other reason for version 9 to be better than modularity?

The main shtick of v9 is the reduced bundle size on the both client/server side. This is what they say

The fully modularized version 9 has these advantages over earlier versions:
Version 9 enables a dramatically reduced app size. It adopts the modern JavaScript Module format, allowing for "tree shaking" practices in which you import only the artifacts your app needs. Depending on your app, tree-shaking with version 9 can result in 80% less kilobytes than a comparable app built using version 8.
Version 9 will continue to benefit from ongoing feature development, while version 8 will be frozen at a point in the future.

v9 is gonna be a refactor I suppose. Things like:
async createUser(profile) {
  const userRef = await client.collection("users").add(
    stripUndefined({
      name: profile.name,
      email: profile.email,
      image: profile.image,
      emailVerified: profile.emailVerified ?? null,
    })
  )
  const snapshot = await userRef.get()
  const user = docSnapshotToObject(snapshot)
  return user
}

is going to be
import { collection, doc, addDoc } from "firebase/firestore";

async createUser(profile) {
  const newUser = stripUndefined({
    name: profile.name,
    email: profile.email,
    image: profile.image,
    emailVerified: profile.emailVerified ?? null,
  })
  const userRef = await addDoc(collection("users", newUser));
  const snapshot = await getDoc(userRef)
  const user = docSnapshotToObject(snapshot)
  return user
}

would like to land https://github.com/nextauthjs/next-auth/pull/2361 first

@atanaskanchev, @wobsoriano the refactored version is available at #183. I would appreciate it if you could look and help to think through how we could utilize Firebase 9.
My biggest question is how to provide all the necessary methods because I would like to keep the adapter APIs for different adapters similar. Meaning most adapters pass a client object, which contains all the necessary methods. I am thinking we will have to require the users to write some boilerplate and create a FirebaseClient object that is passed as an argument to FirebaseAdapter.

Oh wait... looks like web only has the modular update. Not sure

So it might not be necessary to upgrade after all? 😁

Just tried "firebase": "9.0.0-beta.7" in a node environment and it's actually supported too.
So yeah, your example will do 😁
import { FirebaseAdapter } from '@next-auth/firebase-adapter'
import { initializeApp } from 'firebase/app'

import {
  getFirestore,
  collection,
  query,
  getDocs,
  where,
  limit,
  doc,
  getDoc,
  addDoc,
  updateDoc,
  deleteDoc
} from 'firebase/firestore'

const firebaseApp = initializeApp({ /* config */ });

const db = getFirestore(firebaseApp)

const firebaseClient = {
  db,
  collection,
  query,
  getDocs,
  where,
  limit,
  doc,
  getDoc,
  addDoc,
  updateDoc,
  deleteDoc
}

export default NextAuth({
  adapter: FirebaseAdapter(firebaseClient)
})

Thanks, will have a look then!

@atanaskanchev, @wobsoriano I did the rewrite<EMAIL_ADDRESS>requires at least Firebase 9. All tests pass, but could any of you test it out? You will need<EMAIL_ADDRESS>at least.

@atanaskanchev, @wobsoriano I did the rewrite<EMAIL_ADDRESS>requires at least Firebase 9. All tests pass, but could any of you test it out? You will need<EMAIL_ADDRESS>at least.

Will there be a new official update to the Firebase Adapter Docs?
Because the current version of the docs isn't compatible with firebase v9.
I had to add the FirebaseClient like mentioned above in order to work.

Still haven't decided. See #183 What's the status of Firebase 9? Is it stable yet?

import { initializeApp } from 'firebase/app'

import {
  getFirestore,
  collection,
  query,
  getDocs,
  where,
  limit,
  doc,
  getDoc,
  addDoc,
  updateDoc,
  deleteDoc
} from 'firebase/firestore'

const firebaseApp = initializeApp({ /* config */ });

const db = getFirestore(firebaseApp)

const firebaseClient = {
  db,
  collection,
  query,
  getDocs,
  where,
  limit,
  doc,
  getDoc,
  addDoc,
  updateDoc,
  deleteDoc
}

export default NextAuth({
  adapter: FirebaseAdapter(firebaseClient)
})

async createUser(profile) {
  const userRef = await addDoc(collection(db, 'users'), newUser);
  const snapshot = await getDoc(userRef)
  const user = docSnapshotToObject(snapshot)
  return user
}

I'm getting FirebaseError: Expected first argument to collection() to be a CollectionReference, a DocumentReference or FirebaseFirestore this way. Anyone had similar issue?

Yes Firebase is stable. It's version ^9.4.1

Still haven't decided. See #183 What's the status of Firebase 9? Is it stable yet?

We should go for v9 then I think. I'm not able to work in the PR, so if anyone interested in picking up, please do 🙏. Would be awesome to figure out if we can integrate with Firebase Authnas well.

Please add support for v9 as soon as possible. It's a humble request.

"as soon as possible" is not really humble. :sweat_smile: Please read https://github.com/nextauthjs/adapters/issues/175#issuecomment-1001813990. Feel free to work on it.
Useful links:
https://next-auth.js.org/adapters/overview
https://next-auth.js.org/adapters/models
https://next-auth.js.org/tutorials/creating-a-database-adapter
See other adapters for example on testing, TypeScript etc.:
https://github.com/nextauthjs/adapters/tree/main/packages

I am not good with words. I am sorry if that was rude in any way. It's just I am stuck on a project where I am using Firebase v9.
It's my first time using any adapter, I don't even know what an adapter is properly, I was just following a tutorial.
But I will definitely try to work on it, whatever I can.

No hard feelings! :green_heart:
Hopefully, the above links should give you a proper introduction. When you've read through all of them and still have questions, don't hesitate to ask.

Please see https://github.com/nextauthjs/adapters/issues/180#issuecomment-1004126562 with regards to firebase auth. The issue seems to be closed. I have also got it in a basic working state here #363

React-Query-Firebase is pretty much working like a firebase adapter right now @srijans38 @SahilAujla
I'm using it until I see a stable fb v9 for next-auth.


Strange Crypto Exception on some Windows machines

On some machines i get a crazy windows encryption exception. The error occurs when protecting the password.
Most likely this issue only happens if users only use a domain user but have no local admin user on the machine.
(have a local fix)

please send pull request

https://github.com/fsprojects/Paket/pull/1347


Using typeof to determine type and assign value accordingly

Flow version: "flow-bin": "^0.92.0"
Expected behavior

Actual behavior

This is refinement invalidation, please inline typeof in if.
Also link to Try Flow would be better than screenshots
https://flow.org/en/docs/lang/refinements/#toc-refinement-invalidations

As @goodmind said ^


Bump net.javacrumbs.shedlock:shedlock-spring in /backend

BACKEND PULL REQUEST
Related Issue

Reopening https://github.com/CDCgov/prime-simplereport/pull/7187 because checks on PR were not running
Please see linked dependabot PR for description and testing results

Changes Proposed

Upgrade shedlock to version 5.10.2

deployed to dev4 and saw the cron job ran again


Remake AbstractMatrixCFcn

Replace AbstractMatrixCFcn and children for a new unique type FuncMatrix. FuncMatrix can be created without any functions and defaults them to identity, multiplication and ctranspose multiplication can be added individually. It also switches the functions when it is ctransposed to know at every moment which function to use.
Simpler to use and there is only one type for the user to handle.

Maybe you are aware but remember that functions have types in 0.5 so it might not be necessary to wrap them in a type. E.g. you can define
Base.eltype(::typeof(f)) = Float64
Base.size(::typeof(f)) = (10,10)
Base.*(::typeof(f), x) = f(x)

I'm not sure if it fits all your needs here but I could get a long way with this recently when I wrote an iterative algorithm.

I think it won't help in this case, the type wraps two functions: multiplication and ctranspose multiplication. When ctransposing it is desirable to have the wrapper to organize which function to use.

Current coverage is 85.59% (diff: 53.84%)

Merging #85 into master will decrease coverage by 0.66%

@@             master        #85   diff @@
==========================================
  Files            17         17
  Lines          1092       1104    +12
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            942        945     +3
- Misses          150        159     +9
  Partials          0          0

Powered by Codecov. Last update 04f9195...670d279

In the code it would be worth adding comments as to what the A and B type parameters mean.

@lopezm94 has tried the function type approach in this branch: https://github.com/lopezm94/IterativeSolvers.jl/blob/FunctionDispatch/src/common.jl#L79-L117
Basically the main problems are that of scope and delaying the evaluation of the definitions.
@andreasnoack any thoughts?

Can either of you explain the "delaying" part to me?
Couldn't it be e.g. Base.eltype(::typeof(mu)) = typ instead or am I missing something?

Reason:
julia> function f(g::Function)
           g(::typeof(g)) = nothing
       end
ERROR: syntax: cannot add method to function argument g

Julia does something ::typeof at compile time, so there is a need to delay the evaluation
julia> function f(g::Function)
           @eval g(::typeof(g)) = nothing
       end
f (generic function with 1 method)

Reason:
julia> function f(g::Function)
           k(::typeof(g)) = nothing
       end
ERROR: syntax: cannot add method to function argument g

Julia does something to ::typeof at compile time, so there is a need to delay the evaluation or it will go out of scope
julia> function f(g::Function)
           @eval k(::typeof($g)) = nothing
       end
f (generic function with 1 method)

julia> f(identity)
k (generic function with 1 method)

And then there's the problem with the scope, if the user deletes the object the functions might be left behind.

Closing in favor of #104


create easily new operators

in pwsh we can expand the language with several form:
we can create cmdlet/function ( binary - script )
we can create module (binary-script-dynamic...)
we can create new keyword ( ast )
we can create dynamic parameter ( class - DYNAMICKEYWORD keyword)
but we can't create operator easily and only contaiement operator is allowed to create,
what if we can create operators in runtime easily ? it's possible ? maybe with a schema 'json' or 'xml' or 'hash' and then compile this schema...for example:
PS > cat code.ps1

#Requires -configuration MyNewOperator

$a,$b = (ls) -all ($_.PSISContainer)
$a # 10 all directory
$b # $true

Operators make no sense without a model/abstraction. @rjmholt did an initial work to support DSLs but without a progress. Perhaps he could share thoughts about this.

This proposal did not raise enough interests, so it will be closed for housekeeping purpose.


Fix #416

Ticket
Closes #416
Reviewers
@.
Changes

Preventing errors when switching tabs.

Definition of Done (from ticket)
QA steps (optional)
Known Issues
This is just preventing the error and allowing the page to load, but is not really fixing anything. The real fix is on https://github.com/clevercanary/data-browser/issues/416 (we might need to revisit the solution?)

This fix was moved to PR #519


Investigate colour/intensity normalisation

This could be arbitrarily hard; could we mark it wontfix?

My thought for this issue is to do some non-linear squashing (something like a sigmoid) and that should do the most of the required job.
I recall @jbanfield suggesting something like asinh.

Alright, let's use asinh; I recall seeing a source saying that asinh has been shown to work well as inputs to these things.

I can't seem to get fitting logistic regression to actually terminate with an asinh stretch (working on it...) but for random forests the performance is improved a fair bit with such a stretch.


'Give the service principal access' correction

'Give the service principal access' part is confusing and causes 403 "AuthorizationFailed" error.
Step 1 & 2 correction example below.

Sign in to the Azure Stack portal [https://portal.local.azurestack.external/].
Select More services > Subscriptions.

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

ID: 4072186c-0214-fdab-a2d1-bd7965d76b66
Version Independent ID: c254d1d2-7fd7-d623-f45b-cd6bc53e3ca2
Content: Deploy a Kubernetes Cluster to Azure Stack
Content Source: articles/azure-stack/user/azure-stack-solution-template-kubernetes-deploy.md
Service: azure-stack
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@yountaechung Thanks for the feedback! We are currently investigating and will update you shortly.

@yountaechung  Thanks for your feedback. I am making the update. Cheers! Matt #sign-off

@yountaechung We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Will the loader work if TEXT > 4kB

https://github.com/bespoke-silicon-group/bsg_f1/blob/8d7948a04f2a20e07fedb87712a46407cfb6a9f3/cl_manycore/libraries/bsg_manycore_loader.cpp#L96
This for loop has the potential to write outside of the 4KB memory space. Or am I missing something?

This discussion seems tightly coupled with #194

Yep
I think anything related to the loader can be subsumed by #194

Fixed in #213


ndtv support broken

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2<IP_ADDRESS>. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v "https://www.ndtv.com/video/shows/walk-the-talk/walk-the-talk-with-george-fernandes-aired-june-2003-287880"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'https://www.ndtv.com/video/shows/walk-the-talk/walk-the-talk-with-george-fernandes-aired-june-2003-287880']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.15 (CPython) - NetBSD-8.0_RC1-amd64-x86_64-64bit-ELF
[debug] exe versions: ffmpeg 4.0, ffmpeg3 3.4.2, ffmpeg4 4.0, ffprobe3 3.4.2, ffprobe4 4.0
[debug] Proxy map: {}
[NDTV] 287880: Downloading webpage
ERROR: Unable to extract video filename; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/pkg/lib/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 793, in extract_info
    ie_result = ie.extract(url)
  File "/usr/pkg/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 508, in extract
    ie_result = self._real_extract(url)
  File "/usr/pkg/lib/python2.7/site-packages/youtube_dl/extractor/ndtv.py", line 91, in _real_extract
    r"(?:__)?filename\s*[:=]\s*'([^']+)'", webpage, 'video filename')
  File "/usr/pkg/lib/python2.7/site-packages/youtube_dl/extractor/common.py", line 983, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
RegexNotFoundError: Unable to extract video filename; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.ndtv.com/video/shows/walk-the-talk/walk-the-talk-with-george-fernandes-aired-june-2003-287880

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
ndtv was a supported site, though of late the support is broken, most probably something changed on the site.

I have written a fix.  You can find it here: https://github.com/rawcoder/youtube-dl/tree/ndtv_fixes
I will send a PR once it is cleaned up.  Not sure if @dstftw would be ready to merge it in its current state.


Cypress 13.6.0 [flat][loader-utils][mocha-7.0.1] Critical vulnerabilites

Hello
I use cypress/browsers:latest to create my base image
My Dockerfile
FROM cypress/browsers:latest

RUN mkdir /e2e
COPY cypress.config.js /e2e
COPY package.json /e2e
WORKDIR /e2e
RUN npm install --only=prod
RUN npm cache clean --force
ENTRYPOINT ["npm","test","run"]

I install cypress manually, this is the content of my package.json

{
    "scripts": {
     "clean:reports": "rm -rf  cypress/reports/mocha/* && rm -rf  cypress/reports/mochareports/*",
      "pretest": "npm run clean:reports",

      "scripts": "cypress run  --headed --browser $BROWSER --spec $ENVTEST ",
      "combine-reports": "mochawesome-merge ./cypress/reports/mocha/*.json > cypress/reports/mochareports/report.json",
      "generate-report": "marge cypress/reports/mochareports/report.json -f report -o cypress/reports/mochareports",
      "posttest": "npm run combine-reports && npm run generate-report",
      "test": "npm run scripts || npm run posttest "

    },

    "dependencies": {
      "cypress": "^13.6.0",
      "cypress-multi-reporters": "^1.4.0",
      "mocha": "^10.0.0",
      "mochawesome": "^6.2.1",
      "mochawesome-merge": "^4.2.0",
      "mochawesome-report-generator": "^5.1.0",
      "cypress-xpath": "^1.6.2"
    }
  }

Content of cypress.config.js

const { defineConfig } = require('cypress')

module.exports = defineConfig({
  failOnNetworkErrors: false,
  returnError: true,
  chromeWebSecurity: false,
  videoCompression: false,
  pageLoadTimeout: 660000,
  reporter: 'cypress-multi-reporters',
  reporterOptions: {
    reporterEnabled: 'mochawesome',
    mochawesomeReporterOptions: {
      reportDir: 'cypress/reports/mocha',
      quite: true,
      overwrite: false,
      html: true,
      json: true,
      video:true
    },
  },
  e2e: {
       specPattern: 'cypress/e2e/**/*.spec.js',
  },
})

I use AWS Inspector to scan vulnerabilities and I get 3 critcal ones :
root/.cache/Cypress/13.6.0/Cypress/resources/app/node_modules/flat/package.json (current 4.1.1)
Update flat to 5.0.2
root/.cache/Cypress/13.6.0/Cypress/resources/app/node_modules/mocha-7.0.1/node_modules/minimist/package.json (current  0.0.8)
Update minimist to 1.2.6
root/.cache/Cypress/13.6.0/Cypress/resources/app/node_modules/loader-utils/package.json (current 1.4.0)
Update loader-utils to 2.0.3
Any idea how can I fix it ?
PS :  I know  "cypress-xpath" is deprecated and no longer supported

@ElyessTekaya
The modules you list with security vulnerabilities are provided as part of the Cypress binary. They cannot be updated by Cypress Docker images and any update needs to come from the core https://github.com/cypress-io/cypress software.

Closing, since the Cypress Docker image cypress/browsers:latest does not contain any Cypress software and therefore the vulnerability report does not concern this repository. Even if cypress/included images were used, any issues need to be addressed directly through the process described on https://github.com/cypress-io/cypress .


AI IDs associated with 0 or multiple RMD records

@srerickson has been querying the RMD API by AI ID and gotten some unexpected results. Quite a few of the IDs he's looking for have either zero records in RMD or more than one record (up to 12). Seems like this might have something to do with duplicates, but I'm not sure exactly what. This is not blocking our work, it's just very unexpected.
These AI IDs are coming from Activity Insight exports that @nmg110 's team runs for publications where the researcher uploads a post-print in RMD. They can have a variety of document types in AI (not necessarily journal articles).
Here are examples of AI IDs with no records associated:
222649597952
218823843840
214945056768
218861940736
219108661248
Here are examples of AI IDs with more than one record associated:
196613644288 (3 records)
203850590208 (2 records)
214769737728 (12 records)
214769721344 (12 records)
218180702208 (11 records)

@anaelizabethenriquez I released a patch to production that fixes the issue with the duplicates.  There was a bug causing this.  The API was returning a record for every organization the users associated with that publication were a member of.
I'm still looking into why some AI IDs are not returning any records.  The records you listed above don't appear to be in RMD, so they must not be getting imported.  Not sure why that is, but still looking.

@anaelizabethenriquez Looking at the records for those AI IDs in Activity Insight:<PHONE_NUMBER>36,<PHONE_NUMBER>68, and<PHONE_NUMBER>40 don't have statuses of "Published" or "In Press", so they weren't imported for that reason.<PHONE_NUMBER>52 and<PHONE_NUMBER>48 look like they should have been imported without issue, however, I am noticing that the error handling in the Activity Insight importer is not ideal.  I added a ticket to address this here: #417.  Basically, the importer will stop importing data for a user if it encounters any kind of error and move to the next user.  Something like a missing title could cause an error.  This is likely the reason these publications are not being imported.  It's hard to pinpoint exactly what the error is and there could potentially be multiple errors.  For example the user (sjc2) for record<PHONE_NUMBER>48 has a handful of publications in Activity insight that do not have titles.
This is probably something that should be prioritized in a future development cycle.  I think this was potentially what we wanted to address here: #188, but it was not fixed quite the way it needed to.  I have a better understanding of the magnitude of the issue now.

Thanks, Alex. I agree about prioritizing this on a future development cycle.
I'm tagging @srerickson so he will be aware of this outcome and @nmg110 so she knows RMD isn't importing as much from AI as we thought.


JOIN should require ON condition

Describe the bug
SELECT ... FROM l JOIN r is not a valid SQL query, the ON condition should not be optional
To Reproduce
DataFusion CLI v43.0.0
> SELECT * FROM (SELECT 1 AS id, 'Foo' AS name) l JOIN (SELECT 1 AS id, 'Bar' as name) r;
+----+------+----+------+
| id | name | id | name |
+----+------+----+------+
| 1  | Foo  | 1  | Bar  |
+----+------+----+------+
1 row(s) fetched.

Expected behavior
The query should be rejected, the ON condition be requiered. In fact, omitting it might be typo with dangerous consequences (expensive cross join).
Trino
trino> SELECT * FROM (SELECT 1 AS id, 'Foo' AS name) l JOIN (SELECT 1 AS id, 'Bar' as name) r;
Query 20241119_124051_00000_rykay failed: line 1:87: mismatched input '<EOF>'.  JOIN (SELECT 1 AS id, 'Bar' as name) r;
                                                  ^

Additional context
No response

Thanks @Dandandan for checking the SQL standard too!

take

Proposed to wait for use case.
https://github.com/apache/datafusion-sqlparser-rs/pull/1552#discussion_r1866317958


JSON intellisense sometimes do not show all the error / warning messages in Visual Studio 2019

When Open a JSON file with some error / warnings, sometimes Visual Studio 2019 do not show all the errors / warnings.

Related issue:
https://github.com/microsoft/vs-dtdl-language-support/issues/6

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

ID: b34d7baa-5b74-4a87-ed21-d7fc8d9366b3
Version Independent ID: 0249ae3d-5f05-812d-46f0-ad5a921358a4
Content: JavaScript and TypeScript in Visual Studio 2019 - Visual Studio
Content Source: docs/javascript/javascript-in-vs-2019.md
Product: visual-studio-windows
Technology: vs-javascript
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@SLDragon Thanks for your feedback; we appreciate it. This looks like a product issue. To make sure that this is logged in the right place, please use the Report a Problem tool to report the issue. When you do this, your report becomes part of the work items that are listed in the in the Visual Studio Developer Community, which allows you to engage directly with the engineering team while they address and fix the problem.

Thanks, I put the issue here: https://developercommunity.visualstudio.com/content/problem/1093682/json-intellisense-sometimes-do-not-show-all-the-er.html


Syncing from upstream zalando/patroni (master)

bt_gitbot

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

10 of 10 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.86%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
12834

Relevant Lines:
12852

💛  - Coveralls


x/tools/cmd/gopls: import organization fails on certain packages

What version of Go are you using (go version)?

$ go version
go version go1.11.5 darwin/amd64

Does this issue reproduce with the latest release?
N/A, I think.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/alex/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/alex/.go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/opt/go/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/opt/go/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/alex/Projects/Transparensee/enclosure-core/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/qs/wpmg19r12_9_nz7pvvs2_82r0000gn/T/go-build353164800=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

What did you expect to see?
Saved file in VSCode that contains a reference to a struct in a different package, within the same module. I expected the import to be added to import.
What did you see instead?
Nothing was added. This usually works, but there appears to be some packages that just don't work with import organization. This works fine with Go's standard library packages as well as third-party packages.
The file is a skeleton main.go, where I added site := models.Site{}. The models package is a package inside the same module.
I don't know much about gopls and how to report diagnostics, but here's the log file with RPC logging enabled: gopls.log.

Does running goimports manually on the file work as intended?

@stamblerre Yes.

Same behavior here.

Is the file formatted and it's just that the missing import isn't added?
Is there a go.mod file present in the directory? Does adding one solve the issue?

Is the file formatted correctly and it's just that the missing import isn't added?

Yes. Also, gopls works fine for some files. It's inconsistent.

Is there a go.mod file present in the directory? Does adding one solve the issue?

I can reproduce the problem both in the root (where go.mod is) and in a sub-folder.

Got it. Thank you.

@atombender: Are you able to offer the contents of the Go file and the go.mod file for us to try to repro, or are they private?

As I said, it's just a skeleton file with just an empty main. I can see if I can reduce it to something I can share, but I'm doubtful. Is there any extra logging I can turn on?

I did see the main file in the logs you attached, but I was more curious about the go.mod file - maybe it's something to do with the dependency you are adding.
Also, just checking, is the file saved on disk? If the file isn't saved yet, gopls won't work at all.

Yes, it's saved. Everything being imported is within the same Go module. I haven't noticed any patterns to the problem. I'll see if I can create a minimally reproducible case here now.
More logging might be useful. After using gopls for a few days in VSCode I've eventually disabled it because it's completely unstable. I frequently get no autocompletions, no reformatting on save, gopls crashing, gopls having to restarted to unclog it, etc. Not sure how much of this is VSCode or gopls itself. The sudden lack of automatic reformatting always trips me up, because it's unclear if it's taking time or it's just stopped working. I've never found anything useful in the log when these things happen.

For the record, my experience with gopls + VSCode on Mac mirrors @atombender; several times a day, I notice that auto-importing and autocompletion have stopped working. Restarting VSCode (and, therefore, gopls) typically solves the issue for awhile. I'd happily run it in some diagnostic mode and report back, if it helps.

We will try to add some more logging to understand better why this happens. I've never had this happen, but there seem to be quite a few people reporting that gopls stops working in certain projects.

I can confirm that as well, even in non-go mod repos.

Same issue for me with both modules or non-modules projects. Restarting vscode does the trick.

Same issue for me... but restarting doesn't fix it.  Nothing seems to fix it.  Import organization only works like 5% of the time for me.

For those who are having this experience, can you attach your gopls logs to this issue? For VSCode users, they can be found by going to "View: Debug Console" -> "Output" -> "Tasks" ->  "gopls".

If those logs include a "no room in queue" output that correlates to roughly when the problem started, then it's likely related to #32368. If so please update and report back.

@freeformz Not in my case. Log is in original issue.

@atombender: When you see the diagnostic from the language server that your package is not imported, can you hover over the error and see a "Quick Fix" suggestion? And if so, does it offer you the "Organize All Imports" quick fix?

If those logs include a "no room in queue" output that correlates to roughly when the problem started, then it's likely related to #32368. If so please update and report back.

I updated yesterday and still got the no room in queue bug :
[Error - 1:47:00 PM] Request textDocument/documentSymbol failed.
Message: no room in queue
Code: -32000
[Error - 1:47:03 PM] Request textDocument/hover failed.
Message: no object for ident sections
Code: 0
[Error - 1:47:03 PM] Request textDocument/hover failed.
Message: no object for ident sections
Code: 0
[Error - 1:47:04 PM] Request textDocument/hover failed.
Message: no identifier found
Code: 0

Might not be related but it also doesn't work most of the time when adding a new file.
For example, I add a new file with simply package x and a struct def, gopls output :
[Error - 8:03:36 PM] unable to check package for [filename]: parse: no package found for [filename]
[Error - 8:03:36 PM] Request textDocument/hover failed.
Message: no AST for [filename]
Code: 0
Once vscode is restarted, new file is correctly recognized.

@ericrenard: Do you mind filing separate issues for these cases? I think it will be easiest to help everyone debug their cases in separate issues - even when the bugs seem similar, they are often unrelated.

Here's a log from a current editing session where imports aren't being reorganized. In this case, it refuses to remove unused imports after I commented out a section of code, and it's not adding a third-party import (gorm, which is github.com/jinzhu/gorm) that I'm using in this project.

gopls.log.gz
main.go.gz

In this case, the "quick fix" button fixes the unused imports, whereas the "quick fix" on the "Undeclared name: gorm" error shows "No code actions available".
goimports is not having any issues on this file.

Can you try adding
 "editor.codeActionsOnSaveTimeout": 3000

to your VSCode configuration? The default value is 750ms, so it might not need to be as large as 3000. This seems to have mitigated a few goimports-related issues for users on Slack.

@stamblerre That didn't help, unfortunately.
I can reproduce this consistently in my project. I have a main.go which is in a sub-folder. If I remove all its imports and save, almost none are automatically put back (the only one is gopkg.in/yaml.v2, curiously). Log: gopls-1.log.gz.
If I then undo to put back all the imports, and save, all the errors remain, even though the imports are back, and the log starts getting no room in queue errors. Log: gopls-2.log.gz.
I can tar up the project and email it to you privately if that would help?

I don't have good, reproducible test cases, but I can add some data points and confirm that the last few times gopls appeared to stop working, the debug output was full of "no room in queue" messages. For example, in this gist, everything was working fine until the ~2:25:16 timestamp.

I've now experienced gopls failing many times in the same way, which I think has not yet been reported. I'll open a file in a package and typically make some change that renders the file uncompilable. Rather than underlining my change with a red squiggle, VS Code underlines the package statement at the top of the file, and hover tells me that
file:///Users/peter/<whatever>/<file>.go is not part of a package

Every interaction with the file fails, even after I delete the change and return the file to the original state. The gopls log looks like this:
[Info  - 12:55:00 PM] Build info
----------
golang.org/x/tools/cmd/gopls
<EMAIL_ADDRESS>h1:mSUCVIwDx4hfXJfWsOPfdzEHxzb2Xjl6BQ8YgPnazQA=
<EMAIL_ADDRESS>h1:8gQV6CLnAEikrhgkHFbMAEhagSSnXWGV915qUMm9mrU=

Go info
-------
go version go1.12.5 darwin/amd64

GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/peter/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/peter"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/Cellar/go/1.12.5/libexec"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.12.5/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/peter/mod/plasma/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/8s/kvkb2dzx37n057vk54y9b_ym0000gn/T/go-build192614899=/tmp/go-build -gno-record-gcc-switches -fno-common"

[Error - 12:55:02 PM] Request textDocument/hover failed.
  Message: no identifier found
  Code: 0
[Error - 12:55:09 PM] unable to check package for file:///Users/peter/<whatever>/<file>.go: parse: no package found for /Users/peter/<whatever>/<file>.go
[Error - 12:55:09 PM] unable to check package for file:///Users/peter/<whatever>/<file>.go: parse: no package found for /Users/peter/<whatever>/<file>.go
[Error - 12:55:09 PM] Request textDocument/hover failed.
  Message: no AST for file:///Users/peter/<whatever>/<file>.go
  Code: 0
[Error - 12:55:09 PM] unable to check package for file:///Users/peter/<whatever>/<file>.go: parse: no package found for /Users/peter/<whatever>/<file>.go
[Error - 12:55:09 PM] unable to check package for file:///Users/peter/<whatever>/<file>.go: parse: no package found for /Users/peter/<whatever>/<file>.go
[Error - 12:55:10 PM] unable to check package for file:///Users/peter/<whatever>/<file>.go: parse: no package found for /Users/peter/<whatever>/<file>.go

(repeat ad nauseum)

The only fix is quitting VS Code and starting it again.

Could you update to the latest version (go get -u golang.org/x/tools/gopls@latest)? We had some stability issues with gopls this week, but things should be more stable now.

Also, @atombender, I'm sorry I dropped the ball on following up with your issue. It seems like you have a more specific case than general gopls problems (hopefully the "no room in queue errors" should be mostly gone with the latest version of gopls). Does import organization work for you in other projects?
We're going to be modifying the way that gopls does imports soon (https://github.com/golang/go/issues/32585), so we will be able to get more insight into your issue then. Right now, we basically just send the file contents we have to imports.Process. You also may be able to get more detailed logging by rebuilding gopls with the imports.Debug set to true.

@stamblerre FWIW, I'm still getting plenty of "no room in queue" errors with the very latest. I update daily just in the hope of getting more stability. I also pkill -f gopls hourly to deal with this.

Does import organization work for you in other projects?

I'm seeing the problem in all my projects. Lately I've noticed that it also affects built-in packages like log, though I don't know if it's the same issue.

Right now, we basically just send the file contents we have to imports.Process.

I'll look into modifying gopls with that debug flag. Maybe I could write a small program to run my file through imports.Process and see what happens?

@stamblerre With debug on (full log), after removing the import of litter, I see that it is resolving the import, but it takes 18 seconds (!), so VSCode ignores it:
[Trace - 3:42:01 PM] Received response 'textDocument/codeAction - (16)' in 18246ms.
Params: [{"title":"Organize Imports","kind":"source.organizeImports"," [...]

Increasing VSCode's formatting timeout to 30 seconds resolves this, but obviously that's not acceptable performance. For comparison, goimports takes less than 1 second:
$ time goimports src/api/builder.go > /dev/null
goimports src/api/builder.go > /dev/null  0.54s user 0.51s system 193% cpu 0.542 total

Why so slow?

(I'm going to test some more and see if this slowness is what's affecting other problematic imports.)

I noticed that gopls is sometimes very late in apparently doing anything on a change that needs to add an import. Here's me adding a new name that needs to be added to the import list:

[Trace - 4:04:21 PM] Sending request 'textDocument/codeAction - (247)'.
Params: {"textDocument":{"uri":"file:///Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go"},"range":{"start":{"line":0,"character":0},"end":{"line":348,"character":0}},"context":{"diagnostics":[{"range":{"start":{"line":343,"character":33},"end":{"line":343,"character":45}},"message":"undeclared name: litter","severity":1,"source":"LSP"}]}}

2019/06/14 16:04:21 fixImports(filename="/Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go"), abs="/Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go", srcDir="/Users/alex/Projects/Transparensee/enclosure-core/src/api" ...
2019/06/14 16:04:22 18.533987ms for GOROOT=/usr/local/opt/go/libexec GOPATH=/Users/alex/.go GO111MODULE= GOPROXY= PWD=/ go [go env GOMOD]
[Trace - 4:04:22 PM] Sending request 'textDocument/documentSymbol - (248)'.
Params: {"textDocument":{"uri":"file:///Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go"}}

[Trace - 4:04:22 PM] Sending request 'textDocument/codeAction - (249)'.
Params: {"textDocument":{"uri":"file:///Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go"},"range":{"start":{"line":343,"character":52},"end":{"line":343,"character":52}},"context":{"diagnostics":[]}}

[Trace - 4:04:22 PM] Sending request 'textDocument/documentLink - (250)'.
Params: {"textDocument":{"uri":"file:///Users/alex/Projects/Transparensee/enclosure-core/src/api/builder.go"}}

2019/06/14 16:04:44 scanning /usr/local/opt/go/libexec/src
2019/06/14 16:04:44 scanned /usr/local/opt/go/libexec/src
2019/06/14 16:04:44 scanning /Users/alex/.go/src
2019/06/14 16:04:44 open /Users/alex/.go/src/.goimportsignore: no such file or directory
2019/06/14 16:04:44 Error statting ignored directory: stat /Users/alex/.go/src/v: no such file or directory
2019/06/14 16:04:44 Error statting ignored directory: stat /Users/alex/.go/src/mod: no such file or directory
2019/06/14 16:04:44 scanned /Users/alex/.go/src
2019/06/14 16:04:44 litter candidate 1/1: github.com/sanity-io/litter in /Users/alex/.go/src/github.com/sanity-io/litter
[...]

Notice the 22-second gap.

I found a package that won't organize with gopls. Goimports works fine. My code:
	r := chi.NewRouter()
	r.Handle("/metrics", promhttp.Handler())

Here,

chi should resolve to github.com/go-chi/chi
promhttp should be github.com/prometheus/client_golang/prometheus/promhttp

...but gopls doesn't resolve them at all. Debug log:
2019/06/18 12:41:46 scanning /usr/local/opt/go/libexec/src
2019/06/18 12:41:46 scanned /usr/local/opt/go/libexec/src
2019/06/18 12:41:46 scanning /Users/alex/.go/src
2019/06/18 12:41:46 open /Users/alex/.go/src/.goimportsignore: no such file or directory
2019/06/18 12:41:46 Error statting ignored directory: stat /Users/alex/.go/src/v: no such file or directory
2019/06/18 12:41:46 Error statting ignored directory: stat /Users/alex/.go/src/mod: no such file or directory
2019/06/18 12:41:46 scanned /Users/alex/.go/src
2019/06/18 12:41:46 chi candidate 1/2: archive/tar in /usr/local/opt/go/libexec/src/archive/tar
2019/06/18 12:41:46 chi candidate 2/2: archive/zip in /usr/local/opt/go/libexec/src/archive/zip
2019/06/18 12:41:46 loading exports in dir /usr/local/opt/go/libexec/src/archive/zip (seeking package chi)
2019/06/18 12:41:46 loading exports in dir /usr/local/opt/go/libexec/src/archive/tar (seeking package chi)
2019/06/18 12:41:46 loading exports in dir /usr/local/opt/go/libexec/src/archive/zip (seeking package chi): scan of dir /usr/local/opt/go/libexec/src/archive/zip is not expected package chi (actually zip)
2019/06/18 12:41:46 loading exports in dir /usr/local/opt/go/libexec/src/archive/tar (seeking package chi): scan of dir /usr/local/opt/go/libexec/src/archive/tar is not expected package chi (actually tar)

For some reason it's looking in archive/zip and archive/tar despite the fact that they don't match the names being searched for.
Can reproduce with HEAD (fdf1049a943acdab3a1008d66838170551d1dedb).

That's really strange, thank you for the report. @suzmue will be working on https://github.com/golang/go/issues/32585 very soon, so hopefully we will be able to offer more assistance with this case soon.

Thanks, looking forward to it. This stuff is a real productivity-sapper.

I'm sorry you've had to deal with it. In the meantime, you can disable formatting and import organization through gopls in VSCode - you can set "format": false in the "go.languageServerExperimentalFeatures" setting.

@stamblerre Thanks, but I'm not seeing anything different with that setting. Gopls is still trying to format and resolve imports, and I can see things like fixImports in the log output.

editor.codeActionsOnSave is supposed to be an object. Did you mean "editor.codeActionsOnSave": {"source.organizeImports": false}?

That can't be right, that disables format-on-save altogether, even Format Document stops working. I have:
  "go.formatTool": "goimports",
  "editor.formatOnSave": true,
  "editor.codeActionsOnSave": {
    "source.organizeImports": true
  },
  "editor.codeActionsOnSaveTimeout": 30000
  "go.useLanguageServer": true,
  "go.languageServerExperimentalFeatures": {
    "diagnostics": true,
    "format": false
  },
  "go.languageServerFlags": [
    "-trace"
  ]

Yes, my apologies, I did mean "source.organizeImports" should be set to false. With this configuration, I am able to get goimports working through the VSCode-Go extension, rather than through gopls:
"go.formatTool": "goimports", // or "goreturns", depending on your preference
"go.languageServerExperimentalFeatures": {
    "format": false,
},
"[go]": {
    "editor.formatOnSave": true,
    "editor.codeActionsOnSave": {
        "source.organizeImports": false,
},

Same as what I had, but I reloaded the window again (which I'd already done) and it suddenly worked, for some reason, thanks!
Also, for some reason for the Format Document action to work I need "editor.defaultFormatter": "ms-vscode.Go", which (if it's not set) appears twice in the list of suggested formatters.

No, with these settings I'm still seeing import processing in the log, and considerable slowness:
2019/06/19 15:36:32 fixImports(filename="/Users/alex/Projects/Transparensee/enclosure-core/src/api/builder_picks.go"), abs="/Users/alex/Projects/Transparensee/enclosure-core/src/api/builder_picks.go", srcDir="/Users/alex/Projects/Transparensee/enclosure-core/src/api" ...
2019/06/19 15:36:32 16.822328ms for GOROOT=/usr/local/opt/go/libexec GOPATH=/Users/alex/.go GO111MODULE= GOPROXY= PWD=/ go [go env GOMOD]
2019/06/19 15:36:44 scanning /usr/local/opt/go/libexec/src
2019/06/19 15:36:44 scanned /usr/local/opt/go/libexec/src
[...]
[Trace - 3:36:44 PM] Received response 'textDocument/codeAction - (55)' in 59849ms.
Params: {}

Also seeing lines like [{"title":"Organize Imports","kind":"source.organizeImports", .... Surely these shouldn't happen with these settings?
As you can imagine, dealing with 60-second delays when saving a file and waiting for formatting to happen is painful.
Killing gopls doesn't help. I went to a different, small file with no compilation errors and tried saving there, no format-on-save. Maybe something is stuck in VScode. Maybe some internal queue of requests? Reloading the window brings back format-on-save, so something is getting stuck.

Right now, it looks like fixImports(...) appears on roughly every code change. I get outdated error markers when editing that take 30-60 seconds to update.
Inserting some debug information, imports.Process is called with FormatOnly: false in its options. Surely that should be true. It's because lsp/source/format.go calls Process with no options.
Turning the fixImports function into a no-op fixes my import slowness, but it seems I have to turn "source.organizeImports": true in VSCode on again to get goimports to be invoked.

Sorry about spamming this issue, but there's definitely something funky going on with the VSCode Go extension. I had a file where imports were not being added, as usual. I'm running gopls with the fixImports disabled altogether now, so goimports should be formatting. Yet while VSCode was formatting my code, it was not reorganizing the imports. Running goimports manually on the command line worked fine, and super fast. So I reloaded the VSCode window, re-saved the file, and boom, imports automatically added.
The goimports run is not logged to the output panel, so this is difficult to diagnose. I've now added a wrapper script for goimports that logs its activity now, so I can see if it is even being invoked. I'll keep an eye on this problem.
In the meantime, I've noticed messages like this:
[Trace - 7:02:07 PM] Received notification 'window/logMessage'.
Params: {"type":1,"message":"unable to check package for file:///Users/alex/Projects/Transparensee/enclosure-core/src/api/sitecache/sitecache.go: go/packages.Load: no packages found for /Users/alex/Projects/Transparensee/enclosure-core/src/api/sitecache/sitecache.go"}

The file in question is a new file in a new folder that I created in VSCode, inside another package. In this new file, Gopls was completely nonfunctional until I restarted it.

Update: I can confirm that after a while, VSCode just stops invoking goimports on changes. If I go to a different open VSCode window and save something, goimports is invoked. So it's clearly something stuck within a single window process. Reloading the window gets it unstuck. I don't see anything in any of VSCode's log files.

Update: Watching the log with some patience, the above problem seems to actually be caused by Gopls blocking VSCode's execution of goimports. On saving a file, I will see log messages like this:
[Trace - 9:24:19 PM] Received response 'textDocument/codeAction - (19)' in 1342ms.
Params: [{"title":"Organize Imports","kind":"source.organizeImports","edit":{"changes":{"file:///Users/alex/Projects/Transparensee/enclosure-core/src/models/utils_test.go":[{"range":{"start":{"line":12,"character":0},"end":{"line":13,"character":0}},"newText":""},{"range":{"start":{"line":13,"character":0},"end":{"line":14,"character":0}},"newText":""},{"range":{"start":{"line":14,"character":0},"end":{"line":15,"character":0}},"newText":""},{"range":{"start":{"line":15,"character":0},"end":{"line":16,"character":0}},"newText":""},{"range":{"start":{"line":16,"character":0},"end":{"line":16,"character":0}},"newText":"\t\tStreetAddress: ptrutils.String(\"123 Some Way\"),\n"},{"range":{"start":{"line":16,"character":0},"end":{"line":16,"character":0}},"newText":"\t\tState:         ptrutils.String(\"NY\"),\n"},{"range":{"start":{"line":16,"character":0},"end":{"line":16,"character":0}},"newText":"\t\tCounty:        ptrutils.String(\"Kings County\"),\n"},{"range":{"start":{"line":16,"character":0},"end":{"line":16,"character":0}},"newText":"\t\tZipCode:       ptrutils.String(\"10012\"),\n"}]}}}]

(As I said before, I commented out the import organization code in Gopls to avoid the weird long stalls, so it shouldn't be doing anything here.)
But goimports isn't being invoked... until much later. Waiting for it, I see a lot of slow actions (Received response 'textDocument/codeAction - (17)' in 5887ms), and a lot of Sending notification '$/cancelRequest'. Eventually, after maybe a minute, goimports is suddenly invoked again.
It's as if Gopls is building up a queue of changes and choking on them. And this isn't like I'm a particularly fast typer or anything. It happens constantly.

It seems like gopls is still doing the import organization, based on the fact that it's still sending those responses. The only thing it does in a code action is imports. Can you try changing the codeAction function in internal/lsp/code_action.go to just return nil, nil?

I didn't realize there were two code paths to import organization. Will try that, thanks.

Still seeing the same weirdness where VSCode just stops invoking goimports after a while. I can report that as a VSCode bug.

@ianthehat How does this fix relate to this issue?

This issue seems to cover multiple bugs that have some symptoms in common. Most of the things in it seem to be related to gopls ceasing to communicate with the editor client, often with the no room in queue message, which should be fixed.

That's just part of it, I think. I'd leave this issue open for now. It also relates to with the currently very slow import organization code that apparently is being rewritten.

Re-opened to track the imports-related issues. This may be a duplicate of https://github.com/golang/go/issues/32360.

@stamblerre You said you re-opened, but it's still closed.

This issue is outdated due to the latest changes to gopls and imports.
To track issues relating to slow import organization, please follow #32750.


How to remove the border highlight of shadcn-ui input component when focusing on it?

<Input
   type="text"
   placeholder="Search"
   className="focus:border-none"
/>

I try to change properties of focus: to border-none, border-0, outline-none, outline-0, ring-none, and ring-0. However, nothing work for me.

Hi, You can update @/components/ui/input file with:
import * as React from "react"

import { cn } from "@/lib/utils"

export interface InputProps
  extends React.InputHTMLAttributes<HTMLInputElement> {}

const Input = React.forwardRef<HTMLInputElement, InputProps>(
  ({ className, type, ...props }, ref) => {
    return (
      <input
        type={type}
        className={cn(
          "flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm ring-offset-background file:border-0 file:bg-transparent file:text-sm file:font-medium placeholder:text-muted-foreground focus-visible:outline-none disabled:cursor-not-allowed disabled:opacity-50",
          className
        )}
        ref={ref}
        {...props}
      />
    )
  }
)
Input.displayName = "Input"

export { Input }

After changing, you can use your Input with no focus ring.

Hi, You can update @/components/ui/input file with:
import * as React from "react"

import { cn } from "@/lib/utils"

export interface InputProps
  extends React.InputHTMLAttributes<HTMLInputElement> {}

const Input = React.forwardRef<HTMLInputElement, InputProps>(
  ({ className, type, ...props }, ref) => {
    return (
      <input
        type={type}
        className={cn(
          "flex h-10 w-full rounded-md border border-input bg-background px-3 py-2 text-sm ring-offset-background file:border-0 file:bg-transparent file:text-sm file:font-medium placeholder:text-muted-foreground focus-visible:outline-none disabled:cursor-not-allowed disabled:opacity-50",
          className
        )}
        ref={ref}
        {...props}
      />
    )
  }
)
Input.displayName = "Input"

export { Input }

After changing, you can use your Input with no border highlight and focus ring.

Thank you! It finally work.


Among many input fields sharing the same state, if one input state  changes, causing all other styled components to re-render in both development and production environment.

I have a big form with 30-40 styled Input elements along with styled buttons, paragraphs, div containers. And if one input changes, it is causing all styled elements to re-render , and as I type it si lagging significantly, and delaying the input. I tried with Regular HTML inputs along with plain CSS, it is working just fine. We have been using Styled-Components heavily in our project and we cannot move to another library or env and I love SC. I shared the code snippet that I wrote which follows kind of the same structure with our actual project. What am I doing wrong here?
I am using ComponentShouldUpdate for now as a workaround, but this is unnecessary and annoying to put it in each component. It is not happening with regular HTML elements, so I think there is a bug or I am misusing SC.
Environment
System:

OS: macOS High Sierra 10.13.5
CPU: x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
Memory: 2.39 GB / 16.00 GB
Shell: 5.3 - /bin/zsh

Binaries:

Node: 8.11.3 - /usr/local/bin/node
Yarn: 1.7.0 - /usr/local/bin/yarn
npm: 5.6.0 - /usr/local/bin/npm

npmPackages:

styled-components: 3.3.3 => 3.3.3

Reproduction
You can see the actual code here:
https://stackblitz.com/edit/styled-components-bug
Expected Behavior
It should only re-render the component which receives different props.
Actual Behavior
Here I uploaded the videos , on the left side Input fields and paragraphs with regular HTML and on the right side Styled HTML Elements with SC
https://youtu.be/KTWx3eGl_VE . Production Env
https://youtu.be/u7T3Av8npcA.  Development Env

Hmm I'm not seeing your bug. I have the React devtools open with "trace on updates" enabled and only the input I'm working on seems to be updating.


example.io_uring isn't getting compiled

Since c9df036823bd4e5719e4aa491c065e564bdb0f3a (stdexect_examples typo). Presumably as a result, it seems to have bitrotted a bit :)
In https://github.com/lukester1975/stdexec/tree/fix-uring-example I've put a cmake fix and a few other hacks to try to get the example working. Happy to separate out the cmake fix in to a PR. I'm sure the rest is largely nonsense...
Presumably context.run() should be context.run_until_stopped() after 5bb97462362a5ebac31fcf49c9fb69f259d22862?
The example still seems a bit racy. Sometimes it works:
Hello, 1!
Hello, 2, stopped.
Hello, 2!
Hello, 1, stopped.
Hello, 0!
Hello, 1!
Hello, 2!
Stop it!
Requesting context stop
Requesting context2 stop
Hello, stopped.
Hello, stopped.
Joining io_thread
Joining io_thread2
Threads finished
The context is stopped!
The context is stopped!
Restarting context
Hello, 2!
Hello, schedule_at!
Finished

Other times:
Hello, 1!
Hello, 2, stopped.
Hello, 2!
Hello, 1, stopped.
Hello, 0!
Hello, 1!
Hello, 2!
Stop it!
Requesting context2 stop
Hello, stopped.
Hello, stopped.
Joining io_thread

I've not looked at that in any depth as it made my brain melt. It did seem context.reset() was required to ever get it to work, though.
Maybe I should open separate issue for this, but should run_until_stopped() restart IORING_ENTER_GETEVENTS if it gets EINTR? Otherwise run_until_stopped() needs wrapping in a loop and try/catch, presumably? Gets interrupted a lot when debugging.
Kernel is 6.5.6-200.fsync.fc38.x86_64 from nobara. Perhaps I should double check with something more vanilla...
Thanks!

Thank you for this!
I will look into it and fix it. Regarding EINTR you are correct. In fact I have a fix (and some improvements) in the pipeline here
https://github.com/maikel/senders-io/blob/static_io_pool/source/sio/io_uring/io_uring_context.cpp#L351
I haven't managed yet to merge it yet but I will try do so today.

I've fixed the trivial stuff. For the other error: I have yet to debug the cause. Currently I'm suspecting some misuse of the io_uring interface. For example, I am ignoring the dropped counter for the submission queue. I will try to allocate time soonish.


How to populate certificate metadata i.e. subject details e.g. OU, Organization etc

HI,
We are testing this add-on in OpenShift with OpenShift-cert-manager operator. The installation worked fine and the initial testing with specified annotations in README worked as expected. However we are looking for annotations that can be used for certificate metadata such as subject - Organization Unit, Organization, locality, state, country etc. Basically we are hoping to have annotations available for all the fields that certificate resource provides from configuration standpoint.
Currently all these fields are coming as blank. Please let us know if we have annotation available or is there any plan for implementing those.
Thanks

Both of the below are requirements

#29 needs to be merged in this repo which adds support for subject annotations, I've been a little slow getting back to the maintainers but I am going to update the PR shortly!
Coincidentally I was also the one who submitted a similar PR for supporting subject fields on ingresses in cert-manager, those are supported as of cert-manager v1.12.0. After the #29 is merged, you would probably want to be running cert-manager v1.12.0 or higher to align with the version used in openshift-routes but it might work with an older version too. OpenShift cert-manager-operator is only at v1.11.4 as it trails upstream by at least 1 minor version.


[Question] Are there javadocs generated for zuul core?

Yes, they're pushed to the public repo for each build.

Could you please link me to the public-repo?

I think this is what you're looking for - JFrog Bintray Netflix. The jars are available under Files or a direct link here.