Using a session configuration file with ssh remoting on mac fails

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Add PS7 subsystem to sshd_config:
Subsystem	powershell	/usr/local/microsoft/powershell/7-preview/pwsh -sshs -nologo -noprofile -configurationfile /Users/steve/test/demo.pssc

Poing to a pssc file:
@{

# Version number of the schema used for this document
SchemaVersion = '<IP_ADDRESS>'

# ID used to uniquely identify this document
GUID = '86335646-db0a-455d-8332-01668112ba77'

# Author of this document
Author = 'slee'

# Description of the functionality provided by these settings
# Description = ''

# Session type defaults to apply for this session configuration. Can be 'RestrictedRemoteServer' (recommended), 'Empty', or 'Default'
SessionType = 'Default'

# Directory to place session transcripts for this session configuration
# TranscriptDirectory = 'C:\Transcripts\'

# Whether to run this session configuration as the machine's (virtual) administrator account
# RunAsVirtualAccount = $true

# Scripts to run when applied to a session
ScriptsToProcess = '/Users/steve/test/pssc.ps1'

ModulesToImport = @()

# User roles (security groups), and the role capabilities that should be applied to them when applied to a session
# RoleDefinitions = @{ 'CONTOSO\SqlAdmins' = @{ RoleCapabilities = 'SqlAdministration' }; 'CONTOSO\SqlManaged' = @{ RoleCapabilityFiles = 'C:\RoleCapability\SqlManaged.psrc' }; 'CONTOSO\ServerMonitors' = @{ VisibleCmdlets = 'Get-Process' } }

# Language mode to apply when applied to a session. Can be 'NoLanguage' (recommended), 'RestrictedLanguage', 'ConstrainedLanguage', or 'FullLanguage'
LanguageMode = 'RestrictedLanguage'

# Cmdlets to make visible when applied to a session
VisibleCmdlets = 'Get-Command', 'out-default', 'get-process'

# Functions to make visible when applied to a session
VisibleFunctions = 'Exit-Session'

}

the pssc.ps1 simply contains:
function Exit-Session {
    exit
}

Expected behavior
Successfully create session

Actual behavior
New-PSSession: [localhost] The background process reported an error with the following message: Remote transport error: Running startup script threw an error: The specified module '/usr/local/microsoft/powershell/7-preview/Modules/Microsoft.WSMan.Management' was not loaded because no valid module file was found in any module directory...

Error details
Exception             :
    Type        : System.Management.Automation.Remoting.PSRemotingTransportException
    ErrorCode   : 2100
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            Message : The background process reported an error with the following message: Remote transport error: Running startup script threw an error: The
specified module '/usr/local/microsoft/powershell/7-preview/Modules/Microsoft.WSMan.Management' was not loaded because no valid module file was found in any module
directory...
            HResult : -2146233087
        CategoryInfo          : ResourceUnavailable: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : System.Management.Automation.Remoting.PSRemotingDataStructureException
    Message     : The background process reported an error with the following message: Remote transport error: Running startup script threw an error: The specified
module '/usr/local/microsoft/powershell/7-preview/Modules/Microsoft.WSMan.Management' was not loaded because no valid module file was found in any module directory...
    HResult     : -2146233087
TargetObject          : System.Management.Automation.RemoteRunspace
CategoryInfo          : OpenError: (System.Management.A…tion.RemoteRunspace:RemoteRunspace) [New-PSSession], PSRemotingTransportException
FullyQualifiedErrorId : 2100,PSSessionOpenFailed
ErrorDetails          : [localhost] The background process reported an error with the following message: Remote transport error: Running startup script threw an
error: The specified module '/usr/local/microsoft/powershell/7-preview/Modules/Microsoft.WSMan.Management' was not loaded because no valid module file was found in
any module directory...
InvocationInfo        :
    MyCommand        : New-PSSession
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 25
    Line             : new-pssession -HostName . -UserName steve
    PositionMessage  : At line:1 char:1
                       + new-pssession -HostName . -UserName steve
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : new-pssession
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1
PipelineIterationInfo :

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.0-preview.8
PSEdition                      Core
GitCommitId                    7.3.0-preview.8
OS                             Darwin 21.6.0 Darwin Kernel Version 21.6.0: Wed Aug 10 14:28:35 PDT 2022; root:xnu-8020.141.5~2/RELEASE_ARM64_T8101
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

This only occurs when SessionType is Default, setting it to RestrictedRemoteServer seems to work

It seems that remote initial session state explicitly imports engine modules where two are not on Unix systems.
However, after removing those for Unix, it also looks like remote server mode depends on Select-Object and Measure-Object to handle command metadata so those cmdlets have to be available.

I am removing WG-Remoting tag because this is not a remoting issue.  If you want a secure remote session, you should use:
SessionType = 'RestrictedRemoteServer'

And remove the LanguageMode setting as it is not needed.
I tested this and everything works as expected because the session is configured for remoting.
But I agree that this is an engine bug where InitialSessionState should work correctly on non-Windows platforms.

@SteveL-MSFT : It looks like you want to do powershell remote via SSH with JEA Endpoint Configuration? Does the current PS remote via SSH really support JEA Endpoint Configuration?


The update is deleting data

When upgrading to the new version (3.0), I lost all my connections and snippets.
I performed the updated via .deb file.

Hi @Leonardofreua ,
Are you using OmniDB app or server? What version were you running before the upgrade?
Also, if it's server, do you run it as root or a different operating system user?
Regards,
William

I'm using the OmniDB app and the previous version was 2.16.

Hi @Leonardofreua ,
Two days ago we released 3.0.1b with an improvement to the automatic migration process. If you install the newest beta package now, it should migrate your users, connections, snippets and monitoring units from 2.17.
We are preparing release of 3.0.2b, it should be out later today.
Best regards,
William

Ok, I will install this version. Thanks for attention.

Hi all,
I'm closing this issue now that the fix is available on the latest version.
Please, feel free to open new issues. Thanks for your contribution.
Best regards


docs: add Simplified Chinese version

add Simplified Chinese version.

Hi @geminiyellow this looks great, and can you squash it down into a single commit?

@wsargent ok, zipzipzip

Thanks!

@geminiyellow are you okay with being added as a contributor so you can merge your own commits to the chinese doc?

@wsargent oh, my honor .

@geminiyellow okay done!


connect

./util/connect ch1 ch2 leads to a connection hanging unconfirmed in ch2 while it appears to be confirmed (permissions given) in ch1. Issuing the same command the other way round seems to do nothing.

Issuing the same command the other way round (./util/connect ch2 ch1) now removed the connection from ch2 while it still appears confirmed in ch1. Not good.
Tried ./util/connect ch1 ch2 again but nothing happend in ch2. Still looks confirmed in ch1.

OK, this one is going to need some coffee to sort out the bidirectional stuff. That will probably have to wait until tomorrow morning.

Well I had a look and it's working fine for me. I connected ch1 to ch2. This showed ch1 as connected but with no permissions. Looking at ch2, ch1 was 'pending' (awaiting approval). That's the way it should look. Then I connected ch2 to ch1. At that point both showed as connected from each channel with normal permissions in each direction and neither was pending.


Add ERC: Smart Blobs

https://ethereum-magicians.org/t/erc-7689-smart-blobs/19672

@SamWilsn we've made revisions based on your feedback. Would be great if you could re-review. Cheers!

Agreed, this probably should be abstracted from arweave per se, but maybe just generalized "permanent storage network with probabilistic DA guarantee". winston_byte_price probably should be storage_network_byte_price or so.
I think arweave belongs much more to "implementation detail". In fact, the more mediums you store/fetch historical data from - the more secure overall Smart Blobs scheme becomes.

Just checking in to see if you're still interested in pursuing this proposal.


New version: REINERSCT.cyberJackDriverPackage version 1.4.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue? #186061

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@microsoft-github-policy-service agree

@wingetbot run

Automatic Validation ended with:
(Automated response - build 897.)

Automatic Validation ended with: Edit:

Edit: Hi @nimra98,
Our pipeline uses 32-bit OSes to validate x86 packages.
I meant to suggest a change, but forgot to click "Start a review".

Thanks for the help. I missed that the installer now only supports x64 Win 10 and x64/arm64 Win11, as I only copied the old entry and modified the installer location.
I also added the arm64 for Win11 and hope the pipeline runs through.


Add support for with_repeated_options and with_arguments

Per our discussion on the RubyTerraform RFC - add support for with_repeated_options and with_arguments as well as incorporate the conditional logic into with_option to ignore switches with no value.

Thanks for the review @jonassvalin.
I'm happy to merge this now and make the small changes. That way we can get a release out for @sa73917 to continue the ruby_terraform PR.
Thanks again all.


Fix false positive for style.css

Previously any path with style.css at the end got matched, including editor-style.css. Not ideal, this patch fixes that including converting line endings.

Pull Request Test Coverage Report for Build 69

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

Totals

Change from base Build 67:
0.0%

Covered Lines:
1647

Relevant Lines:
1647

💛  - Coveralls


Support for Elliptic Curve algorithms

Hi,
is there a good reason, why the ES* algorithms aren't in the list of available algorithms (tokens.py). They are creating significantly smaller JWTs compared to the RS* algorithms. A quick test showed me that just adding them as "allowed" works out of the box.
I am happy to create a pull request in case there is no technical/security reason why this shouldn't be supported in the first place.

one way to "circumvent" this situation is:
import rest_framework_simplejwt.backends
rest_framework_simplejwt.backends.ALLOWED_ALGORITHMS += ('ES256',)

having cryptography installed this will allow the usage of ES256

Duplicate #170

(Not a security expert). It seems like ECDSA is just not preferred due to needing randomness whereas RSA is deterministic:
https://tools.ietf.org/html/rfc6979

Thanks and sorry.. I didn't find the issue #170 when I searched for that.


Clean up asset folder

It seems to me that the asset folder is outdated. I don't actually know what is what. @sophieeng, could you check for every item whether we actually currently use it (in the extension, readme, store listing, elsewhere), and delete those items we do not use.
I already removed the Privacy Without Compromise screenshot from the repo and the Firefox Store because that landing page no longer exists.

I created a new branch and deleted useless items. The extension is still working properly. I am not sure how I can check the store listing. 'OptMeowt_Chrome_Web_Store.psd' seems to be something that we use in the store listing, so I did not delete it. It is also possible that I might delete items that are used in the store listing.

Here is what is used in the Chrome Web Store.

So, it does not look like it is used there. 'OptMeowt_Chrome_Web_Store.psd' is a Photoshop file, probably, more for the icon.png. So, let's rename it to icon.psd in  case anyone wants to modify the icon going forward.
Also, if you could make sure that nothing breaks in the readme. Because we have some assets in there as well.
Then, feel free to merge.

Actually Sophie updated chrome-firefox-store-assets very recently. The store listing asset folder should be fine.
Everything looks fine now after cleaning. I am closing this.


Personally identifiable information in message exception.

Is it possible to avoid including personally identifiable information (PII) in exception messages?
When a unique constraint violation SQLException occurs, the exception message may contain PII such as email addresses or logins. Entity Framework logs these messages, and there's no built-in way to prevent this. Is there a way to exclude columns containing PII when generating the SQLException message?
Example of what i get:
Microsoft.Data.SqlClient.SqlException (0x80131904): Cannot insert duplicate key row in object 'dbo.Users' with unique index 'IX_Users_Email'. The duplicate key value is<EMAIL_ADDRESS>
Example of what i want:
Microsoft.Data.SqlClient.SqlException (0x80131904): Cannot insert duplicate key row in object 'dbo.Users' with unique index 'IX_Users_Email'.

Thank you for reporting this issue. At this time, the Microsoft.Data.SqlClient driver does not prevent personally identifiable information inside the SqlException message coming from the Sql Server to be displayed at the client application,
It might be possible to avoid including personally identifiable information (PII) in exception messages by using the dynamic data masking and masking policy of the Sql Server.

Also please note that many of these error messages are transmitted from the server.
The client faithfully surfaces them, without looking at the content.


[Feature] Add the option to automtically collapse large messages

Motivation
Large messages (read about ~100MB) are fetched and displayed as is by MQTTX.
Not only is this not practical but it also makes the application lag and even crash.
Detailed design
There should be a limit for the maximum length for messages being displayed.
If that limit was exceeded then the message should be automatically collapsed.

Thank you for bringing this to our attention. We hadn't considered the display issue with large messages before. I will take your suggestion into account and work to implement it as soon as possible.

Thanks @Red-Asuka !

@TommyDuc Hi, before we optimize this, could I ask you two questions? What is the scenario that requires such a large size of data? What is the current performance and show of MQTT X when receiving and sending this large amount of data? We want to be able to reproduce this when optimizing fully.

@ysfscream

The scenario is that the MQTT broker is used to transfer files (read firmware updates). We know that MQTT was not meant to transfer files but it does support messages up to 256MB :) and so it was quick and cheap way to do it.
For uploading I haven't tried it but when file-messages, which are often retained, then opening MQTT X on the file topic causes it to freeze for multiple minutes or even crash.

OK. Thanks for your details. We will optimize this data display, but you can wait for the large file transfer function of EMQX, which will work very well for you!

It's updated to https://github.com/emqx/MQTTX/releases/tag/v1.10.0, welcome to try and thanks again.


Update-Pax8SubscriptionById

Not sure whether this is an issue with the Module or Pax8 but I cannot use the Update-Pax8SubscriptionById. I get the below error:
Invoke-Pax8MasterRequest : The remote server returned an error: (500) Internal Server Error.
I am using the latest version of the PS Module.

That's typically an issue with the REST server. Can you make the call manually? Use the code snippet below with your values:
# Update these variables with your values
$accessToken = "Your access token here"
$subscriptionId = "Your subscription Id here"

$headers = New-Object "System.Collections.Generic.Dictionary[[String],[String]]"
$headers.Add("Content-Type", "application/json")
$headers.Add("Authorization", "Bearer $accessToken")

# Update the body to reflect your values
$body = @"
{
    "quantity":11,
    "startDate":"2020-02-10",
    "endDate":"2021-08-24",
    "price":15.39,
    "billingTerm":"Monthly",
    "provisioningDetails":[
        {
            "key":"userEmailAddress",
            "values":[
<EMAIL_ADDRESS>            ]
        }
    ]
}
"@

$response = Invoke-RestMethod "https://api.pax8.com/v1/subscriptions/$($subscriptionId)" -Method 'PUT' -Headers $headers -Body $body
$response | ConvertTo-Json

Hi, thanks for the help - when I call it manually I get the same error. I assume I will need to go back to Pax8?

No problem. Yeah, it sounds like it could be an issue on their end so I'd reach out to them. Let me know if I can help with anything else once you hear back.

Before I could raise a ticket with Pax8 the code both the REST code and PowerShell module have both started working so all sorted now - thanks for the help!


Latest to the latest build conventions and workflows

Reviewer checklist

[ ] Read the contributing guide
[ ] PR should be motivated, i.e. what does it fix, why, and if relevant, how
[ ] Ensure relevant issues are linked (description should include text like "Fixes #")
[ ] Ensure any appropriate documentation has been added or amended

Coverage: 100.0%. Remained the same when pulling 6b4ad1b55c9c5a18e1ed191b962c74caa4aa80ed on conventions into 923963d81ad283d14502bbf67ad2c6f1cfa564d8 on main.


Add mp4 option for when downloading from streaming sites like Disney+

Is your feature request related to a problem? Please describe.
I'm aiming to download some movies from Disney+, but it downloads them as encrypted .ts files. I need that they be downloaded as mp4 files.
Describe the solution you'd like
Described above.
Describe alternatives you've considered
Idk how i can decrypt an encrypted .ts  movie files downloaded via this your program.
Additional context
Thanks for solving this issue.

Did you ever solve this? I also want to download some videos from Disney+. I've tried lots of downloader programs but they all cost and I can't find a good crack that works.

Nope, not yet, sadly.
If you have a discord.com account, tell your username here and a friend i know of can help you with this your task.
Once you showed your discord username here, i'll tell him to add and contact you in there.

Thanks, but I have actually found something that works relatively stable. So far I've downloaded 3 seasons with relatively few trouble. Only downside is that it takes around 30 min per episode (20 min). It's not the download but the decryption that takes so long.
But I'm fine with that if I eventually get my series.

The first step would be to add the ability to save/convert files in mp4 format first though, as requested here.


[TODO] Installer + Auto Update

Use of InstallForge to create an installer that will add the files necessary for the application to function properly into C:\Program Files (x86)
Have a permanent storage.
Be able to automatically download the last update using a script.

I found some interesting infos in here :

Host all releases in Source Forge
Updates folder
updatescript.ini file
to move releases in Updates folder, we can connect via sftp command shell
> sftp<EMAIL_ADDRESS>> cd frs/project/PROJECTNAME #lowercase
> rename target dest


Removed unused method from ExternalsUtil

Coverage increased (+0.3%) to 77.275% when pulling 9e8b0c3cce7c08fc8e3b3c2aa7763d3d2f3d12b8 on some-fixes-mikko into 92a5fc03d52a0da90c62a298ba21aa6d26b6e2d1 on master.


Validation Error importing OpenAPI planner when OpenAI credentials not in environment

System Info
Name: langchain, Version: 0.0.180
Name: openai, Version: 0.27.7
macOS Mojave 10.14.6
Who can help?
@vowelparrot
Information

[X] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[X] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
Steps:

Do not load open ai key into env with the intention of wanting to pass it as a parameter when instantiating the llm

from dotenv import dotenv_values
openai_api_key = dotenv_values('.env')['OPENAI_API_KEY']

Load the planner:

from langchain.llms.openai import OpenAI
from langchain.agents.agent_toolkits.openapi import planner

Expected behavior
A validation error should not be raised during the importing of the module.
We should be able to pass the open api key as an argument.
That is, the following should work:
from langchain.llms.openai import OpenAI
from langchain.agents.agent_toolkits.openapi import planner

llm = OpenAI(model_name="gpt-4", temperature=0.0, open_api_key=open_api_key)

Thanks for raising this issue - we'll get a fix in for the next release


Create Initial Data Hub Kotahi Docmaps API

https://github.com/elifesciences/data-hub-issues/issues/775

Is this PR still relevant?


fixes, tests, CI

Codecov Report

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

@@          Coverage Diff           @@
##             main      #9   +/-   ##
======================================
  Coverage        ?   4.53%
======================================
  Files           ?      30
  Lines           ?    2182
  Branches        ?       0
======================================
  Hits            ?      99
  Misses          ?    2083
  Partials        ?       0

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


Formula realization

x = sqrt_alphas_cumprod[t_start] * (2*x-1) + sqrt_1m_alphas_cumprod[t_start] * torch.randn_like(x)
Is the code here corresponding to that formula? Looking forward to your reply.

Hi @ConanMusk ,
you can find relevant discussion regarding t_start at #23.
In short, this is an initialization strategy.

May I ask where in your deblur code you introduced the conditionals, it seems that I didn't find out how you introduced the conditionals. Looking forward to your reply.

May I ask where you implemented the code implementation of these formulas? Looking forward to your reply.

HI @ConanMusk

May I ask where in your deblur code you introduced the conditionals, it seems that I didn't find out how you introduced the conditionals. Looking forward to your reply.

We first construct the paired dataset from HR images and a given degradation operator (blurring here);
in the sampling process, we do precalculation on condition LR $y$ https://github.com/yuanzhi-zhu/DiffPIR/blob/main/main_ddpir_deblur.py#L236
then solve the data subproblem: https://github.com/yuanzhi-zhu/DiffPIR/blob/main/main_ddpir_deblur.py#L284

May I ask where you implemented the code implementation of these formulas?

https://github.com/yuanzhi-zhu/DiffPIR/blob/main/main_ddpir_deblur.py#L344
https://github.com/yuanzhi-zhu/DiffPIR/blob/main/main_ddpir_deblur.py#L346


"contributors" is spelled wrong in GH team name

https://github.com/orgs/hackedu/teams/contributers
should be "contributors", not "contributers"

Updated: https://github.com/orgs/hackedu/teams/contributors
Thanks @tejasmanohar!


Title: Add new project - Bitwise Calculator

Is your feature request related to a problem? Please describe.
Bitwise operations are very frequent when it comes to electronics engg, Competitive programming, physics, mathematics and many more areas, Creating one such calculator by own self can test JS skills of basic calculations; and also help users in many ways
Describe the solution you'd like.
Bitwise calculator which Calculates results of different bitwise operations based on inputs. AND NAND OR NOR XOR XNOR ; All these operations will be available...
Describe alternatives you've considered.
I have searched online calculations of bitwise operations iteratively many times
Add any other context or screenshots about the feature request here.
No response

@debangi29 Thank you for providing your insight but it is not required as of now

Okay , no problem
Thank you


Add VM instruction logging option

What current issue(s) does this address, or what feature is it adding?
A continuation of #294 , specifically point 3

add instruction logging
rationale; having a historical log of all calls executed gave me nearly all the information needed to exactly pin point which code line of my SC the VM was executing (by looking at function calls, DROP/RET statements etc).

How did you solve this problem?
Add some file writing statements to ApplicationEngine/ExecutionEngine and StateReader.
How did you make sure your solution works?
write tests + make test
Are there any special changes in the code that we should be aware of?
no
Please check the following, if applicable:

[X] Did you add any tests?
[X] Did you run make lint?
[X] Did you run make test?
[X] Are you making a PR to a feature branch or development rather than master?
[X] Did you add an entry to CHANGELOG.rst? (if not, please do)

Note: I specifically choose to implement the file writing in the Execute() function over the StepInto() function because the latter would mean you'd get hundreds of file open/closes per script execution (literally for every instruction).

Coverage decreased (-0.03%) to 78.202% when pulling 2357acf6ac129d7da021572f41ac4c28936a0744 on ixje:sc_debug_2 into 0e3c324b4699d421fe645f3deb3a0b9aff765743 on CityOfZion:development.

The concern raised was speed, which is indeed important to keep in mind. Of course there's some impact to code additions, therefore I did a basic optimisation on the calls that get called every instruction. This yielded a 9% improvement to offset the <2% loss by the new functionality.

Looks good! Tests seem to be wanting timeit from the neo.Utils module

This will be very helpful for many people until we get some actual debugging in place!


Clean up redstone-power related data and properties

Currently, redstone power levels are represented in the API through a combination of PoweredData, RedstonePoweredData, PoweredProperty, and IndirectlyPoweredProperty.
PoweredData and RedstonePoweredData are not clearly defined as to whether they deal with input or output power. As a consequence, they are implemented differently depending on the block, making it difficult to determine what they actually mean.
PoweredProperty and IndirectlyPoweredProperty do function properly, but their usefulness is greatly decreased by the fact that they deliberately hide the actual value, instead only returning a boolean.
As it stands currently, the represntation of redstone power in the API needs a general overhaul, both to improve usability and clearly define what is actually being represented.

I've had a go at this here, https://github.com/SpongePowered/SpongeAPI/compare/feature/clean-redstone
I feel PoweredData makes sense, because it states if a lever, button etc is currently in its powered state.
As for RedstonePoweredData, I changed that to RedstoneOutputData, as that's what it signifies.
As for the two properties, I changed them to integers and clarified in the javadocs what they do, and that they aren't applicable to all blocks.


SLOW

The speed is too slow, disable syntax highlighting is also slow. Are there other ways to speed things up?

That's a bit surprising if it's still slow after disabling syntax highlighting. Which terminal app do you use? Some terminals handle colors and fonts much better. If you're using iTerm in particular, one perf issue I had was when I had "Use ligatures" checked in the font settings that made text rendering much slower.


Security token does not match. You must login again...

Hi,
After running the container and try to access the management console am getting the below error,
Security token does not match. You must login again...

Unable to repeat this case


fix fix test_numbers case

bp: #38687
we use order_qt rather than qt to promise the order of results.

run buildall


chore: grouping ALL common modules in one big chunk

SUMMARY

There was a concern that there were TOO MANY chunks generated by webpack. This PR undoes some custom "vendors" chunking, instead going with Webpack's default of grouping shared modules in node_modules. The result? One Giant Chunk! This might take the browser longer to parse, but it might be more highly cached, and reduce loading when moving around between Superset's react apps. All in all, it removes about 100 other chunk files. We could set a max chunk size and break it down into a few for parallel loading, but right now, this PR is more of a discussion point.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Bundle analyzer screengrabs to come...
TESTING INSTRUCTIONS

I'll spin up an ephemeral environment, and people can poke around at it in a browser and take a look.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Thanks for PRing to start the conversation! @rusackas could you link some screenshots with the bundle size and count before and after?


The title bar of projects looks draggable

We fixed the fact that you could drag to resize in #77, but if you hover over the line the cursor still changes to a drag-handle.

We can hide the drag-bar completely (and create a non-draggable replacement if we want the line there). Does that sound like a good solution to you? And if so, should we make a replacement bar?

Never mind, - splitView:shouldHideDividerAtIndex: isn't actually for hiding the divider, but is used to determine the behavior when it goes off screen. I should, however, be able to disable dragging completely using - splitView:effectiveRect:forDrawnRect:ofDividerAtIndex:.

:clap: :clap: :clap:


[Bug]: vllm/vllm/_C.cpython-310-x86_64-linux-gnu.so: undefined symbol error

Your current environment
The output of `python collect_env.py`

Collecting environment information...
PyTorch version: 2.1.2+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.3 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.27.6
Libc version: glibc-2.35

Python version: 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-5.15.0-73-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 12.2.140
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA GeForce RTX 3090
Nvidia driver version: 535.104.05
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.9.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.9.5
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Address sizes:                   46 bits physical, 57 bits virtual
Byte Order:                      Little Endian
CPU(s):                          64
On-line CPU(s) list:             0-63
Vendor ID:                       GenuineIntel
Model name:                      Intel(R) Xeon(R) Gold 6326 CPU @ 2.90GHz
CPU family:                      6
Model:                           106
Thread(s) per core:              2
Core(s) per socket:              16
Socket(s):                       2
Stepping:                        6
CPU max MHz:                     3500.0000
CPU min MHz:                     800.0000
BogoMIPS:                        5800.00
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 invpcid_single intel_ppin ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb intel_pt avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local split_lock_detect wbnoinvd dtherm ida arat pln pts avx512vbmi umip pku ospke avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg tme avx512_vpopcntdq la57 rdpid fsrm md_clear pconfig flush_l1d arch_capabilities
Virtualization:                  VT-x
L1d cache:                       1.5 MiB (32 instances)
L1i cache:                       1 MiB (32 instances)
L2 cache:                        40 MiB (32 instances)
L3 cache:                        48 MiB (2 instances)
NUMA node(s):                    2
NUMA node0 CPU(s):               0-15,32-47
NUMA node1 CPU(s):               16-31,48-63
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Mmio stale data:   Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Enhanced IBRS, IBPB conditional, RSB filling, PBRSB-eIBRS SW sequence
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected

Versions of relevant libraries:
[pip3] numpy==1.22.2
[pip3] onnx==1.14.0
[pip3] pytorch-quantization==2.1.2
[pip3] torch==2.1.2
[pip3] torch-tensorrt==0.0.0
[pip3] torchdata==0.7.0a0
[pip3] torchtext==0.16.0a0
[pip3] torchvision==0.16.0a0
[pip3] triton==2.1.0+e621604
[conda] Could not collectROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: N/A
vLLM Build Flags:
CUDA Archs: 5.2 6.0 6.1 7.0 7.2 7.5 8.0 8.6 8.7 9.0+PTX; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    NIC0    NIC1    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      NODE    NODE    0-15,32-47      0               N/A
NIC0    NODE     X      PIX
NIC1    NODE    PIX      X

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

NIC Legend:

  NIC0: mlx5_0
  NIC1: mlx5_1

How you are installing vllm
pip install -i http://ftp.daumkakao.com/pypi/simple --trusted-host ftp.daumkakao.com -e .

I tried to import vllm,but it failed.
python
Python 3.10.12 (main, Jun 11 2023, 05:26:28) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import vllm
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/sjlim/workspace/vllm/vllm/__init__.py", line 3, in <module>
    from vllm.engine.arg_utils import AsyncEngineArgs, EngineArgs,
  File "/home/sjlim/workspace/vllm/vllm/engine/arg_utils.py", line 6, in <module>
    from vllm.config import (CacheConfig, ModelConfig, ParallelConfig,
  File "/home/sjlim/workspace/vllm/vllm/config.py", line 9, in <module>
    from vllm.utils import get_cpu_memory
  File "/home/sjlim/workspace/vllm/vllm/utils.py", line 8, in <module>
    from vllm._C import cuda_utils
ImportError: /home/sjlim/workspace/vllm/vllm/_C.cpython-310-x86_64-linux-gnu.so: undefined symbol: _ZN2at4_ops19empty_memory_format4callEN3c108ArrayRefINS2_6SymIntEEESt8optionalINS2_10ScalarTypeEES6_INS2_6LayoutEES6_INS2_6DeviceEES6_IbES6_INS2_12MemoryFormatEE

Is it on an nvidia container which has transformers-engine package in it? If yes, uninstall it using pip and then try

Thanks for letting me know.
But the problem now is that the pip install ends successfully, but when I import, I get the same problem as above. I will fix the tag as bug.

我也遇到了一样的问题

When you install vllm using pip, you might frequently run into errors related to 'undefined symbol', often because of conflicting versions with pytorch or something.
It's safer to build vllm from the source.
https://github.com/vllm-project/vllm/issues/129#issuecomment-1805088950

I built it from the source. still i get the undefined symbol error
root@b0da1a1ca0fd:~# python3
Python 3.10.13 | packaged by conda-forge | (main, Dec 23 2023, 16:04:32) [GCC 12.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import triton
>>> import vllm
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/root/vllm/vllm/__init__.py", line 4, in <module>
    from vllm.engine.async_llm_engine import AsyncLLMEngine
  File "/root/vllm/vllm/engine/async_llm_engine.py", line 12, in <module>
    from vllm.engine.llm_engine import LLMEngine
  File "/root/vllm/vllm/engine/llm_engine.py", line 16, in <module>
    from vllm.model_executor.model_loader import get_architecture_class_name
  File "/root/vllm/vllm/model_executor/model_loader.py", line 10, in <module>
    from vllm.model_executor.models.llava import LlavaForConditionalGeneration
  File "/root/vllm/vllm/model_executor/models/llava.py", line 11, in <module>
    from vllm.model_executor.layers.activation import get_act_fn
  File "/root/vllm/vllm/model_executor/layers/activation.py", line 9, in <module>
    from vllm._C import ops
ImportError: /root/vllm/vllm/_C.cpython-310-powerpc64le-linux-gnu.so: undefined symbol: cuPointerGetAttribute

build the container with DOCKER_BUILDKIT=0, the probelm will be solved. (building needs gpu)

Apologies for the delayed response. I was able to resolve the issue after reinstalling NVIDIA-Docker :)


[charts/redis-ha] Removing PSP & Adding Kyverno adjustments to securityContexts.

What this PR does / why we need it:
Adds the necessary adjustments to deprecate PSP and instead use appropriate securityContext options for containers/initContainers and tests.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #204

Special notes for your reviewer:
@joebowbeer @pierluigilenoci Please triple check and confirm you're happy with these changes. These are inline with the policy defined here: https://github.com/kyverno/policies//pod-security
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

On second look, a couple of questions and a suggestion
Where is the Dockerfile for the default haproxy image?
The ArgoCD version runs as user 1000, but it runs the alpine-based image, which has a default nonroot user of 1000.
https://github.com/argoproj/argo-cd/blob/master/manifests/ha/base/redis-ha/chart/upstream.yaml#L1090
Suggestion: Remove the duplicated (pod) securityContext elements from the container securityContext values. That is, if the pod declares runsAsUser 1000 then remove the dup setting from the container.

@joebowbeer https://github.com/docker-library/haproxy/blob/master/Dockerfile.template#L13
UID 99 apparently. I'll apply and test with that adjustment.
I'll remove the dupes on the containerSecurityContext in a sec.

@DandyDeveloper wrote:

UID 99 apparently. I'll remove the dupes on the containerSecurityContext in a sec.

If the different containers (redis vs haproxy) are running as different users then I suggest moving their user/group ids out of the PodSecurityContext and into their dedicated containerSecurityContext.

@joebowbeer There's some need for duplicate fields between securityContext and containerSecurityContext because of the test pods using containerSecurityContext.
Just FYI there are two securityContext fields in the values. One for HAProxy and another for Redis.
As far as I can tell, this is "as good" as we'll get it without me spending unnecessary time on edge cases / conditionals in the test pods.

As far as I can tell, this is "as good" as we'll get it without me spending unnecessary time on edge cases / conditionals in the test pods.

np. I was thinking it would be better in some sense to move { runAsUser: 99, fsGroup: 99 } into the container-specific context because this is the only part that actually changes per Dockerfile container, and it would avoid any confusion due to unintended inheritance. But really whatever works is great.

@joebowbeer Yeah I 100% understand. Can you provide another review if you're happy to proceed?

Uh... Woops.


⚠ Bofrost.de has degraded performance

In 868918b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1003 ms

Resolved: Bofrost.de performance has improved in 8b446b0 after 19 minutes.


Trained model for HTC ResNet 50 with DCN

Hello, I would like to ask if it is possible to ask to train the model HTC ResNet 50 with DCN on the COCO dataset and add a trained model to Model Zoo, since the capabilities of my graphics card do not allow me to do this quickly. I would be very grateful, thanks.

We try the best to maintain the codebase but sorry that we cannot help training additional models, which is out of our resources.


Quartz format as mentioned in README is not fully supported

jobs are not getting registered when those three values are used in schedule:
The ‘L’ character is allowed for the day-of-month and day-of-week fields. This character is short-hand for “last”, but it has different meaning in each of the two fields. For example, the value “L” in the day-of-month field means “the last day of the month” - day 31 for January, day 28 for February on non-leap years. If used in the day-of-week field by itself, it simply means “7” or “SAT”. But if used in the day-of-week field after another value, it means “the last xxx day of the month” - for example “6L” or “FRIL” both mean “the last friday of the month”. You can also specify an offset from the last day of the month, such as “L-3” which would mean the third-to-last day of the calendar month. When using the ‘L’ option, it is important not to specify lists, or ranges of values, as you’ll get confusing/unexpected results.
The ‘W’ is used to specify the weekday (Monday-Friday) nearest the given day. As an example, if you were to specify “15W” as the value for the day-of-month field, the meaning is: “the nearest weekday to the 15th of the month”.
The ‘#’ is used to specify “the nth” XXX weekday of the month. For example, the value of “6#3” or “FRI#3” in the day-of-week field means “the third Friday of the month”.

Yeah, sorry, we don't fully support the format Quartz, we only support the first parameter for the optional seconds.


Remove bare except

Fixes #23

The changes are already covered up by the current tests, but i guess i can add a test for the corrected_hour_angle


Materials with OpenSimplexNoise won't load in 4.0alpha 6

Godot version
4.0 alpha 6
System information
Linux Manjaro, Vulkan, Intel Kaby Lake
Issue description
Title.
This is understandable because OpenSimplexNoise was removed in favor of FastNoiseLite, but without some sort of an automatic conversion - or tips on how to fix the .tres file - this is a problem.
Steps to reproduce
Sample file attached.
Minimal reproduction project
road_shadermat.tres.zip

From a quick search:

A typical use case using IntelliJ or Android Studio IDE with gradle build tool. If you have a project that uses JDK 14 with gradle 6.3 and you upgrade the JDK to version 16 then you get the error Unsupported class file major version 60 because gradle 6.3 is not compatible with Java 16. So the root cause is independent from your IDE. It can be also Eclipse, Visual Studio Code or any other.

The solution is to use comaptible JDK and gradle versions. See Java major numbers and gradle compatibility matrix.


URGENT: pac4j jars disappeared from maven central !

see https://repo1.maven.org/maven2/org/pac4j/pac4j/3.6.1/
same for any other pac4j version.
P.S. other libs still have jars:
https://repo1.maven.org/maven2/org/apache/spark/spark-core_2.11/2.4.3/
so we get build failure on a fresh jenkins branch, which do not have cached jars:
....
[warn] 	[FAILED     ] org.pac4j#pac4j;3.6.1!pac4j.jar: invalid sha1: expected=<!doctype computed=3a08b17598cc540c37049370768c2d42c88ece7e (600ms)
[warn] 	Detected merged artifact: [FAILED     ] org.pac4j#pac4j;3.6.1!pac4j.jar:  (0ms).
[warn] ==== local: tried
[warn] ==== public: tried
[warn]   https://repo1.maven.org/maven2/org/pac4j/pac4j/3.6.1/pac4j-3.6.1.jar
[warn] ==== Shibboleth: tried
...
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	::              FAILED DOWNLOADS            ::
[warn] 	:: ^ see resolution messages for details  ^ ::
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[warn] 	:: org.pac4j#pac4j;3.6.1!pac4j.jar
[warn] 	::::::::::::::::::::::::::::::::::::::::::::::
[error] sbt.librarymanagement.ResolveException: download failed: org.pac4j#pac4j;3.6.1!pac4j.jar

@leleuj

Indeed, that's very strange! Let's wait a few minutes to see if it comes back. If it doesn't, I'll try to reach the guys from Maven...

it disappeared around or before yesterday evening. so at least 15h so far.

interesting,  https://mvnrepository.com/artifact/org.pac4j/pac4j/3.6.1 now lists pac4j as pomOnly, so as a aggregate of dependencies without a jar:
// https://mvnrepository.com/artifact/org.pac4j/pac4j
libraryDependencies += "org.pac4j" % "pac4j" % "3.6.1" pomOnly()

maybe the jar shouldn't actually exist (as other jars exist, like pac4j-core.jar ), and maven central just did some cleanup? or is it the inverse - maybe this pomOnly shown as a result of some fuckup on maven central?
@leleuj

Fixed by marking "org.pac4j" % "pac4j" % "3.6.1" pomOnly() as pomOnly()

Ah yes, it was the pom dependency! pac4j-* dependencies are still there. You shouldn't pull the pac4j dependency, it doesn't make any sense, you should pull pac4j-oauth, pac4j-oidc, ...


[Dailymotion] Support password-protected videso (was: Dailymotion video 'title' extraction error)

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 that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.06.14. 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 2016.06.14

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)

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 -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
chris@zbox3:~$ youtube-dl -i -k --write-sub --sub-lang en --sub-format srt -u givemebaozi -p kyss http://www.dailymotion.com/video/x4gh4bl_eng-sub-yandai-byway-ep5-p2-2_fun --verbose
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-i', u'-k', u'--write-sub', u'--sub-lang', u'en', u'--sub-format', u'srt', u'-u', u'PRIVATE', u'-p', u'PRIVATE', u'http://www.dailymotion.com/video/x4gh4bl_eng-sub-yandai-byway-ep5-p2-2_fun', u'--verbose']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.06.14
[debug] Python version 2.7.11+ - Linux-4.4.0-24-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 2.8.6-1ubuntu2, avprobe 2.8.6-1ubuntu2, ffmpeg 2.8.6-1ubuntu2, ffprobe 2.8.6-1ubuntu2, rtmpdump 2.4
[debug] Proxy map: {}
[dailymotion] x4gh4bl: Downloading webpage
ERROR: An extractor error has occurred. (caused by KeyError(u'title',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 342, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/dailymotion.py", line 145, in _real_extract
    self._check_error(metadata)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/dailymotion.py", line 277, in _check_error
    '%s said: %s' % (self.IE_NAME, info['error']['title']), expected=True)
KeyError: u'title'
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 676, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 348, in extract
    raise ExtractorError('An extractor error has occurred.', cause=e)
ExtractorError: An extractor error has occurred. (caused by KeyError(u'title',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.
The Problem is reported as "ERROR: An extractor error has occurred."
User and password are as shown in the command line.
The video plays through the dailymotion website OK.
Thank you for looking at this problem.
Chris.

Password-protected videos on Dailymotion are not supported yet.

Hi,
I have user-name and password protected files that youtube-dl downloaded from Dailymotion in April 2015 still on my hard drive. This is why I didn't expect any problem downloading this file.
Has Dailymotion changed their setup so that youtube-dl is no longer able to access these sorts of files?
Regards,
Chris.

Has Dailymotion changed their setup

I'm not sure. Maybe you're right.

Thanks for redirecting me to this issue, I understand the problem now. Is there any estimation of when or indeed whether password-protected Dailymotion videos are expected to be supported?


ODBC and MFC - thinks ODBC 13 is the latest.

This article states that the Microsoft ODBC Driver 13 for SQL Server is the latest driver.
Is that true? I've successfully been running a highly complex CDatabase/MFC monolith with ODBC 17 and ODBC 18 drivers.
Should this article link to a version agnostic ODBC Driver page so that it's less brittle in future?
Or is there really a compatibility issue between MFC that restricts it to the 13 driver?

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

ID: e37d6a9d-d5ba-6246-063f-ede88395ace1
Version Independent ID: 8c8ae57a-7f08-799b-7118-0cc7c1b8281b
Content: ODBC and MFC
Content Source: docs/data/odbc/odbc-and-mfc.md
Product: visual-cpp
Technology: cpp-data
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Closing as part of backlog grooming. We have accrued more issues than our resources can manage.
Criteria for closing include severity of the issue, how broadly applicable it is, whether the issue enough information is provide to act on the issue, etc.
If you think this issue should not have been closed, reopen it with a comment explaining why.


KubernetesCommandExecutor throw an exception if pod exceeds expire of TemporalConfigStorage

KubernetesCommandExecutor now throws an exception when the execution time of a pod exceeds the defaultPodTTL.
But we also need to consider that the pod execution time may exceed expiration date of TemporalConfigStorage.
This is because pod error detection can be done early.
For example, if we continue processing when the pod's execution time is beyond the expiration of TemporalConfigStorage, the curl command will raise an error when trying to upload the archive to the expired TemporaryConfigStorage at the end of the operation.
reopened https://github.com/treasure-data/digdag/pull/1359

Failed to follow the remote master, so I'll rebuild it with another PR.


Second getting started example is not correct anymore

Description
The current second example in the main readme with the created MLP is not correct anymore. During implementing this example with version 0.15 i saw, that the MLP constructor awaits at least 3 parameters (input, output, hidden) but in the current example only two are used
Will this change the current api? How?
No
Who will benefit from this enhancement?
New interested people who see this and will not be blocked
References

@TjayGs
Thanks you for pointing out. I create a PR to address this issue.


[BUG]: Unintentional player update after switching to a new architecture

Version
6.8.0
What platforms are you having the problem on?
Android
System Version
Android 13 (API Level 33)
On what device are you experiencing the issue?
Real device
Architecture
New architecture with interop layer
What happened?
On start/pause by changing state the player is reloaded: onLoad, onLoadStart events are fired.
Reproduction Link
repository link
Reproduction
Step to reproduce this bug are:
Reproducable if bufferConfig parameter is set (even empty) for <Video/> component

yep, already tracked here: https://github.com/TheWidlarzGroup/react-native-video/issues/4261
And a workaround was provided. Need to implement a clear fix for sure !


NPM Error

I can't get past the first step. I get a 404 with npm/yarn
npm ERR! 404 Not Found: protobufjs@https://registry.npmjs.org/protobufjs/-/protobufjs-5.0.2.tgz

Sorry, looks like it was an npm issue. https://github.com/dcodeIO/protobuf.js/issues/1056


empty tabs on android

Hi
first thanks for such a nice work. It works really fine on iOS but when I switched to Android, all the tabs are empty.
Here are some screenshot

Any idea ?
I search for old issues like this but could only find trouble about orientation or stuff like "when changin tabs everything is back" wich is not the case for me.
I should mention that nothing inside my views are statics, it's stuff loaded from my api.
Thanks
Best

Found it :
removing
"justifyContent: 'center',
alignItems: 'center'"
in the style of ScrollableTabView did the trick


[website] Refactor page context with next router

The issue is I got unexpected error when copy installation page to docs/pages/material/getting-started/installation (it should not throw error).
The error occurs in AppTableOfContent because there is no active page. However, it does not make sense that the AppTableOfContent should know about the active page. Instead, it should just read the browser URL (provided by nextjs) and apply a hash to the URL when clicked.
I feel that this is simpler and easier to maintain, so open this small PR to check that I don't miss anything important.

[x] I have followed (at least) the PR section of the contributing guide.

@oliviertassinari Do you see anything that I might be missing?

Actually, I can see why docs/src/modules/components/AppTableOfContents.js makes sense in isolation. It's more flexible. For the change docs/src/modules/components/Link.tsx I don't follow the reasoning. I would expect the pathname to never include the language, so the extra check to be a topology.

For the change docs/src/modules/components/Link.tsx I don't follow the reasoning. I would expect the pathname to never include the language, so the extra check to be a topology.

@oliviertassinari The change is correct because in AppTOC, the href use router.asPath (the URL) so it can contain $user_language/*. It makes sense for the Link to consider this case so other component does not need to think about languages.

@mnajdova Could you review this PR? thanks 🙏


Palindrome Checker use multiple languages

Can you tell me about , what the issue is this, so may be i can help in solving this issue..... ?

simple pallindrome checker between strings, number and all.. Use multiple coding language to implement

any updates?


Update CI.yml

update versions

Codecov Report

Merging #101 (cae2d32) into main (731f97b) will decrease coverage by 0.29%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #101      +/-   ##
==========================================
- Coverage   90.08%   89.79%   -0.29%
==========================================
  Files          29       29
  Lines         696      696
==========================================
- Hits          627      625       -2
- Misses         69       71       +2

Impacted Files
Coverage Δ

src/Elements/Node.jl
75.00% <0.00%> (-12.50%)
:arrow_down:

src/StructuralSolvers/StructuralSolvers.jl
87.93% <0.00%> (-1.73%)
:arrow_down:

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


Remove call comment in Ruby discovery sample gen

Currently Ruby compile check sample parsing relies on a "Perform the call" comment inconsistent with samples for other languages. It should be removed.

I've gone ahead and done this as part of the Ruby refactoring - hopefully you don't mind!

Super.


Scanning the root directory is allowed

The extension asks grype to scan the directory specified byvscode.workspace.rootPath.
It appears that certain circumstances (such as opening a live share link) result in this value being /, which leads grype to scan the entire filesystem.
Scanning / takes a long time, and doing so can result in an orphaned grype process if VS Code exits before the grype child process has completed its execution.
At a minimum, two actions should be taken:

vscode.workspace.rootPath is deprecated and should be replaced by a recommended method of accessing the workspace directory.
A guard should be included to prevent against grype being told to scan /.

Duplicate of #37


Vtol weathervane

Fixes: https://github.com/PX4/Firmware/issues/3679

After reviewing this further, I believe we should clamp the yaw control demand, instead of disabling it completely. So that the system does only spend a little effort in maintaining heading. In light wind it will then maintain it, in heavy wind it will weather-vane.

@LorenzMeier i agree, now it might be hard to understand what is going on in low wind conditions. How, and with what threshold should it disable control?

10-20% of the maximum yaw demand might make sense, that would be 0.1-0.2 (you have to limit -0.2..0.2, since its symmetric around zero). That matches the intent to not give up too much control over roll and pitch for yaw.

@LorenzMeier @sanderux I need to check the mixer if this would even make any sense. I believe yaw is mixed in at the very end so that it doesn't violate thrust, roll, pitch

@tumbili This discussion is not about yaw handling. Its about orienting the plane into the wind so that wind gusts do not completely ruin the position and attitude control response. Its very different from mixer limits.
We reserved a minimum for yaw. And it would put the motors at their maximum spread. It would mean that you would see the system constantly yaw, coupled with roll and pitch. We can't handle this on a best-effort basis in the mixer. We need to do this in the controller already.

This will apply only to QuadPlanes (classic plane with tail and rudder) not the quad delta wings, right?

@tubeme
It will apply to all VTOL types where the setting is enabled. if you have a standard vtol on delta wing i would suggest leaving the VT_OPT_WV_LND set to 0 (default).

@sanderux OK I understand. Thanks.

@LorenzMeier is this more what you had in mind?

Replaced by https://github.com/PX4/Firmware/pull/3706


check and equality methods don't verify optional parameters have the right shape/axes

write_calfits can produce files that read_calfits cannot read. The total quality array isn't being checked, so it can be the wrong shape.

This was fixed by PR #179.


feat: autocomplete built-in plugin names and locales

@tcampbPPU, is there anything holding us back from merging this PR (including the other one made by Daniel too)?


⚠ Bofrost.es has degraded performance

In 7441a87, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Bofrost.es performance has improved in 9563a77 after 9 minutes.


Fixed #25513 -- Extracted admin pagination to EllipsisPaginator.

Supersedes #5866 & #10328 for ticket-25513.
Notes on changes:

Pagination code in the admin is now 1-indexed. Backward incompatibility note added as requested. I think this will be less confusing in the long run, despite Carlton's reservations. It is unlikely that a bookmark to a specific page will show consistent results over time anyway.

DOT constant is now ELLIPSIS using '…' which makes it easier to use the correct character for display purposes and better fits with the new EllipsisPaginator class.

Calculations have been corrected to avoid inserting ELLIPSIS to replace one number, e.g. [1, 2, ELLIPSIS, 4, 5, …] should be [1, 2, 3, 4, 5, …]

Documentation has been updated and moved to the correct location below the documentation for Paginator.

Outstanding questions:

Should ELLIPSIS be moved to EllipsisPaginator.ELLIPSIS? Then it can be easily documented and used in code that iterates over the ellipsized page range.

Should we look to merge this into the base Paginator class? We can't add it to page_range as that is a property - we need to pass the current page number - so it'd just be adding the new method instead of defining it on a separate class.

Should we perform any validation on the values provided to on_each_side and on_ends?

"ellipsized" is a neologism, but the "ellipsed" used in a previous version of this doesn't quite feel right either. Any thoughts?

... despite Carlton's reservations.

Good. I'm glad that my initial response looking at this now was the same as last time. At least I'm consistent. 😀 — I shall try to convince myself that the behaviour change is OK here.

"ellipsized" is a neologism, but the "ellipsed" used in a previous version of this doesn't quite feel right either. Any thoughts?

I think the verb is to elide, so elided, even though it's not identical.

I think the verb is to elide, so elided, even though it's not identical?

So, yes, ellipsed is definitely not right. Thanks, elided fits much better. The only other options I can think of are "contracted" or "truncated". I'll change this.

I shall try to convince myself that the behaviour change is OK here.

I think that the existing behaviour here is unexpected. Being a lazy fellow who likes to use the keyboard and avoid the mouse, I sometimes will edit the location bar and append ?p=5 if I want to jump to the fifth page rather than scroll to the bottom of the page and click a link. Also, if the page is in the elided range there is no button available anyway. I've always assumed this would take me to the fifth page and not the sixth.
The only minor concern I can see here is whether users are bookmarking certain pages, but I cannot see this being robust in any case as search results are affected significantly by data insertions, updates and deletions, as well as other factors such as filtering and sort order.

Should we look to merge this into the base Paginator class?

Thinking more about this, I'm +1 for doing this if you agree. Essentially we are only adding an additional method and developers can simply change paginator.page_range to paginator.get_elided_page_range(number) to benefit immediately without needing to change the class.
The main question is whether we add the on_each_side and on_ends arguments to the __init__() as we currently do or, since get_elided_page_range() must be called with the current page number anyway, whether we simply add them as keyword arguments there and keep everything self contained?

Thinking more about this, I'm +1 for doing this if you agree....

OK, if you want to look at that, I won't object on principle. 🙂

...whether we simply add them as keyword arguments there and keep everything self contained?

This. Complicating __init__() on the base class for an edge behaviour is making everyone think about it for only a (small?) subset of uses. Optional kwargs on get_elided_page_range(), as you say, keeps everything self contained, so is better I think.

Ok. So I've separated out two commits to fix the admin pagination to be 1-indexed and to avoid elision of a single page number.
I've also condensed everything based on my comments above and expanded the tests significantly to cover all edge cases. It has turned out rather nicely. I think this is mostly there short of some tweaks to documentation and a few extra test assertions related to ensuring an iterator/generator is returned and the appropriate exceptions are raised.

Hey @pope1ni:

... a few extra test assertions related to ensuring an iterator/generator is returned and the appropriate exceptions are raised.

If you just want to add those we can push this in. (General outline is 👍 — Thanks for your work!)


Use "login" executable by default on OSX (was: zprofile and zshrc isn't sourced)

Description
~/.zprofile and $ZDOTDIR/.zshrc isn't sourced by default
Environment

Contour Version or Git commit hash: 0.1.1 132
Operating System (name / distribution / version): macOS(11.3.1)
Contour configuration: stock
TERM environment variable: xterm-256color truecolor
Compiler version: didn't compile from source

Steps to Reproduce
Have .zshrc under $ZDOTDIR and not under home and set ZDOTDIR to the directory .zshrc is in .zprofile

Don't you need to set $ZDOTDIR through .zshenv first? Can you give us examples on which terminals it works and/or can you give us your dotfiles?
Resources:
https://github.com/romkatv/zsh4humans/issues/2
https://zsh.sourceforge.io/Intro/intro_3.html#SEC3

@uspasojevic96 I have set ZDOTDIR in .zprofile, it seems to work under alacritty and iTerm
I tried checking if ZDOTDIR is set via echo $ZDOTDIR but it is empty in contour. I have set zdotdir to ~/.config/zsh
export ZDOTDIR="$HOME/.config/zsh"

So does contour not start login shell by default?

That's a question for @christianparpart , while we wait for him, can you try setting the env in .zshenv?

I set $ZDOTDIR in .zshenv and it says tmux(or other stuff) not found (maybe because I set all $PATH stuff in .zprofile), but on sourcing ~/.zprofile in ~/.zshenv it worked, but this feels like unnecessary hack because other terminals work without it

Ok, then that confirms it that zsh isn't spawned as login interactive shell

Also is it a known issue that font rendering is bad on macos or do I make another issue for that

It's known, HiDPI isn't working properly yet

* Contour Version or Git commit hash: 0.1.1 132

(I really need to release 0.2.0 damn. there are worlds between those two releases) :-)

Have .zshrc under $ZDOTDIR and not under home and set ZDOTDIR to the directory .zshrc is in .zprofile

Contour (I think like most (if not all) terminal emulators do by default start the executable that is configured as your default shell. So that's working apparently, great. This implies that it's not boot up as "login shell". This can be tweaked yourself by adding that to your profile:
profiles:
    your_profile:
        shell: "/bin/zsh"
        arguments: ["--login"]

EDIT:
Ok, found it. OS/X plays the extra-chicken and wants you to run login executable instead of bash --login or zsh --login (etc). So another way around would be to set shell: "login" with arguments: ["-fp", "YourUserName"]. Seems like all the other terminal emulators have suffered the same extra-chicken-game and worked around that. https://github.com/alacritty/alacritty/issues/1071 & https://github.com/kovidgoyal/kitty/issues/247 at least (and iTerm2 mentioned via screenshots).
This can be easily fixed (btw: without any deeper knowledge of VTE's :) )

Wait, I didn't notice the version he was running, that means that he isn't talking about HiDPI but about actual font issues that I used to face

I tried to grab a build from CI and it works without any workaround

I tried to grab a build from CI and it works without any workaround
Edit: I get this error when I added scientifica in font section, with other fonts it works fine

Lovely. We are still in early stages. So we would love to get feedback and bug reports as much as your time permits. Do, enjoy and many thanks for your interest. :)

On using open it launches the terminal and everything other than tmux is sourced, but on double pressing the app via finder(and launchpad) it opens fine with tmux
https://user-images.githubusercontent.com/57838468/128821831-28e18b38-ab9c-46d0-8500-0e1a8ba46f90.mov
Everything other than tmux is sourced, very weird

Hey @mTvare6
many thanks for that video demo. I will be booting into my MBP soon to test/fix and actually implement this. :)

@christianparpart did you check the edit, it happened because open passed the TMUX variable to contour so it thought tmux is running, also I had mentioned it worked without any workaround, but apparently I already edited the yaml file to have
        shell: "login"
        arguments: ["-fp", "name"]

sorry if I had confused or wasted your time

@christianparpart did you check the edit, it happened because open passed the TMUX variable to contour so it thought tmux is running, also I had mentioned it worked without any workaround, but apparently I already edited the yaml file to have
        shell: "login"
        arguments: ["-fp", "name"]

sorry if I had confused or wasted your time

Hey, I just read you now.
This was certainly no waste of time. I did not even know of that behavior of OS/X (as I am mainly using Linux, and only rarely OS/X or Windows). I still try hard to make this terminal emulator working the same on all platforms.
The default startup shell is the one that I suggested you to do manually. It's still an improvement, so many thanks. :)


[AUTO] terminate declaration FE

As NHS admin I want to upload files with register to automatically terminate declarations.
In order to do that we need to add 3 additionaly pages:

file upload
file entries statistic
files statistic
Requirements here

@ivchenkodima please, on the page with Reigster_entries rename first column ID --> ID файлу
also add to search by document tax_id (ІПН)


Impliment standard Icon sizes

So @Fidesnoella The way the component overrides work is that they override the small and large fontSizes for the icons. So when you want an icon to be large, all you have to pass is the fontSize="large" prop and the override will take our styles for it.
<SettingsIcon fontSize="large" data-testid={DATA_TEST_ID.APP_HEADER_ICON_SETTINGS} />

@Fidesnoella Would be nice if we could show off these styles in the storybook under styles? Could you add that?

Also @Fidesnoella We should go through all the files and find icons that are not setting their sizes this way. For example the sidebar icons...

@Fidesnoella when starting work on a new task you should fetch the latest main and create a new branch from that.
Your branch is old and you now have to merge the changes... this is extra work without any reason.


Feature/v2 updates

Pull Request
Are you a current Fivetran customer?

Internal
What change(s) does this PR introduce?

This PR applies the Ad Reporting V2 updates:

Adds in the ad_squad_hourly_report and campaign_hourly_report source tables and accompanying staging models, and the following field descriptions and tests for the new models in the yml files
Adds additional fields to existing models that were previously not brought in
Applies README updates
Introduces the identifier variable for all source models
Casts all timestamp fields using dbt_utils.type_timestamp()

Did you update the CHANGELOG?

[x] Yes

Does this PR introduce a breaking change?

[x] Yes (please provide breaking change details below.)
[x] No  (please provide an explanation as to how the change is non-breaking below.)

Did you update the dbt_project.yml files with the version upgrade (please leverage standard semantic versioning)? (In both your main project and integration_tests)

[x] Yes

Is this PR in response to a previously created Bug or Feature Request

[ ] Yes, Issue/Feature [link bug/feature number here]
[x] No

How did you test the PR changes?

[x] CircleCi
[ ] Local (please provide additional testing details below)

Select which warehouse(s) were used to test the PR

[x] BigQuery
[x] Redshift
[x] Snowflake
[x] Postgres
[x] Databricks
[ ] Other (provide details below)

Provide an emoji that best describes your current mood

:dancer:
Feedback
We are so excited you decided to contribute to the Fivetran community dbt package! We continue to work to improve the packages and would greatly appreciate your feedback on our existing dbt packages or what you'd like to see next.

[x]  Do you feel there is a desire or a need to add passthrough metrics to the Snapchat report models? Most of our other packages include these passthrough metrics. If there are other metrics you feel customers would want to include in these models, then I say we add this functionality to the *_hourly_report models. Otherwise, we can keep the models as they are.

Good point! I do see that being relevant so I will add that metric functionality in. I updated the readme as well.

[ ]  Do you know why for all of our is_most_recent_record fields we use _fivetran_synced within the partition order by instead of the updated_at field within the source? It seems creative_url_tag_history seems to be the only one that uses updated_at.

Good question... I glanced at the sample data we had and it looks like it doesn't make much of a difference nor do I think it would when used on real data. I changed the _fivetran_synced ones to updated_at to make it in line with the other ad packages


Add flag to expose only the metric

Idk if this is really helpful, but we just want to know if a reboot is required and perform the reboot manually.
If there's anything to improve the PR, please let me know.

Hi @Thubo - I'd rather understand why you want to reboot manually, and see if we can address that - can you explain?

Hi @awh ,
we have applications (pods) which are rather painful to restart, because they handle state-full connections (websockets or databases). Therefore we want to be in control about the restart of the underlying nodes (e.g. trigger the reboot manually outside business-hours and monitor the system closely).
I know this is not ideal - but we still want to be notified when a reboot is required. (A custom metric would to the same job, but I like the idea of not reinventing the wheel.)

@Thubo does #46 satisfy your requirements?

@Thubo does #46 satisfy your requirements?
Yes it does. However, as you stated in the PR:

Try not to abuse this mechanism - it's better to strive for
restartability where possible. If you do use it, make sure you set
up a RebootRequired alert as described in the next section so that
you can intervene manually if reboots are blocked for too long.

We probably wouldn't abuse it, but in the end we would block pretty much all our nodes due to the topology of our services/clusters. But I guess that would be fine and we are able to pretty much achieve what we are looking for.


Make definition of locals nicer

Right now, most of a view's logic sits inside a single #locals method where it returns the hash of locals it wants passed to the template. This is not very nice, and leads to a lot of awkward methods like this.
It'd be better if we could have each of our view's locals provided via a dedicated method that is marked as a local or an "exposed" method, so instead of this:
def locals(options)
  # setup stuff here

  super.merge(
    foo: something,
    bar: something_else,
  )
end

we have something like this:
local def foo
  something
end

local def bar
  something_else
end

Where local or maybe expose is a class-level macro that accepts a method name (e.g. local :foo, like in that example above) and then makes it part of the set of locals passed to the template.
The tricky thing here is that our view instances here are long-lived, with all the per-request data coming into #call only, so that data can't really sit on the instance and be available for instance methods to play with. We might be able to do some trickery to make it look like this though 😏

Revisited this for a think. I think that doing some trickery to try lift methods out of one context and plant them into another won't ever be intuitive. It'll more likely confuse than help.
So I think perhaps something like this might be better:
module Main
  module Views
    module Posts
      class Index < Main::ViewController
        include Main::Import["persistence.repositories.posts"]

        configure do |config|
          config.template = "posts/index"
        end

        prepare do |input|
          posts = controller.posts.listing(
            controller.posts.listing(
              page: input.page,
              per_page: input.per_page
            )
          )

          expose :posts, Decorators::PublicPost.decorate(posts)

          expose :indexed_posts, posts.map.with_index { |post, i|
            OpenStruct.new(index: i, post: post)
          }

          expose :paginator, posts.pager
        end
      end
    end
  end
end

So, instead of #locals where you munge the input and return a hash, have a class-level .prepare block that is executed with the input, and provides a helper like #expose to allow the class author to pass locals to the view one by one, and have a bit more space to set things up nicely.
(Underneath, this would all be a nicer API on top of something similar to what is happening now in terms of preparing a hash of locals. Class authors could always just go ahead and build custom #call or #locals methods directly if they wanted).
Thoughts?

How about:
class ViewController
  def call(input)
    view_locals = self.class.exposures.each_with_object({}) do |(name, blk), h|
      h[name] = blk ? instance_exec(input, &block) : __send__(name, input)
    end
    # the reset of stuff
  end

  def self.expose(name, &block)
    exposures << [name, block]
  end

  def self.exposures
    @exposures ||= []
  end
end

class Index < ViewController
  expose :posts
  # slightly trickier but totally doable (we can look at block arg names and
  # if they match other exposures then we tsort them by deps and execute in
  # the right order to get arguments for other exposures, like "posts" here)
  expose :paginator { |posts| posts.pager }

  def posts(input)
    # do stuff to return posts
  end
end

This way you can either use inline blocks or method names, and we can easily aggregate all the needed locals using isolated methods/blocks, no need for a nested DSL.
WDYT?

@solnic this looks very nice! And a little closer to what I was thinking about in the first place.
My only concern would be making it possible for exposure blocks/methods to access common things that you don't want to expose, e.g. the equivalent of this:
def locals(options)
  shared_thing = do_something_with(options[:foo])

  {
    one: prepare_for_view(shared_thing)
    two: more_stuff_happening_to(shared_thing)
  }
end

Perhaps we need another keyword besides expose that does everything else apart from including it in the view's locals?

@timriley views are not singletons so we can use their state to share common stuff between exposure methods, wdyt?

@solnic I dunno, that doesn't seem right? Since they're not meant to carry state from one #call invocation to another?

That said, if we're sharing objects (private or otherwise) between exposures, it'd make sense to capture/memoize them somehow, because otherwise we'd be wasting cycles in rebuilding them for each usage. This was part of the motivation in putting together that other .prepare block example, since it makes it clear we only set up that posts object once.

@timriley yeah they must be be memoized, I think we can avoid any prepare blocks, it would be good to keep these objects relatively simple


Command "swagger2cscontroller" is generating "return" statement for void methods

I tried to generate code from swagger template for PetStore and found there is a 'return' statement added to void method.

name that need to be updated
/// Updated user object
[System.Web.Http.HttpPut, System.Web.Http.Route("user/{username}")]
public void UpdateUser(string username, User body)
{
## return _implementation.UpdateUserAsync(username, body);
}
Error:
Since PetStoreController.DeleteUser(String,User) returns void, a return keyword must not be followed by the object expression.

Please test with the latest CI artifacts: https://ci.appveyor.com/project/rsuter/nswag-25x6o


Fix: Pipeline publish path incorrect

This is PR for #9959.
I have added a variable for functiomAppName and used it while copyTask to get correct build artifacts

@ankur198 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ankur198 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ankur198 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ankur198 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off

#sign-off

#sign-off


What To Do With Damage Boosting?

Expected Behavior
Pressing the direction you're sent after you get knocked back doesn't conserve that momentum.
Relevant Information to Reproduce
Press the direction you're sent after you take damage and are knocked back.
Category of Issue
Question

I'll keep it, seeing as it doesn't break the engine, can be a fun glitch to perform, and could be potentially leveraged in speedruns (were a speedrunning community to come to the game).


Docs for refreshUserIds publisher api method

Relates to https://github.com/prebid/Prebid.js/pull/5819

@TLadd Looks good to be, although I'd suggest to remove the => undefined part.

part of 4.12


X-Plex-Client-Identifier is missing

having trouble getting any of the code examples to work, no matter what I do I get {"errors":[{"code":1000,"message":"X-Plex-Client-Identifier is missing","status":400}]}
I even tried using one of the examples effectively verbatim:
import getpass
from plex_api_client import PlexAPI

client = PlexAPI(
    client_id="gcgzw5rz2xovp84b4vha3a40",
    client_name="Plex Web",
    client_version="4.133.0",
    client_platform="Chrome",
    device_name="Linux",
)
res = client.authentication.post_users_sign_in_data(request={
    'login'<EMAIL_ADDRESS>    'password': getpass.getpass('plex password'),
    #'verification_code': getpass.getpass('TOTP'),
})

Traceback (most recent call last):
  File "~/.pyenv/versions/3.10.7/lib/python3.10/code.py", line 90, in runcode
    exec(code, self.locals)
  File "<input>", line 1, in <module>
  File "~/.pyenv/versions/plex-playlist-sync/lib/python3.10/site-packages/plex_api_client/authentication.py", line 686, in post_users_sign_in_data
    raise errors.PostUsersSignInDataBadRequest(data=data)
plex_api_client.models.errors.post_users_sign_in_data.PostUsersSignInDataBadRequest: {"errors":[{"code":1000,"message":"X-Plex-Client-Identifier is missing","status":400}]}

am I doing something wrong here? what am I missing? is there any actual documentation anywhere?

removed unnecessary comments

Hey @Commod0re,
This SDK and all of the others listed here, are all generated from one OpenAPI specification. The CTO at Plex was kind enough to share a bunch of markdown documentation that I began converting to OpenAPI in effort to expand it further and better document the API.
So far, I and two other individuals have contributed to it in our free time, and we welcome any assistance the community wants to provide!

am I doing something wrong here? what am I missing? is there any actual documentation anywhere?

The readme contains documentation for the methods and types present in the SDK, and additional API focused documentation can be found here.
This documentation is also sourced from the same OpenAPI specification, so as we continue to expand and correct the OpenAPI specification it should also expand.

how long has this been broken?

That specific endpoint you mentioned was very recently added to the Spec, it looks like a required parameter just needs to be added

do y'all even test this thing?

Some! Tests are all written manually in the main spec repo, so far only written by @Jasonlandbridge.
In the near future, we will be looking to add a few additional types of tests at different points to each generated SDK to validate some sets of functionality.

a handful of incomplete examples that don't actually work, a list of functions with almost no useful information... is this representative of the types of libraries you like to work with?

This feedback feels less constructive.
We have tried to piece together as much information on this API as we can using traditional data-gathering methods. Which In this context are pretty much exclusively reviewing calls in the network explorer inside a browser while interacting with the Plex website and by throwing API requests at the Plex fridge via Postman spaghetti style.
If you have any specific knowledge you would like to share or if you want to contribute meaningfully we would welcome that.

worst of all it's almost impossible to work around the poor documentation by reading the code because the code is super indirect and neither commented, nor written clearly enough to be self-documenting

Again, this SDK, along with all the others linked are fully generated. Not hand written. It is just a few folks working to try and document the Plex API as thoroughly as possible, and provide SDKs for the community in as many languages as possible.

what a frustrating mess

Have a wonderful day, and happy development!

@LukeHagar You're too polite, I would have told this rude and ungrateful brat off.

you are right, I was wrong to allow my frustration to get the best of me, I apologize for being rude

No worries @Commod0re, my goal is to document as much of the plex API as I can, and provide SDKs in an many languages as I can.
Time is definitely a factor for me, and I welcome any and all contributions!

@Commod0re Thank you for your apology and admittedly, this project is still a very much a work in progress. If you happen to use Python then I can recommend python-plexapi which is a more mature project.


Failed to load shared library

I compiled and installed the latest code (0.1.5) in Windows 11 with Python 3.10.11 and CUDA 11.8. The compilation with cuBLAS flag and installation were successful. When importing the module via from llama_cpp import Llama, python reported
Failed to load shared library 'C:\Users\teristam\Anaconda3\envs\textgen\Lib\site-packages\llama_cpp\llama.dll':
Could not find module 'C:\Users\teristam\Anaconda3\envs\textgen\Lib\site-packages\llama_cpp\llama.dll' (or one of its dependencies). Try using the full path with constructor syntax.

But when I change
https://github.com/abetlen/llama-cpp-python/blob/d90c9df32639397078a439c86ac6a474043ce57d/llama_cpp/llama_cpp.py#L55
to ctypes.CDLL(str(_lib_path),winmode=0), the dll is able to load succesfully. It may be related to a bug in ctype in Windows:
https://stackoverflow.com/questions/59330863/cant-import-dll-module-in-python/64472088#64472088
https://bugs.python.org/issue42114

@teristam thanks for the bug report / solution, I'll update it so winmode=0. Are there any side-effects?

The winmod=0 alone did not fix this issue for me. I had add CUDA_PATH to dll dir.  #225


openid connect idtoken iss mismatch problem

With WSO IS 5.3.0, reproducing steps as below:
1, create service provider with "Inbound Authentication Configuration"/"OAuth/OpenID Connect Configuration" to configure and have password granttype enabled.
2, in /opt/wso2/repository/conf/identity/identity.xml having this in OpenIDConnect element:
<IDTokenIssuerID>${carbon.protocol}://${carbon.host}:${carbon.management.port}/oauth2/oidcdiscovery/</IDTokenIssuerID>
3, get idtoken by running:
curl -H "Authorization: Basic X0FvMEozN2cxMlkzSEtiaU1tbWVYdHVvM2FFYTpCeVBZQUprQnZLWlNmOURHeDF0S2xvUTdKcGdh" -H "Content-Type: application/x-www-form-urlencoded" -k -d "grant_type=password&username=admin&password=admin&scope=openid" https://localhost:9443/oauth2/token
4, returned idtoken is partially something as below decoded:
{"exp":1487335376,"sub":"admin","azp":"Dibo_uMHzySCIxrf55uvMGWjGEUa","at_hash":"_8q5TmtJRsdEj4V_dL4-Zg","aud":["Dibo_uMHzySCIxrf55uvMGWjGEUa"],"iss":"https:\/\/localhost:9443\/oauth2\/token","iat":1487331776,"acr":"urn:mace:incommon:iap:silver"}
By openid connect spec, iss should be https://localhost:9443/oauth2/oidcdiscovery/. The mismatch will make validation of idtoken fail.

@gaord the spec doesn't say it has to be the discovery endpoint. It just needs to be some identifier confirming to http://openid.net/specs/openid-connect-core-1_0.html#IDToken


boundimage: Add tmt tests for bound images

These tests validate the basic use case of switching and upgrading an image with bound images that need to be pulled.

At the end of doing this, what did you think? I personally ended up somewhat ambivalent about the nushell usage for tests...my overall feeling was I could have written 70% of it faster in bash, except there's that 20% where you want to parse some JSON or operate on arrays and I need to go look up syntax and about 10% of horrible bash pitfalls.

I like it, I think these will be way easier to maintain in the long run. It was a little frustrating digging through the nu shell docs at times and chat gpt (and other llms) wasn't very helpful. Still, after the initial (relatively low) learning curve I think this a lot nicer than bash. I eventually got familiar enough with testing stuff directly in a nu shell prompt which seems more ergonomic than zsh or bash. I'll probably switch over to it as my main shell soon.

marking this as a draft because I started seeing some odd behavior related to handling reboots. I ran out of time today to figure out the cause and I'll be AFK for awhile. AFAICT the problem is that on each reboot all of the nu shell tmt tests run. So, trying to read the reboot count in each test gets tricky. I think for now we could just dump all the tests in a single file, but we probably want to isolate each test to a separate VM and run them in parallel or something.

Oh wow...yes, oops.

but we probably want to isolate each test to a separate VM and run them in parallel or something.

Yes, this is quite doable, I'll take a look at figuring out the relevant tmt incantations.

Let's track this issue here https://github.com/containers/bootc/issues/735

Want to try rebasing on top of https://github.com/containers/bootc/pull/736 ? It will require something like:
cp plans/test-20-local-upgrade.fmf plans/test-21-bound-images.fmf and then replacing the test executed inside that fmf file

$ tmt run plans -n test-21-logically-bound
/var/tmp/tmt/run-003

/plans/test-21-logically-bound
...
total: 1 test passed

:tada:


Copy-editor's changes to Chapter 1

Still missing:

[ ] We should align the print & web style so that we refer to "gray outline" or "black frame" and that's the same in both. We can't do that before we're looking at proofs, though, so not attempted.
[ ] In the book we refer to figures as "in Figure X.Y", on the web as just "below" since they are in line. Would be nice to have that in code somehow, not done yet.
[ ] Similarly, figures have captions in print but not on the web, and not all figures are used in both. We'll align this in time as the figures are redrawn to be better.
[ ] The exercises are numbered in print; we should do that, that's obviously a practical improvement. Not done here.

It would be ideal to do a pass through feedback before shipping the latest "frozen" version to the copyeditors.

I found a few cases where what you wrote differed from the copyeditor, and fixed it.


Markdown rendering of code blocks broken in light theme

When the how-to-fix text of a rule violation contains code blocks they are always using a dark theme, making them unreadable when the UI is using the light theme:

@mnonnenmacher can you copy-paste the markdown message string as-is? I would need to know how the markdown marks a code block.

@mnonnenmacher  Could you please check if #1524 fixes it for you?

Once again didn't autoclose.

No comments received, and I checked from "real world" runs that this is fixed for now, so let's close it.

@Etsija Yes, it's working, thanks!


Accessing the current build logs

Hi,
I've been meaning to write a custom build step to analyze the current build logs and fail if a condition is met (ex: if the string 'error' is found, fail the current build).  This is a feature I'm currently using in TeamCity and would really enjoy seeing it in VSTS.
Is it possible to access the build logs within a VSTS task?
Thanks,
Yohan

Yep, you have full access to the disk.   You can write a custom task (and publish to the market place to share).  Use the lib in node or powershell.  Docs are included and our tasks here are OSS as a reference.
https://github.com/Microsoft/vsts-task-lib
There's also variables to use:
https://www.visualstudio.com/en-us/docs/build/define/variables
Another option is to write an ad-hoc powershell task, use the variables above (like source directory) and run with the powershell task.  That's more appropriate for your custom use.
Guidance - not a specific task issue in this repo so closing.


Create OpenAPI from Folder in collection

In my postman collection I have some folders which are used for Collection Runner / Newman execution. The contain duplicate API calls for different integration testing, it would be great if theres a flag to stipulate which folders are included in the conversion.
I can strip these out unwanted folders by pre-parsing the postman collection file with JQ but it's nice when its baked into your tool natively.

Hi @alexevansigg,
In this case this feature is out of the scope of the library, the goal is to transform a postman collection into open API specs but things like filtering, sorting... Should be done in another place I think.
A possible solution for you is just to move to another collection this folders that you don't want to transform, I normally follow this approach, I maintain a collection with my API request with full descriptions and ready to use for customers and then other request in a separate collection with similar made but with a suffix like "utils"
Best regards