API: progammable API?

First thanks for the great job on this gallery. It is one of the best I've seen so far.
I'm trying to implement some url mechanism to track which photo is currently displayed and correctly handling the back button. Problem is I have a hard time manipulating the gallery programmatically. Currently I would need:

open an image via its number
switch to any image
close the full width display of an image

What do you think? Would it be possible?

I'm glad to hear that!

if you mean opening a gallery on the specific image, that's a bit tricky thing to do as images are identified by a composite key (gallery ID, index). Much easier for now would be if you triggered basic click() event on desired thumbnail element.
this can be done easily, supplementing showNext, showPrevious methods.
do you mean simply closing the gallery? If so, this also can be done.

well that's already what I'm currently doing, but this is convoluted. I need to store the number of the item, and when restoring the display of an image I need to find the image and then trigger the click. Click that won't trigger anything if the gallery is already open.
Just using those methods requires work on my end. :)
I mean doing what is done when afterHide is triggered. Right now I have to trigger a click on the overlay of the gallery.

What I expect from such an API would be:

onChange should provide all information needed to move to a given state
have a method show that would move the gallery and accept the previously given information
have a close that will be the opposite of show

As for the composite id, no issue here for me. I would either store both info or make a single out of that :)

Sorry, I don't quite understand this use case. When you get a NodeList of images, each index should represent the nth image. I haven't tried that in practise, but seems straightforward.

Click that won't trigger anything if the gallery is already open.

Is that a wrong behaviour?

I didn't mean using these methods, I just wanted to say that the proposed show() method is easy to implement :)
As above

onChange, what should this do? What kind of state?
show, previously given information? Do you mean this method should also open a gallery, right?
close, opposite of show in what way? Isn't it enough to just hide the gallery or what else do you need?
The problem with this composite ID is that the galleries are identified only by a reference to an array, something you can't access from the outside.


How to close the terminal after running the Alright wrapper?

Are you referring to closing the WhatsApp window or Terminal?

@LouisSchieferle , if you intend to close the browser, the method for that has been implemented, check it out!
However, if you do want to close the terminal itself, try something like exec python ./my_script.py instead of simply python ./my_script.py.
Hope this is helpful.

Additional tip, @LouisSchieferle: if you intend to close the browser, you can use messenger.browser.quit() on your code. Hope this is helpful!


Vim9: crash when first line in heredoc starts with #

Describe the bug
In Vim9 script, Vim might crash when using a heredoc whose first line starts with a # character.
To Reproduce
Run this shell command:
vim -Nu NONE -S <(cat <<'EOF'
    vim9script
    def Func()
        let lines =<< trim END
        # aaa
        END
    enddef
    defcompile
EOF
)

Vim crashes.
Expected behavior
Vim doesn't crash.
Environment

Vim version: 8.2 Included patches: 1-1489
OS: Ubuntu 16.04.7 LTS
Terminal: XTerm(358)

Additional context
For some reason, the issue is only triggered if the first line in the heredoc starts with #.  The next lines can start with #; it doesn't make Vim crash:
vim9script
def Func()
    let lines =<< trim END
    bbb
    # aaa
    END
enddef
defcompile

The issue only occurs inside a :def function.  Not in a legacy function:
vim9script
fu Func()
    let lines =<< trim END
    # aaa
    END
endfu
Func()

Nor at the script level:
vim9script
let lines =<< trim END
# aaa
END

It's caused by a regression introduced in the patch 8.2.1243, which fixed the issue https://github.com/vim/vim/issues/6482.

backtrace
valgrind log
asan log


Release 1.0.0 doesn't include bunldex.exs file

Release 1.0.0 doesn't include bunldex.exs file, which leads to errors during the compilation of the project:
==> video_mixer
could not compile dependency :video_mixer, "mix compile" failed. Errors may have been logged above. You can recompile this dependency with "mix deps.compile video_mixer", update it with "mix deps.update video_mixer" or clean it with "mix deps.clean video_mixer"
** (Code.LoadError) could not load /Users/radoslawszuma/dev/membrane_live/deps/video_mixer/bundlex.exs. Reason: enoent
    (elixir 1.14.0) lib/code.ex:1807: Code.find_file!/2
    (elixir 1.14.0) lib/code.ex:1237: Code.require_file/2
    (bundlex 1.0.1) lib/bundlex/project.ex:162: Bundlex.Project.load/1
    (bundlex 1.0.1) lib/bundlex/project.ex:141: Bundlex.Project.get/1
    (bundlex 1.0.1) lib/tasks/compile.bundlex.ex:29: Mix.Tasks.Compile.Bundlex.run/1
    (mix 1.14.0) lib/mix/task.ex:421: anonymous fn/3 in Mix.Task.run_task/4
    (mix 1.14.0) lib/mix/tasks/compile.all.ex:92: Mix.Tasks.Compile.All.run_compiler/2
    (mix 1.14.0) lib/mix/tasks/compile.all.ex:72: Mix.Tasks.Compile.All.compile/4

I've just released v1.0.1 which includes the bundlex.exs file. Thanks!


Decide when to stack

Is your feature request related to a problem? Please describe.
I'm implementing a work planning which would have for each item a label above that might be longer than the owning item itself.

By far, i only show the label when the length of item is larger than a certain threshold.
The current implementation of stacking items, i believe, bases on the calculation of start_time and end_time. Is there anyway to override the stacking process ?

@perfectstrong I am not sure I understand what you are trying to implement but I think itemRenderer might solve your issue.
as for stack items you can't control that

Sorry for not clarifying my intent. Sometimes, the label on each item is long enough to overlap each other. For example here, i have a label "M. Toto DUPONT DUPONT DUPONT" extremely long.

Hi @perfectstrong
I currently have the same problem with stacking when the labels are larger than the items.
Do you already have a solution for this?

@jeuhen i didn't work on this issue for a long time and by far, no solution has been found yet.


tokio-core is deprecated.

Any progress on this?

Just a short note (as I otherwise don't have the time to work on this library): moving away from tokio_core to tokio would have to account for the use of the former in tokio_proto, also deprecated but unfortunately the foundation of ldap3. I don't see a straightforward way to excise tokio_proto without a massive rewrite of this crate's internals.

Yes, I was realizing that last night when I was trying to do the migration myself.

Hi @inejge , I had a go at trying to port to tokio today, and have made some progress. It looks like the blocker is tokio_proto more than anything, for the multiplex tcp connections.
I'm due to release a program that will be deployed at clients in the coming days, so I don't have a choice but to "make it work".
I'm going to try something from the tower ecosystem and see if it works. I'll provide feedback in a few days

Right, the tokio_proto multiplexer is the tricky part, and having it available is what made initial development comparatively straightforward. For your immediate problem, I'd recommend starting a dedicated thread for LDAP connection(s), with its own tokio_core event loop, and communicating with the rest of the system via oneshot/mpsc channels. I believe these can work across tokio flavors.

I found https://github.com/red75prime/tokio-proto/commit/cd1af828791fcd60dc518eb713a53d00e9aacf91, which while not ideal, still allowed me to get the crate to build. I just got it to build now actually.

With the port to Tokio 0.2, this can now be closed.

Thanks @inejge


vis2x.: Syntax highlighting and autocomplete does not work

Describe the bug
Syntax highlighting and autocomplete does not work (f. ex. in basic html widget).
Expected behavior
Syntax highlighting and autocomplete should work same as in vis1.x (or in CSS section of vis2.x).
Versions:

Adapter version: ioBroker.vis v2.0.6
JS-Controller version: v4.0.23
Node version: v16.18.1
Operating system: ioBroker.docker v7.2.1

Syntax highlighting and autocomplete works as expected in v2.0.9.
thank you very much!


Cannot claim Front end dev cert

Hi, this looks to be the same as issue #5174, which is closed.
When I try to claim my front end development certificate at:
https://www.freecodecamp.com/challenges/claim-your-front-end-development-certificate
When I get to step 3, and I click "Open link in new tab (this unlocks this step)", I get an "Oops" error.
Console log says:
Failed to parse SourceMap: https://www.freecodecamp.com/challenges/rx.all.map
Screenshot below:

I don't know if it helps but I started the front end curriculum in Sep 2015, then put it on hold for a few months (Nov, Dec, Jan). When I came back, I noticed the curriculum had changed, some challenges were added and a couple projects I had completed had been removed, although those I had completed still showed. Maybe the parser is expecting the most up to date curriculum but choking on some of the old ones I did..?

I am having the same problem getting the back end certificate :cry:
It seems to mess up at the same step. I managed to trick it to let me through that step... which may have been a mistake because now the back-end certificate challenge is ticked, but my certificate has not been added to my portfolio :disappointed:

I have the same problem, I cannot get my Front End Certificate.

I would recommend each person experiencing this to create an issue, so that
it will get attention.
On Apr 8, 2016 9:49 AM, "Stephane Rousseau"<EMAIL_ADDRESS>wrote:

I have the same problem, I cannot get my Front End Certificate.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7963#issuecomment-207508926

I did updated an issue on git.
Thank you,
Stephane
Confidentiality Warning: This message, including any attachment, is sent
only for the use of the intended recipient; it is confidential and may
constitute privileged information. If you are not the intended recipient,
you are hereby notified that any printing, copying, distribution or other
use of this message is strictly prohibited. If you have received this email
in error, please notify the sender immediately by return email, and delete
it. Thank you!
2016-04-08 12:57 GMT-04:00 Yulgene Suh<EMAIL_ADDRESS>
I would recommend each person experiencing this to create an issue, so that
it will get attention.
On Apr 8, 2016 9:49 AM, "Stephane Rousseau"<EMAIL_ADDRESS>wrote:

I have the same problem, I cannot get my Front End Certificate.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
<
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7963#issuecomment-207508926

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/7963#issuecomment-207514237

ᐧ

I'm working on a solution. Thank you all for your patience.

Fix is in production. Thanks for you patience.
Happy Coding!


Seed data migrations don't compensate for on delete cascade

Hi,
I have a configuration file that automatically seed data in my database.
After I create a first migration, all tables are filled with data (including relation tables) when I update the database with the following command :
dotnet ef database update
Here is an example of data from migration file that I want to insert in my database :
migrationBuilder.InsertData(
    table: "REL_FUNCTION_MEASURE",
    columns: new[] { "MEASURE_UID", "FUNCTION_UID" },
    values: new object[,]
    {
         { 4, 3 },
         { 4, 4 },
         { 4, 5 },
         { 5, 5 },
         { 5, 2 },
         { 5, 3 },
         { 5, 4 },
         { 4, 2 },
         { 5, 1 },
         { 4, 1 },
         { 3, 2 },
         { 3, 4 },
         { 3, 3 },
         { 3, 1 },
         { 2, 5 },
         { 2, 4 },
         { 2, 3 },
         { 2, 2 },
         { 2, 1 },
         { 1, 4 },
         { 1, 3 },
         { 1, 2 },
         { 1, 1 },
         { 3, 5 },
         { 1, 5 }
    });

And here is the data in the seed file :
modelBuilder.Entity<MeasureEntity>().HasData(

	new MeasureEntity { Id = 1, Name = "Quantity", Description = "Quantity measure", Type = MeasureTypeEnum.QUANTITY },
	new MeasureEntity { Id = 2, Name = "Unit price", Description = "Unit price measure", Type = MeasureTypeEnum.UNIT_PRICE },
	new MeasureEntity { Id = 3, Name = "Unit margin", Description = "Unit margin measure", Type = MeasureTypeEnum.UNIT_MARGIN },
	new MeasureEntity { Id = 4, Name = "Unit weight", Description = "Unit weight measure", Type = MeasureTypeEnum.UNIT_WEIGHT },
	new MeasureEntity { Id = 5, Name = "Amount", Description = "Amount measure", Type = MeasureTypeEnum.AMOUNT }
);

modelBuilder.Entity<FunctionEntity>().HasData(

	new FunctionEntity { Id = 1, Name = "Sum", Description = "Sum function", Type = FunctionTypeEnum.SUM },
	new FunctionEntity { Id = 2, Name = "Min", Description = "Min function", Type = FunctionTypeEnum.MIN },
	new FunctionEntity { Id = 3, Name = "Max", Description = "Max function", Type = FunctionTypeEnum.MAX },
	new FunctionEntity { Id = 4, Name = "Average", Description = "Average function", Type = FunctionTypeEnum.MEAN },
	new FunctionEntity { Id = 5, Name = "Median", Description = "Median function", Type = FunctionTypeEnum.MEDIAN }
);

modelBuilder.Entity<FunctionMeasureRelationEntity>().HasData(

	new FunctionMeasureRelationEntity { FunctionId = 1, MeasureId = 1 },
	new FunctionMeasureRelationEntity { FunctionId = 2, MeasureId = 1 },
	new FunctionMeasureRelationEntity { FunctionId = 3, MeasureId = 1 },
	new FunctionMeasureRelationEntity { FunctionId = 4, MeasureId = 1 },
	new FunctionMeasureRelationEntity { FunctionId = 5, MeasureId = 1 },

	// Unit price functions
	new FunctionMeasureRelationEntity { FunctionId = 1, MeasureId = 2 },
	new FunctionMeasureRelationEntity { FunctionId = 2, MeasureId = 2 },
	new FunctionMeasureRelationEntity { FunctionId = 3, MeasureId = 2 },
	new FunctionMeasureRelationEntity { FunctionId = 4, MeasureId = 2 },
	new FunctionMeasureRelationEntity { FunctionId = 5, MeasureId = 2 },

	// Unit margin functions
	new FunctionMeasureRelationEntity { FunctionId = 1, MeasureId = 3 },
	new FunctionMeasureRelationEntity { FunctionId = 2, MeasureId = 3 },
	new FunctionMeasureRelationEntity { FunctionId = 3, MeasureId = 3 },
	new FunctionMeasureRelationEntity { FunctionId = 4, MeasureId = 3 },
	new FunctionMeasureRelationEntity { FunctionId = 5, MeasureId = 3 },

	// Unit weight functions
	new FunctionMeasureRelationEntity { FunctionId = 1, MeasureId = 4 },
	new FunctionMeasureRelationEntity { FunctionId = 2, MeasureId = 4 },
	new FunctionMeasureRelationEntity { FunctionId = 3, MeasureId = 4 },
	new FunctionMeasureRelationEntity { FunctionId = 4, MeasureId = 4 },
	new FunctionMeasureRelationEntity { FunctionId = 5, MeasureId = 4 },

	// Amount functions
	new FunctionMeasureRelationEntity { FunctionId = 1, MeasureId = 5 },
	new FunctionMeasureRelationEntity { FunctionId = 2, MeasureId = 5 },
	new FunctionMeasureRelationEntity { FunctionId = 3, MeasureId = 5 },
	new FunctionMeasureRelationEntity { FunctionId = 4, MeasureId = 5 },
	new FunctionMeasureRelationEntity { FunctionId = 5, MeasureId = 5 }
);

However when I create a new migration, I found out that relations table are not filled with data from my seed file (FunctionMeasureRelationEntity as exemple) in my database.
I checked the new migration files and I noticed that I didn't find data insertions in my relation table (that I had in my previous migration).
Is that classic comportement from EF Core and I forgot to set some configuration in my code to keep insertion process for my relation tables ?
Further technical details
EF Core version: 2.2.3
Database Provider: Pomelo MySQL
IDE: Visual Studio Code version: 1.33.0

@smerliere I'm likely not 100% following what you are saying, but if the data did not change between the first and second migration, then there should not be anything to do in the second migration, and hence it should not include the seed data. In other words, the first migration will insert the data, and the second migration does not need to because the first migration already did it.
I would only expect to see data inserts/updates in subsequent migrations if the data changes.
If this doesn't help, then can you file a small, runnable project/solution that demonstrates the behavior you are seeing so that we can investigate further.

Thank you for your answer.
I send you a runnable project.
TestSeeding.zip
Don't forget to configure a MariaDB's database (version:10.3) and modify the configuration in appsettings.json
In this project, we have three entities that represent our database tables:
-MeasureEntity :
/// <summary>
    /// Measure entity.
    /// </summary>
    [Table("DIM_MEASURE")]
    public class MeasureEntity : EntityDateBase, IEntityBase<int>
    {
        /// <summary>
        /// Id.
        /// </summary>
        [Key]
        [Column("MEASURE_UID")]
        public int Id { get; set; }

        /// <summary>
        /// Name.
        /// </summary>
        [Column("MEASURE_NAME")]
        [Required]
        public string Name { get; set; }

        /// <summary>
        /// Description.
        /// </summary>
        [Column("MEASURE_DESCRIPTION")]
        [Required]
        public string Description { get; set; }

        /// <summary>
        /// Type.
        /// </summary>
        [Column("MEASURE_TYPE")]
        [Required]
        public MeasureTypeEnum Type { get; set; }

        /// <summary>
        /// Functions relation list.
        /// </summary>
        public IEnumerable<FunctionMeasureRelationEntity> Functions { get; set; }
    }

-FunctionEntity
    /// <summary>
    /// Function entity.
    /// </summary>
    [Table("DIM_FUNCTION")]
    public class FunctionEntity : EntityDateBase, IEntityBase<int>
    {
        /// <summary>
        /// Id.
        /// </summary>
        [Key]
        [Column("FUNCTION_UID")]
        public int Id { get; set; }

        /// <summary>
        /// Name.
        /// </summary>
        [Column("FUNCTION_NAME")]
        [Required]
        public string Name { get; set; }

        /// <summary>
        /// Description.
        /// </summary>
        [Column("FUNCTION_DESCRIPTION")]
        [Required]
        public string Description { get; set; }

        /// <summary>
        /// Type.
        /// </summary>
        [Column("FUNCTION_TYPE")]
        [Required]
        public FunctionTypeEnum Type { get; set; }

        /// <summary>
        /// Measures relation list.
        /// </summary>
        public IEnumerable<FunctionMeasureRelationEntity> Measures { get; set; }
    }

-MeasureFunctionEntity (relation many to many between MeasureEntity and FunctionEntity)
    /// <summary>
    ///  FunctionMeasure relation entity
    /// </summary>
    [Table("REL_FUNCTION_MEASURE")]
    public class FunctionMeasureRelationEntity
    {
        /// <summary>
        /// Measure Id.
        /// </summary>
        [Column("MEASURE_UID")]
        [Required]
        public int MeasureId { get; set; }

        /// <summary>
        /// Function Id.
        /// </summary>
        [Column("FUNCTION_UID")]
        [Required]
        public int FunctionId { get; set; }

        /// <summary>
        /// Measure relation.
        /// </summary>
        public MeasureEntity Measure { get; set; }

        /// <summary>
        /// Function relation.
        /// </summary>
        public FunctionEntity Function { get; set; }
    }

There is already an initial migration that you can run with :
dotnet ef database update
to create tables and seed associated datas included in ApplicationDbContextSeed.cs
To reproduce our problem, you can add a property into MeasureEntity like the following example :
/// <summary>
    /// Measure entity.
    /// </summary>
    [Table("DIM_MEASURE")]
    public class MeasureEntity : EntityDateBase, IEntityBase<int>
    {
        /// <summary>
        /// Id.
        /// </summary>
        [Key]
        [Column("MEASURE_UID")]
        public int Id { get; set; }

        /// <summary>
        /// Name.
        /// </summary>
        [Column("MEASURE_NAME")]
        [Required]
        public string Name { get; set; }

        /// <summary>
        /// Description.
        /// </summary>
        [Column("MEASURE_DESCRIPTION")]
        [Required]
        public string Description { get; set; }

        **/// <summary>
        /// Column testing.
        /// </summary>
        [Column("MEASURE_COLUMN_TESTING")]
        [Required]
        public string TestingColumn { get; set; }**

        /// <summary>
        /// Type.
        /// </summary>
        [Column("MEASURE_TYPE")]
        [Required]
        public MeasureTypeEnum Type { get; set; }

        /// <summary>
        /// Functions relation list.
        /// </summary>
        public IEnumerable<FunctionMeasureRelationEntity> Functions { get; set; }
    }

Then, you can create and apply a new migration. You will can see that datas in Measure and Function tables are here. However the table REL_FUNCTION_MEASURE is empty.

@ajcvickers  I am on the same issue and I just tested with SQLite to isolate the problem of MariaDB and Pomelo and I confirm that I have the same problem.
At the second migration, the data in the relationship table is empty.

The issue here is that the FKs on REL_FUNCTION_MEASURE have OnDelete set to Cascade, but we don't compensate for this when dropping rows.
The workaround would be to manually copy the InsertData for the relationship tables to the migrations that drop principal rows.

@AndriySvyryd Ok I understand why we are losing relationship data.
On the other hand I find that your compensation solution is not very practical in the case where we need to change the database several times because it would be necessary to copy / paste the isnert data for each relation tables in each new migration.
It's just not feasible for a production environment ...

@aborderon Sure, you can also change the FKs to DeleteBehavior.Restrict, add the migration, than change them back and add another migration.

I tried to add OnDelete.(DeleteBehavior.Restrict) on one relation of the intermediate table like :
        /// <Summary>
        /// Function Measure relation configuration.
        /// </Summary>
        /// <param name="builder"></param>
        void ConfigureFunctionMeasureRelation(EntityTypeBuilder<FunctionMeasureRelationEntity> builder)
        {
            builder.HasKey(fm => new { fm.MeasureId, fm.FunctionId });

            builder.HasOne(fm => fm.Measure)
                .WithMany(m => m.Functions)
                .HasForeignKey(fm => fm.MeasureId);

            builder.HasOne(fm => fm.Function)
                .WithMany(f => f.Measures).IsRequired()
                .HasForeignKey(fm => fm.FunctionId)
                .OnDelete(DeleteBehavior.Restrict);
        }

But suddenly I get a constraint error during the second update (SQLite Error 19: 'FOREIGN KEY constraint failed'.). On the EF core doc it is specified:

Note that in this case your code must synchronize the child entities and their foreign key values manually. Otherwise, constraint exceptions will be lifted.

@aborderon On SQLite many schema operations are not supported, so you would have to use the first workaround.

Yes, I realized it right after, I connected the initial Pomelo.MySQL provider to my MariaDB database but I get the same problem during the second update:
MySql.Data.MySqlClient.MySqlException (0x80004005): Cannot delete or update a parent row: a foreign key constraint fails (ibis_api.REL_FUNCTION_MEASURE, CONSTRAINT FK_REL_FUNCTION_MEASURE_DIM_FUNCTION_FUNCTION_UID FOREIGN KEY (FUNCTION_UID) REFERENCES DIM_FUNCTION (FUNCTION_UID)) ---> MySql.Data.MySqlClient.MySqlException (0x80004005): Cannot delete or update a parent row: a foreign key constraint fails (ibis_api.REL_FUNCTION_MEASURE, CONSTRAINT FK_REL_FUNCTION_MEASURE_DIM_FUNCTION_FUNCTION_UID FOREIGN KEY (FUNCTION_UID) REFERENCES DIM_FUNCTION (FUNCTION_UID))
I also tried to define the relationship directly on each tables (MeasureEntity and FunctionEntity) but without any possitive results.

@aborderon Make sure that you are changing the OnDelete value in the same migration as the seed data.
Also you can avoid needless reseeding by not setting CreatedDate and UpdatedDate to DateTime.Now.
Post your migrations if you are still having issues.

Hello @AndriySvyryd ,
I just did different tests and I understood the problem.
Effectively, by removing the Datetime.Now assignment on the CreatedDate and UpdateDate properties there is no problem even with a cascading configuration it works fine.
On the other hand, I use MariaDB in production database and SQLite in database for unit tests and functional tests.
That is to say at the launch of unit tests, migrations and seeds are also launched to be in line with the production environment.
The problem is that SQLite obviously does not support the ValueGeneratedOnAdd propertybuilder defined in the DbContext;
builder.Entity<FunctionEntity>().Property(c => c.CreatedDate).ValueGeneratedOnAdd();
builder.Entity<MeasureEntity>().Property(c => c.CreatedDate).ValueGeneratedOnAdd();
builder.Entity<FunctionEntity>().Property(c => c.UpdatedDate).ValueGeneratedOnAddOrUpdate();
builder.Entity<MeasureEntity>().Property(c => c.UpdatedDate).ValueGeneratedOnAddOrUpdate();

I also tried to force it with the annotation [DatabaseGenerated (DatabaseGeneratedOption.Identity)] for the property CreatedDate and [DatabaseGenerated (DatabaseGeneratedOption.Computed)] for UpdatedDate but nothing to do, I always get the error:
Microsoft.Data.Sqlite.SqliteException: SQLite Error 19: 'NOT NULL constraint failed: DATA_CUSTOMER.CREATED_DATE'.
I remember why I had forced DateTime.Now on the properties ...
Do you have a solution to handle the case of SQLite for testing ?

@aborderon You can add a default value just for SQLite:
if (Database.IsSqlite())
{
    builder.Entity<FunctionEntity>().Property(c => c.CreatedDate).HasDefaultValueSql("CURRENT_TIMESTAMP");
    builder.Entity<MeasureEntity>().Property(c => c.CreatedDate).HasDefaultValueSql("CURRENT_TIMESTAMP");
    builder.Entity<FunctionEntity>().Property(c => c.UpdatedDate).HasDefaultValueSql("CURRENT_TIMESTAMP");
    builder.Entity<MeasureEntity>().Property(c => c.UpdatedDate).HasDefaultValueSql("CURRENT_TIMESTAMP");
}

Hello,
I had the same idea that you yesterday afternoon, it confirms me on my idea.
I thank you for your help so I will close the ticket.


Failed to dynamically deploy this plugin

After building the hpi, when I tried to upload the plugin to Jenkins (1.612) and the following error was encountered.
java.io.IOException: Failed to dynamically deploy this plugin
at hudson.model.UpdateCenter$InstallationJob._run(UpdateCenter.java:1328)
at hudson.model.UpdateCenter$DownloadJob.run(UpdateCenter.java:1126)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at hudson.remoting.AtmostOneThreadExecutor$Worker.run(AtmostOneThreadExecutor.java:110)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.io.IOException: Failed to install build-notification plugin
at hudson.PluginManager.dynamicLoad(PluginManager.java:474)
at hudson.model.UpdateCenter$InstallationJob._run(UpdateCenter.java:1324)
... 5 more
Caused by: jenkins.ExtensionRefreshException: hudson.plugins.persona.random.RandomPersonaFinder@867523e doesn't support refresh
at jenkins.model.Jenkins.refreshExtensions(Jenkins.java:2224)
at hudson.PluginManager.dynamicLoad(PluginManager.java:467)
... 6 more

Well, this is not something related to this plugin. It's clear in the stacktrack that the problem is the Persona Plugin that doesn't support a refresh and you're trying to install a plugin without a restart in Jenkins.
I believe you're facing this problem here:
https://issues.jenkins-ci.org/browse/JENKINS-12803

Sorry. I just figured that out! I restarted Jenkins and I can see the plugin installed already. Apologies. Thanks.

No problem. And thanks for using this plugin!


Improve formatter to support partial expression nodes

Purpose

This PR intends to fix issue #39813

Fixes #39813
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@AzeemMuzammil were you able to look into the windows test failures in here?


Adding more error types

#6

Thanks for working on this. Hope I'm not micromanaging too much.

Not at all, it's been fun to do

Sweet. There are a couple unresolved issues and one that I had a follow-up with. Then I should be good to approve this bad boy.
Lmk if you have any questions.


Gmail Helper

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0c87c29). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   93.81%
=======================================
  Files           ?        1
  Lines           ?      194
  Branches        ?       41
=======================================
  Hits            ?      182
  Misses          ?        8
  Partials        ?        4

Continue to review full report at Codecov.

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


feat(tagmanager): add mpv2 user data transformation

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

9 of 212 (4.25%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 10.861%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/config/googleadsconversion.go
0
1
0.0%

pkg/provider/tracify/server.go
0
1
0.0%

pkg/provider/facebook/server.go
0
2
0.0%

pkg/provider/tracify/server/tag/tracify.go
0
2
0.0%

pkg/provider/facebook/server/tag/conversionsapitag.go
0
3
0.0%

pkg/config/facebook.go
0
5
0.0%

pkg/tagmanager/server/template/jsonrequestvalue.go
0
6
0.0%

pkg/provider/googleanalytics/server.go
0
14
0.0%

pkg/tagmanager/server/variable/mpv2data.go
0
16
0.0%

cmd/tagmanager/tags.go
8
25
32.0%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.4%

Covered Lines:
381

Relevant Lines:
3508

💛  - Coveralls


A question regarding the lr scheduler

Hi, is that correct to set milestones to 200ep? Thanks.
milestones:
    - 200ep

https://github.com/mosaicml/diffusion/blob/2eb6228ef205832aaf7d3111b64a38cdfa4c06f6/yamls/hydra-yamls/SD-2-base-256.yaml#LL62C14-L62C14

Hi, this is a hold over from a previous experiment. For our training LR schedule, we used linear warmup then a constant learning rate. As long as your number of epochs is < 200, the config you posted will work.
Below is an alternative more accurate config:
scheduler:
  _target_: composer.optim.LinearWithWarmupScheduler
  t_warmup: 10000ba
  alpha_i: 1.0
  alpha_f: 1.0

Thanks.
Can you provide the accurate SD-2-base-256.yaml and SD-2-base-512.yaml that were used in your experiment? So that we can reproduce the SD-2-base you have trained.

The yaml you referenced in the repo are the ones we used for training except for the missing dataset paths.
The alternative scheduler I posted above was an adjustment we wanted to make, but did not have a chance to add it.

So I should use the alternative scheduler to reproduce your model training, am I correct?

Apologies for the confusion. The config we used to train was the one you specified above:
milestones:
    - 200ep

The alternative scheduler should be equivalent and we plan to use it for future runs, but we did not use the alternative scheduler for our blog post training run.

it's actually the same, right? since the training won't reach to 200ep, so the lr will be the constant value=1e-4 after warmup

BTW, does this yaml file contain all those five techniques to speed up the training? Or should I add them by myself? Thanks~

Yes, they are the same.
All five techniques should be enabled in the yaml. A caveat is that precomputing the VAE and CLIP latents must be done before training (if you want to use precomputed latents).


/tiller.sh: line 188: 69870 Terminated: 15

I do a helm tiller run -- helm delete --purge [release], it basically hangs for a while. The release is stuck in a DELETING state... I can't get rid of it because of this
What does this mean?
.helm/plugins/helm-tiller/scripts/tiller.sh: line 188: 69870 Terminated: 15          ./bin/tiller --storage=${HELM_TILLER_STORAGE} --listen=<IP_ADDRESS>:${HELM_TILLER_PORT} ${PROBE_LISTEN_FLAG} --history-max=${HELM_TILLER_HISTORY_MAX}  (wd: ~/.helm/plugins/helm-tiller)

does the helm tiller run -- helm ls work for you?
the terminated means that helm delete --purge [release] was finished and tiller was killed

I had almost the same issue.
it was showing me a terminate message from tiller sub shell on bash 4.4 but not on bash 5.
To fix it i had to update this line
https://github.com/rimusz/helm-tiller/blob/master/scripts/tiller.sh#L163
to be like this
( ./bin/tiller --storage=${HELM_TILLER_STORAGE} --listen=<IP_ADDRESS>:${HELM_TILLER_PORT} ${PROBE_LISTEN_FLAG} --history-max=${HELM_TILLER_HISTORY_MAX} & 2>"${HELM_TILLER_LOGS_DIR}")

found here
https://stackoverflow.com/questions/81520/how-to-suppress-terminated-message-after-killing-in-bash
@rimusz do you want me to make a PR for this?

yes, please

https://github.com/rimusz/helm-tiller/pull/58


Broken styles?

I've opened Soundcleod today and there seems to be an issue with broken styling as you can see with the screenshot:

The Home and Collection links are now not clickable either. I'm running Soundcleod 0.18 on OSX 10.10.5.

Looks like a bug on soundcloud.com that affects Safari 8 too:
https://www.browserstack.com/screenshots/74c58c1163e0b89d435c83c452b9d6207db4432c

Confirmed to be due to dropped Safari 8 support on soundcloud.com (hopefully temporary). Unfortunately this is the same engine SoundCleod uses and there is no easy way to upgrade (at least on Yosemite or older).
Watch this space for updates.

And fixed on soundcloud.com.


Rebased and resolved conflict.

Nice! Rebased and will merge once 🟢 !


[WIP] Add Project Comments

Add Comment model
Add comments table and model Factory
Add Project model morph map
issue #11

Project comment done.


Assignment 3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Issue with the notebook:
Invalid Notebook
The Notebook Does Not Appear to Be Valid JSON
Using nbformat v5.10.4 and nbconvert v7.16.1


tools: Update default NEXT_PUBLIC_API_HOSTNAME for the new api routing

With the new routing of API requests through next.js, we need to update the default NEXT_PUBLIC_API_HOSTNAME value set in the .env file when running setup.

Merging to unblock another user that's having the same issue


Clarification on user credentials used

I believe the SQL Server connector creates a connection that caches the user accounts provided when the account is setup.  If that is correct, this is UNLIKE the other commonly used connectors such as SharePoint and OneDrive which the PowerApps user must authorize the first time they run the app and THUS end up being the user account used for authentication via the (ShPt/OneDrive) connector.  In other words, please note on this page that the SQL Connector will impersonate the user account provided when a Connection is created FOR ALL USERS WHO USE AN APP THAT LEVERAGES A SQL SERVER CONNECTOR.

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

ID: bbcbf308-e9b9-d59d-5b1b-ba224dcc1aa9
Version Independent ID: 5959353c-ab28-1bc6-8c21-3b186c1bf61d
Content: Overview of the SQL Server connection - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/connections/connection-azure-sqldatabase.md
Service: powerapps
GitHub Login: @lancedMicrosoft
Microsoft Alias: lanced

Thanks for reaching out. You can read more about sharing resources & connections at share app resources - connections section.


Fix R package compilation error

This fixes the R package which does not compile (commit reference: 0ddb3ef).

Hi @Laurae2, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


monir: 添加变量type

Codecov Report

Merging #763 (307ddbc) into v2.6.x_develop (48bfa74) will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           v2.6.x_develop     #763   +/-   ##
===============================================
  Coverage           61.67%   61.67%
===============================================
  Files                 445      445
  Lines               28443    28443
===============================================
  Hits                17542    17542
  Misses              10901    10901

Continue to review full report at Codecov.

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


Query GO:0004012, phospholipid-translocating ATPase activity and children

From #17164
@vanaukenk:

I have a question about GO:0004012, phospholipid-translocating ATPase activity.
Is that activity transmembrane transport, or does it refer to translocation of phospholipids from one monolayer to the other, e.g. cytoplasmic leaflet to exoplasmic leaflet?

@pgaudet:

Hi @vanaukenk
Good point. These are described as flippases or scramblases, see https://en.wikipedia.org/wiki/ATP10A "The aminophospholipid translocases transport phosphatidylserine and phosphatidylethanolamine from one side of a bilayer to another."
I will not touch these for now - these will probably need an annotation review.
Thanks, Pascale

@ValWood:

should be placed under
GO:0120013 intermembrane lipid transfer activity
or GO:0140303 intramembrane lipid transporter activity
which should also have a common parent, probably.

corrected above to
GO:0140303 intramembrane lipid transporter activity

but
phospholipid-translocating ATPase activity (GO:0004012)
appear to be defined as GO:0140303 intramembrane lipid transporter activity (inner to outer leaflet)

possibly equivalent?

the intermembrane ones are the
sec14 cytosolic factor family

Also consider the position of
GO:0017128    phospholipid scramblase activity
Catalysis of the movement of phospholipids from one membrane bilayer leaflet to the other, by an ATP-independent mechanism.
(the ATP independent intra membrane term)

Right, thanks !
It seems annotations need to be reviewed - for example P08183 looks like transmembrane transporter (PMID:8898203) - @RLovering

I used the matrix to identify violations
GO:0045332
should not probably overlap with
GO:0022857
or
GO:0055085

http://amigo.geneontology.org/amigo/search/bioentity?q=*:*&fq=isa_partof_closure:"GO:0055085"&fq=isa_partof_closure:"GO:0045332"

unfortunately, you cant switch this to an annotation view to see the evidence
but these are the sources:

both of the PomBase ones will be fixed by
https://github.com/geneontology/go-annotation/issues/2329

GAh this should be on teh annotation tracker...will migrate

Not sure what the action is here, but assigning.

I can do it. I think the action will be either a merge or a better definition of the terms.

@marcfeuermann and I looked at all these terms and their annotations. I am changing the labels to make the direction of the lipid transfer as follows:

'ATPase-coupled ceramide transporter activity' -> ceramide floppase activity
'ATPase-coupled phosphatidylcholine transporter activity' -> phosphatidylcholine floppase activity
'ATPase-coupled phosphatidylserine transporter activity'  -> phosphatidylserine floppase activity

Since all papers show that they are floppases (moving the lipid from the cytoplasic side leaflet to the exocytoplasmic leaflet of the membrane).
Thanks, Pascale

and fixed the definitions:
"Catalysis of the movement of [x] from the cytosolic to the exoplasmic leaftlet of a membrane, using energy from the hydrolysis of ATP."

Also:
GO:0015437 ATPase-coupled intramembrane lipopolysaccharide transporter activity -> lipopolysaccharide floppase activity
GO:0034202 ATPase-coupled intramembrane glycolipid transporter activity -> glycolipid floppase activity

For
GO:0004012ATPase-coupled phospholipid transporter activity and
GO:0090555 ATPase-coupled phosphatidylethanolamine transporter activity
it looks like there are both flippases and floppases. I will create both terms and ask for annotation reviews.

GO:0090555 ATPase-coupled phosphatidylethanolamine transporter activity
-> change label to 'phosphatidylethanolamine flippase activity' and ask for annotation review, as there may be one floppase:
https://github.com/geneontology/go-annotation/issues/2341

Updated comment to
'Nomenclature note. Scramblases are ATP-independent, non-selective, inducing non-specific transbilayer movements across the membrane. Flippases and floppases are ATP-dependent transbilayer lipid translocators. According to an extensively used, though not universal, nomenclature, they catalyze lipid transfer towards the inward monolayer (flippases) or towards the outward monolayer (floppases). Some authors however use the name floppase to denote any enzyme catalyzing transbilayer lipid motion, in some cases even in the absence of ATP requirements (PMID 20043909).'

The P4 ATPases are usually considered as "flippases". In the litterature, the term "flippase" is used for most lipid translocating enzymes, whether they have flippase or floppase activity.
Many ABC transporters have also been shown to transport lipids from one side to the other side of a membrane, mostly from inside (cytoplasmic) to outside (exocytoplasmic), in addition to perform transmembrane transport of various other compounds. The mechanisms involved in the flip/flop of lipids by ABC transporters is still under debate, but it is usually accepted as flippase activity (in fact mostly floppase, when considering the direction).

[ ] Change label sphingolipid-translocating ATPase activity (1 annotation, under review) https://docs.google.com/spreadsheets/d/1Y3v52aUxi0a8sgd83z8PDHC4RYllgIx4z9sRpmqv2wM/edit#gid=0

This branch is now all revised.


Html css start

I made a header for all the web pages based off of the design for the home page header which is located in the /websitePlans folder. I also added some photos needed for the website like the officers photos.

Changes have been made. Ready to merge


Compare to Azure Service

Compare the performance of the table tools with Azure services: https://learn.microsoft.com/en-ca/azure/ai-services/document-intelligence/overview?view=doc-intel-3.1.0

This project is not continuing for now. Can potentially revisit in the future.


Add function to get list of all ref ID -> labels from Head

Using this as part of https://github.com/prometheus/prometheus/pull/4588 (remote write via WAL)
Signed-off-by: Callum Styan<EMAIL_ADDRESS>@gouthamve @krasi-georgiev
cc: @fabxc

can you summarise the use case to save me reading the comments and PR code?
btw there is already an interface to get all label values and a pr for the label names https://github.com/prometheus/tsdb/pull/369

@krasi-georgiev Yeah sorry I should have done that already. It's essentially the same as the other PR to get all reference ID's. As far as I can tell, Head is the source of truth for which series references are active and what their labels are. If Prometheus is started from an existing WAL, Head will rebuild the structures containing the series references and labels that it knows about. We can just ask Head for that info rather than rereading the WAL again in the new remote code. The other PR, that just adds a function to get all the series references. I want to use this when the WAL is truncated and a checkpoint is created, since in the new remote write code we're caching the series id's and labels as well.
#369 gets me the labels values, but I also need #402 for the truncate case.


Update z2ui5_cl_core_http_get.clas.abap

updated to reflect latest changes (sorry some local changes were los)

hey @oblomov-dev
sorry somehow some local changes were lost and i did not commit the very latest version.
Please find corrected :) -> Component now instantiated
Best,
Michael


Accept an arbitrary path to /migrations in config

Bug Description
PyPI installs the fidesops /migrations directory outside of the package source tree, in a sibling directory of /fidesops. Currently there is no way to specify a path to DB migrations that exists outside of the /fidesops package source tree.
Steps to Reproduce

pip install fidesops
docker run -it --rm fidesops_fidesops bash
Inside the container, observe:$ tree -L 2 /usr/local/lib/python3.9/site-packages/
site-packages/
├─ fidesops/
│  ├─ __init__.py
│  ├─ ...
├─ migrations/
│  ├─ versions/
│  ├─ __init__.py
│  ├─ ...
├─ ...

Expected behavior
Fidesops may accept an arbitrary file path to the /migrations directory via ENV variable configuration.
Additional context
This is required for fidesops-plus to start successfully.

Thinking more about the changes on #613, it might be better to simply move the /migrations directory inside of /fidesops instead. Thoughts @seanpreston @sanders41 ?

I haven't seen migrations inside the package directory before, but this is a unique case. I think the idea of putting it in /fidesops makes sense here. pip install fidesops resulting in migrations being installed as it's own module in the virtual environment makes it unclear why the module is there, or where to find it when wanting to use it.

@sanders41

I haven't seen migrations inside the package directory before

It turns out that it's done this way in fidesctl. That's where I got the idea.

pip install fidesops resulting in migrations being installed as it's own module in the virtual environment makes it unclear why the module is there, or where to find it when wanting to use it.

I completely agree. Closing this issue and #613 in favor of #615.

Trying to fix


Pin twine to 3.8.0 for Windows

As title.
Fixes b/222133958.

https://fusion2.corp.google.com/invocations/c08466d5-7aa2-4811-953d-9169e27640de


How ad-hoc should our dataPropertiesToRepair be?

We've been adding data properties to dataPropertiesToRepair.js on an as needed basis, since few need to be repaired, but there's no general principle to decide which to repair.
However, https://github.com/Agoric/SES/pull/171 repairs TypeError.constructor and TypeError.name  because of https://github.com/nodejs/readable-stream/blob/edd8c2d87c0f43f4f1d78407ac972fc193e45b40/errors-browser.js#L32-L33 and https://github.com/nodejs/readable-stream/blob/edd8c2d87c0f43f4f1d78407ac972fc193e45b40/errors-browser.js#L3 . It seems too ad-hoc to repair one subclass of Error and not the others. Given that we're doing TypeError, should we go ahead and whitelist these two properties on the other FooError.prototypes?

This whitelist is now at https://github.com/Agoric/SES-shim/blob/master/packages/ses/src/enablements.js
The last expansion of the whitelist is at https://github.com/Agoric/SES-shim/commit/f0e11387fb2b84bae89d808ea7b720ca92cb6129#diff-fcd6ac2a5973959dd77e284ca6b8ca48 on May 15, to accommodate cases from "tape". It seems to me that the initial premise is holding --- that expanding the whitelist on demand as we encounter cases is an adequate coping strategy. After a moderate amount of growth, further growth slows to a crawl.
This issue is in the "Reassess" category of the SES board https://github.com/orgs/Agoric/projects/4#card-43564316
I propose that we reassess our current strategy as a successful to continue. Assuming this is uncontroversial, I am closing this bug. If you disagree, please reopen and discuss.


Update RX100 port.c interrupt vector pragma

This updates the RX100 port.c #pragma interrupt to work with
CC-RX V3.00.00 compiler.
Resolves https://github.com/aws/amazon-freertos/issues/1006

Description
NOTE: This now matches the syntax of RX200 and RX600 and RX600v2 port.c files.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This is a duplicate of https://github.com/aws/amazon-freertos/pull/1013

Closing w/o merging as it is a duplicate of previously merged #1013


Create BinarySearch.java

@ar7aditya can you add the labels for hacktoberfest also

i have already added that
On Sat, Oct 1, 2022 at 2:55 PM Harikesh Pallantla @.***>
wrote:

@ar7aditya https://github.com/ar7aditya can you add the labels for
hacktoberfest also
—
Reply to this email directly, view it on GitHub
https://github.com/ar7aditya/DSA/pull/13#issuecomment-1264304868, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATN4OEI62VURAFB3JSV5S2TWA77O5ANCNFSM6AAAAAAQ2L2KFI
.
You are receiving this because you were mentioned.Message ID:
@.***>

i have already added that
…
On Sat, Oct 1, 2022 at 2:55 PM Harikesh Pallantla @.> wrote: @ar7aditya https://github.com/ar7aditya can you add the labels for hacktoberfest also — Reply to this email directly, view it on GitHub <#13 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATN4OEI62VURAFB3JSV5S2TWA77O5ANCNFSM6AAAAAAQ2L2KFI . You are receiving this because you were mentioned.Message ID: @.>

You need to add the labels to each PR, you added those only for the repo and issue.


Introduce new Error Classes that can be raised by planning backend through the PlannerInterface

The goal is to have more detailed Exceptions exposed to the user for debugging in case of failure.
These Exceptions can be raised by compas_fab backend adapter during pre-planning and post-planning phase, or raised by the backend client and relayed by the adapter.

The current plan is to design a few Exceptions related to different steps in the planning process.

Exception
Planning Phase

InvalidStartStateError
Check Start State (e.g. Collision, Joint values in range)

InvalidTargetError
Check Target (e.g. Reachability / Attached Object Collision)

NoTargetConfigurationFoundError
Target Sampling (e.g. cannot sample FrameTarget / PointAxisTarget)

NoTrajectoryFoundError
Path Sampling (e.g. No Collision-free Path)

PostPlanningCheckError
Final Sanity Check (e.g. Joint Jump Violation)


Device name after connection in Home Assistant (CON-454)

I've successfully connected matter lighting app to Home assistant. As I understand the product name is already displayed on the screen. When I try to connect my devices (other six devices on the screenshot below) with my custom product name it isn't displayed on the screen. How can I achieve this?

Hi @Gonta01 I never tried esp-matter with Home Assistant, I've asked someone from Home Assistant to take a look into this.

I have changed a Product name and Vendor name by using a mfg tool, but it hasn't been represented in the Home Assistant.

I am not familiar with the mfg tool I am sorry. But the "M5STAMP Lightning App" string definitly originates from the CHIPProjectConfig.h https://github.com/NabuCasa/matter-example-apps/blob/main/lighting-app-example/esp32-c3-m5stamp/0001-Add-Project-configuration-for-M5-Stamp.patch.

@agners mfg_tool.py is a python based tool to generate the factory partition which would contain discriminator, verifier, vid, pid, and some more informations that is expected to go into factory partition.
If you are familiar with generate_esp32_chip_factory_bin.py, its same but with some more changes.

Ok I see.

I have changed a Product name and Vendor name by using a mfg tool,

Well, it seems that this tool also allows to change product/vendor name then? I guess the question is what is different to the method I've used (adjusting the upstream Matter CHIP_DEVICE_CONFIG_DEVICE_VENDOR_NAME and CHIP_DEVICE_CONFIG_DEVICE_PRODUCT_NAME configs) and that mfg tool.
@Gonta01 after using the mfgtool, did you remove and readd that device to Home Assistant?

@Gonta01 after using the mfgtool, did you remove and readd that device to Home Assistant?

Yes I did. The result is the same.

@Gonta01 Hi, had this problem too, solved it by manually setting NodeLabel attribute of basic cluster, code snippet follows, hope it would help you
auto endpoint = esp_matter::endpoint::get(node, 0);
auto cluster = esp_matter::cluster::get(endpoint, chip::app::Clusters::BasicInformation::Id);
auto attribute = esp_matter::attribute::get(cluster, chip::app::Clusters::BasicInformation::Attributes::NodeLabel::Id);
std::string data = "Test Bench";
auto val = esp_matter_char_str(data.data(), data.length());
esp_matter::attribute::set_val(attribute, &val);

@rfortunatov thanks for your reply, your answer helped. I close this issue.


Fixed RegExp for a few countries

Turkish Tax Identification Number - reference
Mexican RFC number - reference
Chilean TIN - reference
Argentinian Tax ID - reference

Hello!
Thanks for the PR.
Was testing it and it seems the Mexican one does not work so I kept the @dwelle's one. For turkey, I see there are 10 digits and TIN also has 11 digits.
The Chile changed regex is too forgiving: \b[0-9]{2}\.[0-9]{3}\.[0-9]{3}-[0-9kK]\b this one should be it per: https://www.oecd.org/content/dam/oecd/en/topics/policy-issue-focus/aeoi/chile-tin.pdf
Also would be great to update examples so we can test it against it and let users know how it looks like

Thanks for your feedback.
I had a few thousand records to process with this library and the changes I submitted reflect what I had to do in order for it to work. I did a quick investigation on the ones that broke in my dataset and noticed that the regexps where not matching what was described in the links I posted.
This way I was able to process and manually fix a few records to make them match the updated regexps. Ultimately, the whole dataset went through Stripe with no issues, apart from a few obvious outliers, so I'm really happy about the outcome and I thank you very much for providing the library.
That said, I don't presume my data set to be comprehensive enough to represent a confirmation that my fix is 100% correct. I only used the updated regexp to reduce the number of those I would have needed to process by hand, and trusted Stripe to ultimately reject my work if the ids wouldn't comply. That didn't happen but, again, I only had a few thousand records.
Also, this was just a one shot job for me, so I'm offering my fixes as is. Unfortunately I don't have much time to work on this further.
See what you can do with this PR.
And thanks again!
Cheers
Stefano


Migrate codebase to TypeScript

[x]  Add typescript config
[x] Rename js/jsx to ts/tsx
[x] Add types
[x] Resolve type errors

This closes #5

Closes #5


Add support for ZHAOXIN CPU

Add support for CPU of vendor ZHAOXIN

@gchatelet @toor1245 @Mizux
Are there any other modification requirements? Can I successfully submit the patch that supports ZHAOXIN CPUs?

@gchatelet, @Mizux, please review PR

Hi, @gchatelet  I don't think amd64 Windows /msvc (pull_request) error: " Configure: could not find any instance of Visual Studio." is my fault. Is it?

Hi, @gchatelet I don't think amd64 Windows /msvc (pull_request) error: " Configure: could not find any instance of Visual Studio." is my fault. Is it?

No, my bad, we are using windows-latest github runner vm so they just migrate to VS 2022 so we should change our CMake Generator...
https://github.com/google/cpu_features/runs/5228975155?check_suite_focus=true#step:1:7
So you can safely ignore it, I'll create an issue to track and fix it...

@AnvilaWang, added fix https://github.com/google/cpu_features/pull/223, could you rebase?


New package: Jingque.Fanyiniao version 2.2.0

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335141
2024-06-29 00:51:16.708 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Jingque.Fanyiniao.2.2.0\extracted\翻译鸟.exe

Automated error analysis suggests -1978335141 may mean Unable to locate nested installer. Does the RelativeFilePath have the previous version number in it?
(Automated response - build 886.)

Automatic Validation ended with:

Installation failed with exit code -1978335141
2024-06-29 00:51:16.708 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Jingque.Fanyiniao.2.2.0\extracted\翻译鸟.exe

Automated error analysis suggests -1978335141 may mean Unable to locate nested installer. Does the RelativeFilePath have the previous version number in it?
(Automated response - build 886.)

The RelativeFilePath is correct. Is it an encoding issue that the filename in the zip file is not properly interpreted?

Hi @SpecterShell,
Manual Validation also ended with:

Installation failed with exit code -1978335141
2024-06-29 00:51:16.708 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\Jingque.Fanyiniao.2.2.0\extracted\翻译鸟.exe

The name appears to misencode into τ┐╗Φ»æΘ╕ƒ.
Verify:
But when replacing the characters with their misencoding, the package manager is able to find the nested installer file. This isn't the best solution - how should this PR proceed?

This might be a filename encoding issue of the zip archive. I would prefer to close the PR as it may result in errors on devices with different administrative locales.


fix double compiling of node_modules

Description
https://github.com/nozzle/react-static/commit/ad87562fd912acf2479c2843937ceee8cc7e3673 introduced some double compiling.
like here: https://github.com/glimmerjs/glimmer-application-pipeline/issues/87
so excluding node_modules we must!
Changes/Tasks

[x ] Changed code

Motivation and Context
Types of changes

[ ] Refactoring/add tests (refactoring or adding test which isn't a fix or add a feature)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My changes have tests around them

I was having trouble with an exclude here. It was killing just about every external import from node_modules (emotion, react-table, etc) from properly importing (trying to import module.default.default haha).
Have you tested this with any of those?
Also, I removed the exclude mostly based on what CRA had done by simply deciding to compile everything from node_modules. It ended up being the only way they could allow people to use ESM and commonjs modules in the same projects.
Thoughts?

no i didnt tested it with ESM packages. but without excluding it gets even worse here:
_typeof is not a function in the webpack bundle, and nothing works. i dont know which package caused this problem. i guess its happening because of the double transpile of some already babel transpiled code. which makes it even worse i think.
are there ESM only packages out there? if yes we should find some other way like this:

exclude node_modules but not node_modules/react-table and not node_modules/emotion and so on

or we leave the exclude option, but not excluding node_modules by default? so one can exclude if it have to?
or leave it up to the local babel config? so many questions.

For reference, this is where CRA double compiles to avoid this problem: https://github.com/facebook/create-react-app/blob/next/packages/react-scripts/config/webpack.config.prod.js#L313-L353

@tannerlinsley if #775 works we can close this one...


chore(deps): update common-dev-assets digest to 3d1a2f8

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
247369d -> 3d1a2f8

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This issue has been resolved in version 1.1.12 :tada:
The release is available on:

GitHub release
v1.1.12

Your semantic-release bot :package::rocket:


Make a table of how much data is thrown out by various filters

SWIRE data set
count
percentage

all
30853
100%

training
25933
84%

training < 0.9 stellarity
19671
64%

training < 1000 µJy flux
25062
81%

testing
4920
16%

testing < 0.9 stellarity
3684
12%

testing < 1000 µJy flux
4763
15%

I suggest that you do the splitting into training, validation and test AFTER you do the filtering. That way, you don't need to report percentages for training and testing above.

Merge into #105.


[FEATURE] Selectively render utility classes

Detailed description
Let developers toggle which utilities to render.
Context

Request from one of the internal developers:

This will greatly improve DX if they wish to use both hashi utility classes and tailwind utility classes at the same time
Possible implementation

❔ This snippet of text is taken from the Hashi specifications

Every utility initializer comes with the $exclude and $include parameters. This allows you to exclude certain
utility classes, or include the ones you only need.
@use 'node_modules/@devprotocol/hashi';
@use 'node_modules/@devprotocol/hashi/hs-utils';

@include hashi.init {
  @include hs-utils.color-utils(
    $include: ('text', 'border')
  );
  @include hs-utils.layout-utils(
    $exclude: ('d', 'gap', 'flow')
  );
}

However, you can only use one or the other parameter. Using both will result in an error.
Selecting the utilities to include/exclude is done by passing the class prefix of the utilities. Example: text
is mapped to text-[color]-[tone] utilities.
@use 'node_modules/@devprotocol/hashi';
@use 'node_modules/@devprotocol/hashi/hs-utils';

@include hashi.init {
  @include hs-utils.color-utils(
    $include: ('text', 'bg')
  );
}

Additional information
No response
Code of Conduct

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

Implemented on v.1.2.0


[BUG] Go lint is deprecated and frozen need to find another alternative

Describe the bug
The Golint library is deprecated and frozen, see
Need to investigate a good enough alternative, there are many options listed here

Go to Go Code Reviews
Golint section

We can use the golangci-lint instead. Enabling more linters using .golangci.yaml:
linters:   enable:     - bodyclose     - contextcheck     - dupl     - goconst     - gocritic     - godot     - godox     - revive     - misspell     - nestif     - nlreturn     - stylecheck     - tagliatelle     - tenv     - whitespace     - wsl     - gofmt     - govet     - deadcode     - errcheck     - gosimple     - ineffassign     - staticcheck     - structcheck     - typecheck     - unused     - varcheck


TERRAIN DRAWING

Fix the holes in medium level of detail. Whole patches were lost when moving the camera.

Fixed 3acd17354f2c61902bc9889791fcaf87a993c1d6


:hocho: supplyPlotProps

Fixes #436

@VeraZab this is now waiting on your renaming/visibility stuff and then I can 🔪 the rest of the supplyPlotProps machinery

Hmmm now that I look at it, it may be easier to merge my change in first... I could help you rebase :)

yeah, looks like it'd be easy to rebase this one..
well, looks good to me, I'll look a little closer at the code tomorrow, but looks ready for a merge
thanks for the api fixes!

💃


Feedback: Licensing FAQs - what-constitutes-hosting-cockroachdb-as-a-service

Page: https://cockroachlabs.com/docs/v24.2/licensing-faqs.html
What is the reason for your feedback?
[x] Missing the information I need
[ ] Too complicated
[ ] Out of date
[ ] Something is broken
[ ] Other
Additional details
I am after a clarification on https://www.cockroachlabs.com/docs/stable/licensing-faqs#what-constitutes-hosting-cockroachdb-as-a-service.
Specifically - if CRDB is used in a service offering that allows external users to modify table schemas in some other database product - not in CRDB - but CRDB is used internally as part of the system as a whole - where the CRDB table schemas themselves are only modified by internal employees and contractors - does that constitute a licensing breach?
Jira issue: DOC-11102

If the interpretation is really supposed to be liberal - it should be something like:

Hosting CockroachDB as a service means creating an offering that allows third parties (other than your employees and contractors) to operate a database in CockroachDB. Specifically, third parties cannot modify table schemas in CockroachDB.

Otherwise - it leaves room for legal teams to be uncomfortable. Very understandably so, imo.


sim_info sometimes is unworkable

Hi,
my test device : iphone 11 / iOS 14.3
when I called sim_info in flutter to get sim info, sometimes I got the null result. Once I retry again,  it is workable to get correct sim info. It is so strange.
here is my source code
=============source code =========================
Future getSimInfo() async {
print("starting getSimInfo");
String allowsVOIP = await SimInfo.getAllowsVOIP;
String carrierName = await SimInfo.getCarrierName;
String isoCountryCode = await SimInfo.getIsoCountryCode;
String mobileCountryCode = await SimInfo.getMobileCountryCode;
String mobileNetworkCode = await SimInfo.getMobileNetworkCode;
print("mobileCountryCode:$mobileCountryCode, mobileNetworkCode:$mobileNetworkCode");
setState(() {
  _allowsVOIP = allowsVOIP;
  _carrierName = carrierName;
  _isoCountryCode = isoCountryCode;
  _mobileCountryCode = mobileCountryCode;
  _mobileNetworkCode = mobileNetworkCode;

});

}

I got the error description as the below.
How can I solve the issue?
2021-01-14 18:05:23.674190+0800 Runner[4201:1355438] [Client] Updating selectors after delegate removal failed with: Error Domain=NSCocoaErrorDomain Code=4099 "The connection to service on pid 91 named com.apple.commcenter.coretelephony.xpc was invalidated from this process." UserInfo={NSDebugDescription=The connection to service on pid 91 named com.apple.commcenter.coretelephony.xpc was invalidated from this process.}

I got the error description as the below.
How can I solve the issue?
2021-01-14 18:05:23.674190+0800 Runner[4201:1355438] [Client] Updating selectors after delegate removal failed with: Error Domain=NSCocoaErrorDomain Code=4099 "The connection to service on pid 91 named com.apple.commcenter.coretelephony.xpc was invalidated from this process." UserInfo={NSDebugDescription=The connection to service on pid 91 named com.apple.commcenter.coretelephony.xpc was invalidated from this process.}

I also getting this same problem in ios OS 14.7.1


Feature: Update mousecursor when moving around.

I can also notice that the mouse cursor is not updated - it remains the default arrow even on clickable items where it should change.

Idea: Update cursor property on


Inspect fails to retrieve IP address when inside a privileged Docker container

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
Description

Steps to reproduce the issue:
export NAME=redis
podman run -d --name=$NAME redis:4-alpine
podman inspect --type=container $NAME | jq -r '.[] | select(.State.Running).NetworkSettings.IPAddress'

Describe the results you received:
ERRO[0000] error getting container "48bf1ed1672dd2e0620a068de32ce1930d9f32cb47532b59daffe00ab1c0c48f" IP: error retrieving network status of container 48bf1ed1672dd2e0620a068de32ce1930d9f32cb47532b59daffe00ab1c0c48f: Unexpected command output nsenter: failed to execute ip: No such file or directory
 with error: exit status 1

Describe the results you expected:
The IP address of the container.
Additional information you deem important (e.g. issue happens only occasionally):
Strangely, this works fine, so I'm not sure what the message about failing to execute ip is:
/usr/bin/podman exec $NAME ip addr show eth0 | /bin/grep "inet " | /usr/bin/awk '{print $2}' | /usr/bin/cut -f1 -d'/'

I have only been able to see this issue when it I run podman inside a Docker container. On a "real" machine, inspect works fine.
Output of podman version:
Version:       0.4.4-dev
Go Version:    go1.10
OS/Arch:       linux/amd64

Output of podman info:
host:
  MemFree: 163012608
  MemTotal:<PHONE_NUMBER>
  SwapFree: 923910144
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 2
  hostname: util-kitchen
  kernel: 4.9.87-linuxkit-aufs
  os: linux
  uptime: 1h 31m 9.77s (Approximately 0.04 days)
insecure registries:
  registries: null
registries:
  registries:
  - docker.io
store:
  ContainerStore:
    number: 4
  GraphDriverName: vfs
  GraphOptions:
  - vfs.override_kernel_check=true
  GraphRoot: /var/lib/containers/storage
  GraphStatus: {}
  ImageStore:
    number: 5
  RunRoot: /var/run/containers/storage

Additional environment details (AWS, VirtualBox, physical, etc.):

At present, we get the IP Address we show in inspect by running ip on the host inside the container's network namespace (via nsenter). I suspect the container with Podman in it doesn't have IP in it.
As an aside, we should no longer be doing this - we now maintain the IP addresses of the container in the database, so we don't need to execute a separate program to get them anymore.

The weird thing is that ip is available in the container. This works:
/usr/bin/podman exec $NAME ip addr show eth0 | /bin/grep "inet " | /usr/bin/awk '{print $2}' | /usr/bin/cut -f1 -d'/'

That being said, fetching from the DB sounds better.

@ipmb To clarify - we presently run ip from the host machine, IE whatever podman is running on top of, inside the container's network namespace. So podman inspect has a dependency on whatever package provides ip on the host. I think that might not be reflected in some of our packages, but we are removing it with that patch.


fixes module loading issues

See issue #116 "package.json missing dependencies":
package.json is missing the dependencies declaration for leaflet. This breaks loading leaflet.label in a non-browser tools (i.e., npm install).
 {
    "name": "leaflet.label",
    "version": "0.2.1",
    "description": "Labels for leaflet maps",
+   "dependencies": {
+       "leaflet": "~0.7.0"
+   },
    "devDependencies": {
        "jshint": "~2.1.4",
        "uglify-js": "~2.3.6"
    },
    "main": "dist/leaflet.label.js",
       ...
}

See issue #115 "AMD & CommonJS (Node) Support"
build.js (used by Jakefile.js) loads leaflet (L) by referencing window.L.
Rather than relying on browser globals (which breaks for non-browser environments), can we implement AMD / CommonJS, and use browser global as a fallback?
See https://github.com/umdjs/umd/blob/master/jqueryPluginCommonjs.js
(function (factory) {
    if (typeof define === 'function' && define.amd) {
        // AMD. Register as an anonymous module.
        define(['leaflet'], factory);
    } else if (typeof exports === 'object') {
        // Node/CommonJS
        module.exports = factory(require('leaflet'));
    } else {
        // Browser globals
        factory(L);
    }
}(function (L) {

    // leaflet plugin code goes here
    L.labelVersion = '0.2.1';

    // ... etc ...

}));

Pinging @jacobtoye and @danzel.
This PR is related to https://github.com/Leaflet/Leaflet.label/pull/34, where the package.json file was originally introduced.
I hope you guys will look into merging this. With these changes, npm install will work properly, and will also enable other npm-based package managers (i.e., jspm, which is the ES6 future!)

+1 please, merge this

a merge would be nice :)

Sorry, this PR is super old at this point, so I'm just closing it.


Title positions differently when deployed (vs in localhost)

Hello!
As the title says, I want my blog to look the same as it shows on the localhost. The first image is what my blog looks like in the localhost and the second image is what it looks like in my Github blog page. I can't seem to figure out what the issue is, so I would appreciate if someone can help me with this one.
localhost:8000
github.io/blog
Thanks!

I assume you're building your GitHub site with --prefix-paths and set pathPrefix: "/blog" in gatsby-config.js?
In that case it's the following line that causes the problem:
https://github.com/gatsbyjs/gatsby-starter-blog/blob/master/src/layouts/index.js#L11
A quick hack is to change that line to if (location.pathname === '/blog/' || location.pathname === '/') {
However, ideally the layout file should read the pathPrefix from gatsby-config.js  and use that value in the if statement if the environment is production. That way you can change the prefix in one place and the code will continue to work.
@KyleAMathews would you mind pointing me in the right direction for a good implementation. Should I add a GraphQL query to reads pathPrefix to src/layouts/index.js? Or how would you recommend we solve the problem?
If you can nudge me in the direction I'll prepare a PR.

@HeinrichFilter thanks for tackling this! Doing something like we do in gatsby-link probably makes the most sense https://github.com/gatsbyjs/gatsby/blob/master/packages/gatsby-link/src/index.js#L7

@KyleAMathews I know this is closed but looking into this I noticed some oddity as I installed it tonight and followed the proper methods by adding   pathPrefix: '/blog',  it seems with some testing after deploying to my repo titled blog and using gh-pages that the location.pathname returns as / even with pathPrefix, when you goto a blog post it is /hello-world.   The site does handle the assets properly but I think the logic in /layouts/index.js may not be entirely correct.

@jmenglis happy to take a PR with the fix.


chore: add cargo deny ignore for gtk3-rs

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 15.66%. Comparing base (23044f6) to head (23461b1).

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #230       +/-   ##
===========================================
+ Coverage   12.28%   15.66%    +3.37%
===========================================
  Files        2246      187     -2059
  Lines      191068    16755   -174313
  Branches   174985      670   -174315
===========================================
- Hits        23469     2624    -20845
+ Misses     166401    14131   -152270
+ Partials     1198        0     -1198

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Better ways to notify the user about the feeds that failed to load

I the current version of newsboat there is no way to know if one of the feed failed to download. The only way to discover that there is something wrong is by noticing the absence of the new posts, which is not very obvious for feeds that contains less frequent posts.
It would be nice if newsboat could:

show the error message when one or more feeds failed to download new posts
create entry in some error log, so user can check which feeds failed.

Can you please implement such features in the one of the next releases?
Kind regards,
Piotr

I've also wanted a better view & visibility for download errors, and the issue #2765 has quite similar goal (it would play well with option 1.).
There are three ways I know of, that can show this information currently:

The feedlist has format option S for download status. I put this at the start of the line, so I see which feeds failed to load. This does not show the reason.
Logging gives a message with a reason. The error-log option is good for this, but other logging options also work (but have more unrelated messages).
The logged error message is also briefly shown in the UI, but there's only room for one message and it gets quickly overwritten (e.g. by the reload progress messages). So this is not practical.

@juntuu: Looks like option 1 is the one that can do for now. :)
Thanks for pointing out this feature.

Maybe additional error log window and on the status line a message like:
Some errors occured. Press <define_key> to see the log

I had similar idea about error log window inside the application.
I implemented a demo in https://github.com/juntuu/newsboat/tree/message-view

UserError level log messages are kept in memory, and shown in the view as a simple list
The view is adapted from help dialog (I don't really know stfl or the Newsboat's UI logic)
The view is bound to m and should work from most other views

This does not have any indication about new messages in status bar or similar. It could also be good to allow clearing the messages (other than restarting the program).

Another idea I had is adding the number of failed reloads to the notification format. I don't use the notification system myself, but this would make sense to me. Demo implementation here https://github.com/juntuu/newsboat/tree/notify-on-failed-reloads

Failed reloads are counted
The count is exposed as E(rrors) format identifier
E is empty if there are no errors, so that this works with conditional format
Notification is shown if any errors happened

These are pretty rough drafts, but hope they bring ideas for the discussion.

Together with this change it would also to be nice to cover following scenario:
My default view on the feeds is to see only these with unread articles (with config option show-read-feeds no).
In this case, if one of the feed update will fail, I will not know that unless I press l key to see all the feeds.
It would be nice to have some configuration option which would allow failed feeds to be displayed on show-read-feeds no view. It could be like always-show-failed-feeds <yes|no>.
I use the S variable in my feeds view, so I can distinguish failed feeds, but for now I cannot see them unless pressing l key from time to time to check for errors.


⚠ Bofrost.fr has degraded performance

In cda8074, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 993 ms

Resolved: Bofrost.fr performance has improved in 36aa7d9.


Can't deploy Rails apps using sprockets 3

Sprockets 3 released recently changed the manifest path:
Renaming /u/.../releases/.../public/assets/manifest-ab5dee974c76c3e103b9ea6ce1d15fc0.json to /u/.../releases/.../public/assets/.sprockets-manifest-0998110c17d5a019c2dd8b92b490aa8d.json

So then the backup_manifest task fails:
cp: cannot stat `/u/.../releases/.../public/assets/manifest*': No such file or directory

I'll try to submit a PR to fix this latter this week.

Awesome, I was just looking for an answer for this the last few days. I was going crazy thinking no one else was having a problem, but I wasn't sure how to fix it :+1:

:+1:

Thanks for logging the issue, it helps reducing grey hairs :)

I think this has been fixed in capistrano-rails 1.1.3.

Yep, new capistrano-rails fixed it for me.

So, I'll just close the issue then… mmkay?

Just pushed this. Thanks. :+1:


Version 1.11.x has exception not caught in CharacterReader.bufferUp

Exception is not caught and not possible to catch in CharacterReader.bufferUp
Tested with version 1.11.2,  with both
Doc doc = Jsoup(....)
or
Response reponse = Jsoup(...)
Document doc = response.parse();

The method tested with both throws IOException, UncheckedIOException, SocketException and also try ... catch (Exception e)
None of them can catch the exception. But using 1.10.3 will never find the problem.
This is the error output:
Exception in thread "Thread-8" org.jsoup.UncheckedIOException: java.net.SocketException: Connection reset

at org.jsoup.parser.CharacterReader.bufferUp(CharacterReader.java:60)
at org.jsoup.parser.CharacterReader.Current(CharacterReader.java:85)
at org.jsoup.parser.TokeniserState.readData(TokeniserState.java:1669
at org.jsoup.parser.TokeniserState.access$200(TokeniserState.java:8)
at org.jsoup.parser.TokeniserState$6.read(TokeniserState.java:76)
at org.jsoup.parser.Tokeniser.read(Tokeniser.java 45)
at org.jsoup.parser.TreeBuilder.runParser(TreeBuilder.java:51)
at org.jsoup.parser.TreeBuilder.parse(TreeBuilder.java:45)
at org.jsoup.parser.Parser.parseInput(Parser.java:39)
at org.jsoup.helper.DataUtil.parseInputStream(DataUtil.java:151)
at org.jsoup.helper.HttpConnection$Response.parse(HttpConnection.java:832)
at MyApp.mediasWeb(Instagrammjava:280)
at MyApp.Mediascanner.run(Mediascanner.java:31)

@jhy is this the same stuff as #980 ? You may want to close also this one.

Thanks, yes, this is the same issue/fix as #980


Added the tests, code formatting changes, and the Range Request feature.

Had to reconfigure some of the tests but I'll merge this feature even if the tests don't pass.


Use Macro.camelize/1

Fixes a bug with package names that contain underscores like package test_service.v1.FooBar. This will generate a module named TestService.V1.FooBar in Elixir.
Currently if a package with underscore is used, it results in the following list ["Test_service", "V1", "FooBar"] and fails Module.safe_concat/1 because this atom does not exist. Test_service should be TestService.

I would have preferred an issue for better conversation, but 🤷
You have removed the test for upcase_first/1 and the only place that it is called, but you didn't remove the now unused and untested def upcase_first(...). Please also remove the function definition when it is unused and untested.
Otherwise this looks good

Sorry about that. Seemed like a simple issue so I opened a PR. Will remember to start a discussion in the future.
Removed upcase_first.

Thanks for the work @christopherlai !

Thanks for the work @christopherlai !

Happy to help, but it was purely self-serving. Needed this fix for myself.
What's the schedule for cutting a new release hex?

I'll see about a patch release with this and another bugfix (already merged) probably tomorrow


onTap should not be triggered when tap occurs to stop momentum scrolling

When an user is scrolling on a mobile there's "momentum scrolling" effect that makes the content continue to scroll even after the touchend event, according to the movement velocity (also called "fling scroll")
Often, the user may want to stop that momentum scrolling by tapping the screen. He does not care what he taps on, he just want to stop scrolling. But unfortunatly when the scrollable content has a lot of onTap listeners, the probability to trigger one of them is high and the user ends up opening unintentionally new popups / menus / content...
Other references on this issue here:
http://stackoverflow.com/questions/27040241/detect-if-a-touchstart-touchend-has-cancelled-a-scroll-momentum-scroll
http://stackoverflow.com/questions/10573080/mobile-safari-vertical-scrolling-how-to-detect-when-the-window-has-stopped-mov
It seems a workaroud has been implemented for FastClick#42
and the commit is here:
https://github.com/ftlabs/fastclick/commit/2a31072ca469def511f7ddbedbcaf305d37ab934
I may have time to work on this as we need it at at work some point, just tell me if you agree with the idea

PRs welcome...


More accurate way to calculate drivers' urls with IPv6 support

Fix drivers.GetUrl() to build proper IPv6 addr.
Use net.JoinHostPort() as it has support for IPv6 literals, which should be enclosed in [].
fix #2219 but depends on docker/docker#15557 which adds support for valid IPv6 addresses in $DOCKER_HOST. Unfortunately it hasn't been landed to any docker's release yet.

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 "fix-dirvers-geturl" git@github.com:narqo/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 @narqo for taking the time to check each & every driver.
I wish we vould do some simple unit testing on those cases, but unfortunately the way things are put together makes this quite hard.
Sounds like docker/docker#15557 is merged, so here's my
LGTM

LGTM


update multi json to reflect the dependencies of cucumber

This now matches the multi_json dependencies of the edge version of cucumber

Hey @charlierudolph, would you take a look at this?

@charlierudolph would be nice if you can :)

Sorry haven't touched this repo in a long time. What exactly is wrong with the current version? It appears it doesn't allow above 2.0 and is only allowing some older version of multi_json. Is that correct?


Please add support for Markdown document outlines or [TOC]

Please add support for Markdown document outlines or [TOC]

Thanks for the suggestion. This is a dupe of #34 so I'll close this in favour of that. Please go give that issue a thumbs up to log your interest.


linux/hardened/patches/6.5: init at 6.5.3-hardened1

Description of changes
Previous: https://github.com/NixOS/nixpkgs/pull/242946
6.5 through 6.5.3 released a couple hours ago: https://github.com/anthraxx/linux-hardened/releases/tag/6.5-hardened1 through https://github.com/anthraxx/linux-hardened/releases/tag/6.5.3-hardened1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Currently building

Booted into the system, works. :3


cleanup(userspace/libsinsp): remove legacy references

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
/area build
/area libsinsp
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This cleans up legacy references that are not used anymore in the codebase.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: jasondellaluce
Once this PR has been reviewed and has the lgtm label, please assign andreagit97 for approval by writing /assign @andreagit97 in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 420c9fac0a984f1e05e9af9a98cebaabf32347cf

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Ensure default branches on github are set correctly

Currently we need to manually go through and set the proper default branches... e.g. when we created the 1.0 or 4.0 branches, the repos were still on 1 or 4 respectively.

I think next minor (1 / 4) makes sense as the default branch, and this has been the standard through the CMS 4.x lifecycle, so we can close this one out for now.


How to call MallocExtension::instance()->ReleaseFreeMemory() from gdb?

Hi,
My C (not C++) code is linked with -ltcmalloc with gcc(not g++) and it succeed.
I want to call MallocExtension::instance()->ReleaseFreeMemory() from gdb, anyone know how to do it?
THanks!!

Consider calling C version. See malloc_extension_c.h


Bring feat-2.0 up to date with dev since f3b762d

This PR brings all recent commits since f3b762d from the dev branch to the feat-2.0 branch.
Merge had no conflicts.

bors r+


MVP 681 police force

Hi guys,
So I think this might be a better solution. Its the police force concept 2 link in the concepts section...
So what I've done is directly after we ask when it was reported to the police, we ask where its being investigated (country) and then the force depending on what they choose. The journey then resumes as normal till location where I've removed the country questions and it only asks location then resumes... Hope that makes sense. You should be able to get it from the link.
This change would just be an update to the route and some content tweaks so much easier for the devs to implement and means we are not adding/ or removing pages. Just rearranging a little.
I toyed with the idea of asking country on the location page?
@johnkane1 for the title of the country page - this was the best I could come up and think makes sense? I wondered about saying 'investigating' as though it's currently ongoing when it could have been investigated years ago? Just a thought.
Let me know what you guys think!!

@maxinedivers I can't view concept 2. It crashes after When did the incident happen

@maxinedivers, ignore above, John has access and has shown me it.

I think it would be worth discussing it tomorrow as a group as we're getting tied up in knots because we're trying to avoid using a drop down at all cost. Some ineligible applicants may have reported it to an English force even if it happened 'somewhere else'. I know its small numbers but it does happen so we need to ask location country to route out ineligible cases.


[Bug]

🐛 Bug

To reproduce
Code snippet to reproduce
// Your code goes here
// Please make sure it does not require any external dependencies

Stack trace/error message
// Paste the output here

Expected Behavior

System information
Please complete the following information:

Additional context
Add any other context about the problem here.
Can't send assets, please help!?

cannot send assets to other wallets


Support workload granularity metrics dynamicly turn on with kmeshctl

What type of PR is this?
/kind enhancement

What this PR does / why we need it:
add workload metrics trigger in kmeshctl.
Enable workload metrics on demand.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

/lgtm


[New Port Request] log4cpp

Library name: log4cpp
Library description:
Log4cpp is library of C++ classes for flexible logging to files, syslog, IDSA and other destinations. It is modeled after the Log4j Java library, staying as close to their API as is reasonable
Source repository URL: https://sourceforge.net/p/log4cpp/codegit/ci/master/tree/
Project homepage (if different from the source repository): http://log4cpp.sourceforge.net/

log4cpp.zip
tested windows x64 dynamic

Hi @iceseyes  Thanks for posting this request. This port has been added to vcpkg. Please try to update vcpkg and use this port.


feat: create migration for support_requests table

Add a database migration file to create the 'support_requests' table.

This commit adds a database migration file to create the 'support_requests' table. The table includes columns for storing support request details such as user_email, first_name, last_name, email, support_message_title, support_message_text, file (blob), created_at, and updated_at.


"This element does not support ARIA roles, states, and properties" for noscript

The noscript tag gives this warning (AX_ARIA_12) if it has aria-hidden. This doesn't make sense, as it's (sometimes) a rendered node. (We are including it so that chromevox doesn't read the contents of the tag, because it does by default, even with JS enabled).

@calebegg it's based on this: http://www.w3.org/TR/html-aria/#document-conformance-requirements-for-use-of-aria-attributes-in-html
There you will see that for noscript, in the column "ARIA roles, states and properties which may be used"

"No role or aria* attributes"

@calebegg thats a bug in chromevox and needs to be reported, I suggest filing a bug http://crbug.com|crbug.com - tag with UI-Accessibility


Fatal error: 0x00000001: Failed to create vertex shader.

Some old Gpu device will Failed to create vertex shader.
Adapter 1: NVIDIA GeForce 310M
D3D11 loaded successfully, feature level used: 41216
$input a_position, a_texcoord0
$output v_texcoord0

/*
 * Copyright 2011-2019 Branimir Karadzic. All rights reserved.
 * License: https://github.com/bkaradzic/bgfx#license-bsd-2-clause
 */

#include "../common/common.sh"

void main()
{
	gl_Position = vec4(a_position, 1.0);
	v_texcoord0 = a_texcoord0;
}

Fatal error: 0x00000001: Failed to create vertex shader.

Try creating shader with SM4.


Class Property Directive for Documenting Property Name & Type Only

Feature Request
Actual Behavior
TypeDoc generates documentation for class properties as they are defined in code
Expected Behavior
TypeDoc generates documentation for class properties using options to only include property name and type in generated docs
Currently, TypeDoc does not seem to have a way to create documentation for class properties in a clean manner. You can see what I'm talking about here TypeDoc VariableConverter Class Documentation. If you scroll down to look at the "supports" property of the "VariableConverter" class you can see that not only does it include the name of the property and its type, but it also includes its set value (the = along with the value that comes after the =). This makes for unsightly documentation, and is currently an undesirable effect of generating documentation for a project we are working on.

As of 0.20.1, properties will include their initializer only if deemed to be "simple" (a numeric or string literal)


[Enhancement]: Using a different marker for live location in the map widget

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Appsmith devs who collect location from the app viewers can use it to show a live location indication for the app-viewer in the map widget.
Similar to how google maps support blue location indicator for the current user.
Why should this be worked on?
We can then differentiate the user's location from the locations being fetched from databases, it will help app-viewers navigate better.

Reach: 42 in 6 months
Impact: Medium


Add fingerprint protection

please add fingerprint / password protect for this App

Since https://github.com/google/google-authenticator/issues/236 already asks for PIN, I'm changing this feature request to be about fingerprint.

@ThomasHabets Hope you guys implement this security feature

FYI: The version in Google Play Store / Apple App store is not the same as this opensource version. They've diverged. This opensource version is also unlikely to end up in the app stores. This open source version doesn't get much love, but I'll accept well-written pull requests. But don't expect this feature to be implemented by Google.

On my Huawei P10 Lite phone, which have Android 7.x, one can put an app lock on an app, meaning that the security part of Android will require a successful authentication using a PIN code or a finger print, before that app will be started.