post urlencoded to login

hi, I need to POST url encoded username/password to login for for some reason auth is intercepting the login and failing
the config looks like:
export default defineNuxtConfig({
  // ...
  auth: {
    // middleware
    globalMiddleware: true,

    strategies: {
      local: {
        scheme: 'refresh',

        token: {
          property: 'access_token',
        },

        endpoints: {
          login: {
            url: `${apiPrefix}/auth/token`,
            method: 'post',
          },
          logout: {
            url: `${apiPrefix}/auth/me`,
            method: 'delete',
          },
          refresh: {
            url: `${apiPrefix}/auth/refresh`,
            method: 'post',
          },
          user: {
            url: `${apiPrefix}/auth/me`,
            method: 'get',
          },
        },

        user: {
          property: '',
          autoFetch: true,
        },
      },
    },
  },

but in $auth login and refresh are replaced with /_auth/local/local/authorize which dies with Invalid JSON body if I url encode the body.
    const res = await $auth.loginWith('local', {
      body: encodeURIComponent(state),
      Headers: {
        'Content-Type': 'application/x-www-form-urlencoded; charset=UTF-8',
      },
    });

any suggestion about what I'm missing?

Operating System: Linux
Node Version:     v18.19.1
Nuxt Version:     3.10.3
CLI Version:      3.10.1
Nitro Version:    2.9.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      srcDir, telemetry, devtools, extends, modules, eslint, fontMetrics, googleFonts, typescript, runtimeConfig, auth, routeRules, vite
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

this problem became critical for me as I use nuxt generate and this hardcoded authorize thing doesn't exist unless nitro is running.
at least for the local/refresh strategy adding ssr: false makes no change :sob:

I'm not sure if it helps, but it works for me
const response = await $auth.loginWith("cookie", {
      body: new URLSearchParams(state)
    })

I added JSON support to the backend at the end, but now my main problem is that this authorize interceptor doesn't exist when using nuxt generate and that means I can't log in. any suggestion on how to disable it when using local?


Default Export-FormatData to pretty-printing  or add an option

Related: #6680
One use case for Export-FormatData - even though about_Format.ps1xml doesn't currently mention it - is to use existing formatting definitions as a starting point for creating new ones.
To that end, outputting pretty-printed XML would be much more convenient - currently, a single-line XML string is output.

If changing the existing behavior is not a backward-compatibility concern:

Simply output pretty-printed XML by default (it will consume more space, however)

Otherwise:

Introduce a switch such as -Format (or, to follow the  - inverted - logic of ConvertTo-Json, perhaps -Uncompress?) that results in pretty-printed output.

Environment data
Written as of:
PowerShell Core v6.0.2

@mklement0 This sounds like a good thing to do. (I would have thought we were already doing it actually.) Anyway, as long as the result is valid XML, simply pretty-printing it should fall within acceptable changes.

Pretty printed xml is well-formed and valid, so there's no functional impact.  Seems like a good idea and should be a small change.


Chdir to execute in a directory the user can access

Brew needs to be able to retrieve information about the current working directory when it runs.
If we're running puppet as root (e.g. from something like cron), and we're managing homebrew for a non-root user account, it's very likely we'll be running with a current working directory that isn't readable by the homebrew user (e.g. /var/root). This means that when we execute a brew command, brew fails with errors about not being able to access the CWD. In effect, the current version of the providers are doing this:
# cd /var/root   # (or any dir inaccessible to someuser)
# sudo -u someuser brew list --versions

Which results in:

shell-init: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
chdir: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
chdir: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
job-working-directory: error retrieving current directory: getcwd: cannot access parent directories: Permission denied
The current working directory doesn't exist, cannot proceed.

In puppet, these errors look like:

Could not prefetch package provider 'brew': Could not list packages: Execution of '/usr/local/bin/brew list --versions' returned 1: shell-init: error retrieving current directory: getcwd: cannot access parent directories: Permission denied

In this PR, we change the providers to cd to the home dir of the brew user just before the command executes. (The assumption is that a user's own homedir is accessible to them.)

Thanks, nice catch!


also delete release when deleting tag

this attempts to fix issue where sometimes our canary release is marked as 'draft' and re-runs won't help as it will just update the existing release (which is already draft).
I still don't have a good answer as to why the release is getting marked as draft (except someone reporting same issue here https://github.com/ncipollo/release-action/issues/336#issue-1728946267). may be if we push to main while previous release is being tagged?

this just happened again (before the merge), so I am going to merge this and monitor if it improves the state.


Fixe rotation direction on reorient()

Description
Fixe #3006
I think the rotation from exif is not in the good direction
Change the rotation direction on image reorient()
Maybe reorient methode should be add in the doc
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

I have no idea how this is supposed to work. Can you share any resources that indicate what the correct process should be?

Haven't checked the back-bone of rotate(), but GD and ImageMagick uses to different values for the same rotation. So I guess @paul45 are using ImageMagick? Or it's wrong in GD. xD

@jreklund i use ImageMagick but i think it is the same in GD because the reorient() method is in the basehandler.php
@MGatner
https://www.impulseadventure.com/photo/exif-orientation.html
for exemple if the orientation code from exif is 6 we need rotate 90° in CW

Okay, then we can't accept this as is. As GD and ImageMagick uses different values for the same thing. I wrote a similar rotation function but based off Imagick::rotateImage, and had to apply a conversion based on this post, in another project.
I can't see any conversion being made in CodeIgniter at the moment, so that function needs to be as well. Personally I applied the conversion in ImageMagick, as the GD are the main one used.

ok you are right @jreklund the actual reorient() is good for GD but not for ImageMagick
maybe the reorient() method will be put in GDHandler.php and in ImageMagickHandler.php(with this change) instead of BaseHandler.php

@paul45 I think that is the right way to go. Would you be willing to tackle this?

an other way is to add a optional direction parameter to rotate() in both class for select CW or CCW
and use this parameter in reorient()

Removing reorient from BaseHandler would be a breaking change if I'm not wrong, as it's a public function. So it would need to be converted in either GD or ImageMagick.
Hard to tell if someone are depending on the current orientation values at the moment, by extending the BaseHandler. So the safe bet are to converting it in ImageMagick I'm afraid.

@paul45 consensus seems to be that we move this to ImageMagick and leave BaseHandler's version in place. I think a test to demonstrate proper rotation would help me a lot, since I only look at this every 3 weeks 🙄

maybe a solution without duplicate code, with protected $rotateAngle like @jreklund said should be more elegant ?
i don't know how to make test sorry

It's less duplicated code at least, if everyone are fine with adding two new variables. :-)

I will merge this at the current state and add some tests later today.


Clean up PlatformAgnostic SystemInfo

This change was primarily motivated by removing ChakraCore's dependency on the GetPhysicalMemory Win32 API on Windows. This is problematic from a UWP compliance perspective, so I removed usage of that API
altogether.
Also did some cleanup- I refactored the ch-specific platform agnostic code to its own directory under ch.  We will merge the two PlatformAgnostic components once PlatformAgnostic can be moved out of Runtime.
Also made some stylistic changes to make the code more consistent with the existing style in the file.

@digitalinfinity I have couple of questions in order to understand the changes better;

GetPhysicalMemory is part of ChakraCore and for now the consumer was ch. Assuming ch is not for UWP... and UWP has nothing to do with GetPhysicalMemory... wouldn't be nicer to manage that via ifdef etc. stuff instead of moving the codes into ch?

IIRC; We had just recently refactored the GetBinaryLocation etc. parts from CH back to PlatformAgnostic for reuse purposes. What do we gain by moving these back to CH?

Perhaps this new ch\PlatformAgnostic directory should actually be the start of the new root-level one, and then we can add things from .\lib\Runtime\PlatformAgnostic to .\PlatformAgnostic as we see fit? As far as I remember, Intl-related code only depends on the PAL, ICU, and Utf8Codex -- that last one may be an issue, but I would also suggest that Utf8Codex shouldn't have any external dependencies (which it currently may or may not, I am not sure). I know Utf8Helper.h has some frustrating level of dependency on lib, which may take some time to factor out.

Codex is not really a concern since it's mostly standalone. The bigger problem is the dependency on ThreadContext because that's have WinGlob is managed today. We should avoid any dependency on these classes for PlatformAgnostic.
I marginally prefer having module specific PlatformAgnostic directories before we do one wholesale effort to combine the two- I think it's a tad less confusing that way.

p.s. this PR overrides #2297 and the PR #3221

@dotnet-bot test OSX static_osx_osx_debug please

Closing in favor of https://github.com/Microsoft/ChakraCore/pull/4141


feat: 🎸 Update compatible REST API and SQ version for 5.3.0

Description
Update SQ and REST API version compatible with chain 5.3.0
Breaking Changes
NA
JIRA Link
DA-563
Checklist

[ ] Updated the Readme.md (if required) ?

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

npm package (@latest dist-tag)
GitHub release
v5.10.0
v5.10.0

Your semantic-release bot :package::rocket:


Scroll bar bg. color

Scroll bar's background color should change with change in theme  . and should also have same color as other page .
also try to add same js to make it smooth transition while changing with theme button .
Note: plz send a short video for review before pr

hey @harshvardhansb can you assign this issue to me i can work on it

I would like to work on this as well!


[Unified Web Console] EXPLAIN PLAN FOR throws org.apache.calcite.sql.parser.SqlParseException

Affected Version
0.15.0
Description

A simple SQL statement like "EXPLAIN PLAN FOR SELECT 1 + 1;" from the unified web console is resulting in a org.apache.calcite.sql.parser.SqlParseException.

Unknown exception / Encountered "( EXPLAIN" at line 1, column 15. Was expecting one of: <IDENTIFIER> ... <QUOTED_IDENTIFIER> ... <BACK_QUOTED_IDENTIFIER> ... <BRACKET_QUOTED_IDENTIFIER> ... <UNICODE_QUOTED_IDENTIFIER> ... "LATERAL" ... "(" ... "(" "WITH" ... "(" "+" ... "(" "-" ... "(" "NOT" ...  "(" "TRUE" ... "(" "FALSE" ... "(" "UNKNOWN" ... "(" "NULL" ... "(" <LBRACE_D> ... "(" <LBRACE_T> ... "(" <LBRACE_TS> ... "(" "DATE" ... "(" "TIME" ... "(" "TIMESTAMP" ... "(" "INTERVAL" ... "(" "?" ... "(" "CAST" ... "(" "EXTRACT" ... "(" "POSITION" ... "(" "CONVERT" ... "(" "TRANSLATE" ... "(" "OVERLAY" ... "(" "FLOOR" ... "(" "CEIL" ... "(" "CEILING" ... "(" "SUBSTRING" ... "(" "TRIM" ... "(" "CLASSIFIER" ... "(" "MATCH_NUMBER" ... "(" "RUNNING" ... "(" "PREV" ... "(" "NEXT" ... "(" <LBRACE_FN> ... "(" "MULTISET" ... "(" "ARRAY" ... "(" "PERIOD" ... "(" "SPECIFIC" ... "(" <IDENTIFIER> ... "(" <QUOTED_IDENTIFIER> ... "(" <BACK_QUOTED_IDENTIFIER> ... "(" <BRACKET_QUOTED_IDENTIFIER> ... "(" <UNICODE_QUOTED_IDENTIFIER> ... "(" "ABS" ... "(" "AVG" ... "(" "CARDINALITY" ... "(" "CHAR_LENGTH" ... "(" "CHARACTER_LENGTH" ... "(" "COALESCE" ... "(" "COLLECT" ... "(" "COVAR_POP" ... "(" "COVAR_SAMP" ... "(" "CUME_DIST" ... "(" "COUNT" ... "(" "CURRENT_DATE" ... "(" "CURRENT_TIME" ... "(" "CURRENT_TIMESTAMP" ... "(" "DENSE_RANK" ... "(" "ELEMENT" ... "(" "EXP" ... "(" "FIRST_VALUE" ... "(" "FUSION" ... "(" "GROUPING" ... "(" "HOUR" ... "(" "LAG" ... "(" "LEAD" ... "(" "LAST_VALUE" ... "(" "LN" ... "(" "LOCALTIME" ... "(" "LOCALTIMESTAMP" ... "(" "LOWER" ... "(" "MAX" ... "(" "MIN" ... "(" "MINUTE" ... "(" "MOD" ... "(" "MONTH" ... "(" "NTH_VALUE" ... "(" "NTILE" ... "(" "NULLIF" ... "(" "OCTET_LENGTH" ... "(" "PERCENT_RANK" ... "(" "POWER" ... "(" "RANK" ... "(" "REGR_SXX" ... "(" "REGR_SYY" ... "(" "ROW_NUMBER" ... "(" "SECOND" ... "(" "SQRT" ... "(" "STDDEV_POP" ... "(" "STDDEV_SAMP" ... "(" "SUM" ... "(" "UPPER" ... "(" "TRUNCATE" ... "(" "USER" ... "(" "VAR_POP" ... "(" "VAR_SAMP" ... "(" "YEAR" ... "(" "CURRENT_CATALOG" ... "(" "CURRENT_DEFAULT_TRANSFORM_GROUP" ... "(" "CURRENT_PATH" ... "(" "CURRENT_ROLE" ... "(" "CURRENT_SCHEMA" ... "(" "CURRENT_USER" ... "(" "SESSION_USER" ... "(" "SYSTEM_USER" ... "(" "NEW" ... "(" "CASE" ... "(" "CURRENT" ... "(" "CURSOR" ... "(" "ROW" ... "(" "(" ... "(" "SELECT" ... "(" "VALUES" ... "(" "TABLE" ... "UNNEST" ... "TABLE" ... / org.apache.calcite.sql.parser.SqlParseException
While issuing a curl for the same query works and outputs a plan.
curl -X POST \
   -H "Content-Type:application/json" \
   -d \
'{
  "query":"EXPLAIN PLAN FOR SELECT 1+1"
}' \
'http://localhost:8888/druid/v2/sql'

[{"PLAN":"BindableValues(tuples=[[{ 2 }]])\n"}]
=> Inspecting the request payload from the Network activity console, the query issued is
"SELECT * FROM (EXPLAIN PLAN FOR SELECT 1 + 1) LIMIT 5000"

Looks like the console is modifying the query with the prefix "SELECT * FROM" in this case which is resulting in the issue.
curl of the modified query "SELECT * FROM (EXPLAIN PLAN FOR SELECT 1 + 1) LIMIT 5000" fails as expected.

Same behaviour is observed for WITH clause.

WITH wikipedia as (SELECT 1 + 1) FOR SELECT * FROM wikipedia LIMIT 1; throws org.apache.calcite.sql.parser.SqlParseException for the same reasons mentioned in 1.

Yeah the query UI tries to wrap your query, that does not work well for the queries you posted.
You can disable that like so:

Make sure your run button says Run as is

We should make the query wrapper smarter

We have made the query wrapper smarter as part of Druid 0.16.0


Fix #91 check logger is nil

fix #91

?       github.com/surrealdb/surrealdb.go/internal/mock [no test files]
?       github.com/surrealdb/surrealdb.go/internal/rpc  [no test files]
?       github.com/surrealdb/surrealdb.go/pkg/gorilla   [no test files]
?       github.com/surrealdb/surrealdb.go/pkg/rand      [no test files]
?       github.com/surrealdb/surrealdb.go/pkg/websocket [no test files]
ok      github.com/surrealdb/surrealdb.go       0.354s
ok      github.com/surrealdb/surrealdb.go/internal/benchmark    (cached) [no tests to run]
ok      github.com/surrealdb/surrealdb.go/pkg/logger    0.002s

Seems like something is wrong with CI, I tested with two different machines.

waiting for #88

for information: this PR removed 1.20 golang version for slog


Disable access to ToolPanel

Looking for a way to complete disable access to the ToolPanel?   The only thing I can find is the ability to control its display (showToolPanel) but nothing to complete remove access?
Is this possible and I am missing something?  I have scoured the docs and have not found anything

the only way to show it is via the menu, and you can customise the menu to remove this item.

@ceolter Awesome, that is what I wanted to know.,


[feat] Watch addresses

Feature Request
Describe the Feature Request
A function to watch addresses and to have an event sent or triggered or a query responding in the positive when an address hits a certain limit.
Describe Preferred Solution
Respond to a query on an address as to whether or not it has reached a pre set limit set for that address, or raise an event when that limit is reached.
Additional Context
Use case: Makes it easier to accept doge for purchases via in-app purchases as a developer may generate an address

I think about this a lot in various contexts. I'd use this in two ways:

register some sort of trigger if an address receives a confirmed transaction (or a mempool transaction)
get a list of all transactions to an address after a specific timestamp or blockheight

I prefer to decouple non-chain statefulness from the library. In other words, I wouldn't want the library to track things like transaction amounts to a threshold; I'd rather push that to a stateful service under my control, like a database or other persistence mechanism. That reduces the responsibility for the library to taking action when something interesting happens to an address or allowing me to query for actions since I last finished processing events.


Module not found sentiment_analyzer (Python3.5.2, NLTK3.1)

This piece of code should, according to my best knowledge, work:
import nltk.sentiment
nltk.sentiment.util.demo_sent_subjectivity('This is great')

c:\users\user\appdata\local\programs\python\python35-32\lib\site-packages\nltk\sentiment\util.py in demo_sent_subjectivity(text)
598         print('Cannot find the sentiment analyzer you want to load.')
599         print('Training a new one using NaiveBayesClassifier.')
--> 600         sentim_analyzer = demo_subjectivity(NaiveBayesClassifier.train, True)
601
602     # Tokenize and convert to lower case
c:\users\user\appdata\local\programs\python\python35-32\lib\site-packages\nltk\sentiment\util.py in demo_subjectivity(trainer, save_analyzer, n_instances, output)
537     :param output: the output file where results have to be reported.
538     """
--> 539     from sentiment_analyzer import SentimentAnalyzer
540     from nltk.corpus import subjectivity
541
ImportError: No module named 'sentiment_analyzer'

@njbuch Thank you. I fixed the problem in #1290 .
The error was in the imports. As stated in the Package Structure Wiki:

imports from this package have the form from nltk.package import name[...].


[Feature Request] Tags Comicinfo Metadata Field

Hi Lordwelch,
I was wondering could you add the ability to set a Tags metadata field for ComicRack comicinfo files. Right now this isn't available. This isn't a default metadata field for comicinfo. However, it is a part of the anansi projects draft of the comicinfo 2.1 standard, if that matters. Right now it is supported by nhentai, kavita, and komga, but I would love to see it expand. I was able to kind of get it working under the command line using some modifications that broke CBL format, but a more official way would be greatly appreciated,
Sincerely,
SR2017

My only issue is that I want to make sure that ComicTagger shows a difference between the metadata the ComicRack supports and what the newer Anansi project supports as many readers only support the original set of metadata that ComicRack included. My current plan for this is to make it so that there is a plugin-type capability for metadata and then a new metadata type can be defined.

Thank you for responding. Good to know that you are working on a solution.

Out of curiosity, any update on this?

When this PR is merged adding this will be possible #195
Before I finish that PR I am re-working the settings in #363

Any chance of this getting implemented now that both are merged? Totally cool if not, I know the hassle of maintaining a project and the constant requests for various features. Just wanted to get an idea if it was planned at all, so I can plan accordingly.

I have some decent code for turning metadata styles into plugins, (I have to actually fix a hardware issue with the machine it's on before I can get to it again) but right now I'm trying to separate ComicAPI out of ComicTagger into it's own library with https://github.com/OzzieIsaacs/ComicAPI once that is done then I plan on working on metadata plugins again

This is now supported via https://github.com/comictagger/comicinfoxml the comicinfoxml plugin is included in all binary downloads starting with 1.6.0-alpha.9: Groonk

Nice


SQL formatter doesn't like ~ in statements

I'm trying to use the SQL Beautifier with a query, but it throws an error if there's a tilde in the query. I'm using version 1.20.3.
Example:
SELECT bar, foo FROM foo_bar WHERE customer_prefix_name ~ '^.*\\d{9}.*$' GROUP BY bar
Error:
Parse error: Unexpected "~ '^.*\\d{" at line 1 column 57
Expected output:
SELECT
  bar,
  foo
FROM
  foo_bar
WHERE
  ~ customer_prefix_name '^.*\\d{9}.*$'
GROUP BY
  bar

Hi, thanks for the report! The tool is using sql-formatter library (https://github.com/sql-formatter-org/sql-formatter) to do the formatting. Maybe this should be reported there or check if there's any open issues about this?


How to read property from last invoked command?

Hi,
I'm automating some hardware config backup job and command: Invoke-SSHCommand -SessionId $sessionId -Command "backup" gives me output:
Output     : {File written to the location, "/backup/backup-2016-07-05.cfg"., Configuration saved.}
ExitStatus : 0
Error      :
Host       : <IP_ADDRESS>
Duration   : 00:00:03.3760740
if i could access output property i could use:
$output -split '"' -match '/backup/([\s\S]*)')
already tried:
$output = Invoke-SSHCommand ... | Select -ExpandProperty Output
$output = $(Invoke-SSHCommand ...).Output
and pass it to Get-SCPFile in order to download the backup file.
I dont want to use streams because thats how im currently backing up with SshShell module

Because it is an array you need to do a foreach against the content of the property or do a -join to turn it in to a string. The turning of the output in to an array will possibly be reverted to be a string with an option to make it in to an array in the next version. It is the reason I have not updated the version in the gallery because it is a braking event.

On Jul 5, 2016, at 12:08 PM, munchie45<EMAIL_ADDRESS>wrote:
Hi,
I'm automating some hardware config backup job and command: Invoke-SSHCommand -SessionId $sessionId -Command "backup" gives me output:
Output : {File written to the location, "/backup/backup-2016-07-05.cfg"., Configuration saved.}
ExitStatus : 0
Error :
Host : <IP_ADDRESS>
Duration : 00:00:03.3760740
if i could access output property i could use:
$output -split '"' -match '/backup/([\s\S]*)')
already tried:
$output = Invoke-SSHCommand ... | Select -ExpandProperty Output
$output = $(Invoke-SSHCommand ...).Output
and pass it to Get-SCPFile in order to download the backup file.
I dont want to use streams because thats how im currently backing up with SshShell module
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/darkoperator/Posh-SSH/issues/100, or mute the thread https://github.com/notifications/unsubscribe/AAf0HianFG9B1M4P_n0vZZ-DZaXSAMmIks5qSoGXgaJpZM4JFRx4.

ahhh, i was wondering why there are commas... thanks! anyway, i have another issue. when i try to download the file it says:
PS C:\cmd> Get-SCPFile -ComputerName $IP -Credential $(Get-Credential) -RemoteFile /backup/backup.txt -LocalFile c:\cmd\test.txt -Verbose -Debug
cmdlet Get-Credential at command pipeline position 1
Supply values for the following parameters:
VERBOSE: Using SSH Username and Password authentication for connection.
VERBOSE: Fingerprint matched trusted fingerprint for host <IP_ADDRESS>
VERBOSE: Connection successful
VERBOSE: Downloading backup.txt
Get-SCPFile : scp: "/backup/backup.txt": No such file or directory.
At line:1 char:1
When i use pscp it downloads the file with no issues. From the logs on the remote hardware i can see a difference in issued commands:
pscp:
executed batch command: 'scp -f /backup/backup.cfg'
Get-SCPFile:
executed batch command: 'scp -pf "/backup/backup.cfg"'
is there any way to control what switches are used in Get-SCPFile and remove quotations marks from the filename?

Because it is not $(get-credential) but just (get-cresential)
Sent from my iPhone

On Jul 6, 2016, at 5:45 AM, munchie45<EMAIL_ADDRESS>wrote:
ahhh, i was wondering why there are commas... thanks! anyway, i have another issue. when i try to download the file it says:
PS C:\cmd> Get-SCPFile -ComputerName $IP -Credential $(Get-Credential) -RemoteFile /backup/backup.txt -LocalFile c:\cmd\test.txt -Verbose -Debug
cmdlet Get-Credential at command pipeline position 1
Supply values for the following parameters:
VERBOSE: Using SSH Username and Password authentication for connection.
VERBOSE: Fingerprint matched trusted fingerprint for host <IP_ADDRESS>
VERBOSE: Connection successful
VERBOSE: Downloading backup.txt
Get-SCPFile : scp: "/backup/backup.txt": No such file or directory.
At line:1 char:1
When i use pscp it downloads the file with no issues. From the logs on the remote hardware i can see a difference in issued commands:
pscp:
executed batch command: 'scp -f /backup/backup.cfg'
Get-SCPFile:
executed batch command: 'scp -pf "/backup/backup.cfg"'
is there any way to control what switches are used in Get-SCPFile and remove quotations marks from the filename?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorry just woke up and miss read, no there is no way to control the underline switches the library sends. What gear is this?
Sent from my iPhone

On Jul 6, 2016, at 5:45 AM, munchie45<EMAIL_ADDRESS>wrote:
ahhh, i was wondering why there are commas... thanks! anyway, i have another issue. when i try to download the file it says:
PS C:\cmd> Get-SCPFile -ComputerName $IP -Credential $(Get-Credential) -RemoteFile /backup/backup.txt -LocalFile c:\cmd\test.txt -Verbose -Debug
cmdlet Get-Credential at command pipeline position 1
Supply values for the following parameters:
VERBOSE: Using SSH Username and Password authentication for connection.
VERBOSE: Fingerprint matched trusted fingerprint for host <IP_ADDRESS>
VERBOSE: Connection successful
VERBOSE: Downloading backup.txt
Get-SCPFile : scp: "/backup/backup.txt": No such file or directory.
At line:1 char:1
When i use pscp it downloads the file with no issues. From the logs on the remote hardware i can see a difference in issued commands:
pscp:
executed batch command: 'scp -f /backup/backup.cfg'
Get-SCPFile:
executed batch command: 'scp -pf "/backup/backup.cfg"'
is there any way to control what switches are used in Get-SCPFile and remove quotations marks from the filename?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Cisco IronPort

sadly I'm not able to replicate on linux or BSD hosts, I do not have Cisco gear to test

Is there anyway i can help you with troubleshooting this?
Of course i cannot allow you to use our hardware

sadly it is hardcoded in the library the scp -pf ""' command, there is no way to change this without me changing the library and probably breaking it in other platforms. sadly without access to the specific gear there is no way for me to troubleshoot it.


fix: publish not working in hlx3 without host

Fix #70

This is getting really complicated ;)

Agreed ... I think once we have migrated all projects to Helix 3, we should rewrite the Sidekick.

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


Fix get_quicklook method for onda provider

This PR aims to solve the issue of getting overviews for ONDA provider:
TypeError                                 Traceback (most recent call last)
<ipython-input-7-3c773c92be6c> in <module>
      1 workspace = "eodag_workspace_overview"
      2 quicklooks_dir = os.path.join(workspace, "quicklooks")
----> 3 product.get_quicklook(base_dir=quicklooks_dir)

~/eodag/eodag/api/product/_product.py in get_quicklook(self, filename, base_dir, progress_callback)
    399             ):
    400                 with open(quicklook_file, "wb") as fd:
--> 401                     fd.write(self.properties["quicklook"])
    402                 return quicklook_file
    403

TypeError: a bytes-like object is required, not 'str'

The quicklook in this case is a base64 string which should be decoded.

@drnextgis many thanks for this PR!

Is there anything preventing this PR from being merged?

@drnextgis no problem, it is done. Thanks again!


ARM32 call convention does not handle 8-byte integers correctly

Describe the bug
I am working with a 32-bits ARM binary that has the good idea to use ulonglongs (8-bytes) at several places.
More particularely, it uses functions with signatures like this: void fun(uint a, ulonglong b, uint c).
According to the "Procedure Call Standard for the Arm Architecture", the parameters should be handled as such:

a is written to r0
b is written to r3:r2 (note that double-words are passed in registers, and not the stack. Also note that r1 is skipped because we can only use r1:r0 OR r3:r2 for 8-bytes values)
c is written to the stack (r1 was skipped).

This is the behavior that I see in this binary, and seems consistent with the behavior of GCC.
So this brings two problems with Ghidra.
First, it assumes that 8-byte values are passed on the stack. Alright, we can tweak Ghidra/Processors/ARM/data/languages/ARM.cspec to fix that:
--- ARM.cspec.orig	2022-02-07 23:36:21.791024369 +0100
+++ ARM.cspec	2022-02-08 00:43:29.940553723 +0100
@@ -64,12 +64,18 @@
         <pentry minsize="1" maxsize="4" extension="inttype">
           <register name="r1"/>
         </pentry>
+        <pentry minsize="5" maxsize="8" extension="inttype">
+          <addr space="join" piece1="r1" piece2="r0"/>
+        </pentry>
         <pentry minsize="1" maxsize="4" extension="inttype">
           <register name="r2"/>
         </pentry>
         <pentry minsize="1" maxsize="4" extension="inttype">
           <register name="r3"/>
         </pentry>
+        <pentry minsize="5" maxsize="8" extension="inttype">
+          <addr space="join" piece1="r3" piece2="r2"/>
+        </pentry>
         <pentry minsize="1" maxsize="500" align="4">
           <addr offset="0" space="stack"/>
         </pentry>

But now there's a bug left: Ghidra will correctly assign a and b, but it will assign c to r1, because it's the first register large enough and not used yet.
I tried to tweak the cpsec file, but I couldn't find a way to make this work properly.
Did I miss something ? Would it make sense to implement a strategy to tell Ghidra to "not look back" at pentries that were skipped ?
To Reproduce
Steps to reproduce the behavior:

Download and import arm_cc_bug.so in Ghidra (see attachments).
Go to my_function
See that the decompiled code does not match the source (b is passed in stack)
Patch ARM.cspec with the above patch and run ReloadSleighLangauge.java
Still be sad (c is passed in r1).

Expected behavior

The registers should be assigned as mentionned above.

Attachments
arm_cc_bug.zip
// arm-none-eabi-gcc -g -shared -o arm_cc_bug.so arm_cc_bug.c

#include <stdint.h>
#include <stdio.h>

void my_function(uint32_t a, uint64_t b, uint32_t c)
{
    printf("a=%x\n", a);
    printf("b=%llx\n", b);
    printf("c=%x\n", c);
}

Environment (please complete the following information):

OS: Ubuntu 20.04.3 LTS
Java Version: openjdk version "11.0.13"
Ghidra Version: 10.1.2
Ghidra Origin: official ghidra-sre.org distro

I notice that I just threw a link to the ARM32 call convention without explaining the relevant information.
To be precise, we can see in the section "Handling values larger than 32 bits" that:

A double-word sized type is passed in two consecutive registers (e.g., r0 and r1, or r2 and r3).

Also, in the "Parameter Passing" section:

C.3. If the argument requires double-word alignment (8-byte), the NCRN is rounded up to the next even register number.

Where NCRN stands for "Next Core Register Number", which always goes forward.

I would piggyback here and add that it also doesn't handle arrays and structures < 0x10 bytes in size (which are passed in registers according to the ABI docs).  So I created more joined register spaces in my ARM. cspec file, which makes things a LOT better.  But then I see two additional issues (one of which you mentioned).
ARM.cspec:
--- ARM.cspec-orig	2022-04-27 14:02:43.773558271 -0400
+++ ARM.cspec	2022-04-28 12:36:33.220212374 -0400
@@ -70,6 +70,24 @@
         <pentry minsize="1" maxsize="4" extension="inttype">
           <register name="r3"/>
         </pentry>
+        <pentry minsize="13" maxsize="16">
+          <addr space="join" piece1="r3" piece2="r2" piece3="r1" piece4="r0" />
+        </pentry>
+        <pentry minsize="9" maxsize="12">
+          <addr space="join" piece1="r3" piece2="r2" piece3="r1" />
+        </pentry>
+        <pentry minsize="9" maxsize="12">
+          <addr space="join" piece1="r2" piece2="r1" piece3="r0" />
+        </pentry>
+        <pentry minsize="5" maxsize="8">
+          <addr space="join" piece1="r3" piece2="r2"/>
+        </pentry>
+        <pentry minsize="5" maxsize="8">
+          <addr space="join" piece1="r2" piece2="r1"/>
+        </pentry>
+        <pentry minsize="5" maxsize="8">
+          <addr space="join" piece1="r1" piece2="r0" />
+        </pentry>
         <pentry minsize="1" maxsize="500" align="4">
           <addr offset="0" space="stack"/>
         </pentry>

Additional problems:

8 to 12 byte arrays or structs are 4-byte and not 8-byte aligned according to the calling convention, but 8-byte doubles and ints are 8-byte aligned.  So for instance calling a function void foo(int x, double d, int y); should place:
x - r0
d - r3,r2
y - Stack
whereas calling a function void foo(int x, struct_type_8byte s, int y); places
x - r0
s - r2,r1
y - r3
AFAICT Ghidra only aligns by size, and it is set so 8-byte items are 8-byte aligned.  So after the above mods to cspec, doubles and int64s work correctly, but structs & arrays do not.

(the one you mentioned) Ghidra doesn't appear to following the incrementing core register number convention, so if it skips r1 due to alignment, it will go back and try to use r1 for a later parameter.

I've attached an example file that illustrates these cases.  I compile with ```arm-none-eabi-gcc example.c -o example.bin --specs=nosys.specs``
example.zip
`


[LICENSE] [v1.8.x] Change intgemm to a submodule instead of fetch.

Description
This resolves the issue that LICENSE referred to 3rdparty/intgemm/LICENSE which was not present in the source because it was downloaded:
https://lists.apache.org/thread.html/r669576b6506f67ea96e9b1bd385b04f59c4e6e7fa1b13a6dc0abc7d9%40<general.incubator.apache.org>
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

Changes

[x] Remove fetch from CMakeLists.txt
[x] Add submodule in 3rdparty
[x] Mention catch in top-level LICENSE

Note
Expect v1.x and master versions of this too.

sanity check is complaining about license headers in the intgemm module. you can whitelist it in https://github.com/apache/incubator-mxnet/blob/v1.8.x/tests/nightly/apache_rat_license_check/rat-excludes#L40

@mxnet-bot run ci windows-cpu
I didn't mess with batchnorm.
[2020-10-22T17:36:18.243Z] ======================================================================
[2020-10-22T17:36:18.243Z] FAIL: test_operator.test_batchnorm
[2020-10-22T17:36:18.243Z] ----------------------------------------------------------------------
[2020-10-22T17:36:18.243Z] Traceback (most recent call last):
[2020-10-22T17:36:18.243Z]   File "C:\Python37\lib\site-packages\nose\case.py", line 198, in runTest
[2020-10-22T17:36:18.243Z]     self.test(*self.arg)
[2020-10-22T17:36:18.243Z]   File "C:\jenkins_slave\workspace\ut-python-cpu\tests\python\unittest\common.py", line 218, in test_new
[2020-10-22T17:36:18.243Z]     orig_test(*args, **kwargs)
[2020-10-22T17:36:18.243Z]   File "C:\jenkins_slave\workspace\ut-python-cpu\tests\python\unittest\test_operator.py", line 1990, in test_batchnorm
[2020-10-22T17:36:18.243Z]     gamma_grad_req, beta_grad_req)
[2020-10-22T17:36:18.243Z]   File "C:\jenkins_slave\workspace\ut-python-cpu\tests\python\unittest\test_operator.py", line 1958, in _test_batchnorm_impl
[2020-10-22T17:36:18.243Z]     ), running_var.asnumpy(), atol=atol, rtol=rtol)
[2020-10-22T17:36:18.243Z]   File "C:\jenkins_slave\workspace\ut-python-cpu\windows_package\python\mxnet\test_utils.py", line 749, in assert_almost_equal
[2020-10-22T17:36:18.243Z]     raise AssertionError(msg)
[2020-10-22T17:36:18.243Z] AssertionError:
[2020-10-22T17:36:18.243Z] Items are not equal:
[2020-10-22T17:36:18.243Z] Error 1.017045 exceeds tolerance rtol=5.000000e-02, atol=5.000000e-02 (mismatch 25.000000%).
[2020-10-22T17:36:18.243Z] Location of maximum error: (1,), a=0.42502668, b=0.50137496
[2020-10-22T17:36:18.243Z]  ACTUAL: array([0.3841784 , 0.42502668, 0.36781523, 0.37314323], dtype=float32)
[2020-10-22T17:36:18.243Z]  DESIRED: array([0.41967845, 0.50137496, 0.3869521 , 0.397608  ], dtype=float32)
[2020-10-22T17:36:18.243Z] -------------------- >> begin captured stdout << ---------------------
[2020-10-22T17:36:18.243Z]
[2020-10-22T17:36:18.243Z] *** Maximum errors for vector of size 4:  rtol=0.05, atol=0.05
[2020-10-22T17:36:18.243Z]
[2020-10-22T17:36:18.243Z]   1: Error 1.017045  Location of error: (1,), a=0.42502668, b=0.50137496
[2020-10-22T17:36:18.243Z]
[2020-10-22T17:36:18.243Z] --------------------- >> end captured stdout << ----------------------
[2020-10-22T17:36:18.243Z] -------------------- >> begin captured logging << --------------------
[2020-10-22T17:36:18.243Z] common: WARNING: Error seen with seeded test, use MXNET_TEST_SEED=1495319684 to reproduce.
[2020-10-22T17:36:18.243Z] --------------------- >> end captured logging << ---------------------
[2020-10-22T17:36:18.243Z]
[2020-10-22T17:36:18.243Z] ----------------------------------------------------------------------

@mxnet-bot run ci [unix-gpu]
Why was it aborted?


Table: scroll jumping when table is bigger than viewport

If you create a long table and start clicking oh a top cells you'll be scrolled down to the bottom of table:

This has been caused by  keytrap.focus(). Just found out that there is an 'preventScroll' option for el.focus(). I remember we had some strange scroll problems in the past, similar to this, with Surface grabbing focus.

Fixed via #910


CountryCodes doesnt seem to work

1.9.3p0 :002 > E164::CountryCodes['1']
NameError: uninitialized constant E164::CountryCodes
from (irb):2
from /Users/jdyer/.rvm/rubies/ruby-1.9.3-p0/bin/irb:16:in `<main>'

Will updating the gem with the country_codes method fix the issue for you or should I do some more research on it?

I sent a push request, i think I patched what I needed.

The pull request has been merged and an update gem released!


⚠ DesiTorrents has degraded performance

In b77591f, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 4319 ms

Resolved: DesiTorrents performance has improved in 8a81404.


Link to Swagger doc broken in reverse-proxy scenario

My app was originally set up this way:
app.UseRouting();
app.UseSwagger();
app.UseSwaggerUI(options =>
{
    options.SwaggerEndpoint("/swagger/v1/swagger.json", "My API v1");
});

However I need to deploy it in a reverse-proxy scenario for production. So I modified the code as suggested in #1253:
app.UsePathBase("/some/root/path");
app.UseRouting();
app.UseSwagger();
app.UseSwaggerUI(options =>
{
    options.SwaggerEndpoint("v1/swagger.json", "My API v1");
});

Now the Swagger UI is accessible from both

/swagger
/some/root/path/swagger

... and the Swagger doc is accessible from both

/swagger/v1/swagger.json
/some/root/path/swagger/v1/swagger.json

Which is perfect.
However, looking at the Swagger UI, the link to the Swagger doc is broken. Here's the markup being generated:

        "v1/swagger.json"

Obviously, clicking on it brings up an empty about:blank page instead of the expected swagger.json.
I wasn't sure if this issue belonged over at swagger-ui instead; but since I'm following #1253, I thought I'd post here first.

.NET Core 3.1 application
Swashbuckle.AspNetCore 5.3.1

This is indeed an upstream issue with the swagger-ui library.
However, the latest version of Swashbuckle (v5.3.2) contains a workaround to address the issue in lieu of an upstream-fix. Please pull that down and let me know if that works?

Hi. I just upgrade from v5.3.1 to v5.3.3 just now and confirms that the current workaround is working.

Hi. I just upgrade from v5.3.1 to v5.3.3 just now and confirms that the current workaround is working.

Which workaround @takato1314 ?

This is indeed an upstream issue with the swagger-ui library.
However, the latest version of Swashbuckle (v5.3.2) contains a workaround to address the issue in lieu of an upstream-fix. Please pull that down and let me know if that works?

I was referring to this workaround that @domaindrivendev mentioned. Basically i got the same problem as the the author while using v5.3.1 and after upgrading it to the latest (v5.3.3), then the same problem was resolved on my side.

Sorry, I got pulled away onto other things for a while. I brought down the latest library (5.4.1) and it's working for me too. Thanks for your help!


Add vimbindings

Is Your Feature Request Related to a Problem? Please Describe

Solution You'd Like

Alternatives You've Considered

Additional Context

It's currently possible to hide all UI by pressing CmdOrCtrl+Shift+H

Can you elaborate on how vim bindings would work?

CmdOrCtrl+Shift+H does noting 4 me. still have a title bar/tab bar and adressbar.
install vimium on chrome or vimvixen on firefox and have a look for a while.
https://www.youtube.com/watch?v=t67Sn0RGK54
press f to select clickable link with links like q or s or as or b.
and lots of others.

+1 and I would also strongly support vim mode for the editor. Vim keybindings for browsing are nice, but vim for editing is essential.


Telegram problem

I have been trying to do this bot for several hours, but now I have this problem, how can I fix it? thank you so much
[root] There was an issue sending to Telegram due to an invalid request: 401 -> {"ok":false,"error_code":401,"description":"Unauthorized"}

I got the same, but then I appended /sendMessage at the end of my webhook and worked

I got the same, but then I appended /sendMessage at the end of my webhook and worked


cloudflare_spectrum_application nested syntax for edge_ips "Error: missing expected {"

Confirmation

[X] My issue isn't already found on the issue tracker.
[X] I have replicated my issue using the latest version of the provider and it is still present.

Terraform and Cloudflare provider version
Terraform v1.5.0
Provider plugins:

hashicorp/aws v5.2.0
cloudflare/cloudflare v4.7.1

Affected resource(s)
cloudflare_spectrum_application
Terraform configuration files
resource "cloudflare_spectrum_application" "endpoint" {
  zone_id      = cloudflare_zone.main.id
  traffic_type = "direct"

  dns {
    type = "ADDRESS"
    name = [redacted]
  }

  edge_ips {
    type = "static"
    ips  = [redacted]
  }
  protocol = "tcp/80"

  origin_dns {
    name = cloudflare_load_balancer.main.name
  }
  origin_port    = [redacted]
  tls            = "off"
  ip_firewall    = false
  proxy_protocol = "v2"
}

Link to debug output
n/a
Panic output
n/a
Expected output
Successful terraform plan run
Actual output
Planning failed. Terraform encountered an error while generating this plan.
╷
│ Error: missing expected {
│
│   with module.nessy.cloudflare_spectrum_application.endpoint,
│   on ../../modules/proxy/main.tf line 147, in resource "cloudflare_spectrum_application" "endpoint":
│  147: resource "cloudflare_spectrum_application" "endpoint" {
│
╵
Steps to reproduce

Upgrade Cloudflare provider to v4.x
Update cloudflare_spectrum_application to use nested syntax for edge_ips per https://registry.terraform.io/providers/cloudflare/cloudflare/latest/docs/resources/spectrum_application
i.e. replace

edge_ips = [redacted]

with
edge_ips {
    type  = "static"
    ips   = [redacted]
}

Run terraform plan

Additional factoids
Works fine when
edge_ips {
    connectivity = "all"
    type         = "dynamic"
}

References
Introduced in https://github.com/cloudflare/terraform-provider-cloudflare/pull/2219

you'll need to remove the resource from state and reimport it to get the structure correct. there wasn't an automatic migration for this due to the resource previously allowing invalid configurations that we weren't comfortable copying forward or breaking the upgrade path.


Allow more than 16 vCPUs

Currently https://github.com/docker/hyperkit/blob/master/src/include/xhyve/vmm/vmm_common.h#L33 restricts the maximum number of vCPUs to 16. For compute-heavy workloads, this arbitrary restriction may be undesirable (see docker/for-mac#1144). This number is mainly used to size static arrays but, as @ijc25 notes, there may be work checking that ACPI tables are set up correctly to offer enough cores correctly.

acpitbl.c takes a number of CPU argument and scales the various ACPI tables and their content by it. So it may work...

Just be aware of table sizing/overflow (http://docs.FreeBSD.org/cgi/mid.cgi?2ebbc1f9-4b1e-74f2-60fc-c0976b7924a1)

@grehan-freebsd thanks for the heads up!


feat(NumberTheory/LSeries): completed L-functions

Define the completed L-function for a function on ZMod N, and prove its functional equation.

This is great, thanks!
Can you please split this in 3 PRs with:

the addition to EvenFunction (this can be merged very quickly)
the stylistic modifications you made to LSeries.ZMod (again can be merged very quickly)
the main one.

Feel free to ask my review.

Thanks for the feedback! I have split off #16911 for the material on even/odd functions, and I will do another PR shortly for the stylistic changes / refactors of existing results in LSeries.ZMod.

Thanks for the review!
bors r+


Fix config.get for schemas with objects

The config.get method does not return the default values for objects with properties in a config schema.
Example
For example the editor config has:
(excerpt taken from config-schema.coffee)
...
  backUpBeforeSaving:
    type: 'boolean'
    default: false
    description: 'Ensure file contents aren\'t lost if there is an I/O error during save by making a temporary backup copy.'
  invisibles:
    type: 'object'
    properties:
      eol:
        type: ['boolean', 'string']
        default: '\u00ac'
        maximumLength: 1
      space:
        type: ['boolean', 'string']
        default: '\u00b7'
        maximumLength: 1
      tab:
        type: ['boolean', 'string']
        default: '\u00bb'
        maximumLength: 1
      cr:
        type: ['boolean', 'string']
        default: '\u00a4'
        maximumLength: 1
...

A call to atom.config.get('editor') returns an object with an 'invisibles' value that is an empty object. We are expecting an object containing the default values for the properties in 'invisibles'.
This result is caused by _.defaults() in @getRawValue() not being recursive. The lack of a _.deepDefaults() lead me to use a bit of a workaround with _.deepExtends(). Now with _.deepExtends(), the default values are the base while any new values read from the config.cson overwrite the defaults. This way all default values, even nested ones, appear in the result.
Side-Effect
A side effect of this issue was that atom/settings-view does not properly present the available configuration options from the config schema. The appendObject() method in settings-view/lib/settings-panel.coffee uses the values provided by config.get(packageName). The lack of keys in the value provided to appendObject would lead to objects in the schema disappearing from the SettingsPanel. With this proposed change, the SettingsPanel now properly displays objects and their properties.

It seems that this method does not return a deep clone of the value. I am currently looking for a better alternative to cloning both the value and default value.

Thanks for helping out! This will require at least one test to verify functionality and to prevent regressions.

Also note that this currently has failing core specs, and 1 failing settings-view spec.

Sorry to trouble you, but how would I go about seeing exactly which specs this is failing? I tried going into atom safe mode and running the atom specs, but that displays errors on both the master branch and my fix branch.

Never mind, it seems to have passed this time. I recommend that the deepDefaults method I introduced be moved to the atom/underscore-plus repository. If you want, I can take that code and submit a pull request there.

:zap: Thanks for tracking this down and fixing it!

I recommend that the deepDefaults method I introduced be moved to the atom/underscore-plus repository.

I actually think it's ok to leave this function where it is, because we have specific concerns here about only merging plain objects, not instances of custom constructors like Snippet.

it seems to have passed this time.

Awesome. Can you still add a test for this functionality? You might want to just add one or more assertions to this test.

I went ahead and added two assertions to the test you specified.

Great work! I'm so glad you fixed this. The failing travis build is something that was failing on master briefly and has been fixed now.


Updates

Category

[X] Content fix, updates

Docs Build status updates of commit ab8ebdc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Community/principles-of-communication.md
:bulb:Suggestion
View
Details

Community/TOC.md
:white_check_mark:Succeeded
View

Community/index.yml
:white_check_mark:Succeeded
View

Community/library-scenarios.md
:white_check_mark:Succeeded
View

Community/select-an-effective-navigation-strategy.md
:white_check_mark:Succeeded
View

Community/site-design-life-cycle.md
:white_check_mark:Succeeded
View

Community/principles-of-communication.md

Line 44, Column 0: [Suggestion-disallowed-heading] Heading 'Feedback' matches the text of a disallowed Docs template heading. NOTE: This Suggestion will become a Warning in late September, 2020.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


bugfix:seata-server can's start in IDE.

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixex #1052
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1053 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             develop   #1053   +/-   ##
=========================================
  Coverage       38.5%   38.5%
  Complexity      1076    1076
=========================================
  Files            220     220
  Lines           8774    8774
  Branches        1108    1108
=========================================
  Hits            3378    3378
  Misses          4966    4966
  Partials         430     430

Continue to review full report at Codecov.

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

ps: the reason https://github.com/seata/seata/pull/942

Had fixed it in #942

Had fixed it in #942

That PR hasn't been merged yet.We can fix it in this small PR ,otherwise the people who cloned the code and run the server in IDE will get confused.


PHP 8 support

composer.json limits PHP version to 7.x.
Any chance of supporting PHP 8.0 or even 8.1?

does it work out of the box? If you send a non braking PR I'll merge it. Sadly I don't have time to test it right now.


swupdate-common: fix sha256 regex

Change sha256 regex brackets to parenthesis to apply only on line with 'sha256'
and 'version'.

Fix an issue taht can be reproduced easily: add in any entry:
hello = "@something"
The regex will match and swupdate will look for file 'something'
Or try the python code:
line = "    hello = \"@something\""
re.match(r"^(?P<before_placeholder>.+)[sha256|version].+[=:].*(?P<quote>[\'\"])@(?P<filename>.*)(?P=quote)", line)

which returns a match

Seriously. Mailing lists... googlegroups... How could you make that terrific combo that repulse many new user to contribute ? What about making a step in 21st century ?

integrated in 23c0ed1dc33f7408f0f02607e27f8946503893fa


Standard shader ball asset

Finally, the standard shader ball PR is here :)
Thanks for everyone's patience and contributions, especially Andre Mazzone and Thomas An. And thanks in advance for input and feedback on the PR.

Just a quick note: the src directory has the scripts and hip file used to produce all of the images. This is mainly in case higher-res maps/data is needed/desirable, but also just to provide instructions examples. The USD data itself should be considered the "canonical" source, as there will surely be contributions that don't original from Houdini. But if/when that happens, it would be nice (but not a requirement) if those files could also end up in the src directory if possible.

Apropos the conversation today, does everything pass usdchecker?

Great work! 👍
I'm getting following warning:
Warning: in _AddParameterInputsToTerminalNode at line 451 of F:\USD-23.08\pxr\imaging\hdMtlx\hdMtlx.cpp -- NodeDef not found for Node 'standard_surface'
Warning: in HdMtlxCreateMtlxDocumentFromHdMaterialNetworkInterface at line 556 of F:\USD-23.08\pxr\imaging\hdMtlx\hdMtlx.cpp -- Validation warnings for generated MaterialX file.
Node interface error: Input 'emission_color' doesn't match declaration: <standard_surface name="Surface" type="surfaceshader">

because in ./layers/shaderball.usda:85, a color3 input is connected to a vector3 image node:
color3f inputs:emission_color.connect = </standard_shader_ball_scene/materials/neutral/mtlx/mtlximage2.outputs:out>
<!-- ... -->
def Shader "mtlximage2"
{
  uniform token info:id = "ND_image_vector3"
  float2 inputs:texcoord.connect = </standard_shader_ball_scene/materials/neutral/mtlx/mtlxtexcoord1.outputs:out>
  string inputs:uaddressmode = "clamp"
  string inputs:vaddressmode = "clamp"
  vector3f outputs:out
}

(which seems to be a common issue - see https://github.com/AcademySoftwareFoundation/MaterialX/issues/1038)

Just pushed up a new update; I squashed the various commits, because I'd let some misplaced files in. But the latest commit has the following updates:

Fixed some strange tmp paths (thanks @AlexSchwank)
Fixed type mismatches reported from MaterialX (thanks @pablode)
Simplified and corrected the camera (thanks @drwave), removed DoF, and re-rendered the images w/ the new camera
Fixed the broken usdpreview example materials
Removed the mistaken ‘layers’ folder and usd files from ‘maps’, and the 'material_surface_geo' folder from the layers dir

This latest version also passes usdchecker, @jcowles  :)

Merging! 🎉 🎉 🎉


matrix-synapse: 1.35.1 -> 1.36.0

Motivation for this change
https://github.com/matrix-org/synapse/releases/tag/v1.36.0rc1
https://github.com/matrix-org/synapse/releases/tag/v1.36.0rc2
https://github.com/matrix-org/synapse/releases/tag/v1.36.0
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

@dotlambda done


Update login method

Steam create new login method, sessions keep alive longer than previos method.
I found one library which have implementation for it.
https://github.com/sometastycake/pysteamauth

Сделал :)


Allow casks with outdated caskfiles to be reinstalled

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

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

The problem occurred when calling brew reinstall on a cask with an out of date caskfile as described in issue #10833.
To solve the problem Cask::Installer#uninstall_existing_cask has been changed to catch a possible CaskInvalidError when trying to load outdated caskfiles using Cask::CaskLoader#load.
Fixes #10833.

Great work, thanks again @apainintheneck!


trouble starting chroot

Please paste the output of the following command here: sudo edit-chroot -all
```name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
/usr/local/bin/enter-chroot: 511: cannot create /run/crouton/mnt/stateful_partition/crouton/chroots/xenial/etc/crouton/name: Directory nonexistent
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

#### Please describe your issue:
when i try to create a chroot with sudo enter-chroot or sudo enter-chroot startxfce4 i get this
No chroots found in /mnt/stateful_partition/crouton/chroots
why is this, i have tried updating xenial, but that hasn't worked

#### If known, describe the steps to reproduce the issue:
![screenshot 2017-07-29 at 10 33 40 pm](https://user-images.githubusercontent.com/28377119/28750133-33e89e8e-74ae-11e7-9334-836a2daa8ec4.png)

@aghasemi,
You may have some permission problems with your chroot.
Try following this wiki guide for help: 'Common issues and reporting'
Hope this helps,
-DennisL

Are you starting with root perms?

i have already read and tried the common issues page and looked at updating
it, but i get the same errors, and i have rootfs and my root/sudo passwd.
On Sat, Aug 5, 2017 at 7:04 AM, 9291Sam<EMAIL_ADDRESS>wrote:

Are you starting with root perms?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dnschneid/crouton/issues/3346#issuecomment-320437542,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbEAH8V3ydglMpgiurik9Hf6HeqJ4Fv5ks5sVExUgaJpZM4Onpsb
.

Have a you (seriously) tried turning it off and back on again and then if that does not work reinstall the chroot after deleting it

@arcaniussainey,
Please show us what is displayed when you enter the following two commands:

ls -l /usr/local/chroots
ls -l /mnt/stateful_partition/crouton/chroots

Hope this helps,
-DennisL

Uninstall/re-install worked, and i installed gnome.


[OCP-4.X] Check scale up playbook with Azure cluster

Validate that the OpenShift 4 scale playbook works with Azure and make changes if necessary to support both AWS and Azure.

#33 works with Azure


Display profile images in buddy list

At present, it can be hard to quickly find the user you are looking for in the buddy list.
We should make this easier by adding an option to show the user's profile picture to the left of their name. The status icon should appear in the bottom right of the picture, similarly to other apps (e.g. Discord, Gmail, Slack):

The vertical spacing between users' names will need to be increased to accommodate reasonably-sized pictures.
Because showing profile photos in the buddy list takes up more vertical space, we should have a display setting to control whether or not to do so.
We can consider adding this feature to the PMs in the left sidebar as well, but it may be best to sequence this change after #19447.
CZO discussion thread

I think we want whether to display avatars in the sidebar to be an optional UX setting; it has a significant cost in terms of how many users one can see in the sidebar, how visually busy that part of the screen is, etc., which are tradeoffs that not every user will want.

Hi @timabbott and @alya
Thanks to you and all the developers for this attractive product.
It's been always an struggle for me and my colleagues to find the users among the other ones.
It's not just about the avatars. But also the arrangement of the users. Sorting the users by their online status doesn't work for us. (someone who has recently become offline would be moved to the end of list... we should scroll down to send him/her a direct message)
However we can find the users who we've recently chatted with on the left pane (direct messages), but it seems like it is duplicated and they can be somehow merged)
Sincerely yours

@zulipbot claim

@Chinemereem please use @zulipbot abandon to unclaim an issue.
You may work on any help wanted issue that seems approachable, but we ask that you tackle one issue at a time to start with. Please see https://zulip.readthedocs.io/en/latest/overview/contributing.html for more info.

@zulipbot claim


Allow exclusion of other resource types

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: mattinger:x: Matthew IngerMatthew Inger seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Add] Shadcn/ui to frontend ui-libraries

Site URL
https://ui.shadcn.com/
Description of the above provided site
It provides you beautifully designed components that you can copy and paste into your apps. Accessible. Customizable. Open Source. Built using Radix UI and Tailwind CSS.
Category
Frontend
Sub Category
Ui Libraries
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am participating in Hacktoberfest 2023

@Hrithik0112
I don't think it is a library.

@Anmol-Baranwal it's reusable component library to be specific.

@Anmol-Baranwal but it's getting popular day by day, many experienced Dev's address this as a UI library.

@Anmol-Baranwal but it's getting popular day by day, many experienced Dev's address this as a UI library.

I've used it, and I'm aware.

Let me check where it fits.

@Hrithik0112
Since Mamba UI is added in UI Generators.

You can add this in UI Generators. Update the description and I'm assigning.

@Anmol-Baranwal thank you for guiding me.


Add tooling to deploy compiled smart contracts

Problem
Currently, there is no tooling available in neo-storm to deploy compiled smart contracts.
Solution
We could create a native Go implementation for deploying them, a better solution would be to create some kind of Python wrapper that enables this without to much pain for the end-user.
Discussion can be continued in comments below

@ansrivas This would something similar yes. Any suggestions you may have?

@ansrivas That seems like a very good approach! I will play with it tomorrow.

I think it's fine when deploying to private network with neo-local + neo-python.
I have deployed a couple of smart contracts to the MainNet and it's a pain to have to wait for neo-python to finish syncing. Slow and unreliable. (If I were good at python I would totally contribute to neo-python)
It would be great if we have a way to deploy on either TestNet or MainNet without relying on a node to be fully synced. All it needs is just the UTXO that has enough gas to deploy as a transaction input.  Of course, we still have to rely on UTXO's API server like neoscan or NEL to return that UTXO data but that seems better in my opinion.

@apisit @ansrivas I think we better wait for the updates on the CLI tooling in neo-local. After that is implemented we can use that to bootstrap neo-storm on top of it. What do you guys think?


Refactor settings.scss

I merged a batch of early commits, and stopped when I saw the issue posted above.

Overall, thanks for writing this in a nice, readable, reviewable way!  And let me know when you've fixed the above.  (Easily visible with git show if you have the color-moved feature enabled).

I merged many of these commits after fixing the small bug above; let me know when you've rebased this.  Thanks @majordwarf!

Okay, I did rebase with upstream/master, anything that needs to be done now? A bit confused here :thinking:

@timabbott Seems like I messed up something while resolving the conflicts, other commits are appearing too in the history.

@majordwarf let me know when you have this updated; I showed you how to get your original code back.

The commits here seem to be all broken from rebasing; @majordwarf can you just redo this PR?


Update Golang status

I believe Loadsmart is waaay past the "trial" phase when it comes to Golang. It is a established language in the Engineering team now.

Agreed 💯


Fix minecart/boat stopping on collide with vanished player

Minecarts and boats stop moving if they collide with a vanished player. This mixin fixes that (for both, even tho its only in the boat class... Mojang why...)

Note: If the player in the boat can see vanished players, they will still collide on the client (but not on the server). Not really sure if there is a way to fix that, but I don't think its a big concern since you can see the thing your running into. This doesn't seem to effect minecarts at all.

Just realized this should probably be tied to one of the interaction configs, will update tomorrow.


Separates out focusedInput from DOM Focus and focuses on the calendar for withPortal implementations

This, as far as I can tell, should be a fix for https://github.com/airbnb/react-dates/issues/246.
I'm going to favor this PR over https://github.com/airbnb/react-dates/pull/409 because it paves the way better for https://github.com/airbnb/react-dates/pull/301.
@airbnb/webinfra @moonboots PTAL

Coverage increased (+0.04%) to 87.359% when pulling 23fa1f18887dc88df609a9389cc33422a800dfa3 on maja-fix-done-bottom-bar-on-ios into f99432a463037988db7d18c2d2a0c6f848ec5df8 on master.

@moonboots PTAL

Coverage decreased (-0.04%) to 87.282% when pulling 93fde60815fe509c30f0c3a6b65751f21cd9ff8d on maja-fix-done-bottom-bar-on-ios into f1cee85aeeb8b92119ccfc8a5a424ab0107eb307 on master.

Coverage decreased (-0.009%) to 87.313% when pulling 93fde60815fe509c30f0c3a6b65751f21cd9ff8d on maja-fix-done-bottom-bar-on-ios into f1cee85aeeb8b92119ccfc8a5a424ab0107eb307 on master.

Coverage decreased (-0.009%) to 87.313% when pulling 03d53260b6c5966b54270344bba71a30d7da9ea3 on maja-fix-done-bottom-bar-on-ios into f1cee85aeeb8b92119ccfc8a5a424ab0107eb307 on master.


Authenticate no longer works

Hello Overv, at first thx for the great work, but since a few days, the request.GetResponse() returns always a 'Internal Server Error'.

Sorry, that was my problem, I have changed the user agent. Your project works great.


[Button] - Icon Button Group Should center icon inside button

icons inside an icon button group appear off-center
example in button docs
the issue seems to be located here:
// src/definitions/elements/button.less:594
.ui.button:not(.icon) > .icon:not(.button):not(.dropdown) {
  margin: @iconMargin;
}

This is a regression that occurred sometime recently. I'll try to address shortly.

how can I add font awesome icon ??


Polymorphic types, easy way to select by type (___type field)

kotlinx-serialization does not require an explicit discriminator column when serializing polymorphic types. These end up in the db as ___type.
Since there is no explicit property there, I don't believe there is a type-safe API to create a filter expression for documents matching a specific type or types.
It would be great if kmongo exposed a type-safe api for type discrimination.

I'm using this extension as a work-around for now:
fun <T: Any, S: T> KProperty<T?>.subType(subType: KClass<S>): Bson =
  Filters.eq("${path()}.___type", subType.qualifiedName)

By the way, You can change the ___type discriminator using the configuration var https://litote.org/kmongo/dokka/kmongo/org.litote.kmongo.serialization/configuration.html - I'm going to add these properties:
/**
 * Provides the path for subtype discriminator, in order to write this kind of filter:
 *
 * eq(subtypePath, B::class.subtypeQualifier)
 */
val subtypePath: String get() = configuration.classDiscriminator

/**
 * Provides the path for subtype discriminator, in order to write this kind of filter:
 *
 *  eq(A::myProperty.subtypePath, B::class.subtypeQualifier)
 */
val KProperty<*>.subtypePath: String get() = "${path()}.${configuration.classDiscriminator}"

/**
 * Provides the value of subtype discriminator.
 */
val KClass<*>.subtypeQualifier: String get() = qualifiedName ?: error("no qualified name for class $this")

Thank you. I find the syntax that falls back to the mongo drivers Filter.* calls a bit messy. I've created these functions:
infix fun <T: Any> KProperty<T?>.isASubTypeOf(ofType: KClass<*>): Bson =
  Filters.eq(subtypePath, ofType.subtypeQualifier)

infix fun <T: Any> KProperty<T?>.isNotASubTypeOf(ofType: KClass<*>): Bson =
  Filters.ne(subtypePath, ofType.subtypeQualifier)

which can be used like this:
Foo::bar isASubTypeOf Bar::class


Rename Interaction::Click

What problem does this solve or what need does it fill?
When I saw this variant, I thought that it works like in usual in UI - if I press and release the RMB over the button, then it is considered as "clicked". Also all examples check Interaction::Click for button pressing.
But it's not how it works! Interaction::Click just represents the pressed state of a button.
What solution would you like?
Rename Interaction::Click into Interaction::Pressed. This will by much clear. No functional changes.
Additional context
I think Interaction represents the state of the button and it's handy for changing the appearance.
But to check for clicking, I would add additional mechanism and I think it should be an event, not a state (an RMB press and then release over the button).

I don't mind giving this one a go if no one else is doing it :)

@AnthonyKalaitzis do it, maybe we could get it to 0.11 since it's a simple rename.

@Shatur When you say Interaction::Click, do you mean Interaction::Clicked? The Interaction enum appears to be in bevy_ui/src/focus.rs and I don't see any others so just double checking that's what you mean.

@AnthonyKalaitzis Yes :)


Data quality checks

We should go through the whole site and check that we are reporting the same numbers everywhere. Dashboard, reports, explore data and download data.

All priority issues have been moved to the private meerkat_issues repo.


split packages in spring-cloud-commons prevents usage of java modules when reactor core is used

Describe the bug
When using the java module system, there are split packages between the reactor core library and spring-cloud-commons. This prevents the system from loading the modules since it violates the package rule.
The following error is thrown when attempting to start a spring boot application:
Error occurred during initialization of boot layer
java.lang.module.ResolutionException: Modules reactor.core and spring.cloud.commons export package reactor.core.publisher to module spring.boot.starter.actuator

Sample
Create a project with the following dependencies:
<dependency>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starter-actuator</artifactId>
</dependency>
<dependency>
    <groupId>org.springframework.boot</groupId>
    <artifactId>spring-boot-starer-web</artifactId>
</dependency>
<dependency>
    <groupId>org.springframework.cloud</groupId>
    <artifactId>spring-cloud-starter-cloud-discovery</artifactId>
</dependency>
<dependency>
    <groupId>io.projectreactor</groupId>
    <artifactId>reactor-core</artifactId>
    <version>3.4.2</version>
</dependency>

Define the required modules in module-info.java:
requires spring.web;
requires spring.boot;
requires spring.boot.autoconfigure;
requires spring.boot.starter.web;
requires spring.boot.starter.tomcat;
requires spring.boot.starter.actuator;
requires org.apache.tomcat.embed.core;
requires org.apache.tomcat.embed.websocket;
requires com.fasterxml.jackson.databind;

requires reactor.core;

exports my.personal.beans;

opens my.personal.beans to spring.core;

Possible solution:
Rename the package reactor.core.publisher from spring-cloud-commons so it doesn't conflict with the same package name found in the reactor core library

Unfortunately, it uses package protected class is reactor.core.publisher.
There's a note that this functionality will be moved to reactor and can then be removed here. @TYsewyn and @simonbasle would need to give us an update.

Unfortunately, it uses package protected class is reactor.core.publisher.
There's a note that this functionality will be moved to reactor and can then be removed here. @TYsewyn and @simonbasle would need to give us an update.

Unfortunately, it uses package protected class is reactor.core.publisher so it can't just be renamed.
There's a note that this functionality will be moved to reactor and can then be removed here. @TYsewyn and @simonbasle would need to give us an update.

If the only package-protected classes are the implemented interfaces like SourceProducer, it should be possible to do without them (although it would probably imply at least replacing these occurrences by the interface(s) they themselves extend).
That said, from 3.4.0 onwards we now have Flux.firstWithValue, would that fit the bill @TYsewyn @spencergibb ?

Unfortunately, it uses package protected class is reactor.core.publisher so it can't just be renamed.
There's a note that this functionality will be moved to reactor and can then be removed here. @TYsewyn and @simonbasle would need to give us an update.

If the only package-protected classes are the implemented interfaces like SourceProducer, it should be possible to do without them (although it would probably imply at least replacing these occurrences by the interface(s) they themselves extend).
That said, from 3.4.0 onwards we now have Flux.firstWithValue, would that fit the bill @TYsewyn @spencergibb ?

I’d need to look into that (new?) functionality.
In any case, I should have fixed this some time ago.
I’ll check this later today.

I’d need to look into that (new?) functionality.
In any case, I should have fixed this some time ago.
I’ll check this later today.

@spencergibb, we'll have to discuss if Flux.firstWithValue might make sense for us.
There's a small but very distinct difference between the 2 constructions.
Where Flux.firstWithValue is only taking the onNext (so first value) into account, CloudFlux.firstNonEmpty is also taking the first onError signal into account.
I believe this was set up to have a "fail fast" approach, rather than waiting for the first DiscoveryClient who would respond to the getInstances call in case you have multiple discovery clients configured.
If it doesn't fit our use case (or we want to support both using a flag, maybe?) I'll move that piece of functionality over to Reactor @simonbasle

@spencergibb, we'll have to discuss if Flux.firstWithValue might make sense for us.
There's a small but very distinct difference between the 2 constructions.
Where Flux.firstWithValue is only taking the onNext (so first value) into account, CloudFlux.firstNonEmpty is also taking the first onError signal into account.
I believe this was set up to have a "fail fast" approach, rather than waiting for the first DiscoveryClient who would respond to the getInstances call in case you have multiple discovery clients configured.
If it doesn't fit our use case (or we want to support both using a flag, maybe?) I'll move that piece of functionality over to Reactor @simonbasle

are we sure that the same cannot be achieved by composing some operators over the sources in Flux.firstWithSignal?
I guess that if you had to add this custom operator, it was because the first (now firstWithSignal) also shortcuts on empty sources, and as a consequence I'm assuming that such empty sources can happen for DiscoveryClient?
But if that is the case, could switchIfEmpty(Mono.delay(d).then(Mono.empty())) work to protect against such sources?
The idea being that d is set high enough (typically congruent with a .timeout) that other sources have a chance to resolve or error faster. The downside being that if ALL sources are empty, then the request will take d to produce nothing (but again, I expect a timeout after that).

are we sure that the same cannot be achieved by composing some operators over the sources in Flux.firstWithSignal?
I guess that if you had to add this custom operator, it was because the first (now firstWithSignal) also shortcuts on empty sources, and as a consequence I'm assuming that such empty sources can happen for DiscoveryClient?
But if that is the case, could switchIfEmpty(Mono.delay(d).then(Mono.empty())) work to protect against such sources?
The idea being that d is set high enough (typically congruent with a .timeout) that other sources have a chance to resolve or error faster. The downside being that if ALL sources are empty, then the request will take d to produce nothing (but again, I expect a timeout after that).

So just doing the following
+++ b/spring-cloud-commons/src/main/java/reactor/core/publisher/FluxFirstNonEmptyEmitting.java
@@ -30,7 +30,7 @@ import reactor.util.annotation.Nullable;
 /**
  * @author Tim Ysewyn
  */
-final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements SourceProducer<T> {
+final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements Scannable, Publisher<T> {

        final Publisher<? extends T>[] array;

@@ -131,6 +131,11 @@ final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements SourceProduc
                return null; // no particular key to be represented, still useful in hooks
        }

+       @Override
+       public String stepName() {
+               return "source(" + getClass().getSimpleName() + ")";
+       }
+

and FluxFirstNonEmptyEmittingTests all pass.
Is that sufficient for now and we can change the package?

So just doing the following
+++ b/spring-cloud-commons/src/main/java/reactor/core/publisher/FluxFirstNonEmptyEmitting.java
@@ -30,7 +30,7 @@ import reactor.util.annotation.Nullable;
 /**
  * @author Tim Ysewyn
  */
-final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements SourceProducer<T> {
+final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements Scannable, Publisher<T> {

        final Publisher<? extends T>[] array;

@@ -131,6 +131,11 @@ final class FluxFirstNonEmptyEmitting<T> extends Flux<T> implements SourceProduc
                return null; // no particular key to be represented, still useful in hooks
        }

+       @Override
+       public String stepName() {
+               return "source(" + getClass().getSimpleName() + ")";
+       }
+

and FluxFirstNonEmptyEmittingTests all pass.
Is that sufficient for now and we can change the package?

Looks like we're dependent on other package private interfaces :-(

Looks like we're dependent on other package private interfaces :-(

That test suite is good because just switching naively to Flux.firstWithSignal() fails two tests.

That test suite is good because just switching naively to Flux.firstWithSignal() fails two tests.


[WIP] Better shape checks at test time

This PR ensures that if an estimator can deal with variable-length inputs, then the only constraint at test time is that the number of features is equal to that provided at fit time.
This is a bugfix for #220

Hello @rtavenar! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file tslearn/utils.py:

Line 62:80: E501 line too long (98 > 79 characters)
Line 83:5: E303 too many blank lines (2)

Codecov Report

Merging #223 into dev will decrease coverage by 0.41%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##              dev     #223      +/-   ##
==========================================
- Coverage   94.01%   93.59%   -0.42%
==========================================
  Files          25       25
  Lines        3493     3530      +37
==========================================
+ Hits         3284     3304      +20
- Misses        209      226      +17

Impacted Files
Coverage Δ

tslearn/svm.py
97.69% <66.66%> (-0.75%)
:arrow_down:

tslearn/tests/sklearn_patches.py
90.41% <79.31%> (-1.04%)
:arrow_down:

tslearn/utils.py
91.89% <84.61%> (-1.30%)
:arrow_down:

tslearn/clustering.py
93.56% <100.00%> (-0.24%)
:arrow_down:

tslearn/neighbors.py
86.50% <100.00%> (-0.54%)
:arrow_down:

tslearn/preprocessing.py
94.04% <100.00%> (+0.14%)
:arrow_up:

tslearn/shapelets.py
94.59% <100.00%> (+0.02%)
:arrow_up:

tslearn/datasets.py
88.69% <0.00%> (-1.74%)
:arrow_down:

tslearn/tests/test_estimators.py
88.09% <0.00%> (-1.20%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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

Great work @rtavenar I couldn't review yesterday, but I took a quick glance and it looks good! Especially that you created a new custom test for it is really nice.
I could give a more extensive review later today if need be!

That would be great @GillesVandewiele

Link to Travis CI job: https://travis-ci.org/github/rtavenar/tslearn/builds/680423839


script to determin if sentence is a question

Close #53
Task Description
This code is uses nltk library to determining if a sentence is a question or statement.
Time Spent

Activity
Time Spend

Research
2 hours

writing code and testing
4 hours

Instruction

Run the code
Enter a sentence .
You should get result based on the sentence you entered.

Result

What if we have a statement like "you shouldn't do that."? won't this script think it's a question because it has the word "shouldn't" in it?

What if we have a statement like "you shouldn't do that."? won't this script think it's a question because it has the word "shouldn't" in it?

No it should not because its only checking for the first word.


MySQL Table 'foo' from one of the SELECTs cannot be used in global ORDER clause

TPC-DS Q76, Q66, Q60, Q56, Q33
Query Error Execution error: Unable to query arrow: Server error: `ERROR 42000 (1250): Table 'foo' from one of the SELECTs cannot be used in global ORDER clause

Should be fixed by https://github.com/spiceai/datafusion/pull/39

Should be fixed by spiceai/datafusion#39

can confirm, the fix has resolved these issues for MySQL. Thanks!


Why is the code in the article not highlighted?

First of all, thank you for this open source project that provided me with the inspiration to build my own blog!!
I noticed that the code in your online deployed blog post was not highlighted. What's going on?
And I also use nuxt-content as my CMS, and use @tailwindcss/typography as the .md style renderer. It can be highlighted normally in the dev environment, but once deployed to vercel, I will find that it cannot be highlighted.

you can see that there is no shiki classes.

this is the normal condition in dev env.
I don't know whether you've encountered the same problem as I have? but I don't know how to solve it (T o T)

I found that when I build it and preview at localhost:3000 it works... I think it might be a Vercel deployment issue??????


[Add] Chai aur Code Youtube Channel

Site URL
https://www.youtube.com/@chaiaurcode
Description of the above provided site
This is an underrated youtube channel mainly focussed on full stack development.
Category
Youtube
Sub Category
web-development
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am participating in Hacktoberfest 2023

@pratikkumar399
I have assigned you the issue. Please proceed with it, and don't hesitate to tag me @Anmol-Baranwal or @CBID2 if you require any assistance.

Hi, @Anmol-Baranwal I have opened a pr. Please check if it fine.


v2.1 Identity authentication middleware

As I understood there is new way for authorization in .netCore 2.1. As I created a new fresh project with the following dotnet cli command:
dotnet new mvc -au individual
It sets me to use identity, which I wanted, now I want to intercept the authentication so I can insert a bit more middle ware logic in there. As I need to port the info .net framework, I need to be able to do additional things to authentication.
I have tried to add something like this:
services.AddAuthentication(options =>             {                 options.DefaultAuthenticateScheme = CookieAuthenticationDefaults.AuthenticationScheme;                 options.DefaultChallengeScheme = CookieAuthenticationDefaults.AuthenticationScheme;             });
and with this after a login the cookie is present in browser but the browser doesn't show any user info, looks like the user did not login.
From the above I my feeling is that there is a default authentication. How can I intercept that?

What sort of things are you wanting to do? There might be a better way than middleware, for example events

Hi. We're closing this issue as no response or updates have been provided in a timely manner and we have been unable to reproduce it. If you have more details and are encountering this issue please add a new reply and re-open the issue.


AssemblyFileRevisionType AutoIncrement

I just wanted to verify where the AutoIncrement base number is pulled from.
It appears it's reading the base value from the AssemblyInfo.* file.
MSBuildExtensionPack/Solutions/Main/Framework/Framework/AssemblyInfo/AssemblyInfo.cs
If I pull clean sources from my scm on each build, I am assuming (and tests show) that the AutoIncrement schema will not work.
Is there an alternate storage mechanism for the AutoIncrement number I'm missing?

IIRC that will operate on any files passed into the AssemblyInfoFiles property. If you get a file at the same version each time then the result will be the same each time, unless your are committing that version change.


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=530425602
https://github.com/Azure/azure-cli-extensions/commit/d095ff237a5488e5b4229a61396bb16b6db6879e

Docs Build status updates of commit 3dcb170:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix validation error when resizing window

Sometimes when resizing the window we may get the following validation error:
ERROR: VALIDATION - Message Id Number: -370888023 | Message Id Name: VUID-vkAcquireNextImageKHR-semaphore-01286
	Validation Error: [ VUID-vkAcquireNextImageKHR-semaphore-01286 ]
Object 0: handle = 0xdcc8fd0000000012, type = VK_OBJECT_TYPE_SEMAPHORE; | MessageID = 0xe9e4b2a9 | vkAcquireNextImageKHR: Semaphore must not be currently signaled or in a wait state. The Vulkan spec states: If semaphore is not VK_NULL_HANDLE it must be unsignaled (https://vulkan.lunarg.com/doc/view/<IP_ADDRESS>/linux/1.2-extensions/vkspec.html#VUID-vkAcquireNextImageKHR-semaphore-01286)

In VulkanContext::prepare_buffers the problem was that vkAcquireNextImageKHR returned VK_SUBOPTIMAL_KHR but it already signaled the semaphore (because it is possible to continue normally with a VK_SUBOPTIMAL_KHR result).
Then we recreate the swapchain and reuse the
w->image_acquired_semaphores[frame_index] which is in an inconsistent state.
Fixed by recreating the semamphores along the swapchain.
Fix #80570

I'm not sure how to judge this as I've not dived deep enough into this part of the Vulkan API so I'm not sure if there is a better way to do this, but I'm happy for this to be merged.
I would add a remark in the code why we're recreating the semaphores because someone looking at this code later on isn't going to get the why and might break this again.

Good point.
Done. I've updated the code with a comment explaining why we destroy the semaphore.

Looks good! Could you squash the commits? We usually keep tightly related changes in the same commit, such as here adding context to the change in the first commit (and we don't use the Squash on Merge strategy for a number of reasons, we prefer the history to be final in the PR itself).

Looks good! Could you squash the commits?

Done.


Support discrete values in Parameters.

Should strongly consider merging attributes and parameters to minimize the impacts on provider developer and allow for more reuse.

Attributes and parameters definitely need merged into one. Validation for user entered values should be pulled out into a separate class.