[C++][Parquet] parquet cpp - improve examples

It would be a great to have examples of using parquet arrow high-level API for the following 2 cases

Storing nested data types (storing nested data types is touted as major merit of parquet, so I think this case should be included as an example). Ideally, an example of how to use arrow::StructArray nested with several primities types, list types and other nested types would cover every case of nested hierarchy of complex data representations
Buffered or Batched writes to parquet file. Parquet is meant to be used for large amounts of data. The current examples stores all of the data as in arrow data structures, before writing to parquet file, which has a huge memory footprint, proportional to the amount of data being stored. An example of writing directly to row groups and columns, can nicely demonstrate how to store data with smaller memory footprint. The current example creates a arrow::Table, which needs to be filled with arrow::Array(s) of entire data, size of which is bounded by the amount of RAM. Ideally, an example which generates some data in several arrow::Array(s), and then stores (appends) them as a new Row Group (or Column Chunk) in a parquet::arrow::FileWriter, using NewRowGroup and WriteColumnChunk functions, thus demonstrating a lower memory footprint for writing a parquet file with huge amounts of data

Reporter: Rajeshwar Agrawal
Note: This issue was originally created as PARQUET-1526. Please see the migration documentation for further details.

Gang Wu / @wgtmac:
This has been resolved in [ARROW-14161: [C++][Docs] Improve Parquet C++ docs by wjones127 · Pull Request #14018 · apache/arrow (github.com)|https://github.com/apache/arrow/pull/14018]


Updating "bns" environment

Do we need to add dask module to the environment?

The current yml has distributed which will install dask accordingly.

Thank you very much!


Add gradient scrollbar

Implement a scrollbar with changing gradients on the scroll.

Can you assign me to this


Subsetting grib2 files for North America area and adding another APCP variable

Description

This PR added a utility function to subset grib2 files for North America area.
Added another variable APCP to represent an accumulation of the predicted precipitation amounts since model initialized.

Linked Issues

Closes https://github.com/NOAA-EMC/graphcast/issues/32

Blocking Dependencies

None
Anticipated Changes
None
Input data

[x] No changes are expected to input data.
[ ] Changes are expected to input data:

[ ] New input data.
[ ] Updated input data.

Needed libraries

None

@SadeghTabas-NOAA Please let me know the preferred subsetting bounding box. Thanks.

@SadeghTabas-NOAA Please let me know the preferred subsetting bounding box. Thanks.
@LinlinCui-NOAA Please consider an approximate bound like this figure, if we needed we can correct the values. Thanks

@LinlinCui-NOAA Please do your modifications in utility scritp, do not update run_graphcast.py. These are experimental and we don't want to update the main code. Thanks

@LinlinCui-NOAA Please do your modifications in utility scritp, do not update run_graphcast.py. These are experimental and we don't want to update the main code. Thanks

The upload function uploads every file in the output directory to s3, please have a look and see if it uploads the grib2 files in the subset folder in the right directory. I'll send you the credentials for S3 bucket. Thanks

The upload function uploads every file in the output directory to s3, please have a look and see if it uploads the grib2 files in the subset folder in the right directory. I'll send you the credentials for S3 bucket. Thanks

@SadeghTabas-NOAA I am not sure if subfolder "north_america" must be pre-created to upload files there. But the script does upload files in the subfolder:
root is forecasts_13_levels
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186
.
.
.
root is forecasts_13_levels/north_america
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186

The upload function uploads every file in the output directory to s3, please have a look and see if it uploads the grib2 files in the subset folder in the right directory. I'll send you the credentials for S3 bucket. Thanks

@SadeghTabas-NOAA I am not sure if subfolder "north_america" must be pre-created to upload files there. But the script does upload files in the subfolder:
root is forecasts_13_levels
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186
.
.
.
root is forecasts_13_levels/north_america
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186

@LinlinCui-NOAA As we talked about today, we will not upload the subset results to S3 and we won't modify the code as well, please just make a sample subset of the grib2 file for f006 (over North America) and share it with @junwang-noaa.  So, your current PR should only reflect the new APCP message in the grib2 files. Please update the PR title as well. Thanks

@LinlinCui-NOAA Also, if you overwrite something on S3, please let me know so I can regenerate those. Thanks

The upload function uploads every file in the output directory to s3, please have a look and see if it uploads the grib2 files in the subset folder in the right directory. I'll send you the credentials for S3 bucket. Thanks

@SadeghTabas-NOAA I am not sure if subfolder "north_america" must be pre-created to upload files there. But the script does upload files in the subfolder:
root is forecasts_13_levels
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/graphcastgfs.t00z.pgrb2.0p25.f186
.
.
.
root is forecasts_13_levels/north_america
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f102
uploading forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186 to graphcastgfs.20240215/00/forecasts_13_levels/north_america/graphcastgfs.t00z.pgrb2.0p25.f186

@LinlinCui-NOAA As we talked about today, we will not upload the subset results to S3 and we won't modify the code as well, please just make a sample subset of the grib2 file for f006 (over North America) and share it with @junwang-noaa. So, your current PR should only reflect the new APCP message in the grib2 files. Please update the PR title as well. Thanks

@SadeghTabas-NOAA This was posted before the meeting. The latest commit has commented out subsetting function.

@LinlinCui-NOAA Also, if you overwrite something on S3, please let me know so I can regenerate those. Thanks

@SadeghTabas-NOAA I did not upload files to S3, just printed file names on both sides to check.


Update Functional and Non-functional requirements

#20 Added week 2 requirements

Is this issue closed now?


DNM: testing apple/swift-installer-scripts#177

Please test with following PRs:
apple/swift-installer-scripts#177
@swift-ci please build toolchain Windows platform


Improve taproot builder conversion errors

The two conversion functions on TaprootBuilder currently return the same error type even though it contains a variant that cannot be returned for try_into_node_info.
Split the error up in the customary fashion.
Please note this PR shows a new favouring of named fields in struct errors instead of using a tuple struct. The struct has a private field so non_exhaustive is omitted.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-15 of 18 (16.67%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 84.053%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

bitcoin/src/taproot/mod.rs
3
18
16.67%

Files with Coverage Reduction
New Missed Lines
%

bitcoin/src/taproot/mod.rs
1
77.79%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
19386

Relevant Lines:
23064

💛  - Coveralls

Note also that this PR attempts to clear up the terminology used around "finalized" and "complete" - please holla if you think I am incorrect here.

One more thought: are prefixes in array-like hex things actually used anywhere? I've never seen a transaction, hash or other byte array entity written as 0x.... So maybe it makes sense to just get rid of all that prefix stuff entirely?

I'm not happy with this PR, converting to draft.


Add features for TestRunner

Fix #108
Add features option to TestRunner

Hi @dtolnay,
There is a crate using trybuild for testing, and there are some features not easy to test without additional features.  May you take a look on this PR, and give me some suggestions about how to make this PR be accepted, thanks.

@yanganto did you find solution of the problem without getting the PR merged into trybuild? any alternative to trybuild?

@yanganto if you are aware about alternative please share

Hi @Wandalen,
I use cargo hack to solve this issue and not use try build for feature test. :smile:


##[warning]Atleast for one assembly start time was not obtained due to tag not present or parsing issue, total run duration will now be summation of time taken by each assembly

Describe the issue
In some pipeline runs I get this warning when publishing the test results:
##[warning]Atleast for one assembly start time was not obtained due to tag not present or parsing issue, total run duration will now be summation of time taken by each assembly
I've hunted it down to that this is depending on the time format of the container. I get this in the Finnish localization, and they have a dot (.) as time separator - not colon (:) as many others.
So in the test result file we get run-time="11.25.21" instead of run-time="11:25:21", which is unexpected by DevOps...
Now I found out that the datetime.ToString("HH:mm:ss") does not do what you expect. The : in the format string says that it should use the current configured time separator, not : as we write. (ref: https://docs.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings#timeSeparator)
The solution is to wrap the : inside '
datetime.ToString("HH':'mm':'ss")
A PR to fix this is on it's way...

What version is this? - is this when writing the XUnit compatible test results?

I'm running <IP_ADDRESS>, but the code is the same in latest version.
Yes, the XUnitResultFileName is set.

I got this before:
  <assembly name="50105 NABCI General Tester" test-framework="PS Test Runner" run-date="2020-05-28" run-time="14.04.22" total="7" passed="7" failed="0" skipped="0" time="69.121">
And after my fix I get this:
  <assembly name="50105 NABCI General Tester" test-framework="PS Test Runner" run-date="2020-05-28" run-time="14:25:32" total="7" passed="7" failed="0" skipped="0" time="68.326">

it's easy to repro in PowerShell, just set you regional settings to Finnish and run below:
[datetime]$dt = Get-Date
$dt.ToString("HH:mm:ss")
$dt.ToString("HH':'mm':'ss")


Refactor gardener-seed-admission-controller

How to categorize this PR?

/area quality dev-productivity
/kind cleanup test
/priority normal
What this PR does / why we need it:
This PR refactors/enhances several things regarding the seed-admission-controller:

migrate the component to a full controller-runtime component (only usage of manager was left)
enable both admission review versions (including admission/v1) in the webhook configs, as it is now supported by the c-r webhook handlers
decrease the timeouts of the extension deletion protection webhook to 10s according to best pratices
add an envtest-style integration test for the extension deletion protection webhook
migrate the handcrafted integration test for the scheduler name webhook to an envtest-style test

Which issue(s) this PR fixes:
Part of https://github.com/gardener/gardener/issues/3109
Special notes for your reviewer:
The PR got a bit bigger than expected, but it's still split into several commits, so I hope this helps with the review.
/squash
Release note:

`gardener-seed-admission-controller`'s webhooks now also accept reviews in version `admission/v1`.
Also, webhook timeouts have been lowered to `10s` for the extension deletion protection webhooks.

/assign
/rebase

/assign
/rebase

/lgtm
I have a general question: I used to write envtest while working on other projects in the past (SEN). I think it's the first such test added to g/g, possibly because it requires a controller runtime component (is this correct?). Are there plans to migrate other components (e.g. gardenlet) to controller runtime as well, so we could write add such tests for them as well?

I think it's the first such test added to g/g, possibly because it requires a controller runtime component (is this correct?).

It's not the first one, but the second (https://github.com/gardener/gardener/pull/2830 was the first one) 😉
You're right though, there are not many envtests in g/g currently.
I think, we should be able to leverage envtest for integration tests for some of our controllers as well, at least for the ones that already implement the reconciler.Reconcile interface. Though, we haven't done any of those yet.

Are there plans to migrate other components (e.g. gardenlet) to controller runtime as well, so we could write add such tests for them as well?

I plan to do the refactoring for gardener-admission-controller this week (similar to this PR).
I think, the general goal is to do the same for the controller components as well, though it will be much more effort and might reveal some difficulties. As of now, there are no concrete plans for this, I guess.

Rebased to resolve conflicts.


didFailedtoPlay delegate method

I have integrated ZFPlayer in my project successfully. I want to add delegate method if player did fail to play video from url.
How to integrate delegate method for didFailedtoPlay.

This is very easy. You can write your own PlayerControlView (ZFPlayerControlView for reference) and add your custom delegate. Then call the didFailedtoPlay protocol in the zf_playerItemStatusFailed function.
override func zf_playerItemStatusFailed(_ error: Error!) {
	customDelegate?.didFailedtoPlay()
}


Added xSQLServerScript Resource

xSQLServerScript resource to extend DSCs Get/Set/Test functionality to T-SQL

This change is

Hi @dcrreynolds, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (David Reynolds). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

The automated test are not fans of using UserName and Password, however, it directly matches how Invoke-SqlCmd works. I opted to leave it matching Invoke-SqlCmd to stay consistent with the cmdlet, but am open to changing it too.

Any opinions of this? Do we follow the PowerShell standard and use a PSCred object, or use the Invoke-SqlCmd language and update the test to not fail here.

I believe everything else in xSqlServer is utilizing PSCred Object so for consistency I would go with PSCred

Reviewed 1 of 5 files at r2, 2 of 3 files at r3.
Review status: 3 of 5 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

README.md, line 278 [r3] (raw file):

* **SetFilePath**: (Key) Path to SQL file that will perform Set action.
* **GetFilePath**: (Key) Path to SQL file that will perform Get action. SQL Queries returned by this function are returned by the Get-DscConfiguration cmdlet with the GetResult parameter.
* **TestFilePath**: (Key) Path to SQL file that will perform Test action. Any Script that does not throw an error and returns null is evaluated to true. Invoke-SqlCmd treats SQL Print statements as verbose text, this will not cause a the Test to return false.

Typo here - "this will not cause a the Test to return false."

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 8 [r3] (raw file):

# TODO: Customize these parameters...
$Global:DSCModuleName      = 'MSFT_xSQLServerScript' # Example xNetworking

Take out comments from template - also the tests templates recently got updated to change these variables from global to script, so can you change both of these to be $script: variables?

Comments from Reviewable

Thanks for the feedback! I'll get this fixed up.

@dcrreynolds - My thoughts would be to deal with the last comment as per #91 (with a better suggestion in there too) in a separate PR just thought it needed to be called out as being something that needed to be thought of thats all

@mbreakey3 I've added your changes in, however, Reviewable keeps crashing and will not let me mark the comments as done. @kilasuit I would agree with updating the resource as one PR for #91 it will help ensure the fix is done uniformly.

Reviewed 1 of 2 files at r4.
Review status: 4 of 5 files reviewed at latest revision, 5 unresolved discussions.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 29 [r4] (raw file):

try
{
    InModuleScope $DSCResourceName {

We're also not using the InModuleScope anymore for testing exported functions due to the fact that it can lead to conflicts.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 89 [r4] (raw file):

            It "Should return true if Invoke-SqlCmd returns null" {
                Mock -CommandName Import-Module -MockWith {}

Since you're mocking these same commands for almost all of these It statements you can create a Context block (which is almost the same as a Describe block) and put all of the It statements that use that mocked function within that Context block and just declare the mock once at the top of that Context block

Comments from Reviewable

Just started over with the test using the current unit test template.

Reviewed 1 of 5 files at r2, 2 of 3 files at r3, 1 of 2 files at r4, 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 8 unresolved discussions.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 30 [r3] (raw file):
Previously, kilasuit (Ryan Yates) wrote…
> With the latest updates to SSMS the SQLPS module was renamed to SQLServer so baring this in mind should we not have a check in here for the change of module name for this along the lines of
>
> ```powershell
>
> If (Get-Module SQLPS -ListAvailable)
>
>     {
>
>         Write-Verbose "SQLPS Module Found"
>
>         $module = Get-Module SQLPS -ListAvailable
>
>     }
>
>     else
>
>     {
>
>     Write-Verbose "SQLPS Module Not found - Using updated SQLServer module "
>
>     $module = Get-Module SQLServer -ListAvailable
>
>     }
>
>
>
> Import-Module $module -WarningAction SilentlyContinue -ErrorAction Stop
>
> ```
>
>
>
> Though this would need to be actioned in the same way across all of the resources where this is currently in place
>
>
>
>
>
> Referenced this in #91 as well

It could be changed to use the helper function `Import-SQLPSModule` (in the xSQLServerHelper module). Then we only have to change the logic of which module is loaded (the #91 issue) in one place. Is this doable?

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 7 [r5] (raw file):

    param
    (
        [parameter(Mandatory = $true)]

Can you change this to [Parameter (using capital P). If you can do this here and in all other places in the resource code?

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 49 [r5] (raw file):

    $returnValue = @{
        ServerInstance = [System.String]$ServerInstance

Can you change this to [System.String] $ServerInstance (white space between type and variable). The same for the rest too below.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):

    {
        Write-Verbose $_
        return $false

Everytime an error occurs in this try-block you will send $false back to LCM so it will run the Set-method. Is this your intended behaviour? You should not throw and error instead so an error is reported back to the user? Ie. throw New-TerminatingError

Comments from Reviewable

Review status: all files reviewed at latest revision, 8 unresolved discussions.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> Everytime an error occurs in this try-block you will send $false back to LCM so it will run the Set-method. Is this your intended behaviour? You should not throw and error instead so an error is reported back to the user? Ie. `throw New-TerminatingError`

With 'everytime'  I meant that it will return false on all errors, even connection problems.

Comments from Reviewable

Review status: all files reviewed at latest revision, 9 unresolved discussions.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 108 [r5] (raw file):

function Test-TargetResource

A suggestion or and idea. Might not work for your intended behavior.
Seems like most of the code is the same as in the Get-method. Would it work to call the Get-method from the Test-method instead? Looks like you could save a few lines of code.
Also the Get-method isn't called from LCM (currently) so right now that method don't do anything (somebody correct me if I'm wrong). Which I think means that if you leverage the Get-method from Test method, you might be able to remove the GetFilePath parameter since it seems they do the same (that is "get current status" and Get returns current status, and Test returns true or false).

Comments from Reviewable

Review status: all files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 29 [r4] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> We're also not using the InModuleScope anymore for testing exported functions due to the fact that it can lead to conflicts.

LGTM

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 89 [r4] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> Since you're mocking these same commands for almost all of these It statements you can create a [Context](https://github.com/pester/Pester/wiki/Context) block (which is almost the same as a Describe block) and put all of the It statements that use that mocked function within that Context block and just declare the mock once at the top of that Context block

LGTM

Comments from Reviewable

Reviewed 1 of 1 files at r5.
Review status: all files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

Comments from Reviewable

Review status: 2 of 5 files reviewed at latest revision, 9 unresolved discussions.

README.md, line 278 [r3] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> Typo here - "this will not cause a the Test to return false."

Done.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 30 [r3] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> It could be changed to use the helper function `Import-SQLPSModule` (in the xSQLServerHelper module). Then we only have to change the logic of which module is loaded (the #91 issue) in one place. Is this doable?

Done.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 7 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> Can you change this to `[Parameter` (using capital P). If you can do this here and in all other places in the resource code?

Done.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 49 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> Can you change this to `[System.String] $ServerInstance` (white space between type and variable). The same for the rest too below.

Done.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 108 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> A suggestion or and idea. Might not work for your intended behavior.
>
> Seems like most of the code is the same as in the Get-method. Would it work to call the Get-method from the Test-method instead? Looks like you could save a few lines of code.
> Also the Get-method isn't called from LCM (currently) so right now that method don't do anything (somebody correct me if I'm wrong). Which I think means that if you leverage the Get-method from Test method, you might be able to remove the GetFilePath parameter since it seems they do the same (that is "get current status" and Get returns current status, and Test returns true or false).
>

Good point about Test and Get being similar. I broke it into a separate function that would also supports Set. GetFilePath is used to extend the regular use of Get-DscConfiguration to a SQL script.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> With 'everytime'  I meant that it will return false on all errors, even connection problems.

A connection problem is an interesting case, but what you describe is the intended behavior. The SQL script at $TestFilePath will be run, if an exception is thrown or output generated then it is treated as a fail. For example, the Test SQL script may check for the existence of an account (or any config for that matter) and use raiserror() if it is not found. Then Set would run a Set SQL script to create the account. Get-DscConfiguration would run the SQL script at $GetFilePath and return any information about the account returned by the get SQL script.
This has been really helpful environments that have large SQL scripts that they have been using, but now want to add them to their DSC deployment.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 8 [r3] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> Take out comments from template - also the tests templates recently got updated to change these variables from global to script, so can you change both of these to be $script: variables?

Done.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 29 [r4] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> LGTM

Done.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 89 [r4] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> LGTM

Done.

Comments from Reviewable

Reviewed 1 of 3 files at r6.
Review status: 3 of 5 files reviewed at latest revision, 7 unresolved discussions.

README.md, line 334 [r6] (raw file):

### Unreleased
* Added resources
  - xSQLServerScript

Script or replication? It should be the same across the board

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 2 [r6] (raw file):

<#
.Synopsis

.SYNOPSIS should be in all caps - also can you indent the contents of the block comment?

Comments from Reviewable

Review status: 3 of 5 files reviewed at latest revision, 7 unresolved discussions.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 30 [r3] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> Done.

LGTM

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 7 [r5] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> Done.

LGTM

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 49 [r5] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> Done.

LGTM

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 108 [r5] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> Good point about Test and Get being similar. I broke it into a separate function that would also supports Set. GetFilePath is used to extend the regular use of Get-DscConfiguration to a SQL script.

LGTM

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> A connection problem is an interesting case, but what you describe is the intended behavior. The SQL script at $TestFilePath will be run, if an exception is thrown or output generated then it is treated as a fail. For example, the Test SQL script may check for the existence of an account (or any config for that matter) and use raiserror() if it is not found. Then Set would run a Set SQL script to create the account. Get-DscConfiguration would run the SQL script at $GetFilePath and return any information about the account returned by the get SQL script.
>
> This has been really helpful environments that have large SQL scripts that they have been using, but now want to add them to their DSC deployment.
>

Invoke-Sqlcmd returns `SqlPowerShellSqlExecutionException` type using `$_.Exception.GetType().Name` when using both `RAISEERROR()` and `THROW`. Better approach would be using that as the type for the catch-statement, if possible.
Could you check for that exception, and return $false only when that exception is thrown by Invoke-SqlCmd, and re-throw any other exception to the user? Would that improve error handling and still making the resource work as you intended?

Comments from Reviewable

Review status: 2 of 5 files reviewed at latest revision, 4 unresolved discussions.

README.md, line 334 [r6] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> Script or replication? It should be the same across the board

Script. It got missed in the merge. Should be correct now.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):
Previously, johlju (Johan Ljunggren) wrote…
> Invoke-Sqlcmd returns `SqlPowerShellSqlExecutionException` type using `$_.Exception.GetType().Name` when using both `RAISEERROR()` and `THROW`. Better approach would be using that as the type for the catch-statement, if possible.
>
> Could you check for that exception, and return $false only when that exception is thrown by Invoke-SqlCmd, and re-throw any other exception to the user? Would that improve error handling and still making the resource work as you intended?

Awesome suggestion! Done.

Tests/Unit/MSFT_xSQLServerScript.Test.ps1, line 2 [r6] (raw file):
Previously, mbreakey3 (Mariah) wrote…
> .SYNOPSIS should be in all caps - also can you indent the contents of the block comment?

Done.

Comments from Reviewable

Reviewed 3 of 3 files at r7.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

DSCResources/MSFT_xSQLServerScript/MSFT_xSQLServerScript.psm1, line 167 [r5] (raw file):
Previously, dcrreynolds (David Reynolds) wrote…
> Awesome suggestion! Done.

LGTM

Comments from Reviewable

Reviewed 1 of 3 files at r6, 3 of 3 files at r7.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


Are fee schedule integer sizes big enough?

Right now we have a mix of uint32_t and uint64_ts for the various fees in fee_schedule_type.  They're all large enough to hold their default values, but some of the uint32_t types don't leave us a lot of room to adjust up in the future.  For example, asset_create_fee is<PHONE_NUMBER>.  We can only increase that by ~40% before overflowing, so if we decide it needs to be more expensive or if CORE drops against USD by that amount, we'll have problems.
I assume this structure will only be transmitted on the network infrequently with global_parameters_update_operation.  Can we just bite it on a few hundred bytes and make 'em all uint64_t?
https://github.com/cryptonomex/graphene/blob/master/libraries/chain/include/graphene/chain/types.hpp#L350-L397

Personally, I think it makes a lot of sense to just make all fee values into uint64_t and be done with, because as you said these should be updated very infrequently, as long as you also implement a scaling parameter (see https://github.com/cryptonomex/graphene/issues/66#issuecomment-114676452) on all the fee values where the scaling parameter can be updated independently.
Another optimization that could be done (although likely not worth it) is to have a bit field defining which of the parameters in the global parameters (which would require a specified sequence in their definition) are being updated in the global_parameters_update_operation and only include the update values in the transaction. That means the number of uint64_t values included in the transaction would be equal to the number of 1 bits in the accompanying bit field.

The real reason was the minimize the size of the "largest operation" because every operation reserves space for the "largest" one.   Right now that space is being largely dominated by this fee structure.
With a little work we could dynamically allocate the fee schedule in the operation and bypass this concern.

Fee types all increased to uint64_t.  Dynamic allocation is a separate discussion, let's continue it in a new ticket (#109).


feat!: @waku/discovery

Problem
We currently have 3 discovery options available within Waku:

DNS Discovery
Peer Exchange
Local Peer Cache (recently added: #1811

For this, we have 3 separate packages, which might only increase with time.
Solution
Create a generalised package @waku/discovery to encapsulate all the available discovery mechanisms.
Notes

Resolves https://github.com/waku-org/js-waku/issues/1862

left some comments

Thanks for the comments! Addressed :)


upgrade to llvm 18.1.5

Description
LLVM 18.1.5 support

Upgrades LLVM to 18.1.5 in setup scripts
process_bpf_source_file.cmake reduce optimization to -O1 for test bpf programs

see this note in #1111

regenerate expected bpf output programs in tests/bpf2c_tests/expected/

Closes #1111
Testing
Existing tests
Documentation
Updated LLVM links to version 18.1.5
Installation
Now expects LLVM 18.1.5 to be installed instead of 11 (and setup scripts now install 18.1.5 from choco)

What is the delta in compiled program size (.o, .dll, and .sys) compared to llvm-11 with -O2?
What is the delta in CPU performance with the bpf_performance runs?
I don't know yet whether this is a win or not, since my intuition is that llvm-11 with -O2 might be better for CPU bound workloads, whereas llvm-18 with -O1 might be better only for features that are cpu=v4.   I'd want CI/CD to test both llvm-18 and llvm-11.

Closing as we switched to the version of LLVM that ships with Microsoft Visual Studio for supply chain protection.


SSL: CERTIFICATE_VERIFY_FAILED: Self signed certificate in certificate chain _ssl.c.1106

Hi Team,
We have installed splunk/splunk-add-on-microsoft-azure latest version 4.0.3 on Splunk HF. We are getting below certificate error. We use company provided internal ROOT CA certificates. We faced similar issue with MS Cloud Services Add-on but resolved when appended our root certificates under $SPLUNK_HOME/etc/apps/<TA_NAME>/lib/certifi/cacert.pem. But not working this time. Splunk version is 9.0.6
Looks like this TA looks other path for ssl certificate. Could you please share the exact path were we should append our internal certificates.
Error as below:

message from "$SPLUNK_HOME/etc/apps/TA-MS-AAD/bin/MS_AAD_signins.py"  HTTPSConnectionPool(host='login.microsoftonline.com', port=443) Max retries exceed with url:/<tenant_id>/oauth2/v2.0/token (caused by sslerror(SSL: CERTIFICATE_VERIFY_FAILED: Self signed certificate in certificate chain _ssl.c.1106))

In $SPLUNK_HOME/etc/apps/TA-MS-AAD/lib/certifi add your root certificate to cacert.pem


[RESOLVED] Unavailable by ICMP ping

Problem started at 11:53:14 on 2023.02.09
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Operational data: Down (0)
Original problem ID: 4836
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23130&eventid=4836

Problem has been resolved in 3m 0s at 11:56:14 on 2023.02.09
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Original problem ID: 4836


Error manager

this is only sugest (because I have lot of useless constants) how is posible solve errors
(again you are free, I am only share ideas with you ...)

Thanks!
This is a lot of content, I can't check everything, but I'm fine to merge on the new branch I created (I just swapped the target of your PR: https://github.com/Zyko0/go-opencl/tree/feature/constants-paths-definitions)
Then I'll refactor it a bit!
Tell me when you're okay to merge this PR on the branch!

Thanks!
This is a lot of content, I can't check everything, but I'm fine to merge on the new branch I created (I just swapped the target of your PR: https://github.com/Zyko0/go-opencl/tree/feature/constants-paths-definitions) Then I'll refactor it a bit!
Tell me when you're okay to merge this PR on the branch!

do not hurry, I will not remove it ..., I will finish my fork (of blackcl), I hope I can use some your ideas (I will mention you as inspiration, ok?)

if I can ask, I see a lot of uint, would it be uint32?
And I have biggest sugest, I create lowest level (from your) https://github.com/MatejMagat305/purego-opencl/tree/master/pure, could I create package "midle" with your wrapper method, and package "high" with my high level wrapper? Then I would add you as second owner, and we would provide together?

if I can ask, I see a lot of uint, would it be uint32?

No particular reason, except sometimes one or the other would give me an error, so I figured some of these by trial and error!

could I fill package "midle" with your wrapper method(your fork, but calling "pure" package"?

Of course, please fill free to use this base as you wish!

I would like add you as second owner, could we provide together?

I stopped working on openCL at the moment and have no active project using it anymore, so I kinda let it on the side for now.
I don't think I'll have the bandwidth to work on openCL more at the moment unfortunately!

if I can ask, I see a lot of uint, would it be uint32?

No particular reason, except sometimes one or the other would give me an error, so I figured some of these by trial and error!

could I fill package "midle" with your wrapper method(your fork, but calling "pure" package"?

Of course, please fill free to use this base as you wish!

I would like add you as second owner, could we provide together?

I stopped working on openCL at the moment and have no active project using it anymore, so I kinda let it on the side for now. I don't think I'll have the bandwidth to work on openCL more at the moment unfortunately!

ok, thank you


Update build-and-deploy-docs-workflow.yml

Bump GH action to fix warning

I've already fixed this in the PR I'm working on to fix up other issues with the workflow.


Dataset

Hi
I find the class data in CORD is unbalanced, the menu classes take up a lot, does this affect the result? Do you have any way to deal with this imbalance? I am very interested in this project. Thank you for sharing your knowledge

Hi @MinhQuang2021
You can use different weights for individual classes if you use cross-entropy loss.
However, I found it was not necessary at least for the CORD task.
You may check the Table 2 of this paper where the model is trained with 800 receipts.
Wonseok


Initial go-tuf-metadata client and CI

The following PR adds an initial go-tuf-metadata client CLI.
Note:
I've noticed that some of the flag names do not match what we had in the initial document but decided to stick to what we have now, and later we can update the tool along with the client CLIs.
Disclaimer:
There's also a simple CI config file plus a few make targets added with this PR. It's not a huge change so they should be easily reviewed altogether, but I completely agree if we decide to split it to separate PRs.

On the changes compared to the initial doc: Sorry, I did not modify the doc -- all of the changes have some kind of reason behind them, but it's been a couple of weeks so I'd have to look again to remind myself.
Now that we're adding some infrastructure I want to clearly document that the clients and the infra is here for two reasons:

testing the conformance test suite itself
quick-and-dirty test rig while we change things

and that the real setup for running the suite against an implementation should be that the implementation (like python-tuf)

maintains a client wrapper in their code base
installs the suite in CI and
runs the suite with their client wrapper in their CI

basically I want to avoid maintaining a list of go dependencies (and rust dependencies and npm dependencies) for just "test data" ... but I don't think this documentation  needs to be in this same PR.
I think I'll just merge this so we can enable the CI and see how it goes


update run-on-minikube doc

update run-on-minikube doc

thanks for the review @gongweibao, updated.


Contact Page Submit Button

positioning of submit button in contact page is not correct you can see in SS. kindly assign this issue to me @SurajPratap10 .

This should be like that in center

Assigned @9520prashant


Native notebook: can not copy cell output

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur

Use the editor for a while, open a side editor. Sorry I have not figured out the exact steps to reproduce

Actual behavior
I select the text in the output and press Ctlr+C, but the text does not get copied to clipboard.
Instead a Cannot read property 'contains' of undefined error pops up
Expected behavior
The text gets copied to clipboard  and can be pasted

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running: Remote

Extension version: v2020.10.296162911-dev

VS Code version: 1.50-insider

Python and/or Anaconda version: 3.7

OS: Local windows, remote ubuntu

Virtual environment: conda

Python Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer, @joyceerhl

Related to https://github.com/microsoft/vscode/issues/104095 ?

Being tracked by https://github.com/microsoft/vscode/issues/104095. Dupe.


"Logging to syslog" configuration docs

Docs for rabbitmq/rabbitmq-server#1600
Refactor logging page, add it to the TOC.

@hairyhum - do you think it would be useful to provide a minimal working configuration example for a syslog server like syslog-ng? The reason I bring it up is that, being new to syslog-ng, I spent quite a bit of time wondering why the network() source wasn't working when in fact I should have been using syslog(). I have an example config here that could be a starting point.

I would prefer to not have to document syslog on the rabbitmq website, especially because there are several syslog implementations. A link to a doc page with "how to enable network" would be nice if such doc page exist.

We will add some guidance around syslog-ng configuration later.


Test Windows and MacOS

I thought we had an issue, but can't find it...

Adds Windows and MacOS (arm and intel, I think, based on this list)

These are experimental, as we can't currently guarantee support/functionality, but this makes it easier to see and fix issues, and hopefully we can get to a point where they are all supported fully.
Currently there seem to be at least a couple that we could address soon:

Installing dgl for windows
Setting the correct device for chgnet (on MacOS or any OS?)

(I don't suggest fixing them as part of this PR)
Unfortunately I think even "allowed" failures lose the green tick, which is annoying.

Errors on latest MacOS should hopefully be fixed in the next CHGNet release (https://github.com/CederGroupHub/chgnet/issues/181).

The Windows issue seems to be due to the availability of built distributions on PyPI. The most recent releases are:
2.2.1, which only seems to include Windows built distributions
2.2.0, which only seems to include MacOS built distributions
2.1.0, which only seems to include linux and MacOS
2.0.0, seems to include built distributions for Windows, MacOS and linux
So a potential solution to ensure compatibility might be reverting to 2.0.0.
Alternatives include:

Follow their suggested installation, but this requires specifying the torch version, and will not work Winows/MacOS after 2.2.1:

pip install  dgl -f https://data.dgl.ai/wheels/torch-2.1/repo.html

Install from source, which requires different instructions for each OS

https://docs.dgl.ai/install/index.html#install-from-source

These will be tested on self-hosted runners e.g. via #334.


Window button visual effect as default

Can you implement window button visual effect.
Glopos has that effect, can you implement something like that in fluent_ui packages.
Glopos pub.dev: https://pub.dev/packages/glopos

This effect was removed in the latest version of win-ui, and was even removed from the docs.

Oh, i don't know that. However, i think it pretty cool 😁


Initialization error

Hello,
I'm facing an issue while initializing the editor. It seems icons.svg is not found
How to reproduce the bug?
I just followed the steps on homepage to integrate the plugin into my project but I got the following error in the javascript console:
GET https://ajax.googleapis.com/ajax/libs/jquery/2.1.3/ui/icons.svg 404
I don't understand because the file icons.svg is available within the folder of the plugin. Why trying to get it
Regards

My bad I didn't see the option : $.trumbowyg.svgPath = '/assets/my-custom-path/icons.svg';


bTrem Guidelines don't match schema

From<EMAIL_ADDRESS>on September 28, 2012 03:42:16
the slash attribute is not available on the bTrem element, though it's explained in the Guidelines
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=122

From<EMAIL_ADDRESS>on September 28, 2012 02:44:23
Status: Duplicate
Mergedinto: 106


(WIP) [SDP-1147] Store admin migrations & tables in the admin schema

What
Refactor the database migration-related code in preparation for SDP-1147, when admin migrations will stop being done in the public schema and start being done in the (new) admin schema. Here are the changes:

The information needed for migrations, such as the migrations table config, and file ystem path, were moved into their own package, at db/migrations/migration_router.go.

This allowed us to remove the code repetition between db/migrate.go and db/dbtest/dbtest.go.
It also triggered the changes in the method signature of multiple other methods.

The *.sql migration files stopped mentioning the public.* schema, unless needed.
Refactored tss_migrations.go into schema_migration_manager.go, to make things more generic and less tied to the TSS.

Also:

Refactored the tests in db/migrate_test.go, to reduce code repetition, since the tests were the same for each schema

Checklist
PR Structure

[x] This PR has a reasonably narrow scope (if not, break it down into smaller PRs).
[x] This PR title and description are clear enough for anyone to review it.
[x] This PR does not mix refactoring changes with feature changes (split into two PRs otherwise).

Thoroughness

[x] This PR adds tests for the new functionality or fixes.
[x] This PR contains the link to the Jira ticket it addresses.

Configs and Secrets

[x] No new CONFIG variables are required -OR- the new required ones were added to the helmchart's values.yaml file.
[x] No new CONFIG variables are required -OR- the new required ones were added to the deployments (pr-preview, dev, demo, prd).
[x] No new SECRETS variables are required -OR- the new required ones were mentioned in the helmchart's values.yaml file.
[x] No new SECRETS variables are required -OR- the new required ones were added to the deployments (pr-preview secrets, dev secrets, demo secrets, prd secrets).

Release

[x] This is not a breaking change.
[x] This is ready for production.. If your PR is not ready for production, please consider opening additional complementary PRs using this one as the base. Only merge this into develop or main after it's ready for production!

Deployment

[x] Does the deployment work after merging?

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com

stellar-disbursement-platform-backend-preview is available here:SDP:      https://sdp-backend-pr265.previews.kube001.services.stellar-ops.com/healthAP:       https://sdp-ap-pr265.previews.kube001.services.stellar-ops.com/healthFrontend: https://sdp-backend-dashboard-pr265.previews.kube001.services.stellar-ops.com


[FLINK-27645] Update overview / supported features page for 1.0.0

Restructured the feature summary
Added links for quick access
Added Known issues and limitations

cc @wangyang0918 @mbalassi

There are some missing pieces, but wanted to open it on for feedback, and other parts of the documentation is still moving, so links probably need to be corrected/added.

Merging now. Let's create new tickets if we get more feedbacks from users.


[Refactoring] Saved object and related event analytics refactoring

Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@mengweieric should this be from https://github.com/opensearch-project/dashboards-observability/tree/explorer-modular-and-hooks

@mengweieric should this be from https://github.com/opensearch-project/dashboards-observability/tree/explorer-modular-and-hooks

yes it should


Tag on merge whenever version in cmake changes.

Do you know if this will work @timkpaine ? I am not sure if the CI action has permission to do the tagging or if there would be some github action magic needed.

lgtm, I think whatever allows us to (safely) eliminate my silly git checks is best


Can not find the chinese version of vaildate

Chinese - https://github.com/chilijung/validate.js
the url can not be reach

@chilijung Did you move this repo somewhere?


[NUMBERS-38]: Added Junit test cases for LanczosApproximation

Coverage remained the same at 83.124% when pulling 9b0367e153c51804d5022051b6722e1419f1b016 on ameyjadiye:master into c2bfc4fbf8fe325eaf254c37128948b7644e5687 on apache:master.

Coverage remained the same at 83.124% when pulling 0701cbfe7492510dc4ffe9d083b89b403f236b42 on ameyjadiye:master into c2bfc4fbf8fe325eaf254c37128948b7644e5687 on apache:master.


Upgrade to v1.9.3 of the Linode Terraform Provider

Do you know why we get so many C# diffs for such a seemingly minor change? Is the new codegen somehow applied at the same time?

@mikhailshilkov the new CSharp codegen went out on 2.0 - I think these are maybe residual changes post 1.9.3 upgrades - I don't think it's anything to worry about - there would be WAY more otherwise


Create contributing doc

Need a CONTRIBUTING.md so that other people can help

Pretty sure this is done


Readme

The readme.md has two broken links at the bottom.

I think Stu means the readme under the subfolder ui-extensions-dev-toolkit not the readme at the top of this repo.
https://github.com/ibm-wch/sample-custom-element-ui/tree/master/ui-extensions-dev-toolkit

removed


Move JS binding code into a JSContext

This PR creates a new JSContext object which implements JS APIs like innerHTML and stores the dukpy object and the node/handle dictionaries. The main advantage is that the Tab class avoids growing super long.
Note: This PR requires the ch9-upgrade PR #193 be merged first.

Please rebase so I can review..


enable versioning for s3?

Photos are one of the few things we don't back up, so when they get deleted by users accidentally (or worse, by us accidentally due to a bug), there's just no way to recover them. I think we could change this by enabling versioning on S3, which would mean DELETE requests would not actually delete them from S3 (though I'm not sure what effect they would have, then). The downside is we apparently can't turn versioning off after we turn it on. I'm not sure if there's any impact on pricing aside from the fact that we never stop paying to host those archived versions. @pleary, can you think of any other reasons not to turn it on?

You can delete object versions and even define a lifecycle for them, you just can't delete them with the DELETE command. The effect of the DELETE command becomes to hide the object from the list of objects in the bucket.
See here: http://docs.aws.amazon.com/AmazonS3/latest/dev/DeletingObjectVersions.html

This too: http://docs.aws.amazon.com/AmazonS3/latest/dev/object-lifecycle-mgmt.html

We ultimately decided to ditch this in favor of #690, largely b/c there are a bunch of versions we don't want to keep in s3, like the original file upload and any changes to the resized pics (sm, med, large, etc), and managing that would involve a lot of version management.


Connection errors with on-premise SQL Server using Apple Silicon 14.0.1

Azure Data Studio Version: 14.0.1
OS Version: macOS 13.0.1

Steps to Reproduce:

Connect to an on-premise SQL Server
Run any query

Partial results are displayed into the grid and then the following error occurs
Started executing query at Line 1
(1000 rows affected)
Query failed: Invalid operation. The connection is closed.
Total execution time: 00:00:02.289

Does this issue occur when all extensions are disabled?: Yes

I'm seeing this as well in 1.40.1 and not in Insiders.

@dzsquared only reproduces on Mac, right?

@alanrenmsft I've only been on macOS in ADS recently, so difficult to say

@gbougiakas I was able to reproduce the issue and you can work around it by:

Delete the connections from Connections view
Restart Azure Data Studio

please let me know if you are still having issue.
@cheenamalhotra , assigning to you for investigation.

I figured out this issue is indeed fixed in Insiders build with PR: https://github.com/microsoft/azuredatastudio/pull/21206, specifically this change:
https://github.com/microsoft/azuredatastudio/pull/21206/files#diff-8e6e02b2056f8700c383eabfb6390f5906007f1c68e9ff9d857b88afad9cc1ceR971
Previously if 'expiresOn' property was present in connection properties, it wasn't checked whether the connection profile is using Azure MFA auth, and reconnect was issued, this led to dropping existing connection in use by query.
I'm looking further into avoiding adding this property for non-AzureMFA, as that seems to be happening in the Insiders build.


[develop] Conan

Change Description

Added full support for using Conan to manage dependencies when building EOSIO:

Buildkite pipeline functionality now matches pinned/unpinned builds. Supply USE_CONAN=true in Buildkite to run a build using Conan to manage dependencies.
You can also provide -DUSE_CONAN=true to cmake when building locally to have Conan manage dependencies.
Docs added in .conan directory to walkthrough steps of using Conan for each supported platform.

CI steps are dynamically generated from these documents.
Document added in scripts directory to detail build process and dependencies when not using Conan.

Restructured .cicd/platforms to support Conan images.

Created partial Dockerfiles used for Conan builds.
Separate all platforms into pinned, unpinned, and conan folder.
Updated all references to pinned IMAGE_TAGs to include -pinned as a suffix.

Updated .cicd/generate-base-images.sh to support use in Anka tag creation and generating Conan images.

Anka tag generation now calls this script instead of the Mac scripts directly.
Dynamically append to Conan Dockerfiles and Mac scripts based on Environment Steps section of Conan documentation per platform.

Updated .cicd/build.sh to support Conan.

Dynamically generate build steps for Conan based on Build Steps section of Conan documentation.

Updated .cicd/test.sh to support Conan.

Dynamically write the Dockerfile at this step again to ensure we grab the correct Docker image for testing Conan. Otherwise, the image would not exist.

Updated .cicd/package.sh to support new platforms structure.
Updated .travis.yml to support new IMAGE_TAG and PLATFORM_TYPE variables.

Other changes:

Removed latest tag in Contracts Builder. This would flip-flop between being a image based off different branches and was unreliable.
Updated scripts/serial-test.sh and scripts/long-running-test.sh to not start mongod if it does not exist.

Added bonus of no longer needing to install Mongo dependencies during Mac builds on Travis, speeding that build time up.

Updated Ubuntu 18 unpinned to use Clang-7 to match AmazonLinux2 unpinned.

See:
https://buildkite.com/EOSIO/eosio/builds/18227 | Pinned
https://buildkite.com/EOSIO/eosio-build-unpinned/builds/760 | Unpinned
https://buildkite.com/EOSIO/eosio-conan/builds/413 | Conan build with LRTs.
https://travis-ci.com/EOSIO/eos/builds/132238360 | Travis build (Mac failures are known.)
Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

With #8118 merged, we can remove the gettext dep in here now

I attempted to follow the instructions for a Ubuntu 18.04 build and failed twice in two different ways. On my first attempt (that was in a container that I've used previously) I would get an error when trying to compile gmp:
checking C++ compiler g++ -DNDEBUG -D_GLIBCXX_USE_CXX11_ABI=1 -m64 -O3 -fPIC -stdlib=libstdc++... no
configure: error: C++ compiler not available, see config.log for details
gmp/6.1.2@bincrafters/stable:
WARN: libcurl/7.66.0@bincrafters/stable: requirement zlib/1.2.11 overridden by your conanfile to zlib/1.2.11@conan/stable
WARN: openssl/1.1.1d: requirement zlib/1.2.11 overridden by libcurl/7.66.0@bincrafters/stable to zlib/1.2.11@conan/stable
gmp/6.1.2@bincrafters/stable: ERROR: Package '244633fabf6cb8b674d1d43097294d0f40cc421a' build failed
gmp/6.1.2@bincrafters/stable: WARN: Build folder /home/ubuntu/.conan/data/gmp/6.1.2/bincrafters/stable/build/244633fabf6cb8b674d1d43097294d0f40cc421a
ERROR: gmp/6.1.2@bincrafters/stable: Error in build() method, line 66
        autotools = self._configure_autotools()
while calling '_configure_autotools', line 62
        self._autotools.configure(args=configure_args, configure_dir=self._source_subfolder)
        ConanException: Error 1 while executing source_subfolder/configure '--disable-assembly' '--enable-shared' '--disable-static' '--enable-cxx' '--prefix=/home/ubuntu/.conan/data/gmp/6.1.2/bincraf
ters/stable/package/244633fabf6cb8b674d1d43097294d0f40cc421a' '--bindir=${prefix}/bin' '--sbindir=${prefix}/bin' '--libexecdir=${prefix}/bin' '--libdir=${prefix}/lib' '--includedir=${prefix}/include'
'--oldincludedir=${prefix}/include' '--datarootdir=${prefix}/share'
CMake Error at .conan/conan.cmake:399 (message):
  Conan install failed='1'
Call Stack (most recent call first):
  .conan/conan.cmake:485 (conan_cmake_install)
  CMakeLists.txt:52 (conan_cmake_run)

Then I started from a fresh container and performed the following
# just update everything to the latest first
sudo apt-get update
sudo apt-get upgrade
# now follow the instructions (more or less; the instructions don't say you need to run some stuff as root, or that installing cmake to /usr/local/cmake needs to have it now in its path)
$ sudo apt-get install -y clang-7 llvm-7-dev python3 python3-dev python3-pip git curl automake jq python2.7 python2.7-dev
$ sudo pip3 install conan
$ curl -LO https://github.com/Kitware/CMake/releases/download/v3.15.3/cmake-3.15.3-Linux-x86_64.sh
$ sudo mkdir -p /usr/local/cmake
$ chmod a+x cmake-3.15.3-Linux-x86_64.sh
$ sudo ./cmake-3.15.3-Linux-x86_64.sh --prefix=/usr/local/cmake --skip-license
$ PATH=/usr/local/cmake/bin/:$PATH
$ git clone -b conan-poc-v2 --recursive https://github.com/EOSIO/eos
$ cd eos
$ cmake -DCMAKE_BUILD_TYPE='Release' -DPKG_CONFIG_USE_CMAKE_PREFIX_PATH=ON -DCMAKE_CXX_COMPILER='clang++-7' -DCMAKE_C_COMPILER='clang-7' -DLLVM_DIR='/usr/lib/llvm-7/lib/cmake/llvm' -DUSE_CONAN=true -Bbuild

But this failed with
- Conan: checking conan executable in path
-- Conan: Found program /usr/local/bin/conan
-- Conan: Version found Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 574, in _build_master
    ws.require(__requires__)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 892, in require
    needed = self.resolve(parse_requirements(requirements))
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 783, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (six 1.11.0 (/usr/lib/python3/dist-packages), Requirement.parse('six==1.12'), {'astroid'})

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/bin/conan", line 6, in <module>
    from pkg_resources import load_entry_point
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3088, in <module>
    @_call_aside
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3072, in _call_aside
    f(*args, **kwargs)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 3101, in _initialize_master_working_set
    working_set = WorkingSet._build_master()
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 576, in _build_master
    return cls._build_from_requirements(__requires__)
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 589, in _build_from_requirements
    dists = ws.resolve(reqs, Environment())
  File "/usr/lib/python3/dist-packages/pkg_resources/__init__.py", line 783, in resolve
    raise VersionConflict(dist, req).with_context(dependent_req)
pkg_resources.ContextualVersionConflict: (six 1.11.0 (/usr/lib/python3/dist-packages), Requirement.parse('six==1.12'), {'astroid'})

CMake Error at .conan/conan.cmake:530 (if):
  if given arguments:

    "VERSION_LESS" "1.0.0"

  Unknown arguments specified
Call Stack (most recent call first):
  CMakeLists.txt:41 (conan_check)

Thanks for the feedback @spoonincode . I'll investigate this more.
As an update, I'm splitting this PR into 2 smaller PRs; one for the Conan specific changes and the second for the supporting CI changes.


Failed to fetch error in Next during static build or when Hono server is offline

Description:
When building a fully static app with Next.js that uses authClient from better-auth, I encounter a TypeError: Failed to fetch during the static build process or when the Hono server (which provides the API) is offline. This error occurs despite wrapping calls in try/catch, which does not seem to handle the failure gracefully.
Error Message:
Uncaught (in promise) TypeError: Failed to fetch

Relevant Code:
The following code initializes authClient with the Hono server URL from the environment variable NEXT_PUBLIC_API_URL:
import { createAuthClient } from 'better-auth/react'

export const authClient = createAuthClient({
  baseURL: process.env.NEXT_PUBLIC_API_URL,
})

The error occurs when using the authClient for example here in a custom hook.
import { authClient } from '@/lib'

export const useSession = () => {
  const {
    data: session,
    isPending,
    error,
  } = authClient.useSession()

  return {
    session,
    isPending,
    error,
  }
}

Expected Behavior:

During static builds or when the API server is offline, the code should handle the failed network request gracefully without breaking the build or causing unhandled promise rejections.

Steps to Reproduce:

Set up a static Next.js project with authClient from better-auth pointing to a Hono API.
Build the project (next build) or run it with the API server offline.
Observe the TypeError: Failed to fetch error.

Suggested Solution:
An option to handle or bypass network requests during authClient initialization could prevent breaking the build when the API is offline. Alternatively, if authClient could detect and handle offline scenarios gracefully, that would resolve the issue.
Or it would be even better if it wouldn't be hard throwing an error when the API is offline.

+1

+1

The "fetch failed" error occurs when fetch is called with an invalid URL during the build process. This typically happens if you attempt to statically build a page that requires fetching session. Since the session isn't available at build time (since there is no server and no user session), the build process fails.
To fix this, refactor your app so that pages requiring a user session are not statically built. Instead, make sure the request is fetched properly at runtime.

+1


Need to switch over to php

Php is a much faster language and is not a microframework. Because of this your should move over to php.

not needed


add functionality  for supporting new native web api s

🔍 Search Terms
add functionality  for supporting new native web api
✅ Viability Checklist

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
[x] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
[x] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

⭐ Suggestion
add functionality for supporting new native web api i get error when i try using new native web apis such as cookieStore navigation and etc it is so anoyying always geting errors there are planty native web api in mdn web api section
📃 Motivating Example
document.cookieStore.setCookie("my cookie","the path","expire date");
💻 Use Cases
document.cookieStore.setCookie("my cookie","the path","expire date");

Duplicate of #47314.

Not a duplicate of that issue; OP is asking for a general way of using a new Web API without waiting for TS implementing it. CookieStore is only an example.
However TS already allows you to use new APIs. You can define relevant types by yourself.

OP is asking for a general way of using a new Web API without waiting for TS implementing it.

Then it's just: https://www.typescriptlang.org/docs/handbook/2/type-declarations.html

i think those api that are mentioned in mdn and are not exprimenatl  should be implemened


log.format flag missing since 0.10.0

What did you do?
Tried to upgrade to 0.10.0
What did you expect to see?
Alertmanager to run
What did you see instead? Under which circumstances?
Alertmanager failed to start with the error
flag provided but not defined: -log.format
Environment

System information:
I confirmed this on two platforms:

Linux 3.10.0-514.26.1.el7.x86_64 x86_64
Darwin 16.7.0 x86_64

Alertmanager version:

alertmanager, version 0.11.0 (branch: HEAD, revision: 30dd0426c08b6479d9a26259ea5efd63bc1ee273)
build user:       root@3e103e3fc918
build date:       20171116-17:43:56
go version:       go1.9.2

Prometheus version:
NA

Alertmanager configuration file:
I have verified that log.format does not show up in the output of -h starting in version 0.10.0. This happens with or without a configuration file.

Prometheus configuration file:
NA

Logs:
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: flag provided but not defined: -log.format
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: Usage of /usr/local/heimdall/alertmanager/alertmanager:
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: -config.file string
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: Alertmanager configuration file name. (default "alertmanager.ym
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: -data.retention duration
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: How long to keep data for. (default 120h0m0s)
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: -log.level value
Dec 04 10:16:27 default-base-rhel7.vagrantup.com alertmanager[15205]: Only log messages with the given severity or above. One of: [de

I checked the output of -h back until I found it again.
$ ./alertmanager-0.11.0.darwin-amd64/alertmanager -h
Usage of ./alertmanager-0.11.0.darwin-amd64/alertmanager:
-config.file string
Alertmanager configuration file name. (default "alertmanager.yml")
-data.retention duration
How long to keep data for. (default 120h0m0s)
-log.level value
Only log messages with the given severity or above. One of: [debug, info, warn, error] (default info)

$ ./alertmanager -h
Usage of ./alertmanager:
-config.file string
Alertmanager configuration file name. (default "alertmanager.yml")
-data.retention duration
How long to keep data for. (default 120h0m0s)
-log.level value
Only log messages with the given severity or above. One of: [debug, info, warn, error] (default info)

$ ./alertmanager-0.9.1.darwin-amd64/alertmanager -h
Usage of ./alertmanager-0.9.1.darwin-amd64/alertmanager:
-config.file string
Alertmanager configuration file name. (default "alertmanager.yml")
-data.retention duration
How long to keep data for. (default 120h0m0s)
-log.format value
Set the log target and format. Example: "logger:syslog?appname=bob&local=7" or "logger:stdout?json=true" (default "logger:stderr")
-log.level value
Only log messages with the given severity or above. Valid levels: [debug, info, warn, error, fatal] (default "info")

This was lost in the migration to go-kit/log. @juliusv does it make sense to add back? or is there another way to configure it? Prometheus is not supporting the flag as well (after the migration)

The removal was intentional. Can you explain why you want this?

Because I prefer to be able to deal with text based logs without having to redirect stdout/stderr in a service or use journalctl and I didn't realize it was intentional (I didn't see anything in the release notes about it. Please let me know if there was and I was looking in the wrong place.)
Digging into journald a bit it looks like I can get everything I want with that system so I agree with ditching it. Closing this issue.


Is on:* working?

I have seen this code here:
  <button
    use:ripple
    class={classes}
    {...props}
    {type}
    {disabled}
    on:click={() => (value = !value)}
    on:click
    on:mouseover
    on:* <!-- this is what intrigued me -->
  >
    {#if icon}
      <Icon class={iClasses} {small}>{icon}</Icon>
    {/if}
    <slot />
  </button>

especially this is what intrigued me:
on:*

I tried in my Svelte project and it doesn't work.

Button.svelte:

<button on:*>Click me please</button>

App.svelte

<Button on:click={()=>console.log("Clicked")} />

Why is this working there?

on:click is passed up explicitly. I don't think Svelte can have a proper generic UI library without on:* but it takes very long time for features like this to be considered: https://github.com/sveltejs/svelte/pull/3349.

Ok, so it doesn't work, like I thought.
Thanks for your work!


Resolve: Run generated migration in MultiDbSecondaryMigrationTest and other pertinent test file #6

Fixes https://github.com/php-openapi/yii2-openapi/issues/6

Thank you!


Update intercompany-how-manage-intercompany-inbox.md

Updated to reflect current state of Intercompany Auto. Accept feature as decided RFH #283088

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/intercompany-how-manage-intercompany-inbox.md
:white_check_mark:Succeeded

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-nav-app/intercompany-how-manage-intercompany-inbox.md
:white_check_mark:Succeeded

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


[Critical Issue] Mistake in README file which can mislead a beginner

While reading I found that there was a question under OOPS section that
Can you list 8 primitive types in java?
Present ans:
byte
short
int
long
float
double
char
String
boolean
Expected ans:
byte
short
int
long
float
double
char
boolean
String is not a primitive data type of Java.
This point may mislead a beginner.

Updated


S3 Content-Length-Range for Pre-Signed Requests

For S3, is there currently any way in Rusoto to pre-sign a POST request and enforce a limit on file size? I'm generating a pre-signed PUT request using:
let req = PutObjectRequest {
    bucket: bucket.to_owned(),
    key: key.to_owned(),
    ..Default::default()
};

let request_option = PreSignedRequestOption {
    expires_in: ::std::time::Duration::from_secs(60 * 10),
};

let presigned_url = req.get_presigned_url(
    &region,
    &credentials.credentials().wait().unwrap(),
    &request_option,
);

println!("{}", presigned_url);

This works, however, I can't find a way to set the Fields and Conditions (specifically Content-Length-Range).
Basically, what I'm after in Python using Boto3:
presigned_url = generate_presigned_post(
    Bucket=bucket,
    Key=key,
    ExpiresIn=expires_in,
    Fields={
        'Content-Type': content_type,
    },
    Conditions=[
        ["content-length-range", 1, 5 * 1024 * 1]
    ],
)

The returned presigned_url is a dict containing the actual URL to POST to and a bunch of fields that need to be included in the request along with the actual payload/file.

Rusoto doesn't currently support that behavior. 😢
It should be added in https://github.com/rusoto/rusoto/blob/master/rusoto/services/s3/src/custom/util.rs where the existing presigned request code is at. 👍
I'd love a PR to get this addressed!

I gave it a shot trying to get a prototype working. Unfortunately, I've been largely unsuccessful in generating a correctly signed URL (AWS responds by just terminating the connection, so the signature is clearly invalid).
A few of my observations along the way (please let me know where I'm wrong):

Regarding the request struct, PostObjectRequest should be introduced. This would have to implemented in the generator part of Rusoto. However, I suspect that this struct would have the same layout like PutObjectRequest.

For PostObjectRequest, get_presigned_url would have to be implemented. Again, this should be largely the same like the impl for PutObjectRequest. Maybe things could be refactored here to avoid code duplication. The possibility needs to be added to pass any user defined fields and conditions.

Headers/fields/policies would all be encoded as part of the payload instead of putting them into the URL, so the resulting URL should simply resemble https://s3.foobar.com/bucket/file. request.generate_presigned_url would have to be modified to accommodate this.

Signing of the payload is already supported, however, I'm unsure to what the format should be (I can't seem to find the proper docs for this). After checking the Python implementation, the body seems to be Base64 encoded JSON containing headers and fields. After some testing, the bare minimum the payload should include seems to be the fields expiration, bucket, conditions and custom fields (if any).

After the payload of relevant headers and fields has been signed by request.generate_presigned_url, a value should be returned that specifies the URL and a list of all fields that are part of the signature (instead of just the URL). These fields need to be then included as the key/value body of the POST request that is used to upload the file, while the file itself is part of its file list.

I intended to use Rusoto in a project as an excuse to slowly inch my way back into Rust, but working on a proper implementation of a PR for this is unfortunately above my current knowledge of Rust (but I'm keen to learn if anyone is willing to work on this with me :smile: )

Hey @SilentByte did you ever figure this one out? I'm having this same issue and would really like a solution. Maybe we can collaborate on it? I don't know much Rust either but two heads are better than one?

I haven't made any more progress on this, unfortunately, and I'm not quite sure what else to try. :disappointed: If you've got any ideas, let me know!

Okay, I'm going to try and make a reddit thread on /r/rust to see if anyone else has any ideas.

Since I had to solve this same issue, I opened a PR with a possible solution


Support for extended_entities. Includes multiple photos and videos.

https://dev.twitter.com/overview/api/entities-in-twitter-objects#extended_entities
I can't seem to access extended_entities from Twython REST responses.

Which calls are you looking for the extended entities for? That documentation is just explaining what's in an entity/what kind of entities can be returned.
https://dev.twitter.com/rest/reference/get/statuses/home_timeline in that api call, it shows the entities are extended automatically.
The only time entities won't be included is if include_entities is sent to False

@michaelhelmick I am hitting the search endpoint: https://dev.twitter.com/rest/reference/get/search/tweets.
To reproduce the exact steps:
twitter = Twython(
         settings.TWITTER_CONSUMER_KEY,
         settings.TWITTER_CONSUMER_SECRET,
         settings.OAUTH_TOKEN,
         settings.OAUTH_TOKEN_SECRET,
     )

response = twitter.search(q="#twittervideo", count=100, result_type='recent')

One of the tweets in that response is: https://twitter.com/AmedArosemena/status/638056188777775105
The JSON returned in the Twython response for that tweet is:
{
    "contributors": null,
    "truncated": false,
    "text": "#TwitterVideo  #30segundos Tema #PazEmocional #domingo http://t.co/faoWeC4pXx",
    "is_quote_status": false,
    "in_reply_to_status_id": null,
    "id":<PHONE_NUMBER>76060161,
    "favorite_count": 0,
    "source": "Twitter for Android",
    "retweeted": false,
    "coordinates": null,
    "entities": {
        "symbols": [],
        "user_mentions": [],
        "hashtags": [
            {
                "indices": [
                    0,
                    13
                ],
                "text": "TwitterVideo"
            },
            {
                "indices": [
                    15,
                    26
                ],
                "text": "30segundos"
            },
            {
                "indices": [
                    32,
                    45
                ],
                "text": "PazEmocional"
            },
            {
                "indices": [
                    46,
                    54
                ],
                "text": "domingo"
            }
        ],
        "urls": [],
        "media": [
            {
                "source_user_id": 37500362,
                "source_status_id_str": "637998362646544384",
                "expanded_url": "http://twitter.com/AmedArosemena/status/637998362646544384/video/1",
                "display_url": "pic.twitter.com/faoWeC4pXx",
                "url": "http://t.co/faoWeC4pXx",
                "media_url_https": "https://pbs.twimg.com/ext_tw_video_thumb/637998127094431747/pu/img/NbFCuetHmMmRiIAG.jpg",
                "source_user_id_str": "37500362",
                "source_status_id":<PHONE_NUMBER>46544384,
                "id_str": "637998127094431747",
                "sizes": {
                    "small": {
                        "h": 604,
                        "resize": "fit",
                        "w": 340
                    },
                    "large": {
                        "h": 1280,
                        "resize": "fit",
                        "w": 720
                    },
                    "medium": {
                        "h": 1067,
                        "resize": "fit",
                        "w": 600
                    },
                    "thumb": {
                        "h": 150,
                        "resize": "crop",
                        "w": 150
                    }
                },
                "indices": [
                    55,
                    77
                ],
                "type": "photo",
                "id":<PHONE_NUMBER>94431747,
                "media_url": "http://pbs.twimg.com/ext_tw_video_thumb/637998127094431747/pu/img/NbFCuetHmMmRiIAG.jpg"
            }
        ]
    },
    "in_reply_to_screen_name": null,
    "in_reply_to_user_id": null,
    "retweet_count": 0,
    "id_str": "638072857076060161",
    "favorited": false,
    "user": {
        "follow_request_sent": null,
        "has_extended_profile": false,
        "profile_use_background_image": true,
        "default_profile_image": false,
        "id": 37500362,
        "profile_background_image_url_https": "https://abs.twimg.com/images/themes/theme1/bg.png",
        "verified": false,
        "profile_text_color": "333333",
        "profile_image_url_https": "https://pbs.twimg.com/profile_images/627619751137804288/QhNwlE17_normal.jpg",
        "profile_sidebar_fill_color": "DDEEF6",
        "entities": {
            "url": {
                "urls": [
                    {
                        "url": "http://t.co/5Q99Nqbgku",
                        "indices": [
                            0,
                            22
                        ],
                        "expanded_url": "http://www.amedarosemena.blogspot.com",
                        "display_url": "amedarosemena.blogspot.com"
                    }
                ]
            },
            "description": {
                "urls": []
            }
        },
        "followers_count": 13402,
        "profile_sidebar_border_color": "C0DEED",
        "id_str": "37500362",
        "profile_background_color": "C0DEED",
        "listed_count": 59,
        "is_translation_enabled": false,
        "utc_offset": -18000,
        "statuses_count": 48616,
        "description": "#PANAMAaLas12  Lunes a Viernes de 12md a  1pm por 104.7 F.M.\n\n#Instagram y #Periscope  @amedarosemena",
        "friends_count": 8890,
        "location": "Panam\u00e1",
        "profile_link_color": "0084B4",
        "profile_image_url": "http://pbs.twimg.com/profile_images/627619751137804288/QhNwlE17_normal.jpg",
        "following": null,
        "geo_enabled": true,
        "profile_banner_url": "https://pbs.twimg.com/profile_banners/37500362/1430253268",
        "profile_background_image_url": "http://abs.twimg.com/images/themes/theme1/bg.png",
        "screen_name": "AmedArosemena",
        "lang": "en",
        "profile_background_tile": false,
        "favourites_count": 580,
        "name": "AmedArosemena",
        "notifications": null,
        "url": "http://t.co/5Q99Nqbgku",
        "created_at": "Sun May 03 20:58:15 +0000 2009",
        "contributors_enabled": false,
        "time_zone": "Central Time (US & Canada)",
        "protected": false,
        "default_profile": true,
        "is_translator": false
    },
    "geo": null,
    "in_reply_to_user_id_str": null,
    "possibly_sensitive": false,
    "lang": "in",
    "created_at": "Sun Aug 30 19:36:38 +0000 2015",
    "in_reply_to_status_id_str": null,
    "place": null,
    "metadata": {
        "iso_language_code": "in",
        "result_type": "recent"
    }
}

Even though that tweet has a twitter video in it, the returned JSON does not have an extended_entities field which has the twitter video information. The 'media:type' still says 'photo' (https://dev.twitter.com/overview/api/entities-in-twitter-objects#media) and it gives the .jpg urls except on the expanded_url.

This might be a twitter issue. I've tried accessing the url via curl:
curl --get 'https://api.twitter.com/1.1/search/tweets.json' --data 'count=100&include_entities=true&q=%23+twittervideo&result_type=recent' --header 'Authorization: OAuth oauth_consumer_key="www", oauth_nonce="zzz", oauth_signature="yyy", oauth_signature_method="HMAC-SHA1", oauth_timestamp="1441380248", oauth_token="xxx", oauth_version="1.0"'

And got the same response, without video info (as described in the extended entities).
I'd file an issue with twitter on their developer forum and link them to this issue.
Since this isn't a Twython-specific issue, I'm going to close this. Good luck!

P.S. I tried with and without include_entities in the data.

Yep, seems like it has been an issue for a while now. https://twittercommunity.com/t/twitter-video-support-in-rest-and-streaming-api/31258/37?u=harisibrahimkv


bcftools: init at 1.3.1

Motivation for this change
Add bcftools to NixOS
Things done

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

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Could you remove the changes that are not related to bcftools from this patchset?

At least split them* up into separate commits in this PR, since they're too trivial to justify another.
*: Not sure if the whitespace fix is too trivial or not, I don't mind though.

Thanks a lot for all the advices @kragniz and @nckx. All fixed.
I have a little question.
HTSlib also should to be added for providing by itself bgzip, htsfile, and tabix utilities? And if the answer is yes, both samtools and bcftools linked to it?
Thanks again.

HTSlib also should to be added for providing by itself bgzip, htsfile, and tabix utilities? And if the answer is yes, both samtools and bcftools linked to it?

So {sam,bcf}tools currently build against a bundled copy of htslib? In that case, it would indeed be much nicer to to package htslib separately and use the --with-htslib to build the *tools against that instead of the bundled cruft.
One last detail: when adding multiple commits like this, make sure you keep everything working in between. As it is now, you first add a package referencing maintainers.mimadrid, but that variable is only added in the next commit. Evaluation in between the two commits is broken, making e.g. git bisect more difficult.

@nckx I think the same and I will link both with htslib as soon as possible.
I very sorry but I don't quite understand how the evaluation in between the two commits is broken by referencing the same variable and I don't want to repeat the error in the future.
Could You explain it in other way?
Thanks a lot.

 Of course you're right: there's no problem here.
Thanks for all the work so far.

Don't worry @nckx. I hope have some time before the weekend for:

Linking both to htslib
ncurses library optional for samtools because it is only required by the 'tview' subcommand and my mistake by adding it to bcftools also.

Regards

Hi, @nckx I think I got it but before upload all I have two question.

Where is the best place for htslib? applications/science/biology/... or  development/libraries/science/biology/...

Only installing samtools or bftools linked to htslib but without a explicit htslib installation I can't use bgzip, tabix and htsfile commands. Did I do something wrong?

Thanks in advance.

Where is the best place for htslib?

Definitely libraries, since it's mainly intended as a library. Plenty of libraries ship a handful of /bin tools (glibc has 22).

Only installing samtools or bftools linked to htslib but without a explicit htslib installation I can't use bgzip, tabix and htsfile commands. Did I do something wrong?

Not at all. This is how Nix works, and how it can work at all. :-)
On other distributions, installing a package immediately places all dependencies in your environment, e.g. installing something that depends on Qt might add qtconfig to your $PATH whether you want it or not. 97% of people don't.
In Nix & friends, a dependency can be installed without being added to your environment (‘profile’ in Nix). In fact, most aren't. For example:
~$ ls /nix/store/*/bin/qtconfig
/nix/store/47279xy8rywgjp04gyjhwa9qrdm4v9v8-qt-4.8.7/bin/qtconfig
/nix/store/4j367a0vj5qpqb19f4d32h448qmwqf11-qt-4.8.7/bin/qtconfig
/nix/store/74fq35wjnfsb618zfqlp8wsh8ji955ay-qt-4.8.7/bin/qtconfig
# ... much more output ...
~$ qtconfig
bash: qtconfig: command not found

Even though qt4's qtconfig is in my /nix/store (about 20 times), it's not in my profile. I'd have to explicitly run nix-env -iA qt4 to ‘see’ it.
Anyway: do samtools and/or bftools depend on any htslib binaries? If not, the current behaviour is probably what we want.

Thanks for all @nckx. I still am very newbie with NixOS so review all the changes with care.

I think it's ready now.

LGTM. Thank you.


How do you turn off debug logging

Almost every 2nd line of my stack traces is DEBUG: Chewy strategies stack: and I'd like to turn off debug logging, but I can't figure out how.

Chewy.logger.level = Logger::INFO

But probably it will change Rails.logger level as well, so maybe you have to assign another Chewy.logger instance

Yeah, that did change the Rails logger level as well. It would be nice if Chewy's logging could be set independent of Rails.logger.

That said, using the following:
Chewy.logger = Logger.new(STDOUT)
Chewy.logger.level = Logger::INFO

I'm able to get what I wanted.


AWS EC2 Warning

As per https://forums.aws.amazon.com/thread.jspa?messageID=723498, sending emails from an EC2 instance requires you to ask Amazon to set up rDNS (Reverse DNS) for the IP addresses sending email from the server which makes the addresses static and takes them off the DUL blocklist.
It might be worth having a provider-specific page in the Wiki for this as postal users will experience deliverability issues until they ask Amazon to carry out the above.
For completeness, both IP addresses on our EC2 instance are Elastic IP addresses.

In my experience Digital Ocean is far better than Amazon for Postal, cheaper and better.

I am actually a Digital Ocean customer and appreciate their awesomeness but in our corporate context, AWS is a much better fit and block list checking is a not publicised-enough factor when researching and becoming an email sender

Setting up RDNS is listed on the FAQ about message delivery. I don't really want to maintain a page of provider specific oddities. I'd hope a provider like Amazon would actually let you set your own PTR records for your IPs rather than needing to contact them.
https://github.com/atech/postal/wiki/FAQs#e-mails-sent-through-postal-are-going-to-spam


V0.0 Dash

Merge dev guide

@akd13
Some of the back-quotes ``` in the first column of the User Stories section are not inserted correctly.

@ChuaPingChan fixed back-quotes

Do you still need this PR? If not we can close it.

Closing since the other PR has been made.


Normalize polarization vectors

If a user specifies a polarization like [0,1,1], then the Rabi rate for this transition will be off by √2. Mistakes like that can be avoided if the program were to automatically normalize the polarization vectors, or report an error when a vector is not normalized.

added in ad1fb793c1147be48cfe49f60eaae97df7616929, now normalizes the polarization to length 1 by default with normalize_pol = True


[Feature][UI Next][V1.0.0-Alpha] Serial number/number text (title) display error

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
序号/编号文字展示错误

What you expected to happen
与老UI展示一致

How to reproduce
1、点击首页
Anything else
No response
Version
dev
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Thank you for your feedback. Will be processed later.

You can refer to this https://github.com/apache/dolphinscheduler/issues/8634 issue. In the new UI, all tables will be uniformly processed as #. Issues related to this do not need to be opened.


add CONTRIBUTING.md

Lets add contributing file so that we can work together.

Let's work together.

Let's do this


WIP: PERF: Overload SpatialObject IsInsideInObjectSpace without name argument

Added virtual IsInsideInObjectSpace(const PointType &) member function
to itk::SpatialObject, as an overload of the original one,
IsInsideInObjectSpace(const PointType&, unsigned depth, const std::string& name).
Removed the virtual keyword from the original member function.
Ensured that the 11 derived classes now override the new overload
(without the name parameter), instead of the original one. The
original member function now calls the new overload.
A major performance improvement was observed by this commit, especially
because it avoids many expensive GetTypeName().find(name) calls.
IsInsideInObjectSpace often appears more than twice (and sometimes
even more than 3 times) as fast as before.
Fixes issue #820 ("SpatialObject IsInsideInObjectSpace PERF penalty
GetTypeName().find")

Tip of the day: when reviewing this commit on github, consider Diff settings ->  Hide whitespace changes (Just for now)!
The most important changes are in itkSpatialObject.h  and itkSpatialObject.hxx
And yes, it's really a major performance improvement  :-)


Some clean-ups

Thanks to @christianhaller

Can't help myself but while I read this, instantly this article from an opera employee pops out of my brain ;-) http://coding.smashingmagazine.com/2011/11/11/our-pointless-pursuit-of-semantic-value/

Just read the reply article.

:)


additional consideration for minimal technical metadata is an "Is Public" field

          One additional consideration for minimal technical metadata is an "Is Public" field. Sources: JGI IMG/M &  PLaBA-db (requirement unknown). NME 30May2022.

Originally posted by @nernster in https://github.com/NFDI4Microbiota/MetadataStandards/issues/1#issuecomment-1140922652

We can add, options along the lines of : "Data available upon reasonable request", "Public", "Available upon request", "Ethics committee approval required" ?

I agree that we need more options. Maybe we can fall back on existing ontologies to coin terms?
E.g. the EGA/ GA4GH created a Data Use Ontology DUO_0000017.
DUO is an ontology which represent data use conditions. I think it was build for human-associated data, so we would need to double-check if the ontology terms fit our expectations, but the granularity of terms/ concepts is a plus IMO.


issue-132 - dedupe insertStyle in output files

Thanks for the work on this plugin!
Description
This PR aims to remove the  duplication of insertStyle function in each chunk / bundle produced by rollup

fixes #132
fixes #69

Proposed solution
When insert option is set to true,
instead of adding the code of insertStyle on the top of each file via the intro hook,
the insertStyle function is now included via an import which points to src/insertStyle.ts file.

[!NOTE]
Using the import, rollup will take care to include the function code only once per bundle (or output only one file if preserveModules function is used).

[!IMPORTANT]
Since rollup process ESModules the insertStyle file should not be compiled using CommonJS  modules but using ES6 modules.
Because of this I added 2 tsconfig.build files to produce the two different outputs.

[!TIP]
When migrating this package to ESM these two files can be safely removed.

Well done sir! (👏)  Good work !!
Writing some unit tests - unless you already are?

In order to test this scenario I updated 'should insert CSS into head tag' test to better match that insertStyle is included once.
In addition I also added a new test: 'should import *.scss and *.sass files with default configuration' to get rid of coverage decrease error.

[!NOTE]
Since we are basically testing strings, for the two tests involved in this PR, I opted out to use snapshot testing.
The snapshots are stored inside test/snapshots folder, let me know if I need to change their location.

Thank you for the good work @marcalexiei 👍 .


Bug: Response.body() returning null

According to this comment the Response.body() method may never return null.
As can be seen in this gist, there are cases in which this method can return null.
I think this comment either needs to be updated to be more accurate, or the functionality should be updated to match the claim made in the comment.

It’s an issue with the comment. It should really say “Never null on responses returned from servers”. It is possible to create instances of Response without providing a body, and we also use this in the cacheResponse and networkResponse members of Response.

@swankjesse Agreed, makes sense. Can i submit a PR to make that change to the comment or what is the process with milestones/releases?