Add quorum attribute to delivery_push_job

This pull adds a quorum attribute to delivery_push_job resource and associated classes. Quorum is the absolute number of nodes required in order for the job to run. This argument was not previously used when submitting jobs to the Chef server. I believe it had defaulted to 100%, the length of the nodes array. This behavior is preserved in this implementation.
This resource still assumes that every node must be successful in order for the job to be considered successful.
The next step after this would be to allow for quorum to be specified in delivery-truck as either a config.json or node attribute.

Functionality looks great! I'm pushing this change through our Automate server to verify this change. I'll keep y'all posted!  🎉

Tests passed!

@mattlqx Could you take care of the comments above and also also squash your commits on a single one? (one of them seems a duplicate and it is not signed off)

Thanks guys!

Woow that was quick!

Change got approved! This will get sync with github as soon as we demo this change so it gets delivered... 👍

Closing in favor of https://github.com/chef-cookbooks/delivery-sugar/commit/587ea28fc3f5cbda5c00f0ed77705c1250750d95


update gisce/react-formiga-table to v1.1.0-rc.1

Automated changes by create-pull-request GitHub action

:tada: This PR is included in version 2.0.1-rc.1 :tada:
The release is available on:

npm package (@v2-develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.2-rc.1 :tada:
The release is available on:

npm package (@v2-develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.5-rc.1 :tada:
The release is available on:

npm package (@v2-develop dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Inform Partners When Distribution Content Changes

Resolves #1847
Description
When a Distribution content changes, at the distribution confirmation email, we want to add the changes that had been made in the Distribution content.

Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I tested manually locally and created some specs that emulate the situation and make sure everything is alright.

Screenshots

Regular email, with no content changes

Regular email, with an updated item

Regular email, with an item removed from the content

Hey @rodolfoasantos looks like oyu have a couple rubocop violations, can you run rubocop -a before we get to the review?


Keeping Input

is there any way to integrate awesomplete without changing the input field design language ??

What do you mean by “design language‘?


Add "licence" tag to package.json

It may be useful to add the license to package.json so one might use automated tools to generate a license report.

Can do.


DNS Anomaly Inspector

WIP - Still under development

[x] basic app structure
[x] load SageWorks FeatureSets
[ ] change page content based on chosen FeatureSet
[ ] Fix color map for cluster ids
[ ] Table:

[x] color row based on cluster
[ ] select row, highlighted
[ ] on row selected, highlight selected point on scatter plot and violin plots

[ ] Scatter plot:

[ ] match colors based on cluster
[ ] size based on anomaly_score
[ ] improve labels and font sizes
[ ] on point selected, highlight selected row on table and point on violin plots
[ ] make title optional on create function

@giupb like we talked about in the meeting this should be a copy of the existing FeatureSets page with 'sample()' replaced with 'anomalies()'

Well, the old mockup was deleted, and I implemented the colors in the new directory that @brifordwylie  created. It seems that the colors in the scatter plot table are consistent.

I've saved this PR in a local repo for save keeping if we need it. Closing this as it's in conflict with current main.


Bug fix and Minor Changes

I didn't had time yet to fully understand your code, but I decided to commit this minor bug fix because I don't know how much time it will take for me to fully read and understand your code and write some modifications... But I already see a lot of possibilities that maybe I can help... We can discuss this later mastodon or here too.
 (I'm sorry for typos... English isn't my language)

Thanks for doing this!
Yeah, I probably will make those custom exceptions you mentioned later, that's a good idea


php7.4 Function get_magic_quotes_gpc() is deprecated

Affected Version
Framework 3.7.5
PHP 7.4
Description
Deprecated: Function get_magic_quotes_gpc() is deprecated in /vagrant/www/framework/core/Constants.php on line 172

get_magic_quotes_gpc was hardcoded to always return false in php 5.4 because magic quotes were removed. As of php 7.4 the method is also deprecated. This causes a deprecation notice to be printed unless error reporting is setup to not show notices.
A temporary workaround is to change error_reporting, but I'd really like to see this fixed in 3.7.

Unfortunately, Silverstripe 3 has entered limited support in June 2018. This means we'll only be fixing critical bugs and security issues for Silverstripe CMS 3 going forward.
You can read the Silverstripe Roadmap for more information on our support commitments.


Move remaining syscall definitions out of solana-program

Problem
There are still some syscall definitions that are imposing a solana_program dep on other crates.
Summary of Changes

Move remaining syscall definitions out of solana_program::syscalls::definitions and into solana_define_syscall::definitions. This causes no measurable increase in build time for solana_define_syscall
Re-export the syscalls with deprecation notice
Add a doc note saying that the new definitions module is only for syscall definitions that bring in no extra deps

Done, left out sol_get_return_data too as that depends on solana-pubkey


[bug] Nested resources not being bundled correctly

Describe the bug
When creating installers, tested only on Windows, nsis and msi targets do not install all folders recursively specified under resources with the glob pattern. In the reproduction, the library folder is excluded from the installers.
Reproduction
Folder layout:
root/
├─ resources/
│  ├─ data.db
│  ├─ library/
│  │  ├─ static.json
├─ src/
├─ src-tauri/
│  ├─ tauri.conf.json

Relevant config for tauri.conf.json
  "bundle": {
    "active": true,
    "targets": ["nsis", "msi"],
    "resources": ["../resources/*"],
    // ...
  },

Run tauri build to generate the installers.
Expected behavior
I expect ../resources/* to recursively install all files and folders during installation. However, only top-level files are installed, e.g. _up_/resources/data.db.
Files expected to be installed:

_up_/resources/data.db
_up_/resources/library/static.json

Full tauri info output
[✔] Environment
    - OS: Windows 10.0.19045 x86_64 (X64)
    ✔ WebView2: 128.0.2739.67
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.81.0 (eeb90cda1 2024-09-04)
    ✔ cargo: 1.81.0 (2dbb1af80 2024-08-20)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.17.0
    - npm: 10.8.1
    - bun: 1.1.27

[-] Packages
    - tauri 🦀: 2.0.0-rc.11
    - tauri-build 🦀: 2.0.0-rc.10
    - wry 🦀: 0.43.1
    - tao 🦀: 0.30.0
    - @tauri-apps/api : 2.0.0-rc.4
    - @tauri-apps/cli : 2.0.0-rc.13

[-] Plugins
    - tauri-plugin-shell 🦀: 2.0.0-rc.3
    - @tauri-apps/plugin-shell : not installed!
    - tauri-plugin-updater 🦀: 2.0.0-rc.3
    - @tauri-apps/plugin-updater : 2.0.0-rc.2

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:1420/
    - framework: Vue.js (Quasar)
    - bundler: Vite

Additional context
This used to work on older versions of v2, I'm not sure which version broke this change since my last app release was with version v2-beta.23.
Workaround
  "bundle": {
    // ...
    "resources": ["../resources/*", "../resources/library/*"]
  },

i tried that before moving the issue here. echo ** lists files on my end.

As far as i am aware i did not modify bash's default behavior.


Airport's tax and fee codes

I found some metadata information for identifier code of taxes and fees.
How do you think you can add a new data file of them?
Thanks.
Refer to direct URL:
https://www.singaporeair.com/flightsearch/passengerDetail.form (search keyword: taxes)
https://booking.vietnamairlines.com/statics/applications/booking/dynamicContent/1.0.83/en-GB.json (search keyword: global.taxes)

It's an interesting idea, but I don't think it will make the next few rounds of updates.
What you could do is add your own tags to airports for different taxes and fees.  I could include user tags in the CSV output without too much difficulty.


.NET 6 changes for Host ASP.NET Core in a Windows Service

Fix #23387 by  copying over Create the Windows Service
Update all the old SDK and other versions to the current version.
Worker Service template. Don't call it ASP.NET Core worker service,  call it .NET worker service and link that to Worker Services in .NET
What parts of the article change by using the new minimal hosting model?
App configuration

Point to Minimal API Windows Service Sample
Update instructions

@tothalexlaszlo please review this issue and provide feedback on what needs to be updated for .NET 6

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

ID: b0bd8cb8-7e28-8c4d-a550-3aea8f3ab4dd
Version Independent ID: 4b60d24e-ef05-15e5-10b4-483094081bf3
Content: Host ASP.NET Core in a Windows Service
Content Source: aspnetcore/host-and-deploy/windows-service.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@halter73  please review outline for changes.

The outline looks good to me. As for "What parts of the article change by using the new minimal hosting model?", it's still being designed. We're going to have to wait on the outcome of https://github.com/dotnet/runtime/pull/68580.

@halter73 can you take another look


[react-scripts - coalesceLocales] Allow TS apps to use a src/index.js file

Don't just depend on an app using src/index.tsx to apply a tsconfig for getting the translations in deps.
Tested on tree-person-r9 with src/index.tsx and src/index.js to make sure that all translations come through and they do.
See https://familysearch.slack.com/archives/C0FPL3ZL7/p1681949517152119 for more information.
I found out about paths.appTsConfig because I had printed out paths to see what all was in there.

This looks good. @joeycozza, do you still make releases manually for react-scripts?

No, I don't make manual releases, or for weird branches that we need to test real quick. This'll publish just like normal once it is merged into frontierMaster.


NullPointerException when subclassing Gauges

there will be a NullPointerException when a gaug is subclassed e.g. with SuperGauge extends Gauge end then new SuperGauge();
WARNUNG: Stacktrace: java.lang.NullPointerException
at eu.hansolo.medusa.Gauge.getUserAgentStylesheet(Gauge.java:5219)
This is due to the assumption that the gauge.css style sheet is always available in the resources. I think it would be better to throw an Exception stating this or be lenient and only issue a warning via the logs. Since the css doesn't contain anything interesting at this time the following code works for me:
  @Override     public String getUserAgentStylesheet() {       URL u = getClass().getResource("gauge.css");       if (u!=null)         return u.toExternalForm();       else         return null;     }

Since the resource is part of the medusa library, it'll always be available. This code should always work:
@Override
public String getUserAgentStylesheet() {
  return Gauge.class.getResource("gauge.css").toExternalForm();
}


Unstructured Options

Implement UnstructuredOptions which contain TextProfilerOptions and DataLabelerOptions, and is stored within ProfilerOptions. Required some integration with saving and loading profiles since the labelers had to be removed and restored.

Big refactor, summary of changes:

Now, Profiler and UnstructuredProfiler both can take in either ProfilerOptions or StructuredOptions(for Profiler)/UnstructuredOptions(for UnstructuredProfiler)
Profiler.options is stored as type StructuredOptions and UnstructuredProfiler.options is stored as type UnstructuredOptions

This solves the need for both profilers to remove and restore labeler objects from both, and just concern themselves with the options class that applies to the profiler.
This required extensive rewriting of any calls to self.options in both classes since now there is no need to go one step further into self.options.(un)structured_options.___.
Unrelated, I also changed the signature of Profiler._update_profile_from_chunk to not pass options since they are stored within Profiler any way, and this way it matches the signature of how it is in UnstructuredProfiler.
Some tests were rewritten to accomodate the refactor, the last order of business from the first wave of comments is to rewrite the UnstructuredOptions tests to reduce the amount of redundancy that is common when testing options.


Added configuration for Pioneer WT018GLFI19HLD

I added support for the Pioneer WT018GLFI19HLD Mini Split, which I am currently using.

@DDriggs00
Any chance you can add the Eco buttons, please?
I'm using mine in an RV and need to limit power usage off solar. My unit has an eco mode that's controllable by the app only as the remote doesn't have the button. I 99% think that blasting the eco codes would accomplish this 🙏🏼

@rossfisherkc Unfortunately, Eco Mode/similar modes are not currently supported by SmartIR in general. If they are at some point in the future, I will give adding it a shot.


After deleting the Selection renderer, it is not created when selection is performed.

After deleting the Selection renderer, it is not created when selection is performed. As a result, selection is never displayed, which is counter-intuitive.

Introduced cuemolui.chgMolSel(Obj) method to unify the selection routine, and in this method, presence of the selection renderer is checked so as to create if does not exist.


Edit

The generator worked, but no edit page or view page will show, just got errors. Using laravel 5.2.
Search and pagination works.
Im my table, petid is primary key, I get
SQLSTATE[42S22]: Column not found: 1054 Unknown column 'id' in 'where clause' (SQL: select * from `pets` where `id` = 4 limit 1)

I tryed changing all occurances of id to petid, still nothing.
In model
namespace App;

use Illuminate\Database\Eloquent\Model;

class Pet extends Model {

    public $guarded = ["id","created_at","updated_at"];

    public static function findRequested()
    {
        $query = Pet::query();

        // search results based on user input
        \Request::input('petid') and $query->where('petid',\Request::input('petid'));
        \Request::input('petname') and $query->where('petname','like','%'.\Request::input('petname').'%');
        \Request::input('ownerid') and $query->where('ownerid',\Request::input('ownerid'));
        \Request::input('petowner') and $query->where('petowner','like','%'.\Request::input('petowner').'%');
        \Request::input('species') and $query->where('species','like','%'.\Request::input('species').'%');
        \Request::input('sex') and $query->where('sex',\Request::input('sex'));
        \Request::input('ostreet') and $query->where('ostreet','like','%'.\Request::input('ostreet').'%');
        \Request::input('odate') and $query->where('odate',\Request::input('odate'));
        \Request::input('ocheck') and $query->where('ocheck',\Request::input('ocheck'));
        \Request::input('dogpic') and $query->where('dogpic','like','%'.\Request::input('dogpic').'%');

        // sort results
        \Request::input("sort") and $query->orderBy(\Request::input("sort"),\Request::input("sortType","asc"));

        // paginate results
        return $query->paginate(15);
    }

    public static function validationRules( $attributes = null )
    {
        $rules = [
            'petid' => 'required|integer',
            'petname' => 'required|string|max:30',
            'ownerid' => 'integer',
            'petowner' => 'string|max:30',
            'species' => 'string|max:20',
            'sex' => '',
            'ostreet' => 'string|max:15',
            'odate' => 'date',
            'ocheck' => '',
            'dogpic' => 'string|max:100',
        ];

        // no list is provided
        if(!$attributes)
            return $rules;

        // a single attribute is provided
        if(!is_array($attributes))
            return $rules[$attributes];

        // a list of attributes is provided
        $newRules = [];
        foreach ( $attributes as $attr )
            $newRules[$attr] = $rules[$attr];
        return $newRules;
    }

}

I tryed changing
\Request::input('petid') and $query->where('petid',\Request::input('petid'));

to
\Request::input('petid') and $query->where('petid',\Request::input('id'));

also tryed changing
public $guarded = ["id","created_at","updated_at"];

to
public $guarded = ["petid","created_at","updated_at"];

I have tryed various combinations of id and petid.

Hi @jimgwhit
At the moment NVD CRUD generator accepts only 'id' column as the primary key. Change 'petid' to 'id' in your database table and the problem will be gone.
P.S. Having a primary key other than 'id' will be supported in upcoming versions. But right now, you have to follow the convention.

Thanks for the reply, any idea how soon the version will be out that supports other than id?

It's the first thing on my Todo list.

Do you have any news?


proxy: shuffle endpoints

Shuffle endpoints to avoid being "stuck" to a single cluster member.

LGTM

@bakins Thanks!


Butterworth filtering function for AnalogSignal

Added a Butterworth filtering function butter() in a newly created module signal

I had used matplotlib.mlab.psd() in the unit tests, but I noticed that elephant is not dependent on matplotlib and so this function shouldn't be used.
In the latest commit I replaced all occurrences of matplotlib.mlab.psd() with an (almost) equivalent function in scipy, i.e. scipy.signal.welch().

Coverage decreased (-0.11%) to 99.42% when pulling 5566f7a5c65b0245a4688335ba748880af57acbb on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.11%) to 99.42% when pulling 5566f7a5c65b0245a4688335ba748880af57acbb on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.15%) to 99.38% when pulling 29116f92f69d8b06af5b9a941d391fe34f94437f on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.15%) to 99.38% when pulling 29116f92f69d8b06af5b9a941d391fe34f94437f on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.15%) to 99.38% when pulling 29116f92f69d8b06af5b9a941d391fe34f94437f on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.1%) to 99.42% when pulling 96ab41a99f9e38217510431f35aa1a365a964a17 on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.1%) to 99.42% when pulling b69e82b15cf791ee03c453aba9d60ceb35508774 on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

Coverage decreased (-0.1%) to 99.42% when pulling b69e82b15cf791ee03c453aba9d60ceb35508774 on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.

according to the discussion in PR #22, the module name was changed.

By the way, my apologies if my comments seem rather brief and hence peremptory. It is just that I am typing with one hand this week following a ski accident so I'm avoiding too much typing! Of course all comments are just suggestions and are up for discussion!

i've made fixes according to the comments.
I've also made changes to properly deal with inputs given as Quantity array or NumPy ndarray.
Please review the code again.

Coverage decreased (-0.17%) to 99.35% when pulling 15a20b1b477fd1026d1cb45305272d815c64c6c9 on Junji110:butter into 37946e29566210dce1d0b9dd972e23da631f7a21 on NeuralEnsemble:master.


Lack of automation Steps 1 -5  in README

I'll handle adding the automation around the first 5 steps of the README for setup, opening this to track and note it should be done.
Will take some time to create and test.

@danehans We can close this once #18 is accepted since it removes those steps from the user.


Fix acme storage file docker mounting example

It is not possible to mount a relative file. A mount path has to be be absolute.
docker: Error response from daemon: invalid volume specification: '/my/host/acme.json:acme.json': invalid mount config for type "bind": invalid mount path: 'acme.json' mount path must be absolute.
See 'docker run --help'.

 Hope this is now the desired form.


[dashboard] Fix underlying index name for dashboards

Description
Based on https://github.com/opensearch-project/terraform-provider-opensearch/pull/41, this fixes a bug in 1.x from https://github.com/opensearch-project/terraform-provider-opensearch/pull/17. The internal index name is still .kibana and not .dashboard (as described in opensearch issues, e.g. https://github.com/opensearch-project/OpenSearch-Dashboards/issues/1270).
Issues Resolved
Misnamed internal index
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.

Thanks LGTM.


Method signature (alias / spread) User experience

https://github.com/Azure/autorest.java/issues/2678

close it, but it may need update, as the current PR to typespec-azure contains invalid tsp

update doc for "spread is spread"
https://github.com/Azure/typespec-azure/pull/1143


report crosshair coordinates?

Howdy-
Is there a function/setting to display the (x,y,z) coordinates of crosshair location, so the user knows where they are looking in the brain?  I browsed a bit, but did not see one.
Thanks,
pt

@mrneont ,
You will need to supply an onLocationChange callback function like this:
function locationChangeHandler(data) {
    console.log(data);
    // or get an element in the html document and display the data.string
   // there are also other properties on the data object
   // https://github.com/niivue/niivue/blob/main/src/niivue.js#L7379
}
const nvobj = new niivue.Niivue(
  {
     logging:true,
     show3Dcrosshair: true,
     onLocationChange: locationChangeHandler
  }
)

one of the live demos shows this as well: https://github.com/niivue/niivue/blob/main/demos/features/basic.multiplanar.html#L36

Thanks, I have this working now.  I was trying to find where 'data.string' was created---I like that information, but would like to change the formatting of the coordinates and intensity values.  I haven't been able to backtrack it, though (even trying to search for the unicode \times symbol).

@mrneont,
the line in niivue is here:
https://github.com/niivue/niivue/blob/main/src/niivue.js#L7352
the data object that is returned has the properties:
data = {
  mm, // mm coords
  axCorSag, // slice
  vox, // voxel location
  frac, // clip space
  xy, // mouse xy
  values, //  voxel values for all loaded volumes
  string // location coords as string
}

We can prob document this better to help make it more clear in the docs :). Thanks for pointing it out! I think others would find it helpful to know what's returned.

@mrneont the string is generated by createOnLocationChange().  However, if you do not like the formatting of the string, you can always apply your preferred formatting:
function handleLocationChange(data){
  document.getElementById('location').innerHTML = 'voxel location: ' + data.vox + ' ' + data.values[0].value
}

Thanks, that is useful.
Maybe I will mention what I want to do explicitly, and how I have started clunkily doing this for suggestions of a better way to go.  I would like to report the (x,y,z) coords and values of the underlay, overlay and threshold volumes.  I am aiming for this format of text string:
xyz:[0.5L, 21.5A, 16.5S], UOT:[123, -4.321, 7.890]

So, my first approximation to this is including this function in my HTML directly:
  function reportCoorAndValues(data) {
    //document.getElementById("intensity").innerHTML = "  " + data.string;
    document.getElementById('intensity').innerHTML = 'xyz:[' + data.mm[0] + ', ' + data.mm[1] + ', '+ data.mm[2] +'], UOT:[' + data.values[0].value + ', ' + data.values[1].value + ', ' + data.values[2].value + ']'
      console.log(data.string);
  }

... which gets the numbers in the right spot, but with faaaar too many decimal points, and without converting the signed coordinate values to more useful coordinate magnitude-with-letter format.

For the decimal question, how should I best include the dynamicDecimals() functionality, without duplicating a lot of code?
For the coordinate magnitude-with-letter format, I believe NiiVue always works in a frame where left/posterior/inferior are all negative valued, is that correct?  Therefore, I would like to do the following, for a given (X, Y, Z) coordinate triple, for each of the coords (where this is a kind of psudocode):

let xmag = str(abs(X))
let xlet = (X<0) : 'L' ? 'R'
let xfinal = xmag + xlet

Does that make sense?  Should I do those pieces within NiiVue src, or just within the HTML?

@mrneont you can also have your wrapper generate a location change event. This is useful for showing information when you load a new image.
nv1.createOnLocationChange();

@mrneont here are two functions for converting numbers to a string with a fixed string with a limited number of digits. The difference is that the second function retains the trailing zeros:
// node flt2str.mjs
async function main() {
  function flt2str(flt, decimals = 0) {
    //return parseFloat(flt.toFixed(decimals)); //return number
    return parseFloat(flt.toFixed(decimals))+""; //return string
  }
  let val = 0.251;
  console.log('flt2str type ', typeof flt2str(val));
  console.log('flt2str(val,0): ', flt2str(val, 0));
  console.log('flt2str(val,1): ', flt2str(val, 1));
  console.log('flt2str(val,2): ', flt2str(val, 2));
  console.log('flt2str(val,3): ', flt2str(val, 3));
  console.log('flt2str(val,4): ', flt2str(val, 4));
  function flt2str0(flt, decimals = 0) {
    //retain trailing zero
    return flt.toFixed(decimals); //return string
  }
  console.log('flt2str0 type ', typeof flt2str0(val));
  console.log('flt2str0(val,0): ', flt2str0(val, 0));
  console.log('flt2str0(val,1): ', flt2str0(val, 1));
  console.log('flt2str0(val,2): ', flt2str0(val, 2));
  console.log('flt2str0(val,3): ', flt2str0(val, 3));
  console.log('flt2str0(val,4): ', flt2str0(val, 4));
}
main().then(() => console.log('Done'))

You can observe the behavior by running this script from the command line:
$ node flt2str.mjs
flt2str type  string
flt2str(val,0):  0
flt2str(val,1):  0.3
flt2str(val,2):  0.25
flt2str(val,3):  0.251
flt2str(val,4):  0.251
flt2str0 type  string
flt2str0(val,0):  0
flt2str0(val,1):  0.3
flt2str0(val,2):  0.25
flt2str0(val,3):  0.251
flt2str0(val,4):  0.2510
Done

Great, thanks, that is working for me.
And for reporting the directionality of coords, I wrote a function to take coords where L/P/I are neg (and R/A/S are pos) and use the signum information to report the coordinate with the appropriate letter instead of the sign:
/* more string formatting numbers, below ... WITH directionality here,
   assuming the coords are what AFNI calls LPI (and what some other
   software call RAS); basically, L/P/I are neg coords, and R/A/S are
   positive.
*/
function flt2str0_dir(flt, ndec = 0, dir = '' ) {
    //retain trailing zero
    if ( dir == '' ) {
      return flt.toFixed(ndec); //return string
    } else if ( dir == 'RL' ) {
      let aflt = Math.abs(flt);
      let lab  = (flt < 0 ) ? 'L' : 'R';
      return aflt.toFixed(ndec) + lab;
    } else if ( dir == 'AP' ) {
      let aflt = Math.abs(flt);
      let lab  = (flt < 0 ) ? 'P' : 'A';
      return aflt.toFixed(ndec) + lab;
    } else if ( dir == 'IS' ) {
      let aflt = Math.abs(flt);
      let lab  = (flt < 0 ) ? 'I' : 'S';
      return aflt.toFixed(ndec) + lab;
    } else {
      return '';
    }
}

... and then when using that:
      let str_c = '&nbspxyz:[' +
                  flt2str0_dir(data.mm[0], 3, 'RL') + ', ' +
                  flt2str0_dir(data.mm[1], 3, 'AP') + ', ' +
                  flt2str0_dir(data.mm[2], 3, 'IS') + '], '


Fixes default expire time in creating access token

Fixes the default expire timedelta used in the function to create an access token.
The default timedelta in create_access_token is updated from '15' to the config setting of ACCESS_TOKEN_EXPIRE_MINUTES.
This matches the behavior of the create_refresh_token function right below it, which uses the config setting of REFRESH_TOKEN_EXPIRE_DAYS for its default expire timedelta. (ACCESS_TOKEN_EXPIRE_MINUTES was also already imported into the file for this likely purpose while not being used.)

Nice catch!


Disallow contributing configuration defaults for launch

We should disallow contributing configuration defaults for launch
At present, extensions can contribute default value for launch using configurationDefaults contribution point. Even though user can customise it in user settings, this is not real Setting. This is just a user launch configuration. If multiple extensions contribute default values of launch then this can lead to unexpected behaviour. There is also a performance impact that this is pulling the complete launch schema when opening package.json.
CC @connor4312 @aeschli

Provided a metadata property for configuration to define if the setting should allow contributing configuration defaults or not
	/**
	 * Disallow extensions to contribute configuration default value for this setting.
	 */
	disallowConfigurationDefault?: boolean;

And adopted launch to use this.


Flip cull mode when rendering flipped Light2D and LightOccluder2D

Fixes #32993.
Discussion on IRC about the proper fix: 2019-12-10 22:33:57.
Not sure about the performance implications, but I tried to keep them to a minimum.

Thanks!


Invalid CA after uploading the certificate

The document is wrong.. (in my setup at least)
You should use x.509 v3 ca basic constraints with CA:true when creating the CA root certificate. Otherwise, you will get Invalid ca health probe status.
Instead of this:
openssl x509 -req -sha256 -days 365 -in contoso.csr -signkey contoso.key -out contoso.crt
Use this one:
openssl req -new -x509 -days 3000 --extensions v3_ca -key contoso.key -out contoso.crt
As a result, the certificate will be marked with CA:true and application gateway health probe will be fine:
...
  X509v3 Basic Constraints: critical
                CA:TRUE
....

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

ID: 3b0e716d-54fc-0b48-e2aa-9f5b126072ae
Version Independent ID: cda1facc-d299-a154-e824-a1b695f812d8
Content: Generate self-signed certificate with a custom root CA - Azure Application Gateway
Content Source: articles/application-gateway/self-signed-certificates.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

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

@rlevchenko Thank you for your patience while we were working on this issue. I have looked into the issue further and have also tried to reproduce it by following the commands as given in the document i.e.,
openssl x509 -req -sha256 -days 365 -in contoso.csr -signkey contoso.key -out contoso.crt
Using the above command, I do not see any issues with the health probe. The health probe is getting 200 OK consistently. If you are getting Invalid CA health probe, there must something else that is causing it. I get a similar error if under my http settings, hostname override with specific domain name is incorrect i.e., does not match the Common Name of the certificate exactly. Please check the same and confirm that it matches the CN of the certificate.
I hope this is helpful. If you have any further questions, please feel free to reach out anytime and we will be glad to assist. Thanks you!

@SaiKishor-MSFT thanks for the response. take a look at this doc You need to have a cert. chain actually . 1) create a ca certificate   2) sign the client certificate by the CA certificate and then AppGw works fine. I used the same CN, so not sure that the issue was related to subject/cn cert properties.
openssl x509 -req -sha256 -days 365 -in contoso.csr -signkey contoso.key -out contoso.crt - doesn't create a Root CA certificate.

@rlevchenko
I understand your explanation about needing Certificate chain and that you used the same CN name. I also see you mentioned that using the command given in the document, you are unable to create a root CA certificate. However, I was able to create a root CA certificate using the given command (please note that the Root CA certificate that I have, does not have basic constraints value with CA field set as true, however, it still works and gives me a root certificate and the health probe gives me a 200 OK). I also tried the command that you mentioned i.e.,
"openssl req -new -x509 -days 365 --extensions v3_ca -key contoso.key -out contoso.crt"
And I see the certificate now has the basicconstraints value with CA field set to true. However, this does not make any difference to the the way the APP GW is running. i.e., it still succeeds with a 200 OK.
In summary, I tried both the commands and the certs that I got from both ways work perfectly fine and health probe returns a 200 OK. I suppose the document is right and it works fine. If you further have any questions/concerns regarding this issue, please let me know or you can also reach out to the Azure Support team via chat/call and work with them directly.
Thank you!

@SaiKishor-MSFT I see. Thanks. It's strange enough you can't reproduce this 'cause I tested multiple times and got no luck. Will update this topic if face with the same error next time.

@rlevchenko  What is the error message that you are getting currently for the health probe and what do you get for using a wrong hostname?

@SaiKishor-MSFT don't have any errors with CA=true certificate. If you mistyped hostnames in the cert, the app gw shows straightforward message - hostnames aren't the same. If the root cert is not marked as CA, app gw shows - invalid CA certificate. The main point is that error messages in both cases are different.

@rlevchenko
Thank you for the update. Yes, I am not getting any errors when I follow the document as it is. I am glad that its working for you when CA is given as TRUE. If in any case, you want further assistance, Azure Support can help better by taking a look at your resources and environment. I will close this issue as mentioned. Thank you!


[BUG] Selection of inline code is not visible in edit mode

Describe the bug
Selection of code formatted with `` is not visible. If I want to select a part of code, I don't see selection at all.
To Reproduce

Insert text:
Code: this is code formatted text.
Try to select it by mouse.

Expected result
There's a selection rectangle covered all selected text.
Actual result
Selection rectangle isn't visible at code block.
Screenshots

There's no problem in rendered mode:

Version [Desktop or Mobile]:
v0.12.19 with latest theme version.
Additional context
Comparison with Blue Topaz theme:

Hello! Thanks for reporting it, I'll work on fixing it in time for next update's release.

Please confirm if the issue has been fixed! :)

Selection rectangle is still behind white background:

Now there's just a thin line under the code:

Oh, I see what this is. I don't think it's possible in regular edit mode, unless you assign a background with some transparency, but then you'd lose the contrast between inline code and regular code (besides the color of the text, ofc).
You can try adding the following snippet:
.cm-s-obsidian span.cm-inline-code:not(.cm-formatting):not(.cm-hmd-indented-code):not(.obsidian-search-match-highlight),
span.cm-formatting.cm-formatting-code.cm-inline-code {
	background-color: hsla(var(--hsl-color-primary-w), .5);
}


typo in docs

Simple typo in docs

thanks for the fix @dbfreem and the reminder @prathmeshranaut !


Docs/readme cleanup

slight edits to language

clarifying which tools

Update from five to seven sub-commands
Header on Usage Details
Added section for show_downstream

@rsaporta thank you so much!!


Add Observer module

The Observer class lets developers observe BasePart instances that may stream in and out of existence when StreamingEnabled is turned on.

Changed names to Streamable and StreamableUtil to properly reflect that this impacts parts that may be streamed in & out during runtime.


Update readme to include section on missing analysis

Fixes #598
Merge / deployment checklist

[x] Confirm this change is backwards compatible with existing workflows.
[x] Confirm the readme has been updated if necessary.
[n/a] Confirm the changelog has been updated if necessary.

The new text in the readme is mostly copied from the original docs page, which describes the issue so nicely I didn't want to make any changes.


The note about the suffix for naming should be placed higher in the doc

There is a note a bit down the documentation saying:
_Note
The naming format for the function app and storage account has changed. To guarantee global uniqueness, a random and unique string is now appended to the names of these resource._
This note is placed way after step 4 in Deploy Feature which just states "Type a name that is a valid URL path". The note appending suffixes to this name (which is good in theory) should be placed there or even better should be integrated into step 4.

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

ID: 472a3f66-eb6d-4699-5a5d-558b69c60b4f
Version Independent ID: 9529161c-f2b8-0c5b-e62a-dcee86601d47
Content: Deploy Start/Stop VMs v2 (preview)
Content Source: articles/azure-functions/start-stop-vms/deploy.md
Service: azure-functions
Sub-service: start-stop-vms
GitHub Login: @ggailey777
Microsoft Alias: glenga

@DennisGaida The note is referring to the screenshot above it. The unique strings are appended as part of the deployment templates, making the expected names of the resources to be different than as shown in the screenshot.
So, Step 4 is still correct.

Thank you for your prompt reply @PramodValavala-MSFT!
I'm not suggesting step 4 is wrong, I am saying that at the point of creating the resources the user should know that the chosen name will not be the final name. Only later in the process (after having executed the deployment) we see a note that the names are actually adjusted (which is good).
I had trouble with this because I actually personally added a random string suffix to my names to make them unique. Only after having created resources I saw the note and had to delete the whole resource group again because it was unnecessary for me to add a unique suffix.
All I am saying is to move that note to where it actually matters: Where a user chooses a name ("Please note, user, the name will have a unique suffix added on creation").

@balas77 can you help with this?

Yeah will take care to update the doc to move the note section ahead of the user inputs to let them aware. Also will include this in the help text for the fields that we add the suffix.

The note section on the naming format has been moved up. Please close this issue.

Thanks @balas77 #please-close


Fix did_type

Fix did_type in order to allow the frontend code recognize the type of the DID

I need to fix something.


fix, chore, refactor(egen): hardcodes scikit-learn version to 1.2, changes python version from 3.9 to 3.10, adds numpy==1.26.4 installation, adds code to undeploy model from endpoints

This PR,  includes the following changes:

Adds project_id param in get_dataframe component
Changes python base image version from 3.9 to 3.10
Changes scikit-learn version from 0.24 to 1.2
Adds numpy==1.26.4 in packages to install to resolve dependency issues
Adds code to undeploy model from endpoints, delete model and endpoints in cleanup section
Adds code to delete locally generated files
Changes REGION to LOCATION
Includes colab restart kernal and authentication as per template guidelines
Adds collab enterprise logo
Makes structural changes according to new notebook template
Removes the use of future tense

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

/gcbrun


Some cleanup of multiple checks and a more generic hostname generation approach

Hi,
i´ve made the following changes to the Code:

Fixed debian/changelog to be valid for the parsechangelog script
Added a more generic hostname generation approach for metric names
-> uses the full fqdn and reverse it, so you get some grouping in graphite
Added "path" in memcached, mongodb and mysql collector configs

I think I got everything other then ccbb044 already. If I'm wrong, please let me know


bypass for this?

do you have bypass mechanism or userscript for intermediate pages on some sites eg
https://downloadhub.ink/animal-factory-2000-hindi-dual-audio-300mb-bluray-480p-esubs/
when downloading from this site a intermediate site is there forextuner.com where you need to click to verify,wait,double click all that sorts of nuisance etc
i want to bypass that because its present on many sites (i guess it called soralink)
https://downloadhub.ink
https://deadtoonsindia.info/beyblade-burst-turbo-season-3-episodes-hindi-eng-dual-audio-download-480p-720p-1080p-hd/
https://desiremovies.cam
https://2filmygod.in
https://moviesverse.in/download-the-prom-2020-hindi-480p-720p-1080p/
https://9moviesflix.com/download-indoo-ki-jawani-2020-hindi-movie-web-dl-480p-400mb-720p-1gb/
https://bollyflix.best/naxalbari-s01-hindi-zee5-originals-web-series/
https://hdhub4u.life/torbaaz-2020-hindi-webrip-all-episodes/
https://katmoviehd.se/the-night-manager-s01-vo/
https://www.mkvcinemas.uk/a-call-to-spy-2020-amzn-movie-webrip-dual-audio-hindi-eng-400mb-480p-1-3gb-720p-4gb-9gb-1080p/
https://southfreak.surf/goutham-nanda-rowdy-rajkumar-2-2017-uncut-720p-hevc-hdrip-dual-audio-hindi-800mb/
https://9xmovie.run
https://hdmovie2.in
https://bolly4u.ninja/durgamati-the-myth-2020-web-dl-hindi-movies-download-720p/
https://hdmovie99.org
https://thekhatrimaza.com/ip-man-kung-fu-master-2020-english-720p-bluray-850mb-download/
https://khatrimaza1.in/shark-attack-3-megalodon-2002-hindi-dual-audio-330mb-hdrip-esubs-download/
https://hdmoviezhub.club/my-cousin-rachel-hindi-480p-720p/
https://bollyflixhd.ml
https://moviezcrush.com/download-the-christmas-2-2020-full-movie-hindi/

& list is big (bypass used to work in universal bypass but after their dev legal battle with linkvertise ,it seems development is stalled there)

Links are hardcoded --
Watch Online

There's no js manipulation involved.

used to work https://github.com/Sainan/Universal-Bypass/issues/193 don't know how?

Better ask them then, I don't know how they do it, but there's no client-sided javascript involved.


Add vouchers to draft orders

Use case:
• Add voucher code to draft order
• After finalizing the order, voucher should redeem (voucher use should increment)
• Voucher should be validated, for example if it is applicable to the user, basket value etc. Same as checkout

This has been dekoderem in Saleor 3.19+


what is a solution anyway?

can probs abstract the solution/particle/bee/ant/penguin/grenade/simp/etc to a separate module?
could call it freelunch.dodgy_analogies??

I think a solution is an animal, going to change the name of the base class to animal in the near future.

I think a solution is an animal, going to change the name of the base class to animal in the near future.

Have created a new submodule freelunch.zoo in 632eea7
Have left existing classes in tech for the time being.
The plan is to have all optimisation algos use this file going forward. @TimothyRogers Can you update the PSO and KH whenever you get the chance?
Will close once classes are removed from tech.

Have created a new submodule freelunch.zoo in 632eea7
Have left existing classes in tech for the time being.
The plan is to have all optimisation algos use this file going forward. @TimothyRogers Can you update the PSO and KH whenever you get the chance?
Will close once classes are removed from tech.

Closed by #8

Closed by #8


PHPUnit - Selenium Testing with Filters Disabled

I have a test that launches all of my selenium tests using Selenium Server and Selenium RC
<?php

class SeleneseTest extends PHPUnit_Extensions_SeleniumTestCase
{
    public static $seleneseDirectory = 'app/tests/selenium';

    protected function setUp()
    {
        $this->setBrowser('*googlechrome');
        $this->setBrowserUrl('http://localhost:8888/');
    }
}

?>

but since it is running in my testing environment, the filters are disabled in Illuminate\Routing\RoutingServiceProvider
protected function registerRouter()
{
	$this->app['router'] = $this->app->share(function($app)
	{
		$router = new Router($app['events'], $app);

		// If the current application environment is "testing", we will disable the
		// routing filters, since they can be tested independently of the routes
		// and just get in the way of our typical controller testing concerns.
		if ($app['env'] == 'testing')
		{
			$router->disableFilters();
		}

		return $router;
	});
}

Calling Route::enableFilters(); or $this->app['router']->enableFilters(); does not seem to work here. Can anyone think of a solution? Possibly extending Illuminate\Routing\RoutingServiceProvider to disable the feature altogether or to permanently enable the filters somewhere else?

Yeh, this was removed in L5. Please see https://github.com/laravel/framework/pull/6114.


[Bug]: Coolify first reports deployment/build failure and then success after a while

Description
I have an angular app that is built using nixpacks via npm install and npm build.  The build takes a while and Cooliy first reports a failure.  However the build is still running and then a couple of minutes later reports a success.
Is there a build timeout or something I can tweak to get coolify/nixpacks to wait longer?
Minimal Reproduction (if possible, example repository)
Angular 16 App Build via npm build
Exception or Error
No response
Version
v4.0.0-beta.177

This happened today while deploying a MSSQL docker container.
As can be seen from this screenshot, already indicated as Failed but docker download is still ongoing.  Once all done and container is running, a success will also be reported.

I'm having the same issue.

This should be fixed. If not, please reopen.


Add instructions for users on ZIP file upload limitations

For ZIP uploads, users should not upload objects that contain more than 25k files.  We should provide some user instructions to let them know this (i.e. number of files inside the ZIP should be less than 25k).  It will not be enforced with validation (we don't need to say this though).
Need exact language and location of message.

Current instructions look like this:

These should be updated to say:
Upload a single zip file (containing fewer than 25,000 files and less than 10GB total), which will be unzipped, including any hierarchy.
Viewers will see the file hierarchy and be able to download individual files. Unzipping may take several minutes if your zip file contains a large number of individual files.
Warning: this file upload option will automatically replace all existing files on your item with the content in the new zip file.


Improve ChecksummedBytes::extend and clarify data integrity guarantee

Description of change
ChecksummedBytes maintains a data buffer and a checksum and guarantees that only validated data can be accessed. Transformations such as split_off, extend, or slice (introduced in this change), may trigger a validation (and return an IntegrityError on failure), or propagate existing checksum(s) if possible, allowing for later validation.
This change clarifies the data integrity guarantee in the docs and optimizes the extend method to avoid re-validation when the checksums for both slices can be combined. It also avoid a redundant buffer allocation.
Also moves ChecksummedBytes and related types into a new root module.
Does this change impact existing behavior?
No.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

For a clearer diff of the changes in ChecksummedBytes see just commit 711d37d.


Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@sveltejs/kit' on production build

Describe the bug
When I create a new SvelteKit project with the node adapter and try to use the files in the build directory, I get the following error:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@sveltejs/kit' imported from [...]

Logs
internal/process/esm_loader.js:74
    internalBinding('errors').triggerUncaughtException(
                              ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@sveltejs/kit' imported from [REDACTED]/app.js
    at packageResolve (internal/modules/esm/resolve.js:655:9)
    at moduleResolve (internal/modules/esm/resolve.js:696:18)
    at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:810:11)
    at Loader.resolve (internal/modules/esm/loader.js:86:40)
    at Loader.getModuleJob (internal/modules/esm/loader.js:230:28)
    at ModuleWrap.<anonymous> (internal/modules/esm/module_job.js:56:40)
    at link (internal/modules/esm/module_job.js:55:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

To Reproduce

create a new SvelteKit skeleton project
add adapter-node to it
make sure all the deps have "next" as version
yarn && yarn build
isolate the build directory (I only want the production files in my production environment) by moving it somewhere else
add a package.json file to the directory with the contents: { "type": "module" } (for ESM)
node . <- This is what triggers the bug

Expected behavior
I expect the build directory to contain everything that is needed to run the app in isolation. Adding the small package.json file is another thing that I do not want to do myself.
When running node . the server should just run.
Information about your SvelteKit Installation:

Diagnostics
  System:
    OS: macOS 11.2.3
    CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
    Memory: 5.37 GB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.16.0 - ~/.volta/tools/image/node/14.16.0/bin/node
    Yarn: 1.22.10 - ~/.volta/tools/image/yarn/1.22.10/bin/yarn
    npm: 7.7.0 - ~/.volta/tools/image/npm/7.7.0/bin/npm
  Browsers:
    Chrome: 90.0.4430.85
    Firefox: 87.0
    Safari: 14.0.3
  npmPackages:
    @sveltejs/kit: ^1.0.0-next.74 => 1.0.0-next.91
    svelte: ^3.29.0 => 3.37.0
    vite: ^2.2.2 => 2.2.3

Your adapter: Node

Severity
It's kind of annoying as I really want my production setup to be ready, but it doesn't stop me from development.
Additional context
All the above steps are done in a Dockerfile for production but I was able to reproduce it outside as well

Dockerfile
FROM node:14-alpine as build

ARG VITE_API_URL

RUN mkdir /svelte-build
WORKDIR /svelte-build

ENV VITE_API_URL=${VITE_API_URL}

COPY . .
RUN yarn && yarn build

FROM node:14-alpine as production

RUN addgroup -S app && adduser -S app -G app

RUN mkdir /app && chown app:app /app

USER app

WORKDIR /app

RUN echo '{ "type": "module" }' > /app/package.json
COPY --chown=app:app --from=build /svelte-build/build/ /app

EXPOSE 3000

CMD node index.js

I ran into this the other day. (cc @Rich-Harris) Rich agreed that this was probably actually a bug. The server-side output that's sent to the Node adapter should probably be re-bundled within the adapter itself, like is happening for some other adapters. @sveltejs/kit shouldn't be an external prod dependency of the built, adapted Node app. When this re-bundling happens, probably everything in devDependencies (that is, @sveltejs/kit and whatever other dependencies you've added) should be included in the server-side bundle.
In the meantime as a workaround, you can move @sveltejs/kit and any other dependencies your app needs at runtime from devDependencies to dependencies but this obviously isn't a great solution.

Same issue and above workaround did NOT work for me. Now it complains about missing svelte module. Does not matter if svelte is in devDependencies or dependencies.

Same issue with me. Trying deploying to heroku using adapter-node
complete log:

2021-05-26T13:05:39.000000+00:00 app[api]: Build succeeded
2021-05-26T13:05:41.632730+00:00 heroku[web.1]: Starting process with command `npm start`
2021-05-26T13:05:48.381759+00:00 app[web.1]:
2021-05-26T13:05:48.381783+00:00 app[web.1]: ><EMAIL_ADDRESS>start /app
2021-05-26T13:05:48.381784+00:00 app[web.1]: > node build
2021-05-26T13:05:48.381784+00:00 app[web.1]:
2021-05-26T13:05:48.727284+00:00 app[web.1]: internal/process/esm_loader.js:74
2021-05-26T13:05:48.727483+00:00 app[web.1]: internalBinding('errors').triggerUncaughtException(
2021-05-26T13:05:48.727485+00:00 app[web.1]: ^
2021-05-26T13:05:48.727485+00:00 app[web.1]:
2021-05-26T13:05:48.727486+00:00 app[web.1]: Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@sveltejs/kit' imported from /app/build/app.js
2021-05-26T13:05:48.727487+00:00 app[web.1]: at packageResolve (internal/modules/esm/resolve.js:655:9)
2021-05-26T13:05:48.727487+00:00 app[web.1]: at moduleResolve (internal/modules/esm/resolve.js:696:18)
2021-05-26T13:05:48.727488+00:00 app[web.1]: at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:810:11)
2021-05-26T13:05:48.727488+00:00 app[web.1]: at Loader.resolve (internal/modules/esm/loader.js:88:40)
2021-05-26T13:05:48.727489+00:00 app[web.1]: at Loader.getModuleJob (internal/modules/esm/loader.js:241:28)
2021-05-26T13:05:48.727489+00:00 app[web.1]: at ModuleWrap. (internal/modules/esm/module_job.js:56:40)
2021-05-26T13:05:48.727490+00:00 app[web.1]: at link (internal/modules/esm/module_job.js:55:36) {
2021-05-26T13:05:48.727490+00:00 app[web.1]: code: 'ERR_MODULE_NOT_FOUND'
2021-05-26T13:05:48.727491+00:00 app[web.1]: }
2021-05-26T13:05:48.783848+00:00 app[web.1]: npm ERR! code ELIFECYCLE
2021-05-26T13:05:48.783850+00:00 app[web.1]: npm ERR! errno 1
2021-05-26T13:05:48.799210+00:00 app[web.1]: npm ERR<EMAIL_ADDRESS>start: `node build`
2021-05-26T13:05:48.799211+00:00 app[web.1]: npm ERR! Exit status 1
2021-05-26T13:05:48.799212+00:00 app[web.1]: npm ERR!
2021-05-26T13:05:48.799213+00:00 app[web.1]: npm ERR! Failed at the<EMAIL_ADDRESS>start script.
2021-05-26T13:05:48.799214+00:00 app[web.1]: npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
2021-05-26T13:05:49.642211+00:00 app[web.1]:
2021-05-26T13:05:49.654457+00:00 app[web.1]: npm ERR!  I'm using the following successfully to deploy in production for several weeks now (same as the workaround as @Conduitry mentioned):
  "dependencies": {
    "@sveltejs/adapter-node": "next",
    "@sveltejs/kit": "next"
  }

@utkarshkukreti tried with the 'workaround' and still, it then complains on svelte missing:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'svelte' imported from /app/build/app.js, as @Discorainman reported. By the way, I tried also adapter-begin and it fails even earlier in building in my machine. But I am still investigating any error on my part.

@katkrasinsk what if you add svelte to dependencies too?

@utkarshkukreti  it does work with adding svelte as dep, but I got a much bigger build, anyway, I am with this solution in order to continue developing the app.

I saw noticed that I didn't have "@sveltejs/adapter-node" in my dependencies. Now it's building fine. I wonder if others are doing this by mistake as well.

I have the same error after latest updates:
 [ERR_MODULE_NOT_FOUND]: Cannot find module '/usr/local/lsws/nodes/bitfinicon/.svelte-kit/output/server/internal.js' imported from<EMAIL_ADDRESS>

I encountered the same error: Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/helena/svelte/personal-website/.svelte-kit/output/routes/blog/0/+page.md' imported from /Users/helena/svelte/personal-website/.svelte-kit/output/server/chunks/index2.js

@bitdom8 @helenawsu commenting on 2-year-old closed issues will not usually get a helpful response. If you're encountering a new bug with SvelteKit, please open a new issue with a minimal reproduction

I encountered the same error: Error [ERR_MODULE_NOT_FOUND]: Cannot find module '/Users/helena/svelte/personal-website/.svelte-kit/output/routes/blog/0/+page.md' imported from /Users/helena/svelte/personal-website/.svelte-kit/output/server/chunks/index2.js

The reason to this was I was using adaptor-static and used dynamic import. I can't build static with dynamic import.

Does quitting VSCode and restarting it back up work for anyone? It worked for me.


Add atombrella to plugin-clearcase

Description
I made some fixes to the clearcase plugin, and would like to be able to release it, as well as merging some of the PRs in this repo.
CC @ewelinawilkosz @praqma-thi @MadsNielsen
https://github.com/jenkinsci/clearcase-plugin
Submitter checklist for adding or changing permissions
Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[ ] Add link to resolved HOSTING issue in description above

For a new permissions file only

[ ] Make sure the file is created in permissions/ directory
[ ] artifactId (pom.xml) is used for name (permissions YAML file).
[ ] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[ ] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[ ] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[ ] Make sure to @mention the users being added so their GitHub account names are known if they require GitHub merge access (see below).
[ ] All newly added users have logged in to Artifactory at least once

Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

I'm aware of the background to this approved

I'm aware of the background to this approved


_WD_HighlightElements() refactoring is needed

I will do this in future
currently this is just a reminder, I will propose new PR soon

Func _WD_HighlightElements($sSession, $aElements, $iMethod = Default)
	Local $iHighlightedElements = 0

	If $iMethod = Default Then $iMethod = 1

	For $i = 0 To UBound($aElements) - 1
		$iHighlightedElements += (_WD_HighlightElement($sSession, $aElements[$i], $iMethod) = True ? 1 : 0)
	Next
	Return ($iHighlightedElements > 0 ? SetError(0, $iHighlightedElements, True) : SetError($_WD_ERROR_GeneralError, 0, False))
EndFunc   ;==>_WD_HighlightElements

It's hard to maintain - I had to go over the topic several times to find out what it was about.
There is one issue.
If there are provided 10 elements to highlight and error ocurs on 5th element then procesing should be stoped with error, but it will process to the end, and finallly report success.
I will do some fix for this.

Rewritten in #172


Hugo site problem

Hi, thanks for making this. This is exactly what I was looking for, but I can't make it work. I copied functions folder into root of my site and uploaded zip to cloudflare. I added CFP_PASSWORD into pages environment variable. I tried encrypted an clear text passwords. I tried every posible combination i could think of but I simply can not get login page on my site. Can you give me any hint what else I can try.Thanks.

After 2 days I found out that i need to use wrangler to upload project with functions folder.  Anyway is there option to use authentication on one or two pages? Because now i need to exclude all pages and leave only one. And how can I exclude root page?


Describe bug in DEFAULT "cipher string" behavior

Actual behavior of DEFAULT is different than currently described.
Rather than acting as cipher string, DEFAULT cannot be combined using
logical operators, etc.
Fixes #5420.
Checklist

[x] documentation is added or updated

Probably a better alternative to #5428, this describes the DEFAULT keyword mess in terms of BUGS section and also with greater detail (it mentions two particular counter-intuitive examples).

I just realized that the original definition of DEFAULT is ambiguous:

(A) DEFAULT just stands for the list of ciphers that would be selected by ALL:!COMPLEMENTOFDEFAULT:!eNULL, ie. the permanent effect of ! would not apply.

(B) whole expression is just included, ie. the effect is preserved.

So far I've read it as (A), but @kroeckx's comment seems to suggest (B). I'm not sure which one is it.  (Haven't read what the code does.)

So far I've read it as (A), but @kroeckx's comment seems to suggest (B). I'm not sure which one is it.

It's (B).

This version is confusing the non-BUGS and BUGS behaviours, putting everything in BUGS.
The fact that DEFAULT is special, and can only be listed first is NOT a bug.  The fact that "DEFAULTfoo" is treated the same as "DEFAULT:foo" is a BUG, that we might not fix, since it might break existing practice,
Correct behaviour would be to only recognize "DEFAULT" if it is the entire cipherlist or when it is "DEFAULT:rest-of-cipherlist".  All other strings starting with "DEFAULT" are just invalid constructs, and should have been skipped (restarting parsing after the first ":").

So #5428 is closer to the right fix, but is "missing" the "BUGS" documentation, namely, funny syntax of "DEFFAULT+FOO", "DEFAULT-FOO", and "DEFAULTFOO", where these instead become:
"DEFAULT:+FOO", "DEFAULT:-FOO" and "DEFAULT:FOO" respectively.  That's "wrong", but we might need to maintain backwards-compatible buggy parsing.

#5428 was merged in preference of this.  Closing.


PlacesData

all the places must be in a single object, in places where there is a need to divide depending on the tag (like in gone/togo mappings) use filter by tag

also transfer the data to firebase


Implement dpr in loop break to collision detection

implemented dpr to all collisions ref point. (cx, cy, cw, ch)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


crash when transform to json string

Invalid number value (infinite) in JSON write
Invalid number value (NaN) in JSON write
Is there any workaround to avoid this?

我也遇到这个问题，版本3.0.10

@qmkCamel @shannonchou  It's better to provide a demo or code  that can reproduce this issue, Thus ,someone can quickly find the problem and solve it,Thank you!

@qmkCamel Check your code to avoid produce infinite number & NaN number. That is developer's job to do.


make webpack ready

https://github.com/aurelia/webpack-plugin#configuration-options

is imho webpack ready


Support for specifying additional arguments to concord tasks

This provides a mechanism to pass additional arguments to concord. Concord API docs which show how to do this are here: https://concord.walmartlabs.com/docs/api/process.html#form-data
I added some additional tests for the concord work, which didn't appear to have any. When running make test, there was an error (before I made any changes) because go fmt seems to disagree with goimports on the order of things. I updated goimports to no avail.
I'm not certain how to test this with concord. Do y'all have a test harness for that?

Codecov Report

Merging #607 into master will increase coverage by 0.63%.
The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master     #607      +/-   ##
==========================================
+ Coverage   55.00%   55.64%   +0.63%
==========================================
  Files          62       62
  Lines        5303     5314      +11
==========================================
+ Hits         2917     2957      +40
+ Misses       1959     1920      -39
- Partials      427      437      +10

Impacted Files
Coverage Δ

pkg/loadtester/concord.go
29.41% <93.75%> (+29.41%)
:arrow_up:

Continue to review full report at Codecov.

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

When running make test, there was an error

Make sure you're using go 1.14, the fmt changed from 1.13 to 1.14

@stefanprodan I'm happy to make that change. Can you update the docs here that the go version is 1.14, not 1.13? https://docs.flagger.app/dev/dev-guide#setup-dev-environment  I looked for the repo on github, but couldn't find it, else I'd submit a PR for it.

@justinabrahms the dev docs are here https://github.com/weaveworks/flagger/blob/master/docs/gitbook/dev/dev-guide.md

https://github.com/weaveworks/flagger/pull/608 for that other change.

@tebo-ernstberger can you please review this PR? I'm not familiar with concord

cc @ta924


✨ - Export/Import settings

Describe the solution you'd like
It would be very useful to be able to export and import settings.
Describe alternatives you've considered
When trying to export settings, a json file may be created.
Additional context
Also being able to decide which of the settings to choose, to import.

All settings should be stored and synced with ~/.codeedit/preferences.json


HBASE-26809: Report client backoff time for server overloaded

Enable metric reporting on backoff time when a server is overloaded.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 56s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 46s
master passed

+1 :green_heart:
compile
0m 20s
master passed

+1 :green_heart:
shadedjars
3m 53s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 20s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 36s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
3m 48s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
1m 17s
hbase-client in the patch passed.

19m 38s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4729/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4729

JIRA Issue
HBASE-26809

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux be2094ddca86 5.4.0-124-generic #140-Ubuntu SMP Thu Aug 4 02:23:37 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b44bfc52cc

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4729/1/testReport/

Max. process+thread count
199 (vs. ulimit of 30000)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4729/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 20s
master passed

+1 :green_heart:
compile
0m 38s
master passed

+1 :green_heart:
checkstyle
0m 15s
master passed

+1 :green_heart:
spotless
0m 39s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 38s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 16s
the patch passed

+1 :green_heart:
compile
0m 38s
the patch passed

+1 :green_heart:
javac
0m 38s
the patch passed

+1 :green_heart:
checkstyle
0m 14s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
7m 53s
Patch does not cause any errors with Hadoop 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 38s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 46s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

23m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4729/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4729

JIRA Issue
HBASE-26809

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 16bd8c3e0270 5.4.0-124-generic #140-Ubuntu SMP Thu Aug 4 02:23:37 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / b60d2eb0b6

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
64 (vs. ulimit of 30000)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4729/3/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Add support for CT.2 Thermostat

CT.2 thermostats don't seem to return certain array keys, this PR fixes it.

Hi @sebastian-enz Have you tested with with other devices other than the CT2 ?

Hi @tggm unfortunately I only have CT.2 Thermostats, nothing else. But these failed because these thermostats don't seem to return certain array keys, which resulted in a fatal error. I would kindly ask you to check with your own thermostats if they still work after the update, for me, the CT.2 works great now via the Home Assistant integration (which relies on this SDK)


added adjustdeferr action & removed addstate action

Adds adjustdeferr action to change the current deferral percentage
Adds fixassign action to add APPROVED_DEFERRED_PERC_X100 item to existing assignments
Fixes assignment HYPHA token calculation when proposed
Removes unused addstate action

@josejulio Looks like test are failing due to 1 second offset in the vote date, do you know why this could happen?


[Tracker] : Breaking Changes in API

A tracker to collect the breaking changes in API's in the next major release.

resource -> check in https://github.com/kyverno/chainsaw/blob/b01fe35c698d50c78fed66543de12a01f63d80c5/pkg/apis/v1alpha1/file_ref_or_check.go#L11
namespace -> change to a struct https://github.com/kyverno/chainsaw/blob/3f8124483e043f1c123a68595fa0c5dcfde749b0/pkg/apis/v1alpha1/test_spec.go#L31
add metadata to mimick structure of an object https://github.com/kyverno/chainsaw/blob/3f8124483e043f1c123a68595fa0c5dcfde749b0/pkg/apis/v1alpha1/object_reference.go#L8
separate test and config in two different groups
rename podLogs -> logs
group report fields
remove TestStep CRD
support file in delete

You may want to standardize on how to refer to existing cluster resources across all operations so you avoid having to do this:
  - name: 04 - Delete good PDBs
    try:
    - delete:
        ref:
          apiVersion: policy/v1
          kind: PodDisruptionBudget
          name: good-pdb01
  - name: 05 - Update existing Deployments to have one replica
    try:
    - patch:
        resource:
          apiVersion: apps/v1
          kind: Deployment
          metadata:
            name: alpha-busybox
          spec:
            replicas: 1

separate test and config in two different groups

i don't remember why we wanted that 🤔


aiohttp response binary format support

Closing in favor of https://github.com/python-openapi/openapi-core/pull/710


Add optional parameter to fetch response headers

Description
Adds an optional parameter fetchResponseHeaders to getEventsFromJournal. When this parameter is set to true, the result contains the complete response headers in result.responseHeaders.
Example:
{
  "access-control-allow-credentials": [
    "true"
  ],
  "access-control-allow-headers": [
    "X-Requested-With, Content-Type, Authorization, x-api-key, x-request-id"
  ],
  "access-control-allow-methods": [
    "GET, POST, PUT, DELETE, OPTIONS, PATCH"
  ],
  "access-control-allow-origin": [
    "*"
  ],
  "access-control-max-age": [
    "60"
  ],
  "connection": [
    "close"
  ],
  "date": [
    "Wed, 14 Jul 2021 00:31:27 GMT"
  ],
  "link": [
    "</events/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569>; rel=\"events\", </events-fast/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?since=rabbit:e458d64e-a9c9-4c3c-87f7-fd39a2767dc8.penguin:f5544d7b-8bd5-4adf-a3a3-b4c177c46673.ce5880d7-0c45-4fa9-86b4-d64c778361b1.0.1625710722.vjpw1rd0pbvmp38-mevw>; rel=\"next\", </count/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?since=rabbit:e458d64e-a9c9-4c3c-87f7-fd39a2767dc8.penguin:f5544d7b-8bd5-4adf-a3a3-b4c177c46673.ce5880d7-0c45-4fa9-86b4-d64c778361b1.0.1625710722.vjpw1rd0pbvmp38-mevw>; rel=\"count\", </events/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?latest=true>; rel=\"latest\", </events/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?since={position}&limit={count}>; rel=\"page\", </events/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?seek={duration}&limit={count}>; rel=\"seek\", </events-validate/organizations/52381/integrations/183697/5103d8b7-9cbb-4404-97b3-fbe48617a569?since=rabbit:e458d64e-a9c9-4c3c-87f7-fd39a2767dc8.penguin:f5544d7b-8bd5-4adf-a3a3-b4c177c46673.ce5880d7-0c45-4fa9-86b4-d64c778361b1.0.1625710722.vjpw1rd0pbvmp38-mevw>; rel=\"validate\""
  ],
  "retry-after": [
    "10"
  ],
  "server": [
    "openresty"
  ],
  "x-request-id": [
    "MCwsTLWLs5E9WPzFCEkovbRiEWxs1YgL"
  ]
}

Related Issue

https://jira.corp.adobe.com/browse/CI-4613
Motivation and Context

It is useful to have the complete response headers when browsing events from Journaling. These response headers are available when using the Journaling API.
How Has This Been Tested?

Passed all existing unit tests
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@pratyushbanerjee The response to getEventsFromJournal API call already fetches the headers. Why don't we just return response headers always instead of giving a flag to add it to result?

@sandeep-paliwal  it only fetches certain links as part of the response. this option wil add all other headers like request id and other links.


[tcgc] add client accessor support to client.tsp

Currently, TCGC doesn't support public client accessors from parent to subclient.
The canonical use case is something like storage, where there is the access from parent BlobServiceClient to child ContainerClient, with BlobServiceClient.getContainerClient(containerName: string) as a client accessor between the two.
By default, with parents and subclients, unless an explicit client accessor method is defined, there is no generated public accessor method from parent to subclient. If a user wants a public accessor method from parent to subclient, they will define it in client.tsp like so
// client.tsp

op getContainerClient(containerName: string): Azure.Storage.Blob.Container;

If you want to define a client accessor and have access still be internal, the @access decorator can still be applied. This use case would be if there are special internal parameters that you have to pass to the subclient that we don't want users to deal with
TODO

We need to make sure that clients / namespaces / interfaces can be listed as a return type in an operation. This might require working with the typespec team.

I still feel that having an explicit definition is better, instead of going down a logic map way of initialization and accessor. I guess the root of the issue is, besides storage and some other individual examples, we don't have a good feel if a public client has a public subclient, whether we have to add a separate ctor.
The thing with explicitly defining is we don't have to tie ourselves to logic. I don't think we're 100% blocked by typespec support actually, we can return namespaces and interfaces from operations (ex here). Finally, I also think we want to give support to spec authors to explicitly state what parameters should get passed from parent to child etc

First, wondering whether we have different logic for accessor without parameter and accessor with parameter. My following comment is on assumption that we didn't distinguish the 2. Both are just client accessor.
Now, wondering for how many languages, this statement is currently true?

By default, with parents and subclients, unless an explicit client accessor method is defined, there is no generated public accessor method from parent to subclient.

In this default mode, then how the subclient get initialized?
Java actually fit this (as default flatten-client mode), but I think Java is really an exception -- in such mode the subclient is initialized via a Builder of parent client.
I think for .NET, there is always a .getSubClient() from parent client to subclient (at least for cases of no additional parameters required in subclient). @pshao25 Is it correct?
I do not have good recommendation.

If we enforce that service must write op getContainerClient(containerName: string); in client.tsp, it seems redundant in most cases, and won't easily work in unbranded (the default being all subclient not initializable and not usable?)
On the other hand, it may not be easy to devise another "default" mode. For @cilentInitialization, without this decorator there is still some default (is it correct?) for initializing the client. If we had similar "default" in client accessor, it may not be apparent to dev who writes the typespec.

PS: currently Java (when in subclient mode) would not require dev write additional client accessor method. The client accessor method is automatically generated (with a name) parameters in the @cilentInitialization -- logic be: if containerName param is required in @cilentInitialization, it is also a required param in client accessor method.

If we enforce that service must write op getContainerClient(containerName: string); in client.tsp, it seems redundant in most cases, and won't easily work in unbranded (the default being all subclient not initializable and not usable?)

agree. i think in most cases, the explicit definition is redundant. actually, i got some feedback that the way we build our client hierarchy is not so convenient for emitter, as they only need the hierarchy info, no need to know the method info. since different language do have different way to deal with the client ctor or client accessor, i prefer just to construct the hierarchy with parent or children linkage, instead of to redefine the accessor method. with @clientInitialization, user could bump the method parameter to client and make the ctor public. with maybe @clientAccessor, user could redefine the client accessor method and make it public.

i think this is a good topic for today's sync, let me make note of it


Widgets

Summary
We are adding a new Platform 2.0 primitive called widgets.
Documents

Product spec

See more in the canvas for #feat-platform-widgets
Testing

Special notes

Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've ran deno task test after making the changes.

Closing this abandoned feature.


Custom UI Demo and Docs

More public fields to make the UI fully customizable

@AbdulazizAlrabiah could you please prepare ios-pod

Working on it


add information on setting up firebase to the readme

I had to think a bit about what values to put in the .env file, and once I figured it out, I thought it would be helpful to add that information explicitly in the readme so other developers could speed up local development.

and actually, it looks like FIREBASE_MEASUREMENTID isn't even necessary to run locally, so I can add a commit to remove that from the .env.example if you want.


Allow users to set the preferred way of Auto Layout

We should allow users to alternate between different Auto-Layouts (e.g. Horizontal, Vertical, freeform).
This would, in effect, override their existing Step Positions, so there should be a notification to the user to explain that.
On hold until #579 is resolved, as how the positions are stored will change.

Closed in #889


[Android] setTimoute inside componentDidMount never fires.

I am registering a setTimeout inside componentDidMount for several Components . What I observe is , componentDidMount does gets executed. but the setTimeout callbacks never.
componentDidMount() {
    console.log("Option list Mounted")
    setTimeout(() => {
      this.refs.OPTIONLIST.measure(this.getMeasureAndUpdateState);
    },0);
  }

Node Version 4.1.1
ReactNative : 0.16
Thanks.

I cannot reproduce this with RN master.
componentDidMount() {
    setTimeout(
      () => console.log("componentDidMount with timeout!"),
      0
    )
  },

12-14 08:56:24.460 2256-3064/com.scrollviewtest I/ReactNativeJS: componentDidMount with timeout!

@theoriginalgri Register multiple and I am facing this on Android only.. However I will try to setup a example.

@gs-akhan have you looked at https://facebook.github.io/react-native/docs/timers.html#timermixin

We strongly discourage using the global setTimeout(...) and recommend instead that you use this.setTimeout(...) provided by react-timer-mixin. This will eliminate a lot of hard work tracking down bugs, such as crashes caused by timeouts firing after a component has been unmounted.

@gs-akhan Feel free to open this up again if you still have problems when switching to this.setTimeout.

This bug was closed without justification.
The original problem was about the callback not firing at all, rather than firing when it shouldn't.
In addition, React components are not the only parties that might use setTimeout() - third party RN-agnostic libraries might do so too, and there is no way to make them use the mixin without rewriting them.
setTimeout() must do what specs say it should do, that is, actually cause the callback to be fired after the given amount of time with 100% certainty, unless clearTimeout() was used. This bug should be reopened and remain open until this issue is resolved, rather than some vaguely similar issue.


Duplicate ICPSR IDs

Our data assigns the same ICPSR IDs to multiple members for:
1252
1330
9152

This was fixed long ago.


<EMAIL_ADDRESS>cannot be installed though the latest version is 0.4.1 on npm

According to https://www.npmjs.com/package/ts-gyb, the latest version should be 0.4.1.
However,
$ yarn add -D<EMAIL_ADDRESS>yarn add v1.22.10
[1/4] 🔍  Resolving packages...
Couldn't find any versions for "ts-gyb" that matches "0.4.1"

I found a similar ticket (mentioned above) and filed a support ticket to npm.

Got a reply. I tried npm install<EMAIL_ADDRESS>according to their suggestion and it works.
Replied to ask the reason.

I also encountered this. I thought it was a cache issue because I was trying minutes after I published. Specifying the version worked.

According to the linked issue mentioned above, it should be related to cache.
Let's move forward with #62 and then publish a new version.

Does the version v0.4.2-rc1 still have this issue?

Does the version v0.4.2-rc1 still have this issue?

No. But I'd like to wait one more day for their reply.

The npm team helped us to refresh the cache and the issue should be resolved.


Can the #include not be a syntax error at least to allow formatted to still work?

Hello,
in my engine as I suppose many people did over the years, I retrofitted a #include support same way as it works in C++.
But I realized now after trying this plugin out that my way will make the formatted to not work because #include is detected as a syntax error.
So, well, I guess would be amazing to jsut allow #include or at least not make it a syntax error forbidding formatter to work.
What do you think? Maybe I should adapt otherwise? I tried with #pragma include and it works but looks very ugly with the red " highlighted... So for now I resorted to ///include but it is misleading.
Thanks!


update mir version

Looks good to me. I don't have merge rights on this repo for some reason.
A new release would be very useful, 0.3.0 because upgrading mir is a breaking change.
@atilaneves could you do the honours?

Merged, pushed, and tagged.

Thankyou both


Mapbox source tiles not rendering

Fairly new to Maputnik but recently, I've noticed that I'm unable to render tiles when choosing Mapbox Sources (ie., mapbox://mapbox.mapbox-streets-v7).The map is empty at all zoom levels. Inspect Mode shows no layers either.  OpenMapTiles are working fine. I've tried across all browsers though I'm primarily working in Chrome.
I'm fairly certain this was working for me a few days ago but not anymore.  Am I doing something wrong or is this a bug? Thanks in advance for any help.

Hi @jonathanlevy,

I'm unable to render tiles when choosing Mapbox Sources

this is because we don't have currently a Mapbox access token available, this is tracked in #385.
With the current Maputnik online version you may use your own MB access token via "Style Settings".
Make sure to close the modal using the close button (cross (X) on the top right corner).

It could be that you need to refresh the browser (F5) afterwards.

Thank you. Apologies for the duplicate, I didn't see that before. I think it was a matter of using the "x" to close for me.

No worries, thanks for your feedback!

I think it was a matter of using the "x" to close

Yes, this behaviour is indeed confusing. There is already a fix for it (#416 / #418) which will included in the next release.


Invalid content /en/testnets/cardano/get-started/installing-and-running-the-cardano-node/

Where does the issue occur?
https://developers.cardano.org/en/testnets/cardano/get-started/installing-and-running-the-cardano-node/
Describe the issue
Hyperlinks of the 4 ways of building a node are broken; the target destination pages, in fact, exists, but are not correctly codified the url's in the hyperlinks
Expected changes
Fix the following 4 hyperlinks to the correct urls
Building a node using Nix
Building a node from source
Running a node using Docker
Running a node on an AWS instance

Solved here

Thank you for your feedback, the above links were fixed, and one remaining is being fixed in PR #440

Thank you for your feedback, the above links were fixed, and one remaining is being fixed in PR #440


[Gluten-4732][CH] delta-mergetree support update/delete/upsert/insert in a more native delta way

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #4732)
How was this patch tested?
(Please explain how this patch was tested.


Provision light-client state in mu-ra; fix w2 startup

This implementation should actually be independent of the polkadot-v0.9.42 update, I will also rebase this branch on master and file a standalone pr.

I can't explain currently, why I see the following erros in the failing integration tests:
[init ../../../psw/ae/aesm_service/source/core/ipc/UnixCommunicationSocket.cpp:225] Failed to connect to socket /var/run/aesmd/aesm.socket
[2023-08-01T05:58:25Z ERROR integritee_service::ocall_bridge::ffi::init_quote] [-]  Failed to init quote: InitQuote(SGX_ERROR_SERVICE_UNAVAILABLE)
[2023-08-01T05:58:25Z ERROR itp_stf_executor::executor] Fatal Error. Failed to attempt call execution: Sgx(SGX_ERROR_SERVICE_UNAVAILABLE)

[init ../../../psw/ae/aesm_service/source/core/ipc/UnixCommunicationSocket.cpp:225] Failed to connect to socket /var/run/aesmd/aesm.socket
[2023-08-01T05:58:25Z ERROR integritee_service::ocall_bridge::ffi::init_quote] [-]  Failed to init quote:
InitQuote(SGX_ERROR_SERVICE_UNAVAILABLE)
[2023-08-01T05:58:25Z ERROR itp_stf_executor::executor] Fatal Error. Failed to attempt call execution: >Sgx(SGX_ERROR_SERVICE_UNAVAILABLE)

This sometimes happens, because we are really pushing those machines when running the tests in parallel, and we suspect that the  aesmd service might not be really suited for this type of workload. Rerunning usually solves it. (The solution would be to have as much runners as needed but this error rarely happens.)

Alright, thanks a lot! 👍

Rebased on master and merge into master here #1399


Parameterizing localization strings with arguments

Following this example from wiki @key|only|@arg I'm trying to pass argument to localized string as parameter:
<vislabel>@key|{arg}</vislabel>

But as result parser searches for key|{arg} key in bundle (and crushes when not found).
Am I missing something?

Could you provide a code sample or a small project that recreates the issue? Or paste the complete stack trace of the exception? When I used this snippet in gdx-lml-tests, it worked fine:
<:argument key="sample" value="test"/>
@bundleLineWithArgument|{sample}

This prints Received argument: test. as expected.

@metaphore Is this still an issue?

@czyzby sorry, had no time to prepare code sample and check it once again. Will do it till the end of this week

Not a problem, take your time.

I had few experiments but for good (or bad) can't reproduce it anymore. I think it should be closed now and if problem will ever occur, I will supply you with details in that issue.
BTW, is there a way to conjunct two expressions in one line like: @someString{someValue}?

BTW, is there a way to conjunct two expressions next to each other like: @someString{someValue}?

I think that in this particular example it would currently replace someValue and look for I18N line named someStringSomeValueResult.
But labels are not multiline by default (you have to enable it first), so something like this:
@someString
{someValue}

...would result in a single string, joining the two inputs.

@czyzby probably I found what is origin problem was about. If I try to parametrize localized string with a value started with . (e.g. file extension) it results in
I18N bundle not found for bundle line: @key.arg

And I guess it's pretty obvious why - LML treats key as a bundle name. But still it looks like there is no way to put any value that starts from dot. May be is it possible to add some kind of escape character for such case?

Can you post a self-contained example of a LML template and the bundle line argument that reproduce the issue, so i can easily test it? I'll look into it before the next release.
Anyway, I suggest using a workaround for now, like using a Java method for building this particular string or avoiding parametrization (if possible) when an argument with dot is present.

Sure, thanks. I believe this should be enough to reproduce the case.
bundle.properties
key = text {0} text

template.lml
<?xml version="1.0"?>
<!DOCTYPE table SYSTEM>

    <label<EMAIL_ADDRESS>


Missing command error handling

currently if there is missing command
for example from tests https://github.com/zio/zio-cli/blob/c96393348bb58102cc8b37caaed7b8d3585f8dd8/zio-cli/shared/src/test/scala/zio/cli/CommandSpec.scala#L148
CliApp
https://github.com/zio/zio-cli/blob/c96393348bb58102cc8b37caaed7b8d3585f8dd8/zio-cli/shared/src/main/scala/zio/cli/CliApp.scala#L119
cmd
git abc

will fail on error like
Missing command name: rebase

error message is confusing
also, it is look like there is not way for custom error handling in CliApp or for Command.parse  recovery, to do something like: print error message and show help

/bounty $100


Localization with Areas

I've been searching a lot on how to use Localization and Resource Files when Areas are used. I have multiple cases in the web app we're creating where I need to have localizable content but it's located in an area. A concrete example we have is a controller in an area "Company". The path to access the controller is then : "Solution/Areas/Company/Controllers/HomeController.cs". My resources folder has the following structure and doesn't seem to work : "Resources/Areas/Company/Controllers/HomeController.resx" and it doesn't seem to find my resource file. Any help in regards to that?
Also, maybe unrelated, but the _resourceBaseName of the ResourceManager that is created is "Solution..Areas.Company.etc.". Are the two dots normal behaviour?

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

ID: d1833676-ff1b-c587-cc9c-81afffa8ef68
Version Independent ID: ad989395-3f49-4d55-521e-cf6fe56281ee
Content: Globalization and localization in ASP.NET Core
Content Source: aspnetcore/fundamentals/localization.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hi guys! Thank you for the help and sorry for the disturbance, but I solved my issue.
My main problem all came from this bad boy :
services.AddLocalization(options =>             {                 options.ResourcesPath = "/Resources";             });
Removing the "/" resolved my problem.
For those curious, my Resource file structure is : "[Root]/Resources/Areas/{MyAreaName}/Controllers/{MyControllerName}.resx.
In summary, I "copied" the same folder structure of my areas.
Again, thank you for your help.
Might I suggest updating the doc to include the areas as a possible scenario. I may not be the only one that face this question.

@hishamco  should we update and say something about areas?
@boppercr  what do you think we should add to the doc for areas?

In the section "Resource file naming", something along the lines of : "The relative path from your root to the file you want localized must be reflected in the Resources folder." And maybe follow that by a couple of examples that are most likely to happen, like a Model, a View with and without areas.
"/Resources/Models/Home/IndexViewModel.resx"
"/Resources/Views/Home/Home.resx"
"/Resources/Areas/MyArea/Controllers/HomeController.resx"
"/Resources/Areas/MyArea/Models/Home/IndexViewModel.resx"
Also, in the same section, it might be good to specify not putting a "/" in front of the ResourcePath = "Resources" since it will break everything.
Thanks again.

@boppercr  thanks for the feedback and glad you solved your problem.

Ya no need to have a /, so no action needed here @Rick-Anderson


Fix body parameter for octect-stream and byte[] parameter

If a method consumes an octect-stream the the schema for a byte[] body parameter must be changed.
before
"schema" : {             "type" : "array",             "items" : {               "type" : "string",               "format" : "byte"             }           }
after
"schema" : {             "type" : "string",             "format" : "byte"           }
With this changeset, the created C# Client works as expected.

@ChristianLutz This PR looks good. Please resolve the conflicts so I can merge it.

Hello @who I tried to rebase my changes to the current master. I will delete this PR and create a new one.


Add scrollbar on the side and reduce the height of the panel

Currently, if they are too many audio streams open, the height of the menu becomes too big. That's for sure depending on the screen resolution/size but still. I would really appretierte if there is a scrollbar or a way to somehow decrease the height of the menu. Furthermore as you can see on the screenshot below, this prevents expanding the menus on the bottom e.g. wifi, bluetooth.

Same issure. Can you move it inside a menu or something similar