log file stops at [P4] Writing C2 table . madmax

i have 8 simillar servers with identical setups and software.
somewhy on 1 of them plotman logs stops at [P4] Writing C2 table . looks like plots are still finish good, so issue just in logs:
Phase 3 took 1584.15 sec, wrote<PHONE_NUMBER>4 entries to final plot
[P4] Starting to write C1 and C3 tables
[P4] Finished writing C1 and C3 tables
[P4] Writing C2 table

expected behavior - 3 more lines like this :
[P4] Finished writing C2 table
Phase 4 took 401.263 sec, final plot size is<PHONE_NUMBER>83 bytes
Total plot creation time was.....

It looks like you deleted the entire template guiding how to file a report.
The short answer is, as soon as the plotting process has been launched it is out of plotman's hands.  Debug the plotter I guess.  You can check ps aux | grep chia_plot while a plot is running to see the command line and then run that yourself to test it without plotman being involved at all.

It looks like you deleted the entire template guiding how to file a report.
The short answer is, as soon as the plotting process has been launched it is out of plotman's hands. Debug the plotter I guess. You can check ps aux | grep chia_plot while a plot is running to see the command line and then run that yourself to test it without plotman being involved at all.

ok , thanks, than its madmax issue.


Catalina 10.15.7 cannot perform check on ichs

As described, Catalina 10.15.7 doesn't seem to be able to perform a check on iCanHazShortcut 1.3.0 and won't let me run it at all.
Installed it using:
 /u/l/d/presentations   master ±  brew cask install icanhazshortcut
Updating Homebrew...
==> Auto-updated Homebrew!
Updated 2 taps (homebrew/core and homebrew/cask).
==> Updated Formulae
Updated 2 formulae.
==> Updated Casks
disk-drill                                            staruml                                               vellum
==> Deleted Casks
daedalus

==> Downloading https://github.com/deseven/icanhazshortcut/releases/download/1.3.0/ichs.dmg
Already downloaded: /Users/erasmus/Library/Caches/Homebrew/downloads/6068311fd91b42dfe6219683f6613280c65062b5488b8ed9aae335b36b8dbaa6--ichs.dmg
==> Verifying SHA-256 checksum for Cask 'icanhazshortcut'.
==> Installing Cask icanhazshortcut
==> Moving App 'iCanHazShortcut.app' to '/Applications/iCanHazShortcut.app'.
🍺  icanhazshortcut was successfully installed!

macOS responds with :
“iCanHazShortcut” can’t be opened because Apple cannot check it for malicious software.

This software needs to be updated. Contact the developer for more information.

Homebrew Cask downloaded this file today at 13:56 from github.com

(The same happens when I install not using brew)

I don't currently have an active Apple developer account, so i can't do notarization, only signing.
To run apps that have no signing or notarization you can open Finder, go to /Applications, right click (or ctrl-click) on the app you want to run and select "Open". You will get a one-time confirmation and the app will run normally after that.

That is unless someone wants to buy me Apple developer subscription, but i doubt this will happen :)


Create/Save without mass-assignment

Hi :smile:
I keep getting these errors when trying to create a model instance without using mass assignment
Creating default object from empty value
As I understand the associated translation row is not created in such a case, so how would you create it in a beautiful way ?
Have you planned something like getTranslationOrNew() for the future ?
Thanks a lot,
Martin

I'm not totally sure what you're asking, could you please explain a bit more?

Hi, sorry for my flimsy explanation, let met try again :stuck_out_tongue:
So I'm using your package to translate my models, everything worked fine until I had to translate a model of mine that I can't use mass-assignment on, which is basically
Model::create();

So what I do to create/save the model is :
public function createDummyModel()
{
    $dummy = new Dummy();

    $dummy->attribute = "This is an attribute of my dummy model.";

    $dummy->save();
}

But I'm unable to save the instance because I keep getting Creating default object from empty value errors in this case. So I guess the matching translation is not created at this time when not using mass-assignment which results in the error.
So I was looking for a way to check if the related translation exists, and create it if it doesn't.
It is better ? (By the way, apologies for my english, French is my mothertongue :wink: )

Dows your Dummy model contain any other attributes except the translated fields?

Yes it does, but only those I need to translate, the others aren't in the translatedAttributes array.

Have you added all fields (including translated attributes) to the $fillable array?

Yes (But I'm still going to check again),
But isn't the $fillable variable only used when using the create() method ?

That is correct. What is the purpose of creating an empty object anyway? Note that the create() method saves the instance in the background.

Oh sorry, my previous code was just an example, the model that I'm having problems with is in fact much more complex (It represents a "Board Game" in a shop), in the end I manually set a good dozen of variables (I'm not using mass-assignment on this one because I have to make a lot of modification to the request payload + it made the code look messy as hell)

Okay, could you maybe share the code you have issues with then I could take a look?

Yes of course !
Here's my model :
<?php namespace App;

use Illuminate\Database\Eloquent\Model;
use Vinkla\Translator\Translatable;
use Vinkla\Translator\Contracts\Translatable as TranslatableContract;

class Game extends Model implements TranslatableContract {

    use Translatable;

    /**
     * The database table used by the model.
     *
     * @var string
     */
    protected $table = 'games';

    /**
     * @var string
     */
    protected $translator = 'App\Models\Translations\GameTranslation';

    /**
     * @var array
     */
    protected $translatedAttributes = ['name', 'short_description', 'description', 'material', 'video', 'media', 'about'];

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = ['name', 'slug', 'short_description', 'description', 'avg_players', 'avg_time', 'min_age','author', 'artist', 'material', 'video', 'media', 'about', 'active'];

    /**
     * The attributes excluded from the model's JSON form.
     *
     * @var array
     */
    protected $hidden = [];

    public function gallery()
    {
        return $this->morphMany('App\Gallery', 'imageable');
    }

    public function category()
    {
        return $this->belongsTo('App\GameCategory', 'category_id');
    }

    public function publisher()
    {
        return $this->belongsTo('App\Publisher', 'publisher_id');
    }
}

And the function that I use to create a record of my model (I removed harmless code so it's more legible for you):
public function postNew()
    {
        $game = new Game;
        $input = Request::all();

        $publisher = Publisher::find($input['publisher']);
        if ($publisher) {
            $game->publisher()->associate($publisher);
        }

        $category = GameCategory::find($input['category']);
        if ($category) {
            $game->category()->associate($category);
        }

        $game->name = $input['name'];
        $game->slug = $input['slug'];
        $game->author = $input['author'];
        $game->artist = $input['artist'];
        $game->min_age = $input['min_age'];
        $game->avg_players = $input['avg_players'];
        $game->avg_time = $input['avg_time'];
        $game->material = $input['material'];
        $game->video = $input['video'];
        $game->media = $input['media'];
        $game->about = $input['about'];
        $game->short_description = $input['short_description'];
        $game->description = $input['description'];
        $game->active = Request::has('active') ? true : false;

        $game->save();
        return redirect('/adm/games');
    }

This is the code generating the error?

Yep, sorry for not mentioning.

No problem. Do you know which line that fails?

When you added the translator package, did you move all translated attributes to the translations table?

I'm using Whoops and it doesn't give me much informations, apparently that would be this line :
$game->name = $input['name'];

But that doesn't make sense.
Actually, I created them in the translation table, but the fields are still in the base table (I was keeping them as some sort of "backup", so that I could copy it to te translation table afterwards), could it be causing the issue ?

Yeah, could be that. Try removing them.

Just removed them, still having the error. :/

To bad :/ Here: https://github.com/vinkla/translator/blob/master/src/Translatable.php#L112-L113

Ok, But this piece of code doesn't seem to get called. as I receive null in my case ?

I was playing a little bit with php artisan tinker, apparently the problem occurs when calling this method from Translatable.php for the first time :
    public function setAttribute($key, $value)
    {
        if (in_array($key, $this->translatedAttributes)) {
            return $this->getTranslation()->$key = $value; // Error is thrown here
        }

        return parent::setAttribute($key, $value);
    }

Will do a few tests and come back to you !

Cool, let me know how it goes!

Just had enough of trying to make it work on my complex model so I made a test one with only one attribute, no relations or stuff like that, and :
$test = new Test;
$test->content = "This is a test";
$test->save();

Gives me the same error, can you confirm that this works on your end ?

@dtnmr show your migrations code.
i am using this package and it works flawlessly. even on more complex models where i have polymorphic relations and nested sets.

Here are the migrations for the code above :
class CreateTestTable extends Migration
{
    public function up()
    {
        Schema::create('tests', function (Blueprint $table) {
            $table->increments('id');
            $table->timestamps();
        });
    }

    public function down()
    {
        Schema::drop('tests');
    }
}

class CreateTestTranslationTable extends Migration
{

    public function up()
    {
        Schema::create('tests_translations', function (Blueprint $table) {
            $table->increments('id');
            $table->string('content');

            $table->integer('test_id')->unsigned()->index();
            $table->foreign('test_id')->references('id')->on('tests')->onDelete('cascade');

            $table->integer('locale_id')->unsigned()->index();
            $table->foreign('locale_id')->references('id')->on('locales')->onDelete('cascade');

            $table->timestamps();
        });
    }

    public function down()
    {
        Schema::drop('tests_translations');
    }
}

Same here, but it works after I changed line 188 to getTranslation(false), and I'm 100% sure that it worked earlier without this change
return $this->getTranslation(false)->$key = $value;

Your setup looks fine. We should really get some tests in here. Will try to add some in the next major version. We're open to PRs if any of you find a solution.

And as @dtnmr said it works with Model::create but not when we try to set attributes to object separately and then save it

Are you using HHVM ? I am using it as FastCGI wrapper, and I have to admit that it has caused me issues (It does not process PHP the same way as other runtimes, so sometimes I had to tweak my code a bit) eversince I started using it.
Maybe this could be related ?

@dtnmr if it works with Model::create([]) then try to define a mutator in your model
public function setContentAttribute($value)
    {
        $this->attributes['content'] = $value;
    }

i didn't test it ... but just throwing a guess.

@elieandraos @dtnmr @michakpl How did it work out on this in the end? Did any of you solve this issue?


scripts: compliance: add a compliance check to keep blocks of code sorted

scripts: compliance: add a compliance check to keep blocks of code sorted
This allows defining a list within two marker (zephyr-keep-sorted-start and zephyr-keep-sorted-stop), and have the CI validate that the block is kept sorted every time stuff gets added to it.
This is mainly for Kconfig and CMake include lists so that there's no ambiguity on where to add new stuff.

nice :)

Can you also update the Kconfig Tips document to reference these new markers?  I'm not sure if there is a good spot to mention this for the CMake files.

   * What if you have a list where each element may span across multiple lines due to line length limit or other syntactical and style-related requirements?

Ok good point, changed so it ignores blank lines and comments within the block, and so it detects multi-line entries, just checking for lines starting with spaces or tabs, nothing super fancy.
This way you add comments and it won't complain about it, it also allowed to do the subsys and driver lists in a single block.

2. There cannot exist any variation among the list elements.

   * What if I want to mix `add_subdirectory_ifdef` and `add_subdirectory_ifndef` in the same list?

That I have problems with, I can fairly easily add a set of string substitutions while reading the line to discard the known suffixes (_ifdef, _ifndef), but then I feel like the resulting order would become a bit of a nightmare to understand and you could not really just grab the block and sort it from the editor (which is the common case I had in mind). I would rather live with the compromise that unconditionals, _ifdef and_ifndef are grouped together and accept it as a limitation of the tool, at least for now.
The alternative would be to make the error propose a "correct list" that one could copy paste but it sounds like a very cumbersome thing to use (talking from first hand experience, we have some internal tooling that sorts python include list, it's not just lexicographical and I find it very frustrating).

   * What if I want to add a comment above `add_subdirectory_ifdef`?

Handled.

Rebased.

Rebased :-)


installation docs not quite correct?

When following the part of the installation docs, "Now it is time to set up Home Assistant:" it didn't mach the actual way I had to do it.
To start with I had two Autodiscovered integrations, the first came right after onboardning, the second after installing the addon.
Both of them wanted to install the integration without the option to uncheck the box.
When I instead chose "other z-wave", I got a dialog where I could add the server.
Now the option is gone, so I can't write down exactly what I did without starting all over I guess...

Tested the steps and concluded the existing documentation matches the latest version of Home Assistant (2022.10 development).
../Frenck


Suggestion: breakpoints for debugging directly in browsers

I always debug my js code in DevTools, directly within the browser. In this context, VSCode breakpoints doesn't make any difference whatsoever.
I wish it would add a "debugger;" statement immediately before the line I've checked the breakpoint, and that line could be hidden. Likewise, any debugger statements in some js code could be hidden and signalized with a breakpoint interchangeably.
I know its not so trivial as it may seem (e.g.: multi-statements lines, line numbers, etc), and maybe there's even a better solution in VSCode today that I may not know of.

I don't understand why a "debugger;" statement should be added automatically. Isn't a debugger statement just a workaround if breakpoints don't work?

@weinand as I said, I may not know a better existing approach for this. The extension mentioned is indeed a good way to go. I'll try using that.
I opened the issue even with my lack of knowledge because I thought this could be a general way of doing this, without any extension and in whatever browser the user wants, not strictly chrome. But to do that concurrently with the extension is trickier and maybe not worth it (20-80 case).

Glad you got the Chrome debugger working!


RangeError: Maximum call stack size exceeded

I tried to get all the rows from one of my tables and got this error.
I used several ways to solve this. (await Currency, await Currency({unit_currency_fl: 'eth'}) and so on)
If i directly used the query that is maded by openrecord, it works. I don't know why this happens.
at Function.keys ()
at Object.exports.clone (C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:87:12)
at C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:91:29
at Array.forEach ()
at Object.exports.clone (C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:87:22)
at C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:91:29
at Array.forEach ()
at Object.exports.clone (C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:87:22)
at C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:91:29
at Array.forEach ()
at Object.exports.clone (C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:87:22)
at C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:91:29
at Array.forEach ()
at Object.exports.clone (C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:87:22)
at C:\Users\lcs\projects\api.v2.ms1.coint.me_alpha\node_modules\openrecord\lib\utils.js:91:29
at Array.forEach ()

Hi @Feelthewind
it seems that you have some kind of circular dependency in your model definition.
Please post your model definition of Currency.
await Currency({unit_currency_fl: 'eth'})should not work! I guess it's a condition, so the following code will do it: await Currency.where({unit_currency_fl: 'eth'})


Add default templates

0.4.0 release (not bumping to 1.x as its not on aldryn marketplace)

👍

Coverage remained the same at 83.772% when pulling 3fdeacad48ce00a9c508b7cb72fd2b700fabdf63 on feature/release into 24e42d9f46aeb18130d25147834474c38dd0fb50 on master.


Add option to do strict validation check

I ran into a 'bug' where the system said the VAT number is valid, when it was actually not.
checkVAT('BE0740-509975', countries) return true, because the library automatically clears the special characters (https://github.com/se-panfilov/jsvat/blob/master/src/lib/jsvat.ts#L107)
Would it be possible to add an attribute that specifies if the client actually wants to remove those characters?
I don't mind submitting a PR, if you don't have the time to implement this.
Thanks for your library!

@se-panfilov
having the exact same issue! ☝
checkVAT('DE124725-365', [germany]) strips the dash from the string and sees DE124725365 as a valid VAT ID.
The checkVAT function should not strip special characters from the string by default.


[For discussion] WorkManager refresh job

Schedule a daily refresh job using WorkManager, including images.
Also use it for fetching data.

TODO

[ ] check error conditions (network)
[ ] confirm data fetching hits caches for images


[AutoComplete] Popover top value inconsistently calculated

This <AutoComplete> instance is inside a <ToolbarGroup>. The top property seems to either be rendered at 70 or 0, depending on its mood I guess.
This is with the latest 0.14.0 release.

It seems to be an issue with how the values from getTargetPosition are calculated and/or parsed in applyAutoPositionIfNeeded in the <Popover/> component.
In the instances where the offset is correct (e.i, not overlapping), getTargetPosition is returning an object that looks like:
{
bottom: 0
center: 0
left: 0
middle: 0
right: 0
top: 0
}

and 'setPlacementcalculatestargetPositionas{top: 70, left: 1224}. These values are passed to applyAutoPositionIfNeeded` which uses the following logic to check if the top value should be updated:
 if (targetPosition.top < 0 || targetPosition.top + target.bottom > window.innerHeight) {
     // Update logic and stuff
    }

Since targetPosition is 70 and target.bottom is 0 it doesn't recalculate the top value and it renders fine.
But, in the instances where it overlaps, getTargetPosition is returning:
{
bottom: 879
center: 439.5
left: 0
middle: 128
right: 256
top: 0
}

Where bottom is actually the value of window.innerHeight (if you resize the window, the bottom value change accordingly on the next render)
So since the window.innerHeight + 70 is always greater than window.innerHeight its recalculating the top value, which is returning a negative value, and the Math.max(0, newTop) call is returning 0 instead of the 70.

This issue is, of course, resolved if <AutoComplete> sets the canAutoPosition prop on the <Popover> to false

This is a very odd and moody bug. Thanks for the detailed report :+1: :+1: We'll look into it. although a PR is also welcome :grin: :grin:

Do you think setting canAutoPosition to false would be acceptable for the <AutoComplete> component, or would that have unexpected consequences? If that would work, I'd be happy to get a PR going

I'm not sure. I wan't around when this component was being written. @oliviertassinari Can you take the lead on this one?

@chrismcv Should be the best in place to answers this :grin:

Hi, I want to point out a related problem. When the keyboard is open on mobile (i.e. screen height is limited), the Popover will also cover the input.
Setting canAutoPosition to false will fix this, but will also cause part of the Popover to go off the screen, since the maxHeight of the Popover content is still determined by window.innerHeight. We might need another way to constrain the height of the Popover content.

@oliviertassinari @chrismcv @alitaheri any word on this?

@oliviertassinari @chrismcv @alitaheri sorry to keep pestering you on this, but we'd like to get this taken care of before pushing our project to production. Can I can an update here? Does my proposed fix sound reasonable?

@Aweary I'd say make that PR for now so we can evaluate the effects. it it's a breaking change we'll ease into it. otherwise we'll merge. Either way, you can always use your own fork for the production. at least for now.

Thanks @alitaheri I'll get a PR going. Forking would have been a last ditch option, but we'd rather support the project then fork it. Thanks for the response.

Your approach is appreciated :+1: Thanks for the PR :grin:

fixed by https://github.com/callemall/material-ui/pull/3620


Add support (flutter extension) for zed code editor

Use case
.
Proposal
.

Closing from here. See https://github.com/zed-industries/extensions/issues/247


Are there desktop images for ROS2 Jetson?

I see there are images for the ros_base and such but none for the Desktop versions of ROS in Noetic, Foxy and Galactic. Are there plans to release prebuilt containers for these distros?

Hi @sisaha9, IIRC there was some trouble building the full desktop packages so I don't have the pre-built images for them.

Hi @dusty-nv , now that Jetpack 5.0 is out is it possible to create Desktop images for the above distros? I haven't seen a Jetpack 5.0 l4t-base image so I am unable to attempt it

In theory it should be possible, although I haven't migrated my ros2 dockerfiles over to installing OSRF's focal packages and am still building from source.   l4t-base is up here:  docker pull nvcr.io/nvidia/l4t-base:r34.1

@sisaha9 - start with the container that has everything you want apart from ROS and then install that ROS distro requirements and then do a full desktop install.  Will probably have to be build from scratch unless your container is Ubuntu 20.04 (e.g. newer JetPack on Orin or Xavier)


Fix typo in readme.

I guess it’s a typo.

Sure is. Thanks.

Your welcome :octocat:
Thank you for that plugin :ok_hand:


Proposal to inject built-in HttpAuthenticationMechanism beans

A proposal to address https://github.com/jakartaee/security/issues/188.
This is to make it possible to choose programmatically which of the built-in authentication mechanisms should be used based on configuration or user input. It would be possible to inject one or more mechanisms configured using the existing definition annotations (e.g. @BasicAuthenticationMechanismDefinition) and delegate authentication to them from a custom HttpAuthenticationMechanism bean.
See the README for more details.

An alternative option would be to support qualifiers for built in auth mechanisms, like this:
@BasicAuthenticationMechanismDefinition( qualifier = BasicAuth.class /* user-specified CDI qualifier*/ )
@FormAuthenticationMechanismDefinition(
   qualifier  = FormAuth.class /* user-specified CDI qualifier*/,
   loginToContinue = @LoginToContinue(loginPage = "/login.xhtml"))
@Dependent
public class MyAuthMechanism implements HttpAuthenticationMechanism {
    @Inject
    @FormAuth
    HttpAuthenticationMechanism formAuthentication;

    @Inject
    @BasicAuth
    HttpAuthenticationMechanism basicAuthentication;
}

More boiler-plate code but does the same job.


Enter UAC Code

Motivation and Context

New social acceptance test for entering a UAC as a respondent into Respondent Home.
What has changed

Added feature file, steps, and RH page.
Tweaked some return statements.
Added UAC to context.
Added EQ_URL as a config variable.
How to test?

make TEST_TARGET=acceptance_tests/features/social_enter_uac.feature social_acceptance_tests
Links

https://trello.com/c/eotv2PRA

 EXT: Enter UAC Code

@benjefferies all this is being tested within RH already. This was more to increase confidence and visibility of RH -> eQ. Is the idea to deprecate this entire repo in favour of services running their own acceptance tests?

@liamtoozer Do capabilities want to merge this?

Closing for now as we have more tests that do the same thing in the RH repo.


Faster IndexOfVectorized

from https://github.com/dotnet/corefx/pull/16222
Not quite sure how to interpret the benchmark results (from corefxlab\scripts\PerfHarness) but they suggest is x4.5 - x6 faster than .IndexOf for everything except length = 0
Likely want to verify it by someone who understands how the benchmarks work; might all be the same length and 0 is higher as its jitting

If you run the harness till the end (as opposed to stopping it after some tests pass), the harness will analyze the raw file and create another file with relatively nicely tallied results.

I ran the benchmark with your changes.
Before: 0.113638251
After: 0.075090739
Nice improvement!

I added some tests for different indexes. Here are the results:
With your changes:
Test Name	Average
IndexOfBench.SpanIndexOf(at: 1000)	0.367757911
IndexOfBench.SpanIndexOf(at: 30)	0.017418343
IndexOfBench.SpanIndexOf(at: 0)	0.003753878
IndexOfBench.VectorizedIndexOf(at: 1000)	0.084792347
IndexOfBench.VectorizedIndexOf(at: 30)	0.017610471
IndexOfBench.VectorizedIndexOf(at: 0)	0.016239759
Without your changes:
Test Name	Average
IndexOfBench.VectorizedIndexOf(at: 1000)	0.174605217
IndexOfBench.VectorizedIndexOf(at: 30)	0.092288951
IndexOfBench.VectorizedIndexOf(at: 0)	0.084601138
Nice improvement. I will merge the changes.
Thanks!!

What's it like on the early numbers? (non-zero) compared to regular IndexOf?

Here are some "early numbers":
Test Name	Average
IndexOfBench.SpanIndexOf(at: 1000)	0.369605184
IndexOfBench.SpanIndexOf(at: 30)	0.018005964
IndexOfBench.SpanIndexOf(at: 2)	0.004584118
IndexOfBench.SpanIndexOf(at: 4)	0.005525728
IndexOfBench.SpanIndexOf(at: 8)	0.007806724
IndexOfBench.SpanIndexOf(at: 1)	0.004488092
IndexOfBench.SpanIndexOf(at: 0)	0.003797033
IndexOfBench.VectorizedIndexOf(at: 1000)	0.082512277
IndexOfBench.VectorizedIndexOf(at: 2)	0.017165913
IndexOfBench.VectorizedIndexOf(at: 4)	0.017223139
IndexOfBench.VectorizedIndexOf(at: 30)	0.018337207
IndexOfBench.VectorizedIndexOf(at: 1)	0.01692914
IndexOfBench.VectorizedIndexOf(at: 8)	0.01734747
IndexOfBench.VectorizedIndexOf(at: 0)	0.01703912

I am going to merge this and then we can cleanup depending what goes or not into Span directly.


MINOR: add upgrade note for KIP-173 topic configs

@guozhangwang, @bbejeck, @mjsax

LGTM. Merged to trunk.


Fix sample link

Fixes #16937

@bot-gradle test and merge

Your PR is queued because #16519 is being built.

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


Fix path in dev-mode

Supersede #300 and fix https://github.com/quarkiverse/quarkus-quinoa/issues/299

@melloware I will be off for a bit more than 10 days, could you cherry-pick this commit on the 1.x branch please.


No Phone Verification

What is your username and discriminator?
Linley#8686
Why did you join Discord Linux?
I'm a hobby developer and want to learn more about creating stuff using Discord and Linux.
Where did you hear about Discord Linux?
From another programming discord server
What is the passphrase that was assigned to you by the bot?
sudo -i tourist-enthusiasm-buffer-squeal

From simonizor#0420 :
Looks good.  I've given you a temporary role so that you can enter the passphrase in the welcome channel.


[197] Issue #197 - November 04, 2021

To contribute to this issue, simply leave a comment here. Please also review our contributing guidelines.
The current draft for this issue in _drafts/. If you want to contribute directly, feel free to open a pull request.

https://forums.swift.org/t/pitch-package-manager-command-plugins/53172

https://forums.swift.org/t/pitch-pointer-family-initialization-improvements/53168

https://twitter.com/niw/status/1455258442541711365?s=21

https://forums.swift.org/t/asyncstream-constructor-which-also-returns-its-continuation/53251

https://forums.swift.org/t/se-0328-structural-opaque-result-types/53248

https://forums.swift.org/t/pitch-guarded-failpoints-adding-enhanced-logic-to-guard-statements/53222

https://forums.swift.org/t/mqttnio/53238

https://forums.swift.org/t/extending-swift-docc-to-support-objective-c-documentation/53243

https://forums.swift.org/t/swiftnio-swift-version-support/53232

https://twitter.com/AirspeedSwift/status/1455735695855677447


Need help on the implementation of the 2D-RNN

The current idea is, map the original (Nsample, timestep ,input_dim) to a grid  (Nsample, grid_shape[0], grid_shape[1] ,input_dim).
Then apply nested scan to scan column by column.
At each step, updates the current state and activation based on the row and column predecessors.
So the initial_states is [row_states] + [col_states].

class MultiDimRNN(Recurrent):
    '''grid_shape: define the 2D shape of the grids. Eg. there are in total 5*6 grids'''

    def __init__(self, output_dim, grid_shape ,
                 init='glorot_uniform', inner_init='orthogonal',
                 activation='sigmoid', inner_activation='hard_sigmoid',
                 **kwargs):

        self.output_dim = output_dim
        self.grid_shape = tuple(grid_shape)

        self.init = initializations.get(init)
        self.inner_init = initializations.get(inner_init)
        self.activation = activations.get(activation)
        self.inner_activation = activations.get(inner_activation)
        super(MultiDimRNN, self).__init__(**kwargs)

    def build(self):
        input_shape = self.input_shape
        if self.stateful:
            self.reset_states()
        else:
            # initial states: all-zero tensor of shape (output_dim)
            self.states = [None for _ in range(np.sum(self.grid_shape))]
            #for state_id in range(np.sum(self.grid_shape)):
            #    self.states.append(K.zeros((input_shape[0], self.output_dim)) )

        input_dim = input_shape[2] #Nsample*timestep*input_dim
        self.input_dim = input_dim

        self.W = self.init((input_dim, self.output_dim))
        self.U_row = self.inner_init((self.output_dim, self.output_dim))
        self.U_col = self.inner_init((self.output_dim, self.output_dim))
        self.b = K.zeros((self.output_dim,))
        self.params = [self.W, self.U_row, self.U_col, self.b]

        if self.initial_weights is not None:
            self.set_weights(self.initial_weights)
            del self.initial_weights

    def get_initial_states(self, X):
        # build an all-zero tensor of shape (samples, output_dim)
        initial_state = K.zeros_like(X)  # (samples, timesteps, input_dim)
        initial_state = K.sum(initial_state, axis=1)  # (samples, input_dim)
        reducer = K.zeros((self.input_dim, self.output_dim))
        initial_state = K.dot(initial_state, reducer)  # (samples, output_dim)
        initial_states = [initial_state for _ in range(len(self.states))]
        return initial_states

    def reset_states(self):
        assert self.stateful, 'Layer must be stateful.'
        input_shape = self.input_shape
        if not input_shape[0]:
            raise Exception('If a RNN is stateful, a complete ' +
                            'input_shape must be provided ' +
                            '(including batch size).')
        if hasattr(self, 'states'):
            for state_id in range(np.sum(self.grid_shape)):
                K.set_value(self.states[state_id],
                            np.zeros((input_shape[0], self.output_dim)))
        else:
            self.states = []
            for state_id in range(np.sum(self.grid_shape)):
                self.states.append(K.zeros((input_shape[0], self.output_dim)))

    def step(self, x, row_state, col_state):
        # states only contains the previous output.
        #assert len(states) == 1
        #prev_output = states[0]
        h = K.dot(x, self.W) + self.b
        output = self.activation(h + (K.dot(row_state, self.U_row) + K.dot(col_state, self.U_col))/2 )
        return output, [output]

    def get_output(self, train=False):
        # input shape: (nb_samples, time (padded with zeros), input_dim)
        X = self.get_input(train)
        assert K.ndim(X) == 3
        # get input, then map it to the grid_shape
        inner_data = K.reshape(X, (-1,) + self.grid_shape + (self.input_shape[2],))  # for example 16*16 *900 (16 * 16 grids and each contains 900 pixels)

        if K._BACKEND == 'tensorflow':
            if not self.input_shape[1]:
                raise Exception('When using TensorFlow, you should define ' +
                                'explicitly the number of timesteps of ' +
                                'your sequences. Make sure the first layer ' +
                                'has a "batch_input_shape" argument ' +
                                'including the samples axis.')

        mask = self.get_output_mask(train)

        if mask:
            # apply mask
            X *= K.cast(K.expand_dims(mask), X.dtype)
            masking = True
        else:
            masking = False

        #grid_mask = K.reshape(mask, (-1,) +  self.grid_shape)

        if self.stateful:
            initial_states = self.states
        else:
            initial_states = self.get_initial_states(X) #build an all-zero tensor of shape (samples, output_dim)

        last_output, outputs, states = self.rnn_grid(self.step, inner_data, self.grid_shape,initial_states,
                                             go_backwards=self.go_backwards,
                                             masking=masking)

        if self.stateful:
            self.updates = []
            for i in range(len(states)):
                self.updates.append((self.states[i], states[i]))

        if self.return_sequences:
            return K.reshape(outputs, (-1,) + (np.prod(self.grid_shape), self.input_shape[2]))
        else:
            return last_output

    def rnn_grid(self ,step_function, inputs, grid_shape,initial_states,
        go_backwards=False, masking=True):
        row_states = initial_states[0:grid_shape[0]]
        col_states = initial_states[grid_shape[0]:]

        inputs = inputs.dimshuffle((2,1, 0, 3))  # col, row, nsample, dim
        #inputs = input.dimshuffle(tuple(range(1,len(grid_shape)+1))+(0, len(grid_shape)+1) )

        def _step(input,row, row_state, col_state): #[row_states] + [col_state]
            #'''it needs the left, and upper grid as it's predecessor,
			#   it only update the col_state after process one local patch.
			#   although it needs two states to compute each step, but it actually only updates the upper column state, for the
			#   left row_state, it only use it.
			#'''
            states = [col_state]
            output, new_states = step_function(input, row_state, states[0])  # this function actually should only return one state

            if masking:
                # if all-zero input timestep, return
                # all-zero output and unchanged states
                switch = T.any(input, axis=-1, keepdims=True)
                output = T.switch(switch, output, 0. * output)
                return_states = []
                for state, new_state in zip(states, new_states):
                    return_states.append(T.switch(switch, new_state, state))
                return [output] + return_states
            else:
                return [output] + new_states

        def loop_over_col(coldata, col_ind,col_state, rows_states, rows_model):
            #'''when loops over a column, it needs the left column as row_states,
			#row_states, col_states are already concatenated to be tensors for scan to process before call this function.
			#it return the results as well as this column response as the row_states to next column.
			#it should return the last column_state
			#'''
            results , _ = theano.scan( fn = _step,
                                       sequences = [coldata, rows_model, rows_states],
                                       outputs_info=[None] +  [col_state]
                                       )

            new_row_states = results[1:] #'''list of tensor of row_size * nsample * out_dim, but we need to modify it to be list'''
            col_vals = results[0]
		   #'''tensor of row_size * nsample * out_dim'''
		   #'''the length is the number of states for each single actual step'''
            single_state_num = len(new_row_states)
            returned_row_states = []
            for timestep_id in range(len(rows_model)):
			    #'''new_row_states is a list of tensor of size row_size * nsample * out_dim
				#   returned states should be [timestep_1_state_1,...,timestep_1_state_n, ...., timestep_m_state_1,...,timestep_m_state_n],
				#   n is the number of required states for each single actual step function, most time it is 1. but CWRNN has 2.
				#   m is the row_size, = grid_shape[0]
				#'''
			   for state_id in range(single_state_num):
				    returned_row_states.append(T.squeeze(new_row_states[state_id][timestep_id]))
            returned_col_state = []
            for state_id in range(single_state_num):
			  returned_col_state.append(T.squeeze(new_row_states[state_id][-1]))
            #'''output should be list corresponding to the outputs_info'''
            return [col_vals] + T.concatenate(returned_row_states, axis =0), T.stack(returned_col_state, axis =0)

        inputs_model = dict(input = inputs, taps=[0])
        rows_model = T.arange(self.grid_shape[0])
        cols_model = T.arange(self.grid_shape[1])

        #'''loop_over_col(coldata, col_ind, rows_states, col_states, rows_model)'''
		#'''Maybe I need to concatenate all states list to tensor at this stage, scan deal with list of tensor variable, not
		#list of list.'''
        results, _ = theano.scan( fn = loop_over_col,
                                    sequences = [inputs_model, cols_model, T.stack(col_states, axis =0)],
								#'''return grid_results, row_states, col_state'''
                                    outputs_info=[None, T.stack(row_states, axis =0), None],
                                    non_sequences = [rows_model]
            )
        #update col_states here
		#'''results are list of [grid_tensor, row_state_tensor, col_state]'''
        outputs =  results[0]  #'''tensor type'''
        row_states_tensor  = results[1][-1]  #'''we only need the last column as the row_states, tensor type'''
        col_states_tensor  = results[2]  #'''tensor type'''

        # deal with Theano API inconsistency
        #if type(results) is list:
        #    outputs = results[0]
        #    states = results[1:]
        #else:
        #    outputs = results
        #    states = []
        outputs = T.squeeze(outputs)
        last_output = outputs[-1,-1]

        #outputs  col, row, nsample, dim
        outputs = outputs.dimshuffle((2,1,0,3))

        row_states = [T.squeeze(state[-1]) for state in row_states_tensor]
        col_states = [T.squeeze(state[-1]) for state in col_states_tensor]
        return last_output, outputs, row_states + col_states

    def get_config(self):
        config = {"grid_shape": self.grid_shape,
                  "output_dim": self.output_dim,
                  "init": self.init.__name__,
                  "inner_init": self.inner_init.__name__,
                  "activation": self.activation.__name__}
        base_config = super(MultiDimRNN, self).get_config()
        return dict(list(base_config.items()) + list(config.items()))

The current idea is, map the original Nsampletimestepinput_dim to a grid,.
Then apply nested scan to scan column by column.
At each step, updates the current state and activation based on the row and column predecessors.
So the initial_states is [row+states] + [col_states].
The states stuff really confuse me,  I am not sure if I implement it right.
@wxs   @fchollet   @davidbuniat

Hello, I'm also trying to write a nD version LSTM.
Did you test the code you wrote, does that work? nesting two scan, will that be extremely slow?

@WeidiXie  I did not test it. I believe it won't work in its current shape. I posted it here looking for help from experienced theano programmar.

@shampool  Did you eventually get it work?

@WeidiXie  not yet, I am still struggling with the bugs of returned states of nested scan.

I'm currently not by the pc,  and will go to the lab later, maybe we can debug it together. I need it as well.  Could you drop me an email.

@WeidiXie。sure！<EMAIL_ADDRESS>

latest update, I can compile it. but not sure will it generate valid results.

Any news?


try

Can one of the admins verify this patch?

ok

jbuild

jbuild

jbuild

jbuild

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

jbuild


No source on pipy

Hi,
Really appreciate your tool! Could you publish the source (tar.gz) on https://pypi.org/project/rawpy/#files? It would facilitate the automatic cross-compilation for embedded systems.
Thanks!

You should be able to achieve what you want with pip install git+https://github.com/letmaik/rawpy?
I generally don't publish source packages anymore for any of my binary Python packages as those more often than not lead to failed builds with obscure error messages and more people complaining and I don't have the time to handle the support requests. I treat building from source as an advanced use case that shouldn't happen by accident (pip automatically falls back to source packages if there are no binary wheels for the target Python version and/or platform, which is unfortunate).

Hi,
No. I am using for example buildroot to build custom pre-compiled Linux systems for embedded systems. The default is to download the source from pypi (https://buildroot.org/downloads/manual/adding-packages-python.txt). Most package developers also upload the source and therefore facilitate the usage of such automated build systems.
I do understand that you do not have the time to help troubleshoot obscure errors. However, it would be nice to support advanced users in using your nice tool by simply publishing the source code also on pypi.


R11_EAC and RG11_EAC textures formats

What we don't support:
        BC6H = 19,
        BC7 = 20,
        ATI2N = 21,
        ETC2 = 23,
        ETC2_EAC = 24,
        R11_EAC = 25,
        RG11_EAC = 26,
        ATI1N = 27,

Valve uses https://github.com/google/etc2comp

Out of these, Underlords only uses ETC2 and ETC2_EAC

ETC2 and ETC2_EAC are implemented, and the remaining formats have not been seen used anywhere.

The new SteamVR beta updates have added a lightmap baker to the tools. The output of it gets saved as vtex_c files compressed with BC6H and BC7 inside the map vpk.

Can you upload an example file?

https://drive.google.com/open?id=173tAi7CmLA-C-l4647IAB9OEAtI_68TR
Here is the set of two it generates, as well as the .exr source files that the lightmapper caches (not sure which or if any of them get compiled directly).
It seems a lot like the Unity directional lightmapper, storing one texture with lighting data + one with the direction of the dominant incoming dir and the directionality ratio in the alpha.

Are you able to upload png images of the source so it's easier to compare what it's supposed to look like? Thanks.

SteamVR has ATI1N textures:
materials/default/default_metal_tga_9e174e3d.vtex_c
materials/default/default_ao_tga_559f1ac6.vtex_c
in \SteamVR\tools\steamvr_environments\game\core\pak01_dir.vpk

HL:A has ATI2N textures.
ATI1N textures sometiems crash with Index was outside the bounds of the array.

Hey @lewa-j, perhaps you would like to take a look at ATI1N/ATI2N texture formats? I added code for ATI1N some time ago, and it doesn't appear to be fully correct as it crashes on certain textures in HL:A.

Alright. But i need sample textures, especially those that crashes.

In HL'a VPK some ati1n textures that crash:
materials/models/props/plastic_barrel_1_ao_tga_2850e499.vtex_c
materials/models/props/plastic_tiling_rough_tga_e0c24fc1.vtex_c
materials/models/props/hotel/hotel_laundry_cart001_ao_tga_b83eddab.vtex_c
metro_body_ao_tga_58b6a7f3.vtex_c
ati2n textures:
materials/models/props/hotel/hotel_tarp_001_freedom_psd_993397bd.vtex_c
materials/models/props/misc_debris/debris_piles_med_normal_tga_2446c4a8.vtex_c
materials/xen/tile/antlion_infestation_blend_normal_tga_d5e4101f.vtex_c
Also, I noticed that BC7 decoder produces some corrupted images (like some pixels are not fully decoded or something), for example:
metro_body_normal_tga_e5bc8f85.vtex_c

Thanks. I'll look into it.
I tested BC7 decoder only on lightmaps.

#206 Should fix corruption on BC7 textures. It was related to alpha.

ATI1N done

ATI2N done. They share a lot of code with DXT5. Can be refactored.

Decoding NormalizeNormals just like DXT5

sky_space_hdr.vtex_c in hlvr using BC6H looks corrupted to me:

This is odd. light_sky_c17_zoo.vtex_c has the same format and all the other parameters and looks fine. Need to investigate.

A lot of BC6H textures in HLA are broken (in skybox folder), it does feel like it has something to do with the (un)compressed sizes.


Website rendering incorrectly

I forked the agency-jekyll-theme repo and I'm attempting to view the website at https://tillykrishna.github.io/, but I can't get the website to render as it does in the demo. This is what I'm currently getting:

In my _config.yml, I have:
url: https://tillykrishna.github.io
baseurl: /agency-jekyll-theme/
Is there something else I'm missing? Thanks so much!

I'm experiencing the same thing. I have set it up on a local repository on Windows and it builds and runs perfectly with no errors in the browser dev tools. I've been trying to figure out if it is not compatible with github pages but there's no error message from github other than that it failed to build.

Hi, maybe it's already late but can be the answer for the next people who may have this issue.
You need to change the baseurl on _config.yml file. The base URL should be the name of your repository.
Hope it works! I had the same issue and now it's solved


Tests do not compile

[14 of 16] Compiling IPATextToDescriptionSpec
phonetics-modeling/test/IPATextToDescriptionSpec.hs:20:35: error:
Not in scope: type constructor or class ‘SpecM’
|
20 | pulmonicEgressiveConsonantSpec :: SpecM () ()

I made some changes. Now, tests compile and run without problems.
This issue is resolved as of commit c14e1f56cf817a40cf8b55d71dddc25a8de664dd


Question: Running only selective tests

Describe the bug
This is not a bug. I want to confirm whether I can run tests selectively in TaskCat
For example, I have the following taskcat.yml
---
global:
  owner<EMAIL_ADDRESS>  qsname: sample-taskcat-project
  regions:
    - ap-southeast-1
tests:
  first-test:
    parameter_input: debug-input-1.json
    regions:
      - ap-southeast-1
    template_file: cloudfront.yml
  second-test:
    parameter_input: debug-input-2.json
    regions:
      - ap-southeast-1
    template_file: s3.yml

Is there any way I can just run the first test, for example like this:
taskcat -c ci/taskcat.yml -m first

I want to use this functionality because I have to test many individual components of AWS using CloudFormation, but I do not want to make taskcat.yml for every component. So let me know if this feature exists already or any reasoning for this not being required.
Thanks in advance

@lavk7 - at the moment, this feature doesn't exist. We're in the last stages of our 0.9.x release branch, and I anticipate this (or equivalent) would be available in 0.9.1 or perhaps a little later.

@andrew-glenn  Got it.
Writing a secondary config file is what I have been doing for now. Well, I'll wait for it then.
Thanks for the quick response.

This was added via https://github.com/aws-quickstart/taskcat/pull/525. It's available in our current development release, and will be in 0.9.18.
If you're interested in the dev release:
pip install taskcat==0.9.18.dev3


[stable/nginx-ingress] fix volumes condition

What this PR does / why we need it:
Currently when using .Values.controller.extraVolumes, spec.template.volumes: is not output in Deployment or DaemonSet manifest.
This PR fixes it.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/lgtm
Thanks for the PR!


SPIFFS_fflush while open for read.

I am using SPIFFS in Arduino ESP8266 with a single quite large file. I think I see the issue described below, and it would be nice to know if my diagnosis is correct.
The file is constantly open for writing (and reading) in "r+" mode in file descriptor 1.
The file is updated periodically using a SPIFFS_SEEK_SET and write to some place in the file.
The file now also becomes open for reading in file descriptor 2.
Reading takes some time because the receiver is slow.
While the file is open for reading in file descriptor 2, we need to update the file using file descriptor 1.
I set the file pointer using a SPIFFS_SEEK_SET, write 32 bytes and does a SPIFFS_fflush to ensure that the data are physically written to the file.
Can SPIFFS handle this situation?
I suspect that the SPIFFS_fflush, when it sees that the file is being read in another descriptor really "flushes" the cache, without executing the physical SPI write.
Is that correct?

Hmm, I have to look into this, but I'll direct you to these test cases for the time being.


Add docs on why we introduced Spring-(WebFlux/Redis/MongoDB) and removed them finally

Done. I have written about why we try to not use deps in https://turms-im.github.io/docs/intro/redevelopment.html#关于依赖库的使用.


Multilabel classification training data

Hi guys, just have a question regarding the training data for multilabel classification.
So, for multiclass classification, you can play around with the number of samples (K) per label, and of course, the higher the K, the performance increases; this is straightforward as there will be only 1 label per sample.
However, for multilabel classification, where there can be more than 1 label per sample (also in many different combinations), how are we supposed to construct our training data? for example, would it be best to give an equal number of samples for every combination of labels? this would exponentially increase the training data required which would defeat the purpose of few-shot learning...?
I am asking this question as I have tried training the model without considering the above question and getting not-so-great results (around 65% accuracy with 12 labels).
Thank you!

I have used SetFit when there is a limited number of labeled training data available. If you have ample data, you can also use other frameworks. I have used the following code to balance unbalanced training data classes.
model = SetFitModel(
                    model_body=SentenceTransformer('all-MiniLM-L6-v2'),
                    model_head=OneVsRestClassifier(LogisticRegression(class_weight="balanced")),
                    multi_target_strategy="one-vs-rest"
                    )

@MattiL I am not sure if you understood my question correctly. I am talking about multilabel problem where the input data can have multiple labels at the same time out of many labels unlike multiclass problem where the data can have exactly one label out of many.
To balance the training data for the multiclass problem, it's easy, you just balance the data or adopt the method like you have. However, for the multilabel problem, since you can have sum(nCr) combinations, you w

I have tried to use the balancing code for multilabel classification. I guess that might improve the accuracy. Multiclass has had little support in SetFit.

You could use scikit-multilearn to create a balanced training dataset. Use iterative_train_test_split

to all wondering how the data should look like. Here is a sample format
text,vocab,slang,idioms,simple "Exhausted from a relentless schedule, the formidable entrepreneur persevered through arduous obstacles to achieve unprecedented success.",1,0,0,0 "After witnessing the catastrophic impact of deforestation firsthand, the environmentalist fervently advocated for reforestation measures to mitigate irreversible ecological damage.",1,0,0,0 The arduous task of persuading the obstinate toddler to eat his vegetables became an exhausting battle of wills.,1,0,0,0 "Amidst unprecedented global turmoil, the populace strives relentlessly to find solace in the pursuit of self-actualization.",1,0,0,0 The arduous pursuit of success often leads individuals to forsake their personal relationships in favor of their professional ambitions.,1,0,0,0 "Despite encountering numerous obstacles throughout his journey, the resilient entrepreneur persistently strategized and ultimately achieved remarkable success in his business venture.",1,0,0,0


Remove application tag in library module

<application> tag in AndroidManifest.xml is not necessary for a library module.
Sometimes it causes compile error even:
Error:Execution failed for task ':app:processDebugManifest'.
> Manifest merger failed : Attribute application@label value=(@string/app_name) from AndroidManifest.xml:8:4-36
	is also present at [com.github.moove-it:fakeit:v0.2] AndroidManifest.xml:13:9-49 value=(@string/application_name).
	Suggestion: add 'tools:replace="android:label"' to <application> element at AndroidManifest.xml:5:2-20:16 to override.

@fython Thanks for the contribution, will be working on this in next release!


update file lfi-os-files.data

There is no information on where this list comes from. We need to get some sources and update the list.

Started with sources:

Fuzzing/LFI/LFI-gracefulsecurity-linux.txt


Drag target reacting to drag position

Hey, I'm trying to recreate this effect I found on Dribbble. Note that as the item is dragged around, a blob follows the pointer until release, and then particles flow from the object to the target. It seems the particles could be done via the current API callbacks since it passes the offset of the DragTarget on drag finish (although this effect would not be straightforward to implement, and I'm OK with ignoring it for now.)
The blob effect, however, seems impossible at the moment. The API for DragTarget does not allow you to react to the position of the DragAnchor as a drag occurs inside (or outside) of its bounds. Any ideas on how you might be able to achieve this type of reaction to a drag based on its current position within the target?
In short, we can react to the position of a drag when it finishes, but not while it's occurring. Is my understanding correct, and if so, are there any quick wins in terms of workarounds?

Hi @VerTiGoEtrex
This platform is not meant for assistance on personal code.
Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow.
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you


Add an option to use an EC2 instance's IAM role

Uses the AWS Go SDK's role provider to provide credentials with use of the --amazonec2-use-iam-role flag.
I wasn't quite sure about the flag name, so more than happy to change it if needed!
Fixes #3500

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "3500-use-instance-role" git@github.com:siphonnetworks/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Thanks for the PR.
Is it possible to just do so automatically?  Like, if an IAM profile for the instance exists, then use that, otherwise fall back to trying to read them from file, and otherwise fall back on the Machine flags / env vars.  Seems like we should just use the profile automatically if possible.

@nathanleclaire My concern with doing that is that it would potentially break existing installs - where docker-machine might be on an instance with a profile, but using credentials for a different role. It would then be impossible to override the instance's profile.
I could also make it work without the flag as a fallback - i.e. try taking the credentials from the command line, then file, then if none specified use the profile, but this would cause delays when run on a non EC2 box as it would try and contact the metadata server every time before throwing the error - I'm not sure if there's an easy way to check if run on an EC2 instance without trying to contact the metadata server.
Also, the same problem exists, if you have the credentials file in place for a different use but want to use the instance role then there's no way to override and say 'use the role'.
Very happy to take direction here as to what would be best.

I think this is fixed with https://github.com/docker/machine/pull/3799 ?

Please re-open if I'm mistaken!  Thanks


Bedrock: support for custom endpoints

Is there a change to add extra configuration to define a custom endpoint to test Bedrock?
I was provided an URL similar to the following one:
https://{instance}.execute-api.{region}.amazonaws.com/{env}/v1/chat/completions

but noticed it goes internally to a fixed one, so I receive this kind of error
Failed to access underlying url 'https://bedrock.us-east-1.amazonaws.com/model/anthropic.claude-v2/invoke

thanks

This would be lovely - would welcome a PR or maybe @Benyuel or @vitaly-ps would be up for a crack at it?

@ladrians, is that a valid bedrock url?  I'm not seeing it in the documentation: https://docs.aws.amazon.com/bedrock/latest/APIReference/API_Operations.html
Also, currently the bedrock LLM is tightly coupled to the InvokeModelWithResponseStream endpoint and it's response format with streaming, so I'm not sure if we should expose/allow other urls (if there are some).

Yes it is, we have access to the following models referencing the initial url mentioned, 'anthropic.claude-v1', 'anthropic.claude-v2', 'anthropic.claude-instant-v1', 'amazon.titan-tg1-large', 'ai21.j2-ultra', 'ai21.j2-mid'.

Fixed by #2614 and will be live in the next release (0.0.149)!


Set ContryUtils public

To allow access of getFlagDrawableResId method

Great update! This is valid for some cases.


[4742] Fix: funding not cleared properly when changing courses multiple times

Context
https://trello.com/c/F9QZeh9e/4748-course-dates-get-lost-on-registered-trainees-when-changing-course
PublishCourseDetailsForm wasn't showing itt_start_date or itt_end_date because it was wrapped in a conditional which I don't think is relevant anymore. I think the conditional was added before we created the ITT dates page.
Changes proposed in this pull request

Remove conditional from PublishCourseDetailsForm#itt_start_date and PublishCourseDetailsForm#itt_end_date. The idea here is that if the IttDatesForm has stashed dates, we have to assume it was used previously and PublishCourseDetailsForm should default to its values first before falling back to the course dates.
Extract method to clear all stashes used beforehand and put it in CourseFormHelpers so it can be use by CourseDetailsForm which was clearing all the stashes after it saves its data.

Guidance to review

Start with a registered trainee on a publish course
Select 'change course'
Pick a new course and following the journey
The confirmation page should display dates entered earlier if asked

Important business

[ ] Does this PR introduce any PII fields that need to be overwritten or deleted in db/scripts/sanitise.sql?
[ ] Does this PR change the database schema? If so, have you updated the config/analytics.yml file and considered whether you need to send 'import_entity' events?

NB: Please notify the #twd_data_insights team and ask for a review if new fields are being added to analytics.yml

 Course dates get lost on Registered trainees when changing course.


IHittable

This PR creates an IHittable interface that plottable objects can implement so when they are clicked the control's OnLeftClickedPlottable event is fired. resolves #1844

Hi @swharden,
I looked at your work and want to leave a few comments.

Do we really need to support a unique cursor for each Plottable on the MouseOver event? It's hard for me to imagine such scenarios. Everything would be greatly simplified if this cursor were set the same in the Control itself.
IHittable may contain the ability to disable clicks by analogy with IDraggable.

public interface IHittable
{
  bool ClickEnabled {get; set};
  //or
  bool HitTestEnabled{get;set;}
...
}

You will not need to save the state in the plottable if you also transfer the current Plot.Dims to HitTest(), which are already saved. The hit calculation will be done every time HitTest() is called and it needs to be fast to be able to bind it to OnMouseMove events.

public bool HitTest(Coords c, PlotDimensions dims);

Hi @StendProg, thank you for your feedback! I really value your input. I found this problem to be surprisingly difficult, and it I put it down for a few days and am now coming back to it.
1. Do plottables require unique cursors?
I think so. Here are some potential use cases.

Plottable
Cursor

draggable vertical line

draggable horizontal line

draggable crosshair center

selectable point

draggable point

2. HitTestEnabled
public interface IHittable
{
  bool HitTestEnabled { get; set; }
...
}

Great suggestion! I will implement this.
3&4. Instead of storing state, can you pass dims into HitTest()?

The hit calculation will be done every time HitTest() is called and it needs to be fast to be able to bind it to OnMouseMove events.

Unfortunately the size of bounding box requires string measurement, and that is costly enough that I don't think it is worth performing on every mouse move event. Not finding a better solution this problem is why I put this PR down and haven't picked it up in two weeks 😅 It is much more difficult than I initially thought it would be. I can't think of a better way to achieve this, but after I merge this I welcome additional thoughts you have or suggestions for improvements here or in #1844.


Starter app's source code has hot reload instructions only for terminal, doesn't mention our IDE

From a UX study:
The comments/instructions for Hot Reload in the starter app's source code (after "new ThemeData") are written for someone using the terminal. Do we need to also add IntelliJ instructions here?

cc @mit-mit

I think we should be careful here. There's lots of IDEs that people could be using. IntelliJ is what we recommend, but isn't a requirement. We can probably make the text clearer though so that people aren't misled.

I don't see an issue with mentioning IntelliJ in addition to terminal.


GH-626 Fix handling of script placeholder properties

Fixes #626

Applied to main (=2.1) and branch 2.0.


Add iterators

makes validate(), pt.DataFrame.validate() & pt.Model.validate() return the dataframe being validated
adds filter_columns: bool option to all the validate()s
adds iter_models() to pt.DataFrame which returns an iterator of patito model instances
adds to_list() to pt.DataFrame which returns a list of patito model instances
adds iter() to pt.Model which returns an iterator of patito model instances
adds to_list() to pt.Model which returns a list of patito model instances

Questions

happy to remove the ListableIterator
should I add an example to the readme?
how to update docs (by the way I don't think the website docs are up-to-date)

Hi @joloppo! First and foremost, thank you for an excellent PR! Jakob and I really appreciated the work you put into this, it's clearly very thought-through!
We've accepted most of your additions. I suggest you take a look at the commits which have explanations for our modifications. I'm going to merge this right now, and we can follow up with anything else in an additional PR.
Thank you for your work!


Use a more general regex and improve error handling.

I added the following alias:
alias cmt='git commit -m "Update TODO"'

using:
 camer add cmt "git commit -m \"Update TODO\""

But I couldn't remove it later using camer remove cmt because the regex that Camer uses(shown below) to find aliases doesn't allow double quotes ; Camer is unable to find cmt when camer remove cmt is called, even though it exists in .camrc.
alias ([0-9a-zA-Z_]*)='([0-9a-zA-Z_ ]*)'

I noticed that we have been using a more general regex for ls -- matching non-space chars --, so I changed the code to use the same for remove and check_name_exist. Hopefully, this will not cause major issues later.
Also, I tried to print a useful message when remove fails to find the given alias-name to remove.

@hubvue any thoughts?

@harshshredding  Sorry to reply to you only now. this is a real problem, the project started out with the idea of just doing command alias, now it seems to be able to store single lines of code, scripts etc, great changes.


Support for EU (IC2 Classic)

Hello!
I am beta testing IC2 Classic and would love if you could implement support of EU on the controller. I understand power transfer will not be possible with the powertiers. But being able to power the controller to use for items and logic would be great.
I see Refined Storage supports power from IC2 and RF/tesla.

I stay here for assistance if you need any help implementing it.

I also had an idea, not sure if it is possible. But to support IC2 power transfer with xnet. You could maybe place down a (advanced) connector into a ic2 cable and it would get its properites from that cable. Regarding transfer limits and loss. So I could for instance have a copper cable going into a interface, run xnet cable then add another interface going into a new copper cable and create a energy network for that.


Add Norwegian localization for olympic sports

Added Norwegian localization for summer and winter olympics.

Thanks for your work @Majid-J !


This PR also adds type alias for the most commonly used types from the package and a simple example for using the router.

Example of using the router from this PR: https://github.com/radu-matei/spin-kv-explorer/blob/feat/golang/golang-explorer/main.go


[Suggestion] Cosmetic Armor Reworked compability?

Any possibility to add compability with Cosmetic Armor Reworked mod (https://www.curseforge.com/minecraft/mc-mods/cosmetic-armor-reworked)? Currently when we die the armor just stays on the ground next to the grave.

Yeah definitely! Sort of new to neoforge (or forge) mods so I don't know which mods there are to support very well. Will add compat with Cosmetic armor reworked in the next update though!

The mod has now been updated, containing compat with the cosmetic armor mod


Ballerina Streaming V2 Initial Implementation

Purpose

This PR contains the initial features related to Ballerina Streaming V2. This is implemented on top of Ballerina language. This PR only supports Filtering and Select capabilities.

Don't merge but review. I am waiting for a fix for Streams subscription functionality.

:x: Build ballerina-lang 108 failed (commit https://github.com/ballerina-platform/ballerina-lang/commit/481e8b6b85 by @mohanvive)

:x: Build ballerina-lang 134 failed (commit https://github.com/ballerina-platform/ballerina-lang/commit/97be7cbc62 by @mohanvive)

:white_check_mark: Build ballerina-lang 136 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/ab893bff94 by @mohanvive)

:white_check_mark: Build ballerina-lang 140 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/544342eeba by @mohanvive)

:white_check_mark: Build ballerina-lang 142 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/1baa4a60ec by @mohanvive)

:white_check_mark: Build ballerina-lang 149 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/93e5707780 by @mohanvive)

:white_check_mark: Build ballerina-lang 166 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/5627565ce1 by @mohanvive)

:white_check_mark: Build ballerina-lang 190 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/0ba19dbded by @mohanvive)

:white_check_mark: Build ballerina-lang 218 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/eab578c0f1 by @mohanvive)


replace dancing ACU with sane gif

For the love of god please replace the dancing ACU with a normal loading gif.
I even made one for you that you can use:

(it is a white circle on transparent background so it is suited for the dark client but looks not so good in a preview on white background)

Debatable - I like dancing ACU.

I also like the dancing ACU

Yes a setting to choose is probably needed

The dancing ACU consumes 20MiB of memory

Yeah that might be fixed by making a gif with less quality

or we use the spinning circle gif and all people that want the dancing acu can use a theme. Theme support is right around the corner, so we don't even need to code a setting to choose the gif anymore.

Check if we can use a sane format APNG instead of gif.
Gif is a horrible format and only has 1 bit transparency "channel" so you cannot create transparent gif that works on bright and dark background without artefacts, as seen in the gif above.

APNG isn't supported by JavaFX https://bugs.openjdk.java.net/browse/JDK-8092356

Has animating an SVG path via PathTransition been tried? That should cost basically no memory and should work with UI scaling without any artefacts.

Bump. I wanna know how many of the people that like this actually play ladder regularly.

There is an article that explains how to make a theme. A theme for replacing the gif is mostly a matter of putting the gif in the correct folder...

https://github.com/FAForever/downlords-faf-client/wiki/Themeing

The ACU should be in a theme, not the other way around.

Something like this can indeed be nice for a theme but should by no way be the default. It's way too cringe for that

The new tmm UI will not have that gif

TMM killed the ACU

TMM killed the ACU


Found a Bug, how to fix?

I noticed that when I run an account the script gets stuck when searching the webite when a cookies popup comes up and stops it from being able to do the quiz, and therefore just skips a whole lot. What I've done is gone into debugging mode and then whilst its running I accept the cookies quickly whilst the script is on. This does fix the issue and allows the script to work, and when I do this I do not receive any sort of error in the errors folder.
Is it possible for me to modify the code so I can make it automatically accept the cookies popup? Whenever I reopen the program the cookies seem to reset so I need to redo the whole process.

Hello. Sorry for the late response. I've taken a small break from this project to complete some school work. Ill be back soon to help with this.
(I'm also currently investigating this problem. When i have an answer i will get back to you)


Incorrect info on page

[Enter feedback here]
https://docs.microsoft.com/en-us/azure/defender-for-iot/quickstart-onboard-iot-hub
States that it is possible to add Defender on an existing IoT Hub.
Step 3 states:
Click Secure your IoT solution and complete the onboarding form.
There is no such button/link on the panel. See screenshot

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

ID: 9fb40bba-9d0f-dae4-bcbf-662de2aa0cd1
Version Independent ID: 5482893c-25ad-19ef-ea7c-e71379940e32
Content: Onboard to Defender for IoT agent-based solution
Content Source: articles/defender-for-iot/quickstart-onboard-iot-hub.md
Service: defender-for-iot
GitHub Login: @shhazam-ms
Microsoft Alias: shhazam

Hi, Thanks for writing. I am including @ElazarK  as he is dealing with the agent-based solution.

Hi @lucarv I am looking in to this now.

Hi, You can enable that, but it is not on the overview panel, it is on the security alerts panel.

@lucarv  i'm not sure what you mean by "You can enable that, but it is not on the overview panel, it is on the security alerts panel."
I reworded and clarified the instructions and added images to better reflect the correct procedure.

please let me know if solves your issue

Hello,
If you go to the portal, and click on security overview for an existing IoT Hub, you don't get that message. You only have this message if you have not previously done that action, i.e secured your IoT Hub. You could do that before Azure Defender for IoT came along.
However, if you click on the item below Overview, named Security Alerts, that is when you see that message with the option to activated it. Even if you have previously activated security for it hub.

@lucarv  you are correct, if you select any option under the security section, it will direct you to the "Secure your IoT solution" button.
I will correct the wording to reflect that.

@lucarv Thank you for the feedback. the documentation has been updated and will be published in the next publishing cycle.

#please-close


Parquet writes empty Rowgroup at the end of the file

At then end of a parquet file, it may writes an empty rowgroup.
This happens when: numberOfRecords mod sizeOfRowGroup = 0
Reporter: Tim / @tsdeng
Assignee: Tim / @tsdeng
Note: This issue was originally created as PARQUET-104. Please see the migration documentation for further details.

Julien Le Dem / @julienledem:
Issue resolved by pull request 66
https://github.com/apache/incubator-parquet-mr/pull/66


add an actual term model

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

85 of 101 (84.16%)  changed or added relevant lines in 5 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 81.994%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/dao/term.py
55
57
96.49%

sis_provisioner/management/commands/create_file.py
0
5
0.0%

sis_provisioner/models.py
20
29
68.97%

Files with Coverage Reduction
New Missed Lines
%

sis_provisioner/models.py
1
65.91%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
592

Relevant Lines:
722

💛  - Coveralls


Router is not work

I used ./install_app.sh to install a debug app on my mobile and failed to start trending activity.
Related log:
04-11 23:03:45.627 12155-12155/com.github.piasy.octostars E/Router: com.chenenyu.router.RouterBuildConfig
04-11 23:03:45.928 12155-12155/com.github.piasy.octostars E/Router: Could not find an Activity that matches the given uri.

First line is raised when Router.initialize(). RouterBuildConfig is not generated at compile-time, then Class.forname failed.
Second line is raised when Router.build(...).go(...) at splash view. Initialization is failed and it cannot find corresponding activity.
BTW, you forgot to add Router.openLog() at https://github.com/Piasy/AndroidTDDBootStrap/blob/master/contrib/business/src/main/java/com/github/piasy/octostars/BootstrapApp.java#L96 :)
FYI: update the router version to latest (0.7.0) could help.

Sorry that I was too busy these days, I'll try to take a look into it maybe a week later :)


[Quilt 1.20.1] Indium requires at least version 3.2.0 or any newer version of fabric-renderer-api-v1, but only a different version is present: 3.1.2

Crash Report:
crash-2023-10-07_<IP_ADDRESS>22-quilt_loader.txt

You need to install a recent version of Quilt Standard Libraries / Quilted Fabric API.


feat: implement range as a factory that returns a class

Just an idea. The range function could be a factory that returns an instance of a Range class.

class Range {
    // ...
}

export const range = (start: number, stop?: number, step?: number) => {
    if (stop === undefined) {
        return new Range(0, start, 1);
    }
    if (step === undefined) {
        return new Range(start, stop, 1);
    }
    return new Range(start, stop, step);
}

Advantages

Could simplify the current generator function by decoupling the creation from the implementation. The Range class would have three non-optional parameters.
Could implement other features of the python range object like len, in, maybe even reversed. These would have to be ECMA-ified though.

Disadvantages

Could be slower? At least for instantiation?
It's already been done, but not in TypeScript: https://github.com/michal-perlakowski/range.

It would be better to implement elsewhere. Going to start deno_itertools to try it out. Accessing the generator from a Sybmol.iterator method would be clunky.


Add an FAQ to documentation

This adds an FAQ to our documentation and links to it from the main README.

@timostamm Made the requested updates.  Let me know what you think.


[stack-ibm-core-security-services] Add a test to test existing resources

Set up a test that spins up some existing resources (SM, KMS and others when they are supported) using terraform, and then deploy the stack in the test

Pr merged - hence closing the issue


Collect pii

Adds collectPii API and updates the usage docs.

closing while i fix my fork due to repo private change.


standard capitalization option

Per this conversation with experts, it appears to me that the most widely accepted standard for PostgreSQL with respect to capitalization is that:

all keywords (SELECT, DELETE, UPDATE, ON, AND, etc.) are upper case, but
all datatypes (uuid, text, int, etc.) are lower case.

Unfortunately, I do not see a DBeaver setting that supports this dichotomy.
It would be nice to have a separate protocol for data-type capitalization which is distinct from keyword capitalization.

seems that it is already implemented. You just need to change highlighted options as it shown at the attached picture

I was going to disagree, but when I went to check, I found that that things like uuid and int are left alone, an it's just the TEXT data type that gets converted to upper case, though I'd rather it remain lower case like uuid and int.
Is there some reason why TEXT is treated this way?

issue with text data types will be resolved in the scope of #10180


Error when final_snapshot_identifier is not set

Hi, I've been using the module for a while but after the last version I'm getting an error after every terraform plan when final_snapshot_identifier is not set.
* module.db_instance.aws_db_instance.this: only alphanumeric characters and hyphens allowed in "final_snapshot_identifier"

It may be related to this closed issue

Thanks for reporting this issue. The problem was that the default value (empty string) was not good enough even though it is not used. Fixed in v1.0.8


Docs improvements to Healthwatch Metrics listing

Changing order of 2 Diego metrics in order to match screen order (similar to other listings). Also updated for next patch release which will introduce PCF 1.12 support. Also updated "tests" to "reports" where the word is more appropriate to the metrics being described.

Thanks.


Wrap CanvasTools in a React-component

Add a React-wrapper for the library with SVG-hosting code and glue code.

Keep as vanilla-js project.


Assets are 500?

Both of the links here are 500:
http://openvault.wgbh.org/exhibits/leary_and_jobs/media#tabs
The record has been ingested, so it's a render problem: http://openvault.wgbh.org/catalog/V_AD9E0BC353BF435E83F28DEF165D4F40.pbcore

Or a discrepancy between names: In the xml:
<pbcoreAnnotation annotationType="Special Collection">from_the_vault</pbcoreAnnotation><pbcoreAnnotation annotationType="Special Collection Tag">from_the_vault</pbcoreAnnotation>

But the name of MD is just 'vault.md'. Either could change, but they need to agree. @Muraszko ?

(We're getting 500s now, but if they were ingested into the current system, they would have failed validation.)

fixed.


DRAFT: Implement "last seen" warnings on the front-end

Untested because last seen stuff only works for Postgres DB right now, but I think this should be more-or-less done now


Write a Kotlin program to find volume of a cuboid

Description
Write a Kotlin program to find volume of a cuboid
Volume of a cuboid = Length × Width × Height
Length : 2
Width  : 3
Height : 4

Volume : 24

How to contribute

Save the solution in FindVolumeOfACuboid.kt file
Add FindVolumeOfACuboid.kt file in find-volume-of-a-cuboid folder

!assign


nitweb: search field

Introduce a search field in the top navbar connected to the /search service.
Added a stub implementation of a search service depending on APIList.

Demo: http://nitweb.moz-code.org/

when I click on a search result, the content of the page changes but the search result box is not closed and stays over the page

@privat should be better now

+1 search algo need tweaks but this is good enough

Looks great. However, I would like to click to a search result without having to click on the entity name (that is, the whole box would be a link).
:+1: +1

If you dont want to click, use your keyboard.
On Jun 3, 2016 09:21, "jcbrinfo"<EMAIL_ADDRESS>wrote:

Looks great. However, I would like to click to a search result without
having to click on the entity name (that is, the whole box would be a link).
👍 +1
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nitlang/nit/pull/2157#issuecomment-223576713, or mute
the thread
https://github.com/notifications/unsubscribe/AAjl6KY8RLABdZfIEQ7DR11n_ioFCd8Pks5qICozgaJpZM4ItE5_
.

What I mean is that when I look at a menu entry, I expect that the whole entry (the whole rectangle) to be clickable, not just a part of the text in it.

Yup, I understood your statement the first time.
On Jun 3, 2016 09:32, "jcbrinfo"<EMAIL_ADDRESS>wrote:

What I mean is that when I look at a menu entry, I expect that the whole
entry (the whole rectangle) to be clickable, not just a part of the text in
it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/nitlang/nit/pull/2157#issuecomment-223579380, or mute
the thread
https://github.com/notifications/unsubscribe/AAjl6C2zmrRA8SN7FgzNdisMS1A2iB_6ks5qICzRgaJpZM4ItE5_
.


how to get buffering progress on Android ?

Current behavior
I don't know how to get buffering progress on Android.Do you have any API?
Platform
Android
Version
react 16.5.0
react-native 0.57.2
react-native-video 4.20

What values are you getting back in your onProgress event?


JsonSubTypes support always assumes a @type property

Describe the bug
The generator for @JsonSubTypes assumes that there will always be a property named @type. This is the case with the @JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY) definition used with various built-in classes but it's not necessarily the case with other representations.
The property name can be customized with an optional property attribute on the @JsonTypeInfo annotation. If it's not defined, then the default value is based on the use value. NAME defaults to @type whereas e.g. MINIMAL_CLASS defaults to @c and DEDUCTION doesn't use any additional property at all.
Expected-behavior
An identifying property is injected into the generated TypeScript only when Jackson would actually include one in the generated JSON values. When a property is injected, then the correct name is used for that property.
Reproduction
Example without any additional property.
@JsonTypeInfo(use = JsonTypeInfo.Id.DEDUCTION)
@JsonSubTypes({ @Type(A.class), @Type(B.class) })
public class Base { }

public class A {
  private final String a;
  public A(String a) { this.a = a; }
  public String getA() { return a; }
}

public class B {
  private final String b;
  public B(String b) { this.b = b; }
  public String getB() { return b; }
}

An example with a different default property name can be seen by changing the built-in Filter to like this:
@JsonTypeInfo(use = JsonTypeInfo.Id.MINIMAL_CLASS, include = JsonTypeInfo.As.PROPERTY)
@JsonSubTypes({ @Type(value = OrFilter.class),
        @Type(value = AndFilter.class),
        @Type(value = PropertyStringFilter.class) })
public class Filter {

}

An example with a non-default property name can be seen by changing the @JsonTypeInfo of Filter like this:
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.PROPERTY, property = "t")

An example with a non-default type structure can be seen by changing the @JsonTypeInfo of Filter like this:
@JsonTypeInfo(use = JsonTypeInfo.Id.NAME, include = JsonTypeInfo.As.WRAPPER_OBJECT)

System Info
Hilla 2.3.2

Supporting all the JsonTypeInfo options is a huge amount of work. I mean huge. That's why we chose to limit to one syntax.
At the same time, it should probably skip generation if the syntax is different.

I agree. It's better to avoid injecting any property in the cases that are not explicitly supported since it's more convenient for the developer to include an additional property than to exclude a property that TypeScript thinks must be there.


feat: doc tool generate provides referenced by information on schema

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y
fix #165

2. What is the scope of this PR (e.g. component or file name):

kcl-lang/kcl-go/pkg/tools/gen
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[ ] N
[x] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

The above approach is wrong and causing errors. Working on using GetFullSchemaType function to extract the referenced by info from the base information being returned by it. Would push the changes soon.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

22 of 33 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.06%) to 61.863%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/tools/gen/gendoc.go
22
33
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
3254

Relevant Lines:
5260

💛  - Coveralls

I have done the following changes @Peefy:

Modified the GetFullSchemaType function in api.go to provide base schema information.
Updated genopenapi.go to get ReferencedBy info from base schema info being returned by GetFullschemaType.

Now, I have some doubts about adding the refernced by info to gendoc.go, since a type mismatch is occuring when trying to add the Referenced by info while rendering, and I need some reviews here to move ahead.

@shruti2522 Thank you, I believe there is no problem with the current overall implementation approach. I have left some details and methods for modifying test cases.

I guess there is some issue with the rendering logic too in gendoc.go, since the template is not able to render the Referenced By section. Working on the fix.

I guess there is some issue with the rendering logic too in gendoc.go, since the template is not able to render the Referenced By section. Working on the fix.

OK. But I think the current referencedBy logic may have been reversed firstly.

Schema A inherits B, indicating that B's referencedBy contains A.
The field of Schema A $ref B,  indicating that B's referencedBy contains A.

I guess there is some issue with the rendering logic too in gendoc.go, since the template is not able to render the Referenced By section. Working on the fix.

OK. But I think the current referencedBy logic may have been reversed firstly.

Schema A inherits Schema B, indicating that B's referencedBy contains A.
The field of Schema A $ref Schema B,  indicating that B's referencedBy contains A.
I see. I'll review the logic and make sure that Schema A properly references Schema B in the referencedBy field. Will update accordingly once fixed.

I see. I'll review the logic and make sure that Schema A properly references Schema B in the referencedBy field. Will update accordingly once fixed.

Hello @shruti2522, are you still working on this PR? Because we are preparing to release a new version of kcl-go in the next two days.

Yes @Peefy, I will push the changes by tomorrow at the latest.


No support for K95 RGB Platinum

The program doesen't have support for a k95 rgb platinum keyboard, I managed to get the G1-G6 keys working by using the freestyle tool and selecting every one of the G1-G18 keys on the on-screen layout. But I can't get the top bar of the RGB Platinum to work, which would be very nice to be a health bar in CS:GO. I hope you can add support for the Corsair K95 RGB Platinum! Thanks!

Duplicate of #445


The option to change the value with a mouse wheel should be switchable.

I use a mouse wheel to scroll the window in long forms where I transfer the data from Excel books into the Atomic Server forms. If I enter the data and a mouse pointer is still in the field, this attempt to scroll changes the value, so I need to re-enter that again, which is terrible and prone to additional errors. The option to change the values with the mouse wheel should be switchable for the Atomic Server instance.

Is this for number values? Could be a weird browser default

Yes, only for number values, both in Chrome and Edge. Actually, I think this is because of that element with up and down arrows on the right, can you turn it off and leave plain input?


PuLID stays persistent

It seems, once a sampling process with Flux PuLID started, it stays persistent even if the node is removed:
1st samling process:

2nd sampling process:

Could you please check?
Kind regards!

perhaps you loose the seed and run again

@sipie800 the PuLID still stays persistent even I change the seed:

My workaround is to set weight to 0, which 'resets' the model - not perfect but works for my use case.
https://github.com/balazik/ComfyUI-PuLID-Flux/issues/41 - this is the original issue

@jaszczw Unfortunately that does not appear to be a proper workaround. I found this issue while I was trying to integrate FluxPuLID into a more complex workflow which samples at first without FluxPuLID, later with FluxPuLID. As soon I restart the whole workflow all sample processes are using FluxPuLID.

Yeah, you would need to put pulid with basically anything in it with weight 0 - which I do.

@jaszczw Your suggestion does not work:

This is because Apply pulid flux is monkeypatching the unet, the current comfyui modelpatcher doesn't support the required block patching needed for plux so it's basically a hack (all implementations for flux not just this repo).
So if you use apply pulid, that unet is officially THAT pulid, it can't just be swapped again later or in a different line like above...
Use a different unet loader into the second apply pulid, though i dont know if that will work given internal comfy caching of models, but worth a try.


[FEATURE REQUEST] Custom border property in tooltips

I would like to be able to give the tooltips a border color and thickness.
eg
borderColor: '#000000',
borderWidth: '2px'

Done in #3872 which will be included in the next release 😄


[BUG]: document processor unavailable

How are you running AnythingLLM?
AnythingLLM desktop app 1.5.5 to 1.5.7.
What happened?
I am using a corporate desktop running windows 10, with restricted admin right controlled via domain. When I starts, I am unable to upload any files, showing 'Document Processor unavailable, we can't upload your files right now because the document processor offline. Please try again later'
The app is installed in C:\user<username>\AppData\Local\Programs\Anythingllm-Desktop.
Are there known steps to reproduce?
No response

The image failed to upload, but in general the collector app should attempt to bind to port 8888 and if unavailable the process will bind to another port. I presume the app has been fully quite and rebooted post-update and was not updated while open.
Are you able to retrieve logs from the appropriate debug method? https://docs.useanything.com/debug

Here is the log using cmd windows. Thanks for looking into the issue.
`[Preferences] preference config stored at C:\Users\WindowsUser\AppData\Roaming\anythingllm-desktop\config.json
Prisma schema loaded from prisma\schema.prisma
Datasource "db": SQLite database "anythingllm.db" at "file:C:/Users/WindowsUser/AppData/Roaming/anythingllm-desktop/storage/anythingllm.db"
Already in sync, no schema change or pending migration was found.
Running generate... (Use --skip-generate to skip the generators)
Error: request to https://binaries.prisma.sh/all_commits/61e140623197a131c2a6189271ffee05a7aa9a59/windows/query_engine.dll.node.gz.sha256 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
Prisma schema loaded from prisma\schema.prisma
Error: request to https://binaries.prisma.sh/all_commits/61e140623197a131c2a6189271ffee05a7aa9a59/windows/query_engine.dll.node.gz.sha256 failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
[OllamaProcessManager] Ollama will bind on port 11434 when booted.
[Preferences] Will load window with last know bounds.
node:internal/modules/cjs/loader:1087
throw err;
^
Error: Cannot find module 'uc.micro'
Require stack:

C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\linkify-it\build\index.cjs.js
C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mailparser\lib\mail-parser.js
C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mailparser\index.js
C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mbox-parser\dist\Parser.js
C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mbox-parser\dist\index.js
C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\collector.js
at node:internal/modules/cjs/loader:1084:15
at Function._resolveFilename (node:electron/js2c/utility_init:2:3040)
at node:internal/modules/cjs/loader:929:27
at Function._load (node:electron/js2c/asar_bundle:2:13327)
at Module.require (node:internal/modules/cjs/loader:1150:19)
at require (node:internal/modules/cjs/helpers:110:18)
at Object. (C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\linkify-it\build\index.cjs.js:3:16)
at Module._compile (node:internal/modules/cjs/loader:1271:14)
at Object..js (node:internal/modules/cjs/loader:1326:10)
at Module.load (node:internal/modules/cjs/loader:1126:32) {
code: 'MODULE_NOT_FOUND',
requireStack: [
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\linkify-it\build\index.cjs.js',
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mailparser\lib\mail-parser.js',
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mailparser\index.js',
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mbox-parser\dist\Parser.js',
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\node_modules\mbox-parser\dist\index.js',
'C:\Users\WindowsUser\AppData\Local\Programs\anythingllm-desktop\resources\backend\collector.js',
undefined
]
}

Node.js v18.16.1
[TELEMETRY ENABLED] Anonymous Telemetry enabled. Telemetry helps Mintplex Labs Inc improve AnythingLLM.
prisma:info Starting a sqlite pool with 13 connections.
[TELEMETRY SENT] {
event: 'server_boot',
distinctId: '2a0d551c-df4a-410f-a197-fe78c56bd74f',
properties: { runtime: 'desktop' }
}
Hot loading of AnythingLLMOllama - LLM_PROVIDER is anythingllm_ollama.
[NativeEmbedder] Initialized
[CommunicationKey] RSA key pair generated for signed payloads within AnythingLLM services.
[production] AnythingLLM Standalone Backend listening on port 3001
[OllamaProcessManager] SINGLETON LOCK: Using existing OllamaProcessManager.
[OllamaProcessManager] [windows] Ollama subprocess running. Port 11434. PID 5496.
OllamaAPI offline - retrying. 1/3`

@jackfood Is the system running the app offline? I only ask as the main issue is

Error: Cannot find module 'uc.micro'

Which appears to only be a dependency for parsing mbox types files. This should obviously not block boot up and others do not seem to suffer from it. So want to see what is going on client side that could prevent that.
I strongly suspect this is some local AV software stripping the uc.micro library from the system because it does not like that "extension" even though it is a folder. Reference
If you allowlist AnythingLLM in your AV - does the problem go away?
closing for now, since it appears to be client side, but discussion can still continue

Thanks for your reply. I do not have control of AV whitelist as it is corporate laptop controlled by domain (encrypted by bitlocker). However, the same type of laptop (unencrypted) using same AV using mklink to bring anythingllm to D:\  solves this issue of uploading.