more than 1 promise .catch() causing query to be executed more than once

Environment
Knex version:
from knex=0.12.9 + pg=6.1.2
to knex=0.15.2 + pg=7.4.3 (included)
OS: osx, windows, linux
Select applicable tempalate from below.
If issue is about oracledb support tag @ atiertant. For MSSql tag @ smorey2 .
Rest of dialects doesn't need tags.
Bug

Explain what kind of behaviour you are getting and how you think it should do
Chaining multiple .catch() traps, for a total of Ncatch, to insert() causes the insert to be repeated for (Ncatch -1) + 1 times (the -1 being the catch it skips, and +1 been the expected one we actually created).
Expected behaviour: chaining multiple catches to a promise interface should behave like a promise without weird side effetcs to the query.

Error message
If the target table has constrain than we'll have a duplicate insert error, otherwise we'll just find multiple entries in out tables without any evidence of what caused it.

Reduced test code, for example in https://npm.runkit.com/knex or if it needs real
database connection to mysql or postgresql, then single file example which initializes
needed data and demonstrates the problem.

Consider the following code:
const knex = require('knex');

const cred = {
    dialect: 'postgres',
    debug: true,
    connection:{
        host: 'localhost',
        port: '5432',
        user: 'docker',
        password: 'docker',
        database: 'c8_accounts',
        ssl: false
    },
    pool: { min: 5, max: 25 }
};

const conn = knex(cred);
const tableName = 'clients';
const data = { name: `whatever-${Date.now()}`, status: 'whatever-test' };

const writer = conn(tableName).insert(data);
writer.then((result) => {
    console.log(`add record to ${tableName} (%o) result: `, data, result);
    return result;
}).catch((error) => {
        console.error(`add record to ${tableName} (%o) error: `, data, error);
    return Promise.reject(error);
});

This has the expect output (debug: true):
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532357467509', 'whatever-test' ],
  __knexQueryUid: '5a5cdebd-7b09-43e0-9b96-fa1f7aff9348',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }

add record to clients ({ name: 'whatever-1532357467509', status: 'whatever-test' }) result:  Result {
  command: 'INSERT',
  rowCount: 1,
  oid: 0,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }

Now if we add another catch statement (ex: writer.catch((error) => console.log('just because: ', error))):
const knex = require('knex');

const cred = {
    dialect: 'postgres',
    debug: true,
    connection:{
        host: 'localhost',
        port: '5432',
        user: 'docker',
        password: 'docker',
        database: 'c8_accounts',
        ssl: false
    },
    pool: { min: 5, max: 25 }
};

const conn = knex(cred);
const tableName = 'clients';
const data = { name: `whatever-${Date.now()}`, status: 'whatever-test' };

const writer = conn(tableName).insert(data);
writer.then((result) => {
    console.log(`add record to ${tableName} (%o) result: `, data, result);
    return result;
}).catch((error) => {
        console.error(`add record to ${tableName} (%o) error: `, data, error);
    return Promise.reject(error);
});

writer.catch((error) => console.log('just because: ', error))

Suddenly something unexpected happens:
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532357719493', 'whatever-test' ],
  __knexQueryUid: 'f7ced535-6b3c-440a-8e37-527a6453bc72',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532357719493', 'whatever-test' ],
  __knexQueryUid: '9edfa735-37e6-4684-ae72-4930045eefa2',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }

the knex-provided debug output has now 2 jobs to do (with different __knexQueryUid, so they are actually different things)
and we get both the success log line:
add record to clients ({ name: 'whatever-1532357719493', status: 'whatever-test' }) result:  Result {
  command: 'INSERT',
  rowCount: 1,
  oid: 0,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }

as well as the 2nd catch output (please note the 1st catch was ignored):
just because:  { error: duplicate key value violates unique constraint "clients_name_key"
    at Connection.parseE (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:277:12)
    at readableAddChunk (_stream_readable.js:262:11)
    at Socket.Readable.push (_stream_readable.js:217:10)
    at TCP.onread (net.js:638:20)
  name: 'error',
  length: 217,
  severity: 'ERROR',
  code: '23505',
  detail: 'Key (name)=(whatever-1532357719493) already exists.',
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: 'public',
  table: 'clients',
  column: undefined,
  dataType: undefined,
  constraint: 'clients_name_key',
  file: 'nbtinsert.c',
  line: '433',
  routine: '_bt_check_unique' }

The more catch() we add:
const knex = require('knex');

const cred = {
    dialect: 'postgres',
    debug: true,
    connection:{
        host: 'localhost',
        port: '5432',
        user: 'docker',
        password: 'docker',
        database: 'c8_accounts',
        ssl: false
    },
    pool: { min: 5, max: 25 }
};

const conn = knex(cred);
const tableName = 'clients';
const data = { name: `whatever-${Date.now()}`, status: 'whatever-test' };

const writer = conn(tableName).insert(data);
writer.then((result) => {
    console.log(`add record to ${tableName} (%o) result: `, data, result);
    return result;
}).catch((error) => {
        console.error(`add record to ${tableName} (%o) error: `, data, error);
    return Promise.reject(error);
});

writer.catch((error) => console.log('just because1: ', error))
writer.catch((error) => console.log('just because2: ', error))
writer.catch((error) => console.log('just because3: ', error))
writer.catch((error) => console.log('just because4: ', error))

the more times knex tries to execute our query 5 times (4 additional catches + our insert):
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358106864', 'whatever-test' ],
  __knexQueryUid: '3483dba3-c371-48bc-a6f2-0528ae7f5805',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358106864', 'whatever-test' ],
  __knexQueryUid: '8657dbeb-f192-47e5-a511-aed25f7af91e',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358106864', 'whatever-test' ],
  __knexQueryUid: '58eee2a1-e39c-486c-a563-23c62eb95a73',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358106864', 'whatever-test' ],
  __knexQueryUid: '185e8e27-e5f2-482c-a12d-fdd376a5dd32',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358106864', 'whatever-test' ],
  __knexQueryUid: '21945a1d-0b35-45c4-a1c9-dd2be76e68b5',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }

(please not how all the __knexQueryUid are different)
and we still get our success:
add record to clients ({ name: 'whatever-1532358106864', status: 'whatever-test' }) result:  Result {
  command: 'INSERT',
  rowCount: 1,
  oid: 0,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }

as well as a (totalCatches)-1 number of failure outputs:
just because2:  { error: duplicate key value violates unique constraint "clients_name_key"
    at Connection.parseE (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:277:12)
    at readableAddChunk (_stream_readable.js:262:11)
    at Socket.Readable.push (_stream_readable.js:217:10)
    at TCP.onread (net.js:638:20)
  name: 'error',
  length: 217,
  severity: 'ERROR',
  code: '23505',
  detail: 'Key (name)=(whatever-1532358106864) already exists.',
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: 'public',
  table: 'clients',
  column: undefined,
  dataType: undefined,
  constraint: 'clients_name_key',
  file: 'nbtinsert.c',
  line: '433',
  routine: '_bt_check_unique' }
just because4:  { error: duplicate key value violates unique constraint "clients_name_key"
    at Connection.parseE (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:277:12)
    at readableAddChunk (_stream_readable.js:262:11)
    at Socket.Readable.push (_stream_readable.js:217:10)
    at TCP.onread (net.js:638:20)
  name: 'error',
  length: 217,
  severity: 'ERROR',
  code: '23505',
  detail: 'Key (name)=(whatever-1532358106864) already exists.',
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: 'public',
  table: 'clients',
  column: undefined,
  dataType: undefined,
  constraint: 'clients_name_key',
  file: 'nbtinsert.c',
  line: '433',
  routine: '_bt_check_unique' }
just because3:  { error: duplicate key value violates unique constraint "clients_name_key"
    at Connection.parseE (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:277:12)
    at readableAddChunk (_stream_readable.js:262:11)
    at Socket.Readable.push (_stream_readable.js:217:10)
    at TCP.onread (net.js:638:20)
  name: 'error',
  length: 217,
  severity: 'ERROR',
  code: '23505',
  detail: 'Key (name)=(whatever-1532358106864) already exists.',
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: 'public',
  table: 'clients',
  column: undefined,
  dataType: undefined,
  constraint: 'clients_name_key',
  file: 'nbtinsert.c',
  line: '433',
  routine: '_bt_check_unique' }
just because1:  { error: duplicate key value violates unique constraint "clients_name_key"
    at Connection.parseE (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:553:11)
    at Connection.parseMessage (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:378:19)
    at Socket.<anonymous> (/Users/lucio/tmp/dbexp/node_modules/pg/lib/connection.js:119:22)
    at Socket.emit (events.js:182:13)
    at addChunk (_stream_readable.js:277:12)
    at readableAddChunk (_stream_readable.js:262:11)
    at Socket.Readable.push (_stream_readable.js:217:10)
    at TCP.onread (net.js:638:20)
  name: 'error',
  length: 217,
  severity: 'ERROR',
  code: '23505',
  detail: 'Key (name)=(whatever-1532358106864) already exists.',
  hint: undefined,
  position: undefined,
  internalPosition: undefined,
  internalQuery: undefined,
  where: undefined,
  schema: 'public',
  table: 'clients',
  column: undefined,
  dataType: undefined,
  constraint: 'clients_name_key',
  file: 'nbtinsert.c',
  line: '433',
  routine: '_bt_check_unique' }

This erratic behaviour can be prevented by simply coercing insert's to a canonical native Promise:
const knex = require('knex');

const cred = {
    dialect: 'postgres',
    debug: true,
    connection:{
        host: 'localhost',
        port: '5432',
        user: 'docker',
        password: 'docker',
        database: 'c8_accounts',
        ssl: false
    },
    pool: { min: 5, max: 25 }
};

const conn = knex(cred);
const tableName = 'clients';
const data = { name: `whatever-${Date.now()}`, status: 'whatever-test' };

const writer = new Promise(function (resolve, reject) {
    conn(tableName).insert(data).then(resolve).catch(reject);
});

writer.then((result) => {
    console.log(`add record to ${tableName} (%o) result: `, data, result);
    return result;
}).catch((error) => {
        console.error(`add record to ${tableName} (%o) error: `, data, error);
    return Promise.reject(error);
});

writer.catch((error) => console.log('just because1: ', error))
writer.catch((error) => console.log('just because2: ', error))
writer.catch((error) => console.log('just because3: ', error))
writer.catch((error) => console.log('just because4: ', error))

and we're back to an expected output:
{ method: 'insert',
  options: {},
  timeout: false,
  cancelOnTimeout: false,
  bindings: [ 'whatever-1532358850252', 'whatever-test' ],
  __knexQueryUid: '5b781898-eee5-4fdf-a11d-c9011cc1d4c3',
  sql: 'insert into "clients" ("name", "status") values (?, ?)',
  returning: undefined }
add record to clients ({ name: 'whatever-1532358850252', status: 'whatever-test' }) result:  Result {
  command: 'INSERT',
  rowCount: 1,
  oid: 0,
  rows: [],
  fields: [],
  _parsers: [],
  RowCtor: null,
  rowAsArray: false,
  _getTypeParser: [Function: bound ] }

Long story short, this is pretty much like having something simple like:
function executeInsert(table, data) {
    return conn(table).insert(data)
    .catch((error) => {
        console.error('DB ERROR: ', error);
        return Promise.reject(error);
    });
}

function addSomething(data) {
    return executeInsert('myTable', data)
        .catch(function(error) {
            console.error('Cannot add something. Error: ', error);
        })
}

addSomething({name: 'nothing'}).then(function (result) {
    console.log('Somehing was inserted: ', result);
})

unexpectedly adding something a number of times.

When you call .catch() on a Promise you're not "adding" a generic error handler that will only run when there is an error or something, you're actually running this: .then(null, function() { /* ... */ }), so what you are seeing is perfectly expected since you're running multiple queries in parallel, each one started when you call .catch() on the insert Promise.
What you really want is this:
writer.then((result) => {
    console.log(`add record to ${tableName} (%o) result: `, data, result);
    return result;
}).catch((error) => {
        console.error(`add record to ${tableName} (%o) error: `, data, error);
    return Promise.reject(error);
}).catch((error) => console.log('just because: ', error))

You can chain .catch() calls to eternity like that and only one query will run, but if you call .catch() on writer it will simply start another Promise chain in parallel with the first one.

@ricardograca thanks for your answer, however catching and throwing between catches in a promise is something I use quite a bit (usually for logging purposes) and I never observed a behavior like this.
Also: how do you explain wrapping it in a "js native" promise fixes the problem?

Not really sure, but maybe it's related to the use of Bluebird in Knex instead of using native Promises.

Unless I'm mistaken this is primarily because knex interface is a "thenable", not a "real promise" if you will. For example if you call .then on a query builder multiple times it may not always yield the same result, since it actually does run the query every time its called, unlike a "real promise" which would immediately resolve with the original result.
See here and here.
It has been lifted as an issue multiple times before. I believe in @tgriesser 1.0 refactor branch this was changed to more accurately fulfill the promise spec, right?

Yep, you are calling then / catch for query builder multiple times. Its pretty much completely by valid functionality according to Promise A+ specs for thenable. If that behaviour is changed it will actually make using await with knex harder, because you wouldn't be able to execute the same query builder multiple times by awaiting it. There are many issues about this where all the effect are explained better for example here https://github.com/tgriesser/knex/issues/1952#issuecomment-285636299.
I really hope that this will not be changed for 1.0 release. Closing as duplicate.

Some related issue also https://github.com/tgriesser/knex/issues/2434


Revise Sweetalert asset load

Use Hugo Pipes to load the Sweetalert script when needed. One benefit is that the build would now fail if the script is missing, rather than silently missing some site behavior.
/kind cleanup
/area web-development

/approve
Thanks!


Update to readable-stream 3.0.0

Closes #17
Closes #22
Closes #24

rs@3 is significantly different in the browser and in node so we probably need some browserified tests here. it also drops support for some browser versions but i think its support is now the same as Buffer's so that's ok.

Anyone reviewing this? readable-stream v3 is now 3.0.6 btw.

there are still some problems with this unfortunately. i had some runtime crashes involving util.inspect.custom (which IIRC is not yet available in browserify's util)

status update: saucelabs approved the request quickly, but I ran into some trouble claiming it. I only just finally emailed them about it, hopefully it can be resolved soon!

@goto-bus-stop would you consider forking and publishing this update yourself? I'm currently looking into updating these builtins in jspm (https://github.com/jspm/jspm-core) and this would be great to avoid the duplication that now comes into effect since http has upgraded.

Saucelabs says yes! :package<EMAIL_ADDRESS>

Amazing, persistence pays off :)


mkFlake/options: add pathToOr for modules options

I still prefer the modules = ./modules/module-list.nix pattern, this makes that possible without having to use import.

bors try

bors r+

bors r+

I hope so too, I figure if overlays has the same thing, we should be ok. And we used to do this. I checked all the flake outputs too to make sure everything is there.


Xnero: Watch myhalo.life

Xnero requests the watch of the watch_keyword myhalo\.life. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

myhalo\.life has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


Categories with pipe suffix

Categories with pipe suffix
Fixes {GitHub issue number #1826 and title Categories with pipe suffix}
Description (required)
Certain Categories in MediaDetailsFragment used to show up suffixed with pipe. Because of which , when such categories were clicked, the associated images did not show up, because server could not find any images associated because of the pipe suffix.
Fixes {GitHub issue number #1826 and title Categories with pipe suffix}
Changes made :
-Certain category names used to show suffixed with strings prefixed with pipe '|'. Removed everything after the pipe. As per the discussion on the thread, its safe to remove everything after the pipe, including the pipe
{Describe the changes made and why they were made.}
As a fix, I have looped the category names list and and for each category name if it contains a pipe '|'. It removes everything from the string proceeding the pipe including the pipe. As a result of which, the categories would be shown without the pipe and the clicks on them would show appropriate images.
Tests performed (required)
Tested on {27 & name of Pixel 2}, with {build variant, ProdDebug}.
Screenshots showing what changed (optional)

Codecov Report

Merging #1873 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1873      +/-   ##
=========================================
- Coverage    3.61%   3.61%   -0.01%
=========================================
  Files         188     188
  Lines        9839    9842       +3
  Branches      873     874       +1
=========================================
  Hits          356     356
- Misses       9457    9460       +3
  Partials       26      26

Impacted Files
Coverage Δ

...fr/free/nrw/commons/media/MediaDetailFragment.java
0% <0%> (ø)
: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 3f52211...13f8fba. Read the comment docs.

Thanks for PR @ashishkumar468 , can you please include issue title to PR title too? It would be a lot easier for tester while checking all PRs from list:)

@neslihanturan I have made the required changes, if you could review :)

@neslihanturan Bump?

Ready, works for me. Thanks @ashishkumar468 !


[FEATURE] Add cell settings

Description

Add cell settings to Table panel. It's allowing users to custom cells depending of the cell value. For now, user can change the display name, the text color and the background color.
Screenshots

Checklist

[x] Pull request has a descriptive title and context useful to a reviewer.
[x] Pull request title follows the [<catalog_entry>] <commit message> naming convention using one of the
following catalog_entry values: FEATURE, ENHANCEMENT, BUGFIX, BREAKINGCHANGE, DOC,IGNORE.
[x] All commits have DCO signoffs.

UI Changes

[x] Changes that impact the UI include screenshots and/or screencasts of the relevant changes.
[x] Code follows the UI guidelines.
[x] Visual tests are stable and unlikely to be flaky.
See Storybook
and e2e docs for more details. Common issues
include:

Is the data inconsistent? You need to mock API requests.
Does the time change? You need to use consistent time values or mock time utilities.
Does it have loading states? You need to wait for loading to complete.

Hi! I see that this feature implements "value mapping" but is it only for table panel ? Or would it be possible to make it more generic to use, for example, in StatPanel, and possibly other panels ? For example, In Grafana schema valueMapping is a univesal property that can be applied to almost any panel.
P.S. We are trying to implement statusHistory panel which also should need this functionality.

Opened #2337 for tracking common mapping value feature


Fleet is unreachable after VM is suspended/restored

Closing as there's no great solution here and the workaround is documented on the README

I am new and posting this for others for clarity.  When I provisioned my logger vm, the https://<IP_ADDRESS>:8412 service was not reachable.  I ran the commands above and the service became reachable.


Could not install package sqlite-net-pcl on NETFramework,Version=v4.6

I'm trying to install sql-net-pcl into my project. But I get an error:
Could not install package 'sqlite-net-pcl 1.7.335'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.6', but the package does not contain any assembly references or content files that are compatible with that framework.
How can I fix this?

I tried installing 'sqlite-net-standard' and it worked on my .net462 project.


[5.8] add new lines to empty rows in multi-line syntax highlighted code

Explanation: Fixes an issue where empty new lines are not rendered properly.
Scope: Small JS change that improves syntax highlighted code
Issue: rdar://103087953
Risk: Small potential for regressions with multiline highlighted code listings
Testing: Added unit test and visually inspected formatting.
Reviewer: @mportiz08
Original PR: #502

@swift-ci test


How to setup dynamic mask?

Looking for dynamic brazilian phone like http://codepen.io/lagden/pen/meemqG as mentioned at https://github.com/BankFacil/vanilla-masker
How can I configure this with VueJS directive?

We do not support dynamic mask in Awesome Mask, we have removed this feature.
Here at Moip we are using two inputs with v-if instead of one with dynamic mask.
We think that change the mask on the fly is not good to  UX.
https://github.com/moip/awesome-mask/commit/0f4f1ddc68914ccfa59c8f1f33974fa6590ef899
We are also having problems to support the Samsung predict keyboard with dynamic mask.

Hi man, Could you give me an example about how do you use these two inputs with v-if instead of a dynamic mask?

+1

<button @click="select('cpf')">CPF</button>
<button @click="select('cnpj')">Cnpj</button>
<input type="text" v-show="isCpf" v-mask="'999.999.999-99'" >
<input type="text" v-show="!isCpf" v-mask="'99.999.999/9999-99'" >

I can`t share the design yet, but its simple like this.
@JoohnCruz @alexsandro-xpt

@iCaio it's not a better example. A better exemple is with brazilian phones when thoses may have the optional 9 number front of others number.

@alexsandro-xpt We use two fields, our designers makes the design with this limitation in mind.

@iCaio I'm not right about that...

I am using as of below:
<input mask="telefoneMask">

computed: {
    // Define máscara do número do telefone
   telefoneMask() {
       return this.telefone.tipo_telefone_id == 1 ? '(99) 999 99-9999' : '(99) 9999-9999';
   }
},

Hope it helps!

@evertramos What is the range types of this.telefone.tipo_telefone_id ? Type 1 for Brazil? Type 2 for others?

@alexsandro-xpt this.tleefone.tipo_telefone_id type 1 it´s like mobile phone number format and the rest would be home phone number format (brazilian's format).

@evertramos Alright, but We know, that only fix home number is 8 chars, and work phone or commercial phone can be a dynamics 8 or 9 chars.
And now, this is a problem.

@alexsandro-xpt I see what you mean... not sure if length would work on that... sorry not being able to help.

@evertramos Vanilla Mask has dynamic mask https://github.com/vanilla-masker/vanilla-masker#dynamic-mask here is sample for brasilian phone numbers.

@alexsandro-xpt also awesome-mask since 31March https://github.com/moip/awesome-mask/releases/tag/0.4.2

@caioincau and @joohncruz I fix this issue removing set maxlength attribute at those lines:
https://github.com/moip/awesome-mask/blob/fb3de2e8d6dc6154f92da2a1c5995c23f80cfb58/src/index.js#L21-L22
https://github.com/moip/awesome-mask/blob/fb3de2e8d6dc6154f92da2a1c5995c23f80cfb58/src/index.js#L49
https://github.com/moip/awesome-mask/blob/fb3de2e8d6dc6154f92da2a1c5995c23f80cfb58/src/index.js#L57
And creating this method at my component for @input="numberInput" input event:
numberInput (ev) {
  this.$v.number.$touch();
  if (ev.length > 14) {
    this.fields.number.mask = '(99) 99999-9999';
  } else {
    this.fields.number.mask = '(99) 9999-9999';
  }
}

@alexsandro-xpt the maxlength is needed to work well in Samsung Internet with Samsung Prediction keyboard.

@alexsandro-xpt also awesome-mask since 31March https://github.com/moip/awesome-mask/releases/tag/0.4.2

Can you give me an example of use?

We wanted people to be able to input mobile phones or home phones on the same input since we had tested previously that multiple fields or a button to choose what kind of phone input had worse conversion rates.
This is our workaround using this library:
computed: {
    phone: {
      set(value) {
        this.rawPhone = value

        if (value.length === 14) {
          this.rawMask = '(99) 9999-99999'
        } else if (value.length > 14) {
          this.rawMask = '(99) 99999-9999'
        } else {
          this.rawMask = '(99) 9999-9999'
        }
      },

      get() {
        return this.rawPhone
      }
    }
}


r11 Deprecation warnings on Chrome 63 /  GainNode.gain.value setter smoothing  and  OscillatorNode.frequency.value setter smoothing

I don't know if it requires any action. Just in case, I'm opening this ticket.
[Deprecation] GainNode.gain.value setter smoothing is deprecated and will be removed in M64, around January 2018. Please use setTargetAtTime() instead if smoothing is needed. See https://www.chromestatus.com/features/5287995770929152 for more details.
[Deprecation] OscillatorNode.frequency.value setter smoothing is deprecated and will be removed in M64, around January 2018. Please use setTargetAtTime() instead if smoothing is needed. See https://www.chromestatus.com/features/5287995770929152 for more details.

Thanks for the note! I actually opened a bug about this with Chrome about a year and a half ago. The setter smoothing they had implemented until now was not defined in the specification and was usually not the desired behavior, so i'm glad to finally see it corrected.


npm install still fails: Invalid tag name ">=^1.0.0"

#22 & #23 were closed but this bug still very much exists.
$ npm i @k6-contrib/list-plugins
npm ERR! code EINVALIDTAGNAME
npm ERR! Invalid tag name ">=^1.0.0": Tags may not have any characters that encodeURIComponent encodes.

Workaround: npm i --legacy-peer-deps @k6-contrib/list-plugins

I think this is resolved by my PR https://github.com/keystonejs-contrib/k6-contrib/pull/27

this is closed in #29


[bug]: Unable to install 3.2.0: InvokeAI-Installer[onnx]' is not installable.

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
cuda
VRAM
6GB
What version did you experience this issue on?
3.2.0
What happened?
Install crashes after a while, giving me this weird error...
I'm doing a clean install, having deleted a previous version of InvokeAI.
pip 23.2.1 is installed in my system's global pip installation, for some reason it doesn't detect it...
Screenshots

Additional context
No response
Contact Details
<EMAIL_ADDRESS>

Ive had the same issue, when installing from the Invokeai/installer directory.
For me I could fix it by running:
/home/<USER>/invokeai/.venv/bin/pip install --require-virtualenv --use-pep517 '/home/<USER>InvokeAI-3.2.0[onnx]' --extra-index-url https://download.pytorch.org/whl/rocm5.4.2

As it seems that it was linking to the InvokeAI/installer directory instead of the InvokeAi directory which houses the pyproject.toml needed for the install.

Ive had the same issue, when installing from the Invokeai/installer directory. For me I could fix it by running:
/home/<USER>/invokeai/.venv/bin/pip install --require-virtualenv --use-pep517 '/home/<USER>InvokeAI-3.2.0[onnx]' --extra-index-url https://download.pytorch.org/whl/rocm5.4.2

As it seems that it was linking to the InvokeAI/installer directory instead of the InvokeAi directory which houses the pyproject.toml needed for the install.

Helping someone with this issue, any idea what this would look like for windows??
Thanks in advance!

The installer and install docs have been redone for v4. Please create a new issue if the problem persisted with the latest release (v4.1.0).


lost layout and setting after reboot

Hi wida! once rebooted (from button on wemos) or when powering it off the wemos came up with the right zone but in manual mode.
mqtt data (are storage expect the server ip), so it need to be inserted again.
Did i flashed it wrong? or is a know issue?
thank you

Let's continue the discussion here:: https://github.com/widapro/wledPixel/issues/24
Try the new beta version from the link above


Add Cash for tower defence isnt working

It just gets rid of your cash. could you get it fixed plz?

we can try

I can push an update


3PCD Deprecation Trial Expiration and the Updated Timeline to 2025

Hello,
Will the 3PCD Deprecation Trial expiration date be pushed back to sync with the updated timeline to Q2 2025? If so, will we need to register for and add a new Deprecation Trial token or would the existing token expiration be automatically updated?

Apologies for the very long delay in responding!
Please see Updates to temporary third-party cookie exceptions in Chrome for more information regarding the status of the 3PCD deprecation trials.
In particular:

Chrome is extending the grace period that allows sites and services demonstrating user-facing, non-ads breakage to continue to access third-party cookies. Origins using this grace period don't need to deploy deprecation trial tokens; origins that have already deployed tokens don't need to keep them in place. We plan to maintain the grace period at least until we introduce a new experience in Chrome that lets people make an informed choice that applies across their web browsing.

Hope that helps.

Closing this as completed, but feel free to reopen.


ARO-11073: Configure identities requirements config file in P1 (dev) environment

What this PR does
Allow the configuration of azure operators managed identities in the P1 environment.
Jira: https://issues.redhat.com/browse/ARO-11073
Link to demo recording:
Special notes for your reviewer
This is oped as draft for now as it depends on https://issues.redhat.com/browse/ARO-11071 being done at which point, I'll also bump the image.

This will be re-worked by someone else. Closing for now.


Crash running Coreelec 8.90.4

I'm submitting a ...

[ x] bug report
[ ] feature request
[ ] support request

General infomration
I'm running Coreelec 8.90.4 on an Odroid C2 and netflix plugin 0.13.10
Prerequisites

[x ] Are you running the latest version?

Description
Getting a crash during video playback when bringing up any overlays.
[Description of the bug or feature]
Steps to Reproduce

Playback a video
Use remote to bring up any overlay
crash

Expected behavior: [What you expected to happen]
Crashed
Actual behavior: [What actually happened]
Playback continues
Context (Environment)
Installation

[ ] I installed the plugin via zip from the Releases page
[x] I´m using the Netflix Repo
[ ] I´m using a different source (Please tell which)

Operating System

[ ] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

Additional informatin on the environment
[Descripe your environment a bit more detailed, are you using LibreElec f.e.]
Debug log
http://ix.io/1fZL
[Please include a link to your debug log (use something like http://sprunge.us/) or similar, please do not paste]
Other information
[e.g. detailed explanation, related issues, suggestions how to fix, links for us to have context, etc.]
Screenshots
[Please add a screenshot if that helps understanding your problem]
[You can erase any parts of this template not applicable to your Issue.]

Same here. Tried #707 build and #629. Nothing helps.
If I install the plugin, Im able to login but not play any episode. If i restart the device, login fails...

Please provide a full debug log (turn on debug log settings in Kodi).


functional: Rework all of them

Our cc-runtime is a sort of hybrid of runc right now because we still have old flags (not supported anymore by latest runc) such as --console along with newer flags --console-socket.
I am not saying we should not be backward compatible (runc is not by the way), but our functional tests should be cleaner.
I think we could have a set of functional tests working with runc v1.0.0-rc2 and another set running with the latest runc version.
Right now, simply using one runc version or another simply does not work. I have spent a day trying to figure out why one of our tests was failing with the new patch I was bringing in for the agent, and I have been wondering if the problem was coming from my patch or from the test itself.
This kind of situation would not happen if we were sure that our functional tests are properly passing with runc as a reference. And to make sure we don't break our tests over time, we should run them first with runc, and then with cc-runtime. In both cases, they should pass.
@sameo @grahamwhaley @jodh-intel @devimc WDYT ?

Related: https://github.com/kata-containers/tests/issues/59.


Qiskit installation error for Python 3.11

Informations

Qiskit version: pip install qiskit
Python version: 3.11
Operating system: Windows 10

What is the current behavior?
pip install qiskit
or
pip install qiskit-aer
Steps to reproduce the problem
What is the expected behavior?
ERROR: Could not build wheels for qiskit-aer, qiskit-terra, tweedledum, which is required to install pyproject.toml-based projects
Suggested solutions

Qiskit doesn't support Python 3.11 yet. Python 3.11 was only just released a couple of days ago, and many of the upstream dependencies of Qiskit do not fully support python 3.11 yet (or haven't published python 3.11 wheels on pypi yet). This is just a matter of time as we wait for those packages to release and add support for python 3.11. Once all of Qiskit's dependencies have versions that support python 3.11 we can push out a qiskit version that supports 3.11. Until Python 3.11 is officially supported in qiskit I'd recommend you stick with python <=3.10 where everything is supported already.
It is probably possible to build everything from source, but it requires having a C/C++, and Rust compiler installed along with many system libraries (like BLAS, GMP, etc) to be able to build all of Qiskit's dependencies from source. You can see documentation on building Qiskit from source here: https://qiskit.org/documentation/getting_started.html (on the Install from source tab). But the issue you'll hit there is being able to build all the dependencies from source. In particular tweedledum and symengine (although as a windows user symengine isn't in the required dependencies list) are quite difficult to build from source. I'd probably not recommend going that route unless you absolutely need to run on Python 3.11.

Qiskit Terra has supported Python 3.11 for about a month now, and Qiskit Aer (the other missing component) will as soon as it makes its next patch release, which is due very soon - the PR adding support is Qiskit/qiskit-aer#1642, which has already merged.
Somewhat ironically, we can't update this version metadata on PyPI without cutting a new release of the package, but when we cut new releases, we can also make sure the new wheels are built...  Strictly, there's nothing in Aer that actually prevents it from running on Python 3.11 already, other than the compiled wheels aren't distributed.  It should be possible for pip to automatically build it from the source distribution, but only if you have all the necessary build requirements installed, which most people don't.

This issue should be solved at this point. Please, comment/reopen otherwise.


Spanish starter pack can not be downloaded

With the latest plugin release 7.1 we have the UI translated to Spanish. The initial animation and the Welcome screen is in Spanish.
When proceeding in the Welcome screen the Spanish starter pack is not downloaded though - the progress stays at zero percentage.
The URL that is tried is http://<IP_ADDRESS>:58297/es-es/explore/#/download/spanish/0001
The starter pack is defined in https://github.com/endlessm/endless-key-collections/blob/main/json/spanish-0001.json - other starter packs have the same naming format
https://github.com/endlessm/endless-key-collections/blob/main/json/artist-0001.json
The UI itself is translated fine. This is just about the starter pack not being downloaded correctly.

I see the JSON in the latest release, so is not that: https://github.com/endlessm/endless-key-collections/releases/tag/v0.3.1

I reproduced it. On the frontend I see this error:
TypeError: Cannot read properties of null (reading 'metadata') at DownloadPage.vue:129:1

On the backend I see a 500 error when start-download is called in ~/.var/app/org.endlessos.Key/data/kolibri/logs/kolibri.txt:
INFO 2023-09-26 10:20:45,937 cherrypy.access <IP_ADDRESS> - - "POST /explore/api/ek-collections/start-download" 500 0 "http://<IP_ADDRESS>:44041/es-419/explore/" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"

Unfortunately the backend logs don't contain the Python traceback that's potentially causing that 500 error. The debug.txt file is empty.

The fix seems to be as easy as adding "spanish" to the list here. It seems we missed the backend change in https://github.com/endlessm/kolibri-explore-plugin/issues/786

Tested with 7.3 - download of the starter pack is working fine now.


fix(Default Value) Save default value at correct location

Current coverage is 92.18% (diff: 0.00%)

Merging #302 into master will decrease coverage by 0.43%

@@             master       #302   diff @@
==========================================
  Files            16         16
  Lines           705        909   +204
  Methods         143        163    +20
  Messages          0          0
  Branches        174        263    +89
==========================================
+ Hits            653        838   +185
- Misses           52         71    +19
  Partials          0          0

Powered by Codecov. Last update f3b8dc3...ce96927


How can I get HWND of the window in a C++ WinUI3 Application ?

How can i get hwnd of the window in a C++ WinUI3 Application ?
In WinUI C# version i used this code :
[ComImport]
[InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
[Guid("EECDBF0E-BAE9-4CB6-A68E-9598E1CB57BB")]
internal interface IWindowNative
{
      IntPtr WindowHandle { get; }
}

and obtained hwnd using
 var hwnd = this.As<IWindowNative>().WindowHandle;
but how can i port it for WinUI3 C++ Application
Note: The handle should be obtained in MainWindow,xaml.cpp file and not in app.cpp file

I Got the answer from other issues:
#include <Microsoft.UI.Xaml.Window.h>
HWND GetHWNDFromWindow(winrt::Microsoft::UI::Xaml::Window const& window)
{
    HWND nativeWindow{ nullptr };
    winrt::check_hresult(window.as<IWindowNative>()->get_WindowHandle(&nativeWindow));
    return nativeWindow;
}

This solved my problem

I am reopening this issue, i was able to get handle to a window, but if i need to get the handle of MainWindow in C++ WinUI3 what can i do?

duplicate of #5230


asa custom containers for frontend (node.js) and python microservices (az-openai) are failing to start

Custom containers on port 1025 are not starting for frontend and python services when deployed to ASA standard

It just deployed successfully, and the frontend page can show the data responded from backend,

Can't reproduce this issue. The deployment was successful on my side.

Closing this issue now for cannot reproduce it.


update cni to v1.1.1-calico+go-1.21.3

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

TBD

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

Closing this since we've made an update as a part of https://github.com/projectcalico/calico/pull/8199


when i change file with shell  webpack,Will only take effect once

when I use the shell to change file, webpack watch just working once.
The second time, without any response,how can i make it work?
code: touch index.js && echo sss >> index.js

Can you re-open with more details at Stack Overflow? Thanks! 👍

@bebraw  please see it .http://stackoverflow.com/questions/38888612/webpack-doesnt-rebuild-after-i-execute-npm-install


Website Provider is detected by Avast antivirus as threat

Steps to reproduce

execute run.bat with Avast antivirus running

Expected result
App should run
Actual result
App does not run as gets blocked.
Technical information
Starcounter version <IP_ADDRESS>46
Involved apps' version
https://github.com/StarcounterApps/Website/commit/5a955a365cbe4b288c1da68a8e280d835acb3dc7
Browsers affected
n/a
Screenshot

I believe that there is nothing specific in WebsiteProvider.exe, this is a problem between Avast and Starcounter. Closing this, as it is tracked in https://github.com/Starcounter/level1/issues/1939


Ruby gem install fails on Alpine 3.6

Fails with grpc 1.2.5 and 1.3.4. Also tried to run with the installed libc6-compat package. Using libressl-dev.
current directory: /usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4/src/ruby/ext/grpc
/usr/bin/ruby -r ./siteconf20170706-17813-1lf0xo3.rb extconf.rb
Building internal gRPC into /usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4/src/ruby/ext/grpc/libs/opt
make: Entering directory '/usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4'
make: Circular /usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4/src/ruby/ext/grpc/libs/opt/libares.a <- /usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4/src/ruby/ext/grpc/libs/opt/libz.a
dependency dropped.
  Check the mkmf.log file for more details.  You may
need configuration options.

Provided configuration options:
	--with-opt-dir
	--without-opt-dir
	--with-opt-include
	--without-opt-include=${opt-dir}/include
	--with-opt-lib
	--without-opt-lib=${opt-dir}/lib
	--with-make-prog
	--without-make-prog
	--srcdir=.
	--curdir
	--ruby=/usr/bin/$(RUBY_BASE_NAME)

extconf failed, exit code 1

Gem files will remain installed in /usr/lib/ruby/gems/2.4.0/gems/grpc-1.3.4 for inspection.
Results logged to /usr/lib/ruby/gems/2.4.0/extensions/x86_64-linux/2.4.0/grpc-1.3.4/gem_make.out

An error occurred while installing grpc (1.3.4), and Bundler cannot continue.
Make sure that `gem install grpc -v '1.3.4'` succeeds before bundling.

Please advise.

Ok, running gem update --system helps as described ’#6525


Single comp player

Trello ticket: https://trello.com/c/93qR2hmV/44-simple-computer-player
Added in or added to:

ComputerPlayer class
validate computer_player move
computer_player marks board
method for take_turns when computer player vs human players
Menu to choose between human vs human and human vs computer (will prompt after each game)
Updated acceptance tests with input for new computer player menu

Not yet implemented:

Right now, the human has to choose both symbols (if they are not playing with X and O), even when they choose human vs computer

Considerations:

The human player always goes first in this version - is that what we want?
Do I want to include messaging that's different if the computer wins (instead of just saying, "Player O, you're the winner" - something like, "Aw man, you lost to the computer!")

Have you tried a play through in Spanish? both against the HvC and HvH? I wonder if the messages are all working correctly? and if not, are there tests to add to ensure that they do?

Have you tried a play through in Spanish? both against the HvC and HvH? I wonder if the messages are all working correctly? and if not, are there tests to add to ensure that they do?

When I added in a SpanishLanaguage test, the tests were failing but I have it expecting a Spanish Message now instead of a MockMessage so it passes.

Have you tried a play through in Spanish? both against the HvC and HvH? I wonder if the messages are all working correctly? and if not, are there tests to add to ensure that they do?

When I added in a SpanishLanaguage test, the tests were failing but I have it expecting a Spanish Message now instead of a MockMessage so it passes.

I was asking if you did a play-through, not adding more tests.


Support for Python3.9

https://github.com/aws-greengrass/aws-greengrass-docker/blob/1c19905c2afdb72c209f00755b9f5eb16310a403/Dockerfile#L40
Can we add support for Python3.9?

For testing purpose, next release we will add python3.9 support along with it together

Any update on this?  Python has moved along and greengrass is stuck in the past.  Will any new release be happening with this?

We recommend that all customers use this dockerfile provided here as a base to start from. If you need a newer version of Python, please edit the dockerfile to install the version that you need.


Include imported code when generating reflection data

The basic idea is simple: be sure to enumerate code in __imported modules when generating reflection info

Note that we don't currently allow an entry point to appear in an imported module, so we only consider globlal-scope parameters

Although there isn't currently a real implementation of namespacing, I went ahead and ensured that parameters in imported modules are treated as distinct from parameters in the user's code, even if they have the same name.

GitHub doesn't seem to be picking up my fixup changes to this branch, and so further CI builds aren't being triggered. I may have to close and re-open the PR.


Issue on aem cloud author | fontIconPicker is not a function

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEMaaCS
[ ] ACS AEM Commons Version: 6.0.14
[ ] Reproducible on Latest?

Expected Behavior
On the pages can be added components and could choose assets/pages via pathbrowser
Actual Behavior
In AEM cloud dev and stage environemnts while loading page we see following js error in console, which is comming form acs-commons.widgets.icon-picker library
Uncaught TypeError: $(...).fontIconPicker is not a function
at HTMLDocument. (dialogstyles.lc-9f9fdaa8982a0f444a4720a2098c372d-lc.min.js:1:108)
at HTMLDocument.dispatch (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:126:359)
at k.handle (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:120:65)
at Object.trigger (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:124:489)
at HTMLDocument. (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:139:230)
at Function.each (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:52:175)
at c.fn.init.each (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:48:132)
at c.fn.init.trigger (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:139:206)
at HTMLDocument. (coralui3.lc-c4cd39f7fcd599057827d5ecc5a83ab1-lc.min.js:76:4583)
at p (jquery.lc-f9e8e8c279baf6a1a278042afe4f395a-lc.min.js:101:96)
Actually it can't load fontIconPicker function from icon-picker.js (https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/master/ui.apps/src/main/content/jcr_root/apps/acs-commons/touchui-widgets/icon-picker/source/icon-picker.js)
$(document).on('cui-contentloaded.data-api', function (e) {
$('[data-init~=graphiciconselect] > select').fontIconPicker({
'theme' : 'fip-coral'
});
});
Steps to Reproduce
Set acs-commons 6.0.14 version to your aem project deploy it cloud enviroment, and try to add cmp.

Any update, we're facing the same issue on-premise 6.5 latest SP with acs-commons 6.6.4 while developing a new set of edtitable templates based on core component page v3.


Update ACR TSG with info about retrying blob upload

Fixes https://github.com/Azure/azure-sdk-for-net/issues/35253

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Containers.ContainerRegistry


One-hot format for output data

It seems that keras needs one-hot format for the targets in order to compute the loss,
Pytorch however requires integer format. It makes sense to default to the pytorch format
and have the keras implementation call to_categorical on it rather than having pytorch
undoing the categorical (which means twice as much computation)

I agree (also needs much less memory). Feel free to make the necessary
changes in the pytorch branch.
Prof. Dr. Mike Kestemont | www.mike-kestemont.org | Twitter:
@Mike_Kestemont |<EMAIL_ADDRESS>|<EMAIL_ADDRESS>|
University of Antwerp | City Campus, Prinsstraat 13, room D. 118 I B-2000
Antwerp, Belgium | tel. +32 (0)3 265.42.54

Check out our documentary on Digital Humanities and Hildegard of Bingen:
watch it in HD on Vimeo: https://vimeo.com/70881172

On Mon, May 8, 2017 at 11:40 AM, Enrique Manjavacas <
<EMAIL_ADDRESS>wrote:

It seems that keras needs one-hot format for the targets in order to
compute the loss,
Pytorch however requires integer format. It makes sense to default to the
pytorch format
and have the keras implementation call to_categorical on it rather than
having pytorch
undoing the categorical (which means twice as much computation)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hipster-philology/pandora/issues/14, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AELJL9yctsEryNG-zgcCTf7umBn-8dTXks5r3uMAgaJpZM4NTtls
.

Done as per 007db3f1ffd82dfe5690a3ccd5afa7cefb22da41. To keep it simple I just decided to add an argument to Preprocessor constructor categorical specifying whether the data should be turned into binary format or not.


Create Min cut Square.cpp

DP problem Medium lvl
Please fill the details before contributing.

⚠Important!!

[x] Done reading CONTRIBUTING.md?

Adding Code PRs?🔥
⚠Do not add multiple files for the same solution!!!

[x] Added problem statement(Readme.md) with solution?

[ ] Contributing new file?
[ ] Appending your code in already existing file?

From Platform:

[ ] Leetcode
[x] GFG
[ ] Interviewbit
[ ] Others(only acceptable after DISCUSSIONS)

Migration PRs?🤝

[ ] Done deleteing file from home directory?
[ ] Added file with correct name and correct directory?

dp problem

.


Defining world_vertex_coords render mode leads to shader crash

Godot version
bf322bacdd99cdb81bd7870b0b196ef01652581c
System information
Windows 10
Issue description
Because scene_data.inverse_normal_matrix as well as scene_data.camera_inverse_binormal_matrix/camera_inverse_tangent_matrix are not defined in the scene_data struct (scene_forward_clustered_inc.glsl).
Steps to reproduce
Create the following shader:
shader_type spatial;

render_mode world_vertex_coords;

Minimal reproduction project
Too easy to reproduce.

Duplicate of https://github.com/godotengine/godot/issues/47480.


Add link to Bolt memory

;)

Thanks!
Btw you may want to consider tagging your library with a version :)


cosmiconfig throws error when searching for config

Hi, I've just started using this library in a vscode extension that I've been working on.  I need to find the Jest configuration in a particular directory and cosmiconfig seemed like the ideal solution.  This is the utility function that I've written - let me know if I've got something wrong:
const getJestConfigInDirectory = async (directory: string): Promise<string | null> => {
  const result = await cosmiconfig("jest").search(directory);
  if (result === null) {
    return null;
  }
  // TODO in future we may be able to make use of the config object that is returned.
  return result.filepath;
};

But basically I'm finding that the search call is throwing an error.  The error seems to be happening in the loadJs loader in the statement:
const result = importFresh(filepath);

So the issue may be in a import-fresh.  The actual error is:

[TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
at validateString (internal/validators.js:112:11)
at Object.dirname (path.js:583:5)
at c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:63551:36
at loadJs (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57684:18)
at Explorer.loadFileContent (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57119:32)
at Explorer.createCosmiconfigResult (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57124:36)
at Explorer.loadSearchPlace (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57105:31)
at async Explorer.searchDirectory (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57076:29)
at async run (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57049:22)
at async cacheWrapper (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57462:18)
at async Explorer.search (c:\Users\rossk\Documents\source\vscode-jest-test-adapter\dist\extension.js:57041:20)]

Debugging further indicates that import-fresh tries to find the parent path of itself which happens to return undefined - which causes the later error.  Please let me know if this is something you are aware of, or if you think I need to raise this issue against import-fresh instead.

Ah, it may have been a transient error.  Can't repro now...

I currently have the same issue when I use babel it's ok but when I use ncc to pack I have this error


use lib mode to build a package which includes a worker,it can't be loaded currently

Describe the bug
like this

this code is contained in a sdk project of my team, I use vite lib mode to build,when serviceWorker.register reload the file,it will directly reload /assets but not add prefix path about my sdk package.
Reproduction
not have
System Info
System:
    OS: macOS 12.4
    CPU: (8) arm64 Apple M1
    Memory: 74.41 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 16.15.1 - ~/Library/Caches/fnm_multishells/11049_1661476897981/bin/node
    Yarn: 1.22.19 - ~/Library/Caches/fnm_multishells/11049_1661476897981/bin/yarn
    npm: 8.11.0 - ~/Library/Caches/fnm_multishells/11049_1661476897981/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 104.0.5112.101
    Edge: 104.0.1293.70
    Firefox Developer Edition: 104.0
    Safari: 15.5

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I also encountered this problem


SAK-29035 caps-lock-checker.js changes styles inline with javascript which is bad practise

https://jira.sakaiproject.org/browse/SAK-29035
caps-lock-checker.js changes styles inline with javascript which is bad practice. The javascript should only change the CSS class on the element, and the actual styles should be in a stylesheet.
Also, the spacing in the .js file is inconsistent, and there are some instances of the 'evil' equality operators rather than the 'good' twins.

Looks good +1

Thanks Sam!


Docs: fix header height on SSR

Summary
As noted in the Jira bug linked below, our docs header looks wonky on certain mobile devices.

I thought at first this was related to very narrow width screen sizes, but it's actually related to height. Our logic around screen height caused a hydration mismatch on SSR, leading to the wonky behavior. This PR modifies the logic to ensure that we're only checking header height on the client, preventing any mismatch. I also cleaned up the Header markup a bit and made sure it looks good on the tiniest of mobile screens*.
* Zoolander phones not accounted for

I believe this also fixes #1732, though @rlingineni should verify.
Links

Jira

ON iPhone 5 'm seeing a weird spacing


get latest mono in CI

#4392 allows us to get the latest stable Mono prior to build+testing

This is working just fine,


Some windows base width from earlier context when should resize

Describe the bug
Code action windows seem to hold a size from the list of code actions, the action preview should resize if there are actions longer than original code action names...

  {
    "nvimdev/lspsaga.nvim",
    dependencies = { "neovim/nvim-lspconfig" },
    event = "LspAttach",
    config = function()
      local present, lspsaga = pcall(require, "lspsaga")

      if not present then
        return
      end

      lspsaga.setup {
        lightbulb = {
          enable = false,
          enable_in_insert = true,
          sign = true,
          sign_priority = 40,
          virtual_text = true,
        },
      }
    end
  }

I have to turn off lightbulb because it causes lines with diagnostics in editor to shift left a column or two...
Steps to reproduce
request code actions that dont seem to have an immediate preview available on the initially selected action?
This was reproduced on gopls, lua_ls, and vtsls
Expected behavior
resize on action preview width changes...
define a fixed width for the action preview window?

THis screen is from hardcoding 60 for the action_preview window opts.width
Neovim version (nvim -v)
0.10.0
lspsaga commit
master
Terminal name/version
wezterm

It's nothing to be sorry about. This is my plugin. When something goes wrong it's my responsibility to fix it. lsp and neovim have too many cases to handle. This is normal.

@glepnir
I just know that in the OSS space it is rare for someone to jump onto issues/requests/features or even respond to users because it's not their first priority/moneymaker.
When I find that an open source developer has better response/SLA/relation with their users than companies with software support contracts in the $$$$$$ of dollars... it definitely should be shown appreciation!

Not quite. More because of love :). For example I don't get paid for my work at neovim.

@glepnir hah! yes, but not all have your sense of responsibility.
I am also not trying to apologize.
I am trying to praise you!

lol thanks for praise. have a good day :)


fix(CodeBlock): Fix dark mode issues

Kayla's team is working out a more PatternFly-compliant system for user messages. This not yet ready. However, she approved this as a short-term gap fix so we can address the bug Rohit's team filed. Rohit's team was seeing problems with inline code and also block-level code headings on dark mode. This should address both for now.
Fixes https://github.com/patternfly/virtual-assistant/issues/137

Preview: https://virtual-assistant-pr-virtual-assistant-141.surge.sh


Add progress bar for virtualization publishing status

Adds progress bar for status updates when virtualization is being published, similar to integrations progress bar.

renames IntegrationProgress to ProgressWithLink and moves it into /Shared.  Update usages, stories and tests accordingly.
VirtualizationPublishStatusDetail component is updated to use the shared component
Add VirtualizationPublishingDetails model, and add function in VirtualizationUtils to return the publishing details for a virtualization
updates data stories

PR Storybook available here

Cool!  I notice with this change we'll need to maybe increase the width for the integration-progress class, but I can do that in a subsequent PR.  I'll let @riccardo-forina have a look too.

PR Storybook available here


Latest 18.1.0 release breaks type remapping for args with spaces in them

As the title says:
Having this line in the rsp:
--remap
__arglist=@params string[] args

Used to work in previous versions by now throws:
Error: Invalid key/value pair argument: string[]. Expected 'name=value'
Error: Invalid key/value pair argument: args. Expected 'name=value'

I was trying to track down the issue and it seems like it's something to do with how the arguments are directly parsed, not the actual logic for extracting them in ParseKeyValuePairs. It seems like maybe context.ParseResult.GetValueForOption maybe splits on spaces so it's resolving multiple arguments instead of a single one?

What was the previous version you were on?
There was likely some break or change in System.CommandLine that was missed and we'll need to add a workaround.


Gui applications in wsl2 have misaligned context menus with fancyzones enabled

Microsoft PowerToys version
0.84.0
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
FancyZones
Steps to reproduce

Setup fancyzones
Install WSL2 Ubuntu
install gitkraken through instructions here to ensure all the required dependencies get deployed. Running this in /tmp prevents some apt related issues.
Launch gitkraken, observe the window appear
Open a repository with it and right click a commit, or open any of the drop-down menus in the application at the top.
Observe that the context menus are visually misaligned from the cursor and difficult to use
add msrdc.exe to the exclude list for fancyzones (or exit powertoys)
Observe the correct behavior for context menus.

There's a related issue opened for wsgl with video etc. examples, but I opened a bug ticket here because this seems like it may have a fix / workaround in fancyzones itself https://github.com/microsoft/wslg/issues/1116
✔ Expected Behavior
Context menus work as expected
❌ Actual Behavior
Context menus are misaligned with the cursor and often un-usable

Other Software
Windows Subsystem For Linux 2 w. Ubuntu
Remote desktop client

on my laptop i have "move newly created windows to their last known zone" enabled, disabling that fixes the issue. enabling it makes it come back. On my desktop this doesn't happen. Unclear if this has something to the laptop having two screens even if they are not active at the same time. I'm somewhat concerned uploading powertoys logs on my work laptop as an attachment here :)

After a reboot this morning on the laptop, the issue persists even with the feature tag mentioned above disabled. excluding msrdc.exe fixed the problem. The original poster of the bug on wslg, @pkmnsnfrn, has a three monitor system. Speculating this is related to multi-monitor.


Start community-provided dataset docs

Continuation of #736 with clean fork.
Old description
This is what I did to get the pseudo-labels updated. Not sure if it generalizes, but I figured I would write it down. It was pretty easy because all I had to do was make properly formatted directories and change URLs.
In slack @thomwolf called it a user-namespace dataset, but the docs call it community dataset.
I think the first naming is clearer, but I didn't address that here.
I didn't add metadata, will try that.

Oh, really cool @sshleifer!


Integrated trace in attributes

A function that computes integrated trace of post-stack seismic amplitude data. Vectorized of course.

Needs to have a window argument.
https://agilescientific.com/blog/2011/12/28/i-is-for-integrated-trace.html
Description of Landmark's old implementation:

Integration integrates the trace using Fourier transform. This is done by transforming the trace, dividing each frequency by (2Pifreq*sqrt(- 1)), and zeroing the zero frequency component (removing dc). This gives a result similar to the simple (trace(i) = trace(i) + trace (i-1)), but has a more accurate 90 degree phase response at all frequencies. The frequency domain method is also not dependent on dc biases caused by small amplitude variations of the first few samples.
Integration produces an output trace in which each output sample value is the sum of the original samples, including this original sample.

Another description here, from CREWES, look for BLIMP: https://crewes.org/ForOurSponsors/ResearchReports/1996/1996-21.pdf


refactor: put all filters in one interface

It is strange that integration tests are not running on this PR.

More and more I think of this, I lean to have it to separate interfaces. If forces users more to think what they want to achieve with the filters.
Basically a generic filter is about ignoring the resources. And the others really makes think the users in what situation it should be used.


E19 when jumping back to fern buffer in nvim

In vim-fern directory, run nvim (0.10.2) as follows:nvim --clean --cmd 'set rtp^=.' -c 'Fern .'

Move the cursor to README.md, and <CR> on it.
Jump back to the fern buffer with <C-o>.
Jump to README.md with <C-i>
Jump back to the fern buffer with <C-o>
E19: Mark has invalid line number

Interestingly, the line number of the position in jumplist keeps increasing after each <C-o>, <C-i> cycle:
# In README.md

:ju
 jump line  col file/text
   2     1    0 fern://79154091/file:///home/user/.vim/plugged/vim-fern$
   1    41    0 fern://79154091/file:///home/user/.vim/plugged/vim-fern$
>  0     1    0 # 🌿 vim-fern

# run <C-o> <C-i>

:ju
 jump line  col file/text
   2     1    0 fern://79154091/file:///home/user/.vim/plugged/vim-fern$
   1    49    0 fern://79154091/file:///home/user/.vim/plugged/vim-fern$

Vim (9.1.771) does not raise E19, but you can see the line number increasing after each <C-o>, <C-i>.

Reproduced.

https://github.com/lambdalisue/vim-fern/blob/2cf57c60df508d15878e03c1e8fb12506384212a/autoload/fern/internal/viewer.vim?plain=1#L49
Using bufhidden=hide solve the issue but I'm not sure if it should.


Training

How can i train SDXL with own dataset, can you give me a help

same question

Convert your dataset to webdataset format.
Then, specify the tar file location in cfg file.

@dae-sun can you provide a sample code ?


Allow customizable turn on action for LG WebOS tv

Description:
Currently, the webostv media_player component allows for WoL to turn the TV on, as turning it on via network is not possible. Not all LG WebOS TV models have the WoL functionality however all (I believe) of the TVs do have some form of a RS-232 input which allows you to directly communicate with the TV to do things like turn it on (even if it is off) among other things.
I have added the ability to specify an action. We now have two ways of turning this TV on - WoL or action. Currently WoL takes precedence over default action if both are specified.
Input Required: As indicated, the current functionality allows for WoL and now custom action, however I feel like this is a bit clunky. There should only be 1 way to do this. HASS already has a WoL switch which can be created specifically for the TV and referenced through the turn on action (and set to hidden to keep hass clean). This would allow me to get rid of the current WoL functionality for the LG WebOS tv, however this is a breaking change and I would be hesitant to do this without getting the OK of someone higher up.
Alternatively, I can leave it as is with a note in the docs about the precedence or I can somehow raise an error if both are specified. Please let me know which way you'd like.
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Once I get an answer to the Input Required section above, I will update the documentation and update the PR to reflect this.
Example entry for configuration.yaml (if applicable):
media_player:
  - platform: webostv
    host: lgtv.example.com
    name: Living Room TV
    turn_on_action:
      service: persistent_notification.create
      data:
        message: "foobar!"

Checklist:
If user exposed functionality or configuration variables are added/changed:

[NO] Documentation added/updated in home-assistant.github.io
Once I get an answer to the Input Required section above, I will update the documentation and update the PR to reflect this.

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[na] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[na] New dependencies are only imported inside functions that use them ([example][ex-import]).
[na] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[na] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Great one.Not only RS232 also the HDMI CEC can be used!

I am waiting for comments from anyone in power re: getting rid of the default WoL behavior in favor of the configurable turn on option before continuing on the PR.
Is there anything any of you need from me for this?

I'm not "in power" but have been thinking about this change a bit and wanted to weigh in... it seems a shame to have to add additional config (both enabling the WoL component and then specifying a service call to call it with the TVs MAC address for the on action) if your proposed additional change were to be implemented.
Currently this is handled by just specifying the MAC address of the device in config alone (for supported TVs and network setups); I'm not sure if WebOS devices can be autodiscovered, but if they are, and support WoL, wouldn't that break the ability to turn on for a discovered device and require manual config? (at least in the case that the WoL component is not enabled)

@andersonshatch I believe you would still need to specify the mac address for the TV even if there were a way to autodetect a model has WoL, which would mean leaving it as is would be the same effect.

@balloob @pvizeli Hey! Sorry to bug you about this but is there anything you'd need from me before being able to discuss the breaking change idea I had?

We have actually been switching components over from built-in wake on lan to just supporting the script syntax, in that case people can still call the wake_on_lan service that is provided by the wake_on_lan component.
However, if LG WebOS TVs support wake on lan and thus is a supported feature, we should keep that available. So in this case I would suggest we keep support for both and make sure the user can only configure one (like this).
While you're in the code, would be awesome if you could remove the wakeonlan requirement and make it depend on the wakeonlan component and call that service instead.

@balloob Not all LG WebOS TVs support wake on lan. From a very cursory investigation, only the highest end models do, so a lot of LG TVs cannot use this. I feel it is a fringe case which is why my first thought was to drop the WoL aspect from this component completely and let people configure their preferred turn on methods via script (WoL, RS-232, HDMI-CEC) because between those three, I believe, we can support turn on for all LG WebOS TVs.
Saying that, if you would like to continue to support both WoL and Scripts, I will update the component as requested. Let me know and thanks for your time!

ping @balloob I'm hoping to finish this up this weekend (understanding it won't make it into this release) once I get finalized direction. Otherwise I might just finish it up on my own and hope for the best.

Well, fighting with houndci aside, what do you think @balloob?
If approved, I can get the docs PR up and update this one to reflect that.

I believe this is ready to go. I have been using this version of webostv.py as a custom component for a few days and it has been working great. Any possible way to get this into .54 @balloob ?

Us the problem with Travis that this PR is not included?

@moskovskiy82 possibly. I will update everything sometime this week, including documentation. Make any changes from there I guess.

As WebOSTV has a configuration to it, the WOL option is most used across all users. RS232 is barely available on webos 3+ devices and barely anyone has their HA server connected to their TV via HDMI. Those both are a niche implementations and I expect a lot of questions on Discord from confused users.
In a way this went from plug-n-play to "how do I find the MAC address?" implementation.
@cmsimike can you please add to the docs for everyone to know how to implement the WOL change please?

Hi @ferdydek,
You might be misremembering, but the wake-on-lan functionality was never plug and play.  Here is the documentation diff:
https://github.com/home-assistant/home-assistant.github.io/pull/3442/files
The current documentation clearly indicates the turn-on  functionality is a user-definable action.
https://home-assistant.io/docs/automation/action/

Hey @cmsimike
Sorry to reply so late. I guess this depends on WebOS version <3, 3.0 or 3.5.
With 3.0 + if you allowed the TV to be autodiscovered, while the Mobile App option on the TV was already on you would not need mac address.
I have nothing against new/additional power on functions but since the volume of users will be rather using newer devices with WebOS 3.0+ I strongly advocate to keep old behavior as default and add new option that would override defaults is specified.
Sorry for not being more helpful in how to code this in.
I had at least 4 questions about how to use this in the last week (forum + discord) that is why first of all I think we should go back to previous build and add non-default actions without creating breaking changes and then also I'm sure we need to add info about WOL Power On action as example on the docs.
Please see:
https://community.home-assistant.io/t/lg-webos-tv-configuration-problem/30708/9?u=ferdydek
https://github.com/home-assistant/home-assistant/issues/10005
If I can help out in any way poke me on discord @ferdydek

PRs are not the place for discussion. Use the forums or issues on Github instead.


[BUG]Packet overload upon special ability use with almost all legendaries and boss attacks

Example:
Minecraft: 1.20.1
Fabric: 0.15.11
Soulslike-Weaponry: <IP_ADDRESS>
This is the crash I have been having issues with. It is the same issue the darkin scythe had before the most recent patch update (lovely patch btw). Upon the use of any attack that has particles, the client will have a packet overload and immediately crash. this happens inconstantly but many examples is The Nightfall's smash attack, the darkin blade's smash attack, comet spear calculated fall landing, Genesis fracture ability, Returning night and fallen icon smash attacks, all of the deity attacks, some night shade attacks, and I think there are more. but it regards every attack using particles and it's completely random when it happens. (By the way, you DID fix the darkin scythe's issue. So everything should be under the same list, but it seems like a lot)
No logs are generated as the game just reverts to before the crash for single player, and in multiplayer the only thing logged is the join and leaving of the player that crashed.

Expected behavior: It shouldn't crash :)
(Hey, if you're interested in joining an smp, you're more than welcome to join mine which is the main reason for me asking for the patches :D. Just thought I should mention lol)
A video of me and another member of the smp fighting a supremely config-buffed fallen icon and then crashing:
https://youtu.be/c-3Qz3eLeiw
I have not tested it by itself on this version yet, but on version 1.1.3 these crashes were still prevalent while the mod was isolated.

Thanks for the video, seems like it still is an issue then. I will look into it, fixing the Umbral Trespass one did give some insight into why this happens.
And thanks for the invite! Maybe I'll join another time in the future, got a lot going on right now, but I'll let you know if the time comes.

Yeah, it is still prevelant lol.
And of course, by all means.
By the way, I will open a new thread for this but I've completed a theme (Ima have you judge it) for the returning knight/fallen icon. I feel like fallen icon should have a 2nd phase theme and since fallen icon and returning knight are essentially the same, they get the same theme.

Hi! Was wondering if this was fixed in a recent update? A server we host has this mod in its modpack with the version <IP_ADDRESS>, and we have been experiencing clientside crashes on certain boss attacks
Should I get the pack devs to update the mod somehow? (the version it is on is the latest according to curseforge)

It is not fixed yet, but I do believe marium is doing as best they can to resolve the issue 🔥

@mariumbacchus if by any chance could you give us an update on this situation? :D
I have been having the issue of this happening all the time against fallen icon and all the other bosses. The bosses make the crash happen alot more than the weapons, but it happens with those too.

I am currently working through all the issues, but updating through 6 versions (fabric 1.18.2 to 1.20.4 + forge 1.18.2 and 1.20.1) takes time. I just read something interesting in the other crash logs about packets causing memory crash, I belive you read it yourself a few days ago too, namely forcing the Java version to 17.0.11 fixes the issues. If you could try this and see if that helps, it would be greatly appreciated.

I appreciate you working on it, I thank you greatly. But If I may ask how would I force java version to 17.011

Huh, i will try out prism launcher and see if the crash still happens, and if it fixes Id like to find a way to do it with curseforge launcher

YES, this is a fix! I used prism and installed java 17.0.11 and no boss or weapon could crash me. I had 5 of my testers help me battle fallen icon and the deities simultaneously, and they crashed a ton, and I did not. To be sure, I joined using my curseforge client and crashed equally as much as they did. However I rejoined with prism and did not crash once. I even spawned 20 night prowlers and everyone except me crashed immediately. Now I just need to test and see if I can change this on curseforge.

Do note that java 17 alone does NOT work

I figured out how to do it with curseforge, and it works as it does with prism. The only issue is that it is a tad annoying to do, as you have to manually download java 17.0.11 from https://download.oracle.com/java/17/archive/jdk-17.0.11_windows-x64_bin.exe
and then go to your minecraft settings in curseforge, and change the directory of java 17

Ok nice, now I at least know why this crash is happening. Thanks for testing it out, I'll try to figure out a fix.

One of my admins made a tutorial for it, It is below.
Click the link to download Java 17.0.11: https://download.oracle.com/java/17/archive/jdk-17.0.11_windows-x64_bin.exe
Click the link to view the video: https://youtu.be/-JV00P-hdr0

Alright nice. Maybe I'll put this up on the mod page or in the README file. It is a bit much to do though, so I will still look for a potential fix.

Is there a way to fix this on Tlauncher?

The fix is the same for every launcher, just make the correct java version your main thing. Either you can set the version you want as your "java home", making that version be universal for all other applications, or there should be a way inside TLauncher too. A quick google search should give you the details.


ENH: Add out-of-sample prediction to IV

Add out-of-sample prediction to IV models

This pull request introduces 6 alerts - view on lgtm.com
new alerts:

6 for Potentially uninitialized local variable

Comment posted by lgtm.com


Ionic 1.3.1 + iOS ghost click

On iOS 9.2.1 when I tap on an entry in the side menu which looks like this:
<ion-item ng-repeat="item in menu.items" menu-close ui-sref="{{::item.sref}}" ui-sref-active="activated" > <h2>{{::item.title}}</h2> </ion-item>
the keyboard automatically pops up. In the next view there is a the textarea at the same co-ordinates as the menu item just tapped. This behaviour is made more strange by the fact that the textarea isn't actually focussed, it is just the keyboard that is up.  This has been an issue for a while (see https://github.com/driftyco/ionic/issues/4993) Does anyone have any thoughts?
Thanks for your help,
Matt

@mklilley I have the same issue on iOS 9.3.1 with the same Ionic version.  Below i have another button which expands the div above. When i click the expand button the buttons of the expanded div are clicked accordingly to the touch coordinate.
In my case, this doesn't happens always. Is it the same with you, or does it happens all the time?

@mklilley i have a similar issue (#6750). In my case it doesn't happen always, is that your case?

Same issue (on ionic 1.0.0) on iOS 9.2.1. data-tap-disabled="true" fixes the problem, although adds a 300ms delay.

This issue was moved to driftyco/ionic-v1#133


Qwen2 CrossLayerAttention (CLA) + FP8 KV

What does this PR do?

LGTM feel free to merge

I will also add the changes for llama modeling.


Support for anonymous users to purchase a subscription and activate upon registration or at a later date

Currently it seems like this project requires users to be authenticated before they can purchase a subscription.
I think there's a strong use case for anonymous users being able to purchase a subscription before having to register and login.  Instead of creating/activating the customer immediately for the authenticated user, all relevant data could be stored in a database with a unique validation key that is emailed to the purchaser upon received payment.  The purchaser can then register for an account at a later date and activate the subscription with the validation token at their discretion.
I'll be working to develop this on a branch.  Thoughts?

@codydjango this is an interesting idea. i think current pinax-stripe (master) supports this but you'd need to hook up your own view(s) and integration into your site.  I'd be interested to hear more though.


LET/D variables in attrib-handling list structure

Reported by ansley b manke on 12 Jan 2016 18:08 UTC
The linked-list structure that stores datasets, variables and attributes does not take into account user-defined variables that are associated with datsets using LET/D.  All user variables are stored in this structure in a fake dataset with dataset number -1. This means that if the same variable name is defined for multiple datasets with LET/D, the definition of each new variable wipes out previous ones in this structure, that have the same name.
yes? use coads_climatology
yes? let/d=1/title="coads foobar" foobar = 1
yes? sh dat 1
     currently SET data sets:
    1> /home/users/tmap/ferret/linux/fer_dsets/data/coads_climatology.cdf  (default)
 name     title                             I         J         K         L         M         N
 SST      SEA SURFACE TEMPERATURE          1:180     1:90      ...       1:12      ...       ...
 AIRT     AIR TEMPERATURE                  1:180     1:90      ...       1:12      ...       ...
 SPEH     SPECIFIC HUMIDITY                1:180     1:90      ...       1:12      ...       ...
 WSPD     WIND SPEED                       1:180     1:90      ...       1:12      ...       ...
 UWND     ZONAL WIND                       1:180     1:90      ...       1:12      ...       ...
 VWND     MERIDIONAL WIND                  1:180     1:90      ...       1:12      ...       ...
 SLP      SEA LEVEL PRESSURE               1:180     1:90      ...       1:12      ...       ...
 ------------------------------
 FOOBAR[= 1

yes? sh att foobar
     attributes for user-defined variables
 foobar.long_name = coads foobar
 foobar.missing_value = -1.E+34

yes? show data/att 1
[result listing does not show foobar](D=coads_climatology])

yes? use levitus_climatology
yes? let/d=2/title="Levitus foobar" foobar = 2
yes? sh dat 2
     currently SET data sets:
    2> /home/users/tmap/ferret/linux/fer_dsets/data/levitus_climatology.cdf  (default)
 name     title                             I         J         K         L         M         N
 TEMP     TEMPERATURE                      1:360     1:180     1:20      ...       ...       ...
 SALT     SALINITY                         1:360     1:180     1:20      ...       ...       ...
 ------------------------------
 FOOBAR[= 2

yes? sh att foobar
     attributes for user-defined variables
 foobar.long_name = Levitus foobar
 foobar.missing_value = -1.E+34

yes? sh att sst[d=1](D=levitus_climatology])
     attributes for dataset: /home/users/tmap/ferret/linux/fer_dsets/data/coads_climatology.cdf
 SST.missing_value = -1.E+34
 SST._FillValue = -1.E+34
 SST.long_name = SEA SURFACE TEMPERATURE
 SST.history = From coads_climatology
 SST.units = Deg C

yes? sh att foobar[**ERROR: variable unknown or not in data set: foobar[d=1](d=1]
)

yes? sh att foobar[**ERROR: variable unknown or not in data set: foobar[d=2](d=2]
)

The variables are defined using the non-attribute-handling data structures in Ferret; only code that depends on the attribute-handling structure is affected.
yes? sh var
 Created by DEFINE VARIABLE:
     FOOBAR[= 1
         "coads foobar"
     FOOBAR[D=levitus_climatology](D=coads_climatology]) = 2
         "Levitus foobar"
 >>> Definitions that replace any file variable of same name:
     FOOBAR = 0
         "Global foobar"

Instead of being in the fake dataset # -1, the info for variables should probably be added to the list for the dataset where they are defined; with an additional tag saying they're user-defined. CANCEL VAR should remove the information from the linked list structure.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2352

Comment by ansley.b.manke on 18 Feb 2016 19:15 UTC
duplicate of #2134


netbird gui improvement for URL sanitization / user feedback

Describe the problem
Management URLs are not sanitized against user input. If the user enters invalid characters nothing happens when they click connect and they are left scratching their heads. Users copy/paste the Management URL. Sometimes they get extra characters in the clipboard.
To Reproduce
Steps to reproduce the behavior:
In this example, a carriage return was included by the user over-selecting during copy/paste. To reproduce simply over-copy a URL to the next line so a '/r' gets invisibly added to the clipboard entry and paste it into the gui management URL field.
Expected behavior
The gui should sanitize user inputs against this type of obvious error
or
provide an error pop-up for the user without having to dig into the client.log
Are you using NetBird Cloud?
Self-hosted
NetBird version
0.26
Context
From the client.log (domain masked):
2024-02-27T15:27:46+04:00 ERRO client/internal/config.go:337: failed parsing Management URL URL https://netbird.domain.com:33073
: [parse "https://netbird.domain.com:33073\r ": net/url: invalid control character in URL]

Hi @TSJasonH,
thanks for pointing this out. We will work on improving the validation of the fields in the desktop client UI.


[BUG] Unable to use terraform-provider-rancher2 to provision RKE2 cluster well on Rancher v2.9.0-alpha5

Describe the bug

Given the imported Harvester v1.3.1 in Rancher v2.9.0-alpha5.
When we try to provision the v1.29.5 RKE2 cluster using terraform-provider-rancher2 (3.1.1 and latest) using

Backend automation test script
Manual execution

The RKE2 cluster all stuck in "waiting for cluster agent to connect"

Lots of pods in Pending state
ubuntu@rke2-terraform-pool1-ea67ef05-hm945:/etc/rancher/rke2$ sudo kubectl --kubeconfig=rke2.yaml get pods -A
NAMESPACE         NAME                                                          READY   STATUS      RESTARTS   AGE
calico-system     calico-kube-controllers-86bd9c4fb4-m4bl8                      0/1     Pending     0          3h59m
calico-system     calico-node-q79j7                                             0/1     Running     0          3h59m
calico-system     calico-typha-8d75c767d-phmcc                                  0/1     Pending     0          3h59m
cattle-system     cattle-cluster-agent-78c64dbc79-7kxc2                         0/1     Pending     0          4h
kube-system       etcd-rke2-terraform-pool1-ea67ef05-hm945                      1/1     Running     0          3h59m
kube-system       harvester-cloud-provider-86bb5648b4-4xrlk                     1/1     Running     0          3h59m
kube-system       harvester-csi-driver-controllers-56877cb574-9cdkk             0/3     Pending     0          3h59m
kube-system       harvester-csi-driver-controllers-56877cb574-hdfdc             0/3     Pending     0          3h59m
kube-system       harvester-csi-driver-controllers-56877cb574-xbbwv             0/3     Pending     0          3h59m
kube-system       helm-install-harvester-cloud-provider-sr5pk                   0/1     Completed   0          4h
kube-system       helm-install-harvester-csi-driver-mk67w                       0/1     Completed   0          4h
kube-system       helm-install-rke2-calico-crd-xp99l                            0/1     Completed   0          4h
kube-system       helm-install-rke2-calico-k9j6d                                0/1     Completed   2          4h
kube-system       helm-install-rke2-coredns-xt5jr                               0/1     Completed   0          4h
kube-system       helm-install-rke2-ingress-nginx-xpqhd                         0/1     Pending     0          4h
kube-system       helm-install-rke2-metrics-server-8wv5n                        0/1     Pending     0          4h
kube-system       helm-install-rke2-snapshot-controller-crd-vm8nq               0/1     Pending     0          4h
kube-system       helm-install-rke2-snapshot-controller-qxn92                   0/1     Pending     0          4h
kube-system       helm-install-rke2-snapshot-validation-webhook-t47sd           0/1     Pending     0          4h
kube-system       kube-apiserver-rke2-terraform-pool1-ea67ef05-hm945            1/1     Running     0          4h
kube-system       kube-controller-manager-rke2-terraform-pool1-ea67ef05-hm945   1/1     Running     0          4h
kube-system       kube-proxy-rke2-terraform-pool1-ea67ef05-hm945                1/1     Running     0          4h
kube-system       kube-scheduler-rke2-terraform-pool1-ea67ef05-hm945            1/1     Running     0          4h
kube-system       rke2-coredns-rke2-coredns-5b7d84d764-mrn7r                    0/1     Pending     0          3h59m
kube-system       rke2-coredns-rke2-coredns-autoscaler-b49765765-8jxvr          0/1     Pending     0          3h59m
tigera-operator   tigera-operator-795545875-ph9wt                               1/1     Running     0          3h59m

To Reproduce
Steps to reproduce the behavior:
Pre-requisite

Use helm to provision the Rancher v2.9.0-alpha5

Repo url: https://releases.rancher.com/server-charts/alpha

Create a v1.3.1 Harvester cluster
Import Harvester to Rancher

Using automation script

Clone the harvester tests repo

git clone https://github.com/harvester/tests
cd tests

Trigger the terraform Rancher automation tests suitetox -e py36 --result-json=test_result_terraform.json -- -l harvester_e2e_tests/integrations --json-report --html=test_report/test_result_terraform.html -m 'not delete_host and not upgrade' -k 'test_z_terraform_rancher'

Check the RKE2 cluster state

Using manual install

Prepare the terraform rancher provider.tf file
Get the kubeconfig file of Harvester and place in the local path
Update the provider.tf file with the correct value
providers.tf.txt
Execute following command to trigger terraform to provision RKE2 cluster from Rancer$ terraform init
$ terraform plan
$ terraform applyt

Check the RKE2 cluster state

Expected behavior

Should be able to provision downstream v1.29.5 RKE2 guest cluster well using terraform-provider-rancher2
Support bundle

supportbundle_a94a8926-f7b3-4f92-937f-4cbcb56bc9a1_2024-07-01T11-54-02Z.zip
Environment

Harvester ISO version: v1.3.1
Rancher version: v2.9.0-alpha5
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630): Single nodes on baremetal machine
RKE2 version: v1.29.5+rke2r1
terraform-provider-rancher2: v3.3.1 and latest

Additional context
Use regular steps from Rancher UI can provision the v1.29.5 RKE2 cluster well

@TachunLin Could you try with terrform provider latest version v4.1.0?

I tried with the terraform provider rancher2 v4.1.0 in the manual and automation manner
Both of them also encounter the waiting for cluster agent to connect while provisioning the v1.29.5 RKE2 cluster.

For manual execution, I use the following provider.tf file

providers.tf.example.txt

For backend automation, I update the tests/config.yml file to the terraform provider rancher2 v4.1.0
# script location for terraform related test cases
terraform-scripts-location: 'terraform_test_artifacts'
# Rancher provider version, leave empty for the latest version. e.g. '' or '3.1.1'.
terraform-provider-rancher: '4.1.0'

Use the different way to test the terraform provider provisioning RKE2 cluster.
Compared with the previous test, this time I remove the cloud provider part in the providers.tf file
    # ########## Generate Manually ##########
    # gen_harvester_cloud_provider_kubeconfig.sh
    # #######################################
    machine_selector_config {
      config = {
        cloud-provider-config = file("${path.module}/harvester131-kubeconfig")
        cloud-provider-name = "harvester"
      }
    }

This time I can provision the RKE2 v1.29.5 cluster well on the Rancher v2.9.0-alpha5

Attached the success provider main tf file for the reference
main.tf.txt
I think maybe it related to the cloud-provider-config file did not well generated.
I would investigate more in depth to try again.

@TachunLin If it is working with v4.1.0, let's close this. Also, if needed update the automation script.

Hi David, thanks for reporting the issue. I reopen the issue because there is a bug in harvester-cloud-provider. We should be able to provision RKE2 server without modifying providers.tf.  Thank you.
Using manual install

Prepare the terraform rancher provider.tf file
Get the kubeconfig file of Harvester and place in the local path
Update the provider.tf file with the correct value
providers.tf.txt
Execute following command to trigger terraform to provision RKE2 cluster from Rancer$ terraform init
$ terraform plan
$ terraform applyt

Check the RKE2 cluster state

Thanks for @FrankYang0529 provided information.
Just tried the manual manner to provision RKE2 cluster with terraform rancher2 v4.1.0
We use the gen_harvester_cloud_provider_kubeconfig.sh to generate the cloud config file
#!/bin/bash

RANCHER_SERVER_URL="https://rancher.<IP_ADDRESS>.sslip.io"
RANCHER_ACCESS_KEY="xxx"
RANCHER_SECRET_KEY="xxx"
# Refer to https://<IP_ADDRESS>/dashboard/harvester/c/c-m-f9zsbp9t/harvesterhci.io.dashboard#vm
HARVESTER_CLUSTER_ID="c-m-9dvfsnj8"
CLUSTER_NAME="harv-local"
curl -k -X POST ${RANCHER_SERVER_URL}/k8s/clusters/${HARVESTER_CLUSTER_ID}/v1/harvester/kubeconfig \
   -H 'Content-Type: application/json' \
   -u ${RANCHER_ACCESS_KEY}:${RANCHER_SECRET_KEY} \
   -d '{"clusterRoleName": "harvesterhci.io:cloudprovider", "namespace": "default", "serviceAccountName": "'${CLUSTER_NAME}'"}' | xargs | sed 's/\\n/\n/g' > ${CLUSTER_NAME}-kubeconfig

And use the correct provider tf file content in the following
main.tf.txt
This time we can also include the cloud provider and correctly provision the RKE2 cluster on v4.1.0

I would check again later using the existing automation script for a cross comparision.
And also we need to test again once after the PR of handle empty namespace have been merged.

Double checked on Rancher v2.8.5 with Harvester v1.3.1
Using backend automation script can correctly provision the v1.28.10 RKE2 cluster.