Siad doesn't handle out of disk space well

Loading...
(0/6) Loading siad...
(1/6) Loading gateway...
(2/6) Loading consensus...
(3/6) Loading transaction pool...
(4/6) Loading wallet...
(5/6) Loading host...
(6/6) Loading renter...
Finished loading in 1.057140199 seconds
goroutine 856293 [running]:
runtime/debug.Stack(0x4457c7, 0x0, 0xc420da7c80)
	/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
	/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Severe(0xc420da7e58, 0x2, 0x2)
	/home/luke/go/src/github.com/NebulousLabs/Sia/build/critical.go:32 +0x92
github.com/NebulousLabs/Sia/persist.(*Logger).Severe(0xc420326800, 0xc420da7e58, 0x2, 0x2)
	/home/luke/go/src/github.com/NebulousLabs/Sia/persist/log.go:71 +0xe0
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc4249e4a40, 0xc42022e668, 0x0, 0x0, 0x0, 0xf05908, 0x0, 0x0)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:201 +0x357
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:209 +0x1e5
Severe error: Unable to properly initialize WAL file, crashing to prevent corruption: unable to write WAL metadata: write /home/sokoow/.config/Sia-UI/sia/host/contractmanager/contractmanager.wal_temp: no space left on device
goroutine 856293 [running]:
runtime/debug.Stack(0x4457c7, 0x0, 0xc420da7aa0)
	/usr/local/go/src/runtime/debug/stack.go:24 +0x79
runtime/debug.PrintStack()
	/usr/local/go/src/runtime/debug/stack.go:16 +0x22
github.com/NebulousLabs/Sia/build.Severe(0xc420da7c18, 0x2, 0x2)
	/home/luke/go/src/github.com/NebulousLabs/Sia/build/critical.go:32 +0x92
github.com/NebulousLabs/Sia/persist.(*Logger).Severe(0xc420326800, 0xc420da7c18, 0x2, 0x2)
	/home/luke/go/src/github.com/NebulousLabs/Sia/persist/log.go:71 +0xe0
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).appendChange(0xc42022e668, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlog.go:151 +0x46e
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc4249e4a40, 0xc42022e668, 0x0, 0x0, 0x0, 0xf05908, 0x0, 0x0)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:208 +0x423
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:209 +0x1e5
Severe error: Unable to write state change to WAL: write /home/sokoow/.config/Sia-UI/sia/host/contractmanager/contractmanager.wal_temp: no space left on device
panic: unable to append a change to the WAL, crashing to prevent corruption

goroutine 856293 [running]:
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).appendChange(0xc42022e668, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlog.go:152 +0x4b8
github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit.func2(0xc4249e4a40, 0xc42022e668, 0x0, 0x0, 0x0, 0xf05908, 0x0, 0x0)
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:208 +0x423
created by github.com/NebulousLabs/Sia/modules/host/contractmanager.(*writeAheadLog).commit
	/home/luke/go/src/github.com/NebulousLabs/Sia/modules/host/contractmanager/writeaheadlogsync.go:209 +0x1e5

You disk is full. You need to free some space.

I know that, yet it should return some nice error message, not crash, right ? :)

I agree

I would recommend changing the issue title to something like "Improve error message regarding Disk Space Issues" and see if one of the devs will tag it with the "Programming Task" label.  Otherwise, I would close this issue as the code is performing as expected (but not as desired).

We are tracking this Feature Request in #2407
Recommendation to Devs: Close


Make some options user-configurable

Some various constants could do with being configurable in _config.yml:

MAX_SECONDS_BETWEEN_POSTS
OOC_START_DELIMITERS

This may even tie in to #15, allowing RpLogGenerator::RP_KEY to be specifying in the config file (or taken from its collection name)

Added in cf213569ea6efd55e39af79d827f962de88e4877


Oracle connection string

With your example, init mysql database connection with these code:
db, err := otelsql.Open("mysql", mysqlDSN, otelsql.WithAttributes( 		semconv.DBSystemMySQL, 	))
i think with oracle, init database will be:
db, err := otelsql.Open("oracle", oracleDSN, otelsql.WithAttributes( 		semconv.DBSystemOracle, 	))
i dont know with oracle, what will connection string "oracleDSN" look like? please guide me. Thanks a lot!

Hi @CongSon, the driver name and the DSN is up to the implementation of the driver, not the dashbase/sql or the otelsql.
Take go-sql-driver as an example, it has a section describing its DSN format. https://github.com/go-sql-driver/mysql#dsn-data-source-name
If you'd like to connect to oracle, make sure to check the driver name and the DSN format of that certain oracle driver.

Thanks @XSAM . i'm totally understand!!!


Add embed Google Map on Places pages.

It would be cool to have a Google Map preview along with a place's address on pages like this http://eventsinwichita.dev/places/1

I'm interested in picking this one up, I did something similar at dtravie.com/emorinlaw. Is this what you're looking for?

I think eventually I'd like the map to have a different color scheme to better match the site, but I can create another issue for that when we get there.

Places have been removed for now in the rewrite


Upload button is not appearing in the website

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Any log messages given by the failure

Expected/desired behavior

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?)

azd version?

run azd version and copy paste here.

Versions

Mention any other details that might be useful

Thanks! We'll be in touch soon.

The upload button is an optional feature, as outlined in the deploy_features.md document.
To enable it, you'll first need to configure login and document-level access control. After that, set the environment variable azd env set USE_USER_UPLOAD true to activate the feature.

That's right, thanks @jeannotdamoiseaux for sharing the links. Closing this issue.


Revert ignored test once CI infra is fixed

Revert this https://github.com/microsoft/ApplicationInsights-dotnet/pull/1155

closed with https://github.com/microsoft/ApplicationInsights-dotnet/pull/1157


feat: move problem engagement MVs

Description
This PR moves the problem engagement to an MV. A testing model called fact_problem_engagement_v2 has been created to compare the old model to the new one.
In the new model, records with empty section_subsection_name are skipped.
Author concerns
The data in the new MVs will only be updated once a new problem event is triggered

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

This seems ok to me, once fact_problem_engagement_v2 is renamed. It seems like the downstream problem engagement reports will all need to be changed to use the new models?

@Ian2012 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.


How can I always fetch latest version of packages from requirements.txt?

Hello,
How can I always do the equivalent of pip install --upgrade -r requirements.txt on heroku with this buildpack?

That is not possible, at this time.
This functionality has been requested by someone before, and was heavily considered — however, at this time it was opted against. Thanks for your feedback! I've added it to my list of things to evaluate as improvements (perhaps with an environment variable switch).
✨🍰✨

environment flag seems simple enough maybe in Procfile or elsewhere


adyenAction hitting quota limitation

Hello,
While placing an order using a Credit Card with a 3D Secure test card
Visa | 4917 6100 0000 0000 | 03/2030 | 737
Adyen cartridge is trying to store payload into paymentInstrument but it is hitting system quota, referring;
if (handlePaymentResult.threeDS2) {     Transaction.wrap(function () {       paymentInstrument.custom.adyenAction = handlePaymentResult.action;     });
And the issue;

Is there a fix or workaround already for that?

Hi @suaterken ,
Thanks for reaching out.
Which version of the cartridge are you using?
Can you make sure that the adyenAction attribute's type is set to text?
https://github.com/Adyen/adyen-salesforce-commerce-cloud/blob/develop/metadata/site_import/meta/system-objecttype-extensions.xml#L99
Please note that this value used to be string in older versions of the cartridge.
Hope this helps.
Kind regards,
Bas Maassen
Adyen

Hello @maassenbas,
We have been using an old version (19.1.0) to a new one 20.1.0.
But indeed, after changing the type from string to text, it seems to be working after all.
Thank you for the quick response.


Prepublish checklist error message on Video Alt text missing is incorrect.

Bug Description
As Videos don't have Alt text (see #5332 ) we should not be presenting prepublish checklist messages about missing Alt text

source - https://stories-new-wordpress-amp.pantheonsite.io/wp-admin/post.php?action=edit&post=13619#page=%2522b1abc9e1-6802-4027-a3eb-7782cf5ceca5%2522
Instead this message should be updated to display the following when no video description has been provided:
Video missing description
Add a video description. Descriptions help with indexability and accessibility (for videos without captions). Include any burned-in text inside of the video.
Expected Behaviour

Steps to Reproduce

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Tested 5417
Message re: missing description not appearing (and seeing "too small" message appear twice):

This was fixed in the image PR; so this is somewhat a duplicate ticket as well. If we do not see this working on main, we can re-open this. The duplicate message is another bug ticket that is being addressed.

Checking Main and seeing this:

Which doesn't match the copy above
(Video missing description
Add a video description. Descriptions help with indexability and accessibility (for videos without captions). Include any burned-in text inside of the video.)

This is somewhat confusing. There are currently two separate checks here, one for missing alt and one for missing title, with two different messages.
https://github.com/google/web-stories-wp/blob/d7223976202e7cb2a89c623d9d0a04e99abaab20/assets/src/edit-story/app/prepublish/constants.js#L163-L169
https://github.com/google/web-stories-wp/blob/d7223976202e7cb2a89c623d9d0a04e99abaab20/assets/src/edit-story/app/prepublish/constants.js#L191-L198
Not sure why only one was updated, but we're actually combining these two fields into one in #5332, so we can keep the one with the right messaging.
Since this is already in main, I suggest to leave this to #5332 to address.

Verified in QA


chore: url error logging

Description

one of the main causes of url errors is from params (e.g. slugs coming through as empty strings)
this PR updates the messaging for the resolveUrl function, so that it's clearer what the underlying cause could be

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


Tuples Insertion with float & double values lose the decimal portion

Laravel Version: 5.6.11
PHP Version: 7.2.3-1
Database Driver & Version: mysqlnd 5.0.12

Description:
On insertions or updates, like when you use the save method from Eloquent model, float and double variables are losing their decimal part
Steps To Reproduce:
Starting by supposing that you have an accounting table in your database as it follows
Table Criation code:
CREATE TABLE IF NOT EXISTS `accounting` (
    `id` INT NOT NULL AUTO_INCREMENT,
    `description` VARCHAR(250) NOT NULL,
    `value` DECIMAL(25,2) NOT NULL,
    `bDeleted` TINYINT(1) NULL DEFAULT 0,
    PRIMARY KEY (`id`))
ENGINE = InnoDB

And that your Eloquent model and Controller have the following code:
Eloquent Model code:
namespace App;
use Illuminate\Database\Eloquent\Model as Model;
class accounting extends Model {
    protected $table = 'accounting';
    public $timestamps = false;
}

Controller code:
require_once FILE_INDEX_BASE_PATH.'/App/accounting.class.php';
class AccountingController extends Controller {
    public function __construct() {
        parent::__construct();
        $this->model = new accounting;
    }

    private function createDemoMovement() {
        $this->model->description = 'Monthly incomes';
        $this->model->value = (float) 128.75;
        $this->model->bDeleted = 0;
        $this->model->save();
    }

    public function viewAccounts() {
        $this->createDemoMovement();
        print_r($this->model->get());
    }
}

When sending a request to the viewAccounts operation from the Accounting controller, the expected output would be:
[{
    "id":1,
    "description":"Monthly incomes",
    "value":"128.75",
    "bDeleted":"0"
}]

But current output is:
[{
    "id":1,
    "description":"Monthly incomes",
    "value":"128.00",
    "bDeleted":"0"
}]

Notice that the current output shows how the decimal part is lost when the tuple is inserted into the table, and you can confirm it by checking directly into the database.
We found out that the issue might be related to the validation applied on the bindValues method, from illuminate\database\MySqlConnection.php file, where the data type bound to the value is defined on a ternary if as it follows:
 is_int($value) || is_float($value) ? PDO::PARAM_INT : PDO::PARAM_STR

so, by changing the validation to:
is_int($value) ? PDO::PARAM_INT : PDO::PARAM_STR

we get the expected output for the given example, just by basically removing the "is_float" portion from the validation

Please add the migration of the value column.

Do you have any casts, accessors or mutators in your accounting model?

No, I don't have any cast, accessors nor mutations in my model.
The code used to replicate the bug is as simple as the one I posted and there are no methods, functions or process in between the assingment of the value and writing in the database
About the definition of the value column, my apologize, missed it when writing the post

It works for me.
Does it work if you use a float/column column in your table?

And try using the decimal column with a string:
$this->model->value = '128.75';

When you use a decimal column, you should use a string in PHP.
When you use a float/double column, you should use a float in PHP.

Even if you use string how does this condition make sense?
  is_int($value) || is_float($value) ? PDO::PARAM_INT : PDO::PARAM_STR
Its basically saying, if its a float then pass on the type as INT when, clearly it not the same data type.
Its pretty clear in the phpdocumentation  the data type of the constant.

I just tried on your recommendations and got the following results:

By setting value column data type to float I get the same result as when it was decimal (the decimal part is lost on the insertion)
By casting the value attribute/variable to string I get the desired behavior, something that I already knew when analyzing the origin of this issue

The problem is that our application is accounting oriented so setting a monetary destined field to float is totally discouraged due the lack of precission, and forcing all the variables to be casted as string before the insertion seems to be unpractical in my opinion taking into consideration that;

Before upgrading from Laravel 5.4 it was working smoothly
If any complementary operations were needed before the insertion, a string type variable would just trigger warnings all over PHP7.2 when trying to perform mathematical operations

I just tested on two different database engines and found out that this issues only occurs with MariaDB engines, the one we are currently using is MariaDB 10.1.31

Laravel doesn't officially support MariaDB.
For monetary values you should definitely use string in PHP. There's no point in using float with a decimal column. Use BCMath for calculations.

Or as integer.

@ardiefernandes @tillkruss
What I think it was missed in this issue is the PDO option ATTR_EMULATE_PREPARES.
When this option is set to true on php7.2 the float values are saved without decimals.
On php 7.0 seems to work fine.
Indeed, the below "if" located in bindValues method in MySqlConnection.php is a bit odd. I think that is_float check should be removed?
is_int($value) || is_float($value) ? PDO::PARAM_INT : PDO::PARAM_STR

@ionutantohi: Why should it be removed?

Why would you set PDO::PARAM_INT for a float in the first place?
Until now, that check was harmful, I don't know yet what is causing this, but something changed from php7.0  to 7.2 or PDO
But now, with php7.2 it's causing issues in some conditions. An one of them is having PDO::ATTR_EMULATE_PREPARES = true
Steps to reproduce:

create a fresh laravel project on a box with php 7.2
set PDO::ATTR_EMULATE_PREPARES = true in config/database.php
perform a insert within a table which contains a DOUBLE (12,2) column with a float value.

Output:

with PDO::ATTR_EMULATE_PREPARES = true, the decimals are not inserted
with PDO::ATTR_EMULATE_PREPARES = false, the value is inserted correctly

If is_float check will be removed, the float value will be inserted correctly, not matter of ATTR_EMULATE_PREPARES option, because in that method, it will default to PDO::PARAM_STR

There's a reason for using PDO::PARAM_INT: https://github.com/laravel/framework/pull/16069

@ionutantohi: Cool, feel free to submit a PR with good test cases to avoid this change causing havoc.

@staudenmeir I understand this, but a workaround it needed for both cases.
Here is what happened to me in real life.
I have a project started on Laravel 5.5 and php7.0. In config/databases.php I had ATTR_EMULATE_PREPARES = true
Inside the projected is a script which saves currency exchange rates multiple times a day.
We upgraded to laravel 5.6 and php7.2. All went ok until we spotted that the exchange_rates were not saved correctly.
Imagine what can happen to a billing project.

Hi @themsaid @taylorotwell, I don't want to bother to much, but I would like to hear your opinion on this before continuing.
In the initial issue, the user compained that the decimals are lost when saved in database.
I experienced the same issue and it seems to be related to PDO::ATTR_EMULATES_PREPARE. When this is set to true, the above scenario happens.
On my project, I can't switch that to false because the database is on a remote server and having ATTR_EMULATES_PREPARE = true gives a performance boost.
Ie:
Query from sessions table:
ATTR_EMULATES_PREPARE=true: 5.9ms
ATTR_EMULATES_PREPARE=false: 89.1ms
database server is in the same LAN
Also, is seems to relate how a value is bound in MysqlConnection.php. A float value is bound as PDO::PARAM_INT and in correlation with ATTR_EMULATES_PREPARE=true produces the above issue.
If that check will be removed, I don't know yet the impact on this issue: https://github.com/laravel/framework/issues/16063
Here are some tests with current codebase of how a value is bound, including PARAM_INT for floats
php7.0
ATTR_EMULATES_PREPARE=true : float is inserted correctly
ATTR_EMULATES_PREPARE=false : float is inserted correctly
php7.2.4
ATTR_EMULATES_PREPARE=true : float looses decimals
ATTR_EMULATES_PREPARE=false : float is inserted correctly
tested on both mysql 5.5/5.7
Thank you!

thanks @ionutantohi, i encounther this issue too because add ATTR_EMULATES_PREPARE. it is needed for 000webhost to work properly and then i move to another hosting without deleting the options. after deleting that, float saved without losing decimal. although i do not measure performance as it is not my focus (for now)

any news on this? Or recommendation how to proceed?
Should I cast every single float value to string when I use laravel 5.6 with PHP 7.2 and ATTR_EMULATES_PREPARE=true?

It looks like this is a 7.2 Issue. In my Custom system I always used PARAM_INT with decimal. Suddenly on Upgrade now I have to go through my whole system and find the Decimal fields and update them to PARAM_STR or it wont save. Gotta love this unanounced change.

For those who monitor this.
I made a pull request (#26817) containing a failing test to demonstrate the issue, but it was closed with the reason that we shouldn't use emulate prepares anymore. In my opinion this is wrong. Emulate prepares is an option in PDO which laravel doesn't handle it correctly on php 7.2
I tried to fix this but with no luck. The pull request which introduced this (#16069) has no tests and to be honest, I was afraid to not break something else.

Change  ATTR_EMULATE_PREPARES=true to ATTR_EMULATE_PREPARES=false does not solve the problem.
If the application use some "mysql view", it enables an "MySQL Bug"  like: Prepared statement needs to be re-prepared MySQL
Please. remove this "is_float" from "MySQLConnection" class

In PHP 7.2, the behavior of emulated queries was fixed, so it makes sense to remove the is_float check for MySQL https://bugs.php.net/bug.php?id=73234 https://bugs.php.net/bug.php?id=77954 @vitordm @driesvints @taylorotwell

I have the same issue: Eloquent saves float as int.
Php 7.4.3, lumen 8.2.3


Verify Developer mode is set on Windows before running extension

@compnerd, @tristanlabelle Is there an easy way to find this out?

What is it precisely that you are after? Developer Mode not being enabled doesn't prevent the functioning of the extension. I wonder what exactly you are trying to determine.

I believe this comes from seeing that the lack of developer mode breaks testing functionality due to the missing symlink.
Developer mode can be tested through a registry key: https://learn.microsoft.com/en-us/windows/apps/get-started/developer-mode-features-and-debugging#use-regedit-to-enable-your-device . This NPM package could be used: https://www.npmjs.com/package/regedit

If it is just symbolic link support, I think that querying for SeSymbolicLinkPrivilege should be checked for. Some people, including myself, do not enable Developer Mode, but do have the privilege which allows everything to work as expected.

SeSymbolicLinkPrivilege

So what key and what value would I test for? Also are there instructions somewhere for an individual to enable this?

There are instructions at https://portal.perforce.com/s/article/3472 (and various other sites, including the steps I've posted to the forums previously).
SeCreateSymbolicLinkPrivilege would be the key - that is the privilege itself that needs to be checked for. https://github.com/microsoft/Windows-classic-samples/blob/1d363ff4bd17d8e20415b92e2ee989d615cc0d91/Samples/ManagementInfrastructure/cpp/Process/Provider/WindowsProcess.c has an example of how to enable the privilege and should be helpful for determining how to check the privilege (just need to check the attributes for SE_PRIVILEGE_ENABLED I believe).

Verified with e548b63188333a84ade036e183a2b90656d64b4f


Fix broken jupyter notebooks

The jupyter notebooks don't work anymore due to the recent Math and Craco reorganizations. This needs to be fixed.

One thing: We should probably wait with this until the import paths are actually fixed, i.e. we have a library name.

Everything but the protocol tutorial works now. The protocol tutorial has the following exception occuring in the Zero-Knowledge part:
---------------------------------------------------------------------------
java.lang.NullPointerException: null
	at ReplyCorrectnessProof.getChallengeSpaceSize(#48:1)
	at org.cryptimeleon.craco.protocols.arguments.fiatshamir.FiatShamirProofSystem.<init>(FiatShamirProofSystem.java:21)
	at .do_it$Aux(#49:1)
	at .(#49:1)


Simple introductory video

It seems like novice Embereños find it increasingly hard to get started with ember-light-table. Most already struggle with passing a table to their {{light-table}} component invocation in the template and get this error:
[ember-light-table] table must be an instance of Table

The live demo page and cookbok are great for the more experienced developers, but are way too complicated for beginners. Many don't seem to understand the simple-table mixin.
We're happy to assist users in our #e-light-table slack channel, but having to answer the same questions all over again is needlessly time-consuming and frustrating, for supporters and users alike.
One user mentioned that there was a vimeo video showcasing ember-light-table, but it was apparently removed.
tbh I didn't even know about that video, but I think it's an awesome idea. Someone (?) should create a short introductory video, that shows how to create a very simple ELT component. No mixins, no ember-concurrency, no special stuff. A second video might show these advanced concepts and maybe transform this basic table into the simple-table example from the demo page.

Exemplary issues:
https://github.com/offirgolan/ember-light-table/issues/21 https://github.com/offirgolan/ember-light-table/issues/407

There already is a video (starts at around 2:06:30) with slides.
Didn't look into it yet, but we should already add that. :smiley:

I know this is a little old, but the original post states the users are encountering this error: "[ember-light-table] table must be an instance of Table", but doesn't explain the error... which I've encountered.


Increase timeout for SKE delete cluster wait handler

Related to https://github.com/stackitcloud/terraform-provider-stackit/issues/241


Proposed fix for #73, open for suggestions

Coverage decreased (-2.8%) to 84.55% when pulling 7ed3b5e693a3c743e991a1b4b6ca651f0fffa2d4 on webbushka:master into 1629a026da17a1e1d8999a02f3fe6b6b60aaac9c on mzabriskie:master.

duplicates #49


Construct Binary Tree from Inorder and Postorder

💥 Proposal
Given two integer arrays inorder and postorder where inorder is the inorder traversal of a binary tree and postorder is the postorder traversal of the same tree, construct and print the binary tree.
Have you read the Contributing Guidelines on Pull Requests?
YES

Kindly assign this to me as a part of GSSOC'21.

/assign

@ankitaggarwal23 Kindly confirm can I do the question.


422 Validation Failed [{Resource:Issue Field:assignees Code:invalid Message:}]

Used the following values for github team and token:

But I'm getting the following error:
Note: The same setup was working in other repo 2 days back but today I created a new repo but not working.
error creating issue: POST https://api.github.com/repos/EQWorks/eq-mwaa/issues: 422 Validation Failed [{Resource:Issue Field:assignees Code:invalid Message:}]

So for the same setup, one repo was able to expand the github team for the list of users and was able to create an issue tagging them but this new repo with the same setup wasn't able to create an issue with same team but when I specifically mentioned user names then the issue was resolved, I'm not sure why this behavior.

Yes, I have some issue on this

  wait-for-approval:
    runs-on: ubuntu-latest
    permissions:
      issues: write
    needs: [get-some-vars,build-to-acr,deploy-to-sit1]
    steps:
      - uses: trstringer/manual-approval@v1
        env:
          current_version: ${{ needs.get-some-vars.outputs.release_number }}
        with:
          secret: ${{ github.TOKEN }}
          approvers: user01,user02,team01
          minimum-approvals: 1
          issue-title: "Deploying ${{ env.current_version }} to pre"
          issue-body: "Please approve or deny the deployment of version ${{ env.current_version }}."
          exclude-workflow-initiator-as-approver: false
          additional-approved-words: ''
          additional-denied-words: ''

I got the same problem: error creating issue: POST https://api.github.com/repos/xxx/xxx/issues: 422 Validation Failed [{Resource:Issue Field:assignees Code:invalid Message:}]
The code used:
    - name: Approval
      uses: trstringer/manual-approval@dd1555b1d17966cb58500676d79a181c53bb6c12
      with:
        secret: "......."
        approvers: "my-group"
        minimum-approvals: 1
        issue-title: "......"
        issue-body: "......"
        exclude-workflow-initiator-as-approver: false

It used to work and suddenly it broke when new user was added  to the group. I tested it on multiple repositories and all comes down to single specific user. If group has this user or this user is specified directly than it fails.

In my case , we had a user with no repository access granted in GHE setup. Removing the user from the approvers list solved the issue.


The "paste direction" is totally unnecessary

On Android when you know how to copy you also know how to paste!
You make a long press on the input field and choose paste!
Nobody needs an additional button here!
You can simplify that screen so much by removing 2 unnecessary buttons.

Thank you for voicing your concerns. We'll take them into account in future redesigns.


Cisco Room Kit: standby being Bool OR String causes errors in placeos/source

The Cisco Room Kit driver works fine.
Though when placeos/source is active, source throws this error:
2022-03-20T07:22:18.421461738Z failure writing points to database: partial write: field type conflict: input field "standby" on measurement "VidConf" is type boolean, already exists as type string dropped=1 (Flux::ClientError)
2022-03-21T04:13:39.278367773Z failure writing points to database: partial write: field type conflict: input field "standby" on measurement "VidConf" is type string, already exists as type boolean dropped=1 (Flux::ClientError)

So I think somewhere in the Cisco Room Kit driver code, it's "standby" state is flipping between a Bool and a String.
Would be better to make it just a consistent Bool OR a String, so that it's value can be stored and analysed in Influx properly.

cant fix this is an issue with the cisco status being multiple types, on the cisco end.
Safe to ignore.


Debugging Guidance Seems out of date in README

https://github.com/microsoft/vscode-test-cli/blob/main/README.md#debugging
States you can make a launch config with a TestConfiguration parameter to specify a test and have this launch a extensionhost test similar to what vscode-test does. This does not seem to work at all, it launches a window but nothing gets triggered or started and TestConfiguration is reported as an unknown parameter. I tried in Insiders too. Is this something that is TBD or maybe was abandoned?
No such parameter exists in the vscode-js-debug options for ExtensionHost debugging: https://github.com/microsoft/vscode-js-debug/blob/main/OPTIONS.md

Same problem for me.
Version: 1.85.1 (user setup)
Commit: 0ee08df0cf4527e40edc9aa28f4b5bd38bbff2b2
Date: 2023-12-13T09:49:37.021Z
Electron: 25.9.7
ElectronBuildId: 25551756
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

@JustinGrote  @reduckted
Did you find any workaround for debugging using this cli ?
Configuration

node: 17
.vscode-test.mjs

import { defineConfig } from '@vscode/test-cli';

export default defineConfig([{
  label: 'Workspace_1',
  workspaceFolder: './src/test/integration/workspace_1',
  files: 'out/test/integration/**/*.test.js',
  mocha: {
    preload: "ts-node/register",
  }
}]);

Attempts
Create a configuration inside launch.json to execute the cli.

Execute the tests: :heavy_check_mark:
Debugging available: :negative_squared_cross_mark:

{
	"name": "Integration tests with vscode-test cli program",
	"type": "node",
	"request": "launch",
	"program"<EMAIL_ADDRESS>	"args": [
		"--label",
		"Workspace_1"
	],
	"console": "integratedTerminal"
}

Workaround
My current workaround is to use a custom runner. But the goal of this cli is to provide an easier method.
CF: https://code.visualstudio.com/api/working-with-extensions/testing-extension#advanced-setup-your-own-runner
{
	"name": "Integration tests with custom runner",
	"type": "extensionHost",
	"request": "launch",
	"runtimeExecutable": "${execPath}",
	"args": ["--extensionDevelopmentPath=${workspaceRoot}", "--extensionTestsPath=${workspaceRoot}/out/test/integration" ],
	"outFiles": [ "${workspaceRoot}/out/test/integration/**/*.js" ],
	"preLaunchTask": "${defaultBuildTask}"
}

Did you find any workaround for debugging using this cli ?

No, unfortunately not.

FYI looks like this was fixed and will be usable in the next build.
https://github.com/microsoft/vscode-test-cli/pull/15#issuecomment-1989465139

It was released to stable last week.


Babelify hackery

@sapegin @carpeliam If you have time would you mind giving this branch a go. I'm hopeful it'll address https://github.com/wheresrhys/fetch-mock/issues/21 (in case you're not familiar with the shorthand for branch installations on npm it's npm install wheresrhys/fetch-mock#babelify-hackery)

Thanks @wheresrhys!

Seems to work fine in Node 0.12. Thanks!


feat(translator): add history_translator

history_translator@history
history:
input: ls
size: 1
initial_quality: 1000

輸入ls時，可以從commit_history中取出歷史記錄。
按backspace和return時，會清空歷史記錄，暫時無法自定義清空鍵。
https://github.com/rime/librime/blob/develop/src/commit_history.cc#L24

Almost LGTM, but
I only meant the default value input: ls was inappropriate because it is based on a particular input method ("pinyin" in this case). This is an assumption that I argue we should avoid in providing a platform default value.
Other fields like tag_, size_, initial_quality_ had default values that should make sense to most users. Please add them back.


Source for the AllJoyn samples?

I wonder why there are no project and source-files available for the AllJoyn samples. Will you add this code?

The source files are available in the AllSeen git -  https://git.allseenalliance.org/cgit/dsb.git. We will update the samples to make it more clear.
Hope that helps,
Lou

Perfect :) Thanks Lou!


WITH RECURSIVE Common Table Expression doesn't parse case of select queries correctly

A clear and concise description of what the bug is.
How to reproduce the bug
While trying to execute a query using WITH RECURSIVE Common Table Expression on RDS MariaDB. I encounter the following error:
Sample SQL query:
WITH RECURSIVE t(n) AS (
    VALUES (1)
  UNION ALL
    SELECT n+1 FROM t WHERE n < 100
)
select sum(n) FROM t

DB engine Error

Only SELECT statements are allowed against this database.

This may be triggered by:
Issue 1022 - Database does not allow data manipulation.

However, I found simply changing the case of the final select query to uppercase works. I understand that WITH RECURSIVE was supported after this bug report but I think the functionality needs to be extended or the error message should be updated accordingly to reflect this.
Expected results
The query should be able to execute successfully with either select and SELECT
Actual results
Instead it results in the following error
DB engine Error

Only SELECT statements are allowed against this database.

Screenshots
select lower case:
[

]
SELECT upper case:

Environment

browser type and version: Firefox
superset version: Version: 2.1.1

Checklist

[ X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ X] I have reproduced the issue with at least the latest released version of superset.
[ X] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Tempted to close this as stale, and because it's reported on Superset 2.x, where we're only supporting 3.1 and 4.0 now.
That said, I would bet that it's still happening... can you verify this? We can keep it open if it's still happening. I'm wondering if the work done recently around SQLGlot might have resolved this along the way (cc @betodealmeida @john-bodley )

Hello @rusackas
I checked and I'm still able to reproduce this behavior on superset version 3.0.3, as we've upgraded our version of Superset since this issue was initially reported.

Thanks! Officially, we support 3.1.x and 4.0.0 now, but I'm guessing the same issue exists there too.

I'm happy to take a look if this is still happening.


https://developer.mozilla.org/ja/docs/Web/API/Window/scrollBy

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

https://developer.mozilla.org/ja/docs/Web/API/Window/scrollBy
I'm not sure, if it's typo or not.
window.scrollBy(X,Y);
window.scrollTo(options)

I think in Syntax section of scrollBy page
window.scrollTo(options) should be
window.scrollBy(options) .

Yup, a typo; fixed!


Custom labels specified not used

Hello, we are making use of the custom labels feature released in 0.9.0 so that we can differentiate between PR's that should be automatically merged and those that should be reviewed. In our case, minor and patch bumps would be automatically merged, while major bumps aren't. The label we are trying to use is simply "automerge".
The reason why we currently can't use the autocomplete and auto approve is because our branch policy requires two reviewers. There are steps after dependabot in the pipeline that will find all PR's with this label and complete them by bypassing policies using the devops rest api.
Here is the script we're using in the pipeline:
      docker run --rm -t \
        -e GITHUB_ACCESS_TOKEN=$(dependabot-gh-pat) \
        -e AZURE_HOSTNAME=dev.azure.com \
        -e AZURE_ACCESS_TOKEN=$(dependabot-pat) \
        -e AZURE_ORGANIZATION={redacted} \
        -e AZURE_PROJECT=${{ parameters.project }} \
        -e AZURE_REPOSITORY=${{ parameters.repository }} \
        -e DEPENDABOT_DIRECTORY=${{ parameters.directory }} \
        -e DEPENDABOT_MILESTONE=${{ parameters.parent_wi }} \
        -e DEPENDABOT_PACKAGE_MANAGER=${{ parameters.pkg_mgr }} \
        -e DEPENDABOT_VERSIONING_STRATEGY=auto \
        -e DEPENDABOT_OPEN_PULL_REQUESTS_LIMIT=${{ parameters.pr_limit }} \
        -e DEPENDABOT_IGNORE_CONDITIONS='${{ parameters.ignore_cond }}' \
        -e DEPENDABOT_EXTRA_CREDENTIALS={redacted} \
        -e DEPENDABOT_LABELS='["automerge", "dependencies"]' \
        -e AZURE_SET_AUTO_COMPLETE=${{ lower(parameters.autocomplete) }} \
        tingle/dependabot-azure-devops:0.10

I see the following note in the github dependabot docs stating that the labels must already exist, so I ensured the label was added to other PRs first:

You can use labels to override the default labels and specify alternative labels for all pull requests raised for a package manager. You can't create new labels in the dependabot.yml file, so the alternative labels must already exist in the repository.

After the pipeline completes, we see the dependencies label but not the automerge label. What am I missing?

@mburumaxwell Dang. Thanks for the reply. Given what I could describe in my original description, does that sound like a reasonable approach to autocompleting minor/patch bumps only? Would you recommend any other way?

Also, it looks like the PR in dependabot-core is waiting for your review

@mburumaxwell Dang. Thanks for the reply. Given what I could describe in my original description, does that sound like a reasonable approach to autocompleting minor/patch bumps only? Would you recommend any other way?

When you are already using the DevOps REST API and the docker image directly in place of the task extension then you are already a pro (not the Apple version). If you are able to bypass policy validation, then by all means you have succeeded at doing the necessary. I cannot see any other way to go around given the policies on Azure DevOps can only filter against paths.
One alternative is to have a develop branch alongside the main only this time without the strenuous policies; direct dependabot to create PRs against the develop branch then occasionally merge the develop branch onto the main (fast forward). It creates an extra step.
While we are here, may I ask if there is a specific reason you use the docker image directly instead of the task?

Want to contribute a fix?

Submitted https://github.com/tinglesoftware/dependabot-azure-devops/pull/403

Custom labels finally merged into upstream in https://github.com/dependabot/dependabot-core/pull/5767

@mmoreno79 there's still no new release for dependabot since 0.215.0 but the new bits are now available here. Could you test using the specific input dockerImageTag: 0.14.2-ci.7?


Correct zgeadd_k prototype

Not sure why this obvious-looking fix would induce build lockups - the appveyor one was not reproducible, and the travis failures were already intermittent when I tried this PR before the update to netlib 3.8.0. (With the exception of subtask 7, the mingw cross-compile, which timed out reproducibly on travis, but still ran without any issues locally). Any travis experts in the house ?

The appveyor failure is reproducible. For some reason, this change (or something else) extended the build time from 50 minutes to one hour, which is the exact amount of time that will lead to a timeout. I suspect it was the LAPACK update.

I opened #1375 to enable build caching, which should resolve these issues. Note that you should merge #1375 immediately because PRs don't cache. Then restart this PR to determine if the build speed is improved. If it doesn't work, you can just revert the PR with GitHub's revert button.

Thanks, we'll see how it goes.

Doesn't seem to have helped unfortunately. (Note that yesterday's PR - already after the LAPACK update - ran normally in particular with travis, though the one appveyor job took 56 minutes there already. So perhaps there is something in the float->double prototype change that influences compilation)

What are you talking about? The appveyor PR has not yet completed.

Sorry, need to get some sleep first it seems. Totally misread that your fix is "only" for the appveyor case, while I am more worried about the travis failures with "no output received within 20 minutes".

Still weird why the build time with appveyor should jump from 50 to 51 minutes to (dangerously close to) 1 hour just from making the zgeadd_k prototype in the header match the actual function.

Unfortunately I did not adjust the test directory. #1376 should address that.

Wait for it to pass before merging though.

Not sure why this obvious-looking fix would induce build lockups - the appveyor one was not reproducible, and the travis failures were already intermittent when I tried this PR before the update to netlib 3.8.0. (With the exception of subtask 7, the mingw cross-compile, which timed out reproducibly on travis, but still ran without any issues locally). Any travis experts in the house ?

Travis will error out if there is no output in the last 10 minutes. While openblas is building, the only outputs are the warnings. When you fix the warnings, there's no output, leading to a gap of 10 minutes with no output. :smile:

Hehe. I was sort of hoping for that explanation, but are you sure travis monitors stderr and not stdout for build activity ? There would still be the regular reports from make/gcc whenever a new file is compiled...

Travis build script uses QUIET_MAKE=1. See https://github.com/xianyi/OpenBLAS/blob/develop/.travis.yml#L20


move code generation scripts to istio/pkg from istio/istio

istio/pkg references istio/istio scripts from go generate directives.
Since we can no longer assume istio is in GOPATH, the scripts need to be updated.
/cc @geeknoid

Also the existing scripts in istio/pkg (fmt.sh) does not appear to be functional.

Fixed by https://github.com/istio/pkg/pull/9


operators community-windows-machine-config-operator (5.1.1)

Signed-off-by: Alina Ryan<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[x] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/hold for manual testing

/hold cancel

/lgtm

/approve

@alinaryan consider adding replaces to 5.1.1.

/merge possible


Engine -> Framework roll stopped on JNI reference leak and Dual Screen breaking change

The Engine roll https://github.com/flutter/flutter/pull/93941 caused failures on framework CI
https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_android cull_opacity_perf__timeline_summary/3117/overview
Additionally, the Engine change here https://github.com/flutter/engine/pull/29585 is a breaking change and can't land in the framework without compensating changes here https://github.com/flutter/flutter/pull/93240.
/cc @jason-simmons @andreidiaconu @blasten

Does this mean that everything might get rolled back until this is fixed?

Looks like we are back in business.


Zoomed images getting stuck

What
Hello! I'm not sure if this is the right channel, but quite often (and now) while I'm looking at something in the player the images gets 'stuck' to my mouse after zooming in/out (it means I can't zoom/in out or centre the image). It's fine after a refresh, but not great if spent time trying to locate a page. Is this a known issue? Or is it me?! Thanks :slightly_smiling_face:
Reported by Nicola Cook
https://wellcome.slack.com/archives/C294K7D5M/p1593446743395400
Where

This is happening on wellcomelibrary.org's universal viewer.
From Jenn: We can't do any development on the Viewer on wellcomelibrary.org. The version we're using is well out of date, and our in-house team has never done any development work on that site. All we can do it let Digirati know about it, but my assumption is that we can't fix it without upgrading to a newer version, which we're not going to do.


[iOS] reduce cursor size for multiline(TextInput)

Summary
Currently in multiline input the cursor touches the previous line.
So this reduces its height sets its position so that I does not touch previous line.
This PR will also fix the issue #28012 (Problem with TextInput lineHeight on iOS)
Changelog
[IOS] [ADDED] - Fixed cursor height on multiline text input

Test Plan
UI Before the change

UI After the change

@javache I have updated my PR based on the comments
Can you please review and say if this approach is correct?

This PR shouldn't be merged at all. It causes bigger issue.
It doesn't completely fix cursor issue on iOS multiline TextInput.
After reducing cursor height, it aligns to top, while text aligns to bottom:
Also, selected range cursor doesn't match.
Before:
https://user-images.githubusercontent.com/108292595/226881193-288a7747-48b1-4e25-bab5-4bc2aa6ecbf5.mp4
After:
https://user-images.githubusercontent.com/108292595/226881219-4aa7d5cf-79df-49c1-b495-70db9ec266db.mp4
The ideal solution should be to align center both cursor and text, same as Text component.

@javache this code should be enough for test. Just return in root App.js
    <View style={{flex: 1, paddingTop: 200}}>
        <View style={{marginHorizontal: 50}}>
            <Text>Text</Text>
            <Text style={{
                height: 110, borderWidth: 1, lineHeight: 50, fontSize: 16,
            }}
            >
                sdj fdsjflj slafjal ksdj afklakjfkajdskfj; asjfjs;dfs klfjalsjf j;ksd fkjalkj sfjlks j
            </Text>
        </View>
        <View style={{marginHorizontal: 50, marginTop: 50}}>
            <Text>TextInput</Text>
            <TextInput
                multiline
                placeholder="sdj fdsjflj slafjal ksdj afklakjfkajdskfj; asjfjs;dfs klfjalsjf j;ksd fkjalkj sfjlks j"
                style={{
                    height: 110, borderWidth: 1, lineHeight: 50, fontSize: 16,
                }}
            />
        </View>
    </View>

@soumyajit4419 do you think the issue was fixed competely?
Still has issues:

suggestion highlight rectangle is still aligned to the top
selection rectangle still takes full height
text and caret should be center aligned like Text view, not bottom

https://user-images.githubusercontent.com/108292595/227418790-7c400909-c105-405b-a7a9-ee0d103ac3bf.mp4


Copy-pasting doesn't work in Mac OS X

Tested with mofowallet.app (workaround: use the separate fimk release, works fine).

Hywää wappua Dirk @fimkrypto, ota kalja @ChangeTip!

Hi @fimkrypto, @evaarnas sent you a Bitcoin tip worth 1 kalja (24,277 bits/€5.00), and I'm here to deliver it ➔ collect your tip.
Learn more about ChangeTip

Hi @evaarnas ,
Sorry again for this (extreme) late reaction (email notification of issues seems to be mis-configured).
If you would be so kind please verify if this issue is still the case since the most recent mofowallet MacOSX desktop release?
Thanks
Dirk
ps. thanks for the beer!

I finally managed to install the latest mofo lompsa. Unfortunately this problem still exists. :-(
Dirk @fimkrypto, kalja taisi ehtiä väljähtyä, tässä uusi @ChangeTip!

Hi @fimkrypto, @evaarnas sent you a tip worth 1 kalja (22,503 bits/€5.00), and I'm here to deliver it ➔ collect your tip.
Learn more about ChangeTip


loss spike when training qwen1.5 with sample_packing:true

Please check that this issue hasn't been reported before.

[X] I searched previous Bug Reports didn't find any similar reports.

Possible solution

Could it be because there are multiple samples in one input and the loss is not averaged over them?
Does this have any effect on training?
let me know if any other information is needed

Which Operating Systems are you using?

[X] Linux
[ ] macOS
[ ] Windows

Python Version
3.10/docker
axolotl branch-commit
main/decb66e17013
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this bug has not been reported yet.
[X] I am using the latest version of axolotl.
[X] I have provided enough information for the maintainers to reproduce and diagnose the issue.

Hey, is this behavior consistent? Does this happen with other datasets or on another retry? How's your eval loss?

It is consistent through reruns and across different subsets of this dataset. However, I currently don't have access to test it with a totally different dataset

You may try the datasets in the example configs for testing though they're a bit small.


does it support parasitic power?

does it support parasitic power?

Hi @youxiaojie
Yes, it detects the power mode automatically. It even supports mixing devices with different power modes.

1.if different type of sensors, 1822 18b20 18s20 in the same line, is it ok?
2. 3 sensors in system, so It will initialize 3 instances from class and get temp then print it. the question is how do I which one represent which component? and if I remove one, how the program do? eg, I hope to show:points1:23C point2 25C point3 28C for 3 sensors, if the middle sensors is removed, I hope to show:points1:23C point2 NC point3 28C. could you give me some advice.

Yes, I believe that should work fine. Please let me know if you encounter any problems.


why judge degenerate by eigenvalue less than 10 ?

https://github.com/RobustFieldAutonomyLab/LeGO-LOAM/blob/aa37d0a9553f94ec5c59755cf5fb0a15f84a53f4/LeGO-LOAM/src/featureAssociation.cpp#L1337
I don't know why judge degenerate as above method?

I have the same question. Why do u judge degeneracy by value 10? does anyone know?


Hanif

@YSchneiter  now u can merge with ag_dev b

Coverage decreased (-1.2%) to 33.123% when pulling 82427af8ddfbd6767fe6aec46fe32f247b7e4036 on Hanif into dc4577a93bd1c953e76887dbb9ce8d22cb464bd4 on ag_dev.


Add recipe annotation to prevent cd-ing to justfile

Proposed along with a few other features in #761, moving into its own issue to track separately.
It might be nice to have a recipe annotation that makes just use the invocation directory as the current working directory for that recipe, like so:
#[no_cd]
foo:
  pwd # will print invocation directory, not justfile directory

Just a stylistic suggestion: perhaps #[cwd] is more idiomatic than #[no-cd]

Until this is implemented, you can add
#!/usr/bin/env bash
cd -

which will take you back to the directory where you executed the just command.

I can't get this to work:
foo:
  #!/usr/bin/env bash
  pwd
  cd -
  pwd

$ just foo
/Users/rodarmor/tmp/cd
/var/folders/b5/q2c4mxk534v6kwdtx2mpw91m0000gn/T/justF2cWhz/foo: line 4: cd: OLDPWD not set
/Users/rodarmor/tmp/cd

Hmmm, I can only get it to work when my Justfile is located in $HOME/Justfile. Didn't pay attention to that...

We have recipe attributes now, so this was very simple to implement. It's available on master if you want to give it a shot!


[QST] Is there any example on how to "Sequential Recommendations"?

❓ Questions & Help
Details
Following here: https://github.com/NVIDIA-Merlin/Transformers4Rec/discussions/384#discussion-3941348

@whoknowsB hello. We do not have an example for Sequential Recommendations in our GH repo yet. Our examples are particularly on session-based models. you can easily adapt them for sequential recommendation. You can groupby by user_id instead of session_id, and create sequential interactions per user.

@whoknowsB can we close this issue?


[nrf noup] net: sockets: Add new DTLS and RAI socket options

Introduce new nRF91 specific TLS and RAI socket options.
Additionally, rework how local extensions to the networking headers are handled downstream. Instead of modifying the upstream socket.h header, introduce a new one, specifically for NCS extensions, and keep all of the local changes in there. The only change needed in the original header is an inclusion of the new header. This should help to keep the local changes in order and facilitate the future upmerge process.

I would like to get this in because we will introduce a new socket option with libmodem v1.2.0 which we'd want to keep in the separate file that is being created in this PR.

We need to wait for the upmerge...


Colegiul National Emanuil Gojdu Oradea

Website of the school: http://lego.rdsor.ro

@clau100 Please provide us with:

the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@clau100 You might miss my previous comment. Please react.

I did not receive a notification for the first message, I am so sorry for
the inconvenience. I will ask my school for the papers and will get back to
you as soon as possible.
On Mon, Mar 14, 2022, 17:13 Philip Torchinsky @.***>
wrote:

@clau100 https://github.com/clau100 You might miss my previous comment.
Please react.
—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/13744#issuecomment-1066923794,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ73ICIR3AWUFPMFPTFEXLLU75JQLANCNFSM5QPXOZ6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

@clau100 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!
You can also apply for a free educational license with an official document that proves that you are a student. You can do it using the "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.


What about older versions of android?

Can you be more specific with your question?
In general, this project is more like a proof of concept, we don't do any compatibility testing.

I got it to work on android 28.
Here is the out put of my sdkmanager
./sdkmanager --sdk_root=${ANDROID_HOME} --list_installed --verbose
Installed packages:
build-tools;30.0.2
Description:        Android SDK Build-Tools 30.0.2
Version:            30.0.2
Installed Location: /home/mongoose/android-sdk/build-tools/30.0.2
build-tools;32.0.0
Description:        Android SDK Build-Tools 32
Version:            32.0.0
Installed Location: /home/mongoose/android-sdk/build-tools/32.0.0
cmake;<IP_ADDRESS>88404
Description:        CMake <IP_ADDRESS>88404
Version:            3.10.2
Installed Location: /home/mongoose/android-sdk/cmake/<IP_ADDRESS>88404
emulator
Description:        Android Emulator
Version:            31.1.4
Installed Location: /home/mongoose/android-sdk/emulator
ndk;21.3.6528147
Description:        NDK (Side by side) 21.3.6528147
Version:            21.3.6528147
Installed Location: /home/mongoose/android-sdk/ndk/21.3.6528147
patcher;v4
Description:        SDK Patch Applier v4
Version:            1
Installed Location: /home/mongoose/android-sdk/patcher/v4
platform-tools
Description:        Android SDK Platform-Tools
Version:            31.0.3
Installed Location: /home/mongoose/android-sdk/platform-tools
platforms;android-28
Description:        Android SDK Platform 28
Version:            6
Installed Location: /home/mongoose/android-sdk/platforms/android-28
platforms;android-30
Description:        Android SDK Platform 30
Version:            3
Installed Location: /home/mongoose/android-sdk/platforms/android-30
sources;android-28
Description:        Sources for Android 28
Version:            1
Installed Location: /home/mongoose/android-sdk/sources/android-28

I applied the following installation procedure to run the app on android 28, based on the info provided in this post:
yes | $ANDROID_HOME/cmdline-tools/bin/sdkmanager --sdk_root=${ANDROID_HOME} --licenses
$ANDROID_HOME/cmdline-tools/bin/sdkmanager --sdk_root=${ANDROID_HOME} "build-tools;30.0.2" "build-tools;32.0.0" "cmake;<IP_ADDRESS>88404" "emulator" "ndk;21.3.6528147" "patcher;v4" "platform-tools" "platforms;android-28" "platforms;android-30" "sources;android-28"

I used the commandlinetools-linux-9123335_latest.zip package from here.
Some of the installs by the sdkmanager may be redundant, but I did not check this.
Note that ANDROIDVERSION?= line in Makefile has to be changed to version 28 prior to running make keystore and make run commands.
Otherwise, INSTALL_FAILED_OLDER_SDK error occurs during $(ADB) install -r $(APKFILE) step in make run.
Thank you @SelfTide, for your helpful post.


Add option to authenticate via proxies and with verification cerificates

It is currently not possible to pass proxies and  verify parameters upon connection. This imposes substantial restrictions to data access for users working in institutions under proxies.
It would be an easy change amounting to including default parameters proxies=None and verify=True to a number of functions such as

All calls to requests
pyam.iiasa.Connection so that this can be called e.g. as conn = pyam.iiasa.Connection(proxies=PROXIES['https'], verify=where())
pyam.read_iiasa
etc

Would you kindly be able to push a new version accounting for this?

I believe PRs are welcome so feel free to contribute the changes needed and ask for a review


feat(vm-snapshots): add validation for ability to revert a vm

https://github.com/bwsw/cloudstack-ui/pull/1495

Pull Request Test Coverage Report for Build 1500

6 of 9 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.007%) to 53.798%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/root-store/server-data/vm-snapshots/vm-snapshots.actions.ts
6
9
66.67%

Totals

Change from base Build 1499:
0.007%

Covered Lines:
8861

Relevant Lines:
14552

💛  - Coveralls


Timezone support for events coming from Calendars

Hello,
Thank you for the great plugin; I love it so far! I added my work calendar (secret iCal from Google Calendar sharing), and it loaded events correctly, for the most part. Some of the events were sent from India and have the India Timezone attached to the event; When Time Ruler pulls this specific event, it shows to be scheduled at 9 PM instead of 8:30 AM.

Raw event with details omitted.
BEGIN:VCALENDAR
BEGIN:VEVENT
DTSTART;TZID=Asia/Kolkata:20231024T210000
DTEND;TZID=Asia/Kolkata:20231024T213000
RRULE:FREQ=WEEKLY;BYDAY=TU
DTSTAMP:20231023T205553Z
ORGANIZER;CN=[omitted]
UID:[omitted]
ATTENDEE;CUTYPE=INDIVIDUAL;ROLE=REQ-PARTICIPANT;PARTSTAT=ACCEPTED;CN=[omitted];X-NUM-GUESTS=0:mailto:[omitted]
X-GOOGLE-CONFERENCE:[omitted]
CREATED:20220118T181438Z
DESCRIPTION:[omitted]
LAST-MODIFIED:20231023T204757Z
SEQUENCE:1
STATUS:CONFIRMED
SUMMARY:Eng Hangout
TRANSP:OPAQUE
END:VEVENT
END:VCALENDAR

Would you prefer the event be shown relative to the current timezone, or all show up when they are scheduled regardless of timezone?

yes, events should be displayed in the current timezone.

This should work in 1.5.2.

I can confirm it's fixed for some events, but not for others. I'm not sure why Google Calendar uses two different styles for the ical event, but I went ahead and isolated a couple of examples (coming directly from my own calendar) and put it all in a jsfiddle for you. Let me know if you need anything else to make this work.
https://jsfiddle.net/kalbasit/gz8pxujo/  You'll notice in my jsfiddle the local timezone is hardcoded to mine but you can comment that line and uncomment the one above to get the one in your local timezone.

seems to be fixed

sorry, thought i was responding to another fix.  haven't tested by will

https://github.com/peterbraden/ical.js/issues/50 looks like the ical library you're using doesn't really handle this style of timezone.

Yes, I'll probably have to parse the data myself or find a new one

actually,  the library is apache 2 license.  just use and modify.  you can also submit a pull back to the original to add the capability.  less work.
one thing to remember is that the starting/ending times can be in different timezones

Alright, I'll give that a shot.

I did a quick look and for some reason the code uses a regex to pick up the date.  Not sure why as the date function of javascript ccan parse an ISO date (including timezone).  You might also want to look at this obsidian pluging that parses ical entries.  Haven't checked if it handles timezones though

I'll close this issue until someone wants to PR it.


Loosen requirement on rugged ?

Is it possible to loosen the gem version requirement for rugged ? We are currently using the 1.2.0 version of rugged.
Maybe it's possible to use  < 2.0, similar to https://rubygems.org/gems/licensee ?

Oh this was already done in https://github.com/grodowski/undercover/blob/master/undercover.gemspec#L30. @grodowski, is it possible to publish a new version of the gem ?

Thanks for bringing this up @kuahyeow, pushing a new tag soon!

Thanks ! Looking forward to using this in GitLab (see https://gitlab.com/gitlab-org/gitlab/-/merge_requests/74859)

Done and included in 0.4.4. Thanks for reporting @kuahyeow


wbr tag needs a css compatibility fix for some browsers

See comment in PR

@jdillard can you please review this one?


[Redesign] Spinning loading indicator does not spin in Edge

https://preview-dev.nugettest.org/stats/packages/BaseTestPackage?groupby=ClientName
No high contrast, logged out.
Try clicking the checkboxes, notice that loading indicator appears, but does not move

Apparently, Edge doesn't support animations on SVGs.
See https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/8487752/

Addressed with [Redesign]Fix loading Spinner in Edge/IE #4398


Beta transaction details

still WIP, will add more later tonight or tomorrow

noticing something a little off with the history. when I go to a block, and navigate to a tab that's not the first tab, pick a txn, and then go back, it directs me back to the first tab in the block


Add package lint tool

it would be nice to lint packages for correctness.
things like:

are you installing binaries into /opt?
are you installing any files into / that are more than X in size?

Using something else than a SHA-1 sum in the revision= field should be disallowed, following this reasoning.

* Using something else than a SHA-1 sum in the `revision=` field should be disallowed, following [this reasoning](https://github.com/matteodelabre/toltec/pull/1#discussion_r483542742).

Unless we allow another mechanism to validate the source like makepkg allows.

Sounds like this needs a configuration language, not an effectfull programming language.
I'm not sure what the constraints of toltec are but if you want to stay with shell, maybe shellcheck should at least be part of the linting. Maybe it can even be tuned to verify new things you need? My 2 cents.

Sounds like this needs a configuration language, not an effectfull programming language.
I'm not sure what the constraints of toltec are but if you want to stay with shell, maybe shellcheck should at least be part of the linting. Maybe it can even be tuned to verify new things you need? My 2 cents.

We do actually have shellcheck running in the linting right now, but it can't lint the non-shell specific stuff we have in the package definition itself. That requires context of the system that shellcheck doesn't have.

Sounds like this needs a configuration language, not an effectfull programming language.

The good side of this choice is that we can use the metadata fields inside the recipe functions, e.g. to parameterize sources with the version number. This is the model that Arch Linux uses. But of course we can transition to something else if it is warranted.

I'm not sure what the constraints of toltec are but if you want to stay with shell, maybe shellcheck should at least be part of the linting. Maybe it can even be tuned to verify new things you need? My 2 cents.

shfmt has a -tojson flag which exports the internal AST. We could repurpose that to check project-specific syntax constraints.

Lint check: error if an unknown field or unknown function is declared and does not start with _. (See #217.)

Lint check: error if an unknown field or unknown function is declared and does not start with _. (See #217.)


Downlink to LoRa devices

Hi everybody.
This is a question for an integration.

What do you want to do?
use a watteco temp/hum sensor

What steps did you take?
Add the device on thethingnetwork, thats work

What went wrong or what is missing?
1 - I don't understand why thethingnetwork send a message with empty payload after the temp or hum frame. Where this message come from?
2 - I want to send configuration message to my device but i don't understand how to do.
I try in the dowlink tab... that appear in application data but nothing more...
Do you have explanation?

Do you have Screenshots?

What kind of OS/Browser/Gateway are you using? Which version?
packet_forwarder on pi with picoGW
drive thethingnetwork by chrome on windows 7

What are the IDs and EUIs of your Device/Gateway? (if applicable)
gateway : eui-414456414e54454e

@mazareth thanks for reporting.

Payload on FPort 0 are MAC commands, these are sent by the network server as per LoRaWAN specification;
Seems like you do the configuration right. You can send the downlink by pressing the button. You will see it appear scheduled in the Application Data, and then sent off the next time there's an opportunity to send

Hi Johan,
Thanks for 1.
For 2 - Do you know where the opportunity to send is calculated?
What's the Fport change?
What's the confirmed checkbox change?

@mazareth yes, on class A the network has an opportunity to send only directly after an end device's uplink transmission.
I'm closing this issue as I think it is resolved.
Did you know that there's a #support channel on our Slack? Also, there's the public forum.

@johanstokking  no i don't know where i can't ask question.
Could you give me a link?
And how can i change the class of the network?

Class C is experimental in TTN Stack V2 (this version), but class B and C will be generally available in TTN Stack V3, launched 2018 Q1.

See this post on the forum


GUI-Alpha Kubernetes Installation does not work

I have tried all possible ways to install from the load balancer IP but the webpage times out every single time. Please try and fix this since Keycloak has a stable Kubernetes deployment but Gluu still lacks one. Thanks

Bug: Installer does not load. IP times out.

Your bug report is not following any report standards. You need to be clear about your report. Most importantly, the GUI is alpha version which means it's not meant to be used to install production systems. We have  stable pathways for installing Gluu on Kuberenets and all are currently being used by several customers.
Please refer to our docs : https://gluu.org/docs
The stable version of the GUI will be out in Gluu 5.0, so at the moment please use our stable pathways and open a bug report with detailed information as the template walks you through that . Skipping that does not help at all. The pathways that the GUI installer uses have been tested and are recurringly tested per commit , making a claim needs evidence and a path of reporducing it so we can address and fix .
Please open an issue with more information.
Thanks


Deposit verifier waits certain period to take effect

Fixes #174

I think we don’t need a swap. We can just put a
if(now > period) useNew else useOld.
In register, we put the new to old and update the new one with new period.
(Sorry for not well formatted comment, done by phone)

I think we don’t need a swap. We can just put a
if(now > period) useNew else useOld.
In the getter.
And then in register, we put the new to old and update the new one with new period.
Then the cost of deposit would be constant.

Technically yes, but is it clean? I mean the code becomes dirtier, because you newer know which verifier will be used by just the variable name.
I think I prefer the swapping solution.

We can wrap functions for what client would be interested instead of relying directly on the variable in. Eg. currentEffectingContract() nextIncomingContract() nextIncomingContactEffectingTime()
Well...it’s more or less do we want to be customer obsessive and add some complexity to us or not.

But still you do not have a single variable ex currentVerifier

If we’ve provide enough api for client the currentVerifier is of no interest to anyone then we just keep it private?

I will leave it to you two to choose one and move forward. Personally, I prefer two steps or the abstracted apis to be customer obsessive.
I think both two steps or abstracted apis are kind of complexity to us in a different way though. (Or there might be better option? :)

I've updated solution description in the comment


analytics: report to InfluxDB by default.

Now that this is ready: let's roll it out to everyone in 4.0.0.
Do not merge until we've done some final testing.

Review period skipped due to critical label.


EDA

Get larger datasets
Check how the Data look
Check dataset balance
see if we can predict the text with our own eyes

@EsterIlyasov we need to add the new dataset to the repo, so we could use it from here instead of Google Drive


Remove race in setting/reading kubelet.podCIDR

LGTM.

Need leads LGTM for this?

Bouncing to @alex-mohr for another LGTM.

Does this close an issue?

LG, merging.

It is a potential fix for #10622. No proof, but it was our best lead, and an obvious bug.


Improve exception handling behaviour of InstrumentingClasspathFileTransformer

Include instrumented file name in the timeout exception.
Don't try to create the receipt file when a concurrent writer wins.

@bot-gradle test QFL

OK, I've already triggered QuickFeedbackLinux build for you.


Change TFM to netcoreapp2.0

cc @kichalla  @Eilon

Change the Internal.AspNetCore.Sdk version to 2.1.0-*.

@pranavkm It already is 2.1.0-* https://github.com/aspnet/JsonPatch/blob/jbagga/netcoreapp2.0/build/dependencies.props#L5


fix xDS protocol mishandling

Signed-off-by: Brandon Cook<EMAIL_ADDRESS>fixes #499

@phylake can you please try this diff instead
% git diff
diff --git a/internal/grpc/xds.go b/internal/grpc/xds.go
index 9718d5d..693f837 100644
--- a/internal/grpc/xds.go
+++ b/internal/grpc/xds.go
@@ -16,6 +16,7 @@ package grpc
 import (
        "context"
        "fmt"
+       "strconv"
        "sync/atomic"

        "github.com/envoyproxy/go-control-plane/envoy/api/v2"
@@ -100,7 +101,7 @@ func (xh *xdsHandler) stream(st grpcStream) (err error) {
                        log.Info("stream_wait")

                        // now we wait for a notification, if this is the first time through the loop
-                       // then last will be zero and that will trigger a notification immediately.
+                       // then last will be less than zero and that will trigger a notification immediately.
                        r.Register(ch, last)
                        select {
                        case last = <-ch:
@@ -117,10 +118,10 @@ func (xh *xdsHandler) stream(st grpcStream) (err error) {
                                }

                                resp := &v2.DiscoveryResponse{
-                                       VersionInfo: "0",
+                                       VersionInfo: strconv.Itoa(last),
                                        Resources:   resources,
                                        TypeUrl:     r.TypeURL(),
-                                       Nonce:       "0",
+                                       Nonce:       strconv.Itoa(last),
                                }
                                if err := st.Send(resp); err != nil {
                                        return err

Closing in favor of #1016


What does EFV:NLI means?

Hello,
Emails I am trying to send from my custom domain ends in outlook.com with this header:
X-Forefront-Antispam-Report: EFV:NLI
Could you elaborate on the EFV:NLI so that I can "Determine the reason why the message was marked as spam" ?
Have a good day !

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

ID: bbab747e-1a43-037e-09ca-9c402deec602
Version Independent ID: 1c6d8e28-034d-24d6-5732-b96475411d6b
Content: How to prevent false positives from happening in Office 365
Content Source: microsoft-365/compliance/prevent-email-from-being-marked-as-spam.md
Service: o365-seccomp
GitHub Login: @stephow-MSFT
Microsoft Alias: stephow

@Yanal-Yves  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Hello @Yanal-Yves , have you tried any option from https://answers.microsoft.com/en-us/outlook_com/forum/all/email-ist-going-to-junk-folder-although-antispam/08262593-ef1a-4436-8a23-182ca43015ff
I would recommend you to open up a support case with Office365 support team for its further investigation. I would recommend checking if your domain not marked as blacklist on the internet.  For this purpose you can use any domain blacklist check tool.
Regards,
Imran.

@Yanal-Yves - From our understanding, the issue you raised has been answered by joinimran. If you feel it hasn't been answered, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@mypil ,
I don't think that the question "What does EFV:NLI mean?" has been answered.
Where does EFV:NLI stand for? And what triggers it?
I am having the same issue as @Yanal-Yves

@Yanal-Yves @leonNL - I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let me know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.

@joinimran, the link you are suggesting didn't help me to get the meaning of X-Forefront-Antispam-Report: EFV:NLI.

SPF and DKIM are correctly configured according to mail-tester.com.
Our email provider is maiJet and according to the dashboard our domain is correctly configured.
Emails sent from our domain are not falling in Gmail spam

I read somewhere that using a new IP to send emails is a reason to fall in the outlook spam.
So I will wait and see if our emails continue to fall in the spam.
Anyway, I wasn't raising an issue to get support for my particular issue but simply get the anti spam report directive documented for the community and then maybe I was hopping to find a solution for my particular problem.
Do you guys at Microsoft knows what X-Forefront-Antispam-Report: EFV:NLI means?

Did you review the EOP codes on this link? https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/anti-spam-message-headers

Hi @Yanal-Yves Any headers not documented here (https://docs.microsoft.com/en-us/microsoft-365/security/office-365-security/anti-spam-message-headers) are internal and used exclusively by the Microsoft anti-spam team for diagnostic purposes.

@LauraWi (Principal Content Experience Manager at Microsoft)
Are you stating that EFV:NLI is only for DIAGNOSTIC reasons and does is NOT has effect how mail are handled (e.g. Outlook) and that it NOT causes the mail to arrive in unwanted messages folder?

@LauraWi, so if any hearders not documented in the page you mentioned is internal, maybe the resolution for this ticket is to add this information to the wiki page. That would avoid (or at list limit) this kind of ticket in the future :-).

Hi Yanal -- The topic already includes this information in the statement " Other fields in this header are used exclusively by the Microsoft anti-spam team for diagnostic purposes." But we can look at making the information more prominent (perhaps in a note), if confusion persists. Thanks.


git clone information missig

The information that it is required to add --recurse-submodules to the git clone is missing in readme.

The sequence of commands:
git clone https://github.com/OPCF-Members/UA-IIoT-StarterKit.git
cd UA-IIoT-StarterKit
git submodule update --init

should do the same thing.

Closing a stale issue.


Yusup

Check bang

Terimakasih Bang 🙏
On Wed, Dec 14, 2022, 08:54 raihanprojecthub @.***>
wrote:

Merged #12 https://github.com/TIM-SUKSES/similu/pull/12 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/TIM-SUKSES/similu/pull/12#event-8031836593, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A3ARQTINTN4PNXDGT3OOYWLWNESFJANCNFSM6AAAAAAS54BGWE
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix: use more precise types for move_requires_*

... and add move_requires_4, a companion to forall_intro_4 that was missing.
I noticed that move_requires_n don't use dependent types like forall_intro_n do, which I needed, hence this PR.

Thanks Théophile! This looks good. Just that somehow it makes LowStar.Monotonic.Buffer go into an infinite loop... I'll take a look but I killed the CI for now.

Apparently randomly, this query was causing Z3 to go into an infinite loop without burning rlimit. Some tweaks avoid. It's a good reminder to bump the default Z3 version.
Thanks again, sorry for the snafu.


Feature/sample from grids

This pull request contains both the additional feature of sampling from grids and cleaning up the model test libraries namespaces.  Review should be focused only on the first two commits. Don't waste your time reviewing all the files related to the namespace change.
Currently this will not pass tests as the Sample Points functionality for a raster is not yet available in the RASMapper libraries that are currently under development. I'll coordinate with our friends over there to get that accomplished.

Libraries were updated. This is now fully implemented and functional

By the way thanks for the heads up to only look at the first two commits


Update meta.yaml

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Last time I checked, Ansible playbooks cannot be executed on Windows hosts (but they can be used to provision Windows machines).
Has this changed?

You probably need to define an entrypoint to get the ansible batch test to pass

You probably need to define an entrypoint to get the ansible batch test to pass

@SimeonStoykovQC You should get these entrypoints by regnerating the recipe using https://github.com/conda-incubator/grayskull

This PR should really be closed. You cannot run Ansible on Windows. See here

This PR should really be closed. You cannot run Ansible on Windows. See here

Damn.


[Bug] when where clause column type is bigint , value equal faild

Search before asking

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

Version
1.1.5-rc02
What's Wrong?
TABLE ：
CREATE TABLE test_bigint (
day_id int(11) NOT NULL COMMENT "分区字段,如:20211001",
device_id varchar(255) NULL COMMENT "ID",
analysys_id bigint(20) NULL COMMENT "ID"
) ENGINE = OLAP DUPLICATE KEY(day_id, device_id)
COMMENT "OLAP" DISTRIBUTED BY HASH(day_id) BUCKETS 1 PROPERTIES (
"replication_allocation" = "tag.location.default: 1",
"in_memory" = "false",
"storage_format" = "V2"
);
SQL :

What You Expected?
Fix it
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

我这边发现并无异常

sorry , the bug is doris web only show<PHONE_NUMBER>5660600 , realy value is<PHONE_NUMBER>566060097

same problem : https://github.com/apache/doris/issues/10313


Get Tomorrow's date

I'm trying to get tomorrow's date like this, but it just returns today's.
moment().add(1, "days").format("MMMM do")
This works fine in Javascript
moment().add(1, "days").format("MMMM Do")
Your Environment
Swift 4.0
Xcode Version 9.2 (9C40b)

let currentDateMoment = moment(currentDate)
let numberOfDateMoment = currentDateMoment.add(numberOfDays, TimeUnit.Days)
return numberOfDateMoment.date
let current date is your from date
how manys to added to number of days


annotations.md: added useful annotations

Other label schema projects include the following items that are not yet
included in OCI Images. This PR adds support for version, vendor and
revision to bring parity to OCI defined image labels.
Signed-off-by: Stephen J Day<EMAIL_ADDRESS>

Added licenses field.

so, are these fields effectively duplicating what label-schema has defined?
so users might expect to see both?
On Wed, Apr 5, 2017 at 3:35 PM Stephen Day<EMAIL_ADDRESS>wrote:

Added licenses field.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/pull/636#issuecomment-291972645,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6eJH26KoqAbxEZlel7nio0EpD0Y_ks5rs-0bgaJpZM4MzQF1
.

so, are these fields effectively duplicating what label-schema has defined?

Doesn't that make sense to do if the fields are useful? Maybe I am misunderstanding label schema but isn't it just a proposal for the naming, relying on it for useful information as part of the image spec standard does not make much sense.

so, are these fields effectively duplicating what label-schema has defined?

OCI already has about 70% of what label-schema.org provides. Should we delete those fields, as well?

so users might expect to see both?

These are annotations. What is the issue with that? They are completely compatible.
I am extremely confused about the push back here.

I am extremely not pushing back, but rather curious
On Wed, Apr 5, 2017, 16:23 Stephen Day<EMAIL_ADDRESS>wrote:

so, are these fields effectively duplicating what label-schema has defined?
OCI already has about 70% of what label-schema.org provides. Should we
delete those fields, as well?
so users might expect to see both?
These are annotations. What is the issue with that? They are completely
compatible.
I am extremely confused about the push back here.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/pull/636#issuecomment-291985719,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEF6VH6W7sAoRlkjm9LhtuKSuAneLDjks5rs_hMgaJpZM4MzQF1
.

I am extremely not pushing back, but rather curious

:)
In general, I think adding these to OCI will create better adoption. Considering we have the bulk of these already, going the extra distance seems quite harmless.

I kind of agree with https://github.com/opencontainers/image-spec/pull/635#issuecomment-291928199 - ending up with duplicate annotations for exactly the same thing seems unfortunate, especially when these are both quite nascent efforts.
Turning this around, let's assume we merge this PR, I wonder if label-schema folks would be okay making a reference to the official values here? I am not hugely fussed about whether the conventions live here (as I endorsed at a while back), or externally in label-schema, but it would nice if we could consolidate.

Maybe I am misunderstanding label schema but isn't it just a proposal for the naming, relying on it for useful information as part of the image spec standard does not make much sense.

I don't understand this point; what's the difference here exactly?

@jonboulle While I wasn't for adding these labels in the beginning, I do see that there is a desire for them in the community. As it stands, 70% of the label-schema labels were already in OCI, minus the *.cmd labels, so I thought it would be better if we filled in that gap here. In general, it has a much better chance of succeeding here than elsewhere.
And adding them here doesn't mean label-schema has to go way. They are namespaced and should generally be compatible. If a user only needs these base labels, they can use those. If more or different functionality is required, then they can adopt something else.

After some amount discussion, while the adding of these keys is innocuous, I think the issue is:
For folks that are already using the org.label-schema. prefix, there is no benefit to having an org.opencontainers. equivalent. And for any tooling that operates on one, now may have to respect both.
This effectively is seeking to shadow the patterns put forth by the label-schema.org, without providing a path to why they should switch.
While I would rather just point reference to an existing convention, perhaps the conversation here would be one of bringing in the label-schema (adopting it) such that their existing prefix is reserved and the fields are directly honorable. (with discussion around not including the docker and rkt ones). This obviously would require a opening the conversation with the maintainers of label-schema.
Regardless, the addition of these fields or adoption conversations were agreed to not be blocking of a v1.0.0-rc6 or final release, but rather a good-to-have discussion.

I hope it's ok for me to jump in and add my 2p's worth as a label-schema maintainer: I would far rather see a single set of label names rather than two!
We started the label-schema project with the idea of helping the community settle on a single convention for common labels. Two conventions is worse than one, and confusing. If the OCI spec is the right place to define the convention, I for one say we should retire label-schema in its favour and point at the OCI spec.
That said, there are now over a thousand public images that have already adopted the label-schema convention. Is there usage out there for org.opencontainers labels?
Whatever, we should just settle on ONE set of names so that we can achieve the original objective of saving users from defining multiple labels to convey the same information about their image to multiple vendors or tool.

Also, I'll add that I absolutely don't have a preference about whether the convention gets documented within the OCI doc itself, or it references the label-schema doc. It makes very little difference to end users!

@lizrice thanks for your input and openness! Agreed on all fronts - one is
better than two :-)
Am 12.04.2017 18:55 schrieb "Liz Rice"<EMAIL_ADDRESS>
Also, I'll add that I absolutely don't have a preference about whether the
convention gets documented within the OCI doc itself, or it references the
label-schema doc. It makes very little difference to end users!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opencontainers/image-spec/pull/636#issuecomment-293640984,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACewNxIvKQwGf0beOrpsywXOKfPiD_NAks5rvQFygaJpZM4MzQF1
.

@vbatts @jonboulle @lizrice If we have the end users in mind, adopting this PR is the best approach.
Indeed, there are roughly 1000 images that include label-schema labels, but this is out of 400,000 public images, which is about 0.25%. The actual usage of these images and the number of users impacted by a split will be low. While the number of OCI images in existence is currently very low, it will have almost immediate adoption across several container platforms, easily eclipsing the current adoption of label-schema.
I understand the idea that creating a "duplicate" namespace for these labels seems redundant, but we've already duplicated other parts of images in the pursuit of OCI. For example, OCI created an entirely new set of mediatypes for objects already specified and implemented in Docker. Creating a "conversion schema" is the approach taken thus far in OCI and I don't see why we would depart from this model. From a technical perspective, this provides much better control over the specified components, without breaking compatibility and providing an well-defined upgrade path.
I'd also like to point out that the licensing, ownership, governance and copyright of label-schema.org isn't quite clear. For something that we envision to have large adoption, this ambiguous situation isn't ideal. Given that OCI already has the legal basis covered, we can side step the possible issues that may come up. I'm sure this could be resolved, but I think hesitation here against an OCI release will have a negative effect adoption.
If our goal is industry uniformity, adopting this PR into OCI is best chance. We will have clear adoption, clear governance, copyright, licensing and ownership. While doing so will mean that label-schema may go away, the very existence will have ensured that they landed here, achieving the original goals of the project. That sounds like a success to me!

Taking advantage of the work OCI have done to sort out legals etc SGTM. For all the folks who already use or contributed to the existing label-schema work, it would be nice to see that acknowledged and agree a way to communicate the advantages of moving over to an OCI-blessed standard (rather than just have it silently "go away"). Conversation about this f2f in Austin next week? @amouat and I will both be in town.

@lizrice definitely supportive of referencing the effort in the spec and/or website for proper acknowledgement, @stevvooe what do you think?

@lizrice @caniszczyk Should we also reference Project Atomic Generic Labels, as well? Seems like we should have a whole section that includes back references to source material for OCI, including the original specification links for the image config and manifest types. This could also serve to provide solid historical context and help with implementors trying to support several formats.
As far as label-schema's messaging is concerned, that is purely up to the maintainers of that project. If they want to endorse OCI's labels and move on to something else, that would be great for achieving the goal of centralization.
We can figure out the details in the f2f.

@stevvooe @lizrice SGTM, added it to the agenda and I think the suggestion is fair, do the @opencontainers/image-spec-maintainers have any other comments?
https://docs.google.com/document/d/1z5Cv7CdFeSmYJDH8xznPc1SWhjigtxG_mTfl6aqYabk/edit

@vbatts Let's get this merged, per the agreement made in the OCI F2F meeting.

LGTM

LGTM

From: v1.0.0.batts<EMAIL_ADDRESS>Sent: Wednesday, April 26, 2017 8:43:42 AM
To: opencontainers/image-spec
Cc: Brendan Burns; Team mention
Subject: Re: [opencontainers/image-spec] annotations.md: added useful annotations (#636)
LGTM
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fopencontainers%2Fimage-spec%2Fpull%2F636%23issuecomment-297453019&data=02|01|bburns%40microsoft.com|41169792f1384b29646f08d48cbb06c9|72f988bf86f141af91ab2d7cd011db47|1|0|636288182283530482&sdata=k9btHTlUfh%2Fq8C8zVkbyYM3i4SMaR2WOBsIK2FphmGM%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAFfDgsC0s7GXuYYuo0RGm3u3zrtERuOWks5rz2YugaJpZM4MzQF1&data=02|01|bburns%40microsoft.com|41169792f1384b29646f08d48cbb06c9|72f988bf86f141af91ab2d7cd011db47|1|0|636288182283530482&sdata=q%2BhAgalUZDKjdD2wkp9syFCgcmTYAYc4Vvg4uSzQnI8%3D&reserved=0.


Docs: fixing typos in admin/supervisor

Fixing typos in docs for Administrate/Using supervisor with Docker
Signed-off-by: Tomasz Kopczynski<EMAIL_ADDRESS>

thanks! LGTM


Invalid args in completion

Recently I upgraded Emacs packages in my env. After that, while I am using bingo and lsp-mode, I got this invalid completion for functions arguments. It doesn't occur for all go project, just for big ones. I attached recordings below.

I just noticed it was duplicated with https://github.com/saibing/bingo/issues/168. Close it.


papercut: drop down list for service_endpoints

Description
This should be doable by explicitly inputing options in the ibm_catalog.json
see example: https://github.com/terraform-ibm-modules/terraform-ibm-watsonx-saas-da/blob/b2a2bea04930ad8bd3f7ee2010f29d194ce005d7/ibm_catalog.json#L206

PR Merged