Headers on different rows other than 1

Hey Warren -
I ran into a slight issue with the Import-XLSX Script. I have a business XLSX file that has a top row that has some images embedded and custom formatting/branding for my company that is customer facing. My headers actually start on row 2. So I went in and modified the Import-XLSX function to support changing the RowStart and HeaderStart with a couple parameters. Here is the quick and dirty code I modified to do this..
function Import-XLSX {
    <#
    .SYNOPSIS
        Import data from Excel

    .DESCRIPTION
        Import data from Excel

    .PARAMETER Path
        Path to an xlsx file to import

    .PARAMETER Sheet
        Index or name of Worksheet to import

    .PARAMETER Header
        Replacement headers.  Must match order and count of your data's properties.

    .PARAMETER FirstRowIsData
        Indicates that the first row is data, not headers.  Must be used with -Header.

    .PARAMETER Text
        Extract cell text, rather than value.

        For example, if you have a cell with value 5:
            If the Number Format is '0', the text would be 5
            If the Number Format is 0.00, the text would be 5.00

    .EXAMPLE
        Import-XLSX -Path "C:\Excel.xlsx"

        #Import data from C:\Excel.xlsx

    .EXAMPLE
        Import-XLSX -Path "C:\Excel.xlsx" -Header One, Two, Five

        # Import data from C:\Excel.xlsx
        # Replace headers with One, Two, Five

    .EXAMPLE
        Import-XLSX -Path "C:\Excel.xlsx" -Header One, Two, Five -FirstRowIsData -Sheet 2

        # Import data from C:\Excel.xlsx
        # Assume first row is data
        # Use headers One, Two, Five
        # Pull from sheet 2 (sheet 1 is default)

    .NOTES
        Thanks to Doug Finke for his example:
            https://github.com/dfinke/ImportExcel/blob/master/ImportExcel.psm1

        Thanks to Philip Thompson for an expansive set of examples on working with EPPlus in PowerShell:
            https://excelpslib.codeplex.com/

    .LINK
        https://github.com/RamblingCookieMonster/PSExcel

    .FUNCTIONALITY
        Excel
    #>
    [cmdletbinding()]
    param(
        [parameter( Mandatory=$true,
                    ValueFromPipeline=$true,
                    ValueFromPipelineByPropertyName=$true)]
        [validatescript({Test-Path $_})]
        [string[]]$Path,

        $Sheet = 1,

        [string[]]$Header,

        [switch]$FirstRowIsData,

        [switch]$Text,

        [int]$RowStart,

        [int]$HeaderStart

    )
    Process
    {
        foreach($file in $path)
        {
            #Resolve relative paths... Thanks Oisin! http://stackoverflow.com/a/3040982/3067642
            $file = $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath($file)

            write-verbose "target excel file $($file)"

            Try
            {
                $xl = New-Object OfficeOpenXml.ExcelPackage $file
                $workbook  = $xl.Workbook
            }
            Catch
            {
                Write-Error "Failed to open '$file':`n$_"
                continue
            }

            Try
            {
                if( @($workbook.Worksheets).count -eq 0)
                {
                    Throw "No worksheets found"
                }
                else
                {
                    $worksheet = $workbook.Worksheets[$Sheet]
                    $dimension = $worksheet.Dimension

                    $Rows = $dimension.Rows
                    $Columns = $dimension.Columns
                }

            }
            Catch
            {
                Write-Error "Failed to gather Worksheet '$Sheet' data for file '$file':`n$_"
                continue
            }
            if ($RowStart) {
                $RowStart = $RowStart
            }
            else {
                $RowStart = 2
            }
            if($Header -and $Header.count -gt 0)
            {
                if($Header.count -ne $Columns)
                {
                    Write-Error "Found '$columns' columns, provided $($header.count) headers.  You must provide a header for every column."
                }
                if($FirstRowIsData)
                {
                    $RowStart = 1
                }
            }
            else
            {
                if ($HeaderStart) {
                    $Header = @( foreach ($Column in $HeaderStart..$Columns)
                    {
                        if($Text)
                        {
                            $worksheet.Cells.Item($HeaderStart,$Column).Text
                        }
                        else
                        {
                            $worksheet.Cells.Item($HeaderStart,$Column).Value
                        }
                    } )
                }
                else
                {
                    $Header = @( foreach ($Column in 1..$Columns)
                    {
                        if($Text)
                        {
                            $worksheet.Cells.Item(1,$Column).Text
                        }
                        else
                        {
                            $worksheet.Cells.Item(1,$Column).Value
                        }
                    } )
                    }
            }

            [string[]]$SelectedHeaders = @( $Header | select -Unique )

            Write-Verbose "Found $(($RowStart..$Rows).count) rows, $Columns columns, with headers:`n$($Headers | Out-String)"

            foreach ($Row in $RowStart..$Rows)
            {
                $RowData = @{}

                if ($HeaderStart) {
                    $HeaderStartColumns = 0..($Columns - $HeaderStart)
                }
                else
                {
                    $HeaderStartColumns = 0..($Columns - 1)
                }

                foreach ($Column in $HeaderStartColumns)
                {
                    $Name  = $Header[$Column]
                    if($Text)
                    {
                        $Value = $worksheet.Cells.Item($Row, ($Column+1)).Text
                    }
                    else
                    {
                        $Value = $worksheet.Cells.Item($Row, ($Column+1)).Value
                    }

                    Write-Debug "Row: $Row, Column: $Column, Name: $Name, Value = $Value"

                    #Handle dates, they're too common to overlook... Could use help, not sure if this is the best regex to use?
                    $Format = $worksheet.Cells.Item($Row, ($Column+1)).style.numberformat.format
                    if($Format -match '\w{1,4}/\w{1,2}/\w{1,4}( \w{1,2}:\w{1,2})?')
                    {
                        Try
                        {
                            $Value = [datetime]::FromOADate($Value)
                        }
                        Catch
                        {
                            Write-Verbose "Error converting '$Value' to datetime"
                        }
                    }

                    if($RowData.ContainsKey($Name) )
                    {
                        Write-Warning "Duplicate header for '$Name' found, with value '$Value', in row $Row"
                    }
                    else
                    {
                        $RowData.Add($Name, $Value)
                    }
                }
                New-Object -TypeName PSObject -Property $RowData | Select -Property $SelectedHeaders
            }

            $xl.Dispose()
            $xl = $null
        }
    }
}

Cool. Thank you

Hi! Great idea, have run into a few of these. Took a quick stab at this in 62c267a with a few alterations - added 'RowStart' and 'ColumnStart' parameters. If you have a moment, let me know if it works for you!

Going to assume 62c267a took care of this - thanks again!


Bug fix: Display flex on grid-container

Without display: flex; on .grid-containers, this scenario fails.

Item 1

Item 2

Item 3

The container is just meant to center content inside a grid-content element. It shouldn't be a flex container, because it's just supposed to contain plain-old flow content. We might need to update the documentation to make this more clear, though.
An example of its use would be:

So, assuming I am wanting the proposed scenario to have a max-width and center up, I would have to do something like this?

Item 1

Item 2

Item 3

Yep, that should work. If not, I can try to make a more complex example.

Understood. The circumstance I was in looked something like this and I just thought it looked redundant at first glance as I assumed grid-container was just a grid-block with a max-width:

Item

Item

Item

Yeah, that's a really tricky thing. We want mixins for all of our components so they'll be more flexible, but at some point you still need to mimic our markup structure to an extent for it to work. This grid in particular is that way because of how flexbox works, and because it's new.


[macOS Monterey] Cannot find libpq-fe.h – how to disable PostgreSQL dependency

After installing all brew dependencies (brew install autoconf automake bison freetype gd gettext icu4c krb5 libedit libiconv libjpeg libpng libxml2 libzip pkg-config re2c zlib), I'm stuck at PostgreSQL check:
checking for PDO_DBLIB support via FreeTDS... no
checking for Firebird support for PDO... no
checking for MySQL support for PDO... yes
checking for MySQL UNIX socket location... no
checking for PDO includes... checking for PDO includes... /var/folders/x5/0v2t1jt94xx3k5n6v764v8qw0000gn/T/php-src-php-7.2.34/ext
checking Oracle OCI support for PDO... no
checking for ODBC v3 support for PDO... no
checking for PostgreSQL support for PDO... yes
checking for pg_config... not found
configure: error: Cannot find libpq-fe.h. Please specify correct PostgreSQL installation path

Is there a way to disable psql support by default?
I'm on macOS Monterey, with no system PHP installed.

You can use this command to disable it:
PHP_WITHOUT_PDO_PGSQL=yes asdf install pup <version>


Proptypes error with react 15.5.X and up

 Warning: Accessing PropTypes via the main React package is deprecated, and will be removed in  React v16.0. Use the latest available v15.* prop-types package from npm instead. For info on usage, compatibility, migration and more, see https://fb.me/prop-types-docs

Any updates on this?

Problem of react-i18nify package itself, solved by 1.9.2 release.
I guess this can be closed?


Tweaked version ranges

This fixes incorrect fabric version range

I've been testing with 1.21.1 and it had no issues running btw


Fix wrong config for Webpack

Config seems to be following Babel config way but it's not working as expected and Webpack es complaining.
We need to use an object in order to define the implementation.

LGTM


Cannot create a menu using your docs

I cannot for the life of me create a button, or a menu at all from the documents.
		r := b.NewMarkup()
		r.Text("idk")

This doesn't work
Neither do your menu docs

read doc:
https://pkg.go.dev/gopkg.in/tucnak/telebot.v3#section-readme:~:text=m%2C "new caption")-,Keyboards,-Telebot%20supports%20both


rename _wurst/assets/Abilities.wurst

https://github.com/wurstscript/WurstStdlib2/blob/9559ad20266a0417ab9eda1db5a989f4aac70939/wurst/_wurst/assets/Abilities.wurst#L4
"Abilities" is a very generic naming, maybe it could be called something like "SpellModels"

Abilities is in line with all the other packages, which would also have to be changed.

Yeah I know, just dues to conventions, I thougth it should had been given some other naming.


Update moe doc

@mrwyattii -- please review this and merge as time permits. Just updating our moe tutorials to point to the main branch as we merged moe support in megatron-deepspeed.

@jeffra  -- needs one more approval to be merged.


⚠ Fetchers has degraded performance

In 07e537b, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 339 ms

Resolved: Fetchers performance has improved in d77872c.


Clean up GitHub Projects boards

https://github.com/pi-top/pi-top-Python-SDK/projects
Move to issues where appropriate, otherwise close. Either make use of it or close it.

Migrate to a pinned Issue with roadmap (e.g. https://github.com/EthWorks/useDApp/issues)


Opening an incorrect table path causes an empty directory to be made

Environment
Delta-rs version: 0.22.3
Binding:  python
Environment:

Cloud provider: :upside_down_face:
OS: Linux/amd64
Other:

Bug
What happened:
>>> from deltalake import DeltaTable
>>> dt = DeltaTable('i_just_made_this_up_lol')
[2024-12-16T14:15:26Z DEBUG deltalake_core::table::builder] creating table builder with file:///home/tyler/source/github/delta-io/delta-rs/python/i_just_made_this_up_lol
[2024-12-16T14:15:26Z DEBUG deltalake_core::table::builder] build_storage() with file:///home/tyler/source/github/delta-io/delta-rs/python/i_just_made_this_up_lol
[2024-12-16T14:15:26Z DEBUG deltalake_core::table::builder] Loading a logstore based off the location: Url { scheme: "file", cannot_be_a_base: false, username: "", password: None, host: None, port: None, path: "/home/tyler/source/github/delta-io/delta-rs/python/i_just_made_this_up_lol", query: None, fragment: None }
[2024-12-16T14:15:26Z DEBUG deltalake_core::logstore] Found a storage provider for file:/// (file:///home/tyler/source/github/delta-io/delta-rs/python/i_just_made_this_up_lol)
[2024-12-16T14:15:26Z DEBUG deltalake_core::logstore] Found a logstore provider for file:///
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/tyler/source/github/delta-io/delta-rs/python/deltalake/table.py", line 415, in __init__
    self._table = RawDeltaTable(
_internal.TableNotFoundError: no log files
>>>
(venv) plantain% ls i_just_made_this_up_lol
(venv) plantain%

What you expected to happen:
No directory should be made :smile:
How to reproduce it:
More details:

Encountered this issue while using the deltalake Python package.
After a quick lookup into the Rust implementation of RawDeltaTable, it seems this is a deliberate design in DeltaTableBuilder::from_uri calling ensure_table_uri?
https://github.com/delta-io/delta-rs/blob/72311ed2fc0c459dd3d5c20358b04efe0ad8d001/crates/core/src/table/builder.rs#L384

/// The location could be:
///  * A valid URL, which will be parsed and returned
///  * A path to a directory, which will be created and then converted to a URL.
///
/// If it is a local path, it will be created if it doesn't exist.

Why is there this requirement of creating an empty directory if the URI is local?
From a user perspective, directory creation should only be performed when actually creating a Delta Table (DeltaTable.create), not trying to open an existing one.


Update auth.md

Changes

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[ ] Includes docs (if user facing)
[ ] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
Describe any user facing changes here, or delete this block.

Examples of user facing changes:
- API changes
- Bug fixes
- Any changes in behavior

Hey @wutongjie23hao, thanks for the PR… but it's not needed. serviceAccount changed to serviceAccountName here : https://github.com/tektoncd/pipeline/pull/1291 ; to be coherent with the rest of kubernetes.
The docs folder on master reflects the current master code and features. If you want to read the docs of the current release, it's on its own branch as listed here — for 0.7.0, here
/close


error while compiling

The last make produces error. How can I compile?
$ make
[ 45%] Built target picoquic-core
[ 47%] Linking C executable picoquic_ct
/usr/bin/ld: cannot find -lprofiler
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/picoquic_ct.dir/build.make:448: picoquic_ct] Error 1
make[1]: *** [CMakeFiles/Makefile2:114: CMakeFiles/picoquic_ct.dir/all] Error 2
make: *** [Makefile:84: all] Error 2

Saw issue #10 now.
I had installed google-perftools as per https://pquic.org/ but error exists despite that. Installing libgoogle-perftools-dev solved it.


Extend benchmarking module

This PR augments the benchmarking functionality of emutkit by:

passing a complete loop state, rather than just X and Y, to the outer loop during benchmarking
enable the outer loop to have multiple event handler in order to allow to measure multiple metrics
adds a new metric that computes the cumulative cost after each function evaluation
update the unit test to the new interface

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #202 into master will decrease coverage by 0.27%.
The diff coverage is 47.61%.

@@            Coverage Diff            @@
##           master    #202      +/-   ##
=========================================
- Coverage   89.47%   89.2%   -0.28%
=========================================
  Files         105     105
  Lines        2861    2871      +10
  Branches      292     295       +3
=========================================
+ Hits         2560    2561       +1
- Misses        249     258       +9
  Partials       52      52

Impacted Files
Coverage Δ

emukit/core/loop/loop_state.py
100% <ø> (ø)
:arrow_up:

...ukit/benchmarking/loop_benchmarking/benchmarker.py
0% <0%> (ø)
:arrow_up:

emukit/core/loop/outer_loop.py
91.66% <100%> (ø)
:arrow_up:

emukit/benchmarking/loop_benchmarking/metrics.py
82.5% <54.54%> (-11.25%)
:arrow_down:

emukit/core/event_handler.py
50% <0%> (-33.34%)
:arrow_down:

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 eb95c10...9405f0b. Read the comment docs.

Codecov Report

Merging #202 into master will decrease coverage by 0.27%.
The diff coverage is 47.61%.

@@            Coverage Diff            @@
##           master    #202      +/-   ##
=========================================
- Coverage   89.47%   89.2%   -0.28%
=========================================
  Files         105     105
  Lines        2861    2871      +10
  Branches      292     295       +3
=========================================
+ Hits         2560    2561       +1
- Misses        249     258       +9
  Partials       52      52

Impacted Files
Coverage Δ

emukit/core/loop/loop_state.py
100% <ø> (ø)
:arrow_up:

...ukit/benchmarking/loop_benchmarking/benchmarker.py
0% <0%> (ø)
:arrow_up:

emukit/core/loop/outer_loop.py
91.66% <100%> (ø)
:arrow_up:

emukit/benchmarking/loop_benchmarking/metrics.py
82.5% <54.54%> (-11.25%)
:arrow_down:

emukit/core/event_handler.py
50% <0%> (-33.34%)
:arrow_down:

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 eb95c10...9405f0b. Read the comment docs.

This looks good apart from the EventHandler stuff and a unit test for the new CumulativeCost metric


Automated cherry pick of #9910

Cherry pick of #9910 on release-6.5.
/cc  @AshishDhama
NONE

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

Trying to auto merge this PR.

Pull Request successfully merged
SHA: e9cde1b7be545c414125ffc43cf080fb114dbdf9


Minor updates

@jovanpop-msft : Thanks for your contribution! The author, @allenwux, has been notified to review your proposed change.

#ignore I have sent PR in another private branch.


add engagement model to app file

supporting the proposal in https://gitlab.cee.redhat.com/app-sre/contract/-/merge_requests/109
adding engagementModel to app files will help determine the engagement level with each different service.
this attribute can be added to either child apps or parent apps, illustrating the engagement level per component if needed.
starting out with this attribute as optional so we don't create a breaking change that requires every instance to add this attribute to all app files. we can make it mandatory down the line, or not. perhaps the default is Standard, unless otherwise specified.

Do we need this if the status of the onboarding is different? The related contract MR mentions a different onboarding status.


relax dependency version requirements

@jamescallmebrent Good! 🙆

@mokumoku Thanks! Mind publishing release 0.2.2 to cococapods?

@jamescallmebrent oops! 😱 I forgot it!
Now I pushed to CocoaPods. Please pod update 🙏
Thank you!


Update jupyter_embed.py

On Windows, I needed to change the encoding to 'utf-8', since that isn't the default on Windows (and is the default of Linux and mac).
After the simple update I was able to use the embed function in my notebooks.

sorry for the delay here! Will do a release soon.


[WIP] hotfix for v2.2.3 (production branch) - reset migrations and add mysql to travis

work in progress, don't merge

[ ] reset migrations -

[x] delete all existing migration files
[x] recreate new migration files
[ ] look at production DB and make sure the actual state corresponds to the new migrations
[ ] write deployment procedure, basically need to fake the migrations / update the state manually in DB

[ ] add travis to mysql
[ ] update documentation - add clear instructions about how to use migrations

this PR will be pointless if afterwards the same mistakes will be made again

@ArtyomKa feel free to continue working on it, I'll get back to it only tomorrow / later this week

@OriHoch IIUC, resetting the migrations will "force" them on the DB, right? what will happen with the existing data?

@ArtyomKa - my tought was to manually sync the DB to the migrations
knex uses a migrations table in DB - you can manually edit it to "fake" as if all migrations ran, then you manually make sure DB is in sync with migrations (e.g. that running migrations produces the same DB state as you have in production)
after that you need to start working with migrations properly - e.g. make new migrations each time, not edit existing migrations, test them properly etc..

@OriHoch
Got the migrations and the tests passing. Do you want me to update this PR? How is it done? i just push to your branch or open a PR?

you should be able to push to my branch
or - push to a new branch and open a new pr, then close this pr

OK, I pushed to the branch on your fork. I removed all obsolete volunteers tables so there are less FK to worry about.
I think most of the problems that relations between tables where defined in the migrations and there are some type mismatches. like: signed integer references unsigned integer.
Sqllite does not realy support FK anyways, so it does not really care, but MySQL won't have it.. and migrations fail.
Also I'm not sure that defining many operations on different table is such a good idea, cause there is no way to control how Promise.all([]) executes them.

@OriHoch Now, that I have a better understanding about what your plan is, I'm not sure that my commit is good, as i removed some of the volunteers tables. So if they are present in the DB this commit should be reverted.

Please take a look at https://github.com/Midburn/Spark/pull/338
I have done some work on the DB connectivity and made a lot of fixes for the migrations to allow them run on MySql. This might save you some time.

closing, see the updated description for details


后端运行出错

后端运行后提示表不存在
2016-08-18 10:34:51 INFO     auto_block.py:62 Read hosts.deny from line 87
2016-08-18 10:35:54 INFO     auto_block.py:62 Read hosts.deny from line 89
2016-08-18 10:36:00 ERROR    db_transfer.py:468 Traceback (most recent call last):
 File "/root/shadowsocks/db_transfer.py", line 462, in thread_db
   rows = db_instance.pull_db_all_user()
 File "/root/shadowsocks/db_transfer.py", line 264, in pull_db_all_user
   cur.execute("SELECT " + ','.join(keys_detect) + " FROM detect_list wheretype = 1")
  File "/usr/lib/python2.6/site-packages/cymysql/cursors.py", line 133, in execute
    self.errorhandler(exc, value)
  File "/usr/lib/python2.6/site-packages/cymysql/cursors.py", line 35, in errorhandler
    self.connection.errorhandler(self, errorclass, errorvalue)
  File "/usr/lib/python2.6/site-packages/cymysql/connections.py", line 118, in errorhandler
    raise errorvalue
ProgrammingError: (1146, u"Table 'sspanel.detect_list' doesn't exist")

解决了么，我也出现这个问题了

请升级前端

升级过后出现这样  是因为添加节点出现问题么
2016-08-22 10:48:10 INFO     db_transfer.py:411 db start server at port [1026] pass [gr3jZO] protocol [origin] obfs [plain]
2016-08-22 10:48:10 INFO     server_pool.py:137 starting server at [::]:1026
2016-08-22 10:48:10 INFO     server_pool.py:163 starting server at <IP_ADDRESS>:1026
2016-08-22 10:49:10 INFO     auto_block.py:62 Read hosts.deny from line 33

已解决，这属于正常日志，防火墙的锅


Is it possible to produce a Groovy DSL script from an OpenAPI specification generated in JSON format

Hi,
Is there a method, plugin, or dependency that can be used to transform an OpenAPI Specification produced in JSON format to a Groovy contract?

Thank you for your response, @marcingrzejszczak. Although it's feasible to utilize the OpenAPI specification as the contract directly, we need the contracts to be in Groovy format. Hence, we are striving to generate Groovy contracts automatically. I'll attempt to produce the contracts in Groovy by utilizing the dependency and method mentioned above.

Check this class https://github.com/spring-cloud/spring-cloud-contract/blob/6fd4280d12a5186eb2a009eb018b8c0943baf85d/spring-cloud-contract-verifier/src/main/java/org/springframework/cloud/contract/verifier/util/ToFileContractsTransformer.java and this one https://github.com/spring-cloud/spring-cloud-contract/blob/main/spring-cloud-contract-tools/spring-cloud-contract-converters/src/main/java/org/springframework/cloud/contract/verifier/wiremock/WireMockToDslConverter.java . The latter can convert a Wiremock Json to a Groovy contract. You could write sth similar.

Hi @marcingrzejszczak ,Could you provide more details on the process of generating a Groovy contract from an OpenAPI generated JSON specification file? I am feeling uncertain about the steps I need to take, and I would appreciate it if you could elaborate on them.


Feature/572

Was like this.

Now it like this.

Create new component 'ImageUploadPatternContainer' (../containers/image-upload-container/image-upload-patternContainer).
Add it to 'pattern-form.js'

move to our Horondi board

remove empty file

move text 'Фото' and 'Фото для конструктора' over image

remove empty file

It`s not empty.


[Snyk] Security upgrade setuptools from 40.5.0 to 70.0.0

Snyk has created this PR to fix 1 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements.txt

⚠ Warning
```
selenium-wire 5.1.0 requires blinker, which is not installed.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiIyNGYyZjkxNS01MDI4LTRjNDQtYmQxYS1jZWI1OGE5ZjhiMzciLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjI0ZjJmOTE1LTUwMjgtNGM0NC1iZDFhLWNlYjU4YTlmOGIzNyJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/klept0/project/220b1b64-2bc2-422b-9afe-f0e51b42dbfb?utm_source=github&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/klept0/project/220b1b64-2bc2-422b-9afe-f0e51b42dbfb?utm_source=github&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Improper Control of Generation of Code ('Code Injection')](https://learn.snyk.io/lesson/malicious-code-injection/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"setuptools","from":"40.5.0","to":"70.0.0"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-SETUPTOOLS-7448482","priority_score":768,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"7.5","score":375},{"type":"scoreVersion","label":"v1","score":1}],"severity":"high","title":"Improper Control of Generation of Code ('Code Injection')"}],"prId":"24f2f915-5028-4c44-bd1a-ceb58a9f8b37","prPublicId":"24f2f915-5028-4c44-bd1a-ceb58a9f8b37","packageManager":"pip","priorityScoreList":[768],"projectPublicId":"220b1b64-2bc2-422b-9afe-f0e51b42dbfb","projectUrl":"https://app.snyk.io/org/klept0/project/220b1b64-2bc2-422b-9afe-f0e51b42dbfb?utm_source=github&utm_medium=referral&page=fix-pr","prType":"backlog","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["pkg-based-remediation","updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-SETUPTOOLS-7448482"],"patch":[],"isBreakingChange":false,"remediationStrategy":"dependency"}'

wtf is it looking at to not realise it has been >=70.0.0 for over 2 weeks now

it wants us to upgrade blinker - I know it was hardcoded to that to prevent an issue. @cal4 are we good to remove the version and let it upgrade after the updates?


Module not found: Error: Can't resolve 'fs' in ...

Encountered below error while trying to deploy it

did you ever resolve?


Can you give me an option to prevent 3Dtiles from retrieving Sequence?

Can you give me an option to prevent 3Dtiles from retrieving Sequence?
Can you add a Bool value to the blueprint to check?

On solution could be to add an explicit reference to the level sequence actor that we intend on using, similar to the references to credit system, camera manager, etc...

I see


improve the online parser layout

in xs mode the grid is not full width
equal width colums would look better
add a little description of the project
add labels to each box (input and output)
consider adding the "readme.md" as an "about" section (may require routing)

nice to have in the future:

build pokemon cards with the pokemon icons
show Pokemon type coverage charts for: resistances and stab types

I think the "nice to have" should go in a separated project. We can build cards with the icons if the project doesn't have to host the icons individually. Using a spritesheet could help.


Fixes MicrosoftDocs/azure-docs#8861

It should be StandardLRS or PremiumLRS, there is no underscore.

@MicahMcKittrick-MSFT : Thanks for your contribution! The author, @ramankum, has been notified to review your proposed change.


Add tool to extract glb from i3dm.

Part of #34

Looks good.  Can you add the instructions to tools/README.md and add tests?

Bump @bagnell.

@pjcozzi @lilleyse This is ready for another look.

Works great, thanks!


ci(build-test-analyze): addition of build, test and analyze script

This commit bears the addition of CI scripts that will be used to trigger build, test and analyze automatically for the core library whenever any new commit is pushed.
closes #3

Closing this PR as this is outdated now and scripts are already configured in main branch.


Composing character disappear when click other block or place

Do you want to request a feature or report a bug?
a bug
What is the current behavior?
When i typing Korean and it's during composing mode.
if move cursor, my composing character is disappear...
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/stopachka/m6z0xn4r/.
it's happened on Facebook.com, Draft.js

What is the expected behavior?
move cursor and composing character is composite done(it means, not disappear)
Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
Mac Book pro 13": 10.12.5
Chrome: 59.0.3071.115(official build)
FireFox: fine
Safari: fine
Windows, sorry i can't tested...

I was able to reproduce this - thanks for reporting!
Looks like this does work as expected in other text areas on the web - just checked gmail and quip and they both do the right thing here.
Of the top of my head I'm not sure of the cause or when this was introduced. Seems serious though - I'll open a task internally for us to look into it.
@spicyj any ideas on this one? Just tagging you since you've looked at Korean input bugs before.

Hm. We need to force the composition to resolve, maybe in response to the onSelect event? Not sure what we're doing now without looking at the code.

This bug was kind of fixed after DOMObserver was introduced. You can take a look at them from the master.
However, it can also be cured by modifying DraftEditorCompositionHandler.js for the older version 0.9.1
https://gist.github.com/goodhyun/3539a16951f368ce756da205d37c85e5


Integration Test - Release - 2022/04/11 17:28:35

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

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

Docs Build status updates of commit 5495076:
:warning: Validation status: warnings

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md
:warning:Warning

Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md

Line 4, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 4, Column 8: [Warning: duplicate-titles - See documentation] Title 'VS Code Authoring Extension' is duplicated in: 'Demo/demo.md(4,8), Demo/v-caxian/AuthorNotification.md(4,8), Demo/v-caxian/auto-merge.md(4,8), Demo/v-caxian/auto-merger.md(4,8), IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md(4,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


perform some path sanitization over the generated names

The current scheme for the file generation uses the function label,
which uses ':' as a parameter separator.  However, this character is not
valid on all file systems.  This results in partial content generation
on Windows.  Add a sanitization step which replaces "invalid" characters
with _ instead.  This does not impact the rendering, only the
filenames and the generated URL references between the pages.  With
this, we can generate and link function documentation on Windows.

@compnerd Thanks for your patience on this. Once CI comes back ✅ , I'll merge this in.

@mattt - do I need to redo the patch or can you merge this (for the signing)?

@compnerd Sorry for the delay. This is now merged in.


Tapo C200, all move are anaivalaible

Description
Camera model : C200
Hi,
Don't know since when, but all move related function are unavailable ( worked very well for a long time )

Reproduction Steps
Cannot access to move command of the camera
Expected behavior
Have access to move command of the camera
If applicable, add error logs.
No response
Device Firmware
1.3.9 Build 231019 Rel.37378n(4555)
Integration Version
updated
Using stream component
Yes
Does camera work via official integrations?
Yes
Camera has all attributes filled out in developer tools
Yes
HASS Environment
HA OS raspberry
Search for similar issues
Yes
Additional information
No response

Hi post logs and proper version in template.

Oh so strange !
For giving you debug log, tested more...   and :

movement ( like left / right, up and down, despite unavailable, are working
movement for goind to position, ARE NOT
nothing about my C200 camera in the log ( searched for name, firmware version and type name )  ( log are full of message of my another camera, a C310, a static one )

So not sure what can i post you :(

Please disable your other camera and then enable debug logs for this integration and post that when you see this issue.
Integration version is still incorrect.
I will reopen this once the above is sent.

Intégration 5.4.17
Here is the log with only one camera :
debug.log

Thank you could you please post screenshots of all the entities? I  see “_2” in the entities. Maybe these are duplicate and there are other for movement ?

Sure, here are the screen shot, as i said, all was working before, surely a recent update ( HA or extension ? ) broke this...
( first capture is reseaching 'mov' )

Thank you.
As the next steps:

Please try to remove camera, restart, and add it again
If you are still experiencing the issue, please post full debug logs from the start of HA (the last ones you posted are just a small section while it was running).

ok will do this ! thanks for helping !

So, deleted all, restard HA, reconfigure the camera and strangely all entity are ok this time ( name without _2 of course )
All is ok, so no better information in log to understand what was the problem.
Thanks for your time btw.


Update to latest dependencies

Minor changes due to:

reverting the use of libsodium for KES;
removal of SafeBeforeEpoch consensus feature
simplification of time lock script representation

bors merge


raidboss: fix cn / jp translate error in r1n

Current CN translate opposite the cardinals and intercardinals directions
Also found that Leaping One-Two Paw gimmick might have ambiguity?
https://github.com/OverlayPlugin/cactbot/blob/7517edf518f52d5857e930ca7c036969fc5638f3/ui/raidboss/data/07-dt/raid/r1n.ts#L284-L298
I thought the word West => East at marker could be parsed as Get to current Boss's west, then get to the marker's east or Get to the marker's west, then its east. At first time, I and @Echoring understood the word as the former one, so we translated the at marker part only on the East word (Chinese and Japanese require the at marker locative adverbial before the modified noun, which is different from English / French)
So pinging @JLGarber or @valarnin to check if we understand it right now, and if the English / French / German word should be adjusted.

Yes, the intent of the English wording there was that all actions for that mechanic are relative to the marker. Your example of Get to the marker's west, then its east is correct. Sorry that wasn't clearer!

Yes, the intent of the English wording there was that all actions for that mechanic are relative to the marker. Your example of Get to the marker's west, then its east is correct. Sorry that wasn't clearer!

Thank you for the rapid response, I am going to merge this.


HTML classes support

I'm submitting a ...  (check one with "x")
[ ] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[x] feature request

Current behavior
When I hade some HTML tags the tags are added in she string. For example if my translation string is "This is bold" the output in my HTML page would be exactly the same and not "this is bold".
Expected/desired behavior
I wondered if it was possible to add some HTML classes support. For example by adding a param in the translation function.
Thanks,

You can already do that with innerHTML:

with:
{
  "SOME_KEY": "This is bold"
}

Thanks a lot and sorry !


publish doc with travis

Publish Ascii-Doc files to github pages

This change is

Changes Unknown when pulling c82f71a37ad0c4deeab132669b8d1c488799d2af on publish-doc into ** on master**.

Reviewed 1 of 1 files at r1.
Review status: 1 of 2 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Changes Unknown when pulling b8c2638f22b044b758c4bd9ea70b52c7727d4b08 on publish-doc into ** on master**.


upgrade azurerm to latest 3.x version

Reference Issues or PRs
Fixes https://github.com/nebari-dev/nebari/issues/2806

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

How to test this PR?

Any other comments?

This looks good but I wanted to confirm everything works as expected by running our Azure tests from this branch: https://github.com/nebari-dev/nebari/actions/runs/11628815929
Unfortunately, they failed because of a Vault issue, completely unrelated to this PR. Once we have fixed that, I'll re-trigger them and see if the deployment works in Azure.

Thanks Marcelo, I haven't tested it yet.

@Adam-D-Lewis I re-triggered the Azure deployment workflow now that Vault is working again and found another issue when it tries logging-in: https://github.com/nebari-dev/nebari/actions/runs/11628815929/job/32469625611#step:6:33
Running Azure CLI Login.
/usr/bin/az cloud set -n azurecloud
Done setting cloud: "azurecloud"
Federated token details:
 issuer - https://token.actions.githubusercontent.com/
 subject claim - repo:nebari-dev/nebari:ref:refs/heads/upgrade_azurerm_provider
Attempting Azure CLI login by using OIDC...
Error: AADSTS700213: No matching federated identity record found for presented assertion subject 'repo:nebari-dev/nebari:ref:refs/heads/upgrade_azurerm_provider'. Please note that the matching is done using a case-sensitive comparison. Check your federated identity credential Subject, Audience and Issuer against the presented assertion. https://learn.microsoft.com/entra/workload-id/workload-identity-federation Trace ID: ea8ccd7f-2663-42b5-9130-450be9707200 Correlation ID: aaf97b77-ef83-4a2e-b225-8ebe0106d6f7 Timestamp: 2024-11-04 10:49:52Z

Error: Interactive authentication is needed. Please run:
az login

Error: Login failed with Error: The process '/usr/bin/az' failed with exit code 1. Double check if the 'auth-type' is correct. Refer to https://github.com/Azure/login#readme for more information.

Do you recognize this error?

I'll take a look

Sorry I put this as a draft since it's not ready for merge.  I'm sure there'll need to be some adjustments made for this to deploy correctly.


Add repetition counting extractor for ComplexTextStims

To be able to easily model the effects of word repetition in NeuroScout, we can add a RepetitionCountExtractor class that takes a ComplexTextStim as input, and outputs the number of times each word has been previously seen before (with optional log transformation of values).

Closed by #364


Dubbo new website design

The Dubbo new logo has been accepted by the community, we need to create the design based on the new logo.

Hi guys，I have designed the Dubbo Portal.Dubbo's product attributes is primarily connected, so the design uses lines as elements to achieve brand penetration.Let's take a look at the design.

Nice design, clean and simple

like it.

looks good to me.

Very professional ！If the homepage offers scrolling panels, it may be even better to have some wonderful videos in the community.

Clean and simple, nice design, the whole looks very comfortable

I like this design

Cool. +1

Awesome design
Jerrick Zhu<EMAIL_ADDRESS>于2018年6月26日周二 下午4:19写道：

Cool. +1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-dubbo-website/issues/12#issuecomment-400222447,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAye6OOPJToe1JQx6jdppjMqhbuav7vBks5uAe6bgaJpZM4Uzk3H
.

颜色太亮，看着眼睛好累

nice design.
maybe you can show more information in the frist screen of  each page.

辣眼睛，nice

Haha, now I display the original size of the design for easy viewing. Welcome everyone to make valuable suggestions.

@YujieQiyue  Really nice work! This will definitely help the developers to implement the website!

颜色确实有些亮，有些文字与背景颜色区分度不显著，看着眼睛有些累。建议照顾弱色或色盲群体，声明本人眼睛是正常的。😁

+1, much much better than the current.

Thanks everyone for the comments.
This design has been implemented by https://github.com/apache/incubator-dubbo-website/pull/13

@EdwardLee03 Thank you for your advice! The color here uses the logo brand color, and it seems that it should adjust its brightness according to the specific scene.


Prevent router from some paths

How to prevent router to intercept some paths?
Example:
My landing page use Vaadin Router, but I also have a /documentation path that I don't want to be handled by the router, because the documentation is another static app.
With the current behavior, and using the fallback route (path: '(.*)'), when you click on a documentation link (Documentation), you see the fallback route instead of the documentation static app.

A community member has suggested a solution:

to exclude individual links, add an vaadin-router-ignore attribute on the  tagExternal Page

This requires a small change in the CLICK navigation trigger.
to exclude a group of links at once, create a route with a special action:routes = [
  // this may be the first route in the config
  {
    path: '/pattern/to/exclude/(.*)',
    action: (ctx, commands) => {
      window.location.pathname = ctx.pathname;
    }
  },
  ...
];

ToDo:

modify the CLICK navigation trigger to support the vaadin-router-ignore attribute: https://github.com/vaadin/vaadin-router/blob/940df38dfc96e6d0966e88d8f4627a26ec037be9/src/triggers/click.js#L67
add a test to verify the new feature in the CLICK trigger: https://github.com/vaadin/vaadin-router/blob/master/test/triggers/click.spec.html
add a demo that shows the feature. Add it to the end of the Getting Started demo page: https://github.com/vaadin/vaadin-router/blob/master/demo/vaadin-router-getting-started-demos.html
update the description of the CLICK navigation trigger in the Navigation Triggers demo page that tells about the vaadin-router-ignore attribute: https://github.com/vaadin/vaadin-router/blob/master/demo/vaadin-router-navigation-trigger-demos.html

Are we sure about the attribute name? I feel that's it is too long. How about just ignore?

Are we sure about the attribute name? I feel that's it is too long. How about just router-ignore?

Yep, vaadin-router-ignore sounds a bit too long. The reason why I do not like the short ignore option is that it's lacking context:
Admin

Should this link be ignored?
Some other options I have in mind are:

vaadin-ignore
router-ignore

Would any of these be better?

The reason I lean towards router-ignore is that first of all it has context and second we don't have vaadin in any other attributes. So It's more consistent with the other attributes.

Hi,
I have  2 html files -> index.html ( vaadin + lit ) and chat.html ( react ). the above doesn't seem to work as when I navigate manually from 1 app to the other -> I still get a " route not found " error.
Can anyone help with a bit of guidance how to ignore navigating from 1html file to another from being picked up by the router ?


Generated columns (PostgreSQL)

Problem
Postgres generated columns are currently not supported. That's not a big deal, since we can always declare them as regular columns in the schema.prisma. The only flaw is we need to declare them as optional, otherwise the client will always complain about them missing in the create input. This means the generated type field will always be "nullable", when it's actually not.
Suggested solution
Introduce a custom schema attribute ( ex @generated ) to mark a column as a postgres generated column. This way it will not be included in the create input type but it will also be not null in the model type.

this should marked as bug instead of feature request. declare generated columns as optional manually is cumbersome, user need remember to do that after every introspect

I think you can already use @default(dbgenerated()) here to specify that field is computed. We already support that. https://www.prisma.io/docs/reference/api-reference/prisma-schema-reference#dbgenerated
What we don't do here is automatically add this for computed fields when we introspect so that is something that we should do. cc @do4gr

@pantharshit00 unfortunately for generated columns, dbgenerated produces an invalid syntax, here's to reproduce:
in schema:
createdAtMonth Int?                @default(dbgenerated("GENERATED ALWAYS AS (date_part('month', "createdAt")) STORED")) @db.SmallInt
which produces in sql:
    "createdAtMonth" SMALLINT DEFAULT GENERATED ALWAYS AS (date_part('month', "createdAt")) STORED,
If we remove the DEFAULTpart, it should work

Possibly stupid idea @hyusetiawan: What happens if you add a 0  at the beginning?
If I am changing the generated SQL it will trigger a new migration every time I migrate and it breaks the SQL, unfortunately. So my current solution is to just have it as a regular optional column and then generate a new manual migration and write the SQL by hand.

@janpio oof nice try here's the error that it produced:
this is the SQL generated:
202     "createdAtMonth" SMALLINT DEFAULT 0 GENERATED ALWAYS AS (date_part('month', "createdAt")) STORED,
and the error is:
ERROR: both default and generation expression specified for column "createdAtMonth" of table "Renewal"
and
DbError { severity: "ERROR", parsed_severity: Some(Error), code: SqlState("42601"), message: "both default and generation expression specified for column "createdAtMonth" of table "Renewal"", detail: None, hint: None, position: Some(Original(5455)), where_: None, schema: None, table: None, column: None, datatype: None, constraint: None, file: Some("parse_utilcmd.c"), line: Some(825), routine: Some("transformColumnDefinition") }

It was worth a try.
Sorry, then using dbgenerated(...) will not work for generated columns. (When Harshit wrote that, the function was to be used without parameters - but that changed in the meantime).
Unfortunately I am not aware of any handy workaround for this.

@janpio it's worth a shot indeed :)
so the current workaround is to set the column to be nullable and then create another manual migration on top of it and hand code the SQL the GENERATED part

@janpio it's worth a shot indeed :)
so the current workaround is to set the column to be nullable and then create another manual migration on top of it and hand code the SQL the GENERATED part

I would appreciate an example of the migration workaround, if you may. Did you found if it's better to go with triggers, or simply fill the default value?

Quite note that those generated columns are a great way to store the tsvector lexems for full text search when another column is inserted or updated. For example:
ALTER TABLE se_details ADD COLUMN ts tsvector
    GENERATED ALWAYS AS (to_tsvector('english', event_narrative)) STORED;

I'm seeing a related issue which is that prisma inspects the database schema for a generated column I want it to not attempt to manage and writes into any new migrations ALTER TABLE <> ALTER COLUMN <generated_thing> DROP DEFAULT;.
😢
Of course, the generated default is kinda important here. 😄   It does seem to take an attempt to enumerate the generated default so far and so I believe I have a workaround for now.

Quite note that those generated columns are a great way to store the tsvector lexems for full text search when another column is inserted or updated. For example:
ALTER TABLE se_details ADD COLUMN ts tsvector
    GENERATED ALWAYS AS (to_tsvector('english', event_narrative)) STORED;

I would like to store the value of array of aggregation of another table for indexing to compute tf-idx for finding related articles.

I'm seeing a related issue which is that prisma inspects the database schema for a generated column I want it to not attempt to manage and writes into any new migrations ALTER TABLE <> ALTER COLUMN <generated_thing> DROP DEFAULT;

@ryanmt I've just run into this after upgrading to Prisma 4 - these unwanted (+invalid!) DROP DEFAULT migrations were not being generated previously, so now I've hit a blocker preventing me creating new migrations.
Are you using Prisma v4 also? Could you possibly share more about your workaround?
~~
In my schema.prisma (worked for the past months with Prisma v3):
...
<column> String? @db.VarChar(255) @default(dbgenerated())
...

A previous custom migration in my migrations history results in the following schema in Postgres (which has been working fine):
"<column>" character varying(255) GENERATED ALWAYS AS (
<SQL expression>) STORED

Upgrading to Prisma v4 now generates these new DROP DEFAULT migrations that fail with error:
Database error:
ERROR: column "<column>" of relation "<table>" is a generated column
HINT: Use ALTER TABLE ... ALTER COLUMN ... DROP EXPRESSION instead.

I think this should be a bug 🐛

We do not support generated columns and do not claim to do so, so for us, this is a feature request. Bugs are only when things are broken or do not work as expected and documented.

Update: issue created #14799

I'm seeing a related issue which is that prisma inspects the database schema for a generated column I want it to not attempt to manage and writes into any new migrations ALTER TABLE <> ALTER COLUMN <generated_thing> DROP DEFAULT;

@ryanmt I've just run into this after upgrading to Prisma 4 - these unwanted (+invalid!) DROP DEFAULT migrations were not being generated previously, so now I've hit a blocker preventing me creating new migrations.
Are you using Prisma v4 also? Could you possibly share more about your workaround?
~~
In my schema.prisma (worked for the past months with Prisma v3):
...
<column> String? @db.VarChar(255) @default(dbgenerated())
...

A previous custom migration in my migrations history results in the following schema in Postgres (which has been working fine):
"<column>" character varying(255) GENERATED ALWAYS AS (
<SQL expression>) STORED

Upgrading to Prisma v4 now generates these new DROP DEFAULT migrations that fail with error:
Database error:
ERROR: column "<column>" of relation "<table>" is a generated column
HINT: Use ALTER TABLE ... ALTER COLUMN ... DROP EXPRESSION instead.

Sorry I mean't this, but I see there is an issue for it. Please ignore my bug comment 😳

There you are absolutely correct. #14799 is fixed though, did that not work for you? If so, please open a new issue @homerjam so we can look into that.

In case anyone is interested in this feature find this issue, I just want to mention that there is a workaround for using GENERATED columns in Prisma, although they're not natively integrated in the Prisma schema yet. Here's an example repo that explains the workflow: https://github.com/nikolasburk/generated-columns

This is a very much needed feature

Heyyy, everyone that previously comment, subscribed or upvoted. Could you please share as many examples of generated columns SQL here as possible?
We want to figure out how much we want to or can understand and represent the "expression" of a generated column in Prisma Schema Language (and use that to for example validate the function before it is created in the database, by checking field/column names and so on). Otherwise we just treat them as magic SQL strings without understanding what they do, and "only" take care of migrating and introspecting them.
Thanks!

Some examples I found in other issues:

https://github.com/prisma/prisma/issues/6336#issuecomment-1210604043
ALTER TABLE se_details ADD COLUMN ts tsvector
    GENERATED ALWAYS AS (to_tsvector('english', event_narrative)) STORED;

https://github.com/prisma/prisma/issues/3001
CREATE TABLE `example` (
  `idZak` smallint(5) unsigned NOT NULL,
  `idObdobi` smallint(5) unsigned NOT NULL,
  `idSkup` smallint(5) unsigned DEFAULT NULL,
  `idSkupFilled` smallint(5) GENERATED ALWAYS AS (coalesce(`idSkup`,0)) STORED NOT NULL,
  PRIMARY KEY (`idZak`,`idObdobi`,`idSkupFilled`)
  /*CONSTRAINT `vylep_skupina` FOREIGN KEY (`idSkup`) REFERENCES `skupiny` (`idSkup`),CASCADE*/
);

https://github.com/prisma/prisma/issues/17313 - Supabase schema
CREATE TABLE auth.identities (
    ...
    email text GENERATED ALWAYS AS (lower((identity_data ->> 'email'::text))) STORED
);

CREATE TABLE auth.users (
    ...
    confirmed_at timestamp with time zone GENERATED ALWAYS AS (LEAST(email_confirmed_at, phone_confirmed_at)) STORED,
    ...
);

https://github.com/prisma/prisma/issues/17734#issuecomment-1454954128 - More Supabase schema
    "email" TEXT GENERATED ALWAYS AS (lower((identity_data->>'email'::text))) STORED,
...
    "confirmed_at" TIMESTAMPTZ(6) GENERATED ALWAYS AS (LEAST(email_confirmed_at, phone_confirmed_at)) STORED,

https://github.com/prisma/prisma/issues/4849
CREATE TABLE person (
    primary key (person_id),
    person_id      bigint,
    first_name     text not null,
    last_name      text not null,
    full_name      text NOT NULL GENERATED ALWAYS AS (first_name || ' ' || last_name)
)

https://github.com/prisma/prisma/issues/16913
ALTER TABLE `users` ADD COLUMN `updated_date` DATE GENERATED ALWAYS AS (DATE(updated_at)) STORED;

https://github.com/prisma/prisma/issues/12914#issuecomment-1411883969
-- AlterTable
ALTER TABLE "channels" ADD COLUMN     "textsearchable_index_col" tsvector
  GENERATED ALWAYS AS (to_tsvector('english', coalesce(domain, '') || ' ' || coalesce(title, ''))) STORED;

-- CreateIndex
CREATE INDEX "channels_textsearchable_index_col_idx" ON "channels" USING GIN ("textsearchable_index_col");

https://github.com/prisma/prisma/issues/5089#issuecomment-1479901365 Workaround for missing range type
alter table "Example"
add column timestamp_range tsrange generated always as (tsrange(start_timestamp, end_timestamp)) stored;

https://github.com/prisma/prisma/issues/16249
domain       text GENERATED ALWAYS AS ( split_part(url, '/', 3) ) STORED,

https://github.com/prisma/prisma/issues/3394#issuecomment-1233094477
CREATE TABLE IF NOT EXISTS User (
  id BIGINT NOT NULL auto_increment,
  foo INT NOT NULL,
  bar INT NOT NULL,
  mycomputed VARCHAR(255) GENERATED ALWAYS AS (
    CONCAT_WS(',', foo, bar)
  ) VIRTUAL NOT NULL,
  PRIMARY KEY (id)
);

https://github.com/prisma/prisma/issues/5006#issuecomment-1224036491 Workaround for MySQL's 0000-00-00 00:00:00
ALTER TABLE 'target_table' ADD COLUMN 'generated_date_column' DATETIME(0)
GENERATED ALWAYS AS (IF(target_date_column = "0000-00-00 00:00:00", NULL, target_date_column)) NULL;

https://github.com/prisma/prisma/issues/14799
CREATE TABLE "table" (
    ...

    -- manually edited migration to make column generated
    "hereBeDragons" TEXT NOT NULL GENERATED ALWAYS AS (
'this row ID is: '::text || "id") STORED,

    ...

https://github.com/prisma/prisma/issues/14799#issuecomment-1266204769
ALTER TABLE "event" ADD COLUMN "fts" tsvector GENERATED ALWAYS AS (
  to_tsvector('english',
    id::text -- add the cuid to the vector
    || ' ' || COALESCE("name", '') -- concat more fields (or empty string if null)
  )
) STORED;
CREATE INDEX event_fts_idx ON "event" using gin (fts); -- generate the index

https://github.com/prisma/prisma/issues/4368#issuecomment-1165692766
ALTER TABLE "my_table" ADD "myDateGenerated" DATE GENERATED ALWAYS AS (CASE WHEN "myDate" IS NULL THEN '1999-12-31' ELSE "myDate" END) STORED

https://github.com/prisma/prisma/issues/2378#issuecomment-627285131
CREATE TABLE `group` (
  `idGroup` smallint(5) unsigned NOT NULL AUTO_INCREMENT,
  `name` varchar(45) COLLATE cp1250_czech_cs NOT NULL,
  `idCategory` smallint(5) unsigned NOT NULL,
  `color` int(11) NOT NULL COMMENT 'C style color 0xAABBGGRR',
  `hex` varchar(7) COLLATE cp1250_czech_cs GENERATED ALWAYS AS (concat('#',substr(hex(((`color` >> 8) | ((`color` & 0xffff) << 16))),1,6))) STORED,
  PRIMARY KEY (`idGroup`)
);

Simplified example supporting a foreign key that can refer to a row coming from several different tables - in this example an identity that could be either a user or API key.
Using generated column for each related table to extract the foreign key when relevant. Used in turn to support Prisma relations so we have the fluid apis etc for accessing the related models:
schema.prisma:
model AuditLog {
  ...

  createdBy String? @db.VarChar(255)

  // generated fields - see migration.sql
  createdByUserId String? @db.VarChar(255) @default(dbgenerated())
  createdByApiKeyId String? @db.VarChar(255) @default(dbgenerated())

  createdByApiKey ApiKey? @relation("AuditLogCreatedByApiKey", fields: [createdByApiKeyId], references:[id])
  createdByUser User? @relation("AuditLogCreatedByUser", fields: [createdByUserId], references:[id])

  ...
}

migration.sql:
CREATE TABLE audit_log (
    ....

    created_by character varying(255),

    created_by_api_key_id character varying(255) GENERATED ALWAYS AS (
        CASE
            WHEN ((created_by)::text ~~ 'dkey_%'::text) THEN created_by
            ELSE NULL::character varying
        END) STORED,

    created_by_user_id character varying(255) GENERATED ALWAYS AS (
        CASE
            WHEN ((created_by)::text ~~ 'user_%'::text) THEN created_by
            ELSE NULL::character varying
        END) STORED,
);

(@map() attributes in prisma schema mapping column names from camelCase to snake_case omitted for brevity)

Another example in Supabase is with its storage schema and path_tokens:
CREATE TABLE storage.objects (
    id uuid DEFAULT extensions.uuid_generate_v4() NOT NULL,
    bucket_id text,
    name text,
    owner uuid,
    created_at timestamp with time zone DEFAULT now(),
    updated_at timestamp with time zone DEFAULT now(),
    last_accessed_at timestamp with time zone DEFAULT now(),
    metadata jsonb,
    path_tokens text[] GENERATED ALWAYS AS (string_to_array(name, '/'::text)) STORED,
    version text
);

I've been trying to use the workaround suggested here: https://github.com/nikolasburk/generated-columns, but prisma seems to revert any changes I make in a migration as soon as I apply it by dropping the columns and adding those specified in the prisma schema. Is there any workaround currently? My case looks like this - in prisma schema Event model:
  dates              DateTime[]
  startDate          DateTime?      @map("startDate")
  endDate            DateTime?      @map("endDate")

Adding this to migration before applying:
ALTER TABLE "Event"
    DROP COLUMN "startDate",
    DROP COLUMN "endDate";

ALTER TABLE "Event"
    ADD COLUMN "startDate" TIMESTAMP(3) GENERATED ALWAYS AS (
        CASE WHEN array_length(dates, 1) > 0 THEN dates[1] ELSE null END
        ) STORED,
    ADD COLUMN "endDate" TIMESTAMP(3) GENERATED ALWAYS AS (
        CASE
          WHEN array_length(dates, 1) IS NOT NULL THEN dates[array_upper(dates, 1)]
          ELSE NULL
        END
    ) STORED;

any updates on the above issue?

No update being posted means there is no update.

for users that stumbled upon:

In case anyone is interested in this feature find this issue, I just want to mention that there is a workaround for using GENERATED columns in Prisma, although they're not natively integrated in the Prisma schema yet. Here's an example repo that explains the workflow: https://github.com/nikolasburk/generated-columns

newer versions also need @default(dbgenerated()) - works with 4.16 - see https://github.com/prisma/prisma/issues/14799

I did find @Notalifeform's advice to work https://github.com/prisma/prisma/issues/6336#issuecomment-1650970223 using the latest prisma 5.3.1. I will note that it seems like I've lost then the ability to simply do a npx prisma db push and have the database be in the correct initial state while in my local dev/test environments.
Though I would guess this would be expected since the prisma schema file wouldn't have knowledge of the custom column definition that's defined in the migration.
Switching over to using npx prisma migrate dev every time to reset my database gave me the desired effect though to have the generated column show up in my live database schema.
Perhaps unique to my setup is that I'm using docker, and am also wiping the database first with a docker compose down. If you're not doing this with docker and instead have a local install of postgres, I'm suspecting you might need to do both:
npx prisma db push
npx prisma migrate dev

to get your schema in the state you would expect in dev.
Thanks to all those that helped contribute to documenting this workaround. This is preferable IMO to using a Prisma Client extension.

I'm running into the same issue as @rijenkii, but with a slightly different generation.
model Question {
  id String @id @default(cuid()) @db.VarChar(255)

  question String
  answer   String @default("")

  searchTsVector Unsupported("tsvector")? @default(dbgenerated()) @map("search_ts_vector")

  @@map("questions")
}

ALTER TABLE questions
ADD COLUMN search_ts_vector tsvector GENERATED ALWAYS AS (
  CASE
    WHEN answer IS NOT NULL THEN to_tsvector ('english', question || ' ' || answer)
  END
) STORED;

version 5.7.1

@rijenkii @mindblight This (continuous attempt of prisma migrate dev to undo the manual change to the migration SQL, and this not working with db push) are expected behaviors as long as this feature is not implemented by Prisma as a first class feature. Sorry.

It can be a preview feature and when provided then the @@generated something like could be used in which it takes an array like :
fullName String @@generated([firstName, ' ' ,lastName])

TO use this,
generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["generatedColumns"]
}

Kind of concept could be used which enables the use of @@generated option for the models.

@v1ctr @rijenkii @mindblight updating to version 5.14.0 fixed the issue for me
I think this now leads to a new error, where prisma tries to insert a default error on to the generated column

Error in batch request 2: Error occurred during query execution:
    ConnectorError(ConnectorError { user_facing_error: None, kind: QueryError(PostgresError { code: "428C9", message: "cannot insert a non-DEFAULT value into column \"myColumn\"", severity: "ERROR", detail: Some("Column \"myColumn\" is a generated column."), column: None, hint: None }), transient: false }

@rijenkii @mindblight This (continuous attempt of prisma migrate dev to undo the manual change to the migration SQL, and this not working with db push) are expected behaviors as long as this feature is not implemented by Prisma as a first class feature. Sorry.

Updating to 5.14.0 fixed the issue for me. However, it seems like there should be a way to tell prisma to entirely ignore interacting with something. My understanding is that Unsupported essentially tells prisma that the field is there, but that it should be ignored (except when creating the migration). Is the correct thing to do use @ignore or the like? I access this field via raw SQL anyways, so I just want Prisma to more or less ignore it.

Want to add: @ignore might work for Unsupported, but in my case I want to expose the column:
number: Float? @default(dbgenerated())
With this setup I am still getting an error with 5.14.0.

I was searching for something that looked like Object annotation as Django has and stumbled upon this. @janpio Asked for more examples so here is a list of similar issues I've found.
https://github.com/prisma/prisma/issues/4554
https://github.com/prisma/prisma/issues/3394#issuecomment-1416715544
https://github.com/prisma/prisma/issues/18543
https://github.com/prisma/prisma/issues/7181#issuecomment-1379155060
Though I expect the solution to be quite difficult to do typesafe. There are many ways how you could calculate a column in mysql. In my case i had 2 date columns and needed to sort based on the greatest of both:
ALTER TABLE 'exampleTable' ADD COLUMN 'maxOfBoth' DATETIME(3) GENERATED ALWAYS AS (GREATEST(columnA, columnB)) NULL;

It can be a preview feature and when provided then the @@generated something like could be used in which it takes an array like :
fullName String @@generated([firstName,lastName])

Here one catch can be concat only the values that are provided. Like, keep in mind the schema for that model too. So, if lastName is not provided, its value doesn't get concatenated to the firstName.

TO use this,
generator client {
  provider        = "prisma-client-js"
  previewFeatures = ["generatedColumns"]
}

Kind of concept could be used which enables the use of @@generated option for the models.

Can't this be one way to consume the generatedColumns feature


Black screen when in multiplayer

When it is played in multiplayer, anyone who joins gets a black screen and their model doesnt load.

Are you sure that this is caused by Poltergeist? I've used it on the current version and didn't encounter that issue. If you can, send the log file when this occurs.

My friend was getting this spammed every frame with poltergeist enabled
[Warning: Unity Log] [Netcode] NetworkObject (2) children not resolved to parents by the end of frame
[Info   : Unity Log] soumdmanager: True; False
When we disable poltergeist it works fine

This could be a CSync issue as Arachnophilia has a similar issue with people not joining and having a black screen

Hm, okay, I'll look into it.

I can't figure out what it causing your issue, but I don't think it's caused directly by Poltergeist.
When I boot your modpack (with or without Poltergeist) I see a massive error for FixPluginTypesSerialization that could be causing problems?
I will say that Poltergeist works fine in my modpack, so I think it's not an inherent issue with the mod.

If I can find the incompatible mod, i'll respond here with it

Also CSync requires FixPluginTypesSerialization


Document TypeHelper usage in the readme :)

Yes... that would be great. I'm a bit stuck here. @kevmoo it would be great if you could maybe add a comment here if you don't have the time to update the docs.
In my specific case, I need to convert Strings "123.45" to Decimal: Decimal.parse(value).

Fwiw, instead of trying to use the TypeHelper stuff you can just use the fromJson and toJson optional args on the JsonKey annotation to use a custom function. The main issue with that is you have to do it per field though.

@enyo – does the comment from @jakemac53 help?

@kevmoo yes definitely! That is a completely acceptable workaround for now.

Hey, sorry to open this again but I struggle with implementing the workaround...
what I try is something like:
  @JsonKey(
    name: "rate",
    fromJson: Decimal.parse(value),
    toJson: value.toString(),
  )
  Decimal rate;

fromJson doesn't work because of Arguments of a constant creation must be constant expressions. and toJson because of The argument type 'String' can't be assigned to the parameter type 'Function'.
I get that toJsoncan't work this way but also no Idea how to get this - and for fromJson I don't really know what the error message means tbh :/
Thx for any hints

Great to hear, @elmcrest

For others who are looking for away to do this with Decimal without having to create static methods that you have to reference everywhere. This is the way I did it. It would be helpful for there to be more documentation about trying converters for types from other packages. I do not know if this is the "right way" but to me the implementation is cleaner.
@JsonSerializable(nullable: false)
@_DecimalStringConverter()
class DecimalClass {
 Decimal myDecimal;
}

class _DecimalStringConverter implements JsonConverter<Decimal, String> {
  const _DecimalStringConverter();

  @override
  Decimal fromJson(String json) => json == null ? null : Decimal.parse(json);

  @override
  String toJson(Decimal object) => object?.toString();
}

Honestly, I want to get away from expose the type helper stuff.
It's a lot of internal plumbing that's just got more complex.


cryptsetup: 2.0.5 -> 2.0.6

Motivation for this change
https://gitlab.com/cryptsetup/cryptsetup/blob/master/docs/v2.0.6-ReleaseNotes
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Oops, didn't see this. Update was done in #52366 as well.

All good, thanks!


Simplify scheduler.cohortsUsage; Cleanup FlavorResourceQuantities

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Simplify cohortsUsage, by no longer taking it as a pointer receiver. We also change it to FlavorResourceQuantitiesFlat, so we no longer have to do allocations/null checks inside of loops.
Make more progress towards flattening FlavorResourceQuantities #2447
Does this PR introduce a user-facing change?
NONE

/cc @macsko

Thanks for the review, @macsko


Add version for gluster block provisioner.

Signed-off-by: Humble Chirammal<EMAIL_ADDRESS>

@wongma7 PTAL :)

needs rebase, would you like me to push the image with tag v0.5 to quay.io/external_storage after this is merged?

@wongma7 rebased. Yeah, it would be good to push the image with tag v0.5, however let me ping you before you push the image. :)


pseudoUrl to match URL's not ending in jpg or png

Hi,
with PseudoUrl, I want to match URLs not ending in jpg or png (not image).
It's Ok with this regular expression http://example.com/(?!\S+(?:jpe?g|png|bmp|gif))\S+, but it's KO with Apify.PseudoUrl
Example :
const purl = new Apify.PseudoUrl('http://example.com/[(?!\S+(?:jpe?g|png|bmp|gif))\S+]');
if (purl.matches('http://example.com/a-propos/lab/lemoussel/')) console.log('Match!'); // => Not maching
if (purl.matches('http://example.com/wp-content/uploads/iamge0.png')) console.log('Match!');

Maybe I misuse Apify.PseudoUrl but I can't find a way to solve it.

Hi there, thanks for reporting this, we'll investigate it. As a workaround, you can initialize the PseudoUrl class using a full regular expression.

What do you want to indicate by using a full regular expression?
Is that it?

const purl = new Apify.PseudoUrl('[http://example.com/(?!\S+(?:jpe?g|png|bmp|gif))\S+]');

Something like:
const purl = new Apify.PseudoUrl(/http\:\/\/example\.com\/(?!\S+(?:jpe?g|png|bmp|gif))\S+]/);

Found why in escaping \S.
Solution :  const purl = new Apify.PseudoUrl('http://example.com/' + '[(?!\\S+(?:jpe?g|png|bmp|gif))\\S+]');

https://github.com/apifytech/apify-js/commit/cd0c2192c951c68cd5d693a3451c9e2667dcdff4
Consider fixed by updating documentation.
Thank you for the issue @LeMoussel .


Add nightly python

Current coverage is 0.00% (diff: 100%)

Merging #7 into master will not change coverage

@@           master    #7   diff @@
===================================
  Files           5     5
  Lines         103   103
  Methods         0     0
  Messages        0     0
  Branches        6     6
===================================
  Hits            0     0
  Misses        103   103
  Partials        0     0

Powered by Codecov. Last update 0a7ec47...4cd9dae


fix: when no operation command, execute the dev in the remix cli

Currently, remix occurred error when we not give operator command to remix cli
$ npx remix
TypeError: Cannot read properties of undefined (reading 'startsWith')
    at Object.run (/app/packages/remix-dev/dist/cli/run.js:171:17)
    at Object.<anonymous> (/app/packages/remix-dev/dist/cli.js:16:11)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)
    at node:internal/main/run_main_module:28:49

Because, there are parts of cli that expect operation command to always exist. We originally expect the shortcut to the dev command to run, so I modified it.

[ ] Docs
[x] Tests

Testing Strategy:
I added tests so, you can check with run test runner.
pnpm jest packages/remix-dev/__tests__/cli-test.ts

Or, package build and run cli.
pnpm build
node packages/remix-dev/dist/cli.js

Hi, @brophdawg11
This PR is a minor fix, but it will save people who are confused. Can this be merged?

Thank you for the PR but we're not looking to make further functional changes to the Remix v2 CLI.  You are free to open a proposal for the coming react-router CLI in v7.
My personal opinion is that the lack of a command should print the help contents - not default to a specific command.

That's okey. Thank you for let me know.


Add a Value conformance for Foundation NSURL

Thanks, can you merge in the current mainline with your branch? The build fails now because the underlying CI image has changed.

There is only one branch to merge into

No, I meant to update from the base branch.
See https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/keeping-your-pull-request-in-sync-with-the-base-branch


[PE-D][Tester B] swap-task does not work when swapped with a task number that does not exist

When i specify a task number that does not exist, no error message is printed. It just remains in the same position
swap-task m/cs2103t ts/1 4 does not work when i only have 3 tasks-->no error message printed

Labels: type.FunctionalityBug severity.Medium
original: Kennethsim2000/ped#8

Duplicate of #136


Can't see Quickfix selected file name

This colorscheme is awesome, I was update my config, but found a small issue with quickfix.

Should look a lot better now 🙁 thanks for raising


Feature request: floating nodes

Please describe the feature that you want to propose
I am looking to add a node at a fixed position on the canvas such that it doesn't move when panning or zooming in/out? Are there any plans to support this in the future? I see the Node type has a positionAbsolute attribute, but this is not documented.
Thank you!

Should it scale when you zoom? Is it connected with other nodes?


enable_active_mesh errored out

Getting this error while using spoke modules, need to fix the code
Error: Unsupported argument │  │   on .terraform/modules/spoke_azure_1/main.tf line 27, in resource "aviatrix_spoke_gateway" "default": │   27:   enable_active_mesh                    = var.active_mesh │  │ An argument named "enable_active_mesh" is not expected here.

Hi Satya, thanks for reporting this. As per controller version 6.6, enable_active_mesh is no longer a supported argument for the aviatrix_spoke_gateway which is part of this module. That is what's causing this error to arise.
In stead of updating this module, we have release a multi-cloud spoke module that is usable for Azure as well as other clouds. I would like to suggest to start using that one. Migrating shouldn't be to complex, but I'm happy to help if you need support.
You can find the module here:
https://github.com/terraform-aviatrix-modules/terraform-aviatrix-mc-spoke
Some small behavior changes that you have to account for:

Prefix/suffix addition of "avx-" and "-spoke" to the name has been removed. If you were using this (which is default), you would have to add these to the name to prevent a change in the name.
As this module is multi-cloud, you need to add an argument to inform the module for which cloud it is deploying. It's very simple. Just add the below code to calling the module.

cloud = "azure"
Please let me know if you have any questions.

Hi Dennis,
Thanks for the response. I will take a look at MC modules.
Regards,
Satya
From: Dennis Hagens @.>
Date: Wednesday, March 9, 2022 at 4:11 AM
To: terraform-aviatrix-modules/terraform-aviatrix-azure-spoke @.>
Cc: Satyanarayana Prasad Chalavadi @.>, Author @.>
Subject: Re: [terraform-aviatrix-modules/terraform-aviatrix-azure-spoke] enable_active_mesh errored out (Issue #15)
Hi Satya, thanks for reporting this. As per controller version 6.6, enable_active_mesh is no longer a supported argument for the aviatrix_spoke_gateway which is part of this module. That is what's causing this error to arise.
In stead of updating this module, we have release a multi-cloud spoke module that is usable for Azure as well as other clouds. I would like to suggest to start using that one. Migrating shouldn't be to complex, but I'm happy to help if you need support.
You can find the module here:
https://github.com/terraform-aviatrix-modules/terraform-aviatrix-mc-spokehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fterraform-aviatrix-modules%2Fterraform-aviatrix-mc-spoke&data=04|01|schalavadi%40aviatrix.com|a7929c00d68943eb6f1008da01b53256|4780055ece374f02b33dfdad8493a4b6|0|0|637824175005309581|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=CdnkuNfT06b%2Bxr26jbcCwbAUGwVCp1fSuz6h8%2FOHd74%3D&reserved=0
Some small behavior changes that you have to account for:

Prefix/suffix addition of "avx-" and "-spoke" to the name has been removed. If you were using this (which is default), you would have to add these to the name to prevent a change in the name.
As this module is multi-cloud, you need to add an argument to inform the module for which cloud it is deploying. It's very simple. Just add the below code to calling the module.

cloud = "azure"
Please let me know if you have any questions.
—
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fterraform-aviatrix-modules%2Fterraform-aviatrix-azure-spoke%2Fissues%2F15%23issuecomment-1062760335&data=04|01|schalavadi%40aviatrix.com|a7929c00d68943eb6f1008da01b53256|4780055ece374f02b33dfdad8493a4b6|0|0|637824175005309581|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=oZtccPe0L4gxX6uzXtZQViLt%2FNnK%2Fu6B6XtHxHko0Co%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAT3T3A2CYZAO4RT2W2GM2TTU7B2NNANCNFSM5QH6KDXQ&data=04|01|schalavadi%40aviatrix.com|a7929c00d68943eb6f1008da01b53256|4780055ece374f02b33dfdad8493a4b6|0|0|637824175005309581|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=k9ekRVMD8VyphLutjV3ca6JkuuENZgUPnW88e9uUfCo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

Let me know if you need help. Closing this issue.


Update people.md

According to Linkedin , Joydeep Sensarma is no longer working at Qubole but Clearfeed.ai
https://www.linkedin.com/in/joydeeps/

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

Is the change a dependency upgrade?

How was this patch tested?

This is to be updated only by the folks themselves, so unless Joydeep wants to do that himself, nobody else is suppose to play with that.
Closing


ValueError: I/O operation on closed file.

I am getting this with pipenv 3.4.2 and 3.5.6. Running any pipenv command, Ubuntu 16.04. pew is installed, per issue 195. pip is upgraded to latest 9.0.1.
Traceback (most recent call last):   File "/usr/local/bin/pipenv", line 11, in <module>     sys.exit(cli())   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/core.py", line 722, in __call__     return self.main(*args, **kwargs)   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/core.py", line 697, in main     rv = self.invoke(ctx)   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/core.py", line 1043, in invoke     return Command.invoke(self, ctx)   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/core.py", line 895, in invoke     return ctx.invoke(self.callback, **ctx.params)   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/core.py", line 535, in invoke     return callback(*args, **kwargs)   File "/home/rmsadmin/.local/lib/python3.5/site-packages/click/decorators.py", line 17, in new_func     return f(get_current_context(), *args, **kwargs)   File "/usr/local/lib/python3.5/dist-packages/pipenv/cli.py", line 756, in cli     ensure_project(three=three, python=python)   File "/usr/local/lib/python3.5/dist-packages/pipenv/cli.py", line 134, in ensure_project     ensure_pipfile(validate=validate)   File "/usr/local/lib/python3.5/dist-packages/pipenv/cli.py", line 99, in ensure_pipfile     if validate and project.virtualenv_exists:   File "/usr/local/lib/python3.5/dist-packages/pipenv/project.py", line 43, in virtualenv_exists     return os.path.isdir(self.virtualenv_location)   File "/usr/local/lib/python3.5/dist-packages/pipenv/project.py", line 80, in virtualenv_location     loc = c.out.strip()   File "/usr/local/lib/python3.5/dist-packages/delegator.py", line 89, in out     self.__out = self.std_out.read() ValueError: I/O operation on closed file.

Thanks for opening this @generalavc, this is actually a bug in delegator.py and a dup of #310. Please follow the instructions there to fix the issue.

super! thanks, I didn't find that before.


Use with self-hosted runners?

I am using self-hosted runners via https://github.com/actions/actions-runner-controller and have bumped into this issue:
##[debug]Failed to delete archive: Error: ENOENT: no such file or directory, unlink ''
Recording preexisting Docker images. These include standard images pre-cached by GitHub Actions when Docker is run as root.
docker image list --format "{{ .Repository }}:{{ .Tag }}"
Error: Error: spawn /usr/bin/bash ENOENT

Is use of this a no-no with self-hosted runners?

Does your runner have Bash?


fix for 1429

when both the data length and facet data length are undefined, suppress the warning
fixes #1429
Note: I initially added arquero as a devDependency and @juba’s penguins test, but removed it as I felt it adds too much cruft. The test was:
import * as Plot from "@observablehq/plot";
import * as d3 from "d3";
import * as aq from "arquero";

export async function penguinArqueroFacet() {
  const data = await d3.csv<any>("data/penguins.csv", d3.autoType);
  const penguins = aq.from(data).toArrow();
  return Plot.plot({
    facet: {data: penguins, x: "island"},
    marks: [Plot.dot(penguins, {x: "culmen_depth_mm", y: "body_mass_g"}), Plot.frame()]
  });
}

Ah ho! You found the bug. Excellent!


Update Inspire command, added xmas flag for Xmas Season!

new flag on inspire commanf for XMAS season.
Screenshot to show it gives images related to xmas:

-1 on this one. Seasonal commands don't make a lot of sense imo

Well yeah it is seasonal, but it's not like it gonna get removed from API. and it's just 1 flag so shouldn't matter much. v:

Flag it as season then accepting string argument, because maybe they'll add more seasons to their query > &season=xmas


tsnet integration

it's possible to have a simple option to run sniproxy out of Tailscale directly, using the tsnet library. it'd be good to build that out for users that want to manage their ACLs through Tailscale/Headscale rather than repeating it through sniproxy.

tailscale now ships with its own sniproxy so this is potentially not needed anymore.


List comprehension issues

Issue with setting link['lists'] to the incorrect value due to list
comprehension. Probably similar issue with link['tags']

Hey @vwheezy22, can you maybe elaborate on the problem here so I understand it better? :)


Disabling flaky Navigation Redirect test

Please review the release process for BrowserServicesKit here.
Required:
Task/Issue URL: https://app.asana.com/0/1205237866452338/1205344247172464/f
iOS PR:
macOS PR:
What kind of version bump will this require?: Patch
Description:
Disables a flaky test

Steps to test this PR:

Confirm test testClientRedirectToSameDocument is skipped

OS Testing:

[ ] iOS 14
[ ] iOS 15
[ ] iOS 16
[ ] macOS 10.15
[ ] macOS 11
[ ] macOS 12

Internal references:
Software Engineering Expectations
Technical Design Template

@mallexxx since there wasn't much of a stack trace / error message in the failed unit test run (✖ testClientRedirectToSameDocument, failed -  🤷♀ ), let me know if you'd prefer if this unit test is kept enabled for now to help narrow down the cause


[Sema][SR-2041] Quick fix for off-by-count errors in pattern binding diagnostics

What's in this pull request?

Rather than return the last var pattern, only match on the ones that
match the vardecl the diagnostic is warning about.
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci please test and merge.


Previous code

How is it possible to copy the code which you wrote before in last lines?

Closing since this seems to be a mistakenly created issue


Hyundai CAN-FD: remove 90° steering lockout

panda PR: https://github.com/commaai/panda/pull/1098
TODO:

[ ] Decide if we need torque winddown bit.
Certain cars are not affected immediately by dropping torque actuation bit and maintain full torque

I just passed the torque_fault along from carstate.py to hyundaicanfd.py. we definitely need torque winddown bit still since else we get a slapping noise on the EPS. I tried this a few weeks back and now
I did the same with the branch here. See video for reference: https://discord.com/channels/469524606043160576/538741329799413760/1019205700251029554
I searched for days for the correct winddown but without success.
"STEER_REQ": 1 if lat_active and not torque_fault else 0,

Tried out 1 frame multiple times a second vs. 2 frames once a second, and we indeed need to send two invalid messages consecutively to make this fault-free: https://imgur.com/a/DM8xwh6

@ioniq5-cz do you have HDA2 or 1? Does openpilot send LFA or LKAS for your car? I noticed tonight that LFA does the winddown for LKAS, so if we can send LFA we improve the loss of torque a bit. Sending LKAS and setting the bit low, the Driving ECU actually ramps our requested torque down and up each time

HDA2

Ah, so make sure openpilot longitudinal is enabled and let me know how it goes when you try it again!


Central problem

I'm trying to compile the code as is (on Ubuntu machine), but keep getting an error
"Failed to execute goal on project cassandra-binding: Could not resolve dependencies for project com.yahoo.ycsb:cassandra-binding:jar:0.1.4: Could not find artifact com.yahoo.ycsb:core:jar:0.1.4 in central (http://repo.maven.apache.org/maven2)"
Core compiles fine. I'm a newbie to Java and Maven etc, but this seems to indicate there's a problem in the central repository that YCSB is trying to access. Looking at it I dont find the relevant jar file
http://search.maven.org/#browse|-2026234819
Am I doing something wrong, or has something changed?
Thank you

Were you able to fix this. I am facing same issues

Check that you are actually using the central maven repo, you can try adding to you pom file this:

central
Central
http://repo1.maven.org/maven2

I also know for sure some dependencies like mapkeeper is not deleted from the central maven repo so you could comment it out from the pom file until it is fixed.

@haishmg  can you give more details on what command you're running? building top level currently works from master AFAICT.

closing as stale. if you are still facing this issue with the current master branch, please reopen.


Show that thec invoke service is currently broken

ping @jsantos17

This needs merge from master.

This is fixed in the import branch I am working on and about to PR


Add changes to support tf12

1 - change google_compute_health_check to support all http params
2 - delete some unnecessary []
regarding #3 #10

@morgante I've check examples and they depend on another module which still tf11. So what will be the best way - update GoogleCloudPlatform/managed-instance-group/google and then refactor these examples or what?

@evkuzin please update the examples to use terraform-google-modules/vm/google//modules/mig.

Update with examples. But examples also use module with external LB which is still tf11. So I erased it from examples.

@omazin please take this on.